[SPARK-19092] [SQL] [Backport-2.1] Save() API of DataFrameWriter should not scan all the saved files #16481

What changes were proposed in this pull request?
This PR is to backport https://github.com/apache/spark/pull/16481 to Spark 2.1

DataFrameWriter's save() API is performing a unnecessary full filesystem scan for the saved files. The save() API is the most basic/core API in DataFrameWriter. We should avoid it.
How was this patch tested?
Added and modified the test cases

Test build #71402 has started for PR 16588 at commit a246ae9.

Test build #71402 has finished for PR 16588 at commit a246ae9.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71402/
Test FAILed.

retest this please

Test build #71403 has started for PR 16588 at commit a246ae9.

Test build #71403 has finished for PR 16588 at commit a246ae9.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71403/
Test PASSed.

cc @cloud-fan

thanks, merging to 2.1

Thanks!


Chattriggers compatibility

Describe the bug
Chattriggers freezes my display totally.
To Reproduce
Steps to reproduce the behavior:

install Minecraft
Click on Install forge 1.8.9
Add the chattriggers mod
OS freezes after a while

Expected behavior
Normally i should have a working problem
Screenshots
Can't screenshot on a frozen pc
Setup (please complete the following information):

OS: Ubuntu linux
ChatTriggers Version: latest, downladed today on the home page - Minecraft Version: 1.8.9
Java Version: java 8

Additional context
Add any other context about the problem here.

Hmm, I've never had any issue with my WHOLE computer freezing, but I did with MC + CT freezing when using the Minecraft Launcher's bundled JRE. Try setting your game profile to use a manually downloaded JRE 8 version, and see how that works.

Im already using a manually installed java version. The base jre u get is java 11, and i needed it to be 8

Or am i talking jive

Are you using OpenJDK? If so try with an oracle jre/jdk.

i think im using oracle

If you want to, join our discord server (https://discord.gg/chattriggers), its easier to provide support there.

sorry
WHat i meant is i dont think i am


Feature request. Add hook for calling sendPasswordResetEmail

Hi,
It would be nice to have a wrapper with loading/error state for auth.sendPasswordResetEmail(email). Thanks.

This has just been done here https://github.com/CSFrequency/react-firebase-hooks/commit/08489c5de26b366669149fc98df449527fe59df6 and will be released in the upcoming v5 in the next week or two

v5 is now released: https://github.com/CSFrequency/react-firebase-hooks/releases/tag/v5.0.0


Fix examples and refactor code to ensure working examples

This PR addresses tasks in issue #2.  It will be work-in-progress until all the tasks are completed.  Will comment when completed.

All tasks are done.  Note that the AMQP example is a little buggy (doesn't completely close out everything).  Appears to be a problem with underlying library (node-amqp).  Before v1.0.0 release we should upgrade to squaremo/amqp.node.


Appstudio update test-component-pac-dloi

Pipelines as Code configuration proposal

Pipelines as Code CI/test-component-pac-dloi-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
7 seconds
init

✅ Succeeded
15 seconds
clone-repository

✅ Succeeded
1 minute
build-container

✅ Succeeded
21 seconds
deprecated-base-image-check

✅ Succeeded
20 seconds
inspect-image

✅ Succeeded
3 minutes
clair-scan

✅ Succeeded
51 seconds
clamav-scan

✅ Succeeded
18 seconds
sbom-json-check

✅ Succeeded
8 seconds
optional-label-check

✅ Succeeded
9 seconds
label-check

✅ Succeeded
20 seconds
show-sbom

✅ Succeeded
7 seconds
show-summary

Pipelines as Code CI/test-component-pac-dloi-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
7 seconds
init

✅ Succeeded
15 seconds
clone-repository

✅ Succeeded
1 minute
build-container

✅ Succeeded
21 seconds
deprecated-base-image-check

✅ Succeeded
20 seconds
inspect-image

✅ Succeeded
3 minutes
clair-scan

✅ Succeeded
51 seconds
clamav-scan

✅ Succeeded
18 seconds
sbom-json-check

✅ Succeeded
9 seconds
label-check

✅ Succeeded
8 seconds
optional-label-check

✅ Succeeded
20 seconds
show-sbom

✅ Succeeded
7 seconds
show-summary


at:put: for ByteArray needs to work if value is a Character

... or did I fix this in trunk? Cannot remember...

Duplicate #68.
at:put: does not work with characters on Cog, either. The problem is the primitive fallback code for replaceFrom:to:with:startingAt:.


SSR Method Server-side rendering with Javascript disabled?

Package Versions:
next: 14.1.0
@apollo/client: 3.9.10
@apollo/experimental-nextjs-app-support: 0.9.1

Hi,
I'm using Nextjs with the App router, and have followed the instructions for incorporating the SSR method from this library. I'm also using it with the useSuspenseQuery hook. Is it possible to have the page render right away, even with JS disabled? As soon as I introduce this hook, all I see is the loading.ts content, even on reloads where it seems to have cached the content server-side.
In my testing it works perfect fine when using the RSC method and passing in the data, so the client components can render fine on the server. Id just like to be able to use SSR/useSuspenseQuery to have proper hydration/client-side cache-and-network to have the latest data.
Here's my code:
// apollo/ApolloWrapper.tsx
'use client';
declare module '@apollo/client' {
  export interface DefaultContext {
    token?: string;
  }
}

import { ApolloLink, HttpLink } from '@apollo/client';
import { setContext } from '@apollo/client/link/context';
import {
  ApolloNextAppProvider,
  NextSSRInMemoryCache,
  NextSSRApolloClient,
  SSRMultipartLink,
} from '@apollo/experimental-nextjs-app-support/ssr';
import typePolicies from './typePolicies';

const uri = `${process.env.NEXT_PUBLIC_GRAPHQL_URL}/graphql`;

// have a function to create a client for you
function makeClient() {
  const authLink = setContext(async (_, { headers, token }) => {
    return {
      headers: {
        ...headers,
        ...(token ? { authorization: `Bearer ${token}` } : {}),
      },
    };
  });

  const httpLink = new HttpLink({
    // this needs to be an absolute url, as relative urls cannot be used in SSR
    uri,
    // you can disable result caching here if you want to
    // (this does not work if you are rendering your page with `export const dynamic = "force-static"`)
    // fetchOptions: { cache: 'no-store' },
  });

  return new NextSSRApolloClient({
    cache: new NextSSRInMemoryCache({
      typePolicies,
    }),
    link:
      typeof window === 'undefined'
        ? ApolloLink.from([
            new SSRMultipartLink({
              stripDefer: true,
            }),
            authLink.concat(httpLink),
          ])
        : authLink.concat(httpLink),
  });
}

// you need to create a component to wrap your app in
export function ApolloWrapper({ children }: React.PropsWithChildren) {
  return (
    <ApolloNextAppProvider makeClient={makeClient}>
      {children}
    </ApolloNextAppProvider>
  );
}

// app/layout.tsx
export default function RootLayout({ children }: { children: ReactNode }) {
  return (

        <ApolloWrapper>
          <AppProviders>
            <HeaderWrapper>{children}</HeaderWrapper>
          </AppProviders>
        </ApolloWrapper>

  );
}

// app/content/[id]/page.tsx
export default async function Page({ params }: Props) {
  const id = params.id
  return <ContentPage id={id} />;
}

// app/content/[id]/loading.tsx
export default async function Loading() {
  return
Loading...

}

export default function ContentPage({
  id,
}: {
  id: number;
}) {
  const { data } = useSuspenseQuery(GetContentDocument, {
    variables: {
      id,
    },
    fetchPolicy: 'cache-and-network',
  });

  return
{data?.content?.title}

}

It would work if your browser were a search engine, because in that case, Next.js renders the whole page on the server and flushes it when it is finished - but in the case of normal browsers, Next.js uses React's "out of order" streaming and that requires JS to be enabled.
Your page will still load with the .js files 100% blocked, but there is a little bit of JS required to "stitch it together" correctly, because it it streaming over parts of the page as they finish rendering on the server, not in the order they actually appear in the DOM.

It would work if your browser were a search engine, because in that case, Next.js renders the whole page on the server and flushes it when it is finished - but in the case of normal browsers, Next.js uses React's "out of order" streaming and that requires JS to be enabled.

Ok, good to know! Search engines/SEO is my main concern, so it seems like the next/vercel magic will do that with this lib. Thanks for all your expertise and great work on this library!


fix k8s version to 1.9.9

fix #1121

Coverage increased (+17.3%) to 72.187% when pulling 19c757977061c96ce16c999f1872b29d413ead3f on yife/fix into ae6f7d520d9891876ffbc5d7ae1399395bd9df5b on master.

Coverage increased (+17.3%) to 72.187% when pulling 19c757977061c96ce16c999f1872b29d413ead3f on yife/fix into ae6f7d520d9891876ffbc5d7ae1399395bd9df5b on master.

cherry-picked to #1165


stepmania: 5.0.12 -> 5.1.0-b2

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/stepmania/versions.
meta.description for stepmania is: '"Free dance and rhythm game for Windows, Mac, and Linux"'.
Release on GitHub
Compare changes on GitHub

Checks done (click to expand)

built on NixOS

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

directory tree listing: https://gist.github.com/2a371e1b362825f16898b736a1d4bf2a

du listing: https://gist.github.com/be13b71fc46a9b3274db5dabf76c7304

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)
1 package rebuild(s)
1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
stepmania

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/awp56c7zrigcacxh4pfz58d7fh2y9wj5-stepmania-5.1.0-b2 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A stepmania https://github.com/r-ryantm/nixpkgs/archive/161bbb9dbf930f4a83cb2459a729e3a69027eb43.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/awp56c7zrigcacxh4pfz58d7fh2y9wj5-stepmania-5.1.0-b2
ls -la /nix/store/awp56c7zrigcacxh4pfz58d7fh2y9wj5-stepmania-5.1.0-b2/bin

beta release


Archive availabilities reflect availability of children

I think the tests will explain this better than I can

You'll need to freeze the Catalogue API and pipeline before you merge this, and then we'll fold it into the next reindex.


feat: use quick_select for statusline

I think bester if we use quick_select for statusline.
The priority: tag name -> quick_select -> index

Great idea, thanks! Just a couple clean-up suggestions.

Updated, can you take a look

Fixed a small typo. Merged. Thanks!


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/28878

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/29197

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/29512

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/49793


Command line parameter is not fully read

Detected in https://github.com/flutter/flutter/issues/84270
With the below example:
void main(List<String> args) {
  print('args: $args');
}

Running
>dart test.dart "^(?!Golden).+"

Expected
^(?!Golden).+

Actual:
(?Golden).+

Dart SDK version: 2.14.0-228.0.dev (dev) (Sat Jun 19 18:06:49 2021 -0700) on "windows_x64"
Windows 10

@asashour is it always the first character that's being dropped, or is it just the ^ character?

It seems that regardless of the position, ^ and ! are always removed.
>dart test.dart "a^b!c"
abc

>dart test.dart "a!@#$%^&*()b"
a@#$%&*()b

@asashour which terminal are you using? Command Prompt, Powershell, Linux Subsystem for Windows, Cygwin, etc.?
I'm unable to reproduce with my initial tests on bleeding edge on Windows and on Linux (although I ran into some weird bash behavior when using ! with double quotes). I'll checkout 2.14.0-228.0.dev next and give that a shot.

I am using windows command prompt.

I think this is just the CMD doing something weird and you might just need to escape some characters. Can you try running this: dart test.dart "^^(?^^!Golden).+"?

Indeed, it can be escaped by prefixing both characters with ^.
However, I don't think this is CMD-related, since Java works without escaping.
You can close this issue if you believe so.
public class Test {
  public static void main(String[] args) {
    System.out.println(args[0]);
  }
}

>dart test.dart "^^(?^^!Golden).+"
args: [^(?^Golden).+]

>dart test.dart "!!"
args: []

>dart test.dart "\!"
args: [" ]

>dart test.dart "^!"
args: [!]

C:\use\flutter>dart test.dart "^^(?^!Golden).+"
args: [^(?!Golden).+]

>java Test "^(?!Golden).+"
^(?!Golden).+

>java -version
openjdk version "11.0.11" 2021-04-20
OpenJDK Runtime Environment AdoptOpenJDK-11.0.11+9 (build 11.0.11+9)
OpenJDK 64-Bit Server VM AdoptOpenJDK-11.0.11+9 (build 11.0.11+9, mixed mode)

I cannot reproduce either. Windows 10, built from bleeding edge:
λ dart test.dart "^(?!Golden).+$"
args: [^(?!Golden).+$]

λ dart --version
Dart SDK version: 2.14.0-edge.28c784d87b0def07e63a1d5053fb9e7b6240c635 (be) (Wed May 19 12:35:16 2021 +0000) on "windows_x64"

You say you're getting Dart through Flutter. Is Flutter possibly using a different compiler, with a preamble which works differently on the command line before passing it to the main function (the C++ one, not the Dart main, that comes later).

Flutter is using dart.bat wrapper https://github.com/flutter/flutter/blob/master/bin/dart.bat which I think causes this behavior because it does SETLOCAL ENABLEDELAYEDEXPANSION.

because it does SETLOCAL ENABLEDELAYEDEXPANSION

That's definitely the issue, so this is a problem with Flutter. Will follow up on flutter/flutter#84270.


Support Swift Package Manager in Xcode 11

Hey!
In Xcode 11 Swift Package Manager is built in and works extremely well as a dependency manager, automatically resolving dependencies without messing around with the project structure like Cocoapods does.
I'm not sure how it works with frameworks written in Objective-C, whether there can be a Swift wrapper around the framework which is exposed in a Swift Package, but since Xcode 11 this will very quickly kill of Cocoapods it's a really good idea to start thinking about how to support this now.
The majority of new Xcode projects are Swift-first and now with SPM built in, people will expect to be able to import frameworks without Cocoapods.

Hey @KaneCheshire! Thanks for the thoughtful note. I'll review this with my team and work on next steps.
Stay tuned.

Assets are now supported :) Any chance this can go ahead now.

Hi @vrutings . Stay tuned!

Hi @CaseyApptentive. Looking forward to the SPM support as Apptentive is one of the few libraries that are blocking my team to remove Cocoapods. Thank you.

No problem @scier, we appreciate the patience as we worked to support this :)


Have a way to get the current stack trace without throwing exceptions.

There's currently no way to get the full, current stack trace. This means that there's also no way for an asynchronous method to give an accurate stack trace when it has an error. For example:
  Future<String> getLinks(String url) {
    return fetchPage(url).transform((html) => parseLinks(html));
  }
Now suppose fetchPage detects an error asynchronously and wants to display a stack trace for that error. A useful stack trace would show that it had originally been called by getLinks, but there's no way for it to know that.

Never say never - https://github.com/dart-lang/sdk/commit/027b8dca39a753a95b01a9845d4fc5f7cd08be1a


Path must be a string. Received undefined.

Please provide us with the following information:

OS?  Windows 7, 8 or 10. Linux (which distribution). Mac OSX (Yosemite? El Capitan?)
--> Debian 8.0 Jessie 64-bit

Versions. Please run ng --version. If there's nothing outputted, please run
in a Terminal: node --version and paste the result here:
--> Node 6.5.0
-->  angular-cli: 1.0.0-beta.11-webpack.8 node: 6.3.1 os: linux x64

Repro steps. Was this an app that wasn't created using the CLI? What change did you
do on your code? etc.

The log given by the failure. Normally this include a stack trace and some
more information.Path must be a string. Received undefined TypeError: Path must be a string. Received undefined  at assertPath (path.js:7:11)  at Object.resolve (path.js:1148:7)  at Object.getWebpackCommonConfig (/home/ferdinand/dev/project/client/node_modules/angular-cli/addon/ng2/models/webpack-build-common.ts:140:35)  at new NgCliWebpackConfig (/home/ferdinand/dev/project/client/node_modules/angular-cli/addon/ng2/models/webpack-config.ts:13:30)  at Class.exports.default.Task.extend.run (/home/ferdinand/dev/project/client/node_modules/angular-cli/addon/ng2/tasks/serve-webpack.ts:19:22)  at /home/ferdinand/dev/project/client/node_modules/angular-cli/addon/ng2/commands/serve.ts:84:26  at process._tickCallback (internal/process/next_tick.js:103:7)

Mention any other details that might be useful.

This happens with serve as well as with build, regardless of whether an environment argument (e.g. --dev) is passed or not.
Debugged webpack-build-common.ts at the mentioned position and everything seems to be fine there, meaning that both appRoot and appConfig.environments['source'] (or appConfig.environments['dev'] ) when running ng serve --dev are a string containing a path.

Thanks! We'll be in touch soon.

I was receiving this same error after upgrading from RC5 to RC6 on:
angular-cli: 1.0.0-beta.11-webpack.8
node: 6.5.0
os: win32 x64
I added three lines to my angluar-cli.json that seemed to fix the issue.
      "scripts": [],
      "styles": [],
      "assets": "assets",

I looked through webpack-build-common.ts and it looks like scripts and styles should have defaulted to [] if they were undefined but they weren't for some reason. I am not sure if that is a bug or an issue with something that I was doing. Assets didn't seem to have any default value if it was not defined so that explains the "Received undefined TypeError".
I still have to debug other errors that I believe are unrelated before I can confirm this fixes the issue.

I ran into this issue as well and it was because angular-cli.json was out of date. I updated it based on what ng new would generate for a new project and now things work.

@fmccready adding the "assets" property solved it for me. Thank you. Unfortunately there's no documentation on that.


added cluster setting stats.jobs_log_expiration

refers to https://github.com/crate/crate/pull/4648

Please use the pattern in StatsTables to implement the functionality.
You need a variable to hold the last setting and one that holds the initial setting to revert to when reset command is issued (StatsTables.NodeSettingListener)


No verification string is displayed after the UTF-8 character is signed.

Test case:
./bin/keytool sign SCMURBUOY76KQK6MUAIU47XVKJG3H7P5O6HOLVVOOGO57KIJJIHSLE4N "헨리"
Signed message: 38B280A70B76560956D114A6A4E9A6A4BFAC04FA051B8D5B51246A4F2165B8EDEB4F768F8351F765B1E8DCA8391FF37AEF6EB1F8F1BA4EDFB8D9B90B27FE4907ED97A8EBA6AC
./bin/keytool verify GD2MCBTM6JKDU6AONX2RLWG46HQGWBF32OIDVHKF4UCZC2SEBS36FYUM 38B280A70B76560956D114A6A4E9A6A4BFAC04FA051B8D5B51246A4F2165B8EDEB4F768F8351F765B1E8DCA8391FF37AEF6EB1F8F1BA4EDFB8D9B90B27FE4907ED97A8EBA6AC
Result: ED97A8EBA6AC
Result as string is not displayed.
Is it correct that UTF-8 does not support it?

Assigned to @AndrejMitrovic

@TrustHenry : Can you confirm the issue is fixed ?

renames account to @TrustAndrej


chore: dependabot combined packages/e2e updates

Describe your changes
Ticket: https://pins-ds.atlassian.net/browse/ASB-2300
PR combines all the dependabot updating dependancies in packages/e2e-tests:

https://github.com/Planning-Inspectorate/applications-service/pull/1012
https://github.com/Planning-Inspectorate/applications-service/pull/1008
https://github.com/Planning-Inspectorate/applications-service/pull/1007
https://github.com/Planning-Inspectorate/applications-service/pull/1006
https://github.com/Planning-Inspectorate/applications-service/pull/1005

The main change is cypress updating to 13.6.4 and some other indirect dependancy updates only updating on the package-json.json. Running e2e tests locally caused some failures due to timeouts not being long enough for submissions, so have increased that. Some of these tests do fail but that is roughly the same as main - due to class name changes and required maintenance changes that will come in the future.
Note:

Combining the changes in one PR will allow for all the changes in this package to be validated at once and dependabot should automatically recognise this and remove the alerts for these dependancies.
Dependabot commits were too long and failed commit lint checks so I've squashed it for the PR and listed the individual PRs above.

Type of change 🧩

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update
[x] Other (please explain in the description section above)

Checklist before requesting a review

[x] I have performed a self-review of my own code
[x] I have double checked this work does not include any hardcoded secrets or passwords
[x] I have made corresponding changes to the documentation
[x] I have provided details on how I have tested my code
[x] I have referenced the ticket number above
[x] I have provided a description of the ticket
[x] I have included unit tests to cover any testable code changes

:tada: This PR is included in version 2.19.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.24.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


NoMethodError when triggering a jsonapi_render_not_found error

Hi there,
Firstly, thank you for this gem, it makes using JR much easier when dealing with API's more complex than the simple examples in the docs. I really appreciate it.
I'm getting this weird error when triggering a jsonapi_render_not_found error. It seems the params hash gets lost somewhere. Anyway, here's the code (it's for a multi-tenant system, which is why I'm searching from the current_user.
module API
  module V1
    class AccountsController < AuthenticatedController
      include JSONAPI::ActsAsResourceController

      def show
        jsonapi_render json: current_user.accounts.find(params[:id])
      end
    end
  end
end

And here's the exception and stacktrace... I hope you can help...
 I do really appreciate your feedback :-)
In order to make sure that I will lead you to the right answer, I would like to know which version of JR and JU you are using in your project.

Hey Tiago,
Thanks for getting back to me :), this is straight from my Gemfile.lock
jsonapi-resources (0.7.0)
jsonapi-utils (0.4.0)

Great! I've caught the bug. I will be merging a hotfix asap.

Fixed :-)

Awesome sauce!!! Muito obrigado!


Invoke-SSHCommand slow to many hosts over slow link

Invoke-SSHCommand is slow if you are running against a number of systems over a slow link.  Looking at the Invoke-SSHCommand code, it looks like it should be possible to do an BeginExecute on each connection, assign the results to an array, then iterate through them in the "end" section of the script.  This way they would execute in parallel, but you could get the results from them after the fact.  This should make some scripts much faster when ran against multiple systems.

That would mean me following state per connectiononly working with x number of connections at once, finish those, grab the next ones and probably working with runsspaces. A change that big with my current workload would not even be possible for months for me to experiment. But I do accept pull requests ;)
Sent from my iPhone

On May 29, 2015, at 4:51 PM, FireInWinter<EMAIL_ADDRESS>wrote:
Invoke-SSHCommand is slow if you are running against a number of systems over a slow link. Looking at the Invoke-SSHCommand code, it looks like it should be possible to do an BeginExecute on each connection, assign the results to an array, then iterate through them in the "end" section of the script. This way they would execute in parallel, but you could get the results from them after the fact. This should make some scripts much faster when ran against multiple systems.
—
Reply to this email directly or view it on GitHub.

Shouldn't have to go to the extreme of using runspaces.  You already are using async methods, it just has to go into an array.  But I'll try to get things setup so I can put in a pull request.  Haven't done anything with git yet, so it will take me a bit to figure it out.

So far my experience is that powershell sucks at asynchronous tasks and they have to be handled by jobs, runsspaces or as a cmdlet in C#. Also if running against multiple sessions it should have a control to run against x amount of sessions at once like other cmdlet a that have a limit of 32 parallel tasks at a time.
Sent from my iPhone

On May 29, 2015, at 5:32 PM, FireInWinter<EMAIL_ADDRESS>wrote:
Shouldn't have to go to the extreme of using runspaces. You already are using async methods, it just has to go into an array. But I'll try to get things setup so I can put in a pull request. Haven't done anything with git yet, so it will take me a bit to figure it out.
—
Reply to this email directly or view it on GitHub.

Well.  I had some time so I put together a quick version that does it async.  It doesn't have limits on parallel tasks, but it is using remote threads instead of local.  And the main cost with async of establishing the connection is already done.  I use the variable $AsyncProcessing to keep track of the different connections and their status.   In my case, it took getting results from over 200 servers from 25 seconds to 9 seconds.  It should be better, but it isn't really multi-threaded.  So it has to start the commands sequentially and then end them again sequentially.
I'm actually not sure how bad it is not to have a threshold on it.  Long term it could be added, but short term it is more useful to have it faster, but not a perfect design.  The biggest reason to limit parallelism, is that resources are consumed locally to set up all the connections.  In this case, the connections are already established, so the cost doesn't really come into play.
function Invoke-SSHCommand
{
    [CmdletBinding(DefaultParameterSetName='Index')]
    param(

        [Parameter(Mandatory=$true,
                   Position=1)]
        [string]$Command,

        [Parameter(Mandatory=$true,
                   ParameterSetName = 'Session',
                   ValueFromPipeline=$true,
                   Position=0)]
        [Alias('Name')]
        [SSH.SSHSession[]]
        $SSHSession,

        [Parameter(Mandatory=$true,
                   ParameterSetName = 'Index',
                   Position=0)]
        [Alias('Index')]
        [int32[]]
        $SessionId = $null,

        # Ensures a connection is made by reconnecting before command.
        [Parameter(Mandatory=$false)]
        [switch]
        $EnsureConnection,

        [Parameter(Mandatory=$false,
                   Position=2)]
        [int]
        $TimeOut = 60

    )

    Begin
    {
        $ToProcess = @()
        switch($PSCmdlet.ParameterSetName)
        {
            'Session'
            {
                $ToProcess = $SSHSession
            }

            'Index'
            {
                foreach($session in $Global:SshSessions)
                {
                    if ($SessionId -contains $session.SessionId)
                    {
                        $ToProcess += $session
                    }
                }
            }
        }

        # array to keep track of all the running jobs.
        $AsyncProcessing = @()
    }
    Process
    {
        foreach($Connection in $ToProcess)
        {
            if ($Connection.Session.IsConnected)
            {
                if ($EnsureConnection)
                {
                    try
                    {
                        $Connection.Session.Connect()
                    }
                    catch
                    {
                        if ( $_.Exception.InnerException.Message -ne 'The client is already connected.' )
                        { Write-Error -Exception $_.Exception }
                    }
                }
                #$cmd = $Connection.session.CreateCommand($Command)
            }
            else
            {
                try
                {
                    $Connection.Session.Connect()
                }
                catch
                {
                    Write-Error "Unable to connect session to $($Connection.Host) :: $_"
                }

            }

            if ($Connection.Session.IsConnected)
            {
                $cmd = $Connection.session.CreateCommand($Command)
                $cmd.CommandTimeout = [timespan]::FromMilliseconds(50) #New-TimeSpan -Seconds $TimeOut

                # start asynchronious execution of the command.
                $Duration = [System.Diagnostics.Stopwatch]::StartNew()
                $Async = $cmd.BeginExecute()

                $AsyncProcessing  += [pscustomobject]@{
                    cmd = $cmd
                    Async = $Async
                    Connection = $Connection
                    Duration = $Duration
                    Processed = $false
                }
            }
        }
    }
    End
    {
        $Done = $false
        while ( -not $Done )
        {
            $AsyncProcessing |
            % {
                # Check if it completed or is past the timeout setting.
                if ( $_.Async.IsCompleted -or $_.Duration.Elapsed.TotalSeconds -gt $TimeOut )
                {
                    $Output = $_.cmd.EndExecute($_.Async)

                    # Generate custom object to return to pipeline and client
                    [pscustomobject]@{
                        Output = $Output
                        ExitStatus = $_.cmd.ExitStatus
                        Error = $_.cmd.Error
                        Host = $_.Connection.Host
                        Duration = $_.Duration.Elapsed
                    } |
                    % {
                        $_.pstypenames.insert(0,'Renci.SshNet.SshCommand');

                        #Return object to pipeline
                        $_
                    }

                    # Set this object as having been processed.
                    $_.Processed = $true
                }
            }

            # Check if the list is done or sleep for 50 ms.
            $AsyncProcessing = @($AsyncProcessing | ? { -not $_.Processed } )
            if ( $AsyncProcessing.Count -eq 0 )
            { $Done = $true }
            else { Start-Sleep -Milliseconds 50 }
        }
    }
}


[Fab] Inconsistent click event behavior

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
Props of a Fab component like "name" and "value" are often inaccessible when handling click events.
Expected Behavior 🤔
Those props should always be accessible inside event.target.value and  event.target.name.
Steps to Reproduce 🕹
Codesandbox comparing the click event values for Fab, SpeedDialAction and button
(SpeedDialAction is included because it uses the Fab component and I had originally an issue with it).
This bug is quite odd... the events seem to register when clicking the bottom or top of a FAB but not anywhere else. Maybe it's an issue with the icons ?
Your Environment 🌎

Tech
Version

Material-UI
v4.9.5

React
16.13.0

Browser
Firefox and Chrome

You probably want to use currentTarget instead of target.


Server import function not available on 0.6.6?

Hello,
i search the Import function to import a existent Server on the New Version, but i can not find this function?

There isn't an import function, but the equivalent is to use restore from backup.
Select the server profile you want to import to.
Select the restore file
Save settings
Then click restore selected file
It will import the file to you save game folder for the selected profile. It will rename the file to be the correct name for dedicated servers.
It works with server files, zip files containing a server file, and directories containing zip files containing server files

ok thanks


Running Whisper on Cloud GPU by 3rd Party Services

It is unlikely that Whisper can be used on the cloud as an extra option, as it requires a powerful GPU to run efficiently. However, some software companies offer Whisper as an API, such as Replicate. This allows users to access Whisper's functionality without needing to have a powerful GPU. More information about using Whisper through Replicate can be found at the following link: https://replicate.com/openai/whisper.

I second that! OpenAI recently released their official API and it would be great to have an option to set up an API key and access larger models from the cloud.
I tested out a basic python setup with the API and was able to transcribe a 13 minute file in just one minute. Compared to my laptop using the medium model which took 58 minutes, the difference is massive. Even though the API isn't free, the pricing makes it a very viable option.

I second that! OpenAI recently released their official API and it would be great to have an option to set up an API key and access larger models from the cloud.
I tested out a basic python setup with the API and was able to transcribe a 13 minute file in just one minute. Compared to my laptop using the medium model which took 58 minutes, the difference is massive. Even though the API isn't free, the pricing makes it a very viable option.

Yes there are others similar to the company you cited. However, there are issues linked to cloud usage vs local processing (I.e. pricing, security,privacy etc). However, local transcription has been moving very rapily in terms of speed improvements. The work being carried out by Chidi and others will shrink the differences between offline and cloud offerings...just stay tuned :-)

It seems Cidi has added support for Whisper API: https://github.com/chidiwilliams/buzz/pull/371

You can create API key on the OpenAI Website
Currently they gift $18.00 for the new users.
The only thing to note is that the file size is currently limited to 25MB. But since we already have ffmpeg, I guess it would not be too hard to slice the files before sending them to the API.
@chidiwilliams Should I start a new issue with a slicing suggestion?

Yes, Whisper API is supported in v0.7.2. I'll have a look at slicing.


:bug: fix hcl appscan, issue #9279

issue #9279

Could you take a look and review this also please @mtesauro ?


Add support for ISO calendar week to Time

This PR adds two methods to Time for handling ISO calendar week:

Time#calendar_week returns the year and week number. The year is important because calendar weeks don't match up with calendar years and the calendar week might actually be in a different year than the regular calendar year (returned by #year).
Time.week_date creates a Time instance from a week date specified by year, week number and day of week.

It also adds directives %G, %g, %V which utilize these methods to Time::Format.
The algorithms for calculating the week number and back are completely annotated in code. They're based on the description on Wikipedia and the implementation in Go's stdlib but with a few improvements.
Closes #2533
Depends on #6555

needs a rebase

Rebased.

@straight-shoota Sorry, could you rebase again this on master so we can merge in a more tidy fashion?

Sure, done. But I'm not sure what it adds, it could just be fast forwarded.

Thanks! A rebase will not leave a text reference to the PR. Squash or merge commit will. But is better to avoid long standing branch merged and keep the repo as thin as possible.


Inconsistent behaviour of CSVDataset with eager disabled/enabled

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): macOS
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: N/A
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): v1.10.0-12-g4dcfddc5d1 1.10.1
Python version: 3.6.5
Bazel version (if compiling from source): N/A
GCC/Compiler version (if compiling from source): N/A
CUDA/cuDNN version: N/A
GPU model and memory: N/A
Exact command to reproduce: see below

Describe the problem
The following code snippet works without any errors in eager mode but fails when eager is disabled
import tensorflow as tf
data = tf.contrib.data.CsvDataset("foo.csv", [tf.constant(0, dtype=tf.int64)])
data.make_one_shot_iterator().get_next()

wheer foo.csv is generated by echo 42 > foo.csv. The issue is fixed by making 42 a single-element list following the example in the CsvDataset documentation.
Source code / logs
Eager execution disabled
>>> data.make_one_shot_iterator().get_next()
Traceback (most recent call last):
  File "[...]/tensorflow/python/framework/ops.py", line 1576, in _create_c_op
    c_op = c_api.TF_FinishOperation(op_desc)
tensorflow.python.framework.errors_impl.InvalidArgumentError: Shape must be rank 1 but is rank 0 for 'CSVDataset' (op: 'CSVDataset') with input shapes: [], [], [], [], [], [], [0], [].

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "[...]/tensorflow/python/data/ops/dataset_ops.py", line 170, in make_one_shot_iterator
    six.reraise(ValueError, err)
  File "[...]/six.py", line 692, in reraise
    raise value.with_traceback(tb)
ValueError: Shape must be rank 1 but is rank 0 for 'CSVDataset' (op: 'CSVDataset') with input shapes: [], [], [], [], [], [], [0], [].

Eager execution enabled
>>> data.make_one_shot_iterator().get_next()
(<tf.Tensor: id=53, shape=(), dtype=int64, numpy=42>,)

Thanks @rachellim!


Ghost Vacuum Chest [Bug] [1.10.2]

Issue Description:
On a Server, I administrate a Player had a 100% vanilla blaze grinder built with 1 Exception, he had placed a Vacuum chest at the bottom of it to collect the drops. At 1 point he decided to break the vacuum chest. When he broke the chest, it never dropped or showed in his inventory. He also at that point was unable to get drops from blazes, at that point he had me come to investigate the problem.
I search the area and find no reason his blazes were not dropping stuff, till I had the Idea of throwing junk out of my inventory and fix it getting sucked into a piece of vanilla netherrack which was placed in the spot of his old vacuum chest. I searched the entire area made sure there was no hidden vacuum chest. I then had the idea to throw ~20 stacks of netherrack into the spot it was getting sucked up till it stopped. I have placed and removed blocks from the exact spot his chest was/ where items were getting sucked too, so it wasn't a simple matter of a block getting broken client side, but not server side.
This event did take place inside a FTB utilities claim. I have been unable to repeat this bug, nor do I have the faintest clue on what happened. Nor do I know if FTB led to this bug being created.
What you expected to happen:
You break a vacuum chest and you pick it up, and the area no longer tries to vacuum up items.

Affected Versions

EnderIO: 1.10.2-3.1.157
EnderCore: 1.10.2-<IP_ADDRESS>-beta
Minecraft: 1.10.2
Forge: <IP_ADDRESS>5
Other Mods that Might caused the problem FTBLib-1.10.2-3.2.3 & FTBUtilities-1.10.2-3.2.4

I see what's happening. However, Ender IO is not involved here, that is pure Mojang code acting here. So this is either a vanilla bug or some other mod is messing with the world state (yes, FTBUtils is a good candidate). Ender IO certainly is not changing the way the world state is manipulated when a block is broken (and we won't start doing so, not even to make sure the state is consistent...)


Hide internal modules

Could export them in an Internal.* module.

Ended up moving them from exposed-modules to other-modules in the cabal file.


Issue with Kino.Input.number & Firefox

The Arrows of the Number Input are not recognized (at least in Firefox).
Keyboard Arrow Up and Down Works.

Update
I just checked it in Chrome and there it is working

We use phx-debounce="blur", so the change is reflected when the input is focused, changed and blurred. In other browsers hitting the up/down arrows focuses the input, so it is blurred on submit as expected. In Firefox however, hitting up/down doesn't change focus and it looks like a known bug (or at least known behaviour). So this applies to LV in general, cc @chrismccord.

Closing this a known firefox bug. :(


[Feature request] Possibility to sort by "New" and "Updated" in non-default App Loaders

I'm currently using 3 App Loaders:

https://espruino.github.io/BangleApps/
https://hilmarst.github.io/BangleApps/
https://banglejs.com/apps/

Only the last one has the possibility to sort by "New" and "Updated" - if I want to see what's new, I have to look through the closed pull request.
It would be nice to have these sort possibilities also on the non-default App Loaders.

Answered in your duplicate post here: http://forum.espruino.com/conversations/371794/#comment16347776

Funny that "TheDod" opened it in the forum at the same time - but it's not me!

Heh - sorry! It was so close I just assumed it's the same :)


When I import BetterTransfomer, It Return Syntax Error

System Info
optimum = 1.7.1
Python Version == 3.7.11

Who can help?
Thank you everyone, when I try to import the bettertransformer, It return syntax error.
from optimum.bettertransformer import BetterTransformer
Error
`Traceback (most recent call last):
File "/opt/conda/lib/python3.7/site-packages/IPython/core/interactiveshell.py", line 3457, in run_code
exec(code_obj, self.user_global_ns, self.user_ns)
File "/tmp/ipykernel_1227/438332947.py", line 1, in
from optimum.bettertransformer import BetterTransformer
File "/home/jovyan/.local/lib/python3.7/site-packages/optimum/bettertransformer/init.py", line 15, in
from .transformation import BetterTransformer
File "/home/jovyan/.local/lib/python3.7/site-packages/optimum/bettertransformer/transformation.py", line 30
**def raise_save_or_push_incompatible(dummy, /, *_, __):
^
SyntaxError: invalid syntax`
Please help to solve this problem, Thank you very much!
Information

[ ] The official example scripts
[ ] My own modified scripts

Tasks

[ ] An officially supported task in the examples folder (such as GLUE/SQuAD, ...)
[ ] My own task or dataset (give details below)

Reproduction

pip install --user optimum[onnxruntime]
from optimum.bettertransformer import BetterTransformer`

Expected behavior
I expected to successfully load the BetterTransformer.

This is due to the syntax raise_save_or_push_incompatible(dummy, /, *_, __) that is valid only from python 3.8 onwards.
This was fixed in https://github.com/huggingface/optimum/pull/881 and will be included in a patck release this week. As a general recommendation, I would advise to use python >=3.8 though, as python 3.7 end of life comes soon: https://devguide.python.org/versions/


Shortcut to sort Projects, Labels, or Filters in the side panel?

No clue if this is possible, but I like having my Projects, Labels, & Filters in alphabetical order in the side panel, and it's really annoying that I have to tediously do this manually every time I add or rename one of them...
Thanks for a great extension, otherwise!

Hey, thanks!  Glad you appreciate the extension!
I don't think such a feature is within the scope of todoist-shortcuts, though, so closing this feature request as "wontfix".  If only I had infinite time for all the things!


[5X backport] Fix aocsseg inconsistency when rollback add column

Before current commit we initailized pg_aocsseg entries with frozen
tuples to make these entries implicitly visible even after rollback.
It is a working approach for AO tables, but AOC tables store additional
"vpinfo" structure with serialized information about every column EOF.
This can cause inconsistency between "vpinfo" structure in pg_aocsseg
entries and the actual number of columns in a table when we modify the
amount of columns under explicit transaction and rollback it later.
As a result we fail on asserts in "GetAllAOCSFileSegInfo_pg_aocsseg_rel()"
or (in a case of a build without asserts) in some unpredictable places
in the code when retrieve metadata from pg_aocsseg.
As a fix this commit inserts simple tuples to pg_aocsseg but still insert
frozen tuples to gp_fastsequence (as it generates row numbers for segment
files and should never be rollbacked to avoid inconsistency in index pointers
of the AO/AOC tables).
Backport of a5e28ec66dd07b4e1863bff785c951ffbd74aa52
Co-authored-by: Vasiliy Ivanov<EMAIL_ADDRESS>

TODO: fix mixed tabs/spaces

@Stolb27 I have inspected aocssegfiles.c file and it has a mix of tabs and spaces inside (mostly tabs are used). This mix comes from the legacy initial GP5x commit  and I don't want to change it. My modifications also use tabs (as the most popular indentation in the file), so I don't like the idea to change it to spaces.


helmfile: 0.130.1 -> 0.131.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/roboll/helmfile/releases.
meta.description for helmfile is: "Deploy Kubernetes Helm charts"
meta.homepage for helmfile is: "https://github.com/roboll/helmfile"
meta.changelog for helmfile is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 0.131.0 with grep in /nix/store/q37cmv9wx92jhslk4z87k6nnsbg7q03k-helmfile-0.131.0

directory tree listing: https://gist.github.com/0e470f9e8561157f53d6bcdbfd26ac3a

du listing: https://gist.github.com/438fcf372eb6294b96307606e081f69a

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
helmfile

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/q37cmv9wx92jhslk4z87k6nnsbg7q03k-helmfile-0.131.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A helmfile https://github.com/r-ryantm/nixpkgs/archive/b116c54879127228670ac2ff3a76eb59b59c9273.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/q37cmv9wx92jhslk4z87k6nnsbg7q03k-helmfile-0.131.0
ls -la /nix/store/q37cmv9wx92jhslk4z87k6nnsbg7q03k-helmfile-0.131.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review 1

1 package built:

helmfile

Maintainer pings
cc @pneumaticat @yurrriq @kalbasit for testing.

thanks!


How to create item without target/group?

Is there any way to create item w/o target? Consider this scenario, when Catalog is only used to group several links and there is no such page?

Catalog

Products
Categories

Unfortunatelly it's not possible right now. Currently I just use some other page as target (like products page).
It's definitely not a good solution and I plan to add support for that

It is possible to use javascript:void(); as target, but you have to rewrite code of carrooi menu to catch javascript: as new type of target.

With 2.0.0 you don't need to specify any link or action in which case # will be used as a link. Or you can specify your own link with the link option:
Catalog:
  link: 'javascript:void();'
  items:
    Products:
      action: :Products:default
    Categories:
      action: :Categories:default


Does it support Always Free Autonomous Databases creation?

Error returned by Database Service
Http Status Code: 403.
Error Code: Forbidden.
Message: Always Free Autonomous Databases do not offer the following features and operations: CPU scaling, storage scaling, storage shrinking, private endpoints, manual backups, restore, preview version software, Oracle Database edition, Oracle Service Gateway connect, DB tools, wallet rotation with grace period, and license updating. Paid Autonomous Database instances cannot be created in a Free-Tier only account. Paid Autonomous Databases cannot be cloned to Always Free Autonomous Databases.

Hi @mathiasmaciel97, the ADB controller doesn't only support paid ADBs in the current release. I'm working on the enhancement to support Always Free ADBs.


Fix for MSBuild command parser confused by Fully Qualified Path to proj files in Linux root #1894

The FileUtilities.LooksLikeLinuxPath() only returns true if the first path
segment is a directory. This can't be true for files in the root directory
on *nix systems. This commit adds an additional check to determine if the
parameter is a file.

@dotnet-bot test OSX Build for CoreCLR please

@Sarabeth-Jaffe-Microsoft any chance someone can look at the OSX build failure?
I'm getting this from the console output of the build...
06:41:47 Run condition [Current build status] enabling prebuild for step [[Archive the artifacts]]
06:41:48 [_OSX_CoreCLR_prtest] $ /bin/sh -xe /var/folders/zz/zyxvpxvq6csfxvn_n0000000000000/T/hudson3370030304074114963.sh
06:41:48 + ./cibuild.sh --scope Test --target CoreCLR
06:41:48 Installing dotnet cli...
06:54:06 curl: (56) SSLRead() return error -9806
06:54:06 ./sdk/1.0.0-preview3-003686/Microsoft.ApplicationInsights.dll: (Empty error message)
06:54:06 tar: Error exit delayed from previous errors.
06:54:06 Restoring BuildTools version 1.0.27-prerelease-00927-05...
06:54:06 Failed to load , error: dlopen(, 1): no suitable image found.  Did find:
06:54:06 	/usr/local/lib/: not a file
06:54:06 	/usr/lib/: not a file
06:54:06 ERROR: Could not restore build tools correctly. See '/Users/dotnet-bot/j/workspace/Microsoft_msbuild/master/_OSX_CoreCLR_prtest/init-tools.log' for more details.1
06:54:06 Initializing BuildTools...
06:54:06 /Users/dotnet-bot/j/workspace/Microsoft_msbuild/master/_OSX_CoreCLR_prtest/init-tools.sh: line 122: /Users/dotnet-bot/j/workspace/Microsoft_msbuild/master/_OSX_CoreCLR_prtest/packages/Microsoft.DotNet.BuildTools/1.0.27-prerelease-00927-05/lib/init-tools.sh: No such file or directory
06:54:06 ERROR: An error occured when trying to initialize the tools. Please check '/Users/dotnet-bot/j/workspace/Microsoft_msbuild/master/_OSX_CoreCLR_prtest/init-tools.log' for more details.1

I re-triggered the build in case it was a once off.

@dotnet-bot test OSX Build for CoreCLR please

@mmitche Our OS X build is failing because of a weird curl SSL issue.  Do you know if anything has changed with the OS X machines lately or have any idea what could cause this?
07:50:25 curl: (56) SSLRead() return error -9806
07:50:25 ./shared/Microsoft.NETCore.App/1.0.1/System.Net.Sockets.dll: (Empty error message)

@jeffkl @patros Depends on what the test is.  The lab that the mac machines are in is having severe network issues at the moment.  Did you receive an outage notification (Vs Engineering Outage mail)

Did you receive an outage notification

Apparently I did, sorry I missed that.  Since the action that is failing is downloading a .tar, we'll just assume its related to the network issue.  Thanks!

@jeffkl Almost certainly.  Speeds are extremely low.

@jeffkl changed tack a bit to make it easier to unit test. Looks like the OSX build servers are working again too.

@rainersigwald @cdmihai it is a terrible check to have to do. I couldn't think of a good way around it though. I've made it more explicit as requested.

Yeah, that check is wayyyy better than my original one

@patros, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, .NET Foundation Pull Request Bot

@dotnet-bot test Windows_NT Build for Desktop please

Thanks for the fix, @patros!


Check if the user is logged before accessing pages

I've noticed that there is no way to check if access to a page is allowed, maybe after a login.
I would like to know if there is a way to prevent a non-logged user from accessing pages, either with a 403 error or with a redirect.
Thanks.

I think pc.redirect could work for this where you check if the user is logged in before redirecting, in terms on not loading the page you could check if the users is logged in before loading any data on the page.
If you want something more robust like prohibiting pages based on something in your state I can look into implementing this.

You can use this statement :
if request.user.is_authenticated:     #code

You can just use this :
if request.user.is_authenticated:
    #redirect
else:
    #403 page

I think pc.redirect could work for this where you check if the user is logged in before redirecting, in terms on not loading the page you could check if the users is logged in before loading any data on the page.
If you want something more robust like prohibiting pages based on something in your state I can look into implementing this.

Thanks for your answer.
Yes with pc.redirect we can do this. But i'm looking for a way to prohibit access to pages if navigated from url directly.
I will try to explain better.
I've for example a login page at /login and a main page at /main.
The user must login before entering the main page, if it navigate to /main it can see the page, even without data, but the structure of the page with hardcoded data is also exposed and i think that this is not good.
I've tried this but i don't know if there is a better solution.
return pc.cond(State.is_logged, home(), error404()) where is_logged is a boolean variable set to True after the login.

You can just use this : if request.user.is_authenticated:  #redirect else:   #403 page

Where should i use this, inside the auth method?
But if i go directly to the link? That's my problem.
I want to prevent users to access some pages without auth.

Where should i use this, inside the auth method?
But if i go directly to the link? That's my problem.
I want to prevent users to access some pages without auth.

You would need to utilize cookies for that. As much as I know, Pynecone does not support that at the moment, if ever.
Pynecone is not a suited framework for websites that have authentication. I don't understand why some people like Line Indent do tutorials on a login page and Pynecone supporting it, when it isn't really useful anyway because anyone could just skip to the next url.

@HellAmbro I think your method of using the cond makes sense. You can add an on_load event to your page that will trigger when the page loads - and it can redirect them if they are not logged in. This can prevent them from accessing the page.
@nshout We're still pretty new but our goal is to support all features of traditional webdev, including cookies in the future.

@HellAmbro I think your method of using the cond makes sense. You can add an on_load event to your page that will trigger when the page loads - and it can redirect them if they are not logged in. This can prevent them from accessing the page.
@nshout We're still pretty new but our goal is to support all features of traditional webdev, including cookies in the future.

Thanks, i've tried but it doesn't seem to work, maybe i was wrong.
app.add_page(home, title="Aleica 2022", on_load=State.is_authenticated)
class State(pc.State):

    username: str
    logged_in: bool = False

    def is_authenticated(self):
        if not self.logged_in:
            return pc.redirect("/403")

logged_in is correctly set (I've followed Twitter example) if I login, but when i try to navigate to /home the page is still loaded.
How can i fix that and redirect the user to error page?
Thanks

Use this mayeb:
if request.user.is_authenticated:
    do something
else:
403page

@Alek99
pc.redirect doesn't do anything if it is in "State" and loaded from an on_load in add_page. (Chained)
The label is not "enhancement" at this point anymore, its a "bug"/"needs investigation".
I have not seen this issue as an already reported issue, if it is, apologies.
As seen on this code by @HellAmbro
class State(pc.State):

   username: str
   logged_in: bool = False

   def is_authenticated(self):
       if not self.logged_in:
           return pc.redirect("/403")

And as stated by @Lendemor

However, on_load event chain are still broken (the function execute, but not the one chained (in your case pc.redirect) so once this is fixed it will worked.

@picklelo

@nshout We're still pretty new but our goal is to support all features of traditional webdev, including cookies in the future.

I totally understand that, I forgot how new Pynecone actually is and that it's not ready for production as of now. I truly have high hopes, as Pynecone seems to enable what many Python devs are looking for or are dreaming of.
Apologies for everyone that I tagged.


Vue devtools freezes pages. Stuck on 'normalizeRouterInfo' function.

Devtools causes page to freeze on normalizeRouterInfo -> deepClone  -> clone. The clone function keeps calling itself. Sadly I do not know how to reproduce this.

Can you share the vue-router config or try to give a mini repo 🙏


Fix most of the examples

Compilation with gcc tested.
The file fftdata is missing in fft_test_r2c.
This should help https://github.com/xcompact3d/2decomp-fft/issues/28

Tested with intel compiler : make CMP=intel BUILD=debug && make CMP=intel BUILD=debug examples is working

Hi Guys
with Paul we added
call MPI_COMM_RANK(MPI_COMM_WORLD,nrank,ierror)
call MPI_COMM_SIZE(MPI_COMM_WORLD,nproc,ierror)

in  decomp_2d_init because without this none of the examples is going to work. At the moment I cannot find this not in the main branch nor in any other.
Cedric your branch fix-examples does not work with openmpi,
rfj82982@ceg-rolfo-2 2decomp-fft % make FC=mpif90-openmpi-mp examples
mpif90-openmpi-mp  -cpp -fallow-argument-mismatch -O3 -march=native -J mod -Imod  -DDOUBLE_PREC -DVERSION=\"b3ad163\"  -c src/decomp_2d.f90 -o obj/decomp_2d.o
mpif90-openmpi-mp  -cpp -fallow-argument-mismatch -O3 -march=native -J mod -Imod  -DDOUBLE_PREC -DVERSION=\"b3ad163\"  -c src/d2d_log.f90 -o obj/d2d_log.o
mpif90-openmpi-mp  -cpp -fallow-argument-mismatch -O3 -march=native -J mod -Imod  -DDOUBLE_PREC -DVERSION=\"b3ad163\"  -c src/./glassman.f90 -o obj/./glassman.o
mpif90-openmpi-mp  -cpp -fallow-argument-mismatch -O3 -march=native -J mod -Imod  -DDOUBLE_PREC -DVERSION=\"b3ad163\"  -c src/fft_generic.f90 -o obj/fft_generic.o
ar rcs libdecomp2d.a obj/decomp_2d.o obj/d2d_log.o obj/./glassman.o obj/fft_generic.o
/Applications/Xcode.app/Contents/Developer/usr/bin/make -C examples
/Applications/Xcode.app/Contents/Developer/usr/bin/make -C test2d test2d
mpif90-openmpi-mp   -cpp -fallow-argument-mismatch -O3 -march=native -J mod -I../../mod -c test2d.f90
f951: Warning: Nonexistent include directory 'mod' [-Wmissing-include-dirs]
mpif90-openmpi-mp -o test2d test2d.o -L../../ -ldecomp2d
ld: library not found for -lSystem
collect2: error: ld returned 1 exit status
make[2]: *** [test2d] Error 1
make[1]: *** [test2d] Error 2
make: *** [examples] Error 2

In my case works (i.e. build) only with mpich. Later on I’ll try to fix and start to run some tests.

It looks like something has gone wrong with the mod directories, although strange that it would be ok with MPICH... Let me try here what's happening

It's the space between J and $(MODDIR), I think I have a fix...

@rfj82982 are you talking about the commit below ?
https://github.com/mathrack/2decomp-fft/commit/67931a2b6cb28dd538cadcc15acb67f6e5165917

@pbartholomew08 the space was added at some point for the intel compiler

yes, although I don't know why as I have a space in the variable definition unless someone stripped it, i.e. the variable was defined as MODFLAG = -module  (note space at the end of line).

Assuming the space is the issue it should (I think) be fixed by #38

@rfj82982 which compiler are you wrapping with mpif90? Using the same command line you show it complains about non-existant mod dir when building examples (see above), however it does build with mpif90+gfortran-9.4

In my case I have gnu 12.1 on Mac. The same that I am using also with mpich and in that case I have only warnings but i have all the executables
rfj82982@ceg-rolfo-2 2decomp-fft % mpif90-openmpi-mp -v
Using built-in specs.
COLLECT_GCC=/opt/local/bin/gfortran-mp-devel
COLLECT_LTO_WRAPPER=/opt/local/libexec/gcc/aarch64-apple-darwin21/12.1.0/lto-wrapper
Target: aarch64-apple-darwin21
Configured with: /opt/local/var/macports/build/_opt_bblocal_var_buildworker_ports_build_ports_lang_gcc-devel/gcc-devel/work/gcc-12-branch-a613c586b5461d7fb99fa076ef7e59eca4227fae/configure --prefix=/opt/local --build=aarch64-apple-darwin21 --enable-languages=c,c++,objc,obj-c++,lto,fortran,jit --libdir=/opt/local/lib/gcc-devel --includedir=/opt/local/include/gcc-devel --infodir=/opt/local/share/info --mandir=/opt/local/share/man --datarootdir=/opt/local/share/gcc-devel --with-local-prefix=/opt/local --with-system-zlib --disable-nls --program-suffix=-mp-devel --with-gxx-include-dir=/opt/local/include/gcc-devel/c++/ --with-gmp=/opt/local --with-mpfr=/opt/local --with-mpc=/opt/local --with-isl=/opt/local --with-zstd=/opt/local --enable-stage1-checking --disable-multilib --enable-lto --enable-libstdcxx-time --with-build-config=bootstrap-debug --with-as=/opt/local/bin/as --with-ld=/opt/local/bin/ld --with-ar=/opt/local/bin/ar --with-bugurl=https://trac.macports.org/newticket --enable-host-shared --with-gxx-libcxx-include-dir=/opt/local/libexec/llvm-13/include/c++/v1 --with-pkgversion='MacPorts gcc-devel 12-20220615_0+enable_stdlib_flag' --with-sysroot=/Library/Developer/CommandLineTools/SDKs/MacOSX12.sdk
Thread model: posix
Supported LTO compression algorithms: zlib zstd
gcc version 12.1.0 (MacPorts gcc-devel 12-20220615_0+enable_stdlib_flag)

In my case I have gnu 12.1 on Mac. The same that I am using also with mpich and in that case I have only warnings but i have all the executables

ok - I'll try again later on a different computer/compiler

One day, we will have automatic tests for compilation for more compilers / MPI library ! See this PR for a way to have automatic tests with both gnu and intel compilers.

@rfj82982 could you add a MacOs build in the following PR ? https://github.com/xcompact3d/2decomp-fft/pull/39
It should be possible : https://docs.github.com/en/actions/using-github-hosted-runners/about-github-hosted-runners#supported-runners-and-hardware-resources

The example test2d is looking slightly better now.
Please note that the present PR could use the one looking at the debug level : https://github.com/xcompact3d/2decomp-fft/pull/35/files

The above commit - Add a "memory" to Makefile execution... creates a file Makefile.settings with your FFLAGS, etc. and includes this on subsequent runs - anything you specify will overwrite this. @mathrack can you check this does what you wanted?

After a quick test it seems to work as expected for a debug build GNU + MPICH, thank you @pbartholomew08

I have a feeling there will be ways to break it so raise any issues you experience. I forgot to say, make clean will also remove the settings file

Halo test now runs in memory and calls MPI_Abort with non-zero error code on failure

I suspect halo subroutines are broken. For instance here :
https://github.com/mathrack/2decomp-fft/blob/fix_examples/src/halo_common.f90#L32
If nproc = 1, the data can be in y-pencil or z-pencil, the test will claim it is using x-pencil. More generally, if the data is in y-pencil and p_row = 1, the test will claim it is using x-pencil.

that's how that code has always been, I'm not sure how we could fix it without breaking use cases

However we can at the very least update the test to try and check whether it does break in those circumstances

I think the update_halo subroutine needs one more argument to clarify the pencil orientation of the given array. On this ground, we can add some check on p_row and p_col to make sure the indices are not out of bound.
To fix this, we need feedback from people using this halo subroutine.

This PR will be closed. Work will resume in the branch https://github.com/xcompact3d/2decomp-fft/tree/fix_examples after https://github.com/xcompact3d/2decomp-fft/pull/42 is merged.


SPI: Implement interrupt-driven transfer_in_place_async

This PR now also demonstrates a way to hide driver-private State, without necessarily hiding peripheral Info. Further (out of scope) work is necessary to actually fully describe SPI peripherals (i.e. include all signals), but at that point their Instance traits and Info structs should be considered complete for existing hardware.

If a common Instance trait is used for multiple peripherals, those traits should not have any logic implemented in them.

But it was already the case before this PR so I'm not going to push it.

What are you referring to, what trait has logic in it? The only trait in the SPI master driver that is even intended to ever become stable is PeripheralInstance, and even that will see changes. Driver is not a trait, it is the low-level(ish) implementation, and it is also private.

If a common Instance trait is used for multiple peripherals, those traits should not have any logic implemented in them.

But it was already the case before this PR so I'm not going to push it.

What are you referring to, what trait has logic in it? The only trait in the SPI master driver that is even intended to ever become stable is PeripheralInstance, and even that will see changes. Driver is not a trait, it is the low-level(ish) implementation, and it is also private.

Nevermind, perhaps I misunderstood the spirit of that API guideline.


Error in "stripepriceid" field.

As soon as the lines
stripePriceID:
  extends: fields/stripe-price-id

are used in a blueprint, the page in the admin panel takes up to ten second to load and returns this error :
Operation timed out after 10000 milliseconds with 0 bytes received in file: path/to/site/kirby/src/Http/Remote.php line: 251

Maybe because you don't have any priceId set up on Stripe yet ?

actually in /sck/api/products, id and default_price seems to properly fetch the data...
Is this related to something else?


josh: cherry-pick upstream patch to fix build with rust 1.73

Description of changes
Our josh builds broke on staging with the new rustc 1.73:
https://hydra.nixos.org/build/239619692/log/tail
This commit cherry-picks the fix (already merged upstream)
https://github.com/josh-project/josh/pull/1290

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

https://github.com/NixOS/nixpkgs/issues/265948


bf8 eltwise kernels

I have added two versions of conversion routines.

Efficient : F32 -> F16 -> BF8
Generic : F32-> BF8 (this could be extended to include weird exponent offset and other corner case handling)

some minor question in the driver.
Can you please also adjust the test scripts for AVX512?
Will you submit also ReLU, etc. in the PR?

All done.
Added stochastic rounding support.


MeshCollider raycast bug

// in TriangleShape::raycast
const decimal sign = (localHitPoint - ray.point1).dot(pq) > FLT_MIN ? 1.0f : -1.0f;
// hitFraction must multiply sign here, sometimes the point is not on the ray .
const decimal hitFraction = sign * (localHitPoint - ray.point1).length() / pq.length();
if (hitFraction < decimal(0.0) || hitFraction > ray.maxFraction) return false;

Can I please ask you to explain the issue with text? Where does this code coming from exactly? What do you expect and what is wrong?

I am using reactphysics3d for camera collider in my project, by moving away the camera from the meshcollider that obstruct the view. But sometimes  the raycast not work well, here is my test data.
ray : {
point1 = (x = -11.6768284, y = 7.62946081, z = -6.41031027)
point2 = (x = -6.94290876, y = 7.38909339, z = -5.27299595)
maxFraction = 1
},
mesh triangle : {
[0] = (x = -11.5843544, y = 10.4978685, z = -7.63218307)
[1] = (x = -12.3325634, y = 5.11047888, z = -6.35258769)
[2] = (x = -12.3325634, y = 10.4978685, z = -6.35258769)
},
the raycast result point is: {
(x = -12.2221985, y = 7.65715217, z = -6.54133463)
},
but the result is not on the ray, this cause my bug, it's on the positive of  the ray, you can try my test data.
That code is copy from TriangleShape.cpp in reactphysics3d source, I add ‘const decimal sign = (localHitPoint - ray.point1).dot(pq) > FLT_MIN ? 1.0f : -1.0f;
’ fixed the bug.
Here is a video to explain my problem:
https://user-images.githubusercontent.com/3447313/218084213-a08a5814-c16d-4433-9d71-0fedec87b493.MP4


Support installations from npm 3.x

added a path cascade to try to find phantomcss and casperjs in likely locations for npm 2x or 3x installs.
updated phantomcss to 0.11.1 (which includes fix for same problem with resolvejs)
tests pass with npm 3x, and works in a "live" npm 3x installation.

This should fix https://github.com/micahgodbolt/grunt-phantomcss/issues/6

Tried this with node .10 and current. Appears to do the trick. Still wondering if there is a more elegant way to do all of this, but working is better than broken!

I agree there should be a better approach, but I ended up going with what the PhantomCSS project did with resolveJS.  When in Rome....

Turns out you can use require.resolve() to retrieve the path of any node module you have installed.
Also turns out CasperJS is NOT a node module, it is only installed via NPM, and therefore you can't use require.resolve() on it :-1:


Investigate building a JavaScript library

Not yet sure what a good project structure for this can be - ideally someone javascript-affine has a look here - I would love the reuse of code - but I am not really a javascript kinda guy..

Relevant talk: https://www.youtube.com/watch?v=ZbANCuZ_qqw KotlinConf 2017 - What's New & Cool in Kotlin Tools by Dmitry Jemerov

Things just got more relevant: https://blog.jetbrains.com/kotlin/2017/11/kotlin-1-2-released/

Hi, I m interested in your issue opening.
Please share me more details about your problem.
Thank you.


⚠ Bofrost.de has degraded performance

In 29eee82, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1222 ms

Resolved: Bofrost.de performance has improved in c53497c.


Use @inlinable and remove redundant helper function

This is a tiny PR that changes two things:

Use Dictionary(grouping:by:) instead of the custom implementation that was used previously
Set @inlinable instead of @usableFromInline on internal helper functions, that are small enough to benefit from inlining. This allows for better compiler optimisations, since the content of the inlined helper function can be inlined itself, instead of it just being able to be called from within an inlined function.

@nkristek somehow the Unit Tests aren't running, potentially due to the force push. I also don't have the option to re-run on my end. Could you push something to retrigger the checks?

@Marcocanc could you please retry running the workflow? I've pushed two small commits that improve the description of some error cases.

I've just noticed, that in the example project the snapshot tests were broken. They're fixed as well now.

Running now. I also changed the settings so you don't need approval for every run.


Invalid URI characters in password or username are not reencoded

https://github.com/slimphp/Slim-Psr7/blob/5dbd98f5a0194db8aeee4da3441e56a1d816360b/src/Factory/UriFactory.php#L69
createFromGlobals takes the PHP_AUTH_USER,PHP_AUTH_PWD as-is without rencoding them.
]This causes to Uri::__tostring to ouput an invalid RFC3986 string
Uri->withUserInfo rencodeds the given values and should be used instead of passing them directly to the constructor.

If PHP_AUTH_PWD contains a ?, $, # (ie: 'abc?def')
This causes Uri->tostring() to generate URLs that can't be parsed by parse_url because they are not encoded.
Constructor has a different mecanism than using withUserInfo

@lletourn good catch, you're more than welcome to raise a PR so we can fix it!

Is the 3.X branch still maintained? The bug is there too so I'd need to have 2 PRs one for 4.x and one for 3.x
Thanks

@lletourn the 3.x branch is in security mode only at the moment. It's a real pain to do releases but I might do an exception for this as I'm sure you're not the only one affected.


Change docs link to use HTTPS instead of HTTP

ack https://github.com/coala-analyzer/website/pull/42/commits/40c1b00063d81e82a6cb1dba4ef9b9e1fd3d9f53

@rultor merge

@rultor merge

@sils1297 OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@sils1297 @underyx Oops, I failed. You can see the full log here (spent 11s)
+ set -e
+ set -o pipefail
++ dirname ./run.sh
+ cd .
+ echo 1321
+ echo '2.0-SNAPSHOT BUILD'
2.0-SNAPSHOT BUILD
+ date
Fri Jun  3 00:00:21 UTC 2016
+ uptime
 00:00:21 up 13 days, 19:23,  0 users,  load average: 13.15, 9.92, 9.50
+ fork=git@github.com:underyx/website.git
+ fork_branch=patch-1
+ head=git@github.com:coala-analyzer/website.git
+ head_branch=gh-pages
+ pull_id=42
+ pull_title='Change docs link to use HTTPS instead of HTTP'
+ vars=('--env=fork=git@github.com:underyx/website.git' '--env=fork_branch=patch-1' '--env=head=git@github.com:coala-analyzer/website.git' '--env=head_branch=gh-pages' '--env=pull_id=42' '--env=pull_title=Change docs link to use HTTPS instead of HTTP')
+ image=yegor256/rultor
+ directory=
+ scripts=()
+ container=coala-analyzer_website_42
+ as_root=false
+ mkdir -p /home/rultor/.ssh
+ echo -e 'Host github.com\n\tStrictHostKeyChecking no\n'
+ chmod 600 /home/rultor/.ssh/config
+ git clone git@github.com:coala-analyzer/website.git repo
Cloning into 'repo'...
+ cd repo
+ git config user.email<EMAIL_ADDRESS>+ git config user.name rultor
+ '[' -z '' ']'
+ '[' -e pom.xml ']'
+ '[' -e Gemfile ']'
+ '[' -e build.xml ']'
+ [[ -e build.sbt ]]
+ [[ -e project/Build.scala ]]
+ echo 'I can'\''t guess your build automation tool, see http://doc.rultor.com/basics.html'
I can't guess your build automation tool, see http://doc.rultor.com/basics.html
+ exit -1
'cid' file is absent, container wasn't started correctly


Allow colons in config values

The CI failure is because we pinned to an older version of pytest for compatibility with python 3.8 betas.  Now 3.8 is out officially, so I think we can remove that pin now (which broke because pytest-asyncio recently upgraded to an API not available in our chosen pytest version).
I think you can just remove the version requirement on pytest in test-requirements.txt and repush.  If the tests pass then it's good to go 🤞


should merging of #183 necessary?

Right, according to pep8, we need #<space> instead of #. But this kind of behavior can be implemented by let g:NERDSpaceDelims = 1. Actually, isn't it necessary to add a space between the comment delimiter of any language to make it clean? The requirement of pep8 should not make python a special case. Or at least the committer should take into account of g:NERDSpaceDelims. Someone argues that #183 makes it simpler for python commenting from the start. But before the merging I just need to config
let g:NERDSpaceDelims = 1

now I have to config
let g:NERDSpaceDelims = 1
let g:NERDCustomDelimiters = {'python': {'left': '#'}}

besides two hours wasted on locating the reason why <Leader>cc only cause comments in python two spaces instead of one.

Agree.
Now I have to config for python specifically. It is annoying, really.

Is making NERDSpaceDelims=1 the default a potential option? That would be pep8-compliant by default and avoid making python a special case. And as pointed out above:

Actually, isn't it necessary to add a space between the comment delimiter of any language to make it clean?

On a somewhat tangential note, the current fix doesn't seem to even address pep8 block requirements:

Block comments generally apply to some (or all) code that follows them, and are indented to the same level as that code. Each line of a block comment starts with a # and a single space (unless it is indented text inside the comment).

Emphasis mine. The current implementation yields:
# some_block = {
    # 'a': 1,
    # 'b': 2}

in which pep8/pycodestyle tool correctly identifies "unexpected indentation", rather than:
# some_block = {
#     'a': 1,
#     'b': 2}

I think you can meet Pep8 requirements with:
let NERDDefaultAlign="left"

Put it inside .vim/after/ftplugin/python.vim to avoid changing the setting for other filetypes.


changed the social section

before

after

@Spyware007

Hi, there seems to be a merge conflict, please resolve it.

Hi, are you still working on this?

a different pull request was merged , issue #279 has been closed


Option to use Alternative Titles as Badge Titles

As I don't really want to use the table layout but still need the context on which badge is which, using the already present alt attribute values (e.g. Statements, Branches, etc.) as titles for the badges (instead of Coverage for every badge) would be neat. 🐢

Hello, @CREEATION thanks for the issue! We will definitely look into it and I think you are right. This should come as default, indeed. We may also create a way of making it fully customizable as well.

@CREEATION is this what you mean? Disregard it being in a table though, see the badges naming.

Hey, that was quick!
By "alt" I meant the literal "alt" attribute from the markdown.
e.g. here it's "Functions" (alt), not "functions" (hashKey):
![Functions](#functions#)
   ^ this | not this ^

This way, I'd imagine, it's customizable very easily by just changing the markdown.

Oh, I think the library is not ready to work as you suggested :( what do you think of these proposed changes? Do they work for you? https://github.com/olavoparno/istanbul-badges-readme/pull/34/files
We decided to postpone a customization where one would be able to change each hashKey as desired.

Sure, that's already looking good for me in contrast to all badges being "Coverage".
Is it possible to add a CLI argument so people who want the previous behaviour don't get angry because of me opening this issue? 😄
Thanks for the quick responses! 🐢

Yeah, it passed through our minds this idea of istanbul-badges-readme --functions="funciones" --statements="declaraciones" of sorts but you may be right. Are you in a hurry? We will come back to you as soon as possible. Thank you again for the issue and the prompt response.

Nah, it's not urgent at all.


Use app.join() instead of waiting for signal

What is the reason for this PR?

Less code and nicer code.

Hm, there was a reason to use signal handlers to control the termination of the process and being able to enter the console. Have you tested, that this is still working as before?

Yes, I tested that. Will double check now.

Consider to write a test for this
http://click.pocoo.org/5/api/#testing

Double checked. Let me know if you can see other side effects.

Hey @ms83,
I tried to run from your branch. It seems you forgot some commits? Here is what I get:
% pyethapp --data-dir /tmp/pea run
...
Traceback (most recent call last):
  File "/tmp/newenv/bin/pyethapp", line 9, in <module>
    load_entry_point('pyethapp', 'console_scripts', 'pyethapp')()
  File "/tmp/newenv/local/lib/python2.7/site-packages/click-6.2-py2.7.egg/click/core.py", line 716, in __call__
    return self.main(*args, **kwargs)
  File "/tmp/newenv/local/lib/python2.7/site-packages/click-6.2-py2.7.egg/click/core.py", line 696, in main
    rv = self.invoke(ctx)
  File "/tmp/newenv/local/lib/python2.7/site-packages/click-6.2-py2.7.egg/click/core.py", line 1060, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/tmp/newenv/local/lib/python2.7/site-packages/click-6.2-py2.7.egg/click/core.py", line 889, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/tmp/newenv/local/lib/python2.7/site-packages/click-6.2-py2.7.egg/click/core.py", line 534, in invoke
    return callback(*args, **kwargs)
  File "/tmp/newenv/local/lib/python2.7/site-packages/click-6.2-py2.7.egg/click/decorators.py", line 17, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/tmp/pyethapp/pyethapp/app.py", line 215, in run
    app.join()
AttributeError: 'EthApp' object has no attribute 'join'

@konradkonrad  Are you in sync with pydevp2p develop branch? EthApp inherits from BaseApp which has join method.
https://github.com/ethereum/pydevp2p/blob/develop/devp2p/app.py#L43

alright, thanks for the hint :)

@heikoheiko not sure, how you want to test this with a click test-runner? pyethapp isn't exactly a one-off app, whose exit code you could test.
Consider this:
from click.testing import CliRunner
from pyethapp.app import app

def test_run():
  runner = CliRunner()
  with runner.isolated_filesystem():
    result = runner.invoke(app, ["run"])
    assert result.exit_code == 0

It will never return...

we want to test the full application. especially config and cli params handling.
monkeypatching it so, that it will exit after a certain condition should be sufficient to let it exit.


README looks bad on mobile devices

It's nothing urgent, but something to consider fixing.

I don't care enough, closing.


fceux install failed

==> Downloading https://downloads.sourceforge.net/project/fceultra/Source Code/2.2.2 src/fceux-2.2.2.src.tar.gz
Already downloaded: /Library/Caches/Homebrew/fceux-2.2.2.tar.gz
==> Patching
patching file SConstruct
patching file SConstruct
patching file src/drivers/sdl/SConscript
patching file src/drivers/sdl/gui.cpp
patching file src/drivers/sdl/sdl-video.cpp
==> /usr/local/opt/scons/bin/scons GTK=0 GTK3=1
#include <gd.h>
^
1 error generated.
scons: *** [src/drivers/videolog/nesvideos-piece.o] Error 1
scons: building terminated because of errors.
READ THIS: https://git.io/brew-troubleshooting
If reporting this issue please do so at (not Homebrew/homebrew):
https://github.com/homebrew/homebrew-games/issues

Could you post brew gist-logs fceux, please?

Closing due to lack of response, but please reopen if you're still having issues. If you are, please include the logs!


general text channels for every supported language (german & french until now)

Proposal:
Proposal written by DanNick#8108
What is the title of your proposal?: general text channels for every supported language (german & french until now)
Is it a suggestion for the website or for the bot?: server
Explain your proposal more accurately (It's best to give as much information as possible, so that we can implement the proposal better): Both German and French should have general text channels to discuss, have small-talk etc. again. The German text channel was very popular until it got deleted and now users are starting to talk off-topic in <#441580549564530699>. General text channels for different languages would resurrect the Discord server since many users on this server probably can't speak English and are inactive due to that.
Why we should add this feature?: See "Explain your proposal more accurately"
ReportID: 55

➤ LenoxBot commented:
Approve:
Would be nice but there were those channels and they got removed
KeinEmxl#3887

➤ LenoxBot commented:
Approve:
good idea!
ƨʌvʌʛєƈɾєєƥєɾ28™✔#0662

➤ LenoxBot commented:
Deny:
We removed the channels with a reason, so it isn't a good suggestion
Nickkel#6515


Allow passing arguments through to 'start'

The entrypoint needs to pass any arguments passed to it
through to the specified 'start' file. We do this with some
more argparse setup.
We must exec the entrypoint process rather than spawn them with
subprocess. Otherewise we will end up with zombie processes.
ENTRYPOINT directive should use the 'array' syntax rather than the
'string' syntax, to prevent an extra invocation of shell
args = argparser.parse_args()

Thanks @yuvipanda !


Kotlin IO

Added examples for FileTreeWalk and various File extension functions

Looks good to me, thanks  Johanne! @JohanneA


[minor BUG] good night in the day

Good Night, XXX.
    It's 12:36 PM on a beautiful Tuesday!

It's not night... It's probably one of the libraries doing this. It's up to you to report it upstream. I'm assuming this is a simple bug.

What was the actual time and date for you then?

https://github.com/mov-cli/mov-cli/commit/4b172828122fb48f9e98e26f3401fd76705a21e5

Yeah that was correct.

Wait what was the actual time for you? (24 hour format)


bug: ion-input has ion-invalid and ng-valid classes after clearing errors

Bug Report
I use reactive form in Angular and custom form validation function passed to FormBuilder.group function. In the validator I clear errors using AbstractControl.setErrors function. The problem is that the ion-invalid class is not removed from the control if it is not edited control.
In my case I have two inputs

password
repeated password
The ion-invalid class does not disappear from repeated password even if values are the same when I edit "password".

Here is a screenshot from element inspector:
https://pasteboard.co/J0Ncc1M.png
where you can see ion-invalid and ng-valid next to each other.
The form validation status is correct thou.
Ionic version:

[x] 5.0.4
Current behavior:

Expected behavior:

Paired classes:
ion-invalid  and ng-invalid
ion-valid and ng-valid
Steps to reproduce:

Related code:

insert short code snippets here

Other information:

Ionic info:

insert the output from ionic info here

Thanks for the issue. I am going to close this as a duplicate of https://github.com/ionic-team/ionic/issues/20650.


Added Json::Verification to capture verification fields and make them…

… available to parsers

:+1:

Merged #26


[JS] Topic: JavaScript Static Methods

Directory
Topics/Static_Methods

Description
I want to add topic JavaScript Static Methods
Domain
JavaScript
Type of Contribution

[ ] Documentation
[ ] Video
[ ] Audio

@paramsiddharth pls assign this issue to me. I would like to work on this issue.

Is this issue still open? I'd like to help!

I would like to work upon this

@paramsiddharth

@daminimehra @7sakshi7 @joshi-kaushal Please read the modified issue description. Good luck!

/assign  please assign this to me


Make the netcdf-c library compile with gcc -ansi.

Primary fixes to get -ansi to work.

Convert all '//' C++ style comments to /.../ or to use #if 0...#endif
It turns out that when -ansi is specified, then a number of
functions no longer are defined in the header -- but they are still
in the .so file.
The big example is strdup(). So, added code to include/ncconfig.h to define
externs for those missing functions that occur in more than one place.
These are enabled if !_WIN32 && STDC == 1 (STDC is supposed to
be the equivalent compile time flag to -ansi). Note that this requires
config.h (which references ncconfig.h) to be included in files where it is
currently not included. Single uses will be only in the file that uses them.
Added mmap test for the MAP_ANONYMOUS flag to configure.ac. Apparently
this is not always defined with -ansi.
fix some large integer constants in nc_test4/tst_atts3.c and nc_test4/tst_filterparser.c
to avoid compiler complaints.
fix a double constant in nc_test4/tst_filterparser.c to avoid compiler complaints.

[Note I suspect numbers 4 and 5 will be a problem on big-endian machines, but we have no way to test]
Misc. Changes:

convert more instances of _MSC_VER to _WIN32.
added some debugging code to include/nctestserver.h
added comment about libdispatch/drc.c always being compiled.
modify parser generation in ncgen to remove unneeded files.

Beware: this pr causes failures under cygwin. It looks like there is some sort of
difference in the handling of floating point values when -ansi is specified.
One difference is the NC_ERANGE is not be returned. Another is that floating point
comparisons fail when they should succeed. No idea what is the root cause.

I would suggest that compiling on cygwin is far more important than compiling with -ansi.

Agreed. It is not a show-stopper. I can do the check for -ansi compliance under my
ubuntu VM. I am not sure in practice how/why anyone would be using -ansi
under cygwin anyway.

Correct. ok without -ansi, test failures with -ansi.

These fixes break compilation on OSX; removing from #1249 for the moment and will revisit with details.

As I was the one suggesting to use -ansi I'll had a look at the PR.
I think this PR goes definitly in the right direction but I think the mass of changes makes it difficult to review. How about making a PR with only the comment change from // -> /* */? This should make make the diff much smaller.
Regarding the missing strdup declaration, this function is only defined in POSIX and not in C itself, see http://man7.org/linux/man-pages/man3/strdup.3.html.

BTW agree about doing this in smaller chunks; my bad. I got carried
away in doing all the ansi fixes in one fell swoop.

See pr https://github.com/Unidata/netcdf-c/pull/1252

Still seeing the following on OSX, will try to debug this.
In file included from /Users/wfisher/Desktop/gitprojects/netcdf-c/build/config.h:596:
/Users/wfisher/Desktop/gitprojects/netcdf-c/include/ncconfigure.h:45:15: error: expected parameter declarator
extern size_t strlcat(char*,const char*,size_t);
              ^
/usr/include/secure/_string.h:115:47: note: expanded from macro 'strlcat'
                __builtin___strlcat_chk (dest, __VA_ARGS__, __darwin_obsz (dest))
                                                            ^
/usr/include/secure/_common.h:39:62: note: expanded from macro '__darwin_obsz'
#define __darwin_obsz(object) __builtin_object_size (object, _USE_FORTIFY_LEVEL > 1 ? 1 : 0)
                                                             ^
/usr/include/secure/_common.h:30:32: note: expanded from macro '_USE_FORTIFY_LEVEL'
#    define _USE_FORTIFY_LEVEL 2

Let me propose a wild idea:
In nccconfigure.h, change:
extern size_t strlcat(char*,const char*,size_t);
to
extern size_t strlcat(chardst,const charsrc,size_t count);
That is, add actual parameter names.


Fix ArcGIS and WMS sandcastle examples

A fix to allow these examples to load over https...
https://cesiumjs.org/Cesium/Apps/Sandcastle/index.html?src=ArcGIS MapServer.html&label=All
https://cesiumjs.org/Cesium/Apps/Sandcastle/index.html?src=Web Map Service (WMS).html&label=All

Thanks @chris-cooper!


Yanking and Pasting into *.ju.py file creates duplicated lines

Describe the bug
When pasting text that's in your clipboard/register(s)/etc, the notebook open in the browser seems to post a number of duplicate lines which then causes a de-sync between the file open in neovim and the text in the notebook. For example, I yanked the text "pd.read_sql(sql, CONNECTION_STRING)" from another neovim buffer I had open (a normal .py file), and then pasted it in my .ju.py file, and then the notebook open in the firefox window shows a number of duplicates, or to be exact:
pd.read_sql(sql, CONNECTION_STRING)
pd.read_sql(sql, CONNECTION_STRING)
pd.read_sql(sql, CONNECTION_STRING)
pd.read_sql(sql, CONNECTION_STRING)
pd.read_sql(sql, CONNECTION_STRING)

This issue also pops up when you have something like autopairs that completes closing quotes (e.g.
typing """ in python and it automatically generates the closing """. It seems to always result in 4 additional copies, as far as I can tell.
To Reproduce
Steps to reproduce the behavior:

Start up jupynium as described in readme
Have another buffer/tab/etc open containing other code
Yank some text/code
Paste it in your .ju.py file
Results in duplicate lines in notebook

Expected behavior
Expected behavior is to have pasted text in neovim buffer to result in same exact text in notebook
Output when using jupynium command
If you launch Jupynium server using the command line program, it gives you more helpful logs.

N/A (using neovim plugin rather than command line tool)

Output of jupynium --version
N/A (using neovim plugin rather than command line tool)

Output of nvim --version
NVIM v0.9.1
Build type: Release
LuaJIT 2.1.0-beta3

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/usr/local/share/nvim"

Run :checkhealth for more info

Additional context
N/A

FWIW I'm also seeing the same issue.

I never had this problem yet but I'll need to confirm this.
I think this should be the neovim problem, because I'm updating using their buffer update event APIs. Maybe neovim is giving updates in a wrong way..

Indeed, there seems to be multiple events generated from neovim:
jupynium.events_control:  197 - INFO - Event from nvim: ['notification', 'on_lines', [1, ['print("hello, world")'], 7, 7, 8]]
jupynium.events_control:  197 - INFO - Event from nvim: ['notification', 'on_lines', [1, ['print("hello, world")'], 7, 7, 8]]
jupynium.buffer:  443 - INFO - Converting to code cells: [2]
jupynium.events_control:  197 - INFO - Event from nvim: ['notification', 'on_lines', [1, ['print("hello, world")'], 7, 7, 8]]
jupynium.buffer:  443 - INFO - Converting to code cells: [2]
jupynium.events_control:  197 - INFO - Event from nvim: ['notification', 'CursorMoved', [1, 7, 7]]
jupynium.buffer:  443 - INFO - Converting to code cells: [2]

The above log was generated by yanking and pasting the line print("hello, world") once.

Hmm, after updating my neovim plugins (with lazy) I no longer see this issue. Will test a bit more tomorrow and report back if it resurfaces!

Nevermind, the issue is still there, but I made a discovery:
The first time the file is opened with JupyniumStartSync, the issue is gone.
However, after closing the notebook and re-opening by calling JupyniumStartSync, pasting produces two copies of the line (and two events).
If I close the notebook again, and re-open, pasting produces three copies of the line (and three events).
My guess is that there is some event handler that is not cleaned up properly after a notebook is closed.

The issue happens regardless of how the notebook is closed, e.g., by closing the browser tab or calling JupyniumStopSync.

@joh Okay, thank you. Now that's more clear. I think that it's not closing the event updates and forcing to receive another events when you sync again.
In this case it's not the neovim bug. If this is the case, however, you should have two copies for all letters you type not only the copy pasting, though.


Come discuss with us (from PrismarineJS team)

Hi,
At PrismarineJS we have followed your work for a while, and this is a topic that interest some of our member (including me).
It would be awesome if you could join us on our discord https://discord.gg/GsEFRM8 to discuss about AI in minecraft.
Karang, on behalf of the PrismarineJS team

Thank you for the invitation @Karang. I really appreciate it 💖


Creating empty commit for release purposes

See https://github.com/pypa/packaging-problems/issues/74

This is a courtesy to demonstrate exactly which commit the specific pypi package we will distribute is running against. It will make it easier to recreate this pypi package, if that must be done in the future.


Support type int32_t and int64_t for Split op

We have a customer speech model, and when we convert it, we get a split op where the input is int type. We need onnxruntime support this.

Thank you! do we need int32 or int64 or both please? Meanwhile, I'd also understand when "double" is used. The reason is that we do want to control our binary size now.

@linkerzhang currently when we convert speech synthesis model, the python type is int and it is mapped to C int32_t.  It would be better for us to support both int32_t and int64_t. Otherwise we may revisit this if int64_t is needed.

rethink this, let's support it on demand because of controlling binary size. Let's just support int32_t for this PR.

Will use https://github.com/Microsoft/onnxruntime/pull/563