Share icon is grey while edit icon is black

On product view, the Share icon is grey while edit icon is black. They should both be black and the same size.

Could I know if anyone is still working on it or can I work on it @teolemon ?

@Karljoones have you started on this?

I guess it's fine to go ahead.

Thanks!! will be submitting PR soon !!

Sorry, haven't had a chance, had great ambition to get a lot of bugs closed but life got in the way!

@teolemon  i would love working on this

#486

Closed duplicate #486


Publish SNAPSHOT to Maven Central

Done in https://github.com/fbiville/hands-on-neo4j-devoxx-ma-2018/commit/64f064e6898ba980d20235523c5f1cd75f6257fd


Fix issue where using the same object causes issues with Build optimisers

The existing code does not work with Angular, and possibly other Webpack bundlers where they optimize code.
The issue is that this code gets generated:
key: 'normalizeData', value: function(t) {
  return 'object' !== r(t) && (t = {}), t.text = t.text || '', t.level = parseInt(t.level) || this.defaultLevel.number, t;
}

When running this, t's existing properties are readonly so it cannot overwrite them, this patch fixes it.

Hi @talyguryn @gohabereg sorry to ping you, but this PR has been open for two weeks.
Would it be possible to merge this change?  In my code to fix this, I extend the class but it's currently breaking my tests and this quick fix would resolve that.
Thanks

Hi @tanepiper ,
Sorry for delay! Fix is ok, could you please patch the version, build the package using yarn build and include build files to PR.
Thanks!

@gohabereg No problem, done.

There are conflicting files, I guess you need to pull the master and rebuild the package


feat: added aitools.inc

Added the aitools.inc directory.

@mahseema - Is this okay for the PR or need anything else from me to suggest a new directory?

Hey Lucas, Thanks for submit your directory


Release: 2.0.5-alpha.1

@thanhsonng Released

@namdaoduy Thank you very much for your fine pipeline


Add chart's option to define a hardcoded  "start" and "end"

It should help to focus on a custom period of time

Idea of before/after behaviour

then i "crop" the start/end of my graph

The render show a gradiant when bar were cut
Date can be shown on the bar
I'm still without any clue how the UI should propose this function.


why cpu event would have park unpark function?

i use async profiler to get cpu event.
and tran it to pprof.
the result shows most of the cpu costed in park.
is this correct?

Unfortunately, I can't provide support for third party software (pprof) on async-profiler github.
To inquire about profiling results, please provide the original async-profiler's recording in .jfr format.

Unfortunately, I can't provide support for third party software (pprof) on async-profiler github. To inquire about profiling results, please provide the original async-profiler's recording in .jfr format.

file: cpu (2).jfr.zip
jfr print:
jdk.ExecutionSample {
  startTime = 2023-08-02T10:15:00.250261066Z
  sampledThread = "Log4j2-TF-1-AsyncLogger[AsyncContext@5d099f62]-1" (javaThreadId = 12)
  state = "STATE_DEFAULT"
  stackTrace = [
    libpthread-2.17.so;649be141 (deleted).pthread_cond_timedwait() line: 0
    libjvm.so.Unsafe_Park() line: 0
    sun.misc.Unsafe.park(boolean, long) line: 0
    java.util.concurrent.locks.LockSupport.parkNanos(Object, long) line: 215
    java.util.concurrent.locks.AbstractQueuedSynchronizer$ConditionObject.awaitNanos(long) line: 2078
    com.lmax.disruptor.TimeoutBlockingWaitStrategy.waitFor(long, Sequence, Sequence, SequenceBarrier) line: 38
    com.lmax.disruptor.ProcessingSequenceBarrier.waitFor(long) line: 56
    com.lmax.disruptor.BatchEventProcessor.processEvents() line: 159
    com.lmax.disruptor.BatchEventProcessor.run() line: 125
    java.lang.Thread.run() line: 748
  ]
}

I see only 5 execution samples in your .jfr profile, 2 of them coming from sun/misc/Unsafe.park, which looks fine, considering that awaitNanos is called in a loop. The kernel part of the stack trace (ending with finish_task_switch) confirms the CPU time is  spent on context switching caused by thread parking.

Right - it's not uncommon for park/unpark to take much CPU resources, for example, see #174 and #577.
In this case, however, pprof graph looks suspicious: normally, cpu profile does not show AsyncGetCallTrace - so, it's obviously not a pure cpu profile.


shell scripts

I think we might simply delete the two shell scripts. You can put exactly the same command into the travis file. Or do they serve any other purpose than CI?

Just put the very same commands in the travis  file. Something like this
- cd doc/tutorial; python simulate.py; cd ../..


fix: Allow #top and the empty string as special-case non-missing IDs

As noted in #12367, #top is a perfectly valid hashlink for the top of the document. (TODO on rest of description, I'm going to bed.)
Also, AFAICT, there were no tests for handleMissingId in the first place, so I added some.

Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[x] Ideally, include a test that fails without this PR but passes with it.

Tests

[x] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[ ] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpm changeset and following the prompts. Changesets that add features should be minor and those that fix bugs should be patch. Please prefix changeset messages with feat:, fix:, or chore:.

Edits

[x] Please ensure that 'Allow edits from maintainers' is checked. PRs without this option may be closed.

Not sure why but the added test is failing.

@elliott-with-the-longest-name-on-github I managed to get the CI tests passing. Was there anything else you wanted to do for this PR besides the changeset?

Thanks!


Remove delegate how-to page + supporting changes

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[ ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

Preview environment: https://ci.pr.harness.io/pr-421

Preview environment: https://ci.pr.harness.io/pr-421


Add some explanation/distinction about the web and local versions.

Let's update the readme to explain the distinction between the web version of this pvlcoe.nrel.gov and the local version, including information about how to track down which version is/was live on pvlcoe.nrel.gov. It will often be the case that the github version here will be ahead.

We should also handle the change log as follows: Update with the date of github release and version along with a note that github and nrel.gov release dates might be different.


Prepare 7.24.0

Changes proposed:

[x] Prepare

@robertohuertasm Can you please be so kind, and do the changes as well as commit them, so we can proceed with the release?

@JimiC, all ready to tag.The honour is all yours!

@JimiC superb work with the publication flow!!


Is it possible to add an auxiliary output to an algorithm?

Hi! I am trying to add an auxiliary output predicting a feature of the state vector onto an A2C agent with an LSTM actor critic policy. However, my efforts so far have come up short. How can I do this? Thanks in advance.

Hey. These issues on Github are mainly for bugs and enhancement purposes, not for technical support. The only way to go about it is to see where loss is created and trying to fit in your modifications there.
Also, if LSTM is not necessary, I suggest taking a look at stable-baselines3 which is done in PyTorch. It should be much easier to modify.

Hey. These issues on Github are mainly for bugs and enhancement purposes, not for technical support. The only way to go about it is to see where loss is created and trying to fit in your modifications there.
Also, if LSTM is not necessary, I suggest taking a look at stable-baselines3 which is done in PyTorch. It should be much easier to modify.


reject_build_waiting_for_review is expiring the previous build if build is in Waiting for review and In-review status on testflight

The reject_build_waiting_for_review new logic is expiring the previous builds if it's in betareview instead deleting from the review it's getting expired and making the build unavailable for the external testers in the testflight, before 2.221.0 versions is
reject_build_waiting_for_review logic is not expiring the build if it's in betareview it will correctly remove the previous build from review and submit the new build for the review, please bring back that change as soon as possible 🙏🙏 @joshdholtz

This is the PR causing the issue
https://github.com/fastlane/fastlane/pull/21995

@joshdholtz any update?

Could anyone please help here

@joshdholtz/any one can help here @AliSoftware @mokagio


Important vpiActual nodes only with -elabuhdm flag

To have the proper type of result variable in https://github.com/chipsalliance/Surelog/issues/1449, the -elabuhdm option is required.
Is it intentional?
These nodes are needed in uhdm-verilator, but full elaboration option can include many other nodes that are not needed and may cause performance problems.

Yes, it's mandatory. I'm not going to develop an a-la-carte elaboration option. That would be a maintenance nightmare.
 That will take care of the performance issues you see.


Save filename to elastic

Issue #, if available:
Closes #248
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

I'm not sure why github is showing such a long commit hisotry for this PR. The only commit that matters is this one:
https://github.com/aws-solutions/aws-media-insights-content-localization/pull/249/commits/e379356bcc6f28ec025e6e51c3bf7166a8cdc87a
Maybe try recreating this PR...


apply should re-perform tasks if the top-level configuration data changes

We might be able to do something smarter than "did file change", we may even be able to determine "which" file configs changed.

we also probably want some kind of dry-run command (maybe?)

Implemented here: https://github.com/vkottler/rcmpy/pull/13


Initiative assignments doesn't assign role assignments

When assigning policy initiatives containing policies with DeployIfNotExists actions through blueprints, role assignments are not included.
This works fine when assigning individual policies.
The policy documentation states that using command-line approaches to assigning policies, the role assignments must be assigned as a post policy assignment task.
https://docs.microsoft.com/en-us/azure/governance/policy/how-to/remediate-resources
This is something that is already handled in blueprints for individual policies, but it seems not so for initiatives.
Is this a bug, missing feature, or expected behavior?

This is a missing feature. The recommendation is to assign the policy initiative via ARM Template artifact and you can also do the role assignment in that template.


support recent closed feature for query editor and notebook

This PR fixes #20103
notebook and query editor didn't implement the toUntyped() method that is used by vscode to determine whether an editor should be added to the recent closed list when an editor is getting closed.
https://github.com/microsoft/azuredatastudio/blob/main/src/vs/workbench/services/history/browser/history.ts#L724
I am not adding this to the untitled query editor and notebook because the underlying resource can't be mapped to the correct  input, it doesn't seem to be trivial to implement. giving the fact that open a new query editor and new notebook is easy enough, i am going to leave these 2 scenarios as is.

updated description with screenshots.

Make sure there's a test case added for this too


[3.x] Refactor Android platform export structure

This is a manual cherry-pick from https://github.com/godotengine/godot/pull/50771 for the Android platform.

Thanks!


website not working

hello @ansonbenny
i tried to deploy the site on netlify but i got a error "error facing issues". Ijust separated the client and server folder into different repositories. and hosted the client directory o n netlify and the server directory on adaptible.io and addded the url into the client .env file Creating a new value called Backend_url. Even after doing this iam getting the same issue.
Am i doing something wrong
can u guide me through the process pls.
i looked at ur hosted site on render after signup the songs only play for 30 seconds i thought i should mention it so that u ca fix it
my hosted site url "https://music-streamerx.netlify.app/"
u have some other projects also in the same server and client directory type just guide me how can i deploy them.
i will be waiting for ur reply.

Yes Even i'm facing the same issue. Can you look into these please.if possible

It doesn't work on netlify
On Thu, Apr 25, 2024, 12:27 PM ShaikHaaris @.***> wrote:

Yes Even i'm facing the same issue. Can you look into these please.if
possible
—
Reply to this email directly, view it on GitHub
https://github.com/ansonbenny/Music-Streamer/issues/2#issuecomment-2076499181,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWNWLFCUJCVQUOKY2KXRQ6TY7CSM3AVCNFSM6AAAAABFMHHNMGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANZWGQ4TSMJYGE
.
You are receiving this because you were mentioned.Message ID:
@.***>

It doesn't work on netlify
…
On Thu, Apr 25, 2024, 12:27 PM ShaikHaaris @.> wrote: Yes Even i'm facing the same issue. Can you look into these please.if possible — Reply to this email directly, view it on GitHub <#2 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AWNWLFCUJCVQUOKY2KXRQ6TY7CSM3AVCNFSM6AAAAABFMHHNMGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANZWGQ4TSMJYGE . You are receiving this because you were mentioned.Message ID: @.>

Hey @ansonbenny should I deployed it to render or vercel. If not can u recommend which service I should use. And the song on ur site are playing only for 30 secs.

@skgupta507 Spotify doesn't provide full song, i suggest to host on render. before hosting on render, build client and move dist folder to server folder then host server folder on render. then will work both client and server.

@ShaikHaaris follow readme steps for setup .env, you can generate MAIL_SECRET from google app password, for spotify id & secret https://developer.spotify.com/documentation/web-api/tutorials/getting-started

@skgupta507 Spotify doesn't provide full song, i suggest to host on render. before hosting on render, build client and move dist folder to server folder then host server folder on render. then will work both client and server.

hey @ansonbenny sorry for disturbing but there is only public and src folders in the client directory which folder shuld i move to server from this both.

@ShaikHaaris You can connect me on LinkedIn https://in.linkedin.com/in/anson-benny

@skgupta507 yes, run this command on client side npm run build


[Network] az network vnet subnet: Add parameter --default-outbound-access

Related command

Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Network


Cleanup PCodec soft dependency

As with other soft dependencies in Numcodecs, the pattern is to attempt to import them and if their dependencies cannot be resolved skip them. Have applied this pattern to PCodec to be consistent with the other optional codec dependencies.
TODO:

[ ] Unit tests and/or doctests in docstrings
[ ] Tests pass locally
[ ] Docstrings and API docs for any new/modified user-facing classes and functions
[ ] Changes documented in docs/release.rst
[ ] Docs build locally
[ ] GitHub Actions CI passes
[ ] Test coverage to 100% (Codecov passes)

@dstansby could you please review this one?

Resolved conflicts and added a changelog entry


Bug forwarding fragmented IP frames

Short
Fragmented packets are not forwarded
Proposal
With large packets, like video frames, packets are split into multiple IP frames. Our TUN interface processes and decodes an l4 packet starting from a single frame which is not always the case. When a L4 packet is split across multiple frames, the decode fails, and the packet is dropped.
Solution
We need a packet buffer to sum together all the incoming frames into a single packet.
Status
Looking for an efficient solution
Checklist

[ ] Discussed
[ ] Solved
[ ] Tested

fixed with #53


Fix doc: delete annotation fields

Ⅰ. Describe what this PR does
Delete annotation fields in doc, the fields will be use in the future? if yes, i will close the pr ：）
Ⅱ. Does this pull request fix one issue?

NONE
Ⅴ. Special notes for reviews
@cheyang

Codecov Report

Merging #140 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #140   +/-   ##
=======================================
  Coverage   20.66%   20.66%
=======================================
  Files          16       16
  Lines         329      329
=======================================
  Hits           68       68
  Misses        241      241
  Partials       20       20

Continue to review full report at Codecov.

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

/lgtm

Thanks @xieydd


Move Index to use kclient

This allows:

Proper HasSynced support for Index
Allow things using index to use kclient
Allow proper use of filtered informer with Index

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
ee0c259a48321a5c9c985e163cf8f02037fdbd3f
link
true
/test unit-tests-arm64

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
ee0c259a48321a5c9c985e163cf8f02037fdbd3f
link
true
/test unit-tests-arm64

lint_istio
ee0c259a48321a5c9c985e163cf8f02037fdbd3f
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
ee0c259a48321a5c9c985e163cf8f02037fdbd3f
link
true
/test unit-tests-arm64

lint_istio
ee0c259a48321a5c9c985e163cf8f02037fdbd3f
link
true
/test lint

gencheck_istio
ee0c259a48321a5c9c985e163cf8f02037fdbd3f
link
true
/test gencheck

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
ee0c259a48321a5c9c985e163cf8f02037fdbd3f
link
true
/test unit-tests-arm64

lint_istio
ee0c259a48321a5c9c985e163cf8f02037fdbd3f
link
true
/test lint

gencheck_istio
ee0c259a48321a5c9c985e163cf8f02037fdbd3f
link
true
/test gencheck

unit-tests_istio
ee0c259a48321a5c9c985e163cf8f02037fdbd3f
link
true
/test unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
ee0c259a48321a5c9c985e163cf8f02037fdbd3f
link
true
/test unit-tests-arm64

gencheck_istio
ee0c259a48321a5c9c985e163cf8f02037fdbd3f
link
true
/test gencheck

unit-tests_istio
ee0c259a48321a5c9c985e163cf8f02037fdbd3f
link
true
/test unit-tests

lint_istio
b8e487fb56810b754331d790b9ade5d00d8fad94
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
ee0c259a48321a5c9c985e163cf8f02037fdbd3f
link
true
/test unit-tests-arm64

gencheck_istio
ee0c259a48321a5c9c985e163cf8f02037fdbd3f
link
true
/test gencheck

unit-tests_istio
ee0c259a48321a5c9c985e163cf8f02037fdbd3f
link
true
/test unit-tests

lint_istio
b8e487fb56810b754331d790b9ade5d00d8fad94
link
true
/test lint

integ-pilot-istiodremote-mc_istio
b8e487fb56810b754331d790b9ade5d00d8fad94
link
true
/test integ-pilot-istiodremote-mc

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

gencheck_istio
ee0c259a48321a5c9c985e163cf8f02037fdbd3f
link
true
/test gencheck

unit-tests_istio
ee0c259a48321a5c9c985e163cf8f02037fdbd3f
link
true
/test unit-tests

lint_istio
b8e487fb56810b754331d790b9ade5d00d8fad94
link
true
/test lint

integ-pilot-istiodremote-mc_istio
b8e487fb56810b754331d790b9ade5d00d8fad94
link
true
/test integ-pilot-istiodremote-mc

integ-security-multicluster_istio
b8e487fb56810b754331d790b9ade5d00d8fad94
link
true
/test integ-security-multicluster

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests_istio
ee0c259a48321a5c9c985e163cf8f02037fdbd3f
link
true
/test unit-tests

lint_istio
b8e487fb56810b754331d790b9ade5d00d8fad94
link
true
/test lint

integ-pilot-istiodremote-mc_istio
b8e487fb56810b754331d790b9ade5d00d8fad94
link
true
/test integ-pilot-istiodremote-mc

integ-security-multicluster_istio
b8e487fb56810b754331d790b9ade5d00d8fad94
link
true
/test integ-security-multicluster

gencheck_istio
b8e487fb56810b754331d790b9ade5d00d8fad94
link
true
/test gencheck

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
b8e487fb56810b754331d790b9ade5d00d8fad94
link
true
/test lint

integ-pilot-istiodremote-mc_istio
b8e487fb56810b754331d790b9ade5d00d8fad94
link
true
/test integ-pilot-istiodremote-mc

integ-security-multicluster_istio
b8e487fb56810b754331d790b9ade5d00d8fad94
link
true
/test integ-security-multicluster

gencheck_istio
b8e487fb56810b754331d790b9ade5d00d8fad94
link
true
/test gencheck

integ-ambient_istio
b8e487fb56810b754331d790b9ade5d00d8fad94
link
true
/test integ-ambient

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
b8e487fb56810b754331d790b9ade5d00d8fad94
link
true
/test lint

integ-pilot-istiodremote-mc_istio
b8e487fb56810b754331d790b9ade5d00d8fad94
link
true
/test integ-pilot-istiodremote-mc

integ-security-multicluster_istio
b8e487fb56810b754331d790b9ade5d00d8fad94
link
true
/test integ-security-multicluster

gencheck_istio
b8e487fb56810b754331d790b9ade5d00d8fad94
link
true
/test gencheck

integ-ambient_istio
b8e487fb56810b754331d790b9ade5d00d8fad94
link
true
/test integ-ambient

integ-telemetry_istio
b8e487fb56810b754331d790b9ade5d00d8fad94
link
true
/test integ-telemetry

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-pilot-istiodremote-mc_istio
b8e487fb56810b754331d790b9ade5d00d8fad94
link
true
/test integ-pilot-istiodremote-mc

integ-security-multicluster_istio
b8e487fb56810b754331d790b9ade5d00d8fad94
link
true
/test integ-security-multicluster

integ-ambient_istio
b8e487fb56810b754331d790b9ade5d00d8fad94
link
true
/test integ-ambient

integ-telemetry_istio
b8e487fb56810b754331d790b9ade5d00d8fad94
link
true
/test integ-telemetry

lint_istio
a22b2519b4bd9a04cfa3722fda06035ae67fbbc0
link
true
/test lint

gencheck_istio
a22b2519b4bd9a04cfa3722fda06035ae67fbbc0
link
true
/test gencheck

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

gencheck_istio
8832aad5ca31d14ab1dfc219004387848e79d386
link
true
/test gencheck

lint_istio
8832aad5ca31d14ab1dfc219004387848e79d386
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

gencheck_istio
8832aad5ca31d14ab1dfc219004387848e79d386
link
true
/test gencheck

lint_istio
8832aad5ca31d14ab1dfc219004387848e79d386
link
true
/test lint

unit-tests-arm64_istio
72e148ec789a34e12676d4fb550399c28437f31b
link
true
/test unit-tests-arm64

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


add clear functions to remove all tier listeners and features Loaded listeners

The tiers can add listeners, however they can not remove them. I added 2 functions to clear all the listeners.

Thanks!


文章收集

并发相关

[x] Address Sanitizer
[x] Thread Sanitizer Algorithm Overview
[x] ruigulala/ConAnalysis TSan 扫描, lldb 复现
[x] ConVul : An Effective Tool for Detecting Concurrency Vulnerabilities (ASE2019) 使用 Intel Pin Framework 插桩, FastTrack 检测竞争事件, 鉴定为屎
[x] Understanding Real-World Concurrency Bugs in Go (a survey, ASPLOS 2019) 分析介绍go常见漏洞类型
[ ] Language-based information-flow security (SAC 2003)
[ ] FastTrack: Efficient and Precise Dynamic Race Detection. 动态事件顺序检测算法, 高引.
[x] WAFFLE, exposing memory ordering bugs efficiently with active delay injection. 基于 delay-injection 来动态触发竞争
[x] Eraser: a dynamic data race detector for multithreaded programs. [Savage et al.'97]
[ ] Time, Clocks, and the Ordering of Events in aDistributed System. [Lamport '78]

Books: The Architecture of Open Source Applications (Volume1)

[x] The Bourne-Again Shell (Bash)
[x] LLVM

并发fuzz参考文献推荐：

Let's stop counting centuries
https://corecursive.com/building-powershell-with-jeffrey-snover/#intro
https://blog.sbensu.com/posts/demand-for-visual-programming/
Seven habits of effective text editing -- Bram Moolenaar
what's your most productive shortcut with vim


ref(client): improve client error message

This PR improves the resultant client error message they get when the returned response is
incorrect. This is especially important for determining the error message
Old message when using a v1 client with a v2 cluster:
http://deis.<IP_ADDRESS>.xip.io does not appear to be a valid Deis controller. Make sure that the
Controller URI is correct and the server is running.

New message:
http://deis.<IP_ADDRESS>.xip.io/v1/ does not appear to be a valid Deis controller. Make sure that the
Controller URI is correct and the server is running.

Notice the appended /v1/ path, which users can assume this is a v1 client.

Good thinking! I tested this change locally:
$ make -C client/ build
$ deis login http://not.real
http://not.real/v1/ does not appear to be a valid Deis controller.
Make sure that the Controller URI is correct and the server is running.
Error: Get http://not.real/v1/: dial tcp: lookup not.real on <IP_ADDRESS>:53: no such host

Should this also be ported forward for Deis v2?

I had a branch available but github wouldn't let me PR it lol. Thanks for porting the fix!

Could we even take it a step farther and emit a very specific "You must upgrade your client" error when the client detects "/v2" in the controller URL? I'm not sure it's obvious to users who aren't all that familiar with Deis that a `v2

Could we take it a step further and emit a specific "You must upgrade your client" error when detecting "v2" in the controller URL with a v1 client? I don't think that including the full controller URL with /v2 is obvious to the average Deis user -- it could just be an API version or somesuch that is inconsequential.

Could we take it a step further...

The client doesn't know that a parallel /v2/ endpoint does exist, only that /v1/ didn't. And I think this message could be triggered by conditions other than a Deis version mismatch, so I would argue this is good enough. At least there is a way to tell now what endpoint deis was trying.

Or were you suggesting the CLI have additional probing code to see if there is specifically a version mismatch?

I don't think we should do a version probe or else we're gonna have to do that for every major release going forward. However, @slack's recommendation at https://github.com/deis/workflow-cli/pull/34#issuecomment-206439428 would be a great middleground.

Or were you suggesting the CLI have additional probing code to see if there is specifically a version mismatch?

I was thinking that it'd be nice to do a major version mismatch check.

I don't think we should do a version probe or else we're gonna have to do that for every major release going forward.

Agreed.

@carmstrong we actually do a check for the DEIS_API_VERSION header. The problem is that the v2 server does not respond to /v1/ requests so we get a 404 with no DEIS_API_VERSION header :'(

Considering that https://ci.deis.io/job/test-ec2/4030/ passed and 4031 is only a error message change (which we already changed the message before and it passed in 4030), I think this is safe enough to merge to get the LTS release out the door today. If something breaks I owe @mboersma a :beer: :)


Python 3.9 support

Hello,
Thanks for this project, would it be possible to add Python 3.9 support ?

#19

Hi,
First of all, thanks for creating and maintaining this amazing library! Also, happy new year.
I was wondering if @wilsonzlin might release a new version of this library to PyPI with support for 3.9? I've seen the proposed patch was merged in #19. I'm developing on an M1 Mac and it's the only version of Python I can use at the moment.
Thanks in advance!

Hi,
First of all, thanks for creating and maintaining this amazing library! Also, happy new year.
I was wondering if @wilsonzlin might release a new version of this library to PyPI with support for 3.9? I've seen the proposed patch was merged in #19. I'm developing on an M1 Mac and it's the only version of Python I can use at the moment.
Thanks in advance!

I just made a release with a build for Python 3.9 and macOS 11, version 0.3.12, let me know if it works for you. I have noticed some issues around Big Sur and Apple Silicon, so would be interested to hear your experience with the package.

I just made a release with a build for Python 3.9 and macOS 11, version 0.3.12, let me know if it works for you. I have noticed some issues around Big Sur and Apple Silicon, so would be interested to hear your experience with the package.

@wilsonzlin The new release works well from my side

@wilsonzlin The new release works well from my side

Thanks @wilsonzlin! It will probably be a few days before I have any actionable feedback, but consider no news as good news.
Thanks again for putting this together!

Thanks @wilsonzlin! It will probably be a few days before I have any actionable feedback, but consider no news as good news.
Thanks again for putting this together!

Tried it out for a bit and all seems to work as intended. Things I tried:

minify with minify_js=True

Tried it out for a bit and all seems to work as intended. Things I tried:

minify with minify_js=True

🎉 Sounds good, closing this issue.

🎉 Sounds good, closing this issue.


Establish the project communication channels using Discussions.

Establish initial guidelines around Discussions and moderation around how the project communications would occur.

Trying out discussions at the moment.


HHH-11147 handleWrite does not hit the db

@sebersole @gbadner @Sanne apart the need for code refactoring and for a better name for the new methods, do you think the changes 1c6ccf0 have any sense? Basically what I'm trying to do is to avoid to hit the db for EnhancementAsProxyLazinessInterceptor#handleWrite() (it seems it is what happen for LazyAttributeLoadingInterceptor).
I added a test 4b70c8c but probably I'm missing a lot of cases that can fail with this change.
as a side effect it seems also the failing test i added a couple of days ago 4b70c8c works with the change.

yes you are right avoiding the select will always cause the update (and I haven't think about that), it just seemed to me this was the behaviour with the proxy but I may be wrong, not really an expert on this part :( Tomorrow I will try to doublecheck and add few more tests.

Sorry, that's not what I was trying to imply.
I had thought you meant that you were seeing unnecessary UPDATE statements.  But I see now you probably just meant that calling a setter does not trigger initializing the proxy.  Yes, that should happen.  Though only least the lazy group (and the base group) to which the "writing attribute" belongs.

@sebersole you are right, my change looks wrong.

Why do you say it is wrong?  What exactly is wrong?

I believed that with an HibernateProxy a call to a setter method should not trigger a query but it seems I was wrong.

Right, it should do the select


Missing event in external mode: editCancel

ng2-smart-action-edit-cancel button does not trigger any event, which makes things tricky to know the edit state of the grid in external mode.

we can know if a user pressed the edit button by using the edit Output
we can know if a user pressed the edit confirm button by using the editConfirm Output
but we CAN'T know is a user pressed the edit cancel button directly

A workaround is to listen to rowHover event and check for isInEditing: false in pair with an instance var storing the index of the last edited row.
But this is tricky and not ideal because this event is triggered a lot (on mouse hover).
<ng2-smart-table
 [settings]="settings"
 [source]="datasource"
 (rowHover)="onRowHover($event)"
>
</ng2-smart-table>

  onRowHover(row) {
    if (row.isInEditing) {
      this.currentlyEditedRowIndex = row.index
    }
    if (!row.isInEditing && row.index === this.currentlyEditedRowIndex) {
      console.log(`Edition has been cancelled for row ${row.index}`)
      this.currentlyEditedRowIndex = null
    }
  }

  onEdit(row) {
    console.log(`Edition has been triggered for row  ${row.index}`)
    // Set current row in edit mode
    row.isInEditing = true;
  }

Would that be possible to see added editCancel event?

Any update on the feature?


CSS is Broken

changes that break css are in css-broken branch
After TS standardisation this warning was thrown in build:
warn - The content option in your Tailwind CSS configuration is missing or empty. warn - Configure your content sources or your generated CSS will be missing styles. warn - https://tailwindcss.com/docs/content-configuration

fixed by simply not swapping tailwindcss config to ts


Additional costume slots are not awarded upon level increase

Summary
Experiencing a bug where costume slots aren't awarded at level 20 as expected. I'm not getting a new costume slot until level 23.

The code looks right, and doesn't appear to have changed. Will need to dig further.

I can't replicate this upon further testing... soooo :man_shrugging:


Tabs.Tab ref does not allow clicking / selection

My goal is to programmatically select a tab.
When using useRef with Tabs.Tab, doing ref.current.click() doesn't work.
e.g. <Tabs.Tab ref={ref} anchor="compare">
The ref never seems to get populated correctly.
As a workaround, if I assign an id:
<Tabs.Tab id="compare" anchor="compare">
Then I can do document.getElementById("compare")?.click();
Is ref not being set correctly internally?

I identified the issue and will try to push a fix as soon as possible. I will let you know once I do.

Thank you for the fast response, Nick!
On Sat, Jun 24, 2023 at 7:26 AM Nick Dunas @.***> wrote:

I identified the issue and will try to push a fix as soon as possible. I
will let you know once I do.
—
Reply to this email directly, view it on GitHub
https://github.com/rewindui/rewindui/issues/55#issuecomment-1605403901,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACZQW26LCR7KVZOQVPR673XM3FGXANCNFSM6AAAAAAZR6SCSQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

Just verified - the fix worked. Thanks again!
On Mon, Jun 26, 2023 at 9:31 AM Nick Dunas @.***> wrote:

Hello again @ianmacartney https://github.com/ianmacartney
I have just published a new release fixing the issue you have encountered.
Please install version 0.10.1 and let me know if everything is working fine
now.
I have also updated the docs on both Tabs and Selector components giving
similar examples:
https://rewind-ui.dev/components/tabs#ref
https://rewind-ui.dev/components/selector#ref
Thank you for addressing this!
—
Reply to this email directly, view it on GitHub
https://github.com/rewindui/rewindui/issues/55#issuecomment-1607483691,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACZQW5QTADQ2JSXUCTCD4TXNGFMJANCNFSM6AAAAAAZR6SCSQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


throw error if profile not found, improve parsing of cli kwargs

improve error handling of kwargs
switch from '\n'-delimited filter, search results to ' '-delimited

Changes Unknown when pulling d8127796c8af4f5fab2fd8ad1eb3511c27a84604 on fs into ** on staging**.

Changes Unknown when pulling 18d4ec02adf0ece795877df4dcb3f06cf53ba57c on fs into ** on staging**.


Document various config parameters in https://github.com/OpenDevin/OpenDevin/blob/1fe290adf9140de639c58475023dcd57bd5bae20/opendevin/core/schema/config.py#L31

Describe your question
There is no document on config parameters.
Additional context
Encountered exception like
AgentFinishAction.__init__() got an unexpected keyword argument 'reason'

wondering if it is due to memory or bad response from llm. Want to change config parameters so see we can avoid exceptions.

Please take a look here: https://opendevin.github.io/OpenDevin/modules/python/opendevin/core/config


add assert of response code, and fix the bug of parent test

add assert of response code, and fix the bug of parent test.

Thanks @WangXiangUSTC
But CI still failed. 😭


Run-time Example - Windows 7 / Vista source

Hi
Is it possible to please share the source of the Windows 7 run example image? I cannot see how to create those awesome looking buttons!
Thanks
Originally posted: 2016-08-18T10:01:46

You can use the Vista Controls library. https://www.nuget.org/packages/WindowsFormsAero/
Originally posted: 2016-08-19T10:51:51

I wrote this for a diagnostic tool to allow me to use that style on XP as well. It will use the native button if available, and will custom draw if needed.
http://taskscheduler.codeplex.com/SourceControl/latest#TaskService/TaskSchedulerConfig/CommandLink.cs
Originally posted: 2016-08-19T15:00:53


chore(Sync Main and Release): One time fix

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v1
Commit ID: f16284dee18349a711055f9f8904c92d38a0fbd5
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


[Android] - Image is being saved in the Pictures folder even though the storageOptions is not specified

The documentation says:
storageOptions:
If this key is provided, the image will get saved in the Documents directory on iOS, and the Pictures directory on Android (rather than a temporary directory)
I'm not providing the storageOption property and the pictures are being saved in the 'Pictures' folders. ex:
var options = {
  title: 'Select Avatar',
  customButtons: {
    'Choose Photo from Facebook': 'fb',
  },
};

Shouldn't the image be saved in a temp folder as per document? or am I missing something?

With camera or gallery ?

both

it happens with both

Can you try out v0.22 on npm and see if this is still happening?

hello, i use image picker on ios, my ViewController frame is CGRectMake(0, 20, UIScreenWidth, UIScreenHeight - 20). when I present the image picker, and selected the photo or cancel, dismiss image picker, my ViewController frame is  CGRectMake(0, 20, UIScreenWidth, UIScreenHeight - 20), that is not terrible. what happend it? Can you help me? Thank you.

This should be fixed in v0.22.2

Same thing still happening with the latest version (0.28.1) on Android 5.1.1


Missing more information about testing libraries

Hi, thank you for the amazing work, it looks very promising.
I noticed the docs don't cover some more complex components, eg, a data-table.
A data table is composed of the main component (table) and children (columns) that depend directly on the parent. For such situation using shallow on the Table would render the table partially, using it on columns would directly not work.
The solution seems to wrap it all up in a dummy component and mount that component instead. But that seems not ideal for the workflow vue-test-utils suggest and I am not sure how to proceed about this, thus opening this ticket.

There was talk about including a depth option to shallow, so that you could choose how many children to render. This way you could add depth: 2 and have both your Table and Column components render.
Are you having problems using mount for this?

Thanks @eddyerburgh for your quick answer.
Not a problem, but I think I cannot use it if have children. Most likely I'm missing something here. This is the code I want to test, a simple snapshot teste.
<vk-table middle divider :data="rows">
  <vk-table-column header="Website" cell="website" />
</vk-table>

Using mount or shallow I could render vk-table, but I don't see how can I set the vk-table-column as it children.

https://vue-test-utils.vuejs.org/en/api/options.html#slots
mount(VkTable, {
  slots: {
   default: `<vk-table-column header="Website" cell="website" />`
  }
})

*(assuming vk-table was already registered as a component on VkTable

Thanks @LinusBorg, I tried this again and this time it just worked... the first time I did there was some issue that most likely confused me.
Thank you @eddyerburgh as well.

Sorry to bother again guys, but I reproduced now the issue that misled me initially.
As @LinusBorg and docs suggest the slots allows to render a component and pass it down to the mounted component. But, and at least in my tests, no matter what I set there, I got the same result in the $slots reference, a generic vnode.
At first, I thought it was because of a wrong component registration, but then I got the same results registering the component globally or locally.
This is my code so far:
import Vue from 'vue'
import { mount } from 'vue-test-utils'
import { createRenderer } from 'vue-server-renderer'

import VkTable from '~/components/table/table'
import VkTableColumn from '~/components/table/table-column'

Vue.component('VkTableColumn', VkTableColumn)

const wrapper = mount(VkTable, {
  slots: {
     default: '<vk-table-column head="Name" cell="name" />'
  }
})

Internally, VkTable will try to access VkTableColumn vnode through $slots.default and even though there is a component, its definitely not VkTableColumn.
And as mentioned, this code would behave exactly the same:
const wrapper = mount(VkTable, {
  slots: {
     default: '<div />'
  }
})

What am I missing this time :)

@miljan-aleksic When you use mount, it uses a local vue instance to do all of it's magic, so in a localized test you will need to either pass the components to the wrapper or add the Column component to your Table's components.
  const wrapper = mount(Table, {
    slots: {
      default: '<Column cell="name"></Column>'
    },
    components: [Column]
  });

Side question, if that is what you are looking for and you end up closing this could you update the title to have something about slots in it so that others will more easily find?

Hi @Austio, done but please feel free to change it as you think will best help the community.
I tried as suggested and is not working, then I tried passing a localVue registering the component to it. None does work, unfortunately. I tested with beta6.
I have few comments:

A slot that cannot resolve a component should warn about it. Among others, it will make it more easy to detect if issues like this are because of a bad configuration

Mixing mount options with Vue API is confusing. Is not clear what is being declared; a component instance or a wrapper. Separating concepts should be considered.

If mount use a local Vue by default what is for the localVue option? The docs imply polluting global Vue should be avoided, but in fact that would not matter at all as mount will never use it.

Please take my comments as observations of a first time user who is not much experienced with testing tools.

Hmmm, I don't have any other thoughts then,  could create a jsfiddle or repro with minimal needed to reproduce?

A slot that cannot resolve a component should warn about it. Among others, it will make it more easy to detect if issues like this are because of a bad configuration

The issue here isn't bad configuration, it's to do with how we compile strings that are passed in the slots option.
At the moment only basic tags are supported as strings. Under the hood, we pass them to the vue-template-compiler compileToFunctions method, but we don't pass any vm or options.
I'll add an intend to implement tag and look into fixing this so that you can use a component as the slot like you intended 🙂

Mixing mount options with Vue API is confusing. Is not clear what is being declared; a component instance or a wrapper. Separating concepts should be considered.

I agree this is problematic. When we started this project, I took the decision to merge the two for aesthetic reasons:
const wrapper = mount(Component, {
  propsData: { text: 'hello' },
  mocks: { $route }
})

const wrapper = mount(Component, {
  mocks: { $route }
}, {
  propsData: { text: 'hello' },
})

In effect, everything that isn't a vue-test-utils mounting option is passed to the component when it's instantiated. We need a longer section in the docs explaining this, but I don't think we should separate out the options.

If mount use a local Vue by default what is for the localVue option? The docs imply polluting global Vue should be avoided, but in fact that would not matter at all as mount will never use it.

localVue is created by extending the global Vue class, so anything added to the global Vue will exist in a localVue. In other words, you might as well think of it as the global Vue class. It's a localVue internally because it means we don't accidentally pollute the global Vue class when we add the mounting options.

@eddyerburgh, thank you very much for your detailed explanation and intend to implement the missing feature. I deeply appreciate it ^^


[question] SSL certs management and DNS configuration

Hi,
Thanks again for this amazing project, just created a new cluster in few minutes, it's magic!
I installed addons helm, rook, ingress and cert-manager, but I don't really known how to deal with SSL certs.

What I need to do to issue a new cert please?
I've created a HA cluster ( 3 master, etcd included, kubernetes control self-hosted ). Which master IP address I need to use in my registrar DNS configuration please?

Thanks!

Hey, thanks for the feedback.
About 1. Please take a look at the conversation in #45. There are links to resources, which describe how to either use the kube-lego style or the new way by using crd.
About 2., At the moment you should use the IP of the first master. That's because the other IPS are not setup for the other master nodes. If they would be added there, you could easily balance over these.

I put the links here for the record (not yet tested) :
https://github.com/jetstack/cert-manager/tree/master/docs/api-types/issuer
https://github.com/jetstack/cert-manager/blob/master/docs/user-guides/ingress-shim.md


Is "FREE" Windows 1-Click Installer?!

On the page is only paid option. Even 5$ is NOT "free"! Only to be correct, thanks.

@milen-prg
It's free if you set it up yourself.
RTFM

Understood.


Move Redirect handling from middleware to ...path/page and resolve them on demand

Major advantage is that we don't have to load all redirects into memory and get independent of the number of redirects.
Theoretically incompatible: if a redirect and page with the same path exist, previously the redirect was used, now the page is used.
One tricky part is that redirects are not language scoped, but pages are:

for redirects I take the language param and put append it to the source path
to support redirects with "invalid" languages (from the pagetree perspective) - for example /redirect-10 I had to avoid sending an invalid scope.language (would be redirect-10) by adding the @skip directive - else the api responds with an Access Denied error as the scope is not allowed

Theoretically incompatible: if a redirect and page with the same path exist, previously the redirect was used, now the page is used.

Since this is Demo-only, I wouldn't regard this as breaking.


spring messaging @ConnectMapping not called for routes

hey o/
not sure if I'm right here 😊
(Just thought so since it works in a plain Java App. see here )
I've got a java (spring) controller with a ConnectMapping for a custom route:
@Controller
public class WsController {

    @ConnectMapping("setup")
    public Mono connect() {
        System.out.println("Never called :-(");
        return Mono.empty();
    }
}

The JavaScript RSocketClient  sends the setup frame with the custom route:
const client = new RSocketClient({
    serializers: {
        data: JsonSerializer,
        metadata: JsonSerializer,
    },
    setup: {
        keepAlive: 999999,
        lifetime: 999999,
        dataMimeType: "application/json",
        metadataMimeType: "message/x.rsocket.routing.v0",
        payload: {
            metadata: String.fromCharCode("setup".length) + "setup"
        }
    },
    transport: new RSocketWebSocketClient(
        {
            debug: true,
            url: `ws://${window.location.host}/rsocket`,
        }
    ),
});

client.connect().then(...)

But WsController#connect is never called.
It works, however, with a default @ConnectMapping and not sending a payload within the setup frame.
I've added a sample project that can be found here: https://github.com/bseber/spring-boot-rsocket-example
Should it work with the custom route? Do I have to configure the RSocketClient somehow?

I"m getting the same issue when I use composite metadata

url: `ws://${window.location.host}/rsocket`,

url: ws://${window.location.host}:{port}/rsocket,
i think that you need insert ws's port to url

I use wss so I don't think thats the issue

I use wss so I don't think thats the issue

i'm getting same issue when using composite metadata but no problem with Seber's issue

@baudiachatb indeed, window.location.host includes the port. in my example app that would be 8080. however using the hard coded "ws://localhost:7000/rsocket" url still doesn't work.
did you checkout my example project? or do you have a working example? would be nice if this "just works" and I just have a stupid error on my side :-)

Is this the same behaviour in this stackoverflow entry ? https://stackoverflow.com/questions/63225352/connectmapping-not-getting-triggered-even-after-sending-setup-frame-from-rsocke

I use wss so I don't think thats the issue

i'm getting same issue when using composite metadata but no problem with Seber's issue

Did you use the code from @bseber exactly as he posted it? If not can you post your code here? Maybe we were missing a little piece.

Any news about this problem? Maybe it must be opened on the spring project?

Hi all,
The issue here is a few things:

It's best to use composite metadata when defining a route
When using composite metadata, its important to use IdentitySerializer for metadata

IdentitySerializer will pass the value of metadata through as-is, without any modification. This is important as the value for metadata is already encoded as it should be from encodeCompositeMetadata(...).
Please see https://github.com/bseber/spring-boot-rsocket-example/pull/1/files for example changes.

nice, thanks @viglucci!


Multi-Tenanted Rabbit MQ & Azure Service Bus connections

For a JasperFx client.
What we need for both Azure Service Bus & Rabbit MQ (for now) is the ability to specify a separate Azure Service Bus namespace or Rabbit MQ virtual host for different tenant ids.

Going well right now. Do need to add a way to "tell" individual queues or endpoints that they should opt out of being multi-tenanted


New Swift release broke everything again

No, XCGLogger still works fine with most recent release, but Swift 2.2 marked stuff which will be removed in Swift 3.0 as deprecated and XCGLogger utilizes a lot of them.
[16:31:09]: ▸ ⚠  XCGLogger.swift:592:118: __FUNCTION__ is deprecated and will be removed in Swift 3, please use #function
[16:31:09]: ▸ public class func logln(@autoclosure closure: () -> String?, logLevel: LogLevel = .Debug, functionName: String = __FUNCTION__, fileName: String = __FILE__, lineNumber: Int = __LINE__) {
[16:31:09]: ▸                                                                                ^~~~~~~~~~~~
[16:31:09]: ▸ ⚠ XCGLogger.swift:592:151: __FILE__ is deprecated and will be removed in Swift 3, please use #file
[16:31:09]: ▸ public class func logln(@autoclosure closure: () -> String?, logLevel: LogLevel = .Debug, functionName: String = __FUNCTION__, fileName: String = __FILE__, lineNumber: Int = __LINE__) {
[16:31:09]: ▸                                                                                ^~~~~~~~
[16:31:09]: ▸ ⚠  XCGLogger.swift:592:179: __LINE__ is deprecated and will be removed in Swift 3, please use #line
[16:31:09]: ▸ public class func logln(@autoclosure closure: () -> String?, logLevel: LogLevel = .Debug, functionName: String = __FUNCTION__, fileName: String = __FILE__, lineNumber: Int = __LINE__) {
[16:31:09]: ▸                                                                                ^~~~~~~~
[16:31:09]: ▸ ⚠  XCGLogger.swift:596:81: __FUNCTION__ is deprecated and will be removed in Swift 3, please use #function
[16:31:09]: ▸ public class func logln(logLevel: LogLevel = .Debug, functionName: String = __FUNCTION__, fileName: String = __FILE__, lineNumber: Int = __LINE__, @noescape closure: () -> String?) {
[16:31:09]: ▸                                                                                ^~~~~~~~~~~~
[16:31:09]: ▸ ⚠  XCGLogger.swift:596:114: __FILE__ is deprecated and will be removed in Swift 3, please use #file
[16:31:09]: ▸ public class func logln(logLevel: LogLevel = .Debug, functionName: String = __FUNCTION__, fileName: String = __FILE__, lineNumber: Int = __LINE__, @noescape closure: () -> String?) {

#124 fixes this

Thanks for this. Should be all fixed now, not sure why I left this issue open?


Added search bar in Command overview and details page

Added search bar in Command overview and details page.

It is a search-as-you-type instead of a search button. Let me know if you prefer the button.
In the details page, I only applied the search in the Match history and not on the Historical winrate. Let me know if you filter out the historical data as well.
There is no styling. Any suggestion on styling? Do we have a UX design mock? 😆

can you include a screenshot


SF unable to authenticate with primary certificate.md

"Or you can revert to the old algorithm by using following command
Set-AzureRmServiceFabricSetting -ResourceGroupName rgname -Name clustername -Section "Security" -Parameter "UseSecondaryIfNewer" -Value "false"
The above statement is kind of misleading based on my observations. After changing UseSecondaryIfNewe to false, ONLY the primary certificate will work. (The secondary will stop working)

I updated the statement to make it more clear.  You are correct in your observation, the older algorithm will only use the primary certificate.  The secondary slot was only used to stage new certs in preparation for rollover


DBZ-1895 (Skipped ops for MySQL)

https://issues.redhat.com/browse/DBZ-1895
Supersedes #1360.

@jpechane, this should be good to review now. Thanks!

@blcksrx @gunnarmorling Applied, thanks!


No documentation on combining TcpListener with non-TCP tokio types

Is it possible to run a TcpListener & another tokio type (like tokio-process, or reading from a pipe) on the same Core object?
The examples in the documentation, like from here, only ever run a single TcpListener:
fn serve<S>(s: S) -> io::Result<()>
    where S: NewService<Request = String,
                        Response = String,
                        Error = io::Error> + 'static
{
    let mut core = Core::new()?;
    let handle = core.handle();

    let address = "<IP_ADDRESS>:12345".parse().unwrap();
    let listener = TcpListener::bind(&address, &handle)?;

    let connections = listener.incoming();
    let server = connections.for_each(move |(socket, _peer_addr)| {
        let (writer, reader) = socket.framed(LineCodec).split();
        let service = s.new_service()?;

        let responses = reader.and_then(move |req| service.call(req));
        let server = writer.send_all(responses)
            .then(|_| Ok(()));
        handle.spawn(server);

        Ok(())
    });

    core.run(server)
}

Using TcpServer from tokio-proto is out of the question here, since it only runs TCP services & blocks the thread.

From IRC I've heard it could be done by spawning server with handle.spawn(server), rather than running it. Is that right? Could that feature more prominently in the documentation somewhere?

Yeah I think this is mostly a documentation issue, and this should definitely be handled in the coming weeks as we revamp the documentation. I don't think we'll gain a lot from having a specific bug like this open, though, so I'm going to close this.


New package: Golizza.RMHCentralServer version 3.50.13

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.5 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

@wingetbot waivers Add Validation-Domain

Hi @Golizza,

Since they have both server and client, the better PackageIdentifier would be Golizza.RMHCentral.Server or, since it is a GitHub release, Golizza.RMHFlash.Central.Server

I agree that this would be a better way to organize these packages. What do you think?

Manual Validation ended with:

Manual Validation ended with:

Description: A .NET application failed. Application: RMH.Central.Communication.Central.Assistant.exe Path: C:\Program Files (x86)\Retail Hero\RMH Central Flash Server\RMH.Central.Communication.Central.Assistant.exe Message: You must install or update .NET to run this application.  App: C:\Program Files (x86)\Retail Hero\RMH Central Flash Server\RMH.Central.Communication.Central.Assistant.exe Architecture: x64 Framework: 'Microsoft.AspNetCore.App', version '6.0.0' (x64) .NET location: C:\Program Files\dotnet\  No frameworks were found.  Learn about framework resolution: https://aka.ms/dotnet/app-launch-failed  To install missing framework, download: https://aka.ms/dotnet-core-applaunch?framework=Microsoft.AspNetCore.App&framework_version=6.0.0&arch=x64&rid=win10-x64

(Automated response - build 489.)

With both dependencies, it gives a different message:

thank you @stephengillie. I will close this PR and submit another later.


Create dedicated test for BeginBlocker

Currently we have logic in a transfer test to verify begin blocker is executed. We should move this out into its own dedicated test.
Originally posted by @damiannolan in https://github.com/cosmos/ibc-go/pull/4563#discussion_r1316000446

Is this issue still up for development?
If yes then would it make sense to just shift restart test to capability_test or genesis_state and use it wherever required? Seems fairly doable

Now that capabilities is removed, I'm unsure if the begin block check still exists? We can close this issue?


Bugfix: Added node properly in from_causal_graph

Description and Motivation
This PR:

Fixed a bug in cai_causal_graph.time_series_causal_graph.TimeSeriesCausalGraph.from_causal_graph for cai_causal_graph.time_series_causal_graph.TimeSeriesCausalGraph where the method was not adding the node correctly.
Added max_time_lag and min_time_lag properties to cai_causal_graph.time_series_causal_graph.TimeSeriesCausalGraph to return the maximum and minimum lag of the graph, respectively. The property maxlag was removed to avoid confusion.

Additional Context

How Has This Been Tested?

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Improvement (non-breaking change which improves functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Refactor (moving code around, general refactoring improvements)
[ ] Documentation (documentation)

PR Checklist:

[ ] I have reviewed my own PR? (review the PR as you are reviewing someone else's PR)
[ ] I have implemented all requirements? (see JIRA, project documentation)
[ ] I am affecting someone else's work? If so: I have added those people as reviewers?
[ ] I have added comments to all the bits that are hard to follow
[ ] At a bare minimum, I tested this manually
[ ] I have added unit test(s)
[ ] Is this a bugfix? If so have I added a regression test?
[ ] Does this PR satisfy the one PR, one concern rule?
[ ] If needed, documentation has been added/updated?
[ ] I have updated the appropriate changelog with a line for my changes
[ ] If this PR breaks reproducibility, have I added a note in the changelog explaining the break in reproducibility
and its extent?

Screenshots (if appropriate):

This is super confusing. The max lag will actually be the longest time forward while min lag is the longest time backwards. I think we need to change the naming a bit.
If we have X-3 -> X-2 -> X-1 -> X -> X+1 -> X+2, then max lag will be +2 and min lag will be -3. I would think max lag is -3 and max future is 2. Min lag is really -1 in this case. Just by name I would think:
max_lag = min(node.time_lag for node in nodes if node.time_lag < 0)
min_lag = max(node.time_lag for node in nodes if node.time_lag < 0)

max_future = max(node.time_lag for node in nodes if node.time_lag > 0)
min_future = min(node.time_lag for node in nodes if node.time_lag > 0)

Sorry I have not been following too deeply, but how does it make sense at all to have any notion of future?  Why would you ever want to do that?  IMO, it should just be min_lag and max_lag and those should be positive integers.

This is super confusing. The max lag will actually be the longest time forward while min lag is the longest time backwards. I think we need to change the naming a bit.
If we have X-3 -> X-2 -> X-1 -> X -> X+1 -> X+2, then max lag will be +2 and min lag will be -3. I would think max lag is -3 and max future is 2. Min lag is really -1 in this case. Just by name I would think:
max_lag = min(node.time_lag for node in nodes if node.time_lag < 0)
min_lag = max(node.time_lag for node in nodes if node.time_lag < 0)

max_future = max(node.time_lag for node in nodes if node.time_lag > 0)
min_future = min(node.time_lag for node in nodes if node.time_lag > 0)

Sorry I have not been following too deeply, but how does it make sense at all to have any notion of future? Why would you ever want to do that? IMO, it should just be min_lag and max_lag and those should be positive integers.

Yes I agree they should be positive integers. I think @matteo-causalens needs these properties to address the coerce time tiers question from Tadhg. However, the current implementation on this PR is too confusing. We need an idea of future in the TimeSeriesCausalGraph as we can extend forward and backwards in time. So future needs to exist. However, I agree on Max's point on always positive. I would define them as so:
max_lag = max(abs(node.time_lag) for node in nodes if node.time_lag < 0)
min_lag = min(abs(node.time_lag) for node in nodes if node.time_lag < 0)

Then you would get 3 for max_lag and 1 for min_lag.

This is super confusing. The max lag will actually be the longest time forward while min lag is the longest time backwards. I think we need to change the naming a bit.
If we have X-3 -> X-2 -> X-1 -> X -> X+1 -> X+2, then max lag will be +2 and min lag will be -3. I would think max lag is -3 and max future is 2. Min lag is really -1 in this case. Just by name I would think:
max_lag = min(node.time_lag for node in nodes if node.time_lag < 0)
min_lag = max(node.time_lag for node in nodes if node.time_lag < 0)

max_future = max(node.time_lag for node in nodes if node.time_lag > 0)
min_future = min(node.time_lag for node in nodes if node.time_lag > 0)

Sorry I have not been following too deeply, but how does it make sense at all to have any notion of future? Why would you ever want to do that? IMO, it should just be min_lag and max_lag and those should be positive integers.

Yes I agree they should be positive integers. I think @matteo-causalens needs these properties to address the coerce time tiers question from Tadhg. However, the current implementation on this PR is too confusing. We need an idea of future in the TimeSeriesCausalGraph as we can extend forward and backwards in time. So future needs to exist. However, I agree on Max's point on always positive. I would define them as so:
max_lag = max(abs(node.time_lag) for node in nodes if node.time_lag < 0)
min_lag = min(abs(node.time_lag) for node in nodes if node.time_lag < 0)

Then you would get 3 for max_lag and 1 for min_lag.

It is useful in other cases (not only Tadhg's request). I agree that min and max are confusing but this is due to the fact that we discussed with Marcin time ago regarding the property time_lag and are  the names should change and reflect more the idea of time. Because min and max in the current implementation are correct names actually because they are signed. If we want them to be always positive then we must have both past and future lags. But also having max_lagand min_lag are confusion both in terms of names and in terms of value if you want to refer them as past value.
Moreover, I don't need the intermediate ones (i.e. min_lag and min_future) and I don't see them super useful but I can add them.
So what I would propose is max_past_lag and max_future_lag, both positive.

This is super confusing. The max lag will actually be the longest time forward while min lag is the longest time backwards. I think we need to change the naming a bit.
If we have X-3 -> X-2 -> X-1 -> X -> X+1 -> X+2, then max lag will be +2 and min lag will be -3. I would think max lag is -3 and max future is 2. Min lag is really -1 in this case. Just by name I would think:
max_lag = min(node.time_lag for node in nodes if node.time_lag < 0)
min_lag = max(node.time_lag for node in nodes if node.time_lag < 0)

max_future = max(node.time_lag for node in nodes if node.time_lag > 0)
min_future = min(node.time_lag for node in nodes if node.time_lag > 0)

Sorry I have not been following too deeply, but how does it make sense at all to have any notion of future? Why would you ever want to do that? IMO, it should just be min_lag and max_lag and those should be positive integers.

Yes I agree they should be positive integers. I think @matteo-causalens needs these properties to address the coerce time tiers question from Tadhg. However, the current implementation on this PR is too confusing. We need an idea of future in the TimeSeriesCausalGraph as we can extend forward and backwards in time. So future needs to exist. However, I agree on Max's point on always positive. I would define them as so:
max_lag = max(abs(node.time_lag) for node in nodes if node.time_lag < 0)
min_lag = min(abs(node.time_lag) for node in nodes if node.time_lag < 0)

Then you would get 3 for max_lag and 1 for min_lag.

It is useful in other cases (not only Tadhg's request). I agree that min and max are confusing but this is due to the fact that we discussed with Marcin time ago regarding the property time_lag and are the names should change and reflect more the idea of time. Because min and max in the current implementation are correct names actually because they are signed. If we want them to be always positive then we must have both past and future lags. But also having max_lagand min_lag are confusion both in terms of names and in terms of value if you want to refer them as past value. Moreover, I don't need the intermediate ones (i.e. min_lag and min_future) and I don't see them super useful but I can add them.
So what I would propose is max_past_lag and max_future_lag, both positive.

As I already mentioned to Andrew, I would like to be aware of the sign lags sometimes. For example, the graph X(t-3) -> X(t-2) would have -3 and -2 in my implementation. Instead if we move to abs values we loose this information

This is super confusing. The max lag will actually be the longest time forward while min lag is the longest time backwards. I think we need to change the naming a bit.
If we have X-3 -> X-2 -> X-1 -> X -> X+1 -> X+2, then max lag will be +2 and min lag will be -3. I would think max lag is -3 and max future is 2. Min lag is really -1 in this case. Just by name I would think:
max_lag = min(node.time_lag for node in nodes if node.time_lag < 0)
min_lag = max(node.time_lag for node in nodes if node.time_lag < 0)

max_future = max(node.time_lag for node in nodes if node.time_lag > 0)
min_future = min(node.time_lag for node in nodes if node.time_lag > 0)

Sorry I have not been following too deeply, but how does it make sense at all to have any notion of future? Why would you ever want to do that? IMO, it should just be min_lag and max_lag and those should be positive integers.

Yes I agree they should be positive integers. I think @matteo-causalens needs these properties to address the coerce time tiers question from Tadhg. However, the current implementation on this PR is too confusing. We need an idea of future in the TimeSeriesCausalGraph as we can extend forward and backwards in time. So future needs to exist. However, I agree on Max's point on always positive. I would define them as so:
max_lag = max(abs(node.time_lag) for node in nodes if node.time_lag < 0)
min_lag = min(abs(node.time_lag) for node in nodes if node.time_lag < 0)

Then you would get 3 for max_lag and 1 for min_lag.

It is useful in other cases (not only Tadhg's request). I agree that min and max are confusing but this is due to the fact that we discussed with Marcin time ago regarding the property time_lag and are the names should change and reflect more the idea of time. Because min and max in the current implementation are correct names actually because they are signed. If we want them to be always positive then we must have both past and future lags. But also having max_lagand min_lag are confusion both in terms of names and in terms of value if you want to refer them as past value. Moreover, I don't need the intermediate ones (i.e. min_lag and min_future) and I don't see them super useful but I can add them.
So what I would propose is max_past_lag and max_future_lag, both positive.

As I already mentioned to Andrew, I would like to be aware of the sign lags sometimes. For example, the graph X(t-3) -> X(t-2) would have -3 and -2 in my implementation. Instead if we move to abs values we lose this information

For extending the graph it's not an issue as we compute the minimal graph first (X(t-1) -> X(t)) and then we do the rest. So for now we can leave it as it is but I think we should revisit this in the future together with the time_lag name as @mmiklitz already suggested.


Move whatsnew items to correct folder

Checks failed.... 😕

(timeout exception).
Happy to accept this as it stands.


Enhancement for excludeNotDocumented option

Search Terms
excludeNotDocumented, option
Problem
I have a codebase with large set of non documented functions, classes, e.t.c

If excludeNotDocumented = true I can see only documented classes, functions. But even if a whole class has documentation his properties, methods without docs are hidden.
excludeNotDocumented = false I can see all non-documented properties, methods from documented classes but at the same time I see a rest of non-documented functions, classes which I don't want to include.

Suggested Solution
A mode when class, enum, interface, ... shows all it non documented members if it was documented on top level

Hmm, this ideally should be configurable at a more granular level, where you can specify what kinds of reflections should be removed by excludeNotDocumented... excludeNotDocumentedKinds isn't a horrible name, having difficulty coming up with something better.


added initial view toggle author control

The SEO team wants to help us get the multi-image thumbnails on SERP:

Our current templates pages load the images too small. With this code change, the authors will have control over which size we render the templates initially on page load.
More details in ticket.
Please test using mobile viewport.
Resolves: MWPW-141135
Test URLs:

Before: https://main--express--adobecom.hlx.page/express/templates/brochure/company?martech=off
After: https://template-size-change--express--adobecom.hlx.page/express/templates/brochure/company?martech=off

Codecov Report
Attention: 18 lines in your changes are missing coverage. Please review.

Comparison is base (4d33185) 33.82% compared to head (e854d52) 33.76%.
Report is 10 commits behind head on stage.

Files
Patch %
Lines

express/blocks/template-list/template-list.js
0.00%
11 Missing :warning:

express/blocks/template-x/template-x.js
0.00%
7 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##            stage     #653      +/-   ##
==========================================
- Coverage   33.82%   33.76%   -0.07%
==========================================
  Files         131      129       -2
  Lines       25423    24897     -526
==========================================
- Hits         8599     8406     -193
+ Misses      16824    16491     -333

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


[deploy] Host preinstall CUDA checking

Problem
previous IT cluster issue:
Potential suggestion enhance pai
checking at driver installation or probe to tell deployer to keep clean of host CUDA.
quick fix for current user
remove cuda and reboot server could temp fix this error
Our exp:
driver installed by pod or use users
CUDA and cuDNN is installed by in job container dockerfile.

not our obligation.


⚠ 한경주요지수-코스피200 has degraded performance

In c81f2cc, 한경주요지수-코스피200 (https://markets.hankyung.com/indices/kospi200) experienced degraded performance:

HTTP code: 200
Response time: 16410 ms

Resolved: 한경주요지수-코스피200 performance has improved in 48d3a39.


(apache-httpd) Fixed compatibility with Powershell v2

Fixed the non-Powershell v2 compatibility comlet of Get-NetTCPConnection
@majkinetor @AdmiringWorm @gep13

Description
This solution uses netstat for retrieval of the information requested using the windows 8 or newer system comlet of Get-NetTCPConnection. Has been tested on a windows 7 machine.

Motivation and Context

How Has this Been Tested?

Screenshot (if appropriate, usually isn't needed):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Migrated package (a package has been migrated from another repository)

Checklist:

[x] My code follows the code style of this repository.
[ ] My change requires a change to documentation (this usually means the notes in the description of a package).
[ ] I have updated the documentation accordingly (this usually means the notes in the description of a package).
[x] All files are up to date with the latest Contributing Guidelines
[ ] The added/modified package passed install/uninstall in the chocolatey test environment.
[x] The changes only affect a single package (not including meta package).

Original Location

Original Repository
Open Issues Add the different issues underneath, and tick those that are fixed in this PR

[X] Issue #828
[ ] Issue 2 Link

[ ] Include the link to the opened PR that removes the package from the original location
[ ] The migration guidelines have been followed

:white_check_mark: Package verification completed without issues. PR is now pending human review

:white_check_mark: Package verification completed without issues. PR is now pending human review

@majkinetor
Changes made and Appveyor agrees 👍

:x: Package verification failed, please review the Appveyor Logs and the provided Artifacts before requesting a human reviewer to take a look.

:white_check_mark: Package verification completed without issues. PR is now pending human review

@majkinetor
I made the match so it will old match if that number is found not inside a port

@majkinetor
Fixed code style and indentation

:white_check_mark: Package verification completed without issues. PR is now pending human review


Broken wiki example

The following example from the wiki
mf(vi, sampler; x=[1.5, 2.0]) = begin
  s = Turing.assume(sampler,
                    InverseGamma(2, 3),
                    Turing.VarName(vi, [:c_s, :s], ""),
                    vi)
  m = Turing.assume(sampler,
                    Normal(0,sqrt(s)),
                    Turing.VarName(vi, [:c_m, :m], ""),
                    vi)
  for i = 1:2
    Turing.observe(sampler,
                   Normal(m, sqrt(s)),
                   x[i],
                   vi)
  end
  vi
end
mf(Turing.VarInfo(), nothing)

throws the following error:
ERROR: MethodError: no method matching sqrt(::Tuple{Float64,Float64})
Closest candidates are:
  sqrt(::Float64) at math.jl:425
  sqrt(::Float32) at math.jl:426
  sqrt(::Float16) at math.jl:950
  ...
Stacktrace:
 [1] #mf#1(::Array{Float64,1}, ::Function, ::Turing.VarReplay.VarInfo, ::Void) at ./REPL[3]:6
 [2] mf(::Turing.VarReplay.VarInfo, ::Void) at ./REPL[3]:1

The culprit appears to be that s is a tuple rather than a Float64. What is the appropriate work around? This example worked a couple of weeks ago; what changes to the internals have occurred to cause this problem?

In the last PR I changed the API of assume, observe a bit. Basically now they also return log-probability: I evaluate the log-probability inside the function and return. Coming with this change, I set a local variable _lp inside the model function.
The reason to do this is that ReverseDiff.jl somehow is very slow with the type is unsafe and using this local variable to accumulate probability somehow makes Turing.jl work fast with ReverseDiff.jl. Though I didn't figure out why this really happened, I guess it's something related with type stability (see related issues #400, https://github.com/yebai/Turing.jl/issues/331#issuecomment-367823550)
So now this program should be changed to below
mf(vi, sampler; x=[1.5, 2.0]) = begin
  _lp = zero(Real)

  s, __lp = Turing.assume(sampler,
                    InverseGamma(2, 3),
                    Turing.VarName(vi, [:c_s, :s], ""),
                    vi)
  _lp += __lp
  m, __lp = Turing.assume(sampler,
                    Normal(0,sqrt(s)),
                    Turing.VarName(vi, [:c_m, :m], ""),
                    vi)
  _lp += __lp
  for i = 1:2
    _lp += Turing.observe(sampler,
                   Normal(m, sqrt(s)),
                   x[i],
                   vi)
  end
  vi.logp = _lp
  vi
end
mf(Turing.VarInfo(), nothing)

Tested in commit 43017ad8ded8036e2c763aabc1cec245792c8764 as the current master seems broken?

Makes sense, thanks for the quick response.


window is not defined

I feel there must be more to this error than what is described in the docs. I have spent a long time trying to get various scripts to work in Nuxt, and each time have failed to get passed the window is not defined error, despite adding:
if (process.BROWSER_BUILD) {
  require('external_library')
}

I won't list all the scripts I tried here, but I'll give one example: Vue.Isotope
nuxt.config.js
build: {
  vendor: [ 'vueisotope'],
}

.vue file
if (process.BROWSER_BUILD) {
  require('vueisotope')
}
import isotope from 'vueisotope'
export default {
  components: {
    isotope
  }
}

Hi , I tried your example component .
I have interest . especially I will use Isotope layout near future.
I provide my try for your reference
yarn add vueisotope

index.vue
<template lang="pug">
section
  vueisotope(:list="list",id="root_isotope",class="isoDefault",:options='option')
    div(v-for="element in list", @click="selected=element", :key="element.id")
      | {{element.name}}
      br
      | {{element.id}}

</template>
<script>
const vueisotope = process.BROWSER_BUILD ? require('vueisotope') : '';

export default {
  data() {
    return {
      list: [{ name: 'John', id: 25 },
      { name: 'Joao', id: 7 },
      { name: 'Albert', id: 12 },
      { name: 'Jean', id: 100 }],
      selected: null,
      option: {
        getSortData: {
          id: 'id',
        },
        sortBy: 'id',
      },
    };
  },
  head() {
    return {
      title: 'sitename',
    };
  },
  components: {
    vueisotope,
  },
};
</script>

I used eslint-config-airbnb-base , so , I need use global import here.
but I got some error message .
ERROR in ./~/isotope-layout/js/layout-modes/masonry.js
Module not found: Error: Can't resolve 'masonry/masonry' in 'C:\Users\ausir\nuxt-axios-exercise\node_modules\isotope-layout\js\layout-modes'
 @ ./~/isotope-layout/js/layout-modes/masonry.js 12:4-16:15
 @ ./~/isotope-layout/js/isotope.js
 @ ./~/vueisotope/dist/vue_isotope.min.js

so I tried find source , and I found this
http://isotope.metafizzy.co/extras.html#webpack
masonry.js
define( [
        '../layout-mode',
        'masonry/masonry'
      ],

so, I add 2 alias in nuxt.config..js
build: {
    /*
    ** Run ESLINT on save
    */
    extend(config, ctx) {
      config.resolve.alias['masonry'] = 'masonry-layout';  // <-- this
      config.resolve.alias['isotope'] = 'isotope-layout';  // <-- and this
      if (ctx.isClient) {
        config.module.rules.push({
          enforce: 'pre',
          test: /\.(js|vue)$/,
          loader: 'eslint-loader',
          exclude: /(node_modules)/,
        });
      }
    },
  },

vueisotope is work !
but I didn't try generate func yet ..

Thanks for the info @ausir0726
So it looks like the key difference is
const vueisotope = process.BROWSER_BUILD ? require('vueisotope') : '';

VS
if (process.BROWSER_BUILD) {
  require('vueisotope')
}
import isotope from 'vueisotope'

Maybe the former should be in the docs?

import need in first ( head ) line
then follow require .
but import will have error for SSR
so we need if ( process.BROWSER_BUILD )
but import can't use in if condition
so we just can use require
but for my eslint-config-airbnb-base rules
require also can't write in if(){} condition ( global-requrie )
so , I need wirte const vueisotope = process.BROWSER_BUILD ? require('vueisotope') : '';
then I can pass airbnb-base linter .
if you just use eslint-config-standard
I think you can just write require in if condition
if (process.BROWSER_BUILD) {
  const isotope  = require('vueisotope');
}

It should be work ...

for some library you don't a variable or import
but usually need require assign variable
like. sweetalert2  smoothscroll-polyfill

Thank you @ausir0726 for your help
In the upcoming release, we are adding an ssroption in the plugins key of the nuxt.config.js, so you will be able to integrate Vue.Isotope this way @smth:
nuxt.config.js
module.exports = {
  plugins: [
    { src: '~plugins/vue-isotope.js', ssr: false }
  ],
  build: {
    vendor: ['vueisotope'],
    extend (config, { isClient }) {
      if (isClient) {
        config.resolve.alias = Object.assign(config.resolve.alias, {
          'masonry': 'masonry-layout',
          'isotope': 'isotope-layout'
        })
      }
    }
  }
}

plugins/vue-isotope.js
import Vue from 'vue'
import isotope from 'vueisotope'

Vue.component('isotope', isotope)

This option ssr: false is required until Vue.Isotope becomes compatible with SSR.
In the meantime, you will have this warning from Vue:

[Vue warn]: The client-side rendered virtual DOM tree is not matching server-rendered content. This is likely caused by incorrect HTML markup, for example nesting block-level elements inside <p>, or missing <tbody>. Bailing hydration and performing full client-side render.

It still doesn't work when using generate! Please reopen issue
ERROR in ./node_modules/isotope-layout/js/layout-modes/masonry.js
Module not found: Error: Can't resolve 'masonry/masonry' in '/nuxt-examples/node_modules/isotope-layout/js/layout-modes

Sorry, I fix it already. I had a wrong condition in nuxt.config if (ctx.dev && ctx.isClient) so it was working only in Dev.


Fixed multiple instantiations of DependencyTrackingTelemetryModule

Fix Issue https://github.com/Microsoft/ApplicationInsights-dotnet/issues/613.
Prevents multiple instantiations of DependencyTrackingTelemetryModule.

[ ] I ran Unit Tests locally.

For significant contributions please make sure you have completed the following items:

[ ] Changes in public surface reviewed

[ ] Design discussion issue #

[ ] CHANGELOG.md updated with one line description of the fix, and a link to the original issue.

[ ] The PR will trigger build, unit tests, and functional tests automatically. If your PR was submitted from fork - mention one of committers to initiate the build for you.
If you want to to re-run the build/tests, the easiest way is to simply Close and Re-Open this same PR. (Just click 'close pull request' followed by 'open pull request' buttons at the bottom of the PR)

Please follow [these] (https://github.com/Microsoft/ApplicationInsights-aspnetcore/blob/develop/Readme.md) instructions to build and test locally.

This fix is incomplete, as it does not deal with improper disposing of DependencyTrackingTelemetryModule which disposes HttpDesktopDiagnosticSourceListener ...

@mkosieradzki Thank you for your contribution to SDK!
From here,
https://github.com/Microsoft/ApplicationInsights-aspnetcore/blob/develop/src/Microsoft.ApplicationInsights.AspNetCore/Extensions/ApplicationInsightsExtensions.cs#L128
it looks like AddApplicationInsightsTelemetry is already idempotent.
Can you give more information about the specific problem you are hitting with repo steps.? So we can understand better whats wrong and then decide on a fix.

Closing in favor of proper fix in #622


add kafkauser csr approval support

Description
Add CertificateSigningRequests approval support for KafkaUser resources.
It's used when pkiBackend is k8s-csr, and signerName contains clusterissuers.cert-manager.io. So, it can be used in parallel with the csr-operator.
Type of Change

[ ] Bug Fix
[x] New Feature
[ ] Breaking Change
[ ] Refactor
[ ] Documentation
[ ] Other (please describe)

Checklist

[x] I have read the contributing guidelines
[x] I have verified this change is not present in other open pull requests
[x] All code style checks pass
[x] All new and existing tests pass

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


F0130 complete draw linear

ready to merge into master

Coverage remained the same at 100.0% when pulling c963bb4aa3e6c222196bd062d53b1b025d530a6e on F0130_completeDrawLinear into f12e6ec40275206d1c144200d8f047727c957809 on master.


Write Wiki to have instructions for ArcGIS Pro instead or in addition to ArcMap

@jdrisdelle I think the Wiki should reference how to do this in ArcGIS Pro. That's what I am using and some other people that I work with are using. Also, we submit RA jobs from ArcGIS Pro, so it would be beneficial to have the instructions written from ArcGIS Pro.

I will update the wiki @gbrunner

Added a section to the wiki for using python raster function in Pro.


Possible shape issue when using CRF Loss

I get the following error
ValueError: weights can not be broadcast to values. values.rank=0. weights.rank=2. values.shape=(). weights.shape=(None, 200).
When running this simple sample model
_________________________________________________________________
Layer (type)                 Output Shape              Param #
=================================================================
word_input (InputLayer)      [(None, 200)]             0
_________________________________________________________________
embedding (Embedding)        (None, 200, 100)          100000
_________________________________________________________________
time_distributed_4 (TimeDist (None, 200, 7)            707
_________________________________________________________________
forward_crf (CRF)            (None, 200, 7)            49
=================================================================

Using categorical crossentropy lets it train with no issue, but using the loss from the forward_crf layer, give the error instantly. Could this be an issue in the CRF loss?

Could this possibly be related? https://github.com/tensorflow/neural-structured-learning/issues/21

I run the test code and there is no problem. Could you give me an example code?


release 0.25.0

We need an official release which includes the contrib/exporters/uber binary and configuration files.
Target date is 15.9.2019.

not relevant anymore as I intent pushing binaries to skydive-binaries repo


Qpdf view text in text editor

i have used qpdf  with

qpdf --qdf orig.pdf expanded.pdf

I can see all objects in text editor. But i cannot view the text in the pdf.
I have test with "Hello world" text in pdf. But i can't see this text after decompress. can someone help, how to view the text with objects
Thank you.
File tried with
Pdf file

This isn't something qpdf can do directly. Text in PDF content streams is often encoded using subsetted fonts where you have to look up each byte in a table to map it into the character, and even when the text appears directly, it is often placed on the page character by character to achieve specific text positioning, so you might not be able to discern individual words. If you were trying to really understand text encoding in PDF, qpdf in qdf mode would help, but if you're just trying to extract text, it will only work for a small class of PDF files.
There are some tools, like ghostscript, that come with pdf extractors. This is somewhat outside my area of expertise though -- probably a good google search would yield better results. I'm just confirming here that just looking at the uncompressed PDF content stream data is not sufficient in the general case to see the text.
I'll close this issue, but feel free to reopen if you have any qpdf-specific concerns.


support attachments

this is not yet reading attachments, which breaks images, audio messages and link enrichment

fixed by https://github.com/phonetonote/phonetonote-logseq/commit/979ff75e2fa31460fb1b4c8614b04cf0eea4f03f


MSYS2 libdwarf support

is possible add option to use libdwarf  on windows msys2 platform ?
at this moment  BACKWARD_HAS_DWARF is inside BACKWARD_SYSTEM_LINUX
I try this https://github.com/davea42/libdwarf-code
and sample dwarfdump display all symbol in exe

Indeed right now it is specific to Unix/Linux. The choice of which
implementation to compile is done with the preprocessor.  You could change
the ifdefs in backward.hpp to see how much change would be required.
On Sat, Jun 24, 2023, 02:56 mariuszmaximus @.***> wrote:

is possible add option to use libdwarf on windows msys2 platform ?
at this moment BACKWARD_HAS_DWARF is inside BACKWARD_SYSTEM_LINUX
I try this https://github.com/davea42/libdwarf-code
and sample dwarfdump display all symbol in exe
—
Reply to this email directly, view it on GitHub
https://github.com/bombela/backward-cpp/issues/300, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABUZDCWFDYJXYFGO7NYMHLXM22VRANCNFSM6AAAAAAZSNKXAY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I have my first experiments
Build with BACKWARD_HAS_DWARF on msys2  and don't link msvcr90
not everything works yet :(
no line number :(
Problem is here:
    // Try to get an ELF handle. We need to read the ELF sections
    // because we want to see if there is a .gnu_debuglink section
    // that points to a split debug file
    dwarf_elf_t elf_handle;
    elf_handle.reset(elf_begin(file_handle.get(), ELF_C_READ, NULL));
    if (!elf_handle) {
      return r;
    }

Always return r;
I will think about it

Right, because on MS Windows the executable should be a PE (Portable
Executable) file, not ELF (Executable and Linkable Format).The fact it
compiled and run is promising.
On Tue, Jun 27, 2023, 14:26 mariuszmaximus @.***> wrote:

I have my first experiments
Build with BACKWARD_HAS_DWARF on msys2 and don't link msvcr90
not everything works yet :(
no line number :(
Problem is here:
// Try to get an ELF handle. We need to read the ELF sections
// because we want to see if there is a .gnu_debuglink section
// that points to a split debug file
dwarf_elf_t elf_handle;
elf_handle.reset(elf_begin(file_handle.get(), ELF_C_READ, NULL));
if (!elf_handle) {
  return r;
}

Always return r;
I will think about it
—
Reply to this email directly, view it on GitHub
https://github.com/bombela/backward-cpp/issues/300#issuecomment-1610242231,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABUZDGCOKX2HGZPOYIIBRDXNNFY3ANCNFSM6AAAAAAZSNKXAY
.
You are receiving this because you commented.Message ID:
@.***>

Mea culpa I look at samples in https://github.com/davea42/libdwarf-code
and cmake find libelf
Now I checked ! libelf  is not used in any way in libdwarf-code  samples
I gave up with libdwarf
I found a better solution:https://github.com/bombela/backward-cpp/pull/301


C Program Display Its Code as Output

A Simple C Program Which Display Its source Code as Output

@shivamsdk It is automated script using python
Add projects or find bugs related to these


Enhancement: Allow filtering and sorting other watched lists #276

Changes made
Added a new store variable 'watchedListErrorOccurred' set to true if getPublicWatchedList() promise errors.
The aim is that the filter and sort functionality only show if the watched list loads in successfully.

Hi @IRHM - I totally agree. Upon further consideration, watchedListErrorOccurred adds unnecessary complexity. I've updated the code accordingly. I've made the changes and pushed the new commit just now for your review and feedback.

Thank you very much @iamericfletcher, this works great!