View lifecycle hook naming

About the detached() lifecycle hook. The documentation says it is invoked when the view that contains the extension is detached from the DOM. Is the view still part of the DOM when detached() is invoked? If it is, detaching() might be a better name for the hook.

It is fired after it is removed from the DOM. We have selected this naming based on the Web Components spec. I agree that it would be nice to have a detaching callback as well perhaps, but we want to stick with the WC callbacks as a basis for our behaviors at the moment.


CORE API: Add PDF download integration

GET /articles/get/{coreId}/download/pdf

That's so good to know. Frankly, I'd never thought about programmatic access to PDFs -- that could help resolve some of the errors that we've had at that stage of the workflow.

I thought we want to use the api's to be able to find and download PDFs where possible

@ceteri Should a pdf_lookup method return a PDF object or should it return a link to the PDF of the publication?

Good question. That depends on whether the returned URL has limitations:

does it require a cookie?
is there potentially a time limit for its use?
are there any other limitations?

Ideally, if there aren't these or other limitations then we'd prefer to simply lookup the PDF URL and it to the KG for later processing.  Alternatively we can refactor the PDF download to an early stage of the workflow if needed.
I've noticed a trend where publishers are claiming to have "open access" articles, but in reality you must be logged in and use your browser/cookie "watermarks" where the PDF link requires some JavaScript to run -- in other words, it's not a direct download URL.  So many of the errors that we see (e.g., Wiley, OUP ScienceDirect) appear to be due to limitations:
https://github.com/Coleridge-Initiative/rclc/blob/master/errors.txt
Although maybe we need to troubleshoot that download code?


Request show the name of the forum thread as well as the post #number.

When fetching https://myanimelist.net/forum/?topicid=2026447#msg66799688 using the api we could have the topic title and the reply number as well like "[MAL Bunkasai: Send decor and event ideas to the Student Council (#37)]

Sorry but we have no plans to support Forum parsing as the data is available through the MAL API.
MAL Docs: https://myanimelist.net/apiconfig/references/api/v2#operation/forum_topic_get


Update to Compose Navigation

Hi, Should I work on this?

@Rajnish23 Yes buddy you can 👍

Thanks. I've started implementing the changes. I found that a lot of things in jetpack compose is being deprecated.
I've updated the gradle versions and started making changes to deprecated components.
Is tthat okay to you, Should I continue making changes? @Spikeysanju

@Rajnish23 Yes buddy, You're in the direction. Please go-ahead and update the changes 👍

@Rajnish23  PR Merged 🎉 , Thank you so much for your contribution 👍


Can the iox-roudi service skip the user/group level when starting pre-allocated memory

Brief feature description
In our usage scenario (communication with cyclonedds shm), sub-service A starts normally, while sub-service B starts with sudo, and the iox-roudi service cannot provide services for these two scenarios at the same time. In order to ensure the normal use of sub-service B, iox-roudi also needs Start with sudo.
So I guess that iox isolates users/groups. I would like to ask if there is a way to solve this problem, such as through iox configuration?
my configuration:
[general]
version = 1

[[segment]]

[[segment.mempool]]
size = 1088
count = 512

[[segment.mempool]]
size = 16448
count = 1024

[[segment.mempool]]
size = 262208
count = 256

[[segment.mempool]]
size = 1048640
count = 256

[[segment.mempool]]
size = 4194368
count = 256

[[segment.mempool]]
size = 8388672
count = 128

When service A starts with sudo and iox-roudi starts normally, an error is reported:
2022-12-19 15:55:25.507 [Warning]: Process cannot send message over communication channel
2022-12-19 15:55:25.507 [Warning]: ICEORYX error! POSH__ROUDI_PROCESS_SEND_VIA_IPC_CHANNEL_FAILED
/root/iceoryx/iceoryx_hoofs/source/posix_wrapper/unix_domain_socket.cpp:367 { iox::cxx::expected<iox::posix::IpcChannelError> iox::posix::UnixDomainSocket::initalizeSocket() -> iox_connect }  :::  [ 13 ]  Permission denied
permission to create unix domain socket denied "/tmp/iceoryx_rt_1312_1671436518216473058"
/root/iceoryx/iceoryx_hoofs/source/posix_wrapper/unix_domain_socket.cpp:238 { iox::cxx::expected<iox::posix::IpcChannelError> iox::posix::UnixDomainSocket::timedSend(const string&, const iox::units::Duration&) const -> iox_setsockopt }  :::  [ 9 ]  Bad file descriptor
invalid file descriptor for unix domain socket ""
2022-12-19 15:55:25.527 [Warning]: Process cannot s

Detailed information
E.g. a list of pros and cons of different considerations and how iceoryx and its user could benefit from it

@YeahhhhLi Thanks for opening this issue. Did you have a look at the access control example? There, the setup regarding users and groups is described in detail. Unfortunately, user/groups configuration via the .toml file is not possible at the moment and you need to build a custom RouDi as described in the example. Typically, there is no need to start app from an iceoryx perspective with root and I also wouldn't recommend to do so for security concerns. Would that be possible in your setup?

Thanks for reply

Did you have a look at the access control example? There, the setup regarding users and groups is described in detail.

Yes I did.
However, when we use the cyclonedds shm communication method, the iox service is involved, and we hope that we can meet our needs by modifying the .toml file.
So, is it possible to support the cancellation of shm read and write permissions in the future (of course, it is up to the business side to decide, and the security risk is also borne by the business side)
At the same time, there is another problem. I mentioned this issue before but wrongly mentioned it in the rust repo. (https://github.com/eclipse-iceoryx/iceoryx-rs/issues/67)
Although C++ iox lib is not supported now, is it possible to support it in the future, so that we can start the iox service in our own service initialization phase

@YeahhhhLi

we hope that we can meet our needs by modifying the .toml file.

Turns out I was mistaken. It should be possible to configure the groups with a .toml file. Can you try the following? I used the same values as in the example.
[general]
version = 1

[[segment]]
reader = "unprivileged"
writer = "privileged"

[[segment.mempool]]
size = 128
count = 10000

I think none ever used this feature. Maybe so pull requests are needed for fine tuning ;)
If you wanna extend ice_access_control and contribute a .toml which does the same thing as the custom build RouDi, this would be much appreciated :)

So, is it possible to support the cancellation of shm read and write permissions in the future

Everything described in the example is possible to configure. The basic requirement is: Every user needs to be in the iceoryx group, which needs to be configured in your underlying operating system. I think this group might be missing for your root user? Again, running applications as root can easily become a security risk and probably another solution should be preferred.

At the same time, there is another problem. I mentioned this issue before but wrongly mentioned it in the rust repo. (https://github.com/eclipse-iceoryx/iceoryx-rs/issues/67)

Let's take the discussion over to #970 and continue there.

I think this group might be missing for your root user?
@mossmaurice  Hello. I just tried to associate the user and user group, and then restarted, and found that some permission errors will be reported

Logs printed by self-service:
Log level set to: [Warning]
/root/iceoryx/iceoryx_hoofs/source/posix_wrapper/file_lock.cpp:53 { iox::cxx::expected<iox::posix::FileLockError> iox::posix::FileLock::initializeFileLock() -> iox_open }  :::  [ 13 ]  Permission denied
permission to access file denied "/tmp/iox-unique-roudi.lock"
2022-12-20 12:20:36.228 [ Fatal ]: Error occurred while acquiring file lock named iox-unique-roudi
2022-12-20 12:20:36.229 [ Error ]: ICEORYX error! ICEORYX_ROUDI_MEMORY_MANAGER__COULD_NOT_ACQUIRE_FILE_LOCK

Logs printed by iox-roudi service:
2022-12-20 12:21:28.557 [Warning]: Process cannot send message over communication channel
2022-12-20 12:21:28.558 [Warning]: ICEORYX error! POSH__ROUDI_PROCESS_SEND_VIA_IPC_CHANNEL_FAILED
2022-12-20 12:21:30.127 [Warning]: Application iceoryx_rt_2712_1671510036337922976 not responding (last response 1569 milliseconds ago) --> removing it
/root/iceoryx/iceoryx_hoofs/source/posix_wrapper/unix_domain_socket.cpp:367 { iox::cxx::expected<iox::posix::IpcChannelError> iox::posix::UnixDomainSocket::initalizeSocket() -> iox_connect }  :::  [ 13 ]  Permission denied
permission to create unix domain socket denied "/tmp/iceoryx_rt_2712_1671510036337922976"
/root/iceoryx/iceoryx_hoofs/source/posix_wrapper/unix_domain_socket.cpp:238 { iox::cxx::expected<iox::posix::IpcChannelError> iox::posix::UnixDomainSocket::timedSend(const string&, const iox::units::Duration&) const -> iox_setsockopt }  :::  [ 9 ]  Bad file descriptor
invalid file descriptor for unix domain socket ""

Let me describe our usage scenario again so that you can understand it in detail:
1、Start iox-roudi:
my iox toml file:
[general]
version = 1

[[segment]]

[[segment.mempool]]
size = 1088
count = 512

[[segment.mempool]]
size = 16448
count = 1024

[[segment.mempool]]
size = 262208
count = 256

[[segment.mempool]]
size = 1048640
count = 256

[[segment.mempool]]
size = 4194368
count = 256

[[segment.mempool]]
size = 8388672
count = 128

Then I start the service by the following command:
/abs/path/iox-roudi -c aos/transport/cyclone/config/iox_config.toml

2、Start our own service

Opening with sudo is due to some other needs of our service

sudo ./my_services  {argv}

When the two services are started successively, the error at the beginning is reported.
And I found some files about iox  in  /tmp:
total 4
-rw------- 1 root  root            0 Dec 20 12:19 iceoryx_rt_2459_1671509945237909683.lock
-rw------- 1 root  root            0 Dec 20 12:20 iceoryx_rt_2712_1671510036337922976.lock
-rw------- 1 ye.li domain_users    0 Dec 20 12:18 iox-unique-roudi.lock
srw-rw---- 1 ye.li domain_users    0 Dec 20 12:19 roudi
-rw------- 1 ye.li domain_users    0 Dec 20 12:18 roudi.lock

Is it related to these temporary files?

@YeahhhhLi Thanks for the detailed post!
Which version of iceoryx are you using?

just tried to associate the user and user group, and then restarted, and found that some permission errors will be reported

On some embedded systems permissions changes might not be permanent and will be reset. Could you try again without a restart?
For setting up the shared memory configuration, iceoryx uses sockets to communicate from the application to RouDi and vice-versa during startup. The permissions of the sockets are configured so that every user in the same group can access it as you can see in your output:
srw-rw---- 1 yeah domain_users    0 Dec 20 12:19 roudi

Here, all users in the group domain_users can write to RouDi's socket.
Additionally, every application creates a .lock file, to make sure RouDi and the other applications can only be started once. The lock file of course is only available to the users and not to groups:
-rw------- 1 root  root            0 Dec 20 12:19 iceoryx_rt_2459_1671509945237909683.lock
-rw------- 1 yeah domain_users    0 Dec 20 12:18 roudi.lock

What I can tell from the above logs is that, your my_service application is started as root. Therefore, if you want to keep using the root user you need to adapt the groups in such a way that the socket of RouDi can be accessed. And vice-versa RouDi can access the socket of the application.

Opening with sudo is due to some other needs of our service

Again, I would suggest you to think about lowering the requirement to run your own service as root for security reasons. Could you refactor  ./my_services  {argv} in such a way that running it as root is not needed?

Is it related to these temporary files?

It's very likely, if the lock files in /tmp where created with the wrong permissions before they will not be removed and re-created. Hence, before any new try you should always delete them with rm -rf /tmp/*.
Summary:

rm -rf /tmp/* before starting the apps
All users need to be in a common group like in the ice_access_control example.

Hope this help!

@YeahhhhLi can this issue be closed?

@YeahhhhLi closing this issue. Feel free to reopen it if you still have questions


Broken self-referential link in main readme

Line 93 of the main readme.md needs to have the repo name updated to reflect the new ai-notes schema.

ty!


Add format date is expected in.

I seem to have mucked up my signature :(
I was playing with the yubikey and broke everything...


Add request notification to checkout person view of user profiles

Just at bass media checkout; requested reservations dont show up under user profiles in the checkout view.

all done!


unification of workflow descriptions

[Environment]: Kubernetes, bare metal
[Known affected releases]: master (includes all releases)
Currently, there are two workflow descriptions: 1) ASL and 2) KNIX-specific.
We should extend ASL that is accepted by the platform, so that we can support KNIX-specific functionalities (e.g., session functions, checkpointing flag) and can have a unified workflow description.

Started implementation on branch  feature/asl_knix_flags.

Added some test cases that utilize session functions as well as checkpoints flag. These tests pass.
Need to update GUI for validation checks.

The complete list of KNIX-specific flags:

SessionFunction (bool): specifies whether this function is a session function (default: false)
SessionFunctionParameters (dict): specifies the parameters of a session function (default: {})
EnableCheckpoints (bool): specifies whether checkpoints between states should be taken (default: true)
AllowImmediateMessages (bool): specifies whether functions can send immediate messages to other functions to trigger them asynchronously (i.e., no response). The usage of sendToFunctionNow() API call can cause unexpected behavior, because the triggered function can continue making transitions in the specified state machine. (default: false)

In the API docs, we should add warnings to the API functions that are relevant to KNIX-specific features. These include:

session related functions
immediate message functions
dynamic workflow manipulation functions
CRDT operations
any others?


Converting notebooks for DC2 using the Gen2 Butler

This is a PR to track the conversion of Stack Club notebooks to DC2 Gen2.

[x] Basics/ButlerTutorial.ipynb (@DouglasLeeTucker)
[x] Basics/Calexp_guided_tour.ipynb (@DouglasLeeTucker)
[x] Basics/Exploring_A_Data_Repo.ipynb (@DouglasLeeTucker)
[x] ~Basics/Gen3ButlerTutorial.ipynb (@DouglasLeeTucker)~ (not suitable for DC2)
[x] Basics/afw_table_guided_tour.ipynb  (@DouglasLeeTucker)
[x] ~Basics/dm_butler_skymap.ipynb~ (never ran on RSP)
[x] Deblending/scarlet_tutorial.ipynb (@kadrlica)
[x] Deblending/lsst_stack_deblender.ipynb (@kadrlica)
[x] GettingStarted/FindingDocs.ipynb (@kadrlica)
[x] GettingStarted/HelloWorld.ipynb (@kadrlica)
[x] GettingStarted/ImportTricks.ipynb (@kadrlica)
[x] ImageProcessing/BrighterFatterCorrection.ipynb (@kadrlica)
[x] Measurement/UndersampledMoments.ipynb (@kadrlica)
[x] ~Measurement/DwarfGalaxySrcOverdensity.ipynb~ (not suitable for DC2)
[x] Measurement/AsteroidLightCurve.ipynb (@MelissaGraham)
[x] SourceDetection/Footprints.ipynb (@kadrlica)
[x] SourceDetection/LowSurfaceBrightness.ipynb (@kadrlica)
[x] Validation/image_quality_demo.ipynb (@DouglasLeeTucker)
[x] Validation/verify_demo.ipynb (@DouglasLeeTucker)
[x] Visualization/AFW_Display_Demo.ipynb (@kadrlica)
[x] Visualization/LsstCameraGeometry.ipynb (@kadrlica)
[x] Visualization/bokeh_holoviews_datashader.ipynb (@leannep)

@DouglasLeeTucker are you willing to sign off on the review for this?

Yes, definitely!
From: Alex Drlica-Wagner @.>
Date: Friday, April 23, 2021 at 11:53 AM
To: LSSTScienceCollaborations/StackClub @.>
Cc: Douglas L Tucker @.>, Mention @.>
Subject: Re: [LSSTScienceCollaborations/StackClub] Converting notebooks for DC2 using the Gen2 Butler (#252)
@DouglasLeeTuckerhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_DouglasLeeTucker&d=DwMCaQ&c=gRgGjJ3BkIsb5y6s49QqsA&r=R-57Mm7OJX1IswbY4ezVTQ&m=0ASCHzlDnjFjfceZUWbbk8xKfno3hmuIHc7odfD-_Jo&s=09KvOiCx1M8_3vW3GCZMcOy-eT88XkRLB2MM4Ucn7Ls&e= are you willing to sign off on the review for this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_LSSTScienceCollaborations_StackClub_pull_252-23issuecomment-2D825786734&d=DwMCaQ&c=gRgGjJ3BkIsb5y6s49QqsA&r=R-57Mm7OJX1IswbY4ezVTQ&m=0ASCHzlDnjFjfceZUWbbk8xKfno3hmuIHc7odfD-_Jo&s=GDvGQ37N5O8NdW3_h8R8mI80kFv3gdpSwRGNG4LSEnU&e=, or unsubscribehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AARAHOJ4N63P3WCOGT6KC5DTKGQXFANCNFSM4ZBF44AA&d=DwMCaQ&c=gRgGjJ3BkIsb5y6s49QqsA&r=R-57Mm7OJX1IswbY4ezVTQ&m=0ASCHzlDnjFjfceZUWbbk8xKfno3hmuIHc7odfD-_Jo&s=PqHGYuUjjgG4OTjtJZdqSj8mL01yBnHYgX9_EP8LaR8&e=.

Looks good!  Let's go with it!


Integrate modelprop tsunami wrapper into docker compose files

This is the change to include the modelprop for the tsunami case.
We had already had a lot of stuff prepared, so that it looks like it could work:

There was already a seperate ModelpropTsWrapper that has default constraints to use Medina & Suppasri Schemas.
The constraint management already seperates the Modelprop for earthquakes & tsunamis as it works on the wrapper name (eq-modelprop vs ts-modelprop).
The deus wrapper already filters for modelprop results that are done with the earthquake schemas (currently only sara).

So as far as I can see the only thing that was needed was to adjust the docker-compose files:

rename the modelprop container name (so that we know it is the earthquake specific one)
add the wrapper for the tsunami modelprop.
rename the input & output topics for those wrappers.

Things still todo:

[ ] test with the frontend
[ ] eventually adjust the input topcis for the tsunami modelprop so that we only call it after we run the tsunami run

Just for the completeness:

I wondered about listening to the tsunami service before this here starts.
But there is really no constraint that would require to wait.

I guess if it's not required, we might as well leave it out, right?


Resources go out of synch in GitOps deployments

Bug description
Some resources continuously go out of synch when deploying with a GitOps tool, eg. ArgoCD:

capsule-ca and capsule-tls secrets
capsule-mutating-webhook-configuration MutatingWebhookConfiguration
capsule-validating-webhook-configuration ValidatingWebhookConfiguration

How to reproduce

Deploy Capsule Helm chart with ArgoCD

apiVersion: argoproj.io/v1alpha1
kind: Application
metadata:
  name: capsule
  namespace: argocd
spec:
  destination:
    namespace: capsule-system
    server: https://kubernetes.default.svc
  project: default
  source:
    repoURL: 'https://github.com/clastix/capsule'
    path: charts/capsule
    targetRevision: v0.0.5
  syncPolicy:
    syncOptions:
      - CreateNamespace=true
    retry:
      limit: 3

Check the synch status on ArgoCD

Expected behavior
Once deployed resources should not go out of synch
Logs
$ kubectl -n capsule-system get secret capsule-ca -w
NAME         TYPE     DATA   AGE
capsule-ca   Opaque   2      2m16s
capsule-ca   Opaque   0      2m35s
capsule-ca   Opaque   2      4m21s
...

Additional context

Capsule version: (capsule --version)
Helm Chart version: (helm list -n capsule-system)
Kubernetes version: (kubectl version)

A possible solution for this is to let Capsule create or update the {Mutating,ValidatingWebhookConfiguration} resources.
But we got a problem with the Secret ones, because we're mounting them and, if they don't exist, the capsule-controller-manager Pod cannot be started.
We got two options here:
1⃣ not mounting Secrets but let the Operator write the content to its roots
2⃣ creating an additional Job that checks for the Secret resources and, if they're not there, creating it with the desired content; this would require an additional command in the Capsule code-base.
This is happening only in the scenario where a CA and related certificate is not provided, but I think it's common practice.

2⃣ creating an additional Job that checks for the Secret resources and, if they're not there, creating it with the desired content; this would require an additional command in the Capsule code-base.
the same way, as we do in https://github.com/clastix/capsule-proxy/blob/master/charts/capsule-proxy/templates/post-install-job.yaml?

the same way, as we do in https://github.com/clastix/capsule-proxy/blob/master/charts/capsule-proxy/templates/post-install-job.yaml?

I was thinking about this, yes, but in the end, I think we can easily address this with the opinionated ArgoCD options.
I tested the scenario in an ArgoCD environment: the issue is that ArgoCD doesn't play so well with the 3-way merge patch as Helm 3 does, in fact, it complains regarding the outdated Secret resources content and webhooks CA.
We can easily ask ArgoCD to ignore these fields, using the /spec/ignoreDifferences key, as follows.
spec:
  destination:
    namespace: capsule-system
    server: https://kubernetes.default.svc
  ignoreDifferences:
  - group: ""
    jsonPointers:
    - /data
    kind: Secret
    name: capsule-tls
  - group: ""
    jsonPointers:
    - /data
    kind: Secret
    name: capsule-ca
  - group: admissionregistration.k8s.io
    jsonPointers:
    - /webhooks/0/clientConfig/caBundle
    kind: MutatingWebhookConfiguration
    name: capsule-mutating-webhook-configuration
  - group: admissionregistration.k8s.io
    jsonPointers:
    - /webhooks/0/clientConfig/caBundle
    - /webhooks/1/clientConfig/caBundle
    - /webhooks/2/clientConfig/caBundle
    - /webhooks/3/clientConfig/caBundle
    - /webhooks/4/clientConfig/caBundle
    - /webhooks/5/clientConfig/caBundle
    - /webhooks/6/clientConfig/caBundle
    - /webhooks/7/clientConfig/caBundle
    - /webhooks/8/clientConfig/caBundle
    - /webhooks/9/clientConfig/caBundle
    kind: ValidatingWebhookConfiguration
    name: capsule-validating-webhook-configuration
  project: default
  source:
    path: charts/capsule
    repoURL: https://github.com/clastix/capsule
    targetRevision: v0.0.5
  syncPolicy:
    retry:
      limit: 3
    syncOptions:
    - CreateNamespace=true

Honestly, since this is just an issue with ArgoCD, I wouldn't change the code-base to fix a specific problem.
@bsctl feel free to close the issue if it's good enough.

@prometherion thanks, I think yes. Closing it.


Training model logs 'nan' in message

While training the model, I can see below warning;
Training
/Users/akekane/instructlab/venv/lib/python3.12/site-packages/instructlab/train/lora_mlx/lora.py:130: RuntimeWarning: invalid value encountered in scalar divide
return np.sum(all_losses) / ntokens
Epoch 1: Iter 1: Val loss nan, Val took 0.002s
Epoch 50: Iter 50: Val loss nan, Val took 0.000s
"Val loss nan" is nan expected?

Is this still an issue for you? What version of ilab are you using? Have you tried again?

This was the initial version, unfortunately I had to format my machine and haven't configured it again, But AFAIK this issue is still active.

Closing this as the code has changed drastically since this was opened. If you are still seeing this problem, please open an issue in the instructlab repo


New EC2 instance and can no longer put to S3.

The EC2 node front end of a system I used to maintain was retired recently.  The CEO contacted me frantically because the front end was down, I discovered the retirement, and attempted to restart on the new instance.
At that point. I get this message.
#### Uncaught exception: Error: CDN: Error: Hostname/IP doesn't match certificate's altnames ####
error: Uncaught exception: Error: CDN: Error: Hostname/IP doesn't match certificate's altnames

I'm frankly stymied: the code is shrinkwrapped, I'm getting the same old versions of everything (node v0.10.40 and express-cdn 0.1.3) the source matches git.

google this error, seems to be S3?  did someone change the certs?
On Nov 21, 2015 3:02 PM, "Charlie Martin"<EMAIL_ADDRESS>wrote:

The EC2 node front end of a system I used to maintain was retired
recently. The CEO contacted me frantically because the front end was down,
I discovered the retirement, and attempted to restart on the new instance.
At that point. I get this message.
Uncaught exception: Error: CDN: Error: Hostname/IP doesn't match certificate's altnames
error: Uncaught exception: Error: CDN: Error: Hostname/IP doesn't match certificate's altnames
I'm frankly stymied: the code is shrinkwrapped, I'm getting the same old
versions of everything (node v0.10.40 and express-cdn 0.1.3) the source
matches git.
—
Reply to this email directly or view it on GitHub
https://github.com/niftylettuce/express-cdn/issues/91.

Did google the error. No certs have been changed, and if I launch with
NODE_TLS_REJECT_UNAUTHORIZED=0 npm start

I get past the exception but get a succession of messages saying the files can't be copied to S3.  I'm pursuing this with AWS support too.
These folks current use case doesn't really require a CDN, so another approach would be to simply take the CDN out; any advice you can offer that way would be helpful.  This wasn't my code to start with and I hadn't even looked at it in almost a year, so a little leverage would make a big difference.


give a sane view of local couchbase log files?

If I'm running couchbase server locally and using cbsh... would love to have a cbsh command that...

opens the local couchbase server log files

and merges them by timestamp

optionally like tail -f to follow what's going on

so that i can grep through the trail of evidence using cbsh.

and, fantasy -- cbsh can optionally apply to the ongoing log stream some library of well-tended diagnostic patterns or recipes to tell me why the #@#$*&^ my app is getting that inscrutable error and why that API ain't working right for me.

and, fantasy x2 -- tell me some hints of what I probably did wrong as an app developer with the API.

The project for that is couchbaselabs/observability (a.k.a. CMOS).  That'll be coming soon.  It goes all the way through fantasy bullet #1.


Building API documentation should be disabled when Doxygen is not installed.

Bug description
A clear and concise description of what the bug is.
==> Processing plain cmake package: 'beluga'
==> cmake /mypath/src/beluga/beluga -DCMAKE_INSTALL_PREFIX=/mypath/devel_isolated/beluga -G Unix Makefiles in '/mypath/build_isolated/beluga/devel'
-- Build testing enabled.
-- Build API documentation enabled.
-- Could NOT find Doxygen (missing: DOXYGEN_EXECUTABLE)
-- Configuring done
-- Generating done
-- Build files have been written to: /mypath/build_isolated/beluga/devel
==> make -j4 -l4 in '/mypath/build_isolated/beluga/devel'
Scanning dependencies of target beluga_docs
Scanning dependencies of target gtest
Scanning dependencies of target clang_tidy_findable
Scanning dependencies of target benchmark_beluga
[  2%] Generating API documentation with Doxygen
/bin/sh: 1: DOXYGEN_EXECUTABLE-NOTFOUND: not found
make[2]: *** [docs/CMakeFiles/beluga_docs.dir/build.make:58: docs/CMakeFiles/beluga_docs] Error 127
make[1]: *** [CMakeFiles/Makefile2:1195: docs/CMakeFiles/beluga_docs.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....

Platform (please complete the following information):

OS: Ubuntu Focal, ROS noetic
Beluga version: ed7dfe38bcab566a900b636378e7dbf05981120a

How to reproduce
List steps to reproduce the issue:

make sure doxygen is not installed
clone into catkin_ws/src
catkin_make_isolated

Code snippets or minimal examples are always helpful, if not necessary.
Expected behavior
A clear and concise description of what you expected to happen.
No failures.
Actual behavior
A clear and concise description of what actually happened.
see above
Additional context
Any other information you think could be meaningful to this issue.

Also, doxygen should be listed as doc_depend.


How does the Topics API infer topics related to travel preferences, such as "Adventure Travel" or "Beach Vacations", from a user's browsing history on a travel agent website?

The Topics API infers topics related to travel preferences, such as "Adventure Travel" or "Beach Vacations", from a user's browsing history on a travel agent website through a machine-learning classifier that analyzes the user's browsing behavior. Here's a high-level overview of how it works:

Browsing history collection: The Topics API collects the user's browsing history on the travel agent website, including the URLs and hostnames visited.

Hostname analysis: The API analyzes the hostnames visited to identify patterns and topics. For example, if a user visits multiple websites related to hiking and outdoor activities, the API may infer that the user is interested in "Adventure Travel".

URL analysis: The API also analyzes the URLs visited to identify specific topics. For example, if a user visits a webpage about "Beach Vacations in Hawaii", the API may infer that the user is interested in "Beach Vacations".

Machine-learning classification: The API uses machine-learning algorithms to classify the user's browsing behavior into predefined topics, such as "Adventure Travel" or "Beach Vacations". These algorithms take into account the frequency, regency, and duration of the user's visits to specific websites and web pages.

Topic scoring: The API assigns a score to each topic based on the user's browsing behavior, with higher scores indicating a stronger interest in the topic.

Topic inference: The API infers the user's most likely topics of interest based on the scores and returns them to the travel agent's website.

This reads as LLM-generated. Closing, as I don't see a real question?


Create an operating model for AI development that is compatible with the governance framework

At the moment we are drafting a governance framework that sets out the control required to safely develop and deploy AI applications. Ideally this would be accompanied by an operating model (i.e. more detailed guidance about tools, processes and approaches) for the "safe" development of AI applications, that would then (most likely) adhere to the governance framework.

At the moment we are drafting a governance framework that sets out the control required to safely develop and deploy AI applications. Ideally this would be accompanied by an operating model (i.e. more detailed guidance about tools, processes and approaches) for the "safe" development of AI applications, that would then (most likely) adhere to the governance framework.

NCSC and CISA have put together comprehensive guidelines for this purpose. Perhaps we could lean on some of the standards set there? Just a quick thought, but instead of developing an entirely new model, we predicate our "safe/secure" AI practices on the Secure Design, Development, Deployment, & Operation/Maintenance framework they use in the guidelines?
https://www.ncsc.gov.uk/collection/guidelines-secure-ai-system-development


ui wont scroll up when it types a long answer

I asked a question and it typed out a long answer. It continued past the bottom, but I have to scroll up to see it. Would be nice to keep the latest text in view

Going to close this since we're not concentrating on the UI right now.


Bugfix: Accessing AsyncComputed status property should trigger computed function

Same as value, error, and complete already do. Otherwise, only reading status isn't sufficient for a dependency to get tracked.

Oh, this behavior was somewhat intentional. I think I just copied the comment that mentioned that status ran the function incorrectly.
The idea is that while value and error don't really make sense to access if the function has never run, you may want to access status to see if the task has be initiated. Otherwise the initial state is very hard to ever observe.
I think we should keep an eye on this and see if anyone wants a non-triggering status getter. Maybe there's a separate getter for that.

The class comment should be updated too, since it doesn't mention status:
 * Compute functions are run when the `value`, `error`, or `complete` properties
 * are read, or when `get()` or `run()` are called, and are re-run when any
 * signals that they read change.

The idea is that while value and error don't really make sense to access if the function has never run, you may want to access status to see if the task has be initiated. Otherwise the initial state is very hard to ever observe.

Ah, I see. I found it surprising that complete would trigger a run, but status didn't, so I figured this was an oversight. How about peekStatus for the side-effect free version?


[cocoon] Have cocoon validate the .ci.yaml file on PRs

Although we have a validate_ci_config builder for the framework: https://ci.chromium.org/p/flutter/builders/prod/Linux validate_ci_config, we need a generalized solution for all repos. In addition, if .ci.yaml is malformed, this test will never be scheduled.
cc @CaseyHillers

This has rolled into prod from https://github.com/flutter/cocoon/pull/1229
@christopherfujino do you think we need a post submit version as well?

This has rolled into prod from flutter/cocoon#1229
@christopherfujino do you think we need a post submit version as well?

implementing this check from cocoon post-submit seems non-trivial right?

Yeah. There's a bit of overhead to add (such as possible retries, and logs), and i'm not sure it's worth the effort. I'll mark this as completed.


Facing issue in windows docker agent start.ps1 file.

Determining matching Azure Pipelines agent...
Invoke-RestMethod : An error occurred while sending the request.
At C:\azp\start.ps1:32 char:12

$package = Invoke-RestMethod -Headers @{Authorization=("Basic $base64 ...

       ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : InvalidOperation: (Method: GET, Re...1.14393.100
0
}:HttpRequestMessage) [Invoke-RestMethod], HttpRequestException
FullyQualifiedErrorId : WebCmdletWebResponseException,Microsoft.PowerShe
ll.Commands.InvokeRestMethodCommand

Cannot index into a null array.
At C:\azp\start.ps1:33 char:1

$packageUrl = $package[0].Value.downloadUrl

  + CategoryInfo          : InvalidOperation: (:) [], RuntimeException
  + FullyQualifiedErrorId : NullArray

Downloading and installing Azure Pipelines agent...
New-Object : Cannot find type [System.Net.WebClient]: verify that the assembly
containing this type is loaded.
At C:\azp\start.ps1:39 char:7

$wc = New-Object System.Net.WebClient

  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : InvalidType: (:) [New-Object], PSArgumentExcepti
on
FullyQualifiedErrorId : TypeNotFound,Microsoft.PowerShell.Commands.NewOb
jectCommand

You cannot call a method on a null-valued expression.
At C:\azp\start.ps1:40 char:34

$wc.DownloadFile($packageUrl, "$(Get-Location)\agent.zip")

                             ~~~~~~~~~~~~

CategoryInfo          : InvalidOperation: (:) [], RuntimeException
FullyQualifiedErrorId : InvokeMethodOnNull

Expand-Archive : The path 'agent.zip' either does not exist or is not a valid
file system path.
At C:\azp\start.ps1:42 char:1

Expand-Archive -Path "agent.zip" -DestinationPath "\azp\agent"

  + CategoryInfo          : InvalidArgument: (agent.zip:String) [Expand-Arch
 ive], InvalidOperationException
  + FullyQualifiedErrorId : ArchiveCmdletPathNotFound,Expand-Archive

Configuring Azure Pipelines agent...
Cleanup. Removing Azure Pipelines agent...
.\config.cmd : The term '.\config.cmd' is not recognized as the name of a
cmdlet, function, script file, or operable program. Check the spelling of the
name, or if a path was included, verify that the path is correct and try again.
At C:\azp\start.ps1:68 char:3

.\config.cmd remove --unattended `

+ CategoryInfo          : ObjectNotFound: (.\config.cmd:String) [], Comman

dNotFoundException
+ FullyQualifiedErrorId : CommandNotFoundException
.\config.cmd : The term '.\config.cmd' is not recognized as the name of a
cmdlet, function, script file, or operable program. Check the spelling of the
name, or if a path was included, verify that the path is correct and try again.
At C:\azp\start.ps1:48 char:3

.\config.cmd --unattended `

+ CategoryInfo          : ObjectNotFound: (.\config.cmd:String) [], Comman

dNotFoundException
+ FullyQualifiedErrorId : CommandNotFoundException

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

ID: b31d4f6c-d16d-2ba1-1294-6aa94e328046
Version Independent ID: 88aec599-4410-697d-1002-1c41b1c17bb6
Content: Running a self-hosted agent in Docker - Azure Pipelines
Content Source: docs/pipelines/agents/docker.md
Product: devops
Technology: devops-cicd-agents
GitHub Login: @steved0x
Microsoft Alias: sdanie

You need to enable TLS 1.2 by adding the following line to the top of your Start.ps1 script
[System.Net.ServicePointManager]::SecurityProtocol = [System.Net.SecurityProtocolType]::Tls12

@atedsimple I have enabled TLS 1.2 as u have asked, now it is throwing different error.

Determining matching Azure Pipelines agent...
https://vstsagentpackage.azureedge.net/agent/2.168.2/vsts-agent-win-x64-2.168.2.zip
Downloading and installing Azure Pipelines agent...
New-Object : Cannot find type [System.Net.WebClient]: verify that the assembly
containing this type is loaded.
At C:\azp\start.ps1:39 char:7

$wc = New-Object System.Net.WebClient

  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : InvalidType: (:) [New-Object], PSArgumentExcepti
on
FullyQualifiedErrorId : TypeNotFound,Microsoft.PowerShell.Commands.NewOb
jectCommand

You cannot call a method on a null-valued expression.
At C:\azp\start.ps1:40 char:34

$wc.DownloadFile($packageUrl, "$(Get-Location)\agent.zip")

                             ~~~~~~~~~~~~

CategoryInfo          : InvalidOperation: (:) [], RuntimeException
FullyQualifiedErrorId : InvokeMethodOnNull

Expand-Archive : The path 'agent.zip' either does not exist or is not a valid
file system path.
At C:\azp\start.ps1:42 char:1

Expand-Archive -Path "agent.zip" -DestinationPath "\azp\agent"

  + CategoryInfo          : InvalidArgument: (agent.zip:String) [Expand-Arch
 ive], InvalidOperationException
  + FullyQualifiedErrorId : ArchiveCmdletPathNotFound,Expand-Archive

Configuring Azure Pipelines agent...
Cleanup. Removing Azure Pipelines agent...
.\config.cmd : The term '.\config.cmd' is not recognized as the name of a
cmdlet, function, script file, or operable program. Check the spelling of the
name, or if a path was included, verify that the path is correct and try again.
At C:\azp\start.ps1:68 char:3

.\config.cmd remove --unattended `

+ CategoryInfo          : ObjectNotFound: (.\config.cmd:String) [], Comman

dNotFoundException
+ FullyQualifiedErrorId : CommandNotFoundException
.\config.cmd : The term '.\config.cmd' is not recognized as the name of a
cmdlet, function, script file, or operable program. Check the spelling of the
name, or if a path was included, verify that the path is correct and try again.
At C:\azp\start.ps1:48 char:3

.\config.cmd --unattended `

+ CategoryInfo          : ObjectNotFound: (.\config.cmd:String) [], Comman

dNotFoundException
+ FullyQualifiedErrorId : CommandNotFoundException

Seems like you don't have the correct versions of Framework in your base docker image
Have a look at what we use to build the Docker Image for both 2016 and 2019 server
https://docs.modalitysoftware.com/vsts-agent/customise.html

@atedsimple thanks.

@ashutoshbarve Thank you for the question.
@atedsimple Thank you for providing a solution, consider becoming a contributor.
For further issues, please provide a reference to the specific documentation example and we will assign an author for review.
To receive general support, refer to the Azure DevOps support page.

Closing this issue now.

Hi, I am having the same issue in 2021, as mentioned by @ashutoshbarve ashutoshbarve , tried the soluition by @atedsimple but it didn"t work. What was the final solution to solve the Invoke WebRequest problem?

@masoodamunchen with all due respect we have been using this daily in a production environment and it does work. What are you doing that is different? Maybe you are running this from behind an internet proxy? Also we are now only using Server 2019 hosts as they seem much better than 2016


Building issue with GitHub action (billing limit)

Related to fixing https://github.com/chocoteam/pychoco/issues/26
I think that the building process is fixed (or almost) on the dev_actions branch. I cannot understand why but the building process is failing with windows-2019, so I removed it to have only windows-2022.
Everything is building well with Windows, also on Ubuntu, except the fact that some jobs do not start due to a lack of workers. On Mac, nothing starts and we get this message: The job was not started because recent account payments have failed or your spending limit needs to be increased. Please check the 'Billing & plans' section in your settings.
Apparently we have an issue with GitHub actions limits, which I cannot further investigate because it is linked to the Choco organization. @cprudhom , could you check this when you have the time to see what's the problem? When we have this solved we'll have to make a tag and a new minor version to trigger the pypi update.

We have a Github Free plan with the following features:

Unlimited public/private repos
Unlimited collaborators
2,000 Actions minutes/month
500MB of Packages storage
120 core-hours of Codespaces compute per developer
15GB of Codespaces storage per developer
Community support

We are concerned by the third point.
But, when I look at the Billing plans, in front of Github Actions there is this mention:

Use execution minutes with GitHub Actions to automate your software development workflows. Write tasks and combine them to build, test, and deploy any code project on GitHub. Minutes are free for public repositories.

There is also a link to Learn more about billing. In particular, "Jobs that run on Windows and macOS runners that GitHub hosts consume minutes at 2 and 10 times the rate that jobs on Linux runners consume". So we reach the limit very quickly.
I have no immediate answers other than to drastically reduce the matrix.

It occurs to me that there is a quite work-around.
Indeed, actions are triggered on each push, ignoring wether it is related to dev or release.
So, each time:

the macOS's matrix generates 5 (macOS versions) x 6 (python versions)= 30 jobs,
the windows' matrix generates 1 (windows version) x 6 (python version) = 6 jobs,
the ubuntu' matrix generates 3 (ubuntu version) x 6 (python version) = 18 jobs.

We should consider to run those jobs only when release tag is pushed and in dev mode, running 1 job per OS should be enough. In addition, we should consider reducing the number of job for MacOS to fewer versions of macOS or python.

I tried but it doesn't work as expected.
I had to give up but I think it worth a try

Okay, I'm updating GraalVM version and Choco version, I will also restrict Python version to 3.9+ (https://devguide.python.org/versions/), we'll see if it works.

The billing limit problem is now solved, closing this issue


Timestamp format

Add support for more timestamp formats.

Codecov Report

Merging #1507 into develop will increase coverage by 0.1%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           develop    #1507     +/-   ##
==========================================
+ Coverage    71.41%   71.52%   +0.1%
==========================================
  Files          128      128
  Lines        14196    14212     +16
==========================================
+ Hits         10138    10165     +27
+ Misses        4058     4047     -11

Impacted Files
Coverage Δ

botocore/model.py
97.62% <ø> (ø)
:arrow_up:

botocore/serialize.py
100% <100%> (ø)
:arrow_up:

...ndored/requests/packages/urllib3/connectionpool.py
46.07% <0%> (+0.35%)
:arrow_up:

botocore/credentials.py
98.8% <0%> (+0.39%)
:arrow_up:

botocore/vendored/requests/adapters.py
51.42% <0%> (+0.57%)
:arrow_up:

...vendored/requests/packages/urllib3/packages/six.py
65.89% <0%> (+0.92%)
:arrow_up:

...e/vendored/requests/packages/urllib3/util/retry.py
60.2% <0%> (+4.08%)
:arrow_up:

Continue to review full report at Codecov.

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


cookie-store in core no longer supported?

I cannot use it in the lastest version. Is there any other usage that can substitute the *cookie-store*?

@lianxiangru hmm.. looking at the code for it, the support is still there, can you describe the problem you're seeing with it? It may end up being a bug

@dakrone Hi! I have read the code, only the core_old.clj file contains *cookie-store*. It appears in the definition of this dynamic variable and the request function.
Both are deleted in the core.clj.

Ahh okay, it looks like when I added the cookie store - https://github.com/dakrone/clj-http/blob/master/src/clj_http/core.clj#L276-L277 I only added the support for the :cookie-store key in the option map and not *cookie-store*. I can re-add that in. In the meantime, you can pass the store with :cookie-store in your requests

Also, the behavior of cookie store in core.clj and core_old.clj are different.
In old version ([clj-http "2.1.0"]) I have
(let [cookie-store (clj-http.cookies/cookie-store)]
  (client/get "http://zhihu.com" {:cookie-store cookie-store})
  (print (clj-http.cookies/get-cookies cookie-store)))
=> {_xsrf {:discard true, :domain www.zhihu.com, :path /, :secure false, :value 627eb084f45cd6d7d6f42489ca6fc5f3, :version 0}, cap_id {:discard false, :domain zhihu.com, :expires #inst "2016-05-18T14:36:31.000-00:00", :path /, :secure false, :value "NjA0MjZmYjFlY2YxNGEyMWJmNzIyMDkyNWJhMGM4YWU=|1460990191|cf88dbb83f5f49e0e39ac2061e7d830f1f1876fd", :version 0}, l_cap_id {:discard false, :domain zhihu.com, :expires #inst "2016-05-18T14:36:31.000-00:00", :path /, :secure false, :value "ZTdhNjc4NTFhNzQyNDExMDg2YTc0ZGY2NWU2NjdlZmQ=|1460990191|af86b27c05c85bf4052ddb4cffce4396dea2c781", :version 0}, l_n_c {:discard true, :domain zhihu.com, :path /, :secure false, :value 1, :version 0}, n_c {:discard true, :domain zhihu.com, :path /, :secure false, :value 1, :version 0}, q_c1 {:discard false, :domain zhihu.com, :expires #inst "2019-04-18T14:36:31.000-00:00", :path /, :secure false, :value 78bec6726d054f51b721412df40497ae|1460990191000|1460990191000, :version 0}}

In the new version ([clj-http "RELEASE"]) with the same code I have
(let [cookie-store (clj-http.cookies/cookie-store)]
  (client/get "http://zhihu.com" {:cookie-store cookie-store})
  (print (clj-http.cookies/get-cookies cookie-store)))
=> {_xsrf {:discard true, :domain www.zhihu.com, :path /, :secure false, :value 35d9ed038e614a3ce2ae0e8f2ae0ca3d, :version 0}, l_n_c {:discard true, :domain zhihu.com, :path /, :secure false, :value 1, :version 0}, n_c {:discard true, :domain zhihu.com, :path /, :secure false, :value 1, :version 0}}

This actually causes some operations like loggging in failure in the new version. Is this a bug in the new version?

@lianxiangru the actual behavior of the cookie store itself I haven't touched, the only thing that has changed has been an Apache HTTP version upgrade, it sounds like perhaps that might be what's causing this?

@dakrone Probably, let me change it back to 4.5.1 to see whether it works.

@dakrone Still not working.
user=> (let [cookie-store (clj-http.cookies/cookie-store)]
  #_=>   (client/get "http://zhihu.com" {:cookie-store cookie-store})
  #_=>   (print (clj-http.cookies/get-cookies cookie-store)))
{_xsrf {:discard true, :domain www.zhihu.com, :path /, :secure false, :value ac799097c4ac9ad579435efdc55747e1, :version 0}, l_n_c {:discard true, :domain zhihu.com, :path /, :secure false, :value 1, :version 0}, n_c {:discard true, :domain zhihu.com, :path /, :secure false, :value 1, :version 0}}nil

It still returns the shorter cookie even with the older 4.5.1 dependency:
  :dependencies [[org.apache.httpcomponents/httpcore "4.4.4"]
                 [org.apache.httpcomponents/httpclient "4.5.1"]
                 [org.apache.httpcomponents/httpmime "4.5.1"]
                 [commons-codec "1.10"]
                 [commons-io "2.4"]
                 [slingshot "0.12.2"]
                 [potemkin "0.4.3"]]

Closing this for now since it's old and the user has deleted their profile, we can re-open if anyone else sees it.


Feature/mst cli

What is this pull request?
Command line instruments for multi-signatures transactions.
Why do you implement it? Why do we need this pull request?
Need to sign uncompleted/multi-signature transactions
Details/Features
List of features / major commits

Add
Refactor interactive cli
Add interactive multisig
Add new query: get_uncompleted_transactions

Codecov Report

Merging #706 into feature/mst-trunk_rebased will decrease coverage by 0.06%.
The diff coverage is 6.89%.

@@                      Coverage Diff                      @@
##           feature/mst-trunk_rebased     #706      +/-   ##
=============================================================
- Coverage                      42.68%   42.62%   -0.07%
=============================================================
  Files                            328      328
  Lines                          12973    12995      +22
  Branches                        7253     7264      +11
=============================================================
+ Hits                            5538     5539       +1
- Misses                           720      740      +20
- Partials                        6715     6716       +1

Impacted Files
Coverage Δ

iroha-cli/client.hpp
50% <ø> (ø)
:arrow_up:

irohad/model/converters/pb_query_factory.hpp
100% <ø> (ø)
:arrow_up:

irohad/model/queries/get_transactions.hpp
0% <0%> (ø)
:arrow_up:

...odel/converters/impl/pb_query_response_factory.cpp
41.86% <0%> (-0.5%)
:arrow_down:

iroha-cli/client.cpp
35.71% <20%> (-4.29%)
:arrow_down:

irohad/model/converters/impl/pb_query_factory.cpp
34.61% <6.66%> (-2.71%)
:arrow_down:

Continue to review full report at Codecov.

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


Bump version and fix package.sh

Taking the new Github Actions for a spin.

@AndKaminer super useful the Github Actions :+1:  thanks.


[2pt]  Display joined missions in My profile - Filters

Render a list of all joined missions (use filter()) on the "My profile" page.

There is a PR for this issue - https://github.com/clintonjosephs/Space-traveler/pull/43


Error after install - 2019-06-22 : 08:41 AM

Unhandled Error
TypeError: Cannot read property 'editor' of undefined
at Object.reportButton (app://./js/app.d520568f.js:1:85933)
at handleError (C:/Program Files/kbide/resources/app.asar/node_modules/electron-unhandled/index.js:58:13)
at error (C:/Program Files/kbide/resources/app.asar/node_modules/electron-unhandled/index.js:80:4)
at invokeFunc (C:/Program Files/kbide/resources/app.asar/node_modules/lodash.debounce/index.js:160:19)
at trailingEdge (C:/Program Files/kbide/resources/app.asar/node_modules/lodash.debounce/index.js:207:14)
at timerExpired (C:/Program Files/kbide/resources/app.asar/node_modules/lodash.debounce/index.js:195:14)

Any error with the latest commit?
Could you please share screenshot of errors?

it ok now


WiFi will not recover after lost then restored

Issue reported here: https://forums.balena.io/t/balena-wifi-connect-issue/361651/10?u=brownster
Summary:
Initial setup works great. The hotspot works great and captive portal works great sending configuration and establishing a client mode connection while killing the hotpsot.
Test sequence:
Kill the WiFi
Hotspot starts as it should
Bring WiFi back
Device will not reconnect to WiFi. Hotspot stays active
A manual restart will fix the issue an reconnect to Wifi
Interesting: if you try to connect to the hotspot (prior to rebooting) the captive portal will list any available WiFi networks except for the original one it was connected to.

Would be great to have this feature. We've used this example before and had issues in our fleet where an intermittent connection forced it into Hotspot mode. The WiFi automatically recovered, but WIFI Connect stopped it from getting back to the internet. It required asking someone locally to use their phone to re-enter the WiFi SSID/Password...


add env variable to format ddev validate command

What does this PR do?
Adds a step to set a changed env variable in the validations workflow in order to set format ddev command dynamically depending on github event.
Motivation
Currently, ddev validate is not run on push to master, as the validations workflow has ddev validate <COMMAND> changed and no integrations are changed.
AI-2263
Additional Notes

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

Test on feature branch
Test PR: https://github.com/DataDog/integrations-core/pull/11889
on pull validations run: https://github.com/DataDog/integrations-core/runs/6150439183
on push validations run: https://github.com/DataDog/integrations-core/runs/6150483066


Feature/functional arithmetic

Codecov Report
Base: 99.58% // Head: 99.62% // Increases project coverage by +0.03% :tada:

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

Additional details and impacted files
@@               Coverage Diff               @@
##           development      #45      +/-   ##
===============================================
+ Coverage        99.58%   99.62%   +0.03%
===============================================
  Files               38       39       +1
  Lines             1214     1316     +102
===============================================
+ Hits              1209     1311     +102
  Misses               5        5

Impacted Files
Coverage Δ

tests/concepts/arithmetic_operators.cpp
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.


Contributing suggestion of improvements

Hello @meilisearch/devrel-team 👋
Here is a suggestion for contributing improvements to avoid this kind of issue we sometimes have: https://github.com/meilisearch/documentation/issues/1882, https://github.com/meilisearch/meilisearch-kubernetes/issues/129
My suggestion is to

update the default template for PRs, see my draft PR here https://github.com/meilisearch/.github/pull/11
update our current CONTRIBUTING.md to be more permissive and let people create PRs without creating necessarily issues for not-really-impactful changes like typo fixes. We can also add a warning a creating a PR without any approval for the change can be closed.

Currently, we sometimes have double notifications for not really important changes.
But most of all, I think it's important to remove the maximum of steps to let people contribute since we are not flooded by contributions yet. Closing PRs is of course something we allow.
I would love to have your opinion @dichotommy @brunoocasali on this question.

I liked the idea @curquiza, and yeah, reducing the notifications burden is something I appreciated ❤

This is a really great idea! Especially with Hacktoberfest coming up!

Thanks for starting the discussion @curquiza .
I love the idea of removing any additional steps which are not super required.
I'm wondering how we could explain that contributors can sometimes skip the usual process: where to communicate about it and what to say? And also what are the criteria to determine if they can skip the process?

I'm wondering how we could explain that contributors can sometimes skip the usual process: where to communicate about it and what to say? And also what are the criteria to determine if they can skip the process?

Indeed, this is the part that is not easy to describe in the CONTRIBUTING...
I can try something, open a PR, and make you in review?

sure, happy to help if I can

Hello, I'm late to the party, but I agree with both of your suggestions @curquiza . Fewer notifications for us and fewer barriers for contributors, what's not to like! Happy to review once a PR has been created for the contributing guide.


Docs

Add a separate guide and describe available methods. (or at least provide some examples in the README)

@im-adithya Can you please add an example of how you would boost() someone to the README?
Would love to see a complete example using LightningAddress to fetch keysend details + send a boostagram with this data.

Opened #18


Fix for #175 Remove unnecessary msg.Clone calls

Referring to #175
This will remove msg.Clone from producers perviously cloning the original message for retry.
As RetryMessage already relies on the original stored within a message this is not necessary anymore.
In addition to this, all producers that formerly used specific key formatters have ben adjusted to retrieve the key from metadata.

Coverage decreased (-0.2%) to 65.19% when pulling d402f34e395c5b32eb97090db8ba73a3262afc2d on fix/175_Superfluous_clones into 4cc95505594edaf7dfed67db3284af1db03291d1 on master.

Coverage decreased (-0.06%) to 65.391% when pulling 7e513b41779d9d62a1517cd3f0438113959568b0 on fix/175_Superfluous_clones into 2f7e766a0e5574c6139bbe2f81680a77016c3895 on master.

Coverage decreased (-0.06%) to 65.391% when pulling 7e513b41779d9d62a1517cd3f0438113959568b0 on fix/175_Superfluous_clones into 2f7e766a0e5574c6139bbe2f81680a77016c3895 on master.


How to activate zlib compression?

The docs show how to activate ezstd but don't show how to enable zlib:

To activate zstd compression....

How do you turn on zlib compression?

compressors=zlib should activate zlibcompression.


xSQLServerEndpoint: Should Be Cluster Aware

Details of the scenario you tried and the problem that is occurring:
When the xSQLServerEndpoint resource is applied to all of the nodes in a Failover Cluster Instance (FCI), all of the nodes will connect to the instance and ensure the desired state is true even if the node is not currently hosting the instance.
The DSC configuration that is using the resource (as detailed as possible):
When the following configuration is applied to two or more nodes in a FCI.
xSQLServerEndpoint SQLConfigureEndpoint-Instance1
{
    EndpointName         = 'HADR'
    SQLInstanceName      = 'INST1'

    PsDscRunAsCredential = $SysAdminAccount
}

Version of the Operating System, SQL Server and PowerShell the DSC Target Node is running:
Any
What module (SqlServer or SQLPS) and which version of the module the DSC Target Node is running:
Any
Version of the DSC module you're using, or 'dev' if you're using current dev branch:
Any

The resource should evaluate if the current node is actively hosting the SQL instance. If it is not hosting the instance, it should write a verbose message stating that it does not own the instance, and then silently exit.

To resolve this issue, a parameter called ProcessOnlyOnActiveNode should be created on the Set-TargetResource and Test-TargetResource functions.
The Get-TargetResource function will execute the helper function Test-ActiveNode to determine if the current node is actively hosting the SQL instance and return the result via the read-only property IsActiveNode.
The Test-TargetResource function will use the value of ProcessOnlyOnActiveNode and IsActiveNode to determine if it should continue. If ProcessOnlyOnActiveNode is $true and IsActiveNode is $false, Test-TargetResource will return $true.
I put together this gist as a guide to updating the resource.
Also, see the xSQLServerAlwaysOnAvailabilityGroup resource for a completed example.


Change onConflict() to only pass pushError as argument

serverRecord and clientRecord values can be obtained from pushError using pushError.getServerRecord() and pushError.getClientRecord() respectively.

@danderson00

Looks fine


Updates to tests, configuration and policy files

JSSE and Activation have not been tested for a very long time, tests are being reactivated to test existing code, new code and atomic serialization.

Tests are now running with JSSE and Activation as default for some time in the QA Suite.


Logo change

Please note: This repository is being rebuilt to accept the new volume of token additions and modifications. PR merges will be delayed.

I agree to not ping anybody on Discord/Twitter/email about this pull request. Instead I will inquire by posting a new comment in the pull request if needed.

PRs are reviewed in bulk and and can take up to two weeks to be merged.
This repository is managed using an auto merge action. Please ensure your PR has no deleted lines, and it will be merged.
Please provide the following information for your token.
Please include change to the src/tokens/solana.tokenlist.json file in the PR.
DON'T modify any other token on the list.
At minimum each entry should have

Token Address:
Token Name:
Token Symbol:
Logo: (logo should be uploaded under assets/mainnet//*.<png/svg>)
Link to the official homepage of token:
Coingecko ID if available (https://www.coingecko.com/api/documentations/v3#/coins/get_coins__id_):

Auto merge requirements
Your pull request will be automatically merged if the following conditions are met:

Your pull request only adds new tokens to the list. Any modification to existing
tokens will require manual review to prevent unwanted modifications.

Your pull request does not touch unrelated code. In particular, reformatting changes to unrelated
code will cause the auto merge to reject your PR.

Any asset files added correspond to the token address you are adding. Asset files
must be PNG, JPG or SVG files.

Your change is valid JSON and conforms to the schema. If your change failed validation,
read the error message carefully and update your PR accordingly.

No other tokens shares the same name, symbol or address.

For example, this change would be rejected due to unrelated changes:

The bot runs every 60 minutes and bulk-merges all open pull requests to prevent conflicts.
This means that you need to wait up to 60 minutes for your pull request to be merged or reprocessed.

I'm submitting a ...
[ ] bug report
[x] feature request
[ ] question about the decisions made in the repository
[ ] question about how to use this project
Hi there,
I would like to have the token logo changed.
The PR is https://github.com/solana-labs/token-list/pull/20017
Thanks

added in https://github.com/solana-labs/token-list/commit/402d203d744a9d1290f68158986a5799a9786e10 by @Acumen-Protocol
@Acumen-Protocol can you please reply approving these changes?

I'm submitting a ...
[ ] bug report
[x] feature request
[ ] question about the decisions made in the repository
[ ] question about how to use this project
Hi there,
I would like to add some extensions to ACM
The PR is https://github.com/solana-labs/token-list/pull/20017
Thanks :)


Move comments under variables, to make it visible for VSCode preview

Small changes that improve developer experience with this library
Before:
documentations is not visible in IDE

After:
documentations is visible in IDE and easy to understand for developers

Fixed in v2.0.0 and higher.


Format code to align unaligned code.

Motivation:
Format code to align unaligned code.
Modification:
Reformat the code

@qeesung thanks a lot!


Accessing deeply nested collections

Hi @mikkopaderes, thanks for a great adapter!
I have collections clients -> athletes and clients -> teams. everything works great. I can access athletes and teams with calls like client.get('athletes) and client.get('teams').
when i add a many-to-many relationship between athletes and teams everything mostly works as expected but i can no longer use accessors to retrieve child items. eg. if in a model hook i have loaded a team then any attempt to load a team's athletes via team.get('athletes') will return an empty recordset and i can only retrieve a team' s athletes via a store.query.
If I inspect the buildReference for a team's athletes, the path appears as team\:team_id\athletes. I'm guessing this would need to be clients\:client_id\teams\:team_id\athletes but I don't know how this would be doable in a model definition?

May I see how you've defined your models?

there are no belongsTo anywhere
// team model
export default Model.extend(Validations, {
  name: attr('string'),
  athletes: hasMany('athlete', {
    isRealtime: true
  }),
 ...

// client model
export default Model.extend(Validations, {

  users: hasMany('clientUser'),

  athletes: hasMany('athlete', {
    isRealtime: true,
  }),

  teams: hasMany('team', {
    isRealtime: true,
  }),
  ....

// athlete model

export default Model.extend(Validations, {

  teams: hasMany('team'),

If I inspect the buildReference for a team's athletes, the path appears as team:team_id\athletes. I'm guessing this would need to be clients:client_id\teams:team_id\athletes but I don't know how this would be doable in a model definition?

Yes it should be teams/:teamid/atheletes by default. Does your that subcollection have any documents?

Does that subcollection have any documents?

Yes, a collection of documents that each contain a single referenceTo field to
clients/:client_id/athletes/:athlete_id and where the document id is the same as  that :athlete_id

Ok, got it. Just to confirm, here's the data structure that you have:
{
  "clients": {
    "client_1": {
      "athletes": {
        "athelete_1": { ... }
      }
    }
  },

  "teams": {
    "team_1": {
      "athletes": {
        "athlete_1": {
          "referenceTo": "/clients/client_1/athletes/athlete_1"
        }
      }
    }
  }
}

This should work fine as it is. I'll try to find some time to reproduce it. As a follow up question:

I'm curious what are the contents of /clients/client_1/athletes/athlete_1?
How do you do the store.query to make it work?

Also, if store.query works for you, maybe a better temporary workaround is to override the buildReference of teams.get('athletes') in the model definition.

The structure is a little different from that. A client has 1 or more  athletes and 0 or more teams. Athletes can be in 0 or more teams.
Its:
{
  "clients": {
    "client_1": {
        "athletes": {
            "athlete_1": {
                 .... other doc attributes ....
                 "teams": {
                    "team_1": {
                       "referenceTo": "/clients/client_1/teams/team_1"
                     }
                 }
             }
         },
        "teams": {
           "team_1": {
              "athletes": {
                   "athlete_1": {
                      "referenceTo": "/clients/client_1/athletes/athlete_1"
                   }
               }
           }
        }
    }
  }
}

for the query, i have to know the client id and team id to get a teams athletes
    this.store.query('athlete', {
      buildReference(db) {
        return db.collection('clients')
        .doc(_this.get('client.id'))
        .collection('teams').doc(_this.get('team.id')).collection('athletes');
      }
    })

I just want to make the point also that I'm looking to store all of a client's info in a tree structure within a client, ie no root level collections with cross-client info.

did you get a chance to look at this @mikkopaderes ?

No, unfortunately. Currently in a long vacation and wouldn't be back till mid-December.

Ok, reading back through this I think what you should need to do is to use buildReference() in the model definition since your documents lives under specific sub-collections. Does this work for you?

It sort of does. I need to revisit it. It would involve injecting a service into the model which would have the doc ids needed to build the reference tree - which is probably an anti-pattern in Ember. Doing something with collection groups could also be a possibility.
I'm happy to close the issue and come back again with something more concrete?

I may be wrong but looking at your structure, it seems that you only need client id and team id to get a team's atheletes. You already have access to team id so you'll just have to find a way to make client id available to the team record if it isn't yet.

that's true but I gave the minimum viable example to explain my issue. there are further levels of nesting under athletes with resources that have varying relationship types

I think it's a little bit challenging because of how you structured your data with very deep nesting. There's not much I could do for this as the code is working as intended. Maybe you are better of using query for the very complex nesting.

agreed Mikko. closing the issue.


Implement ASCII-safe mode

There should be an option to omit non-ASCII Unicode characters. This would then allow legacy terminals to also work (in conjunction with a 'plain' aka no-ANSI theme).

See here: https://rich.readthedocs.io/en/stable/reference/console.html?highlight=ascii#rich.console.ConsoleOptions.ascii_only


Make the Blockchain Event Viewer pretty

TBD
Going to be similar to https://hubble.figment.io/livepeer/chains/mainnet

Good enough for now, will tweak over time


Add "Limit-WinGetPackage"

Description of the new feature / enhancement
The Microsoft.WinGet.Client module needs to have parity with the capabilities for winget pin.
Proposed technical implementation details
Refactor code as needed into COM so the WinGet client and the PowerShell modules are using the same interface.

Additional discussion regarding the cmdlet names at:

https://github.com/microsoft/winget-cli/discussions/2980#discussioncomment-4999565


Fix static declarations

As titled.

Thanks! I doubt anyone was relying on these symbols being link-able, especially without any formal extern declarations in the headers.

Thanks! I doubt anyone was relying on these symbols being link-able, especially without any formal extern declarations in the headers.

🍻

🍻


Typescript support

Naturally, people will begin asking about this.

This means, essentially we need to remake the entire thing in Typescript.

Please tell me in which file you are having the issue.


PUGX badges

http://pugx.github.io/
Similar to PyPI, uses shields but has very broken rendering right now. Reach out to PUGX when service gets rolling.

Hi, could you clarify what PUGX is? I'm having a difficult time understanding from the website.


Add LGTM code quality badges

Hi there!
I thought you might be interested in adding these LGTM code quality badges to your project. They indicate how you care about code quality and encourage your future contributors to do the same.
To get an idea of the analyses reflected by these grades, check the alerts discovered by LGTM.
N.B.: I am on the team behind LGTM.com, I'd appreciate your feedback on this initiative, whether you're interested or not, if you find time to drop me a line. Thanks.

Merging.


countdown descending

Hello good afternoon,
someone could give me a help with angular-timer module please
I <timer start-time = "1462916931934" end-time = "1462917003389" finish-callback = "finished ()" interval = "1000">.
as achieving countdown begins descending

Use the attribute countdown  instead of start-time
.
Original doumentation here:
http://siddii.github.io/angular-timer/

By Countdown time runs really fast , is it possible that it decrements exactly like it get incrementing


Repo sync for protected CLA branch

The pull request is created from main638036155649574012sync_temp to main to fix git push error for protected CLA branch

Learn Build status updates of commit a336dac:
:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Fixed Sphinx errors

I got some errors while building the docs with Sphinx.
These changes fix them.
Additionally, a repetition in "barcode.rst" was fixed.

Merged! Thanks for contributing! :)


workflows: publish tembo-pg-cnpg for Pg 14 and 16

This looks to be missing the changes I merged last night.


Work on improved hand menu + improved subsystem query API

Overview
This introduces an improved hand menu script that is completely independent of the solvers/constraints system, and offers much improved visual polish + responsiveness.

[x] Improve our subsystem query API to cache and reuse scratch lists for alloc-free every-frame-queries
[x] "Flipping"-style flip-up effect
[x] Dynamic anchoring of Canvas content
[x] Dampening movement when opposite hand is near
[x] Switching menu between hands
[x] Require open hand
[x] Require FOV cone
[ ] Gaze integration
[x] Work with controllers
[ ] Port existing hand menu UI prefabs to use the new hand menu script

Changes

Fixes: #10811 , #10848 , etc by no longer using HandConstraintPalmUp for hand menus
Improves GrabBar material for fading/smoothing
Adds implicit cast operator for HandJointPose to Pose
Adds TransformPose(Pose pose) method to PlayspaceUtilities
Adds IsShowingHands getter to RiggedHandMeshVisualizer
Adds HandMenu.cs, a standalone, dedicated hand menu script
Refactors some of our XRSubsystemHelpers to cache based on type for improved memory allocation behavior
Adds hand mesh enable/disable to our sample scene hand menu

We appreciate your contribution and thank you for the pull request.
Microsoft Mixed Reality Toolkit version 2 (MRTK2) is currently in limited support. This means that Microsoft is only fixing high priority issues. Unfortunately, this pull request does not meet the necessary priority and will be closed. If you strongly feel that this change deserves more attention, please open a new pull request and explain why it is important.
Microsoft recommends that all new HoloLens 2 Unity applications use MRTK3 instead of MRTK2.
Please note that MRTK3 was released in August 2023. It features an all-new architecture for developing rich mixed reality experiences and has a minimum requirement of Unity 2021.3 LTS. For more information about MRTK3, please visit https://www.mixedrealitytoolkit.org.
Thank you for your continued support of the Mixed Reality Toolkit!

@AMollis uhhhhhhhhhhhh this is an MRTK3 PR


[Feature Request]: Add option to propagate successful storage-api write

What would you like to happen?
Successful storage-api writes should be available via WriteResult
Issue Priority
Priority: 3 (nice-to-have improvement)
Issue Components

[ ] Component: Python SDK
[X] Component: Java SDK
[ ] Component: Go SDK
[ ] Component: Typescript SDK
[X] Component: IO connector
[ ] Component: Beam examples
[ ] Component: Beam playground
[ ] Component: Beam katas
[ ] Component: Website
[ ] Component: Spark Runner
[ ] Component: Flink Runner
[ ] Component: Samza Runner
[ ] Component: Twister2 Runner
[ ] Component: Hazelcast Jet Runner
[ ] Component: Google Cloud Dataflow Runner

@reuvenlax can you just brief me more about the new feature to be added

Successfully written records will be forwarded to an output PCollection after they are written.


[Typo] Fix the typo 'retriable', which means 'retryable'.

What is the purpose of the change
(For example: This pull request makes task deployment go through the blob server, rather than through RPC. That way we avoid re-transferring them on each deployment (during recovery).)
Fix the typo 'retriable', which using 'retryable' is better.
Brief change log
(for example:)

The TaskInfo is stored in the blob store on job creation time as a persistent artifact
Deployments RPC transmits only the blob storage reference
TaskManagers retrieve the TaskInfo from the blob cache

Verifying this change
Please make sure both new and modified tests in this PR follows the conventions defined in our code quality guide: https://flink.apache.org/contributing/code-style-and-quality-common.html#testing
(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (100MB)
Extended integration test for recovery after master (JobManager) failure
Added test that validates that TaskInfo is transferred only once across recoveries
Manually verified the change by running a 4 node cluster with 2 JobManagers and 4 TaskManagers, a stateful streaming program, and killing one JobManager and two TaskManagers during the execution, verifying that recovery happens correctly.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

Closing this PR because the Pulsar connector has been externalized to https://github.com/apache/flink-connector-pulsar/


Fix build & test on linux and osx, add ubuntu CI

Sry made a bad rebase, please squash and merge

Thank you!


Fixed Broken Links (Issue #618)

As suggested by @SauravMaheshkar, I have fixed the 3 links and redirected them to examples/old/ directory.

@imshashank looks like tests are still failing, please advise!

I know @mynameisvinn and I'm really sorry for creating confusion by 2 PR's. You can go forward with this one itself.


define sector_type

This PR introduces a new function sector_type as a high level method to access the sector type without relying on implementation details such as the existence of LabelledNumbers.jl. The objective is to use GradedUnitRanges.sector_type as a generic method to access sector for various types:
GradedUnitRanges.sector_type(S::Type{<:AbstractSector}) = S
GradedUnitRanges.sector_type(F::Type{<:FusionTree}) = ...
GradedUnitRanges.sector_type(FT::Type{<:FusionTensor}) = ...

To be decided:

currently sector_type errors for non-labelled axes. It may instead return NoLabel(), or a custom struct.

The tests are failing now due to an issue in LabelledNumbers. They will pass with https://github.com/ITensor/LabelledNumbers.jl/pull/7.


Heapchecker does not detect mismatched delete

Originally reported on Google Code with ID 471
The following should detect an error, but is not.

    int * p = new int[10];
    delete p;

Must use delete [] when allocated with new [].  This is seen with gperftools-2.0

Reported by oscarl on 2012-10-15 18:25:24

debug malloc will detect this mismatch.


When having both named subscription and queries, query is sent over websockets

When having both named subscription and queries like this

and selecting to send query a over the wire, it is send via websockets (without operationName, see #605).
I suppose this is a bug?

The same here. Graphiql performs the same behavior with named Mutation and Subscription combination. None of them work unless I split them into separate tab/window.

tl;dr I have verified that operationName is not sent went using subscriptions, but the root cause is not in the GraphiQL package itself.

Reproducing
I temporarily hacked in support for GraphQL over websocket to the test GraphQL server in the graphiql package to help diagnose other subscriptions/ws issues.
I found that if I do a query like this...
query a {
  test {
    id
  }
}

subscription b {
  subscribeToTest(id: "my") {
    test {
      id
    }
  }
}

... and run operation a or b, you will be able to see successful interaction over the websocket, but you'll see if you drill into the network dev tools messages like this being sent, with operationName missing:
{
    "query": "\nquery a {\n test {\n id\n }\n}\n\n# query b {\n# test {\n# id\n# }\n# }\n\n\nsubscription b {\n subscribeToTest(id: \"my\") {\n test {\n id\n }\n }\n}\n\n",
    "variables": null,
    "type": "subscription_start",
    "id": 0
}

You can wrap the  GraphiQL fetcher with a function including a debug/console.log and verify that GraphiQL's call to the fetcher includes the operationName.
Root cause
As I noted above, GraphiQL is correctly handing the operationName to the fetcher, but the common fetcher to use with GraphiQL for subscriptions,  https://github.com/apollographql/GraphiQL-Subscriptions-Fetcher does not pass the operationName along.
https://github.com/apollographql/GraphiQL-Subscriptions-Fetcher/blob/master/src/fetcher.ts#L34
Also, rather importantly, that repo has been archived!
Possible next steps:

Close this issue since the heart of the matter is not GraphiQL itself.

However, I'm not 100% satisfied with that answer. No one is likely to fix this issue or provide a workaround as the source of the issue (one that we reference in our test that used in yarn dev) is archived. So maybe we could:

Provide an example GraphiQL fetcher for subscriptions that works.

Migrate https://github.com/apollographql/GraphiQL-Subscriptions-Fetcher/blob/master/src/fetcher.ts into this repo (maybe as a different package?)
Update our test GraphiQL example (the one used with yarn dev) to use that fetcher.

To help support the approach of doing (2) I did some quick testing and verified that we can pass the operationName along if we were able to change the fetcher.

Thanks so much for this very well constructed & researched bug report.
We should indeed figure out a more robust solution for subscribing to wss
for graphiql. I can ask Uri if this code is still alive elsewhere, mayhaps?
And yes, lastly creating our own client just for graphiql makes sense,
eventually as a plugin (as will all ws/wss subscriptions support)
On Fri, Nov 8, 2019 at 4:52 PM Sam Parsons<EMAIL_ADDRESS>wrote:

tl;dr I have verified that operationName is not sent went using
subscriptions, but the root cause is not in the GraphiQL package itself.
Reproducing
I temporarily hacked in support for GraphQL over websocket to the test
GraphQL server in the graphiql package to help diagnose other
subscriptions/ws issues.
I found that if I do a query like this...
query a {
test {
id
}
}
subscription b {
subscribeToTest(id: "my") {
test {
id
}
}
}
... and run operation a or b, you will be able to see successful
interaction over the websocket, but you'll see if you drill into the
network dev tools messages like this being sent, with operationName
missing:
{
"query": "\nquery a {\n test {\n id\n }\n}\n\n# query b {\n# test {\n# id\n# }\n# }\n\n\nsubscription b {\n subscribeToTest(id: "my") {\n test {\n id\n }\n }\n}\n\n",
"variables": null,
"type": "subscription_start",
"id": 0
}
You can wrap the GraphiQL fetcher with a function including a debug/
console.log and verify that GraphiQL's call to the fetcher includes the
operationName.
Root cause
As I noted above, GraphiQL is correctly handing the operationName to the
fetcher, but the common fetcher to use with GraphiQL for subscriptions,
https://github.com/apollographql/GraphiQL-Subscriptions-Fetcher does not
pass the operationName along.
https://github.com/apollographql/GraphiQL-Subscriptions-Fetcher/blob/master/src/fetcher.ts#L34
Also, rather importantly, that repo has been archived!
Possible next steps:

Close this issue since the heart of the matter is not GraphiQL
itself.

However, I'm not 100% satisfied with that answer. No one is likely to fix
this issue or provide a workaround as the source of the issue (one that we
reference in our test that used in yarn dev) is archived. So maybe we
could:

Provide an example GraphiQL fetcher for subscriptions that works.

Migrate
https://github.com/apollographql/GraphiQL-Subscriptions-Fetcher/blob/master/src/fetcher.ts
into this repo (maybe as a different package?)
Update our test GraphiQL example (the one used with yarn dev) to
use that fetcher.

To help support the approach of doing (2) I did some quick testing and
verified that we can pass the operationName along if we were able to
change the fetcher.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/graphql/graphiql/issues/606?email_source=notifications&email_token=AAKOFFYUGE7RYI6N2GVPLSTQSXUUPA5CNFSM4D4PN3T2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDTS5SY#issuecomment-552021707,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKOFF45VXUMFT5XV674MKTQSXUUPANCNFSM4D4PN3TQ
.

Also, so we don't lose track, this issue is related to the same underlying issue:
https://github.com/graphql/graphiql/issues/605

thanks @sjparsons! hopefully we can get to this soon after resolving some higher priority bugs

this should be fixed here:
https://github.com/graphql/graphiql/pull/664


ARTEMIS-2414 Sync before closing file in case data loss

#2740 added sync before closing file in case data loss. Sync would also be called while closing read-only files. This would result in perf penalty in the case of files opened and closed constantly, such as large message deliver/delete, page file read by many queues, etc. Don't sync if file is only opened for reading.

Pushed to keep explicit :)

this is merged but i will open the PR to make sure we have some discussions about it, ok?

Did you see .sync actually taking time on a profiling?
The information I had from a few kernel guys is that a sync call when there's no sync needed would be moot and not cause any issues. Although I can see some barrier between Java and Native layer here.
is that what you saw? a barrier between native and Java, or an actual sync ocurring when nothing was needed?

this is being followed by #2774

@wy96f I figured out: AIOSequentialFile::close(sync) was not closing the file if sync=false. I am fixing it. it's a previous issue that became unveiled when you changed the close.
BTW: Did you see .sync actually taking time on a profiling?
The information I had from a few kernel guys is that a sync call when there's no sync needed would be moot and not cause any issues. Although I can see some barrier between Java and Native layer here.
is that what you saw? a barrier between native and Java, or an actual sync ocurring when nothing was needed?

It might not be the actual sync. A few sync calls would not incur perf penalty, but a great many would.
The thing is, during development testing of #2750 , I found message of PagedReference was cleared by gc while delivering, which caused reading page file to retrieve message. There was no problem in PageReader cache version for the cached PageReader is always open. However in PageReader per queue version, when we moved to next page meaning the last PageReader was closed and removed, we had to reopen PageReader to retrieve message. This resulted in lots of sync when opening and closing, and perf penalty was emerging.