verifyTypeScriptSetup.js writes mixed line endings into tsconfig.json

In verifyTypeScriptSetup.js:
    fs.writeFileSync(fileName, JSON.stringify(object, null, 2) + os.EOL);

the code uses a combination of JSON.stringify and os.EOL - however JSON.stringify is specified to (and does) always use 0x000A as its linefeed, even on Windows.
The net result is that after building a create-react-app project on Windows, then tsconfig.json contains LFs all down the file until the last line, which is CRLF, as shown here:

This is not a user misconfiguration, it is the out-of-the-box behaviour
It occurs whenever verifyTypeScriptSetup.js rewrites this file on Windows
tsconfig.json ends up with linebreaks as shown above regardless of whether it was initially pure LF or initially pure CRLF

It is possible to achieve (what seems to be fairly evidently) the intent of the code by replacing all LF characters in the JSON.stringify output with os.EOL as follows:
    fs.writeFileSync(fileName, JSON.stringify(object, null, 2).replace(/\n/g, os.EOL) + os.EOL);

NB Even this is NOT necessarily what the user actually wants (for a Windows user, it depends on whether they are working with Windows linefeeds, or perhaps in Unix linefeeds because it is a cross-platform project).
I'd be happy to do a PR, but suspect there may well be other places where this approach is used, and also note that there are two other uses of os.EOL in the same file (which again may or may not be what the user really wants for the reason just mentioned).

If anybody from the create-react-app team is looking at this, I'd be happy to put in the fixed line above as a PR - but note the following which I think need to be resolved/discussed first:

There seem to be potentially related issues which this alone might not fix (as mentioned above)
The simple fix (as suggested above) to the code line which has the problem will achieve what (it seems clear) the code was intended to do, but still won't always do what the user actually wants

Any suggestions as to any way to make the line ending used default to the correct for the OS - which I can do - but also be configurable if the user needs it to be would be good
Actually, off the top of my head, I'd suggest that the tsconfig.json file itself could have a new config value saying which line endings it should use if/when re-written, with values 'windows', 'unix' and 'native' available, and defaulting to 'native' (c.f. this eslint project issue - which somewhat naturally suggests that the new tsconfig.json setting could natually be called 'linebreak-style' - but perhaps would be better as 'self-linebreak-style', to make it clear what it applies to)
Again, I'd be happy to do the above as a 'more complete' PR (in the sense that it would actually, rather than only partially, solve the problems which the very fact of a re-write can potentially cause), and it should be fairly simple code in the style of the existing file (if the approach just suggested seems reasonable to the team?), but still see the following point

I am not aware of what tests it might be appropriate to add for such a fix, and a suggestion as to where and what would help

I think .replace(/\r\n|\r|\n/g, os.EOL) should be sufficient for most everyone.

@ianschmitz I believe that .replace(/\n/g, os.EOL) is sufficient, since JSON.stringify always and only produces \n (it would be in breach of its contract otherwise).
A PR for that and then (if I or someone cares enough!) a separate Issue and discussion as to whether or not it's with doing anything at all about the small class of Windows users who have Git set to check out Unix linefeeds, would make sense.

Dear Ian, I've put up a PR as requested, I'd obviously be happy to change it to use the more defensive coding you've suggested, if you'd all prefer.
For some reason, CI builds on the project are failing for some reason nothing to do with the code pushed up - in the eject tests, the (docker?) image is causing the process to fail with a missing package error.

Just wanted to leave a note that the fix as applied works for users using native linefeeds (i.e. most people!), so many thanks! But still won't work for users using Unix linefeeds on Windows (or any other weird combination, but this is probably much the most likely one, and is even the recommended setting by eslint!).


Large bundle size compared to the standard @apollo/client

Hi,
I have noticed when running npm run build the first load JS is incredibly large when using any of the hooks from this library when compared with the standard @apollo/client
Import size using 'Import Cost' extension of both the libraries:

vs

npm run build results

vs

Thanks

I've seen something similar in #95 where using NextSSRApolloClient instead of ApolloClient in registerApolloClient (so for RSC, not SSR) drove bundle size a lot - is that maybe happening here?

I am using the NextSSRApolloClient, however I just tested with a fresh project and even without any wrapper the package size is 75+kb for any import from @apollo/experimental-nextjs-app-support/ssr.
Fresh project, only containing page.tsx and layout.tsx causes the same issue:
"use client";

import { useQuery } from "@apollo/experimental-nextjs-app-support/ssr";

console.log(useQuery);

export default function Home() {
  return (
    <main>
    </main>
  );
}

https://github.com/apollographql/apollo-client-nextjs/issues/95#issuecomment-1721965647 - I have also noticed strange behaviour in other instances with bundle sizes.
Just raised a bug with MUI earlier today showing similar behaviour - https://github.com/mui/material-ui/issues/39814

@phryneas I have just re-tested this on 14.0.4 but still the same issue. The MUI bug I mentioned above appears to be fixed on 14.0.4 so I had some hope

Looking into this, this looks like some bug in the next.js build report.
If I build the pages (working with our integration test here), I get these entries in my manifest (.next/app-build-manifest.json):
{
  "/cc/dynamic/useSuspenseQuery/page": [
    "static/chunks/webpack-570b061e254f9b2e.js",
    "static/chunks/fd9d1056-2db6654aae4e30bf.js",
    "static/chunks/472-ae7a0071c9c24f93.js",
    "static/chunks/main-app-8059d80e590c3de5.js",
    "static/chunks/521-f6286d24f3481bd1.js",
    "static/chunks/679-3e2ead1f48a6bd5f.js",
    "static/chunks/100-a5bf1a60e71cf4e8.js",
    "static/chunks/app/cc/dynamic/useSuspenseQuery/page-112ed6fcb4431919.js"
  ],
  "/cc/dynamic/useSuspenseQueryAC/page": [
    "static/chunks/webpack-570b061e254f9b2e.js",
    "static/chunks/fd9d1056-2db6654aae4e30bf.js",
    "static/chunks/472-ae7a0071c9c24f93.js",
    "static/chunks/main-app-8059d80e590c3de5.js",
    "static/chunks/521-f6286d24f3481bd1.js",
    "static/chunks/app/cc/dynamic/useSuspenseQueryAC/page-bd72f692458105ca.js"
  ]
}

Where "/cc/dynamic/useSuspenseQuery/page" uses import { useSuspenseQuery } from "@apollo/experimental-nextjs-app-support/ssr" and "/cc/dynamic/useSuspenseQueryAC/page" uses import { useSuspenseQuery } from "@apollo/client";.
Now you will notice that the import from @apollo/client indeed does not contain the two dependencies
    "static/chunks/679-3e2ead1f48a6bd5f.js",
    "static/chunks/100-a5bf1a60e71cf4e8.js",

If you look at these, 679-3e2ead1f48a6bd5f contains the @apollo/client package and 100-a5bf1a60e71cf4e8 contains @apollo/experimental-nextjs-app-support.
So... The one where we import @apollo/client directly does not contain @apollo/client. That seems weird. And it is.
Until you look further, at the layout:
{
    "/cc/layout": [
      "static/chunks/webpack-570b061e254f9b2e.js",
      "static/chunks/fd9d1056-2db6654aae4e30bf.js",
      "static/chunks/472-ae7a0071c9c24f93.js",
      "static/chunks/main-app-8059d80e590c3de5.js",
      "static/chunks/db4cc85f-52d9a110b29d1dde.js",
      "static/chunks/ab91f1f5-c5e022c50fe46f19.js",
      "static/chunks/aaea2bcf-9807dd94c29a0ac5.js",
      "static/chunks/521-f6286d24f3481bd1.js",
      "static/chunks/679-3e2ead1f48a6bd5f.js", // @apollo/client
      "static/chunks/988-f8ecbd3dba53a260.js",
      "static/chunks/100-a5bf1a60e71cf4e8.js", // @apollo/experimental-nextjs-app-support
      "static/chunks/app/cc/layout-a809e480e8f29bda.js"
    ],
}

You see: loading the layout already loads those two anyways - no matter which of the two imports we actually use.
So, both of these should actually be equally-sized, until you look at the build report:
├ λ /cc/dynamic/useSuspenseQuery                       435 B           208 kB
├ λ /cc/dynamic/useSuspenseQueryAC                     5.26 kB         109 kB

So, the page /cc/dynamic/useSuspenseQuery reports to use 99kb more. But only because it counts one file twice?
If you look further, though, /cc/dynamic/useSuspenseQueryAC, by importing from @apollo/client, the page size suddenly increased from 435B to 4.26kB. That's because for some reason, here the bundler starts inlining a few functions from @apollo/client - from a library that at that point in time has already loaded anyways.
So not only is the reporter broken here, but using @apollo/client instead of @apollo/experimental-nextjs-app-support actually leads into actual double-shipping of an additional 5kB of code. 🤦

At this point I'm actually not that sure about the tree-shaking here...
graphql should tree-shake down to 12kB (4kB gzipped) for Kind, visit, print (which is all that AC uses) or 40kB (10kB gzipped) if you also take into account parse, which is used by graphql-tag.
But if you look at this, it's 500kB (44kB gzipped) instead, which is crazy!

@tombryden could you give #161 a try? It should shake off a big part of the graphql package.
npm i<EMAIL_ADDRESS>

Many thanks for looking into this.
I have given this a quick test, this change definitely helps - still not near the size of @apollo/client though unfortunately.
Fresh project, npm run build with Next.js 14.0.4

Import Cost Extension (originally 76.6k gzipped so nice improvement here)

npm run build including useQuery from @apollo/experimental-nextjs-app-support/ssr (originally 211kB)

and to compare - regular @apollo/client import of useQuery

@tombryden  I believe your "import cost" calculator makes the same mistake as the Next.js build report itself here - it "forgets" that the layout imports ApolloClient etc. anyways - and while those now turn up in your page, they already have been sent to the user with the Layout - this is likely not actually additional bundle size.
Can you measure this another way, e.g. by calling the page in the browser and looking at the actually transmitted data in both cases?

That would make sense, tbh the bundler from next.js 13 onwards seems flaky atm, there are other bugs I am following relating to it.
Here are the results of loading the page:
useQuery from @apollo/experimental-nextjs-app-support/ssr

useQuery from @apollo/client

Then I tested using dynamic importing to fully isolate the component with useQuery
Dynamic import with @apollo/experimental-nextjs-app-support/ssr

Note _app-pages-browser_src_comps_test_tsx.js size
Dynamic import with @apollo/client

Very strange!

Could it be that you just tried this in dev, not with the next build/next start artefact? The page.js in the next build variant is <1kB for me.

Whoops... apologies
Here are the same results using build
useQuery from @apollo/experimental-nextjs-app-support/ssr

useQuery from @apollo/client

Dynamic import with @apollo/experimental-nextjs-app-support/ssr

Dynamic import with @apollo/client

Think you hit the nail on the head - looks like an issue with the bundle reporter

Yeah :/ Unfortunate that Next reports it like that.
But it wasn't for naught - we got that optimization in #161 out of it :)
Are you okay if I close this issue though? I don't think I can really address this from our side.

Will close it now. Many thanks for https://github.com/apollographql/apollo-client-nextjs/pull/161 and looking into this!


Proxying requests still getting blocked by CORS

Describe the bug

CRA is being served on http://localhost:3000
In package.json I have this line: "proxy":  "http://localhost:8000"
My backend is uvicorn serving a FastAPI application
Making the request (GET http://localhost:8000/api/features) with Postman is successful
The request in React (using axios) is blocked

Did you try recovering your dependencies?
Yes: updated npm, deleted node_modules and package-lock, reinstalled
Which terms did you search for in User Guide?

Proxy
CORS

Environment
Environment Info:
current version of create-react-app: 4.0.3
System:
OS: macOS
Binaries:
Node: 13.11.0 - ~/.nvm/versions/node/v13.11.0/bin/node
Yarn: Not Found
npm: 7.6.2 - ~/.nvm/versions/node/v13.11.0/bin/npm
Browsers:
Chrome: 89.0.4389.82
Firefox: 86.0
npmPackages:
react: ^17.0.1 => 17.0.1
react-dom: ^17.0.1 => 17.0.1
react-scripts: 4.0.3 => 4.0.3
npmGlobalPackages:
create-react-app: Not Found
Steps to reproduce

Have a local uvicorn/FastAPI backend running on http://localhost:8000
Add "proxy": "http://localhost:8000" to your package.json
Start your local CRA app running on http://localhost:3000
Make a request from CRA to your backend
Note the error messages

Expected behavior
The proxy usually works fine for me, I don't understand why it's not working in this new project. It seems the browser is receiving the response as expected, but it's still getting blocked despite the proxy.
Actual behavior
The request seems to be getting blocked by CORS in both Chrome and FF

Reproducible demo
I don't know how to reproduce this exact set of circumstances. The proxy normally works fine for me. Hopefully someone can give feedback based on the response screenshots I shared.

I'm having the same problem


Feature Request: Adding Files within Code Editor

Hello! Is is possible to add a feature to add files within the code editor?
Thanks!

Hello and thank you for reaching us. Yes. It is possible, but you need to open the folder and right click your explorer, you should see New file on the right click. Thank you!

Thanks!!!!


Add zsh support

Would love to see this.

Maybe sometime, I'll have some time to figure out zsh API for autocomplete (and fish too) :)
Or someone can do it?


Propagate HADOOP_PROXY_USER on launching Flink engine

Why are the changes needed?

How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

Codecov Report

Merging #3604 (f5731b2) into master (2cb34c4) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master    #3604      +/-   ##
============================================
- Coverage     51.75%   51.75%   -0.01%
  Complexity       13       13
============================================
  Files           483      483
  Lines         27023    27025       +2
  Branches       3777     3778       +1
============================================
+ Hits          13985    13986       +1
- Misses        11672    11673       +1
  Partials       1366     1366

Impacted Files
Coverage Δ

...n/scala/org/apache/kyuubi/engine/ProcBuilder.scala
80.36% <0.00%> (-0.38%)
:arrow_down:

...rg/apache/kyuubi/ctl/cmd/log/LogBatchCommand.scala
80.00% <0.00%> (ø)

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

cc @yanghua @SteNicholas, do you think it's a bug fix or new feature? should we merge it into branch-1.6?

Thanks, merging to master

@dongkelun the HADOOP_PROXY_USER has side-effects and causes #4762 on non-kerberized env, any thoughts?

@dongkelun have done some investigation, seems Flink does not support proxy user, so there is no ideal solution IMO now.
I will simply revert this patch from branch-1.7(master already removed it in b315123a6b6dfa7b03a5ab7875856bdfd4e0eaed), as a workaround, users can pass the env vars to the engine by kyuubi.engineEnv.HADOOP_PROXY_USER=<> to achieve this capability


ETAs for code and checkpoints ?

Hi,
Wanted to know if there are any ETAs for code and ckpt files?
Thanks

Thanks for your interest. We will provide detailed codes in the coming months. Appreciate your patience.

Thanks for your interest. We will provide detailed codes in the coming months. Appreciate your patience.

@EasonXiao-888 waiting for your great work and source code. Please asap.

We will release our code and checkpoints as soon as possible. Please stay tune!

@dragen1860 @amantanejams Sorry for the wait!!! We have uploaded a simple version of the code and will continue to update it later. Thanks for your interest!!!

Okay, that's great. Thanks ! What about the license? Can you please add that as well?

Okay, that's great. Thanks ! What about the license? Can you please add that as well?

Our codes are under MIT license.


Overlay problem when opening modal in ion-header

Short description of the problem:
I try to open modal dialog from ion-header and i can see that function is getting called and ion-backdrop is overlay screen. If i tap 2 more times on button then i can see a flash of modal that has been created.
What behavior are you expecting?
Modal open.
Steps to reproduce:
<ion-header>
<ion-navbar>
<ion-buttons start>
<button (click)="navigateProfile()">
<ion-icon name="contact"></ion-icon>
</button>
</ion-buttons>
<ion-title>
Club Room
</ion-title>
<ion-buttons end>
<button>
<ion-icon name="add" (click)="createGroup()"></ion-icon>
</button>
</ion-buttons>
</ion-navbar>
</ion-header>
<ion-content class="clubroom" fullscreen>

Typescript file
createGroup() { console.log('Create group');
let modal2 = this.modalCtrl.create(ModalCreateGroupPage,
{ groupId: 'this.groupId', groupName: 'this.groupName',       inputPostMessage: 's' });
console.log('Create group2');
modal2.present();
console.log('Create group3');
}

Ionic-angular 2.0.0-beta.11

Cordova CLI: You have been opted out of telemetry. To change this, run: >cordova telemetry on.
6.3.0
Ionic Framework Version: 2.0.0-beta.11
Ionic CLI Version: 2.0.0-beta.36
Ionic App Lib Version: 2.0.0-beta.19
ios-deploy version: Not installed
ios-sim version: 5.0.8
OS: Mac OS X El Capitan
Node Version: v5.12.0
Xcode version: Xcode 7.3.1 Build version 7D1014

A user had similar problem and added in app.core.scss but this not working for me either..
ion-modal {
  z-index: 9999;
}

Hello! Thanks for using Ionic! Are you getting any errors in the console? Also did this issue also happen in beta.10?

I search for this issue for ours.... and when when i post it here i found problem...
Problem is that there were a ngModel variabel that wasnt sett properly in modal ts page where i instansiated new FormGroup... :(


Super high cpu usage by Ghost

Hello, I have been running my blog for over two years now on Ghost. However, I notice very high CPU usage on my duo-cpu server. Htop reports that it's node which runs Ghost. Linode sends me alerts everytime CPU spikes to over 120% and from htop graphs node is pushing to 100%.
My blog (http://davidokwii.com/) isn't really a high traffic blog at all. So I am wondering why Ghost is taking up so much resources.
Technical details:

Ghost Version:0.11.3
Node Version: v4.2.6
Browser/OS: ubuntu 16.04
Database: Sqlite

Happened to me just today. I have 1 vCPU and 2 GB ram. CPU barely goes above 10%-15% but today the site hanged. I checked Htop, and it showed the node is using 100% CPU. I restarted the instance, and it got fixed.
But still, it's weird. I don't want my blog to just periodically hang.
I am running the latest Ghost version with MySQL on Ubuntu 20.04 (arm64).

Hey @AmreshSinha - maybe you could share some more details - ideally on the forum? This is very unlikely to be related to an issue in Ghost from 2018.


Projects files with  files don't work properly

    <Compile Include="..\sr.fs">
      <Link>ErrorText\sr.fs</Link>
    </Compile>

Project such as FCS use file links and it would be great if they could opened and edited in Atom:
This is an ideal project to aim to support:
https://github.com/fsharp/FSharp.Compiler.Service/blob/master/src/fsharp/FSharp.Compiler.Service/FSharp.Compiler.Service.fsproj

Hmm. That's interesting case. But one of the main point of Atom is that it's using file system instead of project files. So I'm not sure if it's even possible.
Maybe workaround would be parsing fsproj files and creating temporary symbolic links in file system for ever file link from project file.

Is it not possible to create virtual entries in Atom to represent the links?
Its a pity if this doesn't work, my main use case for using Atom or Sublime is generally editing compiler source, it would be nice to have tooltips and completion and also without red underlined errors everywhere like this:

Hmm, well looks like the problem is with not finding fsproj file for currently opened file. Can You open fsproj file to force parsing it, come back to file You are trying to edit and check if there still is an issue?

If I open the project file and reopen the file the errors disappear, although tooltips don't work for all the types on the page.  If I then open the file where one of the missing tooltips is defined like Symbols.fs and go back to Expr.fs then the errors reappear.

When you force parse the project file are you starting Background parsing for the project?

When users open new file (and it's F# file) I try to find fsproj file in same folder as file. If I find project file I'm calling fsautocomplete.exe and executing https://github.com/fsharp/fsharpbinding/blob/master/FSharp.AutoComplete/Program.fs#L392.
Same story for fsproj file, if user open file I just execute same thing.

Hmm, from what I can see it just parses the project file, it doesn't start a background compilation of the project, if it did then a request for any file in the project to be parsed would return results based on the background check.


how do i make

i dont understand can you explain it more in death and how do i open framwork thanks

Which part of the building process  dont you understand?

Please see the building manual for instructions. If that doesn't help you, maybe this mod isn't meant for you.


Branches blocked

I don't think I can create new branches on this repo getting a 403 error.

Write access to repo granted now. dev branch is protected from direct push. Pull requests are still possible.


Think about behaviour of EnvelopeEncryption when the KMS is unavailable

          We do need to think about what we should do with produce and fetch requests when then KMS is unavailable I'm not sure we should just reject them because they may contain records for un-encrypted topics which we should still pass through (graceful degradation)

Originally posted by @SamBarker in https://github.com/kroxylicious/kroxylicious/pull/932#discussion_r1482210307

See #811

also potentially addressed by #972

This is different from the question about should we let the KMS response dictate whether we encrypt. There are questions here about whether we could let partial produce data through and make better use of Kafka response codes, rather than just killing the request/connection.

@SamBarker can you please update the description with more detail about the problem

We should also think about the test cases we'll want to add too.


feat(k8s-driver): use custom ServiceAccount for task pods

As mentioned in #36 being able to use a custom ServiceAccount would be great e.g. when deploying applications from a pipeline to the local cluster.
I kept the default behavior that if there's no ServiceAccountName is configured, the driver won't mount the ServiceAccount but it's now possible to use a custom ServiceAccount for a task

@baez90 Thanks for the PR.
About the implementation of this PR, since this is related to the k8s executor, the ServiceAccountName shouldn't be propagated to types, runconfig etc... but live only inside the executor.

I'd happy for some advice where it's actually necessary :smile: I started at one point (can't remember which one) and propagated it to the others because it didn't work in the first place :sweat_smile:


Fix typo in SvelteKit getting started docs

A typo in the section that asks you to check for typos, that's embarrassing
thanks!


How to switch loading bar condition based on views in angular js

What version of angular-loading-bar are you using?
angular-loading-bar v0.9.0
What version of angular are you using?
I am using  AngularJS v1.5.8
What is the issue ?
Right now i have added cfpLoadingBar in my app.config file.,
it is coming to all the page prior loading.
It is ok.
but I want the spinner loading bar to all the page except few other views.
I have 4 views used in application.

Home page view
Detail page view
Contact page view
About Us page view

I want spinner loading to contactus & aboutus page.
means  "cfpLoadingBarProvider.includeSpinner = true;"
and for Home page and details page I want to set template to show the welcome message HTML.
means " cfpLoadingBarProvider.spinnerTemplate = "
Welcome to my site
;"  "
How can I handle single angular loading bar with different condition
can you help me on this.
is there any possibility?
What did you expect to happen?

Home page view - it should come with "welcome message"
Detail page view - it should come with "welcome message"
Contact page view - it should come with spinner as usual
About Us page view - it should come with spinner as usual

If not, then show me different way how to add condition to full fill my query..
What actually happened?
Right now, spinner loader coming to all the page views same, I want to differentiate based on views.

Hi,
From what I understand, angular config is setup once during module
initialisation. Hence I think you will bot be able to modify or control
it's behaviour dynamically.
Regards,
Nitin
On Oct 26, 2016 12:39 PM, "Abhi"<EMAIL_ADDRESS>wrote:

What version of angular-loading-bar are you using?
angular-loading-bar v0.9.0
What version of angular are you using?
I am using AngularJS v1.5.8
What is the issue ?
Right now i have added cfpLoadingBar in my app.config file.,
it is coming to all the page prior loading.
It is ok.
but I want the spinner loading bar to all the page except few other views.
I have 4 views used in application.

Home page view
Detail page view
Contact page view
About Us page view

I want spinner loading to contactus & aboutus page.
means "cfpLoadingBarProvider.includeSpinner = true;"
and for Home page and details page I want to set template to show the
welcome message template.
means " cfpLoadingBarProvider.spinnerTemplate = "Welcome to my site;" "
How can I handle single angular loading bar with different condition
can you help me on this.
is there any possibility?
What did you expect to happen?

Home page view - it should come with "welcome message"
Detail page view - it should come with "welcome message"
Contact page view - it should come with spinner as usual
About Us page view - it should come with spinner as usual

What actually happened?
Right now spinner coming to all the page views
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/chieffancypants/angular-loading-bar/issues/333, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ATeo6ryq61fkPkoVNd78NY_1e1Y3CKnBks5q3vypgaJpZM4Kg1CP
.

@nitinsaxena27  you are right. it doing config time..
How can I do?
can you tell me some tips & tricks?
Thank you.

Trick is you can use the same spinner template in your main html file and
control it's visibility using Ng if and bind it to a model which is set in
the controller. This will mimic the same behaviour. Cheers.
On Oct 28, 2016 3:51 PM, "Abhi"<EMAIL_ADDRESS>wrote:

@nitinsaxena27 https://github.com/nitinsaxena27 you are right. it doing
config time..
How can I do?
can you tell me some tips & tricks?
Thank you.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/chieffancypants/angular-loading-bar/issues/333#issuecomment-256887158,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATeo6hp0TuO-3pykIdGXubyYj3l-WdjPks5q4cy2gaJpZM4Kg1CP
.


add initializer uninstall cmd in setup

as titled

Jenkins job istio.github.io/presubmit passed

Jenkins job istio.github.io/presubmit passed

Jenkins job istio.github.io/presubmit passed


Question about ts/js deprecated hint

VSCode Version: 1.47.3
Commit: 91899dcef7b8110878ea59626991a18c8a6a1b3e
Date: 2020-07-23T13:08:29.692Z
Electron: 7.3.2
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Darwin x64 18.6.0 macOS Mojave 10.14.5

The function type is overload, and deprecated part of signature
I have no idea whether the hint is right or wrong
Does this issue occur when all extensions are disabled?: No
But it came out when installed the plugin of [JavaScript and TypeScript Nightly]

/cc @Kingwl @sandersn

I can repro this with the following program:
// @filename: first.ts
/** @deprecated */
export declare function tap<T>(next: null);
export declare function tap<T>(next: T): T;

// @filename: second.ts
import { tap } from './first'

Note that uses are marked correctly, just the import is incorrectly marked deprecated:
tap // ok
tap(null) // deprecated
var twelve = tap(12) // ok

The import needs to be treated the same as the uncalled function on the first line.
@mjbvz I can't figure out how to move this to the typescript repo, but I'll start looking at this to be fixed before the full 4.0 release.

Fix is available at microsoft/TypeScript#39950.

This is now merged and will be available in Typescript 4, although it didn't make it into the RC.


Local Artwork not working anymore

My local cover in www folder doesnt't show up

Can you provide more details? Platform, OS version, plugin version, any other pertinent details.

I had to run 'ionic state reset' which installs all platforms and plugins again. It stopped working after that.
Manually reinstalling the platform or plugin did not work, too.
Noticed on Android 5.1 and Android 6.0.1
Adding a remote image still works.

It worked again after updating to 1.4. (Even tho i manually updated to 1.4 before and it didn't fix it).
Closed for now.


DS-1946 Upgraded to Ruby 3 / Rails 7 to allow console_sso to upgrade similarly

Overview
DS-1946
What is the change?
Upgrade this gem to Ruby 3 / Rails 7 to allow console_sso to be upgraded
How was this change made?

Followed the upgrade guide. See the CHANGELOG.md for more details
Moved CI/CD from Travis -> Jenkins

How do I use/reproduce this change?
Notes

Incorrectly created this PR


Feature: Edit Multiple Flags at Once

Trello Link (if applicable): https://trello.com/c/2oR6QFRH
Forum Link (if applicable):
Explanation of Bugfix/Feature/Modification:
This one has been wanted for a while and it was pretty straight forward so I just went ahead and added it! Simply adds two buttons to the transaction edit modal to add/remove flags. If multiple transactions were selected, it updates all of them.

Thank you, guys, so much! Thank you, thank you, thank you! :-)


Fix #62 selected date not being displayed

There is no code in the pull request and as far as I can tell, no bug to fix.


One question about OpenFSAT and Matlab

Dear guys,
I would like to know how can the dynamic link library be called in Matlab, or how can it be used in Matlab?
Thanks!

Duplicate of #1897 and #1898


TypedFactory Disposes Before Component That Depends on it

If a typed factory is a dependency of a disposable component, it is already disposed by the time dispose is called on that component.
If I directly implement the factory myself rather than using the typed factory, everything behaves as I would expect.
This SO Question also details the issue
I have attached a test solution below that shows this behavior (although it is not actually useful itself).
WindsorTypedFactoryDisposalTest.zip

I am having the same problem. If I use a factory to construct an IDisposable, I am the one responsible to put it in a using statement or call Dispose. That is kind of the whole point of injecting a Func<DbContext>- so that I have absolute control of creation and disposal of every DbContext. I need that control.

Here is another repro, very tiny and to the point: Test Castle Windsor dispose out of order.zip

This is causing our application to crash in production. Is there a workaround?

@jnm2 I ended up not using the typed factory feature at all and ended up creating my own full factory class instead. When you create your own factory, and wire it up along with everything else it will behave as expected.

@pnquest That would be unnecessarily disruptive if there's anything else I can do. I'm using delegate factories and the delegate factories are what is keeping track of releasing the dependencies of the components they provide.
@hammett Any idea what the ETA might be?

I think I found a workaround for the time being- replace dependencyFactory.Invoke() with TypedFactoryWorkaround(dependencyFactory):
private T TypedFactoryWorkaround<T>(Func<T> typedFactory)
{
    try
    {
        return typedFactory.Invoke();
    }
    catch (ObjectDisposedException)
    {
        // Workaround for https://github.com/castleproject/Windsor/issues/114
        Dispose();
        throw new OperationCanceledException();
    }
}

@jnm2 That works fine if you are looking to just gracefully fail, in my case there was still data flowing through that needed to be handled before shutdown could be finalized. In that situation your workaround will not help.

@pnquest Exactly.

I'm still running into this big time. @hammett Did you make any progress on this?

I'm tracking this at https://github.com/castleproject/Windsor/pull/61
I believe it's the same issue, correct?

@hammett I wasn't positive, it's been so long. Is there anything I can do to help crack this one?

Aside from debugging it and fixing it yourself, no. This is new code to me as well since I haven't written that part of Windsor

On Jun 9, 2017, at 1:11 PM, Joseph Musser<EMAIL_ADDRESS>wrote:
@hammett I wasn't positive, it's been so long. Is there anything I can do to help crack this one?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@hammett I'll be able to try my hand at it in two weeks.

We are thinking of deprecating this facility, please join the discussion at https://github.com/castleproject/Windsor/issues/339

@fir3pho3nixx Looks like both of you are thinking of actually keeping this facility which is good, because otherwise I'd have to cut my losses now. 😄

Yep, the Typed Factory facility is staying as is UsingFactoryMethod, the Factory Support facility is the one we are looking at deprecating. Reopening.

Whoops wrong factory!

@fir3pho3nixx @jonorossi I want to pick your brains on this issue. As a practical matter, things have come to a head with this issue and I've been looking at fixing it.
Simple failing test: https://github.com/castleproject/Windsor/compare/master...jnm2:fix_typed_factory_out_of_order_dispose
Reason: TypedFactoryInterceptor must not be disposed until all handlers which use typed factories are disposed, because the handlers may need to invoke their typed factory dependencies in order to be disposed.
The problem is, DefaultKernel.Dispose contradicts this necessity by doing things in this order:

Calls DisposeComponentsInstancesWithinTracker which disposes TypedFactoryInterceptor
Calls DisposeHandlers which disposes a service whose Dispose method invokes a typed factory

I've looked and tried things for a few hours but the complexity of the internals is daunting. What approach would you take to fix this?
https://github.com/castleproject/Windsor/blob/8e139da134c0640dceefb4a8a043af1958e60927/src/Castle.Windsor/MicroKernel/DefaultKernel.cs#L233-L245

Do we need to invent a way to postpone an interceptor's disposal until after the handlers have been disposed?

Or maybe we need a way to make sure the handler for TypedFactoryInterceptor, created by InterceptorReference.IReference<IInterceptor>.Resolve, ends up in the DisposeHandlers() list instead of the DisposeComponentsInstancesWithinTracker() list.
That way it could be topologically sorted, so long as the dependencies are reflected properly between it and the handlers that use typed factories.

The TypedFactoryInterceptor already does end up in the DisposeHandlers, and it is topologically sorted properly. So we just need a way to keep it from being disposed by DisposeComponentsInstancesWithinTracker, which just releases the container's lifetime policy.

Wait... is it really this easy? I thought I had already tried switching the order of the DisposeHandlers and DisposeComponentsInstancesWithinTracker calls, but now it's working!
DisposeComponentsInstancesWithinTracker makes sense as a backstop for anything DisposeHandlers didn't clean up organically via reference counting, and no tests tested to fail, so I'm satisfied with this...

https://github.com/castleproject/Windsor/pull/344

@jnm2 great work working through that, now that we've got a unit test that defect is now squished forever.


Update future-friday.md

add link to jira board

I didn't even know we had a board. in other teams we use jira labels or notion pages. others use nothing, just personal notes. maybe we should skip this link, and keep any FF info in team docs?

@dblandin Thanks, I've updated the name of the link to reflect that


Components in splitter panels not updating

Components only seem to update when they deref a ratom, but not on changing arguments.
(def cnt (r/atom 0))
(defn myspan1 [] [:span @cnt])
(defn myspan2 [t] [:span t])
(defn demo []
  [:div
    [re-com/button :label (str "inc " @cnt) :on-click #(swap! cnt inc)]
    [re-com/h-split :panel-1 [:div "works: " [myspan1]]
                    :panel-2 [:div "doesn't: " [:span @cnt] ", " [myspan2 @cnt]]]])

@ZTO  two things:

there's just not enough information here for anyone to help you.  Which components?  What arguments changed?  If you ask for help please take the time to distill the problem down and explain it well.
This is not the right place to ask this question.  This place is to issues, not asking for help.

Regarding point 2, please take this up in the various help forums like:

http://clojurians.net/ (#reagent channel)
https://groups.google.com/forum/#!forum/clojurescript

In the meantime does this page help?  https://github.com/Day8/re-frame/wiki/When-do-components-update%3F

This is a bug report, not asking for help.
I think I distilled it into a minimal, self-explanatory example that demonstrates the issue.
I can also tell you that replacing the splitter component with a regular div solves the update problem, hence my suspicion that the splitter is the culprit.

@ZTO, the behaviour you are experiencing is the way reagent atoms and components work.

@ducky427 are you referring to passing ratoms vs. derefed (and changed) values? If you look at my code, I'm doing the latter, which should trigger a rerender. Again, it does work if I put everything in a basic div, so the problem doesn't seem to be with Reagent or my understanding thereof. It only breaks once I put it in a splitter.

Okay, thanks, I see what you mean now.

Ahh,  yes, and I can see the bug now.  These were the very first ever components we wrote for re-com  and, sure enough, they have a rookie mistake in them.
But I'll have to figure out what the right fix is.
Thanks for reporting it and staying persistent despite my attempts at moving you on . :-)

For the record, a fix (incomplete) is simply to change this line:
https://github.com/Day8/re-com/blob/master/src/re_com/splits.cljs#L122
to  (fn [& {:keys [panel-1 panel-2]}]
The current version leaves panel-1 and panel-2 as the original parameter value, and never a rerendered value. The proposed change allows new "values" for panel-1 and panel-2 to be rendered by the parent.
But that still leaves the question:  are these the only two parameters which can change?  What about width or style.   Some thought required.

@mike-thompson-day8, your suggestion above did indeed fixed the component. This will be included in the next alpha release


stable release after a month in beta testing

do not merge unless all tests pass.

Need to disable this test from the pipeline
testPackage.NetworkInterceptionTest.interceptShaftLogoAndReplaceItWithYoutubeLogo


Provide a command-line compiler (as its own npm package)

Input files can be in any extension, and the
output will just be sameFileName.tpl.
NOTE: chai and mocha were added as deps because otherwise it
would throw:
$ ./node_modules/ts-node/dist/bin.js src/main.ts

/Users/andres/Documents/Code/ivy-bitcoin/ivy-compiler/node_modules/ts-node/src/index.ts:329
        throw new TSError(formatDiagnostics(diagnosticList, cwd, ts, lineOffset))
              ^
TSError: ⨯ Unable to compile TypeScript
Cannot find type definition file for 'chai'. (2688)
Cannot find type definition file for 'mocha'. (2688)
    at getOutput (/Users/andres/Documents/Code/ivy-bitcoin/ivy-compiler/node_modules/ts-node/src/index.ts:329:15)
    at Object.compile (/Users/andres/Documents/Code/ivy-bitcoin/ivy-compiler/node_modules/ts-node/src/index.ts:515:11)
    at Module.m._compile (/Users/andres/Documents/Code/ivy-bitcoin/ivy-compiler/node_modules/ts-node/src/index.ts:402:43)
    at Module._extensions..js (module.js:646:10)
    at Object.require.extensions.(anonymous function) [as .ts] (/Users/andres/Documents/Code/ivy-bitcoin/ivy-compiler/node_modules/ts-node/src/index.ts:405:12)
    at Module.load (module.js:554:32)
    at tryModuleLoad (module.js:497:12)
    at Function.Module._load (module.js:489:3)
    at Function.Module.runMain (module.js:676:10)
    at Object.<anonymous> (/Users/andres/Documents/Code/ivy-bitcoin/ivy-compiler/node_modules/ts-node/src/bin.ts:145:12)

@danrobinson should I care rebasing this PR or will you keep not reviewing it?

This project is dead. You might want to fork it or take a look at Spedn and CashScript.

BCash? lol

Thanks for doing this—LGTM! If you don't mind rebasing it I can test it locally and then merge.

@danrobinson thanks for the feedback! I just rebased it.

@danrobinson ping?

Is there any more work that needs to be done before ivy-bitcoin 0.0.34 can be published on the npm registry? It would make it easier to test this out.

Published version 0.0.34.

@danrobinson The suggested changes were applied with the latest public ivy-bitcoin package.

@danrobinson The suggested changes were applied

@danrobinson ping

@danrobinson ping

I ran npm update in the ivy-compiler dir. This fixed the build errors I was seeing so I've added a commit with the updated package-lock.json

@danrobinson ping :)

@danrobinson
Outputting to a file with a name based on the input file name is more compiler-y though. Not that I have a strong preference either.

Exactly, I don't know any unix-y compiler that outputs to stdout by default.

@danrobinson Would it be possible to publish ivy-compiler as it's own npm package?

@canndrew You could go ahead and do that yourself, couldn't you? Not sure you need me


Strange behavior when extending arbitrary::arbitrary/2 predicate

Hello!
I am observing some strange behavior when creating my own generators for subsequent use with quickcheck and the lgtunit which I suspect is somehow related to the interaction of the arbitrary library with the random library.
Here is the necessary code to reproduce the issue:
:- object(wwtypes).

:- multifile(arbitrary::arbitrary/1).
arbitrary::arbitrary(fluent).
:- multifile(arbitrary::arbitrary/2).
arbitrary::arbitrary(fluent, Arbitrary) :-
   random::member(Functor, [at, out, carries, alive, dead]),
   handle_functor(Functor, Arbitrary).

:- private(handle_functor/2).
handle_functor(at, at(Entity, Place)) :-
   random::member(EntityBase, [agent, wumpus]),
   random::member(PlaceBase, [exit, wwplace, pit]),
   type::arbitrary(list(positive_integer,3),[X,Y,Id]),
   Entity=..[EntityBase, Id],
   Place=..[PlaceBase, X, Y].

:- end_object.

When testing this code, I observe the following very strange behaviour:
?- type::arbitrary(fluent, D).
D = at(agent(724), pit(93, 444)).

?- type::arbitrary(fluent, D).
false.

What's more, I am unable to step through the code with the debugger, i.e., running debugger::trace before performing the above calls has no effect, even when compiling with the debug(on) flag.
I do not see why this would be. I would greatly appreciate any help or ideas on how to minimize this issue.
I am running Logtalk 3.51.0 on ubuntu 20.04 LTS, installed from the .deb on the logtalk website, if it helps.
Thank you very much!
-ccr185

Hi. In your code, the handle_functor/2 predicate have a fixed value, at, in the first argument. Therefore, when the random::member(Functor, [at, out, carries, alive, dead]) goal unifies Functor with one of the other atoms, the call to the handle_functor/2 predicate fails. For example:
?- {debugger(loader)}.
% [ /Users/pmoura/logtalk/tools/debugger/debuggerp.lgt loaded ]
% [ /Users/pmoura/logtalk/tools/debugger/debugger.lgt loaded ]
% [ /Users/pmoura/logtalk/tools/debugger/debugger_messages.lgt loaded ]
% [ /Users/pmoura/logtalk/tools/debugger/dump_trace.lgt loaded ]
% [ /Users/pmoura/logtalk/tools/debugger/loader.lgt loaded ]
% (0 warnings)
true.

?- debugger::trace.
   Debugger switched on: tracing everything for all objects compiled in debug mode.
true.

?- type::arbitrary(fluent, D).
   Rule: (0) arbitrary::arbitrary(fluent,_27512) ?
   Call: (1) random::member(_28328,[at,out,carries,alive,dead]) ?
   Exit: (1) random::member(at,[at,out,carries,alive,dead]) ?
   Call: (2) handle_functor(at,_27512) ?
   Rule: (2) handle_functor(at,at(_30614,_30616)) ?
   Call: (3) random::member(_31138,[agent,wumpus]) ?
   Exit: (3) random::member(agent,[agent,wumpus]) ?
   Call: (4) random::member(_240,[exit,wwplace,pit]) ?
   Exit: (4) random::member(pit,[exit,wwplace,pit]) ?
   Call: (5) type::arbitrary(list(positive_integer,3),[_1512,_1518,_1524]) ?
   Exit: (5) type::arbitrary(list(positive_integer,3),[93,444,724]) ?
   Call: (6) _226=..[agent,724] ?
   Exit: (6) agent(724)=..[agent,724] ?
   Call: (7) _228=..[pit,93,444] ?
   Exit: (7) pit(93,444)=..[pit,93,444] ?
   Exit: (2) handle_functor(at,at(agent(724),pit(93,444))) ?
D = at(agent(724), pit(93, 444)).

?- type::arbitrary(fluent, D).
   Rule: (7) arbitrary::arbitrary(fluent,_7410) ?
   Call: (8) random::member(_8108,[at,out,carries,alive,dead]) ?
   Exit: (8) random::member(dead,[at,out,carries,alive,dead]) ?
   Call: (9) handle_functor(dead,_7410) ?
   Fail: (9) handle_functor(dead,_7410) ?
false.

A possible fix would be to rewrite the handle_functor/2 predicate as:
:- private(handle_functor/2).
handle_functor(Functor, Term) :-
   Term =.. [Functor, Entity, Place],
   random::member(EntityBase, [agent, wumpus]),
   random::member(PlaceBase, [exit, wwplace, pit]),
   type::arbitrary(list(positive_integer,3),[X,Y,Id]),
   Entity=..[EntityBase, Id],
   Place=..[PlaceBase, X, Y].

This will give:
?- {*}.
% Redefining object wwtypes
% [ /Users/pmoura/arb/arb.lgt reloaded ]
% (0 warnings)
% Reloaded all Logtalk source files modified or that required
% recompilation due to a change to the compilation mode
true.

?- debugger::nodebug.
   Debugger switched off.
true.

?- type::arbitrary(fluent, D).
D = carries(agent(312), wwplace(946, 502)).

?- type::arbitrary(fluent, D).
D = dead(wumpus(667), wwplace(598, 916)).

?- type::arbitrary(fluent, D).
D = carries(agent(143), exit(478, 597)).

?- type::arbitrary(fluent, D).
D = alive(agent(160), wwplace(210, 698)).

?- type::arbitrary(fluent, D).
D = out(agent(458), wwplace(559, 215)).

I don't know how I didn't see that, thank you so much for the prompt response

P.S. When defining clauses for multifile predicates that only call local, auxiliary predicates, as in your case here, best to encapsulate them in a category instead of in an object as there's really no public interface for the object.

P.S. When defining clauses for multifile predicates that only call local, auxiliary predicates, as in your case here, best to encapsulate them in a category instead of in an object as there's really no public interface for the object.

I see, note taken :)
Thank you very much both for your help and your promptness in responding!

I have followed your advice and now my generators work perfectly, however, I am seeing quickcheck behave very strangely, and it does not seem related to the aforementioned problem.
To illustrate, here's what's happening concretely:
?- lgtunit::quick_check(tests::holds_prop(+non_empty_list(fluent)),[v(true)]).
*     Failure:   tests::holds_prop([at(agent(946),pit(444,724)),dead(wumpus(502)),out(wumpus(312)),dead(wumpus(598))])
*     quick check test failure (at test 1 after 0 shrinks):
*       tests::holds_prop([at(agent(946),pit(444,724)),dead(wumpus(502)),out(wumpus(312)),dead(wumpus(598))])
*     starting seed: seed(3172,9814,20125)
false.

?- tests::holds_prop([at(agent(946),pit(444,724)),dead(wumpus(502)),out(wumpus(312)),dead(wumpus(598))]).
true.

That is, it seems the goal fails unexpectedly even when called normally it does succeed, which would run counter to what one would expect.
If it helps, here's the definition of the property in question and an alternative definition of the QuickCheck test as part of a test suite that exhibits the same behavior:
:- public(holds_prop/1).
	holds_prop(Fluents) :-
		forall(member(Fluent, Fluents),::holds(Fluent, Fluents)).

	quick_check(
		holds,
		(holds_prop(+non_empty_list(fluent,5))),
		[v(true),n(10)]
	).

Where is the predicate holds/2 defined? In the same object that contains the holds_prop/1 predicate? If so, why the message to self in the forall/2 call?

It is defined in a category outside the test suite, which imports it (I though perhaps the easiest way to get it in there since it's a category I use in other objects)

Here's a more complete version of the test object:
:- object(tests, extends(lgtunit), imports(sflux)).
	:- public(holds_prop/1).
	holds_prop(Fluents) :-
		forall(member(Fluent, Fluents),wwstate<<holds(Fluent, Fluents)).

	quick_check(
		holds,
		(holds_prop(+non_empty_list(fluent,5))),
		[v(true),n(10)]
	).
:- end_object.

And then you would have:
:- category(sflux).
:- public(holds/2).
holds(F, [F|_]).
holds(F, Z) :- Z=[F1|Z1], F\==F1, holds(F, Z1).
:- end_category.

Does the tests object imports that category?

it does indeed, would that be an issue? I don't see how else to do so?

Where is the predicate holds/2 defined? In the same object that contains the holds_prop/1 predicate? If so, why the message to self in the forall/2 call?

The message to self, I took it from the documentation that says to call imported predicates from categories in this manner.

Even changing the property to the following still produces the same behavior (i.e. calling an object that itself imports the sflux category):
holds_prop(Fluents) :-
		forall(member(Fluent, Fluents),wwstate(_,_)::holds(Fluent, Fluents)).

There are two bugs in your code:

There's no non_empty_list(Type, Length) type. That causes an immediate failure (for performance reasons, the existence of the type is not checked).
There's a definition for generating arbitrary values for the type fluent but not definition that verifies that a term is a valid fluent.

The first bug is easy to correct. Use one of the other defined list types (see https://logtalk.org/library/arbitrary_0.html).
The second bug requires defining clauses for the multifile predicates type::type/1 and type::check/2 predicates (see https://logtalk.org/library/type_0.html) so that terms of type fluent can be checked (which is required by the QuickCheck implementation).

Thank you so much, you're absolutely right. What an embarrassing mistake, it's not an issue at all. I'd suggest you delete this issue altogether, if possible.

Nothing to be embarrassed about. Happy logtalking!


HHH-18201 - Addition/subtraction of duration to/from temporal parameter wrongly resolved as duration during query parsing

See Jira HHH-18201
When duration is added/subtracted to/from temporal parameter, operation result is incorrectly resolved as duration instead of temporral.

It seems to me that the proper fix would be to change the SqlAstNodeRenderingMode for certain pattern variables on some databases to achieve the casting. Patterns shouldn't be touched IMO. I'll take a look to see if I can improve the situation a bit.

I tried applying the changes I mentioned and removed some seemingly unnecessary changes. Feel free to reset your branch to mine if you want to further work on this (i.e. add more tests).
https://github.com/hibernate/hibernate-orm/pull/8509

@beikov If you think that this approach is better, I will not change anything in my branch.
And I do not think that additional tests are necessary until existing are failing.

Since #8509 is now fixing all the problems, I will close this PR.


refactor(changelog-github): update src/index.ts in order to display contributors thumbnails in GitHub releases

Hello,
I propose this change in order to use built-in "Contributors" section of GitHub releases that shows automatically profiles thumbnails and links.
For this, we just need to replace this markdown:
Thanks [@lauthieb](https://github.com/lauthieb) - ...

To this markdown:
Thanks @lauthieb - ...

The result will be:
Before
Thanks [@lauthieb](https://github.com/lauthieb) - ...

After
Thanks @lauthieb - ...

Thanks in advance :)
Laurent

Hello, are you interested? :)
Thanks in advance 🙏

Hello @Andarist @Noviny @emmatown, are you interested? :)
Thanks in advance 🙏


Ubuntu wget fails to resolve hostnames

Ubuntu's wget implementation, like most browsers, attempts to resolve IPv6 addresses before falling back to IPv4.  For some reasons, our DNS implementation confuses wget into thinking the hostname doesn't exist and never bothering to fall back to IPv4 dns resolution.
To reproduce, I ran the network test blueprint, and changed the image from alpine, to an ubuntu instance with wget installed.
# wget red.q
--2018-02-02 22:56:44--  http://red.q/
Resolving red.q (red.q)... failed: Name or service not known.
wget: unable to resolve host address 'red.q'

However, if you run with only IPv4 resolution, you get a connection refused error instead of a dns resolution error (note there's not HTTP server running where I tested, hence connection refused is the expected result).
# wget --inet4-only red.q
--2018-02-02 22:57:06--  http://red.q/
Resolving red.q (red.q)... <IP_ADDRESS>
Connecting to red.q (red.q)|<IP_ADDRESS>|:80... failed: Connection refused.

Note this issue does not occur on alpine because it uses an ipv4 only wget implementation.

It looks like the network test blueprint is already using Ubuntu https://github.com/kelda/kelda/blob/master/integration-tester/tests/10-network/network.js#L7 -- are you using a different blueprint?

I tried adding wget to the Dockerfile linked above, but couldn't repro.
Wget version:
GNU Wget 1.17.1 built on linux-gnu.


Statistical Functions

Statistical Functions added as a part of library

Standard Deviation
Variance
Standard Error
Statistical Range
Mean
Median
Mode

Coverage decreased (-8.3%) to 86.922% when pulling ca454d92b76cccadb1fc621b76c0520eb227522f on kpkalwa:master into b713f5a6d75b12c8c57fb3f410df029497c2a43f on jashkenas:master.

The changes has been made and pushed to the master branch of the repository can you review and close the pull request @jgonggrijp

The changes has been made and pushed to the master branch of the repository can you review and close the pull request @jgonggrijp

@jgonggrijp I was still working on the changes all the changes has been completed you can review the code.
Migrating to Underscore-contrib will take time for me as the the structure is totally different and not modularized.

@jgonggrijp The changes has been Implemented with the two white spaces


[$1000] Reply in Thread page flickers when navigating from the message

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open App and Go to any chat
Send a message
Press Reply in Thread on the message

Expected Result:
The thread page directly opens with the main thread message.
Actual Result:
The thread page shows the chat header for a second and then shows the thread message.
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.47-2
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/43996225/81b90056-627e-45ac-912b-d8be5d0e58d7
https://github.com/Expensify/App/assets/43996225/5fa011e5-c215-414e-9d62-e21af6ec3c7f
Expensify/Expensify Issue URL:
Issue reported by: @jayeshmangwani
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1690459548064709
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01b6896bd2259fc866
Upwork Job ID:<PHONE_NUMBER>250601984
Last Price Increase: 2023-07-28

reproducible - I'm not seeing a matching issue open but I think this is known and my instinct is this is an internal issue

Same RCA as https://github.com/Expensify/App/issues/23638 and someone is working on this

thank you @s-alves10! I think i will close this for now as it will be fixed by #23638


Update README.md

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[ ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

Preview environment: https://ci.pr.harness.io/pr-253

Preview environment: https://ci.pr.harness.io/pr-253


web_benchmarks_html text_canvas_layout.html.multi_line.layout.outlierAverage needs a baseline.

Can we just auto-baseline this?
cc @yjbanov

Archiving all outlierAverages.


Fix commit status index problem

Replace #14160 and fix #14059

Is the ID column is necessary? Could we use a composite pk?

Is the ID column is necessary? Could we use a composite pk?

Composite primary key makes things more complicated, I would like to add a ID column.

@zeripath PASS now.


virtualservice and gateway resource flake

Describe the bug
I deploy the bookinfo sample app shortly (between 30-60 seconds) after deploying Istio. I receive the following error, even though the associated CRDs have been created and show type: Established and status: "True":
$ kubectl create -f istio-0.8.0/samples/bookinfo/routing/bookinfo-gateway.yaml
unable to recognize "istio-0.8.0/samples/bookinfo/routing/bookinfo-gateway.yaml": no matches for kind "Gateway" in version "networking.istio.io/v1alpha3"
unable to recognize "samples/bookinfo/routing/bookinfo-gateway.yaml": no matches for kind "VirtualService" in version "networking.istio.io/v1alpha3"

Expected behavior
I expect the following:
$ kubectl create -f istio-0.8.0/samples/bookinfo/routing/bookinfo-gateway.yaml
gateway.networking.istio.io "bookinfo-gateway" created
virtualservice.networking.istio.io "bookinfo" created

Steps to reproduce the bug
Follow the official docs to deploy Istio, then kubectl create -f samples/bookinfo/kube/bookinfo.yaml and kubectl create -f samples/bookinfo/routing/bookinfo-gateway.yaml
Version
$ kubectl version
Client Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.1", GitCommit:"d4ab47518836c750f9949b9e0d387f20fb92260b", GitTreeState:"clean", BuildDate:"2018-04-12T14:26:04Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.1", GitCommit:"d4ab47518836c750f9949b9e0d387f20fb92260b", GitTreeState:"clean", BuildDate:"2018-04-12T14:14:26Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"linux/amd64"}

$ istioctl version
Version: 0.8.0
GitRevision: 6f9f420f0c7119ff4fa6a1966a6f6d89b1b4db84
User: root@48d5ddfd72da
Hub: docker.io/istio
GolangVersion: go1.10.1
BuildStatus: Clean

Is Istio Auth enabled or not?
I used Option 1 from https://istio.io/docs/setup/kubernetes/helm-install/
 controlPlaneAuthPolicy: NONE
Environment
Running on VMware VMs:
    operatingSystem: linux
    osImage: Ubuntu 16.04.4 LTS

Terminal Log
$ helm template install/kubernetes/helm/istio --name istio --namespace istio-system --set sidecarInjectorWebhook.enabled=false > $HOME/istio.yaml
$ kubectl create namespace istio-system
$ kubectl create -f $HOME/istio.yaml
<SNIP>
customresourcedefinition.apiextensions.k8s.io "virtualservices.networking.istio.io" created
customresourcedefinition.apiextensions.k8s.io "destinationrules.networking.istio.io" created
customresourcedefinition.apiextensions.k8s.io "serviceentries.networking.istio.io" created
customresourcedefinition.apiextensions.k8s.io "gateways.networking.istio.io" created
customresourcedefinition.apiextensions.k8s.io "policies.authentication.istio.io" created
customresourcedefinition.apiextensions.k8s.io "httpapispecbindings.config.istio.io" created
customresourcedefinition.apiextensions.k8s.io "httpapispecs.config.istio.io" created
customresourcedefinition.apiextensions.k8s.io "quotaspecbindings.config.istio.io" created
customresourcedefinition.apiextensions.k8s.io "quotaspecs.config.istio.io" created
clusterrole.rbac.authorization.k8s.io "istio-ingress-istio-system" created
clusterrole.rbac.authorization.k8s.io "istio-mixer-istio-system" created
clusterrole.rbac.authorization.k8s.io "istio-pilot-istio-system" created
clusterrole.rbac.authorization.k8s.io "prometheus-istio-system" created
clusterrolebinding.rbac.authorization.k8s.io "prometheus-istio-system" created
clusterrole.rbac.authorization.k8s.io "istio-citadel-istio-system" created
role.rbac.authorization.k8s.io "istio-cleanup-old-ca-istio-system" created
clusterrole.rbac.authorization.k8s.io "istio-sidecar-injector-istio-system" created
clusterrolebinding.rbac.authorization.k8s.io "istio-ingress-istio-system" created
clusterrolebinding.rbac.authorization.k8s.io "istio-mixer-admin-role-binding-istio-system" created
clusterrolebinding.rbac.authorization.k8s.io "istio-pilot-istio-system" created
clusterrolebinding.rbac.authorization.k8s.io "istio-citadel-istio-system" created
rolebinding.rbac.authorization.k8s.io "istio-cleanup-old-ca-istio-system" created
clusterrolebinding.rbac.authorization.k8s.io "istio-sidecar-injector-admin-role-binding-istio-system" created
service "istio-egressgateway" created
service "istio-ingress" created
service "istio-ingressgateway" created
service "istio-policy" created
service "istio-telemetry" created
service "istio-statsd-prom-bridge" created
deployment.extensions "istio-statsd-prom-bridge" created
service "istio-pilot" created
service "prometheus" created
service "istio-citadel" created
service "istio-sidecar-injector" created
deployment.extensions "istio-egressgateway" created
deployment.extensions "istio-ingress" created
deployment.extensions "istio-ingressgateway" created
deployment.extensions "istio-policy" created
deployment.extensions "istio-telemetry" created
deployment.extensions "istio-pilot" created
deployment.extensions "prometheus" created
deployment.extensions "istio-citadel" created
deployment.extensions "istio-sidecar-injector" created
job.batch "istio-cleanup-old-ca" created
horizontalpodautoscaler.autoscaling "istio-egressgateway" created
horizontalpodautoscaler.autoscaling "istio-ingress" created
horizontalpodautoscaler.autoscaling "istio-ingressgateway" created
mutatingwebhookconfiguration.admissionregistration.k8s.io "istio-sidecar-injector" created

$ kubectl create -f samples/bookinfo/kube/bookinfo.yaml
service "details" created
deployment.extensions "details-v1" created
service "ratings" created
deployment.extensions "ratings-v1" created
service "reviews" created
deployment.extensions "reviews-v1" created
deployment.extensions "reviews-v2" created
deployment.extensions "reviews-v3" created
service "productpage" created
deployment.extensions "productpage-v1" created

$ istioctl create -f samples/bookinfo/routing/bookinfo-gateway.yaml
unable to recognize "istio-0.8.0/samples/bookinfo/routing/bookinfo-gateway.yaml": no matches for kind "Gateway" in version "networking.istio.io/v1alpha3"
unable to recognize "istio-0.8.0/samples/bookinfo/routing/bookinfo-gateway.yaml": no matches for kind "VirtualService" in version "networking.istio.io/v1alpha3"

$ kubectl get crd/gateways.networking.istio.io -o yaml
apiVersion: apiextensions.k8s.io/v1beta1
kind: CustomResourceDefinition
metadata:
  creationTimestamp: 2018-06-26T23:44:27Z
  generation: 1
  labels:
    app: istio-pilot
  name: gateways.networking.istio.io
  resourceVersion: "2789091"
  selfLink: /apis/apiextensions.k8s.io/v1beta1/customresourcedefinitions/gateways.networking.istio.io
  uid: dcf63fc2-799a-11e8-b32b-0050569353a1
spec:
  group: networking.istio.io
  names:
    kind: Gateway
    listKind: GatewayList
    plural: gateways
    singular: gateway
  scope: Namespaced
  version: v1alpha3
status:
  acceptedNames:
    kind: Gateway
    listKind: GatewayList
    plural: gateways
    singular: gateway
  conditions:
  - lastTransitionTime: 2018-06-26T23:44:27Z
    message: no conflicts found
    reason: NoConflicts
    status: "True"
    type: NamesAccepted
  - lastTransitionTime: 2018-06-26T23:44:27Z
    message: the initial names have been accepted
    reason: InitialNamesAccepted
    status: "True"
    type: Established

$ kubectl create -f<EMAIL_ADDRESS>unable to recognize<EMAIL_ADDRESS>no matches for kind "Gateway" in version "networking.istio.io/v1alpha3"
unable to recognize<EMAIL_ADDRESS>no matches for kind "VirtualService" in version "networking.istio.io/v1alpha3"

$ kubectl create -f<EMAIL_ADDRESS>unable to recognize<EMAIL_ADDRESS>no matches for kind "Gateway" in version "networking.istio.io/v1alpha3"
unable to recognize<EMAIL_ADDRESS>no matches for kind "VirtualService" in version "networking.istio.io/v1alpha3"

$ kubectl get crd/virtualservices.networking.istio.io -o yaml
apiVersion: apiextensions.k8s.io/v1beta1
kind: CustomResourceDefinition
metadata:
  creationTimestamp: 2018-06-26T23:44:27Z
  generation: 1
  labels:
    app: istio-pilot
  name: virtualservices.networking.istio.io
  resourceVersion: "2789084"
  selfLink: /apis/apiextensions.k8s.io/v1beta1/customresourcedefinitions/virtualservices.networking.istio.io
  uid: dcf4059c-799a-11e8-b32b-0050569353a1
spec:
  group: networking.istio.io
  names:
    kind: VirtualService
    listKind: VirtualServiceList
    plural: virtualservices
    singular: virtualservice
  scope: Namespaced
  version: v1alpha3
status:
  acceptedNames:
    kind: VirtualService
    listKind: VirtualServiceList
    plural: virtualservices
    singular: virtualservice
  conditions:
  - lastTransitionTime: 2018-06-26T23:44:27Z
    message: no conflicts found
    reason: NoConflicts
    status: "True"
    type: NamesAccepted
  - lastTransitionTime: 2018-06-26T23:44:27Z
    message: the initial names have been accepted
    reason: InitialNamesAccepted
    status: "True"
    type: Established

$ kubectl create -f<EMAIL_ADDRESS>unable to recognize<EMAIL_ADDRESS>no matches for kind "Gateway" in version "networking.istio.io/v1alpha3"
unable to recognize<EMAIL_ADDRESS>no matches for kind "VirtualService" in version "networking.istio.io/v1alpha3"

# I wait 2 minutes...

$ kubectl create -f<EMAIL_ADDRESS>gateway.networking.istio.io "bookinfo-gateway" created
virtualservice.networking.istio.io "bookinfo" created

$ kubectl get crd/virtualservices.networking.istio.io -o yaml
apiVersion: apiextensions.k8s.io/v1beta1
kind: CustomResourceDefinition
metadata:
  creationTimestamp: 2018-06-26T23:44:27Z
  generation: 1
  labels:
    app: istio-pilot
  name: virtualservices.networking.istio.io
  resourceVersion: "2789084"
  selfLink: /apis/apiextensions.k8s.io/v1beta1/customresourcedefinitions/virtualservices.networking.istio.io
  uid: dcf4059c-799a-11e8-b32b-0050569353a1
spec:
  group: networking.istio.io
  names:
    kind: VirtualService
    listKind: VirtualServiceList
    plural: virtualservices
    singular: virtualservice
  scope: Namespaced
  version: v1alpha3
status:
  acceptedNames:
    kind: VirtualService
    listKind: VirtualServiceList
    plural: virtualservices
    singular: virtualservice
  conditions:
  - lastTransitionTime: 2018-06-26T23:44:27Z
    message: no conflicts found
    reason: NoConflicts
    status: "True"
    type: NamesAccepted
  - lastTransitionTime: 2018-06-26T23:44:27Z
    message: the initial names have been accepted
    reason: InitialNamesAccepted
    status: "True"
    type: Established

$ kubectl get crd/gateways.networking.istio.io -o yaml
apiVersion: apiextensions.k8s.io/v1beta1
kind: CustomResourceDefinition
metadata:
  creationTimestamp: 2018-06-26T23:44:27Z
  generation: 1
  labels:
    app: istio-pilot
  name: gateways.networking.istio.io
  resourceVersion: "2789091"
  selfLink: /apis/apiextensions.k8s.io/v1beta1/customresourcedefinitions/gateways.networking.istio.io
  uid: dcf63fc2-799a-11e8-b32b-0050569353a1
spec:
  group: networking.istio.io
  names:
    kind: Gateway
    listKind: GatewayList
    plural: gateways
    singular: gateway
  scope: Namespaced
  version: v1alpha3
status:
  acceptedNames:
    kind: Gateway
    listKind: GatewayList
    plural: gateways
    singular: gateway
  conditions:
  - lastTransitionTime: 2018-06-26T23:44:27Z
    message: no conflicts found
    reason: NoConflicts
    status: "True"
    type: NamesAccepted
  - lastTransitionTime: 2018-06-26T23:44:27Z
    message: the initial names have been accepted
    reason: InitialNamesAccepted
    status: "True"
    type: Established

This is actually k8s issue. They just has fix here https://github.com/kubernetes/kubernetes/pull/63068

I have met same issue in 1.4.1 k8s cluster deployed by kubeadm.


Issue programing multiple devices at the same time

Hi,
I'm currently trying to flash multiple i.MX6 boards at the same time using release 1.5.21 on a Win10 PC. To this end I'm using multiple cmd instances calling uuu with the -m option to address specific USB devices.
Unfortunately, only one flash procedure appears to be performed at a time.
Having started one such flash procedure for one board always results in the other instances being aborted with errors "Fail HID(W):LIBUSB_ERROR_IO" or "Fail HID(W):LIBUSB_ERROR_TIMEOUT".
Using option -d allows to serialize these procedures. But what I'm aiming for is to parallelize them to save time during production.
I've already found issue #194 being very similar to what I'm facing right now and that it was solved using version 1.3.102. However, using other versions of the tool didn't help in my case.
No such issues are observed using the same setup to flash a single board.
Since one of the key features listed for the tool is "Multi devices program support" I'm quite sure that it's somehow possible to flash multiple boards at the same time.
Any ideas on what I'm doing wrong?

Can you post your cmd and error log? I am little confused

Hi and thanks again!
'uuu -lsusb' command leads to output
uuu (Universal Update Utility) for nxp imx chips -- libuuu_1.5.21-0-g1f42172

Connected Known USB Devices
        Path     Chip    Pro     Vid     Pid     BcdVersion
        ==================================================
        6:8422   MX6ULL  SDP:    0x---- 0x----   0x0001
        6:7422   MX6ULL  SDP:    0x---- 0x----   0x0001

(VID and PID are blanked out)
What can be observed most of the time is the following:
For the 1st instance running the command 'uuu -d -m 6:7422 u-boot-devel.imx' leads to:
Success 1    Failure 0     at path 6:7422
In a seperate CMD window for the 2nd instance running the command 'uuu -d -m 6:8422 u-boot-devel.imx' immediately after having started the 1st run leads to:
Success 0    Failure 1     at path 6:8422
6:8422   1/ 2 [HID(W):LIBUSB_ERROR_IO                ] SDP: boot -f "u-boot-devel.imx"

(LIBUSB_ERROR_IO can be observed with and without -d option)
And if this is of any help, (occasionally) even the first flash procedure fails with a timeout error:
Success 0    Failure 1     at path 6:7422
6:7422   1/ 2 [HID(R):LIBUSB_ERROR_TIMEOUT           ] SDP: boot -f "u-boot-devel.imx"

Strange, I use this ways every days.


Updated Glew to version 2.0.0

Updated Glew to latest stable version 2.0.0. Tested on Windows 10 with Visual Studio 2015.

Hi @sol-prog, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

Yes, it is a type, 2.0.0 is the correct version number.

I'll change this during the merge, thanks!

Great. Thanks.


8.4.0RC2 - File manager: context menu options for multiple folders do not work and most don't make sense

Requirement:
More than one folder exists in the file manager.
How to reproduce:

Select two folders, right-click, the context menu opens

In the context menu select any option, for example "Move to Folder"

Result: the indicator of the progress bar expands slowly to the right, stops just before finishing, the spinner turns forever, nothing else happens.

I'm seeing more than one issue here:

None of the options the context menu offers if multiple folders are selected work.
Most of these options don't even make sense for multiple folders:

Download (i don't believe that this should create a ZIP file containing all files of all subfolders)
Properties (doesn't make sense for multiple folders)
Sets (doesn't make much sense for multiple folders)
Rescan (this would make a lot of sense, but doesn't work either)
Move to Folder (this is what i wanted to do, since i see no other possibility to move folders into other folders that are not displayed on the same page of the File Manager)
Storage Location (doesn't make sense for multiple folders)
Delete (would make sense of course, but doesn't work)

Tested this on a clean installation 8.4.0RC2.

I can confirm that using the context menu/bulk command menu to move multiple folders into another folder fails.
@africanforest

Download (i don't believe that this should create a ZIP file containing all files of all subfolders)
Properties (doesn't make sense for multiple folders)
Sets (doesn't make much sense for multiple folders)
Rescan (this would make a lot of sense, but doesn't work either)
Move to Folder (this is what i wanted to do, since i see no other possibility to move folders into other folders that are not displayed on the same page of the File Manager)
Storage Location (doesn't make sense for multiple folders)
Delete (would make sense of course, but doesn't work)

I think these features would be useful for multiple folders, allowing for bulk changes.

Yes, the issue here is that the bulk file commands are showing up when selected folders instead of tree nodes. In this case we should show no active menu commands – there are no bulk commands that can apply to folders.

This is fixed in 8.4.1.

Folders can only be moved by drag & drop.


Sync projections and base model deleted state with CUSTOM_MODEL_CLASS

Projections and base models now have a synced up deleted state.

@hjdivad I'll transition the whole file to using native property access at once, dealing with schemas for each individual test was a bit obnoxious


Remove pagecontrol when max step is 1

Would be swell to add an option to remove the page control when items being used in the AlertOnboarding is only 1, so this can also be used as an alert.

On second thought disregard this, the would also mean changing the delegate options and e.t.c


Add reduction parameter to mean_pairwise_squared_error loss

Also increased clarify of the documentation for the function

Could you please let me know where I can add unit tests? Thanks.

(Fine for API change)

@ispirmustafa Could you please let me know where I can add unit tests? Thank you.

You can add tests here: python/kernel_tests/losses_test.py

Nagging Awaiting Response: It has been 14 days with no activityand the awaiting response label was assigned. Is this still an issue?

I'll close this for now, please reopen if you find time to add tests.


Konrads Issue 2.b

In order to stop manually dealing with escape characters, line breaks, etc.  as a member of Group 8, I can refactor chirp to use an external library for CSV handling.
Acceptance criteria:

Makes CSV handling easier

Label: Enhancement

VERY NICE


Use Path.Combine for Unzip and Nupkg paths

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

Project coverage is 66.75%. Comparing base (4c9354c) to head (7793f7c).

Additional details and impacted files
@@                      Coverage Diff                      @@
##           feature/sbom-targets-task     #663      +/-   ##
=============================================================
- Coverage                      69.69%   66.75%   -2.94%
=============================================================
  Files                            277      276       -1
  Lines                           8653     8586      -67
  Branches                        1009     1001       -8
=============================================================
- Hits                            6031     5732     -299
- Misses                          2105     2345     +240
+ Partials                         517      509       -8

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


[Feature Request]merge into diffusers so as more developers and integrate PulID into there system

Hi, I'm a developer working on SD and its relevant pipelines. PulID is a great tool for maintaining fidelity when generating new images. However, I find the code difficult to integrate with the Diffusers pipeline.
I would appreciate it if you could make it part of the Diffusers library.

Definitely this would be great.


Lax version requirement on Rails

This makes administrate-field-image compatible with Rails 5.

Closing in favor of https://github.com/thoughtbot/administrate-field-image/pull/6 and https://github.com/thoughtbot/administrate-field-image/pull/4.
@nickcharlton - all yours!


How do you load the parasitic information of the the design

I have a pre-routed design which has a dummy inverter/buffer cells already placed. I want to optimize the sizing of these gates based to get the lowest latency possible. Does that look like the correct use of the OpenPhySym tool?
I managed to get the tool running using docker and tried buffer_fanout_transformation.
I was wondering how to provide the parasitic information to the tool to get or does it calculated based on the tech.lef + def file provided?

Hi @ganeshgore, thanks for your interest.
The current approach relies on using an average wire parasitics estimation which is set using set_wire_rc  to extract the average values from the metal layers or set_wire_rc <res_per_micron> <cap_per_micron> to give custom wire RC.
The timing information is extracted from OpenSTA which supports read_spef command to feed parasitics information although we did not test this aspect so far.
I also recommend using repair_timing command instead which is the most comprehensive command in the tool to perform various optimization (you can check the different options it supports in the README file. The tool performs different optimizations to meet the timing constraints and solving electrical violations, if you want to meet higher constraints you can try tightening the clock period.
Let me know if anything is not clear, and feel free to open a suggestion ticket for features/enhancements that you think might be useful for your flow.


Move pkg/client package from neofs-api-go

Blocked by #61
Consider using param structure requests and responses.

Closed via #73.


Idea for better rails/ActiveRecord integration

I've been trying to see if shale would be a good choice for a rails app I'm working on. I saw in https://github.com/kgiszczak/shale/issues/16 that you pointed to the Custom Model documentation which seems helpful, but the downside is that in order to use that developers have to redefine the attributes that are already defined in schema.rb.
If I set up a very basic rails app with a schema that looks like this:
(Created a migration with rails generate migration CreatePerson first_name:string last_name:string address:string)
ActiveRecord::Schema[7.0].define(version: 2023_10_05_210646) do
  create_table "people", force: :cascade do |t|
    t.string "first_name"
    t.string "last_name"
    t.string "address"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
  end
end

Then I'm able to call Person.attribute_types and get something that has the type for each attribute key along with other schema-adjacent data:
irb(main):001> Person.attribute_types
=>
{"id"=>
  #<ActiveRecord::ConnectionAdapters::SQLite3Adapter::SQLite3Integer:0x00007f455dd81cc0
   @limit=nil,
   @precision=nil,
   @range=-9223372036854775808...9223372036854775808,
   @scale=nil>,
 "first_name"=>
  #<ActiveModel::Type::String:0x00007f455dd81310
   @false="f",
   @limit=nil,
   @precision=nil,
   @scale=nil,
   @true="t">,
 "last_name"=>
  #<ActiveModel::Type::String:0x00007f455dd81310
   @false="f",
   @limit=nil,
   @precision=nil,
   @scale=nil,
   @true="t">,
 "address"=>
  #<ActiveModel::Type::String:0x00007f455dd81310
   @false="f",
   @limit=nil,
   @precision=nil,
   @scale=nil,
   @true="t">,
 "created_at"=>#<ActiveRecord::Type::DateTime:0x00007f455dd413f0 @limit=nil, @precision=6, @scale=nil>,
 "updated_at"=>#<ActiveRecord::Type::DateTime:0x00007f455dd413f0 @limit=nil, @precision=6, @scale=nil>}

So I'm wondering if it would be feasible for shale to use that data so that instead of having to redefine attributes in our mapper like this:
class PersonMapper < Shale::Mapper
  model Person

  attribute :first_name, Shale::Type::String
  attribute :last_name, Shale::Type::String
  attribute :address, AddressMapper
end

we could do something like this and let shale create the attributes based on what's already defined on the model:
class PersonMapper < Shale::Mapper
  model Person

  attributes Shale::Schema.from_active_model(Person)
end

Hey Brooks, thanks for suggestions, but I want to keep the scope of this project limited.
Integrations with other projects is probably better put in external gems.
Also integration with rails is so trivial I don't think it needs specific treatment.
All you need is a mapping from rails to shale types. Here's a sample:
RAILS_TO_SHALE_TYPE_MAPPING = {
  integer: Shale::Type::Integer,
  string: Shale::Type::String,
  datetime: Shale::Type::Time,
}

class PersonMapper < Shale::Mapper
  model Person

  Person.attribute_types.each do |name, att|
    attribute name.to_sym, RAILS_TO_SHALE_TYPE_MAPPING[att.type]
  end
end

That makes sense, thank you for the feedback! I'll work on a solution like your example.


graphql-sequelize: Include support has been removed

Brilliant work on graphql-sequelize-crud!  Looking to make use of this package with latest graphql (v0.7.2) and graphql-sequelize (v4.0.0) updates, but when attempting to run graphql-sequelize-crud Demo after updates get AssertionError:  "Include support has been removed in favor of dataloader batching".
Error arises from: graphql-sequelize\lib\resolver.js:41:24 (https://github.com/mickhansen/graphql-sequelize/search?utf8=✓&q="Include+support+has+been+removed+in+favor+of+dataloader+batching")
After loading dataloader-sequelize package get same error.  Have not looked far into a solution, but as you are likely already aware of the issue, was wondering if graphql-sequelize-crud will be updated  for  graphql-sequelize (v4.0.0)?

Note: After removing all instances of include: false property from resolver functions in src/index.js the Demo runs again without issue.  There was a comment in src/index.js that include was set to false to disable auto including of associations based on AST - default: true.  While the Demo now works again are there other implications of include support being removed from graphql-sequelize in favor of the dataloader-sequelize package that need to be considered?

Pull Request #15 has been merged. Hope that helps!

Hi,
First off, awesome job on this!    Heads up, I pulled the package from npm and it still has the include: false properties in its lib/index.js file.  I ran into this same error, modified my local copy and got up and running fairly quickly.  I see the github version is modified, so not sure what to make of that.
Thanks!

Thanks for the reminder, @jblank46. I need to publish latest to npm.

Published v0.3.0


Consider adding sort to Iterator.

Just like the rev function, sort would alter the order depending on the factors you supply. Like, sort from low to high, sort from high to low. You know. All that sorting stuff.
However, since it's an iterator it would probably be more efficient than collecting and sorting.

Collecting and then sorting is really the only way to sort; ultimately we'd need to at least buffer the results of the iterator somewhere, so this isn't in any way an improvement over collecting and then sorting as far as I know. There's some convenience functions which do sort of this in itertools: https://docs.rs/itertools/0.6.0/itertools/trait.Itertools.html#method.sorted.

I thought you only needed to get the smallest element (or, well, according to a closure or et.c), but I guess I was wrong.

We don't have a concept of random access iterator :(
sort could be applied to such iterators without extra space.


Fix warning: more use of == in void context, again

Please see https://github.com/mikel/mail/pull/326

Nice, thanks :)


Default order direction is documented wrong & not explicitly tested for

The documentation here and on postgrest.com states that ?order=col will default to sorting in descending order. In fact, there is a tangentially related test that depends on the opposite (ASC default) being true.
Also, there's no test dedicated to this detail either way.
(Aside: Is there a preferred way to submit documentation issues?)

Uh oh, sorry for the inaccurate docs. The pages you see on postgrest.com come from markdown in the docs/ folder of this repo, for instance https://github.com/begriffs/postgrest/blob/master/docs/api/reading.md
Thanks for pointing out the problem. If you send a pull request to correct this file I'll run the script to deploy your change to the site.


[FEATURE] Support for Thread ID in API Requests for OpenAI Assistant

Current Behavior
Currently, the API does not seem to support thread management. In my usage, I've noticed that the API retrieves a random thread for each call, which makes it challenging to handle specific conversation threads in applications that manage multiple chats simultaneously.
Desired Behavior
The proposed feature would allow users to send a specific thread ID with each API request. This would enable the API to handle requests in the context of the specified thread, rather than randomly selecting a thread. This feature is crucial for building more structured and organized chat management systems using the Flowise API.
Use Case
This feature is particularly useful for applications that need to manage multiple, concurrent chat conversations. By allowing the API to handle specific threads, developers can create more sophisticated and user-friendly chat interfaces.
Questions

Is this feature currently possible with the existing API?
If not, what would be the process for considering and potentially implementing this feature?

I have reviewed the existing code and documentation but haven't found any references to thread management in the API. I look forward to hearing your thoughts on this feature request.
Thank you for considering this enhancement to the Flowise API.

It does. Just specify chatId in your body request as documented here - https://docs.flowiseai.com/memory/threads


config options for nested tables

config_datasets.json = {
"102887025":["hits","sessions","products"]
,"24973611":["hits","sessions","products"]
}
instead of the current config:
config_datasets.json =
{"datasets": ["102887025", "24973611", "83262411"]}

Release v1.3.0 has fixed this issue

Release v1.3.0 has fixed this issue


DisparateImpactRemover is not importing in fresh installation

Sumary
DisparateImpactRemover fails to import on a fresh installation of OS and conda environment.
Steps for reproduction
Package installed using: pip install holisticai
then:
from holisticai.bias.mitigation.preprocessing import DisparateImpactRemover

the outcome is:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ImportError: cannot import name 'DisparateImpactRemover' from 'holisticai.bias.mitigation.preprocessing' (/home/username/miniconda3/envs/venv/lib/python3.9/site-packages/holisticai/bias/mitigation/preprocessing/__init__.py)

Version information

Operating system: Ubuntu 20.04 LTS
Python version: 3.9.16
holisticai version: 0.5.0

Thanks!

@fracarfer5 For mitigators you have to install the full version:
pip install holisticai[methods]

Thanks!! It is working now


Automatic v1.0.x -> master upmerge

The more people are committing to v1.0.x, the more this PR will evolve. We can merge it anytime to keep master in sync, and recreate it right after.

Let's merge it ASAP to fix the broken ObjC tests in master.

Oh, because this isn't from your fork it doesn't get Travis. Should we re-enable?

Go for it.
On Tue, Jul 12, 2016 at 6:14 PM, Jorge Canizales<EMAIL_ADDRESS>wrote:

Oh, because this isn't from your fork it doesn't get Travis. Should we
re-enable?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/grpc/grpc/pull/7357#issuecomment-232228117, or mute
the thread
https://github.com/notifications/unsubscribe/AG8bpvvWX719uQI2WhKJ-pV_iUKCkLTkks5qVDv6gaJpZM4JK96X
.

Enabled. You might need to do a dummy push into v1.0.x for it to trigger though.

@nicolasnoble are you fine if I merge and recreate?

I'm fine with that, yes.


Add Logging Functionality ( #111 feat: log to a file)

This pull request addresses Issue #111.
Changes Made:

Added logger.ts file in c-qube/src/utils directory.
Implemented logging functionality using the popular winston library, including TypeScript typings (@types/winston).
Chose winston for logging due to its flexibility and extensive feature set compared to a simple fs solution.
Included an example of how to use the logger in the code snippet provided.

Usage Example:
const { setupLogger } = require('./utils/logger');

const logger = setupLogger('app.log', 'info');

logger.info('This is an info message');
logger.warn('This is a warning message');
logger.error('This is an error message');

Additionally, while working on this task, I noticed the debug.ts file, which appears to provide similar functionality. Please let me know if this implementation meets the requirements outlined in the issue, or if there is any other specific approach I should consider.
Please review and provide feedback. Thank you!

@ChakshuGautam can you go through this?

can I please review these changes @techsavvyash ?

Hey @Bigguysahaj,
Since you are unresponsive to the changes required for this PR to become mergeable, I am marking this PR as closed. Please feel free to reopen it once you have finished it.
Thanks for your contribution


Encrypt and Decrypt byte[] without string conversion using AES

When using the library to encrypt and decrypt PDF files received through an ASP.NET Core endpoint using AES, the PDF file after en- and decryption would be corrupted. I determined this was due to the byte[] being converted to a string before encryption and decryption.
Implementing the same methods without this conversion solved the issue. The existing EncryptContent and DecryptContent methods for strings were not modified.

TK-SMF First, thank you for your PR and your suggestion.
Secondly, I have created a bug issue with your description.
Back to the code review, after investigating the issue, I agree with you, that when decrypting PDF, it makes the PDF corrupted, no doubt in that. the issue. The solution you have provided will work but still has issues. In one way it solves the PDF corruption issue but it introduces another issue. When I compare the original PDF before encryption and PDF after decryption, some bytes are missing.  After spending some time and unit testing, I am going to make a hotfix ASAP.
The problem really Encoding.ASCII in CryptoNetUtils.cs and the solution is in this answer https://stackoverflow.com/a/49106300/3088349

TK-SMF wait a minute, I am still spending some time to figure out if that is the real solution.

@TK-SMF now it is fixed on branch https://github.com/maythamfahmi/CryptoNet/tree/feature/41-major-imporvement
If you don't mind, pull the branch and test it for your solution, let me know so I can create a new PR and release an update Nuget package.

@TK-SMF I will close this PR and let's communicate on #41


Multi-language support

Currently, the supplement of a super-figure is set to "Figure" explicitly, because it cannot be resolved by user code in the show rule in order to apply it to its children regardless of their kind.
Better support for resolving auto values in typst itself or better native i18n support like in typst#3728 could solve this.
Alternatively, the linguify package or my unfinished translation package could be used to provide translations.

Since linguify doesn't seem to have a built-in mechanism for handling regions, I've hand rolled a simple solution instead.


Disable reply-to does not work on auto-added VPN rules

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[x] I have searched the existing issues and I am convinced that mine is new.

Describe the bug
Auto-added VPN firewall rules do not take the global firewall setting "Disable reply-to on WAN rules" into account.
To Reproduce
Steps to reproduce the behavior:

Setup an IPsec VPN tunnel.
Make sure that auto-added VPN firewall rules are active (uncheck Disable Auto-added VPN rules under VPN -> IPsec -> Advanced Settings).
Output firewall rules on console with command pfctl -sr.
Now globally disable reply-to on WAN rules (Firewall -> Settings -> Advanced).
Output firewall rules on console with command pfctl -sr.

Expected behavior
Auto-added VPN firewall rules should not  being generated with a reply-to expression when reply-to is globally disabled.
Relevant log files
pass out log on pppoe0 route-to ( pppoe0 fe80::xxxx:xxxx:xxxx:xxxx ) proto udp from {any} to {any} port {500} keep state label "37[...]c7" # IPsec: Mobile Devices
pass in log on pppoe0 reply-to ( pppoe0 fe80::xxxx:xxxx:xxxx:xxxx ) proto udp from {any} to {any} port {500} keep state label "27[...]81" # IPsec: Mobile Devices
Environment
OPNsense 21.1.5-amd64

You should be able to find the responsible toggle using the magnifying glass at the end of the automatically generated rules, I expect your looking for " Disable all auto-added VPN rules." in the advanced IPsec settings.

I want to use auto-added VPN rules.
The problem is that OPNsense does not honor the global setting "Disable reply-to on WAN rules" when creating auto-added VPN rules.
Auto-added VPN rules should be written without reply-to expression when reply-to on WAN rules is globally disabled under Firewall -> Settings -> Advanced.

Currently it's probably best to disable the auto generated vpn rules, although it's debatable if it should add route-to in this case, it might have side affects for others (these rules are mainly an inheritance from the past).
For a major update it's no problem to change this, a minor I'm not sure yet.


Example/Stats/Stackdriver: consistent with stackdriver doc example

Same as :
https://cloud-dot-devsite.googleplex.com/monitoring/alpha-opencensus/docs/

Codecov Report

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

@@            Coverage Diff            @@
##             master    #1540   +/-   ##
=========================================
  Coverage     82.94%   82.94%
  Complexity     1659     1659
=========================================
  Files           251      251
  Lines          7720     7720
  Branches        754      754
=========================================
  Hits           6403     6403
  Misses         1097     1097
  Partials        220      220

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 71dba4a...16cb944. Read the comment docs.


Error CORS when call method exportImg

Describe the bug
I have img tag with src url from s3 and when i click exportImg this function can not working.
Link image s3
In nodeContent:

Error CORS when call exportImg

Expected behavior
method exportImg working and export new image

When exporting, org-chart tries to convert images in bas64 string, to do that it needs to access and download images. So, the URL which you specify for images has to be accessible via CORS.
As a workaround, maybe try base64 image URLs instead. Not exactly sure if it will work though. Let me know if it does


Document all the timing loops provided by Bencher

If I implement #23 and #24, the documentation will have to be updated to show how the timing loop looks like and what's its time model. The documentation should also discuss when to use each timing model.

Done.