Problem building pyatac.fragments

Hi @AliciaSchep ,
I'm trying to run nucleoatac run and having an issue:
nucleoatac run --bed test_sorted.bed --bam ctc1_d15_combined.bam --out test_sorted --fasta ../../external_data/hg19/hg19.fa
Command run:  /raid/development/cdeboever/software/anaconda1.9.1/envs/cardips-tc/bin/nucleoatac run --bed test_sorted.bed --bam ctc1_d15_combined.bam --out test_sorted --fasta ../../external_data/hg19/hg19.fa
nucleoatac version 0.1.4
start run at: 2015-05-14 14:03
/home/cdeboever/.pyxbld/temp.linux-x86_64-2.7/pyrex/pyatac/fragments.c:250:31: fatal error: numpy/arrayobject.h: No such file or directory
 #include "numpy/arrayobject.h"                               ^
compilation terminated.
Traceback (most recent call last):
  File "/raid/development/cdeboever/software/anaconda1.9.1/envs/cardips-tc/bin/nucleoatac", line 217, in <module>
    main()  File "/raid/development/cdeboever/software/anaconda1.9.1/envs/cardips-tc/bin/nucleoatac", line 54, in main
    from nucleoatac.run_occ import run_occ
  File "/raid/development/cdeboever/software/anaconda1.9.1/envs/cardips-tc/lib/python2.7/site-packages/nucleoatac/run_occ.py", line 19, in <module>
    from nucleoatac.Occupancy import FragmentMixDistribution, OccupancyParameters, OccChunk
  File "/raid/development/cdeboever/software/anaconda1.9.1/envs/cardips-tc/lib/python2.7/site-packages/nucleoatac/Occupancy.py", line 12, in <module>
    from pyatac.fragmentsizes import FragmentSizes
  File "/raid/development/cdeboever/software/anaconda1.9.1/envs/cardips-tc/lib/python2.7/site-packages/pyatac/fragmentsizes.py", line 10, in <module>
    from pyatac.fragments import getAllFragmentSizes, getFragmentSizesFromChunkList  File "/raid/development/cdeboever/software/anaconda1.9.1/envs/cardips-tc/lib/python2.7/site-packages/pyximport/pyximport.py", line 431, in load_module    language_level=self.language_level)
  File "/raid/development/cdeboever/software/anaconda1.9.1/envs/cardips-tc/lib/python2.7/site-packages/pyximport/pyximport.py", line 209, in load_module
    inplace=build_inplace, language_level=language_level)
  File "/raid/development/cdeboever/software/anaconda1.9.1/envs/cardips-tc/lib/python2.7/site-packages/pyximport/pyximport.py", line 186, in build_module
    reload_support=pyxargs.reload_support)
  File "/raid/development/cdeboever/software/anaconda1.9.1/envs/cardips-tc/lib/python2.7/site-packages/pyximport/pyxbuild.py", line 100, in pyx_to_dll
    dist.run_commands()
  File "/raid/development/cdeboever/software/anaconda1.9.1/envs/cardips-tc/lib/python2.7/distutils/dist.py", line 953, in run_commands
    self.run_command(cmd)
  File "/raid/development/cdeboever/software/anaconda1.9.1/envs/cardips-tc/lib/python2.7/distutils/dist.py", line 972, in run_command    cmd_obj.run()
  File "/raid/development/cdeboever/software/anaconda1.9.1/envs/cardips-tc/lib/python2.7/site-packages/Cython/Distutils/build_ext.py", line 163, in run    _build_ext.build_ext.run(self)
  File "/raid/development/cdeboever/software/anaconda1.9.1/envs/cardips-tc/lib/python2.7/distutils/command/build_ext.py", line 337, in run
    self.build_extensions()  File "/raid/development/cdeboever/software/anaconda1.9.1/envs/cardips-tc/lib/python2.7/site-packages/Cython/Distutils/build_ext.py", line 171, in build_extensions
    self.build_extension(ext)  File "/raid/development/cdeboever/software/anaconda1.9.1/envs/cardips-tc/lib/python2.7/distutils/command/build_ext.py", line 496, in build_extension
    depends=ext.depends)  File "/raid/development/cdeboever/software/anaconda1.9.1/envs/cardips-tc/lib/python2.7/distutils/ccompiler.py", line 574, in compile
    self._compile(obj, src, ext, cc_args, extra_postargs, pp_opts)
  File "/raid/development/cdeboever/software/anaconda1.9.1/envs/cardips-tc/lib/python2.7/distutils/unixccompiler.py", line 122, in _compile    raise CompileError, msg
ImportError: Building module pyatac.fragments failed: ["CompileError: command 'gcc' failed with exit status 1\n"]

I have cython 0.22 and numpy 1.9.1 installed with python 2.7.9. I'm also running on openSUSE 13.2. I'm trying to debug it a bit but I haven't used cython much so I haven't had too much luck.

I have made a change* that I think may fix this issue in a branch called "issue4".  @cdeboever3, would you be able to try checking that branch out to see if the change fixes the problem on your system?
*changing pyximport.install() to pyximport.install(setup_args={"include_dirs": np.get_include()}) within all applicable scripts

Thanks @AliciaSchep that solved it. I was able to use nucleoatac run for a small set of intervals successfully. Cool stuff.

Great, thanks for trying the new branch.  I have merged "issue4"  into the master branch.


Added GetXrmTheme and ImportXrmTheme commands

Not sure why this has included the changes that you'd already merged - I have synced yours back to my fork.  Might just be me not understanding git properly...

@MattTrinder1 thanks this is great. Why do you need the ThemeInfo class? Do you think using an early bound class similar to GetXrmSolutionCommand would have the same effect?

@WaelHamze I was following the same pattern as the XrmSolutionInfo class - keeping it as simple as possible and only exposing the relevant properties needed to export/import a theme.

@MattTrinder1 apologies for delay in response, Can you please out the XrmCommandBase from this PR? This has significantly changed since you submitted the PR and it already handles the TLS.
Once that is done I'll merge it. It might be useful if these Cmdlets can save and import a theme from a json file.

Have merged in all your latest changes, and added a RemoveSolution cmdlet


[Issue]: How can I access follow-up questions from the assistant when human input is set to ALWAYS? Currently, it only interacts via the terminal.

Describe the issue
When I set human input to ALWAYS, I was able to interact via the terminal. How do I make this available to the web UI or frontend?
Steps to reproduce
No response
Screenshots and logs
No response
Additional Information
No response

One way is to TERMINATE on human input, then resume the chat with the additional human input included

The same issue, I now want to predefine multi-turn inputs for humans to make the conversation between humans and machines automatic. Is there a way to predefine the response for an agent?

@Venkysanju246 for UI integration, we will be adding documentation in the v0.4 (preview).
@xin-ran-w. In the v0.2 version, you can register reply functions to customize agent responses. https://microsoft.github.io/autogen/0.2/docs/reference/agentchat/conversable_agent#register_reply. In the v0.4 preview, you are just write Python code to define your agent's message handler and take pre-defined responses.


Any reason why server module is being used?

const server = require('server');
Any reason why server module is being used? For such a simple controller you could've used the "raw" SFCC APIs. This would've made the cartridge work anywhere, be standalone and not be dependent of SFRA anymore.
The translated controller would be something like this:
"use strict";

/**
 * Display development console template
 */
function Show() {
    var URLUtils = require("dw/web/URLUtils");
    var System = require("dw/system/System");
    var ISML = require("dw/template/ISML");

    if (!request.isHttpSecure()) {
        response.redirect(URLUtils.https("Console-Show"));
        return;
    }

    // We only allow on DEVELOPMENT_SYSTEM and disallow on STAGING_SYSTEM as well
    // since that is just as sensitive as PROD (due to data replication)
    if (System.getInstanceType() !== System.DEVELOPMENT_SYSTEM) {
        response.setStatus(418);
        response.getWriter().print("I'm a teapot!");
        return;
    }

    ISML.renderTemplate("dev_console/index", {
        urlPath: URLUtils.https("").toString() + "/default",
        staticPath: URLUtils.staticURL("/").toString(),
    });
}

module.exports.Show = Show;
module.exports.Show.public = true;

/**
 * Run the script and return the response
 */
function Run() {
    var URLUtils = require("dw/web/URLUtils");
    var System = require("dw/system/System");

    if (!request.isHttpSecure()) {
        response.redirect(URLUtils.https("Console-Show"));
        return;
    }

    // We only allow on DEVELOPMENT_SYSTEM and disallow on STAGING_SYSTEM as well
    // since that is just as sensitive as PROD (due to data replication)
    if (System.getInstanceType() !== System.DEVELOPMENT_SYSTEM) {
        response.setStatus(403);
        response.setContentType("application/json");
        response.getWriter().print(
            JSON.stringify({
                error: true,
                message: "Only available in SIG environments",
            })
        );
        return;
    }

    const code = request.getHttpParameterMap().get("code").getStringValue("");
    const maxDepth = request.getHttpParameterMap().get("maxDepth").getStringValue("");

    if (!code || !maxDepth) {
        response.setStatus(418);
        response.setContentType("application/json");
        response.getWriter().print(
            JSON.stringify({
                error: true,
                message: "I'm a teapot!",
            })
        );
        return;
    }

    let result = null;

    try {
        const myFunc = new Function("code", code);
        result = myFunc();
    } catch (e) {
        result = e;
    }

    const serializer = require("../scripts/serializer");
    result = serializer.serialize(result, maxDepth);

    if (typeof result === "string" || typeof result === "boolean" || typeof result === "number") {
        response.setContentType("application/json");
        response.getWriter().print(JSON.stringify([result]));
        return;
    }

    response.setContentType("application/json");
    response.getWriter().print(JSON.stringify(result || {}));
}

module.exports.Run = Run;
module.exports.Run.public = true;

@danechitoaie interesting, and good catch.  Any chance you can fire off a PR for this update?  Then we can test this baby out and make sure everything still works, and you get tagged for the fix ;)

Sure, I can do a PR, but in the morning 😆 , it's 2 AM here.

Done in #24


Implement the JS::Object#to_b method.

A boolean value in JavaScript is always a JS::Object instance from Ruby's point of view.
Therefore, if we use the boolean value returned by a JavaScript function as the condition for an if expression in Ruby, the if expression will always be true.
if searchParams.has('phrase')
  # Always pass through here.
  ...
else
  ...
end

Think of searchParams as an instance of JavaScript's SearchParams class.
This is not the expected behavior.
Therefore, we can implement the JS::Object#to_b method to convert JavaScript boolean values to Ruby boolean values.
It is intended to be used in the following ways:
if searchParams.has('phrase').to_b
  ...
else
  ...
end

I implemented the to_b method by referring to the implementation of the to_s method.
If you prefer a different implementation, please let me know.

We did not consider the point of matching the standard Ruby method definitions.
I agree with the policy to define JS::True and JS::False.


Fix pyarrow and dask parquet issues

This is the final set of fixes identified in issue #86. CI works locally for me now.
The changes are all related to API changes and deprecations in pyarrow between 5.0.0 and 8.0.0. Changes were required in our code to directly deal with these as well as other changes following dask modifications to deal with the same.
I have created a new script _create_testdata.py to create test parquet files that are stored in the new tests/test_data directory and these are checked as part of pytest. The last time the CI definitely worked was July 2021 with pyarrow==5.0.0 and dask==2021.7.2 (and the same for distributed). These files are successfully read with up-to-date pyarrow==8.0.0 and dask==2022.7.1. Similarly, test parquet files create with the up-to-date pyarrow and dask are successfully read with pyarrow==5.0.0 and dask==2021.7.2.

CI passing on github actions on ubuntu using Python 3.8. Expanding test matrix to cover Python 3.7 to 3.10 and all 3 major platforms.


fix: 组件的生命周期响应问题

What kind of change does this PR introduce? (check at least one)

[x] Bugfix
[ ] Feature
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:

[ ] It's submitted to the dev branch for v2.x (or to a previous version branch), not the master branch
[ ] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[ ] All tests are passing
[ ] New/updated tests are included

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:

修改mpType为component时，组件的ready响应有onReady改为ready，对应微信官方组件的生命周期
增加组件的生命周期钩子attached，ready，moved，detached

补充一下 component-lifestyle 相关的测试到 test/mp/runtime 吧

@anchengjian  👌

@anchengjian 大佬测试用例加上了，麻烦看一下是否有问题～

问一下组件生命周期这个在哪个版本可以使用？

最新的版本就可以用，有啥问题么？

这个 pr 解决的场景是什么？ready、attached 这些生命周期只有小程序的 自定义组件 才有，但是目前 mpvue 组件是用 template 实现的，所以不会有 ready 等事件，只有页面的 onReady，而小程序原生的自定义组件应该不经过 mpvue 处理吧


Configuration instructions and Screen shots are outdated for forgerock 7.2.0

Description:
The forgerock 7.2.0 is the latest Forgerock version and the configuration steps has some major changes in the below steps and new screen shots are added delow.
Step 1.1 - Configure OAuth 2.0 Provider
Step 5.
Here an Oauth provider is not attached to the Realm by default in the latest versions. So we first need to go to Realm -> Services and add Oauth2 provider as a new service as follows.

Step 1.2 - Configure an OAuth 2.0 client
Step 1.
Here the scopes needed for DCR should be added to default scopes as well as given below.

Thanks.
Suggested Labels:

Suggested Assignees:

Affected Product Version:
OS, DB, other environment details and versions:
Steps to reproduce:
Related Issues:

Fixed from https://github.com/wso2/docs-apim/issues/6669


Background color in card variant header/footer

Headers and footers in card variants are practically indistinguishable from the card body since the background color is the same, where in the standard card a slightly darker background gives them their own visual semantic.

Is this intended? In my opinion headers and footers in variants should follow the default visual behavior.

+1

Punted on the fix, so I'll punt on this for now, too. See #20649.


Cancelling sort using custom helper causes list to display in incorrect order

Hi, I forked the simple codepen to show my issue.
I have to create a custom helper in my code to fix the offset scroll issue. I have an issue when after I  cancel the sort a few times.
See codepen
Codepen
If you drag the top item to the bottom of the list 3 times it ignores the cancel event. Now the list is displayed incorrectly and if you try sorting other items you see the log shows the model item orders do not match the list displayed in the left hand side.
Been trying to work out what is going on to but am none the wiser.
Please advise.

I was able to replicate. I will try to investigate during the weekend.

Resolved by #493 and released as v1.5.1 .
Please confirm that it works for you.

Works perfect, thks loads


Build MVP pipeline for NEET report

MVP is automated generation of a file containing the details identified:
Seasonally adjusted totals (People - SA tab) for 16-24 Totals, Unemployed, Economically inactive and Total People in relevant group.
Essentially columns 1-4 of this table

https://www.ons.gov.uk/employmentandlabourmarket/peoplenotinwork/unemployment/datasets/youngpeoplenotineducationemploymentortrainingneettable1
Seasonally Adjusted - first 4 figures (no
16-24

Have processed (with little data adornment!) the NEET stats into this file
https://github.com/open-innovations/yff-data/blob/main/data/neet/neet_16_to_24.csv
@taz-chiles Once we have the same for QLFS we should share with client to check

The output NEET data is collected here:

https://github.com/open-innovations/yff-data/tree/main/data/neet
https://github.com/open-innovations/yff-data/blob/main/data/neet/neet_16_to_24.csv

Processing applied via this script: https://github.com/open-innovations/yff-data/blob/main/scripts/neet/transform.py
Steps:

Load People - SA sheet from Excel file
Convert the string representation of quarter (e.g. Jan-Mar 2023) to a proper date time representing the start of the quarter.
Select the 16-24 age range column set
Rename the columns and set to a single level index
Extract the most recent 3 years data and save as a CSV.


Not compatible with leptos v4.0.0 +

When building in release mode with trunk and<EMAIL_ADDRESS>trunk build --release, I'm receiving this error:
the trait leptos::Component<_> is not implemented for fn item fn(leptos_reactive::scope::Scope, IconProps<_>) -> impl leptos_dom::IntoView {leptos_icons::Icon::<_>}

This is also the case with cargo leptos.

This is expected, as only the Component trait from Leptos 0.3 is implemented for the icon component function when you depend on a version of leptos-icons targeting leptos 0.3 and target leptos 0.4 yourself. Different versions of the same type are present when compilation runs leading to this kind of error.

Is there a version which targets 4.0+?
On Sun, 2 July 2023, 6:52 am Lukas Potthast, @.***>
wrote:

This is expected, as only the Component trait from Leptos 0.3 is
implemented for the icon component function when you depend on a version of
leptos-icons targeting leptos 0.3 and target leptos 0.4 yourself. Different
versions of the same type are present when compilation runs leading to this
kind of error.
—
Reply to this email directly, view it on GitHub
https://github.com/Carlosted/leptos-icons/issues/16#issuecomment-1616109844,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATPWGTCNL6W26GO6GL2QMX3XOCEYPANCNFSM6AAAAAAZ2LGD24
.
You are receiving this because you authored the thread.Message ID:
@.***>

Made a PR for it. Until released as leptos-icons 0.0.15, you can already try it out using
leptos-icons = { git = "https://github.com/lpotthast/leptos-icons", branch = "leptos_0.4", features = [
    // ...
] }

This is closed by #17

Thank you 😊
On Mon, 3 July 2023, 3:43 am Charles Edward Gagnon, <
@.***> wrote:

Closed #16 https://github.com/Carlosted/leptos-icons/issues/16 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/Carlosted/leptos-icons/issues/16#event-9700762051,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATPWGTHSCTULWY7BHLXUZRDXOGXK3ANCNFSM6AAAAAAZ2LGD24
.
You are receiving this because you authored the thread.Message ID:
@.***>

FYI: Usually, I don't depend on non release dependencies unless if I faced some blockers. Unfortunately<EMAIL_ADDRESS>is broken again with the git dependency of leptos
based on examples/simple-app, replace leptos dependency to # leptos = { git = "https://github.com/leptos-rs/leptos", features = ["csr"] } and build again to receive these errors:
the trait ComponentConstructor<_> is not implemented for &fn(leptos_reactive::scope::Scope, IconProps) -> impl leptos_dom::IntoView {leptos_icons::Icon}
the trait leptos::Component<_> is not implemented for fn item fn(leptos_reactive::scope::Scope, IconProps) -> impl leptos_dom::IntoView {leptos_icons::Icon}


Runtime crash when default or rest parameters of called function types are not provided

Description:

$title
Steps to reproduce:
1.
public function main() {
    function (boolean|float,string="") returns string? func = (p,q) => "someString";
    string? funcval = func(2.0);
}

will give
Running executable

[2022-03-22 12:40:56,226] SEVERE {b7a.log.crash} - Bad local variable type
Exception Details:
  Location:
    issue.main(Lio/ballerina/runtime/internal/scheduling/Strand;)Ljava/lang/Object; @513: aload
  Reason:
    Type top (current frame, locals[35]) is not assignable to reference type
  Current Frame:
    bci: @513
    flags: { }
    locals: { 'io/ballerina/runtime/internal/scheduling/Strand', 'io/ballerina/runtime/internal/values/FPValue', 'java/lang/Object', 'java/lang/Object', double, double_2nd, 'io/ballerina/runtime/api/values/BString', long, long_2nd, long, long_2nd, 'io/ballerina/runtime/api/values/BString', long, long_2nd, long, long_2nd, 'io/ballerina/runtime/api/values/BString', long, long_2nd, long, long_2nd, 'io/ballerina/runtime/api/values/BString', long, long_2nd, long, long_2nd, 'java/lang/Object', 'io/ballerina/runtime/internal/values/ErrorValue', 'java/lang/Object', integer }
    stack: { 'java/util/function/Function', '[Ljava/lang/Object;', '[Ljava/lang/Object;', integer }
  Bytecode:
    0000000: 2ab4 0011 9900 07b8 0017 bf01 4c01 4d01
    0000010: 4e0e 3904 013a 0609 3707 0937 0901 3a0b
    0000020: 0937 0c09 370e 013a 1009 3711 0937 1301
    0000030: 3a15 0937 1609 3718 013a 1a01 3a1b 013a
.
.
.
    same_frame(@885)
    full_frame(@896,{Object[#13],Null,Null,Null,Double,Null,Long,Long,Null,Long,Long,Null,Long,Long,Null,Long,Long,Null,Null,Null,Integer},{})
    full_frame(@1029,{Object[#13],Object[#92],Object[#4],Object[#4],Double,Object[#281],Long,Long,Object[#281],Long,Long,Object[#281],Long,Long,Object[#281],Long,Long,Object[#4],Object[#187],Object[#4],Integer},{})

        at $_init.$lambda$main$(.)
        at io.ballerina.runtime.internal.scheduling.SchedulerItem.execute(Scheduler.java:610)
        at io.ballerina.runtime.internal.scheduling.Scheduler.run(Scheduler.java:316)
        at io.ballerina.runtime.internal.scheduling.Scheduler.runSafely(Scheduler.java:283)
        at java.base/java.lang.Thread.run(Thread.java:829)

public function main() {
    function (boolean|float,string...) returns string? func = (p) => "someString";
    string? funcval = func(2.0);
    // string? funcval = func(2.0,"jj"); // This will also give the same error
}

will give
Running executable

[2022-03-22 12:46:04,192] SEVERE {b7a.log.crash} - 'java.lang.Object issue.$lambda$_0(io.ballerina.runtime.internal.scheduling.Strand, java.lang.Object, boolean)'
java.lang.NoSuchMethodError: 'java.lang.Object issue.$lambda$_0(io.ballerina.runtime.internal.scheduling.Strand, java.lang.Object, boolean)'
        at issue.$lambda$_0$lambda0$(issue.bal:2)
        at issue.main(issue.bal:3)
        at $_init.$lambda$main$(.)
        at io.ballerina.runtime.internal.scheduling.SchedulerItem.execute(Scheduler.java:610)
        at io.ballerina.runtime.internal.scheduling.Scheduler.run(Scheduler.java:316)
        at io.ballerina.runtime.internal.scheduling.Scheduler.runSafely(Scheduler.java:283)
        at java.base/java.lang.Thread.run(Thread.java:829)

Here also needs to check whether this should be a compilation error unless two args should be there (i.e. func = (p,q) => "someString";) because when we write functions with rest args we write like this
function func(int p, string... q) returns string? {
    return "someString";
}

Affected Versions:
Ballerina 2201.0.2 (Swan Lake)
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

Most probably fixed in https://github.com/ballerina-platform/ballerina-lang/pull/31589, merged to master an hour ago. cc @chiranSachintha

This is fixed with https://github.com/ballerina-platform/ballerina-lang/pull/31589
@Nadeeshan96 Can you recheck on master

I checked with the master branch. The first one is solved, but the second one (one with rest parameters) is not solved.

As the following
public function main() {
    function (boolean|float,string...) returns string? func2 = function (boolean|float p, string... q) returns string? { return "someString";};
    string? funcval1 = func2(2.0);
    string? funcval2 = func2(2.0,"jj");
}

works fine, it seems the correct way to write the expression-bodied function is this.
function (boolean|float,string...) returns string? func = (p, q) => "someString";

But currently it gives a compilation error when written like that and no compilation error when written like this.
function (boolean|float,string...) returns string? func = (p) => "someString";

So, other way around should be the correct way and the compilation error should be corrected.


SPARK-36770 Replace Unbounded Following window functions with Unbounded Preceding window function (First, Last)

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

Can one of the admins verify this patch?


Folder structure

Feature PR
Re-organised the folder structure for route handlers.
Changed the folder structure from having a folder named routes with all the different handlers (i.e. User.routes.ts), to having for each route handler (i.e. User, UserPosts) its own folder.
Then for each endpoint it has been created its own file (i.e. /Users/getAccountInfo.ts , /User/getUserById.ts).
File Changes

fileUploader router
messages router
projectPostReply router
projectPost router
project router
users router
search router
userPosts router

Review

[ ] Tests
[ ] Documentation

Reviewer

[ ] Code Review
[ ] Documentation Review

Bug is being solved.
The bus was only on the local machine.


Is field type mapping possible ?

I might have missed it, but it looks like all fields coming in through the river-csv are strings.
I have used the following minimal PUT request :
{
    "type" : "csv",
    "csv_file" : {
        "folder" : "C:\\localdata\\COUNTER3",
        "first_line_is_header":"true"
    }
}

Can the [fields] parameter be used to specify a field type mapping too?
Or should I specify the mapping on the index with the Elasticsearch mapping api ?

For anyone else with this question:
You can also use dynamic templates to match column names (eg "COUNT") and set the data types (eg LONG) accordingly:
http://www.elastic.co/guide/en/elasticsearch/guide/current/custom-dynamic-mapping.html#dynamic-templates
http://www.elastic.co/guide/en/elasticsearch/reference/current/mapping-root-object-type.html#_dynamic_templates


gaffer.mtd: put nodes into submenus

Currently deals with built-in nodes and nodes in mtoa, htoa and alShaders.
A few things:

This is a bit annoying to review, I guess. Do you want me to add screenshots of the menus?
I have not separated Ai and Al here as that seems to be what you agreed on in that eMail thread? At least that's the last thing I heard about it. I did separate htoa and mtoa, though.
Regarding the crash I mentioned last week: I just removed those nodes here. I don't think anybody uses them.

Thanks.

Thanks Matti - looks good to me. Could we just rename the htoa/mtoa submenus so that they're capitalised? Maybe "Maya" and "Houdini" might be a bit more readable anyway, and save us from trying to determine the right capitalisation (MtoA? MToA?)?

Updated.

Nice one - thanks Matti.


Implement redux folder structure on the project

The project is using simple folder structure, but since we will have to deals with lot of components it is becoming more and more difficult to maintain without redux. Implement redux folder structure on the frontend side of the project.

Just make sure you mean this folder structure by that, not that old actions/reducers/constants folder thingy :)

I just gave you a link to the official Redux Style Guide which contains an example folder structure?

This task completed my me so i am closing this issue


Implement additional modules that don't yet exist

[ ] Node API
[ ] P2P API
[x] Fraud API

[ ] https://github.com/celestiaorg/celestia-node/pull/1285
[ ] https://github.com/celestiaorg/celestia-node/pull/1313
[ ] fraud is done already.


Remove .gitmodules and use CPM to fetch dependencies instead

We are already using CPM for fetching JUCE in the examples. Much nicer to use them for everything instead of .gitmodules that just add needless complexity, confusion and extra step when building 😊

So the reason we didn’t do this is because quite a few of our target environments have network free build stages. If we are going to do cpm we need a bailout which lets you specific a pre downloaded location (which is the strategy we tok with the clap wrapper)
the examples are built in ci and by developers so the constraints are more relaxed, but having those have a juce roof would also be lovely.
But i don’t think we can merge this change as is. It will break some of our client users.

Also it seems the projucer workflows failed to work with the change.
The strategies folks seems to use are

copy the deps. This is how juce deals with vst3 and AU
Submodule so the network access is at checkout time
Cpm with configurable bailout. So do if target checks and look for things like clap root and obey that if set - check out the guarantee functions in the clap wrapper for this

all of those allow a build of the core lib in an un networked env. Moving from 2 to 3 will break some of our users though since they will have to reconfigure their builds to do the checkouts at source assembly time.

Oh another alternative I just thought of. You could use File in cmake to check if the submodule stage has been run. If it has not you could print a warning and run Cpm.  That would retain the submodules for cure t workflow but also self heal with Cpm if you don’t do the proper check out. That’s the least disruptive strategy.
Need to make sure we sync the Cpm and git module versions then so a small workflow cost.
alternately we could have the same check do a clear error message with a fatal error which is lowest complexity

Thank you for getting back to me @baconpaul. Love the project!
You can tell CPM to rely solely on local dependencies, no problem. It will prioritise local packages and revert to download if it doesn't find them
    "-DCPM_USE_LOCAL_PACKAGES=ON"
    "-DCPM_LOCAL_PACKAGES_ONLY=ON

Thank you for getting back to me @baconpaul. Love the project!
You can tell CPM to rely solely on local dependencies, no problem. It will prioritise local packages and revert to download if it doesn't find them
    "-DCPM_USE_LOCAL_PACKAGES=ON"
    "-DCPM_LOCAL_PACKAGES_ONLY=ON

Yeah but then it uses findoackage right? Which isn’t set up with clap helpers and clap
the more I think about it the idea of if the submodule are empty then try cpm seems most appropriate
we can even set up a ci test of that to either not clone or to remove the submodule before running
(If this was an end product I would totally be fine with cpm and we use it for all our test cases but libraries and included things I’ve generally been a bit more careful)

It is also possible to use CPM's "local package override" to try CPM to use a locally downloaded copy of the dependency repo rather downloading it. Not sure exactly where this fits into the conversation, but it felt worth mentioning.
https://github.com/cpm-cmake/CPM.cmake#local-package-override


Menu icon invisible when dark menu theme applied

OS X's dark menu theme hides the menu icon. It is visible when the menu bar is not active, adding a white/lighter icon would fix this issue.

In this screenshot, the icon is to the left of the green bar.

This has been fixed in 0.9.1.


Disable the autouse flag on the postgres_db fixture (#3111)

Pull request opened by the merge tool on behalf of #3111

Merged into branches iso4 in 88b38385af2959ba54bdd573f4242eecb39f75d0


Survey Results

Upload of our survey results
Thank you to all who helped and supported this survey, especially the KCDs who were so kind to share this at their conferences.

Looks like the website is not building 😬
@mkorbi could we put a link to the pdf on the website ->  https://github.com/cncf/tag-env-sustainability/blob/main/website/content/about/_index.md or index one level above.

/retest

Looks like the website is not building 😬

@mkorbi you need to sync your branch with upstream main to update the website.

I will update the website in a followup PR..


Update about page

What this does
Adds a "History of the Caselaw Access Project" section to the about page to provide historical context for the project and additional documentary resources for folks to learn from.
Closes ENG-757.
Screenshot

Thanks (again) Becky!


Fix issue with using UUID primary keys in complex return types

When using a UUID as a primary key with PostgreSQL the grouping was
defaulting to the row which caused incorrect results to be returned.

Thanks for the pull request and catching the problem. Excellent work.


Enable gitea oci registry

This is now working with the following patch for gitea: https://github.com/jessesanford/gitea/blob/c2fffabcd5122d258360d4443b6e583fc62e28f1/routers/api/packages/container/container.go#L119-L131
We need to decide if we want to use this forked version temporarily while we shape a PR for upstream or continue to use an additional registry. @nimakaviani @nabuskey @greghaynes wdyt?

We had some discussions around enabling gitea OCI registry.
I think we should enable it by default if users decides to use. My reasoning is that we already install Gitea by default so we can get this feature for "free". I don't see any harm in enabling this feature by default so long as we stay local.
BUT we need to be careful and not rely on this. End users should be able to use whatever registries they want to use. We should not restrict them to using this registry.

@jessesanford can you plz drop the link to the change you had to make to gitea for its image registry to work? I am curious if we can find a better way to get around the problem than to have to use a custom image.

@nabuskey I agree with everything you said. It just so happens this works very well with the ref-implementation workflows I am developing.
@nimakaviani the unofficial PR for my changes is here... it's pretty simple: https://github.com/jessesanford/gitea/pull/1/files
You can also see the discussion of the issue here on the official repo: https://github.com/go-gitea/gitea/issues/29591
I am bullish on this becoming an actual upstream patch as the project has accepted similar contributions for the git repo functionality in the past. See: https://github.com/go-gitea/gitea/issues/19345

How about adding Harbor as a core application and use as OCI/Image registry? Anyway every platform needs some kind of registry for application images


Infinite loop setting the key

When adding the openAI token, the code show the same error message.

My machine is a macbook pro m2 , running on Ventura 13.12.1

I solved like this;

Create ~/.opencommit
Add your key like .env format to into the that file

OPENAI_API_KEY=sk-##################
emoji=true

save file after then try again set config command.

It solved, thanks!


Ensure that ways do not have more than 2000 nodes

A Geojson file could be provided to this script with a large way with over 2000 nodes. This should be broken up into smaller ways due to the OSM limit.

An open way can be split into two ways with the tags duplicated. A closed way would have to be converted into a multipolygon. A way as part of a relation would have to have the new way added to the relation


Complete the implementation of class fields, private methods and decorators.

Reference documents:

https://tc39.github.io/proposal-class-fields
https://tc39.github.io/proposal-private-methods
https://2ality.com/2019/07/private-methods-accessors-in-classes.html
https://tc39.es/proposal-decorators/#sec-syntax

Class fields implementation

a migration transforms the JSIdentifier of JSMemberExpression in JSIdentifierName because reserved words are permitted as internal fields name like myObject.function.
The migration has also an associated test.

add JSPrivateIdentifier that extends JSIdentifierName:
PrivateIdentifier::
  # IdentifierName

It extends JSIdentifierName and has a different editor.

add ClassElementName:
ClassElementName[Yield, Await]:
  PropertyName[?Yield, ?Await]
  PrivateIdentifier

It is an interface extended by JSIPropertyName and the new interface JSIPrivateIdentifier.

add FieldDefinition:
FieldDefinition[Yield, Await]:
  ClassElementName[?Yield, ?Await]   Initializer[In, ~Yield, ~Await]opt

It's a concept, it implements the three interfaces to pass the parameters to the initializer.

add an alternative to ClassElement:
ClassElement[Yield, Await]:
  FieldDefinition[?Yield, ?Await];

JSFieldDefinition implements the interface JSIClassElement.

add an alternative to MemberExpression:
MemberExpression[Yield, Await]:
  MemberExpression[?Yield, ?Await].PrivateIdentifier

MemberExpression alternatives have MemberExpression to the left, actually we have object: JSPrimaryExpression[1] so this os going to be changed in object: JSAbstractMemberExpression[1]. No migration required because we substitute with a subconcept.
on the right actually we can have JSIdentifier and it's not correct: the grammar has IdentifierName that allows also reserved words, infact myVar.for is permitted. A migration is required. We also need an interface grouping JSIIdentifierName and JSPrivateIdentifierReference, private identifier must be declared before used.

private fields scope is implemented with:

JSClassDeclaration now implements ScopeProvider.
JSIClassElement extends JSBindingProvider so JSFieldDefinition can override the implementation.

the following alternative to CallExpression is not needed:
CallExpression[Yield, Await]:
  CallExpression[?Yield, ?Await].PrivateIdentifier

tests:

for private fields scope.
for construction of this.#privateField.field.
for construction of this.field.field.

examples for private fields and member expression of a member expression (this.a.a).

Private methods implementation
This implementation provides:
class MyClass {
  #privateOrdinaryMethod() {}
  // async, generator private methods will be implemented contextually to async, generator etc
  // * #privateGeneratorMethod() {}
  // set/get #privateGetSet() {}
}

The grammar is:
MethodDefinition[Yield, Await]:
  PropertyName[?Yield, ?Await](UniqueFormalParameters[~Yield, ~Await]){FunctionBody[~Yield, ~Await]}   <-- from
  ClassElementName[?Yield, ?Await](UniqueFormalParameters[~Yield, ~Await]){FunctionBody[~Yield, ~Await]}   <-- to

change methodName : JSIPropertyName[1] to methodName : JSIClassElementName[1] in JSMethodDefinition, no migration required.

let JSMethodDefinition overrides JSBindingProvider to provides scope for its private identifier if present.

Decorators implementation
This was already partially implemented. What we are missing is:

for the alternative ( Expression[+In] ) of DecoratorMamberExpression I've added HasIn to JSParenExpressionDecorator.

JSClassElement should extend JSIDecoratorList so methods and fields can be decorated (JSClassDeclaration is already this way).

JSClassExpression should extend JSIDecoratorList, the concept is present but not yet used.

I'm getting the following error on the current development branch when compiling all-examples:
// More on decorators, part of issue #7.
var deco1;

@deco1
@<!TextGen not found for 'org.mar9000.mps.ecmascript.structure.JSParenExpressionDecorator'!>
@<!TextGen not found for 'org.mar9000.mps.ecmascript.structure.JSDotMemberExpressioDecorator'!>
@<!TextGen not found for 'org.mar9000.mps.ecmascript.structure.JSDotMemberExpressioDecorator'!>
@<!TextGen not found for 'org.mar9000.mps.ecmascript.structure.JSDotMemberExpressioDecorator'!>
@<!TextGen not found for 'org.mar9000.mps.ecmascript.structure.JSDotMemberExpressioDecorator'!>
@deco1(1)
@<!TextGen not found for 'org.mar9000.mps.ecmascript.structure.JSDotMemberExpressioDecorator'!>(3)
class ForDecorators {
  field1 = 4
  ;
  method1() {
  }
  constructor__() {
  }
}

Fixed adding two missing TextGen concepts.


[FEA] Add options to reserve categorical indices in the Categorify() op

Is your feature request related to a problem? Please describe.
Some pipelines trust that categorical features will have some reserved values after preprocessing.
For example, for sequential features (e.g. session-based recommendation, time series, text data) it is important to reserve a value for padding (e.g. 0) in the model side, as not all sequences might have the same length.
It is also important to reserve a value (e.g. 1) to map Out-Of-Vocabulary (OOV) categorical values that might appear in the transform() (e.g. during evaluation or inference)
Describe the solution you'd like
Create one argument for Categorify() op to set the desired oov_index (for which all OOV values will be mapped during the preproc) and another argument start_index to set the first value that should be used to encode known values (e.g. 2). We can also have a similar option for null values (null_index), so that they are mapped to a known index.
In this case, if we call Categorify(oov_index=1, null_index=2, start_index=3), the index 0 could be safely used for padding for example.
I think we don't need an padding_index, because it is specific to sequential features. It is better to give users flexibility to reserve a number of categorical values by using start_index

@gabrielspmoreira Do we really need that level of complexity?  Can't we just specify the offset?  Or does categorify work differently if you specify oov, null, and start_index ranges.
Also do we want to handle the case where people specify ridiculous values?  What if I say oov = 100000000?
I think a simple reserved offset is more straightforward and less error prone.

@EvenOldridge I agree we can simplify the solution.
With the offset (start_index) we solve the issue of reserving 0 for padding.
And we could reserve the first two indices after start_index / offset for null and OOV during transform() (inference).
What do you think @benfred ?

I think we should simplify, and just specify a start_index with a default value of 0.
The implementation of Categorify has gotten pretty complex - so I thought it might help to put some pointers on where to get started with this:

There are two phases in Categorify: the 'fit' which calculates the mapping of input label to categorical id, and the 'transform' which applies the mapping. We'll need to change both of these to handle this option
For the 'fit' stage, we will need to store this option in the 'FitOptions' struct and then use in the _write_uniques function https://github.com/NVIDIA/NVTabular/blob/57855b872444fee1c249ad6f6f190a73aa8a81f3/nvtabular/ops/categorify.py#L889-L904 .
The 'transform' code needs updated in the _encode function to set the unknown values to the start_index specified by the user.  The simple case where we aren't doing any OOV hashing will involve updating these lines https://github.com/NVIDIA/NVTabular/blob/57855b872444fee1c249ad6f6f190a73aa8a81f3/nvtabular/ops/categorify.py#L1110-L1116  , but this will get more complicated with the hash bucket code. It might be worth just trying to get this working without hashing first, and then tackle hashing next.
There is currently a 'na_sentinel' parameter on the categorify op https://github.com/NVIDIA/NVTabular/blob/57855b872444fee1c249ad6f6f190a73aa8a81f3/nvtabular/ops/categorify.py#L146-L147 . Unfortunately I don't think this works appropriately right now, and probably should be removed as an option from Categorify.

@benfred @rjzamora Would one of you have some time today that I could meet with you to ask about how to implement this additional argument? I read over the current implementation of the class, but I am not fully understanding what the particular methods Ben outlined are supposed to do. I tagged you two since it looks like you have spent the most time working on this module. I would appreciate it if so.

@gabrielspmoreira, someone else, would you have a test case available to test the appropriate desired functionality? I found this test from tests/unit/ops/test_ops.py that tests categorify that looks like a helpful template.
def test_categorify_lists(tmpdir, freq_threshold, cpu, dtype, vocabs):
    df = dispatch._make_df(
        {
            "Authors": [["User_A"], ["User_A", "User_E"], ["User_B", "User_C"], ["User_C"]],
            "Engaging User": ["User_B", "User_B", "User_A", "User_D"],
            "Post": [1, 2, 3, 4],
        }
    )

    cat_names = ["Authors", "Engaging User"]
    label_name = ["Post"]

    cat_features = cat_names >> ops.Categorify(
        out_path=str(tmpdir), freq_threshold=freq_threshold, dtype=dtype, vocabs=vocabs
    )

    workflow = nvt.Workflow(cat_features + label_name)
    df_out = workflow.fit_transform(nvt.Dataset(df, cpu=cpu)).to_ddf().compute()

    # Columns are encoded independently
    if cpu:
        assert df_out["Authors"][0].dtype == np.dtype(dtype) if dtype else np.dtype("int64")
        compare = [list(row) for row in df_out["Authors"].tolist()]
    else:
        assert df_out["Authors"].dtype == cudf.core.dtypes.ListDtype(dtype if dtype else "int64")
        compare = df_out["Authors"].to_arrow().to_pylist()

    if freq_threshold < 2 or vocabs is not None:
        assert compare == [[1], [1, 4], [3, 2], [2]]
    else:
        assert compare == [[1], [1, 0], [0, 2], [2]]

I would like to implement and test a feature add for this issue, but I am not exactly sure how to make a test to test Categorify with an extra start_index argument.

@gabrielspmoreira I did not see how to add you as a reviewer for my draft PR, but when you have a chance, would you be able to see whether the implemented tests test your intended functionality? This draft PR consists of the commit  642e5f503e33c8440d6a186c7a9a8c29cc33618b.


MAINT: Update ruff version to ~0.5.5

Changes:

type comparisons now require is: str is str
remove overridden class attribute active_adapter
remove secondary import of fbd_cuda

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.

@yfeng95 it would be great if you could confirm that the fbd_cuda import is redundant and can be safely removed.


Enable linters which do not report anything

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
Those do not introduce any file changes so they can be enabled by default.
Which issue(s) this PR fixes:

None
Special notes for your reviewer:
None
Does this PR introduce a user-facing change?

None

/apply
/lgtm

/approve
/lgtm


[Bug] Missing translation for reset password email

Bug report
What I did
I tried to reset a password from an italian localized installation
I already have config/app.php setup for
'locale' => 'it',
'fallback_locale' => 'en',

And in config/backpack/crud.php I have
'locales' => [
 ...
'en' => 'English',
...
'it' => 'Italian',
...
]

What I expected to happen
I expected to see the reset password email fully localized into italian
What happened
A part of email is not translated

This is the default email.blade.php file for notifications
vendor/laravel/framework/src/Illuminate/Notifications/resources/views/email.blade.php

What I've already tried to fix it
Actually we exported the email.blade.php and forced italian text
Is it a bug in the latest version of Backpack?
After I run composer update backpack/crud the bug... is it still there?
Yes
Backpack, Laravel, PHP, DB version
When I run php artisan backpack:version the output is:
λ php artisan backpack:version
### PHP VERSION:
PHP 8.2.8 (cli) (built: Jul  4 2023 15:53:15) (ZTS Visual C++ 2019 x64)
Copyright (c) The PHP Group
Zend Engine v4.2.8, Copyright (c) Zend Technologies

### LARAVEL VERSION:
<IP_ADDRESS>

### BACKPACK PACKAGE VERSIONS:
backpack/basset: 1.2.1
backpack/crud: 6.3.0
backpack/generators: v4.0.2
backpack/permissionmanager: 7.1.1
backpack/pro: 2.0.18
backpack/theme-tabler: 1.1.1

Hey @realtebo as you can see, that file is part of core Laravel. I am not 100% sure if https://github.com/Laravel-Lang/lang already provide a translation for it.
If not, you can manually add a translation for that string by creating a file in lang/it.json and add that string:
 "If you’re having trouble clicking the \":actionText\" button, copy and paste the URL below\ninto your web browser:": "Se tiver problemas em clicar no botão \":actionText\" , copie e cole o seguinte URL no seu browser:",

More info on https://laravel.com/docs/10.x/localization#defining-translation-strings
Basically, as you can see on the file that you mentioned it runs through @lang directive so you can translate it using Laravel methods.
I am closing this as I don't think there is any bug on our side here.
I may be wrong, so feel free to re-open, or continue the conversation and we can re-evalute.
Cheers 🙏

I confirm, and understand all the line you follow.
But I am pretty sure it should be included in the lang file of backpack, if possibile, so when we start a project, localized, it is complete.

Also here

Having some pieces localized and some not is not ... optimal.
I suggest, at least, to add a clear documentation alert about how to complete localization.

I've edited the docs a little bit more to make it clear that some translations are not provided by backpack here: https://github.com/Laravel-Backpack/docs/pull/523
We think the https://github.com/Laravel-Lang/lang package is an easy way to have all those strings translated in most of the languages. If something is missing you can submit a PR there, I am sure they will appreciate it 🙏
Let me know if I can help you with something else.


OrderedMerge is named too similar to SortedMerge

The names provide no indication as to how they differ, and the key difference is not prominent in the documentation (AFAICT, OrderedMerge is equivalent to SortedMerge().Distinct(), or am I missing something?).
I'm not sure which one should be renamed, though. Maybe these should be overloads of the same method, with a parameter indicating whether to discard repeated values or not.

I agree, the naming is certainly confusing but they do different things. OrderedMerge is like an efficient full outer join so the sequences are joined at keys. The more specific overloads let you decide on the result when either sequence yields an element or both:
Enumerable.Repeat(5, 10)
          .OrderedMerge(Enumerable.Range(5, 10),
                        n => n,                 // identity key
                        a => new { a, b = -1 }, // left only
                        b => new { a = -1, b }, // right only
                        (a, b) => new { a, b }) // both

The result will be:

a
b

1
-1

2
-1

3
-1

4
-1

5
5

6
6

7
7

8
8

9
9

10
10

-1
11

-1
12

-1
13

-1
14

The ordered in OrderedMerge is there to say clearly that it expects ordered sequences otherwise the result is undefined.
In contrast, SortedMerge will line up in order:
string.Join(", ", Enumerable.Range(1, 10)
                            .SortedMerge(OrderByDirection.Ascending,
                                         Enumerable.Range(5, 10)))
// output: 1, 2, 3, 4, 5, 5, 6, 6, 7, 7, 8, 8, 9, 9, 10, 10, 11, 12, 13, 14

OrderedMerge is already there in 1.x so it should stay. However, if you can come up with a way, way better name then we can talk about marking it obsolete in 2.0 while also exposing it under its new name. The older name could then be retired in 3.0. That's assuming if SortedMerge is the perfect name. Another option is to drop SortedMerge for now and introduce it back later when the naming can be clarified. What I want to avoid is a 2.0 release just hanging on this issue.

@atifaziz Interesing. v2 alpha does not have it.
Perhaps SortedInterleave is a better name for SortedMerge? Also, OrderedMerge could be OrderedJoin.
If you find deprecation and removal in an eventual 3.0 acceptable, then there is no need to hold the 2.0 release on this. The Obsolete attribute can always be added later on.

@fsateler v2 alpha doesn't have it because it was added to v1 and released in one of the minor version updates after v2 work had started. It should make it in the next package now that it's been merged in from v1.

Perhaps  SortedInterleave is a better name for SortedMerge? Also, OrderedMerge could be OrderedJoin.

Who ever said naming is hard? :smirk: And of course, SortMergeJoin would just make a terrible name to mean everything and yet nothing. :laughing: If we can borrow a name from a similar operation in another library (on another platform/runtime/language) then it can go a long way to help not tear out hairs on this. Not only many others must have gone through the same motions but chances are good that users will recognize them easily. For example, Zip can be found pretty much everywhere you look from LINQ, Python, F#, Haskell and countless others.

If you find deprecation and removal in an eventual 3.0 acceptable, then there is no need to hold the 2.0 release on this. The Obsolete attribute can always be added later on.

True.


upgrade: make pinned error have correct exit code.

Also, display before saying "no packages to upgrade".
Fixes #3936.

@ilovezfs I see this is meant to fix my submitted issue #3936. However, I dated Homebrew today and still experience this. I see your commit in the git log, though, so I should have the latest code.
$ brew upgrade
Error: Not upgrading 1 pinned package:
unison 2.51.2
==> No packages to upgrade

@echosa This makes the error message match the exit code. However, there's now --ignore-pinned you can use to bypass that message/error.

Oh, there's a new flag. Got it. Ideally, by my original suggestion, I'd still like to be notified that there are pinned formulae not getting update (i.e. not ignore them with a flag) but have that notification not be an error (because, in my mind, it isn't… it's just a notice like "FYI, don't forget, these formulae aren't updating"). When I see "Error" in red, I think something's wrong, but pinned formulae aren't wrong.  ;-)
I guess it's just strange to me that it's considered an error at all. That was the initial surprise that spurred my suggestion. That flag is still useful in its own right, though, for folks who don't care about their pinned formulae and just want to "pin it and forget it".

The flagged option does result in the correct behavior:
$ brew upgrade
Error: Not upgrading 1 pinned package:
imagemagick@6 6.9.9-39
==> No packages to upgrade

$ echo $?
1

$ brew upgrade --ignore-pinned
==> No packages to upgrade

$ echo $?
0

@MikeMcQuaid, shouldn't --ignore-pinned be the default though? If I actively pin a package, the expected behavior is that I wouldn't get an error on brew upgrade, while at the same time the pinned package shouldn't get updated. Seems like this generally doesn't work well though, as there are too many cases where a package is updated anyway (like a dependency and not a leave package). This makes it not well suited to development, where specific versions are required.
I imagine full dependency resolution would be required to satisfy those requirements though, like yum/apt, and not the primary goal for the homebrew project.

Seems like this generally doesn't work well though, as there are too many cases where a package is updated anyway (like a dependency and not a leave package).

This should not happen any longer. If it does to you: please file a bug with a reproducible test case and I will fix this.

I imagine full dependency resolution would be required to satisfy those requirements though, like yum/apt, and not the primary goal for the homebrew project.

Indeed, this is why it's not the default behaviour. Pinning packages and upgrading others will often result in broken packages, unfortunately.

shouldn't --ignore-pinned be the default though?

My two cents (again). I think the default behavior should be a mix of the two. Default behavior is fine to show a reminder of pinned formulae, but shouln't error out, meaning both a.) it should exit with code 0, and b.) it shouldn't show the word "Error" in the output. Doesn't that make the most sense?
In other words, this would the ideal behavior to me:
$ brew upgrade
Not upgrading 1 pinned package:
imagemagick@6 6.9.9-39
==> No packages to upgrade

$ echo $?
0

@echosa have a look at this thread https://github.com/Homebrew/brew/pull/3806, they won't change the message even if it's super clear that is the right thing to do ¯\_(ツ)_/¯

even if it's super clear that is the right thing to do ¯_(ツ)_/¯

@sferrini It's not super clear, though, because myself and other maintainers disagree. Please read our Code of Conduct and adjust your future communications accordingly.

Having read over #3806, I understand a little better why this hasn't changed. I don't necessarily agree with the reasons, but I understand the current situation a little better. I still think that this shouldn't display "Error" (in red, no less) in the brew upgrade output when there's a pinned package.
It's an issue of semantics. I agree with @sferrini in that I don't see pinned formula as an error, even if it breaks programs, as in the example given in #3806. pin is a valid command, which means pinned formula are valid, which means means not upgrading a pinned formula is valid and expected, not an error. However, reminding users of pinned packages not getting upgraded with a "warning" is desired, as well.
All that said, I'm not a maintainer of this project, and this is literally just a mild irritation as setting the word "Error" when there is no error IMO. The commands all still work great, so this isn't exactly a huge deal. Just a personal quality of life request from a user. However, this proposition has been looked at, considered (I assume and hope), and rejected, so I'll respect that. I simply made the suggestion and tried to support my stance on it. I'm not one the who has to take the time to make the change, nor support that code change going into the future. Am I sad that I'll continue seeing "Error" in the output? Sure. Does it affect my ability to keep working and using homebrew? Not at all.
Thanks to the homebrew team for an awesome program, about which the biggest complaint I have is a semantic wording choice. Keep doing what you're doing. It makes life so much better for so many people. <3

Thanks for your kind words @echosa!


Quasiquotes extractor

Hi,
The quasiquotes assignment like this is not highlighted correctly
val q"val $name = $value" = someTree

Are there any plans to add syntax highlighting to the scala code inside the quasiquotes?


MIT license violation

Large parts of this project is violating the MIT license of EventFlow. Please make sure to rectify this and make sure to add links back to the original project which under copyright from myself and eBay Software Foundation.
The original project of which most of the code has been copied is located here.
https://github.com/eventflow/EventFlow

I recommend creating a NuGet dependency on EventFlow instead of having duplicated code and build on the changes, bug fixes and features added to EventFlow since the code was copied.

Recommendation seems unfeasible, as we had a conversation about it before this issue. I will attribute the sums of files and then cc you in on the PR so you can signal your satisfaction or otherwise. :)

please view https://raw.githubusercontent.com/Lutando/Akkatecture/dev/src/Akkatecture/Aggregates/AggregateEvent.cs would this suffice on all affected files?

work has been done just need you to comment on https://github.com/Lutando/Akkatecture/commit/96ac63eca1e15bc27315978a4ee4565779378ce5 and i will apply them to the rest. thanks

@Lutando I talked to our eBay open source legal counsel and everything seems to be in order now.


values not updated on first click on editor (non-compact mode)

Hi
It seems like when a non-compact mode editor just got rendered, if the first click on the component is on the editor's text block (or even other blocks?) and starts modifying the text, the values are actually not updated (can check json)
I'm not sure whether this is an expected behavior or not, just asking

version: 4.4.2
also behaves like this in the examples' non-compact mode (small screen)

thanks!

Please update to the latest version 4.4.3


Animated routes list

Routes list should animate when we search something
Current animation:
https://user-images.githubusercontent.com/16733681/164987576-679bd5b8-2845-4984-9b35-7a169178d0eb.mp4
As an alternative, staggered animation works with a single prop change, but I thought the animation was too distracting if you have a long list
https://user-images.githubusercontent.com/16733681/164987630-1a658d6d-ac61-4e79-8ebc-dc392143e001.mp4

Closing in favor of https://github.com/caribou-crew/mezzo/pull/57


Broken with streamlit 1.12

The latest streamlit moved lib/streamlit/server to a different location in commit https://github.com/streamlit/streamlit/commit/ddfe5c02234d6e8eb5deb26c42605979f623e810
As a result this package is now broken.

Thank you for the report!

Dev note:
With this upgrade, the singleton instance of the server has been removed (Server.get_current() is no longer available).
We have to find an alternative that can be used to store a server-wide object.

Thank you @whitphx for the library and for fixing this so quickly!


Helm3 error

Hello.
I'm trying to use helm3 and cloudwatch-adapter and faced this an issue. When I install adapter helm stop working with such message:
Error: could not get apiVersions from Kubernetes: unable to retrieve the complete list of server APIs: custom.metrics.k8s.io/v1beta1: the server could not find the requested resource

On helm2 everything is OK.
Thanks for helping

Have same issue with Helm 3

which version of the adapter are you using? the call to register with custom.metrics.k8s.io should have been removed in v0.7.0

chankh/k8s-cloudwatch-adapter:v0.7.0

Looks like a problem is with Helm https://github.com/helm/helm/issues/6361

As API point custom.metrics not used. You can delete it and helm starts working after that.
kubectl delete apiservice v1beta1.custom.metrics.k8s.io


Add remember check_box into view, config for rememberable and add cov…

…erage directory into gitignore
Summary

Add checkbox for user choose if remember or not through cookies session persisted, otherwise, session expires when user closes the web browser

Coverage remained the same at 0.0% when pulling f6da568ad7d2877f4ce16ca7fe36361bfd84b3aa on juuh42dias:session-login-#254 into beafe974fcba88d1860e0876ea6b4e7ea9ed51b1 on gorails:master.


[META-DOC] Create Internationalization documentation

Status: content planning
TOC (working draft)

Make languages available
Provide translations on the Internationalization tab for each object you want to internationalize.
Specify a language for the user

@vagimeli - you can close this issue until we have it in our backlog. I know they are adding language support in an upcoming release, but that is tracked under a different issue. Thanks.

Closing meta issue; see specific issues as they are created


problems in using APIrefineHypothesis()

I want to use the APIs to learn Hypothesis like this:
learn()
{
   boolean flag = true;
   learningAlgorithm.startLearning();
   hypothesis = learningAlgorithm.getHypothesisModel();
   while(flag)
        {
           counterExample = equivalenceAlgorithm.findCounterExample();
           if(counterExample == null) {
                flag = false;
                }
           else{
                result= learningAlgorithm.refineHypothesis(counterExample);
                hypothesis = learningAlgorithm.getHypothesisModel();
         }

  //......get the final  HypothesisModel.............
}

but  when i test the program, it  always hung up in refineHypothesis(); will get someHypothesis but no final result. I can't find out what's wrong with it.

Dear @IMHappyZzz,
I can see one potential problem with your code: You store the result of learningAlgorithm.refineHypothesis(counterExample) in the variable result. If you pass a false counterexample to the learning algorithm, which does not trigger an actual hypothesis refinement, the refineHypothesis method will return false. This information is not propagated to your flag variable, so you can potentially run into an infinite loop.
However, without further information, it is hard to tell what is going wrong. A minimal working example (MWE) which would allow us to reprodruce the error would be the best.
It could also be a problem with your system under learning. If it does not exhibit regular behavior, a regular learning algorithm usually runs into infinite loops. Especially systems, which involve some kind of counting (e.g. a stack datastructure) have these kind of problems. You could check, if the size of the hypothesis increases throught the loop iterations to further pinpoint the issue.

I'm closing this issue due to the missing feedback and us not being able to reproduce the problem. If this is actually a problem in LearnLib, feel free to re-open the ticket.
If you have general questions on using LearnLib, make sure to check out our Q&A list, so that we can use the issue tracker for actual issues.


log output difficult to understand for no testcase error

When there are no testcases in Tosca for one of the TestTypes, the output we get is dificult to understand
Is it posible to have a preflight check for the configuration in Tosca and have a more clear message in case there is something missing ?
Log
time="2021-08-04T08:01:50Z" level=error msg="Unknown Error when executing TestSuite installation, error: Tricentis.TCAPIObjects.Exceptions.TCApiFatalException: Object reference not set to an instance of an object.\r\n\r\nServer stack trace: \r\n   at Tricentis.TCAPIObjects.TCConnectionBase.ExecuteTask(TCObjectBase[] objs, TCTaskBase taskToExecute, TCTaskParams taskParams)\r\n   at System.Runtime.Remoting.Messaging.StackBuilderSink._PrivateProcessMessage(IntPtr md, Object[] args, Object server, Object[]& outArgs)\r\n   at System.Runtime.Remoting.Messaging.StackBuilderSink.SyncProcessMessage(IMessage msg)\r\n\r\nException rethrown at [0]: \r\n   at Tricentis.TCAPIObjects.Objects.TCObject.ExecuteTask(String name, TCTaskParams taskParams)\r\n   at Tricentis.TCAPIObjects.Objects.ExecutionList.WriteAutomationObjects(String AutomationSetPath)\r\n   at CIService.Helper.ExecutionHelper.RunExecution(ExecutionTracking executionTracking, List`1 executionListIDs) in C:\\Temp\\toscaci-main\\server\\CIService\\Helper\\ExecutionHelper.cs:line 80"
time="2021-08-04T08:01:50Z" level=error msg="Unknown Error when executing TestSuite integration, error: Tricentis.TCAPIObjects.Exceptions.TCApiFatalException: Object reference not set to an instance of an object.\r\n\r\nServer stack trace: \r\n   at Tricentis.TCAPIObjects.TCConnectionBase.ExecuteTask(TCObjectBase[] objs, TCTaskBase taskToExecute, TCTaskParams taskParams)\r\n   at System.Runtime.Remoting.Messaging.StackBuilderSink._PrivateProcessMessage(IntPtr md, Object[] args, Object server, Object[]& outArgs)\r\n   at System.Runtime.Remoting.Messaging.StackBuilderSink.SyncProcessMessage(IMessage msg)\r\n\r\nException rethrown at [0]: \r\n   at Tricentis.TCAPIObjects.Objects.TCObject.ExecuteTask(String name, TCTaskParams taskParams)\r\n   at Tricentis.TCAPIObjects.Objects.ExecutionList.WriteAutomationObjects(String AutomationSetPath)\r\n   at CIService.Helper.ExecutionHelper.RunExecution(ExecutionTracking executionTracking, List`1 executionListIDs) in C:\\Temp\\toscaci-main\\server\\CIService\\Helper\\ExecutionHelper.cs:line 80"

@segator

I think we can  just check in case execution lists are empty, or not execution lists found at all show a more user friendly error


MFA is optional by default

As a CMS owner, I don't want the ability to completely remove MFA once it's been enabled, so that I know the site will stay secure and user preferences for MFA are still respected.
AC

Once the module is installed, MFA will be optional be default
It is not possible through the CMS to entirely disable MFA on a site

Notes:

Checkbox is removed, radio button is toggled between 'optional' and 'enforced', default on optional

I've updated the designs for Admin SiteConfig to accommodate this. I've

Removed checkbox "Enable MFA for CMS access" from designs
Updated Radio button label text.

Select CMS Admin turns on 2FA:
https://projects.invisionapp.com/dsm/silver-stripe/silver-stripe/folder/components/5c68f96ffb26bb0018525d1a

There's some UI changes required as part of this issue, but speaking to @newleeland, there's some new ideas for wording. It would be good to try include this in your work to avoid rework later.
https://github.com/silverstripe/silverstripe-mfa/issues/140#issuecomment-501065709

Hey @NightJar looking good. I've got a just text changes to make this clearer.
Text changes

Also, is there a reason why the radio labels are not aligned with radio?


Cross-Origin is not supported

Need to support CORS, because usually the user wants to deploy iotx in a single server, while the other web server can access it across domains.

fixed！


Lodestar on bigboy network (2M indexes)

Devops carried out a test to help clients prepare for very big networks. This issue is an epic to track different workstreams. Lodestar in particular showed the following pain points:
Long epoch transitions
Epoch transitions regularly took 7-8 seconds, and 2 run per epoch. This took too many cycles off the main thread causing cascading issues. In current mainnet epoch transitions take 1 sec, so on a 2.1M it should be ~2.5 sec.

Why is the epoch transition 3x slows than it should?
Can the state transition be optimized more?
Why there were 2 runs per epoch instead of 1?

Head size 10-15 GB
The initial anchor CachedBeaconState takes 2.6GB. Since the network was not finalizing there were probably forks maxing the cache, but I could not assert this.

https://github.com/ChainSafe/lodestar/issues/5857

Long fork-choice runs
Re-computing the head takes 1 second, which is in line with observations on mainnet. However, 1 second is too long and not acceptable, and will cause attesters to vote for the wrong head.

Can the re-compute head be optimized more?

Very few peers
The nodes where visibility overloaded, probably causing long response times to network messages. We have not checked with other operators if the nodes were actively banning Lodestar or not, just a hypothesis.

TODO investigate more

closing as stale

epoch transition has been optimized heavily since then
n-historical-states now on by default
fork choice optimized since then


[sqlsmith_01_asan]:Query exceeded time limit of 180 seconds, be crash

Steps to reproduce the behavior (Required)

Select

use sqlsmith
select
  strright(
    cast(subq_0.c7 as VARCHAR),
    cast(subq_0.c6 as INT)) as c0,
  subq_0.c0 as c1,
  subq_0.c7 as c2,
  case when subq_0.c1 <= conv(
        cast(subq_0.c4 as VARCHAR),
        cast(cast(nullif(cast(null as BIGINT),
          case when (subq_0.c6 >= subq_0.c9)
              or (subq_0.c8 <> subq_0.c7) then cast(null as BIGINT) else cast(null as BIGINT) end
            ) as BIGINT) as BIGINT),
        cast(cast(null as BIGINT) as BIGINT)) then subq_0.c0 else subq_0.c0 end
     as c3,
  subq_0.c4 as c4,
  subq_0.c3 as c5
from
  (select
        ref_0.p_name as c0,
        ref_0.p_category as c1,
        ref_1.cc_tax_percentage as c2,
        ref_1.cc_market_manager as c3,
        ref_0.p_color as c4,
        ref_1.cc_market_manager as c5,
        ref_0.p_size as c6,
        ref_0.p_mfgr as c7,
        ref_1.cc_state as c8,
        unix_timestamp() as c9
      from
        sqlsmith.hive_ssb_100g_orc_part as ref_0
          left join sqlsmith.tpcds_100g_call_center as ref_1
          on (ref_0.p_color = ref_1.cc_call_center_id )
      where ref_0.p_container is not NULL
      limit 116) as subq_0
where ((cast(nullif(subq_0.c5,
        subq_0.c7) as VARCHAR) < subq_0.c0)
    or (case when subq_0.c9 is not NULL then subq_0.c9 else subq_0.c9 end
         <= subq_0.c6))
  or (subq_0.c3 is NULL)
limit 74;

Expected behavior (Required)
Successful execution
Real behavior (Required)
Query exceeded time limit of 180 seconds
Be stack
start time: Tue Jun 21 20:13:08 CST 2022
log4j:WARN No appenders could be found for logger (org.apache.hadoop.fs.FileSystem).
log4j:WARN Please initialize the log4j system properly.
log4j:WARN See http://logging.apache.org/log4j/1.2/faq.html#noconfig for more info.


Consider refactoring transaction verifier tests to be property tests

Motivation

The zebra-consensus transaction verifier has many test vectors. They share some common boiler plate code to find a suitable transaction and to call the verifier. It would be interesting to try to refactor some of them in order to try to reduce the amount of test code and run with a larger set of inputs, by:

collecting common behavior in helper functions
using property tests with customized transaction generation strategies
joining tests that have the same assertions but have different inputs into a single property test, with an enum or a test strategy to select between the inputs
creating helper types to prepare valid input for the transaction verifier

Designs

One thing I started experimenting on was a transparent::MockTransfer type that implements Arbitrary. This type could then help with the creation of a source UTXO, its output in a (mock) transaction, and an input to spend it. The idea was to use this in property tests as a replacement for test vectors that use the mock_transparent_transfer helper function.
Related Work

I initially thought of this while trying to write property tests instead of test vectors for #3060.

Hey team! Please add your planning poker estimate with ZenHub @dconnolly @gustavovalverde @jvff @oxarbitrage @teor2345 @upbqdn @conradoplg


Fixed out of bounds error for FuelType(MessageIndex)

If the number of messages sent per batch exceeds 19 then an [Index was outside the bounds of the array] error is generated. This iterates through fuel types ensuring that if you have 19 messages or more you will have generated each fuel type. Future enhancement could make this so that you can select whether the fuel type is randomly generated or the iteration method.

Thanks!


I have no idea how to pull request since this is my first so i'm hoping that i'm not spamming or anything

You have done a great job!
Your PR looks good to me, I'll merge it tomorrow.
Happy open-source journey!
Also leave a star ⭐ to the repository


Error sending PHI20 tokens causing to go to contract address rather than destination wallet

Phi20 tokens are not sending to the wallet address it seems to be sending to the contract address not sure why this is happening. Was working fine before this is a major problem as when users are sending phi20 tokens they are Being lost because they are going to the contract address and not the wallet address user is sending to.
It's only like that on the newest update i don't know why when send phi20 token it is sending to the contract address of the token rather then the destination wallet. Only happening on newest update of alpha wallet please need this fixed asap @hboon

Yes i can confirm it is on the newest update just updated a new device after a successful phi20 transaction to the newest update and it's causing All phi20 transactions to go to the contract address rather than the destination wallet address is this very important to fix. Please if we can get to this as soon as possible so users don't loose funds. Thanks you

Can see when sending 10 Usdc it went to contract address instead of the address added to the receipt. https://phiscan.com/tx/0x31c0710064803618c09f38e1039bc41ff1da9175832dbe00b794d4fad0afec8a

Thanks for reporting. We fix this in #5812

Duplicate of #5811

Why is app not in App Store says not available in country or region. Can you please check this

@Phinetwork took it down until the hotfix is approved by Apple

@Phinetwork took it down until the hotfix is approved by Apple

Ok thank you for update. How long before you think the hot fix is approved by apple. Appreciate you getting to this so quickly.

@Phinetwork usually a few hours to a day

@Phinetwork usually a few hours to a day

Ok great thank you 🙏


plot_finder_image. Unexpected keyword argument: 'radius'

This function does not work properly. I try to follow the example in the tutorial and get the following error
from astroplan import FixedTarget
from astroplan.plots import plot_finder_image
import matplotlib.pyplot as plt

messier1 = FixedTarget.from_name("M1")
ax, hdu = plot_finder_image(messier1)
plt.tight_layout()
plt.show()

---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-96-7d43b094b5fd> in <module>()
      4
      5 messier1 = FixedTarget.from_name("M1")
----> 6 ax, hdu = plot_finder_image(messier1)
      7 plt.tight_layout()
      8 plt.show()

/home/daniel/Software/anaconda3/lib/python2.7/site-packages/astropy/utils/decorators.py in plot_finder_image(target, survey, fov_radius, log, ax, grid, reticle, style_kwargs, reticle_style_kwargs)
    634             name = func.__name__
    635
--> 636         func = make_function_with_signature(func, name=name, **wrapped_args)
    637         func = functools.update_wrapper(func, wrapped, assigned=assigned,
    638                                         updated=updated)

/home/daniel/Software/anaconda3/lib/python2.7/site-packages/astropy/units/decorators.pyc in wrapper(*func_args, **func_kwargs)
    128             # Call the original function with any equivalencies in force.
    129             with add_enabled_equivalencies(self.equivalencies):
--> 130                 return wrapped_function(*func_args, **func_kwargs)
    131
    132         return wrapper

/home/daniel/Software/anaconda3/lib/python2.7/site-packages/astroplan/plots/finder.pyc in plot_finder_image(target, survey, fov_radius, log, ax, grid, reticle, style_kwargs, reticle_style_kwargs)
     86
     87     hdu = SkyView.get_images(position=position, coordinates=coordinates,
---> 88                              survey=survey, radius=fov_radius, grid=grid)[0][0]
     89     wcs = WCS(hdu.header)
     90

TypeError: get_images() got an unexpected keyword argument 'radius'

Other than that, eveyrthing so far seems to work well. Thank you so much for this package! :+1:

The code runs well on my machine. 'SkyView.get_images()` is a part of astroquery. Is it possible you have an old version of astroquery?

Was exactly the reason. Had version 0.2.4, after upgrading with pip install astroquery -U to version 0.3.2 everything works. Thanks


import com.google.pubsub.v1.ProjectTopicName cannot be resolved

So I am using Gradle to build the project and implicitly get your library. If you have some other dependencies with the same packages names inside, the dependency is not grabbed by Gradle. Otherwise the class is missing even from your sources here.
I see "mvn compile " fails for pubsub ... Not expecting this kind of issues here ...
Please advise

@LiviuArln I think we need more details or reproduction step. If I create a new Gradle project and add
dependencies {
	compile 'com.google.cloud:google-cloud-pubsub:0.47.0-beta'
}

everything works fine. Perhaps you're not pulling dependencies pubsub library neds?

The issue seems to have been resolved here. Please ask for reopen if not.


⚠ Bofrost.at has degraded performance

In 296b3c6, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2045 ms

Resolved: Bofrost.at performance has improved in 1d5db29.


Allow control and trials to use different arguments

Allow for control and trial callbacks to use different method signatures to add more flexibility when refactoring code.

Thanks for the review suggestions @localheinz, I've updated this branch.


Dynamic imports not working with additional path after the dynamic part

I'm trying to create a package system using universal-import but when the dynamic part is not the last one, it fails.
This is the UniversalComponent:
const UniversalComponent = universal(
  props => import(`../../packages/${props.page}/src/pwa/index.js`)
)

The chunks create by Webpack have this structure:

This means the names of the chunks are packages/${props.page}-src-pwa-index.js.
But webpack-flush-chunks is trying to find packages/${props.page}/src/pwa/index.js and the SSR fails.
[FLUSH CHUNKS]: Unable to find packages/connection/src/pwa/index.js in Webpack chunks. Please check usage of Babel plugin.
PATH /connection
DYNAMIC CHUNK NAMES RENDERED [ 'packages/connection/src/pwa/index.js' ]
SCRIPTS SERVED [ 'bootstrap.js', 'main.js' ]
STYLESHEETS SERVED [ 'main.css' ]

It works fine if the dynamic part is the last part, like in your examples:
const UniversalComponent = universal(
  props => import(`../../packages/${props.page}`)
)

or if it's not dynamic:
const UniversalComponent = universal(
  () => import('../../packages/general/src/pwa/index.js')
)

In the last case, the chunks created by Webpack don't contain hyphens and packages/connection/src/pwa/index.js is the proper chunk name:

What do you think? Is it easy to fix?

I likely won't be able to get to it, but perhaps the context-replacement-plugin will let you rewrite things:
https://webpack.js.org/plugins/context-replacement-plugin/
If you do wanna do a PR, it's around here:
https://github.com/faceyspacey/babel-plugin-universal-import/blob/master/index.js#L61
and I'm not 100% sure right now, but I'd hope we could replace slashes with hyphens here: quasis[1].
However, if u just want this done, likely the following will work in userland:
``js
chunkNames.map(name => name.replace('-src-pwa-index.js', '/src/pwa/index.js'))

I'll try those solutions and I'll let you know. Thanks James!

This seems to work great in both cases (./pages/${props.page} and ./pages/${props.page}/other/path):
// At the beginning of the file:
function prepareChunkNamePath(path) {
  return path.replace(/\//g, '-')
}

// ...

// The new createTrimmedChunkName function:
function createTrimmedChunkName(importArgNode) {
  if (importArgNode.quasis) {
    const quasis = importArgNode.quasis.slice(0)
    const baseDir = trimChunkNameBaseDir(quasis[0].value.cooked)
    quasis[0] = Object.assign({}, quasis[0], {
      value: { raw: baseDir, cooked: baseDir }
    })

    // <<<< Added this
    if (quasis[1]) {
      const newPath = prepareChunkNamePath(quasis[1].value.cooked)
      quasis[1] = Object.assign({}, quasis[1], {
        value: { raw: newPath, cooked: newPath }
       })
    }
    // >>>>

    return Object.assign({}, importArgNode, {
      quasis
    })
  }
  const moduleName = trimChunkNameBaseDir(importArgNode.value)
  return t.stringLiteral(moduleName)
}

I've forked the repo and added a new snapshot:
'dynamic import (string template with nested folder)': 'import(`./base/${page}/nested/folder`)'

The output is:
import(\`./base/\${page}/nested/folder\`)

      ↓ ↓ ↓ ↓ ↓ ↓

import _path from \\"path\\";
import _importCss from \\"babel-plugin-universal-import/importCss.js\\";
import _universalImport from \\"babel-plugin-universal-import/universalImport.js\\";
_universalImport({
  id: \\"./base/\${page}/nested/folder\\",
  file: \\"currentFile.js\\",
  load: () => Promise.all([import( /* webpackChunkName: 'base/[request]' */\`./base/\${page}/nested/folder\`), _importCss(\`base/\${page}-nested-folder\`)]).then(proms => proms[0]),
  path: () => _path.join(__dirname, \`./base/\${page}/nested/folder\`),
  resolve: () => require.resolveWeak(\`./base/\${page}/nested/folder\`),
  chunkName: () => \`base/\${page}-nested-folder\`
});

The other snapshots don't change so I guess everything is ok.
The only thing I'm not sure of, is the importCss part:
_importCss(\`base/\${page}-nested-folder\`)

but I've tested it in my app and CSS is working great. Now window.__CSS_CHUNKS__ use the proper chunkNames. For example:
window.__CSS_CHUNKS__ = {
  'packages/saturn-app-theme-worona-src-pwa':
    '/static/packages/saturn-app-theme-worona-src-pwa.css',
  'packages/general-app-extension-worona-src-pwa':
    '/static/packages/general-app-extension-worona-src-pwa.css',
};

So I guess is ok.
I've done a PR: https://github.com/faceyspacey/babel-plugin-universal-import/pull/20
We can keep talking there :)


[Improvement] optimization task definition & fix in the task definition list, if one task have more pre task, the task list can't show all task

optimization task definition & fix in the task definition list, if one task have more pre task, the task list can't show all task
closes: #13105

Codecov Report

Merging #13106 (b6346b4) into dev (fa3e2a5) will increase coverage by 0.08%.
The diff coverage is 87.50%.

@@             Coverage Diff              @@
##                dev   #13106      +/-   ##
============================================
+ Coverage     39.20%   39.29%   +0.08%
- Complexity     4273     4277       +4
============================================
  Files          1069     1069
  Lines         40311    40276      -35
  Branches       4634     4614      -20
============================================
+ Hits          15804    15825      +21
+ Misses        22729    22667      -62
- Partials       1778     1784       +6

Impacted Files
Coverage Δ

...duler/api/controller/TaskDefinitionController.java
2.85% <0.00%> (+0.07%)
:arrow_up:

...er/api/service/impl/TaskDefinitionServiceImpl.java
34.48% <93.33%> (+4.60%)
:arrow_up:

...hinscheduler/plugin/alert/script/ScriptSender.java
70.58% <0.00%> (-1.64%)
:arrow_down:

...cheduler/plugin/alert/dingtalk/DingTalkSender.java
34.13% <0.00%> (-0.78%)
:arrow_down:

...rver/master/runner/task/BlockingTaskProcessor.java
75.86% <0.00%> (-0.55%)
:arrow_down:

...inscheduler/plugin/registry/etcd/EtcdRegistry.java
50.69% <0.00%> (-0.35%)
:arrow_down:

.../dolphinscheduler/plugin/task/datax/DataxTask.java
36.62% <0.00%> (-0.26%)
:arrow_down:

.../dolphinscheduler/plugin/task/spark/SparkTask.java
72.56% <0.00%> (-0.25%)
:arrow_down:

...duler/dao/repository/impl/TaskInstanceDaoImpl.java
3.44% <0.00%> (-0.13%)
:arrow_down:

...dolphinscheduler/service/storage/impl/S3Utils.java
0.00% <0.00%> (ø)

... and 19 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Rerun the fail CI


quick diff is showing wrong source

Testing #174853

Checked out a PR. I used this https://github.com/microsoft/vscode/pull/164085
Opened src/vs/platform/request/browser/requestService.ts
Add a new line of text
Click on the quick diff gutter indicator

🐛 Source is shown as GitHub Pull Request

This is by design. The GitHub pull request source shows diffs between your current file state and the PR base.

In that case, how does Git diff option helps here? I mean to ask, will there be any diffs that git provider has but not the PR extension?

The git provider will always be a subset of the PR provider. This is probably true of other extensions that could provide diffs in the future. For example, maybe Gitlens would let you set a specific commit to comapre against. Then you would always compare the current file state to the specified commit.
The diff is always of this format:

Base is provided by extension
Modified is your current file state.


Proper turn lane editing

Turn lanes are inferred as the leftmost/rightmost only, or come from OSM turn restrictions. Also allow changing this.

Duplicate of #70


Updating document with id failed

Hello,
I started to use crate recently and configured a cluster of 3 nodes 2 days back. However seems like one of the nodes will always be throwing the following error. The error occurs continuously and takes all HDD space in hours.
I have this in my config:

node.name: mycrate1
cluster.name: mycluster
discovery.zen.ping.multicast.enabled: false
discovery.zen.ping.unicast.hosts:
  - mycrate1.com:4300
  - mycrate2.com:4300
  - mycrate3.com:4300
discovery.zen.minimum_master_nodes: 2

# I tried to add the following to see if that fixes the issue. Before it was complaining about 50 queue.
threadpool.bulk.queue_size: 100

[2015-04-13 00:35:37,694][ERROR][io.crate.operation.projectors.UpdateProjector] Updating document with id JxLvSdgUSDS1PCKIqMqUiA failed {}
org.elasticsearch.common.util.concurrent.EsRejectedExecutionException: rejected execution (queue capacity 100) on org.elasticsearch.action.support.replication.TransportShardReplicationOperationAction$AsyncShardOperationAction$1@7598c84b
	at org.elasticsearch.common.util.concurrent.EsAbortPolicy.rejectedExecution(EsAbortPolicy.java:62)
	at java.util.concurrent.ThreadPoolExecutor.reject(ThreadPoolExecutor.java:821)
	at java.util.concurrent.ThreadPoolExecutor.execute(ThreadPoolExecutor.java:1372)
	at org.elasticsearch.action.support.replication.TransportShardReplicationOperationAction$AsyncShardOperationAction.doStart(TransportShardReplicationOperationAction.java:415)
	at org.elasticsearch.action.support.replication.TransportShardReplicationOperationAction$AsyncShardOperationAction.start(TransportShardReplicationOperationAction.java:342)
	at org.elasticsearch.action.support.replication.TransportShardReplicationOperationAction.doExecute(TransportShardReplicationOperationAction.java:97)
	at org.elasticsearch.action.support.replication.TransportShardReplicationOperationAction.doExecute(TransportShardReplicationOperationAction.java:59)
	at org.elasticsearch.action.support.TransportAction.execute(TransportAction.java:75)
	at io.crate.operation.projectors.UpdateProjector.setNextRow(UpdateProjector.java:101)
	at io.crate.operation.collect.LuceneDocCollector.collect(LuceneDocCollector.java:180)
	at org.apache.lucene.search.Weight$DefaultBulkScorer.scoreAll(Weight.java:193)

Please help! Thanks in advance!

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/11023508-updating-document-with-id-failed?utm_campaign=plugin&utm_content=tracker%2F505659&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F505659&utm_medium=issues&utm_source=github).

Thanks for reporting. I am investigating this.
What query/queries did you execute to run into this issue?

We use crate for storing users' activities on our site which generates thousands of events per min. the system also does simple SELECT queries to see if the user's id is already in our system. Crate is actually connected with our NodeJS server.

Do you use bulk-inserts? Do you make use of the ON DUPLICATE KEY UPDATE ... clause?

As number of user events can be quite high, the NodeJS app mostly inserts records. However when a user's ID is generated, system updates all old events to have the user id. That's the only update that we run.
No, we do not use ON DUPLICATE KEY UPDATE clause.

Thank you very much!
I'll keep you posted on this.
Until i come up with a fix, does it help to increase the threadpool.bulk.queue_size further as a temporary workaround?

threadpool.bulk.queue_size helped buy couple of hours. threadpool.bulk.queue_size was 50 by default then I changed to 100, 300, 3000, 30000 however the error did show up after a while and the error log started chew up all disk spaces.

As the error says Updating document with id JxLvSdgUSDS1PCKIqMqUiA failed  I removed the "UPDATE" statement in my script and the error stopped showing up!
I run a simple update like: UPDATE table_name set user_id = 'some-user-id' where track-id = 'some-track-id'

The update query is clearly causing this.
The poolsize of the bulk threadpool that is currently used defaults to the number of processors available. If you have a that high concurrency, maybe it is worth to increase the threadpool.bulk.size too.
BTW: we will switch to the index threadpool for the update operation with the next minor release and implement some (internal and fully transparent) retry logic for the update statement execution.

We've spotted the issue and it will be fixed with the next hotfix releases for 0.47 and 0.48.
Thank you very much for reporting! :)

Really feeling great to hear that!


feat: add scripts for running frontend

Description

This change adds some convenient scripts for running development tasks
Testing

I ran the scripts locally
Related Issues

Closes #7

Can you update README PR with instructions on how to boot up the project via Nix


Command pallet blurry

I just discovered the command pallet and when I opened it, it was quite blurry. Here is a screenshot:

(if you zoom in you will see the difference in the icons, command pallet icons vs background icons)
Compared to the rest of the app which is crystal clear it is a bit uncomfortable to my eyes me problem?

I think this is an inherent issue with css transforms. There always seems to be some browser / device / os combination where it loses sharpness. For me it looks fine.
There's a shadcn issue for this where people say removing a class related to border-radius fixes it. I don't understand why that would have anything to do with it, but I'll try it out.
https://github.com/shadcn-ui/ui/issues/711

Alright feel free to tell me when you need beta testing

If you still have your local instance, can you try removing the rounded-md class on this line? That's what people suggest in the thread I linked above.
https://github.com/henrygd/beszel/blob/68824935e95fe330ad76ef150534a894891942ac/beszel/site/src/components/ui/command.tsx#L16

If you still have your local instance

Of course I have it, it's monitoring my entire homelab lol.
Yes man it solved it (and luckily it didn't lose the rounded corners)! You can safely remove it from your codebase and it should be good.

Great, thanks.


How to generate the public key from a private key in eos-Sharp?

Hi,
I couldn't find much information on how to generate the associated public key if you have a private key with you in eos-sharp. I could only see public/private key pairs being generated only when we make use of generateKeyPair. Is there a functionality to generate public key if you have only the private key with you in eos-sharp?

You can check it here: https://github.com/GetScatter/eos-sharp/blob/master/EosSharp/EosSharp.Core/Providers/DefaultSignProvider.cs#L27
Use Secp256K1Manager.GetPublicKey(privKeyBytes, true)


Allow custom ordering of arguments in directives

Currently, this snippet doesn't works:
{%
  include-markdown "{filepath}"
  comments=false
  start="<!-- start -->"
  end="<!-- end -->"
%}

Must be written as:
{%
  include-markdown "{filepath}"
  start="<!-- start -->"
  end="<!-- end -->"
  comments=false
%}

Added in v2.6.0

Added in v2.6.0


tests: adding benchmark tests against master

The output of 07-performance.sh will be something as:
Time gained against master (higher is better):
real 52.0373%
user 27.027%
sys 38.4615%
Signed-off-by: André Martins<EMAIL_ADDRESS>

This pull request is now managed with CodeReviewHub. Refresh.


Improvements to the command framework

Command condition API that allows for filtering and casting, for example the isPlayer condition that will provide a SquirtgunPlayer to a command if the executor is one
Removed CommandNode<T>#getPermission()
Removed CommandContext type argument
Command chains may now be covariant assuming appropriate conditions are specified, for example a node with type SquirtgunPlayer with an isPlayer condition may follow one with type SquirtgunUser

fixes #30

@emilyy-dev would you mind quickly sanity checking my code please

@emilyy-dev would you mind quickly sanity checking my code please

Will do !

Looks good overall, just a few things I noticed could be changed.
There are a lot of inconsistencies between tabs v. spaces across different files, please stick to one convention.

will resolve this in a separate branch after merging, see #32

ok i think we're all good now, anything you notice before i merge?

ok i think we're all good now, anything you notice before i merge?

Looks good!


Format duration including hour

Make sense if someone (e.g. me) uses it for long form podcasts.
I have decided to not use a leading zero for the hour, but one could really debate this.

Good idea @olikami ! Just, could you add appendZero function for hours also? It seems to me it will be more consistent with existing implementation...

We are using leading zero for seconds and minutes. So, I think we should use it for hours also...

@Werter12 done


[AppConfig-BugFix] Fixed bug that FeatureFlagConfigurationSetting overrides all of value when updating property

fixes: https://github.com/Azure/azure-sdk-for-java/issues/34904
FeatureFlagConfigurationSetting can take unknown properties other than the swagger-defined property definition. Such as
featureFlag.setValue("{\"conditions\":{\"requirement_type\":\"All\"}}"); // "requirement_type" is user-defined property

Users can edit the ConfigurationSetting's value directly or through Azure Portal UI.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-data-appconfiguration

Just trying to understand this change a bit more before giving a detailed review. What we're trying to support is FeatureFlagFilters that don't conform to the standard {"name":"name","parameters":{...}}?

Just trying to understand this change a bit more before giving a detailed review. What we're trying to support is FeatureFlagFilters that don't conform to the standard {"name":"name","parameters":{...}}?

The service supports any custom values that the customer wants, or any additional fields the service team wants.
For example we are trying to add a requirement_type field that allows for switching between or/and logic for feature flags. Currently if any modifications are made to the feature flag object any unknown fields are erased which shouldn't happen.
This issue is present in most of the SDKs.

Just trying to understand this change a bit more before giving a detailed review. What we're trying to support is FeatureFlagFilters that don't conform to the standard {"name":"name","parameters":{...}}?

The service supports any custom values that the customer wants, or any additional fields the service team wants.
For example we are trying to add a requirement_type field that allows for switching between or/and logic for feature flags. Currently if any modifications are made to the feature flag object any unknown fields are erased which shouldn't happen.
This issue is present in most of the SDKs.

Okay, given this, I think we should add support for "additional properties" throughout the setting. This is fairly standard functionality where a Map<String, Object> is added to a model and handled fluently in serialization and deserialization scenarios.

Offline discussion with Alan. Adding public API to support "unknown properties" which isn't in the swagger is not a good approach. Will keep no change in the SDK public surface. If supporting "unknown properties" in the public API, it should go through the swagger API archiboard and discussion.


[Feature Request] - Get position of a especific bone.

It would be cool if you added a procedure that allows you to obtain the xyz coordinates of a specific bone. This way, we could spawn particles as special effects coming out of a specific part of a creature (Like a flamethrower coming out of a dragon's head) even if the head is animated.

The values would be clientside only so that's not going to be very useful, plus you need access to the model code to do that so it can't be used in procedures.