OneDrive shared folder and chmod

Windows Version
Microsoft Windows [version 10.0.22631.3527]
WSL Version
<IP_ADDRESS>
Are you using WSL 1 or WSL 2?

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>-2
Distro Version
Ubuntu 22.04
Other Software
OneDrive Windows : Build 24.076.0414.0005 (64 bits)
Repro Steps
I have a OneDrive folder in Windows.
By using this folder in WSL2, there is no support of chmod.
=> on WSL  :
touch ~/OneDrive/test
chmod +x ~/OneDrive/test
chmod: changing permissions of '/home/my/OneDrive/test': Operation not permitted
=> on Native Ubuntu :
It s disturbing because on native linux/ubuntu (24.04), I am able to mount the OneDrive with rclone, and I don't have error of chmod.
chmod +x ~/OneDrive/test
ls -ltr ~/OneDrive/test
ls: /home/me/OneDrive/test: Input/output error
-rw-rw-r-- 1 sah0027 sah0027 0 May  5 17:32 /home/me/OneDrive/test
Yes, in native ubuntu the chmod is not applied , but it s not doing the error on chmod.
( ps : is there a way to have permission support enabled on a OneDrive shared folder ?)
Regards,
Expected Behavior
The issue is to have a difference of behaviour,  personally i would prefer the behaviour Ubuntu native.
=> chmod ok
Actual Behavior
=> on WSL  :
touch ~/OneDrive/test
chmod +x ~/OneDrive/test
chmod: changing permissions of '/home/my/OneDrive/test': Operation not permitted
Diagnostic Logs
No response

Ps:
in /etc/wsl.conf  , I have :
[automount]
options = "metadata"

and with mount , I see :
C:\ on /mnt/c type 9p (rw,noatime,dirsync,aname=drvfs;path=C:;uid=0;gid=0;metadata;symlinkroot=/mnt/,mmap,access=client,msize=65536,trans=fd,rfd=11,wfd=11)
And in wsl,  ~/OneDrive is a symlink to : /mnt/c/XX/OneDrive - YY/

The error you're seeing is because Linux permissions don't really make sense for a OneDrive folder mounted on the Windows side.
Accessing the files themselves should work though. Could you capture logs of the file accesses failing ? /logs


uncompleted wiki

I haven't found a description how to charge e.g. the Flux-Infused JetPlate.

Put it in a Energetic Infuser


fix add BIN as keyword

Proposed changes
Issue Number: close #xxx
Problem summary
This keyword BIN is introduced in #13067 , but forgot to add it to keywords region.
Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[ ] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[ ] No
[ ] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[ ] No Need

Does it need to update dependencies:

[ ] Yes
[ ] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[ ] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 38.35 seconds
load time: 567 seconds
storage size:<PHONE_NUMBER>9 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20221102135448_clickbench_pr_38454.html


MacOS builds are guaranteed to be non-deterministic

MacOS 11 was last to support deterministic builds and it's being slowly retired from GitHub runners, which means the pkgcheck workflows can intermittently fail until they stop working completely.

https://gist.github.com/pudquick/89c90421a9582f88741b21d10c6a155e
It suggests we may be able to get around it by using a different linker.

https://gist.github.com/pudquick/89c90421a9582f88741b21d10c6a155e It suggests we may be able to get around it by using a different linker.

There is differences even before it tries to use linker, so comparing object files already fails... I got as far as having just 1 byte difference, but my knowledge of the executable format used by Mac is quite limited and I don't have recent enough Mac hardware to test, so I can only use the CI to test.


Updated AsyncNinja: latest version and more platforms

Pull Request Description
Update previously added AsyncNinja:

build actions were added for watchOS and swift package manager
revision was updated
swift version changed from 3.0 to 3.1

Acceptance Criteria
To be accepted into the Swift source compatibility test suite, a project must:

[x] be an Xcode or swift package manager project
[x] support building on either Linux or macOS
[x] target Linux, macOS, or iOS/tvOS/watchOS device
[x] be contained in a publicly accessible git repository
[x] maintain a project branch that builds against Swift 3.0 compatibility mode
and passes any unit tests
[x] have maintainers who will commit to resolve issues in a timely manner
[x] be compatible with the latest GM/Beta versions of Xcode and swiftpm
[x] add value not already included in the suite
[x] be licensed with one of the following permissive licenses:

BSD
MIT
Apache License, version 2.0
Eclipse Public License
Mozilla Public License (MPL) 1.1
MPL 2.0
CDDL

[x] pass ./check script run

Ensure project meets all listed requirements before submitting a pull request.

Thanks!


Emitting to rooms not working

Hi, for some reason it looks like the rooms are not working when I use multiple dynos on Heroku. I've enabled session affinity, so assume I'm doing something wrong setting up the redis adapter. But I'm not getting any errors as far as I can tell.
I'm using version 5.3 as I haven't migrated to socket.io version 3 yet.
Here is my code:
const sticky = require('sticky-session');
const express = require('express');

const app = express();
app.set('trust proxy', true);
const server = require('http').createServer(app);
const SocketServer = require('socket.io');
const redisAdapter = require('socket.io-redis');
const keys = require('./config/keys');
const start = require('./server');

const io = new SocketServer(server, { perMessageDeflate: false });

const WORKERS = process.env.WEB_CONCURRENCY || 1;
const PORT = process.env.PORT || 5000;

console.log(`Running ${WORKERS} workers`);

if (!sticky.listen(server, PORT, { workers: WORKERS })) {
  // Master code
  server.once('listening', () => {
    console.log(`Server started on ${PORT} port`);
    io.adapter(redisAdapter(keys.redisURL));
  });
} else {
  // worker code
  start(app, io);
}

In the start function called by the worker code section I call app.set('socketio', io) to make the io instance available globally in my app. Then I'm using the rooms like this:
io.to(roomId).emit(message)
Am I doing something wrong? I appreciate any help here.
Thanks!

@timB73 is it a similar issue to: https://github.com/socketio/socket.io/issues/3673, would one of those comments help?

@timB73 is it a similar issue to: https://github.com/socketio/socket.io/issues/3673, would one of those comments help?

@timB73 this issue fixed it for me: https://github.com/socketio/socket.io/issues/3743, turns out i was using different types of roomIds where one type was number and the other type was string, enforcing string type of roomIds when joining and emitting fixed my problem

@timB73 this issue fixed it for me: https://github.com/socketio/socket.io/issues/3743, turns out i was using different types of roomIds where one type was number and the other type was string, enforcing string type of roomIds when joining and emitting fixed my problem

@timB73 did you find a solution?

@darrachequesne Hi, I believe this was caused by using Node clusters / sticky-session module, rather than multiple dynos. I removed the cluster code to instead use a single process (still multiple dynos) and it worked. Not sure why.

Hey guys the was related with types.
For example
clientSide(angular):
this._socketio.socket.on('connected', () => this._socketio.socket.emit('join', user_id)); // user_id must be string!
serverSide(emitter):
io.to(user_id).emit(event, JSON.stringify(payload));  // user_id must be string!

For future readers: we have created a new adapter which works with Node.js cluster module.
Please see the documentation here: https://socket.io/docs/v4/cluster-adapter/


Convert data to string before requesting inference in FiwareNode

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

Project coverage is 49.09%. Comparing base (b3b37a7) to head (5710816).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #127      +/-   ##
==========================================
+ Coverage   48.99%   49.09%   +0.09%
==========================================
  Files          65       65
  Lines        3023     3023
  Branches     1115     1115
==========================================
+ Hits         1481     1484       +3
+ Misses        716      713       -3
  Partials      826      826

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


Fixed #6163: Force linking to objective C runtime

Sound channel requires the objective C runtime, force linking to
fix build issues on older mac os versions

@ryandesign  this should fix your issue. cause seems to be different indirect linking settings on the older mac os versions.

Works for me, thank you!


[Network] Fix #29565: az network nat gateway create missing --tags argument

Related command

Description

Fix: #29565
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Network


add_watermark

Pull Request Title (short and descriptive)
Description
Added successfully add_watermark function
Type of Change
Please delete options that are not relevant and mark the one that applies:

[ ] Bug fix (non-breaking change that fixes an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update
[ ] Refactoring (code improvements or updates without changing functionality)

Related Issue(s)
List any related issues:

Closes #ISSUE_ID

Proposed Solution
Describe the approach taken to implement the solution.
Checklist
Please review the checklist below and mark items as complete as applicable:

[ ] My code follows the style guidelines of this project.
[ ] I have performed a self-review of my code.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have added necessary unit tests that verify the functionality.
[ ] All new and existing tests pass locally.
[ ] I have checked that no other open PRs address this issue.

Screenshots (if applicable)
Include any screenshots or additional visual aids that might help with the review.
Additional Information
Add any other context about the pull request  @Tinny-Robot

@Tinny-Robot, I have done my task, can you check that.


React: Major Bug Fix and Blogs responsive

@anushbhatia Sir, this PR includes the following:

The responsiveness issue on the blogs page

In addition to that it fixes a major issue coming up always during development. On refreshing the page every time the styled-components used to break out and thus the website broke as well due to which we had to go to different routes every time and come back again.
It was extremely frustrating to work with during development phase.
Turns out it has something to do with how next.js works and finally after searching up it required an additional babel plugin and config to resolve the issue.
Fix:
This GitHub thread from the official next.js  package came highly in use. You can view it here: https://github.com/zeit/next.js/issues/4068
To preview the work:https://page-setup-preview.now.sh/blogs

This was a really difficult task to do well-done @abhu-A-J !!!

@anushbhatia Sir medium was good for it! I don't think this needs hard! 🤣


Wildcard references with headers should skip notes without matching headers

Please select if your request is either something new or an enhancement

[x] Enhancement of an existing Feature.
[ ] Request of a new feature.

Please select the area your request applies to. (Multiple selections are Possible. You can leave blank if you're not sure.)

[ ] Workspace - VSCode workspace, vaults, Intellisense/autocomplete, Dendron settings
[ ] Lookup - Dendron's Lookup Command
[x] Views - Dendron Preview, Tree View, Side Panels in the UI
[ ] Schema - Dendron Schemas
[ ] Pod - Data import from / export to Dendron
[ ] Publish - External Site Publish
[x] Markdown - Markdown features, such as syntax support and features

Is your feature request related to a problem? Please describe
When using a wildcard reference with a header (for example ![[daily.journal.*#meeting]]), the notes that don't have matching headers are included in the render with the error message "anchor meeting not found".
This makes it hard to find only matching notes because there may be many notes in the hierarchy but few that match the header.
Describe the solution you'd like
Instead, the notes that don't have matching headers should be skipped. Perhaps a message could be included to note that there are other notes that are in the hierarchy but don't match the header.
Describe alternatives you've considered
The queries RFC would also solve this problem, but it's a harder syntax to use.

When using the wildcard reference ![[daily.journal.2022.*#new-thoughts,1]], there is an error reported in the preview panel

since I don't have a file named as daily.journal.2022.01.md.

Tracking with [[Wildcard References with Headers Should Skip Notes without Matching Headers|dendron://private/task.2022.02.14.wildcard-references-with-headers-should-skip-notes-without-matching-headers]]

There is already an open (very old) issue referring to the same behaviour: https://github.com/dendronhq/dendron/issues/363

@SeriousBug, is it okay to close this as a duplicate of #363?

Ah good catch, thanks! I'll go ahead and close this.


GETMusic checkpoint download link not working

Fix GETMusic checkpoint download link

The GETMusic checkpoint download link has been deleted，how can i get  GETMusic checkpoint now?

Hello
Version 2.13.2  is available with the build correction. Unity Team is super
efficient for delivery ;-)
No need for the patch now.
Regards
Thierry
Le mer. 13 nov. 2024 à 07:21, 清吾 @.***> a écrit :

The GETMusic checkpoint download link has been deleted，how can i get
GETMusic checkpoint now?
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/muzic/issues/203#issuecomment-2472551462,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFMEQASR5TOFW7O2TR6M7EL2ALVVVAVCNFSM6AAAAABOWNMV4CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINZSGU2TCNBWGI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


clues: add Challenge Scrolls to their clue scrolls

Closes #2740
This keeps the answer overlay visible if the user opens the Challenge Scroll itself, which is required when doing Elite clues.
There are a couple of clues I'm not sure about, specifically any with a number greater than 1,000 that doesn't have commas, but the information seems to be much more accurate than Sherlock's text was.
Also bundled are a few minor clue changes:

Thanks to Cook from OSW, it has been discovered that the HICK JET clue never had a challenge scroll.
The two Mandrith clues have been made distinct; now, one is for the question, and the other is for the Puzzle/Light box.

clues: make Anagram and Cryptic clues check all clue text is not changing cryptic clues to check all clue text? Is there a reason it is all being reordered?

Oops, that should say Cipher not Cryptic. I'll go rename the commit


Fix odoc selection with OCaml 5.0

ocaml-ci installs odoc without telling the solver about it. This means we might not get the latest version, and on OCaml 5 that means it might not install at all. As a hack, when choosing the opam-repository commit to use for builds, make sure it has odoc 2.1.1 at least.

Pushed live for testing.


A window can be focused through it has been hidden on windows

I have found that a window can be focused through it has been hidden on windows.
This led to the previous window can not regain the focus automatically.
Here are my codes:
function initShortcut() {
    globalShortcut.register('alt + shift + x', function() {
        showHideSwitch();
    });
}

function showHideSwitch() {
    if (mainWin.isFocused() && mainWin.isVisible()) {
        // mainWin.emit('blur');
        mainWin.hide();
        //after that case, the mainWin is not visible, but it's focused.
    } else {
        mainWin.show();
        mainWin.webContents.send('focus-on-input');
    }
}

When I input 'alt+shift+x', the main window will show or hide. But when the mainWin  were hidden by shortcut, it will keep up the focus.

Which operating system are you using?

On Windows a hidden window can also be focused, it is a bit weird though.
In your case I think the simplest fix is to check isVisible first:
if (mainWin.isVisible() && mainWin.isFocused())


Disk drive not working.

No matter what I do the disk drive doesn't work. just doesn't. it does not show the 3D texture of sticking out drives and takes 0rf/t. When I open the grid nothing showes up. I don't know what's happening. MC version: 1.12.2

Fill in the issue template


multiple is not working for me.

multiple is not working for me. On plunker it is working fine. But in my application it is not working. Giving Error : "http://localhost:5100/community/selectize/match-multiple.tpl.html"

Selectize doesn't support multiple (there is not template)
Closing as duplicate of #295


pypechain: add events to contract classes

The contract generated by templates/contract.jinja2 permalink has the TODO for this.

Resolved by #876


Port SnapshotVersion for Remote

Discussion
This is part of the C++ migration. Replace FSTSnapshotVersion by C++ SnapshotVersion in the Remote modulo.
Testing
Integration and unit test pass.
API Changes
n/a

Too late to see the message, I've already merged the other PR https://github.com/firebase/firebase-ios-sdk/pull/1182 ... I'll hold off this one.

Actually, now it is open. Merging.


UNRAID Docker startup error

Any thoughts on this error? Just installed, starts but immediately stops.
+++
INFO:     Started server process [1]
INFO:     Waiting for application startup.
spoolman.main              INFO     Starting Spoolman v0.14.0 (commit: 286e519) (built: 2023-12-02 11:16:35+00:00)
spoolman.main              INFO     Using data directory: /home/app/.local/share/spoolman
spoolman.main              INFO     Using logs directory: /home/app/.local/share/spoolman
spoolman.main              INFO     Using backups directory: /home/app/.local/share/spoolman/backups
spoolman.main              INFO     Setting up database...
ERROR:    Traceback (most recent call last):
File "/home/app/spoolman/spoolman/env.py", line 89, in get_port
return int(port)
^^^^^^^^^
ValueError: invalid literal for int() with base 10: 'remove variable if not used'
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/home/app/spoolman/.venv/lib/python3.11/site-packages/starlette/routing.py", line 677, in lifespan
async with self.lifespan_context(app) as maybe_state:
File "/home/app/spoolman/.venv/lib/python3.11/site-packages/starlette/routing.py", line 566, in aenter
await self._router.startup()
File "/home/app/spoolman/.venv/lib/python3.11/site-packages/starlette/routing.py", line 654, in startup
await handler()
File "/home/app/spoolman/spoolman/main.py", line 87, in startup
database.setup_db(database.get_connection_url())
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/app/spoolman/spoolman/database/database.py", line 25, in get_connection_url
port = env.get_port()
^^^^^^^^^^^^^^
File "/home/app/spoolman/spoolman/env.py", line 91, in get_port
raise ValueError(f"Failed to parse SPOOLMAN_DB_PORT variable: {exc!s}") from exc
ValueError: Failed to parse SPOOLMAN_DB_PORT variable: invalid literal for int() with base 10: 'remove variable if not used'
ERROR:    Application startup failed. Exiting.
+++

You can't leave any of the DB variables empty in the Unraid Spoolman config. If you don't use them, remove them.


adding to manufacturing

Docs Build status updates of commit a86e7d5:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-PullRequestMergeConflict] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/MicrosoftDocs/msftdynamicsgpdocs, working directory: W:\e03p-s, target branch: HEAD, target branch commit: 99e65c1651193aeef0d4f1b203e65b24bc66049a, source commit: a86e7d5b1766184618676989f25e94fc6d6aa3f0.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
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.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit e4f8228:
:warning: Validation status: warnings

File
Status
Preview URL
Details

dynamics-gp/distribution/MfgSetup.md
:warning:Warning
View
Details

dynamics-gp/distribution/MfgSetup.md

Line 401, Column 1: [Warning-file-not-found] Invalid file link: 'media/ad05a25ecf94b6674192d1e77f8455cb.jpg'.
Line 519, Column 1: [Warning-file-not-found] Invalid file link: 'media/11e4e8e3673094ecc02c4f9337ec81df.jpg'.
Line 625, Column 1: [Warning-file-not-found] Invalid file link: 'media/a52df266f547d8a3c9978f9e0100909c.jpg'.
Line 683, Column 1: [Warning-file-not-found] Invalid file link: 'media/a2ab0984c7944fade6bb1f40b1e9bb9b.jpg'.
Line 797, Column 1: [Warning-file-not-found] Invalid file link: 'media/66babca6de2ed03cfbc56271d6c2ba7d.jpg'.
Line 880, Column 1: [Warning-file-not-found] Invalid file link: 'media/55bf80301c52d087d8fdb675407688b5.jpg'.
Line 1044, Column 1: [Warning-file-not-found] Invalid file link: 'media/4be79d01041c43492eb6665335d94b86.jpg'.
Line 1174, Column 1: [Warning-file-not-found] Invalid file link: 'media/16d612c45442124e29db5c28d021dec0.jpg'.
Line 1442, Column 1: [Warning-file-not-found] Invalid file link: 'media/19a5e4ca7d736395d9c686c7fb3983f8.jpg'.
Line 1605, Column 1: [Warning-file-not-found] Invalid file link: 'media/a23d2f092bf1cbd15aaa0433c67349fd.jpg'.
Line 1706, Column 1: [Warning-file-not-found] Invalid file link: 'media/a2ab0984c7944fade6bb1f40b1e9bb9b.jpg'.
Line 1928, Column 1: [Warning-file-not-found] Invalid file link: 'media/53fda1cff045b06b9fa37fef151f3aef.jpg'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
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.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit dcf057e:
:warning: Validation status: warnings

File
Status
Preview URL
Details

dynamics-gp/distribution/MfgSetup.md
:warning:Warning
View
Details

dynamics-gp/distribution/MfgSetup.md

Line 401, Column 1: [Warning-file-not-found] Invalid file link: 'media/ad05a25ecf94b6674192d1e77f8455cb.jpg'.
Line 519, Column 1: [Warning-file-not-found] Invalid file link: 'media/11e4e8e3673094ecc02c4f9337ec81df.jpg'.
Line 625, Column 1: [Warning-file-not-found] Invalid file link: 'media/a52df266f547d8a3c9978f9e0100909c.jpg'.
Line 683, Column 1: [Warning-file-not-found] Invalid file link: 'media/a2ab0984c7944fade6bb1f40b1e9bb9b.jpg'.
Line 797, Column 1: [Warning-file-not-found] Invalid file link: 'media/66babca6de2ed03cfbc56271d6c2ba7d.jpg'.
Line 880, Column 1: [Warning-file-not-found] Invalid file link: 'media/55bf80301c52d087d8fdb675407688b5.jpg'.
Line 1044, Column 1: [Warning-file-not-found] Invalid file link: 'media/4be79d01041c43492eb6665335d94b86.jpg'.
Line 1174, Column 1: [Warning-file-not-found] Invalid file link: 'media/16d612c45442124e29db5c28d021dec0.jpg'.
Line 1442, Column 1: [Warning-file-not-found] Invalid file link: 'media/19a5e4ca7d736395d9c686c7fb3983f8.jpg'.
Line 1605, Column 1: [Warning-file-not-found] Invalid file link: 'media/a23d2f092bf1cbd15aaa0433c67349fd.jpg'.
Line 1706, Column 1: [Warning-file-not-found] Invalid file link: 'media/a2ab0984c7944fade6bb1f40b1e9bb9b.jpg'.
Line 1928, Column 1: [Warning-file-not-found] Invalid file link: 'media/53fda1cff045b06b9fa37fef151f3aef.jpg'.
Line 2037, Column 0: [Warning-multiple-h1s] Multiple H1s are not allowed. You can only have one top-level heading.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
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.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


New dict implementation

A new Dict implementation based on Left-Leaning Red-Black trees (LLRB). Performance numbers can be found here: https://discourse.elm-lang.org/t/new-dict-implementation-for-elm-now-done/174/3
The code that is commented out helps validating the invariants of a LLRB tree. It's not something that should be built in production builds, or even used by regular users, but it's invaluable when making changes to the data structure.
Also includes code from justinmimbs (fromSortedList and friends).

New commit strips out the "sorted optimization" to save on compiled asset size.
The new Dict implementation is still faster than the current implementation, and without sorted opts it should also be smaller.


Options using Attributes with uppercase fail

Hi,
Noticed that when you apply options using the "data-date-"-attribute it fails to apply due to a lower-case in the opts_from_el function.
For example, the format will apply but the weekStart will not.
data-date-format="yyyy-mm-dd"
data-date-weekStart="1"
I've made a temporary quickfix on this, replacing the
out[inkey] = data[key];
with
out[opt] = data[key];
where I filter out "opt" based on the names in the defaults array.

The documentation states the following:
Most options can be provided via data-attributes. An option can be converted to a data-attribute by taking its name, **replacing each uppercase letter with its lowercase equivalent preceded by a dash**, and prepending “data-date-” to the result. For example, startDate would be data-date-start-date, format would be data-date-format, and daysOfWeekDisabled would be data-date-days-of-week-disabled

So weekStart should be applied as data-date-week-start, not as data-date-weekStart
<input type="text" class="form-control" id="myDatePicker" data-date-format="dd/mm/yyyy" data-date-week-start="2">

Here is a js fiddle demonstrating this: https://jsfiddle.net/hewhy2mm/3/

Thanks @frederikprijck for the solution! Closing the issue


Antares & chart.js

Discussed in https://github.com/antares-sql/antares/discussions/458

Originally posted by codehangen October 12, 2022
Hi all.
I would like to know if it would be possible to have graphics in antares sql?

It is possible since Antares is written in JS.


Fix unit test cases for code coverage

:thumbsup: Coverage increased from [%] to [69.4875%]

:thumbsup: Coverage increased from [%] to [69.4875%]


Add link to Slack in readme and remove old draft documentation

Add link to our public Slack workspace in the root README.

Looks like the link to Slack is already there.


Make loading extension classes from the Maven project classpath possible

Thanks for this PR. I'll play around with it as soon as I find time to do so.

Thanks for this useful PR, I can see the value in merging it in. I have a few more specific comments around the codebase, please take a look and tell me what you think?


Update audit.md

Added a note  on how to setup the log file and policy file for kube-apiserver deployed as a Pod.

@mokua Do you have a chance to look at @tengqm 's comments?

@mokua can you fix the yaml indenting as suggested by @tengqm. Given the age of this PR with no updates, the next docs reviewer may close it in 7 days. Thanks!

@mokua can you fix the yaml indenting as suggested by @tengqm. Given the age of this PR with no updates, the next docs reviewer may close it in 7 days. Thanks!

The patch has been updated to reflect the correct indentation as highlighted by @tengqm

@mokua there is still one place needs some revision, pls check. Thanks.

@mokua there is still one place needs some revision, pls check. Thanks.

@tengqm Updated the volumeMounts, kindly have a look.

Thanks.
/lgtm
/approve


Errors not passing through express.route

I attempted to write a middleware that can catch the errors that I throw inside app.get (and sister functions) and transform the response to a particular format.
Codebase here, NPM here.
The package works fine and handles the errors that are thrown inside the route definitions, but when routes are added using the app.use method, it does not handle it.
Example
this works completely fine

app.use('/user', UserRoutes)
app.use('/auth', AuthRoutes)

const foo = (req: Request, res: Response) => {
  throw new NotFoundException('err !')
}
app.get('/', foo)

app.use(HttpExceptionTransformer)
export default app

but when an error is thrown in either of UserRoutes or AuthRoutes, it does not handle them, and hangs up the connection. In UserRoutes, i throw an error using:

const router = express.Router()

/**
 * declaring user rotues that are nested under the /users scope
 */
router.get('/', async (req: Request, res: Response) => {
  throw new NotFoundException('why here !')
  const data = await UserController.getAllUsers()
  res.json(SuccessToResponseMapper(data))
})

and the application throws an error with the following trace.

/home/yash/Desktop/files/works/projects/simple-rbs/node_modules/http-exception-transformer/exceptions/HttpException.js:22
        var _this = _super.call(this, message) || this;
                           ^
GenericHttpException: why here !
    at NotFoundException.HttpException [as constructor] (/home/yash/Desktop/files/works/projects/simple-rbs/node_modules/http-exception-transformer/exceptions/HttpException.js:22:28)
    at new NotFoundException (/home/yash/Desktop/files/works/projects/simple-rbs/node_modules/http-exception-transformer/exceptions/NotFoundException.gen.js:22:28)
    at /home/yash/Desktop/files/works/projects/simple-rbs/src/modules/user/routes.ts:15:9
    at step (/home/yash/Desktop/files/works/projects/simple-rbs/src/modules/user/routes.ts:33:23)
    at Object.next (/home/yash/Desktop/files/works/projects/simple-rbs/src/modules/user/routes.ts:14:53)
    at /home/yash/Desktop/files/works/projects/simple-rbs/src/modules/user/routes.ts:8:71
    at new Promise (<anonymous>)
    at __awaiter (/home/yash/Desktop/files/works/projects/simple-rbs/src/modules/user/routes.ts:4:12)
    at /home/yash/Desktop/files/works/projects/simple-rbs/src/modules/user/routes.ts:14:51
    at Layer.handle [as handle_request] (/home/yash/Desktop/files/works/projects/simple-rbs/node_modules/express/lib/router/layer.js:95:5)
    at next (/home/yash/Desktop/files/works/projects/simple-rbs/node_modules/express/lib/router/route.js:137:13)
    at Route.dispatch (/home/yash/Desktop/files/works/projects/simple-rbs/node_modules/express/lib/router/route.js:112:3)
    at Layer.handle [as handle_request] (/home/yash/Desktop/files/works/projects/simple-rbs/node_modules/express/lib/router/layer.js:95:5)
    at /home/yash/Desktop/files/works/projects/simple-rbs/node_modules/express/lib/router/index.js:281:22
    at Function.process_params (/home/yash/Desktop/files/works/projects/simple-rbs/node_modules/express/lib/router/index.js:335:12)
    at next (/home/yash/Desktop/files/works/projects/simple-rbs/node_modules/express/lib/router/index.js:275:10)
[nodemon] app crashed - waiting for file changes before starting...

Hi @YashKumarVerma ! I bet that you are not using express 5 even though you are using async in your route handler. You either need to use express 5 to catch exceptions thrown in async handlers (since it adds async handler support) or use one of the several modules on npm to add this support into express 4 (or just call next with your error).

thanks for the quick help, fixed the problem !


Connection zombies on database!

Driver version
e.g. 12.4.1 or 12.8.1
SQL Server version
Microsoft SQL Server 2019 (RTM-CU27-GDR) (KB5040948) - 15.0.4382.1 (X64)   Jul  1 2024 20:03:23   Copyright (C) 2019 Microsoft Corporation  Developer Edition (64-bit) on Windows Server 2019 Standard 10.0  (Build 17763: ) (Hypervisor)
Client Operating System
Windows 10 Pro 22H2
JAVA/JVM version
Occurs e.g. in both 17.0.8 or 21.0.4
Problem description
SQL Server sp_who shows connections for processes that don't exist anymore.
Expected behavior
If the Java App closes the database connection, those connections should be not visible anymore on the SQL server.
Actual behavior
exec sp_who2   shows misc (many; increasing) "open" connections that are closed by the application.
Any other details that can be helpful
We inspected the thread dump of a Java VM and searched for threads that can be related to the connections. (We name a databse connection with the Java Thread ID !)  But non of the hanging connections belong to still existing Java threads.
Those zombie connections only vanish if we exit the Java application.  As we can't identify still running threads for this connections we think that the driver (or the driver's DLL ...) somehow does not relase those connections. Those DB connections only vanish if we stop/exit the Java application.
With our thread and connection naming, we are sure, that the originators of those connections are our Java application threads and that those threads don't exist anymore (not in thread dump) and those threads Java code have standard closing of connections included.
This does not occur for all connections. Some driver versions earlier, we didn't see this problem.

Download
https://www.mediafire.com/file/7ybpxn8lil1av70/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Wait ... we were able to reproduce 1 thread that left the connection open. We debugged the whole chain of destructors and found a link class that didn't overwrite the destruct() method. We were able to fix this and this connection vanished. => our fault.
I guess there are no finalizers for connections if no one is referenzing them. And finalize is deprecated now. Is there a way to get infos from the Driver what connections are open, who created them (stack trace?) and what their current state is?

Hi @JDBC-0,
We'll look into this.

No, you need to explicitly close all open statements (this is actually a common mistake).
We'll be closing this issue now, but let us know if you have anything further you'd like to discuss.

@Jeffery-Wasty : OK, we introduced some central list to remember open connections but this is not 100% perfect as we might forget some places where we open/close connections. The question is:  is the driver able to offer such lists/infos about open connections/statements? That would be a 100% complete single point of truth about open connections/statements.
Are open ResultSets also a problem concerning open connections?

No, the driver does not offer this. The user is given full responsibility in making sure any resources that need to be closed, are closed. I would need to check if this is offered in our logging somewhere, but that would not be a solution to your problem as keeping logging on continuously would be quite performance intensive.
I'm curious how this Java application is being worked on? Most IDEs and editors should give a warning if there are unclosed resources.
ResultSets should also be closed, but we haven't noticed as many issues related to unclosed ResultSets, as we have for statements and connections.


Add discovery support

@lukas-hetzenecker Here you have the discovery support 🕺

Perfect, works great! :man_dancing:

Is even more coming, or do you want me to publish a new release now? ;)

The final piece is to implement import of YAML config into a config entry and adding a reload service, so the YAML config can be reloaded without having to restart. Should be fairly straight forward to implement.

An oh, @lukas-hetzenecker, you should break out your artistic skills and submit a fabulous icon to the brands repo. It will make things look better.

@postlund that's all I can do, what do you think? :-)

@lukas-hetzenecker Fantastic! 😎

home-assistant/brands#2024 :raised_hands:

@lukas-hetzenecker The new logo looks swell 👍
I'm working on the YAML import now, it's basically done. There's one issue however: api_password. As I didn't implement support for it in the config flow, there's no "easy" way to to support it. Ok, maybe it's not that hard to implement, but since it's deprecated and less secure than access_token, may I suggest that we deprecate it now so I don't have to spend time adding support for it? It's a quick change to do for me.


Load GPU trained model on cpu or vice versa

Signed-off-by: Ketan Umare<EMAIL_ADDRESS>

gonna close this for now


Stop preventing extraction when a type parameter wouldn't bind correctly in a containing scope

It's not an issue because we'll just declare a corresponding type parameter on the extracted function and pass the original as a type argument.
Fixes #18142
Note that the first commit is actually https://github.com/Microsoft/TypeScript/pull/18423

@amcasey,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@amcasey,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


First Name and Full Name type limited by gender

Feature Request
Description of Problem:
The first name and full name data types currently don't have a concept of gender. If your profile has a gender field it would be possible for the profile to output a row with the name Jeff who is a female, for example. This doesn't make sense for a real world data set so a way to limit the sets of names by gender would be required.

There are a few considerations here:

Gender can be neutral, i.e. someone can decide to not associate themselves with male or female as such this should be considered in the generated data. I.e. a 'gender' column should support Male/Female/Neutral
Some names transcend both genders, such as Sam


Memgraph Support?

I see that memgraph support is on the README -- wondering if any of the contributors to this package know off the top of their head the things that would need to change in this library to enable support for Memgraph.
At the very least, version parsing here needs to be updated (looks like memgraph returns v5 instead of a semver).
I really like and appreciate the use of Pydantic at the core of this library, and would be down to contribute this feature so I dont have to use gqlalchemy.

I know there are some differences in indexing and constraints for sure, but i haven't really looked too much into memgraph. I have found this link from the docs listing some of them.
I haven't really got that much time right now since i am busy with some things in my life, but you are welcome to lay out a basic plan for what would have to be changed and how to best integrate this into the existing code base. Memgraph integration was more of a afterthough since the project for which i originally created this library for used Neo4j from the start but we later wanted to migrate to Memgraph for performance reasons.
If you have any ideas keep me posted.

I found some time to look over the codebase and pick out the parts which would probably need some refactoring or at least some small changes to support the syntax differences between the two DBs. Here is a list of things i found (though i might have missed some):

DB version parsing (as mentioned previously)

The parsing is already pretty basic and only exists because i could not be bothered to test this library against older versions of Neo4j since i did not need it

Indexing on Labels/Types/Properties/...

This could potentially be a bit more work since not all index types are supported for each DB

Uniqueness constraints

should be easy to implement since it is only a slight syntax change

Subqueries used in combination with EXISTS (can potentially be solved by using the exists() pattern function)

Could again be a bit more work, but it shouldn't be too complicated since pyneo4j only supports patterns that can not be nested

Conditional expressions using CASE

should be a easy to implement since the syntax is only slightly different but basically provides the same functionality

Usage of elementId is not supported in Memgraph

This is a bit more work since we somehow need to "disable" the elementId operator when Memgraph is used as a DB instead

As for how to best integrate it into the existing code base, i still don't have not decided on the best approach. As far as changes go, these affect the client class (indexing/constraints/version parsing), the query builder (basically all differences in query structure/unsupported functions) and the operator builder (for the elementId operator). Another concern would be typing based on the used DB, since the current typings would be wrong for Memgraph queries.
As for migrations, if the rest works they should work as well since they are basically only a few queries which actually interact with the database.
The best approach would probably be to integrate basic support for indexes/constraints/manual queries and tackle things like query building and operators as the underlying logic comes along.
I am going to keep you posted if i find the time to actually get started on this.


[1.20.1] KJS Ars Nouveau is asking for the wrong version of Ars Nouveau

It asks for version "<IP_ADDRESS>" or above but on Curseforge the mod is only "4.7.4"
Crash Log:
https://mclo.gs/Es7hnFp

Should be fixed with the latest release, it'll be up on Curseforge in the coming hours.


[FEATURE] Save unfinished message in chat room

Currentlny, when user writtes a message and then decides to leave a chat room without sending a message, the message disaperas. It could be saved and when user reopen a chat room, written message would be shown again with auto focus on the text field. For example, similar feature is in Messenger.
Current situation:

Feature implemented in Messenger:

I think, that I could work on that feature, but there is a question that need to be answered before writting an implementation. Should unfinished message be stored in SharedPreference  or maybe in local database with use of Andorid-Room library?

@Pancor Yes, you should save it on SharedPreferences.  Have a look on this class https://github.com/RocketChat/Rocket.Chat.Android/blob/develop-2.x/app/src/main/java/chat/rocket/android/infrastructure/SharedPrefsLocalRepository.kt if you want to work on it.

Does this issue also address the lose of a message when changing to another app and back to Rocket.Chat?
This always happens to me using Rocket.Chat+/1.1.0 on a Samsung Galaxy S6 (Android 7.0).


Multi factor authenication requested

Environment

Spring boot starter:

active directory spring boot starter

OS Type: Windows/Linux/MacOS

Java version:

11

Summary
We want to use azure-active-directory-spring-boot-starter 2.1.7 to login users to our web application. While the login works for some users, other are caught in a redirection loop that is eventually stopped by the browser they use.
In the backend log I can see a list of exceptions like this:

[Correlation ID: 59d22bd4-a14d-40cf-b112-0ee0aadfeed5] Execution of class com.microsoft.aad.adal4j.AcquireTokenCallable failed.
com.microsoft.aad.adal4j.AdalClaimsChallengeException: {"error_description":"AADSTS50076: Due to a configuration change made by your administrator, or because you moved to a new location, you must use multi-factor authentication to access '00000002-0000-0000-c000-000000000000'.\r\nTrace ID: cfa3b062-230d-4b3a-83bb-b78899106a00\r\nCorrelation ID: 59d22bd4-a14d-40cf-b112-0ee0aadfeed5\r\nTimestamp: 2019-09-27 12:47:55Z","error":"interaction_required","error_uri":"https://login.microsoftonline.com/error?code=50076"}

Expected Results
The login should either work or display a screen to the user asking for MFA validation.
Actual Results
Users cannot log in.

Hi:
Thanks for your feedback, We will inspect it and give you a response later.

Another thing that might be relevant is, that we log in using federated accounts. When we login to the azure portal using the same accounts, we have to answer two MFA challenges - one for our company and a second for the tenant owner.

Hi @chufgard,
sorry for the late response, now I am going to investigate your issue now. Could you provide more details? For example, your WebSecurityConfigurerAdapter config, your properties(except sensitive data). And if you can provide a sample project, that will be better.

@chufgard  I have tried some samples, and I can't reproduce your issue.

Hi @chufgard,
sorry for the late response, now I am going to investigate your issue now. Could you provide more details? For example, your WebSecurityConfigurerAdapter config, your properties(except sensitive data). And if you can provide a sample project, that will be better.

Sample project will be difficult - at least since the whole stuff works like a charm if I deploy it on my local machine. Only running in azure it breaks.
We are assuming that the problem might be caused by a federated AD scenario plus a conditional access policy (access only from company IPs) bound to the ad where we are federated into.

Hi @chufgard,
sorry for the late response, now I am going to investigate your issue now. Could you provide more details? For example, your WebSecurityConfigurerAdapter config, your properties(except sensitive data). And if you can provide a sample project, that will be better.

Sample project will be difficult - at least since the whole stuff works like a charm if I deploy it on my local machine. Only running in azure it breaks.
We are assuming that the problem might be caused by a federated AD scenario plus a conditional access policy (access only from company IPs) bound to the ad where we are federated into.

yep, I got your point. But there still need some investigate to prove it and find a solution. Could you provide a sample project, I will try it by running it in Azure. BTW: how do you run your application in Azure? By app service or VM?

The application is deployed in a kubernetes cluster.

The application is deployed in a kubernetes cluster.

okay, I got it. And I still need your properties and WebSecurityConfigurerAdapter or some minimal sample codes, not a sample project. So I can reproduce your issue, otherwise, it is difficult for me to reproduce it. Could you provide these to ours?

hi @chufgard, I got a question, where did you get that error log? In your application backend log? Could you give more logs about the error? Because we don't use aad SDK in AzureActiveDirectorySpringBootStarter, so it's strange that you get that log in the backend.

Hi @chufgard, could you provide more logs?

https://github.com/microsoft/azure-spring-boot/pull/815/files


Request Code

Hi,
Where do we put the request codes? I propose in the ServiceRequest.code field and letting the related observations refer to the service request via the Observation.basedOn field.
An additional question, which code set do we use for the request codes? The international guidance has Snomed CT as an example, but I assume that won't cover every request code? Perhaps we should also specify an order of code sets like we did for the observations?
Best,
Thibault

I propose to do the same thing as in: https://build.fhir.org/ig/hl7-be/hl7-be-fhir-laboratory-report/guidance.html#guidance-for-codes-and-values-in-the-observation-resource

To me that is not an issue for laboratory results; that is for requests - for me this is out of current scope!

The analysis order codes from the order cannot be mapped one to one with the lab result codes from the report. So this is not relevant. This should be considered when doing the order.


Is there a reason to not use the "kubernetes" tfstate backend?

Hi,
I love the idea behind a terraform CRD (I'm watching this space and just discovered your project).
I'm wondering if there is a reason for not chosing https://www.terraform.io/docs/backends/types/kubernetes.html as a default backend? Is this because of the secret-size limit (1MB) ?
Thanks!

I believe that the kubernetes backend is only available with Terraform >= 0.13.0. This project supports >= 0.11.9 (or something like that) so it would make the default backend incompatible with earlier versions.
There is a customBackend feature to allow users to specify their own backends. I just tried, however, and it looks like if "kubernetes" backend is selected, the role for the tf-runner needs to be updated to this:
# rbac role rules for the tf-runner pod
rules:
- apiGroups:
  - ""
  resources:
  - configmaps
  - secrets # needs to be added
  verbs:
  - '*'
# and leases needs to be added
- apiGroups:
  - coordination.k8s.io
  resources:
  - leases
  verbs:
  - '*'

limitations
I could see some limitations in the kubernetes backend since it requires Secrets permissions to access tfstate. Hypothetically, If state data needs to be read from another namespace for a certain module, there would be an issue getting access to it.
solution?
I think I might make some changes to read for backend "kubernetes" to add RBAC permissions to the tf-runner.  That would allow someone to use "kubernetes" backend by adding this to the terraform spec:
  customBackend: |-
    terraform {
      backend "kubernetes" {
        secret_suffix    = "state"
        in_cluster_config  = true
      }
    }

I believe that the kubernetes backend is only available with Terraform >= 0.13.0. This project supports >= 0.11.9 (or something like that) so it would make the default backend incompatible with earlier versions.
There is a customBackend feature to allow users to specify their own backends. I just tried, however, and it looks like if "kubernetes" backend is selected, the role for the tf-runner needs to be updated to this:
# rbac role rules for the tf-runner pod
rules:
- apiGroups:
  - ""
  resources:
  - configmaps
  - secrets # needs to be added
  verbs:
  - '*'
# and leases needs to be added
- apiGroups:
  - coordination.k8s.io
  resources:
  - leases
  verbs:
  - '*'

limitations
I could see some limitations in the kubernetes backend since it requires Secrets permissions to access tfstate. Hypothetically, If state data needs to be read from another namespace for a certain module, there would be an issue getting access to it.
solution?
I think I might make some changes to read for backend "kubernetes" to add RBAC permissions to the tf-runner.  That would allow someone to use "kubernetes" backend by adding this to the terraform spec:
  customBackend: |-
    terraform {
      backend "kubernetes" {
        secret_suffix    = "state"
        in_cluster_config  = true
      }
    }

Hello @isaaguilar and kudos for this nice project!

This project supports >= 0.11.9 (or something like that)
This is an information I failed to find. It means we may encounter some restrictions depending on the versions of the providers we use for eg? How can I quickly figure out the compatibility level with Terraform?

I didn't delve into your code yet but what would be the "cost" to support latest Terraform version? Is it just a matter of adding new attributes to structs and update the CRD or is it more complex? Thanks.

@asaintsever FWIW we're migrating our BDD terraform testing on the Jenkins X project over to using this most excellent operator - totally loving it so far.
We're currently using the kubernetes back end - as it was the simplest thing to do to start with. The only thing to be careful of is make sure the backend namespace is the same as the namespace you are creating the Terraform resources inside otherwise I hit RBAC issues.
e.g. here's our templated Terraform we're using: https://github.com/jstrachan/jxr-versions/blob/tf-operator/.lighthouse/jenkins-x/bdd/tf.yaml#L37-L44
Though the main downside is k8s clusters ideally would be cattle not pets; so using (say) a bucket is easier to backup + survives your k8s cluster being recreated (e.g. if you change region or something)

@jstrachan interesting to know you are more than giving a try to this operator. I am also on the verge on starting a PoC to assess it. Thanks for your template. I will likely also make use of the k8s backend as a starting point but will quickly change to S3.


Dim non-religious units when religion lens is enabled

From the feature request documented in #106 - when the Religion lens is enabled, the flags of all non-religious units are dimmed.  When the lens is disabled, they're restored.
Normal Lens:

Unit that can be moved: Not dimmed (alpha 1.0)
Unit without moves, or sleeping/fortified: Dimmed (alpha 0.65)

Religion Lens Enabled:

Religious Unit that can be moved: Not dimmed (alpha 1.0)
Non-Religious Unit that can be moved: Dimmed (alpha 0.65)
Unit without moves, or sleeping/fortified: Dimmed More (alpha 0.35)

Edit: longer version of the gif where I select the unit to show it does the same thing... can't embed it here, though: https://i.imgur.com/k3s9YbT.gif

As a general comment - this is actualluy a good example of the feature that imho should be toggable via an option.  Plus it is a clear CHANGE from vanilla behavior. Visuals are always subjective and I can easily imagine that some people would NOT like to have such a change forced onto them.

Also - even looking at an example you attached. Since the lens itself is yellow, the actual dimming doesn't stand out much. And there are religion colors much darker in use - I can imagine dimming will be even less visible over there.

Plus another concern - normal units are still "usable" when dimmed. This may be confusing as people are used to dimming meaning "no more moves". The more I think about this whole idea, the less I am convinced that this is a way to go.
And another point for an option to toggle on/off.

Was thinking about that with the other units - normally clicking another unit cancels the lens, which is why I left them there.  Clicking elsewhere has the same effect tho, I think, so making the other units fully invisible may be fine... but if one is clicked on while trying to scroll the map, hmm...
Probably a good idea to also prevent the show path on mouse over from appearing for at least the non-religious units, too.  I will look further at it tonight.

And making this a configurable thing is also something that can be done, yes

I was also thinking about making them invisible. So, maybe the way to go is to have 3 options: vanilla, dimmed, invisible.

Updated.  I like the hiding things, but for the time being stuck with making things transparent as the default

Wow, now it is really clear where religious units are.

I tested it a bit and it is working pretty well.
One issues I noticed.

Put Religion lenses on.
Select a religious unit.
The lenses are toggled OFF - this is weird because when a religious unit is selected they are toggled ON by default.
When the next religious is a autoselected - then the lenses are back ON.

I am not sure if this behavior is because of the changes or is it also in the master CQUI (so, not related to dimming).

Yeah, that behavior exists in the master branch.  Going to look and see if it's something that can be controlled.

I checked to see if that was somehow being done in the OnUnitSelectionChanged functions, but no luck - the Religion lens would still report "Active" upon returning from BASE_CQUI_OnUnitSelectionChanged in both worldinput_CQUI and unitflagmanager_CQUI.
It's something to do with how the lenses are applied when using the Lenses panel.  When you select Religion from that panel, it shows the "Key" box in the upper right corner of the screen,  When clicking on the religious unit, it does not show that Key.  Note if you have a religious unit that has a queued path, that takes priority over the Religious view (when you clikc on it, it shows the path instead of the colored hexes for the religion)

Fixed the branch merge, can I get an approval?  I don't think the issue with the way the lens is applied with the lenses panel should be blocking...


Inline component alert suppression

Is there an existing issue for this?

[X] I have searched the existing issues

Feature description
In certain situations, bumping a component version is not technically possible.  To avoid recurring alerts I would like a feature that allows suppression at either the per-line level or component version level.  This would be more granular than excluding entire manifest files.

Can you share the manifest, and how the exclusion would look like?

Closing due to lack of user response. If you care to provide the details @yeikel requested, happy to reopen.


Add swarm Node field into ContainerInfo

Swarm introduces the new field Node in the /containers/{name}/json endpoint

@vieux  How could I move forward with this one ?


Broken relative imports inserted with multiple module paths

Hi - I have run into some frustrating behaviour with relative import suggestions. I hope this issue can be helpful.
When using a project that has multiple separate module hierarchies, it is impossible to configure the automatic imports in a way that is satisfactory.
The behaviour we want is that imports from the local module are imported relatively, and imports across modules are imported absolutely.
As far as I can tell, there is no way to get this behaviour, no matter what I set in python.analysis.include, python.analysis.exclude and python.analysis.extraPaths.
I should be able to specify which directories are expected on the python path at runtime, but this does not seem possible.
Environment data

Pylance version: v2024.9.2
OS and version: Fedora Linux 40 Workstation
Python version: Python 3.12.6 (system provided)

Code Snippet

A minimal example can be found here: https://github.com/maxeonyx/pyimport-issue-minimal
To run the project, use:
PYTHONPATH=appsrc:depsrc python -m appmodule

Repro Steps

Clone the following repo: https://github.com/maxeonyx/pyimport-issue-minimal

Comment out your VSCode user settings so that they don't interfere with the workspace settings in this project.

Open the repo in VSCode.

Open appsrc/appmodule/__main__.py. Take the suggested action. The created import is not correct

Expected behavior
Relative mode
The following imports should be inserted in relative import mode:
from . import do_thing
from sharedmodule.patterns import retry

Absolute mode
The following imports should be inserted in absolute import mode:
from appmodule import do_thing
from sharedmodule.patterns import retry

Actual behavior
The following imports are what are actually inserted:
Relative mode:
from . import do_thing
from ...depsrc.sharedmodule.patterns import retry

The ...depsrc.sharedmodule.patterns import is NOT correct as per my python.analysis.* settings.
Absolute mode
This works correctly:
from appmodule import do_thing
from sharedmodule.patterns import retry

Note
Note that ...depsrc is NOT expected to be on the python path at runtime. I have set python.analysis.autoSearchPaths to false so the workspace root should not be one of the search paths.
If we have the from ...depsrc.sharedmodule.patterns import retry version, when we run with the following command, we will get ImportError: attempted relative import beyond top-level package.
PYTHONPATH=appsrc:depsrc python -m appmodule

Thanks for the issue. This would be an enhancement request, being able to specify which paths should be absolute and which are relative. Probably requiring a new setting.

Transferring to a discussion item or up votes.


Delay asNavFor on(afterChange)

Is it possible to delay the firing of a asNavFor carousel using .on('afterChange')?
$('.carousel').slick({
  slidesToShow: 1,
  slidesToScroll: 1,
  infinite: true,
  asNavFor: '.carousel-images',
  autoplay: false,
  dots: true,
  speed: 500,
  fade: true
});

$('.carousel-images').slick({
  slidesToShow: 1,
  slidesToScroll: 1,
  infinite: true,
  asNavFor: '.carousel',
  autoplay: false,
  speed: 500,
  fade: true
});

$('.carousel').on('afterChange', function(event, slick, currentSlide, nextSlide){
  // Possibly something here to fire the method of $('.carousel-images')
});

Since you are using fade transitions, this is rather easy.
Delete the asNavFor option from both sliders
Change afterChange to beforeChange
Add this bit of code inside beforeChange
setTimeout(function(){
  .carousel-images.slick("slickGoTo",nextSlide);
},200);

This will delay the moving of .carousel-images by 200ms
For "sliding" transitions, this is something I am still struggling with


Mbed/merge

Hello, I merged the nordic changes in the targets, which caused conflicts. Could you please rebase to resolve conflicts? Thanks

Can you please sign http://developer.mbed.org/contributor_agreement/ ?

@0xc0170 Done the latter, will look at the conflicts.

@0xc0170 which specific commit is the problem?

I merged one PR which touched targets.py (MCU inheritance for nordic targets), I believe this one is causing conflicts. Just fetch and you will see it


Fixing newlines added after curly brace string index access

Fixes #2018
This fixes my most basic use case where I ran into an issue of extra
new lines, so it only deals with assignment of a string character to
another value (see the test for an example of what I mean).  There may
be other cases where newlines are added for this string index access
syntax, though there are language limitations as to what you can do to
a string offset.

Already fixed on 2.x line with #1955
Please, add the following utest:
           array(
                '<?php
                    if (1) {
                        echo $items{0}->foo;
                        echo $collection->items{1}->property;
                    }
                ',
            ),

Thank you @jaydiablo.


Read file with pagination

Description
Users should be able to read a CSV file with a pagination
Checklist

[x] Implement.
[x] Test.
[x] Refactor.
[x] Document.
[x] Update Work in progress section in changelog.

Done.


Setting multithreading  for tensorflowlite does not work！

Hi guys,
I set up tensorflowlite for four threads,but compared with single thread, the time consumption has not decreased. by the way, i quantizate my models.
Hope to get your help,ths!

@hzq-zjm
Can you please fill issue template.
Request you to share colab link or simple standalone code to reproduce the issue in our environment.It helps us in localizing the issue faster.Thanks!

The relevant codes are as follows：
private static Interpreter tflite_segm;
private static Interpreter tflite_ocr ;
/** Options for configuring the Interpreter. */
private static Interpreter.Options tfliteOptions_seg = new Interpreter.Options();;
private static Interpreter.Options tfliteOptions_ocr = new Interpreter.Options();
/** The loaded TensorFlow Lite model. */
private static MappedByteBuffer tfliteModel_segm;
private static MappedByteBuffer tfliteModel_ocr;

        tfliteModel_segm=loadModelFile("frozen_qnt",activity);
        tfliteModel_ocr =loadModelFile("model_12y4r",activity);

        tfliteOptions_seg.setNumThreads(4);
        tfliteOptions_ocr.setNumThreads(4);

        tflite_segm = new Interpreter(tfliteModel_segm,tfliteOptions_seg);
        tflite_ocr = new Interpreter(tfliteModel_ocr,tfliteOptions_ocr);

Only the CPU is used to run the model.

@hzq-zjm What model are you running?
Currently, only Conv2D and DepthwiseConv has multithreaded kernels.
So you will only get benefit of they are significant in your model.

@hzq-zjm What model are you running?
Currently, only Conv2D and DepthwiseConv has multithreaded kernels.
So you will only get benefit of they are significant in your model.

Thks for your reply. There are two models in my project, one of which is trained based on deeplab-v3 +, and the other is OCR model  based on densenet-18.  All models are quantized by post quantization, which makes multithreading not work?

Could you run the performance benchmark(https://www.tensorflow.org/lite/performance/benchmarks) on both models with the --enable_op_profiling=true?
It will give much more details.


Add 1.12pre5 support

Hi, I just helped for add 1.12pre5 support :)

x9nico committed on GitHub 4 minutes ago
if you do that, at least take your time to actually test what you write.....

This PullRequest doesn't include all the packet ID changes in pre5. Myles or I will update it when one of us is home, if nobody else did do it yet. (:


Use upstream yolo3 source as git submodule

Currently this repo uses a modified version of https://github.com/qqwweee/keras-yolo3 with tweaked imports - this should be replaced with a clean git submodule of the upstream source tree

@bleggett - Created initial yolo3-keras here https://github.com/jbondpk/keras-yolo3 taking the codebase from the current inventory and yolo object detection as a starting point.  Please review and fork to T-Mobile and then can then repoint these repos to it as submodules.  Let me know if you have any questions.  Thanks.

@bleggett - Closing this issue as related to prior merge request that was merged.


Assetion in std::string buffer::get_line(bool skip_whitespace), with MSVC build

sorry for awful screenshot :)
Offended code:
	std::string buffer::get_line(bool skip_whitespace) {
		...
		if (skip_whitespace) {
			auto count = mismatch([](char chr) { return !std::isspace(chr); }); // <--
			if (count == -1) {
				position(limit()); // the end of the buffer has been reached
			} else {
				position(position() + count);
			}
		}

		return tmp;
	}

char can be signed on some C++ implementations, and have negative values.
Issue was hit, when parsing world mesh in steam-eng version of gothic2

Hm interesting. To my knowledge isspace is supposed to take an int anyways (see cppreference)? Regardless I'll see if I can reproduce on MinGW on Linux so that I can actually see what's going on here :)

Fixed in 69cd380.


Inconsistent power cycles (cold boot, shutdown, restart) and possible solution

My system (T480, Mac Os Monterey 12.7, EFI 1.3.0.) was plagued with inconsistent power cycles (not shutting down properly, rebooting with errors etc...). Following Dortania's OC post install guide it could be related to USB mapping.
I went the USBToolbox route, generating a new UTBMap.kext in Windows and removing stock USB kext.
The system is stable now.

Thanks for posting this. I have some of the issues you mention. I'll give this a try!


Project is not mandatory to have any builds being run yet.

I think now understood the optional changes you were looking for 😄

The squash merge (・_・;)


Burkholderia ubonensis is not included in the Burkholderia cepacia complex and it should be.

User reports that ubonensis is part of the B. cepacia complex, but this is not reflected in PATRIC.  Looking at NCBI taxonomy, it should be.
https://www.ncbi.nlm.nih.gov/Taxonomy/Browser/wwwtax.cgi?mode=Info&id=101571&lvl=3&lin=f&keep=1&srchmode=1&unlock

Recent updates in taxonomy data at PATRIC has taken care of this. IT is now part of the complex.


Added ROCM specific sanity check for CI.

Added ROCM specific sanity check for CI.
Used the upstream sanity script as a template but removed the following sub tests:
buildifier
pip package licenses check
lib package licenses check
java package licenses check
pip smoke test
pip no cuda deps ubuntu
pip no cuda deps windows

jenkins : retest Ubunty-sanity please

jenkins : retest Ubuntu-sanity please

jenkins : test Ubuntu-sanity please

jenkins : retest Ubuntu-sanity please

jenkins : retest Ubuntu-sanity please

I have undone all the fixes to existing .py files and changed the new script to only capture commits for the current PR.  I am not sure if my approach is good enough since it makes assumptions that I am not sure would be valid in all situations.  Please see the new function get_changed_files_for_this_branch.
The function get_py_files has been modified to call this function to get a complete list of changed files for the current PR.

Redid the script to downgrade pylint and python to the versions used by the upstream Ubuntu-sanity check.

Redid the script to downgrade pylint and python to the versions used by the upstream Ubuntu-sanity check.


jenkins_pkg_url invalid for LTS on RedHat

I believe this line:
https://github.com/geerlingguy/ansible-role-jenkins/blob/master/vars/RedHat.yml#L4
should look like this:
__jenkins_pkg_url: https://pkg.jenkins.io/redhat{{ '-stable' if (jenkins_prefer_lts | bool) else '' }}

or at least without it, the role does not install LTS version of a specific version.

For Debian the same issue.
https://github.com/geerlingguy/ansible-role-jenkins/blob/master/vars/Debian.yml#L4


62 chromes file explorer crashing application

Refactoring handleFileChange in UploadDocumentForm and changing input name for upload-doctype from file to upload-doctype

currently an issue with input html elements on macOS 13.2.1 and Chrome 111. working with legacy chrome versions. merging for minor improvements.

I'm not sure if it'll still crash if we remove the accept attribute or change it to just accept=".pdf, .docx, .doc"
Perhaps the latest version of Chrome/MacOS have a hard time with text or rich text files?


CI is failing

https://github.com/thebrowsercompany/swift-build/actions No new build is available since Mar 08.
The VSdevenv setup fails to finish.
I tried to fix the issue and it seems it was resolved with this patch and this patch. Task was passed on the fork, the job stopped at linking because of out of memory though.
@compnerd may this help to fix the CI.

Close as it's been fixed


Theme for latest Sonarr V3 partially broken (<IP_ADDRESS>9)

Todays update of Sonarr V3 broke the theme in the "add new show" section, see screenshot.
https://imgur.com/a/yIwNFHv
Edit: Some additional info might be good, using Sonarr with nginx (sub filter) not Stylus or sorts.
The first entry is how it looks with mouse over, the others without.

@quorn23 sonarr version?

Fixed on https://github.com/gilbN/theme.park/commit/8c334503c142124ece3c24e60077d9389834de2d

Noted, next time not only in the title :D

no worries :)

I am seeing this same visual behavior on Lidarr <IP_ADDRESS>8.

I am seeing this same visual behavior on Lidarr <IP_ADDRESS>8.

I'm not seeing that. Im on version <IP_ADDRESS>1

I am running a little newer version.  Are you able rerecreate it with 1558?  I just tested it again on a mobile browser this time and still see the same issue.

Are you on develop? I'm using the  linuxserver/lidarr latest

Nightly actually.  The last update was Dec 16th.

@lostandlost
fixed https://github.com/gilbN/theme.park/commit/c0c08561081d8ef332275f4f90c69edcffebcd84


Zone clean-up?

Hello there,
I have used viomi integration for several months, either from you, nergal, or from someone upstream or downstream, can't remember. However it stopped working so I moved to this viomi integration, because of how easy it is to download/update via HACS.
However this flavor of integration is missing zone clean-up, which I miss very much. Any ideas if you plan to add support? Or is it HASS issue? Or maybe I am doing something wrong?
I have used such definitions:
          service: xiaomi_viomi.clean_zone
          target:
            entity_id: vacuum.robot
          data:
            zone:
              - - -1.1 #left
                - 1.3 #up
                - 0.2 #right
                - 0.4 #down
            repeats: 1

Now the service is non-existent in this integration. ANY ideas? Thank you.

This would be very useful indeed.


conformance: fix artifact layers content in setup

This was a typo in the test setup. We create two artifacts A and B with both config.media-type and layers model.

With zot ...
HTML report was created: /local/rchincha/go/src/github.com/opencontainers/distribution-spec/conformance/report.html

Ran 74 of 79 Specs in 0.197 seconds                                                                                SUCCESS! -- 74 Passed | 0 Failed | 0 Pending | 5 Skipped
PASS


fix: use setuptools.find_namespace_packages

Similar to https://github.com/googleapis/google-auth-library-python/pull/1205
https://packaging.python.org/en/latest/guides/packaging-namespace-packages/#native-namespace-packages

This change was overtaken by events. Namely the merge of PR #707 which replaced multiple references related to packaging and namespace.
Closing this PR.


sq inspect should offer a flag to only show summary metadata

Is your feature request related to a problem? Please describe.
sq inspect will shows schema metadata (tables, etc). This is as expected. However, it's sometimes the case that we just want to see the "summary" metadata. Here's the current output of sq inspect -y:
handle: "@sakila/pg12"
location<EMAIL_ADDRESS>name: sakila
name_fq: sakila.public
schema: public
driver: postgres
db_driver: postgres
db_product: PostgreSQL 12.13 on x86_64-pc-linux-musl, compiled by gcc (Alpine 12.2.1_git20220924-r4) 12.2.1 20220924, 64-bit
db_version: "12.13"
user: sakila
size: 17613679
table_count: 29
tables:
- name: actor
  name_fq: sakila.public.actor
  table_type: table
  table_type_db: BASE TABLE
  row_count: 200
  size: 73728

A particular use case for this is when the user submits a bug. It would be good to be able to see the metadata, without the schema info (which may contain sensitive info).
Describe the solution you'd like
There should be a flag --summary or --no-schema that omits the tables field. E.g. the output of sq inspect --yaml --summary might be:
handle: "@sakila/pg12"
location<EMAIL_ADDRESS>name: sakila
name_fq: sakila.public
schema: public
driver: postgres
db_driver: postgres
db_product: PostgreSQL 12.13 on x86_64-pc-linux-musl, compiled by gcc (Alpine 12.2.1_git20220924-r4) 12.2.1 20220924, 64-bit
db_version: "12.13"
user: sakila
size: 17613679

Implemented in v0.39.0.


fix unstable no-cluster test

Windows issue will be debugged in https://github.com/nearform/node-clinic-doctor/pull/248


Question: how can I capture error parameters with revertedWith?

I have custom errors in my contract, that return useful info, such as:
error MaxSupplyReached(uint16 left);

I have test code that expects the revert:
await expect(token.connect(user3).mint(10, {value: tokenPrice(10)}))
        .to.be.revertedWith('MaxSupplyReached');

I want to test that the value of left is 9. I tried using .withArgs but apparently that does not work with revertedWith.
How can i capture error parameters in Waffle?

I have the same issue

Duplicate of https://github.com/TrueFiEng/Waffle/issues/511.

Please refer to https://github.com/TrueFiEng/Waffle/blob/1e598c58f5454235e4b1eaaa2e104f20dc3977eb/waffle-hardhat/test/reverted.test.ts#L27-L34
Note that this works only with hardhat - that is @ethereum-waffle/provider doesn't support custom errors for now.


Codeigniter 3.1.4 and PHP 8.1

I am getting several error messages while running my script. By far, I am not an expert but I am not able to fix it. It seems like an issue due to PHP 8.1 for me.
Severity:    8192
Message:     str_replace(): Passing null to parameter #3 ($subject) of type array|string is deprecated
Filename:    /myproject/system/core/Output.php
Line Number: 447
Backtrace:
File: /myproject/index.php
Line: 294
Function: require_once
PHP Fatal error:  Array and string offset access syntax with curly braces is no longer supported in /myproject/system/libraries/Profiler.php on line 108
Fatal error: Array and string offset access syntax with curly braces is no longer supported in /myproject/system/libraries/Profiler.php on line 108
A PHP Error was encountered
Severity:    Compile Error
Message:     Array and string offset access syntax with curly braces is no longer supported
Filename:    /myproject/system/libraries/Profiler.php
Line Number: 108

https://github.com/bcit-ci/CodeIgniter/issues/5879
use CI latest version (3.1.13 at the moment)

Maybe it would be a good idea to migrate to CI4 since you are using PHP 8.1

the version 3.1.13 is compatible with php 8.1. I have 2 projects working very well with these especifications.

Sorry, but I can't locate any of the mentioned lines/function calls by using the code from the develop branch... If you will point them by using a GitHub link to the actual lines, I will have a look at them.


Add python support to VS. I don't want other crappy IDEs handling my python code anymore make a python IDE part of VS

Like what I said above all the IDEs I find are slow to load for me the VS ones load faster than Photoshop CS6. So, I need such IDE that is great, knows what to highlight, and also knows other languages as well. and is better than Notepad++. As well as a way to debug python code in it and ways that is suggests on making it better as well as a simple way of python 2.x to 3.x conversion without less of functionality. Sadly I have some python 2 code that uses some bytes things that python 3 does not let me use.

This is already a thing

oh ty so it comes with 2015? or just a addon?

I think could be a problems for the libraries, but it's a good idea.

oh btw I did found a few python files that (when cythonized) makes the Incremental Linker use up over 1 GB of RAM when it should not on a computer that has a 4 GB of RAM actually in it (I also seen python.exe use that much RAM to in some cases so I think it is in some way die to the Incremental linker and not it's code).

Personally, I'd like cheap my python things in JetBrains PyCharm. They have free community version.

For me PyCharm Freezes a lot (and sadly so does Visual Studio 2015 but not as much as PyCharm)

@AraHaan The place for feedback on VS is http://connect.microsoft.com/VisualStudio


Remove archived frameworks

I haven't checked them all but the following frameworks are still displayed on the website and don't seem to be maintained.

https://github.com/tsukuyomi-rs/tsukuyomi
https://github.com/finchers-rs/finchers

Closed by #242

Thanks for noticing!


api/server/router/system: fix info versioning

cc @cpuguy83 @vdemeester
Signed-off-by: Antonio Murdaca<EMAIL_ADDRESS>

@thaJeztah should this go into 1.12.3?

Oh, yes, looks like it!
ping @vieux

Actually, looks like it's not in 1.12? https://github.com/docker/docker/blob/1.12.x/api/server/router/system/system_routes.go#L31-L41


TypeSpec Validation result reporting

Is it correct for the job to fail when there are warnings in the chuchu job?
Example: https://github.com/Azure/azure-rest-api-specs-pr/pull/12045/checks?check_run_id=13252511527

@maririos: Are you asking if it's by design that TypeSpecValidation fails when the TypeSpec compiler emits warnings?  If so, I believe the answer is "yes", for two reasons:

The TypeSpecValidation tool was implemented this way before ownership was transferred, so I assume this was the desired behavior.
We want to keep all specs in our repos as clean as possible (no compiler warnings, formatted using tsp format, etc)

In this scenario, I think the spec author should fix the warnings before the PR is merged.


New command: viva engage community set. Closes #6279

Closes #6279

Thanks, we'll try to review it ASAP!

@Jwaegebaert ok, I will update the util based on
https://github.com/pnp/cli-microsoft365/pull/6371/files#diff-5de2c70818e31135768e63ab9bfdd4a0e8e58de9dc4464645f762ce17e6fd382

Hi @Adam-it, I was waiting to avoid update vivaEngage utils multiple times. Update right now, so it's ready for review

Hi @Adam-it, do you plan to review this PR in the following days/weeks? There are other commands related to viva engage community being merged in the meantime, so I'm only rebasing again and again (this one and #6286). It seems to me better to wait until the other commands with higher priority are merged. Now it's useless effort for me.

Hi @Adam-it, do you plan to review this PR in the coming days/weeks? In the meantime, other commands related to the viva engage community are merged, so I'm just rebasing (this and #6286). It seems to me better to wait until the other commands with higher priority are merged. Now it's useless effort for me.

TBH I was going to to that last time but I didn't manage. Today we started internal code freeze till upcoming major release which will be at the end of this month and during this time we only may merge pr-major pr-priority and docs PRs. This means this PR may proceed no sooner than at the beginning of November

ready to merge 🚀

Merged manually. Thank you for your awesome work! You Rock !🤩


can i use crud to upload image ?

Hello
i need to make input file to upload image file and save file name to database table
i need to rename file name before save the file name in DB
any suggestions?
Thank you in advance

Hey!
Thank you for help this is my code to do that
first step create new field type
file_image.blade.php
in laravel_app_root/resources\views\vendor\backpack\crud\fields
`

{{ $field['label'] }}
    @foreach ($field as $attribute => $value)
        @if (is_string($attribute) && is_string($value))
           @if($attribute != 'value' && $attribute != 'type')
                {{ $attribute }}="{{ $value }}"
            @endif
        @endif
    @endforeach

    value="{{ old($field['name']) ? old($field['name']) : (isset($field['value'])?$field['value']:'') }}"
    >
@if (isset($field['hint']))

{!! $field['hint'] !!}

@endif

second step:
in ArticleCrudController in news system or whatever you use
    $this->crud->addField([    // Image
        'name' => 'image',
        'label' => 'Image',
        'type' => 'file_image'//,
    ]);`

third step
install this package
image lib
four step
load lib image and function to the Article model
`
use Image;
// my own functions
public function setImageAttribute($value) {
// use Intervention image or whatever you want to process that image
 	$image=$value;
    $input['imagename'] = time().'.'.$image->getClientOriginalExtension();
    $img = Image::make($image->getRealPath());

    $destinationPath = public_path('/uploads/news');
    $img->resize(750, 450, function ($constraint) {
	    $constraint->aspectRatio();
	})->save($destinationPath.'/'.$input['imagename']);

    $destinationPath = public_path('/uploads/news/thumb');

    $img->resize(100, 100, function ($constraint) {
	    $constraint->aspectRatio();
	})->save($destinationPath.'/'.$input['imagename']);

	//  $image->move($destinationPath, $input['imagename']); // for no resize

   $this->attributes['image'] =   strtolower($input['imagename']);

}

`
Thank you tabacitu
i was going to rebuild the article module from scratch if i couldnt use image file upload

i try to make image upload as you write, and it's failed at this line:
$input['imagename'] = time().'.'.$image->getClientOriginalExtension();
i guess its happened becuase the form has not have the enctype="multipart/form-data"
do you know how can i solve it?

@BIGITIMI , if that's the case, you can use a overwrite the create.blade.php and edit.blade.php files (where the form is instantiated) with your own files, by placing them in resources/views/vendor/backpack/crud. Does this work for you?

@tabacitu Hi, i change the form to use  enctype="multipart/form-data" and it work good,
but i think the "file upload" field must build-in in the backpack, this is a basic field to work with
thanks for your reply.

@BIGITIMI completely agree with you. That's why this issue is still open. Cheers!

Hi guys,
I've just pushed the upload field type, in Backpack\CRUD v3.1.5. Would love to have your feedback and tests. Here's how it works. And here's the commit:
And here's my question to you: should we also automatically delete the file from disk when the entry is deleted? How about if it has soft deletes? This might cause confusion, because I'm pretty sure if somewhere in your app you use batch operations to delete, that won't trigger the "deleted" event on the model.
Thanks, cheers!

@tabacitu I've been having a play around with this feature and have come across 2 bugs
Firstly - When trying to remove files form the multi_uploader, it fails to do this (might be linked to the second bug)
Secondly - When it gives you the preview url - it doesnt take into account the disk path e.g my path was /uploads/folder_1/subfolder_1 - so the file uploaded there correctly, however it doesnt return the preview field with the disk location included.
I've created a video demonstrating
https://drive.google.com/file/d/0Bxqca2RsGkfTd2FiVE9FNzlQcGM/view

Hi @OwenMelbz , thank you for the video - it was perfect to debug the issue.
Let's tackle the first issue first - not removing the files.

My first try would be to use attribute casting on the Model - if you cast "image" to "array" does that still happen? It's now also in the docs, as Step 3.

    protected $casts = [
        'image' => 'array'
    ];

Also, I didn't quite understand the way you used the models. In TeamCrudController I see you're using the Team model, but then you put the mutator on the AdminModule model?

Cheers!

As it happens, I've refactored a lot of what I had, so will need to try out the casting again later to see what happens, although I'm pretty sure I did cast it.
Regards to the AdminModule - that was because all my models extend AdminModule, I just happened to have accidently recorded the video while not in the sub class, so in that instance it was because my Team model was extending it anyway so didn't overly matter, but will do it properly for when I look into casting it again.

Great. Tell me when you see if it is.
For the second issue, I've just pushed an update:

if on your field you don't specify a "disk", the file URL will be generated using asset($your_db_entry);
if you DO specify a disk, the file URL will be generated using the Storage facade;

This way, everyone can use it:

if you store your uploads in the /public folder, you don't need to specify the disk;
if you store your uploads is the /storage folder or in the cloud (S3, etc), you need to specify the disk;

Here's how that looks:
$this->crud->addField([
            'type' => 'upload_multiple',
            'name' => 'photos',
            'label' => 'Photos',
            'upload' => true,
            'disk' => 'public' // <- this line is new
        ]);

This should solve that one. Please confirm though :-)
Cheers!

I've just tried it again and it seems to work for most use cases. If there's still a bug for it, it's most likely something very particular. @OwenMelbz , if you're still having issues, we'll track it down and squash it :-)
I'll go ahead and close this issue so we don't clog the section. Feel free to open it again if you have follow-ups.

i have install the laravel backpack but i don,t have crud folder in resource/view/vendor/backpack i have only two folder in backupmenager and base i want to display image and my other question is that
how can i resize image in laravel backback and how how i change its design kindly help
reply me on<EMAIL_ADDRESS>iam waiting for your response

Hi @aamirkhan37557 ,
You can use any library you want to resize images. The cleanest way would be to use a Mutator in your Model. If you're uploading an image, you have probably already defined that. Just add your resize logic using a library like Intervention Image.
Hope it helps. Cheers!


Feature/add actions

Fancy a quick review of what I have done for DA tools.
They work. Hoping the == ewhelan/AccordDaTools works to save on GitHub "credit"!

Great that AccorDaTools is exploring the use actions too. You don't have to worry about the Github credits. Github actions are free for public repos.

Thanks for the review and the info regarding free actions for public repos!


Upgrade Bitcanna to v3.0.0

Hi @tombeynon
Not sure why test build is failing
Submitted PR before waiting for tests to complete 😬
ERROR: failed to solve: process "/bin/sh -c apt-get update &&   apt-get install --no-install-recommends --assume-yes ca-certificates curl wget file unzip liblz4-tool gnupg2 jq pv &&   apt-get clean" did not complete successfully: exit code: 127
Error: Process completed with exit code 1.

Hey @rodri-r - the image needs to be a debian based one, and Go 1.21 doesn't support buster so we also need to bump that to bullseye. Essentially use:
        GOLANG_VERSION: 1.21-bullseye
        DEBIAN_VERSION: bullseye

You can see the same config used for dYdX: https://github.com/akash-network/cosmos-omnibus/blob/7456b93913b74b211fc5b7b96fed784f80f74fe6/dydx/build.yml#L15-L16

ghcr.io/akash-network/cosmos-omnibus:v0.4.7-rc3-bitcanna-v3.0.0

@rodri-r could you leave the Omnibus version as it was (cosmos-omnibus:v0.4.6 instead of cosmos-omnibus:v0.4.7) and only bump the chain version number? The Omnibus version is updated in bulk when a new release is ready.
If it's a hassle I can fix it before I merge, just let me know :)

Changing now, thought you needed that changed from one of your last comments :)

@tombeynon I think it is good now, although Kava test run is failing for some reason :D


Change rst signal according to _neg / _pos

Currently the reset signal is always defined as "rst".
When one sets the register_reset  to "_neg" the signal does not reflect the active state anymore.
So would it be possible to make it "rstn" when register_reg = <>_neg ?

Yes it would be possible. For now, this was done intentionally in order to simplify templates. However, it looks like simple jinja macro is required.


assertj 3.18.1

Is there some background to this? No strong objection to upgrading if it's blocking you.  I often bump the versions, so not against this. But please explain the context for needing this version?
Otherwise it's just a change that could cause some issues for OkHttp maintainers chasing down some surprising difference.

Is there some background to this? No strong objection to upgrading if it's blocking you.  I often bump the versions, so not against this. But please explain the context for needing this version?
Otherwise it's just a change that could cause some issues for OkHttp maintainers chasing down some surprising difference.


Feature: define credentials with env variables

I see on the version 2.0 roadmap that there are updates planned for the AWS credential management process overall, but I wanted to call attention to an immediate issue I'm having with the current version: Scotty.js doesn't work with CircleCI 2.0.
There's currently a bug on Circle's end where AWS credentials are not getting added to ~/.aws/credentials. Most people are using the aws CLI and can therefore work around it by passing their credentials as build environment variables AWS_ACCESS_KEY_ID and AWS_SECRET_ACCESS_KEY.
After digging into the scotty source code a bit, it looks like the AWS.SharedIniFileCredentials class that's getting used under the hood only looks for credentials at ~/.aws/credentials and doesn't check for credentials in any of the other config settings defined here under "Configuration Settings and Precedence".
I realize this is a bug on Circle's end and not on this repo, but I just wanted to call it out since it's forcing me to drop Scotty.js. I have never used the AWS JS SDK but will try to find an alternative to SharedIniFileCredentials that will support my use case later this week.

@wkentdag I think I did a quick fix on master branch, can you try it? If it works I'll publish it on NPM tomorrow. If not I'll test it tomorrow.

@stojanovic It's working as expected. No credentials issue anymore on master.

@stojanovic sorry for extreme delay - it works! Think you can close this out.

Sorry for delay on my side. Closing it. It's on NPM as 1.10.2.


Create video tutorial showing users how to run their own test cases

Description: Create a tutorial video that guides users through running their own test cases using MotorEase.
User Story: #3 Tutorial Video
Estimated effort: 5 hours
Dependency: #14 #13
Acceptance criteria:

[ ] Brief overview of running custom test cases and it's importance
[ ] Ensure that MotorEase is already set up using Docker or Python environment
[ ] Show what type of data MotorEase requires as input and where to store them within the project directory
[ ] Add real GloVe Embeddings
[ ] Replace placeholder GloVe modle
[ ] Execute the MotorEase Python script MotorEase.py
[ ] Ensure clear video and audio quality

This issue will be closed and merged with another tutorial video


Composition Trait not updated to remaster text yet.

The trait uses the old text which references verbal and somatic components.

Fixed by #13339


Frog-api service

Student, group: Korotkikh Viktor, 3530901/90203
URL: https://github.com/vitekkor/frog-api
Git approach: tbd

`+


Copy paste box

What feature or improvement do you think would benefit Files?
I mentioned this before, I hope this gains attention, all file managers have the same problem.  Its called copy paste, then hunting down the destination folder you want to paste to or move too.   There is only way to solve this problem.

Requirements
speed
Files Version
<IP_ADDRESS>
Windows Version
10.0.22621.2506
Comments
No response

@meemore thank you for your feedback, can you confirm if this is the same as #12490?


test push

test push

can't merge , please auto create test file and test end auto del this file

please modify "github.com/yanunon/oss-go-api/oss" code in this project

all file make and annotate ，please use english

ok,modify later

backend.go please modify to goroutlne

backend load driver please modify to reflection

close


0.13.0 rejects a type parameter which is unused except through its associated type

This program compiles with ouroboros 0.12.0, but fails with 0.13.0:
#[ouroboros::self_referencing]
struct Thing<T: SomeTrait> {
    outer: Vec<<T as SomeTrait>::Assoc>,
    #[borrows(outer)]
    inner2: &'this (),
}

trait SomeTrait {
    type Assoc: 'static;
}

error[E0284]: type annotations needed
 --> src/foo.rs:1:1
  |
1 | #[ouroboros::self_referencing]
  | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ cannot infer type for type parameter `T` declared on the struct `BorrowedFields`
  |
  = note: cannot satisfy `<_ as foo::SomeTrait>::Assoc == _`
  = note: this error originates in the attribute macro `ouroboros::self_referencing` (in Nightly builds, run with -Z macro-backtrace for more info)

This only occurs when T itself is not used in the fields, so T could be replaced with a parameter taking the associated type directly, instead of going through the trait at all, but perhaps whatever the new restriction is could be fixed, or documented so that users know how to handle it.

I can confirm that the original case from which I reduced the example code now also compiles in ouroboros 0.14.2.