macOS support for examples

I don't know if you still plan any support for this project, but the examples don't really work on macOS. The cube examples renders a cube in the very lower left corner and quite tiny. The shaders example has some broken GUI elements and nothing else, etc.

I wouldn't mind supporting MacOS if I had the hardware to do so, which I don't. Once upon a time I did and supported the platform on my C graphics engine. When I did that, I found that there were just oddities in the driver and shader compatibility that I couldn't predict ahead of time and just had to muscle through it.
So...
A) I personally can't support it since I don't have a mac and this won't change in the foreseeable future.
B) I'm open to pull requests claiming to fix MacOS bugs
C) I'm willing to try and give pointers for support

Unfortunately i also do not have a mac, BUT my game 2048 builded on this engine works well on a  mac.


Transparent Screen in Win11 + .NET 7

Describe the bug
In a new project using .NET 7, the entire screen appears transparent. Although nothing is visible, you can still interact with the buttons defined in the window.

To Reproduce
Create a new application WPF application and install latest WPF UI 3.0.0-preview.4
Expected behavior
It should show the default Home,Data and Setting pages
Screenshots

OS version
OS Name:	Microsoft Windows 11 Pro
Version:	10.0.21996 Build 21996
System Model:	HP EliteBook 840 G7 Notebook PC
.NET version
.NET 7
WPF-UI NuGet version
3.0.0-preview.4
Additional context
I have created other applications using WPF UI; however, those applications also don't work, that works in win10

Upgraded the application to 3.0.0-preview.11 and still the same issue

I can see the application when I mouse over it in the task bar in release x64 mode.

This is odd. I created a demo app using the versions mentioned, but could not replicate. Are you sure you're not setting the window opacity to near zero levels?
Can you share a snippet of the <ui:FluentWindow> parameters?

I am not setting the opacity to zero. I have an application which I created a month ago and is production ready. That doesn't work. When I drag the window and make it full screen it shows. I have attached a video for you.
Main Window Code:
<ui:FluentWindow     x:Class="SplaatPhotoOrganizer.Views.Windows.MainWindow"     xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"     xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"     xmlns:d="http://schemas.microsoft.com/expression/blend/2008"     xmlns:local="clr-namespace:SplaatPhotoOrganizer.Views.Windows"     xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"     xmlns:ui="http://schemas.lepo.co/wpfui/2022/xaml"     Title="{Binding ViewModel.ApplicationTitle, Mode=OneWay}"     Width="1100"     Height="650"     d:DataContext="{d:DesignInstance local:MainWindow,                                      IsDesignTimeCreatable=True}"     d:DesignHeight="450"     d:DesignWidth="800"     ui:Design.Background="{DynamicResource ApplicationBackgroundBrush}"     ui:Design.Foreground="{DynamicResource TextFillColorPrimaryBrush}"     ExtendsContentIntoTitleBar="True"     Foreground="{DynamicResource TextFillColorPrimaryBrush}"     WindowBackdropType="Mica"     WindowCornerPreference="Round"     WindowStartupLocation="CenterScreen"     mc:Ignorable="d">     ... </ui:FluentWindow>
https://github.com/lepoco/wpfui/assets/76004649/7f9a4d49-f662-4692-8804-b76828f6c6ac

Thank you for your insights! I appreciate your suggestions. I'll investigate the window sizing issue.
I'll check if window width or height is being handled in the code and make the necessary adjustments. I'll also try using debug mode for the builds and the parameters you suggested:
Width="Auto" Height="Auto" SizeToContent="WidthAndHeight"
If the issue persists, I'll definitely look into SnoopWPF as you recommended. It sounds like a valuable tool for debugging WPF applications.
I'll keep you posted on my progress and let you know if I have any questions.
Thanks again for your help!


leafref changes for path and must statement to point the same LTP instance.

Problem :
For the attribute tdm-container-kind in tdm-container-configuration , pyang tool reports warning with regards to the must statements defined ,

tdm-container-2-0.yang:474: warning: node "tdm-container-2-0::tdm-container-capability" is config false and is not part of the accessible tree

the current "must" statement for the attribute is not pointing to the capabilities of the same LTP instance.
Possible solution :
leaf tdm-container-kind {
                type leafref {
                    path "/core-model:control-construct/core-model:logical-termination-point[core-model:uuid = current()/../../../../core-model:uuid]/core-model:layer-protocol[core-model:local-id = current()/../../../core-model:local-id]/tdm-container:tdm-container-pac/tdm-container:tdm-container-capability/tdm-container:supported-tdm-container-kind-list/tdm-container:tdm-container-name";
                    require-instance false;
                }
                must  'deref(.) = current()';
                description
                    "Type of TDM container.";
            }

Solved with TdmContainer_2.0.0-tsp.220324.1125+yang.1.


how to make -start -end modifiers ignore the reference elements width

I've been running back and forth on the documentation for days already and I cant seem to find what needs to be done
both poppers have exact modifers with placement="bottom-end"
Problem: the above popper is shifting to the start instead of end because the reference element was too small
What im looking for: I want the above popper to behave like the popper below(shifts on the right side even if the reference element is small)

-start and -end modifiers align the left or right border to the left or right border of the reference element, the behavior you are experiencing is correct.
I'm not sure how to achieve what you are looking for, it seems something non linear.

@FezVrasta so what is the recommended way to do something like bottom-left, botttom-right?

also do we have an official reddit/slack channel where we can ask random questions?

There's the Spectrum.chat channel.
I'm not sure how to achieve what you are looking for I'm sorry, a custom modifier will surely be able to do that but I don't know what to put inside it.


Twint command line documentation update

[Yes] Python version is 3.6;
[Yes] Updated Twint with pip3 install --user --upgrade -e git+https://github.com/twintproject/twint.git@origin/master#egg=twint;
[Yes] I have searched the issues and there are no duplicates of this issue/question/request.
Command Ran

twint -u POTUS

Description of Issue

The terminal spits out bash: twint: command not found...
I needed to run
python setup.py install

and then the command would work. This command is not mentioned anywhere in the documentation, I only found it thanks to the recommendation of a user on a video about how to use Twint.
Environment Details

Gnome Terminal 3.28.2 on Fedora 28 virtual machine.

[Yes] Updated Twint with pip3 install --user --upgrade -e git+https://github.com/twintproject/twint.git@origin/master#egg=twint

I guess your installation process did not end well


Use compact format for ISO week & quarter

Minor change in iso format for weeks and quarters

Let me know how to help you with acceptance, just in case...

@jgoizueta can you share the steps to perform the acceptance?. If it has a test to check the behavior, merge it directly.

I'm closing this because we've decided to use the non-compact format, after all


Multiple outputs with Wayland backend

[x] Remove output instead of terminating display when a wayland backend view closed
[x] Fix pointer events

Test plan: env WLR_WL_OUTPUTS=2 build/rootston/rootston

This PR is ready for review.


Improve Information card on Overview pages

Feature Suggestion
The information card on the overview page should show the most relevant metadata about the entity. Today it is just a simple, unweighted, list of attributes:

Prior art (from Spotify's internal dataset information card):

Having a more styled and specialized component would allow to display the tags properly 👍

@Fox32 for sure! That will definitely be part of an updated design.

###Updated About Card
Please check out the specs in our Figma file, which you can access by duplicating from here.

You can click into the card to access the different specs.


how can I get the proper value when using BLE GATT and Hex Values

Hello everyone, this is my first time here. I have a client who needs me to connect to a BLE device and get readings from it using Javascript.
I am successfully connecting to the device. But now it is time to get readings. Here is the information they sent me:
"To provide efficient transmission of data at higher speeds a CondensedDecimalArray type can be used. This will be transmitted as an array of 32-bit signed integer values in little endian format each being a hex string of 4 bytes. The first two bytes of the array are a 16-bit unsigned integer value in little endian format. This is the number of readings in the array. Each reading should be read from the array and processed by the client application."
"To interpret each value use the integer value and insert a decimal point num_decimal_places from the end of the value. The num_decimal_places is stored in a characteristic in the calibration service."
So here is my code block:
const result = await BleClient.read(device.device.deviceId, this.CALIBRATION_SERVICE, this.NUM_DECIMAL_PLACES);
this.decimal_places = result.getInt8(0);
BleClient.startNotifications(device.device.deviceId, this.CURRENT_READING_SERVICE, this.CONDENSED_VALUE_CHR, (value) => {});
I am just struggling on how to accurately get a value from this. I see it is a 32 - bit signed integer value in little endian format. So looking at the docs of the plugin, i can use getInt32(0, true)
So my thoughts are this: use getInt16(0, true) and getInt16(1, true)? That would be the first two bytes of the value array to determine how many readings are in the array? Am i on the right track?
Any help is greatly appreciated.

Do you know what is a DataView and Buffer in Javascript? if not please read about them and make sure you understand them well, I suggest you to start from here.

Do you know what is a DataView and Buffer in Javascript? if not please read about them and make sure you understand them well, I suggest you to start from here.

I'm new to DataView and interacting with Bits in JavaScript. That's why I thought I could reach out for some guidance.

Ok you should read more and test DataView yourself it will give you another superpower in JS :)
read this it might be helpful too
https://javascript.info/arraybuffer-binary-arrays
anyways you should try something like this:
const result = await BleClient.read(device.device.deviceId, this.CALIBRATION_SERVICE, this.NUM_DECIMAL_PLACES); for (let i = 1; i <= result.byteLength/8; i++) {     console.log(result.getInt32(i, true)); }

Ok you should read more and test DataView yourself it will give you another superpower in JS :) read this it might be helpful too https://javascript.info/arraybuffer-binary-arrays
anyways you should try something like this: const result = await BleClient.read(device.device.deviceId, this.CALIBRATION_SERVICE, this.NUM_DECIMAL_PLACES); for (let i = 1; i <= result.byteLength/8; i++) { console.log(result.getInt32(i, true)); }

Okay cool! So I can just loop through the object like that. If the response is a array of 32 bit signed integers. It would be result.byteLength/32? Or are all responses always byteLength/8?

ok, please work more with JS DataView, it has some learning curve but it's important to understand how buffers are represented in a DataView

First of all we are looping on a DataView which has different characteristics of normal arrays or objects, but yes you can use for loop to loop throw its bytes
I used byteLength which is similar to Array.length but for DataView
After reviewing the code I wrote I think I'm mistaken :confounded:  so here is a complete step-by-step code that will make more sense
`
// lets' create a buffer that has "room" of 8 bytes
const buffer = new ArrayBuffer(8);

// let's create a DataView of that buffer
const data = new DataView(buffer)
// let's add to the dataview int32 which mean it will occupy the first 4 bytes
// btw, obviously; in16 is made of 2 bytes and int8 is made of a single byte
data.setInt32(0, 123456789, true);
// now offsets 0,1,2,3 are representing 123456789 in  4 bytes
// we have "room" for another int32 because our buffer is 8 byteLength
// we have to start the offset from 4
data.setInt32(4, 987654321, true);
// what will happen if we try to set another int32 on position 8 ?
// it will throw an error saying that Offset is outside the bounds of the DataView
// now we have two int32 in our DataView
// let's loop, we will start from 0
// in your code maybe you have to ask whoever you integrate the ble service from about the offset of the start
// because sometimes the first bytes are reserved for some ACK or similar signals
//in our case we will start from offset 0 and will set our iterator to i += 4 because we know that each 4 bytes are occupied by int32
for (let i = 0; i < data.byteLength; i += 4) {
console.log(data.getInt32(i, true))
}
`

also please for future discussions open a discussion thread instead of an issue, thanks in advance :)

also please for future discussions open a discussion thread instead of an issue, thanks in advance :)

Sorry, I am more of a lurker. Never really post much. Thank you.
Your example is so much more helpful! I totally understand now and can apply this to my issue. I learn better by example than reading so after reading the DataView Docs and your example, it really makes a lot of sense!
Thank you again! Hopefully this post can help others get an understanding too.


Custom URL for posts?

I'm trying to get a post to have a custom URL/permalink.  Lets say that I have a blog post titled "Uses" and I want it to be located at /uses. I know I could easily create a custom page outside the blog engine, but I want to include comments as well, which means that it makes sense to be a blog post.
I've started hacking on this with custom frontmatter: permalink: /test for right now. I've modified some middleman-blog code to make the article.destination_path equal to the permalink frontmatter.  My sitemap looks like this:
Path: blog/2016-12-10-new-use-page-added.html
Build Path: /test/index.html
URL: /test/
Source File: source/blog/2016-12-10-new-use-page-added.html.markdown
Data: {“title"=>"New Use Page Added", "date"=>"2016-12-10 15:37 UTC", "permalink"=>"/test"}
Options: {:lang=>:en}
Locals: lang

But when I click the link in the http://localhost:4567/test/ I get 404 not found. So it looks like it's not picking up the new route, but looking through the middleman-blog code I can't figure out how it works. I feel like I'm getting close, but it's not 'clicking.' Any help would be appreciated.

Okay, I think I figured it out.  article.destination_path can't start with a forward-slash.
Here is my change to handle the destination_path
if article.data.has_key? 'permalink'
  article.destination_path = ::Middleman::Util.normalize_path article.data['permalink']
else
  article.destination_path = template_path @permalink_template, article
end

Does anyone thing this is a good feature to add? I can do a formal PR with tests if needed.

Hiya
Do you know you can include
---
slug : new-path/to-post
---

In the frontmatter
This will override the blog.permalink  = "{category}/{title}.html" from the config.rb.

@silasb Did my post clear this up - closing - please re-open if its still an issue.


New version: Nutstore.Nutstore version 7.0.8

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[X] Does your manifest conform to the 1.5 schema?

Microsoft Reviewers: Open in CodeFlow

@wingetbot run


source make failed

hi , I install on ubuntu according to the notes ,  when I execute the step make && make install, it shows :
In file included from init.cpp:65:0:
./zmq/zmqnotificationinterface.h:8:10: fatal error: validationinterface.h: No such file or directory
 #include <validationinterface.h>
          ^~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.
Makefile:6783: recipe for target 'libbitcoin_server_a-init.o' failed
make[2]: *** [libbitcoin_server_a-init.o] Error 1
make[2]: Leaving directory '/home/liyaojian/BitcoinDiamond/src'
Makefile:10893: recipe for target 'install-recursive' failed
make[1]: *** [install-recursive] Error 1
make[1]: Leaving directory '/home/liyaojian/BitcoinDiamond/src'
Makefile:774: recipe for target 'install-recursive' failed
make: *** [install-recursive] Error 1

what is the problem?

Did you install libzmq3-dev before, or you can try ./configure --without-zmq before make if you don't need zmq.

thank you for your answer! I have solved .  and do you know where is the source code of bcd explorer, for this explorer.btcd.io?


Option to plain-format some keys like /models/{model}

Is your feature request related to a problem? Please describe.
I'm trying to minimize diffs when using yaml-diff-patch on OpenAPI specs like
openapi: 3.0.0
paths:
  /models/{model}:
    get:
      # etc

The /models/{model} key gets quoted after a roundtrip.
EDIT: I thought this is an implicit key but I think I misinterpreted the spec, so seems like this is valid?
Question: is it actually a valid key?  It seems to parse okay and linting tools seem okay with it.  But, I found in your code that you intentionally quote implicit keys containing { or }, and the YAML spec seems pretty clear on this:

In addition, inside flow collections, or when used as implicit keys, plain scalars must not contain the “[”, “]”, “{”, “}” and “,” characters. These characters would cause ambiguity with flow collection structures.

Describe the solution you'd like
If an implicit key like /models/{model} is actually unambiguous, can we have defaultKeyType: 'PLAIN' not quote it?
Describe alternatives you've considered
If using an unquoted key in this form is bad and not technically valid, I'm wondering if there's a good example why it shouldn't be done I can relay back to folks who are outputting YAML in this format?
Additional context

If the maintainers are open to it, I'm happy to fund a PR.

Sent you an email!


Access apidocs

Hi there,
How would we access the apidocs on local machine?.
I tried apidocs.staffjoy-v2.local/ , it does'nt work.
I also tried looking for /swagger.json url, cant seem to find it.
Thanks,
Ratnakar

You access the API docs at:
account.staffjoy-v2.local/ui/
company.staffjoy-v2.local/ui
The Swagger docs are at:
account.staffjoy-v2.local/swagger.json
company.staffjoy-v2.local/swagger.json


Improve Remix support

I've started experimenting with using Material UI and Joy UI components with Remix and I've run into a few bugs and blockers along the way. Browsing the issues here, I see several open threads where folks are discussing the same problems I've bumped into, so I thought it might be helpful to create an umbrella issue to bring all those discussions to one centralized, up-to-date location where folks can track the progress.
Here are all of the open issues I could find—some are overlapping, others are probably pretty outdated and possibly no longer relevant but I haven't dug too deeply into each one:

#34170
#31957
#30359
#31233
#31140
#31835
#30922
#38965

Problems
Here are the biggest things I see after reviewing these issues:

Emotion doesn't play well with Remix / React 18

renderToPipeableStream() function doesn't work with Emotion, causing hydration issues

supposedly this workaround can resolve the hydration stuff but I haven't tried it yet myself, and it looks like it came before Remix v2 so I don't know if that changes things

MUI doesn't support ES modules - https://github.com/remix-run/remix/issues/7314#issuecomment-1706998472

Remix uses esbuild under the hood - #31835

as mentioned in #39641, it's not ideal to have to customize entry.client.tsx and entry.server.tsx if it can be avoided/minimized

Potential solutions

migrate away from Emotion - WIP - #38137
publish packages as ES modules -  #30671

I've read that you can add serverModuleFormat: 'cjs', to remix.config.js to get around this problem, but it didn't work for me when I tried it—maybe I'm missing something

Examples and docs
There are several things we could do to improve the documentation around Remix + MUI!

We have a nice example of Remix v2 + Material UI: #39229
We could use a similar example for Joy UI: #39641
Base UI + Remix example too: #36193
It would also be really nice to have Integration guides for Remix like we do with the Next.js App Router

but maybe it's better to wait on "official" integration guides until we've resolved the bigger problems?

If there are any other known issues or requests to improve the DX, please share them here!

@samuelsycamore I have MUI working with renderToPipeableStream using this sample:
https://github.com/remix-run/examples/blob/main/chakra-ui/app/entry.server.tsx

i have this issue where if I import like so:
import { useDemoData } from "@mui/x-data-grid-generator";
I get
Uncaught TypeError: Cannot read properties of undefined (reading 'A400')
    at theme.ts:25:17
(anonymous) @ theme.ts:25

the provided remix template is old and not compatible, packages and layout is obsolete. please provide an up-to-date remix admin template for MUI with internationalization

@samuelsycamore I have MUI working with renderToPipeableStream using this sample:
https://github.com/remix-run/examples/blob/main/chakra-ui/app/entry.server.tsx

Hello,
I have just tried this one out and it may be outdated as I got the following error:
Module '"@remix-run/node"' has no exported member 'Response'.
I skipped importing that Response (whatever it is) and tried with the base FetchAPI Response class, and got the following error:
Argument of type 'ReadWriteStream' is not assignable to parameter of type 'BodyInit | null | undefined'.
The full code of trying this one is the following
`import { CacheProvider as EmotionCacheProvider } from '@emotion/react';
import { createReadableStreamFromReadable } from '@remix-run/node';
import { PassThrough } from 'node:stream';
import { RemixServer } from '@remix-run/react';
import { renderToPipeableStream } from 'react-dom/server';
import createEmotionCache from '@emotion/cache';
import createEmotionServer from '@emotion/server/create-instance';
import isbot from 'isbot';
import type { AppLoadContext, EntryContext } from '@remix-run/node';
const ABORT_DELAY = 5000;
export default function handleRequest(
request: Request,
responseStatusCode: number,
responseHeaders: Headers,
remixContext: EntryContext,
// This is ignored so we can keep it in the template for visibility.  Feel
// free to delete this parameter in your app if you're not using it!
// eslint-disable-next-line @typescript-eslint/no-unused-vars
loadContext: AppLoadContext
) {
return isbot(request.headers.get('user-agent'))
? handleBotRequest(request, responseStatusCode, responseHeaders, remixContext)
: handleBrowserRequest(request, responseStatusCode, responseHeaders, remixContext);
}
function handleBotRequest(request: Request, responseStatusCode: number, responseHeaders: Headers, remixContext: EntryContext) {
return new Promise((resolve, reject) => {
const emotionCache = createEmotionCache({ key: 'css' });
let shellRendered = false;
    const { pipe, abort } = renderToPipeableStream(
        <EmotionCacheProvider value={emotionCache}>
            <RemixServer context={remixContext} url={request.url} abortDelay={ABORT_DELAY} />
        </EmotionCacheProvider>,
        {
            onAllReady() {
                shellRendered = true;

                const body = new PassThrough();
                const emotionServer = createEmotionServer(emotionCache);
                // const stream = createReadableStreamFromReadable(body);

                const bodyWithStyles = emotionServer.renderStylesToNodeStream();
                body.pipe(bodyWithStyles);

                responseHeaders.set('Content-Type', 'text/html');

                resolve(
                    new Response(bodyWithStyles, {
                        headers: responseHeaders,
                        status: responseStatusCode,
                    })
                );

                pipe(body);
            },
            onShellError(error: unknown) {
                reject(error);
            },
            onError(error: unknown) {
                responseStatusCode = 500;
                // Log streaming rendering errors from inside the shell.  Don't log
                // errors encountered during initial shell rendering since they'll
                // reject and get logged in handleDocumentRequest.
                if (shellRendered) {
                    console.error(error);
                }
            },
        }
    );

    setTimeout(abort, ABORT_DELAY);
});

}
function handleBrowserRequest(request: Request, responseStatusCode: number, responseHeaders: Headers, remixContext: EntryContext) {
return new Promise((resolve, reject) => {
let shellRendered = false;
const { pipe, abort } = renderToPipeableStream(, {
onShellReady() {
shellRendered = true;
const body = new PassThrough();
const stream = createReadableStreamFromReadable(body);
            responseHeaders.set('Content-Type', 'text/html');

            resolve(
                new Response(stream, {
                    headers: responseHeaders,
                    status: responseStatusCode,
                })
            );

            pipe(body);
        },
        onShellError(error: unknown) {
            reject(error);
        },
        onError(error: unknown) {
            responseStatusCode = 500;
            // Log streaming rendering errors from inside the shell.  Don't log
            // errors encountered during initial shell rendering since they'll
            // reject and get logged in handleDocumentRequest.
            if (shellRendered) {
                console.error(error);
            }
        },
    });

    setTimeout(abort, ABORT_DELAY);
});

}
`

...I got the following error: Module '"@remix-run/node"' has no exported member 'Response'.
I skipped importing that Response (whatever it is) and tried with the base FetchAPI Response class, and got the following error: >Argument of type 'ReadWriteStream' is not assignable to parameter of type 'BodyInit | null | undefined'.

Remix used to ship a specific Response type that allowed you to dump in a ReadWriteStream into the Response, instead of maintaining that full Response object they now maintain a createReadableStreamFromReadable You would do mostly the same as what you had posted, but before slapping it all into the body of the Response you need to put it into the createReadableStreamFromReadable function like:
   // Import
   import { createReadableStreamFromReadable } from "@remix-run/node";
    // Other Code
   //
  //
           resolve(
                    new Response(createReadableStreamFromReadable(stream), {
                        headers: responseHeaders,
                        status: responseStatusCode,
                    })
                );

Is it possible to use material ui with remix spa mode?

The current Remix example is a bit outdated and hard to integrate with new Remix templates. I created a brand new integration (with Remix 2.8 + Vite). It helps alot in quick and smooth MUI + Remix integration and also addresses some challenges mentioned here.
Headsup! I am not great with Emotions. But just cross checking with current Remix + MU integration example, it seems it is working as expected with no need to use complex cache/style management.
cc @oliviertassinari @mnajdova

The current Remix example is a bit outdated and hard to integrate with new Remix templates. I created a brand new integration (with Remix 2.8 + Vite). It helps alot in quick and smooth MUI + Remix integration and also addresses some challenges mentioned here.

Thank you so much for this. I was pounding my head for the last 4 hours until I just found this. Incorporating your MuiProvider.tsx file into my entry files was the trick I needed to incorporate MUI into my Remix app. The current "official" example is impossible to decipher as a newbie to Remix working on the current default template.

The current Remix example is a bit outdated and hard to integrate with new Remix templates. I created https://github.com/mahmoudmoravej/remix-mui/pull/1. It helps alot in quick and smooth MUI + Remix integration and also addresses some challenges mentioned here.

@mahmoudmoravej would you like to update the current example we have and open a PR, we can then check what changes you've made.

@mnajdova I will do it but I am not sure it will be so helpful, becasue:

It is upgraded to Remix/Vite template, so you will see a bunch of unnecssary changes.
The approach I used is completely different, so there will not be a good side-by-side comparison.

I still think the PR I provided above(which simply added MUI on top of the latest version of Remix/Vite template) shows a better understanding of what and how it has been doe.

The official example uses serverModuleFormat: 'cjs' in the remix config which seems to prevent fast hot-reloading of front-end changes.
I used the starter from @mnajdova: https://github.com/mahmoudmoravej/remix-mui and it seems to work very well so far. Much simpler, streaming (defer/suspense/await) seems to work fine, hot-reloading etc all seems to work perfectly.

The current Remix example is a bit outdated and hard to integrate with new Remix templates. I created a brand new integration (with Remix 2.8 + Vite). It helps alot in quick and smooth MUI + Remix integration and also addresses some challenges mentioned here.
Headsup! I am not great with Emotions. But just cross checking with current Remix + MU integration example, it seems it is working as expected with no need to use complex cache/style management.
cc @oliviertassinari @mnajdova

Thank you so much for this! It really helped!

The current Remix example is a bit outdated and hard to integrate with new Remix templates. I created a brand new integration (with Remix 2.8 + Vite). It helps alot in quick and smooth MUI + Remix integration and also addresses some challenges mentioned here.
Headsup! I am not great with Emotions. But just cross checking with current Remix + MU integration example, it seems it is working as expected with no need to use complex cache/style management.
cc @oliviertassinari @mnajdova

my guy - if I could tip you I would. Spent a full day going round and round in circles and this example just works!

I have an issue with this template. If hydration fails and react falls back to client rendering the entire root, then the styles of emotion completely fail to load.
You can test this yourself easily if you just add a {Math.random()} to the root.tsx

I have an issue with this template. If hydration fails and react falls back to client rendering the entire root, then the styles of emotion completely fail to load.

I observe this as well, but did not know what was the cause. I'd be interested in how to fix this flaw.

Did you find a solution for the styles not loading when hydration fails?

Did you find a solution for the styles not loading when hydration fails?

No, I already had a working version (https://github.com/mui/material-ui/tree/master/examples/material-ui-remix-ts) that I wanted to replace with this cleaner setup.

I managed to find a way to make it work even when hydration fails:
MuiProvider.tsx
import { ThemeProvider } from '@mui/material';
import { theme } from './theme';
import { CacheProvider } from '@emotion/react';
import ClientStyleContext from 'ClientStyleContext';
import { useMemo, useState } from 'react';
import createCache from '@emotion/cache';

function createEmotionCache() {
  return createCache({ key: 'css' });
}

export function MuiProvider({
  children,
}: {
  children: React.ReactNode;
}) {
  const [cache, setCache] = useState(createEmotionCache());

  const clientStyleContextValue = useMemo(
    () => ({
      reset() {
        setCache(createEmotionCache());
      },
    }),
    [],
  );

  return (
    <ClientStyleContext.Provider value={clientStyleContextValue}>
      <CacheProvider value={cache}>
        <ThemeProvider theme={theme}>{children}</ThemeProvider>
      </CacheProvider>
    </ClientStyleContext.Provider>
  );
}

entry.server.tsx
const body = await renderToReadableStream(
  <MuiProvider>
      <ServerRouter
        context={routerContext}
        url={request.url}
        abortDelay={ABORT_DELAY}
      />
  </MuiProvider>
)

entry.client.tsx
hydrateRoot(
    document,
    <StrictMode>
        <MuiProvider>
            <HydratedRouter />
        </MuiProvider>
    </StrictMode>
  );

ClientStyleContext.tsx
import * as React from 'react';

export interface ClientStyleContextData {
  reset: () => void;
}

export default React.createContext<ClientStyleContextData>({
  reset: () => {},
});

Layout of root.tsx
import { withEmotionCache } from '@emotion/react';
import ClientStyleContext from './ClientStyleContext';
import { unstable_useEnhancedEffect as useEnhancedEffect } from '@mui/material';

export const Layout = withEmotionCache(
  ({ children }: { children: React.ReactNode }, emotionCache) => {
    const clientStyleData = useContext(ClientStyleContext);

    // Only executed on client
    useEnhancedEffect(() => {
      // re-link sheet container
      emotionCache.sheet.container = document.head;
      // re-inject tags
      const tags = emotionCache.sheet.tags;
      emotionCache.sheet.flush();
      tags.forEach((tag) => {
        (emotionCache.sheet as any)._insertTag(tag);
      });
      // reset cache to reapply global styles
      clientStyleData.reset();
    }, []);

     return (
      <html lang='en'>
        <head>
        ...

@cwagner22 where does <HydratedRouter /> come from?
Also, what specifically of this whole configuration fixed the problem with hydration? It is similar to the material-ui-remix-ts example, so knowing why it was breaking would be helpful. Thanks!

HydratedRouter is for React Router 7, which was RemixBrowser in Remix.
I tried to find the part from material-ui-remix-ts that made it working and it seems it was the part from useEnhancedEffect, which I hope can be improved at some point.

HydratedRouter is for React Router 7, which was RemixBrowser in Remix. I tried to find the part from material-ui-remix-ts that made it working and it seems it was the part from useEnhancedEffect, which I hope can be improved at some point.

Thanks for the clarification. I agree the useEnhancedEffect is likely the fix because it is re-applying the styles directly into the document.
I've implemented it and in dev. So far no errors and it properly refreshes even after I force an error in my code. Pushing out to staging to see if anyone can break it there.

I integrated the content solution provided in this issue into Remix v2.51.1 + SPA mode which supports Vite build (https://github.com/observerkei/remix-spa-material-ui/tree/main) , 🎉 hope it helps.

Is it possible to use material ui with remix spa mode?


Error handling (panics, nan, inf)

I think that a library like decimal should never panic. Generally inputs can come from users (e.g. from an HTML form) and it is much nicer to have an ability to handle errors without using recover or pre-validating decimals before each operation.
Have a look at math/big.Accuracy, I think it is better to use it instead of bool or error where it makes sense. E.g. https://golang.org/pkg/math/big/#Float.Uint64
It might be good idea to introduce +Inf and -Inf values for Decimal objects. This gives natural way to report errors for many operations, such as multiplying two large numbers, division by 0, parsing something like 1e+908940390904920390109201902909201901, etc.

I think that a library like decimal should never panic.

I never felt good about the panics myself. They were in there primarily because I had not worked on incorporating +/- Inf, which leads me to...

It might be good idea to introduce +Inf and -Inf values for Decimal objects.

Yes. I've been working on this tonight and am planning on following the IEEE 754 Inf rules much like the math package does.

BTW, math/big.Float doesn't support NaN. There is https://golang.org/pkg/math/big/#ErrNaN instead. Probably your library should use the same approach.

Yeah. I'm curious how I'd handle cases like -Inf + -Inf where (to my
knowledge) IEEE dictates the result should be NaN.
I'd like to get away from NaN, though, since +/- Inf should be good enough
(I think).
On Thu, Jan 14, 2016 at 6:47 PM kostya-sh<EMAIL_ADDRESS>wrote:

BTW, math/big.Float doesn't support NaN. There is
https://golang.org/pkg/math/big/#ErrNaN instead. Probably your library
should use the same approach.
—
Reply to this email directly or view it on GitHub
https://github.com/EricLagergren/decimal/issues/4#issuecomment-171852444
.

From https://golang.org/pkg/math/big/#Float.Add:

Add panics with ErrNaN if x and y are infinities with opposite signs. The value of z is undefined in that case.

I wonder if it's worth adding traps to the Context so the user can decide
whether they want to have quiet or signaling NaN.
There could be a flag (defaults to signaling?) that determines the behavior
of operations that result in NaN.
On Thu, Jan 14, 2016 at 8:27 PM kostya-sh<EMAIL_ADDRESS>wrote:

From https://golang.org/pkg/math/big/#Float.Add:
Add panics with ErrNaN if x and y are infinities with opposite signs. The
value of z is undefined in that case.
—
Reply to this email directly or view it on GitHub
https://github.com/EricLagergren/decimal/issues/4#issuecomment-171868965
.

I do not think I want to implement Accuracy. I'm not quite sure what benefit it adds, tbh.


Can we get batched jobs done with Leviathan/Jenkins

Do a test job X times, can we do it through Jenkins and not locally.

@klutchell Thought about it as I was thinking about #508
The challenge/issue is not running the tests multiple times or on multiple workers. The challenge is how do we deliver the assets. For a Jenkins job like that, how does one upload the assets to run the test suites? Here's what I was thinking.
(balenaOS image, suites, config.json) - Can't be uploaded, either the image comes from an upstream job or they fetch it with fetchOS helper right in the tests. I like the former option since that way folks can run a meta-balena job for their PR branch with custom tests, and the balenaOS image gets generated. Problem solved.
In the downstream, leviathan jobs we add logic to repeat tests on the basis of a variable in Jenkins that can be configured using the Jenkins "Build with parameters" job. We would also need to tell this special job about where to find the balenaOS artifacts built using the PR branch. Since we shouldn't be interfering with the PR flow anyway. We can do a test job X times for only one device type though.

Can't be uploaded, either the image comes from an upstream job or they fetch it with fetchOS helper right in the tests. I like the former option since that way folks can run a meta-balena job for their PR branch with custom tests, and the balenaOS image gets generated. Problem solved.

The only issue I see here is that meta-balena jobs and their artifacts are only kept around for a few days (I think).
What if we run the yocto- job first, then execute the leviathan job x times per the parameters. That way if artifacts have been purged from your last meta-balena run new ones will be generated for you but only for the device you're testing?
This is getting messy though, maybe we need to rethink this.

Quite frankly, I don't think it's gonna remain as automated as we think this is going to be if we do this with Jenkins. Maybe discussing this async might not be the best step. My alternative was,
We tweak Leviathan to run the tests X number of times over on local testing/development rig instead. How about we do a call after I come back to figure out development testing?

Yeah, we need to take this to another call with Ryan and Alex G I think. Maybe even an arch call.

This no longer feels relevent, based on the improvement introduced here: https://github.com/balena-os/leviathan/pull/809
tests can be marked as unstable: https://github.com/balena-os/leviathan/pull/809


The parser does not respect RFC3986 for scheme parsing

Issue summary
The parser does not fully comply with the RFC3986. As mentioned in the section 3.1. Scheme the - (hyphen) character is a valid one for a scheme.
System informations

Information
Description

league-uri-parser version
1.0

PHP/HHVM version
>=7.0.0

OS Platform
all

Standalone code, or other way to reproduce the problem
(new Parser)('android-app://org.wikipedia/http/en.m.wikipedia.org/wiki/The_Hitchhiker%27s_Guide_to_the_Galaxy')

Expected result
[
    'scheme' => 'android-app',
    'user' => null,
    'pass' => null,
    'host' => 'org.wikipedia',
    'port' => null,
    'path' => '/http/en.m.wikipedia.org/wiki/The_Hitchhiker%27s_Guide_to_the_Galaxy',
    'query' => null,
    'fragment' => null,
]

Actual result
League\Uri\Exception: The submitted uri `android-app://org.wikipedia/http/en.m.wikipedia.org/wiki/The_Hitchhiker%27s_Guide_to_the_Galaxy` contains an invalid path

@Baptouuuu good catch this was an oversight from my part 👍

Glad to help :)

1.0.4 is released


Transparency disabled for 2D, wireframe, and barb render

The TFE editor for the 2D, wireframe, and barb render no longer provides control for transparency.
To reproduce:

Load any data set
Enable one of the renderers listed above and try to modify the transparency in the TFE. The transparency/opacity controls are not enabled.

renable 2D data renderer. OK to leave barb and wireframe as is

The described problem still exists for wireframe and barb renderer.

The initial bug included wireframe and the barb render. However, per the comment above we decided that only the 2D data renderer should be addressed, and the wireframe and barb renderer left alone. Closing.


ability of saving parameters to memory

Description
the save_parameters() method of gluon.nn.HybridBlock only saves model parameters to file. it should be possible to save the model to a file-like object. this lets us combine many models, save them to a dict object, pickle that dict object and load from that (think of saving encoder and decoder into just one file than compress it).
of course the possibilities are endless, and this was just an example.
References
pytorch has an api torch.save and torch.load as far as I can remember.

You can collect the parameters and convert them to numpy.
Also note that as of MXNet v2, the saves to the numpy npz file format (see https://numpy.org/doc/stable/reference/generated/numpy.savez.html).

@leezu is it possible to pickle the collected parameters without any data loss?

@leezu I want to pickle with x86_64, and unpickle using that architecture. so, that should not be an issue. but, my main concern was the ability of saving more than 1 network + additional data like the parameters of the optimizer into 1 file.


Fix for https://github.com/libjxl/libjxl/issues/150

Overriding the color transform in enc_frame can an xyb_encoded <->
xyb mismatch assert to fail.
However, overriding color transform is still necessary to encode other
images correctly with correct color, such as y4m. Do this earlier, in
enc_file, now.

Hi Jon, assigning this to you since you likely remember what the original reason for adding this code was and what alternative fix needs to be done for y4m. Please take a look, thank you :)

Actual fix implemented now, PTAL

LGTM, just got something unrelated in dec_frame.h that ended up in the commit somehow.

All fixed now


Update scripts/autoinstall.sh (see description)

Now it adds PATH for every terminal automatically
It also exports 2 vars that can be used for scripting with blade

Works fine on Linux, but doesn't work on OSX. We can keep the old flow for OSX and maintain the new one for Linux only

Done, the next version I'll try to make it work automatically and checking if user is in OSX or Linux, also integrate other ways with this script. I will also try to learn batch to make a windows version too.


Add ng-bootstrap to UI Component resources list

The ng-bootstrap team just released an alpha 0 version of ng-bootstrap today - we would like the project added to the UI Component section of the resources list here.
Also I notice the list has no particular ordering - perhaps this list should be in alphabetical order?

I am part of that team too (along with @pkozlowski-opensource) and yeah, we would like an entry there. I also agree on the alphabetical order, not giving anyone any "promoted position".
/cc @naomiblack @ericjim

I believe @StephenFluin has access to the database.

This is done now, thnx @StephenFluin ! https://angular.io/resources/
@wesleycho I think this can be closed now.

I still think that list should be alphabetized, but I guess the main part of this issue is done.

I'll look into alphabetizing the content.


Child item HTML is malformed

Problem Description
I am writing an extension with the ability to add multiple child elements to the parent element. I noticed that the HTML output double wraps the child item in two sets of identical divs. This is also the case with the parent/child custom CTA from the divi-extension-example.
Screenshot and/or gif

In the second image, you can see that the divs with classes of 'dicm_cta_child_0' and 'et_pb_module_inner' class are repeated - with the second set nested inside the first. I wasn't aware that the child items would be wrapped this way at all. I can see it causing some problems with both CSS and DOM traversing in jQuery adding in items that are outside of what has been coded in the module.

QA Checklist
Reporter

[ ] Helpful title used

Helpful title format: Scope(s) :: One Sentence To Describe The Issue
Example: VB :: Floating Toolbar :: Jumps To Top Of Page

[ ] Detailed description

Description should usually be 150+ characters.

[ ] Steps to reproduce included
[ ] Support Ticket URL included
[x] Image and/or gif

Developer

[ ] Steps to reproduce proven solved
[ ] Test included and coverage acceptable and all Tests passing?
[ ] PR description has layman's description of problem and solution and "Areas Affected" listed/explained?
[ ] PR branch includes latest commits from master branch (at the time of checking)?
[ ] Estimated points on issue
[ ] Proper code documentation and conventions?
[ ] Proper RTL appearance
[ ] i18n used if applicable
[ ] Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
[ ] Database saved/updated? Data sanitized and validated?
[ ] Variable output properly escaped and validated?

Peer Review

[ ] PR branch includes latest commits from master branch (at the time of checking)?
[ ] Steps to reproduce proven solved
[ ] Affected areas are tested and ok
[ ] Proper code documentation and conventions
[ ] Proper RTL appearance
[ ] i18n used if applicable
[ ] Test(s) included and coverage acceptable
[ ] Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
[ ] Database saved/updated? Data sanitized and validated?
[ ] Variable output properly escaped and validated?

Quality Assurance

[ ] All tests passed on CircleCI.
[ ] Steps to reproduce passed, the fix works.
[ ] No errors in console or output
[ ] Affected areas are tested and ok
[ ] The fix does not change a customer's website in any surprising or unintended way

[ ] If Divi's design is adjusted, settings are migrated gracefully and invisibly

Design Review

[ ] UI is to spec
[ ] UX is to spec
[ ] Design is to spec
[ ] No surprising visual changes to a customer's website will be experienced after updating

[ ] Any/all visual changes are migrated gracefully

Dev Review

[ ] Test(s) included and coverage acceptable
[ ] Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
[ ] Database saved/updated? Data sanitized and validated?
[ ] Variable output properly escaped and validated?
[ ] i18n used if applicable
[ ] Implementation acceptable, DRY, Scaleable, future safe
[ ] Best practices used, cleverness and grokage, ES6 use, etc.

Thanks for the report. We'll take a look at it and get a fix out next week.

I also noticed that when you restore a previous state in the VB, the et_pb_module_inner class is missing in the childs wrapper so you end up with a div with no class.

Same here, a div element with the child module classes and another div element with the class et_pb_module_inner are rendered with every item when vb_support is 'on'.
These aditional wrappers break the entire module. Any workaround while it is fixed? If so, please share so we can adapt our modules to the new Divi API. Thanks in advance! JC

Found this in the ET_Builder_Element class:
// Wrap 3rd party module rendered output with proper module wrapper
		// @TODO implement module wrapper on official module
		if ( 'on' === $this->vb_support && 'render' === $render_method && ! $this->_is_official_module ) {
			$output = $this->_render_module_wrapper( $output, $render_slug );
		}

If the vb_support is set to 'on' the wraper classes are added so this issue has nothing to do with the create-divi-extension, this is how it works for all the modules since Divi 3.1. The new question is:
Should we change our HTML rendering to meet this?? I would like to render my child modules like I always did (without these wrapers) and like the official modules currently does (see the Tab / TabItem output).
This is a very important change, please support, thank you again.

Also noticed that, a simple fix : replace return $this->_render_module_wrapper( $output, $render_slug ); in https://github.com/elegantthemes/divi-extension-example/blob/master/includes/modules/CustomCtaChild/CustomCtaChild.php#L180 with return $output;
For now I don't see any issue doing this replacement.

@MarieComet Thanks for noticing that! Looks like the example extension wasn't updated when we moved the call to that function up into Divi's code instead of in the extensions :man_facepalming:


Flutter version solving Failed

Steps to Reproduce

Run flutter Upgrade on older flutter version

Logs
The current Dart SDK version is 2.1.2-dev.0.0.flutter-0a7dcf17eb.
Because flutter_tools depends on build_test >=0.10.7 which requires SDK version >=2.2.0 <3.0.0, version solving failed.
exit code 1
##What i found
I understand this is due to flutter_tools requiring dart sdk version >2.2.0.
But my internal dart sdk version is 2.1.2.
How do i get arround this.Can i upgrade the dart sdk in flutter?

The issue is already long time not active and without answer.
Could everyone who still has this problem please file a new issue with the exact descriptions what happens, logs and the output of flutter doctor -v please.


Remove truffle support

Finally removing truffle support from mythril.

Oh! yeah, forgot to git add that change


Can't open file by typing path with keyboard

Prerequisites

[X ] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: https://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: https://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
Can't open a file by typing in the file path
Steps to Reproduce

Press Ctrl-O
Try type in /etc/hosts press Enter
See that /etc/hosts has opened

I'm testing on KDE Plasma Ubuntu 18.04 but that's probably irrelevant.
The fileopen dialog that opens doesn't allow me to type in a filepath anywhere.
Expected behavior:
Look at KWrite for example

Actual behavior:
A fileopen dialog opens that's unusable via the keyboard. Extremely unproductive and tediously requires mouse-only operation.

Suggested fix
Fixing this should be trivial. KWrite (for example) shows a more useful (but standard) fileopen dialog that allows entering a path, so it's probably a trivial setting that needs to be changed.
Reproduces how often:
Always
Versions
Atom    : 1.38.2
Electron: 2.0.18
Chrome  : 61.0.3163.100
Node    : 8.9.3
Ubuntu 18.04
Not sure if these are relevant
ii  plasma-desktop                                  4:5.12.7-0ubuntu0.1
libgtk-3-bin                                    3.22.30-1ubuntu3
libgtk2.0-bin                                   2.24.32-1ubuntu1
qt5-default:amd64                               5.9.5+dfsg-0ubuntu2.1
libqtgui4:amd64                                 4:4.8.7+dfsg-7ubuntu1
Similar to previously closed issue
https://github.com/atom/atom/issues/4777

The dialog you are seeing is coming from your OS, Atom has no control over what it looks like. It looks to me like Kwrite has created a custom file open dialog there, but I can't say for sure.
It may be that there is a way to select some "better" dialog from the operating system, but if so that would unfortunately need to be changed within Electron, the platform Atom is built on.
It looks like you have a few options here:

See if you can enable this functionality in the OS file picker
Check if there is an issue on your OS's bug tracker about the lack of functionality (by default?) in their file picker
Use the advanced-open-file package linked in the other issue, or something similar
Check if there is already an issue filed on Electron about this

Since it appears there isn't anything that can be done about this inside Atom itself, I'm going to close this issue. Thanks for taking the time to file an issue though!

Thanks @Arcanemagus in the mean time I've installed and tested advanced-open file. It does indeed get the job done, quite well actually.
I don't think the file open dialog is specific to KWrite. It must be some standard thing.
For interest's sake here's what Pinta's file open dialog looks like:

Interestingly also, sublime has the same crappy file open dialog as Atom.io is showing.
Furthermore, LibreOffice Writer has the exact same fileopen dialog style as KWrite, and the 2 pieces of software aren't related at all.


chore: release 1.1.0

:robot: I have created a release *beep* *boop*
1.1.0 (2019-06-06)
Features

support apiEndpoint override in client constructor (#7) (368a004)

Release is at https://github.com/googleapis/nodejs-phishing-protection/releases/tag/v1.1.0


run_mnist_cpp_example error when compiled with cpu/gpu

environment：CPU: intel i7 8th, with no independent graphics card.
I build with docker and run it without --device='/dev/kfd'.
I can run successfully in ref mode   ./mnist_inference, but I encountered the following three problems when compiled with cpu/gpu.

make error

Can I compile a model on a device that is not AMD cpu/gpu?

cpu mode error

gpu mode error

THANKS!

By the way, I'm building in VMware

The cpu is disabled by default so you need to pass -DMIGRAPHX_ENABLE_CPU=On to cmake or rbuild to build the cpu backend.
The gpu backend only works with AMD gpus supported by the rocm stack.
Also I dont know why the driver failed to build, I would need to see the error.

it's OK,Thank you!

Please reopen if there are further issues.


Why the CSS files aren't imported by the components that use them?

The real pain is keeping track of the styles that are already included
For example, the Menu component should do this in the component file:
import<EMAIL_ADDRESS>import<EMAIL_ADDRESS>import<EMAIL_ADDRESS>
Can you spot the problem here?
import<EMAIL_ADDRESS>import<EMAIL_ADDRESS>import<EMAIL_ADDRESS>import<EMAIL_ADDRESS>import<EMAIL_ADDRESS>import<EMAIL_ADDRESS>import<EMAIL_ADDRESS>import<EMAIL_ADDRESS>import<EMAIL_ADDRESS>import<EMAIL_ADDRESS>import<EMAIL_ADDRESS>import<EMAIL_ADDRESS>import<EMAIL_ADDRESS>import<EMAIL_ADDRESS>import<EMAIL_ADDRESS>import<EMAIL_ADDRESS>import<EMAIL_ADDRESS>import<EMAIL_ADDRESS>
Answer: many styles are repeated, I just copied the Menu styles. Now i have to remove the repeated styles manually.
Importing out of the box shouldn't be an issue because all classes are namespaced with mdc- and the files are not being repeated because webpack loads them once. It will save a lot of time whenever we want to install a new component (avoid creating a custom component that imports these styles).
I know the library tries to be un-opinionated but some opinion is good!

In a project that is using create react app and webpack, I agree with you. As soon as you start to consider other methods of including styles this breaks down.
Not all build systems include css loaders and there are many reasons you would want to keep the css separate (separate build process for css, no build process at all). Also some people use SASS for this project as well which is a separate compile process which means importing the css would be the wrong behavior.
The simplest way that I’ve personally worked around this is to have a styles.js / styles.ts file at the root of the project that uses all of the import statements I need to pull in. I also keep the imports this file alphabetical with the exception of a few imports that should come first (theme is an example).
I’m unsure about the idempotence of importing a css file (will importing the same file twice do a double import?). If webpack automatically resolves duplicate imports then there would be no harm in just dumping all the things in a file blindly. I believe that it will only include the file contents once in the bundle but  it will insert the style tag twice causing weird issues and degraded css performance.
Happy to brainstorm, I just know with certainty that adding inline imports inside the component will break people. I could add a single styles.js file per component to import, but I believe you’ll run into the same issue I mentioned above which is double style tag inclusion.

How about a different file that exports the component and imports the css?
// file: @rmwc/component/button/index.js

import<EMAIL_ADDRESS>export { Button } from "@rmwc/button";

Now all we have to do is
import { Button } from "@rmwc/component/button";

function App() {
  return (
    <Button/>
  )
}

What do you think?

I’m not opposed to it. There’s just a lot to maintain as it stands. Something like you’re describing is easy to setup, easy to forget, and hard to write a unit test for since it relies on a working webpack build system :/

I think there are so many different possibilities on how to include styles and build your web app that I would very much vote to keep the strict un-opinioned separation of js/ts and css.
I am quite certain that somehow some use-case will be destroyed or at the very least will have some performance loss or confusing bugs with such a change.

This is definitely going to be out of scope. @j-o-d-o is right, there are just too many ways to include styles in your app these days. While the suggestion may seem convenient, it's likely to cause edge cases and maintainability issues just for a bit of developer convenience


STORM-2844: KafkaSpout Throws IllegalStateException After Committing to Kafka When First Poll Strategy Set to EARLIEST

Serialize KafkaSpoutMessageId to JSON and add it as part of OffsetAndMetadata
Check if commits to Kafka were committed by this topology to correctly enforce FirstPollOffsetStrategy

@srdo can you please do a first pass review. In the meantime I will address the 2 or 3 TODO with minor cleanups. The following tests are also failing. I am working on fixing them, but if you know from the top of your head how to fix these failures (I don't think they are bugs added by the patch), I would appreciate your tips. Thanks.
Tests in error:
  KafkaSpoutRebalanceTest.spoutMustIgnoreFailsForTuplesItIsNotAssignedAfterRebalance:184 InvalidUseOfMatchers
  KafkaSpoutRebalanceTest.testReassignPartitionSeeksForOnlyNewPartitions:244 » Runtime
  MaxUncommittedOffsetTest.testNextTupleCanEmitMoreMessagesWhenDroppingBelowMaxUncommittedOffsetsDueToCommit:120 » NullPointer
  MaxUncommittedOffsetTest.testNextTupleWillNotEmitMoreThanMaxUncommittedOffsetsPlusMaxPollRecordsMessages:187->failAllExceptTheFirstMessageThenCommit:170 » NullPointer

@srdo I plan on submitting a patch to master, but why does the patch need to go on master first? For example, there could be bugs in 1.x that no longer exist in master because of other changes, and in that case the change will only go in 1.x.

@hmcl I don't know that there's a hard rule that it has to be that way, but it seems like a good way to me to ensure that master is the "most fixed"/best state of the code. It's been done this way in the other PRs I've seen.
This doesn't apply to patches that fix issues that don't exist in later branches, but this issue isn't one of those.

@srdo I created the PR here because some topologies with latest master were not working for me and I worked off 1.x-branch to be able to make progress. I will submit a PR for master as well.

@hmcl Great, thanks.

@srdo I am working on making unit tests pass and then will squash and create master PR. Can you take another look in the meantime. Thanks.

@srdo once I get +1 I will create the PR for master.

@hmcl There were a few more comments, just making sure they weren't missed.
https://github.com/apache/storm/pull/2465#discussion_r157350384
https://github.com/apache/storm/pull/2465#discussion_r157375639
https://github.com/apache/storm/pull/2465#discussion_r157375809

@srdo it should be good now. Can you please take a look. Thanks.

@srdo can you please do one last review. Thanks.

These two changes address your comment: "test that when you call findNextCommitOffset, it returns the metadata you put in)"
https://github.com/hmcl/storm-apache/blob/baaf1fe6ab725ec50ecb09238f8957bc22a4e290/external/storm-kafka-client/src/test/java/org/apache/storm/kafka/spout/internal/OffsetManagerTest.java#L67
https://github.com/hmcl/storm-apache/blob/baaf1fe6ab725ec50ecb09238f8957bc22a4e290/external/storm-kafka-client/src/test/java/org/apache/storm/kafka/spout/internal/OffsetManagerTest.java#L82
As for the other tests, I started working on adding tests for scenarios around EARLIEST/LATEST and topology id. However, after working on it for awhile, it didn't feel natural that to test such a simple thing I had to mock so many objects (including KafkaSpoutConfig) and run the whole spout code. Then I decided that I would evaluate if the tests, code, or both should be considered for refactoring to make it easier to test. I need to dig into this further, but it seems to me that system tests are coupled with unit tests, and that makes it hard to write pure unit tests.
Let's merge https://github.com/apache/storm/pull/2464 and I will put my patch on top of it.

Thanks, I missed those changes.
Regarding testing EARLIEST/LATEST, I don't understand why you'd need to mock KafkaSpoutConfig. I think it can be tested as an integration test, like the ones in SingleTopicKafkaSpoutTest, where you start a spout, emit and commit a tuple, deactivate/reactivate it, verify that it can emit and commit the next tuple, close the spout and create a new one and verify that the new spout starts over/starts at the end.
Alternatively you can do it with mocks like many of the other tests, basically doing the same flow but verifying that the spout calls seekToBeginning/seekToEnd instead of checking which tuples it emits.
Could you put a +1 on https://github.com/apache/storm/pull/2464 so I can merge?

@srdo can you pls take a look. Thanks.

@srdo It should be OK now. If you have further requests lest's file a refactoring JIRA and include in it refactoring some of the unit tests for better code reuse.

@srdo done. Pls check and I will squash the commits right away. I would like to try to merge this in today. I will update the master PR with everything squashed already. Thanks.

+1

@srdo Thanks for the review. I have squashed the commits and it is ready to merge.


Codable support

Is your feature request related to a problem?
You don't support Codable when using capture for events.
Describe the solution you'd like
Getting error: key does not conform to NSCoding and cannot be serialized for delivery.
Describe alternatives you've considered
Allow Codable
Additional context
Thanks! Looking forward to reply.
Thank you for your feature request – we love each and every one!

@JoeFerrucci do you mean https://developer.apple.com/documentation/swift/codable ?

Yes, that is the one. When I try to pass a dictionary into capture(event:properties:) it fails with that error.
Here is a trivial example to help highlight it
struct MyType: Codable {}

posthog?.capture("My Event", properties: ["somekey": MyType()])

This yields the error. Technically PostHog SDK only needs to support Encodable; it doesn't need to be Decodable


Altering closeTimeout property of websocket client

I didn't find any way to do this in the source code. I saw the websocket client has a constructor with a config parameter. Is it alterable by the user?
Is there a way to prevent the socket from closing automatically? I'm trying to communicate with a speech to text service which can have some delays before reporting results.

I'm having the same issues with the closing delay.

Is there any further information regarding this? How to configure this parameters in NodeJS?

@majorgilles @amasad @p4r4n0id - you set it the same way as you would for the server:
var client = new WebSocketClient({
  closeTimeout: 15000
});

However, this property has nothing to do with what you're probably thinking it does. It's not at all related to keep-alive functionality or closing idle connections.
In the documentation, closeTimeout is described as:

The number of milliseconds to wait after sending a close frame for an acknowledgement to come back before giving up and just closing the socket.


Field does not support aggregate operator COUNT

I am receiving the following error when creating a new record
"dlrs_OfficersTrigger: execution of AfterInsert caused by: System.QueryException: field Current__c does not support aggregate operator COUNT Class.dlrs.LREngine.QueryExecutor.query: line 631, column 1 Class.dlrs.LREngine.WithSharingQueryExecutor.query: line 640, column 1 Class.dlrs.LREngine.rollUp: line 229, column 1 Class.dlrs.LREngine.rollUp: line 108, column 1 Class.dlrs.RollupService.updateMasterRollupsTrigger: line 958, column 1 Class.dlrs.RollupService.handleRollups: line 842, column 1 Class.dlrs.RollupService.triggerHandler: line 307, column 1 Trigger.dlrs_OfficersTrigger: line 7, column 1"
Here is the scenario: I set up a DLRS to count total current officer records (different officer positions held by that person) for each contact. Current__c is a formula field with the result as a checkbox. Does Count not work on that field type? Any workarounds you can suggest?

I was able to make a formula with a number result 1 if the checkbox is checked and 0 if not checked. Then changed the field to aggregate to that. The DLRS now works and counts the number.  However, now when I make it so the checkbox is unchecked and the number is 0, the aggregate result field is not updating to zero. (Stays at 1)

I solved this by making another field that is 1 if checked and 0 if not then used that field to COUNT. It is working now. Apparently it can't aggregate a formula field with checkbox result.

You could also try counting Id and filtering on your other field.  Go ahead and mark the issue closed now that you've got things working.


Find out more about Portable PDB

Miguel de Icaza blogged a couple of days ago bout new new Portable Program Database (PPDB) files format.
Portable PDB v0.1: Format Specification Draft
https://github.com/dotnet/roslyn/blob/master/docs/specs/PortablePdb-Metadata.md
Code got merged to Roslyn master 10 days ago:
https://github.com/dotnet/roslyn/pull/3891

Mono 4.2.0 was released with support for the PPDB format which is version 0.1.
http://www.mono-project.com/docs/about-mono/releases/4.2.0/

There is related code in Roslyn:
https://github.com/dotnet/roslyn/tree/master/src/Debugging/Microsoft.DiaSymReader.PortablePdb

The related code above exposes a portable pdb to the existing Windows DiaSymReader for compatibility with existing Windows APIs.

Great progress today and a plan is in place for the cross platform portable pdb format. Portable pdb is the future for .NET.
https://github.com/aspnet/Universe/issues/131#issuecomment-154227813

I proposed that F# switch to the portable pdb format. https://github.com/Microsoft/visualfsharp/issues/718

The discussion for adding .pdb files to the NuGet packages in Roslyn is https://github.com/dotnet/roslyn/issues/3. They will be the trend setter.

It is easy to read in the portable pdb format using System.Reflection.Metadata.
https://www.nuget.org/packages/System.Reflection.Metadata
open System.Reflection.Metadata
open System.Runtime.InteropServices
open System.IO
open Microsoft.FSharp.NativeInterop

#nowarn "9"

let printfn format = Printf.ksprintf System.Diagnostics.Debug.WriteLine format

[<EntryPoint>]
let main argv =
    let f = @"C:\cs\roslyn\Binaries\Obj\PdbUtilities\Release\Roslyn.Test.PdbUtilities.pdb"
    let bytes = File.ReadAllBytes f
    let pBytes = GCHandle.Alloc(bytes, GCHandleType.Pinned)
    let mr = MetadataReader(pBytes.AddrOfPinnedObject() |> NativePtr.ofNativeInt, bytes.Length)
    for dh in mr.Documents do
        let d = mr.GetDocument dh
        let name = mr.GetString d.Name
        let language = mr.GetGuid d.Language
        let hashAlgorithm = mr.GetGuid d.HashAlgorithm
        let hash = mr.GetBlobBytes d.Hash

        // Hex.fs from https://github.com/ctaggart/SourceLink/blob/master/SourceLink/Hex.fs
        printfn "%s %s" (Hex.encode hash) name
    pBytes.Free() // reminder for later
    0

The SHA1 for StringUtilities does indeed match:

Portable PDB support is coming in #121.


how upscaling custom face?

hello, i have a photo of a face, how upscaling face?

Hi，you can upscale the face by bicubic interpolation to specified size like code in data_prepare.py  first, then validate the pretrain model.

I do not understand, I have a blurred face, how do I make it hd?

Hi，you can find it in Inference Alone part.

help infer.py asks me for 3 images sr hr lr, but I only have one image.
the description says "HR (high resolution vanilla images), SR (images need to be processed)"
what is hr? Is it the same image in high resolution? It doesn't make sense if my goal is to go from low resolution to high resolution.

You can upscale the face by yourself to get the SR first. HR is the ground truth image, you can replace it with SR if you did not have ground truth image.


Appium Inspector with WinAppDriver Failed to create session

I'm newer on Appium with Windows Application Driver. But I cannot start session from Appium Inspector. Following is detailed information, am I missing anything here? Thanks in advanced.
Operating System:
Edition: Windows 10 Enterprise
Version: 21H2
OS Build: 19044.1766
Tools installed:
Appium Server GUI 1.22.3-4
Appium Inspector 2022.5.4
Windows Application Driver 1.2.99
Repro Steps

Launch Appium Server GUI and set the settings as this screenshot shows:

Start Server. Following logs shows in Appium Server GUI:

[HTTP] Waiting until the server is closed
[HTTP] Received server close event
[Appium] Welcome to Appium v1.22.3
[Appium] Non-default server args:
[Appium]   allowCors: true
[Appium]   address: <IP_ADDRESS>
[Appium]   relaxedSecurityEnabled: true
[Appium]   allowInsecure: {
[Appium]   }
[Appium]   denyInsecure: {
[Appium]   }
[Appium] You have enabled CORS requests from any host. Be careful not to visit sites which could maliciously try to start Appium sessions on your machine
[Appium] Appium REST http interface listener started on <IP_ADDRESS>:4723

Launch Appium Inspector and fille Remote Path and Desired Capabilities as below screenshot shows:

Click Start Session button.

Excepted Result:
The Windows Store App session will be started successfully.
Actual Result:
This error shows in Appium Inspector:

in remote path you are using wb/hub it should be wd/hub

@anunay1  Thanks for helping checking this.
After change the remote path to wd/hub, then start session, I get this error:

Attachment shows the logs in Appium Server GUI:
appium-server-logs.txt
At the beginning of the logs, it shows Appium Server is listening on <IP_ADDRESS>:4723

[info] [Appium] Appium REST http interface listener started on <IP_ADDRESS>:4723

But after starting session from Appium Inspector, the logs shows "WinAppDriver exists, but the checksum did not match. Was it replaced manually?" and "GET http://<IP_ADDRESS>:4724/wd/hub/status". It seems the session request is calling <IP_ADDRESS>:4723 and seems something wrong when Appium Server communicate with WinAppDriver. Is it the problem? How can I configure Appium Server can communicate with WinAppDriver as normal?

[info] [Appium] Appium v1.22.3 creating new WindowsDriver (v1.19.1) session
[info] [Appium] Applying relaxed security to 'WindowsDriver' as per server command line argument. All insecure features will be enabled unless explicitly disabled by --deny-insecure
[debug] [BaseDriver] W3C capabilities and MJSONWP desired capabilities were provided
[debug] [BaseDriver] Creating session with W3C capabilities: {
[debug] [BaseDriver]   "alwaysMatch": {
[debug] [BaseDriver]     "platformName": "Windows",
[debug] [BaseDriver]     "appium:app": "50582LuanNguyen.NuGetPackageExplorer_w6y2tyx5bpzwa!App",
[debug] [BaseDriver]     "appium:deviceName": "WindowsPC",
[debug] [BaseDriver]     "appium:newCommandTimeout": 3600,
[debug] [BaseDriver]     "appium:connectHardwareKeyboard": true
[debug] [BaseDriver]   },
[debug] [BaseDriver]   "firstMatch": [
[debug] [BaseDriver]     {}
[debug] [BaseDriver]   ]
[debug] [BaseDriver] }
[warn] [BaseDriver] The following capabilities were provided, but are not recognized by Appium:
[warn] [BaseDriver]   connectHardwareKeyboard
[info] [BaseDriver] Session created with session id: 030f22bc-d237-4d13-b12a-51475d940192[warn] [WinAppDriver] WinAppDriver exists, but the checksum did not match. Was it replaced manually?[info] [WinAppDriver] Spawning 'C:\Program Files\Windows Application Driver\WinAppDriver.exe' with args: ["4724/wd/hub"]
[debug] [WD Proxy] Matched '/status' to command name 'getStatus'
[debug] [WD Proxy] Proxying [GET /status] to [GET http://<IP_ADDRESS>:4724/wd/hub/status] with no body
[debug] [WinAppDriver] Windows Application Driver listening for requests at: http://<IP_ADDRESS>:4724/wd/hub

when I run it on vmware , there are no issuees;but when I run it  locally, the same sa your issue.My God

Spawning 'C:\Program Files\Windows Application Driver\WinAppDriver.exe' with args: ["4724/wd/hub"]
but failed


remove quotes in POSTGRES_DATASIZE and POSTGRES_WORKER_NODE_COUNT

Scenario which you are working on
A link to the Jumpstart scenario you are working on.
https://github.com/microsoft/azure_arc/blob/main/azure_arc_data_jumpstart/aks/arm_template/postgres_hs/azuredeploy.parameters.json
Describe the bug
{"error":{"code":"InvalidTemplate","message":"Deployment template validation failed: 'Template parameter JToken type is not valid. Expected 'Integer'. Actual 'String'. Please see https://aka.ms/resource-manager-parameter-files for usage details.'.","additionalInfo":[{"type":"TemplateViolation","info":{"lineNumber":230,"linePosition":26,"path":"parameters.POSTGRES_DATASIZE"}}]}}
if you are deploying the file as is you hit 2 errors for both parameters so maybe just remove the quotes?:

  "POSTGRES_WORKER_NODE_COUNT": {
            "value": <PostgreSQL Hyperscale server group number of workers>
        },
        "POSTGRES_DATASIZE": {
            "value": <PostgreSQL Hyperscale size of data volumes in MB>
        },

Thank you @AwdotiaRomanowna. This is why we have included in the guide a link to the azuredeploy.parameters.example.json so you won't make any mistake.
https://azurearcjumpstart.io/azure_arc_jumpstart/azure_arc_data/aks/aks_postgresql_hyperscale_arm_template/#deployment
Removing the quotes will result in syntax violation.

so maybe better replace it with default values?
like 2 for POSTGRES_WORKER_NODE_COUNT and 1024 for POSTGRES_DATASIZE so the minimum one?

Default values are configured in the template file:
    "POSTGRES_WORKER_NODE_COUNT": {
      "type": "int",
      "defaultValue": 3,
      "metadata": {
        "description": "Number of PostgreSQL Hyperscale worker nodes"
      }
    },
    "POSTGRES_DATASIZE": {
      "type": "int",
      "defaultValue": 1024,
      "metadata": {
        "description": "Size of data volumes in MB"
      }
    },

We wanted to provide a modular deployment option. The question is, assuming you were to take a look at the example file, would you still missed it?

i kind of wanted it to work without much digging like a jumpstart - so my first deployment of hyperscale.
a collegue of mine did the same as me hitting the same error.... so maybe i'm extrapolating too much but might be a more common usability issue


Add time to event list

Fixes Issue #41

Closes #41
Changes proposed
Updated cards to include time - date not included as all data is from the same session/day

Check List (Check all the applicable boxes)

[X] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[X] This PR does not contain plagiarized content.
[X] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

hello @Cahllagerfeld just checking in if there is anything I should modify here.  Also, if I am tagging you too soon or should check in some other way, please let me know

I missed it too! Looks great 👍  thank you both 🔥


Rename FuturesUnordered.push() to add() or insert()

I'm not sure how much can realistically be done about this, but I wanted to give the feedback that I find the FuturesUnordered.push API slightly confusing.
I associate push with Vectors or other ordered containers and as such when I read code using FuturesUnordered (especially in an environment that is not showing type hints) intuitively assume that the ordering in which futures are added to the combinator matters.
Maybe calling it add or insert instead of push would make it more clear to a HashSet and emphasise that the order in which we iterate over the results is not the order in which futures were added.
I'd be interested to hear if there was any deeper reason for that naming choice and understand that this might be hard to change now, so feel free to close this issue...

I would prefer not to do this for consistency with FuturesOrdered's API.

Maybe calling it add or insert instead of push would make it more clear to a HashSet and emphasise that the order in which we iterate over the results is not the order in which futures were added.

Note that set/map's insert is often an operation that may replace an existing value.
Also, I'm not sure if it's common in Rust to use add as the method name for collections. At least std doesn't use it (in std, add is basically a name for arithmetic methods).


Sketch integration

Basically using Sketch as the source of truth for colors, typography, and shapes. This could be done via the style mixin feature.

Marking this as completed since import functionality exists. Also have a few tickets for importing design files into Tandem, so will likely add integration features along with those issues.


upload eclipse plugin (#9)

This change is

Review status: 0 of 4 files reviewed at latest revision, 1 unresolved discussion, some commit checks broke.

stereotype.check.repository/pom.xml, line 3 at r1 (raw file):

<?xml version="1.0"?>
<!-- Copyright 2016 NovaTec Consulting GmbH Licensed under the Apache License,
	Version 2.0 (the "License"); you may not use this file except in compliance

hier hat es die formatierung zerhauen

Comments from Reviewable

Review status: 0 of 4 files reviewed at latest revision, 2 unresolved discussions, some commit checks broke.

stereotype.check.repository/category.xml, line 9 at r1 (raw file):

    You may obtain a copy of the License at

        http://www.apache.org/licenses/LICENSE-2.0

hier hat es glaube auch einen unnötigen change

Comments from Reviewable

Review status: 0 of 4 files reviewed at latest revision, 2 unresolved discussions, some commit checks broke.

stereotype.check.repository/category.xml, line 9 at r1 (raw file):
Previously, simon-weimann (Simon Weimann) wrote…
hier hat es glaube auch einen unnötigen change

Done.

stereotype.check.repository/pom.xml, line 3 at r1 (raw file):
Previously, simon-weimann (Simon Weimann) wrote…
hier hat es die formatierung zerhauen

Done.

Comments from Reviewable

Reviewed 2 of 4 files at r1, 2 of 2 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks broke.

Comments from Reviewable


THRIFT-4623: Create new instance of default value for mutable types

Client: py
Default values have a single instance what's fine for non-mutable types, but any change made for list, class, etc are propagated to all instances
>>> from gen.test import ttypes
>>> b1 = ttypes.B()
>>> b1
B(itm=A(value=None))
>>> b1.itm.value = 1
>>> b2 = ttypes.B()
>>> b2
B(itm=A(value=None))
>>> b1
B(itm=A(value=1))
>>>

With this change, generator will set None as default value for such params and add a check in __init__ method to create new instance when the value is None.
def __init__(self, itm=None,):
    if itm is None:
        itm = A(**{
        })
    self.itm = itm

It works for types defined in the same file, the only limitation is to create recurrence types with a default value of the same type because it generates inf recursion.
Apache Jira ticket: https://issues.apache.org/jira/browse/THRIFT-4623

[x] Did you create an Apache Jira ticket?  (not required for trivial changes)
[x] If a ticket exists: Does your pull request title follow the pattern "THRIFT-NNNN: describe my issue"?
[x] Did you squash your changes to a single commit?  (not required, but preferred)
[x] Did you do your best to avoid breaking changes?  If one was needed, did you label the Jira ticket with "Breaking-Change"?
[x] If your change does not involve any code, include [skip ci] anywhere in the commit message to free up build resources.

I removed the last element in thrift_spec tuples with default instance (it was used only in __init__ to check the param value, but it was always failing). I don't see any other place where it's still in use.


Max limit of poison resistance is wrong

Can only put 2 poison resistance in ffbeEquip and I have 3.
https://i.gyazo.com/b80c291150222751e9082a13f8452ef8.png

This has been corrected and can be closed.


Support simplified allocation schema in config

Make it possible to specify allocation schema without the "all" key.
That is, e.g. instead of:
:
l3Allocation:
all: 90%
mbAllocation:
all: [100%]
:
l3schema:
all: 0x1f
mbschema:
all: [100MBps, 50%]
one can simply specify:
:
l3Allocation: 90%
mbAllocation: [100%]
:
l3schema: 0x1f
mbschema: [100MBps, 50%]
Makes configuration easier/cleaner in the common use case where no
per-cacheId allocations are specified.

Codecov Report

Merging #9 (e1bc2c4) into main (2b83b62) will decrease coverage by 0.18%.
The diff coverage is 60.86%.

@@            Coverage Diff             @@
##             main       #9      +/-   ##
==========================================
- Coverage   50.97%   50.79%   -0.19%
==========================================
  Files           6        6
  Lines         871      880       +9
==========================================
+ Hits          444      447       +3
- Misses        325      331       +6
  Partials      102      102

Impacted Files
Coverage Δ

pkg/rdt/config.go
47.48% <60.86%> (-0.33%)
:arrow_down:

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 2b83b62...e1bc2c4. Read the comment docs.


Add users tests

This PR aims to add tests for the Profile model of the users app in order to ensure that it is working as expected. The tests cover the creation, deletion, updating, and string representation of a Profile object.
The changes made to the code include:

Adding a __str__ function to the Profile model to provide a string representation of the Profile object.
Changing the order of installed apps in the settings.py file.
Adding tests to the tests.py file for the Profile model using pytest and fixtures.

Saar orus<EMAIL_ADDRESS>Resolves #65

Hi Saar, please take a look at my comment from another PR about tests that doesn't test the app business logic https://github.com/redhat-beyond/FitMeet/pull/26#discussion_r1171429021


chore: Add Gitpod support

Hi, Pauline from Gitpod here. 👋🏼
Gitpod is an open-source developer platform automating the provisioning of ready-to-code developer environments. This will help folks wanting to contribute and fix bugs to get started instantly without the local dev! I see that you've got a similar sandbox environment which is fantastic, Gitpod does the same thing but with more features and functionality (e.g. docker support, prebuilds, a full laptop in each browser tab)
🪄 Try out the magic here to test this PR:

Happy to answer any questions! LMK. 🧡

Hey @pawlean. Looks interesting!
Two things:

Is the idea to only "host" the master branch, or is Gitpod made to be used with pull requests as well? If so, what would be the workflow — is there a pull-request Gitpod integration? If so, does it inject the links to Gitpod for given PR as a comment, or as a deployment timeline event similar to what Vercel does (well, Vercel does both: comments and then timeline events)? Ideally we'd not want more PR comments as it pollutes notifications.

As far as I understand it, each user that would use Gitpod to develop for Excalidraw would need to create an account and would be subject to the free-plan limits (4 concurrent workspaces, 50hrs/month). While they can apply for an OSS license, I assume it'd 1) not be granted to just any new OSS contributor, and 2) would create too much friction anyway.
Is it possible to have an OSS license for Excalidraw as a project, and have it extended to anyone who is actually working on a the Excalidraw Gitpod workspace? That way, the hours would not count towards their monthly limit, when working on Excalidraw.

Hi @dwelle!
I'm so sorry for the late reply -- this slipped completely from my radar.

Is the idea to only "host" the master branch, or is Gitpod made to be used with pull requests as well?

Gitpod can be used with PRs as well. This page might help explain the workflows better and this documentation.

As far as I understand it, each user that would use Gitpod to develop for Excalidraw would need to create an account and would be subject to the free-plan limits (4 concurrent workspaces, 50hrs/month). While they can apply for an OSS license, I assume it'd 1) not be granted to just any new OSS contributor, and 2) would create too much friction anyway.
Is it possible to have an OSS license for Excalidraw as a project, and have it extended to anyone who is actually working on a the Excalidraw Gitpod workspace? That way, the hours would not count towards their monthly limit, when working on Excalidraw.

We're happy to give core maintainers of Excalidraw an OSS license. Let me know and I'll get that set up for you.


[Desktop] Walk-In Client

Will be handled by the Branch Manager.

if this is done please close the issue. otherwise move it to the proper column thanks.

Almost Finalized GUI is complete. GUI change may happen when problems occur with consumption of the WS.


I couldn't open OneDrive location to download params manually. Can someone please help?Is the web location provided is correct?

Have you downloaded the params? I can't make it either.


[Bug][TCGC]: No compilation error on naming conflicts with clientName decorator

Describe the bug
When a model is renamed to a name conflicting with existing model, TCGC should throw compilation error
Reproduction
Playground
From the type graph, we can see TCGC only appends the clientName information for csharp, although it is conflicting with an exsiting model

Checklist

[X] Follow our Code of Conduct
[X] Check that this issue is about the Azure libraries for typespec. For bug in the typespec language or core libraries file it in the TypeSpec repo
[X] Check that there isn't already an issue that request the same bug to avoid creating a duplicate.
[X] The provided reproduction is a minimal reproducible example of the bug.

Since [clienName](https://github.com/Azure/typespec-azure/blob/54567f763257300e64603091659d282072631d2a/packages/typespec-client-generator-core/src/decorators.ts#L959-L964) can be  applied for language scope and type.
So, the name per language scope and type should be unique.
The naming validation should be after all clientName decorators applied, so that we have the overall naming map in place.


Merger breaks when Product is BinOp type

The initial value of the builder should be set to 1 instead of 0 when the merger's BinOpKind is * instead of +.

Fixed by #152


Add general menu for edit wand with general purpose functionalities

Triggered by pressing 'f' with the wand
Things to include:

Give instructions
Give blocks
Paste linked puzzle
Reset all puzzles
Help page of some sort?

I don't think we can create a puzzle with the menu wand?


Assets are not loading on iPhone simulator while testing with Flutter driver [flutter_driver]

When running UI tests with flutter driver the assets are not displayed on iPhone simulator. They are displayed when app is run in debug mode from VS Code.
I tried to reproduce it on the example project generated by flutter create -i swift -a kotlin --androidx --org com.example example but without success. I followed exactly the same process of introducing flutter_driver tests in both apps according to this article. Unfortunately, in my app the images are not shown, and in example app they appear just fine. I compared project.pbxproj and pubspec.yaml files and they look fine to me.
For instance my pubspec.yaml:
dev_dependencies:
  flutter_driver:
    sdk: flutter
  test: any
  flutter_test:
    sdk: flutter
  screenshots:

flutter:
  uses-material-design: true

  assets:
    - assets/lang/pl.json
    - assets/images/

I thought that maybe using 3.0x and 2.0x folders might cause this issue but it seems that it shouldn't affect me.
Is there anything that can help me with this issue? What can I do to debug the problem?
Logs
Beware, this is a very long log output.
flutter drive --target=test_driver/app.dart --verbose
https://gist.github.com/orestesgaolin/861c4f191773ca152b400d97ced2daeb
Flutter doctor
flutter doctor -v
[✓] Flutter (Channel stable, v1.7.8+hotfix.4, on Mac OS X 10.14.6 18G87, locale pl-PL)
    • Flutter version 1.7.8+hotfix.4 at /Users/dominik/Library/flutter
    • Framework revision 20e59316b8 (6 weeks ago), 2019-07-18 20:04:33 -0700
    • Engine revision fee001c93f
    • Dart version 2.4.0

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at /Users/dominik/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • ANDROID_SDK_ROOT = /Users/dominik/Library/Android/sdk
    • Java binary at: /Users/dominik/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/191.5791312/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 10.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 10.3, Build version 10G8
    • CocoaPods version 1.7.5

[✓] iOS tools - develop for iOS devices
    • ios-deploy 1.9.4

[✓] Android Studio (version 3.5)
    • Android Studio at /Users/dominik/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/191.5791312/Android Studio.app/Contents
    • Flutter plugin version 37.1.1
    • Dart plugin version 183.6270
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)

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

[✓] Connected device (1 available)
    • iPhone Xs Max • C1A5534A-AE3A-4FB3-95F6-993E351FF2D3 • ios • com.apple.CoreSimulator.SimRuntime.iOS-12-4 (simulator)

• No issues found!

It doesn't seem to be a flutter issue, at least there is no steps to reproduce it to act on it, so closing it.
Please try to find help on StackOverflow or other resources https://flutter.dev/community.

Finally I found the solution which I posted on StackOverflow.
I'll copy it here:
So the solution was to create separate main_test.dart file that wraps whole app in DefaultAssetBundle like so:
runApp(DefaultAssetBundle(
      bundle: TestAssetBundle(),
      child: App(),
    ));

and TestAssetBundle is like so:
class TestAssetBundle extends CachingAssetBundle {
  @override
  Future<String> loadString(String key, {bool cache = true}) async {
    final ByteData data = await load(key);
    if (data == null) throw FlutterError('Unable to load asset');
    return utf8.decode(data.buffer.asUint8List());
  }

  @override
  Future<ByteData> load(String key) async => rootBundle.load(key);
}

And in app.dart you have to reference correct file:
import 'package:flutter_driver/driver_extension.dart';
import 'package:playoo_league_player/main_test.dart' as app;

void main() {
  // This line enables the extension.
  enableFlutterDriverExtension();

  // Call the `main()` function of the app, or call `runApp` with
  // any widget you are interested in testing.
  app.main();
}

Full explanation can be found here<EMAIL_ADDRESS>

Cool, thank you @orestesgaolin  a lot! It's going to be helpful to the community.


Add  json-rcp-relay metrics

Description:

Adding the json-rpc-relay as "target" for metrics scrapping by prometheus
Adding the current dashboard with some modifications for Hedera Local Node for displaying the json-rpc-relay metrics

Related issue(s):
Fixes #324
Notes for reviewer:
Checklist

[ ] Documented (Code comments, README, etc.)
[ ] Tested (unit, integration, etc.)

Nice addition. I just have a few questions:

Is the static ip assignment for the relay necessary?  Something like this should work:

    static_configs:
      - targets: ['json-rpc-relay:7546']

Maybe the dashboard should be named something else, or moved to a different subdirectory. Grafana has a limitation where the name of the dashboard can't match the directory it's placed in.

Yes, I also think is not necessary, but @ebadiere encountered the issue on his local that was not able to reach it using the name as suggested json-rpc-relay and am trying to figure out why is it failing the CI tests so thought it might be a good chance to try that out, but yeah, on a previous commit, before I had it only with the name, and locally it worked. but fails on the CI.
Let me take a look at your other suggestion and make the appropriate change to the folder 👍  thanks!!
Also, might return it to use the name as suggested to simplify the amount of proposed changes, anyways it did not resolve my issue with the CI so better to rollback that and not assign a static IP to the relay.

@AlfredoG87 I've discovered an intermittent issue with local node that sometimes affects runs in the CI. It's been fixed via this PR #344.
It seems you are affected by the same issues. I would suggest rebasing on main to see if the issue is resolved for you.

Thank you @isavov, will try it later today 👍

@isavov Thank you, indeed that rebase fixed my issue, also simplified the changes to a minimum. 👍


chore: test-python-jenkins to 0.0.21

chore: Promote test-python-jenkins to version 0.0.21

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Kirstie's TO DO list - week starting 24th July 2017

Emails/Online

[ ] Arrange meeting with HQ re: Turing policy work
[x] Read around Turing & brain for panel event - deadline 24th July
[x] Accomodation & travel for ICON2017 - Amsterdam
[ ] Accomodation & travel for ECNP2017 - Paris
[ ] Send Ed info on Impact Story
[x] Info to OHBM student/postdoc SIG
[ ] Letters of recommendation for SRA application deadline 26th July
[x] Register for Cam Neural Networks & promote at Turing

NSPN

[ ] Run longitudinal pipeline on HPHI

[ ] Need to be able to actually run code on HPHI first! See #39 😕
[ ] Need to copy over the data from HPC

UPDATE 13/7: this is currently underway for UCHANGE directory

[ ] Also need to check that everything looks good

[ ] Write RA job description deadline 21st July
[ ] Update data management team on Cambridge repository

Blogs/interviews/writing

[ ] 250 words on challenges & solutions for open science for OHBM Conversation piece deadline 25th July
[ ] Edit story engine for Mozilla
[ ] Blog telling people to apply for OpenCon, Moz Open Leadership & MozFest

Invited talks

[ ] Bio/abstract for talk at EPFL library
[ ] Abstract & visa information for ICHBD
[ ] Make talk for ICON (and online brain intensive) - See #65

Fragile Families

[ ] Send email about meeting on Wednesday

Didn't do 🙄 ...but could do for next week!

BrainAge

[ ] Get code installed & running
[ ] Send questions to FL

OpenAdvice

[ ] Migrate to organisation ProjectOpenAdvice & build website there: https://projectopenadvice.github.io/
[ ] Merge jekyll pull request

OpenConCam

[x] Summary email from OpenConCam meeting
[ ] Add speakers to thread OpenConCam/issues/16
[ ] Add budget info to thread OpenConCam/issues/16
[ ] Move speakers to google doc & email around

Closing with BILLIONS OF THINGS NOT DONE! GAAAAAH 😭