Should we fork and embed h5ai-nginx subrepo?

Currently some UI changes are applied to the subrepo. This changes are made by applying a patch, which leaves the repo as dirty.
Instead of this approach should we fork the h5ai-nginx subrepo and apply and commit those changes?

Closing this for now given that the patch approach is working fine at the moment and it doesn't prevent to customize it

https://github.com/hashgraph/hedera-sourcify/issues/83
https://github.com/hashgraph/hedera-sourcify/issues/91
https://github.com/hashgraph/hedera-sourcify/issues/101
https://github.com/hashgraph/hedera-sourcify/issues/103
https://github.com/hashgraph/hedera-sourcify/issues/105


Add support for key management

This includes:

[x] Generating a root key phrase for a new wallet
[x] Allowing input of an existing root key phrase for an existing wallet
[x] Generating public/private key pairs.

The interface to Keychain is implemented. I'm leaving the secure management of the secret mnemonic until it's actually used in a controller. After getting some real-world usage out of this design, we may want to reconsider using Strings as keys, since everywhere else they're just treated as raw Data objects.


Create LICENSE

This is a pull request. Those requests are  git's method of making reviews. In a pull request someone proposes a modification or revision in a branch of a project. The request is sent to reviewers whose job it is to ensure that the revision improves the project. I usually send pull request to fellow project workers. In this case I feel that the revision I propose is more significant then usual, and it is also less technical (or less artistic).
This branch is called add-license-1. The revision I propose is that we add a LICENSE to the SKS_tei data source project. It is a standard CC0 license. The reason for this is that people has asked (recently someone in New Mexico asked wondered about it).
Also I want  all you and KB to be aware that

the SKS_tei data is in a public github project. It has been so since September 2017. I have informed anyone that I believed could have opinions about this.
if you accept this pull request the source will be available on the same terms as the SKS content in text service. Note the formulation: "Materialet er dedikeret til public domain. Man kan kopiere, ændre, distribuere og fremføre værket, også til kommercielle formål, uden at bede om tilladelse. Husk dog altid at kreditere ophavsmanden." (https://tekster.kb.dk/sks)

If you so wish, we can merge the branch add-license-1 into the master. Then CC0 will become the license for SKS_tei data source. Alternatively, we can close the pull request, and make the data in this project private

I am (of course) of the opinion that there should be a license and that the license should indeed be CC0. But I am not a lawyer.
Only minor thing I have to add is that crediting the author is optional under CC0 as well as Public Domain, så maybe it should be "Det anses for god skik at kreditere ophavsmanden", adding who the ophavsmand is in this scenario. I presume it is Søren, but I am not sure.

It is totally correct according to KB’s strategy that we publicize license conditions whenever we publicize KB’s collections.  So Sigge is right, we need a license file.
In the SKS case it also correct that it’s a CC0 license.
The wording of the KB’s CC0 license description on KB’s sites has been extensively discussed internally amongst KB’s legal team.
Unfortunately, (as always) they have produced only a Danish text, which is the following:

Ophavsret
Materialet er dedikeret til public domain. Du kan kopiere, ændre, distribuere og fremføre værket, også til kommercielle formål, uden at bede om tilladelse. Husk dog altid at kreditere ophavsmanden.
Læs CC0-erklæringen

It can be seen, for instance, on the new SKS pages in tekster.kb.dk:  https://tekster.kb.dk/text/sks-pcs-txt-root    (click the ‘Info’ button)
Note that our Danish text includes ”Husk dog altid at kreditere ophavsmanden”, which is exactly what Toke suggests.
I would like that to be added to the CC0 text in Github.  We need KB’s legal team to approve/adjust the text in Github.  They (Louise A.) are already on it.

Note that our Danish text includes ”Husk dog altid at kreditere ophavsmanden”, which is exactly what Toke suggests.

Sorry, but it is not. My point is the opposite: CC0 is the only CC-license that does not require attribution. At least not from a legal point of view. I was asking for

A wording that is more explicit about it being a norm rather than a rule
Who (and how) to credit

References:

https://wiki.creativecommons.org/wiki/CC0_FAQ#Does_CC0_require_others_who_use_my_work_to_give_me_attribution.3F
https://wiki.creativecommons.org/wiki/CC0_FAQ#Can_I_control_how_my_work_is_being_used_once_I_publish_it_using_CC0.3F

Legalese is not any of my strengths. Louise said that attribution is necessary according to Danish law. But @tokee is (obviously) correct when it comes to CC0. (I also noted CC's wording; "Unlike our licenses, there are no conditions contained in CC0." Hence, CC0 is not a CC license.)

And you cannot quote something in an academic context without a proper reference....

After discussions with Lousie, I've merged this.

Fine.  Yes, Louise must decide.

It is totally correct according to KB’s strategy that we publicize license conditions whenever we publicize KB’s collections.  So Sigge is right, we need a license file.
In the SKS case it also correct that it’s a CC0 license.
The wording of the KB’s CC0 license description on KB’s sites has been extensively discussed internally amongst KB’s legal team.
Unfortunately, (as always) they have produced only a Danish text, which is the following:
Ophavsret
Materialet er dedikeret til public domain. Du kan kopiere, ændre, distribuere og fremføre værket, også til kommercielle formål, uden at bede om tilladelse. Husk dog altid at kreditere ophavsmanden.
Læs CC0-erklæringenhttps://creativecommons.org/publicdomain/zero/1.0/deed.da
It can be seen, for instance, on the new SKS pages in tekster.kb.dk:  https://tekster.kb.dk/text/sks-pcs-txt-root    (click the ‘Info’ button)
Note that our Danish text includes ”Husk dog altid at kreditere ophavsmanden”, which is exactly what Toke suggests.
I would like that to be added to the CC0 text in Github.  We need KB’s legal team to approve/adjust the text in Github.  They (Louise A.) are already on it.
Best
Bo
DKU
KB
Tel:  2043 7558
From: Toke Eskildsen @.>
Sent: Tuesday, September 13, 2022 8:49 PM
To: kb-dk/SKS_tei @.>
Cc: Bo Krantz Simonsen @.>; Review requested @.>
Subject: Re: [kb-dk/SKS_tei] Create LICENSE (PR #13)
I am (of course) of the opinion that there should be a license and that the license should indeed be CC0. But I am not a lawyer.
Only minor thing I have to add is that crediting the author is optional under CC0 as well as Public Domain, så maybe it should be "Det anses for god skik at kreditere ophavsmanden", adding who the ophavsmand is in this scenario. I presume it is Søren, but I am not sure.
—
Reply to this email directly, view it on GitHubhttps://github.com/kb-dk/SKS_tei/pull/13#issuecomment-1245825797, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACYSQKICFMJ2KY6XJPPUWODV6DEDTANCNFSM6AAAAAAQLNHDTQ.
You are receiving this because your review was requested.Message ID<EMAIL_ADDRESS>


Setting Project Properties

Useful if you take advantage of property expansion in your SoapUI tests.

cool, done!


[ja] update regular cadence link (version-skew-policy.md)

en PR https://github.com/kubernetes/website/pull/33920/files
/assign @atoato88
/assign @nasa9084

Can you share the issue you are trying to resolve?

Can you share the issue you are trying to resolve?

i sync small PR with english version
details here https://github.com/kubernetes/website/pull/34245#issue-1268377534

/lgtm


Ingestion fails with ledgers not contiguous

What version are you using?
21.0.1
What did you do?
Restart rpc (as part of a K8s cluster upgrade)
What did you expect to see?
Ingestion working
What did you see instead?
time="2024-05-09T18:24:36.715Z" level=error msg="could not run ingestion. Retrying" error="error appending ledgers: ledgers not contiguous: expected ledger sequence 51590893 but received 51594383" pid=1

This happens as part of eventStore.IngestEvents()
This happens after loading the in-memory data structures and Core is running.
Root case theories:

Disconnect between the ledger range in the DB and the one obtained from captive core
DB Corruption
Events in-memory data-structure corruption or malfunctioning

Root case theories:

It was none of the above.
It turns out that the in-memory loading timed out, but we were not reporting the error correctly (PR coming up)
As a result, the in-memory data-structure didn't get to load the full ledger range from the DB (in the error above it only loaded up to ledger 51590893 instead of 51594383)
Later, we attempted to continue loading the in-memory data structure from captive core (which we told to start at the DB boundary: 51594383).
Since the in-memory data structure was only loaded up to 51590893 (due to the time out), it resulted in an error.
We should also extend the default ingestion time out.


cf dev start fails on windows 10 pro 64 bits: Error: failed to provision VM: Process exited with: 127. Reason was: ()

I'm trying to get CF-DEV to run on my local machine but it fails on the same spot:

I run MS Windows 10 Pro, 64 bits edition with an Intel Core i7-4720HQ and 16GB (DDR3) memory. Image files are stored on a RAID 0 drive (using 2 SSD drives). I'm connected to the internet without using a proxy on a wired connection (to prevent time-outs or slow connections)
What I've tried so far?

I've switched (removed) off: Hyper-V. (it was conflicting)
Removed native Docker support.
Ubuntu is still part of the install but I doubt it is causing the issue.
removed virtualBox and installed latest version. (to be sure)
downloaded CF tool and CF-dev again.
tried CMD and powershell
ran cf dev start with 8192MB memory (instead of 4GB it assigned itself)
tried running with no services: cf dev start -s none
Modified the vm NAT (shouldn't be a solution): VBoxManage modifyvm "pcfdev-v0.529.0" --natdnshostresolver1 on
removed all old VM, OVAs and retried the install from scratch again

and all resulted in the same error: Error: failed to provision VM: Process exited with: 127. Reason was: ()
What does work: I can start the image from virtualBox manager (GUI) but can't login on the console:

(admin/admin tried as I found in a different issue as a way to login?)
When dev is started in virtualBox Manager and stopped through the console (cf dev stop) It works as expected. If I try (and fail) to start the dev env from the console (cf dev start) and then try to stop it, I get a new error from VirtualBox:

and:

(instruction xxxxx is always pointing to memory on 0xFFFFFFFFFFFFFFFFFFFFFFFFFF.)

Can you provide the full output of cf dev destroy followed by cf dev start?
Then run cf dev debug, make sure the resulting archive doesn't have an sensitive data in it, and send it to me at<EMAIL_ADDRESS>

@javafreek, I think this is the same issue that I reported a few days ago. #255

I can confirm that this is the same issue. Even though you mention having a stable internet connect, the VM fails to resolve a Github domain during the provisioning stage:
curl: (6) Could not resolve host: raw.githubusercontent.com

This request shouldn't happen. We're going to cut a new release (0.28.0) that removes it.
Closing in favor of #255.


update usefull_links

Appreciate your contribution.
If you have more, do it at once. Please don't generate PR for same resources update. I am not merging for now.

i have for diffrent subjects
On Fri, Oct 13, 2017 at 7:18 PM, Chintan Rathod<EMAIL_ADDRESS>wrote:

Appreciate your contribution.
If you have more, do it at once. Please don't generate PR for same
resources update. I am not merging for now.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/GDGAhmedabad/Learning-Resources/pull/236#issuecomment-336457335,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALORTz38bJ60-He9fC3hhRiA-3gPHJkCks5sr2oYgaJpZM4P4gmC
.

--
[image: --]
Vyas Pranjal
[image: https://]about.me/vyaspranjal
https://about.me/vyaspranjal?promo=email_sig&utm_source=email_sig&utm_medium=email_sig&utm_campaign=external_links

Yes. I can see that my friend. This is not a project so can't say but PR can be sent when you are done with your all commits. You can also merge all your commits in single PR.


Using lemma of a word in lstm-parse.cc file

Hi,
Is there a way to reach lemma of words in the treebank (maybe by extracting them from conll files) inside lstm-parse.cc? Is lemma information kept somewhere in the code?
Thanks,
Betul

There is not. But you can modify the code to do so and use them similarly as postags are used.
Thanks!


fix/632: Hide 'Join now' option when user is logged in

PR Fixes:

1 #632

Resolves #632
Checklist before requesting a review

[x] I have performed a self-review of my code
[x] I assure there is no similar/duplicate pull request regarding same issue

npm install error as there is extra comma in last dependencies and I am not fetched the latest commit and created the pull request, that's why the checks failed, please confirm me @hkirat @devsargam whether I have to close this or this is enough

@pkmanas22
The code you have altered is producing a bug.
Bug: If the user is not loged in, join now and login buttons are not visible to the user.
file:///home/giridhar/Pictures/Screenshots/Screenshot%20from%202024-05-15%2001-57-43.png
my alternative suggestion:
when the session.data is in undefined state try to render a skeleton insted of hiding the buttons.
if(isLoading){
return (
// skeleton code
)
}
return (
// main component
)

@pkmanas22  Yes I totally missed out on that


Containers fail to start on snap edge

Required information

Distribution: Ubuntu
Distribution version: 19.04
The output of "lxc info" or if that fails:

config: {}
api_extensions:
- storage_zfs_remove_snapshots
- container_host_shutdown_timeout
- container_stop_priority
- container_syscall_filtering
- auth_pki
- container_last_used_at
- etag
- patch
- usb_devices
- https_allowed_credentials
- image_compression_algorithm
- directory_manipulation
- container_cpu_time
- storage_zfs_use_refquota
- storage_lvm_mount_options
- network
- profile_usedby
- container_push
- container_exec_recording
- certificate_update
- container_exec_signal_handling
- gpu_devices
- container_image_properties
- migration_progress
- id_map
- network_firewall_filtering
- network_routes
- storage
- file_delete
- file_append
- network_dhcp_expiry
- storage_lvm_vg_rename
- storage_lvm_thinpool_rename
- network_vlan
- image_create_aliases
- container_stateless_copy
- container_only_migration
- storage_zfs_clone_copy
- unix_device_rename
- storage_lvm_use_thinpool
- storage_rsync_bwlimit
- network_vxlan_interface
- storage_btrfs_mount_options
- entity_description
- image_force_refresh
- storage_lvm_lv_resizing
- id_map_base
- file_symlinks
- container_push_target
- network_vlan_physical
- storage_images_delete
- container_edit_metadata
- container_snapshot_stateful_migration
- storage_driver_ceph
- storage_ceph_user_name
- resource_limits
- storage_volatile_initial_source
- storage_ceph_force_osd_reuse
- storage_block_filesystem_btrfs
- resources
- kernel_limits
- storage_api_volume_rename
- macaroon_authentication
- network_sriov
- console
- restrict_devlxd
- migration_pre_copy
- infiniband
- maas_network
- devlxd_events
- proxy
- network_dhcp_gateway
- file_get_symlink
- network_leases
- unix_device_hotplug
- storage_api_local_volume_handling
- operation_description
- clustering
- event_lifecycle
- storage_api_remote_volume_handling
- nvidia_runtime
- container_mount_propagation
- container_backup
- devlxd_images
- container_local_cross_pool_handling
- proxy_unix
- proxy_udp
- clustering_join
- proxy_tcp_udp_multi_port_handling
- network_state
- proxy_unix_dac_properties
- container_protection_delete
- unix_priv_drop
- pprof_http
- proxy_haproxy_protocol
- network_hwaddr
- proxy_nat
- network_nat_order
- container_full
- candid_authentication
- backup_compression
- candid_config
- nvidia_runtime_config
- storage_api_volume_snapshots
- storage_unmapped
- projects
- candid_config_key
- network_vxlan_ttl
- container_incremental_copy
- usb_optional_vendorid
- snapshot_scheduling
- container_copy_project
- clustering_server_address
- clustering_image_replication
- container_protection_shift
- snapshot_expiry
- container_backup_override_pool
- snapshot_expiry_creation
- network_leases_location
- resources_cpu_socket
- resources_gpu
- resources_numa
- kernel_features
- id_map_current
- event_location
api_status: stable
api_version: "1.0"
auth: trusted
public: false
auth_methods:
- tls
environment:
  addresses: []
  architectures:
  - x86_64
  - i686
  certificate: |
    -----BEGIN CERTIFICATE-----
    …
  certificate_fingerprint: a1cd628c78e7f44426bafb64cf53d2f11cc4c55bb3130d761e164e243024e56d
  driver: lxc
  driver_version: 3.1.0
  kernel: Linux
  kernel_architecture: x86_64
  kernel_features:
    netnsid_getifaddrs: "true"
    shiftfs: "false"
    uevent_injection: "true"
    unpriv_fscaps: "true"
  kernel_version: 5.0.0-13-generic
  project: default
  server: lxd
  server_clustered: false
  server_name: michal-laptop
  server_pid: 16852
  server_version: "3.12"
  storage: zfs
  storage_version: 0.7.12-1ubuntu5

Issue description
Containers fail to start after a recent snap edge refresh
Steps to reproduce

snap refresh lxd --edge
lxc start <container>

Information to attach

[x] Container log (lxc info NAME --show-log) lxc-start-fail.txt
[x] Container configuration (lxc config show NAME --expanded)
lxd-container-config.txt

@Saviq Can you show the error that's returned to you as well as lxc info NAME --show-log?

@stgraber both are in the lxc-start-fail.txt file above I believe?
$ lxc start snapcraft-multipass
Error: Failed to run: /snap/lxd/current/bin/lxd forkstart snapcraft-multipass /var/snap/lxd/common/lxd/containers /var/snap/lxd/common/lxd/logs/snapcraft-multipass/lxc.conf:
Try `lxc info --show-log snapcraft-multipass` for more info

Hi,
I'm using LXC 3.1.0-devel version (built from source) and I'm facing this issue. Is it solved in another version?
Thanks,
Ginés.


Getting weather details through geolocation

A feature where user can enter latitude and longitude coordinates to get weather details.
I am willing to contribute on this if assigned. Additionally, I have fixed the minor spelling error and temperature toggling issue.

@uddiptojana10  go for it !

Make sure a person have independency of choosing if he/she wants to get weather as per city or geo location.
You can add toggle or dropdown.

@uddiptojana10 do not work on temperature toggling issue as already it is assigned to someone else .
You can work on , geo location and spelling .

oh okay.


Date Format Incorrect in Gantt Chart for Some Locales

Bug Report
When using the Gantt Chart (at least), the dates displaying above the graph are not displaying in the correct format for all locales it seems. In the "en" locale, dates display as "Mon DD" for example "Dec 29". However, in some other locales, this format should update to be "29 Dec". This apparently should be the case for at least Swedish, Dutch, and French, and maybe others. I am restricting this report to the Gantt Chart since it is what we are using, however, it may apply to other charts which display dates.
Description

When displaying dates above the Gantt chart, the formatting is not localized. The abbreviations seem to be, but not the display order of Mon DD / DD Mon. If needed I can try to provide an example.
Graphs affected
Note: AT LEAST Gantt chart is affected, there may be others.

[ ]  Bar graph
[ ]  Bubble graph
[x]  Gantt chart
[ ]  Line graph
[ ]  Paired result
[ ]  Pie chart
[ ]  Scatter plot
[ ]  Timeline

Steps to Reproduce

Create a Gantt chart using Swedish (sv), French (fr) or Dutch (nl) locale
Populate it with data such that the dates along the top are displayed as "Mon DD"
Note that the formatting is not correct for the locale
Formatting may also be incorrect for other date formats as well, if year is included or if time is involved - I have not checked, but worth investigating.

Link to CodeSandbox

For now I have not created a reproduction. If needed I can do so.
Additional Context / Screenshots

Swedish Locale screenshot showing incorrect date formatting.

Dutch Locale screenshot showing incorrect date formatting.

French Locale screenshot showing incorrect date formatting.
Expected Behavior

Dates in the Gantt chart and other graphs should be have their format localized appropriately, similar to, for example, moment's locale-aware date formatting feature.
Possible Solution

Implement some kind of locale-aware date formatting tool or feature that allows for locale-sensitive arrangement of the date elements.
Environment

Component Name and Version: @cerner/carbon-graphs 2.17.1
Browser Name and Version: Chrome 90.0.4430.212 (Official Build) (x86_64)
Node/npm Version: Node v10.23.1, NPM v6.14.10
Webpack Version: WebPack v4.46.0
Operating System and version (desktop or mobile): macOS Version 10.15.7 (Build 19H524)

@ Mentions

@AshishMotanamGurunadham directed me here to submit an issue. I have not worked with anyone regarding investigation or anything similar.

In retrospect this may have been better classified as a feature request, if the functionality was not intended to be there in the first place - My head was in the space of "bug report" since it was identified as an issue. Feel free to update as you feel appropriate. Thank you for your time.

I logged this: https://github.com/d3/d3-scale/issues/247 to see if we can get some traction to make those partial date phrases internationalizable.

Spike update: Based on the research I have done till now, looks like the only way we can change the date format is by sending in the format argument manually. If format is not specified tickFormat function returns the current format, which defaults to null. A null format indicates that the scale’s default formatter should be used, which is generated by calling scale.tickFormat. Since we are not providing any format for the axis ticks, ticks are displaying as designed by scale.tickFormat.
Our next move on this issue is, we will go through the d3-axis code so that we can suggest changes(if there is any) to the d3 team to fix this date format issue.
The new issue is raised here.
If this issue cannot be fixed from the d3 end, as our last hope we will manually send the desired format as per the locale, lower and upper limits provided by the consumer.
Source: https://stackoverflow.com/questions/40286963/d3-time-format-multi-in-v4-x


Integrated Terminal only shows Powershell upon Start Up

Since the latest update, the terminal always shows the Windows Powershell upon start up. I tried setting the default to other terminals (e.g. Git Bash, Command Prompt, et cetera), but VSCode always shows the Powershell as opposed to the default terminal showing up on start up. I can still switch to my desired terminal, but it's an annoying qualilty of life bug.
Expected: Assigned default terminal to show up on start up such as Git Bash
Actual: Windows Powershell shows up on start up
OS: Windows 10

/duplicate #132150


UUID Generation in Non-Browser Context (eg. NativeScript)

Is your feature request related to a problem? Please describe.
When I import and use uuid in a NativeScript application I receive an error cannot get 'browser' of undefined. Initially I didn't realize this was related to the UUID library. But the moment I removed attempted v4() generation the application compiled without any errors.
Describe the solution you'd like
I would love to use uuid as per the documentation even when in a mobile device situation such as NativeScript. I saw the ReactNative support and was hopeful. However, upon a quick peek it didn't seem to be a user friendly solution and I didn't want to attempt to port it.
Describe alternatives you've considered
I've looked at guid-typescript, however, it is unmaintained even though it's downloaded 67k+ times per week according to NPM.
Another alternative that I have considered is to write a NativeScript plugin that uses Android / iOS functionality to generate the UUIDs but felt that would only benefit a smaller community.
Additional context
I imagine if the solution were implemented very well it could cross all the non-browser contexts. I'm not certain how feasible this is, but I hope this is the right place to ask.
Thanks for this UUID library!

Is this related to #551 and #469?  (mainFields order in webpack)
@BallisticPain I believe this is the first issue we've had related to NativeScript, and I don't know that any of us maintainers have worked with that toolchain before.  A Minimal, Complete, and Verifiable example, ⟪cough⟫ as requested in the bug report template ⟪cough⟫ would be helpful.  E.g. a small repo that we can clone, with instructions for building and reproducing the error.

Duplicate of #314.
@BallisticPain - A Minimal, Complete, and Verifiable example, ⟪cough⟫ as requested in the bug report template ⟪cough⟫ would be helpful.  E.g. a small repo that we can clone, with instructions for building and reproducing the error.

@BallisticPain please check my comment in https://github.com/uuidjs/uuid/issues/314#issuecomment-737761009
If reordering resolve.mainFields doesn't work for you, feel free to reopen this issue and proved an example as outlined by @broofa above.


[entropy_src] V2S review action items

We just had the V2S review meeting today. The review notes can be found here. The following action items have been identified:

[x] V2 sign-off, all V2 AIs are done, see  #16308 -> @martin-lueker
[x] Re-trigger FPV sec_cm tests to update dashboard, this has already happened, thanks @cindychip !
[x] Merge move ack_sm state definition into separate package file, https://github.com/lowRISC/opentitan/pull/16311
[x] Merge updated covergroup names PR to align names in coverage interface and testplan, https://github.com/lowRISC/opentitan/pull/16309
[ ] Update/redo UNR exclusions, this helps to remove tool warnings & might also help for toggle coverage (we were't sure about the latter) -> @martin-lueker
[ ] If not solved by updating UNR exclusions, add explicit exclusions for toggle coverage of lower-level hierarchies such as prim_counts with certain inputs fixed -> @andreaskurth

The updated exclusions have been merged already, Andreas is currently running a full regression and will share the results later today. We expect to close V2S with after this.


swift http libary

The http request adapter interface needs to be implemented as a separate package for swift.
We should also consider how the middleware pipeline infrastructure will work while choosing the client and make sure that client supports HTTP/2

https://developer.apple.com/documentation/foundation/urlsession?msclkid=8713fd77b0d411ec85808abbe4506bd1
It seems that apple already provides an HTTP client API, this would likely be the most straightforward method as it is developed by Apple and would thus require less dependencies on other API's.


Allow integration testing measurement

This PR is not finished yet, I have one problem
Overall I would like to allow measurements of integration tests coverage. Tests scenario are written in Postman and checks if API behaves as expected.
I would like to achieve it by running tested assembly first then running target, After finished tests I kill app. Currently tested app crashes because it cannot find coverlet.tracker.dll. Can you help me with fixing this problem?

Hi @szogun1987, can you explain the scenario you're trying to support?

I have published both application I test and coverlet-console project with dotnet publish --self-contained -f netcoreapp2.1 -r win-x64 maybe it matters.

@szogun1987 why not create a runner application that performs the API calls and then just use coverlet.console to call it? You can use the already existing parameters to control which modules are included and excluded from coverage report.

@pjanotti My changes are required to do this:

I can't use main app as a target: console waits for target to complete execution. In case of web app it would never happens. It is harder to run test-app in valid point of time also.
I can't run mine app before coverlet also because it would unable it to do instrumentation: files would be used by main-app process, it has to be run by coverlet, so additional parameter is required.
Without my changes statistics for main application itself would be skipped no matter what parameters would passed because InstrumentationHelper constains logic to exclude it.

But with my changes exception I mentioned above occurs and I don't know why.

Which process are you planning to end with your change to trigger coverage run stop and what process you plan to be the "self-hosted"?

What do you mean as self-hosted? My change allow coverlet-cli to host both processes: testing and tested one. Once testing one would completed there is no need to keep tested one alive, so Kill invocation is added.
The only problem I have is loading coverlet-tracker by tested process:

it seems to be placed in valid location - tested process main folder
it seems that valid process tries to load it - tested app creashes because lack of it, tester one crashes because it can't find tested one listening on port.

By the way today I would add some code that displays more information in console output once one of the processes would crash.

Sorry, I meant "selfrun" or "selfProcess" from your PR. Anyway, per what you are saying "selfrun" is your "tested" (your web app) and the "process" variable inside coverlet code is the "tester" (Postman in your case).
What I was suggesting is that if you have an app (aka "integration-tester") to launch "tested" (your web app) and launch and wait for the termination of API "tester" (Postman in your case). Then you can use coverlet.console to control what is going to be instrumented (via include, and exclude) and use as the -target the "integration-tester". This should work as it is, without need to modify coverlet.
An argument can be made that your case is common enough to deserve to be supported directly by coverlet, and perhaps it is, but at first it seems that you don't need to change coverlet to satisfy your scenario.
Per your description it seems that coverlet is instrumenting your "tester" while you intended it to have "tested" instrumented, include/exclude options should be able to make it work for you.

Running additional process may be redundant, I have tried to write simple powershell script that runs app, then postman and then kills app, but it also had problems with loading coverlet-tracker.dll.
Parameter that prevents excluding main module should be added, maybe it should have other name, because as I said before coverlet excludes it before even checking include and exclude parameter.

@szogun1987 I think I have to agree with @pjanotti on this. While I understand the reason for your change I don't think it's common enough to warrant a change in Coverlet itself. Can you post the command to run with your current self-run change? Perhaps, I can come up with an approach using Coverlet's stock functionality

Sorry for late response So I have following folders structure:

ApiRuner - result of publication of aplication that invokes app Web Api
Coverlet.Test - result of publication of tested application that exposes API
coverlet-cli - result of publication of mine version of coverlet

After run .\coverlet-cli\coverlet.console.exe 'Coverlet.Test\Coverlet.Test.exe' --self-ru n --target 'ApiRuner\ApiRunnner.exe' I receive following output:
Unhandled Exception: System.AggregateException: One or more errors occurred. (Nie można nawiązać połączenia, ponieważ komputer docelowy aktywnie go odmawia) ---> System.Net.Http.HttpRequestException: Nie można nawiązać połączenia, ponieważ komputer docelowy aktywnie go odmawia ---> System.Net.Sockets.SocketException: Nie można nawiązać połączenia, ponieważ komputer docelowy aktywnie go odmawia
   at System.Net.Http.ConnectHelper.ConnectAsync(String host, Int32 port, CancellationToken cancellationToken)
   --- End of inner exception stack trace ---
   at System.Net.Http.ConnectHelper.ConnectAsync(String host, Int32 port, CancellationToken cancellationToken)
   at System.Threading.Tasks.ValueTask`1.get_Result()
   at System.Net.Http.HttpConnectionPool.CreateConnectionAsync(HttpRequestMessage request, CancellationToken cancellationToken)
   at System.Threading.Tasks.ValueTask`1.get_Result()
   at System.Net.Http.HttpConnectionPool.WaitForCreatedConnectionAsync(ValueTask`1 creationTask)
   at System.Threading.Tasks.ValueTask`1.get_Result()
   at System.Net.Http.HttpConnectionPool.SendWithRetryAsync(HttpRequestMessage request, Boolean doRequestAuth, CancellationToken cancellationToken)
   at System.Net.Http.RedirectHandler.SendAsync(HttpRequestMessage request, CancellationToken cancellationToken)
   at System.Net.Http.HttpClient.FinishSendAsyncBuffered(Task`1 sendTask, HttpRequestMessage request, CancellationTokenSource cts, Boolean disposeCts)
   --- End of inner exception stack trace ---
   at System.Threading.Tasks.Task.Wait(Int32 millisecondsTimeout, CancellationToken cancellationToken)
   at System.Threading.Tasks.Task.Wait()
   at ApiRunnner.Program.Main(String[] args) in C:\Users\wogo\Desktop\Coverlet.Test\ApiRunnner\Program.cs:line 11

Unhandled Exception: System.IO.FileNotFoundException: Could not load file or assembly 'coverlet.tracker, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=3c88f8be3827e966'. Nie można odnaleźć określonego pliku.
   at Coverlet.Test.Program.Main(String[] args)

Self process crashed while testing

Calculating coverage result...
  Generating report 'C:\Users\wogo\Desktop\Coverlet.Test\Lab\coverage.json'

+--------+--------+--------+--------+
| Module | Line   | Branch | Method |
+--------+--------+--------+--------+

First exception comes from testing app and should be ignored.
I also attach code of apps used to reproduce issue code of apps used to reproduce issue.

@szogun1987 Your idea is really awesome! It is such a shame it is not working well.
I think the problem with the exception you posed with System.Net.Http.HttpRequestException is that the HttpContext is highly protected class that is created by unmanaged out of process code for security reasons. (think modifying / reading HTTPContext content by virus)  I have tried to work with it before to create HomeApplicationProticol for AppleHome and it is practically impossible to change or hook into that. I have a repo but its unfinished because I literally have to create my own TCPSockets and manage everything my self.. its difficult.
Unfortunate I do not have a solution for you. You need work out a way to ignore hooking into the context and somehow doing hooks at another more public level, ie your libraries. I may be completely offtrack but I have a hunch that is the problem.
Powodzenia!
If you can get this working it would be a killer functionality. It would mean I wont have to write integration tests, i can just build postman tests and get the same coverage. Please do not give up 💯 🥇

@szogun1987 do you have news on it?Or we can close for the moment?

Sorry, I haven't access to the project that uses coverlet anymore.

Sorry, I haven't access to the project that uses coverlet anymore.

Ok can we close for now(some cleanup)?

Sure

Thank's for quick response!


MWPW-140345-Fix issues in media QR block variant

Fixes issue of QR code being visible on mobile and tablet in media large
Sets the dimension of QR code to 140 x 140 as requested
Also set the spacing between the QR code image and text to 24px
Added unit tests to test it in different viewports

Resolves: MWPW-140345
Test URLs:

Before: https://main--milo--adobecom.hlx.page/fragments/consonant/media/media-qrcode?martech=off
After: https://media-qr--milo--ruchika4.hlx.page/fragments/consonant/media/media-qrcode?martech=off

Worked with Kyungwha Lee, and got it verified on feature branch.


Discord scam link detector

There are many scam links going across Discord at the moment. This feature will detect them and delete them.

PSH bot already does this now


Fix Kubernetes pod waiting flakiness

This fixes the eternal issue with the flaky Kubernetes smoke tests. It also affected the normal operation of the provider in some circumstances.

Azure tests are failing, but it's a transient cloud issue. Merging...


Add yacht exercise

Thanks Isaac! Yes, I went through several iterations on this one, and the valueCount function really helped.


export isl_schedule_node_schedule

Why do you need it?
I thought you guys preferred working on that other schedule tree representation.
In that case, you should use isl_schedule_constraints_set_prefix directly.
Otherwise, you end up converting back-and-forth.

This is the rescheduling call... I don't see how it can be done from another representation directly.

ok, I see now


Update notation key add/remove commands for RC1

Need to decide what we'll do for RC1, and what we'll do post RC1 to help users easily add and remove keys for signing. The assumption is this would transition from the alpha1 config to the RC1 trust stores.

Already implemented.

Spec: https://github.com/notaryproject/notation/blob/main/specs/commandline/key.md
Implementation: https://github.com/notaryproject/notation/blob/main/cmd/notation/key.go


Side Menu Button While NavigationBar is Hidden

how can i use side menu  with  Navigation Controller while NavigationBar is hidden?

Allow me to take a punt here...
in your AppDelegate ...  configure your SideMenuController ...
SideMenuController.preferences.interaction.panningEnabled = true

ok , and how can i configure custom button to show side menu ?

In UIButton Action  sideMenuController?.toggle()  works for me.


[Deploy] Feedly

This is a simple service for the RSS reader Feedly
Service
Read news over RSS through Feedly service
Link to repository
https://github.com/augustocdias/feedly-franz-plugin

Just saw there's already one...


[OSS-ONLY] Update BABEL_3_8_STABLE default branch and upgrade-test-configuration.yml

Description
clone_engine_repo.conf: update BABEL_3_8_STABLE__PG_15_9 to BABEL_3_8_STABLE__PG_15_10
upgrade-test-configuration.yml: update Vole version due to 14.14 is skipped
Check List

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is under the terms of the Apache 2.0 and PostgreSQL licenses, and grant any person obtaining a copy of the contribution permission to relicense all or a portion of my contribution to the PostgreSQL License solely to contribute all or a portion of my contribution to the PostgreSQL open source project.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>5
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.003%) to 73.323%

Totals

Change from base Build<PHONE_NUMBER>7:
0.003%

Covered Lines:
42801

Relevant Lines:
58373

💛  - Coveralls


Android render

Reopening the PR because I need to check Azure Pipelines

Everything in master but the integration tutorial -> Closing.


Change log.

Hello,
I've been looking for the signalr change log. Sometimes the client side got updates and the server not. Without the change log it worries me to update the client. Who knows this client version need to have certain server version. So can we have change log for signalr ?, for the clients and servers. Thank you.

Can you create an issue from the doc page? That will connect the doc to the issue. Go to the bottom of the page and select This page .


Proper guidance on adding Helm repositories

Version: master 5/31

When adding catalogs, the text is as below, while we only allow kind = Helm in the Add Catalog dialog which is using HTTP GET to get the files.

Custom
You can define your own additional custom catalog sources.
Each one needs a unique name and a URL that git clone can handle
(see docs for more info).

The dialog box after clicking Add Catalog shows URL as .git URL, which can be plain HTTP. And branch which is not relevant for HTTP URLs.

Should be made more clear if Git or HTTP URL, and the options.

There are two types of catalog that we currently support:

Git based catalog. All the charts and version are written into a git repository. Examples: https://github.com/rancher/charts, https://github.com/kubernetes/charts. Any valid git url should be fine.
Helm original server. This is a HTTP server that can serve all the necessary files like index.yaml, tgz files. Reference on how to build a helm server: https://docs.helm.sh/developing_charts/#hosting-chart-repositories. Examples: https://kubernetes-charts.storage.googleapis.com/, https://kubernetes-charts-incubator.storage.googleapis.com.

rancher/rancher:master 6/12
Verbiage has been updated to reflect the steps for adding a git-based catalog.
There is nothing to denote that a Helm server can be used instead.
The verbiage displayed is: Each custom catalog needs a unique name, a Git repository URL (HTTP/S), and a branch name.


After creating a job of type FLINK-STREAM, there is currently no entry to start the job, which was previously started in the Task definition page, which now does not exist

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
创建入口:

原启动入口：

现在页面：

缺少了该入口
What you expected to happen
Page not found
How to reproduce
Page not found
Anything else
No response
Version
3.2.x
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

This feature has been deprecated


Strange running error

Not sure if I'm doing anything differently than before, but I can't debug this error:
pangeo-forge-runner bake \ --repo=./ \ --Bake.recipe_id=eNATL60_BLBT02 \ -f configs/config_local.py
FileNotFoundError: [Errno 2] No such file or directory: 'hg'
Happened on hub as well, tried after kept failing on local.
Apparently hg is something called "mercurial"

Okay did not know needed to use object, not id

id: eNATL60_BLBT02
object: "eNATL60:eNATL60_BLBT02"

Can we close this?


opnode: Fix Incorrect error handling when creating an L2 block

Description
This is an issue ToB identified (issue #8). What happens is that
the ethereum.NotFound error would never be returned, but in general
if there is an error in the lookup, the state loop should keep
going.
This also properly wrap errors in l1/source.go.
Note: There is an argument for halting L2 block production if it cannot
find the next L1 origin; however, it should be clearly communicated with
an error.
Metadata

Fixes: ENG-2127

Spoke to Proto - we're good to merge this and handle the testing separately. I'm tracking the testing in a separate Linear.


Issues Locating Missing Files

Game name: (please, specify the game name)
Engine version: (specify engine version, if known)
Platform: (specify platform, if known)
Error message: No files were found
Stack trace:    at uTinyRipper.MixedGameStructure..ctor(IEnumerable1 pathes)    at uTinyRipper.GameStructure.CheckMixed(List1 pathes)
at uTinyRipper.GameStructure.Load(List1 pathes, LayoutInfo layinfo)    at uTinyRipper.GameStructure.Load(IEnumerable1 pathes, LayoutInfo layinfo)
at uTinyRipperGUI.MainWindow.LoadFiles(Object data)

This project intended to work with serialized files, not with apk, ipa or any other.


CamelQuarkusTestSupport: Lifecycle.PER_CLASS may cause following tests to fail

Follows https://github.com/apache/camel-quarkus/issues/4560.
Usage of Lifecycle.PER_CLASS in the reproducer from issue #4560 should work  as a workaround.
From the brief check it seems, that somehow the camelContext is not propagated to parent CamelTestSupport, therefore test fails because the contexts are not the same. This should be fixed.

The error message is: Different context found! ==> expected: <CamelContext(camel-1)> but was: <null>


essentials methods ???

Im trying to call essentials plugin method that are from api docs
like essentials.users.seen in php api but it seems that method doesn't exist ??
I just want to get information when last time user was seen in game to my website
im using JSONAPI-5.6.0-mc1.7.9-R0.2-SNAPSHOT.jar

J'ai le même problème :(


bug: Paloma api error

https://api.paloma.mcbnode.online/staking/validators and  https://api.paloma.mcbnode.online/gov/proposals api requests are failing.

This is not a protocol issue. You'll need to check with MCB  node and ask him to check his setup. This is a devops issue.


Add FutureWarning in hypnogram functions

This is a pretty straightforward PR that adds a FutureWarning to the now-outdated yasa.hypno functions. I did not add a warning for a few functions that are called directly from the class methods (e.g. transition_matrix).
This PR also implements a few minor fixes and improvements in the docstring.
@remrama would appreciate your review if time allows it. Otherwise I can just merge as is since it's a low-risk PR.

Codecov Report
Base: 92.53% // Head: 92.54% // Increases project coverage by +0.01% :tada:

Coverage data is based on head (d144f72) compared to base (3dfe2d6).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #123      +/-   ##
==========================================
+ Coverage   92.53%   92.54%   +0.01%
==========================================
  Files          23       23
  Lines        3106     3112       +6
==========================================
+ Hits         2874     2880       +6
  Misses        232      232

Impacted Files
Coverage Δ

yasa/hypno.py
98.34% <100.00%> (+0.01%)
:arrow_up:

yasa/sleepstats.py
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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

Thanks for the super quick review! Good catch re: yasa.hypno_consolidate_stages. I have decided to remove it since it wasn't part of the public API anyway.
One pending comment for you regarding as_int. If we decide to return a numpy array instead I'll implement in a separate PR.

I did not add a warning for a few functions that are called directly from the class methods (e.g. transition_matrix).

Just wanted to add an argument here for always retaining these functions as isolated utility functions. I'm currently developing some examples for an upcoming PR and I am finding it useful to have functions like transition_matrix and sleep_statistics on their own (i.e., as functions and not methods attached to Hypnograms). I like passing them to pandas methods like apply and transform. If you think of YASA as a more broad sleep analysis utility, others might find these useful too.
I see here you aren't planning to immediately deprecate them, just noting in case it's on the table.

Good point, I actually also use pandas.apply quite a lot. However for sleep_statistics the method implementation is now better than the original function. So maybe we can create a class instance internally and then just return the method? This would also avoid code redundancy.
We can also discuss whether to leave or remove them from the public API in 6-12 months when we get closer to v0.8

Good point, I actually also use pandas.apply quite a lot. However for sleep_statistics the method implementation is now better than the original function. So maybe we can create a class instance internally and then just return the method? This would also avoid code redundancy.
We can also discuss whether to leave or remove them from the public API in 6-12 months when we get closer to v0.8


[Hangfire] Add ability to en-/disable jobs by configuration

As a developer,
i want to be able to enable or disable specific hangfire jobs without deleting them,
so i can use different configurations for different environments (dev, stage, prod)

Implemented and released with #33


docs(material): examples in jsdocs

Feature Description
Currently the jsdocs on components is very minimal, with very little detail
https://github.com/angular/components/blob/39921f43382e3abc90c14fd8602d0cf371239fce/src/material/button/button.ts#L62-L64
Example how how this displays in VS Code

With material-experiential things seem greatly improved
https://github.com/angular/components/blob/39921f43382e3abc90c14fd8602d0cf371239fce/src/material-experimental/mdc-button/button.ts#L30-L38
But I think there is still a few things missing.
There is a link to the material design docs which is great, but it would be awesome to have a link to the Angular Material documentation which has code examples and live demos.
Additionally it would be cool to have code examples in the JSDocs themselves.
This helps with discoverability of different ways of using components, for example showing the mat-flat-button as an example would be quite useful on the mat-button JSDoc
Use Case
Having faster access to documentation will not only help new developers who are not familiar with all the APIs that @angular/material has to offer, but also help refresh the memory of any developers who works in many context, such as backend work.

I think it would make sense to add links to the appropriate doc site, but don't need to include code examples in the JSDocs themselves as those can be found on the site.
This seems like a good issue for an external contributor

but don't need to include code examples in the JSDocs themselves as those can be found on the site.

That's fair

Is anyone working in this? I would like to pick it up.

@bonganiSibiya All yours. Nobody has started this issue yet

Since this issue originally referenced what's now Legacy (and used material-experimental, which is now the main implementation as being the good example), is this still an issue that needs addressing?


Very long ToolTips get clipped

The current directory of WSLv1 tab is /mnt/c/Users/user/Downloads, and the tooltip just reads /mnt. This is not useful since it's very generic, even /mnt/c would be more useful. Maybe even absence of this strange path would be less confusing.

Terminal version: Version: 1.7.1033.0

It just looks like it’s cut off (look at the part of the character to the right of mnt). We should probably fix that.

Maybe best to always put this path it on new line, then there wourld be at least 10-20 letters space. And then it could be ellipse'd...

So, we don’t actually know that it’s a path. That’s the verbatim string that bash reported to us, with the username/hostname/path pre-formatted. Alas.

Summing up three other variants: 1) show PS1 on different line than OS name, 2) make tooltip radically larger, 3) have some heuristics for word-wrapping unknown PS1 (e.g. by slash or non-word characters)

Hmm, it looks like how we're building the tooltips doesn't account for the width of the text:

But then compare to:
  <Button Content="Button with a simple ToolTip."
        ToolTipService.ToolTip="https://this.com/is/a/incredibly-long-hyperlink/that/might/have/too-long-a-tooltip"/>

So TabBase::_UpdateToolTip and
    <Border x:Name="HyperlinkTooltipBorder"
            BorderBrush="Transparent">
        <ToolTipService.ToolTip>
            <ToolTip x:Name="LinkTip"
                     Placement="Mouse">
                <TextBlock IsTextSelectionEnabled="True">
                    <Run x:Name="HoveredUri" /> <LineBreak />
                    <Run x:Uid="HowToOpenRun"
                         FontStyle="Italic" />
                </TextBlock>
            </ToolTip>
        </ToolTipService.ToolTip>
    </Border>

in termcontrol.xaml will probably need to get updated.

Also, not sure why "ctrl+alt+2" shows up at a seemingly arbitrary margin (not left, not right, unclear the intent). With combination with a clipped tooltip, this looks also strange

The ctrl+alt+2 is probably trying to center itself underneath the previous line of text. It just doesn't realize that it's container is not as big as the text is 😉

My proposed tooltip (consistent captialization + PS1 on new line always + FS path wrapping heuristic for the first slash (so that for default Windows/Ubuntu PS1 scheme it looks okay in for short enough paths) + wrapping is prefixed by "... ":
[Ctrl+Alt+2]: Ubuntu 18.04
vadimkantorov@DESKTOP-4UF8FID:
... /mnt/c/Users/user
.... more wrapping if needed

As the one responsible for the mess, I can take a look later on.

I will start with fixing the wrapping (one-liner).
@zadjii-msft   - please let me know if you want to switch to another structure, like @vadimkantorov  suggested.

Nah, let's just fix the wrapping for now. That's easiest - everything else is just heuristics, which is a programmer word for "guessing" 😄 Plus, that doesn't involve any real design changes, which would require a much lengthier discussion

Will this also fix Ctrl capitalization + shortcut placement?

@vadimkantorov - no, I didn't fix any of those, sorry.

Regarding the shortcut placement, @zadjii-msft  - advised to leave it as is meanwhile
Regarding the capitalization we need to do it.. but I guess it has larger scope that just a tool-tip, e.g., it might affect the command palette as well.

The tooltip for the new tab button is too short (or the text is too long) to understand what it says:

Related #5355 and #5484.


ntr for PMID:28827290

increased

FYPO:0005597
abnormal protein localization to centromere

decreased protein localization to centromere FYPO:0006238
increased protein localization to centromere FYPO:0006239
note that the "vegetative" terms have been around for a while
edit file: 884374b24346e939e1e868297533dc49cc5fc21f
release: f08be34c1aaa2f3a32bc54e8fdcd703d89407fe4


(7041, 8321, 8341) Move workshops location

Hi,
As discussed with Michelle, created a folder dba-essentials under version-specific folder odb-21c, which indicates Oracle Database 21c.
Moved the workshops under odb-21c/dba-essentials:

configure-network-env
install-db
remove-db

No change to lab contents.
Please approve the PR. Thank you
~MG

Hi Arabella, thanks for approving the PR.
I do not think I understand the new objectstorage URLs very well. Could you please suggest what to do.
Thanks

Hi Manish! You can create object storage URL for your workshop by combining the object storage prefix of the database repo (https://objectstorage.us-phoenix-1.oraclecloud.com/p/KNnfJD8PskLUcpiy55OJ4GjaMSFvEJD6TKEzBi6E88AXNfBKKgl7HbLVTU2oxdO1/n/c4u02/b/database/o/) and the path to your workshop's index.html file.

Okay. The workshop is at this location in the database repo:
database/odb-21c/dba-essentials/install-db
So, the URL would be:
https://objectstorage.us-phoenix-1.oraclecloud.com/p/KNnfJD8PskLUcpiy55OJ4GjaMSFvEJD6TKEzBi6E88AXNfBKKgl7HbLVTU2oxdO1/n/c4u02/b/database/o/database/odb-21c/dba-essentials/install-db/workshops/freetier/index.html
If this is correct, I will go ahead and update WMS for this workshop and for others too.

Hi Manish! The link should be:
https://objectstorage.us-phoenix-1.oraclecloud.com/p/KNnfJD8PskLUcpiy55OJ4GjaMSFvEJD6TKEzBi6E88AXNfBKKgl7HbLVTU2oxdO1/n/c4u02/b/database/o/**odb-21c/dba-essentials/install-db/workshops/freetier/index.html**. You only need to append your workshop path within the repo. I suggest you wait after the sync between GitHub repo and object storage tonight, verify all the links can render workshops correctly, and then update them in WMS.

Sure. Thanks, Arabella! :thumbsup:


testing CI with upstream changes in ethers-solc

@0xalpharush let's bump ethers to your PR and give this some user testing passes before we merge? always want to be careful with the compilation pipeline changes

@gakonst For sure, not in a hurry. I'm going to work on updating crytic-compile and try to keep it backwards compatible with the old cache. I'll test it on some repos starting with https://github.com/ScopeLift/flexible-voting since it motivated these changes.

SGTM - LMK. I approved the ethers PR, let's run this however is best for you guys.


Encode JSON encode on write rather than read

JSON encoding on write rather than read to reduce memory footprint. For example the memory use of an initial sync of a 35MB/200k row table can be reduced from 50MB to 25MB on the first initial sync (which includes the encoding and the writing to storage) and to 6MB on subsequent initial syncs (where we just read from storage and there's no encoding).
This allows a couple of simplifications:

append_to_log now takes log items rather than the intermediate structure prepared_change
...

Reminder to move this PR to electric — I'm going to close all the PRs tomorrow 🙏


Mark ConfigureLogging and UseLoggerFactory overloads with ILoggerFact…

…ory as obsolete
#1007
This will break downstream repos. I will try to pre-emptively update as many as I can and keep an eye out for build breaks when merging.
Affected Repos:

[ ] AzureIntegration
[ ] BasicMiddleware
[ ] SignalR

@davidfowl the issue has be triaged to preview2. Do you think that is okay or should we pull it back to preview1?

ping @muratg @davidfowl @BrennanConroy @Tratcher since we want to pull this change back to preview1.


how to load module translations that works in global namespace

In each module service provider, we load translation like this:
$this->loadTranslationsFrom(module_path($this->moduleName, 'Resources/lang'), $this->moduleNameLower);
the problem is that loaded translations becomes limited to related module namespace and to access it, you should use for example:
__('student::auth.required'),
so, how do we make loaded translations accessible from laravel and any other module by using:
__('auth.required'),

any news ?


Error messages about pending task which are destroyed

Home Assistant release (hass --version):
0.21.0.dev0
Python release (python3 --version):
Python 3.4.3
Component/platform:
mqtt/http
Description of problem:
Just an error in the log. Seems still to work
Expected:
No error in the log if there is no problem.
Problem-relevant configuration.yaml entries and steps to reproduce:
http:

Traceback (if applicable):
16-06-07 18:38:30 ERROR (ThreadPool Worker 6) [asyncio] Task was destroyed but it is pending!
task: <Task pending coro=<run() done, defined at /home/fab/Documents/repos/home-assistant/lib64/python3.4/site-packages/websockets/protocol.py:389> wait_for=<Future pending cb=[Task._wakeup()]>>
16-06-07 18:38:30 ERROR (ThreadPool Worker 6) [asyncio] Task was destroyed but it is pending!
task: <Task pending coro=<run() done, defined at /home/fab/Documents/repos/home-assistant/lib64/python3.4/site-packages/websockets/protocol.py:389> wait_for=<Future pending cb=[Task._wakeup()]>>

Hmmm, can't reproduce it on a second system.

It's related to HBMQTT as its the only part that uses coroutines and
websockets
On Tue, Jun 7, 2016, 09:53 Fabian Affolter<EMAIL_ADDRESS>wrote:

Hmmm, can't reproduce it on a second system.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/home-assistant/issues/2253#issuecomment-224343788,
or mute the thread
https://github.com/notifications/unsubscribe/ABYJ2ul96Qv2CgQTZGEIlqfb4JfRsY3Sks5qJaIfgaJpZM4IwIct
.

This problem only occurred in my gps branch.


call of reflect.Value.Int on string Value

Just updated hugo from 0.19 to 0.25 via "brew upgrade hugo".  Ran hugo and got
Started building sites ...
WARNING: calling IsSet with unsupported type "invalid" (<nil>) will always return false.

panic: reflect: call of reflect.Value.Int on string Value [recovered]
	panic: reflect: call of reflect.Value.Int on string Value [recovered]
	panic: reflect: call of reflect.Value.Int on string Value [recovered]
	panic: reflect: call of reflect.Value.Int on string Value

goroutine 133 [running]:
text/template.errRecover(0xc4205cb830)
	/usr/local/Cellar/go/1.8.3/libexec/src/text/template/exec.go:146 +0x62
panic(0x1710e80, 0xc4203649a0)
	/usr/local/Cellar/go/1.8.3/libexec/src/runtime/panic.go:489 +0x2cf
text/template.errRecover(0xc4205cab38)
	/usr/local/Cellar/go/1.8.3/libexec/src/text/template/exec.go:146 +0x62
panic(0x1710e80, 0xc4203649a0)
	/usr/local/Cellar/go/1.8.3/libexec/src/runtime/panic.go:489 +0x2cf

etc.
Uninstalled hugo  (brew uninstall --force hugo), reinstalled, with the same result.
MacBook Pro running MacOS 10.12.5.

I need more info to guess what this is. Do you have the site source online for people to look at?

https://github.com/bwbecker/localpr.ca https://github.com/bwbecker/localpr.ca

On Jul 7, 2017, at 9:09 AM, Bjørn Erik Pedersen<EMAIL_ADDRESS>wrote:
I need more info to guess what this is. Do you have the site source online for people to look at?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/gohugoio/hugo/issues/3681#issuecomment-313676182, or mute the thread https://github.com/notifications/unsubscribe-auth/AA_ZRgtcZV-M6pILIHGpEdu5UAGHVNBPks5sLi4CgaJpZM4OQ2u0.

Byron Weber Becker
School of Computer Science
University of Waterloo, Waterloo, Ontario  N2L 3G1
Email<EMAIL_ADDRESS>Phone:<PHONE_NUMBER> x34661
Advising FAQ:  www.cs.uwaterloo.ca/current/faq

@bwbecker that repository does not build out-of-the box. Probably a missing theme.

Sorry; added the theme and pushed.  Pulled to a fresh directory.  Builds with Hugo v0.19 but gives the panic with Hugo v0.20.

@moorereason this test fails for IsSet:
{[]interface{}{1, 2, 3, 5}, "2", false, false, ""},

@bwbecker the panic is unfortunate, and we will fix that, but the .Site.Sections was deprecated in Hugo 0.23 and is now replaced with a slice of Pages, so the isset constructs in your theme does not work anymore. Please take any further questions on https://discourse.gohugo.io/

Thank you!

On Jul 7, 2017, at 1:18 PM, Bjørn Erik Pedersen<EMAIL_ADDRESS>wrote:
@moorereason https://github.com/moorereason this test fails for IsSet:
{[]interface{}{1, 2, 3, 5}, "2", false, false, ""},
@bwbecker https://github.com/bwbecker the panic is unfortunate, and we will fix that, but the .Site.Sections was deprecated in Hugo 0.23 and is now replaced with a slice of Pages, so the isset constructs in your theme does not work anymore. Please take any further questions on https://discourse.gohugo.io/ https://discourse.gohugo.io/
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/gohugoio/hugo/issues/3681#issuecomment-313741696, or mute the thread https://github.com/notifications/unsubscribe-auth/AA_ZRovAvjW_gWCf6MCvy383-T2S52e0ks5sLmhIgaJpZM4OQ2u0.

Byron Weber Becker
School of Computer Science
University of Waterloo, Waterloo, Ontario  N2L 3G1
Email<EMAIL_ADDRESS>Phone:<PHONE_NUMBER> x34661
Advising FAQ:  www.cs.uwaterloo.ca/current/faq


Update NuGet package for .net 5.0 and 6.0

Update NuGet package for .net 5.0 and 6.0

It's a well working tool, would be nice to see it packaged for NET 6
I don't think it needs much (if any) updates, it works on NET 6 with compilation warnings.

I would love to get an updated nuget package, muche more convinient...


When a bulkImport fails to update the MetadataTable, which bulk import failed is hard to determine

While trying to track back issues that were seen with ConstraintViolationExeptions thrown in MetadataTableUtil#update, I noticed that tracking back exactly which bulk import threw the exception was not simple. It would be great if the error message could include more information so that we could more easily track back to which import failed

This issue appears to be addressed with a change in #2086 (Add optional extent to metadata update failure logs). Any objections to it being closed?

Closing as no objections were raised and it is addressed with #2086.


git-add file that accidentally was left out of 44d027beb11398a57a43.

@swift-ci test and merge

@swift-ci test and merge

@swift-ci test and merge


[Docs] Create "Performance Management with Meshery" guide

Current State
Users lack a holistic guide to use the performance management capabilities in Meshery.
This could be managed through the UI, CLI and GitHub Action.
Desired State
Create a new guide here: https://docs.meshery.io/guides
Add sections on performance management through UI, CLI as well as the GitHub action.
Refer: https://docs.meshery.io/guides/smi-conformance

Contributor Guides and Resources

🛠 Meshery Build & Release Strategy
📚 Instructions for contributing to documentation

Meshery documentation site and source

🎨 Wireframes and designs for Meshery UI in Figma
🙋🏾🙋🏼 Questions: Layer5 Discussion Forum and Layer5 Community Slack

Hello @navendu-pottekkat , can you please assign me this issue?

Here you go @Anand-Theertha 🚀

@Anand-Theertha Are you still working on this issue? I could take this up and get this done over the weekend.

@navendu-pottekkat I'm still on it Navendu. I'll get it done.

Sure @Anand-Theertha . This would be a nice to have ASAP. We have been investing a lot on performance management and a lack of documentation really slows us down. Let me know if you need any help.

Hello, sorry for the delay. Please find the pull request to this issue here - #4192


A small refactoring

Hello.
Thank you for SSCustomTabbar.
A small refactoring.

Hello @vraj-s-shah
Any feedback?

@RomanPodymov  Will check and update you.

@RomanPodymov Thanks for your contribution, this will be added in the next release.


How to Migrate AWS Elastic Search To Azure Elastic Search?

How can we migrate AWS Elastic Search to azure elastic search can this be done using data factory?

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

ID: c1f2edef-d233-6125-1499-7f55d86ecb0c
Version Independent ID: e994602f-882a-327e-9333-4d1cf2c89006
Content: Create an Azure data factory using the Azure Data Factory UI
Content Source: articles/data-factory/quickstart-create-data-factory-portal.md
Service: data-factory
GitHub Login: @linda33wj
Microsoft Alias: jingwang

Thanks for the feedback and bringing this to our notice . At this time we are reviewing the feedback and will update the document as appropriate .

@Anmolgan81 : At this time we have support for Azure Search index as sink in Azure . Please refer this doc .

@Anmolgan81 could you please clarify whether you are intending to use
ElasticSearch on top of Azure or Azure Search or some other solution?

We are trying to use elastic search on top of azure.
Regards
Anmol Ganju

From: MartinJaffer-MSFT<EMAIL_ADDRESS>Sent: Tuesday, April 9, 2019 2:20:27 AM
To: MicrosoftDocs/azure-docs
Cc: Anmol Ganju; Mention
Subject: Re: [MicrosoftDocs/azure-docs] How to Migrate AWS Elastic Search To Azure Elastic Search? (#28973)
@Anmolgan81https://github.com/Anmolgan81 could you please clarify whether you are intending to use
ElasticSearch on top of Azurehttps://www.elastic.co/guide/en/elastic-stack-deploy/current/azure-marketplace.html or Azure Searchhttps://docs.microsoft.com/en-us/azure/search/ or some other solution?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/28973#issuecomment-481002683, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AhHKpx0unIX5p5y5u6l_OMdFFCvgHjJrks5ve6uTgaJpZM4chzoj.
This message contains confidential information and is intended only for the individual named. If you are not the named addressee you should not disseminate, distribute or copy this e-mail. Please notify the sender immediately by e-mail if you have received this e-mail by mistake and delete this e-mail from your system. E-mail transmission cannot be guaranteed to be secure or error-free as information could be intercepted, corrupted, lost, destroyed, arrive late or incomplete, or contain viruses. The sender therefore does not accept liability for any errors or omissions in the contents of this message, which arise as a result of e-mail transmission. If verification is required please request a hard-copy version. IFI Techsolutions Pvt. Ltd. : www.ifi.tech

To use the Elastic Search on top of Azure, you will need to set up the resources yourself.  Data Factory can help you migrate any data you have from AWS to Azure services.  The links here detail how to set up the resources, and how to snapshot and then restore the index from there.  The third article seems particularly useful.
https://www.elastic.co/guide/en/elastic-stack-deploy/current/azure-marketplace.html
https://www.elastic.co/guide/en/elastic-stack-deploy/current/azure-marketplace-getting-started.html
https://www.elastic.co/guide/en/cloud/current/ec-migrate-data.html


How to set up link branding

https://sendgrid.com/docs/User_Guide/Settings/Sender_authentication/How_to_set_up_link_branding.html
Good instructions.  Thanks.
Can you correct this sentence right below step 5. please?
Now links and images in your emails will have be from your custom domain.

@taylor10848 good catch!! Thanks! Updated in the PR linked ^^^


SP4: use color palette to define default color for DataLogger and DataStream

Use color palette to define DataLogger and DataStream trace color as described in #2733

Hi @KroMignon, thanks for your work here and in #2735!
After working on the first bullet of #2733 I forgot there were more recommendations there. Thanks for following-up with this PR!


Update UG for tasks section

This PR updates the tasks section of the UG:

Replacing bullets with tables
Adding a "peek" feature (not a command!!!)
Added some screenshot mockups: Tried putting side-by-side before-and-after screenshots and thought it looked pretty neat. @AY2223S1-CS2103T-T10-1/developers let me know what you guys think!

Codecov Report
Base: 76.45% // Head: 76.68% // Increases project coverage by +0.23% :tada:

Coverage data is based on head (fe78552) compared to base (3197dd5).
Patch has no changes to coverable lines.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #164      +/-   ##
============================================
+ Coverage     76.45%   76.68%   +0.23%
  Complexity      831      831
============================================
  Files           120      119       -1
  Lines          2582     2574       -8
  Branches        329      328       -1
============================================
  Hits           1974     1974
+ Misses          484      476       -8
  Partials        124      124

Impacted Files
Coverage Δ

...ress/model/person/NameMatchesKeywordPredicate.java

...in/java/seedu/address/storage/JsonAdaptedTask.java
100.00% <0.00%> (+10.00%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


Somewhat loosen the types for the standard binding handlers.

Allow observable variables to be bound to all standard binding handlers. Also adjust the allowed types for the css and style bindings.

If valueaccessor unwrap observables, approved. If not, not approved.

Valueaccessor does not unwrap. I think it is the responsibility for each binding handler to do if they allow observables to be bound. The normal behaviour in a binding handler is to unwrap the result of the valueaccessor-call.
See example from knockouts documentation on custom binding handlers
ko.bindingHandlers.slideVisible = {
    update: function(element, valueAccessor, allBindings) {
        // First get the latest data that we're bound to
        var value = valueAccessor();

        // Next, whether or not the supplied model property is observable, get its current value
        var valueUnwrapped = ko.unwrap(value);

This means that custom bindinghandlers that does not accept observables cannot use the BindingContextIdentityTransform that we use.

I think it is the responsibility of each binding handler to use ko.MaybeObservable on the type. Else, the binding handler won't accept observables.
When using an observable, the observable should be called and not sent into the binding handler as an observable, if not an observable, is wanted.

Oops


update berkeley

:exclamation: New Conflict detected :exclamation:
This PR conflicts with one of our main branches. As a result below Pull requests were created to aid you in resolving merge conflicts. Each temporary branch contains cherry picked changes from this PR.

Pull request name
 Temporary branch name
 Conflicting branch

  https://github.com/dkijania/webhook_test/pull/119
 fix-conflict-of-berkeley_conflict-and-develop
 develop


make an administrator number of a group?

this possible?

wow, @pabloruan0710 thank for write code, you are super fast

Promote participant Admin Group not working, any update?


Re-add and configure logic to stage artifacts prior to gh-pages deploy

Commands for moving/copying files are currently commented out in the stagePages() function (in the gh-pages module). This code reflects old outputs when using swagger2markup and asciidoc, and should ideally be updated based on where ReDoc outputs are created.
The key thing to ensure with this step is that artifacts are stored in the correct location to ensure that expected URL paths work on GitHub Pages:

For the master branch of the repository, artifacts are created and hosted at the following paths:

[org].github.io/[repo]/swagger-ui/ — Swagger UI for the API spec
[org].github.io/[repo]/docs/ — reference docs for the API
[org].github.io/[repo]/openapi.json — API spec in JSON format
[org].github.io/[repo]/openapi.yaml — API spec in YAML format

For non-master branches, reviewers can preview documentation and other pages under "[org].github.io/[repo]/preview/[branch-name]/"

swagger-ui/ — Swagger UI preview for current version of [branch-name]
docs/ — docs preview for current version of [branch-name]
openapi.json — spec (JSON) preview for current version of [branch-name]
openapi.yaml — spec (YAML) preview for current version of [branch-name]

Caveats:

Because swagger2markup and asciidoc support isn't currently implemented, nothing will actually be created that would be destined for docs/ directories
As a first step, we can probably just replace (or supplement) the swagger-ui/ directories with redoc-ui/ paths; however, I imagine that ReDoc outputs can eventually live under docs/ without the need for an additional UI directory

I realized this issue is irrelevant with the switch to ReDoc (#5).


Errors not showing ..

Issue Type: Feature Request
Somehow i disables the feature which was showing errors
VS Code version: Code 1.60.2 (7f6ab5485bbc008386c4386d08766667e155244e, 2021-09-22T12:00:31.514Z)
OS version: Windows_NT x64 10.0.19042
Restricted Mode: No

What feature? What error?


GitHub bridge confuses commits and tags?

When reading a tag, GitHub seems to return the ID of the tag object. This is not the same as the ID of the commit that has been tagged. This means that e.g. we can't see the CircleCI status of a build (which is presumably recorded against the commit rather than the tag). It also means that git fetch fails. e.g.
> "git" "fetch" "origin" "tags/mac-v1.12.1-beta26.1:git-pull-of-088dbfac25c9291b23340a5d9f88c4a0f19d4251.new"
remote: Counting objects: 303, done.
remote: Compressing objects: 100% (26/26), done.
remote: Total 303 (delta 162), reused 157 (delta 155), pack-reused 121
Receiving objects: 100% (303/303), 3.82 MiB | 841.00 KiB/s, done.
Resolving deltas: 100% (192/192), completed with 74 local objects.
error: cannot update the ref 'refs/heads/git-pull-of-088dbfac25c9291b23340a5d9f88c4a0f19d4251.new': Trying to write non-commit object 088dbfac25c9291b23340a5d9f88c4a0f19d4251 to branch refs/heads/git-pull-of-088dbfac25c9291b23340a5d9f88c4a0f19d4251.new
From github.com:docker/pinata
! [new tag]         mac-v1.12.1-beta26.1 -> git-pull-of-088dbfac25c9291b23340a5d9f88c4a0f19d4251.new  (unable to update local ref)
* [new tag]         mac-v1.12.1-beta26.1 -> mac-v1.12.1-beta26.1

Saw this today, so looks like this isn't fixed:
error: cannot update the ref 'refs/heads/git-pull-of-aac5c4dcbcb2a65de0e845dd429471b5c7892c6e.new': Trying to write non-commit object aac5c4dcbcb2a65de0e845dd429471b5c7892c6e to branch refs/heads/git-pull-of-aac5c4dcbcb2a65de0e845dd429471b5c7892c6e.new

This is working now.

Oops. No it isn't. I just forgot that it might have the commit anyway for some other reason. Here's a failure from today:
[2016-11-02 11:06.42] Fetching PR branch
error: cannot update the ref 'refs/heads/git-pull-of-624daccef414337cc78a7e85c75baf4da26d90e1.new': Trying to write non-commit object 624daccef414337cc78a7e85c75baf4da26d90e1 to branch refs/heads/git-pull-of-624daccef414337cc78a7e85c75baf4da26d90e1.new

what is that .new suffix?

That's the local name of the branch. We're hoping that some remote branch contains the commit we want, but it might not and it might not be the most recent one. So we fetch to a temporary branch (.new) and then try to update the real branch to the actual commit we want.


support for market events (splits and dividends)

For a project of mine I needed to be able to fetch the market events information (splits and dividends). That information is displayed on the web but it was not available through yahoo_finance_api. The patch I propose simply fills this gap.
Hope you'll like it. Thanks a lot for the crate anyways ! :-)
Here is an example of how it is used:
// To make a request, nothing changes from what you're used to.
let conn   = yahoo::YahooConnector::new();
let tick   = "TSLA";
let start  = DateTime::parse_from_rfc3339("2020-08-30T00:00:00.00Z")?.with_timezone(&Utc);
let end    = DateTime::parse_from_rfc3339("2020-09-02T00:00:00.00Z")?.with_timezone(&Utc);
let hist  = conn.get_quote_history(ticker, start, end).await?;

// Accessing the stocks does not change either
println!("{}", ticker);
println!("QUOTES");
for quote in hist.quotes()? {
    let time = DateTime::<Utc>::from(UNIX_EPOCH + Duration::from_secs(quote.timestamp));
    println!("{} | {:.2} | {:.2}", time, quote.open, quote.close);
}

// But in addition you can show the market events as you like
println!("SPLITS");
for split in hist.splits()? {
    let date = DateTime::<Utc>::from(UNIX_EPOCH + Duration::from_secs(split.date));
    println!("{} | {} : {}", date, split.numerator, split.denominator);
}
println!("DIVIDENDS");
for dividend in hist.dividends()? {
    let date = DateTime::<Utc>::from(UNIX_EPOCH + Duration::from_secs(dividend.date));
    println!("{} | {:.3}", date, dividend.amount);
}

I definitely like your contribution! Unfortunately I am quite busy right now, but will hopefully find a slot in the next few days to review your PR.

Your PR is merged to master. I have also added two examples based on your sample usage.


JavaScript

JavaScript
User Story
As a boot camp student
I want the prework notes to be structured on a webpage
So that I can easily find and read the information
Acceptance Criteria
GIVEN a Prework Study Guide website
WHEN I view the study guide
THEN I can see the four topics I learned listed along with a suggestion on what I should study first

Completed JS section of prework-study-guide


User menu's profile is not loading on first time

Describe the bug
Login into ASC using LDAP.
Environment

AEM Version: 6.5.17
Asset Share Commons Version: 3.3.0
Checked on Publish

To Reproduce
Steps to reproduce the behavior:

Login into ASC using LDAP.
See the broken user info section

Expected behavior
User info is rendered properly
Screenshots

Additional context
It loads correctly on page refresh

Fixed via: https://github.com/adobe/asset-share-commons/pull/1001


Adding ROBIN to list as it supports CSI snapshot features.

Updating the list of projects supporting CSI Snapshot.

/lgtm

/assign @kbarnard10


Map the Parent name of the Primary entity when creating a new record

Hello, when I create a new Record using Power APP property OpenQuickCreateForm (ModelDrivenFormIntegration.OpenQuickCreateForm) the lookup field form the parent entity it's not being mapped on the related entity record. But when I use the Quick Create Form native form with the related Sub-grid on the form the parent lookup its being mapped correctly.
new record using powerapps (ModelDrivenFormIntegration.OpenQuickCreateForm)

new record using sub-grid working fine:

@vernetmatias Have you contacted Support regarding this issue you're experiencing? Did you receive a response to your report, or is the issue otherwise resolved? To contact Support: https://powerapps.microsoft.com/support/

Thanks for contacting us. As we haven’t heard back from you, I am closing this issue. If you need further help, please reopen the issue or open a new issue.
@Mattp123 - Closes #861


Make UUID generation lazy

same as https://github.com/getsentry/sentry-python/pull/2131

note that we also have uuid generation on scope.propagation_context now and to make that lazy we first need to encapsulate propagation_context in a class and not just a raw dict as currently implemented.
https://github.com/getsentry/sentry-python/blob/64e0ec53cbc2e4f3b57a5b45562583ce1130b049/sentry_sdk/scope.py#L225-L232
Without also making those lazy, this will not really have a performance impact but we can merge it anyway.

Is there a reason to not land this pull request right now?

we can merge but without the prop context changes it won't have full impact, and that might conflict with the v2 refactor, I will let @sentrivana and @antonpirker decide

I will refactor the propagation context first and then merge this change

This has found its way into the codebase with the refactoring of the propagation context: https://github.com/getsentry/sentry-python/pull/2970

UUIDs in tracing.py are still always generated, as #2970 only made the generation lazy for propagation contexts. Is this intentional?


scalafmt: 0.2.3 -> 0.2.5

You also have to adjust the hash.

ooooops ;)
I did a build but due to the fact that the downloaded file does not contain the version, it just built the old version without caring...


roWholeWordsOnly is not honored

It's not even used by the code, only declared :)

Fixed. roCaseSensitive was also missing.


Basic repo setup

[x] Add CONTRIBUTING.md
[x] Add README.md
[x] Add DCO

closing the issue as i have added all of these.


⚠ Jenkins Plugins Artifactory has degraded performance

In e9ea928, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1154 ms

Resolved: Jenkins Plugins Artifactory performance has improved in d0e44ba after 18 minutes.


KAFKA-9706: Handle null keys/values in Flatten transformation

Fix DataException thrown when handling tombstone events with null value
Passes through original record when finding a tombstone record
Add tests for schema and schemaless data

Signed-off-by: Greg Harris<EMAIL_ADDRESS>Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

This is very similar to #6914 for the InsertField transformation, and uses the same solution and testing strategy.

cc @ncliang @C0urante for review

cc @kkonstantine @rhauch for review

@kkonstantine I fixed the whitespace, can you verify that it's correct?

ok to test

JDK11 green
JDK8 one failure on a single unrelated integration test that is known to be flaky lately.
org.apache.kafka.streams.integration.QueryableStateIntegrationTest.concurrentAccesses
Given the above, I'll move forward with merging this simple bugfix.


node boundaries and attributes for cssClassApplier

Hi Tim,
first thing I have to say: I am very happy about the work you are doing (I would not like to do it ;-)).
I am struggling with the range object (startContainer, endContainer, startOffset and endOffset) in different browsers using rangy 1.3.
Please have a look at the following example:

      Das ist ein Test, der nicht viel Sinn macht.

Selecting "Test" with the mouse, the result in different browser alters (Chrome, Firefox).
While Chrome returns the same textnode ("Test") as startContainer and as endContainer and the corresponding offsets (0 and 4) corresponding to the length of the selection, Firefox would return the node before the selected textnode as startContainer and its full offset (corresponding to its length) and would then return the following node as endContainer with the offset 0.
Using rangy.getSelection().getRangeAt(0) I get a RangyRange, right? Couldn't I expect the same behaviour from all browser?
I am trying to write a generic TextEditor. Now I discovered your CssClassApplierModule which perhaps solves my problems (if I don't have do deal with ranges anymore myself).
As far as I understood, I can add arbitrary cssClasses to the selection using the span element by default and remove them by using toggleSelection, adding and removing of spans with classes is handled automatically
This works fine as far as I have seen. This way I don't have to deal with range objects directly.
There is one thing I have to do: adding atrributes (data-) to the selection. This works with rangy 1.3 (http://stackoverflow.com/questions/20376043/elementproperties-of-rangy-doesnt-work).
My question: each time, I would have to assign a different value to the assigned attribute. Do I have to create another CssClassApplier each time (only to change its elementAttributes)?
Does isAppliedToSelection() only consider the assigned class or does the assigned attribute value also matter?
Thank you very much for your work and your answer. I you are interested in some code (I have tried to write an abstraction layer for the node boundary issue) I would deliver that gladly.
kind regards
Tobias

Hello :)
I really, really like this library and happy to use it :)
In chrome it is working perfectly, but I need to focus on ff ... Unfortunately (as Tobias wrote) it is not working that perfectly on ff, problem is with double-click (due to bug in offset calculating, I think) - when we want to mark word next to previous tag, node gets start offset = 1 , nodeType = 1 (as parent, not 3).
What about this bug? Are you working on this ? :)
Best regards,
Izabella


Consolidate some tags, change multiapps tag

refs #58

Job VTB Documentation on c323e81 wanted to post the following:
View the site here
This comment will be updated on new commits.

Only changes doco, which passed


Fill happiness index

Task
Use the happiness index tool at https://happy-amos.appspot.com/  to submit how you feel about the project.
This is done once at the end of a sprint, during the team meeting retrospective.
Entering your happiness is different from sending stand-up emails; this should not be mixed.
If there are problems with using the service, please switch off ad blockers and the like.

Some people hare having access issues

Issue was resolved


[WIP] feat: add support for functions

This PR adds support for pure functions to AirScript.
TODOs:

[ ] Using function calls as iterables in list comprehension.
[ ] Add inlining tests for functions
[ ] Using non-symbol arguments in function calls.

[ ] Binary expressions (func(a + b))
[ ] Non scalar expressions (func([1, 2, 3, 4]))
[ ] Function calls (func_a(func_b(a)))

Superceded by #351


Pull request review threads GQL broken

This has been broken on GitHub's end for almost 2 months. Been trying to get an answer here but it's not going well.
Probably have to play around w/ other ways to get PR review threads and contents. Bummer.

Got a response, tho its basically "known bug, no ETA".

It’s better than nothing at least :)

How has this still not been fixed 🙄 😞
The fact this isn't a feature really annoys me, more so because there's nothing we can do about it 😕

I know I’m losing my mind not having these. Gonna try to hack a fix today... 😭
Sent with GitHawk

I've already spent an hour trying to find every alternative I could, even looking at the v3 API previews 😅 Couldn't find anything

Lol ya, its totally broken. Updated w/ a reply:
https://platform.github.community/t/pullrequestreviewthread-no-longer-showing-up-in-timeline-for-prs/2755/17
Hoping we can get some clarification on what is going on here. Its impossible to get PR review threaded comments atm w/ the GQL API. Huge disappointment.

Okay so I don't think they're going to fix it... so I've found an alternative. @rnystrom
https://api.github.com/repos/rnystrom/GitHawk/pulls/704/comments?access_token=revoked
https://github.com/rnystrom/GitHawk/pull/704
The above endpoint returns an array of comments, in chronological order. It appears to only include comments from a review (it doesn't include the root comment, or other comments in the feed from my testing). The nodes can contain a "in_reply_to_id" which is always the first comment in a "discussion".
So in the above PR I left a comment "Not sure if whatever rule is causing this..." the ID of which is 146309348. The other three comments in that "discussion" have the "in_reply_to_id" field pointing to the same ID. We obviously then use the time to work out the order of messages!
The same endpoint also has "pull_request_review_id" which can be linked back to the response we already have from the GQL response (By either decoding the id value in the same way we have for comments, or using the deprecated databaseId field). This means we can tie multiple discussions to a single review.
To confirm:

Current GQL response gives us a "Review ID"
Using the endpoint above we can get all the comments

We then need to filter that response down to only the comments with a pull_request_review_id of the review.
Using the original data, we then want to create a new subset which is anything with in_reply_to_id as the ID of the comments we just filtered. (This gives us a dictionary of discussions)

@Sherlouk, I love you.
That is all.
Sent with GitHawk

I needed some loving today 😔
It's amazing what having a 3hr nap at 5pm does to your brain 🤔
Only took me a month

Awesome. So we need to send a second request, but we only need to do it for PRs. I guess paged items might be a little tricky, but maybe we just worry about that later?

Yeah PRs with a lot of review comments will need paging, haven't the foggiest how we would achieve that though (the endpoint is pageable, but wouldn't know when to trigger additional requests)
There's no way that I can see to find out how many comments a review has either, so no way to detect that 🤷♂
I think yeah leave paging for now, get the first 25 or what ever default response is

@rnystrom I'm removing this from the 1.12 milestone as we have got the functionality in now to see review threads.
I'd opt to leave this issue open though so we can further track the original bug with the GQL API and in hopes that they will, eventually, maybe, probably not, fix it!


[FEAT EXTRACTION] CSF (Cerebrospinal fluid) used as a ROI for radiomics descriptors

Description
I'm struggling with feature extraction using the segmented CSF(Cerebrospinal fluid) as the ROI, which is a binary intensity-based segmentation mask. The bottleneck comes during the feature extraction since it appears to not converge at all. My hunch goes for the ill-possed problem underlying the mask sparsity.
PyRadiomics configuration
For instance I'm using a subset of features
firstorder=['Mean', 'Skewness', 'Entropy','Kurtosis']
glcm=['ClusterProminence','Correlation','Contrast']
shape=['MeshVolume','VoxelVolume','SurfaceArea','Elongation','Flatness']
PyRadiomics log file
During processing, the log doesn't move forward in the Computing shape step and stays there forever without showing any progress.
Version (please complete the following information):

OS: Ubuntu 18.04.5
Python version:  3.6.10
PyRadiomics version v3.0

Additional context
Add any other context about your question here.

Can you share the mask file as an example for debugging purposes?

Hi @JoostJM,
Sure thing! Attached to this message is the White Matter mask and also the one I tried out first which is the CSF binary mask.
c3ro_paciente_AS0467_aligned.nii.gz
c4ro_paciente_AS0467_aligned.nii.gz
BTW, something that I tried, in my opinion, to reduce a large number of disconnected components was to use the closing operator (Erode(Dilation()))
Best,
Miguel

To avoid isolated clusters (which could cause errors with feature extraction) you could try using a seed-based segmentation (e.g. on Slicer or ITKSnap) or post-process the mask to remove all clusters except the largest one (both Slicer and FSL have suitable tools).


Feature/review card UI

#35 is merged, can you rebase onto the latest main? @CryoMilo

Guilty apologize for the flaky pipeline XD

done in https://github.com/stamford-syntax-club/course-compose/pull/58


Chisel does not respect claimed blocks [1.17.1]

Using FLAN as claim management system, the Chisels do not respect claimed areas and anyone can start chipping away someone else's blocks.

I am not the mod author?????????

Bits can still be placed in claimed areas after the last commit, both on new blocks and inside existing chiseled blocks.

This issue is still happening on latest 1.18.2 version of the Mod, and it doesn't seems to have been fixed on 1.19 either by looking at the changelogs...


Adds 6th detector from Deeperforensics

I built the image, ran the container, and tested the detector with a video request.  It has passed all checks.


Cannot use OIDC to access subscriptions in another tenant

We have a setup where:

Subscription exists in another tenant from where the OIDC GitHub app exists.
Access is currently granted to users in our tenant by means of Azure Lighthouse: https://docs.microsoft.com/en-us/azure/lighthouse/. We have an AD group in our tenant which has contributor role access in these delegated tenants. Any users in those groups can see and access the subscriptions in these other tenants.
I've successfully configured OIDC login when viewing subscriptions in the same tenant as the OIDC AD Application.
I've added the AD App to the group giving it contributor role to a subscription in another tenant.
When I try and run the GH action, then it says "no subscriptions"

I'm not sure if this is a supported scenario, or something which I'm missing in configuration to support this.

Fixed - this was a configuration issue in our AAD whereby the SP access was assigned to the group with the tenant access, as opposed to the SP being added to the group.


Can’t reach headings from an external source

Can’t reach a heading element (the  internal anchors) from an external source.
e.g. http://somedomain.com/article.html#intro doesn’t work.

It's not clear what you're talking about. If you talking about the anchors generated using the automatic ID generation feature of this library:

The auto-generated anchors do work, based on some quick tests with Edge, Chrome, and Firefox. For example, with this link: https://ndabas.github.io/toc/#Automatic_ID_generation.
Even if they didn't work, that is a non-goal of this library. The only goal is to generate a TOC to show on the page; IDs are generated only as a convenience to allow clicking links in the TOC. If you are linking to headings on the page, you should probably be using IDs assigned manually (or using a build process, on the server-side, or whatever).

If you talking about the anchors generated using the automatic ID generation feature of this library
Yes, that's what I mean.
For example, with this link: https://ndabas.github.io/toc/#Automatic_ID_generation.
It works fine here!
After some tests I detected it's due to jQuery version. Your example uses jQuery v1.12.4, while I was trying to use TOC depending on jQuery v3.1.1.
Try to update jQuery in your example and will notice that it doesn't properly.
Notice that I'm still here talking about targeting an anchor from an external source.

Sure, I will test out the library with newer versions of jQuery, thank you for pointing that out. However, like I said, this is a non-goal of this project, so I wont be testing (or putting in a fix) for this issue specifically. Maybe I can put in some guidance in the docs about the intention and recommended usage of the automatic ID feature.


Bootstrap CDN broken, vendor bootstrap locally

Bootstrap (and other js/css resources) should be vendored locally in case the CDN fails to work. For the last several days the bootstrap CDN has been returning invalid OCSP data, causing Firefox to refuse to load the page:

An error occurred during a connection to maxcdn.bootstrapcdn.com. The OCSP response contains out-of-date information. Error code: SEC_ERROR_OCSP_OLD_RESPONSE

I think you could just used NPM installation.

I do not want to introduce NPM as part of the maintenance of this project. It should be as simple as copying the files to the correct location in the repo and updating the script/style/etc tags in the HTML templates.

I fixed this, going trough the Gerrit submission procedure right now.

@antbbn Link? I was going to spend a bit of time tackling this if you decided not too and couldn't find a Gerrit submission.

Hi, the gerrit submission is here https://go-review.googlesource.com/#/c/23335/


refactor: remove use of payload files

Description
Refactor that removes the use of the payload JSON files. This is expected to significantly improve the ease-of-use of the codebase, and reduce the (duplicate) configuration required.

Pipeline parameters for use in the dev/sandbox environment are defined in the pipeline definition itself (the default arguments for the pipeline function in pipeline.py). These values are also used during the E2E tests, including in CI/CD. Some of these values are taken from environment variables (using os.environ.get(...) so can be altered in the Cloud Build trigger
Pipeline parameters for use in the test and prod environments (via Cloud Scheduler) are defined in the Terraform configuration for those environments
The E2E tests no longer need to override the pipeline_files_gcs_path using an environment variable in trigger/main.py, since the GCS directory used for the assets files is defined as an environment variable anyway (using os.environ.get in the pipeline's default arguments)

How has this been tested?
Unit tests all passing (including trigger-tests)
Full end-to-end test outstanding
Test deployment of changed terraform code outstanding
Checklist

[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have successfully run the E2E tests, and have included the links to the pipeline runs below
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I have updated any relevant documentation to reflect my changes
[ ] I have assigned a reviewer and messaged them

Pipeline run links:

xgboost e2e pipeline failing because of another known issue (unrelated to this PR) https://github.com/GoogleCloudPlatform/vertex-pipelines-end-to-end-samples/pull/15