If the output drive/location no longer exists, 5 separate error windows come up, and the application won't let you change the location.

Describe the bug
If the output drive/location no longer exists, 5 separate error windows come up, and the application won't let you change the location. I tried looking in the files for a config that I could maybe change manually but didn't see one. Click on "set output folder" simply doesn't do anything.
To Reproduce
Steps to reproduce the behavior:
You should be able to test it with a flash drive or external hard drive. I was using an external drive to store them which I then had to remove and I forgot to change the output folder. So set the output to a drive, close application, remove drive, start it back up.
Expected behavior
I believe it should start normally, let you know that the destination is no longer valid, and either go back to the default, or force you to choose a new location.
Screenshots
https://i.imgur.com/AtH9gGx.png

I've added a check to see if the output drive exists. If not, then fallback to default output directory.
Here's a build to test: https://ci.appveyor.com/api/buildjobs/9fydji5meag3n966/artifacts/temp%2FCaptura-Portable.zip.
Please note that this build is a dev build and the features in it are considerably different from those in releases.
If you want to change the config file, it is located at %AppData%\Captura\Captura.json for setup builds, and in Settings folder in the app directory for Portable builds.
There is an OutPath option in the file which you can change manually if needed.


[5.3] Add even and odd to the $loop variable

I don't have a practical use case for this options on my current project, but looks reasonable have it.
@foreach ($users as $user)
        @if ($loop->even)
        	// Do something
        @endif
@endforeach

I would just use modulus operator on index.


Twister with device-testing cannot run tests on target on Windows

Describe the bug
When trying to run twister with --device-testing on Windows, all tests being skipped.
Is this a regression? Yes: this PR https://github.com/zephyrproject-rtos/zephyr/pull/79174 causes the issue
Quick workaround:
--- a/scripts/pylib/twister/twisterlib/testinstance.py
+++ b/scripts/pylib/twister/twisterlib/testinstance.py
@@ -263,11 +263,11 @@ class TestInstance:
         simulator = self.platform.simulator_by_name(simulation)
         if os.name == 'nt':
             # running on simulators is currently supported only for QEMU on Windows
-            if (not simulator) or simulator.name not in ('na', 'qemu'):
+            if simulator and simulator.name not in ('na', 'qemu'):
                 return False

             # check presence of QEMU on Windows
-            if simulator.name == 'qemu' and 'QEMU_BIN_PATH' not in os.environ:
+            if simulator and simulator.name == 'qemu' and 'QEMU_BIN_PATH' not in os.environ:
                 return False

To Reproduce
Run this command:
west twister -p frdm_k64f --short-build-path -M -N --device-testing --device-serial-baud 115200 --device-serial COM33 -O D:\o -T .\samples\hello_world\
Expected behavior
Twister with device-testing can run tests on target on Windows
Impact
Twister with device-testing cannot run tests on target on Windows
Logs and console output
INFO    - Using Ninja..
INFO    - Zephyr version: v4.0.0-1261-g0a80c780cb65
INFO    - Using 'zephyr' toolchain.
INFO    - Building initial testsuite list...
INFO    - Writing JSON report D:\o\testplan.json

Device testing on:

| Platform   | ID   | Serial device   |
|------------|------|-----------------|

Junction created for D:\o\twister_links\test_0 <<===>> D:\o\frdm_k64f_mk64f12\samples\hello_world\sample.basic.helloworld
INFO    - Adding tasks to the queue...
INFO    - Added initial list of jobs to queue

INFO    - 1 test scenarios (1 configurations) selected, 1 configurations filtered (1 by static filter, 0 at runtime).
INFO    - 0 of 0 executed test configurations passed (0.00%), 0 built (not run), 0 failed, 0 errored, with no warnings in 7.10 seconds.
INFO    - 0 of 0 executed test cases passed (0.00%) on 0 out of total 871 platforms (0.00%).
INFO    - 0 test configurations executed on platforms, 0 test configurations were only built.

Environment (please complete the following information):

OS: Windows
Toolchain (e.g Zephyr SDK, ...)
Commit SHA or Version used: baa49f6f32d

Additional context

#82020 could also be related.

Oh, indeed. @Dat-NguyenDuy would you mind creating a PR with your fix please ?


amdgpu_top: 0.1.8 -> 0.1.9

Description of changes
https://github.com/Umio-Yasuno/amdgpu_top/releases/tag/v0.1.9
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, 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/)
23.11 Release Notes (or backporting 23.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

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

1 package built:

amdgpu_top


Rename binary to 'ks';  rename 'kubecfg' references to 'ksonnet'

As a part of the migration work from github.com/ksonnet/kubecfg to this repository, we will change the naming of this command line tool to 'ksonnet'. This implies the following changes:

Calling the executable 'ks', so that we can run commands such as ks apply dev
Rename all references of 'kubecfg' in the codebase to 'ksonnet'.

Partially supports issue #1.

So. I'd like to propose that we only change the public-facing stuff instead of the internal APIs, because after kubecon we're going to have to pull the common stuff out and merge it back into a common repository, and we don't want to make life harder than it has to be.

Fixed this -- only binary name change (+docs) and import path fixes.


Issues with new Processes page

Rancher Versions: master 1/19
Steps to Reproduce:

[x] Run Time column is sorting by ID and not by Time

[x] Name & Exit Reason secondary sorts are not by ID

[x] All processes showing up in Recently Completed

Version - v1.4.0-rc7
Verified fixed


Fixed some crashing

This fixes the second problem in #275

What is the actual cause of the crashes?

TypeError: Cannot read property 'filter' of undefined

Probably missing properties or undefined variable


Also add note about default maxRetries of 3 to maxDuration section of the documentation

The default maxRetries setting of 3 is mentioned in the retries section, however the duration section https://failsafe.dev/retry/#duration reads as if maxRetries is then determined by the max duration.

Yes exactly I ran into this today wondering why only 3 attempts were taking place, only rereading the duration section I was confused. However, after reading the javadoc of withMaxDuration it dawned on me.
Regarding your proposed wording I still find it missleading as I never specified maxRetries so I would not assume other constraints as I already have specified a stop constraint via duration. So maybe also add a note to your proposal, something along the lines:

You can add a max duration for an execution, after which retries will stop (if not stopped by other constraints, like maxRetries/Attempts, Note: maxRetries is set to 3 if not overridden).

@jhalterman - Would you want a wording change here, or an earlier emphasis on how maxRetries/Attempts is always considered, or both?

Maybe a wording change that does both...
It might be good to move the "Duration" section of the Retry docs to be right below the "Attempts" section, since they both serve the same purpose of constraining executions, as you point out. For proposed wording on "Duration", which would directly follow "Attempts", maybe:

In addition to max attempts, you can also add a max duration for an execution, after which retries will stop if the max attempts haven't already been reached.

Would those two changes clarify things?

I just pushed this change and also added a note about disabling retries: https://github.com/failsafe-lib/failsafe.dev/commit/4ae3ec32d0ad5a0d10e2a195770b22ec00ff0a07

@jhalterman much better now, thank you for the quick fix


Update README to include yarn

A user indicated that it was not clear that the setup-node action took care of yarn, and tried using an incompatible third-party action.  Better specify that yarn is included above the fold.

allout58 is correct.  doesn't install and add yarn to path.  only node/npm

allout58 is correct.  doesn't install and add yarn to path.  only node/npm


Support restricted use accounts

SUMMARY
At genesis block, create way to set tokens locked up from investors, grantees, employees to be used for delegation on the network.
ISSUE TYPE
In order to allow for broader token engagement on the network without creating too much liquidity early on, we'd like to have a way for stakeholders to use the tokens promised to them before they're able to transfer them or sell them on an exchange.
COMPONENT NAME
Consensus layer
ADDITIONAL INFORMATION
when does lockup end? -- example would be a tiered lockup with a certain amount of tokens released at 4 months, 8 months, 12 months.

what exactly can be done with locked up tokens -- locked tokens can be delegated and rewards can be earned on those tokens. locked up tokens cannot be sold or transferred.

The safest way to implement this would probably be a separate "restricted" sub-account with its own balance and a policy that would define what the account holder can do. This way the balance would not be part of the general balance.
A downside of this is that it adds more complexity to the consensus layer.

locked up tokens cannot be sold or transferred.

While you can prevent transfer, you can't really prevent selling them. Someone could record the sale off-chain and do the transfer when the tokens thaw.

If you're going to do this (and I'm against it for complexity reasons), it probably should store a vector of balances, just so that the inevitable "we want to add more tokens to the restricted pool with a different thaw date" is easy to implement.

any updates on this?


Small fixes.

Deleted the start_in_vehicle module
Deleted the outdated setting.sqf file in set_face module
Fixed some typos in the call examples of the marker_control module.

So how come start in vehicle is removed?
Also this should target the dev branch please.

So how come start in vehicle is removed?
Also this should target the dev branch please.

I looked at the module a few days ago and noticed no settings file for it so I asked pizza and he said it was a redundant module.

So how come start in vehicle is removed?
Also this should target the dev branch please.

I looked at the module a few days ago and noticed no settings file for it so I asked pizza and he said it was a redundant module.

OK fair can you please just recreate the PR to target the dev branch instead of master?

So how come start in vehicle is removed?
Also this should target the dev branch please.

I looked at the module a few days ago and noticed no settings file for it so I asked pizza and he said it was a redundant module.

OK fair can you please just recreate the PR to target the dev branch instead of master?

it's not in dev branch, it is only left in master because the dev to master merge didn't delete the extra files

Just needs the setting.sqf for the marker control updated so it has both Add and AddLayer examples

I done this today but just need to add it to the fork.

I done this today but just need to add it to the fork.

next time make a branch for the feature/bugfix on this fork so we can add to it

for future reference once you "commit" to a branch that is already in a PR, the PR will automatically update. You can see the total changed files and commits included in the PR.


memory warning and background kill

Memory warning
Terminate app when in background

This seems solved by upstream rlottie with v0.1.0 official version. Please have a update.
https://github.com/Samsung/rlottie/releases

Thanks , I will close it.
I have switched to another


Support FaaS[Noy fully automated yet]

The tool now supports FaaS and CLI, the FaaS is not supported in the pipeline yet.

What is missing?

Making the cli and the faas independent: so if for some reason the faas is broken, the cli is still working. That would be helpful too if we want to link the faas pipeline by faas releases and the same for the cli.
There is no way to use metacall-deploy in an interactive way, those So,
We need a flag for the metacall-deploy --dev to deploy in a non-interactive way with no env variables by default.
Another flag to get the output of metacall-deploy --inspect --dev in the stdout in a non-interactive way.
@viferga @giarve

Cool.
I have answered you in Discord, when running it as non-interactive mode like: docker run -t ... instead of -it, it runs without asking questions.


Refactor and clean up CI config

Refs #278
Refs #275
Refs #293

Okay, this has brought our CI up to speed. All our test tools are independently installed, so that we can test the best range of PHP versions. PRs will likely need to rebase.


more style improvements

This runs now without issues.

I'm not a major fan of splitting up the return and setting the content. How many passes are you thinking of taking over this code? With each one I am afraid we will accidentally introduce some issue, so I comb through them all very carefully.
What is the end goal of this? How will we know when we reached that goal?

I share your concern about errors. Is there a way to make this easier, for example, one PR for each module or type of linting error?
I guess the end goal is to make the JS runtime easier to maintain, the linting check could even be added to the CI server when the linter passes (almost) all checks.

I think this is too out of date to make sense to try to merge. I'm not sure how much more aggressive I want to be with style stuff at the moment. This stuff just makes me nervous!


web demo does not work on MacBook Air with m1 chip

http://fulldecent.github.io/system-bus-radio/
Was not able to catch something using radio receiver at specified frequency 1560 kHz

Thank you for checking in. Please also try the compiled versions too as they should be more potent. I have only had success with the web version on Intel processolrs myself


Astrodon app store

Astrodon desktop store
I know astrodon still in dev and is not yet stable
But I suggess to plan to build ecosystem around astrodon

astrodon store (AS) :
Take as input dev code ( deno-ts + spa ) + app description and other meta
Then build the app for target plateform (on cloud ) or take as input the build directly
save it on the cloud
and return it the zipped executable to astrodon store client on demand
astrodon store client (ASC):
desktop app
allow to the final user to search for a specific app
install astrodon desktop app by click
under the hood ASC ask for needed permissions and install needed component
This will be good for all :
dev who create desktop app based astrodon : it will be easy to deliver product to there client and it will be easy for client to install here app
astrodon author and mentainer
Will have more conroller of app installation and get possibility to monitise this project then they can recrute some dev for full time.
final user can get good user experience.

In some way I aime to get have ecosystem like playStore+androidStudio+... but for desktop app multiplateform base on deno and web tech.
This is just a suggestion.

@elycheikhsmail Thanks for your idea, but I'm sorry to tell you this is not a priority for astrodon at the moment and probably never will, since for us it doesn't have any sense to make something like this. If you want to make an application store that shows only astrodon projects, go ahead. Probably in the future we will implement a showcase on the website but nothing more than that. I'm closing this issue, greetings.

@elycheikhsmail I'm willing to actively participate if you do

@elycheikhsmail I'm willing to actively participate if you do

Thank, I'm waiting for astrodon stable, but we can discuss about this idea in astrondon discod


Replace legacy rust sdk to new sdk

https://github.com/expo/expo-server-sdk-rust is deprecated.

@oliverwelter Could you review it?


Discord invite link

Discord invite link is outdated and does not work. Is Discord server alive?

The discord server was closed because of  inactivity several weeks ago. Github issue tracker seems enough already for a small project like jok.


Missing ARM template support for assigning multiple user assigned managed identities

In the 2018-06-01 api version, it is not possible to assign an array of user-assigned managed identities. The userAssignedIdentities object expects a dictionary, using copy creates an array so it is not possible to assign to dynamically assign a resource an array of managed identities
Duplicate issue: https://github.com/MicrosoftDocs/azure-docs/issues/21258
Solution suggested there will not work

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

ID: 6dada714-f867-2a46-6b94-eaea6e1da966
Version Independent ID: 25dc8f50-52c6-3c2b-ade6-c5d8258b74ec
Content: Configure managed identities on Azure VM using template - Azure AD
Content Source: articles/active-directory/managed-identities-azure-resources/qs-configure-template-windows-vm.md
Service: active-directory
Sub-service: msi
GitHub Login: @barclayn
Microsoft Alias: barclayn

@poyadav2001
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

Any updates on this issue, can you illustrate a way to assign multiple user-assigned managed identities to a resource using latest ARM template api version. If not, turn it into a feature request for the future

@poyadav2001
Thank you for the follow up on this!
I've re-tagged this issue as a doc-bug and will add the author @barclayn for any further updates.
Thank you again for your time and patience throughout this issue.

Any updates on this issue.
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10
From: James @.>
Sent: Friday, July 9, 2021 10:50 AM
To: @.>
Cc: Pooja @.>; @.>
Subject: Re: [MicrosoftDocs/azure-docs] Missing ARM template support for assigning multiple user assigned managed identities (#78033)
@poyadav2001https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fpoyadav2001&data=04|01|Pooja.Yadav%40microsoft.com|a3ab92efe4ae4fd3a31708d9430219c6|72f988bf86f141af91ab2d7cd011db47|1|0|637614498577047581|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=UJpJq%2BmjI9S6lFXriXKWP3pMLYlq263RfOqDsYkJn7U%3D&reserved=0
Thank you for the follow up on this!
I've re-tagged this issue as a doc-bug and will add the author @barclaynhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fbarclayn&data=04|01|Pooja.Yadav%40microsoft.com|a3ab92efe4ae4fd3a31708d9430219c6|72f988bf86f141af91ab2d7cd011db47|1|0|637614498577047581|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=BxUvJBvfSXgxjB3yn%2FDmm%2BasXYxTOHaD9z6xgrOWbvk%3D&reserved=0 for any further updates.
Thank you again for your time and patience throughout this issue.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F78033%23issuecomment-877355755&data=04|01|Pooja.Yadav%40microsoft.com|a3ab92efe4ae4fd3a31708d9430219c6|72f988bf86f141af91ab2d7cd011db47|1|0|637614498577057582|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=t1QmRxs0tIzz8L%2FgCCx4lqji4DTfOVOo6rfNbkdZ4fk%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FASR7GBEHY43Z5QCZ2THNGATTW4ZH5ANCNFSM477PD26A&data=04|01|Pooja.Yadav%40microsoft.com|a3ab92efe4ae4fd3a31708d9430219c6|72f988bf86f141af91ab2d7cd011db47|1|0|637614498577057582|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=yQDh7yire0%2Bw55qlnB2L2McY3by4GiBfgVOIDM%2FITSw%3D&reserved=0.

@poyadav2001 thank you for your submission. We are addressing this internally. #please-close


Posible Bug with video splash in RetroOrangePi 4.1 for Pi PC

Hi.
I have found a fault in your images for OPI PC.
The images of the version 4.1 for the OPI PC do not allow to use videos like splash.
The mp4 video that comes by default (or any video) does not go. It only plays the audio.
I have tried with 2 different SD.
I have tested with 2 different OPI PCs.
I have tested the image of the USA server and the European server.
Just installed the image, loads retro orange pi, go to menu of options, open retropie-setup.
Configuration Tools -> 831 Splashcreen.
Splashscreen enabled.
Randomizer disabled.
Choose Splashscreen -> own / extra -> splash-720-only.mp4
back, back, reboot ...
When you restart, you only hear the splashscreen audio, there is no video.
Thanks.

fixed in v4.2,  release soon


OnShippingChange not returning the FirstName and LastName

Describe the Bug/Feature
onShippingChange callback  from shippingChangeData?.shippingAddress not returning the firstName and lastName. Fields that are present in the Add New Shipping Address.
Describe alternatives you've considered
I was not able to find an explicit way to retrieve the firstName, lastName from Address<- OnShippingChange { // call back} , is it available?If not, is it possible to include it?

Hi @psandeepkumar-concentrix we have a privacy concerns/complaint from exposing the fullName during the XO experience but you should be able to get all the info you need as part of getDetails request on an approved order.
Let me know if you have any additional concerns otherwise I will go ahead and close the ticket :)


needs update for string.data changes

After JuliaLang/julia#19449, soon to be merged for Julia 0.6, you will no longer be able to access the raw bytes of a string via string.data; instead, do Vector{UInt8}(string).  For example, this affects:
https://github.com/Keno/TerminalUI.jl/blob/479a844fb50a4c8171b1cf5b6e758d923ce3f3c0/test/regressions.jl#L62

This was done.


AEM request to keep card width consistent between two-columned and three-columned card layouts

The problem
Right now, the width of a card changes between 4 columns and 5 columns depending on the number of cards. When there are only two cards side-by-side the card width spans 5 of the 12 column grid. When there are three cards side-by-side the card width spans 4 of the 12 column grid.
The solution
The AEM team would like to make this width consistent and have the cards consistently span 4 of the 12 columns regardless of the number of cards in its row.
Additional information
Visual example of request (Resources section towards the bottom)

hi Roberta, any chance, we can get an ETA on this ticket?

Related AEM ticket is https://jsw.ibm.com/browse/ADCMS-522

@kirbyevans could you provide the screenshot or link to the component? Currently, Card group doesn't do that and keeps cards at 4 column wide.

Caught up with Kirby over Slack and this is component he's referring to: https://ibmdotcom-react.mybluemix.net/?path=/story/components-contentgroupcards--default
We want to keep above component unchanged because it's actually by design. To achieve 4 col cards, AEM team should use the Card group component mentioned above. Kirby said he will follow up with devs.


Add collecting payments with Stripe article to community resources

Description (required)

Adding a resource to the community section. Sarah asked me to add this in May, but it completely slipped my mind until now :)

Related issues & labels (optional)

Closes #
Suggested label:

First-time contributor to Astro Docs?
Yep. But I already have a contributor tag in Discord for some reason 🙃.
If you are a member of the Astro Discord, please add your username in the description so we can welcome you there!

zellwk

Hello! Thank you for opening your first PR to Astro’s Docs! 🎉
Here’s what will happen next:

Our GitHub bots will run to check your changes.
If they spot any broken links you will see some error messages on this PR.
Don’t hesitate to ask any questions if you’re not sure what these mean!

In a few minutes, you’ll be able to see a preview of your changes on Vercel 🥳

One or more of our maintainers will take a look and may ask you to make changes.
We try to be responsive, but don’t worry if this takes a few days.

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

en
guides/ecommerce.mdx
Source changed, localizations will be marked as outdated.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


Seller Terminal -  Product Management

Seller Terminal has many hard quoted path , Please change it . below Url should work .
http://ec2-54-255-229-68.ap-southeast-1.compute.amazonaws.com/dev/sellerhub/seller/logincheck

above  link is not working

Seller Terminal Preview
Seller Terminal has many hard quoted path , Please change it . below Url should work .
http://ec2-54-255-229-68.ap-southeast-1.compute.amazonaws.com/dev/sellerhub/seller/logincheck

seller terminal path issue resolved
in product model when we add any  product now product attribute categories into 3 group 0, 1, 2 and those data will appear first for 0 and then 1 after on click checkbox group 3 attribute displayed
condition applied some issue occur on third level where i am working

When i select multiple colour it shows single row of group 3 attributes


Migrate to FFmpeg 6.0

These changes are also compatible with FFmpeg 5.1, which is now minimum supported version.
Also set -Wl,-Bsymbolic flag via target_link_options command which is more correct.

Returned -Wl,-Bsymbolic flag. It seems that it's still needed for some codecs.

@equeim Have you checked logcat?
I've migrated to ffmpeg 6 some times ago locally and it seems the comment :

// Output buffer sizes when decoding PCM mu-law streams, which is the maximum FFmpeg outputs.
is no more true and requires a bigger buffer. Doubling OUTPUT_BUFFER_SIZE_16BIT fixed all my issues. (But I do not support nor test ac3)

Nope, it was wrong option names for SwrContext (and there is no error handling av_opt_set calls so there was nothing in logcat). I think I will change it to use swr_alloc_set_opts2 function which accepts these options as parameters, so that there will be a better chance of discovering this at compile time.

@Tolriq I tried to encode some audio using ffmpeg with pcm_mulaw encoder and wav container and there wasn't any issues with ffmpeg 6.0 and this patch (without any changes to OUTPUT_BUFFER_SIZE_16BIT).

The issue is about decoding media I can send you a file in private that shows the issue.

@equeim file sent please do not spread.

I'm not sure that just doubling OUTPUT_BUFFER_SIZE_16BIT is a good idea. From what I can gather the required size of output buffer depends on numbers of samples in AVFrame structure (AVFrame::nb_samples), and that's can't be determined ahead of time for all codecs. I don't see a way to get "maximum" value either. I see 2 correct solutions:

Allocate buffer dynamically for each ffmpegDecode() call, or at least reallocate it's too small
Rewrite ffmpegDecode() to read AVFrame partially

Both seems a non-trivial changes.

@microkatz @rohitjoins Currently there are changes regarding NDK r26 in this PR, I think it would be better to move them to separate PR. The question, do we want to support NDK r26? It dropped support of SDK levels < 21, and AFAIK currently media3 has minSdkVersion 16. I suppose it's possible to add automatic detection of NDK version in ffmpeg build script, but it won't be obvious to users that their minSdkVersion should be bumped to 21 if they are building with r26.

@Tolriq Actually with your file the issue with output buffer size is present in FFmpeg 4.4 too, so I think I will leave it as is here and make separate PR that adds dynamic reallocation.

@equeim,
Thank you for your contribution.

there are changes regarding NDK r26 in this PR

While file in particular? Should we throw an WARN log in the build script if someone is building with NDK r26?

currently media3 has minSdkVersion 16

yes, and so we mention in the README that our build configuration is tested with NDK r21.

While file in particular? Should we throw an WARN log in the build script if someone is building with NDK r26?

I've removed it since this out scope of this PR.

yes, and so we mention in the README that our build configuration is tested with NDK r21.

But shouldn't we at least be able to compile with the latest LTS version of SDK (which is r26b right now)? Currently build scripts for native libraries don't work at all with r26.

Hi @equeim,
This PR is merged in the main branch. Agreed, NDK r21 is outdated and we should possibly update it to a newer version of NDK. Though we are constrained by the NDK support per Gradle version, please create another PR for supporting NDK r26b, I'll discuss this internally and try to upgrade our extension to it.
Thank you once again for your contributions.

Hi @equeim,
Upon discussing internally it seems we are not constrained by the NDK support per Gradle version and would be happy to upgrade our README to the latest LTS for NDK which is r26b.

Hi @equeim,
Upon discussing internally it seems we are not constrained by the NDK support per Gradle version and would be happy to upgrade our README to the latest LTS for NDK which is r26b.

To do this it will be necessary to replace armv7a-linux-androideabi16- and i686-linux-android16- prefixes in build scripts with ones that set API 21, since NDK r26 does not support pre-21 APIs. This in turn would mean that all extensions that use native libraries will need to raise their minSdk versions to 21. Are you fine with this change? AFAIK minSdk for media3 is still 16.

No, we can't do this. May be we can introduce some optional flag which people can enable to use the newer version of NDK.

No, we can't do this. May be we can introduce some optional flag which people can enable to use the newer version of NDK.

I'm thinking of adding another argument to build_ffmpeg.sh (and friends) scripts to pass ABI level (minSdk) from outside instead of hardcoding 16/21. This way those who have minSdk >= 21 will be able to use latest NDK without modifying build scripts.


revise the "overall documentation strategy"

The iter module and join/scope functions now host the primary docs. The README and crate-level docs mostly forward to those. Some of the detailed material from the README is moved to a FAQ and lightly edited.

cc @KodrAus

@cuviper ok

bors delegate=KodrAus

Let's see if this works!
bors r+


Swiper 9.0.2 + React 18.2.0 in StrictMode - Cannot read properties of undefined (reading 'slideClass')

Check that this is really a bug

[X] I confirm

Reproduction link
https://codesandbox.io/s/swiper-9-react-strict-issue-c83d98
Bug description
Swiper 9.0.2 + React 18.2.0 in StrictMode
If you set loading='lazy' to image element inside SwiperSlide and set React.StrictMode you will see error message
Uncaught TypeError: Cannot read properties of undefined (reading 'slideClass')
    at slideSelector (process-lazy-preloader.js:2:1)
    at processLazyPreloader (process-lazy-preloader.js:3:1)
    at HTMLImageElement.<anonymous> (core.js:484:1)

https://github.com/nolimits4web/swiper/blob/9fe9a9605e41af36e4c34b8877accf0530830975/src/shared/process-lazy-preloader.js#L2
I know that lazy loading reimplemented in Swiper 9.
Is it possible to avoid execution of processLazyPreloader if developer don't want to use preloader spinners?
Expected Behavior
Native lazy loading working without issues in console in React.StrictMode
Actual Behavior
Errors in console:
Uncaught TypeError: Cannot read properties of undefined (reading 'slideClass')
    at slideSelector (process-lazy-preloader.js:2:1)
    at processLazyPreloader (process-lazy-preloader.js:3:1)
    at HTMLImageElement.<anonymous> (core.js:484:1)

Swiper version
9.0.2
Platform/Target and Browser Versions
Win/Mac, Chrome
Validations

[X] Follow our Code of Conduct
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.
[X] Make sure this is a Swiper issue and not a framework-specific issue

Would you like to open a PR for this bug?

[ ] I'm willing to open a PR

fixed in https://github.com/nolimits4web/swiper/commit/4ea0102cfda0be6de5757381b6c0e2aa2649a4e8


Unhelpful error reporting when connected to a non-gRPC server

When the Objective-C (and presumably also any C-based) client connects to a web server that doesn't support gRPC, a message is logged like this one:
E0226 11:35:01.485432000<PHONE_NUMBER>17120 parsing.c:472] Unknown frame type 50
Objective-C clients also receive an NSError object that, when logged, prints something like this:
2016-02-26 11:36:15.447 stickynotes[89409:5881572] ERROR: Error Domain=io.grpc Code=14 "" UserInfo={io.grpc.TrailersKey={
},
To a client application developer, this is fairly useless information. Stepping through the grpc code, I see that the actual server response is the following:
"P/1.1 400 Bad Request\r\nConnection: close\r\nDate: Fri, 26 Feb 2016 19:31:58 GMT\r\nContent-Length: 0\r\nContent-Type: text/plain; charset=utf-8\r\n\r\n\x03"
At a minimum, it would be good to log this server response (perhaps truncated for safety), and it would be even better to include a more descriptive message in the NSError object returned by the Objective C API.

Just hit this connecting to a grpc server from a nodejs client as well - would be nice to see a universal fix (rather than just for ObjC)

This is a duplicate of #49 (our oldest open issue!)
But yes, C core should stick some useful data in the error status that it
bubbles up. We've (clearly) been trying to get to it for a while... there
are some prerequisites needed to make this robust (#4672 springs to mind).
On Fri, Mar 11, 2016 at 3:51 PM Russell Matney<EMAIL_ADDRESS>wrote:

Just hit this connecting to a grpc server from a nodejs client as well -
would be nice to see a universal fix (rather than just for ObjC)
—
Reply to this email directly or view it on GitHub
https://github.com/grpc/grpc/issues/5443#issuecomment-195606104.


Error in inmuneConvoy.sqf

Version: <IP_ADDRESS>
Mods: CBA, RHS, ACE Antistasi Edition
Environment: MP dedi.
.rpt attatched? YES (screenshot)
have you edited the missionfile?: No
Issue:
I saw the following error in the Server log today. This happened a few minutes after everyone leaving the Server yesterday night. Not sure if it causes any bigger issue.

However, searching for the issue I found that the dirTo function (in line 62) should not be used anymore in A3:
https://community.bistudio.com/wiki/BIS_fnc_dirTo             (Description)

Yes, that function should be replaced, but no worries abut an error there, it's harmless


Store results in table instead of column

BACKEND PULL REQUEST
Related Issue
Fixes #3456
Changes Proposed

Starts writing out to the new Result table!

Additional Information

We're still writing to the old results column for now as there are some related pieces of code that require it (dashboard and filtering results, for example). This will be addressed with #3663 and #3664

Testing
Specific things I'd like to verify in dev or test before merging:

Queue items that are pending when this goes in can be edited and submitted
Results are viewable to SR users
Results are visible to patients
Results can be filtered successfully with all filter options
Dashboard works as intended
Test corrections and deletions work as intended
Any other areas that are touched by results

Checklist for Primary Reviewer

[x] Any large-scale changes have been deployed to test, dev, or pentest and smoke tested
[x] Any content updates (user-facing error messages, etc) have been approved by content team
[x] Any changes that might generate questions in the support inbox have been flagged to the support team
[x] GraphQL schema changes are backward compatible with older version of the front-end
[x] Changes comply with the SimpleReport Style Guide
[x] Changes with security implications have been approved by a security engineer (changes to  authentication, encryption, handling of PII, etc.)
[x] Any dependencies introduced have been vetted and discussed

Smoke tested on dev and everything seems to be working correctly (adding results, correcting results, marking as duplicate, pxp, and the dashboard all had the same functionality as before). There's still one failing test though.

@zdeveloper @nathancrtr All tests are now passing, and everything looks good when smoke testing in dev. If you could give this a review it would be greatly appreciated!

LGTM! I might be wrong here, but the multiplex feature feels to me like its getting built in a waterfall method and I would've preferred seeing more thin slices.

Would love to chat more about this offline!


Fix for #348 - IGListStackedSectionController's children need to know numberOrItems before didUpdate is called

Changes in this pull request
The reloadData method in IGListStackedSectionController was being called too soon.  It was moved from the constructor to the didUpdateToObject method.  Tests were updated to reflect the change.
Pull request checklist

[x] All tests pass. Demo project builds and runs.
[x] I added tests, an experiment, or detailed why my change isn't tested.
[x] I added an entry to the CHANGELOG.md for any breaking changes, enhancements, or bug fixes.
[x] I have reviewed the contributing guide

Nice. Thanks @jeffbailey ! 💯

Coverage decreased (-0.0007%) to 97.978% when pulling 1d4de97d7eb6653b25f0354be93ce292f5861115 on jeffbailey:master into c19379eeda6f254c0ba365dbe1110f4ffe37acf0 on Instagram:master.

Coverage increased (+0.09%) to 98.064% when pulling 43d43a46550d3247c60e6ce8c8052bc62f4d7307 on jeffbailey:master into fb4e8238704906c4ec826bf2d677928249586a92 on Instagram:master.

Coverage increased (+0.09%) to 98.064% when pulling 43d43a46550d3247c60e6ce8c8052bc62f4d7307 on jeffbailey:master into fb4e8238704906c4ec826bf2d677928249586a92 on Instagram:master.

Coverage increased (+0.09%) to 98.064% when pulling 43d43a46550d3247c60e6ce8c8052bc62f4d7307 on jeffbailey:master into fb4e8238704906c4ec826bf2d677928249586a92 on Instagram:master.

Coverage decreased (-0.0007%) to 98.064% when pulling 2ec527c9061479e51dbb71b5fb3b39eaeb6f631f on jeffbailey:master into dd3b9eddccd637ff13f5a55ce761c367b441ba44 on Instagram:master.

Coverage decreased (-0.0007%) to 98.064% when pulling 2ec527c9061479e51dbb71b5fb3b39eaeb6f631f on jeffbailey:master into dd3b9eddccd637ff13f5a55ce761c367b441ba44 on Instagram:master.

Friendly ping ✨ any updates here?
@rnystrom if this is good, we can follow up with a changelog entry separately.

The internal header is a great option.  I'll get this done today.  Thanks @rnystrom

Coverage decreased (-0.0007%) to 98.064% when pulling fbfebf9e219104a18031831d38a7025a1028cc4a on jeffbailey:master into dd3b9eddccd637ff13f5a55ce761c367b441ba44 on Instagram:master.

Coverage decreased (-0.0007%) to 98.064% when pulling fbfebf9e219104a18031831d38a7025a1028cc4a on jeffbailey:master into dd3b9eddccd637ff13f5a55ce761c367b441ba44 on Instagram:master.

Coverage decreased (-0.0007%) to 98.064% when pulling fbfebf9e219104a18031831d38a7025a1028cc4a on jeffbailey:master into dd3b9eddccd637ff13f5a55ce761c367b441ba44 on Instagram:master.

Changes Unknown when pulling e0e703143dde640bd1ace8156a169c90f83d56c2 on jeffbailey:master into ** on Instagram:master**.

Changes Unknown when pulling e0e703143dde640bd1ace8156a169c90f83d56c2 on jeffbailey:master into ** on Instagram:master**.

Changes Unknown when pulling e0e703143dde640bd1ace8156a169c90f83d56c2 on jeffbailey:master into ** on Instagram:master**.

Changes Unknown when pulling e0e703143dde640bd1ace8156a169c90f83d56c2 on jeffbailey:master into ** on Instagram:master**.

@jeffbailey looks great! Very last thing is a quick note in the CHANGELOG.md file about this fix and we're g2g. It should go under the 2.2.0 list.

Changes Unknown when pulling 8c5bd9e5a68f7096a94f6e03f6b25465fbbe8184 on jeffbailey:master into ** on Instagram:master**.


Enhance test runner scripts

Two commits:
shlib_wrap.sh: add module and engine paths to environment if unset
The testsuite invokes many openssl commands using the shlib_wrap.sh script.
It used to be possible to copy those commands for testing them standalone
or run them in the debugger (see also description of issue #10628).
With providers this did not work anymore, because the script did not
set the OPENSSL_MODULES and OPENSSL_ENGINES environment variables,
they were already set by the _tests make target.
To make shlib_wrap.sh self-contained again, we set the OPENSSL_MODULES
and OPENSSL_ENGINES environment variables if they are unset. In fact,
that's just what opensslwrap.sh already does.
opensslwrap.sh: make implementation more consistent with shlib_wrap.sh

Closing in favour of #11110.


[ext] Nam dashboard - removed red 'unsprayed' bar from stacked bar chart on the dashboard

@onlyjsmith commented on Thu Jul 19 2018
External feedback source: https://github.com/disarm-platform/user-requests-and-feedback/issues/89
Since we are not subtracting First visit unsprayed - Mop-up unsprayed this number will be inflated - we can just keep the cumulative green bar (# sprayed) and remove the red bar

Has been deployed to stage.

Done


Support key and salt rotating

If I want to change the key and/or salt in production environment (maybe 3 imgproxy servers behind a load balancer), it is very crucial not to stop the service. I found it seems difficult to achieve this requirement. If I change key/salt in my application first, I cannot get imgproxy to process the image for me any more. Likewise if I change key/salt in imgproxy first.
I wonder to know if it is possible to store an array in IMGPROXY_KEY and IMGPROXY_SALT. Here is the example:
IMGPROXY_KEY: ["key1", "key2", ...]
IMGPROXY_SALT: ["salt1", "salt2", ...]
With this kind of settings, imgproxy can check the signature with key1/salt1 first, if it failed, continue to check with key2/salt2. And now it is safe to update each server's key/salt one by one without breaking any service.

Hi,
Is there any significant reason to change key/salt pair so often?
To be honest, I don't like the idea, because it's like setting multiple passwords to DB.

Normally I won't change key/slat often, currently only under two circumstances I need to change the key/salt. The first one is to obey my company's audit policy, I have to change key/salt once every year. The second one is when I observed a suspicious activity.
The solution I proposed was referenced from https://cloud.google.com/kms/docs/rotating-keys, but simplified. I think I can live with any kind of solutions, no matter is it key rotating based. The only requirement is to keep my imgproxy service with zero down time when changing key and/or salt.

Maybe a fail-safe approach will work in this case?  Something like, if the signature is not valid, redirect to the original image URL? I think it's better than showing the error message?

@selul How do I know if the signature is invalid? I just generate a key and give the same key to both imgproxy and my application. If I generate a new key and update the key in my application first, my application will still generate signature fine without error. I won't know whether the signature is valid.
Or do you mean I have to check every image returned by imgproxy to see if it is empty (or error, or not found)? And if it is empty, redirect to original image. This maybe doable, but I think this is cumbersome. If I have a lot of images on my web page, I wonder to know if this will slow down the page loading. Furthermore, this solution will skip the CDN cache, if the original images are pulled from my own website, this will definitely slow down the website, or worse, paralyzed it.

@itsliang I might misunderstand your problem, if so please ignore my reply.
I thought you are worried that the URLs of the images generated with the old keys/salt won't work after you deploy the new one's credentials.
In that case, I was thinking that, if imgproxy detects an invalid token error, don't display the error message and rather redirect to the original image. Yes, this will slow down the website where you use the images, but this is temporary until you will replace the images URLs with the new signature.

@selul Thanks for your suggestion. But I think this approach is a little bit unsafe, because the using of signature is to prevent my imgproxy being used by other people. If imgproxy redirects requests to original image when signature check failed, my imgproxy might be used by malicious software.

Well, though I still don't really like the idea, it's pretty easy to implement. v2.1.0.rc2 (available on Docker Hub) allows specifying multiple keys and salts divided by comma (,).

Just released imgproxy v2.1.0 with this and other awesome features. Check out the changelog for details: https://github.com/DarthSim/imgproxy/blob/master/CHANGELOG.md#v210


test<EMAIL_ADDRESS>

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 80.676%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3301

Relevant Lines:
4064

💛  - Coveralls


Warnings on npm install and build

on build:
Dynamic import can only be transformed when transforming ES modules to AMD, CommonJS or SystemJS.
on install:
npm WARN ERESOLVE overriding peer dependency
npm WARN While resolving<EMAIL_ADDRESS>npm WARN Found<EMAIL_ADDRESS>npm WARN node_modules/rollup
npm WARN   dev<EMAIL_ADDRESS>from the root project
npm WARN   2 more (@rollup/plugin-babel, @rollup/plugin-commonjs)
npm WARN
npm WARN Could not resolve dependency:
npm WARN peerOptional<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm WARN node_modules/@rollup/pluginutils
npm WARN<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm WARN   node_modules/@rollup/plugin-babel
npm WARN   1 more (@rollup/plugin-commonjs)
npm WARN
npm WARN Conflicting peer dependency<EMAIL_ADDRESS>npm WARN node_modules/rollup
npm WARN   peerOptional<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm WARN   node_modules/@rollup/pluginutils
npm WARN<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm WARN     node_modules/@rollup/plugin-babel
npm WARN     1 more (@rollup/plugin-commonjs)

Please review #55

When I run npm install I initially get an error about a damaged lock file, though when I rerun it is fine:
npm-package-template [(HEAD detached at origin/pr/55)]$ npm i
npm WARN reify invalid or damaged lockfile detected
npm WARN reify please re-try this operation once it completes
npm WARN reify so that the damage can be corrected, or perform
npm WARN reify a fresh install with no lockfile if the problem persists.
npm WARN reify invalid or damaged lockfile detected
npm WARN reify please re-try this operation once it completes
npm WARN reify so that the damage can be corrected, or perform
npm WARN reify a fresh install with no lockfile if the problem persists.
npm WARN reify invalid or damaged lockfile detected
npm WARN reify please re-try this operation once it completes
npm WARN reify so that the damage can be corrected, or perform
npm WARN reify a fresh install with no lockfile if the problem persists.

added 1259 packages, and audited 1260 packages in 9s

124 packages are looking for funding
  run `npm fund` for details

found 0 vulnerabilities

@dhowe did you checkout the branch in the PR #55 ?

I checked out the PR, as shown in the prompt above:
npm-package-template [(HEAD detached at origin/pr/55)]$ npm i

Hm this is strange. I have recreated the lockfile with npm 10.2.0, removed everything on have no warnings on a clean install.

@dhowe I recreated the package-lock, can you please try again?

all good now
also, what is that warning about the dynamic imports?

also, what is that warning about the dynamic imports?

I think the iife build for browsers does not support dynamic imports. However, this is something to investigate on. I will create a new issue for this.

seems to show up even when I remove the iife build from the rollup config...


PyObject.clone doesn't actually clone?

I want to copy a python object in my Rust code so that it cannot be mutated by external python code. I tried looking into the PyObject.clone() method. But this only clones the reference to the object.
A minimum example is: EDIT: A more complete (and working) example below: https://github.com/PyO3/pyo3/issues/1394#issuecomment-763130042
#[pyclass]
struct PyObjSaver {
    value: PyObject
}

#[pymethods]
impl PyObjSaver {
    pub fn get(&self) -> PyObject {
        self.value.clone()
    }
    pub fn set(&mut self, value: PyObject) {
        self.value = value.clone();
    }
}

Unfortunately an object (like a python dict) that is passed into the set method can later be mutated. A value retireved from the get method can be mutated as well.
I can get around this using the python copy method like so:
pub fn set(&mut self, value: PyObject) -> PyResult<()> {
      Python::with_gil(|py| {
          let copy_module = PyModule::import(py, "copy")?;
          self.value = copy_module.call1("copy", (value,))?.extract()?
      })
  }

This doesn't seem ideal. How might I copy a PyObject without importing the python copy module?

Why is that not acceptable? After all this functionality isn't a built-in in Python itself.
BTW, clone in Rust is the equivalent of Python's copy.deepcopy, not copy.copy.

Why is that not acceptable? After all this functionality isn't a built-in in Python itself.
BTW, clone in Rust is the equivalent of Python's copy.deepcopy, not copy.copy.

@birkenfeld I think you may have hit on the issue at hand:

BTW, clone in Rust is the equivalent of Python's copy.deepcopy, not copy.copy.
It does not appear to be so. A value n passed to PyObjSaver.set(n) can be changed after the call and both references will be changed. In other words:

t = PyObjSaver()
n = {"a": 1}
t.set(n)
n["a"] = 5
t.get()["a"] == 5

The docs say PyObject: "Can be cloned using Python reference counts with .clone()." Link I'm not sure if that means marked as new reference or if it really is suppose to be a deepcopy.
Thank for the hints with &PyAny and the method block.

@birkenfeld I think you may have hit on the issue at hand:

BTW, clone in Rust is the equivalent of Python's copy.deepcopy, not copy.copy.
It does not appear to be so. A value n passed to PyObjSaver.set(n) can be changed after the call and both references will be changed. In other words:

t = PyObjSaver()
n = {"a": 1}
t.set(n)
n["a"] = 5
t.get()["a"] == 5

The docs say PyObject: "Can be cloned using Python reference counts with .clone()." Link I'm not sure if that means marked as new reference or if it really is suppose to be a deepcopy.
Thank for the hints with &PyAny and the method block.

A more complete example:
Rust Module
use pyo3::prelude::*;

#[cfg(test)]
mod tests {
    #[test]
    fn it_works() {
        assert_eq!(2 + 2, 4);
    }
}

#[allow(unused_variables)]
#[pymodule]
pub fn muta(py: Python, m: &PyModule) -> PyResult<()> {
    m.add_class::<PyObjSaver>()?;
    Ok(())
}

#[pyclass]
struct PyObjSaver {
    value: Option<PyObject>
}

#[pymethods]
impl PyObjSaver {
    #[new]
    pub fn new() -> PyObjSaver {
        PyObjSaver {
            value: None
        }
    }

    pub fn get(&self) -> Option<PyObject> {
        match &self.value {
            Some(t) => Some(t.clone()),
            None => None
        }
    }

    pub fn set(&mut self, value: PyObject) {
        self.value = Some(value.clone());
    }
}

Build
maturin build --interpreter python --release
pip uninstall muta -y
pip install target\wheels\muta-0.1.0-cp37-none-win_amd64.whl

Python console testing
Python 3.7.6 (default, Jan  8 2020, 20:23:39) [MSC v.1916 64 bit (AMD64)] :: Anaconda, Inc. on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> import muta
>>> z = muta.PyObjSaver()
>>> z
<PyObjSaver object at 0x0000027E6113D2F0>
>>> a = {"one": 10}
>>> z.set(a)
>>> z.get()
{'one': 10}
>>> a["one"] = 30
>>> z.get()
{'one': 30}
>>>

I expected the value in from z.get() to be {'one': 10}. Unfortunately I was able to modify a to change the internals of z the PyObjSaver object.

A more complete example:
Rust Module
use pyo3::prelude::*;

#[cfg(test)]
mod tests {
    #[test]
    fn it_works() {
        assert_eq!(2 + 2, 4);
    }
}

#[allow(unused_variables)]
#[pymodule]
pub fn muta(py: Python, m: &PyModule) -> PyResult<()> {
    m.add_class::<PyObjSaver>()?;
    Ok(())
}

#[pyclass]
struct PyObjSaver {
    value: Option<PyObject>
}

#[pymethods]
impl PyObjSaver {
    #[new]
    pub fn new() -> PyObjSaver {
        PyObjSaver {
            value: None
        }
    }

    pub fn get(&self) -> Option<PyObject> {
        match &self.value {
            Some(t) => Some(t.clone()),
            None => None
        }
    }

    pub fn set(&mut self, value: PyObject) {
        self.value = Some(value.clone());
    }
}

Build
maturin build --interpreter python --release
pip uninstall muta -y
pip install target\wheels\muta-0.1.0-cp37-none-win_amd64.whl

Python console testing
Python 3.7.6 (default, Jan  8 2020, 20:23:39) [MSC v.1916 64 bit (AMD64)] :: Anaconda, Inc. on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> import muta
>>> z = muta.PyObjSaver()
>>> z
<PyObjSaver object at 0x0000027E6113D2F0>
>>> a = {"one": 10}
>>> z.set(a)
>>> z.get()
{'one': 10}
>>> a["one"] = 30
>>> z.get()
{'one': 30}
>>>

I expected the value in from z.get() to be {'one': 10}. Unfortunately I was able to modify a to change the internals of z the PyObjSaver object.

Sorry, this was misleading.  I meant that the definition of clone() in Rust on "plain" data types (such as Vec, HashMap) corresponds to deepcopy in Python.
On special types like Arc, or here PyObject it takes a different meaning.  That it only creates new Python references in PyO3 is intended.

Sorry, this was misleading.  I meant that the definition of clone() in Rust on "plain" data types (such as Vec, HashMap) corresponds to deepcopy in Python.
On special types like Arc, or here PyObject it takes a different meaning.  That it only creates new Python references in PyO3 is intended.

On special types like Arc, or here PyObject it takes a different meaning. That it only creates new Python references in PyO3 is intended.

This. As Python is reference-counted, it is a natural translation to Rust for all of PyO3's Python-native objects to behave like Arc.
Also note that copy.deepcopy is implemented purely in Python; there is no Python C-API to "clone" an entire object in the way you ask. IIRC copy.deepcopy works by pickling and unpickling the entire object.
It's not intended for PyO3's behaviour to change in this regard. Perhaps there's some documentation we can improve though.

On special types like Arc, or here PyObject it takes a different meaning. That it only creates new Python references in PyO3 is intended.

This. As Python is reference-counted, it is a natural translation to Rust for all of PyO3's Python-native objects to behave like Arc.
Also note that copy.deepcopy is implemented purely in Python; there is no Python C-API to "clone" an entire object in the way you ask. IIRC copy.deepcopy works by pickling and unpickling the entire object.
It's not intended for PyO3's behaviour to change in this regard. Perhaps there's some documentation we can improve though.

Understood. I don't particularly like the the dual meaning of clone on special vs non-special types in rust- but PyObject is following convention here. Feel free to close this issue, or keep it open if you want to come back around to do some documentation.

I wouldn't particularly class it as "special" vs "non-special" types. I think of it as connected to ownership.
Types like &T, Arc<T>, Py<T> and PyObject all represent shared ownership of the underlying data. To call .clone() on any of these types gives you another handle to that underlying shared data.
Types like Vec<T> or struct Foo represent exclusive ownership; to call .clone() on any of these implies that you'll create a full new copy as the result of the clone.


Change for loop to while loop.

Change for loop to a while loop so that we don't skip items and overshoot the end of the container when erasing.

I'm not sure that I see a difference in what is executed here. Does this fix a crash?

It looks like with the change it's possible to dereference an invalid
iterator.
On Tue, Dec 3, 2019, 3:36 PM Matthew<EMAIL_ADDRESS>wrote:

I'm not sure that I see a difference in what is executed here. Does this
fix a crash?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/swri-robotics/mapviz/pull/663?email_source=notifications&email_token=ABBZWHDACM3WA24IMCNNOQLQW3GNHA5CNFSM4JU64W7KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEF24VWY#issuecomment-561367771,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABBZWHHRYFGUL4BNKAMQ4DLQW3GNHANCNFSM4JU64W7A
.

Where does that occur?

Sorry about that, I misread the code on my phone and thought the continue
statements had been removed. I'm still not sure what problem this change is
addressing.
On Tue, Dec 3, 2019 at 4:38 PM Matthew Grogan<EMAIL_ADDRESS>wrote:

Where does that occur?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/swri-robotics/mapviz/pull/663?email_source=notifications&email_token=ABBZWHG5JR5KE5TVEPHCLQDQW3NVJA5CNFSM4JU64W7KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEF3B6AQ#issuecomment-561389314,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABBZWHBSARTVSZJ6L6I2CR3QW3NVJANCNFSM4JU64W7A
.

Ah, no worries. The erase function returns the next iterator and at the end of the for loop, that iterator will be incremented again. So it potentially skips items and increments the iterator past end.


Consider restoring support for selecting embargo group during submission

In https://github.com/DSpace/dspace-angular/issues/869 , we decided to remove the Group select box from the Embargo step in the Submission process, as the initial implementation was problematic.

The reasons for this were summarized by @cwilper in his analysis of the issues: https://github.com/DSpace/dspace-angular/issues/869#issuecomment-713594267
As noted in my follow-up response, while this feature did exist in DSpace 6, I felt it was slightly "hacky" & needed rethinking to avoid potential security issues.

Workaround: In the above discussion, I noted it was easy enough to simply configure separate group options in your [dspace]/config/spring/api/access-conditions.xml and name them logically (e.g. "Embargo then Public", "Embargo then Campus-Only", etc).  This would provide the same ability for submitters to select from different Embargo options, without them having to determine which (internal) group is most appropriate to select.
This ticket is opened as a reminder to revisit this discussion/decision. If we find this feature is missed by users of 7.0, then obviously we'll need to determine a better solution for allowing Submitters to select from a list of embargo groups.
Related work

REST PR which removed this feature: https://github.com/DSpace/DSpace/pull/3136

The issue can be solved as follow
instead than expose the uuid of the parent group containing the selectable (sub)groups having we would have modified the access condition configuration exposed via REST to directly list the group and the uuid. This should mean a json like that
{
  "id": "default",
  "required": true,
  "max-size": 536870912,
  "accessConditionOptions": [
		{
 			"name": "openaccess"
		},
		{
 			"name": "administrator"
		},
		{
 			"name": "embargo",
 			"options": [{"label": "Group 1", "value": uuid1}, {"label": "Group 2", "value": uuid2} .... ]
 			"hasStartDate": true,
 			"maxStartDate": "2018-06-24T00:40:54.970+0000"
		},
		{
 			"name": "lease",
 			"options": [{"label": "Group 1", "value": uuid1}, {"label": "Group 2", "value": uuid2} .... ]
 			"hasEndDate": true,
 			"maxEndDate": "2017-12-24T00:40:54.970+0000"
		}
  ]
}

the patch operation could have been extended to protect against illegal attempt to select unexpected group (that should lead to a 422 as the parameter is semantically unvalid/unexpected)
Our estimation for these change is 8 hours for the REST/Contract side, 4 hours on the Angular side


Fixed InputNumber Icon alignment

What kind of change does this PR introduce?
Bug fix
What is the current behavior?
Currently InputNumber Icon is not aligned properly.
#276
What is the new behavior?
Fixed InputNumber Icon alignment

:tada: This PR is included in version 0.37.0-beta.5 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 0.37.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


ENHANCEMENT : Tabung > Cetak - Delete (ReactJS)

Expected result:

Icon delete di Tabung > Cetak
Guna endpoint curl --location --request DELETE 'localhost:8082/kutipan/9'

Expected delivery time:
3 days
Comment: "Assign this to me", if you wish to pick this up

PR: https://github.com/Dev4w4n/e-masjid.my/pull/220


Default chunk sizes

Does this make sense?
String chunkSizeArg = is2D ?  "1024,1024" : "96,96,96";
@constantinpape @martinschorb

Sounds reasonable.


Add httpguides.com

This PR is:

[x] Adding a new domain
[ ] Updating existing domain size
[ ] Changing domain name
[ ] Removing existing domain from list
[ ] Website code changes (512kb.club site)
[ ] Other not listed

[x] I used the uncompressed size of the site
[x] I have included a link to the GTMetrix report
[x] The domain is in the correct alphabetical order
[x] This site is not an ultra lightweight site
[x] The following information is filled identical to the data file

I confirm that I have read the FAQ section, particularly the two red items around minimal pages and inappropriate content and I attest that this site is neither of these things.

[x] Check to confirm

- domain:
  url:
  size:
  last_checked:

GTMetrix Report: https://gtmetrix.com/reports/httpguides.com/8l6vybLx/

Thanks!


Add builtin string functions

upper(str)
lower(str)
left(str)
right(str)
lpad(str)
substring(str)
reverse(str)
...

Can we make this optional? Existing applications might not want to expose these functions to their end users.
For example, we are using expr to provide expressions in a data streaming language. We would like full control over the set of functions exposed to our users.
Would be nice to have some kind of config flag "disableBuiltIns" to prevent these functions being exposed when they are not wanted.

Making this optional it would interfere with the docgen.CreateDoc. Then the docgen package also need a method of something like:
AddStringBuiltins(*Context) *Context
I think that would be a bit weird?

But why optional? What is use case for it?

Yeah I would like to see it in stdlib also, i do not have a use case for hiding it. Would you like to see it implemented here: https://github.com/antonmedv/expr/blob/master/builtin/builtin.go ?

But why optional? What is use case for it?

we use it calculate sql function

But why optional? Let’s make them available all the time.


[Bug] Fix override/defaulting of "prediction intervals" adders

Reference Issues/PRs
Fixes #3009
What does this implement/fix? Explain your changes.
This adds the base class _predict_quantiles method into conformal.py to ensure that the _predict_quantiles method for the forecaster that is wrapped with conformal.py is consistent with _predict_interval.
Does your contribution introduce a new dependency? If yes, which one?
No
What should a reviewer concentrate their feedback on?
I have just copied and pasted the base class _predict_quantiles method and removed the implements_interval check . Should this be amended at all?
PR checklist
For all contributions

[x] I've added myself to the list of contributors.
[x] Optionally, I've updated sktime's CODEOWNERS to receive notifications about future changes to these files.
[x] I've added unit tests and made sure they pass locally.
[x] The PR title starts with either [ENH], [MNT], [DOC], or [BUG] indicating whether the PR topic is related to enhancement, maintenance, documentation, or bug.

For new estimators

[ ] I've added the estimator to the online documentation.
[ ] I've updated the existing example notebooks or provided a new one to showcase how my estimator works.

Makes sense!
Question, should this perhaps be an explicit call to BaseForecaster._predict_quantiles instead?
Because if we fix a bug in that, it would not propagate to here.
This is very non-hypothetical, see this bug in the default _predict_quantiles! #3106

Good shout - I'll call directly to BaseForecaster

CI failures unrelated, see https://github.com/alan-turing-institute/sktime/issues/3103

Failures unrelated, caused by #3103


Testing for the Docker driver

I've been going through the Docker driver code and I realized that it currently has no testing, and we should probably add some.
@silvin-lubecki - is there a fake Docker client / CLI / standard way of testing the creation of containers, adding volumes and other typical operations we might execute with the Docker driver?

Hello @radu-matei sorry I missed this one 😅
In the docker/cli there's a FakeClibut put on purpose in theinternal` package because we don't think it's at the right level for tests.
We instead prefer faking the docker client, like here. Unfortunately the client is a giant interface 😞, but using this pattern I guess you only have to implement the methods you are using? Or maybe we should abstract from the docker client/cli and have our own interface in the cnab-go?

We do have the integration tests now, which directly create a container.
That being said, executing that test does require a Docker daemon, and it would be nice if we had a fake client - I'll take a look at the way it's done in the CLI, thanks!


The demo video is missing

http://pivotfinland.com/pytest-sugar/img/video.gif now redirects to https://github.com/Teemu/pytest-sugar/img/video.gif which is a 404 page.
Suggestion: if you still have that file, add it to a git repository (this one if it's not too huge, or maybe some sibling repository pytest-sugar-demo) and link to it using a raw link like https://raw.githubusercontent.com/Teemu/pytest-sugar/master/video.gif.

fyi it's still on wayback
https://web.archive.org/web/20200429105636/https://pivotfinland.com/pytest-sugar/img/video.gif

and it's only: 4.83 KB (4,949 bytes)
I'd recommend re-recording it with asciinema/svg-term-cli: https://github.com/marionebl/svg-term-cli

Oops, this happened when I redirected the website to this repo. Fixed now.


Fixed wasm-bindgen typo in README

@rmliddle Could you kindly check these names as well?

This is good to merge too (y)


do/end matching bug when end is a field of a map/struct

The problem here is that the end in current.range.end matches the fn, which causes everything else to be unbalanced. We'll need to add end used in this case to the list of allowed unmatched brackets or whatever it is.

We may be able to just add something like this and then add the name to unbalancedBracketScopes:
    {
      "match": "\\.end",
      "name": "SOMETHING"
    },

What timing. I just hit this at work today!


HBASE-27354 EOF thrown by WALEntryStream causes replication blocking

In WALEntryStream#readNextEntryAndRecordReaderPosition, it is possible that we read uncommitted data.  If we read beyond the committed file length, then reopen inputStream and seek back.
In our use, we found that the position where seek back may be exactly the length of the file being written, which may cause EOF. The thrown EOF is finally caught by ReplicationSourceWALReader.run, but totalBufferUsed is not cleanup up.
After a long run, all peers will go slow and eventually block completely.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 48s
Docker mode activated.

After a long run, all peers will go slow and eventually block completely.

@ddupg curious if you have any workaround for this on live cluster? do you restart regionservers?

Any updates here @ddupg ? If not, I think we could open a new issue to land the improvement here first, no harm.
Thanks.

@ddupg curious if you have any workaround for this on live cluster? do you restart regionservers?

@virajjasani  Previously we had to restart regionservers, after this patch, the problem still exist, but the frequency has slowed to an acceptable level.


xds_k8s tests: Fix xlang install script sourcing.

This change sources the test driver install script correctly for the xlang tests.
This fixes a mistake in #27462 where this was missed.
Test run from branch: https://fusion2.corp.google.com/invocations/93f8bf03-bf07-42b6-928f-398616a4c63a/targets
@sergiitk

This change sources the test driver install script correctly for the xlang tests.
This fixes a mistake in #27462 where this was missed.


x509: ECDSA verification failure

error: [client-utils.js]: sendPeersProposal - Promise is rejected: Error: Failed to deserialize creator identity, err The supplied identity is not valid, Verify() returned x509: certificate signed by unknown authority (possibly because of "x509: ECDSA verification failure" while trying to verify candidate authority certificate "ca.org1.example.com")
at /home/ercw/go/src/github.com/hyperledger/fabric/examples/fabric-docker-compose-svt/fabric-explorer/node_modules/grpc/src/node/src/client.js:554:15
error: [Channel.js]: Failed Query block. Error: Error: Failed to deserialize creator identity, err The supplied identity is not valid, Verify() returned x509: certificate signed by unknown authority (possibly because of "x509: ECDSA verification failure" while trying to verify candidate authority certificate "ca.org1.example.com")
at /home/ercw/go/src/github.com/hyperledger/fabric/examples/fabric-docker-compose-svt/fabric-explorer/node_modules/grpc/src/node/src/client.js:554:15
[2017-09-08 09:05:29.591] [ERROR] Query - Error: Failed to deserialize creator identity, err The supplied identity is not valid, Verify() returned x509: certificate signed by unknown authority (possibly because of "x509: ECDSA verification failure" while trying to verify candidate authority certificate "ca.org1.example.com")
at /home/ercw/go/src/github.com/hyperledger/fabric/examples/fabric-docker-compose-svt/fabric-explorer/node_modules/grpc/src/node/src/client.js:554:15
[2017-09-08 09:05:29.591] [ERROR] blockscanner - TypeError: Cannot read property 'data' of undefined
at EventEmitter. (/home/ercw/go/src/github.com/hyperledger/fabric/examples/fabric-docker-compose-svt/fabric-explorer/listener/blocklistener.js:14:30)
at emitOne (events.js:96:13)
at EventEmitter.emit (events.js:188:7)
at saveBlockRange (/home/ercw/go/src/github.com/hyperledger/fabric/examples/fabric-docker-compose-svt/fabric-explorer/service/blockscanner.js:35:23)
at next (native)
at onFulfilled (/home/ercw/go/src/github.com/hyperledger/fabric/examples/fabric-docker-compose-svt/fabric-explorer/node_modules/co/index.js:65:19)

Please help with this error. I am running the default settings with TLS disabled.
{
"host":"localhost",
"port":"8080",
"channelsList": ["mychannel"],
"GOPATH":"../artifacts",
"keyValueStore":"/tmp/fabric-client-kvs",
"eventWaitTime":"30000",
"enableTls":false,
"users":[

the same error come out and how to fix it?

You can try update the code.
Then  the config.json modify a property  “enableTls”  and set it value is true and try again
We suggest you can get the code from this url
https://github.com/hyperledger/blockchain-explorer https://github.com/hyperledger/blockchain-explorer
Thank you !

在 2017年9月9日，07:47，yoyokeen<EMAIL_ADDRESS>写道：
the same error come out and how to fix it?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/onechain/fabric-explorer/issues/14#issuecomment-328238364, or mute the thread https://github.com/notifications/unsubscribe-auth/AE8HeYc7_jErwaSvJEOps-ZHYOHuNM4Vks5sgdIBgaJpZM4PRYvi.

Ok, I was able to run fabric-docker-compose-svt with TLS (this is not the default mode) enabled by manually changing start.sh to provide a -s option for ./network_setup.sh and setting environmental variable ENABLE_TLS=true.
Then did as you suggested but now I'm getting this error instead:
Please open Internet explorer to access ：http://localhost:8080/ [2017-09-09 11:46:26.646] [DEBUG] metrics - [ 0, 0 ] [2017-09-09 11:46:26.649] [DEBUG] metrics - [ 0, 0 ] [2017-09-09 11:46:26.658] [DEBUG] mysqlservice -  the getRowsBySQlCase select count(1) c from chaincodes where channelname='mychannel'  [2017-09-09 11:46:26.659] [DEBUG] mysqlservice -  the getRowsBySQlCase select max(blocknum) as blocknum from blocks  where channelname='mychannel' [2017-09-09 11:46:26.668] [DEBUG] mysqlservice -  the getRowsBySQlCase select count(1) c from transaction where channelname='mychannel' [2017-09-09 11:46:26.669] [DEBUG] mysqlservice -  the getRowsBySQlCase select max(blocknum) c from blocks where channelname='mychannel' E0909 11:46:26.718498588   94300 ssl_transport_security.c:921] Handshake failed with fatal error SSL_ERROR_SSL: error:1408F10B:SSL routines:SSL3_GET_RECORD:wrong version number. error: [client-utils.js]: sendPeersProposal - Promise is rejected: Error: Connect Failed     at /home/ercw/go/src/github.com/hyperledger/fabric/examples/fabric-docker-compose-svt/fabric-explorer/node_modules/grpc/src/node/src/client.js:554:15 error: [Client.js]: Failed Installed Chaincodes Query. Error: Error: Connect Failed     at /home/ercw/go/src/github.com/hyperledger/fabric/examples/fabric-docker-compose-svt/fabric-explorer/node_modules/grpc/src/node/src/client.js:554:15 [2017-09-09 11:46:26.723] [ERROR] Query - Error: Connect Failed     at /home/ercw/go/src/github.com/hyperledger/fabric/examples/fabric-docker-compose-svt/fabric-explorer/node_modules/grpc/src/node/src/client.js:554:15 [2017-09-09 11:46:26.723] [DEBUG] blockscanner - Error: Connect Failed     at /home/ercw/go/src/github.com/hyperledger/fabric/examples/fabric-docker-compose-svt/fabric-explorer/node_modules/grpc/src/node/src/client.js:554:15 E0909 11:46:26.734817073   94300 ssl_transport_security.c:921] Handshake failed with fatal error SSL_ERROR_SSL: error:1408F10B:SSL routines:SSL3_GET_RECORD:wrong version number. error: [client-utils.js]: sendPeersProposal - Promise is rejected: Error: Connect Failed     at /home/ercw/go/src/github.com/hyperledger/fabric/examples/fabric-docker-compose-svt/fabric-explorer/node_modules/grpc/src/node/src/client.js:554:15 error: [Channel.js]: Failed Query channel info. Error: Error: Connect Failed     at /home/ercw/go/src/github.com/hyperledger/fabric/examples/fabric-docker-compose-svt/fabric-explorer/node_modules/grpc/src/node/src/client.js:554:15 [2017-09-09 11:46:26.735] [ERROR] Query - Error: Connect Failed     at /home/ercw/go/src/github.com/hyperledger/fabric/examples/fabric-docker-compose-svt/fabric-explorer/node_modules/grpc/src/node/src/client.js:554:15 [2017-09-09 11:46:26.736] [DEBUG] Query - <<<<<<<<<< channel height >>>>>>>>> [2017-09-09 11:46:26.736] [ERROR] blockscanner - TypeError: Cannot read property 'low' of undefined     at getChainInfo.then.response (/home/ercw/go/src/github.com/hyperledger/fabric/examples/fabric-docker-compose-svt/fabric-explorer/app/query.js:242:32) [2017-09-09 11:46:27.651] [DEBUG] metrics - [ 0, 0 ] [2017-09-09 11:46:27.652] [DEBUG] metrics - [ 0, 0 ] [2017-09-09 11:46:27.652] [DEBUG] mysqlservice -  the getRowsBySQlCase select count(1) c from chaincodes where channelname='mychannel'

{
"host":"localhost",
"port":"8080",
"channelsList": ["mychannel"],
"GOPATH":"../artifacts",
"keyValueStore":"/tmp/fabric-client-kvs",
"eventWaitTime":"30000",
"enableTls":false,
"users”:[
You can try modify   property   “enableTls” to true , then try Reboot fabric-explorer again!
Thank you!

在 2017年9月10日，02:52，ercw<EMAIL_ADDRESS>写道：
Ok, I was able to run fabric-docker-compose-svt with TLS (this is not the default mode) enabled by manually changing start.sh to provide a -s option for ./network_setup.sh and setting environmental variable ENABLE_TLS=true.
Then did as you suggested but now I'm getting this error instead:
Please open Internet explorer to access ：http://localhost:8080/ [2017-09-09 11:46:26.646] [DEBUG] metrics - [ 0, 0 ] [2017-09-09 11:46:26.649] [DEBUG] metrics - [ 0, 0 ] [2017-09-09 11:46:26.658] [DEBUG] mysqlservice - the getRowsBySQlCase select count(1) c from chaincodes where channelname='mychannel' [2017-09-09 11:46:26.659] [DEBUG] mysqlservice - the getRowsBySQlCase select max(blocknum) as blocknum from blocks where channelname='mychannel' [2017-09-09 11:46:26.668] [DEBUG] mysqlservice - the getRowsBySQlCase select count(1) c from transaction where channelname='mychannel' [2017-09-09 11:46:26.669] [DEBUG] mysqlservice - the getRowsBySQlCase select max(blocknum) c from blocks where channelname='mychannel' E0909 11:46:26.718498588 94300 ssl_transport_security.c:921] Handshake failed with fatal error SSL_ERROR_SSL: error:1408F10B:SSL routines:SSL3_GET_RECORD:wrong version number. error: [client-utils.js]: sendPeersProposal - Promise is rejected: Error: Connect Failed at /home/ercw/go/src/github.com/hyperledger/fabric/examples/fabric-docker-compose-svt/fabric-explorer/node_modules/grpc/src/node/src/client.js:554:15 error: [Client.js]: Failed Installed Chaincodes Query. Error: Error: Connect Failed at /home/ercw/go/src/github.com/hyperledger/fabric/examples/fabric-docker-compose-svt/fabric-explorer/node_modules/grpc/src/node/src/client.js:554:15 [2017-09-09 11:46:26.723] [ERROR] Query - Error: Connect Failed at /home/ercw/go/src/github.com/hyperledger/fabric/examples/fabric-docker-compose-svt/fabric-explorer/node_modules/grpc/src/node/src/client.js:554:15 [2017-09-09 11:46:26.723] [DEBUG] blockscanner - Error: Connect Failed at /home/ercw/go/src/github.com/hyperledger/fabric/examples/fabric-docker-compose-svt/fabric-explorer/node_modules/grpc/src/node/src/client.js:554:15 E0909 11:46:26.734817073 94300 ssl_transport_security.c:921] Handshake failed with fatal error SSL_ERROR_SSL: error:1408F10B:SSL routines:SSL3_GET_RECORD:wrong version number. error: [client-utils.js]: sendPeersProposal - Promise is rejected: Error: Connect Failed at /home/ercw/go/src/github.com/hyperledger/fabric/examples/fabric-docker-compose-svt/fabric-explorer/node_modules/grpc/src/node/src/client.js:554:15 error: [Channel.js]: Failed Query channel info. Error: Error: Connect Failed at /home/ercw/go/src/github.com/hyperledger/fabric/examples/fabric-docker-compose-svt/fabric-explorer/node_modules/grpc/src/node/src/client.js:554:15 [2017-09-09 11:46:26.735] [ERROR] Query - Error: Connect Failed at /home/ercw/go/src/github.com/hyperledger/fabric/examples/fabric-docker-compose-svt/fabric-explorer/node_modules/grpc/src/node/src/client.js:554:15 [2017-09-09 11:46:26.736] [DEBUG] Query - <<<<<<<<<< channel height >>>>>>>>> [2017-09-09 11:46:26.736] [ERROR] blockscanner - TypeError: Cannot read property 'low' of undefined at getChainInfo.then.response (/home/ercw/go/src/github.com/hyperledger/fabric/examples/fabric-docker-compose-svt/fabric-explorer/app/query.js:242:32) [2017-09-09 11:46:27.651] [DEBUG] metrics - [ 0, 0 ] [2017-09-09 11:46:27.652] [DEBUG] metrics - [ 0, 0 ] [2017-09-09 11:46:27.652] [DEBUG] mysqlservice - the getRowsBySQlCase select count(1) c from chaincodes where channelname='mychannel'
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/onechain/fabric-explorer/issues/14#issuecomment-328296581, or mute the thread https://github.com/notifications/unsubscribe-auth/AE8HeaqOkleIBcYkcY_jwTsMFqIST80Mks5sgt5VgaJpZM4PRYvi.

The error above is with "enableTls":true

Hi Gurus,
Currently，I hit the following errors in using fabric-explorer to another sample, it is FabCar(fabric1.0 sample).Could you please help check what the cause is? thanks!
[2017-10-11 16:21:09.722] [DEBUG] metrics - [ 0, 0 ]
[2017-10-11 16:21:09.723] [DEBUG] metrics - [ 0, 0 ]
[2017-10-11 16:21:09.723] [DEBUG] mysqlservice -  the getRowsBySQlCase select count(1) c from chaincodes where channelname='mychannel'
[2017-10-11 16:21:09.724] [DEBUG] mysqlservice -  the getRowsBySQlCase select count(1) c from transaction where channelname='mychannel'
[2017-10-11 16:21:09.724] [DEBUG] mysqlservice -  the getRowsBySQlCase select max(blocknum) c from blocks where channelname='mychannel'
[2017-10-11 16:21:09.889] [ERROR] Helper - =============Peer=peer1
error: [client-utils.js]: sendPeersProposal - Promise is rejected: Error: Failed to deserialize creator identity, err The supplied identity is not valid, Verify() returned x509: certificate signed by unknown authority (possibly because of "x509: ECDSA verification failure" while trying to verify candidate authority certificate "ca.org1.example.com") at /root/fabric-explorer/node_modules/grpc/src/node/src/client.js:554:15
error: [Client.js]: Failed Installed Chaincodes Query. Error: Error: Failed to deserialize creator identity, err The supplied identity is not valid, Verify() returned x509: certificate signed by unknown authority (possibly because of "x509: ECDSA verification failure" while trying to verify candidate authority certificate "ca.org1.example.com") at /root/fabric-explorer/node_modules/grpc/src/node/src/client.js:554:15
[2017-10-11 16:21:09.897] [ERROR] Query - Error: Failed to deserialize creator identity, err The supplied identity is not valid, Verify() returned x509: certificate signed by unknown authority (possibly because of "x509: ECDSA verification failure" while trying to verify candidate authority certificate "ca.org1.example.com") at /root/fabric-explorer/no
...
The config of network-config.json is as below:
{
"network-config": {
"orderer": [{
"url": "grpc://<IP_ADDRESS>:7050",
"server-hostname": "orderer0.example.com"
}
],
"org1": {
"name": "peerOrg1",
"mspid": "Org1MSP",
"ca": "http://<IP_ADDRESS>:7054",
"peer1": {
"requests": "grpc://<IP_ADDRESS>:7051",
"events": "grpc://<IP_ADDRESS>:7053",
"server-hostname": "peer0.org1.example.com"
},
"admin": {
"key"<EMAIL_ADDRESS>"cert"<EMAIL_ADDRESS>}
}
}
}
And the config.json is as below:
{
"host":"localhost",
"port":"8080",
"channelsList": ["mychannel"],
"GOPATH":"../artifacts",
"keyValueStore":"/tmp/fabric-client-kvs2",
"eventWaitTime":"30000",
"enableTls":false,
"users":[
{
"username":"admin",
"secret":"adminpw"
}
],
"mysql":{
"host":"<IP_ADDRESS>",
"database":"fabricexplorer",
"port":"3306",
"username":"root",
"passwd":""
}
}

I have run into same error,had you fixed?

i have the same error too,

try migrating your code to fabric samples v1.0.2 ..it was a bug in v1.0..
and run the code again...it would work..and do put tls to false in network config.

same problem with 1.1 alfa

Ok so I ran into this same problem and it cost me a couple of hours to figure out, for me this was caused by running generate.sh again to regenerate my artifacts, including the certificates. There is a setting in the docker-compose.yml for the ca.example.com service. It's called FABRIC_CA_SERVER_CA_KEYFILE and it must point to the newly generated certificate file name, since on regeneration the filename changed.

I have changed the FABRIC_CA_SERVER_CA_KEYFILE with the generated artifacts of ca but still got the same error
POST /api/v1/register 401 26 "Untrusted certificate: Failed to verify certificate: x509: certificate signed by unknown authority (possibly because of "x509: ECDSA verification failure" while trying to verify candidate authority certificate "ca.epo1.epo.com")"

Another way this can happen is if the orderer container tries to start before the certificate authority container, you want to make sure that orderer.example.com has something like:
depends_on:
ca.example.com
or in some other way ensure the orderer container starts after the CA


[ENG-6620] Added tests for versioned DOI and CrossRef metadata

Purpose
Add test for versioned DOI and crossref_metadata
Changes
Add test for versioned DOI and crossref_metadata
QA Notes
N/A
Documentation
N/A
Side Effects
N/A
Ticket
https://openscience.atlassian.net/browse/ENG-6620

Update: take a look at related failures in https://github.com/CenterForOpenScience/osf.io/pull/10844/, which may overlap with your changes in this file tests/identifiers/test_crossref.py.


When device language Direction is RightToLeft buttons comes empty

When device language Arabic and use image and title it shows to me empty

This library has been deprecated.


Do not word wrap on git blame

When I do git blame word wrap is always on by default and I always turn it off because when many lines are wrapping the output of blame looks messy. Can we disable word wrap when using git blame?

Add this to the Git Blame syntax specific settings
{
    "word_wrap": false
}

Is there a scratch view that is created where word wrap should be enabled? Most of the git command output is all pre-formatted right?

@brendanator how can I add it to "the Git Blame syntax specific settings"
I've added your code to Package Settings > Git > Settings - User but it does not work.

@sjuchno
First open Git Blame on a file, then find the menu item: Preferences > Settings - More > Syntax Specific. This will be the Git Blame syntax settings file

@brendanator Thank you!

The pull request I originally made was merged.


Import issues from Trello

When importing the Boards from Trello to a newly created board, a newly created board is unclickable. After refreshing page, the newly created board is empty with no imported information.

Fixed in 6128047


How to implement iOS import library in TypeScript

Did you verify this is a real problem by searching [Stack Overflow]
Yes
Tell us about the problem
var firebase = require("nativescript-plugin-firebase");
firebase.init(...
=> This js file code working well.
import * as firebase from "nativescript-plugin-firebase"
=> This TypeScript code not work.
error TS2307: Cannot find module 'nativescript-plugin-firebase'.
Which platform(s) does your issue occur on?
I test in iOS.
Please provide the following version numbers that your issue occurs with:
CLI: (run tns --version to fetch it)
2.4.0
Cross-platform modules: (check the 'version' attribute in the
node_modules/tns-core-modules/package.json file in your project)
2.4.0
Runtime(s): (look for the "tns-android" and "tns-ios" properties in the
package.json file of your project)
2.4.0
Plugin(s): (look for the version number in the package.json file of your
project)

import * as firebase from "nativescript-plugin-firebase"

This one works on my end. What version of the nativescript-plugin-firebase package are you using? Could you put up a project on GitHub that reproduces the problem?

I used nativescript-plugin-firebase-2.1.8.

Create project (tns create fbtest --ng)
Add iOS platform (tns platform add ios)
Add firebase plugin (tns plugin add ./nativescript-plugin-firebase-2.1.8)
import * as firebase from "nativescript-plugin-firebase"
in .ts file
Build (tns build ios)

Maybe.. I don't know importing library in typescript file.
I can't find sample source that loading iOS library for typescript file in NativeScript homepage.
Thanks.
firebase_test.zip

@wantaek It looks like the 2.1.8 version of the FB plugin needs some manual work to get its typings to be recognized by TypeScript. I guess you'd need to add a <reference path="..." /> to the firebase.d.ts file inside the package.
Out of curiosity, why use that old release? The latest version of the plugin has this issue fixed.

@hdeshev I wonder why var firebase = require("nativescript-plugin-firebase"); code work well.
I download 2.1.8 version from plugins.telerik.com homepage (http://plugins.telerik.com/nativescript/plugin/firebase)
where is the lastest version?
I found nativescript-plugin-firebase demo app.
(https://github.com/EddyVerbruggen/nativescript-plugin-firebase-demo)
I try again with this demo sample.
Thanks.


Feature/union source data

Are you a current Fivetran customer?

Nope
What change(s) does this PR introduce?

Add ability to union from multiple data sources
Does this PR introduce a breaking change?

[ ] Yes (please provide breaking change details below.)
[x] No  (please provide explanation how the change is non breaking below.)
Backwards compatible

Is this PR in response to a previously created Issue

[] Yes, Issue [link issue number here]
[x] No

How did you test the PR changes?

[x] CircleCi
[ ] Other (please provide additional testing details below)

Select which warehouse(s) were used to test the PR

[x] BigQuery
[ ] Redshift
[ ] Snowflake
[ ] Postgres
[ ] Databricks
[ ] Other (provide details below)

Provide an emoji that best describes your current mood

:dancer:
Feedback
We are so excited you decided to contribute to the Fivetran community dbt package! We continue to work to improve the packages and would greatly appreciate your feedback on our existing dbt packages or what you'd like to see next.

@fivetran-reneeli feel free to re-request my review once you are ready for me to take another pass at this PR. I just reviewed and it looks like you checked the action items in my last review comment and most look good. However, I don't seem to see the README updates applied?

@fivetran-reneeli feel free to re-request my review once you are ready for me to take another pass at this PR. I just reviewed and it looks like you checked the action items in my last review comment and most look good. However, I don't seem to see the README updates applied?

@fivetran-joemarkiewicz  Thanks for calling that out, looks like I only updated the transforms readme. I've made the changes and checks have passed-- here's a sample queries to check the source relation's been added!
SELECT * FROM dbt-package-testing.dbt_renee_stg_xero.stg_xero__credit_note LIMIT 1000
SELECT * FROM dbt-package-testing.dbt_renee_xero.xero__profit_and_loss_report LIMIT 1000


Various fixes to clauses and data panel

Fixed a bug in my data panel update logic! Should be much faster
Removed clause slice to make data panel updates logically easier
I think made predicate selection / data panel interactions much more intuitive

I am pretty sure I didn't break any of the coloring logic, but was hoping you'd try it out locally as well before I merge it!


Upgrade to billing 5.1 (reverse compatible)

This is build off of @youssefali424 #387 PR because that one had a couple issues. It didn't pass back the prices properly for items that were created in android billing 4.0 amongst other issues.
I created two new dart objects that reflect android 5.0 billing objects SubscriptionOffer and PricingPhase that pass back all the information associated with a play store item, so you can navigate those to extract introductory offers and discounted subscriptions as opposed to just having a single offer price.
I've also kept the price, currency and localizedPrice values and made them backwards compatible with items that were updated from billing 4.0 if your app relies on those (ours did)
Note that the tests fail because of ios and android receipt validation only, which appears to not point to a valid URL anymore.
All tests are passing and extra code coverage has been added

Hi, i've try your PR and i got what i expected, but there is one issue in this PR and i believe its from native side

Also for minors, add toJson() and toString() on SubscriptionOfferAndroid and PricingPhaseAndroid will be much appreciated

@nizwar gotcha, I didn't manually re-test all the methods I didn't modify but that error makes sense to me, I'll try and fix it today and do a bit more hands-on testing

Any chance we can get a timeline on this? Would be great to be able to not have to use a forked plugin in production

@hyochan what's missing for this to be merged in? We've been using it in production for over a month without issue.

@SamBergeron @xPutnikx @nizwar shipped as 5.4.0 🚀

Is this still the reason why I can't see my price after upgrading to 5.4.0? I get both introductoryPrice and introductoryPricePaymentModeIOSall right but nothing else, no localizedPrice. All Android libraries are up-to-date, price is set correctly in Google Play, it still works all right with my current release version out there, just not with the upgraded plugin here. Do I need to make changes in my code or is it just a regression of some sort?


Ability to override deserializer in Kafka consumer

When binding the consumer, the kafka consumer should not be set to use ByteArrayDeserializer for both key/value deserializer. Instead, they need to be used as the default values. Any extended consumer properties for key/value deserializer should override this default deserializer.
Add test

This resolves #55

Tried to run this, but the branch runs into test failures constantly (master runs fine) - can you check @ilayaperumalg ?

@mbogoevici Initially I thought this was because of the inconsistent test behaviour before. Now I realized it was a mistake on my part not to unbind some of the bindings created during the tests. Fixed them and updated the PR.

Cool - thanks @ilayaperumalg!

Rebased, squashed and merged via https://github.com/spring-cloud/spring-cloud-stream-binder-kafka/commit/36c90aa8c0acf1fe29e235b1d2aac05115fc4c90


OCI Artifact manifest support

For https://github.com/containers/oci-spec-rs/issues/115 without IANA media-type. Media types are represented by oci_spec::MediaType::Other(String).
Summary of changes

oci_spec::image::MediaType::ArtifactManifest is added
oci_spec::image::artifact submodule is added, which defines oci_spec::image::ArtifactManifest.

Codecov Report

Merging #116 (b9b54a5) into main (987a296) will increase coverage by 0.40%.
The diff coverage is 27.11%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #116      +/-   ##
==========================================
+ Coverage   24.10%   24.50%   +0.40%
==========================================
  Files          20       21       +1
  Lines        1568     1628      +60
  Branches      770      804      +34
==========================================
+ Hits          378      399      +21
- Misses        525      539      +14
- Partials      665      690      +25

@saschagrunert Should we give out the new version?

@saschagrunert Should we give out the new version?

Yep sounds good, are you happy to cut the release?