Paste prepends to current message rather than inserts in place

Bug description:

When pasting content from the clipboard to an in-progress Hangouts message, the content is prepended to the message rather than inserted at the cursor location.
Steps to reproduce:

Start a Hangouts message to a recipient.
Type something like "Check out this amazing link: "
Copy a URL from the browser and paste it into your message.
The link is inserted at the beginning of the message, not at the cursor location.

Expected behavior

Pasted content should be inserted in context with the message at the current cursor location.

Screenshots

Environment:

Operating System:

MacOS 11.2 (Big Sur)

Ferdi Version:

Server:

Ferdi <IP_ADDRESS>6

Debug information:

Yeah, I'm glad that someone else found this. It is pretty annoying. I also verified that copy-pasting works fine on the website https://hangouts.google.com/ via Chrome.

I've been sending pastes as a separate post. Wish I didn't have to but
it works.
On 3/30/21 8:41 PM, mastercko wrote:

Does anyone have a workaround for this maybe?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/getferdi/ferdi/issues/1199#issuecomment-810669251,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOL5V2OZPAH3IANNZL7BRDDTGJVSFANCNFSM4XSAHJCA.

Yeah, I guess have to unlearn my usual keystroke of "type type type ctrl-v enter" (urggh)

Hey guys, I was unable to reproduce this in Google Hangouts with the latest nightly: https://github.com/getferdi/nightlies/releases/tag/v5.6.1-nightly.33
There were some copy and paste related code changes recently, please let me know if it works for you too.

I switched over to using the Google Hangouts Chat service instead of classic Hangouts, so I can't quickly test this anymore.

@steve-manes - are you able to revert back on whether you are still seeing this as an issue?

I've been using nightly builds and haven't seen the problem in them.
On 8/19/21 4:53 AM, Vijay Raghavan Aravamudhan wrote:

@steve-manes https://github.com/steve-manes - are you able to revert
back on whether you are still seeing this as an issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/getferdi/ferdi/issues/1199#issuecomment-901733110,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOL5V2OB2XOZLC37WMRC3WTT5TBCJANCNFSM4XSAHJCA.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email.

Thanks @steve-manes , @mastercko - closing this ticket based on your confirmations.


Added disable_javascript_update

Added disable_javascript_update to prevent lazy imports from polluting non-jupyter consoles with "<IPython.core.display.Javascript object>". Addresses issue #30.

Any chance this is being followed on?

Sorry for the (very) late reply. I will merge this asap and make a new release. Thanks @gamis for following up on this!


cant be compiled in nodejs 4.1.1

Hi dude, I have tested several times between nodejs v0.12.7 and v4.1.1. It cant be compiled in v4.1.1. so please help.

search for similar issues before posting new ones: https://github.com/EyalAr/lwip/pull/182


[Event Request] Table 7326 - Whse. Worksheet Line - OnBeforePutAwayCreate

Hi AL-Team,
can you please add the following Publisher with Handled Pattern to PutAwayCreate?
procedure PutAwayCreate(var WhsePutAwayWkshLine: Record "Whse. Worksheet Line")
    var
        CreatePutAwayFromWhseSource: Report "Whse.-Source - Create Document";
        Handled: Boolean;
    begin
        // NEW >>>>
        OnBeforePutAwayCreate(Rec, WhsePutAwayWkshLine, Handled);
        if Handled then
            exit;
        // NEW <<<<
        CreatePutAwayFromWhseSource.SetWhseWkshLine(WhsePutAwayWkshLine);
        CreatePutAwayFromWhseSource.RunModal;
        CreatePutAwayFromWhseSource.GetResultMessage(1);
        Clear(CreatePutAwayFromWhseSource);
    end;

    [IntegrationEvent(false, false)]
    local procedure OnBeforePutAwayCreate(Rec: Record "Whse. Worksheet Line";var WhsePutAwayWkshLine: Record "Whse. Worksheet Line"; var Handled: Boolean)
    begin
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Support Key of ProductTypes

http://dev.commercetools.com/http-api-projects-productTypes.html#producttype
Please support the key field in Producttype draft.

Duplicate of #52


gitfs: ls: fix return value if detail=True

fsspec requires it to return a list of infos, not a dict.

For the record: tested with dvc, so this is safe to release 🙂


math to speech improvements (add as you go)

f raised to the negative one power -> f inverse

[ 'Subtract   5 x ', ' from each side.' ] should not be parsed this way for 1 expression ( {text} + {expression} ) is the current format. Conditional if it is the last expression any following text should be added onto same sentence.

mph -> miles per hour

want units to be in same sentence as the values ['we have 60' , 'mph in the 40...'] ->
['we have 60 mph' ,  ' in the 40...']


link dependencies on RHS of file editor to the file in which they are defined

Requested by Luiz @ Liv Up

This was fixed in a previous PR: https://github.com/dataform-co/dataform-co/pull/1928


website

what's the word on it does it need recreated?

Work is currently being done on RiceDB, which is soon to enter beta testing. Once that's done, we'll work on Logos.


Quick file find, or jump to file in explorer by typing filename

VSCode Version:
OS Version:

We are literally working with thousands of files. Currently the user is limited to a mouse scrolling through a large folder structure to find the file and manually selecting the file.
We kindly request that finding files in the Explorer be easier. The Windows Explorer is a parallel example.
For example, provide a box where you can start typing in the file name and the Explorer is limited to showing files that match. Please support wild cards.
For example, allow typing in the Explorer to jump to a file.

Duplicates #10026


client don't try reconnect

hi,
I run ping pong example, first run server and then run client, all is ok.

I hope client will try reconnect if network is disconnect.

Remove reconnect=5 from the example.

Hi, I guess you're using Tcp or FramedTcp right? The way it works is how TCP works.
TCP does know about reconnections. Once the channel is closed by one side (i.e, when you kill the server), the connection is considered closed for the other endpoint. If you want a reconnection mechanism, you should implement this on top of Tcp by yourself. i.e, trying to connect again after some incremental period or similar.


Add ChangeDisplaySettingsEx to Safe Functions

Do you mean EnumDisplaySettings?

You are right. I would prefer to use this one in my code because I am trying to replicate the calls of another application as close as possible. This way also I don't have to decide on the flags.
I guess it depends on the policy of this repo. Looking at the current readme, it seems like we want to cover the whole API? If we only want to implement calls that actually make sense, then I agree with you!

I really never thought about the policy. Maybe is just to be useful to people, and in that sense, it would be good to implement this API, since it's useful to you. Anyway feel free to request anything you need, you're welcome.


AMD Ryzen 7 8845HS w/ Radeon 780M

What is the issue?
use ollama-windows-amd64.7z, it report 0xc0000005 error.
thx to this issue, now it can work.
https://github.com/likelovewant/ROCmLibs-for-gfx1103-AMD780M-APU/issues/8
copy all *.dll to runner/rocm6.1
copy rocblas folder to runner/rocm6.1
now it's a bit practical。 with Qwen2 7B， 17 tokens/s， and in LM studio is 10 tokens/s
thanks for you work
OS
Windows
GPU
AMD
CPU
AMD
Ollama version
0.30.0

OllamaSetup.exe  should be better .otherwise , https://github.com/likelovewant/ROCmLibs-for-gfx1103-AMD780M-APU/issues/8 is an alternative .


Docs: Add PodIPPoolSelector Special Selector to BGP CP

When https://github.com/cilium/cilium/pull/28314 merges, two special selectors will be introduced and should be documented.

/assign


[BUG] Contributor over time module in README is out of service

Description:
The contributor over time module is out of service, suggest remove it from README.
Location: https://github.com/hyperf/hyperf?tab=readme-ov-file#code-contributor-over-time
Click and open link: https://git-contributor.com/?chart=contributorOverTime&repo=hyperf/hyperf , see upper right corner's error message.

Steps To Reproduce:

Open https://github.com/hyperf/hyperf?tab=readme-ov-file#code-contributor-over-time
Click 'Code contributor over time'
See no response in the page provided by api7.ai

Also see

https://github.com/api7/contributor-graph/issues/231 first brought up the issue
https://github.com/api7/contributor-graph/issues/232 validates the issue
https://github.com/pingcap/docs-cn/issues/19086 & https://github.com/pingcap/docs-dm/issues/933 how the other repos deal with the issue, one chooses to use an alternative, and the other deletes the section.


Instagram Shortcode broken getJSON failure

The Instagram Shortcode isn't working, getJSON error even though the addresses are valid and able to be fetched with my browser
Probably related to https://github.com/gohugoio/hugo/issues/5643#issue-403564640
I'll try looking into it and writing a build test. (I'm a noob)

I have merged #5643 -- and I cannot reproduce this particular issue -- the instagram examples on the docs site works fine (I have cleared the cache).

I am having similar issue with tweet shortcode.
Building sites … ERROR 2019/02/01 10:12:24 "/Users/pgrzybek/Sites/pawelgrzybek.com/content/posts/2018-06-30-top-picks-2018-june.md:15:1": failed to render shortcode "tweet": failed to process shortcode: template: _internal/shortcodes/tweet.html:7:13: executing "_internal/shortcodes/tweet.html" at <getJSON $url>: error calling getJSON: failed to get getJSON resource "https://api.twitter.com/1/statuses/oembed.json?id=1003623284829638659&dnt=false": Get https://api.twitter.com/1/statuses/oembed.json?id=1003623284829638659&dnt=false: x509: certificate signed by unknown authority

It happened to me after update from v50 to v53.

I can confirm that version. 0.54 fixed my issues.

Thank you a lot for rapid action @bep


fix(#914): Issue crash app when update environment variable

Fix issue crash app when set environment variable
Fixes https://github.com/usebruno/bruno/issues/914
Contribution Checklist:

[x] The pull request only addresses one issue or adds one feature.
[x] The pull request does not introduce any breaking changes
[x] I have added screenshots or gifs to help explain the change if applicable.
[x] I have read the contribution guidelines.
[x] Create an issue and link to the pull request.

Before (issue)
https://github.com/usebruno/bruno/assets/140475149/501f75dd-6eed-418c-a2be-53e7e64aa9b8
After fixed
https://github.com/usebruno/bruno/assets/140475149/c9a25a73-4dec-4e78-b1ef-d378ef55be3d

This is great! Than you for fixing this @nguyenbavinh-decathlon !


Generate qiskit-ibm-runtime 0.17.0 API docs

This PR adds the API docs for qiskit-ibm-runtime 0.17.0 and converts version 0.16 into a historical one.
Commands used:
npm run make-historical -- -p qiskit-ibm-runtime
npm run gen-api -- -p qiskit-ibm-runtime -v 0.17.0 -a https://github.com/Qiskit/qiskit-ibm-runtime/suites/18863019852/artifacts/1100724937

The CI run used to build the current version:
https://github.com/Qiskit/qiskit-ibm-runtime/actions/runs/7136304937

FYI @kt474

Thanks! Once this is in, how do deployments work? Currently still seeing 0.15.0 as the latest version in https://docs.quantum.ibm.com/api/qiskit-ibm-runtime

Once this is in, how do deployments work?

We sync the docs in an internal repository. Once that is merged, it deploys automatically. I'll DM you the README for how we do that.


Rename Singularity→Apptainer

Rename Singularity to Apptainer and switch from SingularityCE to Apptainer package

Added table for explaining the differences/names
Updated wording accordingly
Fixed small mistakes

looks great, great work @mira-miracoli that table is incredibly useful!


Implement Draco 3D compression for meshes

https://google.github.io/draco/

@cesarecaoduro and @d3ssy Do we need this?

I would like to keep this when we get to serialisation/compression of meshes.

I think this is out of scope for G-Shark. We're getting into the computer graphics and rendering realm here...


SDK version update

Hey guys, how are you?
I've spotted that the lib is using AWS SDK 2.21.15. Looks like there exists already 2.27.5 and there's a BC break introduced somewhere in between (a protected method setOverrides is now final). Which makes it a bit uncomfortable to use the lib if an new SDK version is also present.
Thus the question, do you guys have any plans updating the SDK version?
Thanks.

Think we can close this

Thanks!


Adds evaluation metric based on base metrics

There are four different types of evaluation metrics needed:

Cost function for clustering changelists itself
Baseline metrics (e.g. number of splits, change size)
Metric of what a good changelist looks like
Metrics of what a good changeset looks like

More metrics:

histogram/bot plot of code size
histogram/box plot of split count
language breakdown (e.g. as a slice)
anomaly detection
graph complexity metrics
maybe some distance metrics for symbols
diff metrics (delta, add, rm)


cli mode log feature issues

There are a few well known log feature bugs in the cli mode that will need to be addresses.

[x] log folder and file need to be created if it is not there.
[ ] commands will be appended to log file and never deleted.
[ ] when commands are too large the buttons in the right disappear even in large screen:

Normal look is:

The first item was resolved by https://github.com/bitshares/gwallet/commit/5af64e86d2a319813cc22cb2adfd0b16fdca22ea
The binary will now run stand alone in ubuntu.


FlowRouterMeta is not a function

I'm having an issue:
When i just install FlowRouterMeta
import { FlowRouterMeta, FlowRouterTitle } from 'meteor/ostrio:flow-router-meta';
console.log(FlowRouterMeta);
new FlowRouterMeta(FlowRouter);
new FlowRouterTitle(FlowRouter

Got the following message ...

I20180613-16:31:04.849(2)? undefined
W20180613-16:31:06.419(2)? (STDERR) /home/joel/.meteor/packages/meteor-tool/.1.5.2_2.irtbc9++os.linux.x86_64+web.browser+web.cordova/mt-os.linux.x86_64/dev_bundle/server-lib/node_modules/fibers/future.js:280
W20180613-16:31:06.420(2)? (STDERR) 						throw(ex);
W20180613-16:31:06.420(2)? (STDERR) 						^
W20180613-16:31:06.421(2)? (STDERR)
W20180613-16:31:06.421(2)? (STDERR) TypeError: FlowRouterMeta is not a function
W20180613-16:31:06.421(2)? (STDERR)     at meteorInstall.route.zzz_router.js (route/zzz_router.js:39:1)
W20180613-16:31:06.422(2)? (STDERR)     at fileEvaluate (packages/modules-runtime.js:333:9)
W20180613-16:31:06.422(2)? (STDERR)     at require (packages/modules-runtime.js:228:16)
W20180613-16:31:06.423(2)? (STDERR)     at /home/joel/project/c10po/beta/.meteor/local/build/programs/server/app/app.js:26001:1
W20180613-16:31:06.424(2)? (STDERR)     at /home/joel/project/c10po/beta/.meteor/local/build/programs/server/boot.js:339:34
W20180613-16:31:06.424(2)? (STDERR)     at Array.forEach (native)
W20180613-16:31:06.425(2)? (STDERR)     at Function._.each._.forEach (/home/joel/.meteor/packages/meteor-tool/.1.5.2_2.irtbc9++os.linux.x86_64+web.browser+web.cordova/mt-os.linux.x86_64/dev_bundle/server-lib/node_modules/underscore/underscore.js:79:11)
W20180613-16:31:06.426(2)? (STDERR)     at /home/joel/project/c10po/beta/.meteor/local/build/programs/server/boot.js:158:5
W20180613-16:31:06.426(2)? (STDERR)     at /home/joel/project/c10po/beta/.meteor/local/build/programs/server/boot.js:388:5
W20180613-16:31:06.427(2)? (STDERR)     at Function.run (/home/joel/project/c10po/beta/.meteor/local/build/programs/server/profile.js:510:12)
W20180613-16:31:06.428(2)? (STDERR)     at /home/joel/project/c10po/beta/.meteor/local/build/programs/server/boot.js:387:11

I'm on ubuntu 16.04 and Meteor <IP_ADDRESS>

Hey @jalaux ,
{ FlowRouterMeta, FlowRouterTitle } are Client only libs.
And I see you're getting errors at server.
You can simply wrap it into Meteor.isClient condition

Great.
thanks

Please, support this project by:

Star on GitHub
Star on Atmosphere
Check other packages we have made


USB C PD stack no callback for MSG_NOT_SUPPORTED_RECEIVED policy notify

Describe the bug
I am developing an application on a custom PCB having USB C PD capabilities with the STM32G473 MCU. I used the code provide in the sample.
I am testing the USB C PD application with a PD source which has 4 PDOs (5V/30A, 9V/3A, 15V/3A and 20V/3.25A)
At 15V and 20V the source send a Vendor Defined Message PD_DATA_VENDOR_DEF which I was able to debug and find out from the below code in usbc_pe_snk_states.c, line 424
...
		/* Extended Message Request */
		if (header.extended) {
			extended_message_not_supported(dev);
			return;
		} else if (header.number_of_data_objects > 0) {
			/* Data Messages */
			switch (header.message_type) {
			case PD_DATA_SOURCE_CAP:
				pe_set_state(dev, PE_SNK_EVALUATE_CAPABILITY);
				break;
			default:
				LOG_INF("MSG TYPE = %d", header.message_type);
				pe_set_state(dev, PE_SEND_NOT_SUPPORTED);
			}
			return;
...

The USB C stack throws log messages  like <inf> usbc_stack: PE_Not_Supported but the Policy Notify callback doesn't get called, so the application does not have a chance to handle the error.
Impact
The device's charger not functioning and the application cannot show an error, so has a high impact
Logs and console output
[00001779] <inf> usbc_stack: Unattached.SNK
[00001784] <inf> usbc_stack: AttachWait.SNK
[00001992] <inf> usbc_stack: Attached.SNK
[00001997] <inf> usbc_stack: PE_SNK_Startup
[00001997] <inf> app_ucpd: NOT_PD_CONNECTED
[00001997] <inf> usbc_stack: PRL_INIT
[00001997] <inf> usbc_stack: PRL_HR_Wait_for_Request
[00001997] <inf> usbc_stack: PRL_Tx_PHY_Layer_Reset
[00001997] <inf> usbc_stack: PRL_Tx_Wait_for_Message_Request
[00002002] <inf> usbc_stack: PE_SNK_Discovery
[00002007] <inf> usbc_stack: PE_SNK_Wait_For_Capabilities
[00002010] <inf> usbc_stack: RECV 41a1/4
[00002010] <inf> usbc_stack:    [0]0a11912c
[00002010] <inf> usbc_stack:    [1]0012d12c
[00002010] <inf> usbc_stack:    [2]0014b12c
[00002010] <inf> usbc_stack:    [3]00164145
[00002015] <inf> usbc_stack: PE_SNK_Evaluate_Capability
[00002015] <inf> app_ucpd: SOURCE_CAPABILITIES_RECEIVED
[00002015] <inf> usbc_stack: PE_SNK_Select_Capability
[00002015] <inf> app_ucpd: snk_mv = 20000, snk_ma = 3000, pos = 4
[00002017] <inf> usbc_stack: PRL_Tx_Wait_for_PHY_response
[00002021] <inf> usbc_stack: RECV 03a3/0
[00002021] <inf> usbc_stack: PRL_Tx_Wait_for_Message_Request
[00002021] <inf> app_ucpd: PWR 3A0
[00002026] <inf> usbc_stack: PE_SNK_Transition_Sink
[00002260] <inf> usbc_stack: RECV 05a6/0
[00002265] <inf> app_ucpd: TRANSITION_PS
[00002265] <inf> usbc_stack: PE_SNK_Ready
[00002322] <inf> usbc_stack: RECV 17af/1
[00002322] <inf> usbc_stack:    [0]ff00a002
[00002327] <inf> usbc_stack: MSG TYPE = 15
[00002327] <inf> usbc_stack: PE_Not_Supported
[00002328] <inf> usbc_stack: PRL_Tx_Wait_for_PHY_response
[00002333] <inf> usbc_stack: RECV 03a1/0
[00002333] <inf> usbc_stack: PRL_Tx_Wait_for_Message_Request
[00002339] <inf> usbc_stack: PE_SNK_Ready
[00003996] <inf> app_ucpd: Source Caps:
[00003996] <inf> app_ucpd: PDO 0:
[00003996] <inf> app_ucpd:      Type:              FIXED
[00003996] <inf> app_ucpd:      Current:           3000
[00003996] <inf> app_ucpd:      Voltage:           5000
[00003996] <inf> app_ucpd:      Peak Current:      1
[00003996] <inf> app_ucpd:      Uchunked Support:  0
[00003996] <inf> app_ucpd:      Dual Role Data:    1
[00003996] <inf> app_ucpd:      USB Comms:         0
[00003996] <inf> app_ucpd:      Unconstrained Pwr: 1
[00003996] <inf> app_ucpd:      USB Suspend:       0
[00003996] <inf> app_ucpd:      Dual Role Power:   0
[00003999] <inf> usb_cdc_acm: Device suspended
[00004046] <inf> app_ucpd: PDO 1:
[00004046] <inf> app_ucpd:      Type:              FIXED
[00004046] <inf> app_ucpd:      Current:           3000
[00004046] <inf> app_ucpd:      Voltage:           9000
[00004046] <inf> app_ucpd:      Peak Current:      1
[00004047] <inf> app_ucpd:      Uchunked Support:  0
[00004047] <inf> app_ucpd:      Dual Role Data:    0
[00004047] <inf> app_ucpd:      USB Comms:         0
[00004047] <inf> app_ucpd:      Unconstrained Pwr: 0
[00004047] <inf> app_ucpd:      USB Suspend:       0
[00004047] <inf> app_ucpd:      Dual Role Power:   0
[00004097] <inf> app_ucpd: PDO 2:
[00004097] <inf> app_ucpd:      Type:              FIXED
[00004097] <inf> app_ucpd:      Current:           3000
[00004097] <inf> app_ucpd:      Voltage:           15000
[00004097] <inf> app_ucpd:      Peak Current:      1
[00004097] <inf> app_ucpd:      Uchunked Support:  0
[00004097] <inf> app_ucpd:      Dual Role Data:    0
[00004097] <inf> app_ucpd:      USB Comms:         0
[00004097] <inf> app_ucpd:      Unconstrained Pwr: 0
[00004097] <inf> app_ucpd:      USB Suspend:       0
[00004097] <inf> app_ucpd:      Dual Role Power:   0
[00004147] <inf> app_ucpd: PDO 3:
[00004147] <inf> app_ucpd:      Type:              FIXED
[00004147] <inf> app_ucpd:      Current:           3250
[00004147] <inf> app_ucpd:      Voltage:           20000
[00004147] <inf> app_ucpd:      Peak Current:      1
[00004147] <inf> app_ucpd:      Uchunked Support:  0
[00004147] <inf> app_ucpd:      Dual Role Data:    0
[00004147] <inf> app_ucpd:      USB Comms:         0
[00004147] <inf> app_ucpd:      Unconstrained Pwr: 0
[00004147] <inf> app_ucpd:      USB Suspend:       0
[00004147] <inf> app_ucpd:      Dual Role Power:   0

Environment:

OS: Linux
Toolchain Zephyr SDK zephyr-v3.2.0-3518-g7b5780e007
Hardware: custom hardware using STM32G473

@deyro Sorry, I missed this. I'm looking into it now.

@sambhurst no problem, thank you!

@deyro Sorry, I missed this. I'm looking into it now.


fix: CENTER as default value for vertical alignment in Table Widget

Description
Set "CENTER" as default value for Vertical Alignment property instead of invalid value "LEFT"
Fixes #14885
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Manual

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

/ok-to-test sha=1a73948

@aswathkk do we need to merge this in

@somangshu Yes. Tests were passed. But, the ui-test-result status is not getting reported.


Issue with responsive images and DPR on mobile

When responsive images are used with srcset, it looks as though images that are the right size are highlighted in blue (too small) and images that are too big are highlighted in green (OK) when emulating a mobile device with a high DPR in Devtools. I think this might be to do with using naturalWidth for the image element, rather than looking to naturalWidth of currentSrc - demo page here: https://www.alexrp.co.uk/nl/testresponse.html

Thanks a lot of the bug report. Will try and get this sorted for the 1.6.0 release


[ rook addon ] no persistent volumes available for this claim and no storage class is set

Hi !
I'm trying to deploy my first Helm Chart on the HA cluster installed with this script. I've enable the rook addon ( require 2 times to install correctly as mentioned in another ticket ), but a call to « kubectl -n kube-system get storageClasses » confirm it's correctly installed and set as default storage.
On « kubectl get events --sort-by='{.lastTimestamp}' » I can see the exact error: FailedBinding / persistentvolume-controller / no persistent volumes available for this claim and no storage class is set.
I've tried with storageClass = "", storageClass = "default", storageClass = "standard", and storageClass = "rook-block".
Could you please help me to fix that?
Regards,

Could you give the output of describe for PV and PVC?

I had a similar issue with default openebs installation. The default storageclass was not set.
I could fix it by:
kubectl patch storageclass openebs-standard -p '{"metadata": {"annotations":{"storageclass.kubernetes.io/is-default-class":"true"}}}'

see: https://kubernetes.io/docs/tasks/administer-cluster/change-default-storage-class/

@pierreozoux The describe of pv,pvc is empty under namespace kube-sytem and there is a pvc under the default namespace in Pending named pioneering-guppy-docker-registry (but this seems related to docker-registry addon installed on the same time and not yet used)
BTW, do this mean the rook-block class need to be set as default each time for each namespace? Because for my own chart I have created a new namespace to install it.
@quorak Thanks for the idea, but the patch doesn't works « not patched », this is because it's already set by default.
In fact the most strange thing is this doesn't works even if I set storageClass: "rook-block" :-/

Ok, I've just found an interesting point, this seems related to the namespace.
In the default namespace, by setting "rook-block", this works.
But if I set storageClass = "", still in the default namespace, this doesn't works ( no persistent volumes available for this claim and no storage class is set ).

what if you don't set storage class at all?

@pierreozoux Same error, if set to empty string or if I remove the annotation line :(

May be the rook addon installation need to specify the authorization to be used on every namespaces?

Just figured out I've used a old annotation way ( metadata.annotations.volume.beta.kubernetes ). I've re-tested all cases with the correct way ( spec.storageClassName ), but this still doesn't works ( same error ).
More strange, in the stable way, in default namespace with "rook-block" this doesn't works anymore.

I would try to uninstall rook to re-install it but this give me this error:
2018/03/23 11:45:12 removing addon rook
2018/03/23 11:45:13 Error from server (NotFound): pools.rook.io "replicapool" not found

2018/03/23 11:45:13 > kubectl delete -n rook pool replicapool
2018/03/23 11:45:13
2018/03/23 11:45:13
2018/03/23 11:45:13 run failed:Process exited with status 1

So, even if the « get storageclass » show me « rook-block (default) », may be there is a missing piece of rook somewhere ( "replicapool not found" ) ?

Yes, sometimes, I noticed that if you change config in rook, the better is to just recreate the cluster.
Try with a branc new cluster, I'm sure it will work.

@pierreozoux Ok, cluster deleted, now re-creating... thanks

Fixed after cluster re-creation!
Thanks for your help! :)


Create a Short (YouTube, IG Reel, or TikTok) About Your Goose Experience! 🦢

Create a Short Video About Your Goose Experience! 🦢
Goose is your on-machine developer agent, automating engineering tasks seamlessly within your IDE or terminal. Guided by you, it can intelligently assess your project's needs, generate code modifications, and implement changes on its own.
Learn more about Goose in our documentation.
Once your content is merged, it will be featured on our Community Content site!
 These should be under 60 seconds and in vertical format (9:16 aspect ratio).
🤝 How You Can Contribute
Share things like:

A quick demo of Goose in action
A specific toolkit or feature showcase
A "day in the life" with Goose
Before/after of how Goose improved your workflow
Quick tips and tricks
A specific problem Goose helped you solve

🔑 Tasks and Acceptance Criteria

[ ] Create a short video about your Goose experience
[ ] Host your content on your preferred platform
[ ] Add your content to the content/content.json file
[ ] Create a pull request with your changes

Your contribution in content.json must include:
{
    "type": "short",
    "thumbnail": "./img/your-thumbnail.png",
    "title": "Your Content Title",
    "link": "https://linktoyourcontent.com",
    "contributor": "Your Name"
}

🌟 Resources

Goose Documentation
Goose repo
How to use toolkits
How to create toolkits

Full episodes on Goose to understand more

Getting Started

Comment ".take" on this issue to get assigned
Fork the repository and create a new branch
Create your content
Submit a PR with your content.json addition
Wait for review and address any feedback

Questions?
If you have any questions or need clarification, please comment on this issue!
Happy sharing! 🦢

.take


File sync not working for android versions below Q

Hey hey @lrq3000 ! Could I maybe ask you to have another look at this? I am try to get this to work on my phone which runs on android 9. Unfortunately there the permissions dialog is never opened and without it it doesn't seem to be able to write the file.

Hey @johannesjo, by file upload do you mean filesync? If yes, then that's normal, because scoped storage is only implemented in android 10+ so we open the folder picker dialog only on these, but for lower versions it uses the old permission system. I did not test with Android 9, if it doesn't work, then we can just change the condition to lower:
if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O)
To:
if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.P)
For Android 9.
I'll try various versions of android to ensure if it all works as expected for older versions, I'll post here my results.
2 févr. 2023 20:46:20 Johannes Millan @.***>:

Hey hey @lrq3000[https://github.com/lrq3000] ! Could I maybe ask you to have another look at this? I am try to get this to work on my phone which runs on android 9. Unfortunately there is never a permissions dialog opened and without it it doesn't seem to be able to write the file.
—
Reply to this email directly, view it on GitHub[https://github.com/johannesjo/super-productivity-android/issues/36], or unsubscribe[https://github.com/notifications/unsubscribe-auth/AAIRFXXNOFB3OXAJFC2LBB3WVQFIXANCNFSM6AAAAAAUPOAB3A].
You are receiving this because you were mentioned.[Image de pistage][https://github.com/notifications/beacon/AAIRFXWQ6YBDWAFLLKSYYUTWVQFIXA5CNFSM6AAAAAAUPOAB3CWGG33NNVSW45C7OR4XAZNFJFZXG5LFVJRW63LNMVXHIX3JMTHF3ABYSI.gif]

Yes sorry! File sync of course! I tried changing the if condition, but that didn't do the trick. Would be great if you could have another look at this!

Ok i see then maybe a regression was introduced somewhere, I'll have a look, thanks for the heads up.
3 févr. 2023 13:34:50 Johannes Millan @.***>:

Yes sorry! File sync of course! I tried changing the if condition, but that didn't do the trick. Would be great if you could have another look at this!
—
Reply to this email directly, view it on GitHub[https://github.com/johannesjo/super-productivity-android/issues/36#issuecomment-1415813111], or unsubscribe[https://github.com/notifications/unsubscribe-auth/AAIRFXQPERVWVDIT24P7G5TWVT3OVANCNFSM6AAAAAAUPOAB3A].
You are receiving this because you were mentioned.[Image de pistage][https://github.com/notifications/beacon/AAIRFXUATLTRUV3LRIVLP6DWVT3OVA5CNFSM6AAAAAAUPOAB3CWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTSUMOL7O.gif]

@johannesjo I get a fatal error on just opening SP on Android 9: https://github.com/johannesjo/super-productivity/issues/2451
Do you get the same?


on AEM Cloud, In Brightcove Video Player (HTML5) component dialog, the account and players dropdown option values doesn't show up again to author, after the first time edit and save.

1.It is used as is. Just for note: Only on AEM cloud version we found this issue. in AEM 6.5 it does work.
2. No only one account configured.
Just a quick Observation: The player drop-down options doesn't show up at its expected position , however on every click,  it kind of blinks up on the top left corner of the screen, it's so fast we can't grab a screenshot of it.


Password management

We need to provide some basic tooling surrounding password management.  We have a decent process in place for seeding the initial credentials, but the process for managing these credentials long-term is still a little hairy.
I think one simple thing we could do is provide an fly postgres changepassword <user> <password> command that streamlines the change password process for users and ensures it's performed in a safe manor.
One big question is whether or not it makes sense to provide everyone superuser privileges, by default?
When a new PG cluster is created, users are required to specify their credentials via secrets.  Once the provision has completed, users are supplied with a postgres user with superuser privileges.  The problem is that once users connect, they have the ability to delete users and change passwords via the ALTER USER command.   If this is used to change the flypgadmin, repluser or the postgres user credentials, it will create a disconnect between the supplied secrets and the actual credentials used by PG.  This could inadvertently bring down their cluster.
We have quite a few different routes we could potentially take here:

We provide superuser privileges and accept the possibility that people inadvertently break their cluster.
We provide a normal user by default, which has privileges to create databases, roles, login, etc. and document how to access the superuser if required for their use-case.
We restrict PG clusters to normal users and superusers are reserved for internal administration.
?

I would love peoples thoughts on this.

For just the SU password, I'm ok with ignoring this until it becomes a problem. People can poke around and f'up their clusters, that's totally fine and I bet it's pretty rare. We should definitely give everyone admin permissions! Reduced permissions don't make a lot of sense (they can SSH to their Postgres and do what they want already)
More generally, though, giving people easy ways to manage DB users, create read only users, is useful.
This gets complicated because of linked apps. It would be nice to be able to rotate DB permissions for linked apps, but we'd need to add a new user, update the linked app secret, let it deploy, then remove the old user to make it work.

This gets complicated because of linked apps. It would be nice to be able to rotate DB permissions for linked apps, but we'd need to add a new user, update the linked app secret, let it deploy, then remove the old user to make it work.

Yeah, this does get complicated.  I think the general procedure is fairly straight forward, but monitoring the entire process and knowing when it's actually safe to yank the original credentials seems less straight forward.

We provide a normal user by default, which has privileges to create databases, roles, login, etc. and document how to access the superuser if required for their use-case.

If that user account can create roles etc, what's the difference to being superuser?

We restrict PG clusters to normal users and superusers are reserved for internal administration.

I for one create roles and update their passwords as part of my schema migration mechanism. You can't take away that power, or your PG app becomes less useful.
Perhaps a thing to consider instead is, what's the route to recovery if and when someone does change one of the passwords used by fly infrastructure. Can the user obtain the "supposed value" and set it back to that (are the secrets visible anywhere)? If you don't want to make them visible, you could detect a "bad password" error from Postgres and guide the user to an ALTER ROLE statement that has the SCRAM form of the password...
Now, should flyctl postgres connect default to superuser? Maybe not. But is there a solid answer what account it should use? Should each person running flyctl get their own PG role by default? And also, beyond the existence of flyctl postgres connect --user, is this really Fly's problem?

This issue is quite old and hasn't really been thought about since. :D.  Some password rotation tooling could still be useful, but it's certainly not high on our priority list.

If that user account can create roles etc, what's the difference to being superuser?

In short, superuser bypasses all permission checks.

I for one create roles and update their passwords as part of my schema migration mechanism. You can't take away that power, or your PG app becomes less useful.

We have no intention of forcing a particular process here. There are three different credentials outlined in the README that need to be handled in a specific manor, but you're free to manage other roles/passwords however you see fit.

Perhaps a thing to consider instead is, what's the route to recovery if and when someone does change one of the passwords used by fly infrastructure. Can the user obtain the "supposed value" and set it back to that (are the secrets visible anywhere)? If you don't want to make them visible, you could detect a "bad password" error from Postgres and guide the user to an ALTER ROLE statement that has the SCRAM form of the password...

It's a foot gun, but it hasn't actually come up much.  The credentials outlined within the README can be viewed within the PG environment and are pretty easy to correct.  The secret and the associated password within PG just need to match.

Now, should flyctl postgres connect default to superuser? Maybe not. But is there a solid answer what account it should use? Should each person running flyctl get their own PG role by default? And also, beyond the existence of flyctl postgres connect --user, is this really Fly's problem?

We started out having a non-superuser as the default, but it ended up causing too much friction for some use-cases.  Users just want things to work and not spend too much time fiddling with permissions.


periph: add a PeripheralInfo class for metadata.

As suggested in https://github.com/nmigen/nmigen-soc/pull/11#issuecomment-602738198.
This PR proposes a PeripheralInfo class to gather the resources of a peripheral, with the aim of advertising them to the BSP generator. A ConfigDict class is also provided to collect constants.

I'll need some time to dig into this--please ping me on IRC in a week or two if I don't review this quickly enough.

Updates, based on feedback from the July 6th IRC meeting:

Remove the constant dictionary. Its implementation will be the subject of another PR.
Use an event.Source corresponding to the IRQ line of the peripheral, instead of an event map describing local events. This gives the BSP generator a mean to recover both:

the IRQ number of the peripheral (by calling .index(periph_info.irq) on the interrupt controller event map)
the numbers of local events, in order to clear their pending bits. (by querying periph_info.irq.event_map)

Also, use getter-only properties instead of attributes, for added safety.


UI label for kubernetes connector name

Change Cluster Name to Name on creation

Environment

Walrus version: v0.5.0-rc3

Environment

Walrus version: main a803edf588fc3dc8024ebdb4745b184b3b40bba7
UI version: dev-186c2e0

Test Result: pass


gleam: init at 0.5.0

Motivation for this change
Close #71432.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @NobbZ

This change is

still LGTM


VirtualBox VM in the "Saved" State Cannot be Started

So somehow my VM was in the "Saved" state. Maybe my computer went to sleep and VirtualBox put it to sleep? In any case, the docker-machine stop|start|restart <name> commands would just hang. I managed to get the docker-machine CLI working again by manually opening the VirtualBox GUI and "Restoring" the "Saved" state of the VM, and then stopping it.

I have a similar issue. My work around for this is:

Opened my Virtual Box Manager
I can see the list of my Machines in diff state
Started my 'SAVED' machine (right clicking on the machine and click save)
Maybe this will help in your end.

I experienced the same issue. I'm also using virtual box as my driver. Same workaround using Virtual Box and started the machine in Saved state.

This is a known issue and a fix has already been created. See #750

This should be fixed by https://github.com/docker/machine/pull/751 cc @ehazlett to close

nice!

Just ran into this issue. I'm on OSX 10.10.3 using the latest Docker Engine stable release and the latest Docker Machine (installed both yesterday).
I created a Virtualbox VM, used it a bit yesterday, left it running and closed my MacBook.
Today when I opened it up, Docker Machine reported the VM in "Saved" state:
$ docker-machine ls
NAME   ACTIVE   DRIVER       STATE   URL   SWARM
dev             virtualbox   Saved

I tried to start, stop, restart, and kill the VM, to no avail:
$ docker-machine start dev
exit status 1

$ docker-machine ls
NAME   ACTIVE   DRIVER       STATE   URL   SWARM
dev             virtualbox   Saved

$ docker-machine stop dev
exit status 1

$ docker-machine ls
NAME   ACTIVE   DRIVER       STATE   URL   SWARM
dev             virtualbox   Saved

$ docker-machine restart dev
exit status 1

$ docker-machine kill dev
exit status 1

I was able to rm the VM and re-create it. If I run into this issue again, is there anything I can do to help you guys to reproduce this issue?

Argh, this should work.  I'll try and see if I can reproduce and/or fix.

same issue here. Just happened today.

I can duplicate this and want to get a fix in ASAP.

https://github.com/docker/machine/pull/1487


docker compose run does not output any container log

Describe the bug
Hello, we are using a docker compose run command in a particular step as part of our CI process to run specs against a Dockerized Ruby on Rails application.
Since a few days, we are unable to see the output from Docker in this step.
To Reproduce
I created this sample repo which mimics parts of our pipeline. It is enough to reproduce the issue visible in step "Run Specs with docker compose run".
The relevant workflow part :
- name: Run Specs with docker compose run
  run: |
    # GENERATES NO OUTPUT :
    docker compose -p ci_test -f docker-compose.test.yml run --name ci_test_output ci_test sh run-tests.sh

    # RETRIEVES LOGS FROM PREVIOUS COMMAND AND OUTPUTS CORRECTLY :
    docker logs ci_test_output
- name: Run Specs with docker run
  run: |
    # OUTPUTS LOGS CORRECTLY :
    docker run ci_test:latest sh run-tests.sh

The script which is being run by docker compose run (nothing special):
#!/bin/sh

echo "******* Loading database schema *******"
bundle exec rake db:schema:load
echo "***************************************"

echo "******* Running pending database migrations *******"
bundle exec rake db:migrate
echo "***************************************"

echo "******* Running tests *******"
bundle exec rails test -v

Expected behavior
The output of the docker compose run command should be displayed in the job log output.
Runner Version and Platform
Runner 2.286.1 on Ubuntu 20.04.3 LTS
I also noticed that affected runner has the following docker compose version : Docker Compose (Docker Inc., 2.2.3+azure-1)
Whereas an unaffected job had the following version : Docker Compose (Docker Inc., 2.2.2+azure-1)
What's not working?

Note that we can get the logs by using docker run instead of docker compose run:

But we need to use docker compose run for our real world workflow due to the complexity of the app.
Thanks for your help!

Moving this to https://github.com/actions/virtual-environments and reopening; this looks like an issue in a new version of docker compose in a virtual environment.

@ethomson @ste26054 we will take a look

@ste26054
I've converted your pipeline into the one:

on:
  push:
  workflow_dispatch:

jobs:
  test:
    runs-on: self-hosted
    steps:
      - name: Check Out Repo
        uses: actions/checkout@v2
      - name: Run Specs with docker compose run
        run: |
          sudo rm -frv /usr/local/lib/docker
          sudo mkdir -p /usr/local/lib/docker/cli-plugins
          sudo apt-get remove moby-compose || true
          sudo docker system prune -a -f
          sudo  curl -SL https://github.com/docker/compose/releases/download/v2.2.3/docker-compose-linux-x86_64 -o /usr/local/lib/docker/cli-plugins/docker-compose
          sudo chmod +x /usr/local/lib/docker/cli-plugins/docker-compose
          sudo docker compose version
          echo "***** GOING TO RUN DOCKER COMPOSE RUN. LOGS ARE EXPECTED BELOW THIS LINE *****"
          sudo docker compose -p ci_test -f docker-compose.test.yml run --name ci_test_output ci_test sh run-tests.sh
      - name: Run Specs with docker run
        run: |
          echo "GOING TO DOCKER RUN"
          sudo docker run ci_test:latest sh run-tests.sh
      - name: Run Specs with docker-compose run
        run: |
          sudo docker compose -f docker-compose.test.yml rm -f
          echo "***** GOING TO RUN DOCKER-COMPOSE RUN. LOGS ARE EXPECTED BELOW THIS LINE *****"
          sudo docker-compose -p ci_test -f docker-compose.test.yml run --name ci_test_output ci_test sh run-tests.sh```

          (on a self-hosted runner), it seems the problem exists in moby docker compose 2.2.3, but once I am installing the separate cli plugin the problem is gone. Might be @cpuguy83 has better knowledge on what's going on.

Also checking around with folks at Docker.

update: I was wring, 2.2.3 is not working at all (in therms of logs output), 2.2.2 downgrade helps as I said above :)

@ndeloof thanks a lot for your input!
Given that this is not a bug per se I am going to close this issue right now, if anyone has questions left feel free to reach us out again.

Thanks for the investigation, I have added the option --no-TTY to docker compose run and it is now working as expected!

--no-TTY causes keyword highlighting to disappear, anyway to avoid this?


Not valid DKIM

Hello,
I'm using your package. It works, the DKIM signature added, but both Outlook and mail-tester.com return the error: "Not valid DKIM". Can you please check? Have you tried to check the result DKIM with any services?
Thanks!

Hello, foremost, which version are you using? I don't have an environment on Laravel 9 but @mrjose, who provided the Laravel 9 compatibility version said, that it works for him.
You need to make sure that the "selector" is correct, maybe that is your error? Follow the readme and then it should work.

Hi @hostbrook just ran a test with mail-tester.com and Gmail, using laravel-dkim 2.0.0 and Laravel 9 - DKIM signature reporting as valid. No problems so far.
I would recommend double checking you have your keys setup correctly, including the selector.

Hello again. I followed your package instructions and have double-checked all keys, it seems to me everything is correct:
The file .env:
DKIM_DOMAIN=hostbrook.com
DKIM_SELECTOR=220409
<EMAIL_ADDRESS>
The file private_key.txt is in storage/app/dkim/, UTF-8 LF
DNS is added correctly (TXT record with name 220409._domainkey ) - you can check on dmarcanalyzer.com/dkim/dkim-checker
But still have an issue.
Gmail Original Message result:
Authentication-Results: mx.google.com;
spf=pass (google.com: domain of<EMAIL_ADDRESS>designates <IP_ADDRESS> as permitted sender<EMAIL_ADDRESS>dkim=neutral (body hash did not verify<EMAIL_ADDRESS>header.s=220409 header.b=e7+gyrXD
Received-SPF: pass (google.com: domain of<EMAIL_ADDRESS>designates <IP_ADDRESS> as permitted sender) client-ip=<IP_ADDRESS>;

...

DKIM-Signature: v=1; q=dns/txt; a=rsa-sha256; bh=2s4zP7onDND0eE9/9K0cFNDTHGf9IpHPF6XzZ2HvIx4=; d=hostbrook.com; h=From: To: Subject: MIME-Version: Date: Message-ID<EMAIL_ADDRESS>s=220409; t=1649563196; c=relaxed/relaxed; b=e7+gyrXDx9S4cLgOTytsC4j5eVWlLHHZYItiH3d2jMOjBVpJBOF8WNTBibh6WqaNmcXu+M9S2 tLDYf7c5xfSowS/3hll2s34fAKU6EXzogtZSd4LV6ph2l1RZRCE2BDJFW6kP+BAAFGv5wVGfB rGIbD9uYZYu+teaSde9l55ApA/uYpkc9EtNNQ03gRNsdcp1XhDLUI8rZDAP6N8RSRHx9v7ed7 QdMIm5Gax4b2XkM5Ko9GI/GNO+3NqadnC+w+TJHE74Z4fqJGycRBd4phY1zfqKZrGZ6aavq29 2vTVq5fDKivK+fEC3LaREFLmqnBDLfcfoDp7wrQe2LJINx/6zw==
Content-Type: multipart/alternative; boundary=YN6_aMme

Spam-test from app.mailgenius.com/spam-test/:
(-2 points) Your email is not properly authenticated with DKIM.
Solution:
Update your DNS records with the correct public DKIM key.
The public key, represented by the p= tag, on your DKIM DNS record is unverified or revoked.

Existing DKIM Signature Header:
Version: 1
Domain: hostbrook.com
Algorithm: rsa-sha256
Algorithm_type: ; d=hostbrook.com
Selector: 220409

Spam-test from mailtrap.io:
DKIM_SIGNED Message has a DKIM or DK signature, not necessarily valid
DKIM_INVALID DKIM or DK signature exists, but is not valid

What did I do wrong?
Laravel v9.7.0
Your package, the latest version: 2.0.0
Thank you!

I could offer you paid support as the package does its job correctly and without debugging I have no chance to know where the error is. If you accept my offer, you can contact me on my website: https://www.simonschaufelberger.de/de/kontakt.html

I do not know if you receive my emails via the form provided. Just in case if not, this is what I found. Your package works well and DKIM is valid only in the case if email built either with HTML or PLAIN TEXT template. In my case, all my emails have both HTML and PLAIN TEXT, because this is a good practice. For example, how I build a message:
class ContactForm extends Mailable
{
    ...

    public function build()
    {
        $this->subject($this->mailData['subject'])
            ->view('mail.contact-form_html')
            ->text('mail.contact-form_plain');
        return $this;
    }
}

So, if you use both templates to build the email (what is the right way) you get an invalid DKIM signature. Actually, SWIFT mailer at Laravel 8 didn't have this issue.
Next, I have dug this issue with Symfony mailer and found the possible way on how to fix it, here:
https://github.com/symfony/symfony/issues/39354#issuecomment-894379296
Please double-check it.

Ok then this is really a bug.

I do not think this is a bug in your package, probably this is the issue at Symfony Mailer. But curious if you can bypass it in your package somehow. Anyways, please double-check before closing the issue. Thanks!

@fjahn I would prefer a pull request instead of some code snippets here.

Sorry for that! The code is untested, I just thought that since I dug the source code already, it would be a waste not to suggest these hotfixes for others. I would not recommend doing this, this is just if you desparately need to fix DKIM right now and don't care about compatibility.
Having said that, I will submit a PR nontheless, but I want to point out that this is a severe breaking change that modifies existing behavior of Symphony Mailer.

Fixed in Symfony


Deploy to Azure Automation link not working

Hi all,
Am trying to deploy this module to Azure Automation - however it loads the portal, the blade shows available automation account - and then the original blade displays the broken cloud icon.
I'll attach a screenshot.

https://awsaug01.blob.core.windows.net/temp/boken.png

Closing as it is affecting all module in the PowerShell Gallery


Load the wrong snippets source

I find I couldn't load some snippes in systemverilog.snippets.
I delete  systemverilog.snippets file, But it can load verilog.snippets when I edit *.sv files

Seems the issue is not relevant to cmp, but it's on how snippets are loaded. Could you please check at https://github.com/SirVer/ultisnips/blob/master/doc/UltiSnips.txt#L535


Fix random test failures

Using assert_selector makes Capybara wait for the element to appear. This should fix the not-so-random test failures

:clap:


Should break in eager mode.

If all the tests are passing, we're not testing the eager mode.

@aakashkumarnain @ssaishruthi
You are owners of some files modified in this pull request.
Would you kindly review the changes whenever you have the time to?
Thank you very much.

If all the tests are passing, we're not testing the eager mode.

Sorry, I didn't get this. We aren't testing eager mode?

It's work in progress. I'm trying to get rid of run_all_in_graph_and_eager_mode. At the same time, I noticed that we were not testing some functions in eager mode. This pull request should not be merged. It's just to prove that if a bug happens only in eager mode, we won't know about it.


can the current code run with your results?

can the current code run with your results?

Hi! I don't fully understand your question. Are you getting any errors?
In any case, I removed the cuda specifications in requirements.txt.
It might take a while before the dataset is downloaded and unpacked. I will try to add a progress bar.

thanks for your repid reply. I can run your code now.


participant pages are bugged and half done

Participant pages are only done 50%. Data is missing, some is wrong and some data is nonsensical.
I would think the best approach is to take a parse and then run them by fx Tim Hirsch or Andrea

missing data added and Tim Hirsch have signed them off with a few logged issues


Authentication required each time Code Tunnel is started.

Does this issue occur when all extensions are disabled?: No
I set up new Ubuntu server, and set up code tunnel with code tunnel service install.
But, it is offline.
I used code tunnel service log to see logs.
● code-tunnel.service - Visual Studio Code Tunnel
     Loaded: loaded (/home/xxxxx/.config/systemd/user/code-tunnel.service; enabled; vendor preset: enabled)
     Active: active (running) since Sat 2023-08-05 15:08:24 JST; 2s ago
   Main PID: 6145 (code-tunnel)
      Tasks: 8 (limit: 18834)
     Memory: 4.3M
        CPU: 51ms
     CGroup<EMAIL_ADDRESS>             └─6145 /usr/share/code/bin/code-tunnel --verbose --cli-data-dir /home/tadashi/.vscode/cli tunnel service internal-run
 8月 05 15:07:24 xxxxxx-N100DC-ITX code-tunnel[1970]: Open this link in your browser https://vscode.dev/tunnel/tadashi-n100dc-itx
 8月 05 15:07:24 xxxxxx-N100DC-ITX code-tunnel[1970]: [2023-08-05 15:07:24] trace Found token in keyring
 8月 05 15:08:24 xxxxxx-N100DC-ITX systemd[659]: Stopping Visual Studio Code Tunnel...
 8月 05 15:08:24 xxxxxx-N100DC-ITX systemd[659]: Stopped Visual Studio Code Tunnel.
 8月 05 15:08:24 xxxxxx-N100DC-ITX systemd[659]: Started Visual Studio Code Tunnel.
 8月 05 15:08:24 xxxxxx-N100DC-ITX code-tunnel[6145]: [2023-08-05 15:08:24] debug Starting tunnel with `code /usr/share/code/bin/code-tunnel --verbose --cli-data-dir /home/xxxxxx/.vscode/cli tunnel service internal-run`
 8月 05 15:08:24 xxxxxx-N100DC-ITX code-tunnel[6145]: [2023-08-05 15:08:24] debug starting as new singleton
 8月 05 15:08:24 xxxxxx-N100DC-ITX code-tunnel[6145]: [2023-08-05 15:08:24] trace No token in keyring, getting a new one
 8月 05 15:08:24 xxxxxx-N100DC-ITX code-tunnel[6145]: [2023-08-05 15:08:24] debug [reqwest::connect] starting new connection: https://github.com/
 8月 05 15:08:24 xxxxxx-N100DC-ITX code-tunnel[6145]: To grant access to the server, please log into https://github.com/login/device and use code XXXX-XXXX
 8月 05 15:08:29 xxxxxx-N100DC-ITX code-tunnel[6145]: [2023-08-05 15:08:29] trace refresh poll failed, retrying: Error getting authorization: authorization_pending The authorization request is still pending.
 8月 05 15:08:35 xxxxxx-N100DC-ITX code-tunnel[6145]: [2023-08-05 15:08:35] trace refresh poll failed, retrying: Error getting authorization: authorization_pending The authorization request is still pending.

It seems that every time the service is started it requires authentication.
I already did this the first time I started the service.
And even if I authenticate each time, every time I restart the service or restart the machine itself, it asks me to authenticate again.
How can I avoid having to authenticate every time? I don't know how to solve this problem.

VS Code Version: 1.81.0 x64
OS Version: Ubuntu 22.04.3 LTS

Steps to Reproduce:

code tunnel and authenticate machine
Ctrl + C
code tunnel service install
It isn't online because authorization_pending

I just stumbled across this as I was looking for a fix to a problem I'm having but earlier in my journey I looked at a similar issue which might help you

Please run code-tunnel --version and share the output

Please run code-tunnel --version and share the output

tadashi@tadashi-N100DC-ITX:~$ code-tunnel --version
code-tunnel: コマンドが見つかりません

code-tunnel: command not found
tadashi@tadashi-N100DC-ITX:~$ code --version
1.81.1
6c3e3dba23e8fadc360aed75ce363ba185c49794
x64

tadashi@tadashi-N100DC-ITX:~$ code tunnel --version
error: unexpected argument '--version' found

  tip: a similar argument exists: '--verbose'

Usage: code-tunnel tunnel --verbose

For more information, try '--help'.

With reference to https://github.com/microsoft/vscode-remote-release/issues/8628 and https://github.com/coder/code-server/issues/1861, I run echo "export VSCODE_CLI_USE_FILE_KEYCHAIN=1" >> ~/.bashrc and sudo loginctl enable-linger $USER, but trace refresh poll failed, retrying: Error getting authorization: authorization_pending The authorization request is still pending. still keep occurring.
tadashi@tadashi-N100DC-ITX:~$ code tunnel service log
● code-tunnel.service - Visual Studio Code Tunnel
     Loaded: loaded (/home/tadashi/.config/systemd/user/code-tunnel.service; enabled; vendor preset: enabled)
     Active: active (running) since Sat 2023-08-26 03:50:02 JST; 10s ago
   Main PID: 13630 (code-tunnel)
      Tasks: 5 (limit: 18833)
     Memory: 7.0M
        CPU: 57ms
     CGroup<EMAIL_ADDRESS>             └─13630 /usr/share/code/bin/code-tunnel --verbose --cli-data-dir /home/tadashi/.vscode/cli tunnel service internal-run
 8月 26 03:45:02 tadashi-N100DC-ITX systemd[1182]: Stopping Visual Studio Code Tunnel...
 8月 26 03:45:02 tadashi-N100DC-ITX systemd[1182]: Stopped Visual Studio Code Tunnel.
-- Boot d073733b7ca64bc7b48cc86e487b1a9d --
 8月 26 03:50:02 tadashi-N100DC-ITX systemd[1191]: Started Visual Studio Code Tunnel.
 8月 26 03:50:02 tadashi-N100DC-ITX code-tunnel[13630]: [2023-08-26 03:50:02] debug Starting tunnel with `code /usr/share/code/bin/code-tunnel --verbose --cli-data-dir /home/tadashi/.vscode/cli tunnel service internal-run`
 8月 26 03:50:02 tadashi-N100DC-ITX code-tunnel[13630]: [2023-08-26 03:50:02] debug starting as new singleton
 8月 26 03:50:02 tadashi-N100DC-ITX code-tunnel[13630]: [2023-08-26 03:50:02] debug No code server tunnel found, creating new one
 8月 26 03:50:02 tadashi-N100DC-ITX code-tunnel[13630]: [2023-08-26 03:50:02] trace No token in keyring, getting a new one
 8月 26 03:50:02 tadashi-N100DC-ITX code-tunnel[13630]: [2023-08-26 03:50:02] debug [reqwest::connect] starting new connection: https://github.com/
 8月 26 03:50:02 tadashi-N100DC-ITX code-tunnel[13630]: To grant access to the server, please log into https://github.com/login/device and use code xxxx-xxxx
 8月 26 03:50:07 tadashi-N100DC-ITX code-tunnel[13630]: [2023-08-26 03:50:07] trace refresh poll failed, retrying: Error getting authorization: authorization_pending The authorization request is still pending.
 8月 26 03:50:13 tadashi-N100DC-ITX code-tunnel[13630]: [2023-08-26 03:50:13] trace refresh poll failed, retrying: Error getting authorization: authorization_pending The authorization request is still pending.
 8月 26 03:50:18 tadashi-N100DC-ITX code-tunnel[13630]: [2023-08-26 03:50:18] trace refresh poll failed, retrying: Error getting authorization: authorization_pending The authorization request is still pending.

I deleted ~/.vscode and then did code tunnel service install and it started up fine.
It seems that something was wrong with in ~/.vscode.
Thanks.


fix: add npm semantic release

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Interface to piqp solver

Description
Implements cvxpy interface to the piqp solver.
Main repo for the solver: https://github.com/PREDICT-EPFL/piqp
Documentation: https://predict-epfl.github.io/piqp/
Type of change

[x] New feature (backwards compatible)
[ ] New feature (breaking API changes)
[ ] Bug fix
[ ] Other (Documentation, CI, ...)

Contribution checklist

[x] Add our license to new files.
[x] Check that your code adheres to our coding style.
[x] Write unittests.
[x] Run the unittests and check that they’re passing.
[x] Run the benchmarks to make sure your change doesn’t introduce a regression.

Nice work! I guess this is tested automatically by test_qp_solvers.py

I assume so. I didn't see any individual tests for other QP solvers.


Adding requests to the python container

Add the requests module to the python container
Add tests to assert that module installation for the python container works as expected.

Solves #1190

Any other packages we lost/should include?

Here's my personal list:
pip install beautifulsoup
pip install httplib2
pip install lxml
pip install python-dateutil
pip install scrapy
pip install simplejson
pip install twisted

@markusthoemmes added a simple test for request. I suggest we augment the test to require the installed packages but do not necessarily need to test each one of that.

Added @psuter's list with some more tests here: https://github.com/openwhisk/openwhisk/commit/d2df2c2466786a58f1b9170f8cc2074f865ba7e3.


About Code RAW

you lose the function at internal/dataset.py -- 625col. if you choose the mode is RAW, you don't have self.image_paths in your code.
you should add:
colmap_files = sorted(utils.listdir(colmap_image_dir))
image_files = sorted(utils.listdir(image_dir))
colmap_to_image = dict(zip(colmap_files, image_files))
image_paths = [os.path.join(image_dir, colmap_to_image[f])
for f in image_names]
self.image_paths = np.array(image_paths)
after 625 col.

Thanks for pointing it out. However, our proposed model does not support RAW images - you can run the original multinerf code.


gRPC invoke: Do not return error if message is nil

Should fix https://github.com/dapr/java-sdk/pull/822

/ok-to-test

I have verified that this fixed the regression identified in ITs in Java SDK.

Tests are falling (e2e).

So, there is something else then. I wonder if we just need to change the IT for Java SDK to assert for a different error code now.

I am not surprised this broke some E2E tests since it did change the behavior.
Let me know what you prefer. The previous code was returning "Unknown" and although "internal" isn't necessarily better, I am not sure if unknown is good either.
Otherwise, I will look into fixing the tests for this tomorrow, if quick, or Monday :)

@artursouza

Right now, the HTTP API returns UNKNOWN error code too.

What do you mean? Are you talking about gRPC invoke to HTTP endpoint?

After discussing with Artur, we're closing this as the tests will be updated instead


Vendor specific delivery receipt format breaks Deliver_sm

Hi all,
In the SMPP Protocol specification it states:
" The informational content of an SMSC Delivery Receipt may be inserted into the
short_message parameter of the deliver_sm operation. The format for this Delivery Receipt
message is SMSC vendor specific but following is a typical example of Delivery Receipt report."

In the case of the SMSC which I am integrating to, they omit two fields which org.jsmpp.bean.DeliveryReceipt expects (sub and dlvrd) in its constructor- for example this is what they send:
"id:1526758174 submit date:1701240904 done date:1701240904 stat:DELIVRD err:000 text:hellow"
This causes a Null Pointer when this line is executed (cannot parse null to int):
submitted = Integer.parseInt(getDeliveryReceiptValue( DeliveryReceipt.DELREC_SUB, formattedDeliveryReceipt));
I think the code should maybe use zero as a default in the case that these values do not exist but I am not sure of the implications.
If some can guide me I can make the changes...

There is example code to implement a specific decoder (jsmpp-examples/src/main/java/org/jsmpp/examples/receipts/ExampleDeliveryReceiptStripper.java). This will allow you to decode any format the SMSC will use.
I will change the existing default DeliveryReceipt stripper to not generate the NullPointer exception.

Thank you.


Change column position

What's the simplest way to change column position with xsv?
For example, how to move the last column to the first position?

From the README:

select - Select or re-order columns from CSV data.


Mixed Reality Data Provider type bug

Describe the bug
In the default profiles, the WindowsMixedReality Data provider type is incorrect.
Easily fixed by reselecting the WMR type, but needs fixing by default
To Reproduce
Steps to reproduce the behavior:

Add XRTK to scene
Navigate through configuration to the Input System Data providers
Expand the WMR Controller Data Provider
See error

Expected behavior
It should work by default?
Screenshots
If applicable, add screenshots to help explain your problem.

Resolved by releasing a new version of the WMR package


fix the crash on course result (hopefully)

as zkldi has mentioned in #help-bugs in the bokutachi server, /ir endpoints aren't required to return json. also the endpoint is /import/course and not /import-course

FUCK wrong repo


Is evaluation proper? [concerns]

Hi,
I have some concerns on the evaluations used throughout this code repository.
PPO/TRPO
The evaluation uses the same trajectories as for training. However, you have a horizon which could be a partial trajectory. However, if you do evaluation on these partial trajectories, you may be skewing your results for the worse. For example, say we have a batch size of 2048 (as is the default with mujoco). And we have two trajectories of 1000 timesteps and then a partial trajectory of 48 timesteps, it looks like our average returns will be much lower than they actually are since we're using the partial trajectory in our evaluation.
[EDIT] Nvm, I realized I missed something, it's a bit confusing, but returns only get added here, but partial trajectories can be returned but are only used for the training and not considered in evaluation. https://github.com/openai/baselines/blob/master/baselines/pposgd/pposgd_simple.py#L56-L58
However, that still leaves the question of evaluations are based on an unspecified number of trajectories and then are mixed together with trajectories from past policies since it's a running average of the past 100 trajectories. This is unlike the original TRPO evaluation code it seems, but i guess this is up for debate.
DDPG @matthiasplappert
First, the algorithm seems a bit off, you collect 100 time steps, then train for 40 timesteps and then evaluate for another 100 and then loop this 20 times. This has the feel of n-step returns without making use of n-step returns. As I understand it originally DDPG should take a small number of steps (ideally one) and use the replay buffer. Second, you're actually evaluating across at least 10 different policies it seems for one trajectory.
https://github.com/openai/baselines/blob/master/baselines/ddpg/training.py#L122-L139
This seems very off. I think the proper thing would be to move the evaluation loop outside of the training loop and do N proper full trajectories with a single policy per epoch. And again, here you may not be evaluating the full trajectory as in PPO.
Maybe I'm misunderstanding the codebase due to the MPI details, but it seems like overall the evaluations done are not proper and can either beneficially or detrimentally change results (even with the 100 average return running mean). Please let me know if I'm incorrect in my assumptions!

@Breakend may be I missunderstood your question but what's wrong with rollout and train steps? You can collect experience for n steps and train for k and it's totally ok.
As about evaluation you can train without evaluation at all. It doesn't affect training, how it can change any result?

Sorry if I wasn't clear. For rollout and train steps, I guess this was more of an observation about the difference from the original DDPG paper. If you're taking N-steps, why not just use n-step backups (asynchronous updates?). It shouldn't really affect the results, but it's a difference from the original paper i think. (this is not the main concern)
The main concern with the DDPG evaluations is this. If people publish a paper using the evaluation as is, I'm not sure if it's proper. Taking rollouts which use a mixture of policies doesn't seem correct and could result in improper reporting. Let's look at an example.
Take 100 steps, train for 40 iterations.
Case 1:
Now, my policy knows very well how to handle this 100 steps. So we take these same 100 steps on the evaluation policy and move on. This is what is currently implemented (it seems).
Case 2:
If, however, we reset our evaluation environment and run 5 full rollouts, you may see the case that the policy forgets how to handle the previous 100 steps or doesn't yet know how to handle the rest of the policy so it'll fail.
In Case 1 you've skewed your results in your favour by evaluating in sections instead of a single policy on a full rollout. You avoid the possible problem of catastrophic forgetting (if it is in fact there). By using a mixture of policies that were just trained on the sequence that you're about to evaluate on, you're not really evaluating the policy.
Does that make sense? I'm not sure if I'm right in my assumptions here, but you can see the difference if you look at the rllab implementation of DDPG.

Thanks for more detail description. I'm travelling and replying from my phone, will be able to take a look into the rllab implentation details and refresh my knoweledge of the baselines DDPG only at evening. I'll give at the moment only a brief comment to 1) and I hope I remember all the implementation details correctly.
So after 100 rollouts and 20 training steps the algorithm doesn't know well how to handle 100 rollout steps. There were just added to the replay buffer. And during training batches are randomly sampled from it. When you have a size of relay buffer even 100K the chance that you will get a noticeable number of those 100 rollouts sampling 64x20 transitions are quite small.

That may be true in later epochs (that it doesn't matter), but in earlier epochs this is certainly a large factor. For the first 3 epochs or so (with the default settings), you have a greater than random chance of selecting a sample you just saw. While, at the end of training perhaps this is not true so it doesn't matter so much, it still seems off.
Why not just evaluate on full evaluation trajectories? What's happening now is evaluation is actually done using a mixture of old policies across a single trajectory. Maybe it doesn't affect evaluation results, or maybe it does.

It's the right way to evaluate in the sense of total accumulated reward (as in online learning / regret minimization) where you have to balance exploration and exploitation.


How to trigger the tooltip everywhere without worrying about the points?

I want to trigger the tooltip everywhere in the chart area without worrying about the points. How could I do that?
Expected:

Current:
If I move my mouse to near the data point it will trigger the tooltip and show. otherwise, it will hide. This behavior will make the tooltip not continuously, it makes the tooltip disappear for a certain time.

See intersect in documentation
I guess you want:
tooltips: {
  mode: 'nearest',
  axis: 'x',
  intersect: false
}

@andyngdz totaly curious how you styled that graph - it looks awesome.


exportModelToLionWeb and enum literals

JsonSerialization (and probably other parts of Lionweb) expects enum properties to have values that are instances of EnumLiteral, however exportModelToLionWeb does not export enums to Lionweb enums, causing serialization of an exported Kolasu node with enum attributes to fail with a class cast exception.

Good point.
I think a workaround is to call prepareJsonSerialization.
It should register some specific serializers and deserializers to handle that, I think.

I'm facing other problems down the line, but on top of that, I still (randomly) got the same ClassCastException on enum values even after calling the method.

Hurray!
Yes, the fact that jsonSerialization looks like a field but it is not is very error-prone... in retrospective that was not a great idea, and I should refactor it


CASSANDRA-15164 log unusual EstimatedHistogram serializations and deserializations

...and provide a conservative estimate of the droppable tombstone ratio when a deserialized EstimatedHistogram cannot provide one

when that metric in the sstable is overflowed it will cause errors throughout the rest of the codebase that isn't caught

Yeah, there are a couple other places that might still explode, like GROUP BY queries and range reads if we allow an overflowed histogram. Short of dynamically resizing the number of buckets, I'd propose that for now we sanitize the two estimate histograms in StatsMetadataSerializer, so we can at least differentiate the WARN messages in the log.
CC @blerer

Tests in progress: https://app.circleci.com/pipelines/github/maedhroz/cassandra?branch=CASSANDRA-15164


Cannot find module

I got an error when i try to start dispatch.
dispatch start

module.js:340
    throw err;
          ^
Error: Cannot find module '/usr/local/lib/node_modules/dispatch-proxy/bin/../lib'
    at Function.Module._resolveFilename (module.js:338:15)
    at Function.Module._load (module.js:280:25)
    at Module.require (module.js:364:17)
    at require (module.js:380:17)
    at Object.<anonymous> (/usr/local/lib/node_modules/dispatch-proxy/bin/dispatch.js:2:1)
    at Module._compile (module.js:456:26)
    at Object.Module._extensions..js (module.js:474:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Function.Module.runMain (module.js:497:10)

I installed dispatch-proxy with commands:
git clone https://github.com/Morhaus/dispatch-proxy.git
cd dispatch-proxy
sudo npm install -g dispatch-proxy

node --version
v0.10.15

 uname -a
Linux sancho 3.11.0-14-generic #21-Ubuntu SMP Tue Nov 12 17:04:55 UTC 2013 x86_64 x86_64 x86_64 GNU/Linux

i am facing same problem. Your solution is not working. Please help me.
when i run this command
./node_modules/.bin/grunt
I found this error
bash: ./node_modules/.bin/grunt: No such file or directory

@sofathitesh to run grunt you need first run npm install on the project directory.
Anyways now it gives me another error.
Running "coffee:compile" (coffee) task
>> src/dispatcher.coffee:5:5: error: Can't reference 'this' before calling super in derived class constructors
>>     @connectionsTotal = 0
>>     ^
>> In file: src/dispatcher.coffee
>> On line: 4
>>     @connectionsTotal = 0
>>     ^
Warning: CoffeeScript failed to compile. Use --force to continue.


CSS bugs

Hello, I've had some problems with the css of application.css and formatting.css:
In formatting.css (line 40):
.clearfix:after, section:after {
clear: both;
content: ".";
display: block;
height: 0;
overflow: hidden; /* for height 0, is necesary force with overflow hidden */
visibility: hidden;
}
In application.css (line 4):
/*#page, #site_bar_content, footer, header -> (prevent use of this rule in other context, like a header of an article, adding "#page_container >" for footer and header) */
#page, #site_bar_content, #page_container > footer, #page_container > header {
margin: 0 auto;
width: 1000px;
}
I hope this will be useful :)

I'm not so good when it comes to css - wont it break stylings when user will update to the latest version (1.0.x)?
If you could send a pull request I would merge it in master but I'm still curios about 1.0.x apps.

We can't change the CSS for 1.0.x unless it's bugfixes so maybe this would qualify


WAF-296 - Fix for different plp

Added class for PLP to center tiles on screen as per WAF-296

Closing, this is already on main


[Feedback] Email Feedback

There should be an option to provide a list of email addresses that feedback gets mailed to. Probably this should be implemented as a signal on the creation of feedback items.

Feedback is no longer in Zenaida. You can follow this issue at littleweaver/django-feedback#4


Icon needed

Icon Request

Icon name: arrow-right-up-line
Tags of this icon:
Use case: 
Screenshots of similar icons:

I'm not quite clear, do you want this icon?
https://remixicon.com/icon/arrow-right-up-line


add feature to generate output without namespace

Currently, namespace parameter is required.
If one runs schemats without argument -n (or with empty one) like that :
schemats generate -c postgres://postgres@localhost/osm -t users -o osm.ts
It ends up with the following output in osm.ts :
export namespace undefined {

    export namespace usersFields {
...

Using a namespace for a whole module in typescript is ok, but a bit outdated.
With es2015 modules, it seems preferable to generate directly :
    export namespace usersFields {
...

wich allows to import as following :
import * as myNamespaceInConsumerModule from 'osm.ts'
...

It would be nice to make namespace optional.

You are right.
I'm think about deprecating the namespace parameter all together. The choice of having a namespace was due to my inexperience with Typescript earlier, and mixed information from Typescript 1.5 days. As of today, and using Typescript 2.0, there's really no motivation to keep using namespaces

Totally agree :thumbsup:
As a consequence of semver, it will be schemats 1.0 no ?   💃 :tada:

Fixed by #32 https://github.com/SweetIQ/schemats/commit/64143dd352ff1824417c288b248b016a003b4697


Compliance of numeric tower

The behavior on overlow is not specified. This allows compliant Scheme implementations to exist with de-facto C number semantics.

This seems problematic. C is allowed to crash on overflow. Is Scheme permissive to unexpected behavior and termination of the program in that case?

Hi, thanks for commenting! I am not an expert in the Scheme specification, so I am not entirely sure whether crashing on overflow is technically allowed in R7RS-Small. But please note that no C implementation commonly available today actually crashes on overflow, so it is a moot point. My plans were to have the Java semantics, which are better defined, and crash only on integer division by zero.
One important difference compared top C signed integer overflow, which C considers "undefined behavior". I would try to avoid that at any cost.
Sadly, I "temporarily" 🤣 stopped working on this project in 2020, and even though I still intend to resume it, I am not sure exactly when.

R7RS allows a program to crash on numeric overflow.
R6RS prescribes that implementation must raise an exception on overflow.


When user forgets to input either name or endpoint for newly created KB, but attempts to save at other than the first tab, nothing happens and he/she does not know why

Any ideas? Maybe a simple warning, similar in appearance and location to the one received from the server when something else is amiss.

Upon clicking 'save', the user will be redirected to 'main' tab.
The only problem is, the external ui-tabset component does not work well at all and the tabset cannot be changed programmatically all the time (seems to be working only once per page load). This is really strange. But, it can be changed by clicking on any other ui component.
So what I did is once user clicks 'save', he is ALWAYS redirected to 'main' tab, not only upon a negative form validation. Hope that suffices.


feat: delete installed CRDs

Description
Changes proposed in this pull request:

modify SKR CloudResources CR reconciler to delete CRDs prior to removing the finalizer
moved e2e tests for SKR CloudResources CR reconciler to separate package due to delete CRD conflicts with other tests

Related issue(s)

/retest


Add "onConnectionError" notification events

Hi! Thanks for your work.
It would be a good idea to add "onConnectionError" to events.
Sometimes the application crashes due to an error calling the this.#loadEngine.stop() method because this.#loadEngine is not initialized. There is no way to catch this error.

Thanks!

I had the same issue, and I managed to solve it by making changes to the BandwidthMeasurementEngine class.
If you're interested in the fix and want to see the specific changes, you can find them in this pull request: https://github.com/cloudflare/speedtest/pull/24/files

@itvby thanks for reaching out.
This bug has been fixed, and a new onError method has also been added to the module schema. Please upgrade to v1.3.0 to get the fix.
More details on the PR: https://github.com/cloudflare/speedtest/pull/27


use uuid tokens and sql persistence driver

Upstream osad uses fernet tokens as the default. In rpc we want to
continue to use uuid for the time being, so this commit adds a var to
override the default, as well as setting the token persistence driver back to sql
Partially-Closes: https://github.com/rcbops/rpc-openstack/issues/258

I think this is a bad idea in terms of future proofing an environment, scalability and supportability. I'm not sure why we'd choose to keep with SQL backed UUID tokens at this point.

I think the reason to stick with uuid+sql is that it's still the default in Kilo. Changing from the upstream defaults is an operational change for a 11.0 release where it's supposed to be a Kilo code refresh only,
Second, iirc, anyone who wants to use Federation/ADFS would have to change their config back to uuid [at least until keystone fixes the scoped things, which may or may not happen as a security patch]. Having the default change, to then have to change it again seems like a unnecessary thing to have to do for 11.0.

In keystone the default is uuid+sql however the recommended in kilo going forward is fernet. That said, fernet, in its current form, would be an issue for Federation deployments however IMHO staying with uuid+sql because it is solving one problem, for a single use case, which will likely not be the norm while fernet will solve a lot of problems for most other deployment use cases. All that said, I understand the desire to change even if I don't agree with it.

As has been pointed out, this is part of our commitment to keep the delta between juno, and this initial release of kilo (for rpc) as minimal as possible. Whilst fernet is going to be a better future choice, uuid+sql is what we have now, is what we have tested, is what support are comfortable and knowledgable about, and ultimately is a less risky choice at this time.

While I understand the desire to minimize the delta I think this is the wrong course of action. I disagree that this is something that we've not tested. In truth QE has been deploying off the osad kilo branch and the rpc-openstack submodule which contains the bits that made fernet the default token provider. So rpc has been testing fernet for some time now.  Because of that, I'd also say there little to no risk here. While I'll I agree fernet is not something that support has experience with so there will need to be some documentation on how it works and how to do some of the functions manually. However,  I'd think the change would be welcomed as it solves much of the problems I know support has experienced using uuid+SQL.

:+1: LGTM as per requirements

@mancdaz - It should be noted that UUID+SQL has never been tested in OSAD within the Kilo development time-frame if this is something that we intend to do, we should do it soon so QE can get  to running it through some paces.

@cloudnull yes, that's the plan. QE are scheduled to do an entire run through with it, we just need to get this in so that they can.
Having said that, uuid is what has always been deployed in RPC, so I'd expect no regressions in behaviour.

:+1:, merging with @odyssey4me's +1


This adds colored switches!

Inspiration => http://bootsnipp.com/snippets/featured/material-design-switch
Codepen => http://codepen.io/IamManchanda/pen/EmyOZK

@kball @rafibomb ... Apart from this PR,
There is just a Suggestion .... lets keep this way... I mean,
Not only for switches but we can implement the same way:

Cards - #9867
Accordion
Tabs
Table Header

& simply update the docs on coloring part with a link to the building block.
This way ... we are keeping the core framework as lean as possible but also helping the audience on color variations... what's your thought on this??

Another one merged! You're the man @IamManchanda !

@rafibomb .... thanks but i didn't got the answer of my suggestion??


Update pin mapping form ODIN and C029

A small note, you can use rebase which would rebase your commits on the top of the master, This would not produce the merge branch master commit.


Service file binary place mismatch

According to cluster/binary.yml vault binary will be placed to {{ bin_dir }} and according to inventory/sample/group_vars/all.yml that is bin_dir: /usr/local/bin

CLA signed -- achievement unlocked!


610 fix command output

Closes #610

Changes:

Applied a temporary fix to a failing command in the documentation
Implemented correct handling of ExitSuccess
Removed the dependency of contributing.md on mdBook features (actually, there was only one - #include).

@deemp Great work on your contribution! 🌟 You've earned +4 points, reflecting our commitment to recognizing effort despite challenges. While we missed out on code reviews and comments this time, we value your input. Your balance is now +36. Keep the momentum going - we look forward to your future contributions!


Merge 2.1 to develop

DO NOT MERGE YET: Will need to be merged in order with many other PRs.
This brings develop up to date with current 2.1 functionality and fixes.

Closing, as 2.2 was based off of 2.1, and has been merged into develop at this point.


Pickadate options dont work

Whatever option I try to use doesn't make any change.
{{pick-a-time date=date format='T!ime selected: h:i a' formatLabel='h:i <!i>a</!i>' placeholder="Pick a time"}}
In this example nothing affects the component, it always uses standard format.
Any suggestions?

Try passing in those using the options attribute, everything passed in via options is used when creating the pickadate instance.
So instead of what you have, you could do
{{pick-a-time date=date placeholder="Pick a time" options=opts}}
where opts could be
{ format: 'T!ime selected: h:i a', formatLabel: 'h:i <!i>a</!i>' }
Hopefully that works!

Worked like a charm!
Thanks
Cheers

Great! 😊


[Azure Stack] Increas concurrent VPN connections

Concurrent VPN connections is limited to the number of ports instead of # of local user. In order to increase concurrent VPN connection, Our customer doesn't need to create additional local users but use the command "netsh ras set wanports device="WAN Miniport (L2TP)" maxports=10" and modify the maxports.
Also fix some typo and format issue.

Hi @ostrich75, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (larryzh). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

#sign-off

#sign-off

Hi @ostrich75, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (larryzh). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

#sign-off


Pycharm tidies

Codecov Report

Merging #161 (c691e32) into dev-moreflake8reqs (91948d2) will decrease coverage by 0.06%.
Report is 4 commits behind head on dev-moreflake8reqs.
The diff coverage is 75.00%.

Additional details and impacted files

@@                  Coverage Diff                   @@
##           dev-moreflake8reqs     #161      +/-   ##
======================================================
- Coverage               74.53%   74.47%   -0.06%
======================================================
  Files                      33       33
  Lines                    2022     2018       -4
======================================================
- Hits                     1507     1503       -4
  Misses                    515      515

Files
Coverage Δ

soliket/__init__.py
100.00% <100.00%> (ø)

soliket/bandpass.py
91.26% <100.00%> (ø)

soliket/bias.py
100.00% <100.00%> (ø)

soliket/cash.py
100.00% <100.00%> (ø)

soliket/cash_data.py
90.47% <100.00%> (-0.44%)
:arrow_down:

soliket/ccl.py
93.84% <100.00%> (+0.29%)
:arrow_up:

soliket/clusters/survey.py
53.22% <100.00%> (ø)

soliket/clusters/sz_utils.py
86.50% <100.00%> (ø)

soliket/constants.py
100.00% <100.00%> (ø)

soliket/cosmopower.py
16.90% <100.00%> (-0.59%)
:arrow_down:

... and 11 more


Fixed the maxConcurrency parameter summary

The description for the maxConcurrency parameter was talking about the PartitionKey, probably leftovers from a previous method signature.
This PR fixes that.

@christopheranderson  Are there plans to hook up a CI build to this repo?

Yep. @ausfeldt is on it. We'll be set up to run CI and accept PRs/etc. We have a public VSTS account set up so test results/etc. can be visible.


fix: Missing CURSEFORGE_ID and CURSEFORGE_TOKEN in pipeline.yml

Added curseforge id and token as input for the release action in the pipeline workflow

Also, I wonder how the actions are supposed to build the curseforge pack with the default set to false here
I modified it on my repo, is it the intended way ?

Also, I wonder how the actions are supposed to build the curseforge pack with the default set to false here I modified it on my repo, is it the intended way ?

Yea so, most of the people currently use Modrinth,
this just means if there is no CURSEFORGE or MODRINTH ID/TOKEN, it adds it to the release, in this case if you want to add the curseforge to all the releases, you can just set default-release-curseforge: 'true'!
In the future, this could also be done via an env file!

Aah I see, ok thx for the clarification :)


BoneCP to HikariCP migration slowing down the DB with "Create Sort Index"

Hi @brettwooldridge
Issue
We are migrating from Amazon EMR 5.x to EMR 6.8.
EMR 5.x uses BONECP as connection pooling to connect to Hive metastore (RDS MySQL DB). EMR 6.8 uses HikariCP to connect to metastore mySQL DB.
The backend RDS DB became extremely slow with HikariCP. I created a simple test to fire 100 queries and tested it with BoneCP and HikariCP.
All the DB connections are consumed by resource intensive "Create Sort Index" queries when I use HikariCP. I noticed that BoneCP issues the queries little slower than HikariCP so I added 7 sec sleep between my queries and "Create Sort Index" queries did not start. I have tested it multiple times with BoneCP and HikariCP and the "Create Sort Index" gets started only when its HikariCP + DB is stressed with high number of queries. After all connections are consumed by "Create Sort Index", then we start getting connection timeout. I changed maximumPoolSize=100 and it avoided connection timeout but "Create Sort indexes" still started. We have r5.4xlarge DB (16 vCPU) -->  recommended pool size is 32 so it did not seem like a right solution to increase maximumPoolSize.
Question:
Is there any way to control the rate at which queries are issues to the DB?
Current settings
Below are my setting
connectionTimeout  =  30000
maximumPoolSize  =  10. (experimented with 5,8,32,100)
minimumIdle  =  10
Tests
I have tested with https://github.com/brettwooldridge/HikariCP/wiki/MySQL-Configuration but no luck
I also compared sessions setting between BoneCP and HikariCP and changed autoCommit=false and transactionIsolation to TRANSACTION_READ_COMMITTED but issue persists.
We have been struggling to find the fix. Please let us know if you have any recommendations.
Thank you
Kiran

If you can identify the root cause of your problem and create a unit test reproducing the behavior, I would be happy to take a look. But to be honest, I doubt this is caused by a bug in HikariCP. It could be a coincidence. Or it could be that BoneCP just happened to do something differently in just the right way to cause the problem to not present.
Either way, a unit test that reproduces the issue would be useful.
Even better would be if you have a pull request that also includes the fix for the problem.


Integration of hdt-rdf compression module

I have created the hdt package as mentioned and created a class TripleOps and corresponding Test cases are submitted.

Code has been updated as mentioned. Also updated the code as per scalastyle

Hi @Gezim,
Okay noted. Thank you for letting me know.
See you.
Abakar
On Mon 13. May 2019 at 09:33, Gezim Sejdiu<EMAIL_ADDRESS>wrote:

@GezimSejdiu commented on this pull request.
In
sansa-rdf-spark/src/main/scala/net/sansa_stack/rdf/spark/model/hdt/TripleOps.scala
https://github.com/SANSA-Stack/SANSA-RDF/pull/81#discussion_r283217335:

@return returns RDD[Tripe]

*/
def readRDFFromFile( input : String) : RDD[graph.Triple] = {
val triples : RDD[graph.Triple] = spark.rdf(Lang.NTRIPLES)(input)
triples
}

/**

This is key function of TripleOps that read RDF file and create Dictionaries and Index Table and register them as Spark In memory Table

@param input Input RDF File Path [Either One of the input is require]

@param compressedDir Input compressed-directory Path to read compressed data directly [Either One of the input is require]

@param registerAsTable If true, it register all the DF as Spark table

@return Returns the Tuple4 [IndexDataFrame,SubjectDictDataFrame,ObjectDictDataFrame,PredicateDictDataFrame]

*/
def createOrReadDataSet(input : String, compressedDir : String, registerAsTable : Boolean = true) : (DataFrame, DataFrame, DataFrame, DataFrame) = {

Hi @abakarboubaa https://github.com/abakarboubaa ,
indeed. There is no need to do such a separate. I already did it here:

getHDT
https://github.com/SANSA-Stack/SANSA-RDF/blob/develop/sansa-rdf-spark/src/main/scala/net/sansa_stack/rdf/spark/model/hdt/TripleOps.scala#L80
and readHDTFromDisk
https://github.com/SANSA-Stack/SANSA-RDF/blob/develop/sansa-rdf-spark/src/main/scala/net/sansa_stack/rdf/spark/model/hdt/TripleOps.scala#L110

See you
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/SANSA-Stack/SANSA-RDF/pull/81#discussion_r283217335,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKQTRGANWLN37RWQASGLSKLPVEKVNANCNFSM4HKYOT2A
.


Find ways to get the coin icons

Per reddit users, it would be great if we could include the coin icon for each coin. What your thoughts on how we can handle this?

Well, that is possible. The cryptocompare API returns a url for coin icons for coins that have an icon (coins that don't have an icon are assigned a default image). So we could include a link to the icon in the cryptocurrencies.json dataset. The question is whether or not we should be (or even allowed to per the cryptocompare TOS) "hotlinking" the cryptocompare images, if not, then we would need to push/host the images somewhere else.
If the icons are really small, then another possibility is to embed a base64 encoding of the image within the dataset (obviously, this only works if the images are really small).

Ah I see now. Hmm I don't see anything in their ToS against "hot linking" images, but I would imagine that's because no one has really done it at a large scale. The base64 encoding is concerning as it would increase the actual file size :(.
Could we possibly download the images and host the photos within a subfolder inside this repository  ?

Yeah, the filesize of the module will be pretty large if we want to include (up to) thousands of images, so maybe we can run a few tests and see which way would be the most economical (all things considered).