raise ValueError("Missing these files: " + ",".join(missing_files)) ValueError: Missing these files: files

Hi, i am using the following command for mity INDELS finding using ONT generated data, which has been aligned by minimap2 aligner. Here i am using a single BAM file as a input. i am a bit confuse to troubleshoot the error. Any help will be really appreciable. Thank you guys in advance.
The command i am using is :
mity call --reference hg38 --prefix barcode17 --output-dir /pfs/home/jyotimridha/Mito_ONT/Structural_variants/mity/ --normalise files /pfs/home/jyotimridha/Mito_ONT/Batch1_Barcode_Aug_reanalysed/2_minimap_to_freebayes/barcode17_sorted.bam
The error i am facing:
raise ValueError("Missing these files: " + ",".join(missing_files))
ValueError: Missing these files: files
Overall message i am getting on terminal is here:
mity call --reference hg38 --prefix barcode17 --output-dir /pfs/home/jyotimridha/Mito_ONT/Structural_variants/mity/ --normalise files /pfs/home/jyotimridha/Mito_ONT/Batch1_Barcode_Aug_reanalysed/2_minimap_to_freebayes/barcode17_sorted.bam
mity version 1.1.0
Calling mitochondrial variants
Traceback (most recent call last):
File "/pfs/home/jyotimridha/.conda/envs/fastp/bin/mity", line 33, in
main()
File "/pfs/home/jyotimridha/.conda/envs/fastp/bin/mity", line 29, in main
args.func(args)
File "/pfs/home/jyotimridha/.conda/envs/fastp/lib/python3.12/site-packages/mitylib/commands.py", line 42, in _cmd_call
call.Call(
File "/pfs/home/jyotimridha/.conda/envs/fastp/lib/python3.12/site-packages/mitylib/call.py", line 69, in init
self.run()
File "/pfs/home/jyotimridha/.conda/envs/fastp/lib/python3.12/site-packages/mitylib/call.py", line 84, in run
self.run_checks()
File "/pfs/home/jyotimridha/.conda/envs/fastp/lib/python3.12/site-packages/mitylib/call.py", line 219, in run_checks
self.check_missing_file(self.files, die=True)
File "/pfs/home/jyotimridha/.conda/envs/fastp/lib/python3.12/site-packages/mitylib/call.py", line 323, in check_missing_file
raise ValueError("Missing these files: " + ",".join(missing_files))
ValueError: Missing these files: files

Hi @Jyoti-Mridha,
Mity first tries to look for the files you've specified in the file list. In the command you've specified:
mity call --reference hg38 --prefix barcode17 --output-dir /pfs/home/jyotimridha/Mito_ONT/Structural_variants/mity/ --normalise files /pfs/home/jyotimridha/Mito_ONT/Batch1_Barcode_Aug_reanalysed/2_minimap_to_freebayes/barcode17_sorted.bam
"files" here is part of the list of files that mity is looking for, and the error is saying that the file called "files" can't be found. I'm assuming that you're not actually trying to use mity on a file named "files", so you can just get rid of it and run this:
mity call --reference hg38 --prefix barcode17 --output-dir /pfs/home/jyotimridha/Mito_ONT/Structural_variants/mity/ --normalise /pfs/home/jyotimridha/Mito_ONT/Batch1_Barcode_Aug_reanalysed/2_minimap_to_freebayes/barcode17_sorted.bam

Or formatted a bit differently:
mity call \
--reference hg38 \
--prefix barcode17 \
--output-dir /pfs/home/jyotimridha/Mito_ONT/Structural_variants/mity/ \
--normalise \
/pfs/home/jyotimridha/Mito_ONT/Batch1_Barcode_Aug_reanalysed/2_minimap_to_freebayes/barcode17_sorted.bam

You don't need to put anything in front of your list of files, and feel free to look at the main readme in the github repo for more examples of how to run the commands.
Thanks! Let me know if you're still encountering issues!
Trent Zeng


filesystem, blob, base64 src removed

In a chrome app, adding a newly created blob/filesystem data url into a dom-repeated model data creating an img element with an empty src.
E.g. forms.set('images',['filesystem:chrome-extension...', 'blob:chrome-extension%3A//cgnolaidl...', 'data:image/gif;base64,R0lGO...']
appends

(no difference when using iron-image)
When changing the src of an existing img element with js it works.

The problem is not with urls, it is with dom-bound setting (push not notifying, using set doesn't reset it if the same elements are there for some reason) so I'm closing the issue.


Deploy scripts

Added script to deploy functions to the Azure cloud
Added docs (README.md)

see https://github.com/teamdigitale/digital-citizenship/pull/14


How to get some bigger text?

Thanks for sharing the code. I want to get some bigger text (for example, the text occupies a large part of the image), can you tell me how to operate your code? Thank you very much.

Thank you very much. When I single step executing the code, i meet this problem as follows:
0 of 4
** instance # : 0
Traceback (most recent call last):
File "/home/zy/pycharm-community-2016.2.3/helpers/pydev/_pydevd_bundle/pydevd_frame.py", line 541, in trace_dispatch
self.do_wait_suspend(thread, frame, event, arg)
File "/home/zy/pycharm-community-2016.2.3/helpers/pydev/_pydevd_bundle/pydevd_frame.py", line 71, in do_wait_suspend
self._args[0].do_wait_suspend(*args, **kwargs)
File "/home/zy/pycharm-community-2016.2.3/helpers/pydev/pydevd.py", line 738, in do_wait_suspend
time.sleep(0.01)
File "/home/zy/SynthText-master/common.py", line 38, in signal_handler
raise TimeoutException, colorize(Color.RED, "   *** Timed out!", highlight=True)
TimeoutException:    *** Timed out!
*** Timed out!
1 of 4
** instance # : 0
2 of 4
** instance # : 0
3 of 4
** instance # : 0
4 of 4
** instance # : 0
Process finished with exit code 0
So what is the problem?

Set SECS_PER_IMAGE to None here:
https://github.com/ankush-me/SynthText/blob/master/gen.py#L27

I am very appreciate your help. Thank you!


refactor(evm-ante): Use protobuf ABCI event instead of legacy event

Following from changes in https://github.com/NibiruChain/nibiru/pull/1912

Completed by https://github.com/NibiruChain/nibiru/pull/2053
Commit https://github.com/NibiruChain/nibiru/commit/629aea39d5527dcaeafd4b6f6472da3b8649c8aa

Notes
Now, the EVM events are structured like this:

pending_ethereum_tx - untyped event, emitted within evm ante handler.
Left untyped because it could be used by Tendermint TxSearch which cannot parse wrapper quotes in a typed event value.

eth.evm.v1.EventEthereumTx - typed event, contains more info about ethereum tx. Used for getting TxReceipt.

message - untyped event with attr module (evm), sender and tx_type (legacy, accessList or DynamicFee) - left untyped because it's being used for event subscription (for eth it's used in eth_filterLogs)

eth.evm.v1.EventTxLog - typed event, stores evm execution logs. Used in log queries.

Additional typed events which are emitted depending on Tx action: EventContractDeployed, EventContractExecuted or EventTransfer.

eth.evm.v1.EventBlockBloom - typed event emitted in the end of the block and used for event filtering.

TmTxIndexer vs EVM Indexer
One of the indexers must be turned on public nodes. With both turned off EVM Tx will not get a proper receipt and several queries will fail.
Fix tx_index
Fixed issue with tx_index discrepancy between ante handler event and apply evm event.
Proto Break
Removed typed EventMessage as we have to go with the untyped one. See above.


salam khooooooooti la bghit nvalider les lessons bach man3aya9ch kindie

Describe the bug
A clear and concise description of what the bug is.
Browser

[ ] firefox
[ ] chrome

version:
rosettastone product

[ ] Foundations
[ ] Fluency builder

To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.

olaa bghit n9oss sway3

hhh ydna fih makanch solution


Added value to control setting of the watch filter

Signed-off-by: Marcus Noble<EMAIL_ADDRESS>This PR:

Makes the watch filter configurable

Checklist

[x] Update changelog in CHANGELOG.md.

Closing in favour of https://github.com/giantswarm/cluster-api-app/pull/59


Force new rotate when commercetools scope changes

In the secrets module, the aws_secretsmanager_secret_rotation resource should force a new rotate when the var.scopes changes.
If we could re-create this resource when the scope changes, AWS will trigger a new rotate.
At the moment there doesn't seem to be an easy way to trigger this.

If I read it correctly, we should be able to rotate the token using the API of the AWS secrets manager by calling RotateSecret without the RotationRules parameter.

If you do not include the configuration parameters, the operation starts a rotation with the values already stored in the secret. After the rotation completes, the protected service and its clients all use the new version of the secret.

https://docs.aws.amazon.com/secretsmanager/latest/apireference/API_RotateSecret.html
This should be executed before any other components have started since they will be requesting this token on startup and then cache it.


Broken link on glossary page - 403

Describe the bug
The Infrastructure Provider section of the glossary page has a broken link to https://edpb.europa.eu/system/files/2021-07/eppb_guidelines_202007_controllerprocessor_final_en.pdf that responds with 403 Access denied reported by the link checker job.

Hm, is that the exact same URL that was fixed in a different section in da49c886d8b890ee31711a87a4f489f6a6ef810a ?


Problems with numeric inputmask

Hi,
I used the following mask for my input (Tested on the latest version).
input.inputmask("decimal", {
    placeholder: "0",
    digits: 2,
    digitsOptional: false,
    radixPoint: ",",
    groupSeparator: ".",
    autoGroup: true,
    allowPlus: false,
    allowMinus: false,
    clearMaskOnLostFocus: false,
    removeMaskOnSubmit: true,
    onUnMask: function(maskedValue, unmaskedValue) {
        var x = unmaskedValue.split(',');
        if (x.length != 2)
            return "0.00";
        return x[0].replace(/\./g, '') + '.' + x[1];
    }
});

First I noted that with default options where {radixPoint: ".", groupSeparator: ",", autoGroup: true}, the mask function does not work correctly. For example, if I set "10000" or "10000.00" to my input which comes from my database, the masked value is "100.00".
Then when I used {radixPoint: ",", groupSeparator: ".", autoGroup: true} options, the mask function also does not work and the unmask function does not unmask the value, i.e., the unmask function returns the same value. For example, if my input has value "1.000,00", the unmask function returns "1.000,00". But with the default options, the unmask function returns the value correctly, i.e., taking off the groupSeparator(",") from the value. My solution was to define what to return using onUnMask.
I found another problem when I use {autoGroup: true, clearMaskOnLostFocus: false} to insert groupSeparator. I do not know how to explain well that problem, so I will try to be clear enough. With this options, the input first show the value "0.00". When I focus the input the cursor goes to before radixPoint and I type "123456", the input value will be "123,456.00". But if I focus again and start erasing the digits with backspace until input value show ",00", and then focus out the input value is updated to show "000.00". That happens with {autoGroup: true}, but not with {autoGroup: false}.
I suggest to improve the behavior described when using {autoGroup: true, clearMaskOnLostFocus: false}. But I think that making mask and unmask function work properly is a good idea. I mean that unmask function could return the value with default format, i.e, database decimal format (no groupSeparator and radixPoint:".").
I think that your inputmask is great because offer many options like removeMaskOnSubmit, where usually the values are unmasked or converted on the server side, and other options that I have not explored yet.
Thanks.

Hi, I am facing same issue -
When I insert a digit (for example, "1") I get "100".
Could you please help me to fix this issue.
Regards,
Manish

@manishkrai
I did not understand your problem. I tested your mask using the latest version and worked well.
@RobinHerbots
I noticed another thing. When I use the events 'change' or 'focusout' and try to get unmasked value of the input, the mask does not work well. For example, I focus in the input, select all characters (Ctrl+A) and type any integer number (for example, "20", without entering radixPoint and decimal digits). When the event fire, I just get "20.undefined" although I can see "20,00" in the input.
inputmask used:
input.inputmask("decimal", {
	placeholder: "0",
	digits: 2,
	digitsOptional: false,
	radixPoint: ",",
	groupSeparator: ".",
	autoGroup: true,
	allowPlus: false,
	allowMinus: false,
	clearMaskOnLostFocus: false,
	removeMaskOnSubmit: true,
	autoUnmask: true,
	onUnMask: function(maskedValue, unmaskedValue) {
		var x = unmaskedValue.split(',');
		return x[0].replace(/\./g, '') + '.' + x[1];
	}
});

@FilipeZhou ,
Can you retest with the current version. ... and thx for the feedback!

@RobinHerbots
Thank you for the fixes for my problems.
I found out a strange behavior in the current version when delete characters in special situations with the same inputmask used.
For example, I type "1.456,00" and move my cursor to after '4' like "1.4|56,00" and delete '4', the result is "15|6,00".

Hi @RobinHerbots,
When I enter a single digit number like- 1 or 2 or 3. This displays it as 100, 200, 300 respectively.
Iddeally It should be 1.00, 2.00 and 3.00.
I am using 3.0.33 version of this contro. Kindly suggest me the way around for the same.
Regards,
Manish

@manishkrai ,
Update to the latest version.

@FilipeZhou ,
I see, I will have a look.

@RobinHerbots
Thank you very much for the fixes.
Just a suggestion: I have seen in your code that when you mask a value, according to inputmask, you apply the mask replacing the radixPoint ('.') to the one in inputmask. But when I get the unsmakedvalue, you just take off the groupSeparator from the value. What do you think in replacing back radixPoint to '.' too?
Now I set onUnMask function to my inputmask to do this part:
input.inputmask("decimal", {
	placeholder: "0",
	digits: 2,
	digitsOptional: false,
	radixPoint: ",",
	groupSeparator: ".",
	autoGroup: true,
	allowPlus: false,
	allowMinus: false,
	clearMaskOnLostFocus: false,
	removeMaskOnSubmit: true,
	autoUnmask: true,
	onUnMask: function(maskedValue, unmaskedValue) {
		var x = maskedValue.split(',');
		return x[0].replace(/\./g, '') + '.' + x[1];
	}
});

@FilipeZhou ,
Some systems require that the radixpoint is like specified in the culture setting to be valid. (ASP.NET/MVC) otherwise the binding will be incorrect.
Anyway, the onUnMask is there to be adapted to your needs as you do.

@RobinHerbots
OK. No problems so.
Thank you anyway.

@FilipeZhou
Excelente man
Solucione mi problema del caracter - y +
Saludos.

Hola amigos :)


Bonfire passes tests but does not mark as complete

I've passed all the test cases, but I don't get the "good job" modal, nor does it mark as complete on the map or save my code on my profile. The bonfire in question is http://www.freecodecamp.com/challenges/bonfire-binary-agents.
Here is my passing and yet not passing code:
function binaryAgent(str) {
  var newStr = "";
  arr = str.split(" ");
  arr.forEach(function(current) {
    newStr += String.fromCharCode(current);
  });
  return newStr;
}

OK, so the code is incorrect. I suppose the bug is that it marks all of the test cases as correct when it they are not.

Hey @michael-krebs I've been unable to replicate this.
Closing for now but feel free to open if this is still an issue for you


Don't clear out errorcode or errormessage when using the paramaterless version of ToReturnOptionObject()

My team has noticed that calling ToReturnOptionObject() clears any values set in errorcode or errormessage.  That may be the intended behavior, but it was definitely unexpected.  I would expect to need to set errorcode 0 or the equivalent enum value to clear those fields.  What do you think about not clearing them unless error code 0 is specified?

You are correct. The default method was envisioned a convenient way to return a successful response with the parameterized override allowing for setting other error codes or messages. I had not considered that the error code and message would have been set prior to this step.
I agree that this assumption is incorrect and that this method should not overwrite the existing values if present. However, something worth considering is whether the ToReturnOptionObject() is still needed in this use case if it has already been configured for return.
The ToReturnOptionObject() method performs the following actions to prepare the OptionObject for return:

Clones the OptionObject.
Removes unedited rows to trim down the payload and address some errors previously seen in the myAvatar UI in testing.
Sets the ErrorCode and ErrorMesg.

I suspect in this use case the method is still valuable for step 2 despite step 3 being handled earlier in the process.

Yes, since the guidance given in training is to return an optionobject
containing only the items you wish to change, I think step 2 is still
important.
On Sat, Jan 28, 2023, 12:54 PM Scott Olson Jr @.***>
wrote:

You are correct. The default method was envisioned a convenient way to
return a successful response with the parameterized override allowing for
setting other error codes or messages. I had not considered that the error
code and message would have been set prior to this step.
I agree that this assumption is incorrect and that this method should not
overwrite the existing values if present. However, something worth
considering is whether the ToReturnOptionObject() is still needed in this
use case if it has already been configured for return.
The ToReturnOptionObject() method performs the following actions to
prepare the OptionObject for return:

Clones the OptionObject.
Removes unedited rows to trim down the payload and address some
errors previously seen in the myAvatar UI in testing.
Sets the ErrorCode and ErrorMesg.

I suspect in this use case the method is still valuable for step 2 despite
step 3 being handled earlier in the process.
—
Reply to this email directly, view it on GitHub
https://github.com/rarelysimple/RarelySimple.AvatarScriptLink/issues/52#issuecomment-1407450951,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWTCEJ2L2DIXSGPXABWWYTTWUVMMFANCNFSM6AAAAAAUETGS4M
.
You are receiving this because you authored the thread.Message ID:
@.***
com>


[Bug]: Recaptcha is not working on release

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Every time we click on submit button, recaptchaToken gets generated which we can see on Text widget(after binding). Currently recaptchaToken is not generated on click.
Adding video:

Steps To Reproduce

Add Button widget on canvas
Add recaptcha key for either v2 or v3
Add Text widget and bind with Button widget to see recaptcha token {{Button1.recaptchaToken}}
Click on the Submit button to display recaptcha on Text widget
Observe that token is not visible on Text widget

Note: Same activity has been done on Production and seems everything works fine there but not working on release
Environment
Release
Version
Cloud

@vicky-primathon please check this on priority. If you have a fix, make sure of adding the test case so that we dont have problems with this feature

Failing as getting below error when loading google recaptcha script.

@vicky-primathon Issue still exists on release

Can we use this solution here @somangshu
https://stackoverflow.com/questions/61862932/chrome-extension-refused-to-load-the-script-because-it-violates-the-following-c

@chandannkumar  this issue is fixed on release

@vicky-primathon still same issue. Since we are getting the error as "Violation of content security policy" in console only for release environment. Meanwhile we can confirm this fix after release
cc @somangshu

Reopening this. Since this issue is not working on release because of above reason mentioned.
Adding console error screenshots

@sharat87 can you help us here? Not sure if this is something to do with the release env itself

Added www.gstatic.com for script-src. Can we check once again please? @chandannkumar

This seems working fine now


fix(cta-block): clean up spacing

Related Ticket(s)
Resolves #5779
Description
This PR adjusts some spacing rules for the dds-cta-block element and its intended subcomponents so they better align to the design spec.
Before:

After:

Changelog
Changed

reduce spacing between dds-cta-block's copy and action slots
align dds-cta-block's intended subcomponents to Carbon grid

Thank you for the review, @shixiedesign! Would you mind taking a look at my comment on the related issue (https://github.com/carbon-design-system/carbon-for-ibm-dotcom/issues/5779#issuecomment-919200662)?

Hi @jkaeser i just answered your question on spacing here: https://github.com/carbon-design-system/carbon-for-ibm-dotcom/issues/5779#issuecomment-921716957
I really should have documented the bug more specifically, sorry! But taking a look, changing button group as a whole is too risky right now. It impacts a lot of components. I'd say if fixing the spacing between copy and button group is too hard otherwise, we can temporarily leave it as is (a bit bigger than expected). This does make me wonder if button group is looking correct. Here's a visual of what's going on:

If there's erronous padding on button group, we might need a separate epic to refactor button group and fix regression on all the components touched.

@shixiedesign IMHO taking a more considered look at the button group spacing would be the best decision in the long term. The margin-top on the buttons exists to create vertical space between buttons when they wrap to multiple rows. The fact that it also creates the correct amount of space between components is a side effect, and as this issue demonstrates, that side effect is not always desirable.
For now, I will revert the button margin change and leave the negative margin solution in as a temporary fix. I'll also go ahead and make any necessary changes to address https://github.com/carbon-design-system/carbon-for-ibm-dotcom/issues/5779#issuecomment-921716957.
Thanks, everyone!


refactor: move remaining code in penumbra-crypto to penumbra-component

Closes #2765

I split out #2800 into another PR since this PR is failing the wasm build and I need to debug further to understand why.

We aim to land this PR prior to shipping Testnet 56 (https://github.com/penumbra-zone/penumbra/issues/2761).

For posterity, the issue with the wasm build was:
On main,

penumbra-component depends on penumbra-storage
penumbra-storage depends on tokio

In this PR previously:

penumbra-component is added to the dependencies for penumbra-transaction, among other crates that need to be wasm-friendly -> the problem
so the approach of moving effect_hash.rs to penumbra-component just doesn't work

Two ways address this would be to:

either move effect_hash.rs to penumbra-chain (already a dependency of penumbra-wasm ) OR
make a minimal crate e.g. penumbra-effect-hash that can be used by penumbra-transaction and the various penumbra component crates

I've done approach 1 here to resolve since the amount of code in question is fairly small.


The profile column in the okta_application table does not contain any information

Describe the bug
For any rows from the okta_application table, the profile column never has data
Steampipe version (steampipe -v)
v0.10.0
Plugin version (steampipe plugin list)
v0.1.0
To reproduce
Run select name, profile from okta_application while having at least 1 application
Expected behavior
The profile column should have data or not be defined if the API doesn't return any data for it
Additional context
Application type - https://github.com/okta/okta-sdk-golang/blob/master/okta/application.go#L35

The following response was received from the OKTA support team:
The Profile property is designed for the ListApplicationUsers(), UpdateApplicationUser(), GetApplicationUser() and AssignUserToApplication() methods, which all of them are making an API call to the {{url}}/api/v1/apps/{{appId}}/users/{{userId}} endpoint.
The {{url}}/api/v1/apps endpoint does not contain any property called 'Profile' as it is only available for users.

As a result, it has been decided to remove profile from the list of columns available for the table.


Add sorting by reference

Add sorting by reference in the prestashop categorie page.
| Type         | improvement
| BC breaks?    | No
| Deprecations? | No

This change is

@LameuleFR ,
Do you have an issue/discussion related ? If so, please add the link to your PR description.
If not, please describe How to test your PR description.

thank you @LameuleFR

Some merchants don't use references, so this sorting is not relevant for them.

In most case merchant will use references but we can probably add a configuration field in the module to activate/deactivate this behaviour.

@LameuleFR It would be great to have a simple configuration for every sort option 👍🏻


Add Lago

https://github.com/getlago/lago-api
Found via https://techcrunch.com/2024/03/14/lago-a-paris-based-open-source-billing-platform-banks-22m/

Dupe of https://github.com/eliotsykes/real-world-rails/issues/7982


feature: sass, different fixes

1. In package.json

Added sass with the latest version
Bumped electron to the latest version (17.0.0) which was released a day before

2. In src/main/index.ts

Added a handler that will open external links in the browser, not creating a new window of the application with the page.

3. In src/renderer/index.html

Added a meta tag with Content-Security-Policy which fixes the Electron Security Warning (Insecure Content-Security-Policy) warning in the console

4. Moved all styles to a specified folder
5. In src/renderer/src/main.tsx

Changed imports

6. In src/renderer/src/App.tsx

Changed component to be an arrow function component
Changed class names for the imported Scss module

7. Added styles src/renderer/src/styles/app.module.scss

Remade App.css for Scss (can be converted to Sass and vice-versa)

If this PR gets merged, don't forget to change the repository 'About' section and its topics 😉

fixes the Electron Security Warning
This is great 🎉

The code uses two 'Spaces' instead of 'Tabs' 😄


[REQUEST] Add Game Categories and Online Support Information

Is your feature request related to a problem? Please describe.
No, this feature request is not related to a specific problem. However, it aims to enhance the user experience by providing additional information about games in the HydraLauncher, such as their categories (genre) and whether they support online play. This would make it easier for users to find and select games based on their preferences.
Describe the solution you'd like
I would like to see the implementation of two new features in the HydraLauncher:

Game Categories:
Add a way to categorize games by genre, such as action, adventure, puzzle, etc. This would help users filter and find games that match their interests more efficiently.

Online Support Information:
Introduce an indicator or tag that shows whether a game supports online multiplayer. This would allow users to quickly identify games they can play with friends or others online.

Describe alternatives you've considered
One alternative could be to allow users to manually tag games with this information, but this might lead to inconsistencies. A more structured approach where this information is standardized within the launcher would likely provide a better user experience.
Additional context
These features would align well with the current functionalities of HydraLauncher and could be implemented as optional metadata fields within the launcher’s game library interface. This addition would also support users who prioritize online gaming or who prefer specific genres, making the platform more user-friendly and versatile.

to fix your trouble check this fix, i see it in another issue,
https://github.com/Jostino/RimWorld-it/releases/tag/fix
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble check this fix, i see it in another issue,
https://github.com/Jostino/RimWorld-it/releases/tag/fix
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble check this fix, i see it in another issue,
https://github.com/Jostino/RimWorld-it/releases/tag/fix
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

Yo, this has been around for a while. Everything about it online says it's fix, and you can play with your friends. When you download it, it uses Steam to let you game with your buddies. The password for downloading the games from online fix is online-fix.me.


Zero-centered gamma (Layernorm1p) support for JAX

This PR adds zero_centered_gamma support for JAX layernorm in different modules.
For TransformerLayer, MultiHeadAttention module, the gamma initializer becomes 0s initializer and use (1+gamma) instead of gamma when zero_centered_gamma is used.
It is similar for LayerNorm, LayerNormDenseGeneral, and LayerNormMLP. The default initializer for them is nn.initializers.ones when zero_centered_gamma is False, and it becomes nn.initializers.zeros when zero_centered_gamma is True through __post_init__. Besides, users can still provides their own initializer through scale_init for these three modules.

/te-ci

/te-ci

/te-ci

/te-ci

/te-ci


docs: Fixes a typo in gpt.mdx

Spotted while going through your docs at ETHGlobal Bangkok 2024 ;)
btw the button on the documentation to directly edit the docs doesn't work (wrong link, 404), maybe a configuration to update on your side with the framework you use.

Summary by CodeRabbit

New Features

Introduced a new chat memory management system to enhance user interaction by storing chat histories.
Updated text generation functionality to utilize chat history for improved responses.

Bug Fixes

Corrected minor text errors in the documentation for clarity.

These changes aim to provide a more seamless and engaging user experience while interacting with the chat features.

Thanks! fixed in current PR
https://github.com/ephemeraHQ/message-kit/pull/128


Update AEM Forms solutions with new names.

Update AEM Forms include flags to reflect new solution types
Description
Going forward, AEM Forms will have two different solutions on
AEM as a Cloud Service. These are named:

formsenrollment
formscommunications

As per the existing contract with Cloud Manager during VSTS repository
generation, the updated solution names will be passed instead of the
earler used includeForms in the archetype command.
This requires us to update all usage of the includeForms property within
the archetype with the updated solution names (includeFormsenrollment
and includeFormscommunications).
For backward compatibility reasons the older includeForms check is
left intact in the archetype template but has been removed from the
README.md.
Related Issue
#823

Motivation and Context

This is needed to preserve the contract with Cloud Manager and archetype for some new AEM Forms Features.
How Has This Been Tested?
Run build-time tests and validate output.

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Can we also update the ReadMe description for the includeForms argument? It currently describes the argument as the following but this description does not make any mention of AEM Forms which is a product module that is very different than Core Components.

Includes Forms Core Components dependencies and generates corresponding artifacts.

Arguably, AEM Forms and the includeForms should be different arguments altogether. I would suggest includeCoreForms, includeFormsEnrollment, and includeFormsCommunications if the last two have been vetted by the Product Team.

@GuillaumeCleme, Apologies for the late reply. It was an extended holiday over here.
Let me try to provide a little more context.
So far till now, there was a single "AEM Forms" solution in AEM as a Cloud Service and it was just named "Forms". However going forward from this month onwards, this solution is now named "Forms Enrollment" and there will be an additional solution named "Forms Communications".
So in total, there will be 2 Forms specific solutions on AEM as a Cloud Service parallel to other solutions like Sites, Assets, etc.

Forms Enrollment. ( -DincludeFormsenrollment)
Forms Communications ( -DincludeFormscommunications)

However, for the short term, for backward compatibility reasons we'd still need to support the older "Forms" solution which has the flag ( -DincludeForms). Once these older solutions have been migrated to the newer ones, we should be able to remove includeForms from the archetype.

Arguably, AEM Forms and the includeForms should be different arguments altogether. I would suggest includeCoreForms, includeFormsEnrollment, and includeFormsCommunications if the last two have been vetted by the Product Team

includeFormsenrollment and includeFormscommunicationshave been vetted by the Product Team (Both CMGR and AEM Forms).includeForms` is not needed anymore and is only kept for backward compatibility reasons. Unfortunately we're not in a position to change the naming conventions wrt the name casing at this point (since the contract between CMGR and the archetype is pre-defined)


PreviousNextView not working with textfield focused programmatically

First let me say that this plugin is awesome,
Now second, I have an issue with PreviousNextView. It's working great for the most part, but I have a page set up so that the first textfield is focused as soon as the page is opened (so that the user can start typing right away). The problem is that when I use textfield.focus() then the arrows are messed up for the first textfield. They point up when I have the first textfield selected, and neither arrow work. I have to unselect the textfield, and then selected it back again.
Here I have a playground example with the minimal code to reproduce it (notice the arrows as soon as the second page is opened):
https://play.nativescript.org/?template=play-ng&id=p9cTUW&v=6
So I have a textfield like this:
<TextField
    (layoutChanged)="onNameInputLayoutChanged($event.object)"
    [hint]="'projectName' | translate"
    [(ngModel)]="projectInput.name"
></TextField>

and the .ts file like this:
public onNameInputLayoutChanged(textfield: TextField): void {
   textfield.focus()
  }

If you guys could look into it that would be awesome. Thank you!

Looks like you're focusing that textfield as soon as it's loaded, and perhaps other parts of the page haven't yet. So a little timeout will go a long way. I would actually use an even longer timeout so the screen transition has finished so it looks a little nicer.
Updated playground: https://play.nativescript.org/?template=play-ng&id=p9cTUW&v=7
Changed code in textfields.component.ts:
    public onLayoutChanged(textfield): void {
        setTimeout(() => {
            textfield.focus();
        }, 0);
    }

Hey thanks so much for the fast reply! It does work, but it causes some flickering. I'll probably add some delay time as suggested or maybe use a ViewChild reference to the object once the page is fully loaded.
Thanks again! I wasn't expecting a same day reply


Add support for Informix-4gl files

Fixes vscode-icons/vscode-icons#2514
Changes proposed:

[x] Add
[ ] Delete
[ ] Fix
[ ] Prepare

Attach: vsicons-custom-icons.zip

Please see https://github.com/vscode-icons/vscode-icons/issues/2514#issuecomment-645207742


Update requirements.txt

We need to update requirements.txt to reflect current dependencies for setting up a new development environment.
Currently missing packages:

pytest

nose


[Hold]For migration

What type of PR is this?

What this PR does / why we need it
Which issue(s) this PR fixes

Fixes #
Special notes for your reviewer
Does this PR introduce a user-facing change?

/hold


django-ajax fixed to point to the right min.js file

JQuery AJAX plugin was not loaded because the filename was mistaken.

Merged!. Thanks for the contribution @rtubio


Redirect to last guide

When a logged user visits home again from a different site, he/she should be redirected to the last guide.
That way you keep context and it is not necessary to look for the guide again.
This should not affect flow when you visit it from within mumuki.
A message saying "welcome back, we have recovered you workspace for you" or something like that could also be a good idea.

Any comments? @faloi @jvillarejo @PalumboN @aguspina ?

I like this but redirecting the user sounds a bit agressive to me. Also I'm thinking that the student could be working on different guides at the same time.
Perhaps we can do something like Netflix's "Continue watching" feature:

Yes, that could work, show the last 4 or 5 guides or paths he or she has tried.
But.... that means a redesign of the homepage - which is actually necessary, though. Why not going for the plain redirect now, and then implement this feature in the future?

And I don't think there are too many users doing two guides of different paths at the same time now, do you?

Agree


performance problem

Hello,
I noticed a performance problem as soon as the schema contains the following structure:
... "anyOf": [     {"enum": ["aa", "bb", "cc"]},     {"pattern": "pattern1"},     {"pattern": "pattern2"},     {"pattern": "pattern3"},     ... ] ...
The performance can be massively improved by processing the schema beforehand. All enum values and patterns should be combined to a single pattern as shown in the example below:
... "anyOf": [     {"pattern": "^aa$|^bb$|^cc$|pattern1|pattern2|pattern3"} ] ...
Actually, you iteratively append the enum values and regex patterns to a single regex and compute for every iteration the intersection between the current pattern and ".*". This is very expensive and results in bad performance (for this specific kind of schema).
I added an example json file (anyOf.json) that shows the problem. anyOf.json takes on my machine about 50-60 seconds for the result (LHS :< RHS and RHS :< LHS) when checking the file against itself (command jsonsubschema anyOf.json anyOf.json). Applying preprocessing, it takes about 0.04 seconds. I also attached a python script (smaller_anyOf.py) that contains the preprocessing. The script combines the string-enum-values and all patterns to a single pattern as shown in the example above.
AnyOf.zip
By transforming the string-enum-values to a regex, special regex characters (e.g. ".", "-", ...) are escaped to get an identical expression as regex.
... "enum": ["ab-c"] ...
will be transformed to
... "pattern": "^ab\\-c$" ...
Be careful, this can currently lead to another problem - see #6 .
Best Regards
Michael

Hi Micahel,
Thank you for this issue.
I am not sure your description of the problem root cause is correct.
Isn't your suggestion is what is being done here for string enums
https://github.com/IBM/jsonsubschema/blob/165f8939c9c294d8f6fa01d9e6690a26bd24e9c9/jsonsubschema/_canonicalization.py#L264-L266
and here for anyOf with several patterns
https://github.com/IBM/jsonsubschema/blob/165f8939c9c294d8f6fa01d9e6690a26bd24e9c9/jsonsubschema/_checkers.py#L349-L351


⚠ Empornium has degraded performance

In e59104a, Empornium ($EMP) experienced degraded performance:

HTTP code: 500
Response time: 1451 ms

Resolved: Empornium performance has improved in fd25e77 after 18 minutes.


Renaming Tests Discussion

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
I am starting the discussion on how we want to rename some of the tests post the refactoring of the tests in #1491
Describe the solution you'd like
Some of the tests have names that don't necessarily have anything to do with what they are testing. Most universally the tests that begin with csv_* come to mind. Does that prefix have any value? I think we should just universally remove them. Maybe another suggestion maybe to instead prefix tests with what functionality they are meant to actually test?
Describe alternatives you've considered
Not renaming them?
Additional context
Add any other context or screenshots about the feature request here.

@alamb @xudong963 @liukun4515 Any opinion on this or would you prefer I just run with a PR on my own ideas?

Some of the tests have names that don't necessarily have anything to do with what they are testing. Most universally the tests that begin with csv_* come to mind. Does that prefix have any value? I think we should just universally remove them.

Makes sense to me

Maybe another suggestion maybe to instead prefix tests with what functionality they are meant to actually test?

That would also be a great idea
Sorry for the lack of feedback earlier - I can't keep up with all the tickets 🤯  somedays :)

@alamb @xudong963 @liukun4515 Any opinion on this or would you prefer I just run with a PR on my own ideas?

@hntd187  agree with you.
Some tests have the meaningless prefix, and it is not necessary.
My suggestion is that the prefix should be aligned with intend of the test.
I can start this task. If you finish it, I would like to review it.

Make sense to me, looking forward to your ticket!  I can help review it. @hntd187

@hntd187
any update for this?

I think this is completed in  https://github.com/apache/arrow-datafusion/pull/1491

No @alamb this is renaming which we didn’t do in #1491 we agreed I’d do it after some discussion in this issue. I planned to get to this this weekend.

FYI I plan to move some tests out of context.rs into the sql_integration test -- I'll try and get that done prior to the big rename 🏃


[Backport release-23.11] nextcloud, nextcloud-notify_push: set platforms to linux

Description of changes
Backport of #283970
Also contains backport of https://github.com/NixOS/nixpkgs/pull/268109, which this one depends on.

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Why is the first commit not a cherry-pick -x?

Why is the first commit not a cherry-pick -x?

Thanks, fixed.

Why exactly do we need to backport maintainers/teams: init and add helsinki-systems (#268109, 7b6580dba42d2ec15e8b2b90fb2f1da93bd0d398)?

Good catch, thanks!
My intention with cherry-picking the maintainers commit is to avoid future merge conflicts when backporting via labels (maintainers of notify_push changed). Would you prefer something else?

@emilylange Reconsidered and fixed, thanks for the review.

@emilylange That might be an issue with GitHub, I literally copy pasted those commit revs from the GitHub bot message linked above. The revs are present in this repo as well, somehow it's linking to my fork.

Hmm... yeah, you are right the thing I saw is gone now.
Not sure how that happened.
Could have been GitHub being eventually consistent or just my brain bit flipping or whatever.
sorry^^


Sh/add aqlmuf interval standard value set

What does this PR do?
change AQLMUFInterval to AQLMFrequencyInterval for StandardValueSet
What issues does this PR fix or reference?
https://github.com/forcedotcom/cli/issues/2163

@shetzel,
Hi Steve,
Don't we actually need both of them? AQLMUFInterval and AQLMFrequencyInterval? Because there are two different standard picklists in ASM:

AdQuoteLine.MaximumFrequencyInterval (AQLMFrequencyInterval)
AdQuoteLine.MaximumUserFrequencyInterval (AQLMUFInterval)

And in this PR we only cover the first one - AQLMFrequencyInterval.


Merge current features with each other

I worked on the drafts feature within my branch. The current functionalities are now possible:
Delete a draft by clicking an img element that is tied to it
Post a new draft by various buttons (when exiting and "saving as draft", when signing or not signing)
I'm currently working on showing the draft data within Quill, from there I'll try to make it work so that when a user clicks on a draft item it shows the draft data in the letter editor and finally that when saving again the draft will update

@Leo-Wolff and I worked on mergeing all the filters to the development branch. Yesterday me and @enisokic did the matching and filtering part.
Happy to say that everything works together! just some minor tweaks are needed and some tidy-up to do, but we are almost there

Merge has been completed! Some bugs where formed but fixed that! Project works!


Missing Git tags v1.6.16, v1.7.4, v1.7.5, v1.7.6

The most recent tag here is v1.7.3. Tags from the last few months are missing: v1.6.16, v1.7.4, v1.7.5, v1.7.6.

Thanks. I've pushed them.


Draft initial pass for documentation content

Draft the initial documentation content for the documentation website. Initially planned structure:
Overview

Introduction: Briefly introduce the project, core principles and point towads support and contribution guidelines.
Why Cardinal: Explain the rationale behind why a developer could choose Cardinal over other create-***-app solutions, and bring a overview of the included tech.
Recommendations: This is aimed at users who are still choosing their stack. Provide recommendations based on project type, budget, and developer experience level.

Usage

New Project: Instructions on how to setup a new project with Cardinal.

First steps: Post scaffolding setup instructions. Some scaffolded projects will still require the user to perform post-install tasks, like configuring and supplying correct environment variables for parts of the stack. Most things here will backlink to the specific "stack part" page.

Frameworks: Top level page for frameworks.

Next.js: Next.js specific instructions, mostly pointing to their docs.

Backend: Top level page for backend modules

REST: Describe how REST API's are authored depending on the combination of deployment provider and framework.
GraphQL: Describe how GraphQL API's are authored, and how to use the GraphQL client on server and client components/runtimes.
tRPC: Describe how tRPC API's are authored, and how to call procedures from both server and client runtimes.

Authentication: Top level page for authentication. Reafirm core things about the authentication layer in Cardinal (Lucia, server-only, abstractions for low-level primitives)

Security considerations: A refresher on auth security topics.
Add new authentication methods: A top level overview of how to implement some other authentication methods (tokens, magic link, OAuth).
Session handling: Describe how Lucia creates sessions by default, and cases in which you may want to override it's behavior. Advise caution when doing so.

Database: Top level page for database. Reafirm core things about the database layer in Cardinal (Drizzle, underlying drivers, SQL-like schema declaration and query)

Database connection: Per-platform instructions of where to get the credentials needed to stabilish the database connection
Prototyping and migrations: Some general recommendations on how you can prototype database schemas with either drizzle-kit push (when applicable), or database migrations.

Configuration: Top level page for configuration module

Environment variables: Describe how environment variables should be declared, managed, and how they are checked during runtime.
Prettier & ESLint: Describe how Cardinal scaffolds and pre-configures Prettier and ESLint, and where/how you might want to extend them


Bump ppy.osu.Game to 2020.1009.0

Also fixes the obsoleted code errors introduced by this update.

Thanks! I think I need to start revisiting this project and integrating with dependabot for a future update 😅


create-devstack-local-conf: fix a typo in the template

This is bash environment, not YAML. Sorry for the typo.

ping @wangxiyuan
sorry, I think it's the last PR for today :)

ping @wangxiyuan sorry, I think it's the last PR for today :-)

no warry, feel free to add more patch :)
I'll merge them once I notice.


!bigint tag

I think it'd be nice if there was a custom tag to explicitly tag BigInt values.
However, this isn't quite working.  It seems like yaml.stringify() is turning BigInts into Numbers, even if the tag identifies the value.
Haven't dug into it more than a cursory glance to see that intAsBigInt behavior seems to perhaps be conflicting with what this is trying to do, and it's a nice-to-have at best, so if it's too hard to figure out, then meh. If there's some obvious thing I'm missing, then I'll try to get this over the finish line. :)

That sounds like it's because the !!int tags are grabbing the BigInt values before !bigint. The relevant bit of code is here.
So to get something like !bigint to work, there are two current solutions:

Use the schema: 'core' option, as that only includes !!str for scalars.
Use a function for customTags that prepends rather than appends your custom tags.

We should probably include an example of the latter in the README here.

Updated to prepend the tag to the list, and now it does work as expected :)

Simplest solution was just to support the full range of 0000123n that JS allows. 💪
PTAL, thanks :)

Given that there is a workaround, I'd really rather not risk breaking things by changing that anymore. :/

I think using custom tags at all is probably a pretty advanced feature, so I agree the right approach probably is to just leave it as-is and maybe just add a caveat in the docs.

Replaced the regexp with just a test to remove the n from the end of the string if present.  (Could also remove that if the stringify() didn't put it there, but maybe it shouldn't?)
This does make some of the tests kind of redundant, since now they're just testing the JS BigInt() implementation, but seems harmless to leave in 😅

Let's keep the trailing optional n.

I like it too.  Makes it feel more biginty.


Shuffle completely breaks the playlist's indexies

Hello! First of all, I wanted to say thank you for your package. Compared to just_audio_windows, your solution is much more stable.
Unfortunately, the shuffle feature is broken here: Calling setShuffleModeEnabled(true) results in a weird indexes being returned from the player class:
print(queue.sequence[player.currentIndex]) # Prints current track.
await player.setShuffleModeEnabled(true);

print(queue.sequence[player.currentIndex]) # Outputs the wrong track although the playlist haven't been changed.

Not only currentIndex is broken, but nextIndex (and probably previousIndex as well). Because of that issue, my UI displays the wrong track after user tries to enable the shuffle.
Second, just_audio's setShuffleModeEnabled can be called before calling player.setAudioSource, but media_kit doesn't allow that. Because of that, the playlist doesn't get shuffled.
Comparing just_audio_media_kit with native (?) Android version of just_audio, there aren't any issues with the shuffle: I can call the setShuffleModeEnabled before setting the playlist, and indecies are returning correctly.

Hi! As specified in the 'Caveats' section in the Readme, shuffle indices are currently not supported, and I couldn't work on supporting them for a lack of time, as media_kit itself doesn't support them either as far as I'm aware, so I'd need to manually change the queue (it only supports specifying whether the player has to shuffle or not).
As a temporary solution, you could manually shuffle the queue in your application and set that as the player's queue.

Thank you for your reply.

As specified in the 'Caveats' section in the Readme

Hm... I'm sorry, but I can't find that section in the README.md file. Are you certain that you have pushed the changes to the Github? 👀

shuffle indices are currently not supported, [...] as media_kit itself doesn't support them either as far as I'm aware

I've been using media_kit instead of just_audio in my project before. And yes, media_kit's shuffle implementation is awful, I can agree with that.
Due to awful shuffle/queue implementation on media_kit's side, I've had to rely on Spotube's solution: this implementation stores player's queue in a separate variable, and then feeds the player with currently playing media, one by one, instead of giving the player the complete playlist.

sorry for the late reply

Not a problem, better to be late than never :D
If you aren't planning to continue working on that package, then I may try something myself; can't promise anything tho :D

Hm... I'm sorry, but I can't find that section in the README.md file. Are you certain that you have pushed the changes to the Github? 👀

You're right.. I forgot to publish it. 💀

Spotube's solution: this implementation stores player's queue in a separate variable, and then feeds the player with currently playing media, one by one, instead of giving the player the complete playlist.

Yeah, I wanted to implement something like that in the plugin, so that shuffleIndices would be supported, just like natively on Android.

If you aren't planning to continue working on that package, then I may try something myself; can't promise anything tho :D

I am trying to work on it (also because I use it myself in my app), albeit slowly because of the lack of time. Though you're welcome to make your forks and eventually help me with development by submitting a PR.

Hi, we're also trying to use this package for our media player (Finamp), but it seems like the queue isn't properly working for some reason. When one song is finished, the player stops playback (Stopping audio service) and the current MediaItem becomes null. Skipping to the next track works just fine, so I'm not sure where the problem is.
We're using a ConcatenatingAudioSource to play MP3s via HTTP, but the issue also appears when playing local files.
It seems like that bug is related to the discussion here, but I can also open a separate issue if you prefer :)

Hi, we're also trying to use this package for our media player (Finamp), but it seems like the queue isn't properly working for some reason. When one song is finished, the player stops playback (Stopping audio service) and the current MediaItem becomes null. Skipping to the next track works just fine, so I'm not sure where the problem is. We're using a ConcatenatingAudioSource to play MP3s via HTTP, but the issue also appears when playing local files.
It seems like that bug is related to the discussion here, but I can also open a separate issue if you prefer :)

Hi! The bug is not really related to this discussion, as this discussion is about the shuffle indexes not working (which is also stated in the README.md file). What you're talking about instead, is that the queue is completely broken, so yes, a new issue would be better. I'll start looking at the issue in the meantime, and also at the pull request you've opened.

Coming back to this now that my other issues have been resolved, I'd prefer a solution that fully updates media-kit's playlist instead of only feeding the next track.
I believe that makes more sense with regard to features like precaching (#11), and is also easier to implement (less edge-cases to handle; skipping back, forth, or jumping to a specific index works just like before). That is, unless there are major issues with replacing media-kit's playlist (we basically need to remove everything before and after the current track, and then add the new items before and after the current track).
I'm not sure if I'll have the time to play around with this myself, but I'd be happy to discuss this with you further!

Hello! I've made my own version of just_audio_media_kit that implements the solution I've been discussing before.
Not sure if I should make a PR for this, but here we are: Zensonaton/just_audio_media_kit. Basic features like seek (previous/next) are working as intended, while queue modification methods have not been tested.
In case if you want to check my repo without merging:
just_audio_media_kit:
  git:
    url: https://github.com/Zensonaton/just_audio_media_kit.git
    ref: main

@Zensonaton I just tried your fork, but it doesn't seem to be working for some reason. Toggling shuffle mode doesn't change the order of the tracks, neither in the effectiveSequence nor in what is actually being played.
I do get an error in the logs though:
UnimplementedError: setShuffleOrder() has not been implemented

which seems to originate from here:
https://github.com/ryanheise/just_audio/blob/6952be55c6ddfb01f2567c0737ef098d1bd9fd45/just_audio_platform_interface/lib/just_audio_platform_interface.dart#L124-L128
Is there anything I need to change about my code to make use of your implementation?

@Chaphasilor, thank you for trying my fork.

I just tried your fork, but it doesn't seem to be working for some reason.

Ouch. This is why I wasn't comfortable in creating a PR :D
Interestingly, my own version of just_audio_media_kit works perfectly in my app.

Toggling shuffle mode doesn't change the order of the tracks, neither in the effectiveSequence nor in what is actually being played.

Hmmm, I thought that this is how things were supposed to be working. just_audio returns shuffleOrder only once, in the load method, this is why indecies aren't changing upon calling setShuffleMode multiple times.

setShuffleOrder() has not been implemented

Yeah, both Pato05's and mine implementations of just_audio_media_kit doesn't have a setShuffleOrder method. I've updated my fork to implement setShuffleOrder method. However, be warned! Due to lack of time on my side, I'm unable to properly test that change, so feel free to leave the feedback.

I also noticed that I don't get any changed order when using the published version of just_audio_media_kit. In your original comment you mentioned that the order changes for you, is that still the case?

Yeah- and no. In the original just_audio_media_kit library, shuffleOrder was completely ignored, as also stated in the README.md. My PR changes that behavior by creating its own queue, that plays media from given just_audio's queue one-by-one, while also respecting the shuffleOrder.

It's amazing @Zensonaton you've created a fork covering this! I have seen your commits, and really good job. Though your implementation as of right now would be incompatible with the prefetching I've introduced some commits ago. Maybe we could keep a cache of all the sources and then feed something like 4 at a time to the player, and whenever the shuffle order changes, we could just re-iterate over those few elements and add them to the underlying player.
Something like this would allow the player to pre-fetch the next track, and there wouldn't be any issues regarding the player automatically going to the next track (instead of having it be fed).
If you want, you could make a PR already, and mark it as a draft, so that I can also try suggesting changes. I just have to ask one thing before doing this: please merge the changes, since it's quite behind some commits.

Also @Chaphasilor if you need shuffling in your player, instead of using just_audio's methods, you can just shuffle the queue, and use that as new queue (by calling AudioPlayer.setAudioSource(...)). That's what I'm currently doing in my app.
If there are a lot of items in the queue, you can take a look at the compute function as to avoid freezing the main thread.

Take a look at the compute ...

Wait, I thought that the load method sends the queue to the underlying player in a separate thread. Thank you for telling me that, I'll definitely try that in my app 👀

@Pato05 that's not really an option sadly. Not only would it mean re-implementing a lot of functionality (like our custom ShuffleOrder), but it would probably also break our playback history, and playback would be restarted when toggling shuffle.
Not having shuffle for now is better imo.


Enable edit issue comment for user who posted an comment

Posted Issues and replies to the issue should be editable (ideally for a period of time).
When writing commenting an issue, If you forgot to mention something, only way to add something is to create new comment to the issue. That is not easily readable nor convenient. Also due to multiple errors in parsing of markdown present at the moment in gogs, IMHO this is important feature.
Thanks :)

You're right, thanks your feedback!

+1, hard to use gogs for issue tracking without being able to edit the issue and comments

It is now possible to edit issue and comments! https://try.gogs.io/unknwon/gogs/issues

Closed as implemented. :tada:


Unnabl to run the example in Readme

Hi HF Team
Thanks a lot for this repo.
I was exploring the code and trying to run it with the example provided in Readme file.
But it throws this error related to protobuf.
Here is the error:
`2024-03-31 22:07:00.161046: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libcudart.so.11.0'; dlerror: libcudart.so.11.0: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: /opt/amazon/efa/lib:/opt/amazon/openmpi/lib:/opt/aws-ofi-nccl/lib:/usr/local/cuda/lib:/usr/local/cuda/lib64:/usr/local/cuda:/usr/local/cuda/extras/CUPTI/lib64:/usr/local/cuda/targets/x86_64-linux/lib:/usr/local/lib:/usr/lib:/opt/amazon/efa/lib:/opt/amazon/openmpi/lib:/opt/aws-ofi-nccl/lib:/usr/local/cuda/lib:/usr/local/cuda/lib64:/usr/local/cuda:/usr/local/cuda/extras/CUPTI/lib64:/usr/local/cuda/targets/x86_64-linux/lib:/usr/local/lib:/usr/lib
2024-03-31 22:07:00.161086: I tensorflow/stream_executor/cuda/cudart_stub.cc:29] Ignore above cudart dlerror if you do not have a GPU set up on your machine.
Traceback (most recent call last):
File "/home/ubuntu/for_nlp_tasks/R_and_D/text-clustering/test_it.py", line 1, in
from src.text_clustering import ClusterClassifier
File "/home/ubuntu/for_nlp_tasks/R_and_D/text-clustering/src/text_clustering.py", line 17, in
from umap import UMAP
File "/opt/conda/lib/python3.10/site-packages/umap/init.py", line 7, in
from .parametric_umap import ParametricUMAP
File "/opt/conda/lib/python3.10/site-packages/umap/parametric_umap.py", line 14, in
import tensorflow as tf
File "/opt/conda/lib/python3.10/site-packages/tensorflow/init.py", line 37, in
from tensorflow.python.tools import module_util as _module_util
File "/opt/conda/lib/python3.10/site-packages/tensorflow/python/init.py", line 37, in
from tensorflow.python.eager import context
File "/opt/conda/lib/python3.10/site-packages/tensorflow/python/eager/context.py", line 29, in
from tensorflow.core.framework import function_pb2
File "/opt/conda/lib/python3.10/site-packages/tensorflow/core/framework/function_pb2.py", line 16, in
from tensorflow.core.framework import attr_value_pb2 as tensorflow_dot_core_dot_framework_dot_attr__value__pb2
File "/opt/conda/lib/python3.10/site-packages/tensorflow/core/framework/attr_value_pb2.py", line 16, in
from tensorflow.core.framework import tensor_pb2 as tensorflow_dot_core_dot_framework_dot_tensor__pb2
File "/opt/conda/lib/python3.10/site-packages/tensorflow/core/framework/tensor_pb2.py", line 16, in
from tensorflow.core.framework import resource_handle_pb2 as tensorflow_dot_core_dot_framework_dot_resource__handle__pb2
File "/opt/conda/lib/python3.10/site-packages/tensorflow/core/framework/resource_handle_pb2.py", line 16, in
from tensorflow.core.framework import tensor_shape_pb2 as tensorflow_dot_core_dot_framework_dot_tensor__shape__pb2
File "/opt/conda/lib/python3.10/site-packages/tensorflow/core/framework/tensor_shape_pb2.py", line 36, in
_descriptor.FieldDescriptor(
File "/opt/conda/lib/python3.10/site-packages/google/protobuf/descriptor.py", line 553, in new
_message.Message._CheckCalledFromGeneratedFile()
TypeError: Descriptors cannot be created directly.
If this call came from a _pb2.py file, your generated code is out of date and must be regenerated with protoc >= 3.19.0.
If you cannot immediately regenerate your protos, some other possible workarounds are:

Downgrade the protobuf package to 3.20.x or lower.
Set PROTOCOL_BUFFERS_PYTHON_IMPLEMENTATION=python (but this will use pure-Python parsing and will be much slower).

More information: https://developers.google.com/protocol-buffers/docs/news/2022-05-06#python-updates`
I understand that it's mentionned that this project is under progress, but I was really interested to try it, since I'm working on something similar about arabic text clustering and would like to get some good results.
Thanks a lot

is there any news for that?


Gradient of np.sum raises TypeError

Thanks for sharing this awesome project!
Using the GitHub master of autograd and numpy v1.10.2, I tried the following simple function:
import autograd
import numpy as np

def function(x):
    return -0.5*np.sum(x**2)

autograd.grad(function)(np.random.randn(10))

but it raises the TypeError:
TypeError: make_grad_np_sum() got an unexpected keyword argument 'dtype'

I assume that this keyword must have been added in a recent numpy release but I can't work out exactly when. This error won't block my project (I know how to write a function like this in a way that works) but I figured I should share this issue here just in case.
Thanks again for your work on this module!

You wrote import numpy as np but you need to write import autograd.numpy as np.

To say a bit more, autograd works by wrapping numpy's functions so that when evaluating a gradient they do some extra bookkeeping on the side. That's why you need to replace imports like import numpy as np with import autograd.numpy as np.
It's an easy mistake to make (I did the same in #49!) and the error message is pretty opaque. I just added a hint to the error message, so that when running your test case above this prints out at the bottom of the traceback:
AutogradHint: This error *might* be caused by importing numpy instead of autograd.numpy, so check that you have 'import autograd.numpy as np' at the top of your file instead of 'import numpy as np'.
Sub-exception:
TypeError: make_grad_np_sum() got an unexpected keyword argument 'dtype'

Oh my that is embarrassing! Thanks for the postscript :-)


Correction of data types in JSON data for the all.json end point

Currently most of the data types in the JSON data stores for certain numeric fields are string however, this should not be the case, these should be numbers instead.
Take a look at the state/v1/all.json file for example below. The population and total_area should be numbers instead of strings.
    {
        "administrative_division": "State of Malaysia",
        "state": "Johor",
        "capital": "Johor Bahru",
        "royal_capital": "Muar",
        "population": "3,794,000", <---------------- This should be number
        "total_area": "19,166",  <----------------       This should be number
        "licence_plate_prefix": "J",
        "phone_area_code": "07, 06 (Muar & Tangkak)",
        "abbreviation": "JHR",
        "ISO": "MY-01",
        "FIPS": "MY01",
        "HDI": "0.818",
        "region": "Peninsular Malaysia",
        "head_of_state": "Sultan",
        "head_of_goverment": "Menteri Besar"
    },

This should be corrected for all entries.

please assign this issue to me. I would like to work on it.

Assigned issue. @Frey0-0 👍

I have made a PR for this issue.

Thank you.


[Feature]Allow SkipCertificateCheck

Is your feature request related to a problem? Please describe.
Vault with self-signed certificates currently won't function.
Describe the solution you'd like
Allow switch in secretvault config to bypass TLS checks. Can leverage the SkipCertificateCheck switch built into invoke-restmethod for 6.0 and above.
Describe alternatives you've considered
Tried some of the workarounds like without luck
[System.Net.ServicePointManager]::ServerCertificateValidationCallback = { return $true }
Additional context
Add any other context or screenshots about the feature request here.

This is a great idea. I can think of one way to implement this would be to put it as an option when the Vault is registered. The likely reason your workaround didn't work is because of the way that the module loads up in a run-space (which also makes it inherently tricky to troubleshoot).
Would you want to give it a go on adding this feature? I can help you work on this if you aren't sure where to start.

Sure! I could take this on.

Your Code is now out there on the gallery under 2.0.1-Preview. Please close this issue if the function is working correctly for you.


"Welcome" site in Chrome has odd behavior on last dialog

http://github.hubspot.com/vex/docs/welcome
On a 7" nexus in landscape mode, the demo works fine until that last dialog, what happens repeatedly is that it snaps to bottom left, so effectively when I then try to click the "X" to close it out, it thinks I'm hitting the twitter link
Chrome 37.0.2062.117
Edit: was playing around on tablet initially, when I got on laptop same deal under Chrome, 37.0.2062.120 m

Though we already fixed this with a workaround, the underlying Chrome bug has now been fixed! https://code.google.com/p/chromium/issues/detail?id=426157 :)


Support for AWSSDK.Core 3.5?

Description
When referencing any package that has AWSSDK.Core 3.5 as a dependency then we get the following issue:
Detected package version outside of dependency constraint: XXX requires AWSSDK.Core (>= <IP_ADDRESS> && < 3.4.0) but version AWSSDK.Core <IP_ADDRESS> was resolved.
Reproduction Steps
Install any AWS SDK library version that references AWSSDK.Core 3.5+ alongside this one.
Logs

Severity	Code	Description	Project	File	Line	Suppression State
Warning	NU1608	Detected package version outside of dependency constraint: AWSSDK.CognitoIdentity 3.3.100 requires AWSSDK.Core (>= 3.3.100 && < 3.4.0) but version AWSSDK.Core <IP_ADDRESS> was resolved.	XXX.csproj	1
Warning	NU1608	Detected package version outside of dependency constraint: AWSSDK.CognitoIdentityProvider 3.3.100 requires AWSSDK.Core (>= 3.3.100 && < 3.4.0) but version AWSSDK.Core <IP_ADDRESS> was resolved.	XXX.csproj	1
Warning	NU1608	Detected package version outside of dependency constraint: AWSSDK.Extensions.NETCore.Setup <IP_ADDRESS> requires AWSSDK.Core (>= 3.3.100 && < 3.4.0) but version AWSSDK.Core <IP_ADDRESS> was resolved.	XXX.csproj	1
Warning	NU1608	Detected package version outside of dependency constraint: AWSSDK.SecurityToken 3.3.100 requires AWSSDK.Core (>= 3.3.100 && < 3.4.0) but version AWSSDK.Core <IP_ADDRESS> was resolved.	XXX.csproj	1
Environment

Build Version: 1.0.5
OS Info: Windows 10
Build Environment: Visual Studio
Targeted .NET Platform:  .NET Core 3.1

Resolution
Any ETA on support for AWSSDK.Core 3.5? Thanks!

This is a :bug: bug-report

I am having the same issue. I find it bad that in over a month nobody from AWS has responded.

This should be fixed in Amazon.AspNetCore.Identity.Cognito 2.0.2.

This should be fixed in Amazon.AspNetCore.Identity.Cognito 2.0.2.

Still cant update.


User reports that code linking doesn't work in the guide.

I have a report that code linking is broken in the guide.  It works for me in both Chrome and Firefox on Linux, as well as Chrome/ChromeOS.  But it might be that there's a browser incompatibility somewhere.
Code is: https://github.com/google/codeworld/blob/master/web/js/doc.js#L30

This isn't actionable without more info.


Treeshaking

[ ] For a big file, check what part of the file is relevant for your component and extract it

Acceptance testing

[ ] Check on https://pagespeed.web.dev/ the performances mobile and then desktop

@mahfouzhseineweev don't forget that this needs to be tested as well on site not using meta sections so please when done to add an archived version of the library on geeks website dev, like this I can test there as well

Report:
For this issue as it is mainly about removing unused js codes, I have found a way to analyze the bundlers created upon a build in Nuxt js which is support out of box and can be enabled by adding analyze: true to the build in nuxt.config.js file.
Note that Nuxt js which uses Webpack, supports tree-shaking out of the box and no configurations needed to enable it.
After running the analyze I got the below file created which has a summary of all bundlers contents and their sizes.

The quill.js and quill-emoji.js are taking the biggest size across the bundlers and it seems the problem of unused codes is coming from these 2 scripts.
I still have to confirm this as I still need to check in details using the sources coverage that is supported in the devtools out of the box and can point to unused codes by there lines.
The updates needed to adjust the performance for unused js codes is not going to be straight forward and it is unknown currently what are the updates that has to be performed.
As mentioned, more debugging is needed to find what codes exactly are causing the big amount of unused js codes and to check if there is a clean way to remove them

Report:
For this issue as it is mainly about removing unused js codes, I have found a way to analyze the bundlers created upon a build in Nuxt js which is support out of box and can be enabled by adding analyze: true to the build in nuxt.config.js file.
Note that Nuxt js which uses Webpack, supports tree-shaking out of the box and no configurations needed to enable it.
After running the analyze I got the below file created which has a summary of all bundlers contents and their sizes.

The quill.js and quill-emoji.js are taking the biggest size across the bundlers and it seems the problem of unused codes is coming from these 2 scripts.
I still have to confirm this as I still need to check in details using the sources coverage that is supported in the devtools out of the box and can point to unused codes by there lines.
The updates needed to adjust the performance for unused js codes is not going to be straight forward and it is unknown currently what are the updates that has to be performed.
As mentioned, more debugging is needed to find what codes exactly are causing the big amount of unused js codes and to check if there is a clean way to remove them

If treeshaking is performed out of the box, why is quill's codebase so big ? It was detected as an opportunity to be optimized so I'm assuming that a lot of its code is unused... are you sure treeshaking is being performed on Quill's js ?
Are you sure the way the quill plugin is added is "standard" and allow's nuxt's tree shaking to be performed as it normally should ?

are you sure treeshaking is being performed on Quill's js ?

No this is still unknown

Are you sure the way the quill plugin is added is "standard" and allow's nuxt's tree shaking to be performed as it normally should ?

It is standard and should allow nuxt tree shaking but I am not sure if quill library itself allow this @jfayad

@mahfouzhseineweev what makes you believe that a library can "forbid" tree shaking from taking place ?

@jfayad I am not sure but I have read the below:
Tree shaking relies on the static structure of ES6 modules (using import and export). Tools like Webpack or Rollup analyze the import/export statements to determine which parts of the code are being used.
And because the library we are using is dead and no more maintained, it could be using old ES modules not supported for tree shaking


scsictl: Create files with binary/JSON or text format protobuf data

@rdmark With github's "Hide whitespace" option enabled these differences are invisible. Just tested it with this: It makes a big difference, just give it a try.


chore: Fix deprecation warning for sphinx.testing.path

Fixes
tests/conftest.py:10
  .../sphinx-autodoc-typehints/tests/conftest.py:10: RemovedInSphinx90Warning: 'sphinx.testing.path' is deprecated. Use 'os.path' or 'pathlib' instead.
    from sphinx.testing.path import path

os.path and pathlib are not drop-in replacements for sphinx.testing.path. Sphinx 7.2 dropped support for Python 3.8 and the last version that supports it (7.1) depends on specific functionality from sphinx.testing.path.
You can either drop support for Python 3.8 or close this PR and hope that Python 3.8 goes EOL before Sphinx 9.0 comes out and removes sphinx.testing.path :shrug:


give me Max the Elf

please give me Max the Elf in mixer manor discord server I required him to live...

No :trollface:

Ok fair point you won't see Yuri alive anymore.


transition property on utils/Frame breaks navigation scroll preservation

Describe the bug
When navigating from a page that uses the Navbar component, scroll preservation of a previously visited site is broken, and the scroll is "all over the place" (depending on browser and kind of navigation [e.g. to anchor tag]). After some digging, I found that the culprit is this line in `utils/Frame.svelte:
<svelte:element
  this={tag}
  use:use={options}
  bind:this={node}
  transition:transition={params} // <---- This breaks navigation!
  {...$$restProps}
  class={divClass}
  on:click
  on:mouseenter
  on:mouseleave
  on:focusin
  on:focusout
  {role}>
  <slot />
</svelte:element>

When pasting the compiled HTML back into svelte instead of the Navbar component, or recreating it with a copy and paste of Frame.svelte with the transition omitted, everything works again.
This is mainly visible when using the browser back button or window.history.back(), but also causes weird behavior when navigation to an anchor tag on Safari, where it just scrolls past the tag ^^
Thank you again for this awesome library! :)
Reproduction
https://stackblitz.com/edit/sveltejs-kit-template-default-wvhzf6
Please scroll down until you find the test buttons and wait for hydration on first load :)
Flowbite version and System Info
System:
    OS: Windows 10 10.0.19044
    CPU: (16) x64 AMD Ryzen 7 2700X Eight-Core Processor
    Memory: 6.26 GB / 15.95 GB
  Binaries:
    Node: 18.16.0 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.19 - ~\AppData\Roaming\npm\yarn.CMD
    npm: 8.1.4 - C:\Program Files\nodejs\npm.CMD
    pnpm: 8.6.7 - ~\AppData\Local\pnpm\pnpm.EXE
  Browsers:
    Chrome: 115.0.5790.102
    Edge: Spartan (44.19041.1266.0), Chromium (114.0.1823.82)
  npmPackages:
    @sveltejs/kit: ^1.20.4 => 1.22.3
    flowbite-svelte: ^0.40.1 => 0.40.1
    svelte: ^4.0.0 => 4.1.0
    vite: ^4.3.0 => 4.3.9

This seems to only happen if the Navbar component is inside the exchanged component, not if it lives above (e.g. in the layout).

Just for people to track this issue: https://discord.com/channels/457912077277855764/750401468569354431/1147275274631782551
The thread also includes proposed changes that would add a |nonav modifier that fixes this issue :)

Oh wow... It's just on the Svelte Discord server... I'll just dump the important part here and will update this thread if anything important happens :)

I am currently experiencing a bug where a transition on a page will break scroll restoration on navigation between routes.
This caused me to add a |nonav modifier to transitions that will prevent them from running when Kit navigates.
I implemented this in Svelte and SvelteKit by using globalThis in both the SvelteKit client and Svelte client transition code to communicate that a navigation is happening. The Svelte compiler then adds a little check (just like with |local) to prevent the animation from running if Kit is currently navigating.
Now the real question: Is there any preferred/recommended way to communicate between SvelteKit and Svelte? I don't see how I can rely on beforeNavigate etc. to fix this issue or not modify the compiler itself. The bug is triggered with even an empty null_transition as it's a "race condition" between requestAnimtionFrame and the kit navigation code.
This is actually not a bug per the documentation but breaks a lot of UI and could be simply avoided in a non-breaking way for old sites. I'd like some feedback on this "communication" approach so that I can submit my adjustments as a PR.
Issue reproduction:
https://stackblitz.com/edit/sveltejs-kit-template-default-atnnds
Changes in Kit:
https://github.com/Coronon/kit/commit/fa00aea7a97a9918fcb1857aa1376c70698893f4
Changes in Svelte:
https://github.com/Coronon/svelte/commit/8db526159c81c1a4db58bdb4a64a3244117f5304
Related issue: https://github.com/sveltejs/svelte/issues/9028

Added PR that could fix this with a new condition property on TransitionConfig: https://github.com/sveltejs/svelte/pull/9272

This issue should be opened again. It still exists and have run into it because of using the Navbar component repeatedly across different pages. I was able to work around it by restructuring my layouts so that Navbar is only ever loaded once at a higher level layout.


I want to change Method/Function's color in C#,what should i do?

Do‘t change local/system symbols/fields.
This used Visual Assist, I want like this but not used plugin.

If you have the theme editor installed:
Make a copy of the theme you are using. Click the edit theme button on the copy. In editor tab that comes up select the "Show All Elements" button on the toolbar.  Use the search box to find the "Method" colors. The entries you will want to change will be listed under Text Editor -> User Members.
Hope that helps


casper-node should handle calling endpoints on stored contracts for Version1 transactions

Currently if I try to call an endpoint for a stored contract using the Version1 variant of Transaction I will get a silent error which originates here https://github.com/casper-network/casper-node/blob/release-2.0.0-rc2/node/src/components/contract_runtime/operations.rs#L290
When doing finalized block execution and the Transaction is an EntryPoint call the node needs to check for EntryPointPayment of that EntryPoint prior to BalanceIdentifier handling. And if the EntryPointPayment is "Caller"  - it shouldn't penalize.

The logic referred to was removed in rc4. This should be working in dev net already.

@Jiuhong-casperlabs can you re-test this on dev net to determine if it can be closed or if I need to spend time digging into it?

@Jiuhong-casperlabs can you re-test this on dev net to determine if it can be closed or if I need to spend time digging into it?

It was fixed.
Transaction hash for your reference.


Make discarding empty strings optional

This adds a new option, called discardEmpty, that when set to false will keep empty translations. The current behavior is it always discards empty strings. It also defaults to true, in order to keep old functionality the same.

@tonyskn Here's a small improvement I made. I needed this in a project I'm working on, and this simple solution did it for me.


Followup of #244

As the author of #244 didn't respond in a year, I just wrote a followup to finally get this feature merged.

@mafintosh Sorry to ping you but I thing this feature should definitely be merged. This resolve #225 and #264 allowing plugins to work with the filename.
Thanks for your consideration.

Got excited when I saw this patch as I've been meaning to have a got at it myself, but it turns out it crashes my peerflix when launching with vlc :(


feature: storage rbd add auto migrate on start, if memory not enough

这个 PR 实现什么功能/修复什么问题:
storage rbd add auto migrate on start, if memory not enough
是否需要 backport 到之前的 release 分支:

Codecov Report

Merging #584 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master    #584      +/-   ##
=========================================
- Coverage    7.38%   7.38%   -0.01%
=========================================
  Files         346     346
  Lines       54693   54712      +19
=========================================
- Hits         4041    4040       -1
- Misses      50274   50294      +20
  Partials      378     378

Flag
Coverage Δ

#aFlag
7.38% <0%> (-0.01%)
:arrow_down:

Impacted Files
Coverage Δ

pkg/compute/models/hosts.go
0.37% <0%> (-0.01%)
:arrow_down:

pkg/compute/models/guest_actions.go
0% <0%> (ø)
:arrow_up:

pkg/util/rbacutils/rabc.go
68.56% <0%> (-1.14%)
:arrow_down:

pkg/compute/models/guests.go
0.66% <0%> (ø)
:arrow_up:

pkg/util/seclib2/seclib.go
95.74% <0%> (+4.25%)
:arrow_up:

Continue to review full report at Codecov.

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

/hold

/area region

/lgtm

/hold cancel
/approve


64bit integer / array BE or LE / multidimensional arrays / maybe a different syntax?

I'm submitting a ...

[x] feature request

Summary
I noticed a few changes that could be made to improve even more this amazing lib, here are they:

Support for 64 bit integer

Specify int array to be BE or LE

Support for multidimensional arrays (this one I believe is not extremely necessary, but would be a huge plus!)

For multidimensional array I thought of a new declaration, for example:
const test = new Struct("Test")
.field('fieldA', 'array', 4, 'UInt16LE') // array with explicit LE type
.field('fieldB', 'array', 4, 'array', 10, 'UInt16LE') // 2D example
.field('fieldC', 'array', 12, 'string', 20, 'win1251') // string array
.field('fieldD', 'UInt64LE') // 64 bit integer
.field('fieldE', 'string', 32, 'win1251') // regular string
.field('fieldF', 'array', 2, 'array', 4, 'array', 8, 'UInt32LE') // 3D example ...
.compile()

Will add support soon
Typed arrays do not support endianness, use swap
I think it is a good idea
In this case, there may be problems with type inference in Typescript. I'll think about it

In this case, there may be problems with type inference in Typescript.  I was doing a wrapper for the struct library (pretty old one), and what I did was 4 functions: number, string, struct and array, that worked just fine!


 I build myself a linter for this. Thanks 👍🏻

Does the command work for your use case?

My usecase was in nodejs so I prefer to use the api - i found calling replacer.getOAuth2Response slightly better than the last snippet I posted. I elaborated on it in this blog post.
I think httpyac --flow implicit -e test is not picking up env variables - i am writing a test that proves this.

I have tested it with .env variables. But I was in the same folder as the variables, maybe there is the error.

I get tutoring now:-)

nice, i use the same!


Load onnx model error in trtis 19.09 container?

There is a onnx model that was generated by use the torch.onnx.export interface,pytorch version is 1.3.0,my trtis image version is 19.09,and the error follow:
I0708 02:37:22.406269 72 server.cc:110] Initializing TensorRT Inference Server E0708 02:37:24.532915 72 logging.cc:43] ../rtSafe/coreReadArchive.cpp (31) - Serialization Error in verifyHeader: 0 (Magic tag does not match) E0708 02:37:24.532988 72 logging.cc:43] INVALID_STATE: std::exception E0708 02:37:24.532997 72 logging.cc:43] INVALID_CONFIG: Deserialize the cuda engine failed.
I find the explains that from https://forums.developer.nvidia.com/t/tensorrt-error-rtsafe-corereadarchive-cpp-31-serialization-error-in-verifyheader-0-magic-tag-does-not-match/81872 , Why use the onnx model to predict result that use tensorRT engine and tensorRT?

I have opened the verbose option,got the message:
`I0708 07:20:51.406790 87 ensemble_backend_factory.cc:47] Create EnsembleBackendFactory
I0708 07:20:51.407844 87 autofill.cc:138] TensorFlow SavedModel autofill: Internal: unable to autofill for 'NoName', unable to find savedmodel directory
I0708 07:20:51.407868 87 autofill.cc:149] TensorFlow GraphDef autofill: Internal: unable to autofill for 'NoName', unable to find graphdef file named 'model.graphdef'
I0708 07:20:51.407887 87 autofill.cc:162] PyTorch autofill: Internal: unable to autofill for 'NoName', unable to find PyTorch file named 'model.pt'
I0708 07:20:51.407905 87 autofill.cc:175] Caffe2 NetDef autofill: Internal: unable to autofill for 'NoName', unable to find netdef files: 'model.netdef' and 'init_model.netdef'
I0708 07:20:51.426046 87 autofill.cc:198] ONNX autofill: Internal: onnx runtime error 10: This is an invalid model. Type Error: Type 'tensor(bool)' of input parameter (51) of operator (ReduceSum) in node () is invalid.
E0708 07:21:16.162082 87 logging.cc:43] ../rtSafe/coreReadArchive.cpp (31) - Serialization Error in verifyHeader: 0 (Magic tag does not match)
E0708 07:21:16.177306 87 logging.cc:43] INVALID_STATE: std::exception
E0708 07:21:16.177378 87 logging.cc:43] INVALID_CONFIG: Deserialize the cuda engine failed.
I0708 07:21:16.177433 87 autofill.cc:211] TensorRT autofill: Internal: unable to autofill for 'NoName', unable to create TensorRT runtime and engine
I0708 07:21:16.177464 87 model_config_utils.cc:203] autofilled config: name: "NoName"
max_batch_size: 128
instance_group {
count: 1
gpus: 0
kind: KIND_GPU
}
E0708 07:21:16.177596 87 main.cc:943] error: creating server: INVALID_ARG - must specify platform for model 'NoName'
`
And found ReduceSum does not support tensor(bool) as per the ONNX spec.https://github.com/microsoft/onnxruntime/issues/2848

I have opened the verbose option,got the message:
`I0708 07:20:51.406790 87 ensemble_backend_factory.cc:47] Create EnsembleBackendFactory
I0708 07:20:51.407844 87 autofill.cc:138] TensorFlow SavedModel autofill: Internal: unable to autofill for 'NoName', unable to find savedmodel directory
I0708 07:20:51.407868 87 autofill.cc:149] TensorFlow GraphDef autofill: Internal: unable to autofill for 'NoName', unable to find graphdef file named 'model.graphdef'
I0708 07:20:51.407887 87 autofill.cc:162] PyTorch autofill: Internal: unable to autofill for 'NoName', unable to find PyTorch file named 'model.pt'
I0708 07:20:51.407905 87 autofill.cc:175] Caffe2 NetDef autofill: Internal: unable to autofill for 'NoName', unable to find netdef files: 'model.netdef' and 'init_model.netdef'
I0708 07:20:51.426046 87 autofill.cc:198] ONNX autofill: Internal: onnx runtime error 10: This is an invalid model. Type Error: Type 'tensor(bool)' of input parameter (51) of operator (ReduceSum) in node () is invalid.
E0708 07:21:16.162082 87 logging.cc:43] ../rtSafe/coreReadArchive.cpp (31) - Serialization Error in verifyHeader: 0 (Magic tag does not match)
E0708 07:21:16.177306 87 logging.cc:43] INVALID_STATE: std::exception
E0708 07:21:16.177378 87 logging.cc:43] INVALID_CONFIG: Deserialize the cuda engine failed.
I0708 07:21:16.177433 87 autofill.cc:211] TensorRT autofill: Internal: unable to autofill for 'NoName', unable to create TensorRT runtime and engine
I0708 07:21:16.177464 87 model_config_utils.cc:203] autofilled config: name: "NoName"
max_batch_size: 128
instance_group {
count: 1
gpus: 0
kind: KIND_GPU
}
E0708 07:21:16.177596 87 main.cc:943] error: creating server: INVALID_ARG - must specify platform for model 'NoName'
`
And found ReduceSum does not support tensor(bool) as per the ONNX spec.https://github.com/microsoft/onnxruntime/issues/2848

So the model is invalid it seems.