Question regarding SSH role

Hi,
Thanks for the amazing collection your proposed here.
I have a question reagrding the role openssh and the include directive.
I'm currently running your role on a rocky linux 9 and a subdirectory exists with this directive Include /etc/ssh/sshd_config.d/*.conf
In the subfolder some redhat specific files are existing (sshd_config.d/50-redhat.conf) :
# This system is following system-wide crypto policy. The changes to
# crypto properties (Ciphers, MACs, ...) will not have any effect in
# this or following included files. To override some configuration option,
# write it before this block or include it before this file.
# Please, see manual pages for update-crypto-policies(8) and sshd_config(5).
Include /etc/crypto-policies/back-ends/opensshserver.config

SyslogFacility AUTHPRIV

ChallengeResponseAuthentication no

GSSAPIAuthentication yes
GSSAPICleanupCredentials no

UsePAM yes

X11Forwarding yes

# It is recommended to use pam_motd in /etc/pam.d/sshd instead of PrintMotd,
# as it is more configurable and versatile than the built-in version.
PrintMotd no

What is your recommendation to be able to handle this, change the role or change the optins to reflect the content of this file?
Thanks for your help.

If you would like to modify any of the parameters, try to set Ansible variables. If the values in /etc/ssh/sshd_config.d//50-redhat.conf are okay, you can leave that as is.


Jose review

Hey Cameron. This PR is part of the JOSE pre-review.

I made all of the grammatical changes suggested here. I also addressed most of the review comments by making the requested modifications, or responding to the comments in the pull request.


Use-after-free in quickfix code

Here's a minimal vimrc that exposes this problem:
au FileType qf :cclose

What happens here is that when copen is invoked we jump here when ft is set to qf.
The side effect of setting ft is the execution of the au that kills the buffer we were in so when we get back here the new curbuf doesn't match the old curbuf anymore and varp is a pointer into a block of memory that's been freed.
Possible fixes

Use s instead of *varp
Return an error message when old_curbuf (the curbuf before did_set_string_option) doesn't match curbuf

Notes

Is do_set affected as well?
The return value of set_option_value is often ignored, it might make sense to show it at least when we're setting ft. At the moment you don't know if the au failed (and the same goes for any possible error in after/ftplugin/, I think)

LemonBoy wrote:

Here's a minimal vimrc that exposes this problem:
au FileType qf :cclose

What happens here is that when copen is invoked we jump here when ft is set to qf.
The side effect of setting ft is the execution of the au that kills the buffer we were in so when we get back here the new curbuf doesn't match the old curbuf anymore and varp is a pointer into a block of memory that's been freed.
Possible fixes

Use s instead of *varp
Return an error message when old_curbuf (the curbuf before did_set_string_option) doesn't match curbuf

That autocommand basically aborts the operation, thus giving an error
and aborting would be the appropriate action.

Notes

Is do_set affected as well?
The return value of set_option_value is often ignored, it might
make sense to show it at least when we're setting ft. At the moment
you don't know if the au failed (and the same goes for any possible
error in after/ftplugin/, I think)

Autocommands can have many side effects.  Unfortunately this means we
need to check if the status is still sane after each one.  I have been
wondering about a more generic mechanism, but can't think of one (in
many cases the autocommand changes things in a way the state is still
sane, it depends on the place where it's triggered).
--
BEDEVERE:        Why do you think she is a witch?
SECOND VILLAGER: She turned me into a newt.
BEDEVERE:        A newt?
SECOND VILLAGER: (After looking at himself for some time) I got better.
"Monty Python and the Holy Grail" PYTHON (MONTY) PICTURES LTD
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

That autocommand basically aborts the operation, thus giving an error
and aborting would be the appropriate action.

A simpler solution I've just envisioned is to increment and decrement the curbuf_lock around the call to did_set_string_option, we'd stop executing the autocommand by raising a E788 and the qf windows pops open.

Is do_set affected as well?

It is and the proposed solution does fix this case as well.
If the solution is fine with you I can provide a patch if you wish me to do so.

LemonBoy wrote:

That autocommand basically aborts the operation, thus giving an error
and aborting would be the appropriate action.

A simpler solution I've just envisioned is to increment and decrement
the curbuf_lock around the call to did_set_string_option, we'd
stop executing the autocommand by raising a E788 and the qf windows
pops open.

Yes, that would also work.  Thus if it's simpler, let's do it that way.

Is do_set affected as well?

It is and the proposed solution does fix this case as well.
If the solution is fine with you I can provide a patch if you wish me
to do so.

Welcome to send a patch.
--
ALL:   A witch!  A witch!
WITCH: It's a fair cop.
ALL:   Burn her!  Burn her!  Let's make her into a ladder.
"Monty Python and the Holy Grail" PYTHON (MONTY) PICTURES LTD
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

fixed by 8.0.0677


ci: test latest Node.js

Description

We should test latest / node (alias for same) to fail early (we do not want to change / use 11 when 12 comes out and just maintain the list of LTS versions).

Closes

Types of changes

[ ] Documentation
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (anything that would change an existing functionality)
[x] Maintenance (refactor, code cleaning, development tools...)

Checklist

[x] I have read and follow the CONTRIBUTING.md document.
[x] The pull request title and template are correctly filled.
[x] The pull request targets the right branch (develop or develop-v...).
[x] My commits are correctly titled and contain all relevant information.
[ ] I have updated the documentation accordingly to my changes (if relevant).
[ ] I have added tests to cover my changes (if relevant).

Well, that is bad
Is this good to merge or did the PR contained any sensitive data?

It's good to merge now. I've revoked my token (I should have the files in the global ignore file).


Not getting the wanted results

My terminal doesn't run properly. I type commands and I keep on getting "command not listed" or other things. Can I refresh? Or anything else?
Version: 1.71.2
Commit: 74b1f979648cc44d385a2286793c226e611f59e7
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
Embedder: codespaces

Please provide a screenshot or a gif illustrating the problem (attaching images won't work via email, you need to go to github.com). See this wiki page for more general info on how to report actionable terminal issues.
/label info-needed terminal

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


CollectionError swallows exception

Is your feature request related to a problem? Please describe.
I've noticed when collecting, pytkdocs swallows errors from the module in question. For example if there is an issue importing a module, It returns that:
AttributeError: module 'module.x.y' has no attribute 'z'

However this usually points to something amiss in the module (unmet dependency). If i open up a cli and try to import the module by hand i can see whats going on.
Describe the solution you'd like
It would be nice if pytkdocs could bubble the underlying exception through to CollectionError for easier diagnosis.

I don't think the traceback would be useful, because it happens here. You would only see the stack down to pytkdocs.
This is related to https://github.com/pawamoy/mkdocstrings/issues/127. I'll mark this as error handling :slightly_smiling_face:

You can now use the new_path_syntax option to prevent pytkdocs from swallowing the exception.
In mkdocs.yml:
plugins:
  - search
  - mkdocstrings:
      handlers:
        python:
          selection:
            new_path_syntax: yes


Extract text from pdf using a url

Is it possible to extract text from a pdf file on a url(Example: 'https://example/test.pdf')

@akuokodanieljnr pdftohtmljs does not support streams. However, if you have time then PR is welcome.


adding failed:  fresh openhabian to fresh raspberrymatic on rpi0w

Hi guys,
I'm not sure about the headline or the problem, but I have tried everything I could connecting my Openhab to my Rasperrymatic and it just doesn't work and I have to start somewhere. So let's start with my first "Issue" creation here on Github.
My Setup:
2x New SD Cards and 2x new Raspberry Pi 0 W.
(A) is installed with the latest stable Openhabian. ( with openHAB 2.2.0)
(B) is installed with the latest RaspberryMatic.  ( with Version: <IP_ADDRESS>80324 )
Both are connected with good USB cables to a 2A Power supply, no Screens or input-peripherals connected.
Both are placed next to each other connected to my Fritzbox. The (A) has a Stick for Z-Wave and the (B) has the USB Stick for HmIP-RFUSB connected. (A) has nothing set up  after the standard package, I start fresh with the homematic binding and (B) has one device, a HomematicIP Sensor HmIP-SPI.
So I do Not have any other devices for Homematic, I just have one(1) Homematic IP Device.
I can connect to both using ssh and webinterface. So connecting to Raspberrymatic webinterface, setting the firewall to "Full" is all I thought to be necessary to connect those 2. But the adding in openhab failes with this error:

/var/log/openhab2/events.log:
2018-03-30 18:04:50.864 [hingStatusInfoChangedEvent] - 'homematic:bridge:3b48c490' changed from INITIALIZING to OFFLINE (COMMUNICATION_ERROR): java.net.ConnectException: Connection refused

2018-03-30 22:33:55.257 [DEBUG] [matic.handler.HomematicBridgeHandler] - HomematicConfig[gatewayAddress=<IP_ADDRESS>,callbackHost=<IP_ADDRESS>,xmlCallbackPort=9125,binCallbackPort=9126,gatewayType=auto,rfPort=2001,wiredPort=2000,hmIpPort=2010,cuxdPort=8701,groupPort=9292,timeout=15,socketMaxAlive=900]
2018-03-30 22:33:55.266 [TRACE] [matic.handler.HomematicBridgeHandler] - Registering HomematicDeviceDiscoveryService for bridge 'a2274cc4'
2018-03-30 22:33:55.294 [DEBUG] [org.openhab.binding.homematic       ] - ServiceEvent REGISTERED - {org.eclipse.smarthome.config.discovery.DiscoveryService}={service.id=314, service.bundleid=204, service.scope=singleton} - org.openhab.binding.homematic
2018-03-30 22:33:55.376 [TRACE] [nal.communicator.client.XmlRpcClient] - Client XmlRpcRequest (port 2001):

getDeviceDescription
BidCoS-RF
2018-03-30 22:33:55.408 [DEBUG] [nal.communicator.client.XmlRpcClient] - XmlRpcMessage failure, sending message again 1/1
2018-03-30 22:33:55.432 [DEBUG] [matic.handler.HomematicBridgeHandler] - Disposing bridge 'a2274cc4'
2018-03-30 22:33:55.456 [DEBUG] [very.HomematicDeviceDiscoveryService] - Stopping Homematic discovery scan
2018-03-30 22:33:55.483 [DEBUG] [org.openhab.binding.homematic       ] - ServiceEvent UNREGISTERING - {org.eclipse.smarthome.config.discovery.DiscoveryService}={service.id=314, service.bundleid=204, service.scope=singleton} - org.openhab.binding.homematic

I have attached my log files
homematic-raspi-2018-03-30.log
openhab.log
events.log
it somehow tries to connect to port 2001 on the raspberrymatic, which is not available. The internet suggests that the HMS failed to boot, I think, I just do not see that.
here is a log file from earlier in the day that at least mentions port 2001:
homematic-raspi-2018-03-30_with_2001.log
Additional Infos: I used manuall adding within openhab and added the IP of the (B) leaving the rest on default. I tried (like some forum posts suggest) to play around with different combination also on the fallback port, not successful.
Here is the netstat -l on the raspberrymatic:

netstat -l
Active Internet connections (only servers)
Proto Recv-Q Send-Q Local Address           Foreign Address         State
tcp        0      0 <IP_ADDRESS>:www             <IP_ADDRESS>:*               LISTEN
tcp        0      0 <IP_ADDRESS>:8181            <IP_ADDRESS>:*               LISTEN
tcp        0      0 <IP_ADDRESS>:ssh             <IP_ADDRESS>:*               LISTEN
tcp        0      0 <IP_ADDRESS>:8183            <IP_ADDRESS>:*               LISTEN
tcp        0      0 <IP_ADDRESS>:https           <IP_ADDRESS>:*               LISTEN
tcp        0      0 localhost:705           <IP_ADDRESS>:*               LISTEN
tcp        0      0 <IP_ADDRESS>:1999            <IP_ADDRESS>:*               LISTEN
tcp        0      0 :::www                  :::*                    LISTEN
tcp        0      0 :::8181                 :::*                    LISTEN
tcp        0      0 :::ssh                  :::*                    LISTEN
tcp        0      0 :::2010                 :::*                    LISTEN
tcp        0      0 :::https                :::*                    LISTEN
tcp        0      0 :::9292                 :::*                    LISTEN
udp        0      0 <IP_ADDRESS>:55634           <IP_ADDRESS>:*
udp        0      0 homematic-raspi.fritz.box:ntp <IP_ADDRESS>:*
udp        0      0 localhost:ntp           <IP_ADDRESS>:*
udp        0      0 <IP_ADDRESS>:ntp             <IP_ADDRESS>:*
udp        0      0 <IP_ADDRESS>:snmp            <IP_ADDRESS>:*
udp        0      0 <IP_ADDRESS>:43439           <IP_ADDRESS>:*
udp        0      0 fe80::ba27:ebff:fe95:92aa:ntp :::*
udp        0      0 fd00::ba27:ebff:fe95:92aa:ntp :::*
udp        0      0 ::1:ntp                 :::*
udp        0      0 :::ntp                  :::*
udp        0      0 ::1:snmp                :::*
udp        0      0 :::33967                :::*
Active UNIX domain sockets (only servers)
Proto RefCnt Flags       Type       State         I-Node Path
unix  2      [ ACC ]     SEQPACKET  LISTENING       5829 /run/udev/control
unix  2      [ ACC ]     STREAM     LISTENING       7662 /var/run/dbus/system_bu

Other info's I find strange:

I have another (manual) Openhab 2.2 Instance running on my VMWare Server and have the same problem towards the Raspberrymatic there.
Openhab's Auto-discovery does not find the raspberrymatic
Port 2001 on (B) (Netstat -l) is never open but this log entry in Openhab want it to be:
2018-03-29 00:45:48.433 [hingStatusInfoChangedEvent] - 'homematic:bridge:32ff3af2' changed from OFFLINE (COMMUNICATION_ERROR): java.net.ConnectException: Connection refused to OFFLINE (COMMUNICATION_ERROR): java.net.SocketException: Could not connect to /<IP_ADDRESS>:2001
the SSH is very unstable
the HmIP-SPI has the info that a firmware upgrade is uploading.. (since 2 days now).

Sometimes, I also receive popups in the (B) Webinterface about 3 unavailable Services every several minutes, but I think a better usb cable fixed thoses for now..

HmIP-RF
VirtualDevices
ReGa

That is where I am now, I have no idea what's the problem and where to go from here. Can you help?
Please let me know if I issues this issue correctly or if there is something I forgot / should have included.

a come on, I am searching for a solution for days and then I just post this issue here and like minutes later, I found this:
https://github.com/openhab/openhab2-addons/issues/3309

You need 2.3 for this. But there is a bug in RaspberryMatic <IP_ADDRESS>80225, that prevents finding the gateway. Either use <IP_ADDRESS>80120 or wait for the next release.

I'll try it with 2.3 asap and will update this issue accordingly.

If you only use a HmIP-RFUSB with RaspberryMatic then the BidCos-RF Service is of course not running but only the HmIP services which use a completely different port/API. Thus, you would need a HM-MOD-RPI-PCB or LAN Gateway to get the BidCos-RF services starting correctly so that they are available. On the other hand OpenHab (if supporting HomeMaticIP) should allow to be started without the BidCos-RF service in place.


[FEA] Add Hive UDF support for urlDecoder

Is your feature request related to a problem? Please describe.
With https://github.com/rapidsai/cudf/pull/6972 and possibly other changes to support hive udfs, the plugin should allow for hive udfs with url decoding to work on the gpu.

This is a duplicate of #1352.


doc: Add package and module docstrings for use in help()

Co-Authored-By: Kurt Griffiths<EMAIL_ADDRESS>

[Current coverage][cc-pull] is 100% (diff: 100%)

Merging [#850][cc-pull] into [master][cc-base-branch] will not change coverage

@@           master   #850   diff @@
====================================
  Files          29     29
  Lines        1821   1848    +27
  Methods         0      0
  Messages        0      0
  Branches      307    307
====================================
+ Hits         1821   1848    +27
  Misses          0      0
  Partials        0      0

Powered by Codecov. Last update [1cf9eeb...875fc9b][cc-compare]
[cc-base-branch]: https://codecov.io/gh/falconry/falcon/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/falconry/falcon/compare/1cf9eebe3174134a4995de08d98597fb2fe069c4...875fc9b5d87eda2b3327ad2879cc3c806ad1eff4?src=pr
[cc-pull]: https://codecov.io/gh/falconry/falcon/pull/850?src=pr


Seed documentation for the resolution project

Feature request
There are essentially no docs for resolution currently. Introduce an initial set that covers:

Tutorial on how to get setup and running from scratch with tekton pipelines + resolution + gitresolver
Developer-focused how-to guide on writing a new resolver from scratch
Technical explanation on the current implementation of resolution and reasons for it being the way it is
Technical reference on the types and behaviours that Tekton Resolution has introduced

Use case
Help end-users get started with tekton resolution, help developers attempting to write their own resolver, and given as much background as possible for the reasoning behind resolution's current state. I'm using https://documentation.divio.com/ as a guide to the different kinds of docs that it would be worth having initially.

/assign


chore(deps): update common-dev-assets digest to 8eef612

This PR contains the following updates:

Package
Update
Change

common-dev-assets
digest
22ebbcb -> 8eef612

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

:tada: This PR is included in version 1.10.15 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[WIP] deps: Merge dependabot updates

Commit Message: deps: Merge dependabot updates
Additional Description:
merge the update prs into one to ensure they work together and lessen commit noise
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Deprecated:]
[Optional API Considerations:]

idna cant be updated until requests has been - see https://github.com/psf/requests/issues/5710 and https://github.com/psf/requests/pull/5711

/retest

/lgtm deps


Fixed some deprecation messages

Fixed the following two deprecation messages, which are reported by Symfony's deprecation helper:

1x: Method "Pimple\Container::offsetGet()" might add "mixed" as a native return type declaration in the future. Do the same in child class "ZBateson\MailMimeParser\Container" now to avoid errors or add an explicit @return annotation to suppress this message.
1x: Method "ArrayAccess::offsetGet()" might add "mixed" as a native return type declaration in the future. Do the same in implementation "ZBateson\MailMimeParser\Message\PartChildrenContainer" now to avoid errors or add an explicit @return annotation to suppress this message.

The return annotations were already added by #197, but reverted again with commit c1c4b6857cc35ca4b3be19e17d86a811d9296619 (PHPCSFixer).

thank you!


Optional reagents don't seem to affect missing skill calculation

Hi,
After upgrading to 7.2.2 today, I don't think the calculation of missing skill (and missing skill with inspiration) is working correctly with optional reagents.
Below, you can see the calculated missing skill for a blacksmithing axe, without any optional reagents:

When adding optional reagents, the calculation doesn't change, even though the recipe has a much higher difficulty now:

Steps to reproduce:

Open profession window (tested with blacksmithing and alchemy)
Enter simulation mode
Add mandatory reagents and observe the calculated missing skill (and missing skill with inspiration)
Add any optional reagent that changes the required skill (or a finishing reagent that boosts inspiration skill)
Observe that there is no change in calculated missing skill.

Fixed in 7.3+


Refactor Process Mode

Implements https://github.com/godotengine/godot-proposals/issues/1835#issuecomment-727186192 , extending the pause functionality to a more generic process mode. This allows disabling of nodes, as well as their children.
Keep in mind that disabling nodes only affects the logic, and does not hide them. This goes separate to the visual part.

PauseMode is now ProcessMode, containing the following states:PROCESS_MODE_INHERIT, // same as parent node
PROCESS_MODE_PAUSABLE, // process only if not paused
PROCESS_MODE_WHEN_PAUSED, // process only if paused
PROCESS_MODE_ALWAYS, // process always
PROCESS_MODE_DISABLED, // never process

NOTIFICATION_PAUSED and NOTIFICATION_UNPAUSED are received effectively when the node is paused and unpaused (not any longer when pause mode is set in SceneTree).
Renamed some nodes that used ProcessMode/process_mode to specify a callback type to ProcessCallback to avoid clashes.

Obligatory Screenshot:

Thanks!


update yarn cache

Attempting to update cache, following previous fix for this here: https://github.com/transcom/mymove/pull/4194

CircleCI client test and Happo CI / happo will be the telling factors

Adding pdfjs resolution will remove reliance on canvas module but then storybook fails. Adding it back gives the canvas module missing error.


March 2016 update for increasing-engagement-with-app-install-banners-in-chrome-for-android

removed the Cr-UI-Browser-AppShortcuts paragraph because the link no longer points to any issues
added having a start_url in the manifest as a requirement for the app install banner

The other changes are pretty straight-forward.

Is this post useful now that we have https://developers.google.com/web/fundamentals/engage-and-retain/app-install-banners/ ?

Related: https://github.com/google/WebFundamentals/pull/2617

Thanks for the merge. Is this post useful now that we have https://developers.google.com/web/fundamentals/engage-and-retain/app-install-banners/ ? Should it link to the latter?

Yep, at some point we should add a banner to the top of the updates doc and point to the fundamentals doc.
Essentially the workflow should be new feature in updates, then when it becomes stable and is in other browsers, it goes to fundamentals.


About extracting trained model

When I downloaded the trained model, I could not extract the 'tar' file. I was wondering if there is something wrong with your 'tar' file.

The .tar files are not compressed - they are simply pytorch-saved dictionaries. In other words, load them directly from pytorch without any decompression.

Thank you for your reply. I truly appreciate your help in resolving my problem.
发自我的iPhone
------------------ Original ------------------
From: Fangchang Ma<EMAIL_ADDRESS>Date: Sun,Mar 22,2020 5:06 PM
To: fangchangma/self-supervised-depth-completion<EMAIL_ADDRESS>Cc: Suraimu<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [fangchangma/self-supervised-depth-completion] About extracting trained model  (#39)
The .tar files are not compressed - they are simply pytorch-saved dictionaries. In other words, load them directly from pytorch without any decompression.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Update cgmanifest.json since we don't include source code of nlohmann json anymore

https://github.com/Azure/azure-sdk-for-cpp/blob/master/sdk/storage/azure-storage-blobs/cgmanifest.json#L17

Is this file auto-generated? How should we edit this file? Can I just remove that item of nlohmann json? Will it break something?

Is this file auto-generated? How should we edit this file? Can I just remove that item of nlohmann json? Will it break something?

The cg manifest is meant to declare third party dependencies, and not just for code that is directly included in the SDK.
The file is not auto-generated. If the JSON dependency is coming from core now, we would want to move this to core's cgmanifest.

The cg manifest is meant to declare third party dependencies, and not just for code that is directly included in the SDK.
The file is not auto-generated. If the JSON dependency is coming from core now, we would want to move this to core's cgmanifest.

@ahsonkhan please move/update the declaration of the dependency to make it accurate for the where the dependency is coming from.

@ahsonkhan please move/update the declaration of the dependency to make it accurate for the where the dependency is coming from.

We also need to update the NOTICE.txt files, right? Because it still contains license notice for json library.
https://github.com/Azure/azure-sdk-for-cpp/blob/fb33402844be8aaeedcbcec645b8be9c69fa6242/sdk/storage/azure-storage-blobs/NOTICE.txt#L68

We also need to update the NOTICE.txt files, right? Because it still contains license notice for json library.
https://github.com/Azure/azure-sdk-for-cpp/blob/fb33402844be8aaeedcbcec645b8be9c69fa6242/sdk/storage/azure-storage-blobs/NOTICE.txt#L68


feat(docker-container): create docker image from Dockerfile

As introduced in https://github.com/stellar/soroban-example-dapp/issues/89, this PR (draft) introduces the creation of a Docker image with all the necessary dependencies (depending on the current release https://soroban.stellar.org/docs/reference/releases)

rust
soroban-cli

Tasks:

[x] Create Docker Image (soroban-preview:7)
[x] Start docker container with that image
[x] Make ./initialize.sh to work insithe this docker container
[x] Make docker container to run when quickstart.sh

When trying:
cd docker
bash build.sh

I get this error while installing soroban-cli
 > [ 9/11] RUN cargo install --locked --version 0.6.0 soroban-cli:
#12 0.473     Updating crates.io index
#12 160.1  Downloading crates ...
#12 161.1   Downloaded soroban-cli v0.6.0
#12 161.3   Installing soroban-cli v0.6.0
.
.
.
.
.#12 175.6    Compiling serde v1.0.152
#12 175.8    Compiling serde_derive v1.0.152
#12 176.0 error: linker `cc` not found
#12 176.0   |
#12 176.0   = note: No such file or directory (os error 2)
#12 176.0
#12 176.0 error: could not compile `quote` due to previous error
#12 176.0 warning: build failed, waiting for other jobs to finish...
#12 176.1 error: could not compile `syn` due to previous error
#12 176.1 error: could not compile `libc` due to previous error
#12 176.1 error: could not compile `proc-macro2` due to previous error
#12 176.3 error: could not compile `serde_derive` due to previous error
#12 176.3 error: could not compile `serde` due to previous error
#12 176.3 error: failed to compile `soroban-cli v0.6.0`, intermediate artifacts can be found at `/tmp/cargo-installHGvkgm`
------
process "/bin/sh -c cargo install --locked --version 0.6.0 soroban-cli" did not complete successfully: exit code: 101

Docker compose might not be really needed.
Until now is ok enough. However I'm not able to try it all due to #89

Ready for review! Thanks!

Fixes #89


Add rayon thumbnail generation example

Sorry for taking a while on this, my semester recently started up again and I've been a bit busy with course work.
I did most of the changes, but the error handling part in par_iter() is really ugly in my version, so any help with that would be greatly appreciated.
Right now when I run the program with one corrupt .jpg in the directory I get this output:
Saving 197 thumbnails into "thumbnails"
"broken.jpg" failed: Format error: first two bytes is not a SOI marker
Done

Also, when I updated my fork with git pull upstream master and now cargo test fails on some of the examples, so I'm not sure what I did there...
failures:
    encoding_sect_deserialize_a_toml_configuration_file_line_132
    encoding_sect_handle_invalid_csv_data_with_serde_line_456
    encoding_sect_read_csv_records_with_different_delimeter_line_507
    encoding_sect_serialize_records_to_csv_using_serde_line_403
    net_sect_consume_a_paginated_restful_api_line_632
    net_sect_create_and_delete_gist_with_github_api_line_552
    net_sect_query_the_github_api_line_377
    net_sect_set_custom_headers_and_url_parameters_for_a_rest_request_line_477

test result: FAILED. 68 passed; 8 failed; 0 ignored; 0 measured; 0 filtered out

error: test failed, to rerun pass '--test skeptic'

Also, I couldn't get the AsRef<Path> thing to work because when I tried to call .join() for the output_path I would get an error that I didn't know how to fix. Something to do with AsRef not implementing the .join() method or something.

Sorry for taking a while on this, my semester recently started up again and I've been a bit busy with course work.

no reason to be sorry :)

Also, when I updated my fork with git pull upstream master and now cargo test fails on some of the examples, so I'm not sure what I did there...

Most likely its a skeptic but. cargo clean will fix the problem.

error handling part in par_iter() is really ugly in my version,

we will fix it in the course of the review :)

Also, I couldn't get the AsRef thing to work because when I tried to call .join()

To use the AsRef you would have to

Merged! I've made some minor cleanup and squashed all the commits into a single one.
Thanks for sticking with it @nocduro!


Data grid not loading

hello all
i am using laravel 5.2 and i update to kordy/ticketit:0.2.x-dev
my data grid not loading

script
$('.table').DataTable({
had not load properly


egui ID clash in Selection panel

git checkout 3dc64f9ebb5f04a7e696a4def64b6a66fe7a977d
pixi run rerun

Open any example
Select the recording (press escape)
Scroll horizontally in the selection view

I can't reproduce this, even on this exact commit 🤔


Removed the singletone TyphoonTypeConverterRegistry

This PR fixes the Issue #407. TyphoonTypeConverterRegistry is now not a shared variable, but relates to TyphoonComponentFactory.
Besides it, I've added a couple of new unit tests and a TyphoonNSColorTypeConverter, as an OSX opposite of TyphoonUIColorTypeConverter.

:+1: Nice


data(mental-health): add wgm 2020 report

This PR adds the WGM 2020 dataset on Mental Health.
The original data comes as a questionnaire. We build two types of variables:

Share: share of a particular answer to a particular question.
Count: number of a particular answer to a particular question.

Variables have long naming convention:
Share - Question: [QUESTION] - Answer: [ANSWER] - Gender: [GENDER] - Age_group: [AGE GROUP]

e.g.
Share - Question: After the COVID crisis ends, your government should spend money on preventing and curing dieases ONLY if they pose a risk to poeple in your country - Answer: DK/Refused - Gender: all - Age_group: 15-29

Other points

shared.py contains ID to label mappings.
Percentages are obtained by weighing the answers. Source provides these weights, which aims to minimise bias in survey estimate

Thanks for your feedback, Pablo! 😄


[BUG] Invalid JSON events - Task details JSON logs

Describe the bug
In task details JSON logs opensearch_task_detailslog.json the value of metadata field is not correctly sanitized. That is, the quotation marks are not escaped. Therefore, when processing the event with a JSON parser it fails.
{
    "type": "direct",
    "timestamp": "2023-07-03T21:51:23,773Z",
    "level": "INFO",
    "component": "t.d.search",
    "cluster.name": "wazuh-cluster",
    "node.name": "node-1",
    "message": "taskId:[75900], type:[direct], action:[indices:data/read/search[can_match]], description:[shardId[[wazuh-alerts-4.x-2023.04.11][0]]], start_time_millis:[1688419349792], resource_stats:[{}], metadata:[source[{\"size\":1,\"query\":{\"bool\":{\"must\":[{\"term\":{\"rule.mitre.tactic\":{\"value\":\"Lateral Movement\",\"boost\":1.0}}},{\"term\":{\"rule.mitre.technique\":{\"value\":\"SSH\",\"boost\":1.0}}}],\"adjust_pure_negative\":true,\"boost\":1.0}},\"sort\":[{\"timestamp\":{\"order\":\"desc\"}}]}]]",
    "taskId": "75900",
    "action": "indices:data/read/search[can_match]",
    "description": "shardId[[wazuh-alerts-4.x-2023.04.11][0]]",
    "start_time_millis": "1688419349792",
    "resource_stats": "{}",
    "metadata": "source[{"size":1,"query":{"bool":{"must":[{"term":{"rule.mitre.tactic":{"value":"Lateral Movement","boost":1.0}}},{"term":{"rule.mitre.technique":{"value":"SSH","boost":1.0}}}],"adjust_pure_negative":true,"boost":1.0}},"sort":[{"timestamp":{"order":"desc"}}]}]",
    "cluster.uuid": "eOXG1FbXQB2KVUJrjJeSQQ",
    "node.id": "Cu8rIDeyTeiZVDWOoyubFQ"
}

To Reproduce
Enable task logging through the cluster settings API:
PUT _cluster/settings
{
  "persistent" : {
    "task_resource_consumers.enabled" : "true"
  }
}

Expected behavior
The event should comply with JSON format.
Plugins
opensearch-alerting
opensearch-anomaly-detection
opensearch-asynchronous-search
opensearch-cross-cluster-replication
opensearch-geospatial
opensearch-index-management
opensearch-job-scheduler
opensearch-knn
opensearch-ml
opensearch-neural-search
opensearch-notifications
opensearch-notifications-core
opensearch-observability
opensearch-performance-analyzer
opensearch-reports-scheduler
opensearch-security
opensearch-security-analytics
opensearch-sql
Host/Environment (please complete the following information):

OS: Ubuntu
Version 20.04
OpenSearch 2.6.0

@anasalkouz why is this labeled as "good first issue"? Would it be easy to understand the bug for a newcomer?

Yes "good first issue" labels, means it's going to be straight forward for newcomers to fix. Or at least that what we anticipated to be.

@anasalkouz Thanks for your answer. I'd like to try working on it. Could you give me some guidance (e.g. is there some module I need to check, is there a way to know which component of the software writes those log?) to start?


chore: add has validate basic compiler assertion

Description

Leaving as draft because it needs SDK v0.50.
closes: #XXXX
Commit Message / Changelog Entry
type: commit message

see the guidelines for commit messages. (view raw markdown for examples)

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[x] Targeted PR against correct branch (see CONTRIBUTING.md).
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards and Go style guide.
[ ] Wrote unit and integration tests.
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/).
[ ] Added relevant godoc comments.
[ ] Provide a commit message to be used for the changelog entry in the PR description for review.
[ ] Re-reviewed Files changed in the Github PR explorer.
[ ] Review Codecov Report in the comment section below once CI passes.

Feature branch is now on SDK v0.50, so this PR is ready for review.


Verification codes don't work in Chile due to changes in DST schedule

Original issue 149 created by maralbust on 2012-03-12T15:00:55.000Z:
What steps will reproduce the problem?

change time on my phone
try acces to my gmail account

Doesn´t grant acces to my gmail account.
The codes are date/time dependace and now in my country (Chile), we didn't change the summer time, until march 28.

This is still happening, apparently, on iOS and Android devices. Here in Mexico DST ended yesterday (October 30th, 2016) and now the Authenticator generates codes that are not valid. The timezone and DST settings on the phones are correct, and other apps (for instance, 1Password) generate valid codes.
I already tried on the Android devices to use the "sync time" setting but it did not change anything, and the codes are still invalid.

Marking iPhone since this affects iphone from what I can read in the comments. If android too, then create bug in android project.


Reduce app size

Hey 👋
First, thank you for such an awesome tool. The question I have is related to the app's size.
Freshly installed on mac os it's almost 520mb. As an example, VSCode editor is 230mb, twice less.
So why is graphql client so heavy, maybe it's possible to reduce its size?

Hey, never really looked into why the bundle is so large. I always assumed electron apps have large sizes due to the nodejs runtime that is bundled with the apps.

@imolorhe yep, this is why I added VSCode as an example, it's also an electron app. I also have another electron apps: Twitch (~220mb), RunJs (~190mb), Figma (~140mb), Atom (~590mb).
Interesting if it's somehow possible to reduce Altrair size as there are definitely smaller electron apps :)

Thanks I have found the reason for the large app size. I don't filter out the application content when packaging the app so the electron app is bundled with all the files it finds in the directory.
I'll add a files list to ensure only required files are packaged. 🙂

Digging further, I think I can reduce the app size even more if I have the electron app as a separate package, instead of having it as part of the main repo.

188.7mb ❤

@imolorhe epic 🎉

Assets for v2.3.4:

Assets for v2.3.5:

All assets sizes decreased by half. Thanks for mentioning this 😊

@allcontributors[bot] Please add @vladshcherbin for bugs.

@imolorhe after installation, on mac os the app size dropped from 520mb to 190mb.
Thank you a lot! 🎉👍

Any issues with the new installation? Lost queries, windows, collections?

@imolorhe I haven't used Altair heavily with all features it has. No issues with things I used though, everything seems to be fine 👌

Okay. Good to hear.


Adding I18n

Added a config/locales/en.yml (and also a pt-BR.yml) file, moved a lot of hardcoded strings into it and made references to them using Rails's I18n. Also changed a few date's strftime to use I18n.l method.

I've squashed your commits, added i18n to the gemspec and merged to master: https://github.com/assaf/vanity/commit/906d0017d7da400d47c9b142c35c3dc27b5827fa. Thanks for doing this work @teonimesic!


Collapsible properties

Made the properties on the dashboard collapsible. It says the most recent state in the settings.
Known issue: only will say to USER settings, needs to be able to write to where ever it got the setting from, defaulting to user.

Can we have collapsible icon next to ...? From the screenshot I can't tell if the properties widget is collapsed and can be expanded. Customer can't tell if the widget is collapsed or collapsible until they check the ... button


Theme notification is send twice on some nodes

Godot version
4.x
System information
Windows 10
Issue description
Some nodes receive the theme notification twice.
The problematic code:

The gui_base can be a direct child of theme_base.
In this case the theme_base sets the theme and propagates the notification to all children down in the scene tree, including gui_base.
Then gui_base does the same again.
Steps to reproduce
Found when testing https://github.com/godotengine/godot/pull/70410

Add a breakpoint to the NOTIFICATION_THEME_CHANGED in e.g. editor_audio_buses.cpp

Change the editor theme
See how NOTIFICATION_THEME_CHANGED is called twice

In this case the following condition is true: gui_base->get_parent() == theme_base. There are probably more combinations out there...
Minimal reproduction project
--

cc @YuriSizov - Not a big problem but maybe still worth checking out :)

We plan to refactor EditorNode in 4.1+, and this would also go. I think @AaronRecord had a PR that simplified this theme situation for 4.0, but we didn't merge it yet.

Good idea. I also saw https://github.com/godotengine/godot/pull/62846, which unfortunately was reverted due to problems.
But this is also a good refactoring.
While testing, I also found that NOTIFICATION_ENTER_TREE  paired with NOTIFICATION_THEME_CHANGED is almost never needed and nothing broke when I tried removing it. So definitely worth doing on some point.

This has been improved in the majority of GUI classes, but still needs to be done in the editor code. We now have a virtual method that can be implemented on a Control/Window derivative that you'd use to cache all theme properties as needed. Most of built-in controls no longer need to bind to either notification. Some still do because I had to omit them from my PRs as they've been refactored by other PRs in the process.
See https://github.com/godotengine/godot/pull/65156 and https://github.com/godotengine/godot/pull/65250.
Same changes need to be done to the editor code, where we subscribe to theme changes. It needs to be moved to use the new method and the theme_cache pattern.


ie8 button fix

new rails from my master branch do the trick without code modifications
tested on vm with XP Pro and ie8

Can you please rephrase your issue and make it more clear. I am not quite sure what you are referring to or what your issue is. Thanks!


Clarification on Referring Segmentation

Based on the code:
https://github.com/microsoft/X-Decoder/blob/584123df0792cfbee7500d7a5ea8c3878d146122/datasets/dataset_mappers/refcoco_dataset_mapper.py#L158
https://github.com/microsoft/X-Decoder/blob/584123df0792cfbee7500d7a5ea8c3878d146122/xdecoder/architectures/xdecoder_model.py#L419-L424
Is it true that referring segmentation in X-Decoder is done by segmentation -> classification (matching mask with highest similarity)?

Yes, but we are not directly matching the mask embedding, we compute hungarian matching between pred mask and gt mask, pred text embedding and gt text embedding. During evaluation we use text embedding matching score only.


Fresh install Errors

Hi there,
Im trying to install foxystreams and foxymeta on a fresh install but getting some errors
01_KODI.log

this is what I have not sure if its the same
20:02:50.019 T:14872   ERROR: EXCEPTION Thrown (PythonToCppException) : -->Python callback/script returned the following error<--
- NOTE: IGNORING THIS CAN LEAD TO MEMORY LEAKS!
Error Type: <type 'exceptions.AttributeError'>
Error Contents: 'xbmcaddon.Addon' object has no attribute 'getSettingBool'
Traceback (most recent call last):
File "C:\Users\ED\Documents\KODI\KRYPTON\Kodi\portable_data\addons\plugin.video.foxystreams\launcher.py", line 4, in
addon.run(*sys.argv[:3])
File "C:\Users\ED\Documents\KODI\KRYPTON\Kodi\portable_data\addons\plugin.video.foxystreams\addon.py", line 413, in run
for provider in get_user_debrid_providers():
File "C:\Users\ED\Documents\KODI\KRYPTON\Kodi\portable_data\addons\plugin.video.foxystreams\addon.py", line 112, in get_user_debrid_providers
if router.addon.getSettingBool(
AttributeError: 'xbmcaddon.Addon' object has no attribute 'getSettingBool'
-->End of Python script error report<--

this is what I have not sure if its the same
20:02:50.019 T:14872   ERROR: EXCEPTION Thrown (PythonToCppException) : -->Python callback/script returned the following error<--
- NOTE: IGNORING THIS CAN LEAD TO MEMORY LEAKS!
Error Type: <type 'exceptions.AttributeError'>
Error Contents: 'xbmcaddon.Addon' object has no attribute 'getSettingBool'
Traceback (most recent call last):
File "C:\Users\ED\Documents\KODI\KRYPTON\Kodi\portable_data\addons\plugin.video.foxystreams\launcher.py", line 4, in
addon.run(*sys.argv[:3])
File "C:\Users\ED\Documents\KODI\KRYPTON\Kodi\portable_data\addons\plugin.video.foxystreams\addon.py", line 413, in run
for provider in get_user_debrid_providers():
File "C:\Users\ED\Documents\KODI\KRYPTON\Kodi\portable_data\addons\plugin.video.foxystreams\addon.py", line 112, in get_user_debrid_providers
if router.addon.getSettingBool(
AttributeError: 'xbmcaddon.Addon' object has no attribute 'getSettingBool'
-->End of Python script error report<--


Implement share functionality

Context
The current share menu lacks functionality for users to share content directly via Email, Facebook, X (formerly Twitter), or by copying a link. Adding this functionality will improve user experience and encourage content sharing.
Definition of Done

Share menu is updated to include the following working options: Email, Facebook, X (formerly Twitter), Copy link;
Clicking on each menu item triggers the respective share functionality:

Email: Opens the default email client with a prefilled subject and body containing a link.
Facebook: Opens Facebook's share dialog for sharing the content link.
X: Opens X's sharing interface with a prefilled tweet containing the content link.
Copy link: Copies the content link to the user's clipboard, with a visual confirmation (e.g., toast notification).

Testing is conducted to ensure each share option works correctly

Engineering Details

Update the existing Share component to add the functionality.
Implement sharing functionality using:

mailto: links for Email sharing.
Facebook’s Share Dialog URL: https://www.facebook.com/sharer/sharer.php?u=<URL>.
X’s Tweet Composer URL: https://twitter.com/intent/tweet?url=<URL>.
Clipboard API for copying the link (navigator.clipboard.writeText).

Use Jest and React Testing Library to test the share functionality:

Verify mailto:, Facebook, and X links render with correct href attributes.
Mock the Clipboard API to test that the "Copy link" feature copies the correct URL and shows a confirmation message.

Limitations: Testing frameworks cannot verify the execution of external sharing. Focus tests on DOM interaction and correct link generation.

test

What I have right now:

Is this what we are going for or do we want to be able to share a specific report that we click on?

url changes to this when searching but the page data does not change

Was able to get the url to work for Twitter, email, and copying it -> facebook seems to have an issue with the share so I'll be doing some additional tests

[ ] check if it works with https url
[ ] check if it works with localhost
[ ] document findings


Correct Chrome supported versions for promise rejection tracking

Chrome supports unhandledrejection/rejectionhandled since version 49, not since version 36 (version 36 is the first supported version for Opera).
See also https://www.chromestatus.com/feature/4805872211460096

Thanks!


False positive on shape check in default when an object global var is used

  global current_organization_id -> uuid;
  global current_organization := (
    select Organization filter .id = global current_organization_id
  );

  type Organization;

  type SomeOrgSpecificObject {
    required link organization -> Organization {
      default := global current_organization;
    }
  }

error: SchemaDefinitionError: default expression may not include a shape
   ┌─ query:11:7
   │
11 │       default := global current_organization;
   │       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ error

A very unpleasant bug, the consequence of which is the redundancy of the model code.

Is there any update on this bug?
As soon as I add globals to schema, the abovementioned example becomes very tempting and useful to use (set defaults from globals). But this bug prevents it from using completely.

For edgedb 3.0 the bug is still relevant. When can we expect a fix?

Fix will go out in 3.4


Garbage collection, how much self tuning occurs?

"The garbage collector is self-tuning and can work in a wide variety of scenarios. You can use a configuration file setting to set the type of garbage collection based on the characteristics of the workload"
You're saying it's self-tuning, immediately followed by 'set it yourself'.  Does an application without a setting for <gcServer> automatically switch between workstation and server modes as part of the self-tuning?  If so it means that performance should gradually reach its optimum level, but if not it could be a costly mistake to omit it.

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

ID: d8948cf5-0aec-308c-8046-965cc15763b2
Version Independent ID: a30a0be5-b741-2a9c-e997-55d6d9960ca9
Content: Fundamentals of Garbage Collection
Content Source: docs/standard/garbage-collection/fundamentals.md
Product: dotnet
GitHub Login: @rpetrusha
Microsoft Alias: ronpet

Thanks for submitting your questions, @ro-jo.
@maioni0, can you answer @ro-jo's questions? And do we need to provide some additional detail in the documentation about what self-tuning means and when adding or modifying the <gcServer> and <gcConcurrent> elements is appropriate?

@ro-jo asks a reasonable question but somehow no one ever responded to @rpetrusha's urging about it. @maioni0 or others?

@ro-jo
IIUC,
Server GC is not the default and must be opted into. Once an application is started with either a Server or Workstation GC, it cannot be changed during the application's lifetime.
The self tuning aspects of the GC refer to collection timing and aggressiveness which enables the GC to deliver good performance across many types of workloads.
@rpetrusha I think you meant to call @Maoni0 instead of @maioni0
That being said, the doc could include a paragraph explaining that:

Workstation GC favors making applications feel smoother by using smaller GC pauses and tries to keep overall memory usage lower.
Server GC favors making the application run as fast as possible overall even if that means keeping more memory and occasionally incurring longer GC pause times.

well, I dunno who @maioni0 is and not sure why they didn't respond. and I don't normally read the docs repo so the only way that I would know about an issue in this repo that requires my attention is if someone mentions @Maoni0.
@ro-jo

You're saying it's self-tuning, immediately followed by 'set it yourself'.

"self-tuning" and "set it yourself" refer to different things.
"set if yourself" is referring to setting configs that do not require users to have GC internal knowledge. for example, the kinds of workloads  meaning "are you a process that tends to coexist with many other processes on the machine and does fairly irregular things(Workstation like workload), or a dominant/very active process that does fairly regular things (Server like workload)". it should be a simply decision on the users side.
self-tuning means for the kind of workload GC does not require you to tell it what GC mechanism to use, eg, you don't need to tell the GC things like "compact the heap when the free ratio is > X%" or "use this GC for young generation collections and this other one for old gen".
does this help?

Sorry about that, @Maoni0. I mistyped your GitHub alias.

Dear MSFT, thanks, you've been no help whatsoever.    Spare me the ignominy.
In addition to the paradoxical sentence I highlighted, you answer with: "The self tuning aspects of the GC refer to collection timing and aggressiveness which enables the GC to deliver good performance across many types of workloads." right after implying you must choose the garbage collection based on your workload.  Congratulations.
@Maoni0, in short: no it doesn't help.  You too: "... configs that do not require users to have GC internal knowledge ... " and then " it should be a simply decision on the users [sic] side.". What?
Me: Server GC is slower to initialise and consumes more resources at startup. It improves performance of longer running, resource intensive processes.
The point of my question was to highlight a poorly worded explanation and that it would be useful if there was just one type of garbage collector that could switch to server or workstation mode as part of its self-tuning.  Apart from that, what interest do I have in self-tuning?  Can I control it? Is it a unique innovation in .Net, that no other OS or framework employs? Or is it just the way all high-level memory management systems work?  DON'T ANSWER! Thanks.

@ro-jo
The GC mode can't be changed after the application has started. It's used to categorize the workload on the machine. Generally it should be Server if it's the only program running and Workstation if it's running along with other programs. The application has no idea about this and relies on you to know where and how the program is deployed, that's why it's a setting that you must configure.
Once the basic workload type is configured, the GC can then use that as a guide and will optimize itself as the program runs to ensure collection keeps pace with allocation and pauses are as small as possible. These are the internal settings that the GC will "self-tune" but you can still override if you really need to.
Does that help?

Why do the most vacuous attempts at answers end with the rhetorical phrase 'does this/that help?' ?
I can raise issue in every statement of every such answer on this page. Here are a few, I haven't got time to waste on all of them:
" it should be Server if it's the only program running": that would be never.
" Workstation if it's running along with other programs": that would be always.
"The application has no idea about the environment and relies on you to tell the program how it's deployed": Yes, it (the non-sentient piece of programming) can procure 100% of such information, to reiterate, 100%.
"Once the basic workload type is set": no such thing.  Try to consider the marginal cases that fit somewhere in between big-data/long-running service and short-lived/cooperative applications.
To be honest, the only thing I was expecting to happen was somebody would have a look at the wording and change it.  It's waffle.  But no, you seem to think I've put my projects on hold for nine months waiting for you to tell me what gcServer means - and such wonderful answers they are.  Maybe after a few years you'll make a garbage collector that's able to self-tune properly in all scenarios without hard-coded parameters, and maybe you won't.  Maybe you'll decide there's no need to mention self-tuning at all, since it's useless information.  I'm taken aback by the absurdity that's ensued nine months after raising the point.

The setting is rather clearly described in the existing documentation if you read past the single sentence you take issue with.
It comes down to how resource-intensive the GC should be, which is up to you to decide if you're running many other applications on the same server. There's obviously never only one process running on the entire machine but determining whether a server is being used for a single application or not is a rather straightforward part of app deployment.
I'm not sure how you expect the program to know these kinds of details automatically, and it's no different than how an OS provides CPU affinity and process priority for you to control resources. If you need a middle-ground then stick with Workstation, which is the default anyway.
Regardless, we can't read your mind and your responses show both confusion and disagreement as if you know better so I'm not sure what you were expecting. Your attitude is definitely not conducive to getting any help though. Good luck to you.

@manigandham It's almost as if you haven't read anything I've written.
Look at your answers and then the original question.
The only place I go for help is stackoverflow, so no big deal, it sticks to answering the question and context.

I regret commenting on this thread.
And for context, I don't work for Microsoft.

@rpetrusha thanks, I totally agree with you, my comments were unacceptable. Deleted the offending material, apologies all round to those I offended.

Thank you, @ro-jo, for your response and apology. We look forward to your continuing interest in our documentation and participation in our GitHub repos.


salam

Video
test:
https://github.com/FrameworkPython/RubikaAdBlocker/assets/149888152/7530d40a-1502-4168-8082-d1df7fd969bc

jhb


I can’t connect my k3s

Rancher Server Setup

Rancher version:
Installation option (Docker install/Helm Chart):docker

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc):

Proxy/Cert Details:

Information about the Cluster

Kubernetes version:
Cluster Type (Local/Downstream):

If downstream, what type of cluster? (Custom/Imported or specify provider for Hosted/Infrastructure Provider):

Describe the bug

To Reproduce

Result
Expected Result

Screenshots

Additional context
<!--Add any other context about the problem here.—>

This is my screenshot


7.1 audio getting converted to 5.0 rather than 5.1

I have a few recent rips from MakeMKV where the audio is getting converted to a 5.0(side) layout rather than a 5.1(side) layout.
I believe this is the first time I've encountered this. I'm sure the problem is on my end, probably something to do with how I ripped the disc, but I can't figure out what it would be.
I'm getting this in the ffmpeg log:
[auto_aresample_0 @ 0x7fc3af70d940] ch:8 chl:7.1 fmt:s32 r:48000Hz -> ch:5 chl:5.0(side) fmt:fltp r:48000Hz

Here's what it the log has to say about the input stream:
Stream #0:1(eng): Audio: flac, 48000 Hz, 7.1, s32 (24 bit) (default)
    Metadata:
      title           : Surround 7.1
      BPS-eng         : 2738422
      DURATION-eng    : 02:16:20.640000000
      NUMBER_OF_FRAMES-eng: 85215
      NUMBER_OF_BYTES-eng:<PHONE_NUMBER>
      SOURCE_ID-eng   : 001100
      _STATISTICS_WRITING_APP-eng: MakeMKV v1.17.2 darwin(x64-release)
      _STATISTICS_WRITING_DATE_UTC-eng: 2022-11-30 14:18:59
      _STATISTICS_TAGS-eng: BPS DURATION NUMBER_OF_FRAMES NUMBER_OF_BYTES SOURCE_ID

In case it matters, this is happening with a recently purchased Matrix box set. It seems to be happening both versions (HD & UHD) of each of the first three films. I already had the 4th film ripped, so I don't know if it would have been affected or not.
I've attached the ffmpeg log generated by other-transcode.
The Matrix (1999) - 1080p.mkv.log

Does that happen if you try --mp4?

gave it a try just now, and yes it seems to
I still get the following, when using --mp4:
[auto_aresample_0 @ 0x7fa9a400dc80] ch:8 chl:7.1 fmt:s32 r:48000Hz -> ch:5 chl:5.0(side) fmt:fltp r:48000Hz

I don't have that Blu-ray, but is there a digital copy or something else on it that might offer a different audio track that you could target using MakeMKV? If I remember correctly, MakeMKV doesn't allow older versions to work, so my first idea is to determine whether the issue is isolated to ffmpeg or whether MakeMKV could be involved. I'm using a Mac, so it's fairly trivial to use version 4 instead of 5 of ffmpeg. Could you also try that?

I'll have a closer look at the discs to see if there was an audio track I missed somehow.

If I remember correctly, MakeMKV doesn't allow older versions to work
Can you elaborate on this? Are you saying older versions of ffmpeg are incompatible with MakeMKV's output?

I'm currently using ffmpeg 5.1.2

MakeMKV generally disables the license key for any version other than current. I was suggesting using the ffmpeg command included in your log with ffmpeg 4 rather than ffmpeg 5. Both are available, I think. but Don has taken this on, so no more worries. :)

@donmelton that appears to have done the trick. I was on 0.11 and upgrading to 0.12 get's me the following expected output in the log (I haven't tried a full transcode):
[auto_aresample_0 @ 0x7fb43201bf40] ch:8 chl:7.1 fmt:s32 r:48000Hz -> ch:6 chl:5.1(side) fmt:fltp r:48000Hz

Thanks!

@zcutlip You are very welcome! I'm glad that was the issue.
I will close this out now.

for anyone who comes along later, re-encoding these with the corrected channel layout also fixed an issue where Plex + Apple TV down mixing for headphones caused most of the dialogue to come through the right channel 🎉

@zcutlip Good to know and thank for the update! 👍


Decide where the oracle will initially be hosted

The oracle will need to be hosted somewhere. Make a decision on where.

We will be using the olivia instance at https://h00.ooo/.


[8.x] Fix option overrides

This PR fixes an issue with option overriding introduced in https://github.com/laravel/framework/pull/40954. The problem is that by default a value of 'options' => null gets added when the default value of a HTTP call (get, post,...) is left empty.  Because array_merge_recursive was changed to array_replace_recursive, the default value removed the value of query set with withOptions.
The way I fixed it is to filter out any null value of options before sending the request. It doesn't seems you want to add null values to a request? However, if users are explicitly removing options set with withOptions by assigning null values, this will no longer work. But all tests pass right now and the docs don't note anything about that behavior so I don't think that's expected.
Related: https://github.com/laravel/framework/pull/40954 & https://github.com/laravel/framework/commit/30e341b7fe4e4d9019df42b7eff6c7dfa5ea30e5#
Fixes https://github.com/laravel/framework/issues/41382

Fixed https://github.com/laravel/framework/commit/a54f48102deea2864071e510172fe0b22a1c1d5a

Fixed https://github.com/laravel/framework/commit/a54f48102deea2864071e510172fe0b22a1c1d5a


Fix swagger-ui when "server-proxy-uri" is not set

When server-proxy-uri is not set in postgrest.conf, the swagger.json would report a Base url: <IP_ADDRESS>:3000/, which results in the generated API calls for the examples to have an extra slash i.e. the "//" in:
curl -X GET http://<IP_ADDRESS>:3000//skills -H  "accept: application/json" -H  "content-type: application/json" -H  "Range: undefined" -H  "Range-Unit: items" -H  "Prefer: undefined"

This results in a failure when trying out the API, see:

This PR removes this extraneous tailing "/" for the case where server-proxy-uri is not set.

This change is

Just reproduced locally, thanks for the fix! Want to include a line in the changelog too please?

:+1:

Actually I reverted this PR because it was actually a bug with swagger-ui, not postgrest. https://github.com/swagger-api/swagger-ui/issues/819


Optimise the logic for verifying if a table is empty

Search first

[X] I searched and no similar issues were found

Description
Originally metioned here.
Currently, to check if a table is empty, Liquibase is using JdbcExecutor.queryForInt(..), which isn't efficient for tables with a large number of rows. Furthermore, if a table contains more rows than the maximum Integer number, the run will fail with an exception.
Steps To Reproduce
Create a changeSet that includes the TableIsEmpty precondition, which will call the check function in the RowCountPrecondition class.
If the table that the changeSet is being applied to has over Integer.MAX_VALUE rows, the run will fail with an exception.
Expected/Desired Behavior
The method for checking if a table is empty should be something similar to checking if there is a first row or not, instead of counting all of them and seeing if the number of rows is 0.
Liquibase Version
All
Database Vendor & Version
All
Liquibase Integration
All
Liquibase Extensions
No response
OS and/or Infrastructure Type/Provider
No response
Additional Context
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR (Thank you!)

As this ticket generally affects performance, I'm including it in the next sprint review to see if it's something we want to address.


Support for Decimal128 field types

We should add support for Decimal128 field types similar to realm-java: https://github.com/realm/realm-java/issues/6712

KBSON currently doesn't high/low in the public API. We need to expose that to be able to send Decimal128's into Core

The needed properties have been merged and should be available in a 0.2.0-SNAPSHOT soon


CORS HTTP 405 Status Code

Describe the bug
I have created an API gateway with YARP and have a couple of downstream APIs.
The problem I am facing is that when running locally, my web client app calls the API gateway, I am getting:
https://localhost:64540/xxx/v1/yyy/zzz 405 (Method Not Allowed)
I have researched this as much as possible and have found a handful or blogs e.g. https://blog.agchapman.com/bypassing-cors-with-yarp-proxy/ but I cannot seem to rectify this CORS issue.
Output from API gateway HTTP logging:
[21:28:06 DBG] Connection id "0HN331AO8SB31" accepted.
[21:28:06 DBG] Connection id "0HN331AO8SB31" started.
[21:28:06 DBG] Connection 0HN331AO8SB31 established using the following protocol: Tls12
[21:28:06 INF] Request starting HTTP/1.1 OPTIONS https://localhost:64540/REDACTED/v1/xxx/yyy - null null
[21:28:06 DBG] 1 candidate(s) found for the request path '/REDACTED/v1/xxx/yyy'
[21:28:06 DBG] Endpoint 'REDACTED' with route pattern 'REDACTED/{**catch-all}' is valid for the request path '/REDACTED/v1/xxx/yyy'
[21:28:06 DBG] Request matched endpoint 'REDACTED'
[21:28:06 DBG] The request has an origin header: 'https://localhost:5173'.
[21:28:06 INF] CORS policy execution successful.
[21:28:06 DBG] The request is a preflight request.
[21:28:06 DBG] Connection id "0HN331AO8SB31" completed keep alive response.
[21:28:06 INF] Request finished HTTP/1.1 OPTIONS https://localhost:64540/REDACTED/v1/xxx/yyy - 204 null null 14.9071ms
[21:28:06 INF] Request starting HTTP/1.1 GET https://localhost:64540/REDACTED/v1/xxx/yyy - null null
[21:28:06 DBG] 1 candidate(s) found for the request path '/REDACTED/v1/xxx/yyy'
[21:28:06 DBG] Endpoint 'REDACTED' with route pattern 'REDACTED/{**catch-all}' is valid for the request path '/REDACTED/v1/xxx/yyy'
[21:28:06 DBG] Request matched endpoint 'REDACTED'
[21:28:06 DBG] The request has an origin header: 'https://localhost:5173'.
[21:28:06 INF] CORS policy execution successful.
[21:28:06 DBG] Static files was skipped as the request already matched an endpoint.
[21:28:06 INF] Executing endpoint 'REDACTED'
[21:28:06 INF] Proxying to https://localhost:64660/v1/xxx/yyy HTTP/2 RequestVersionOrLower
[21:28:06 INF] Received HTTP/2.0 response 405.
[21:28:06 INF] Executed endpoint 'REDACTED'
[21:28:06 DBG] Connection id "0HN331AO8SB31" completed keep alive response.
[21:28:06 INF] Request finished HTTP/1.1 GET https://localhost:64540/REDACTED/v1/xxx/yyy - 405 0 null 105.5603ms
[21:28:07 INF] Start processing HTTP request POST http://localhost:4318/v1/traces
[21:28:07 INF] Sending HTTP request POST http://localhost:4318/v1/traces
[21:28:12 INF] Received HTTP response headers after 4077.6925ms - 502
[21:28:12 INF] End processing HTTP request after 4078.9004ms - 502

I have the following code configuration implemented on the API gateway project:
Program.cs
var builder = CommonApi.CreateWebApplicationBuilder(args);

var services = builder.Services;

services.AddHttpLogging(logging =>
{
    logging.LoggingFields = HttpLoggingFields.All;
    logging.RequestBodyLogLimit = 4096;
    logging.ResponseBodyLogLimit = 4096;
});

services.AddApplicationDependencies(builder.Configuration, builder.Environment);

services.AddYarp(builder.Configuration);

services.AddCors(opt =>
{
    opt.AddPolicy("corsGatewayPolicy", policyBuilder =>
    {
        policyBuilder
            .AllowAnyOrigin()
            .AllowAnyOrigin()
            .AllowAnyHeader()
            .AllowAnyMethod();
    });
});

var app = builder.Build();

if (!app.Environment.IsDevelopment())
{
    HttpClient.DefaultProxy = new WebProxy();
}

app.MapReverseProxy();
app.UseCors("corsGatewayPolicy");

app.UseSwagger();
app.UseSwaggerUI(options =>
{
    var config = app.Services.GetRequiredService<IOptionsMonitor<ReverseProxyDocumentFilterConfig>>().CurrentValue;
    foreach (var cluster in config.Clusters)
    {
        options.SwaggerEndpoint($"/swagger/{cluster.Key}/swagger.json", cluster.Key);
    }
});

app.Run();

Extension method to add YARP to servicecollection:
public static IServiceCollection AddYarp(this IServiceCollection services, IConfiguration config)
    {
        services.AddTransient<IConfigureOptions<SwaggerGenOptions>, ConfigureSwaggerOptions>();

        var endpointSection = new EndpointSection();
        config.Bind(endpointSection);

        services
            .AddReverseProxy()
            .LoadFromMemory(YarpRouteService.Routes(endpointSection.Endpoints.BoundImplementations),
                YarpRouteService.Clusters(endpointSection.Endpoints.BoundImplementations))
            .AddSwagger(YarpRouteService.GetSwaggerConfig(endpointSection.Endpoints.BoundImplementations));

        services.Configure((Action<ReverseProxyDocumentFilterConfig>) (overriddenConfig =>
        {
            overriddenConfig.Swagger = YarpRouteService.GetSwaggerConfig(endpointSection.Endpoints.BoundImplementations).Swagger;
        }));

        return services;
    }

Custom YarpService class
public static class YarpRouteService
{
    public static List<RouteConfig> Routes(List<IEndpoint> endpoints)
    {
        var routes = new List<RouteConfig>();

        if (routes == null) throw new ArgumentNullException(nameof(routes));

        foreach (var endpoint in endpoints)
        {
            routes.Add(new RouteConfig
            {
                RouteId = $"{endpoint.RouteName}",
                ClusterId = $"{endpoint.RouteName}-module",
                Order = endpoint.Order,
                CorsPolicy = "corsGatewayPolicy",
                Match = new RouteMatch { Path = $"{endpoint.RouteName}/{{**catch-all}}" },
                Transforms = new []
                {
                    new Dictionary<string, string>
                    {
                        {"PathPattern", "{**catch-all}"}
                    },
                    new Dictionary<string, string>
                    {
                        {"ResponseHeader", "Access-Control-Allow-Origin"},
                        {"Set", "*"}
                    }
                }
            });
        }

        return routes;
    }

    public static List<ClusterConfig> Clusters(List<IEndpoint> endpoints)
    {
        var clusters = new List<ClusterConfig>();

        if (clusters == null) throw new ArgumentNullException(nameof(clusters));

        clusters.AddRange(endpoints.Select(endpoint =>
            new ClusterConfig { ClusterId = $"{endpoint.RouteName}-module",
                Destinations = new Dictionary<string, DestinationConfig>(StringComparer.OrdinalIgnoreCase) {
                {
                    "destination1", new DestinationConfig() { Address = endpoint.Uri }
                }
                }
            }));

        return clusters;
    }

    public static ReverseProxyDocumentFilterConfig GetSwaggerConfig(List<IEndpoint> endpoints)
    {
        var cluster = new Dictionary<string, ReverseProxyDocumentFilterConfig.Cluster>();

        foreach (var endpoint in endpoints)
        {
            cluster.Add($"{endpoint.RouteName}-module", new ReverseProxyDocumentFilterConfig.Cluster
            {
                Destinations = new Dictionary<string, ReverseProxyDocumentFilterConfig.Cluster.Destination>
                {
                    {
                        "destination1", new ReverseProxyDocumentFilterConfig.Cluster.Destination
                        {
                            Address = endpoint.Uri,
                            Swaggers = new[]
                            {
                                new ReverseProxyDocumentFilterConfig.Cluster.Destination.Swagger
                                {
                                    PrefixPath = $"/{endpoint.RouteName}",
                                    Paths = new[] {"/swagger/v1/swagger.json"}
                                }
                            }
                        }
                    }
                }
            });
        }
        return new ReverseProxyDocumentFilterConfig
        {
            Swagger = new ReverseProxyDocumentFilterConfig.SwaggerConfig
            {
              CommonDocumentName  = "YARP", IsCommonDocument = false
            },

            Routes = Routes(endpoints).ToDictionary(_ => _.RouteId, _ => _),

            Clusters = cluster
        };
    }

To Reproduce

Further technical details

Include the version of the packages you are using
The platform (Linux/macOS/Windows)

Why do you believe this is a CORS issue? The client did make the request to the gateway.
Judging by the logs, YARP forwarded the GET request to the backend service (https://localhost:64660/v1/xxx/yyy), and it responded with a 405. Are you sure that the client is using the correct method for this request?
I would recommend looking at the logs in the backend service to see why it decided to reject the request.

Why do you believe this is a CORS issue? The client did make the request to the gateway.
Judging by the logs, YARP forwarded the GET request to the backend service (https://localhost:64660/v1/xxx/yyy), and it responded with a 405. Are you sure that the client is using the correct method for this request? I would recommend looking at the logs in the backend service to see why it decided to reject the request.

You were completely correct, it ended up being a HTTP verb change that was missed when I replaced the legacy API gateway.
Many thanks for the second pair of eyes :)


Select manual filtering.

Pull Request
🤨 Rationale

#1866

👩💻 Implementation
This PR surfaces the necessary APIs for clients to be able to provide their own manual filtering of options in the Select. Additionally, clients can now set a loading-visible attribute on the Select to true to show a visual in the dropdown to indicate that it is in a loading state.
Not in this PR:

Examples that actually exercise the expected client workflow in which they would apply their own custom filtering for options. Instead, there are simply unit tests that validate the necessary notifications for clients to use (in addition to tests regarding the loading visual in the dropdown). The Angular and/or Blazor integration will provide a more complete example of this feature.

🧪 Testing
Unit tests. Chromatic tests.
✅ Checklist

[ ] I have updated the project documentation to reflect my changes or determined no changes are needed.

@m-akinc you mind buddying this?


umd-compatible version

You should  compile an actually umd compatible and loadable version for the browser without require so that it can be used with e.g unpkg.com

Hi @Fohlen
I think the dist version is umd, see here

@rowanwins<EMAIL_ADDRESS>tell's you that there's a bunch of require statements inside. Not quiet umd compatible

@Fohlen @rowanwins We have base library of dropzone where we used require('dropzone)

Webpack can't handle the require() function correctly...

@CinKon that is not entirely true.
@vrajroham I see. Imho you should handle it like this

use the dist folder to upload the bundle (which is currently in docs) as vue2-dropzone.min.js
use the src folder the way it is currently used
include the bundle from the docs, e.g via unpkg or raw GitHub link
use the "main" tag of npm to include the src/index.js as the npm version.

After bundling for production, the console tells me either (void 0) is not a function or n is not a function at the point where your dist/vue2Dropzone.js says require('dropzone')

@CinKon which is the fault because of the way the project's webpack.conf.js is configured. Not the fault of webpack itself.

my understanding is that webpack has nothing to do with the building of dist/vue2Dropzone.js, it's all done by rollup.
I think it may be that I need to add some additional config in the rollup commonjs plugin to make sure that the dropzone dependency gets bundled in, and also rather than using require('dropzone') it probably gets changed to an import statement
https://github.com/rowanwins/vue-dropzone/blob/master/rollup.config.js#L20
I'll do some digging because my intention was for the dropzone depency to be bundled in the dist version

using require('dropzone') it probably gets changed to an import statement

That's what I want ;)
I don't care about the building process of dist/vue2Dropzone.js. I wan't to use vue-dropzone in my project but can't because of require('dropzone').

I have the same problem:
[Vue warn]: Error in mounted hook: "Error: Module name "dropzone" has not been loaded yet for context: _. Use require([])
http://requirejs.org/docs/errors.html#notloaded"
found in
--->

I am having the same problem. Is there a resolution for how to bundle this correctly with webpack for production/deployment? Thanks.

@rowanwins Any update on this, even a work around to get webpack building to work as expected? We used vue2-dropzone because it looked like a helpful wrapper, but if it cannot be built and included with webpack it will have cost us time rather than saved us time. Thanks for any guidance. @Fohlen

Hi various folks
Thanks to some work by @bam-tbf I've now released a new version of this as v3.2.1 . If someone would like to give it a try and check that it works for them that would be great.
Thanks,
Rowan

Closing as im presuming it's resolved


Full site

In This PR.

Add React Routes.
implemented quote page
implemented home page
created navbar with routes
Linked all components and functioned the roots to work properly.

Thank You for Review I Forgot to remove the commented code. Thanks For
Pointing it out
On Thu, Jan 19, 2023, 2:21 AM Mamadou Abdoulaye Livouwane Diallo <
@.***> wrote:

@.**** requested changes on this pull request.
Changes Required ♻ Hi @Itswali https://github.com/Itswali,
[image: AlmostThereReachGIF]
https://user-images.githubusercontent.com/55918904/191301799-36d07166-0e35-4582-a231-7a2d780f7783.gif
Great work on making the changes requested by a previous reviewer 👏
You've done well implementing some of the requested changes, but there are
still some which aren't addressed yet.
(Highlights) Good Points: 👍

You documented your work professionally. ✔
GitHub flow is used correctly. ✔
All linters checks are ok. ✔

Required Changes ♻
Check the comments under the review.
Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this
PR. However, I strongly recommend you to take them into account as they can
make your code better. Some of them were simply missed by the previous
reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something
is not 100% clear.
Please, remember to tag me in your question so I can receive the
notification.
Please, do not open a new Pull Request for re-reviews. You should use the
same Pull Request submitted for the first review, either valid or invalid
unless it is requested otherwise.
As described in the Code reviews limits policy
https://microverse.zendesk.com/hc/en-us/articles/1500004088561 you have a
limited number of reviews per project (check the exact number in your
Dashboard). If you think that the code review was not fair, you can request
a second opinion using this form https://airtable.com/shrQAqnBwek5a0O0s.
In src/App.js
https://github.com/Itswali/Math-Magicians/pull/7#discussion_r1080557184:

+{ /*

MATH MAGICIANS

   HOME

   Calculator

   QUOTE

 */ }

Please remove this code that has been commented on. Leaving
commented code like this in your code base is not a good practice because
it makes it unclean.👌

—
Reply to this email directly, view it on GitHub
https://github.com/Itswali/Math-Magicians/pull/7#pullrequestreview-1260772020,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AY66QL5HPTRJXPUD2R6B52TWTBNHDANCNFSM6AAAAAAT7QFWAI
.
You are receiving this because you were mentioned.Message ID:
@.***>


[DDW-727] Implement process management for decoupled cardano-wallet and node

This PR, into a new base branch, allows the integration of the new cardano-wallet API (https://input-output-hk.github.io/cardano-wallet/api/edge/).
The new base branch will be used as V2 API endpoints are integrated. We will stay away from develop (but keep the HEADs aligned regularly), as this branch will be unstable until:

cardano-wallet is feature complete
We get access to the new Haskell node. We are using an intermediary edge node (http-bridge) to enable development, but this will never see production.

This feature embeds the cardano-wallet launcher as the process that is managed my CardanoNode. GET wallet and addresses has been implemented so that the main Daedalus homepage will render.
Fixing E2E tests are not within the scope of this initial work

@tomothespian due to the number of mocks against the new API (which involved commenting out lots of things to get this started, and hardcoding a few values), I didn't see a lot of value in getting those checks passing at this point, particularly as this is moving into a different base branch for the time being.
I had a conversation about this with @nikolaglumac and @daniloprates on Monday. It is up to you guys though. @daniloprates have you had time to play / look at this?
I'm going hands off on development of this (as my role was just to get the new dependencies into the dev process to enable you guys to start developing against the new API). But if you collectively agree you want the checks passing, I'll do that

@daniloprates @tomothespian I've updated this branch to the most recent version of cardano-wallet which has a few new endpoints.
While I was there I just fixed the lint and flow errors. More things commented out, but it might be easier for you guys that way

@Sam-Jeston after pulling the latest changes from cardano-sl@develop and daedalus@feature/ddw-727-new-wallet-and-node-process-manager I'm not able to connect to cardano anymore 👇

@daniloprates I just tested it on both my mac and Ubuntu installs. No issues. Please provide more info. What is the output of cardano-node.log. Anything strange in the terminal etc etc.
But I imagine this will be easier to just pair on


Add .gitignore and uninstall section to Makefile

Added .gitignore and uninstall section to Makefile

I don't think this is necessary?
The install rule is only there for package managers an end user is not meant to run that.
Even then, You forgot to include the removal of the configuration and stylesheet from data dir.
As for .gitignore, Again i don't feel like that is necessary since make clean before submitting changes is required anyway.

I don't think this is necessary?
The install rule is only there for package managers an end user is not meant to run that.
Even then, You forgot to include the removal of the configuration and stylesheet from data dir.
As for .gitignore, Again i don't feel like that is necessary since make clean before submitting changes is required anyway.

Oops forgot about the configuration files.
Adding a .gitignore eliminates the need to run make clean before submitting changes 😄
Anyway it's upto you

Eh sure why not, LGTM :+1:


Show combat feedback errors

I guess that oUF_CombatFeedback needs to be update too, it throws errors when on.
showCombatFeedback = true,

Should be fixed in #180. It was having trouble getting the font height from the Blizzard code so I just hard coded the size in.


fix: onchain wallet deleted for charge

Steps:

go to satspay extension
open New Charge dialog
select onchain wallet (do NOT click Create Charge yet)
in new tab:

go to watchonly extension
delete the account that the charge was pointing to

back in the satspay  tab click Create Charge

Actual Result

a technical error is shown: NoneType' object has no attribute 'address'
there is an error stacktrace

Expected Result

it fails with a user-friendly message: Wallet '{onchainwallet}' can no longer be accessed.
no stacktrace

Codecov Report

Merging #1235 (50addd7) into main (d0ed93a) will decrease coverage by 0.12%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #1235      +/-   ##
==========================================
- Coverage   49.85%   49.72%   -0.13%
==========================================
  Files         255      255
  Lines       14748    14754       +6
==========================================
- Hits         7352     7336      -16
- Misses       7396     7418      +22

Impacted Files
Coverage Δ

lnbits/extensions/satspay/crud.py
28.39% <0.00%> (-0.72%)
:arrow_down:

lnbits/extensions/satspay/views_api.py
38.55% <0.00%> (-1.96%)
:arrow_down:

lnbits/db.py
81.20% <0.00%> (-10.07%)
:arrow_down:

lnbits/commands.py
73.61% <0.00%> (-1.39%)
:arrow_down:

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


file choose option for choose in playlist section

I'm submitting a ...  (check one with "x")

[x] feature request

Actual Behaviour

Nothing happens when clicked on choose button
Expected Behaviour

opens a window to select the music and plays directly after choosing music

its done


Add option to manually specify file extension inside file names v2

Version 2 of my previous PR #1266, sorry about that. Fixed the remaining issue, here's the original description:
Digikam recently changed the behaviour of XMP sidecar files and now they need to match the full file-name of the associated image, ie. what used to be:
image.jpg
image.xmp

is now:
image.jpg
image.jpg.xmp

This PR adds the option to use %image_ext% in the file naming process so I can change my filename config to say image.%image_ext%.
%image_ext% does not replace the existing extension, adding it to anything other than info files will just duplicate the image's file extension, but who knows maybe someone wants that somewhere, like making directories based on extension I don't know.
The new addition is a bit of a wild one, I really wasn't sure on the best way to handle Ugoiras.
The problem I ran into is that, at the time we create metadata files, we don't yet know what ugoira re-encodes we're going to create, and ugoira's are a bit unique in that the original file-type of a ZIP is not usually retained, at least by me anyway.
After a lot of umming and ahhing I decided to add a big If Statement and just rely on the config file options to create additional sidecars, but only where someone has enabled writeImageXMPPerImage. This config option currently creates additional XMP files for every image in an album, I think the method of creating multiple re-encodes of Ugoiras is similar enough behaviour that I've integrated it into this option.
Not enabling writeImageXMPPerImage will make it write just a single xmp sidecar still. If you use the new %image_ext% in your info filename format then it will be called .zip.xmp, not sure who wants that but maybe someone.
Finally, if you enable writeImageXMPPerImage, have Ugoira re-encodings enabled, but don't specify the %image_ext% in the Info filename format, then it will technically just write multiple XMP files all with the same filename, overwriting each one. I thought about adding some logic to avoid this to prevent the unnecessary writes but it's basically instant anyway so I didn't bother.
Please let me know what you think.
Screenshots
Before:

After:

:eyes:


fix(comp:form): form item label height error when label is

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines
[x] Tests for the changes have been added/updated or not needed
[x] Docs and demo have been added/updated or not needed

What is the current behavior?
表单项 同时配置了 colonless和空格label，高度不正常
What is the new behavior?
修复以上问题
Other information

空格 label ? 为什么会有这种场景？
不想轻易去动 display 属性了。


nodelet manager dies

roslaunch realsense2_camera rs_camera.launch fails as follows:
[FATAL] [1537376861.893268506]: Failed to load nodelet '/camera/realsense2_cameraof typerealsense2_camera/RealSenseNodeFactoryto managerrealsense2_camera_manager'
[camera/realsense2_camera_manager-1] process has died [pid 17026, exit code -11, cmd /opt/ros/kinetic/lib/nodelet/nodelet manager __name:=realsense2_camera_manager __log:=/home/glory/.ros/log/25de489e-bc28-11e8-82e3-28f10e3d3536/camera-realsense2_camera_manager-1.log].
log file: /home/glory/.ros/log/25de489e-bc28-11e8-82e3-28f10e3d3536/camera-realsense2_camera_manager-1*.log
[camera/realsense2_camera-2] process has died [pid 17027, exit code 255, cmd /opt/ros/kinetic/lib/nodelet/nodelet load realsense2_camera/RealSenseNodeFactory realsense2_camera_manager __name:=realsense2_camera __log:=/home/glory/.ros/log/25de489e-bc28-11e8-82e3-28f10e3d3536/camera-realsense2_camera-2.log].
log file: /home/glory/.ros/log/25de489e-bc28-11e8-82e3-28f10e3d3536/camera-realsense2_camera-2*.log
Below is the full stdout and logs are attached
roslaunch realsense2_camera rs_camera.launch
... logging to /home/glory/.ros/log/25de489e-bc28-11e8-82e3-28f10e3d3536/roslaunch-glory-ubuntu-17009.log
Checking log directory for disk usage. This may take awhile.
Press Ctrl-C to interrupt
Done checking log file disk usage. Usage is <1GB.
started roslaunch server http://glory-ubuntu:39433/
SUMMARY
PARAMETERS

/camera/realsense2_camera/accel_fps: 1000
/camera/realsense2_camera/accel_optical_frame_id: camera_accel_opti...
/camera/realsense2_camera/align_depth: False
/camera/realsense2_camera/aligned_depth_to_color_frame_id: camera_aligned_de...
/camera/realsense2_camera/aligned_depth_to_fisheye_frame_id: camera_aligned_de...
/camera/realsense2_camera/aligned_depth_to_infra1_frame_id: camera_aligned_de...
/camera/realsense2_camera/aligned_depth_to_infra2_frame_id: camera_aligned_de...
/camera/realsense2_camera/color_fps: 30
/camera/realsense2_camera/color_height: 480
/camera/realsense2_camera/color_optical_frame_id: camera_color_opti...
/camera/realsense2_camera/color_width: 640
/camera/realsense2_camera/depth_fps: 30
/camera/realsense2_camera/depth_height: 480
/camera/realsense2_camera/depth_optical_frame_id: camera_depth_opti...
/camera/realsense2_camera/depth_width: 640
/camera/realsense2_camera/enable_color: True
/camera/realsense2_camera/enable_depth: True
/camera/realsense2_camera/enable_fisheye: True
/camera/realsense2_camera/enable_imu: True
/camera/realsense2_camera/enable_infra1: True
/camera/realsense2_camera/enable_infra2: True
/camera/realsense2_camera/enable_pointcloud: False
/camera/realsense2_camera/enable_sync: False
/camera/realsense2_camera/fisheye_fps: 30
/camera/realsense2_camera/fisheye_height: 480
/camera/realsense2_camera/fisheye_optical_frame_id: camera_fisheye_op...
/camera/realsense2_camera/fisheye_width: 640
/camera/realsense2_camera/gyro_fps: 1000
/camera/realsense2_camera/gyro_optical_frame_id: camera_gyro_optic...
/camera/realsense2_camera/infra1_fps: 30
/camera/realsense2_camera/infra1_height: 480
/camera/realsense2_camera/infra1_optical_frame_id: camera_infra1_opt...
/camera/realsense2_camera/infra1_width: 640
/camera/realsense2_camera/infra2_fps: 30
/camera/realsense2_camera/infra2_height: 480
/camera/realsense2_camera/infra2_optical_frame_id: camera_infra2_opt...
/camera/realsense2_camera/infra2_width: 640
/camera/realsense2_camera/json_file_path:
/camera/realsense2_camera/rosbag_filename:
/camera/realsense2_camera/serial_no:
/rosdistro: kinetic
/rosversion: 1.12.13

NODES
/camera/
realsense2_camera (nodelet/nodelet)
realsense2_camera_manager (nodelet/nodelet)
ROS_MASTER_URI=http://localhost:11311
process[camera/realsense2_camera_manager-1]: started with pid [17026]
process[camera/realsense2_camera-2]: started with pid [17027]
[ INFO] [1537376860.690312630]: Initializing nodelet with 8 worker threads.
[ INFO] [1537376860.716838794]: RealSense ROS v2.0.3
[ INFO] [1537376860.716867181]: Running with LibRealSense v2.16.0
[ INFO] [1537376861.007192077]: getParameters...
[ INFO] [1537376861.069992569]: setupDevice...
[ INFO] [1537376861.070032505]: JSON file is not provided
[ INFO] [1537376861.070071676]: ROS Node Namespace: camera
[ INFO] [1537376861.070094571]: Device Name: Intel RealSense D415
[ INFO] [1537376861.070111748]: Device Serial No:<PHONE_NUMBER>83
[ INFO] [1537376861.070131971]: Device FW version: <IP_ADDRESS>
[ INFO] [1537376861.070156868]: Device Product ID: 0x0AD3
[ INFO] [1537376861.070175994]: Enable PointCloud: Off
[ INFO] [1537376861.070196408]: Align Depth: Off
[ INFO] [1537376861.070219614]: Sync Mode: Off
[ INFO] [1537376861.070270998]: Device Sensors:
[ INFO] [1537376861.070315639]: Stereo Module was found.
[ INFO] [1537376861.070344149]: RGB Camera was found.
[ INFO] [1537376861.070396163]: (Fisheye, 0) sensor isn't supported by current device! -- Skipping...
[ INFO] [1537376861.070422510]: (Gyro, 0) sensor isn't supported by current device! -- Skipping...
[ INFO] [1537376861.070446271]: (Accel, 0) sensor isn't supported by current device! -- Skipping...
[ INFO] [1537376861.070478484]: setupPublishers...
[ INFO] [1537376861.074595257]: Expected frequency for depth = 30.00000
[ INFO] [1537376861.116156771]: Expected frequency for infra1 = 30.00000
[ INFO] [1537376861.139312376]: Expected frequency for infra2 = 30.00000
[ INFO] [1537376861.159249360]: Expected frequency for color = 30.00000
[ INFO] [1537376861.181719950]: setupStreams...
[ INFO] [1537376861.183606450]: depth stream is enabled - width: 640, height: 480, fps: 30
[ INFO] [1537376861.183856157]: infra1 stream is enabled - width: 640, height: 480, fps: 30
[ INFO] [1537376861.184088482]: infra2 stream is enabled - width: 640, height: 480, fps: 30
[ INFO] [1537376861.186312171]: color stream is enabled - width: 640, height: 480, fps: 30
19/09 19:07:41,193 ERROR [139828003849984] (backend-v4l2.cpp:1002) Video frame arrived in idle mode.
19/09 19:07:41,193 ERROR [139828003849984] (backend-v4l2.cpp:1002) Video frame arrived in idle mode.
[ INFO] [1537376861.198501841]: publishStaticTransforms...
[ INFO] [1537376861.198773568]: RealSense Node Is Up!
[FATAL] [1537376861.893268506]: Failed to load nodelet '/camera/realsense2_cameraof typerealsense2_camera/RealSenseNodeFactoryto managerrealsense2_camera_manager'
[camera/realsense2_camera_manager-1] process has died [pid 17026, exit code -11, cmd /opt/ros/kinetic/lib/nodelet/nodelet manager __name:=realsense2_camera_manager __log:=/home/glory/.ros/log/25de489e-bc28-11e8-82e3-28f10e3d3536/camera-realsense2_camera_manager-1.log].
log file: /home/glory/.ros/log/25de489e-bc28-11e8-82e3-28f10e3d3536/camera-realsense2_camera_manager-1*.log
[camera/realsense2_camera-2] process has died [pid 17027, exit code 255, cmd /opt/ros/kinetic/lib/nodelet/nodelet load realsense2_camera/RealSenseNodeFactory realsense2_camera_manager __name:=realsense2_camera __log:=/home/glory/.ros/log/25de489e-bc28-11e8-82e3-28f10e3d3536/camera-realsense2_camera-2.log].
log file: /home/glory/.ros/log/25de489e-bc28-11e8-82e3-28f10e3d3536/camera-realsense2_camera-2*.log
all processes on machine have died, roslaunch will exit
shutting down processing monitor...
... shutting down processing monitor complete
done
roslaunch-glory-ubuntu-17009.log
camera-realsense2_camera-2-stdout.log

the issue disappears on a fresh installation of the system


Lap time sequence

Hello,
Would like to change the lap time sequence in ascending order for a better experience and ease of identifying the sequence of laps.
Please assign it to me @avinash201199

Hello.
Changed the sequence in which lap times were being displayed in ascending order for improved comprehension, comparison, clarity and error prevention.
Please accept my pull request

https://github.com/avinash201199/stopwatch/assets/92590913/84ba8178-edc5-407d-81ec-2598fda5394d


Adding missing incident deletion cascades

We should also add ondelete="CASCADE" to these secondary tables. You'll need to expand the file to see the selection. The rest lgtm.

We should also add ondelete="CASCADE" to these secondary tables. You'll need to expand the file to see the selection. The rest lgtm.

The link just points to my PR, not sure what I need to add.

Expand src/dispatch/workflow/models.py and you'll see the selection.


Proposal: generic uint_to_bytes

Issue
An alternative to #1746 int_to_bytes and compute_shuffled_index part.

[Typing presentation bug] Status-quo int_to_bytes expects uint64 as the input. But actually we were passing Python int to it in some cases.
Part of https://github.com/ethereum/eth2.0-specs/pull/1746. Open a new PR to discuss this phase 0 change proposal for decreasing ambiguity. IMHO this PR is neater than #1746's casting.

Proposed solution

To make it more explicit, rename int_to_bytes to uint_to_bytes and enforce the input of uint_to_bytes to be SSZ uint type.
Encode the uint object by its length in bytes. i.e., encode uint8 to 1-byte output and uint64 to 8-byte output.
To avoid exposing SSZ APIs, describe uint_to_bytes in prose instead of in code. Similar to hash_tree_root.
To make it more explicit, rename bytes_to_int to bytes_to_uint64.

It's a refactoring change of the current spec and it should not change the consensus logic.

Please review the type-infer branch (based on #1746), it handles many of the more "obvious" type inferences, to keep the spec clean. For compute_shuffled_index it doesn't involve as many casts anymore, by following a simple set of rules to keep the output size the same as the input size. And it fixes + catches new things.


Log named blob mysql db queries for debugging

Codecov Report
Patch coverage has no change and project coverage change: -59.25 :warning:

Comparison is base (fd9943f) 73.27% compared to head (c4b0434) 14.02%.

Additional details and impacted files
@@              Coverage Diff              @@
##             master    #2486       +/-   ##
=============================================
- Coverage     73.27%   14.02%   -59.25%
+ Complexity    11089     2362     -8727
=============================================
  Files           783      783
  Lines         62217    62440      +223
  Branches       7624     7644       +20
=============================================
- Hits          45591     8759    -36832
- Misses        14126    53168    +39042
+ Partials       2500      513     -1987

Impacted Files
Coverage Δ

.../java/com/github/ambry/named/MySqlNamedBlobDb.java
0.00% <0.00%> (-89.66%)
:arrow_down:

... and 628 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.