Changed bhagavadgitaapi.in link

Changed the bhagavadgitaapi.in API link since the old one did not work anymore.

[x] My submission is formatted according to the guidelines in the contributing guide
[x] My addition is ordered alphabetically
[x] My submission has a useful description
[x] The description does not have more than 100 characters
[x] The description does not end with punctuation
[x] Each table column is padded with one space on either side
[x] I have searched the repository for any relevant issues or pull requests
[x] Any category I am creating has the minimum requirement of 3 items
[x] All changes have been squashed into a single commit

Thank you!


Prevent sliding on specific card area touching

Hi,
On mobile, I have a slider with specific buttons on each on my cards.
Buttons work but it doesn't prevent the slide from going to the next or previous slide.
Is there a way to avoid a click on a specific zone from the card to interact with the slider?
Thanks

Solved by adding a touchstart event instead of a single click event on my specific buttons.


Implement compare view

This is the olympia part to https://github.com/mozilla/addons-validator/issues/62
implement a view that compares the results between python and js validator. It appears we might be able to leverage some code from devhub.utils:ValidationComperator.

Don't think this is needed.


I'd like to join this project's community

Your public repo
https://github.com/florihas/gov4git-identity-public.git
Your public branch
main

Approved

 Follow up
On Friday, 16-Aug-24 07:11:44 UTC by Gov4Git dev
@florihas was added to the community.


Incorrect file name in documentation

for the "Change the site name in the QR Code", the file is listed as "/Areas/Identity/Pages/Account/Manage/EnableAuthenticator.cshtml".  Obviously it needs the .cs suffix.

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

ID: 117c950a-0bcb-d936-31cf-5859b4280abb
Version Independent ID: 270bedc7-a636-5bd0-9e2f-e42235deec8c
Content: Enable QR Code generation for TOTP authenticator apps in ASP.NET Core
Content Source: aspnetcore/security/authentication/identity-enable-qrcodes.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

@scottaddie I've got this.

@Rick-Anderson Already have the PR ready to go.

OK


About 'disable auto mode' option

About the option 'disable auto mode' of the plugin, it disables the auto mode of the device in Homekit / Home app or the 'auto' mode in the melcloud web interface? Thanks.

In home app

Thank you for the quick answer.
But, when I activate this option, the 'automatic' function in the AC in the homekit home app continues as an selectable mode. Is this normal? What is the function of auto mode in this case? Thanks qgain:

You can select what a mode will be used for Auto in plugin config, some users do not need auto but use fan or dry instead

Thank you very much


What about modded mobs trophies? And other questions!

Hi! Really love the mod, thank you for making it :)
I was wondering if there was a way to add more trophies that would drop for modded mobs?
I'm also struggling to understand how the different sizes of trophy bases are supposed to work in a survival environment. My gut thinking was that you would be able to craft them and then place the trophies you get from mobs inside, but it doesn't seem to be the case at all since they don't have recipes. Are they just for creative?
And thirdly, I think Trofers would really benefit from having a per-mob drop rate akin to what Just Mob Heads (and Valheim) do. A drop rate of 0.5 for a rare mob is nice, but if you set it for zombies as well, it can easily get annoying when you have to throw away dozens of them all the time.
Thank you 😊

Trofers was originally purely a tool for modpack developers, the entity trophies were added later on as an example of what you can do with it. The larger trophy bases aren't obtainable in survival, and are mainly intended to be used by pack devs.
Adding new trophies is possible, although you'll have to create a data pack to do it. The README explains the data format if you're interested. At the moment there's no easy way to make entities drop them if you're using Fabric though.
Trofers already has compatibility for a couple of mods, you're always welcome to suggest more mods here on the issue tracker. I can't promise to add support for every suggested  mod though.
Because I'd like to keep the default data pack at least somewhat maintainable, unfortunately I don't plan on adding a per-mob drop rate for trophies. Adding trophies for new mobs is already quite time-consuming, and I believe most users would prefer the convenience of setting the drop rate for all trophies through a single config option, even if that means some trophies are much rarer than others.

Right! Thanks for the comprehensive reply. I'm indeed on Fabric.
It would be nice if there was a database where modpack developers could upload what they do. If somebody has already done the work to make a specific mod compatible with Trofers, then there should be no reason for other modpack developers to do it again. Just an idea though!
The main mods I was thinking about here were Creeper Overhaul, The Graveyard and Naturalist.

The creepers from Creeper Overhaul have animations that can't be turned off last time I looked into it, which unfortunately looks rather weird on a trophy. The Graveyard and Naturalist might have similar issues since they also use GeckoLib, but I haven't properly looked into them yet.


Update Get-AzureADServicePrincipalOAuth2PermissionGrant.md

Fixed typo in part of the example Powershell

Docs Build status updates of commit 195427b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

azureadps-2.0/AzureAD/Get-AzureADServicePrincipalOAuth2PermissionGrant.md
:white_check_mark:Succeeded

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

@CelesteDG, good to go.


Revert "Revert "linux: eliminate a read on eventfd per wakeup (#4400)""

Reverts libuv/libuv#4585
I think it looks like this may be a latent synchronization bug in bind9, which (contrary to expectations) was made slightly easier to diagnose because of that PR? It remains to be confirmed whether adding the missing memory fence fixes their test however, which would demonstrate whether their code was always broken or whether some other interaction is causing issues.

So, do I hold the patch release for now?

I'd do the release. This pull request is not critical, the world won't stop turning when we hold it back until the next release.

Linking to discussion in other issue: https://github.com/libuv/libuv/issues/4584#issuecomment-2422528341
(I had to look it up again, might as well save others or my future self the trouble)


⚠ Roblox Site has degraded performance

In 3557982, Roblox Site (https://roblox.com/) experienced degraded performance:

HTTP code: 200
Response time: 3923 ms

Resolved: Roblox Site performance has improved in 4f2eac6 after 12 minutes.


Feature: Pause Autocomplete

This patch allows the user to pause/resume autocomplete (#13) for the duration of the Visual Studio Code session.  The status bar item indicates that autocomplete is paused through the displayed icon as well as the tooltip.
Additional changes

New: Added command llama.pause, "Llama Coder: Pause" -- pause autocomplete until resumed or the session is restarted.
New: Added command llama.resume, "Llama Coder: Resume" -- resume autocomplete after being previously paused.
New: Added command llama.toggle, "Llama Coder: Toggle" -- toggle the paused state of inferencing.
Change: Modified command llama.openSettings now available in the command palette as "Llama Coder: Open Settings".
Change: Clicking on the status bar item will invoke llama.toggle instead of llama.openSettings.

Can we have this? @ex3ndr


Leafwing input example is broken in input-server mode

So is bullet_prespawn.
That's probably because of the query changes I made to make host-server mode work.
Try to find a way to get both listen-server and host-server to work with the same code (no if statements)

Fixed with https://github.com/cBournhonesque/lightyear/commit/c59f2b11aaef39f851a2e7329dccb1b9cd42c726


fix missing " char

Pull Request
📖 Description

🎫 Issues

👩💻 Reviewer Notes

📑 Test Plan

✅ Checklist
General

[ ] I have added tests for my changes.
[ ] I have tested my changes.
[ ] I have updated the project documentation to reflect my changes.
[ ] I have read the CONTRIBUTING documentation and followed the standards for this project.

Component-specific

[ ] I have added a new component
[ ] I have added Unit Tests for my new compontent
[ ] I have modified an existing component
[ ] I have validate Unit Tests for an existing component

⏭ Next Steps

This problem has already been detected and corrected in PR #1447.
https://github.com/microsoft/fluentui-blazor/pull/1447/files#diff-42d9c5c371bc6506fb9b4b212b292710d895898c0ef01f7cf2c4f4fb50563c4a


Document using cocotb installed as Python package

After #836 goes in the only way of installing and the preferred way of using cocotb is through a Python package. We need to have documentation for that.

We should also mention how to do development work.
Is https://pip.pypa.io/en/stable/reference/pip_install/#editable-installs the way to go, for instance?

pip install --user -e . works in a git checkout without root permissions

pip install develop should work also

That's now part of the documentation.


Client database

let me know what you think @bobbinth
I can also make a first draft of the database structure

For the client database, I was thinking something like this:

account_code table, with the following fields:

root: blob - root of the Merkle tree for all exported procedures in account module.
module: blob - serialized MASM code for the account module.

account_storage table with the following fields:

root: blob - root of the account storage Merkle tree.
slots: blob - serialized key-value pair of non-empty account slots.
For now, we'll ignore MerkleStore portion of the storage.

account_vault table, with the following fields:

root: blob - root of the Merkle tree for the account vault.
assets: blob - serialized account vault assets.

accounts table, with the following fields:

id: u64 - account ID.
code_root: blob.
storage_root: blob.
vault_root: blob.
nonce: u64 - account nonce.
status: enum - status of the account record: either pending or committed.

input_notes table with the following fields:

id: blob - hash of the note.
nullifier: blob - note's nullifier.
script: blob- serialized NoteScript, including script hash and ProgramAst.
vault: blob - serialized NoteVault, including vault hash and list of assets.
inputs: blob - serialized `NoteInputs, including inputs hash and list of inputs.
serial_num: blob
sender_id: u64 - account ID of the sender.
tag: u64
inclusion_proof: blob - note's index and a Merkle path from the note to the block's note_root.
recipients: blob - a list of account IDs for accounts which can consume this note.
status: enum - status of the note: either pending or consumed.
commit_height: u32 - block number in which the note was included into the chain.

output_notes table with the following fields:

id: blob - hash of the note.
vault: blob - serialized NoteVault, including vault hash and list of assets.
recipient: blob - serialized note recipient information. This could be either a single digest or full recipient info (note script, note inputs, and serial_num).
sender_id: u64 - account ID of the sender.
tag: u64
inclusion_proof: blob - note's index and a Merkle path from the note to the block's note_root.
status: enum - status of the note: either pending or committed.
commit_height: u32 - block number in which the note was included into the chain.

transactions table with the following fields:

id: blob - transaction ID: this still needs to be defined, but it is probably a hash of account input/output states, input notes, output notes, and tx_script.
account_id - ID of the account against which the transaction was executed.
init_account_state: blob - hash of the account state before the transaction was executed.
final_account_state: blob - hash of the account state after the transaction was executed.
input_notes: blob - serialized list of input note hashes. Another option is to store this a JSON to make it more easily queryable.
output_notes: blob - serialized list of output note hashes. Another option is to store this a JSON to make it more easily queryable.
tx_script: blob - serialized transaction script, including script hash and ProgramAst.
block_num: u32 - block number for the block against which the transaction was executed.
status: enum - status of the transaction: either pending or committed.
commit_height: u32 - block number of the block at which the transaction was included into the chain.

block_headers table, with the following fields:

block_num: u32
notes_root: blob - root of the notes Merkle tree in this block.
sub_hash: blob - hash of all other header fields in the block.
header: blob - serialized block header.

chain_mmr table - fields TBD.

One open question: in many cases we could store complex objects not as serialized blobs but as JSON (stored as text). While this would increase storage requirements quite a bit (and also slow down parsing), for a client, this is unlikely to be a concern. On the other hand, storing data as JSON may improve readability and debugging.

Nice, should we start implementing it?

Yes - I'll try to implement this in the next several days.

Do you consider multiple accounts here as well? Will we have one database per managed account then or would they be all managed in the Accounts table? Is it easier if we start with one account (I think so)?

This structure can accomodate multiple accounts as well (they will all go into the same accounts table) - but we can limit it to 1 account in the CLI. Not sure if it will simplify things a lot though.

Should we store keys somewhere in the client? If so, we don't store them in this database sketch atm, right?

Yes, but I'm not sure if storing them in the database is a good idea. We'll probably have a separate mechanism for storing keys - but I haven't gotten that far yet.

Would it make sense to store the note vault and script separately (as you do with the account vault, storage and code?

For the note script - yes, this is a good idea so that we don't duplicate common scripts. For the vault probably doesn't make too much sense at a note's vault is static and unique to a note.

Yes, but I'm not sure if storing them in the database is a good idea. We'll probably have a separate mechanism for storing keys - but I haven't gotten that far yet.

I think storing the keys in the database should be fine provided we do not store them in plain text.  We should encrypt the keys with a password and when we read the keys from the database we can unencrypt them.  As far as I'm aware this is how metamask and other browser based wallets work.

Yes, but I'm not sure if storing them in the database is a good idea. We'll probably have a separate mechanism for storing keys - but I haven't gotten that far yet.

I think storing the keys in the database should be fine provided we do not store them in plain text. We should encrypt the keys with a password and when we read the keys from the database we can unencrypt them using the password. As far as I'm aware this is how metamask and other browser based wallets work.

I agree with @frisitano, I don't see an issue with storing them more securely in the same store. Created issue https://github.com/0xPolygonMiden/miden-client/issues/30 in order to discuss this specific point.

Closing as most of this has already been addressed, and things which are left over (e.g., input vs. output notes) should be handled in separate issues.


Fix setting basic auth config by hand

Hi @morgoth, thanks for this! But this was not supported on purpose, as a deterrent to avoid people setting secret as their password, forcing having to use Rails credentials and providing an easy way to set those via a command.

Not being able to set it by hand as all other configs would make people confuse when they try it (as me ;-).
Also I can imagine people setting secrets via env variable, which for them it won't be possible to use this feature.

I think the README explains it, in case someone is confused 🤔  I'll wait to see if this actually becomes a problem before changing it.


Document message support for NaN and other special floating point numeric limits

Hello,
Would it be possible to document in Interfaces that you can set constants or default values to special values like NaN? It's not clear what's supported, and whether NAN, NaN, and whether it would be quiet or signaling NaN.
https://en.cppreference.com/w/cpp/types/numeric_limits
I am writing messages that have values that can fail. Rather than add is_X_valid booleans for every field in the message, I would prefer to have support for NaN. This results in significantly less application code and reduces bandwidth when messages are sent over wireless networks using DDS compared to dedicated booleans.
While doing this, it would also be good to document whether infinity is supported.
Related:

https://github.com/ros2/rosidl/issues/351
https://answers.ros.org/question/345861/using-nan-constants-in-messageservice-definitions/

And, the CLI also supports it on humble:
ros2 topic pub /ap/cmd_vel geometry_msgs/msg/TwistStamped "{header: {frame_id: base_link}, twist: {linear: {x: .nan, y: .nan, z: .nan}, angular: {z: 0.01}}}"

Yeah, I think that is a reasonable thing to document; I'd enhance the table in https://docs.ros.org/en/humble/Concepts/Basic/About-Interfaces.html#field-types to add a "Note" column, or something like that.  If you'd like to open a PR to add that, I'd be happy to review it.

Yeah, I think that is a reasonable thing to document; I'd enhance the table in https://docs.ros.org/en/humble/Concepts/Basic/About-Interfaces.html#field-types to add a "Note" column, or something like that. If you'd like to open a PR to add that, I'd be happy to review it.

It doesn't look like it's actually supported.
In a .msg:
# NaN (IEE-754 quiet NaN) signifies no data is available at the location.
float64 my_value NaN

c: error: ‘nanl’ undeclared (first use in this function)
  276 |   msg->my_value = nanl;

c++: error: ‘nan’ was not declared in this scope
  198 |       this->my_value= nan;

Relates to https://github.com/ros2/rosidl/issues/705

Huh, in the IDL 4.2 spec, it says nothing about special floating point values:
https://www.omg.org/spec/IDL/4.2/PDF
See section <IP_ADDRESS>.
Then in the DDS Consolidated JSON 1.0 spec, it says that NaN is supported.
https://www.omg.org/spec/DDS-JSON/1.0/PDF
See section <IP_ADDRESS>.8
Numeric values, such as Infinity, -Infinity, and NaN, which as stated in [ECMA-404] cannot be represented as
sequences of digits, shall be represented using the following JSON strings: "inf", "-inf", and "nan".


How to pass default SearchParameters

With our focus on widget states, we missed one important usecase: being able to define simple SearchParameters that will always be sent.
This goes back to the idea of @bobylito of having a <Configure/> widget. For instance we could imagine having such simple <Configure/> widget that we pass a state to search parameters function that will allow users to set distinct=true for instance or any other Search parameter.
What do you think?

Really need this.
In my own index, I have by default set distinct = true
But in my UI I have use cases where I want to set it to false.

This is planned and need to be done, yes

Hi @vvo I was trying to find how to do it with current version. Any leads?

I would suggest you to clone the projet, create a new "Connector" see how the searchBox connector is done, then see how a generic "configure" component could be done. Should be doable, but you will have to dig in the docs (making your own widget: https://community.algolia.com/instantsearch.js/react/Customization.html) and the code. Good luck.

Got it, I was looking for this page only. I wasnt linked in the current new docs.

Thanks I used the hits connecter from docs https://github.com/algolia/instantsearch.js/blob/v2/packages/react-instantsearch/src/widgets/Hits/connect.js
My getSearchParameters looks like this now.
getSearchParameters(searchParameters, props) {
    searchParameters = searchParameters.setQueryParameter("distinct", false);
    if (
        typeof props.hitsPerPage !== 'undefined' &&
        typeof searchParameters.hitsPerPage === 'undefined'
    ) {
      return searchParameters.setHitsPerPage(props.hitsPerPage);
    }
    return searchParameters;
  }

Awesome! Utilmately we need a simpler API to do that.

What about a function passed as a prop on InstantSearch with the signature SearchParameters => SearchParameters We don't really need to provide it but I think it's easier than having to instantiate it yourself (as a developer)

We don't really need to provide it but I think it's easier than having to instantiate it yourself (as a developer)

Not sure to get what you are proposing

Updated my comment @vvo

We don't really need to provide the SearchParameter object but I think it's easier than having to instantiate it yourself (as a developer)

Yes it is easier to manipulate an existing state rather than having to provide an API to create a state.
As for an option on <InstantSearch/> or a <Configure/> widget, with a <Configure/> widget, you could maybe more easily remove refinements dynamically if you unmount the component. Example:
var WithGeo = <Configure state={state => state.addQueryParameters('geo', '...')} />
var WithDistinct = <Configure state={state => state.addQueryParameters('geo', '...')} />
// later on..
<WithGeo/><WithDistinct/>

// and remove them when needed.

I guess the same is achievable with just a function on <InstantSearch/>?

That's an interesting declarative Api using configure. What do you think @mthuret?

Having a function that configure SearchParameter on the  component feel more natural to me than having several  widget. But if it's really easier for changing refinements dynamically that's a real advantage.
In either case that's great, I will need this feature in a near future :)

having several widget

Sorry the example was misleading, we won't have several widget, only one <configure/> widget.
The issue I see if we put it as an option on <InstantSearch/> is about being able to unmount/remove parameters. I am not sure how that would fit.
This is a good investigation issue.

I've given some thoughts 🤔  on the <Configure/> vs <InstantSearch /> discussion 💡

The issue I see if we put it as an option on  is about being able to unmount/remove parameters. I am not sure how that would fit.

The function on <InstantSearch /> will be a prop. therefore it can be easily switched.
var WithGeo = state => state.addQueryParameter('geo', '...')};
var WithDistinct = state => state.addQueryParameter('distinct', '...')};
// later on..
<InstantSearch searchStateConfig={withGeo}/>
// Or
<InstantSearch searchStateConfig={withGeo}/>
// And it can be composed too
<InstantSearch searchStateConfig={s => withDistinct(withGeo(s)) }/>

Also:

it'll be easier to predict when the function is applied if we control it from this component.
the API will be more familiar and does not rely on non-UI widget widget

I suggest we go for a prop on <InstantSearch /> 🚀

WFM

(Also don't try to millennial-convince us lowl)

it always works better to convey the messsage 😸


[BUG] Follow_link issue with headerlink

Please confirm

[x] I am running the latest version of this plugin

Describe the bug
When using a header link, depending on the cursor position, the output of follow_link is not the same.

If the cursor is on the filename, the picker works as expected
If the cursor is on the header name or block id, the picker only looks for what goes after the #

This is true for headers or bloc-ids links.
To Reproduce
Steps to reproduce the behavior:

Start nvim
Place cursor on the header part of the link (or the block-id part)
See error

Expected behavior
Picker output should be the same no matter the position of the cursor.
Operating system (please complete the following information):

OS: I use Arch (btw)

This can probably be tied to #103


Real optim state  hidden because not passed in

The build job needs to be modified because the API of optim changed. I wasn't going to leave the misnamed state to stand for 'config' and a new 'config' to be the state.
Or do I have to fix the build now?

There is some problem with the github build environment. I tested the Lua5.2 version and it tests just fine without errors. I have gcc 4.8, while travis build uses old 4.6, maybe that's the problem. In any case, it is not my checkin but the build just fails now, because the 'LessThan' test gives the wrong gradient.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Rene Schaub seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


gp mem issues.

 File "/home/msultan/software/anaconda/bin/osprey", line 9, in <module>
    load_entry_point('osprey==1.0.0.dev0', 'console_scripts', 'osprey')()
  File "/home/msultan/software/anaconda/lib/python3.5/site-packages/osprey-1.0.0.dev0-py3.5.egg/osprey/cli/main.py", line 37, in main
    args_func(args, p)
  File "/home/msultan/software/anaconda/lib/python3.5/site-packages/osprey-1.0.0.dev0-py3.5.egg/osprey/cli/main.py", line 42, in args_func
    args.func(args, p)
  File "/home/msultan/software/anaconda/lib/python3.5/site-packages/osprey-1.0.0.dev0-py3.5.egg/osprey/cli/parser_worker.py", line 8, in func
    execute(args, parser)
  File "/home/msultan/software/anaconda/lib/python3.5/site-packages/osprey-1.0.0.dev0-py3.5.egg/osprey/execute_worker.py", line 77, in execute
    sessionbuilder=config.trialscontext)
  File "/home/msultan/software/anaconda/lib/python3.5/site-packages/osprey-1.0.0.dev0-py3.5.egg/osprey/execute_worker.py", line 102, in initialize_trial
    params = strategy.suggest(history, searchspace)
  File "/home/msultan/software/anaconda/lib/python3.5/site-packages/osprey-1.0.0.dev0-py3.5.egg/osprey/strategies.py", line 278, in suggest
    self._fit_model(X, Y)
  File "/home/msultan/software/anaconda/lib/python3.5/site-packages/osprey-1.0.0.dev0-py3.5.egg/osprey/strategies.py", line 200, in _fit_model
    model.optimize(messages=False, max_f_eval=self.max_feval)
  File "/home/msultan/software/anaconda/lib/python3.5/site-packages/GPy/core/gp.py", line 551, in optimize
    super(GP, self).optimize(optimizer, start, messages, max_iters, ipython_notebook, clear_after_finish, **kwargs)
  File "/home/msultan/software/anaconda/lib/python3.5/site-packages/paramz/model.py", line 104, in optimize
    opt.run(start, f_fp=self._objective_grads, f=self._objective, fp=self._grads)
  File "/home/msultan/software/anaconda/lib/python3.5/site-packages/paramz/optimization/optimization.py", line 51, in run
    self.opt(x_init, **kwargs)
  File "/home/msultan/software/anaconda/lib/python3.5/site-packages/paramz/optimization/optimization.py", line 124, in opt
    opt_result = optimize.fmin_l_bfgs_b(f_fp, x_init, maxfun=self.max_iters, maxiter=self.max_iters, **opt_dict)
  File "/home/msultan/software/anaconda/lib/python3.5/site-packages/scipy/optimize/lbfgsb.py", line 193, in fmin_l_bfgs_b
    **opts)
  File "/home/msultan/software/anaconda/lib/python3.5/site-packages/scipy/optimize/lbfgsb.py", line 330, in _minimize_lbfgsb
    f, g = func_and_grad(x)
  File "/home/msultan/software/anaconda/lib/python3.5/site-packages/scipy/optimize/lbfgsb.py", line 278, in func_and_grad
    f = fun(x, *args)
  File "/home/msultan/software/anaconda/lib/python3.5/site-packages/scipy/optimize/optimize.py", line 289, in function_wrapper
    return function(*(wrapper_args + args))
  File "/home/msultan/software/anaconda/lib/python3.5/site-packages/scipy/optimize/optimize.py", line 63, in __call__
    fg = self.fun(x, *args)
  File "/home/msultan/software/anaconda/lib/python3.5/site-packages/paramz/model.py", line 270, in _objective_grads
    self.optimizer_array = x
  File "/home/msultan/software/anaconda/lib/python3.5/site-packages/paramz/parameterized.py", line 325, in __setattr__
    return object.__setattr__(self, name, val);
  File "/home/msultan/software/anaconda/lib/python3.5/site-packages/paramz/core/parameter_core.py", line 118, in optimizer_array
    self.trigger_update()
  File "/home/msultan/software/anaconda/lib/python3.5/site-packages/paramz/core/updateable.py", line 79, in trigger_update
    self._trigger_params_changed(trigger_parent)
  File "/home/msultan/software/anaconda/lib/python3.5/site-packages/paramz/core/parameter_core.py", line 128, in _trigger_params_changed
    self.notify_observers(None, None if trigger_parent else -np.inf)
  File "/home/msultan/software/anaconda/lib/python3.5/site-packages/paramz/core/observable.py", line 91, in notify_observers
    [callble(self, which=which) for _, _, callble in self.observers]
  File "/home/msultan/software/anaconda/lib/python3.5/site-packages/paramz/core/observable.py", line 91, in <listcomp>
    [callble(self, which=which) for _, _, callble in self.observers]
  File "/home/msultan/software/anaconda/lib/python3.5/site-packages/paramz/core/parameter_core.py", line 497, in _parameters_changed_notification
    self.parameters_changed()
  File "/home/msultan/software/anaconda/lib/python3.5/site-packages/GPy/core/gp.py", line 190, in parameters_changed
    self.posterior, self._log_marginal_likelihood, self.grad_dict = self.inference_method.inference(self.kern, self.X, self.likelihood, self.Y_normalized, self.mean_function, self.Y_metadata)
  File "/home/msultan/software/anaconda/lib/python3.5/site-packages/GPy/inference/latent_function_inference/exact_gaussian_inference.py", line 40, in inference
    K = kern.K(X)
  File "/home/msultan/software/anaconda/lib/python3.5/site-packages/GPy/kern/src/kernel_slice_operations.py", line 79, in wrap
    ret = f(self, s.X, s.X2, *a, **kw)
  File "<decorator-gen-125>", line 2, in K
  File "/home/msultan/software/anaconda/lib/python3.5/site-packages/paramz/caching.py", line 282, in g
    return cacher(*args, **kw)
  File "/home/msultan/software/anaconda/lib/python3.5/site-packages/paramz/caching.py", line 178, in __call__
    new_output = self.operation(*args, **kw)
  File "/home/msultan/software/anaconda/lib/python3.5/site-packages/GPy/kern/src/add.py", line 52, in K
    return reduce(np.add, (p.K(X, X2) for p in which_parts))
  File "/home/msultan/software/anaconda/lib/python3.5/site-packages/GPy/kern/src/add.py", line 52, in <genexpr>
    return reduce(np.add, (p.K(X, X2) for p in which_parts))
  File "/home/msultan/software/anaconda/lib/python3.5/site-packages/GPy/kern/src/kernel_slice_operations.py", line 79, in wrap
    ret = f(self, s.X, s.X2, *a, **kw)
  File "<decorator-gen-134>", line 2, in K
  File "/home/msultan/software/anaconda/lib/python3.5/site-packages/paramz/caching.py", line 282, in g
    return cacher(*args, **kw)
  File "/home/msultan/software/anaconda/lib/python3.5/site-packages/paramz/caching.py", line 178, in __call__
    new_output = self.operation(*args, **kw)
  File "/home/msultan/software/anaconda/lib/python3.5/site-packages/GPy/kern/src/stationary.py", line 97, in K
    r = self._scaled_dist(X, X2)
  File "<decorator-gen-137>", line 2, in _scaled_dist
  File "/home/msultan/software/anaconda/lib/python3.5/site-packages/paramz/caching.py", line 282, in g
    return cacher(*args, **kw)
  File "/home/msultan/software/anaconda/lib/python3.5/site-packages/paramz/caching.py", line 178, in __call__
    new_output = self.operation(*args, **kw)
  File "/home/msultan/software/anaconda/lib/python3.5/site-packages/GPy/kern/src/stationary.py", line 149, in _scaled_dist
    return self._unscaled_dist(X/self.lengthscale, X2)
  File "/home/msultan/software/anaconda/lib/python3.5/site-packages/GPy/kern/src/stationary.py", line 121, in _unscaled_dist
    r2 = -2.*tdot(X) + (Xsq[:,None] + Xsq[None,:])
MemoryError

How many previous points do you have?

~1700 but a lot of those failed i think

Yeah if they all failed, then GP doesn't have much to go on.... Also, do you have negative gmrq scores or anything?

nah some definitely passed.


Regression in azure-functions/dotnet-isolated:4 Docker image

Hello, I am seeing a regression in the dotnet-isolated:4 Docker image. A few months ago, trying to run an Azure Function that was built inside this image resulted in the following error:
info: Host.Function.Console[0]
      It was not possible to find any compatible framework version
info: Host.Function.Console[0]
      The framework 'Microsoft.AspNetCore.App', version '6.0.0' (x64) was not found.
info: Host.Function.Console[0]
        - No frameworks were found.

which manifested as a 500 error when trying to invoke a function and the following exception in the Function runtime:
Microsoft.Azure.WebJobs.Script.Description.WorkerFunctionInvoker[0]
      Final functionDispatcher state: WorkerProcessRestarting. Initialization timed out and host is shutting down
fail: Function.Function1[3]
      Executed 'Functions.Function1'

I haven't seen this issue for a while but this week I have noticed it cropping up again. The workaround I have (which I modified from this issue: #441) is to add the following lines to my Dockerfile:
RUN apt-get update -y \
  && apt-get install -y apt-transport-https wget \
  && wget https://packages.microsoft.com/config/debian/11/packages-microsoft-prod.deb -O packages-microsoft-prod.deb \
  && dpkg -i packages-microsoft-prod.deb \
  && apt-get update -y \
  && apt-get install -y aspnetcore-runtime-6.0

This seems like a regression in your Docker image. Do you not package the ASP.NET Core 6.0 runtime into the Docker image?

The Microsoft.AspNetCore.App runtime was not included in the Linux images.  This was fixed in this PR: https://github.com/Azure/azure-functions-docker/pull/562. Rolling out this change is still in-progress. Will update this thread when it is done.
@VpOfEngineering FYI

Just chiming in that I ran into the same issue today. Thankfully, the workaround provided by @esimkowitz works for me too. 👍🏽

@kshyju @VpOfEngineering do you have any updates/ETAs on the rollout of the new Docker image version for azure-functions/dotnet-isolated:4?

We are currently in-progress to roll out the changes. Within the next week, you should see version 4.1.3 appear which includes the new image.

I will also post an update here when it is fully deployed

@kshyju @VpOfEngineering I just tried out 4.1.3 as the base image and it is still failing with the same error

Hi Evan, thank you for filing this important issue. I can see what is happening here. The changes that @kshyju introduced in #562 are available in the -appservice images provided by the Functions team.  Those changes were not included in the non-appservice images that you are using.  For now, you can build based on 4.1.3-appservice tag and you should be unblocked. Sorry about this. Will be taking steps to avoid these problems in the future and will be including aspnet in our next release of the images.

@CooperLink any update on the PR to get this change into all feeds? I see it's been open for 2 weeks

@CooperLink I see that the PR has been merged. Do you know when I can expect it to roll out to the dotnet-isolated:4 feed?

Hi @esimkowitz the rollout for dotnet-isolated:4 will be completed in the next release 4.2.0.  That release process should begin this week.  The image will be available at dotnet-isolated:4 about 1 week after that.  Expected ETA is within one month. For the time being you can use AppService images as mentioned. Sorry for the delay

I've tested this using the dotnet-isolated:4 image and everything looks good, I think we can close this issue


[WIP] copy meta schema json for mi aggregation copy

…of resource copy

Pull Request Checklist:

[ ] Positive Test Case Written by Dev

[ ] Automated Testing

[ ] Sufficient User and Developer Documentation

[ ] Passing Jenkins Build

[ ] Peer Code review and approval

Positive Test Case

[Enter positive test case here]

METRIC
VALUE

NEW VULNERABILITIES
0

NEW SECURITY HOTSPOTS
0

NEW BUGS
0

NEW DUPLICATED LINES DENSITY
60.1626016260163

NEW CODE SMELLS
0

https://sonarqube.cuahsi-workstation.com:9000/dashboard?id=hydroshare-4387


Changes to make rss bot working with zulip 3.3 and Debian 10.7 :

Add init method
Change return hashlib.md5 synthax

Signed-off-by: Manu LN<EMAIL_ADDRESS>Before changes, while trying to get rss infos in RSS stream the script output was
Traceback (most recent call last):
  File ".local/share/zulip/integrations/rss/rss-bot", line 190, in <module>
    entry_hash = compute_entry_hash(entry)  # type: str
  File ".local/share/zulip/integrations/rss/rss-bot", line 131, in compute_entry_hash
    return hashlib.md5(entry_id + str(entry_time)).hexdigest()
TypeError: Unicode-objects must be encoded before hashing

Merged, thanks for the PR @fulljackz!


Restart the tunnel when the client detect network change.

In the case of the device is going in sleep mode or connect in another network we should restore the dns configuration , re-apply the dns configuration and restart the tunnel.
On macosx i tested https://gist.github.com/LiamHaworth/d50ce9167fe0377cd20d6585d77a70b4 who is able to trigger a function on network change.
At start:
2020/12/21 10:59:42 Interface changed state: &net.Interface{Index:1, MTU:16384, Name:"lo0", HardwareAddr:net.HardwareAddr(nil), Flags:0x15}
2020/12/21 10:59:49 Interface changed state: &net.Interface{Index:15, MTU:1420, Name:"utun8", HardwareAddr:net.HardwareAddr(nil), Flags:0x19}

On change:
2020/12/21 11:19:10 Interface changed state: &net.Interface{Index:10, MTU:1484, Name:"awdl0", HardwareAddr:net.HardwareAddr{0xe, 0xa1, 0x41, 0xb4, 0xfb, 0x7d}, Flags:0x13}
2020/12/21 11:19:10 Interface changed state: &net.Interface{Index:8, MTU:1500, Name:"en0", HardwareAddr:net.HardwareAddr{0x54, 0x26, 0x96, 0xd9, 0xc8, 0xb3}, Flags:0x13}

This has been added in the client


introduce http_proxy variable in docker container

What is the problem that this PR is trying to fix?
This is to support http proxies in enterprise, in collaboration with @schultyy.
What approach did you choose and why?
Setting an environment variable on the docker container. In the long-term, it would probably be better to put this kind of logic into travis-build (especially to make this work across infras), but from what I've heard it is currently easier to roll out a newer version of worker on enterprise than travis-build.
So that's what we're going with for now.
How can you test this?
I'm testing it with a local worker. We can also test it with our enterprise test installation.
It can be tested by setting the environment variable:
export TRAVIS_WORKER_DOCKER_HTTP_PROXY=http://proxy:80880

That should make the http_proxy and HTTP_PROXY env vars set in the container.

Getting some docker version issues with my local docker install:
time="2018-10-30T14:11:29+01:00" level=error msg="couldn't run script, attempting requeue" completed=false err="\"env\" requires API version 1.25, but the Docker daemon API version is 1.24" instance_id=08ce1a9 job_id=730669 job_path=igorwwwwwwwwwwwwwwwwwwww/hello-world/jobs/730669 pid=85514<EMAIL_ADDRESS>repository=igorwwwwwwwwwwwwwwwwwwww/hello-world self=step_run_script

We have a worker image to test with: travisci/worker:v4.5.2-12-ga556057.

Looks like we need to set the API version the client uses. There's a config var for that:
export TRAVIS_WORKER_DOCKER_API_VERSION=1.37

Confirmed working!

Timeout waiting for network availability.
...
$ echo $http_proxy
http://proxy:80880

Tested this on a test installation for Enterprise and everything works out of the box. ✔

Thanks guys 🍺


Remove unnecessary app analytics tests

Signed-off-by: Eugene Lee<EMAIL_ADDRESS>Description
Remove App Analytics tests that are not necessary. This is to reduce flakiness.
Issues Resolved
#289
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Rerun https://github.com/opensearch-project/opensearch-dashboards-functional-test/actions/runs/3237324644 to see if it will succeed.

Any backport is needed? @eugenesk24

@eugenesk24 looks like the tests are failing https://github.com/opensearch-project/opensearch-dashboards-functional-test/actions/runs/3237324644/jobs/5392223936
Also if backport is needed, please add the corresponding backport label.

@joshuali925 Do you know if this PR is still needed? Could we close if no active development is on it?

not sure if it's still relevant, @mengweieric?

not sure if it's still relevant, @mengweieric?

@mengweieric Can you confirm? If no longer needed, we can close the PR since it has been sitting in the queue for a while.


WSL, Can't attach with rdbg: "your 131072x1 screen size is bogus, expect trouble"

I'm trying to debug a Remote WSL session using rdbg extension.
When i try to attach the rdbg process into VSCode, i get the following error:
your 131072x1 screen size is bogus. expect trouble
your 131072x1 screen size is bogus. expect trouble

I can't seem to control the size of the terminal that is spawned to run rdbg. So debugging is not feasible
Been looking for fixes in the VSCode repo, and they suggest to ignore this error silently:
VSCode workaround: https://github.com/microsoft/vscode/pull/104277/files
Same workaround, implemented in Omnisharp VSCode extension: https://github.com/OmniSharp/omnisharp-vscode/pull/3961/files
VSCode version: 1.70.2
Using WSL2 with Ubuntu-20.04
Extensions:
Remote - WSL v0.66.3
VSCode rdbg Ruby Debugger v0.0.11

Wow. I'm using WSL and I never see the error. How to repro? Need to buy a big display :p ?

Haha 😅
Nothing much interesting is done.
Here's a screencast of the flow:
https://user-images.githubusercontent.com/4783831/191789953-2247cdbb-b42a-4318-b9e1-98a9afa887f0.mp4

Okay, i gave it more thought to my attach config.
Added the "debugPort" configuration and also gave my rdbg process the flags to allow remote debugging in a certain port.
Back then:
rdbg -n --open=vscode
Now:
rdbg -n -O --port=12345
That way, i can debug without issue from vscode using the extension, and the error message dissapears.

Ah, do you reuse .vscode/ created by rdbg -n --open=vscode?

I ran into the exact same problem last night with my WSL2 setup.

It seems to randomly afflict different folks working with different Linux distros and tooling.
@Leo137 I appreciate your detective work, although you didn't provide literal steps. I'm going to share my configs exactly so that the next people coming along might suffer less in their journey.
Note that your paths and Ruby versions will certainly be different, future traveller. Here is my .vscode/launch.json:
{
  "version": "0.2.0",
  "configurations": [
    {
      "type": "rdbg",
      "name": "Attach with rdbg",
      "request": "attach",
      "rdbgPath": "/home/leastbad/.rvm/gems/ruby-3.1.3/gems/debug-1.7.1/exe/rdbg",
      "debugPort": "20138",
      "localfs": true
    }
  ]
}

web: rdbg -n -O --port=20138 -c -- bin/rails server -p 3000
js: yarn build --watch

I'm using esbuild and foreman/overmind, and so far as I can tell, the port is somewhat arbitrary so long as it's unused.
Order of operations is that you run bin/dev first, then click the "Attach with rdbg" button hiding in the top left of the VS Code UI.

Now, you can place a breakpoint in your code by clicking in the gutter, left of the line number:

Pretty cool! I'm still a passionate Ruby Jard user, but I'm going to give this whole IDE thing an honest try. I haven't used an IDE workflow since VB6!


Implement ApproxEq for all primitives

Currently only f32 and f64 are implemented.

[ ] (T, ..)
[ ] [T]
[ ] RefCell<T>
[ ] Cell<T>

We should write down a list of these!

Added some to the list!

I think @WaDelma was referring to other numeric primitive types like u32, i32, etc.
Though  I'm not sure what ulps would mean for them so it might not be worth it.

Hmm, yeah. Is this a problem with the API? What does approx mean for integers?

What about [T; n]? Could we add it to the list?

Hey! What's the status on tuples? I would find that very useful for lots and lots of my unit tests.


Include deluge as an alternative.

Include deluge as an alternative to rtorrent.
http://deluge-torrent.org/

I'm fine adding most alternatives for most software, but as an FYI I'm aiming so that EVERY script can be run without interfering with each other, meaning that every script can be run and every piece of software will function as expected without overwriting files, messing with incomplete downloads, etc.
Not undoable at all, simply something to keep in mind.


Babel-loader dependency missing

Describe the bug
Running latest rnv causes babel-loader problem.
To Reproduce
rnv run (web platform)
Expected behavior
That run works
Screenshots or copy&paste
https://hastebin.com/opokidazal

Additional context
Same codebase was working ok with 0.29.1-alpha.20. Rnv reran node_modules install but  have also ran bootstrap-clean within our project

should be fixed in 0.31.3


Textstyles lineheight and kerning

Description of changes:
Adds support for setting lineHeight (Line) and kerning (Character) to a TextStyle.

Only adds the properties to the object if the value is actually passed as an argument. By not setting this we still utilise the built in value in the font file. This is especially important for kerning since a value of 0 is not the same as not setting it. Setting kerning to 0 will force kerning pairs to use a fixed spacing of 0 instead of using the built in information in the font kerning pairs.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Amended last commit: (missed getter method for kerning after local rebase)

Wow thank you for this! If possible, could you add Typescript stuff?
https://github.com/amzn/sketch-constructor/blob/master/models/TextStyle/index.d.ts

TypeScript stuff added ✅ 😄

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[fast-lossless] Speedup and enable (if supported) AVX512 by default

With the RLE improvements and the changes here, AVX512 is about 50% faster than AVX2 on a i7-11850H.
Before, AVX2:
build/fast_lossless /tmp/noise-4.ppm
   245.864 MP/s
    14.668 bits/pixel
build/fast_lossless /tmp/noise-6.ppm
   236.304 MP/s
    20.813 bits/pixel
build/fast_lossless /tmp/noise-8.ppm
   241.526 MP/s
    26.878 bits/pixel
build/fast_lossless /tmp/noise-10.ppm
   164.410 MP/s
    32.878 bits/pixel
build/fast_lossless /tmp/noise-11.ppm
   161.683 MP/s
    35.897 bits/pixel
build/fast_lossless /tmp/noise-12.ppm
   159.196 MP/s
    38.912 bits/pixel
build/fast_lossless /tmp/noise-13.ppm
   159.249 MP/s
    41.925 bits/pixel
build/fast_lossless /tmp/noise-14.ppm
   141.446 MP/s
    48.705 bits/pixel
build/fast_lossless /tmp/noise-15.ppm
   117.555 MP/s
    62.039 bits/pixel
build/fast_lossless /tmp/noise-16.ppm
   119.843 MP/s
    66.092 bits/pixel
build/fast_lossless /data/jpeg_xl_data/imagecompression.info.rgb8/artificial.ppm
   401.069 MP/s
     2.677 bits/pixel
build/fast_lossless /data/jpeg_xl_data/imagecompression.info.rgb16/artificial.ppm
   185.927 MP/s
    10.235 bits/pixel

Before, AVX512:
build/fast_lossless /tmp/noise-4.ppm
   288.877 MP/s
    14.668 bits/pixel
build/fast_lossless /tmp/noise-6.ppm
   274.973 MP/s
    20.813 bits/pixel
build/fast_lossless /tmp/noise-8.ppm
   268.379 MP/s
    26.878 bits/pixel
build/fast_lossless /tmp/noise-10.ppm
   176.508 MP/s
    32.878 bits/pixel
build/fast_lossless /tmp/noise-11.ppm
   173.567 MP/s
    35.897 bits/pixel
build/fast_lossless /tmp/noise-12.ppm
   167.652 MP/s
    38.912 bits/pixel
build/fast_lossless /tmp/noise-13.ppm
   164.720 MP/s
    41.925 bits/pixel
build/fast_lossless /tmp/noise-14.ppm
   147.509 MP/s
    48.705 bits/pixel
build/fast_lossless /tmp/noise-15.ppm
   131.819 MP/s
    62.039 bits/pixel
build/fast_lossless /tmp/noise-16.ppm
   133.364 MP/s
    66.092 bits/pixel
build/fast_lossless /data/jpeg_xl_data/imagecompression.info.rgb8/artificial.ppm
   492.252 MP/s
     2.828 bits/pixel
build/fast_lossless /data/jpeg_xl_data/imagecompression.info.rgb16/artificial.ppm
   215.241 MP/s
    10.298 bits/pixel

After, AVX512:
build/fast_lossless /tmp/noise-4.ppm
   401.756 MP/s
    14.668 bits/pixel
build/fast_lossless /tmp/noise-6.ppm
   380.501 MP/s
    20.813 bits/pixel
build/fast_lossless /tmp/noise-8.ppm
   359.672 MP/s
    26.878 bits/pixel
build/fast_lossless /tmp/noise-10.ppm
   232.566 MP/s
    32.878 bits/pixel
build/fast_lossless /tmp/noise-11.ppm
   228.881 MP/s
    35.897 bits/pixel
build/fast_lossless /tmp/noise-12.ppm
   221.053 MP/s
    38.912 bits/pixel
build/fast_lossless /tmp/noise-13.ppm
   224.517 MP/s
    41.925 bits/pixel
build/fast_lossless /tmp/noise-14.ppm
   216.443 MP/s
    48.705 bits/pixel
build/fast_lossless /tmp/noise-15.ppm
   196.745 MP/s
    62.039 bits/pixel
build/fast_lossless /tmp/noise-16.ppm
   198.877 MP/s
    66.092 bits/pixel
build/fast_lossless /data/jpeg_xl_data/imagecompression.info.rgb8/artificial.ppm
   598.934 MP/s
     2.828 bits/pixel
build/fast_lossless /data/jpeg_xl_data/imagecompression.info.rgb16/artificial.ppm
   305.824 MP/s
    10.298 bits/pixel

While avx512 now makes a lot more sense to use by default, I am not entirely comfortable with the idea of getting slightly worse compression just because you have a cpu that can do wider SIMD.
In particular, I can imagine that you'd still want to have the slightly better compression and use avx2 only, which is still much faster than e2.
Is there a way to make avx512 and avx2 produce the same output? If not, then maybe we should add an e0 that does the simd-width-dependent thing, and keep e1 like it is now, independent of simd width (even if that means not using avx512, or at least not where it makes a difference in compression).

While avx512 now makes a lot more sense to use by default, I am not entirely comfortable with the idea of getting slightly worse compression just because you have a cpu that can do wider SIMD. In particular, I can imagine that you'd still want to have the slightly better compression and use avx2 only, which is still much faster than e2.
Is there a way to make avx512 and avx2 produce the same output? If not, then maybe we should add an e0 that does the simd-width-dependent thing, and keep e1 like it is now, independent of simd width (even if that means not using avx512, or at least not where it makes a difference in compression).

You already get this behaviour anyway with avx2 Vs no-avx (which get somewhat better compression still). Given that the difference is minimal (a couple of % in rle-heavy areas), I wouldn't be too worried.

I'm with Luca -- I strongly believe that especially with fast lossless and other fastest encoding options we need to match the coding decisions to the simd width, but we may want to do that even in slower modes, depending on the density/speed impact.
In the fast lossless the weight on density is pretty minimal and we just need a fast coding for the use case of loading an album of images, applying a color correction/processing and saving them back. Think 500 studio quality photographs processed this way. This kind of speed improvements are the thing that put JPEG XL on the map for raw-like storage for photo editing tools. This is not a fully theoretical use case, but one where JPEG XL already is the best performer -- so making it stronger will further increase the appeal and utility.

For the artificial.ppm image, the compression difference is a bit over 5%. This is an image that benefits from rle, but maybe for some screenshot images the difference is even larger.
Of course 600 Mp/s instead of 400 MP/s is very nice, but for most 'normal' use cases, both are just 'more than fast enough' / 'image saves instantly'. In that sense I think 5% compression gain can be preferable over 50% speed improvement.
It's mostly just confusing ux, imo, that doing the same thing from the user pov (saving an image at e1) can produce a noticeably larger file after getting a better computer, while the time to encode is probably not noticeably different (going from 10ms to 7ms is not something humans can generally appreciate).
I am not opposed to doing that anyway by default, but there should at least be an option (preferably a runtime option) to get the 'consistent result' behavior as opposed to the 'best trade-off' behavior.

Ideally there would some more guiding comments on the purpose of the functions.

git and GitHub made an absolute mess at rendering the diff - in practice there are only three new functions (AppendBytesWithBitOffset, StoreToWriter and StoreToWriterAVX512), whose name and type signature I think make the purpose sufficiently clear.

For the artificial.ppm image, the compression difference is a bit over 5%. This is an image that benefits from rle, but maybe for some screenshot images the difference is even larger.
Of course 600 Mp/s instead of 400 MP/s is very nice, but for most 'normal' use cases, both are just 'more than fast enough' / 'image saves instantly'. In that sense I think 5% compression gain can be preferable over 50% speed improvement.
It's mostly just confusing ux, imo, that doing the same thing from the user pov (saving an image at e1) can produce a noticeably larger file after getting a better computer, while the time to encode is probably not noticeably different (going from 10ms to 7ms is not something humans can generally appreciate).
I am not opposed to doing that anyway by default, but there should at least be an option (preferably a runtime option) to get the 'consistent result' behavior as opposed to the 'best trade-off' behavior.

The way I see it, -e 1 = absolute fastest you can get with decent compression, and -e 2 is where you start making more meaningful tradeoffs between speed and compression. There is I think some space for a fancier encoding method at effort 2, for example by doing very simple generic lz77 and having a simple decision tree.

Jon, you can relatively easily increase consistency by not using the avx/avx2/avx512 code paths, but I'd consider it a rare need. I believe in some compilations we generate the code for several simd paths, so it would be technically possible to dynamically choose weaker codes when needed.
Consider discussing this with Jan on highway -- possibly it should be solved and supported at highway level rather than just hacked into libjxl.


Remove log timing

It's unused and I don't like having unused code in repo. if we need it we can still add it back.


[Form] Avoid display error block if empty

Example: http://jsbin.com/mexanositi/edit?html,js,output
Suppose I have a section for displaying error messages within a form; in the example assume such messages could come from the server, regardless from the client-side validation.
Setting true the inline option, error section becomes visible even when empty. I think it could be a way and it would be good idea to avoid this. What do you think?

HTML:
<form class="ui form segment" action="#">

    Label
    <input type="text" name="field">

Submit

</form>

JavaScript:
$('form').form({
    inline: true,
    fields: {
      field: {
        identifier  : 'field',
        rules: [{
            type   : 'empty',
            prompt : 'Please enter a value'
        }]
      }
    }
})

/*
 For later add error messages with something like:
 $('form').form('add errors', messages).addClass('error')
*/

Result:

I do not know CSS, but this patch seems to work at least in Firefox 40.0 and Chrome 44.0.2403.155
diff --git a/src/definitions/collections/form.less b/src/definitions/collections/form.less
index 6d10fbd..d3f0ad9 100755
--- a/src/definitions/collections/form.less
+++ b/src/definitions/collections/form.less
@@ -358,7 +358,7 @@
 ---------------------*/

 /* On Form */
-.ui.form.success .success.message {
+.ui.form.success .success.message:not(:empty) {
   display: block;
 }
 .ui.form.success .icon.success.message {
@@ -366,11 +366,11 @@
 }

 /*--------------------
-        Error
+        Warning
 ---------------------*/

 /* On Form */
-.ui.form.warning .warning.message {
+.ui.form.warning .warning.message:not(:empty) {
   display: block;
 }
 .ui.form.warning .icon.warning.message {
@@ -378,11 +378,11 @@
 }

 /*--------------------
-        Warning
+        Error
 ---------------------*/

 /* On Form */
-.ui.form.error .error.message {
+.ui.form.error .error.message:not(:empty) {
   display: block;
 }
 .ui.form.error .icon.error.message {

Should I make a pull request?

Empty is a bit of an issue since text nodes don't count for the pseudo selector.
So

   Warning you did something bad

Would actually not show up.

You mean if I can just add the div.error block dynamically?... I could, but I like the idea of indicating the location explicitly in the code, you know ... for semantic :smile:
I'm trying with this simple code and the block show for me: http://jsbin.com/porugo/edit?html,css,output ... maybe the issue you mention is with a specific browser or I didn't understand?
 I'm thinking about :empty js selector which does include text nodes

Excellent!. Thank you so much.
Only one minor observation. In my patch, I also corrected the "header comments". It said "Error" when should be "Warning" and vice versa.
Something unimportant, but ... :stuck_out_tongue_winking_eye:


My flutter app does not run!

I cannot run or build any flutter app

I even created a new app, i just ran the code without making any changes to it and it gives me the same thing..

I have had flutter running previously on my machine, i stopped using flutter for like 2 months and now it does not build and run any app..
Using hardware rendering with device AOSP on IA Emulator. If you get graphics artifacts,
consider enabling software rendering with "--enable-software-rendering".
Launching lib\main.dart on AOSP on IA Emulator in debug mode...
Initializing gradle...
(This is taking an unexpectedly long time.)       \
Sending crash report to Google.
Initializing gradle...
Crash report sent (report ID: 6d91bad9874e629a)
Initializing gradle...
(This is taking an unexpectedly long time.)       |Oops; flutter has exited unexpectedly.
(This is taking an unexpectedly long time.)
Crash report written to C:\Users\me\code\mark\app\mark\flutter_02.log;
please let us know at https://github.com/flutter/flutter/issues.
(This is taking an unexpectedly long time.)       /

Hi @marioallaa
have you tried following the instruction in the logs?
Using hardware rendering with device AOSP on IA Emulator. If you get graphics artifacts,
consider enabling software rendering with "--enable-software-rendering".

I see from the pictures that some log has been generated, could please attach it?
Please refrain from posting pictures if not necessary.
thank you

Using hardware rendering with device AOSP on IA Emulator. If you get graphics artifacts,
consider enabling software rendering with "--enable-software-rendering".

it worked when i ran flutter run --enable-software-rendering
Thank YOU!

Could everyone who still has this problem please file a new issue with the exact descriptions what happens, logs and the output of 'flutter doctor -v'  please.
All system setups can be slightly different so it's always better to open new issues and reference related issues.

Above is the error i get when i run flutter.

This was the outcome,when i ran flutter doctor

i have the exact same problem please help me

Could everyone who still has this problem please file a new issue with the exact description of what happens, logs and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


#106 and #94 support for associated researcher tab in metadata template

Added credit record, has credit record, has contributor role, applies to person to nmdc.yaml for Issues #106 and #94
All in support of AssociatedResearchers tab in metadata template https://docs.google.com/spreadsheets/d/1INlBo5eoqn2efn4H2P2i8rwRBtnbDVTqXrochJEAPko/edit#gid=795994701

@turbomam Thanks for putting this together. I think we should discuss this at tomorrow's metadata meeting. This impacts how I've been doing GOLD ETL. It may impact Donny too (but I'm not sure).
cc @cmungall @dehays

Thanks for reviewing @wdduncan. Yes, I look forward to discussing tomorrow if possible.
How does the addition of new classes and slots affect your existing GOLD ETL? If there's an existing mechanism for associated researchers, I will use that.

@turbomam You rightly point out that there isn't a slot for co-investigators (and the like). GOLD only has information concerning PIs, and the ETL process does capture that. However, this change may require how the ETL works on GOLD for PI information.

Here's my current parse of the ProjectInformation and AssociatedResearchers tabs
I'm not trying to say that this is good, necessary or whatever. Just showing my work.  Nothing has been tidied or validated. (For example, the various ID values).
@wdduncan is the different shape between the established principal_investigator and my proposed has credit record part of the problem?
{
  "id": "GOLD:Gs01971387",
  "INSDC bioproject identifiers": "NCBI Accession: PRJNA594403",
  "GOLD study identifiers": "GOLD:Gs01971387",
  "alternate identifiers": [
    "EMSL:29728",
    "JGI:1781",
    "NCBI ID: 594403"
  ],
  "principal_investigator": {
    "has raw value": "Montana Smith",
    "orcid": "0000-0002-8683-0050",
    "email"<EMAIL_ADDRESS>  },
  "has credit record": [
    {
      "type": "credit record",
      "has contributor role": "Project administration",
      "applies to person": {
        "has_raw_value": "Pajau Vangay",
        "orcid": "0000-0002-9231-0692",
        "email"<EMAIL_ADDRESS>      }
    }
  ],
  "title": "Soil microbial response to elevated temperatures and increased carbon availability",
  "description": "Northern latitude peatlands contain large reserves of soil carbon (C) due in part to climatic constraints on decomposition, including low temperatures and water inundated soils.... As expected, the two LMWCC substrate additions increased C respiration....",
  "type": "Study",
  "websites": [
    "https://microbiomedata.org/",
    "https://github.com/microbiomedata"
  ],
  "publications": [
    "https://doi.org/10.1016/j.soilbio.2019.107561",
    "Keiser, Ashley D., et al. Peatland microbial community response to altered climate tempered by nutrient availability. Soil Biology and Biochemistry 137 (2019): 107561."
  ],
  "doi": {
    "has raw value": "https://doi.org/10.25585/1487765"
  }
}

The question is whether we want both a principal investigator slot and a credit record, or just a credit record that includes the principal investigator.

I think the above pattern is more complicated than what we need. From what I've gathered about Montana's needs we have a situation in which studies have researchers assigned to them, and each researcher has a role (or more than one role?) within the study.
IMHO, a more direct route to do this would be to create a researchers slot with range person value, and had a has role slot to person value. E.g.:
study:
  id: ....
  researchers:
    - {has raw value: Jane Doe,  has role: PI}
    - {has raw value: John Doe, has role: co-PI}

If we don't want the range of has roleto be a string, we can create a role class to allow for more information to captured. E.g.:
researchers:
    - has raw value: Jane Doe
    - has role:
         has raw value: PI
         start date: ...
         end date: ...

My suspicion, however, is that after all is said and done, all we need to be able to do is query the researchers that worked on a study. In which case, the simplest thing to do is to have a slot for each kind researcher on the study. E.g.:
study:
  id: ....
  principal investigator: {has raw value: Jane Doe}
  co-investigator:
    - {has raw value: John Doe}
    - {has raw value: Foo}
    - {has raw value: Bar}
  project administrator: {has raw value: King Tut}

Note: In other projects, I've seen studies with more than one PI. So, we may need to allow for this in NMDC too.

I like the idea of making a slot for each kind of contribution if the number of contributions isn't too high. Would we really allow the use of all 14 CRedit terms plus OBI's term for principal investigator?

@turbomam Where did you get the credit enum from? Did someone provide you a list of the roles in the projects?

@turbomam Where did you get the credit enum from? Did someone provide you a list of the roles in the projects?

https://casrai.org/credit/ + OBI's principal investigator role (which could be omitted)

2. Since, has_credit_association takes an array, can you please as an "s": has_credit_associations.

Done

3. You a single role (e.g., Data curation) applying to multiple person values. This is going to be difficult to write a change sheet for. It would be better if has_credit_associations took an array of objects, each object being a different credit role object.

So you want this completely denormalized? Neither the applied role or applies to person should be multivalued? Done.

From Skype:
Bill, 11:53 AM
I approved the PR. Please merge after you write the test.


Can someone implement the equivalent of torch.cat(...) or np.concatenate(...) for grad tensors in TinyGrad?

It would be easier for implementing Transformer based models (BERT, ViT, ...), for example for using the CLS and SEP tokens...

Its probably far from efficient but would this work?
`def cat(tensors, dim=0):
num_dims = len(tensors[0].shape)
# So you can set dim=-1 for last dim
if dim < 0:
dim = dim + num_dims
last_dim = num_dims-1

# If dim is not last, transpose tensor so that the concat dim is last
if dim != last_dim:
    order = np.arange(len(tensors[0].shape))
    order[dim] = last_dim
    order[last_dim] = dim
    tensors = [t.transpose(order=order) for t in tensors]

sizes = [t.shape[-1] for t in tensors]
# Size of result along concat dim
total_size = sum(sizes)

iden = Tensor.eye(total_size)
# Start result as all zeros
concat = Tensor.zeros(*tensors[0].shape[:-1], total_size)
start = 0
for i,t in enumerate(tensors):
    # dot each tensor with a slice of the identity so it has the same size as concat and it's values
    # are in the right spots with zeros everywhere else
    concat = concat.add(t.dot(iden[start:start+sizes[i], :]))
    start += sizes[i]

if dim != last_dim:
    # Undo transpose
    concat = concat.transpose(order=order)
return concat`

can you implement a test and compare your algo of cat wrt torch and numpy (in terms of execution time for example)?

Oh I like this!

Hey, I tested this code for the cat, here some plots:

Linear
Log Scale

in this test I compared the cat function of tinygrad (above), numpy and torch by varying the inputs sizes. Here an idea of the code used:
def measure_time(func, *args, **kwargs):
  to = time.time()
  out = func(*args, **kwargs)
  return out, time.time() - t0

# tinygrad
x1, x2 = Tensor.randn(1, size), Tensor.randn(1, size)
x, t = measure_time(cat, [x1, x2], dim=1) # your function above

# numpy
x1, x2 = np.random.randn(1, size), np.random.randn(1, size)
x, t = measure_time(np.concatenate, [x1, x2], dim=1)

# torch
x1, x2 = torch.randn(1, size), torch.randn(1, size)
x, t = measure_time(torch.cat, [x1, x2], dim=1)

In the plots the measurements are collected num_trials=10 times, and I highlighted the min and the max (colored region) and the mean (line).
It seems that your version has a 'squared' dependency respect to the input size, whereas it seems constant for numpy and torch...
How can we improve that?

I think the backward pass would probably be a lot worse than pytorch's cat too.
I don't know if improving on my algorithm is the way to go. The main concern when writing it was to avoid using anything other than the existing 14 ops. If you're concerned about performance it'd probably be better to just write a new cat op at the same level as the other ops (I think there's already a pull request working on this).
Do you need better performance though? Roughly how many tensors are you going to be concatenating in a real scenario?

Warning about the code I wrote! The backward pass won't work with tensors of rank > 2. I think it's because of how the dot backward pass is implemented.

I agree this is a real issue. I think you can write this as a second order op, Slice (if you go bigger, it pads) + Add.

Done


update sfdx schema to latest, add managed package support to schema extensions

chore: update sfdx JSON schema from forcedotcom, reformatted according forcedotcom repo
feat: add "managed" enum member to "packageDirectory.type"
Checklist
All items have to be completed before a PR is merged

[x] Adhere to Contribution Guidelines
[x] Updates to Decision Records considered?
[x] Updates to documentation at DX@Scale Guide considered?
[x] Tested changes?
[x] Unit Tests new and existing passing locally?

Will revisit this in subsequent release, new versions have schema which is relaxed


What's needed to add prediction intervals for GammaGAM?

I'm using a GammaGAM for forecasting, and I could really use prediction_intervals like they are available in LinearGAM. I understand it's a bit more complicated, and I could give it a shot, but I don't know what's necessary to implement them.
It seems there was an issue #75 already that talked about prediction intervals for the Poisson distribution, but that has been closed (unsolved, right?)
Thanks in advance for any hints!

hey @AlexEngelhardt !
sorry for the slowwww response.
the way to do this for the general case it to use the empirical sampling approach (there's some discussion here)
but here's a quick recap:
the idea is to fit a few models on bootstrapped replicates of the dataset, then generate new data from the fitted models, and compute the percentiles of those data.

import numpy as np
from matplotlib import pyplot as plt

from pygam import PoissonGAM, s, te
from pygam.datasets import chicago

X, y = chicago()
gam = PoissonGAM(s(0, n_splines=200) + te(3, 1) + s(2)).fit(X, y)

# sample 500 new predictions by fitting 5 models on bootstraps of the data
samples = gam.sample(X, y, quantity='y', n_draws=500, sample_at_X=X, n_bootstraps=5)

# now compute percentiles of the sampled data
q = [2.5, 97.5]
percentiles = np.percentile(samples, q=q, axis=0).T

# and plot
plt.plot(y[:500])
plt.plot(percentiles[:500])

@AlexEngelhardt did this work for you?

Hi Daniel,
sorry from me too - I took a long time to respond :)
Your code worked perfectly. Do you consider this just a workaround or do you think this would fit into pyGAM as well? If yes, I can put it in and submit a pull request.

@alexengelhardt im very glad it worked for you.
this is the correct way to compute prediction intervals for non-gaussian distributions.
it definitely belongs in the prediction_intervals method for non-gaussian models.
i would love to get a pull request for this!!
-dani

it should be easy to close https://github.com/dswah/pyGAM/issues/156 with this method.


Sync eng/common directory with azure-sdk-tools for PR 8879

Sync eng/common directory with azure-sdk-tools for PR https://github.com/Azure/azure-sdk-tools/pull/8879  See eng/common workflow

/check-enforcer override
python - template - ci build completed without check suite update


[SPARK-28599][WEBUI][2.4] Support sorting Execution Time and Duration columns for ThriftServerSessionPage

What changes were proposed in this pull request?
This PR add support for sorting Duration column for ThriftServerSessionPage.
Why are the changes needed?
Sorting on duration columns consider time as a string after converting into readable form which is the reason for wrong sort results as mentioned in SPARK_28599.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Tested manually.
Back-port of https://github.com/apache/spark/pull/25892

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Please do not make a PR for the unmarked PR.

Got a lesson. Thank you @dongjoon-hyun.
One query, can we re-open this PR after PR https://github.com/apache/spark/pull/25892 get merged?

No, this PR basically takes @wangyum 's credit.
In the Open Source community, the code authorship is considered very important.

This sounds fair. I will keep this in mind.
Thanks again @dongjoon-hyun .

You can make a backporting PR with @wangyum 's authorship. When the committers merge it, the authorship will be @wangyum and usually the person who made a backporting PR is not visible in branch-2.4.

Still want to back-port 51d350942865302e33480c83324704a54a24a494 to branch-2.4 and soon raise a PR.
Thanks @wangyum and @dongjoon-hyun


Closes: #1

Updated validations.py python script.
Fixed the behavior of validate_user function in validations.py. Now it accepts only usernames starting with letters
Fixes #1

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

Please review


Laravel 6 support

Thanks for opening a PR! I'll continue in #21: I need to update the README and other dependencies.
Thanks


[Issue]: Memcpy to device burns 100% CPU in WaitRelaxed if long-running kernels are present

Edit for future readers who find this on Google:
This is intentional behavior!
CUDA does the same thing. Google hipSetDeviceFlags(hipDeviceScheduleBlockingSync) for how to turn it off.
Previous text:
Problem Description
I've run into this with ComfyUI, but there's been variants of this issue a few times already (just search the issues for WaitRelaxed), but there's usually been some other issue that was fixed eventually, so I want to go ahead and file a bug for just WaitRelaxed burning 100% CPU.
As far as I can tell, this goes back to DmaBlitManager::hsaCopyStaged calling releaseMemoryFence, which, via CpuWaitForSignal, goes into a HSA_WAIT_STATE_ACTIVE WaitRelaxed call, which then spends 80% of CPU load on the now() call in the loop. To add insult to injury, g_enable_mwaitx has somehow become false even though my CPU definitely supports it.
So it just hot-busyloops for many milliseconds waiting for the kernel to finish.
Operating System
Ubuntu 24.04
CPU
AMD Ryzen 7950X3D
GPU
AMD Radeon RX 7900 XTX
ROCm Version
ROCm 6.1.0
ROCm Component
clr
Steps to Reproduce

Launch ComfyUI
Set latent size to 2048x2048 to ensure some nice long-running kernels
Queue Prompt
Observe high CPU load.
Observe rdtsc taking >80% in profiler.

(Optional for Linux users) Output of /opt/rocm/bin/rocminfo --support
ROCk module is loaded
HSA System Attributes
Runtime Version:         1.13
Runtime Ext Version:     1.4
System Timestamp Freq.:  1000.000000MHz
Sig. Max Wait Duration:<PHONE_NUMBER>3709551615 (0xFFFFFFFFFFFFFFFF) (timestamp count)
Machine Model:           LARGE
System Endianness:       LITTLE
Mwaitx:                  DISABLED
DMAbuf Support:          YES
==========
HSA Agents

Agent 1

Name:                    AMD Ryzen 9 7950X3D 16-Core Processor
Uuid:                    CPU-XX
Marketing Name:          AMD Ryzen 9 7950X3D 16-Core Processor
Vendor Name:             CPU
Feature:                 None specified
Profile:                 FULL_PROFILE
Float Round Mode:        NEAR
Max Queue Number:        0(0x0)
Queue Min Size:          0(0x0)
Queue Max Size:          0(0x0)
Queue Type:              MULTI
Node:                    0
Device Type:             CPU
Cache Info:
L1:                      32768(0x8000) KB
Chip ID:                 0(0x0)
ASIC Revision:           0(0x0)
Cacheline Size:          64(0x40)
Max Clock Freq. (MHz):   5759
BDFID:                   0
Internal Node ID:        0
Compute Unit:            32
SIMDs per CU:            0
Shader Engines:          0
Shader Arrs. per Eng.:   0
WatchPts on Addr. Ranges:1
Features:                None
Pool Info:
Pool 1
Segment:                 GLOBAL; FLAGS: FINE GRAINED
Size:                    32568860(0x1f0f61c) KB
Allocatable:             TRUE
Alloc Granule:           4KB
Alloc Recommended Granule:4KB
Alloc Alignment:         4KB
Accessible by all:       TRUE
Pool 2
Segment:                 GLOBAL; FLAGS: KERNARG, FINE GRAINED
Size:                    32568860(0x1f0f61c) KB
Allocatable:             TRUE
Alloc Granule:           4KB
Alloc Recommended Granule:4KB
Alloc Alignment:         4KB
Accessible by all:       TRUE
Pool 3
Segment:                 GLOBAL; FLAGS: COARSE GRAINED
Size:                    32568860(0x1f0f61c) KB
Allocatable:             TRUE
Alloc Granule:           4KB
Alloc Recommended Granule:4KB
Alloc Alignment:         4KB
Accessible by all:       TRUE
ISA Info:

Agent 2

Name:                    gfx1100
Uuid:                    GPU-7b8755ac387dcb5c
Marketing Name:          Radeon RX 7900 XTX
Vendor Name:             AMD
Feature:                 KERNEL_DISPATCH
Profile:                 BASE_PROFILE
Float Round Mode:        NEAR
Max Queue Number:        128(0x80)
Queue Min Size:          64(0x40)
Queue Max Size:          131072(0x20000)
Queue Type:              MULTI
Node:                    1
Device Type:             GPU
Cache Info:
L1:                      32(0x20) KB
L2:                      6144(0x1800) KB
L3:                      98304(0x18000) KB
Chip ID:                 29772(0x744c)
ASIC Revision:           0(0x0)
Cacheline Size:          64(0x40)
Max Clock Freq. (MHz):   2526
BDFID:                   768
Internal Node ID:        1
Compute Unit:            96
SIMDs per CU:            2
Shader Engines:          6
Shader Arrs. per Eng.:   2
WatchPts on Addr. Ranges:4
Coherent Host Access:    FALSE
Features:                KERNEL_DISPATCH
Fast F16 Operation:      TRUE
Wavefront Size:          32(0x20)
Workgroup Max Size:      1024(0x400)
Workgroup Max Size per Dimension:
x                        1024(0x400)
y                        1024(0x400)
z                        1024(0x400)
Max Waves Per CU:        32(0x20)
Max Work-item Per CU:    1024(0x400)
Grid Max Size:<PHONE_NUMBER>(0xffffffff)
Grid Max Size per Dimension:
x<PHONE_NUMBER>(0xffffffff)
y<PHONE_NUMBER>(0xffffffff)
z<PHONE_NUMBER>(0xffffffff)
Max fbarriers/Workgrp:   32
Packet Processor uCode:: 202
SDMA engine uCode::      20
IOMMU Support::          None
Pool Info:
Pool 1
Segment:                 GLOBAL; FLAGS: COARSE GRAINED
Size:                    25149440(0x17fc000) KB
Allocatable:             TRUE
Alloc Granule:           4KB
Alloc Recommended Granule:2048KB
Alloc Alignment:         4KB
Accessible by all:       FALSE
Pool 2
Segment:                 GLOBAL; FLAGS: EXTENDED FINE GRAINED
Size:                    25149440(0x17fc000) KB
Allocatable:             TRUE
Alloc Granule:           4KB
Alloc Recommended Granule:2048KB
Alloc Alignment:         4KB
Accessible by all:       FALSE
Pool 3
Segment:                 GROUP
Size:                    64(0x40) KB
Allocatable:             FALSE
Alloc Granule:           0KB
Alloc Recommended Granule:0KB
Alloc Alignment:         0KB
Accessible by all:       FALSE
ISA Info:
ISA 1
Name:                    amdgcn-amd-amdhsa--gfx1100
Machine Models:          HSA_MACHINE_MODEL_LARGE
Profiles:                HSA_PROFILE_BASE
Default Rounding Mode:   NEAR
Default Rounding Mode:   NEAR
Fast f16:                TRUE
Workgroup Max Size:      1024(0x400)
Workgroup Max Size per Dimension:
x                        1024(0x400)
y                        1024(0x400)
z                        1024(0x400)
Grid Max Size:<PHONE_NUMBER>(0xffffffff)
Grid Max Size per Dimension:
x<PHONE_NUMBER>(0xffffffff)
y<PHONE_NUMBER>(0xffffffff)
z<PHONE_NUMBER>(0xffffffff)
FBarrier Max Size:       32
*** Done ***
Additional Information
No response

@FeepingCreature Internal ticket has been created to investigate this issue. Thanks!

Hi @FeepingCreature, are you able to reproduce this on ROCm 6.2? Also, to clarify, does your kernel proceed while the CPU is spinning or are you getting a hang? I'm currently investigating some other issues with DMA and this might be related. Thanks!

It's proceeding fine, the Python process is just additionally sitting on high CPU. Yes this happens  on 6.2, the dropdown just didn't have that as an option when I filed the bug.

Thanks for the quick response! A few follow-up questions: Is this a new issue you're running into, or have you experienced this CPU spin with multiple configurations (i.e. on other distros, older versions, or on other hardware)? Is the CPU usage constantly high throughout? Does this issue appear only on large workloads like the one you've provided, or are you seeing the CPU spin on smaller tasks as well?

I've always seen Python at high CPU load with ComfyUI, but I never looked at it in a profiler before so I can't confirm it's the same function. Also to be safe I just reproed the issue on a clean ComfyUI install with AnythingXL. Also, the issue seems constant no matter how big I make the tasks, ie. if I run 256x256 for 200 steps, rdtsc still dominates the profiler, though a lot more is burned in pure Python: in that one, rdtsc sits at 18%. Which is still a lot! I suspect the 80% are burnt by Python before Rocm gets a chance to burn it. Only thing I could do to make it cleaner is boot into a Livecd, does that seem worthwhile? Also, do you want me to just upload a perf capture somewhere?

I think a clean install could be helpful just to cover all the bases, but I don't think it's likely to fix your problem, so that's up to your discretion. A configuration issue is possible since high CPU usage doesn't seem to be commonly reported for this workflow, but that could manifest from sources outside of the ROCm install. A profile would be interesting; one possibility is that the spinlock is intentional but the memory transfer that's blocking is taking longer than it should. What settings are you using in ComfyUI, so I can try to reproduce your exact issue?

It happens with the default workflow. I'm testing with AnythingXL, but I really doubt it depends on which SDXL checkpoint you use. I'm trying to set up a clean 22.04 boot and rocm/pytorch install off the 22.04.2 livecd on my end currently, will report back how that behaves.

Yep, reproduced on a livecd chroot. I haven't done these exact steps, but that suggests this should work:

Install Ubuntu 22.04.4
apt update
apt full-upgrade
apt install "linux-headers-$(uname -r)" "linux-modules-extra-$(uname -r)"
usermod -a -G render,video $LOGNAME
wget https://repo.radeon.com/amdgpu-install/6.2/ubuntu/jammy/amdgpu-install_6.2.60200-1_all.deb
apt install ./amdgpu-install_6.2.60200-1_all.deb
apt update
apt install amdgpu-dkms rocm
modprobe amdgpu
apt install linux-tools-common linux-tools-5.15.0-94-generic # for perf
apt install python3-pip
`pip3 install --pre torch torchvision torchaudio --index-url https://download.pytorch.org/whl/nightly/rocm6.1
git clone https://github.com/comfyanonymous/ComfyUI.git
cd ComfyUI
pip3 install -r requirements.txt
download a model such as SDXL or https://civitai.com/models/9409/or-anything-xl to models/checkpoints
python3 main.py --listen
confirm that running with 1024x1024 evinces 100% CPU load in htop while the KSampler in the default workflow is running, and that libhsa-runtime64.so tops the chart in perf top
apt install hsa-rocr-dbgsym
give torch the system libhsa runtime so we can profile it
/home/ubuntu-server/.local/lib/python3.10/site-packages/torch/lib/libhsa-runtime64.so
ln -s /opt/rocm-6.2.0/lib/libhsa-runtime64.so.1.14.60200 /home/ubuntu-server/.local/lib/python3.10/site-packages/torch/lib/libhsa-runtime64.so
observe that rocr::core::InterruptSignal::WaitRelaxed leads the chart while ComfyUI runs.

Thanks for the detailed reproduction instructions! My setup is on Ubuntu 22.04 with ROCm 6.2, with a 7900XTX and 5995WX. I set up ComfyUI with Anything XL and ran it with latent size 1024x1024. Instead of using the nightly torch builds, however, I used specific versions of torch, torchvision, and triton confirmed for ROCm 6.2 support:
wget https://repo.radeon.com/rocm/manylinux/rocm-rel-6.2/torch-2.3.0%2Brocm6.2.0-cp310-cp310-linux_x86_64.whl
wget https://repo.radeon.com/rocm/manylinux/rocm-rel-6.2/torchvision-0.18.0%2Brocm6.2.0-cp310-cp310-linux_x86_64.whl
wget https://repo.radeon.com/rocm/manylinux/rocm-rel-6.2/pytorch_triton_rocm-2.3.0%2Brocm<IP_ADDRESS>0b42334-cp310-cp310-linux_x86_64.whl
pip3 uninstall torch torchvision pytorch-triton-rocm numpy
pip3 install torch-2.3.0+rocm6.2.0-cp310-cp310-linux_x86_64.whl torchvision-0.18.0+rocm6.2.0-cp310-cp310-linux_x86_64.whl pytorch_triton_rocm-2.3.0+rocm<IP_ADDRESS>0b42334-cp310-cp310-linux_x86_64.whl numpy==1.26.4

With this setup, single CPU threads occasionally spin up to 100%, but only one thread at a time. After looking into the source code in our stack where you've pointed (i.e. the WaitRelaxed call), I believe this is intended behavior, as the thread is spinlocking waiting for a memory transfer to complete. If you're observing more than one thread spinning, try those versions of torch, torchvision, and triton.

It sort of looks like WaitRelaxed is supposed to be switching to an event-based sleep. I'm not sure why it doesn't happen.

The matter is starting to clear up. Apparently hipSetDeviceFlags lets you deliberately opt in to this behavior. Trying to figure out who sets this and why.

Resolution! Apparently this is behavior that's been copied from CUDA, where it also occurs. https://github.com/pytorch/pytorch/issues/28224 So I just have to figure out how to hipSetDeviceFlags(hipDeviceScheduleBlockingSync) from Python somehow.
For skimming future readers:
This is intentional behavior.

import ctypes

# Load the HIP runtime library
hip = ctypes.CDLL('libamdhip64.so')

# Define the function prototype
hip.hipSetDeviceFlags.argtypes = [ctypes.c_uint]
hip.hipSetDeviceFlags.restype = ctypes.c_int

# Define the flag value
cudaDeviceScheduleBlockingSync = 0x04

# Call hipSetDeviceFlags
result = hip.hipSetDeviceFlags(cudaDeviceScheduleBlockingSync)

# Check the result
if result != 0:
    print(f"hipSetDeviceFlags failed with error code {result}")
else:
    print("hipSetDeviceFlags succeeded")

I dropped this code (thanks Claude 3.5 Sonnet!) at the top of main.py, and CPU load drops to 10% with 1024x1024. Problem solved. Sorry for wasting your time.

Thanks for the deep dive! I'm glad you found a workaround for your system, and for future readers who also find this to be an issue. I'll make a note of this internally so we can take a closer look if/when it becomes higher priority.
As for the documentation issue, do you mean that you've tried setting hipDeviceScheduleYield and it doesn't have the same function, or hipDeviceScheduleYield is being set internally somewhere when hipDeviceScheduleBlockingSync should be set, or something else?

No, the docs currently say that BlockingSync does not have any effect other than Yield. BlockingSync does a blocking sync, because it sets device->setActiveWait(false). Yield and Spin are the ones that have the same effect.

I see, I'll bring that up to the internal team, thanks!


Rename html-mock.html and move to public folder

To Do

rename the file
move to public folder
update routes/html-routes.js to send the updated file along with the correct path

Done!


Migrates to swift 4.2 with Xcode 10

Migrates to swift 4.2 with Xcode 10

@s4cha does this support Cocoapods installation or only Carthage? When installing using this branch it still installs PryntTrimmerView (2.0.1) which doesn't support Swift 4.2, it should be PryntTrimmerView (3.0.0). Thanks!

@katerynasytnykiOS Thanks for noticing that I'm going to update the pod file :)


Enhanced the Welsh (cy) language dictionaries.

As a native Welsh speaker I've updated some of the Welsh language dictionaries to fill in some omissions.

Thanks much!

Thanks much!


Check Upgrade Requests doesn't ignore system localized decimal point

This causes the HTTP version validation to fail on international systems with a comma decimal separator.

Fixed in c0159f0 submitted with illuminated-g/lv-msg-combo-example#1
<IP_ADDRESS> uploaded to vipm


first pass at updating Billing Project guidance

Thought this would be fairly straightforward, just recommend they use a single Billing Project to start.  But two snags:

[x] It now makes more sense to swap sections 5 and 6, since Google Projects won't exist on GCP until Workspaces have been created, so there won't be anything to track spending for until they create Workspaces.
[x] This means we need to update the overview figure.

~So, here's a start at updating the prose, but I need to address those two issues before this is ready to go~ Done!
~Those might be better addressed in a separate PR?~ Nah, that'll just make merging harder.

@avahoffman @cutsort
Before I started working on Billing videos, my original goal was just to update the Getting Started text to reflect the change to 1 Workspace: 1 Project.
Then I realized I needed to update the overview figure, which seemed like a good opportunity to also make a Billing short...which keeps growing into more videos, haha.
So, in the interest of making some incremental forward progress, here's PR that just updates the prose and overview figure but does not include any of the new videos.  I'll add videos in other PR(s), once they're ready.
Major changes

now recommend they only set up one Terra Billing Project to start (keep it simple)
swap order of steps 5 and 6 (Set Alerts/View spend <-> Create Workspaces)
updated overview figure to reflect the reordering
(also updated the _child_google_billing_view_spend in https://github.com/jhudsl/AnVIL_Template/pull/118)

So, primarily take a look at PI steps 4, 5, and 6 (i.e. chapters 2.5-2.7)


Fix creation of certificate when a keychain is specified.

In this case, the certificate is properly created in the specified
keychain, but the certificate validation used to fail since it kept
searching in the keychain search list.
This patch ensures the verification is performed using the specified
keychain file.

I signed it!


Suggestion to add laravel view components for each theme.

Hi, I really like these starter themes really easy to start your project admin. I have a suggestion to improve these themes more if we use Laravel view components for each theme. Let developer type or copy less html code just laravel components tags to make things more fast so there will be no html div break. we can have different components for each theme like

sidebar or menu component
Bootstrap card component
Input fields components

Above are just basic components list we can add more and make development more fast. if any developer don't want to use these still they can use html.
Looking forward your comments regarding my suggestion.

Laravel/ui doesn't use blade components. Breeze does and themes use it. Of course those components could be improved so feel free to pr :)

@krekas I think that @umefarooq is suggesting exactly that - to add Blade components to bootstrap themes.
I'm not sure that many people would use that, but if you really think this feature would be useful please create a pull request with at least one theme covered with components, and then we could take over for other themes

Here is first look of volt login with component. I have created card component adding form in it. All these are using bootstrap. Here is my suggested component structure all bootstrap component in under component bootstrap directory (App\View\Components\BS) so these all themes can share them.
<x-bs.card>
 <x-slot name="header">
 Header
 </x-slot>
Here you can add text or html elements

 <x-slot name="footer">
 Footer
 </x-slot>
</x-bs.card>

or each theme has their own components?

Themes can't have same components because of different styles

@umefarooq good start, but @krekas is right: the whole problem with components is that they would end up being different, in different themes.
We have this problem clearly in our quickadminpanel.com - that's why it's a very time-consuming work for us to add a new theme in that generator - that every new theme has its own structure, layout ideas, and the same components don't fit.

Yes these components should be theme specific. I was creating form input component everything was working fine but i was not showing validation error. after googling i found this cool package laravel forms components it support bootstrap 4 and 5, tailwind 1 and 2 frameworks easy to use. model binding create form with just component tags
https://github.com/protonemedia/laravel-form-components
Now need your suggestion should we use this or not? if we use this package than we have to create only theme specific components.

@umefarooq no no, we don't want any dependency on any other external packages.
That package looks cool, and I may review it on YouTube, but for larastarters, it should be only "native" components.
That said, if you agree that components are different for all themes, then what is the point of doing components at all? I always thought of larastarters as more or less unified codebase for all themes, and different components would break that idea, also adding more work to support those components in the future.
So, all in all, after some thinking, I will not accept this change into larastarters package.
Closing the issue, and may reconsider if many people request it again and I feel that it's worth supporting.

Why should we reinvent a wheel again, We can use this package only for forms for all themes, other than that each theme can have separate set of UI components.

"there always somebody to take over package and keep rolling it"

Wrong, over my career I've seen dozens of packages abandoned which caused problems for their users

"why should i use this starter kit in my projects"

Because it's a STARTER kit, you use it only once and then can forget about it, even removing it from your vendor. You shouldn't care if Larastarters is upgraded, it's one-time use for new projects.

Generally, I understand your line of thought, it's just the approach that I'm not comfortable with, it's just too much work with those components, and no one else actually requested it. If I see bigger demand, then maybe I would rethink spending my time on this, otherwise, I have more important things to do.


Service Specific Customer Frontend and Backend Template

It would be nice to be able to specify custom frontend and backend templates on a per service basis. Here are the additional headers I'd like to add for certain services. It would probably be a good idea to add them for all services, but not certain if that would mess up for someone
http-request set-header X-Forwarded-Port %[dst_port]
http-request add-header X-Forwarded-Proto https if { ssl_fc }
http-request add-header X-Forwarded-Protocol https if { ssl_fc }
http-request set-header X-Forwarded-Ssl on if { ssl_fc }
http-request add-header X-Url-Scheme https if { ssl_fc }
References:
https://en.wikipedia.org/wiki/List_of_HTTP_header_fields#Common_non-standard_request_headers
http://stackoverflow.com/questions/16042647/whats-the-de-facto-standard-for-a-reverse-proxy-to-tell-the-backend-ssl-is-used

Added addHeader and setHeader reconfigure parameters. They can be used to add or set additional headers before they get forwarded to a destination service.
More info can be found in General HTTP Query Parameters.
New params are available from the release 1.275.
Please let me know if this fits your needs.

Works great. Thanks


Landing UI - Updated

I have also modified the changes which were required.

remove top stories or you can make flex-direction column in media query with max-width: 688px

Great Work @Nayaker, I will merge your PR after resolving conflicts


Adds redux saga support.

Discussed in #234.
Thanks to @rmevans9 for suggesting the feature.
Thanks to @yelouafi for guidance on how to build this puppy.

There's so much more we can do, but this is a usable first start.

This is excellent! Can't wait for this to land.


Update confusedchar-compact.txt

Minor rearrangements
fat arrow
clump similar -÷+
dyslexic j<k

Looks great!

cheers


Content of the sidebar cannot be fully viewed

Description
The content of the navigation sidebar that is displayed on all pages cannot be fully viewed, as it's being pushed down by the Announcement Banner.

Notice in the image above that the section Guides (iOS) is not visible. Removing the announcement banner specific code (top padding on the sidebar and the HTML elements) fixes the issue.
Perhaps the contents of the sidebar should be fully scrollable or the presence of the announcement banner should apply extra styling to the sidebar
This issue is reported from a Macbook Pro with a fixed Dock.
Documentation version
0.63.0

Mine has a scrollbar, check again and ensure if this is not a browser-specific issue

Mine has a scrollbar, check again and ensure if this is not a browser-specific issue

Added some browser specific information to the original report. It is indeed a browser specific issue with Chrome, at least for my setup detailed above.

Hi @eduardopelitti, I have tested the website on Chrome and Safari on macOS and I cannot reproduce your problem (I can scroll both sidebars - left and right). Is this issue still present for you with the live version?

Hi @eduardopelitti, I have tested the website on Chrome and Safari on macOS and I cannot reproduce your problem (I can scroll both sidebars - left and right). Is this issue still present for you with the live version?

Hi! Yes, I still see it, but with further testing I see it's much more narrowly scoped.

I only get the issue in /docs/ pages, on the other sections (Componentes, API) I can scroll the sidebar.
I can do some scrolling, but not all that I need. I've attached a video with the issue I see.


Fix version compare closes #5

Tested fix for version compare. Tested with Ansible 2.9.5 only.

It's also working for 2.7.x :)
Thanks @sspreitzer!


Fix: crash after posting transaction

Closes #3995
NSPredicate(format:) isn't type safe. Is there a better way? Does Realm provide some type-safe magic to form the predicate from the Transaction perhaps?
@oa-s would you help check the other NSPredicate(format:) calls for these? :)

would you help check the other NSPredicate(format:) calls for these? :)

sure

The rest of predicates seems good

@oa-s thanks!