Sass (and no CSS)

Why u no sass?
Even AMP shows GROW to uses Sass - and people doing this level of design have their favorite transpiler.
So people have to modify your work.

And he links why http://github.com/markbirbeck/mdl-sass-mixins

Google: http://github.com/google/web-starter-kit/blob/master/docs/mdl-sass.md

etc.

If you just release Sass mixins for us to BEM.
Now, each of has to do that effort by self.

We understand this is an issue, and have rearchitected our codebaze for v2 to make it much easier to support this use case. I encourage you to go check out the packages/ directory on the master branch, where you'll find individual pluggable sass files for every component, and whose sass dependencies are all installed via npm. We'll have more instructions on Sass integration as we get closer to alpha.
As for v1, as we mention in our issue template we will not be making any major changes to that codebase since we are actively working towards v2.

Thx Travis. We do not want components for @extend ('is a')
I mean composition ('has a') via mixins.
Even the name of this project is 'lite' and then you do heavy even in v2.
A few pages of examples of what we want:
http://google.com/#q=sass+mixin+library
Composition not inheritance. 'Has a', not 'is a'.
Right now when I do a AMP page via @extend Sass it's huge.
If you switch to mix in, it be only a few lines generated in AMP.
Anyway, that is best practice, I either go heavy or we all have to make our own mixins.

The goal of Material Design Lite is to provide a library of components delineated in the Material Design Spec for the web platform. The goal is not to provide a sass mixin library. If you're looking for that then as you mentioned you may want to go with another solution like Bourbon or Compass.


Influx query language improvements

I'm trying to get better support into bosun, and for this I'm trying out different examples from the documentation page and adapting them to the influx() function.
Given the os.diskspace alert, we have this opentsdb query:
q("avg:6h-avg:os.disk.fs.percent_free{host=netlog*,disk=*}", "7d", "")

I think I have a similar query to get the data from influxdb, correct me if i'm wrong:
select mean(value) from "os.disk.fs.percent_free" WHERE host =~ /netlog.*/ and time > now() - 7d group by time(6h), host, disk

The current problem:

influx() wants the time outside the query, or it will complain with

  influx($db, '''select mean(value) from series WHERE host =~ /netlog.*/ group by time(6h), host, disk''', "7d", "")

=> aggregate functions with GROUP BY time require a WHERE time clause
Adding it, will complain too:
  influx($db, '''select mean(value) from series WHERE time < now() - 7d host =~ /netlog.*/ group by time(6h), host, disk''', "7d", "")

=> influx query must not contain time in WHERE
Is there a dependency somewhere in the code on this argument style? I.e: is it possible to ditch the start/end arguments for influx()?

I've pushed some changes to my bosun repo in an experimental branch. This gives me at least a function to simulate these same alerts with.

Ah. This is the influxTag() function at line 27 of influx.go. The influx querylanguage must have some info on how to retrieve those. Perhaps @nathanielc can assist me?

The two errors you are getting are an artifact of the way the query is constructed.
The first aggregate functions with GROUP BY time require a WHERE time clause is just complaining that you can't groupby time if you haven't also added  a where clause on time. Pretty self explanatory.
The second error influx query must not contain time in WHEREis coming from the bosun specific code saying that the influx method wants to add the time clause itself to the query and if there is already one its considered invalid.
The combination of these tow errors creates a cyclic dependency that means you can''t use group by time in your influx queries.
There is a simple solution though. Since bosun is constructing a query with its own time clause, bosun also needs to handle adding the groupby time clause as well. This way the raw query passed to bosun will not have either a time where clause or a groupby time but will be added based on the other parameters passed to the method.
I will not have time to look at this for a while but here is some example code I used before to do this exact same thing:
First: Add another parameter to the bosun method influx  called group_by_time_interval or something.
Second: If that parameter is not empty and is a valid duration string then add a groupby clause to the constructed query like so:
Place this code about here https://github.com/bosun-monitor/bosun/blob/master/cmd/bosun/expr/influx.go#L152
groupByInterval, err = time.ParseDuration(groupByIntervalStr)
if groupByInterval > 0 {
		s.Dimensions = append(s.Dimensions,
			&influxql.Dimension{
				Expr: &influxql.Call{
					Name: "time",
					Args: []influxql.Expr{
						&influxql.DurationLiteral{groupByInterval},
					},
				},
			},
		)
	}

Using your example the influx method would look like this
influx($db, '''select mean(value) from series WHERE host =~ /netlog.*/ group by host, disk''', "7d", "", "6h")

Hopefully that is enough to get you started.

Okay, here is a rough draft for the documentation on #1304, with remaining notes., I'm interested in your views wrt the nil values that Influxdb can return.
InfluxDB Query Functions
influx(db string, query string, startDuration string, endDuration, groupByInterval string) seriesSet
Queries InfluxDB.
All tags returned by InfluxDB will be included in the results.

db is the database name in InfluxDB
query is an InfluxDB select statement
NB: WHERE clauses for time are inserted automatically, and it is thus an error to specify time conditions in query.
startDuration and endDuration set the time window from now - see the OpenTSDB q() function for more details
They will be merged into the existing WHERE clause in the query.
groupByInterval is the time.Duration window which will be passed as an argument to a GROUP BY time() clause if given. This groups values in the given time buckets. This groups (or in OpenTSDB lingo "downsamples") the results to this timeframe. Full documentation on Group by.

Notes:

By default, queries will be given a suffix of fill(none) to filter out any nil rows.


[BUG] in HeteroFL, the accuracy of the global model does not change

Describe the bug
the results of HeteroFL in examples seem doesn't right, the accuracy of the global model does not change
To Reproduce
python examples/heterofl/heterofl.py -c examples/heterofl/heterofl_resnet18_dynamic.yml
Screenshots

Additional context
Compared to FedRolex, the HeteroFL takes much more time on model aggregation, which is weird too, the two methods have the same aggregation steps based on papers.
the HeteroFL aggregation:

the FedRolex aggregation:

@wangxm233 In the implementation of HeteroFL, it contains HeteroFL algorithm according to their paper including the statistical batch normalization. This example shows that statistical batch normalization is a poor design (e.g. may not converge) when the structures of sub models are randomly sampled.
On the other hand, in the example of FedRolex, where we give an example of removing the sBN module. You can disable the sBN module in the HeteroFL by set parameter parameters.model.track to false  in the configuration file and annotate function weights_aggregated and clients_processed in heterofl_server.py or refer to the implementation in FedRolex where sBN module is disabled.
Additional Context
More time spent during mode aggregation is caused by the process of doing statistical batch normalization operations.


Fix bug in lightSync where from is off by one

Issue Number
ADP-1422, ADP-1427
Overview
Light-mode (Epic ADP-1422) aims to make synchronisation to the blockchain faster by trusting an off-chain source of aggregated blockchain data.
In this pull request, I fix an issue with the implementation of lightSync: The from field of a BlockSummary must point to the BlockHeader that immediately follows the BlockHeader to which we are currently synced. This matches the convention for retrieving a list of blocks, where only new blocks are retrieved, never the current one.
Attempt at visuals:
BlockSummary:

  B--------B-----B-----B----B
here      from             to

List:
  B       [B,    B,    B,   B]
here      from             to

This pull request also adds a check on from to the unit test for lightSync.

bors r+


[FEATURE] Trino compatible API

test client: https://github.com/trinodb/trino-python-client
API documentation: https://trino.io/docs/current/develop/client-protocol.html

initial support: https://github.com/mabel-dev/opteryx/blob/main/opteryx/api/trino_api.py

Will define own API


x/tools/cmd/godoc: document behaviour of comments somewhere

The good folks over at stackoverflow don't like my quite reasonable question asking where is the specification for godoc itself
The language specification doesn't mention that go:generate is special or that godoc comments exist.
The full rules should be documented somewhere other than in the gogenerate and godoc tools source.
Repeating that here:

I know godoc is minimal compared with similar tools but I would expect to find a specification for it somewhere. The closest thing to a specification I can find is the blog post announcing it. Is there anything more elsewhere? Even if there isn't a formal specificaiton perhaps a 1 page cheat sheet?
go help doc - documents the "go doc" sub-command but doesn't mention the format
This is also a hard thing to google as you get hits for anything documented using godoc.
I have found a couple of "tricks and tips" pages:
http://elliot.land/post/godoc-tips-tricks
https://godoc.org/github.com/fluhus/godoc-tricks

I know the source is available but I should not have to read the source (which also has no link to a spec) to know how to use the tool.
I am surprised that I should have to ask this question and also that no-one has asked already so perhaps I have missed something obvious?
Some things not mentioned in that blog post which may help show why a spec would help that I have stumbled upon are:
[titles](https://godoc.org/github.com/fluhus/godoc-tricks#titles) (as in link above)
[quotes](https://www.reddit.com/r/golang/comments/35ee2t/quotes_in_godoc/)

Extensions using go commands must not clash with the godoc syntax For example:
[gocontracts](https://github.com/Parquery/gocontracts extends the godoc syntax to support contracts in comments
[apidocjs](http://apidocjs.com/) allows you to embeds API documentation in go comments
go itself uses some comments semantically - e.g. "go: generate"

The language specification itself does not discuss the behaviour of comments beyond the syntax.

The docs are at http://golang.org/doc/articles/godoc_documenting_go_code.html .  There is a link to that page from the godoc documentation at https://godoc.org/golang.org/x/tools/cmd/godoc .
It would help if you could point out areas where that could be improved.  Thanks.

That document is a great one page introduction. It is not a specification. A specification need not be very long but I think it would be useful. It could be similar to a cheat sheet.
The blog page does not:

say what comments will be interpreted by go tools like go doc and go generate.
It does not say whether it is legal to name the function twice.
Will you get two comments or one comment with two bullet points?
Is it safe for third party tools gocontracts to try to extend the syntax by using "FuncName requires" to indicate a pre-condition or apidocjs (not a go tool but a tool that happens to support go) to do what it does.
Do different versions of go have different rules?
What are the reserved words?  (at least "Deprecated: ", "BUG():" & "go:generate")

I just discovered build constraints as another semantic use of comments built into go
https://golang.org/pkg/go/build/#hdr-Build_Constraints
These really all should be documented together.
To an outsider it feels like a design smell that comments can be used for so many different things.

Titles feature is not documented in the article
https://godoc.org/github.com/fluhus/godoc-tricks#Titles

doc comments are now more standardized https://go.dev/doc/comment

This still is an issue.  There's no single place which would tell me what is a Golang comment and what is not.  I stumbled across build tags, //go:, #cgo opening up a new context when combined with import "C" but there might be other dragons which I'm unaware of.

I rewrote https://go.dev/wiki/Comments to document more of these compiler directives.  Please let me know about anything else that should be added here, or feel free to add it yourself.  Thanks.

I rewrote https://go.dev/wiki/Comments to document more of these compiler directives. Please let me know about anything else that should be added here, or feel free to add it yourself. Thanks.

That's actually very helpful, and includes references and historical changes.  Thanks a lot.


Variable Group - Tell whether variable is still being referenced outside of the group before deleting it.

Is there a way to tell whether a variable within a variable group is currently being referenced by a pipeline?
I would like to do some cleanup of our variable groups. However, the task seems quite daunting and impractical if I have to manually verify that each of our pipelines are no longer using the variable and/or group and hope that I don't mistakenly delete one that is still being used by a pipeline.

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

ID: 4827aa99-ae62-bd63-6f2f-a87a4065ed27
Version Independent ID: c9e461ee-7f42-3503-9460-18fa3a807bbb
Content: Variable groups for Azure Pipelines and TFS - Azure Pipelines & TFS
Content Source: docs/pipelines/library/variable-groups.md
Product: devops
GitHub Login: @alexhomer1
Microsoft Alias: ahomer

Thank you for your feedback. At present there is no way to do this. You might like to post a request on the Developer Community website at https://developercommunity.visualstudio.com/spaces/21/index.html, where they are tracked by the product teams. Thank you.


emacs not working but terminal worked

Hi,
I'm using this wakatime-mode, it was good. but when I upgrade a version, it not working in emacs, but working in terminal. some of the snapshot:
the command is essentially:
/usr/bin/zsh -c '~/.wakatime/wakatime-cli --entity /tracked/file/name/xxx.py --plugin "emacs-wakatime/1.0.2" --time<PHONE_NUMBER>.48 --write --key waka_aef****'
the log is:
{"caller":"cmd/run.go:341","file":"/tracked/file/name/xxx.py","func":"cmd.runCmd","is_write":true,"level":"error","message":"failed to run command: sending heartbeat(s) failed: failed making request to "[https://api.wakatime.com/api/v1/users/current/heartbeats.bulk](https://api.wakatime.com/api/v1/users/current/heartbeats.bulk\)": Post "[https://api.wakatime.com/api/v1/users/current/heartbeats.bulk](https://api.wakatime.com/api/v1/users/current/heartbeats.bulk\)": context deadline exceeded (Client.Timeout exceeded while awaiting headers)","now":"2024-03-24T15:54:32+08:00","os/arch":"linux/amd64","plugin":"emacs-wakatime/1.0.2","time":1711266751.88,"version":"v1.90.0"}
same command when I run in WSL2 terminal, it worked and log show working correct
can you help me figure out how is this happening?

It looks like the connection simply timed out, at least from the error log provided.
WakaTime sets a timeout of 120 seconds. The time value<PHONE_NUMBER>.88 is 2024-03-24T15:52:31+0800, basically exactly 120 seconds before the log was sent at 2024-03-24T15:54:32+08:00, which checks out.

It looks like the connection simply timed out, at least from the error log provided.
WakaTime sets a timeout of 120 seconds. The time value<PHONE_NUMBER>.88 is 2024-03-24T15:52:31+0800, basically exactly 120 seconds before the log was sent at 2024-03-24T15:54:32+08:00, which checks out.

thanks for feedback, is it related to I used VPN? How set proxy to this wakatine emacs package?


Optimize single release getting

Changes Proposed
Used dedicated Helm call to get individual release, instead of scanning full list always.

Check List

[x] The title of my pull request is a short description of the changes
[x] This PR relates to some issue: #372
[ ] I have documented the changes made (if applicable)
[ ] I have covered the changes with unit tests

Codecov Report
Patch coverage: 43.75% and project coverage change: +0.27 :tada:

Comparison is base (27eb794) 29.82% compared to head (e0dad24) 30.09%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #373      +/-   ##
==========================================
+ Coverage   29.82%   30.09%   +0.27%
==========================================
  Files          10       10
  Lines        1321     1329       +8
==========================================
+ Hits          394      400       +6
- Misses        888      890       +2
  Partials       39       39

Impacted Files
Coverage Δ

pkg/dashboard/objects/releases.go
8.19% <43.75%> (+1.87%)
:arrow_up:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Add a feature for SVG support in the Async Clipboard API

See https://blogs.windows.com/msedgedev/2024/07/11/seamless-svg-copy-paste-on-the-web/ for some background on this.

We added this in #1703.
(I didn't realize how new this is, will fix a regular web papercut for me!)

I'm going to close this as a duplicate of #1703. If you still think this should be a standalone feature, please merge main into your branch and reopen for discussion/review. Thank you!

I don't think web devs know that the clipboard API doesn't support all formats by default. I don't think many of them know that it only supports text, html, and png out of the box. And, on top of this, that browsers sanitize the html and png data to avoid security problems, so you don't even get to read the same HTML code that you wrote in the clipboard earlier.
Because of this fact, and because the addition of SVG, and web custom formats happened much later than the bulk of the clipboard API, I would still prefer to track these as separate features. I think splitting these gives a better chance for devs to understand the limits of the clipboard API, and the additional features they need to achieve their particular use case (for example, a Figma-like app would want to check the Baseline status for both a clipboard, and a clipboard-svg feature. Or, an excel-like app would want to check the status of a clipboard-web-custom-format feature, to know whether they can pass their custom spreadsheet+macro code, unsanitized, in the clipboard, between a native and a web app).

OK, please reopen after you've merged in main and resolved the errors. The existing PR will fail on merge, due to duplicate keys.


Different scale for skeletal meshes (armatures) and static meshes

Basically every export that has both static meshes and skeletal (parented to armature) meshes ends up with different scale for meshes. Skeletal meshes are always 100x smaller. I guess it's somewhere around SCALE_FACTOR = 100 and armature handling code. Inconsistent scaling is inconvenient if you have many meshes in a single file and also affects physical models and vehicles.
How to reproduce:

Create 2 default cubes (2x2x2m), drag them to Export collection.
Push assets to UE (do not change any settings). Both cubes are 200x200x200 cm.
Create empty armature, drag it to collection, parent any cube to the armature.
Push assets to UE, skeletal mesh cube is 2x2x2, static mesh cube is 200x200x200.

Blender 4.2, Blender Tools 2.5.
Upd. It's not an easy fix, e.g. deleting this line fixes the cubes above but breaks nested hierarchy scale (e.g. wheelcaps).
         # Todo add to FBX addon
         if ob_obj.type == 'ARMATURE':
-            scale = Vector((scale[0] / SCALE_FACTOR, scale[1] / SCALE_FACTOR, scale[2] / SCALE_FACTOR))
             if bpy.context.scene.send2ue.use_object_origin:
                 loc = Vector((0, 0, 0))

A better file to test the hierarchy/armatures/size:

gnomecar.blend.zip (Proper size should be approx 400x400x200 units.)

The easiest way to handle that without fixing the plugin code is to parent armatures to "empty" with scale 100, and check the "Apply Transform" option. I also use "Extensions - Combine assets - Child meshes" plugin setting to get wheels as bones for the car simulation.
Note the generated convex hull on the second picture is 100x smaller than the mesh. This also affects ragdolls because bones are not scaled properly. "Apply Transform" fixes convex hulls and other physics (except nested meshes).

With "Apply Transform"
Without "Apply Transform"

The only way to reliably keep hierarchy and get proper size is to rescale all skeletal meshes 100x in Blender before exporting. Also I had to rotate the mesh in Blender because it faces the wrong way, changing export axes to X Forward, Z Up doesn't work too. I also had to merge wheels and wheelcaps meshes because wheelcaps shrink (hierarchical scale doesn't seem to work).

Yeah I’ve noticed that too, but haven’t dug into it and pinned it on Send2UE. I separately apply transforms to the mesh, and there’s a different option to apply skeleton transforms.
I use CGDives Game Rig Tools and have halfway suspected that’s the culprit too btw.

 I will take a look at what's going on on Monday.

This seems to be an issue with the import in UE 5.5. I can still export to 5.4 just fine. It breaks if i reimport in 5.5 without send2ue as well.

I don't have 5.4. Yea I've noticed 5.5 is a little different, Also there's an issue with bone naming. In 5.4 I imported bone names with spaces and it replaced them with dashes, now 5.5 replaces spaces with underscores (e.g. Bip01-Head is Bip01_Head now) and Mixamo animations made from the old model no longer work.

So I disabled the Interchange Framework plugin and it seems to have fixed that scale issue. I diddn't check the issues with the bone naming though.

That's a lot of dependencies, it also disables Fab and a few other plugins. It does NOT fix the initial 2 cubes issue for me. But general animation import is back to normal as in 5.4 (except bone naming, still underscores). With the Interchange Framework plugin enabled it doesn't even add any animations and collapses the skeleton.

Yeah for now it looks like the best option is stick to UE 5.4 or do imports in a seperate 5.4 project and migrate them until there is an actual fix.


Fix no default export issue

Fix VS Code's no default export issue
closes #11

Thank you very much @mamunonweb!


Make archetypes query zillions of times faster.

Travis tests have failed
Hey @bakert,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.

Ah, yeah, this will not bootstrap.


Fix Clearing Inspector for Remote Node

Resolves: #30731
When you stop debugging... if you were inspecting a remote node, partial information about the remote node remained in the inspector (e.g. name and warning).
To resolve this, called EditorNode::edit_current() instead of EditorInspector::edit(NULL), which will call all the methods required using "NULL" if the current selected object is NULL.
In EditorPath::update_path() if the selected object is NULL, it does not update the path to reflect this change, basically does nothing. To fix this, we nullify everything, before the history loop.

Thanks!


[ColorPicker] Third Grid column width is hardcoded and control is not responsive

How color picker looks with Fluent theme:

Workaround:
// Execute this code when all children were added from the template.
if (((IVisual)ViewPicker).VisualChildren.FirstOrDefault() is Grid grid
    && grid.ColumnDefinitions.Count == 3)
{
    grid.ColumnDefinitions[2].Width = GridLength.Auto;
}

ColorPicker after workaround

Apart from that, NumberUpDown control looks wrong, but it may be global problem - I will check later.


Update for namespace changes

Closes https://github.com/ntt-sic/masakari-deploy/issues/1

@vmtyler Thanks for the fix.
Deployed without problem.
Though, have 2 comments.
(1) Change the resource dpkg_package name from masakari_ to masakari-controller_
cookbooks/masakari_controller_conf/recipes/default.rb
(2) Rename the template files and change the source template names

Rename the template files in cookbooks/masakari_compute_conf/templates/default/
Ex: hostmonitor.conf.erb to masakari-hostmonitor.conf.erb
Change the related source name of the template resources in /cookbooks/masakari_compute_conf/recipes/default.rb

Will do
On Tuesday, January 26, 2016, samP<EMAIL_ADDRESS>wrote:

@vmtyler https://github.com/vmtyler Thanks for the fix.
Deployed without problem.
Though, have 2 comments.
(1) Change the resource dpkg_package name from masakari_ to
masakari-controller_
cookbooks/masakari_controller_conf/recipes/default.rb
(2) Rename the template files and change the source template names

Rename the template files in
cookbooks/masakari_compute_conf/templates/default/ Ex: hostmonitor.conf.erb
to masakari-hostmonitor.conf.erb
Change the related source name of the template resources in
/cookbooks/masakari_compute_conf/recipes/default.rb

—
Reply to this email directly or view it on GitHub
https://github.com/ntt-sic/masakari-deploy/pull/2#issuecomment-175356154
.

Done

@vmtyler thank you again.
Looks good to me. +1

@vmtyler thanks for you pull request.
I checked and tested. It's okay. Let me merge.


Switch from npm to corepack for installation?

See the history and discussion on https://github.com/plone/documentation/pull/1540.
I am not sure of the impact of switching from npm to corepack for installation instructions, and possibly in automation and the cookiecutter. This will need input from Volto core contributors and maintainers.
If approved, I would suggest that we update the Plone 6 documentation with the following.
#### Yarn 3

Install the latest Yarn 3 version (not the Classic 1.x one).

1.  Open a terminal and type:

    ```shell
    corepack enable
    ```

2.  Verify that Yarn v3.x.x is installed and activated.

    ```shell
    yarn set version 3.x
    yarn -v
    ```
    ```console
    3.2.3
    ```

Hi , Can i work on this issue

@dhruvgupta0503 please read and follow Contributing to Plone and First-time contributors, especially Things not to do.

@stevepiercy https://yarnpkg.com/getting-started/install if we look at the install docs they say to use the corepack install so I would remove the install with npm since it is discouraged by yarn install docs.
This is because we offered this way of install as an alternative for the npm install which nowadays is wrong and should be removed.


Edaneshmand/passive joints

Description
Some robots (like bolt) have passive joints. By this pull request, I want the controller to consider passive joints.
It needs a bit of change in all impedance_ctrl.yaml files. For example, for Bolt with passive ankles "is_joint_active:[True, True, True, False]" should be added to both legs.
How I Tested
Do not merge before

[ ] link to dependent pull request

I fulfilled the following requirements

[x] All new code is formatted according to our style guide (for C++ run clang-format, for Python, run flake8 and fix all warnings).
[x] All new functions/classes are documented and existing documentation is updated according to changes.
[x] No commented code from testing/debugging is kept (unless there is a good reason to keep it).

@Lhumd If you are done with the PR you can merge!


Stop inviting proposals for incompatible encodings

If, as @Stebalien said when closing #12, "the ship has sailed", then the readme should no longer be inviting people to propose and discuss breaking changes to the protocol.

No worries; I did wonder whether it was out of date when I first read it, and I think git blame said it was added about four years ago.  I can easily see how its removal could be overlooked, though.


[SPARK-8112][Streaming]Fix the negative event count issue

Merged build triggered.

Merged build started.

Test build #34223 has started for   PR 6659 at commit d255b6e.

Test build #34223 has finished for   PR 6659 at commit d255b6e.

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

Merged build finished. Test PASSed.

Merged build triggered.

Merged build started.

Test build #34250 has started for   PR 6659 at commit a5d7da6.

LGTM. We can test this in PR #6614

Test build #34250 has finished for   PR 6659 at commit a5d7da6.

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

Merged build finished. Test PASSed.


Fix: Be more specific with error message

This PR

[x] attempts to be more specific with error messages when requiring a minimum / maximum number of items in an array

+1


Recover funds after funding transaction timeout (Ethereum)

Problem
As mentioned in #20, a channel opening can fail while channel funding is in progress (e.g., if the funding confirmation is not  received within the specified waiting period). There is currently no convenient method to withdraw the funds in that case.
Solution

Persist the channel before submitting the funding transaction.
If the channel opening fails, wait until the funding transaction is confirmed and initiate channel settling afterwards.

@matthiasgeihs  Currently a fundingTimeout that is same as Challenge Duration(see here).
When this timeout expires, the loop that waits for Funding events within the Funder will abort. If we try to initiate channel settling after this, should we increase the timeout value to accommodate for  genuine delays in the transaction getting mined ?

Or do you think, the Challenge Duration will be sufficient to accommodate for those delays ?

Related to #239

Closed in #349


Declare CLI options as global

By declaring CLI options global we allow their usage in subcommands
for the custom CLI.
Issue : #215

Indeed I didn't think about it thanks !


Support for transitions in iron-pages

Are there any plans to add support for page transitions to iron-pages, similar to what was done for core-animated-pages with transitions="slide-from-right" attribute?

It has already been implemented - <neon-animated-pages> under neon-animation. In any case, you should raise issues under the correct section - in this instance the iron-page Github page.

Thanks!


[Bug Report] Component Pagination can't render by @vue/test-utils API  "mount "

Element UI version
2.5.4
OS/Browsers version
win 10
Vue version
2.6.7
Reproduction Link
sorry for can't offer the [https://jsfiddle.net
but  https://codesandbox.io/s/nn28lx9r80
Steps to reproduce

Open codesandbox URL
Open the Test Tab

What is Expected?
mount can render the component Pagination
What is actually happening?
it can't render

Pagination component is not Element's Pagination  component.


README instructions aren't clear in regards to Docker

Your instructions say:

You should only need to change the IP address here. Replace <IP_ADDRESS> with the IP address that docker-machine ls gives you.

However, this doesn't output anything but:
NAME   ACTIVE   DRIVER   STATE   URL   SWARM   DOCKER   ERRORS
What is going on?
Igor

Just a suggestion - instead of telling folks to run docker-machine ls, tell them instead to run docker ps.  This would yield the following output, and will tell them exactly which ip address the Web UI and App Server are running on:


Intel cAVS v2.5 offline loader and multiprocessing

Enable the MP cores on this hardware and make that configuration default.  Doesn't pass everything yet, but is a lot closer than what we have in tree.  Includes all relevant changes from the unmerged PR #36528

Note that this includes a regression of the XCC linker fix from @dcpleung, which unfortunately seems to misbehave on SDK builds (my guess is that the syntax using a symbol name as a section start address gets ignored?).  So we're likely going to need to revisit that bug again and find a fix that works identically to the historical code.

Push a few fixes, see final three patches (one of which regresses the use of WAITI in Zephyr idle and is really only a workaround).
At this stage I can get passing results from basically all 1-cpu and single-feature tests, and most threading/smp tests in 2-core mode.  Some of that may be latent bugs in the tests: we haven't had a 4-core configuration in CI yet and so testing that many has only been done ad hoc.

Maybe move boards/xtensa/intel_adsp_cavs15/tools/cavs-fw-v25.py under boards/xtensa/intel_adsp_cavs25/tools?

Maybe move boards/xtensa/intel_adsp_cavs15/tools/cavs-fw-v25.py under boards/xtensa/intel_adsp_cavs25/tools?

Actually the second I have a chance to do some housekeeping I think we want to (1) unify both scripts so they work on all hardware and (2) move it to soc/xtensa/intel_adsp and out of the board layer.  Likewise for a bunch of the headers, which differ only in a few addresses that are extractable from devicetree (some of that work is already in this PR).  Likewise for the linker scripts which are also almost all duplicated...
Really the "board" layer for these devices is way too thick.

Updated with current version that is actually passing many tests now, and has a seemingly-working low power idle API (soc_{halt,relaunch}_cpu(id);) in the SOC layer (though no test for that yet -- have to clean up my hello_world-based rig and add it).

The checkpatch whine is a false positive.  Pre-existing formatting for that inline assembly had those spaces, and objectively the current format is more readable than any alternatives.  The warning is supposed to be about trailing spaces in log messages or whatever,  it just doesn't apply.

One last patch in the series (left unsquashed for easier review) seems to have been it.  I haven't had time to run a full suite yet, but I can't find any common test cases that fail.
Build failures on older cAVS should be fixed, but I don't have the board with me to test it right now.

Concerning the APL SMP runtime breakage: there are several changes to SMP code on platforms other than cAVS 2.5 in this PR, I cannot say which of those changes breaks APL. I could bisect, but first I'd need to merge the last fixing commit with the commit, that it's fixing, and then bisect. Maybe it's better to extract a minimum PR from this one, touching only 2.5 and not modifying a single instruction on others. After that we can improve and consolidate step by step.

Add a patch that fixes an SMP regression on APL.  One part is straightforward: I accidentally removed code that unmasked the IDC interrupt on the target core.  That's benign on 2.5 where the startup-on-IDC behavior is handled by a state machine outside the interrupt controller, but on earlier hardware that's a software handler in ROM!
The other bit is sort of voodoo: the new scheme sets up the RPO memory mapping "TLB" registers, which is required on 2.5 where we handle the hardware handoff ourselves.  But trying to re-write them to the same values already set by the ROM is causing hangs.  We never used to do this, and it's needless since the ROM already does it, so remove.  But I don't know why that shouldn't work.  Maybe the cache controller (which has active lines for the region) is messing things up?

And I see that pylint warning about duplicate lines is back.  @nashif you mentioned there was a way to add a comment to suppress that?  Any pointers would be appreciated.

Simple cleanup and an attempt to suppress that pylint duplicate-code error with a pragma comment.

Nope.  Turns out that cross-file pylint warnings can't be disabled (https://github.com/PyCQA/pylint/issues/214).
That makes that issue another genuine false positive.  AFAICT there's nothing that can be fixed with the compliance results.  Pull the trigger and merge, please.

Had to respin to fix a warning that snuck in.

Once more because I forgot to commit both warning fixes, ugh.

Websocket samples?!  Ugh.

Buildkite seems to have two remaining failures on the adp_xc7k_ae350 board.  I can guarantee that's unrelated.

Still not working on APL with multicore

@lyakh Can you be more specific?  The test suite seems fine.  Really, we need to get this merged.  We can work bugs as they appear.

OK, indeed @lyakh found the magic.  That irq_enable() was also setting the Xtensa INTENABLE SR, not just the second level interrupt controller.  So the interrupt would be able to reach the core but would still be masked by the CPU itself.  Under Zephyr-internal code, that was hidden by the fact that the timer driver initialization on the second core (which shares the same L2 interrupt) was then enabling it before any user code ran.
But on SOF, secondary core initialization happens later, and the IPC task run immediately after core start needs to be able to run synchronously on that core, and can't wait for a timer setup.  Instead it was being picked up on core 0 and failing.  (Which is sort of a SOF bug, actually -- if the task has to run on a specific core for correctness it should be pinned to run only on that core, can fix later.)

Also added a few new minor patches from yesterday's work and fixed the identified bisectability/misordering problems.  Probably should have rebased over main, but didn't.  Will do if it needs a respin.  @lyakh, please review and retest?

Unsurprisingly I indeed had to respin to add a dropped patch.  So now it's rebased on current main.

(lightly) tested to work with SOF on APL with multicore topology again - thanks!


[T4A4][W11-B3]Wong Kang Fei

…dress so that each produces a printable string representation of the object.

Accepting this PR without a review because another of your PRs for this tutorial was reviewed in detail.

Some comments added. Please close the PR after reading comments.


Show hashing progress

For better use experience, it would be nice to have a UI element for each loaded file showing the progress of currently executing hash precess.

A nice example is placed here.
(Having this applied would also fix #2)

This example shows a very short way to block wise generate hashes from input bytes. Reading them block wise would bring the wanted result.

Done with 6ea3816ef462e7fa585e3d64ddbd6b90c686411f


Adding after_each command

Summary of changes:

Removed the command that sets the modified time of files in ADD and COPY. Docker 1.8 no longer uses modified time when determining cache.
Added an after_each that runs regardless of pass or fail of the run.
Moved the current trap to run before_each for sub builds (just in case compose commands are run there and it fails)

Let me know what you think. Thanks!

thanks @aheuermann

Quick question @aheuermann . Why do we need sh -xc cmd  why can't we just cmd.
Same fore before_each

Quick question @aheuermann . Why do we need sh -xc cmd why can't we just cmd.

I think you are right, it probably could just be cmd. I think I used the sh -xc because the original before used it. #174

@aheuermann cool. I'll make that change :+1:

Released in 2.26.0 . Also added after_run which behaves like before_run.
Will update docs tomorrow.


[Feature request]  --buffer-size-disk or --buffer-path=/

Same feature as current buffer-size that is being stored in ram while --buffer-size-disk would store it on disk.
Since buffer-size is stored in ram its not feasible to set it to some extremely amount since you will end up out of ram eg buffer-size 25G ( to cover whole movie ) as the buffer is set per file.
It would be important to add new setting in config eg buffer-path=/path/ so we can set where buffer will be stored.

Or maybe instead of making new flag --buffer-size-disk we could just have additional flag -buffer-path=/path where to store buffer/ so without it its stored in ram otherwise in defined path.

What is the use case for such a buffer?
(Assuming that a "buffer" is something that gets cleared once used)

What is the use case for such a buffer?

Most of people Plex Servers are running on fast internet connections but we all have limited amount of ram so its hard to set 10G ram per connection as few streams and your all machine is out of ram ( the cache would still be swaped on disk. however the machine would suffer due no free ram )
If we overcome this by setting buffer file to disk we can easily set 10G, for example my server will in average download 10G within first 3 to 5 minutes and that would mean the whole file is cashed on disk, the rclone can now drop connection to amazon and user can seeks up and down on the file as much as he wants without any kind of problems on plex side.

If you have a server with a fast internet connection, you probably don't need a buffer at all, as @ncw noted in the forum it should handle network glitches (not seeking, not consecutive reads etc...), a small amount of ram per file should be more then enough
I think you are confusing buffering with caching which are two different features

You can have best internet in world and it wont help as ACD is throttling speed a lot and average copy can change from 500K to 50MB so with limited buffer because of memory every seeks will start to redownload ( same case with disk buffer ), so what you want to achieve is to get file down asap and be done with it.
p.s. This is optional flag so those who dont need it dont use it.

I support this feature request as well.  My current setup is a cheap VPS and my media is optimized for direct play on all devices.  I'm just now starting to play with buffer size but I'm only working with 1G of ram so it doesn't take much to use it up.  Additionally, with many people using SSD VPS the access times are just as good as using RAM.  In the meantime, could I just increase my buffer size above 1G and use swap or would that cause issues with other programs?

I see this more as a disk cache rather than a buffer... Cache it to disk, let me set cache total space to something and maybe a timeout, so things cached for 30 days get removed... This way, streaming works better, as on first access or multiple block reads, we grab the whole file to cache. Then when I stop and go back later to rewatch, I can actually resume... I'm more than willing to give up 100GB of space on my hdd for a cache like this. The idea being, I can keep tons of data in ACD, but when reading, it will act like a local disk.


fake change to trigger tests

please ignore, this is to troubleshoot build failure on master branch

@therealmitchconnors: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
d9a1b8b5bbfc42dc692983ab591d35d72f2afc70
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@therealmitchconnors: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
d9a1b8b5bbfc42dc692983ab591d35d72f2afc70
link
true
/test release-notes

gencheck_istio
d9a1b8b5bbfc42dc692983ab591d35d72f2afc70
link
true
/test gencheck

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

should eb fixed now


Add Support for IA3 Adapters in add_weighted_adapter Method, Currently facing issue that 'IA3Model' object has no attribute 'add_weighted_adapter'

Feature request
I propose adding support for IA3 adapters in the add_weighted_adapter method used within the PEFT module of the transformers library. IA3 adapters enhance model adaptability with minimal parameter increase, offering significant benefits for efficient and effective model fine-tuning across various tasks. This feature would allow users to integrate IA3 adapters seamlessly, thereby expanding the functional capabilities of the library.
Motivation
The motivation behind this proposal is to address the current limitation in the PEFT module of the transformers library, where IA3 adapters are supported but the merging of IA3 adapters using the add_weighted_adapter method is not implemented. This gap leads to errors such as 'IA3Model' object has no attribute 'add_weighted_adapter' when attempting to merge IA3 adapters.
The ability to merge IA3 adapters is crucial for users who rely on the PEFT module for efficient model adaptation and fine-tuning. Without this functionality, users are unable to leverage the benefits of IA3 adapters, such as reduced memory footprint and customizable model tuning, within the PEFT architecture.the
Your contribution
I do not have a complete implementation in mind, but I suggest starting with an evaluation of the current add_weighted_adapter function to determine the necessary modifications for supporting IA3 adapters. Collaboration with researchers familiar with IA3 and PEFT could provide insights into feasible approaches.

We had a PR once in #980 but there were a few non-trivial decisions to be made. If you want to work on a PR, please check out the discussion there. Pinging @alexrs also to see if there are any updates.

I am using your branch alexrs:multi-ia3 to test the addd_weighted_adapter for ia3
FIRST_ADAPTER_PATH = "/content/drive/MyDrive/FedMl_test_llm/TrainedModels/WestLake_SFT_IA3_1/Weights/Epoch_1/Job_1"
SECOND_ADAPTER_PATH = "/content/drive/MyDrive/FedMl_test_llm/TrainedModels/WestLake_SFT_IA3_2/Weights/Epoch_1/Job_1"
FIRST_ADAPTER_NAME = "first"
SECOND_ADAPTER_NAME = "second"
model = PeftModel.from_pretrained(quantized_model_4bit, FIRST_ADAPTER_PATH, FIRST_ADAPTER_NAME)
_ = model.load_adapter(SECOND_ADAPTER_PATH, adapter_name=SECOND_ADAPTER_NAME)
adapters_li = [FIRST_ADAPTER_NAME,SECOND_ADAPTER_NAME]
weights_li = [0.5,0.5]
new_adapter= "ADAPTER_WEIGHTED"
if new_adapter in model.peft_config:
model.delete_adapter(new_adapter)
model.add_weighted_adapter(adapters=adapters_li, weights=weights_li, adapter_name=new_adapter)
After this it shows me the error that :
Invalid type <class 'list'> found in target_modules
I meanulay test this using code
loaded_adapters = list(model.peft_config.keys())
print(loaded_adapters)
shows I have two adapters load in model
['first', 'second']
I manually test this:
module_type="target_modules"
adapters=adapters_li
module_types = [type(getattr(model.peft_config[adapter], module_type)) for adapter in adapters]
print(module_types )  shows [list, list]

Please solve my problem I urgently need to merge few ia3 adapters , how can I add them

Thanks for the ping @BenjaminBossan
Hi @Abdullah-kwl, the code I wrote for #980 is very outdated. The HF folks have been shipping lots of code lately! 👏
I guess this is a good moment to start the conversation again (sorry for the very long delay!). We want to implement add_weighted_adapter for $(IA)^3$ adapters in such a way that is comparable to the LoRA implementation. The implementation, however, is a bit different for multiple reasons:

$(IA)^3$ introduces trainable vectors instead of matrices, therefore it does not make sense to support most of the combination types implemented for LoRA.
$(IA)^3$ follows multiplicative operators. When we discussed this issue back in November last year, we did not reach an agreement on whether we should do a linear combination of adapters, or multiply them.

I'd suggest that we can start with a simple implementation of add_weighted_adapter that combines adapters using a weighted average of $(IA)^3$ vectors (equivalent to linear in LoRA if I remember correctly).
Thoughts @BenjaminBossan @pacman100 ?
I can try to prototype something in the next few days if you think this is a good approach!

It is good to start with a simpler linear approach.

I'd suggest that we can start with a simple implementation of add_weighted_adapter

That would be fantastic. Let's start with something simple and not try to have a "feature complete" copy of add_weighted_adapter for LoRA.

I can try to prototype something in the next few days if you think this is a good approach!

Thanks, that would be great. Maybe once you have the first testable version, @Abdullah-kwl can test it out and give feedback on whether it works well or not.

Hi @Abdullah-kwl
I found some time to prototype an implementation. You can find it in https://github.com/huggingface/peft/pull/1701
It is still work in progress, I did not do any manual testing to check that the result is correct. You can give it a try and report any issues!
Assuming you have two $(IA)^3$ adapters, you should be able to use:
peft_model.add_weighted_adapter(ia3_adapters, [0.1, 0.9], "weighted_adapter")

to combine them using a weighted average of the adapters.

I have tested this it is working

i am also facing some issue relate ia3 that, Cannot merge ia3 layers when the model is loaded in 4-bit
mention in this  https://github.com/huggingface/peft/issues/1704

@BenjaminBossan  please also look at this if we can also add this feature (merge_and_unload for ia3 adapters) for both 4bit and 8bit quantized models .

I have tested this, it is working

Thanks for giving this a spin. If you have any numbers to share, like scores before and after merging, or even code, that would be great.

i am also facing some issue relate ia3 that, Cannot merge ia3 layers when the model is loaded in 4-bit

Indeed, this is not yet supported. We will certainly take a look at this at some point, but contributions are also very welcome. (And please don't post the same issue twice)


Replace global CPU feature flag variables with local variable in init_functable

Now, the CPU feature checks is completely thread-safe.
Based on PR #1440
After PR #1440 is merged, this PR will be rebased.

This will be a good cleanup once it is in.

Rebased and ready for review.
@nmoinvaz, @Dead2


Just Enough Breeding 1.20.x for Fabric?

Hi, are you perhaps planning to update the Fabric version of Just Enough Breeding to 1.20.x ?

Yes that I am :)

Woohoo! Also, thanks for the quick reply.


Backup & Restore doesn't preserve ttl's

Hi,
pipeline-dump doesn't preserve ttl's. if it's intended behaviour, could you please document it.
Thanks,
Venu

Merging w/ #1899


Remove limitation to allow for only basic properties in set

I tried this:
chunks = [['chr9', 1, 15000], ['chr9', 15001, 25000]]
input: sos_groups(by=1), for_each = 'chunks'
output: sos_targets(f"{_input} ... <an _input based name>").set('chunks', chunks)

I get:
 Failed to process step output ..: Target properties can only be sequence of basic types: [['chr9', 1, 15000], ['chr9', 15001, 25000]] provided.

So I guess for set it is not bad idea to remove this limitation? Actually this is the only usage for set that I can be convinced useful.

What I need is an easy way to test data types that do not need pickle to transfer...

What I need is an easy way to test data types that do not need pickle to transfer...

Do we not have that, because we keep global around and we do not check it there?

I know it is a hectic morning, but it would be nice if recursive basic types can be added to the supported basic types for now ... I am a bit stuck without being able to do it in my pipeline.  Basically for dict, list and tuple we want to call the check function recursively for each element of it, right?

Not tested. Make it work if it fails. The code is clear enough.

It worked, thanks a lot!! Maybe i'll close it for now and we 'll see if recursive basic types will cover our basis.


profiling app get different key in the map even if it is in same function

bpftrace -e 'profile:hz:1999 /comm=="a.out"/{ @[ustack] = count(); }'

cat 3.c
void moo(){}
void foo(){}
int main(){
    while(1){
    foo();
    moo();
    }
}

get this map
@[
    moo+5
    main+24
    generic_start_main+484
    __strcpy_sse2_unaligned+0
    0x202860f20f983
]: 3
@[
    moo+4
    main+24
    generic_start_main+484
    __strcpy_sse2_unaligned+0
    0x202860f20f983
]: 5

is there a method can merge the same function name.
i think the ustack should add param to remove the offset of function

(when you open a issue/feature, could you wrap the code snippets in ```<here>``` )

i think the ustack should add param to remove the offset of function

What is the use case for this?

(when you open a issue/feature, could you wrap the code snippets in <here> )

i think the ustack should add param to remove the offset of function

What is the use case for this?

i want to profile which function cost the most cpu time, but now, the different offset of same function are not merged

i want to profile which function cost the most cpu time, but now, the different offset of same function are not merged

In that case, why don't you just print the top stack frame, @[ustack(1)] = count()? If you remove the function offset the stack frame becomes useless anyway.
Or you could do this in userspace, remove everything after the + and aggregate using some awk.

thanks for your reply. this tool is very useful for developer


(chore) Update Benefits help text on Job Creation form

Trello card URL:
https://trello.com/c/9NopUhfv/490-employee-benefits-text-revision
Changes in this PR:
Updated the help text on the Benefits input o Job Creation form
Screenshots of UI changes:

 Employee benefits text revision


Wolverine & Marten integration: Deadletter replay not working correctly

Describe the bug
Replaying deadletter'd events of type IEvent<T> does not work.
To Reproduce
Setup Marten and Wolverine, with marten.IntegrateWithWolverine, marten.PublishEventsToWolverine, wolverine.UseDurableLocalQueues and wolverine.AutoApplyTransactionsalso enabled.
Then, write a handler for a Marten event:
public sealed class MyEventHandler()
{
    public MyCascadingEvent Handle(IEvent<MyEvent> @event)
    {
        throw new Exception("asd");
    }
}

This will send the message to the error queue: with message 'Moved message 0190cac1-9ceb-4bad-a3eb-afe8c9a81cee to the Error Queue
No replay the event, by setting replayable to true in the wolverine_dead_letters table to re-run the message. When re-running, wolverine is not able to correctly route the message.
2024-07-19T13:31:11.2280960 info: Wolverine.RDBMS.DurabilityAgent[0]
      Issuing a command to recover 1 incoming messages from the inbox to destination local://ieventmyevent/
2024-07-19T13:31:11.2490510 info: Wolverine.RDBMS.DurabilityAgent[206]
      Recovered 1 incoming envelopes from storage
2024-07-19T13:31:11.2507170 info: Wolverine.RDBMS.DurabilityAgent[0]
      Successfully recovered 1 messages from the inbox for listener local://ieventmyevent/
2024-07-19T13:31:11.2515180 info: Wolverine.Runtime.WolverineRuntime[106]
      e6c8713c-ee8d-4d91-89e5-9990a5700e82: No known handler for EventMyEvent#0190cac1-9ceb-4bad-a3eb-afe8c9a81cee from local://replies/

Expected behavior
Message should be replayed, but it's lost
Desktop (please complete the following information):

OS: Mac OSX

Found another place where IEvent<T> is causing issues:
    opts.OnException<SendEmailException>()
        .RetryWithCooldown(100.Milliseconds(), 500.Milliseconds(), 2.Seconds(), 5.Seconds())
        .Then
        .ScheduleRetry(1.Minutes(), 2.Minutes());

The cooldowns work, but the retry ends up with:
2024-07-20T11:48:01.6811800 info: Wolverine.Runtime.WolverineRuntime[0]
      Locally enqueuing scheduled message 0190cf89-4f6b-4a7c-8ae4-f89d49044d31 of type EventCreated
2024-07-20T11:48:01.6894440 info: Wolverine.Runtime.WolverineRuntime[106]
      19cbeb11-d601-4ee6-a11d-efb7dd8cb89b: No known handler for EventCreated#0190cf89-4f6b-4a7c-8ae4-f89d49044d31 from local://replies/

This was nasty to fix and took longer than every other bug in this release combined. The IEvent publishing is both strategically important and extremely unpleasant to support.


[Ready to Review] User serializer showing user registrations link for user nodes relationship

Purpose
Issue https://openscience.atlassian.net/browse/OSF-5356
Small fix - user serializer showing user registrations link for user nodes relationship
Changes
Added view_category and view_name to UserRegistrations view.
Now user node link is returned for user nodes relationship.

Thanks. Good to merge when TCI passes.

Merged into release/0.60.0


ace: 7.0.2 -> 7.0.3

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/ace/versions.
meta.description for ace is: "ADAPTIVE Communication Environment"
meta.homepage for ace is: "https://www.dre.vanderbilt.edu/~schmidt/ACE.html"
meta.changelog for ace is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

/nix/store/6wgss9l3rsdnvvp2sa26gjx7yw5y313y-ace-7.0.3/bin/ace_gperf passed the binary check.

1 of 1 passed binary check by having a zero exit code.

1 of 1 passed binary check by having the new version present in output.

found 7.0.3 with grep in /nix/store/6wgss9l3rsdnvvp2sa26gjx7yw5y313y-ace-7.0.3

found 7.0.3 in filename of file in /nix/store/6wgss9l3rsdnvvp2sa26gjx7yw5y313y-ace-7.0.3

directory tree listing: https://gist.github.com/079f5afb5559519efc13e060dd946305

du listing: https://gist.github.com/13b7cfb037f25db0172bbba4f6d0a19a

Rebuild report (if merged into master) (click to expand)

6 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
2 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
2 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
ace
yarp

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/6wgss9l3rsdnvvp2sa26gjx7yw5y313y-ace-7.0.3 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A ace https://github.com/r-ryantm/nixpkgs/archive/05f058b9c997086540a2c8fbd468189688af0739.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/6wgss9l3rsdnvvp2sa26gjx7yw5y313y-ace-7.0.3
ls -la /nix/store/6wgss9l3rsdnvvp2sa26gjx7yw5y313y-ace-7.0.3/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

ace
yarp

Maintainer pings
cc @nico202 for testing.

Result of nixpkgs-review pr 132768 at 05f058b9 run on aarch64-linux 1

2 packages built successfully:

ace
yarp


[v3] Make Snagit crash

Hi,
Just discovered that the v3 of KWM, make the snagit app to crash when you try to perform a screen capture. It crashes after you have selected the region you want to capture.
v2 was working fine with it.
Of course, I checked that disabling KWM makes Snagit work.
-- Edit: it makes crash only the version 4, version 3 works fine. I opened a ticket on their side also.

It seems specific to OSX screen capture. I have the same issue with Quicktime screen recording and recordit screen recoring. Although no issues with cmd-shift-4 style screen captures.

I believe this issue is with KWM's overlay/border feature.
When you disable it, snagit works fine

I have no idea how snagit works, does this happen if you create a window-rule to make snagit float? A screencast of this in progress could be useful as well.


Add flash script usage examples/documentation to v2 docs

Docs repo https://github.com/AugurProject/docs.git
Add under source/version2/includes/ as something like _flash.md, like the other md files.


Allow checking if an enchantment is a treasure enchantment or not - Closes #1596

SpongeAPI | SpongeCommon
Closes #1596

I'd be open to a better method name if anyone has one.


get error when package my game

log
1>------ Build started: Project: UE4, Configuration: BuiltWithUnrealBuildTool Win32 ------
2>------ Build started: Project: Studio, Configuration: BuiltWithUnrealBuildTool Win32 ------
3>------ Build started: Project: Game_2018, Configuration: Development_Game x64 ------
3>  Using 'git status' to determine working set for adaptive non-unity build.
3>  Distributing 8 actions to XGE
3>  --------------------Build System Warning---------------------------------------
3>  Failed to connect to Coordinator:
3>      All builds will run in standalone mode.
3>
3>  -------------------------------------------------------------------------------
3>  --------------------Project: Default-------------------------------------------
3>  PCH.UnrealEnginePython.cpp (0:25.73 at +0:00)
3>f:\xermao\documents\unreal projects\game_2018 4.19\plugins\unrealenginepython\source\unrealenginepython\private\Wrappers/UEPyFSoftSkinVertex.h(15): error C3646: “ss_vertex”: 未知重写说明符
3>f:\xermao\documents\unreal projects\game_2018 4.19\plugins\unrealenginepython\source\unrealenginepython\private\Wrappers/UEPyFSoftSkinVertex.h(15): error C4430: 缺少类型说明符 - 假定为 int。注意: C++ 不支持默认 int
3>f:\xermao\documents\unreal projects\game_2018 4.19\plugins\unrealenginepython\source\unrealenginepython\private\Wrappers/UEPyFSoftSkinVertex.h(22): error C2065: “FSoftSkinVertex”: 未声明的标识符
3>
3>  Error executing C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\bin\amd64\cl.exe (tool returned code: 2)
3>  ---------------------- Done ----------------------
3>
3>      Rebuild All: 0 succeeded, 1 failed, 0 skipped
3>
3>  1 build system warning(s):
3>     - Failed to connect to Coordinator
3>
3>
3>
3>UnrealBuildTool : error : UBT ERROR: Failed to produce item: F:\xermao\Documents\Unreal Projects\Game_2018 4.19\Plugins\UnrealEnginePython\Binaries\Win64\UE4-UnrealEnginePython.lib
3>  Total build time: 27.30 seconds (XGE executor: 0.00 seconds)

Fixed in latest code. Thanks for reporting

I'm also getting this unreal build tool error when building on ubuntu 17.10
Steps to reproduce:

install unreal engine 4.19 from the release branch
make a new c++ project (named fouronenine in this case)
install unrealenginepython plugin from master branch
run the build tool

I ran the build tool with this command:

/home/.../UnrealEngine/Engine/Build/BatchFiles/RunUAT.sh BuildCookRun -project="/home/.../UnrealProjects/fouronenine/fouronenine.uproject" -nop4 -build -cook -stage -compressed -platform=Linux -clientconfig=Development -pak -archive archivedirectory="" -utf8output

The error message I received is:

ERROR: UBT ERROR: Failed to produce item: /home/.../UnrealProjects/fouronenine/Plugins/UnrealEnginePython/Binaries/Linux/libfouronenine-UnrealEnginePython.a
Total build time: 637.49 seconds (Local executor: 0.00 seconds)
Took 637.793242s to run mono, ExitCode=5
ERROR: Command failed (Result:5): /home/.../UnrealEngine/Engine/Binaries/DotNET/UnrealBuildTool.exe fouronenine Linux Development -Project=/home/.../UnrealProjects/fouronenine/fouronenine.uproject  /home/.../UnrealProjects/fouronenine/fouronenine.uproject -NoUBTMakefiles  -remoteini="/home/.../UnrealProjects/fouronenine" -noxge -NoHotReload -ignorejunk. See logfile for details: 'UnrealBuildTool-2018.04.02-09.33.55.txt'
(see /home/.../Library/Logs/Unreal Engine/LocalBuildLogs/UAT_Log.txt for full exception trace)
AutomationTool exiting with ExitCode=5 (5)
RunUAT ERROR: AutomationTool was unable to run successfully.

The error that is thrown is:

In file included from /home/.../UnrealProjects/fouronenine/Plugins/UnrealEnginePython/Intermediate/Build/Linux/B4D820EA/fouronenine/Development/UnrealEnginePython/PCH.UnrealEnginePython.h:115:
In file included from /home/.../UnrealProjects/fouronenine/Plugins/UnrealEnginePython/Source/UnrealEnginePython/Private/UnrealEnginePythonPrivatePCH.h:50:
/home/.../UnrealProjects/fouronenine/Plugins/UnrealEnginePython/Source/UnrealEnginePython/Private/Wrappers/UEPyFSoftSkinVertex.h:15:2: error: unknown type name 'FSoftSkinVertex'
FSoftSkinVertex ss_vertex;
^
/home/.../UnrealProjects/fouronenine/Plugins/UnrealEnginePython/Source/UnrealEnginePython/Private/Wrappers/UEPyFSoftSkinVertex.h:22:39: error: unknown type name 'FSoftSkinVertex'; did you mean 'FFinalSkinVertex'?
PyObject *py_ue_new_fsoft_skin_vertex(FSoftSkinVertex);
^~~~~~~~~~~~~~~
FFinalSkinVertex
Runtime/Engine/Classes/Components/SkinnedMeshComponent.h:273:43: note: 'FFinalSkinVertex' declared here
void GetCPUSkinnedVertices(TArray& OutVertices, int32 InLODIndex);

Not sure if this is relevant...
https://forums.unrealengine.com/development-discussion/c-gameplay-programming/1446998-what-happened-to-uskeletalmesh-in-ue-4-19

I'll try pulling new code again.


Provide a function to read standard input other than line-by-line

We are developing a language server written in ballerina. We are using stdio to connect with VSCode. According to language server protocol [1], it sends the following message
Content-Length: ...\r\n
\r\n
{
	"jsonrpc": "2.0",
	"id": 1,
	"method": "textDocument/didOpen",
	"params": {
		...
	}
}

We are not able to read the rest of the JSON file but only the first line that says “Content-Length”. This is because ballerina only provides a way to read line by line in readln. Please provide a function to read content by length or by character.
[1] https://microsoft.github.io/language-server-protocol/specifications/specification-current/#contentPart

@mindula, I'm looking into other languages to come up with a design for this use case. Will you be able to share how other language server implementations handle this scenario?

@mindula @manuranga , WDYT about this workaround?.
import ballerina/io;
import ballerina/lang.value;

public function main() returns error? {
    string fullContent = "";
    string delta = io:readln("");
    fullContent = delta;

    while delta != "" {
        delta = io:readln("");
        fullContent += delta;
    }
    json j = check value:fromJsonString(fullContent);
}

The issue here is not reading multiple lines. Issues is, there is no way to read anything less than a line, eg read char by char or to read given number of bytes.
If we use your above code it will get stuck at second io:readln("") and wait forever, since server is not ending that line (ie no new line char at the end).

IINM, your requirement is to read the standard input partially without pressing the "enter" key.

I tried to implement a similar use case in Java, but it seems, those kinds of implementations are highly platform-dependent and not portable[1, 2].
If you could share some samples, I can look more into this.
[1] https://www.darkcoding.net/software/non-blocking-console-io-is-not-possible/
[2] https://stackoverflow.com/questions/1066318/how-to-read-a-single-char-from-the-console-in-java-as-the-user-types-it

I assume these restriction only apply if we write using a console, in our use case the input is piped.
Maybe simple System.in.read works in this case, please check by piping in some input eg: cat myfile.txt | java -jar myBal.jar and reading char by char.

I assume these restriction only apply if we write using a console, in our use case the input is piped.
Maybe simple System.in.read works in this case, please check by piping in some input eg: cat myfile.txt | java -jar myBal.jar and reading char by char.

@manuranga, I tried a sample and it is working as expected when we use a pipe as you mentioned. We can provide an API to support that. I'll come up with a design and proceed with this task.

Here, we need to provide APIs to cover the following cases:

Read a character from standard input
Read n number of characters from standard input

There are two possible approaches to provide these use cases as per my initial research.
The existing read API is:
string content = io:readln(any a);

Approach 01
Here, the default case is reading a character. If n > 1 then it will return a string.
string|'string:Char content = io:readChars(any a, int n = 1);

Approach 02
'string:Char content = io:readChar(any a);
string content = io:readNChars(any a, int n);

A slight inconsistency is there with readln API. But I guess this will be a better option when we consider other file APIs.
@daneshk @jclark, please provide your feedback on this.

Just to clarify, in our use-case, we only know the amount of bytes not chars. The proposed solution still works for us since we can read char-by-char and concat, but we'll not be able to use the multi char version.

@manuranga, just to clarify. If you have a file myfile.txt and you know the number of bytes to read, isn't that possible to use the following APIs that are already in I/O?.
io:fileReadBytes(string path) returns readonly & byte[]|io:Error;
io:fileReadBlocksAsStream(string path, int blockSize = 4096) returns stream<Block, io:Error?>|io:Error

cat myfile.txt was just an example, in reality it's will be a program, eg vscode-client | java -jar my-ls-server.jar.

If you need byte related APIs, then we can change the aforementioned approaches as follows:
Approach 01
byte[]|byte content = io:readBytes(any a, int n = 1);

Approach 02
byte content = io:readByte(any a);
byte[] content = io:readBytes(any a, int n);

Note, returned byte content can be set as readonly values.

This will work for us, but even the single char one is enough for us. So please take a decision thinking about what is best from the stdlib point of view.

Specifying number of characters doesn't make much sense to me. The underlying system call will take a number of bytes, and you don't know how many bytes a given number of characters will need.

If you need byte related APIs, then we can change the aforementioned approaches as follows:
Approach 01
byte[]|byte content = io:readBytes(any a, int n = 1);

Approach 02
byte content = io:readByte(any a);
byte[] content = io:readBytes(any a, int n);

Note, returned byte content can be set as readonly values.

Here, I meant n as the number of bytes as they already know it.

I don't understand the any a parameter.
I strongly dislike  io:readBytes(any a, int n = 1) with a default of 1. It will not be obvious to a reader that io:readBytes(a) means io:readBytes(a, 1).

Here, the any a parameter is influenced by the existing read API
string content = io:readln(any a);

any a value is the default message that is being printed to the console before the reading.

Arghhhh. That is ghastly! Where did that come from?

There's two different meanings for console io:

in Unix terms, /dev/tty - not redirected; always connected to the terminal that a command-line user is interacting through

stdin/stdout/stderr - for a command-line user when not redirected, same as 1, but can be redirected to a file

These MUST NOT be mixed up.
io:println is doing console IO type (2)
A function that takes a prompt (which should be a string) and reads input makes sense for console IO type (1). This would be for getting input interactively from a user. This needs an option to turn off echo (for reading passwords).
@shafreenAnfar  io:readln needs fixing ASAP

@jclark thanks for your input. We will get it fix as soon as possible and will ship with the next release.

A possible approach to handle such use cases is as follows:
Console Read/Write
// Read from the console. These APIs will block till the user presses enter.
string line = io:consoleReadLine();
string passwd = io:consoleReadPassword();

// Write to the console, write line.
io:consoleWriteLine(io:Printable... values);

Stdin/Stdout/Stderr operations
Currently we have these APIs.
// Write to stdout and stderr
io:fprint(io:stdout|io:stderr , string msg);
io:fprintln(io:stdout|io:stderr, string msg);

Need to implement the following APIs.
// Read from stdin
string line = io:fscanLine()
byte b = io:fscanByte();
byte[] bArr =io:fscanBytes(int size);

@jclark, please provide your feedback on this.

-1 to both those.
console stuff shouldn't be in the io module.
Using scan is not appropriate: C fscanf is reading formatted data, but your scan is not doing this.
Using the f prefix is not appropriate: the f in the print case is there because the first argument is a file descriptor.

For reading a line from stdin, I would do just:
string line = io:readln();

I think the reading bytes feature should be part of a general solution for reading and writing to file descriptors (at least stdin/stdout/stderr).


BUG: Improve Get in touch div

Is there an existing issue for this?

[x] I have searched the existing issues

What happened?
Get in touch div is not looking as much good, the transparent view makes it less attractive and an image is also not visible

It should be redesign.
Record

[X] I agree to follow this project's Code of Conduct
[x] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Hi, I noticed this issue is focused on improving the "Get in Touch" section. I’d like to add a few more observations:
There’s a broken image labeled ctc1 in the form.
The form input fields lack sufficient padding and alignment.
The "Address" section could benefit from better spacing and alignment using Flexbox or Grid.
I’d love to help with these improvements as part of GSSoC. Please let me know if I can take this up or assist in implementing the fixes.


Cannot initialize TextFormField with initialValue (property marked final)

When creating a new TextFormField there is no way to specify the initial value in the constructor.
How are we supposed to set the initial value of a textFormField in a Form?
See FormField.initialValue

The value is maintained by the TextEditingController.  TextFields (like some other widgets, like Animation) have a Stateful component, for maintaining presentation state (like what text is entered but not committed, where the cursor is, etc.)  TextEditingController is that state object.
TextEditingController can be created with an initial value, there is even a helper contructor for that case:
https://docs.flutter.io/flutter/widgets/TextEditingController/TextEditingController.fromValue.html
When you move from having the FormField auto-create its own TextEditingController to creating one yourself, you need to make sure to hold the TextEditingController off of some object which has an appropriate lifetime for that text state.  Typically this is a parent StatefulWidget's State, but it can even be higher up or static depending on what behavior you want to have when navigating away from that scene of widgets and back again, etc.

Thanks, I was not aware of this.

can we place our database value in initialvalue in textformfield in flutter

@mishubham did you find any solution?

final TextEditingController _textFieldController = TextEditingController();
@override
void initState() {
  _textFieldController.text = "4444";
  super.initState();
}

Take a look on How to Supply an Initial value to TextField Widget in Flutter?


Update welcome dashboard with real charts

Only gitbase based charts:

Repository Count
Languages
Commits Evolution
Repositories by number of files
Number of commits per day of the week
Compliance score
Missing files
Files missing summary

All of them transfered manually from demo.
I put all of them on the same dashboard without splitting into tabs
because there are just few of them. We can re-arrange when we have more.
Fix: #102
Signed-off-by: Maxim Sukharev<EMAIL_ADDRESS>Screenshot from dataset ~/go/src/github.com/src-d/

Screenshot from dataset with few repos:

Known problem:

with a large number of repositories, some of the charts will timeout for the first time. Refreshing the page will re-trigger the calculation and the charts will be displayed at the end.
with a huge number of repositories and weak hardware, some of the charts may timeout permanently

@smacker that's already a good baseline, what's the best workflow so that could add others and change these later?

Actually might be better to merge this already, and we make further the changes in a new PR.

what's the best workflow so that could add others and change these later?

run srcd-ce with this change
add new charts / modify current dashboard
export it into json
replace json file in the repository & PR

Was this one exported from the UI? I noticed that exporting from the CLI produces much smaller json files, with much less metadata.
My guess is that since we are also importing from the CLI, it will be less likely to run into problems if we also export from the CLI (docker-compose exec sourced-ui superset export_dashboards)

FYI this is the current WIP dashboard mockup with charts/queries we have added (outlined in green), all pure gitbase so far: https://docs.google.com/presentation/d/1PFUbWZiSKae2VEQclswvW02B-BlHIyNk-PeFupXXVjo/edit#slide=id.p

@carlosms if I export it using cli I get:

Only 2 charts work.

Hm... It happened only after deleting & uploading. With a clean state works ok. Re-pushed.

Can you please take a look at the travis failing tests?

they are failing because travis for some reason can't start postgres and does something wrong with mysql. It is unrelated to changes in this PR.
You can see the same problem in other recent prs as well.


feat: revamp UI for docs

Description
Revamp UI for nextra docs, specifically landing page.
Possible solution(s)

Additional context

@anuragxxd you mentioned removing nerd docs, we can add those docs to ELIXIR's Documentation Hub, in the language specific section for {java,type}script but most of the docs in this are related to monorepo, so idk if we should remove it from there.
Maybe we could remove it from nextra and link it in readme though!


Pseudo-TTY with New-SSHSession

Is there a way to enable pseduo-tty when creating a New-SSHSession ?

Sadly the library I’m using for the module does not support it.

On Aug 17, 2015, at 11:04 AM, Warsame Bashir<EMAIL_ADDRESS>wrote:
Is there a way to enable pseduo-tty when creating a New-SSHSession ?
—
Reply to this email directly or view it on GitHub https://github.com/darkoperator/Posh-SSH/issues/68.

Thanks for the prompt reply.

I believe support for this was added recently to the core library so it should be doableSupport creating shell without PTY · Issue #1418 · sshnet/SSH.NETgithub.comSent from my iPhoneOn Jul 17, 2024, at 7:25 AM, p-gen @.***> wrote:
Do you think there is an alternative to forcing the allocation of a pseudo terminal?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: @.***>

Hi,
thanks for answering.
What I would like is to be able to run a script using Invoke-SSHCommand with the equivalent of “ssh -t” to make sure the remote shell has a tty allocated as some programs need it.

Yes, I think so, but it's a slightly more complex solution for non-interactive commands that still require a terminal.

Problem is that invoke-sshcommand does not create a TTY is has the same behavior as Plink where it executes a given binary with the parameters passed and returns the output.

It's a real shame, so I'm going to use New-SSHShellStream even if it complicates things. Thanks again.


[FR] - Make cardano-api/gen a normal library again due to haskell.nix incompatibility.

Internal/External
External When trying to depend on the cardano-node/cardano-api/gen lib, the ghcWrapper somehow doesn't include the lib in the shell environment. This makes it not really usable as a dependency.
Area
Other   Any other topic (Delegation, Ranking, ...).
Describe the feature you'd like
I would like the gen library to be a normal toplevel library as long as all the bugs with sublibs aren't fixed.
Describe alternatives you've considered
The alternative is fixing haskell.nix and/or cabal
Additional context / screenshots
Also see this issue on haskell.nix

It's not possible to make library cardano-api:gen a separate package because test-suite cardano-api:cardano-api-test depends on library cardano-api:gen which in turn depends on library cardano-api.
test-suite cardano-api:cardano-api-test and library cardano-api are defined in the same package cardano-api, so cabal requires that the gen package exist in the same package as well.
Unfortunately, this means fixing haskell.nix and/or cabal is the only viable option.

Unfortunately, this means fixing haskell.nix and/or cabal is the only viable option.

Improving haskell.nix seems like a good thing!

@MangoIV , @eyeinsky I'm interested to know if this resolves the issue: https://github.com/input-output-hk/cardano-node/pull/4970

Afaict yes.

Can you point to where the library is used?  Is it for consumption via an srp entry in a cabal.project or something else?

I tried it with both srp and nix extra hackage. Neither do worth. Sublibs just don't work. At least for Haskell.nix devShells. See the attached issue. Thank you.

@MangoIV  Instead of making cardano-api/gen a top level library, will adding a .cabal file for the generators fix your issue? I'd rather not add another library to the cardano-node repo.

I will close this issue as I don't depend on this anymore. I still want to note that one probably wants to abstain from using public sublibs as they're not fully supported by cabal yet and haskell.nix cannot deal with them for that reason and stuff will break.


Update history-of-nodered.md

Description
This fixes the error from https://github.com/FlowFuse/website/pull/1620
Related Issue(s)

Checklist

[x] I have read the contribution guidelines
[x] I have considered the performance impact of these changes
[ ] Suitable unit/system level tests have been added and they pass
[ ] Documentation has been updated

Fixed in: https://github.com/FlowFuse/website/pull/1716


added pagination input text on ellipsis click

Hi,
Thanks for this awesome package.
I've modified the pagination section of dataTable to add the "go to page number" feature on user ellipsis click. In other word, when user clicks on ellipsis span on pagination, an input field is presented to the user to take a page number.
It is also nice to add some style for the input field to make it more elegance. Here is a simple one:
.ellipsis input{
            padding: 5px;
            max-width: 5em;
            text-align: center;
        }

That's one massive diff. Github won't even render it :).
Is it something like what the input paging plug-in does? That is the place to send plug-in pull requests. Going to close this one as this is just a dist. repo.

Ok, thanks.
I'll try to make it as plugin.


install problem

hi,all
i want to try this algrithm on virtual machine and my system is ubuntu 16.04.
I had installed: MXNet@(commit 62ecb60) and Cython, opencv-python==<IP_ADDRESS>, easydict==1.6 as required. My python version is 2.7.12. But i came across this problem when i run init.sh
Traceback (most recent call last):
File "setup_linux.py", line 56, in
CUDA = locate_cuda()
File "setup_linux.py", line 44, in locate_cuda
raise EnvironmentError('The nvcc binary could not be '
EnvironmentError: The nvcc binary could not be located in your $PATH. Either add it to your path, or set $CUDAHOME
how can i fix this problem? is it possible to run the demo on virtual machine?

Our code require a GPU to run, please install cuda and I doubt that a VM can do this.


Disable test methods explicitly

A test that was not attempted (disabled / excluded), should not show up
as a passing test.

I extracted this from some other PR but i don't remember why i did that :)
we agreed to use SkipException and i don't want to introduce a new convention just yet.
let me drop this PR and reopen when i have more arguments to steer into some specific direction


[BUG] Problem Building Python Bindings

Describe the bug
Following the minimal python readme steps exactly, I'm getting the following error when trying to run the example:
  File "teaser_python_ply.py", line 5, in <module>
    import teaserpp_python
  File "/home/zhsh/miniconda3/envs/teaser_test/lib/python3.6/site-packages/teaserpp_python/__init__.py", line 1, in <module>
    from .teaserpp_python import *
ModuleNotFoundError: No module named 'teaserpp_python.teaserpp_python'

I've built the c++ version and it works fine on the example, and in the output after cmake ... I see that python bindings have been built:
[100%] Built target pmc
-- Found PythonInterp: /usr/bin/python (found suitable version "3.8.10", minimum required is "3.6")
-- pybind11 v2.4.3
-- TEASER++ Python binding will be built.
-- Python Interpreter Version: 3.8
-- Configuring done
-- Generating done
-- Build files have been written to: /home/zhsh/catkin_ws/src/TEASER-plusplus/build
[ 60%] Built target pmc
[ 86%] Built target teaser_registration
[100%] Built target teaserpp_python

I've also read the previous related issues #131 and #92
Installed Dependencies
Ubuntu 20.04
cmake version 3.16.3
conda 4.13.0
To Reproduce
Execute the commands
sudo apt install cmake libeigen3-dev libboost-all-dev
conda create -n teaser_test python=3.6 numpy
conda activate teaser_test
conda install -c open3d-admin open3d=<IP_ADDRESS>
git clone https://github.com/MIT-SPARK/TEASER-plusplus.git
cd TEASER-plusplus && mkdir build && cd build
cmake -DTEASERPP_PYTHON_VERSION=3.6 .. && make teaserpp_python
cd python && pip install .
cd ../.. && cd examples/teaser_python_ply
python teaser_python_ply.py

Any updates on the issue? I would love to help if I can

I have the same issue, did you resolve it?

@EdoardoPassera can you try the latest develop branch? Thanks

I still get the same error.

me too, did any handsome guy solved this problems?

I have met same problem under Window 10 MinGW toolchain. Here's some steps I took to make it work.

Under Windows, pybind11 tends to generate .pyd instead of .so, we have to change the line 11 at ${ProjectRoot}/python/setup.py.in, from package_data={'': ['*.so]} to package_data={'': ['*.pyd']}. This step make sure that the compiled *.pyd or *.so file can be copied to pip installation path.

Make sure that the FOUND PYTHON INTERPRETER is the one as the same as you use for running py scripts. Otherwise, import can't find the entry of the Library. Pay attention if you use Anaconda Env, pybind11 may use base python.exe, but you may run scripts in your established envs.

Copy the generated *.dll or *.so files under ${ProjectRoot}/build/teaser, and other path (e.g. pmc-build, tinyply-build) to the same path where the py scripts placed, in case that the searching path for dynamic load libraries is not correctly configured.


Perlin noise texture class

https://stackoverflow.com/questions/26549480/how-do-i-declare-a-static-field-in-a-struct-in-rust
was useful in creating a static constant.
https://stackoverflow.com/questions/26033976/how-do-i-create-a-vec-from-a-range-and-shuffle-it
was useful in creating a vector of integers and randomly shuffling it.

At this point, rust doesn't support creating constant structures in impl blocks. So, have to resort to lazy statics as per this thread's suggestion: https://users.rust-lang.org/t/defining-a-const-variable-with-sqrt/24972/2

Rendering two perlin-textured spheres:

Corrected image for perlin-textured spheres (changed the vfov for camera to 20.0):

Render perlin textures with trilinear interpolation

Perlin noise textures with Hermitian Smoothing

Perlin noise texture with higher frequency

Image generated after Vector Smoothing

Added turbulence

Rendering perlin noise marble textures


Client may die and there's no way to restart it

The change in https://github.com/aki017/slack-ruby-gem/pull/23 assumes that the main entry point. For example https://github.com/dblock/slack-gamebot is actually a Puma web server that hosts a Slack client.
As a quick experiment I modified it to do:
      def start
        EM.run do
          ws = Faye::WebSocket::Client.new(@url)

          ws.on :open do |event|
          end

          counter = 0

          ws.on :message do |event|

            data = JSON.parse(event.data)

            puts data['type']

            if counter == 3
              puts "exit!"
              EM.stop
              return
            else
              counter += 1
            end

Here's what happens:
~/source/slack-gamebot/dblock (master)$ foreman start
10:03:33 web.1  | started with pid 48042
10:03:33 web.1  | Puma starting in single mode...
10:03:33 web.1  | * Version 2.11.0 (ruby 2.1.6-p336), codename: Intrepid Squirrel
10:03:33 web.1  | * Min threads: 0, max threads: 16
10:03:33 web.1  | * Environment: development
10:03:34 web.1  | warning: parser/current is loading parser/ruby21, which recognizes
10:03:34 web.1  | warning: 2.1.5-compliant syntax, but you are running 2.1.6.
10:03:34 web.1  | * Listening on tcp://<IP_ADDRESS>:5000
10:03:34 web.1  | Use Ctrl-C to stop
10:03:34 web.1  | I, [2015-05-27T10:03:34.994152 #48042]  INFO -- : Welcome 'gamebot' to the 'dblock' team at https://dblockdotorg.slack.com/.
10:03:35 web.1  | hello
10:03:35 web.1  | I, [2015-05-27T10:03:35.379089 #48042]  INFO -- : Successfully connected to https://dblockdotorg.slack.com/.
10:03:35 web.1  | presence_change
10:03:35 web.1  | presence_change
10:03:37 web.1  | user_typing
10:03:37 web.1  | exit!
# just sits here

Need a way to bounce EM.

@dblock Couldn't you just put a while loop here?  When start ends, the thread here just ends.  Your process is just hanging because the slack thread has ended, but not the main thread running rack.

You're right. I fixed it in https://github.com/dblock/slack-gamebot/commit/617051c61eaf8328c4167f9c11b510defec3399b, and was able to test it using #stop in https://github.com/aki017/slack-ruby-gem/pull/26. Note that you have to re-create a client because you need a new websocket URL.


How to Power Off (new Heltec LoRa Tracker PCB)?

I have the newer Heltech LoRa Tracker board. This has just a P and R button. I can't figure out how to power down the tracker. It's in a case with a battery so disconnecting the power is not really a good option. I did try to use the Power Off option in the menu. This is tricky to reach and also does not allow me to power the board off. Ideas?

As this board has no switch or power management chip: pressing 3 times USER button will make the board go into deep sleep for over a month
This is the only available way to make it use very little current (0,7mA/H)
To power off for real, use an external switch

Perfect thanks. I noted that the reset switch will wake it up again. Thank you for your work on these devices.


Addition of Docker File

Hello,
I noticed the repo does not have a 'dockerfile', If needed may I add one to the repo?

Hi, there are two Dockerfile files in the repo:
1 - The shogun SDK that is used for testing. The image contains all required libraries to build shogun and most optional libraries.
https://github.com/shogun-toolbox/shogun/blob/develop/configs/shogun-sdk/Dockerfile
2 - The shogun Dockerfile. It contains the last major release of shogun, but this is quite outdated now.
You can also find these on https://hub.docker.com/u/shogun

Thank you @gf712 for clarification.
So is any updating required/expected in point 2?

No, it will be updated when we release again

Ok, thank you for looking into it.


Add rediness and liveliness props to the deployment spec

Description:
Readiness probes are designed to let Kubernetes know when your app is ready to serve traffic. When new configuration applies, Pod will pull the base image according to the image pull policy and restart the pod.
Here the issue is if a user sends a request while restarting k8s will response as connection refuse error. Need to avoid this from adding readiness and liveliness props into the deployment spec.
Suggested Labels:
Type/Improvement
Suggested Assignees:

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

We introduced an internal API to check the readiness of the pod.
The following is the configuration used in deployment.yaml file
 readinessProbe:
     httpGet:
         path: /healthz
         port: 9201
     initialDelaySeconds: 5
     periodSeconds: 2

This is available from 1.2.0-m1


unverified breakpoint icon is not strong enough

When creating a breakpoint in an active debug session and this breakpoint cannot be verified, the resulting icon is almost invisible:

The problem is that the icon's stroke is too thin.
In previous versions the icon had a thicker stroke:

I suggest that we restore the previous icon.

Agree that it is a bit too light.
I do not just want to take the previous icon since the color and style are different.
Thus it would be great if @chryw could tackle this
Also fyi @misolori

Not quite as strong as in the 'before' above, but stronger. Marking verified.


Exclude blocked content from search results

List of blocked outpoints is at api.lbry.io/file/list_blocked

@lyoshenka , what is the output of endpoints?

Its a list of outpoints. An outpoint uniquely identifies a file. Its a combination of txid and nout of the publish for the file. It has the form txid:nout. For example, the outpoint for the current winning claim on one is 8e5d6945d238e8508520364d5eae7885b0db15d77d6d5cffee3f860dc13f0b45:0

If possible, I would like lighthouse to return the quantity of blocked URLs (and possibly the list itself).

Yup, this is tested and now live on production! Closing this issue!


pin AMQP dependency to prior to event leak release, resolves RackHD/RackHD#262

closing in favor of #155 (sorry @BillyAbildgaard )


Toolbar issue

How to make the text align to be center default

Not enough info. Can you post a screenshot or at least code example?

@Rameshmariappan
<ReactQuill         style={{           textAlign : 'center',         }}       />


Constraint and Or Graders

Adds a constraint grader functor which takes in a Constraint and a Standard grader. When ran, it first checks to make sure the Constraint grader gives a point value above the threshold. If it does then it has the same results as the Standard grader. If not, then it awards zero points and gives some message.
The or grader functor simply takes in two graders and takes the maximal score of the two. Potentially useful for if we want to accept off-by-one errors (so we don't have to make the original grader handle this in some way).

Yeah, that's what I was thinking! :)

Sooo what's the tldr
I think to add to the conversation, I would prefer the constraint grader to not be just simple multiplication, since that is not how 150 grading works (at least not anymore). We deduct x amount of points, and if you reach below 0, we add it back so you get 0.

Oh in the case of memofib, we explicitly say we deduct 10 points (out of 15) for violating constraints, so would also like it if we could make it so that we can change the number of points we deduct

i think the tl;dr is that ideal the prod graders are rewritten to take in a score "combining" function which can be used to handle all these different use cases

Will this be worked on sometime in the near future?

It seems like it needs a review?

(I think you both should have the ability to review/merge, if it looks good to you!)

Also, should this be closed as superseded by #7, or both make sense?


Align with Bash design guide

@dramasamy
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Let's wait for this pull request - https://github.com/MicrosoftDocs/azure-docs-pr/pull/235304

Done via https://github.com/MicrosoftDocs/azure-docs-pr/pull/235304

#please-close


Earlgrey: prim_ram_1p.sv doesn't work in verilator

Requires prim_generic_ram_1p.sv which issue is described in issue https://github.com/alainmarcel/uhdm-integration/issues/207

ram module is parsed correctly by Surelog. It was added to flow with Surelog
https://github.com/antmicro/verilator/pull/71


In query responses, change result key to results

In the draft spec (0.3.0-alpha), the results are in candidates arrays that are themselves in a results array. To be honest, I find it very confusing, but at least I am sure that the results key is in plural form, not in the singular form "result".

you are right. I implemented now a check against the result scheme as well and adapted the response accordingly.

Can this be closed then?


device disconnected

[ ] I have read the FAQ.
[ ] I have searched in existing issues.

Environment

OS: windows
scrcpy version: 1.14
device model: samsung galaxy s8
Android version: android 9

Describe the bug
the device connects correctly, but after leaving it off (screen locked) when trying to unlock it, the screen goes black and the window closes, the command window stays open
this happens to me being connected by wifi

See #1382.

More precisely:https://github.com/Genymobile/scrcpy/issues/1382#issuecomment-627902816
(Forget about the "error" printed, it's not related and is harmless.)

sorry, but I know almost none of this What option would I have to disable?