fix: add --parsable to sbatch call for a more robust output parsing

Following discussion in https://github.com/snakemake/snakemake-executor-plugin-slurm/issues/121#issuecomment-2270905243 , this implementation seems more robust than match a word with only numbers (which could break in future versions of slurm, or if a cluster name is made of only numbers).

Thank you for your contribution! I will merge this (after fixing the linter issue) - but a new release has to wait until some more contribs are merged. This contrib does not change behaviour, after all. It may wait a day or two.

Sorry for the linting issue, I fixed it. Of course the current implementation works fine so it can wait for the next release :) . Thank you for time!

no need to apologize - thank you for your contribution once more!
(now, after the holidays, I am still pretty busy, but will work on improving the code once more)


Number.toLocaleString polyfill

There's nice little method of Number, allowing to format value to any locale: Number.toLocaleString.
There're some polyfills for that, which, I suppose, might be included into the central repository.
Just a notion.
Any thoughts?

The second polyfill is not completed yet!

MDN is a bit vague on current support. Presumably you're looking for the additional support for the locales argument, rather than basic support which seems to be relatively universal?  Very happy to take a polyfill.  There are updated guidelines on contributing third party maintained polyfills in the contributor guide.

Actually this issue is just a part of polyfilling INTL standard (see MDN) which is quite spacious and will require lots of efforts to be implemented.

Is this issue covered off by our full support for the Intl polyfill and all the locales?

Ping @dfcreative - is our support for Intl solving this?

See also #497.

@triblondon at some point all my browsers, including IE in compatible mode and node, started supporting toLocaleString, so I don’t have a chance to check it. If you can do it - try this code:
<!DOCTYPE html>
<script src="https://cdn.polyfill.io/v2/polyfill.js?features=Intl.~locale.ru-RU|always&gated=0"></script>
<script>(12345).toLocaleString();</script>

Closing this as the Intl polyfill provides a Number.prototype.toLocaleString method - https://github.com/andyearnshaw/Intl.js/blob/20f322dcb47624cec6ba72addcef107c2de94e71/src/core.js#L40


Can not convert custom resource to the new storage api version

I created my own Custom Resource Definition. Let`s call it logging.customresource.com. It has v1 and I created custom resource logging1.
I need to update this CRD and add v2 as storage version. Some of the properties changed its` names in v2 and the old names removed from this new version.
I deployed conversion webhook and configured it in logging.customresource.com configuration. But conversion webhook does not change and save the actual custom resource logging1 to the storage version. It allows only get the resource to my or other requests to this resource, as I understood.
I would like to make it clear: if I want to mark v1 as deprecated and stop to support it soon, I need to make upgrade of the logging1.
According to documentation

If the storage version changes, existing objects are never converted automatically.

What are the ways to convert resource? I mean saving resource to etcd with the last storage version.
But when I try to modify it, apiVersion=v2 was saved (I checked it in etcd), but the new properties did not apply and the old values are lost. And even if logging1 has apiVersion=v2, I can not add new properties. It is just ignored without errors.
In the section Upgrade existing objects to a new stored version in option 2 said:

Write an upgrade procedure to list all existing objects and write them with the same content. This forces the backend to write objects in the current storage version

What does upgrade procedure mean?
Do I need to delete logging1 of version v1 and then recreate it with v2 or how to make work the existing custom resource but with the last api version? And maybe it is better to watch CRD logging.customresource.com and when storage version changed, I need to recreate custom resource (if I would like to support only v2)?
Also there was a comment https://github.com/kubernetes/website/issues/27534#issuecomment-819236827 and good question about describing what happens in different cases with or without conversion webhook, but it is not described in documentation yet unfortunately.
Documentation link: https://kubernetes.io/docs/tasks/extend-kubernetes/custom-resources/custom-resource-definition-versioning/#upgrade-existing-objects-to-a-new-stored-version

/sig api-machinery
/wg api-expression

/language en

This is mostly:
/kind support
although we could improve the phrasing to help readers understand the meaning. Here's a bit of text to consider rewording:

Write an upgrade procedure to list all existing objects and write them with the same content. This forces the backend to write objects in the current storage version

By “upgrade procedure” I think the document means that you either:

write a document aimed at human readers. One part of that document must explain how to list all existing objects and then write back each object, unchanged. When a person follows this procedure, the write that the trigger prompts the storage version upgrade.
write a computer program. Part of the functionality of that progam will be to list all existing objects and then write back each object, unchanged. That scripted write triggers the storage version upgrade.

You get to choose which approach you prefer.

Thanks!
I found the cause why the parameters added in the v2 were lost and faced the next problem:
I have created go reconciler that can change status subresource of my Custom Resource using client.Status().Patch() from  sigs.k8s.io/controller-runtime/pkg/client.
I created the struct Logging with the fields of my custom resource in the apis/customresources/v1. So when I do client.Get() I get the object of type Logging with apiVersion v1.
Then I upgraded the custom resource logging1 to v2 (added new fields too) manually.
Then Im going to update the reconciler image to start work with the new version of struct in apis/customresources/v2, but did not make it yet, reconcile of previous version of reconciler application has started.
And the client.Status.Patch() method receives object of v1 (because client.Get() returned us the version v1, as reconciler requested) and executes patch, but it removes the new fields of v2 that we added recently. So we have lost the new values.
I would like to know, is it expected behaviour of client.Status.Patch()? If it is expected what is the way to deal with the case I described?

Given that this is a request for help, it's the wrong place, but:
/transfer kubernetes


Set a rating or keyword for favorited photos

Is your feature request related to a problem? Please describe.
One of the most important meta data in my workflow is whether a photo is favorited or not. And it's probably a common need for other people too.
Currently, I don't think there is a built-in option to add this to the exported xmp or exif.
Describe the solution you'd like
I propose adding an argument like --favorite-rating, which would set a 5-star rating on the photo. This would have the advantage that it would work out of the box with apps that display photo ratings.
Another option is something like --favorite-keyword that sets the is_favorite keyword.
Describe alternatives you've considered
Currently I am using the following option as a workaround:
keyword_template = [
    "{favorite?is_favorite}",
]

Thanks for the suggestion.  At one point, I actually had this enabled (the code is still there, just commented out).  I disabled it because it wasn't immediately obvious to me that "favorite" and "star rating" would necessarily mean the same thing to many users.
https://github.com/RhetTbull/osxphotos/blob/6c1650b7cffefc223374f66012393f14d443fa72/osxphotos/photoexporter.py#L1704-L1706
I've got a long-standing but not yet implemented issue (#243) to add an --exiftool-template feature that would allow you to set arbitrary metadata fields using the template system, for example,
--exiftool-template "Rating" "{favorite?5,}"
Your suggestion of --favorite-rating might be worthwhile and simpler for the average use case but implementing a change like this requires passing around the state in quite a few places so it'll have to be something I do when I have time to get around to it.

If I implement this for exif metadata should probably also implement for XMP sidecar.  According to the XMP spec:
XMP:Rating    A user-assigned rating for this file. The value shall be -1 or in the range [0..5], where -1 indicates “rejected”
and 0 indicates “unrated”. If xmp:Rating is not present, a value of 0 should be assumed.
NOTE: Anticipated usage is for a typical “star rating” UI, with the addition of a notion of rejection.

I agree that it probably shouldn't be enabled by default, but having the ability to enable it through a command line argument would be great.
And yes, support both XMP and exif would be great.
Either way, I don't think it's a super high priority request as my keyword workaround works well enough. But I just wanted to raise the feature request and provide a workaround for those who are looking for this functionality.

I've implemented this in v0.50.9 as --favorite-rating:
--favorite-rating
                                When used with --exiftool or --sidecar, set XMP:Rating=5 for photos marked
                                as Favorite and XMP:Rating=0 for non-Favorites. If not specified, XMP:Rating
                                is not set.

Amazing... it works perfectly.
Thank you so much for implementing it so quickly!


Add GoogleFetcher

This allows the fetching of items using files.get from Google Drive

Thanks for your contribution to Apache Tika! Your help is appreciated!
Before opening the pull request, please verify that

there is an open issue on the Tika issue tracker which describes the problem or the improvement. We cannot accept pull requests without an issue because the change wouldn't be listed in the release notes.
the issue ID (TIKA-XXXX)

is referenced in the title of the pull request
and placed in front of your commit messages surrounded by square brackets ([TIKA-XXXX] Issue or pull request title)

commits are squashed into a single one (or few commits for larger changes)
Tika is successfully built and unit tests pass by running mvn clean test
there should be no conflicts when merging the pull request branch into the recent main branch. If there are conflicts, please try to rebase the pull request branch on top of a freshly pulled main branch
if you add new module that downstream users will depend upon add it to relevant group in tika-bom/pom.xml.

We will be able to faster integrate your pull request if these conditions are met. If you have any questions how to fix your problem or about using Tika in general, please sign up for the Tika mailing list. Thanks!

@tballison Thanks for review! Honestly, I wasn't expecting one as I'm mostly pushing this to collaborate with @nddipiazza, however, if it makes sense to work as a group, then I will happily do so.
I am pushing to tika-grpc-3x-features as that is the branch we're working off in terms of using the tika-pipes project, but I'm not sure on the overall state of things -- if you have a clearer picture, it'd be great to know.
Happy to hop on a quick call if it's easier. I'll read your other comments in the meantime :+1:

To the degree we can make small/logical changes in main to achieve the project's goals, all the better? This is definitely a standalone PR that can go straight into main, I think.

Sounds great, I'm glad you think so. Let me sort out your feedback and then we'll go from there. Appreciate the response!

Sorry, what I meant was that over on the tika-grpc-3x branch you should change everything to 4.0.0-SNAPSHOT. On this PR, you should only need to do that for the GoogleDriveFetcher, and you should make this PR against main.

Sorry, what I meant was that over on the tika-grpc-3x branch you should change everything to 4.0.0-SNAPSHOT. On this PR, you should only need to do that for the GoogleDriveFetcher, and you should make this PR against main.

No worries, made that change here: https://github.com/apache/tika/pull/2074/commits/d439aca4bef3960215b7c363b00886a318fb3937

Closing this in favour of https://github.com/apache/tika/pull/2077, which is based off main and targeting it. The tika-grpc-3x-features branch needs significant cleanup.


Update buildupdate.mono.sh for rename libpalaso build (master)

This change is

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


AMD GPU Support / Training Questions

I have a couple of questions regarding the hardware required to train the AI. Is it possible to do this with CPU or AMD GPU's? In the future do you plan on making a GUI to simplify the training procedures?

Back to the first actual question...
AFAIU opencv transparent api should already be able to handle many/most functions through opencl.
As for pytorch, there isn't an opencl backend, but there is an HIP one already: https://github.com/pytorch/pytorch/issues/10657#issuecomment-415067478


Alexa stops building cxx object SampleApp ... UIManager.app (97%)

IMPORTANT: Before you create an issue, please take a look at our Issue Reporting Guide.
Briefly summarize your issue:

What is the expected behavior?

What behavior are you observing?

Provide the steps to reproduce the issue, if applicable:

Tell us about your environment:

What version of the AVS Device SDK are you using?
  <x.y.z>

Tell us what hardware you're using:

[ ] Desktop / Laptop
[x] Raspberry Pi
[ ] Other - tell us more:

Tell us about your OS (Type & version):

[ ] Linux
[ ] MacOS
[x] Raspbian Stretch
[ ] Raspbian Jessy
[ ] Other - tell us more:

This issue sometimes happens because of limited resources on the pi. Do you have any other application running on the pi. You could also modify the shell script to reduce the number of jobs.

Many thanks for your reply. My laptop computer ran out of power whilst send the original message and so I hadn’t completed it and I didn’t actually think it had been sent.
I am running a Raspberry Pi 3 off a 64gb SD card with a fresh build of the latest version of Raspian Stretch. I tried using Raspian Duster but got error messages during the initial updates so went back to Stretch. With that in mind I can’t see how it should be running out of resources with such a large card and only the basic build with no extra apps.
I note that there are several versions of how to implement Alexa on a Pi (including one on the Amazon Developer site) and was wondering if you could suggest the most reliable instructions to try again. I have tried several and many times.
I have got as far as the install recognising the Alexa wake up message but then got a message to do with the network not being connected and working in ‘limited mode’. The microphone and playback audio work fine.
Any help such as a modified script or other suggestions would be much appreciated.
Cheers, Martyn.
Sent from my iPhone

On 15 Jul 2019, at 14:42, Mahesh Velegonda<EMAIL_ADDRESS>wrote:
application running on the pi. You could also modify the shell script to reduce the number of jobs.

Hi @GeekTeacher, it sounds like you basically have the SDK compiled & working, however the client entered 'limited' mode since Authorization failed.  I would suggest that you remove the database files (as listed in the AlexaClientSDKConfig.json file) and go through the authorization steps again.

Hi @GeekTeacher, were you able to try @dhpp suggestion? Did it fix the problem? You can also use the 'Factory Reset' option from the limited mode (key 'k') to clean up all the database files and start again.

I am closing this issue due to inactivity. Please feel free to re-open it if it has been closed in error.

The problem comes from :
Low power supply > when CPU Load is high > CPU drains more power > Raspberry reboots, so the installation is interrupted at this level. 🥇
Solution: plug 2 Amps power supply (instead of computer USB plug suplying), then redo the installation, the high CPU load will

I had the same issue as @GeekTeacher , I identified the problem with reference to @monforum  and solved it using the following steps :
I have a screen and I get the HDMI output over there, I connect a mouse and a Keyboard too.
TLDR - I just turned off bluetooth and VNC, rebooted to Commandline Interface (Non GUI and build it again)
**Solution:**
1. Turn off Bluetooth in the Pi, you can use these commands ( I just right clicked and turned off the icon)

$ echo "dtoverlay=pi3-disable-bt" | sudo tee -a /boot/config.txt
$ sudo systemctl disable hciuart

2. Reboot to Command Line Interface (CLI)
         a. $ sudo raspi-config
         b. Go to boot options
         c. Go to "Enable Boot to Desktop/Scratch
         d. Select Console / Text Autologin (CLI)

3. Reboot the system and re run the build script as earlier.

Suggestion - As you re run the script, try to remove your USB mouse and Keyboard so that they don't consume power. Also using SSH instead of VNC should save you more power in case you don't have a screen available. Turning down VNC is also recommended.
Mentions

- Raspberry Pi 3B+
- Raspbian Buster March 2020 Build
- VNC Enabled
- 5V, 2A Power Source used for pi
- 100mA usage based both Keyboard and Mouse connected (200 mA total)

The problem comes from :
Low power supply > when CPU Load is high > CPU drains more power > Raspberry reboots, so the installation is interrupted at this level. 🥇
Solution:
1. unplug all USB devices (mic, speaker...etc) form your Raspberry to remove any power suply drainer

2. plug 2 Amps power supply (instead of computer USB plug suplying)

3. redo the installation, the high CPU load will

increasing swap size solved for me, change it from 100 to 1024 re execute it finishes at this time, changes at power supply  did not make difference for me

I had a similar issue, the steps I followed to expand the swapfile on the sdcard were:
sudo dphys-swapfile swapoff
sudo nano /etc/dphys-swapfile
Increase the number 100 to 1100_
sudo dphys-swapfile setup
sudo dphys-swapfile swapon
Use swapon -s to view the swap space.
I elected to restart afterwards to verify the change worked after a restart.

I had a similar issue, the steps I followed to expand the swapfile on the sdcard were:
sudo dphys-swapfile swapoff
sudo nano /etc/dphys-swapfile
Increase the number 100 to 1100_
sudo dphys-swapfile setup
sudo dphys-swapfile swapon
Use swapon -s to view the swap space.
I elected to restart afterwards to verify the change worked after a restart.


Feature Request: Jump to ticks

Is it possible to add a feature where you can jump to a specific tick/timestamp?

Oooh I've seen that something similar exists

but I can't get it working

I guess you have to set markers right?

Yes, this is a planned feature

Implemented in 6ab06918766e86d18c32a31f54e04a1d21fe93f1


Test Failure: SEGSEGV from TestGetEventTypes.java during JfrNetworkInterfaceName frame

Core Facts
Frequency: Seen once so far.
Platform: Solaris SparcV9
JDK Version: JDK8u382
Test Super Target: extended.OpenJDK
Test Target: jdk_jfr_0
Test: jdk/jfr/api/flightrecorder/TestGetEventTypes.java
Link: https://ci.adoptium.net/job/Test_openjdk8_hs_extended.openjdk_sparcv9_solaris_testList_1/81/consoleFull
Additional Data
This is an upstream issue that I've raised here
Finally, here's a copy of the full error message:
full_test_output.txt

I'm raising this issue here to:

track the progress of the upstream issue associated with this failure (link).
provide updates when we're close to a fix
allowing non-OpenJDK contributors to add related comments
and to track the temporary exclusion of this test, as well as the unexclusion once we have a proven fix.

Also see other crashes, reported https://github.com/adoptium/aqa-tests/issues/4359 (appears to crash on stop events).

There I was trying to find the log files in the empty workspace. :)
Thanks for the information Shelley! I didn't see those issues. Will close this one and append the information to #4359.
I'll leave the upstream issue open, and I'll add the core+log files and the names of the other affected tests.

Thanks for raising the upstream one, I was just noting the others because it seems to have arisen in same and slightly different forms before.  :)


fix: revert service accounts integration

Revert changes introduced in #1716.

@rkpattnaik780 We are looking to release connectors. Service accounts integration should stay on the feature branch as we would need to give more heads up to people as this will introduce some breaking changes.


Search phrases with hyphens (-) return no results

If a search phrase contains a hyphen (-), then no results are returned back. This can be seen in your example when searching for the term "Rover-2".

Thanks for reporting this.
I've taken a look and the problem is due to a mismatch between what the query parser and tokenizer consider a word separator.
It looks like I thought of this, but never came back to it :man_facepalming:
The simplest fix is to use the same set of separators as used by the tokenizer.
I think the right thing to do is to make it configurable, just like with the tokenizer. I'll have a think about the best way to achieve this.

I've just pushed version 2.0.2 that contains a fix for this.
There might be better ways of doing this that offer more customisation or more control in the future, but for now it should be fine.


Tweaks to network config

While looking at some issues with CNI and hostPort connections mentioned in https://github.com/kubernetes-incubator/kube-aws/issues/704#issuecomment-403265649, I found some discrepancies between the kube-aws network configuration and what's expected or perhaps optimal. I've since discovered the cause of my hostPort issues was something unrelated but the items I found are likely worth including in kube-aws. I'm going to list them here first and we can always split them up.

We are not setting the MTU for Calico in the CNI config. It means that for Canal the MTU of the flannel interface is 8951 but all the Calico interfaces are 1500. It still works but I believe it's not the optimal value. According to the Calico docs, we should set this to 8951 to match flannel:

When using flannel for networking, the MTU for the network interfaces should match the MTU of the flannel interface. In the above table the 4th column “Calico MTU with VXLAN” is the expected MTU when using flannel configured with VXLAN.

We should set externalSetMarkChain to KUBE-MARK-MASQ in the CNI config to reuse existing iptables chains. It's not in the default YAMLs but I found it here. It seems our friend redbaron found the same

externalSetMarkChain - string, default nil. If you already have a Masquerade mark chain (e.g. Kubernetes), specify it here. This will use that instead of creating a separate chain. When this is set, markMasqBit must be unspecified.

We should set the --service-cluster-ip-range flag on controller manager as it prevents IPs being assigned in the service CIDR range should it overlap with the pod CIDR range. This doesn't seem to be documented anywhere I could find, but most bootstrap instructions set it and I just found we didn't set it. Dug into the code and found this.

When using Canal or Flannel, we should be defaulting the podCIDR to <IP_ADDRESS>/16. When using Calico it should be <IP_ADDRESS>/16. References - Calico, Canal, kubeadm instructions and code. I'm wondering what the impact of setting these to anything else is as our default right now is <IP_ADDRESS>/16 and it appears to work for basic functionality. Perhaps the impact is in network policy enforcement?

We should update cniVersion from 0.3.0 to 0.3.1

Hi
Good catch.  I vote for all of the above, except 4 - which I don't rightly understand what is the significance of using one cidr range over another?  Can't we decide our own cidr and then accommodate the defaults and kube manifests around that choice?  What is the advantage of using <IP_ADDRESS> with calico?

@davidmccormick I'm not sure at this stage about 4 and why every set of instructions I found seem to indicate it has to be set to that. At least on the surface our defaults work right now so I assume there's some other subtle issue or just everyone has copied the instructions verbatim 😆  (including kubeadm).

:) I think we should stick with the same default podCIDR unless we find hard evidence to change it.
All the others sound like we should merge asap.

Sure, we can at least do that and have the other bit ready.
One thing though, I'm struggling to identify the cause of a full cluster outage so asked for the PR to not be merged yet (see the message there but either related to this change or
https://github.com/kubernetes-incubator/kube-aws/pull/1397). Nodes started marking themselves a NotReady intermittently and sometimes NotScheduleable until gradually all of them fell over. A dev cluster with a few nodes did not experience that issue which is why we promoted the changes to a larger cluster with more use and that's where the problems started. Actually the dev cluster experienced it on one node just after we had successfully restored the downed cluster. Killing that one node and the dev cluster has been fine for another couple of days.
Surprisingly for the downed cluster a brand new cluster using a backup/restore via ark didn't work, the restored pods appeared to make the new cluster fragile in some way. Redeployed all from source and it was healthy. I suspect it was the CIDR change although it could be the newer Calico or the other network changes. I did follow the instructions from your recent PR with cluster downtime. I found a few kube issues related to issues waiting for CNI plugins to respond which seem like they have similar symptoms.
We still have the node problem detector backlog ticket in kube-aws of course but that'll only patch over the root cause.


macOS support builds but doesn't work

By merging #1 we made it compile for macOS, thanks to @linkineo, but it doesn't work for the moment, due to some paths issues.
It can be reproduced easily by cloning https://github.com/header-only/example-dependency and checking out the list-your-deps tag, then running inglued seal as presented below :
Damiens-MacBook-Pro:inglued daminetreg$ cd example/example-dependency/
Damiens-MacBook-Pro:example-dependency daminetreg$ git checkout list-your-deps
Damiens-MacBook-Pro:example-dependency daminetreg$ ../../build/inglued seal
Fetching and glueing in your git repository direct dependencies:
	 fetching boostorg/fusion...libc++abi.dylib: terminating with uncaught exception of type std::runtime_error: Cannot add the dependency, your working tree must not have changes to already commited files. Output is :
 git fetch https://github.com/boostorg/fusion.git boost-1.62.0
warning: no common commits
From https://github.com/boostorg/fusion
 * tag               boost-1.62.0 -> FETCH_HEAD
error: Invalid path 'deps//fusion/.gitattributes'
error: Invalid path 'deps//fusion/doc/Jamfile'
error: Invalid path 'deps//fusion/doc/acknowledgements.qbk'
error: Invalid path 'deps//fusion/doc/adapted.qbk'
error: Invalid path 'deps//fusion/doc/algorithm.qbk'
error: Invalid path 'deps//fusion/doc/changelog.qbk'
error: Invalid path 'deps//fusion/doc/container.qbk'
error: Invalid path 'deps//fusion/doc/extension.qbk'
error: Invalid path 'deps//fusion/doc/functional.qbk'
error: Invalid path 'deps//fusion/doc/fusion.qbk'
error: Invalid path 'deps//fusion/doc/html/images/fusion_org.png'
error: Invalid path 'deps//fusion/doc/html/index.html'

It might be due to the passing of arguments to the bp::child, but might as well be something else. Have to investigate.

Solve in #3

Indeed checked it's solved in #3 .


pagination design

resolves #1200
result:

It looks like the test had failed due to some network hickups or something. I want ahead and restarted them and they're all green now. This looks good from my end, however @nlhkabu is going to have to review this since it's front end/design work.

Hi @nkanaev, thanks for this PR.
My preference here is to use a button group, so that we don't have to add specific CSS for the pagination (less code, less to maintain). It's also more visually consistent with the rest of the design :)
Based on the work you have done, is there any reason why the button group markup wouldn't work?

i do agree that less css is better. i'd actually implemented it as a button set at first. but then noticed making "..." elements as unclickable links doesn't make sense, so I needed a logical way to add span element. in the end i've ended up implementing something bem'ish (though it's not Gallagher's one).
do have any suggestions on spans? maybe we should just use something like:
...

Yes, let's try using a span with a button class.  If necessary, we can modify the button BEM, but it should work out of the box :)
You can also apply a disabled modifier to the span, e.g.
...

This will make it grey and give it a not-allowed cursor.

final result:

Thanks for upading this @nkanaev :)
I'm still not convinced that a new button set is needed.  I'd much rather modify the existing button group to see if we can make it work.
Does this markup do the trick?
<div class="button-group"
  Previous
  1
  2
  3
  4
  ...
  382
  Next

thanks, totally have missed button-group.
had to adjust a little bit - other buttons were overlapping highlight buttons on mouse hover, which is visible only when overlapping button is set to disabled.

Merged :sparkles:
Thanks very much for this @nkanaev :)


[JAV-591] Completable future response mapper

Coverage increased (+0.3%) to 87.437% when pulling ce029f8539a223155f27510c54b549ce041863f9 on wujimin:completableFuture-response-mapper into dcee4097b14b7493b7885cf3a3b32e44828616fe on apache:master.


phys_reader = PhysPkgReader(pkg_file)

When running my programm with Docx i get the error mentioned above. It originates in \Python\Python311\Lib\site-packages\docx\opc\phys_pkg.py

Closing as stale/inactive.


no entrypoints defined error when saving a valid model

Checklist

[X] I have looked into the README and have not found a suitable solution or answer.
[X] I have looked into the documentation and have not found a suitable solution or answer.
[X] I have searched the issues and have not found a suitable solution or answer.
[X] I have upgraded to the latest version of OpenFGA and the issue still persists.
[X] I have searched the Discord community and have not found a suitable solution or answer.
[X] I agree to the terms within the OpenFGA Code of Conduct.

Description
When I try to save a valid model I get
"error":"the definition of relation 'can_enable_logging' in object type 'entity' is invalid: no entrypoints defined"
If I delete the can_enable_logging relation or if I remove the but not from the has_access_to_logging relation
Expectation
It should not return an error.
Reproduction

Try to save the model  or run fga model validate --file model.fga
It returns

{
  "is_valid":false,
  "error":"the definition of relation 'can_enable_logging' in object type 'entity' is invalid: no entrypoints defined"
}

Store data
model
  schema 1.1

type user

type entity
  relations
    define member : [user]

    define contextual_user: [user]
    define contextual_member : member and contextual_user

    define has_logging_product: [entity]
    define block_logging : [user] and contextual_user

    # If you don't include the "But not", the model validates
    define has_access_to_logging : contextual_member from has_logging_product but not block_logging from has_logging_product

    # If you comment this line, the model validates
    define can_enable_logging : has_access_to_logging

OpenFGA version
Latest version of the CLI
How are you running OpenFGA?
As a binary
What datastore are you using?
In-Memory
OpenFGA Flags
no
Logs
No response

May be related - https://github.com/openfga/openfga/issues/1260


fixes project name length issue

Why
If project name exceeds 49 characters length then Jenkins fails to execute the job.
See https://github.com/openshiftio/openshift.io/issues/2527#issuecomment-401681596
Reverts the PR changes of https://github.com/fabric8-launcher/launcher-backend/pull/468. Updated the right class.
Description
To fix this issue added project constraints on project name length
Checklist

[X] I followed the contribution guidelines.
[ ] I created an issue and used it in the commit message.
[X] I have built the project locally prior to submission with mvn clean install.
[X] I kept a clean commit log (no unnecessary commits, no merge commits).
[X] I am happy with my contribution.

@gastaldi my bad for the previous PR.

Thank you! Good job


Add support for Fedora 36

This issue is created to track progress for the work to enable Fedora 36 support for .NET Core 3.1, .NET 6.0 and .NET 7.0.

Do you have any specific issues/tasks in mind here? It looks like we forgot to open a Fedora 36 issue, but I am not aware of any issues that exist on Fedora 36. I use Fedora 36 as my daily driver and I am not aware of any significant issues that impact .NET there.

Do you have any specific issues/tasks in mind here? It looks like we forgot to open a Fedora 36 issue, but I am not aware of any issues that exist on Fedora 36. I use Fedora 36 as my daily driver and I am not aware of any significant issues that impact .NET there.

I opened a tracking issue as Fedora 36 is not listed as a supported distribution even though it was released on 2022-05-05.

EOL - 16 May 2023. Resolving.


Focusable checkbox

Make our checkbox "focusable" so that the user can check/uncheck via keyboard.

Do we need a new class for that?
Would just adding something like this to the existing wfm-checkbox class be enough?
` input[type='checkbox'] {
opacity: 0;
height: 0;
width: 0;
margin: 0;
}
input[type=checkbox]:focus ~ label .wfm-checkbox-toggle:after,
input[type=checkbox]:focus ~ .wfm-checkbox-toggle:after {
@extend .material-trench-1;
}`
Then you can just add tabindex to existing wfm-checkboxes and they would instantly be focusable. It would also reduce the amount of css in the styleguide and be more DRY :)

Also, should it be added to radio buttons and switches as well?

Thanks for your advice! I'll try to change the existing one instead.
As for the radio buttons and switches, I think we can upgrade them one by one.

I will take a look asap :+1:
Yes of course, keep this PR to the check-boxes and if it works we can spread it later


Disabling json5lint, mdjsonlint,

Describe the bug
Disabling json5lint, mdjsonlint, or stylelint throws the error Cannot read property 'includes' of undefined.
The errors come from the related runChecks function related to
if (that.options.config.enabled && !checksToRun || checksToRun.includes(that.options.key)) {

To reproduce
Steps to reproduce the behavior:

In .fluidlintallrc.json disable any or all of the above mentioned checks
Run npm run lint
Notice that an error is thrown.

Expected behavior
The linting should skip the disabled checks.
Technical details
Desktop

OS: macOS 11
Node: 14.15.0

The checks needed parentheses, as in "the check must be enabled" AND "one of these two things must be true".  Fixing it up now.

The checks needed parentheses, as in "the check must be enabled" AND "one of these two things must be true".  Fixing it up now.


⚠ Turnos 24x7 has degraded performance

In 5c66064, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 5870 ms

Resolved: Turnos 24x7 performance has improved in d19ddcb.


Add check if file is already selected

Describe the bug
I noticed that in LocalSend, it is still possible to send the identical files multiple times. So, when I select a file, let's say file_a.jpg, and then select it again, the file is added to the queue again, resulting in the transmission of two files.
Is this the expected behavior of the LocalSend feature, or is it a bug @Tienisto ?
To reproduce

Go to tab Send
Click on File or Media
Select a file or image or video
And then add the same file with pick previously

Expected behavior
I feel that there should be validation within a session to prevent the reselection of files that have already been chosen, unless the sending session has changed.
Screenshots

Desktop (please complete the following information)

OS : MacOs
Version : MacOs Ventura 13.2.1

Smartphone (please complete the following information)

Device : Samsung A50s
OS : Android
Version  : Android 11

Additional context
I tried in MacOs and Android. But i think this is will happens to all platform

You are right, there is currently no check if the same file is selected again

You are right, there is currently no check if the same file is selected again

But should that require validation? If yes, maybe I can help with that.

Yes, a validation would be great

Yes, a validation would be great

ok, i will try on it. do you think for text and copy paste should be add the duplicate validation too?

i already create the PR for this enhancement, hope it will be merge. thx
@Tienisto


Add Fleet resources

Relevant issue: https://github.com/rancher/backup-restore-operator/issues/71

Accidentally closed this. Reopened as draft.

👍 I just migrated Rancher to another cluster, and Fleet decided to remove all managed resources from downstream clusters.
This is a very scary behavior.

Would it make sense to put the fleet things in its own section? I think it would keep the expressions from getting too big by having each main component (rancher, fleet...) separated. And future updates adding more components would be cleaner, add a new section and don't worry about existing.

@dramich had this thought while going through the PR. I think we should section it out. It's getting difficult to read.

@nickgerace @dramich  just wanted to make a note here that for the chart to pick up these changes and include them in the rancher-resource-set, the changes should be made here
This crd file in this repo was only being maintained during the initial development/testing.
It was also suggested (by Darren I think) to move the chart from rancher/charts repo to this repo, and then provide the chart url like its done for rancher-operator here

Thank you @mrajashree for the insight. That makes much more sense. We are moving the chart accordingly and I'll close this PR to avoid confusion. Thank you again and everyone for the initial reviews.


Stage

Pull Request Test Coverage Report for Build 2794

0 of 0   changed or added relevant lines in 0 files are covered.
396 unchanged lines in 3 files lost coverage.
Overall coverage increased (+0.1%) to 66.91%

Files with Coverage Reduction
New Missed Lines
%

dist/util/winston-logging.js
1
87.5%

dist/routes/v2/services/slpdb.js
9
75.42%

dist/routes/v2/slp.js
386
46.37%

Totals

Change from base Build 2772:
0.1%

Covered Lines:
2540

Relevant Lines:
3576

💛  - Coveralls


Question regarding Medformer and Encoder-only models

Hi Authors,
Congratulations on the great work. I would like to know how you modified the forecasting-based methods for classification.
Did you extract the Encoder-only part and use it for classification? (I think this is true based on Appendix C), please correct me if I am wrong.
Also, if you used the encoder-only parts for all transformer variants, could you share the Encoder-only model implementation or teach me how to extract them? I would like to test your approach + other models on some of the in-house clinical time series data.
Thank you

Hi, please refer to the Time-Series-Library repository for the implementation of classification tasks in other models. Our code is built upon their framework.
From our understanding, they primarily use the encoder-only variant for these methods with minor modifications when doing classification tasks. There is no need to explicitly extract this part; the encoder-only model will be automatically applied when running a classification task by passing the --task_name classification argument in the command line.
Check each model in the models folder. Different tasks, such as forecasting and classification, have different code implementations. We have also provided all the scripts for the classification task in the scripts/classification folder.

Thank you so much for the clarification.


Since its packaged now.. how do I edit...

Quick couple questions since it's packaged now.

How do I replace the connect/disconnected image. I changed the class but didn't seem to change the image.
I originally added text after the heart rate number... How can I add this now?
Since this is using websocket could I pull this line into say Reselim's beat-saber-overlay via html / css if I added it as a row in that overlay?
any way to support netbios / wins to connect by computer name? streaming on different networks I usually have to update the ip on the watch.. just an extra step.
whats the native resolution for the overlay / space wise.. is it absolute or is the div just adjusting to fit.

Love the app / overlay been super awesome! Thank you!

The image that shows when the watch is disconnected probably isn't configurable right now. You could change the size or hide it, but that's about it.
Add this to the Custom CSS in OBS .hr::after { content: "bpm" }
Probably not without writing code
You can use something like ZeroTier to set up a VLAN so you can use the same IP address everywhere
The resolution is not absolute. You can change the image/font sizes if you want it to be high quality at larger sizes.

Thanks for the quick response!
as for 3.. I'll see if I can just iframe it as the overlay is local.
Thanks again for all your work.. super awesome app.


Update overlap/translation endpoint (release/113)

Requirements

Filling out the template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion;
Review the contributing guidelines for this repository; remember in particular:

do not modify code without testing for regression
provide simple unit tests to test the changes
the PR must not fail unit testing
if you're adding/updating documentation of an endpoint, make sure you add/update the necessary parameters to the (template) configuration files in the ensembl-rest_private repo

Description
Update behavior of overlap/translation/:id endpoint.
Use case
A recent user query demonstrated that there is a difference in the description
Benefits
If applicable, describe the advantages the changes will have.
Possible Drawbacks
If applicable, describe any possible undesirable consequence of the changes.
Testing
note to submitters and reviewers: documentation-only changes may reflect
changes in other repos that can result in new or different output from
REST endpoints. In turn, these may require new tests or changes to existing tests.
Have you added/modified unit tests to test the changes?
2 subtests were added into the overlap.t test, as well as an update in info.t to handle the extra xref added.
If so, do the tests pass/fail?
Tests are succeeding.
Have you run the entire test suite and no regression was detected?
Yes.
Changelog
Are you changing the functionality of an endpoint? If so, please give a one line summary for the public facing changelog.
[overlap/translation/:id] Use protein feature description if no Interpro xref exists.

realised the build is failing for PR. Better check what's going on

This seems to be happening because the ensembl-vep repo hasn't branched for 113 yet


Fix UI Error/Success Box UI

Fix the UI (sizing/location) of the error/success message box.
Currently, scrolling is needed to see the full message, which is undesirable in a CLI focused product.

Potentially, this can be done with #45, as both issues are closely related.


Arabic text is written in reverse when used in Django

When I use the library as a standalone it works fine and all. But when I use it in Django, it saves weirdly.
The standalone one:

The Django one:

This is my code:
def _fill_form(self, application):
        import requests
        from fillpdf import fillpdfs
        from django.core.files import File
        from io import BytesIO
        from django.forms.models import model_to_dict

        ctx = model_to_dict(application)

        response = requests.get("http://localhost:8000/static/document/form.pdf")
        if not response.ok:
            log.error("couldn't get form")

        in_stream = BytesIO(response.content)
        out_stream = BytesIO()

        fillpdfs.write_fillable_pdf(
            in_stream,
            out_stream,
            ctx,
            flatten=True
        )

        application.arabic_document.save("Some fillpdf file.pdf", File(out_stream))

the issue was caused by the browser I was opening the pdf from.
If you install a plugin that renders Arabic text it should work fine.


test: path_to_regex_no_options.cpp:627 fails

Had to comment this one out.

"Paths with two parameters (or more of the first parameter) where the first contains one lower case letter and the second contains one or more integers should match"

std::regex r = path_to_regex("/:test([a-z])+/:id(\\d+)", keys);
EXPECT_NOT(std::regex_match("/M/1234", r));

There also seems to be some other tests that fails on jenkins(ubuntu), but not locally for me on mac:
path_to_regex_no_options.cpp:734-736:
EXPECT(std::regex_match("/B", r));        // FAILS WHEN NOT CASE SENSITIVE (icase set)
EXPECT(std::regex_match("/Z", r));        // FAILS WHEN NOT CASE SENSITIVE (icase set)
EXPECT(std::regex_match("/Matilda", r));  // FAILS WHEN NOT CASE SENSITIVE (icase set)

# local
/Users/andreas/dev/IncludeOS/test/util/unit/path_to_regex_no_options.cpp:734: passed: Creating path_to_regex with no options: std::regex_match("/B", r) for true

# jenkins
/home/ubuntu/workspace/pull_request_misc/test/util/unit/path_to_regex_no_options.cpp:734: failed: Creating path_to_regex with no options: std::regex_match("/B", r) for false
1 out of 107 selected tests failed.

My compiler
$ /usr/bin/c++ -v
Apple LLVM version 8.0.0 (clang-8<IP_ADDRESS>)
Target: x86_64-apple-darwin15.6.0
Thread model: posix
InstalledDir: /Library/Developer/CommandLineTools/usr/bin

One more, line 895 (890fed1).
It looks like case insensitive /[a-z] matches /A but not /B etc. Maybe @AnnikaH can explain the issue better.

Yes this is an issue we discussed some months ago and couldn't quite figure out - it's an issue with std::regex_constants::icase or our use of it - it doesn't do what it's supposed to (ignore case) in the code: https://github.com/includeos/path_to_regex/blob/master/path_to_regex.cpp#L203

Related? https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71500

Yes, looks very much like it. Nice find!


[TA] (423) A user can view a list of properties (part 2)

Changes in this PR

This brings the premises listing page in line with the wireframe designs

Screenshots of UI changes
Before

After

 A user can view a list of properties

 Update error handling in Apply forms


Make Turbo Frames' IntersectionObserver configurable

This PR adds configuration options akin to those controlling the autoscroll behavior to AppearanceObserver.
Note that I allowed only one threshold value (which could be many, see https://developer.mozilla.org/en-US/docs/Web/API/Intersection_Observer_API#thresholds, but that doesn't make sense here).
Note further that all those options are optional, and just fall back to the default if omitted.

I love it this is really useful to start lazy loading before the element appears.
Little naming comments:

most of the data attributes in Turbo are prefixed by turbo. Shouldn't we have here data-turbo-intersection-root-selector, data-turbo-intersection-root-margin etc

Thanks! FWIW I took inspiration from the autoscroll attributes: https://turbo.hotwired.dev/reference/frames#html-attributes
It's data-autoscroll-... there.

heads up: I've rewritten this in pure JS, will add a test if you think it's worth it ✌

Hello @afcapel
Thanks for the beta of Turbo 8. I was wondering if this PR could be part of the final V8
As per my comment https://github.com/hotwired/turbo/pull/923#issuecomment-1542776338 we could add a turbo prefix. Julian went that way to be consistent with data-autoscroll that was introduced before.
Given that we have a major release this could be a good time to rename data-autoscroll in data-turbo-autoscroll and introduce those new settings for the intersectionObserver

data-turbo-intersection-root-selector
data-turbo-intersection-root-margin
data-turbo-intersection-threshold

I have seen quite a few apps where a custom stimulus controller was created just for that. I think it would be a good DX improvement

I think those options are essential to mimic the native lazy loading from browsers. They load images and iframes before they intersect with the viewport so that it's likely to be ready when lazy loaded elements appear on screen.

Ping @dhh since you seem to be focused on Hotwire at the moment.


Asset Page improvements

[x] Asset page should include asset flags (auth_required, revocable, immutable). You can follow  issuing account link to see this but it would be more comfortable if it's also in the asset page.
[x] Asset holders currently shows the number of total trustlines. Ideally, this should be the number of trustlines that have a balance>0.
[x] To complete the picture, it would be great to also have the total number of trustlines.
[x] In the case of a restricted asset, indicate how many of these are authorized.

Thanks @orbitlens. Yeah, I can see why the authorization status might need some more work. I removed that from the 3rd item and added it to a 4th. Are 1-3 achievable with your current schema?

Sure. Going to implement and release this functionality in two weeks.

Thinking out loud: AUTH_REQUIRED=false and AUTH_REVOCABLE=false are misleading in the (common) case where  AUTH_IMMUTABLE=false, since the issuer can change at anytime.
I wonder if there's a way to indicate that without being not too intrusive. Maybe a red flag that on hover explains that?

AUTH_REQUIRED and AUTH_REVOCABLE flags added to the asset page in v0.8.23.
There is a problem with AUTH_IMMUTABLE, as Horizon doesn't return the value of this flag.
The remaining three features require ingestion engine refactor.

@orbitlens auth_immutable is now in horizon (master)

Implemented in v0.10.7


how to pass a array(no numberic index) to a interface by POST?

talk is cheap,show you the code ~~
$paramPost = array('xxx','yyy','zzz');
$client = new Client('http://myhost/myinterface');
$client->setMethod('POST');
$client->setParameterPost(array(
        'image_name_array' => $paramPost,
));
$client->send();
$responseBody = $client->getResponse()->getBody();

when i print the request raw content,i find a question:
image_name_array%5B0%5D=xxx&image_name_array%5B1%5D=yyy&image_name_array%5B2%5D=zzz

the pretty format is:
image_name_array[0]=xxx&image_name_array[1]=yyy&image_name_array[2]=zzz

but the server expected the follow data: 'image_name_array[]=xxx&image_name_array[]=yyy&image_name_array[]=zzz' how to handle the question?

additional remarks: zf version 2.2.4 ,PHP 5.4.7 (cli) ,use Zend\Http\Client;

but the server expected the follow data: 'image_name_array[]=xxx&image_name_array[]=yyy&image_name_array[]=zzz'

You know there is no difference at all and both notations represent exactly the same array and both arrays have numeric indexes?
image_name_array[]=xxx&image_name_array[]=yyy&image_name_array[]=zzz:
image_name_array(
    0 => 'xxx',
    1 => 'yyy',
    2 => 'zzz',
)

image_name_array[0]=xxx&image_name_array[1]=yyy&image_name_array[2]=zzz:
image_name_array(
    0 => 'xxx',
    1 => 'yyy',
    2 => 'zzz',
)

Perhaps you are not telling the complete story, but at this moment it looks like basic array knowledge to me and not a ZF2 bug :wink:

Closing as a support request and not a bug.

nope, perhaps my description is't clear, i use Http\Client to post data to a server imitate form submit, but the request is not same as form action. the following js code segment working well:
var info=[];
info.push("xxx");
info.push("yyy");
info.push("zzz");
$.ajax({
        type: "POST",
        url: "myinterface",
        data:{ "image_name_array": info },
       success: function(result){
           console.log(result);
       }
});

i think our zf2 should support this post way, though i know the programming way may be cumbersome but practical !

nope, perhaps my description is't clear, i use Http\Client to post data to a server imitate form submit, but the request is not same as form action.

The javascript code generates the same array again:
image_name_array(
    0 => 'xxx',
    1 => 'yyy',
    2 => 'zzz',
)

i think our zf2 should support this post way, though i know the programming way may be cumbersome but practical !

Perhaps you can clarify the practical use for [] instead of [0]? If the use case is clear it might be easier to understand your point.

@Martin-P yes, it may be common in many cases or actual applications.

@Martin-P yes, it may be common in many cases or actual applications.

In that case it shouldn't be a problem to clarify your point. So please provide a use case, because it is still not clear why such a feature would be needed.


Running the demo app is blank

Description

Steps To Reproduce

Code sample

macOS 12.6.8  version< 13.0
macos_ui: ^2.0.9

Expected behavior

Screenshots

Logs

Logs

flutter: Warning: Falling back on slow accent color resolution. It’s possible that the accent colors have changed in a recent version of macOS, thus invalidating macos_ui’s accent colors, which were captured on macOS Ventura. If you see this message, please notify a maintainer of the macos_ui package.

What’s your macOS version?

macOS 12.6.8
macos_ui: ^2.0.9

If you’re targeting older versions of macOS, you need to follow the instructions found here: https://github.com/macosui/macos_ui?tab=readme-ov-file#older-macos-versions


GearVR Controller Support?

I know this is not specifically mentioned as supported, but I tried out the GearVR Controller with a GearVR and a Samsung S6.
Trying the demo page on this project, I got the following results:

Using the new(ish) built in Internet browser on Oculus home - the controller is visible and tracks/points well, but none of the buttons work - I can't interact with the UI elemental via clicking.
Using Samsung Internet with WebVR enabled, I get a black screen when loading the page.

Let me know if there is any useful debugging info I can provide for this controller.
Cheers! - James

I would love to support the GearVR controller! This may be a bit of a pain, but can you download and open Chrome on your phone, plug your phone into your computer, then connect your phone’s open Chrome tab to your computer’s like so: https://developers.google.com/web/tools/chrome-devtools/remote-debugging/
Once you do that you can interact with the JavaScript console. Load up the demo site and then in your console paste and enter this to enable full verbose mode: THREE.VRController.verbosity = 1
Now when you interact with the trackpad or buttons you should get a lot of output in the console. What you need to do is interact with each piece of the controller and see what button index # is receiving that interaction. My guess is the thumpad has a touch, a press, and axes and is index #0. See if that’s what’s showing up in the console and also check out the other buttons. We can use that info to add in explicit support :wink:
Try to be thorough and keep notes. It’s the subtleties that make all the difference! (Does this button have a press state? A touch state? An analog value—and if so at what point does a press begin and is that the same threshold for ending a press? Does +1 on the Y axis mean top or bottom? And so on.) Here’s what I did as an example:
https://github.com/stewdio/THREE.VRController/blob/master/VRController.js#L451

I also just made a super tiny update that should allow your GearVR primary button to work with the example’s "primary press began" event listener even though we haven’t explicitly mapped the buttons yet:
https://github.com/stewdio/THREE.VRController/blob/master/VRController.js#L95
If you can, give that a go and tell me if it works. It makes a pretty big assumption that the primary button is going to be at index === 0 in the buttons array but that’s what I’m seeing with the Vive, Oculus, and Daydream ... so I’m hoping that’s the norm!

I haven't done the full logging yet, but I can confirm I can now:

Open https://stewdio.github.io/THREE.VRController/ on the built in browser
Click enter VR
The main button (clicking the trackpad) works. I can use it to drag the UI and interact with the elements.

The button does not work outside VR mode but this is probably expected.

Wow that is awesome! (Thank you for testing it out!)
When you say the button does not work outside VR mode, are you able to see the controller at all before entering VR? If you don’t see it at all then I think what might be happening is that prior to calling vrDisplay.requestPresent() the GearVR does not engage its VR internals and connect to the controller. (But this is just a guess! It would be similar to how Daydream seems to operate.)

I can see the controller outside the VR mode, but what I am seeing is the GearVR interface and their rendered representation of the controller. The browser is just a floating window inside the GearVR interface/UI.
I can point to items in the scene, they highlight the item I am pointing at them (e.g. the dat.guiVR floating menu) but the click is ineffective on the scene - I can't click the elements, drag the UI, etc - when outside VR mode.
Within VR mode everything works as expected. I guess the above is not that surprising but just an observation.

I'll check this out presently.

I had to fork this project to start working on GearVR support. it should work like the DayDream one. So this makes the thumbpad the primary not the trigger.
'Gear VR Controller': {

		style: 'gearvr',

		//  THUMBPAD
		//  Both a 2D trackpad and a button with both touch and press.
		//  The Y-axis is “Regular”.
		//
		//              Top: Y = -1
		//                   ↑
		//    Left: X = -1 ←─┼─→ Right: X = +1
		//                   ↓
		//           Bottom: Y = +1

		axes: [{ name: 'thumbpad', indexes: [ 0, 1 ]}],
		buttons: [ 'thumbpad' ],
		primary: 'thumbpad'
	},

@danrossi are you still planning to do a PR to add this support? I added Oculus Go yesterday and can do this as well if you no longer want to do so.

@paulmasson I am using Oculus Go with this controller:
https://developer.oculus.com/documentation/unity/latest/concepts/unity-ovrinput/#unity-ovrinput-go-controller
However, I cannot find which event for touchpad event (touch position or swipe action?)

@whatisor if you have the Go connected to your computer and have set THREE.VRController.verbosity = 1 in the Chrome JavaScript console, then you'll see all the events. You're probably looking for thumbpad axes changed.

@paulmasson Thank you, I found it as "axis changed" from aframe sample.


Update load commands to update existing data instead of flushing data

Currently the loadtopics and loadresources commands require the database to delete (flush) all entries and then recreate them. It should instead update existing entries, create new ones, and delete old ones. This is required when users want to create a permanent connection to a resource, topic, etc.

Moving to future milestone as not required for release.

@hayleyavw Do we wish to close this issue? As we are not planning on enabling user accounts for CS Unplugged.

Might as well stay open for external contributors, it's not on our priority list but it should be done at some point shouldn't it? Our current solution of deleting it all then creating everything all over again is not particularly elegant.

At the current point of this project, and with the planned features, it never needs to be implemented. Therefore I believe it should be closed, especially as it's a large amount of work for someone to do that wouldn't be utilised.


vendor: google.golang.org/grpc v1.58.3

Upgrade google.golang.org/grpc to v1.58.3 in preparation for upgrading OTel, which has a dependency on the latest version.
See also:

containerd/containerd#9281

/ok-to-test

FYI I don't think the win2019 failures were related to these changes, it looks like something went horribly awry on the CI runner.

FYI I don't think the win2019 failures were related to these changes, it looks like something went horribly awry on the CI runner.

Can you rebase on the tip of main?  We've disabled this CI job for now.

Latest CI failure is due to throttling 😅
import_test.go:81: failed to copy: httpReadSeeker: failed open: unexpected status code https://mcr.microsoft.com/v2/windows/nanoserver/blobs/sha256:5178da1ca6af8a14a235f58eb31955ca5f4c72f950d35f9bb67a8bd20232d840: 429 Too Many Requests

If someone can kick that job, hopefully this is all set!


add new safe rollback to NBE updates

at the moment, both NBE update and rollback are done live, meaning that the NBE is not uninstalled before the new version comes it.
This can sometime originate conflicts and undesired behaviors that cause both the update and rollback to fail.
To provide extra robustness to the updater, we should implement a fallback to the rollback action, such that in case all fails, the whole NBE is uninstalled (volume included), and a new one (with the new version) is installed.
Because this removes the shared volume, the following needs to be implemented:

[x] add rollback fallback to updater script
[x] add new env vars to agent such that it can take API key and secret via the env, to resume its operation without having to activate the NB again
[x] adjust the remaining NBE components that rely on other files in the shared volume (like the security who waits for the .nuvla-configuration file). Such files need to either be recreated of removed as a dependency
[x] the job-engine-lite needs to pass these env vars to the installer container

finished in https://github.com/nuvlabox/deployment/commit/f4a8248d09e4000cce45d4090cc0c8548962755a


Add KtUnits

https://github.com/sargunster/KtUnits
I'm not sure if this is a large enough library to add, but I think it can be useful.

Nice contribution! Thank you!


Update 2024-07-29-draft-consitution-converted.md

Closing this because it was a test for documentation purposes.


Fix KMT remote launch tasks for macOS

What does this PR do?
This fixes some issues when launching KMT remote machines from macOS. It also improves ssh key finding for cases where the name of the SSH key does not coincide with the file name (such as when using the laptop setup script).
The fixes for macOS are the following:

Build compiler image with an explicit platform (some packages are not available in the ARM builds of Debian) and Go versions.
Adapt env-setup.sh script to the Homebrew package manager
Move libvirt-related commands to only be done in the non-lite mode

Motivation
Additional Notes
Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided. Except if the qa/skip-qa label, with required either qa/done or qa/no-code-change labels, are applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

/merge

/merge


Updating the hint text on the "Date Input" examples to match the "Dates" pattern guidance

The "Dates" pattern gives specific guidance about giving user example dates: https://design-system.service.gov.uk/patterns/dates/#how-to-write-dates
However I noticed that the examples on the 'Date Input' component don't follow that guidance, so this PR updates the examples to match the guidance from the pattern.

@ashward Thanks for flagging this issue and suggesting a fix! Looks good to me, but I'm just going to get team reviews as per our usual process.


[5.3] Eloquent self::update([...]) is updating attributes not given as parameter

Laravel Version: 5.3.28
PHP Version: 7
Database Driver & Version: sqlite

Description:
Within an Eloquent model Foo with 3 attributes foo ; bar ; baz, calling a method doSomething() which updates the model foo ; bar attributes will also update the baz attribute IF an accessor is declared for baz.
Steps To Reproduce:
class Foo extends Model
{
    protected $fillable = [
        'foo',
        'bar',
        'baz',
    ];

    protected function doSomething()
    {
        $this->update([
            'foo' => true,
            'bar' => true,
            // note baz is not set to be updated
        ]);
    }

    public function getBazAttribute($value)
    {
        return 'XXX';
    }

...

So calling $object->doSomething() will update foo ; bar as expected but also baz (not expected) only if the accessor is declared.

Caused by a third party package...
Sorry for that.


Support of sleeping clients

In the MQTT-SN protocol specification (Version 1.2), Section 6.14, the support of sleeping clients is described. This is currently not implemented on emq-sn (emqttd-2.2-beta.2). When could I expect this feature?

Developing this feature now. Please be patient since too many TODOs in my list.

@jpmzometa   sleeping device will be added into next official release.

Hi. I've been playing around with the sleeping client implementation. It does not work as expected. A client doing a CONNECT to the gateway, SUBSCRIBE to a topic, and DISCONNECT with a duration (i.e. going to sleep) work correctly. However, every time the client sends a PINGREQ (client awake) message, it gets from the gateway all the previous messages that have been published (in the topics subscribed by the client) since the client went to sleep via the DISCONNECT with duration.
Put in another words, if there are buffered messages for the client in the gateway (that is, for the topics for which the client has subscribed), after a PINGREQ from the client (i.e. client is awake), the gateway should send all buffered messages and at the end a PINGRESP, and the gateway should clear its buffer (assuming all messages satisfied the QoS). If the client sends a new PINGREQ, the gateway should just reply with a PINGRESP if there are no new messages. At the moment, it appears the buffered messages are not cleared in the gateway (i.e. each new message is added to the buffer, and all messages in the buffer are sent to the client every time a PINGREQ arrives from the client).

@jpmzometa  This bug has been fixed in v2.3-beta1.


Configure repository to prevent merges without ci passing and peer review

DoD

[ ] Can't push directly to the master/main branch.
[ ] Setup team access to the repository.
[ ] Add branch protection rules to enforce that CI is passing and code has at least 1 review before allowing for merge

My suggested setup for Will to review

Requires https://github.com/elifesciences/enhanced-preprints/pull/36 and https://github.com/elifesciences/enhanced-preprints/pull/35 to be merged first.


Unable to transfer data from parent controller to child controller

Hello,
I am using XLPagerTabStrip library latest version with xcode 9.2. I am making an API call on viewdidload() method of parent controller. I am having two child controller i.e. child1 and child2. I am not able transfer the data that I am getting from an API call to child view controller. The problem is that the network call method in the parent view controller is executed after the viewdidload() method of child controller. So If I am passing an array that contain data from network call in the parent controller to child controller. I am getting array count 0 in viewdidload() and viewwillappear() method of both child controller.

[https://stackoverflow.com/questions/51532550/passing-data-between-children-using-xlpagertabstrip/51533257](Use this link)
Modify the above code a little bit and you will be able to solve your problem. Just make child 1 and child 2 view controller as a global variable in your parent view controller then in your API response just access child 1 property and child 2 property and set the corresponding value.
Code Sample:-
class ParentViewController:ButtonBarPagerTabStripViewController{
fileprivate var child1:CreditCardViewController!
fileprivate var child2:PaypalViewController!
override func viewControllers(for pagerTabStripController: PagerTabStripViewController) -> [UIViewController] {
child1 = self.getNextViewController(storyboardName: Constants.StoryboardNames.commonViews, destinationVCIdentifier: "VC1") as! CreditCardViewController
child1.selectedPackage = self.selectedPackage
child1.billingObj = self.billingInfo
    child2 = self.getNextViewController(storyboardName: Constants.StoryboardNames.commonViews, destinationVCIdentifier: "VC2") as! PaypalViewController
    child2.selectedPackage = self.selectedPackage
    child2.billingObj = self.billingInfo

    return [child1, child2]
}

func APICall(){
self.child1.apiData = API_Data //Setting Value
self.child2.secondApiData = API_Data //Setting Value
}
}
class CreditCardViewController{
var apiData = ""
}
class PaypalViewController{
var secondApiData = ""
}


check the motion reduction

it seems to be selecting very few components

likely culprit is that motion artifacts are not zero-meaned. So the average will explain a massive amount of variance.


Is it possible get the file?

I'm getting the string without problem, but I also need get the file and those logged attributes:
'Response = ', { height: 4160,
width: 3120,
type: 'image/jpeg',
fileName: 'image-f3305acd-8ae6-4361-b8d4-8c1074ea9fab.jpg',
fileSize: 4065585,
path: '/storage/emulated/0/Pictures/image-f3305acd-8ae6-4361-b8d4-8c1074ea9fab.jpg',
data: '/9j/...'

Any solution on this?

Okay, I'll try that.
Closing for now, thank you.


[Bug]: multi-line logs are not removed

Bug Description
I use prettier to format my code so when a log line is too long it will spread it across multiple lines.
Then when I try to remove the lines with removeLogs it only removes the first line of the log.
Example:
console.log("[🪲 dlvhdr] beep 🔵 will return", {
  data: [
    {
      id: `cluster-${services[0].k8sClusterName}`,
      name: services[0].k8sClusterName ?? "",
      kind: WorkspaceKind.Cluster,
      value: {
        clusterName: services[0].k8sClusterName ?? "",
      },
    },
  ],
});

Relevant Screenshot
No response
To Reproduce

Create a long log
Run some formatter or manually spread it to multiple lines
Observe removeLogs only removes the first line.

neovim version
0.10.0

Make sure you have done the following

[X] I have updated to the latest version of the plugin.

Yeah, the plugin only removes the lines containing the marker emoji. Trying to detect anything more is not only quite convoluted, but also very error-prone (requiring special considerations for every single language, etc.)
The best approach for dealing with this is to add a prettier-ignore to the log statement, as described in the readme: https://github.com/chrisgrieser/nvim-chainsaw?tab=readme-ov-file#have-your-formatter-ignore-the-log-statements

Oh nice, will it then remove the prettier comment as well?

yes, the %s at the end of the /* prettier-ignore */ // %s adds the emoji marker, ensuring that the line will also be removed by removeLog


Inconsistent ConfigParameters Behavior: Missing samp_freq Field

Hi, sorry to bother you again with this issue.
I recently tried to help someone install NoisePy, but the same versioning issue arose again. When running the main tutorial, I encountered the following error:

ValueError: "ConfigParameters" object has no field "samp_freq"

This error occurs when executing the cell for setting configuration parameters for monitoring:

config = ConfigParameters()

It seems the installed version of the code does not include the samp_freq parameter. This is strange because I have another installation where this error does not occur, and the tutorial works as expected.
Investigation So Far:

I inspected monitoring_utils.py, but samp_freq is not explicitly declared in the ConfigParameters_monitoring class.

I also checked /noisepy/seis/io/datatypes.py in the local installation, but I couldn't locate samp_freq there either. Additionally, I noticed that the io directory under seis does not appear to exist in the GitHub repository, making it difficult to compare the installed files with the repository version.

Could you please help clarify:

Where the samp_freq parameter for the ConfigParameters class is declared or inherited from?
Why this inconsistency might occur between installations when using the same NoisePy tutorial?

Any guidance would be greatly appreciated. Thank you so much for your help, and I apologize for the inconvenience!

Hi @SeismoFelix,
Sorry for the issue you are facing. We are still going through some major change to the package. It's mostly done, but there might still be some changes to the parameters or variables. In short, the problem you are facing comes from a recent change on IO that replace samp_freq into sampling_rate to match the obspy convention.

I recently tried to help someone install NoisePy, but the same versioning issue arose again. When running the main tutorial, I encountered the following error:

ValueError: "ConfigParameters" object has no field "samp_freq"

This error occurs when executing the cell for setting configuration parameters for monitoring:

config = ConfigParameters()

It seems the installed version of the code does not include the samp_freq parameter. This is strange because I have another installation where this error does not occur, and the tutorial works as expected.

We apologize that the tutorial on HPS-book is not updated since the change. It was trying to set the parameter samp_freq, which does not exist anymore. We have updated the notebook on HPS-book. We also recommend using this documentation, which is always synced with our packages.

I inspected monitoring_utils.py, but samp_freq is not explicitly declared in the ConfigParameters_monitoring class.

The ConfigParameters_monitoring is specifically for monitoring. It makes sense there isn't either samp_freq or sampling_rate field.

I also checked /noisepy/seis/io/datatypes.py in the local installation, but I couldn't locate samp_freq there either. Additionally, I noticed that the io directory under seis does not appear to exist in the GitHub repository, making it difficult to compare the installed files with the repository version.

We put IO as a separate repository for easier maintenance. You can find the repo here, and the place ConfigParameters is defined here.
Hopefully this helps solve your problem.

Thanks a lot @niyiyu
I was able to run the tutorial with the newest version of noisepy. I just made a minor modifications to the paremeters that still belonged to the deprecated version:

config.net_list = ["CI"] -> config.networks = ["CI"]

I also updated some lines that still call the old parameters:

raw_store = SCEDCS3DataStore(S3_DATA, catalog, channel_filter(config.net_list, config.stations, ["BH?", "HH?"]),timerange, storage_options=S3_STORAGE_OPTIONS) # Store for reading raw data from S3 bucket

->

raw_store = SCEDCS3DataStore(S3_DATA, catalog, channel_filter(config.networks, config.stations, ["BH?", "HH?"]), timerange, storage_options=S3_STORAGE_OPTIONS)

and also in defining the dt before calculating the reference curve

dt=1/config.samp_freq -> dt=1/config.sampling_rate

Now I am trying to conciliate the legacy parameters with the newest ones for controling the number of dv/v measurements.
In this new version, as is, I am getting 4044 measurements for a month of data:

Which seems to be excesive compared to the previous version of the tutorial, where there were 40 samples:

I changed config.substack_windows = 2 (I assume is the new parameter in exchange of config.substack_len) and I got 30 windows, but the reference curve output looks significatly different from the old version I run a couple of weeks ago. I am hesitant that 10 samples changed the outcome too much.
New version:

Old version:

After the modifications I suggest, with the today noisepy version, the tutorial is functional. Although the outcome is different from what I got some weeks ago. But I believe this is a matter of being familiar with the new parameters. So, feel free to close this issue if you consider  it convenient.
I appreciate a lot how responsive have you been. I am still climbing the learning curve and I will keep this issue update it with the aim to help my colleagues that also are learning.
Cheers

Good news (to me :) )
I was able to to run the tutorial monitoring notebook getting the same results as the obtained ones in the previous version of noisepy. It took me a while to realize than config.substack_len is defined internally now as
config.substack_len = self.substack_windows * self.cc_len. So I had to define config.substack_windows = 24 and cc_len = 600  as it was. Then I could get the 6  segments/windows as in the previous version and reproducing the same curves.
 But I will try to move forward with this version.
Thanks a lot for your help!

@SeismoFelix Thank for more corrections! I just made another updates to the HPS book to sync with the documentation.
Cheers,
Yiyu


add scale flag for compose up

Signed-off-by: ye.sijun<EMAIL_ADDRESS>Add scale flag for compose up.
./nerdctl compose up -d --scale wordpress=3
INFO[0000] Creating network nerdctl_default
INFO[0000] Ensuring image mariadb:10.5
INFO[0001] Ensuring image wordpress:5.7
INFO[0001] Creating container nerdctl_wordpress_3
INFO[0001] Creating container nerdctl_wordpress_2
INFO[0001] Creating container nerdctl_wordpress_1
INFO[0001] Creating container nerdctl_db_1

Thanks, could you also update README.md?
Adding tests would be also appreciated.


Datetime picker glitch in Safari

I'm Submitting a ...

[x] Bug report
[ ] Feature request
[ ] Support request

Steps to Reproduce

Install/run react-datetime demo
In Safari, click on Datetime input under 'Customization props' heading.

Actual Results

If clicking in whitespace area of input, picker appears as intended. If clicking on text inside input, picker appears but then immediately hides.
This issue appears to be specific to Safari. Works as intended in Firefox. Unknown if specific to Safari version noted below.
Other Information (e.g. stacktraces, related issues, suggestions how to fix)

Safari version 11.0
MacOS 10.12.6
react-datetime 2.10.2

I can confirm this in Safari on MacOS 10.12.6.  However, the problem only happens in the Demo app and not the CodePen example.

same thing on Safari 11.0 (126<IP_ADDRESS>.7)

Same issue, Safari 11.0 on macOS 10.13

So this glitch is fixed in alpha.1 but it appears to be back in beta.2. Do you know what caused this issue? I can't really get anymore from the commits, but I'd like to help if you give me a pointer.


Add UI prototype

Created a new folder for the app and a subfolder for the UI. Added PDF of the UI prototype.
The link below will also allow access to view the prototype.
https://www.figma.com/proto/Lx9D2WlgJAJgMgJIuNUlqa/UI-prototype-1?node-id=10%3A25&scaling=scale-down
Resolves #11

Looks good!


Be more resilient to storage errors

While S3 was down, we failed a bunch of requests because we try to read the tile from S3 as a short-cut before querying it from the database. Unfortunately, there are many things that can go wrong with fetching a tile over the network.
I've grabbed a selection of errors from the logs at the bottom of this issue.
The request could have been served by querying the database, but these exceptions thrown mean we return 500 errors instead. Throwing exceptions seems appropriate for tilequeue, where we want to make sure that the job isn't marked complete unless the tile is stored safely. For tileserver, where there's a client waiting on the other end of a connection, perhaps we want to catch and log the exceptions and then continue as if the tile were missing?
After the tile has been rendered, we store it. It looks like if that raises an exception then we just log it and move on rather than stopping, since it happens on a background thread.
Traceback (most recent call last):
   File "/usr/local/lib/python2.7/dist-packages/tileserver/__init__.py", line 204, in __call__    response = self.handle_request(request)
   File "/usr/local/lib/python2.7/dist-packages/tileserver/__init__.py", line 249, in handle_request    tile_data = self.reformat_from_stored_json(request_data, layer_data)
   File "/usr/local/lib/python2.7/dist-packages/tileserver/__init__.py", line 331, in reformat_from_stored_json    tile_data = self.read_tile(coord)
   File "/usr/local/lib/python2.7/dist-packages/tileserver/__init__.py", line 389, in read_tile    raw_data = self.store.read_tile(coord, fmt, 'all')
   File "/usr/local/lib/python2.7/dist-packages/tilequeue/store.py", line 60, in read_tile    key = self.bucket.get_key(key_name)
   File "/usr/local/lib/python2.7/dist-packages/boto/s3/bucket.py", line 192, in get_key    key, resp = self._get_key_internal(key_name, headers, query_args_l)
   File "/usr/local/lib/python2.7/dist-packages/boto/s3/bucket.py", line 199, in _get_key_internal    query_args=query_args)
   File "/usr/local/lib/python2.7/dist-packages/boto/s3/connection.py", line 664, in make_request    retry_handler=retry_handler
   File "/usr/local/lib/python2.7/dist-packages/boto/connection.py", line 1068, in make_request    retry_handler=retry_handler)
   File "/usr/local/lib/python2.7/dist-packages/boto/connection.py", line 942, in _mexe    request.body, request.headers)
   File "/usr/lib/python2.7/httplib.py", line 1017, in request    self._send_request(method, url, body, headers)
   File "/usr/lib/python2.7/httplib.py", line 1051, in _send_request    self.endheaders(body)
   File "/usr/lib/python2.7/httplib.py", line 1013, in endheaders    self._send_output(message_body)
   File "/usr/lib/python2.7/httplib.py", line 864, in _send_output    self.send(msg)
   File "/usr/lib/python2.7/httplib.py", line 826, in send    self.connect()
   File "/usr/lib/python2.7/httplib.py", line 1216, in connect    self.timeout, self.source_address)
   File "/usr/lib/python2.7/socket.py", line 571, in create_connection    raise err
 timeout: timed out

Traceback (most recent call last):
   File "/usr/local/lib/python2.7/dist-packages/tileserver/__init__.py", line 204, in __call__    response = self.handle_request(request)
   File "/usr/local/lib/python2.7/dist-packages/tileserver/__init__.py", line 249, in handle_request    tile_data = self.reformat_from_stored_json(request_data, layer_data)
   File "/usr/local/lib/python2.7/dist-packages/tileserver/__init__.py", line 331, in reformat_from_stored_json    tile_data = self.read_tile(coord)
   File "/usr/local/lib/python2.7/dist-packages/tileserver/__init__.py", line 389, in read_tile    raw_data = self.store.read_tile(coord, fmt, 'all')
   File "/usr/local/lib/python2.7/dist-packages/tilequeue/store.py", line 60, in read_tile    key = self.bucket.get_key(key_name)
   File "/usr/local/lib/python2.7/dist-packages/boto/s3/bucket.py", line 192, in get_key    key, resp = self._get_key_internal(key_name, headers, query_args_l)
   File "/usr/local/lib/python2.7/dist-packages/boto/s3/bucket.py", line 199, in _get_key_internal    query_args=query_args)
   File "/usr/local/lib/python2.7/dist-packages/boto/s3/connection.py", line 664, in make_request    retry_handler=retry_handler
   File "/usr/local/lib/python2.7/dist-packages/boto/connection.py", line 1068, in make_request    retry_handler=retry_handler)
   File "/usr/local/lib/python2.7/dist-packages/boto/connection.py", line 1025, in _mexe    raise BotoServerError(response.status, response.reason, body)
 BotoServerError: BotoServerError: 500 Internal Server Error

Traceback (most recent call last):
   File "/usr/local/lib/python2.7/dist-packages/tileserver/__init__.py", line 204, in __call__    response = self.handle_request(request)
   File "/usr/local/lib/python2.7/dist-packages/tileserver/__init__.py", line 249, in handle_request    tile_data = self.reformat_from_stored_json(request_data, layer_data)
   File "/usr/local/lib/python2.7/dist-packages/tileserver/__init__.py", line 331, in reformat_from_stored_json    tile_data = self.read_tile(coord)
   File "/usr/local/lib/python2.7/dist-packages/tileserver/__init__.py", line 389, in read_tile    raw_data = self.store.read_tile(coord, fmt, 'all')
   File "/usr/local/lib/python2.7/dist-packages/tilequeue/store.py", line 60, in read_tile    key = self.bucket.get_key(key_name)
   File "/usr/local/lib/python2.7/dist-packages/boto/s3/bucket.py", line 192, in get_key    key, resp = self._get_key_internal(key_name, headers, query_args_l)
   File "/usr/local/lib/python2.7/dist-packages/boto/s3/bucket.py", line 199, in _get_key_internal    query_args=query_args)
   File "/usr/local/lib/python2.7/dist-packages/boto/s3/connection.py", line 664, in make_request    retry_handler=retry_handler
   File "/usr/local/lib/python2.7/dist-packages/boto/connection.py", line 1068, in make_request    retry_handler=retry_handler)
   File "/usr/local/lib/python2.7/dist-packages/boto/connection.py", line 1025, in _mexe    raise BotoServerError(response.status, response.reason, body)
 BotoServerError: BotoServerError: 503 Service Unavailable

@iandees Can this be closed?

Yep!


Run codecov on Linux only

Codecov Report

Merging #202 into master will decrease coverage by 0.03%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #202      +/-   ##
============================================
- Coverage     31.77%   31.73%   -0.04%
  Complexity      355      355
============================================
  Files            46       46
  Lines          2990     2990
  Branches        379      379
============================================
- Hits            950      949       -1
  Misses         1921     1921
- Partials        119      120       +1

Impacted Files
Coverage Δ
Complexity Δ

.../java/com/redhat/red/build/finder/BuildFinder.java
16.66% <0%> (-0.13%)
35% <0%> (ø)

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 74a96f0...425651f. Read the comment docs.


status 127

Hello,
Once I ran the ramazon function, I am getting:
WELL DONE!

I @stefan-jovanovski, ramazon currently does not work on Windows environment ( look at issue #5 ). As stated there feel free to develop it :)


Fix/14578 no internet prevents clicking on srs

Description
No internet connections prevents clicking on Self test submission
Link to Jira
https://jira-ibs.wbs.net.sap/browse/EXPOSUREAPP-14578
Screenshots
https://user-images.githubusercontent.com/72390277/214326263-249aa7be-f80d-4239-b4e2-1dd7447175d0.MP4

So IIUC when you are not connected to the internet and click on the tile nothing happens.  Like nothing at all? I don't think that's a good idea! Some sort of error should be shown to the user, otherwise this could lead to very confused users.

@Ein-Tim This is under certain circumstances the situation with 3.0 and this PR will fix the issue.

@thomasaugsten So there will actually be some sort of UI information presented to the user why this doesn't work?

Depends if he never has an internet connection he will receive the error that we cannot check his phone time.
If its a temporary no internet he will receive an no internet error when he tries to submit


[FEATURE] Add efficient k-NN filtering support to Neural query

Is your feature request related to a problem?
Currently, there is no way to use efficient k-NN filtering with a Neural query (documentation)
What solution would you like?
Adding a filter field to the neural query which is pretty similar to the syntax of efficient k-NN filtering:
"neural": {
  "<vector_field>": {
    "query_text": "<query_text>",
    "query_image": "<image_binary>",
    "model_id": "<model_id>",
    "k": 100,

    "filter": {
      ...
    }

  }
}

@SeyedAlirezaFatemi the feature is already supported with neural query clause and works exactly you have defined in the description.
The documentation is not updated to say that it is supported,
cc: @jmazanec15 @vamshin

I think the Java Client for OpenSearch also doesn't support it.

For filtering support, we do mention it here: https://opensearch.org/docs/latest/search-plugins/semantic-search/#step-4-search-the-index-using-neural-search. That being said, its not easy to find. We should have a section on the query itself and the parameters of the query.

I think the Java Client for OpenSearch also doesn't support it.

@SeyedAlirezaFatemi for which java client? high level java rest client or opensearch-java(low level client)?

@navneet1v opensearch-java. NeuralQuery

@SeyedAlirezaFatemi thanks for providing the details. I need to move this issue to opensearch-java repo, as the changes need to happen there.
cc: @VachaShah, @vamshin , @dblock , @jmazanec15

@navneet1v is prefiltering also available for neural sparse search ?

@navneet1v is prefiltering also available for neural sparse search ?

@zane-neo can you please reply on this.

Closing the issue. Please reopen the issue in java client repo.
For sparse vector search, it works in a manner similar to text search. Therefore, the concept of pre-filtering does not apply.


Check that DSN is binary before trying URI.parse

I was setting up Sentry with DSN read from env, and got some crashes when the required variable was not set, because URI.parse was being called with nil, which doesn't match.
To fix this and get a nicer warning message instead of an exception (like #218), I added a is_binary guard in Sentry.Client.get_dsn. Plus a couple of tests to verify that it works as intended.

Thank you for the contribution (and tests)! 💖


Small improvements for the crate search input

Here's the output (the "important" part is at the top right):

Updated.

Thanks for the PR @GuillaumeGomez, but I am not sure about this change, is it really necessary? I prefer current look of it.

It took me a while to:

See this input.
Understand it was an input.

By adding borders, it makes it more obvious and easier to spot.

Yeah, I agree it's hard to see, but adding borders is just making it look like another link in navbar. I don't think it's solving visibility problem.

Might be worth giving it a try and ask users what they think about it no? Reverting the change is easy after all.

@onur I think you have named the problem -- it looks like toolbar text with or without the lines.

I think I'd prefer it with a spyglass icon to the left instead. Anyway, why not post the screenshot on r/rust before pushing instead of afterwards?

I think this should be merged, because it is better then the current solution, and it does not prevent a different solution in the future. I didn't even know it was a search box until I read this pull request.
r? @jyn514

I don't think it's bad, I just don't think it helps very much. I like the spyglass suggestion.

Closing in favor of https://github.com/rust-lang/docs.rs/pull/577


consider tagging a new release please :)

Hi, I would love to see a new release happening, as the last one is a year old and it would be great to get the changes into distros

Yeah, I have also been thinking about a new release. The last one was in December and I was planning (roughly) around the same time frame. Feature wise, not that much changed, so this will be more a bug-fix release.
I'd like to get some issues fixed, then await some last minute issues before the new release. Let me schedule it around the weekend of November 09/10. ping @bling

sounds good, marking my calendar.

v0.11 has been tagged @bling please update the vim.org script. Thanks!

uploaded!

@bling @chrisbra
Thank you!


can't access the full-stack-fastapi-postgresql (running inside vagrant vm) from host machine

Hi
I have setup the stack inside the vm created by vagrant (with creating the setup defined by https://dockerswarm.rocks/). ONly directly setup the stack inside the virtual machine
vagrant@vagrant:/vagrant/$ docker ps
CONTAINER ID        IMAGE               COMMAND                  CREATED             STATUS              PORTS                                                 NAMES
1fe5b95c243c        dpage/pgadmin4      "/entrypoint.sh"         25 minutes ago      Up 4 minutes        80/tcp, 443/tcp, <IP_ADDRESS>:5050->5050/tcp               pgadmin_1
76ad6cf25f40        mher/flower         "flower --broker=amq…"   25 minutes ago      Up 4 minutes        <IP_ADDRESS>:5555->5555/tcp                                flower_1
87bda7d3461d        frontend:latest     "nginx -g 'daemon of…"   25 minutes ago      Up 5 minutes        80/tcp                                                frontend_1
308de721e621        traefik:v2.2        "/entrypoint.sh --pr…"   25 minutes ago      Up 4 minutes        <IP_ADDRESS>:80->80/tcp, <IP_ADDRESS>:8090->8080/tcp            proxy_1
ed25baa2c098        postgres:12         "docker-entrypoint.s…"   25 minutes ago      Up 5 minutes        5432/tcp                                              db_1
368e4a8915e0        rabbitmq:3          "docker-entrypoint.s…"   25 minutes ago      Up 5 minutes        4369/tcp, 5671-5672/tcp, 15691-15692/tcp, 25672/tcp   queue_1
Inside vagrantfile i did forward all the ports mentioned in above table. None of following is working for me on host machine:
Frontend, built with Docker, with routes handled based on the path: http://localhost
Backend, JSON based web API based on OpenAPI: http://localhost/api/
Automatic interactive documentation with Swagger UI (from the OpenAPI backend): http://localhost/docs
Alternative automatic documentation with ReDoc (from the OpenAPI backend): http://localhost/redoc
PGAdmin, PostgreSQL web administration: http://localhost:5050
Flower, administration of Celery tasks: http://localhost:5555
Traefik UI, to see how the routes are being handled by the proxy: http://localhost:8090 (this is understandable as I didn't set this up)
Does the stack needs to be installed on host machine only (and not inside the vagrant virtual machine)? Or maybe I am missing something please advise. Many thanks.

resolved this.

Thanks for reporting back and closing the issue :+1: