Vision Angle / "Hearing" dead creatures

When you play with a vision angle that is not 360° and a token looks away from a as dead marked creature they can still "hear" them.

I assume what you asking is unrelated to the vision angle: you just asking for hearing to not detect dead tokens, correct?

Well if i think about it, yes it is unrelatetd to the vision angle, and yes not hearing dead tokens would be a option?

to be correct - not hearing anything in D&D5 would be nice ... it is not an rule ... and is a very situative option if an creature is dedected by hearing (and then nearly all times only if he moves and not if he standing still .....) - so i dont need "hearing" at all in D&D5 ....

Maybe have a read of this article. It’s a pretty good summary of how invisibility actually works: https://www.belloflostsouls.net/2022/01/dd-invisibility-doesnt-work-the-way-you-think-it-does.html

I dont need an article how the autor think invisibility should work - sorry - but in the official 5e rules nothing is say from "hearing" ....
And even if the hearing would be implemented carefull - where is not working anyway as it is modifyed from so many things the engine could never understand > is the char moving, how fast, how is his closing modify this, how is the underground, how is the overall surounding noise level ... and so one ... < - i simply dont need an dedection mode i cant control as DM - then this mod is an dead end for me.

If you don't want any hearing at all, you can just go and disable the hearing of the tokens in their detection, just set the hearing radius to 0ft and you don't have hearing anymore.
But in general this is not about invisibility or hearing in general. It's about "hearing" a creature which is dead on the floor not doing anything, not able to do anything, and showing up on the screen cluttering the view.
The makeshift solution that we came up in last nights session was to just whenever a creature is dead, the DM also gave the creature the status effect of "Inaudible" that solved the problem somewhat.
We tried to get a macro working to be able to set the creature dead and Inaudible at the same time, but we didn't got it to work properly.
A easy solution to this issue would be a macro. If someone could help me with that, would be highly appreciated.

Do you use Dfred’s CE? You could customise the Dead status effect to also apply Inaudible statusEffect. Just tested and it works great.

It looks i get many solutions (i dont like to say useles - but this i feel...) - but the ground problem dont get away ...
I DONT need or WANT any hearing option in my D&D5 game - simply ....
I dont want to get this, and then i musst disable it on all chars or use macros .... how stupid is this, first to install an mod and then manual disable on many tokens the unwanted part - better not install the mod > simply solution ...
I have used Perfect Vision long time - a good mod, but if i get things i dont like (and it is not global possible to disable it simply in the mod settings) - then i am done with this mod - have fun, but not for me

Added a setting to configure the default range of vision.
Hearing and Tremorsense are no longer able to detect dead (defeated) tokens.

https://github.com/dev7355608/vision-5e/releases/tag/v1.4.0


[BUG] Being in results screen for long enough freezes the game

Describe the bug
If after leaving the results screen for too long in idle, after clicking the "<" Back button it will freeze for a few seconds and display incorrect information, while softlocking you without being able to escape.
To Reproduce
Steps to reproduce the behavior:

Play a song (Tested on Untitled Chords Thing Vocals)
Finish as bot
Wait a few minutes
Try to leave the screen with the back button
Unable to

Expected behavior
Being thrown into main menu
Screenshots

System Info (please complete the following information):

OS: Linux Mint Cinnamon 22 64 bit
Version: 6.2.9

Additional context
Commit ccbec701be1430da9eae660b0609e05285854485
https://github.com/Encore-Developers/Encore/actions/runs/10859419729 - One of the last linux building runs

Update: I am completely unable to escape the results screen. Clicking the Back button makes the texts say nan and it softlocks (no freeze because background is moving)


fix: restore elements and app state in updateScene

fixes https://github.com/excalidraw/excalidraw/issues/4581
Earlier

Now

I checked, and realized this is kinda a hot path, not just for excalidraw.com, but for host apps as well. For example, we call updateScene in collaboration on every update, and pass in the whole scene elements. Restoring them unnecessarily can quickly become a bottleneck (excalidraw collaboration syncing is already super heavy and congests easily as it is).

We can make this optional, but I'd rather avoid this — it wouldn't really solve the issue.

Second option is to optimize the restoring calls.
This would likely consist of versioning of some kind so we can bail if the version is up to date.

Third option is to call restore manually before passing data to updateScene, in places where the data comes from external boundaries where we're not reasonably certain they are already restored.

@zsviczian @ad1992

I think best would be to come up with (2) to optimise restore calls, (3) is upto the host if they want to restore they always can and we can recommend it in readme as well to make sure migrations work properly

I think best would be to come up with (2) to optimise restore calls, (3) is upto the host if they want to restore they always can and we can recommend it in readme as well to make sure migrations work properly

Well, if we wanna put the responsibility onto hosts, then there's not (that much) need for (2).

thanks a lot @ad1992 @dwelle


Add key prefix as a configuation option

It's not uncommon to have single s3 bucket for multiple applications, connected to a single cloudfront domain, separated just by a key prefix, example:
static.example.com/app1/assets...
static.example.com/app2/assets...

ATM there's no easy way to implement this with this very nice package
thanks!

@bubenkoff thanks for raising that point. Can you please suggest a suitable solution? I haven't faced that issue before so any details you can share are much appreciated!

I'm actually facing the same sort of problem on an S3/CloudFront environment as described by @bubenkoff , where I have just one bucket, hosting multiple apps' assets. At the moment I use just one app on the CDN, so I'm good having a public folder on the bucket's root directory, but this could potentially display the package issue if I decide to use the package on the other apps since, in a Laravel environment, you most likely always push content from the public directory, potentially overwriting every apps assets.
A possible solution could be having a config variable in config/cdn.php containing the additional folder/path (defaulting to an empty string) e.g. 'app-folder' => '',, which will then get pushed to the AwsS3Provider'sdefault array.
Then, considering the way you upload the assets (get all local's, compare with server's, remove the not modified from the array), and you are using the AWS PHP SDK putObject method, where the "destination" path is held in the Key key, we will need to prefix the Key key wth app-folder if the string is not empty and the file is brand new (not available on server).
Considering this there will most likely be a bit of refactoring in the AwsS3Provider's upload and getFilesAlreadyOnBucket methods as follows:

In the getFilesAlreadyOnBucket method add a line after 401: $item->IsModified = true; this way we know if the file has actually been modified instead of just created locally;
In the upload method change line 207 to: 'Key' => ($this->default['providers']['aws']['s3']['app-folder'] !== '' && $file->IsModified !== null ? str_replace('\\', '/', $this->default['providers']['aws']['s3']['app-folder'].'/'.$file->getPathName()), : str_replace('\\', '/', $file->getPathName()). This way we'll add app-folder as a prefix to the the actual local app path, so the file gets uploaded where we want. You might not want to expose the variable itself though, and access it from a getter e.g. getAppFolder, so you could apply string transformation on it, as necessary (trim('/') off the top of my head).

I fear though that these edits could "break stuff" (very technical term), as you are most likely not allowed to add a custom key to $item at point 1, but is roughly my idea.
Apolgies if I've written something outrageous or haven't formatted the answer properly, but I'm very inexperienced with GitHub's issue tracker.
What do you think?

@thtg88 Thanks for the details. I'm just afraid that would require too much config in the simple case where you only have one bucket per app. I always lean towards simplicity when it comes to config.
I was thinking whether Bucket in config can be bucket-name/app1, that should work IMO. Can you please give it a try?

@Mulkave Forgot to mention I tried that, unfortunately it throws an exception as the forward-slash '/' character gets urlencoded. I'm not at my PC right now but I can provide the Exception details if needed.
thanks for your reply anyway

@thtg88 sure some more details on the exception would help. I'm also not able to try that atm.

Hi @Mulkave , apologies for the delay. You can find the exception message below:
[2016-11-19 14:38:49] local.ERROR: exception 'Aws\S3\Exception\S3Exception' with message 'Error executing "ListObjects" on "https://s3.amazonaws.com/the-s3-bucket-name%2Fthe-application-assets-subfolder-name?encoding-type=url"; AWS HTTP error: Client error: `GET https://s3.amazonaws.com/the-s3-bucket-name%2Fthe-application-assets-subfolder-name?encoding-type=url` resulted in a `403 Forbidden` response:
<?xml version="1.0" encoding="UTF-8"?>
<Error>SignatureDoesNotMatch<Message>The request signature we calcul (truncated...)
 SignatureDoesNotMatch (client): The request signature we calculated does not match the signature you provided. Check your key and signing method. - <?xml version="1.0" encoding="UTF-8"?>
<Error>SignatureDoesNotMatch<Message>The request signature we calculated does not match the signature you provided. Check your key and signing method.</Message><AWSAccessKeyId>AKIAIY7T2HVCCV34ZZVA</AWSAccessKeyId><StringToSign>AWS4-HMAC-SHA256
20161119T143848Z
20161119/us-east-1/s3/aws4_request
thestringtosign-omitted</StringToSign><SignatureProvided>thsignatureprovided-omitted</SignatureProvided><StringToSignBytes>thestringtosignbytes-omitted</StringToSignBytes><CanonicalRequest>GET
/the-s3-bucket-name/the-application-assets-subfolder-name
encoding-type=url
aws-sdk-invocation-id:60f4df8826b54f3ef8e7a99eb6361085
aws-sdk-retry:0/0
host:s3.amazonaws.com
x-amz-content-sha256:e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855
x-amz-date:20161119T143848Z

aws-sdk-invocation-id;aws-sdk-retry;host;x-amz-content-sha256;x-amz-date
thecanonicalrequest-omitted</CanonicalRequest><CanonicalRequestBytes>thecanonicalrequestbytes-omitted</CanonicalRequestBytes><RequestId>032C69243B67DFE1</RequestId><HostId>thehostid-omitted</HostId></Error>'

exception 'GuzzleHttp\Exception\ClientException' with message 'Client error: `GET https://s3.amazonaws.com/the-s3-bucket-name%2Fthe-application-assets-subfolder-name?encoding-type=url` resulted in a `403 Forbidden` response:
<?xml version="1.0" encoding="UTF-8"?>
<Error>SignatureDoesNotMatch<Message>The request signature we calcul (truncated...)
' in C:\Application\vendor\guzzlehttp\guzzle\src\Exception\RequestException.php:111

Should you need any other details, please let me know.
Thanks!

@Mulkave, Very actual problem! Maybe I propose PR for this?

I'm currently running into this issue. It would be good to have this feature available. The main reason is we have different branches being staged that need different CDN assets.


Minor grammatical fixes

Just adding fixes I found reading through the design doc the first time.

@rothgar thanks for your PR. This project uses DCO. Can you please do the git commit -a -s --amend dance.

I think that worked.


Break DOM structure by create and remove floatThead multiple times

I tried this at the home page by using the "Demo" section.
After running "Run" and "Remove" several times , the document seems totally broken.
The browser I used is Chrome  44.0.2403.157
Thanks.

Yep, looks like I busted something.  Will fix today


Allow for handling non-supabase related deep link /applink urls

Is your feature request related to a problem? Please describe.
Currently, the way that supabase handles app links, it automatically figures out if an incoming link is related to an auth changes, and updates the auth state accordingly, which is great. However, I have a need to also handle other kinds of links in my app, and doing so with uni_links or uni_links_desktop breaks the supabase url handling flow, where uni_links_desktop will catch the url instead of supabase, and auth state is no longer updated. I'd like to be able to handle the following:
supabase auth links, like signing in via a magic link email
supabase reset-password link: com.mycompany.myapp://reset-password < supabase currently does a reset password workflow, but when this link is clicked from a user's email, Supabase-flutter signs the user in, and doesn't expose the url to the app, as far as I can tell.
custom url scheme: com.mycompany.myapp://share/some-share-id
Describe the solution you'd like
If supabase could expose a stream to the uriLinkStream, or expose a stream that allows us to handle links that supabase doesn't do anything with, that would be ideal. AFAICT, in supabase_auth._handleDeepLink(), after checking _isAuthCallbackDeeplink(uri) if the link isn't a supabase auth link, it simply returns without doing anything. If instead, it could emit the url to another stream, or just emit all urls received to a stream (to handle the reset-password case mentioned above), that would be great.
Describe alternatives you've considered
I initially started trying to handle the deepLinks myself, and then using Supabase.instance.client.auth.getSessionFromUrl(uri), but linking on desktop vs mobile is different enough that I encountered unexpected problems. Primarily, on macOS, you can preempt the deep link handler that supabase uses with uni_links_desktop, but on android/iOS, I wasn't able to get preemption working. I'm sure I could get this working eventually, but I feel like the solution would be fairly fragile, since uni_links_desktop is beta, and I'm not sure if preemption would be stable on different versions of the same OS.
Additional context
Love y'alls product, it's been a blast working with supabase!

So you listened to the linkStream, but it didn't emit any events? Did the initial link method return null as well?

Correct, if I set up my own instance of AppLinks, and listen to the uriLinkStream, no events are received. As well, getInitialAppLink returns null, but that's expected since these are links that come in when the app has been running already.
Supabase still handles urls properly in this case.

I thought you are using uni_links? The method you mentioned now are from app_links.

Correct, I did try uni_links, but due to the beta status, and different handling on different platforms (uni_links_desktop is a different package from uni_links), I decided to try using AppLinks instead, since I believe it's using a broadcast stream, and should allow multiple listeners. Trying that, however, also didn't work.

Where are you listening to the stream? I think it has to be after runApp(). So best somewhere in your widget tree. I have no issues to add my own listener that way.

In one of my pages, which is built after runApp is called. I've tried working on this a few different ways, but if it's just a problem in my app, then I suppose it's not really a feature to request for the library.

Closing with this PR merged
https://github.com/supabase/supabase-flutter/pull/848

Reference: https://github.com/supabase/supabase-flutter/pull/605#issuecomment-2267171726
I tried to solve it here https://github.com/llfbandit/app_links/pull/145, let's hope it gets merged

It's now available in 6.2.0 🎉


We defiantly need more functions!

Functions such as:

/trololo
/42

just to name some.

Please give more information about what functions you want.

There could be an option to print e.g. a rofl-copter.

On 12.06.2016, at 21:30, Manuel Bäurle<EMAIL_ADDRESS>wrote:
Please give more information about what functions you want.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/Zaratropia/ma-mensa-bot/issues/4#issuecomment-225455683, or mute the thread https://github.com/notifications/unsubscribe/AH18FOc9b4iBeLBI0UgrUjHay1rlyxZ3ks5qLF5ggaJpZM4Iz1AK.

useful for hs students would be a connection to the wiki of the faculty: "schwarzes brett”.

On 12.06.2016, at 21:30, Manuel Bäurle<EMAIL_ADDRESS>wrote:
Please give more information about what functions you want.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/Zaratropia/ma-mensa-bot/issues/4#issuecomment-225455683, or mute the thread https://github.com/notifications/unsubscribe/AH18FOc9b4iBeLBI0UgrUjHay1rlyxZ3ks5qLF5ggaJpZM4Iz1AK.

Rofl-copter feature is worth a try.
The "Schwarzes Brett" feature is out of scope for this bot and rejected.


MAINTAINER NEEDED

I no longer have time to maintain this repository and no longer work with Angular in my day-to-day, so it makes sense to hand this off to someone with a vested interest.
Please let me know if you want to take this on
cc: @steffenmllr

Let me know if I can participate and maintain

@rahulmr definitely! What is your NPM username? Let me know and I can get the process started

Yeah, glad this is so recent. I want to continue using but it doesn't work unless protractor v5 is being used. Thats all I need and then I can keep using this module

@mattfritz it is rahulmr (@rahulmr)

@rahulmr you should be good to go. Thank you for taking this on!

Just wanted to thank @mattfritz for his work and @rahulmr to continue this project. It nice to see people wanting to contribute. We also don't use angular anymore which was the original reason I created the repo.
@rahulmr Maybe you could ask the angular team to move the repo to the angular project. I'm happy to transfer the repo if they would do it. A lot of people are using this plugin (https://npm-stat.com/charts.html?package=gulp-protractor)

@hah-nan could you please check and report if there are any issues at your end after update to protractor ^5?
@steffenmllr thanks for your appreciation. I am bit new :) with npm and github. I will try to connect with them.  Any pointers will be helpful.


ssh or shell access to cloud run

This does not exist.
In the future you’ll be able to run stuff like gotty.

@ahmetb it worths documenting that officially this does not exist, but there are ways with reverse shell
so get Cloud Run instance to connect to a GCE instance for example on launch and this way a tunnel is happening.

Such methods are currently quite hacky. Cloud Run throttles cpu to ~0% when there are no request. So such tunnels would not work easily.

I am well aware of that, but you can ping and keep the cluster online, also think of web socket is it throttled under them as well?

What do you mean by "cluster" here?
WebSockets are active requests, so not throttled. But as you can see WS isn't supported currently.

Cluster was a wrong word. About WS we will get back to that later.
Anyway a lot of people are chasing and one of the most asked and comment things on Cloud Run are how to debug with shell access.
It would be good so Cloud Run engineers should provide a debug mode, and while that is active

they don't throttle CPU
they keep the instance up

Anyway a lot of people are chasing and one of the most asked and comment things on Cloud Run are how to debug with shell access.

As someone who monitors ~all Cloud Run questions asked on Twitter and Stack Overflow and here, I would not say "a lot of people". Also this repo isn't for discussing feature requests. :)


Re-filled watched inbox gives error

An inbox that has been emptied and re-filled gives an error that smells like there is some unexpected XML or HTML sent to a RDF parser.
Steps to reproduce:

Watch an empty inbox
Add a resource to the inbox
Delete this resource
Add another resource

If this error still occurs, please reopen this issue.


What data does the onSelectedChange of the SelectPanel component require in TypeScript format?

I reviewed the usage guide for components on your latest documentation page （https://primer.style/components/selectpanel/react） and found that the introduction to component properties is not comprehensive. Some properties that can be set are not mentioned in the documentation (this is the case for many components, such as the ability to use the sx property, which is not included in the property descriptions). Additionally, there is a lack of detailed information on the data types and formats used for component properties. This often leaves me uncertain about what data to pass and the format it should have, leading to frequent guesswork and trial-and-error. In some cases, it's challenging to determine these details, especially when developing with TypeScript. For instance, I encountered difficulties when using the SelectPanel component in TypeScript this time.
I followed the example code for the SelectPanel component but encountered issues as the example code is incomplete. Some libraries and methods used in the example are not included, making it difficult for readers to construct these methods on their own. I suggest that the documentation be as comprehensive and detailed as possible, including the necessary import statements for libraries and methods in the example code. For instance, import statements like import {TriangleDownIcon} from '@primer/octicons-react' should be included in the example code.
Furthermore, it would be beneficial if the documentation uses the latest versions of development libraries. For example, in the latest version of React.js, there is no need to explicitly import and use React, and useState() can be used directly without React.useState(). However, the example code still uses the old version, causing errors for less experienced software engineers who might spend time searching for solutions. Keeping the documentation up-to-date with the latest library versions would help avoid these issues.
When using the SelectPanel example code from the new documentation（https://primer.style/components/selectpanel/react）, the first error I encountered was the direct usage of an undeclared items in the example code. I was unsure how to construct this items. Later, I referred to the old documentation（https://primer.style/react/SelectPanel）, which included the declaration for items. However, I encountered another unresolved issue. The example code contains an undeclared data type ItemInput, and I'm uncertain about how to construct this ItemInput. I am using TypeScript (I tested with JavaScript, and it ran successfully, but TypeScript requires type descriptions, which I couldn't provide since I don't know the data type structure of ItemInput).
Many of the previous points were suggestions. I have resolved many issues and reached this point. However, there is still one unresolved problem. The question is: What is the TypeScript type declaration for ItemInput (specifically, the type declaration for the leadingVisual value type)?

@meiyingqishi I'd be glad if you assign me this task.

Hi @meiyingqishi 👋 Thank you for your thoughtful feedback on our documentation. We have an internal issue that collects feedback and I am going to link this issue to it. Also cc'ing our PM @dipree.
@Adityabandaru18 thanks for your willingness to work on this issue. This is more of a feedback than a bug that requires our team to go through it first. Watch this space to pick up any good first issues and we will be happy to help you with it!

Hello @meiyingqishi again, sorry I missed that you asked a question at the end.

What is the TypeScript type declaration for ItemInput (specifically, the type declaration for the leadingVisual value type)?

it should be React.ElementType https://github.com/primer/react/blob/bs/iconbutton-tooltip/src/deprecated/ActionList/Item.tsx#L46
Let us know if you have further questions 🙌

@meiyingqishi thank you for the feedback. It's worth noting that we are working on a new version of the SelectPanel and that we would love to improve the documentation to make it more useful.

Think you for your reply @broccolinisoup.
I still haven't resolved this issue. May I ask how to import ItemInput in TypeScript project?
I tried to build the ItemInput data type, but I still couldn't succeed:
interface ItemInput {
    leadingVisual: React.ElementType;
    text: string;
    id: number;
}

Here is the complete code：
import {
  useState
} from "react";

import {
  BaseStyles,
  Box,
  Button,
  SelectPanel,
  ThemeProvider
} from "@primer/react";

import {
  TriangleDownIcon
 } from "@primer/octicons-react";

import<EMAIL_ADDRESS>
export default function App() {

  // I don't know how to import "ItemInput" and build it myself.
  interface ItemInput {
    leadingVisual: React.ElementType;
    text: string;
    id: number;
  }

  function getColorCircle(color: string) {
    return () => (
      <Box
        borderWidth="1px"
        borderStyle="solid"
        bg={color}
        borderColor={color}
        width={14}
        height={14}
        borderRadius={10}
        margin="auto"
      />
    )
  }

  const items = [
    {leadingVisual: getColorCircle('#a2eeef'), text: 'enhancement', id: 1},
    {leadingVisual: getColorCircle('#d73a4a'), text: 'bug', id: 2},
    {leadingVisual: getColorCircle('#0cf478'), text: 'good first issue', id: 3},
    {leadingVisual: getColorCircle('#ffd78e'), text: 'design', id: 4},
    {leadingVisual: getColorCircle('#ff0000'), text: 'blocker', id: 5},
    {leadingVisual: getColorCircle('#a4f287'), text: 'backend', id: 6},
    {leadingVisual: getColorCircle('#8dc6fc'), text: 'frontend', id: 7},
  ]

  const [selected, setSelected] = useState<ItemInput[]>([items[0], items[1]])
  const [filter, setFilter] = useState('')
  const filteredItems = items.filter(item => item.text.toLowerCase().startsWith(filter.toLowerCase()))
  const [open, setOpen] = useState(false)

  return (
    <ThemeProvider>
      <BaseStyles>
        <main className="d-flex flex-justify-center flex-items-center flex-column">

          <SelectPanel
            renderAnchor={({children, 'aria-labelledby': ariaLabelledBy, ...anchorProps}) => (
              <Button className="mt-3" trailingAction={TriangleDownIcon} aria-labelledby={` ${ariaLabelledBy}`} {...anchorProps}>
                {children || 'Select Labels'}
              </Button>
            )}
            placeholderText="Filter Labels"
            open={open}
            onOpenChange={setOpen}
            items={filteredItems}
            selected={selected}
            onSelectedChange={setSelected}
            onFilterChange={setFilter}
            showItemDividers={true}
            overlayProps={{width: 'small', height: 'xsmall'}}
          />

        </main>
      </BaseStyles>
    </ThemeProvider>
  )
}

Screenshot of code error prompt：

Screenshot of abnormal interface display (the first two options should be checked by default)：

Screenshot of the abnormal interface display (the check box will not be selected by default. After clicking the option again, the text on the button will not be canceled, but duplicate items will be added continuously)：

I have solved the last question I asked (What is the TypeScript type declaration for ItemInput (specifically, the type declaration for the leadingVisual value type) by following the solution replied by the author.
Declare the ItemInput type：
interface ItemInput {
  leadingVisual: React.ElementType;
  text: string;
  id: number;
}

Create getColorCircle function：
function getColorCircle(color: string) {
  return () => (
    <Box
      borderWidth="1px"
      borderStyle="solid"
      bg={color}
      borderColor={color}
      width={14}
      height={14}
      borderRadius={10}
      margin="auto"
    />
  )
}

Construct items data：
const items = [
  {leadingVisual: getColorCircle('#a2eeef'), text: 'enhancement', id: 1},
  {leadingVisual: getColorCircle('#d73a4a'), text: 'bug', id: 2},
  {leadingVisual: getColorCircle('#0cf478'), text: 'good first issue', id: 3},
  {leadingVisual: getColorCircle('#ffd78e'), text: 'design', id: 4},
  {leadingVisual: getColorCircle('#ff0000'), text: 'blocker', id: 5},
  {leadingVisual: getColorCircle('#a4f287'), text: 'backend', id: 6},
  {leadingVisual: getColorCircle('#8dc6fc'), text: 'frontend', id: 7},
]

And move the ItemInput, getColorCircle, and items that I originally placed in the export default function App() outside the App function.


Update to Swift 4.2

What's in this pull request?

Require Swift 4.2 to build the project
Update to the 4.2 package format
Remove deprecated constructs

Breaks my heart to see pods break my build again
https://github.com/CocoaPods/CocoaPods/issues/7708
Disabling pods support until 1.6.0 is released to the public.

⛵


Bug: Phantom proposal on proposal list load

Overview

When doing a hard reload on this page, a “phantom proposal” record appears for a second or two which is then replaced with the correct data (either the empty message, or actual proposals). The “phantom proposal” seems to include DAO-specific and somewhat relevant data which is interesting to me.

#0xc20b...9882 Proposal to execute 7 transactions 7 addresses that will Send 60,000,000 DCNT To 0x0000...0000

Steps to reproduce
n/a
Screenshots / Video
No response
Version
Dev (app.dev.fractalframework.xyz/)
Browser
Chrome (or Chromium based)
Operating System
None
web3 wallet
None

This is feedback from Adam.  @mudrila is this the module transaction from when the DAO was created? Any idea what's going on with this?

@mudrila
this is with the original DAO address that I used when making my initial report: 0xb6b5dEE6427dC1FE63E78B44661f21B542Fa8F3C
https://user-images.githubusercontent.com/706929/214857143-a580fbc7-82b7-43fe-b2ad-5d6566964bd6.mov

I can't reproduce this, so likely a refactor fixed this one @mudrila
Closing this.


Unbreak snappy and tensorflow Haskell packages

Description of changes
This unbreaks the Haskell TensorFlow bindings, and - as it's a dependency - the snappy package.
This obsoletes #119411.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[X] x86_64-darwin
[ ] aarch64-darwin

[X] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[X] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[X] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Sorry, wrong base branch, will try again momentarily.


Add rubocop to travis

Add rubocop to travis and manually disable two cops, which weren't automatically excluded.

@nodo you can feel free to use the cucumber org repo for development work now if you want to - no need to use your own fork.

Thanks @mattwynne :) will do it!


ClassTag instances can cause runtime exceptions

ClassTags are used by scalac during type-based pattern matching. This is a problem for newtypes (but not newsubtypes) since their ClassTag runtime class is Object. As such, we are permitting users via our "safe" API to do unsafe things without it being obvious that what they are doing will definitely result in a runtime exception.
scala> import io.estatico.newtype.macros._, io.estatico.newtype.ops._

scala> @newtype case class Foo(x: String)

scala> (1: Any) match { case x: Foo => x ; case _ => Foo("nope") }
res0: Foo = 1

scala> res0.coerce[String]
java.lang.ClassCastException: java.lang.Integer cannot be cast to java.lang.String
  ... 38 elided

I think the solution for now is to remove ClassTag instances altogether. The only reason I think they're useful is for constructing Arrays, but I think this is the wrong solution. I have an experimental (and incomplete) branch for a different way of handling Arrays here (via an AsArray type class). I'm looking to try to use this to resolve #10 in a type safe way.
In the meantime, I think it's best to remove the ClassTag instances to prevent unsafe code from happening by accident.
Also see my original point about this with regard to the unapply method here.

Sounds good :+1:


Removed the Typography extension

Fixes #236

Removed the Typography extension. It was responsible for converting double quotes to a smart font.
It is also possible to disable specific rules in the Typography extension: https://tiptap.dev/api/extensions/typography. But converting to a smart font seems unnecessary.

@amaldinesh7 _a can you please take a look?

There are some useful typography conversions in the package. I suggest turning off the ones that are not required.
cc: @labeebklatif

@amaldinesh7 _a please have a look.

We can enable the left and right arrows @AbhayVAshokan _a . Rest looks good to me. Proceed with merging after making the change.


Fix typos and replace direct writing to console with proper logging

master only.

Integrated, thanks Adrian!


Value change doesn't trigger change in konb's position

I have value in state:
this.state = {
  value: 50,
};

Slider usage:
          const { value } = this.state;

          <CircleSlider
              value={value}
              onChange={this.handleChange}
            />

When I call this.setState({ value: _a_value_ }) but the knob position never change.

@harry7cao  Thank you for the feedback. Package updated, try it now please.


Increase brightness when using ls100 video lighting?

Describe the problem
Hi, I was wondering if it is possible to increase the overall brightness of the LEDs when using the ambient backlight effect. The leds seem really dim compared to when directly controlled. Is there a way to apply some kind of brightness scaling to the leds before each update, or to set a minimum brightness perhaps? Thank you.
System (please complete the following information):

OS: Windows 10
Board: Arduino Leonardo
IDE: Arduino IDE
Version: 1.8.33
Sketch: AmbientBacklight

Code changes
Did you changed the code of the library or the sketch?
Added CLP::scale function to onUpdateHook

By default all iCUE software playback effects have a maximum brightness of 50% only, which can't be changed in iCUE. You can change this in the onUpdateHook by multiplying all RGB values by 2 before applying gammaCorrection.
This workaround only works in Software Playback mode, in HW the default brightness is 100% and then all colors are broken because of the scaling.

Could you give me an example for the code I would use to multiply the rgb values? Sorry I'm really new to all this

I can do this later today

Okay awesome, thank you

@tchesket Have a look at the new fixIcueBrightness function https://github.com/Legion2/CorsairLightingProtocol/pull/143/files#diff-df06cb0ecab891d52d16b3deb8423144R100

This seems to have worked, thanks for doing that :)
On Thu, Jun 4, 2020, 13:07 Leon Kiefer<EMAIL_ADDRESS>wrote:

@tchesket https://github.com/tchesket Have a look at the new
fixIcueBrightness function
https://github.com/Legion2/CorsairLightingProtocol/pull/143/files#diff-df06cb0ecab891d52d16b3deb8423144R100
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Legion2/CorsairLightingProtocol/issues/142#issuecomment-639016805,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJSHH4IIGKSRVRSEPQ4OBGTRU7PFXANCNFSM4NSHS6XQ
.

Discard that, download latest version and use fixIcueBrightness function
ledController.onUpdateHook(0, [](){
CLP::fixIcueBrightness(&ledController, 0);
});

Oh okay, that's what I'm already using
On Sun, Jun 7, 2020, 19:05 Sanbay<EMAIL_ADDRESS>wrote:

Discard that, download latest version and use fixIcueBrightness function
ledController.onUpdateHook(0, {
CLP::fixIcueBrightness(&ledController, 0);
});
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Legion2/CorsairLightingProtocol/issues/142#issuecomment-640300098,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJSHH4I7CX37JYEU5KFFPP3RVQTK3ANCNFSM4NSHS6XQ
.


[Automated] Update actions

Produced via:
# meta: knative/.github
# main: google/ko
cp $(find "${GITHUB_WORKSPACE}/meta/workflow-templates" -type f -name '*.yaml') \
  "${GITHUB_WORKSPACE}/main/.github/workflows/"

Cron. -knative-prow-robot
/cc @jonjohnsonjr @ImJasonH @mattmoor
/assign @jonjohnsonjr @ImJasonH @mattmoor

Codecov Report

Merging #329 (e7e38b4) into main (ee02868) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #329   +/-   ##
=======================================
  Coverage   38.24%   38.24%
=======================================
  Files          33       33
  Lines        1527     1527
=======================================
  Hits          584      584
  Misses        849      849
  Partials       94       94

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 ee02868...e7e38b4. Read the comment docs.


Implement packdiff

This is basically #2 with squashed commits.

Add headers to files. Add a NOTICE file.


TODO habits are being counted as "green"

First of all, really like how user-friendly your plugin is, and love the visualization! I haven't been able to use it successfully, because it has always marked my TODO items as completed. Haven't done anything special in the settings.
Here are my settings:

All the days that are green there are actually still in TODO and not marked DONE (so they shouldn't be green in the habit tracker):

The days that are red are actually a separate issue where LogSeq isn't putting my journal template in some days that I manually create the journal page for. You can ignore that.
I'm on the latest version of your plugin and LogSeq 0.8.0.

Thanks, @mikob.
It seems the ignore pattern does not match entries where time is tracked (i.e. 0s on the right), due to newline characters. The plugin uses :block/content to match habits, you can observe LOGBOOK inserted after \n to track time by clicking (Dev) Show block data if developer mode has been enabled.
In the meantime, you can change your ignorePattern to TODO (.|\n)*.
I'll provide either a fix or a more sensible default for the ignore pattern in the next version.
Cheers

@c6p nice catch, yes that fixes it!


need more validation of OpImageRead result type

Vulkan requires the result to be a 4-element vector. See here

The Result Type operand of OpImageRead must be a vector of four components.

OpenCL (see Data format for reading and writing images and (https://www.khronos.org/registry/OpenCL/specs/3.0-unified/html/OpenCL_C.html#image-read-and-write-functions )

must be scalar float for depth image
must be 4-element vector otherwise

WebGPU SPIR-V env spec will follow Vulkan.
First attempt at implementing this is #4072

However, Glslang's HLSL compilation flow can produce OpImageRead with result types that are scalar or vectors of the wrong length. See https://source.cloud.google.com/results/invocations/13a8342c-84ba-4683-8ce4-ec07d4033716/log

The failures are only in the Glslang HLSL path.
[  PASSED  ] 1451 tests.
[  FAILED  ] 4 tests, listed below:
[  FAILED  ] ToSpirv/HlslCompileTest.FromFile/hlsl_rw_register_frag, where GetParam() = 16-byte object <CC-16 01-9A 2D-56 00-00 23-EF 02-9A 2D-56 00-00>
[  FAILED  ] ToSpirv/HlslCompileTest.FromFile/hlsl_rw_scalar_bracket_frag, where GetParam() = 16-byte object <E2-16 01-9A 2D-56 00-00 23-EF 02-9A 2D-56 00-00>
[  FAILED  ] ToSpirv/HlslCompileTest.FromFile/hlsl_rw_vec2_bracket_frag, where GetParam() = 16-byte object <13-17 01-9A 2D-56 00-00 23-EF 02-9A 2D-56 00-00>
[  FAILED  ] Hlsl/HlslIoMap.FromFile/spv_rw_autoassign_frag, where GetParam() = 40-byte object <07-3B 01-9A 2D-56 00-00 23-EF 02-9A 2D-56 00-00 05-00 00-00 0A-00 00-00 14-00 00-00 0F-00 00-00 1E-00 00-00 01-01 00-00>

This was fixed by #4072


Bug in dataset.py?

Hi, thanks for your nice code!
In line 53 of dataset.py, I think the code should be
"
if nFrames%2 == 0:
seq = [x for x in range(-tt,tt) if x!=0] # or seq = [x for x in range(-tt+1,tt+1) if x!=0]
else:
seq = [x for x in range(-tt,tt+1) if x!=0]
"
because when nFrames is an even number, " seq = [x for x in range(-tt,tt+1) if x!=0]" will make the number of neigbor equal to nFrames instead of nFrames-1.
You can try running the command "python main.py  --other_dataset True --nFrames 6".

Thank you, I just revised that part.

And so, for the same reason, should the following line
https://github.com/alterzero/RBPN-PyTorch/blob/08757cb463a46a4dccb7f28d8bbf1ef8e21ea2a6/dataset.py#L74
be changed to "seq = [x+4 for x in range(-tt, tt+(0 if (nFrames % 2 ==0) else 1)) if x!=0]" or something?


Playfair Cipher

A digraph substitution cipher where pairs of letters are substituted according to a 5x5 matrix constructed from a keyword.
Add input and output field also so that people can experiment

@shrinidhihegde3 assign me this issue


[Feature Request] Is there a way to load test Postman Newman Scripts / API tests ?

[Feature Request] Is there a way to load test Postman Newman Scripts / API tests ? In our organisation we use Postman to test API functionality, it would be great if we can reuse those scripts to load test our api's rather than having to convert them to JMeter to make it work in Azure Load Testing Feature ..

Hi @rashmipatil If I understood it correctly you would like to import an API collection and directly run on our service? This is a high priority item on our roadmap. Let me check on the timeline and get back.

Yes, I am looking at importing an API collection and running a load test on
the collection using azure load testing service. Glad to know it's a
priority, will you be able to inform me about the timeline of this
functionality? When are you planning on delivering it?
Thank you
On Tue, 28 Dec 2021, 6:27 pm kaumishra, @.***> wrote:

Hi @rashmipatil https://github.com/rashmipatil If I understood it
correctly you would like to import an API collection and directly run on
our service? This is a high priority item on our roadmap. Let me check on
the timeline and get back.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/azure-load-testing/issues/10#issuecomment-1001911035,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABPINFP2N45JFCGL6CSYWFTUTFRGPANCNFSM5KE2B6NA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

Hello! Do you have a plan for this feature on the roadmap? We also use Postman collection for testing our API and we want to use Azure Load Testing to do out load testing.

@adiazcan Thanks for reaching out. Unfortunately we didn't see many customers asking for this, so we couldn't prioritize it. We'll watch out for more feedback so that we can plan properly.


Override display: flow

The flow value seems to behave like block, i.e. after setting it to flow it reads as block afterwards, so the collector cannot determine the support status.
For some reason, CSS.supports('display: flow') returns false in Chrome 114 and true in Chrome 115, but I couldn't find any matching entry in https://chromium.googlesource.com/chromium/src/+log/114.0.5735.199..115.0.5790.99?pretty=fuller&n=10000.
See: https://github.com/mdn/browser-compat-data/pull/24386/files#r1792287025

hm 🤔

[error] custom/overrides.json: SyntaxError: 'MemberExpression' is not allowed in JSON. (1198:3)


Allow the same node to be drawn in more than one location (cell span)

In xUML, for example, you might want to repeat the same imported class in two locations (different spans).
Currently, this won't work since the model element name is used as a dictionary index which prevents duplication in the layout.

Done!


Feature request: new pod select for rolled deployments

When using the deployment as the target, the agent doesn't see that the pod was deleted and the next connection to the port-forwarded port still tries to forward to the old pod. Once "kubectl relay" is restarted this is fixed, but I'd really like not to do that.

@knight42 unfortunately I'm connecting to the "debug" port, which is not exposed via Service. Creating the Service only to support such usecase is not the best user experience, I thought that krelay could help me here, but I set the expectations too high.
Either way, if you choose to implement it, this will greatly add to krelay value. During my development workflow I need to update Deployment frequently and restarting the port-forwarding constantly is really annoying.

Creating the Service only to support such usecase is not the best user experience

@Tarick I also consider this to be a valuable feature, but it might take some time to work it out and I cannot guarantee how long it will take. Before this is actually implemented, you could create a "debug" service or add the "debug" port to your service as a workaround.

@Tarick Hi! This feature has already been implemented, please upgrade krelay to v0.0.2(kubectl krew upgrade krew) to give it a try!

Thanks, @knight42 , everything works perfectly! Nice work!


Unify efforts with BQ datasketches repository

We have shared this repository with some Google engineers.  They were unaware of its existence and have created their own repository with similar objectives.
For example, a recent pull request introduces KLL, Tuple and Theta support to BigQuery:
https://github.com/GoogleCloudPlatform/bigquery-utils/pull/423
Furthermore, they have used a different method for assembling the scripts than what is used in this repository.  This introduces additional complexity but avoids potential problems with memory exhaustion.
Is it worth consolidating efforts?

We are not entirely sure yet how to consolidate efforts. We can try to contribute to Google repo.
On the other hand, they don't seem to have any versioning there, and the datasketches related SQL stuff is mixed with other UDFs. We know how to do Apache process with community voting and semantic versioning. We may want to keep this repo anyway. Let's discuss.

@davecromberge,
We have made progress.  Google BQ has agreed to use this repo for development and they will bring in the adaptors based on release tags.  There are some more details to work out but this is big progress.

@leerho thank you for the update - this is great news.  Should I keep the issue open or should we mark it as resolved?
My team is interested primarily in the CPC and Theta sketches in the BQ environment and would be available to deploy early versions to help with testing at scale if you would find this helpful.

Hi David,
Thanks for the offer, we just might ask for some help in testing!
Cheers,
Lee.
On Tue, Aug 13, 2024 at 1:55 AM David Cromberge @.***>
wrote:

@leerho https://github.com/leerho thank you for the update - this is
great news. Should I keep the issue open or should we mark it as resolved?
My team is interested primarily in the CPC and Theta sketches in the BQ
environment and would be available to deploy early versions to help with
testing at scale if you would find this helpful.
—
Reply to this email directly, view it on GitHub
https://github.com/apache/datasketches-bigquery/issues/4#issuecomment-2285720651,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADCXRQV56T7W6JPAO2KLDM3ZRHCXBAVCNFSM6AAAAABJSE6PNCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOBVG4ZDANRVGE
.
You are receiving this because you were mentioned.Message ID:
@.***>


Monorepo migration - bring "Data Magic" into the primary repo

It's time to get the dirty Data Magic repo cleaned up and merged into our primary repo.
@ArLough is leading this, with advisory assistance from @bigolboyyo, @justBanks, and @shreeup
Tasks:

[ ] Copy over the Flask back-end (App folder)
[ ] Migrate Issue tickets over
[ ] Get the Flask app deployed and API accessible (should we try Vercel or something else?)
[ ] Create a production branch and deployment workflow
[ ] Rename "Data Magic" repo to "Playground" for running community experiments and testing @Yinan0409's AI model
[ ] Update documentation

Thinking ahead to deployment, here is one article I was reading about a possible way to do it
https://testdriven.io/blog/flask-render-deployment/

@shreeup has been playing around with deploying "Data Magic" to Vercel.
If it's a better fit than another host, that's awesome.
My main concern is price - would I have to pay for both Netlify & Vercel? That starts to add up on top of the other project costs.
I suppose whatever we use to host this thing will have a cost to it beyond Netlify? Or can we consolidate?

i am going to delete venv folder from data-magic github repo (https://github.com/climate-tech-handbook/data-magic/tree/main/venv) . i think it is not really used and make the project heavy, in an effort to make it easier to deploy

My main concern is price - would I have to pay for both Netlify & Vercel? That starts to add up on top of the other project costs.
I suppose whatever we use to host this thing will have a cost to it beyond Netlify? Or can we consolidate?

Netlify does have serverless functions as an offering, but I'll have to look into it and see if it fits our use case.

I don't imagine switching to Docusaurus (Node / React) changes anything when it comes to Flask, but if it does let me know, as I am not a developer.
I also don't know if Flask has limitations as the project scales. If so, please share those opinions.

I don't think the Docusaurus switch will affect anything. Flask itself has a reputation as best for side projects and smaller web apps. But, I think it should work at least in the immediate future. There are alternatives that are still Python based like Django and FastAPI that are meant for more production sites and larger websites. But, I think we should be able to make that switch in the future if needed. FastAPI in particular looks very similar to Flask/

I was able to deploy docker image to render platform,
https://climatetechdatamagic.onrender.com/hello
but post endpoints are not really working.. will have to figure this out
Error
Traceback (most recent call last):
Jun 14 09:47:31 PM    File "/usr/local/lib/python3.8/site-packages/flask/app.py", line 2528, in wsgi_app
Jun 14 09:47:31 PM      response = self.full_dispatch_request()
Jun 14 09:47:31 PM    File "/usr/local/lib/python3.8/site-packages/flask/app.py", line 1825, in full_dispatch_request
Jun 14 09:47:31 PM      rv = self.handle_user_exception(e)
Jun 14 09:47:31 PM    File "/usr/local/lib/python3.8/site-packages/flask/app.py", line 1823, in full_dispatch_request
Jun 14 09:47:31 PM      rv = self.dispatch_request()
Jun 14 09:47:31 PM    File "/usr/local/lib/python3.8/site-packages/flask/app.py", line 1799, in dispatch_request
Jun 14 09:47:31 PM      return self.ensure_sync(self.view_functions[rule.endpoint])(**view_args)
Jun 14 09:47:31 PM    File "/usr/local/lib/python3.8/site-packages/flask/app.py", line 1890, in ensure_sync
Jun 14 09:47:31 PM      return self.async_to_sync(func)
Jun 14 09:47:31 PM    File "/usr/local/lib/python3.8/site-packages/flask/app.py", line 1911, in async_to_sync
Jun 14 09:47:31 PM      raise RuntimeError(
Jun 14 09:47:31 PM  RuntimeError: Install Flask with the 'async' extra in order to use async views.
Steps:
Build docker image push to docker hub
docker image build -t datamagic_docker .
docker run --platform linux/amd64 -p 5000:5000 -d datamagic_docker
docker login
docker tag datamagic_docker shreeupadhyaya/datamagic_docker
docker push shreeupadhyaya/datamagic_docker

deploy image from docker hub to render
https://render.com/docs/deploy-an-image

I experienced the same async issue on my local machine. It was resolved by doing 'pip install asgiref' for me
https://stackoverflow.com/questions/70321014/runtimeerror-install-flask-with-the-async-extra-in-order-to-use-async-views

Ok now I am able to invoke post methods in hosted platform.
This is image from docker logs which show that post endpoints invocation was successful and returned 200.

I am also attaching screenshot of postman returning 200 for one of endpoint


az login failure using ubuntu 22.04 terminal

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az login
Errors:
The command failed with an unexpected error. Here is the traceback:
SubscriptionClient.__init__() got an unexpected keyword argument 'user_agent'
Traceback (most recent call last):
  File "/home/uppaluru/.local/lib/python3.10/site-packages/knack/cli.py", line 233, in invoke
    cmd_result = self.invocation.execute(args)
  File "/home/uppaluru/.local/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 663, in execute
    raise ex
  File "/home/uppaluru/.local/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 726, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/home/uppaluru/.local/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 697, in _run_job
    result = cmd_copy(params)
  File "/home/uppaluru/.local/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 333, in __call__
    return self.handler(*args, **kwargs)
  File "/home/uppaluru/.local/lib/python3.10/site-packages/azure/cli/core/commands/command_operation.py", line 121, in handler
    return op(**command_args)
  File "/home/uppaluru/.local/lib/python3.10/site-packages/azure/cli/command_modules/profile/custom.py", line 139, in login
    subscriptions = profile.login(
  File "/home/uppaluru/.local/lib/python3.10/site-packages/azure/cli/core/_profile.py", line 176, in login
    subscriptions = subscription_finder.find_using_common_tenant(username, credential)
  File "/home/uppaluru/.local/lib/python3.10/site-packages/azure/cli/core/_profile.py", line 737, in find_using_common_tenant
    client = self._create_subscription_client(credential)
  File "/home/uppaluru/.local/lib/python3.10/site-packages/azure/cli/core/_profile.py", line 828, in _create_subscription_client
    client = client_type(credential, api_version=api_version,
TypeError: SubscriptionClient.__init__() got an unexpected keyword argument 'user_agent'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az login

Expected Behavior
Environment Summary
Linux-5.15.0-56-generic-x86_64-with-glibc2.35, Ubuntu 22.04.1 LTS
Python 3.10.6
Installer: PIP

azure-cli 2.43.0

Additional Context

@jiasli for awareness

Please see https://github.com/Azure/azure-cli/issues/21189


Issue in understanding the codebase

Hi, I was going through the codebase, and in Spectral Convolution I haven't understood the following line of code can you please explain to me what you are trying to do here?
     # upper block (truncate high freq)
    out_fft[:, :, :self.half_n_modes[0], :self.half_n_modes[1]] = self._contract(x[:, :, :self.half_n_modes[0], :self.half_n_modes[1]],
                                                                          self._get_weight(2*indices), separable=self.separable)
    # Lower block
    out_fft[:, :, -self.half_n_modes[0]:, :self.half_n_modes[1]] = self._contract(x[:, :, -self.half_n_modes[0]:, :self.half_n_modes[1]],
                                                                          self._get_weight(2*indices + 1), separable=self.separable)

Thanks. Looking for your reply.

It applies the convolution theorem and performs a contraction of the (truncated) Fourier transform of the input with the learned weights of the spectral convolution. You can refer to the Fourier neural operator paper for the details.

Hello,
Why are there two weights of size (..., half_modes, half_modes), and why is it applied to the :k_max/2 and -k_max/2:? I don't understand why not it is like that and not directly ":k_max". What am I missing about the implementation of the (R)FFT?


Hide vanished players from /team info

Hi,
Thanks for implementing this!
Do you think you could also implement this check here?:
https://github.com/booksaw/BetterTeams/blob/ec60dae480241ed55ffd2815f6517ac16fa9f69f/src/main/java/com/booksaw/betterTeams/commands/team/InviteCommand.java#L26
Maybe something along the lines of
if (toInvite == null || Utils.isVanished(toInvite)) {
			return new CommandResponse("noPlayer");
		}

Thank you!

Appreciate it. 🙏


Feature/bloss1/add ats file parser

Parse generic input ATS files to allow line by line interpretation. This greatly improves debugging bad input files, allows greater flexibility, readability, and provides a path away from exec() calls in the code.
Python's builtin ast (abstract syntax tree) module is used as a superior alternative to regex/text parsing. ast also allows for sophisticated handling of generic input files in the future.

I want to do a pull and test of this (don't recall if I have done that yet)

Passes my tests.  Looks good to merge


Pendulum Problem in Java

Usually while learning DSA we often indulge ourselves in Stack, queue and such other. But often we forget about the pendulum problems in arrays. I want to add pendulum Problems in java for the DSA for HACTOBER 2022

Go on add a solution for it, also do add a description for the pendulum problem.


Add ORDER BY clause to commands SQL

Function selectAllIncomingOrOutgoingCommandsForCluster did not order
its results, so ORDER BY cluster code then command code (including
manufacturer code in each).
Reorder expected test output to match new order. Fix typo in expected
test output as well.

I've verified that the change to expected output is the same lines, just reordered.

Thank you, @mlepage-google !


System freeze after suspending two instances of Neovim with io_uring enabled

Hi!
When running Arch Linux or Fedora Rawhide and suspending two instances of Neovim, which uses libuv, which uses io_uring, I experience a system freeze. It stops me from typing anything in any shell, or spawn any new shell, but I'm able to run some simple commands over ssh (e.g. ssh myserver ls -la).
dmesg doesn't report anything interesting as far as I could tell, other than some of the apps that were running not being responsive.
Mar 31 11:53:31 billy kernel: INFO: task st:29757 blocked for more than 122 seconds.
Mar 31 11:53:31 billy kernel:       Tainted: P           OE      6.8.2-arch2-1 #1
Mar 31 11:53:31 billy kernel: "echo 0 > /proc/sys/kernel/hung_task_timeout_secs" disables this message.
Mar 31 11:53:31 billy kernel: task:st              state:D stack:0     pid:29757 tgid:29757 ppid:29751  flags:0x00004006
Mar 31 11:53:31 billy kernel: Call Trace:
Mar 31 11:53:31 billy kernel:  <TASK>
Mar 31 11:53:31 billy kernel:  __schedule+0x3e6/0x1520
Mar 31 11:53:31 billy kernel:  schedule+0x32/0xd0
Mar 31 11:53:31 billy kernel:  schedule_timeout+0x151/0x160
Mar 31 11:53:31 billy kernel:  wait_for_completion+0x86/0x170
Mar 31 11:53:31 billy kernel:  __flush_work.isra.0+0x173/0x280
Mar 31 11:53:31 billy kernel:  ? __pfx_wq_barrier_func+0x10/0x10
Mar 31 11:53:31 billy kernel:  n_tty_poll+0x134/0x1e0
Mar 31 11:53:31 billy kernel:  tty_poll+0x57/0xc0
Mar 31 11:53:31 billy kernel:  do_select+0x362/0x880
Mar 31 11:53:31 billy kernel:  ? pollwake+0x50/0xa0
Mar 31 11:53:31 billy kernel:  ? __pfx_pollwake+0x10/0x10
Mar 31 11:53:31 billy kernel:  ? __pfx_pollwake+0x10/0x10
Mar 31 11:53:31 billy kernel:  ? __pfx_pollwake+0x10/0x10
Mar 31 11:53:31 billy kernel:  core_sys_select+0x36b/0x530
Mar 31 11:53:31 billy kernel:  do_pselect.constprop.0+0xe9/0x180
Mar 31 11:53:31 billy kernel:  __x64_sys_pselect6+0x3d/0x70
Mar 31 11:53:31 billy kernel:  do_syscall_64+0x86/0x170
Mar 31 11:53:31 billy kernel:  ? do_syscall_64+0x96/0x170
Mar 31 11:53:31 billy kernel:  ? do_syscall_64+0x96/0x170
Mar 31 11:53:31 billy kernel:  ? exc_page_fault+0x7f/0x180
Mar 31 11:53:31 billy kernel:  entry_SYSCALL_64_after_hwframe+0x6e/0x76
Mar 31 11:53:31 billy kernel: RIP: 0033:0x7541e536b640
Mar 31 11:53:31 billy kernel: RSP: 002b:00007ffec990be70 EFLAGS: 00000202 ORIG_RAX:<PHONE_NUMBER>00010e
Mar 31 11:53:31 billy kernel: RAX: ffffffffffffffda RBX:<PHONE_NUMBER>000000 RCX: 00007541e536b640
Mar 31 11:53:31 billy kernel: RDX:<PHONE_NUMBER>000000 RSI: 00007ffec990bf50 RDI:<PHONE_NUMBER>000005
Mar 31 11:53:31 billy kernel: RBP:<PHONE_NUMBER>000000 R08:<PHONE_NUMBER>000000 R09: 00007ffec990beb0
Mar 31 11:53:31 billy kernel: R10:<PHONE_NUMBER>000000 R11:<PHONE_NUMBER>000202 R12: bff0000000000000
Mar 31 11:53:31 billy kernel: R13:<PHONE_NUMBER>000000 R14:<PHONE_NUMBER>000000 R15:<PHONE_NUMBER>000010
Mar 31 11:53:31 billy kernel:  </TASK>

The freeze doesn't occur after disabling io_uring in libuv using UV_USE_IO_URING=0 or in the kernel with sysctl kernel.io_uring_disabled=1
uname -a from the tested systems

Linux billy 6.6.23-1-lts #1 SMP PREEMPT_DYNAMIC Wed, 27 Mar 2024 07:47:20 +0000 x86_64 GNU/Linux running Arch Linux
Linux yolen 6.8.2-arch2-1 #1 SMP PREEMPT_DYNAMIC Thu, 28 Mar 2024 17:06:35 +0000 x86_64 GNU/Linux running Arch Linux
Linux localhost 6.9.0-0.rc1.20240329git317c7bc0ef03.20.fc41.x86_64 #1 SMP PREEMPT_DYNAMIC Fri Mar 29 14:04:53 UTC 2024 x86_64 GNU/Linux running Fedora Rawhide

Reproduction steps

Be on Arch Linux (bug reproducible on netcup & hetzner servers) or Fedora Rawhide
Install Neovim (pacman -S neovim)
Run Neovim
Suspend Neovim (by pressing CTRL+Z)
Run Neovim again
Suspend Neovim (by pressing CTRL+Z)
Your system is now most likely unresponsive

The suspension can be done in separate shells, or as different users with the same results.
Video showing off the freeze
https://github.com/axboe/liburing/assets/962989/f405bd56-28b8-4a2f-a4b5-3de7d8023010
I'm still able to run certain apps on the system, but not open a shell
If the<EMAIL_ADDRESS>email is a better place for this report let me know and I'll report it there instead.
Originally reported in https://github.com/libuv/libuv/issues/4377

Funky! Thanks for the report, I'll take a look.

Awesome! Glad to hear you were able to reproduce it. Thank you!

Unsure if you're able to test kernel patches, but I believe the below should do it. Looks like we get into an inversion between the events workqueue being flushed for console output, and io_uring ring exits for some weird cases. If not, then I'll get it into 6.9-rc3 end of this week and it can bubble back to stable from there.
diff --git a/io_uring/io_uring.c b/io_uring/io_uring.c
index 5d4b448fdc50..f6277e029d5f 100644
--- a/io_uring/io_uring.c
+++ b/io_uring/io_uring.c
@@ -147,6 +147,7 @@ static bool io_uring_try_cancel_requests(struct io_ring_ctx *ctx,
 static void io_queue_sqe(struct io_kiocb *req);

 struct kmem_cache *req_cachep;
+static struct workqueue_struct *iou_wq __ro_after_init;

 static int __read_mostly sysctl_io_uring_disabled;
 static int __read_mostly sysctl_io_uring_group = -1;
@@ -3161,7 +3162,7 @@ static __cold void io_ring_ctx_wait_and_kill(struct io_ring_ctx *ctx)
 	 * noise and overhead, there's no discernable change in runtime
 	 * over using system_wq.
 	 */
-	queue_work(system_unbound_wq, &ctx->exit_work);
+	queue_work(iou_wq, &ctx->exit_work);
 }

 static int io_uring_release(struct inode *inode, struct file *file)
@@ -4185,6 +4186,8 @@ static int __init io_uring_init(void)
 	io_buf_cachep = KMEM_CACHE(io_buffer,
 					  SLAB_HWCACHE_ALIGN | SLAB_PANIC | SLAB_ACCOUNT);

+	iou_wq = alloc_workqueue("iou_exit", WQ_UNBOUND, 64);
+
 #ifdef CONFIG_SYSCTL
 	register_sysctl_init("kernel", kernel_io_uring_disabled_table);
 #endif

Oh, and if you want a Reported-by: tag in the commit, please do let me know and I'll update it with that. Just need an identity + email for that. Queued up:
https://git.kernel.dk/cgit/linux/commit/?h=io_uring-6.9&id=d1a9cef84784f873b457f2622ca2415c4b4db748

A Reported-by tag would be appreciated yeah! My identity + email below:
Rasmus Karlsson<EMAIL_ADDRESS>I'll see if I can test the patch, or if not test the rc3 kernel when that's released

Perfect, commit updated:
https://git.kernel.dk/cgit/linux/commit/?h=io_uring-6.9&id=e5444baa42e545bb929ba56c497e7f3c73634099

Just applied the patch to my 6.8.2 kernel on a system that previously experienced the issue and I can confirm that it fixes it. Thanks for the quick turnaround!

out of curiousity, why did all shells froze, but ssh didn't?

Perfect, commit updated:
https://git.kernel.dk/cgit/linux/commit/?h=io_uring-6.9&id=e5444baa42e545bb929ba56c497e7f3c73634099

I tested this on top of arch linux kernel 6.8.2-arch2-2 and it seems to work. You have my permission to add Tested-by: Iskren Chernev<EMAIL_ADDRESS>

out of curiousity, why did all shells freeze, but ssh didn't?

For me ssh was freezing too. But I also failed to kill the nvim after it froze... maybe there are a few variations of this.

I am affected by this too, any TTY is affected

Thanks everyone, patch will go upstream later this week, and it'll bubble back to -stable post that. Marking this one as closed as fix exists.


feat: Part 2 of IT tests

Fixes #1553
Now handles 3 URLS
Checklist

[x] Integration tests updated

@flank-it

@flank-it


[FLINK-26798][runtime] Hardens test against unexpected heartbeat

What is the purpose of the change
testJobFailureWhenTaskExecutorHeartbeatTimeout failed due to a heartbeat
being processed during the test. The missing payload of the test
implementation caused an unexpected error. Increasing the heartbeat
interval to a higher value should prevent this test from failing
because of an unexpected heartbeat.
Brief change log

Increased the heartbeat interval. This won't prevent the initial heartbeat (which is ok, because no Executions are scheduled, yet) but let's the next heartbeat being scheduled long time in the future.

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

I adapted the PR. We're removing the (unnecessary) JobMaster.heartbeatFromTaskManager call as the source of the error. It's not necessary for keeping the JobMaster running in the test cases. The heartbeat from the TaskManager to the JobManager is performed normally (i.e. a completed Future is returned) as long as the test isn't triggering a timeout through the TestingHeartbeatServices instance. PTAL

@flinkbot run azure


Adding jquery, tether for Bootstrap 4 and making application layout more plain

Taking some stuff out of application layout.
Adding Jquery.
Adding Tether.

🎉 💯 👍


More robust "all properties" named property enumeration

This is primarily the ability to back off the page size for named property enumeration.  However, with that I found an issue where the object gets invalidated and need to be re-opened when calling GetNamesFromIds. So now the GetAllProps code path will attempt to see if the passed in object is either a IMAPIContainer or IMsgStore object, and obtain the root folder from that object. This will allow the code to reliably re-open the root folder object as needed while trying to enumerate all the properties.

@stephenegriffin - I'll still have some testing to finish up tomorrow on this. I need to make sure I didn't break anything in the GUI and be sure it works in some of the truly broken scenarios.

There actually were some issues due to some code passing null for the tag array which was ** (pointer to pointer). Since all these methods but MAPI are returning the output tags in a vector, we don't need to actually have them take in **, so I converted them all to just pass in * which makes usage easier.


Add Typescript to React.Template/reactnet-webpack

I cloned this sample, compiling it runs.
I added Typescript, had to update npm packages and add some type definitions. It builds, but I get a runtime exception:
JsRuntimeException: TypeError: Object.create: argument is not an Object and is not null on @Html.ReactRouter("Components.RootComponent", new { initialComments = Model.Comments, page = Model.Page }, renderFunctions: chainedFunctions)
webpack.config.js:
const path = require('path');
const { WebpackManifestPlugin } = require('webpack-manifest-plugin');
const webpack = require('webpack');

module.exports = {
	entry: './Content/components/expose-components.tsx',
	output: {
		filename: '[name].[contenthash:8].js',
		globalObject: 'this',
		path: path.resolve(__dirname, 'wwwroot/dist'),
		publicPath: '/dist/'
	},
	mode: process.env.NODE_ENV === 'production' ? 'production' : 'development',
	optimization: {
		runtimeChunk: {
			name: 'runtime', // necessary when using multiple entrypoints on the same page
		},
		splitChunks: {
			cacheGroups: {
				commons: {
					test: /[\\/]node_modules[\\/](react|react-dom)[\\/]/,
					name: 'vendor',
					chunks: 'all',
				},
			},
		},
	},
	module: {
		rules: [
			{
				test: /\.tsx?$/,
				use: 'babel-loader',
				exclude: /node_modules/,
			},
		],
	},
	resolve: {
		extensions: [".tsx", ".ts", ".js"],
		fallback: {
			"stream": false,
			"buffer": false
		}
	},
	plugins: [
		new WebpackManifestPlugin({
			fileName: 'asset-manifest.json',
			generate: (seed, files) => {
				const manifestFiles = files.reduce((manifest, file) => {
					manifest[file.name] = file.path;
					return manifest;
				}, seed);

				const entrypointFiles = files.filter(x => x.isInitial && !x.name.endsWith('.map')).map(x => x.path);

				return {
					files: manifestFiles,
					entrypoints: entrypointFiles,
				};
			},
		}),
		new webpack.ProvidePlugin({
			Buffer: ['buffer', 'Buffer'],
		}),
	]
};

tsconfig.json
{
	"compilerOptions": {
		"target": "es5",
		"module": "commonjs",
		"jsx": "react-jsx",
		"sourceMap": true,
		"strict": false,
		"noUnusedLocals": true,
		"noUnusedParameters": true,
		"noImplicitReturns": true,
		"noFallthroughCasesInSwitch": true,
		"noUncheckedIndexedAccess": true,
		"esModuleInterop": true,
		"skipLibCheck": true,
		"forceConsistentCasingInFileNames": true,
		"plugins": [
			{
				"name": "typescript-plugin-css-modules"
			}
		]
	},
	"include": [
		"Content"
	]
};

What's wrong?

After hours of struggling I have been able to get it to run. I think the problem came from the missing @Html.ReactGetScriptPaths() and @Html.ReactGetStylePaths().

I am still struggling with this sample, added typescript and trying to add a style. Using css modules i get document is not defined, using styled component i get window is not defined.
Is it possible to get it to work or am I wasting my time?

After hours of struggling I have been able to get it to run. I think the problem came from the missing @Html.ReactGetScriptPaths() and @Html.ReactGetStylePaths().

Is it possible you could share the repository where you got Typescript in combination with webpack and Reactjs.NET to run? I could really benefit from a straightforward example, because I'm also struggling with this.

I am sorry, I moved all to client side and defined a pure REST API so I am not using React.NET anymore.


🔬Research impact of using ref structs

Complete The Item Below

[X] I have updated the title without removing the 🔬 emoji.

What To Research
Research the ability to use ref structs for performance reasons.
Good candidates would be the structs like TextureGPUData, TexturureVertexData, and RectGPUData.
Reason:
The reason is because if the structs in the GPUData namespace are ref structs, we can internally have a struct field that is a Span<T>.  This could hold all of the array data that gets sent to the GPU.  Doing this would mean we can avoid using the ToArray() extension methods which cause allocations and perf issues.
Research Results
No response
Acceptance Criteria
This issue is finished when:

[ ] Research complete and issues created (if needed).
[ ] If any issues were created, they have been added to the Related Work section below.

ToDo Items

[X] Priority label added to this issue.  Refer to the Priority Type Labels section below.
[ ] Issue linked to the correct project.

Issue Dependencies
No response
Related Work
No response
Additional Information:
Priority Type Labels

Priority Type
Label

Low Priority
low priority

Medium Priority
medium priority

High Priority
high priority

Code of Conduct

[X] I agree to follow this project's Code of Conduct.

Closed without changes
Reason: This does not apply anymore.  Enough is known about ref structs.


Clarity on action and reaction

For the video_0001
From annotaion i see for frame 363
hand_gesture
and for frame 364
reaction
action
hand_gesture
I am not able to locate what is this "reaction action" in the readme. Could you please elaborate on this. How these lables would be interesting, while predicting the Intention. I would like to use LSTM on the sequence of activities, but not sure about the activites such as "reaction action".
from jaad_data import JAAD
imdb = JAAD(data_path = jaad_path)
anno = imdb._get_annotations("video_0001")
for k,v in anno["ped_annotations"]["0_1_3b"]["behavior"].items():
print (k)
Outputs:
cross
reaction
look
nod
action
hand_gesture
I was expecting "walking" in this output. What are the possible values apart from above output.

Hi There,
So Reactions as the name suggests refers to the reaction of the people with respect to the driver's actions, e.g. clearing path, etc. A good place to look for these tags is in "_map_text_to_scalar" function. Basically, actions are very basic movements of the pedestrian, i.e. 'standing', 'walking'. Reactions are rarer and only are available if there is an interaction taking place. In your case, if you want to do intention, a good starting point would be 'crossing' labels which indicate whether the person will cross '1, or not '0' or even is irrelevant '-1'. When you select 'all' samples to be generated (including the ones without behavioral data), all those pedestrians will be irrelevant. We will try to update the readme sometime soon and include more details.

Hallo, thanks for your quick reply.
One thing not clear, in the first video i could see the lady crossing the road, but from annotation i am not getting that information. sample output for last frames
560 : clear_path-walking-nod?-not_looking-hand_gesture?-not_crossing
561 : clear_path-walking-nod?-not_looking-hand_gesture?-not_crossing
562 : clear_path-walking-nod?-not_looking-hand_gesture?-not_crossing
563 : clear_path-walking-nod?-not_looking-hand_gesture?-not_crossing
564 : clear_path-walking-nod?-not_looking-hand_gesture?-not_crossing
565 : clear_path-walking-nod?-not_looking-hand_gesture?-not_crossing
566 : clear_path-walking-nod?-not_looking-hand_gesture?-not_crossing
567 : clear_path-walking-nod?-not_looking-hand_gesture?-not_crossing
568 : clear_path-walking-nod?-not_looking-hand_gesture?-not_crossing
What does clear_path indicates in this context? I was thinking clear_path indicates pedestrain crossing and clearing the path for Vehicle. Could you please throw some light into it?

So to answer your question,
Our convention for crossing tags is that someone crosses from one side of the road to the other. That particular video that you mentioned, there is no crossing taking place. The lady in the video is doing something in her car and we are simply waiting for her to finish so we can park next to her car.
So what clear path entails? It refers to situations where the pedestrian clears the way for the vehicle to go first. Your point is correct that when someone crosses is also a form of clearing path. However, in this dataset, we only focus on the cases in which the vehicle proceeds before the pedestrian. For example, the pedestrian is standing on the road or starting to cross but when he/she sees the car, he/she goes backward and let the driver proceed.

Hallo, I did not find any interface to extract video names depending on the camera used for recording.
In the paper it is mentioned that below cameras are used with different FOV
Garmin GDR-35 ,GoPro Hero+ ,Highscreen BB Connect
How to categories the videos with above camera types.

Ok, thanks for the info!

Hallo, is there any information about vehicles current speed for a particular frame.

Hi there,
we did not record vehicle information, only text tags for driver actions are available (e.g. slow down or speed up).

I don't have these stats handy. You can use ffprobe to compute video durations.

Yes. Thank you!

I cannot find the video clips on the given link. Are they removed?

The videos were not removed but our server is down. Please check again later.


Fix argparse type error

Resolved the following error on executing run_squad2.py --help
TypeError: %o format: an integer is required, not dict

Thanks Surya!


fix: Resolve issue with center-fixed-web-mercator and center-web-mercator

Description of Changes
It was not taking into account that the center was the geodesic center and not the web mercator center. This caused movement of the polygon as it was resized.
Link to Issue
Potentially resolves #243 but hard to say without more feedback.
PR Checklist

[x] There is a associated GitHub issue
[x] If I have added significant code changes, there are relevant tests
[ ] If there are behaviour changes these are documented

Description of Changes
It was not taking into account that the center was the geodesic center and not the web mercator center. This caused movement of the polygon as it was resized.
Link to Issue
Potentially resolves #243 but hard to say without more feedback.
PR Checklist

[x]  There is a associated GitHub issue
[x]  If I have added significant code changes, there are relevant tests
[ ]  If there are behaviour changes these are documented

Hey， @JamesLMilner . Now the scaling effect experience for the circle has been improved, but if the circle is scaled to a very small time to zoom again, repeated operations, there will be circular irregularities.


@helia/verified-fetch announcement

Adds @helia/verified-fetch announcement blog post
TODO before merging

[ ] Update publish date

you have to pass a contentTypeParser to get automatic content type determining. passing just an Accept header won't get many users the fully fleshed out functionality they want (auto playing videos instead of downloading, etc..)

Good point. I'll mention this in the "Supports a wide range of data types" section and add an example showing how this works and the nuance around it.


Split html-report-page into two exports with different type params

summaryReportContent() and sourceReportContent() implicitly know what reportType to use. So I removed the reportType: opt.type field from their props and let html-report-page.jsx export two components.
This unlocks the ability to specify specific flowtypes for Summary and SourceFile report pages, which we now also have. Should increase coverage, remove optional/unused fields and prevent that threshold issue from happening again.

@ryan953 ... man...  This is exactly that change that I had in mind! :+1:
I'll take a look at the PR in more details asap (during this weekend for sure).

I have some more changes related to the annotation column and those new flags sitting in a branch. but the rebase was getting hairy so i split this out. I'll keep splitting it up as things land so I don't get caught with a big rebase again.


Update to aws-sdk v3

aws-sdk v2 is deprecated

We support V3 already.
V2 is still supported as some users are still actively using V2.

I'm not sure what you mean that v3 is supported already. Could you explain how one would configure this project to use aws sdk v3?

Sorry for the misunderstanding. I thought you were referring to OneTable (dynamodb-onetable) and not the cli.
Yes, the CLI is currently V2.
We are planning upgrading all to V3 over the next quarter.

Thanks for the update


Need CodeQuality issue template

What problem does this solve or what need does it fill?
Users cannot create issues for code quality, only for bugs and new features. For example #5439 is neither a bug (everything works), nor a feature request (the API would possibly change a bit, but the feature set won't).
What solution would you like?
Add a new GitHub template for code quality.
What alternative(s) have you considered?
Ping @alice-i-cecile for each new code quality Issue to remove the C-Bug tag and add the C-CodeQuality one.
Additional context
Surprisingly, Alice seem to prefer the template approach... ¯\_(ツ)_/¯

I am in fact in favor of this!

Should #5279 be merged first before tackling this?

No need; I'm fine to do them in whatever order.


Bump pyjwt to 2.4.0

I want to merge this change because it bumps pyjwt to 2.4.0

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-7059 Update pyjwt to 2.4.0


permissions: move and rename PermissionPolicy input and output types

Hey, I just made a Pull Request!
These types will soon be used in frontend permission clients as we split out the queries that can return conditional decisions to protect users from receiving them unexpectedly. This commit simply moves them from permission-node to permission-common, with a small rename for consistency.
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Replaced by #10375


udev naming convention

https://github.com/RoverRobotics/roverrobotics_ros2/tree/main/roverrobotics_driver/udev udev rules by convention start with a number between (I believe) 1 and 99 rated by importance. This numbering is important in case of colliding udev rules such that the highest priority rules override lower ones in case of conflict. May I suggest something in the mid-30s to 60s.

updating with PR#9 Udev now will be 50-

https://github.com/RoverRobotics/roverrobotics_ros2/tree/05410566dfd83a959345753c6d86352d4bdf6bc6/roverrobotics_driver/udev
Please also update your raw file name

will do!


Updated Deploying doc to 2.3 workflow

This commit updates "Deploying your first project" (deploying.md) to the 2.3 workflow including interactive the zos create command; removed redundant instructions; revised setup instructions.

Closing in favor of #961


Cannot traverse relations when passing objects that came from Prefetch object to other serializer

This happens in the the beta which adds support for Prefetch objects in prefetch_related. This is a known issue that is blocking the beta release.
Will work on this soon.

Note: This only happens when a to_attr is set. Probably an issue with using the to_attr to traverse instead of the prefetch_through, leading to a path that does not exist.

        select_items.extend(return_values[0])

        simple_prefetch = [item.prefetch_through if isinstance(item, Prefetch) else item for item in prefetch_items]
        for value in return_values[1]:
            prefetch_through = value.prefetch_through if isinstance(value, Prefetch) else value
            if prefetch_through in simple_prefetch:
                continue

            prefetch_items.append(value)

        return select_items, prefetch_items

This should replace line 116 (_extend_relation_items)

Initial issue still not fixed


Request: case-insensitive citation abbreviations

For example, if my abbreviations.json file contains
"American Journal of Botany":"Amer. J. Bot.",

then American Journal Of Botany or american journal of botany will not be matched and abbreviated.
I think it makes sense to use a case-insensitive match here.  No journal title is differentiated by case, and there are many scenarios or errors where the correct case might not be used in both the abbreviations file and the references file.

I agree, this makes sense.


Bundle fails on unexpected token when minification is enabled

Description
I noticed an issue where the minified bundling fails on syntax errors, even though the development build works flawlessly.
Bundling the invalid code produces the output:
Loading dependency graph, done.
/Users/aeir/Downloads/TestProject/node_modules/babylon/lib/index.js:4336
  throw err;
  ^

SyntaxError: Unexpected token, expected , (4:0)
    at Parser.pp$5.raise (/Users/aeir/Downloads/TestProject/node_modules/babylon/lib/index.js:4333:13)
    at Parser.pp.unexpected (/Users/aeir/Downloads/TestProject/node_modules/babylon/lib/index.js:1705:8)
    at Parser.pp.expect (/Users/aeir/Downloads/TestProject/node_modules/babylon/lib/index.js:1693:33)
    at Parser.pp$3.parseCallExpressionArguments (/Users/aeir/Downloads/TestProject/node_modules/babylon/lib/index.js:3484:12)
    at Parser.pp$3.parseSubscripts (/Users/aeir/Downloads/TestProject/node_modules/babylon/lib/index.js:3453:31)
    at Parser.pp$3.parseExprSubscripts (/Users/aeir/Downloads/TestProject/node_modules/babylon/lib/index.js:3424:15)
    at Parser.pp$3.parseMaybeUnary (/Users/aeir/Downloads/TestProject/node_modules/babylon/lib/index.js:3394:19)
    at Parser.pp$3.parseExprOps (/Users/aeir/Downloads/TestProject/node_modules/babylon/lib/index.js:3324:19)
    at Parser.pp$3.parseMaybeConditional (/Users/aeir/Downloads/TestProject/node_modules/babylon/lib/index.js:3301:19)
    at Parser.pp$3.parseMaybeAssign (/Users/aeir/Downloads/TestProject/node_modules/babylon/lib/index.js:3264:19)
/Users/aeir/Downloads/TestProject/node_modules/babylon/lib/index.js:4336
  throw err;
  ^

SyntaxError: Unexpected token, expected , (4:0)
    at Parser.pp$5.raise (/Users/aeir/Downloads/TestProject/node_modules/babylon/lib/index.js:4333:13)
    at Parser.pp.unexpected (/Users/aeir/Downloads/TestProject/node_modules/babylon/lib/index.js:1705:8)
    at Parser.pp.expect (/Users/aeir/Downloads/TestProject/node_modules/babylon/lib/index.js:1693:33)
    at Parser.pp$3.parseCallExpressionArguments (/Users/aeir/Downloads/TestProject/node_modules/babylon/lib/index.js:3484:12)
    at Parser.pp$3.parseSubscripts (/Users/aeir/Downloads/TestProject/node_modules/babylon/lib/index.js:3453:31)
    at Parser.pp$3.parseExprSubscripts (/Users/aeir/Downloads/TestProject/node_modules/babylon/lib/index.js:3424:15)
    at Parser.pp$3.parseMaybeUnary (/Users/aeir/Downloads/TestProject/node_modules/babylon/lib/index.js:3394:19)
    at Parser.pp$3.parseExprOps (/Users/aeir/Downloads/TestProject/node_modules/babylon/lib/index.js:3324:19)
    at Parser.pp$3.parseMaybeConditional (/Users/aeir/Downloads/TestProject/node_modules/babylon/lib/index.js:3301:19)
    at Parser.pp$3.parseMaybeAssign (/Users/aeir/Downloads/TestProject/node_modules/babylon/lib/index.js:3264:19)
/Users/aeir/Downloads/TestProject/node_modules/babylon/lib/index.js:4336
  throw err;
  ^

SyntaxError: Unexpected token, expected , (4:0)
    at Parser.pp$5.raise (/Users/aeir/Downloads/TestProject/node_modules/babylon/lib/index.js:4333:13)
    at Parser.pp.unexpected (/Users/aeir/Downloads/TestProject/node_modules/babylon/lib/index.js:1705:8)
    at Parser.pp.expect (/Users/aeir/Downloads/TestProject/node_modules/babylon/lib/index.js:1693:33)
    at Parser.pp$3.parseCallExpressionArguments (/Users/aeir/Downloads/TestProject/node_modules/babylon/lib/index.js:3484:12)
    at Parser.pp$3.parseSubscripts (/Users/aeir/Downloads/TestProject/node_modules/babylon/lib/index.js:3453:31)
    at Parser.pp$3.parseExprSubscripts (/Users/aeir/Downloads/TestProject/node_modules/babylon/lib/index.js:3424:15)
    at Parser.pp$3.parseMaybeUnary (/Users/aeir/Downloads/TestProject/node_modules/babylon/lib/index.js:3394:19)
    at Parser.pp$3.parseExprOps (/Users/aeir/Downloads/TestProject/node_modules/babylon/lib/index.js:3324:19)
    at Parser.pp$3.parseMaybeConditional (/Users/aeir/Downloads/TestProject/node_modules/babylon/lib/index.js:3301:19)
    at Parser.pp$3.parseMaybeAssign (/Users/aeir/Downloads/TestProject/node_modules/babylon/lib/index.js:3264:19)

Uncaught error in the transformer worker: /Users/aeir/Downloads/TestProject/node_modules/react-native/packager/transformer.js

See reproduction section for invalid code example.
Reproduction
# Create new project
react-native init TestProject
cd TestProject/

# Create file with non-minifiable syntax
echo "var barWidth = Math.max(
    (
        (parent.width - 60) // Width of single data point on chart [Removing this comment fixes the issue)
        / data.length // Number of data points in chart
    )
    * 0.8, 5);
" > test.js

# Bundle code
node node_modules/react-native/local-cli/cli.js bundle --dev false --entry-file test.js --bundle-output /dev/null --assets-dest /dev/null

Solution
Would need to dig deep into the react native package to figure out what babel plugins or configurations cause this issue.
Additional Information

React Native version: 0.41.2
Platform: both
Operating System: MacOS

^^ can confirm this is happening on my build too
Information
React Native version: 0.41.2
Platform: both
Operating System: MacOS

We are having a similar problem with release builds on 41.2.
For us, the cause was React Native 41.2 on a fresh npm install is pulling in uglify-js vs 2.8.3.  We shrinkwrapped it back to 2.7.5 and everything is working fine again.

I am seeing this same issue even on RN 0.43.4.

Wow—bizarrely enough, this occurred in my app because I used name as a variable in JSX. For example:
const Thing = ({ name }) => (
  <Text>{name}</Text>
)

I changed that code to the following to fix all errors:
const Thing = ({ ...props }) => (
  <Text>{props.name}</Text>
)

Hi there! This issue is being closed because it has been inactive for a while. Maybe the issue has been fixed in a recent release, or perhaps it is not affecting a lot of people. Either way, we're automatically closing issues after a period of inactivity. Please do not take it personally!
If you think this issue should definitely remain open, please let us know. The following information is helpful when it comes to determining if the issue should be re-opened:

Does the issue still reproduce on the latest release candidate? Post a comment with the version you tested.
If so, is there any information missing from the bug report? Post a comment with all the information required by the issue template.
Is there a pull request that addresses this issue? Post a comment with the PR number so we can follow up.

If you would like to work on a patch to fix the issue, contributions are very welcome! Read through the contribution guide, and feel free to hop into #react-native if you need help planning your contribution.

That's a shame to see this issue closed. Currently it's a pain to debug this kind of issues as you don't have any code related to this. And when you have a huge codebase and make some upgrades, you don't any any clue to what bothering babylon....


Optimize polygon packing

@amakaseev could you take a look at optimizing the polygon packing algorithn. It's really slow at the moment

Increasing the step value from 5 to 20 improves time a lot and only generates a slightly bigger atlas.

it is possible to try to set a step depending on the volume of sprites...

also profiling to measure the slowest place and to try to optimize it


esp_https_ota: Fix build warning when CONFIG_OTA_ALLOW_HTTP is set (IDFGH-5098)

Fix below build warning when CONFIG_OTA_ALLOW_HTTP is set:
esp_https_ota.c:165:13: warning: 'is_server_verification_enabled' defined but not used [-Wunused-function]
While at it, also simplify the is_server_verification_enabled logic.
Fixes: 48987ad9df35 ("esp_https_ota: Enable option of using global_ca_store and x509_crt_bundle")
Signed-off-by: Axel Lin<EMAIL_ADDRESS>

Thanks for your contribution.

Wouldn't it be better to fix https://github.com/espressif/esp-idf/blob/1f54d17503ff17d830de75eed4eb47bc6a3aafc3/components/esp_https_ota/src/esp_https_ota.c#L183-L189
so that it still prints a warning but doesn't return an error?

@negativekelvin
We can do that, but if we look at plain HTTP config option and its help:
https://github.com/espressif/esp-idf/blob/1f54d17503ff17d830de75eed4eb47bc6a3aafc3/components/esp_https_ota/Kconfig#L4
It is clearly stated that this is "test" only configuration and there is security risk associated with it. So if someone is enabling it then they are aware about associated risk.
Given this, current PR looks good to me. However open to suggestions as well.

I think it's better to have the runtime logs for confirmation and understanding than silently remove the function.  I'm sure it will happen at least once that somebody doesn't know it's enabled.

IMHO, one topic per patch. The fix does what the subject line says.

@AxelLin the other way would also fix build warning by always calling is_server_verification_enabled.  If this is merged just have to undo it later.

Hi @AxelLin  I have added a followup commit which incorporates changes suggested by @negativekelvin, It is merged in the internal codebase, should be available on GitHub shortly. Thank you for your contribution in fixing this issue.


Running as non root user

Hi I'm trying to lock down my containers as much as I can and that entails running them as a non root user.
When I add
user: 10002:10002
security_opt:
  - no-new-privileges:true

The container fails to start with the following:
s6-overlay-suexec: warning: unable to gain root privileges (is the suid bit set?)
s6-mkdir: warning: unable to mkdir /run/s6: Permission denied
s6-mkdir: warning: unable to mkdir /run/service: Permission denied
s6-overlay-suexec: fatal: child failed with exit code 111

Is there someway to run it as a non root user?

Found the answer in #61


Replace Bindings implementation by one which is able to keep variable equalities

New implementation of the match_atoms() treats both atoms symmetrically. It returns variables from both left and right arguments. If caller doesn't need this then Bindings::narrow_vars() can be called to leave only variables which are required. Such implementation of the match_atoms() is going to be reused in type checker logic.
To return variables new implementation of Bindings is done. This implementation keeps both variable equalities and variable values. Variable values are kept without full resolution of sub-variables. When instance of Bindings is returned via C API values for all variables are resolved. Separate change in C API is required to keep Bindings as is (see #244).
Also separate change is required to implement Bindings editing logic correctly. Right now code expects that adding a binding for the value doesn't multiply number of Bindings instances. But there are cases when it is not true. For example we are trying to match two expressions ($x $x) and  ((A $y) {space}), where space is a atomspace containing following atoms: (A B), (A C), (D E). Then algorithm first matches $x with (A $y) and adds this match as bindings for variable $x: { $x = (A $y) }. But second match of the $x and {space} will require checking whether {space} and (A $y) can be matched. And indeed they can be matched but there are more then one result of such match: (A B) and (A C). Thus the final match result should contain two instances of Bindings: { $x = (A $y), $y = B } and { $x = (A $y), $y = C }. Current API of Bindings structure doesn't allow representing it. It is explained in TODO comments before corresponding functions. Taking into account such case is relatively rare I postponed it for the future PRs.

@e-gorodnitskiy FIY, it is a PR which contains new Bindings API.

@vsbogd , I skimmed through the PR. I supposed there should be something like this to track variables in consequent bindings, but understanding the implementation in detail would be time-consuming and would unlikely lead to any useful input, so I wonder what you expect from the review. Do you think it is necessary to check if this PR solves the issues like https://github.com/trueagi-io/hyperon-experimental/blob/b94274904cea7c96a619381a14283d68835ee594/python/tests/scripts/b2_backchain.metta#L50 , or it is not intended to solve all the problems and doesn't require such checks before merging?

@Necr0x0Der, first goal is to pay attention to new Bindings representation. In particular I suggest closing #127 because new representation allows representing correctly result of matching (x x) and (y z) as { x = y = z }.
Second goal is to agree on change in unit tests behavior because new logic changes some of them.
In fact I already have second PR with changes to type checker which fixes incorrect type checker behavior which was found by @ngeiswei, but I wanted to merge this PR first to unblock #245.

Reviewing Bindings implementation is a super goal :-) But I agree it is time consuming and not sure how much value it has. But I think we need to agree on API which Bindings has and on relations it represents. To me it is important because before this all implementations we used was more or less kind of map Variable -> Value and it is not true for this one.

BTW your example in b2_backchain.metta  still doesn't work as far as I can see.