Question - QA on Sum of Leaf Gains

This is more of a doubt on my side. The constructor is currently building score points based on self.xgb_scorecard["XAddEvidence"] by default. This is defined on self.extract_leaf_weights from the gain of a leaf. The gain of a leaf is defined here with a formula that as I understand, it approximates a Newton-Raphson update.
My question is which guarantees do we have that the sum of the leaf gains plus the base score actually reproduces the XGBoost predictions (in the logit scale)? Maybe we could test this by comparing the gain against the result of using predict with output_margin=True for a tree and passing the value of the desired leaf? (e.g. LTV < 50% in a tree with that decision split).
Note: I have updated the issue as there was some misunderstanding on my side. I aim to provide some code to check my question above.

Long overdue, but here is a MRE that indeed shows that both approach yield the same result (with XAddEvidence having an extra digit of precision, interestingly). Closing the issue.
import xgboost as xgb
from xbooster.constructor import XGBScorecardConstructor

from sklearn.datasets import load_breast_cancer
import pandas as pd
import numpy as np

data = load_breast_cancer()
X, y = data.data, data.target

# Build and fit classifier with only two trees
model = xgb.XGBClassifier(n_estimators=10, eval_metric='logloss')
model.fit(X, y)

# Retrieve base score using constructor class
scorecard_constructor = XGBScorecardConstructor(
    model, X, y
)
xgb_scorecard = scorecard_constructor.construct_scorecard()

# Check first leaf, save XAddEvidence for later comparison
print("First leaf")
print(xgb_scorecard.iloc[0])
xbooster_logit_contrib = xgb_scorecard.iloc[0]["XAddEvidence"]

# Compare against logit contribution manually calculated
# Extract first tree
tree = [t for t in scorecard_constructor.model.get_booster()][0]

# Prepare artificial data that meets first leaf condition (manually)
X_val = pd.DataFrame(np.zeros((1, X.shape[1]), np.longdouble))
X_val.iloc[0, 10] = 0.63
DM_X_val = xgb.DMatrix(X_val, base_margin=np.zeros(1,))

# Obtain logit contribution from this tree
print("logit contribution from first tree")
print(tree.predict(DM_X_val, output_margin=True)[0])
logit_contrib = tree.predict(DM_X_val, output_margin=True)[0]

if np.isclose(logit_contrib, xbooster_logit_contrib):
    print("Both approaches lead to close results")
else:
    print("Results differ")


Proposed issue labeling to support project management tools

Goals
Keep issue management simple and lightweight, while enabling project management tools.
Dask is used by other companies (the contributors among others) that want to contribute to Dask but need a greater level of coordination across internal and dask projects.
With GitHub Projects (or other tools) milestones, progress, and priority are expressed and tracked, but to do this effectively issues need to have a meaningful set of labels applied to them to express type, size, and component. In GitHub projects, it displays the issue title, tasks, "#xxx opened by user in repo", and the labels.
Labels
Each issue should have a type, size, and component whenever appropriate.
Type label
By default, an issue with no type is an enhancement. For simplicity issues that are dup/WillNotFix/invalid should get a comment and be closed, not kept open.

(default no-label type means enhancement)
bug
question
good first issue
community

Size label
A rough estimation of how long someone familiar with the system would take to complete the issue. This helps in planning and enables new contributors to find smaller tasks.

size minutes
size hours
size days
size weeks

"size weeks" is for community issues discussing roadmap/architecture planning, which are broken into smaller issues for implementation. No single merge should ever be that vast.
Component label
Each project has a minimal but complete set of component labels. The list should be short but correspond to areas of expertise.
Common Components

core (core to functionality of the repo)
documentation
cicd
... + project components. e.g. networking, scheduling, arrays, cicd, ...

Flow
A component and size label means the issue has been triaged (blank type means enhancement).
An issue actively being worked on should have the "assigned" field set when possible.
Default GitHub labels: bug, duplicate, enhancement, help wanted (help is always wanted), invalid, question, wontfix.

cc @TomAugspurger @jcrist @shoyer @jakirkham @ogrisel
Companies like NVIDIA would like to use labels when doing internal project management.  They'd like to avoid having their own issues for work, and centralize on the dask issue tracker.  I think that there is some balance to be struck between adding structure and making sure that that structure is not company specific.  Your input on this topic would be very welcome.

Labels (all of subproject, type of issue and workflow) and assignees are an excellent idea, even if only the companies/people that want to make use of them actually use them. Getting good labels may not be entirely straight-forward, and I think it may be a mistake to force OSS contributors from having to label everything.

No objection to using labels / projects more.
I don't see much in the initial post about milestones. Do we want to use those? Pandas has one milestone per release. Issues are tagged with the milestone we'd like to have it done by (with a bucket "someday" release for things none of the maintainers plan to work on). PRs are added to the milestone they're merged for, making it easy to see what version a PR is available in.
Is a given issue expected to have exactly one of each component? For example, I could see an issue being laeled as "good first issue" and "bug" (though I don't really see the value in a "bug" label).
I'm also somewhat against a "community" label, since that reinforces the distinct between paid and volunteer developers.
Size sounds hard to estimate, some issues more than others. Not to say we shouldn't try.

and I think it may be a mistake to force OSS contributors from having to label everything.

One challenge here is that only people with write access can assign labels.  We're in an odd position where there may be some benefit to giving someone with a corporate project management role write access.  (To be clear, I'm not necessarily in favor of this, just stating a tension).

No objection to using labels / projects more

Same here, I think this would be helpful

I'm also somewhat against a "community" label, since that reinforces the distinct between paid and volunteer developers

I think the community label is more for general project discussions like roadmaps, planning, upcoming sprints, etc. that anyone can be involved in (see https://github.com/dask/dask/issues?utf8=✓&q=label%3ACommunity+ for the issues that have the community label in the Dask issue tracker).

PRs are added to the milestone they're merged for, making it easy to see what version a PR is available in.

As in @martindurant 's comment above, I'm somewhat hesitant to add any maintenance burden to people who are reviewing and merging PRs.  Currently this is usually me, and I'm often a bottleneck.  I'd be much more comfortable with this if we're able to diffuse review and merging burden.

only people with write access can assign labels

I didn't realise that. I suppose that brings in the "supporting entity" concept (or whatever we called it), that any organisation that wants to be able to assign labels should be an official supporter and thereby have at least one proven person with commit access.

I have no objections to giving commit access to anyone who wants to do
project organization tasks. We can generally trust people to do the right
thing and not step beyond their bounds of expertise. The caveat would be
for genuine security risks, but I don't think that's a concern here.
On Tue, Feb 12, 2019 at 12:10 PM Martin Durant<EMAIL_ADDRESS>wrote:

only people with write access can assign labels
I didn't realise that. I suppose that brings in the "supporting entity"
concept (or whatever we called it), that any organisation that wants to be
able to assign labels should be an official supporter and thereby have at
least one proven person with commit access.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask/issues/4477#issuecomment-462915200, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABKS1idLX2TZuPBZkiMwpn5gbu0jcbVkks5vMx-xgaJpZM4a3vCO
.

One common strategy that other OSS projects have is to add a descriptor to the issue title (e.g. ENH, BUG, etc.). We could at a minimum adopt an existing standard along these lines and add this information to our contributing guide and GitHub contributing templates. This would let us empower more users to include this information without needing to worry about a level of trust or having someone necessarily burdened with cleanup.

Continuing the discussion

Seems to be general consensus that labels would be good.
@TomAugspurger is right about the multiple meanings of "community"...  "discussion" may be clearer and more universal to people that are new or coming from other languages.
"bug" to me is useful as a red flag (and it should be the red label) on things that are broken, not matching the API, wrong, or blocking a user from adopting Dask. A bug means at least one person is blocked, and worst case the rest haven't noticed their results may be wrong. Bugs lead to retractions, retractions are very very bad.
Milestones are orthogonal but related to labels, I'd like to keep that as a separate discussion.
I believe @jakirkham is referring to Conventional Commits https://www.conventionalcommits.org/ which I have used before and love because of all the tooling it enables, and the component labels can match the commit scopes, but it requires some discipline by those who are doing the merges. Either way it's a very good idea to adopt. Component names should be single word and lowercase (and mostly are).

The good news is that existing labeling there is appears to be mostly consistent with the above.
Proposed Action Items:

Type "community" -> "discussion"
Create project component lists. (I plan to start with dask/distributed and related)
Use Conventional Commits as preferred commit/merge format, but not required.
Raise separate discussion about releases/milestones/project/etc.

Type "community" -> "discussion"

+1 from me

Create project component lists. (I plan to start with dask/distributed and related)

I'm curious what these would look like.  I interpret this as things like "bag", "array", "dataframe", "scheduling" and in the main repository.  Is this what you had in mind?

Use Conventional Commits as preferred commit/merge format, but not required.

I'm not sure that concensus has been reached on this.  I personally don't find the prefixes like BUG and ENH super helpful and tend to avoid process by default.  I would be curious to hear from people here about when having prefixes like these on all commits would have helped them with Dask maintenance work before moving forward on this.

Raise separate discussion about releases/milestones/project/etc.

Sounds good to me

To clarify my original suggestion, the use of prefixes like BUG and ENH are suggested to be used with issue/PR titles. The point being a random user doesn't necessarily have the option to label an issue/PR, but they can prefix the title with ENH for example. Am not opposed to using them in commits, but as we squash commits this should mostly follow from the PR title. The acronyms are taken from NumPy's dev docs as these are frequently used by projects in the ecosystem.

I've found the "Need Info" label helpful. It's applied when a user submits a bug report that isn't (easily) reproducible or a vague feature request. I'll periodically go through issues labeled with "Needs Info" and either ping the reporter or close them.

This topic was discussed in the meeting today: https://docs.google.com/document/d/1UqNAP87a56ERH_xkQsS5Q_0PKYybd5Lj2WANy_hRzI0/edit?usp=sharing
A brief summary:
We're pretty happy with the proposed labels.

Size estimates are difficult. Trialing for the next month, and it's up to the maintainer to assign them (or not) as they see fit.
Assigning issues is difficult: We don't want to discourage someone else from picking up an issue because it's been "assigned" and dropped. Trialing for the next month; maintainers can assign as they see fit and we'll check in.

What are the next steps? Do we need to ensure that all the repositories of interest (need to specify exactly what these are) have the same "label type" and "label size" labels, and "component" labels specific to that repository?

From last week's meeting and summarizing the above discussion, I was going to start labeling issues (to the best of my abilities).  We are going to keep the labels at least through SciPy sprints and evaluate if the labels were helpful/harmful/indifferent in the lead up to and during the sprints.
Before proceeding I would like to get consensus on the the labels:
Type Label
bug
question
good first issue
discussion
need info
Component label
documentation
cicd
bag
array
dataframe

I generally like this proposed list of labels @quasiben, thanks for continuing this discussion. I'd put forward that we also keep the current "io" label as it's used relatively frequently.

Also, I'm happy to help with labeling existing (and new) issues. We should probably make that "discussion" label so we can label this issue : )

@quasiben, your proposal seems reasonable. Do you know if there is a way to ask users to select a type and then some components?

I think we can ask uses to optionally select labels when creating an issue.  The issue template will need to be amended:
https://help.github.com/en/articles/creating-issue-templates-for-your-repository
Additionally, for me to start applying labels I think I need write access to the repository.  Perhaps this should wait until https://github.com/dask/governance/pull/12 is merged

@quasiben , you now have rights.
This issue could be closed (we agreed in principle), although setting of the issue template has not yet happened.

closing


Question about mouse_released field in Io

Is there any reason as for why this is not a public field? This is useful for, say, reimplementing IsMouseReleased. I haven't seen this addressed in an issue here before, so I thought I'd ask.
Obviously - I can use imgui::sys::igIsMouseReleased, but I don't think that it's particularly elegant - which is why I think, wouldn't it be better for the library to simply implement such (useful) methods inside the Io impl?

Sorry - didn't notice it's in mouse.rs


Documentation

Please provide some pages with documentation containing examples of how to use this module. Here in the README and also at https://pypi.org/project/unicodedata2/ if possible

This is a backport - the documentation is the same as the unicodedata package in python 3. The examples are there. I’ll add a link to that.

Thanks but still missing description at https://pypi.org/project/unicodedata2/ (The author of this package has not provided a project description) and link to https://pypi.org/project/unicodedata2/ in https://github.com/mikekap/unicodedata2/blob/master/README.md

I'll fix that, but that will only show up for the next release (and I don't think it's worth doing another release just for that).


Just jotting down notes.

steamcmd +@NoPromptForPassword 1 +login anonymous +workshop_download_item<PHONE_NUMBER>22600 +quit

https://github.com/dgibbs64/SteamCMD-Commands-List/blob/master/steamcmdcommands.txt


implement account with seed in system program

This PR implements create_account_with_seed similar to
https://github.com/solana-labs/solana-web3.js/blob/44f32d9857e765dd26647ffd33b0ea0927f73b7a/src/system-program.ts#L743-L771

@michaelhly please let me know if I should make any changes before merge

I'm not a maintainer but you'll want to write a test for this in tests/unit/test_system_program,py. Should be simple to imitate an existing test

@kevinheavey thanks for the advice! I've added a unit test test_create_account_with_seed and implemented the  decode_create_account_with_seed function that was missing from the original commit

Looks good. But you need to run make fmt. Thanks!


Footer images are highlighted with inverted color

If you highlight everything on the page with Ctrl + A, everything should be highlighted with the same color (in my case, blue). However, currently the images in the footer are the inverse of that color (in my case red), because of the filter: invert(1) CSS declaration.

Interesting finding :D
A fix could be to change the colors in the SVG files or use some kind of CSS trick to change the SVG fill color.


Linux CLI do not support Copy/Paste

Hi, it depends on terminal app you are running, for some of them Ctrl + Shift + V works, somewhere Insert button, and somewhere you need to right click and paste/copy.


Add bicep template for creating Azure resources

@shpathak-msft Can we add a teardown mechanism as well?


fix go-lint error

Signed-off-by: Sally O'Malley<EMAIL_ADDRESS>

fixes this:
pkg/rpm/rpm.go:8:1: comment on exported function Version should be of the form "Version ..."

Merged.


Deserialization error with generically-typed events

I'm trying to add metadata to events (see #780, IMO this should be supported by the framework at a basic level), and went the route of adding a wrapper around each event, as such:
public class Envelope<T>
{
    public T Value { get; set; }
    public Guid ExecutingUserId { get; set; }
}

public class Created
{
    public Guid Id { get; set; }
}

public class Updated
{
    public String UpdateValue { get; set; }
}

Code to manage event streams:
public async Task StartStream<TEvent>(Guid streamId, TEvent _event)
{
    using(var session = _store.OpenSession())
    {
        session.Events.StartStream(streamId, (object)new Envelope<TEvent>
        {
            Value = _event,
            ExecutingUserId = GetUserIdFromSomewhere()
        });
        await session.SaveChangesAsync();
    }
}

public async Task Commit<TEvent>(Guid streamId, TEvent _event)
{
    using(var session = _store.OpenSession())
    {
        session.Events.Append(streamId, (object)new Envelope<TEvent>
        {
            Value = _event,
            ExecutingUserId = GetUserIdFromSomewhere()
        });
        await session.SaveChangesAsync();
    }
}

Events commit happily, updated projections correctly, and even persist the correct event type to the database.
Querying the event stream throws an exception though. Stream loading code:
public async Task<IReadOnlyList<IEvent>> LoadStream(Guid id)
{
    using(var session = _store.OpenSession())
    {
        return await session.Events.FetchStreamAsync(id);
    }
}

Attempting to load a stream with two different events (Envelope<Created> and Envelope<Updated>) results in all events attempting to be deserialized using the type of the first event (Envelope<Created>).
The error that surfaces is a JSON parsing issue (JsonReaderException: Unexpected character encountered while parsing value), but I ran intermediate code under the JSON parser and discovered that the second event is attempting to be deserialized using the Type of the first event.
Troubleshooting I've done:

Fetch just the second event (Envelope<Updated>) to verify that there's no inherent problem with that record. Using QueryAllRawEvents() and skipping the first record, it deserializes correctly and I get a nice Envelope<Updated>
Remove my Envelope<> wrapper and just persist the events (maybe my code is the problem). This works, which means that the problem is probably down in the Marten framework itself.

Hopefully that's useful debugging information - if you can point me in the right direction I may have a go at fixing it myself.

@eouw0o83hf It's not a serialization error per se, it's just that Marten is probably being too naive about how it stores the event type name. Closed generic event types really wasn't something we anticipated here. That's really not a case I'm excited about supporting because the reflection wok around that is always a huge mess.
It'd be easier to deal with this with a simple layer supertype for your event classes, even without waiting for us to add the event metadata

@jeremydmiller I'll do some more research, but I seem to remember getting correct types out if I queried the data in different ways, which led me to believe that Types are being persisted correctly but maybe reinstantiated naively.
I wound up going the supertype route, it's probably an overall less-complex way to deal with it.
I have some time coming up to get the Marten solution setup locally, I'll see if I can knock this out.

Fixed in a local branch. This will be in Marten 2.10.0 and 3-alpha-2 tomorrow (8/22)


 I would like to as is there a way to turn it off or make it disappear? thanks

btw, in the preferance->help-> auto connections, I already turned off all the options, but it doesn't help

Sorry, that's not possible.

Tools --> Preferences --> Completing and linting --> Introspection --> Enable hover hints
Untincking Enable hover hints worked for me.

Tools --> Preferences --> Completing and linting --> Introspection --> Enable hover hints
Untincking Enable hover hints worked for me. It was so frustrating hints appearing at desktop prior to switching tasks.

Great help !!! many thanks !!!

Untincking Enable hover hints worked for me. It was so frustrating hints appearing at desktop prior to switching tasks.

This was fixed in 4.1.3. Please update.


[Bug report]

Version
4.5.2
Platforms
dart, Android, iOS
Device Model
all device
flutter info
[✓] Flutter (Channel stable, 3.10.2, on macOS 13.3.1 22E772610a darwin-x64, locale zh-Hans-CN)
    • Flutter version 3.10.2 on channel stable at /Users/linvi7/project/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 9cd3d0d9ff (7 days ago), 2023-05-23 20:57:28 -0700
    • Engine revision 90fa3ae28f
    • Dart version 3.0.2
    • DevTools version 2.23.1
    • Pub download mirror https://pub.flutter-io.cn
    • Flutter download mirror https://storage.flutter-io.cn

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/linvi7/Library/Android/sdk
    • Platform android-33, build-tools 30.0.3
    • Java binary at: /Applications/Android Studio 2.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E222b
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.2)
    • Android Studio at /Applications/Android Studio 2.app/Contents
    • 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 17.0.6+0-17.0.6b802.4-9586694)

[✓] IntelliJ IDEA Ultimate Edition (version 2019.1.4)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 44.0.1
    • Dart plugin version 191.8593

[✓] VS Code (version 1.77.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension can be installed from:
      🔨 https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Connected device (3 available)
    • iPhone 14 Pro Max (mobile) • C063CAE9-DEAA-486A-A8DC-4D8132C42B5E • ios            • com.apple.CoreSimulator.SimRuntime.iOS-16-4 (simulator)
    • macOS (desktop)            • macos                                • darwin-x64     • macOS 13.3.1 22E772610a darwin-x64
    • Chrome (web)               • chrome                               • web-javascript • Google Chrome 113.0.5672.126

[✓] Network resources
    • All expected network resources are available.

How to reproduce?
when I build FlutterBeanFactory and generated document, and then it throw some error logs after I run the project.
I try to restart my pc, upgrade Android Studio, but it doesn't work.
Logs
lib/generated/json/base/json_convert_content.dart:26:51: Error: Member not found: 'fromJson'.
		(GetRequestEntity).toString(): GetRequestEntity.fromJson,
		                                                ^^^^^^^^
lib/generated/json/base/json_convert_content.dart:27:53: Error: Member not found: 'fromJson'.
		(PostRequestEntity).toString(): PostRequestEntity.fromJson,
		                                                  ^^^^^^^^
lib/generated/json/base/json_convert_content.dart:28:53: Error: Member not found: 'fromJson'.
		(GetResponseEntity).toString(): GetResponseEntity.fromJson,
		                                                  ^^^^^^^^
lib/generated/json/base/json_convert_content.dart:29:55: Error: Member not found: 'fromJson'.
		(PostResponseEntity).toString(): PostResponseEntity.fromJson,

Example code (optional)
@JsonSerializable()
class GetRequestEntity {

	GetRequestEntity();

	factory GetRequestEntity.fromJson(Map<String, dynamic> json) => $GetRequestEntityFromJson(json);
	String? getNumber;
	String? getName;

	Map<String, dynamic> toJson() => $GetRequestEntityToJson(this);

}

@JsonSerializable()
class PostRequestEntity {

	PostRequestEntity();

	factory PostRequestEntity.fromJson(Map<String, dynamic> json) => $PostRequestEntityFromJson(json);
	String? postNumber;
	String? postName;

	Map<String, dynamic> toJson() => $PostRequestEntityToJson(this);
}

...

class JsonConvert {
	static final Map<String, JsonConvertFunction> convertFuncMap = {
		(GetRequestEntity).toString(): GetRequestEntity.fromJson,
		(PostRequestEntity).toString(): PostRequestEntity.fromJson,
		(GetResponseEntity).toString(): GetResponseEntity.fromJson,
		(PostResponseEntity).toString(): PostResponseEntity.fromJson,
	};

......

}

Contact
No response

Thanks


Not syncing in Linux (.deb package)

Hi,
I tried to install and try the utility in Linux (Pop_OS) but it never finishes syncing initial data (gets stuck at 63%). I would try to fix it myself but I am not yet familiarized with the project.

The Dat repository that it's trying to sync isn't fully available. I'm waiting for either the original full source to come back online or it may need to be recreated.

@bencevans thank u for the info. Let's hope it to be fixed. I see that the project is still on the go. I had my doubts.

@txusser I was able to successfully sync on Pop!_OS today. Is it working now for you?

For each datasource there are two Dat repositories, Feed and Content.
The initial sync your seeing is the sync of the Feed repository for the default eLife datasource.
The eLife datasource is currently fully available and mirrored by my server and I think on HashBase so the initial sync should be completing without issue.
The problem currently is that not all the papers from the Content repository is being mirrored thus only partially available. When you try to download some papers you may find that they won't finish downloading and this is because they're the files that aren't currently available.
To make them available a user that has a copy needs to come online so that my server and anyone else can download and mirror. This is being tracked in #172.


[New Issue]: Useless PRs created by wingetbot

Everyday @wingetbot creates many PR for packages like https://github.com/microsoft/winget-pkgs/pull/138683 https://github.com/microsoft/winget-pkgs/pull/138680 https://github.com/microsoft/winget-pkgs/pull/138661 https://github.com/microsoft/winget-pkgs/pull/138671 https://github.com/microsoft/winget-pkgs/pull/138607
When u see the changes in file, u can see that it only changes 1 line:
-  # Automatically updated by the winget bot at 2024/Feb/09
+  # Automatically updated by the winget bot at 2024/Feb/10

There is no actual new version added or old version removed.
Also installer hash is not changed.

I've been tracking this and reporting it to the team internally. We had about 45 of these last week. I believe there is a fix for this set to be added soon.

I've been tracking this and reporting it to the team internally. We had about 45 of these last week. I believe there is a fix for this set to be added soon.

Has this now been fixed since the update? Bot looks to be running better now.


Updated Microservices Diagram

Updated the microservices diagram with better clarity on interaction.

Nice diagram. Better than mine.
What editor did you used?

This is using power point and then saved as png


allow arguments to be passed to functions

probably something in there that will break things, but passes all tests and works so far for simple cases

Closing this as there are new discussions about this topic


Add new tests for Anlogic architecture

Problems/questions:
- memory.ys: ERROR: Failed to import cell gate.mem.0.0.0 (type
EG_LOGIC_DRAM16X4) to SAT database.
Why EG_LOGIC_DRAM16X4, not AL_LOGIC_BRAM?
- Internal cell type $TBUF  is present.

Created #1434 to continue work on this


Hide commands

namespace App\Telegram\Commands;

use SergiX44\Nutgram\Handlers\Type\Command;
use SergiX44\Nutgram\Nutgram;

class HiddenCommand extends Command
{
    protected string $command = 'admin';

    protected ?string $description = 'Admin command';

    public function handle(Nutgram $bot): void
    {
        if ($this->isHidden($bot)) {
            return;
        }

        $bot->sendMessage('Shhh!');
    }

    public function isHidden(Nutgram $bot): bool
    {
        return $bot->getUserData('commands', default: [])[$this->command]['hidden'] ?? true;
    }
}

🤔 Sorry, but I cant find a real use case for this. The isHidden method is used internally to check if a command should be registered or not when setting the bot commands. So is invoked way before an handler, so it doesnt make sense passing an instance that is processing an update


[v1.5-Tests-'AdaptiveCard.Rtl.True.json']: Voice over focus get trapped at once place and does not move to another control and text in 'AdaptiveCard.Rtl.True.json' card.

https://microsoft.visualstudio.com/OS/_workitems/edit/37857685

As per above comment, this card is not for public user and also not available in latest adaptive card build.
Verified and Closed already


Difficulty in understanding how plush works.

So I'm trying to add float support to plush mainly to be able to test floats in a higher level language.
Now, I'm not fully sure about how the front end for plush works.
My main point of confusion is the fact that the parser has some parts written in cpp and others in plush.
Moreover, the runtime is written in plush but the code generation in cpp.
So the runtime.pls is firstly interpreted and it seems that it adds support to overloaded symbols, such as +
The code generation in cpp produces .zim code and when it encounters an overloaded symbol it uses functions defined in runtime
What about parser.pls? Is it ever used?

The reason it might be confusing is that there are two plush implementations. There is a C++ implementation (parser.cpp, codegen.cpp, etc), and there is also a plush-written-in-plush implementation (parser.pls). The runtime.pls file is, as you pointed out, used to define certain symbols. It also implements the behavior of primitive operators in plush (what happens when you do x + y). Both implementations make use of it.
When you compile a file with cplush (the C++ implementation), it gives you a standalone zim file. Now, cplush is used to compile parser.pls into packages/lang/plush/0/package.
You might be wondering why this is useful. The reason is, once we have a compiled parser package, Zeta has support for plush. When you run a source file with a line that says `#language "lang/plush/0" at the top, it's telling ZetaVM to load this parser package and use this to parse the source code found in the file.
Eventually, Zeta will have a package manager, and people will be able to upload packages that implement parsing/codegen for their own languages... And by doing that, they will be giving everyone instant access to their new language in a way that will feel native. What's more is that code written in plush will be able to import code written in any other language supported by ZetaVM. That is a big part of my "grand plan".
Back to the original problem. The C++ Plush implementation is more bare-bones and less user-friendly. I created it so that I would have a tool to write the self-hosted code with, but it only has to have just enough features. It doesn't support the "import" statement, for example. Eventually, I might like for parser.pls to replace the C++ implementation, but at this stage, I think it's wise to keep the C++ implementation in working order... If only because it gives people an example of how to write a standalone compiler, in a language that they are likely to be familiar with.
You could add FP support to either the C++ or the self-hosted parser.pls implementation. It might make more sense to implement it in parser.pls, but I understand if you find that confusing and would prefer to write C++. If you want, you can do C++ support and I can write the parser.pls version. I will let you choose :)

I'll try to implement floats in the plush version!
I'll try my best to have it ready as soon as possible, but till Wednesday I'm a different conferences so I'll be a bit busy.
Thanks for the clarification.

@Maaarcocr No problem. Deep learning talks at work for me today.
I did make some changes since your last PR. I changed the int type to int32. Make sure to do a git pull when you look at the code again.
Enjoy the conferences :)

Small update:
I'm sorry, but I'm actually going to implement is in c++. I just find it easier to implement it due to the presence of atof.
I may actually implement floats into the c++ one, so that I can use str_to_f32 in the plush integration later on.
I hope that my change of mind doesn't cause any problem.

Sure, no problem. This project is meant to be fun too :)

It is fun :) I'm learning a good amount of things, thanks!


Include Citizen Science

Add topic Citizen Science.

For all topic suggestions we have an etherpad collecting material and ideas: https://pad.okfn.org/p/OpenScience101topicsuggestions


Update OpenFaaS + AKS post

Add basic auth by default
Remove old section pointing to basic auth through proxy
Use helm chart repo instead of git clone
Point to workshop as next step at end
Create namespaces through official method

Signed-off-by: Alex Ellis<EMAIL_ADDRESS>cc @justindavies @squillace

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: alexellis sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Hi @alexellis - In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just a few minutes: Sign the CLA. Thank you.

@alexellis : Thanks for your contribution! The author, @justindavies, has been notified to review your proposed change.

@alexellis : Thanks for your contribution! The author, @justindavies, has been notified to review your proposed change.

CLA should be handled now.

@GitHubber17 : Thanks for your contribution! The author, @justindavies, has been notified to review your proposed change.


[Bug] IntIndexer causes a memory leak

Describe the bug
Using the SOMA IntIndexer is causing a memory leak which isn't present when using the regular Pandas indexer.
To Reproduce
import gc

import numpy as np

import cellxgene_census
from cellxgene_census.experimental.pp._highly_variable_genes import _get_batch_index
import tiledbsoma as soma

experiment_name = "homo_sapiens"
batch_key = ["suspension_type", "assay"]

census = cellxgene_census.open_soma(census_version="2023-12-15")
query = census["census_data"][experiment_name].axis_query(measurement_name="RNA")

batch_index = _get_batch_index(query, batch_key=batch_key)
batch_indexer = soma.IntIndexer(batch_index.index.to_numpy(), context=query.experiment.context).get_indexer
for i, arrow_tbl in enumerate(query.X("raw").tables()):
    print(i)
    _batch_take = batch_indexer(arrow_tbl["soma_dim_0"])
    gc.collect()

Versions (please complete the following information):
The bug is present on the most recent tiledbsoma version.

I managed to cut down the reproducer a little further, and think I've identified where the leak lives.
Starting with a similar setup:
import numpy as np
import pyarrow as pa

import cellxgene_census
from cellxgene_census.experimental.pp._highly_variable_genes import _get_batch_index
import tiledbsoma as soma

rng = np.random.default_rng()

experiment_name = "homo_sapiens"
batch_key = ["suspension_type", "assay"]

census = cellxgene_census.open_soma(census_version="2023-12-15")
query = census["census_data"][experiment_name].axis_query(measurement_name="RNA")

batch_index = _get_batch_index(query, batch_key=batch_key)
batch_indexer = soma.IntIndexer(batch_index.index.to_numpy(), context=query.experiment.context).get_indexer

This does not cause memory to grow:
%load_ext memory_profiler
%memit
# peak memory: 3287.59 MiB, increment: 0.00 MiB

for i in range(10_000):
    x = rng.choice(len(batch_index), 100_000, replace=False)
    _ = batch_indexer(x)

%load_ext memory_profiler
%memit
# peak memory: 3288.59 MiB, increment: 0.00 MiB

But if we instead pass the array as a pyarrow array, we see memory usage increase:
for i in range(10_000):
    x = pa.array(rng.choice(len(batch_index), 100_000, replace=False))
    _ = batch_indexer(x)

%load_ext memory_profiler
%memit
# peak memory: 10932.71 MiB, increment: 0.00 MiB

We can see that there is a branch for this here: https://github.com/single-cell-data/TileDB-SOMA/blob/765a12c4eaecb232f5af13c839f7a7194d9b3a01/apis/python/src/tiledbsoma/_indexer.py#L82-L86
Which ends up calling the c++ code here: https://github.com/single-cell-data/TileDB-SOMA/blob/765a12c4eaecb232f5af13c839f7a7194d9b3a01/apis/python/src/tiledbsoma/reindexer.cc#L86
Which is probably where the leak is.

Repro: https://gist.github.com/johnkerl/120a2ffa329726a963d8a6bbc7dfa161
[sc-48535]


Some Images Are Failing For X-Platform But Not iOS

Figure what what's going on here.

It's looking like there might be multiple invertebrates (which are keyed by their name) using the same image. During image sync switching to a QMultiMap solves that problem.
I think that I've also got a problem with how I'm slicing (mid-ing) the list of image titles.

Not a slice issue. I hadn't completely implemented the system for matching invertebrates to their images because most times the file name is the same, but sometimes the file names have their spaces replaced with underscores, and other sometimes the underscores are replaced by spaces.
This is fixed in settings.

Merged.
p.s. It'd be good to have this fixed on the server side so that the app doesn't have to try and guess what the image is really named. Still at this point it's handled and not too burdensome.


"Bălțați", "Bircii", "Chițeasca", "Constantinești", "Jitaru", "Mărgineni-Slobozia", "Mihăilești-Popești", "Mogoșești", "Negreni", "Piscani", "Rusciori", "Șuica", "Teiuș" are listed in the drop-down menu

Description:
When user writes "Bălțați", "Bircii", "Chițeasca", "Constantinești", "Jitaru", "Mărgineni-Slobozia", "Mihăilești-Popești", "Mogoșești", "Negreni", "Piscani", "Rusciori", "Șuica" and "Teiuș" the locations appear in the drop-down menu next to the comune and the county which is a part according to the law
Precondition:
The website is up an running.

Step 1
Write in the search bar "Bălțați"
Expected results
The location appears in the drop-down menu as "Sat Bălțați, OLT (SCORNICEȘTI)"

Step 2
Press "x" button.
Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Bircii"
Expected results
The location appears in the drop-down menu as "Sat Bircii, OLT (SCORNICEȘTI)"

Step 4
Press "x" button.
Expected results
The location was deleted from the search bar.

Step 5
Write in the search bar "Chițeasca"
Expected results
The location appears in the drop-down menu as "Sat Chițeasca, OLT (SCORNICEȘTI)"

Step 6
Press "x" button.
Expected results
The location was deleted from the search bar.

Step 7
Write in the search bar "Constantinești"
Expected results
The location appears in the drop-down menu as "Sat Constantinești, OLT (SCORNICEȘTI)"

Step 8
Press "x" button.
Expected results
The location was deleted from the search bar.

Step 9
Write in the search bar "Jitaru"
Expected results
The location appears in the drop-down menu as "Sat Jitaru, OLT (SCORNICEȘTI)"

Step 10
Press "x" button.
Expected results
The location was deleted from the search bar.

Step 11
Write in the search bar "Mărgineni-Slobozia"
Expected results
The location appears in the drop-down menu as "Sat Mărgineni-Slobozia, OLT (SCORNICEȘTI)"

Step 12
Press "x" button.
Expected results
The location was deleted from the search bar.

Step 13
Write in the search bar "Mihăilești-Popești"
Expected results
The location appears in the drop-down menu as "Sat Mihăilești-Popești, OLT (SCORNICEȘTI)"

Step 14
Press "x" button.
Expected results
The location was deleted from the search bar.

Step 15
Write in the search bar "Mogoșești"
Expected results
The location appears in the drop-down menu as "Sat Mogoșești, OLT (SCORNICEȘTI)"

Step 16
Press "x" button.
Expected results
The location was deleted from the search bar.

Step 17
Write in the search bar "Negreni"
Expected results
The location appears in the drop-down menu as "Sat Negreni, OLT (SCORNICEȘTI)"

Step 18
Press "x" button.
Expected results
The location was deleted from the search bar.

Step 19
Write in the search bar "Piscani"
Expected results
The location appears in the drop-down menu as "Sat Piscani, OLT (SCORNICEȘTI)"

Step 20
Press "x" button.
Expected results
The location was deleted from the search bar.

Step 21
Write in the search bar "Rusciori"
Expected results
The location appears in the drop-down menu as "Sat Rusciori, OLT (SCORNICEȘTI)"

Step 22
Press "x" button.
Expected results
The location was deleted from the search bar.

Step 23
Write in the search bar "Șuica"
Expected results
The location appears in the drop-down menu as "Sat Șuica, OLT (SCORNICEȘTI)"

Step 24
Press "x" button.
Expected results
The location was deleted from the search bar.

Step 25
Write in the search bar "Teiuș"
Expected results
The location appears in the drop-down menu as "Sat Teiuș, OLT (SCORNICEȘTI)"


[BACKEND] Fix fma mixed-precision

and expose the allow_tf32 argument to the matmul op
@shunting314

The bitwise difference seems to be caused by changes of kernel names. @zahimoud

The bitwise difference seems to be caused by changes of kernel names? @zahimoud

Yes, seems like it. We can't tell from this comparison whether the kernels that changed names are still bitwise identical, I think that would have to be manual work for now, if really needed..

Thanks. May I know where do I find the name of the updated kernel? In other words, which tests did you use to assemble kernel names in kernels.yml?

Thanks. May I know where do I find the name of the updated kernel? In other words, which tests did you use to assemble kernel names in kernels.yml?

We stopped using kernels.yml, we currently clear the cache and run test/unit/operators, and then try to match kernel names in the current cache versus main cache: here are the CI steps

Thanks. May I know where do I find the name of the updated kernel? In other words, which tests did you use to assemble kernel names in kernels.yml?

We stopped using kernels.yml, we currently clear the cache and run test/unit/operators, and then try to match kernel names in the current cache versus main cache: here are the CI steps

I see, in this case, do I need to update anything for the bitwise warning? Maybe not?

I confirmed that this PR fixed https://github.com/openai/triton/issues/2156#issuecomment-1689009505 . But there are more issues surfaced with differnet combination of dtypes. I'll add more details in the GH issue.

Thanks. May I know where do I find the name of the updated kernel? In other words, which tests did you use to assemble kernel names in kernels.yml?

We stopped using kernels.yml, we currently clear the cache and run test/unit/operators, and then try to match kernel names in the current cache versus main cache: here are the CI steps

I see, in this case, do I need to update anything for the bitwise warning? Maybe not?

No, if you are confident that the kernels with changed names are still bitwise identical, then you can just ignore the warning.

Thank you!


[TEST] Use different Rocky Linux AMI

What's the test to develop? Please describe
The Rocky Linux AMI we use for testing does not have the latest versions and doesn't have ARM images. Use the official image instead.

@ejweber is this finished? It seems all PRs have been merged. If all good, let's close it directly.
cc @longhorn/qa

https://github.com/longhorn/longhorn-tests/pull/1426

@innobead and @longhorn/qa:
We still need https://github.com/longhorn/longhorn-tests/pull/1426 to ensure Rocky testing with SELinux enabled works correctly. Since the issue that owned it was closed, I've tagged it here. We should close this directly as requested as soon as that one merged.

[ ] https://github.com/longhorn/longhorn-tests/pull/1426


[Bug] C++ cast caerEventPacketHeader + vector.data() change

After work in https://github.com/inilabs/caer/issues/77 , I discovered a sublime bug in libcaer's C++ behavior.
Or actually it behaves correctly, but it's highly unintuitive.
Use-case:

Working with C++ headers, I read data of packet header (28B), now I need to get eventType to construct proper <typed>EventPacket (eg PolarityEventPacket).

Code I used:
    auto packet = this->net.rcvAll(CAER_EVENT_PACKET_HEADER_SIZE);
    auto header =(caerEventPacketHeader)packet.data();
    // append packet header + data to form a full packet
    const size_t DATA_SIZE = header->eventSize * header->eventCapacity;
    const auto data = net.rcvAll(DATA_SIZE);
    packet.insert(end(packet), begin(data), end(data));
// STEP 3: parse polarity packets
    if ((int)header->eventType == (int)POLARITY_EVENT) { //do your work ...

Do you see a bug in there?
...I wasn't getting any packets processed by my code
 // append packet header + data to form a full packet
    const auto data = net.rcvAll(DATA_SIZE);
cout<< "eType="<<header->eventType<<" POLAR_T="<<POLARITY_EVENT<<endl; //1: prints eg eType=1 POLAR_T=1
    packet.insert(end(packet), begin(data), end(data));
cout<< "eType="<<header->eventType<<" POLAR_T="<<POLARITY_EVENT<<endl; //2: always prints eType=0 POLAR_T=1
    header =(caerEventPacketHeader)packet.data(); //hack,fix
cout<< "eType="<<header->eventType<<" POLAR_T="<<POLARITY_EVENT<<endl; //3: correctly prints again eg eType=1 POLAR_T=1

I'd like to fix this behavior! Some proposed solutions:

proper C++ EventPacketHeader class that owns the resource:
Expected usage:

  auto header = EventPacketHeader(packet);
  cout << header->eventType;
  packet.insert(...);
  cout << header->eventType; //same!

allow generic EventPacket()
Which already has a constructor using caerEventPacketHeader + data, but throws assert on "use proper EventPacket to ensure correct RTTI ..."
Can we allow to construct with the header only (to access the header values) + add method

T castTo<T>(vector data) {
  assert(T==eventType);
  vector new_v = headerData_.insert(...data);
  auto newHeader = (caerEventPacketHeader)new_v.data();
   return T(newHeader);
}
```

@llongi what do you think? Do you see a better approach?

The whole issue stems from your usage of a vector to hold the bytes. If you want the raw data, you need to access .data(), and the moment you .insert() on it, it gets reallocated and the old "header" pointer is invalid. There's nothing strange here, you're using a tool and need to be aware of its quirks.
An EventPacket is a block of memory with both a header and its data, and I don't want to break/change that assumption in C++, which is why the various EventPacket constructors expect a "full" event packet, and that will remain an invariant for them. Even if vector were to be supported directly, to avoid having to remember to call .data(), it would expect a full vector with all the bytes. And I'm not convinced it needs to be vector-aware, as that's just one of dozens of representations for "a bunch of bytes".
If you want to avoid having to check the type to build an EventPacket, in libcaercpp/events/utils.hpp there already is a couple of functions doing exactly that: makeSharedFromCStruct() and makeUniqueFromCStruct().
Now I agree that often you only want to look at the header and gets some info from it, so a solution similar to (1) is what I'm thinking about. Of course without the magic vector support.

After thinking about this some more, I've decided to not go the route of an extra C++ EventPacketHeader. The already present C version and its few, simple functions (caerEventPacketHeaderGet*() and friends) are perfectly sufficient for the job of extracting the few things one might need in this particular case of network clients. I have added caerEventPacketGetDataSize() and caerEventPacketGetSize() to simplify size calculation a bit, so now you can do:
auto packet = this->net.rcvAll(CAER_EVENT_PACKET_HEADER_SIZE);
const size_t DATA_SIZE = caerEventPacketGetDataSize((caerEventPacketHeader) packet.data());
const auto data = net.rcvAll(DATA_SIZE);
packet.insert(end(packet), begin(data), end(data));
std::unique_ptr<EventPacket> eventPacket = makeUniqueFromCStruct((caerEventPacketHeader) packet.data(), false);
if (eventPacket->getEventType() == POLARITY_EVENT) { ... }

That's easier and not so error prone. Also the make*FromCStruct() functions now can forward the 'takeMemoryOwnership' parameters, since memory is owned by your vector in this case.

Thank you for a very swift fix! And for the example :+1:

@llongi one more follow up, I using the new code, how would I cast the EventPacket to matching <typed>EventPacket?
Say,
std::unique_ptr<EventPacket> eventPacket = makeUniqueFromCStruct((caerEventPacketHeader) packet.data(), false);
if (eventPacket->getEventType() == POLARITY_EVENT) {
  PolarityEventPacket pp = (PolarityEventPacket)*eventPacket; //this does not work, somehow cast to avoid re-creating PolarityEventPacket
}

The C++11 way to do this is:
std::unique_ptr pp = static_cast<PolarityEventPacket *>(eventPacket.release());
Or if you don't want to keep the unique_ptr but just a basic pointer:
PolarityEventPacket *pp = static_cast<PolarityEventPacket *>(eventPacket.release());

Thank you!
I was just looking at the makeUniqueFromCStruct() code, can that be changed to return directly the specific cast? I'm assuming all the <type>EventPackets will have the common get*() from EventPacket header.
std::unique_ptr<EventPacket> makeUniqueFromCStruct(caerEventPacketHeader packet, bool takeMemoryOwnership = true) {
        switch (caerEventPacketHeaderGetEventType(packet)) {
                case SPECIAL_EVENT:
                        return (std::unique_ptr<SpecialEventPacket>(new SpecialEventPacket(packet, takeMemoryOwnership)));
                        break;

So that auto eventPacket = makeUnique..(); would already be PolarityEventPacket (or any other, specific); The user wouldn't have to specify anything, as the type can be recognized from the header.

std::unique_ptr pp = static_cast<PolarityEventPacket *>(eventPacket.release());

That did not seem to work, so I had to use dynamic_cast, which I found at
https://stackoverflow.com/questions/17417848/stdunique-ptr-with-derived-class
    if (eventPacket->getEventType() == POLARITY_EVENT) {
auto *tmp = dynamic_cast<PolarityEventPacket*>(eventPacket.get());
std::unique_ptr<PolarityEventPacket> polarityPacket;
if(tmp != nullptr)
{
    eventPacket.release();
    polarityPacket.reset(tmp);
}

At the first glance this seems slower (?) than reconstructing the object again as

const PolarityEventPacket polarityPacket = PolarityEventPacket((caerPolarityEventPacket) packet.data(), false);

No, that can't be done automatically. C++ resolves its types at compile time, so you can't change the return type of something based on a runtime integer somewhere in memory, the best you can do is return the closest common type (EventPacket in this case). I'm not sure why the .release() method doesn't work, it's the same we're using for implementing copy(). What is the error?
In the end there should be no slow-down at all for both methods, since the memory is still owned and part of the vector, what you're moving around is just a pointer to it. And sure, you're constructing an EventPacket class, but that is just a pointer and a boolean, VS constructing a std::unique_ptr, which is just a pointer usually. So the differences in memory usage and what's happening are negligible from a performance point of view, though building the std::unique_ptr should be marginally faster (less checks) and uses 4-8 bytes of memory less.

I'm not sure why the .release() method doesn't work, it's the same we're using for implementing copy(). What is the error?

error: no viable conversion from 'libcaer::events::PolarityEventPacket *' to
      'std::unique_ptr<PolarityEventPacket>'
std::unique_ptr<PolarityEventPacket> pp = static_cast<PolarityEventPacket *>(eventPacket.release());
                                     ^    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/usr/bin/../lib/gcc/x86_64-linux-gnu/5.4.0/../../../../include/c++/5.4.0/bits/unique_ptr.h:200:17: note: candidate constructor not viable: no known
      conversion from 'libcaer::events::PolarityEventPacket *' to 'nullptr_t' for 1st argument
      constexpr unique_ptr(nullptr_t) noexcept : unique_ptr() { }
                ^
/usr/bin/../lib/gcc/x86_64-linux-gnu/5.4.0/../../../../include/c++/5.4.0/bits/unique_ptr.h:205:7: note: candidate constructor not viable: no known conversion
      from 'libcaer::events::PolarityEventPacket *' to 'std::unique_ptr<libcaer::events::PolarityEventPacket,
      std::default_delete<libcaer::events::PolarityEventPacket> > &&' for 1st argument
      unique_ptr(unique_ptr&& __u) noexcept
      ^
/usr/bin/../lib/gcc/x86_64-linux-gnu/5.4.0/../../../../include/c++/5.4.0/bits/unique_ptr.h:356:7: note: candidate constructor not viable: no known conversion
      from 'libcaer::events::PolarityEventPacket *' to 'const std::unique_ptr<libcaer::events::PolarityEventPacket,
      std::default_delete<libcaer::events::PolarityEventPacket> > &' for 1st argument
      unique_ptr(const unique_ptr&) = delete;
      ^
/usr/bin/../lib/gcc/x86_64-linux-gnu/5.4.0/../../../../include/c++/5.4.0/bits/unique_ptr.h:220:2: note: candidate template ignored: could not match
      'unique_ptr<type-parameter-0-0, type-parameter-0-1>' against 'libcaer::events::PolarityEventPacket *'
        unique_ptr(unique_ptr<_Up, _Ep>&& __u) noexcept
        ^
/usr/bin/../lib/gcc/x86_64-linux-gnu/5.4.0/../../../../include/c++/5.4.0/bits/unique_ptr.h:228:2: note: candidate template ignored: could not match
      'auto_ptr<type-parameter-0-0>' against 'libcaer::events::PolarityEventPacket *'
        unique_ptr(auto_ptr<_Up>&& __u) noexcept;
        ^

In the end there should be no slow-down at all for both methods ...

Yes, I observed the slowdown with dynamic_cast (which was the only cast I managed), and it's known to be slower than the other casts.


Page URL

Page URL
@https://docs.flutter.dev/development/add-to-app/ios/project-setup#option-a---embed-with-cocoapods-and-the-flutter-sdk
Page source
No response
Describe the problem
When https://github.com/flutter/flutter/pull/101943 makes it to a tagged release, update The flutter_post_install method (recently added to Flutter), line to instead indicate the version in which it's available.
Introduced in https://github.com/flutter/website/pull/7032#issuecomment-1100241706.
Expected fix
No response
Additional context
No response
Originally posted by @jmagman in https://github.com/flutter/website/issues/7039

Closing as this is just a copy of an existing issue on the flutter website repo


Bug fix: remove htableName toUpperCase, for HBase tableName is case sensitive

Description
HBase is case sensitive，tableName.toUpperCase() will lead to the created table could not be found.
If my tableName of HBase is lowercase, kylin can't find the table created in the HFile generated step.
Improvement
just remove toUpperCase() for HBase tableName on master-hadoop3.1.

Can one of the admins verify this patch?

Codecov Report

:exclamation: No coverage uploaded for pull request base (master-hadoop3.1@4fc5e4a). Click here to learn what that means.
The diff coverage is 0%.

@@                 Coverage Diff                 @@
##             master-hadoop3.1     #717   +/-   ##
===================================================
  Coverage                    ?   24.99%
  Complexity                  ?     5972
===================================================
  Files                       ?     1385
  Lines                       ?    82254
  Branches                    ?    11533
===================================================
  Hits                        ?    20562
  Misses                      ?    59760
  Partials                    ?     1932

Impacted Files
Coverage Δ
Complexity Δ

...apache/kylin/storage/hbase/steps/CubeHFileJob.java
0% <0%> (ø)
0 <0> (?)

Continue to review full report at Codecov.

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

refer-jira: https://issues.apache.org/jira/browse/KYLIN-4058

Commit cherry-picked at https://github.com/apache/kylin/commit/55cdbab208de99079badb3de6e5b288231a81d52


UI to back-end web-socket connection.

Need a minimal wrapper over web-socket to handle some leanings from Shuttle demo.

Back pressure. Since we are trying to send real-time messages over a streaming protocol we must make sure that we do not overload the other side. Or the whole ting starts to lag. So we need some back-pressure protocol.
Keep alive messages. If there have been to long without messages we need to send a keep alive message.
Reconnect. We need to be able to automatically reconnect web sockets connections if its broken.

[ ] Find or design a back pressure protocol. RSocket?
[ ] Implement it on front and back.
[ ] Implement keep alive if necessary.
[ ] Implement reconnect. If necessary.
[ ] Monitoring.

We are archiving this repo.


Embedded image does not display on browser preview

Laravel Version: 7.0
PHP Version: 7.2.5
Database Driver & Version: mysql

Description:
the embedded image on browser preview showing nothing but ok in mailing
Steps To Reproduce:

Hey there,
Unfortunately we don't support this version anymore. Please check out our support policy on which versions we are currently supporting. Can you please try to upgrade to the latest version and see if your problem persists? We'll help you out and re-open this issue if so.
Thanks!


Pass multiple values to startsnotwith annotation

[x] I have looked at the documentation here first?
[x] I have looked at the examples provided that may showcase my question here?

Package version eg. v9, v10:
v10
Issue, Question or Enhancement:
I'm trying to use the Baked-in validation for string.
I would like to make sure a string field doesn't start with . or space or ,.
Is it possible to pass multiple values to the startsnotwith annotation?
Code sample, to showcase or reproduce:
	Details           *string                       `json:"details,omitempty" validate:"startsnotwith=.`

You can just chain them using OR. It would be something like this
Details string `json:"details,omitempty" validate:"startsnotwith=.|startsnotwith= |startsnotwith=,`

I did use the solution you provided and it works well.
I was hoping there is a shortened way to write this expression, something like:
Details string json:"details,omitempty" validate:"startsnotwith= . | 0x2C | ".
Thanks a lot.

I did use the solution you provided and it works well.
I was hoping there is a shortened way to write this expression, something like: Details string json:"details,omitempty" validate:"startsnotwith= . | 0x2C | ".
Thanks a lot.

It's a pity that there isn't.


Multiline prompt

I couldn't figure this out from the docs or the source code, so there may not currently be a way to do this. Right now there doesn't seem to be a way to have prompt-toolkit issue prompts on any line but the first.  For example, in xonsh, we see the current behaviour with scopatz/xonsh#601
scopatz@localhost ~ $ def g():
                          print(2)

scopatz@localhost ~ $ g()
2

However, what I would like to see is something like the following:
scopatz@localhost ~ $ def g():
.....................     print(2)
.....................
scopatz@localhost ~ $ g()
2

I think that this is more readable.  Basically the problem I was running into is that the get_tokens_func seems to only affect the first line.  Any ideas?

Hi @scopatz,
This has been implemented in one of the last commits today.
Have a look at this example: https://github.com/jonathanslenders/python-prompt-toolkit/blob/master/examples/get-multiline-input.py#L6
Please let me know if this doesn't work for you.

Thanks a million @jonathanslenders, this seems like it should work.  Any idea when this will make it into a release? I am hesitant to add code to xonsh master that hasn't been released by prompt-toolkit.

Hi @scopatz,
I will first try to create pull requests for all prompt-toolkit projects that are still using older versions to upgrade to the latest prompt-toolkit. (These are only a few.) Then I'll see whether the IPython guys have any special requirements. And then I'll do a new release. (Just trying to avoid releasing too quick and often.)
Jonathan

Sure thing! Is there a mailing list or anything where these get announced on?  They just sort of seem to happen unless you are watching the repo (which I am not).

No, right now we don't have a mailing list yet. (I will think about it, but often my mailbox is flooded enough that I easily forget to answer e-mails.)

my mailbox is flooded enough that I easily forget to answer e-mails

yeah.... I really hate email myself.  That is one of the reasons why I don't want to watch this repo.  It would be that many more emails.

For all the users of prompt-tookit that are in Debian, mycli, pgcli, and softlayer-python, they should work fine with the current release (due to the backward compatibility changes of mine you merged) - unless you added more changes that will break backward compatibility.  For those at least there's no need to do anything to have them update (and softlayer-python requires you to sign a CLA if you're going to contribute code to it).


Question: What is the correct CPE for ninja?

We're having trouble matching away false positives CVE's for this project given ninja is reasonably common name. Does this project even have an assigned CPE?

I don't think there is any (and I am in no way an authority on this project).
From searching the database at https://nvd.nist.gov/products/cpe/search, it looks like CMake uses [cpe:2.3:a:cmake_project:cmake:-:*:*:*:*:node.js:*:*] and GNU Make uses [cpe:2.3:a:gnu:make:-:*:*:*:*:*:*:*].
I would suggest using [cpe:2.3:a:ninja_build:ninja:-:*:*:*:*:*:*:*] as a CPE for Ninja, but frankly don't know how to register anything like that.


Cancels Moonkin Form Unnecessarily

Expected behaviour
Pressing ravm created macro should use random mount while in Moonkin Form.
Actual behaviour
Pressing ravm created macro while in Moonkin Form cancelauras Moonkin Form then uses a random mount.
Steps to reproduce this behaviour

Be in Moonkin Form
press /ravm
this is odd behavior because it is unnecessary to remove moonkin while mounting and doing so makes us use a global every time we dismount to get back to moonkin

@waldenp0nd

Hey @lesath-lestrange,
Sorry to just get back to this, I took a small break from the game.
However, I do think this has now been fixed in https://github.com/RavenousAddons/ravMounts/releases/tag/v2.8.3 which includes a fix that should direct Ravenous Mounts to cancel form only when the Travel Form option is ticked and the player is in Travel Form.  This means that there is no longer a delay when summoning your mount from Moonkin form.
Thanks for the report!.


LiquidThreads fix

halfak/Mediawiki-Utilities#12

Just to note, this pull request only makes it so that liquidthreads are ignored when encountered.  I created a new bug to track the fact that we still don't support liquid threads.  See https://github.com/halfak/Mediawiki-Utilities/issues/40
Thanks for your contribution.


one_hot cuda numbering error

File "train.py", line 211, in
main()
File "train.py", line 136, in main
loss = criterion(output, aff_cls.long())
File "/usr/local/lib/python3.5/dist-packages/torch/nn/modules/module.py", line 532, in call
result = self.forward(*input, **kwargs)
File "/workspace/new_aff_2020/nets/loss/dice.py", line 56, in forward
dice_loss = self.dice(output, target)
File "/usr/local/lib/python3.5/dist-packages/torch/nn/modules/module.py", line 532, in call
result = self.forward(*input, **kwargs)
File "/workspace/new_aff_2020/nets/loss/dice.py", line 23, in forward
target = make_one_hot(target.unsqueeze(dim=1), classes=output.size()[1])
File "/workspace/new_aff_2020/nets/loss/dice.py", line 10, in make_one_hot
target = one_hot.scatter_(1, labels.data, 1)
RuntimeError: arguments are located on different GPUs at /pytorch/aten/src/THC/generic/THCTensorScatterGather.cu:307

def make_one_hot(labels, classes):
    one_hot = torch.FloatTensor(labels.size()[0], classes, labels.size()[2], labels.size()[3]).zero_().cuda(device=labels.device)
    target = one_hot.scatter_(1, labels.data, 1)
    return target

fixed

Hi @kape67
Thanks for the correction.


feat: Add minimal function/trigger support in order to enable an audi…

…t feature.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

497 of 510 (97.45%)  changed or added relevant lines in 21 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.07%) to 96.709%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/sqlalchemy_declarative_extensions/trigger/base.py
38
42
90.48%

src/sqlalchemy_declarative_extensions/dialects/postgresql/trigger.py
75
84
89.29%

Totals

Change from base Build<PHONE_NUMBER>:
-0.07%

Covered Lines:
2170

Relevant Lines:
2237

💛  - Coveralls


Building Page URLs

They are not displaying, references only the /map main URL. We need the unique URLs to display when on a given page within the map.

We need to implement the HTML5/Javascript history API here
On Fri, Jul 10, 2015 at 11:35 AM, Chris Paddock<EMAIL_ADDRESS>wrote:

They are not displaying, references only the /map main URL. We need the
unique URLs to display when on a given page within the map.
—
Reply to this email directly or view it on GitHub
https://github.com/WheatonCollege/map/issues/47.


Please add a "Open with Visual studio code" button, in the context menu

right click on a directory or file
show a button in the context menu, that opens a directory or file with visual studio code

Visual studio already has such button, stating: "Open with Visual Studio"
It might be great if Visual studio code also got a button like that:

If you got questions, feel free to ask,
Regards
Dave

VS Code creates this entry, but you need to check corresponding options during installation


Support for T-Series possible?

Hi there,
I am using a Foxess T-13 inverter. Is it possible to add support for this inverter? My idea would be to connect Modbus via USB adapter. There is a repository https://github.com/assembly12/Foxess-T-series-ESPHome-Home-Assistant that provides support, but uses an ESP based solution which I would like to avoid.
Do you see any change realizing this? Is there is any possibility for me to contribute?
Regards,
Niko

Reading through the project you linked, it looks like the T-series works in a different way to the H-series.
The H-series expose a number of registers over modbus. Each register holds a different sort of value (PV1 voltage, etc), and we query individual registers using the modbus protocol.
The T-series doesn't seem to expose a modbus interface, but instead just transmits a set of values every few minutes.
It should be possible to use an off-the-shelf device RS485 to USB converter, or a device which receives packets over RS485 and re-transmits them over TCP (the W610 looks like it might be able to do this, but the documentation is rubbish), which would give you a route to avoiding ESPHome.
However as things stand, this integration isn't really set up to do that: it would need a fair amount of rework, and to be honest it would probably be easier to start again with a new integration. That's going to be much easier for someone who has a T-series inverter to develop and test against: doing it entirely removely will be a bit painful.
So, I'm afraid the straight answer is probably "no". But, with a bit of work it should be possible to use an off-the-shelf RS485 to USB converter, however you will need to invest some time into writing a new integration.


Feature Request: PsN Default Number of Threads

PsN by default has 5 threads for NONMEM execution.  While it's possible to change this on the fly, it's preferable to me to have it default to the number of licenses I've purchased.  For that, I'd propose the following change to the Dockerfile:
# in the preamble
ARG NUM_THREADS=4
# After running setup.pl
  && sed s/threads=5/threads=$NUM_THREADS/ /usr/local/share/perl/*/PsN_4*/psn.conf > psn.conf.tmp \
  && mv psn.conf.tmp /usr/local/share/perl/*/PsN_4*/psn.conf \

Isn't this something we could set as a ENV with a default value and let someone overwrite at runtime, rather than worrying about build time? or does it affect builds/testing?

no should not affect builds/testing - an env variable with a default of 4 would be most reasonable, as that would be relevant for a single license pack.

I set it as an ARG because users can input their number of licenses and then it becomes the default for all executions in their installation.  Is there a significant downside to an ARG with a default argument as opposed to an ENV?

The effect on build is related to the config file setting.  Because it affects many commands within PsN, I think that this is the simplest way to do set the default number of threads (though I'm happy to be proven wrong).

Anything that is "environment specific" such as running on one machine vs. another should be a ENV that's changeable at runtime. An ARG is only used at build time, which is fine if the value never needs to change during runtime. Sounds like we might need a docker-entrypoint.sh, which is the standard for how to change config files during container startup. We can have docker-compose.yml have the ENV in it, and document that anyone can change that value before startup, then docker-entrypoint.sh runs on startup and writes changes to config files.

OK so I've updated some things in master. Latest commits fix this issue I believe.

I've added some getting started info to README.md that may help understand this process.
The assumption is the standard user is using docker-compose commands for this and using our pre-built images, but once someone knows what they are doing they can use the alternate compose file and an override file to tweak things.
NUM_THREADS is now an ENV that writes to /root/psn.conf at container runtime. Just update the docker-compose.yml or create a docker-compose.override.yml to modify. No need to rebuild images.


correct python package workflow link

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (930e17f) 100.00% compared to head (8513328) 100.00%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##            master       #49   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            5         5
  Lines          121       121
=========================================
  Hits           121       121

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


clean design verification [dv] folder

I approve of this PR.  Thanks.
From: Mike Thompson @.>
Sent: Wednesday, December 21, 2022 10:25 AM
To: openhwgroup/cve2 @.>
Cc: Subscribed @.***>
Subject: Re: [openhwgroup/cve2] clean design verification [dv] folder (PR #26)
@MikeOpenHWGroup approved this pull request.
LGTM @christian-herber-nxphttps://github.com/christian-herber-nxp. Given that this PR deletes the entire dv directory, I would like to ensure that at least one member of the team from Intrinix also signs-off on this PR.
—
Reply to this email directly, view it on GitHubhttps://github.com/openhwgroup/cve2/pull/26#pullrequestreview-1226288766, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AIFRB447Z6MBZNUWEC6QFWTWOMOM7ANCNFSM6AAAAAATFQGPDU.
You are receiving this because you are subscribed to this thread.Message ID<EMAIL_ADDRESS>

Thanks @LeeHoff.


Enhancement of AzureDevopsProvider with new functionalities

User description
This PR enhances the AzureDevopsProvider class by implementing new methods for publishing code suggestions, labels, and removing comments. It also extends the AzureDevopsProvider with GitProvider and adds placeholder methods. The PR also includes some refactoring and reformatting of the existing code.

Type
Enhancement

Description

The AzureDevopsProvider class now inherits from GitProvider, and implements several new methods.
Added publish_code_suggestions method to publish code suggestions as comments on the PR.
Added remove_comment method to remove a specific comment.
Added publish_labels method to publish labels to the PR.
Added get_pr_labels method to retrieve labels from the PR.
Refactored and reformatted the existing code for better readability and maintainability.

Changes walkthrough
Relevant files                                                                                Enhancement

azuredevops_provider.py

        pr_agent/git_providers/azuredevops_provider.py
The AzureDevopsProvider class has been significantly enhanced with new functionalities. It now inherits from GitProvider and implements methods for publishing code suggestions, labels, and removing comments. The class also includes placeholder methods for future functionalities. The code has been refactored and reformatted for better readability and maintainability.

 +292/-90

/review

/improve --pr_code_suggestions.summerize=false

Preparing review...

Persistent review updated to latest commit https://github.com/Codium-ai/pr-agent/commit/b776e5069c148ed5699a5c4fc462945555851f25

/review

Preparing PR description...

PR Description updated to latest commit (https://github.com/Codium-ai/pr-agent/commit/b776e5069c148ed5699a5c4fc462945555851f25)

Preparing PR description...

PR Description updated to latest commit (https://github.com/Codium-ai/pr-agent/commit/b776e5069c148ed5699a5c4fc462945555851f25)


[Discussion] Should we change link from overlapping to concentrated

We changed the UI text from "overlapping" into "concentrated", but the code & URL didn't change.

Should we change the path in the URL ?

This would require a bunch of changes since values in the URL are used in the code as keyword, therefore we would have to add an abstraction around it, or change every reference of "overlapping" to "concentrated" in the code.

there is no requirement for this task.
only cleaning the code up.


Is there some limit in azure data explorer to difference between columns mapping and table schema?

I am experiencing some very strange behavior when ingesting data to kusto using dotnet client.
In one table, we decided that there were some columns that we don't want to store anymore.
So I removed them from the mapping, in my dotnet application (29 out of 94 columns), and suddenly the data stopped being stored in some other column for which I hadn't removed the mapping.
And to my surprise when testing, if I restore one column from mapping (only some work, not any of them) from the 29 removed, then the data into this other column appears to be stored again.
I also noticed that if I drop the same columns from the schema that I removed from mapping, then the data is stored correctly again.
So my question is, what is the rule for how much different the mapping can be from the schema?
(if needed I can provide sample code for reproducing the issue)
Thanks!

It would best if you could provide the table schema, the mapping and a sample data.
Also, please check if there are update policies on your table which might calculate the value of some columns.

Hi @yogilad
Thank you very much for taking time to reply, I added you and David to a private repo containing minimum code that allows to reproduce the issue.
I added table schema in comments inside the Program.cs file.
Any doubts please let me know !


feat(examples): Add custom host

custom email server used like mail jet so we quickly add on environment variable URL

Please review my code if you like my little bit contribution please add Thanks

Thanks @sharozraees802 🎉


Vood 78 data race when forwarding quickly

Actually, this doesn't work. Imagine 2 messages: 1.Message: increment this.calls and start processing 2.Message: increment this.calls and start processing 2.Message: finish processing and discard changes because this.calls === 2 2.Message: decrement this.calls 1.Message: finish processing and accept changes because this.calls === 1
=> incorrect data schown

I changed the handler on how to detect obsolete calls, which should solve this problem. The calls aren’t counted anymore. The event handler saves the sequence number of the last message and compares it with the sequence number of the original call before rendering the result.


sqlite3.OperationalError on Alpine Linux

Hi!
I just installed mtv_dl on an Alpine Linux VM.
I was able to do a couple of list commands, and then all of a sudden, this error message popped up:
2020-11-05T10:34:15  CRITICAL <class 'sqlite3.OperationalError'>: unable to open database file

                                File "/home/matti/.local/bin/mtv_dl", line 8, in <module>

                                  sys.exit(main())
                                File "/home/matti/.local/lib/python3.8/site-packages/mtv_dl.py", line 1202, in main
                                  showlist = Database(filmliste=cw_dir / FILMLISTE_DATABASE_FILE,
                                File "/home/matti/.local/lib/python3.8/site-packages/mtv_dl.py", line 414, in __init__
                                  self.connection = sqlite3.connect(filmliste_path.absolute().as_posix(),

Rebooting didn't change anything, also there is enough space on the drive.

Ugh, hard to tell what the exact reason for this is, but in general I would say the tool is unable to access the Filmliste database file.
The database file is located at "{current working directory}/.Filmliste.{script_version}.sqlite". This can be caused by permission issues but probably also by a corrupted file. I would recommend deleting the file. The mtv_dl tool will detect that it's missing and generate a new one. The Filmliste sqlite file is there for caching purposes only.
Did this help?

Well, it worked once: I deleted the Filmliste file, rebooted and it worked.
After another reboot, the Filmliste file was gone (thatsweird) and i get the same error message. (Thats weird, because it should be recreated as you said above).
I have a feeling there is something wrong with my VM image.

Probably, yes. Maybe it helps to put the working directory to some different place with --dir=PATH.
If I can help you to make it work from the mtv_dl side, please let me know - but it feels like it's coming from the system side.

I think i found something: The problem occurs after I create a .mtv_dl.yml file in my home directory with this content:
high: true
dir: /media/downloads
target: '{dir}/[{topic} {date}] {title}{ext}'

If i delete the file, it works again.
This happened on a Debian VM and my MacOS.
Should I open another bug ticket for this?

yeah, well, i feel stupid now. i didn't realize that "dir" is where the database file is stored. so if i change that in the config file, the script will be looking there. sorry to have bothered you. ;)

No problem. Glad that it works now.


Plugin Repository

I believe there is a need for a Hyperledger Aca-Py Plugins master repository.
Currently, there is no real documentation about how to (I believe work in this area is underway):

create a plugin
best practices for loading/using a plugin
build out a custom image with a set of plugins
how to distribute your plugin

BC Gov is working on a series of plugins for multi-tenant enhancements traction and need to make those more accessible for public use.
Ontario Gov is working on a webhook security enhancement and they are restricted from creating an open source repository for their plugin, but can contribute to existing open source projects/repos.
We (BC Gov) created a list of plugins we could find as we were investigating how to make plugins and what we could do the architecture. It is unknown what is active, what is working (if it ever did).
Having a hyper ledger repository should make it easier for the community to find and leverage plugins and would be a good place for how-to and other guides for leveraging the plugin architecture.

@swcurran I think having a central repo for the documentation @usingtechnology described and at least links to plugin repository makes sense. I would not recommend moving the entirety of the plugin to such a repo, though. We should have consistency as much as possible across the plugins but I don't think it would be beneficial to change the processes and workflows already established on the toolbox plugin.
Perhaps it would make sense to have a "centralized" repo that pulled in common groupings of plugins via submodule and built images from them. This could be a good way to get the benefits of having the plugin "in-repo" without moving it.

I think a centralized list of plug-ins is a good idea (could just be maintained in a doc in the aca-py repo) but I don't think we should have a centralized repo for the plug-ins themselves.

See https://github.com/hyperledger/aries-acapy-plugins

Repository is created... aries-acapy-plugins


[SCB-882] Split mixed it case

Coverage decreased (-0.05%) to 86.0% when pulling 3c986cbccfa198e8dcd94ac0cd0b932410ae92be on wujimin:split-mixed-it-case into 36f897fd9facaead112f57e360993b0878e5d013 on apache:master.


Sweep feature

The sweep feature is not working (yet). Selected 10 phunks to buy but the "Buy selected" button is not highlighted/active. Then tried just one using the feature, still no joy.

This is disabled until testing is complete.

This is disabled until testing is complete.

Ah ok, got it. Cheers


extract the getPosition/updatePosition method

just extracts the getTickSize-> getPositionInMarket->upsertPositionInMarket pattern

merging to avoid large conflict in block seq pr


Project Infrastructure

Improving documentation formatting and building scripts.

Nice, would the Dockerfile also need to be updated to reflect the new location of the binary ?
Step 15/17 : COPY --from=builder /go/src/github.com/redhat-developer/build-operator/out/operator /usr/local/bin/build-operator


Bug

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
1.9.2
Does this issue reproduce with the latest release?
I don't know.
What operating system and processor architecture are you using (go env)?
windows/amd64,
What did you do?
I was have a text file with size 6.79 GB. Then i read it with code :-

f, err := os.Open("filepath")
checkErr(err)
// get the file size
stat, err := f.Stat()
checkErr(err)
buf := make([]byte, stat.Size())
fmt.Println(stat.Size())
f.Read(buf)

Finally i need to write the buffer as a new file with this code :-

ioutil.WriteFile("temp.csv", []byte(buf), 0644)

What did you expect to see?
I expect the new written file will the same size as the file i was read which size is 6.79GB.
What did you see instead?
The new written file size was 2.79GB.

 I don't see error checks from your ioutil.WriteFile.
Oh, and you're also not using io.Reader correctly. See the docs on the https://golang.org/pkg/io/#Reader
For questions about Go, see https://golang.org/wiki/Questions.


niv-updater-action: Make sure it finishes in time

The action didn't pull niv from the cache and wasted many GitHub Actions
minutes building niv. We use our fork where the issue is fixed until
knl/niv-updater-action#46 is merged.
This also sets a timeout of 2mn.
Thanks to @nmattia for noticing and fixing.

This PR does not affect the produced WebAssembly code.


TypeError: Cannot read property 'startRecordingFilesWithChangedResolutions' of undefined at InferredProject.Project.updateGraph

TypeScript Version:
2.5.3-insiders.20170919
VSCode Version
1.17.0-insider
command requested
getApplicableRefactors
hit count
11993
stack
at InferredProject.Project.updateGraph in tsserver.js (line: 79253, column: 28)
at InferredProject.Project.getLanguageService in tsserver.js (line: 78994, column: 26)
at IOSession.Session.getApplicableRefactors in tsserver.js (line: 82616, column: 32)
at Session.handlers.ts.createMapFromTemplate._a.(anonymous function) in tsserver.js (line: 81586, column: 61)
at IOSession.Session.executeWithRequestId in tsserver.js (line: 82790, column: 28)
at IOSession.Session.executeCommand in tsserver.js (line: 82799, column: 33)
at IOSession.Session.onMessage in tsserver.js (line: 82819, column: 35)
at Interface.<anonymous> in tsserver.js (line: 83945, column: 27)
at emitOne in events.js (line: 96, column: 13)
at Interface.emit in events.js (line: 191, column: 7)
at Interface._onLine in readline.js (line: 241, column: 10)
at Interface._normalWrite in readline.js (line: 384, column: 12)
at Socket.ondata in readline.js (line: 101, column: 10)
at emitOne in events.js (line: 96, column: 13)
at Socket.emit in events.js (line: 191, column: 7)
at readableAddChunk in _stream_readable.js (line: 178, column: 18)
at Socket.Readable.push in _stream_readable.js (line: 136, column: 10)
at Pipe.onread in net.js (line: 560, column: 20)

resolutionCache is undefined only if project is closed. To investigate this further, it would be helpful to have tsserver log to get the information as to why closed inferred project is used to get the applicable refactors

These issues are crash reports, we will not get more information from these. so we need a speculative and defensive fix here. otherwise we will need to add additional asserts.

Duplicate of #18928


Scroll to Tab on Mobile with SetHash set to True

I think it would be a good idea to add in a default option where the page would scroll to the active Tab on mobile device.
If I open (or activate) a new accordion item that is below an already opened, long accordion item, the new active item will move up outside of the viewport. It would be nice to have the window scroll up with the accordion item.
Can this be done now with the tab-active callback? If so, can you explain how that can be done?
Thanks

This is actually an already existing option! You can enable the auto scrolling with the scrollToAccordion option: https://github.com/jellekralt/Responsive-Tabs#scroll-to-accordion-panel

That's great! I must have overlooked that option.


Allow override of k8s namespace during deployment/generation

🚀 Feature Description
As a user, I'd like to be able to supply the namespace during deployment or generation, in order to deploy somewhere other than the default namespace.
✔ Goals

[ ] create a CLI option for providing a namespace. If not supplied defaults to default.

PS. Apolgies if this is already available.  I wasn't able to find it in the CLI help or the docs...

Great idea
I'll add support this weekend 😄

Handled in release 0.1.20-preview thats just been pushed.
Can be seen in docs here https://prom3theu5.github.io/aspirational-manifests/generate-command.html
Its --namespace or the env var ASPIRATE_NAMESPACE when running the generate command


List boards per vendor

This PR proposes adding switches to 'west boards' to filter the board list on a board vendor or SoC vendor basis.
Two commits make up this PR:

code change to west's boards as well as to list_boards.py
changes to the yaml files for which NXP is the vendor.

The switches and yaml file fields proposed are optional.

Please also add https://github.com/zephyrproject-rtos/zephyr/tree/main/boards/arm/s32z270dc2_r52 :)

Please have a look at #50305


http://cdn.binaryjs.com/0/binary.js = 404

I'm having some throubles with this directory. How to preceed?

i found the answer here : https://github.com/binaryjs/binaryjs/issues/117


Ability to skip rest test cases if one fails

🚀 Feature Proposal
Provide ability to skip rest test cases in a file or describe block if one test fails.
Motivation
For E2E testing, the test cases are usually written to be run sequentially, and a failed test should stop the rest from running.
Example
test('should login', async () => {});

test('should edit profile', async () => {});

test('should render profile', async () => {});

Or with describes:
describe('#1', () => {
  test('should login', async () => {});

  describe('#1.1', () => {
    test('should edit profile', async () => {});

    test('should render profile', async () => {});
  });
});

keyword: step
prior art: https://github.com/rprieto/mocha-steps

Duplicate of https://github.com/facebook/jest/issues/6527 (for hooks and tests)

No it's not a duplicate. We are requesting mocha-steps not jasmine-fail-fast.
TL;DR: skip the following steps after the first fail within the current describe and not exiting after the first failure

No it's not a duplicate. We are requesting mocha-steps not jasmine-fail-fast.
TL;DR: skip (and auto fail) the following steps after the first failing step within the current describe (not exit on first failure)
@jeysal please reopen.

Okay then, we can keep this open to track fail-fast within a describe block (although the OP does not explicitly request this).
I think it's unlikely that this goes into Jest core though, while #6527 might as it's a logical extension of bail.

Personally I think this makes sense, and would love to see it in the jest core under step, as it lets you break tests into smaller parts (allowing you to describe them better, since they each have an it/step message), and increase performance for when you're testing something that is dependent on prior tests.
Take for example this test:
describe('.root', () => {
  it('contains the root package.json', () => {
    // eslint-disable-next-line @typescript-eslint/no-require-imports
    const requirePackageJson = () => require(path.join(pathTo.root, 'package.json'));

    expect(() => requirePackageJson()).not.toThrow();
    expect(requirePackageJson()).toStrictEqual(packageJson);
  });
});

It's checking that pathTo.root points to a directory with a package.json, and that the package.json is the actual root package.json (by comparing the object returned by the require to one imported via the normal module loader).
Since require throws an error if it can't find the file (the first expect), it makes no sense to test contents of the file (the second expect) if this happens.
But, since its are independent of each other, I either will have two tests failing (which while technically correct means I've got pointless noise that adds nothing to the situation), or one "big" it that combines two tests.
Meanwhile, if we had step:
describe('.root', () => {
  // eslint-disable-next-line @typescript-eslint/no-require-imports
  const requirePackageJson = () => require(path.join(pathTo.root, 'package.json'));

  step('has the root package.json', () => {
    expect(() => requirePackageJson()).not.toThrow();
  });
  step('contains the root package.json', () => {
    expect(requirePackageJson()).toStrictEqual(packageJson);
  });
});

I know that step is pretty self explanatory in its advantages, I felt like it could be valuable to put one to paper - I've got a lot of tests that are similar to that.
I don't mind step not being in the jest core, just so long it's as easy to use as it, describe, #each, etc are today.
I'm just really excited at the idea of having this feature in jest, as I've come across this problem a lot! 😄

Smoke tests in front of more substantial tests save people time. Without the ability to short circuit subsequent test, the time saving is mitigated.
Where does this functionality need to be added? Would this live in a test runner or is this a jest issue? (Many people don't even realize the distinction between jest and test runners...)

Here is my temporary solution.
describe("test", () => {
  let skipOther = false
  test("check something", async () => {
    try {
      await checkSomething()
    } catch(e) {
      skipOther = true
      throw new Error("error")
    }
  })

  test("might be skipped", async () => {
    if(skipOther) throw new Error("error occured in previous tests.")
    console.log("display this, if there is no error before this.")
  })
})

I would be interested in implementing a test.step function in jest circus, I don't see a way to do this outside of the jest-circus core though, as it requires adding some stuff to the state.

I created a PR that implements asynchronously canceling tests as described in #8604. With that in place, one could implement this functionality through easy monkey patching in user land like this:
let prevFailed;
const _describe = global.describe;
global.describe = (...args) => {
	prevFailed = false;
	return _describe(...args);
};
global.test.step = (name, fn) => {
	test(name, async function() {
		if (prevFailed) {
			return this.skip();
		}
		try {
			return await fn();
		} catch (e) {
			prevFailed = true;
			throw e;
		}
	});
};

@MrLoh where's the PR? I only see the issue

#9944 But I doubt it will ever be merged, I haven't heard back a single word from any contributor.

My solution https://github.com/facebook/jest/issues/6527#issuecomment-760092817

My solution https://github.com/facebook/jest/issues/6527#issuecomment-760092817

Here is my solution #7245 to skip test programmatically and on run time after test began. the key is to call pending()

Playwright already provides a test.step https://playwright.dev/docs/api/class-test/#test-step
And this wouldn't also make sense for jest specially for e2e tests.

Here is the solution to make a Jest Describe Block as Sequence of Steps
https://gist.github.com/Raaghu/ca8323a414ea0041f3704559fe6d04e4
Features

All previous tests are run before current test (even in case of only one test is selected to run)
future tests are skipped if current test fails

.

.


Add support for dependency conventions

Adds support for dependency conventions using the convention object approach.
Reviewing cheatsheet
Before merging the PR, comments starting with

❌ ❓must be fixed
🤔 💅 should be fixed
💭 may be fixed
🎉 celebrate happy things

@bot-gradle test rFM

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle test QuickFeedback

I've triggered the following builds for you. Click here to see all build failures.

QuickFeedback build

@bot-gradle test QuickFeedback

I've triggered the following builds for you. Click here to see all build failures.

QuickFeedback build


Add a test that placeholder are preserved in translations

What

Add a test that placeholder are preserved:
"share_product_text": "Have a look at this product on Open Food Facts: {url}",
"share_product_text": "Open Food Facts'teki bu ürüne bir göz atın",
https://github.com/openfoodfacts/smooth-app/pull/2384
https://github.com/openfoodfacts/smooth-app/pull/2389/commits/2bb16c82b5d1c8286977fe6b6beaa9963d2884d7

Bumping to P1, since I loose time everyday on this

Looks similar to #2406 and #2407: @lsaudon, do you feel like fixing that one too?

I feel like it's not the same thing.
If I understood, you have to make a test that verifies that the {stuff} does not disappear from the translations?

Hi @lsaudon!
Actually I imagined that for #2406 and #2407 you used a magic trick that identified dead code (useless files and useless parameters). I assumed that the magic trick was a parameter in lint check, was it?
And I assumed that there was something similar regarding translations, in that case checking that for each label, all the translations have the same parameters.
For instance, in a translation file (in .dart_tool/flutter_gen/gen_l10n/app_localizations_??.dart) you'll see something like that:
  @override
  String share_product_text(String url) {
    return 'Have a look at this product on Open Food Facts: $url';
  }

And perhaps there's a way to detect: there's a parameter (url) in the parent method, but it is not used in that implementation (it would be the case with return 'Have a look at this product on Open Food Facts!';).

Working on it...


Upload example

Hi,
Could you create additional example how to use react-dropzone to upload file?

I am also curious on how to get it done

It's the same as you would do with file form field, but instead of getting the 'files' property from the field, you listen to the onDrop callback to handle the files. Simple explantion here: http://abandon.ie/notebook/simple-file-uploads-using-jquery-ajax
The onDrop provides you with an array of Files which you can then send to a server. For example, with SuperAgent as a http/ajax library:
javascript
onDrop: function(files){
var req = request.post('/upload');
files.forEach((file)=> {
req.attach(file.name, file);
});
req.end(callback);
}

Or if you use a Flux architecture like me: for every file in the onDrop trigger an action, which will then call a function in your api-utils, which will then do the request like above, which will eventually trigger another action when complete or erroneous.

Closing this one. Will accommodate @nextminds note in the README so everyone can read.

Hey there, I'm using superagent and following the steps @nextminds outlined, but I'm having trouble getting file upload to work (in my case it's a .xlsx file).
Has anyone else had success with this? Are there any tips to follow. I'm using express and the req.body is empty on the backend.