Improve getting started page

Try to make https://kubernetes.io/docs/setup/ better [preview]

link to the downloads page
reword existing text
add “What's next” hyperlinks

Hi @sftim .
The changes look good.
With these updates, what do you think about removing the "Learning Environment" sub-directory and section _index.md page?
/lgtm

Thanks @sftim .
/label tide/merge-method-squash
/approve


chore!: simplify discriminants handling in schema derives

There’s no need for a separate discriminant variable and assignment
statement in the derived code.  The value of the discriminant can be
included directly in the entry for the variant.

cargo-doc failure is not of my doing.


Idea: Test Alert Conditions Against Historical Data for Different for: Values

Proposal
Hello Prometheus team,
We are happy long-time users of Prometheus. One functionality we’ve missed on multiple occasions is a way to test alert conditions with different for: values against historical data. This would be helpful during alert development to understand when specific conditions would have been triggered with different for: values in the past to refine our alert conditions accordingly.
Currently, we’ve created a makeshift approach by wrapping queries in subqueries to simulate Prometheus’s alert evaluation behavior. Here’s an example of such a query:
clamp_max(count_over_time(( sum(rate(fluentbit_output_proc_records_total{prometheus="monitoring-kube-prometheus-stack/gke-main-devel-prometheus"}[5m])) by (prometheus) >= 1000 )[5m:30s]) >= 10, 1)

To generalize, this approach can be represented as:
clamp_max(count_over_time(( {{ALERT-QUERY}} )[{{FOR}}:30s]) >= {{FOR / 30s}}, 1)

While this method works, it is a bit cumbersome to handle. We believe integrating this functionality into the Prometheus UI could simplify this process. For instance, a feature where users could input alert conditions and for: values, and the UI would display when the alert would have been triggered historically using stored time-series data, could make this workflow much more intuitive.
We’re willing to contribute to developing such a feature but wanted to first discuss whether it aligns with the project’s direction and whether it makes sense in general as well as a part of Prometheus’s core functionality.
If there’s already a way to achieve this that we might have missed, we’d appreciate any pointers to the relevant documentation or examples. Otherwise, we’d like to suggest this as a feature request for consideration.
Thanks!
Stephan

I believe this is what was attempted by https://github.com/prometheus/prometheus/pull/4277 long ago.
This functionality has later been implemented in the 3rd party tool promutil, see https://github.com/kadaan/promutil?tab=readme-ov-file#web .
If anyone wants to implement such a feature directly in Prometheus, I would certainly like it (but probably needs some discussion if other developers like it, too).


Updated ListView to return synched ListView.Selection and ListView.Selec...

Updated ListView to return synched ListView.Selection and ListView.SelectionIndex. These were out of sync when Filtering was used.

@kkashi01 LGTM. Note I added back the AI lists comment. I also had to make a small modification to deal with a change Hal made. The items in the AdapterArray are now Spannable Strings. I've pushed it up to Gerrit prior to merging and have asked Hal to take a quick look. But it looks OK to me.
-Jeff

@kkashi01 MERGED


Curly Brackets instead of Square Brackets

Hello! Anyone knows how to implement the toString() method so that it would print the curly brackets instead of the square brackets? I tried calling map.values() but it returned me the square bracket version.
Codecrunch:

Mine:

Thanks a lot!

You can remove the square bracket by using substring method since the curly bracket is the first and last character, use substring(1, stringname.length() - 1). Just to note however calling the values() method is not allowed codecrunch will reject your program

You can also try override the toString() method!

Hi! I am also facing the same issue, may i know how u resolved it since we are not allowed to use values()?

did u use ImList instead of a dictionary? anyway just hardcode the curly brackets in that’s what i did


Installing zeek-Kafka error

Summary of the issue
I am trying to install zeek-kafka
But facing error
Expected behavior
...
I expect it to get installed
Steps to reproduce
...
$ curl -L https://github.com/edenhill/librdkafka/archive/v1.4.2.tar.gz | tar xvz
$ cd librdkafka-1.4.2/
$ ./configure --enable-sasl
$ make
$ sudo make install
/opt/zeek/bin/zkg install seisollc/zeek-kafka
Where applicable, consider providing a patch that uses the end to end testing environment.
Logs, errors, etc.
...
`root@CT:/opt/zeek/bin# ./zkg install seisollc/zeek-kafka
The following packages will be INSTALLED:
zeek/seisollc/zeek-kafka (v1.0.0)
Verify the following REQUIRED external dependencies:
(Ensure their installation on all relevant systems before proceeding):
from zeek/seisollc/zeek-kafka (v1.0.0):
librdkafka ~1.4.2-RC1
Proceed? [Y/n] Y
zeek/seisollc/zeek-kafka asks for LIBRDKAFKA_ROOT (Path to librdkafka installation tree root) ? [/usr/local]
Saved answers to config file: /opt/zeek/etc/zkg/config
Running unit tests for "zeek/seisollc/zeek-kafka"
error: "zeek/seisollc/zeek-kafka" tests failed, inspect contents of /opt/zeek/var/lib/zkg/testing/zeek-kafka for details, especially any "zkg.test_command.{stderr,stdout}" files within /opt/zeek/var/lib/zkg/testing/zeek-kafka/clones/zeek-kafka
Proceed to install anyway? [N/y] y
Installing "zeek/seisollc/zeek-kafka"...................................................
Installed "zeek/seisollc/zeek-kafka" (v1.0.0)
Loaded "zeek/seisollc/zeek-kafka"
root@CT:/opt/zeek/bin# ./zeek -N Seiso::Kafka
error in /opt/zeek/share/zeek/base/init-bare.zeek, line 1: cannot load plugin library /opt/zeek/lib/zeek/plugins/packages/zeek-kafka//lib/SEISO-KAFKA.linux-x86_64.so: /usr/local/lib/librdkafka++.so.1: undefined symbol: rd_kafka_commit_transaction
fatal error in /opt/zeek/share/zeek/base/init-bare.zeek, line 1: aborting after plugin errors
root@CT:/opt/zeek/bin# ^C
`
zkg.test_command.stderr
kafka.l2s-l2e-no-overlap ... failed
% 'zeek ../../../scripts/Seiso/Kafka/ /opt/zeek/var/lib/zkg/testing/zeek-kafka/clones/zeek-kafka/tests/.tmp/kafka.l2s-l2e-no-overlap/l2s-l2e-no-overlap.zeek > output' failed unexpectedly (exit code 1)
% cat .stderr
error in /opt/zeek/share/zeek/base/init-bare.zeek, line 1: cannot load plugin library /opt/zeek/var/lib/zkg/testing/zeek-kafka/clones/zeek-kafka/build//lib/SEISO-KAFKA.linux-x86_64.so: /usr/local/lib/librdkafka++.so.1: undefined symbol: rd_kafka_commit_transaction
fatal error in /opt/zeek/share/zeek/base/init-bare.zeek, line 1: aborting after plugin errors
kafka.l2s-set-l2e-set ... failed
% 'zeek ../../../scripts/Seiso/Kafka/ /opt/zeek/var/lib/zkg/testing/zeek-kafka/clones/zeek-kafka/tests/.tmp/kafka.l2s-set-l2e-set/l2s-set-l2e-set.zeek > output' failed unexpectedly (exit code 1)
% cat .stderr
error in /opt/zeek/share/zeek/base/init-bare.zeek, line 1: cannot load plugin library /opt/zeek/var/lib/zkg/testing/zeek-kafka/clones/zeek-kafka/build//lib/SEISO-KAFKA.linux-x86_64.so: /usr/local/lib/librdkafka++.so.1: undefined symbol: rd_kafka_commit_transaction
fatal error in /opt/zeek/share/zeek/base/init-bare.zeek, line 1: aborting after plugin errors
kafka.l2s-set-l2e-unset ... failed
% 'zeek ../../../scripts/Seiso/Kafka/ /opt/zeek/var/lib/zkg/testing/zeek-kafka/clones/zeek-kafka/tests/.tmp/kafka.l2s-set-l2e-unset/l2s-set-l2e-unset.zeek > output' failed unexpectedly (exit code 1)
% cat .stderr
error in /opt/zeek/share/zeek/base/init-bare.zeek, line 1: cannot load plugin library /opt/zeek/var/lib/zkg/testing/zeek-kafka/clones/zeek-kafka/build//lib/SEISO-KAFKA.linux-x86_64.so: /usr/local/lib/librdkafka++.so.1: undefined symbol: rd_kafka_commit_transaction
fatal error in /opt/zeek/share/zeek/base/init-bare.zeek, line 1: aborting after plugin errors
Your environment

Version of Zeek  : 4.0.5
Version or commit hash of the zeek-kafka package : latest
Operating System and version Ubuntu 18.04

I cannot reproduce this in our docker environment ( which was already using 1.4.2 but is using ubuntu 20x ) I have updated the zkg meta requirement to drop the rc.

How I download your latest commit ,
this command will fetch original i guess curl -L https://github.com/edenhill/librdkafka/archive/v1.4.2.tar.gz | tar xvz
Can you give me a git clone command so that i get that

It was just merged to main

It was just merged to main

I went with Manual Installation , it was installed but still local.zeek can't find package


"For more information, see this article" redirects back to this article

The "this article" link at the end of the article redirects back to the article that contains the link:

Support for file system plug-ins and customizations while the executable of the main application is in a VFS is planned for the next major release of Windows. You can preview this support starting in Windows 10 Insider Preview Build 18312 or later. For more information, see this article.

I suspect the entire paragraph should be removed, because the article already contains the "Modification packages on Windows 10, version 1903, and later" section.

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

ID: 34ef494e-4c2f-e7dd-d8c6-915e0aadab18
Version Independent ID: fd2a9514-09d3-ab99-291e-96fcf97ee357
Content: Customize your Enterprise apps with modification packages - MSIX
Content Source: msix-src/modification-packages.md
Product: msix
GitHub Login: @dianmsft
Microsoft Alias: diahar

This has been fixed

This has been fixed

Apparently in https://github.com/MicrosoftDocs/msix-docs/commit/cd323db697c7bbec66aa528ed18c0c6a70a8f7dc.


Replace boid representation

Currently boids are represented by a single circle. This should be replaced with something that indicates the direction of flight and could be something as simple as a triangle.

Commit 8a933bb replaces circle representation with triangles showing the direction of travel. However Canvas is not being correctly cleared between frames.


Add Elixir dependency to yml build

workflow cant build project to run tests as it doesnt have Elixir

Uncomment Elixir version in test.yml


(fix) 'Bad Connection' modal not showing up on Electron app

ASANA Ticket: [bug] the disconnect info modal not always show up on connectivity issues
Issue: bitfinexcom/bfx-hf-ui#592
For Electron users, the 'Bad Connection' modal was not showing up because of local API and Data servers being used, so when the user was having internet problems, it had no effect on the localhost connection.
This is why a new websocket connection for Electron users has been added.
UI Demonstration:

@Blaumaus I wanted to try this PR locally first to see if we could use some helper fn of ufx/bfx-containers. But I could not login/reset pw, looks like I am not proceeding because of following check. Please help fix it
  const offline = Object.keys(sockets).some(s => sockets[s].status !== 'online')
socket state
{ bfx-hf-data-server: {status: 'online', lastActivity:<PHONE_NUMBER>577} bfx-hf-server: {status: 'online', lastActivity:<PHONE_NUMBER>573} bfx-wss-pub-server: {status: 'offline', lastActivity: null} }

Thank you for addressing this!

@Blaumaus I wanted to try this PR locally first to see if we could use some helper fn of ufx/bfx-containers. But I could not login/reset pw after using this branch, looks like I am not proceeding because of following check. Please help fix it
 const offline = Object.keys(sockets).some(s => sockets[s].status !== 'online')
socket state
{ bfx-hf-data-server: {status: 'online', lastActivity:<PHONE_NUMBER>577} bfx-hf-server: {status: 'online', lastActivity:<PHONE_NUMBER>573} bfx-wss-pub-server: {status: 'offline', lastActivity: null} }

fixed

@Blaumaus I wanted to try this PR locally first to see if we could use some helper fn of ufx/bfx-containers. But I could not login/reset pw after using this branch, looks like I am not proceeding because of following check. Please help fix it
 const offline = Object.keys(sockets).some(s => sockets[s].status !== 'online')
socket state
{ bfx-hf-data-server: {status: 'online', lastActivity:<PHONE_NUMBER>577} bfx-hf-server: {status: 'online', lastActivity:<PHONE_NUMBER>573} bfx-wss-pub-server: {status: 'offline', lastActivity: null} }

fixed

sorry for delayed reply.
when I tried with latest pull, still ping-worker is not working since it always return from readyState check (https://github.com/bitfinexcom/bfx-hf-ui-core/blob/main/src/redux/middleware/ws/index.js#L70:L72)
I have proposed a solution in following PR using getWSConencted selector from ufx-ui/bfx-containers.
getWSConencted already reflects state of REACT_APP_UFX_WSS_URL , and it is pinged already from wsPingSaga of bfx-containers.
@Blaumaus could you please review PR  https://github.com/Blaumaus/bfx-hf-ui-core/pull/1  ?

closing since implemented in https://github.com/bitfinexcom/bfx-hf-ui-core/pull/107


Unable to cache a POST request

Hi, please excuse my lack of knowledge as just starting mycoding journey.
I am having the same error occur in my console relating to the above
the error is -
cacheWrapper.mjs:97 Uncaught (in promise) TypeError: Request method 'POST' is unsupported
    at Object.put (cacheWrapper.mjs:97)
put @ cacheWrapper.mjs:97

I have read that background sync might solve this but cant get it to work, i have the following code in my SW file
const bgSyncPlugin = new workbox.backgroundSync.Plugin('myQueueName', {
  maxRetentionTime: 24 * 60 // Retry for max of 24 Hours (specified in minutes)
});

workbox.routing.registerRoute(
  /\/api\/.*\/*.json/,
  new workbox.strategies.NetworkOnly({
    plugins: [bgSyncPlugin]
  }),
  'POST'
);

That code looks fine. (There's more context in https://github.com/GoogleChrome/workbox/issues/1584)
If you are using new workbox.strategies.NetworkOnly as the handler for your route, there should not be any attempt to call put(), so I think your error is coming from another portion of your service worker file.
Can you share your entire service worker, and confirm which URL is being requested when you encounter this error message?

Hi Jeff, thank you.
SW is attached.  URL is - https://storage.googleapis.com/workbox-cdn/releases/_private/cacheWrapper.mjs
sw.txt

say error comes in line 97 or cacheWrapper URL

Any of those routes which use NetworkFirst might be triggering this error. NetworkFirst will, under the hood, attempt to store the request/response combination in a local cache, and that attempt will fail if your request has a mode of POST.
If you could say which URL your web app is POSTing to, I could give some advice as to which route to change.
(Also, your
workbox.routing.registerRoute(
  '/application/templates/sidebar.php',
  '/application/templates/admin-header.php',
  '/application/templates/admin-footer.php',
  '/application/templates/frontend-footer.php',
  '/application/templates/frontend-footer.php',
  '/application/index.php',
  new workbox.strategies.NetworkFirst()
  );

route is invalid, as you can only list one parameter as the matchHandler when calling registerRoute. Listing a whole bunch of strings won't work.)

ok, i believe the error is coming from the  "importScripts('https://storage.googleapis.com/workbox-cdn/releases/4.3.1/workbox-sw.js');"    line
with regards to the register route, so i need a handler for each string?

It's not coming from that line—that line is responsible for importing the Workbox runtime code, which in turn allows you to use Workbox to define your routes and caching behavior.
An error like that would only come from once you actually start running your service worker and it intercepts a POST request. What I'd need from you is the URL of that POST request. You should be able to see it from, e.g., the Network panel in Chrome's DevTools.
You can use a regular expression in place of that array of strings. For example, new RegExp('/application/.*\\.php') seems like it may work for you, if you really want to match all of those URLs.

apologies for my lack of knowledge on this.   See screenshot of the console and network panel.
what are you looking for on here?
URL's i can see are
https://babsdemo.co.uk/application/message-popup.php
https://babsdemo.co.uk/application/ajax/counter.php

There's a "Method" option that you could check to get the method logged in DevTools:

If I had to guess, based on your traffic, which of those requests were a POST I'd say it's likely for counter.php (since doing things that involve changing state, like incrementing a counter, are usually done via a POST).

ok, so both are POST

Its starting to cause network failures now -

I would recommend removing
workbox.routing.setDefaultHandler(
  new workbox.strategies.NetworkFirst()
);

That ends up picking up your POST requests as well and caching them, and you don't want that.
If you want to set up a route that specifically does match the POST requests and uses the background sync plugin, that's possible... but I don't really think it's necessary for your use case. You wouldn't want to sync a failed login POST request, for instance.

@jeffposnick in my case I have a Jekyll based AMP website so my hands are really tied when fixing this. I identified that the only POST request going out to Google Tag Manager (https://www.googletagmanager.com/gtag/amp?__amp_source_origin=https%3A%2F%2Fcsaba.page) and even that is indirectly generated by the amp-analytics component. My plan was to apply a networkOnly strategy for that domain, but I still get the error:
workbox.routing.registerRoute(
  /^https?:\/\/www\.googletagmanager\.com/,
  workbox.strategies.networkOnly()
)

What could be still a reason for the service worker to try to cache a POST request? I thought networkOnly will disable any caching.


Add the aurora operation notebook. Enable nbsphinx.

This pull request generates a notebooks that outlines how to create an MTH5 file and run it through the Aurora processing pipeline with an emtf function being output.
This PR also enables nbsphinx so the the operate_aurora.ipynb notebook, and other notebooks, can be embedded into the documentation.

Test failures seem to be in main branch because the failed test in this PR are the same as the same test in the main branch. This PR is not associated with the failing test cases.


Review SPR_SEEDLOT packages

Review the SPR_SEEDLOT package and the related objects in order to identify any risks/issues in using its add procedure to generate a SEEDLOT_NUMBER in the old SPAR Oracle database.
This is a spike activity related to the task:
https://github.com/bcgov/nr-spar/issues/237

In order to create and lock a new seedlot number there are a few steps that needs to be executed:

First call the init procedure to set all attributes to null;
Then set all relevant attributes by calling the set procedures for each attribute;
Finally call the add procedure that generates the seedlot number and inserts the record on SEEDLOT table.

After adding a new seedlot by providing only the Species, Genetic Class, and Applicant Agency (default values), those are the columns populated on SEEDLOT table:

SEEDLOT_NUMBER
SEEDLOT_STATUS_CODE
VEGETATION_CODE
GENETIC_CLASS_CODE
REGISTERED_SEED_IND = N
UTM_ZONE = 0
UTM_EASTING = 0
UTM_NORTHING = 0
ORIGINAL_SEED_QTY = 0
STORED_CLI_NUMBER
STORED_CLI_LOCN_CD
APPLICANT_CLIENT_NUMBER
APPLICANT_CLIENT_LOCN
ENTRY_USERID
ENTRY_TIMESTAMP
UPDATE_USERID
UPDATE_TIMESTAMP
REVISION_COUNT = 2


Make the code sample clearer

@RedTachyon This is to fix https://github.com/openai/gym/issues/2716
I confirmed the code runs.
I should have opened the original issue on this repo.

I kind of think that another proposed comment of just adding a comment that the policy function will have to be provided by the user makes a lot more sense?

Fixed, is this what you meant?

Ya that is.
@trigaten this can be merged when applicable


Testcases: Added testcases for WorldCountries Component

Types of Changes

[ ] Bug fix (non-breaking change that fixes an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Description
Testcases: Added testcases for WorldCountries Component and improved code coverage
How Has This Been Tested?

Screenshots (if appropriate):

Fixed issue that was failing CI.

@all-contributors please @NitinRamnani for test

@all-contributors please add @NitinRamnani for test


Allow @latest specifications in keyfiles

Fixes #797

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

14 of 14 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.5%) to 97.437%

Totals

Change from base Build<PHONE_NUMBER>:
-0.5%

Covered Lines:
1363

Relevant Lines:
1387

💛  - Coveralls


⚠ Bofrost.es has degraded performance

In b6c7a0c, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1432 ms

Resolved: Bofrost.es performance has improved in bf7ebff after 6 minutes.


Brackets

Are brackets necessary for every statement? The example here doesn't show them.

No... An "if" statement doesn't need braces if the "then" clause contains only one statement.


App Crashed Without Exception on Media Picked Event

Bug Information
Version Number of Plugin: 4.4.7 - beta
Device Tested On: LGE LM-G710 - Android 9
Simulator Tested On: N/A
Version of VS: 2017 - 15.9.5
Version of Xamarin: <IP_ADDRESS>
Versions of other things you are using:
Steps to reproduce the Behavior
Taking a picture as outlined in all the tutorials and sample docus.
Expected Behavior
Clicking the camera button takes the picture and gives you the option to use or cancel the picture. Selecting OK to use the picture returns a MediaFile from TakePhotoAsync()
Actual Behavior
On selecting OK to use the picture, the handler for the MediaPickerACtivity.MediaPicked event is never fired. The app just closes without throwing an exception. When debugging, the debugging session closes. Breakpoints within the handler are never hit.
Code snippet
try
{
    var cameraPermission = await CrossPermissions.Current.CheckPermissionStatusAsync<CameraPermission>();
    if (cameraPermission != PermissionStatus.Granted)
    {
        if (await CrossPermissions.Current.ShouldShowRequestPermissionRationaleAsync(Permission.Camera))
        {
            //Show toast
        }

        cameraPermission = await CrossPermissions.Current.RequestPermissionAsync<CameraPermission>();
    }
    switch (cameraPermission)
    {
        case PermissionStatus.Granted:
            {
                var photo = await CrossMedia.Current.TakePhotoAsync(
                    new StoreCameraMediaOptions
                    {
                        SaveToAlbum = false,
                        PhotoSize = PhotoSize.Medium,
                        DefaultCamera = CameraDevice.Front
                    }
                );

                if (photo == null)
                {
                    //show toast
                }
                break;
            }
        case PermissionStatus.Denied:
            //Show Toast
            break;
    }
}
catch (Exception ex)
{
    Toast.ShowLongMessage("An unexpected error occured");
}

I took out the nuget package and added the project manually to my solution. I can step through the code and I get right up to TakeMediaAsync() android's MediaImplementation.
private Task<MediaFile> TakeMediaAsync(string type, string action, StoreMediaOptions options, CancellationToken token = default(CancellationToken))
        {
            var id = GetRequestId();

	        if (token.IsCancellationRequested)
				return Task.FromResult((MediaFile) null);

            var ntcs = new TaskCompletionSource<MediaFile>(id);
            if (Interlocked.CompareExchange(ref CompletionSource, ntcs, null) != null)
                throw new InvalidOperationException("Only one operation can be active at a time");

			context.StartActivity(CreateMediaIntent(id, type, action, options));

///THIS HANDLER IS NEVER HIT AFTER I SELECT A PICTURE
			void handler(object s, MediaPickedEventArgs e)
			{
				var tcs = Interlocked.Exchange(ref CompletionSource, null);

				MediaPickerActivity.MediaPicked -= handler;

				if (e.RequestId != id)
					return;

				if (e.IsCanceled)
					tcs.SetResult(null);
				else if (e.Error != null)
					tcs.SetException(e.Error);
				else
					tcs.SetResult(e.Media.FirstOrDefault());
			}

			token.Register(() =>
	        {
		        var tcs = Interlocked.Exchange(ref CompletionSource, null);

		        MediaPickerActivity.MediaPicked -= handler;
		        CancelRequested?.Invoke(null, EventArgs.Empty);
		        CancelRequested = null;

		        tcs.SetResult(null);
	        });

			MediaPickerActivity.MediaPicked += handler;

			return CompletionSource.Task;
		}

After further testing it looks like this works on different devices. At least the Samsung S4. It looks might it be specific to only LGE phones. I thought it might be caused by #718 but it doesn't seem so. Might be something to do with an LGE update.
Any thoughts anyone?

Could this be related to #696 ?


Fix 72: AnnotationsPane now checks for currentAnnotations.clusters_text

PR Overview

Fixes #72
Classifications page on localhost:3000/classify would have issues rendering some Subjects (presumably, those without previous markings)
Turns out this was due to AnnotationsPane.renderPreviousAnnotations not checking the existence of currentAnnotations.clusters_text before trying to .map() it.
Fix is a simple check before execution.

NOTE

@wgranger it occurs to me that the reason I didn't catch this earlier was because the currentAnnotations.clusters_text was always provided... which hints that Caesar is likely now live with the new text aggregation updates, wooo ( https://github.com/zooniverse/aggregation-for-caesar/pull/34 )

Status
Ready for review just in case there's anything else obvious I missed. Otherwise, since this is minor, I'll merge within 24 hours.

EDIT: just checked Slack, and I realise now that @wgranger is already aware of this issue. (Sorry, I missed Friday's updates.) This PR is now marked irrelevant and closed, as I assume Will is already working on a more generalised PR that will update the Annotations Pane to handle the new data format.
However, in addition to said data format updates, please remember to also run the necessary existential checks before running, e.g., .map() functions. Simultaneously, I'll make a note to double-check for this when I run my PR reviews next time.

This PR has been resurrected and will be merged immediately.
I just had a talk with Will, and while he is working on a new branch with the relevant Caesar JSON data format updates, that will take a while and the data format is still expected to change further. This means we need a fix for the front-end crashing ASAP, so this should take care of that.
Also, to reiterate what I mentioned on Slack: the front end should never crash if an external service is unavailable. It should either fall back to a failsafe (e.g. if Caesar is down, we should still be able to do normal Classifications... and the metadata should record that the aggregation service was unavailable) or fail gracefully (e.g. if Panoptes is down, we need a "lol sorry, no Zooniverse for you" message)


[stable/mercure] Automatically Roll Deployments

Automatically Roll Deployments on configmap and secret changes
Signed-off-by: Patrick Desnoyers<EMAIL_ADDRESS>Is this a new chart
No
What this PR does / why we need it:
Automatically Roll Deployments on configmap and secret changes
Which issue this PR fixes
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x ] DCO signed
[x ] Title of the PR starts with chart name (e.g. [stable/mychartname])

/ok-to-test

/approve

/lgtm


ui.grid dependency missing. needs merging commit fbe16d6 to 1.5.x-HBase1.x

Hello,
The changes introduced in commit fbe16d6 are missing from the client side code.
Ui-grid dependency missing the the javascript bundle.
Can you please merge the fbe16d6 commits to the 1.5.x-HBase1.x branch.
Thank you

Latest commits in master branch have been merged into 1.5.x-HBase1.x branch, including the fix for ui.grid.

Thanks,
I build the latest version of this branch and it's working fine now.
Somehow the chinese symbol for sorting made it into the resulting dataset table in the query page.
Please check attached image.

I've created a JIRA to track this issue:
https://issues.apache.org/jira/browse/KYLIN-1688


SoftDelete does not work with all query

Hello,
If I'm not mistaken, SoftDelete with deleted_at value does not work with all query. All items returned ...
I use "use SoftDeletes" in my model. And delete method to remove item.

any updates ? Were you able to solve your issue?

one thing i noticed, by watching the logs (query), it uses the alias or collection name together with the deleted_at, which generally a non-existent field (e.g. my_collection.deleted_at vs just deleted_at) in that collection

i found out the issue. this happens when you use SoftDeletes trait from Laravel, not from this library. so it should be from Jenssegers\Mongodb\Eloquent\SoftDeletes.


Fix nil pointer panic in kvstore

Signed-off-by: Michael Gasch<EMAIL_ADDRESS>
Changes

Fix bug in kvstore causing nil pointer panic on an empty map (map without data)

/kind  bug

Fixes #2001

Release Note

Docs

Downstream tests are unrelated to this change and being worked on in downstreams.

Downstream tests are unrelated to this change and being worked on in downstreams.

Thanks!
/lgtm
/approve

Thanks!
/lgtm
/approve

/retest
metric flakes

/retest
metric flakes


nix-plugin-pijul: 0.1.5 -> 0.1.6

Fixes compilation with nixVersions.git.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

The "Error: Cannot get path of un-named identity"? That's a Pijul bug unfortunately, broke half a year ago and hasn't been fixed since: https://nest.pijul.com/pijul/pijul/discussions/953


add restart always policy to services in compose.prod.yml ?

I just had an error report from an LIS GCV user that turned out to be a case of one of the microservices having exited (for reasons not clear to me):
lis-genes-1                           "python3 -u -m genes"    genes                           exited (0)
seems like adding a restart policy to the compose.prod.yml for each service would make things more robust, though perhaps there are reasons this is a bad idea. I think we discussed something similar here:
https://github.com/legumeinfo/graphql-server/issues/102
and ultimately decided it was a good thing to do (although there were other things we wanted to address there as well).
Some opinions about docker compose best practices here for reference:
https://release.com/blog/6-docker-compose-best-practices-for-dev-and-prod
I guess making the policy configurable via .env as they have done might be preferable to just hard-coding it as "always".

either way, happy if you have some time to do it and it's not urgent I just thought I'd raise the issue while fresh in my mind.


Cant start controller for LLaVa "llava' is already used by a Transformers config, pick another name"

Describe the issue
Issue:
Controller/Webserver wont start
ive tried everything chatgpt and bard suggested, no results.
-pip uninstalling reinstalling llava (wich is just a part of it i guess since it took mere seconds)
-pip install transformers --upgrade
-renamed the whole thing which lead to other errors with wrongly named files
i am also very new to all of this so sorry if this is really stupid.
thank you, -benik
Command:
python -m llava.serve.controller --host <IP_ADDRESS> --port 10000

Log:
Traceback (most recent call last):
  File "C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.10_3.10.3056.0_x64__qbz5n2kfra8p0\lib\runpy.py", line 187, in _run_module_as_main
    mod_name, mod_spec, code = _get_module_details(mod_name, _Error)
  File "C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.10_3.10.3056.0_x64__qbz5n2kfra8p0\lib\runpy.py", line 110, in _get_module_details
    __import__(pkg_name)
  File "C:\Users\Niklas\llava\llava\__init__.py", line 1, in <module>
    from .model import LlavaLlamaForCausalLM
  File "C:\Users\Niklas\llava\llava\model\__init__.py", line 1, in <module>
    from .language_model.llava_llama import LlavaLlamaForCausalLM, LlavaConfig
  File "C:\Users\Niklas\llava\llava\model\language_model\llava_llama.py", line 110, in <module>
    AutoConfig.register("llava", LlavaConfig)
  File "C:\Users\Niklas\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.10_qbz5n2kfra8p0\LocalCache\local-packages\Python310\site-packages\transformers\models\auto\configuration_auto.py", line 1128, in register
    CONFIG_MAPPING.register(model_type, config, exist_ok=exist_ok)
  File "C:\Users\Niklas\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.10_qbz5n2kfra8p0\LocalCache\local-packages\Python310\site-packages\transformers\models\auto\configuration_auto.py", line 828, in register
    raise ValueError(f"'{key}' is already used by a Transformers config, pick another name.")
ValueError: 'llava' is already used by a Transformers config, pick another name.

me too

me too

im not quite sure but i think running this fixed it for me:
git pull
pip install -e .

What version of transformers are you using?
In the latest version, the LLaVA already exists so you could (1) install the version specified in this repo or (2) change the name for registering the config.

explain more

This line:
File "C:\Users\Niklas\llava\llava\model\language_model\llava_llama.py", line 110, in <module>
    AutoConfig.register("llava", LlavaConfig)

File "C:\Users\Niklas\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.10_qbz5n2kfra8p0\LocalCache\local-packages\Python310\site-packages\transformers\models\auto\configuration_auto.py", line 828, in register
    raise ValueError(f"'{key}' is already used by a Transformers config, pick another name.")
ValueError: 'llava' is already used by a Transformers config, pick another name.

tells you that you cannot register a config with the name 'llava' bc it's already used. This is the case when using transformers > 4.35 (I think where HF added LLaVA). So check version of transformers which shoulde be 4.31.. OR you can check the line where you register to AutoConfig.register("llava_org", LlavaConfig) or something.
Best to install the right dependencies though to avoid other conflicts.

@HB-ai-cf can you explain more in detail how to do that?

The latest code base shall have this fixed. Thanks.

Transformer version is 4.37.1
How to  fix (check the line where you register to AutoConfig.register("llava_org", LlavaConfig or other solution)

It is most likely because the transformer version is incompatible. Try the following:
pip list
This will list your packages in your environment.
Then use:
pip uninstall transformers
And then finally:
pip install transformers==4.28.1
This worked for me

for me it was:
I am debugging llava locally  on my computer and did a  pip install -e .  to a  local folder  then few weeks later ,  I did a pip  install on another version of transformer. So, causes a conflict.  so do another pip install -e . where the LLava folder is and the problem will go away.

my solution is change the model_type param in the customized model config before calling any "xxx.register(...)", then the conflict disappears


Rotation for ellipse and svgPath

This PR adds high-level support for rotating svgPaths, and ellipses.
Some notes for @Hopding :

I couldn't figure out why the rotating didn't have any effect on the ellipse on my first attempt, but I went on, and implemented the same thing for svgPaths.

For svgPaths, it worked as expected. So, as a workaround/fix, I've changed the ellipse-drawing method to use the drawSvgPath as well: it uses the same bezier curves as before, but they're now defined in the path string. I think it's even better this way, because the codebase got a little bit smaller (no need for the drawEllipse methods anymore).

I've modified some tests: test1, and test12 now includes rotation for ellipse, and for svgPath respectively.

There's a "break" in functionality though: in test1, when you create that "mask" effect for the text with the 2 ellipses. That no longer works as expected. There seems to be a pretty huge offset for some reason, but I couldn't figure out the solution. Some additional notes about this:

If I give proper colors to the ellipses (instead of undefined), they appear on the right position and size
If I change these 2 lines in PDFPage.ts from this:

const x = -xScale;
const y = -yScale;

to this:
 const x = centerX - xScale;
 const y = centerY - yScale;

Then the "mask" effect seems to be fine, but all the visible ellipses are having an offset. I think it must be related to something like a "context position" that is being (or needs to be) modified (with page.moveTo), or something. Because the original ellipse drawing methods used something like that. And this bring me to my last note:

I tried to create the same "mask" effect with rectangles instead of ellipses, but it didn't work for me neither with, nor without this PR. So I think it was already a quite unique, "hacky" (or at least low-level) solution before, that it only worked with ellipses.

Hey @Hopding ,
As you requested: the drawing method is now reverted to the "standard" one for ellipses, and I also figured out the rotation problem, it seems. Not exactly sure of what the problem was, I basically followed how the rectangle is being drawn. Maybe it has to do something with the lack of translate operator in the original method. Anyway, it works now. Please check and let me know!

@soadzoor I was just running through the full suite of integration tests before cutting a new release with this feature. However, while doing so I noticed a problem. Here's what integration test #1 looks like in Adobe Acrobat prior to this PR:

Of course, the orange ellipse is not rotated here. But note the magenta text overlaid on top of it. Here's what that same test looks like in Acrobat after this PR:

The ellipse is rotated, but the magenta text is now missing. Interestingly, this does not seems to be the case for all PDF readers. So far I've tested in Acrobat, Foxit, and Preview. The magenta text is missing in Acrobat and Foxit, but not in Preview:

I usually test PR changes with Preview and only prior to cutting a release do I run the full suite of tests in all readers and environments (because it's slow and time consuming). That's why I didn't catch this until now.
I haven't had a chance to look into what's causing this. And it's intriguing that not all readers have the same problem. But regardless, I'm postponing the release until you have some time to investigate this. Note that I won't revert the PR merge just yet. But I'll have to do that prior to the next release of other features, unless we're able to merge a fix for this.

Hey @Hopding ,
I noticed this right away (because I usually test PDFs with chrome's built-in reader). I didn't know that it's still okay in some readers, though.
See my original post on this PR, from this part: There's a "break" in functionality though..., it's all about this issue.
My guess is that some readers don't respect transformations for the clipping (if that's possible..?). So I believe there's not much we can do about it, because we need transformations for rotating. Note, that we were only able to use ellipses for this overlaying-mask-text-effect (prior to this PR) anyway. It didn't work with any other shapes as far as I know. So it was already a very special, "unstable", "unreliable", and also low-level feature meaning the user needed to do some low-level stuff to get it done. And again: it only worked with ellipses prior this PR.
Now, that ellipses are also created with some transformations (just like the rest of the shapes), it no longer works (at least not with all the pdf readers). It's a pity, but at least it's consistent with the rest of the shapes. And if I got to choose which one is more important:

The ability to use low-level code to draw masked texts with ellipses, and with ellipses only.
or
The ability to rotate the ellipse with a well-accustomed high-level method (just like the rest of the shapes)

I'd go with the second option, because:

It's most likely a feature that more users need. For one example, me and my team, we're long waiting for the second ability, and we'll never use the first.
It also makes the lib more consistent (ellipses would behave much more like the rest of the shapes). No special treatment / additional functionality anymore.

Considering all the above, I wouldn't hesitate much.
Of course, it's your decision, not mine.

@soadzoor Thanks for your thoughtful reply. You may well be correct that the current behavior is anomalous and doesn't make sense to try and preserve. If that is the case, I'm not necessarily opposed to releasing this change. However, before I commit to doing so, I need to better understand what exactly is going on here. The interplay between clipping, transformations, and inconsistent behavior between different readers is frustrating and I haven't had the time to investigate this yet.
I also tend to agree with you that having the ability to rotate ellipses would be more useful than the ability to mask with only ellipses. What do you think about the following approach?

If no rotation is passed (rotation === undefined), emit the same sequence of operators as the current version (probably using drawEllipsePath).
If a rotation is passed, use the new drawEllipseCurves function that you've introduced in this PR.

Implementing this approach would amount to a simple ternary condition in the drawEllipse function.
This approach would avoid introducing a breaking change, so we would not need to delay the release of this feature any longer. Admittedly, it's a bit awkward and may seem like overkill. But the only alternatives that I see are:

I release this breaking change without fully understanding what's going on here, which makes me uneasy.
I delay the release of this feature until I have to time to investigate what's happening and determine if a breaking change is the right solution, which might take several weeks.

My apologies for the wait you and your team have had to put up with to get what would seem to be an obvious, simple feature. I appreciate your attentiveness and persistence in getting this feature delivered!

@Hopding Thank you for your respone! I'm absolutely fine with your proposed approach, thank you! It seems like a decent solution for now, and I hope you'll get a chance to get to the bottom of this in the future! It surely seems like a frustratig anomaly

Version 1.10.0 is now published with this feature. The full release notes are available here.
You can install this new version with npm:
npm install<EMAIL_ADDRESS>
It's also available on unpkg:

<EMAIL_ADDRESS><EMAIL_ADDRESS>
As well as jsDelivr:

<EMAIL_ADDRESS><EMAIL_ADDRESS>

Thank you @Hopding  ! For documenting purposes, could you perhaps add the "add rotation to drawSvgPath" to the release notes (beside the "add rotation to drawEllipse")? Thanks

@soadzoor good catch! I forgot to mention that. I'll get it added to the release notes 👍


Listen to Mouse Events on the Grid Area

One-line summary [问题简述]
Would it be possible to request a feature that allows the user to listen to all mouse events on a grid area, and retrieve the x/y value at that point, even if you're not hovering over an point/bar on the chart.
We're trying to implement a feature where as you move the mouse across the chart, a text span underneath the chart is updated with statistics.
Version & Environment [版本及环境]

ECharts version [ECharts 版本]:
Browser version [浏览器类型和版本]:
OS Version [操作系统类型和版本]:

Expected behaviour [期望结果]
ECharts option [ECharts配置项]

option = {

}

Other comments [其他信息]

+1 forthis feature

👍

+1 - would be nice to have

+1


Long running tasks get HTTP 401 when using the UAA for authentication

I'm using the UAA for authentication with the Director. The default configuration proposed at bosh.io sets the lifetime of the token to 60s.
When executing long-running tasks that take longer than the token lifetime, the token seems to expire before the task is completed. Maybe it doesn't use the refresh-token to obtain a new token?
Steps to reproduce:

Deploy the Director using bosh-init with UAA user management as described on bosh.io
git clone cf-release
bosh upload release releases/cf-222.yml. This takes a long time to build the release tarball. Afterwards, the call to /releases fails:

...
Uploading release
release.tgz:   100% |ooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooo|   3.4GB  10.6MB/s Time: 00:05:31
HTTP 401: Not authorized: '/releases'

Smaller releases work. Setting the access_token_validity to some higher value resolves this problem as well:
uaac token client get admin
uaac client update bosh_cli --access_token_validity 600

Seems like the time between the cli making the call and passing the token and the actual call to /releases is relevant.

Problem here is that because upload to nginx takes so long (for large releases), by the time token makes it into the Director it has already expired. There is no good way to fix this unless uploading is separated into another step so that it returns a token that never expires. I think for now we should just set the access token validity to some reasonable time that is greater than maximum upload time.
docs-bosh PR?

Updated https://github.com/cloudfoundry/docs-bosh/commit/8ef20b2a0dd81e0919b68fe333886949c72e07b0.
Closing in favor of recommended solution to increase access token validity.


Added Notes Oproxy Docs

Preview Site Available
Congratulations! The automatic build has completed successfully.
A preview site is available at: https://xsoar-pan-dev--pull-request-1663-j9317vc3.web.app

Important: Make sure to inspect your changes at the preview site.
Detected modified urls:

OProxy

Preview Site Available
Congratulations! The automatic build has completed successfully.
A preview site is available at: https://xsoar-pan-dev--pull-request-1663-j9317vc3.web.app

Important: Make sure to inspect your changes at the preview site.
Detected modified urls:

OProxy


md-flex-small ignored when md-flex-medium is defined

Hello,
For example:
<md-layout md-flex-xsmall="100" md-flex-small="50" md-flex-medium="30"></md-layout>

The xsmall and medium parameters are applied, but not the small one. It is because the class md-flex-small-50 does not take precedence over mx-flex-medium-30. The order of the rules is wrong in vue-material.css.
The weird part is that the order of includes is correct in mdLayout.scss but it is not preserved in the css output ("small" is moved before the other ones). So I guess it might be a bug with node-sass.

+1

I have a similar issue where in:
<md-layout md-flex="10" md-flex-offset-large="10" md-flex-offset-xlarge="10">

the md-flex-offset-large gets applied to all smaller sizes as well. It doesn't assume 0 for other sizes. There's no way to "disable" the offset for medium sized screen for example.
I tried md-flex-offset-medium= false and 0 to no avail

Closing this issue as our focus is on the new 1.0.0 version.


chore: fix stylua's inline hint

This allows formatting the full-file  with null-ls without issues

ty, noticed that this comment only works on my work machine today for some reason @_@ truly mysterious

ty, noticed that this comment only works on my work machine today for some reason @_@ truly mysterious

check:
stylua --version

It might've been a change since a recent version or something.


316 type alignment

Fixes #316 Screenshots in comments
Creator

[ ] Build relevant tests if any apply
[ ] Ensure there are no new warnings
[ ] Upload an animated GIF showcasing the solution (if it applies)
[ ] Set a relevant reviewer

Reviewer

[ ] Run test and make sure all tests pass
[ ] Review function and form on iOS
[ ] Review function and form on Android
[ ] Review function and form on Web
[ ] Review new test logic

Before

After

Before

After


E2e test for vwan network connection

Overview/Summary
Include E2E test steps for vWan network connection described in 104479
This PR fixes/adds/changes/removes

Add step to check for changes in the infra-as-code/bicep/modules/vnetPeeringVwan path excluding *.md and *.png files.
Update condition to deploy spoke Vnet if there are changes to infra-as-code/bicep/modules/vnetPeeringVwan
Add step to deploy Vwan Network connection to spoke vnet.

Breaking Changes
N/A
Testing Evidence
TBC
As part of this Pull Request I have

[x] Checked for duplicate Pull Requests
[x] Associated it with relevant ADO items
[x] Ensured my code/branch is up-to-date with the latest changes in the main branch
[ ] Performed testing and provided evidence.


FSPageManager default buffering considered harmful

Before I forget, I looked into the performance characteristics quite a lot and one easy and rather impactful improvement was to change the FileSystemPageManager buffer defaults.
Right now, it defaults to writeBufferSize * PageSize, where writeBufferSize defaults to 100, or worse 1000 when constructed via StorageFactory with caching enabled and default cache settings.
https://github.com/VictorScherbakov/DataTanker/blob/master/DataTanker/Core/PageManagement/FileSystemPageManager.cs#L727
https://github.com/VictorScherbakov/DataTanker/blob/master/DataTanker/Core/StorageFactory.cs#L337
i.e. given a default 4096 page size, the usual case is 1000 * 4096 = 4096000 passed to the FileStream as the preferred buffer size.
Given that page access is extremely random during normal operation (as is a result of B+/Radix trees), the probability that two successive reads are close enough to each other that the seek to the next requested page offset does not discard the file streams internal buffer (and uninterrupted by writes, which would also discard writes) is extremely low. That means most of the time (almost always for a lot of work loads) the file stream buffer is discarded after being used to read only a single page. But the buffer was filled with 1000 pages, 999 of which are never accessed before being discard, creating a lot of useless disk I/O.
I have a real world program that was read-only at 65k page size
Unmodified:

678 FetchPages
676 disk I/O ReadFiles of 65536000 bytes each, meaning 2 reads were served from the buffer which is a hit rate of 0.3%.
Total disk reads:<PHONE_NUMBER>0 bytes = 41.26 GiB
ufff (thanks to the OS disk cache, it wasn't this many hard reads, thankfully).

Modified to use writeBufferSize = 1:

678 FetchPages,
678 disk I/O ReadFiles of 65536 bytes each (obviously)
Total disk reads: 44433408 bytes = 0.04 GiB
Now that's more like it.

The difference is a whooping 41 GiB in ReadFile requests to the OS. Of course, the larger the page size, the worse it gets.
Conclusion
Given the low probability that the file streams buffer will be useful, I recommend to chose writeBufferSize = 1 for all defaults.

Looks like a very nice improvement.
Yep... in normal cases, there is no need to set up a FileStream with buffer size larger than the page size.
Even if we don’t take into account the FileStream allocates an additional buffer...
It seems that a bufferSized chunk of a file is read in any way (unless we reach EOF), even if one byte is requested.
https://github.com/microsoft/referencesource/blob/master/mscorlib/system/io/filestream.cs#L1526
That's why you've seen such a bad I/O statistics:

676 disk I/O ReadFiles of 65536000 bytes each.

Many thanks for the contribution!


react 17 support

npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving: hls-player@undefined
npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/react
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/react-hls-player
npm ERR!   react-hls-player@"*" from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!
npm ERR! See /Users/ibrahimabushammah/.npm/eresolve-report.txt for a full report.
npm ERR!      /Users/ibrahimabushammah/.npm/_logs/2021-11-12T17_24_37_235Z-debug.log

getting this too. F

Same issue here. Any thouhts?

this is more than likely because it's setup to only support this you can either change your node version or you can attempt to update the peerDepedencies to support the latest version
"peerDependencies": {
    "react": "^16.13.1",
    "react-dom": "^16.13.1"
  },

try again after execute this code
npm config set legacy-peer-deps true


MixedVariableMutation crashes in scipy==1.7.0

Below is a minimal implementation that reproduces the bug, the code works well in scipy==1.6.3, bug crashes with ValueError: Unsupported dtype object in scipy==1.7.0, I'm using pymoo==4.2.2
import numpy as np
from pymoo.model.problem import Problem
from pymoo.optimize import minimize
from pymoo.factory import get_algorithm
from pymoo.operators.mixed_variable_operator import MixedVariableMutation
from pymoo.factory import get_mutation

class MyProblem(Problem):
    def __init__(self):
        super().__init__(n_var=2,
                         n_obj=1,
                         n_constr=0,
                         xl=np.array([-2, -2]),
                         xu=np.array([2, 2]),
                         )

    def _evaluate(self, x, out, *args, **kwargs):
        out['F'] = (x ** 2).sum(axis = 1, keepdims = True)

mask = ['real', 'real']
mutation = MixedVariableMutation(mask, {
    'real' : get_mutation('real_pm', eta = 20),
    'int'  : get_mutation('int_pm',  eta = 20)
})
problem   = MyProblem()
algorithm = get_algorithm('ga', pop_size = 100, mutation = mutation)
res       = minimize(problem, algorithm, ("n_gen", 100), verbose=True, seed=1)
print(res.X)

This is the error message

Compiled modules for significant speedup can not be used!
https://pymoo.org/installation.html#installation

To disable this warning:
from pymoo.configuration import Configuration
Configuration.show_compile_hint = False

=============================================
n_gen |  n_eval |     fopt     |     favg
=============================================
    1 |     100 |  0.138459616 |  2.993702771
Traceback (most recent call last):
  File "see.py", line 29, in <module>
    res       = minimize(problem, algorithm, ("n_gen", 100), verbose=True, seed=1)
  File "/home/alaya/.local/lib/python3.8/site-packages/pymoo/optimize.py", line 85, in minimize
    res = algorithm.solve()
  File "/home/alaya/.local/lib/python3.8/site-packages/pymoo/model/algorithm.py", line 226, in solve
    self._solve(self.problem)
  File "/home/alaya/.local/lib/python3.8/site-packages/pymoo/model/algorithm.py", line 321, in _solve
    self.next()
  File "/home/alaya/.local/lib/python3.8/site-packages/pymoo/model/algorithm.py", line 246, in next
    self._next()
  File "/home/alaya/.local/lib/python3.8/site-packages/pymoo/algorithms/genetic_algorithm.py", line 93, in _next
    self.off = self.mating.do(self.problem, self.pop, self.n_offsprings, algorithm=self)
  File "/home/alaya/.local/lib/python3.8/site-packages/pymoo/model/infill.py", line 40, in do
    _off = self.eliminate_duplicates.do(_off, pop, off)
  File "/home/alaya/.local/lib/python3.8/site-packages/pymoo/model/duplicate.py", line 26, in do
    pop = pop[~self._do(pop, None, np.full(len(pop), False))]
  File "/home/alaya/.local/lib/python3.8/site-packages/pymoo/model/duplicate.py", line 75, in _do
    D = self.calc_dist(pop, other)
  File "/home/alaya/.local/lib/python3.8/site-packages/pymoo/model/duplicate.py", line 66, in calc_dist
    D = cdist(X, X)
  File "/home/alaya/.local/lib/python3.8/site-packages/pymoo/util/misc.py", line 90, in cdist
    return scipy.spatial.distance.cdist(A, B, **kwargs)
  File "/home/alaya/.local/lib/python3.8/site-packages/scipy/spatial/distance.py", line 2954, in cdist
    return cdist_fn(XA, XB, out=out, **kwargs)
ValueError: Unsupported dtype object

Thanks for reporting this issue!
Adding a cast of the array before calling scipy resolves the issue (pymoo.util.misc.cdist).
def cdist(A, B, **kwargs):
    return scipy.spatial.distance.cdist(A.astype(float), B.astype(float), **kwargs)

Could you update pymoo release for this issue so we do not have to modify the source code?
Thank you
Jenny

I am planning to create a bigger release soon. I am about to automate the release process to update such fixes quicker.
Currently, it is a time-consumer process because of all the compiled packages for Windows and Mac for different Python version.
Until the next release please use the code below which basically is the same fix without changing any code in the framework.
import numpy as np
from pymoo.model.duplicate import DefaultDuplicateElimination
from pymoo.model.problem import Problem
from pymoo.optimize import minimize
from pymoo.factory import get_algorithm
from pymoo.operators.mixed_variable_operator import MixedVariableMutation
from pymoo.factory import get_mutation
from scipy.spatial.distance import cdist

class MyDuplicatedElimination(DefaultDuplicateElimination):

    def calc_dist(self, pop, other=None):
        X = self.func(pop).astype(float)

        if other is None:
            D = cdist(X, X)
            D[np.triu_indices(len(X))] = np.inf
        else:
            _X = self.func(other).astype(float)
            D = cdist(X, _X)

        return D

class MyProblem(Problem):
    def __init__(self):
        super().__init__(n_var=2,
                         n_obj=1,
                         n_constr=0,
                         xl=np.array([-2, -2]),
                         xu=np.array([2, 2]),
                         )

    def _evaluate(self, x, out, *args, **kwargs):
        out['F'] = (x ** 2).sum(axis=1, keepdims=True)

mask = ['real', 'real']
mutation = MixedVariableMutation(mask, {
    'real': get_mutation('real_pm', eta=20),
    'int': get_mutation('int_pm', eta=20)
})
problem = MyProblem()
algorithm = get_algorithm('ga', pop_size=100, mutation=mutation, eliminate_duplicates=MyDuplicatedElimination())
res = minimize(problem, algorithm, ("n_gen", 100), verbose=True, seed=1)
print(res.X)

Thank you. Looking forward to the new bigger release! 👍


Error-prone use of an array as a key-value object

Look at the function getResults starting on line 69 and how the results array is used as a key-value data structure:
function getResults(content) {
    var results = []

    for (var i = 0; i < content.length; i++) {
        var fileMatch = content[i].value;
        if (fileMatch.match !== null) {
            results[fileMatch.filename] = {
                matches: fileMatch.match,
                count: fileMatch.match.length,
                line: fileMatch.lines
            };
        }
    }

    return results;
}

Although being valid JavaScript, this seems quite unconventional and leads to weird behaviour. For example, results.length equals zero. Is this a programming error? If the author intended to preserve the order, this is not the way how to do it. Arrays are objects and the order of the keys of an object is unspecified.
I guess there are two solutions. The first is simply var results = {}. The second option would be to still use an array but instead push objects into it, with additional key filename:
results.push({
  filename: fileMatch.filename,
  matches: fileMatch.match,
  count: fileMatch.match.length,
  line: fileMatch.lines
});

Thanks for pointing this out, I agree with you here. I'll try to handle this as soon as I can. A PR would be most welcome, time is a rare resource atm :-)

Thanks for taking action! Unfortunately I am incapable to make a PR: time is precious and also the issue affects the API, that I think it is up to you to decide. Yet, to help in the decision, I believe var results = {} has the smallest probability to cause backward-incompatibility problems.

Thanks for the input – I'll try to handle it as soon as possible :-)

Oh, all hail to @kaesetoast! :)


Make tags trigger automated actions

i.e. "morse" makes it drop a morse code image in there, "crossword" makes it give wordplay's top crossword answers to clues

"drop a morse code image in there" - Is "there" the sheet or the Discord channel?
For the crossword suggestion, are you imagining a workflow where we

Create a puzzle
Fill the Google Sheet with crossword clues via copy to clipboard (or some other copy mechanism)
We tag the puzzle in Cardboard with "crossword" and then we trigger an action that reads through the clues in the sheet, makes requests to Wordplay for each clue, and fills in the top answer(s) into the sheet.

I guess sheet? I feel like that one might be less useful and we also don't have a morse code tag usually.
Another workflow I can think of is automatically setting up sheets formulas for acrostics, not sure how easy that is.

Acrostics are a pain, but I don't know if there is a common enough format for their Copy to Clipboards that would allow us to set up formulas automatically. We can make a template that would make it easier to set up via formulas, but would still require manual effort.


warn and continue in argocd login error

will not work, since argocd code exists (Fatal) on that error.
ended up making a special binary for the customer instead (that just skips argocd login call)


locationd: Unit tests for different scenarios

Add unit tests for locationd response to different scenarios like:

base
gps_off
gps_off_midway
gps_on_midway
gps_tunnel
gyro_off
gyro_spike_midway
accel_off
accel_spike_midway

FYI, this is one of the slowest tests. Do you know why?  Do you know why?  That is what takes most time. (~100 secs totally). I'll try making the logs much smaller.


Introduce standard annotations for client libraries

There is value in introducing more standard annotations to be used across all client libraries, for both static code analysis and runtime benefits. This issue is a catch-all to enumerate any idea that may be worth implementing.

[ ] @ServiceMethod for all client library methods that call out to Azure.
[ ] @Tracing should be applied on all @ServiceMethod methods, to specify the tracing operation name to be set in all spans. RestProxy can then be configured to retrieve the value from here for the operation name. It may be the case that this will need to be applied to methods within the service proxy interface, rather than the service client - which will have ramifications for the JavaDoc idea listed below.
[ ] @Builder for all client library builder classes. Could reference the class that the builder creates.
[ ] @ServiceClient for all client classes.
[ ] @Fluent \ @NonFluent for all classes. This will help to ensure that we have fluent API when expected, and non-fluent API when expected.

Beyond the introduction of these annotations, we should introduce additional CheckStyle rules and custom JavaDoc doclets. Ideas include:

[ ] A JavaDoc doclet that will include the full tracing name for each method into the generated JavaDoc.

Implemented in PR #4189


Remove Name input on Section.Define

Name = $"{GroupName}, {TypeName}, {SizeName}";

[ ] Update the example on Docusaurus
[ ] Update Example 5 - Using custom sections

I have just created an example for Grasshopper regarding the custom cross section (which is really powerful) and I had some doubt as well regarding the inputs.
<section name="Concrete sections, Rectangle, 120x250" fd_name_code="Concrete sections" fd_name_type="Rectangle" fd_name_size="120x250">

I generally don't appreciate the "name" that FEM-Design is assigning to the sections but we should follow their rule (which is closer to the one you suggest)

I have changed the milestone to 21.5.0 as the feature is needed for some example/contents for 21.5.0


Include the dependency type to the Analyzer output

Currently the "scopes" section of the Analyzer YAML output provides a tree of package dependencies. Each entry consists of a package "id" and a list of "dependencies".
When using the Analyzer output for license analysis, the type of the dependency ("static link", "dynamic link" or "other") can have important implications for the license conditions. (E.g.: LGPL licensed code is only allowed in proprietary software if it is dynamically linked.)
Suggestion is to extend the existing elements in the "scopes" tree with a new field indicating their "type" of dependency on the level above, and extend all supported package manager implementations to provide this information.

That element is actually already supported, but it's only serialized if it deviates from the default of "dynamic linkage" to reduce the file size. See e.g. https://github.com/oss-review-toolkit/ort/blob/264ec63076d8d5b6afa389ed19d4c5dbbdaa2b84/analyzer/src/funTest/assets/projects/synthetic/cargo-expected-output.yml#L24.

I concluded it was not supported because for Pub packages (Flutter / Dart) the dependencies were not indicated as statically linked.
Should I report that as a separate issue?

I just did that. Please indicate in that new ticket whether you have any hints on how to find out whether a Pub package is statically or dynamically linked.


Searchinator delivering incomplete results

Searchinator is not delivering comparable results to the Family Services Directory

here is an example where a provider in the FSD data is not coming up in the Searchinator results yet it is within 300 metres of a provider that does.
How is Searchinator searching.docx

@choirboy07 We are seeing a bit of a discrepancy between the doc you've sent and what we are seeing online here.
With our searchinator - we see 6 results
On your website we are seeing - 21 results (11 local)
The document you've provided says there should be 7. Can you let us know if that's a typo or could also send through a screenshot of what you are seeing, as that will be really helpful

Hi, I have 7 in the document as they have a Gisborne City  Street address - the other 4 on the main Directory are Ruatoria addresses, which is 100+kms from Gisborne City.

@dlouise64 @JacOng17 feedback from @choirboy07 for you.

in case it's useful, I've done a comparison between Find Whanau Services (Searchinator) and Family Services Directory (FSD):
Searchinator vs FSD.xlsx
The TL;DR is: I can explain the difference in results between the two sites for 4/6 results, but there are 2 results where I can't explain why they aren't showing in Searchinator.
Those results are for:
Provider 7292 Open Home Foundation (OHF) - Wairarapa, and
Provider 5303 Rangitane O Wairarapa Incorporated

I have done some testing and issue appears to be where the physical address has no numbers (Street or Post code) that can be reference to match the location.  This appears to be for any physical address and not just those that start with Cnr - see last entry on spreadsheet. This issue also relates to card 201.   Note I am not sure why Stand Children's Services for Manukau is not coming up in keyword search only.
Searchinator Location Searching .xlsx

@lamorrison  the summary of what we've found when looking specifically for  Open Home Foundation (OHF) - Wairarapa (provider 7292) is:

Enter topic or organisation: [ open home ] - result set includes the provider

Enter address [ Masterton ] - result set includes the provider

Enter address [ 2 Akura Road, Masterton 5810] - result set includes the provider

Enter address [340 Queen Street, Masterton 5810] - result set DOES NOT include the provider

So the address search can identify that Open Home Wairarapa is in Masterton, and it can identify it being close to another street in Masterton, but for some reason it can't identify that it's in close proximity to Queen Street Masterton.
Not sure what the likely cause is, but in case it's helpful... !

I have made a separate card for the bug relating to Stand Children's Services, #216

Hi @sarahneeson , I tried the search you mentioned for OHF Wairarapa today and it seems to be working for me. Could you confirm if it's still missing from the 340 Queen Street search for you?

Hi @lamorrison  - Yep, it's still missing.  Apologies, I should have specified it was in combination with the category Parents and Caregivers.
Here's the search where it doesn't appear:
https://serviceinnovationlab.github.io/FSD/#/FSD/?address=340 Queen Street%2C Masterton 5810&category=Parents and Caregivers&latitude=-40.9540232833&longitude=175.65468015&radius=50

Hi, I've attached a table to show search outcomes for the same search constructs using 340 Queen Street and then Masterton.  I haven't figured out the why yet but it does highlight differences in results between the 2 "Addresses".
Open Home Searches Table.xlsx

@lamorrison @JacOng17 I bumped the limit to 500 now I can see "Open Home Foundation (OHF) - Wairarapa". Ready for dev testing.

Two of the previous searches that weren't showing Open Home Foundation (OHF) - Wairarapa in the results are now showing that provider as expected.
I still have a question about the placement of this provider in one of the searches, but will raise this question in a separate issue.


[coverage] Return bool instead of using OT_UNREACHABLE()

          Isn't this reserved for truly impossible branches? The documentation states that behavior is undefined if control reaches this point — I'm not sure if this means the entire branch, including LOG_ERROR is now undefined.

Originally posted by @dmcardle in https://github.com/lowRISC/opentitan/pull/18745#discussion_r1205988422

CC @cfrantz @timothytrippel

Should this be fixed for PROD?

I think this is a minor cleanup we can ignore for the time-being and revisit if we have time in the future.


My Profile: unable to fill out any form

I'm trying to use the input masks including Twitter Handle, etc. but both with Google Chrome and Firefox, I am unable to type anything (no characters are appearing in the form).
Steps to reproduce: just login to fever dreams app. Click on "My Profile", start typing.
If I inspect e.g. the about element (id="bio" class="chakra-textarea") [event] says invalid (main.b338ac5d.js:1:215515)

Fixed in https://github.com/entmike/fd-web-ui/pull/31


Field data is not removed when a Field has been unmounted

Are you submitting a bug report or a feature request?
Action @@redux-form/UNREGISTER_FIELD doesn't delete Field value when a component is unmounted. The problem this causes is that if you have conditional fields, the data is being submitted from fields that have been unmounted.
What is the current behavior?
Field data is kept after unmounting.
What is the expected behavior?
Field data should be removed after unmounting has happened. This is the same behavior as what would happen when redux-form is not being used on a form.
Sandbox Link
https://codesandbox.io/s/PZ19V5Pwy
What's your environment?
Tested with 6.8.0 and 6.3.0
Other information

This is intentional for the Wizard forms use case, where you want to build up your form state across pages, or across field changes.  Even with conditional fields it is a valid use case to want the data to return if the field is re-enabled, so outright scrapping the value is not really an option.
Since we can't call this a bug, I'm guessing it's a feature request, but I'm not exactly sure what the appropriate feature requested would be here.
If it's an issue of data submission exclusion, this is something that can be solved in user land, by managing the same relevancy rules applied to your form elements inside the submission function. Likely there are libraries for managing relevancy rules like this, even if not, I think it's not something that should live in redux-form itself.
If it's an issue of nonetheless wanting to be able to force a value to be erased on deregistering, a) do you have another use case than exclusion, and b) if so, it would be interesting to have something similar like a destroyOnUnregister which applies at the Field level.
Perhaps you could clarify your needs before I go wandering off into abstract possibilities!

Closing this due to non-response, and also it's essentially a duplicate of #2761.


Check the validity and update the specs for the core types

We have section of our specification spec/core describing the core types adopted in CometBFT. While some minor changes have been made recently, the documentation itself appears to need a thorough review, for the sake of accuracy and possibly also completeness.
In particular, some validity predicates appear to be imprecise, for instance the restrictions regarding round numbers (https://github.com/tendermint/tendermint/issues/10041), that probably also apply to other int fields (e.g., validator index) that are mistakenly restricted to be >0 when the 0 value is valid.
I also wonder whether there is a better way to keep this documentation consistent with the implementation (namely, the referenced data types) without manually editing Markdown tables.

Could you hook this issue into #23 ?


pip is not recognised

Type: Bug
Hi, please I am a novice at programming and will appreciate any help I can get to sort this issue. I have checked stackoverflow but I can't seem to find the help that can sort this issue for me specifically.
PS C:\Users\hp laptop> pip
pip : The term 'pip' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was
included, verify that the path is correct and try again.
At line:1 char:1

pip

  + CategoryInfo          : ObjectNotFound: (pip:String) [], CommandNotFoundException
  + FullyQualifiedErrorId : CommandNotFoundException

VS Code version: Code 1.77.0 (7f329fe6c66b0f86ae1574c2911b681ad5a45d63, 2023-03-29T10:02:16.981Z)
OS version: Windows_NT x64 10.0.19044
Modes:
Sandboxed: Yes

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-6300U CPU @ 2.40GHz (4 x 2496)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
undefined

Memory (System)
7.88GB (1.20GB free)

Process Argv
--crash-reporter-id 79e84415-8a2c-4b28-856b-c5fedab112ac

Screen Reader
no

VM
0%

Extensions (10)

Extension
Author (truncated)
Version

chatgpt-vscode
gen
0.0.7

vsc-python-indent
Kev
1.18.0

chatgptplus
mol
2.0.0

python
ms-
2023.6.0

vscode-pylance
ms-
2023.3.40

jupyter
ms-
2023.3.1000892223

jupyter-keymap
ms-
1.1.0

jupyter-renderers
ms-
1.0.15

vscode-jupyter-cell-tags
ms-
0.1.8

vscode-jupyter-slideshow
ms-
0.1.5

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492:30256859
vslsvsres303:30308271
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vsdfh931:30280409
vshan820:30294714
vstes263cf:30335440
vscoreces:30445986
pythondataviewer:30285071
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593:30376534
pythonvs932:30410667
cppdebug:30492333
vscaat:30438848
vsclangdf:30486550
c4g48928:30535728
dsvsc012:30540252
pynewext54:30695312
azure-dev_surveyone:30548225
vscccc:30610679
nodejswelcome1cf:30587006
3biah626:30602489
pyind779:30671433
89544117:30613380
pythonsymbol12:30671437
a9j8j154:30646983
vsctsb:30677850
azdwalk:30687957
pythonms35:30701012

Install pip then add the path to the system variable as:

1.Just install python in your pc, pip will automatically be downloaded and you can easily use it.
2 If you have installed pip then just add its path to system environment variables i think it will work.
First method will work definitely.


Addthis is not defined

When starting index.html, the console shows that addthis is undefined.
Uncaught ReferenceError: addthis is not defined:
jquery.min.js: 16

Yes. Need to troubleshoot. This happens after recent merge with PR#10
Sent from my iPhone

On Jan 3, 2016, at 11:08 AM, Omar Shaikh<EMAIL_ADDRESS>wrote:
When starting index.html, the console shows that addthis is undefined.
Uncaught ReferenceError: addthis is not defined:
jquery.min.js: 16
—
Reply to this email directly or view it on GitHub.

Please see why it is not working after yesterday's merge.

Is this issue fixed now?

Resolved in #14


Refresh OpenSearch nodes version in cluster state after upgrade

Description
This the draft PR for fixing the issue https://github.com/opensearch-project/OpenSearch/issues/827
Fix:

As soon as OpenSearch node becomes master, it refreshes the nodes version in cluster state by comparing against their channel version. Right now, it doesn't wait for all Elasticsearch nodes to leave the cluster before refreshing the version.
There was bug in DiscoveryNode BWC serialization logic where any ES version < 7.10.2 was getting over-written with 7.10.2

Pending unit test & integs (In Progress), raised it to get early feedback.
Issues Resolved
https://github.com/opensearch-project/OpenSearch/issues/827
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

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

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

I like it.

+1 on the approach. How close are you on Integration and Unit tests? Not sure this will make it for GA so we probably want to get it out in a bugfix relatively quickly.

@shwetathareja Any update on getting Integration and Unit tests done?  Do you think you could get this in for GA? If not we will need to get a bugfix release out pretty quickly after GA.

@shwetathareja Hey -- we're running out of runway for 1.0.0 and this seems like something we'd want to get in.  Can you get your tests in?

@nknize @CEHENKLE sorry for the delay, working on it, will raise it in next couple of hours.

Thanks, @shwetathareja !  I'll look for it in our AM :)

start gradle check

start gradle check

@nknize @adnapibar @dblock please review.

start gradle check

start gradle check

The changes look good but I see the a few bwc tests for mixed cluster are failing (these pass in main)
./gradlew :qa:mixed-cluster:v7.10.2#mixedClusterTest

Tests with failures:
 - org.opensearch.backwards.MixedClusterClientYamlTestSuiteIT.test {p0=cat.health/10_basic/Empty cluster}
 - org.opensearch.backwards.MixedClusterClientYamlTestSuiteIT.test {p0=cat.health/10_basic/Help}
 - org.opensearch.backwards.MixedClusterClientYamlTestSuiteIT.test {p0=cat.health/10_basic/With ts parameter}

The changes look good but I see the a few bwc tests for mixed cluster are failing (these pass in main)
./gradlew :qa:mixed-cluster:v7.10.2#mixedClusterTest

Tests with failures:
 - org.opensearch.backwards.MixedClusterClientYamlTestSuiteIT.test {p0=cat.health/10_basic/Empty cluster}
 - org.opensearch.backwards.MixedClusterClientYamlTestSuiteIT.test {p0=cat.health/10_basic/Help}
 - org.opensearch.backwards.MixedClusterClientYamlTestSuiteIT.test {p0=cat.health/10_basic/With ts parameter}

Looks like the BWC tests have become flaky and I can also see these in the main branch.  I have created an issue to investigate and resolve these separately https://github.com/opensearch-project/OpenSearch/issues/922

@shwetathareja Heya Shweta -- we're holding the build for this fix, so if you could fix your DCO, we're going slam the door on 1.0.0 behind you.  So if you can fix that and merge when you get in, we should be good to go in our AM :)
Thanks,
/C

start gradle check


Math Typo on Qiskit Pulse Documentation

On the Qiskit documentation for Introduction to Transmon Physics, in the section on Qubit Drive, one of the terms has a sign swapped. Specifically, I think the raising operator in the underlined portion of the below image is supposed to be a lowering operator such that the equation makes sense.

Not a big issue, but it'd be nice to correct it!

I think this issue is resolved, please close this issue.

This repository is no longer maintained. It has been superseded by IBM Quantum Learning.


Search for struct body span after any generic arguments

Fixes #5273
Previously, rustfmt searched for the start of a struct body after the opening {. In most cases this works just fine, but const values can also be defined inside { }, which lead to issues when rewriting the struct body (as highlighted by the linked issue).
Now, rustfmt will search for the { after the generic argument list to guarantee that the { it finds is the start of the struct body.

@ytmimi re: dialog #4220, I'd think ideally we'd do what you have here but use the span of the where clause instead as opposed to doing a full span search (where clause should be available as a field on the generics iirc)

Yup the where clause is is available on the Generics. I'm surprised that the span on the struct doesn't encompass the where clause as well.

Yup the where clause is is available on the Generics. I'm surprised that the span on the struct doesn't encompass the where clause as well.

I'm surprised that the span on the struct doesn't encompass the where clause as well.

I know it seems odd at first blush for the span of the containing node to not extend to the end of the span of one its child nodes, but remember with certain item definitions there can be an awful lot of non-generics content between the end of the generics and the start of the where clause, which is why the spans are structured they way that they are


SyntaxError: Cannot use import statement outside a module

Using fresh install of recoil-nexus v0.3.10

Hi there looks weird, which version of Recoil? Is your project js/ts?
Any info to reproduce is helpful
L

I'm using Typescript with Recoil v0.5.2 and this is what I have done:

Install with npm i recoil-nexus
Add RecoilNexus under RecoilRoot

...
<RecoilRoot>
    <RecoilNexus />
    ...

Use setRecoil

import { setRecoil } from 'recoil-nexus'
//...
setRecoil(currentAtom, currentValue)

and after that I'm getting the SyntaxError

It works now, steps to solve:

Change "module" property in the recoil-nexus tsconfig.json from "esnext" to "commonjs"
Run npm run build


Farming Station planting wrong seeds

Issue Description:
The Ender IO Farming Station is deploing the wrong seeds if u use Barley seeds from the Natura Mod.
instead of Barley it's planting Cotton seeds from the Natura Mod but it's consuming the Barley seeds.
What happens:
Farming Station using Barley seeds but planting Cotton
What you expected to happen:
Farming Station using Barley seeds to plant Barley
Steps to reproduce:
1.Setup Farming Station
2.Put in Barley seeds
3.Watch
...

Affected Versions (Do not use "latest"):

EnderIO: 1.10.2-3.1.180
EnderCore: 1.10.2-<IP_ADDRESS>-beta
Minecraft: 1.10.2
Forge: <IP_ADDRESS>
-Minecraft Forge: <IP_ADDRESS>4

Your most recent log file where the issue was present:
[pastebin/gist/etc link here]

Just to clarify some things - you only put barley seed in the farming station and no other type of seed (that's what I'm going to assume).
As I currently do not have a pack with Natura can you check the item ID of these seed (by using F3-H to show debug tooltips).
I assume that both seeds share the same item ID with a different meta data.

Can anyone confirm the data with Natura? I have:

grown barley block: "Natura:N Crops" meta=3
grown cotton block: "Natura:N Crops" meta=8
freshly planted cotton block: "Natura:N Crops" meta=4
barley seed item: "Natura:barley.seed" meta=0
cotton seed item: "Natura:barley.seed" meta=1

current Pack im using is FTB Beyond 1.5.3
Natura Version is: 1.10.2-<IP_ADDRESS>
Barley seeds ID: (#7711/0) 7711
Cotton seed ID: (#7711/1) 7711:1
tile.natura.barley_crop.name (#7709/0)
tile.natura.cotton_crop.name (#7710/0)
Block ID shown with AgriCraft Debugger Tool (#4634)
Planted Barley block ID: natura:barley_crop Meta: 0
Planted Cotton block ID: natura:cotton_crop Meta: 0

I can confirm the naming Chelebi pointed out for the blocks, the seeds have the ID "natura:overworld_seeds".
Not sure if this is best put here or if it warrants an issue of its own, but besides the barley/cotton thing, Natura berry bushes also aren't handled by the Farming Station correctly. because they were renamed since 1.7.10. The new naming scheme is "natura:(overworld|nether)berrybush[berryname]" as far as I can tell, each berry being a separate block/item.

2 things:

Natura changed its mod-id, so our Natura code is not engaged at all. So all Natura seeds and plants are handled by the vanilla code using whatever vanilla interfaces those items/blocks implement. So that both seeds generate the same kind of plant is a bug in Natura and should be reported there.
New Farming Station code for Natura is in progress.

Ok, I'm through with the new Natura integration. However, I cannot recommend Natura at all at the moment---too many bugs there.

Thanks a lot for the quick change! IMO, this issue can be considered closed.
As a sidenote, I'm only using Natura berries since they are a great source of food when combined with the HarvestCraft juicer :D


Error at startup: 403 invalid client secret

Just installed via HACS and created a client_id and generated an oauth token. Same result if I remove secrets and enter client and token directly into the config. I've tried generating a new auth with no change.
  - platform: twitch_helix
    client_id: !secret twitch_client_id
    client_secret: !secret twitch_client_secret
    own_channel_id: !secret twitch_channel_id
    channel_ids:
      - "502430815" # desertheartsrecords
      - "436417492" # PykaMusic
      - "504331394" # hotdubtimemachine
      - "408679510" # jessu
      - "535661629" # cool_guy_club
      - "646868791" # djnewgirl
      - "503461554" # thefunkhunters
      - "513403459" # ShintaLovesHouseMusic

Logger: homeassistant.components.sensor
Source: custom_components/twitch_helix/sensor.py:52
Integration: Sensor (documentation, issues)
First occurred: 4:53:31 PM (1 occurrences)
Last logged: 4:53:31 PM

Error while setting up twitch_helix platform for sensor
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 249, in _async_setup_platform
    await asyncio.shield(task)
  File "/usr/local/lib/python3.9/concurrent/futures/thread.py", line 52, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/config/custom_components/twitch_helix/sensor.py", line 52, in setup_platform
    client = Twitch(client_id, client_secret, target_app_auth_scope=scopes)
  File "/usr/local/lib/python3.9/site-packages/twitchAPI/twitch.py", line 168, in __init__
    self.authenticate_app(target_app_auth_scope if target_app_auth_scope is not None else [])
  File "/usr/local/lib/python3.9/site-packages/twitchAPI/twitch.py", line 424, in authenticate_app
    self.__generate_app_token()
  File "/usr/local/lib/python3.9/site-packages/twitchAPI/twitch.py", line 407, in __generate_app_token
    raise TwitchAuthorizationException(f'Authentication failed with code {result.status_code} ({result.text})')
twitchAPI.types.TwitchAuthorizationException: Authentication failed with code 403 ({"status":403,"message":"invalid client secret"}
)

Hi! I've just tried on my setup with the same config using my own secrets and it seems to work.
The error message seems to indicate the client secret is wrong.
How did you setup the Twitch app?
This is an example of how mine looks like (sorry it's on Danish, but should like kinda the same):
Remember to hit the "save" button at the bottom of the app.


Add support for .avax domains.

This PR integrates .avax forward & reverse resolution into revoke.cash.
Included Functionality

In the Header SearchBar component, if the user enters a string that terminates with .avax, the app attempts to forward resolve the string into an 0x address.
When visiting /address/name.avax, the app attempts to forward resolve name.avax into an 0x address and load the wallet details.
When viewing a wallet, the app will attempt to reverse-resolve the 0x address into a .avax domain name. If reverse resolution is successful, the domain name gets displayed.
After connecting a wallet, the app will attempt to reverse-resolve the connected 0x address into a .avax domain name. If reverse resolution is successful, the .avax domain name gets displayed.

Omitted Functionality
This PR does not reverse resolve 0x addresses found in the Authorized Spender column in the Allowances table as we did not find any evidence of reverse resolution for ENS / UNS being performed on these columns. However, if it is acceptable, we can optionally add an operation to attempt to reverse resolve all of these addresses to .avax names in a single RPC call.
Avvy Client Configuration
The Avvy Domains client relies on RPC calls to an Avalanche C-Chain RPC. This PR includes a hard-coded URL for the RPC. We would like to include this value in the .env, but wanted to first consult Revoke.Cash to determine the best way to allow user configuration of the RPC.

Thanks for the PR. I will slightly reword the search bar text after merging since it's getting a bit long if I keep adding other domain name systems to the search bar, so I'll replace the search bar placeholder with "Search accounts by address or domain". Then I'll add a bit to the FAQ that lists all the supported domain name systems.

I'm trying to merge origin/master into this branch so that the PR can be merged, but I'm getting a permission denied. Did you check the "Allow changed from maintainers" checkbox for this PR?

I can't find the "Allow changes from maintainer" toggle. Have instead merged master from RevokeCash/revoke.cash and have also added you as a writer on avvydomains/revoke.cash in case anything else is necessary. @rkalis


conda installation

Hi,
I'm trying to install spyder-vim with latest conda installation version 4.3.25, but it doesn't seems to work.
Option 1: conda install spyder-vim -c spyder-ide
-> This says that the package is missing in the repository
Option 2: pip install spyder-vim
-> This seems to install, but then I see no option (or different behaviour) in Spyder.
Any ideas how to solve it?
Thanks!
Cheers,
Hendrik

This package has not been released yet. That's why there are no conda nor pip packages available, sorry.

When are you planning to release the package? I'm sure lots of people like me are eager to use it!

@dcasasor @hendrikwout we will try to make a release before the end of the year

Hi there. Although, a package is not available, is it possible to install this plugin from source on conda? I tried the following, but it didn't seem to work:

Cloned spyder-vim repository
Copied spyder_vim folder to anaconda3/pkgs/spyder-4.0.1-py37_0/lib/python3.7/site-packages/spyder/plugins/
Restarted Spyder.

Not sure if this process is correct for Conda users? Hope someone can advise.

Hi all. I installed Spyder on Windows with the standalone installer. I then cloned and pip installed spyder-vim in the virtual environment whose python my Spyder points to. If I open Spyder directly then I do not see the vim command line bar. If I open Spyder from within the gitbash terminal, I can see the vim commandline that seems to work but then Spyder segfaults within 30 secs every time. Is there a workaround for this?

@kcmeehan, our installers are completely self-contained, so our third-party plugins can't be installed in them. We plan to fix that limitation later this year.

Hope the conda package can be released eventually! My org only allows conda packages internally. Thanks for the good work.

Closing as duplicate of #22
To follow up the progress towards the initial release for this plugin/package you can check #78


Please store checkbox values as a bit

Usage of keywords to store bit values is not recommended. LAConfigs is storing values as on and off and in other tables we have enabled and disabled.
To be consistent it would be much better to store it as a bit 0 or 1 and in the interface you might you enums to translate the setting

Just changed these in the views and the seeder, if interested, please check my fork end of the day to pull it
thnx

@mgazelle, LAConfigs is not just limited to Checkboxes.
Having value in on and off won't affect anything.
Regards,

That is not the point. I know it is not limited to Checkboxes but saving the checkbox value as on and off was my comment.

On 15 Oct 2016, at 19:18, Ganesh Bhosale<EMAIL_ADDRESS>wrote:
@mgazelle https://github.com/mgazelle, LAConfigs is not just limited to Checkboxes.
Having value in on and off won't affect anything.
Regards,
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/dwijitsolutions/laraadmin/issues/70#issuecomment-253997709, or mute the thread https://github.com/notifications/unsubscribe-auth/AFPCHJgmQBe4qH5dkKToVRe1ORzqpwAIks5q0QregaJpZM4KXpeM.

Does it matter ?


Problem540

Pull Request Template
What have you Changed(must)
Added Java implementation for problem 540 - Single Element in a Sorted Array
Issue no.(must) - #750
Pr will be close and marked as spam. If issue number not found or issue was assigned to someone else.
Marking as spam can block your account from HacktoberFest.
Self Check(Tick After Making pull Request)

[x] This issue was assigned to me if not it will be marked as spam.
[x] My file is in the proper folder
[x] I am following clean code and Documentation.
[x] I have added the title and what program will do first in the file
[x] I have named the file as (Problem #). (Problem Title(in Camel Case)).(extension)
ex. 2. AddToNumbers.py

README - How to Contribute

Oops! Just realized that I used my secondary GitHub account, grabinow2, instead of the account that was assigned this issue to commit and push. I hope this doesn't invalidate my contribution.


input bam files refinements

Hi.
In some SNP discovery protocols it is suggested to perform pcr duplicate labeling and base quality score recalibration if possible. Would you also recommend this before running dysgu?
As for pcr duplicates, in case, is tagging enough or is removal better?

Thanks a lot @kcleal !


fix: should not stringify loader loading requests

Summary
Introduced in https://github.com/web-infra-dev/rspack/pull/7623
Should not JSON.stringify again as eval is removed.

Checklist

[ ] Tests updated (or not required).
[ ] Documentation updated (or not required).

TODO: add node20 esm-loader test :D

Should enable test for Node 20: packages/rspack-test-tools/tests/configCases/loader/type-module/test.filter.js


Update install guide with driver apply info

At least on Windows 10 Version 2004 build 19041.746, it is possible to enable the driver just by re-plugging the mouse and no restart is required at all.

nice find, i always thought it was loaded on boot
one thing that bothers me is that it's not unloaded dynamically, for example:

uninstall version A and dc all mice—communication with A is still possible (gui loads fine)
then install version B and connect device—it runs on A until reboot

so i think it's simpler to leave this unmentioned, or if we do add it, also add a note for reinstall


Missing available STT systems in docs

In the online docs are missing available languages in STT section. The missing ones:
Kaldi: Italian, Spanish, Russian
Deepspeech: French, Spanish, Italian

Thanks! I'm updating this now :+1:


partial fix chef_client_schedule_task

Description
This PR  is a partial fix for making chef_client_schedule_task resource idempotent. Testing is remaining.
Issues Resolved
Partial fixes https://github.com/chef-cookbooks/chef-client/issues/490

@tas50 , @btm, while trying to make frequency: monthly idempotent, we came across some issues.
If frequency_modifier: 4 is set with frequency :monthly, the schedule task sets months: Apr, Aug, Dec in the XML.
So in order to set the current_resource.frequency_modifier, we can use some formula like 12 / months_arr.length.
But there's also an attribute called months which also sets the same XML months element.
Suppose the user writes a recipe:
windows_task "my_task" do
  frequency   :monthly
  months       "Jan, Mar"
end

This will also set current_resource.frequency_modifier if we apply the logic that I mentioned above.
We cannot be sure whether the existing scheduled task was created by passing frequency_modifier attribute or months attribute in case of frequency :monthly.
It's difficult to make the resource idempotent in this case.
The solution to this problem can be to allow setting only one attribute for frequency :monthly... either frequency_modifier or months.

I'm closing this out since this bandaids the fix in a way that I'm uncomfortable doing and also switches from resource driven to cookbook driven config logic, which we're trying to avoid.


rpm spec mode fixes

When developing an rpm syntax highlighting plugin for Brackets I made a number of fixes to the rpm spec mode.

CSS classes names didn't exist. Updated to use classes that do exist.
Add support for more architectures, preambles and sections.
Fix clearing of control flow state.

Thanks. Merged as 4f7a6fb