Cannot get infromation of Minecraft server on same host

When I try to get the information of my locally running minecraft server on my linux vps, I get an error (doesn't matter if link URL or localhost). But if I try to remotely get the minecraft server information (with my Windows VPS) it works like a charm.
And yes, I use the standard port (but have another running on a custom one, I can't get information of either of those when running the programm on the same server).

Not sure how to reproduce this without more information. Please re-open if this is still an issue.


merge CONTEXT into RUNTIME_CONTEXT for reduce ThreadLocal lookup to  improve performance

two ThreadLocal means twice lookup
Please answer these questions before submitting a pull request

Why submit this pull request?

[ ] Bug fix

[ ] New feature provided

[*] Improve performance

Related issues

Bug fix

Bug description.

How to fix?

New feature or improvement

Describe the details and related test reports.

Also, this merge will confuse other language agent maintainers. They don't implement all of those like JAVA.

Those two are different from the concept perspective. This change could improve the performance a little, yes. But currently, there is no concern about this. We should not change that, tp make codes more complex.

This PR just refactor the code
这种代码重构，没有功能性改变，仅内部实现上的调整，对使用者来说是透明的，没有增加复杂度一说吧:)
不过每个人的考虑角度有所不同吧，有需要的可自己git cherry-pick此PR后自己构建的agent:)


added polish translation

I've added polish translations.
There were few problems I had with this.

First of all in polish there is no single case for plural months and weeks.
For example for months:
"2,3,4 months ago" is "2,3,4 miesiące temu"
"5 months ago" is "5 miesięcy temu" (this is used for all other cases)

I've made it as "%d miesiące/-y temu" to handle all cases with this single variable.

Second of all I'm not sure why for colloquial_p_yy there is an exception comparing to all other cases like colloquial_p_mm and there is a year passed to this variable, instead of amount of years that passed. I would expect this to be:

colloquial_p_yy = "%d years ago"
Same question for future years.

Hi,
For first question we should move the codebase to stringsdict in order to support these variants; I'll start working on it before 5.x.
Second question: replacing %d years ago with the specific year's value is used because in colloquial format you don't say "5 years ago" but simply "2012". It's just a choice for clarity and nothing more.
Can I merge your PR into the next release? I don't understand how you have solved the first point.
Thanks for your support.

Sure you can merge it.
The first points was solved, by using kind of written "or" statement.
It's used in many official papers all around in our language, that it says:
"X miesiące/-y", which can become "2 miesiące temu" or "5 miesięcy temu", so "e/-y" means "e or y".
As I said it's fully legit to go this way as it is how they usually do that in government forms etc...


refactor!: remove location path utility methods

This PR was generated using Autosynth. :rainbow:
Log from Synthtool
synthtool > Executing /tmpfs/src/git/autosynth/working_repo/synth.py.
synthtool > Ensuring dependencies.
synthtool > Cloning googleapis.
synthtool > Pulling Docker image: gapic-generator-typescript:latest
latest: Pulling from gapic-images/gapic-generator-typescript
Digest: sha256:a7f89dbac851b1daf372db435f555fcbd77551a66bfb6ea3a21457e616be156d
Status: Image is up to date for gcr.io/gapic-images/gapic-generator-typescript:latest
synthtool > Generating code for: google/cloud/redis/v1beta1.
synthtool > Generated code into /tmpfs/tmp/tmpf9mpld22.
synthtool > Pulling Docker image: gapic-generator-typescript:latest
latest: Pulling from gapic-images/gapic-generator-typescript
Digest: sha256:a7f89dbac851b1daf372db435f555fcbd77551a66bfb6ea3a21457e616be156d
Status: Image is up to date for gcr.io/gapic-images/gapic-generator-typescript:latest
synthtool > Generating code for: google/cloud/redis/v1.
synthtool > Generated code into /tmpfs/tmp/tmpq5vtnvjp.
.eslintignore
.eslintrc.yml
.github/ISSUE_TEMPLATE/bug_report.md
.github/ISSUE_TEMPLATE/feature_request.md
.github/ISSUE_TEMPLATE/support_request.md
.github/PULL_REQUEST_TEMPLATE.md
.github/release-please.yml
.jsdoc.js
.kokoro/common.cfg
.kokoro/continuous/node10/common.cfg
.kokoro/continuous/node10/docs.cfg
.kokoro/continuous/node10/lint.cfg
.kokoro/continuous/node10/samples-test.cfg
.kokoro/continuous/node10/system-test.cfg
.kokoro/continuous/node10/test.cfg
.kokoro/continuous/node12/common.cfg
.kokoro/continuous/node12/test.cfg
.kokoro/continuous/node8/common.cfg
.kokoro/continuous/node8/test.cfg
.kokoro/docs.sh
.kokoro/lint.sh
.kokoro/presubmit/node10/common.cfg
.kokoro/presubmit/node10/docs.cfg
.kokoro/presubmit/node10/lint.cfg
.kokoro/presubmit/node10/samples-test.cfg
.kokoro/presubmit/node10/system-test.cfg
.kokoro/presubmit/node10/test.cfg
.kokoro/presubmit/node12/common.cfg
.kokoro/presubmit/node12/test.cfg
.kokoro/presubmit/node8/common.cfg
.kokoro/presubmit/node8/test.cfg
.kokoro/presubmit/windows/common.cfg
.kokoro/presubmit/windows/test.cfg
.kokoro/publish.sh
.kokoro/release/docs.cfg
.kokoro/release/docs.sh
.kokoro/release/publish.cfg
.kokoro/samples-test.sh
.kokoro/system-test.sh
.kokoro/test.bat
.kokoro/test.sh
.kokoro/trampoline.sh
.nycrc
.prettierignore
.prettierrc
CODE_OF_CONDUCT.md
CONTRIBUTING.md
LICENSE
README.md
codecov.yaml
renovate.json
samples/README.md
synthtool > Replaced '/compute/docs/' in src/v1beta1/cloud_redis_client.ts.
synthtool > Replaced '/compute/docs/' in src/v1/cloud_redis_client.ts.
npm WARN npm npm does not support Node.js v12.13.1
npm WARN npm You should probably upgrade to a newer version of node as we
npm WARN npm can't make any promises that npm will work with this version.
npm WARN npm Supported releases of Node.js are the latest release of 6, 8, 9, 10, 11.
npm WARN npm You can find the latest version at https://nodejs.org/
npm WARN deprecated<EMAIL_ADDRESS>core-js@<3 is no longer maintained and not recommended for usage due to the number of issues. Please, upgrade your dependencies to the actual version of core-js@3.
npm WARN deprecated<EMAIL_ADDRESS>One of your dependencies needs to upgrade to fsevents v2: 1) Proper nodejs v10+ support 2) No more fetching binaries from AWS, smaller package size

><EMAIL_ADDRESS>postinstall /tmpfs/src/git/autosynth/working_repo/node_modules/core-js
> node -e "try{require('./postinstall')}catch(e){}"

><EMAIL_ADDRESS>postinstall /tmpfs/src/git/autosynth/working_repo/node_modules/protobufjs
> node scripts/postinstall

><EMAIL_ADDRESS>prepare /tmpfs/src/git/autosynth/working_repo
> npm run compile

npm WARN npm npm does not support Node.js v12.13.1
npm WARN npm You should probably upgrade to a newer version of node as we
npm WARN npm can't make any promises that npm will work with this version.
npm WARN npm Supported releases of Node.js are the latest release of 6, 8, 9, 10, 11.
npm WARN npm You can find the latest version at https://nodejs.org/

><EMAIL_ADDRESS>compile /tmpfs/src/git/autosynth/working_repo
> tsc -p . && cp -r protos build/

npm notice created a lockfile as package-lock.json. You should commit this file.
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})

added 972 packages from 1018 contributors and audited 7161 packages in 23.046s
found 0 vulnerabilities

npm WARN npm npm does not support Node.js v12.13.1
npm WARN npm You should probably upgrade to a newer version of node as we
npm WARN npm can't make any promises that npm will work with this version.
npm WARN npm Supported releases of Node.js are the latest release of 6, 8, 9, 10, 11.
npm WARN npm You can find the latest version at https://nodejs.org/

><EMAIL_ADDRESS>fix /tmpfs/src/git/autosynth/working_repo
> gts fix && eslint --fix samples/*.js samples/**/*.js

/tmpfs/src/git/autosynth/working_repo/samples/quickstart.js
  24:38  error  "@google-cloud/redis" is not found  node/no-missing-require

✖ 1 problem (1 error, 0 warnings)

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>fix: `gts fix && eslint --fix samples/*.js samples/**/*.js`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>fix script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/kbuilder/.npm/_logs/2019-12-10T12_21_48_511Z-debug.log
installing<EMAIL_ADDRESS>installing<EMAIL_ADDRESS>installing<EMAIL_ADDRESS>synthtool > Cleaned up 2 temporary directories.
synthtool > Wrote metadata to synth.metadata.

Same as https://github.com/googleapis/nodejs-translate/pull/402. Please do not merge until the logic is fixed in the generator. Cc: @xiaozhenliu-gg5

Closing this, the next publish should be fine.


image.load  loads an image into a torch.Tensor  Error!!!

th train-SRResNet.lua -model_name 9x9-15res-LR24
{
iter_end : 10000000
checkpoint_start_from : ""
beta : 0.9
model_name : "9x9-15res-LR24"
arch : ""
checkpoint_save_iter : 10000
lr : 0.001
}
ResNet-34 ImageNet
ImageNet loaded, # of imgs:0
+++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
image.load
loads an image into a torch.Tensor

usage:
image.load(
string                              -- path to file
[number]                            -- force destination depth: 1 | 3
[string]                            -- type: byte | float | double
)
+++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
/Users/anthonyyuan/torch/install/bin/luajit: ...s/anthonyyuan/torch/install/share/lua/5.1/dok/inline.lua:738: <image.load> missing file name
stack traceback:
[C]: in function 'error'
...s/anthonyyuan/torch/install/share/lua/5.1/dok/inline.lua:738: in function 'error'
...s/anthonyyuan/torch/install/share/lua/5.1/image/init.lua:345: in function 'load'
./src/util.lua:13: in function 'crop_SR_LR_patches'
./src/util.lua:52: in function 'setBatch'
train-SRResNet.lua:104: in main chunk
[C]: in function 'dofile'
...yuan/torch/install/lib/luarocks/rocks/trepl/scm-1/bin/th:145: in main chunk
[C]: at 0x01092febc0

Have you ran th prepImageNet.lua first?
You have 0 ImageNet images loaded


no module named 'utils.compute_overlap'

When I try to run inference.py I get
Traceback (most recent call last):
File "inference.py", line 1, in
from model import efficientdet
File "/EfficientDet/model.py", line 17, in
from utils.anchors import anchors_for_shape
File "/EfficientDet/utils/anchors.py", line 5, in
from utils.compute_overlap import compute_overlap
ModuleNotFoundError: No module named 'utils.compute_overlap'
Do I have to change something to make it work?

Hi @hermanius ,you can pull the setup.py' and run python setup.py build_ext --inplace`.

Hi @xuannianz , that solved the issue, thank you very much for the patch.
I now have the following error running inference.py:
(Let me know wether I should open this as a separate issue)
Traceback (most recent call last):
File "inference.py", line 71, in
main()
File "inference.py", line 28, in main
score_threshold=score_threshold)
File "/EfficientDet/model.py", line 233, in efficientdet
features = backbone_cls(input_tensor=image_input, freeze_bn=freeze_bn)
File "/EfficientDet/utils/init.py", line 58, in wrapper
return func(*args, **kwargs)
File "/EfficientDet/efficientnet.py", line 471, in EfficientNetB1
**kwargs)
File "/EfficientDet/efficientnet.py", line 395, in EfficientNet
x = BatchNormalization(freeze=freeze_bn, axis=bn_axis, name='stem_bn')(x)
File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/keras/engine/base_layer.py", line 842, in call
outputs = call_fn(cast_inputs, *args, **kwargs)
File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/autograph/impl/api.py", line 237, in wrapper
raise e.ag_error_metadata.to_exception(e)
ValueError: in converted code:
/EfficientDet/layers.py:22 call  *
    if not training:
/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/autograph/operators/control_flow.py:893 if_stmt
    basic_symbol_names, composite_symbol_names)
/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/autograph/operators/control_flow.py:931 tf_if_stmt
    error_checking_orelse)
/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/util/deprecation.py:507 new_func
    return func(*args, **kwargs)
/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/ops/control_flow_ops.py:1174 cond
    return cond_v2.cond_v2(pred, true_fn, false_fn, name)
/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/ops/cond_v2.py:93 cond_v2
    verify_captures(_COND, [true_graph, false_graph])
/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/ops/cond_v2.py:768 verify_captures
    b1name=branch_names[i]))

ValueError: Tensor stem_bn_trainable:0 in true_fn is accessed from false_fn.

I'm using the efficientnet-b4_weights_tf_dim_ordering_tf_kernels_autoaugment.h5 model from 'master' of github.com:Callidior/keras-applications

I just saw the Callidor models are efficientnet backbone models and not efficientdet models, so of course it's not possible to just use them as pretrained models for your repo.
I suppose you don't have good efficientdet pretrained models available that you can share, and that can be used for finetuning (transfer learning)?


[HybridApp] NewDot app is not loading with chats

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

Version Number:
Reproducible in staging?: needs reproduction
Reproducible in production?: needs reproduction
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @quinthar
Slack conversation: https://expensify.slack.com/archives/C05LX9D6E07/p1719022224303489
Action Performed:

Launch the NewDot app

Expected Result:
App should be loading without any issue
Actual Result:
Not opening at all
Workaround:
unknown
Platforms:

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

[x] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/Expensify/App/assets/38435837/107b0df4-1b16-41fc-a2bd-795ec5216dcb
https://github.com/Expensify/App/assets/38435837/7be4175d-9012-48d2-b9a2-fbd274b7cb80

View all open jobs on GitHub

This issue has been resolved

Happening again.

Posted updated video in thread here: https://expensify.slack.com/archives/C05LX9D6E07/p1719861112788059?thread_ts=1719022224.303489&cid=C05LX9D6E07

Still not working.

Still happening

We are working on this @MelvinBot

Still working on it!

Can you try this again @quinthar ?

Tested, still happening; discussing in Slack

Seems to no longer be happening. Closing out for now. Reopen if it re-occurs!


ADD: PY3 requirements patch

Reviewers:

[ ] @mention

What's the purpose of the PR?

Description of changes

Notes to reviewers

Keep in mind while reviewing code:

Is relevant code tested?
Are added functions/methods documented?
Separation of concerns (SOC)
Don't repeat yourself (DRY)
Limit the number of positional arguments
Is function/method length reasonable?
Can code be broken down into smaller components?
Where do added functions/methods belong?
Are variable names descriptive?
Are errors handled appropriately?
Can logic be simplified?
Does the code make sense in context? (expand the diff)

Tests failing are due to server-side issues.  Merging.


[8.0.0-beta.4] Module not found: Can't resolve 'fs'

Describe the bug
Getting the following error when running the app:
wait  - compiling...
error - ./node_modules/next-i18next/dist/commonjs/serverSideTranslations.js:28:0
Module not found: Can't resolve 'fs'

Occurs in next-i18next version
8.0.0-beta.4
Steps to reproduce
You must provide a reproducible repository, or copy/paste blocks of code which are sufficient to reproduce the bug.
Expected behaviour
You must provide a clear and concise description of what you expected to happen.
Screenshots

Hi @kdelmonte – this is most likely because you have imported serverSideTranslations in some place where client side code is being run. It can only be used in server side data fetching methods such as getStaticProps and getServerSideProps. Hope that helps!

Thank you @isaachinman , I am only using it inside getServerSideProps:
export const getServerSideProps = async ({ locale }) => ({
  props: {
    ...(await serverSideTranslations(locale, ['common', 'loginPage'])),
  },
})

Please advise.

@kdelmonte You'll need to provide a reproducible repo.

@isaachinman I found this issue. I was exporting getServerSideProps from another file into the Next.js page file. Moving getServerSideProps directly to the page file fixed this issue. Thanks.

I have same issue, in 8.0.0-beta.4, but I did all the imports correctly and I totally tried to do it like in read.me file and example from the repository

wait  - compiling...
error - ../node_modules/next-i18next/dist/commonjs/serverSideTranslations.js:28:0
Module not found: Can't resolve 'fs'
null
react-i18next:: You will need to pass in an i18next instance by using initReactI18next

_app.tsx

import { appWithTranslation } from "next-i18next";
import { AppProps } from "next/app";

const App: React.FC<AppProps> = ({ Component, pageProps }) => {
  const MainLayout = (Component as any).MainLayout || DefaultLayout;
  const NestedLayout = (Component as any).NestedLayout || DefaultLayout;

  return (
    <>
     /.../
              <MainLayout>
                <NestedLayout>
                  <Component {...pageProps} />
                </NestedLayout>
              </MainLayout>
         /.../
    </>
  );
};

const DefaultLayout: React.FC = ({ children }) => <>{children}</>;

export default appWithTranslation(App);

file where i use serverSideTranslations:
import { serverSideTranslations } from "next-i18next/serverSideTranslations";
import { GetStaticProps } from "next";
import Grid from "@material-ui/core/Grid";
import { Header } from "../components/Header";

const AppLayout: React.FC = ({ children }) => (
  <>
    <Grid container direction="column" alignItems="center">
      <Header />
      {children}
    </Grid>
  </>
);

export const getStaticProps: GetStaticProps = async ({ locale }) => ({
  props: {
    ...(await serverSideTranslations(locale, ["translation"])),
  },
});

export default AppLayout;

file where i use useTranslation
import { useTranslation } from "next-i18next";

import { NavLink } from "../../../../components/NavLink";

export const Header = () => {
  const { t } = useTranslation();
  return (
/ .../
          <NavLink title={t("something1")} href="/#"  />
          <NavLink title={t("something2")} href="/#"  />
          <NavLink title={t("something3")} href="/#"  />
/ .../
  );
};

next-18next-config.js
import { initReactI18next } from "react-i18next";

const path = require("path");

module.exports = {
  defaultLocale: "en",
  locales: ["en", "ru"],
  localePath: path.resolve("./public/translations"),
  use: [initReactI18next],   // this line did not affect the error in any way
  debug: true,
};

next.config.js
module.exports = {
  images: {
    domains: ["..."],
  },
  webpack(config) {
    config.module.rules.push({
      test: /\.svg$/,
      issuer: {
        test: /\.(js|ts)x?$/,
      },
      use: ["@svgr/webpack"],
    });

    return config;
  },
  async redirects() {
    return [
      {
        source: "/",
        destination: "/...",
        permanent: false,
      },
    ];
  },
  i18n: {
    defaultLocale: "en",
    locales: ["en", "ru"],
  },
};

I'm resolve this issues, this code
export const getStaticProps: GetStaticProps = async ({ locale }) => ({
  props: {
    ...(await serverSideTranslations(locale, ["translation"])),
  },
});

must be in pages folder, but i'm use it in  components folder

This is exactly what happened to me. Beware, future readers.

Just want to note that this is a general NextJs "gotcha", and isn't necessarily specific to next-i18next. It's to do with how chunks are separated by webpack, despite there being client and server code in the same files (pages).

@isaachinman I am using NextJS and Storybook together and I did not share any code SSR related code within both separate builds.
I still realise when I use the useTranslation hook in the Storybook environment I receive the error:
ERROR in ./node_modules/next-i18next/dist/esm/config/createConfig.js
Module not found: Error: Can't resolve 'fs'

I could solve it for now with importing useTranslation from react-118next instead of next-i18next - I am not sure if thats the right way to do this.
// import { useTranslation } from 'next-i18next' // => throws error
import { useTranslation, UseTranslationOptions, UseTranslationResponse } from 'react-i18next'

type TranslationNamespace = 'common'

const useLmTranslation = (
  ns?: TranslationNamespace,
  options?: UseTranslationOptions
): UseTranslationResponse<string> =>
  useTranslation((ns as string) ?? 'common', options)

export default useLmTranslation

Hey @dohomi – hopefully we can solve this together.
As a package, next-i18next runs in both server and browser environments. For the createConfig code your error is referencing, we check if process.browser is true.
Strange that process.browser is not true in your Storybook env.
The swap from typeof window === 'undefined' to process.browser was done in #926 as a bundle optimisation. cc @felixmosh – we may need to also add typeof window === 'undefined' back in.

@isaachinman, sound OK, I will prepare a PR

Fixed by #974.

@isaachinman @felixmosh I updated to 8.0.1 and the error still exists and Storybook is not building:
ERROR in ./node_modules/next-i18next/dist/esm/config/createConfig.js
Module not found: Error: Can't resolve 'fs' in '/node_modules/next-i18next/dist/esm/config'
 @ ./node_modules/next-i18next/dist/esm/config/createConfig.js 54:15-28
 @ ./node_modules/next-i18next/dist/esm/appWithTranslation.js
 @ ./node_modules/next-i18next/dist/esm/index.js
 @ ./lib/hooks/useLmTranslation.ts

I'm only using the useTranslation hook the rest is run by react-i18next

Try to add a DefinePlugin to your story book build config with process.browser with a value of true

EDIT:
I add an alias to point via Webpack config to different alias to solve the error for now.
webpackFinal: async (config, {configType}) => {
    config.resolve.alias = {
      ...config.resolve.alias,
      'next-i18next': 'react-i18next'
    }
    return config
  }

Putting this in my /Storybook/.main.js solved it for me, cheers

I'm resolve this issues, this code
export const getStaticProps: GetStaticProps = async ({ locale }) => ({
  props: {
    ...(await serverSideTranslations(locale, ["translation"])),
  },
});

must be in pages folder, but i'm use it in components folder

This solved it for me

What to do if I need to use it in components folder?

you can still use the useTranslation hook in components

What to do if I need to use it in components folder?

you can still use the useTranslation hook in components

Thank you, I figured out 😅

@dohomi Did you figure out a workaround other than the alias to Webpack config?

I add an alias to point via Webpack config to different alias to solve the error for now.
webpackFinal: async (config, {configType}) => {
    config.resolve.alias = {
      ...config.resolve.alias,
      'next-i18next': 'react-i18next'
    }
    return config
  }

The root cause of this issue in storybook is related to this line.
Next replaces process.browser to false && typeof window to undefined by default.
Probably storybook doesn't do that, and with Webpack 5 there is no default mocking of fs.
You have 2 choices to solve the issue:

disable fs by

//webpack.config.js
webpackFinal: async (config, { configType }) => {
  config.resolve.fallback = {
    ...config.resolve.fallback,
    fs: false, // <------
  };
  return config;
};

if the first option doesn't works, change it to work with browserify-fs

webpackFinal: async (config, { configType }) => {
  config.resolve.fallback = {
    ...config.resolve.fallback,
    fs: require.resolve("browserify-fs"), // <------
  };
  return config;
};

@felixmosh Think we'd be better off relying directly on import at this point?

@isaachinman I'm not sure I got the idea, import of what?
The "problem" is that the code is isomorphic, and contains code of server side, import won't change anything (because you want to lazy load the server code on the server only).

import of whatever we are currently using fs to achieve. Dynamic imports on the server side should be possible.

import of whatever we are currently using fs to achieve. Dynamic imports on the server side should be possible.

import of what?

Hi!
when I follow this approach to get this work in storybook then I start getting this errors
ModuleNotFoundError: Module not found: Error: Can't resolve 'next-i18next/serverSideTranslations' in '/Users/sauldeleon/Projects/mine/whemotion/packages/blog/src/pages'

I bet this is because storybook is running in the same folder as the next.js app, and is loading I dont know why a file with that import.
Anybody has faced the same issue?

nextjs 12.2 add Edge SSR(https://nextjs.org/blog/next-12-2#edge-server-rendering-experimental) feature, it can also cause this issue.

I'm using getServerSideProps inside pages but still facing the issue

Сan anyone explain how to solve the problem using components outside of pages?

In my case, that was the use of
import { useTranslation } from 'next-i18next';

Replace all the occurences with
import { useTranslation } from 'react-i18next';

Importing serverSideTranslations as an util works fine for me
export async function getServerSideProps() {
	return {
		props: {
			...(await getI18nProps()),
		},
	};
}

Hey, I'm trying to get my head around the internationalization stuff but running into this 'Can't resolve  fs' error. I'm using Next.js 13 and with the app routing option. Has anyone managed to find a fix/work around for this?

Create a i18n.js file in the root of your project with the following:
`const NextI18Next = require('next-i18next').default;
module.exports = new NextI18Next({
defaultLanguage: 'en',
otherLanguages: ['fr'], // Add your supported languages here
localeSubpaths: {},
localePath: typeof window !== 'undefined' ? 'public/locales' : 'locales',
});`

I personally identified that dependencies serverSideTranslations and useTranslation cannot be together. It works for me.
./i18n/useTranslation.ts
import { useTranslation as useOfficialTranslation } from 'next-i18next';
import { UseTranslationResponse } from 'react-i18next';

import { namespaces } from '@/i18n/helpers/namespaces';

export function useTranslation (ns: string | string[]) {
  const translation = useOfficialTranslation(namespaces(ns));
  const res: UseTranslationResponse<string> = { ...translation };
  res.t = (key: string | string[], options?: any) => {
    if (typeof key === 'string' && typeof ns === 'string' && key.split(':').length === 1)
      return translation.t(`${ns}:${key}`, options);
    return translation.t(key, options);
  };
  return res;
}

./i18n/serverSideTranslations.ts
import { serverSideTranslations as officialServerSideTranslations }
  from 'next-i18next/serverSideTranslations';

import { namespaces } from '@/i18n/helpers/namespaces';

export async function serverSideTranslations (locale: string | undefined, ns: string | string[]) {
  return await officialServerSideTranslations(locale ?? 'de', namespaces(ns));
}

./i18n/helpers/namespaces.ts
export function namespaces (ns: string | string[]) {
  return ['common', ...(typeof ns === 'string' ? [ns] : ns)];
}

With this configuration I managed to bypass the explicit use of common. I will implicitly have the specific translation that I require and common.

getStaticProps

Hello @ivnmood But I want to use it in the components folder because I'm using the navbar as a component.
Here is mode I'm using.
import { serverSideTranslations } from 'next-i18next/serverSideTranslations';
import { UseTranslation } from 'next-i18next';

import Link from 'next/link';
import Language from './Language';

const links = [
    { href: '/', label: 'Home' },
    { href: '/about', label: 'About' }
];

export default function MenuItems() {
    const { t } = UseTranslation('menus');
    return (
        <ul className='flex'>
            {
                links.map((link, index) => (
                    <Link
                        key={index}
                        href={link.href}
                        className=" text-[#414052] hover:text-[#AC6DDE] transition-colors mx-[10px]"
                    >
                        {link.label}
                    </Link>
                ))
            }
            <Language />
            {t('about')}
        </ul>
    )
}

export async function getStaticProps({ locale }) {
    return {
        props: {
            ...(await serverSideTranslations(locale, ['menus']))
        }
    };
}

https://github.com/i18next/next-i18next#translate-in-child-components


Validate conf.py before build docs

This build https://readthedocs.org/projects/wagtail/builds/7918857/ produced a log into Sentry as an ERROR https://sentry.io/read-the-docs/readthedocs-org/issues/712546002/?query=is:unresolved which is annoying. These exceptions are raised to communicate a specific message to the user and shouldn't be logged as ERROR for us to take a look at.

I'm not sure if this is already implemented in v2 of our configuration or not.

There is some related work at #4495 and #4590.

I think this happens when we try to find aconf.py file for the user and we found more than one. With v2 we still keep the find automatically magic.

@stsewd So it this supposed to be fixed? If yes, I would like to take a stab at it.

@invinciblycool we can't validate this on the config module because we put None when rtd needs to find the file automatically, I think there is a case where rtd fails to find one or there are multiple config files, we need to dig a little more to see the entire flow and see how we want to fix this.

We haven't seen this in ~2 months, we should test if this can be replicated. I think it was raised when the project has 2 conf.py files, and isn't using a config file.

I was able to replicate this locally
Traceback (most recent call last):
  File "/home/stsewd/rtd/readthedocs.org/readthedocs/doc_builder/backends/sphinx.py", line 176, in append_conf
    self.config_file or self.project.conf_file(self.version.slug)
  File "/home/stsewd/rtd/readthedocs.org/readthedocs/projects/models.py", line 719, in conf_file
    ProjectConfigurationError.MULTIPLE_CONF_FILES,
readthedocs.projects.exceptions.ProjectConfigurationError: We found more than one conf.py and are not sure which one to use. Please, specify the correct file under the Advanced settings tab
 in the project's Admin.

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/stsewd/rtd/readthedocs.org/readthedocs/projects/tasks.py", line 565, in run_build
    outcomes = self.build_docs()
  File "/home/stsewd/rtd/readthedocs.org/readthedocs/projects/tasks.py", line 830, in build_docs
    outcomes['html'] = self.build_docs_html()
  File "/home/stsewd/rtd/readthedocs.org/readthedocs/projects/tasks.py", line 847, in build_docs_html
    html_builder.append_conf()
  File "/home/stsewd/rtd/readthedocs.org/readthedocs/doc_builder/backends/sphinx.py", line 183, in append_conf
    ).with_traceback(trace)
  File "/home/stsewd/rtd/readthedocs.org/readthedocs/doc_builder/backends/sphinx.py", line 176, in append_conf
    self.config_file or self.project.conf_file(self.version.slug)
  File "/home/stsewd/rtd/readthedocs.org/readthedocs/projects/models.py", line 719, in conf_file
    ProjectConfigurationError.MULTIPLE_CONF_FILES,
readthedocs.projects.exceptions.ProjectConfigurationError: A configuration file was not found. Make sure you have a conf.py file in your repository.

with this structure
├── conf.py
├── more
│   └── conf.py
├── README.rst
└── requirements.txt

@stsewd
Is this still valid?

Yeah, I think we fail with a general exception with the above structure https://github.com/rtfd/readthedocs.org/issues/4745#issuecomment-470754025

@stsewd
I am not sure as what needs to be done here.
Can you provide some details here.

www.mynewish.com


Reporting window size events in stdin

Windows Terminal version
1.16.10261.0
Windows build number
10.0.19044.0 (21H2)
Other Software
Steps to reproduce
I am setting my input handle with ENABLE_VIRTUAL_TERMINAL_INPUT | ENABLE_WINDOW_INPUT | ENABLE_MOUSE_INPUT |  ENABLE_EXTENDED_FLAGS,
Stdout's mode is left as default (0x7) with requests for focus reporting, mouse button reporting, and disabling mouse_any reporting: \ESC[?1004h\ESC[?1003l\ESC[?1002h.
Then I read from my stdin.
Expected Behavior
Some sort of binary/VT sequence of bytes to represent a window resize event.
Actual Behavior
Keyboard events are present in stdin (in their VT form), as are mouse and focus events. However, I have nothing when I resize the WT window (both via mouse or via fullscreen toggle via F11).
The phrase "User interactions that change the size of the console screen buffer are reported in the console's input buffer" for ENABLE_WINDOW_INPUT on the page https://learn.microsoft.com/en-us/windows/console/high-level-console-modes has lead me to believe that I should get at least something - maybe not a VT escape sequence per se, but something.
Is there anything in my configuration amiss? Is there a gentle \ESC[?10XXh equivalent of "please give me window resize events, too"?
The example in https://learn.microsoft.com/en-us/windows/console/reading-input-buffer-events works fine without additional configuration.
Cheers!

Hey @ShrykeWindgrace! This is one place where we could seriously do better. Alright, here goes.
Right now, the window size events come in in the form of INPUT_RECORD structures when you use ReadConsoleInputW. Even in VT mode.
We don't have a VT encoding for window size events for one really annoying reason: nobody does! On non-Windows platforms, these things come in via an asynchronous signal SIGWINCH, and are read out of the tty via the TIOCGSZ IOCTL. We don't have either of those things on which to build a good window size notification primitive, so instead it comes in as raw events in the input stream.
So, I can give you these two things:

A "template" for how we deal with it, when all we want is a simple application that reads VT and knows when the window changes size. class ConsoleInputReader sort-of encapsulates all of that annoying stuff and turns your input stream into (1) a buffer of raw VT and (2) callbacks for window size events.
A link to #281, one of our oldest open issues, which tracks (despite the name) teasing apart how and where and when we generate window input events.

I briefly had a prototype (branch on this repo that mentions l9 and hax) for how the console host could deliver asynchronous signals to client applications for things like this, but it felt too dangerous x low-priority to invest in.
Hope that helps! I'm gonna mark this issue up as a question with resolution=answered and later convert it to discussion. Please feel free to come back with any followups!

Thanks a lot for the comprehensive answer! I'll see what I can do with this info =)


Rasa_core failed to start

Rasa Core version:  0.12.2
Python version: 2.7.15
Operating system (windows, osx, ...): osx ( also tried with python:2 image for docker )
Issue:
Traceback (most recent call last):
  File "chatbot/dialogue_management_model.py", line 8, in <module>
    from rasa_core.agent import Agent
  File "/home/workspaceChatbot/rasa_core/rasa_core/agent.py", line 41
    def load_from_server(interpreter: NaturalLanguageInterpreter = None,
                                    ^
SyntaxError: invalid syntax

Need help please :'(

Resolved using python:3.5 docker image :)


pki match: remove return values that are always nil

Simplifies the usage of pki match functions by removing the return values that are always nil.
Kind
/kind cleanup
Release Note
NONE

/approve


fix: ticker response timestamp attributes

Change ticker response naming timestamp to exchange timestamp as per [kite connect doc](fix: timestamp ticker response attributes).

This will be a breaking change, and since there is only one Timestamp in the ticks, I doubt there will be confusion for users of the library. Max we can do is add a godoc on top of this in models. Like so:
// Tick represents a single packet in the market feed.
type Tick struct {
...
        // Timestamp contains the exchange timestamp
	Timestamp          Time
...
}

But in my opinion even this might not be neccessary.
Thoughts @vividvilla ?


feat(tutorial): complete step 5 Hursley

:+1::skin-tone-2:   Looks good - I have recorded your completion of this step!


Upgrade dependencies (yarn/maven plugins/java)

Some frontend dependencies updates are previously done in 423f6cc65b57c4bb101f186ea8a6aa3312c9da7a.
Some dependencies upgrade can not be done because of detected issue :

maven-javadoc-plugin : https://issues.apache.org/jira/browse/MJAVADOC-694
revapi-maven-plugin : https://github.com/revapi/revapi/issues/252

Some can not be done because we are stuck to java7 for the lib and java8 for servers demo. (see https://github.com/eclipse/leshan/issues/924)

Tested at : https://ci.eclipse.org/leshan/job/leshan-hudson-test/69/


New-DbaLogShippingPrimaryDatabase

Function to save information for log shipping about the primary database

This function executes the procedure "master.dbo.sp_add_log_shipping_primary_database " which does a couple of things:

Add a backup job which will be disabled
If not already created, create an alert job which will be enabled
Set values in several tables. The following tables will be written:
SELECT * FROM msdb.dbo.log_shipping_monitor_primary
SELECT * FROM msdb.dbo.log_shipping_primary_databases
SELECT * FROM msdb.dbo.log_shipping_monitor_alert
SELECT * FROM msdb.dbo.log_shipping_primary_databases

Downside to testing this function is that the values cannot be easily removed. At least I don't know if a manual removal will be successful.
The way I removed it essentially is by executing the entire Invoke-LogShipping for that database and remove it through the GUI. This is something I'm going to look into.
If everything works out fine the results would look similar to this


[DEP-40] - BUG: yarn dependencies identifier

Adjusting regex that gets version to avoid looking for letters and just checks numbers and special caracteres.

I think the tests could be tweaked. That exists column seems weird to me.


Fix Null Step Configuration/Parameters Error

Describe changes
Some users faced errors when writing steps to our database with None step configuration or parameters. This was caused by too large configurations/parameters that exceed the 4K character limit after encoding, which SQLModel apparently silently set to None then. To fix this, I changed the type of those columns to TEXT.
While at it, I also adjusted some code in the metadata store to ensure those parameters can never be set to None, no matter what is written in MLMD.
Pre-requisites
Please ensure you have done the following:

[x] I have read the CONTRIBUTING.md document.
[ ] If my change requires a change to docs, I have updated the documentation accordingly.
[ ] If I have added an integration, I have updated the integrations table and the corresponding website section.
[ ] I have added tests to cover my changes.

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Other (add details above)

This generally looks good to me. I'm wondering whether something like this is going to bite us at some point in the future if we use the text type too liberally?:

I think it should be fine since we are using SQLAlchemy's TEXT, which is translated to different types depending on the underlying engine, so I would expect that TEXT will simply get translated to another data type (perhaps VarChar(65535)?) once the TEXT type no longer exists in MySQL.


Change indexing in UnstructuredRepertoire intra batch comp

Related issues: N/A
Quick fix to select the current fitness in intra batch comp in the add method of unstructured repertoire. The current indexing will fail if any of the descriptors in the batch have nan values (e.g. when re-initialising the repertoire in AURORA, after training the auto-encoder).
This PR introduces:

change indexing of fitnesses to use current_index in intra batch comp rather than 0th knn_relevant_indices.

Checks

[x] a clear description of the PR has been added
[ ] sufficient tests have been written
[ ] relevant section added to the documentation
[ ] example notebook added to the repo
[ ] clean docstrings and comments have been written
[ ] if any issue/observation has been discovered, a new issue has been opened

Future improvements
[List here potential observations made and/or improvements that could be made in the future. If relevant, open issues for those.]

Hi @LisaCoiffard, thank you very much for the PR! Could you fix the style issue in the CI, so that I can merge it? :D

Fixed this issue! You can merge it :)


Option for combined GOPATH

Add an option to prepend the GOPATH, instead of replacing it

replace
prepend

Closing as go modules make gows no longer needed.


Conversion of special characters in template data

I have a template that prints out the version of my Helm chart
    version: "{{ .chartVersion }}"

where the version string may sometimes contain characters like +. Example: 1.5.0-PullRequest0031.33.20220104212638.d0533b9
When rendered the output looks like this:
    version: "1.5.0-PullRequest0031.33+20220104212638.d0533b9"

Is there a way to get special characters to not be encoded?

Possibly due to the use of "html/template" instead of "text/template"?
https://github.com/chuhlomin/render-template/blob/main/main.go#L7
See https://stackoverflow.com/a/48527166/576333

Thanks for burning this up! There is no reason to use html/template package.
Switched to text/template in v1.4 release. Please, give it a try.


fix(examples): add logging messages and force build

Alaska Airlines Pull Request
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Summary:
Add example logging and force a release build
Type of change:
Please delete options that are not relevant.

[ ] New capability
[ ] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[x] Other (please elaborate)

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

:tada: This PR is included in version 1.9.10 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


The shape of the logits

the output 'logits' are of shape (1,768,8,7,7), but it should be (batch, num_class). How to adapt the code to classify videos?

The fc layer is defined in (https://github.com/SwinTransformer/Video-Swin-Transformer/tree/master/mmaction/models/recognizers)/base.py   from the official implementation.

I'm not completely sure, but in my opinion, you can perform video classification by applying softmax or sigmoid.


Inconsistency with "Duplicate Line"

Describe the bug
I feel there is some inconsistency regarding the "duplicate and increment" feature.
In the docs it is mentioned that it "duplicates the line" which it does when you just have the cursor in the current line. It does not duplicate the full lines however when you make a selection.
IntelliJ internally has two different commands, "duplicate" and "duplicate line", where the latter always duplicates the full lines.
I think it would make more sense to duplicate full lines rather than just selections (or at least make it configurable in the settings).
To Reproduce
Steps to reproduce the behavior:

Have a statement that takes more than one line, e.g.

int x1 = 5;
int x2 = 10;
System.out.println("some very very very very very very long text plus the content of " +
    x1);

Now you want both lines 3 and 4 duplicated and x1 turned into x2.

In case you properly select everything:

and then choose "duplicate and increment", this will work as expected:
System.out.println("some very very very very very very long text plus the content of " +
    x1);
System.out.println("some very very very very very very long text plus the content of " +
    x2);

However, if you just select just somewhere in the middle, like this:

this will produce broken code:
      int x1 = 5;
      int x2 = 10;
      System.out.println("some very very very very very very long text plus the content of " +
           x1y very very very very long text plus the content of " + x2);

Expected behavior
The line duplication should work the same way as the "duplicate line" feature in IntelliJ, duplicating whole lines instead of just the selection.
Environment :
IntelliJ IDEA 2020.3.3 (Ultimate Edition)
Build #IU-203.7717.56, built on March 15, 2021
Licensed to Bernhard Streit
Subscription is active until April 24, 2021.
Runtime version: 11.0.10+8-b1145.96 x86_64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
macOS 10.16
GC: ParNew, ConcurrentMarkSweep
Memory: 5391M
Cores: 16
Registry: analyze.exceptions.on.the.fly=true, ide.tooltip.initialDelay=674, ide.appIcon.requestAttention=false
Non-Bundled Plugins: AceJump, IdeaVIM, org.jetbrains.IdeaVim-EasyMotion, com.github.holgerbrandl.pasteimages/, google-java-format, com.crunch42.openapi, EclipseCodeFormatter, GrepConsole, String Manipulation, com.dubreuia, com.krrrr38.idea.mockito.postfix, com.sunny.plugin.MockitoGenPlugin, com.jetbrains.codeWithMe, idea.plugin.protoeditor, org.sonarlint.idea, MavenRunHelper, intellij.prettierJS, org.jetbrains.kotlin, org.bitstrings.idea.plugins.TestInsanity, org.mapstruct.intellij, PlantUML integration

Where is mentioned that it duplicates the line?

Oh, found it :) That is wrong :-)

Not quite the fix I hoped for but fair enough 😂

Duplicating selection is more universal, not sure if it is worth to make a line feature.

I agree, easiest solution would be to have a "duplicate selection / whole line" toggle button in the settings where you could change this globally for all StringManipulation duplicate actions.
Would love to help, but honestly never done an IntelliJ plugin...


Update sbt-scalafix to 0.12.0

I see that #161 got stuck a little bit. It's a separate PR since I can't commit to the original PR.

@rossabaker @armanbilge feel super dumb every time I request a review in FOSS, sadly, it's often the same folks being tagged. Sorry if this contributes to your anxiety!
But I see only you merge PRs in this repository. Just a friendly reminder about this PR, to thrust it forward to unlock several downstream libraries.

the churn in the output is going to be from the transitive dependency

much likely yes

not something that impacts the ability to use it downstream

can only hope that it's so


Better coverage and addition of coveralls

Description
Better coverage, more tests and added coveralls submission through travis ci
Motivation and Context
Tests helps to control project.
Types of changes

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

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.
[x] I have read the CONTRIBUTING document.
[x] All new and existing tests passed.

This pull request introduces 1 alert when merging 4f6a3fa01c9ca0820268a58e4493dad21af6477a into 591da6dfacac7aa716ae024285e78ea6afdd45a1 - view on LGTM.com
new alerts:

1 for 'import *' may pollute namespace

Comment posted by LGTM.com

This pull request introduces 1 alert when merging 7c0a79a12f88e31aa3bc12adb805968ebeb121f8 into 591da6dfacac7aa716ae024285e78ea6afdd45a1 - view on LGTM.com
new alerts:

1 for 'import *' may pollute namespace

Comment posted by LGTM.com


SchemaComparator does not handle references to $defs

I am using SchemaComparator to check that there are no compatibility issues between two version of the same json schema, which is decoded as apispec.Schema.
Following program highlights the problem
object RefsProblem {

  import sttp.apispec.validation.SchemaComparator
  import sttp.tapir
  import sttp.tapir.docs.apispec.schema.TapirSchemaToJsonSchema

  object v1 {
    case class Child(name: String) derives tapir.Schema
    case class Parent(name: String, children: Seq[Child]) derives tapir.Schema
  }
  object v2 {
    case class Child(name: String, age: Int) derives tapir.Schema
    case class Parent(name: String, children: Seq[Child]) derives tapir.Schema
  }

  private def apiSpecSchema[T: tapir.Schema] = TapirSchemaToJsonSchema(
    implicitly[tapir.Schema[T]],
    markOptionsAsNullable = true,
  )

  private def defs(schema: apispec.Schema): Map[String, apispec.Schema] = schema.$defs.getOrElse(Map.empty)
    .collect { case (k, s: apispec.Schema) => k -> s }

  def main(args: Array[String]): Unit = {
    val writer = apiSpecSchema[v1.Parent]
    val reader = apiSpecSchema[v2.Parent]
    val comparator = new SchemaComparator(defs(writer), defs(reader))
    comparator.compare(
      writer.copy($schema = None, $defs = None),
      reader.copy($schema = None, $defs = None),
    ).map(_.toString).foreach(println)
  }
}

And to avoid the problem specified in #170, I reset $schema and $defs and pass $defs as arguments to new SchemaComparator.
Such comparison does not identify new field age: Int in v2.Child schema, which it should detect.
The reason for this is that there is no $ref lookup implemented for #/$defs/ and it only supports #/components/schemas/.
After playing around with code of SchemaComparator and adding DefsRef along with LocalRef, I was able to make it work and produce the desired validation check
private object DefsRef {
  private val DefPrefix = "#/$defs/"
  def unapply(ref: String): Option[String] =
    Option.when(ref.startsWith(DefPrefix))(ref.stripPrefix(DefPrefix))
}

incompatible schema for property children:

incompatible schema for items:
target schema requires properties: age

I can submit a PR to fix this and #170 but I need a reaction from the maintainer to validate both issues.

Sure this sounds like something we should support, do you maybe have some pointer as to when $defs should be used, comparing to #/components/schemas/? So that we follow the spec
Also cc @ghik

Yes, the comparator is limited in many ways and generally optimized for schemas generated from tapir endpoints. $def is not understood at all. More details in the PR: https://github.com/softwaremill/sttp-apispec/pull/157

@adamw it is a good question.
Looking at the latest specification of json schema, $ref can have multiple ways of referring to a schema, the one which is missing is called Schema with a JSON Pointer and the format is { "$ref": "#/$defs/string" }.
Such reference is defined here  https://github.com/softwaremill/tapir/blob/91a95b77009ed887fea9407eb8f8f6948e8984c6/docs/apispec-docs/src/main/scala/sttp/tapir/docs/apispec/schema/TapirSchemaToJsonSchema.scala#L20

$def is not understood at all

@ghik This is what I also noticed :)
But it is relatively easy to support it. If it is done then TapirSchemaToJsonSchema can be used to generate apispec.Schema and such schemas could be used in SchemaComparator.
I have it working on my machine without major changes to the comparator but I had to refactor the resolution/normatlization logic

If it's a simple change - why not, but I suspect there must be some difference between references using $ref + $def and #/components/schemas/?

That is, when should one be used, and when the other?

@adamw I believe $defs is a feature of JSON Schema (in isolation), while #/components/schemas is an OpenAPI specific thing. For this reason I assumed that there shouldn't be much reason to use $defs in schemas generated from tapir endpoints.

Oh, it seems my assumption was somewhat subconscious and, in fact, not a right one, looking at this code which uses $defs.
IMO it would still be more natural to use #/components/schemas for that, but I might be missing some context.

@ghik I think you're right, $defs allows you to reuse schemas within a single JSON schema, that is in the context of OpenAPI in within a single body definition, for example. While components/schemas allows you to reuse schemas across  different json schemas, for different bodies.
In the tapir schema -> json schema conversion, we cannot use the shared pool (as it doesn't exist in that context), so we're using $defs.
Anyway, seems that supporting $defs in the same way as supporting components/schemas seems the way to go, so @seveneves if you'd like to contribute that change, that would be great :)


Fix: Password storage issue with PostgreSQL backend

Description
This pull request addresses an issue where the password field of the User struct was being stored as an empty string in the PostgreSQL database, while working correctly with the SQLite backend (not sure why). The issue was caused by the custom MarshalJSON and String methods defined on the PrivateString type, which always returned an empty string.
Changes

Implemented custom Value and Scan methods for the PrivateString type to handle database serialization and deserialization correctly.

The Value method returns the actual string as a driver.Value when storing the user record in the database.
The Scan method properly assigns the string from the database to the PrivateString field when querying the user record.

The custom methods ensure that the string is stored in the PostgreSQL database, while still keeping the password/hash hidden in the JSON and string representations of the PrivateString type.

Testing

Verified that the password field is stored correctly in the PostgreSQL database after applying the changes.
Local user creation and login work normally.
Local user login works normally after password changed by admin.

thanks


License

Hey @yxgeee ,
thanks for sharing your code :)
Would you mind providing a license file for it?
Thank you and best regards,
Patrick

Thanks for your reminder. I have added a license file.

Thanks a lot :)


[BUG] Exception Trace in console becuase of undefined editor

Hello,
I've seen the trace below in the console log.
Every time you 'request' the editor from vscode.window.activeTextEditor you should check if it is not undefined.
Anyway, thanks for your extension.
Kind regards
Frank
Trace:
mainThreadExtensionService.ts:76 TypeError: Cannot read properties of undefined (reading 'document')
	at Object.updateContextKey (vscode-file://vscode-app/home/fru/.vscode-insiders/extensions/takumii.markdowntable-0.9.1/out/commands.js:12:24)
	at vscode-file://vscode-app/home/fru/.vscode-insiders/extensions/takumii.markdowntable-0.9.1/out/extension.js:21:89
	at u.invoke (vscode-file://vscode-app/usr/share/code-insiders/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:58:145)
	at w.fire (vscode-file://vscode-app/usr/share/code-insiders/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:58:1856)
	at vscode-file://vscode-app/usr/share/code-insiders/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:86:66000
	at u.invoke (vscode-file://vscode-app/usr/share/code-insiders/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:58:145)
	at w.fire (vscode-file://vscode-app/usr/share/code-insiders/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:58:1856)
	at _.acceptDocumentsAndEditorsDelta (vscode-file://vscode-app/usr/share/code-insiders/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:85:75835)
	at _.$acceptDocumentsAndEditorsDelta (vscode-file://vscode-app/usr/share/code-insiders/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:85:73924)
	at i._doInvokeHandler (vscode-file://vscode-app/usr/share/code-insiders/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:89:13826)
	at i._invokeHandler (vscode-file://vscode-app/usr/share/code-insiders/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:89:13510)
	at i._receiveRequest (vscode-file://vscode-app/usr/share/code-insiders/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:89:12171)
	at i._receiveOneMessage (vscode-file://vscode-app/usr/share/code-insiders/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:89:10841)
	at vscode-file://vscode-app/usr/share/code-insiders/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:89:8947
	at u.invoke (vscode-file://vscode-app/usr/share/code-insiders/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:58:145)
	at w.fire (vscode-file://vscode-app/usr/share/code-insiders/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:58:1856)
	at d.fire (vscode-file://vscode-app/usr/share/code-insiders/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:66:19030)
	at vscode-file://vscode-app/usr/share/code-insiders/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:104:34269
	at u.invoke (vscode-file://vscode-app/usr/share/code-insiders/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:58:145)
	at w.fire (vscode-file://vscode-app/usr/share/code-insiders/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:58:1856)
	at d.fire (vscode-file://vscode-app/usr/share/code-insiders/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:66:19030)
	at o._receiveMessage (vscode-file://vscode-app/usr/share/code-insiders/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:66:23611)
	at vscode-file://vscode-app/usr/share/code-insiders/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:66:21145
	at u.invoke (vscode-file://vscode-app/usr/share/code-insiders/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:58:145)
	at w.fire (vscode-file://vscode-app/usr/share/code-insiders/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:58:1856)
	at w.acceptChunk (vscode-file://vscode-app/usr/share/code-insiders/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:66:15861)
	at vscode-file://vscode-app/usr/share/code-insiders/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:66:14991
	at Socket.P (vscode-file://vscode-app/usr/share/code-insiders/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:104:13797)
	at Socket.emit (node:events:390:28)
	at addChunk (node:internal/streams/readable:315:12)
	at readableAddChunk (node:internal/streams/readable:289:9)
	at Socket.Readable.push (node:internal/streams/readable:228:10)
	at Pipe.onStreamRead (node:internal/stream_base_commons:199:23)

@frudolph77 Thanks to your feedback!


CI: replace compiletest with trybuild

in the same vein as rtfm-rs/cortex-m-rtfm#268

bors try

bors r+

bors +

bors +

bors r=perlindgren


Emmet doesn't work in sass and stylus files in 1.20

Have the below 2 extensions installed

robinbentley.sass-indented
sysoev.language-stylus

Open a .sass and .stylus file and run the Emmet: Expand Abbreviation command on any emmet abbreviation like say m10
Sample content for both sass and stylus files:
.foo
  margin: 10px
  padding: 10px

  .bar
    font: 14px
    left: 10px

This is a regression

Fixed in master, waiting for triage on Monday to cherry-pick the fix to release


add script to test the old apt-repo

ping @tianon @cpuguy83
we can run this on jenkins to ensure people know the old apt repo works for various versions that are not in the new repos :)
related: #12461

@mattmoor you will hopefully like this :)

What's the correct way to run this? just run hack/make.sh test-old-apt-repo fails unless you have docker in your $PATH

hack/make/test-old-apt-repo

+1

nice!

LGTM

Oh I like that ! :wink:
LGTM


gpg sig for navcoin-6.0.2-x86_64-linux-gnu.tar.gz?

I suggest that the sha256sums on this page
https://github.com/NAVCoin/navcoin-core/releases/tag/6.0.2
be labeled "sha256sums" instead of just "hashes". I see nothing like a gpg signature for the hashes or navcoin-6.0.2-x86_64-linux-gnu.tar.gz. I'd feel a lot better about it if it were signed with a signature that could be verified with multiple public keyservers.

you can find signatures for the different gitian builds for each commit here:
https://build.nav.community/binaries/master/gitian.sigs/
signed with https://keys.openpgp.org/vks/v1/by-fingerprint/1BF9B51BAED51BA0B3A174EE2782262BF6E7FADB<EMAIL_ADDRESS>


chore(main): release 1.2.1

:robot: I have created a release beep boop
1.2.1 (2023-06-22)
CI

fix wrong release tag and name (a3a06cc)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/itsHenry35/ledu/releases/tag/v1.2.1 :sunflower:


After addressing it, you can label the PR with ready to publish etc


KEP-4008: CRDValidationRacheting - Beta Docs

~placeholder for~ https://github.com/kubernetes/enhancements/issues/4008

/remove-language hi
/remove-language zh

Hey @alexzielenski 👋 please take a look at Documenting for a release - PR Ready for Review to get your PR ready for review before Tuesday March 12th 2024 18:00 PST. Thank you!

/retitle [WIP] KEP-4008: CRDValidationRacheting - Beta Docs
This looks like it's missed the deadline; if the feature should graduate, please do add the associated docs.

Updated with changes for beta. Ready for review
/cc @jpbetz @cici37 @sttts
for technical review

Hey @alexzielenski 👋 please take a look at Documenting for a release - PR Ready for Review to get your PR ready for review before Tuesday March 12th 2024 18:00 PST. Thank you!
This looks like it's missed the deadline; if the feature should graduate, please do add the associated docs.

Isn't the deadline later today?

Isn't the deadline later today?

It might be. Make sure to mark it as not work in progress before then.

/remove-area blog

Hey @jpbetz @deads2k  :wave: please provide a technical review for this PR by March 26th 2024 18:00 PT to get this into the release. Thank you!

/milestone 1.30

/sig api-machinery

Hello @alexzielenski @jpbetz @deads2k   :wave:!
I'm reaching out from the Docs team. Just checking in as we approach Docs Freeze on Tuesday March 26th 18:00 PDT. This documentation appears to still be under review. To meet the Docs Freeze, this PR must have a technical review as well as lgtm and approve labels applied, without any unaddressed comments or concerns from SIG Docs. The status of this enhancement is marked as at risk for docs freeze. Thank you!

/lgtm on tech side. Thanks

Technically this looks good.  Added one minor cleanup comment and agree with all of @sftim's proposed improvements.
/approve

Updated with all feedback

For beta, I'd expect to see the conceptual explanation in https://kubernetes.io/docs/concepts/extend-kubernetes/api-extension/custom-resources/ (it is OK to retain a mention of ratcheting inside the task page)
@alexzielenski: would you be willing to migrate this content to where it belongs? SIG Docs can help explain what's involved, but it's pretty much a cut and paste thing.

There is already a linked reference to ratcheting and further docs on this page: https://github.com/kubernetes/website/blob/e894028da7b101c3e4cef0ebafd8ff2dabddce77/content/en/docs/concepts/extend-kubernetes/api-extension/custom-resources.md#L212
Did you mean to migrate all the ratcheting docs? I did not see a natural place for it on that page (especially since the context about OpenAPI schemas & validation is also not fully explained, instead linking to the task page.).

Just a heads up that Doc Freeze is approaching fast tomorrow March 26th at 18:00 PDT and starting this cycle, we need to file an Exception after this time.
This one looks pretty close and we'd like to help you get it over the finish line!
@kubernetes/sig-docs-en-owners Does this one look good for an approve from y'all? Thanks!

removed duplicated paragraph

@kubernetes/sig-docs-en-owners Feedback has been addressed, does this one look good to approve?

/lgtm

/approve

/milestone 1.30


Misalignment of method parameters located on separate lines

VSCode Version: 1.10.2
OS Version: Windows 10

Steps to Reproduce:
Current Format
this.authService.resetPassword(payload)
    .map((data: { message: string }) => data.message)
    .subscribe(
    (message: string) => this.onResetPassword(message),
    (error: any) => console.error(error)
    );

Expected Format
this.authService.resetPassword(payload)
    .map((data: { message: string }) => data.message)
    .subscribe(
        (message: string) => this.onResetPassword(message),
        (error: any) => console.error(error)
    );

Thanks. This looks like it falls under: https://github.com/Microsoft/TypeScript/issues/14586 which is being tracked upstream on the TypeScript side. I'll add your example to that bug


Better Documentation

Couple documentation issues...

WebVTT integration is mentioned http://mediaelementjs.com but it isn't clarified how to add it.

From the source of the page it is:

 elements include just: subtitles, captions, descriptions, chapters & metadata.  In which case I think there's a problem with how you implemented it on the example site above.  Shouldn't WebVTT files be captions rather than subtitles?

The start language I'm assuming is a 2 character ISO code, right?

// automatically selects a  element
startLanguage: 'en',

I'm unclear what Google Translate can do as it can't work from the video, but I'm assuming would have to go off of a pre-defined .  It can't do the initial captioning, right?
// automatically selects a  element
startLanguage: '',
// a list of languages to auto-translate via Google
translations: [],
// a dropdownlist of automatic translations
translationSelector: false,
// key for tranlsations
googleApiKey: ''

Also, is there a way to provide a different audio tracks? Doesn't seem like HMTL5 has allowed for that:
http://www.w3.org/TR/html5/video.html#the-track-element

From this slideshow I just got some other useful info:
http://blog.gingertech.net/wp-content/uploads/2010/12/HTML5AuDesc/#slide31
      <video poster="video.png" controls>
        <source src="video.mp4"  type="video/mp4">
        <source src="video.webm" type="video/webm">
        <track ref="#description" label="description" srclang="en">
      </video>

      <audio id="description">
        <source src="description.mp3"  type="audio/mp3">
        <source src="description.ogg" type="audio/ogg">
      </video>

But strangely they didn't list the  kind in their example.

Thanks John.
I just blogged about my efforts here - http://openconcept.ca/blog/mgifford/mediaelement-accessibility-conference-drupal
I haven't captioned the piece yet, but just looking for more documentation to make implementation easier.
I hadn't heard of the "subtitles" vs. "captions" debate.  Guess HTML5 is still fluid enough it's going to be a matter of what assistive technology & browsers decide to support.
So, it should be possible technically to offer different audio tracks for the same video?  It's something that could in the roadmap?
Thanks for your other clarifications.

Looks great!
I'd love to build multiple audio tracks, but I'm  not sure when we'll do it because the specs are still being developed for native implementations.


deCONZ has all my devices, but Phoscon App (and Homebridge) do not

To be honest, I am quite lost. After having a very stable system based on 2.05.77 for a while (even though without VNC) I did a Docker container pull for the latest version, and an up -d to restart. All was fine, Phoscon App gave me 2.05.78 as current version and all devices were still there, working perfectly fine with the Homebridge Hue plugin. I left that +/-24hrs until I decided to look into the RaspBee II firmware update again, since now the logs show there is an update available (I still have 26520700, logs show 26580700). So I did how described in the readme.md - stop the container, and initiate the FWflasher. Since it (again) gave me the error of missing GLIBC version I stopped the process and brought my Docker container back with up -d, no problem.
Now it's the point I cannot wrap my head around.
To check things, I access the Phoscon App. The Phoscon App lost my gateway name, and asks me for a new one and to set a password.  Phoscon App is empty, rooms and devices are gone. Bummer.
Desperate I try to access deCONZ via VNC (and did not expect it to work, since it did not work in .77). But hello, VNC works like a charm and I have a connection. AND all my devices are there in deCONZ. WHAT???
Ok, back to Phoscon App to check things, and now there is a Group 420 showing up, and the lights section has my two bulbs, one outlet and the range extender. Lights and outlet are in Group 420 (my former basement room), but all sensors, switches, and the roller blinds are missing (all passive battery devices).
My blinds still work, and I can move them up/down with the Ikea up/down switch (they were paired before). But all of the devices are not available in Homebridge, and not in HomeKit (ALL of them).
So a bit context, but I really do not see why this is happening:

Pi 3B+ running Homebridge in docker, all latest versions
the same Pi running deCONZ in docker
Homebridge Config-Ui-X running on 192.168.1.xxx:8080
Phoscon App running on 192.168.1.xxx:8282
VNC (now again) running on 192.168.1.xxx:5900
docker ps gives me two running containers (homebridge and deconz) as expected

Any ideas guys? Is it the missing firmware update that causes this? Why did Phoscon App lose everything, then partly reappearing, but nothing is exposed to anywhere?
Help is really appreciated. Let me know if further details (e.g. logs) might help.

Cross-posted here: https://github.com/dresden-elektronik/deconz-rest-plugin/issues/2974

As there hasn't been any new replies and as there are newer versions, i'm closing this issue. If this needs to be re-opened, please ping me :)


FIXED: Messed up screen geometry after return from VIRTUAL1

Hello, a short info:
In Ubuntu 24.04 X11, after I ended Immersed in VIRTUAL1, I sometimes had screen geometry broken, or missing wallpaper. I have added these commands after running Immersed to fix everything. eDP1 is name of my laptop screen:
    xrandr --output VIRTUAL1 --off
    sleep 1
    xrandr --delmode VIRTUAL1 "3200x1800_60.00"
    sleep 1
    xrandr --rmmode "3200x1800_60.00"
    sleep 1
    xrandr --output eDP1 --off && xrandr --output eDP1 --mode 1920x1080

Jan

Thanks for letting me know!
Can you try the new shared Intel virtual monitor script? https://github.com/augustoicaro/Immersed-Linux-Virtual-Monitors/blob/main/scripts/intel-scripts/immersed_tie_fighter.sh
If the problem also happens in the script, we can add your steps at the end of it.


make color controls prettier/smaller/simpler

If we still need to do this, add an issue in https://github.com/openconnectome/NeuroDataViz/
Maybe re-evaluate in 2016?


data required

Hello Jae Yoon Heo. can you please provide me the link of  dataset where it is stored. i need it urgently. i will be realy thankful to you.

The dataset was obtained from AI Hub, a public DB in Korea, but the post seems to have disappeared after the website renewal.

Okay,. thank
you so much
Thanks & Regards
Shashank Chauhan
MS Candidate, School of Industrial Engineering,
University of Ulsan, Ulsan, South Korea
On Wed, Jun 7, 2023 at 4:15 PM jaeyoonheo @.***> wrote:

The dataset was obtained from AI Hub, a public DB in Korea, but the post
seems to have disappeared after the website renewal.
—
Reply to this email directly, view it on GitHub
https://github.com/jaeyoonheo/action-classifier/issues/4#issuecomment-1580075307,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATRL6BL7Z4JEUGG33JEKGUDXKATBJANCNFSM6AAAAAAY5KCSQU
.
You are receiving this because you authored the thread.Message ID:
@.***>


cli not working

after running this command
sutando init

i get this error
/home/.nvm/versions/node/v20.10.0/bin/sutando: line 1: syntax error near unexpected token `(' 'home/.nvm/versions/node/v20.10.0/bin/sutando: line 1: `const { program } = require('commander');

Thank you for your feedback.
I have fixed the issue and released a new version, 1.1.1. Please update to the latest version and try running the command again.
If you encounter any further issues after updating, please let me know.

yes it works fine
but i am using es6 import method and typescript

#7


Consider running user-provided exception handler even when response has started

Created from https://github.com/aspnet/AspNetCore/issues/12301
If the response has already started, the Exception Handler middleware exits early and doesn't write a response. This makes total sense. However, it also doesn't call the user-provided handler (if any) and it doesn't signal the diagnostic source. This makes diagnosing issues more difficult. We should consider refactoring this.

The user provided handler is also intended to generate a response, so running that seems problematic. Signaling the diagnostic source makes sense though.

The user provided handler is also intended to generate a response

I don't know that I'd agree. The handler is a RequestDelegate yes, but that's not inappropriate here. There's nothing that expects the handler to generate a response here. It's just giving it the full context (the request) and the opportunity to generate responses as necessary. The handler can just as easily check if the response has started.
I would agree that this could be a behavior break though since the handler may not have been expecting to run when the response has already started. We can talk further about how we want to do that (or if we want to call the handler at all). Signalling the diagnostic source definitely seems like a no-brainer here :).

@anurse Is that something I as a consumer of the framework can hook in to and have a block of code provide a response to the request with?
No matter what we need to be able to provide some sort of response that explains either that the request was somehow not valid or that the server was unable to handle it for what ever reason.

No matter what we need to be able to provide some sort of response that explains either that the request was somehow not valid or that the server was unable to handle it for what ever reason.

If the response has already started it's not possible for you to provide a different response. You'd be writing in the middle of the existing response and corrupting the HTML, JSON, etc.. If you want to be able to replace a response then you have to fully buffer it to prevent it from being sent to the client. This adds a lot of overhead so we don't do it by default.

Yeah, I agree with @Tratcher here. This middleware is not designed to allow you to produce an error response after the main response has already started. If you want to be able to do that, you'd need to build a new middleware that buffers the response in memory and then dumps that buffer only if there is no error executing the rest of the pipeline.
What we are discussing here (and still not at all committed to) is the idea of allowing you to run code in the handler even after the response has started. You still would not be able to write a new response or affect the status code if the response has already started, but you could send the error to a diagnostic system of some kind (for example).

Yeh that's not unreasonable at all @Tratcher .
I found a scenario where I could setup an OData controller (I know OData is not core aspnet but bear with me) ... in the business logic for the action I could construct an entity and not populate it correctly so the entity was not valid.
EF for example would fail to put it in to the DB with a SQL exception.
The OData framework seems to validate properties as it serializes them resulting in the response failing during serialization so the response is literally a half backed blob of JSON.
The key issue I had there of course is that the framework just stopped processing at that point.
Being able to hook on to any part of the request lifecycle as a global catch all and at least log the true exception information is key to helping devs treat the framework right.
I do also accept that some scenarios are just simply not resolvable (like my OData one) in the ideal manner.
@anurse It sounds like you're proposal is much like what I would expect with some basic ability.
So a "pseudo code" concept (with little detail of course) of what I think the framework should do as core behaviour ...
MiddlewareObject stack;
try
{
     stack = BuildMiddlewareStack();
     stack.ExecuteOn(request);
}
catch(Exception ex)
{
      CallDefaultHandler(ex);
      if(defined) { CallUserHandler(ex); }
}
finally
{
     Cleanup(stack, request);
}

... I'm sure it's way more complex than that but this at the root level ensures that "worst case" the user still gets notified when the framework simply can't handle our garbage ;)

and at least log the true exception information

The framework should already be logging the exception at the middleware or server level, even if it can't produce a response.

The bulk of logging that has any value seems to be dropped in to the VS console ... is that what you mean by the "middleware or server level"?
I'm thinking take that VS console stuff too and be able to handle that with the global exception handler.
I'm already doing this ...
app.UseExceptionHandler(errorApp =>
{
    errorApp.Run(async context =>
    {
        var ex = context.Features.Get<IExceptionHandlerPathFeature>();
        log.Error(ex.Error.Message + "\n" + ex.Error.StackTrace);
          // code clipped out here for berevity
        var innerEx = ex.Error.InnerException;
        while (innerEx != null)
        {
            log.Error(ex.Error.Message + "\n" + ex.Error.StackTrace);
            innerEx = innerEx.InnerException;
        }
    });
});

... but I've hit scenarios where the output window in VS has information that would really help and doesn't get passed to this handler.
It would be really cool if this middleware also got all the info passed to the VS console window ... maybe if that is the case though put some flag / logging level info  in the errorApp object as it might produce a lot of noise for some scenarios that people wouldn't want.
I think @anurse was talking to me about this in another ticket actually. ah yeh, it's #12301 linked above. That would probably solve both of these tickets at least in part.

The bulk of logging that has any value seems to be dropped in to the VS console ... is that what you mean by the "middleware or server level"?

It's logged to the ASP.NET Core logging system. You can plug in a number of outputs, including custom ones.

See https://docs.microsoft.com/en-us/aspnet/core/fundamentals/logging/?view=aspnetcore-3.0 for more details on logging.

I use Log4Net as my logging framework (legacy reasons basically) ... This looks really good though ... I ma have to migrate to it and see if it's any better (anything that reduces further external dependencies can't be that bad right).

The advantage to integrating with the built-in framework is that the entire ASP.NET Core framework logs to that system, at a variety of detail levels. There are third-party providers than integration our system with other logging frameworks, more detail on that in the docs. I don't see log4net in that list, but a quick google search for log4net microsoft.extensions (The logging framework is in the Microsoft.Extensions.Logging namespace) does yield some results (though I don't know how well those integrations work). You can always integrate the two systems and then continue to log directly to log4net in your app, while also having ILogger messages go through to log4net.


File Transfer sessions fail to open (continuously prompt for credentials) V.<IP_ADDRESS>

Describe the bug
On connecting via "File Transfer Option" it request for login credentials over and over again...
To Reproduce
Steps to reproduce the behavior:

Go to 'Right Click -> File Transfer'
It Request login credentials over and over again...

Expected behavior
It should connect right away.
Additional notes
IT WORKS ON PREVIOUS VERSION V.<IP_ADDRESS>
Not similar to #717

Confirmed. It asks always for passwords.

I never used <IP_ADDRESS> so cannot confirm that this is a regression, but I also experience this, visibly with any session.
Is anyone successfully using file transfer on <IP_ADDRESS>?
Thank you for reporting this @CollapsedMetal . Would you mind retitling this ticket to something like "File Transfer sessions fail to open (continuously prompt for credentials)"?

I never used <IP_ADDRESS> so cannot confirm that this is a regression, but I also experience this, visibly with any session.
Is anyone successfully using file transfer on <IP_ADDRESS>?
Thank you for reporting this @CollapsedMetal . Would you mind retitling this ticket to something like "File Transfer sessions fail to open (continuously prompt for credentials)"?

I never used File Transfer. But if I understand correctly what it offers, an alternative is to use WinSCP, which is somewhat integrated in SuperPuTTY.
WinSCP is a free software application which needs to be installed separately, and SuperPuTTY needs to have its path configured.

I never used File Transfer. But if I understand correctly what it offers, an alternative is to use WinSCP, which is somewhat integrated in SuperPuTTY.
WinSCP is a free software application which needs to be installed separately, and SuperPuTTY needs to have its path configured.

@Chealer issue title changed

@Chealer issue title changed

I am using the v1.5 (available only x32) and I have the same problem.
The location of putty.exe is at putty x64 (same with pscp.exe location), if this is important

Describe the bug On connecting via "File Transfer Option" it request for login credentials over and over again...
To Reproduce Steps to reproduce the behavior:

Go to 'Right Click -> File Transfer'
It Request login credentials over and over again...

Expected behavior It should connect right away.
Additional notes IT WORKS ON PREVIOUS VERSION V.<IP_ADDRESS>
Not similar to #717

select "Options-->GUI-->Security-->Allow plain text passwords on putty command line", in this way, the file transfer can be successfully connected.(It's translated by software, haha)


Missing Arguments for useTrackEvent in AppInsights Micorsoft Docs for React Plugin

Is your feature request related to a problem? Please describe.
useTrackEvent hook is missing the arguments eventData, and skipFirstRun in Microsoft docs for AppInsights React plugin docs.

The useTrackEvent Hook is used to track any custom event that an application may need to track, such as a button click or other API call. It takes two arguments, the first is the Application Insights instance (which can be obtained from the useAppInsightsContext Hook), and a name for the event.

so it actually takes four arguments, three are required, and one optional.
I was able to predict what those extra two arguments do as they are intuitive from the types but still not sure how the code will act with/out providing them.
Describe the solution you'd like
Update docs for useTrackEvent hook to include  eventData, and skipFirstRun arguments.
Thanks,

@rmarskell my bad, I have updated the wordings in the doc, but didnot update the snippet. Now its updated in this pull request


Request for a Windows-like search functionality

What feature or improvement do you think would benefit Files?
Hello Yair. I'm new to Github.
First of all, your app is absolutely fantastic! Microsoft should have done something like this when Windows 11 came out - I guess you've been told that tons of times already. I'll now come to the point of my request.
Whenever I type something into the Windows File Explorer search bar, it not only checks the names of the files within that folder, it also checks the contents of all the files.
I think it would be nice to be able to choose in the settings whether the search bar should only search for names or both names and contents.
I find this would be useful when you can't remember the file where the content you are looking for is located. I personally use it all the time, especially when I have to write something in a document and I know I have already written it or have written something similar somewhere, but can't remember exactly where.
I hope I didn't waste your time with an unfeasible request.
Have a good one!
Requirements

This proposal will accomplish X
This proposal will accomplish Y
This proposal will accomplish Z

Files Version
<IP_ADDRESS>
Windows Version
10.0.22621.675
Comments
No response

Thanks for the feedback. Files already uses the same syntax and system as Windows File Explorer so you can look up the filters you want to use. A UI for filtering is being tracked in https://github.com/files-community/Files/issues/4813. You can check there or click subscribe in the right sidebar for updates.

Thank you for the positive feedback, as Josh mentioned, you can set the flags using the advanced search syntax available in Windows.


js not working in IOS?

Framework7 version: 5.7.1
Platform and Target: iphone 7 iOS 13.4.1
It seems that the use of js in IOS does not work. On chrome it works fine.
Example: {{js "new Date(this.StartDate).getDate()"}} returns "19" fine on Chrome but NaN in IOS.
Please advise,
Kind regards,
Mario

It means that what you have in this.StartDate is in wrong or not parseable format by Safari


This will make Raw Accel 100% better.

Hello there,
so not everyone is a low level programmer and in fact using low level to make a UI isn't very smart.
Because people who know UI very good don't use low level programming.
I would rather make the driver in C/C++ and use a higher level for UI so more people can contribute.
UI is just to configure the driver and even if it eats up much more than needed (like electron apps) it doesn't matter as the tiny high performance driver written in C/C++ will be running all time.
There are many UI things missing, just for example the way of making the "curves" is just bad, I would rather have some UI where I can use some Bezier Curves and then move some points and have it how I want it. Set min and max value of my sens.
I would much prefer this UI
https://mouseacceleration.com/home.html
How I use accel is by lowering my sens, so my flicking is on 1.0 and it works amazing, but then I use like 0.85-0.9 for tracking.
Sadly as soon as I set a limit of 1.0 the curves don't work... so the only thing I use is the rotation.
Look at FanControl on github :) Very easy to use, nice UI and runs really good, a magnitude better than any Mouse driver.

The UI for Raw Accel is written in C# using windows forms, not c, c++, or any other "low level programming" (by which I'm guessing you mean not memory-managed): https://github.com/a1xd/rawaccel/blob/master/grapher/Form1.cs
If you want to start a curve at x and go to 1, set a cap or limit of 1/x and a sensitivity multiplier of x.
You, or anyone, is free to create proposed changes which add desired features. We've had a few folks do that already. The writer.dll is C# and contains the function call needed to pass values to the driver, so it can be used to write an entire new UI should you or anyone be inclined to do so.

How does the "values" look like? Is it JSON or what is it exactly?


Add UDT: Unlock Discount Token

[X] I understand that token listing is not required to use the Polygon Wallet with a token.

Please provide the following information for your token.
Token Name: Unlock Discount Token
Token Symbol: UDT
Decimals: 18
Polygon Address: 0xf7e78d9c4c74df889a83c8c8d6d05bf70ff75876
Ethereum Address: 0x90de74265a416e1393a450752175aed98fe11517
Logo URI: https://github.com/unlock-protocol/unlock/blob/master/design/brand/1808-Unlock-Identity_Unlock-LogoMark-Circle.svg
Project Name: Unlock Protocol
Project Summary: The Unlock Protocol is a protocol for memberships where members cards are NFT with an expiration date
Project Contact<EMAIL_ADDRESS>Project Website: https://unlock-protocol.com/
Mainnet/Testnet: Yes
Additonal Notes: NA

Mainnet/Testnet: Mainnet

Included in AllTokens v1.2.0.


SPOI-7087 #resolve #comment Add JDBC dimension output operator to Malhar

@ilooner please review and merge


Heap Sort with Java

I want to implement it. Please assign it to me.

I would like to contribute. Please assign @OrchiDorchi

Hello @OrchiDorchi, this issue already exists and has been assigned. We request you to look for an issue that interests you, or create a new on if it does not exist.
Have a good time contributing :)

@Asifa23 , this has already been assigned to you.


Lab2 luliang li

Fixes #268
Review of colleague's PR #182
Changes proposed in this PR:

implemented bubble/insert sort
implemented test cases

Codecov Report

Merging #176 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master   #176   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           2      3    +1
  Lines          18     38   +20
=====================================
+ Hits           18     38   +20

Impacted Files
Coverage Δ

02_bubble/luliangLi/main.go
100% <100%> (ø)

Continue to review full report at Codecov.

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

Apologies for the confusion, You need to produce a copy of the original array. As it is, your solution modifies the original array. Can you please change this and write a test to ensure the original array is not modified.

The go-course is now closed. Thank you very much for participating.


[BUG] Multivariate_normal.rst has syntax issues

The syntax issues all of which I have not been able to pinpoint yet have been causing pdf compilation errors.
Someone with good experience of writing in rst will help expedite finding the fix.

one identified section causing this error is https://github.com/QuantEcon/lecture-python/blob/master/source/rst/multivariate_normal.rst#one-dimensional-intelligence-iq . Text up to this section is okay.
sections after this section still need to be verified

another is https://github.com/QuantEcon/lecture-python/blob/master/source/rst/multivariate_normal.rst#another-representation

Finally, the reason for this was that the lecture had italicized math symbols, one example:
n = 50
𝜇𝜃, 𝜎𝜃, 𝜎y = 100., 10., 10.

𝜇_IQ, Σ_IQ, D_IQ = construct_moments_IQ(n, 𝜇𝜃, 𝜎𝜃, 𝜎y)
μ_IQ, Σ_IQ, D_IQ

which needed to be converted to:
 n = 50
 μθ, σθ, σy = 100., 10., 10.

 μ_IQ, Σ_IQ, D_IQ = construct_moments_IQ(n, μθ, σθ, σy)
 μ_IQ, Σ_IQ, D_IQ

And newline characters in math directives, one example:
.. math::

   \Sigma_y = E Y Y^{\prime} = \Lambda \Lambda^{\prime} + D \\
   E Y f^{\prime} = \Lambda \\
   E f Y^{\prime} = \Lambda^{\prime}

which needed to be converted to:
.. math::
   \begin{aligned}
   \Sigma_y = E Y Y^{\prime} = \Lambda \Lambda^{\prime} + D \\
   E Y f^{\prime} = \Lambda \\
   E f Y^{\prime} = \Lambda^{\prime}
   \end{aligned}


Error when start app with Angular 11 with oauth2-oidc

I have an error when I started my angular using oauth2-oidc, below the error and warning in my console
Error: ./src/app/app.module.ts
Module not found: Error: Can't resolve 'node:process' in 'C:\Users\Desktop\anager-version-ui\src\app'

** Angular Live Development Server is listening on localhost:61909, open your browser on http://localhost:61909/ **
Error from chokidar (C:\): Error: EBUSY: resource busy or locked, lstat 'C:\hiberfil.sys'
Error from chokidar (C:\): Error: EBUSY: resource busy or locked, lstat 'C:\pagefile.sys'
Error from chokidar (C:\): Error: EBUSY: resource busy or locked, lstat 'C:\swapfile.sys'

But in my app.module, when I put in comment the provider AuthConfig, I have no errors
providers: [
    { provide: HTTP_INTERCEPTORS, useClass: AuthInterceptor,deps: [ConfigService], multi: true },
    {
      provide: APP_INITIALIZER,
      useFactory: configFactory,
      deps: [ConfigService],
      multi: true
    },
    { provide: LocationStrategy, useClass: HashLocationStrategy },
    **{ provide: AuthConfig, useValue: config },**
    AuthorizationService,
]

What I missed in my configuration ?

I think this is often/typically caused on Windows with VSCode and Angular. VSCode might do some magic (e.g. auto import upon intellisense auto completion) causing this to happen. Rebooting vscode, angular processes, etc. typically solves the issue for me.
Not sure if the community for this library can do much for you.

Hi @jeroenheijmans  .. Yes it is this thank you so much .. the auto import failed .. Thank so much ..


Feature/sort obj table

Will fix #83 when complete.

First version uses a bubble sort, with DOM updates mid-stream.  This performs about as well one might expect.  Needs a better sort, and separate sort from DOM manipulation.


Automatic PR from RestAPI-PR1387

Generated from RestAPI PR: https://github.com/Azure/azure-rest-api-specs/pull/1387

Codecov Report

Merging #1292 into master will increase coverage by <.01%.
The diff coverage is 40%.

@@            Coverage Diff             @@
##           master    #1292      +/-   ##
==========================================
+ Coverage   55.83%   55.83%   +<.01%
==========================================
  Files        2622     2622
  Lines       70003    70004       +1
==========================================
+ Hits        39086    39087       +1
  Misses      30917    30917

Impacted Files
Coverage Δ

...ure/mgmt/sql/models/transparent_data_encryption.py
71.42% <ø> (ø)
:arrow_up:

...tions/server_azure_ad_administrators_operations.py
15.31% <ø> (ø)
:arrow_up:

...e/mgmt/sql/models/database_blob_auditing_policy.py
35.71% <100%> (ø)
:arrow_up:

.../azure/mgmt/sql/operations/databases_operations.py
19.08% <25%> (+0.1%)
:arrow_up:

Continue to review full report at Codecov.

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

Contains https://github.com/Azure/azure-sdk-for-python/pull/1290 , #1287
0.8.0

Breaking change: Database blob auditing policy state is required
Bug fix: fixed KeyError in server_azure_ad_administrators_operations.get

This should be good to publish

In https://github.com/Azure/azure-sdk-for-python/pull/1413


This application cannot tree shake icons fonts.

/Users/tva/development/flutter/bin/flutter --no-color build apk

Running Gradle task 'assembleRelease'...
This application cannot tree shake icons fonts. It has non-constant instances of IconData at the following locations:
  - file:///Users/tva/.pub-cache/hosted/pub.dev/json_theme-6.2.3/lib/src/codec/theme_decoder.dart:5260:16
Target aot_android_asset_bundle failed: Exception: Avoid non-constant invocations of IconData or try to build again with --no-tree-shake-icons.

/Users/tva/development/flutter/bin/flutter doctor --verbose
[✓] Flutter (Channel stable, 3.13.0, on macOS 13.4.1 22F770820d darwin-x64, locale en-BR)
    • Flutter version 3.13.0 on channel stable at /Users/tva/development/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision efbf63d9c6 (9 days ago), 2023-08-15 21:05:06 -0500
    • Engine revision 1ac611c64e
    • Dart version 3.1.0
    • DevTools version 2.25.0

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.2)
    • Android SDK at /Users/tva/Library/Android/sdk
    • Platform android-33, build-tools 33.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E300c
    • CocoaPods version 1.12.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)

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

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 13.4.1 22F770820d darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 116.0.5845.96

[✓] Network resources
    • All expected network resources are available.

• No issues found!

This isn't an issue per se.  It's actually documented in the README.md:

Important Note
Because this library allows for icons to be dynamically referenced, Flutter cannot tree shake icons because it has no way of knowing what icons may or may not be referenced. If you include this library, then you will need to add the --no-tree-shake-icons flag to your Flutter build or you will get an error stating that Flutter cannot tree shake the icons.


Next.js Practice App

To create an app that presents the registries projects in a neat format.
To practice the team's technologies and get up to speed.
Using: React, Node, Next
Talk to Oamar for help
Acceptance Criteria

[ ] Hit the  registry api and pull a list of all the current projects
[ ] Present those projects in an easy to read format

Private Zenhub Image


How to use an optimizer in TFSharp?

How can we use an optimizer, such as Gradient Descent or Adam optimizer?

I think it should be like this: https://stackoverflow.com/questions/49291781/how-to-train-model-with-tensorflowsharp