Support for getServerSideProps

Is your feature request related to a problem? Please describe.
When using getServerSideProps as opposed to getInitialProps results in a namespacesRequired warning, despite the fact that I am returning namespacesRequired inside the required props key
I don't know if this is factual or not, or if the warning is a red herring
Describe the solution you'd like
Ideally I'd like getServerSideProps to be supported.
Apologies if this is vague or unclear, or a completely unreasonable request.
Thanks in advance

NextJs 9.3 was released a week ago. None of the new data fetching methods are supported yet, but in general I'd lean towards getStaticProps over getServerSideProps. Typically it's better to associate translation changes with releases, rather than having changes at runtime.
The next-i18next package will be entirely rewritten once NextJs plugin support lands, as quite a few things have changed since its inception.

I'm kind of newbie so sorry for asking this question. As I've figured out your amazing library currently fully supports Next js 9.2, right?

It fully supports NextJs 9.3 as well, just not the new data fetching methods.

In NextJs 9.3, Page components can not have getInitialProps. How could it support 9.3 ?

That is entirely false. That'd be a breaking change, and would go against semver. Read the docs.

No deprecations are introduced and getInitialProps will continue to function as it currently does.

It fully supports NextJs 9.3 as well, just not the new data fetching methods.

May I know if there is any schedule for supporting getServerSideProps and getStaticProps?

I do not currently have much time for development on this project due to other factors in life. I don't see this changing for the next few weeks, at least.
As I've stated, next-i18next will be rewritten once plugin support (finally) lands. If you'd like to see this (and many other features) supported, please let the NextJs team know that you are waiting for plugin support!

Just noticed that you can use i18next inside getServerSideProps in the same way as you would do it in the express application, i.e. create & configure plain i18next instance and use it instead

@maxmalov Can you explain what you mean? We still have a dependency on the i18next browser detector middleware.

getServerSideProps is being called on the server-side only, consider the example
import i18next from 'i18next';

const createI18Next = (lang) => {
  // create & configure i18next instance
}

export const serverSideMethod = ({ params, req, res }) => {
  // detect lang from req somehow (probably can reuse express middleware)
  const lang = detect(req);
  const i18n = createI18Next(lang);
  const { t } = i18n;

  // do whatever you want with t
}

So later one can use serverSideMethod inside getServerSideProps and get already translated content. For now, it's just a theory. We're working on the AMP first app and have to reuse the existing 18next resources we use in the separate single-page app. So in a few days, we eventually find out whether it works or not

Yes, that approach will work if you want to roll your own implementation, but has nothing to do with this package. Let's keep discussion here on topic.

Any updates here?
It can be a workaround for now, or a way to silence

You have not declared a namespacesRequired array on your page-level component: Page. This will cause all namespaces to be sent down to the client, possibly negatively impacting the performance of your app. For more info, see: https://github.com/isaachinman/next-i18next#4-declaring-namespace-dependencies"

Thanks

Hey @pettomartino - no update from my side. You can always silence warnings by setting strictMode to false. Please do feel free to contribute if you so choose!

@isaachinman thank for so short reply. Gonna try to implement something then. Any hints?

@pettomartino Do you mean contributing to next-i18next? If we want to support getServerSideProps, we'll need to update the appWithTranslation HOC to also look for that data fetching method in addition to getInitialProps.

There is another library solution, but it needs you to change the library...
https://github.com/zeit/next.js/blob/canary/examples/with-i18n-rosetta/README.md

@pettomartino there is a demonstration that the i18n solution works in SSG. Hope it helps
https://github.com/zeit/next.js/pull/11841

since we cant use serverSideProps together with initialProps. we can make use of React FunctionComponent.defaultProps
here is the workaround that works for me, and it still works with initialProps (https://github.com/isaachinman/next-i18next/issues/615#issuecomment-575578375)

particular page
Page.defaultProps = {    i18nNamspacves: ['home-page'] }

main app / _app.tsx
`MyApp.getInitialProps = async (appContext: AppContextType) => {
const appProps = await App.getInitialProps(appContext)
   const defaultProps = appContext.Component.defaultProps

return {
...appProps,
pageProps: {
namespacesRequired: [...(appProps.pageProps.namespacesRequired || []), ...(defaultProps?.i18nNamespaces || [])],
}`

since we cant use serverSideProps together with initialProps. we can make use of React FunctionComponent.defaultProps
here is the workaround that works for me, and it still works with initialProps (#615 (comment))

particular page

Page.defaultProps = {
      i18nNamespaces: ['home-page']
}

main app / _app.tsx

MyApp.getInitialProps = async (appContext: AppContextType<Router>) => {
    const appProps = await App.getInitialProps(appContext)
    const defaultProps = appContext.Component.defaultProps
    return {
        ...appProps,
        pageProps: {
            namespacesRequired: [...(appProps.pageProps.namespacesRequired || []), ...(defaultProps?.i18nNamespaces || [])]
        }
    }
}

this works. thanks!

Is it correct that the defaultProps must be set on the result of the HOC:
This works for me:
const page = withTranslation('toto')(TotoComponent);

page.defaultProps = {
    i18nNamespaces: ['toto','bar']
}

export default page;

This does not:
TotoComponent.defaultProps = {
    i18nNamespaces: ['toto','bar']
}

export default withTranslation('toto')(TotoComponent);

I'm using this version, so that it picks up recursive namespaces from WrappedComponents when using HOC
MyApp.getInitialProps = async (appContext: AppContextType<Router>) => {

    const recursiveNamespaces = (component: NextComponentType & { WrappedComponent?: NextComponentType }, acc: string[] = []): string[] => {

        const ns = component.defaultProps?.i18nNamespaces;
        const a = (typeof ns === "string") ? [...acc, ns] : [...acc, ...(ns || [])];
        let wrappedComponent = component.WrappedComponent;
        if (wrappedComponent) {
            return recursiveNamespaces(wrappedComponent, a)
        } else
            return a
    }

    const appProps = await App.getInitialProps(appContext)
    const namespaces: any = recursiveNamespaces(appContext.Component)
    return {
        ...appProps,
        pageProps: {
            namespacesRequired: [
                ...(appProps.pageProps.namespacesRequired || []),
                ...namespaces]
        }
    }
}

@rparree I think you're commenting on the wrong issue. This issue is about getServerSideProps.

@isaachinman I was referring to the comment with the workaround submitted by @icrona above  (https://github.com/isaachinman/next-i18next/issues/652#issuecomment-644618517)

Sorry, what exactly is that workaround achieving?

With the solution in their comment you can use defaultProps to set the namespaces when using a component that uses getServerSideProps. My code looks for defaultProps recursively when using a HOC.

As far as I can tell, that doesn't achieve anything except slightly modifying the API. It won't enable SSG.

since we cant use serverSideProps together with initialProps. we can make use of React FunctionComponent.defaultProps
here is the workaround that works for me, and it still works with initialProps (#615 (comment))

particular page

Page.defaultProps = {
      i18nNamespaces: ['home-page']
}

main app / _app.tsx

MyApp.getInitialProps = async (appContext: AppContextType<Router>) => {
    const appProps = await App.getInitialProps(appContext)
    const defaultProps = appContext.Component.defaultProps
    return {
        ...appProps,
        pageProps: {
            namespacesRequired: [...(appProps.pageProps.namespacesRequired || []), ...(defaultProps?.i18nNamespaces || [])]
        }
    }
}

how would that work for typescript?

@isaachinman It does the trick for me. I am able to use a component like this (and it will be SSR)
export const getServerSideProps = async function (ctx: NextPageContext) {
  …
 return {
    props: {
      totoProp : ""
   }
 }
}

TotoComponent.defaultProps = {
    i18nNamespaces: ['toto','bar']
}

export default withTranslation('toto')(TotoComponent);

since we cant use serverSideProps together with initialProps. we can make use of React FunctionComponent.defaultProps
here is the workaround that works for me, and it still works with initialProps (#615 (comment))
1. particular page

Page.defaultProps = {
      i18nNamespaces: ['home-page']
}

1. main app / _app.tsx

MyApp.getInitialProps = async (appContext: AppContextType<Router>) => {
    const appProps = await App.getInitialProps(appContext)
    const defaultProps = appContext.Component.defaultProps
    return {
        ...appProps,
        pageProps: {
            namespacesRequired: [...(appProps.pageProps.namespacesRequired || []), ...(defaultProps?.i18nNamespaces || [])]
        }
    }
}

This approach breaks SSR for me.

To get all namespaces loaded in case none are defined on the page, you would have to adapt icronas solution like this:
NextApp.getInitialProps = async (appContext) => {
  const appProps = await App.getInitialProps(appContext);
  const { defaultProps } = appContext.Component;
  let pageProps = {}
  if(appProps.pageProps.namespacesRequired || defaultProps?.i18nNamespaces) {
    pageProps = {
      namespacesRequired: [...(appProps.pageProps.namespacesRequired || []), ...(defaultProps?.i18nNamespaces || [])],
    }
  }
  return {
    ...appProps,
    pageProps
  };
};

@mattcrn The next-i18next package will already load in all namespaces if you don't define any on the page. Let's keep this issue strictly about the support of getServerSideProps.

@isaachinman yes, but using icrona's solution will break this functionality since it will return an empty array for namespacesRequired and the package is simpy checking if namespacesRequired  is defined for loading all namespaces.
So, namespacesRequired: [] means no namespaces are being loaded.

@pettomartino Do you mean contributing to next-i18next? If we want to support getServerSideProps, we'll need to update the appWithTranslation HOC to also look for that data fetching method in addition to getInitialProps.

As far as I understand it, getServerSideProps is not part of the page component, so we would have to create a higher order function to wrap the pages getServerSideProps. Or is there a better way?

I think when this feature lands, it would be ideal to support both getServerSideProps and getStaticProps.  For most users of this library, local translations and staticProps will be fine, but the serverSideProps may be useful in projects that are using https://locize.com/ or something similar, where translations are managed outside of the project.

Any update since then ?
Looking forward to have this thing implemented. :+1:

@Divlo Feel free to submit a PR.

+1

since we cant use serverSideProps together with initialProps. we can make use of React FunctionComponent.defaultProps
here is the workaround that works for me, and it still works with initialProps (#615 (comment))

particular page

Page.defaultProps = {
      i18nNamespaces: ['home-page']
}

main app / _app.tsx

MyApp.getInitialProps = async (appContext: AppContextType<Router>) => {
    const appProps = await App.getInitialProps(appContext)
    const defaultProps = appContext.Component.defaultProps
    return {
        ...appProps,
        pageProps: {
            namespacesRequired: [...(appProps.pageProps.namespacesRequired || []), ...(defaultProps?.i18nNamespaces || [])]
        }
    }
}

Well, i thought it would work good, but it works with delay:

At the first loading, it does not load the skill-test namespaces, i need to reload it to get that namespace active and the proper localization strings.
Inspecting it for the first time we get this:

After reloading, we can find the required namespace:

Both namespaces are in the option, but just the first one, defined on _app.js by getInitialProps, is loaded.
Why does this happen?

@SalahAdDin This will be solved by #869.

Fixed in<EMAIL_ADDRESS>

Thanks a lot for this @isaachinman

@isaachinman Does the new release mean we must deleted the whole workaround for this matter and use the new hook? It is not so much clear for me.

@SalahAdDin Correct, there is no need for a workaround in v8.

Import the serverSideTranslations
import { serverSideTranslations } from "next-i18next/serverSideTranslations";
Now from the getServerSideProps, pass the ..(await serverSideTranslations(locale, ["common"])), with the props.
export const getServerSideProps: GetStaticProps = async ({
  locale,
  locales,
  query
}: any) => {
  return {
    props: {
      ...(await serverSideTranslations(locale, ["common"])),
    }
  }
};

Now add your language string inside
/public/locales/en/common.json
For example
{
  "home": {
    "Test": "Test"
  }
}

You can add more language strings inside the locales directory.
Restart your Next.js app. It will work.


Info panel type inference does not work

Describe the bug
Sometimes the Info Panel doesn't show correct type information.
To Reproduce
Steps to reproduce the behaviour:

Clone elmish repo
Run paket install, dotnet restore etc to setup the project.
Open elmish folder in VSCode.
Open cmd.fs. Click the function Cmd.OfFunc.attempt. The Info Panel shows the return type as Unknown.

Expected behaviour
Cmd<'msg> should be the correct return type as the screenshot shows (which is inferred correctly in the codelens).
Screenshots

Environment (please complete the following information):

OS: Windows 10 or CentOS 7
Ionide version: 4.2
VSCode version: 1.39.2
dotnet SDK version: 3.0.100
mono / .Net Framework version: None

Additional context
N/A

This seems to be working nowadays


WiFly Board doesnt send UDP Packets when sending to Gateway

Hi I tried to send some UDP Data to my Router via the Example Code. When I send Data to my own PC in the Network, the Data gets sent, but when I set the IP to xxx.xxx.x.1 No Data is transmitted. Wireshark doesnt see any Packets either. When I send some Data from my PC to the Router it works, and i see the Packages with Wireshark. What am I doing wrong?

@mrst0n3 did you get this working?

@harlequin-tech No, I just decided I do not care anymore and did not want this ticket to be in my open issues tab.


Fix issues with Disconnect method in WatsonTcpClient

Reset _Token upon disconnect. This addresses an issue where calling connect after disconnect would result in a TaskCanceledException being thrown.

This was observed when running the Test.Reconnect sample.

Check IsCompleted instead of Status for the _DataReceiver and _IdleServerMonitor tasks. This addresses an issue where the task Status may be WaitingForActivation instead of Running.

After fixing the first issue, I encountered a problem in the Test.Reconnect sample where calling Disconnect would result in an InvalidOperationException. This was due to the _DataReceiver task setting Connected to false. It turns out that the Status of the _DataReceiver task was WaitingForActivation when Disconnect was called, so the task was still active when Connect was called a second time.

Hello, Might this solve the issue when there's people connected and dont disconnect correctly or client side crash and server doesnt detect client is not anymore connected? I have a really heavy issue with this library in many projects, CPU gets high and we discovered is on WatsonTCP  AppendBytes (while loop). It keeps like looping even if there're not clients connected.. I\m getting desperate

is this related to this? https://github.com/dotnet/WatsonTcp/discussions/166
Thanks

merged into 6.0.8 manually


Version 2.2.0: Added blurChangeMode to TextFieldRenderer and DateFiel…

…dRenderer to tackle issue #18. Added eagerChangeMode to TextFieldRenderer to tackle issue #18. Added support for different DateResolutions to DateFieldRenderer. Added HtmlButtonRenderer to collection

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Add more languages for example.py

Why Japanese was not included though it is the original one? :smile:
And I added Korean because it's my mother language.

Thanks for this request.
Also it seems Japanese is breaking the bot :(


add LoadedBridgeCliConfig

Description
Describe the changes or additions included in this PR.
Test plan
How did you test the new or updated feature?

Release notes
Check each box that your changes affect. If none of the boxes relate to your changes, release notes aren't required.
For each box you select, include information after the relevant heading that describes the impact of your changes that a user might notice and any actions they must take to implement updates.

[ ] Protocol:
[ ] Nodes (Validators and Full nodes):
[ ] Indexer:
[ ] JSON-RPC:
[ ] GraphQL:
[ ] CLI:
[ ] Rust SDK:

https://github.com/MystenLabs/sui/pull/17729


DuckPlayer: Temporary Fix for Watch In Youtube

Task/Issue URL:https://app.asana.com/0/1204099484721401/1208531438887018/f
Tech Design URL:
CC:
Description:
There is an issue in the Frontend that causes allowFirstVideo to remain 'true' even if it shouldn't.  This issue is triggered by sending more than one User-Settings Message per webpage view.
Moving this into the enrollment conditional, will buy us time to fix the issue in the FE and C.S.S and release as part of the regular release train.

Steps to test this PR:

[ ] Set DuckPlayer to "Always Ask"
[ ] Go to https://m.youtube.com/watch?v=B_HSa1dEL9s
[ ] Tap "Turn On Duck Player"
[ ] Tap "Watch in Youtube"
[ ] Tap the first video from the "Recommended list below"
[ ] Confirm DuckPlayer Overlay appears again.

cc @shakyShane .   I've found a place where I 'reset' the values after enrolling in the experiment to make sure DuckPlayer is shown immediately if you're enrolled.   Moving this into a conditional stops sending this 'unexpected' update it seems...


add --version flag to fn start

e.g. fn start --version 0.3.690 to start a specific version of
fnproject/fnserver docker image
tested ^ and latest. I did not do a lot of validation, but we don't appear to
do much of the other args either (that's my excuse and i'm sticking to it!),
it will fail pretty obviously with 'image not found' if someone specifies the
wrong version and we can't too easily check that without registry calls (we
don't need that), though I concede we could check for semver, latest doesn't
fit that either and is most convenient to keep current behavior. alas.
closes #574

LGTM (and sorry for the long delay!)


[REQUEST] Don't redownload video if ID is the same but title different

Checklist

[x ] I'm reporting a feature request
[x ] I've verified that I'm running youtube-dl version 2021.05.16
[x ] I've searched the bugtracker for similar feature requests including closed ones

Description

WRITE DESCRIPTION HERE
If I download a playlist, and the title is changes but ID is the same, it will re-download the video. My requrest is that there's some option like --ignore-title-change so the file newcar-nepf8cMXRUs.mp4, already downloaded, won't be download as a separate file with newvideo-nepf8cMXRUs.mp4

Isn't this covered by --download-archive <your-text-file-name.txt> ?
e.g.
--download-archive youtube-dl-history.txt
line from file...
youtube Ve8h5EQ9cFY

@baffled-mart no idea didn't try. But that implies I have to export the playlist to a text file and update it accordingly as the playlist adds new items. On the other hand it's much easier to send the playlist URL to youtube-dl.exe directly. Unless the youtube-dl can also create such text file and allow me to update it easily...

Isn't this covered by --download-archive <your-text-file-name.txt> ?
e.g.
youtube-dl  "https://www.youtube.com/watch?v=Ve8h5EQ9cFY"  --download-archive youtube-dl-history.txt
line in youtube-dl-history.txt ...>
youtube Ve8h5EQ9cFY
Resulting in...

youtube-dl  "https://www.youtube.com/watch?v=Ve8h5EQ9cFY"  --download-archive youtube-dl-history.
txt
[youtube] Ve8h5EQ9cFY: Downloading webpage
[download] Setting up Vonets VAP11N Wi-Fi Bridge & Triple-Dragon USB adaptor has already been recorded in archive

@superbonaci
If your text file doesn't exist the youtube-dl will create it, if the file does exist then each download is appended.
some examples...
brightcovenew<PHONE_NUMBER>001
generic a8399956455a8839f38a0e0b3065fe52
dailymotion x810zjl
This works for both individual downloads and playlists (e.g. one line per video download, not one line for playlist).
While it may be a pita, you will have to retrospectively create your own history for all past downloads, but will be fine for anything in future.
I've found it very helpful, especially when the same video occurs in different playlists.

Here is a command I use regularly
youtube-dl -f mp4 --no-check-certificate --ignore-errors --restrict-filenames --download-archive youtube-dl-history.txt --add-metadata  --batch-file playlist.txt -o "%(uploader)s/%(upload_date)s-%(title)s_%(id)s.%(ext)s"
History is contained in youtube-dl-history.txt
List of playlists in playlist.txt
I hope this makes sense.

OP, you should always have the site-specific ID in the filenames to be able to find dupes without having to check literally every file.
My format for YT:
TITLE RESOLUTION #ID#.EXT

RESOLUTION could become NA, which is the case with audio, or if the data can't be fetched.
For ex:
Call of Duty - Black Ops 4 - All Weapon Reload Animations in 9 Minutes 854x480 #so_vULkIZFQ#.mkv

This way I can just search for #so_vULkIZFQ#. to find dupes of that.

Yeah but where do you insert the playlist url, say https://www.youtube.com/playlist?list=PLcetZ6gSk968l1s4WuxwyhiyEUmg5GOZC

@rautamiekka how do you get to use ID and check for it with a simple download command, say
youtube-dl.exe https://www.youtube.com/playlist?list=PLcetZ6gSk968l1s4WuxwyhiyEUmg5GOZC

@superbonaci
Even if I am downloading only one playlist I still use  --batch-file playlist.txt  (with just a single entry)
Example contents for one or more playlists,
The first line is all the playlists created on one channel.
The second and third lines are single playlists.
playlist.txt
https://www.youtube.com/channel/UC1KYlOjI34ob_hicXVBuukA/playlists
https://www.youtube.com/playlist?list=PLWKjhJtqVAbnqBxcdjVGgT3uVR10bzTEB
https://www.youtube.com/playlist?list=PLcetZ6gSk968l1s4WuxwyhiyEUmg5GOZC

@baffled-mart I see so you can put playlists inside the batch file. I though you could only put single video URLS. Could help if you explain better dude you give lots of things for granted.

Apologies for the formatting.  I also don't seem to be able to edit my posts!  Grrr.
"how do you get to use ID and check for it with a simple download command, say"
youtube-dl --get-id  https://www.youtube.com/playlist?list=PLcetZ6gSk968l1s4WuxwyhiyEUmg5GOZC > this-list-id.txt
youtube-dl --get-title  https://www.youtube.com/playlist?list=PLcetZ6gSk968l1s4WuxwyhiyEUmg5GOZC > this-list-title.txt
I do not know how to get one file with a nicely formatted id,title so I do it with two runs and merge the files.
The following gives a single line for each id and each title, which doubles file length and isn't so easy to read;
youtube-dl --get-id --get-title https://www.youtube.com/playlist?list=PLcetZ6gSk968l1s4WuxwyhiyEUmg5GOZC > this-list.txt

Very useful thanks.

@superbonaci
I see so you can put playlists inside the batch file. I though you could only put single video URLS. Could help if you explain better dude you give lots of things for granted.
Sorry chap.
Also, I think I misread this (so ignor previous post?)
"how do you get to use ID and check for it with a simple download command, say"
Not sure what you mean.

@baffled-mart basically you are doing that with the file youtube-dl-history.txt. I wanted to have an option to not redownload files with same id, like I said in the example:
youtube-dl.exe --no-duplicates https://www.youtube.com/playlist?list=PLcetZ6gSk968l1s4WuxwyhiyEUmg5GOZC

Except that that option doesn't exist, so the way you do it works the same.

Inside a batch file you can put...
a single video
https://www.youtube.com/watch?v=gdIskHlqRwc
a whole channel (obviously beware, could be thosuands...)
https://www.youtube.com/user/bbclearningenglish/videos
a single playlist
https://www.youtube.com/playlist?list=PLcetZ6gSk968eYHxKIIMe0N1yDQUfZx-G
all playlists for a channel
https://www.youtube.com/user/bbclearningenglish/playlists
Any combination of the above.
For what could be a very large number of videos like, https://www.youtube.com/user/bbclearningenglish/videos , I would always run the --get-title option to a text file first to find out how many videos I might be downloading and see if I do not want any.
Also be aware, as I am sure you are, that one video can be in more than one playlist on the same site and if you use the option --download-archive youtube-dl-history.txt (or whatever you want to call your history file) the video will only be downloaded once and placed from the first playlist it is found in.
I use the following output commands to keep playlists together, either
-o "%(playlist)s/%(playlist_index)s-%(title)s_%(id)s.%(ext)s"
or
-o "%(playlist)s/%(upload_date)s-%(title)s_%(id)s.%(ext)s"
I hope this helps and is a little clearer.  It is past my bedtime! :-)

The date options can be useful too (as only the video title has changed, the upload date should be the same).
Videos can be filtered by their upload date using the options --date, --datebefore or --dateafter. They accept dates in two formats:
Absolute dates: Dates in the format YYYYMMDD.
Relative dates: Dates in the format (now|today)[+-][0-9](day|week|month|year)(s)?

Examples:
Download only the videos uploaded in the last 6 months
$ youtube-dl --dateafter now-6months
Download only the videos uploaded on January 1, 1970
$ youtube-dl --date 19700101
$ # Download only the videos uploaded in the 200x decade
$ youtube-dl --dateafter 20000101 --datebefore 20091231

Here is a command I use regularly
youtube-dl -f mp4 --no-check-certificate --ignore-errors --restrict-filenames --download-archive youtube-dl-history.txt --add-metadata --batch-file playlist.txt -o "%(uploader)s/%(upload_date)s-%(title)s_%(id)s.%(ext)s"

History is contained in youtube-dl-history.txt
List of playlists in playlist.txt
I hope this makes sense.

If a video is removed from the playlist you still keep it on your computer right?
Also, is possible to put comments inside playlist.txt?
#List of music
https://www.youtube.com/channel/UC1KYlOjI34ob_hicXVBuukA/playlists
#List of movies
https://www.youtube.com/playlist?list=PLWKjhJtqVAbnqBxcdjVGgT3uVR10bzTEB
#List of shorts
https://www.youtube.com/playlist?list=PLcetZ6gSk968l1s4WuxwyhiyEUmg5GOZC

@baffled-mart I see so you can put playlists inside the batch file. I though you could only put single video URLS. Could help if you explain better dude you give lots of things for granted.

As far as ytdl (and by extension Python) is concerned you can give an infinite list of links at once. You'll only be limited by

RAM.
whatever bugs in Python or ...
underlying problems in the C lib[rarie]s you might find

, and if you're giving all the links in the command instead of a download list file (or a for loop), whatever the char[acter] limit the shell/terminal (in Window$ case the Command Prompt or PowerShell) has. I think you're only limited by RAM on non-Window$ if you force a diff[erent] limit, but in any case there's a pre-existing RAM-unrelated limit.
^ So, it's a good idea to use those list files, or at least in the case of YT give only the vid/playlist ID unless it's absolutely necessary for the goal, since ytdl defaults to YT if the link is incomplete, and creating entire commands to download something will make long strings of text (I know from 1st-hand experience since I've a long Batch file for the purpose).

If a video is removed from the playlist you still keep it on your computer right?
Also, is possible to put comments inside playlist.txt?
#List of music
https://www.youtube.com/channel/UC1KYlOjI34ob_hicXVBuukA/playlists
#List of movies
https://www.youtube.com/playlist?list=PLWKjhJtqVAbnqBxcdjVGgT3uVR10bzTEB
#List of shorts
https://www.youtube.com/playlist?list=PLcetZ6gSk968l1s4WuxwyhiyEUmg5GOZC

The status of a vid on the website doesn't matter as far as the already downloaded ones are concerned.
Yes, that's how the comments work.

Also be aware, as I am sure you are, that one video can be in more than one playlist on the same site and if you use the option --download-archive youtube-dl-history.txt (or whatever you want to call your history file) the video will only be downloaded once and placed from the first playlist it is found in.
With the option you provided you just sort videos by Author (channel), so is pointless to speak about sorting them by playlists. I think that's the best idea since there can't be more than one author for the same video ID.


Error while udating property 'src' of a view managed by "RCTPDFViewAndroid

I alway got the error "Error while udating property 'src' of a view managed by "RCTPDFViewAndroid".
May I know how to set local file?
let pdfpath = require("../../../pdf/en/0.pdf")
<PDFView ref={(pdf)=>{this.pdfView = pdf;}}
src={pdfpath}
onLoadComplete = {(pageCount)=>{
this.pdfView.setNativeProps({
zoom: 1.5
});
}}
style={styles.pdf}/>

Ok, I  found the issue. src only support absolute path. It can only view from device's local storage file.

now how to show dynamic path ?


Drivers: Sensor: Bosch Bmp390 merge into bmp388 driver

Bmp390 is a drop-in replacement for the Bmp388 with same registers and features but different
chip-id.

bmp388 = 0x50
bmp390 = 0x60

This patches add property to the bmp388 dts , the chip-id. And replace the fix check against a define with
a check against the device->data->chip-id. This implement with minimal code changes.
Additionally:
replace a manual bit-shift operation with safe sys_get_le24()
use cast for int16_t struct members when read the calibration data from the chip.
cleanup macros from tailing spaces and apply clang comply format to it.
Please review. Thanks.

From other in tree sensors a driver for multiple sensor of the same series is usually done by having one compatible "bmp3xx". You can peek at this sensor for example:
https://github.com/zephyrproject-rtos/zephyr/blob/40414f72e9fcec733b0ad4d046c3c042a579bb94/drivers/sensor/maxim/ds18b20/ds18b20.c#L367
Regarding the commits generally:
Most of the maintainers do not want pure formatting commits as it creates noise in the files history. If you need to change one line and can make its formatting more consistent, great. Otherwise most of the time do not bother with the formatting.

I see, - ok noted with the formatting. I will drop this patch.
and I understand the solution you pointed out. Give me a little time to change it.
@faxe1008 you recommend to change this Pull-request to a draft until I think be ready?

Since you did not open it as a draf, assignees are already there and will be getting notifcations I think. You can to make the process more transparent but does not change anything really.

sorry guys, something goes wrong. I pulled my main and updated my branch. only 4 files had been changed.
Let me fix this , sorry for the noise.


Maven Release

I added maven support in order to get builds easier :)
A jenkins server should be created to get development builds automatically :D

*bump

Thanks for this, I'll check it out shortly.

Have you checked it?

Just having some issues resolving the dependencies (not used Maven for a project before), sorry it's taking a bit longer than it should :P

No trouble :)

@Lmmb74 If you wouldn't mind, I'd appreciate it :)
I've imported the project as a Maven project into Eclipse, however I'm getting missing artefacts for bungeecord.jar and spigot.jar.

I am importing the project in Eclipse right now to check the problem.

Seems strange, I had no trouble importing.

@Lmmb74 Are you using the File -> Import -> Maven -> Existing Maven Projects method of importing? I've tried a couple of times, and I'm still getting the missing artifact errors for both com.namelessmc:bungeecord:jar:1.0-SNAPSHOT and com.namelessmc:spigot:jar:1.0-SNAPSHOT.

Yes. You are just importing the main one (the modular one ), right?

That will allow the dependencies to be located in the parent repositories.

I'm importing everything within the Nameless-Plugin-master folder

Then I'm out of ideas because I am not getting any trouble. Would you mind making a maven debug inside Eclipse and a maven clean install execution where you cloned the repo?

The error within the Maven debug is this.
Unfortunately I can't try a clean install as I don't have Maven installed outside Eclipse on this machine at the moment.

OOOOOOOOOOOOOOOOOOOOOOOOWWWWWWWWWWWWWWWWWWWWWWWWwwwwwwwwwwwwwwww. It's trying to get the dependencies from the repositories provided in the parent pom when it only has to resolve them from the other modules and shade them. I had this trouble before when I was making the maven project. I do not usually use Eclipse for maven :P

Gimme a minute and I will tell you how to solve it ;)

Fixed

Btw, changed typo cause I prefer tab-spacing format rather than 4-space format :).

Thanks, works perfectly! ;)

:D


Cache by host as well as the path ("url")

https://trello.com/c/vFPSSGb7/175-activate-continuous-deployment-for-5-more-apps
Previously we only used the path of a request as the cache key,
which leads to surprising outcomes, since the value of the cache
is an absolute URL. This stops overriding the Varnish default [2],
so we cache based on the host of the request, as well as the path.

For example, if a request to 'www-origin/foo' return a redirect
to 'www-origin/bar', we would cache this using '/foo' as the key;
a subsequent reques to 'www/foo' would then return the cached
redirect of 'www-origin/bar', instead of the correct redirect of
'www/bar'. Issues caused by this:

- In production, a cached 'www-origin' redirect would lead to a
network error for users, since 'www-origin' is behind a firewall.

- In integration, a cached 'www-origin' redirect would lead to a
401 Unauthorized response, since it requires basic auth to access.

We think this is likely the cause of many a transient Smokey test
failing in Integration, in steps where we click on a button and
get redirected. Note that Smokey, while does provide the required
basic auth credentials, it only does this for the 'www' domain [1].

[1]: https://varnish-cache.org/docs/3.0/reference/vcl.html
[2]: https://github.com/alphagov/smokey/commit/d8847c4d03b0c416cf9f5d29607184acd566c4fd#diff-57013262a6cdf1f0f6cb2c64aa762c68R49

Example of this problem in production

@kevindew yup, it doesn't seem to affect anything:

requests to various pages continue to work (CDN and -origin)
no noticeable change in machine stats for *frontend* machines

Update: deployed, and I'm no longer able to replicate this problem.
Testing:
# fill the cache
while true; do curl -I "https://www-origin.publishing.service.gov.uk/transition-check/questions?page=12&foo=bar1234"; done

curl -I "https://www.gov.uk/transition-check/questions?page=12&foo=bar1234"
...
location: https://www.gov.uk/transition-check/results


Adds support for a new Image variant and an accompanying src prop.

Hi there, I'm a big fan of the boring-avatars library, but need to be able to display either an uploaded user photo or a placeholder avatar, depending on whether the user has uploaded a photo or not.
To address this, I've added a new image variant that can be used with a new src prop to display an image. The Image variant supports both the round and square modes, making it easy to "upgrade" the avatar without changing the component.
I hope that others will find this new variant useful as well!

Can we get this merged in?

This should be merged in.


[Bug]: CI does not catch Junit test failures

Is there an existing issue for this?

[X] I have searched the existing issues

Description
When the PR is created in github multiple tests are run and if all of these tests run without failures then we can merge the changes to release branch.
But the Junit test failures are not accounted for this. Even though there are failures in JUnit this is not considered and one can merge the PR to release branch.
https://theappsmith.slack.com/archives/CPQNLFHTN/p1654535891273319

Findings :
The CI was skipped due to the incompatibility issue between surefire plugin and junit 4. So only the tests using junit 5 were run and junit 4 dependent tests did not run. Hence they were not caught in CI.

1 PR still pending, opening again.

Cannot be verified


whisper failure

failing in regression as of 2/15/24
@vtangTT will do some bisect analysis

After bisect, this is the commit that broke it (feb 14)
#4003: use faster TMs in optimized ttnn whisper
a41f2ffb4c03836e5b7d4a5ec3e6a934c58a1588

@arakhmati

has this been fixed?

The test is fixed


Privacy setting require_terms

When testing the require terms setting using the latest Dev code, if checked in Admin > Settings > Login & Register > Privacy Settings (tab) for Comments the value for require_terms in the options table is set to 0,y and for Newsletter is set to 0,y,y, and when that page is reloaded the setting is not detected and therefore the boxes are not checked. Should both those option values be either y or n?
If 'require terms' is checked in the Comments module settings then the value is 'y'.
If 'require terms' is checked in the Newsletter module settings then a new option record is added with the value of 'y' for option group mw-main-module-backend. Surely this should be newsletter?
For the Contact Form module require_terms is saved with the option group value of contact_form_default - ie it includes the template name. Surely this should be just contact_form?
Also should the contact form module include a privacy_settings.php file to be consistent with the comments and newsletter modules?

The option value is changed from y to 0,y,y or 1,y,y in file \userfiles\modules\microweber\api\options.js around line 140 where it fetches multiple elements with the name require_terms. However adding data-value-unchecked="n" data-value-checked="y" to the checkbox input tag eg in \userfiles\modules\comments\privacy_settings.php seems to fix the value issue.

Also the user can submit the contact form without completing name, email, and message, there only appears to be validation on the security code.
After the form has been submitted there is a modal confirmation message, but there seems to be no way to close it. It would be good it the dialog could be closed by clicking outside.

Hopefull the Privacy settings update fixes the above noted issues. However the inconsistancy between contact form option group names still prevails; Should the default contact form option group be contact_form_default or contact_form?


Implement shard API for PostgreSQL metastore

[ ] open_shards
[ ] acquire_shards
[ ]  list_shards
[ ] delete_shards

@fmassot, @kamalesh0406 is going to give you a hand for this one. There are four calls to implement: open_shards, acquire_shards, list_shards,  and delete_shards. Let's open one PR for each so it's easy to split the work between you. Please leave a comment here and update the to-do when one of you starts working on a new one.
Implementation pointers:

there already exists a reference implementation for the file-backed metastore
the test suite for the Shard API lives in this file: please fill in the blanks and add more tests as you see fit.

I am starting the work for the open_shards @fmassot

I added the implementation for list_shards(unit test pending :) ) to my existing PR as well. I will see if I can get delete/acquire_shards done over the next couple of days.

Closed via #4403.


Improve multimonitor support and Display

We need to improve multi-monitor support and Display.

Test with multi-monitor with different DPI in case "per monitor dpi"
Add wxVideoMode property in Display
Add event void OnDPIChanged(wxDPIChangedEvent& event);
Add event void OnSysColourChanged(wxSysColourChangedEvent& event);
Was added? virtual bool ShowFullScreen(bool show, long style = wxFULLSCREEN_ALL) wxOVERRIDE;

Related info:
https://docs.wxwidgets.org/3.2/overview_high_dpi.html
Related images:

When dpi is changed-> incorrect TabControl resize

We need to move dip to pixel multiply code from c++ to c#


OrderService Crud?

Hİ,
The OrderService microservice is described as a simple CRUD API in the table of services description, but nowhere in the code there exists any crud operation. The service  just gets the orders from the Virtual Customer microservices and publishes them onto the pubsub event bus as far as I can see.
Is there anything that I am missing? Could you please clarify?

Hey @feveden, that is correct. VirtualCustomers, today, is just used as a simple load generator to drive the dashboard metrics. OrderService is the entrypoint into the system in which you could easily stand up a PowerApp or web-based order entry UI in front of to handle human transacted orders. The OrderService is responsible for taking a CustomerOrder model (which doesn't contain attributes such as UnitPrice) as input and transposing it into an OrderSummary (after doing a product lookup) that the rest of the services utilize. So, you are correct in noting that the OrderService doesn't contain the Read, Update and Delete portions of CRUD.


PyStemmusScope_Octave Runing return an error

@SarahAlidoost @BSchilperoort When i used PyStemmusScope on Crib to run the Octave version (since the Matlab Runtime is not yet ready on Crib), i encountered the following error
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
Cell In [6], line 3
      1 # setup the model
      2 # here you can change the location and start & end time
----> 3 config_path = model.setup(
      4     Location="ZA-Kru",
      5     StartTime="2001-01-01T00:00",
      6     EndTime="2001-01-02T01:30",
      7 )
      9 # new config file genertaed to run the model
     10 print(f"New config file {config_path}")

TypeError: setup() got an unexpected keyword argument 'Location'

It seems you are not using the latest version of PyStemmusScope. Please install the latest i.e. version 0.2.0, see https://pypi.org/project/PyStemmusScope/0.2.0/ . Also, see the release note of stemmus_scope https://github.com/EcoExtreML/STEMMUS_SCOPE/releases

It seems you are not using the latest version of PyStemmusScope. Please install the latest i.e. version 0.2.0, see https://pypi.org/project/PyStemmusScope/0.2.0/ . Also, see the release note of stemmus_scope https://github.com/EcoExtreML/STEMMUS_SCOPE/releases

Hi Sarah, i am not sure if this is an idea to put an comment or error/reminding message that suggest users always to update it to the latest version?


feat: generate profiles for smart charging

Pull Request doc
Purpose
What is the larger goal of this change?
To add the Python translation of the smart charging code
What the code is doing
How is the purpose executed?
This calculates the smart charging profiles for either LDVs or LDTs by iterating through the NHTS data. For each vehicle, all of its trips will help determine the optimized charging by considering various factors such as dwelling activity and charging costs.
Testing
How did you test this change (unit/functional testing, manual testing, etc.)?
This has not been tested yet but once it's been optimized to run faster, we can compare the output against the matlab output.
Where to look

It's helpful to clarify where your new code lives if you moved files around or there could be confusion/
The new code is located here: "PreREISE/prereise/gather/demanddata/transportation_electrification/".

What files are most important?
There are two new files. smart_charging.py is the main file with the algorithm. dwelling.py contains helper functions to calculate the charging power during a vehicle's dwelling period.

Usage Example/Visuals
How the code can be used and/or images of any graphs, tables or other visuals (not always applicable).
Time estimate
How long will it take for reviewers and observers to understand this code change?
It could about a few days for the reviewers to understand this code change.

It would be helpful to assess the validity of the input parameters, e.g., LDT vs ldt, etc. Raising some ValueError would be helpful to guide the user.


All filters creating a red image

I'm running XCode 7.3.1. I've triple checked I've followed the set up instructions.
Any filter I run, via any method (filterWithOperation, filterWithPipeline, or using PictureOutput()) the outputImage is always just a red fill.
Maybe I'm doing something wrong, but maybe others are running into this?

The plot thickens, it seems once I call .filterWithOperation twice on the same source image, the filter works. If I call it just once, the output is red.

This should be fixed now. The PictureInput was triggering an automatic reprocessing when it was attached as a target, but before it had been told to do so, leading to an ordering problem. I added a check around this in the latest commit, which seems to work on Mac, Simulator, and iOS devices.


Investing using Clang for Window builds.

The current compiler toolchain is capable of building for Windows. We need the appropriate SDK setup.

Taking at least temporarily, since I'm working on testing locally to see if there are any source-level or GN-level issues that need to be addressed to support clang builds.

At the moment clang builds are working for me with a manually installed toolchain, so for now we're just waiting on the clang binary being available. (Things may regress before then, since we don't have bots building that configuration, so I'll recheck regularly.)

This was fixed by the PR above, I just forgot to close the bug.


examples/windowsize: FPS doesn't reach 120 Hz after entering fullscreen by a key

Ebitengine Version
00e67b841996aa45ec3b8824549bd2ad30a1573f
Operating System

[ ] Windows
[X] macOS
[ ] Linux
[ ] FreeBSD
[ ] OpenBSD
[ ] Android
[ ] iOS
[ ] Nintendo Switch
[ ] PlayStation 5
[ ] Xbox
[ ] Web Browsers

Go Version (go version)
go version go1.21.6 darwin/arm64
What steps will reproduce the problem?

FPS reaches 120.
What happens instead?
FPS cannot reach 120.
Anything else you feel useful to add?
                    'c.
                 ,xNMM.
               .OMMMMo
               OMMM0<EMAIL_ADDRESS>     .;loddo:' loolloddol;.      -------------------------------------
   cKMMMMMMMMMMNWMMMMMMMMMM0:    OS: macOS 14.2.1 23C71 arm64
 .KMMMMMMMMMMMMMMMMMMMMMMMWd.    Host: Mac15,6
 XMMMMMMMMMMMMMMMMMMMMMMMX.            Shell: zsh 5.9
 kMMMMMMMMMMMMMMMMMMMMMMMMWd.    Resolution: 1512x982
 .XMMMMMMMMMMMMMMMMMMMMMMMMMMk   DE: Aqua
  .XMMMMMMMMMMMMMMMMMMMMMMMMK.   WM: Quartz Compositor
    kMMMMMMMMMMMMMMMMMMMMMMd     WM Theme: Blue (Light)
     ;KMMMMMMMWXXWMMMMMMMk.      Terminal: vscode
       .cooc,.    .,coo:.        CPU: Apple M3 Pro
                                 GPU: Apple M3 Pro
                                 Memory: 2690MiB / 18432MiB

This also happens with v2.6.3.
Pressing the green button causes an expected result.

This change also fixes an issue where a mysterious blinking on the upper side of a window. I failed to create a minimal reproducible case.

The cause might be the difference of FPS (60 vs 120), not architectures. Let's investigate this.

Refresh rates didn't matter. I've tested only MacBook Pro 2020 and MacBook Pro 2023 (M3), and it seems pretty hard to detect the true factor. Architectures (GOARCH) might not be the true factor but this seems to work...


Figure 1 in higher resolution available?

The quality of the image is very low... Do we have it in higher resolution?

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

ID: 43c84773-337e-9df7-7fbd-fc9d94280711
Version Independent ID: 223a78a0-67fd-8b96-be3f-153bb400101b
Content: Microsoft Teams call flows - Microsoft Teams
Content Source: Teams/microsoft-teams-online-call-flows.md
Service: msteams
GitHub Login: @CarolynRowe
Microsoft Alias: crowe

@BenK89 Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

@BenK89 Thank you for raising this issue! I'll be looking into it!

@CarolynRowe I have made the replacement images that are requested in this issue but am unable to upload them to the media folder. Would you be able to assist in this?

@CarolynRowe Hi could you please assist with this issue? Thanks!

@CarolynRowe Hi could you please assist with this issue? Thanks!

@CarolynRowe Hi could you please assist with this issue? Thanks!

Can someone provide the updated graphics?

@CarolynRowe Yes, the first 2 images in the document have a much lower resolution than the rest of the images. I recreated them. Here are the links for them:
Image 1:  https://i.imgur.com/7u4OumQ.png
Image 2:  https://i.imgur.com/YPjwrMZ.png

@CarolynRowe Hope everything is going well. @TokyoScarab has provided the images.

@CarolynRowe Just checking in on this and getting the images changed out. Thanks!

@CarolynRowe Just checking in on this and getting the images changed out. Thanks!

@CarolynRowe Just following up on this. Thanks!

@CarolynRowe Just following up on this, thanks!

@CarolynRowe Any news?

@CarolynRowe Just following up on this, thanks!

@CarolynRowe Just following up on this, thanks!


fix(libsinsp): ignore maybe-uninitialized for valijson

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area API-version

/area build

/area CI

/area driver-kmod

/area driver-bpf

/area driver-modern-bpf

/area libscap-engine-bpf

/area libscap-engine-gvisor

/area libscap-engine-kmod

/area libscap-engine-modern-bpf

/area libscap-engine-nodriver

/area libscap-engine-noop

/area libscap-engine-source-plugin

/area libscap-engine-savefile

/area libscap-engine-udig

/area libscap

/area libpman

/area libsinsp

/area tests

/area proposals

Does this PR require a change in the driver versions?

/version driver-API-version-major

/version driver-API-version-minor

/version driver-API-version-patch

/version driver-SCHEMA-version-major

/version driver-SCHEMA-version-minor

/version driver-SCHEMA-version-patch

What this PR does / why we need it:
Building libs with -Werror results in maybe-uninitialised in the valijson adapters. This fixes it.
Which issue(s) this PR fixes:

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

NONE

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: therealbobo
Once this PR has been reviewed and has the lgtm label, please assign molter73 for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

LGTM label has been added.  Git tree hash: 3ec38d45436e66aa47f0c07a174d2ee936067015

/milestone 0.14.0

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, therealbobo
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP]

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

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, jasondellaluce, therealbobo
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP,jasondellaluce]

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


Added import for Spark Context

Added import for Spark Context so that methods such as reduceByKey may work.

Can one of the admins verify this patch?

Hm, but it already compiles right? You do not need these imports.

Actually no, it doesn't. This is what I got when I tried it without imports:
/mllib/fpm/FPGrowth.scala:131: value reduceByKey is not a member of org.apache.spark.rdd.RDD[(Item, Long)]
[error] possible cause: maybe a semicolon is missing before `value reduceByKey'?
[error]       reduceByKey(partitioner, _ + _).

The code plainly compiles in master, as you can see from Jenkins builds. What are you compiling?

Oh! I see. I might have messed up a few files between different versions on my end.
I'll recheck the code on my end. Sorry for the useless request.

OK. Mind closing this PR?

Right away.


Update Character Generator to be able to create characters of type IB…

…1 and RAW

Starting this now. Will update the character extractor and AN scripts to handle the new character types

Think it's about ready. I'm going to merge this in soon and get to work on the guide how-to


error: flake 'git+file:///Users/shelbernstein/mojo-playground.nix' does not provide attribute 'packages.aarch64-darwin.default' or 'defaultPackage.aarch64-darwin'

HI, thank you for building this package.
When i try to git clone and then "nix build", i get this error
error: flake 'git+file:///Users/shelbernstein/mojo-playground.nix' does not provide attribute 'packages.aarch64-darwin.default' or 'defaultPackage.aarch64-darwin'

was wondering if you had any suggestions? both chatgpt and nixos-wiki turned up blank
Thank you :)

Sorry @adnanwahab, I only noticed this issue now. Which OS/architecture are you on?

no worries, i resolved it (didnt realize mojo only supported amd64 arch and not ARM/risc)


Not possible to use flatpickr with no theme

I'd like to use the default all white styles, but it seems like a theme is required (since it defaults to dark).   Is there a way to use no theme?

@jgadbois I think you can set theme to default

Just kidding. I will release a fix for this. Please stand by.

Thanks!  I actually tried that first, should have mentioned that.

Released 1.1.4 which should fix this

Thanks!  It is still showing the dark theme for me with:
  var app = new EmberApp(defaults, {
      flatpickr: {
        theme: null·
      }
      // Add options here
    });

Could your default be overriding null?

Scratch that i didn't update via npm yet

There is no default anymore. It just uses the vanilla flatpickr.css when there is no theme specified. You should be able to remove the config entirely.

Works great, thanks!


HAWKULAR-1121 Fix wrong path on MetricsAlerting itest

This PR needs a release of metrics and alerting.
The bug on the itest masked some issues on the intregation side.
I will update the PR once these releases are out.
Please don't merge as pom.xml is provisional.

This one depends on https://github.com/hawkular/hawkular-metrics/pull/600, https://github.com/hawkular/hawkular-metrics/pull/601, and https://github.com/hawkular/hawkular-alerts/pull/203
Also, new releases are necessary to include those fixes.

Updated pom.xml with metrics 0.19.2.Final and alerting 1.1.4.Final

/cc @stefannegrea @jshaughn
This fix is ready to merge unless there are any other issue.

@lucasponce Sorry, I don't have permission to merge.


[FALSE-POSITIVE]

Domains or links
[<IP_ADDRESS>]
More Information
How did you discover your web site or domain was listed here?
Incorrectly marked as Phishing
Have you requested removal from other sources?
NA
Additional context
Blacklisted after heavy RAM load which has now lowered
Send a Pull Request for faster removal
Users who understand github and creating Pull Requests can assist us with faster removals by sending a PR to mitchellkrogza/phishing repository, on the falsepositive.list file

https://github.com/mitchellkrogza/phishing/blob/main/falsepositive.list
Please include the same above information to help speed up the whitelisting process.

IP is not listed by us
https://www.virustotal.com/gui/ip-address/<IP_ADDRESS>


typescript 2.0 @type types not being loaded

when using typescript 2.0 @type definitions from npm, gulp-typescript is failing to recognize types in the node_module @type directories.  I've created a demo repo here that shows the problem:
https://github.com/mrohr/gulp-typescript-2.0-types-demo
when running tsc the index.ts compiles fine, but running gulp will result in this error:
src/index.ts(1,11): error TS2304: Cannot find name '$'.

The command line (tsc) is loading ALL the @types folder automatically without asking you...
gulp-typescript doesn't, try to use import

I'll take a look at this issue this weekend.

@gilamran using import isn't working for me. If I have
import * as fs from 'fs';

it doesn't work even after doing npm install @types/node

my workaround if anyone needs it:
gulp.src([
  paths.source + '**/*.ts',
<EMAIL_ADDRESS><EMAIL_ADDRESS>.pipe(ts(project));


Make error message consistent when invalid script is passed to -File, better error when passed ambiguous arg

Since -File is now the positional parameter for powershell.exe, made the error message consistent with
-Command when passed an invalid file.
If ambiguous arg is passed, give a better error message:
PS C:\users\poker\repos\PowerShell> powershell -no -file foo
Invalid argument '-no', did you mean:

        -nologo
        -noexit
        -noprofile
        -noninteractive
PowerShell v6.0.0-beta.5-42-g3ed9a816b2452551aaad1ac1245a1cbb35e87612-dirty
Copyright (C) Microsoft Corporation. All rights reserved.

Enable -WindowStyle to work
Fix https://github.com/PowerShell/PowerShell/issues/4351

Not sure why some of the new tests fail in AppVeyor but not on my desktop

No host?

Figured out why it's failing on non-Windows.  It appears that exit codes from programs have to be in the range 0-255 and anything greater than 255 is mod 256.  I verified this with a simple console c# app.  In this case, the value for ExitCodeBadCommandLineParameter mod 256 == 0.  Hmmm.
Although a breaking change, I think we should conform to libc standards which means this error code should be 64 "command line usage error".  We should also update all the other error codes that powershell console returns.
In general, I don't think this should be a problem as typically 0 means success and non-0 means error so although a breaking change, I suspect it have limited real world impact.
cc @PowerShell/powershell-committee

@lzybkr are you referring to -WindowStyle and everything else as two commits?

@lzybkr sure, I can make that two commits in the same PR.

Split into 3 commits (Thanks @lzybkr for the tip on git add -p!)

@PowerShell/powershell-committee reviewed this and is ok with changing the exit codes

@iSazonov and @SteveL-MSFT  thanks as always!


Error with Server restart....pls help

/usr/local/bin/arkmanager: Zeile 592: 23162 Speicherzugriffsfehler  "$arkserverroot/$arkserverexec" "$arkserveropts" "${arkextraopts[@]}"
11:21:13: Bad PID ''; expected '23162'
11:21:13: exited with status 0
11:21:13: restarting server
11:21:13: Running /home/ark/ARK/ShooterGame/Binaries/Linux/ShooterGameServer TheIsland?MaxPlayers=100?Port=7777?QueryPort=27015?RCONEnabled=True?RCONPort=32330?ServerAdminPassword=#####?ServerPassword?SessionName=.....
11:21:13: Server PID: 15562
[S_API FAIL] SteamAPI_Init() failed; SteamAPI_IsSteamRunning() failed.
Setting breakpad minidump AppID = 346110
11:22:32: server is up

OK - so the server exited due to a segmentation fault, and the script automatically restarted the server, after which it successfully came up.
What happened when the server crashed?

The server starts regularly reassessed with this error. 1-2 times a day
The server looks every 30 minutes for updates.
The script automatically restarted the server.

What mods are you using?  Is anything special occurring just before the server restarts?

http://steamcommunity.com/sharedfiles/filedetails/?id=510590313
http://steamcommunity.com/sharedfiles/filedetails/?id=503245171
http://steamcommunity.com/sharedfiles/filedetails/?id=534838287
No nothing happens special

Mod 503245171 was last updated 6 months ago, says it is discontinued, and points to http://steamcommunity.com/sharedfiles/filedetails/?id=533175507 as a replacement, which itself was last updated 4 months ago.

why should it but be because when it comes only so irregular?
I'm going to mod uninstall and watch it once, thank you

Closing old issues.


docs: add missing dependency to typescript guide package.json

Typescript project setup guide includes installing @types/node package as a step but the dependency was missing in final package.json

That dependency is not really needed, that's why it is not there. We don't include it in any of our TS templates.
But maybe we should, I know it can have weird side effects for type inference if this package is missing (from other projects, not aware of any issues connected to crawlee in this regard). Thoughts @vladfrangu?

We probably should include node types by default in our templates, yea

I was trying to use Buffer and VS Code's intellisense was not showing it without installing @types/node.
Also pnpm install  warns about  @types/node not being installed as it's a peer dependency for ts-node.
It would just make it easier to use the provided package.json without having to manually install @types/node 😄


add delimiter option for solving read error

The example file(titanic.csv) is a csv file, but the delimiter is a semi-colon.
Added a delimiter option so that the sample code can run right out of the box.

thanks! I agree it's a good idea to have this mentioned in te samples :)


How to config Autoprefixer？

How to config Autoprefixer to make browsers >= IOS8?

Eject app and configure in "config/webpack.config.*.js"

@miraage Is this the only way?

@lewis617 unfortunately yes

Hope this tool will support this feature in the future

Where I can find "config/webpack.config.*.js"?

Where I can find "config/webpack.config.*.js"?

oufresh read from the top bro :-)


Strange issues with latest redis-parser

Since node-redis update to 2.6 brought redis-parser 2.0.0, we've been seeing a lot of errors like this:
{ [AbortError: Fatal error encountert. Command aborted. It might have been processed.]
  code: 'NR_FATAL',
  command: 'QUIT',
  origin: [ReplyError: Protocol error, got "8" as reply type byte. Please report this.] }

where the "8" there varies but seems to be a whole load of different values.
For the time being I'm going to lock down the redis dependency here to ~2.5.0 until we have time to investigate further and figure out what exactly is going on with the newer redis parser.

@jbt what commands and what options do you use?

I've been trying to narrow in on a base case to figure out whether the bug is here in redis-clustr, in node_redis, or in redis-parser. It's proving to be a little tricky to reduce down but as soon as I've got a better idea of exactly what's causing the problems then I'll open issues in the relevant places

Slowly beginning to track this down. It looks like the error is coming most often from a cluster slots command. More information as and when I can find it out.

Putting here mostly for my own reference - I'll pick this up tomorrow and open some proper issues:
It seems not to be an issue with redis-clustr. The following test case seems to throw the same error with<EMAIL_ADDRESS>but not<EMAIL_ADDRESS>although only very sporadically:
var redis = require('redis');
[
  // big long list of { host: blah, port: blah }
].forEach(function(c){
   var cli = redis.createClient(c.port, c.host);

   cli.on('error', function(err) {
     console.error(err);
   });

   cli.on('ready', function() {
     cli.cluster('slots', function(){});
   });
 })

With only one host in the list it seems to be ~10% of the time, and it'll error with a [ReplyError: Protocol error, got "3" as reply type byte. Please report this.] (where the "3" can be "5" or "\r" or many other things).
Occasionally I'll also see this:
/home/jt/Development/testfoo/node_modules/redis/index.js:710
    if (typeof command_obj.callback === 'function') {
                          ^

TypeError: Cannot read property 'callback' of undefined
    at normal_reply (/home/jt/Development/testfoo/node_modules/redis/index.js:710:27)
    at RedisClient.return_reply (/home/jt/Development/testfoo/node_modules/redis/index.js:816:9)
    at JavascriptRedisParser.Parser.returnReply (/home/jt/Development/testfoo/node_modules/redis/index.js:188:18)
    at JavascriptRedisParser.execute (/home/jt/Development/testfoo/node_modules/redis/node_modules/redis-parser/lib/parser.js:392:12)
    at Socket.<anonymous> (/home/jt/Development/testfoo/node_modules/redis/index.js:267:27)
    at emitOne (events.js:77:13)
    at Socket.emit (events.js:169:7)
    at readableAddChunk (_stream_readable.js:153:18)
    at Socket.Readable.push (_stream_readable.js:111:10)
    at TCP.onread (net.js:531:20)

I'll check up on what redis versions are doing this (pretty sure there're some servers in my test list running old versions) to make sure it's not a silly redis bug, and then open some issues tomorrow when I haven't been staring at it for hours.

The error is very likely not an error on your side. The parser error can pretty much just be returned if there's a mistake in the parser code (or Redis returned faulty data and this is highly unlikable). That's why the error explicitly asks for being reported in the error message ;-).
But currently I don't really know where to search for the error. Please outcomment line 330 of the parser, increase the stack trace limit to a high value (e.g. 20) in the ReplyError line 7 and activate the debug mode (NODE_DEBUG=redis). That output will hopefully help a lot finding the bug.
Ping @Salakar

Ok I tried commenting out the decreaseBufferPool interval and I'm still seeing both these weird errors.
I said yesterday that I could reproduce it with a single redis instance but I think I was wrong - it looks like it only happens with more than one client open. So it looks to me like it could be some sort of shared buffer being used by multiple clients.
Debugging output from two such cases here (for the Cannot read property 'callback' of undefined error) and here (for the reply type byte error).
In both cases all the redis servers in question were running redis 3.0.5, with the test script from my earlier comment. The error doesn't happen reliably but I sometimes have to run the script a few times before seeing the error.
Would you like me to open an issue / multiple issues on redis-parser for this? Happy to provide any other information that'll help track this down.

I also tried uncommenting the buffer_from_socket.toString() log in node_redis line 266 and that produced this output which might help. In that case I'm using two hosts (<IP_ADDRESS>:6380 and <IP_ADDRESS>:6380)

@BridgeAR I did warn you before that the bufferPool was being shared in the module scope and that multiple instances of the parser would cause conflicts ;p

I found the issue. I forgot that Buffer.slice() does not copy the buffer but instead returns a reference. Now if there are multiple parsers that receive data concurrently and the parser still holds some data from a former chunk that was sliced out of the bufferPool and the other parser uses the bufferPool too, the first parser holds faulty data in it's buffer from now on.

Glad I could help with tracking this down :) Would you like me to open an issue on node-redis-parser or shall I leave it to you to take it from here?

Oops, I forgot to close this when I published v1.5.0. All is good now :+1:


Introduce frequency_meter

#90

Waiting on https://github.com/tscircuit/schematic-symbols/pull/114 for better text support.


Integration with third party authenticator applications

Integration with third party authenticator applications

Authy
Trusona
Lastpass
Onespan
Deepnet

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


[Unnamed Game] I am trying to install KakarotModManager

It tells me that the game is null even though I own the game. I even put the mod manager in the directory of the exe file on steam.
KMM Info:
Version: 1.0-4
Args:
StartDir: C:\Program Files (x86)\Steam\steamapps\common\DRAGON BALL Z KAKAROT
Process Level: Administrator
Game: Unnamed Game
Exception:
Type: NullReferenceException
Message: Object reference not set to an instance of an object.
Source: KakarotModManager
Function: System.Collections.Generic.List`1[KakarotModManager.SteamGame] SearchForGames(System.String)
StackTrace:
at KakarotModManager.Steam.SearchForGames(String preference)
at KakarotModManager.App.Main(String[] args)
InnerException:

Forgot Steam made some changes to some files. Does this work?
KakarotModManager.zip

Works! thank you! You are awesome!


Best practice to enable HTTPS for s3ninja

Hi, our client app (java) is forcing HTTPS to S3 so in order to mock it I must provide HTTPS support.
I can go with local http server that proxies the requests to localhost:9444 but thought maybe you have another option to do that?

sirius-web now supports HTTPS / SSL:
add this to application.conf:
http {
ssl {
enabled = true
port = 9443
keystore = "Path to .jks file"
password = "Password for keystore"
alias = "Alias for the default domain"
}
}


Cannot call a class as a function

Hello i am trying to follo your example at https://github.com/styled-components/styled-components/blob/master/docs/tips-and-tricks.md#media-templates
I have a copy of this code

importing it like

and using like this

I am getting error:

I am using version:

We can't provide support for inferno-compated apps I'm afraid. That definitely sounds like a problem with your setup, as that code works perfectly fine in my standard React app.


Add BEPUphysics

Game/BEPUphysics - https://github.com/bepu/bepuphysics1
BEPUphysics is a pure C# 3D physics library by BEPU.

Hey, thanks for the PR. It looks like v1 of the engine is there only for historical records and all development effort is in v2 - https://github.com/bepu/bepuphysics2 . Could you please update the PR with the v2?

Hey @zihotki
The reason I added the version 1 ist that the 2nd version is still in alpha. Should I anyways updtae the PR?

@fxredeemer I think it's better to do that way, eventually it'll hit the release. And we don't make decisions about projects based on whether it's in alpha stage or RTM. It's up to the user decide about whether it's stable for him


[abap2xlsx] - Issues with image and formula

Hi,
I would like signal two issues.

Image in a cell.
When I insert a image in a cell, I can't decide the distace from top and from the side of cell.
In case I have a cell with edges drawn, the image covers the lines of the edges.

Formula in a cell.
I save a file with a formula in a cell, after, I open that file and I decide to close it, I display a popup for save the change, but,I don't have changed anything.

I wrong anything?
Thank you all in advance.
Alessandro Ieva
PS.
Congratulation for abap2xlsx. It is very helpfull :)

Sorry for late answer.

You may use the parameters IP_ROWOFF and IP_COLOFF of the method SET_POSITION. If you copy the program ZDEMO_EXCEL16 and add the 2 parameters as shown below, you will see a little shift of the image (NB: big numbers for little shift):  lo_drawing->set_position( ip_from_row = 3
                            ip_from_col = 'B'
                            ip_rowoff   = 100000
                            ip_coloff   = 50000 ).

Usually, Excel doesn't ask to save. Do you have an Excel file that we can check?
Thank you.

No answer. Closing.


Grpc.Net.Client : "Received message exceeds the maximum configured message size."

When orchestration output  exceeds ~3MB "message exceeds..." exception is thrown.
Is there a way to change the limit?
Current hack to make work:

I'm sure there is a better way to do this.
Running on Azure Functions .NET 7

I think this is a limitation of the gRPC channel between the durable client and the durable extension running in the host.
We might need to fix this by updating the host to increase the max gRPC message size. Ideally, we could use the same configuration that exists for the Functions host gRPC message size.
Java is likely impacted by this issue as well.
@jviau, would you be able to help with this? Hopefully it's a simple fix?

Closing as won't fix as per my comment above.


Partial class support in runtime templates

Hello MrJul,
I hacked together a very raw support for partial classes in runtime templates.
Essentially, I've switched one limitation, no partial class support, for another - members defined in .tt are now reported as duplicates (except for TransformText, which is generated with different name). For me the trade-off is ok, I have lots of runtime templates with partial classes and I tend to put all member declarations to partial classes anyway. Having at least some support is better than none at all.
I don't know if you find it useful - you've most probably thought of the possibility already. Maybe you could align me in better direction of implementing partial class support? I could give it a second try.
Thanks for your great work,
O.

This plugin has been forked and is now maintained by JetBrains here, so I'm closing this PR as it's no longer relevant. Thanks!


Time Entry Autocomplete

Implement an searchable autocomplete feature for charge code, client and project dropdowns on time entry table.

Implement a custom form control component that looks like the HQ input and acts like an autocomplete.
It should have an input for items and use content projection for option templates.

https://select2.org/selections#templating


Bug#4706 add event.cancelable check

Please do not update the README or Change Log, we will do this when we merge your PR.
This PR (delete as applicable)

Fixes a bug

Describe the changes below:
added event.cancelable check to if statement in this.onTouchStart, this.onTouchEnd and this.onTouchMove to fix #4706

Is there anything you need me to do to move this along?

👍


More flexible notification options

Integrity by default notifies after every build. We think this is too much spam, and have a local patch to notify on failure only. It is trivial (+1 line in each notifier we use) but not quite ideal: what I would really like is notification on every failure and a notification on recovery (that is, success after failure).
Thinking about upstreaming these changes I realized that different notifiers may need different notification policies. For example, a notifier that updates a status somewhere might want to be called on every build, whereas a notifier that sends emails in the same project would only be called on a change of status.
Therefore I came up with the following additional options per notifier:
Notify on success:

Always
Never
After failure

Notify on failure:

Always
Never
After success

(The bullet points are radio buttons.)
This should cover all cases I can think of. Thoughts?

I didn't see this come through.
I have an implementation of the same thing, but it is done outside of integrity - I would love to see it brought into Integrity. Let's do a pull request for it?


network connection: apply real back pressure during read disable

note: existing tests cover this case well.

@lyft/network-team @htuch this is a scary change so I will do a little smoke testing and report back.

smoke test looks good

@htuch updated. As long as I was making changes again, I fixed it so that when you make a file event you have to specify the initial watch events. It didn't make much sense that it was fixed to read/write at construction time.

LGTM


Setting intra flag

How to set -intra flag in FFmpeg-python?
I've tried setting it as a parameter on output but it doesn't work...

Can you provide more details about code you have tried and also if possible a ffmpeg command using an intra flag ?
If you are talking about setting fixed/close GOP , you can set it using specific codec params

Been trying to achive this as well
ffmpeg -i GH010281.MP4 -intra output.MP4
from: https://video.stackexchange.com/questions/16958/ffmpeg-encode-in-all-i-mode-h264-and-h265-streams
Not sure how the alias "-intra" should be handled, this does not work atleast
.output("output.MP4", **{'intra':[]}


Add "Library Construction Approach" and "Species" to get Data Summary

List the library construction approaches and species  in the query results.
We can add a +10 more in a future ticket when these get big.

Projects                                          1
Species                                           Homo sapiens
Library Construction Method                       Smart-seq2, 10X v2 sequencing
Donors                                            8
Specimens                                         8
...

Let long lists wrap to next line.


ValidateVar coerces values instead of validating them

Here: https://github.com/cranetm/yii2-json-rpc-2.0/blob/master/JsonRpc2/Validator/ValidateVar.php#L80-L93
This library is advertised as one with strict type validation, so it is expected that values will be validated, and not silently coerced.
Also, default case is omitted in that switch, is it intentional? Shouldn't it throw an exception if the type is not recognized?

Please, give an example with that "unrecognized" type.

If there is a typo or a misspelling in a type declaration ValidateVar does not perform any actual validation and just silently returns the original value.

I understand, this is a kind of "protection of Fools".
For some developers it can be an issue, i agree.
I will add corresponding exception, thank you for explanation.

Well, not only from fools. For example, int/integer, bool/boolean are acceptable in phpdocs, but only short forms will be "validated" here. It is not mentioned in the docs and can become a source of errors.
And what about strict type validation? If implemented it will be a BC break for sure. How do you think it should be done?
Also, on a related note, why not use Yii's \yii\base\Model as the base DTO class? Its validation facilities provide much better extensibility and error reporting and are already known by yii users.

About short and long forms i agree, them could be added to switch-case.

why not use Yii's \yii\base\Model
Because this library was developed(in my old project) for another framework and I adopted it for yii.
yii\validators\Validator has many interesting validations about dates, email, url, ip, match by regex, trim.
I'll look at this Model class, maybe it can be useful.


<EMAIL_ADDRESS>setScheduler not behaving as expected

It seems that setScheduler(new RAFScheduler()) does not behave properly because the number of job objects are out of sync with the DOM objects that are created (more DOM object are created that actually exists in the jobs stream). The only way I can have the list of jobs synced with the DOM is by using syncRAF.
import { defAtom, defCursor } from<EMAIL_ADDRESS>import * as rx from<EMAIL_ADDRESS>import { $compile, $list, setScheduler, RAFScheduler } from<EMAIL_ADDRESS>
// setScheduler(new RAFScheduler());

const db = defAtom<State>({ jobs: [] });
const jobs = defCursor(db, ["jobs"]);
const jobsView = rx.syncRAF(rx.fromAtom(jobs))
const jobCreate = rx.stream<Job>();
jobCreate.subscribe({
  next: ({ start, end, ...rest }) => {
    jobs.swap((prev) => [
      ...prev,
      { ...rest, start: new Date(start), end: end && new Date(end) },
    ]);
  },
});

tx.run(
  tx.sideEffect((x) => jobCreate.next(x)),
  getJobs(), // not implemented here returns Job[]
);

const root = document.getElementById("root")
$compile($list(jobs, "div", {}, (x)=> `${x.start}`)).mount(root)

closing this since, schedulers have been removed with v1.0.0 (back in January 2024) - see commit message above for details...


Please note the version id.

Version 2 still displays 1.0.0.
Gabor Szots, CCRL

Version 2 still displays 1.0.0.
Gabor Szots, CCRL

Thank you. I will update the binaries.

@SzotsGabor I have updated the binaries in the Svart 2 release. Let me know if the issue has been resolved.

Yes, it is now 2.0.0.
BTW, do you prefer to remain anonym? We like to include author name and country in our lists. It is completely your decision, it is not a requirement for your engine to be tested.
I see you are from Sweden. I guess Svart means sword, right? Just out of curiosity, I am fond of languages.

Yes, it is now 2.0.0.
BTW, do you prefer to remain anonym? We like to include author name and country in our lists. It is completely your decision, it is not a requirement for your engine to be tested.
I see you are from Sweden. I guess Svart means sword, right? Just out of curiosity, I am fond of languages.

Good guess! It means black in Swedish, svärd would be sword.
Regarding the name, I am fine with my first name being used, my name is Cristopher; but I would prefer to not reveal my last name. My name could be displayed as Crippa / Cristopher or just one of them, that is up to you. 🙂


TypeError: Cannot read property 'name' of undefined

Description
I'm getting the the following error.
I think the problem is that the index of inputTypesMap should be a string but in my case typeToInputTypeAssociation[type.name] is an array.
You can find the affected line here.
Steps to reproduce
I think it's reproducible by adding something like this to the datamodel.prisma:
type Team {
  id: ID! @unique
  members: [User!]!
  calendar: [Calendar!]!
}

Expected results
No error is thrown
Actual results
Error in my console:
(node:72993) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'name' of undefined
    at /Users/sapkra/Documents/git/server/node_modules/graphql-resolver-codegen/dist/generators/ts-generator.js:80:282
    at Array.map (<anonymous>)
    at Object.generate (/Users/sapkra/Documents/git/server/node_modules/graphql-resolver-codegen/dist/generators/ts-generator.js:80:10)
    at generateCode (/Users/sapkra/Documents/git/server/node_modules/graphql-resolver-codegen/dist/index.js:87:28)
    at /Users/sapkra/Documents/git/server/node_modules/graphql-resolver-codegen/dist/index.js:181:28
    at step (/Users/sapkra/Documents/git/server/node_modules/graphql-resolver-codegen/dist/index.js:33:23)
    at Object.next (/Users/sapkra/Documents/git/server/node_modules/graphql-resolver-codegen/dist/index.js:14:53)
    at fulfilled (/Users/sapkra/Documents/git/server/node_modules/graphql-resolver-codegen/dist/index.js:5:58)
    at <anonymous>
(node:72993) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:72993) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

After that I also checked the content of the variables.
type.name = 'Team'
typeToInputTypeAssociation[type.name] = [ 'UserWhereInput', 'CalendarWhereInput' ]
Versions

graphql-resolver-codegen: 0.3.0
OS name and version: Mac OS 10.14

I'm having this issue as well

Can you check whether you still have this problem with the latest version of graphqlgen?

I can confirm that this is still and issue, this PR fixes it: https://github.com/prisma/graphqlgen/pull/135

I am also getting this error: TypeError: Cannot read property 'name' of undefined
<ion-item>
        <ion-label>Name</ion-label>
        <ion-input [(ngModel)]="item.name" placeholder="Name"></ion-input>
      </ion-item>

although this works:
{{item?.name}}


Making trainingOption updater/getter less rigid

Description

I removed a lot of the hardcoded handling in _updateTrainingOptions, and instead have it iterate over all the trainingDefault options and set them if they exist. I didn't want to use a spread operator, as this keeps it limited to the options we want passed. this isn't a big deal IMO, and it could make this very simple to implement if we don't care about extra options.
Motivation and Context

This is in response to some comments not being submitted before #137 was merged
How Has This Been Tested?

yarn test - all passed
Screenshots (if appropriate):
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Author's Checklist:

[x] My code focuses on the main motivation and avoids scope creep.
[x] My code passes current tests and adds new tests where possible.
[x] My code is SOLID and DRY.
[x] I have updated the documentation as needed.

Reviewer's Checklist:

[ ] I kept my comments to the author positive, specific, and productive.
[ ] I tested the code and didn't find any new problems.
[ ] I think the motivation is good for the project.
[ ] I think the code works to satisfies the motivation.

just fix up conflicts and we're good to go


I need more than 10 seconds

When adjusting the subtitle timeline, I found that it can only be adjusted for a maximum of 10 seconds, and I hope that this limitation can be removed!

Hey @Hyacinth1013, thanks for your observation! You're absolutely right. With a single click you can only adjust the subtitles by 10 seconds at a time. Have you tried using the shortcuts g and h to sync the subtitles?


Updates API ref for jobs. Adds doc on Kubernetes Scheduler persistent

DO not merge this yet. @JoshVanL will add one more update.

@msfussell updated- please take a look!


Kong 2.0.3 hasn't yet been published on dockerhub

#62 introduces support for Kong 2.0.3, but this causes build issues if using master, as only 2.0.2 is yet available on dockerhub. Certainly, this issue can be avoided by checking out a specific revision, but it might be more usable if this repo had tags for each supported Kong version.

Waiting on Docker to review / merge https://github.com/docker-library/official-images/pull/7750
In the meantime we publish unofficial Docker Kong images at kong/kong along with daily builds when we do releases
https://hub.docker.com/r/kong/kong/tags?page=1&name=2.0.3

@nhynes my bad, I merged the PR for 2.0.3 too soon. Will probably implement a fallback to an unoffical image (that doesn't have the review/merge lead-time) in the future to prevent this from happening again.
You can simply bypass by specifying the Kong version;
KONG_VERSION=2.0.2 pongo run

fyi fallback has been implemented. Just pull the latest Pongo.


CloudEventMessageBuilder creates URI with surrounding quotes

Currently we're migrating some of our messages to Cloud Events. If we use
CloudEventMessageBuilder.withData(this)
        .setId(UUID.randomUUID().toString())
        .setSource("/my/source")
        .setSpecVersion("1.0")
        .setType("my.type")
        .setHeader(KafkaHeaders.KEY, myId)
        .build(CloudEventMessageUtils.KAFKA_ATTR_PREFIX)

the resulting message apparently has a header ce_source="/my/source" and other clients which validate the incoming messages fail with "/my/source" not being a valid URI.
If I just set the header myself using .setHeader(CloudEventMessageUtils.SOURCE, "/my/source") everything seems fine.

@kschlesselmann it is fixed. Please give it a shot and let me know before we do 4.2.1

@olegz Sure, which release should I try? Is the BinderHeaderMapper included in the reactive binder? I tried to upgrade to org.springframework.cloud:spring-cloud-stream-binder-kafka-reactive:4.2.1-SNAPSHOT but this doesn't seem to be enough.

You are too quick, since I believe the snapshot has not been generated yet. You can either build locally yourself OR wait few hours, but yes 4.2.1-SNAPSHOT would be what you are looking for

Actually I just noticed that the fix does not work with spring-cloud-stream-binder-kafka-reactive and looking at the code I am afraid that the header mapping and serialization is happening outside of the actual binder.
So I am reopening it and will dig some more to see if there is anything that can be done on our end

Good news, found the way to reuse the same BinderHeaderMapper, will push shortly

Awsome. Give me a ping and I'll build it on my machine :-)

@kschlesselmann give it a try. I tested with your app and it's working for me


fix: unconfirmed middle txs

Related to #799, #768

Description
Two bugs are fixed here:

The syncing column of addresses were never updated to false, meaning they were always syncing and that the syncing of missing transactions was not working
The syncing of missed transactions would skip unconfirmed transactions that happened to be caught in the middle of confirmed transactions. That only could happen because of the bug above, but I fixed so that even if that happens, it will set these txs to confirmed

#768 and #799 should both be fixed here, except for the part that mentions triggers (I am assuming this is about triggers being send after downtime, which will be implemented in another PR)
Test plan
This is already running in prod, so there should be no more old unconfirmed txs nor missing txs.
To test in local, one can manually set a old tx in the db to confirmed=0, or delete an old tx and then reset ONLY the paybutton-dev container and see if it was fixed. This reset would probably need to be done twice, since the first time would only fix bug 1. (set the syncing flag to false) and the second time would fix bug 2. (sync the old txs)

Changed to draft for debugging, tests are failing in github but not in my local env.


🚀 Feature: CLI tool to communicate with Backstage

🔖 Feature description
A lot of developers work on their terminals. Having a CLI tool to communicate with features in Backstage would be super useful for the users. Extra cool feature would be to allow plugins to somehow register to the CLI tool with their own subcommand. Authentication is something that might be difficult based on the selected authentication provider, most likely some kind of API token should be generated for each user to bypass the default authentication used in the web version.
🎤 Context
The tool could utilize the core features and existing APIs to for example search from Backstage, list and see catalog items, run software templates, see TechDocs etc. directly from command line.
✌ Possible Implementation
No response
👀 Have you spent some time to check if this feature request has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
None

There's been a lot of discussion about this over the years 😅, also internally before we open sourced Backstage. It's really tricky to create and maintain this. Just like Backstage addresses the problem of a sprawl of internal web admin panels, it could also do the same for CLIs. I think the problem generally exists for CLIs too, but the solution likely looks very different.
We are probably inching closer to at least something like an OSS Backstage CLI, with the existing TechDocs CLI, and a CLI for the Scaffolder being discussed too. Making that extensible is a significant extra step though, and I'm not sure it's worth spending time there at this point.
@taras in case you want to chime in :grin:

I do :)
@drodil I advocated for a similar CLI in my talk about BackstageCon
It's a tricky problem to solve because we would want most of the functionality to be controlled by the backend; otherwise, you will push many updates to the CLI for small changes. There could be ways to around that, but we also want it to be low-maintenance for Backstage operators. kubectl is a nice CLI, but their CLI is a thin wrapper around the REST API that's very structured and has a schema. Most of the APIs on /api endpoint are not structured to the same degree and don't have schemas. All of this makes creating a good flexible CLI experience very tricky. It could be fairly easy to create a CLI that only works with core plugins, but we don't have a consensus on how to make the CLI extensible.
Long way of saying, I completely agree with you.

Just spit balling here, but what about providing building blocks for creating a Backstage CLI in the same way that Backstage is a platform for building developer portals? So rather then focus a complete solution, focus on exposing parts that allow org interrogators can use to build a CLI experience for their organization.
This would leave the structuring of commands/sub commands up to the integrator. As far as being opinionated, I think the building blocks would naturally need to be TS based. I think in many cases if a plugin has -cli or -node package that exposes the MyPluginClient for that plugin it could provide some of the heavy lifting. However, this also implies that some of existing frontend APIs might need to be node compatible or duplicated to work this way. In essence the overall approach would be to provide a pattern where plugins can expose modules that could be consumed by CLI instead of frontends. Anyways, obviously a fair bit of work any way you cut it ...

Let me know if you're interested in trying the CLI from my BackstageCon. I can put together instructions on how to set it up locally.
@andrewthauer I have some thoughts on how this might work, but I need time to organize might thoughts. I'll share them early next week.

Bump

Bump again. Any updates @taras ?

I'm still interested in some of the use cases this could unlock.
However, I still believe this is ideally achieved in a way that you can integrate into an existing CLI first. Probably through exposing APIs clients through isomorphic packages like the catalog client.

I'm still interested in this!

Bump

I was curious if I could write a CLI using the same constructs as the new backend system. Turns out you can and it's pretty powerful, modularity built in 😁 . The branch here.
https://github.com/backstage/backstage/assets/34432188/26c2ad2b-a5db-4b79-8551-abb957a8b30e
Demo above shows me using a catalog CLI plugin I wrote that calls out to the /entities endpoint and the /entity/by-ref/{ref} endpoint and showing the result. To get that to work, I created 2 new services, a rootCommanderService to store the commander instance that everything would be attached to and a commanderService that could be used in each plugin to write commands, prefixed with the pluginId as the namespace. For example, my catalog plugin would be given control over cli-builder catalog list and cli-builder catalog get <entityRef> and could define those commands as just .command('list') and .command('get <entityRef').
As you can see in the demo and the code, the CLI implementation doesn't quite work with the idea of the backend recovering well from errors. If this were an interesting path, there would be a bit of work to tailor the existing new backend constructs for a CLI version.

Somewhat related to #23817

🤞 Still hoping for movement on this

not stale

+1

#27680 is very related here.


HHH-16935 Fix inconsistent method name

https://hibernate.atlassian.net/browse/HHH-16935

@sebersole Please review.

That looks reasonable, thanks! I'll let @sebersole decide about the final method name, but could you re-introduce the old method name as a deprecated delegate please? Otherwise I'm not sure which platforms we might break, and we wouldn't want to do that in a micro.

@Sanne Updated by introducing deprecated default method as your suggestion.

It's not just renaming but also logic error fix, please check it ASAP. @Sanne @sebersole
https://github.com/hibernate/hibernate-orm/blob/0897a90a9e674d1a56633622fcd6a240d7dbda41/hibernate-core/src/main/java/org/hibernate/boot/internal/MetadataBuilderImpl.java#L973-L976

What's the decision?

Sorry, missed the notifications from this.  :+1:   Jan is going to rebase this and apply

Sorry, missed the notifications from this. 👍 Jan is going to rebase this and apply

Done.

It should be backported to 6.2.x and 6.3.x.

Hello, I think this change it is related with this bug
https://hibernate.atlassian.net/jira/software/c/projects/HHH/issues/HHH-16881?filter=allissues
I don't know if you have time to check it, but currently it is not possible to use RevisionListener with Spring Boot 3.1.X
Thanks in advance

Hello, I think this change it is related with this bug
https://hibernate.atlassian.net/jira/software/c/projects/HHH/issues/HHH-16881?filter=allissues
I don't know if you have time to check it, but currently it is not possible to use RevisionListener with Spring Boot 3.1.X
Thanks in advance

I missed checking Helper.allowExtensionsInCdi().

I created a PR yesterday
https://github.com/hibernate/hibernate-orm/pull/7446