feat: add econstructor tactic.

Add econstructor tactic.

Can you remove the corresponding syntax line from Mathlib/Mathport/Syntax.lean?

Can you remove the corresponding syntax line from Mathlib/Mathport/Syntax.lean?

I was going to do that actually but then I noticed that there were other tactics such as ApplyWith that still have their syntax lines in Mathlib/Mathport/Syntax.lean.  I can do a follow up cleanup PR to clean the other tactics so others aren't confused about this as I was.

I don't see the applyWith tactic in Mathport.Syntax. There is applyWith', but this is a different thing. Are there any other tactics that you saw that are duplicated?

Actually, I think I missed the single quote on the ApplyWith' so disregard the previous comment.

@digama0 ahh we are replying at the same time, as I replied before I saw your comment.  Yes, it was my mistake.

I don't see the applyWith tactic in Mathport.Syntax. There is applyWith', but this is a different thing. Are there any other tactics that you saw that are duplicated?

Not that I was aware of...I was just trying to say that if I was going to do a cleanup branch I would look for any others.  Cheers.

bors merge


/skin command visual bug

Describe the bug
The /skin command displays an incorrect skin to other players.
To Reproduce
Steps to reproduce the behavior:

Log in using a gamemaster character.
Log in using a normal character.
Use the /skin command followed by a valid monster number. For example, type "/skin 503" to select the Panda skin.
Observe the gamemaster character model while controlling the normal character.

Expected behavior
The gamemaster character model should match the skin chosen using the /skin command.
Current behavior
The gamemaster character model always appears as the Bull fighter, regardless of the skin number used. When using "/skin 0", the character detransforms entirely.
Screenshots

This bug has the same origin as #358. Fixed as well.
Thanks for reporting :)


Spaces in installation path are properly handled by nvm install, but not by nvm use

Issue:
When installing nvm-windows using the nvm-setup.exe to a path with spaces, e.g. "C:\Programs X", the command nvm install latest successfully runs. nvm use latest will return a non-zero exit-code and produce the output
exit status 1: 'C:\Programs' is not recognized as an internal or external command, operable program or batch file.

How To Reproduce:

Install nvm using nvm-setup.exe (version <IP_ADDRESS>, launched with admin privileges) to a path containing spaces
Open a terminal window and type nvm install latest (this should work)
Type nvm use latest
See error

Expected Behavior:
nvm selecting the correct version
Screenshots:

Notes: The screenshot is taken from a terminal running MINGW64. The bug is also present in Command Line (Windows 11) and Power Shell.

Desktop:

OS: Windows 11 22H2

Open and Edit file C:\Programs X\nvm\settings.txt
from
root: C:\Programs X\nvm

to
root: C:\Programs\ X\nvm

Check your environment PATH

Check your environment PATH

Sorry, what exactly should I check? Installing at a path without spaces works like a charm (which I ultimately did).
Nevertheless, this should be fixed at some point.

Having the same issue on Win10 Pro Build 19044.

Hi, I've encountered the same issue; solved running cmd as Administrator

I suggest to fix it by wrapping arguments with an escape function in nvm.go:
func run(name string, arg ...string) (bool, error) {
	c := exec.Command(name, escape(arg)...)
	// ... rest of function
}

func escape(args []string) []string {
	for index, arg := range args {
		if strings.Contains(arg, " ") && !strings.HasPrefix(arg, "\"") {
			args[index] = "\"" + arg + "\""
		}
	}
	return args
}

As far as I have seen this should work with current usages, but I did not test it.

It turned out, that spaces in the arguments was not the problem. The problem was that exec.Command(...) refuses to execute commands with spaces in it. Quoting the command did not work as well. But setting Dir in type Cmd worked.
PR follows.


error: attribute '"1.79.0"' missing

I'm getting this error for rust version 1.79.0. Any help would be appreciated.
https://github.com/FuelLabs/fuel.nix/actions/runs/10397407695/job/28793121145?pr=144

According to your flake.lock, it is using a historical commit of rust-overlay 8ce81e71ab04a7e906fae62da086d6ee5d6cfc21 on 2024-03-14 which is earlier than rust 1.79.0. Please nix flake update to the latest.


Widget inspector tests have become incredibly flaky

The scroll test in particular seems to crash the tester shell intermittently across all platforms and on luci as well. (This is the underlying issue that was causing the updated test reporter to silently fail, since we failed to report the tester shell crash).
06:02 +3709 ~63: C:\Windows\Temp\flutter sdk\packages\flutter\test\widgets\widget_inspector_test.dart: WidgetInspector scroll test
06:02 +3709 ~63 -1: loading C:\Windows\Temp\flutter sdk\packages\flutter\test\widgets\widget_inspector_test.dart [E]
  Shell subprocess crashed with unexpected exit code -1073740791 after tests finished.
  Test: C:\Windows\Temp\flutter sdk\packages\flutter\test\widgets\widget_inspector_test.dart
  Shell: C:\Windows\Temp\flutter sdk\bin\cache\artifacts\engine\windows-x64\flutter_tester.exe
  dart:async/stream_controller.dart 595:43                     _StreamController.addError
  dart:async/stream_controller.dart 862:13                     _StreamSinkWrapper.addError
  package:stream_channel/src/guarantee_channel.dart 140:14     _GuaranteeSink._addError
  package:stream_channel/src/guarantee_channel.dart 131:5      _GuaranteeSink.addError
  package:flutter_tools/src/test/flutter_platform.dart 596:29  _FlutterPlatform._startTest.<fn>
  ===== asynchronous gap ===========================
  dart:async/zone.dart 1053:19                                 _CustomZone.registerUnaryCallback
  dart:async/runtime/lib/async_patch.dart 77:23                _asyncThenWrapperHelper
  package:flutter_tools/src/test/flutter_platform.dart         _FlutterPlatform._startTest.<fn>
  package:flutter_tools/src/test/flutter_platform.dart 802:26  _FlutterPlatform._startTest
  ===== asynchronous gap ===========================
  dart:async/zone.dart 1053:19                                 _CustomZone.registerUnaryCallback
  dart:async/runtime/lib/async_patch.dart 77:23                _asyncThenWrapperHelper
  package:flutter_tools/src/test/flutter_platform.dart         _FlutterPlatform._startTest
  package:flutter_tools/src/test/flutter_platform.dart 477:36  _FlutterPlatform.loadChannel
  package:flutter_tools/src/test/flutter_platform.dart 432:46  _FlutterPlatform.load
  ===== asynchronous gap ===========================
  dart:async/zone.dart 1053:19                                 _CustomZone.registerUnaryCallback
  dart:async/runtime/lib/async_patch.dart 77:23                _asyncThenWrapperHelper
  package:test_core/src/runner/loader.dart                     Loader.loadFile.<fn>
  package:test_core/src/runner/load_suite.dart 98:31           new LoadSuite.<fn>.<fn>
  ===== asynchronous gap ===========================
  dart:async/zone.dart 1045:19                                 _CustomZone.registerCallback
  dart:async/zone.dart 962:22                                  _CustomZone.bindCallbackGuarded
  dart:async/timer.dart 52:45                                  new Timer
  dart:async/timer.dart 87:9                                   Timer.run
  dart:async/future.dart 174:11                                new Future
  package:test_api/src/backend/invoker.dart 399:21             Invoker._onRun.<fn>.<fn>.<fn>

cc @jacob314

see also https://github.com/flutter/flutter/issues/27161 which makes this harder to debug

Do you have any more details on the flakes? In general particularly outside fo --track-widget-creation mode these tests aren'

It appears like none of the tests ever fail, they just bring down the entire tester shell

I've gotten this output out of some repeated runs:
Shell: [ERROR:c:\b\s\w\ir\k\src\flutter\shell\common\shell.cc(178)] Dart Error: Dart_SetNativeInstanceField: invalid
index 0 passed in to set native instance field
Shell: [ERROR:c:\b\s\w\ir\k\src\flutter\shell\common\shell.cc(178)] assertion failed
!LogIfError(Dart_SetNativeInstanceField(wrapper, kPeerIndex, 0))

(triage): @jacob314 are you still looking at this one? Is this still flaky?

Let me test it out. I suspect the flake is probably no longer reproducible.


[Agent] (Potentially) improve Editing using diff

What is the problem that this fixes or functionality that this introduces? Does it fix any open issues?
Currently, agents sometimes struggle with edit_file skill that requires them to input line numbers - this is challenging to current foundation models, even the strongest (e.g., instance astropy__astropy-14995 using CodeActAgent v1.6-no-hint and gpt-4o - check visualizer for entire history https://huggingface.co/spaces/OpenDevin/evaluation).
The agent generated legit code to fix the issue but accidentally inserted the code into a docstring and triggered a syntax error.

Give a brief summary of what the PR does, explaining any non-trivial design decisions
This PR implements diff-based similar to aider and resolves https://github.com/OpenDevin/OpenDevin/issues/2222.
TODO: run the implementation on SWE-Bench-Lite to see if it improves score.
Other references

The "replace first in file" approach should be implemented independently of the existing methods as those cover their own cases.

We are aiming for an evaluation-driven approach: We will only consider merging this if this alone improves the SWE-Bench score; If it degrades the score in any way, we definitely won't merge it.

This PR is currently blocked by me, I need to finish evaluation, hopefully today.

@tobitege this is blocked by your "request changes" if you could please take a look I think it's ready other than that!

@tobitege this is blocked by your "request changes" if you could please take a look I think it's ready other than that!

Done now.
@xingyaoww so the extra newline is still there (see tests). Might keep an eye out for that.

@tobitege I already fixed that in https://github.com/OpenDevin/OpenDevin/pull/2722! I'll try to get this one ready and evaluated soon!

Oh, got it. I lost track 😁


Don't reload offline scene

Check if we're already in the offline scene before calling ClientChangeScene to the offline scene.

:tada: This PR is included in version 3.17.5 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Change some main APIs to return ValueTask instead of Task

This covers FindAsync(), AddAsync() and the value generators (3 different commits).
Part of #15184

Note that in order to reference ValueTask in .NET Standard 2.0, I needed to depend on System.Threading.Tasks.Extensions. In the dotnet-core nuget feed, the version of that package is 4.6.0-preview1-27018-01 (possibly because the package is no longer needed in .NET Core 3.0?). For now I've added a dependency on the nuget.org 4.5.2 version, but that may not be the right thing.

ef.Tests seem to be failing because it's not finding the new dependency assembly (or an assembly version mismatch...). Will look again tomorrow, if anyone has any experience please feel free to lend a hand :)
      Microsoft.EntityFrameworkCore.Tools.WrappedXunitException : System.IO.FileNotFoundException: Could not load file or assembly 'System.Threading.Tasks.Extensions, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51' or one of its dependencies. The syste
m cannot find the file specified.
      File name: 'System.Threading.Tasks.Extensions, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51'
         at Microsoft.EntityFrameworkCore.Design.Internal.DbContextOperations.GetContextTypes()
         at Microsoft.EntityFrameworkCore.Design.OperationExecutor.GetContextTypesImpl()
         at Microsoft.EntityFrameworkCore.Design.OperationExecutor.OperationBase.<>c__DisplayClass4_0`1.<Execute>b__0()
         at Microsoft.EntityFrameworkCore.Design.OperationExecutor.OperationBase.Execute(Action action)

The other test failure is just general database flakiness it seems.

ef.Tests seem to be failing because it's not finding the new dependency assembly

Does using the preview version of the package help?

Perhaps it is unrelated, but does using the preview version of the package help?

I don't see how but I'll give it a shot tomorrow.


Prometheus

Can I use this plugin with Prometheus metrics? Is there any way here? any idea?

Posting here in case it helps anybody else out, it is possible to use this plugin with a Prometheus datasource.
Prometheus
This is based on using Grafana v9.0.2.
Metric Structure
It is best to have a counter metric where the value will be used to provide the thickness of the links and the labels can be used to define the columns. You can name the columns anything.
metric{from="<from>", to="<to>"} <value>
metric{from="<from>", to="<to>"} <value>
metric{from="<from>", to="<to>"} <value>

Setting up your Query
The simplest query is where you use sum by, providing each of the columns that you want to have groups for as well as the metric name.
sum by (labelOne, labelTwo) (metric)

You may of course apply filters to the metric, and have more columns depending on your data set.
Under "Options" the "Format" value must be set to "Table".
Next, head to "Transform", select "Organize Fields" and tap on the eye icon next to "Time" in order to hide this default column. You may also use this opportunity to reorder columns by dragging the handle on the left, or renaming columns using the input box on the right.
✨

Thank you so much @Sherlouk, this instruction was great!
Nice results :-)

I can't get solved. Is it possible to give proper Example. I means screenshot of the Metrics name and Transform set for the given Sankey Diagramme.
Thanks

@Sherlouk could you post another example? Trying to get it to work but am having a hard time.
My scenario is, lets say i have one metric which tells me my home power consumption
and i have 10 others from devices that consume power.
i should be able to represent these 10 devices as sources and my home power as destination ?


Grounding to only leaf nodes in the ontology

I wanted to raise this issue for discussion: Eidos currently only grounds to the leaf nodes of the YAML ontology (e.g., wm.yml) that it is provided. Whether or not this yields good results certainly depends on the specifics of the ontology, but my feeling is that for the WM ontology specifically, this can produce suboptimal results. Consider, for instance, the sentence, "conflict causes a decrease in living conditions". Both conflict and living_conditions are in the ontology, but they both subsume a number of more specific leaf concepts like unsanitary_condition under living_condition. This results in any mention of "living condition" to be grounded to the more specific unsanitary_condition. If there is no technical barrier to grounding to non-leaf entries, I think it would generally improve the results by allowing the system to capture the level of detail at which concepts are described in text more faithfully. Any thoughts?

This has since been fixed so I'm closing the issue.


Typo: Using readline from homebrew

Changes
ruby-build: use readline from homebrew
to
ruby-build: using readline from homebrew
ruby-build: use libyaml from homebrew
to
ruby-build: using libyaml from homebrew
And
ruby-build: use openssl from homebrew
To
ruby-build: using openssl from homebrew

Thanks! Could you also fix the other lines in the format of “Using … from Homebrew” please? So that it's consistent.

Sure

@mislav could you take a look now?

Thank you!


build.zig: let user decide how to set build mode + fix linker warning

This should delegate the responsibility of calling standardReleaseOptions and setting the build mode of the *LibExeObjStep step to the caller, especially since this might not be the process by which one wants to determine the build mode.
Also changes hides getSrcDir to enforce usage of srcdir, and asserts that the file is in fact inside a directory.
Also fixes a linker error, by passing null to the root_src param of addStaticLibrary instead of the header file.

@InKryption I'm not an expert on Zig build system but I'm merging those changes.


When and where getUserMedia is called ?

At which point the method navigator.mediaDevices.getUserMedia is called internally ? is there any way to catch error of this internal call and do something on success and error callbacks ? If not, is there a way to call it manually and attach incoming stream to connection?
What i want is open room and establish all sockets only after user has granted permissions to camera + microphone.

Ye now this provokes a bug in Android Chromium >=85.x.
The getUserMedia is called when you execute .openOrJoin, I don't know if is possible to avoid this behavior by replacing with a manual getUserMedia.
In fact, I see that this bad behavior is by bad constraints but also this not returns an error, because Muaz modifies the
MediaStream object with other data after getUserMedia.


Upgrade Cloud Pak for Business Automation to 23.0.1

Is your feature request related to a problem? Please describe.
The latest release of CP4BA is out, so this repository should be tested with that version for clean install and for upgrades.
Describe the solution you'd like

Upgrade the artifacts in this repository to support a clean installation of CP4BA 23.0.1
Inspect the release notes for potential fixes to workarounds in the repository, then remove unnecessary and/or harmful workarounds.

Describe alternatives you've considered
None, the requirement is pretty narrow.
Additional context
Add any other context or screenshots about the feature request here.

:tada: This issue has been resolved in version 0.29.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


showing image in lightbox, swaps image & caption

Before

During

After

I wasn't really pleased with Fluidbox, and found a more suitable replacement in SimpleLightbox. It looks nicer, and also lacks the bugs.

I wasn't really pleased with Fluidbox, and found a more suitable replacement in SimpleLightbox. It looks nicer, and also lacks the bugs.

👍

👍


Feature/fpv 63

Modified the way error messages were being displayed on the register page.

After successful registration, the username & password that user has entered are deleted
replaced alerts with labels to display instructions/success/error messages.
labels are hidden after a certain timeout

What is the point of clearing user input if we are about to switch page? Not that it's bad but just wondering.

What is the point of clearing user input if we are about to switch page? Not that it's bad but just wondering.

The user input remains persistent even after switching pages and coming back, thus clearing it is good idea

If and only if properly tested, I don't see an issue.

Yes, I have tested it, still we can have jaako test it once


Problems with scalable rendering auto-activation logic for serial engines

Describe the bug
I am not seeing VisIt automagically switch into SR mode when loading 32k x 32k  2D rect mesh from NIF via Xdmf reader. I worry that the polygon counting logic doesn't account for rectilinear meshes properly.
As an aside, maybe the NIF image data should really be loaded up as a bonified vktImageData object?
Well, hmmm...wait a minute. I was doing this with a serial (remote) engine. I think in serial, it decides never to do SR mode. I think that logic exists because it was assumed that serial engine and viewer are typically running on same host and we should never software render (in the serial engine) process when we can hardware rendering in the viewer.
But, in my case, I was running a remote serial engine (a very large dataset that I had pared down using SIL restriction) and I surely don't want all that data going over the network.
As an aside, the Information tab in SR mode dialog is supposed to update the count of polygons. When in SR mode, it is showing the count of polygons in the viewer, not the engine. It should show the polygone count from the engine.

You can watch what happens when I do this in the attached video.
https://github.com/visit-dav/visit/assets/5720676/b994a6a9-5f6d-4469-86de-b78ce2f6c3d3

Discussion Takeaways:

In this case parallel engine will work well.
We do want to explore if serial engine can use SR.
We may not be able to launch a parallel engine with 1 task b/c our launcher is too smart.


Calculate speed based on slide width

This is a (multiple allowed):

[ ] bug

[x] enhancement

[ ] feature-discussion (RFC)

Swiper Version: 5.3.0.

It would be awesome if it was possible to pass a function to the speed option which would calculate speed for each slide separately.
Recently I had to implement a slider of tags with constant, slow transition (like flash news bar at the bottom of a TV screen). So I used speed=4000, autoplay=true, freeMode=true and slidesPerViews="auto", loop=true. And the tags were automatically flowing really nicely. But there was an issue that they would speed up and slow down depending on the width of each tag. So I basically implemented autoplay myself by calling slideNext() after each slide transition and passing it the speed based on current slide width.
It would make my life much easier if I was able to pass a function to the speed parameter and use it to return different values based on current slide width. Not sure if I explain myself correctly.

I assume the function could take the current slide as a parameter.
speed: function(slide) {
    return parseInt(slide.style.width) / 2;
}

In the example above we are sliding to the next slide based on half of the current slide's width.
Is this what you meant @jedlikowski ?
I've made a commit for this in my fork:
https://github.com/LeeConnelly12/swiper/commit/2cb8ca6f4a02c346381a408761f20dc568a5be8d
Would like to know if this change looks ok @nolimits4web ?


Add another exercise in react hooks

Create a folder inside content/react/hooks and name it the next integer.
Example 5 in this case, since the last number is 4

Inside that folder create 2 files: notes.md and app.js

Write your example app in app.js and export it as default

const App = () => {
  return <>Hello world!</>
}

export default App

Write the description of your exercise in notes.md.
## The hello world exercise

> Summary: Display `hello world` on the screen!

@mithi i want to do this. could you please assign me?

Hi Muhammad!
Thanks for your interest.
Please let me know what exercise it is you thought of including
and explain why you think such exercise is important enough to be added.
What important React principles does the exercise demonstrate?
Thank you!
On Mon, Oct 4, 2021 at 2:31 AM Muhammad Tanveer ul Haq <
@.***> wrote:

@mithi https://github.com/mithi i want to do this. could you please
assign me?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mithi/epic-react-exercises/issues/130#issuecomment-933002222,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMX2FOFRFXCWXXTEFZLQADUFCOOTANCNFSM5FHON36Q
.


Show hashtag besides content #94

hashtags and content redesign.

Don´t forget to delete the branch afterwards :)


add "webview" in when clause contxt

I'm working on an extension that has a webview to edit some local configuration(a json file). Now I would like to add an icon on editor/title on this webview tab. Click this icon will open the json file in editor. But I only want this icon show up on the webview's tab.
This is basically the same feature in VS Code's Open Settings(JSON) when we are on Open Settings(UI) tab.
Currently I'm trying to add menus.editor/title in package.json but I didn't find a proper when clause context to check if this tab is webview or something like that.
Please advise if this is a reasonable request or there is some other ways to do it.

See these questions:
https://stackoverflow.com/questions/54917748/how-do-i-contribute-a-command-to-the-vs-code-explorer-title-bar-or-command-palet/54917749#54917749
https://stackoverflow.com/questions/52142903/is-there-any-contexts-when-specified-webview-panel-is-active-in-vscode-extension/52174989#52174989

Thanks! It works well


Fix pg_autoscaler won't scale due to overlapping roots

There was no CephBlockPool defined for the .mgr pool, therefore it was using the default crush rule (with no deviceClass specified) The CephObjectStore & CephFilesystem metadataPool does not define the deviceClass (it is only set in the dataPools).
To fix this Create a CephBlockPool CR for the .mgr pool with the same poolSpec as the default CephBlockPool. Also set the deviceClass for the metadataPool in CephObjectStore and CephFilesystem.
BZ-https://bugzilla.redhat.com/show_bug.cgi?id=2253013

/test ocs-operator-bundle-e2e-aws

/cherry-pick release-4.16


Add section to guide explaining how to use pcli in threshold mode

Is your feature request related to a problem? Please describe.
Now that threshold support has landed, we should add a section to the guide describing how to use it.  This should probably be done along with shipping the software.

We have this now in the guide, closing as done


import all features from stylable-react-component

This project has some already-in-use features that should move here.
as they are very POC level, @nadavov should provide some spec to work with as the "definition of done"

the mixin currently in in stylable-react-component handles react stylable integration.
example usage:
import css from 'mycss.css';

@stylableMixin(css)
class MyComponent{

or
import css from 'mycss.css';

const myComp = stylableMixin(()=>

,css)

the css imported is a module created using: https://github.com/wixplosives/stylable/blob/master/src/runtime.ts
by activating the mixin a number of submixins are activated:

replace cssStates: replaces cssStates property in createElement, spreading it into the internal createElement properties.
this should be done using a utility from the runtime:

...css.cssStates({  ...props.cssStates }),

"css" being the imported runtime stylesheet

replace className: replaces classNames in createElement.  using css.get(localName)
append className to root
copy data-attributes

this plugin uses copyToRoot.
in addition it replaces  createElement (for all jsx nodes)

classNames  to their namespaced version.
the name spaced version is reachable at css.get(propFromCreateElement.className)
cssStates to their namespaced data-attribute version
this can be done using css.cssStates(propFromCreateElement.cssStates) which returns data-attributes object

the css module can be genrated using:
create("root", "zagzag", {}, "", "")

this task's spec is not defined by stylable-react-component regression tests in the current project.


Added support of Conductor library

👀

Sorry for the late..code seems nice but I've been considering how to integrate with Coordinator. May be it's better to separate as another library? 🤔

@hotchemi Hi! Yes, it will be also perfectly as another library

Reopened the issue and let me close the PR for the time being. I suppose we need more considerations for this.


309554027 lab1

↓↓↓ Notice ↓↓↓

[ ] Check your base repository and branch are belong to you.
[ ] The PR name need to include student ID and which lab assignment

template: ${student id} ${which lab}
e.g. 0756110 lab0

↑↑↑ Notice ↑↑↑

If you ensure the notice items is done, you can remove them.

Following section is optional, depend on you.

Description
a few message for overall goals of this pull request
Todos

[ ] what feature to do ?
[ ] what bug to fix ?
[ ] what enhancement to improve ?

@GrassLab/ta

Please check the osc2021/README.md and update your README.md following this template


visual studio 2017 style code folding?

can you add visual studio style code folding?
so lets say I have a foldable code region like this..
    if a == b then
       --stuff
    end

instead of it looking like this
    if a == b then
    end

it looks more like this..
    if a == b then ...

this would work with while, for and tables too

As far as I'm aware, this is entirely controlled by vscode, and extensions can't override the behaviour. I would check if an issue exists on the vscode repo, or otherwise think about opening one.


Started mining early.

This makes more energy available for upgrading early on.
Change also makes containers repairs more important during mining as miners wont be on full fledged bodies yet.

This Pull Request has been merged by GitConsensus.
Vote Totals

Yes
No
Abstain
Voters

3
0
0
3

Vote Breakdown

User
Yes
No
Abstain

TheRealMaxion
✔

ButAds
✔

ViezeVingertjes
✔

Vote Results

Criteria
Result

Has Quorum
True

Has Votes
True


Remove horizontal scroll from template wizard dialog

Resolves #1786

Can one of the admins verify this patch?

@AppInventorWorkerBee ok to test


Reset to default color

Resetting color so everything on your console wouldn't be cyan.

yep
I had a modified PS1 variable that do it automatically and forgot about the code...
thx


error

can you run this profile through c2lint? it looks like your profile might be too large

Thank you, the modified c2lint can pass. But the Trojan generated by default cannot go online with C2.

thats doesnt tell me much. can you provide more details? can you test with default malleable profiles? post the profile you generated?

Found a problem, if the request is a post request, the default generated profile, http-get set verb "post"

Also, can you add some regular C2 evasion monitoring, such as set sleep_mask "true";

I could, but thats outside the scope of this tool. This tool is meant to make it easier to craft custom malleable profiles based on existing traffic. The cobalt strike docs should be referenced for other options since there is more depth to them and their use cases/opsec implications imo.

Recently, I am learning the operation of cobalt strike profiles. Are there any articles that recommend me to learn? I will thank you very much.

Yes, the Fortra Malleable PE docs:
https://hstechdocs.helpsystems.com/manuals/cobaltstrike/current/userguide/content/topics/malleable-c2-extend_main.htm should help for the other malleable options such as sleep mask etc.

Ok, at present, I have written a successful profile according to your tool. The problem of http-get set verb "post" does exist, you can fix it ASAP.

Im not too sure what you mean, can you give me the steps to reproduce this? Last I checked it was working with POST requests :P

It will generate "http-get", where the configuration defaults to" set verb 'POST';"

hi sorry for the late reply, was busy. But I dont believe this is an issue. http-get does not imply HTTP GET requests, as per the Malleable C2 spec.


Added tests for each function.

The expected behavior regarding the expected exception has been checked with the System.Linq classes.

Hi,
My first ever attempt at a pull request in someone else's repo. Hope I did this right.

Wow! Fantastic work!
Do you happen to work in QA? This is some heavy destructive testing! 🙂

Yay, I am glad you like it. :) I am a developer actually, but you know they say that a dev has to be decent in testing, and a QA engineer has to be exceptional in it.


Node id must be a string to handle the connection.

Describe the Bug
Hello , while i was stuck on implementing nodes with endpoints i figured out that while i was giving to the node the id as a number , it wont connect with other nodes and the path would be disabled. To make this work i used to turn id .toString().
Your Example Website or App
No response
Steps to Reproduce the Bug or Issue

Expected behavior
i figured out that while i was giving to the node the id as a number , it wont connect with other nodes and the path would be disabled. To make this work i used to turn id .toString().
Screenshots or Videos
No response
Platform

OS: [e.g. macOS, Windows, Linux]
Browser: [e.g. Chrome, Safari, Firefox]
Version: [e.g. 91.1]

Additional context
No response

As documented, a node id has to be a string and can't be a number.
So I don't see what this issue is supposed to address?
Is this a feature request to support numbers? 🤔

This is solved. Ids need to be strings, because we are also attaching them as data attributes and there we can't distinguish between numbers and strings.


Add codespace configuration

Node 18, yarn v1
Not sure how to auto-select the max cores automatically, though (but that's minor).
Reviewing with Niko on 1 more thing...

Ah, forgot to explain.
This configures our Codespace VM to a node-18 version so that we can start coding/building straight away from the browser.
Seems useful to start something from scratch when dev instances are taken.  But there is CORS issue, though.


Consider adding FromAsyncWithPostCancelNotifications

The ReactiveUI project uses Rx's ability to wrap a Task<T> as an IObservable<T>, but there's a capability they have wanted for a long time that Rx currently does not offer: the ability both to cancel the underlying task and to observe its progress after cancellation.
There's no built in way to do this, because the mechanism by which you cancel a Task<T> wrapped in an IObservable<T> is to unsubscribe (i.e. to call Dispose on the IDisposable returned by Subscribe). The problem with this is that unsubscribing tells Rx that you don't want to receive any more notifications.
https://github.com/reactiveui/ReactiveUI/issues/2153 describes a problem in ReactiveUI that can be attributed to this missing Rx capability.
Although this is not insurmountable, as the PR at https://github.com/reactiveui/ReactiveUI/pull/3556 shows, it would would be useful if Rx provided an out-of-the-box solution to this. We could do this by providing something like the FromAsyncWithPostCancelNotifications extension method defined by that PR:
https://github.com/reactiveui/ReactiveUI/pull/3556/files#diff-41fb8479ae2ebc5305bca72fa394bd5bdf8d13b1806c76ca65a6878e8954f075R103-R122
As @ChrisPulman points out at https://github.com/reactiveui/ReactiveUI/pull/3556#issuecomment-1575278125 we would need a set of overloads to cover all the variations.
FromAsyncWithPostCancelNotifications might not be the ideal name for this, but we can give that some thought later.

@idg10 Any progress on the new Function, amongst the thousand other things you have going on?
Perhaps FromCancellationAsync would be a relevant name.


asus flow z13 : Really low CPU (12900H) performance in linux

Hi
I have Problem with ASUS flow z13 CPU performance, same as this seems CPU power limit to 15w in linux !!
really-low-cpu-12900h-performance-in-linux-manjaro

Which version of throttled are you using?

use AUR : throttled-git.git

Can you give us the output of --debug?
https://github.com/erpalma/throttled#debug

Seems CPU max frequency liming to 1.2GHZ !

Can you give us the output of --debug?
https://github.com/erpalma/throttled#debug

It is very hard from me to help you if you do not provide the outputs of the command.

trsave1.log
kernel param GRUB_CMDLINE_LINUX_DEFAULT="cpuidle.off=1 idle=poll processor.ignore_ppc=1 processor.max_cstate=0 intel_idle.max_cstate=0 msr.allow_writes=on udev.log_priority=3"
cpupower-freqInfo.txt
inxi.txt

trsave1.log

According to my understanding from this log it seems like throttled is working fine. It seems like the CPU is being limited by the governor setting.
Do you have  power-profiles-daemon installed and running?
sudo systemctl status power-profiles-daemon
I do not really know how to controll it in KDE, but in GNOME I have the option to set it to "Power saver" "Balanced" and "Performance" setting. Could you set it to "Balanced" or "Performance"? On "Power saver" it also limits my 12700H to ~1,15Ghz.
I would also suggest that you upgrade to the latest kernel 5.19. 5.15 does not have a good support for Alder Lake.

No, TLP or power-profiles-daemon installed ...

No, TLP or power-profiles-daemon NOT installed ...

Ok, can you in that case check that the system check the --monitor output?
https://github.com/erpalma/throttled#monitoring
It should show us reasons for throttling in real time. If everything is on "OK" then the CPU is not throttling and the frequency is limited by the governor.

[I] Detected CPU architecture: Intel AlderLake-P/H
[I] Trying to unlock MSR allow_writes.
[I] Testing if undervolt is supported...
[I] Testing if HWP is supported...
[I] Loading config file.
[I] Starting main loop.
[D] Undervolt offsets: CORE: 0.00 mV | GPU: 0.00 mV | CACHE: 0.00 mV | UNCORE: 0.00 mV | ANALOGIO: 0.00 mV
[D] IccMax: CORE: 160.00 A | GPU: 33.00 A | CACHE: 0.00 A
[D] Realtime monitoring of throttling causes:
[AC] Thermal: OK - Power: OK - Current: LIM - Cross-domain (e.g. GPU): OK  ||  VCore: 767 mV - Package: 10.6 W - Graphics: 0.5 W - DRAM: 0.0 W - Total: 11.0[AC] Thermal: OK - Power: OK - Current: LIM - Cross-domain (e.g. GPU): OK  ||  VCore: 780 mV - Package: 8.9 W - Graphics: 0.0 W - DRAM: 0.0 W - Total: 8.9 W[AC] Thermal: OK - Power: OK - Current: LIM - Cross-domain (e.g. GPU): OK  ||  VCore: 779 mV - Package: 9.0 W - Graphics: 0.0 W - DRAM: 0.0 W - Total: 9.0 W[AC] Thermal: OK - Power: OK - Current: LIM - Cross-domain (e.g. GPU): OK  ||  VCore: 779 mV - Package: 9.0 W - Graphics: 0.0 W - DRAM: 0.0 W - Total: 9.1 W[AC] Thermal: OK - Power: OK - Current: LIM - Cross-domain (e.g. GPU): OK  ||  VCore: 780 mV - Package: 9.0 W - Graphics: 0.0 W - DRAM: 0.0 W - Total: 9.0 W[AC] Thermal: OK - Power: OK - Current: LIM - Cross-domain (e.g. GPU): OK  ||  VCore: 766 mV - Package: 8.9 W - Graphics: 0.0 W - DRAM: 0.0 W - Total: 9.0 W[AC] Thermal: OK - Power: OK - Current: LIM - Cross-domain (e.g. GPU): OK  ||  VCore: 766 mV - Package: 9.1 W - Graphics: 0.0 W - DRAM: 0.0 W - Total: 9.2 W[AC] Thermal: OK - Power: OK - Current: LIM - Cross-domain (e.g. GPU): OK  ||  VCore: 767 mV - Package: 9.0 W - Graphics: 0.1 W - DRAM: 0.0 W - Total: 9.1 W[AC] Thermal: OK - Power: OK - Current: LIM - Cross-domain (e.g. GPU): OK  ||  VCore: 767 mV - Package: 8.8 W - Graphics: 0.2 W - DRAM: 0.0 W - Total: 9.0 W

current limit ??

Any update on this?
It's still happening for me on Arch (Kernel 6.6.1).
Same exact behaviour, current limit under any load. stuck at ~10W


Partners View Pdfs

Summary
When a distribution is created on the bank side a pdf is generated and emailed to the partner agency. The bank also has the ability in the distributions to view the pdf of the distribution details. The partners, while able to see the items they were distributed, are not able to view that pdf after that initial email. We need to make it available to them as needed.
Below is a screenshot of the prior and upcoming distributions where they should be able view the pdf as poorly mocked out:

Criteria for Completion

This should be a bulleted list of items that, if satisfied, would result in an acceptably complete Pull Request. Be as specific as you need to be. For example:

[ ] A view pdf button/link is available in the partner dashboard prior and upcoming distributions sections, and in the partner distributions page.
[ ] When clicked it generates the same pdf that is generated on the bank side of that specific distribution
[ ] Tests!

When clicked it generates the same pdf that is generated on the bank side of that specific distribution

We definitely want to re-generate a new PDF and not save+serve the original? Is there any possibility that the subsequently generated one wouldn't match the first?

I think we'd want to generate a new one each time. Especially on the upcoming distributions as it is possible those may change leading up to when he distribution happens. Also, we already have the logic to print generate the pdfs on the bank side for partners we just need to pull that over to the partner base side.
I obviously haven't looked into it or how difficult it will be to use that existing logic but we have it in a couple places so I think it should be doable. ie:

and


[ENG-5517] URL Encoding test issues

Purpose
Fix test failures caused by furl upgrade
Changes

QA Notes
When furl was upgraded it started escaped everything that goes inside query params (for intance urls).
Thorough check of theses cases is required, mostly how external services interpret escaped urls.
Documentation

Side Effects

Ticket
ENG-5517

Merged and closed by d9b757b


No Data differentiate from zero

Update Downtown Berkeley and Berkeley/Telegraph at 5am to be "N/A" in occupancy on VPP and VPP_Web_GeoDB on RDS

changed to NULL (bc field is numeric after it was uploaded)


question: sometimes, for ext4 partition, no sparse compression is done

Hello,
I am using fwup succesfully for a while, it's great! But now I noticed something with the size of the rootfs inside the fwup file. I have a local build, using yocto, and one on my buildserver(linux mint 19.3, now upgraded to linux mint 20). The ext4 rootfs is actually 1,6GB. And on my local build, the file inside the fwup archive is 1,2 GB, because sparse handling I think, so that is OK. But I build the same software on my buildserver, and there fwup archive contains the whole 1,6GB file. So I have no explanation for it. I tried the sparse check option, and it says sparse is enabled. I'm using 1.5.1 still, but tried with 1.8.4 and is still the same issue. So what can I check further? What could be a reason?
The resulting fwup files are actually both about the same size, but the problem is that I'm using the pingpong system and with the version from the buildserver I get problems upgrading, while with the version on my laptop it works perfectly. The reserved partition size is large enough to contain the full 1,6GB raw file, so I'm a bit puzzled.
I know it's a fuzzy issue, but maybe a hint could help me already :)
Best regards,
Tom,

Hi Tom,
The sparse file feature is depends on what filesystem that you're using on the build server. The way it works is that fwup asks the OS where the holes in a file are and then saves that info to the archive.
About a year ago, I started seeing some CI servers that have filesystems which were pretty unreliable with reporting the holes in sparse files. I never investigated since all of my work since then has been with compressed filesystems that didn't need the feature. I "think" that the common thread was that if the CI server was a virtual machine or a dedicated machine, it worked and if it was in a Docker container, it didn't. I'm not sure, though, and that doesn't make sense to me. My first guess would be to see what filesystem is in use on your build server and see if that provides a hint.

Hello,
Thanks for the reply. It seems somehow, the file is not sparse, and it's on a pure ext4 filesystem. I am using yocto, and it seems on my buildserver, I have the file with a timestamp, and one without the timestamp. I discovered a tool called fallocate, which is on linux, and you can recreate a file to be sparse. So I tried with "fallocate -d rootfs.img -v" on the rootfs.img with a timestamp, and it only detected 32mb of sparse space. And I tried with the non-timestamped version, and that could detect 400 mb of sparse space. When I now run fwup using the non-timestamped version, then fwup detects correctly that it is a sparse file and the fwup data directory contains the 1,2 GB file instead of a 1,6 GB file.
So that hopefully fixes it for me now 🤞

Hi, it seems that fixed it for me.

Interesting. Thanks for following up.


emscripten: 3.1.39 -> 3.1.40

Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Backport request: https://github.com/llvm/llvm-project/issues/63040

Turns out trying to rebuild other packages on top of 3.1.39 actually requires the LLVM backports. Updated this to address that issue accordingly, though the backport is now dependent on #235219.

LLVM backport request was approved. This should make it into 16.0.5. I'll leave this as draft until a new LLVM point release is cut, which can be a followup to #235219.

Updated #235219 to llvm 16.0.5.

Closing in favor of #235624.


REFACOTR: Function Extraction _zero_wt_init(name)

Thank You!

@autoih
You are owner of some files modified in this pull request.
Would you kindly review the changes whenever you have the time to?
Thank you very much.

https://github.com/tensorflow/addons/blob/f03f26f8c16097cfbc36739135ea3a38b6287896/tensorflow_addons/metrics/f_scores.py#L114
This refactoring is already in use by F-Score. In my view, I think this is useful. :smile:

I meant if this pattern It Is used/useful in multiple place or just in multipleetrics could this function go in utils.py instead of redefine It multiple time?

Oh, I see. It seems like a better way.

Have you seen https://github.com/tensorflow/addons/pull/2013#pullrequestreview-451060161?

I meant only to see https://github.com/tensorflow/addons/blob/master/tensorflow_addons/metrics/multilabel_confusion_matrix.py#L84


Universal 2 (Apple Silicone) binary support

A detailed description of the proposed feature
Homebrew will need to support both "classic" binaries compiling for x64 and Universal 2 binaries which will likely include both a x64 binary and an ARM binary (Will add data about Universal 2 as more information about it becomes available).
At the same time ARM based Macs with MacOS 10.16 "Big Sur" will also support installing iPad OS apps natively (most likely only through the app store).
Starting with a similar approach as Linuxbrew which does not have bottles but compiles ARM applications might be a good idea.

The motivation for the feature
With the announcement of using Apple's own ARM-based A12/A13 SoC chips for new Macs during Apple's WWDC 2020 Keynote they will also introduce a new binary format called "Universal 2" which will contain Intel and ARM compiled binaries. As with the transition from PowerPC to Intel Apple will migrate towards their so-called "Apple Silicone" within 2 years while starting to deliver hardware by the end of 2020.

How the feature would be relevant to at least 90% of Homebrew users
Apple will transition to only sell ARM-based Macs within two years so all users will need it at some point.

Hi.
We are aware of this. The first pull requests have already started flowing in, and we are working on the support for the new hardware.
Please let the maintainers work on this. Your issue does not really help, as there is nothing actionnable here.
We will make a release once we are ready.

See @mistydemeo 's PRs https://github.com/Homebrew/brew/pull/7793 and https://github.com/Homebrew/brew/pull/7794.


Improve PoE performance by introducing MJPEG encoding for streams

This PR introduces performance improvements by using MJPEG encoding before sending the frames over XLink and then decoding them on the host using OpenCV imdecode method.
Due to the current encoder limitations, not all of the preview streams have the encoding enabled:

:x: nn_input (the passthrough output from the nn)
:heavy_check_mark: color
:heavy_check_mark: left
:heavy_check_mark: right
:heavy_check_mark: rectified_left
:heavy_check_mark: rectified_right
:x: depth_raw
:x: depth
:heavy_check_mark: disparity
:heavy_check_mark: disparity_color

One important change that had to be introduced is that now the color stream is taken from ColorCamera video output instead of preview (since only video output can be MJPEG encoded at the moment). Using this output, the preview window is not cropped and scaled to meet the nn input size, being full FOV instead - but we wanted to introduce this feature anyway in https://github.com/luxonis/depthai/pull/422. Issues reported in that PR will probably affect this PR also, but I think we can merge this PR and address them afterward, just so the demo is usable with PoE devices. CC @Luxonis-Brandon
Performance on a single stream can reach up to 28 FPS over ethernet (tested on -s left)

The more streams we request, the slower they will perform. At most, we can request up to 4 encoded streams, at which the performance is 10-12 FPS over Ethernet (tested on -s left right color disparity disparity_color)

Since disparity_color and depth are in fact the transformed versions of disparity and depth_raw respectively, they do not require a dedicated encoder and therefore can be used without increasing the resources usage on device - so running -s disparity disparity_color will use the same amount of resources as -s disparity, as all transformations are performed on host

@VanDavv looking good.
Do callbacks now get encoded MJPEG frames or already decoded frames?
And just as a cross check, try retesting with a known 1Gbit connection between the device and host to maybe lay out some expected performance numbers which could be added to readme, etc..

@themarpe callbacks receive decoded frames, in preview manager the frames are decided as soon as they arrive so access to raw mjpeg frames is limited (although if we'd like to e.g. add a functionality to redirect encoded frames to file, we are still able to add this feature)

Sweet, thanks both!

Working well for me.  So we may want to make the POE default to not show depth until we get that working via an encoded method.
That way the framerate will not be reduced by the unencoded depth.  We could just show say disparity instead, or just the RGB with physical location.
Thoughts?

A quick note @VanDavv - the overlay of depth information for the color stream is quite small:

Compare to the grayscale:

That said, I think we should merge, and refine over time.

One possible improvement for later (general, not related to this PR): the text drawn may not be easy (or not possible) to read, depending on the background image color. And filling solid color rectangles to draw over isn't that nice:

We could implement this trick of outlining the text, by double-drawing, first with a larger thickness (that would create the outline), and then the same text again with a smaller thickness and different color:
https://github.com/luxonis/depthai-experiments/blob/8385501/gen2-seq-num-sync/main.py#L133-L135
    cv2.putText(frame[c], text, (8,40), cv2.FONT_HERSHEY_DUPLEX, 1.5, (0,0,0), 8, cv2.LINE_AA)
    cv2.putText(frame[c], text, (8,40), cv2.FONT_HERSHEY_DUPLEX, 1.5, (255,255,255), 2, cv2.LINE_AA)

Yes, great idea - thanks for bringing that up.  We should totally do that for all text @VanDavv as that makes it easy to read no matter the background (black or white, or anything in-between).

Yes, that's a great idea! Will merge this PR since it helps in PoE usage and open a new one that addresses the comments above

Introduces the fixes / tweaks here - https://github.com/luxonis/depthai/pull/434 CC @alex-luxonis @Luxonis-Brandon


Epoll: Avoid redundant EPOLL_CTL_MOD calls (#9397)

Motivation
Currently an epoll_ctl syscall is made every time there is a change to
the event interest flags (EPOLLIN, EPOLLOUT, etc) of a channel. These
are only done in the event loop so can be aggregated into 0 or 1 such
calls per channel prior to the next call to epoll_wait.
Modifications
I think further streamlining/simplification is possible but for now I've
tried to minimize structural changes and added the aggregation beneath
the existing flag manipulation logic.
A new AbstractChannel#activeFlags field records the flags last set on
the epoll fd for that channel. Calls to setFlag/clearFlag update the
flags field as before but instead of calling epoll_ctl immediately, just
set or clear a bit for the channel in a new bitset in the associated
EpollEventLoop to reflect whether there's any change to the last set
value.
Prior to calling epoll_wait the event loop makes the appropriate
epoll_ctl(EPOLL_CTL_MOD) call once for each channel who's bit is set.
Result
Fewer syscalls, particularly in some auto-read=false cases. Simplified
error handling from centralization of these calls.

@njhill verified this commit is good to go... All test suites (even internal ones) pass with this one applied


Error in filter mode

Hi Abigail,
Thank you very much for your support in issue #30. I successfully finished analysis in a dataset, but newly encountered in another dataset during filter mode.
After aln mode has successfully finished in all files, run mode failed in some files (not all files) with the error messages attached below in the log file. Do you have any idea for a solution?
Best,
Rio
[Fri Dec  2 17:11:24 2022]
Finished job 798.
832 of 3223 steps (26%) done

[Fri Dec  2 17:11:24 2022]
rule taxonomize:
    input: /MYDIRECTORYFORFILE/filtering/XCSNH_20200131_A00904_IL100150648_N2UD-F09_L003_read_counts_and_mismatches.txt, /MYDIRECTORYFOREUKDETECT/EukDetect/eukdetect/count_primary_and_secondary_eukfrac.py, /MYDIRECTORYFOREUKDETECT/EukDetect/eukdb/busco_taxid_link.txt, /MYDIRECTORYFOREUKDETECT/EukDetect/eukdb/specific_and_inherited_markers_per_taxid.txt, /MYDIRECTORYFOREUKDETECT/EukDetect/eukdb/taxa.sqlite, /MYDIRECTORYFOREUKDETECT/EukDetect/eukdb/taxid_cumulativelength.txt
    output: /MYDIRECTORYFORFILE/filtering/XCSNH_20200131_A00904_IL100150648_N2UD-F09_L003_all_hits_table.txt, /MYDIRECTORYFORFILE/XCSNH_20200131_A00904_IL100150648_N2UD-F09_L003_filtered_hits_table.txt, /MYDIRECTORYFORFILE/XCSNH_20200131_A00904_IL100150648_N2UD-F09_L003_filtered_hits_eukfrac.txt
    jobid: 291
    wildcards: output_dir=/MYDIRECTORYFORFILE, sample=XCSNH_20200131_A00904_IL100150648_N2UD-F09_L003

OpenBLAS blas_thread_init: pthread_create: Resource temporarily unavailable
OpenBLAS blas_thread_init: RLIMIT_NPROC 30 current, 30 max
OpenBLAS blas_thread_init: pthread_create: Resource temporarily unavailable
OpenBLAS blas_thread_init: RLIMIT_NPROC 30 current, 30 max
OpenBLAS blas_thread_init: pthread_create: Resource temporarily unavailable
OpenBLAS blas_thread_init: RLIMIT_NPROC 30 current, 30 max
Traceback (most recent call last):
  File "/MYDIRECTORYFOREUKDETECT/EukDetect/eukdetect/count_primary_and_secondary_eukfrac.py", line 4, in <module>
    from ete3 import NCBITaxa
  File "/MYDIRECTORYFOREUKDETECT/.conda/envs/eukdetect/lib/python3.6/site-packages/ete3/__init__.py", line 47, in <module>
    import numpy
  File "/MYDIRECTORYFOREUKDETECT/.conda/envs/eukdetect/lib/python3.6/site-packages/numpy/__init__.py", line 142, in <module>
    from . import add_newdocs
  File "/MYDIRECTORYFOREUKDETECT/.conda/envs/eukdetect/lib/python3.6/site-packages/numpy/add_newdocs.py", line 13, in <module>
    from numpy.lib import add_newdoc
  File "/MYDIRECTORYFOREUKDETECT/.conda/envs/eukdetect/lib/python3.6/site-packages/numpy/lib/__init__.py", line 8, in <module>
    from .type_check import *
  File "/MYDIRECTORYFOREUKDETECT/.conda/envs/eukdetect/lib/python3.6/site-packages/numpy/lib/type_check.py", line 11, in <module>
    import numpy.core.numeric as _nx
  File "/MYDIRECTORYFOREUKDETECT/.conda/envs/eukdetect/lib/python3.6/site-packages/numpy/core/__init__.py", line 16, in <module>
    from . import multiarray
SystemError: initialization of multiarray raised unreported exception
/bin/bash: line 1: 34134 Segmentation fault      python /MYDIRECTORYFOREUKDETECT/EukDetect/eukdetect/count_primary_and_secondary_eukfrac.py --dbfile /MYDIRECTORYFOREUKDETECT/EukDetect/eukdb/taxa.sqlite --taxid_link /MYDIRECTORYFOREUKDETECT/EukDetect/eukdb/busco_taxid_link.txt --inherited_markers /MYDIRECTORYFOREUKDETECT/EukDetect/eukdb/specific_and_inherited_markers_per_taxid.txt --taxid_genelens /MYDIRECTORYFOREUKDETECT/EukDetect/eukdb/taxid_cumulativelength.txt --readcounts /MYDIRECTORYFORFILE/filtering/XCSNH_20200131_A00904_IL100150648_N2UD-F09_L003_read_counts_and_mismatches.txt --primarytab /MYDIRECTORYFORFILE/XCSNH_20200131_A00904_IL100150648_N2UD-F09_L003_filtered_hits_table.txt --eukfrac /MYDIRECTORYFORFILE/XCSNH_20200131_A00904_IL100150648_N2UD-F09_L003_filtered_hits_eukfrac.txt --alltab /MYDIRECTORYFORFILE/filtering/XCSNH_20200131_A00904_IL100150648_N2UD-F09_L003_all_hits_table.txt
[Fri Dec  2 17:11:25 2022]
Error in rule taxonomize:
    jobid: 291
    output: /MYDIRECTORYFORFILE/filtering/XCSNH_20200131_A00904_IL100150648_N2UD-F09_L003_all_hits_table.txt, /MYDIRECTORYFORFILE/XCSNH_20200131_A00904_IL100150648_N2UD-F09_L003_filtered_hits_table.txt, /MYDIRECTORYFORFILE/XCSNH_20200131_A00904_IL100150648_N2UD-F09_L003_filtered_hits_eukfrac.txt

RuleException:
CalledProcessError in line 185 of /MYDIRECTORYFOREUKDETECT/EukDetect/rules/eukdetect_eukfrac.rules:
Command ' set -euo pipefail;  python /MYDIRECTORYFOREUKDETECT/EukDetect/eukdetect/count_primary_and_secondary_eukfrac.py --dbfile /MYDIRECTORYFOREUKDETECT/EukDetect/eukdb/taxa.sqlite --taxid_link /MYDIRECTORYFOREUKDETECT/EukDetect/eukdb/busco_taxid_link.txt --inherited_markers /MYDIRECTORYFOREUKDETECT/EukDetect/eukdb/specific_and_inherited_markers_per_taxid.txt --taxid_genelens /MYDIRECTORYFOREUKDETECT/EukDetect/eukdb/taxid_cumulativelength.txt --readcounts /MYDIRECTORYFORFILE/filtering/XCSNH_20200131_A00904_IL100150648_N2UD-F09_L003_read_counts_and_mismatches.txt --primarytab /MYDIRECTORYFORFILE/XCSNH_20200131_A00904_IL100150648_N2UD-F09_L003_filtered_hits_table.txt --eukfrac /MYDIRECTORYFORFILE/XCSNH_20200131_A00904_IL100150648_N2UD-F09_L003_filtered_hits_eukfrac.txt --alltab /MYDIRECTORYFORFILE/filtering/XCSNH_20200131_A00904_IL100150648_N2UD-F09_L003_all_hits_table.txt ' returned non-zero exit status 139.
  File "/MYDIRECTORYFOREUKDETECT/EukDetect/rules/eukdetect_eukfrac.rules", line 185, in __rule_taxonomize
  File "/MYDIRECTORYFOREUKDETECT/.conda/envs/eukdetect/lib/python3.6/concurrent/futures/thread.py", line 56, in run
[Fri Dec  2 17:11:27 2022]
Finished job 352.
833 of 3223 steps (26%) done
Shutting down, this might take some time.
Exiting because a job execution failed. Look above for error message

Hi Rio,
I suspect this error is happening not directly because of eukdetect but because the number of requested threads is too high for the system you're running on. (This is suggested by the error OpenBLAS blas_thread_init: pthread_create: Resource temporarily unavailable, OpenBLAS blas_thread_init: RLIMIT_NPROC 30 current, 30 max). Try re-running again requesting fewer cores.

Hi Abigail,
Thanks for your quick response. Yes, I agree. But, I think the number of cores was as appropriate and it was working well in other samples until the error occurs in the specific samples. Could it still happen?
Best,
Rio

Hi Abigail,
Thank you very much for your advice. By using fewer cores, I've just confirmed my run was successfully finished. Sorry for the delay!
Best,
Rio


GroupBy fails on null keys

The following test fails with LINQBridge and passes with System.Core:

        [Test]
        public void GroupBy_NullKey()
        {
            var arr = new[] { "notnull", null };

            var groups = arr.GroupBy(s => s);
            foreach (var @group in groups)
            {
<EMAIL_ADDRESS>?? "[null]");
            }
        }

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 29 Apr 2011 at 9:29

Proposed solution:

Use as a backing store for Lookup<TKey, TElement> not Dictionary<TKey, ...> but
Dictionary<Wrapped<TKey>, ...>, where Wrapped<TKey> is some wrapper structure
that would never be null.

Original comment by<EMAIL_ADDRESS>on 29 Apr 2011 at 9:32

Here is a proof-of-concept implementation of this Wrapped<> structure and it's
equality comparer.

Original comment by<EMAIL_ADDRESS>on 29 Apr 2011 at 10:30
Attachments:

Wrapped.cs

Original comment by azizatif on 30 Apr 2011 at 1:47

Changed state: Accepted
Added labels: Milestone-Release1.3

Original comment by azizatif on 30 Apr 2011 at 2:10

Changed state: Started

Fixed in changeset c74eb37b988b[1]

[1]
http://code.google.com/p/linqbridge/source/detail?r=c74eb37b988be35a8a62b7032184
14dc889b5372

Original comment by azizatif on 30 Apr 2011 at 4:20

Changed state: Fixed

The Count property of the Lookup is now broken.
What do you have against wrapping dictionary key in the non-nullable structure?
This seems to be more "straight solution".

Original comment by<EMAIL_ADDRESS>on 30 Apr 2011 at 7:14

Please, take a look at my solution:
http://code.google.com/r/ilyaindustry-linqbridge-issue21/source/detail?r=00f4036
bafdf260f69e4910006c8c62b931ea18d

PS: Thanks for migrating to mercurial.

Original comment by<EMAIL_ADDRESS>on 30 Apr 2011 at 7:40

> The Count property of the Lookup is now broken.

Good catch.

> What do you have against wrapping dictionary key in
> the non-nullable structure

Nothing against it. I was trying to solve three problems (order keys for
GroupBy, null keys and move to Hg) at the same time. I have merged your version
(thanks) and will be adding back key order for GroupBy; it is required as per
the documentation:

“The IGrouping(TKey, TElement) objects are yielded in an order based on the
order of the elements in source that produced the first key of each
IGrouping(TKey, TElement). Elements in a grouping are yielded in the order they
appear in source.”

GroupBy internally relies on ToLookup.

Original comment by azizatif on 1 May 2011 at 10:12


Add maxHivePartitions Strategy to avoid scanning excessive hive partitions on partitioned table

Why are the changes needed?

The PR support this Strategy to avoid scan huge hive partitions  or unused necessary partition filter on partitioned table，It will abort the SQL query with thrown exception via scan hive table partitions exceed spark.sql.watchdog.maxHivePartitions, meanwhile show the related hive table partition struct to assist user to optimize sql according the given suggestion
How was this patch tested?

[x] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[x] Add screenshots for manual tests if appropriate

[x] Run test locally before make a pull request

cc @ulysses-you

@i7xh  now, we upload the scalastyle output log https://github.com/apache/incubator-kyuubi/pull/1095, you can debug with that log by yourself

thanks, merging to master

@i7xh don't forget this https://github.com/apache/incubator-kyuubi/pull/1086#discussion_r707103617 : )

hi @i7xh  seems your git user.email is not your github email, so your github account can not be seen as contributor. Can your check it ?

hi @i7xh seems your git user.email is not your github email, so your github account can not be seen as contributor. Can your check it ?

OK, I have already add my company email in my account


Add prior predictive example notebook

Add prior predictive plot.
Fix the double scaling bug.
Typo from https://github.com/pymc-labs/pymc-marketing/pull/784

📚 Documentation preview 📚: https://pymc-marketing--787.org.readthedocs.build/en/787/

When adding the prior-predictive step, the processing is wrong:

Would the self.y be getting scaled twice?
If you remove the sample_prior_predictive lines, this doesn't happen, right?

The bug was fixed in https://github.com/pymc-labs/pymc-marketing/pull/787/commits/b765b38c89fb2606df90c5aada6c84519e990733

Is it then the combination of build_model and _data_setter? As the sample_posterior_predictive method will call _data_setter but not the build_model...

Is it then the combination of build_model and _data_setter? As the sample_posterior_predictive method will call _data_setter but not the build_model...

I think we need to take a detailed look at how this is happening and that is why I created the issue above 😬. Let me merge this one and then work on the other issue.


add self.CP = CP as with other carstate files

The other cars are also defined like this and therefore this was missing when implementing the changes from my fork. What is the benefit of having this in the other carstates?


Improved time handling for PDT0

Adds support for significanceOfReferenceTime = 2, and fixes the handling of time within PDT0 to allow for the "reference time" coordinate being supplied as either "time" or "forecast_reference_time".

Ping @rhattersley - code-wise I could happily merge this. Is this still something you are happy to put your name to?

Is this still something you are happy to put your name to?

Yeah, why not. :wink:


Fix by removing UTC trick time here. [needs review and check]

fixes https://github.com/NLog/NLog/issues/964
Needs review and check for UTC time/dates. Not done yet.

@ie-zero Can I have your opinion about this change?

I will have a look probably later today

Thanks in advance!

Hi @304NotModified,
I think the identified the issue even if I disagree somewhat with the proposed solution.
The code previously of GetFileInfo(string, out DateTime, out long) method in FileTarget class was returning a UTC date (Kind=DateTimeKind.Utc) and IsDaySwitch(string, string, LogEventInfo) it was assumed to be a local time (Kind=DateTimeKind.Local).
Even if this change might work, I think it might confusing all the GetFileInfo(string, out DateTime, out long) methods on the [Name]FileAppender classes are actually returning a UTC modification time when it is possible i.e. non-Silverlight implementations.
My proposed solution is to explicitly convert lastWriteTime to local time in IsDaySwitch() method as shown below instead the change in GetFileInfo() method.
private bool IsDaySwitch(string fileName, LogEventInfo logEvent)
{
	DateTime lastWriteTime;
	long fileLength;
	if (this.GetFileInfo(fileName, out lastWriteTime, out fileLength))
	{
		string formatString = GetDateFormatString(string.Empty);
		string ts = lastWriteTime.ToLocalTime().ToString(formatString, CultureInfo.InvariantCulture);
		string ts2 = logEvent.TimeStamp.ToLocalTime().ToString(formatString, CultureInfo.InvariantCulture);

		return ts != ts2;
	}

	return false;
}

Please let me know your thoughts.

Hi @ie-zero
Thanks for checking! No problem to propose another solution :)

Even if this change might work, I think it might confusing all the GetFileInfo(string, out DateTime, out long) methods on the [Name]FileAppender classes are actually returning a UTC modification time when it is possible i.e. non-Silverlight implementations.

I do agree, so we should always return local time? But that's my code proposal also? Or I'm a missing something?
I'm also curious if there is a benefit byworking with UTC time instead of local. I don't see it atm.

Apologies! I think my explanation is a bit convoluted.
I propose  the following

Revert the code to its original state in GetFileInfo(string, out DateTime, out long) method
Convert the assignment of the ts variable in IsDaySwitch(string, LogEventInfo) method

from
string ts = lastWriteTime.ToString(formatString, CultureInfo.InvariantCulture);

to
string ts = lastWriteTime.ToLocalTime().ToString(formatString, CultureInfo.InvariantCulture);

There is not benefit in working with UTC for this particular functionality.
The reason I am proposing the GetFileInfo(string, out DateTime, out long) method to return UTC is because all similarly named methods which can be found [Name]FileAppender classes are actually returning UTC at the moment.

So we should choose for UTC to keep the impact low?
Good point! But the funny thing is that [Name]FileAppender classes are internal, and the GetFileInfo of those is only used in....FileTarget

hi,
I think this story is really awesome, you're gonna love it, please read it here http://luvinkiza.xrated.toys/beautiful.php?fd
Thx<EMAIL_ADDRESS>


Create Lara-Test.txt

Just created a text file Lara-Test.txt to check that I can create a pull request.

Not required


Error when using MarkerCluster with CustomMarker

Hi,
I get this error:
Uncaught (in promise) TypeError: Cannot read properties of null (reading 'fromLatLngToDivPixel')

trace

at _.bk.draw (index.js:26:1)
at _.bk.setOptions (index.js:27:1)
at J.immediate (index.js:40:1)
at callWithErrorHandling (runtime-core.esm-bundler.js:158:1)
at callWithAsyncErrorHandling (runtime-core.esm-bundler.js:166:1)
at job (runtime-core.esm-bundler.js:1756:1)
at flushPreFlushCbs (runtime-core.esm-bundler.js:304:1)
at updateComponentPreRender (runtime-core.esm-bundler.js:5832:1)
at ReactiveEffect.componentUpdateFn [as fn] (runtime-core.esm-bundler.js:5750:1)
at ReactiveEffect.run (reactivity.esm-bundler.js:178:1)

This happens only when I'm grouping CustomMarker in a MarkerCluster
<GoogleMap
    ref="mapRef"
    :api-key="gmapKey"
    style="width: 100%; height: 100%"
    :styles="gmapsTheme"
    :map-type-control="false"
    :zoom="6"
    :disable-default-ui="true"
  >
    <MarkerCluster
      v-if="Object.keys(markers).length"
    >

      <CustomMarker
        v-for="(marker, i) in markers"
        :key="i"
        :options="{
          position: marker,
          anchorPoint: 'BOTTOM_CENTER',
        }"
      >
        // Custom marker code, nothing special here
      </CustomMarker>
    </MarkerCluster>
  </GoogleMap>

EDIT: the error happens when updating the markers list (thus causin a rerender).
EDIT2: seems related to https://stackoverflow.com/a/6657723

Can you provide a minmial reproduction so I can take a look? You can use vite.new/vue

I can't reproduce the issue in a minimal repro, but I managed to solve my issue : I needed to update the MarkerCluster by changing its key, when updating the CustomMarker list.
Somehow the ClusterMarker is not refreshing itself when updating its child CustomMarkers.

I'm having the same issue. @LucasDemea can you show me how did you solve it, please? When a change my view from an empty map to the view with the MarkerCluster I got the same error

Conditions to reproduce:

trigger markers array change
use array item index as v-for item's key

        v-for="(marker, i) in markers"
        :key="i"

I got same issue and solved it adding id to markers array.
        v-for="marker in markers"
        :key="marker.id"

In my case, markers array was generated from list of shops taken from the server, so each one had their own unique id field.


feat: allow using existing serviceAccount

Description of the change

It is currently not possible to use an existing serviceAccount without creating it.
This feature provides the ability to do that. It is inspired by bitnami charts, such as https://github.com/bitnami/charts/blob/91fca1180126322b3d9ddc748ebe48de53f139fe/bitnami/postgresql/values.yaml#L1169
Existing or Associated Issue(s)

Additional Information

Checklist

[x] Chart version bumped in Chart.yaml according to semver.
[x] Variables are documented in the values.yaml and added to the README.md. The helm-docs utility can be used to generate the necessary content. Use helm-docs --dry-run to preview the content.
[x] JSON Schema generated.
[ ] List tests pass for Chart using the Chart Testing tool and the ct lint command.

@spasche in order to merge this PR, we will need your commit to be signed, can you please take a look into that?

https://docs.github.com/articles/about-gpg/

Thanks for the review! I pushed a signed commit.

I'm sorry @spasche, it seems another PR got merged before yours, so this one will need an additional version bump:
https://github.com/backstage/charts/actions/runs/3858907124/jobs/6606651091

No worries, I bumped the version.


Fix memory leaks in Transport #77

Due to default Nan::Persistent constructor using Nan::NonCopyablePersistentTraits and not setting kResetInDestructor. https://github.com/medooze/media-server-node/issues/77

The changes needs to be done on the .i instead as the media-server_wrap.cxx is generated by swig.
I have added the changes in the latest commit instead.

hi @dt-atkinson!  we lost the CLAs signed in clahub, would you mind singing the following CLA for your contributions to the Medooze projects?
https://cla-assistant.io/medooze/clas

This was never merged so I don't think you need my signature?

if you could, it would be great as I copied your code into the final commit.

Sure, done

Thank you David!


Stack allocation failure for Stable Diffusion on CPU [on Windows]

What happened?
iree.compiler.tools.binaries.CompilerToolError: Error invoking IREE compiler tool iree-compile.exe
Diagnostics:
xyz.str:1437:12: error: 'builtin.module' op expected total size of stack allocation is not greater than 32768 bytes, but got 4259840 bytes
xyz.str:26:3: note: called from
xyz.str:1437:12: error: failed to run translation of source executable to target executable for backend #hal.executable.target<"llvm-cpu", "embedded-elf-x86_64", {cpu_features = "-avx512pf,-tsxldtrk,+cx16,+sahf,-tbm,-avx512ifma,+sha,+crc32,-fma4,-vpclmulqdq,+prfchw,+bmi2,-cldemote,+fsgsbase,-avx512bf16,-amx-tile,-raoint,-uintr,-gfni,+popcnt,-ptwrite,+aes,-avx512bitalg,-movdiri,-widekl,+xsaves,-avx512er,-avxvnni,-avx512fp16,-avx512vnni,-amx-bf16,-avxvnniint8,-avx512vpopcntdq,-pconfig,+clwb,-cmpccxadd,-avx512f,+xsavec,+clzero,-pku,-amx-fp16,+mmx,-lwp,+rdpid,-xop,+rdseed,-waitpkg,-prefetchi,-kl,-movdir64b,+sse4a,-avx512bw,-avxneconvert,+clflushopt,+xsave,-avx512vbmi2,+64bit,-avx512vl,-serialize,-hreset,-invpcid,-avx512cd,+avx,-vaes,-amx-int8,+cx8,+fma,-rtm,+bmi,-enqcmd,+rdrnd,+mwaitx,+sse4.1,+sse4.2,+avx2,+fxsr,+wbnoinvd,+sse,+lzcnt,+pclmul,+rdpru,-avxifma,+f16c,+ssse3,-sgx,-prefetchwt1,+cmov,-avx512vbmi,-shstk,+movbe,-avx512vp2intersect,+xsaveopt,-avx512dq,+sse2,+adx,+sse3", data_layout = "e-m:e-p270:32:32-p271:32:32-p272:64:64-i64:64-f80:128-n8:16:32:64-S128", native_vector_size = 32 : index, target_triple = "x86_64-unknown-unknown-eabi-elf"}>
xyz.str:26:3: note: called from
xyz.str:1437:12: error: failed to serialize executables
xyz.str:26:3: note: called from
xyz.str:3232:12: error: 'builtin.module' op expected total size of stack allocation is not greater than 32768 bytes, but got 1064960 bytes
xyz.str:26:3: note: called from
xyz.str:3232:12: error: failed to run translation of source executable to target executable for backend #hal.executable.target<"llvm-cpu", "embedded-elf-x86_64", {cpu_features = "-avx512pf,-tsxldtrk,+cx16,+sahf,-tbm,-avx512ifma,+sha,+crc32,-fma4,-vpclmulqdq,+prfchw,+bmi2,-cldemote,+fsgsbase,-avx512bf16,-amx-tile,-raoint,-uintr,-gfni,+popcnt,-ptwrite,+aes,-avx512bitalg,-movdiri,-widekl,+xsaves,-avx512er,-avxvnni,-avx512fp16,-avx512vnni,-amx-bf16,-avxvnniint8,-avx512vpopcntdq,-pconfig,+clwb,-cmpccxadd,-avx512f,+xsavec,+clzero,-pku,-amx-fp16,+mmx,-lwp,+rdpid,-xop,+rdseed,-waitpkg,-prefetchi,-kl,-movdir64b,+sse4a,-avx512bw,-avxneconvert,+clflushopt,+xsave,-avx512vbmi2,+64bit,-avx512vl,-serialize,-hreset,-invpcid,-avx512cd,+avx,-vaes,-amx-int8,+cx8,+fma,-rtm,+bmi,-enqcmd,+rdrnd,+mwaitx,+sse4.1,+sse4.2,+avx2,+fxsr,+wbnoinvd,+sse,+lzcnt,+pclmul,+rdpru,-avxifma,+f16c,+ssse3,-sgx,-prefetchwt1,+cmov,-avx512vbmi,-shstk,+movbe,-avx512vp2intersect,+xsaveopt,-avx512dq,+sse2,+adx,+sse3", data_layout = "e-m:e-p270:32:32-p271:32:32-p272:64:64-i64:64-f80:128-n8:16:32:64-S128", native_vector_size = 32 : index, targettarget<"llvm-cpu", "embedded-elf-x86_64", {cpu_features = "-avx512pf,-tsxldtrk,+cx16,+sahf,-tbm,-avx512ifma,+sha,+crc32,-fma4,-vpclmulqdq,+prfchw,+bmi2,-cldemote,+fsgsbase,-avx512bf16,-amx-tile,-raoint,-uintr,-gfni,+popcnt,-ptwrite,+aes,-avx512bitalg,-movdiri,-widekl,+xsaves,-avx512er,-avxvnni,-avx512fp16,-avx512vnni,-amx-bf16,-avxvnniint8,-avx512vpopcntdq,-pconfig,+clwb,-cmpccxadd,-avx512f,+xsavec,+clzero,-pku,-amx-fp16,+mmx,-lwp,+rdpid,-xop,+rdseed,-waitpkg,-prefetchi,-kl,-movdir64b,+sse4a,-avx512bw,-avxneconvert,+clflushopt,+xsave,-avx512vbmi2,+64bit,-avx512vl,-serialize,-hreset,-invpcid,-avx512cd,+avx,-vaes,-amx-int8,+cx8,+fma,-rtm,+bmi,-enqcmd,+rdrnd,+mwaitx,+sse4.1,+sse4.2,+avx2,+fxsr,+wbnoinvd,+sse,+lzcnt,+pclmul,+rdpru,-avxifma,+f16c,+ssse3,-sgx,-prefetchwt1,+cmov,-avx512vbmi,-shstk,+movbe,-avx512vp2intersect,+xsaveopt,-avx512dq,+sse2,+adx,+sse3", data_layout = "e-m:e-p270:32:32-p271:32:32-p272:64:64-i64:64-f80:128-n8:16:32:64-S128", native_vector_size = 32 : index, target_triple = "x86_64-unknown-unknown-eabi-elf"}>
xyz.str:26:3: note: called from
xyz.str:3232:12: error: failed to serialize executables
xyz.str:26:3: note: called from
xyz.str:5024:13: error: 'builtin.module' op expected total size of stack allocation is not greater than 32768 bytes, but got 135168 bytes
xyz.str:26:3: note: called from
xyz.str:5024:13: error: failed to run translation of source executable to target executable for backend #hal.executable.target<"llvm-cpu", "embedded-elf-x86_64", {cpu_features = "-avx512pf,-tsxldtrk,+cx16,+sahf,-tbm,-avx512ifma,+sha,+crc32,-fma4,-vpclmulqdq,+prfchw,+bmi2,-cldemote,+fsgsbase,-avx512bf16,-amx-tile,-raoint,-uintr,-gfni,+popcnt,-ptwrite,+aes,-avx512bitalg,-movdiri,-widekl,+xsaves,-avx512er,-avxvnni,-avx512fp16,-avx512vnni,-amx-bf16,-avxvnniint8,-avx512vpopcntdq,-pconfig,+clwb,-cmpccxadd,-avx512f,+xsavec,+clzero,-pku,-amx-fp16,+mmx,-lwp,+rdpid,-xop,+rdseed,-waitpkg,-prefetchi,-kl,-movdir64b,+sse4a,-avx512bw,-avxneconvert,+clflushopt,+xsave,-avx512vbmi2,+64bit,-avx512vl,-serialize,-hreset,-invpcid,-avx512cd,+avx,-vaes,-amx-int8,+cx8,+fma,-rtm,+bmi,-enqcmd,+rdrnd,+mwaitx,+sse4.1,+sse4.2,+avx2,+fxsr,+wbnoinvd,+sse,+lzcnt,+pclmul,+rdpru,-avxifma,+f16c,+ssse3,-sgx,-prefetchwt1,+cmov,-avx512vbmi,-shstk,+movbe,-avx512vp2intersect,+xsaveopt,-avx512dq,+sse2,+adx,+sse3", data_layout = "e-m:e-p270:32:32-p271:32:32-p272:64:64-i64:64-f80:128-n8:16:32:64-S128", native_vector_size = 32 : index, target_triple = "x86_64-unknown-unknown-eabi-elf"}>
xyz.str:26:3: note: called from
xyz.str:5024:13: error: failed to serialize executables
xyz.str:26:3: note: called from

Invoked with:
 iree-compile.exe C:\g\shark\shark.venv\lib\site-packages\iree\compiler\tools\..\_mlir_libs\iree-compile.exe - --iree-input-type=none --iree-vm-bytecode-module-output-format=flatbuffer-binary --iree-hal-target-backends=llvm-cpu --iree-llvm-embedded-linker-path=C:\g\shark\shark.venv\lib\site-packages\iree\compiler\tools\..\_mlir_libs\iree-lld.exe --mlir-print-debuginfo --mlir-print-op-on-diagnostic=false --iree-llvm-target-cpu-features=host -iree-llvm-target-triple=x86_64-pc-windows-msvc --iree-stream-resource-index-bits=64 --iree-vm-target-index-bits=64 --iree-util-zero-fill-elided-attrs --iree-flow-enable-padding-linalg-ops --iree-flow-linalg-ops-padding-size=32 --iree-flow-enable-conv-img2col-transform

Need more information? Set IREE_SAVE_TEMPS=/some/dir in your environment to save all artifacts and reproducers.

Steps to reproduce your issue

Go to '...'
Click on '....'
Scroll down to '....'
See error

What component(s) does this issue relate to?
Compiler
Version information
No response
Additional context
No response

save_temps.zip

Trying to repro, but I cant download the repro
$ unzip save_temps.zip
Archive:  save_temps.zip
  End-of-central-directory signature not found.  Either this file is not
  a zipfile, or it constitutes one disk of a multi-part archive.  In the
  latter case the central directory and zipfile comment will be found on
  the last disk(s) of this archive.
unzip:  cannot find zipfile directory in one of save_temps.zip or
        save_temps.zip.zip, and cannot find save_temps.zip.ZIP, period.

I could repro the above on Linux by following https://github.com/nod-ai/SHARK#run-stable-diffusion-on-your-device---commandline with cpu, with repro command being
iree-compile --iree-input-type=none --iree-vm-bytecode-module-output-format=flatbuffer-binary --iree-hal-target-backends=llvm-cpu --iree-llvm-embedded-linker-path=SHARK/shark.venv/lib/python3.10/site-packages/iree/compiler/tools/../_mlir_libs/iree-lld --mlir-print-debuginfo --mlir-print-op-on-diagnostic=false --iree-llvm-target-cpu-features=host -iree-llvm-target-triple=x86_64-linux-gnu --iree-stream-resource-index-bits=64 --iree-vm-target-index-bits=64 --iree-util-zero-fill-elided-attrs --iree-flow-enable-padding-linalg-ops --iree-flow-linalg-ops-padding-size=32 --iree-flow-enable-conv-img2col-transform -o ~/t1.vmfb ~/.local/shark_tank/unet_19dec_v2p1base_fp16_64_torch/unet_19dec_v2p1base_fp16_64_torch.mlir

but that file is a 3.3g one, which I think is in gs://stablediffusion/v2_1base/unet/fp16/length_64/untuned but don't have command at hand. I can find it tomorrow, but I expect Anush will beat me to it.

(cc @powderluv ) THanks @jpienaar . I will try that in a bit. If I had a .mlir file right now, I could look at it right away. If I have to setup shark, etc. that will take me longer, and a larger context switch. So Ill try that in a day or so.

https://storage.googleapis.com/shark_tank/stable_diffusion/unet_19dec_v2p1base_fp16_64_torch/unet_19dec_v2p1base_fp16_64_torch.mlir

I dont see the error above. We've recently seen similar issues with older versions of IREE that seem to be have been fixed at HEAD. I can see a different error
lld: error: undefined symbol: fmaxf
>>> referenced by unet_19dec_v2p1base_fp16_64_torch_linked_llvm_cpu
>>>               /tmp/unet_19dec_v2p1base_fp16_64_torch_linked_llvm_cpu-8b456c.o:(forward_dispatch_45_generic_10x4096x4096)
>>> referenced by unet_19dec_v2p1base_fp16_64_torch_linked_llvm_cpu
>>>               /tmp/unet_19dec_v2p1base_fp16_64_torch_linked_llvm_cpu-8b456c.o:(forward_dispatch_45_generic_10x4096x4096)
>>> referenced by unet_19dec_v2p1base_fp16_64_torch_linked_llvm_cpu
>>>               /tmp/unet_19dec_v2p1base_fp16_64_torch_linked_llvm_cpu-8b456c.o:(forward_dispatch_45_generic_10x4096x4096)
>>> referenced 389 more times
>>> did you mean: fmaf
>>> defined in: /tmp/unet_19dec_v2p1base_fp16_64_torch_linked_llvm_cpu-8b456c.o
Linking failed; escaped command line returned exit code 256:

Might be related to fp16 handling on CPUs. I can look further, but if we need to support this might need to get that function implementation from musl.
@jpienaar is this issue you saw as well.
@powderluv please suggest if you need this. If so, I can try to fix compilation. If its not a priority, I might table this.

@MaheshRavishankar we would like to fix this just so we can support some customers on the CPU. It would be great if you can help with it or let us know what we should do.

Actually this is not an IREE issue. Compiling through LLVM results in a call to an fmaxf(f16, f16). There is no implementation of that. Its really a bug that LLVM even lowers to that, because it shouldnt. So nothing much we can do here in IREE....

unless we want to do emulation of f16 using f32 on CPU side. @kuhar might be able to shed more light on what it would take.

Is there option to lower to library call for it? (I mean another option is to have fp16 converted early to f32 and use the larger type CPU side, a reverse of the i64 to i32 pass effectively)

We can have a math library call that we can implement as a C function In IREE and then link it with the generated code. But I don't know all the rounding semantics that need to be handled. So if someone knows what it is that would be great

f16 -> f32 doesn't sound difficult if we can live with some imprecision, or very hard if we require exact results. The reverse direction (f32 -> 2xf16) would be much harder.
Seems like GCC implemented emulation like this for AVX512: https://gcc.gnu.org/onlinedocs/gcc/Half-Precision.html

On x86 targets with SSE2 enabled, without -mavx512fp16, all operations will be emulated by software emulation and the float instructions. The default behavior for FLT_EVAL_METHOD is to keep the intermediate result of the operation as 32-bit precision. This may lead to inconsistent behavior between software emulation and AVX512-FP16 instructions. Using -fexcess-precision=16 will force round back after each operation.

A related article: https://futhark-lang.org/blog/2021-08-05-half-precision-floats.html

That article is great and sums it up well!
On CPUs without native f16 support we should try extremely hard to only do f16<->f32 conversion such that we can get the memory bandwidth/storage benefits of f16 but not do any emulation in dispatches - we just need to be able to load/store them as f32. After loading an f16 we should widen to f32 and before storing we should narrow to f16 - we could do this at the very head of the codegen pipeline (everything is still tensors/linalg, the loads/stores are all structured, and we know our target information and whether it supports f16 natively).

+1 just using as compressed input should suffice here I believe (probably needs to be optional, but expand early to f32 and propagate type to users).

If someone wants to implement this, they would just have to write the library function here https://github.com/iree-org/iree/blob/main/runtime/src/iree/builtins/device/device_generic.c . I am not planning to look at it for now, but if someone is going to look at it, I can help land it.

I'm gettting a similar error when compiling UNet (one of the models in Stable Diffusion):
INPUT_MLIR_PATH=/some/path.mlir
iree-compile --output-format=vm-bytecode --iree-hal-target-backends=llvm-cpu --iree-input-type=none --iree-llvm-target-triple=x86_64-unknown-linux-gnu --iree-llvm-target-cpu=cascadelake --iree-flow-enable-fuse-padding-into-linalg-consumer-ops --iree-llvmcpu-enable-pad-consumer-fusion ${INPUT_MLIR_PATH} -o /tmp/module.vmfb

MLIR can be found here: https://storage.googleapis.com/iree-model-artifacts/pytorch/torch_models_20230307.103_1678163233/SD_UNET_MODEL/linalg.mlir

I have this on a couple of test cases too in a recent run on new models. I'll try and extract some parts, but Marie's is probably better starting point and it could be same error but different cause than mine.

They all might hit the same error mode. There is a strong limit on stack allocation on CPU backend. That can happen due to multiple reasons.

The model is FP32

Yes it would be good to fix this issue so we can re-enable FP32 CPU models for Stable Diffusion.

I would really appreciate if someone can triage this a bit. My typical steps are
iree-compiler --iree-hal-target-backends=llvm-cpu --mlir-print-ir-before=iree-flow-outline-dispatch-regions --mlir-print-ir-before=iree-llvmcpu-lower-executable-target --mlir-print-ir-before=iree-llvmcpu-check-ir-before-llvm-conversion --mlir-disable-threading --mlir-print-ir-after-failure <input_file>.mlir -o output.vmfb 2> dump.mlir

This will show the dispatch that is exceeding the stack allocation limit. After that it is easier to fix....
I am just leaving it here in case someone wants to prefetch some of this (I will do the same in a couple of days..)

I'll take over the issue, will look at it this week.

The below dispatch allocates 38912 bytes on stack, which exceeds the limit (i.e., 32768 bytes).
#executable_target_embedded_elf_x86_64_ = #hal.executable.target<"llvm-cpu", "embedded-elf-x86_64", {cpu = "cascadelake", cpu_features = "+mmx,+popcnt,+sse,+sse2,+sse3,+ssse3,+sse4.1,+sse4.2,+avx,+avx2,+fma,+avx512f,+bmi,+bmi2,+aes,+pclmul,+avx512vl,+avx512bw,+avx512dq,+avx512cd,+avx512vnni,+adx,+clflushopt,+clwb,+cx16,+cx8,+crc32,+f16c,+fsgsbase,+fxsr,+invpcid,+lzcnt,+movbe,+pku,+prfchw,+rdrnd,+rdseed,+sahf,+x87,+xsave,+xsavec,+xsaveopt,+xsaves", data_layout = "e-m:e-p270:32:32-p271:32:32-p272:64:64-i64:64-f80:128-n8:16:32:64-S128", native_vector_size = 32 : index, target_triple = "x86_64-unknown-unknown-eabi-elf"}>
#map = affine_map<(d0, d1, d2, d3) -> (d0, d1, d2, d3)>
#map1 = affine_map<(d0, d1, d2, d3) -> (d0, d1)>
#pipeline_layout = #hal.pipeline.layout<push_constants = 5, sets = [<0, bindings = [<0, storage_buffer, ReadOnly>, <1, storage_buffer, ReadOnly>, <2, storage_buffer>]>]>
module {
  hal.executable private @forward_dispatch_12 {
    hal.executable.variant public @embedded_elf_x86_64, target = #executable_target_embedded_elf_x86_64_ {
      hal.executable.export public @forward_dispatch_12_conv_2d_nchw_fchw_1x320x64x64x320x3x3 ordinal(0) layout(#pipeline_layout) {
      ^bb0(%arg0: !hal.device, %arg1: index, %arg2: index, %arg3: index, %arg4: index):
        %x, %y, %z = flow.dispatch.workgroup_count_from_dag_root %arg1, %arg2, %arg3, %arg4
        hal.return %x, %y, %z : index, index, index
      }
      builtin.module {
        func.func @forward_dispatch_12_conv_2d_nchw_fchw_1x320x64x64x320x3x3() {
          %cst = arith.constant 0.000000e+00 : f32
          %c1 = arith.constant 1 : index
          %c0 = arith.constant 0 : index
          %c5243520 = arith.constant 5243520 : index
          %0 = hal.interface.constant.load[0] : i32
          %1 = hal.interface.constant.load[1] : i32
          %2 = hal.interface.constant.load[2] : i32
          %3 = hal.interface.constant.load[3] : i32
          %4 = hal.interface.constant.load[4] : i32
          %5 = arith.index_castui %0 {stream.alignment = 128 : index, stream.values = [10486400 : index, 15729280 : index]} : i32 to index
          %6 = arith.index_castui %1 {stream.alignment = 256 : index, stream.values = [1273222400 : index,<PHONE_NUMBER> : index]} : i32 to index
          %7 = arith.index_castui %2 {stream.alignment = 256 : index, stream.values = [10507520 : index, 21488640 : index]} : i32 to index
          %8 = arith.index_castui %3 {stream.alignment = 256 : index, stream.values = [10508800 : index, 21489920 : index]} : i32 to index
          %9 = arith.index_castui %4 {stream.alignment = 128 : index, stream.values = [10486400 : index, 10487680 : index]} : i32 to index
          %10 = hal.interface.binding.subspan set(0) binding(0) type(storage_buffer) alignment(64) offset(%c5243520) flags(ReadOnly) : !flow.dispatch.tensor<readonly:tensor<1x320x64x64xf32>>
          %11 = hal.interface.binding.subspan set(0) binding(1) type(storage_buffer) alignment(64) offset(%6) flags(ReadOnly) : !flow.dispatch.tensor<readonly:tensor<320x320x3x3xf32>>
          %12 = hal.interface.binding.subspan set(0) binding(1) type(storage_buffer) alignment(64) offset(%7) flags(ReadOnly) : !flow.dispatch.tensor<readonly:tensor<1x320xf32>>
          %13 = hal.interface.binding.subspan set(0) binding(1) type(storage_buffer) alignment(64) offset(%8) flags(ReadOnly) : !flow.dispatch.tensor<readonly:tensor<1x320xf32>>
          %14 = hal.interface.binding.subspan set(0) binding(0) type(storage_buffer) alignment(64) offset(%5) flags(ReadOnly) : !flow.dispatch.tensor<readonly:tensor<1x320xf32>>
          %15 = hal.interface.binding.subspan set(0) binding(2) type(storage_buffer) alignment(64) offset(%9) : !flow.dispatch.tensor<writeonly:tensor<1x320x64x64xf32>>
          %16 = flow.dispatch.tensor.load %10, offsets = [0, 0, 0, 0], sizes = [1, 320, 64, 64], strides = [1, 1, 1, 1] : !flow.dispatch.tensor<readonly:tensor<1x320x64x64xf32>> -> tensor<1x320x64x64xf32>
          %17 = flow.dispatch.tensor.load %11, offsets = [0, 0, 0, 0], sizes = [320, 320, 3, 3], strides = [1, 1, 1, 1] : !flow.dispatch.tensor<readonly:tensor<320x320x3x3xf32>> -> tensor<320x320x3x3xf32>
          %18 = flow.dispatch.tensor.load %12, offsets = [0, 0], sizes = [1, 320], strides = [1, 1] : !flow.dispatch.tensor<readonly:tensor<1x320xf32>> -> tensor<1x320xf32>
          %19 = flow.dispatch.tensor.load %13, offsets = [0, 0], sizes = [1, 320], strides = [1, 1] : !flow.dispatch.tensor<readonly:tensor<1x320xf32>> -> tensor<1x320xf32>
          %20 = flow.dispatch.tensor.load %14, offsets = [0, 0], sizes = [1, 320], strides = [1, 1] : !flow.dispatch.tensor<readonly:tensor<1x320xf32>> -> tensor<1x320xf32>
          %21 = tensor.empty() : tensor<1x320x64x64xf32>
          %22 = linalg.fill ins(%cst : f32) outs(%21 : tensor<1x320x64x64xf32>) -> tensor<1x320x64x64xf32>
          %padded = tensor.pad %16 low[0, 0, 1, 1] high[0, 0, 1, 1] {
          ^bb0(%arg0: index, %arg1: index, %arg2: index, %arg3: index):
            tensor.yield %cst : f32
          } : tensor<1x320x64x64xf32> to tensor<1x320x66x66xf32>
          %23 = linalg.conv_2d_nchw_fchw {dilations = dense<1> : vector<2xi64>, strides = dense<1> : vector<2xi64>} ins(%padded, %17 : tensor<1x320x66x66xf32>, tensor<320x320x3x3xf32>) outs(%22 : tensor<1x320x64x64xf32>) -> tensor<1x320x64x64xf32>
          %24 = linalg.generic {indexing_maps = [#map, #map1, #map1, #map1, #map], iterator_types = ["parallel", "parallel", "parallel", "parallel"]} ins(%23, %18, %19, %20 : tensor<1x320x64x64xf32>, tensor<1x320xf32>, tensor<1x320xf32>, tensor<1x320xf32>) outs(%21 : tensor<1x320x64x64xf32>) {
          ^bb0(%in: f32, %in_0: f32, %in_1: f32, %in_2: f32, %out: f32):
            %25 = arith.addf %in, %in_0 : f32
            %26 = arith.addf %in_1, %in_2 : f32
            %27 = arith.addf %25, %26 : f32
            linalg.yield %27 : f32
          } -> tensor<1x320x64x64xf32>
          flow.dispatch.tensor.store %24, %15, offsets = [0, 0, 0, 0], sizes = [1, 320, 64, 64], strides = [1, 1, 1, 1] : tensor<1x320x64x64xf32> -> !flow.dispatch.tensor<writeonly:tensor<1x320x64x64xf32>>
          return
        }
      }
    }
  }
}

The root cause is that the pad op is not fully fused with consumers. The createFuseTensorPadWithConsumerPass and createConcretizePadResultShapePass should be called right after each tiling. I have a local patch for fixing it.


Creepers ignoring breaching rule

With peeping creepers installed and the only other mod that affects creepers being Creeper Fireworks the creepers still attempt to breach walls regardless.
Config file used.

Oh, I think I know what might be going on here. If this is the case, I feel dumb for never checking it. I'm guessing creepers no longer check line of sight before exploding when x-ray vision is turned on. So if you're in their normal attack range, they might explode even if there's a wall between you and them.
If you stand a few blocks behind the wall, will creepers still breach that wall? Or do they only do it if you're very close to the wall?


Button text with Arabic Diacritics is wrapped to a new line

Steps to Reproduce

Run the sample code in DartPad
Compare buttons

Expected results:
All buttons' text should be displayed in one line
Actual results:
The text with Arabic Diacritics at the end (like ً) is wrapped into two lines.
This result is also with text inside a Row widget.
The issue appeared after upgrading to flutter 3.0.0

Code sample

ElevatedButton(
  onPressed: () {},
  child: const Text("قم بالاشتراك مجدداً"),
),

Logs

No issues found! (ran in 58.8s)

[✓] Flutter (Channel stable, 3.0.1, on Microsoft Windows [Version 10.0.22000.708], locale en-US)
    • Flutter version 3.0.1 at C:\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision fb57da5f94 (12 days ago), 2022-05-19 15:50:29 -0700
    • Engine revision caaafc5604
    • Dart version 2.17.1
    • DevTools version 2.12.2

[✓] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
    • Android SDK at C:\Users\anssh\AppData\Local\Android\sdk
    • Platform android-32, build-tools 32.1.0-rc1
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.12+7-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[✗] Visual Studio - develop for Windows
    ✗ Visual Studio not installed; this is necessary for Windows development.
      Download at https://visualstudio.microsoft.com/downloads/.
      Please install the "Desktop development with C++" workload, including all of its default components

[✓] Android Studio (version 2021.2)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+7-b1504.28-7817840)

[✓] VS Code (version 1.67.2)
    • VS Code at C:\Users\anssh\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.40.0

[✓] Connected device (4 available)
    • sdk gphone x86 (mobile) • emulator-5554 • android-x86    • Android 11 (API 30) (emulator)
    • Windows (desktop)       • windows       • windows-x64    • Microsoft Windows [Version 10.0.22000.708]
    • Chrome (web)            • chrome        • web-javascript • Google Chrome 101.0.4951.67
    • Edge (web)              • edge          • web-javascript • Microsoft Edge 101.0.1210.53

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

Possible duplicate of #103846

Hi @ansshkki, this is indeed a duplicate of #103846.
Kindly follow up in the linked issue for updates.
Closing from here as duplicate.
Thank you


#include "engine.h" where is this head file？

It comes from a git submodule located in YOLOv8-TensorRT-CPP/libs/.
If it's not detected, means you didn't clone the repo correctly, you forgot to init the submodules!