[BUG] Fix console warnings about 'Query' export

Describe the bug
When I try to run the OUI documentation site, I see a number of warnings in my console regarding a missing export 'Query'.
To Reproduce
Steps to reproduce the behavior:

Fork the OUI repo and clone it to your local machine
Run yarn install and then yarn start
Visit localhost:8030 on your browser (Chrome, in my case)
Right click on browser screen, choose 'Inspect'
Click on the 'Console' tab

Expected behavior
These warnings should not appear. :)
Screenshots

Host/Environment (please complete the following information):

OUI Version: 1.22.19
OSD Version (if applicable): N/A
OS: macOS Ventura 13.5.2
Browser and version: Chrome (Version 116.0.5845.187 (Official Build))

Additional context
N/A

This appears to be an issue with (maybe?) Typescript and exporting the Query/Ast types (the logs in the console that say they're warnings). See https://github.com/opensearch-project/oui/pull/770#discussion_r1199627364
I've been investigating this for a little bit and can't find anything on my end. What Babel outputs (which runs after Typescript) has correct exports, so this is maybe just a bug in Webpack.
This will probably be a matter of changing lines that say
import { Query } from './query';

into
import { Query } from './query/query';

Most likely will need this change in

src/components/search_bar/index.ts
src/components/search_bar/search_bar.test.tsx
src/components/search_bar/search_bar.tsx
src/components/search_bar/search_filters.test.tsx
src/components/search_bar/search_filters.tsx

You can see how those files are change in #770 for these changes

Is it okay to also address the issue with the 'Ast' import as a part of this bug fix, or should I just focus on the 'Query' import?

Is it okay to also address the issue with the 'Ast' import as a part of this bug fix, or should I just focus on the 'Query' import?

Yeah definitely. No need in creating multiple PR/issues if they are related :)

I was able to update the path names like you suggested and get the site to run without warnings. It appears that ./query was directing to .query/index (see screenshot below), so changing the path names to .query/query did the trick.

The issue with AST was that the original class had been named _AST but it was being imported in a couple other files as AST (no underscore).
Here is what my console looks like now. Much better!

Hi, tried to reproduce this error, but my console doesn't show any warnings.

So, I'm not sure what is causing this particular warning. Any thoughts?

Yeah, most likely you'll need to make a separate export in src/components/search_bar/search_bar.tsx:
export { Query } from './query/query';
export { AST as Ast } from './query/ast';

I tried the fix you suggested, but I'm still getting the warning in my browser console:

I've done some digging, and, as best I can tell, I think the issue is arising when 'AST' is being exported under an alias. I tried getting rid of the alias in this file and the other files that use it, and the warning disappears from my browser console:

@JohnathonBowers Go ahead with that fix to de-alias AST. I still wish we had a better understanding of what's causing the issue, and why it only appeared recently. @AMoo-Miki Is this something you recognize at all from other bundling/building work?

Sounds good, @joshuarrrr . Is it okay to open up a separate issue for this, or would you prefer that I keep the fix in the same PR as the Query path fix?

separate PR/issue is always welcome

@AMoo-Miki Is this something you recognize at all from other bundling/building work?

@joshuarrrr yes. Something similar came up when upgrading webpack: https://github.com/opensearch-project/oui/pull/770#discussion_r1199627364

I said this in Slack, but my theory is something along the lines that Webpack has some sort of bug on Mac in v4 and always in v5 that causes this. Potentially some sort of platform-specific optimization in v4, since this only popped up when people started using Mac's to develop. I checked the Babel output for both CJS and ESM for these files, and they seem just fine, even work without any problems with Node. In my mind, that only leaves Webpack as the culprit of this issue.


SimulatorManager does not reliably find all data folders

i have several apps whose app data folders cannot be found by SimulatorManager. however, they are successfully found by https://github.com/somegeekintn/SimDirs

Let me take a look on the app. May be I will find something useful :)

Thanks @bunnyhero. I updated the app already. Please try latest version :) https://github.com/tue-savvy/SimulatorManager/releases/tag/1.3

awesome, it works! :+1: thank you :)


fix (CLI) list vault secrets & service envs by name

Before
$ kontena vault list
NAME                 CREATED AT
B_SECRET        2016-01-15T14:48:21.777Z
A_SECRET        2016-02-15T14:48:38.611Z

After
$ kontena vault list
NAME                 CREATED AT
A_SECRET        2016-02-15T14:48:38.611Z
B_SECRET        2016-01-15T14:48:21.777Z

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

Nice improvement, thanks!


Font Rendering Bug Report: Misalignment of Unicode  ̂ (U+00302) when using the JuliaMono font

Description:
There appears to be an issue with the rendering of the Unicode combining circumflex accent ( ̂, U+00302) in Visual Studio Code (VSCode) when using certain fonts like JuliaMono. The accent is misaligned, shifting rightward instead of being positioned directly on top of the base Unicode Latin characters.
Example Code:
ŷ::Int, 𝐲̂::String, 𝐲̂, 𝖸̂, 𝕐̂, 𝐱̂₁³, 𝑥̂, 𝑸̂, 𝒜̂, 𝓐̂, 𝔲̂

Observed Behavior:
In the example provided, the combining circumflex accent is visibly shifted to the right for characters like 𝐲̂, 𝐱̂₁³, 𝑥̂, and 𝑸̂, resulting in misalignment. The issue occurs consistently with combining accents above these extended Latin characters.
Expected Behavior:
The circumflex accent should be correctly aligned on top of the base Unicode Latin characters in VSCode, just as it renders correctly in other text editors and terminal emulators (e.g., CotEditor, iTerm) when using the JuliaMono font.
Reproducibility:
This issue is reproducible in Visual Studio Code with JuliaMono font, but does not occur in other editors or terminal emulators that also support the font.
Steps to Reproduce:

Apply the JuliaMono font in VSCode using "editor.fontFamily": "'JuliaMono'".
Display the provided example code.
Observe the misalignment of the circumflex accent ( ̂) over specific Unicode Latin characters.

Screenshots:

Environment:

Font: JuliaMono v0.056
Operating System: macOS 14.6.1 (23G93)
Application: Visual Studio Code (VSCode) Version: 1.93.0

Version: 1.93.0
Commit: 4849ca9bdf9666755eb463db297b69e5385090e3
Date: 2024-09-04T13:02:38.431Z
Electron: 30.4.0
ElectronBuildId: 10073054
Chromium: 124.0.6367.243
Node.js: 20.15.1
V8: <IP_ADDRESS>-electron.0
OS: Darwin arm64 23.6.0

Additional Information:
See https://github.com/cormullion/juliamono/issues/214

Also, this only happens in the main editor, not in the search input bars and integrated terminal:

Hope this helps!


[formal] Find out how to reduce the number of data pattern in Jasper

If an assertion checking the the correctness of the data output (or stored) based on the given input data data_i and mask_i in prim_packer, FPV takes more than a few hours. FPV seems try to populate all the combinations of data_i` pattern.
Assume the data to be inside { values, ... } doesn't help. Need to figure out (or discuss with AE).
CC: @cindychip

Hi Eunchan,
Sorry it took so long to address this issue.
I think by adding an assumption here for data (data_i inside {...}) does help reduce the run time. What makes it still runs very slow is the mask_i variable. Due to the size of mask_i (32 bits), formal engine has to run all the combinations, thus takes a very long time.
If you add another constraint for mask_i, the tb can converge very soon.
But I think if we constrain the mask_i, it limits the functionality check of the prim_packer. I think limiting the input data width might be a better solution. I made a PR here (https://github.com/lowRISC/opentitan/pull/2431) for you to review.
Thanks,
Cindy

Thanks for the follow-up. OK I should limit the mask_i too. in HMAC :) I will close this issue.


Rollback redis to 3.0.2

problem related to https://github.com/NodeRedis/node-redis/pull/1508

Actually https://devcenter.heroku.com/changelog-items/1932


Investigate state lookup mechanism in xfrmDeleteConflictingState

Currently the logic in xfrmDeleteConflictingState doesn't exactly match the logic the kernel follows during a state lookup when adding a XFRM state. In Cilium we also check for the source IP address, while the kernel does not, see the __xfrm_state_lookup kernel function.
The following commands can easily show that the kernel does not respect the source IP when doing a state lookup during add:
> ip x s add src <IP_ADDRESS> dst <IP_ADDRESS> proto esp spi 3 mark 0x155f3e00 aead "rfc4106(gcm(aes))" 0x900d51508868a4cdeddeca7038caa20b4ffc7494 128
// no error

> ip x s add src <IP_ADDRESS> dst <IP_ADDRESS> proto esp spi 3 mark 0x155f3e00 aead "rfc4106(gcm(aes))" 0x900d51508868a4cdeddeca7038caa20b4ffc7494 128
RTNETLINK answers: File exists

We should investigate whether xfrmDeleteConflictingState should match this behavior or if we have good reasons to keep our current matching logic. In the case of the latter maybe even the kernel implementation could be fixed to match our logic.

An example where additionally checking the source address can cause issues is described in #27280.


Add barret py script

Changes:

Add a Python Barret reduction implemementation if we ever need it in the future.

@fkrause98 what is the status of this PR?

@ilitteri
It's on hold since we are not going to need Barret, should we close it?

Yeah, let's close it. We can re-open it if needed.


Why does the dataloader in RaSa return the value like this?

This is a great job!
Thank you for your open source!
Why train_ In the loader, there will be phenomena in the image, and the text can still be understood. One is the original text, and the other is the enhanced text. But the images are all transformed through the same transformer, and they are the same. Why is this design necessary?

Hi @shams2023 ,
Thank you for your interest in our work. We apply data augmentation to images to enhance the generalization capability of our model. Inspired by the contrastive learning approaches like MoCo and SimCLR, we further align the two augmentated views of the images to learn transformation-invariant visual representations.
Please let me know if you have any further questions.

嗨，
感谢您对我们工作的关注。我们将数据增强应用于图像以增强模型的泛化能力。受MoCo和SimCLR等对比学习方法的启发，我们进一步对齐图像的两个增强视图，以学习转换不变的视觉表示。
如果您有任何其他问题，请告诉我。

thanks,sogood!


BugFix: fix the bug for generating code features

Hi @martinezmatias,
When I tried code extraction from the command line and unit tests, I encounter some bugs. So I am sending this PR to fix the bugs in feature generation.
First, in P4JFeatureAnalyzer.java, I change the setting of byGenerator = False in P4J, to disable P4J generating code on the buggy location. This version only returns the original diff features. And null pointer exceptions in the file.
Second, in FeaturesOutput.java, fixed another null pointer exception and avoid generate duplicate features when no feature type specified.
Third, I reopen two unit test cases to test the feature generation. They were ignored because of the it took quite long to generate features for folder " Defects4J_all_pairs". I  have updated to "pairsD4j" with only two diffs. Now, these two functions executed for 10 seconds. I think it would be good to have these two tests to specify the code feature functions.

Hi @SophieHYe
Thanks!
It seems that test  MainComingTest.testFeaturesMain:120 is failing. is that related to one of those changes?

@SophieHYe  thanks a lot!
Very minor point: in this commit you propose 3 changes, I imagine that there are not strong relation between them. So, an alternative is to create 3 commits (and eventually one PR for each), where each one targets one of those changes.
Thanks for the PR, merged!

to create 3 commits and eventually one PR for each

yes, that's a better practice
Thanks a lot for your contribution!


GitHub Action: Even more flake8 tests

Tweeks to get us down to flake8 --ignore=F841,W503
I ran https://github.com/psf/black on the W504 files.  If that made too many changes then please close this PR and I can open another that changes the W504 files in a more surgical manner.

I ran https://github.com/psf/black on the W504 files.

Ah, I see. I jump right in to the code change without noticing the updated comment.


Adding comments

Adding comments for SameSiteMode enum values
Addresses #7486

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

@natery2000 thanks for the PR! The lack of doc comments on this enum has been a recent complaint.

Looks good to me!
@Tratcher - can you do a quick once-over and merge when ready?

Are these seemingly random failing tests anything of concern?

They're unrelated flakey tests. I'll re-run the build and hopefully you get a pass and can merge.

/AzurePipelines run AspNetCore-ci

/AzurePipelines run AspNetCore-ci

Thanks.


Is there a way to customize the tooltip backgroup color dynamically ?

Hi,
Currently, we can change the tooltip background color with the backgroundColor property on the tooltip configuration object but is there a way to change this color dynamically (if I want the color to depends on the point being displayed)? I know we can customize some colors with the labelColor callback but I would like to change the entire background color.
Here is my use case:

I display a time graph with more than one line.
When the mouse is hover a point, I would like to:

Use the line color as the entire background if there is only one point at this coordinates.
Use the default color if there are more than one point at this coordinates.

Here is an example:

In this chart, I would like to have the background color of the tooltip to be the same at the line color so, not black but purple).
Do you think it's possible with the current API?

@mjeanroy this is not possible with the current API. You might have to replace the tooltip draw function. Even better, you could factor out the background draw into a new method on the tooltip then in your case you only have to replace the drawBackground method

Thank you for your answer @etimberg (and thank you for your great library by the way 😉 )!
I'm not sure what you mean by:

You might have to replace the tooltip draw function

Is it something I can do with the current version of Chart.js or do I have to write a custom plugin?

@mjeanroy I was thinking that you would add the following before creating charts. For the most part you'd copy the implementation from https://github.com/chartjs/Chart.js/blob/master/src/core/core.tooltip.js#L673 except for https://github.com/chartjs/Chart.js/blob/master/src/core/core.tooltip.js#L692 where you'd determine the color based on the tooltipItems arrays per your requirements.
Chart.Tooltip.prototype.draw = function() {
  // New implementation
};

@etimberg Ok, that's clear, thank you.
Are you open for a PR to extract a drawBackground method (as you suggested before)?

Basic codepen example using tooltipModel:
https://codepen.io/ezra_siton/pen/dyoQeGe?editors=1011
https://www.chartjs.org/docs/2.7.3/configuration/tooltip.html#tooltip-model


This branch is out-of-date with the base branch

Hi all,
Currently, the PRs get blocked as soon as a branch gets out of date. This is really counter-productive especially in a repository as busy as this.
Note that this is a Github feature enabled in the repository settings and has nothing to do with Git merging and conflict resolution. This check "ensures pull requests targeting a matching branch have been tested with the latest code". See also: https://stackoverflow.com/questions/67332877/how-to-setup-this-branch-is-out-of-date-with-the-base-branch-in-github-repo
The purpose is make sure changes merged from other PRs are considered inside pending PRs. But how often to we actually go back and repeat those test, especially if they involve manual steps? Are there maybe better ways to automatically test independent changes as part of the CI process?
It is very often the case that changes across PRs have nothing to do with each other and this process only hinders the contributors. One could be a port change in the snapcraft file, while another is a feature addition in the code, or a typo fix in README.

Just curious if we enable this, wouldn't one of the end results be a plethora of "merge master into " commits?

@tonyespy:

Just curious if we enable this, wouldn't one of the end results be a plethora of "merge master into " commits?

If you are referring to the checkbox highlighted by Ernesto, it is already enabled in this repo. That's why we need to rebase or merge master into our branches (e.g. 1, 2) as soon as they go out of date (the topic of this issue).

@farshidtz Correct. I believe it's checked for all EdgeX repos.

Will discuss during the architects meeting Dec 20th

Added to Monthly architect's meeting for December

Per Monthly Architect's meeting: Make the change on edgex-docs (require branch to be up to date check box).  For edgex-go, ask devops to give us indications of what the impact would be to CI pipelines (what happens if a build failure occurs with this new workflow; can they put a verify job in place?).  Allow the TSC to make a decision after getting the readout from Devops.
@ernestojeda - please reach out to @farshidtz @tonyespy @bnevis-i

can they put a verify job in place?

To elaborate: the suggestion in the meeting was to replace the enforced branch update on the user-side (and git history) with the same operation done as part of the current CI tests. As a result, the CI workflow would first merge main into the PR's branch (in its local copy) and then perform the tests as usual and raise any possible failures.
This is to make sure that in the absence of enforced branch updates, relevant changes added by independent branches still get tested together before merging a PR.

Right.  The enforced branch update adds 30 minutes to merge every approved PR, during which time another PR that is up-to-date can be committed, resulting in anther 30 minute round-trip.  Not only that, but the commit log gets cluttered with "merge from master" commits.  Give the loosly-coupled nature of edgex, it is rare that a behavior change in one component will be incompatible with another behavior change made in another component to justify the additional round trips.

@bnevis-i , We should be using Squash and Merge to avoid the cluttered commits.

Instead, in GitHub do "Fetch upstream" on the main branch of your fork, then on your branch
git fetch
git rebase -S origin/main
git push --force

As of today Jan 13, 2022, the branch protection rules have been modified for the following repositories:

edgex-go
edgex-docs

We discussed this at length on the Core Working Group call (27th Jan) - recording available at https://wiki.edgexfoundry.org/display/FA/Core+Working+Group
Conclusion is that this is likely to be a rare occurrence and we need to be aware of it, but it is considered an acceptable risk right now. Our DevOps pipeline process will catch errors & test fails and we'll need to fix those if/when it happens. Let's monitor to see if and how frequently this occurs, but closing this ticket for now.


Fedora 37, not able to initialise

Hello!
Problem on Fedora 37

$ ./build/dist/Outline-Client.AppImage
/usr/bin/AppImageLauncher: /lib64/libcurl.so.4: no version information available (required by /usr/bin/../lib/x86_64-linux-gnu/appimagelauncher/libappimageupdate.so)
QSocketNotifier: Can only be used with threads started with QThread
another instance is running - exiting
Outline is starting
libva error: vaGetDriverNameByIndex() failed with unknown libva error, driver_name = (null)
loading web app from file:///tmp/.mount_Outlini0v1lU/resources/app.asar/www/index_electron.html?appName=Outline
MESA-LOADER: failed to retrieve device information
MESA-LOADER: failed to open nvidia-drm: /usr/lib64/dri/nvidia-drm_dri.so: cannot open shared object file: Permission denied (search paths /usr/lib64/dri, suffix _dri)
failed to load driver: nvidia-drm
MESA-LOADER: failed to open zink: /usr/lib64/dri/zink_dri.so: cannot open shared object file: Permission denied (search paths /usr/lib64/dri, suffix _dri)
failed to load driver: zink
MESA-LOADER: failed to open kms_swrast: /usr/lib64/dri/kms_swrast_dri.so: cannot open shared object file: Permission denied (search paths /usr/lib64/dri, suffix _dri)
failed to load driver: kms_swrast
MESA-LOADER: failed to open swrast: /usr/lib64/dri/swrast_dri.so: cannot open shared object file: Permission denied (search paths /usr/lib64/dri, suffix _dri)
failed to load swrast driver
Could not load active tunnel:  [Error: ENOENT: no such file or directory, open '/home/stark/.config/Outline/connection_store'] {
  errno: -2,
  code: 'ENOENT',
  syscall: 'open',
  path: '/home/stark/.config/Outline/connection_store'
}
Checking for update

touch ~/.config/Outline/connection_store
Solve the issue

Same issue with Fedora 38, Outline 1.10.1
touch ~/.config/Outline/connection_store
Not working for me

@maddyhof
The problem is that Outline requires the shasum package to run, which is not in the standard Fedora delivery.:
[root@svolf-pc Apps]# ./Outline-Client.AppImage --no-sandbox
[40297:0527/223100.057129:ERROR:bus.cc(398)] Failed to connect to the bus: Could not parse server address: Unknown address type (examples of valid types are "tcp" and on UNIX "unix")
[40297:0527/223100.057199:ERROR:bus.cc(398)] Failed to connect to the bus: Could not parse server address: Unknown address type (examples of valid types are "tcp" and on UNIX "unix")
Outline is starting
src/gbm_drv_common.c:57: GBM-DRV error (get_bytes_per_component): Unknown or not supported format: 538982482

src/gbm_drv_common.c:57: GBM-DRV error (get_bytes_per_component): Unknown or not supported format: 943215175

src/gbm_drv_common.c:57: GBM-DRV error (get_bytes_per_component): Unknown or not supported format: 842094158

src/gbm_drv_common.c:57: GBM-DRV error (get_bytes_per_component): Unknown or not supported format: 842094169

src/gbm_drv_common.c:57: GBM-DRV error (get_bytes_per_component): Unknown or not supported format: 808530000

src/gbm_drv_common.c:57: GBM-DRV error (get_bytes_per_component): Unknown or not supported format: 538982482

src/gbm_drv_common.c:57: GBM-DRV error (get_bytes_per_component): Unknown or not supported format: 943215175

src/gbm_drv_common.c:57: GBM-DRV error (get_bytes_per_component): Unknown or not supported format: 842094158

src/gbm_drv_common.c:57: GBM-DRV error (get_bytes_per_component): Unknown or not supported format: 842094169

src/gbm_drv_common.c:57: GBM-DRV error (get_bytes_per_component): Unknown or not supported format: 808530000

src/gbm_drv_common.c:57: GBM-DRV error (get_bytes_per_component): Unknown or not supported format: 538982482

src/gbm_drv_common.c:57: GBM-DRV error (get_bytes_per_component): Unknown or not supported format: 943215175

src/gbm_drv_common.c:57: GBM-DRV error (get_bytes_per_component): Unknown or not supported format: 842094158

src/gbm_drv_common.c:57: GBM-DRV error (get_bytes_per_component): Unknown or not supported format: 842094169

src/gbm_drv_common.c:57: GBM-DRV error (get_bytes_per_component): Unknown or not supported format: 808530000

src/gbm_drv_common.c:57: GBM-DRV error (get_bytes_per_component): Unknown or not supported format: 538982482

src/gbm_drv_common.c:57: GBM-DRV error (get_bytes_per_component): Unknown or not supported format: 943215175

src/gbm_drv_common.c:57: GBM-DRV error (get_bytes_per_component): Unknown or not supported format: 842094158

src/gbm_drv_common.c:57: GBM-DRV error (get_bytes_per_component): Unknown or not supported format: 842094169

src/gbm_drv_common.c:57: GBM-DRV error (get_bytes_per_component): Unknown or not supported format: 808530000

src/gbm_drv_common.c:57: GBM-DRV error (get_bytes_per_component): Unknown or not supported format: 538982482

src/gbm_drv_common.c:57: GBM-DRV error (get_bytes_per_component): Unknown or not supported format: 943215175

src/gbm_drv_common.c:57: GBM-DRV error (get_bytes_per_component): Unknown or not supported format: 842094158

src/gbm_drv_common.c:57: GBM-DRV error (get_bytes_per_component): Unknown or not supported format: 842094169

src/gbm_drv_common.c:57: GBM-DRV error (get_bytes_per_component): Unknown or not supported format: 808530000

loading web app from file:///tmp/.mount_OutlinkFcmb4/resources/app.asar/www/index_electron.html?appName=Outline
Could not load active tunnel:  [Error: ENOENT: no such file or directory, open '/root/.config/Outline/connection_store'] {
  errno: -2,
  code: 'ENOENT',
  syscall: 'open',
  path: '/root/.config/Outline/connection_store'
}
Checking for update
Update for version 1.10.1 is not available (latest version: 1.10.1, downgrade is disallowed).
connecting to 3714382e-cfff-4dfe-a4c3-1b87fdc84846...
using tun2socks to check connectivity
[EXIT - tun2socks]: Exited with code 0
UDP support: true
starting routing daemon
Routing daemon socket setup failed Error: connect ENOENT /var/run/outline_controller
    at PipeConnectWrap.afterConnect [as oncomplete] (node:net:1157:16) {
  errno: -2,
  code: 'ENOENT',
  syscall: 'connect',
  address: '/var/run/outline_controller'
}
could not connect: w (routing daemon is not running)
tun2socks started
[EXIT - tun2socks]: Exited with code 0
tun2socks exited with no errors
disconnected from 3714382e-cfff-4dfe-a4c3-1b87fdc84846
installing outline routing service...
copying service installation files to /tmp/iIsyUN
all service installation files copied to /tmp/iIsyUN successfully
trying to run command as root:  trap "/usr/bin/chattr -R -i /tmp/iIsyUN" EXIT; /usr/bin/chattr -R +i /tmp/iIsyUN && /usr/bin/echo "e0644a3750b6c770e4f2f2e2e7de0de1e46386eed733d7841f35d9e4aa53c1cd  /tmp/iIsyUN/install_linux_service.sh" | /usr/bin/shasum -a 256 -c && /usr/bin/echo "0a2d2b874d4d200145b1e3a7cc5818fb27e5c0ed69ebb4f4057e066e0b7fdc4a  /tmp/iIsyUN/OutlineProxyController" | /usr/bin/shasum -a 256 -c && /usr/bin/echo "8bfab08036441bcbb558927bdecd97eebf8c2d365a2a7fdb6c0032c2afc965a2  /tmp/iIsyUN/outline_proxy_controller.service" | /usr/bin/shasum -a 256 -c && "/tmp/iIsyUN/install_linux_service.sh" "root"

/bin/bash: строка 1: /usr/bin/shasum: Нет такого файла или каталога
/usr/bin/echo: ошибка записи: Обрыв канала

command is running as root but failed:  Error: Command failed: cd "/home/svolf/Apps"; "/usr/bin/pkexec" --disable-internal-agent /bin/bash -c "echo SUDOPROMPT; trap \"/usr/bin/chattr -R -i /tmp/iIsyUN\" EXIT; /usr/bin/chattr -R +i /tmp/iIsyUN && /usr/bin/echo \"e0644a3750b6c770e4f2f2e2e7de0de1e46386eed733d7841f35d9e4aa53c1cd  /tmp/iIsyUN/install_linux_service.sh\" | /usr/bin/shasum -a 256 -c && /usr/bin/echo \"0a2d2b874d4d200145b1e3a7cc5818fb27e5c0ed69ebb4f4057e066e0b7fdc4a  /tmp/iIsyUN/OutlineProxyController\" | /usr/bin/shasum -a 256 -c && /usr/bin/echo \"8bfab08036441bcbb558927bdecd97eebf8c2d365a2a7fdb6c0032c2afc965a2  /tmp/iIsyUN/outline_proxy_controller.service\" | /usr/bin/shasum -a 256 -c && \"/tmp/iIsyUN/install_linux_service.sh\" \"root\""
/bin/bash: строка 1: /usr/bin/shasum: Нет такого файла или каталога
/usr/bin/echo: ошибка записи: Обрыв канала

    at ChildProcess.exithandler (node:child_process:408:12)
    at ChildProcess.emit (node:events:526:28)
    at ChildProcess.emit (node:domain:475:12)
    at maybeClose (node:internal/child_process:1092:16)
    at Socket.<anonymous> (node:internal/child_process:451:11)
    at Socket.emit (node:events:526:28)
    at Socket.emit (node:domain:475:12)
    at Pipe.<anonymous> (node:net:687:12) {
  killed: false,
  code: 127,
  signal: null,
  cmd: 'cd "/home/svolf/Apps"; "/usr/bin/pkexec" --disable-internal-agent /bin/bash -c "echo SUDOPROMPT; trap \\"/usr/bin/chattr -R -i /tmp/iIsyUN\\" EXIT; /usr/bin/chattr -R +i /tmp/iIsyUN && /usr/bin/echo \\"e0644a3750b6c770e4f2f2e2e7de0de1e46386eed733d7841f35d9e4aa53c1cd  /tmp/iIsyUN/install_linux_service.sh\\" | /usr/bin/shasum -a 256 -c && /usr/bin/echo \\"0a2d2b874d4d200145b1e3a7cc5818fb27e5c0ed69ebb4f4057e066e0b7fdc4a  /tmp/iIsyUN/OutlineProxyController\\" | /usr/bin/shasum -a 256 -c && /usr/bin/echo \\"8bfab08036441bcbb558927bdecd97eebf8c2d365a2a7fdb6c0032c2afc965a2  /tmp/iIsyUN/outline_proxy_controller.service\\" | /usr/bin/shasum -a 256 -c && \\"/tmp/iIsyUN/install_linux_service.sh\\" \\"root\\""'
}
connecting to 3714382e-cfff-4dfe-a4c3-1b87fdc84846...
using tun2socks to check connectivity
[EXIT - tun2socks]: Exited with code 0
UDP support: true
starting routing daemon
Routing daemon socket setup failed Error: connect ENOENT /var/run/outline_controller
    at PipeConnectWrap.afterConnect [as oncomplete] (node:net:1157:16) {
  errno: -2,
  code: 'ENOENT',
  syscall: 'connect',
  address: '/var/run/outline_controller'
}
could not connect: w (routing daemon is not running)
[EXIT - tun2socks]: Killed by signal SIGTERM
tun2socks terminated due to Error: Process terminated by signal: SIGTERM
disconnected from 3714382e-cfff-4dfe-a4c3-1b87fdc84846
installing outline routing service...
copying service installation files to /tmp/iX37J6
all service installation files copied to /tmp/iX37J6 successfully
trying to run command as root:  trap "/usr/bin/chattr -R -i /tmp/iX37J6" EXIT; /usr/bin/chattr -R +i /tmp/iX37J6 && /usr/bin/echo "e0644a3750b6c770e4f2f2e2e7de0de1e46386eed733d7841f35d9e4aa53c1cd  /tmp/iX37J6/install_linux_service.sh" | /usr/bin/shasum -a 256 -c && /usr/bin/echo "0a2d2b874d4d200145b1e3a7cc5818fb27e5c0ed69ebb4f4057e066e0b7fdc4a  /tmp/iX37J6/OutlineProxyController" | /usr/bin/shasum -a 256 -c && /usr/bin/echo "8bfab08036441bcbb558927bdecd97eebf8c2d365a2a7fdb6c0032c2afc965a2  /tmp/iX37J6/outline_proxy_controller.service" | /usr/bin/shasum -a 256 -c && "/tmp/iX37J6/install_linux_service.sh" "root"
/tmp/iX37J6/install_linux_service.sh: OK
/tmp/iX37J6/OutlineProxyController: OK
/tmp/iX37J6/outline_proxy_controller.service: OK
user root has been added to outlinevpn group

+ readonly PREFIX=/usr/local
+ PREFIX=/usr/local
+ readonly SERVICE_DIR=/etc/systemd/system
+ SERVICE_DIR=/etc/systemd/system
+ readonly SERVICE_NAME=outline_proxy_controller.service
+ SERVICE_NAME=outline_proxy_controller.service
+ readonly GROUP_NAME=outlinevpn
+ GROUP_NAME=outlinevpn
++ dirname /tmp/iX37J6/install_linux_service.sh
+ readonly SCRIPT_DIR=/tmp/iX37J6
+ SCRIPT_DIR=/tmp/iX37J6
+ /usr/sbin/groupadd -f outlinevpn
+ /usr/bin/id root
+ /usr/sbin/usermod -aG outlinevpn root
+ /usr/bin/echo 'user root has been added to outlinevpn group'
+ /usr/bin/cp -f /tmp/iX37J6/OutlineProxyController /usr/local/sbin
+ /usr/bin/cp -f /tmp/iX37J6/outline_proxy_controller.service /etc/systemd/system/
+ /usr/bin/id root
++ id -u root
+ owneruid=0
+ /usr/bin/sed -i s/--owning-user-id=-1/--owning-user-id=0/g /etc/systemd/system/outline_proxy_controller.service
+ /usr/bin/systemctl daemon-reload
+ /usr/bin/systemctl enable outline_proxy_controller.service
Created symlink /etc/systemd/system/multi-user.target.wants/outline_proxy_controller.service → /etc/systemd/system/outline_proxy_controller.service.
+ /usr/bin/systemctl restart outline_proxy_controller.service
+ /usr/bin/sleep 2

outline routing service installed successfully
connecting to 3714382e-cfff-4dfe-a4c3-1b87fdc84846...
using tun2socks to check connectivity
[EXIT - tun2socks]: Exited with code 0
UDP support: true
starting routing daemon
tun2socks started
connected to 3714382e-cfff-4dfe-a4c3-1b87fdc84846
reconnecting to 3714382e-cfff-4dfe-a4c3-1b87fdc84846
reconnected to 3714382e-cfff-4dfe-a4c3-1b87fdc84846
using tun2socks to check connectivity
[EXIT - tun2socks]: Exited with code 0
[EXIT - tun2socks]: Exited with code 0
tun2socks exited with no errors
disconnected from 3714382e-cfff-4dfe-a4c3-1b87fdc84846

I solved this by adding this package manually:
sudo dnf install shasum

@SnowVolf

BUT, Outline working only if i launch client as root user with sudo su, this is a critical flaw...

Please check non-sudo after restarting. In my case shasum was an issue for both. Only difference, that I was able to see it with sudo.

@kaktoforum
in my case all shasum binaries stored in propper paths:
[svolf@svolf-pc bin]$ which shasum
/usr/bin/shasum

still not working without sudo permissions

On ubuntu its work fine. On Fedora 38 its looks like working, can connect to server, but as a result have no internet on machine.  When try to run it with sudo - its not working at all
``
`sudo ./Outline-Client.AppImage --no-sandbox
[sudo] password for user:
[6768:0824/131503.500402:ERROR:bus.cc(398)] Failed to connect to the bus: Could not parse server address: Unknown address type (examples of valid types are "tcp" and on UNIX "unix")
[6768:0824/131503.500461:ERROR:bus.cc(398)] Failed to connect to the bus: Could not parse server address: Unknown address type (examples of valid types are "tcp" and on UNIX "unix")
Authorization required, but no authorization protocol specified
Authorization required, but no authorization protocol specified
Authorization required, but no authorization protocol specified
Authorization required, but no authorization protocol specified
[6831:0824/131503.560219:ERROR:angle_platform_impl.cc(43)] Display.cpp:977 (initialize): ANGLE Display::initialize error 12289: Could not open the default X display.
[6831:0824/131503.560303:ERROR:gl_surface_egl.cc(852)] EGL Driver message (Critical) eglInitialize: Could not open the default X display.
[6831:0824/131503.560343:ERROR:gl_surface_egl.cc(1489)] eglInitialize OpenGL failed with error EGL_NOT_INITIALIZED, trying next display type
Authorization required, but no authorization protocol specified
[6831:0824/131503.561323:ERROR:angle_platform_impl.cc(43)] Display.cpp:977 (initialize): ANGLE Display::initialize error 12289: Could not open the default X display.
[6831:0824/131503.561359:ERROR:gl_surface_egl.cc(852)] EGL Driver message (Critical) eglInitialize: Could not open the default X display.
[6831:0824/131503.561390:ERROR:gl_surface_egl.cc(1489)] eglInitialize OpenGLES failed with error EGL_NOT_INITIALIZED
[6831:0824/131503.561422:ERROR:gl_ozone_egl.cc(21)] GLSurfaceEGL::InitializeOneOff failed.
Outline is starting
[6831:0824/131503.571451:ERROR:viz_main_impl.cc(186)] Exiting GPU process due to errors during initialization
loading web app from file:///tmp/.mount_OutlintNah7N/resources/app.asar/www/index_electron.html?appName=Outline
Could not load active tunnel:  [Error: ENOENT: no such file or directory, open '/root/.config/Outline/connection_store'] {
errno: -2,
code: 'ENOENT',
syscall: 'open',
path: '/root/.config/Outline/connection_store'
}
Checking for update
(outline-client:6768): libappindicator-WARNING **: 13:15:03.676: Unable to get the session bus: Unknown or unsupported transport “disabled” for address “disabled:”
(outline-client:6768): LIBDBUSMENU-GLIB-WARNING **: 13:15:03.676: Unable to get session bus: Unknown or unsupported transport “disabled” for address “disabled:”
Authorization required, but no authorization protocol specified
Authorization required, but no authorization protocol specified
Authorization required, but no authorization protocol specified
Authorization required, but no authorization protocol specified
[6847:0824/131503.679905:ERROR:angle_platform_impl.cc(43)] Display.cpp:977 (initialize): ANGLE Display::initialize error 12289: Could not open the default X display.
[6847:0824/131503.679986:ERROR:gl_surface_egl.cc(852)] EGL Driver message (Critical) eglInitialize: Could not open the default X display.
[6847:0824/131503.680021:ERROR:gl_surface_egl.cc(1489)] eglInitialize OpenGL failed with error EGL_NOT_INITIALIZED, trying next display type
Authorization required, but no authorization protocol specified
[6847:0824/131503.680132:ERROR:angle_platform_impl.cc(43)] Display.cpp:977 (initialize): ANGLE Display::initialize error 12289: Could not open the default X display.
[6847:0824/131503.680176:ERROR:gl_surface_egl.cc(852)] EGL Driver message (Critical) eglInitialize: Could not open the default X display.
[6847:0824/131503.680210:ERROR:gl_surface_egl.cc(1489)] eglInitialize OpenGLES failed with error EGL_NOT_INITIALIZED
[6847:0824/131503.680238:ERROR:gl_ozone_egl.cc(21)] GLSurfaceEGL::InitializeOneOff failed.
[6847:0824/131503.681028:ERROR:viz_main_impl.cc(186)] Exiting GPU process due to errors during initialization
Authorization required, but no authorization protocol specified
Authorization required, but no authorization protocol specified
Authorization required, but no authorization protocol specified
Authorization required, but no authorization protocol specified
[6863:0824/131503.696115:ERROR:angle_platform_impl.cc(43)] Display.cpp:977 (initialize): ANGLE Display::initialize error 12289: Could not open the default X display.
[6863:0824/131503.696184:ERROR:gl_surface_egl.cc(852)] EGL Driver message (Critical) eglInitialize: Could not open the default X display.
[6863:0824/131503.696217:ERROR:gl_surface_egl.cc(1489)] eglInitialize OpenGL failed with error EGL_NOT_INITIALIZED, trying next display type
Authorization required, but no authorization protocol specified
[6863:0824/131503.696316:ERROR:angle_platform_impl.cc(43)] Display.cpp:977 (initialize): ANGLE Display::initialize error 12289: Could not open the default X display.
[6863:0824/131503.696348:ERROR:gl_surface_egl.cc(852)] EGL Driver message (Critical) eglInitialize: Could not open the default X display.
[6863:0824/131503.696377:ERROR:gl_surface_egl.cc(1489)] eglInitialize OpenGLES failed with error EGL_NOT_INITIALIZED
[6863:0824/131503.696413:ERROR:gl_ozone_egl.cc(21)] GLSurfaceEGL::InitializeOneOff failed.
[6863:0824/131503.697243:ERROR:viz_main_impl.cc(186)] Exiting GPU process due to errors during initialization
Authorization required, but no authorization protocol specified
Authorization required, but no authorization protocol specified
Authorization required, but no authorization protocol specified
Authorization required, but no authorization protocol specified
[6870:0824/131503.714984:ERROR:angle_platform_impl.cc(43)] Display.cpp:977 (initialize): ANGLE Display::initialize error 12289: Could not open the default X display.
[6870:0824/131503.715063:ERROR:gl_surface_egl.cc(852)] EGL Driver message (Critical) eglInitialize: Could not open the default X display.
[6870:0824/131503.715102:ERROR:gl_surface_egl.cc(1489)] eglInitialize OpenGL failed with error EGL_NOT_INITIALIZED, trying next display type
Authorization required, but no authorization protocol specified
[6870:0824/131503.715219:ERROR:angle_platform_impl.cc(43)] Display.cpp:977 (initialize): ANGLE Display::initialize error 12289: Could not open the default X display.
[6870:0824/131503.715263:ERROR:gl_surface_egl.cc(852)] EGL Driver message (Critical) eglInitialize: Could not open the default X display.
[6870:0824/131503.715298:ERROR:gl_surface_egl.cc(1489)] eglInitialize OpenGLES failed with error EGL_NOT_INITIALIZED
[6870:0824/131503.715331:ERROR:gl_ozone_egl.cc(21)] GLSurfaceEGL::InitializeOneOff failed.
[6870:0824/131503.716288:ERROR:viz_main_impl.cc(186)] Exiting GPU process due to errors during initialization
Authorization required, but no authorization protocol specified
Authorization required, but no authorization protocol specified
Authorization required, but no authorization protocol specified
Authorization required, but no authorization protocol specified
[6882:0824/131503.734329:ERROR:angle_platform_impl.cc(43)] DisplayVkXcb.cpp:59 (initialize): xcb_connect() failed, error 1
[6882:0824/131503.734402:ERROR:angle_platform_impl.cc(43)] Display.cpp:977 (initialize): ANGLE Display::initialize error 12289: Not initialized.
[6882:0824/131503.734455:ERROR:gl_surface_egl.cc(852)] EGL Driver message (Critical) eglInitialize: Not initialized.
[6882:0824/131503.734496:ERROR:gl_surface_egl.cc(1489)] eglInitialize SwANGLE failed with error EGL_NOT_INITIALIZED
[6882:0824/131503.734540:ERROR:gl_ozone_egl.cc(21)] GLSurfaceEGL::InitializeOneOff failed.
[6882:0824/131503.735411:ERROR:viz_main_impl.cc(186)] Exiting GPU process due to errors during initialization
Authorization required, but no authorization protocol specified
Authorization required, but no authorization protocol specified
[6889:0824/131503.745306:ERROR:gpu_init.cc(481)] Passthrough is not supported, GL is disabled, ANGLE is
[6889:0824/131503.749923:ERROR:x11_software_bitmap_presenter.cc(141)] XGetWindowAttributes failed for window 31457283
Update for version 1.11.0 is not available (latest version: 1.11.0, downgrade is disallowed).

@DanilBorchevkin I had the same issue on Fedora 39, your instruction has helped, thanks.

@DanilBorchevkin you are the best !!!


Test CodePrinter and NoopCodePrinter

Actually, for trivial stuff like this, I'm just going to merge in.


Add 3rd/4th tier Solar Panels

I am working with solar generation and have about 10 advanced generators powering my ender quarry. My problem is trying to bridge the gap between this lower tier power, and getting a reaction chain built for a reactor. Anyway we could add more tiers to the solar panels?

I'm looking into some form of power to bridge the gap between low power (wind, solar) and high power (fusion) generation- maybe hydroelectric, maybe small-scale fission.  I'd rather not flesh out tiers for generators, too much work and not very fun

Thanks for the replay. I appreciate it. If you ever need any assistance, I
wouldn't mind learning Java Programming.
On Mon, Jul 6, 2015 at 2:16 PM, Aidan<EMAIL_ADDRESS>wrote:

Closed #2790 https://github.com/aidancbrady/Mekanism/issues/2790.
—
Reply to this email directly or view it on GitHub
https://github.com/aidancbrady/Mekanism/issues/2790#event-348911733.

--
Brad


Warning level codacy issues: Unnecessary use of fully qualified name ''{0}'' due to existing {2}import ''{1}''

Unnecessary use of fully qualified name ''{0}'' due to existing {2}import ''{1}''
https://app.codacy.com/manual/jdi-testing/jdi-dark/issues?&filters=W3siaWQiOiJMYW5ndWFnZSIsInZhbHVlcyI6W119LHsiaWQiOiJDYXRlZ29yeSIsInZhbHVlcyI6W119LHsiaWQiOiJMZXZlbCIsInZhbHVlcyI6W119LHsiaWQiOiJQYXR0ZXJuIiwidmFsdWVzIjpbIjU1NDEiXX1d

Done.


ollama run fails on start

➜  ~ ollama run llama2
⠋   Error: template: :2:11: executing "" at <.Context>: can't evaluate field Context in type struct { First bool; System string; Prompt string }

➜  ~ ollama --version
ollama version 0.1.13

Ran into this issue after running ollama run llama2. Occurs if server is running or if not & issue persisted after updating the server.

Can you provide some more info on your setup? How did you install ollama? What platform are you on?

Hey @j-sneh can you try ollama pull llama2 and then try running it again?

I think #1244 will partially fix this, but I think you're running a really old model that needs to be updated.

I'm on an M1 Mac, macOS Moneterey — re-pulling the model fixed the issue. My old copy of the model was a couple of months old.


Made migrations MySQL safe

In two migration queries, the following part of the statement didn't play well with MySQL:
WITH statement_summary AS ( ...

This pull request breaks such nested queries into atomic statements. Alternatively, I'd also tried using dialectOptions: { multipleStatements: true } for Sequelize, but that was causing the test setup to fail 🤷.

Looks like there are genuine build failures. I'll take a look at these over the course of the week!

@kunagpal does the existing migration fail silently? Not sure why build wasn't previously failing. Or is there some other issue with view query?
As for changing an existing migration, we'd need to make sure the views are altered on existing SQLPad setups that have run these migrations already.
Ideally, we'd add another migration if possible to drop the view that doesn't work, then add 2 new views if that's the route we'd like to go.

@rickbergfalk There's an error message about the migration query failing:
{"level":30,"time":"2021-09-07T04:02:17.206Z","pid":8,"hostname":"sqlpad","name":"sqlpad-app","msg":"Running migrations"}
{"level":30,"time":"2021-09-07T04:02:17.232Z","pid":8,"hostname":"sqlpad","name":"sqlpad-app","msg":"== 04-00400-query-history-to-batch: migrating ======="}
{"level":50,"time":"2021-09-07T04:02:17.236Z","pid":8,"hostname":"sqlpad","name":"sqlpad-app","name":"SequelizeDatabaseError","parent":{"code":"ER_PARSE_ERROR","errno":1064,"sqlState":"42000","sqlMessage":"You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'statement_summary AS (\n          SELECT \n            batch_id, \n            SUM(' at line 1","sql":"CREATE VIEW vw_query_history AS \n        WITH statement_summary AS (\n          SELECT \n            batch_id, \n            SUM(row_count) AS row_count, \n            MAX(CAST(incomplete AS UNSIGNED)) AS incomplete\n          FROM \n            statements\n          GROUP BY \n            batch_id\n        )\n        SELECT \n          b.id,\n          b.query_id,\n          b.name AS query_name,\n          b.connection_id,\n          c.name AS connection_name,\n          b.status,\n          b.start_time,\n          b.stop_time,\n          b.duration_ms,\n          b.selected_text AS query_text,\n          b.user_id,\n          u.email AS user_email,\n          ss.row_count,\n          ss.incomplete\n        FROM \n          batches b\n          LEFT JOIN users u ON b.user_id = u.id\n          LEFT JOIN connections c ON b.connection_id = c.id\n          LEFT JOIN statement_summary ss ON b.id = ss.batch_id"},"original":{"code":"ER_PARSE_ERROR","errno":1064,"sqlState":"42000","sqlMessage":"You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'statement_summary AS (\n          SELECT \n            batch_id, \n            SUM(' at line 1","sql":"CREATE VIEW vw_query_history AS \n        WITH statement_summary AS (\n          SELECT \n            batch_id, \n            SUM(row_count) AS row_count, \n            MAX(CAST(incomplete AS UNSIGNED)) AS incomplete\n          FROM \n            statements\n          GROUP BY \n            batch_id\n        )\n        SELECT \n          b.id,\n          b.query_id,\n          b.name AS query_name,\n          b.connection_id,\n          c.name AS connection_name,\n          b.status,\n          b.start_time,\n          b.stop_time,\n          b.duration_ms,\n          b.selected_text AS query_text,\n          b.user_id,\n          u.email AS user_email,\n          ss.row_count,\n          ss.incomplete\n        FROM \n          batches b\n          LEFT JOIN users u ON b.user_id = u.id\n          LEFT JOIN connections c ON b.connection_id = c.id\n          LEFT JOIN statement_summary ss ON b.id = ss.batch_id"},"sql":"CREATE VIEW vw_query_history AS \n        WITH statement_summary AS (\n          SELECT \n            batch_id, \n            SUM(row_count) AS row_count, \n            MAX(CAST(incomplete AS UNSIGNED)) AS incomplete\n          FROM \n            statements\n          GROUP BY \n            batch_id\n        )\n        SELECT \n          b.id,\n          b.query_id,\n          b.name AS query_name,\n          b.connection_id,\n          c.name AS connection_name,\n          b.status,\n          b.start_time,\n          b.stop_time,\n          b.duration_ms,\n          b.selected_text AS query_text,\n          b.user_id,\n          u.email AS user_email,\n          ss.row_count,\n          ss.incomplete\n        FROM \n          batches b\n          LEFT JOIN users u ON b.user_id = u.id\n          LEFT JOIN connections c ON b.connection_id = c.id\n          LEFT JOIN statement_summary ss ON b.id = ss.batch_id","stack":"SequelizeDatabaseError: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'statement_summary AS (\n          SELECT \n            batch_id, \n            SUM(' at line 1\n    at Query.formatError (/usr/app/node_modules/@rickbergfalk/sequelize/lib/dialects/mysql/query.js:265:16)\n    at Query.run (/usr/app/node_modules/@rickbergfalk/sequelize/lib/dialects/mysql/query.js:77:18)\n    at processTicksAndRejections (internal/process/task_queues.js:95:5)\n    at async /usr/app/node_modules/@rickbergfalk/sequelize/lib/sequelize.js:619:16\n    at async Object.up (/usr/app/migrations/04-00400-query-history-to-batch.js:21:3)","type":"Error","msg":"Error starting SQLPad"}

I'm able to run the migration with either: atomic queries or dialectOptions: {multipleStatements: true}. The view itself is valid, it's just a question of how the setup is happening. More importantly, the setup I'm working with uses MySQL 5.7, while the build has been running on MySQL 8 😅

Ah ok. I'm wondering if we should add another round of tests for MySQL 5.7 as a backing database

@kunagpal a potential solution if you want to try that doesn't require changing the dialect options is to change the view to use a subquery instead of a CTE. So it'd be something like
CREATE VIEW vw_query_history AS
  SELECT
    b.id,
    b.query_id,
    b.name AS query_name,
    b.connection_id,
    c.name AS connection_name,
    b.status,
    b.start_time,
    b.stop_time,
    b.duration_ms,
    b.selected_text AS query_text,
    b.user_id,
    u.email AS user_email,
    ss.row_count,
    ss.incomplete
  FROM
    batches b
    LEFT JOIN users u ON b.user_id = u.id
    LEFT JOIN connections c ON b.connection_id = c.id
    LEFT JOIN (
      SELECT
        batch_id,
        SUM(row_count) AS row_count,
        MAX(CAST(incomplete AS ${castType})) AS incomplete
      FROM
        statements
      GROUP BY
        batch_id
    ) ss ON b.id = ss.batch_id

@rickbergfalk Looks like you're right on both counts 😁

Since SQLPad uses GitHub actions, could you add a 5.7 variant to the build too?
I'll go ahead and try this subquery locally and let you know how that goes. The pull request will be modified accordingly!

@rickbergfalk Thanks for the left join suggestion, looks like we're good to go.

@rickbergfalk I couldn't replicate the build failure with Node 14 locally. Could you rerun the build so I can confirm that this isn't a one-off? 😄

Thanks @kunagpal!

@rickbergfalk Looks like the same test that failed before is now failing for mariadb and postgres. I'll check this out later today and see what's going on 🤔


Request refresh button

Expected behavior
Refresh list of container
Actual behavior
No button (Or i haven't seen)
Information about the Issue
Affter I run or remove container it's not refresh in the kitematic
Steps to reproduce the behavior

I run or remove instance container by command line
In kitematic it's no change about container

Same here +1

+1

any update on this?

it like to have one too


ChoiceInput Node Edit

This is an edit to the ChoiceInput "side node" - I'm not sure if it has an official name.
the box has gotten a little too small and it crushes in on the circle, making it look odd. We also need to match up the Hoover state shape.
The image below show that we have the top to radius but this one has rounded on the bottom as well.
hoover color should match the other nodes..

Mac 10.15.2
Chrome

addressed via https://github.com/microsoft/BotFramework-Composer/pull/1950


[1Password] Search should show login names instead of category

Extension – 1Password
Author: @khasbilegt
Describe the feature and the current behavior/state.
Current behavior
When I search for a login in Raycast, and that search result shows multiple items with the same title, I'm unable to distinguish between accounts.
See image:

Desired behavior
By default, 1Password saves a login with just the service name as its title. This is ok because within 1Password or its extension, a search shows both the title (by default, the service name), as well as the login name. This helps to distinguish between which accounts are present for the service.
This behavior would be useful within the Raycast plugin. Basically if I search for something, I should see the title of the item, but also the login.
Perhaps this is unique to items in the Login category.
Who will benefit from this feature?
Anyone who has multiple accounts for the same service stored within 1Password

@khasbilegt any thoughts here?


FLUID

Hello. Perhaps you like something to remove these containers with liquids of different mods ? Lag when scrolls .
http://imgur.com/9G7wWcd.

There's already a config option for that.


Migrate app-package to type module

This PR migrates Secvisogram's app package to type "module" which aligns it with modern package standards. This is needed to integrate the new csaf-validator-lib.

LGTM.
@domachine: Just out of curiosity: Why do we rename some files from .js to .cjs?

.cjs is the common js compatibility extension of nodejs: https://nodejs.org/api/esm.html#interoperability-with-commonjs . These are configuration files from tools, that not yet support the ecma module system.

@domachine Just fixed the permission issue. This seems to be ready to merge :-)


Fixed dynamic properties deprecation in PHP 8.2

Fixed upstream and released, https://github.com/packbackbooks/lti-1-3-php-library/releases/tag/v5.2.5. Will find its way into this repo once I rebase


Comma handling

Four-word titles ending in a two-word phrase after a comma, the last two words should sometimes both be in the subtitle.  Hard to say exactly when.

Thanks for the idea! I might close this one as out of scope though. It's a bit of a corner case, and could cause a bit of trouble modeling it correctly. For now, I'll leave word spacing >3 words to Glench's logo generator.


Branch 125491759

Jenkins, test this please.

Jenkins, test this please.

Jenkins, test this please.

Greetings @maciekcc @maciekcc. Is this 'sync to master' inthis PR


Improve typescript types for prisma nextjs docs

Describe this PR
Improves the typescript types and operators in documentation to be more accurate.
This is because prior to instantiation, the global value for prisma is actually undefined as nothing has been instantiated.
Changes
Add undefined to global type definition.
Replace || with ??
What issue does this fix?
Link to issue
Addresses #4566
Any other relevant information
N/A

@all-contributors please add @JingLi1998  for doc and example


[QUESTION] about the focus_on_activate function

Description
This is a very useful feature that I unfortunately cannot use.
The function is very useful when I click on a link in a third-party application and instantly go to the browser automatically, but if, for example, I play some online game and I receive a message from Telegram, then I automatically go to the workspace where Telegram is located. If a letter arrives, this does not happen with Thunderbird. Can anything be done about this?

I believe you can disable that in telegram. This is a dupe of some other issue here

I believe you can disable that in telegram. This is a dupe of some other issue here

Unfortunately, there is no such setting in Telegram and the problem relates to the focus_on_activate function

https://github.com/hyprwm/Hyprland/issues/2732

#2732
Thank you sir!

#2732

Thank you sir!


Results

I follow the steps of github to train and test, but the results I get is different from the paper result，I wonder if there is something I haven't noticed?

Looking forward your reply!

Hello, may I ask, in the paper, the training is carried out in two stages, but in the code at that time, the training is carried out in three stages, why? How do they correspond to each other

Could you leave a contact information？ I want to ask you some questions

Query-based methods are less stable than anchor-based methods, and THUMOS14 is a tiny dataset.
So we think a 0.5% gap is acceptable.
You can re-run the experiment several times and may get a better result.


Modifying code for the F0 board

Can anyone give some directions on how to modify and upload the code to the F0 board, what IDE to use, workflow. I have an ST-LINK V2 and CubeIDE installed
Thank you

Hi, what changes are you trying to make? If it's something that would be useful for others too, I'd be happy to implement them and get them in to official builds.

I am trying to make an analog output. I modified the PCB and added an opamp with RC filter at PB1 (TIM3_CH1) in order have outpt 0-5V analog. Since timer 3 is set up at high frequency it should be easy to add an extra PWM out right?

I am trying to make an analog output.
have you seen F103 boards doing this with a DAC?

I am trying to make an analog output.

have you seen F103 boards doing this with a DAC?

Yes ofcource but i want to make this board work with analog output too.


Performance penalty?

Hi thanks for the lib! I wonder whether this will introduce some performance penalty? Because theoretically speaking, the library uses a binary search to see what text size fits the layout. Thus, it invokes text layouting algorithm for a few more times.

Maybe this does not answer your question completely, but the readme says:

AutoSizeText is really fast. In fact, you can replace all your Text widgets with AutoSizeText.
Nevertheless you should not use an unreasonable high fontSize in your TextStyle. E.g. don't set the fontSize to 1000 if you know, that the text will never be larger than 30.
If your font size has a very large range, consider increasing stepGranularity.


Update misleading title

The article refers to your own DNS server, not an Azure DNS, something that is very useful for Azure HCI stack cases.

Learn Build status updates of commit 5d98ecb:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/virtual-network/virtual-networks-name-resolution-ddns.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Apologies for not reviewing sooner. I am not getting notifications when I am assigned as a reviewer.
It looks like the title of this article has already been changed, so the PR should just be closed.
The article does indeed describe dynamic DNS client registration on a VM, not Azure DNS.

#please-close


Eddie branch

Resolved issue where the database was not writing the data, changed where the signup user was called, and the data being parsed over.
Some code clean up will be required.

This request also includes a code clean up on the scripts.js to remove a huge chunk of commented out data.


buildEnv: support ignoreFileOutputs

Add option to ignore single-file output instead of failing, which would be useful to fix Mic92/nixpkgs-review#408.

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Close to correct the feature branch name.


test: fix the inverted conversion

The resulting string a byte string and needs to be converted to a string
like object.  This was meant to be a decode, not encode.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@swift-ci please smoke test

CC: @tbkka @drexin  @tachoknight


Bump apt and Ruby buildpacks; bump Ruby and Bundler minor versions

Contains security fixes.
See release notes:
https://github.com/cloudfoundry/ruby-buildpack/releases/tag/v1.8.49
https://github.com/cloudfoundry/apt-buildpack/releases/
https://www.ruby-lang.org/en/news/2021/11/24/ruby-2-7-5-released/

Could be worth updating here too: https://github.com/UKGovernmentBEIS/beis-opss-psd/blob/master/redex/manifest.yml#L5

Codecov Report

Merging #1684 (a4f9678) into master (59a8c3b) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1684      +/-   ##
==========================================
- Coverage   89.83%   89.82%   -0.02%
==========================================
  Files         402      402
  Lines        8639     8639
  Branches     1243     1243
==========================================
- Hits         7761     7760       -1
- Misses        699      700       +1
  Partials      179      179

Impacted Files
Coverage Δ

app/forms/product_form.rb
97.01% <0.00%> (-1.50%)
:arrow_down:

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 59a8c3b...a4f9678. Read the comment docs.


When  component is used together with  and  components, it will cause the development environment route to jump to a blank page.

Version
4.1.6
Reproduction link
github.com
Steps to reproduce
When I use nested routes and use the <transition> and <keep-alive> components on the layouts page, I edit the script code on the menu1 page and then click to jump to the menu2 page, and the page goes blank.
This problem only occurs in the development environment and will return to normal after refreshing the page.
https://user-images.githubusercontent.com/1574903/211184150-f7482d33-132d-48f8-95e5-6d5deb8ad4ab.mp4
<!-- layouts/index.vue -->
<template>
  <div>
    <router-link to="/menu1">menu1</router-link> | <router-link to="/menu2">menu2</router-link>
    <router-view v-slot="{ Component, route }">
      <transition name="main" mode="out-in" appear>
        <keep-alive>
          <component :is="Component" :key="route.fullPath" />
        </keep-alive>
      </transition>
    </router-view>
  </div>
</template>

<style scoped>
.main-enter-active {
  transition: 0.2s;
}

.main-leave-active {
  transition: 0.15s;
}

.main-enter-from {
  opacity: 0;
  margin-left: -20px;
}

.main-leave-to {
  opacity: 0;
  margin-left: 20px;
}
</style>

If I delete either <transition> or <keep-alive> component, everything works fine.
What is expected?
After modifying the code and jumping to the page can be displayed normally
What is actually happening?
Cannot be displayed properly

Duplicate of https://github.com/vuejs/core/issues/7121


GSD Request

--- GSD JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v5.16 to before v5.16.10",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=660a6126f8c3208f6df8d552039cda078a8426d1",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=4ad70fad65d7faf7d3d3126218eba48651d0674a"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "660a6126f8c3208f6df8d552039cda078a8426d1",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "4ad70fad65d7faf7d3d3126218eba48651d0674a",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "Revert \"gfs2: check context in gfs2_glock_put\"\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.16.10 by commit 4ad70fad65d7faf7d3d3126218eba48651d0674a, it was introduced in version v5.16 by commit 660a6126f8c3208f6df8d552039cda078a8426d1. For more details please see the references link."
}
--- GSD JSON ---

This issue has been assigned GSD-2022-1000497


Acer Spin 13 (akali360) microphone issues PopOS

The microphone doesn't show up anywhere, not even in pavucontrol. I checked discord to see whether it shows up and it did, but the name seemed like a device address of some type (?)
My speakers do show up and work properly. My OS is PopOS!
Boardname
Akali 360
Logs
debug-logs.tar.gz

Try the testing branch, ./setup-audio --branch common-codecs.

Looks like stock firmware is broken (unsurpriringly).


use identityserver4 in php core

Hi everyone,
I need to use latest news from fiba website and they provided an API which using identityserver4 as authorization system,
https://bbm-api.fiba.com/swagger
Our application is PHP and I tried to find something related to php and identityserver but I could not,
I would be grateful if help me how it is working with php and show me right way.I am really need to do this project asap

This seems to be a general question about IdentityServer - not a bug report or an issue.
Please use one of the our free or commercial support options
See here for more details.
Thanks!


Support association of connection with connection handler actor

It would be great if there were a means to name the connection handler actor based on some information available in the upgrade request. I'm thinking, for example, of a custom header that provides an ID, and that ID (presumably URL encoded) can be used as the name of the connection handler actor. The problem, of course, is that the connection handler actor already exists, and I don't think there's a way to name the actor after the fact. So presumably the implementation would create a new connection handler with the right name, register it as the handler, and stop the existing connection handler.
The goal is for clients to be able to identify themselves, and for the application to be able to use Akka routers, ActorSelections, etc. to communicate with specific clients (I'm assuming that's what the send() methods are for).
Does this make any sense? Is there a straightforward way to do it?

Paul,
Using the WebSocket in order to maintain bidirectional communication between the client and the server can be acheived with no "identification" of the client end.
What you might actually want is to have a specific handler actor responsible for a particular URL such as I tried to exemplify in my template at:
https://github.com/cuali/SprayEasterEggs/blob/wandoulabs/src/main/scala/reactive/api/ReactiveApi.scala#L29-L34
In this way, the connection handler remains anonymous while you have a clear route to some explicitly named actor that will handle the effective communication messages.
Would this resolve your concern?
Regards,
Alain


I want to run mysql-operator in ARM server

If I want to run this operator in ARM, do I need to make other configurations to the makefile, or can I just use Dockerfile directly

try https://github.com/ksmartdata/mysql-operator/pkgs/container/mysql-operator

try https://github.com/ksmartdata/mysql-operator/pkgs/container/mysql-operator

Thank you, but I am using version 0.6.2. Are these versions compatible?


chore!: update eigensdk

And with it, the alloy dependencies.
Also added a symbiotic feature to the SDK, symbiotic-rs will need to have its alloy dependency versions changed
Draft for now, this required changing quite a bit, need to make sure it's all good.
closes #478

Ok I bumped symbiotic-rs to 0.1.1


Fix memory leak on flag components remove

The generated code for flag components always adding the same static component as new component but using the same method to remove them increasing the stack all the time. I created a new method for removing flag components  without pushing them to the stack.

Hi, I will try to find another solution as we also have methods like e.RemoveAllComponents() which would still pool those static components.

Fixed by checking the componentPool first. Probably not the best solution but the fastest.
There's another open issue that suggests a different approach of dealing with flag components altogether, so fixing this with an easy quickfix should be fine for now. https://github.com/sschmid/Entitas-CSharp/issues/238#issuecomment-302856144

@IDNoise I think you were interested in this fix too. There you go


root: 6.30.06 -> 6.32.00

ROOT 6.32.00 will be out in a few days. When this is happening, this PR should be updated to use the released sources instead of the git release candidate tag.
@veprbl

Hi @nbraud, you're right I used the wrong hash! It should work now.
@veprbl, FYI

Oh okay, yes that makes sense. I'll look into it. Root requires C++ 17 or 20, and all libraries that are build with ROOT as a dependency need to be built with the same C++ standard for these dependents to work in the context of the ROOT C++ Interpreter, called Cling.

Result of nixpkgs-review pr 310309 run on x86_64-linux 1

8 packages built:

hepmc3
ns-3
python311Packages.hepmc3
python312Packages.hepmc3
root
root.debug
xfitter
yoda-with-root


Add Sindre Sorhus

What type of PR is this? (check all applicable)

[x] 🚀 Added Name
[ ] ✨ Feature
[ ] ✅ Joined Community
[ ] 🌟 ed the repo
[ ] 🐛 Grammatical Error
[ ] 📝 Documentation Update
[ ] 🚩 Other

Description
My profile. If it doesn't load, try refreshing some times. It loads eventually.
Add Link of GitHub Profile
https://github.com/sindresorhus

I see what you did there @sindresorhus 🤣

It Looks Great to me, but could you please resolve conflicts?

What conflicts? If you mean the checks, they were failing for unrelated reasons: https://github.com/abhisheknaiidu/awesome-github-profile-readme/pull/78/checks?check_run_id=883519367 (Cloudflare is down)

and also could you place https://github.com/sindresorhus/sindresorhus instead of https://github.com/sindresorhus

FYI, there are some other entries that link directly to the profile too.

@sindresorhus Okk Fine, I'm merging nw
yea I just saw that there is one, that links to the profile 😅
Thanks for pointing out!

Welcome to the repo, @sindresorhus 😍


[v2.2.0] user define hook configuration is override each time module is installed

I write a package that configure pre-push and pre-commit for my project.
/* myHooks/bin/install */

validate.configureHook('pre-push', ['lint', 'validate', 'test']);
validate.configureHook('pre-commit', ['lint', 'validate', 'test']);

I override the default configuration for pre-commit
/* package.json */
{
...,
  "pre-push": [
    "lint",
    "validate",
    "test"
  ],
  "pre-commit": []
}

then I do
npm i myHooks

My package.json is reset to default hook configuration
/* package.json */
{
...,
  "pre-push": [
    "lint",
    "validate",
    "test"
  ],
  "pre-commit": [
    "lint",
    "validate",
    "test"
  ]
}

We are seeing the same here. Most likely has been introduced in #51 (cc @nlf @jonathan-fielding).

The tests are actually red in HEAD:
  22) configureHook() won't overwrite existing hook settings:

      actual expected

      [
        "test",
        "lint"
      ]

      Expected [ 'test', 'lint' ] to equal specified value

      at /Users/jfeth/dev/git-validate/test/utils.js:297:57

I tried writing a fix for this, but I don't think it is possible without changing the API of configureHooks - alternatively we can release a 2.2.1 that reverts the change and publish a new version as a 3.0.0, however we are for example running git-validate on postinstall, which means that on our CI server it would modify the package.json regardlessly, which in my eyes is not an acceptable default behaviour.

I opened #54, which reverts the changed default behaviour and adds a flag that can be passed to configureHook which @jonathan-fielding could use to auto-upgrade package.json.
It removes the behaviour of merging the old with the new values.


Images are not displayed to players

Hi,
Since one/two weeks, images are not uploaded in the chat but in a black space between the bottom writing DM space and the above system information (attacks, etc).
As a result, images are not displayed to players.
Thanks for fixing.
Ugun

Please provide some images. I don't understand the problem

Hi Marian,
Please find attached.
I tried with using upload button or simple drag and drop, with button activated or not.
Thanks !
Ugun
De : Marian @.>
Envoyé le :samedi 1 octobre 2022 17:38
À : @.>
Cc : @.>; @.>
Objet :Re: [bmarian/chat-images] Images are not displayed to players (Issue #61)
Please provide some images. I don't understand the problem
—
Reply to this email directly, view it on GitHubhttps://github.com/bmarian/chat-images/issues/61#issuecomment-1264401003, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AV7J7YPPUASLSLQKCQTB5CLWBBLJBANCNFSM6AAAAAAQ2PZCUU.
You are receiving this because you authored the thread.Message ID: @.***>

I was asking for an image with the actual problem.
Does your chat look like in the video https://github.com/bmarian/chat-images?
If yes? Just hit enter after adding the images.

I can't see attachments from emails, please respond on github.

Issue solved :) Thanks for your help !


Secure HTTPS connection

Need to figure out how to enable https in Jupyter notebooks.

https://www.cloudflare.com/en-gb/learning/ssl/what-is-an-ssl-certificate/
At the beginning, we might consider a self-signed certificate. Alternatively, Let's Encrypt is the way to go I guess.


migrate workloads from not-ready clusters to healthy spare clusters

What type of PR is this?
kind/feature
What this PR does / why we need it:
Add a feature gate to migrate workloads from not-ready clusters to healthy spare clusters
Which issue(s) this PR fixes:

depends on #673
Special notes for your reviewer:

Codecov Report

Merging #674 (12b7653) into main (e2ffa5f) will decrease coverage by 0.38%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #674      +/-   ##
==========================================
- Coverage   13.60%   13.22%   -0.38%
==========================================
  Files          66       71       +5
  Lines        7492     7839     +347
==========================================
+ Hits         1019     1037      +18
- Misses       6391     6717     +326
- Partials       82       85       +3

see 41 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Credit Card UI Design

Description
Credit Card UI mockup using CSS
Fixes #86
Type of change

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

Explain the Testing instructions
Test Configuration:

Operating system: Windows 10
Text-editors used: Sublime Text Editor

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] My changes generate no new warnings

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK

@arpit456jain u can merge it


Remove offending call to this(null) in default Response constructor

I believe I've located what was preventing a successful build in #6, and removed it. mvn test and mvn install completed on my machine (and in a Heroku deployment) without error.
Let me know if you have any questions, or if there's any other changes you'd like to make as part of this pull request.
Thanks, all!

@mcelicalderon Can you take a look?

It was broken indeed, thanks @skylineproject !

Thanks, @mcelicalderon!


feat: Add ordered navigation handler for plugins, closes #7306

What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Docs
[ ] New Binding issue #___
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change?

[ ] Yes, and the changes were approved in issue #___
[x] No

Checklist

[x] When resolving issues, they are referenced in the PR's title (e.g fix: remove a typo, closes #___, #___)
[x] A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.
[x] I have added a convincing reason for adding this feature, if necessary

Other information

@lucasfernog these changes are great. I right about making some similar ones, but didn't want to be responsible for breaking changes.

No need to worry about it right now since it targets v2. Thanks for the contribution!


Many names account/public domain/global License-CC0

Parallel

I have no clue what you meant with this bug, can you explain?

thanks for the support


nixos: add preferLocalBuild=true; on derivations for config files and closureInfo

Motivation for this change
When using remote builders, nix would copy the closure of the config files to the remote builder when only the path is really needed, which can take a sizeable amount of time depending on the network speed. This is notably annoying for closure-info.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) : I tested the installer test only; it works except one test which fails with

Path length (109) is longer than maximum supported length (108) and will be truncated at /nix/store/0ingn8cwwnl84i374hcl6nafsm2c5m2p-perl-5.28.0/lib/perl5/5.28.0/x86_64-linux-thread-multi/Socket.pm >
boot-after-install# qemu-system-x86_64: -monitor unix:./monitor: Failed to connect socket ./monitor: No such file or directory

which is independent of this change (it was failing before)

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

Tested on top of current nixos-unstable, not tested after rebase on master.

Looks okay to me. Not to bikeshed here but I wonder if runCommand should just always do preferLocalBuild? Those are rarely big enough for it to be worthwhile to do remotely.

Perhaps allowSubstitutes = false also should be included?

What is the purpose of allowSubstitutes = false ?

@symphorien If it is false then the derivation will always be build, no substitution will be attempted. I.e., no network traffic. At least that is my interpretation of the description in the manual.

So I tried to add allowSubstitutes=false. Installer tests fail this way:
machine: must succeed: nixos-install < /dev/null >&2
machine# > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > building the configuration in /mnt/etc/nixos/configuration.nix...
warning: unable to download 'https://rt.perl.org/Public/Ticket/Attachment/1502646/807252/0001-Fix-missing-build-dependency-for-pods.patch': Timeout was reached (28); retrying in 342 ms
warning: unable to download 'https://rt.perl.org/Public/Ticket/Attachment/1502646/807252/0001-Fix-missing-build-dependency-for-pods.patch': Timeout was reached (28); retrying in 669 ms
warning: unable to download 'https://rt.perl.org/Public/Ticket/Attachment/1502646/807252/0001-Fix-missing-build-dependency-for-pods.patch': Timeout was reached (28); retrying in 1018 ms
warning: unable to download 'https://rt.perl.org/Public/Ticket/Attachment/1502646/807252/0001-Fix-missing-build-dependency-for-pods.patch': Timeout was reached (28); retrying in 2370 ms
builder for '/nix/store/4x3famhx2zy5q0khadzdj2wsp3glg6n3-0001-Fix-missing-build-dependency-for-pods.patch.drv' failed with exit code 1; last 1 log lines:
machine#   error: unable to download 'https://rt.perl.org/Public/Ticket/Attachment/1502646/807252/0001-Fix-missing-build-dependency-for-pods.patch': Timeout was reached (28)
cannot build derivation '/nix/store/3nnfzfb8a9kw19k28fyznjxl9wd4nmix-perl-5.28.0.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/0d2w8f99r9f1r469b0c4mxm9wykdslwk-bison-3.1.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/sgyk07dap5m36zp7960ajnsm42fdqfc1-binutils-2.30.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/2gn5waz0i53qyxiiqvlz5x5hbb0285v5-binutils-wrapper-2.30.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/bd7bd91gzd4q354ndmmaygfyfd9mq0kf-binutils-wrapper-2.30.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/2yvxznr3simh127q5g6kspgna95pwnb9-bootstrap-stage2-gcc-wrapper.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/rycm5cr87gxs68c286wghi5ds5krnkfw-bootstrap-stage3-gcc-wrapper.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/06ingf8raac2684mmrpsm2bpvrxbwbpm-bootstrap-stage3-stdenv-linux.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/sj1g4r65ws715ssj88hgqng56vvc08jl-zlib-1.2.11.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/3da3gxyccpksnvgichy0crk9s7k8659z-curl-7.61.1.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/mzp034izmcmwjpa3as6iayz7lixbx8v3-certdata2pem.py.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/xlik0ggbzgpqk32h8db3z360qb1b8xq0-nss-3.39.tar.gz.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/04yr17rzp7hwlpq0fcsqx7fb8lp2rf95-system-path.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/vsx64y5bjixbisi9if0z4zqhng14sihv-nixos-system-nixos-19.03.git.428164a.drv': 1 dependencies couldn't be built
[0 built (1 failed), 300 copied (1086.6 MiB)]
error: build of '/nix/store/vsx64y5bjixbisi9if0z4zqhng14sihv-nixos-system-nixos-19.03.git.428164a.drv' failed
machine: exit status 1
machine: output:
error: command `nixos-install < /dev/null >&2' did not succeed (exit code 1)
command `nixos-install < /dev/null >&2' did not succeed (exit code 1)
cleaning up
killing machine (pid 600)
builder for '/nix/store/7mrrmpaj77xd56z10n4ij2l1bx8sbwwa-vm-test-run-installer-simple.drv' failed with exit code 255
error: build of '/nix/store/7mrrmpaj77xd56z10n4ij2l1bx8sbwwa-vm-test-run-installer-simple.drv' failed

instead of
machine: must succeed: nixos-install < /dev/null >&2
machine# > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > building the configuration in /mnt/etc/nixos/configuration.nix...
[180 built, 302 copied (1042.1 MiB)]

I have no idea what causes this error. Here is the branch if you want to debug it: https://github.com/NixOS/nixpkgs/commit/428164a4d53306982f07961cfe1d697166a80b7f

So the reason seems that the installer uses the installation medium's store as a substituter:
https://github.com/NixOS/nixpkgs/blob/master/nixos/modules/installer/tools/nixos-install.sh#L95-L97
so if we add allowSubstitutes = false on some derivation, then nixos-install will need to build it even if it is present on the live media, which usually requires internet access.
Therefore, allowSubstitutes = false seems to do more harm than good. I think the PR can be merged as it (ie adding only preferLocalBuild = true;).

Is there something I should add to get the PR merged ?

I fixed the merge conflict.

Oh, there is another merge conflict. I expect more to come, so if someone wants to merge this PR, tell me and I will resolve the conflict :)

@symphorien Please do. I've asked in #nixos-dev whether there are any reservations about merging this and I'll try to follow up on it so that we can get this in.

I rebased, and ran the installer.simple test succesfully.

Nice!

Thanks! :+1:


🌐 Add Korean translation for Tutorial - Header Parameters

Hello, there.
I've referred to the previous translation Pull Requests.
Please let me know if any problems.
This PR translates tutorial/header-params.md in Korean.
related: #2017

Thanks for your contribution @mode9 ! :clap: :tada:
And thanks for the reviews @PandaHun and @hard-coders ! :cake:


Error, while converting fastrcnn torchvision model

Used docker
docker run --gpus all -it  nvcr.io/nvidia/pytorch:20.12-py3

Convertion scipt:
import torchvision
import torch
from torch2trt_dynamic import torch2trt_dynamic as torch2trt

model = torchvision.models.detection.fasterrcnn_resnet50_fpn(pretrained=True)
model.cuda().eval().half()

data = torch.randn((1, 3, 800, 800)).cuda().half()

model_trt = torch2trt(model, [data])

error:
Traceback (most recent call last):
  File "./scripts/export_torch_to_tensorrt_example.py", line 24, in <module>
    model_trt = torch2trt(model,
  File "/var/www/auto-carpart-detector/torch2trt_dynamic/torch2trt_dynamic/torch2trt_dynamic.py", line 518, in torch2trt_dynamic
    outputs = module(*inputs)
  File "/opt/conda/lib/python3.8/site-packages/torch/nn/modules/module.py", line 744, in _call_impl
    result = self.forward(*input, **kwargs)
  File "/opt/conda/lib/python3.8/site-packages/torchvision/models/detection/generalized_rcnn.py", line 99, in forward
    proposals, proposal_losses = self.rpn(images, features, targets)
  File "/opt/conda/lib/python3.8/site-packages/torch/nn/modules/module.py", line 744, in _call_impl
    result = self.forward(*input, **kwargs)
  File "/opt/conda/lib/python3.8/site-packages/torchvision/models/detection/rpn.py", line 332, in forward
    anchors = self.anchor_generator(images, features)
  File "/opt/conda/lib/python3.8/site-packages/torch/nn/modules/module.py", line 744, in _call_impl
    result = self.forward(*input, **kwargs)
  File "/opt/conda/lib/python3.8/site-packages/torchvision/models/detection/anchor_utils.py", line 154, in forward
    anchors_over_all_feature_maps = self.cached_grid_anchors(grid_sizes, strides)
  File "/opt/conda/lib/python3.8/site-packages/torchvision/models/detection/anchor_utils.py", line 142, in cached_grid_anchors
    anchors = self.grid_anchors(grid_sizes, strides)
  File "/opt/conda/lib/python3.8/site-packages/torchvision/models/detection/anchor_utils.py", line 118, in grid_anchors
    shifts_x = torch.arange(
  File "/var/www/auto-carpart-detector/torch2trt_dynamic/torch2trt_dynamic/torch2trt_dynamic.py", line 312, in wrapper
    converter['converter'](ctx)
  File "/var/www/auto-carpart-detector/torch2trt_dynamic/torch2trt_dynamic/converters/mul.py", line 12, in convert_mul
    input_a_trt, input_b_trt = trt_(ctx.network, input_a, input_b)
  File "/var/www/auto-carpart-detector/torch2trt_dynamic/torch2trt_dynamic/torch2trt_dynamic.py", line 139, in trt_
    dtype = check_torch_dtype(*tensors)
  File "/var/www/auto-carpart-detector/torch2trt_dynamic/torch2trt_dynamic/torch2trt_dynamic.py", line 113, in check_torch_dtype
    assert (dtype == torch.int32)  # , 'Tensor data types must match')
AssertionError

I think this is due to the fact that tensors with different data types are fed to the mul.
If we convert to one data type, then at the next stage we get an error:
[TensorRT] ERROR: (Unnamed Layer* 3651) [Slice]: slice size cannot have negative dimension, size = [-1]
[TensorRT] ERROR: (Unnamed Layer* 3651) [Slice]: slice size cannot have negative dimension, size = [-1]
Traceback (most recent call last):
  File "./scripts/export_torch_to_tensorrt.py", line 53, in <module>
    model_trt = torch2trt(model_w,
  File "/var/www/auto-carpart-detector/torch2trt_dynamic/torch2trt_dynamic/torch2trt_dynamic.py", line 518, in torch2trt_dynamic
    outputs = module(*inputs)
  File "/opt/conda/lib/python3.8/site-packages/torch/nn/modules/module.py", line 744, in _call_impl
    result = self.forward(*input, **kwargs)
  File "./scripts/export_torch_to_tensorrt.py", line 41, in forward
    return self.model(x)
  File "/opt/conda/lib/python3.8/site-packages/torch/nn/modules/module.py", line 744, in _call_impl
    result = self.forward(*input, **kwargs)
  File "/opt/conda/lib/python3.8/site-packages/torchvision/models/detection/generalized_rcnn.py", line 99, in forward
    proposals, proposal_losses = self.rpn(images, features, targets)
  File "/opt/conda/lib/python3.8/site-packages/torch/nn/modules/module.py", line 744, in _call_impl
    result = self.forward(*input, **kwargs)
  File "/opt/conda/lib/python3.8/site-packages/torchvision/models/detection/rpn.py", line 338, in forward
    concat_box_prediction_layers(objectness, pred_bbox_deltas)
  File "/opt/conda/lib/python3.8/site-packages/torchvision/models/detection/rpn.py", line 100, in concat_box_prediction_layers
    box_regression = torch.cat(box_regression_flattened, dim=1).reshape(-1, 4)
  File "/var/www/auto-carpart-detector/torch2trt_dynamic/torch2trt_dynamic/torch2trt_dynamic.py", line 312, in wrapper
    converter['converter'](ctx)
  File "/var/www/auto-carpart-detector/torch2trt_dynamic/torch2trt_dynamic/converters/view.py", line 15, in convert_view
    input_trt = trt_(ctx.network, input)
  File "/var/www/auto-carpart-detector/torch2trt_dynamic/torch2trt_dynamic/torch2trt_dynamic.py", line 149, in trt_
    num_dim = len(t._trt.shape)
ValueError: __len__() should return >= 0

Hi,
Please don't feed fp16 model and input to the convertor, if you want to enable fp16 model, set fp16_mode=True is enough.
And ... honestly, I am not sure if all the layers inside fastrcnn in torchvision can be converted in this repo. I will try to add the support, this might take some time.


Features list in Readme & user docs

Adding a quick chapter to the user doc to help explain
the goals and benefits of MAGE. Also adding
a bullet point list version of this list to the readme,
as well as a link to the related user doc section.
Closes #55
@Tatsujinichi feel free to provide any feedback on the content written here.

@tatsujinichi mentions that what he was looking for was more of a problem - solution - example kind of description.
TBH I am struggling a bit with the first part: I feel we could do a better job at describing the specific problems MAGE addresses before we go into each solutions.
As for the examples, I think the user doc already
contains all we need, but we will need to create better linking beteen the solution description and the examples provided by the user doc.
As I said, I am having a bit of an issue comin up with the right content and format  so any suggestions would be most welcome.

Adding more people to the conversation; I don't have much time right now to try to get this one merged but would love any input and suggestions in the meantime.

ty for the update

Updated the content. I think this is more in the spirit of what @Tatsujinichi was looking for, but please let me know if anything is still missing.

Added a link to the development blog.


Failing subscription returns nil instead of true

Per the README:
it "rejects when no room id" do
    subscribe
    expect(subscription).to be_rejected
end

This test fails with the following message:
Failure/Error: expect(subscription).to be_rejected
       expected `#<MyChannel:0x007fb69d350fb0 @connection=#<ActionCable::Channel::ConnectionStub:0x007fb69d351460 ...ue:0>, @reject_subscription=nil, @subscription_confirmation_sent=true, @_streams=["my_channel"]>.rejected?` to return true, got nil

Should we be testing for nil here instead of rejected??

That means that subscription hasn't been rejected.
Could you please share your connection/channel code?
rejected? is just an accessor for @reject_subscription variable (https://github.com/rails/rails/blob/master/actioncable/lib/action_cable/channel/base.rb#L153), which is nil by default.

#channel
def subscribed
    stream_from "counts_channel"
end

#connection
class Connection < ActionCable::Connection::Base
    identified_by :current_token

    def connect
      self.current_token = verify_auth
    end

    def verify_auth
      # use query string token param to verify
    end
end

You do not reject the subscription, that's why the test is failing.

Thank you.  This should have been a connection test checking for a failed connection.  For this to success, the subscription would need to be rejected, as you correctly pointed out.


🎉 4.0.21

ℹ About this release

Version: 4.0.21
Type: patch
Last Release: 4.0.20 (3/26/2024, 5:36:14 PM) [?]
Commits to merge: 6 [?]

📦 Dependencies

Update semantic-release from ^23.0.5 to ^23.0.8
Update simple-git from ^3.23.0 to ^3.24.0

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


fix milvus kb service typo

for users using milvus vs, this will fix exception.

we fix this in local and will push it to dev later


Install Kubernetes Tools fails at multiple points - eksworkshop-admin instance profile

When following this guide: https://catalog.us-east-1.prod.workshops.aws/workshops/afee4679-89af-408b-8108-44f5b1065cc7/en-US/020-prerequisites/k8stools
I've raised an issue for a typo on line 184, but source setup doesn't work when called, which is one part of the issue.  But using ./setup at least starts the setup
The output consistently gets stuck at this point below
Install OS tools
Update OS tools
Update pip
Uninstall AWS CLI v1
Install AWS CLI v2
Found preexisting AWS CLI installation: /usr/local/aws-cli/v2/current. Please rerun install script with --update flag.
AWS_REGION is eu-west-1
export ACCOUNT_ID=318781710241
export AWS_REGION=eu-west-1
export AZS=(eu-west-1a eu-west-1b eu-west-1c)
export TF_VAR_region=eu-west-1
eu-west-1
Check Cloud9 AWS Managed temporary credentials are disabled - in AWS Settings
Setup Terraform cache
Setup kubectl
Install kubectl v1.24.12
install eksctl
eksctl completion
helm
add helm repos
"eks" has been added to your repositories
kubectx
ssh key
ssm cli add on
install tfsec ...
pip3
git-remote-codecommit
CHANGED: partition=1 start=4096 old: size=20967391 end=20971487 new: size=67104735 end=67108831
Filesystem     1M-blocks  Used Available Use% Mounted on
/dev/nvme0n1p1     32756  6994     25763  22% /
Verify ....
jq in path
aws in path
wget in path
kubectl in path
terraform in path
eksctl in path
helm in path
kubectx in path
Enable bash_completion
PASSED: AWS_REGION is eu-west-1
PASSED: TF_VAR_region is eu-west-1
PASSED: ACCOUNT_ID is xxxxxxxxxxxx
Associate eksworkshop-admin

An error occurred (InvalidParameterValue) when calling the ReplaceIamInstanceProfileAssociation operation: Value (eksworkshop-admin) for parameter iamInstanceProfile.name is invalid. Invalid IAM Instance Profile name
ERROR: Encountered error associating instance profile eksworkshop-admin with Cloud9 environment
For final checks - run ./check

The eksworkshop-admin role has not been created in IAM and there does not appear to be any instance profile associated with the cloud9 instance.

Are you running this in your own environment - or in an AWS event (via workshop studio) ?

typo fix committed for line 184 in setup
instructions changed to run:  source ./setup
Also removed the workshop section on running in your own account - as this needs more testing / different instructions
So workshop only tested / usable in workshop studio (AWS events) for now.

It was in my own environment Andy.  I'll look forward to seeing it again in the future.
In the meantime I'm going to try and create my own TF and piece together the k8s side from the tfekscode.