.help json value request

for example:
{
    "name": "bash",
    "files": [
        {
            "path": "${HOME}/.bash_history",
            "movable": true,
            "help": "Export the following environment variables:\n\n```bash\nexport HISTFILE=\"${XDG_STATE_HOME}\"/bash/history\n```\n"
        }
    ]
}

currently, i am trying to compeletely declutter and optimize xdg-ninja.
please close this issue immediately if you do not agree with me.
i request to simply change the json files to have markdown rendering removed. so the output could be something along the lines of:
Export the following environment variables:
$ export HISTFILE=\"${XDG_STATE_HOME}\"/bash/history

instead of a very colorful and in my opinion, bloated output.
most programs follow the 'standard' i had said above and doing this could actually save me some time trying to optimize xdg-ninja.

i can't really optimize xdg-ninja without a clear understanding of apply_shell_expansion() and decode_string(), i thought doing something like this wouldn't be so hard. but so far i've reached 80 SLOC

Sorry, but I won't remove the markdown.
What exactly are you trying to optimise?

Sorry, but I won't remove the markdown.

Okay. i guess i will try to add it in my optimized version.

What exactly are you trying to optimise?

Everything. There is many duplicate lines and ton's of complicated scripting that could have been so simple, i can show you how it looks so far but it doesn't work as i completely removed the markdown rendering as that could make the script much more simpler and easier to manage.

apply_shell_expansion expands environment variables in strings,decode_string does what the comment says: Replace \n with literal newline and " with ", normalize number of trailing newlines to 2.

expands environment variables in strings

Example?

➜ apply_shell_expansion "\$XDG_CONFIG_HOME"
 /home/b3nj4m1n/.config

Hmm. Okay. i will soon open a PR and explain each change i did.


The Width of NavigationViewItem can not be auto-sized

Describe the bug

To Reproduce
Run Gallery -> Navigation -> NavigationView -> WPF UI Fluent NavigationView
Expected behavior
"DashBoard","Settings" can be showed fully.
Screenshots
No response
OS version
windows 11
.NET version
.NET 8
WPF-UI NuGet version
3.0.3
Additional context
No response

Hello,
any update on that Bug - or is there a Workaround?
Thanks!


AL: Go! does not create the .alpackages folder

Describe the bug
When using AL: Go! to create a new package, the .alpackages folder and files are not created.
To Reproduce
Ctrl+Shift+P to activate the command pallet
Select AL: Go!
Define folder
Select 9.0 Business Central 2022 release wave 2
Select Microsoft cloud sandbox
New package is built but with the following error:
The package cache c:\Users\my username\Documents\AL\ALProject./.alpackages could not be found.
Expected behavior
The result of the AL: Go! process should be a new package that compiles cleanly
Business Central: Version: W1 Business Central 20.0.41086.41354

Hasn't it always worked this way, and you just need to Download Symbols first?

Hasn't it always worked this way, and you just need to Download Symbols first?

Yes, the symbols have to be downloaded from the server.

Closing this since downloading symbols will fix this issue, and there is an easy shortcut via creating the folder manually.

The problem is that it cannot download the symbols. Also creating the folder manually is not a solution but a woraround. Anyway have a nice day


fuse3: Set FD_CLOEXEC on duplicated fuse session fd

The duplicated file descriptor returned by dup will not have FD_CLOEXEC flag set:

The two file descriptors do not share file descriptor flags (the close-on-exec flag).  The close-on-exec flag (FD_CLOEXEC; see fcntl(2)) for the duplicate descriptor is off.

(The file descriptor returned by libfuse3 has FD_CLOEXEC set as of https://github.com/libfuse/libfuse/commit/4a6fb6768aada05e456b2e8cbbcf5660637564ce)
It's good practice to set the close-on-exec flag on file descriptors to avoid "leaking" fds when calling exec*().

Turns out this contributes to test flakiness for us - with enough test concurrency and leaking enough fds things go boom. There are other ways to fix our particular issue, but it seems apt to set FD_CLOEXEC on this fd.
(In addition from the "leaking" fd from fuser, libfuse3 also leaks another fd)

lgtm, thanks!

Thanks for the fast response time, appreciated!


wrong method-name in warning when not explicitly define precisions for decimals

The (new) warning produced by EF when not explicitly define precisions for decimals seems to suggest the usage of the wrong Fluent API method-name. (ForHasColumnType() instead of HasColumnType()).
The warnings xml-name value is LogDefaultDecimalTypeColumn.
See here: https://github.com/aspnet/EntityFrameworkCore/blob/c786c334ff1a403a3d72f33c9374eb0a70e7c3f8/src/EFCore.SqlServer/Properties/SqlServerStrings.resx#L145
Here is the link to the method in question:
https://docs.microsoft.com/en-us/dotnet/api/microsoft.entityframeworkcore.relationalpropertybuilderextensions.hascolumntype?view=efcore-2.0
Further technical details
EF Core version: 2.1.0-rc1-final
Database Provider: Microsoft.EntityFrameworkCore.SqlServer
Operating system: Any
IDE: Any

@riscie - Would you like to submit a PR to correct this?

Sure, I would like to do so 👍

@riscie - Can you open a PR so we can look at what is going wrong? Perhaps current code is not propertly generated. For this small change hand editing .resx & designer.cs should also be fine.

@smitpatel - I did the change by hand.


XLSX file names get switched to XLS, causing format incompatibility import issues

From: https://github.com/CartoDB/cartodb/blob/0418ccd6bc640b92291c7a035e97b721bdca50db/services/importer/lib/importer/downloader.rb#L395
Should fix: https://github.com/CartoDB/cartodb/issues/1896 and https://github.com/CartoDB/cartodb/issues/6278

Seems to be happening only when name_from_http_headers is not retrieving anything, so it gets name_in. It seems that AWS is not setting the Content-Type correctly for .xlsx files (it's actually empty).

[283, 292] in /home/ubuntu/www/production.cartodb.com/current/services/importer/lib/importer/downloader.rb
   283:         self.source_file  = SourceFile.new(filepath(name), name)
   284:       end
   285:
   286:       def name_from(headers, url, custom=nil)
   287:         debugger
=> 288:         name =  custom || name_from_http(headers) || name_in(url)
   289:
   290:         if name == nil || name == ''
   291:           name = random_name
   292:         end
(byebug) name_in(url)
"202.xls"
(byebug) url
"https://dl.dropboxusercontent.com/u/xxxxx/cartoDB_upload_test%202.xlsx"
(byebug) name_from_http(headers)
"cartoDB_upload_test 2.xlsx"


ShortcutDialog improvements.

As @Neverbirth suggested, possibly closes #784.
The change fixes numerous bugs caused from the previous structure, and also adds new features:

All conflicts are highlighted in Color.DarkRed.

View custom shortcuts without moving your mouse to click the check box. Start the filter with the key "*".

View all conflicts with the filter starting keyword "?".

Have options filter keywords combined in any order.

The dialog is now very very grumpy that it won't close until you resolve all conflicts.

All changes are only applied when the dialog closes.

Removing the check box and more improvements on ShortcutDialog comes in a different branch that will come if this PR is merged.

Updates from this PR

Updates from the branch that can come after this:
...
...AAAAHHH I CAN'T CHECKOUT MY CURRENT BRANCH ADFJHSDFUAKHS GIT PLZ WHY

After 069fda0 the dialog checks for conflicts on open.
Then I figured out that the default shortcuts have some conflicts...

I don't know about those particular cases nor how well FD allows for this but some shortcuts may be shared depending on the context and state of the application.

If that is the case, then I think it only over complicates the shortcut structure...
We would need to implement a VS like structure where the dialog also lets you specify the context in a combo box, that if there is a 'Global' shortcut present it conflicts with shortcuts in other contexts...
Also I'm sure they're not shared shortcuts with different contexts. Have a look at those: (don't mind my horrible mouse-drawing skillz)

Pressing Ctrl+Shift+T only does "Undo Close".

Once this PR is merged, there are some more cool things to come.

As I said, I don't know about all of those, but for example Ctrl+F10 is that way, when you are debugging it takes precedence over Edit Macros.

I think RunToCursor taking precedence when active is just coincidence, since the name "FlashDebugger.dll" is before "MacroManager.dll", therefore FlashDebugger registering its event handler before MacroManager. If FlashDebugger was named like NewDebugger, RunToCursor would never fire.
I think if contextual shortcuts are meant to be implemented, the registration of shortcuts require a context parameter, which a Global context value takes the lowest precedence when dispatching.

I think the grumpy dialog could say: There are conflicts, is that ok?

This PR has now conflicts.

@elsassph sounds good :)
but kind of a different matter but I think at least the default shortcuts shouldn't have any conflicts...

True, what is in conflict atm? @Meychi ?

@elsassph the word "conflict" now has conflicts...
If you meant Git conflicts, they are resolved
If you meant shorcuts, this comment has the picture of the default shortcut conflicts.
Otherwise, ignore and continue :P

This can also wait for #1013 to use FindImage16() for the button images.


Incorrect english in warning message

c:>az login
Note, we have launched a browser for you to login. For old experience with device code, use "az login --use-device-code"
Should be:
Note, we have launched a browser for you to login. For the old experience with device code, use "az login --use-device-code"

Hello,
I have just started to look into this issue. You should be expecting a pull request within the next few minutes. Thanks!

@jiasli thank you for letting us know. Would you like for the warning to be completely removed in the PR?

PR is merged.


[riverpod_lint]Multiple plugins can't be enabled

Describe the bug
I don't think it's a riverpod_lint issue...but
When I try to set multiple "plugins" in analysis_options.yaml, it points out "Multiple plugins can't be enabled".
Is there any workaround?
To Reproduce
I tried to set dart_code_metrics and riverpod_lint.

Expected behavior
I want both "plugins" to work.

No. Pick one :)
Nothing I can do.
But custom_lint support any number of custom_lint_plugin technically dart_code_metrics could be implemented with it


Strange whitespace when using cookie controller

There is a strange white space on the top of page when cookies are set. which disappears as soon as cookies are deleted.

I had the same issue. There seems to be a little bit of front-end code that shipped with the extension to solve this:
st.markdown(
    f"""
        <style>
            .element-container:has(iframe[height="0"]) {{ display: none; }}
        </style>
    """,
    unsafe_allow_html=True,
)

Not sure why it does not work? Maybe this fix broke with the update of streamlit. Anyway I added this to the bottom of my app.py as a workaround for now.


research for user evaluation

research papers added for user evaluation

Research papers added for future references look good.
Retrospective updates for the Sprint 9 still need some changes.


[rparticle] Proposal particle system for raylib

Even if nothing stops you to make our particle system, maybe is a good idea to have a module who bring you a simple but effective way to make particles and particle Emitters. Using GPU particles for better performance.
Libraries like Libgdx, Monogame or LOVE2D has very interestin particle system
https://github.com/libgdx/libgdx/wiki/2D-ParticleEffects
http://rbwhitaker.wikidot.com/2d-particle-engine-3
https://love2d.org/wiki/ParticleSystem
What do yo think?

@uSources raylib also has one: https://github.com/dbriemann/libpartikel 😜

Yes, I saw yesterday, but its on Alpha stage and seen discontinued, the last commit is at 31 May 2019.


Datagrid String Filter malfunction or not working at all

Describe the bug
The issue resides in the fact that after we fetch the data that we want to populate cells with from an endpoint, and create a datagrid, after filtering the results of a specific column (let's name it 'name') by providing a string that does not exist in the column (some gibberish data), and clearing the filter afterwards, no data is populating the datagrid.
If we filter based on an existing string, then no problem arises.
The example I've given in Stackblitz vaguely represents the issue I have. It has another issue however - the applied string filter does not work at all.
How to reproduce
The example is based on an already opened bug.
link to Stackblitz
Steps to reproduce the behavior:

Go to the Name colum filter and click it
Enter some data
Observe that even when entering strings, subset of items present in the name of some column, no result is shown after the filter is applied
After clearing the filter, we observe that no results actually populate the datagrid

Expected behavior
Versions
App

Angular: [9.1.12]
Node: [10.19] - locally
Clarity: [3.1.7]

Device:

Type: [MacBook]
OS: [Mojave]
Browser [FF]
Version [80.0.1]

This seems more like a question about how to implement server driven datagrids. The stackblitz provided seems to mock a backend response but I'm confused for how it filters items in the clrDgRefresh method using the state.
Here is an example that implements a server driven datagrid and mocks the backend. Filtering / sorting etc is already set up.
In a real application, a network call is made to the server that handles the filtering, sorting and pagination with the state information passed when the network call is made. When the response is resolved, the datagrid item list is updated for that state.
I'm going to convert this to a discussion because it is related to implementing a server driven datagrid and not directly related to an issue with the datagrid itself. Let me know if I am missing something here and I am happy to adjust accordingly.


benchmarking related updates

mostly to bring local script more inline with cloud versions and add parameterization via env vars.

build

build


Fix INC model loading

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Failed to add Reference

I am getting this error when installing to my android project :
Failed to add reference. The package 'MonkeyCache' tried to add a framework reference to 'System.Runtime' which was not found in the GAC. This is possibly a bug in the package. Please contact the package owners for assistance. Error HRESULT E_FAIL has been returned from a call to a COM component.
Target Framework : .Net Standard 2.0 ( converted from PCL)
Xamarin.Forms version : <IP_ADDRESS>0555
Android Target Framework Version : 8.0

try <IP_ADDRESS>-beta


Added validation to image is already in use

what's is inside?

Added validation to check if name is already in use.

How is it build?

Added function to retrieve image register by name.
Check if result data from database.
Added valiadation on ValidateRequest function.
Create new Error string.

/retest


Rename QtcCppcheck -> Cppcheck

Just a proposal. None of Qtc plugins (at least the built in ones) has this prefix...

Hi! At first, thanks for all these PRs.
About this one: i don't think that plugin's name matters a lot, but changing it might cause some inconvenience in installing/upgrading/scripting. So i'll keep the old one for now.

How about changing it, but keeping the library target file name only? I'll try that.

AFAIK Qtc identifies plugin by its name, so it would treat renamed as new one. And "wrong" file's name with "correct" plugin's name (and visa versa) doesn't sound better then "wrong" names everywhere :)
So, although the renaming idea is right i general, i don't think it's worth making it.

It works fine here. Qtc uses the json file for the name, the file name shouldn't really matter.
If you insist, I'll keep the plugin name unchanged, but I'd still want to change at least internal and display names.

I'm still holding the point, that having equally (even not so precise) named entities is better then having a mess. Renaming plugin's name in json file or internal constants will lead to loss of user settings. Renaming file's name - to duplicate plugins after updating. Changing namespace/class names is acceptable, but i've told about it in the beginning.
And the main doubt is a reason of changes: i just don't see any real advantages here. Such renaming doesn't clarify anything or fix conflicts - it's just a matter of taste.

Actually what disturbed me were the display names. I care less for the file name or the plugin name, but having a tab named "QtcCppcheck" in Analyzer is bad...

Well, that's acceptable. UI text does not impact on anything except translations. I'll change it.


Add Iris flower Classification

I would like to add a project on iris flower classification as part of GSSOC'24 where I will use different models like KNN, Random Forest, and SVM. I have worked on this before, and I believe it can be completed in one day.
please assign this issue to me @sanjay-kv

Make sure you create seperate folder and add the same

Sir I would Like to add coding the project Bcauz I have already done IRIS Classification Project If you want to see my project Can You Assign Me as a contributor.


Recent versions of the chrome driver support the attach_file feature (fixes #135)

could you please also remove the test for the chrome attach_file disallowance?
https://github.com/dmfrancisco/splinter/blob/master/tests/test_webdriver_chrome.py#L26

Yes, I will also run all the tests to check if everything is okay

Ok, I tested it and it runs the 449 tests with 0 failures and 0 errors


Style Transfer with Neural Networks

Related Issues or Bug

Issue: Style transfer results may sometimes generate outputs with undesirable artifacts or loss of content structure.
Bug: Some users might encounter issues related to runtime efficiency or memory overflow when working with large images.

Fixes: #209  (Replace with actual issue number if applicable)

Proposed Changes

Improved the model’s content and style weight parameters to better balance content retention and style application.
Enhanced image preprocessing pipeline to handle higher-resolution images more efficiently.
Added more detailed comments and documentation in the code to guide users on adjusting key parameters like content_weight, style_weight, and num_steps for personalized results.

Additional Info
Some users reported difficulty understanding how to adjust the loss parameters for better results. The README provides examples and clearer instructions within the notebook.

Screenshots

Original
Updated

These changes aim to enhance both the user experience and the model’s performance when applying style transfer techniques.

@UppuluriKalyani I have completed the task. You can review it further. Thank you!


[core] Add stereotype property to project/distribution

Valid values: [none, cli, desktop, web, mobile]
Packagers may use this property to determine which template should be used.
Distributions inherit the set value from project.
Default value is none.

The following should happen:

add a Stereotype enum with the following values: NONE, CLI, DESKTOP, WEB, MOBILE.
add a stereotype property to Project and Distribution model types (core model & gradle model).
ProjectValidator should make sure the property has a value, use NONE as default.
DistributionValidator should make sure the property has a value. Inherit the value from project if no value is set.

cc @diakogiannis


Convert sphere format to wav format  find: unknown predicate `--wav_dir'

After I installed the dependency following the instructions, I ran './run.sh --stage 0 --tag first --task sep_noisy --id 0' to start but found this  problem:
/home/dell/users/lpp/ss/asteroid-master/egs/wham/ConvTasNet Convert sphere format to wav format find: unknown predicate --wav_dir'`
So I print the sphere_dir and wav_dir in convert_sphere2wav.sh, found the 'Sphere_dir' is '--wav_dir' and 'wav_dir' is empty:
Sphere_dir: --wav_dir wav_dir:
I am confused now. Should I fill the 'sphere_dir' in run.sh？ And what should i fill? Or why the readme not tell this?
I will appreciate it for any help.

Can you please show me the top of the run.sh file?

Can you please show me the top of the run.sh file and the exact command you ran?

`#!/bin/bash
Exit on error
set -e
set -o pipefail
Main storage directory. You'll need disk space to dump the WHAM mixtures and the wsj0 wav
files if you start from sphere files.
storage_dir=
If you start from the sphere files, specify the path to the directory and start from stage 0
sphere_dir=  # Directory containing sphere files
If you already have wsj0 wav files, specify the path to the directory here and start from stage 1
wsj0_wav_dir=
If you already have the WHAM mixtures, specify the path to the directory here and start from stage 2
wham_wav_dir=
After running the recipe a first time, you can run it from stage 3 directly to train new models.`

Can you please show me the top of the run.sh file and the exact command you ran?

Actually, i donot kown how to fill 'sphere_dir' .

Do you have the wsj0 data?
(Please use triple quotes ``` for  blocks of code)

Do you have the wsj0 data?
(Please use triple quotes ``` for blocks of code)

OK, I see. I donot have waj0 dataset. Is this the key to this problem? But I cannot find this dataset. Would you like to share the dataset with me? I will appreciate it very much.

It's under license, I can't.
Have a look at LibriMix instead

It's under license, I can't.
Have a look at LibriMix instead

OK. If I start from the sphere files, Do I still need wjs0 dataset ?

Or wjs0 is made of sphere files?


Set the dist file suffix from the auto-detected version.

To override the suffix, set adaptorlib.version on the command line.
Previously, adaptorlib.suffix was set before adaptorlib.version was
set, so you always had to set adaptorlib.suffix on the command line.
Now, most uses of the "package" target, including releases, can be
done without command line arguments.

Brett pointed out that the this adds a "v" prefix to the file name suffix. Previously, the JAR file manifest used the v, but not the file suffix. I'll take a look at that.

Will there be a later update to http://googlegsa.github.io/adaptor/docs/ReleaseProcess.html ? (which is different repo)


Code coverage on CI

As It was done for kubernetes-ingress-controller , would it be relevant to have a code coverage report on go-kong ?

Yes, we'd love a PR!


load_dataset gives "403" error when using Financial phrasebank

When I try to load this dataset, I receive the following error:
ConnectionError: Couldn't reach https://www.researchgate.net/profile/Pekka_Malo/publication/251231364_FinancialPhraseBank-v10/data/0c96051eee4fb1d56e000000/FinancialPhraseBank-v10.zip (error 403)
Has this been seen before? Thanks. The website loads when I try to access it manually.

Hi! You seem to be using an outdated version of datasets that downloads the older script version. To avoid the error, you can either pass revision="main" to load_dataset (this can fail if a script uses newer features of the lib) or update your installation with pip install -U datasets (better solution).


Bump BPMN deps

Closes https://github.com/camunda/camunda-modeler/issues/4035
Closes https://github.com/camunda/camunda-modeler/issues/3070
Closes https://github.com/camunda/camunda-modeler/issues/4070
Closes https://github.com/camunda/camunda-modeler/issues/4051
Closes https://github.com/camunda/camunda-modeler/issues/4037
Closes https://github.com/camunda/camunda-modeler/issues/3991
Closes https://github.com/camunda/camunda-modeler/issues/3895

This PR is currently blocked as it fails locally (and in the distribution) with the following error message:
ERROR client omit is not a function
    at ../node_modules/bpmnlint-plugin-camunda-compat/index.js (webpack://camundanode_modules/bpmnlint-plugin-camunda-compat/index.js:37:0)
    at __webpack_require__ (webpack://camunda-modeler-client/webpack/bootstrap:19:0)
    at __webpack_require__.t (webpack:/camunda-modeler-client/webpack/runtime/create fake namespace object:10:0)
    at async (webpack://camundanode_modules/@camunda/linting/lib/Linter.js:145:0)
    at async (webpack://camundanode_modules/@camunda/linting/lib/Linter.js:73:0)
    at async (webpack://camunda-modeler-client/src/app/App.js:945:0)
    at async file:///home/nikku/camunda/projects/bpmn.io/camunda-modeler/client/build/bundle.js:78366:14

Weird. We've been using omit in there for forever. 🤔

Found during integration testing the changes: https://github.com/camunda/camunda-modeler/issues/4098.


Potential Update: Emojis

Using Inquirer

Ask a user if they want to include emoji's in their header.

Include or exclude emojis based on response

Feature added as of: commit 396ac8c


High CPU usage in timeline on homepage

Sometimes in case there are 20+ items on a timeline the page becomes very slow responsive. Some JS script constantly consumes 100% of a single CPU core.
The main suspect is plugins_public/jquery.ba-resize.min.js

Actions menu in Timeline card was updated in Ticket #3026.
Current menu doesn't have so many recalculations.


Duplicated uuid on 2 Android app use react-native-device-info

I am developing 2 different apps use react-native-device-info package. At now, I installed 2 apps on an Android device and 2 apps have the same uuid. What's the problem with me?
React Native version: 0.46.4
react-native-device-info: 0.11.0
Device info: Nexus 5 with Android 6.0.1

Documented in README.md


Looted items do not transfer to player inventory.

This is the same issue as https://github.com/jopeek/fvtt-loot-sheet-npc-5e/issues/76 , which states "Player can loot the item, the looting appears on chat but the item dissapears and is not added to the player inventory." I am using LootSheet 2.5.1 and Foundry 0.7.9, 5e Ruleset 1.2.2, and no other mods.
Of note is that newly created campaigns in 0.7.9 do not have any issues, but the problem is present in a previous campaign that I created in 0.6.6 and LootSheet 2.1.X. If it would be helpful, I can upload the campaign in question.
(I'm not super familiar with github so I wasn't sure if I should create a new issue or attach it to the existing closed one. Apologies if I submitted the report incorrectly)

Updated potential workaround: Creating a new character DOES allow the looting functionality to work properly in my old campaign, but characters existed before the campaign, etc. were updated are apparently broken. Duplicating old characters does not fix the functionality.

Updated potential workaround: Creating a new character DOES allow the looting functionality to work properly in my old campaign, but characters existed before the campaign, etc. were updated are apparently broken. Duplicating old characters does not fix the functionality.

I’m also using an old campaign like you and it seems to work for my players there. Does it happen on newly created lootsheet characters too?

On Jan 17, 2021, at 6:00 AM, Kailan7401<EMAIL_ADDRESS>wrote:

This is the same issue as #76 , which states "Player can loot the item, the looting appears on chat but the item dissapears and is not added to the player inventory." I am using LootSheet 2.5.1 and Foundry 0.7.9, 5e Ruleset 1.2.2, and no other mods.
Of note is that newly created campaigns in 0.7.9 do not have any issues, but the problem is present in a previous campaign that I created in 0.6.6 and LootSheet 2.1.X. If it would be helpful, I can upload the campaign in question.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

I’m also using an old campaign like you and it seems to work for my players there. Does it happen on newly created lootsheet characters too?

On Jan 17, 2021, at 6:00 AM, Kailan7401<EMAIL_ADDRESS>wrote:

This is the same issue as #76 , which states "Player can loot the item, the looting appears on chat but the item dissapears and is not added to the player inventory." I am using LootSheet 2.5.1 and Foundry 0.7.9, 5e Ruleset 1.2.2, and no other mods.
Of note is that newly created campaigns in 0.7.9 do not have any issues, but the problem is present in a previous campaign that I created in 0.6.6 and LootSheet 2.1.X. If it would be helpful, I can upload the campaign in question.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

I see yeah the sheet data structured changes at some point. You’ll have to create new characters from scratch I’m affairs.

On Jan 17, 2021, at 6:08 AM, Kailan7401<EMAIL_ADDRESS>wrote:

Updated potential workaround: Creating a new character DOES allow the looting functionality to work properly in my old campaign, but characters existed before the campaign, etc. were updated are apparently broken. Duplicating old characters does not fix the functionality.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

I see yeah the sheet data structured changes at some point. You’ll have to create new characters from scratch I’m affairs.

On Jan 17, 2021, at 6:08 AM, Kailan7401<EMAIL_ADDRESS>wrote:

Updated potential workaround: Creating a new character DOES allow the looting functionality to work properly in my old campaign, but characters existed before the campaign, etc. were updated are apparently broken. Duplicating old characters does not fix the functionality.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

This seems to be fixed?


Create work dir before service starts as it does not persist across restarts

As /var/run is mounted as tmpfs in ubuntu 16.04 it doesn't persist across system restarts.
I have made user permissions only apply to the start job, and the pre start job can then create the work dir in /var/run (/run) which is owned by root. Therefore the prestart job runs as root and creates the dir if it doesent already exist.

I am however, contemplating this as a better alternative?
https://www.freedesktop.org/software/systemd/man/systemd.exec.html#RuntimeDirectory=
I am not yet that well versed with systemd...

[Current coverage][cc-pull] is 73.68%

Merging [#64][cc-pull] into [master][cc-base-branch] will increase coverage by 22.10%

@@             master        #64   diff @@
==========================================
  Files             2          2
  Lines            95         95
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits             49         70    +21
+ Misses           46         25    -21
  Partials          0          0

Powered by Codecov. Last updated by [504136d...94f1ff8][cc-compare]
[cc-base-branch]: https://codecov.io/gh/johnbellone/vault-cookbook/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/johnbellone/vault-cookbook/compare/504136de3d0ee1ec08da1e983499a703236fbf30...94f1ff890aec3d40685929e38e91d7c38ee546bf
[cc-pull]: https://codecov.io/gh/johnbellone/vault-cookbook/pull/64?src=pr

I have switched to the runtimedirectory option which is the preferred method from what i have read.

Perhaps that directory should be set to <%= @directory %>?

Ah, it helps to read the documentation!


Error when using merge-conflicts on files with Windows UNC path

I'm running into an issue while trying to use merge-conflicts and I'm hoping someone might be able to advise a solution or work-around. I'm running Atom on a Windows 10 machine and editing files over a Windows mount, using DOS/Windows UNC path, such as:
\\servername\path\to\file.txt
Everything I've done up until now outside of merge-conflicts works great in the editor (git status coloring in file tree, loading/saving, syntax highlighting, linters) but when I tried today to use to run merge-conflicts:detect for the first time, I get the following error message:
abnormal git exit: 128 '\servername\path\to' CMD.EXE was started
with the above path as the current directory. UNC paths are not supported.

I noticed the path reported in the error message is missing one of the leading backslashes, so perhaps backslashes need to be escaped before attempting to invoke the shell, at least on Windows machines. However, the error message seems to indicate something appeared to detect it was a UNC path, so maybe escaping isn't the issue.
Other plugins, such as linter, that execute external programs to parse files over the mount, as well as whatever Atom does with the file tree to colorize files and folders according to their status in the git repository works wine over the mount, as well.
Does anyone have any ideas, other than mapping the Windows UNC path to a drive letter, for getting merge-conflict (or its dependencies) to work with a Windows UNC path?
Thank you for your time.

I was able to work around this issue by adding this registry entry:
[HKEY_CURRENT_USER\SOFTWARE\Microsoft\Command Processor]
"DisableUNCCheck"=dword:00000001
Marking this closed.


test-sp is also reachable as /simplesaml

The simplesamlphp installation on the test-sp is also reachable under /simplesaml/.  However, then is is misconfigured (no IdPs are visible).

There was a stale (package) installation of simplesaml that enable global alias for /simplesaml. Fixed by removing package.


Add Kruskal's Algorithm in the following language

The following Algorithm needs to be implemented in the following languages.

[ ] C
[ ] C++
[ ] Python
[ ] Java

Ground rules:

Whoever claims the Issue first, should send a Pull Request within 24 hours else it'll be assigned to another claimant and further Pull Requests regarding that issue will not be entertained.
Pull Requests sent without claiming and referencing the Issue will not be entertained.
Make sure your Pull Requests have just one commit in them. Squash multiple commits into one if you have to.
Make sure you put in the proper folder. If you do not know where to push the code for the Algorithm, feel free to ask in the comments section below. :smile:

I would like to work on the C++ version.

I would like to work on C version.

I have started working on the JAVA version.

I would like to work on the Python version.

I would like to work on python version.


First draft of separate staging environment for Mac infra

What is the problem that this PR is trying to fix?
We would like a separate staging environment for testing on the Mac infrastructure. This is the first draft of the first iteration towards that goal.
What approach did you choose and why?
I've created a new graph for macstadium-staging and removed its content from macstadium-pod-1. and macstadium-pod-2. I have renamed macstadium-pod-1 and macstadium-pod-2 to macstadium-prod-1 and macstadium-prod-2 to indicate that they represent the production environments versus the staging environment. I've renamed version variable names like jupiter_brain_staging_version which reference staging since that will no longer be necessary, and I've set wjb to install the newest Ubuntu for staging.
How can you test this?
I'm interested in starting with a visual review of the changes. It's my expectation that we'll discuss the changes and then revise rather than approve and merge.
What feedback would you like, if any?
Anything – I figure there is some content here to discuss. Thanks Matt!

Let me know if this looks ready to go, thanks!

This looks good to me. I'd like to review the Terraform plan, since this will mean taking things out of production.

Hey friends! I'm taking over this change, and I've made some commits that I think get this ready to go. When deployed, this will bring remove workers and more that are for staging from the prod machines, and instead bring them up on the staging WJB machine. Would love another set of eyes on it.
The staging environment will use the same version of Terraform as other infras now. For the time being, macstadium production still uses v0.9.

you can take out the threatstack keys for the staging module
https://github.com/travis-pro/team-blue/issues/836


Feature/change UUID generator

Migrated away from satori/go.uuid to gofrs/uuid due to a critical bug found in the former package.
Refactored the code a bit to not depend on our uuid wrapper package.


[Compute]: Warnings during the compilation

neon_utils.c:15:1: warning: no previous prototype for 'HexDecodeChar' [-Wmissing-prototypes]
   15 | HexDecodeChar(char c)
      | ^~~~~~~~~~~~~
neon_utils.c:33:1: warning: no previous prototype for 'HexDecodeString' [-Wmissing-prototypes]
   33 | HexDecodeString(uint8 *result, char *input, int nbytes)
      | ^~~~~~~~~~~~~~~
neon_utils.c:55:1: warning: no previous prototype for 'pq_getmsgint32_le' [-Wmissing-prototypes]
   55 | pq_getmsgint32_le(StringInfo msg)
      | ^~~~~~~~~~~~~~~~~
neon_utils.c:69:1: warning: no previous prototype for 'pq_getmsgint64_le' [-Wmissing-prototypes]
   69 | pq_getmsgint64_le(StringInfo msg)
      | ^~~~~~~~~~~~~~~~~
neon_utils.c:80:1: warning: no previous prototype for 'pq_sendint32_le' [-Wmissing-prototypes]
   80 | pq_sendint32_le(StringInfo buf, uint32 i)
      | ^~~~~~~~~~~~~~~
neon_utils.c:89:1: warning: no previous prototype for 'pq_sendint64_le' [-Wmissing-prototypes]
   89 | pq_sendint64_le(StringInfo buf, uint64 i)
      | ^~~~~~~~~~~~~~~
neon_utils.c:100:1: warning: no previous prototype for 'disable_core_dump' [-Wmissing-prototypes]
  100 | disable_core_dump()
      | ^~~~~~~~~~~~~~~~~
extension_server.c: In function 'neon_download_extension_file_http':
extension_server.c:37:11: warning: unused variable 'postdata' [-Wunused-variable]
   37 |  char    *postdata;
      |           ^~~~~~~~
extension_server.c: At top level:
extension_server.c:78:1: warning: no previous prototype for 'pg_init_extension_server' [-Wmissing-prototypes]
   78 | pg_init_extension_server()
      | ^~~~~~~~~~~~~~~~~~~~~~~~
control_plane_connector.c: In function 'ConstructDeltaMessage':
control_plane_connector.c:123:3: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
  123 |   HASH_SEQ_STATUS status;
      |   ^~~~~~~~~~~~~~~
control_plane_connector.c:155:3: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
  155 |   HASH_SEQ_STATUS status;
      |   ^~~~~~~~~~~~~~~
control_plane_connector.c:172:5: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
  172 |     char    *encrypted_password = get_role_password(entry->name, &logdetail);
      |     ^~~~
control_plane_connector.c:191:2: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
  191 |  JsonbValue *result = pushJsonbValue(&state, WJB_END_OBJECT, NULL);
      |  ^~~~~~~~~~
control_plane_connector.c: In function 'SendDeltasToControlPlane':
control_plane_connector.c:239:2: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
  239 |  char    *message = ConstructDeltaMessage();
      |  ^~~~
walproposer_pg.c:391:1: warning: no previous prototype for 'replication_feedback_set' [-Wmissing-prototypes]
  391 | replication_feedback_set(PageserverFeedback *rf)
      | ^~~~~~~~~~~~~~~~~~~~~~~~
walproposer.c: In function 'AdvancePollState':
walproposer.c:477:15: warning: unused variable 'wp' [-Wunused-variable]
  477 |  WalProposer *wp = sk->wp;
      |               ^~
walproposer_pg.c: In function 'StartProposerReplication':
walproposer_pg.c:975:13: warning: variable 'currTLI' set but not used [-Wunused-but-set-variable]
  975 |  TimeLineID currTLI;
      |             ^~~~~~~
walproposer.c: At top level:
walproposer.c:1490:1: warning: no previous prototype for 'ParsePageserverFeedbackMessage' [-Wmissing-prototypes]
 1490 | ParsePageserverFeedbackMessage(WalProposer *wp, StringInfo reply_message, PageserverFeedback *rf)
      | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
control_plane_connector.c:251:2: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
  251 |  const int num_retries = 5;
      |  ^~~~~
control_plane_connector.c: In function 'MergeTable':
control_plane_connector.c:350:3: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
  350 |   DbEntry    *entry;
      |   ^~~~~~~
control_plane_connector.c:395:3: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
  395 |   RoleEntry  *entry;
      |   ^~~~~~~~~
control_plane_connector.c: In function 'HandleCreateDb':
control_plane_connector.c:489:2: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
  489 |  DefElem    *downer = NULL;
      |  ^~~~~~~
control_plane_connector.c:499:2: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
  499 |  bool  found = false;
      |  ^~~~
control_plane_connector.c: In function 'HandleAlterOwner':
control_plane_connector.c:530:2: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
  530 |  const char *name = strVal(stmt->object);
      |  ^~~~~
control_plane_connector.c:540:2: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
  540 |  const char *new_owner = get_rolespec_name(stmt->newowner);
      |  ^~~~~
control_plane_connector.c: In function 'HandleDbRename':
control_plane_connector.c:553:2: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
  553 |  bool  found = false;
      |  ^~~~
control_plane_connector.c: In function 'HandleDropDb':
control_plane_connector.c:590:2: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
  590 |  bool  found = false;
      |  ^~~~
control_plane_connector.c: In function 'HandleCreateRole':
control_plane_connector.c:607:2: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
  607 |  bool  found = false;
      |  ^~~~
control_plane_connector.c: In function 'HandleAlterRole':
control_plane_connector.c:636:2: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
  636 |  DefElem    *dpass = NULL;
      |  ^~~~~~~
control_plane_connector.c:653:2: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
  653 |  bool  found = false;
      |  ^~~~
control_plane_connector.c: In function 'HandleRoleRename':
control_plane_connector.c:[674](https://github.com/neondatabase/neon/actions/runs/7261297170/job/19782211123?pr=6184#step:5:675):2: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
  674 |  bool  found = false;
      |  ^~~~
control_plane_connector.c: In function 'HandleDropRole':
control_plane_connector.c:[712](https://github.com/neondatabase/neon/actions/runs/7261297170/job/19782211123?pr=6184#step:5:713):2: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
  712 |  ListCell   *item;
      |  ^~~~~~~~
control_plane_connector.c: At top level:
control_plane_connector.c:807:1: warning: no previous prototype for 'InitControlPlaneConnector' [-Wmissing-prototypes]
  807 | InitControlPlaneConnector()
      | ^~~~~~~~~~~~~~~~~~~~~~~~~
control_plane_connector.c: In function 'InitControlPlaneConnector':
control_plane_connector.c:838:2: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
  838 |  const char *jwt_token = getenv("NEON_CONTROL_PLANE_TOKEN");
      |  ^~~~~
file_cache.c: In function 'neon_get_lfc_stats':
file_cache.c:772:13: warning: 'value' may be used uninitialized in this function [-Wmaybe-uninitialized]
  772 |   values[1] = Int64GetDatum(value);
In file included from /usr/local/pgsql/include/server/postgres.h:47,
                 from hnsw.c:1:
hnsw.c: In function 'hnsw_check_available_memory':
hnsw.c:152:15: warning: format '%n' expects a matching 'int *' argument [-Wformat=]
  152 |   elog(ERROR, "Failed to get amount of RAM: %n");
      |               ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/usr/local/pgsql/include/server/utils/elog.h:141:4: note: in definition of macro 'ereport_domain'
  141 |    __VA_ARGS__, errfinish(__FILE__, __LINE__, PG_FUNCNAME_MACRO); \
      |    ^~~~~~~~~~~
/usr/local/pgsql/include/server/utils/elog.h:233:2: note: in expansion of macro 'ereport'
  233 |  ereport(elevel, errmsg_internal(__VA_ARGS__))
      |  ^~~~~~~
hnsw.c:152:3: note: in expansion of macro 'elog'
  152 |   elog(ERROR, "Failed to get amount of RAM: %n");
      |   ^~~~
hnsw.c:152:46: note: format string is defined here
  152 |   elog(ERROR, "Failed to get amount of RAM: %n");
      |                                             ~^
      |                                              |
      |                                              int *

It seems that these warnings cannot be reproduced on the dev server. So I guess it's specific to a compiler version when building our compute node image.

And we are using gcc 10 https://packages.debian.org/bullseye/gcc in our build image

I copied these warnings from CI

Yep, it's specific to gcc 10 and some combination of compile flags. I'm extracting those info from CI and fixing them.
Also -Wall generates way more warnings than CI. In the future we should enable that, but for now I'll just fix the ones reported in CI.

Fixed -Wmissing-prototypes first and looking into other types of warnings + treating warnings as errors in CI in the future

Newer versions of Ubuntu (GitHub Actions, local Docker on Mac) don't produce this.

decided not to fix -Wdeclaration-after-statement as it is unlikely to cause any runtime errors and fixing it makes the code harder to read (in my opinion)


Potentially misleading log from client or owner exit in ownership pool

This is a pretty classic 3-way exit signal race condition where we technically log the wrong reason:

X does ownership checkout
X link/monitors Y
Y gets allowed on connection owned by X through allow or shared mode
Y does connection checkout
Y exits and sends exit/monitor signal to X
X exits due to signal from Y
X sends monitor signal to Ownership.Owner process
Ownership.Owner receives :DOWN from X, logs owner exited (but it was client that triggered exit) and shuts down
Y sends monitor signal to Ownership.Owner process

OR

X does ownership checkout
Y links to X
Y gets allowed on connection owned by X through allow or shared mode
Y does connection checkout
X exits and sends exit signal to Y
Y exits due exit signal from X
Y sends monitor signal to Ownership.Owner process
Ownership.Owner receives :DOWN from Y, logs client exited (but it was owner that triggered exit) and shuts down
X sends monitor signal to Ownership.Owner process

Fortunately we disconnect in either case so the pool is secure. However it might be that we can give better information or at least try to prevent a user from being mislead that a different process caused the crash.
When we have 2 monitors active (one on owner and one on client) and we receive a :DOWN we could use Process.alive? to see if the other process is still alive and use this in the message. If the other process is not alive we can block to receive their :DOWN and log both. If the process is alive it might be helpful to add that information to (e.g. "(with client #PID<..>)"). We may also want to special case the situation where the owner is the client.

@josevalim do you think this is worth looking into otherwise will close?

I vote for close, yes. :)


Genetic map file for T2T genome (CHM13)

Hi,
Thanks for this wonderful tool! I was trying to use Shapeit5 for phasing a set of samples processed based on the new released T2T human genome (CHM13). Could you please help pointing the right genetic mapping files? I can only find b37 and b38 genomes in the folder.
Thanks very much for your help!
Hang

Hi,
We do not have T2T maps available in this repo yet, but you can liftover the provided maps to the T2T reference.
I have also seen a publicly available github repo made by Joseph Lalli providing lifted over maps. Just, before using them, please make sure with Joseph you are allowed to.
Simone

Thanks for the information! This is very helpful! will do!

Hi,
We do not have T2T maps available in this repo yet, but you can liftover the provided maps to the T2T reference. I have also seen a publicly available github repo (https://github.com/JosephLalli/phasing_T2T/tree/master) maintained by Joseph Lalli providing lifted over maps. Just, before using them, please make sure with Joseph you are allowed to.
Simone


[Packaging Request] extension-manager

Project description
A native tool for browsing, installing, and managing GNOME Shell Extensions.
Metadata

homepage/source URL: https://github.com/mjakeman/extension-manager
license: GPL-3.0
platforms: linux

PR was merged, closing.


[Bug]: parse SID failed in list columns interface when use alias in sql

ODC version
odc 41x and 42x
OB version
all
What happened?

What did you expect to happen?
The list columns interface returns 200 status code
How can we reproduce it (as minimally and precisely as possible)?
as show in what happend
Anything else we need to know?
No response
Cloud
No response

pass


feat(NODE-3750): make maxConnecting configurable

Description
NODE-3750, changes implement the following spec commit: ce242d
What is changing?

Added maxConnecting as a pool option that is externally configurable as a client option

per the spec, maxConnecting <= 0 should warn, so, in line with our warning = error convention, maxConnecting <=0 will error (note that there are two different errors: one from uint validation, and one to handle 0 specifically)

Synced new cmap test: pool-checkout-custom-maxConnecting-is-enforced
Updated uri options connection-pool-options test file with new tests and implemented maxConnection option parsing in the test runner
Updated existing node driver "all options" tests to include maxConnecting - there isn't any complicated option parsing happening, so it didn't seem to require an additional stand-alone test beyond what is covered by the URI spec tests themselves

Is there new documentation needed for these changes?
Yes, in addition to the API docs, we should also update the connection docs page for the node driver since it lists other pool options there.
What is the motivation for this change?
Spec
Double check the following

[ ] Ran npm run check:lint script
[ ] Self-review completed using the steps outlined here
[ ] PR title follows the correct format: <type>(NODE-xxxx)<!>: <description>
[ ] Changes are covered by tests
[N/A] New TODOs have a related JIRA ticket

@durran good catch, I didn't even realize I missed them


chore(main): release never-bot 2.1.0

:robot: I have created a release beep boop
2.1.0 (2024-04-10)
Features

streamline deployment workflow (04c10e0)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/neverased/NeverBot/releases/tag/never-bot-v2.1.0 :sunflower:


reinitialize / redraw on resize ?

hi,
what is the best way to redraw the wavefrom once the browser size change ?
Is there a smarter way than calling WaveSurfer.create ?
~
Marc

@daslicht  RTFM!

wavesurfer.drawer.fireEvent('redraw') it works for me


Print warnings on conflicts in values and secret values

There may be multiple values-files passed to the werf. If next values file redefines some keys from previous, then it is OK.
There may be multiple secret-values-files passed to the werf. If next secret-values file redefines some keys from previous, then it is OK.
If something is redefined using --set, then it is OK.

Need to merge all values into the single values-map and merge all secret-values into the single secret-values-map and merge all sets into set-map. Print a WARNING if:

secret-values-map redefines something from values-map;
set-map redefines something from secret-values-map.

Merging values from different places is pretty complex right now. We did a bit of refactoring in werf v2. Now we only need to ensure that the priority order is consistent with what we describe in our documentation (and it looks like is). Warnings are unnecessary since nothing wrong is happening here (unless we messed up something in values priorities, which will be a bug).


AADSecops Detection and hunting Queries

Fixes #
Proposed Changes
Detections and Hunting Queries for AAD Secops Guide

Looks like you have 2 templates with ID 34c5aff9-a8c2-4601-9654-c7e46342d03b (one detection and one hunting). It shouldn't be an issue technically but it might be best to change one to make identification easier.

Also looks like you'll need to exclude 34c5aff9-a8c2-4601-9654-c7e46342d03b (detection) from KQL validation as its not seeing the correct schema. I have confirmed the query is correct in Sentinel data.

Looks like you have 2 templates with ID 34c5aff9-a8c2-4601-9654-c7e46342d03b (one detection and one hunting). It shouldn't be an issue technically but it might be best to change one to make identification easier.

Fixed duplicate template Ids.

Also looks like you'll need to exclude 34c5aff9-a8c2-4601-9654-c7e46342d03b (detection) from KQL validation as its not seeing the correct schema. I have confirmed the query is correct in Sentinel data.

Removed column name for now as its not necessary in most cases


[BUG] Snapshots REST API returns certain segments of Disk Access ID in uppercase

API Spec link
https://github.com/Azure/azure-rest-api-specs/blob/main/specification/storagesync/resource-manager/Microsoft.StorageSync/stable/2020-03-01/storagesync.json
API Spec version
2020-03-01
Describe the bug
I'm working on integrating disk_access_id into the Azure Terraform provider for Snapshots. I'm encountering an issue where Terraform is throwing a diff error for the disk access ID based on what's retrieved from the REST API.
Expected behavior
Expected behavior would be to have the Disk Access ID returned formatted as it was set during creation.
Actual behavior
Actual behavior is capitalization of certain segments of the Disk Access ID.
Reproduction Steps
// Create snapshot resource
PUT https://management.azure.com/subscriptions/***/resourceGroups/example-resources/providers/Microsoft.Compute/snapshots/acctestss?api-version=2022-03-02
{
  "name": "acctestss",
  "id": "/subscriptions/***/resourceGroups/example-resources/providers/Microsoft.Compute/snapshots/acctestss",
  "type": "Microsoft.Compute/snapshots",
  "location": "eastus",
  "tags": {},
  "properties": {
    "creationData": {
      "createOption": "Copy",
      "sourceUri": "/subscriptions/***/resourceGroups/example-resources/providers/Microsoft.Compute/disks/acctestmd"
    },
    "diskSizeGB": 20,
    "encryptionSettingsCollection": {
      "enabled": false
    },
    "incremental": false,
    "diskAccessId": "/subscriptions/***/resourceGroups/example-resources/providers/Microsoft.Compute/diskAccesses/test-da",
    "networkAccessPolicy": "AllowPrivate",
    "publicNetworkAccess": "Enabled",
    "provisioningState": "Updating"
  }
}

// Read snapshot resource
GET https://management.azure.com/subscriptions/***/resourceGroups/example-resources/providers/Microsoft.Compute/snapshots/acctestss?api-version=2022-03-02

{
  "name": "acctestss",
  "id": "/subscriptions/***/resourceGroups/example-resources/providers/Microsoft.Compute/snapshots/acctestss",
  "type": "Microsoft.Compute/snapshots",
  "location": "eastus",
  "tags": {},
  "sku": {
    "name": "Standard_LRS",
    "tier": "Standard"
  },
  "properties": {
    "creationData": {
      "createOption": "Copy",
      "sourceResourceId": "/subscriptions/***/resourceGroups/example-resources/providers/Microsoft.Compute/disks/acctestmd",
      "sourceUniqueId": "***"
    },
    "diskSizeGB": 20,
    "encryptionSettingsCollection": {
      "enabled": false,
      "encryptionSettings": [],
      "encryptionSettingsVersion": "1.0"
    },
    "encryption": {
      "type": "EncryptionAtRestWithPlatformKey"
    },
    "incremental": false,
    "diskAccessId": "/subscriptions/***/resourceGroups/EXAMPLE-RESOURCES/providers/Microsoft.Compute/diskAccesses/test-da",
    "networkAccessPolicy": "AllowPrivate",
    "publicNetworkAccess": "Enabled",
    "timeCreated": "2024-05-21T15:53:26.5564391+00:00",
    "provisioningState": "Succeeded",
    "diskState": "Unattached",
    "diskSizeBytes":<PHONE_NUMBER>0,
    "uniqueId": "***"
  }
}

Environment
Terraform local provider development MacOSX

@anpint Please help have a look, thank you.


Fix nodesDir scan when node package has js/html in sub dir to package.json

fixes #3861
Types of changes

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)

First, please read through the issue detailed in this comment
Proposed changes

Normalise path before splitting (loader.js)
Skip calling statSync on empty string  (loader.js)
Shortcut scanning of a directories files/folders if the directory contains a package.json (localfilesystem.js)
Prevent crashing node-red when set.types is not set  (registry.js)
Add more tests that load nodes with js/html files in a sub directory (registry.js)

Checklist

[x] I have read the contribution guidelines
[ ] For non-bugfix PRs, I have discussed this change on the forum/slack team.
[x] I have run grunt to verify the unit tests pass
[x] I have added suitable unit tests to cover the new/changed functionality

Coverage decreased (-0.03%) to 68.311% when pulling 6a19d8246c533f825e73876d9d7b2176322ebffa on Steve-Mcl:improve-nodesdir-scan into a5d7f7acceb4c77d1e91feca63a3619164d8b06b on node-red:master.


build-fhs-userenv: fixes for mount points

Motivation for this change
Fixes #110636.
/etc/profiles was not being mounted, which meant that any user-installed package would not be available under the FHS environment. This change alone fixes all the issues reported in #110636 since the AppImage environment can now find the browser executable and the correct cursor.
/etc/fonts was also broken since we were mounting the host's /etc/fonts but it was getting overriden by the default /etc/fonts from fontconfig that was part of the FHS environment.
I only tested these changes on AppImages and it solved all the issues I reported in #110636.
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

/marvin opt-in
/status needs_reviewer

/marvin opt-in
/status needs_reviewer

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

2 packages failed to build:

houdini
quartus-prime-lite

74 packages built:

Sylk
android-studio (androidStudioPackages.stable)
androidStudioPackages.beta
androidStudioPackages.canary
androidStudioPackages.dev
anki-bin
appimage-run
appimage-run-tests
apple-music-electron
arduino-ci
arduino-cli
bitscope.chart
bitscope.console
bitscope.display
bitscope.dso
bitscope.logic
bitscope.meter
bitscope.proto
bitscope.server
chrysalis
clonehero
conda
deltachat-electron
devdocs-desktop
dropbox
dropbox-cli
electronplayer
esphome
fahclient
flutter (flutterPackages.stable)
flutter-beta (flutterPackages.beta)
flutter-dev (flutterPackages.dev)
hover
irccloud
joplin-desktop
keeweb
kodiPlugins.steam-launcher
ledger-live-desktop
left4gore-bin
lens
lightworks
lunar-client
lutris
lutris-free
marktext
mate.caja-dropbox
minetime
molotov
mycrypto
notable
nuclear
p3x-onenote
platformio
plex
plexamp
protontricks
radicle-upstream
rambox-pro
runwayml
sidequest
ssb-patchwork
standardnotes
station
steam
steam-run
steam-run-native
steamcmd
timeular
tusk
unityhub
vmware-horizon-client
wootility
zettlr
zulip

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

2 packages failed to build:

houdini
quartus-prime-lite

74 packages built:

Sylk
android-studio (androidStudioPackages.stable)
androidStudioPackages.beta
androidStudioPackages.canary
androidStudioPackages.dev
anki-bin
appimage-run
appimage-run-tests
apple-music-electron
arduino-ci
arduino-cli
bitscope.chart
bitscope.console
bitscope.display
bitscope.dso
bitscope.logic
bitscope.meter
bitscope.proto
bitscope.server
chrysalis
clonehero
conda
deltachat-electron
devdocs-desktop
dropbox
dropbox-cli
electronplayer
esphome
fahclient
flutter (flutterPackages.stable)
flutter-beta (flutterPackages.beta)
flutter-dev (flutterPackages.dev)
hover
irccloud
joplin-desktop
keeweb
kodiPlugins.steam-launcher
ledger-live-desktop
left4gore-bin
lens
lightworks
lunar-client
lutris
lutris-free
marktext
mate.caja-dropbox
minetime
molotov
mycrypto
notable
nuclear
p3x-onenote
platformio
plex
plexamp
protontricks
radicle-upstream
rambox-pro
runwayml
sidequest
ssb-patchwork
standardnotes
station
steam
steam-run
steam-run-native
steamcmd
timeular
tusk
unityhub
vmware-horizon-client
wootility
zettlr
zulip

Links in rambox-pro still doesn't work for me, but this may be related to #108995 since I use release-20.09.
My only issue with this PR is that it affects everything that uses buildFHSUserEnv, so the implications are probably bigger than I have as context.
Maybe @Atemu or @abbradar can help reviewing here :thinking: ?

Links in rambox-pro still doesn't work for me, but this may be related to #108995 since I use release-20.09.
My only issue with this PR is that it affects everything that uses buildFHSUserEnv, so the implications are probably bigger than I have as context.
Maybe @Atemu or @abbradar can help reviewing here :thinking: ?

I am only using two appimage apps: rambox and ledger-live-desktop. I had issues on both which are fixed by this PR. I think besides appimages the other big user of FHS is Steam, which I don't use and therefore haven't tested. Probably makes sense to test manually before merging this PR.

I am only using two appimage apps: rambox and ledger-live-desktop. I had issues on both which are fixed by this PR. I think besides appimages the other big user of FHS is Steam, which I don't use and therefore haven't tested. Probably makes sense to test manually before merging this PR.

Links in rambox-pro still doesn't work for me, but this may be related to #108995 since I use release-20.09.

Maybe you have some other issue related to XDG then. For me the issue was that my browser was not on the FHS's path since it is symlinked in /etc/profiles/per-user/{username}.

Links in rambox-pro still doesn't work for me, but this may be related to #108995 since I use release-20.09.

Maybe you have some other issue related to XDG then. For me the issue was that my browser was not on the FHS's path since it is symlinked in /etc/profiles/per-user/{username}.

Happy to report that I have installed Steam and played some games, everything seemed to be working fine.

Happy to report that I have installed Steam and played some games, everything seemed to be working fine.


Use the _get_comm_info method to reconstruct all widget models

Seems to fix a lot of bugs with persistence in general.
@SylvainCorlay woohoo!
closes #133

@jdfreder woohoo. I will look at it tonight.

@jdfreder the comm_info_[request/reply] shell message allows you to only request comms corresponding to a given target name. (You can see this in #75.)
It would make sense to make the workaround that we have here do the same thing so that it will be easier to replace when the notebook 4.1 is released like here: https://github.com/SylvainCorlay/ipython_widgets/commit/78984210255f71f1e01c01f331fb978a75feebb6

@SylvainCorlay comments addressed

:+1: :tada: :balloon:
I am all good with this. I can merge later this afternoon, just to let the time to other people to comment. Any input @jasongrout?

hop?


Root command completion

I wanted to start discussion about this topic.
When someone would like to have very simple command without subcommands.
Let's say this:
package main

import (
	"fmt"

	"github.com/jxskiss/mcli"
)

type Config struct {
	field string
}

func cmdTestArguments(ctx mcli.ArgCompletionContext) []mcli.CompletionItem {
	return []mcli.CompletionItem{
		{Value: "cmd one", Description: "description one"},
		{Value: "cmd two", Description: "description two"},
	}
}

func cmdRoot(ctx *mcli.Context) {
	var args struct {
		Names bool   `cli:"-n, --names, Display certificate common names only."`
		Arg   string `cli:"arg, Dummy demo arg"`
		V     string `cli:"value"`
	}
	funcs := make(map[string]mcli.ArgCompletionFunc)
	funcs["arg"] = cmdTestArguments
	funcs["value"] = cmdTestArguments
	ctx.Parse(&args, mcli.DisableGlobalFlags(), mcli.WithArgCompFuncs(funcs))
	fmt.Println("Doing something")
}

func main() {
	app := &mcli.App{Description: `Very Simple App`}
	app.AddRoot(cmdRoot)

	app.Options.EnableFlagCompletionForAllCommands = true
	app.AddCompletion()
	app.AddHelp()
	app.Run()
}

What do you think about strategy for this?
I have started draft PR, it's still in progress.

You may try the code in branch tmp_root_cmd_completion, it should work as a base point.
But the root command support is not well designed and is buggy, e.g. for the program you provided, program help does not print help and exit, instead it runs the root command.
Originally, I thought comlicated needs should use sub-commands, root command was thoughout to be a simple default action when user does not provided any sub-command.

Ok, I am writing to share some thoughts.
I have some mvp. But it needs some tweaks around flag suggestions on root command now.
I think checkLastArgForCompletion is way too complex. It needs refactor to reduce complexity.
When starting writing on root, I have planned procedure:

suggest subcommands
0 subcommand suggestions, try to suggest completion for root function
when not both above not doable because of condition, suggest subcommands/flags

When command with subcommands is not matched, this should handle more than one completion for root.
I still need to write more tests.
About help. Maybe in that case I would need to provide global -h flag, which would trigger ctx.PrintHelp() without adding a new command.
Then state it as prerequisite in order to use completion functions on this level.
What do you think about it?

I am stuck on one thing. Flags are not get evaluated in some specific conditions.
When having root command, flags values does not change.
This minimal example can reproduce the problem:
package main

import (
	"fmt"
	"os"

	"github.com/jxskiss/mcli"
)

type GlobalFlags struct {
	AcceptTOS bool `cli:"-a, --accept-tos, By setting this flag to true you indicate that you accept the current Let's Encrypt terms of service."`
}

var globalFlags GlobalFlags

func Execute() {
	app := mcli.App{}
	app.SetGlobalFlags(&globalFlags)
	app.AddRoot(Invoke)

	app.AddCompletion()
	app.Options.EnableFlagCompletionForAllCommands = true
	app.Run()
}

func main() {
	Execute()
}

func Invoke(_ *mcli.Context) {
	fmt.Println(os.Args)
	var args struct {
		Commands []string `cli:"command"`
		DryRun   bool     `cli:"-r, --dry-run, Dry run / don't execute"`
	}
	funcs := make(map[string]mcli.ArgCompletionFunc)
	funcs["command"] = CompleteComUnits
	mcli.Parse(&args, mcli.WithArgCompFuncs(funcs))

	fmt.Println(args)
	fmt.Println(globalFlags)
}

func CompleteComUnits(ctx mcli.ArgCompletionContext) []mcli.CompletionItem {
	return []mcli.CompletionItem{}
}

Then in command line:

This problem is visible even without my changes from branch.

Ok. Found it. It was caused by flag package.
When scanning arguments list it was failing when first encountered element was non flag.
Now I need to write down more tests.

Ok. I saw merge, so I am closing issue.


Change type of Queue#connection and Queue#nowPlaying to include undefined

Queue#connection can be undefined when the queue hasn't joined a channel yet. I was quite surprised when my program crashed due to this connection property being undefined, despite the types saying it can't be undefined. This PR removes the "strictPropertyInitialization": false in tsconfig so that these incorrect types hopefully don't happen again. (I'm not quite sure why this was set to false in the first place, so I'm happy to revert it if needed.)
This PR also changes the type of Queue#nowPlaying to Song | undefined (instead of Song) because it can be undefined when the queue isn't playing anything.

On an unrelated note, would you consider enabling some ESLint to enforce consistent formatting? Some indentation in this project not properly aligned (not a multiple of 4) and the formatting of if statements is inconsistent if (foo) vs if(foo).

Thanks for the PR. Sorry for that I didn't notice that it can be undefined. Good catch.

I will be pushing it later today.

On an unrelated note, would you consider enabling some ESLint to enforce consistent formatting? Some indentation in this project not properly aligned (not a multiple of 4) and the formatting of if statements is inconsistent if (foo) vs if(foo).

maybe we can add a .prettier file too

Thank you for this pr, pushing to npm in few minutes.

Hey, the update is live! Thanks once again.
https://www.npmjs.com/package/discord-music-player/v/8.3.1


Evaluate Feasibility of creating Eligibility Policy outside of the TEAM solution

Describe the bug
@tawoyinfa We are looking for a way to avoid doing TEAM activity via TEAM Console. Such as creating eligibility and approver policy. It would be really great if you can provide some input on this.
We are exploring the feasibility of developing an eligibility or approver policy outside of the team console, perhaps through APIs or another programmatic method? Currently, we manage everything manually through the team console—creating eligibility policies, approver policies, approving the request, revoking approvals, and so on. To minimize management overhead, we need programmatic or Api's way of executing the above activities.
Expected behavior
We can execute most of the TEAM activity using API's or programmatically, to avoid management overhead.
Desktop (please complete the following information):

OS: Windows 11
Browser Edge
Version 127.0.2651.74 (Official build) (64-bit)

Additional context
We are looking for a way to avoid doing TEAM activity via TEAM Console. It would be really good if we can do this TEAM activities such as creating eligibility and approver policy using api's, or programmatically.

Perhaps https://aws-samples.github.io/iam-identity-center-team/docs/deployment/configuration/cognito_machine_auth.html#retrieve-machine-authentication-credentials and https://aws-samples.github.io/iam-identity-center-team/docs/deployment/configuration/cognito_machine_auth.html#using-the-terraform-provider would be helpful.


Fix block sync problem

related #1587 .
Fix the problem : 1.Split the network and can't merge them, and the node can't set LIB.

Codecov Report

Merging #1620 into dev will decrease coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##              dev    #1620      +/-   ##
==========================================
- Coverage   95.62%   95.62%   -0.01%
==========================================
  Files         398      398
  Lines       18619    18618       -1
==========================================
- Hits        17804    17803       -1
  Misses        815      815

Impacted Files
Coverage Δ

src/AElf.OS/Jobs/BlockSyncJob.cs
100% <100%> (ø)
:arrow_up:


Updating Pipeline in the middle of Progressive Update could result in Pipeline deemed "Running" prior to Pipeline reconciled

Describe the bug
The way the Progressive process works, the "updating" Pipeline would get replaced if a new update occurs in the middle of the process (i.e. there are still just 2 but the new one is now even newer). This would be rare in the current state, but we could declare the new Pipeline to be "Running" prior to Numaflow Controller having reconciled the very latest update.
In the future, we will have metrics to determine the true health of our Pipeline before taking the old one down, meaning this race condition could become more likely if it takes longer to evaluate the true health. Moreover, we will want our metrics to only represent the very latest state of our Pipeline and not the previous state.
Therefore, we should reevaluate how we want our process to be, either:

Prohibit new updates to the Pipeline until the old ones have gone through completely and we are down to 1 Pipeline
Remove the old "updating" Pipeline and put the new one in, starting the process over

Message from the maintainers:
Impacted by this bug? Give it a 👍. We often sort issues this way to know what to prioritize.

this is done


[6.0] Add In-process plugin server path

Cherry-pick #1616 into release/6.0
(copied from https://github.com/swiftlang/swift/pull/74740)

Explanation: swift-syntax libraries in compiler has been complied with -enable-library-evolution because the toolchain provides them as a public interface for building plugins (i.e. lib/swift/host/*.swiftmodule). But that was not great for the performance in the compiler. This PR changes that by separating swift-syntax libs for the compiler from the one for the plugins. So the compiler and the plugin now can use different version/ABI of swift-syntax libraries from the compiler. Now swift-syntax for the compiler are compiled without library evolution.
Scope: Macro plugins
Risk: Mid. This changes the mechanism of in-process plugins significantly. Now in-process plugins are communicate with the compiler using the same messaging serialization with other plugins (i.e. swift-plugin-server, and other executable plugins), instead of directly calling the Macro.Type expansion method. Also, to separate the swift-syntax libraries, this introduces another set of shared swift-syntax libraries in the toolchain.
Testing: Passes current test suite.
Issue: rdar://130532628
Reviewer: Hamish Knight (@hamishknight) Alex Hoppen (@ahoppen) Ben Barham (@bnbarham)

https://github.com/apple/swift/pull/74740
https://github.com/apple/swift-syntax/pull/2702
https://github.com/apple/llvm-project/pull/8929
https://github.com/apple/swift-installer-scripts/pull/309
@swift-ci Please test

Decided not to merge


Chrome Store page not working.

The chrome store page shows error.
The requested URL was not found on this server.

It was removed by Google like three months ago. You can download the "extension.zip" and load it manually.

Thanks.


optimize_projections rule breaks some view operation

Describe the bug

commit: ee7ab0b83ff5bf2934519cd6718ccea620edea1e
If comment out optimize_projections rule, then works fine.
To Reproduce
create table users (id int, name varchar);
insert into users values (1, 'Tom');
create view v as select count(id) from users;
select count(1) from v;

Expected behavior
No response
Additional context
No response

I wonder if @mustafasrepo 's PR https://github.com/apache/arrow-datafusion/pull/8960 could possible fix this issue

I wonder if @mustafasrepo 's PR #8960 could possible fix this issue

No.

I would like to try this one

It seems there is already a fix here https://github.com/apache/arrow-datafusion/pull/8992

It seems there is already a fix here #8992

seems like it is fixed, thanks!


Finite difference pricer adjustments

This branch is still a work in progress
This PR is for addressing the issues raised by @avhz in this comment - the code in question was initially merged from this PR to address  issue 98.
There were some errors in implementation, namely:

Call/Put boundary conditions had implemented $e^{-rT - t}$ instead of $e^{-r(T - t)}$
The time steps were iterating (1..self.time_steps) instead of (0..self.time_steps)
The boundary condition for the Crank-Nicolson method was implemented as self.call_boundary(t + 1, T, delta_t)                               - self.call_boundary(t, T, delta_t) when it should be self.call_boundary(t + 1, T, delta_t)                               + self.call_boundary(t, T, delta_t) (same amendment made for put option boundary

Although the above amendments have been made, there are still issues with convergence in some places. For brevity, I have limited the cases to European call options.
The cases below were run with the following configuration:
strike_price:  10.0;
risk_free_rate:  0.05;
volatility: 0.5;
evaluation_date: Some(date!(2024 - 01 - 01));
expiration_date: date!(2025 - 01 - 01);
time_steps: 10000; // increased 10-fold for stability - see note below
price_steps: 100;
type_flag: TypeFlag::Call;
exercise_flag: ExerciseFlag::European;

with a distinct value for initial_price in each case:
At the money
initial_price: 10.0
Target: 2.179260421286683

Method
Value
Error

Explicit
2.1752124177257035
0.004048003560979563

Implicit
2.1751386801265697
0.004121741160113324

Crank-Nicolson
2.175175548933553
0.004084872353129931

In the money
initial_price: 15.0
Target: 6.064426504411616

Method
Value
Error

Explicit
6.064451145206732
0.00002464079511632633

Implicit
6.06441080128941
0.00001570312220611214

Crank-Nicolson
6.064430972254538
0.000004467842922295517

Out the money
initial_price: 1.0
Target: 1.0140475395001201e-6

Method
Value
Error

Explicit
-1.0119200252625027
1.0119210393100422

Implicit
-1.011847275245909
1.0118482892934484

Crank-Nicolson
-1.011883645221771
1.0118846592693105

There is some level of convergence for the at/in the money cases, although I think there is room for improvement for accuracy. Whilst experimenting with different time step sizes, it seems that there is an issue with consistency i.e. $\Delta t \rightarrow 0 \Rightarrow \text{Error}\rightarrow 0$ - this suggests there is still an issue with implementation - Note that there is convergence, but just not to the target value. The out the money case further confirms that there is something wrong.
Stability issues
It was noticed that explicit() was blowing up in some configurations. It is likely that this indicates an issue with stability, which can be resolved by having a larger time steps, which in turn provides smaller $\Delta t$. Hence, in the config above, I changed the time step from 1000 to 10000 (though 5000 suffices for stability).

Hi, is this still a work in progress?

Hi @avhz - yes this is still a work in progress, I will push something out before the end of this week. I have made some major changes to the code which I will document in this PR.

Just checking, thanks for the update !


Throw better exception in usersecrets when entry assembly is null

cref https://github.com/aspnet/EntityFramework/issues/6918
cc @divega
Also re-syncs Resource.Designer.cs with the resx.

Superceded by #544


[HOPS-463] and [HOPS-506]

[HOPS-463] The transaction title for accessing client cache is set to CONCAT.Update the name to CLIENT_CACHE.
[Hops-506] Added unit tests for STO. Added Hierarchical locking
Make sure there is no duplicate PR for this issue

Please check if the PR fulfills these requirements

[x] Tests for the changes have been added and passed (for bug fixes / features)
[x] HOPS JIRA issue has been opened for this PR
[x] All commits have been squashed down to a single commit
[x] The commit message has the following format: [HOPS-XXX] message

Post a link to the associated JIRA issue
https://hopshadoop.atlassian.net/browse/HOPS-506
https://hopshadoop.atlassian.net/browse/HOPS-463

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
bug fix

What is the new behavior (if this is a feature change)?

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
nope

Other information:

Can one of the admins verify this patch?

Does this fix all the subtree operations problems or only the subtree vs subtree?
Have you done an evaluation of the performance impact?

Please put a link to the pull request dependencies in dal and dal-impl in your description.

Jenkins test this please.

The following tests consistently fail both on Jenkins and on my machine, while they don't fail on the master branch:
TestDFSClientRetries.testNotYetReplicatedErrors
TestAuditLogs.testAuditDenied
TestSubtreeLock.testRetry
TestDFSPermission (timeout)
TestHDFSCLI (timeout)

Should work now. All tests passing and no dependency on DAL-IMPL layer.

Jenkins test this please.

Jenkins test this please.

Jenkins test this please.

Jenkins test this please.

Jenkins test this please

Jenkins test this please.


Add dll_write_to_testcase_ptr callback

This pull request allows custom payload delivery (e.g., via memory as opposed to always via file).

LGTM


Can i use other language dynamic libraries in flutter?

Steps to Reproduce

I wanna  know is there any way provided to use  other language libraries in flutter project.
Like java call C++ libraries by JNI.
Because i wanna call golang library to finish my project.
thanks
Logs

Duplicate of https://github.com/flutter/flutter/issues/7053


test/utils: hide deprecatedBeforeAll inside operator tests

I don't understand why operator.go uses the forbidden function deprecatedBeforeAll. But at least make sure that this is the only user of this deprecated function.
/sig code-quality
NONE

/lgtm

Could we move this to the testsuite setup, if it is absolutely necessary and not possible to do it in a self contained way?

Thanks for the lgtm. I am aware that this is a very small step. As I declared in my commit message, it is clear that there's a lot to be corrected here, but I would leave it to future, better engineers than me.

Don't you need to lower some numbers?

Don't you need to lower some numbers?

I don't understand the question. Do you refer to wc -l tests/utils.go ?

Yes, I do.

Yes, I do.

@0xFelix has a PR touching this, I do not want to conflict.


style: comply with MD022 and fix minor typos

Description

Resolves #60

Comply with MD022
Fix minor typos

Motivation and Context

See #60.
How Has This Been Tested?

I checked with grep.
Screenshots / Output (if appropriate):
None
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] Documentation (no code change)
[ ] Refactor (refactoring production code)
[ ] Other

Checklist:

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

Codecov Report

Merging #61 (90b3ec2) into main (e1da611) will increase coverage by 0.25%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #61      +/-   ##
==========================================
+ Coverage   40.90%   41.15%   +0.25%
==========================================
  Files          13       13
  Lines         824      824
  Branches      215      215
==========================================
+ Hits          337      339       +2
+ Misses        378      374       -4
- Partials      109      111       +2

Impacted Files
Coverage Δ

git-cliff-core/src/repo.rs
29.17% <0.00%> (+2.09%)
:arrow_up:

Continue to review full report at Codecov.

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


feat(settings): Implement missing calltimer setting

What this PR does 📖

Implements the missing call timer setting in audio

Which issue(s) this PR fixes 🔨

Resolves this issue here: https://issues.satellite.im/b/issues/text-is-unreadable-when-highlighted-in-the-save-recovery-phase-screen/

@Flemmli97 @stavares843 @phillsatellite The automated tests for mac and windows are failing now because currently in dev, Call Timer is set as disabled by default, and on this branch, the Call Timer is enabled by default
I can quickly update the tests to reflect this change once this PR is merged, in case that having this setting enabled by default is fine
On Dev:

On this branch:


[WIP] Add WebSocket Support

Fixes #1195
Opening this PR to track progress.

This is superseded by https://github.com/tmpvar/jsdom/pull/2088.


Sample information in trun box should be used to get the start time & duration for WebVTT in MP4

What version of Shaka Player are you using?
v2.0.5 on demo page.

Can you reproduce the issue with our latest release version?

Can you reproduce the issue with the latest code from master?

Are you using the demo app or your own custom app?
The demo app on https://shaka-player-demo.appspot.com/demo/
which says "Shaka Player v2.0.5-debug"

If custom app, can you reproduce the issue using our demo app?

What browser and OS are you using?
Chrome on Linux.
Google Chrome	56.0.2924.87 (Official Build) (64-bit)
Revision	0e9a9a6f3676ae439b78cd9b3f62b4193c3ac7d5-refs/branch-heads/2924@{#895}
OS	Linux

Steps:

Goto https://shaka-player-demo.appspot.com/demo/
Select "Sintel 4k (multicodec, VTT in MP4)"
Play.
Seek to around 1:50.

Expected result:
"It has shed much innocent blood." should show up from 1:51.800 to 1:55.800 and
The line with "You're a fool for traveling alone." should show up from 1:58.000 to 2:01.450.
Actual result:
Both cues show up at 1:50 (before 1:51) and goes till 2:00.
Additional info:
https://storage.googleapis.com/shaka-demo-assets/sintel-mp4-wvtt/s-eng.mp4
tl;dr I think the player is creating a cue per 'mdat' box. For the content above, the 12-th 'mdat' box contains the cues. But the sample timing information is actually in moof > traf > trun box.
Looking at mp4_vtt_parser.js, shaka.media.Mp4VttParser takes start and end timestamps, and all the cues in 'mdat' box are converted to VTTCues using the timestamps. IIUC, these values are coming from the sidx box, hence both cues show up at the same time and for the same duration.
The sidx box contains the subsegment information but not the actual cue timings. Instead info in 'trun' box should be used for the corresponding 'mdat' box.
I am working on a patch to generate MP4 files with WebVTT, and this is the file that I generated
https://storage.googleapis.com/wvtemp/rkuroiwa/vtt_in_mp4_test_delete_me_when_done/sintel-subtitle-eng.mp4
The MPD that uses it is at https://storage.googleapis.com/wvtemp/rkuroiwa/vtt_in_mp4_test_delete_me_when_done/with_subtitle.mpd
This mp4 file is not subsegmented and all the cues are in an mdat box so all of them show up from the beginning of the video.

@rkuroiwa thanks for the detailed report!
@theodab @vaage-google if one of you guys wants some practice with our text parsers and mp4 this might be a good bug for it. (There is a program called iso viewer that translates mp4 files into human-readable form, another tool is http://thumb.co.il/).
Otherwise, I'll look at this next week if @TheModMaker doesn't get to it first :)

@ismena I can look into it.

Hello, I have submitted a merge request which I believe will fix this issue.
https://github.com/google/shaka-player/pull/721

@kvarro, we have a fix in progress, but we are taking a little extra time to refactor the MP4 parser class.  Thanks for the contribution, though!

The fixes related to this will not cherry-pick cleanly to v2.0.x, so this fix will first appear in v2.1.0.


exec: Rely on oci.CompatOCIProcess instead of specs.Process

With recent changes from virtcontainers, we have to rely on the compatible structures we have added so that we don't end up with unmarshalling errors regarding LinuxCapabilities field.

Coverage increased (+0.03%) to 34.971% when pulling 4194d122bede3b4f959805fc8e49ce3b2a0485ca on sboeuf/fix_exec into 4739d0be15bb041a6666ee3cf3ab09c9dfd1f76c on master.

tested fine.
lgtm.


Program running killed without errors

Hi,
I am trying to run SATURN on the integration of human and pig data. I successfully ran the tutorial in the Vignettes/frog_zebrafish_embryogenesis/Train SATURN.ipynb. When I ran SATURN on my data. The running was just killed without any errors.
The code I used is below:
python3 code/train-saturn.py --in_data=data/saturn/run_hu_ss.csv --in_label_col=manual_anno --ref_label_col=manual_anno --num_macrogenes=2000 --hv_genes=8000 --centroids_init_path=results/saturn/hp_centroids.pkl --score_adata --ct_map_path=data/saturn/cell_type_map_hu_ss.csv --work_dir=results/saturn --device_num=0
The running output is
Global seed set to 0  Using Device 0 Set seed to 0 After loading the anndata human View of AnnData object with n_obs × n_vars = 53686 × 17384 ... After loading the anndata pig View of AnnData object with n_obs × n_vars = 85375 × 13350 ... Killed
I am using A100 (80G)
Any idea about the error?
Thanks
Yile

Hi, I think the crash could be happening because of the system running out of memory when creating the centroids, which is unrelated to the GPU.

Hi,
Thanks for the reply.
Okay. I will try a bigger memory on the hpc.
I am closing it.
Thanks
Yile


Added The Crown

Added The Crown (2016) in Series

Thank you so much for the submissions! I ran finalize (a command in ImportBuddy) to generate the disc json files and I fixed up some of the disc names. I will get this submission up on the site during the next data import (probably in the next day or so)


fix(openrpc.json): schema refs

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

1.1.1
GitHub release
1.1.1

Your semantic-release bot :package::rocket:


Document TypeScript mode

From this link, it seems like Sweet.js supports a TypeScript mode, but this isn't referenced anywhere in the docs.

It doesn't seem like it even properly supports TypeScript (see #482), so sweet.js is rather useless for this right now. Nevermind then. :-/

Yes, TS support is planned but not implemented yet.


[Feature Request] GIT-LFS Image Transformations in Dev

TL;DR: It would be terrific if image transformation feature worked via netlify dev.
(Originally posed to the forum but posting here as suggested by @fool.)
Scenario: I'm using Netlify's image transformation feature to transform images uploaded via git-lfs for a static website. I can test all aspects of the website locally except for image resizing. To test this, I have to push to the CI as the cropping arguments are ignored in dev.

+1
Would also love to see such a feature... Have been tinkering around that and while working it's still inefficient.

Yes being able to test this in development would be really helpful.

Large media is becoming deprecated.


Introduce --retry

This introduces a --retry option as discussed in #1942
1- Change the behaviour of --quake to abort as early as possible. That is, when successes have met the low bar, or when it's impossible for them to meet it.
2- Add a --retry N option that is an alias for --quake 1/N
3- Add an optional /k suffix to the argument of --quake to not abort early (mimicking the previous behaviour)
4- Some tweaking of error and debug messages so there's no confusion between --quake and --retry. Internally they are pretty much the same thing, but the user does not need to care about that (but should know that using one disables the other).
Tagging @nikswamy and @aseemr in case they have comments on the feature and/or output messages (I'm pretty confident in the code itself).
Fixes #1942

I'm adding an improved FStar.Math.Lemmas to this PR which passes --quake 100. I hadn't merged before since I had removed some lemmas that were used externally. I'll check across everest this time and possibly write an interface.
(Ironically, the only proof that I've seen randomly fail now is distributivity_add_left... but not sure what I can do about that besides spelling everything out by induction.)

I've merged the part about --retry, I think it's reasonable enough. Working on the Math.Lemmas thing to get something consistent considering everest too.


radiobuttons (checkbox-groups) added

I have recently modified the core-script src/fpdm.php in such a way that it now manages to cope with groups of checkboxes.
At least I think so;)
I'm not familiar with the github-habbits, yet: Shall I upload it here? Maybe after further testing and eliminiting some possible bugs left the modificatiosn might be worth another update.
The example-Form to integrate also contained some ugly umlauts and also a hyphen in the field-definitions. As for this I'd recommend to change the intial regex definition (Line 41) to
$FPDM_REGEXPS= array(
...
"name"=>"//([-#\w]+)/",
...
);
though I think this needs some more testing.

Right now I came across FPDM and noticed that radio buttons are not supported yet. Therefore it would be highly appreciated if you made your modifications available.

@OliverHenniges anyway you can share your modified core?


Update GES model to interact with database

GES model version that takes data from database, runs calibration, runs simulation and returns simulation results to the database.
Also added in to ingress - scripts to read in data from WeatherWarm.csv and MonitoredSynthetic.csv files.
Note these scripts have paths that need re-writing to more generalisability.

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

Project coverage is 90.85%.  Consider uploading reports for the commit d0e3f7b to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #250      +/-   ##
==========================================
+ Coverage   90.83%   90.85%   +0.01%
==========================================
  Files          78       78
  Lines        5674     5674
==========================================
+ Hits         5154     5155       +1
+ Misses        520      519       -1

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

This draft PR is to mark this branch. This branch has the code for the GES model. It has been integrated into DTBase but requires reviewing. There is also a question of whether this should be merged into the main because it may not be considered a 'standard' model. However, it could be very useful for others.


x/sys/unix: add support for new fscrypt ioctls from Linux v5.4

For https://github.com/google/fscrypt/pull/148, we need the sys/unix package to support the latest fscrypt UAPI from Linux v5.4.  This includes new ioctls and some other updates.
I'm working on a patch for this.

/cc @ianlancetaylor @bradfitz @tklauser per owners.


jetson Orin devkit DP 50.1 ubuntu 22.04 [ upgraded from 20.04] ros Humble - the node won't build

Preliminary Checks

[X] This issue is not a duplicate. Before opening a new issue, please search existing issues.
[X] This issue is not a question, bug report, or anything other than a feature request directly related to this project.

Proposal
steps which are supported to build the node for humble ?
Use-Case
-- stderr: zed_components
CMake Error at /workspaces/zed_ws/install/zed_interfaces/share/zed_interfaces/cmake/ament_cmake_export_targets-extras.cmake:18 (message):
Failed to find exported target names in
'/workspaces/zed_ws/install/zed_interfaces/share/zed_interfaces/cmake/export_zed_interfaces__rosidl_generator_cExport.cmake'
Call Stack (most recent call first):
/workspaces/zed_ws/install/zed_interfaces/share/zed_interfaces/cmake/zed_interfacesConfig.cmake:41 (include)
CMakeLists.txt:71 (find_package)

Failed   <<< zed_components [2.00s, exited with code 1]
Anything else?
No response

Hey @AndreV84 I am experiencing the same error but using ROS2 Galactic in Ubuntu Bionic. The weird thing is that it was working before. Did you find any solution?

ROS2 Humble is not yet supported by the ROS2 wrapper. We will work on this feature in the new few weeks.
ROS2 Galactic will not be officially supported because it's not an LTS version and the EOL date is very close.
For more information:
https://docs.ros.org/en/galactic/Releases.html

PS Upgrading a Jetson board to a version of the Operating System not officially supported is not a good practice and the final result will be an unstable machine.
In this particular case, the Jetpack 5.0.1 is based on Ubuntu 20.04 and it should not be upgraded to Ubuntu 22.04 because Nvidia does not provide drivers for this version and on our side, we do not provide a ZED SDK version for Ubuntu 22.04 running on Jetson.

I close the issue because it's not valid.

@AndreV84 I fixed the issue by downgrading CMake to version 3.23.2. Apparently ament_cmake has an issue with CMake 3.24+
See: https://github.com/ament/ament_cmake/issues/396

@Myzhar
How do you add the wrapper into existing ros humble container [nvblox nav nvblox humble provided by nvidia] ?
from running container terminal it won't appear that sdk file installation results in satisfaction of requirements for the wrapper as it would fail with
[Processing: nvblox_performance_measurement, realsense_splitter, zed_interfaces]
--- stderr: zed_interfaces
* Found ROS2 humble
* ROS2 humble is officially supported by this package.
---
Finished <<< zed_interfaces [2min 0s]
Starting >>> zed_components
--- stderr: zed_components
* Found ROS2 humble
* ROS2 humble is officially supported by this package.
CMake Error at CMakeLists.txt:78 (find_package):
  By not providing "FindZED.cmake" in CMAKE_MODULE_PATH this project has
  asked CMake to find a package configuration file provided by "ZED", but
  CMake did not find one.

  Could not find a package configuration file provided by "ZED" (requested
  version 3) with any of the following names:

    ZEDConfig.cmake
    zed-config.cmake

  Add the installation prefix of "ZED" to CMAKE_PREFIX_PATH or set "ZED_DIR"
  to a directory containing one of the above files.  If "ZED" provides a
  separate development package or SDK, be sure it has been installed.

---
Failed   <<< zed_components [1.56s, exited with code 1]

zed sdk is already installed systemwide
trying to install it from container gets through some of steps but will throw
To continue you have to accept the EULA. Accept  [Y/n] ?y
Installing...
Installation path: /usr/local/zed
Checking CUDA version...
OK: Found CUDA 11.4
cp: cannot create directory '/usr/local/zed/tools': Permission denied

at another steps

The error means that you have not installed the ZED SDK.
PS please never post in a closed issue.
If it's a bug kindly create a new issue.
If it's a question or a support request please ask on the Stereolabs forum

The error means that you have not installed the ZED SDK.
PS please never post in a closed issue. If it's a bug kindly create a new issue. If it's a question or a support request please ask on the Stereolabs forum

but it is not a closed issue:
I am still not able to run the zed ros wrapper since " Aug 20" when it was emerged
Does Zed SDK need to be installed two times? both outside the container also within the container? or only within the container? but the internal installation would say it can not acceee /usr/local/zed   - shal  Icreate new question/ issue for that concern necessarily?