Can't record audio in PCM format

When I try to record with -a -P pcm_s16le (or any other PCM format), it gives me an error saying

Failed to read from PulseAudio stream: Invalid argument

I can record with ffmpeg -f pulse -i default -c:a pcm_s16le test.mkv so it's not a ffmpeg issue.
It has no issue recording in opus.
inxi -SMAxxx:
System:
  Host: getgot Kernel: 6.6.4-1-cachyos-lto arch: x86_64 bits: 64
    compiler: clang v: 16.0.6 clocksource: tsc Desktop: sway
    v: 1.9-dev-ae3acf26f info: waybar vt: 1 dm: SDDM Distro: Arch Linux
Machine:
  Type: Desktop System: ASUS product: N/A v: N/A serial: <superuser required>
  Mobo: ASUSTeK model: TUF GAMING B550-PLUS v: Rev X.0x
    serial: <superuser required> UEFI: American Megatrends v: 2803
    date: 04/27/2022
Audio:
  Device-1: AMD Navi 21/23 HDMI/DP Audio driver: snd_hda_intel v: kernel pcie:
    speed: 16 GT/s lanes: 16 bus-ID: 08:00.1 chip-ID: 1002:ab28 class-ID: 0403
  Device-2: AMD Starship/Matisse HD Audio vendor: ASUSTeK
    driver: snd_hda_intel v: kernel pcie: speed: 16 GT/s lanes: 16
    bus-ID: 0a:00.4 chip-ID: 1022:1487 class-ID: 0403
  API: ALSA v: k6.6.4-1-cachyos-lto status: kernel-api
  Server-1: sndiod v: N/A status: off
  Server-2: PipeWire v: 1.0.0 status: active with: 1: pipewire-pulse
    status: active 2: wireplumber status: active 3: pipewire-alsa type: plugin
    4: pw-jack type: plugin

which version of wf-recorder are you using at the time?
Providing -P like you report seems to be incorrect, I get an error "Invalid codec option pcm_s16le". I think the -P param should be in the form of -P <option_name>=<option_value> (according to the man page).
Otherwise, can you also try running with wf-recorder --log <other params> to get more insights on what's happening.
Hope it helps!

which version of wf-recorder are you using at the time?

0.4.1

Providing -P like you report seems to be incorrect, I get an error "Invalid codec option pcm_s16le". I think the -P param should be in the form of -P <option_name>=<option_value> (according to the man page).

yeah that seems to be a copy paste issue, it's supposed to be -a -C pcm_s16le

Otherwise, can you also try running with wf-recorder --log <other params> to get more insights on what's happening.

unfortunately did not provide me any info other than this:
Choosing sample format s16 for audio codec pcm_s16le
[SWR @ 0x77ff2442c3c0] Using fltp internally between filters

which ffmpeg should be able to handle just fine

Hope it helps!

thank you
flac seems to kinda work so i think ill use that, the cpu time cost is negligible


Revamp and extend the deployment docs / tutorials to reflect real world use cases

Adding Envoy to one's stack requires deployment and maintenance of one more service. Revamped and extended deployment docs would make it easier to understand what the true cost of Envoy is, compared to the cost of maintaining one or more databases per user with Cloudant.
Wishlist:

More details on how to get Envoy running reliably in IBM Cloud (The Deploy-to-bluemix button is currently broken, and there are no links to other relevant docs on the subject)
Include an example how provisioning of new Envoy users is done
Mention https://github.com/glynnbird/envoy-serverless as an option where "live" replication is not necessary
A docker-compose.yml example for getting Envoy and Cloudant Local Developer edition running locally so that Envoy can be tested easier
Scaling caveats explained (does Envoy scale well - performance-wise? Cost-wise?)

Great suggestions! Thanks @motin . I think I'll break this down into a number of issues:

deploy to  IBM Cloud broken
link to Envoy Serverless
Docker/Container deployment
Scaling

The documentation side might be best tackled in the Wiki?


Question about long input and difference between streaming-llm and dense attention.

Thank you for your nice work and I have read the issue #33, thank you for your patient explanation for the difference between StreamingLLM and Dense Attention, based on your answer, I have futher question:

As you mentioned in FAQ.3, I guess streaming-llm processes long input in a truncation manner. But if I don't mind the expensive time consuming and handle the long input text in a streaming-llm manner (recompuation from the very begining of the text to the end with attention sinks window by window), theoretically, will it perform better than truncation? do you perform some experiments?

For a model with large context size (say 16k), we can still implement streaming llm use shorter context size(say 4k), have you perform comparison between dense attention with 16k and streaming llm with 4k? Theoretically, streaming-4k should perform worse than dense-16k, what's the gap? this is important if one want to use the streaming-llm to approximate larger window size performance.

Hello,
Thank you for your thoughtful questions. Let's delve into the details:

Regarding streaming-llm processing with truncation vs. re-computation:

In our paper, we have results that touch on this topic. The baseline you're referring to is the "sliding window with re-computation." If you refer to Figure 3 of our paper, you'll see that StreamingLLM's perplexity is in line with this baseline. So, in essence, StreamingLLM performs comparably when handling long inputs in the manner you described.

Comparison between dense attention at 16k and StreamingLLM at 4k:

Firstly, dense attention is designed to function within its pre-training range. So, for the example you provided, it operates within the 0-16K range.
Within the [0, 4k] range, dense attention and StreamingLLM have equivalent perplexity scores.
For the range [4K, 16K], dense attention is likely to outperform StreamingLLM because it retains information from previous tokens, thus having a broader context. This gap should be more apparent when the relevant information is evicted from the window of StreamingLLM.
Beyond the 16K mark (i.e., [16K, ∞]), the dense attention model won't be operational, whereas StreamingLLM will continue to function.

I hope this helps explain the questions you asked.
Guangxuan

@Guangxuan-Xiao Got it. thank you for the answer. btw, hope to see some quantitatively results of the gap in question 2. : )


[Bug] Cannot use non-default value for pulsar.service url

Search before asking

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

Version
Pulsar 3.1
Tested in both MacOS and Docker-compose
Minimal reproduce step
Follow the steps in
https://pulsar.apache.org/docs/3.1.x/io-cdc-debezium/#configuration-2
but change pulsar.service.url to something else, e.g. pulsar://pulsar-server:6650 for docker-compose
What did you expect to see?
Its is always "pulsar://localhost:6650" no matter what I change the config to…
I can see logs:
2023-09-28T16:31:36,502-0700 [pulsar-client-io-1-3] INFO  org.apache.pulsar.client.impl.ConsumerStatsRecorderImpl - Pulsar client config: {"serviceUrl":"pulsar://localhost:6650","authPluginClassName":null,"authParams":null,"authParamMap":null,"operationTimeoutMs":30000,"lookupTimeoutMs":30000,"statsIntervalSeconds":60,"numIoThreads":8,"numListenerThreads":8,"
and
2023-09-28 16:33:46 2023-09-28T23:33:46,136+0000 [pulsar-client-io-1-2] WARN  org.apache.pulsar.client.impl.ConnectionPool - Failed to open connection to localhost/:6650 : io.netty.channel.AbstractChannel$AnnotatedConnectException: Connection refused: localhost/<IP_ADDRESS>:6650
I need to change the url to something else so that I can work in docker-compose or production.
Thanks!
What did you see instead?
I had to use network_mode: host for now as workaround in https://github.com/xdblab/xdb/pull/25
Anything else?
No response
Are you willing to submit a PR?

[X] I'm willing to submit a PR!

Not sure yet for 100%, but seems related https://github.com/apache/pulsar-client-node/issues/343#issuecomment-1872903969

I also encountered the same problem，

Do you have any good solutions on your end


Set default service.name and telemetry.sdk.* group according to spec

According to the spec, the resource attribute service.name must be set to a default value if not configured by the use:
https://opentelemetry.io/docs/specification/otel/resource/semantic_conventions/#service
And the telemetry SDK must attach various self-describing resource attributes:
https://opentelemetry.io/docs/specification/otel/resource/semantic_conventions/#telemetry-sdk
These are both called out in Attributes with special handling.
To me personally, these seem quite noisy and not universally useful. I'd love to leave this in the hands of the user, but on reading the spec today it doesn't seem that it would be the appropriate approach. Please prove me wrong if anyone can :-)

Aren't telemetry-sdk attributes only added when you call AddTelemetrySdk on the ResourceBuilder? At least that's what I experience over here.

Could be - but that seems out of sync with the spec, which is pretty clear on the group being mandatory 🤔

Could be - but that seems out of sync with the spec, which is pretty clear on the group being mandatory 🤔

That's so weird man.... you might wanna double check this with the folks in the contrib repo just in case.
Having an explicit method to add it while the spec is saying it is mandatory is such a big conflict.

Thanks for taking a look! I'll try to find some supporting info 👍

https://github.com/open-telemetry/opentelemetry-dotnet/pull/4369

For these cases, I think that the OpenTelemetry sink falls in an odd location. Those requirements are really aimed at developers of full SDKs for particular programming languages. It makes sense that the full .NET OpenTelemetry SDK is following them. However, this sink is/will not be a full SDK; it only sends data provided by the developer to an OpenTelemetry endpoint.
I'd prefer to leave the decision to include these attributes to the developer and not insert any attributes of dubious value into the payload.

Thanks for the reply @loomis-relativity 👍
I drew much the same conclusion about the dubious value of some of the attributes. More of these might be on the way as the spec grows, so I'm thinking about how to set consumers up for success with whatever collectors/targets might expect, while leaving some room to avoid payload bloat 🤔
I would personally turn the telemetry.sdk.* group attributes off in the environments I currently work in, whether from the .NET SDK or from this library, but I'd also assume that the sink would be conservative and align as closely to the spec as is reasonable by default. Some collectors might assume the presence of these attributes based on the spec, though, and degrade experience when they're absent. The spec doesn't really leave any room for the idea of "not using an SDK" (which is strange IMHO, but I'm not the one driving the spec ..).
What about if we were to use the IncludedData mechanism to control this? E.g. in the IncludedData flags, we'd include a new member SpecRequiredResourceAttributes, and only fill in missing service.name or telemetry.sdk.* attributes when the flag is set?
The sink would (I assume) include this flag in the default set, but we'd demonstrate leaving it out in examples showing how to customize included data, since I think that option will appeal to many users (me included :-)).
Thoughts?


ISPN-15348 Static analysis

https://issues.redhat.com/browse/ISPN-15348

[2023-11-29T16:02:42.299Z] 11:02:42.029 [INFO] -------------------------------------------------------------
[2023-11-29T16:02:42.299Z] 11:02:42.029 [INFO] -------------------------------------------------------------
[2023-11-29T16:02:42.299Z] 11:02:42.029 [ERROR] COMPILATION ERROR :
[2023-11-29T16:02:42.299Z] 11:02:42.030 [INFO] -------------------------------------------------------------
[2023-11-29T16:02:42.299Z] 11:02:42.030 [ERROR] /home/infinispan/workspace/Infinispan_PR-11532/jcache/embedded/src/test/java/org/infinispan/jcache/JCacheLoaderTest.java:[100,67] error: cannot find symbol
[2023-11-29T16:02:42.299Z]   symbol:   method intValue()
[2023-11-29T16:02:42.299Z]   location: class Object
[2023-11-29T16:02:42.299Z] 11:02:42.030 [ERROR] /home/infinispan/workspace/Infinispan_PR-11532/jcache/embedded/src/test/java/org/infinispan/jcache/JCacheLoaderTest.java:[110,71] error: cannot find symbol
[2023-11-29T16:02:42.299Z]   symbol:   method intValue()
[2023-11-29T16:02:42.299Z]   location: class Object

Yes, I'll fix it once https://github.com/infinispan/infinispan/pull/11522 is in

This is now ready

Can we separate https://github.com/infinispan/infinispan/pull/11532/commits/b2db606d54977227dbe8799edb6bfde437c040c4 regarding the command modules to a different PR? I would like to look at this separately and it isn't as trivial as many of the other changes.

I have removed the ModuleProperties changes, to be issued in a separate PR. I have also reintroduced the LinkedHashMap to AttributeSet while keeping the parsing/serialization order changes.


Unit testing Post requests to your Azure Function

I extended the CreateHttpRequest to allow testing post requests to the function that this documentation didn't cover
public static DefaultHttpRequest CreateHttpRequest(string queryStringKey = null, string queryStringValue = null, string body = null)
    {
        var request = new DefaultHttpRequest(new DefaultHttpContext())
        {
            Query = new QueryCollection(CreateDictionary(queryStringKey, queryStringValue))
        };
        if (!string.IsNullOrEmpty(queryStringKey) && !string.IsNullOrEmpty(queryStringValue))
            request.Query = new QueryCollection(CreateDictionary(queryStringKey, queryStringValue));

        if (body != null)
        {
           var stream = new MemoryStream();
            var writer = new StreamWriter(stream);
            writer.Write(body);
            writer.Flush();
            stream.Position = 0;
            request.Body = stream;
        }

        }

        return request;
    }

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

ID: a571957f-41ce-e7ea-5f0b-1aefb8b219fa
Version Independent ID: 6c676eb8-581c-c5c5-0047-c7a7531fa423
Content: Testing Azure Functions
Content Source: articles/azure-functions/functions-test-a-function.md
Service: azure-functions
GitHub Login: @craigshoemaker
Microsoft Alias: cshoe

@p0onage Thanks for contributing the PR and help us in improving the documentation. Should we proceed with closure of this and If there are further questions regarding this matter, please tag me in your reply.

@DixitArora-MSFT I have created a PR for you to look at https://github.com/p0onage/azure-functions-tests/tree/%2343553-TestingFactory-Improvements

@p0onage Content team will review the PR and suggest further action if required. So for now I will proceed with closure of this and If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Eddi "total kW" entity (grid + green)

Thank you for making this great integration I stumbled across it recently and it's really useful.
Is the avaliblecin the myenergi api an entity to give us the total current kW value (i.e a sum of the green kW + grid kW values) , I have a need to create an action to pause (stop) the eddi when the total kW value exceeds 10kW otherwise the PRV on my unvented water cylinder blows and dumps alot of the hot water from cylinder.
Im not a programmer so I can't understand how to take the two existing seperate kW entities that already exist in this integration and add them together and then make some code from that to trigger the action to stop the eddi.
Many thanks.

You should be able to combine the two with a Group Helper

Good suggestion @olej24 , does would best be done in Home Assistant. Did you get it working @BifferPAG ?


Restrict importing data constructors

Consider newtype Foo = Bar Int. I'd like to allow importing Foo, but not Foo (Bar) or Foo (..). I think currently the best I can do is
- functions:
  - {name: [Bar], within: [...]}

But this only generates a hint when using Bar, not when importing Bar. One can still import Foo (Bar) and use coerce to convert between Foo and Int.
And there's no way to disallow importing Foo (..) either.

I don't think you can do this at the moment. However, you can customise what imports are allowed through the modules section in the restrictions, see https://github.com/ndmitchell/hlint#restricting-items. It seems pretty reasonable to extend that with restrictions on what you are allowed to import from that module, which would solve the problem you are looking for, I think?

Yeah, agreed. We can also modify the behavior of functions, such that
- functions:
  - {name: [Bar], within: [...]}

generates a hint when importing Bar. But that doesn't cover the import Foo (..) case.

I would suggest that functions is about use, not importing. In general, if you import but don't use you get a warning, and it seems a bit odd that saying "you can't use Bar" is picked up by an import. I appreciate the newtype Coerce is a bit odd though


Support replaceState

When using Immutable data structures replaceState must be used instead of setState. See for reference this issue: https://github.com/facebook/immutable-js/issues/202

Any news on this? Will replacing state be supported?

Can't merge right now. :-S Also there is no change to README documenting this.
Not sure if it's a good idea to depend the update method if the key is given or not. It sort of goes against principle of least surprise. Or does it?

Closing in favor for #243 instead


Ajax search failing when using PostgreSQL

Our Laravel app is using PostgreSQL v9.6 for it's database because of our need of geo functions.
When trying to use the search feature with ajax tables we are getting the following error
Next Illuminate\Database\QueryException: SQLSTATE[42883]: Undefined function: 7 ERROR:  operator does not exist: integer ~~ unknown
LINE 1: ... or "email" like $3 or "password" like $4 or "id" like $5) a...
                                                             ^
HINT:  No operator matches the given name and argument type(s). You might need to add explicit type casts. (SQL: select count(*) as aggregate from "users" where ("first_name" like %marc% or "last_name" like %marc% or "email" like %marc% or "password" like %marc% or "id" like %marc%) and "users"."deleted_at" is null) in <SNIP>/vendor/laravel/framework/src/Illuminate/Database/Connection.php:770

In https://github.com/Laravel-Backpack/CRUD/blob/master/src/app/Http/Controllers/CrudFeatures/AjaxTable.php#L23 it is explicitly adding the primary key as a field to search, but in all of our tables the primary key is an integer which cannot have the LIKE operator used on it.
Is there a simple way of changing this to either not search the primary key or use = instead?

I'm having the same issue as you :(
This appears to be a workaround to get the ID in the response from EloquentDataTable.
I'd assume if there was a fix it would have to be there - here, more specifically - to check if the LIKE operator is available for the current column.

I found a really hacky workaround - to define the LIKE operator that is missing in PostgreSQL yourself.
CREATE OR REPLACE FUNCTION public.int_like(leftop integer, rightop text)
 RETURNS BOOLEAN
 LANGUAGE SQL
AS $function$
SELECT $1::text LIKE $2;
$function$;

CREATE OPERATOR PUBLIC.~~ (LEFTARG=integer, RIGHTARG=text, PROCEDURE=int_like);

(The LIKE operator is actually an "alias" to the ~~ operator in PostgreSQL, which is the one you want to define.)
Credits<EMAIL_ADDRESS>

Hmm... What if you define a search() method on the controller in question, with the exact same content as this method, but without the highlighted line? Does that fix it for you? Does the search still work?

@tabacitu Then the search works but the buttons don't, as the ID is needed for the edit/delete buttons

Anybody found a solution in the meantime?
Cheers!

@tabacitu I ended up going with the hacky solution. Here's the migration I created for it.
It will only run if the database driver is pgsql. Maybe that could be added to the CRUD installation?
<?php

use Illuminate\Database\Migrations\Migration;

class CreateIntegerLikeOperator extends Migration
{
    /**
     * Run the migrations.
     *
     * @return void
     */
    public function up()
    {
        if (DB::connection()->getDriverName() != 'pgsql') return;

        DB::connection()->getPdo()->exec('
            CREATE OR REPLACE FUNCTION public.int_like(leftop integer, rightop text)
             RETURNS BOOLEAN
             LANGUAGE SQL
            AS $function$
            SELECT $1::text LIKE $2;
            $function$;

            CREATE OPERATOR public.~~ (LEFTARG=integer, RIGHTARG=text, PROCEDURE=int_like);
        ');
    }

    /**
     * Reverse the migrations.
     *
     * @return void
     */
    public function down()
    {
        if (DB::connection()->getDriverName() != 'pgsql') return;

        DB::connection()->getPdo()->exec('
            DROP OPERATOR IF EXISTS public.~~ (integer, text);
            DROP FUNCTION IF EXISTS public.int_like(integer, text);
        ');
    }
}

Just as an idea... searching by primary key is useful, to find... erm... element with primary key = x !
I mean, most of the time if ypou want to search by primary key, you want one single result, the ID of the row, so maybe a compromise solution would be to modify the ->merge($this->crud->model->getKeyName()) search to generate a OR 'primary_key' = 'X' instead of OR 'primary_key' LIKE '%X%' just for this case, only the primary key.
That would:

Fix this problem
Ensure faster searchs by primary key

The only downside is it would not find "199" if you search primary key "19", but I really think that's a corner case

Hi guys,
I think I've recently solved the issue, but I need your help to confirm :-)
I've rewritten the entire search() method and the AjaxDataTables functionality on the upgrade branch, and I expect to launch everything as Backpack 3.3 by the end of the week.
It now no longer searches the primary key because, as @MarcosBL pointed out, it doesn't really make sense. It only searches the columns you define. Plus, when you define a column you now have the ability to define a custom search logic, as a closure, or tell CRUD to not make that column searchable at all (searchLogic = false):
        $this->crud->addColumn([
                                'name' => 'slug_or_title',
                                'label' => 'Title',
                                'searchLogic' => function ($query, $column, $searchTerm) {
                                    $query->orWhere('title', 'like', '%'.$searchTerm.'%');
                                }
                            ]);

I think this fixes this issue AND provides a way to customize, in case the issue arises with other columns (integer, etc).
What do you guys think?
To use the upgrade branch you can specify the version "dev-upgrade" for Backpack\CRUD in your composer.json file, then run composer update.

Hi @tabacitu!
Thanks for putting your effort into that. Your solution looks awesome! The way of providing a custom search implementation for each column is really useful.
I won't have the time to test the new branch so soon, but when I do it I'll reach out with feedback. :D

CREATE OR REPLACE FUNCTION public.int_like(leftop integer, rightop text)
RETURNS BOOLEAN
LANGUAGE SQL
AS $function$
SELECT $1::text LIKE $2;
$function$;
CREATE OPERATOR PUBLIC.~~ (LEFTARG=integer, RIGHTARG=text, PROCEDURE=int_like);
CREATE OR REPLACE FUNCTION public.numeric_like(leftop numeric, rightop text)
RETURNS BOOLEAN
LANGUAGE SQL
AS $function$
SELECT $1::text LIKE $2;
$function$;
CREATE OPERATOR public.~~ (LEFTARG=numeric, RIGHTARG=text, PROCEDURE=numeric_like);

@geekappsmx - the above code snippet is cool - but how does it relate to the current bug?

@geekappsmx - Oh that's a fix for it?

@macecchi @tabacitu  did you guys get a chance to check this? Is this still an issue?

This is still an issue but it's really not a Backpack related one - it's much more a Postgres type one.
Let's close for the moment.
@tabacitu Some of this would be useful to add to the docs.


Issue templates

https://docs.github.com/en/communities/using-templates-to-encourage-useful-issues-and-pull-requests/configuring-issue-templates-for-your-repository#creating-issue-forms

Looks like errors.  GitHub is saying "Body must have unique ids" when I click on your Bug Template.

Ha reproducer was there twice


Added Audio Input with multiple languages and Deadline with remainder

I have made the following updates:
Removed pycache Directories: Cleared all pycache files for a cleaner codebase.
Code Formatting: Used Black to format Python files according to PEP 8 standards.
Added Documentation: Enhanced documentation in HTML templates and relevant models/forms for better clarity.

@Prer74
migrations/pycache : still exists in directories please add it into git ignore to not track or upload to github

Can you please check it again its in feature branch.
Yes I used this sql/add.sql to delete all previous content in table, I forgot to remove!

Thank You!!


two bug

i have meet two bugs:

$.browser.msie is undefined.
this is because of jquery don't support msie from 1.9
and i fixed.

Uncaught Error: Syntax error, unrecognized expression: td[date=19 Mar 2014]

have no idea..

Regarding 2 mistake:
you need to change next lines:
line 207:
if (matches = string.match(/^(\d{1,2}) ([^\s]+) (\d{4,4})$/)) {
replace it with:
if (matches = string.match(/^(\d{1,2}) (\d{1,2}) (\d{4,4})$/)) {
and line 215:
return date.getDate() + " " + this.short_month_names[date.getMonth()] + " " + date.getFullYear();
replace it with:
return date.getDate() + "-" + date.getMonth() + "-" + date.getFullYear();


Links shouldn't be underlined unless hovered/focussed.

TP: https://rockabox.tpondemand.com/entity/11568

Good to Merge @rockabox/qa over to you :+1:

Can be merged for QA


Google Drive doesn't show up under Cloud Drives

Description
I have Google Drive for Desktop installed, which makes Google Drive available as logical drive G:.  I also have OneDrive installed. Only OneDrive shows under Cloud Drives - I have to navigate to G: to locate Google Drive.
Steps To Reproduce

Open Files App with both Google Drive for Desktop and OneDrive installed.
Only OneDrive appears under Cloud Drives.

Requirements
Google Drive should show up under Cloud Drives
Files Version
<IP_ADDRESS>
Windows Version
10.0.22631.2715
Log File
debug.log

Thanks for the report, this has been fixed in the preview builds. https://files.community/download

@Josh65-2201 the only change in preview was to hide it from the regular drives section, this issue sounds like it's not showing up at all.

Sorry, misread it

@swoodvitsoe is Google Drive open on your device?

If by open, you mean was I browsing the files - no. It is available via logical drive G but doesn't appear under Cloud Drives at all.

@swoodvitsoe I meant is the Google Drive client opened in the background?

@swoodvitsoe


[SPARK-9903] [MLLIB] skip local processing in PrefixSpan if there are no small prefixes

There exists a chance that the prefixes keep growing to the maximum pattern length. Then the final local processing step becomes unnecessary. @feynmanliang

Merged build triggered.

Merged build started.

Test build #40665 has started for   PR 8136 at commit e130245.

Test build #40665 has finished for   PR 8136 at commit e130245.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

The change LGTM, though it got me thinking about some other parts of the code unrelated to this PR

Merged build triggered.

Merged build started.

Merged build finished. Test FAILed.

test this please

Merged build triggered.

Merged build started.

Test build #40700 has started for   PR 8136 at commit 80383bc.

Test build #40700 has finished for   PR 8136 at commit 80383bc.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Merged into master and branch-1.5.


decimal conversion of binary to python object causes memory leak

In the python iterator InformixCdc_iternext we call ifx_lo_read from the Informix CSDK to read N bytes from the internal SBLOB that the engine has setup for us to send CDC records.
I'm pretty confident there's a memory leak (or unrestrained memory growth) related to this ifx_lo_read call.
If I force the iterator to call ifx_lo_read more often by limiting the number of CDC records returned to 1 (forcing more calls to ifx_lo_read), the memory grows faster than when max records is set to 250.
I build a debugging version of python that can be used with Valgrind according to these steps:
http://pythonsweetness.tumblr.com/post/151938194807/debugging-c-extension-memory-safety-with-valgrind
some output from ~/src/cpython/python setup.py build_ext && valgrind --leak-check=full --show-leak-kinds=all -v --tool=memcheck ~/src/cpython/python informixcdc.py
looks liks this
==10700== 197,632 bytes in 772 blocks are still reachable in loss record 1,655 of 1,656
==10700==    at 0x4C2DB8F: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==10700==    by 0x6EFE438: cl_ext_malloc (in /opt/informix-csdk-3.70.FC5/lib/esql/libifgen.so)
==10700==    by 0x7375E01: _gl_ext_malloc (in /opt/informix-csdk-3.70.FC5/lib/esql/libifgls.so)
==10700==    by 0x7342567: _gl_alloc_ptr (in /opt/informix-csdk-3.70.FC5/lib/esql/libifgls.so)
==10700==    by 0x733FCC4: alloc_flag_arr (in /opt/informix-csdk-3.70.FC5/lib/esql/libifgls.so)
==10700==    by 0x733FCFD: create_flag_arr (in /opt/informix-csdk-3.70.FC5/lib/esql/libifgls.so)
==10700==    by 0x73407BC: fscanf_unc_multi_arr_t (in /opt/informix-csdk-3.70.FC5/lib/esql/libifgls.so)
==10700==    by 0x73408FB: fscanf_unc_multi_arr_t (in /opt/informix-csdk-3.70.FC5/lib/esql/libifgls.so)
==10700==    by 0x73408FB: fscanf_unc_multi_arr_t (in /opt/informix-csdk-3.70.FC5/lib/esql/libifgls.so)
==10700==    by 0x7340C25: _gl_fscanf_multi (in /opt/informix-csdk-3.70.FC5/lib/esql/libifgls.so)
==10700==    by 0x7373CAC: read_cv_file (in /opt/informix-csdk-3.70.FC5/lib/esql/libifgls.so)
==10700==    by 0x73741A7: gl_cv_load (in /opt/informix-csdk-3.70.FC5/lib/esql/libifgls.so)
==10700==
==10700== 395,264 bytes in 772 blocks are still reachable in loss record 1,656 of 1,656
==10700==    at 0x4C2DB8F: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==10700==    by 0x6EFE438: cl_ext_malloc (in /opt/informix-csdk-3.70.FC5/lib/esql/libifgen.so)
==10700==    by 0x7375E01: _gl_ext_malloc (in /opt/informix-csdk-3.70.FC5/lib/esql/libifgls.so)
==10700==    by 0x7342567: _gl_alloc_ptr (in /opt/informix-csdk-3.70.FC5/lib/esql/libifgls.so)
==10700==    by 0x73406D5: alloc_data_arr (in /opt/informix-csdk-3.70.FC5/lib/esql/libifgls.so)
==10700==    by 0x734072C: create_data_arr (in /opt/informix-csdk-3.70.FC5/lib/esql/libifgls.so)
==10700==    by 0x73407ED: fscanf_unc_multi_arr_t (in /opt/informix-csdk-3.70.FC5/lib/esql/libifgls.so)
==10700==    by 0x73408FB: fscanf_unc_multi_arr_t (in /opt/informix-csdk-3.70.FC5/lib/esql/libifgls.so)
==10700==    by 0x73408FB: fscanf_unc_multi_arr_t (in /opt/informix-csdk-3.70.FC5/lib/esql/libifgls.so)
==10700==    by 0x7340C25: _gl_fscanf_multi (in /opt/informix-csdk-3.70.FC5/lib/esql/libifgls.so)
==10700==    by 0x7373CAC: read_cv_file (in /opt/informix-csdk-3.70.FC5/lib/esql/libifgls.so)
==10700==    by 0x73741A7: gl_cv_load (in /opt/informix-csdk-3.70.FC5/lib/esql/libifgls.so)

No longer think this is a "memory leak".
When I run CDC over a slow network the memory grows, but when I run it locally it doesn't. So I think this may be some kind of network buffering in ifx_lo_read or something going crazy.
Buuuut, it could be some weird Ubuntu (laptop, slow network) vs Centos (local connection) issue.
I'm not going to worry about it too much.

Further testing shows I'm still seeing the memory growth even when connecting locally.
HCL thinks it might be related to GLS and maybe a big on their end there. They're looking into it.
That might explain why I'm seeing the memory growth when testing against HCL on AWS which is a UTF-8 locale and not seeing it on a en_us locale.

Locale used doesn't seem to make a difference after testing against two different locales.

Oh. My. Gosh.
I'm not Py_DECREF'ing all kinds of PyObjects in the loop where I build the PyList of column PyDicts. I need to rethink/rework this whole section (and some others, too).
 There may still be a little bit of leaking in the Informix ESQL/C libraries, they're looking into it, but it isn't as bad as what I'm doing.
I created a fake_ifx_lo_read() to read from a file of binary data to simulate ifx_lo_read(), but it was at 4 AM and I can't remember if the results were good or bad. I think it was good as in I determined that ifx_lo_read was not leaking.
All of my work is in the parse-boring-events branch, which started out pretty boring then morphed into interesting stuff when I added transactional consistency on a restart, this memory leak cleanup, and the start for threading support. I should probably focus on one thing per branch soon.

I'm sure the leak is due a symbol conflict ("initdatetime") between Python and one of the Informix ESQL/C libraries ("libifgen.so"). It's not just the ifx_lo_read() ESQL/C call. Any call which needs to involve GLS will have the same effect.

Yep, that is what I'm being told.
I've started port all of my ESQL/C calls to ODBC, which is not very fun IMO.

Umm, it's that's due the leak issue? hope nop. The InformixOpen Python driver (IfxPy) uses ODBC, it's not perfect, but can give you some ideas.

Do you know where I can find example code that will help me understand how to call the following in ODBC?
execute function informix.cdc_opensess(?, 0, ?, ?, 1, 1)
and how to get the return value from the function call?

Attached is a complete C sample which gets a CDC session opened. It doesn't go as far as decoding the CDC packages, but it does all the SQL calls needed (cdc_opensess, cdc_startcapture, etc)
cdc_c.txt

Awesome! Thank you very much, that code cleared up A LOT of confusion I was having with the ODBC API!

;) glad it helps.
It may be old, but the Redbook below is still a "best seller" ;). Full of quick and simple Informix examples:
https://www.redbooks.ibm.com/redbooks/pdfs/sg247884.pdf

using ODBC and connecting, enabling, activating and reading from the sblob with no memory increase.
thanks again for the help!


Feature request:  Support for saved room groups in Sonos S2

This is a feature request to support saved room groups as introduced with Sonos S2.
Saved room groups are essentially shortcuts for selecting a set of rooms which are frequently played together. Groups saved in thw Sonos first party S2 app on one device are visible on other devices connected to the system. Only the OS and Android apps support saved groups. The desktop apps do not.
Requested functionality:

Room groups created with the Sonos first party apps are exposed via SoCo and can be the target for play/pause/etc. actions.
Room groups may be created via SoCo which would then be visible and usable in the Sonos first party apps.

(from https://support.sonos.com/s/article/3391?language=en_US)

This functionality is not exposed in the S2 desktop app, and I don't have a way of WireShark tracing the interactions between mobile controllers and the Sonos players.
Any suggestions for how to do this ... or better still , does anyone have the traces available?


How exactly does a widget use ui_base?

I read the dashboard 2 docs and understood that widgets are node-red nodes, and that they "extend" the ui_base config node, but I could not understand how exactly this happens. Somehow the widget's node has some additional methods called register/deregister. Probably has other, but I did not read everything yet. So, when/how exactly does a widget receive those 2 methods?
I'm planning to recreate it as a base class of Node class called, in the nrg project.
export default class WidgetNode extends Node {

    // specific WidgetNode properties

    constructor(config){
        super(config);
        //initialize specific WidgetNode properties, such as state store
    }

    // specific WidgetNode methods, like register/ deregister
}

export default class MyWidget extends WidgetNode {

    constructor(config){
        super(config);
    }
}

The UI Base is the single parent object for a Dashboard.
In the constructor for each Widget, they get reference to the UI Base (normally via their assigned group/page). They call base.register, and pass themselves into thag function call

@joepavitt what do you think of my idea of making ui_base into a parent class called WidgetNode, and then widgets would extends this class?  I think it is possible. Could you help me to see problems I would have trying to do it?

The UI Base has settings and configuration of its own. The widgets aren't necessarily extensions in a Class sense. They are different entities. There is only ever one Base. Having a Node an extension of Base would mean you suddenly have 100's of Dashboards/Based
If we consider Class hierarchy, there could be benefit in having a common WidgetNode class that the nodes, in order to share feature sets, etc.
The base.register function is the Widget's way of communicating to the Dashboard to say "Hey, I exist"

My use of the word "extends" here is likely misleading.
From a Dashboard Hierarchy we have:

Base

Theme
Page

Group

Widget

That doesn't mean that there is Class Inheritance here though. Fundamentally, these are the Classes, then each node in a flow (or config node) in an instance of the respective class

Also worth noting, and I may be remembering this wrong, the nodes themselves dont need a register/deregister function.
The base.register() is just called as part of the Node's constructor
Maybe they do need a deregister function?

@joepavitt

Maybe they do need a deregister function? I can't quite remember how I handle that, as I'm not at a keyboard

I believe that if a node can call this.deregister(), maybe there are situations where the node would also call this.register() afterward. Maybe to do a "hard reset" somewhere? Do you know if there is any Widget that does it?

The base.register() is just called as part of the Node's constructor

I can ensure this is called inside the parent's constructor, or in the mixin. This way devs won't need to manually write it. The same way I did for RED.nodes.registerType or RED.nodes.createNode
https://github.com/AllanOricil/nrg/blob/main/templates/server/entrypoint.handlebars
https://github.com/AllanOricil/node-red-node

@joepavitt
What do you think of this model?
export default class WidgetNode extends Node {

    #scope: Base | Theme | Page | Group;  // each instance of a widget can have its own location

    constructor(config){
        super(config);
        //initialize specific WidgetNode properties, such as state store
    }

    // specific WidgetNode methods, like register/ deregister

    register(){
         this.#scope.register(this);
    }

    deregister(){
         this.#scope.deregister(this);
    }
}

That could work, bare in mind though some widgets are permitted to be multiple, e.g. ui-template can be switched between group, page and UI

That could work, bare in mind though some widgets are permitted to be multiple, e.g. ui-template can be switched between group, page and UI

How does a Widget switch between "ui scopes" (have to find a better name)?

It is defined in template node's configuration

It is defined in template node's configuration

@colinl every widget has those fields shown in that image? How can I get to that configuration form?

I started to have a better understanding after reading the config nodes

I think I can start modeling it into classes. I will try to do the base ones, and a button.

One last question.
@colinl @joepavitt
What are all possible event names people can use?
const evts = {
            beforeSend: async function (msg)
}

group.register(node, config, evts)

export default Widget exnteds UIEntity {

     /*
     * It can be implemented by a child class
     * @abstract
     */
     static onBeforeSend(msg){}
}

export default MyWidget extends Widget {

      constructor(config){
      }

      static onBeforeSend(msg) {
            ....
      }

      // all other event handlers
}

The inside the Mixin, I will do some sorcery js to make sure these "on" events are registered.

@joepavitt @colinl could you help with the last comment?

Not me, I don't know the answer.

@AllanOricil https://dashboard.flowfuse.com/contributing/guides/registration.html#evts

every widget has those fields shown in that image? How can I get to that configuration form?

Drop a Dashboard "template" node into the Editor, then double click it to configure it. Only the "Template" node as the option to switch between "parent scope" like this.

@AllanOricil https://dashboard.flowfuse.com/contributing/guides/registration.html#evts

Is the onInput from evts the same as the node's on input this.on("input", onInputHandler())?

Not quite, the one you've quoted there is the client-side handler, the onInput I linked to is the server-side handler

Not quite, the one you've quoted there is the client-side handler (Vue Widget), the onInput I linked to is the server-side handler (the Node-RED node)

So all evts handlers are sent to the client? I thought they executed on the server, because they are declared in the server js of a node.

No, they're separate. What a node does on input server-side (defined in the .jsfile) will differ the the client-side behaviour of a widget when receiving a message (defined in thevue` file)

Sorry, I thought this.on("input", onInputHandler()) had been taken from a vue file as it's very similar to our code there too.
Yes, if you checkout the node.register function in ui-base.js, you'll see that the on('input') handler is setup for the relevant node and the evts.onInput is passed there (if provided)

Sorry, I thought this.on("input", onInputHandler()) had been taken from a vue file as it's very similar to our code there too.
Yes, if you checkout the node.register function in ui-base.js, you'll see that the on('input') handler is setup for the relevant node and the evts.onInput is passed there (if provided)

I would need a new name for that handler to put it on the class alongside the node's this.on("input")
I can't have 2 onInput methods in the class.  What about onData for the evts.onInput?

not sure why the class needs that definition though? It needs evts, which is then an object mapped to x functions/objets/booleans

not sure why the class needs that definition though? It needs evts, which is then an object mapped to x functions/objets/booleans

When one of those handlers is triggered, can't they access "instance" data? I can have multiple widget nodes, all with the same handler, but each with its own instance scope. Those events are part of the Widget, so I'm trying to encapsulate them normalized as single functions, instead of using an evts attribute. It looks cleaner in my opinion.


Maybe figure out how to make the video poster images responsive, or just make them smaller

Maybe just use jpg: https://github.com/derekenos/derekenos.com-generator/issues/21


fixconfix bug of converting outer join probe block to nullable

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Implement economizer for HVAC cooled buildings

Free mechanical cooling at night at certain indoor and outdoor temperature conditions.

possible duplicate of #256 (Free cooling)

@gabriel-happle if it is a duplicate, can this be closed?

Actually Economizer is totally different from free cooling. I didn't know at the time. I learned that in Singapore context the conditions are never favorable to use an Economizer. In Switzerland however, this is different. Maybe @martin-mosteiro has to decide?

Are we talking about night time flushing to cool the building? Probably good to have in Switzerland, no?


Add additional customer fields to MTO

Description
Add additional fields to customer section on draft MTO page:

Email
Phone
Current Address [street 1, street 2, city, state, zip]
Destination Address [street 1, street 2, city, state, zip]
Branch of Service

Setup
make db_dev_e2e_populate
make server_run
make office_client_run

Visit http://officelocal:3000/too/customer-moves/ and the additional fields should be present in the Customer section.
Code Review Verification Steps

[ ] Have the Jira acceptance criteria been met for this change?

References

Jira story for this change

@chrisgilmerproj @garrettqmartin8 @gidjin @lynzt @rdhariwal @Ryan-Koch @tinyels seems like this can't be merged until one of you approve. The migration adding the columns looks like it makes this fall under new "code owner review required" check.

I'm not seeing branch of service in the customer info.

I'm not seeing branch of service in the customer info. I think you missed that column both in the front and back-end.

It was already there is called agency. Missed it on the frontend though. Updated in b92767895.


Adding classname prop for a blank slate svg

💬 Description
I would like to add some custom classes to the SVG rendered by BlankSlate. This PR adds that exact prop in!
✅ Checklist

[ ] Tests written
[ ] Browser tested
[ ] Added to documentation
[ ] Added to storybook

 Make blank slate smaller in general comment sidebar


Proper way of uploading the contract to the testnet

Hi everyone,
Currently I'm in the works of running this contract and am having an issue instantiating the contract.
I ran the following commands:
make
and next optimize the contract using:
docker run --rm -v "$(pwd)":/contract \   --mount type=volume,source="$(basename "$(pwd)")_cache",target=/code/target \   --mount type=volume,source=registry_cache,target=/usr/local/cargo/registry \   enigmampc/secret-contract-optimizer
Having done this, I start the local testnet:
docker run -it --rm \  -p 26657:26657 -p 26656:26656 -p 1337:1337 \  -v $(pwd):/root/code \  --name localsecret ghcr.io/scrtlabs/localsecret
Then, inside the docker (docker exec -it localsecret /bin/bash and run cd code after), I store the contract:
secretd tx compute store contract.wasm.gz --from a --gas 1000000 -y --keyring-backend test
Running secretd query compute list-code, no output is obtained. So maybe something goes wrong here.
Nonetheless, trying to instantiate the contract, I run the following command:
secretd tx compute instantiate 1 '{"name": "test", "symbol":"TTT", "decimals":10, "prng_seed":"VGVzdA=="}' --label test --from a
This returns the following error:
Error: unexpected end of JSON input
Do you know how to resolve this error?

Are you sure that the store tx succeeded? Maybe you need to increase the gas limit

Thank you for your fast reply. Indeed, increasing the gas limit resolved this problem.


add set function

Change-Id: Ib15c9585f0083805a06fb709f12d65620618cf51

Thanks for the PR but I can't merge it because I don't have plans on adding "set" like functions to GJSON at this time. There's the SJSON that provides this functionality.


Update to toml 0.8 and itertools 0.11

@gdesmott Can we get a new release with this? :)

Sure, I just pushed  6.1.2


add doc about modality and library tags

related to:
https://github.com/huggingface-internal/moon-landing/pull/10595
https://github.com/huggingface-internal/moon-landing/pull/10641

It looks like you've updated documentation related to model or dataset cards in this PR.
Some content is duplicated among the following files. Please make sure that everything stays consistent.

modelcard.md
docs/hub/model-cards.md
docs/hub/model-card-annotated.md
src/.../modelcard_template.md (huggingface_hub repo)
src/.../datasetcard_template.md (huggingface_hub repo)
src/.../repocard.py (huggingface_hub repo)
datasetcard.md
docs/hub/datasets-cards.md

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


[11.x] Adds once memoization function

Inspired by @taylorotwell's tweet and Spatie's once package, this pull request proposes the addition of the once memoization function in Laravel 11.
This function is extremely useful for caching a costly computation without the need to create a temporary like a nullable property on the class itself. Here is a practical example:
class User extends Model
{
    public function stats(): array
    {
        return once(function () {
            // Expensive operations to generate user stats, multiple db queries, etc...

            return $stats;
        });
    }
}

Behind the scenes, this function is incredibly smart. It ensures that a callable is called only once, returning the same result on subsequent calls. If the once function is called within an object instance, the memoized result will be associated only with that instance.
// Assuming you have two User instances: $userA and $userB

$userA->stats();
$userA->stats(); // cached result from previous line...

$userB->stats(); // Not using $userA cached results, because $userB !== $userA
$userB->stats(); // cached result from previous line...

Things to Be Decided:

[x] Should we flush the cache of once at the framework level at any lifecycle stage, such as RequestHandled or JobProcessed? We decided to flush Once after each job, after each test case, and after each request (at Octane listener level)

Limitation: This function currently has a limitation where once calls within the same line are considered the same call. For example: $a = once(fn () => 1); $b = once(fn () => 2); echo $b; // outputs 1.

Could the same functionnality be done WITHOUT return a once() call from a Model (or any class) method BUT by setting a custom PHP Attribute on the method ?
Something like:
#[Once]
public function stats()
{
return ['test']; // or int,float, etc...
}

I personally am not a fan of Attributes, but specifically in this case I don't think that's a good route, as it locks users in to memoization happening at the function/method level.  By having a once() function, the user is free to use it however, wherever they like.  this means part of the body of their function could be memoized, while another part might not be.

What about allowing to pass an optional key to remember the value based on this key?
Example:
public function issueToken(string $basicAuthorization) {
    return once(
        fn() => Http::post(...)->json('access_token'),
        'basic-auth:' . md5sum($basicAuthorization)
    );
}

Curious why not use Spatie's package directly?

@khepin we didn't really want to require a package for 2 files, and once as a concept was originally written by me.

@nunomaduro what about the adding documentation for this?

This functionality is missed.
https://github.com/spatie/once#usage


[We See a Way] - Home Page Design - mini teaser

mini teaser with vertical and horizontal line configurable

Figma design link :

this implementation has been done on home page.
https://main--danaher-ls-aem--hlxsites.hlx.page/us/en/we-see-a-way.


[SC-273] Tests for transfers

We need to check farmed amounts consistency after all the transfers.


chore: database schema and typeorm setup

How to test this PR
Cloning the repository
git clone https://github.com/hngprojects/hng_boilerplate_nestjs.git
cd hng_boilerplate_nestjs
git checkout chore/database-setup

Setting Up the Environment
Prerequisites:

Ensure you have NodeJs and Npm installed.
You can download and install Node.js from nodejs.org.
Check your installation here:

node -v
npm -v

Ensure you have NestJs installed
Install the NestJS CLI globally using npm:

npm install -g @nestjs/cli

Check your installation

nest -v

Running The Application Locally
npm install

npm run start:dev

Database Setup

Ensure PostgreSQL is installed and running.

Update the .env.development.local file with your database credentials
PROFILE=local
NODE_ENV=development
PORT=3008
DB_TYPE=dt_type
DB_USERNAME=your_username
DB_PASSWORD=your_password
DB_HOST=localhost
DB_DATABASE=db_name
DB_ENTITIES=dist/**/*.entity{.ts,.js}
DB_MIGRATIONS=dist/db/migrations/*{.ts,.js}

Create And Apply Migration Files

Migration files should be placed in the db/migration directory.
Generate migration files automatically with typeorm

npm run migration:generate

To manually create migration file

npm run migration:create

Run migration

npm run migration:run

Create Sample Data

The seed data is run once when app starts

Testing the Endpoint
You can test the endpoint using curl or Postman

Using Curl

curl -X GET http://localhost:3008/api/v1/users/

Using Postman

Open Postman.
Create a new GET request to http://localhost:3008/api/v1/users.
Send the request and verify the response.

Expected Response

 {
    "id": "d6aa5dc9-a1c3-4516-ad4e-31d169893510",
    "first_name": "John",
    "last_name": "Smith",
    "email"<EMAIL_ADDRESS>    "password": "$2b$10$lOsaGJVjYxxZsVQ2WNsiwe./MEu.MEp2QiXKAS1FwP3gQtctOM2tG",
    "is_active": null,
    "attempts_left": null,
    "time_left": null,
    "created_at": "2024-07-20T13:08:28.273Z",
    "updated_at": "2024-07-20T13:08:28.523Z",
    "profile": {
      "id": "6f8984f8-682a-4bb9-a618-6b14ea109bd3",
      "username": "johnsmith",
      "bio": "bio data",
      "phone": "1234567890",
      "avatar_image": "image.png"
    },
    "products": [
      {
        "id": "61c30739-1b7e-447a-b58e-3b05e0e7c0d3",
        "product_name": "Product 1",
        "product_price": 100,
        "description": "Description 1"
      }
    ],
    "organisations": [
      {
        "org_id": "c52d7c25-8632-4b08-91d1-1f9a837f1861",
        "org_name": "Org 1",
        "description": "Description 1"
      },
      {
        "org_id": "4b42ea8e-e1e2-407e-88cb-9893c05a4167",
        "org_name": "Org 2",
        "description": "Description 2"
      }
    ]
  }

There are 2 default users created when you start the application.
To get the sample data, use this route after starting the application:

http://localhost:3008/api/v1/seed/users

Also, you can get the users by signing in to your database and query the users table using:
SELECT * FROM users;

Could you link the issue to this PR by commenting it please?

https://github.com/hngprojects/hng_boilerplate_nestjs/issues/268


Reconsider how the editor loads data into Trellis

While working on https://github.com/LD4P/sinopia_exporter/issues/53—a ticket to add subject URIs to JSON-LD representations of resources exported from Trellis—I learned that these subject URIs are blank in Trellis because of how the editor is loading data. Namely, the N-Triples specification does not allow for relative (or empty) subject URIs and that is what we currently send.
It is also true that when we load new data from the editor, we do not yet have a subject URI. Thus, to resolve this, we should consider sending RDF data from the editor to Trellis using a serialization of something other than N-Triples.
This also means all of the data we've persisted thus far in every environment is non-conformant and may ultimately need to be migrated.

We currently pin the n3 module that we use for RDF parsing and serializing to version  1.1.1, the current version 1.3.0. In the past the editor has failed when we updated to the latest n3 version likely due to ntriples serialization format not supporting relative URIs.

Let's do an investigation (<= 1 day) to try loading Trellis with a different RDF format.

@mjgiarlo said:

Namely, the N-Triples specification does not allow for relative (or empty) subject URIs and that is what we currently send.

I don't believe that is true as the grammar rule specifically allows: <>

IRIREF | ::= | '<' ([^#x00-#x20<>"{}\|^] |  UCHAR)* '>'`

@jcoyne I was relaying @acoburn's explanation, which I may have mangled in transit. Paging @acoburn!

@jcoyne This is how the thread resolved the last time it came up, AFAICT: https://github.com/trellis-ldp/trellis/issues/419#issuecomment-504766300

@jcoyne :speech_balloon:

I don't believe that is true as the grammar rule specifically allows: <>

IRIREF  ::=  '<' ([^#x00-#x20<>"{}\|^] |  UCHAR)* '>'`

As @acoburn mentioned in the issue discussion above, this is at odds with section 2.2 of the N-Triples spec:

IRIs may be written only as absolute IRIs.

Okay, that looks like it agrees with what Greg Kellogg thinks too:
graph = RDF::Graph.new << [RDF::URI.new, RDF::Vocab::DC.title, "Hello, world!"]
graph.dump(:ntriples)
ERROR Statement #<RDF::Statement:0x3fc7cb070198(<> <http://purl.org/dc/terms/title> "Hello, world!" .)> is invalid

graph.dump(:ttl)
=> "\n<> <http://purl.org/dc/terms/title> \"Hello, world!\" .\n"


backport: python27Packages.soco: 0.18.1 -> 0.19, remove broken

Motivation for this change
Fixes the default build of beets in NixOS 20.03
Things done

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

[x] NixOS
[ ] macOS
[ ] 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)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Also quickly tested beets with my local library, and the compilation of beets also contained a quite significant amount of tests ^^

@GrahamcOfBorg  python38Packages.soco  python37Packages.soco

@minijackson You need to use -x in the cherry-pick, per the backport guide.

I wanted to cherry-pick the automated PR, but it didn't remove the broken status. Should I split it into 2 separate commits ?

@minijackson you can use git commit --amend to also remove the unbroken flag.

Done !


Assign a different color to each mini-equation

closes staltz/matrixmultiplication.xyz#4
This assigns a different color to each mini-equation, which makes it easier to read.
This PR is based on the example code provided by @JanCVanB in the github issue, but generalized to support more colors.
Uses a list of (10) colors defined in the styles.ts file.
If there are more columns in matrix B than colors defined, it will begin repeating colors.
Screenshot below:

PS: Awesome project!!

@dvincent1337 Thank you for turning my issue into a PR! I haven't thought about this visualization in 5 years, but it's just as satisfying today as it was when I first saw it 🤩

My pleasure! I saw this project first time last night, and couldn’t agree
more!
On Fri, Jan 21, 2022 at 1:42 PM Jan Van Bruggen @.***>
wrote:

@dvincent1337 https://github.com/dvincent1337 Thank you for turning my
issue into a PR! I haven't thought about this visualization in 5 years, but
it's just as satisfying today as it was when I first saw it 🤩
—
Reply to this email directly, view it on GitHub
https://github.com/staltz/matrixmultiplication.xyz/pull/12#issuecomment-1018806682,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AASV4QI7W7SMSJIX3ZQH52DUXGZKXANCNFSM5MP32Q4Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>


[ISSUE-4674][feat] update jdbc and trino ec properties

What is the purpose of the change
Add jdbc and trino engine does not exit configuration parameter by default.
Related issues/PRs
Related issues: #4674
Brief change log

add wds.linkis.engineconn.max.free.time=0 to linkis-engineconn.properties.

Checklist

[x] I have read the Contributing Guidelines on pull requests.
[ ] I have explained the need for this PR and the problem it solves
[x] I have explained the changes or the new features added to this PR
[ ] I have added tests corresponding to this change
[ ] I have updated the documentation to reflect this change
[ ] I have verified that this change is backward compatible (If not, please discuss on the Linkis mailing list first)
[ ] If this is a code change: I have written unit tests to fully verify the new behavior.

LGTM


android focusble doesn't work

This widget can't be ficusable, o doesn't work

I'll will look into that, thanks

Closing for inactivity. Feel free to reopen if necessary.


restart in case of preserve on reload

If preserve on reload is enabled, every browser reload re-uses the same application.
Sometimes it would be nice to do a restart....
We don't get any reload info from the browser reload, so we need a solution in our implemtation... Maybe an overlay if we press a special key?

If preserve on reload is enabled, the user-settings will display an option to restart the application, we need a visible field for this user-setting in the application.xml to potentially disable the item

I implemented it the way that I expect the property to be called "restart" in the application-settings-response

userRestart

default: true - if not available as property

The property is now available in mobile.xml


Move all basic macros to top level

Move all basic macros to top level

Not going to do this now since not needed and will make it a whole lot harder to undo later


Send notifications when project species are added or updated

Send notifications when project species are updated
Send out app notification and email when species is added or edited

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#1951  👈
#1946
#1945
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @nickgraz and the rest of your teammates on  Graphite


GuildMessageUpdateEvent - how do i get the message before it is updated

General Troubleshooting

[x] I have checked for similar issues.
[x] I have updated to the latest JDA version
[x] I have checked the wiki and especially the FAQ section for similar questions.

Question

Example Code

You need to cache it yourself

To quote the Troubleshooting Guide:

When discord emits a message_delete or message_update they only provide the new content of the message. Since JDA does not keep a cache of messages it is unable to provide the previous content. Instead you will have to track content of messages yourself.


{AKS} Fix test cases related premium sku

This checklist is used to make sure that common guidelines for a pull request are followed.
Fix the following 2 cases

test_aks_create_with_premium_sku
test_aks_update_with_premium_sku

Related command

General Guidelines

[ ] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
[ ] Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)
[ ] My extension version conforms to the Extension version schema

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

AKS

Queued live test to validate the change. Test passed!


Webpack breaks with ES6

Hi @ericelliott,
Thanks for autodux, it's really nice and useful! :)
I know this is still in "developer preview mode", so what folows is more of a suggestion than a bug report.
Please consider releasing as ES5 too.
In a new project with @angular/cli: 1.2.6 the build breaks with

ERROR in vendor.42b3b802a29f068797b9.bundle.js from UglifyJs
Invalid assignment [/home/nuba/IdeaProjects/understudio-angular-quickstart/~/autodux/source/index.js:3,0][vendor.42b3b802a29f068797b9.bundle.js:20545,47]

And with @angular/cli we're quite limited when it comes to tweaking webpack's behaviour to add a preprocessor or something.
I worked around it by embedding autodux (as it's really small) in my ts code (which then had ES5 as target in tsconfig.json). But just calling tsc --allowJs -t ES5 index.js --outFile index.ES5.js did the trick ;)
Thanks, keep rocking!
Nuba

I think an ES5 bundle would be a fine addition, but it's not on my priority list, at the moment. I'm open to a pull request.

Released as 1.0.5


Get a weekend

Hello. I need to get a weekend from the current date to the month. It is to get the dates of the weekend in the array. This can be done?

We don't really support the concept of 'weekend' as it varies between cultures. Beyond that, I'm not really sure what you are asking. You are wondering how to get all of the saturdays and sundays in a month?

Yes, I need to receive the dates of all Saturdays and Sundays within a month from the start date.

@maggiepint Yes, I need to receive the dates of all Saturdays and Sundays within a month from the start date.

@LusenkoSasha yes, you just need to do a little programming. You can start with a moment, set it to the beginning of the month, find that saturday, then in a loop add a week until you're not in the month anymore, adding to the array as you go.


LibWeb: Don't match text across block elements for find in page queries

Find in page will no longer match text that spans across block elements. Previously, given the markup WH
F
, the query WHF would find a match. We would now match WH and F separately, but not WHF.
This PR also fixes another Find In Page issue where the Find Next button needed to be pressed twice before the match index would be updated.

I've added a commit to this PR to ensure we don't include non-visible text (either display: none or visibility: hidden in find in page results.
An example of this can be seen on Wikipedia:
https://github.com/LadybirdBrowser/ladybird/assets/2817754/439a397e-6a84-4ad2-b4f6-555de4b03f75

That video makes me think that perhaps we could "refresh" any active find-in-page state after performing a layout.
That way you wouldn't have to search again :)
We don't have to do that in this PR ofc!


Add the dexterity as strength and fitness

On death of character, the dexterity does not work. Must check to either change the prof giver to OnCreatePlayer or make another one for that event.

Fixed in 0.1.2


Update interested-technical-writers.md

This is my proposed change with my profile in it. please review it and make the merge.

@googlebot I signed it already.

Hi! Would you please make a new pull request adding your information in alphabetical order by surname/family name? If you only use one name, alphabetize under that name. If you have questions about where your name would go in order, please reply to this comment. (Please alphabetize by more than the first letter: for example, Cat would go after Call but before Cave.)
The names are LISTED firstname, surname, but they are alphabetized by surname. So if you have:
Anwar Able
Bill Lion
Caesar Sun
Someone named Amy Pleasant would go between Bill Lion and Caesar Sun.


Incorrect add_node types for Callable

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangGraph/LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangGraph/LangChain rather than my code.
[X] I am sure this is better as an issue rather than a GitHub discussion, since this is a LangGraph bug and not a design question.

Example Code
from langgraph.graph import StateGraph
from langchain_core.messages import HumanMessage
from typing import TypedDict
from langchain_core.runnables.config import RunnableConfig

class MyState(TypedDict, total=False):
    input: HumanMessage

def some_node(state: MyState, config: RunnableConfig) -> MyState:
    return state

workflow = StateGraph(MyState)
workflow.add_node("some_node", some_node) # Will have a type error

Error Message and Stack Trace (if applicable)
error: Argument 2 to "add_node" of "StateGraph" has incompatible type "Callable[[MyState, RunnableConfig], MyState]"; expected "Runnable[Any, Any] | Callable[[Any], Any] | Callable[[Any], Awaitable[Any]] | Callable[[Iterator[Any]], Iterator[Any]] | Callable[[AsyncIterator[Any]], AsyncIterator[Any]] | Mapping[str, Any]"  [arg-type]

Description
add_node has a type of RunnableLike, however RunnableLike's type does not account for a Callable that takes both the state and the RunnableConfig. In the how-to guides for passing runtime values into tools, it mentions that the second argument is a RunnableConfig.
System Info
python=3.10.4
langgraph==0.1.8
langchain==0.2.8
langchain-community==0.2.7
langchain-core==0.2.19
langchain-experimental==0.0.61
langchain-openai==0.1.16
langchain-text-splitters==0.2.2

@vbarda think this is generally applicable to RunnableLambda too ya?

I have previously opened a similar discussion https://github.com/langchain-ai/langgraph/discussions/1013
Changing the function to
def some_node(state: MyState, *, config: RunnableConfig | None = None) -> MyState:
    return state

will make the function has signature Callable[[MyState], MyState] which is compatible with RunnableLike``

@vbarda think this is generally applicable to RunnableLambda too ya?

yea i believe so

I had the same problem, prompting type error.
I once used RunnableLambda to wrap it and successfully eliminated the type warning.
But when I read the event "on_chain_stream" with name="xxx" it produces extra output.
This bothered me a lot and I ended up removing RunnableLambda.
from typing import Annotated, TypedDict

from langchain_core.runnables import RunnableConfig, RunnableLambda
from langgraph.graph import StateGraph, add_messages

class MyState(TypedDict):
    messages: Annotated[list, add_messages]

def some_node(state: MyState, config: RunnableConfig) -> MyState:
    return {"messages": [("human", "Foo Bar!")]}

workflow = StateGraph(MyState)
workflow.add_node("some_node", some_node)
# ^ Will have a type error hint. (But it can run)
workflow.set_entry_point("some_node")
workflow.set_finish_point("some_node")
app = workflow.compile()

workflow2 = StateGraph(MyState)
workflow2.add_node("some_node", RunnableLambda(some_node))
# No type error, but it generated some extra events in astream_event.
workflow2.set_entry_point("some_node")
workflow2.set_finish_point("some_node")
app2 = workflow2.compile()

async def main() -> None:
    async for event in app.astream_events({"messages": []}, version="v2"):
        if event["name"] == "some_node" and event["event"] == "on_chain_stream":
            print(event["data"])  # This will print the message once

    print("-" * 20)

    async for event in app2.astream_events({"messages": []}, version="v2"):
        if event["name"] == "some_node" and event["event"] == "on_chain_stream":
            print(event["data"])  # This will print the same message twice

if __name__ == "__main__":
    import asyncio

    asyncio.run(main())

@gbaian10 if the only reason to use RunnableLambda is to avoid type warnings, then definitely wrong

closing because issue seems solved in LangGraph 0.2

Think this PR: https://github.com/langchain-ai/langchain/pull/26295


Switch from Sabre\Event to ReactPHP

ReactPHP can use OS level event loops and has more interoperability with libraries like RxPHP.
The event loop is not global like Sabre's.

When checking React you should also check amphp which is developed more active and by more people

What makes you think that?

https://packagist.org/packages/amphp/amp/stats
https://packagist.org/packages/react/event-loop/stats

I see you are a contributor to amphp, if you see any advantages amphp has, I would be curious to hear

Under the amphp umbrella the guy behind icicle.io (another async php framework) and a few php core contributors joined forces to create a react php successor mainly with performance in mind as react php was too slow for them.
They created a great php based webserver https://github.com/amphp/aerys which proves the high performance usecases.
These guys are also the ones behind https://github.com/async-interop which aims to provide common abstractions so you can mix components of the different frameworks.
Regarding react: last time I looked up the github Organisation it was rather inactive. Today I can see that they are working again on the libs, so this argument seems no longer be accurate.
The number of downloads on packagist show more that the components are used instead of beeing actively developed. So take those with a grain of salt.

Thanks for the info. async-interop seems to be dead though.
How do the two compare feature-wise?

No support by RxPHP would be a deal breaker for me: https://github.com/ReactiveX/RxPHP/blob/master/src/Scheduler/EventLoopScheduler.php

With amp v2 (soon to be released), we're natively supporting React Promises in our coroutines and combinators, as well as also providing a proper React adapter (https://github.com/amphp/react-adapter) which will enable you to also use RxPHP too.
Also, amp provides a global event loop, which is superior here (no need to DI it everywhere; there anyway typically only ought to be a single event loop).
Yeah, async-interop is dead, amp v2 is going to be based on top of it.

I don't 100% see how support for ReactPHP Promises helps with supporting RxPHP Schedulers?
I wouldn't see global singleton event loop as that much of a benefit, I generally disliked that part of Sabre's API. I don't have a problem with passing explicit dependencies. You can make the argument of "typically only a single x" for a lot of things, Tracers, DB connections, ...

Right, but while you can execute multiple queries on multiple connections simultaneously (non-blocking), you can have only one event loop running at the same time. [Otherwise it blocks other loops.] That's why we went with that design choice.
100% support for ReactPHP Promises and the React Loop, will enable you to use RxPHP fully.

That's true, but there could be cases where you want to mock it in tests per-test.
Did you look at the scheduler class I linked? It's not related to promises.
Are there any other feature differences? Better perf is good of course

That's why we have https://github.com/amphp/phpunit-util
You require that repo and add the listener into the phpunit file (see the readme).
The scheduler is using the react loop though, which we have an adapter for (as linked above).
Better performance is definitely a goal of Amp (and perf is already very good - better than React).
Apart from that, I consider our libraries a bit more mature and feature complete than Reacts. Still - if you go with Amp, you still can use all the react libraries you wish to via the adapter.

Looking at the docs more (at least the available) I gotta say as a JS dev, I don't like the promise implementation very much. It is basically repeating the same mistakes and iterations JS went through until the community settled on Promises/A+. Error-first callbacks were exactly what we wanted to move away from, you want one catch at the end of your chain. Deferred objects were replaced by the revealing constructor pattern. The argument that the JS pattern doesn't make sense in a language with generators is weird given that JS has generators and coroutines too, and promises are what powers them. ReactPHP stays close to Promises/A.
The streams + loop API with the "cancel ID" looks a bit weird too.
Could you elaborate a bit why you thing amp is more mature / feature complete?

JS didn't have generators and coroutines when Promises/A+ was developed, and now they are stuck with them. There's no reason to chain callbacks on promises in a world with coroutines. Amp's promises reflect this: promises should be yielded in coroutines or passed to combinator functions (such as Amp\Promise\all()). Promise::onResolve() should almost be considered internal – only to be used by library code.
Instead of writing callbacks, code is written like synchronous code, only interjecting a yield where a function returns a promise. Exceptions are caught using a try/catch block instead of defining a callback in catch at the end of a chain.
If you're really insistent on then-chaining, I plan on releasing a library implementing both React's and Amp's promises in a single object.
The loop API is more like that of C event-loop libs and supports features that React's does not, such as multiple watchers on the same stream and signal handling.
As @bwoebi pointed out, an advantage to using Amp is that you have access to any lib written for React as well as any written for Amp.
Amp's libraries are generally more fully-featured than React's. The DNS library automatically reads your local resolver config instead of relying on Google's resolver by default. Aerys is a production-ready HTTP/2 & HTTP/1.1 + websocket server including middlewares, routing, and performance and security features. Amp has an async MySQL, Postgres, Redis, Stomp, and Beanstalk libs. amphp/parallel creates workers using other processes or threads that allows slow or blocking work to be done in a way that does not block the main process.
RxPHP does not require React's event loop for scheduling. Any function can be passed for creating timers, using the React event-loop is only a convenience feature. If using Amp, an instance of ReactAdapter can be given or a function calling Loop::delay() can be defined.


请问上传文件大小限制在哪里修改

The issue will be closed without any reasons if it does not satisfy any of following requirements:

Please speak English, we have forum in Chinese.
Please post questions or config/deploy problems on our forum: https://discuss.gogs.io, here are bugs and feature requests only.
Please take a moment to search that an issue doesn't already exist.
Please give all relevant information below for bug reports; incomplete details considered invalid report.

You MUST delete above content including this line before posting; too lazy to take this action considered invalid report.

Gogs version (or commit ref):
Git version:
Operating system:
Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gogs.io:

[ ] Yes (provide example URL)
[ ] No
[ ] Not relevant

Log gist (usually found in log/gogs.log):

Description
...

默认只能上传小于3M的文件，请问在哪里修改上传大小限制

https://github.com/gogs/gogs/blob/master/conf/app.ini

@DrayChou 请问使用docker 部署的gogs，可否通过环境变量的方式修改这个限制呢？


[SPARK-33675][INFRA][FOLLOWUP] Schedule branch-3.1 snapshot at master branch

What changes were proposed in this pull request?
Currently, master/branch-3.0/branch-2.4 snapshot publishing is successfully migrated from Jenkins to GitHub Action.

https://github.com/apache/spark/actions?query=workflow%3A"Publish+Snapshot"

This PR aims to schedule branch-3.1 snapshot at master branch.
Why are the changes needed?
This is because it turns out that GitHub Action Schedule works only at master branch. (the default branch).

https://docs.github.com/en/free-pro-team@latest/actions/reference/events-that-trigger-workflows#scheduled-events

Does this PR introduce any user-facing change?
No.
How was this patch tested?
The matrix triggering is tested at the forked branch.

https://github.com/dongjoon-hyun/spark/runs/1519015974

Test build #132441 has started for PR 30674 at commit 2d21e5b.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37042/

Thank you so much, @viirya !

Merged to master.

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37042/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/37042/

Test build #132441 has finished for PR 30674 at commit 2d21e5b.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/132441/

Thanks @dongjoon-hyun!

Thank you, @HyukjinKwon . Yes, this is completed now and Jenkins packing job are removed via SPARK-33712 . :)


Prevent service instance sharers from attempting to share with themselves

This PR restores the changes that were merged and reverted as part of #992.
It appears that there has been some misunderstanding/confusion.  We think this is what happened:
#988 was originally created for this story which resulted in #152759320 being created in CAPI's tracker. That PR was closed by SAPI as it needed reworking, instead #922 was opened. It appears that the original #988, which we assume had the acceptance performed against it, did not function. However we have deployed and checked #992 and it does work. So this PR simply restores the original commit that was reverted.
Hope that makes sense, let us know if you have any questions!
Thanks,
Sapi Team (Sam & @ablease)

[x] I have reviewed the contributing guide

[x] I have viewed, signed, and submitted the Contributor License Agreement

[x] I have made this pull request to the master branch

[x] I have run all the unit tests using bundle exec rake

[x] I have run CF Acceptance Tests

@elenasharma This is the reopened version of #992 & addresses the rejection of #152759320.
Thanks!


The lib works fine, but have a memory leak in events

Was wondering if you found any solution to the memory leak, I've been experiencing a similar leak.
Alternatively, is there another library that lets you set system volume in node without a leak?

Was wondering if you found any solution to the memory leak, I've been experiencing a similar leak.
Alternatively, is there another library that lets you set system volume in node without a leak?

@karacas How were you able to determine memory usage over an extended period of time? I'm having trouble getting a profiler to attach to the main node process.

@karacas How were you able to determine memory usage over an extended period of time? I'm having trouble getting a profiler to attach to the main node process.

+1


[Consensus] Add startup sync processor

Motivation
#2188
The startup sync processor is responsible for processing peer messages before an
event processor can be constructed. It will use the sync info in the messages to
try to get blocks(3-chain) for peers.
Test Plan
cargo x test -p consensus, current tests all passes
Will add additional tests when sync processor is being used in chained_bft_smr
TODOs
See Issue

@bors-libra r+

:pushpin: Commit 82e972b has been approved by ZhenjieRuan

:hourglass: Testing commit 82e972baf841228e66b429c0fadcf3ad4c98e25f with merge 1e959cd75861145cbb4c0badd3904fc758e23f5a...

:sunny: Test successful - checks-circle_commit_workflow
Approved by: ZhenjieRuan
Pushing 1e959cd75861145cbb4c0badd3904fc758e23f5a to master...


Error when using lookup table from tf.contrib.keras

I'm trying to build a simple model that includes string input and lookup in a lookup table, and embedding.
I get an error that appears to caused by using non keras layers in a keras model
the code:
import tensorflow.contrib.keras as keras
import tensorflow

input_layer = keras.layers.Input(shape=(1,), name='input', dtype='string')
lut = tensorflow.contrib.lookup.HashTable(tensorflow.contrib.lookup.KeyValueTensorInitializer(['a', 'b', 'c'], [1, 2, 3]), default_value=-1)
lut_layer = lut.lookup(input_layer)
embed = keras.layers.Embedding(input_dim=10, output_dim=5)(lut_layer)

model = keras.models.Model(inputs=[input_layer], outputs=[embed])

the error:
Traceback (most recent call last):
  File "/Users/ophir/dev/ophir/tf_keras_2.py", line 10, in <module>
    model = keras.models.Model(inputs=[input_layer], outputs=[embed])
  File "/Users/ophir/anaconda3/envs/tf_master/lib/python3.6/site-packages/tensorflow/contrib/keras/python/keras/engine/topology.py", line 1640, in __init__
    build_map_of_graph(x, seen_nodes, depth=0)
  File "/Users/ophir/anaconda3/envs/tf_master/lib/python3.6/site-packages/tensorflow/contrib/keras/python/keras/engine/topology.py", line 1631, in build_map_of_graph
    next_node = layer.inbound_nodes[node_index]
AttributeError: 'NoneType' object has no attribute 'inbound_nodes'

Process finished with exit code 1

using tensorflow on macos, built from source revision:<PHONE_NUMBER>5ca3b728d6eec298717c8a8558639f

@fchollet any idea?

For the time being, any op that is not a Keras layer should be wrapped inside a Keras Lambda layer:
inputs = keras.layers.Input(shape=(1,), name='input', dtype='string')
lut = tensorflow.contrib.lookup.HashTable(tensorflow.contrib.lookup.KeyValueTensorInitializer(['a', 'b', 'c'], [1, 2, 3]), default_value=-1)
lut_output = keras.layers.Lambda(lut.lookup)(inputs)

If the wrapping is broken at any point, then it prevents you from being able to build a Keras model with your inputs/outputs later on.
We will automate this wrapping in a later version.

@fchollet, I am also getting this problem, even after wrapping with a Lambda layer. Particularly:
permuted = Lambda((lambda x: K.permute_dimensions(x, pattern=(0,2,1))))(output)
When I try to define the model which includes this layer, I get the same error as @ophiry posted above.
Is there any way around this problem?

@fchollet, what if the non-Keras layer has a backprop function and/or trainable params? How can these be added to a Keras model? A Lambda layer would not handle the backpropagation / trainable params, correct?

If you have parameters to train, you should write a custom layer:
https://keras.io/layers/writing-your-own-keras-layers/
On 13 July 2017 at 00:40, Hans Gaiser<EMAIL_ADDRESS>wrote:

@fchollet https://github.com/fchollet, what if the non-Keras layer has
a backprop function and/or trainable params? How can these be added to a
Keras model? A Lambda layer would not handle the backpropagation /
trainable params, correct?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/8617#issuecomment-314997898,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AArWbwXyeZL-v4ydQ-GqQVoEez-NE5dLks5sNcoJgaJpZM4Mky9s
.

If you have parameters to train, you should write a custom layer:
https://keras.io/layers/writing-your-own-keras-layers/

Thanks @fchollet, I think I'm beginning to understand the situation. In my specific case I'm tring to use a layer with no trainable weights (RoiPooling) but it does have a backward propagation function. If I register the op in the C API with REGISTER_OP, wrap it in a Keras Lambda layer and register the backward propagation function with the same op name (currently only possible in python with the @ops.RegisterGradient decorator), then tensorflow (and thus Keras) should be able to compute correct gradients. I believe this is correct and I will continue to work under that assumption. If it is not correct then I would gladly hear about it :)

Keras delegates gradient computation to TF. If you have custom
differentiable ops that don't have weights, a Lambda layer works fine.
On 15 July 2017 at 09:37, Hans Gaiser<EMAIL_ADDRESS>wrote:

If you have parameters to train, you should write a custom layer:
https://keras.io/layers/writing-your-own-keras-layers/
Thanks @fchollet https://github.com/fchollet, I think I'm beginning to
understand the situation. In my specific case I'm tring to use a layer with
no trainable weights (RoiPooling) but it does have a backward propagation
function. If I register the op in the C API with REGISTER_OP, wrap it in
a Keras Lambda layer and register the backward propagation function with
the same op name (currently only possible in python with the
@ops.RegisterGradient decorator), then tensorflow (and thus Keras) should
be able to compute correct gradients. I believe this is correct and I will
continue to work under that assumption. If it is not correct then I would
gladly hear about it :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/8617#issuecomment-315545819,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AArWb_IW-Mm04tVs1YHIzNBJBhZenWc7ks5sOOragaJpZM4Mky9s
.


Update modman

The problem here is that if you leave it as it is, and after you have installed the latest patch on your magento installation, since the composer strategy is changed to copy, the mapping is completely wrong. So every time I do a composer install, I see Adyen/Payment/Fee/Fee. Then again  Adyen/Payment/Fee/Fee/Fee and on and on. This needs to fixed and released because can cause major problems.

Had the same issue, never knew this solved it. THanks for fixing it!

Hi @cstergianos,
Thanks for the PR. I will merge in into our branch.
Regards,
Rik
Adyen

@rikterbeek facing the same issue, when will there be a new release? Having a hard time installing 2.9.1 properly.. :sweat_smile:


While configuring a "regular expression" filter, Weevil shall provide the option to select the case sensitivity without requiring a RegEx modifier.

Obsoleted by:

#394


Add new commands for filtering tests to run

Add new commands for executing subset of test classes:

[x] Current open Test Class
[x] All open Test Class
[ ] All tests for the current Namespace
[ ] All tests for the current Class

Good stuffs! :+1:


dump only dumping one tabe

When I run rake db:seed:dump only one (my first) table is being dumped to my seeds.rb file :(
Am i doing something wrong ?
Thanks,
Jon.

:+1:
I am having the same issue except only 3 of my 32 tables are being dumped when I run "rake db:seed:dump". I cannot see anything I am doing wrong.

@ljonny18 I created a PR that fixes this issue but if you want a quick solution, open seed_dump/lib/seed_dump/dump_methods.rb and change this line:
record.attributes.select {|key| key.is_a?(String)  }.each do |attribute, value|
to
record.attributes.select {|key| key.is_a?(String) || key.is_a?(Symbol) }.each do |attribute, value|

Thanks @XXChester, I merged your PR.


daily clifile editor not properly handling exceptions in threads

Found stacked up daily_clifile_editors with exceptions not properly handled within threads.

The issue here is that the exception is happening within the callback, which is a big no-no.


Setting start timecode of clips split with split-video

Similarly to https://github.com/Breakthrough/PySceneDetect/pull/291#issuecomment-1288973981
It would be cool to be able to force the start timecode of a clip to be baked as the "ffmpeg seek timecode", either through the custom ffmpeg options or maybe a global one like -timecode -tc like how -crf is used in pyscenedetect

I am not 100% clear on what the feature being requested is. What is your intended usage? Looking at #291 do you want an option to specify a start time for splitting the video and ignoring all the video/scenes prior to that start time?


error: failed to load icon quote-alt-left

When I use the icon component in react, and I try to use
<Box-icon name="quote-alt-left" type="solid" />

I get the following error (icon displays correctly, though)
Failed to load icon: quote-alt-left
Error: 404 Cannot find '/svg/regular/bx-quote-alt-left.svg' in<EMAIL_ADDRESS>
I took a peek in the js, and I couldn't find anything amiss. Thanks!

Same issue here.

I don't know its working when I try it on code pen Here

I'm not saying it's not working - it simply gives 404 errors in console for some of the icons(in my case pinterest, instagram and twitter, solid type).

Tbh I have no clue then. Is it because of the cdn server being down?

Huh. Working for me as well now. Feel free to close.


performance:  Stopwatch takes its time

This issue was originally filed by @tatumizer

Stopwatch, widely used to benchmark performance, itself performs very badly! One invocation of elapsedMicroseconds takes 0.3 microseconds in 32-bit mode! Which makes it nearly impossible to implement any performance tool based on stopwatch - you will be benchmarking Stopwatch instead of a program.
The culprit: external method _now that returns huge number that doesn't fit in smi.
There's very easy fix: store start time as double and return _now as double. 52 integer bits in double is more than enough. This will do wonders in terms of speed. On 64-bit processors, performance will be more or less the same as with ints.

Fwiw, I just ran this on my machine, and if I read the data correctly, it's now much faster.
Running the program on my machine takes 556413 (desktop) or 605918 (notebook) microseconds, which means that the elapsedMilliseconds getter took ~60 nanoseconds.
This was tested on my desktop (Intel(R) Xeon(R) CPU E5-2690 0 @ 2.90GHz) and my notebook (Intel(R) Core(TM) i7-4600U CPU @ 2.10GHz).
elapsedTicks is ~10% faster.

From my last measurements it looks like elapsedMilliseconds has acceptable performance now.


Should Dependabot be updating lock (such as Gemfile.lock) files?

Is there an existing issue for this?

[X] I have searched the existing issues

Package ecosystem
Bundler
Package manager version
No response
Language version
No response
Manifest location and content before the Dependabot update
The /Gemfile.lock file was updated for a new version of a dependency.
diff --git a/Gemfile.lock b/Gemfile.lock
index 56f03bb..a407842 100644
--- a/Gemfile.lock
+++ b/Gemfile.lock
@@ -195,7 +195,7 @@ GEM
       rspec-mocks (~> 3.12)
       rspec-support (~> 3.12)
     rspec-support (3.12.1)
-    rubocop (1.54.1)
+    rubocop (1.54.2)
       json (~> 2.3)
       language_server-protocol (>= 3.17.0)
       parallel (~> 1.10)

dependabot.yml content
version: 2
updates:
  - package-ecosystem: "bundler"
    directory: "/"
    schedule:
      interval: "weekly"

Updated dependency
rubocop from 1.54.1 to 1.54.2
What you expected to see, versus what you actually saw
It's not clear to me if Dependabot updates transitive dependencies.
What I saw when running bundler outdated --strict and bundler update for rubocop matches, but these commands also updated other dependencies. However, it only shows me that this particular case works. It does not demonstrate that all necessary rules for transitive dependencies would be applied or rules across the Gemfile would be applied.
At this point, I don't have any evidence that I can trust Dependabot's PRs to do all of the necessary updates. At most, I can use them as a flag to run bundler outdated --strict and bundler update and apply those changes. And at this point, I'd rather have an issues created and managed rather than a PR.
Native package manager behavior
I ran bundle outdated --strict on the same branch. This showed 4 gems that could be updated: irb, public_suffix, rspec-mocks, and rubocop. Of these, only rubocop is in the Gemfile. The others are transitive dependencies that can be updated and still meet the requirements in the Gemfile.
The output for rubocop was exactly what Dependabot suggested. This only indicates that no transitive updates needed to be updated in a way that changed the ruled in the Gemfile.lock and says nothing about the correctness of the behavior should they need to be updated.
Images of the diff or a link to the PR, issue, or logs
No response
Smallest manifest that reproduces the issue
No response

Have you tried the following config in dependabot.yml:
    allow:
      - dependency-type: all

@armin-joellenbeck Looking at the documentation, I don't see how that helps.
Dependabot shouldn't be touching lock files. For a Ruby project, Dependabot should only be looking at the Gemfile. Similarly, for a Node project, it should only be looking at the package.json file. Although it is good to know if an update exists in a transitive dependency, it is dangerous to update a transitive dependency by editing a lock file. The packages that use that dependency may not work with a different version. You could make a claim that if everyone is using semantic versioning and has correct version pinning, it "should work", but that's a risky assumption.
That doesn't mean that Dependabot shouldn't look at lock files. Knowing that a transitive dependency is outdated or has a vulnerability is a good thing. But if an update is going to be suggested, it needs to be updated in a safe way, and making changes to a lock file is not safe. The default behavior should not be an unsafe operation.
If I'm mistaken, I'd like someone to point me to why updating a lock file is a safe and accepted practice. Otherwise, I think the right course of action is to ensure that Dependabot pull requests are only made against the user-controlled dependency file. If desired, issues could be open to inform the author of outdated or vulnerable transitive dependencies so the maintainer can review and take the appropriate action.

I think the proposed config makes the difference.
With it dependabot updates also indirect dependencies in Gemfile.lock.
Without it dependabot only updates dependencies in Gemfile.
Which choice is preferable, is up to you. I tend to the first one. But there are valid reasons to prefer the second.
I hope I have not misunderstood your pupose. Sorry, if this is the case.

@armin-joellenbeck OK, I think I see what you're saying.
If I'm understanding, you're suggesting that Dependabot may be acting like bundle update --conservative by default. This prevents Bundler from updating indirect/transitive dependencies.
If I make your proposed change, then Dependabot should act like bundle update.
Coming from a Ruby background, this is very strange behavior. I can't remember a time when I ran bundle update --conservative - I don't think I ever had. It's not part of the recommended workflow, either.
I do have to experiment to confirm, but I'd still consider this a bug in Dependabot. It doesn't fit what I would consider to be the expectations of a Ruby developer. For the bundler ecosystem, if you are correct, then dependency-type: all should be the default behavior, since that is closer to the standard workflow for a Ruby developer updating dependencies on their own. It would be extremely helpful if the default, out-of-the-box behavior of Dependabot mirrors the manual behaviors of Ruby developers.

I see, that we agree about the preferred workflow. But if the developers of dependabot prefer the conservative approach as default, it is ok for me, because it is configurable.
Nonetheless I am interested in the outcome of your experiments.


Hello! Why not try my GL553VD EFI?

it is in this : https://git.lug.ustc.edu.cn/QiuChenly/ASUS_FX53VD_10.13.1EFI
What's not working:
1.Keyboard backlight control and FN.
2.you tell me.
What's working:
The sleep, the trackpad, the Apple ALC, and all the other special features.

his efi is currently having the same issues as you

Hi.
because this laptop has a USB keyboard and it's aura keyboard it needs Asus aura core API for Linux !!
but sleeping, trackpad, APPLEALC (id 16 add for gl553vd ) and others are working good.