[ENHANCEMENT]

Is your enhancement related to a problem? Please describe.

Display and monitor the percentage of used space per Virtual Machine volume in Monitoring / Grafana for each VM
Describe the solution you'd like

Adding graphs in the grafana panel "Harvester VM Info Detail"
Each volume (virtual disks allocated to the VM) appear in a distinct graph.
Please check "harvester-monitoring-Harvester_VM_Info Detail-SuggestedView.png" for a suggested representation.
Describe alternatives you've considered

Grafana setup
Additional context

Just to clarify, do you mean the disk's restorable size or do you mean the sum of data allocated across all of the disk's replicas?

Just to clarify, do you mean the or do you mean the sum of data allocated across all of the disk's replicas?
Hello,
What is "the disk's restorable size" ?

I don't think I can rephrase this (sorry, english is not my first language) :
"Display and monitor the used space percentage per Virtual Machine allocated volume"
Did you check the bottom of the second screenshot ? I've tried to represent at the bottom what I'd be looking for.


Pass actual ActorWfrp4e instance to scripts

Right now, on effects such as Prepare Data, the args.actor holds Actor's Data Model. This breaks some effects that rely on this, such as Malaise that uses args.actor.hasCondition() and args.actor.addCondition().
Proposed PR will change it instead to Actor's actual instance so effect scripts can have unchanged access to the entire Actor Document.

Iffy on this, though mainly because I want to keep as much congruity as I can with how I implemented it in IM (which doesn't pass the parent).
I would need to see how big of an impact this has, i.e. if this doesn't get changed, how many scripts are affected.

Unless I'm mistaken, all prepareData and similar effects will be affected and need a rewrite, because even a simple:
args.actor.system.characteristics -= 10;

Is no longer valid, since the Data Model is already the system and would need script like this:
args.actor.characteristics -= 10;

I feel like changing it from Document to Data Model is huge change that breaks compatibility with almost everything that does effects using any of:

prePrepareData
prepareData
prePrepareItems
preWoundCalc
woundCalc

Additionally, I think it's bad, because it would bring a lot of confusion from people asking:

Why sometimes in effects args.actor is actor and sometimes it's not?

A compromise could be providing both, or (if you really are against passing Document instance), changing argument's name from actor to model to remove confusion.
(and you could always update IM too if you want to reuse components...)

you could always update IM too

Well, the effects have all already been written. Nothing for WFRP4e has been rewritten yet for the new effects. It looks like changing this would prevent needing to rewrite many effects though so it's worth considering.

Could always add model as additional arg ;)

🤷 what's the point, args.actor.system gets you there.


header align bug

First open file, header like this:

and after the buffer is switched again, the buffer becomes normal

header config:
            headings = {
                enable = true,
                shift_width = 0,
                textoff = 9,
                heading_1 = {
                    style = 'label',
                    align = 'center',
                    hl = 'CursorLineNr',
                    icon = '',
                    sign = '',
                    sign_hl = 'CursorLineNr',
                },
                heading_2 = {
                    style = 'label',
                    align = 'center',
                    hl = 'Character',
                    icon = '',
                    sign = '',
                    sign_hl = 'Character',
                },
                heading_3 = {
                    style = 'label',
                    align = 'center',
                    hl = 'DiagnosticInfo',
                    icon = '',
                    sign = '',
                    sign_hl = 'DiagnosticInfo',
                },
                heading_4 = {
                    style = 'label',
                    align = 'center',
                    hl = 'Identifier',
                    icon = '',
                    sign = '',
                    sign_hl = 'Identifier',
                },
                heading_5 = {
                    style = 'label',
                    align = 'center',
                    hl = 'Identifier',
                    icon = '',
                    sign = '',
                    sign_hl = 'Identifier',
                },
                heading_6 = {
                    style = 'label',
                    align = 'center',
                    hl = 'Identifier',
                    icon = '',
                    sign = '',
                    sign_hl = 'Identifier',
                },
            },

You don't happen to use a file-tree plugin, do you?
The plugin gets the width of the current window(as it's impossible to get the width of the window containing the buffer since multiple windows with various sizes can have the same buffer). So, if you opened it via some other plugin it will look like that.

Yes, i use the neo-tree plugin.
I understand, thank you

It should work now. Can you check again?

@hyawara, It should work now. Can you check again?

Give me a second. I'll try. I'll get back to you

https://github.com/user-attachments/assets/1f588c5f-8155-4ab1-8af7-567f641af318
I need to move it to make it work.
But it worked out. You were very efficient, thanks.

Does it work now?

I'll try it tomorrow morning. 😁

There are still some problems @OXY2DEV
https://github.com/user-attachments/assets/5c848f0c-643c-4c39-93c3-911ac74dcff0

It should be fixed now.

Do you know how to listen for buffer window size changes? I think it should be the problem.
https://github.com/user-attachments/assets/8a3ea436-8106-48fa-aa70-6b132f0d3076

WinResized won't be used by the plugin.
For example, you have a file opened in a window and hit :vsp.
Now, you have 2 windows containing the same buffer. Now if you resize one of them that window will have the correct alignment but the other window will have an incorrect alignment.

There's no relation between buffers & windows. So, there's no good way to use WinResize to update the buffer.

Also, someone famous once said, "If you always have your file browser open, I have a lot of questions and the only important one is, 'Why?'"
So, there's no point in adding something for every niche case.

Could you provide me with a copy of your nvim configuration for my reference?  thank you

OXY2DEV/nvim


[BUG?] Build fails on latest version (using g++)

I'm using the latest version (v3.1.5) and the CI/CD is throwing some glaze errors, it's building fine on windows
https://github.com/edunad/rawrbox/actions/runs/10037621909/job/27737794760#step:7:352
Using the following struct and passing it down to a template
struct TestSettingsData {
	std::string VERSION = "0.0.2";

	std::map<std::string, float> video = {{"scale", 0.5F}, {"monitor", 2.F}};
	std::map<std::string, std::string> controls = {{"jump", "A"}, {"crouch", "L_CNTRL"}};

	std::string username = "MISSING";
};

(i also added requires guards write_json_supported and read_json_supported and they pass fine)
https://github.com/edunad/rawrbox/blob/features/popup/rawrbox.utils/include/rawrbox/utils/settings.hpp#L14
https://github.com/edunad/rawrbox/blob/features/popup/rawrbox.utils/tests/settings.spec.cpp#L6

I added test cases for this use case in #1201 and they pass with GCC 12, 13, and 14. I think the problem is that your CI/CD is using GCC 11, which hasn't been supported for a while and now fails all the time with Glaze version 3 and beyond.
So, I think it is a compiler bug that I don't expect will be fixed in GCC 11.


⚠ deces.matchid.io search API has degraded performance

In e2415cd, deces.matchid.io search API (https://deces.matchid.io/deces/api/v1/search?q=pompidou) experienced degraded performance:

HTTP code: 200
Response time: 2831 ms

Resolved: deces.matchid.io search API performance has improved in 1dcb12b.


Goals (Мої цілі) in Personal Cabinet

As a registered logged in user I want to see my goals in personal cabinet so that I can track my tasks to be done.
Mockups:

Acceptance criteria:

My goals section contain:
a. title "My goals"
b. tooltip with text "Here you can set your goals and complete them!" or list of chosen/added goals" and checkboxes near each goal.
c. Text "Review all my goals" with hyperlink
d. "Manage goals" button

WHEN registered logged in user just registered and logged in the personal cabinet under title "My goals" he can see tooltip with text "Here you can set your goals and complete them!" and "Manage goals" button.

WHEN registered logged in user clicks on "Manage goals" button, he can see popup. This feature is described in user story #334 #347

WHEN registered logged in user clicks "Review all my goals", list of my goals expands with all my goals (already done and not done) and "Show less" button is available.
WHEN registered logged in user clicks "Show less" button, list roll up to 3 goals.

Assumption and constraints:

Not relevant.


Error during IBI csg Tutorial of liquid Hexane

I am trying to perform Hexane tutorial. I finished the atomistic simulation of hexane and got the A-A, A-B, B-B rd's and bond and angle distribution also. Now, while I am trying to to the IBI coarse graining an occurred and this was mentioned in the onverse.log file:
For a more verbose log see: inverse.log
csg_get_property: No value for 'cg.inverse.mpi.tasks' found in /root/votca/csg-tutorials/hexane/ibi_all/settings.xml, trying /share/votca/xml/csg_defaults.xml
csg_get_property: returning emtpy value for 'cg.inverse.mpi.tasks'
csg_get_property: No value for 'cg.inverse.mpi.cmd' found in /root/votca/csg-tutorials/hexane/ibi_all/settings.xml, trying /share/votca/xml/csg_defaults.xml
csg_get_property: returning emtpy value for 'cg.inverse.mpi.cmd'
csg_get_property: No value for 'cg.inverse.parallel.tasks' found in /root/votca/csg-tutorials/hexane/ibi_all/settings.xml, trying /share/votca/xml/csg_defaults.xml
csg_get_property: returning emtpy value for 'cg.inverse.parallel.tasks'
csg_get_property: No value for 'cg.inverse.parallel.cmd' found in /root/votca/csg-tutorials/hexane/ibi_all/settings.xml, trying /share/votca/xml/csg_defaults.xml
csg_get_property: returning emtpy value for 'cg.inverse.parallel.cmd'
csg_get_property: No value for 'cg.inverse.gromacs.mdrun.bin' found in /root/votca/csg-tutorials/hexane/ibi_all/settings.xml, trying /share/votca/xml/csg_defaults.xml
csg_get_property: returning emtpy value for 'cg.inverse.gromacs.mdrun.bin'
csg_get_property: No value for 'cg.inverse.espresso.bin' found in /root/votca/csg-tutorials/hexane/ibi_all/settings.xml, trying /share/votca/xml/csg_defaults.xml
csg_get_property: returning emtpy value for 'cg.inverse.espresso.bin'
csg_get_property: No value for 'cg.inverse.scriptdir' found in /root/votca/csg-tutorials/hexane/ibi_all/settings.xml, trying /share/votca/xml/csg_defaults.xml
csg_get_property: returning emtpy value for 'cg.inverse.scriptdir'
csg_get_property: No value for 'cg.inverse.gromacs.grompp.topol' found in /root/votca/csg-tutorials/hexane/ibi_all/settings.xml, trying /share/votca/xml/csg_defaults.xml
csg_get_property: returning emtpy value for 'cg.inverse.gromacs.grompp.topol'
csg_get_property: No value for 'cg.inverse.gromacs.grompp.index' found in /root/votca/csg-tutorials/hexane/ibi_all/settings.xml, trying /share/votca/xml/csg_defaults.xml
csg_get_property: returning emtpy value for 'cg.inverse.gromacs.grompp.index'
csg_get_property: No value for 'cg.inverse.gromacs.g_rdf.topol' found in /root/votca/csg-tutorials/hexane/ibi_all/settings.xml, trying /share/votca/xml/csg_defaults.xml
csg_get_property: returning emtpy value for 'cg.inverse.gromacs.g_rdf.topol'
csg_get_property: No value for 'cg.inverse.convergence_check' found in /root/votca/csg-tutorials/hexane/ibi_all/settings.xml, trying /share/votca/xml/csg_defaults.xml
csg_get_property: returning emtpy value for 'cg.inverse.convergence_check'
csg_get_property: No value for 'cg.inverse.convergence_check_options.name_glob' found in /root/votca/csg-tutorials/hexane/ibi_all/settings.xml, trying /share/votca/xml/csg_defaults.xml
csg_get_property: returning emtpy value for 'cg.inverse.convergence_check_options.name_glob'
csg_get_property: No value for 'cg.inverse.convergence_check_options.limit' found in /root/votca/csg-tutorials/hexane/ibi_all/settings.xml, trying /share/votca/xml/csg_defaults.xml
csg_get_property: returning emtpy value for 'cg.inverse.convergence_check_options.limit'
csg_get_property: No value for 'cg.inverse.espresso.table_end' found in /root/votca/csg-tutorials/hexane/ibi_all/settings.xml, trying /share/votca/xml/csg_defaults.xml
csg_get_property: returning emtpy value for 'cg.inverse.espresso.table_end'
csg_get_property: No value for 'cg.inverse.gromacs.traj_type' found in /root/votca/csg-tutorials/hexane/ibi_all/settings.xml, trying /share/votca/xml/csg_defaults.xml
csg_get_property: returning emtpy value for 'cg.inverse.gromacs.traj_type'
csg_get_property: No value for 'cg.inverse.gromacs.topol_out' found in /root/votca/csg-tutorials/hexane/ibi_all/settings.xml, trying /share/votca/xml/csg_defaults.xml
I checked the settings.xml file; but above mentioned things are not present there. I am unable to understand the error.
Please help.

On Tue, Sep 1, 2020, 03:52 mangesh-bhendale<EMAIL_ADDRESS>wrote:

I am trying to perform Hexane tutorial. I finished the atomistic
simulation of hexane and got the A-A, A-B, B-B rd's and bond and angle
distribution also. Now, while I am trying to to the IBI coarse graining an
occurred and this was mentioned in the onverse.log file:
For a more verbose log see: inverse.log
csg_get_property: No value for 'cg.inverse.mpi.tasks' found in
/root/votca/csg-tutorials/hexane/ibi_all/settings.xml, trying
/share/votca/xml/csg_defaults.xml
csg_get_property: returning emtpy value for 'cg.inverse.mpi.tasks'
csg_get_property: No value for 'cg.inverse.mpi.cmd' found in
/root/votca/csg-tutorials/hexane/ibi_all/settings.xml, trying
/share/votca/xml/csg_defaults.xml
csg_get_property: returning emtpy value for 'cg.inverse.mpi.cmd'
csg_get_property: No value for 'cg.inverse.parallel.tasks' found in
/root/votca/csg-tutorials/hexane/ibi_all/settings.xml, trying
/share/votca/xml/csg_defaults.xml
csg_get_property: returning emtpy value for 'cg.inverse.parallel.tasks'
csg_get_property: No value for 'cg.inverse.parallel.cmd' found in
/root/votca/csg-tutorials/hexane/ibi_all/settings.xml, trying
/share/votca/xml/csg_defaults.xml
csg_get_property: returning emtpy value for 'cg.inverse.parallel.cmd'
csg_get_property: No value for 'cg.inverse.gromacs.mdrun.bin' found in
/root/votca/csg-tutorials/hexane/ibi_all/settings.xml, trying
/share/votca/xml/csg_defaults.xml
csg_get_property: returning emtpy value for 'cg.inverse.gromacs.mdrun.bin'
csg_get_property: No value for 'cg.inverse.espresso.bin' found in
/root/votca/csg-tutorials/hexane/ibi_all/settings.xml, trying
/share/votca/xml/csg_defaults.xml
csg_get_property: returning emtpy value for 'cg.inverse.espresso.bin'
csg_get_property: No value for 'cg.inverse.scriptdir' found in
/root/votca/csg-tutorials/hexane/ibi_all/settings.xml, trying
/share/votca/xml/csg_defaults.xml
csg_get_property: returning emtpy value for 'cg.inverse.scriptdir'
csg_get_property: No value for 'cg.inverse.gromacs.grompp.topol' found in
/root/votca/csg-tutorials/hexane/ibi_all/settings.xml, trying
/share/votca/xml/csg_defaults.xml
csg_get_property: returning emtpy value for
'cg.inverse.gromacs.grompp.topol'
csg_get_property: No value for 'cg.inverse.gromacs.grompp.index' found in
/root/votca/csg-tutorials/hexane/ibi_all/settings.xml, trying
/share/votca/xml/csg_defaults.xml
csg_get_property: returning emtpy value for
'cg.inverse.gromacs.grompp.index'
csg_get_property: No value for 'cg.inverse.gromacs.g_rdf.topol' found in
/root/votca/csg-tutorials/hexane/ibi_all/settings.xml, trying
/share/votca/xml/csg_defaults.xml
csg_get_property: returning emtpy value for
'cg.inverse.gromacs.g_rdf.topol'
csg_get_property: No value for 'cg.inverse.convergence_check' found in
/root/votca/csg-tutorials/hexane/ibi_all/settings.xml, trying
/share/votca/xml/csg_defaults.xml
csg_get_property: returning emtpy value for 'cg.inverse.convergence_check'
csg_get_property: No value for
'cg.inverse.convergence_check_options.name_glob' found in
/root/votca/csg-tutorials/hexane/ibi_all/settings.xml, trying
/share/votca/xml/csg_defaults.xml
csg_get_property: returning emtpy value for
'cg.inverse.convergence_check_options.name_glob'
csg_get_property: No value for
'cg.inverse.convergence_check_options.limit' found in
/root/votca/csg-tutorials/hexane/ibi_all/settings.xml, trying
/share/votca/xml/csg_defaults.xml
csg_get_property: returning emtpy value for
'cg.inverse.convergence_check_options.limit'
csg_get_property: No value for 'cg.inverse.espresso.table_end' found in
/root/votca/csg-tutorials/hexane/ibi_all/settings.xml, trying
/share/votca/xml/csg_defaults.xml
csg_get_property: returning emtpy value for 'cg.inverse.espresso.table_end'
csg_get_property: No value for 'cg.inverse.gromacs.traj_type' found in
/root/votca/csg-tutorials/hexane/ibi_all/settings.xml, trying
/share/votca/xml/csg_defaults.xml
csg_get_property: returning emtpy value for 'cg.inverse.gromacs.traj_type'
csg_get_property: No value for 'cg.inverse.gromacs.topol_out' found in
/root/votca/csg-tutorials/hexane/ibi_all/settings.xml, trying
/share/votca/xml/csg_defaults.xml
I checked the settings.xml file; but above mentioned things are not
present there. I am unable to understand the error.

There aren't errors, it just means there is no default value for these
properties.
What are the last couple of lines before the error message? Or attach the
whole inverse.log here.
Christoph

Please help.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/votca/csg/issues/590, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADLJO7P5CHP5CDFD4AYY5DSDS743ANCNFSM4QRQRFLA
.

Yes, I did not find any default values of those parameters. Where can I find them?
I am attaching the inverse.log file here.
inverse.log

On Tue, Sep 1, 2020 at 07:29 mangesh-bhendale<EMAIL_ADDRESS>wrote:

Yes, I did not find any default values of those parameters. Where can I find them?
I am attaching the inverse.log file here.
inverse.log

If you look at the very end it says “gmx_d grompp” failed and if you
scroll a bit up
Fatal error:
The group cutoff scheme has been removed since GROMACS 2020. Please use the
Verlet cutoff scheme.
In short, to do coarse-graining you basically need to use gmx-2019.*
(or lammps) as the current gromacs version has no support for
tabulated interactions.
(see and comment on https://gitlab.com/gromacs/gromacs/-/issues/1347)
Cheers,
Christoph

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

A warning was added a while back to CMake in #452.


Merging PR#62

This fixes the issues for the discussion in PR #62 and removed the explicit dependency on the import statement.

Hey @JensRavens any chance a tag could be pushed (and also a pod trunk push) so I could consume the change with Swift 3.2 in my library?

@loudmouth good idea. done! :)

woot! 🙏


Resize the keyboard vertically for optimal fit

Hi!
It is impossible to change the vertical keyboard.
    public function execute()
    {
        $keyboard = new Keyboard([
            ['text' =>'Hello'],
        ]);

        //Return a random keyboard.
        $keyboard
            ->setResizeKeyboard(true)
            ->setOneTimeKeyboard(true)
            ->setSelective(false);

        $chat_id = $this->getMessage()->getChat()->getId();
        $data = [
            'chat_id'      => $chat_id,
            'text'         => 'Text',
            'reply_markup' => $keyboard,
        ];

        return Request::sendMessage($data);
    }

What I do not so?
Screenshot https://www.dropbox.com/s/mmbwfwwpt3i6i46/2016-12-18_223313.png?dl=0

This bug is fixed with ae6c3ef2066abeec233fac951cc0b6fbef56c5c0
Could you try by manually adding the change? Just to make sure that it was that.

Thanks It works)


Error type assertion gives 2 options as optional

I'm not quite sure how this works it seems to be equivlant to value,ok := map[get] where there is the value and then an optional, bool.
Here is the code example:
type argError struct {
    arg  int
    prob string
}
func f2(arg int) (int, error) {
	if arg == 42 {

		// In this case we use `&argError` syntax to build
		// a new struct, supplying values for the two
		// fields `arg` and `prob`.
		return -1, &argError{arg, "can't work with it"}
	}
	return arg + 3, nil
}

func (e *argError) Error() string {
 return fmt.Sprintf("%d - %s", e.arg, e.prob)
}

func main() {
    _, e := f2(42)
    if ae, ok := e.(*argError); ok {
        fmt.Println(ae.arg)
        fmt.Println(ae.prob)
    }
}

Need to figure out some method to handle this as there is only the option for one variable or multiple variables but they all must be captured with a destruct.

Not sure it matters in this example, but in Go, to be an "error", a type must have a method Error() that returns string. So in the code above, argError is not a Go error type because it does not conform to the Error interface. See https://blog.golang.org/error-handling-and-go.
Returning multiple values is quite normal in Go, using their positions to name them. Indeed assignments can have multiple values too. Some examples are at https://riptutorial.com/go/example/2193/multiple-variable-assignment.
My recollection is that TardisGo used Haxe anonymous structures to return multiple values, with names "r0","r1","r2"... to give the positional information.
In hx2go's AST->AST conversion, that is more complicated for one of the above examples, where a and b would need to be remapped to r0 and r1 in the Haxe anonymous structure to be returned:
func multipleReturn2() (a int, b int) {
    a = 3
    b = 4
    return
}

And yes, the Haxe code generated requires the returned anonymous structure to be put into a temporary variable and then the individual return values taken out of it.

Apologies, I updated the code snippet I left out the receiver function (the code comes from gobyexamples's Errors) . I currently have a system for anonymous variable names with positions at the end, and not yet for named variables. You bring up a good idea to create temporary variables at the start of the function expr if the return values are named, and then return them through an anonymous structure. Thank you for the info and I'll go through the Go's blog post on Errors that should be quite useful.

This seems to be working fine for the latest few test cases I've stumbled upon this, and future new cases can be setup accordingly.


Add Epidemic Sound as Adopter

Signed-off-by: Safaa El Allouche

[ ] I added CHANGELOG entry for this change.
[x] Change is not relevant to the end user.

Changes

Added Epidemic Sound as new item in the adopters.yaml list
Added Epidemic Sound logo

Verification
Following guidelines from: https://thanos.io/tip/contributing/how-to-contribute-to-docs.md/#adopters-logos

There's an unrelated failure in the docs:
M	docs/components/compact.md

I think we should merge this PR and fix in main


Update cocoapods requirement

Without this 'make setup' fails with :
[!] The `master' repo requires CocoaPods 0.32.1 -

jamiebullock I am not able to run the sample project .Could you please help me.


add radar and wait functions

This PR adds two functions from Mindustry Logic missing in Mindcode's built-in functions:

radar. Locate units from a building.

building = radar(prop1, prop2, prop3, sortby, sort) ⇔ radar prop1 prop2 prop3 sortby building sort
propX must be one of any, enemy, attacker, ally, player, ground, flying, or boss. These must be hardcoded and can't be substituted with an indirect reference (seems like an issues with Mindustry).
sortby must be one of distance, health, shield, armor, or maxHealth.

wait. Delays for delay seconds. Can be a decimal number.

wait(delay) ⇔ wait delay

Test cases have been added LogicInstructionGeneratorTest.java.

Thank you very much for your contribution. This was merged and shall be automatically deployed.


Bring compatibility with http4s 0.22

closes https://github.com/guardrail-dev/guardrail/issues/1199
blocked on https://github.com/http4s/http4s/pull/5109

@blast-hardcheese can you please run the CI again, I can't do it :(
Regarding
scala> org.http4s.dsl.io.Path.apply("this/is/a/test", "").toString().init.tail
res0: String = this%2Fis%2Fa%2Ftest

vs
@ org.http4s.dsl.io.Path.Segment.apply("this/is/a/test").toString
res2: String = "this%2Fis%2Fa%2Ftest"

Aren't the " around the string an issue? Or is it better if it contains them?
Regarding the support for multiple http4s version -- I like the plan, but I'm not sure this is the right time to do that. http4s 0.21 is End Of Life anyway, so it's not worth doing it for the sake of 0.21.
The currently alive versions of http4s are 0.22 and 0.23, but I would hope that they are source compatible (the reason for 0.23 is AFAIK support for cats-effect 3 which is fundamental redesign over 2, but I would hope that http4s stays the same :smiley: ).

I've just checked, and the current changes work not only with 0.22, but also 0.23 and 1.0.0-M24.

The tests are now fixed. Can you please run the CI? :pray:

I've fixed the tests again. Please run the CI :pray:

OK, hopefully now everything passes :crossed_fingers: Please give the CI one more kick :smile:

So the problem was -Xss being too small in whatever sbt wrapper GitHub Actions sets by default. TIL we're not actually customizing which sbt launcher we're using. I'll open a PR to swap the setup-java with setup-scala which will resolve this by at least giving us a stable sbt version.

🎉


Autolocalhost

Add function to create a Resource object by reading the topology of the host machine. ( closes #9 )
Fix a bug in the MPI communicator with the partial bcast/gather when sending data multiple times. ( closes #10 )

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

12 of 12 (100.0%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.3%) to 83.119%

Files with Coverage Reduction
New Missed Lines
%

src/godrick.cpp
1
64.52%

Totals

Change from base Build<PHONE_NUMBER>:
0.3%

Covered Lines:
581

Relevant Lines:
699

💛  - Coveralls


npm error

i got an error when "npm install"
163 error Error: spawn ENOMEM
163 error     at errnoException (child_process.js:988:11)
163 error     at ChildProcess.spawn (child_process.js:935:11)
163 error     at exports.spawn (child_process.js:723:9)
163 error     at exports.execFile (child_process.js:607:15)
163 error     at /usr/share/npm/lib/cache.js:470:7
163 error     at /usr/lib/nodejs/which.js:55:18
163 error     at Object.oncomplete (fs.js:107:15)
164 error If you need help, you may report this log at:
164 error     <http://github.com/isaacs/npm/issues>
164 error or email it to:
164 error<EMAIL_ADDRESS>165 error System Linux 3.4.0+
166 error command "/usr/bin/nodejs" "/usr/bin/npm" "install"
167 error cwd /root/pokemon-go-mitm-node
168 error node -v v0.10.25
169 error npm -v 1.3.10
170 error syscall spawn
171 error code ENOMEM
172 error errno ENOMEM

and i use the TAOBAO npm mirror http://npm.taobao.org
Does it cause the error?

error Error: spawn ENOMEM seems like you're running out of memory?

So how do i solve this error?

uh, i found that i use node-v0.10.25, does it cause the error?

Does your computer have enough free memory to run Node?

how much memory does Node need? although i just have 600M free memory..

when i update my node version, there is a new error like this:

Are you using Cygwin or the Windows 10 Linux runtime? Your install is missing the pkg-config tool.

thx for @trisk 's help, i solved this error. But i get another error when run 'coffee example.logTraffic.coffee'. There is the log below:

when i change the Node version to the new stable(not 0.xx.xx) and reinstall the repo('npm uninstall' and 'npm install'), i solve this, now i can enjoy it, thank you all! :)


fix links

might help with #120

Thanks! Still one broken link, shall I merge like this, or would you be willing to fix the last link too?
This Travis build really helps to keep the amount of broken links down, but the sheer number of links makes it hard to keep up. Appreciate the help :-)

https://vim.sourceforge.io/scripts/script.php?script_id=4293 is the link I added, after testing it. They have http://vim.sourceforge.io/scripts/script.php?script_id=4293 also, but that was "broken" before. I presume Travis is using a user-agent that they classify as a bot, and reject the connection.
So, IMHO this can be merged, unless the links are broken for you as well.

Ok, lets merge like this. Definitely improves matters :-)

Thanks again, updated some more links (#123) to finally get back to 0 dead links. :-)

🎉 You're welcome :-)


header_card won't resize according to window size

Wave SDK Version, OS

Mac OS
 h2o-wave==0.25.2

Actual behavior
At the moment the card is cut towards the end since cards don't seem to resize.

Expected behaviour
It would be nice if the header card resizes when a window is resized. Is this possible to fix somehow with the current version ?

Steps To Reproduce
async def serve(q: Q):
  q.page['header'] = ui.header_card(
          box='1 1 10 1',
          title='Wave App',
          subtitle='',
          icon='FlashAuto',
          color="card",
          items=[ui.menu(image=image, items=[
              ui.command(name='profile', label='Profile', icon='Contact'),
              ui.command(name='preferences', label='Preferences',
                         icon='Settings'),
              ui.command(name='logout', label='Logout', icon='SignOut'),
          ])]
      )
   await q.page.save()

Hi @Venkat-Rajgopal. The reason your code above is not working is because you have used a fixed grid layout (box='1 1 10 1'). If you want to achieve responsiveness, you need to reach for flex layout instead.
# Layout / Responsive
# How to create a #responsive #layout.
# ---
from h2o_wave import main, app, Q, ui

@app('/')
async def serve(q: Q):
    content = '![Wave University](https://raw.githubusercontent.com/h2oai/wave/master/assets/brand/wave-university-wide.png)'

    # The meta card's 'zones' attribute defines placeholder zones to lay out cards for different viewport sizes.
    # We define three layout schemes here.
    q.page['meta'] = ui.meta_card(box='', layouts=[
        ui.layout(
            # If the viewport width >= 0:
            breakpoint='xs',
            zones=[
                # 80px high header
                ui.zone('header', size='80px'),
                # Use remaining space for content
                ui.zone('content')
            ]
        ),
    ])

    q.page['header'] = ui.header_card(
        # Place card in the header zone, regardless of viewport size.
        box='header',
        title='Lorem Ipsum',
        subtitle='Excepteur sint occaecat cupidatat',
        nav=[
            ui.nav_group('Menu', items=[
                ui.nav_item(name='#menu/spam', label='Spam'),
                ui.nav_item(name='#menu/ham', label='Ham'),
                ui.nav_item(name='#menu/eggs', label='Eggs'),
            ]),
            ui.nav_group('Help', items=[
                ui.nav_item(name='#about', label='About'),
                ui.nav_item(name='#support', label='Support'),
            ])
        ],
    )
    q.page['controls'] = ui.markdown_card(
        # If the viewport width >= 0, place in content zone.
        # If the viewport width >= 768, place in sidebar zone.
        # If the viewport width >= 1200, place in sidebar zone.
        box='content',
        title='Controls',
        content=content,
    )
    await q.page.save()

Can I resize the cards which are then placed in the content box. ?

Yes, everything you need is in the docs.


[Bug] List of all the compose box related bugs that we need to get fixed soon

Coming from https://expensify.slack.com/archives/C01SKUP7QR0/p1666245169556899
I have categorized the issues based on the type/cause of the issue. Feel free to add more info/issues or update things to make it more clear and help us track them better.
Focus and cursor-related issue

[Looking for proposals] #11093

Composer issue during transition

[PR review in progress] #11471
[Hold to Deprecate sequence number] #11293
[Proposal review in progress] #8592

Emoji related issues

[Proposal review in progress] #11438
[Waiting for payment] #10356

Offline related issue

[Looking for proposal] #11265

Miscellaneous

[Looking for proposals] #11921

Problem
Recently we have been seeing a lot of bugs related to compose box where some of them quite serious ones like Message deletes as you type, and message content staying in composer box even after the message was sent. Since it is one of the most important components of a chat app polishing it perfectly should be our end goal. Some of the issues that have popped up recently are regression that went unnoticed during tests so having a bird's eye view will keep things neat and in control.
Goal

Look into the list before you create a new issue to make sure there is no duplicates issue in the picture.
While testing a PR related to compose box make sure you are not introducing any of the existing/completed bugs that are listed.

https://github.com/Expensify/App/issues/11437 is deployed on staging and has been checked off from the deploy blocker list. Once it's closed we can examine https://github.com/Expensify/App/issues/11968, and https://github.com/Expensify/App/issues/11885 and we will be good to close them.

Addressed #11346 and we should have a fix soon. (But I am a bit concerned over here because of the general behaviour when placeholder content is long will not get fixed). So rather than fixing the copy shall we look into the main issue, here?

I will take a look in the later part of the day and will post an update here.

Categorized the issue based on the type/origin and added progress info to each of the issues. Feel free to start a discussion around any of the above issues.

@techievivek It looks like most of the issues catalogued in this tracking issue are done, held on other initiatives, or being worked on elsewhere by Rn experts. What do you think about updating this OP to highlight the issues that are with contributors and that either need proposals or that we need to help get merged?

Just wanted to follow up on that question above?

Hi @jason, looking into other tracking issues it makes sense to just track the issues there either need help or are actively being handled by contributors. I will update the OP with the changes. Thanks

Ok cool. Let me know if you want help re-testing any of the linked issues once PRs are merged.

Are we keeping tracking GHs weekly? I think it certainly makes sense to! Just want to ensure we are on the same page.

Yeah I think that's best for now.

@michaelhaxhiu Do you want to assign yourself to this tracking issue. I noticed we need a BZ assignee.

👋 Hey-o! What are we accomplishing with this tracking issue? It seems undefined currently, so I'm not quite sure I understand its purpose. Looking at the solution ("goals") stated in the OP:

Look into the list before you create a new issue to make sure there is no duplicates issue in the picture.

This is the expectation across the board, to check the repo for duplicate issues.

While testing a PR related to compose box make sure you are not introducing any of the existing/completed bugs that are listed.

How does anyone know to do this? Equally, you shouldn’t have to x-ref a tracking issue to avoid introducing new bugs in the code. That should be done through bettering documentation, the PR checklist etc.
We have other tracking issues that are actively working on a "wider" refactor or revisit to a flow/implementation to clean-up a bunch of bugs that have spawned from it. It doesn't seem like we're doing that here in this case.

All great points!

@trjExpensify

This is the expectation across the board, to check the repo for duplicate issues.

I just updated the OP yesterday with the latest update but we did see a lot of duplicate issues in this domain. Given the number of new issues popping up daily, it was hard to find duplicate ones(thanks to external contributors for helping us figure that out). And given the component is almost the first point of contact in any chat app I thought prioritizing this will definitely be a good call.
Most of the issues are closed as duplicates or are being worked on by RN experts we have got few which still need proposals. And since no new issues/regression is coming around this component I think we are safe to close this. Also, it would be good to make sure there are a bunch of tests related to compose box in the testRail. I am not very familiar with the testRail for now but I will be happy to look into it and make sure we have included enough tests for it.

Also, it would be good to make sure there are a bunch of tests related to compose box in the testRail. I am not very familiar with the testRail for now but I will be happy to look into it and make sure we have included enough tests for it.

Okay cool, so this seems like an actionable step if we think this is under tested. In which case, this issue should be updated to focus on that goal of identifying and writing missing tests.
For reference, here is the compose box section of the Test Cases that exists in TestRail right now.


Bump sbt-slamdata to 0.0.13

Pending slamdata/sbt-slamdata#17

@drostron slamdata/sbt-slamdata#17 is merged.

@alissapajer Excellent, I'll restart the build once the sbt-slamdata artifact is available.

Codecov Report

Merging #1975 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1975   +/-   ##
=======================================
  Coverage   64.87%   64.87%
=======================================
  Files         380      380
  Lines       18772    18772
  Branches      225      232    +7
=======================================
  Hits        12178    12178
  Misses       6594     6594

Impacted Files
Coverage Δ

web/src/main/scala/quasar/api/zip.scala
90.74% <0%> (ø)
:white_check_mark:

...cala/quasar/frontend/logicalplan/LogicalPlan.scala
89.02% <0%> (ø)
:white_check_mark:

...cala/quasar/physical/marklogic/xquery/syntax.scala
11.32% <0%> (ø)
:white_check_mark:

...hysical/sparkcore/fs/hdfs/parquet/ParquetRDD.scala
100% <0%> (ø)
:white_check_mark:

...ontend/src/main/scala/quasar/sql/SqlRelation.scala
85.29% <0%> (ø)
:white_check_mark:

frontend/src/main/scala/quasar/codec.scala
95.4% <0%> (ø)
:white_check_mark:

...ala/quasar/frontend/logicalplan/LogicalPlanR.scala
81.2% <0%> (ø)
:white_check_mark:

sql/src/main/scala/quasar/sql/semantics.scala
91.05% <0%> (ø)
:white_check_mark:

...uasar/physical/marklogic/xquery/FunctionDecl.scala
6.66% <0%> (ø)
:white_check_mark:

web/src/main/scala/quasar/api/MessageFormat.scala
94.25% <0%> (ø)
:white_check_mark:

... and 10 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 09a4191...9548a21. Read the comment docs.


Allow show/hide functionality for passwords in ui.textbox()

Is your feature request related to a problem? Please describe
While building Login Forms using h2o.ai Wave, There is a password property in ui.textbox(). However, unlike login forms on web apps on the internet, there is no option to make the icon of the field clickable in order to toggle between the show text and hide text option for the password field.
Describe the solution you'd like
It would be nice to have an inbuilt functionality to show/hide the text being typed in password style textboxes. Or atleast make the icons within a textbox clickable so we it can be implemented through code.
Describe alternatives you've considered
A simple workaround right now is to add a button alongside the textbox to toggle between showing and hiding the typed text. However, it isn't very flattering from a UX perspective, as the button is selected on hover and the boundaries of the textbox is also visible.

I would like to work on this issue.
Please assign it to me.


When release the second phase training code

Hi, thanks for sharing the code,
When release the second phase training code and the inference code?
Thanks

Thanks for your interest in our work. Please stay tuned. We are currently refactoring the codes. It should be released soon.

Hi, the second-phase training is available now.


⚠ Bofrost.be has degraded performance

In 0554d2a, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1722 ms

Resolved: Bofrost.be performance has improved in 079a289.


New Sample: Identify raster cell

Another fun and interactive sample.
I think the logic for freezing/unlocking the callout worked out ok, but let me know if you have any other ideas.

@Rachael-E would you mind having a second look after the changes?

Thanks for the review @Rachael-E.
@tschie should be ready for your review now. Thanks!

Hi @mbcoder, would you mind reviewing when you have some time?

@mbcoder @Rachael-E since the discussion is ongoing, I've reverted the build.gradle to only include the relevant javafx modules.
@mbcoder , would you please do another review so we could merge this sample?

Thanks both!


Use Arrows for_json for JSON output

Arrow provides a for_json() function encoding Arrow objects for JSON output.
Using the default-parameter of json.dumps(), this function is invoked on Arrow objects which can then be encoded for JSON output.
This PR fixes #329

Sure, I added a short description to the changelog and provided some basic tests.
Let me know if the tests should cover more cases, I provided ones for the two that came to my mind.


feature request - allow an action-row to be at top of expansion panel

Bug, feature request, or proposal:
Feature - allow action-row to be at top of expansion panel
What is the expected behavior?
I was hoping I could put an action-row at the top of an expansion panel, just below the title.
What is the current behavior?
https://stackoverflow.com/questions/48551683/include-action-row-at-the-top-of-expansion-panel
What are the steps to reproduce?
Providing a StackBlitz reproduction is the best way to share your issue.
StackBlitz starter: https://goo.gl/wwnhMV
What is the use-case or motivation for changing an existing behavior?
It would be nice to have action buttons at the top and the bottom of an expansion panel, especially if the content of the expansion panel is tall.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Is there anything else we should know?

This is actually not allowed, because the entire <mat-expansion-panel-header> within the expansion panel is a button.  Placing another button in it as described in the request would break the a11y of the component.

damn! I guess I can put my own buttons in the top right tho right

It is possible to place action buttons inside the <mat-panel-description> and in the event of (click) execute: event.stopPropagation()


Add class selector to transformStyle & backfaceVisibility utilities

The transformStyle and backfaceVisibility utility classes are missing the class selector ., was receiving this error:
ERROR  in ./assets/css/tailwind.css

Syntax Error: ModuleBuildError: Module build failed (from ./node_modules/postcss-loader/src/index.js):                                                                            friendly-errors 00:39:37
SyntaxError

(121:1) Variant cannot be generated because selector contains no classes.

  119 | }
  120 |
> 121 | @tailwind utilities;
      | ^
  122 |

I am having the same issue.

Thank you!


Auto Generated ActiveEnum String Values And Model Column Names

PR Info

Closes #2160, #2125

New Features

[x] Add support for column_name_case attribute in DeriveEntityModel for automatically generating column names for all fields based on case style provided.
[x] Add support for column_name_case attribute in fields within a model that derives DeriveEntityModel for automatically generating column name based on case style provided.
[x] Add support for rename_all attribute in DeriveActiveEnum for automatically generating string values for all variants of the enum based on case style provided.
[x] Add support for rename attribute for variants of an ActiveEnum that derives DeriveActiveEnum for automatically generating string value for it based on case style provided.
[x] Finalise names of newly added attributes.
[x] Finalise allowed cases for column names
[x] Finalise newly added error messages
[x] Add documentation for new attributes

Dependents

https://github.com/SeaQL/seaql.github.io/pull/121

@tyt2y3 @billy1624 Can I ask for your opinions for these:

Finalise names of newly added attributes.
Finalise allowed cases for column names
Finalise newly added error messages

Hi @tyt2y3 @billy1624, sorry to ping you about this! Did you get a chance to look at it yet?

@billy1624 This is ready for your review in my opinion.

Thanks @billy1624

@tyt2y3 Please review.

@tyt2y3 @billy1624 Thanks for all your help as well.

@anshap1719 Just wanted to inform you that this no longer works:
#[derive(EnumIter, DeriveActiveEnum)]
#[sea_orm(
    rs_type = "String",
    db_type = "String(Some(1))",
    rename_all = "camelCase"
)]
pub enum Category {
    BigTask,
    SmallWork,
}

Using sea-orm 1.0.0-rc.5.


Add FTP protocol for external service entry

Describe the feature request
Add the FTP protocol to Service entry to allow file download from external FTP. FTP start the connection with TCP and then switch to UDP. I did find a way to allow two protocols on the same port.
Describe alternatives you've considered
Affected product area (please put an X in all that apply)
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ x] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Additional context

We don't yet support UDP so we are blocked by https://github.com/istio/istio/issues/1430

Closing as duplicate of https://github.com/istio/istio/issues/15972


[core] Remove default body from HaltException

The purpose of this PR is to remove the default body message (Execution halted) from the HaltException.
The message itself doesn't relay any sort of useful information outside of a debug environment. When short-circuiting logic i was often finding myself writing code to remove the default message, for example:
if (message == null) {
    throw new HaltException(HttpServletResponse.SC_NOT_FOUND, "");
}

If anyone has any objections, please post it.

I'd also consider changing the default value of status code to different one than 200.
I'd expect that if someone (or framework) throws halt exception, it halts for a reason that is abnormal to the execution flow.

Since it's not an unexpected error I prefer 400 over 500.  I'm okay with 400, I don't really have strong feelings on the matter.

The reason for me choosing 500 was to serve as a warning to the developer: 'Hey, you're short cirtuiting your logic without providing an appropriate response code, is this what you want?'. But i don't mind settling for 400.

Nah, no content is no content, not an unexpected halt, IMO.

Edit: Maybe we can use 204 (No Content)?

No, you can set a message and not a status, so that won't work. I think it either has to be 200 or 400.

4xx are client errors, meaning the client who used the API has performed an erroneous action.
5xx indicates that the server failed to fulfill it's promises, and that the responsibility is up to the developer of the application to fix.
A halt seems to me that the server wants to do something but is either hindered in some way or experiencing an issue.
For me it feels more right to use an 5xx as this is, while an intentional halt from the developer, not the happy path and something went wrong. My 10c :)

@pkkummermo
I find your thinking valid, but...
My point of view was rather that it focused on when would I ever want to halt.
In a program that is written correctly, unless client has provided a bad input, I usually do not have a need to halt the request. If I need to halt because there is something bad in my internal state, that usually is 'runtime in your face' 500.
If I need to halt because of other IO things, I have no controll over are doing something bad, then we have separate http status codes for that.
We could also remove this default value enitrely and there would be no confusion at all :) If you halt, you say why (or optionally you don't and you just rely on status code interpretation)

We could also remove this default value enitrely and there would be no confusion at all :)

That's probably the best suggestion seeing how everyone expects it to behave differently.

You can remove the default status too in this PR @HexRealm.

I've updated the PR removing the default status code, thank you everyone for your input.

Looks good, thanks!


Request to add NFS client to check the volumes mounted using NFS

Motivation-

To support the PR https://github.com/litmuschaos/litmus/pull/1248, Requesting a NFS client to continuously check the mounted volume using the NFS.

Approach

Install an alpine image with nfs-utils
using showmount -e NFS_SVC_IP, it will give the mounted volume list
Make a infinite loop to check volume from the mounted volume list
Exit on unable to find the volume from the volume list
Ref- https://github.com/litmuschaos/litmus/issues/1247#issuecomment-595630408

Repository- https://github.com/litmuschaos/test-tools

/assign @rajdas98
/reviewer: @ksatchit

PR merged
/close

@ksatchit Can you add this issue to the project board 1.2 ?


Visualize filters learned by GMMConv

How to visualize filters learned by GMMConv?
I have GMMConv layers:
self.conv1 = GMMConv(1, 16,dim=2, kernel_size=5,separate_gaussians=True)  self.conv2 = GMMConv(16, 1,dim=2, kernel_size=5,separate_gaussians=True)
Some questions:

Here, conv1 has 16  5X5 filters, right?
These filters should look like CNN filters if I am using a grid graph, right?
If I set separate_gaussians=False, does GMMConv act like isotropic GNNs?

Hi and thanks for your interest! The GMMConv is a bit different from classical CNNs since it will learn kernel_size Gaussians which define the kernel points (in fact, it's similar to attention based on edge features).
Those are accessible via self.mu and self.sigma`, and you can plot them similar to Figure 1 in the MoNet paper.
Therefore:

It will have 16 * 5 filters.
Not exactly, since kernel points are modelled as Gaussians and are trainable.
In case separate_gaussians=False, the Gaussians are shared across the features, so it will only learn 5 filters (which corresponds to the original MoNet formulation).


Hierarchical clustering with different min-seq-id values

Expected Behavior
I wish to have multiple cluster runs, each with a different value of min-seq-id value. Ideally, the resultant representative form hierarchical subsets, i.e. representative sequences for min-seq-id = 50 form a subset of those for min-seq-id = 90, and vice versa.
Current Behavior
Neither 1) runs of different min-seq-id nor 2) runs of the same min-seq-id agree.
Related Issue(s)
https://github.com/soedinglab/MMseqs2/issues/663

You can achieve this by manually cascading your clustering steps. We have implemented something like this a long time ago for Uniclust (https://github.com/milot-mirdita/uniclust-pipeline/blob/9b37347d1e9bb0153efcb986b5d828f9f4a316f0/uniclust_workflow.sh#LL67C7-L128C24). However, this script is not up-to-date, but the general strategy is sound.
Regarding the consistency itself, please refer to this answer:
https://github.com/soedinglab/MMseqs2/issues/115#issuecomment-423111623

I am unable to access the uniclust page (page not found). Would you mind sharing another link?

Sorry this one i meant:
https://github.com/soedinglab/uniclust-pipeline/blob/ec52afc312baab5178ce709caebb41bbae9bb3f0/uniclust_workflow.sh#L49-L133

You need to drop all instances of --max-seq-len and replace --target-cov 0.95 with -c 0.95 --cov-mode 1.
Lines 112-114 are quite expensive. I think that could be done smarter now, since we have the cluster reassignment mode in the Clustering workflow (which we didn't have back then).
You can replace that with (didn't try it out):
mmseqs cluster $INPUT "$TMPPATH/pref_step$STEP" "$TMPPATH/tmp_clu" --cluster-reassign 1 -c 0.8 --min-seq-id 0.3
$RUNNER mmseqs align $INPUT $INPUT "$TMPPATH/pref_step$STEP" "$TMPPATH/aln_step$STEP" -e inf -a

This issue is fixed in Git, but not part of a release yet. I think I fixed it shortly after the last release. You can download precompiled binaries at mmseqs.com/latest.
These 5 lines are intended to call the clust module which implements the actual clustering algorithm, not the whole cluster workflow with the searches etc:
mmseqs cluster $INPUT "$TMPPATH/aln_redundancy" "$TMPPATH/clu_redundancy" ${CLUSTER_FRAG_PAR}
mmseqs cluster $INPUT "$TMPPATH/aln_step$STEP" "$TMPPATH/clu_step$STEP" ${CLUSTER0_PAR}
mmseqs cluster $INPUT "$TMPPATH/aln_step$STEP" "$TMPPATH/clu_step$STEP" ${CLUSTER1_PAR}
mmseqs cluster $INPUT "$TMPPATH/aln_uniclust50" "$TMPPATH/clu_uniclust50" ${CLUSTER2_PAR}
mmseqs cluster $INPUT "$TMPPATH/aln_step$STEP" "$TMPPATH/clu_uniclust30" ${CLUSTER2_PAR}

should be
mmseqs clust $INPUT "$TMPPATH/aln_redundancy" "$TMPPATH/clu_redundancy" ${CLUSTER_FRAG_PAR}
mmseqs clust $INPUT "$TMPPATH/aln_step$STEP" "$TMPPATH/clu_step$STEP" ${CLUSTER0_PAR}
mmseqs clust $INPUT "$TMPPATH/aln_step$STEP" "$TMPPATH/clu_step$STEP" ${CLUSTER1_PAR}
mmseqs clust $INPUT "$TMPPATH/aln_uniclust50" "$TMPPATH/clu_uniclust50" ${CLUSTER2_PAR}
mmseqs clust $INPUT "$TMPPATH/aln_step$STEP" "$TMPPATH/clu_uniclust30" ${CLUSTER2_PAR}

For a 70% clustering you can adapt the 50% clustering strategy:
# cluster down to 70%
mmseqs filterdb "$TMPPATH/aln_step$STEP" "$TMPPATH/aln_uniclust70" \
    --filter-column 3 --filter-regex '(0\.[7-9][0-9]{2}|1\.000)'
mmseqs cluster $INPUT "$TMPPATH/aln_uniclust70" "$TMPPATH/clu_uniclust70" ${CLUSTER2_PAR}
mmseqs mergeclusters "${SEQUENCE_DB}" $OUTDIR/uniclust70_$RELEASE \
    "$TMPPATH/clu_frag" "$TMPPATH/clu_redundancy" $TMPPATH/clu_step0 $TMPPATH/clu_step1 $TMPPATH/clu_uniclust70

The idea here is to go directly from 90 to 30 to avoid any possible transitivity issues. In the MMseqs2 cascaded clustering you can get representative->member hits below the given thresholds after putting clusters from previous clustering steps together.
You can reconstruct the 50 (and a 70) clustering from the 30% alignments, and that's what it does (it does it with a roundabout regex, because we didn't have expression filters back then).

I built mmseqs from the latest Git and the issue is solved. There is now another warning.
there must be an error: 37 deleted from 4 that now is empty, but not assigned to a cluster
there must be an error: 49 deleted from 80 that now is empty, but not assigned to a cluster
there must be an error: 42 deleted from 52 that now is empty, but not assigned to a cluster
there must be an error: 42 deleted from 27 that now is empty, but not assigned to a cluster
there must be an error: 128 deleted from 120 that now is empty, but not assigned to a cluster
Total time: 0h 0m 0s 170ms

Interestingly it only happens at 50% level. Not 30% or 70%. Let me know if you identify it as a problem.

Another issue I notice is that not all representative sequences in 30% are found in 50% or 70%, i.e. the clustering is not "hierarchical".
I added the following lines to extract the representative sequences from the clustered.
mmseqs createsubdb $OUTDIR/uniclust50_${RELEASE} $SEQUENCE_DB $OUTDIR/uniclust50_${RELEASE}_rep
mmseqs convert2fasta $OUTDIR/uniclust50_${RELEASE}_rep uniclust50_rep.fasta

One more question. Which part of the script should be changed if I am interested in doing the same with foldseek?

One more question. Which part of the script should be changed if I am interested in doing the same with foldseek?

Did you solve the issue with the clustering not being hierarchical? This should produce a hierarchical clustering.
Foldseek is quite a bit more tricky since it uses different module will need a lot of parameter changes.

Yes, I am still experiencing the same issue, i.e. some sequences in clust30 do not show up in clust50 / clust70.


Stat Randomizer

EventHandler will call statRandomizer() to shuffle a total of 8 extra ability points between the 6 ability categories:
Strength
Dexterity
Consitituion
Intelligence
Wisdom
Charisma.
Plan of improvement:  will be to start with 8 extra points and move on to the standard 27.

Implemented the Dice roll calculation for stat generation.


[MobileNerf] Integrating result to unity or omniverse

First, my thanks for releasing the MobileNerf code.
This is regarding the polygon rendering method in the paper. Does this also mean that these results can be directly integrated to tools like Unity or Omniverse? Traditionally, the Nerf results are first converted to meshes and then can probably be imported but the result of the mesh is often terrible.

Yes. Some people have already implemented that. For example, https://github.com/julienkay/MobileNeRF-Unity-Viewer


Revert protocol generator using reflection/extension to ensure generation is simple and consistent

This move back to single protocol generation, with few changes:

Instead of multiple parameters, one json configuration file to drive the generation
This file, now helps define dependency between modules, since the generator can compute what classes the other modules generated, ensuring proper imports, etc
For ElasticSearch, I kept a custom generator until I can fully unwind the requirements, and absorb them in the standard generator.

/easyCLA


gitian: invalid option: --detach-sign

Generating report
f0f58a3c4f262cd5d8ba91774f0c0ad643ca83c14a6a4991918f8a02c87fa483  bitcoin-0.16.0-aarch64-linux-gnu-debug.tar.gz
...
b801f1eeeb3ad4aeb7f1b58b91b8a382fe138399223430d8de4b790a4452918d  bitcoin-linux-0.16-res.yml
Done.
./bin/gsign:52:in `<main>': invalid option: --detach-sign (OptionParser::InvalidOption)

I got the same error last summer. The correct syntax on Debian seems to be gsign -p true

That was with Debian 9. Not sure if related, but I don't get this error on Debian 8 with:
./gitian-build.sh --setup --detach-sign --no-commit -b sjors 0.16.0rc2

+1 for a Python rewrite, ideally with lots of checks for missing dependencies, misconfigured routes, etc.

Let me know if this is still an issue with the most recent version of gitian-build.py


Add blacklist commands for keywords

Prompted by this: http://chat.stackexchange.com/transcript/message/33655049#33655049
I (and bwDraco) think that it would be great if we could implement a !!/blacklist-keyword command or similar.
Since we already have the !!/blacklist command implemented for websites, and the bad keyword regexes have already been separated into a text file, it should be fairly easy. All the infrastructure is there; we just need to copy/paste some code and change a few things.
While we're doing this, it might also be a good idea to rename the existing !!/blacklist command to !!/blacklist-website. I've noticed that a few people recently have misunderstood the use of the blacklist command, and changing its name should make that clear.
What are your thoughts?
If we reach an agreement that this would be a good idea, I could do this on the weekend (as long as no-one beats me to it).

I second this.
To build on this, I'd also like to see a !!/blacklist-username command as well. A bit earlier (see #295), I had to manually fork the repo, edit the code, and submit a pull request in order to get a username added to the blacklist.

I third this.
With that said, we've had some discussion in CHQ on this topic. Consensus seems to be that we can move plaintext (or close to plaintext) lists to resource files, but regex patterns should stay in the code (such as pattern matching websites).

@ArtOfCode- Yeah, those were my thoughts. I originally was going to propose to do it for most blacklists, but then I realised that keywords and usernames were about the only other appropriate ones to move out into other files.
And I agree with the idea that regexes should stay in code while plaintext can be moved into resource files.

In 21 days, 5 commits, 2 of which involved no change (ended up being reverted). With this level of activity...

With this level of activity, it's easy to forget what you're doing with the code, so adding a new robust command to do it for is a great idea! Just what I was thinking, @hichris1234.

It's easy to forget how to add a line to a text file? Unfortunate times we live in...

Oh, definitely. I mean you've gotta remember which file it is, for a start, and our consistent naming schemes make that really difficult. And then you've gotta remember where the newline goes! And don't forget actually typing, who knows how to do that these days?

I was going so start work on this, but I ran into an obstacle that I need some input on.
Often, a blacklisted username or keyword will actually be a blacklisted phrase, where there are spaces in the 'keyword' or username. However, our current system for chat commands separates arguments by spaces, and hence if a keyword/username were to have a space in it, it would be considered to be two separate keywords/usernames.
So is there a way to get around this?

Modify this line somehow so that it treats a space preceded by \ as escaped, and hence does not split the command at that point
Use a different character in place of spaces, then swap it out with a space behind the scenes?
Use " ".join(message_parts) to make the command single-argument only?

Does anyone have any other ideas? Or will one of the ones above be sufficient?

@angussidney Regexes can use the \s character for spaces. I'd say use the same system as blacklisted websites - split on space for multiple arguments, and blacklist entries that need spaces can use \s.


Fix compare bug + alignment

A previous bug fix to the compare modal introduced a new one. When the pinned row is ranked #1, no other locations are populating the modal (image below). This fixes.

Also, alignment of the compare filters looked a bit off. This addresses that.
Before:

After:

Testing Notes
1. The pinned locations shows up twice, once pinned and once in the list below.

Go to https://covid-projections-git-casulin-fix-compare-modal-bug.covidactnow.vercel.app/us/georgia-ga/county/taliaferro_county/?s=1602544
Sort by population ascending
Open the modal

this rocks! very thorough 🙏🏼
to each point:
1- this is the intended behavior. the location should only be removed from the list if it ranks #1
2- this is a bug. going to fix in a separate PR.
3- also intended behavior (same as point #1)
4- hm yeah. the rank is 0 because we are filtering for metro counties, and the current/pinned county isn't a metro county. instead of 0, it should probably read --. will fix in a separate pr.


fix(html/parser): potential bug in spec

Description:
I found a bug in HTML spec, so I started to fix it in spec and our parser + tests for html5tests-lib for other parser
Original issue - https://github.com/whatwg/html/issues/7922
So we are showring potential fix in spec/parser here (and no problems with other tests). Let's merge when HTML spec marked it as bug and fix it in spec.
BREAKING CHANGE:
No
Related issue (if exists):
Ref: https://github.com/whatwg/html/issues/7922
Ref: https://github.com/html5lib/html5lib-tests/pull/150

@kdy1 Ready to merge, the bug was fixed in spec


Remove Telekinesis

Telekinesis is no longer an enchant so.... byeee 👋

https://www.youtube.com/watch?v=LB05rma3wto note: thanks for making me spend a hour on this

Lmaoooo 🤣 🤣 🤣 🤣


fix typo

By analyzing the blame information on this pull request, we identified @michalgr to be a potential reviewer.

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign up at https://code.facebook.com/cla - and if you have received this in error or have any questions, please drop us a line at<EMAIL_ADDRESS>Thanks!

@facebook-github-bot import

@facebook-github-bot shipit

I cannot do that @hramos, because @king6cong does not have an active CLA on file.

I cannot do that @hramos, because @king6cong does not have an active CLA on file.

CLA signed up

Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!

@facebook-github-bot shipit

Thanks for importing.If you are an FB employee go to Phabricator to review internal test results.


Can we use only vue-charist with only Series data?

Coud I use this library with using only 'series' data?
as Like 'http://gionkunz.github.io/chartist-js/getting-started.html#switching-axis-type'
(Please see the code)
in this code,
  series: [[
    {x: 1, y: 100},
    {x: 2, y: 50},
    {x: 3, y: 25},
    {x: 5, y: 12.5},
    {x: 8, y: 6.25}
  ]]
}, {
  axisX: {
    type: Chartist.AutoScaleAxis,
    onlyInteger: true
  }
});```

There is no label info in array, but presented in Series array as 'x' coloumn.

Fix to the next release


Question about the split

Hi! Thank you for your simple but great job! I see that in your code you use the same split as U2PL. But since many works such as UniMatch, iMAS, AugSeg find the split used in U2PL is unfair and reproduce U2PL, have you tried to reproduce your work with the split from UniMatch? Using the split from UniMatch will naturally boost the mIOU performance. Also, is directly comparing your work with UniMatch but with different splits a little bit unfair? Looking forward to your reply. Thanks!

This work use the same split of U2PL, a new version of the code is uploaded and you can find it in the arxiv version of the paper in few days

Please refer to oficcial repo https://github.com/shaiTheKimhi/s4mc


[MM-47226]: fixed a bug with currentTeamId not being set on mount

Summary
fixed bug where returning to the app from system_console broke the app
Additionally fixed some import sorting issues eslint was complaining about
Ticket Link
MM-47226
Related Pull Requests
reverting some changes made in #11135
Screenshots
n/a
Release Note
NONE

E2E tests not automatically triggered, because PR has no approval yet. Please ask a developer to review and then try again to attach the QA label.

@michelengelen There are some failing tests here.

/update-branch

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.

Trying to auto merge this PR.

Pull Request successfully merged
SHA: d51758ddaec2f3d42dba28227575540b6512f98c

Cherry pick is scheduled.


"share" operator

I think the answer is no, but I would I like to confirm. Is there any combination of available operators that would result in the same behaviour as the share operator?
If not, I'll send a PR

Not available as a operator itself, but can by done by using observable.multicast().refCount(), see test here: https://github.com/renggli/dart-rx/blob/68bddf39dc44e7cbcf058cef4064f2a075d09e17/test/operators_test.dart#L1719


Replace StringCollection with generic List

PR Summary

Replace StringCollection with List<string>.
Replace StringCollection with string[] when length is known.
Rename variables for clarity.
Refactor GenerateMultiLineRowField as static method.
Cache loop invariant integers.

Changes not made in the following:
src\System.Management.Automation\engine\LanguagePrimitives.cs: complex call hierarchy
src\Microsoft.PowerShell.Commands.Diagnostics\CounterSet.cs: parameter in public API
PR Context
Performance using the generic List<string> is much better than using the non-generic StringCollection.
See https://github.com/PowerShell/PowerShell/pull/13425#issuecomment-673180364 for a benchmark.
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[x] None
OR
[NA] Experimental feature(s) needed

[NA] Experimental feature name(s):

User-facing changes

[x] Not Applicable
OR
[NA] Documentation needed

[NA] Issue filed:

Testing - New and feature

[x] N/A or can only be tested interactively
OR
[NA] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[x] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[NA] I have considered the user experience from a tooling perspective and enumerated concerns in the summary. This may include:

Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode (which runs in a different PS Host).
Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.
Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).
Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

@rjmholt Test are failing only Windows, can you restart?

@rjmholt Could we make changes in LanguagePrimitives.cs?

Existing formatting was left unchanged, hence Codacy issues.

@iSazonov can you approve?

rebased to remove merge commit

@xtqqczze Please resolve merge conflicts.

@xtqqczze I think it is better to split the PR to some small PRs by cmdlet/folder to get more fast progress.

Please look CI fails.

Replace StringCollection with List<string>.
Replace StringCollection with string[] when length is known.
Rename variables for clarity.
Refactor GenerateMultiLineRowField as static method.
Cache loop invariant integers.

and maybe split this by kind of changes since there is a risk of regression.


HTTP: Redirected request is accounted twice in cumulatedResponseTime

I checked the operation with the latest source(master branche at 2017/11/20)
I set FollowRedirect is enabled and
define a scenario consisting only of groups containing one request.
(group name is　'group1', request name is 'request1' )
After receiving the response of request 1, request 'request1 Redirect 1' is generate.
I execute this simulation and expect the following results.
If 'request1' response time is A and  'request1 Redirect 1' response time is B,
'group1' cumulatedResponseTime is A+B.
But cumulatedResponseTime is A+A+B.
The whole file I used is below.
It is necessary to change the proxy settings to use.
CD.scala.txt

Thanks for reporting!


How to set env variables while testing a groovy class.

class SampleClass implements Serializable{
	private def steps
	private def readObj

	SampleClass(def steps){
		this.steps = steps
	}

	void sampleMethod(String path){
		if(steps.env.NODE_NAME){
			steps.println("Jenkins env.NODE_NAME is set so it is inside a node")
			readObj = readYaml(file: path)
		}
	}
}

This is how I am testing:
class SampleTesting extends JenkinsPipelineSpecification{

	SampleClass obj

	void setup(){
     obj = new SampleClass(this)
 	}

	void "Testing sampleMethod"(){

		when:
		obj.sampleMethod("/tmp/xyz.yaml")

		then:
		1 * getPipelineMock("readYaml")("/tmp/xyz.yaml") >> {
			def map = [name: 'Gromit', likes: 'cheese', id: 1234] as LinkedHashMap
			return  map
		}

	}
}

This test will always fail as I can't find a way to set env variable. The only way avaiable is using loadPipelineScriptForTest()
def s = loadPipelineScriptForTest("path/to/pipelineScript")
s.getBinding().setVariable("env", [ NODE_NAME: "testNode" ])

but this is a groovy class and we cannot use loadPipelineScriptForTest()
Is there any way I can set env variable in such cases?

SampleClass expects its steps member variable to have a property defined named env.
Why not just
void setup() {
	steps = [
		"env": [
			"NODE_NAME": "value"
		]
	]
	obj = new SampleClass( steps )
}

Beyond that, though, I might suggest refactoring SampleClass and/or sampleMethod so that they don't rely on the env that Jenkins creates. I note that readYaml is also a pipeline step, and so the "Groovy Class" cannot really stand on its own outside of a Jenkins pipeline.
Is it really appropriate to put that code in a standalone Groovy class? You could use a Shared Library Global Variable instead. If you did that, the existing examples for setting env with loadPipelineScriptForTest(...) would work properly.

I agree that the groovy classes should be independent of the Jenkins pipeline step, but that will involve too much refactoring to get the test running. Current I am doing the following to get pass this issue.
getPipelineMock("env.getProperty")("NODE_NAME") >> "testNode"

So the whole test case looks like:
class SampleTesting extends JenkinsPipelineSpecification{

	SampleClass obj

	void setup(){
     obj = new SampleClass(this)
     getPipelineMock("env.getProperty")("NODE_NAME") >> "testNode"
 	}

	void "Testing sampleMethod"(){

		when:
		obj.sampleMethod("/tmp/xyz.yaml")

		then:
		1 * getPipelineMock("readYaml")("/tmp/xyz.yaml") >> {
			def map = [name: 'Gromit', likes: 'cheese', id: 1234] as LinkedHashMap
			return  map
		}

	}
}

This works :)
Thanks for all the help.
Share your thoughts, I will close this issue on Monday

SampleClass expects its steps member variable to have a property defined named env.
Why not just
void setup() {
	steps = [
		"env": [
			"NODE_NAME": "value"
		]
	]
	obj = new SampleClass( steps )
}

Beyond that, though, I might suggest refactoring SampleClass and/or sampleMethod so that they don't rely on the env that Jenkins creates. I note that readYaml is also a pipeline step, and so the "Groovy Class" cannot really stand on its own outside of a Jenkins pipeline.
Is it really appropriate to put that code in a standalone Groovy class? You could use a Shared Library Global Variable instead. If you did that, the existing examples for setting env with loadPipelineScriptForTest(...) would work properly.

If I do
steps = [
		"env": [
			"NODE_NAME": "value"
		]
	]
	obj = new SampleClass( steps )

this won't work
1 * getPipelineMock("readYaml")("/tmp/xyz.yaml") >> {
			def map = [name: 'Gromit', likes: 'cheese', id: 1234] as LinkedHashMap
			return  map
		}

I need to pass this while creating object
obj = new SampleClass(this)

In Jenkins, this is probably the Pipeline script execution.
In jenkins-spock, this is probably the JenkinsPipelineSpecification test suite.
In actual use, you're passing the this into your Groovy class so that the class can call pipeline steps, right? This happens to work because even though the JenkinsPipelineSpecification isn't a running Jenkins Pipeline script, it catches the attempts to call pipeline steps (like steps.println) and forwards them to the correct mocks.
It might be more ideologically correct to prepare additional objects - in this case, a dummy object could stand in for the Jenkins Pipeline. Something like:
def setup() {
    def mockPipeline = new Object()
    addPipelineMocksToObjects( mockPipeline )
    obj = new SampleClass( mockPipeline )
}

The end result should be identical, but you're explicitly creating a dummy/stand-in for the running Jenkins Pipeline script.
I'm not really sure what 'best practice' would really be here, though: no one's asked this specific question about this specific situation before.
For the future, I'd recommend putting Pipeline of code in a shared library Global Variable, and keeping Groovy classes "pure groovy" without any Jenkins-specific dependencies.

Thanks for the quick response. Yup, it works the same way. I will go ahead with the one you suggested.


Assert revert with specific assert error

resolve #209

Codecov Report

Merging #259 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #259   +/-   ##
=======================================
  Coverage   97.41%   97.41%
=======================================
  Files          81       81
  Lines        3129     3129
  Branches       20       20
=======================================
  Hits         3048     3048
  Misses         78       78
  Partials        3        3

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ece48e9...b89469d. Read the comment docs.


Support storage and access of normal file

Sometimes I need to upload lots of voice files instead of images. Hope it could support normal files besides images.

This feature is belong to File Storage System and zimg just deal images.
I'm doing something in developing a file storage system which support images, videos and any other object files' storage. You can wait this project.

refer to #60


systemd.unit.create: Add Environment and EnvironmentFile options (HMS-3814)

Support the Environment and EnvironmentFile options in the Service section of the unit file.
Updated the test to include an empty file in the a.json (before file) so that we get content hashes in the diff instead of just "added files".

Can we make the linked issue HMS-3814 public? Is there more information in the issue than is mentioned in this PR?

I made the new property an object  {key: value} and added the same property schema as an alternative to the existing org.osbuild.systemd.unit stage (which creates drop-in files).

Rebased and resolved conflicts.

Tests are failing because of the new options, but I see they were added to the schema json so I'm not sure why.
"message": "Additional properties are not allowed ('Environment', 'EnvironmentFile' were unexpected)",


Passing multiple child components to a component under test in C# based testing

Inspired by #139, we should make sure that it is possible to do the following using the component parameter factory methods or the component parameter builder:
<Parent>
    <Child/>
    <Child/>
</Parent>

E.g. call RenderComponent<Parent>(...) and pass two or more child components into its ChildContent parameter.
Closes #139

Hello @egil, I'm wondering if is this now possible or is it a work in progress?
So far I have only used C# based testing and this will help me with a current test case.
Btw, congrats for you nice library, it's very helpful.

Hi @eliudgerardo
It's work in progress. Should land in the next beta, and hopefully in the nightlies in a week or so.
You can follow along in the in #203.

Fix available in the latest nightly available on GPR. See https://github.com/egil/bUnit/discussions/209 for guide on how to test it.


User-defined types in MPI-CUDA

User-defined types with no array members should now work fine in the develop branch. Array members still remain to be fixed, and moved to issue #16.


Add token scope(s) required in function help

Summary of the new feature/enhancement
Referencing the Venafi docs for the appropriate token scope for each function is cumbersome.  It is also difficult since it's not necessarily known which api is called.  One could use -verbose to get the api and then look it up, but not ideal.  Also, many functions make multiple api calls so each one would need to be looked up.  This function help enhancement is to add the token scope(s) required to execute each function.  The info is most likely best in the description.
Proposed technical implementation details (optional)

This!
I find myself playing the "API Scope" whack-a-mole game waaaaay too often.


why the performance is different from the paper ?

Hello,
Thank you for your work
I have a question:
why the performance is different from the paper ?, for example in paper table Table 5: Comparison with static type inference tools
when using the static part only:
the performance for return value HiTyper 0.86, but in GitHub  Return Values 0.69

Thanks for your interest!
This is because the two "ExactMarch" scores are different.
If you read the part discussing Table 5 in the paper, you will know the "ExactMatch" score in Table 5 represent the precision, and that's why there is another column named "number of correct annotations" to approximate recall. 0.86 means that 86% of HiTyper results match the GT. We did not use the "Exact Match" as a recall in the paper because static analysis tools can easily fail to infer types.
However,  the "ExactMatch" score used in README represents the recall. 0.69 means that HiTyper can correctly infer 69% of return values in the dataset. That's why the score of arguments is very low, and the score of return values is different from the paper.

why in READM you mention that
"Old results:
If you want the exact experiment results stated in the paper, please download them at this [link]."
Do you mean the results are different from paper when we use the whole tool(DL+static part)? why two results old and new?

This is because:

This repo is actively maintained, and the code is frequently changed.
The benchmark dataset is also evolving. The benchmark dataset used in the paper is updated now.
Therefore, we do not provide any guarantee that the tool now can yield the same results as in the paper.
We do not recommend you directly copy the results in the paper without actually implementing the tool. If you want to compare with HiTyper, just run it on your own dataset to acquire the latest results. However, we still provide the original results in case you want to directly use them.

I was just asking out of curiosity, thanks for answer


Cache clipId when instantiated

...rather than calculate a new id on each render.

@imranolas LGTM. Merging.


I can't submit the kata whatever I do

When I run the code everything runs fine, but it just goes until it times out on purpose and then the submit button doesn't show up
To Reproduce
def is_square(n):
if(n == 0):
    return True
elif(n < 0):
    return False
for num in range(1, n):
    if(n/num == num):
        return True
    elif(num == int(n/2)):
        return False

Expected behavior
It should give true or false for if the number is a perfect square
Screenshots
If applicable, add screenshots to help explain your problem.

Browser
Tried on both Chrome and Edge (newest versions)

OS: [e.g. macOS]
Browser [e.g. chrome, firefox, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Your code is hugely inefficient, and submit tests rightly time it out. This is not a Codewars issue, it's ( at best ) a kata issue, which you can raise on the discourse page of the kata itself.


Sunset Subversion support in GitHub Enterprise Server

Summary
As announced last year, Subversion support will be turned off in GitHub Enterprise Server 3.13, which is tentatively scheduled for release in June 2024.
Subversion support was removed from GitHub.com on January 8, 2024.
Intended Outcome
Subversion support is little-used by GitHub customers (in aggregate) but represents an above-average ongoing support and maintenance burden. Removing it will free up resources to focus on features and support that is more relevant to the vast majority of our customers.

This is complete with the release of GHES 3.13 on June 18, 2024.


x/net/html: removing first child node of pre causes doubling of newline at beginning of following text node

Go version
go version go1.21.4 darwin/amd64
What operating system and processor architecture are you using (go env)?
GO111MODULE='on'
GOARCH='amd64'
GOBIN=''
GOCACHE='/Users/matloob/Library/Caches/go-build'
GOENV='/Users/matloob/Library/Application Support/go/env'
GOEXE=''
GOEXPERIMENT=''
GOFLAGS=''
GOHOSTARCH='amd64'
GOHOSTOS='darwin'
GOINSECURE=''
GOMODCACHE='/Users/matloob/go/pkg/mod'
GONOPROXY=''
GONOSUMDB=''
GOOS='darwin'
GOPATH='/Users/matloob/go'
GOPRIVATE=''
GOPROXY='https://proxy.golang.org,direct'
GOROOT='/usr/local/go'
GOSUMDB='sum.golang.org'
GOTMPDIR=''
GOTOOLCHAIN='auto'
GOTOOLDIR='/usr/local/go/pkg/tool/darwin_amd64'
GOVCS=''
GOVERSION='go1.21.4'
GCCGO='gccgo'
GOAMD64='v1'
AR='ar'
CC='clang'
CXX='clang++'
CGO_ENABLED='1'
GOMOD='/Users/matloob/pkgsite/go.mod'
GOWORK=''
CGO_CFLAGS='-O2 -g'
CGO_CPPFLAGS=''
CGO_CXXFLAGS='-O2 -g'
CGO_FFLAGS='-O2 -g'
CGO_LDFLAGS='-O2 -g'
PKG_CONFIG='pkg-config'
GOGCCFLAGS='-fPIC -arch x86_64 -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -ffile-prefix-map=/var/folders/qb/r47yxsb14sx6nz0m6_1pglm8004_sl/T/go-build411072747=/tmp/go-build -gno-record-gcc-switches -fno-common'

What did you do?
I ran the following program: https://go.dev/play/p/ywoeY3SDD87 pasted below
package main

import (
	"bytes"
	"fmt"
	"log"
	"strings"

	"golang.org/x/net/html"
)

func main() {
	doc, err := html.Parse(strings.NewReader("
\ntext\n
"))
	if err != nil {
		log.Fatal(err)
	}
	pre := doc.FirstChild.LastChild.FirstChild

	fmt.Println("before:")
	var buf bytes.Buffer
	html.Render(&buf, pre)
	fmt.Println(buf.String())

	pre.RemoveChild(pre.FirstChild) // Remove the  tag

	fmt.Println("after:")
	buf.Reset()
	html.Render(&buf, pre)
	fmt.Println(buf.String())
}

What did you expect to see?

text

after:

text

What did you see instead?

text

after:

text

CC @neild, @ianlancetaylor.

CC @nigeltao if you are interested

Yeah, in the "after" case, inserting the newline is deliberate (Working As Intended):
https://github.com/golang/net/blob/08a78b1eeae5f15e658ca8972aa74b6857e3b37b/html/render.go#L187-L195
If you delete those lines of code, the tests will fail. For example, this WebKit test case should produce a "\nfoo" text node with exactly one leading '\n', even though the input has two.

Per the WHATWG HTML spec (see: A start tag whose tag name is one of: "pre", "listing"), a '\n' immediately after a
 would be ignored. In your example code, to keep the Text Node's text being "\ntext\n" (with a leading '\n'), the renderer inserts an extra one.


Add safety when receiving meta deltas

Otherwise the plugin may crash with:
TypeError: Cannot read properties of undefined (reading 'forEach')

Thank you for contributing @bergie


Meta: Allow setting local dev team

Fixes #5202
Partially undoes https://github.com/refined-github/refined-github/commit/a79f49bdb7e571bd8402f3cddba8154220bd0e0c

You should now be able to create the file safari/LocalOverrides.xcconfig with the content:
CODE_SIGN_IDENTITY = "Apple Development";
DEVELOPMENT_TEAM = YG56YK5RN5

Untested except on CI, does it work for you?

MARKETING_VERSION can be set by a custom version script inside an additional GeneratedOverrides.xcconfig file

Yes, it would be nice to not have to duplicate this.

CURRENT_PROJECT_VERSION I think Xcode takes care of this? I think I've seen it automatically bump this number before

Not as far as I know. Xcode has a CLI tool to bump it.
We could maybe set the build version based on commit count or tag count (+ padding to be higher than existing number).
https://gist.github.com/dimitribouniol/24f0406e0878e5880f3a64dea3c528b6

This works, but not quotes on the code sign identity:
CODE_SIGN_IDENTITY = Apple Development
DEVELOPMENT_TEAM = YG56YK5RN5


Introduce richtext-markdown as a generic Markdown renderer

Markdown unfortunately does not have a universally agreed standard. Different flavors, libraries can parse Markdown content in their own ways. richtext-commonmark uses Commonmark spec and parser to convert any given markdown String into an intermediate Abstract Syntax Tree that has been defined internally since its release.
The idea has always been to extract this ASTNode structure out of this module to enable custom parsing logic to be supplied from outside, and make richtext-commonmark only responsible for rendering the given tree using richtext-ui.
This change introduces a new module named richtext-markdown which is actually just a rename of the existing richtext-commonmark. This new module also exposes its internal AstNode APIs to enable other modules to render their own AstNode structures.
richtext-commonmark becomes a helper module which can parse any given string into an AstNode via CommonmarkAstNodeParser. Also, it still provides a RichTextScope.Markdown function that simply does the conversion asynchronously, then calls the Markdown(AstNode) flavor from richtext-markdown.

#123


add tutorial for Bedrock Guardrails

Description
[Describe what this change achieves]
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

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

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

Suggest also add how to use the guardrail model for input_guardrail output_guardrail, like this test example
POST /_plugins/_ml/models/_register?deploy=true
{
    "name": "Bedrock Claude V2 model with openai guardrails model",
    "function_name": "remote",
    "model_group_id": "ppSmpo8Bi-GZ0tf1i7cD",
    "description": "Bedrock Claude V2 model with openai guardrails model",
    "connector_id": "xnJjDZABNFJeYR3IPvTO",
    "guardrails": {
        "input_guardrail": {
            "model_id": "o3JaDZABNFJeYR3I2fRV",
            "response_accept": "^\\s*\"[Aa]ccept\"\\s*$"
        },
        "output_guardrail": {
            "model_id": "o3JaDZABNFJeYR3I2fRV",
            "response_accept": "^\\s*\"[Aa]ccept\"\\s*$"
        },
        "type": "model"
    }
}

@kolchfa-aws , could you help have another review?


No development setup

Hi there,
Currently there is no local development setup in place. This is something I need to add on my end, and I would be happy to contribute back.
I intend to add:

docker compose file to run a local database
some commands to easily bring it up as required and migrate it
test setup allowing running tests against a database (using https://testcontainers.com/)
github workflows for running linting and unit tests
integration of test coverage and code analytics using (https://deepsource.com/)

I might also run e2e tests later, but that is out of scope for me at first.
Tell me if anything is relevant and I will add it as a PR here first and sync

Hi @Goldziher!
Thank you very much for proposing to add new features. I appreciate your willingness to contribute. However, I prefer to keep the repository simple and accessible for all types of users, updating it only as needed.
That said, you can fork the repository and add these features on your side. I'd be happy to provide a link in the readme/documentation to your repository for those who are interested.
Thanks again for your contribution!


[Docs] README revamp

This PR revamps README.md for better clarity. Revamp of project landing page will follow up soon.

@junrushao Thanks for the comments! Updated and please take another look :-)


⚠ Processors has degraded performance

In bc6dedd, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 135 ms

Resolved: Processors performance has improved in 85a259f.


Error：Tuple from_py overload match problem

The following code snippet will result in a compilation error
@python
def t3() -> tuple[pyobj, pyobj, pyobj]:
    return (1, 2, 3)

@python
def t2() -> tuple[pyobj, pyobj]:
    return (1, 2)

@python
def t33() -> tuple[pyobj, pyobj, pyobj]:
    return (1, 3, 5)

def test1(a, b, c):
    return a + b + c

def test2(a, b):
    return a + b

print(test1(*t3()))
print(test2(*t2()))
print(test1(*t33()))

test_py_dec.py:10:1-41: error: 'Tuple[pyobj,pyobj,pyobj]' does not match expected type 'Tuple[T1,T2]'
╰─ test_py_dec.py:21:14-17: error: during the realization of t33()

Hi @victor3d
Thank you for reporting this---looks like a typechecker bug. Will need to check what is going on here.

Fixed in #604.


Setting Timezone

How can I set the Timezone, for both ELK and the Extensions eg: Appsearch?

Some context would help, but the short answer is: you can not and should not. Servers use UTC by convention, and if you need to show a date in your local timezone you should do it on the client side (e.g. Kibana allows you to define your local timezone).

Closing due to inactivity.


clear is not a function

I have trying to rebuild my current JS implementation to typescript.
Now i have faced an issue on the world

If i am not mistaking this is referencing to the following line:
https://github.com/3d-dice/dice-box/blob/759b885544cb11291583ca03220b155187c92937/src/WorldFacade.js#L369
I need to be honest that i wasn't able to understand the code at this place.
My hope was that you have an instant idea.

Arg.... the issue was simple. It seems the world was not initialized due to a race condition.

Sorry to reopen this, but how did you fix this problem?
I'm getting this error as well

@marvink I have the same problem, do you remember the exact solve?


Better Deploy exception detection throwing and/or logging

Issue type
[X] Feature request
[ ] Bug report
[ ] Documentation

Environment

Python version:  3.5.3
Peering Manager version: v0.99-dev

Description
Deploy should be able to pull errors from napalm, like discern if the config is locked (I think this is returned by napalm?) or if the device is unreachable (I think this too is returned by napalm?) or if an error is thrown during the deploy.
I just tried to deploy a config that I knew would introduce changes to the config, but upon completion, it reported there were no differences in the config. After reviewing the logs, I saw there was in fact an issue that PM didn't log or throw an exception for:
2018-10-28 12:04:14 | DEBUG | unable to merge configuration on bfr01.1250ReneLevesqueBl01.YUL reason "Invalid input entered:

    maximum-prefix 0 95 restart 60
                   ^
% Invalid input detected at '^' marker.
"
2018-10-28 12:04:14 | DEBUG | closing connection with bfr01.1250ReneLevesqueBl01.YUL

Indeed that's quite an interesting topic that you've just raised. Actually this is a topic I've been working on when releasing the code to apply the configuration using NAPALM.
NAPALM does raise exceptions when trying to merge a configuration into another. What it does is that if there are any kind of issue we expect NAPALM to raise a MergeConfigException. We then have to inspect the message of the said exception to know what actually happened.
The code handling this in Peering Manager is here. Changing the set_napalm_configuration return value could help knowing if an actual error occurred or if it was changed that no changes had to be done. Yeah the current code returns a value interpreted as a boolean equaled to false if no changes have been applied (meaning even in case of an error).
Changing the return value of the function using maybe a tuple could be an idea like: return (success, message). success would be a boolean value telling if the operation was successful. message could be used as a variable to carry, either the changes to be applied or the actual error message.


Fix typo

java -> Java

@changeworld : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Thanks for your dedication to our documentation. This PR has been triaged and approved.

#sign-off


GOENV_GOPATH_PREFIX does not work as expected

Hi there, I'm running the HEAD version of goenv and the env var GOENV_GOPATH_PREFIX does not seem to work. More specifically, the env var is not set, goenv will automatically manage and update the GOPATH to be $HOME/go/$VERSION however, if the env var is set the GOPATH is set to the GOENV_GOPATH_PREFIX only and the version is not included.
I can't figure out if it's a bug or something misconfigured in my system. Any idea?
Thanks a lot for your help.

Should be working as described in https://github.com/syndbg/goenv/blob/master/ENVIRONMENT_VARIABLES.md#environment-variables

Also, maybe unrelated but, is it normal that when I execute go local I then need to reopen a new shell session to have the right version of go, and then manually re-execute goenv rehash to have GOPATH set properly?

@syndbg sorry for pinging you but I was wondering if you saw my previous message?

@debo I would suggest closing this as a dupe of #209 -- looks like other people are being bit by this as well.