feat!: better composite key support

The main idea is to always register the type command, but when there is no need to take over user input (generally in insert mode), forward the received inputs to the default:type command.
The implementation is to judge the input before forwarding. If it matches, call the configured command, otherwise continue to forward to the default:type
At present, this implementation is normal, but it feels that forwarding input causes a slight delay in input (this may be a psychological effect). Need help testing this. If there is indeed a delay, consider forwarding inputs only when the user needs the composite keys. If the user does not need to use composite keys, then just dispose the command when there is no need to take over user input. ( Even if there is a delay, it is very slight.)

New configurations:

compositeTimeout
compositeKeys

Removed commands: vscode-neovim.compositeEscape1 vscode-neovim.compositeEscape2
Usage

jj to escape

{
    "vscode-neovim.compositeKeys": {
        "jj": {
            "command": "vscode-neovim.escape"
        }
    }
}

jk to escape and save

{
    "vscode-neovim.compositeKeys": {
        "jk": {
            "command": "vscode-neovim.lua",
            "args": ["vim.api.nvim_input('<ESC>')\nrequire('vscode-neovim').action('workbench.action.files.save')"]
        }
    }
}

close #563
close #460
close #330
close #493
close #644
close #892

FWIW, I've been using this most of today and haven't noticed any(?) perceptible delays on input

I used to have jk mapped to composeEscape 1 and 2 using the following:
{
"command": "vscode-neovim.compositeEscape1",
"key": "j",
"when": "neovim.mode == insert && editorTextFocus",
"args": "j"
},
{
"command": "vscode-neovim.compositeEscape2",
"key": "k",
"when": "neovim.mode == insert && editorTextFocus",
"args": "k"
},
after switching to:
"vscode-neovim.compositeKeys": {
"jk": {
"command": "vscode-neovim.lua",
"args": [
"vim.api.nvim_input('')\nrequire('vscode-neovim').action('workbench.action.files.save')"
]
}
}
I am getting a bug where when I have my cursor position set in insert mode (say on line x), then use my mouse to click elsewhere (say on line y), then press jk, my cursor ends up going back to line x instead of staying on line y. This throws me off and did not occur with the old settings.

@MaheshNat  Change args to
require('vscode-neovim').action('vscode-neovim.escape') require('vscode-neovim').action('workbench.action.files.save')

Composite escape should only be for insert mode commands.

Hey thanks for the feature!
This actually breaks my use case. My escape is ",,", which worked with the previous version but not this one.
Another feedback is that it's not clear in the documentation that we are removing a key config and replacing it with something settings.json.

hi! would it allow to use alt-key as escape key for all the keys used by neovim?
i.e. say, i press alt-j in insert mode, then it would become equivalent to pressing escape then j thus bringing me in normal mode and then navigate down.
this would help me immensly.

https://github.com/helix-editor/helix/issues/3162

@goyalyashpal please open a separate issue for this, as it's pretty unrelated to this PR

The new setting will trigger a file save in "interactive python window" which I have no intention to save the results there. Can we use anything other than 'workbench.action.files.save so it does not trigger the file saving please?

{
    "vscode-neovim.compositeKeys": {
        "jj": {
            "command": "vscode-neovim.escape"
        }
    }
}

@echaya
You can use any commands you want. workbench.action.files.save is just for example.

Hey thanks for the feature!
This actually breaks my use case. My escape is ,, (two commas), which worked with the previous version but not this one.
Another feedback is that it's not clear in the documentation that we are removing a key config and replacing it with something settings.json.

I had made this error and was placing the code snippet in the keyboard shortcuts.json file. It would really help if I this specific change was mentioned on the composite Keys section of the Details panel. I had to find out about this by asking on Reddit.

Hey thanks for the feature!
This actually breaks my use case. My escape is ,, (two commas), which worked with the previous version but not this one.
Another feedback is that it's not clear in the documentation that we are removing a key config and replacing it with something settings.json.

I had made this error and was placing the code snippet in the keyboard shortcuts.json file. It would really help if I this specific change was mentioned on the composite Keys section of the Details panel. I had to find out about this by asking on Reddit.

I second this. I also had to come to this post to figure out it is settings.json that i need to add the new setting.

If there is a problem, please open an issue.

Hey thanks for the feature!
This actually breaks my use case. My escape is ,, (two commas), which worked with the previous version but not this one.
Another feedback is that it's not clear in the documentation that we are removing a key config and replacing it with something settings.json.

I had made this error and was placing the code snippet in the keyboard shortcuts.json file. It would really help if I this specific change was mentioned on the composite Keys section of the Details panel. I had to find out about this by asking on Reddit.

I second this. I also had to come to this post to figure out it is settings.json that i need to add the new setting.

Same! But now that I figured out settings.json (not keybindings.json) good to go. Thanks.


[MINOR] Resolve a number of miscellaneous build warnings

What changes were proposed in this pull request?
This change resolves a number of build warnings that have accumulated, before 2.x. It does not address a large number of deprecation warnings, especially related to the Accumulator API. That will happen separately.
How was this patch tested?
Jenkins

Test build #59560 has started for PR 13377 at commit a77be63.

Test build #59560 has finished for PR 13377 at commit a77be63.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/59560/
Test PASSed.

Merged build finished. Test PASSed.

Merged to master/2.0


When added emojiview in textinput view after that my custom view click not working

Please try the new version
pod 'ISEmojiView', '0.1.1'

Please try the new version
pod 'ISEmojiView', '0.1.1'

Thanks , now working fine

Hello Iscade,
Working fine but one issue is need to click on twice on topview option so can please let me know what is a solution

how to open emoji on button click in swift4

Sorry, I am late.
Fixed now, thanks for feedback, please try the new version (0.2.3).


Check checksum issue between uploader and ingest service

Original from @rassokhin-s https://rfcx.slack.com/archives/G018NMPCBHS/p1711100798915549

This issue shows us the upload ID, so when I want to check any error related to the checksum mismatch I see nothing in the grafana log:

Original problem :
// Checks file metadata. Throws IngestionError if data is invalid
if (upload.checksum && upload.checksum !== meta.checksum) {
    throw new IngestionError('Checksum mismatch.', db.status.CHECKSUM)
}

upload id = 65fd51d23bdf6973f9f7d18a
file checksum from db =  4b848e8c77412de247bbc22aa16db97caf5f68d1
file checksum from audio meta = b8ac3e27554f8d2cacf4ea9f210cb77c8958caa5
stream id from audio: vu171ayzkm23
stream id from db : hlcbw0daxt1e
audio meta = {"format":"flac","duration":60,"sampleCount":2880000,"channelLayout":"mono","channelCount":1,"bitRate":399277,"sampleRate":48000,"codec":"flac","tags":{"comment":"Recorded at 15:55:00 26/12/2023 (UTC-5) during deployment CA67479D277F247E at medium gain while battery was 4.2V and temperature was 26.7C.","artist":"AudioMoth 248D9B0260375875","encoder":"Lavf58.24.101"},"size":2994579,"checksum":"b8ac3e27554f8d2cacf4ea9f210cb77c8958caa5"}
Upload metadata from database  {"_id":"65fd51d23bdf6973f9f7d18a","streamId":"hlcbw0daxt1e","userId":"d3e551f1-043a-45bf-aeab-cd41968e0c06","status":10,"timestamp":"2023-12-26T20:55:00.000Z","originalFilename":"20231226_155500.WAV","checksum":"4b848e8c77412de247bbc22aa16db97caf5f68d1","createdAt":"2024-03-22T09:39:30.949Z","updatedAt":"2024-03-22T09:39:56.465Z","__v":0}

device db hasn't any information about deleted device:

The user created the site vu171ayzkm23 via the guardian app 15:12:2023 and manually created the second site hlcbw0daxt1e 20:12:2023, these sites have different names

Hi @rassokhina-e, Thanks for looking into this.

From your comments: https://github.com/rfcx/arbimon-uploader/issues/215#issuecomment-2018041009 https://github.com/rfcx/arbimon-uploader/issues/215#issuecomment-2018044123

I saw that you find out the suspicious case on Device API -- when a site got removed from Core/Arbimon, Device API doesn't know. That case might cause confusion to the user if we still allow them to upload the file to a site that has been removed.

From https://github.com/rfcx/arbimon-uploader/issues/215#issuecomment-2018070482

For this case, I think we should allow the user to upload same file name to different sites.

This https://github.com/rfcx/arbimon-uploader/issues/215#issuecomment-2018030116

Does seem to relate to checksum problem, but I'm still couldn't get my head around it. Can you summarize what was causing the checksum problem, please?

Hi @naluinui , @koonchaya !

For this case, I think we should allow the user to upload same file name to different sites.
Sorry, I put all details during the investigation process and looks like it isn't understandable completely :)

The issue occurs when the user is trying to upload the same name file, but a different checksum to the same site

@rassokhina-e Is it possible that the checksum is incorrect? I think for the Audiomoth recordings there should not be duplicate checksum when user recorded continuously.

Is it possible that the checksum is incorrect

it is not possible, the checksums were correct, the user uploaded the recordings to the site, where the recording name already exists, with a different checksum, it means the user selected not correct site to upload

I think for the Audiomoth recordings there should not be duplicate checksum when user recorded continuously.

from the second recording we see the site related to this recording has different stream id, it means my feedback about selecting not correct site by the user is correct

@rassokhina-e Is there any issue for checksum? What should we do next?

Can I close this ticket?


[Dev] Simplifying the format of *eval_results.json

So far, the schema for *eval_results.json is not easy to understand and use.
https://github.com/evalplus/evalplus/blob/3d1aa126253646b02dc33ef97c99e33c2585803f/evalplus/evaluate.py#L221
I want to propose a better-engineered schema, where for each task_id we have a line of JSON dictionary:

task_id : str
solution : str: the actual code being executed
base_status : str: $\in\{pass, fail, timeout\}$
plus_status : str: $\in\{pass, fail, timeout\}$
base_fail_tests : Optional[List[Any]]: the "base" test inputs that make the solution fail; None if base_status == "pass"
plus_fail_tests : Optional[List[Any]]: the "plus"...; None if plus_status == "pass"

In this way hopefully, model developers can more quickly identify what's wrong and debug/visualize them.

Could we use a list of JSON lines is this format, when it has multiple solutions for each "task_id".
Like
[{task_id, solution, base_status, plus_status, base_fail_tests, plus_fail_tests}, ...]

Thanks for the proposal. To support such a map-like structure I think we can consider:
Each line is a dictionary:

Key: task_id
Value: A list of solution results, each of which follows what I proposed in https://github.com/evalplus/evalplus/issues/95#issue-2125209303


Implementation exposed when deleting missing bucket with CLI

If you attempt to delete a bucket that doesn't exist you get an error message from Bolt bubbling up. We should avoid exposing implementation details in these sorts of error messages.
edd@Alameda:~/go/src/github.com/influxdata/influxdb|master
⇒  influx bucket find
ID			Name		Retention	Organization	OrganizationID
03433857f5c1e000	eddbucket	0s		eddorg		03433857f581e000
edd@Alameda:~/go/src/github.com/influxdata/influxdb|master
⇒  influx bucket delete -i 03433857f5c1e000
ID			Name		Retention	Organization	OrganizationID		Deleted
03433857f5c1e000	eddbucket	0s		eddorg		03433857f581e000	true
edd@Alameda:~/go/src/github.com/influxdata/influxdb|master
⇒  influx bucket delete -i 03433857f5c1e000
bolt/FindBucketByID: <not found> bucket not found

don't close


[FEATURE] Currently, the UI has the NAM Loader, and Wav loader stretched across the entire UI. If adding both another Nam loader, and Wav loader to the plugin is being planned, you could just minimize the empty space by shifting both current icons to the left, cut the long rectangular space for the loader in half, and add the other two loaders to the right, with their own respective icons.  The reason for another wav loader would be longer IR's such as reverbs.  The reason for the additional Nam loader would be a Boost/OD/Fuzz and so on.  So basically the chain would be this:  Top left=Boost, Top right=NAM, bottom left=Impulse Response, bottom right=Wav loader for Reverb IR's.  I think creating two more icons would be appropriate.

Thanks for taking the time to write a feature request! Use the following prompts to help you describe what you're looking for.  The more info you provide, the easier it'll be for me to understand what I can do to make it happen for you without having to come back and ask you questions.
Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Thanks for the suggestion, but I believe that NAM Universal already does what you're looking for :)


fix(key-auth) preflight: revert workaround, implement migration

This builds on top of #2743, but targets the next branch. The additional commit reverts the workaround and implements the proper migrations.

Shall we deprecate (close) #2743 since this is the proper long term solution (it follows our policy of running migrations to include a new configuration property).


I suggest alter to Ctrl/Cmd + Alt + I.


Consider deprecating the split JSON settings editor

Edit - Since this has so many 👎, I need to be very clear, we will never remove editing settings.json files. This is about the UI in the JSON editor, e.g. the pencil icons, the split view, the search bar, etc.

Now that the settings UI is stable, we should consider getting rid of the split JSON settings UI. It would just mean less code to maintain, and new UX feature investment can be focused on the UI.
I think that by default, we would open the default settings and settings.json side by side, and the setting workbench.settings.openDefaultSettings would still control whether the default settings editor is opened.
We need a better understanding of how many people are using it over the settings UI.

I need to be clear, we will never remove editing settings.json files. This is about the UI in the JSON editor, e.g. the pencil icons, the split view, the search bar, etc.

While I don't care much about the pencils, I definitely like the split view and filtering. I would still consider that quite a loss

Would the simpler split view (just two editors like keybindings uses) suffice? And you would still get intellisense in settings.json which I find sometimes more useful anyway.

It looks like this has already been implemented in Insiders... First bad move VS Code has made IMO. 🙁

No, I haven't made any change for this.
I think some people are misunderstanding what this change is about. settings.json is not going away, you can even set it as the default if you want.

Got it. Thanks, Rob. The link in the rich settings UI to open in settings.json seems to be gone in the latest build so that's why I assumed. I'll have to hunt it down...

Oh, under the ... button? Yes that was moved to an editor action button, so now you can get to it with a single click 😄

@roblourens for me I think just a normal split view would suffice. Sometimes I like the search filtering, but I don't think I'd miss it much.

Personally I don't care too much about the pencil (as long as intellisense will still work) but I do like the search/filtering. For settings I'm not a UI fan, I have set the json view as my default

@roblourens The new settings UI doesn't provide configuration for more complicated extensions. For example look up latex extension, 40% of the extension settings cant be configured in the new editor.

@carlocardella Thanks for the heads up that there's a setting for settings. 😂 Just switched mine to be JSON by default! 🎉

When I saw the json settings editor for the first time I thought it was a genius approach.  Especially given that vs code target audience is developers. The pencil button is great, the search is great, comments in the code. It feels so simple and easy. It's easy to find what I'm looking for and edit the settings as needed. The new UI is also simple and easy to use, and is great, but I don't feel it's worth effort to migrate to new UI.

Exactly my reaction: I did not complain when the Settings UI was proposed (I understand people may have different requirements and taste) but given being a developer tool I had not expected the need for a UI, since the Json split view works very well.

Coming from 1.29 to the Insiders build, it took me way too long to figure out that the {} icon was for opening the settings.json file. I thought you guys removed the feature or something. My initial instinct was to think that it would be under the ... button on the tab bar because it had been under the ... before.
The location of the icon got moved to the same row as the upper tab bar which makes it difficult for users to discover where the new button for accessing the JSON is.

Before
After

Can we move the {} button to where the previous ... button was: on the same row as User Settings and Workspace Settings?

@roblourens

This actually makes it more consistent with the rest of vscode, e.g. similar to the button to navigate from a git diff view to the individual file. I understand it might be a road bump for people who have gotten used to the second ... menu but I think it's better long term to be consistent. Also, this puts it a single click away instead of two clicks which was a common complaint.

My suggestion would be sort of a middle ground, so that accessing the JSON is still one click away, but in the previous location. I think it makes more sense to put it on the lower row because it applies only to the Settings view.
There will probably be many reports by users asking where that setting went when 1.30 is released.

Could you open a new issue for that since it's not quite related to removing the UI in the JSON editor?

Sure, created an issue here: https://github.com/Microsoft/vscode/issues/63806
Other than discovering the new button after upgrading I don't think there are major issues with the change.

Maybe we should consider deprecating the GUI settings editor? The JSON settings editor in a developer tool is a brilliant idea I wish more apps would adopt.
I would love an option to disable the GUI settings editor completely to make the VS Code a bit faster and simpler.

If you don't like it, don't use it 😁 See workbench.settings.editor

@roblourens then why is this issue even discussed if you don't plan to to get of the split view editor, but default to the GUI editor instead?

Benefits of the JSON editor

Easy copy&paste of configuration.
Quick overview of your personal settings (the defaults one has chose to override).
Extensions, all of them(?), show the settings as JSON. For example, see here: https://code.visualstudio.com/docs/editor/codebasics. Maintaining images to show of settings is very cumbersome.
You get green squiggly lines on settings that you have configured but that do not longer exist, which gives a quick visual indication.
Possibility to configure more complex settings.

Benefits of the GUI editor

You can click stuff.

Feel free to add to the list.
To reiterate: I've already understood this is not about removing the JSON editor. But you are aiming to do "something" and that "something" I do not like whatever it is :)
But I have no idea why the GUI editor is the default. Do Microsoft employees actually prefer that? As coders? Another topic for another day maybe.

I am moving this issue to https://github.com/Microsoft/vscode/issues/64932
@thernstig Those are all good reasons that you may prefer to use JSON and you are still welcome to do so 😁
But there are many benefits of the GUI editor for new users. Not every vscode user is an experienced coder.  The JSON-based experience is overwhelming to many people who just want to change the font size with a minimal learning curve. When there is a power user option and a friendly option, the friendly option is going to be the default.


Use "returningQuery" function on Select / SelectOne Query

Select query don't use "returningQuery" function as other queries. FInd for which reason and maybe fix it.

returningQuery is to generate the RETURNING part of a query.
Like with update:
UPDATE table SET col="value" WHERE id="id" RETURNING id, col;

Since it's not part of the select syntax, there is no reason for select and selectOne to use it.


fix(module/asg): tag specification

Description
Fixing the launch template to tag resources correctly.
See the attach issue

/sca

/help

/validate paths="modules/asg"

/plan paths="examples/combined_design_autoscale"

/idempotence paths="examples/combined_design_autoscale"


clippy-preview and rls-preview missing on various nightly toolchains

After a fresh rustup installation, I see adding component does work for stable and beta, but not for nightly.
On stable:
$ rustup component list --toolchain stable | grep clippy
clippy-preview-x86_64-apple-darwin (installed)
$ rustup component add clippy-preview --toolchain stable
info: downloading component 'clippy-preview'
info: installing component 'clippy-preview'

On beta:
$ rustup component list --toolchain beta | grep clippy
clippy-preview-x86_64-apple-darwin
$ rustup component add clippy-preview --toolchain beta
info: downloading component 'clippy-preview'
info: installing component 'clippy-preview'

On nightly:
$ rustup component list --toolchain nightly | grep clippy
$ rustup component add clippy-preview --toolchain nightly
error: component 'clippy-preview' for 'x86_64-apple-darwin' is unavailable for download

I think this a generic issue not only related to Mac. My CI is complaining for Linux and Windows too. 😞

I agree that this is larger than OSX.
error: component 'clippy-preview' for 'x86_64-unknown-linux-gnu' is unavailable for download

This has been fixed since 2018-11-06, so let's just close the issue. (Please feel free to re-open if it needs investigation.)

Hey @behnam on clippy is still missing on x86_64-apple-darwin.
I don't know if still makes sense to comment here.


Simplify ParquetExec::new()

Note there is an alternate approach that is not an API change that  uses a dedicated builder in https://github.com/apache/datafusion/pull/10636, but then I realized that
there were many fields (like enable_page_index etc that could already be set with builder apis on ParquetExec so making them redundant with ParquetExec seemed uncessary
Which issue does this PR close?
Part of #10546
Rationale for this change
While working on https://github.com/apache/datafusion/pull/10549 it was cumbersome
to create a ParquetExec -- specifically there are a few fields on ParquetExec::new that
must always be provided even though they are Options and most of the rest of the options can be set via builder style
APIs like with_schema_adapter_factory
In addition, some fields like schema_adapter_factory can not be set by constructor but are instead set
by builder function with_schema_adapater_factor.
Thus it is confusing and imposes a cognitive load that the properties setting is inconsistent.
What changes are included in this PR?

Remove all arguments except FileScanConfig from ParquetExec::new
Add builder style APIs
Improve documentation
Add examples

Are these changes tested?
Yes by existing CI

Are there any user-facing changes?
This is a user API change for anyone who creates ParquetExec's directly (as we do in InfluxDB IOx for example)
There is also better documentation

The more I think about this the more I like https://github.com/apache/datafusion/pull/10636 and deprecate the ParquetExec::new function...


Wrong pert_id in two files

Hi,
First off, thank you for releasing this dataset, it's a great resource!
I noticed that in two of the dataset files, critical_id6_addition.json and major_id5_pp_removed.json, thepert_id value used in the files is 8 even though I believe it should be 6 and 5 respectively. The pert_name value is correct in both files.

Hi thank you, I am in the middle of cleaning up some issues and will aim at releasing v2 soon.


HAWT-FIX: Review, Q&A PUT behavior

What I did or pizza

If pen_name is sent as empty string (""), the Review/Q&A will set itself as non-anonymous and user_name will be shown.
add course_type field to QuestionCreateSchema to get the course Q&A will be creating on.
change test to test for the new Response message

Oh you push new commit. I will recheck again


Relationship issue in Account model

In the past, when using rodauth-rails, I've always had a relationship like this in the account model:
has_one :profile, dependent: delete

This had never resulted in any issues. I created a new rails app and noticed when I have that relationship in the model, and I attempted to run Account.new in the console or any command related to the Account model, I got this error:

I also noticed that when I install rodauth-rails, the include line in the model is include Rodauth::Rails.model. In the past, it was: include Rodauth::Model(RodauthMain).
Are these new and expected behaviours? Cause I'd not experienced them before.

I figured it, I was missing a : 🤦


Game crash with android custom build and using vibrate

Godot version:
3.2 RC3
OS/device including version:
Android 7
Issue description:
I am developping a game with godot 3.2 RC3. I use smartphone vibrate thank the function Input.vibrate_handheld(200).
I try to use the custom build. The export on my android phone is good.
But when the smartphone have to vibrate, the game crash.
Steps to reproduce:
Use the default custom build for android.
Make the smartphone vibrate.
Minimal reproduction project:

Is there any way to hotfix this inside a live project without using a custom build of Godot? E.g. installing the missing dependency as a library?


fix PluginLoader tests

(they still fail because the jars that are used in the tests need to be
updated to reflect the new Plugin API)

ack


Why Builder.limitWriteTo erase limit condition?

When outputting SQL for debugging( using Builder.ToSQL ) before executing SQL, only LIMIT and OFFSET conditions were lost.
I looked up and found the following sentence. Why does builder erase b.limitation ?
https://github.com/go-xorm/builder/blob/145c9968bfe732a56d7160439d25154c5aaa2f35/builder_limit.go#L23

If not, there will be an endless loop. We just make a copy of current limit conditions for following handling. Futhermore,  you can use Bulider.ToBoundSQL instead of Builder.ToSQL,  then you'll have a clearer output.

Thank you for answering. I am glad that it was not a bug.
And thank you for teaching Builder.ToBoundSQL. I will use this, or enable xorm.Engine.showSQL if needed. :)


herbstluftwm: 0.7.0 -> 0.7.1

Motivation for this change
Things done
Note that commit https://github.com/herbstluftwm/herbstluftwm/commit/e5a52f2de0900277767c6ce6e10b5911e448e2fc introduces a new variable FISHCOMPLETIONDIR in config.mk, thus a new line in patchPhase is needed. The package does not seem to have been tested with a non-standard PREFIX (or it was, but mkdir /usr/share/fish/vendor_completions.d worked on a non-Nixos system), which anyway does not seem to be a priority (https://github.com/herbstluftwm/herbstluftwm/pull/107 is opened since 1 year).

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

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

[ ] 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 nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

cc @the-kenny

Thanks! Nice to see that they are still working on herbstluftwm.

@GrahamcOfBorg build herbstluftwm

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: herbstluftwm
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: herbstluftwm
Partial log (click to expand)
/nix/store/jcibdjyk3wyzyg6aw69zlg372sv9w3ag-herbstluftwm-0.7.1/share/doc/herbstluftwm/examples/herbstcommander.sh: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23/bin/bash"
/nix/store/jcibdjyk3wyzyg6aw69zlg372sv9w3ag-herbstluftwm-0.7.1/share/doc/herbstluftwm/examples/dumpbeautify.sh: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23/bin/bash"
/nix/store/jcibdjyk3wyzyg6aw69zlg372sv9w3ag-herbstluftwm-0.7.1/share/doc/herbstluftwm/examples/layout.sh: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23/bin/bash"
/nix/store/jcibdjyk3wyzyg6aw69zlg372sv9w3ag-herbstluftwm-0.7.1/share/doc/herbstluftwm/examples/execwith.sh: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23/bin/bash"
/nix/store/jcibdjyk3wyzyg6aw69zlg372sv9w3ag-herbstluftwm-0.7.1/share/doc/herbstluftwm/examples/toggledualhead.sh: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23/bin/bash"
/nix/store/jcibdjyk3wyzyg6aw69zlg372sv9w3ag-herbstluftwm-0.7.1/share/doc/herbstluftwm/examples/scratchpad.sh: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23/bin/bash"
/nix/store/jcibdjyk3wyzyg6aw69zlg372sv9w3ag-herbstluftwm-0.7.1/share/doc/herbstluftwm/examples/savestate.sh: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23/bin/bash"
/nix/store/jcibdjyk3wyzyg6aw69zlg372sv9w3ag-herbstluftwm-0.7.1/bin/dmenu_run_hlwm: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23/bin/bash"
checking for references to /build in /nix/store/jcibdjyk3wyzyg6aw69zlg372sv9w3ag-herbstluftwm-0.7.1...
/nix/store/jcibdjyk3wyzyg6aw69zlg372sv9w3ag-herbstluftwm-0.7.1

Success on aarch64-linux (full log)
Attempted: herbstluftwm
Partial log (click to expand)
/nix/store/yyxn746pisccgl2vi2dhb88al5g9938s-herbstluftwm-0.7.1/share/doc/herbstluftwm/examples/keychain.sh: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/fqm2x6kiay1q4vg7pqp4wp17bdijlyc3-bash-4.4-p23/bin/bash"
/nix/store/yyxn746pisccgl2vi2dhb88al5g9938s-herbstluftwm-0.7.1/share/doc/herbstluftwm/examples/herbstcommander.sh: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/fqm2x6kiay1q4vg7pqp4wp17bdijlyc3-bash-4.4-p23/bin/bash"
/nix/store/yyxn746pisccgl2vi2dhb88al5g9938s-herbstluftwm-0.7.1/share/doc/herbstluftwm/examples/floatmon.sh: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/fqm2x6kiay1q4vg7pqp4wp17bdijlyc3-bash-4.4-p23/bin/bash"
/nix/store/yyxn746pisccgl2vi2dhb88al5g9938s-herbstluftwm-0.7.1/share/doc/herbstluftwm/examples/float-maximize.sh: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/fqm2x6kiay1q4vg7pqp4wp17bdijlyc3-bash-4.4-p23/bin/bash"
/nix/store/yyxn746pisccgl2vi2dhb88al5g9938s-herbstluftwm-0.7.1/share/doc/herbstluftwm/examples/execwith.sh: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/fqm2x6kiay1q4vg7pqp4wp17bdijlyc3-bash-4.4-p23/bin/bash"
/nix/store/yyxn746pisccgl2vi2dhb88al5g9938s-herbstluftwm-0.7.1/share/doc/herbstluftwm/examples/exec_on_tag.sh: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/fqm2x6kiay1q4vg7pqp4wp17bdijlyc3-bash-4.4-p23/bin/bash"
/nix/store/yyxn746pisccgl2vi2dhb88al5g9938s-herbstluftwm-0.7.1/share/doc/herbstluftwm/examples/dumpbeautify.sh: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/fqm2x6kiay1q4vg7pqp4wp17bdijlyc3-bash-4.4-p23/bin/bash"
/nix/store/yyxn746pisccgl2vi2dhb88al5g9938s-herbstluftwm-0.7.1/share/doc/herbstluftwm/examples/dmenu.sh: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/fqm2x6kiay1q4vg7pqp4wp17bdijlyc3-bash-4.4-p23/bin/bash"
checking for references to /build in /nix/store/yyxn746pisccgl2vi2dhb88al5g9938s-herbstluftwm-0.7.1...
/nix/store/yyxn746pisccgl2vi2dhb88al5g9938s-herbstluftwm-0.7.1


When will foundation 7 be released.

Is foundation 7 still in the works as promised two years ago in the roadmap. Should we consider the project suspended? I love foundation but I am really concerned about its future.
🤔

Seems to be dead. Or at least not maintained.

Not dead. Just slow.

PRs are not reviewed since July.

https://github.com/foundation/foundation-sites/discussions/12501#discussioncomment-4381419

Yes, I’ve seen that. Does it mean that F6 will not get fixes anymore? Should we just wait for v7?

any new news?

anything?

Foundation seems to be meeting the same fate as SemanticUI.

I think it's one guy doing everything.
People need to donate or otherwise chip in to keep projects they care about, going.
I don't like that Foundation uses jQuery.
I was thinking about building a component library and layout system using Tailwind to replicate Foundation but of those I talked with, conversations went dead.
People are moving on, and it's a shame.
With modern js frameworks, Foundation as it is now, doesn't fit into the ecosystems well.
If there is to be a version 7 or 8, it's going to have to take people actually doing something other than complaining.
@joeworkman I'm sure makes nothing on this and has to volunteer his time to keep it moving.

it's going to have to take people actually doing something other than complaining

What exactly do you want people to do? With pull requests being ignored for years, there's very little one can do 🤷🏻♂.  Last PR merged almost 2 years ago.

I was thinking about building a component library and layout system using Tailwind to replicate Foundation but of those I talked with, conversations went dead.

IMO Tailwind is good as it is. I'm actually currently migrating my project to Tailwind from Foundation.

@joeworkman I'm sure makes nothing on this and has to volunteer his time to keep it moving

I have an impression that he uses Foundation in one of his commercial products, which probably was the main reason for taking Foundation over.

@ameotoko You should be grateful that people actually care about foundation and expect regular updates. This indicates that a community of developers are committed to the project success and have choose foundation over hundreds of other frameworks. If the framework is dead, simply archive it and notify the community. I personally detest the fact that jQuery still exists within foundation as such turned to other solutions.
I suggest that the project should be archived if development has halted.

it's going to have to take people actually doing something other than complaining

What exactly do you want people to do? With pull requests being ignored for years, there's very little one can do 🤷🏻♂.  Last PR merged almost 2 years ago.

I was thinking about building a component library and layout system using Tailwind to replicate Foundation but of those I talked with, conversations went dead.

IMO Tailwind is good as it is. I'm actually currently migrating my project to Tailwind from Foundation.

@joeworkman I'm sure makes nothing on this and has to volunteer his time to keep it moving

I have an impression that he uses Foundation in one of his commercial products, which probably was the main reason for taking Foundation over.

I make less than $300/mo. USD and despite 2 years working on another source project, couldn't get a single donation of any amount, otherwise I would have considered forking this project and running with it. Did manage to get $20, but all the people I spoke with who previous expressed an interest in discussing the project, all failed to discuss the project with me. I abandoned my efforts because people don't' seem to be all that serious.
So in summary, I tried to take a lead, but without those who follow, I'm not the right leader. I've done my part.

Hi @joeworkman this roadmap has not been followed and you have made no comments about this topic. Should we assume the project will be archived without further development. Thanks


Tracking quality of (developer) life enhancements

Building and Testing changes from multiple repositories

[ ] #1116
[ ] #1115
[ ] #1117

/lifecycle frozen


Unable to download postman folder

I am unable to download postman older mentioned here:
Import the folder postman in the downloaded zip file instructions into Postman.

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: f10dcb86-8529-c107-a208-36e09a27cd9b
Version Independent ID: f10dcb86-8529-c107-a208-36e09a27cd9b
Content: Auto attendant and call queue historical reports - Microsoft Teams
Content Source: Teams/aa-cq-cqd-historical-reports.md
Service: msteams
GitHub Login: @DaniEASmith
Microsoft Alias: danismith

@CLYVR For your review of this issue.

@kdbhatt I have added the postman folder and it should now be available if you download the file again.  Apologies for the inconvenience.

I can see the postman folder now.


method 'save' can not download the PDF on mobile phone browser

Feature requests, bug reports etc. are very welcome as issues. But questions are directed to stackoverflow with the tag jspdf.
Note that bug reports should follow these guidelines:

A bug should be reported as an mcve
Make sure code is properly indented and formatted (Use ``` around code blocks)
Provide a runnable example.
Try to make sure and show in your issue that the issue is actually related to jspdf and not your framework of choice your setup.

Could you please follow the issue report guidelines you just posted? Please state exactly which browser on which operating system.


Availability of cell painting results for JUMP-MOA in cellpainting-gallery?

Thank you for this resource. Are the cellpainting images for this compound set available within the cellpainting gallery? I see other publications with much larger datasets, but I wonder if the MOA set is a subset of any of them, or collected as part of larger initiatives. Thank you.

Hi,
The majority of cpg0001 was treated with this compound plate; you can check this table to see the exact batches and plate barcodes (for any sets using JUMP-MOA you will see JUMP-MOA in the Treatments column). Hope that helps!


Cannot unpack Jdownloader

Issue
Cannot install Jdownloader. After running brew cask install jdownloader, that's what I get below.
jdownloader requires Java. You can install the latest version with

  brew cask install java

==> Downloading http://installer.jdownloader.org/JD2Setup.dmg
######################################################################## 100.0%
==> No checksum defined for Cask jdownloader, skipping verification
Error: Uh oh, could not figure out how to unpack '/Library/Caches/Homebrew/jdownloader-latest.dmg'

Java is already installed according to java -version though.
Here's the output of brew doctor if that helps
Your system is ready to brew.
Here's the output of brew cask doctor if that helps
==> OS X Release:
10.11
==> OS X Release with Patchlevel:
10.11.3
==> Hardware Architecture:
intel-64
==> Ruby Version:
2.0.0-p645
==> Ruby Path:
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/bin/ruby
==> Homebrew Version:
Homebrew 0.9.5 (git revision 277f; last commit 2016-03-09)
==> Homebrew Executable Path:
/usr/local/bin/brew
==> Homebrew Cellar Path:
/usr/local/Cellar
==> Homebrew Repository Path:
/usr/local
==> Homebrew Origin:
https://github.com/Homebrew/homebrew
==> Homebrew-cask Version:
0.60.0 (git revision 35bb6; last commit 3 hours ago)
Error: undefined method `join' for "/usr/local/Cellar":String
  Most likely, this means you have an outdated version of homebrew-cask. Please run:

      brew uninstall --force brew-cask; brew untap phinze/cask; brew update; brew cleanup; brew cask cleanup

  If this doesn’t fix the problem, please report this bug:

      https://github.com/caskroom/homebrew-cask#reporting-bugs

/usr/local/Library/Taps/caskroom/homebrew-cask/lib/hbc/cli/doctor.rb:186:in `render_install_location'
/usr/local/Library/Taps/caskroom/homebrew-cask/lib/hbc/cli/doctor.rb:15:in `run'
/usr/local/Library/Taps/caskroom/homebrew-cask/lib/hbc/cli.rb:83:in `run_command'
/usr/local/Library/Taps/caskroom/homebrew-cask/lib/hbc/cli.rb:121:in `process'
/usr/local/Library/Taps/caskroom/homebrew-cask/cmd/brew-cask.rb:26:in `<top (required)>'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/rubygems/core_ext/kernel_require.rb:55:in `require'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/rubygems/core_ext/kernel_require.rb:55:in `require'
/usr/local/Library/brew.rb:21:in `require?'
/usr/local/Library/brew.rb:89:in `'

Thank you for the help

Create the folder.
"I was able to resolve the issue by creating the the folder Cellar in /usr/local/ since this is a fresh install /usr/local/Cellar did not exist." - from the last time this was an issue

Thank you for the response. Unfortunately, it didn't solve this issue.
Here's the created folder located in /usr/local/.
 0 drwxr-xr-x   2 master  wheel       68  9 Mar 07:21 Cellar
However the brew cask doctor command gives more information:
==> OS X Release:
10.11
==> OS X Release with Patchlevel:
10.11.3
==> Hardware Architecture:
intel-64
==> Ruby Version:
2.0.0-p645
==> Ruby Path:
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/bin/ruby
==> Homebrew Version:
Homebrew 0.9.5 (git revision 277f; last commit 2016-03-09)
==> Homebrew Executable Path:
/usr/local/bin/brew
==> Homebrew Cellar Path:
/usr/local/Cellar
==> Homebrew Repository Path:
/usr/local
==> Homebrew Origin:
https://github.com/Homebrew/homebrew
==> Homebrew-cask Version:
0.60.0 (git revision 35bb6; last commit 13 hours ago)
==> Homebrew-cask Install Location:
<NONE>
==> Homebrew-cask Staging Location:
/Users/master/Casks
==> Homebrew-cask Cached Downloads:
/Library/Caches/Homebrew
/Library/Caches/Homebrew/Casks
0 files, 0B
==> Homebrew-cask Default Tap Path:
/usr/local/Library/Taps/caskroom/homebrew-cask
==> Homebrew-cask Alternate Cask Taps:
<NONE>
==> Homebrew-cask Default Tap Cask Count:
3005
==> Contents of $LOAD_PATH:
/usr/local/Library/Taps/caskroom/homebrew-cask/lib
/usr/local/Library/Homebrew
/Library/Ruby/Site/2.0.0
/Library/Ruby/Site/2.0.0/x86_64-darwin15
/Library/Ruby/Site/2.0.0/universal-darwin15
/Library/Ruby/Site
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/vendor_ruby/2.0.0
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/vendor_ruby/2.0.0/x86_64-darwin15
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/vendor_ruby/2.0.0/universal-darwin15
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/vendor_ruby
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/x86_64-darwin15
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/universal-darwin15
==> Contents of $RUBYLIB Environment Variable:
<NONE>
==> Contents of $RUBYOPT Environment Variable:
<NONE>
==> Contents of $RUBYPATH Environment Variable:
<NONE>
==> Contents of $RBENV_VERSION Environment Variable:
<NONE>
==> Contents of $CHRUBY_VERSION Environment Variable:
<NONE>
==> Contents of $GEM_HOME Environment Variable:
<NONE>
==> Contents of $GEM_PATH Environment Variable:
<NONE>
==> Contents of $BUNDLE_PATH Environment Variable:
<NONE>
==> Contents of $PATH Environment Variable:
PATH="/Users/master/.rvm/gems/ruby-2.1.4/bin:/Users/master/.rvm/gems/ruby-2.1.4@global/bin:/Users/master/.rvm/rubies/ruby-2.1.4/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/usr/local/git/bin:/Users/master/bin:/Users/master/.composer/vendor/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/usr/local/git/bin:/Users/master/.rvm/bin:/usr/local/Library/Taps/caskroom/homebrew-cask/cmd:/usr/local/Library/ENV/scm"
==> Contents of $SHELL Environment Variable:
SHELL="/usr/local/bin/bash"
==> Contents of Locale Environment Variables:
LANG="en_CA.UTF-8"
==> Running As Privileged User:
No

Same here.
It seems that the correct file can not be downloaded.
The JD2setup.dmg file is just 2kb in size, but should be at least 5.6mb i guess ..

Looks like the download link changed

Download link based on homepage is still - "http://installer.jdownloader.org/JD2Setup.dmg"
Can you check if that direct link works for you?

Yes it does work. However, all download links from this page points to mega.nz.

Seems like that's not the case
 Download Installer Mac OS X Version 10.7 or higher

That's weird
 Download Installer Mac OS X Version 10.7 or higher

@volchub @franatieu I can't seem to reproduce - is this still an issue?

Yes I still get the same error. I think it could be a java related issue. Like mentioned above, whenever I brew cask install jdownloader, I get this message:
jdownloader requires Java. You can install the latest version with

brew cask install java

Even though I run the command brew cask install java, I still get the message every single time.

Hmm, can't currently spin up a clean test environment.
Pinging @vitorgalvao, if you could take a look when you're available.
In the meantime, does a manual install work?

In the meantime, does a manual install work?

Yes it works

That message appearing every time is normal.
I can reproduce the error.

This is sad news. As far as I see every download redirects to mega. Downloading from mega using command line does not look easy (at least it is not possible using a normal curl request).
https://github.com/tonikelope/megadown looks promising, but my openssl does not include the required features.
@winkelsdorf You also worked on this this cask and stated you where in contact with the devs. What do you think about asking them to provide an extra/alternate download option?

If we can’t fix it, we’ll have to remove the cask.

Sad news about their change to use Mega as only available CDN..
@tsparber: I have no other contact as mailing the support. They check the support tickets internally with their dev team. But today I contacted their support asking for another official download link, if any.
I'll let you know as soon as I receive a response. But honestly I wouldn't expect other download links being available, the reason for picking a CDN is obvious.
Right now removing the cask was the right way. We can restore it from history if something changes.

@tsparber Can you please give me feedback if the JDownload cask previously had shown a GUI with progress during it's installation (just for a few seconds)? I received feedback from their team and am currently doing some tests..

@winkelsdorf Sorry, I'm not sure if there's been a GUI. Maybe for a short period (splash screen), but certainly there was no user interaction required. As far as I remember there was also no progress output while the installer is downloading the required files. With a slow internet connection the brew cask install  might seem to stuck.

@tsparber Thanks for the quick reply! Yes, that's what I though too, iirc there previously was some kind of a short splash screen, too. The progress is only visible if java is previously not installed (I am testing this on a clean VM). But no user interaction is required.
Anyway, the cask is now working again :) PR created https://github.com/caskroom/homebrew-cask/pull/20333
Thanks to Appwork and the JDownloader Team :+1:

Thank you for caring :+1:

@tsparber You're welcome :)
@vitorgalvao Cleaned the mess with my repo up, sorry about that. New PR added.

To all following the issue, jdownloader is now available (again) - Thanks to everyone for their contributions!


Increasting the sensitivy of the docker storage check

Increasing the paging sensitivity to 15%, as we have seen customers seeing warning before hitting the 10% mark.

👍


Fix/scenarios restructure

Implement shared modules for scenarios
Resolve #40

[x] scenario 1
[x] scenario 2
[x] skeleton.sh

Fix by @anton-sidelnikov will resolve #47

Will be ready for review after adding fix for #47


Implemented raw block storage and RPC endpoint for retrieving raw block by number

Resolves https://github.com/0xPolygonMiden/miden-node/issues/316

@bobbinth:

The number of block we'll create per year is probably going to be over 10 million and I'm not sure much of a problem it would be. Regardless of this, we may want to figure out some strategy for storing blocks in sub-directories (e.g., define an epoch and store blocks for each epoch in a separate sub-directory). Let's create an issue for this.

Yes, I agree with you, such a big number of files in a single directory might be inconvenient to manage. I introduced epochs in this PR, please take a look! The main idea here is that we get first 16 bits of block number as an epoch, it means, each epoch will store 65536 blocks, it's about 7.5 days per epoch with current block issue frequency.


IC2 Classic machines do not connect with flux plug - 1.19.2

IC2 Classic - https://www.curseforge.com/minecraft/mc-mods/ic2-classic
IC2 Classic uses EU, which is supported by your mod according to the list on the curseforge page.
When I try connecting a flux plug to any type of cable, energy storage unit, or power producing object (like a solar panel), the flux plug does not receive any EU.
I have tried creating a separate network just for using EU, in case that would be a problem, but that did not fix the issue.
When testing, I also tried building out a controller and network storage, just in case the lack of storage was causing an issue, but that did not fix the problem.
I am also seeing that I am unable to connect a flux point to any IC2 Classic machine that uses EU for energy.

I heard that the ic2exp use different energy transfer method from ic2c...


[WIP] Update requirements to latest versions available with Conda

[x] Update requirements.txt and requirements-dev.txt
[ ] Test Julia and R examples.
[ ] Test prediction server.

Coverage remained the same at 100.0% when pulling 2f6a31a2a09bf82a6b1422ce1f5c2bc5121ec1f1 on feature-update-requirements into 6805789f51e7cce4488b6554261b646cb43e8e54 on develop.

Coverage remained the same at 100.0% when pulling 0792a8d4cd39cb45efc7b60c0a6df15905da7d7e on feature-update-requirements into 6805789f51e7cce4488b6554261b646cb43e8e54 on develop.


handle errors

if backed returns non-200, show message from server

by default show an alert, but be able to override with a custom handler


[rom,rom_ext] Consider using SRAM readback feature

Description
Currently, we introduced the #23212 readback feature to the SRAM.
When enabled, each read request to the SRAM will be performed twice and checked. On a write, the written data is read back and compared. On an error, a fatal alert is triggered.
#23395 adds the DIF for using this feature.
In ROM/ROM_EXT we should consider using this feature.

@vogelpi FYI

Thanks for creating the issue Pascal. I am putting this into the ROM milestone. We can already enable it in ROM based on an OTP setting @moidx .

@cfrantz OK to close this issue?


Step to execute some code

Everything can not be planned, so a step to execute an Action would be nice

This things require some redesign of steps to be able to easily precise when to launch it


Control+Backspace Shortcut Causes Node Deletion in ReactFlow Application

Describe the Bug
I'm currently developing a ReactFlow application that involves the use of text inputs within nodes. However, I've come across an unexpected behavior related to the Control+Backspace shortcut. When attempting to use Control+Backspace to erase text within a text input, the action seems to propagate to ReactFlow itself, resulting in the deletion of the entire node.
Your Example Website or App
No response
Steps to Reproduce the Bug or Issue
Launch the ReactFlow application.
Create a node containing a text input element.
Input some text into the text field.
Attempt to erase the text by pressing Control+Backspace.
Expected behavior
The Control+Backspace shortcut should erase the text within the text input field, without affecting the ReactFlow nodes.
Screenshots or Videos
No response
Platform

OS: [e.g. macOS, Windows, Linux]
Browser: [e.g. Chrome, Safari, Firefox]
Version: [e.g. 91.1]

Additional context
No response

😱 what is happening here? we will investigate!

😱 what is happening here? we will investigate!

I tried a workaround to stop the event when you hit "Control + Backspace" on input. But I'm thinking there's probably a better way. Hoping everything goes smoothly. Thanks for understanding!

could you try to add the class name "nokey" to your inputs?

Hi @moklick,
I attempted to include the class in the input as follows: className="nokey", but I didn't observe any changes. Could you please confirm if this is the correct approach to address the error?

Hi @moklick,
I have been looking for a fix within useKeyPress hook (packages/core/src/hooks/useKeyPress.ts) and found out that changing L57 fixes the issue:
from
if (isMatchingKey(keyCodes, pressedKeys.current, false) {

to
f (isMatchingKey(keyCodes, pressedKeys.current, false) && !modifierPressed.current) {

PS: coming from https://github.com/wbkd/react-flow/issues/3416)

Hello, thx for working on this. Do you have an aprox. date for the 11.9.0 release with the fix?

Next week 🚀

this is fixed in 11.9.0


Fix comparison inconsistencies

ProPhoto: https://en.wikipedia.org/wiki/ProPhoto_RGB_color_space#Encoding_function
Rec.2020: https://en.wikipedia.org/wiki/Rec._2020#Transfer_characteristics

Rec. 2020 is probably fine since the value is chosen to be continuous. No idea about the other guy. (The human eye would be able to tell neither, but I agree sticking to the spec is a good idea.)

Yeah, it is purely a "correctness" issue in regards to the spec. I don't imagine there would be any real world issues.

Thank you @facelessuser!


Adding icon submission guidelines (solves JOINDIN-695)

As in title.
I've added a guideline in the sidebar as suggested in the issue, and also a short help message below the upload field in the form, so it's immediately visible, even on mobile.

Can one of the admins verify this patch?

add to whitelist


add terp network rpc resources

hey @discoverdefiteam - thanks for this PR. Couple comments:

We're moving away from hardcoding templates to reading them from config files). See https://github.com/akash-network/console/issues/98 for context.
We want to prioritize offering ready templates for things based on community request - are there any instances of this being requested by the community and/ or popularity of the chain (based on likes, social etc)?

With the above in mind, I'm closing this request for now but feel free to resubmit a PR that updates the config files (rather than code), assuming you have some data for 2. Thanks :)


What are some of these spells?

I've figured out that some of the spells just don't work and thats why i was confused. that said, what are some of these spells? Like whats charge? Does it recharge my energy? Does it refuel a vehicle? The spell book just says when its unlocked and not what it does.
Also How is spawning shotgun shells supposed to work do i need to have a chest near me? Does it need to be one i conjured? Im very confused.

Also i think the point system needs a heavy rework, I get spells way before i can even realistically use them it seems. like i got fast belts but unless i was dedicated all 5 of my points per level to mana exclusively i wouldn't be able to use them, and it only casts one belt anyway, which is infinitely worse (even though its mostly free) than just crafting them, and maybe its because i have an enemy mod that changes how enemies work, but killing seems drastically worse than just crafting and moving around aimlessly, by a wide margin.

Woops! Wrong button - however. All spells are confirmed to work and are just as intended.
Charge is a similar spell from the known game Tibia.

Also i think the point system needs a heavy rework
Thank you for your input - however this will not change.

Originally it was an issue i had posted, i'm not used to github and had missed the issue i had was pinned, apologies if editing it to something less of an issue is the incorrect chain of command but since I'm here, i raise the following issues.
I had to google Tibia and hope the spell charge was named the same to find out its a move speed thing, a description of its own would be greatly helpful though, Like i presumed shotgun shells worked like belt or wall, where it'd give me a couple shotgun shells, it wasnt until i tried out HE and AP shell that i realized its a spell that casts shotgun blast on the cursor.
but I still think everything needs a major rework except maybe dex and biter summoning.
I understand having a system where you get multiple points but the system currently only has any effect if you put all 5 points per level into a stat, so why not just make it one point per level? Aside the other benefits (of which made no impact on my gameplay) str gives an inventory slot every 5 points, so obviously im going to always put 5 into it when i level str, mana you need hundreds of and get 10 every 5 points, so im always going to put the full amount into it, vitality.... well 10 health isnt much, but its still better than putting one point and getting 2HP. putting the full level into dex also helps level you faster as well, hand crafting gets you the most XP it seems, so getting the most amount of crafting speed you can per level is a given (heck, its probably OP)
I get that casting a bunch of belts for the cost of a fish and some mana is kind of OP but in a game where you are already making thousands of belts, pipes, walls, etc is kind of like trying to move water with an eye dropper when the water fall next to you is doing the job anyway.
I think it might also be more useful to have assemblers, power poles and maybe miners or smelters instead of belts, pipes and boxes, burner inserters could also be good, and having some mana at the start would be very helpful for actually utilizing some of these factory related spells we get by being level one.


[Usage] How can I implemet few shot learning on LLaVa

Describe the issue
Hi there,
I have some images and some custom explain.
So I want to implement few shot learning to make summaries of my images.
This is my current implement:
templates = [
    {
        "url": "",
        "explain": """""",
    },
    {
        "url": "",
        "explain": """""",
    },
    {
        "url": "",
        "explain": """"""
    },
    {
        "url": ",
        "explain": """"""
    },
    {
        "url": "",
        "explain": """"""
    },
]

My code to build prompt:
from PIL import Image
import cv2
import numpy as np
import requests

"""Make image summary"""
img_prompt = "User: <image>\n"+"\nASSISTANT:"

prompt = (
    "You are an assistant tasked with summarizing images for retrieval. "
    "These summaries will be embedded and used to retrieve the raw image. "
    "Give a concise summary of the image that is well optimized for retrieval."
)
print(prompt)

images = []

for i, temp in enumerate(templates):
    image_i = Image.open(requests.get(temp['url'], stream=True).raw)
    eplain_i  = temp["explain"]
    example_i = f"\nUser: <image{i}>"+"\nASSISTANT:" + eplain_i + "\n"
    prompt += example_i
    images.append(image_i)

prompt += f"\nUser: <image{len(templates)}>"+"\nASSISTANT:"
print(prompt)
print('-'*100)
print("Examples:", len(images))

Inference:
target = Image.open("figures/figure-2-5.jpg")

out = model_multi_modals(
    images=images+[target],
    prompt=prompt,
    generate_kwargs={"max_new_tokens": 2048})

And my error:
ValueError: The input provided to the model are wrong. The number of image tokens is 0 while the number of image given to the model is 1. This prevents correct indexing and breaks batch generation.

In-context learning or fine tuning

Is it solved? Because I use SGLang for batch inference and I also need this feature for ICL and multiple discussions or few shot.

image{len(templates)}

Describe the issue
Hi there,
I have some images and some custom explain. So I want to implement few shot learning to make summaries of my images.
This is my current implement:
templates = [
    {
        "url": "",
        "explain": """""",
    },
    {
        "url": "",
        "explain": """""",
    },
    {
        "url": "",
        "explain": """"""
    },
    {
        "url": ",
        "explain": """"""
    },
    {
        "url": "",
        "explain": """"""
    },
]

My code to build prompt:
from PIL import Image
import cv2
import numpy as np
import requests

"""Make image summary"""
img_prompt = "User: <image>\n"+"\nASSISTANT:"

prompt = (
    "You are an assistant tasked with summarizing images for retrieval. "
    "These summaries will be embedded and used to retrieve the raw image. "
    "Give a concise summary of the image that is well optimized for retrieval."
)
print(prompt)

images = []

for i, temp in enumerate(templates):
    image_i = Image.open(requests.get(temp['url'], stream=True).raw)
    eplain_i  = temp["explain"]
    example_i = f"\nUser: <image{i}>"+"\nASSISTANT:" + eplain_i + "\n"
    prompt += example_i
    images.append(image_i)

prompt += f"\nUser: <image{len(templates)}>"+"\nASSISTANT:"
print(prompt)
print('-'*100)
print("Examples:", len(images))

Inference:
target = Image.open("figures/figure-2-5.jpg")

out = model_multi_modals(
    images=images+[target],
    prompt=prompt,
    generate_kwargs={"max_new_tokens": 2048})

And my error:
ValueError: The input provided to the model are wrong. The number of image tokens is 0 while the number of image given to the model is 1. This prevents correct indexing and breaks batch generation.

I think The error is because of the  token. In the prompt, the image token should be given as  and not by image id or image index. I got a similar error in my setup for multi-prompt.
BTW, the model is not capable of performing directly on multiple images and prompts simultaneously, as is evident from the following conversations by the author and others.
https://discuss.huggingface.co/t/llava-multi-image-input-support-for-inference/68458
https://github.com/haotian-liu/LLaVA/issues/197#:~:text=Due to the current way of training%2C we do not observe the model having very good capability referring to / comparing with multiple images. We are working on improving this aspect as well%2C stay tuned!
https://github.com/haotian-liu/LLaVA/issues/57#:~:text=Due to the current way of training%2C we do not observe the model having very good capability referring to / comparing with multiple images.
https://github.com/haotian-liu/LLaVA/issues/57#:~:text=Due to the current way of training%2C we do not observe the model having very good capability referring to / comparing with multiple images.
https://huggingface.co/YouLiXiya/tinyllava-v1.0-1.1b-hf/discussions/1#:~:text=The training is based on a single image. Multiple images are not supported

Hi guys, you can use our implemented codebase for ICL. https://github.com/ys-zong/VL-ICL