Scalar returning function's weird (not backward-compatible) response

Definition like the following
CREATE FUNCTION postgrest_foo()  returns text as $$ SELECT 'bar'::text $$ LANGUAGE SQL
that used to be returning in version 3.2 response like this
[{"postgrest_foo":"bar"}]
in version 4.* emits just
"bar"
Is that behavior by design or some little lapse while dealing with corner cases?
Tested on
PostgreSQL 9.6.4 on x86_64-pc-linux-gnu, compiled by gcc (Debian 6.3.0-18) 6.3.0 20170516, 64-bit

by design, "bar" is valid json

So, the issue is response compatibility - not json validity.
CREATE OR REPLACE FUNCTION postgrest_fooset(single_first boolean)
  RETURNS setof text
AS $$
	SELECT foo
	FROM (
		SELECT *, row_number() over() AS rn
		FROM ( VALUES ('bar'::text), ('bazz'::text) ) foo(foo)
	) foo
	WHERE NOT single_first OR rn = 1
$$ LANGUAGE SQL

Apparently, this sequence of interactions is by design too
POST /rpc/postgrest_fooset -d {"single_first":false}
HTTP/1.1 200
[
    {
        "postgrest_fooset": "bar"
    },
    {
        "postgrest_fooset": "bazz"
    }
]

POST /rpc/postgrest_fooset -d {"single_first":true}
HTTP/1.1 200
"bar"

?

the behaviour for the first function is correct, it returns a string, thus you get a string.
for the section function it looks like it not very good, i guess for it it should have been a array of strings.

On master branch:
POST /rpc/postgrest_fooset -d {"single_first":false}
HTTP/1.1 200
[
    {
        "postgrest_fooset": "bar"
    },
    {
        "postgrest_fooset": "bazz"
    }
]

POST /rpc/postgrest_fooset -d {"single_first":true}
HTTP/1.1 200
[
    {
        "postgrest_fooset": "bar"
    }
]

If on master branch the topic-starting function yields
[ { "postgrest_foo": "bar" } ]

then the issue should be closed.

The problem, possibly, remains.
CREATE FUNCTION postgrest_foo() returns text as $$ SELECT 'bar'::text $$ LANGUAGE SQL

docker start pg_rest4_4
curl -i -X POST localhost:3000/rpc/postgrest_foo

HTTP/1.1 200 OK
Transfer-Encoding: chunked
Date: Thu, 19 Apr 2018 13:17:07 GMT
Server: postgrest/<IP_ADDRESS> (f9e770b)
Content-Type: application/json; charset=utf-8
Content-Range: 0-0/*

"bar"

curl -i -X POST localhost:3000/rpc/postgrest_foo2

HTTP/1.1 404 Not Found
Transfer-Encoding: chunked
Date: Thu, 19 Apr 2018 13:17:15 GMT
Server: postgrest/<IP_ADDRESS> (f9e770b)
Content-Type: application/json; charset=utf-8

{"hint":"No function matches the given name and argument types. You might need to add explicit type casts.","details":null,"code":"42883","message":"function public.postgrest_foo2() does not exist"}

CREATE FUNCTION postgrest_foo2() returns text as $$ SELECT 'bazz'::text $$ LANGUAGE SQL

curl -i -X POST localhost:3000/rpc/postgrest_foo2

HTTP/1.1 200 OK
Transfer-Encoding: chunked
Date: Thu, 19 Apr 2018 13:17:24 GMT
Server: postgrest/<IP_ADDRESS> (f9e770b)
Content-Type: application/json; charset=utf-8
Content-Range: 0-0/*

[{"postgrest_foo2":"bazz"}]

docker exec pg_rest4_4 bash -c 'kill -HUP `pgrep postgrest`'
curl -i -X POST localhost:3000/rpc/postgrest_foo2

HTTP/1.1 200 OK
Transfer-Encoding: chunked
Date: Thu, 19 Apr 2018 13:18:37 GMT
Server: postgrest/<IP_ADDRESS> (f9e770b)
Content-Type: application/json; charset=utf-8
Content-Range: 0-0/*

"bazz"

That's expected behavior, the db structured needs to be cached and refreshed if there's a change, for this to be automated in development there's now https://github.com/begriffs/pg_listen. There are instructions of how to use it in https://postgrest.com/en/v4.4/admin.html#schema-reloading.

But that means the answer to the above comment is effectively NO

To be clear.
Specification declares

By default PostgREST returns all JSON results in an array

but actual tests reveal that scalar function does not follow the spec (unless it's beyond PostgREST’s database schema cache)

That's true, we should add to the docs the RPC dependency on the db cache and warn that the restart is needed.
As it is now, there's no way to differentiate scalar from row return type in the function unless we query the db cache.
I can see how that can be a problem for development environments, that's why I added the suggestion to use the utility above, however in production environments changes are done with care and there's likely some sort of CI that can do a SIGHUP to pgrst after the db migration, so the cache is no problem for that case, it should always be refreshed on deployment.

And, also, the docs should probably specify that for now scalar function returns its result in a JSON-value, not a JSON-array (which is, i suppose, breaking change from v3.2)


Portable version?

Hello,
is this a portable version?
https://github.com/leeanchu/ROFL-Player/releases/tag/1.1
If not you can create a portable version?

That release is not technically portable since it writes some settings in the user app data folder. If you copy it to another computer some settings will reset to defaults.
The current version of ReplayBook (the successor to ROFLPlayer) is portable however.

Hello,
that's good, but where can I find the portable version for ReplayBook?
Can you add it to releases now? https://github.com/leeanchu/ReplayBook/releases

Right now it is in closed alpha, and the builds are in the Discord . Just join and ask to join the alpha.

I will like to note however, that by nature these programs can't really be portable. They all depend on specific file locations to exist and copying the program to another computer could cause it to crash.


Drilldown Menu change Heights like Foundation 5?

In foundation 5 the dropdown menu would change heights to fit whatever amount of links were in the menu, so 3 links would be for example 150px height dropdown, then if the next submenu was 6 links it would expand the height to fit the 6 links for example 500px.
Go here on your phone or make your browser narrow to see what I am referring to, I tried CSS solutions but in Foundation 5 it worked by applying a height dynamically using javascript.
Curious what the reason for the change is, still love this menu but miss the dynamic height changing/
http://foundation.zurb.com/sites/docs/v/5.5.3/components/topbar.html  (Old menu that changed heights)
Thanks. Let me know if I can clear anything up.

OK still trying to get this to work. Definitely no CSS solution as the way to dynamically change the height of the drilldown menu based on how many elements are in it seems to be a javascript only solution. If anyone knows where to adjust the code in the core files, it would be great to have this added, possible a data-option to dynamically change the drilldown heights.

I am having a similar issue this somewhat fixed it
div.is-drilldown {
height:auto;
}
However the the hidden nested menus still take up layout space, so the largest nested menu will push other elements down the page even when it is hidden

Closing in favor of #7821. We've gotten a number of requests for this feature now, so it's looking more and more like something we'll want to implement.


Fix dynamic ScrubberWidget and add throttling

Fixes two issues with the ScrubberWidget:

A few releases ago we added the ability to use the ScrubberWidget on a DynamicMap as long all  its dimensions define values, however for this to work the plots had to expand the values into their cross-product. We subsequently reverted the expansion change because it was very slow and expensive for large cross products but did not disable the scrubber. This PR ensures that this works even without expanding the whole cross-product.

This PR adds throttling to the ScrubberWidget by using the same approach used by the selection widgets.

[x] Fixes https://github.com/ioam/holoviews/issues/2571

Ready for review and merge.

Looks good and I'm glad the most tricky it is contained within a well-defined utility. I'll merge once the pr build goes green too.

The failing pr test was a transient (now restarted). Merging.


Minimal example of inference

Hello, cool work! I am trying to use this in the context of loop closure detection (first trying with KITTI).
I tried to use the model as follows. Could you help to sanity check my code?
# relevant imports, include PIL, numpy etc.
from torchvision import transforms as tvf
import torch

# initialize model, load to GPU
vpr_model = torch.hub.load("amaralibey/bag-of-queries", "get_trained_boq",
                                            backbone_name="dinov2", output_dim=12288)
vpr_model.to('cuda')
boq_in_size = (322, 322)  # to be used with DinoV2 backbone
base_image_transform = tvf.Compose([
                tvf.ToTensor(),
                tvf.Resize(boq_in_size, interpolation=tvf.InterpolationMode.BICUBIC, antialias=True),
                tvf.Normalize(mean=[0.485, 0.456, 0.406], std=[0.229, 0.224, 0.225])])

# Load an actual image. replace image_full_path with your stuff
image = PIL.Image.open(image_full_path).convert("RGB")
img_pt = base_image_transform(image).to('cuda')[None, ...]

global_descriptor = vpr_model(img_pt)[0] # 2nd elem of tuple is the device
gd_np = global_descriptor.cpu().numpy()

# use gd_np for similarity

As a follow up, this code worked for my use-case. I just had to set a lower score threshold than for AnyLoc descriptors, but it seems to perform better.
Thanks!

Thank you @tianyilim for your input.
Please, feel free to reopen if you have other questions.
Best,


interactive-node API reference seems to be missing

From https://dev.mixer.com/guides/mixplay/introduction I follow a link to https://mixer.github.io/interactive-node/ hoping to find API reference, but the content in the panel on the right seems not to have been generated.  I see only setWebSocket, and no other functions.
On this same page ( https://mixer.github.io/interactive-node/ ) there's a specific reference to the "Client.open" method that links to https://mixer.github.io/interactive-node/classes/client.html#open which returns a 404, which I assume is the same issue.

Thanks!, i opened: https://github.com/mixer/interactive-node/issues/108 on the client awhile back to track there too.


Honor the templates.default.includeDate configuration setting

Disabling the build date in the footer is important for reproducible builds. We already observe other settings from the JSDoc default template (staticFiles, outputSourceFiles), so do the same with this one.
Code copied from jsdoc3/jsdoc@254391e138853e773d374db67ebefd1f3997e21f.

Looks good 👍


Hide extension in the included files [Feature request]

I would like to hide the extension in any of the files included in the bundled .dmg. macOS automatically hides the extension of the .app file, but dmgbuild does not do that automatically for all the embedded files. Currently, it is not possible to select which files show they extension and which do not. This would be a nice feature, especially if someone wants to include scripts with long .command extensions.
As a workaround, it is currently possible to eliminate the extension by hiding the file (by adding a . before its name), including it in the .dmg, and then creating a symlink using "symlinks:" that points on the hidden file. The created symlink can have an arbitrary name without any extension. The hidden file will not appear in the created .dmg as expected.

On 27 Sep 2017, at 22:46, Alberto Sottile<EMAIL_ADDRESS>wrote:

I would like to hide the extension in any of the files included in the bundled .dmg. macOS automatically hides the extension of the .app file, but dmgbuild does not do that automatically for all the embedded files. Currently, it is not possible to select which files show they extension and which do not. This would be a nice feature, especially if someone wants to include scripts with long .command extensions.
As a workaround, it is currently possible to hide the file by adding a . before its name, include it in the .dmg, and then create a symlink using "symlinks:" that points on the hidden file. The created symlink can have an arbitrary name without any extension. The hidden file will not appear in the created .dmg as expected.

It’s probably also worth mentioning, in the context of the file you wanted to hide the extension of, that you don’t need to name something “.command” to get it to run in Terminal.  If you start the file with
#! /bin/sh
or
#! /bin/bash
and then make sure its “x” bit(s) are set, you can call it whatever you like.
Kind regards,
Alastair.
--
http://alastairs-place.net

Thanks for having implemented this!


MODOAIPMH-292: Incorrect cursor in resumptionToken

Jira: https://issues.folio.org/browse/MODOAIPMH-292
PURPOSE
Fix incorrect cursor value returned within resumption token.
Add an index to the request_id column in order to improve the request processing with date range parameters. Previously without index, the request responded with the timeout error.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: MikhailFokanov:x: IlliaDaliekIlliaDaliek seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

It would be great if you could add an "Approach" section to the PR description, with summary list of what's being done here...
e.g.

Add an ID column to the instance_ids table for purposes of sorting / maintaining the same order on the initial and subsequent requests.  This is simple integer incrementing from 0, 1, 2, ... N.
Fix the stale resumption token check such that .... etc.


get best from leaderboard

Types of changes

[ ] Docs change / refactoring / dependency upgrade
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Motivation and Context / Related issue
I'd like to have a "auto-suggest" for cases in which the only goal is to find the best candidate.
We do not want to compare methods; we just want an "autosuggest" for doing better than before,
using the current best as a "auto-suggest".
How Has This Been Tested (if it applies)

Checklist

[x] The documentation is up-to-date with the changes I made.
[x] I have read the CONTRIBUTING document and completed the CLA (see CLA).
[x] All tests passed, and additional code has been covered with new tests.

What do you mean by auto suggest???

What do you mean by auto suggest???

I mean:

some of the ExperimentFunction could have a bit "ThisCanDoWarmStart"
when this bit holds true, the optimizers who like doing this could do "get_best_from_leaderboard" and use it as a suggest.

Useful for the experiments with @AnMoreau --- we want some long term runs, which can be started, stopped, resumed, and always focus on solving the problem (for research on physics) rather than on solving from scratch for optimization purpose.


Utilize SafeJsonResponse().

Issue #122. Using safeJsonResponse() also fixes the "UI-glitch" (and also partially affects #128).

Which UI glitch do you mean? 🙂

I mean the one that's detailed in #122.

So there's a meaningful error message in place of the star icon.

This still raises so I don't get how it resolves the issue. Let's talk about it but feel free to merge.

I don't get this. How could you reproduce the issue?

To clarify: without safeJsonResponse(), the response coming from the GitHub API is still valid even if there's an error since it can be parsed to JSON.

{message: "Not Found", documentation_url: "https://developer.github.com/v3/repos/#get"}

That's the reason why we could see the "star" icon without any context (title, description and tech icons).

(and also partially affects #128)

I don't recall this anymore, sorry. I'll take a look into it to what extent safeJsonResponse() fixes the error described there.

So there's a meaningful error message in place of the star icon if something goes wrong.

Please don't display anything in this case.

This still raises so I don't get how it resolves the issue. Let's talk about it but feel free to merge.

I don't get this. How could you reproduce the issue?

To clarify: without safeJsonResponse(), the response coming from the GitHub API is still valid even if there's an error since it can be parsed to JSON.

I meant in the code but this makes sense then, nevermind.


Electrical Power Measurement Cluster XML has to be updated as per spec

Description
XML  has to be updated as per spec:
In MeasurementRangeStruct, the Constraint has to be added for below mentioned fields.

EndTimestamp
EndTimestamp
EndTimestamp
EndTimestamp

Hi,
The spec says that the constraint is:

EndTimestamp should be "min (StartTimestamp+1)"
Given that StartTimestamp is not a static value I don't think it is possible to encode that into the XML.
If it was a value (e.g. min 0) then we could add it.
So I don't think this issue is a bug in the XML.
It could be checked in a test case update to validate this automatically. However I don't think this is a cert blocker. Perhaps something to raise a follow up issue for after SVE or in 1.4 given the late timescale.


Adding Hover effect on images

I would like to work on this
please assign me

hey can you  assign this to me under gssoc'24 EXTENDED with label gssoc'24

@PREM-A261
Please assign me this issue.
Also please mention what needs to be corrected exactly

Can i too work on this issue.


Failed to execute 'setItem' on 'Storage': Setting the value of 'content:.' exceeded the quota.

Describe the bug
The gallery is failing to load and I'm seeing these errors in the browser console:
DOMException: Failed to execute 'setItem' on 'Storage': Setting the value of 'content:.' exceeded the quota.

DOMException: Failed to execute 'setItem' on 'Storage': Setting the value of 'content:2015-06-27 Rock and Roll Hall of Fame and Museum' exceeded the quota.

This might just be that I have too many photos for pigallery2. 487 folders, 15537 photos, and 164 videos.
Environment:

OS: Server runs alpine-3.11-default_20200425_amd64
Browser: (everything I tested) Chrome v84.0.4147.105 in Windows and Linux, Firefox v79.0 in Windows

Used app version:
Release 1.8.0

That does not look good.
This should be fixed. Until then (as a mitigation), you can manually delete the local storage in your chrome browser. It is only used for caching, so it wont cause any issue. (A page reload is recommended afterwards)
You can also disable caching (Settings (enable advanced mode) -> Other -> Cache) so this issue wont come up again

The latest commit should fix this.


Cannot install lm_eval without changing other packages' version

Hi, Im trying to install lm_eval via 'pip install -e .' in the repo directory. But it always uninstall my old package and install a new one such as torch... however i dont want to change my env settings.
For torch as an example, I've already installed 'torch=1.12.1+cu113', and i check the 'pyproject.toml' of this repo, only requires torch>=1.8. However, when i tried 'pip install -e .', it show like this

How can i solve it, anybody helps?

And i tried to reinstall torch=1.12.1, but it did not satisfy the requirements of 'perf' 😂

its a dilemma now 😂

—upgrade-strategy to-satisfy-only

Thanks for reply.
I tried pip install --upgrade --upgrade-strategy=to-satisfy-only -e . in repo directory, and it shows option --upgrade-strategy: invalid choice: 'to-satisfy-only' (choose from 'only-if-needed', 'eager'). In the pip docs you provided, the option to-satisfy-only is undocumented, which seems should be avoided. Moverover, This is actually the “default” upgrade strategy when --upgrade is not set said in the doc. so i think --upgrade --upgrade-strategy=to-satisfy-only is already the default setting.
The problem of dependency seems to be weird and complicated 😂 Maybe upgrading torch is the easiest way to solve.

Yes, you should be able to fix this by force-reinstalling torch without any of its dependencies as an override.
Closing as I think this isn't lm_eval specific any more, but feel free to post on discord or if you find a workaround for this, feel free to post it here so that others who might experience it can see the fix!

Hello, I want to ask how exactly to operate


Error during metrics collection results in 500

Hi,
As soon as I added logging.googleapis.com  to the prefixes it started to crash.
Here is the config:
[
    {
        "name": "STACKDRIVER_EXPORTER_GOOGLE_PROJECT_ID",
        "value": "eflow-dev"
    },
    {
        "name": "STACKDRIVER_EXPORTER_MONITORING_METRICS_TYPE_PREFIXES",
        "value": "cloudsql.googleapis.com,datastore.googleapis.com,pubsub.googleapis.com,bigquery.googleapis.com,logging.googleapis.com"
    },
    {
        "name": "STACKDRIVER_EXPORTER_WEB_LISTEN_ADDRESS",
        "value": ":8080"
    },
    {
        "name": "STACKDRIVER_EXPORTER_MONITORING_METRICS_INTERVAL",
        "value": "5m"
    }
]

The error seems to come from prometheus itself and only gets printed when you query /metrics
An error has occurred during metrics collection:

138 error(s) occurred:
* collected metric stackdriver_monitoring_logging_googleapis_com_log_entry_count label:<name:"cluster_name" value:"benchmark-cluster" > label:<name:"container_name" value:"j-clicker" > label:<name:"instance_id" value:"272634823886272651" > label:<name:"log" value:"j-clicker" > label:<name:"namespace_id" value:"default" > label:<name:"pod_id" value:"j-clicker-vvwnw" > label:<name:"project_id" value:"eflow-dev" > label:<name:"resource_type" value:"gke_container" > label:<name:"severity" value:"INFO" > label:<name:"unit" value:"1" > label:<name:"zone" value:"us-east1-c" > counter:<value:0 >  has label dimensions inconsistent with previously collected metrics in the same metric family
... and so on

@OlivierBoucher Thanks for reporting this. I've identified the problem but I'm still trying to find the right solution for this.
When the exporter is reporting metrics it adds several labels, the ones that came from the metric itself, and the ones that came from the monitored resource. Usually the monitored resource is the same (because metrics are namespaced), but logging.googleapis.com is a bit special, because the monitored resource might be different. This means that depending on the resource emitting logs, the labels will differ, and hence the has label dimensions inconsistent with previously collected metrics error.

This has been fixed at v0.2.0. Please be warned that metric's names have changed in this release (ie, the above metric has changed fromstackdriver_monitoring_logging_googleapis_com_log_entry_count to stackdriver_gke_container_logging_googleapis_com_log_entry_count, note that instead of monitoring now it has the resource type gke_container).

@frodenas Thank you very much


Use updated sftp Server constructor

The sftp package has undergone some changes to make it easier to wire in server options. This updates our code to use the new constructor format.
If/when merged, please bump the src/github.com/pkg/sftp submodule in diego release to the current master.

Thanks for the update, @sykesm! We'll pull it in soon.
Best,
Eric


Missing API functionality

I think this API is missing the ability to obtain the item name by material and damage value.
It consist the ability to get the item by material.
String itemName = localeManager.queryMaterial(material);

But I'd also like to add a damage value to get what type of the material to use.
String itemName = localeManager.queryMaterial(material, (short) 5);

Or even make it obtainable through the player:
String playerLangItemName = localeManager.queryMaterial(player, material, (short) 5);

I think this is only possible for english.
But to obtain the english translation is still a big goal.

Thanks for the input, but I'm afraid it's missing because it's impossible afaik. The LocaleQuery class merely converts a Material into a key, which is then sent to the client for translation. For more information, visit https://www.spigotmc.org/resources/localelib.65617/
If you aware of a way to retrieve the translated string back from the client, PRs are welcome.

I will add a PR on how it can retrieve the item name from the key from the server's locale


Don't use segment hashCode in MemorySegmentVectorFloat

This hashcode depends on segment base/offset in the heap rather than contents. This breaks testing around PQVectors hashcodes added in #370. Rather than this being a regression uncovered in #370, this is due to increased test assertions added in #370 that only fail when using the native provider. We adopt the same approach used by ByteSequence in #370.

I've been thinking of MemorySegment as a more flexible ByteBuffer, but it looks like the hashcode implementations diverge. Here is ByteBuffer's:
    /**
     * Returns the current hash code of this buffer.
     *
     * <p> The hash code of a byte buffer depends only upon its remaining
     * elements; that is, upon the elements from {@code position()} up to, and
     * including, the element at {@code limit()} - {@code 1}.
     *
     * <p> Because buffer hash codes are content-dependent, it is inadvisable
     * to use buffers as keys in hash maps or similar data structures unless it
     * is known that their contents will not change.  </p>
     *
     * @return  The current hash code of this buffer
     */
    public int hashCode() {
        int h = 1;
        int p = position();
        for (int i = limit() - 1; i >= p; i--)

            h = 31 * h + (int)get(i);

        return h;
    }


Smarter formatting for year channels?

In a case like this (data), it’d be nice to avoid the commas for the year axis:

Plot.plot({
  width,
  color: { legend: true },
  marks: [
    Plot.rectY(overview, { x: "Year", y: "Value", fill: "Type", interval: 1 })
  ]
})

Of course, you can do it with x: {tickFormat: ""}, but could Plot figure this out automatically?
Similarly when you do something like title: "Year", it’s a bummer that the automatic formatting for numbers shows a comma. I think we could maybe track a hint that looks for (case-insensitive) “year” and avoids the comma.

related #355

In fact this is one of the first questions that came up in a workshop last Friday. But it was happening in Inputs.table, we hadn't started plotting the data yet. The offending field was called "annee" (French for year).

I think this is most common problem/gotcha for those starting with plot. This handling could reduce friction for new users who are not familiar with JavaScript data handling features.
I had posted exactly this question in forums:
https://talk.observablehq.com/t/handling-date-column-during-file-import/6333
https://observablehq.com/@arky/disasters-in-south-eastern-asia-1900-2021

Here's a related paper. I wonder if there are implementations of something like this out there in open-source land, it's such a common issue.

Question: a possible strategy would be for the default format to depend on the domain. If the domain is contained in, say, [1500, 2200], set the default formatter to be d => '${d}' rather than Intl.NumberFormat. This heuristic might be slightly surprising in the odd case, but it would fix the very common issue of years being poorly formatted, while still giving nice numbers by default in general.

If the domain is contained in, say, [1500, 2200], set the default formatter to be d => '${d}' rather than Intl.NumberFormat.

We should also ensure that all channel values are integers (or at least sample the first ~40? channel values). If we see a fractional value such as 1500.1231041240 we should use the default number format rather than assuming it is a year.

If the scale has an interval option that is a year (or a multiple of a year), it seems like we could at least special-case that to use the %Y format and drop the -01-01.

I tried just dropping -01-01 (and -01) from isoformat in #1556, but I think we need to be a little smarter and detect intervals, since otherwise with ordinal scales you are more likely to end up with inconsistent formatting of dates. (Admittedly this is already a problem with sub-daily intervals, such as hours that fall on midnight, but it does exacerbate the problem.)
For example below, we could check the domain of the x scale, and the shortest format that applies to all of the dates in the domain (YYYY-MM-DD) rather than choosing the shortest format each value independently.

#1790 handles this for temporal data. The only challenge left here is that we have very little signal that these represent years rather than arbitrary numbers (which should have commas). The possible heuristics are:

Look at the channel name, and see it’s “year”.
Look at the values, and check if they are integers in the range 1900–2100 (exact range TBD).

(1) is English only, which isn’t great. (2) is brittle; it wouldn’t work well for historical data, and it’ll have some false positives for other data (e.g., melting points of metals in Fahrenheit). But it’s also not the end of the world if there’s a false positive, since the only difference would be a missing comma. And we don’t really need commas for four or fewer digits anyway. So we could extend the heuristic to integers in the range 0–9999.


alttab: doCheck+gpl3Plus

Motivation for this change
alttab has a make check target since 1.6.0. This PR enables running the target as part of CI, via doCheck = true;.
I also clarified the license to gpl3Plus, as the license header in e.g. https://github.com/sagb/alttab/blob/1c0754e6d2da346a08b06725ba716350553edf48/src/alttab.c clearly says "any later".
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Tests fail.

They don't fail per say but execute nothing according to the logs of ofborg.

They don't fail per say but execute nothing according to the logs of ofborg.

They fail if you merge this with master (where alttab was updated 1.6.0).

Urgh, yes. I'll rebase and try again!


How to deal with discrete (integer) variables?

Hi,
Here's another question, for discrete variables, such as age and other count variables (integers), should we set them as continuous columns?
From my understanding of your paper, age was assigned as a continuous column. However, when I do that, the synthetic output of age would not longer be integers.
Thank you!

I have the same question. Can you explain more on the continuous columns parameter.


Execute the PAL Auto Smoothing algorithm (Forecast App)

Tutorial URL: https://developers.sap.com/tutorials/hxe-aa-forecast-15.html
Please specify the step you are referring to
inconsistent naming of files and references  ... "auto_smoothing"  and "auto_smooting"
=> route not found

while the syntax does change from one section to another the smoothing action can be performed, correct?


(Missing) Bridging instructions for CDK Rollups

Hi,
While we have bridging covered in the CDK Validium quickstart tutorial, there seems to be no mention of setting up a Bridge Service / UI with any of the CDK Rollup tutorials. If plausible, we'd like some instructions for interacting with the bridging functionalities of the rollup.
Thanks!

Hi @sahil3Vedi you are absolutely right, thank you for raising this issue. We are working on adding bridging instructions to our docs. I'll keep this issue open and ping you when we have some developments.

Hi @sahil3Vedi as part of our new documentation on the CDK kurtosis stack, there are now bridging instructions. This information is related to any CDK stack. Can you check it out here: https://docs.polygon.technology/cdk/get-started/how-to/use-native-token/.
I'm gonna close this issue now but feel free to raise a new one if you have any trouble following the bridging instructions in our new doc.
Thank you :)


Question about camera extrinsics in batch

Dear Professor，
Thank you very much for seeing this email in your busy schedule.
I have a question for you here: In your ''Cross-view Transformers for real-time Map-view Semantic Segmentation '' paper, when encoding image information, the rotation matrix and translation of the camera external parameters are relative to the ego-vehicle coordinate system, but the ego-vehicle coordinates are also useful in the code to the world coordinate system. So I don't know what the external parameters that go into the encoded batch in the code represent exactly。
I hope you can answer a question in your free time, thank you very much！

I found that the camera extrinsics generated by CVT are different from those generated by Fiery, I'm confused, too. Do you know why now?


Purpose of accessibility button on English video player

Describe the Issue
There is an "accessibility" button in the bottom right corner of the video player for the new English courses. Currently, the only function of this button seems to be toggling the captions on and off. Is this button going to do anything else? If not, then I suggest we use a closed caption icon instead and change the name of the button to "closed captions" and use the aria-pressed attribute to manage its state.
Affected Page
https://www.freecodecamp.org/learn/a2-english-for-developers/learn-greetings-in-your-first-day-at-the-office/task-1
Steps to Reproduce
N/A
Expected behavior
Captions off:
<button aria-label="closed captions" aria-pressed="false"> ... </button>

Captions on:
<button aria-label="closed captions" aria-pressed="true"> ... </button>

Screenshots
No response
System
N/A
Additional context
No response

can you pls provide the icon you want to replace , i will work to change it .

https://github.com/freeCodeCamp/freeCodeCamp/pull/52543#issuecomment-1858694870
Sounds like it is currently planned to just toggle captions. In this case, we can go with changing the icon used here: https://github.com/freeCodeCamp/freeCodeCamp/blob/34347235df5a93291798e90623179ae4b5df31b9/client/src/templates/Challenges/components/scene/scene.tsx#L203
As an aside, I would also change the logic to showDialogue.

YouTube adds that red, bottom border:

Unpressed
Pressed


processDomCallbacks not emptying 'once' array

Version

Phaser Version: 3.16.0 Beta 4

Operating system: Windows 10

Browser: Chrome

Description
I'm using the addUpCallback to open a new window using a Phaser "button". The upOnce callback is being added to the domCallbacks array each time I click the button even though it should be emptying the array so that it only happens once.

Looking through the source code I noticed you empty the array in line 888:
https://github.com/photonstorm/phaser/blob/master/src/input/InputManager.js#L888
I logged it and it empties the array, but when I check the InputManager in-game the callback is still in the array.
In the example code, it will open google just fine the first time, but then the second time it will open 2 and then 3 and so on.
Example Test Code
this.add.image(40, 80, 'patchNotes')
            .setOrigin(0)
            .setInteractive({useHandCursor: true})
            .on('pointerdown', () => {
                game.input.addUpCallback(() => {
                    window.open('https://www.google.com.com/docs');
                }, true)
            });

Additional Information

I changed
once = [];

to
this.domCallbacks.upOnce = [];
this.domCallbacks.downOnce = [];
this.domCallbacks.moveOnce = [];

and it works as intended.

Thank you for submitting this issue. We have fixed this and the fix has been pushed to the master branch. It will be part of the next release. If you get time to build and test it for yourself we would appreciate that.


update nsb yaml

What type of PR is this?
Add one of the following kinds:

enhancement/feature : Update nsb initial yaml file. Including create a slice booking session, delete and inquiry session.

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for reviewers:
Changelog input
 release-note

Additional documentation
This section can be blank.
docs

Does SLA Targets need to include maximum delay?


java.lang.UnsupportedOperationException

I am getting
java.lang.UnsupportedOperationException
When I try to generate code from postgres 10
Full stack trace
java.lang.UnsupportedOperationException
at java.util.Collections$UnmodifiableMap.put(Collections.java:1457)
at com.speedment.generator.standard.lifecycle.GeneratedMetadataTranslator.makeCodeGenModel(GeneratedMetadataTranslator.java:78)
at com.speedment.generator.standard.lifecycle.GeneratedMetadataTranslator.makeCodeGenModel(GeneratedMetadataTranslator.java:46)
at com.speedment.generator.translator.AbstractJavaClassTranslator.get(AbstractJavaClassTranslator.java:144)
at com.speedment.generator.translator.AbstractJavaClassTranslator.get(AbstractJavaClassTranslator.java:62)
at java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:193)
at java.util.ArrayList$ArrayListSpliterator.forEachRemaining(ArrayList.java:1374)
at java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:481)
at java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:471)
at java.util.stream.ReduceOps$ReduceOp.evaluateSequential(ReduceOps.java:708)
at java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:234)
at java.util.stream.ReferencePipeline.collect(ReferencePipeline.java:499)
at com.speedment.generator.core.internal.translator.TranslatorManagerHelper.accept(TranslatorManagerHelper.java:142)
at com.speedment.generator.core.internal.translator.DefaultTranslatorManager.accept(DefaultTranslatorManager.java:58)
at com.speedment.tool.core.internal.util.ConfigFileHelper.generateSources(ConfigFileHelper.java:416)
at com.speedment.tool.core.internal.component.UserInterfaceComponentImpl.lambda$generate$10(UserInterfaceComponentImpl.java:358)
at java.util.concurrent.CompletableFuture.uniHandle(CompletableFuture.java:822)
at java.util.concurrent.CompletableFuture$UniHandle.tryFire(CompletableFuture.java:797)
at java.util.concurrent.CompletableFuture$Completion.exec(CompletableFuture.java:443)
at java.util.concurrent.ForkJoinTask.doExec(ForkJoinTask.java:289)
at java.util.concurrent.ForkJoinPool$WorkQueue.runTask(ForkJoinPool.java:1056)
at java.util.concurrent.ForkJoinPool.runWorker(ForkJoinPool.java:1692)
at java.util.concurrent.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:157)

ok. Thanks. I have scheduled this issue to release 3.0.22

Can you upload the speedment.json file for your project to this issue?

speedment.json file uploaded as requested
speedment.zip

Thanks

I think I can see the problem. It is a bug introduced when we added a version check to make sure the code is executed with the same version of Speedment as it was generated with.
I will fix this now and make sure it is included in the 3.0.22-release. Until then, you can either go back to 3.0.20 that doesn't have this issue or build Speedment 3.0.22-SNAPSHOT yourself by checking out the develop-3.0-branch.

Hm, it didn't fis the problem entirely. It seems to be related to generating code from the tool. Using mvn speedment:generate seems to work.

I have fixed this now. I can verify that I get the same exception with 3.0.21 and it doesn't show up in 3.0.22-SNAPSHOT. We should do an immediate release of 3.0.22 since this will affect most users.


python3Packages.django-cryptography: include missing source files

Description of changes
Some source files are missing from the package. This change ensures they are included.
Before:
$ find result/lib/python3.10/site-packages/django_cryptography | grep -v pycache
result/lib/python3.10/site-packages/django_cryptography
result/lib/python3.10/site-packages/django_cryptography/fields.py
result/lib/python3.10/site-packages/django_cryptography/conf.py
result/lib/python3.10/site-packages/django_cryptography/__init__.py

After:
$ find result/lib/python3.10/site-packages/django_cryptography | grep -v pycache
result/lib/python3.10/site-packages/django_cryptography
result/lib/python3.10/site-packages/django_cryptography/core
result/lib/python3.10/site-packages/django_cryptography/core/signing.py
result/lib/python3.10/site-packages/django_cryptography/core/__init__.py
result/lib/python3.10/site-packages/django_cryptography/fields.py
result/lib/python3.10/site-packages/django_cryptography/conf.py
result/lib/python3.10/site-packages/django_cryptography/__init__.py
result/lib/python3.10/site-packages/django_cryptography/utils
result/lib/python3.10/site-packages/django_cryptography/utils/__init__.py
result/lib/python3.10/site-packages/django_cryptography/utils/crypto.py

Things done

Built on platform(s)

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

[x] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 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.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2183

Can we fetch https://github.com/georgemarshall/django-cryptography/pull/88 instead?

@SuperSandro2000 The raw upstream patch did not apply properly, so I generated one and included it in this PR.


Adding up profile

fix #244
add ing up my profile info

I’ve recently contributed to the repository . Could you please add the gssoc-ext label to my contributions so that they are recognized under the program?
Thank you for your help!


Remove dependencies on ruby-ng and redis2

There are so many cookbooks to install ruby and redis nowadays that it doesn't make any sense to require user to upload those particular two cookbooks.
Even if you set "ruby_install_type" and "redis_install_type" to "none" it'll still require those cookbook to be on your chef-server:
% knife cookbook upload lita
Uploading lita           [0.0.3]
ERROR: Cookbook lita depends on cookbooks which are not currently
ERROR: being uploaded and cannot be found on the server.
ERROR: The missing cookbook(s) are: 'ruby-ng' version '~> 0.1.1', 'redis2' version '~> 0.5.0'

One option is to replace "depends" in metadata with "recommends" or "suggests":
https://docs.getchef.com/essentials_cookbook_metadata.html#settings
But using Berkshelf (which is a recommended way to manage cookbooks versions) they'll still be resolved and uploaded to the chef-server. You don't want any non-used cookbooks to be present on your chef-server, right?
I think having a section in docs just like https://github.com/opscode-cookbooks/jenkins#java should be enough for users to figure out dependencies and there's no need for ruby-ng and redis2 to be in chef-lita cookbook at all.

I've decided to leave this as-is long term. I was going to take on maintaining "mini-versions" of the rubyng and redis recipes in the  for those that wanted a turnkey solution. But that seems like a bad idea the more I think about it. I could be persuaded, but for now, I'm closing this.


Enable local snapshot restore

Closes: #29

PR checklist

[ ] Tests written/updated
[ ] Changelog entry added in .changelog (we use unclog to manage our changelog)
[ ] Updated relevant documentation (docs/ or spec/) and code comments

Keeping this draft until we finish the review of #728.

Different solution pursued.


pgmanage: fix build

Motivation for this change
Recent backport of postgresql to 17.09 broke few things, including pgmanage (https://hydra.nixos.org/eval/1437029). While I do not see why this is related, this fixes pgmanage. I’ll check later how this is affecting master.
The configure script uses the command builtin command which is bash
specific[1] while having a "#!/bin/sh" head.
This forces the use nix default shell (bash)
[1] https://www.gnu.org/software/bash/manual/html_node/Bash-Builtins.html#Bash-Builtins
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[X] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[X] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

/bin/sh is at the moment an awkward busybox sh in nix 2.0, that's why.

@GrahamcOfBorg test pgmanage

Success on x86_64-linux (full log)
Partial log (click to expand)
one: exit status 1
syncing
one: running command: sync
one: exit status 0
test script finished in 18.25s
cleaning up
killing one (pid 140)
vde_switch: EOF on stdin, cleaning up and exiting
vde_switch: Could not remove ctl dir '/tmp/nix-build-vm-test-run-pgmanage.drv-0/vde1.ctl': Directory not empty
/nix/store/2m0bbvby2rbxzq0nwcflk5kq4l96mxd5-vm-test-run-pgmanage

There is a similar problem for pgadmin3. Does it makes sense I spent some time fixing this (for 17.09) or do we just wait for this root error to be fixed?

@lsix yes, I don't think it will be changed back.

Success on x86_64-linux (full log)
Partial log (click to expand)
Installation Successful
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/badk522ww4jagql7gr1hh0h4ppqp8y4b-pgmanage-10.0.2
shrinking /nix/store/badk522ww4jagql7gr1hh0h4ppqp8y4b-pgmanage-10.0.2/sbin/pgmanage
gzipping man pages under /nix/store/badk522ww4jagql7gr1hh0h4ppqp8y4b-pgmanage-10.0.2/share/man/
stripping (with flags -S) in /nix/store/badk522ww4jagql7gr1hh0h4ppqp8y4b-pgmanage-10.0.2/sbin
patching script interpreter paths in /nix/store/badk522ww4jagql7gr1hh0h4ppqp8y4b-pgmanage-10.0.2
checking for references to /tmp/nix-build-pgmanage-10.0.2.drv-0 in /nix/store/badk522ww4jagql7gr1hh0h4ppqp8y4b-pgmanage-10.0.2...
moving /nix/store/badk522ww4jagql7gr1hh0h4ppqp8y4b-pgmanage-10.0.2/sbin/* to /nix/store/badk522ww4jagql7gr1hh0h4ppqp8y4b-pgmanage-10.0.2/bin
/nix/store/badk522ww4jagql7gr1hh0h4ppqp8y4b-pgmanage-10.0.2

@Mic92 I just fixed it.

Success on aarch64-linux (full log)
Partial log (click to expand)
Installation Successful
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/lr706ndfnifdfh65kmhlrf2v2afgrggc-pgmanage-10.0.2
shrinking /nix/store/lr706ndfnifdfh65kmhlrf2v2afgrggc-pgmanage-10.0.2/sbin/pgmanage
gzipping man pages under /nix/store/lr706ndfnifdfh65kmhlrf2v2afgrggc-pgmanage-10.0.2/share/man/
stripping (with flags -S) in /nix/store/lr706ndfnifdfh65kmhlrf2v2afgrggc-pgmanage-10.0.2/sbin
patching script interpreter paths in /nix/store/lr706ndfnifdfh65kmhlrf2v2afgrggc-pgmanage-10.0.2
checking for references to /build in /nix/store/lr706ndfnifdfh65kmhlrf2v2afgrggc-pgmanage-10.0.2...
moving /nix/store/lr706ndfnifdfh65kmhlrf2v2afgrggc-pgmanage-10.0.2/sbin/* to /nix/store/lr706ndfnifdfh65kmhlrf2v2afgrggc-pgmanage-10.0.2/bin
/nix/store/lr706ndfnifdfh65kmhlrf2v2afgrggc-pgmanage-10.0.2

Success on aarch64-linux (full log)
Partial log (click to expand)
Installation Successful
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/5hj3mnh7jkh3jv5ggz3b884wdfa0672a-pgmanage-10.0.2
shrinking /nix/store/5hj3mnh7jkh3jv5ggz3b884wdfa0672a-pgmanage-10.0.2/sbin/pgmanage
gzipping man pages under /nix/store/5hj3mnh7jkh3jv5ggz3b884wdfa0672a-pgmanage-10.0.2/share/man/
stripping (with flags -S) in /nix/store/5hj3mnh7jkh3jv5ggz3b884wdfa0672a-pgmanage-10.0.2/sbin
patching script interpreter paths in /nix/store/5hj3mnh7jkh3jv5ggz3b884wdfa0672a-pgmanage-10.0.2
checking for references to /build in /nix/store/5hj3mnh7jkh3jv5ggz3b884wdfa0672a-pgmanage-10.0.2...
moving /nix/store/5hj3mnh7jkh3jv5ggz3b884wdfa0672a-pgmanage-10.0.2/sbin/* to /nix/store/5hj3mnh7jkh3jv5ggz3b884wdfa0672a-pgmanage-10.0.2/bin
/nix/store/5hj3mnh7jkh3jv5ggz3b884wdfa0672a-pgmanage-10.0.2

Success on x86_64-linux (full log)
Partial log (click to expand)
Installation Successful
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/i6j445ngsfsd4vslpj8445086l7smqcy-pgmanage-10.0.2
shrinking /nix/store/i6j445ngsfsd4vslpj8445086l7smqcy-pgmanage-10.0.2/sbin/pgmanage
gzipping man pages under /nix/store/i6j445ngsfsd4vslpj8445086l7smqcy-pgmanage-10.0.2/share/man/
stripping (with flags -S) in /nix/store/i6j445ngsfsd4vslpj8445086l7smqcy-pgmanage-10.0.2/sbin
patching script interpreter paths in /nix/store/i6j445ngsfsd4vslpj8445086l7smqcy-pgmanage-10.0.2
checking for references to /tmp/nix-build-pgmanage-10.0.2.drv-0 in /nix/store/i6j445ngsfsd4vslpj8445086l7smqcy-pgmanage-10.0.2...
moving /nix/store/i6j445ngsfsd4vslpj8445086l7smqcy-pgmanage-10.0.2/sbin/* to /nix/store/i6j445ngsfsd4vslpj8445086l7smqcy-pgmanage-10.0.2/bin
/nix/store/i6j445ngsfsd4vslpj8445086l7smqcy-pgmanage-10.0.2

@GrahamcOfBorg test pgmanage

Success on x86_64-linux (full log)
Partial log (click to expand)
one: exit status 1
syncing
one: running command: sync
one: exit status 0
test script finished in 198.48s
cleaning up
killing one (pid 143)
vde_switch: EOF on stdin, cleaning up and exiting
vde_switch: Could not remove ctl dir '/tmp/nix-build-vm-test-run-pgmanage.drv-0/vde1.ctl': Directory not empty
/nix/store/7lphrmm9sdmd2w7pyg7hh9qzl4d2a5nm-vm-test-run-pgmanage

Failure on aarch64-linux (full log)
Partial log (click to expand)
installing
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/qi7xks1hdbik3kjl44cdcsw54p1bpyy3-nixos-test-driver
stripping (with flags -S) in /nix/store/qi7xks1hdbik3kjl44cdcsw54p1bpyy3-nixos-test-driver/lib  /nix/store/qi7xks1hdbik3kjl44cdcsw54p1bpyy3-nixos-test-driver/bin
patching script interpreter paths in /nix/store/qi7xks1hdbik3kjl44cdcsw54p1bpyy3-nixos-test-driver
/nix/store/qi7xks1hdbik3kjl44cdcsw54p1bpyy3-nixos-test-driver/bin/.nixos-test-driver-wrapped: interpreter directive changed from " /somewhere/perl -w" to "/nix/store/b5a764yzqpgx03xyhd4kayipzpy84a6m-perl-5.24.3/bin/perl -w"
checking for references to /build in /nix/store/qi7xks1hdbik3kjl44cdcsw54p1bpyy3-nixos-test-driver...
cannot build derivation '/nix/store/mjmh60i2xnr4fv92x6q48rrvy1c60n3i-nixos-test-driver-pgmanage.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/d4h9vhn5rgs835v0rddkga3gh07z4qfr-vm-test-run-pgmanage.drv': 1 dependencies couldn't be built
error: build of '/nix/store/d4h9vhn5rgs835v0rddkga3gh07z4qfr-vm-test-run-pgmanage.drv' failed

Failure on aarch64-linux (full log)
Partial log (click to expand)
cannot build derivation '/nix/store/vxmpamlfmpd1jiir2qa5sr5dlnw7yslf-system-units.drv': 7 dependencies couldn't be built
cannot build derivation '/nix/store/yavhjwwbyrbb58bgwws0x03q12jwb76f-user-units.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/37m9yhm89lybxxgn3kg5bpivv67jf3ck-etc.drv': 7 dependencies couldn't be built
cannot build derivation '/nix/store/v73ld84grd10krmgbissq3yxi04kji2h-nixos-system-one-17.09.git.db3b01b.drv': 7 dependencies couldn't be built
cannot build derivation '/nix/store/lr5dkagf4l8yqf52swz1vxjf5jvah0jc-reginfo.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/53i07qpzkjjnpax7cqzlz30r6icgkr3v-run-nixos-vm.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/k4f24xzacfc09cd0h68pa7s0z4g2viac-nixos-vm.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/hi45x31kc3yhfgs55r4rrzvrxb1z93m1-nixos-test-driver-pgmanage.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/49dwvy6l3m7gnh2i6cjksrp0n7402l2s-vm-test-run-pgmanage.drv': 1 dependencies couldn't be built
error: build of '/nix/store/49dwvy6l3m7gnh2i6cjksrp0n7402l2s-vm-test-run-pgmanage.drv' failed


Fix rbac ServiceAccount imagePullSecrets template

Fix rbac ServiceAccount imagePullSecrets template.
Problem: NGF fails to deploy with NGINX+ because of the rbac ServiceAccount imagePullSecrets template error.
Solution: Fix the spacing which moves imagePullSecrets from being in the ServiceAccount's metadata to its own field.
Testing: Created Kubernetes secret and Docker Config Secret, then deployed NGF with NGINX+ and saw that the error was no longer there.
Closes #1952
Checklist
Before creating a PR, run through this checklist and mark each as complete.

[x] I have read the CONTRIBUTING doc
[ ] I have added tests that prove my fix is effective or that my feature works
[x] I have checked that all unit tests pass after adding my changes
[ ] I have updated necessary documentation
[x] I have rebased my branch onto main
[x] I will ensure my PR is targeting the main branch and pulling from my branch from my own fork

Release notes
If this PR introduces a change that affects users and needs to be mentioned in the release notes,
please add a brief note that summarizes the change.

Fix rbac ServiceAccount imagePullSecrets template which caused errors when running NGF with NGINX+

Fix rbac ServiceAccount imagePullSecrets template.
Problem: NGF fails to deploy with NGINX+ because of the rbac ServiceAccount imagePullSecrets template error.
Solution: Fix the spacing which moves imagePullSecrets from being in the ServiceAccount's metadata to its own field.


Segmentation fault (core dumped)

Hello, thank you for your work!
I am really interested in this repo, but I run into a issue which I don't think can be addressed without help.
I ran the progress in the official knowLM container:
(zhixi) root@b2896f53f126:/app/KnowLM# CUDA_VISIBLE_DEVICES=0,1 python examples/generate_lora_web.py --base_model ./zjunlp_knowlm-13b-zhixi
Loading checkpoint shards: 100%|███████████████████████████████████████████████████████████████████████████████████████████████████████████████| 3/3 [02:50<00:00, 56.81s/it]
/app/KnowLM/examples/generate_lora_web.py:198: GradioDeprecationWarning: Usage of gradio.inputs is deprecated, and will not be supported in the future, please import your component from gradio.components
  gr.inputs.Textbox(
/app/KnowLM/examples/generate_lora_web.py:198: GradioDeprecationWarning: `optional` parameter is deprecated, and it has no effect
  gr.inputs.Textbox(
/app/KnowLM/examples/generate_lora_web.py:198: GradioDeprecationWarning: `numeric` parameter is deprecated, and it has no effect
  gr.inputs.Textbox(
Running on local URL:  http://<IP_ADDRESS>:7860

To create a public link, set `share=True` in `launch()`.

Then I submit the instruction and inputs, and I got this:
Segmentation fault (core dumped)
I need to reboot the server to bring it back to normal.
I also tried single GPU (RTX 4090), this time it report out of memory.
I also tried CUDA_VISIBLE_DEVICES=0,1 accelerate launch examples/generate_lora_web.py --base_model ./zjunlp_knowlm-13b-zhixi
and I got:
The following values were not passed to `accelerate launch` and had defaults used instead:
        `--num_processes` was set to a value of `2`
                More than one GPU was found, enabling multi-GPU training.
                If this was unintended please pass in `--num_processes=1`.
        `--num_machines` was set to a value of `1`
        `--mixed_precision` was set to a value of `'no'`
        `--dynamo_backend` was set to a value of `'no'`
To avoid this warning pass in values for each of the problematic parameters or run `accelerate config`.
Loading checkpoint shards:   0%|                                                                                | 0/3 [00:00<?, ?it/s]WARNING:torch.distributed.elastic.multiprocessing.api:Sending process 74 closing signal SIGTERM
ERROR:torch.distributed.elastic.multiprocessing.api:failed (exitcode: -9) local_rank: 1 (pid: 75) of binary: /opt/conda/envs/zhixi/bin/python
Traceback (most recent call last):
  File "/opt/conda/envs/zhixi/bin/accelerate", line 8, in <module>
    sys.exit(main())
  File "/opt/conda/envs/zhixi/lib/python3.9/site-packages/accelerate/commands/accelerate_cli.py", line 45, in main
    args.func(args)
  File "/opt/conda/envs/zhixi/lib/python3.9/site-packages/accelerate/commands/launch.py", line 906, in launch_command
    multi_gpu_launcher(args)
  File "/opt/conda/envs/zhixi/lib/python3.9/site-packages/accelerate/commands/launch.py", line 599, in multi_gpu_launcher
    distrib_run.run(args)
  File "/opt/conda/envs/zhixi/lib/python3.9/site-packages/torch/distributed/run.py", line 753, in run
    elastic_launch(
  File "/opt/conda/envs/zhixi/lib/python3.9/site-packages/torch/distributed/launcher/api.py", line 132, in __call__
    return launch_agent(self._config, self._entrypoint, list(args))
  File "/opt/conda/envs/zhixi/lib/python3.9/site-packages/torch/distributed/launcher/api.py", line 246, in launch_agent
    raise ChildFailedError(
torch.distributed.elastic.multiprocessing.errors.ChildFailedError:
===================================================
examples/generate_lora_web.py FAILED
---------------------------------------------------
Failures:
  <NO_OTHER_FAILURES>
---------------------------------------------------
Root Cause (first observed failure):
[0]:
  time      : 2023-09-26_10:14:23
  host      : b2896f53f126
  rank      : 1 (local_rank: 1)
  exitcode  : -9 (pid: 75)
  error_file: <N/A>
  traceback : Signal 9 (SIGKILL) received by PID 75
===================================================

As a beginner, I will appreciate it if you would response.

Hello, as this is a 13B model, it is recommended to use a GPU with more than 26GB of VRAM for inference. Since you are using a 24GB GPU, you will need at least two GPUs for multi-GPU inference. The inference command for multi-GPU setup is as follows:
CUDA_VISIBLE_DEVICES=0,1 python examples/generate_lora_web.py --base_model zjunlp/knowlm-13b-zhixi

Hello, multi-gpu inference requires using the first command. Regarding the error message, may I ask if your environment has been configured as required? You can first use the following command to check if it can run successfully:
CUDA_VISIBLE_DEVICES=0,1 python examples/generate_lora.py --base_model zjunlp/knowlm-13b-zhixi --run_general_cases

Hello, multi-gpu inference requires using the first command. Regarding the error message, may I ask if your environment has been configured as required? You can first use the following command to check if it can run successfully:
CUDA_VISIBLE_DEVICES=0,1 python examples/generate_lora.py --base_model zjunlp/knowlm-13b-zhixi --run_general_cases

Thank you! I tried your command and I got this:
(zhixi) root@b2896f53f126:/app/KnowLM# CUDA_VISIBLE_DEVICES=0,1 python examples/generate_lora.py --base_model ./zjunlp_knowlm-13b-zhixi --run_general_cases
testing general abilities!
load_8bit=False
Loading checkpoint shards: 100%|███████████████████████████████████████████████████████████████████████████████████████████████████████████████| 3/3 [02:47<00:00, 55.90s/it]
{'num_beams': 3, 'temperature': 0.1, 'repetition_penalty': 1.3} 请你背诵《静夜思》
GenerationConfig {
  "num_beams": 3,
  "repetition_penalty": 1.3,
  "temperature": 0.1,
  "top_k": 40,
  "top_p": 0.75,
  "transformers_version": "4.28.1"
}

Traceback (most recent call last):
  File "/app/KnowLM/examples/generate_lora.py", line 216, in <module>
    fire.Fire(main)
  File "/opt/conda/envs/zhixi/lib/python3.9/site-packages/fire/core.py", line 141, in Fire
    component_trace = _Fire(component, args, parsed_flag_args, context, name)
  File "/opt/conda/envs/zhixi/lib/python3.9/site-packages/fire/core.py", line 475, in _Fire
    component, remaining_args = _CallAndUpdateTrace(
  File "/opt/conda/envs/zhixi/lib/python3.9/site-packages/fire/core.py", line 691, in _CallAndUpdateTrace
    component = fn(*varargs, **kwargs)
  File "/app/KnowLM/examples/generate_lora.py", line 207, in main
    print(evaluate(instruction, num_beams=cfg["num_beams"], temperature=cfg["temperature"], repetition_penalty=cfg["repetition_penalty"]))
  File "/app/KnowLM/examples/generate_lora.py", line 192, in evaluate
    generation_output = model.generate(
  File "/opt/conda/envs/zhixi/lib/python3.9/site-packages/torch/autograd/grad_mode.py", line 27, in decorate_context
    return func(*args, **kwargs)
  File "/opt/conda/envs/zhixi/lib/python3.9/site-packages/transformers/generation/utils.py", line 1524, in generate
    return self.beam_search(
  File "/opt/conda/envs/zhixi/lib/python3.9/site-packages/transformers/generation/utils.py", line 2810, in beam_search
    outputs = self(
  File "/opt/conda/envs/zhixi/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1194, in _call_impl
    return forward_call(*input, **kwargs)
  File "/opt/conda/envs/zhixi/lib/python3.9/site-packages/accelerate/hooks.py", line 165, in new_forward
    output = old_forward(*args, **kwargs)
  File "/opt/conda/envs/zhixi/lib/python3.9/site-packages/transformers/models/llama/modeling_llama.py", line 687, in forward
    outputs = self.model(
  File "/opt/conda/envs/zhixi/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1194, in _call_impl
    return forward_call(*input, **kwargs)
  File "/opt/conda/envs/zhixi/lib/python3.9/site-packages/transformers/models/llama/modeling_llama.py", line 577, in forward
    layer_outputs = decoder_layer(
  File "/opt/conda/envs/zhixi/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1194, in _call_impl
    return forward_call(*input, **kwargs)
  File "/opt/conda/envs/zhixi/lib/python3.9/site-packages/accelerate/hooks.py", line 165, in new_forward
    output = old_forward(*args, **kwargs)
  File "/opt/conda/envs/zhixi/lib/python3.9/site-packages/transformers/models/llama/modeling_llama.py", line 292, in forward
    hidden_states, self_attn_weights, present_key_value = self.self_attn(
  File "/opt/conda/envs/zhixi/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1194, in _call_impl
    return forward_call(*input, **kwargs)
  File "/opt/conda/envs/zhixi/lib/python3.9/site-packages/accelerate/hooks.py", line 165, in new_forward
    output = old_forward(*args, **kwargs)
  File "/opt/conda/envs/zhixi/lib/python3.9/site-packages/transformers/models/llama/modeling_llama.py", line 196, in forward
    query_states = self.q_proj(hidden_states).view(bsz, q_len, self.num_heads, self.head_dim).transpose(1, 2)
  File "/opt/conda/envs/zhixi/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1194, in _call_impl
    return forward_call(*input, **kwargs)
  File "/opt/conda/envs/zhixi/lib/python3.9/site-packages/accelerate/hooks.py", line 165, in new_forward
    output = old_forward(*args, **kwargs)
  File "/opt/conda/envs/zhixi/lib/python3.9/site-packages/torch/nn/modules/linear.py", line 114, in forward
    return F.linear(input, self.weight, self.bias)
RuntimeError: CUDA error: CUBLAS_STATUS_NOT_INITIALIZED when calling `cublasCreate(handle)`

Hello, I suspect it's an environmental issue. If there is sufficient memory, I suggest running it on the CPU first to see if it throws any errors. I'm also conducting tests concurrently.

Hello, I suspect it's an environmental issue. If there is sufficient memory, I suggest running it on the CPU first to see if it throws any errors. I'm also conducting tests concurrently.

Thank you again for your response and efforts.
FYI, I just pulled the official image and made no change to the environment except that I replaced the repo with the one of latest version to avoid loading lora.

@BeasterYong cc
Hello, I'm not in charge of the Docker image. It can run smoothly following the steps here and I just tested it again a moment ago.

Sorry you had a problem using it! Have you installed nvidia-container-runtime? We will try to reproduce your problem and troubleshoot it soon.

@BeasterYong cc Hello, I'm not in charge of the Docker image. It can run smoothly following the steps here and I just tested it again a moment ago.

Thank you.

Sorry you had a problem using it! Have you installed nvidia-container-runtime? We will try to reproduce your problem and troubleshoot it soon.

Thank you, I'm looking forward to it!

Sorry you had a problem using it! Have you installed nvidia-container-runtime? We will try to reproduce your problem and troubleshoot it soon.

Thank you, I'm looking forward to it!

Hello, thank you very much for your issue, we have also reproduced the same issue. We are trying to troubleshoot the cause, and we recommend you to prefer non-docker deployment of the model~

This issue is caused by insufficient GPU memory resources, we recommend you to release the remaining processes that are consuming GPU resources, or use GPU resources with sufficient memory to deploy the model. It has been verified that the model reasoning runs normally after all the remaining processes have been cleared!

This issue is caused by insufficient GPU memory resources, we recommend you to release the remaining processes that are consuming GPU resources, or use GPU resources with sufficient memory to deploy the model. It has been verified that the model runs normally after all the remaining processes have been cleared!

This issue is caused by insufficient GPU memory resources, we recommend you to release the remaining processes that are consuming GPU resources, or use GPU resources with sufficient memory to deploy the model. It has been verified that the model runs normally after all the remaining processes have been cleared!

Thank you!


GraphStorm needs to check the input data and model config.

Traceback (most recent call last):
  File "gsgnn_ep.py", line 141, in <module>
    main(args)
  File "gsgnn_ep.py", line 111, in main
    trainer.fit(train_loader=dataloader, val_loader=val_dataloader,
  File "/graph-storm/python/graphstorm/trainer/ep_trainer.py", line 143, in fit
    loss = model(blocks, batch_graph, input_feats, None, lbl, input_nodes)
  File "/usr/local/lib/python3.8/dist-packages/torch/nn/modules/module.py", line 1194, in _call_impl
    return forward_call(*input, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/torch/nn/parallel/distributed.py", line 1040, in forward
    output = self._run_ddp_forward(*inputs, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/torch/nn/parallel/distributed.py", line 1000, in _run_ddp_forward
    return module_to_run(*inputs[0], **kwargs[0])
  File "/usr/local/lib/python3.8/dist-packages/torch/nn/modules/module.py", line 1194, in _call_impl
    return forward_call(*input, **kwargs)
  File "/graph-storm/python/graphstorm/model/edge_gnn.py", line 119, in forward
    pred_loss = self.loss_func(logits, labels[target_etype])
  File "/usr/local/lib/python3.8/dist-packages/torch/nn/modules/module.py", line 1194, in _call_impl
    return forward_call(*input, **kwargs)
  File "/graph-storm/python/graphstorm/model/loss_func.py", line 51, in forward
    return self.loss_fn(logits, labels)
  File "/usr/local/lib/python3.8/dist-packages/torch/nn/modules/module.py", line 1194, in _call_impl
    return forward_call(*input, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/torch/nn/modules/loss.py", line 1174, in forward
    return F.cross_entropy(input, target, weight=self.weight,
  File "/usr/local/lib/python3.8/dist-packages/torch/nn/functional.py", line 3026, in cross_entropy
    return torch._C._nn.cross_entropy_loss(input, target, weight, _Reduction.get_enum(reduction), ignore_index, label_smoothing)
RuntimeError: "nll_loss_forward_reduce_cuda_kernel_2d_index" not implemented for 'Int'

In my case I was using some old argument names in the YAML file when running training and the processes just exited without any error given.
We should have some parameter validation for the YAML files, including parameter names and values.


Death plot added

looks good to me. close https://github.com/SABS-R3-Epidemiology/epistrains/issues/66; close

Created test for the death plot
created test for saving and plotting
fixed return plt statement in make_plot (normal compartment plot)
added comments
Need to: check flake8


Trying to make work in headerless linux

Hi just learned about this app, looks great, thanks for putting it together, will be so much better than using SSH. I installed it onto a proxmox host hoping to get some details into HA. The problem is that after entering in all the details im not seeing any attempts to connect on the MQTT broker. Is there any troubleshooting steps you could recommend or any logs i can see from the app?
EDIT:
looking at previous issues i found the below command and the output from it (if you would prefer issues in the github please me know)
root@proxmox:~# systemctl --user status lnxlink
● lnxlink.service - LNXLink
Loaded: loaded (/root/.config/systemd/user/lnxlink.service; enabled; vendor preset: enabled)
Active: active (running) since Mon 2022-01-31 15:29:55 AEDT; 13s ago
Main PID: 96503 (python3)
Tasks: 1 (limit: 38084)
Memory: 16.9M
CPU: 109ms
CGroup<EMAIL_ADDRESS>└─96503 /usr/bin/python3 /opt/lnxlink/run.py
Jan 31 15:30:07 proxmox python3[96503]:     return _bootstrap._gcd_import(name[level:], package, level)
Jan 31 15:30:07 proxmox python3[96503]:   File "", line 1030, in _gcd_import
Jan 31 15:30:07 proxmox python3[96503]:   File "", line 1007, in _find_and_load
Jan 31 15:30:07 proxmox python3[96503]:   File "", line 986, in _find_and_load>
Jan 31 15:30:07 proxmox python3[96503]:   File "", line 680, in load_unlocked
Jan 31 15:30:07 proxmox python3[96503]:   File "", line 790, in exec>
Jan 31 15:30:07 proxmox python3[96503]:   File "", line 228, in _call_with_fra>
Jan 31 15:30:07 proxmox python3[96503]:   File "/opt/lnxlink/modules/media.py", line 1, in
Jan 31 15:30:07 proxmox python3[96503]:     from dbus.mainloop.glib import DBusGMainLoop
Jan 31 15:30:07 proxmox python3[96503]: ModuleNotFoundError: No module named 'dbus'
...skipping...
also dbus is installed
root@proxmox:~# sudo apt-get install dbus
Reading package lists… Done
Building dependency tree… Done
Reading state information… Done
dbus is already the newest version (1.12.20-2).
root@proxmox:~# pip3 install pydbus
Requirement already satisfied: pydbus in /usr/local/lib/python3.9/dist-packages (0.6.0)
tried installing sudo apt install python3-dbus
but then i get this error
● lnxlink.service - LNXLink
Loaded: loaded (/root/.config/systemd/user/lnxlink.service; enabled; vendor preset: enabled)
Active: active (running) since Mon 2022-01-31 20:32:09 AEDT; 50s ago
Main PID: 149058 (python3)
Tasks: 1 (limit: 38084)
Memory: 20.1M
CPU: 136ms
CGroup<EMAIL_ADDRESS>└─149058 /usr/bin/python3 /opt/lnxlink/run.py
Jan 31 20:33:00 proxmox python3[149058]:     from pynput import keyboard
Jan 31 20:33:00 proxmox python3[149058]:   File "/usr/local/lib/python3.9/dist-packages/pynput/init.py", line 40, in
Jan 31 20:33:00 proxmox python3[149058]:     from . import keyboard
Jan 31 20:33:00 proxmox python3[149058]:   File "/usr/local/lib/python3.9/dist-packages/pynput/keyboard/init.py", line 31, in
Jan 31 20:33:00 proxmox python3[149058]:     backend = backend(name)
Jan 31 20:33:00 proxmox python3[149058]:   File "/usr/local/lib/python3.9/dist-packages/pynput/_util/init.py", line 76, in backend
Jan 31 20:33:00 proxmox python3[149058]:     raise ImportError('this platform is not supported: {}'.format(
Jan 31 20:33:00 proxmox python3[149058]: ImportError: this platform is not supported: ('failed to acquire X connection: Bad display name ""', DisplayNameError(''))
Jan 31 20:33:00 proxmox python3[149058]: Try one of the following resolutions:
Jan 31 20:33:00 proxmox python3[149058]:  * Please make sure that you have an X serve

As you suggested i have tried deleting these files:
send_keys.py
media.py
notify.py
and now get this error. Thanks again
● lnxlink.service - LNXLink
Loaded: loaded (/root/.config/systemd/user/lnxlink.service; enabled; vendor preset: enabled)
Active: failed (Result: exit-code) since Wed 2022-02-02 10:35:55 AEDT; 1s ago
Process: 1690 ExecStart=/usr/bin/python3 /opt/lnxlink/run.py (code=exited, status=1/FAILURE)
Main PID: 1690 (code=exited, status=1/FAILURE)
CPU: 144ms
Feb 02 10:35:55 proxmox systemd[1664]: lnxlink.service: Scheduled restart job, restart counter is at 5.
Feb 02 10:35:55 proxmox systemd[1664]: Stopped LNXLink.
Feb 02 10:35:55 proxmox systemd[1664]: lnxlink.service: Start request repeated too quickly.
Feb 02 10:35:55 proxmox systemd[1664]: lnxlink.service: Failed with result 'exit-code'.
Feb 02 10:35:55 proxmox systemd[1664]: Failed to start LNXLink.

It worked for me after I deleted the files you mentioned and removed the entries from /opt/lnxlink/config.yaml

thanks for the tip I just removed them from the config.yaml in the control and monitoring but still get the same error. I restarted the service after saving changes. the only other thing different i have done is that i had to install python3-dbus

after trying a few time the error only occurs after i delete the send_keys.py, deleting the other 2 files is fine. The error also doesnt occur if i just rename the send_keys.py.

IMHO deleting or renaming should have the same effect. Maybe it was just a coincidence. I have some errors with the network module. I get errors like KeyError: 'network' or AttributeError: 'LNXlink' object has no attribute 'Addons' when starting lnxlink. Different behaviour each time I start it even though I didn't change anything. Must be some race condition, maybe something mqtt related.

it works now running it manually like this
systemctl --user stop lnxlink.service
cd /opt/lnxlink
/usr/bin/python3 /opt/lnxlink/run.py

found that when i tried to change this value it would throw a network error
statsPrefix: windows-monitor/stats

Seems that the issue has been resolved, so I am closing it.


What's the minimum nodejs version  supported.

I want to confirm if it supports this version，and my nodejs version is v0.10.48 now.

This module supports node >= 0.6
Your version is supported.

Thank you very much.


Wait until exists

Same as #6, with the latest aws-sdk-go and targeting develop.
@ljfranklin

Thanks @jmcarp, running it through CI now!


"Font metrics not found for font" error

Try typesetting the randomly chosen unicode symbol 𝕭 (this can be tested e.g. in https://katex.org/)
KaTeX will produce an error:
Font metrics not found for font: .

This is a bit brutal. Maybe KaTeX could handle this more gracefully?

I think we can do better than just an improved error message. See PR #3777.

thanks! looking good

would be good if #3777 got approved and merged, just saying...


[Feature] Put some instructions in the read me like right click to move

[Feature] Put some instructions in the read me like right click to move

Good feedback! Will make sure to update the README :)

Instructions have now been added!


Issue with indentation in rails

Some issue makes the rule
rule a
  [a-z] {
    def b
      0
    end
  }
end

throw ArgumentError (Negative Argument) in Treetop.load.
while the same code indendated slightly altered is valid:
rule a
  [a-z] {
  def b
    0
  end
  }
end

This problem occures in rspecs when I require 'rails_helper', but not when I only require 'treetop' on its own. I unfortunately do not have the time for more testing right now, but I will leave you my Gemfile.lock for information.
https://gist.github.com/bonflintstone/b915b5ee52ccac73ebffa292bd229d5e

Sorry, insufficient clues. I cant see a problem with any of that. How about sending a stack trans and a copy of your rails_helper.rb, at least?

     Failure/Error: Treetop.load(Rails.root.join('app', 'grammars', 'document.treetop').to_s)

     ArgumentError:
       negative argument
     # /home/bonflintstone/.rvm/gems/ruby-2.3.0/gems/treetop-1.4.12/lib/treetop/ruby_extensions/string.rb:25:in `tabto'
     # /home/bonflintstone/.rvm/gems/ruby-2.3.0/gems/treetop-1.4.12/lib/treetop/compiler/ruby_builder.rb:17:in `<<'
     # /home/bonflintstone/.rvm/gems/ruby-2.3.0/gems/treetop-1.4.12/lib/treetop/compiler/node_classes/inline_module.rb:18:in `block in compile'
     # /home/bonflintstone/.rvm/gems/ruby-2.3.0/gems/treetop-1.4.12/lib/treetop/compiler/ruby_builder.rb:26:in `indented'
     # /home/bonflintstone/.rvm/gems/ruby-2.3.0/gems/treetop-1.4.12/lib/treetop/compiler/ruby_builder.rb:38:in `module_declaration'
     # /home/bonflintstone/.rvm/gems/ruby-2.3.0/gems/treetop-1.4.12/lib/treetop/compiler/node_classes/inline_module.rb:17:in `compile'
     # /home/bonflintstone/.rvm/gems/ruby-2.3.0/gems/treetop-1.4.12/lib/treetop/compiler/node_classes/parsing_rule.rb:12:in `block in compile_inline_module_declarations'
     # /home/bonflintstone/.rvm/gems/ruby-2.3.0/gems/treetop-1.4.12/lib/treetop/compiler/node_classes/parsing_rule.rb:11:in `each'
     # /home/bonflintstone/.rvm/gems/ruby-2.3.0/gems/treetop-1.4.12/lib/treetop/compiler/node_classes/parsing_rule.rb:11:in `each_with_index'
     # /home/bonflintstone/.rvm/gems/ruby-2.3.0/gems/treetop-1.4.12/lib/treetop/compiler/node_classes/parsing_rule.rb:11:in `compile_inline_module_declarations'
     # /home/bonflintstone/.rvm/gems/ruby-2.3.0/gems/treetop-1.4.12/lib/treetop/compiler/node_classes/parsing_rule.rb:6:in `compile'
     # /home/bonflintstone/.rvm/gems/ruby-2.3.0/gems/treetop-1.4.12/lib/treetop/compiler/node_classes/declaration_sequence.rb:14:in `block in compile'
     # /home/bonflintstone/.rvm/gems/ruby-2.3.0/gems/treetop-1.4.12/lib/treetop/compiler/node_classes/declaration_sequence.rb:13:in `each'
     # /home/bonflintstone/.rvm/gems/ruby-2.3.0/gems/treetop-1.4.12/lib/treetop/compiler/node_classes/declaration_sequence.rb:13:in `compile'
     # /home/bonflintstone/.rvm/gems/ruby-2.3.0/gems/treetop-1.4.12/lib/treetop/compiler/node_classes/grammar.rb:11:in `block in compile'
     # /home/bonflintstone/.rvm/gems/ruby-2.3.0/gems/treetop-1.4.12/lib/treetop/compiler/ruby_builder.rb:26:in `indented'
     # /home/bonflintstone/.rvm/gems/ruby-2.3.0/gems/treetop-1.4.12/lib/treetop/compiler/ruby_builder.rb:38:in `module_declaration'
     # /home/bonflintstone/.rvm/gems/ruby-2.3.0/gems/treetop-1.4.12/lib/treetop/compiler/node_classes/grammar.rb:7:in `compile'
     # /home/bonflintstone/.rvm/gems/ruby-2.3.0/gems/treetop-1.4.12/lib/treetop/compiler/metagrammar.rb:39:in `compile'
     # /home/bonflintstone/.rvm/gems/ruby-2.3.0/gems/treetop-1.4.12/lib/treetop/compiler/grammar_compiler.rb:24:in `ruby_source_from_string'
     # /home/bonflintstone/.rvm/gems/ruby-2.3.0/gems/treetop-1.4.12/lib/treetop/compiler/grammar_compiler.rb:42:in `load_from_string'
     # /home/bonflintstone/.rvm/gems/ruby-2.3.0/gems/treetop-1.4.12/lib/treetop/compiler/grammar_compiler.rb:35:in `block in load'
     # /home/bonflintstone/.rvm/gems/ruby-2.3.0/gems/treetop-1.4.12/lib/treetop/compiler/grammar_compiler.rb:32:in `open'
     # /home/bonflintstone/.rvm/gems/ruby-2.3.0/gems/treetop-1.4.12/lib/treetop/compiler/grammar_compiler.rb:32:in `load'
     # ./spec/grammars/document_treetop.spec.rb:5:in `block (2 levels) in <top (required)>'

ENV['RAILS_ENV'] ||= 'test'
require File.expand_path('../../config/environment', __FILE__)
# Prevent database truncation if the environment is production
abort("The Rails environment is running in production mode!") if Rails.env.production?
require 'spec_helper'
require 'rspec/rails'
# Add additional requires below this line. Rails is not loaded until this point!

# Requires supporting ruby files with custom matchers and macros, etc, in
# spec/support/ and its subdirectories. Files matching `spec/**/*_spec.rb` are
# run as spec files by default. This means that files in spec/support that end
# in _spec.rb will both be required and run as specs, causing the specs to be
# run twice. It is recommended that you do not name files matching this glob to
# end with _spec.rb. You can configure this pattern with the --pattern
# option on the command line or in ~/.rspec, .rspec or `.rspec-local`.
#
# The following line is provided for convenience purposes. It has the downside
# of increasing the boot-up time by auto-requiring all files in the support
# directory. Alternatively, in the individual `*_spec.rb` files, manually
# require only the support files necessary.
#
# Dir[Rails.root.join('spec/support/**/*.rb')].each { |f| require f }

# Checks for pending migration and applies them before tests are run.
# If you are not using ActiveRecord, you can remove this line.
ActiveRecord::Migration.maintain_test_schema!

RSpec.configure do |config|
  # Remove this line if you're not using ActiveRecord or ActiveRecord fixtures
  config.fixture_path = "#{::Rails.root}/spec/fixtures"

  # If you're not using ActiveRecord, or you'd prefer not to run each of your
  # examples within a transaction, remove the following line or assign false
  # instead of true.
  config.use_transactional_fixtures = true

  # RSpec Rails can automatically mix in different behaviours to your tests
  # based on their file location, for example enabling you to call `get` and
  # `post` in specs under `spec/controllers`.
  #
  # You can disable this behaviour by removing the line below, and instead
  # explicitly tag your specs with their type, e.g.:
  #
  #     RSpec.describe UsersController, :type => :controller do
  #       # ...
  #     end
  #
  # The different available types are documented in the features, such as in
  # https://relishapp.com/rspec/rspec-rails/docs
  config.infer_spec_type_from_file_location!

  # Filter lines from Rails gems in backtraces.
  config.filter_rails_from_backtrace!
  # arbitrary gems may also be filtered via:
  # config.filter_gems_from_backtrace("gem name")
end

There is nothing particular special going on in my app, I will try to repeat the error with some minimal rails app I can send you

https://github.com/bonflintstone/treetop-issue
There you go, tell me if you need any more information. All I did for this app was installing treetop and rspec,  initializing rspec and creating the files under app/grammars and spec/grammars
Thanks a lot!

Treetop is now version 1.6.8. You are using treetop version 1.4.12.
Update, and you'll find that this error does not occur.
For what it's worth, the error occurs in a previous version of this line.
(* is the method which gets a negative argument)
https://github.com/cjheath/treetop/blob/master/lib/treetop/ruby_extensions/string.rb#L28
This problem was fixed years ago.

I see, sorry for bothering you. Thank you for checking!
On Fri, Aug 12, 2016 at 8:31 AM, Clifford Heath<EMAIL_ADDRESS>wrote:

Treetop is now version 1.6.8. You are using treetop version 1.4.12.
Update, and you'll find that this error does not occur.
For what it's worth, the error occurs in a previous version of this line.
(* is the method which gets a negative argument)
https://github.com/cjheath/treetop/blob/master/lib/
treetop/ruby_extensions/string.rb#L28
This problem was fixed years ago.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nathansobo/treetop/issues/43#issuecomment-239371431,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOdD9iVodMt0h5AYWkS9yF4WqyjrW2GLks5qfBNdgaJpZM4JiN45
.


Update manifest.webmanifest

👷 Deploy Preview for famous-marshmallow-cfbd61 processing.

Name
Link

🔨 Latest commit
207279087b94f12f79b3f6872d1ffca79c8d368d

🔍 Latest deploy log
https://app.netlify.com/sites/famous-marshmallow-cfbd61/deploys/64a4a40c27a0420008359952


Cache?

Hi,
looks like a great repo, thanks in advance for that!
By the way, can it be cached?

Hey,
glad you like my work!
If you look for (GitHub-) caching, you might want to take a look at this actions file.
As I am not able to call other actions inside my action (or at least this was the case some time ago), you'll need to add the caching step to your actions file manually. From the file linked earlier (minimal example):
# Retrieve the cache, uses cache@v2, you may want to change that as this version may be quite old
- name: Cache boost
  uses: actions/cache@v2
  id: cache-boost
  with:
    # Set the default path as the path to cache (or at least this should be the default path...I think)
    # If you want to change this, check the documentation on how to do that
    path: ${{env.GITHUB_WORKSPACE}}/boost/boost
    # Use the version as the key to only cache the correct version
    key: boost-${{BOOST_VERSION}}

# Actual install step (only runs if the cache is empty)
- name: Install boost
  if: steps.cache-boost.outputs.cache-hit != 'true'
  uses<EMAIL_ADDRESS>  with:
    # Set the boost version (required)
    boost_version: ${{BOOST_VERSION}}

I hope this helps and if you have any further questions, please let me know.
If this was not what you were searching for, feel free to re-open this issue.
Have a nice day!

Hmm, by the way, if boost is cached, how do I know the output path from the step? Or should I just assume it's always workspacedir/boost/boost?

If you are referring to this action, the default install location is ${{env.GITHUB_WORKSPACE}}/boost/boost,  but this can be changed by setting the boost_install_dir option in your actions file. So if you want to change the install directory, I'd recommend setting an environment variable for setting the path for both actions/cache and this action. Although the BOOST_ROOT output variable gets changed accordingly if you choose to change the install directory, in the setup I suggested (which can only work this way as you'll need to know if boost was found in the cache), this action runs after actions/cache, thus, leaving BOOST_ROOT unset once the cache step runs.
If you are referring to actions/cache, that action always stores the data retrieved from the cache where it was originally located (read more about that here). So if you set the path argument to ${{env.GITHUB_WORKSPACE}}/boost/boost, the cache will always be restored to that path. If you change this, the cache will be restored to the directory you specified. But you may want to test different directory setups as I don't know how actions/cache handles subdirectories and non-existent ones, as the last time I tried this was about a year ago (with a different setup), but this should work, theoretically.
But again, remember if you set the boost_install_dir parameter of this action, you'll have to adapt the path parameter of actions/cache accordingly. So your actions file may actually look like this (using environment variables):
jobs:
  build:
    runs-on: windows-latest
    env:
      # Set your boost version
      BOOST_VERSION: 1.77.0
      # Set you boost path to the default one (I don't know if you can use variables here)
      BOOST_PATH: ${{env.GITHUB_WORKSPACE}}/boost/boost

    steps:
    - uses: actions/checkout@v2

    # Additional steps...

    # Retrieve the cache, uses cache@v2, you may want to change that as this version may be quite old
    - name: Cache boost
      uses: actions/cache@v2
      id: cache-boost
      with:
        # Set the path to cache
        path: ${{env.BOOST_PATH}}
        # Use the version as the key to only cache the correct version
        key: boost-${{env.BOOST_VERSION}}

    # Actual install step (only runs if the cache is empty)
    - name: Install boost
      if: steps.cache-boost.outputs.cache-hit != 'true'
      uses<EMAIL_ADDRESS>      with:
        # Set the boost version (required)
        boost_version: ${{env.BOOST_VERSION}}
        # Set the install directory
        boost_install_dir: ${{env.BOOST_PATH}}


Fix TerraformTest-Modules Account in Control Tower

When migrating TerraformModules-Test from the Staging to the Sandbox OU the enrollment broke as it tried to re-enroll the account but all the resources were already existing from the previous enrollment.
So in order to fix this we probably need to do the following:

[ ] Unenroll the account from the service catalogue.

You can do this by terminating the account in the service catalogue

[ ] Re-enroll the account in the new OU

You do this in control tower.

[ ] Re-run AFT Customizations

Terminated the account in the service catalogue

Update: following docs regarding config recorder and delivery channel


support for auto paginating variants of paginatable methods

fixes #1319
impl xxx_pages fns as default trait methods
add pagination methods to result and request struct for getting and setting tokens
added trait bounds on service traits required for ^^
create a type alias for RusotoStream
based on work by softprops<EMAIL_ADDRESS>Please help keep the CHANGELOG up to date by providing a one sentence summary of your change:

The tests are failing as I only checked in the S3 generated file as an expample to make reviewing it easier. I'll generate and commit the rest if you are happy with this approach.


Add shortcode

with attribute for product id

Hint: Are you waiting for the same feature to be implemented? Please react with ❤ to this issue description (features with more reactions will get higher priority)

Hi, I just downloaded your plugin, and unfortunately, it's not working on my product page because I am using a theme that is probably incompatible with your plugin. I can edit product page manually with a plugin I am using but I need a shortcode of your plugin. I tried [wc_price_history] but it's not working. Are you planning to make it work? Thank you for your update!

this fix is not merged yet (you can see this looking on the right sidebar here, and there is green icon with two arrows which are not connected yet) not released.
for time being you can try to use this code to show price:
$storage = new \PriorPrice\HistoryStorage();
echo $storage->get_minimal( $product_id );

in place of $product_id of course provide the product ID

Thank you! Since I am not coder and I don't really know what to do with the code you provided. I will wait for the shortcode. Thank you :)

@Kubeec108 it is merged now and will be released today if I didn't make any troubles :) Please look for 1.2 release in your wp admin.
when updated, you will be able to use this shortcode as follows:

If you want to display the lowest products price in other place than default, you can use shortcode wc_price_history. A few examples:
Display the lowest price on single product page (without passing product ID as argument), currency symbol attached:
This product low is [wc_price_history]

Display the lowest price of the other product, currency symbol attached:
The product with ID 3 had the lowest price [wc_price_history id=3]

Display without currency symbol:
The product with ID 3 had the lowest price [wc_price_history id=3 show_currency=0]

Thank you for amazing work!


how to run demo?

i can not find test_pipeline in the code

We will accomplish the readme (including datasets, commands, and results) in a few months. You can refer to BasicSR to use the following commands for efficient SR.
cd codes
python basicsr/test.py -opt ../options/test/test_x4.yml


Adds a shadow node implementation for sliders & autolinking

What's broken?
When we enable fabric all of our sliders have zero height and aren't interactable as a result. This works in old arch because there's a custom shadow node implementation in java that allocates a slider, measures it, and provides the measurements.
How'd we fix it?
Inspired by the switch shadow node implementation in react native core we added a cpp shadow node to measure the slider to duplicate what was previously happening with the old architecture
How do I test this? (in discord)

Build with new arch enabled
Go to voice setting under profile
Try to slide the volume sliders

no issues on old arch right?
🔥

Yep all good on old arch!


fix no available capacity issure

Isn't this more or less the same as the closed https://github.com/ljedrz/pea2pea/pull/8? You haven't responded to my reasoning there (where I mentioned those limits can easily just be extended as much as you want in your Config), and I'd rather discuss introducing changes that aren't trivially resolved with adjusting configuration.
That being said, I might actually use an UnboundedChannel for the channels inside the protocols (in the handler objects), as the growth of their depth is bounded by the connection limits. If you'd like to perform this change just there (and without changes to docs), I'd likely accept it.

By testing mpsc::unbounded_channel performs better than mpsc::channel in this Pea2Pea ，
mpsc::unbounded_channel test result：

mpsc::channel  test result：

test code：
use bytes::Bytes;
use once_cell::sync::Lazy;
use rand::{distributions::Standard, rngs::SmallRng, Rng, SeedableRng};
use tokio::time::sleep;
use tracing::*;
mod common;
use pea2pea::{
protocols::{Reading, Writing},
Config, Node, Pea2Pea,
};
use std::time::Duration;
use std::{convert::TryInto, io, net::SocketAddr, time::Instant};
const NUM_MESSAGES: usize = 10_000;
const MSG_SIZE: usize = 128 * 1024;
static RANDOM_BYTES: Lazy = Lazy::new(|| {
Bytes::from(
(&mut SmallRng::from_entropy())
.sample_iter(Standard)
.take(MSG_SIZE - 2)
.collect::<Vec<_>>(),
)
});
#[derive(Clone)]
pub struct P2PNode(pub Node);
impl Pea2Pea for P2PNode {
fn node(&self) -> &Node {
&self.0
}
}
#[async_trait::async_trait]
impl Reading for P2PNode {
type Message = bytes::Bytes;
fn read_message<R: io::Read>(
    &self,
    _source: SocketAddr,
    reader: &mut R,
) -> io::Result<Option<Self::Message>> {
    let vec = crate::common::read_len_prefixed_message::<R, 2>(reader)?;
    Ok(vec.map(bytes::Bytes::from))
}

async fn process_message(&self, _src: SocketAddr, _msg: Self::Message) -> io::Result<()> {
    Ok(())
}

}
impl Writing for P2PNode {
type Message = bytes::Bytes;
fn write_message<W: io::Write>(
    &self,
    _target: SocketAddr,
    payload: &Self::Message,
    writer: &mut W,
) -> io::Result<()> {
    writer.write_all(&(payload.len() as u16).to_le_bytes())?;
    writer.write_all(payload)
}

}
impl P2PNode {
pub async fn new<T: Into>(name: T) -> Self {
let config = Config {
name: Some(name.into()),
..Default::default()
};
Self(Node::new(Some(config)).await.unwrap())
}
fn send_periodic_broadcasts(&self) {
    let self_clone = self.clone();
    tokio::spawn(async move {
        loop {
            if self_clone.node().num_connected() != 0 {
                self_clone.send_broadcast(RANDOM_BYTES.clone()).unwrap();
            } else {
                info!(parent: self_clone.node().span(), "meh, I have no frens to chat with",);
            }
            sleep(Duration::from_micros(1)).await;
        }
    });
}

}
#[tokio::test]
async fn broadcast_benches_example() {
// tracing_subscriber::fmt::init();
// println!("{}", RANDOM_BYTES.len());
let random_nodes = common::start_nodes(4, None)
    .await
    .into_iter()
    .map(P2PNode)
    .collect::<Vec<_>>();
for rando in &random_nodes {
    rando.enable_reading().await;
}

let broadcaster = P2PNode::new("chatty").await;

broadcaster.enable_writing().await;
broadcaster.send_periodic_broadcasts();

for rando in &random_nodes {
    broadcaster
        .0
        .connect(rando.node().listening_addr().unwrap())
        .await
        .unwrap();
}

let now = std::time::Instant::now();
loop {
    tokio::time::sleep(std::time::Duration::from_millis(1)).await;
    if now.elapsed() > std::time::Duration::from_secs(60) {
        break;
    }
}

let r = broadcaster.node().stats().sent();
println!("broadcaster send msgs({}) bytes({})", r.0, r.1);
for rando in &random_nodes {
    let r = rando.node().stats().received();
    println!("receiver receive msgs({}) bytes({})", r.0, r.1);
}

}

Yes, and adjusting the Config would have the same effect.

I might actually use an UnboundedChannel for the channels inside the protocols (in the handler objects), as the growth of their depth is bounded by the connection limits.

Implemented in https://github.com/ljedrz/pea2pea/commit/cacf7904af8e60befbdda7f9db9b0d8d8e1b6e73.

Closing due to the aforementioned reasons and lack of activity; as mentioned, one of the components of the PR was merged.


feat: simple cli test for ng client backup&restore

Tests still failing, I'd also recommend putting an info! log before every section so one roughly knows where things go wrong. The error looked like an e-cash reissue failure at first glance because that's not the case yet.

@elsirion Changing to a fixed operation id for mint restoration broke subsequent restoration. First one works. Following ones don't (or maybe they will, but waiting for them finishes prematurely).
Looking at it, I don't think hardcoding the ID is a right approach. ClientModule::restore could return Vec<OperationId>, and then the restore cli could wait for all of them, or something like that. Special casing mint restoration in the general client doesn't feel right.

@elsirion I wanted something like 3cc89e3fa6c607e4f4a87016857009985937c050 , but I don't know if we have any way to get notifications for subscriptions for all the modules?

@elsirion Changing to a fixed operation id for mint restoration broke subsequent restoration. First one works. Following ones don't (or maybe they will, but waiting for them finishes prematurely).

Doesn't that just mean that we should probably completely clear our DB before restoring? Restoring with old state still being around sounds like a bad idea anyway.

Completely wiping the DB sounds reasonable, we don't really want to get into conflict resolution between old and new state. How to best do that is tbd imo (add a clear db command to API, let the user figure it out, …)

Moved to #2655 2655