Tendermint34Client "min height 8014 can't be greater than max height 8013"

First off, I'm really happy to be putting CosmJS to work!  Thanks for this project.
Things are working pretty well, but I'm running into intermittent failures when trying to do provable queries from a web browser to a local chain.  My client is currently issuing a new query every block or so (4-6 seconds).
Here a sample error message I get in the browser:
 Error: {"code":-32603,"message":"Internal error","data":"min height 8274 can't be greater than max height 8273"}
  at HttpClient.execute (httpclient.ts:89:1)
  at async Tendermint34Client.doCall (tendermint34client.ts:341:1)
  at async QueryClient.getNextHeader (queryclient.ts:614:1)
  at async QueryClient.queryVerified (queryclient.ts:523:1)

the stack trace is async after that.
Every error I've seen is of the format "min height ${N+1} can't be greater than max height ${N}".  It looks like there's some bad state somehow?
Here is the query I'm making:
const key = new Uint8Array([
     51,   0, 112, 117,  98, 108, 105, 115, 104, 101, 100,
      0, 119,  97, 108, 108, 101, 116,   0,  97, 103, 111,
    114, 105,  99,  49,  50, 102, 108,  99, 106, 109, 113,
     55,  48,  48, 102, 117, 107,  48,  52,  50, 100, 103,
    100, 119,  50,  99,  97, 101, 116, 107,  99, 106, 106,
     55, 118,  55, 119,  52, 107, 119,  55, 104
  ]);
await queryClient.queryVerified('vstorage', key, undefined)

I'm using @cosmjs/tendermint-rpc 0.28.7.

Thanks for any help you can offer,
Michael.

Hey Michael, which Tendermint version are you connecting to?
Assuming Tendermint does not mix up things, the error message makes sense. Personally I'd say [8274, 8273] is an empty interval and not an error case, but fine.
So where do the values come from? The only place where we have min and max height in one call is the public async blockchain(minHeight?: number, maxHeight?: number): Promise<responses.BlockchainResponse> { function in the Tendermint client. This is called here: https://github.com/cosmos/cosmjs/blob/v0.28.7/packages/stargate/src/queryclient/queryclient.ts#L614. At this place height (min height) seems to be greater than searchHeight (max height). However, I don't see how this is possible with const searchHeight = height + 1;.
If you have the chance to debug the input value of line 614 (e.g. manually hacking the .js file in your node_modules), that would be great.


k3s: update script is failing

Describe the bug
k3s: update script is failing for 1.29.4+k3s1

No auto update branch exists
Old version 1.29.3+k3s1" not present in master derivation file with contents: [...]

Steps To Reproduce
Logs:

https://r.ryantm.com/log/k3s/2024-04-30.log
https://r.ryantm.com/log/k3s_1_28/2024-04-27.log
https://r.ryantm.com/log/k3s_1_27/2024-04-26.log

Expected behavior
Bot should open a PR updating to 1.29.4+k3s1 successfully.
Additional context
Works locally.
Notify maintainers
@euank @yajo @Mic92

Thanks for noticing and reporting it!
I looked into this a little, but I didn't figure out why it broke, here's my notes.
The error we see in the logs is from here:
https://github.com/nix-community/nixpkgs-update/blob/e2050fb126d1353d6c33b626a8aaf2067b7abe4c/src/Nix.hs#L227-L234
That's looking for the version in the attrPath, which it gets from here:
https://github.com/nix-community/nixpkgs-update/blob/e2050fb126d1353d6c33b626a8aaf2067b7abe4c/src/Nix.hs#L138
Which is:
$ EDITOR=echo nix edit k3s_1_28 -f
<nixpkgs>/pkgs/applications/networking/cluster/k3s/builder.nix

It's that value because nixpkgs pulls that value from meta.position, which is from meta.description:
https://github.com/NixOS/nixpkgs/blob/3fb8aadfd03c5fd44ca84a38a41def7c4631dfb3/pkgs/stdenv/generic/make-derivation.nix#L519-L520
https://github.com/NixOS/nixpkgs/blob/9a1693d3dd2bfc5cd150d41669059aa2a8a15455/pkgs/applications/networking/cluster/k3s/builder.nix#L83
So all k3s derivations have a meta.position of builder.nix, which doesn't have the version.
I'm able to reproduce this error myself, both with old nixpkgs-update versions and the current one, so I don't think a nixpkgs-update change broke this.
I guess next up is to git bisect nixpkgs itself, and see if that leads to enlightenment as to when this broke :smile:
Alternatively, we could work around this by inverting it so the k3s derivations are the "root" (k3s/1_28/default.nix has meta set for position, and also contains the versions.nix stuff, and imports builder), instead of having default.nix import versions.nix + builder.nix and combine them.
I kinda like the idea of refactoring them to that structure anyway since it makes nix edit work correctly, and feels more "normal"

The go compiler patch no longer cleanly merges. (Needs adaptation.)
And go compiler check only happens at runtime. Meaning, builds won't fail, but tests will fail.
Maybe that could influence the update script.

@superherointj https://github.com/NixOS/nixpkgs/pull/309110

FYI to folks running it manually: cd'ing into the k3s dir and running the update script by hand does work.


Feature/1221 frontend add single question page with text component

Was

Now

With additional features

Known:

Bugs exist in JS handling of drop menu items and will be addressed in separate ticket
Header wrapping of form name has already been called out on previous ticket
Currently 1 failing acceptance test affecting destination page (new form page edit) after interacting with functionality on this and, not expected to be failure of changes relating to this actual (form overview) page.

Have discussed failing test with @tomas-stefano
He is ok with it and will address/fix when appropriate.
Screenshot of failing test added to Trello ticket.


fix(navbar): add back onClick callback to navbar links

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Feature] Spark connector supports to specify fields to write

Proposed changes

By default , Spark connector must write all fields value to Doris table .
In this feature , user can specify part of fields to write ,  even specify the order of the fields to write.

eg:
I have a table named student which has three columns (name,gender,age) ,
creating table sql as following:
create table student (name varchar(255), gender varchar(10), age int) duplicate key (name) distributed by hash(name) buckets 2;

Now , I just want  to write values to two columns : name , gender.
The code as following:
    val df = spark.createDataFrame(Seq(
      ("m", "zhangsan"),
      ("f", "lisi"),
      ("m", "wangwu")
    ))
    df.write
      .format("doris")
      .option("doris.fenodes", dorisFeNodes)
      .option("doris.table.identifier", dorisTable)
      .option("user", dorisUser)
      .option("password", dorisPwd)
      //specify your fields or the order
      .option("doris.write.field", "gender,name")
      .save()

Refactor spark connector unit test code .

Types of changes
What types of changes does your code introduce to Doris?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Documentation Update (if none of the other choices apply)
[ ] Code refactor (Modify the code structure, format the code, etc...)
[ ] Optimization. Including functional usability improvements and performance improvements.
[ ] Dependency. Such as changes related to third-party components.
[ ] Other.

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[ ] I have created an issue on (Fix #ISSUE) and described the bug/feature there in detail
[x] Compiling and unit tests pass locally with my changes
[x] I have added tests that prove my fix is effective or that my feature works
[x] If these changes need document changes, I have updated the document
[ ] Any dependent changes have been merged

It is best to give an example of this use in the documentation, which is more convenient for users to refer to when using it

Add a example of a specified field option

one more question, do you think doris.write.fields sounds more reasonable than doris.write.field ?

one more question, do you think doris.write.fields sounds more reasonable than doris.write.field ?

As to the filed name , I just wanted to  be consistent with the field doris.read.field
http://doris.apache.org/master/en/extending-doris/spark-doris-connector.html#sql-dataframe-configuration

fix option doris.write.field to doris.write.fields
remove comment line in TestSparkConnector.scala
update en/cn doc for doris.write.fields
pls remove RestService.java in second commit , it shuld not been commited.


Toast component

Summary
Created a toast component using snackbar from mui. Added dynamic capabilities to show success, error, warning, and info messages as well as user choice to dismiss or automatically dismiss after x seconds. Made color differences for each type of message and smooth enter/exit appearance.
Testing
Created a sample button on home page that calls the toastbar component to show up.
closes #29

Wasn't able to get this working on my end? Couldn't see the button on the home page.

Need to revise the toast button as past of your revision!

fixed toast component
https://github.com/user-attachments/assets/9fe06b82-1100-4c5f-b760-d792d15a88f7


Fix issue with state_publisher.py

edited the src/svea_core/scripts/state_publisher.py
SPIN_RATE -> self.SPIN_RATE

Closes #21.


forwardport of [log] View log repeat output #32084

forwardport: [log] View log repeat output #32084

There is no such thing as "forward porting" a fix to 2.6 because there are currently no separate dashboard branches for 2.5 vs 2.6 vs master

Closing based on https://github.com/rancher/rancher/issues/32128#issuecomment-819913918


[sct_resample] BUG when downsampling by more than 10

command:
sct_resample -i /Users/alzaia/ariane/test/multilabel_template_seg_reg_crop_reg.nii.gz -f 0.1

BUG:
Traceback (most recent call last):
  File "/Users/alzaia/spinalcordtoolbox/scripts/sct_resample.py", line 365, in <module>
    main()
  File "/Users/alzaia/spinalcordtoolbox/scripts/sct_resample.py", line 362, in main
    resample()
  File "/Users/alzaia/spinalcordtoolbox/scripts/sct_resample.py", line 109, in resample
    R[i, i] = n[i] / float(n_r[i])
ZeroDivisionError: float division by zero

data in foldersahred/sct_issue/20170106_issue1103

Other bug for an other image same folder..
command:
sct_resample -i /Users/alzaia/ariane/test/multilabel_histo_seg_crop.nii.gz -f 0.5

Load data...
Traceback (most recent call last):
  File "/Users/alzaia/spinalcordtoolbox/scripts/sct_resample.py", line 365, in <module>
    main()
  File "/Users/alzaia/spinalcordtoolbox/scripts/sct_resample.py", line 362, in main
    resample()
  File "/Users/alzaia/spinalcordtoolbox/scripts/sct_resample.py", line 64, in resample
    nii = nipy.load_image(param.fname_data)
  File "/Users/alzaia/spinalcordtoolbox/python/lib/python2.7/site-packages/nipy/io/files.py", line 62, in load
    return nifti2nipy(ni_img)
  File "/Users/alzaia/spinalcordtoolbox/python/lib/python2.7/site-packages/nipy/io/nifti_ref.py", line 550, in nifti2nipy
    raise NiftiError("With less than 3 dimensions we cannot be sure "
nipy.io.nifti_ref.NiftiError: With less than 3 dimensions we cannot be sure which input and output dimensions

Please assign for me, I don't have rights


I can not found the function of language setting

Submit feature requests to http://www.mattermost.org/feature-requests/. File non-security related bugs here in the following format:
Summary
i found the react-native-i18n.but,I can not found the function of language setting in “mattermost-mobile/app/screens/settings/display_settings”
Environment Information

Device Name: Android 7.1
OS Version:
Mattermost App Version: Mobile version 1.7.1
Mattermost Server Version:mattermost-4.9.0-linux-amd64

Steps to reproduce
build and run Mobile version 1.7.1 on Android device.
Expected behavior
I used the 'mattemost classic' on IOS device.But, I can not found the function of language setting in the source code.
Observed behavior
Possible fixes

@AL-Cui I do not know exactly what you mean, but in any case the translation files can be found here https://github.com/mattermost/mattermost-mobile/tree/master/assets/base/i18n let me know if this isn't what you mean

Ok,I know the i18n.I want to say that I can not find the button of language setting

Hi @AL-Cui
If you're looking to change the language setting from within the UI, you'll need to change it in a browser first by going to the "hamburger menu", selecting "Account Settings" > "Display" > "Language".  This will change the language and then when you log in on mobile, the language will be what you changed it to.
We're still working on incorporating these settings into the mobile app.

@lindy65 Hi,
I compiled the "mattermost-mobile" source code and run it on Android device.Then,I selecting "Account Settings" > "Display",i can not find the "Language". There is only one button,"Clock display"。
Does the "language" function have not been done yet，or,I downloaded the wrong source code.I downloaded the source code from this link,"https://github.com/mattermost/mattermost-mobile.git".
Thank u vary much!

Hi @AL-Cui
The feature cannot be set on mobile yet.  You'll need to log into your account on browser, set the language and then log in on Android device.

@lindy65
OK,Thank u!
Do you know that how to download the "mattermost classic"source code.I found a APP named "mattermost classic" rather than "mattermost beta".The interface of this APP  is similar to the  browser that you said. And,this APP has the "Language Setting"function. Do you know this?And ,Do you know how to download the source code about mobile device.

Hi @AL-Cui
My pleasure,
You can find information on compiling your own apps here
Hope this helps?

@lindy65
Thank you vary much!
I found the source code what I want from here

Fantastic, thanks for posting back @AL-Cui :)  I'll close this issue off here then ...


Config Abstraction/Keybindings Birth WIP

Abstraction
We have the need for config other than just user config... such as keybindings, and colors. So I abstracted it, and now it should be easily manipulable for adding new kinds of configs.

Allows for keybindings and config not to get muddied up.
Will allow for oni.color not to be muddied with config/keybindings.

Keybindings  #567 #20 #366 #404
Allow the user to select Onis keybindings

You provide an array of strings, these strings will then test if you hit them with it's string alias/key, and if any of those are hit, fire the action.

TODO :

I'm not sure if keybindings reload if you change the file...
Ensure keybindings work with plugins.
Make sure keybindings files work
Add tasks action
Document
Add tab keybindings ([b ]b I think this is in vim somewhere I didn't look for it.)
Check if keys are duplicate? I never had issues with this in vim... so it's up for debate, I always figured if I overrode a key and didn't notice I didn't need it anyway... and if I did I would realize I DID use that key... but some may see it differently.

Other issues this will affect:

Make sure the abstraction will work well for oni specific colors. (status bar, foreground, background, blah blah)  #604 #33
Make a nice activate/binding method for config #610 #619

Example
// Opened menu keys
        "keybindings.openedMenu.nextMenuItem"         : ["<C-n>"    , "<C-j>"  , "<down>"] ,
        "keybindings.openedMenu.previousMenuItem"     : ["<C-p>"    , "<C-k>"  , "<up>"]   ,

Last gif showing what was accomplished.
You can tell I don't use my mouse much when it comes time to open a folder lol...

@cyansprite , thanks for putting together this PR and the thought you've put into the keybindings!
I like the abstraction you put together for the configuration - it's great to separate the loading / watching logic from all the values.
The model being proposed for the keybindings seems like it addresses the short-term problem of not being able to map keys - but what I want for the keybindings is to be able to bind to commands (specified by Commands.ts right now, as well as functions.
Also, I'd like for our keybinding-story to be able to address all the switching / nesting / hardcoding in the keydown handler in NeovimEditor.tsx. This change gets rid of the key hardcoding, but we still need all the special cases for different pieces of UI being opened.
Ideally, this whole block could be replaced by something like:
const keyboard = new Keyboard()
keyboard.on("keydown", (key: string) => {
    const wasKeyHandled = inputManager.handleKey(key)

    // If the key couldn't be handled in the Oni layer, pass it to Neovim
    if (!wasKeyHandled) {
         this._neovimInstance.input(key);
    }
})

So there's really 4 things I'd like to see from the keymapping work:

Configurable - ability to specify via some configuration (either config.js or keybindings.js).
Multiple mappings per key - We should map key -> action (whether it's a command string, or a function). We might have multiple mappings for the same key, that would be dependent on the current mode, for example. This couldn't be expressed in the current model.
Handling commands AND functions - I want to handle both lambdas as well as existing commands defined in Commands.ts. It might mean that we have to port over some of the functionality, like menu next, menu previous, etc to commands, and have a way for the UI elements to listen for these commands.  I believe that we could reuse the commands registered as opposed to defining new settings.
Generalize all the special-casing in NeovimEditor.tsx - Some commands will have different behavior depending on the edit mode (normal, insert, visual, etc), or based on UI that is available (like, 'is a menu visible'). We should have a way to handle those cases. If we address that, than the key handling in NeovimEditor can just defer that to the input manager to resolve. This will guarantee that our key-handling is flexible enough to support new and novel elements.

The concern I have is that, if these 4 bullet points aren't meant, and we introduce a short-term paradigm, it will actually incur complexity / debt - because now we have to maintain backwards compatibility and also help our users to figure out two different paradigms. So my preference would be to have a holistic solution, as opposed to a short-term solution. The proposed solution solves the first bullet point, but not the subsequent 3.
I'd like to see something that addresses all 4 of the points above.
I'll try and see if I can get time this weekend to put something more concrete together in terms of what I'm thinking, since some of the thoughts above might be too vague to be actionable.  I know the  lack-of-configurability with the input is a huge pain point right now, so I'm definitely on-board with solving it!

Ideally, this whole block could be replaced by something like:

keyboard.on("keydown", (key: string) => {
    const wasKeyHandled = inputManager.handleKey(key)

    // If the key couldn't be handled in the Oni layer, pass it to Neovim
    if (!wasKeyHandled) {
         this._neovimInstance.input(key);
    }
})

I like the above example.

Generalize all the special-casing in NeovimEditor.tsx - Some commands will have different behavior depending on the edit mode (normal, insert, visual, etc), or based on UI that is available (like, 'is a menu visible'). We should have a way to handle those cases. If we address that, than the key handling in NeovimEditor can just defer that to the input manager to resolve. This will guarantee that our key-handling is flexible enough to support new and novel elements.

// temp solution
"keybindings.openedMenu.nextMenuItem"
// more specific solution
{
   "keybindings.openedMenu": {
         "nextMenuItem": ["C-n", "c-j", "<down>"]
     }
}

Also noticed i used c-j instead of C-n, we need to convert all c to C so we can still handle them because if it's c we ignore, and the user doesn't care about case or at least I don't want to.

The concern I have is that, if these 4 bullet points aren't meant, and we introduce a short-term paradigm, it will actually incur complexity / debt - because now we have to maintain backwards compatibility and also help our users to figure out two different paradigms. So my preference would be to have a holistic solution, as opposed to a short-term solution. The proposed solution solves the first bullet point, but not the subsequent 3.

There is no way I would want to merge this right now, I just wanted to put a base review so we could discuss it much like you did :).
I'm kind of new to github and I'm not sure exactly how we should discuss things... so I open prs and issues... is there a better way of doing it?

Or we could do plugin the actual command or lambda

There is no way I would want to merge this right now, I just wanted to put a base review so we could discuss it much like you did :).

I'm kind of new to github and I'm not sure exactly how we should discuss things... so I open prs and issues... is there a better way of doing it?

Oh yes, this is great! I don't think there is a better way to start a discussion than with some code 😄  It gets us out of talking in the abstract and into the real meat of the issue.
Sorry if it came across as negative / critical - didn't mean it like that at all. I appreciate the time, thought, and expertise you put in (after all, you're taking time out of your day to contribute!). So I try and get concerns out early, out of respect for your time! But ya I still feel relatively new to this too, so lots of learning for me - I'm still finding the right balance there.  😄 Thanks for being open to the considerations!

If you could outline the specifics of what we need and build tasks off of a story that way you could define exactly what to complete would be great. I have a general idea and your bullets are a GREAT start.

Sounds good! I know it's vague right now but I wanted to at least set up what the "north star" looks like with those bullet points. I'll post up some more concrete ideas soon.
Thanks again for thinking about this, and getting the conversation started. It will be a huge step forward for this project to have this solved 👍

Sorry if it came across as negative / critical - didn't mean it like that at all. I appreciate the time, thought, and energy you put in (after all, you're taking time out of your day to contribute!). So I try and get concerns out early, out of respect for your time! But ya I still feel relatively new to this too, so lots of learning for me - I'm still finding the right balance there.  :smile: Thanks for being open to the considerations!

It wasn't negative at all, thanks for the feedback, I just want to make sure I'm doing the best I can.
The quick feedback is always appreciated, I would love to help Oni get to release 1.0 as quick, but as good/correct as possible. Not only for my own editing but I want to make an text editor/IDE I believe in.

Sounds good! I know it's vague right now but I wanted to at least set up what the "north star" looks like with those bullet points. I'll post up some more concrete ideas soon.

Less that it's vague more of I want to manage it properly, maybe a check list?

The quick feedback is always appreciated, I would love to help Oni get to release 1.0 as quick, but as good/correct as possible. Not only for my own editing but I want to make an text editor/IDE I believe in.

Cheers to that! Really appreciate your help.

Less that it's vague more of I want to manage it properly, maybe a check list?

Ah ya, github has a check list feature, I just always forget to use it.. That's a good idea - I'll work on breaking down the thoughts I have into an actionable list.

Ah ya, github has a check list feature, I just always forget to use it.. That's a good idea - I'll work on breaking down the thoughts I have into an actionable list.

That would be nice but also thinking of something like trello.

Putting this on hold until #633 is processed.

Looks like this is no longer applicable since #633 is in, and we now have colors customization with #950 -  so I'll close it out. Thanks for the thoughts around this!


Support custom incorrect messages in basic_agent

This PR contains:

[x] New features
[ ] Changes to dev-tools e.g. CI config / github tooling
[ ] Docs
[ ] Bug fixes
[ ] Code refactor

What is the current behavior? (You can also link to an open issue here)
When using the basic_agent solver, if the model submits an incorrect answer, the user can only return a pre-defined static string (such as Your answer was incorrect.)
What is the new behavior?
When using the basic_agent solver, if the model submits an incorrect answer, the user can return a custom string based on the TaskState and Score. This allows the user to incorporate a Score's explanation, or other state information, into feedback for the model. One use case for this: if the model writes some code, and the scorer executes that code and finds that it raises an exception, the exception message can be passed back to the model.
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
No
Other information:
See conversation on Slack here: https://inspectcommunity.slack.com/archives/C080K7SQ4SG/p1731605753325889

Thank you @jjallaire for the extremely quick review and for your helpful advice on Slack! :)

FYI @samiranne made a couple of tweaks here (mostly just supporting a list of scores rather than a single score): https://github.com/UKGovernmentBEIS/inspect_ai/pull/850


Incompatible with AGP 8

Hello!
AGP 8 is now recommended for stable versions of android studio, however, with AGP 8, this plugin causes issues because it relies on gradle-api:7.4.2.
For example:
Unresolved reference: generateLocaleConfig

This is a feature dependent on AGP 8+ that our project relies on.

https://github.com/LSPosed/LSParanoid/commit/90606653dcf7c9bc950f4d2faa2e4ffb3092e7bf


Make load functions compatible with pathlib.Path

Hi !
So far vedo loaders work with filenames as strings, I usually use  pathlib.Path to manage file locations. The current implementation is not compatible as some functions specific to strings are called. What do you think about making it compatible with pathlib.Path ?  An easy way would be to convert pathlib.Path to string and let the rest of the loaders as before, something like that should work:
import os
from pathlib impoer Path

def load(filename: str | os.PathLike, ...):

   is isinstance(filename, Path):
       filename = str(filename)

Thanks,
Louis

Thanks Louis! I'm very open to it, please feel free to open a PR if you wish!


chore: Removing copying data from dictionary values into CometDictionary

Which issue does this PR close?

Closes #.
Rationale for this change

What changes are included in this PR?

How are these changes tested?

cc @sunchao

Merged. Thanks @sunchao @andygrove


switcher_aio error in event call

hello,
For the last days i'm getting an error when trying to events in the switcher, i can read the status but cant set.
2018-08-22 12:51:15 ERROR (MainThread) [homeassistant.core] Error executing service <ServiceCall switcher_aio.turn_on (c:87d0f6db676f4543bb38a681cbc0e2d6): entity_id=['switcher_aio.control_device_switch']>
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/site-packages/homeassistant/core.py", line 1117, in _event_to_service_call
    await service_handler.func(service_call)
  File "/config/custom_components/switcher_aio/__init__.py", line 1007, in async_switcher_control
    _LOGGER.debug("received: " + service.service + " with call id: " + service.call_id)
AttributeError: 'ServiceCall' object has no attribute 'call_id'

there are some other people that getting that error also, see in that thread:
https://community.home-assistant.io/t/switcher-v2-all-in-one-full-control/50652 #

Happens to me as well, would be useful to have it fixed

anyone solve that issue?
I will be glad to know :)

sorry for the delayed response, the issue was actually fixed int the following commit


[BUG?] MagicCitizensTrait cannot use Fury spell

I'm not certain if this classifies as a bug or simply a limitation in how NPCs work.
When I set up a Citizens NPC with the Magic trait and set the spell to fury, the spell does initiate casting and the first effects do play, however no targets seem to be picked from there.
The spell simply vanishes instantly and nothing further happens.
What could be the cause behind that and is there any way I can modify the fury spell to work or create a spell similar to fury that does target enemies when cast from an NPC? If this is a bug or limitation on NPCs end that can be fixed, it stands to reason that similarly functioning spells will also benefit from it.

I think I found the cause, though it is a bit awkward, it seems the spell needs to specify that target_game_modes includes creative as this seems to be the gamemode the npc is standard in and AreaOfEffect attempts to target the npc itself.
Moreover, the fury spell requires the npc to not auto-target the player, for which there was already an option available, I will have to get my own plugin to override that paramater on demand too.


Changes to the weekly-summaries data files

Two more changes requested for the visualization data

[ ] add horizon column to map_data.csv
[ ] sort data in forecast.csv alphabetically with US on top

Some notes from implementing this. Rscript get_forecast_data.R --reference_date 2024-12-21 --base_hub_path ../ --horizons_to_include 0 1 2
One
Warning messages:
1: In readLines(file, warn = readLines.warn) :
  incomplete final line found on '../model-metadata/JHU_CSSE-CSSE_Ensemble.yml'
2: In readLines(file, warn = readLines.warn) :
  incomplete final line found on '../model-metadata/MOBS-GLEAM_COVID.yml'
3: In readLines(file, warn = readLines.warn) :
  incomplete final line found on '../model-metadata/NEU_ISI-AdaptiveEnsemble.yml'

Two No exclusions for reference date: 2024-12-21...the excluded_locations.toml file needs to be updated for each reference dates...the only contents of the excluded_locations.toml file seem to be:
2024-11-23 = [
  "02",
  "04",
  "05",
  "06",
  "08",
  "10",
  "11",
  "12",
  "13",
  "17",
  "18",
  "19",
  "20",
  "21",
  "22",
  "23",
  "24",
  "25",
  "26",
  "27",
  "28",
  "29",
  "30",
  "31",
  "32",
  "33",
  "34",
  "36",
  "37",
  "39",
  "40",
  "42",
  "48",
  "49",
  "51",
  "53",
  "55",
  "56",
  "72",
  "US"
]

Want me to change this? i.e. copy these locations across all reference dates and then regenerate each of the files?

Re https://github.com/CDCgov/covid19-forecast-hub/issues/207#issuecomment-2557225366
One:
I believe these are resulting from the way meta-data files are formatted. I do not want to change anything here since these are submitted by the teams themselves. I am happy to just ignore these warnings until we can't.
Two:
The exclusions for 2024-11-23 exists due to a data reporting issue that affected the indicated locations. We do not expect exclusions for other dates unless something goes wrong again (let's hope not).

This issue also covers, in get_forecast_data.R, renaming "United States" to "US" in the location_name column. This procedure is currently in place for get_covid_hosp_data.R and get_map_data.R.


Can cargo kani support --features option?

I'd like to analyse a crate with a particular set of its non-default features enabled.
Can "cargo kani" support the "--features" option in the same style as "cargo build" please?

Yes, we can. Thanks for the request!

Looks like we also need --no-default-features


CL Hive Integration

Integrate hive tests into Lodestar, references:

Consensus Clients Hive Integration
https://github.com/ethereum/hive/pull/569

Lodestar successfully runs in Hive tests for withdrawals


Feat: multi-targets support

Command options "target" can accept more than one targets now
For files uploading, will pick first target as main target to do uploading, and then copy files from main target to other targets
For files deletion, will delete from targets one by one
All metadadata files updating will happened sequentially
All s3 operations will happen asynchronously

I'll merge this first. If any problems found later then will do fix.


I can't seem to login to Azure using the Logic Apps (Consumption) Extension

I have installed Azure Logic Apps (Consumption) extension for VS Code v1.2.9.  When I click on the Azure button and then 'Sign in to Azure...' the sign in page pops up in the browser and I sign in (the browser says 'You can close this page') but nothing appears to happen in VS code and it still says 'Sign in to Azure...'

I am then unable to connect to existing Logic Apps or create new ones.
Any ideas what I'm doing wrong?
thanks

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

ID: 47be9e62-a0a7-b842-ab8d-e3611c3ea75a
Version Independent ID: 15fb316f-4166-dc73-e4bb-31dab82903fd
Content: Quickstart - Create integration workflows with Azure Logic Apps in Visual Studio Code - Azure Logic Apps
Content Source: articles/logic-apps/quickstart-create-logic-apps-visual-studio-code.md
Service: logic-apps
GitHub Login: @ecfan
Microsoft Alias: estfan

@mike-urnun-msft
Thanks for your feedback! We will investigate and update as appropriate.

Interestingly when I logged in today (to show a colleague that it didn't work) it now does work.  Not sure what the issue was but I can see the LAs now.


Allow users to manage which notifications they receive.

Placeholder for a user request. Not a top priority at the moment.

Collection manager and reviewer options have been implemented. At this time, I think it's important for depositors to get all the messages, so there is no further work to do here.


Publishing latest (svelte) package

Hi,
First of all, I just want to thank you for your work.

This is more of a question than an 'issue';
I'm wondering when the latest version of @nxext/svelte is being published (to npmjs)?
As I'm waiting for #110 to become available for my side-project 😄
Since the other packages (@nxext/stencil and now @nxext/tailwind) were updated this week, I was hoping @nxext/svelte would arrive soon.

Thanks again for your time and effort!

Hi,
sorry about that. You're right, I stumbled on another bug but now I released a patch version with that changes in place as well.
Please try version 10.1.3 :-)

No worries.
Just tested the latest version, and my app now builds correctly 🙂
Cheers!


make get_build generic and safety

get_build was creating new Build instance with path.empty() = true  by using its default constructor . if we  use path.empty() for new CMakeBuild instance  we will get the same result and don't need to mess with type hierarchy(lCMake and Project::CMake was different classes., Project::Build and Project::CMake castings are dangerous too).   It will be great to provide the same functionality for MakeBuild, SconsBuild and so on in the future. And all internals in make.h, scons.h  the same way as cmake.h.

Thank you, but the point of Project::get_build is to deduce the build management system when other management systems than CMake gets supported. The build might not be C/C++ related either. One do not want to specify build management system type when calling Project::get_build. For instance:
https://github.com/cppit/jucipp/pull/203/files#diff-00b5229d483ad0c01de611ff630723beL175
is better than
https://github.com/cppit/jucipp/pull/203/files#diff-00b5229d483ad0c01de611ff630723beR175
By the way, what do you mean by "Project::Build and Project::CMake castings are dangerous"?
I agree that Project::CMake should be called Project::CMakeBuild. And I guess Project::Language's subclasses should be renamed similarly too.

I took a look at previous pull request. And It looks like the line:
return std::unique_ptr<Project::Build>(new Project::Build());

is not solve the problem but make code complicated.
If you decide to add new build tool, you will need to rewrite a lot of code in progect.cc and source.cc for example.So it does make this function "generic", but make complicated other stuff. If I would make a decision I would make something like this:
template <typename T>
class Build<T>

where T is a build tool for example CMakeBuild and we will need template specialization only for creating a Build instance.
auto cmake_build = std::make_unique<Build<CMakeBuild> >;

the benefit is that it will force to implement the same interfaces for new build tool or error at compile time. But even this decision I am not consider like the best. There is not a lot of  "build tools" (make, ninja, cmake, autotools, scons). So I would just make Build class as abstract(interface) class and don't complicate code with templates at this level (no fancy compile errors). Build class should be the interface for build tools and nothing more. But this is all IMHO. But this is all IMHO. I can be mistaken.

To support build management systems other than CMake the main changes would be to the src/project_build.* files. The CMake reference in Project::get_language is only currently to identify clang projects. Furthermore, one might want add something in Project::on_save as well to automate the build further, like what is done when saving CMakeLists.txt. There are no references to Project::CMake in src/source.cc.
Additionally, one would have to update Project::get_build in order to identify the build management system in the given directory structure, and return the correct Project::Build subclass object (of for instance the types Project::CMakeBuild, Project::AutotoolsBuild or Project::SconsBuild). It has to be subclass object, since it must be identified at runtime.
Ninja or make files are currently created in the build directory based on the cmake parameters. This would be the same if autotools or scons for instance would be supported I guess, although I'm not very familiar with these tools.

Oh yes. I understood an idea, I don't understand your approach. I just trying to say that you can avoid run-time polymorphism, just make Project::Build "pure" abstract class. I mean you don't need to iterate through build tools for example, or support many build tools in one project. Just keep code as simple as possible.
And this should be enough:
if(path.filename()=="CMakeLists.txt")
    auto build = std::make_unique <CMakeBuild>*(path);
else if((path.filename()=="SConstruct")
    auto build = std::make_unique <SConsBuild>*(path);

and add one method to check which build tool is using, if the logic is not generic.
auto build_type = build.type();

IMHO it should be much simpler, and less error prone in the future.
For example right now if get_build() returns base class instance Build() but not CMake() you do all these checks anyway.


improve boosted print mode

When printing some boosted component, background color and icons disapear.
Need to identify why and try to improve it
Boostwatch · Boosted.pdf

Not printing background is default behaviour for browsers.
There is no cross-browser solution and fixing this can get hacky.
Easiest way is to configure the browser print options (here in Firefox) :

Boostwatch · Boosted.pdf

Yes quite okay for Chrome, but here are the main problem on Firefox59 and IE11, not all the page was printed only 1 page.
Maybe need to check if it's related to a browser bug.

Hello @Lausselloic I looked around on the internet and I found out that the issue you raise, happens only on Firefox and IE11.
As mentioned here, for firefox, the reason is that developers focus on a proper screen rendering than proper pagination for print out.
Concerning IE11, there is a lot of solutions but none of them have worked on my computer.
This is one of hte most shared fixed.


Broken links

Hi, there are a few broken links:

[ ] http://jxnblk.com/colorable/ (file: README.md, status code: 404)
[ ] http://www.pictaculous.com/ (file: README.md, error: ENOTFOUND)
[ ] http://jxnblk.com/shade/ (file: README.md, status code: 404)
[ ] http://www.checkmycolours.com/ (file: README.md, error: ENOTFOUND)
[ ] http://www.dasplankton.de/ContrastA/ (file: README.md, status code: 500)
[ ] http://colorsupplyyy.com/app/ (file: README.md, error: EAI_AGAIN)
[ ] http://www.colourlovers.com/ (file: README.md, error: ECONNRESET)
[ ] http://www.lolcolors.com/palettes/popular (file: README.md, status code: 404)
[ ] http://color.farm/ (file: README.md, error: ECONNRESET)
[ ] https://blog.graphiq.com/finding-the-right-color-palettes-for-data-visualizations-fcd4e707a283#.k1zjxtfet (file: README.md, error: ENOTFOUND)
[ ] https://medium.com/life-tips/the-unfinished-basics-of-colour-292858f62e62#.b1z1ejmsg (file: README.md, status code: 410)
[ ]<EMAIL_ADDRESS>(file: README.md, status code: 410)
[ ] https://journal.helabs.com/oiling-the-color-wheel-with-jade-sass-5688ceada87c#.frc7e0rj5 (file: README.md, error: ENOTFOUND)
[ ] http://www.tigercolor.com/color-lab/color-theory/color-theory-intro.htm (file: README.md, error: ENOTFOUND)
[ ] http://www.ifweassume.com/2013/05/cubehelix-or-how-i-learned-to-love.html (file: README.md, status code: 404)

Hi @ndsvw, thanks for pointing this out!
Do you want to raise a PR to fix these links?


docs: get started with styling page

This PR adds a new page to Get Started called "Styling the table body", which covers the basics of styling with loc.body().
Note that I tried to structure Get Started a bit more, by splitting it into Table Structure, Format and Style, and Extra Topics for now:

Happy to discuss / change it to anything useful! It seems like once they're implemented we could add additional pages, like:

footnotes to the Table Structure section
styling other locations (like header) to the Format and Style section

TODO:

Should titles use title case, or just capitalize the first letter?

I'm think using 'capitalize first letter' is not a bad way to go!

More specifically, I like the way it's done in the quartodoc pages, where the main headings are in title case and the subheadings start with a capital letter.


Eeprom v5

v5
Added sol_str_slice_to_string and use it.
sched_yield(); on sysfs busy waiting.
Fix some casts.
v4
No more initialising statics with compound literals, so gcc < 5 can be happy.
v3:
To avoid simplify i2c paths, now keeping a 'shadow' map with resolved path. To avoid duplicating entries, sol_memmap_map structure was changed to have entries as a pointer to an array. So, generator and conffile were changed accordingly.
v2:
Keep generated memory mapped map const.
No more internal data on user available struct: keeping an internal array with resolved i2c paths.
After some tests with memmap persistence on Calamari EEPROM, some patches:
int/accumulator: Now with an option to send initial packet and a port to set its initial value (useful to use an accumulator with persisted data).
persistence: Saving ending NUL on strings. Memmap has a fixed size for string fields, so we need to write terminating NUL byte to avoid issues on overwrite.
memmap-storage: path property now accepts create,i2c,... commands (the same from IIO) to create an I2C device - useful for Calamari EEPROM.
And finally, a sample was added to show persistence on Calamari.

+1

Other than those, +1.

Integrated with fixes to issues pointed by @glima. Tks!


support dryrun mode in pack command

allow cmf pack to run without creating the final zip file

this should be useful to run in PR Builds


Assert conn_pool_stream_handle_ is nullptr

Ensures we never reach an invalid state.
Somewhat related to #13737.
Signed-off-by: Raul Gutierrez Segales<EMAIL_ADDRESS>

(Possibly not ready for review yet, need to ensure integ is happy).

OS X timed out building:
https://dev.azure.com/cncf/envoy/_build/results?buildId=55383&view=logs&j=fa3d3e18-6969-5713-c3e7-3581195704fd&t=ba370bef-ca49-52f5-0c93-7c0c0f27c465


Fallback more URL cases

Fallback URLs:

Host starts www or not.
Path is lower or upper.
Start with http or https.
End with / or .txt or .php or .html.

This addresses a few challenges we are running into - possible to get this merged?

May handled by #316.

Don't believe that #316 addresses the same issues.  As I understand it #316 is about downloading/transforming the SPDX license list; that list doesn't include the multitude of aliases/typos for license URLs, which is what this issue is addressing.


A new method for the greedy optimizer

The following changes have been made in this PR:

Added a new optimizer "Hyper-Greedy". In each step, instead of directly selecting the contraction pair with the lowest cost, here we sampling one of them according to the cost $\exp(-L / t)$, where $L$ is the cost and $t$ is the temperature, thus it may given better results that greedy.
Deps of BetterExp is removed, since it has been merge to Base since<EMAIL_ADDRESS>and currently exp2 provided by Base seems better:

julia> @btime exp2(1.0);
  0.791 ns (0 allocations: 0 bytes)

julia> @btime BetterExp.exp2(1.0);
  1.625 ns (0 allocations: 0 bytes)

(betterexp) pkg> st
Status `~/temp/betterexp/Project.toml`
  [6e4b80f9] BenchmarkTools v1.5.0
  [7cffe744] BetterExp v0.1.0

Added multi-threads support for tree_greedy.

Added a new struct GreedyStrategy for method of GreedyMethod.
GreedyStrategy include parameters $\alpha$ and temperature, thus MinSpaceOut and MinSpaceDiff now are special case of GreedyStrategy.

These commented parts have been removed, and the doc strings are revised.


Fix screenshot directory path

@jlipps please review. This hardcoded path doesn't help run this project on any other machine.

published as 3.0.2


ghost: init at 8.0.0

Motivation for this change
Ghost Framework is an Android post-exploitation framework that exploits the Android Debug Bridge to remotely access an Android device.
https://github.com/EntySec/ghost
Related to #81418
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[x] Fits CONTRIBUTING.md.


Added Markdown formatting to examples/mnist_mlp.py

Summary
Added Markdown formatting to examples/mnist_mlp.py.
Result:

Related Issues
#12219
PR Overview

[ ] This PR requires new unit tests [y/n] (make sure tests are included)
[ ] This PR requires to update the documentation [y/n] (make sure the docs are up-to-date)
[x] This PR is backwards compatible [y/n]
[ ] This PR changes the current API [y/n] (all API changes need to be approved by fchollet)

Thanks for the PR.
We are postponing the addition of new examples on keras.io pending a future refactor of the docs website (which will have a dedicated examples section).


Progress window on long operation

At first, thank you for awesome extension!
This PR solves next problem:
When I trying implement interfaces not from stdlib, execution of impl takes very long time (up to 5-10 seconds). While it running for me as user not clear: is it running or not started? So having a progress window reassures me that the process is underway and there will be a result soon.

Hi @neonxp. Thank you for your contribution. The progress window will be added in the new release.


conformance: fix cross-mount behavior when 'from' is missing

fixes issue #442
Signed-off-by: Ramkumar Chinchani<EMAIL_ADDRESS>
What type of PR is this?
bug

Which issue does this PR fix:
issue #442
What does this PR do / Why do we need it:
'from' query param is not support, so conform per spec.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Ran 60 of 65 Specs in 0.147 seconds
SUCCESS! -- 60 Passed | 0 Failed | 0 Pending | 5 Skipped
You're using deprecated Ginkgo functionality:
=============================================
Ginkgo 2.0 is under active development and will introduce (a small number of) breaking changes.
To learn more, view the migration guide at https://github.com/onsi/ginkgo/blob/v2/docs/MIGRATING_TO_V2.md
To comment, chime in at https://github.com/onsi/ginkgo/issues/711

  You are using a custom reporter.  Support for custom reporters will likely be removed in V2.  Most users were using them to generate junit or teamcity reports and this functionality will be merged into the core reporter.  In addition, Ginkgo 2.0 will support emitting a JSON-formatted report that users can then manipulate to generate custom reports.

  If this change will be impactful to you please leave a comment on https://github.com/onsi/ginkgo/issues/711
  Learn more at: https://github.com/onsi/ginkgo/blob/v2/docs/MIGRATING_TO_V2.md#removed-custom-reporters

To silence deprecations that can be silenced set the following environment variable:
  ACK_GINKGO_DEPRECATIONS=1.16.4

PASS


Analyze text with the Text Analytics Service

Module:  week1
Lab/Demo: 1
Task: 1b
Step: 03
Description of issue
getting this error when compiled "speaking-clock.ps1"
13 |  $result = Invoke-RestMethod -Method Post `
|            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| Name or service not known
Repro steps:

Hello @ParthasarathiReddy6621
I have tried with both a Speech service and a Cognitive Services resources, I ran through all the steps and I wasn´t able to reproduce your error.
The error message “Name or service not known” typically occurs when the Invoke-RestMethod command is unable to resolve the hostname specified in the URL. This could be due Proxy issues (if you are behind a proxy you will need to update  PowerShell´s proxy settings) or there is a connectivity issue with your environment (check Virtual network settings, security groups and firewall , so that they don´t block the URL), or even a temporary network issue.
I would suggest trying running this command in Powershell to delete the folder : rm -r ai-900 -f, clone again the repo (to make sure there are no issues with the speaking-clock file) , create a resource , update the key and region and try again. Also, makesure you have a stable network connection without any firewalls or security group rules that might block internet access.

@Mihai-Ac, thanks so much for the response!
@ParthasarathiReddy6621, if you are still having trouble, one of the common errors with speech is that you need to enter the location (for example, eastus) alongside your key. You could be getting an error because you are trying to use your key and endpoint.

Additionally, you will want to make sure the location is copied and pasted exactly as it is displayed in Azure:

Hope that helps!


simplify cloud deployment

How could we simplify cloud deployment of gdsfactory?
For example, we could add https://github.com/proppy/rad-lab/tree/tuning/radlab-launcher
@proppy
@HelgeGehring
@flaport

I should already part of it, see: https://github.com/proppy/rad-lab/blob/tuning/modules/silicon_design/scripts/build/images/provision/environment.yml#L33

nice!
How about including a radlab documentation entry on the gdsfactory docs?

@joamatab following up on the "one click"'ish deployment idea, I think it would possible to leverage https://github.com/marketplace/google-cloud-build with the https://github.com/proppy/rad-lab-deploy repo; rather than the rad-lab UI/CLI.
Do you want to give it a try?


zone aware balancing question

If i wanted to configure envoy to be zone aware, is it correct that i would need a rotor process in each zone?  or can i use a single rotor instance.  thanks!

Yes, rotor can only serve a single envoy configuration. You'll need to modify ROTOR_XDS_STANDALONE_ZONE for each zone, and you'll need to pass static cluster configs to properly configure the clusters (endpoints can remain dynamic via EDS)

@9len Thanks is this still true if looked into using Houston?

cool. closing this for now, let me know if you have any other questions, or if you're interested in talking about Houston more.


Pan/Horizontal Scroll

If I create a graph similar to the "Total of Steps" graph here
https://camo.githubusercontent.com/0858db57496e5802ed511310896b44430b9ac551/687474703a2f2f692e696d6775722e636f6d2f725149683371442e676966
does it support panning or horizontal scroll if the entire graph doesn't fit the width of the screen?

Sorry for late response, missed your question. Macaw doesn't provide any additional scroll capabilities, however MacawView is a regular UIView, so you can use standard iOS technics for scrolling.


Add support for Ipv6 network/binds

This PR implements a IpNetwork configuration option, to choose between a underlying Ipv4 or Ipv6 network.
The network parameter are used for:
a) granting addresses either from a <IP_ADDRESS>/16 address space, or from fe80:::/64
b) binding sockets to associated unspecified addresses <IP_ADDRESS> / :::
The default configuration stays an Ipv4 network with <IP_ADDRESS>/16.

Thanks for submitting the PR, and sorry for the delayed response. I'm curious to hear a bit more about why you need this feature for your simulation needs.

No problem. We were trying to develop some units tests for Axum-based APIs in a system, that for the most part does only support Ipv6 addressing. Changing the addressing either to Ipv4Addr or to IpAddr would be a lot of work and may result in changes in the systems behavior. In short, the codebase is quite large, so we would like to use the least intrusive method to simulate it, which would be Ipv6 support on sockets.


question?

require the scala version is above 2.11.xx ,   what if 2.10.4 ? can this tool successfully running?

No, this only works for 2.11 / 2.12 now I'm afraid.


[Refactor][doc]Unify the switch values of unconventional compilation parameters in the official documentation of the old version

Unify the switch values of unconventional compilation parameters in the official documentation of the old version

Proposed changes
Issue Number: close #xxx
Problem Summary:
Describe the overview of changes.
Checklist(Required)

Does it affect the original behavior: (Yes/No/I Don't know)
Has unit tests been added: (Yes/No/No Need)
Has document been added or modified: (Yes/No/No Need)
Does it need to update dependencies: (Yes/No)
Are there any changes that cannot be rolled back: (Yes/No)

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

@FreeOnePlus please rebase the master to pass the check

Hi @FreeOnePlus , sorry that we have just refactor the document #9272
So you need to rebase the master and apply you changes to the new document


[Bug] The Beast Fur Shawl and ES Regen via Zealot's Oath

The Beast Fur Shawl's ES Recovery Rate (and also Trickster Patient Reaper) should be applied to ES Regen gained from Zealot's Oath.

The stat from The Beast Fur Shawl seems to be working for me; although during testing I did discover that the ES regen percentage in the Calcs tab wasn't displaying the correct value when an ES Recovery modifier was present. The flat regen amount, as displayed there and in the sidebar, is correct, though.
As for Patient Reaper, I haven't added support for the Recovery portion of that passive yet. It should be easy though.

Ah I didn't calculate the actual number just noticed the % didn't change when toggling Zealot's Oath on and off.  And that Beast was was not listed when hovering over the regen.  Calculating the % manually it does show the correct regen, so just a % display bug.  Thanks for the great tool!

Patient Reaper will be properly supported as of the next update, so I think it's safe to close this.


Anyway to configure IPv4/6 settings for crouton's apt usage?

I've had trouble across all of my Linux machines (laptop, desktop, etc.) and all of my Linux distros (Mint, Ubuntu, Elementary, Chrome, etc.) when using apt. They all hang for 5-10 minutes when making the initial connection to the standard ubuntu package archives. Crouton attempts to reconnect to the archives a dozen+ times during various stages of chroot installation. This means that every chroot that I want to create will take 2-3 hours to complete (with Chrome constantly trying to go to sleep, so it needs constant attention).
I've gotten around this problem by changing /etc/gai.conf in all of my Linux installs so that IPv4 is used instead of IPv6. Is there any way to modify crouton to do the same?

@PastyPilgrim,
You meant the way of disabling IPv6?
The top-rated (temporary) method didn't work properly?
sysctl net.ipv6.conf.all.disable_ipv6 = 1
sysctl net.ipv6.conf.default.disable_ipv6 = 1

Regards.

I'm sorry, I haven't seen this method. Where are these changes being made?

@PastyPilgrim,
Run those with sudo inside the chroot (but it's temporal way so reboot could reset this).
Regards.

Wouldn't that just change the settings for that chroot? I want crouton to use IPv4 when installing chroots, I've already gotten my installs using IPv4 by editing gai.conf.

@PastyPilgrim,
Means "disabling IPv6 during crouton install"?
OK. those would work on crosh shell as well.
Regards.

That does the trick, thanks.


faiss::FaissException when using vss_search with better-sqlite3 on Node (WSL)

Hi Alex, awesome job with these sqlite extensions. I'm trying to get up and running with sqlite-vss and better-sqlite3 on Node, in Ubuntu using WSL.
I have followed the installation instructions and have set up a simple test script here:
https://github.com/larryhudson/testing-sqlite-vss/blob/main/index.js
I think I am storing the vectors and setting up the virtual table properly, but I am running into a faiss::FaissException error when I get to the vss_search stage:
terminate called after throwing an instance of 'faiss::FaissException'
  what():  Error in virtual void faiss::IndexFlat::search(faiss::idx_t, const float*, faiss::idx_t, float*, faiss::idx_t*, const faiss::SearchParameters*) const at /home/runner/work/sqlite-vss/sqlite-vss/vendor/faiss/faiss/IndexFlat.cpp:34: Error: 'k > 0' failed
Aborted

Could you please check out the test script and let me know if I am doing something wrong? Thanks again.

Just tried reproducing the error on macOS (M1) and I'm getting the same error, so I don't think it's a WSL error. Must be something wrong with how I'm setting up the tables or storing the vectors.

Ah great, that has fixed it! I missed that instruction when I was going through the Observable article.
Thanks for the quick response :)

And it looks like I'm running SQLite version 3.39.5 on macOS, by the way.

Glad to hear it got resolved!
One note, since you're on 3.39.5 (which contains a bug that was fixed in 3.41+, you'll have to use the vss_search(text_embedding, vss_search_params(?, 20)) syntax instead of vss_search(text_embedding, ?) limit 20. There's some info about this in the docs, which I should also clean up a little...


Mutliple tensors dataloader

Allow passing of multiple tensors in DataSplit and Folds
Make changes to path construction for networks (return as path and features)

Codecov Report

Merging #49 (dbf6852) into develop (4f20c9f) will increase coverage by 1.19%.
Report is 1 commits behind head on develop.
The diff coverage is 73.86%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@             Coverage Diff             @@
##           develop      #49      +/-   ##
===========================================
+ Coverage    55.85%   57.05%   +1.19%
===========================================
  Files            8        8
  Lines          990     1006      +16
===========================================
+ Hits           553      574      +21
+ Misses         437      432       -5

Files Changed
Coverage Δ

src/nlpsig/data_preparation.py
75.16% <0.00%> (+1.20%)
:arrow_up:

src/nlpsig/classification_utils.py
94.14% <100.00%> (+0.61%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


where can i get "faces_ms1m_112x112.pickle"

i try the   python train.py
but "faces_ms1m_112x112.pickle" not me
where can i get "faces_ms1m_112x112.pickle"

Please follow the instruction in README, run pre_process.py first.


Changes in Readme.md

Related issue

What changes does this PR do?
Added my user id
Screenshots (If any)
Questions

@vielmetti  can you please merge it with the main branch

@schmelto Added above change

@vielmetti  can you please merge it with the main branch


fix(build): output directory actually uses host arch instead of target

I actually found this out on GitHub Actions, where I listed the .build contents when compiling the aarch64-apple-ios target: https://github.com/tauri-apps/tauri/actions/runs/4105459372/jobs/7083446720#step:6:217


Have 4k Display Partially working...

I was able to patch Flex Launcher to support 4k displays. And the images and text are now rendered in HiDPI. However When I launch an emulator, it gets rendered at a non-4K resolution.  The child process for instance is 1080p but not 2x the pixel density.
While I have looking great within the Launcher, I don't have the same effect in stuff that is being launched.
geo.screen_height = display_mode.h * 2;
...
SDL_WINDOW_FULLSCREEN_DESKTOP | SDL_WINDOW_MAXIMIZED | SDL_WINDOW_ALLOW_HIGHDPI | SDL_WINDOW_OPENGL```

This keeps the pixel density to the same as my HiDPI settings in the system.
#ifdef __unix__
    SDL_SetHint(SDL_HINT_VIDEO_X11_NET_WM_BYPASS_COMPOSITOR, "1");
    setenv("SDL_VIDEODRIVER", "x11", 0);
#endif
    SDL_SetHint(SDL_HINT_RENDER_SCALE_QUALITY, "3");
    SDL_SetHint(SDL_HINT_VIDEO_ALLOW_SCREENSAVER, config.inhibit_os_screensaver ? "0" : "1");
    if (config.gamepad_enabled)
        sdl_flags |= SDL_INIT_GAMECONTROLLER;

    // Initialize SDL
    if (SDL_Init(sdl_flags) < 0)
        log_fatal("Could not initialize SDL\n%s", SDL_GetError());

    SDL_GetDesktopDisplayMode(0, &display_mode);
    geo.screen_width = display_mode.w;
    geo.screen_height = display_mode.h;
    refresh_period = 1000 / (Uint32) display_mode.refresh_rate;
    geo.screen_margin = (int) (SCREEN_MARGIN * (float) geo.screen_height);
}

// A function to create the window and renderer
static void create_window()
{
    window = SDL_CreateWindow(PROJECT_NAME,
                 SDL_WINDOWPOS_UNDEFINED,
                 SDL_WINDOWPOS_UNDEFINED,
                 0,
                 0,
                 SDL_WINDOW_FULLSCREEN_DESKTOP | SDL_WINDOW_MAXIMIZED | SDL_WINDOW_ALLOW_HIGHDPI | SDL_WINDOW_OPENGL
             );

works for me

Removed the two unix statements which seemed to either conflict with 4k rendering or didn't add anything to the mix.
Added SDL_WINDOW_MAXIMIZED | SDL_WINDOW_ALLOW_HIDPI | SDL_WINDOW_OPENGL
Set SDL_HINT_RENDER_SCALE_QUALITY to "3"


Update MEW button

Add left/right image icons
Add left/right Material Design icons
Adjust side paddings

can you pull in the latest from master and redo this pr since I already added the different icon options ?


Reportback caching returns flagged reportbacks to anon users

BUG
Current Behavior

If an admin user hits a flagged reportback and then an anon user tries to hit that same endpoint, reportback caching will reveal the unauthorized RBs to anon users. This only happens for /reportbacks/:id, /reportback-items/:id works correctly.

Desired Behavior

Flagged/unauthorized RBs should never be returned to an anon user.
In the above situation, a 403 status code and error message Access denied. should be returned when any time an anon user tries to retrieve a flagged reportback.

Steps to Replicate

Sign in as an admin on staging.
Hit /reportbacks/5 endpoint - reportback should appear.
Log out and sign in as an anon user.
Hit the same endpoint and you'll still see the reportback.

Why This Matters

Only authorized users/admins should be able to view flagged reportbacks.

Log out and sign in as an anon user.

Just to note, this would be a auth user, not anon user. Try the same endpoint, but in an incognito window without signing in to be an "anon" user :smile:

ah yes, thanks for catching that Diego! Also a way more efficient way of testing :)

@weerd I actually just tried this with the new code from #6337 and I don't think it is caching! Is it cool to close out this issue once I merge and test to make sure everything is ok with #6337 ?

@weerd confirmed that #6337 works on staging too and the cache doesn't reveal flagged reportbacks/reportback items to non admin users! Going to close this out unless there's another scenario I'm not thinking about with the cache?

hokay!


Optimize list datasets info

This is a lot faster especially in Teams through API. Was basically unusable before.
Do you think anyone relies on / would miss the num_samples part of the info?

Right. A little better than loading every dataset completely (which is like n * 4 db calls), but still not ideal.


Stripping whitespaces as default for QATask ICL eval

Running tests to confirm it doesn't break any other evals.
What does this PR do?
If normalization is off (like in gsm8k), leading/trailing whitespaces can cause mismatch which leads to low accuracy even when the model gets the answer right. Adding a .strip() to be default behavior here.
Before submitting

[x] Have you read the contributor guidelines?
[ ] Is this change a documentation change or typo fix? If so, skip the rest of this checklist.
[ ] Was this change discussed/approved in a GitHub issue first? It is much more likely to be merged if so.
[x] Did you update any related docs and document your change?
[ ] Did you update any related tests and add any new tests related to your change? (see testing)
[ ] Did you run the tests locally to make sure they pass?
[x] Did you run pre-commit on your change? (see the pre-commit section of prerequisites)

Running gauntlet on Llama-2 and OPT to make sure things run smoothly. Track run at llama-opt-gauntlet-eval-H9qpEW. Numbers so far look right to me:
|:-------------------------|---------------:|------------------:|------------------------:|-------------------------:|---------------------------:|------------------------:|
| meta-llama/Llama-2-7b-hf |         0.4165 |          0.510723 |                0.355324 |                 0.656446 |                   0.224974 |                0.335033 |

Please review and merge.
cc @bmosaicml @dakinggg

Should we also strip the answers as well?

@ksreenivasan I feel if we apply strip to the outputs, we should also apply them to the labels. I agree that most processed datasets will not leave whitespace before or after the official answer, but this feels more internally consistent. Other than that I approve


jest not resolve cjs modules as nodejs

🐛 Bug Report

In esm modules the logic of resolving cjs modules is not applied (provided in nodejs by default)
This behavior makes it impossible to use many modules that use other cjs modules. For example, you can't use named import from react in imported module
To Reproduce
Steps to reproduce the behavior:
import.test.js
import {color} from './reexport.cjs'

reexport.cjs
module.exports = require('./export.cjs')

export.cjs
exports.color = 'red'

package.json
{"type": "module"}

run NODE_OPTIONS=--experimental-vm-modules npx jest
Expected behavior

To work correctly with the modules used for nodejs, jest esm modules must resolve cjs files according to the general rules used in nodejs, which means they processed re-export
Link to repl or repo (highly encouraged)

replt.it
envinfo

SyntaxError: The requested module './export.cjs' does not provide an export named 'color'

This works correctly in jest@next. Jest 27 will hopefully be released as stable this month

I just ran in to this, and I see this:
❯ yarn jest tests/integration/node.test.cjs
yarn run v1.22.10
...
  10 files checked.
  testMatch:  - 0 matches
  testPathIgnorePatterns: /node_modules/ - 10 matches
  testRegex: \.test\.c?js - 1 match
Pattern: tests/node.test.cjs - 0 matches
error Command failed with exit code 1.

❯ yarn jest --version
27.0.0-next.7

❯ yarn jest

 PASS  tests/index.test.js
  Component Invocation
    ✓ self-closed (6 ms)
    ✓ block, no args (1 ms)

Test Suites: 1 passed, 1 total
Tests:       2 passed, 2 total
Snapshots:   0 total
Time:        0.198 s, estimated 1 s
Ran all test suites.

❯ ls tests/
index.test.js  node.test.cjs  -utils.js

not related, this issue is about imports inside a file being resolved as ESM imports.
Please note this issue tracker is not a help forum. We recommend using StackOverflow or our discord channel for questions.


There was an error while executing VBoxManage, a CLI used by Vagrant for controlling VirtualBox. The command and stderr is shown below : Error while vagrant up

Please provide the output of the following command:
cat /proc/cpuinfo | grep "svm\|vmx"

For virtualbox to be able to create VMs, the VT-x should be enabled in the BIOS. The above commands lets you check if it is enabled or not.
Also, Is your ubuntu an VM or a Host OS (directly installed on laptop or server)?

I got following while doing vagrant up
cb@satyamz:~/containers/k8s-demo$ vagrant up kubemaster-01
Bringing machine 'kubemaster-01' up with 'virtualbox' provider...
==> kubemaster-01: Importing base box 'openebs/k8s-1.6'...
==> kubemaster-01: Matching MAC address for NAT networking...
==> kubemaster-01: Checking if box 'openebs/k8s-1.6' is up to date...
==> kubemaster-01: Setting the name of the VM: k8s-demo_kubemaster-01_1495104505994_382
==> kubemaster-01: Clearing any previously set network interfaces...
==> kubemaster-01: Preparing network interfaces based on configuration...
    kubemaster-01: Adapter 1: nat
    kubemaster-01: Adapter 2: hostonly
==> kubemaster-01: Forwarding ports...
    kubemaster-01: 22 (guest) => 2222 (host) (adapter 1)
==> kubemaster-01: Running 'pre-boot' VM customizations...
==> kubemaster-01: Booting VM...
There was an error while executing `VBoxManage`, a CLI used by Vagrant
for controlling VirtualBox. The command and stderr is shown below.

Command: ["startvm", "91c21dd6-2613-4a3a-9267-474bfba2aabd", "--type", "headless"]

Stderr: VBoxManage: error: The virtual machine 'k8s-demo_kubemaster-01_1495104505994_382' has terminated unexpectedly during startup with exit code 1 (0x1)
VBoxManage: error: Details: code NS_ERROR_FAILURE (0x80004005), component MachineWrap, interface IMachine

This issue produces when we have multiple versions of VirtualBox are installed on the machine.
OpenEBS needs following :
vagrant  >= 1.9.1
virtualbox >= 5.1

You can resolve this issue by uninstalling old VirtualBox . You can do following :
cb@satyamz:~/containers/k8s-demo$ sudo apt-get remove --purge virtualbox*
cb@satyamz:~/containers/k8s-demo$ sudo rm ~/"VirtualBox VMs" -Rf
cb@satyamz:~/containers/k8s-demo$ sudo rm ~/.config/VirtualBox/ -Rf

Again install VirtualBox 5.1 :
sudo apt-get install virtualbox-5.1

More info on installing VirtualBox can be found here
Once your installation is done, check the versions of VirtualBox and Vagrant.
It should look like :
cb@satyamz:~/containers/k8s-demo$ vboxmanage --version
5.1.22r115126
cb@satyamz:~/containers/k8s-demo$ vagrant --version
Vagrant 1.9.1

Finally do the
cb@satyamz:~/containers$ vagrant up

That's  all !

One of solution which worked for me was shutting down the system & restarting it.It work's.

Any new solutions for this issue in the end of 2018?

I'm using vagrant on EC2 and getting same error above!

I am facing the same issue on Ubuntu 16.04. I tried everything above. I am using latest versions.
virtualbox version 6.0.8
vagrant version 2.2.5
I think I need to downgrade my versions ?
Help ??

I am facing the same issue on Ubuntu 16.04. I tried everything above. I am using latest versions.
virtualbox version 6.0.8
vagrant version 2.2.5
I think I need to downgrade my versions ?
Help ??

Try to run 'sudo /sbin/vboxconfig' command after the installing the newest VMBox. I fixed this issue for vagrant 2.2.5 and VM 6.0.8 (after the 5.2) with ubuntu 16.04

Bonsoir, j'ai le même problème veillez m'aider.
$ vagrant up
Bringing machine 'homestead' up with 'virtualbox' provider...
==> homestead: Checking if box 'laravel/homestead' version '9.4.0' is up to date...
==> homestead: Clearing any previously set forwarded ports...
==> homestead: Clearing any previously set network interfaces...
==> homestead: Preparing network interfaces based on configuration...
homestead: Adapter 1: nat
homestead: Adapter 2: hostonly
==> homestead: Forwarding ports...
homestead: 80 (guest) => 8000 (host) (adapter 1)
homestead: 443 (guest) => 44300 (host) (adapter 1)
homestead: 3306 (guest) => 33060 (host) (adapter 1)
homestead: 4040 (guest) => 4040 (host) (adapter 1)
homestead: 5432 (guest) => 54320 (host) (adapter 1)
homestead: 8025 (guest) => 8025 (host) (adapter 1)
homestead: 9600 (guest) => 9600 (host) (adapter 1)
homestead: 27017 (guest) => 27017 (host) (adapter 1)
homestead: 22 (guest) => 2222 (host) (adapter 1)
==> homestead: Running 'pre-boot' VM customizations...
==> homestead: Booting VM...
There was an error while executing VBoxManage, a CLI used by Vagrant
for controlling VirtualBox. The command and stderr is shown below.
Command: ["startvm", "968c1f09-e707-440e-9c9e-589ae01bd124", "--type", "headless"]
Stderr: VBoxManage.exe: error: Not in a hypervisor partition (HVP=0) (VERR_NEM_NOT_AVAILABLE).
VBoxManage.exe: error: VT-x is not available (VERR_VMX_NO_VMX)
VBoxManage.exe: error: Details: code E_FAIL (0x80004005), component ConsoleWrap, interface IConsole

I am facing the same issue on Ubuntu 16.04. I tried everything above. I am using latest versions.
virtualbox version 6.0.8
vagrant version 2.2.5
I think I need to downgrade my versions ?
Help ??

Try to run 'sudo /sbin/vboxconfig' command after the installing the newest VMBox. I fixed this issue for vagrant 2.2.5 and VM 6.0.8 (after the 5.2) with ubuntu 16.04

Yes, sorry for the late reply, but it worked! Thanks a lot!

I am facing the same issue on windows 10.  I am using latest versions.
virtualbox version 6.1.26r14595
vagrant version 2.2.18
who can help me??

I had this issue after upgrading to macOS Monterey
The solution was:
Create a /etc/vbox/networks.conf (on your host computer, not in the vm) to let you go back to the original IP default range:
(I had to create the vbox folder and the conf file)
I put in the following contents:
* <IP_ADDRESS>/0 ::/0
After that I did:
vagrant halt
vagrant up

And everything started working finally!
thanks to rtpHarry

@deepakgl
I would advice you if you're using mac to check this:
Go to System Preferences > Security & Privacy Then hit the "Allow" button to let Oracle (VirtualBox) load.
Reason is that after the update, macOs reset the permission for the VirtualBox
if you find that access request there and allow it you might solve that problem 👆

Yes, Its working now
Thankyou


Resolve overloaded ACLs in Postgres leading to "row is too big" error

Description/Context

https://dba.stackexchange.com/questions/341796/how-to-work-around-row-is-too-big-for-role-definitions-without-recreating-the
Plan/Design

After lots of digging and poking, this is the set of queries that I would like to test out for repairing our handicapped databases.
-- Explicitly remove any readonly user ACLs from the relacl row for every table
do $$
declare
stmt TEXT;
r record;
begin
for r in select relname from pg_class where relkind in ('r', 'S') and relnamespace = (select oid from pg_namespace where nspname = 'public') loop
stmt := 'update pg_catalog set relacl = array_agg(trimmed_acl) from unnest((select relacl from pg_class where relname = ''' || quote_ident(r.relname) || ''')) as trimmed_acl where trimmed_acl not in (select (''"'' || rolname || ''"=r/mitxonline'')::aclitem from pg_roles where rolname like ''%readonly%'') where relname = ''' || quote_ident(r.relname) || ''';';
EXECUTE stmt;
end loop;
end; $$
;

-- Manage shdepend entries for readonly
delete from pg_shdepend where refobjid in (select oid from pg_roles where rolname like '%readonly%');

Relevant discussion threads that I've found:

https://stackoverflow.com/questions/49904883/postgresql-revoke-all-permissions-from-all-users-and-roles
https://dba.stackexchange.com/questions/244772/how-to-reset-object-permissions-to-default

This is still untested, but I think the safest route is to snapshot one of the problematic DBs, spin up an instance from the snapshot, and try out those queries to see if the issue persists.
To verify before and after, you can run
-- Drop all readonly users from the database
do $$
declare
stmt TEXT;
u record;
begin
for u in select usename from pg_catalog.pg_user where usename like '%readonly%' loop
stmt := 'REVOKE ALL PRIVILEGES ON ALL TABLES IN SCHEMA public FROM "' || u.usename || '";DROP ROLE "' || u.usename || '";';
EXECUTE stmt;
end loop;
end; $$
;

Prior to the fix it will error out with the row is too big problem, and with any luck it should work afterwards.

While I was able to get a snapshot created pretty quickly, I've been having trouble trying to export an RDS snapshot to S3.
I selected AWSServiceRoleForRDS for the IAM role for the export but I keep seeing this error:
The principal export.rds.amazonaws.com isn't allowed to assume the IAM role arn:aws:iam::610119931565:role/aws-service-role/rds.amazonaws.com/AWSServiceRoleForRDS or the IAM role arn:aws:iam::610119931565:role/aws-service-role/rds.amazonaws.com/AWSServiceRoleForRDS doesn't exist.
I saw a suggestion online to add export.rds.amazonaws.com to the allowed assumed roles under Trust relationships but I don't think I have the permissions to do that.

There's no need to export the snapshot to S3. You will want to launch a new RDS instance from the snapshot so that we can replicate the permisisons bug in a separate environment that won't impact any of the running applications.

I was able to spin up and connect an RDS instance from the snapshot: https://us-east-1.console.aws.amazon.com/rds/home?region=us-east-1#database:id=testing-overloaded-acls;is-cluster=false
I was unable to delete from the dependencies tables (pg_depends / pg_shdepends) and unsuccessful in finding a workaround using existing roles. The pg_catalog tables are owned by rdsadmin which is a protected role. Reassigning and dropping ownership was not permitted.
ERROR:  permission denied to reassign objects
ERROR:  permission denied to drop objects

The drop role command also fails:
ERROR:  role "v-aws-conc-readonly-020Dt0NfQ05nF5bmLSem-1688042833" cannot be dropped because some objects depend on it
DETAIL:  255 objects in database mitxonline

The database surgery approach isn't going to work due to restrictions in RDS and concerns around stability. Instead we will need to go the "export and load" approach to rebuild the databases without all of the current ACL assignments. To that end, we need to identify all of the database instances that are suffering from this problem and determine which tables are affected. We may be able to only dump/load specific tables rather than the entire DB.


On-edit diagnostic generation (full text diagnostic generation)

Currently ruffd generates diagnostics only on-save
This is due to the desire for incremental edits to not incur the requirement of parsing the full text for any change, regardless how trivial.
The eventual solution to this will be incremental parsing, which should be possible when https://github.com/charliermarsh/ruff/pull/742 is merged and https://github.com/Seamooo/ruffd/issues/15 is resolved.
In the meantime to satisfy the first implementation of this feature, full diagnostic generation on-edit is acceptable and should be submitted to resolve this issue

This issue is blocking https://github.com/charliermarsh/vscode-ruff/issues/1


"justifyContent" and "minRows" props not recognised by React

These warnings appear when the justifyContent prop is used for the Grid component, and when minRows is used for the TextField component.

It doesn't look like this bug report has enough info for one of us to reproduce it.
Please provide a CodeSandbox (https://mui.com/r/issue-template or https://mui.com/r/ts-issue-template), a link to a repository on GitHub, or provide a minimal code example that reproduces the problem.
Here are some tips for providing a minimal example: https://stackoverflow.com/help/mcve

I can confirm that both properties work for both components in the latest version.
Check out https://codesandbox.io/s/columnsgrid-material-demo-forked-uroktq

Probably caused by other conflicting react packages, shall just use a workaround for now. Thank you


Bug Report: Can't change the BPM of songs in the charter

If you are playing a downloaded version of the game, what operating system are you using?
Windows 10 64-bit
What version of the game are you using? Look in the bottom left corner of the main menu.
1.6.2 - Full Release
Please describe your issue. Provide extensive detail and images if possible.
If you go into a song and change it's Init BPM event to another value, you are no longer able to navigate through your chart with the scroll wheel or arrow keys. Pressing enter will load the song just fine, but pressing 7 to return to the charter will cause your game to crash.
I've made a video to more easily describe the issue: https://streamable.com/godpti
This is the console output upon crash:
source/ChartingState.hx:142: false
source/ChartingState.hx:531: 640
source/ChartingState.hx:535: height of 636
source/Conductor.hx:68: new BPM map BUDDY []
source/ChartingState.hx:227: goin
source/ChartingState.hx:2873: RECALCULATING SECTION TIMES
source/ChartingState.hx:272: Song length in MS: 75590

Very easy fix, the reason why this happens is the saved typed doesn't actually get declared as any "valid type" which means if you load into the song and then save the bpm change event, it will be nothing because the saved type is nothing. a fix is to simply click on the bpm change type drop down and then click save again. A little work around until a bug fix is pushed.
https://github.com/KadeDev/Kade-Engine/commit/17281d85cd492b570b3c8bcba62199a8996e492d


added my git link

Related issue

What changes does this PR do?
Screenshots (If any)
Questions

closing due to inactivity feel free to reopen


skip GPO params check for goerli/op

The endpoint was shut down, causing tests to fail on main. This is the minimal change to get them passing again.

Yep that's in the pipeline #115


Yjdh-209: terms and conditions link + decrease copy-paste from summary page

Description :sparkles:
Localized External link to terms and conditions checkbox
Issues :bug:
Testing :alembic:
Screenshots :camera_flash:
Additional notes :spiral_notepad:

Codecov Report

Merging #303 (e901549) into YJDH-114-invoicer-fields (c67a7a6) will increase coverage by 95.01%.
The diff coverage is n/a.

@@                      Coverage Diff                      @@
##           YJDH-114-invoicer-fields     #303       +/-   ##
=============================================================
+ Coverage                          0   95.01%   +95.01%
=============================================================
  Files                             0       88       +88
  Lines                             0     2851     +2851
=============================================================
+ Hits                              0     2709     +2709
- Misses                            0      142      +142

Impacted Files
Coverage Δ

backend/shared/shared/audit_log/audit_logging.py
97.72% <0.00%> (ø)

backend/shared/shared/oidc/utils.py
95.00% <0.00%> (ø)

...companies/migrations/0003_company_eauth_profile.py
100.00% <0.00%> (ø)

backend/kesaseteli/companies/models.py
100.00% <0.00%> (ø)

backend/shared/shared/audit_log/tests/conftest.py
100.00% <0.00%> (ø)

...plications/migrations/0006_refactor_attachments.py
100.00% <0.00%> (ø)

...ions/migrations/0004_application_default_status.py
100.00% <0.00%> (ø)

backend/kesaseteli/applications/views.py
97.61% <0.00%> (ø)

backend/shared/shared/ytj/ytj_client.py
93.10% <0.00%> (ø)

...ions/migrations/0002_summer_voucher_attachments.py
100.00% <0.00%> (ø)

... and 78 more

Continue to review full report at Codecov.

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


SP2019

What about sharepoint 2019 deployment options ? Is includeClientSideAssets supported ? Thank you

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

ID: 1320b802-156c-5d77-db79-b211b66d4b2f
Version Independent ID: efe018a3-001e-e097-a96c-66ab1786afe5
Content: SharePoint Framework development with SharePoint 2016 Feature Pack 2
Content Source: docs/spfx/sharepoint-2016-support.md
Product: sharepoint
Technology: sharepoint-framework
GitHub Login: @spdevdocs
Microsoft Alias: spdevdocs

Nope... that feature uses the Office 365 CDN which doesn't exist on-prem.


Many different script addresses with the introduction of SegWit

All WriteLines outputs different script addresses.

Is my P2SH and P2SH-P2WPKH identification correct?
What are the ones I denoted with ???.

var n = Network.Main;
var sk = new BitcoinSecret("L4LDNWrzAaeCKaUn2ZhWDSVD1NiJvtVtJCJVogTSGD4aHmXgHwJ7");
PubKey pk = sk.PubKey;

KeyId pkh = pk.Hash;
WitKeyId wpkh = pk.WitHash;

WriteLine(pkh.ScriptPubKey.GetScriptAddress(n)); // P2SH
WriteLine(wpkh.ScriptPubKey.Hash.ScriptPubKey.GetScriptAddress(n)); // P2SH-P2WPKH
WriteLine(wpkh.ScriptPubKey.GetScriptAddress(n)); // ???
WriteLine(wpkh.ScriptPubKey.WitHash.ScriptPubKey.GetScriptAddress(n)); // ???

WriteLine(wpkh.ScriptPubKey.Hash.ScriptPubKey.GetScriptAddress(n)); // P2SH-P2WPKH
No this is actually a P2SH-P2SH-P2WPKH. (which is no sense)
Should be
WriteLine(wpkh.ScriptPubKey.Hash.GetAddress(n)); // P2SH-P2WPKH
WriteLine(wpkh.ScriptPubKey.GetScriptAddress(n)); // ??? it is //P2SH-P2WPKH (equal to above)
WriteLine(wpkh.ScriptPubKey.WitHash.ScriptPubKey.GetScriptAddress(n)); // ??? this is none sense. Would be P2SH-P2WSH-P2WPKH. As explained in another of your issue P2WSH-P2WPKH should never be done, as it is equivalent to everybody can spend.

For the record, this three equals:
WriteLine(wpkh.ScriptPubKey.Hash.GetAddress(n));
WriteLine(wpkh.ScriptPubKey.Hash.ScriptPubKey.GetDestinationAddress(n));
WriteLine(wpkh.ScriptPubKey.GetScriptAddress(n));

Where wpkh.ScriptPubKey.Hash.ScriptPubKey is documented in the book.

yes. I am wondering if I should deprecate some stuff. Or maybe just adding comment ?

I'd just comment. Also writing a KeyManager class, that implements BIP44 and BIP49 could make sense, similar to my Safe: https://github.com/nopara73/HiddenWallet/tree/master/HiddenWallet/KeyManagement
I was long planning to write it and will eventually, just always got off track by the small inconsistencies of BIP44.


calling detach on a numpy array in art.estimators.regression.pytorch

Describe the bug
Lines 270-272 in art.estimators.regression.pytorch.py are:
output = output.detach().cpu().numpy().astype(np.float32)
if len(output.shape) == 1:
    output = np.expand_dims(output.detach().cpu().numpy(), axis=1).astype(np.float32)

Line 270 sets output to be a numpy array, but then line 272 calls detach() on output, resulting in a type error since detach() is a method for tensors, not numpy arrays.
To Reproduce
Call predict() on a PyTorchRegressor object that returns a 1-D  output.
Expected behavior
the code should not cause a bug and crash.
Screenshots
The exact error is "'numpy.ndarray' object has no attribute 'detach'"
System information (please complete the following information):

OS: Mac Monterey 12.4
Python version: 3.8.10
ART version or commit number: main branch as of 8/18/22
TensorFlow / Keras / PyTorch / MXNet version: PyTorch version 1.12.1

Hi @annapmeyer Thank you very much for discovering and fixing this bug!


notebook server die when using datashader support in holoviews

I'm brand new in Holoviews/datashader/bokeh, but loves them. I was trying to use datashader in Holoview, but constantly ran into "server has died" message.
My setup:

conda distribution (Python 3.5) under Linux Mint virtual machine.
Originally installed Holoviews 1.6.2, and discovered my dataset is too large to plot and would like to use the datashader support.
Then I clone the master branch, installed using python setup.py install,  and ran the example notebook: https://anaconda.org/philippjfr/holoviews_datashader/notebook

I got a server died message every time when executing any of the aggregate cell. Can anyone please provide some pointer on how to get this working? Greatly appreciated.

Are there any messages on the terminal where you started Jupyter that seem relevant (or maybe in the JavaScript console, which is less likely to be relevant)?  When I've seen problems like that, I've found messages on my terminal from some dependencies of datashader (not datashader itself).  E.g. there is a message from an HDF5 library that comes up, which is basically saying that various different libraries installed have incompatible versions, but I don't recall the details.  If you could post such messages if there are any, we can probably give advice.

The Jupyter notebook command line output:

Intel MKL FATAL ERROR: Cannot load libmkl_def.so.

locate libmkl on console and returns:

/home/fz/anaconda3/lib/libmkl_avx.so
/home/fz/anaconda3/lib/libmkl_avx2.so
/home/fz/anaconda3/lib/libmkl_avx512.so
/home/fz/anaconda3/lib/libmkl_avx512_mic.so
/home/fz/anaconda3/lib/libmkl_core.so
/home/fz/anaconda3/lib/libmkl_def.so
/home/fz/anaconda3/lib/libmkl_intel_ilp64.so
/home/fz/anaconda3/lib/libmkl_intel_lp64.so
/home/fz/anaconda3/lib/libmkl_intel_thread.so
/home/fz/anaconda3/lib/libmkl_mc.so
/home/fz/anaconda3/lib/libmkl_mc3.so
/home/fz/anaconda3/lib/libmkl_rt.so
/home/fz/anaconda3/lib/libmkl_sequential.so
/home/fz/anaconda3/lib/libmkl_vml_avx.so
/home/fz/anaconda3/lib/libmkl_vml_avx2.so
/home/fz/anaconda3/lib/libmkl_vml_avx512.so
/home/fz/anaconda3/lib/libmkl_vml_avx512_mic.so
/home/fz/anaconda3/lib/libmkl_vml_cmpt.so
/home/fz/anaconda3/lib/libmkl_vml_def.so
/home/fz/anaconda3/lib/libmkl_vml_mc.so
/home/fz/anaconda3/lib/libmkl_vml_mc2.so
/home/fz/anaconda3/lib/libmkl_vml_mc3.so
/home/fz/anaconda3/pkgs/mkl-11.3.1-0/lib/libmkl_avx.so
/home/fz/anaconda3/pkgs/mkl-11.3.1-0/lib/libmkl_avx2.so
/home/fz/anaconda3/pkgs/mkl-11.3.1-0/lib/libmkl_avx512.so
/home/fz/anaconda3/pkgs/mkl-11.3.1-0/lib/libmkl_avx512_mic.so
/home/fz/anaconda3/pkgs/mkl-11.3.1-0/lib/libmkl_core.so
/home/fz/anaconda3/pkgs/mkl-11.3.1-0/lib/libmkl_def.so
/home/fz/anaconda3/pkgs/mkl-11.3.1-0/lib/libmkl_intel_ilp64.so
/home/fz/anaconda3/pkgs/mkl-11.3.1-0/lib/libmkl_intel_lp64.so
/home/fz/anaconda3/pkgs/mkl-11.3.1-0/lib/libmkl_intel_thread.so
/home/fz/anaconda3/pkgs/mkl-11.3.1-0/lib/libmkl_mc.so
/home/fz/anaconda3/pkgs/mkl-11.3.1-0/lib/libmkl_mc3.so
/home/fz/anaconda3/pkgs/mkl-11.3.1-0/lib/libmkl_rt.so
/home/fz/anaconda3/pkgs/mkl-11.3.1-0/lib/libmkl_sequential.so
/home/fz/anaconda3/pkgs/mkl-11.3.1-0/lib/libmkl_vml_avx.so
/home/fz/anaconda3/pkgs/mkl-11.3.1-0/lib/libmkl_vml_avx2.so
/home/fz/anaconda3/pkgs/mkl-11.3.1-0/lib/libmkl_vml_avx512.so
/home/fz/anaconda3/pkgs/mkl-11.3.1-0/lib/libmkl_vml_avx512_mic.so
/home/fz/anaconda3/pkgs/mkl-11.3.1-0/lib/libmkl_vml_cmpt.so
/home/fz/anaconda3/pkgs/mkl-11.3.1-0/lib/libmkl_vml_def.so
/home/fz/anaconda3/pkgs/mkl-11.3.1-0/lib/libmkl_vml_mc.so
/home/fz/anaconda3/pkgs/mkl-11.3.1-0/lib/libmkl_vml_mc2.so
/home/fz/anaconda3/pkgs/mkl-11.3.1-0/lib/libmkl_vml_mc3.so
Any suggestions?
Thanks.

Sounds like it is indeed issues with dependency libraries.  You could try conda remove mkl, as mkl should speed things up somewhere but shouldn't be strictly required by anything.

Thank you. I uninstalled mkl, and it works. I then conda install mkl again. It installed a newer version (2017.0.1-0) as well as downgrading a number of packages. Now it no longer complain about mkl

Sounds good!


BUG: cupyx/scipy/signal: fix mpmath test

fixes https://github.com/cupy/cupy/issues/8255
The test is a clone of the eponymous test in scipy, so drop in the mpmath-using file from scipy and actually test the cupyx.scipy function.

@ev-br Thanks! Could you cherry-pick the change #8258 in this PR to test this PR in CI?

@kmaehashi done

Thanks!
/test

/test mini