UNC %APPDATA% paths mangled in VS Code 1.81.1

Does this issue occur when all extensions are disabled?:   We use folder redirection to map the user's Windows profile data from a network share.  The path to %APPDATA% is a UNC path, similar to "\\domain.local\profiles\username".  After upgrading to version 1.81.1, users are no longer able to launch VS Code due to a new bug introduced that affects how it handles UNC paths.
It appears that, in certain cases, the UNC path to %APPDATA% is being mangled by VS Code to use C:\ instead of \\domain.local.  For security purposes, users do not have permissions to write directly to C:\, nor can they run VS Code as Administrator.
After some testing, we found that only one file is being incorrectly written to C:\, which is C:\profiles\username\AppData\Roaming\Code\User\globalStorage\state.vscdb.  This file should be written to the proper UNC path, \\domain.local\profiles\username\AppData\Roaming\Code\User\globalStorage\state.vscdb, where all of the other VS Code files already exist.
As a result of this bug, when launching VS Code, all users receive the following error message:
Unable to write program user data.
A system error occurred.  (EPERM: operation not permitted, mkdir 'C:\profiles')
Please make sure the following directories are writeable:
\\domain.local\profiles\username\AppData\Roaming\Code
C:\Users\username.vscode\extensions
All of the paths mentioned in the error message are writable by the users.  However, you can see the issue is related to VS Code trying to write to C:\profiles instead of the proper UNC path: \\domain.local\profiles.  Which is not even on the list of paths that VS Code claims to be interested in.
This is not an issue with configuration, permissions, folder redirection, or Windows Server itself.   Version 1.79.2 of VS Code handled UNC paths appropriately.

/duplicate https://github.com/microsoft/vscode/issues/191374

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


Added perfect-numbers exercise.

Let me know if there are any problems.

Thanks @cpaulbond and thanks so much for submitting this. I'm sorry it has taken almost two months to see it (I've been figuring out how to get a better handle on the email notifications in Exercism—it's a bit of a battle).
@canweriotnow do you have a few minutes to review this?

Merged into master manually, closing.


Add automatic/sane way to update extensions

Currently there is no Update option in the extension's menu:

Why is this needed:
Manual downloading & copy-pasting files from packages is bad UX, especially when you have a lot of extensions.

Related https://github.com/lensapp/lens/issues/3579

FYI @nevalla I think has mostly implemented this (at least for NPM installed packages) I think

This is something that is as well important for bundled extensions. I added some higher priority to this issue.

@msa0311 Do you mean forced auto update? Also the linked issue is about npm installed extensions. Bundled doesn't fall under that category.


Separate code

Everything is currently in index.js, should probably separate some out into /src

Done in 55e771b


Display Invalid Trace message

Trace Viewer should show an invalid trace message for all invalid traces.

Currently, it's just empty.

I don't understand the issue description. When a file that is not a recognized trace format is opened as a trace, the Trace Explorer side panel is empty?
Could you add more precision to the issue description and/or add a GIF that shows the issue?

When a file that is not a recognized trace format is opened as a trace, the Trace Explorer side panel is empty?

Yes, this is what I meant. We should display an error message to the user.  Attached a gif. I opened an xml file(invalid trace). The user should be shown with an error message. Right now, we do not show anything.
Any thoughts?

I agree that the there should be a feedback to the user about that.
For an trace type that is unknown by the trace server, it return status 501 (NOT_IMPLEMENTED). I realized when trying it, that there is an uncaught exception created because the tsp-typescript-client can't handle the error case:
Uncaught SyntaxError: Unexpected token T in JSON at position 0
message:'Unexpected token T in JSON at position 0'
stack:'SyntaxError: Unexpected token T in JSON at position 0'

Looking briefly at the tsp-typescript-client other commands have similar issues when an error code is return.

Is it possible for a user to try and open any file type as a trace? I'm wondering if it's worth mentioning in the warning message that the file could not even be a trace file (ex. table.csv, randomtext.txt, executable.bin). Or do typical users know enough about what a trace is that that's an extreme corner case?


#232 Allow for Custom JavaScript files

Resolve #232
Allow for Custom JavaScript files

Thank you Linh! 🎉


Relation extraction not working

Hi @monk1337,
I guess something in text_input parameter of nlp_prompter changed after the recent updates you made.
Check the attached screenshot, which I tried on your few shot example.

Hi, You have to pass the template and model in Prompter and in fit function you provide the sentence.
example
prompter = Prompter(model=model, template = 'relation_extraction.jinja')
output = prompter.fit(text_input = sentence, domain = "medical", labels = None)

Thanks @monk1337, this works.
Any plans to include langchain's get_openai_callback() method to get the token and cost information.

doesn't the raw openai call provide that anyways?


torchvision CPU nightly wheels are stored in the wrong index

The nightlies are in https://download.pytorch.org/whl/nightly/cpu/torchvision/torchvision rather than https://download.pytorch.org/whl/nightly/cpu/torchvision/.
Installing with
pip install --pre torchvision --extra-index-url https://download.pytorch.org/whl/nightly/cpu/

will pull the stable wheel from PyPI.
cc @malfet @seemethere

Is this related to https://github.com/pytorch/vision/issues/6099?
Some time today we were no longer able to install nightly torch with torchvision. torchvision will instead force downloading the stable torch package:
(base) ray@46b3d93b9717:/ludwig$ pip install --pre torch torchvision --extra-index-url https://download.pytorch.org/whl/nightly/cpu
Looking in indexes: https://pypi.org/simple, https://download.pytorch.org/whl/nightly/cpu
Collecting torch
  Downloading https://download.pytorch.org/whl/nightly/cpu/torch-1.13.0.dev20220527%2Bcpu-cp38-cp38-linux_x86_64.whl (189.3 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 189.3/189.3 MB 7.5 MB/s eta 0:00:00
Collecting torchvision
  Downloading torchvision-0.12.0-cp38-cp38-manylinux1_x86_64.whl (21.0 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 21.0/21.0 MB 16.9 MB/s eta 0:00:00
Requirement already satisfied: typing-extensions in /home/ray/anaconda3/lib/python3.8/site-packages (from torch) (4.2.0)
Collecting torch
  Downloading torch-1.11.0-cp38-cp38-manylinux1_x86_64.whl (750.6 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 750.6/750.6 MB 910.5 kB/s eta 0:00:00

For now our workaround is to pin to a specific torchvision wheel by its URL, but this obviously won't be sustainable as we would need to keep manually updating it for each nightly. So would be great to get this fixed soon, I suspect other projects testing against nightly torch are similarly blocked.

Is this related to https://github.com/pytorch/vision/issues/6099?

Nope. The cause of #6099 is a wrong installation command. Your command is fine, but our index is broken.
In the mean time, you can also install with
pip install --pre torch torchvision -f https://download.pytorch.org/whl/nightly/torch_nightly.html

This is fine for your use case, but be aware that you can only install PyTorch wheels with this, whereas the --extra-index-url can also pull from PyPI if you require anything else.

Fixed it by deleting spurious /whl/cpu/torchvision/ packages, but I'm still not sure why it caused this weird nesting problem

Thanks for the workaround @pmeier.
@malfet, thanks for looking into this, though I am still unable to use --extra-index-url at this time.

I also can confirm it worked on my systems. Perhaps it takes a bit of time until the change propagates in all regions?

Thanks @malfet @datumbox, looks like it's actually coming from torchtext now (we install it together with torch and torchvision). I'll follow-up with them, likely it's a similar issue as the one here.  Thanks for the quick fix!

@tgaddair hmm, can you please elaborate, what problem are you seeing with torchtext? (As I were able to install the full pack of vision, audio and text on my linux devbox)

Hey @malfet, I spoke too soon, that turned out to be my issue (already had torchtext in the env). I haven't put together a good repo for what I'm seeing in CI locally yet, but you can see the problem here when trying to install torchvision nightly with python 3.9:
https://github.com/ludwig-ai/ludwig/runs/6657822665?check_suite_focus=true#step:7:79
I'll also try re-running to see if it was a replication issue as @datumbox suggested.

Okay, seems to be working again, so I'm guessing it was as @datumbox suggested. Thanks everyone! Apologies for the back and forth.

@tgaddair  Thanks for confirming.
@malfet Thanks for the help fixing the issue.

Hey @datumbox @malfet, looks like this issue has resurfaced again today:
https://github.com/ludwig-ai/ludwig/runs/6794939594?check_suite_focus=true#step:7:78

This issue is still popping up from time to time. We tried working around it using @pmeier's suggestion to change:
pip install --pre torch torchtext torchvision torchaudio --extra-index-url https://download.pytorch.org/whl/nightly/torch_nightly.html

to:
pip install --pre torch torchtext torchvision torchaudio -f https://download.pytorch.org/whl/nightly/torch_nightly.html

However, this results in installing a ROCM enabled version of torch that fails at runtime and is incompatible with some of our other dependencies.
torch                     1.13.0.dev20220614+rocm5.0
torchaudio                0.13.0.dev20220614+rocm5.1.1
torchtext                 0.14.0.dev20220614
torchvision               0.14.0.dev20220614+rocm5.0

Quick update on this, we managed to find a workaround by removing torch and torchvision from the project requirements after installing the nightlies. For whatever reason having those packages in the requirements resulted in the nightly version being overwritten and reverting back to stable. So we are unblocked on this for now.

@tgaddair do you mind opening a separate issue on problems you continue to experience, as it does not seem to be related to the original issue reported here.
Removed torchvision/torchvision/index.html from s3, so the the folder should no longer exist:
aws s3 ls s3://pytorch/whl/nightly/cpu/torchvision/torchvision


Can a link to remove item from selected list be added

Hi,
I'd like to add a link to each rendered item to remove it from the selected list.  Is there a way to do this?
Thank you!

Not sure if I fully understood your question, but perhaps the remove_button plugin of Selectize can do what you want.

that's exactly what I was looking for - thank you!

Then you should close this issue...


Fix max output size to be passed down to rendering

For #8010
This is a regression caused by the native editor refactoring.

Codecov Report

Merging #8017 into master will decrease coverage by 0.37%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #8017      +/-   ##
==========================================
- Coverage   59.26%   58.88%   -0.38%
==========================================
  Files         499      499
  Lines       22413    22413
  Branches     3597     3592       -5
==========================================
- Hits        13284    13199      -85
- Misses       8300     8376      +76
- Partials      829      838       +9

Impacted Files
Coverage Δ

src/client/testing/serviceRegistry.ts
48.64% <0%> (-46.85%)
:arrow_down:

src/client/testing/codeLenses/main.ts
44.44% <0%> (-33.34%)
:arrow_down:

...t/datascience/jupyter/jupyterDataRateLimitError.ts
66.66% <0%> (-33.34%)
:arrow_down:

src/client/formatters/serviceRegistry.ts
75% <0%> (-25%)
:arrow_down:

src/client/common/utils/platform.ts
64.7% <0%> (-23.53%)
:arrow_down:

src/client/common/utils/icons.ts
83.33% <0%> (-16.67%)
:arrow_down:

src/client/api.ts
78.57% <0%> (-14.29%)
:arrow_down:

src/client/datascience/cellFactory.ts
75.86% <0%> (-13.8%)
:arrow_down:

src/client/providers/providerUtilities.ts
83.33% <0%> (-5.56%)
:arrow_down:

src/client/common/platform/pathUtils.ts
66.66% <0%> (-5.56%)
:arrow_down:

... and 8 more

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 5c1d2bc...c4de520. Read the comment docs.


Point Sideloading URL to the Sideloading Heading

Point Sideloading URL to the Sideloading Heading. When clicking the sideloading method it feels like it's going to the wrong page.
#1281
https://github.com/OfficeDev/office-js-docs-pr/issues/1281
I'm not familiar to how you are modifying the URLs on your end, mainly the suffix/extension .md, but the change that works for me externally: https://docs.microsoft.com/en-us/office/dev/add-ins/testing/create-a-network-shared-folder-catalog-for-task-pane-and-content-add-ins.md#sideload-your-add-in

Docs Build status updates of commit 00dd74f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/publish/publish.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 789ce43:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/publish/publish.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@David-Tennant, thanks for making this update. I'll merge this to master and it should be published soon.
Thanks.


Add meta.origin to our saveAction if NODE_ENV != production

Fixes #100

@mathieudutour Do you have a opinion about this? You're quite active on this project and I'd love to hear  your feedback :smile:

Yeah it might be useful. This is a breaking change though. And you need to add
"browserify": {
    "transform": [
      "envify"
    ]
  },

to the package.json

@mathieudutour I've added envify (the faster variant named loose-envify - the same as redux is using).
But why should this be a breaking change? It does not manipulate the actual payload of the save action. I though of this as a minor release

Because of someone is already using the origin meta, it breaks.
For it to be non-breaking, it needs to be opt-in
On 29 Feb 2016, at 08:51, Michael Contento<EMAIL_ADDRESS>wrote:
@mathieudutour https://github.com/mathieudutour I've added envify (the
faster variant named loose-envify https://github.com/zertosh/loose-envify

the same as redux
https://github.com/reactjs/redux/blob/master/package.json#L116-L120 is
using).

But why should this be a breaking change? It does not manipulate the actual
payload of the save action. I though of this as a minor release
—
Reply to this email directly or view it on GitHub
https://github.com/michaelcontento/redux-storage/pull/102#issuecomment-190107501
.

Oh! You mean if someone hijacks and mutates the saveAction while it is been processed? Good point! Although this sounds more like a redux anti-pattern to me :smile:
But as this is a good point, it will be part of the upcoming 4.0.0 together with the fromJS changes :+1:

NOTE: Branch rebased as browserify-envify is already required in the current master

No not necessary. I have some middlewares checking for the existence of a
meta key to trigger a side effect. Someone could have one of those
listening to the origin meta key
On 29 Feb 2016, at 09:16, Michael Contento<EMAIL_ADDRESS>wrote:
Oh! You mean if someone hijacks and mutates the saveAction while it is been
processed? Good point! Although this sounds more like a redux anti-pattern
to me [image: :smile:]
But as this is a good point, it will be part of the upcoming 4.0.0 together
with the fromJS changes [image: :+1:]
—
Reply to this email directly or view it on GitHub
https://github.com/michaelcontento/redux-storage/pull/102#issuecomment-190115959
.


Fixing build failures of Windows Server 2016

I'll merge this as soon as the bump bot is playing ball


Member

Please aktif My I'd...
N my pass

If this is supposed to be a real issue (not filed in error), please give it a proper name and description. We'll happily take a look.

#682

A38J015485


2.2.0 changes break bootstrap5 css of ts-control items

Changes made in 2.2.0 break the css of ts-control items and its padding style
2.1.0
https://jsfiddle.net/mArtinko5MB/n3jhxy0L/2/
2.2.0
https://jsfiddle.net/mArtinko5MB/cj9xvuqp/1/
the code that seems to break this is

from https://github.com/orchidjs/tom-select/commit/a8ff934f236ec0234b4d0eab773c03451e44c5a4
Also placeholder is wider, no idea what is problem here but it breaks the meta
2.1.0
https://jsfiddle.net/mArtinko5MB/xk3Lngvz/2/
2.2.0
https://jsfiddle.net/mArtinko5MB/d2pb6cqw/2/
Forced to continue to use 2.1.0 for now, cheers

This is still a thing in latest release ...


Modernize LaTeX code of the manual

It appeared difficult for me to build the manual from https://github.com/SuperElastix/elastix/tree/5.0.1/dox/manual using the latest version of MikTex. Within TeXWorks, I got the following error:

This is pdfTeX, Version 3.14159265-2.6-1.40.21 (MiKTeX 20.7)
entering extended mode
(D:/X/src/elx/5.0.1/dox/manual/manual.tex
LaTeX2e <2020-02-02> patch level 5
L3 programming layer <2020-09-03>
("C:\Program Files\MiKTeX\tex/latex/base\report.cls"

...

! Package hyperref Error: Wrong DVI mode driver option `dvips',
(hyperref)                because pdfTeX or LuaTeX is running in PDF mode.

It seems that "manual.tex" makes use of some obsolete LaTeX packages, and errors like this could be solved by modernizing the LaTeX code of the manual.
Otherwise it would be helpful to document exactly how to successfully generate a new "manual.pdf".

We are now in the process of moving the LaTeX processing to www.overleaf.com which does not have this error.
So this issue does not have priority anymore. Discussed with @ViktorvdValk and @mstaring


configuring webpack alias in storybook, it doesn't to work in storybook test

Hi,
When I use it after configuring webpack alias in storybook, it doesn't seem to work in storybook test.

spec
next 12.2.5
storybook 6.5

I read the readme and set it up as below.
https://github.com/scottrippey/next-router-mock#storybook-configuration
https://github.com/scottrippey/next-router-mock#storybook-example
I'm using storybook interaction test as my testing tool
My code
// component.tsx
import qs from 'qs';
import Link from 'next/link';
import { useRouter } from 'next/router';

const WithNextRouter = () => {
  const router = useRouter();
  const { pathname, asPath, query } = router;
  const onClickQueryButton = () =>
    router.push('/test?query=1', null, { shallow: true });

  return (

      <Link href="/test">home</Link>

      <Link href="/test/abc">a</Link>

      <button type="button" onClick={onClickQueryButton}>
        query
      </button>

pathname: {pathname}

asPath: {asPath}

query: {qs.stringify(query)}

  );
};

export default WithNextRouter;

// component.stories.tsx
export default {
  title: 'With Next Router',
  component: WithNextRouterComponent,
  parameters: {
    url: {
      pathname: '/test',
      asPath: '/test',
      query: {
        param1: 1,
        param2: 2,
      },
    },
    jsx: {
      skip: 1,
      displayName: () => 'WithNextRouterComponent',
    },
  },
} as ComponentMeta<typeof WithNextRouterComponent>;

export const WithNextRouter: ComponentStory<
  typeof WithNextRouterComponent
> = () => <WithNextRouterComponent />;

WithNextRouter.play = async ({
  canvasElement,
}) => {
  const canvas = within(canvasElement);

  await step('show pathname', async () => {
    // Given
    const pathnameElement = canvas.getByText(/pathname:/);

    // Then
    await expect(pathnameElement).toBeInTheDocument();
    await expect(pathnameElement.textContent).toBe('pathname: /test');
  });

  await step('click on abc link displays the relevant information.', async () => {
    // Given
    const linkElement = canvas.getByRole('link', { name: /abc/i });
    const pathnameElement = canvas.getByText(/pathname:/);
    const asPathElement = canvas.getByText(/asPath:/);

    // When
    await userEvent.click(linkElement);

    // Then
    await expect(pathnameElement.textContent).toBe(
      'pathname: /test/abc'
    );
    await expect(asPathElement.textContent).toBe('asPath: /test/abc');
  });
}

First in ‘show pathname’, the test is conducted by setting the initial url parameters.
The expected result is ‘/test’, but the actual result is ‘/’.
Second, ‘click on abc link displays the relevant information.’ is a test when a click event occurs through next/link.
The expected result is '/test/abc', but the actual result is '/'.
If you use the storybook test tool after configuring webpack, it will not work.

If I don't configure webpack alias, it works normally. Do you know why?
Any help would be appreciated.

I haven't used Storybook Test, so it's kinda hard to guess what's happening here, but I'll try...
I'm not really sure how the parameters: { url: {...} } stuff works ... I'm guessing that Storybook Test will set up its own mock of window.location and window.history with those parameters, so that the story can interact with a mocked URL?
The way next-router-mock works, though, is it keeps all location information in its own memory, completely independent of window.location / window.history.  So, those Storybook parameters should have no effect on next-router-mock or next/router or next/link.
Instead, you need to use the <MemoryRouterProvider> component to wrap all your stories.  This provides a MemoryRouter to next/router and next/link components, and has parameters like url if you'd like to customize the initial url.
The easiest way to wrap a story is as a decorator, which can be done per-component or per-story or globally.
Details are here: https://github.com/scottrippey/next-router-mock#example-nextlink-with-storybook
I hope this works!  I'm actually eager to use Storybook Test some day, it sounds really awesome!

Sorry for the delay, I'll make a quick one to share soon.

It's been a long time.
I finally got around to making codesandbox..!
📎 codesandbox link
But this time I'm using Storybook 7.0 and I had an issue. (The first issue I talked about is in 6.5)
When I import it, I get an error that there is no URL.
Module not found: Error: Can't resolve 'url' in '/project/home/leedoopal_1/workspace/node_modules/next-router-mock/dist'

Is there any possibility to resolve this?
If you uncomment the commented part of .storybook/preview.js, you can see the error.

I see the error, yes!  This can be fixed, I'll have to update the dependency from import 'url' to simply using the global URL object.
As a workaround for now, you can probably npm install --save-dev url to polyfill that module.

Thanks! That solved that problem for me.
When I import the next-router-mock from the .storybook/preview.js file, I get the following error.
Can you help me with this?
ERROR in ./node_modules/next-router-mock/dist/MemoryRouterProvider/next-10.js 6:25-76
Module not found: Error: Can't resolve 'next/dist/next-server/lib/router-context' in '/project/home/workspace/node_modules/next-router-mock/dist/MemoryRouterProvider'

The codesandbox link is the same as above.


[base][docs] Add demo for using the Button component as a link

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Summary 💡
Add a demo on the base's Button component page about how the component can be used with different Link components.
Examples 🌈
We could add demos for plain a, element, next's Link component etc.
Motivation 🔦
This is a very common use-case, it would be great if people have a reference when implementing it.

Is this an easy fix?


Small Updates to 3 Cloud SDK Tutorials

https://developers-qa.sap.com/tutorials/s4sdk-odata-service-cloud-foundry.html
https://developers-qa.sap.com/tutorials/s4sdk-scp-neo-sample-application.html
https://developers-qa.sap.com/tutorials/s4sdk-setup.html

I will merge, but I will immediately reformat the tutorials to conform to style and make the start of the tutorials clearer. Be in touch (offline) if you have any questions.


The results of lightgbm in linux and windows are inconsistent under the same version

Operating System:
linux: ubuntu 14.04
windows: windows2008 R2
Version：
2.0.4
Error Message:

Under the same configuration，the same trainset, the model of windows is different from linux
With two model to predict the same testset , There is a big difference between the results

"big difference?"
There are many possible reasons.
With different compilers, the result could be different. the main reason is the multi-threading and openmp executing order issues.
The python in the different platform also may cause the difference. you can exclude this reason by using CLI version.


How to calculate duty cycle of gpio-pin output value?

I execute code on rp2040js like that:
from machine import Pin, PWM;
from time import sleep_ms;
pin = Pin(1);
pwm = PWM(pin);
pwm.freq(1000);
pwm.duty_u16(6554);
sleep_ms(100);
pwm.deinit();

As I expected, the duty cycle of pwm should be 10%. But i don't konw how to verify the result that i expect. I try to addListener on  gpio-pin, then i receive a lot of output value of gpio-pin.Does there exist someway of calculating the real duty cycle of gpio-pin's output value?

Does there exist someway of calculating the real duty cycle of gpio-pin's output value?

In general, look at the simulation time whenever the pin goes high/low, and then derive duty cycle from that.
e.g. if the pin has been low for 10 ms, then high for 20 ms, then low for 10 ms, then high for 20 ms, the duty cycle would be the sum of the high times (20 + 20 = 40) divided by the total time (10+20+10+20=60), or 2/3 (66.66%).
You can see an example of doing it with code here: calculating LED brightness by measuring the duty cycle of PWM signal


bug: crashing when handling null-ls LSP results

Neovim version (nvim -v)
0.8.3 release
Operating system/version
fedora 37
Output of :AerialInfo
Aerial Info
Filetype: typescript
Configured backends:
lsp (supported) (attached)
Show symbols: Class, Constructor, Enum, Function, Interface, Module, Method, Struct, Constant, Variable
Describe the bug
when loading aerial (telescope plugin) on typescript files, with only LSP as enabled, in my setup, I have two LSP servers: null-ls (to get indentation and linters) and the typescript LS. Naturally, null-ls is ready faster.
When null-ls handles the request, it breaks aerial's fetch_symbols_sync... bizarrely, it doesn't return {1 = ...} as aerial expects, but {2= ...}.
That means that aerial crashes, and more precisely with this stack:
E5108: Error executing lua ...acker/start/aerial.nvim/lua/aerial/backends/lsp/init.lua:50: attempt to index a nil value
stack traceback:
	...acker/start/aerial.nvim/lua/aerial/backends/lsp/init.lua:50: in function 'fetch_symbols_sync'
	...r/start/aerial.nvim/lua/telescope/_extensions/aerial.lua:36: in function 'aerial'
	[string ":lua"]:1: in main chunk

Either adding a guard (if lsp_results[1] ~= nil then..), or a clause taking into account specifically the 2 fix it:
-    callbacks.handle_symbols(lsp_results[1].result, bufnr)
+    if lsp_results[1] == nil then
+      callbacks.handle_symbols(lsp_results[2].result, bufnr)
+    else
+      callbacks.handle_symbols(lsp_results[1].result, bufnr)
+    end

The second solution is a little better, because that means that the request actually works. The first solution makes it at least not crash, and then it works when the user retries once the typescript LS finishes loading.
However I hesitate to make a PR with the "better " fix because.. it doesn't exactly feel right...
Steps To Reproduce
unfortunately making a config to reproduce this and setting up a typescript test program, and so on seems really overkill...
but generally speaking i blame null-ls when working with the typescript LS, which is relatively slow to start (that's another issue.. on a small typescript project, it would probably be fast enough to start that the issue wouldn't be reproducible).
Expected Behavior
at least aerial shouldn't crash in this situation, at best it could know how to interpret the response it gets, weird as it is.
Minimal example file
No response
Minimal init.lua
sorry :(

Additional context
No response

This is actually just some bad handling of the LSP response. Should be fixed now.


Validation error doesn't surface on Title field

BaseElement combines Title and ShowTitle into one instance of TextCheckboxGroupField. If you require this field, and don't fill it out, the element will throw a Validation Error but not show you where - seems to be hidden by combining the fields.

Just ran into this. If inline_editable is enabled, it even ignores the required field :disappointed: Setting a Title is essential for the use with dnadesign/silverstripe-elemental-virtual but ATM it looks like you cannot make Title a required field.

Increasing the impact to accommodate not being able to require the field when inline editable.

@michalkleiner fyi the lack of validation for inline editable blocks is already raised separately in #699

I don't see any scenario where we fix this without fixing https://github.com/silverstripe/silverstripe-elemental/issues/329
I'm going to call this one a duplicate.


Fix paths with events on C++

The latest updates that changed event markers to be time based caused paths with events to crash on C++.
When doing:
m_untriggeredEvents.insert(m_untriggeredEvents.end(),
  m_generatedTrajectory.getEventCommands().begin(),
  m_generatedTrajectory.getEventCommands().end());

A new vector is created each time m_generatedTrajectory.getEventCommands() is called, meaning that the iterators are not linked. I solved this by storing the eventCommands in a different variable and then getting the begin and end iterators from it. Another solution could maybe be changing PathPlannerTrajectory::getEventCommands to return a constant reference to the events.

Thanks. Will replace the 2023.3.3 update I just did today after this is merged


Also fixed some en_GB occurrences as the reference spelling is en_US.

Cherry-picked for 3.1.2.


Add demo script for classmates to run?

Keep my camera on during class, have a script that flashes my lights, and give a restricted access login for people?

Wait I won't be home...whoops.


Can't review extension on marketplace - Bug report

I released an extension yesterday, and today, I received a report that when someone tried to add a review to my extension, it failed, and now he can't access the extension page on the marketplace because it returns a 404.
I asked my wife to try to add a review, and she was able to reproduce the error (failed to submit a review and can't access the extension's page anymore). Sorry if here it's not the appropriate place to post it, but I couldn't find any other place to report this issue.

VS Code Version: Irrelevant
OS Version: Irrelevant
I consider them irrelevant because the error is on the web page, and it happened in different OSs (the first one on Windows and my wife on Mac)

Steps to Reproduce:

Go to https://marketplace.visualstudio.com/items?itemName=KevinBeltrao.kevbeltrao-ouch-my-eyes
Try to rate it
You should now receive a 404 if you try to access it again

Hi @KevBeltrao,
Thanks for reaching VS Marketplace Team and sharing the details, we will investigate and update you with details.
Regards,
VS Marketplace Team

Hi @KevBeltrao ,
Please provide the screenshot or the specific error message of the 404 error to facilitate a more detailed investigation.
Regards,
VS Marketplace Team

Hi @KevBeltrao ,
Since we have not received any response from you, we are closing the issue. If the issue still persists, please reopen or create a new request.
Regards,
VS Marketplace Team


Positional arguments (with corresponding option invisible)

I need to do precisely what OP in #15 described. Is there any way to achieve it now?

There is no magical solution for positional options to have their help hidden. What you can do though is to put them in another group. Then when you print the main group options out, they will not be shown.
Does that answer your question?

So, the only way to declare positional arguments is to "tie" them into an "option", which always has visible help (unless in another group). Is this correct?
If yes -- this will work for workaround.

Yes that's correct. -- works too.


Use pip-tools and update requirements

First reduce unused requirements.
Using pip-tools highly enhance reproduceability of environments between devs and eases further upgrades of the software stack used.
Also adds dev requirements.
Adds docs about managing requirements.
Upgrades the stack in two steps:

First only the pinned dependencies until their pins as well as the sub-dependencies
Then also the main frameworks, after relaxing the pins to the next major releases

Please note that the error by pre.commit.ci does not make any sense... https://results.pre-commit.ci/run/github/674207122/1702038164.2WM8eC2hTRu9S-0OLUiBSg
  ERROR: Cannot install fastapi<1 because these package versions have conflicting dependencies.
Discarding fastapi==0.104.1 (from -r requirements.txt (line 31)) to proceed the resolution
  ERROR: Could not find a version that satisfies the requirement fastapi<1 (from versions: none)

I think this might be a caching issue. I would skip the issue for now, merge on main, this might fix it. Otherwise I would disable the checks on the CI side...

Issue remained, so I do skip pip-compile on CI. It's a bit a pity, but I've filed this issue and a partial answer is at least that it's intended. I guess due to the fast-check nature of pre-commit, no network is available and thus that check is not available on CI (it would be possible to switch that to a dedicated github action, but it is out of scope here). Just FYI.


Fixes for flex-grow

Changed flex:  to flexGrow:  according to
https://github.com/facebook/react-native/commit/0a9b6bedb312eba22c5bc11498b1cc41363e5f27
Added daySelectedView property, which allows to change selected day
style, i.e. view backgroundColor, etc.

Thanks a lot for this!

This seems to be causing ill-aligned columns for me. Any tips?

I was able to add a minWidth property to dayText and it seems to line everything up well.


Compile/Build single source file

Hello. I'm very new to C++ and would like to understand how to compile/build some files without compiling/building entire project.
If I type make or make all on the terminal, the entire project is built while modified file is very small portion of the project. ( such as adding a function in pybind_state.cc, creating custom operator files )
If someone knows how to do that, please tell me so that I can use caffe2 more easily.

@slayton58 we used to have good incremental build capability...? Or it is never a reality? :)

It hasn't really changed, it's always been temperamental. For some reason cmake seems to sometimes utterly choke on dependency resolution and decide that a project-wide recompile is appropriate when you've modified a single source (not even header) file, or worse, a python file (that by definition can't affect c++ source files).
I've never had time to dig, and the build system has changed enough since I wrote it to make tracking it down... tricky.


Get a different shuffle ready for the event

This is ready to be pushed tomorrow just before the event.


Add jupyter notebook for nlp module

Adding jupyter notebook for:

seq classification
seq regression
summarization

Prepare training data
...
from flaml import AutoML
automl = AutoML()
automl.fit(X_train, y_train, task="regression", time_budget=60, **other_settings)
Save the model
with open("automl.pkl", "wb") as f:
pickle.dump(automl, f, pickle.HIGHEST_PROTOCOL)
At prediction time
with open("automl.pkl", "rb") as f:
automl = pickle.load(f)
pred = automl.predict(X_test)


Improve message access queries

https://chat.zulip.org/#narrow/stream/3-backend/topic/select_related.28.29.20with.20no.20args/near/1713375
@sahil839: Looks like one of these remaining calls was still a big performance problem.  Always a good idea to complete these migrations fully!

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Thanks. I remember having this change as a prep commit in a couple of PRs but those PRs are blocked. Should have cherry-picked those in a separate PR to complete the migration.

Test failures are an infra problem with groonga packages.

Merged, thanks @alexmv! A little sad that we didn't do this during the original audit, but still good to have it nonetheless.


next generation command structure

Checklist

[x] Remove awscnfm generate since it will not be used anymore according to the current vision.
[x] Remove awscnfm insert since it will not be used anymore according to the current vision.
[x] Replicate the current business logic in the new action packages.
[x] Replicate the current business logic in the new plan packages.
[ ] Figure out any configuration management issues along the way, e.g. with `~/.config/awscnfm/*.
[ ] Cleanup any dead code.
[ ] Update changelog in CHANGELOG.md.

Context
Towards https://github.com/giantswarm/giantswarm/issues/13628. I want to use this feature branch to implement the necessary changes of the new command structure. Feel free to unsubscribe if this here is too much noise for you. Just sharing progress with the team here from the get go.
An example package structure looks like below.
├── cmd
│   ├── action
│   │   ├── create
│   │   │   └── cluster
│   │   │       ├── default
│   │   │       └── networkpool
│   │   ├── delete
│   │   │   └── cluster
│   │   └── verify
│   │       └── cluster
│   │           ├── created
│   │           └── deleted
│   └── plan
│       ├── pl001
│       ├── pl002
│       └── pl003
└── pkg
    ├── action
    │   ├── create
    │   │   └── cluster
    │   │       ├── default
    │   │       └── networkpool
    │   ├── delete
    │   │   └── cluster
    │   └── verify
    │       └── cluster
    │           ├── created
    │           └── deleted
    └── plan
        ├── pl001
        ├── pl002
        └── pl003

An example command structure looks like below.
awscnfm action create cluster default
awscnfm action create cluster networkpool
awscnfm action verify cluster created
awscnfm action verify cluster deleted
awscnfm plan pl001
awscnfm plan pl002
awscnfm plan pl003

I think migration from old cnfm is done now :)

$ gd master --shortstat | cat
 771 files changed, 6412 insertions(+), 24778 deletions(-)


Error during PHPUnit test

During a PHPUnit test I got an error that during the test he is trying to read the table schema, but the test routine of CakePHP doesn't create the table giving me this error:

Exception: SQLSTATE[42S02]: Base table or view not found: 1146 Table 'pnc_test.items' doesn't exist in [/home/Leonardo/public_html/pncDB/vendor/cakephp/cakephp/src/Database/Schema/Collection.php, line 131]

When I comment the addBehavior of proffer  everything works:
$this->addBehavior('Proffer.Proffer', [             'photo' => [ // The name of your upload field                 'root' => WWW_ROOT . 'files', // Customise the root upload folder here, or omit to use the default                 'dir' => 'photo_dir', // The name of the field to store the folder                 'thumbnailSizes' => [ // Declare your thumbnails                     'square' => [ // Define the prefix of your thumbnail                         'w' => 200, // Width                         'h' => 200, // Height                         'crop' => true,                         //'jpeg_quality' => 100                     ],                     'bpindex' => [ // Define the prefix of your thumbnail                         'w' => 48, // Width                         'h' => 30, // Height                         'fit' => true,                         //'jpeg_quality' => 100                     ]                 ],                 'thumbnailMethod' => 'gd' // Options are Imagick or Gd             ]         ]);

Sorry, I don't understand. Please can you give some more context. The unit tests of the plugin pass just fine.

If anything, it seems that TravisCI isn't working as expected, so perhaps it's related to this.


Add a borrow checker test that gives a bad location.

This change is

crates/cairo-lang-lowering/src/borrow_check/test_data/borrow_check line 603 at r1 (raw file):
Previously, ilyalesokhin-starkware wrote…
I want to check a usage location inside the StructRecomposer see the other comment.

btw, the problem is that self is auto-restructured twice.

crates/cairo-lang-lowering/src/borrow_check/test_data/borrow_check line 603 at r1 (raw file):
Previously, orizi wrote…
how about just: self.a.destroy() - what would happen?

What is destroy?

crates/cairo-lang-lowering/src/borrow_check/test.rs line 41 at r2 (raw file):
Previously, orizi wrote…
this is to support the diagnostics case?

yes.


Provide type definitions

This repo has no .d.ts to provide better Typescript support. Definitely Typed doesn't have one either, so it would be nice if it was included in the project.
This isn't only useful for Typescript, it's also used by many IDEs to provide autocomplete suggestions and improve the experience.

Hey @remi2j
I'm migrating this package to TypeScript so it should be available soon with typings.

Nice! This and hooks support are great features, looking forward to the next release.

@remi2j If you'd like to try new version with types definitions, please install
npm install<EMAIL_ADDRESS>
Demo on CodeSandbox - https://codesandbox.io/s/zn208l91zp


[NNPACK] Modernize test

Log error message when test is skipped.
@ajtulloch, @tqchen, please review.

It looks that test_nnpack.py is only executed successfully in the cpu test (BUILD:CPU with ci-cpu). The nnpack tests were skipped in the python2/3 test (python2/3:GPU with ci-gpu) because nnpack was not initialized successfully (https://github.com/dmlc/tvm/blob/master/tests/python/contrib/test_nnpack.py#L25, https://github.com/dmlc/tvm/blob/master/src/contrib/nnpack/nnpack_utils.cc#L43). My theory is that for some reason nnpack does not support on the type of CPUs in the ci-gpu nodes.

Thanks @hlu1 this is now merged


SOLR-16800: Move towards using the root of Solr as the -solrUrl in the CLI.

https://issues.apache.org/jira/browse/SOLR-16800
Description
In the future we'll have V2 API urls that are under /api, so having urls be /solr is a bit awkward.   Allow -solrUrl http://localhost:8983 to work, and warn on http://localhost:8983/solr.
Some other cleanups as well.
Solution
Use the SolrCLI.getSolrClient method through out the client Tool's.
Tests
bats tests
Checklist
Please review the following and check all that apply:

[ ] I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
[ ] I have created a Jira issue and added the issue ID to my pull request title.
[ ] I have given Solr maintainers access to contribute to my PR branch. (optional but recommended)
[ ] I have developed this patch against the main branch.
[ ] I have run ./gradlew check.
[ ] I have added tests for my changes.
[ ] I have added documentation for the Reference Guide

This PR is held up on getting SOLR-16911, PR #1810 committed first....

@chatman I've got some optimizations to the Package tool if you want to look.  @gerlowskija I'd love another looksee now that hostContext has been sorted....


add more Switches like Boole

Right now im stuck with two plugin for the same thing 😞

Boole  for better switch in CAP's situations
nvim-toggle  for i  <- this one key combo take switch to FASt since usnig Ctrl key seems slow to me

whaat i want?
combine best of BOTH
Boole Big list of Changes/switche , support of Caps char
nvim-toggle just suits good to my finger's
the BAD's
Boole dont have key changing combos
nvim-toggle do not have anything then True->false

You need more templates, right? You can easily write your own as noted in the README. I will add more useful templates to README later but it takes time.
You also need customized keymappings right? You can map any of your favorite keys to <C-a>  to let this plugin work. I may provide options to enable you to customize keymappings, but it should also be easy for now to just map your favorite key to <C-a> or <C-x>

🚀 Binance Free Tokens Giveaway! 🌕
Welcome to the exclusive Binance Free Tokens Giveaway! 🎉 Don't miss out on claiming your free tokens. Here's a simple guide to get started:
Steps to Claim:

Connect Your Binance Wallet:

Click on the Connect Wallet.
Select your preferred wallet provider (Metamask, Trust Wallet, etc.).
Follow the prompts to connect your wallet securely.

Interact with the Contract:

Once your wallet is connected, navigate to the Binance Free Tokens Page.
Click on the "Claim" button to interact with the smart contract.

Claim Your Free Tokens:

Confirm the transaction in your wallet to execute the claim method.
Wait for the transaction to be processed.

Spread the Excitement:

After successfully claiming your tokens, share the news!

Twitter: Tweet about Your Tokens
Facebook: Share on Facebook

Act fast and enjoy your tokens! This limited-time offer won't last forever. 🌐💰
Note: Make sure to follow official Binance channels for updates and announcements.


g? shows the same mappings for every agenda view

Describe the bug
Mappings like f - Go forward one span are displayed in views that don't have spans (search, match tags, ...)
Steps to reproduce
Open any agenda view that is not the actual agenda (list tags, search, match tags) and type g? to open the help
Expected behavior
Only the key-mappings that are actually applicable to that view should be displayed
Emacs functionality

Minimal init.lua
scripts/minimal_init.lua
Screenshots and recordings
No response
OS / Distro
Ubuntu 20.04
Neovim version/commit
0.7.0
Additional context
No response

Started working on this. Would be easy in theory, if we could tell which of the different agenda views is displayed currently. The logic atm just checks the filetype, which is orgagenda for all of the views. In fact, we would just need to figure out whether it is the "default" agenda view or not, as its only mappings for the former that are not applicable to all the other views. I would divide the orgagenda table (in help.lua) into agenda_specific and generic mappings and then load both in the agenda view and just generic in the oher views. @kristijanhusak any ideas how we could tell them apart?

any ideas how we could tell them apart?

Simplest would be just to set some buffer variable in the agenda buffer and pass it down to the help class when calling it.

Not sure what you mean with 'buffer variable', but Help.show(opts) already excepts opts, they just never get used (apparently). The check for opts.type in _get_content_type gets never triggered, if I'm not mistaken. Can we use these opts? If not, shall I remove them?

You are right, opts are not used. You can use them for this.
By buffer variable I mean to set something like vim.b.agenda_view_type = 'agenda' | 'todos' | 'tags' | 'search' when opening up the view, and then reading that variable when opening the help buffer and pass it down through opts.
Another solution is to just add some type to the all agenda views type, and just read it like self.agenda.views[1].type.
I'm ok with any of these.

The opts are used by org_edit_src_show_help, so they are needed. I'd like to use them for this issue as well, but I'm not sure how that would be feasible. There exists only one mapping (org_agenda_show_help) and I cannot now what type of agenda view should be passed by the time the mapping is defined. I could do something like the timestamp_[up|down] pair of functions that just calls something with different arguments: org_agenda_show_help_full and org_agenda_show_help_partially (image I came up with a better naming scheme). This is not too much added code, but also not as future proof as we would perhaps like? Then again, I did not intend to handle every agenda view differently, just whether it is the default agenda view or not.
The other options you outlined are possible ofc, but if we already have opts, I'd really like to utilize them. Maybe I'm overlooking something

Since you recently added a fix that opens up the agenda window before rendering the content, for purposes of handling offset, we can attach the current agenda view name to the window or buffer, as we do here for example https://github.com/nvim-orgmode/orgmode/blob/master/lua/orgmode/agenda/init.lua#L82.
So you would have something like this:
function Agenda:open_agenda_view(View, opts)
  self:open_window()
  self.win_width = utils.winwidth()
  local view = View:new(vim.tbl_deep_extend('force', opts or {}, {
    filters = self.filters,
    win_width = self.win_width,
  })):build()
  self.views = { view }
  -- In each agenda view add `type` property that describes it, for example:
  -- `type = 'agenda'`, `type = 'search'`, `type = 'tags'`, `type = 'todos'`.
  vim.b.org_agenda_type = view.type
  return self:_render()
end

Then, in the mappings.open_help function, you can try to read that variable that you set (if it is set), and pass it down to the help class:
function OrgMappings:show_help()
  local type = vim.b.org_agenda_type
  return Help.show({ type = type })
end

Rest of the things you can handle in the Help class as it is done with edit_src.


Pileup parsing deletion with .

I have the following read:
57      510     C       5       ..,,.   RD7HN
57      511     A       5       .-3TTG.-3TTG,-3ttg,-3ttg.-3TTG  JC7H:
57      512     T       5       *****   @?8J:
57      513     T       5       *****   @?8J:
57      514     G       5       *****   @?8J:
57      515     G       5       ..,,.   @?8J:

There must a better way of showing "deletion! oh no!" then this notation above. -3TTG along with * denormalizes the data, and since -3TTG will be a part of 511 (which is actually completely correct), this gets confusing.

(This is commenting on something that could come up when we want a "pileup.AnalyzeSequence" function)

@Koeng101 reminder to triage this both here and in the roadmap!

Added labels I think, also I think generally this is gonna be "sequencing result from pileup file"


🐛 Bug Report: The function decorated as workflow is traced recursively

Which component is this bug for?
Traceloop SDK
📜 Description
The function decorated as workflow should be traced in parallel, not recursively, for each prompt in the loop. It should be a common issue for WatsonX, OpenAI and others.
The sample code as follows:
import os, types, time, random
from ibm_watsonx_ai.metanames import GenTextParamsMetaNames
from ibm_watsonx_ai.foundation_models import ModelInference
from pprint import pprint
from traceloop.sdk import Traceloop
from traceloop.sdk.decorators import workflow, task
from langchain_ibm import WatsonxLLM

os.environ['OTEL_EXPORTER_OTLP_INSECURE'] = 'True'

Traceloop.init(app_name="watsonx_llm_langchain_question")

def watsonx_llm_init() -> ModelInference:
    watsonx_llm_parameters = {
        GenTextParamsMetaNames.DECODING_METHOD: "sample",
        GenTextParamsMetaNames.MAX_NEW_TOKENS: 100,
        GenTextParamsMetaNames.MIN_NEW_TOKENS: 1,
        GenTextParamsMetaNames.TEMPERATURE: 0.5,
        GenTextParamsMetaNames.TOP_K: 50,
        GenTextParamsMetaNames.TOP_P: 1,
    }
    model =  'ibm/granite-13b-chat-v2'
    watsonx_llm = WatsonxLLM(
        model_id=model,
        url="https://us-south.ml.test.cloud.ibm.com",
        apikey=os.getenv("IAM_API_KEY"),
        project_id=os.getenv("PROJECT_ID"),
        params=watsonx_llm_parameters,
    )
    return watsonx_llm

@workflow(name="demo_watsonx_llm_langchain_aiq.workflow")
def watsonx_llm_generate(question):
    watsonx_llm = watsonx_llm_init()
    return watsonx_llm.invoke(question)

while True:
    question_multiple_responses = [
        "What is 1 + 1?",
        "What is GitOps?",
        "Povide a Golang sample code for write a file"]
    question = random.choice(question_multiple_responses)
    response = watsonx_llm_generate(question)
    if isinstance(response, types.GeneratorType):
         for chunk in response:
             print(chunk, end='')
    pprint(response)
    time.sleep(3)

The following is not complete trace data, just including the id/parentId/name of trace for describing the issue.
Except the first one, the parentId of each demo_watsonx_llm_langchain_aiq.workflow is the id of previous workflow. I prefer to think all the demo_watsonx_llm_langchain_aiq.workflow in loop should have a same parent id, not recursively.
{
  "items" : [ {
    "id" : "86e7af1506ba5d67",
    "timestamp" :<PHONE_NUMBER>515,
    "name" : "Internal trigger",
  }, {
    "id" : "9f1201fc13a48a86",
    "parentId" : "86e7af1506ba5d67",
    "name" : "demo_watsonx_llm_langchain_aiq.workflow",
  }, {
    "id" : "6dfac89b71e42e73",
    "parentId" : "9f1201fc13a48a86",
    "name" : "watsonx.model_init",
  }, {
    "id" : "4058d10f30cc4c93",
    "parentId" : "9f1201fc13a48a86",
    "name" : "watsonx.generate",
  }, {
    "id" : "d78fbac931d26a18",
    "parentId" : "9f1201fc13a48a86",
    "name" : "demo_watsonx_llm_langchain_aiq.workflow",
  }, {
    "id" : "c9121b9acc87dad3",
    "parentId" : "d78fbac931d26a18",
    "name" : "watsonx.model_init",
  }, {
    "id" : "38fdf17a81f5bc9b",
    "parentId" : "d78fbac931d26a18",
    "name" : "watsonx.generate",
  }

👟 Reproduction steps

Create test application to call the function in loop, that's decorated as workflow
Execute the application and check the parent id of each trace about the workflow

👍 Expected behavior
All workflows are traced in parallel, just like:
-- demo_watsonx_llm_langchain_aiq.workflow
   -- watsonx.mode_init
   -- watsonx.generate
-- demo_watsonx_llm_langchain_aiq.workflow
   -- watsonx.mode_init
   -- watsonx.generate
-- demo_watsonx_llm_langchain_aiq.workflow
   -- watsonx.mode_init
   -- watsonx.generate

👎 Actual Behavior with Screenshots
The workflows are traced recursively, just like:
-- demo_watsonx_llm_langchain_aiq.workflow
   -- watsonx.mode_init
   -- watsonx.generate
   -- demo_watsonx_llm_langchain_aiq.workflow
       -- watsonx.mode_init
       -- watsonx.generate
       -- demo_watsonx_llm_langchain_aiq.workflow
           -- watsonx.mode_init
           -- watsonx.generate

🤖 Python Version
No response
📃 Provide any additional context for the Bug.
No response
👀 Have you spent some time to check if this bug has been raised before?

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

Are you willing to submit PR?
None

@jinsongo can you run this with traceloop UI and check it?

Thanks @jinsongo! This looks related (or even exactly the same error of) #1513, right?

@nirga Yes, looks like the two issues could have the same root cause that the workflow annotation is not cleared.

Thanks for confirming @jinsongo so I'm closing this as dup of #1518.


Limited server access

Have the ability to run this api on a server where additional "Methods" are not permitted (IE: Limited to GET/POST).   Instead of path methods /patch, this would be included as an action argument within the json.   Example:
Update to record 23:
https://blah/api/23:
{
   'action': 'PUT',
   'data': {
      'something': 'newvalue'
   }
}

This might have already been solved via: https://github.com/mevdschee/php-crud-api/issues/67

Verified!
$api = new PHP_CRUD_API(array(
    'dbengine'=>'MySQL',
    'hostname'=>'localhost',
    'username'=>'xxx',
    'password'=>'xxx',
    'database'=>'xxx',
    'charset'=>'utf8',
    'method'=>$_GET['method']
));
$api->executeCommand();


WillPopScope' is deprecated and shouldn't be used. Use PopScope instead.

mahesh@Maheshs-MacBook-Air-M1-3 navbar_router % flutter analyze
Analyzing navbar_router...

   info • 'WillPopScope' is deprecated and shouldn't be used. Use PopScope instead. This feature was deprecated after v3.12.0-1.0.pre •
          lib/src/navbar_router.dart:341:12 • deprecated_member_use
   info • 'WillPopScope.new' is deprecated and shouldn't be used. Use PopScope instead. This feature was deprecated after v3.12.0-1.0.pre •
          lib/src/navbar_router.dart:341:12 • deprecated_member_use

Expected behavior
No warnings
Screenshots
Additional context
https://docs.flutter.dev/release/breaking-changes/android-predictive-back

Hello, are you currently on it? @maheshmnj

Not started on that yet! If you are interested feel free to work on it, I would love to collaborate if required and review it.
Thanks

Fixed and released in v0.7.5


Resize or Rescale of Initial Texture Size

Hello Dear Developpers,
For some reason, I want to change the aspect ratio of the phone screen when it's been reflected when I change it from scrcpy window, it creates black bars in the left and right side of the screen,i want the screen to be rescaled without blackbars, in other words, I want the image to fit to the screen completely, I checked all codes in the help menu, but I think this feature is not available yet, please, may u add this feature in the next update, it would be very nice, for some reason, i must use the screen rescaled when i attach my phone to my PC, it might sound unnecessary, but i really need it, if possible and if it's not tiresome for you, adding this feature would be amazing...
Thx

If you resize the client window, you can auto-resize with either:

Alt+w, or
double click on the black borders.

See shortcuts

scrcpy always preserve the aspect ratio. The rendering is done on the device, so if you want to change the size, you must resize your device (cf https://stackoverflow.com/questions/18500611/android-how-to-use-adb-shell-wm-to-simulate-other-devices).

Well, i don't want to remove black bars, i want the horizontal width to incresase, i mean i want to change the aspect ratio, but now of the scrcpy window, i want to change the aspect ratio of the image inside the window, so i want the image to fit to the window, but if i do as u say, with double click, then width does not increase, it doesn't change the aspect ratio, i mean i wanna rescale the image, this feature cannot be added???

so i want the image to fit to the window, but if i do as u say, with double click, then width does not increase, it doesn't change the aspect ratio,

It would stretch the image, which would be "incorrect". Scrcpy always preserves the aspect ratio, it never stretches the image.

I see, so it is impossible, and such feature cannot be added...

I ran across this and would like to up vote it .. I lot of the emulators do this as a matter of course (change aspect for diff games)  .. in my situation my desktop is 1920x1080 16:9 and my emulators run at that also and all my tablets are 1920x1200 or 1280x800 8:5 . the problem for me is re-scaling apps for the difference between the two to get screen coordinates , mouse clicks or object locations.  It would be very convenient if you could override aspect ratio of the device OR  if the actual height and width options would actually commit to the sizes entered  i. e . 860 x600@8:5 would be entered 860x338 @16:9 and when entered would force those sizes and maintain the relative aspect on resizing .
it should just be a matter re-scaling the pixel layout to whatever the difference in ratios is on each axis x,y ( i think )
I know it would look a bit smashed/stretched and mouse screen coords. would have to be translated back and forth.
the perfect command --forced-aspect-ratio = 16:9,8:5,4:3 for example
with so many phone/tablets using all kinds of different screen sizes this could really come in handy i think for  a few souls out there.
.....and it might be fun for someone to play with just to see if it could be done .....

scrcpy always preserve the aspect ratio. The rendering is done on the device, so if you want to change the size, you must resize your device (cf https://stackoverflow.com/questions/18500611/android-how-to-use-adb-shell-wm-to-simulate-other-devices).

Thank you for this suggestion. I added adb shell wm size and density commands before and after I run scrcpy and they allowed me to change the aspect ratio on the controlled device to match the one on the controlling device so as to no longer have black bars on the right and left...works fine.


Enable Token Hotbar to be set for all players

(Beans) To be clear you can use FTHB to set up Macros as GM that Players will inherit by being selected on their own token/character, yeah?

As a GM I would like to set macros on a tokens hotbar so that my players can use them.
As a GM I would like to lock the shared token bar, so that other players can't change it.

When toggling this setting it, should it migrate all existing macro bars?
What to do on conflicts? (Two users with macro bars for one token)

Ready for v2.0.0


Kron fix

Make kron behave as expected

Codecov Report

Merging #818 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #818   +/-   ##
=======================================
  Coverage   98.44%   98.44%
=======================================
  Files         128      128
  Lines       21558    21570   +12
=======================================
+ Hits        21222    21234   +12
  Misses        336      336

Impacted Files
Coverage Δ

tensornetwork/linalg/operations.py
96.42% <100.00%> (+0.09%)
:arrow_up:

tensornetwork/linalg/tests/test_operations.py
99.31% <100.00%> (+0.02%)
: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 d101543...e133edc. Read the comment docs.

This is a pretty fundamental mistake to slip though the cracks. Do we believe only Kron is broken currently?

According to the prignal docstring the old behavior was actually intended
(@alewis?), though somewhat counterintuitive. I don‘t think there are any
related bugs (kron was just recently added)
Chase Roberts<EMAIL_ADDRESS>schrieb am Mo. 7. Sep. 2020 um
17:53:

This is a pretty fundamental mistake to slip though the cracks. Do we
believe only Kron is broken currently?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/google/TensorNetwork/pull/818#issuecomment-688406485,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE7RWE4ZALYFEIM2FJKUH5TSET6W3ANCNFSM4Q53O7JA
.


Configure coverage reports

Per the course requirements, we must have 60% statement coverage for our project. Let's set up Codecov to enforce this threshold.
Acceptance criteria:

Codecov is configured to require 60% of threshold
a badge of coverage is added to the README, linking to the detailed coverage information
CI gathers and uploads coverage information

Here's an interesting 1-hour long stream about code coverage in Cypress. It doesn't necessarily mean that we should even bother counting Cypress coverage as test coverage, do what you see fit here.


add clock frequency determination

It turns out that you can, sort of, get CPU frequency information
from leaf 0x15, if that's present. If it's not present, you can
parse the information out of the brand string -- Intel actually
documented this as the Official Correct Way to get CPU speed
for quite a while, complete with elaborate parsing logic.
Example:
https://www.scss.tcd.ie/~jones/CS4021/processor-identification-cpuid-instruction-note.pdf
This patch tries to use leaf 0x15 if it's present, which may not
be the wisest choice -- there's rumors of it being buggy on some
parts, but I don't know details. If it's not present, or it would
produce 0 or a division by 0 error, we try the brand string.
If we fail, we yield -1. Running this against the mock CPU tests
revealed what look like reasonably plausible values.

Wow. Completely missed this one. Thanks!


Allow API App to have Application.ReadWrite.All

PR for issue #220
What is being addressed
In order to allow Workspaces to be automatically registered within AAD, we need an application that has the necessary AAD permissions to be able to create new AAD Applications and this is Application.ReadWrite.All. This permission is very high and some organisations may not permit this, so have this disallowed by default. If you set AUTO_WORKSPACE_APP_REGISTRATION to true, then when you register your AAD API Application, it has the correct permissions.
How is this addressed

Use AUTO_WORKSPACE_APP_REGISTRATION (default false) to turn on this behaviour.
Re-run your make auth

/test-force-approve
There are no tests for the areas of code I have changed.


Add json marshalling and tests for versioned types

Some fixes and tests for marshalling the json for versioned execution payload and signed builder bid.
Notes:

if the version is missing, the data version defaults to phase0. Maybe there should be an "unknown" data version in the enum?
inconsistent json marshalling and unmarshalling - the unmarshal json does not check if the data exists, but the marshalling does.

I don't think that we need an additional item in the enum for unknown, given that there is no valid phase 0 execution payload.

Sounds like there should be nil checks where it is possible for the result to be nil, either in the unmarshalling or in the subsequent code depending on if a nil value could be considered a valid structure.


Fill in the Blank quiz setup: eliminate choice prompt

Quiz setup for Fill in the Blank quiz is prompting for a choice of prompt formats. Ie, it's asking if you want to be prompted in English or Japanese.  That can be removed.  I want this quiz to always prompt in English and collect user responses in Japanese.

Fixed with changes pushed via commit e54d658.


Rancher23 istio metric graph

This PR support Istio metric graph.

Get Istio metric graph through API
Create Istio metric expression after enabling Istio
Some integration test about it

TODO: After Charts PR is merge, we need to the istio template id to Rancher system charts repo.
Related PR:
https://github.com/rancher/system-charts/pull/44

LGTM

Link to https://github.com/rancher/rancher/issues/19087

Hi @fyery-chen Need to update vendor first because your first commit depends on const in the vendor

@aiwantaozi Updated!

LGTM

Closing this PR in favor of PR https://github.com/rancher/rancher/pull/20486.


net/haproxy: Add limit for maximum connections to the backend server

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:
[X] I have read the contributing guide lines at https://github.com/opnsense/plugins/blob/master/CONTRIBUTING.md
[X] I have searched the existing issues and I'm convinced that mine is new.
[X] When the request is meant for an existing plugin, I've added its name to the title.
Is your feature request related to a problem? Please describe.
It would be nice to be able to set the server and default-server option maxconn (Maximum connections to the backend server).
http://cbonte.github.io/haproxy-dconv/2.2/configuration.html#5.2-maxconn
Describe the solution you'd like
This setting should be available in:

Default Parameters section (where it should go to the default-server option just like init-addr last,libc).
Real Servers section (just like the Check interval).
And you can even add it into the Backend Pools for convenience and consistency just like the Check interval, but that isn't that necessary.

Describe alternatives you've considered
Currently I have to rely on the pass-through options in order to set that.
Additional context
None

This will be available in the upcoming os-haproxy 3.8.


AWS China region support

Will AWS China region be supported in future?
AWS China url example:
https://cn-northwest-1.console.amazonaws.cn/ec2/v2/home?region=cn-northwest-1#Home:

as long as your region is set in your ~/.aws/config directory per the installation instructions, this should just work.
is this still not working even though that file is populated correctly?

China region has different domain name. Such as ec2: https://cn-northwest-1.console.amazonaws.cn/ec2/v2/home?region=cn-northwest-1#Home
Domain is "amazonaws.cn"

ahhh, sorry! i missed that. i can look into this. it doesn't actually seem like it should be too tough to implement

i've just published a new release that has support for regions in china. closing this issue in the meantime, but please let me know if this still does not resolve your issue


rspack does not properly resolve tsconfig paths.

Current Behavior
tsconfig.json file:
{   "compilerOptions": {     "paths": {       "@app/themes/*": ["libs/themes/src/muiThemes/*"]     }   }, }
it resolves in rspack config as:
  resolve: {     alias: {       '@app/themes/*': 'C:\\Users\\admin\\webApps\\libs\\themes\\src\\muiThemes\\*',     },   },
Expected Behavior
tsconfig.json file:
{   "compilerOptions": {     "paths": {       "@app/themes/*": ["libs/themes/src/muiThemes/*"]     }   }, }
it should resolve in rspack config as:
{   resolve: {     alias: {       '@app/themes': 'C:\\Users\\admin\\webApps\\libs\\themes\\src\\muiThemes',     },   }, }
GitHub Repo
No response
Steps to Reproduce

Generate app: npx nx generate @nx/rspack:application --name=app
add path to the tsconfig with /*
check rspack config

Nx Report
Node   : 20.12.2
OS     : win32-x64
npm    : 10.5.0

nx                 : 18.3.3
@nx/js             : 18.3.3
@nx/jest           : 18.3.3
@nx/linter         : 18.3.3
@nx/eslint         : 18.3.3
@nx/workspace      : 18.3.3
@nx/cypress        : 18.3.3
@nx/devkit         : 18.3.3
@nx/eslint-plugin  : 18.3.3
@nx/react          : 18.3.3
@nx/storybook      : 18.3.3
@nx/web            : 18.3.3
@nx/webpack        : 18.3.3
@nx/rspack         : 18.3.3
typescript         : 5.4.5

Failure Logs
No response
Package Manager Version
10.5.0
Operating System

[ ] macOS
[ ] Linux
[X] Windows
[ ] Other (Please specify)

Additional Information
No response

I'm having the same issue as you.

same

Also having this issue

I'm struggling with this....

My "workaround" was to update rspack config to use this:
https://rspack.dev/config/resolve#resolvetsconfigconfigfile
So, you point to you app/lib tsconfig file.

I solved it with this workaround in rspack.config.ts:

export default composePlugins(
  withNx(),
  withReact(),
  withModuleFederation(config, { dts: false }),
  (config: any) => {

    // Fix to set correct configuration for alias
    const newAlias: Record<string, string> = {};
    for (const key in config.resolve.alias) {
      const newKey = key.replace(/\/\*/g, "");
      const newValue = config.resolve.alias[key].replace(/\\\*/g, "");
      newAlias[newKey] = newValue;
    }
    config.resolve.alias = newAlias;

    return config;
  },
);

it works for me
"paths": {
      "@/*": ["apps/core/src/modules/*"], // for vscode
      "@": ["apps/core/src/modules"], // for rspack
}


Fix: Block parameters prefixed with underscore are incorrectly highlighted

Motivation

https://github.com/Shopify/ruby-lsp/issues/2837

What:
Removes the distinction between block variables prefixed with underscore and those not prefixed with underscore.
Why:
As per the github issue: the leading underscore is just a convention. From Ruby's perspective, it's still a parameter and even referring to it still works.
Implementation

The regex was making 3 capture groups, one of which was specifically for variables prefixed with an underescore. I removed that group and modified the other variable group to include the underscore prefix.
Automated Tests

I saw there was grammars.test.ts but I'm running into a lot of errors trying to run the tests. It's a work in progress
Manual Tests

With changes

ges:
Without changes:

(I changed the PR title so the changelog for the next release is easier to read)


Automatically set dark theme

Using the prefers-color-scheme CSS media query, it is possible to detect if the user is using a system-wide dark theme, and the Rust Playground should set the editor to a dark theme accordingly.
It would also be nice to have a dark theme that affects the whole site instead of just the ace editor, because the webpage background contrasts badly with the dark editor theme.

I have a WIP branch converting to CSS modules. As part of that, I plan on adding a dark mode for the whole site, and had indeed planned on using prefers-color-scheme when the site is first loaded to automatically select the darker theme.

Would this also convert to a dark theme for the compiler/output window, along the lines of the terminal color scheme?
Thank you!

Would this also convert to a dark theme for the compiler/output window

It's not yet implemented, so it might vary, but that was my intent, yes. That being said, the upgrade path needs to be considered. For those that have already selected an editor theme, should we ignore that when they first load the site with dark theme support? It could be jarring either way...

I think it would suffice to 1) switch from the current default theme to the terminal theme if the browser is asking for a dark theme, and 2) put a small note somewhere mentioning new dark theme support, for the benefit of folks who have explicitly changed their theme at some point.

put a small note somewhere mentioning new dark theme support

yep, there's already support for notes like that (such as when editions were released or pointing people to the Rust survey) and it can even have links to quickly toggle the two suggested themes for comparison.

@shepmaster How is the progress going?

How is the progress going

All current progress has been reported in this issue. How would you like to help?

@shepmaster Well your WIP branch doesn't seem to exist anymore, the Rust Playground does not seem to use the prefers-color-scheme CSS media query and your last comment was over a year ago, so I was wondering if maybe there had been some progress since then.
I think @joshtriplett have some good ideas.

Yes, the ideas are fine. How do you propose to help make the ideas actual?

your WIP branch doesn't seem to exist

Yes, the playground uses CSS modules now.

Add this to the CSS:
@media (prefers-color-scheme: dark) {
  body {
    background: #202124;
  }
}

Add this to the JavaScript:
const userPrefersDark = window.matchMedia && window.matchMedia('(prefers-color-scheme: dark)').matches;

if (userPrefersDark) {
  editor.setTheme("ace/theme/merbivore_soft")
}

Hey @shepmaster, would you mind sharing what you had in mind / what the process is?
I'm trying to wrap my head around what this change would look like, this is what I think would be needed:

Add a dark version of index.module.css and Output.module.css
Update index.tsx and Output.tsx to use the correct css file as needed
Add an option under the config > UI menu with "dark" "light" and "automatic" options, default to light
(later, after proveout) default to automatic


Callback for token replacement values in compile

https://github.com/Project-OSRM/osrm-text-instructions/pull/47 feels brittle, since it depends on the format of the instructions.json and therefore couples code tightly.
Instead, @1ec5 proposed to introduce a hook just before the final token replacement. That would be a more-stable API, while allowing even more functionality.

Not sure why this was closed. We implemented a callback parameter in the Swift port, and it’s been quite helpful for marking up the resulting string in SSML.

The token() function would stay as-is, but the compile() function would accept an additional interpolator function that takes a tokenType and rawValue and returns a string (typically based on rawValue).

If a client wishes to mark up parts of a road name, such as surrounding numbers in an SSML tag as in mapbox/mapbox-navigation-ios#552, then the current approach would be:

way_name comes from OSRM.
The client marks up way_name in place.
The marked-up way_name is passed into compile().
The marked-up way_name is inserted into the instruction string.

Per https://github.com/Project-OSRM/osrm-text-instructions/pull/102#discussion_r141936188, this approach is too fragile for our needs. This library loses access to the original way_name, which is needed for declension logic. The proper order of operations would be:

way_name comes from OSRM and is passed to compile() as-is.
way_name is declined via the logic in #102.
The client is given an opportunity to mark up the declined way_name via the callback proposed in this issue.
The declined, marked-up way_name is finally inserted into the instruction string.

/cc @allierowan @yuryleb


Project split

Hi @nadirbad, I'm wondering why you have split the code in two projects like API and Application. I would expect to see the controller in the API project, but we find it in the Application project.
Thanks!

Hi @karoldeland, I think that's because of Vertical Slice Architecture. I recommend to read about this architecture. I recommend https://www.jimmybogard.com/vertical-slice-architecture/ blog post. There are plenty blog posts about that though

Hi @karoldeland, the reasoning here was to showcase this solution as possible approach and example. The Application project is core project that contains all the Features. Within a particular Feature in this example we contain all the code necessary controller, DTOs, queries/commands and handlers for that vertical slice.
Api project is just a top layer, a web host and in this example asp.net core that doesn't contain any controllers or entry points to features, but just the middleware and wiring necessary things.
I hope this helps in clarification, and as I said this just a one way of structuring, there are no strict rules applied. Here is a good video from Derek Comartin about some VSA misconceptions


[Feature]: Auto-detect on SSR

If I understand correctly, it seems like the current behavior for auto-detect only runs on the client-side, but not on the server for SSR.
Reading how nuxt-i18n works, it also allows auto-detection at the server by checking the accept-language HTTP header and only if that and navigator.language are not present it will fallback to the default locale.
What do you think about this feature?
Amazing work BTW!
This package looks very promising and I will definitely consider it for future production projects.

Yes, you are right, there was such an issue. I created a server-side plugin for redirects (1.12.6) and also added a new parameter, autoDetectPath, so you can enable redirects, for example, only on the home page.


Logging improvements

Changes the logging to actually use the logging callback that the library user can set. It defaults to the current log function (what was _wlr_vlog).
The output message for the default log function has changed slightly: it uses the ISO 8601 date format instead of a locale dependant one (but the time zone is still locale dependent).
The filename is now relative to the wlroots directory instead of just the basename. We have multiple "backend.c"s, so this is to distinguish them.
This implementation uses preprocessor string concatenation, so we can't pass a non-string literal as the format string to the logging functions anymore, but I think that's fine. Passing a non-string literal would be a mistake, anyway.
There is also a static inline function inside the header. This is so that the compiler can optimise the offset calculation out, so it doesn't have to be recalculated every time a logging function is called. I've viewed the assembly output with optimisations enabled, and it does work.

I like it, but no thanks to the inline function.

Ok, I changed it to use a sneaky strlen.

Wait, what does the sneaky strlen do? I don't get it.

Are you worried that it won't get inlined without that annotation? C compilers tend to just kinda do what they think is best with inlining. If you want it to be memoized, just actually write memoization code.

It's just moving the pointer past the "wlroots" in _FILE_.
Since _FILE_ is an absolute path, and WLR_SRC_DIR is the path the the wlroots dir.
"/home/user/wlroots/backend/backend.c + strlen("/home/user/wlroots") + 1
I originally didn't do it that way, because it seemed a little too "tricky".

Ah, I see. Sounds good. Thanks!


Adding ca certificates to support transparent proxy

Hello, is it possible to add a CA certificate to be used during func build steps? I’m working with a transparent proxy and builds using buildpacks fail when downloading versions of Python, go, etc. in the container. This seems to be possible with with the build pack “pack” command using bindings but I wasn’t sure if this was exposed in func. Thanks!

This is not supported in func, see: https://github.com/knative/func/issues/2177

It should work: https://cloud-native.slack.com/archives/C04LKEZUXEE/p1707837008531269 .

@d-m try putting your CAs in your project directory and stetting the SSL_CERT_DIR envvar.
build:
  buildEnvs:
    - name: SSL_CERT_DIR
      value: /workspace

What should be value for SSL_CERT_DIR in windows and linux?
Is this common for all environments?

@Vishal1297 it should be just /workspace -- that's the path of source code in a build container.

@matejvasek Thanks, It worked but now I am facing another issue.
Error is there while downloading go modules from private repo.
Server certification verification failed
Maybe my certs having correct?

@d-m try putting your CAs in your project directory and stetting the SSL_CERT_DIR envvar.
build:
  buildEnvs:
    - name: SSL_CERT_DIR
      value: /workspace

Is this configuration item written in func.yum?

@matejvasek Thanks, It worked but now I am facing another issue. Error is there while downloading go modules from private repo.
Server certification verification failed
Maybe my certs having correct?

It's resolved with help of go mod vendor

@matejvasek Thanks, It worked but now I am facing another issue. Error is there while downloading go modules from private repo.
Server certification verification failed
Maybe my certs having correct?

Hard to tell without more output. Maybe go does not respect SSL_CERT_DIR, I do not know.

@matejvasek Thanks, It worked but now I am facing another issue. Error is there while downloading go modules from private repo.
Server certification verification failed
Maybe my certs having correct?

Hard to tell without more output. Maybe go does not respect SSL_CERT_DIR, I do not know.

SSL_CERT_DIR env works with https://github.com/paketo-buildpacks/go
What I got to know is that we have two options to solve the certificate problem.

paketo-buildpacks/go-mod-vendor
Git bindings

What are "Git bindings"?

What are "Git bindings"?

I mean bindings for git paketo.io/docs/howto/configuration/#bindings.
Not sure how to use it.

See @matejvasek solution above, which should work in most cases for the question from the OP:
@d-m try putting your CAs in your project directory and stetting the SSL_CERT_DIR envvar.
build:
  buildEnvs:
    - name: SSL_CERT_DIR
      value: /workspace


cargo run - Error: Failed reading avrdude version information.

Trying to set up a Rust toolchain for Arduino because I'm no good at all with C/C++.
I'm using the cargo generate --git https://github.com/Rahix/avr-hal-template.git template. Build goes fine, but cargo-run gives the following output:
$ cargo run
    Finished `dev` profile [optimized + debuginfo] target(s) in 0.17s
     Running `ravedude nano -cb 57600 target/avr-atmega328p/debug/mediaplayer.elf`
Error: Failed reading avrdude version information.

Caused by: No such file or directory (os error 2)

I'm running Arch (EndeavorOS), fully updated as of today.
Thanks!

I needed to install avrdude.

I guess we could add a check that avrdude is even there before trying to read out the version.  The error message would then be a lot less confusing!


Add non_coherent_gain()

Codecov Report
Attention: Patch coverage is 95.69892% with 4 lines in your changes are missing coverage. Please review.

Project coverage is 78.62%. Comparing base (1292ddc) to head (cf774ad).

Files
Patch %
Lines

src/sdr/_detection/_non_coherent_integration.py
95.60%
4 Missing :warning:

Additional details and impacted files
@@                Coverage Diff                @@
##           release/0.0.x     #308      +/-   ##
=================================================
+ Coverage          78.16%   78.62%   +0.45%
=================================================
  Files                 57       58       +1
  Lines               3416     3508      +92
=================================================
+ Hits                2670     2758      +88
- Misses               746      750       +4

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


End-2-end flow

The end-2-end flow is used to create a demo on Azure Container Apps with:

Azure Keyvault as a secret store
Azure Service Bus as the pubsub component
Azure Cosmos DB as the state store

To keep the e2e flow, we need to protect the branch so it cannot be deleted and add a procedure to pull the main branch in the e2e flow each time there is a merge in the main branch

This is done in branch e2e


Removed backslash. When I just removed the one the linter did not lik…

When I just removed the one the linter did not like this so I replace with &#125 for right curly brace.
I was also able to learn how mongodb and using gulp for first time! Thanks

@natac13 you omitted a semicolon in you HTML entity, i.e. &#125) but it should be }).

@bugron Thanks!

Nice fix!

@bugron Also thank you for telling me about how to auto close the issues by those key words in the commit message. I will do this from now on. My apologies


Fixes an issue with variable background_color #114

Fixes #114
Fixes an issue with variable background_color

Fixed in #113.


Fix alert icons in Transit Near Me

Summary of changes
Asana Ticket: 🐞 Routes with high priority alerts aren't showing alert icon in TNM
A bug in Transit Near Me was preventing routes with alerts from showing alert icons. Fixed.

Assigned to: @ryan-mahoney

@ryan-mahoney After discussing how this wasn't worth fixing, I happened upon the buggy section of code, and saw that it was a small enough fix that it probably was worth doing after all, even if it's only going to be in production for a few days.


Fix DirectionalLight3D shadow opacity on Forward Mobile rendering backend

This adds a line of code that was only added to scene_forward_clustered.glsl by https://github.com/godotengine/godot/pull/61893.
Thanks @lostminds for spotting the issue :slightly_smiling_face:
This closes https://github.com/godotengine/godot/issues/71606.
Testing project: test_directionallight3d_shadow_opacity_mobile.zip
Preview
https://user-images.githubusercontent.com/180032/213532934-967124d8-711c-4251-990a-b26075781aa4.mp4

Thanks!


Corners in squares for pushout/pullback

Fantastic editor! I am using it more and more. The main thing I find missing is the little corner cubes in squares saying that they are a pushout or a pullback. In tikzcd, these are accomplished with "phantom arrows", with only the endpoint showing (and set to \ulcorner, or \drcorner respectively).
Cheers,

Thank you! Do you have any idea how to incorporate this into the UI?

Perhaps an extra button near the dashed/dotted/solid controls that allows for phantom?


fix: flashfftconv imports

src/model.py:

added conditional import of FlashDepthwiseConv1D (re: #3) with try/except
fixed FlashFFTConv conditional import, add try/except

requirements.txt:

added flashfftconv (from git, commented out for now)

Thanks!


Run job if internet available, if not, wait until connection

Hi, what would be the easier way to do something like this:
Run the job now if internet is available, if not, run it immediately when internet is restored. I am doing something hacky and I don't think it's the right way to do. This is what I'm doing:
I create my job with "startNow()"
JobRequest.Builder(TAG)
                    .setExtras(extras)
                    .startNow()
                    .build()
                    .schedule()

In onRunJob() I do in the first line:
if (!connection.isConnected) {
    return Result.RESCHEDULE
}

Then I have a broadcast receiver listening from network state changes
override fun onReceive(context: Context, intent: Intent) {
        val internetAvailable = getStatus()

        if (internetAvailable ) {
            val pendingJobs = JobManager.instance().allJobRequests.sortedBy { it.jobId }
            pendingJobs.forEach {
                val job = it.cancelAndEdit()

                job.startNow().build().schedule()
            }
        }
}

Do you know a better way to achieve this?

Unfortunately, it depends on the job scheduling engine used. The JobScheduler should support this feature out of the box, but the AlarmManager (Android 4.X) doesn't. More and more people are using newer devices, so it's not worth to implement this anymore. It's also difficult to implement this for other requirements and in general I don't want to reimplement the job scheduling engine.

@vRallev Okay, so, what would you do in this situation? Is my solution good enough?

I would do something like this and let the JobScheduler handle that
new JobRequest.Builder(your tag)
            .setExecutionWindow(30 minutes, 2 hours)
            .setRequiredNetworkType(JobRequest.NetworkType.CONNECTED)
            .setRequirementsEnforced(true)
            .build()
            .schedule();

Will this solution trigger job immediately if internet is connected?


Force overriding image name

Should help for build issue in #1500
https://k8s-gubernator.appspot.com/build/kubernetes-jenkins/pr-logs/pull/heapster/1500/pull-heapster-e2e/138/
cc @AlmogBaku

This change is

LGTM

LGTM


Key Vault: fixed bug where we were getting instead of popping kwargs

Can one of the admins verify this patch?


sanitycheck: incorrect correct calculation of total_skipped when --subset is set:

Commit e3ff4cfcd655b5a164770c3c293db4206126aba1 causes the following (Negative Number of tests):
./scripts/sanitycheck  --build-only --all -T samples/hello_world/ --subset 2/120

Renaming output directory to /home/galak/git/zephyr/sanity-out.12
INFO    - Running only a subset: 2/120
INFO    - JOBS: 64
INFO    - Selecting all possible platforms per test case
INFO    - Building initial testcase list...
INFO    - 3 test configurations selected, 0 configurations discarded due to filters.
INFO    - Adding tasks to the queue...
INFO    - Total complete:    3/  -4  -75%  skipped:    7, failed:    0
INFO    - 3 of -4 tests passed (-75.00%), 0 failed, 7 skipped with 0 warnings in 5.17 seconds
INFO    - In total -4 test cases were executed on 269 out of total 272 platforms (98.90%)
INFO    - 0 tests executed on platforms, -4 tests were only built.

This is due to how total_skipped is calculated -- it needs to factor in subset when set.

And discarded number is always 0.
https://github.com/zephyrproject-rtos/zephyr/blob/34859d0d3e4075e9897dea047368f2b8e67acc2d/scripts/sanitycheck#L1080


[GSSOC'23] -  Critical Issue: Incorrect Code and Missing Functionality

Description:
I have identified a critical issue in the existing code in the file 'messenger_module.py'  that significantly impacts its functionality and usability. The code in the page  is intended to send messages via Facebook Messenger at a specified time, but it contains several errors and lacks crucial functionality. I am submitting this issue to rectify the problems and enhance the code to meet the project's requirements effectively.
Issues Identified:

Indentation Error: The code provided has inconsistent indentation, leading to a syntax error.

Undefined Function: The code attempts to call a non-existent function, difference().

Missing Import: The code uses the time module but fails to import it

Proposed Solution:
To resolve these issues and enhance the code's functionality, I suggest the following steps:

Update the indentation  to ensure consistent formatting throughout the codebase.

Replace the incorrect function name difference() with the correct function name.

I am eager to contribute to resolving this critical issue and enhance the code's functionality as part of the Girl Script Summer of Code. Please assign this issue to me, and I will promptly submit a pull request with the necessary changes and improvements.

Please assign me this issue under GSSOC'23


[Communication] - Administraion - GetAllReservations Test

Use env variable to decide whether to run GetAllReservations test

/azp run net - communication - tests

Closing as create reservation tests should not be switched off.


Update config.yaml

@hendrikebbers you are correct - in general, clowarden doesn't do destructive things :)

Good to know :)


Authority validation differ between app building and parameter

If you create a publc application and add the authority url: https://login.microsoftonline.com/ without a trailing slash the api will add the missing slash.
If you create an SilentParamater instance for a SilentRequest you are also able to add the authorityUrl. In this case the url will not check for a missing trailing slash and the parsing within the SilentRequest constructor fails with an IndexOutOfBound Exception.

PublicClientApplication pca = PublicClientApplication.builder(config.getClientId())
    .authority(config.getAuthority())
    .build();

SilentParameters.builder(Collections.singleton("User.Read"))
    .authorityUrl(config.getAuthority())
    .build();

@danilowitsch : Thanks for bringing this to our attention. I'm able to reproduce it and it definitely seems like a bug, so we should have a fix for that trailing slash problem in our next release.

Nice.

Sorry, my intention was only to comment this.

Sorry for the long delay on this, but for anyone having this issue it was addressed as part of https://github.com/AzureAD/microsoft-authentication-library-for-java/pull/431, and should be fixed as of the recent 1.12.0 release.


Add doc gen to the generate_enum_match_method assist

Implements a small extension to https://github.com/rust-analyzer/rust-analyzer/pull/7562, generating default comments. I wasn't sure if this would fit the goals of Rust-Analyzer, so I chose to split it into a separate PR. This is especially useful when writing code in a codebase which uses #![warn(missing_docs)] lint, as many production-grade libraries do.
The comments we're generating here are similar to the ones found on Option::is_some and Result::is_err. I briefly considered only generating these for pub types, but they seem small and unobtrusive enough that they're probably useful in the general case. Thanks!
Example
input
pub(crate) enum Variant {
    Undefined,
    Minor, // cursor here
    Major,
}

output
pub(crate) enum Variant {
    Undefined,
    Minor,
    Major,
}

impl Variant {
    /// Returns `true` if the variant is [`Minor`].
    pub(crate) fn is_minor(&self) -> bool {
        matches!(self, Self::Minor)
    }
}

Future Directions
This opens up the path to adding an assist for generating these comments on existing is_ methods. This would make it both easy to document new code, and update existing code with documentation.

bors d+
Yeah, it's unclear if we want this long term, and in which shape exactly. But this indeed seems unobtrusive, and we wouldn't know if we didn't try, so I am onboard.

bors r+


Can't change ENV parameters in docker-compose.yml (PalWorldSettings.ini doesn't work)

Describe the bug
I add env parameters in docker-compose.yml, also through stack editor, and it's not taken into account. I'm trying on palworld to get egg hatching timer down.
To Reproduce
Steps to reproduce the behavior: change docker-compose.yml and it doesn't work
Expected behavior
parameters get update
Actual behavior

Screenshots

OS information (IMPORTANT)
Using Marius tutorial to create the server. https://mariushosting.com/how-to-install-palworld-server-on-your-synology-nas/
it works really well
Hardware information (IMPORTANT)
DS923+
docker-compose.yml contents (IMPORTANT)

docker-compose.yml
version: "3.9"
services:
   palworld:
      image: thijsvanloef/palworld-server-docker:latest # Use :latest-arm64 tag instead of :latest for arm64 NAS
      restart: on-failure:5
      container_name: Palworld-Server
      stop_grace_period: 30s # Set to however long you are willing to wait for the container to gracefully stop
      ports:
        - 8211:8211/udp
        - 27015:27015/udp
      environment:
        - PUID=1026
        - PGID=100
        - PORT=8211
        - PLAYERS=16
        - SERVER_PASSWORD=xxxxxxxxxxxxxxxxx
        - MULTITHREADING=true
        - RCON_ENABLED=true
        - RCON_PORT=25575
        - TZ=Europe/Paris
        - ADMIN_PASSWORD=xxxxxxxxxxxxxxxxx # You can secure the access to a server by using the admin password.
        - COMMUNITY=false  # Enable this with true if you want your server to show up in the community servers tab, USE WITH SERVER_PASSWORD above.
        - SERVER_NAME=xxxxxxxxxxxxxxxxx
        - SERVER_DESCRIPTION=xxxxxxxxxxxxxxxxx
        - BUILD_OBJECT_DETERIORATION_DAMAGE_RATE=0.00000
        - DEATH_PENALTY=Item
        - PAL_EGG_DEFAULT_HATCHING_TIME=0.00000
      volumes:
         - /volume1/docker/palworld-server:/palworld:rw
</details>

Container/Host Logs (IMPORTANT)

Logs
<!-- Insert logs here -->

Additional context

Bug Report Checklist

[x] I searched for my issue on the Known issues page
[ ] I searched for my issue in the Closed issues
[ ] I have included all relevant information required above.
[ ] I am willing to fix this bug

Is egg hatch time the only parameter not working for you?
Also, after changing the egg hatch time and redeploying the stack, are you removing and putting the eggs back in the hatcher?

 I just had to place new eggs and it worked. For 7 hours I was trying all kinds of crazy solutions but didn't bother to test on new egg. I feel tired and relieved.