[Bug] Crafting Inventory

edit:
I just read this (https://forums.spongepowered.org/t/tools-actioncontrol-v1-1-control-crafting-mining-combat-and-more/10372/76) post, so I assume this will be fixed as soon as the Sponge team updates their inventory API
Assume you set any rule that disables crafting of a specific item. Now if you put this item in a crafting menu, you can place it there, but you won't be able to put it back in your inventory.
Example:
player-filter = "!{'permissions': {'item.craft': true}}",
action-rules {

    craft {
        filter {
            result = "{'type': 'minecraft.planks'}"
        },
        response {
            match = [
                "deny"
            ]
        }
    }
}

There is also some weird behaviour, when I use regex.
player-filter = "!{'permissions': {'item.craft': true}}",
action-rules {

    craft {
        filter {
            result = "{'type': r'minecraft.*'}"
        },
        response {
            match = [
                "deny"
            ]
        }
    }
}

This example has exactly the same behaviour as the upper one, but applies to all minecraft vanilla items.
player-filter = "!{'permissions': {'item.craft': true}}",
action-rules {

    craft {
        filter {
            result = "{'type': r'.*'}"
        },
        response {
            match = [
                "deny"
            ]
        }
    }
}

This example should deny every crafting recipe, including mods. It has a different behaviour, because now you cannot even put items in a crafting menu. Also if you try to put them back in your inventory after trying to put them in a crafting menu, the first time it won't work, but every subsequent try will work

I will look into this.

Yeah all of these issues are caused by using the incomplete inventory api and should be fixed in the next ActionControl update.


Fix duplicate initializers appearing in dist js file

(Note: This error could only be seen when using a project using https://github.com/ronco/ember-cli-meta-tags. If just running ember-cli-meta-tags, this error doesn't appear.)
The error:
When running ember s on our project (which we'll call project-name), we would get this console error: "Assertion Failed: The instance initializer 'head-browser' has already been registered". (https://puu.sh/w7HJm/fd018e7dc1.png)
On investigation, in dist/assets/project-name.js, there were two initializers when there should only have been one, which is what was causing this error to appear.
The first one:
define('project-name/instance-initializers/head', ['exports', 'ember', 'bidvine-web/config/environment'], function (exports, _ember, _environment) {
  'use strict';

  Object.defineProperty(exports, "__esModule", {
    value: true
  });
  exports.initialize = undefined;
  function _initialize(owner) {
    if (_environment.default['ember-cli-head'] && _environment.default['ember-cli-head']['suppressBrowserRender']) {
      return true;
    }

    // clear fast booted head (if any)
    _ember.default.$('meta[name="ember-cli-head-start"]').nextUntil('meta[name="ember-cli-head-end"] ~').addBack().remove();

    var component = owner.lookup('component:head-layout');
    component.appendTo(document.head);
  }

  exports.initialize = _initialize;
  exports.default = {
    name: 'head-browser',
    initialize: function initialize() {
      if (typeof FastBoot === 'undefined') {
        _initialize.apply(undefined, arguments);
      }
    }
  };
});

The second one:
define('project-name/instance-initializers/browser/head', ['exports', 'ember', 'bidvine-web/config/environment'], function (exports, _ember, _environment) {
  'use strict';

  Object.defineProperty(exports, "__esModule", {
    value: true
  });
  exports.initialize = initialize;
  function initialize(owner) {
    if (_environment.default['ember-cli-head'] && _environment.default['ember-cli-head']['suppressBrowserRender']) {
      return true;
    }

    // clear fast booted head (if any)
    _ember.default.$('meta[name="ember-cli-head-start"]').nextUntil('meta[name="ember-cli-head-end"] ~').addBack().remove();

    var component = owner.lookup('component:head-layout');
    component.appendTo(document.head);
  }

  exports.default = {
    name: 'head-browser',
    initialize: initialize
  };
});

My guess of what the problem is
Between 0.2.0 and 0.2.1, the non-fastboot initializers were moved from {{appFolderName}}/instance-initializers/browser/head.js to {{appFolderName]}/instance-initializers/head.js, while the fastboot initializers remained in {{appFolderName}}/instance-initializers/fastboot/head.js.
In ember-cli-head/app/index.js, broccoli-merge-trees is used to select the path to be used for the initializer. I think the different folder depth level of the fastboot vs non-fastboot initializers is causing broccoli-merge-trees to not merge the paths correctly, causing two initializers to be in the final distributed file.
The fix
Moved the non-fastboot initializers back to their original folders, in {{appFolderName}}/instance-initializers/browser/head.js. Regardless of what the precise cause is, I tested this solution and it does indeed fix the problem.

/cc @simonihmig @kratiahuja

There doesn't exists app/instance-initializer/browser/head.js any longer so how is that getting built for you?
Can you put in a minimal reproduction with only ember-cli-head in here? Also what version of ember are you using?

The removal of the initializers within a browser folder is due to FastBoot 1.0 not supporting this anymore. And you can see this in the failing tests of your PR, the e2e tests for FastBoot 1.0 are failing.
What is puzzling me is that the code sample you posted for the second initializer does not even exist in this repo in the current master branch (and so in release 0.2.1). Search for "head-browser" (https://github.com/ronco/ember-cli-head/search?utf8=✓&q=head-browser&type=), and you see that there are two initializers, however both have a FastBoot guard around their initialize function, while you second code sample does not have it. Looks like an older implementation (prior to #21). I have no idea where that comes from!?
I don't think there can be two addons of the same name (different versions) active, right? Did your app maybe try to override instance-initializers/browser/head.js? Other than that, my last resort would be rm -rf node_modules tmp and reinstall/rerun... 🤔

@slannigan can you create a minimal reproduction with a dummy app? This will help us nail down the issue to come up with the correct solution.

The issue is we were on v3.1.6 of https://github.com/tim-evans/ember-page-title which was tied to 0.2.0 of ember-cli-head. With https://github.com/tim-evans/ember-page-title/pull/70 (v3.2.0), we no longer have this issue.

To be clear, I assume it was the conflict of having both ember-page-title and ember-cli-meta-tags in our repo with them pointing at 2 different versions of ember-cli-head which was causing the issue.


Test test_dist_table_scan failed

Introduced in this PR #492
This job contains more information.
Some error messages
2022-11-15T04:02:42.657639Z  WARN meta_srv::service::heartbeat: Quit because it is no longer the leader
2022-11-15T04:02:42.657788Z  INFO meta_srv::service::heartbeat: Heartbeat stream broken: "unknow"
2022-11-15T04:02:42.657971Z  INFO meta_client::client::heartbeat: Success to create heartbeat stream to server: HeartbeatResponse {
    header: Some(
        ResponseHeader {
            protocol_version: 1,
            cluster_id: 1000,
            error: Some(
                Error {
                    code: 2,
                    err_msg: "Current server is not leader",
                },
            ),
        },
    ),
    payload: [],
}
2022-11-15T04:02:42.658338Z  INFO datanode::heartbeat: Heartbeat handling loop exit.

Closed via https://github.com/GreptimeTeam/greptimedb/pull/510


feat(primitives): SVG parser/renderer

This adds a new SVG project that implements parser and renderer for a decent part of the SVG spec.  Not all of them work, and some have been duplicated and tweaked a bit to work around missing features, but these are good test cases for future development.
Here's a selective list of features that have been implemented, and that IMO would be good additions to have, for anyone wanting to contribute:

[ ] Geometry

[x] <circle>
[x] <ellipse>
[x] <line>
[x] <path>

[x] M (Move To, absolute)
[x] m (Move To, relative)
[x] L (Line To, absolute)
[x] l (Line To, relative)
[x] H (Horizontal Line, absolute)
[x] h (Horizontal Line, relative)
[x] V (Vertical Line, absolute)
[x] v (Vertical Line, relative)
[x] C (Cubic Bezier Curve, absolute)
[x] c (Cubic Bezier Curve, relative)
[x] S (Cubic Bezier Curve, implied start control point, absolute)

[x] Following C
[ ] Following c
[x] Following S
[ ] Following s

[x] s (Cubic Bezier Curve, implied start control point, relative)

[ ] Following C
[x] Following c
[ ] Following S
[x] Following s

[x] Q (Quadratic Bezier Curve, absolute)
[x] q (Quadratic Bezier Curve, relative)
[x] T (Quadratic Bezier Curve, implied control point, absolute)

[x] Following Q
[ ] Following q
[ ] Following T
[ ] Following t

[ ] t (Quadratic Bezier Curve, implied control point, relative)

[ ] Following Q
[ ] Following q
[ ] Following T
[ ] Following t

[x] A (Elliptical Arc Curve, absolute)
[x] a (Elliptical Arc Curve, relative)
[x] Z, z (Close Path)
[x] argument chaining/implied commands

[x] <polygon>
[x] <polyline>
[x] <rect>

[x] rounded corners

[ ] Text
[x] <g>
[ ] <defs>
[ ] <use>
[x] Paint

[x] Color

[x] Hex
[x] Named colors
[ ] functions (rgb(...))
[ ] currentColor

[ ] funciri (i.e. references)
[ ] inherit
[ ] gradients

[ ] Presentation attributes

[x] fill
[x] stroke
[x] stroke-width
[ ] stroke-*
[ ] fill-*
[ ] opacity
[ ] ...

[ ] Style
[ ] Units

[x] User coordinates (i.e. unitless)
[ ] em
[ ] ex
[ ] px
[ ] in
[ ] cm
[ ] mm
[ ] pt
[ ] pc
[ ] %
[ ] auto (element-specific behaviour, partly implemented)

[ ] Transforms

[x] viewBox
[ ] preserveAspectRatio
[ ] rotate
[ ] translate
[ ] scale
[ ] skew
[ ] matrix

Could someone approve this so I can get it merged to keep it from going stale again? It doesn't really interfere with much else, so it's very low risk.

The TODO list isn't completed ? DO you want I review it ?

It's not really a TODO list, but rather a list of what's been implemented and what's not. If you'd like to review it that'd be great!


Rdkit upgrade

Updated pip README to include m chip
Currently dc 2.7.1 will install rdkit 2023.9.6. Pinned to use rdkit 2024.03.3.

Hi, it needs to be 2023.03.03. There's no good test to see if it fails, but the warning that pops up here needs to be gone:
https://github.com/rdkit/rdkit/issues/7364

Changed the pip to rdkit 2023.3.3.
Verified with test provided in rdkit/rdkit#7364 that no warnings appear.
Ran ampl unit tests. No errors found.


Disable uppercase limitation for Logger names and so on except for LoggerConfig file & folder names

This is solved now because we don't use .json file based storge any more.


Help in parsing this xml format

Could you please help me in parsing this format.
I am not able to correctly parse this kind of format: I want to get the 'text' attributes value of all prop given 'child' index value.
For ex: get all prop's 'text' value given 'child' index 1
root
child index="1" name="child name 1"
prop index="1" text="prop name1" /
prop index="2" text="prop name2" /
/child
child index="2" name="child name 2"
prop index="1" text="prop name1" /
prop index="2" text="prop name2" /
/child
/root

Hey @imran20487, I noticed you closed this issue - were you able to figure this out?

@drmohundro Yes, I was able to figure it out. First it was confusing for me as I was new in this xml thing. Now it is super easy to use with your library. Thanks.
Here is how I did it:
let xml = SWXMLHash.parse(data)
let childs = xml["root"]["child"].withAttr("index", "(childID)")
let elem = childs["prop"][propID]
propName = ""
if let attr = elem.element?.attributes
{
propName = attr["text"]!
}

Great, happy you were able to get it working!


patch for #422

fixed issue new row skew 1 td left

corrected PR - This PR is a fix for issue #422, this PR also adds support for item numbering in paging, and also support for checkbox type render (like in editor)

@NordRagnarok could you merge that, please?

@ricardodantas  I don't have merge permissions on this branch..
You can take these changes from my fork

Any news about this?


Loading

This updates the loading section per https://github.com/sveltejs/learn.svelte.dev/pull/105/files#r1038350603.
Couple of things to note:

it uses the /// file: some-file-name.js convention from the Kit docs, but there's no CSS in place for that just yet, so it looks a bit funky
the event.fetch(...) call in the Hacker News example doesn't work, because the response is truncated. I can only assume this is a problem with WebContainers somehow, but I haven't investigated yet
We need a way to rename/delete files. Also, up till now app-b was assumed to be additive, but in reality there are times when app-a contains files that shouldn't exist when the exercise is solved

I think it's probably worth merging the content into tutorial-routing and worrying about the functionality later

Deleting files is already possible. It's not that pretty, but if you create a folder with the file __delete in it, it will mark the path above it as " will be removed". So if you want to delete a file you would do src/routes/+page.svelte/__delete . You can see it 'in action' in the advanced SvelteKit tutorial

What do you think about requiring app-b to be complete, and expecting any files that are in app-a but not app-b to have been deleted? Do we think that would create more work or less work in the long run?

I see that you added notes on $types - frankly, I would completely remove those references, and worry about that later. Right now, they don't work at all in the context of the tutorial, and it will inevitably lead to "but where's the TypeScript version" questions.


NER plugin Refactor and example

Refactored NER plugin
Added GermEval based NER example and notebook with placeholders

Codecov Report

Merging #32 (a1565d9) into main (43dc838) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #32   +/-   ##
=======================================
  Coverage   73.10%   73.10%
=======================================
  Files          20       20
  Lines        1372     1372
=======================================
  Hits         1003     1003
  Misses        369      369

Flag
Coverage Δ

unittests
73.10% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

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


Fixing broken Docker URL

Super simple PR, just fixing the broken link for docker hub :)
Signed-off-by: Christopher P. Maher<EMAIL_ADDRESS>

Hey @krnowak ! Thanks for the suggestions. I have updated those two files and added them to this PR. Good catch! :)


Calendar scatter with problems in some months

One-line summary
When using series type as scatter and coordinateSystem as calendar some months have the days out of position and the month as his line out of place too. In this cases the calendar and day labels also change positions for some weird reason.
Version & Environment

ECharts version: 4.0.4
Browser version: Firefox Quantum 59.0.2
OS Version: Windows 10

Expected behaviour:
I would expect the same behaviour as in the other months. Check the screenshots and ECharts option section in this issue!
ECharts option
Access to this page and in the function getVirtulData change the following variables:
var date = +echarts.number.parseDate('2018-10-01');
var end = +echarts.number.parseDate('2018-11-01');

Also change the range property in option object to '2018-10'.
Other comments
All the screenshots have been taken width the same window size and position to make comparison easier!
Working month (2018-09): (calendar in the middle)

Non-working month (2018-10): (calendar in the bottom for some reason)

+1


Search in home page could search full database

Search box in home page could search full database. Right now it searches into Tasks. As a user, I am prompted to check if a new submission I'd like to add is present in the database from the homepage.

We've added a site-wide "search" bar! (It's really rather a "filter," but it combines all taxonomy categories.)


How to use https ?

How to use https？

You can just pass it into the connection object as the URL (https://github.com/mrtazz/restclient-cpp/blob/master/test/test_connection.cc#L24) or via the static methods directly (https://github.com/mrtazz/restclient-cpp/blob/master/test/test_restclient.cc#L89)


Initialize on startup of shell

Is your feature request related to a problem? Please describe.
I want to open a terminal and have the inshellisense already running. Currently if I add is to my bashrc, I can exit from the terminal twice, because is generates a new shell.
Describe the solution you'd like
A command to silently start in the same shell, the ishellisense.
Describe alternatives you've considered
I've tried adding is to the  ~/.zshrc, but it doesn't work in an optimal way.
Additional context
Add any other context or screenshots about the feature request here.

for what it's worth, you can achieve this with a custom profile in your terminal:

I already tried that. It generates a session within a session of the terminal. You can exit twice.

guess it depends on the terminal emulator. mine closes on the first exit.

@naquiroz Would you be interested in contributing the feature?
I think we could add something like a flag --parent-term-exit that will update the exit code to terminate the parent ppid which is either another shell or worst case the terminal emulator of the shell.

Would something like this work?
  term.onExit(({ exitCode }) => {
    if (process.ppid) {
      process.kill(process.ppid);
    }
    process.exit(exitCode);
  });

Would something like this work?
  term.onExit(({ exitCode }) => {
    if (process.ppid) {
      process.kill(process.ppid);
    }
    process.exit(exitCode);
  });

This is looking great so far! Why don't we check for a commander flag instead of if the ppid exists (since for our use case we know it will always exist), because we want users to be able to opt in for this.
You could add the option to the type here and add the flag itself here like we for the other options

Done! (I think)

Is it release automatically?

No I have to create a manual release. I'll most likely be Monday with a fix for #205


exp/services/soroban-rpc: Use Rust to obtain preflights

Instead of using Core (through its now deprecated /preflight HTTP endpoint) to obtain preflights, do it locally through the soroban-env-host library.
This is done through a small C-ABI layer communicating Go (using CGO) with Rust's FFI. The Rust code is compiled to a static library (libpreflight.a). The compilation is done through go generate
Unfortunately it doesn't still work since we need a way to supply soroban-env-host with:

The raw Ledger entries it requests.
Ledger information.

(all of the above marked with TODOs in the code)
~Additionally we need to decide whether we want to try to link the resulting binary statically. See https://github.com/2opremio/soroban-go-rust-preflight-poc/issues/3 for more details.~
I decided to link dynamically against glibc in Linux for now, since the musl targets (which allow linking truly statically) don't seem to allow unwinding Rust panics (See https://github.com/stellar/go/actions/runs/3484488730/jobs/5829099175 , in which you can see the rust call causing an ABORT signal)

do you think this should target the soroban-rpc source that has been relocated to soroban-tools:
https://github.com/stellar/soroban-tools/tree/main/cmd/soroban-rpc
the exp/services/soroban-rpc path will be removed, possibly during this sprint, need to confirm with @tsachiherman on timing though :
https://github.com/stellar/go/issues/4679

I can definitely move the PR there, thanks

@sreuland I can't move the PR just yet, the code is not in sync. See https://github.com/stellar/soroban-tools/pull/251#issuecomment-1324125295

https://github.com/stellar/go/pull/4698 (if successful) should be able to fulfill the remaining requirements of this PR.

I've migrated this PR to https://github.com/stellar/soroban-tools/pull/269 .
This PR is not going to be updated further but I haven't closed it yet because some of the changes aren't migratable:

Preflight method removal in the core client
Test adjustments

I will apply those once we remove RPC-server from this repo
CC @tsachiherman

we won't be using this one anymore, right ?

No, but I left it open because there is still work to do as mentioned above. That's why I left it open.

No, but I left it open because there is still work to do as mentioned above. That's why I left it open.

Yes, I missed these. Thanks!

Yes!


Create NamedPlanPerforming and NamedPlan protocol

NamedPlan protocol must be conformed to by plans that support being named.
Spec: https://material-motion.gitbooks.io/material-motion-starmap/content/specifications/runtime/named-plans.html

Closed by 0d6f6d682f30f0abd59fbfcafc1bde29773c7879.


IoT Sample

It shows compile error as following.
AppDelegate.swift
let configuration = AWSServiceConfiguration(
region: AwsRegion,
credentialsProvider: credentialProvider)
-> Compile error -> Cannot convert value of type '(regionType:AWSRegion, identityPooldId: String)' to expected argument type 'AWSCredentialProvider'

Hi @paultwin0202 ,
The region is supposed to be the IoT Region in your AWS IoT console for example AWSRegionType.USWest2. The credentialsProvider is supposed to be the pool ID of your cognito for example "us-west-2:xxxxxxx-xxxx-xxxx-xx-xxx"
Thanks,
Fengyi

Please re-open if the issue still exists


Please add change IV for HP and defense

Really love this editor! just a little request to make the change IV function also be able to change defense and HP IV as well? Thank you very much

Second this! I've looked into it and it seems like HP/Defense are IV's (Not sure if they're independent of each other, but I imagine they are?) Would be great to add a function that increases all IV's to MAX. I'm sure ATK/Defense is 100/100 but is HP a value or percentage?

Defence has been added. I need to look at HP because although "Talent_HP" is a thing, there are also two other HP variables and I need to figure out how to change all 3 of those without breaking the Pal.

Defence has been added. I need to look at HP because although "Talent_HP" is a thing, there are also two other HP variables and I need to figure out how to change all 3 of those without breaking the Pal.

Awesome! And ah three health variables, much more complicated than I imagined haha. Is there also any plans on being able to increase the stats of the Pals the same way it's increased by the statue of power? (The ones that require Pal souls to do).

Statue of Power feature is coming in a future update


Enable pairing for multiple devices

Problem
CHIP Controller API doesn't support pairing of multiple devices. The controller class stores information of one paired device.
Summary of Changes

Refactor Controller API to squeeze out device specific information to CHIPDevice class.
Split the API into Controller and Commissioner interfaces
Rewrite old controller class using new APIs, to keep backward compatibility for time being (until all controller apps move to the new interface).

Fixes #2795

rebased

rebased

rebased


Decouple Claiming a Square and Changing it's colour

Currently: One has to claim a square and change its colour at the same time with the added option to edit the colour later.
Instead: I suggest we start with the claim exclusively (i.e. keep the name and keep the colour), the user then has the same options of managing their owned squares

Currently: One has to claim a square and change its colour at the same time with the added option to edit the colour later.
Instead: I suggest we start with the claim exclusively (i.e. keep the name and keep the colour), the user then has the same options of managing their owned squares

We could also have that square change to the selected state on the canvas once claimed so its easier to change colour and name as soon as it it claimed

Agreed, this decoupling makes sense. To clarify, do you mean that when you claim a square, you don't choose a colour, and that colour changes always happen in the squareManager part of the app?

Yes, I think the user shouldn't need to change colour (or even enter the hex of the current colour) in order to claim that square.
Colour changes could happen in 2 locations within the app:

Within the squareManager
Within the sidePanel IFF that particular square which is highlighted is owned by the user


Upstream commits for parent/child surface id

This patch change should fail to apply once we get the upstream commits

++

@bridiver https://crrev.com/5d086e08de60 has been cherry-picked in 65.0.3325.162 (https://crrev.com/7353ab9ab66a)


filesystem support for solarish.

close #3890

unfortunately, libc crate does not contain it.

In that case please add a comment so that we can remember to add it in the future.

unfortunately, libc crate does not contain it.
In that case please add a comment so that we can remember to add it in the future.

This comment is still missing

:umbrella: The latest upstream changes (presumably #4016) made this pull request unmergeable. Please resolve the merge conflicts.

broader question would be should we wait the libc crate update or do we keep the missing field as TODO ?

TODO is fine for now.

@rustbot ready

Thanks :)


Add flexible BGO suppression to TTigress

I've (re-)added reasonably flexible BGO suppression to the TTigress class. This required the recreation of the TBgoHits and their storage within TTigress (rather than TTigressHits) to allow for suppression from BGOs belonging to neighbouring detectors.
Suppression is now done on the fly during GetAddbackMultiplicity(), and is evaluated much in the same way as addback. I've also included a fairly basic suppression as a default. I've tested it with data and it does a good job without oversuppressing good events (I see no loss of events for a 1564-keV gamma-ray in the data I tested).
Suppression is considered on by default, but can be toggled using the SetDoSuppression function, which also resets the addback.
Additionally I fixed a bug in TS3 where the fitted times weren't being copied over.

This is good stuff, i never liked the bgo in the hits. However, this won’t work with my new TChannel builder, so Please do not merge this. I will merge this onto my computer and try to get it working with the new builder before we merge it into the main code.
On Aug 3, 2016, at 1:39 PM, jhenderson88 <notifications<EMAIL_ADDRESS>wrote:
I've (re-)added reasonably flexible BGO suppression to the TTigress class. This required the recreation of the TBgoHits and their storage within TTigress (rather than TTigressHits) to allow for suppression from BGOs belonging to neighbouring detectors.
Suppression is now done on the fly during GetAddbackMultiplicity(), and is evaluated much in the same way as addback. I've also included a fairly basic suppression as a default. I've tested it with data and it does a good job without oversuppressing good events (I see no loss of events for a 1564-keV gamma-ray in the data I tested).
Suppression is considered on by default, but can be toggled using the SetDoSuppression function, which also resets the addback.
Additionally I fixed a bug in TS3 where the fitted times weren't being copied over.

You can view, comment on, or merge this pull request online at:
https://github.com/GRIFFINCollaboration/GRSISort/pull/736
Commit Summary

Add flexible BGO suppression to TTigress

File Changes

A include/TBgoHit.hhttps://github.com/GRIFFINCollaboration/GRSISort/pull/736/files#diff-0 (48)
M include/TTigress.hhttps://github.com/GRIFFINCollaboration/GRSISort/pull/736/files#diff-1 (17)
M include/TTigressHit.hhttps://github.com/GRIFFINCollaboration/GRSISort/pull/736/files#diff-2 (6)
M libraries/TGRSIAnalysis/TS3/TS3.cxxhttps://github.com/GRIFFINCollaboration/GRSISort/pull/736/files#diff-3 (2)
M libraries/TGRSIAnalysis/TS3/TS3Hit.cxxhttps://github.com/GRIFFINCollaboration/GRSISort/pull/736/files#diff-4 (2)
M libraries/TGRSIAnalysis/TTigress/LinkDef.hhttps://github.com/GRIFFINCollaboration/GRSISort/pull/736/files#diff-5 (1)
A libraries/TGRSIAnalysis/TTigress/TBgoHit.cxxhttps://github.com/GRIFFINCollaboration/GRSISort/pull/736/files#diff-6 (47)
M libraries/TGRSIAnalysis/TTigress/TTigress.cxxhttps://github.com/GRIFFINCollaboration/GRSISort/pull/736/files#diff-7 (104)
M libraries/TGRSIAnalysis/TTigress/TTigressHit.cxxhttps://github.com/GRIFFINCollaboration/GRSISort/pull/736/files#diff-8 (2)

Patch Links:

https://github.com/GRIFFINCollaboration/GRSISort/pull/736.patch
https://github.com/GRIFFINCollaboration/GRSISort/pull/736.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/GRIFFINCollaboration/GRSISort/pull/736, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AFLTwJH69LZzlScOdRqio_RdOL2P_uOmks5qcNJugaJpZM4Jb4iU.


Expose column spec information for the _iter interface

Related to #271.
In #276, information about column names became available for the results of Session::query and Session::execute through the QueryResult::col_specs field.
We should also expose this information through the iterator object returned by Session::query_iter and Session::execute_iter methods.

I implemented it when adding query_all to Session, but eventually these changes were dropped.
There is a commit on this branch: https://github.com/cvybhu/scylla-rust-driver/commits/queryall-backup:
https://github.com/cvybhu/scylla-rust-driver/commit/6bf71829821a1969a8c19f05b43eadba17242040

Fixed by b46f913


cue: proto support

Originally opened by @mpvl in https://github.com/cuelang/cue/issues/5
Tracking issue for adding protobuf support to CUE:

import command reads text and binary proto instance
import command reads proto definitions
export command supports the converse
core libs

Original reply by @mpvl in cuelang/cue#5 (comment)
It is not implemented so far. Should be straightforward, but stabilizing the new evaluator has priority.

Hello, is there any update/estimate on the timelines for this request?

With regards to Protobuf support, the documentation has been improved since 2021, but still does not make it clear that Cue can't generate protobuf files form Cue.  Cue can import .proto to .cue, but cannot go the other way.
https://cuelang.org/docs/concept/how-cue-works-with-protocol-buffers/
References:
https://github.com/cue-lang/cuelang.org/pull/371
https://github.com/cue-lang/cue/issues/581
https://github.com/cue-lang/cue/discussions/983
Instead of cue, we have settled for the protobuf validation via:
https://github.com/bufbuild/protovalidate


Tutorials for scIB

Hi there,
I think that having tutorials for the different metrics would be very cool.

Yes :). Maybe we can make this a milestone and do project management via github here.


Update keydown function that opens the search field

…Default is only applied when command keys are also pressed at the same time
Before submitting the PR:

[ ] Does your PR reference an issue? If not, please chat to the team on Discord or GitHub before submission.
[X] Did you update and run tests before submission using npm run test?
[X] Does your branch follow our naming convention? If not, please amend the branch name using branch -m new-branch-name
[ ] Did you update documentation related to your new feature or changes?

What does your PR address?
This issue was mentioned in Discord. When opening the search bar, it wasn't possible to type "k" into the search field since the keydown function was preventing defaults of it.
I updated the keydown function to use && instead of || and also changed e.code === 'KeyK to e.key === 'k', since Ctrl + Shift + k is a shortcut in Firefox to open the web console, but this was prevented here. On the SvelteKit site Ctrl + Shift + k also doesn't open the search field: https://kit.svelte.dev/, so perhaps they are doing it this way as well?
I don't have Mac so I can't check if these changes also work on there, so I'll create it as a draft PR for now.
Tips

Tap "convert to draft" to indicate this is work in progress.
Link to an issue using the verbiage Fixes #XX
Linked issues will auto-close when the PR is merged.

Thanks @bennymi. FYI I checked the failed preview build and it looks to be related to the Vite 4 peer dependency. Most likely due to the SvelteKit v1.0 launch stuff happening today!
I'll review this later this afternoon though, after the announcements. Thanks!

@bennymi was able to squeeze in a quick review. Much simpler logic and seems to be working better. Tried it in all Mac browsers without issue. I'll go ahead and merge. Thanks!


Review and enhance logging support for email service (Python / Ruby)

As OpenTelemetry Logging has not fully stabilized, this list of requirements may change in future releases.

[ ] Log events in existing services should be converted to span events where appropriate (i.e., the event occurs within a transaction)
[ ] Logs should be emitted with OpenTelemetry resources and context for correlation where possible.
[ ] Logs should be scraped by OpenTelemetry Collectors for further filtering and processing.

Referencing issue: https://github.com/open-telemetry/opentelemetry-demo-webstore/issues/44
Service: https://github.com/open-telemetry/opentelemetry-demo-webstore/blob/main/src/emailservice/README.md
Ruby dependent on this item: https://github.com/open-telemetry/opentelemetry-demo-webstore/issues/37

Closing as logs are being pushed out


OneBitAdam Incompatible with Pipeline Parallelism

So after a bit of work we finally got 1-bit Adam working over at https://github.com/EleutherAI/gpt-neox
But it seems not to be compatible with Pipeline Parallelism. My hypothesis is that the WaitAll here https://github.com/microsoft/DeepSpeed/blob/master/deepspeed/runtime/custom_collectives.py#L99 is waiting for all ranks to reach that point, but since the pipeline stages don't all execute concurrently, some never reach it, and it errors out.
The actual error code / stacktrace I'm getting is:
    engine = PipelineEngine(args=args,
  File "/src/deepspeed/deepspeed/runtime/pipe/engine.py", line 52, in __init__
    super().__init__(*super_args, **super_kwargs)
  File "/src/deepspeed/deepspeed/runtime/engine.py", line 174, in __init__
    self._configure_optimizer(optimizer, model_parameters)
  File "/src/deepspeed/deepspeed/runtime/engine.py", line 572, in _configure_optimizer
    self.optimizer = self._configure_fp16_optimizer(basic_optimizer)
  File "/src/deepspeed/deepspeed/runtime/engine.py", line 628, in _configure_fp16_optimizer
    optimizer = FP16_Optimizer(
  File "/src/deepspeed/deepspeed/runtime/fp16/fused_optimizer.py", line 104, in __init__
    self.initialize_optimizer_states()
  File "/src/deepspeed/deepspeed/runtime/fp16/fused_optimizer.py", line 112, in initialize_optimizer_states
    self.optimizer.step()
  File "/src/deepspeed/deepspeed/runtime/fp16/onebit_adam.py", line 340, in step
    self.Compressed_Allreduce(exp_avg,
  File "/src/deepspeed/deepspeed/runtime/fp16/onebit_adam.py", line 157, in Compressed_Allreduce
    cupy_sign_list_packed, cupy_recvbuf_sign, cupy_worker_scale, cupy_recvbuf_scale = gather_host(rank,
  File "/src/deepspeed/deepspeed/runtime/custom_collectives.py", line 99, in gather_host
    MPI.Request.Waitall(requests)
  File "mpi4py/MPI/Request.pyx", line 124, in mpi4py.MPI.Request.Waitall
mpi4py.MPI.Exception: MPI_ERR_IN_STATUS: error code in status

To test my hypothesis I ran a pipeline model with a single stage (so no actual parallelism, but still using the Pipeline Module / Pipeline Engine classes), and this works fine.
I wonder if this is something you've encountered, or potentially something that's fixed by https://github.com/microsoft/DeepSpeed/pull/817 ?
@conglongli ?

updating to  #817 gives me a different error
  File "/usr/local/lib/python3.8/dist-packages/torch/_utils.py", line 264, in _flatten_dense_tensors                                               │
    flat = torch.cat([t.contiguous().view(-1) for t in tensors], dim=0)                                                                            │
RuntimeError: There were no tensor arguments to this function (e.g., you passed an empty list of Tensors), but no fallback function is registered $│
or schema aten::_cat.  This usually means that this function requires a non-empty list of Tensors.  Available functions are [CPU, CUDA, QuantizedC$│
U, BackendSelect, Named, AutogradOther, AutogradCPU, AutogradCUDA, AutogradXLA, UNKNOWN_TENSOR_TYPE_ID, AutogradMLC, AutogradNestedTensor, Autograd│
PrivateUse1, AutogradPrivateUse2, AutogradPrivateUse3, Tracer, Autocast, Batched, VmapMode].                                                       │

Getting an identical error with nccl backend

@sdtblck, thank you for your interest in 1-bit Adam. Indeed, we have not supported/tested it with pipeline parallelism (PP). Given that 1-bit Adam is intended for users with limited bandwidth, we have only tested it on smaller scales (up to 64 GPUs) on relatively smaller models like BERT-Large. The communication in 1-bit Adam code path is not hooked up with groups yet so it will fail. Are you interested in this particular combination of 1-bit and PP? We can put it on our list of things to do. If you want to contribute, you are more than welcome to open a PR for it. Let us know your thoughts :)

Hi @awan-10 this is definitely a combination we're interested in!
I've actually been working on a PR today, and I think I have something working, although it doesn't seem to offer much speedup. Maybe someone at the deepspeed team could take a brief look and tell me where I'm going wrong. I'll submit the PR shortly.

Hi @awan-10 , I've opened a PR here: https://github.com/microsoft/DeepSpeed/pull/819. Would appreciate if you could take a look when you get time.

Hi @sdtblck, thank you for the feedback and contribution. Unfortunately currently our team is quite occupied by different kinds of upcoming release items (it's been a while since our last Sep major release), so we might not be able to implement/test new features for 1-bit Adam at this moment. I skimmed over your PR https://github.com/microsoft/DeepSpeed/pull/819 and it looks correct to me (but @awan-10  and @samyam has more expertise in this area). About why you observed only minimal speedup, one hypothesis is that with more pipelines you will need to incur more number of compressed communications. And we observed that in those cases the performance gain can reduce (i.e., not only the communication volume but also the number of communications affect performance).
In the mean while, I'd like to update the doc to mention the limitation of current 1-bit Adam v2. So I have two questions:

Have you or your teammates observed any nontrivial speedup in any of your experiments with 1-bit Adam? All we heard from github issues and emails is that 1-bit Adam "didn't work" or "provide no speedup". So it'd be encouraging if there is any positive results to share :)
Have you observed any other limitation of 1-bit Adam? For example, I believe Shivanshu Purohit is your teammate and he said that 1-bit Adam doesn't work with model parallelism? https://github.com/microsoft/DeepSpeed/issues/781

Hi @conglongli .
Once we got it working correctly, and without model parallelism, we observed a significant speedup, in a similar range to what we should expect given the paper / documentation.
That's an interesting note about the param groups. It looks to me like you do a compressed allreduce for every grad/param even , unless I'm misreading the code?
        for group, grads_this_group in zip(self.param_groups, grads_group):
            ...
            for p, grad in zip(group['params'], grads_this_group):
                ...
                if self.adam_freeze_key is False:
                    ...
                else:
                    if 'non_freeze' in group.keys() and group['non_freeze'] is True:
                        ...
                    else:
                        ...
                        if self.size > 1:
                            exp_avg.set_(
                                self.comm_backend_handle.compressed_allreduce(
                                    exp_avg,
                                    state['worker_error'],
                                    state['server_error'],
                                    self.deepspeed.local_rank))

Could this not be optimized by bucketing, similar to how the deepspeed backend does it?
I'm not sure about the issue linked above, I'll have to ask Shivanshu. Our code is changing quite quickly these days, so it could be that it's due to a bug that we've now fixed.
I'm yet to do proper ablations of the speedups with megatron-style model parallelism, but it does at least run. I'll run a test and get back to you on that.

@sdtblck Glad to hear the positive results! And your observation about when the compressed allreduce happens is right in certain situation. The reason I said param group instead of param is because we used 1-bit Adam as FP16 fused optimizer, so all params in the same group are fused/flatten. This fusion is actually critical to the performance because without it you will have much more number of communications. I would say it'd helpful if you could find out how many times the compressed_allreduce is called in one step. If my assumption is correct, you will probably see some number at least close to 100 if you only see minimal speedup.

Hi @conglongli, the issue that I had opened was simply caused by the NCCL/MPI conflict we were facing. which was resolved by using deepspeed.utils.distributed. 1-bit adam definitely works with model parallelism, after resolving this issue. It also shows non-trivial speed-ups after freeze_steps are finished.

@conglongli Just realised we never responded to this - We experimentally tested your theory and in fact compressed_allreduce is only called once or twice per step, depending on the number of optimizer param groups.

Hi @sdtblck , @awan-10 and I just had a discussion about this. One thing is that 1-bit Adam's performance gain depends on the size of the model and the bandwidth of the system. With pipeline parallelism, essentially each pipeline will have smaller model size, which could affect the perf gain. Could you try increase your model size and see if that helps?
On the other hand, for you PR https://github.com/microsoft/DeepSpeed/pull/819, we do want to merge it regardless of the perf issue because it should have been like this in the first place. But because we changed the filename later (onebitadam.py to adam.py) and made some changes to the code, the PR has some conflict. Just wonder if you have some bandwidth to update the PR or create a new one? If not, do you mind us to create a new PR for this?

Hi @conglongli !
I can create a new PR for this this weekend, if you haven't already

Hi @sdtblck, actually in the near future there will be another code release related to 1-bit compression (a 1-bit LAMB optimizer https://arxiv.org/abs/2104.06069) and I'm preparing the PR which also touches some 1-bit Adam's code. So to avoid conflicts, if you don't mind I can work on adding pipeline parallelism support (to both 1-bit Adam/LAMB) on my side. I will definitely give credit to your original PR and will also invite you to review the upcoming PR (for the pipeline parallelism part). Does this work for you? Thanks.

@sdtblck I just created the 1-bit LAMB PR https://github.com/microsoft/DeepSpeed/pull/970. So feel free to choose any of the following options:

Grant me the permission to adapt your code and add 1-bit Adam/LAMB pipeline parallelism support into 1-bit LAMB PR (or later in a separate PR). I'm happy to do it as long as that's fine to you :)
Create the pipeline parallelism PR to merge into the 1-bit LAMB PR before it is merged into master
After 1-bit LAMB PR is merged into master (around next Tue), create the pipeline parallelism PR to merge into master

Hi @conglongli - I've updated my old PR to be consistent with the latest changes here https://github.com/microsoft/DeepSpeed/pull/972
Hopefully this works for you!

Thanks @sdtblck, it looks good to me so I have merged it into 1-bit LAMB PR. On our side we will add a unit test and apply the same change to 1-bit LAMB optimizer. For MPI implementation, we might leave it as it is (and document the limitation), because the NCCL implementation has superior usability and performance, so we really don't recommend to use MPI implementation (and currently not enough bandwidth to maintain it).
Another thing is that do you mind if I close this issue after merging everything into master? There still remains an open-ended question about "what kind of models can benefit from 1-bit Adam/communication compression in general". We can reopen this issue or create new issue if any of us has new findings about it :)

Sure @conglongli happy to close this.
I'll make sure to try with larger models at some point in the future and report back when i do.

Hi @conglongli, the issue that I had opened was simply caused by the NCCL/MPI conflict we were facing. which was resolved by using deepspeed.utils.distributed. 1-bit adam definitely works with model parallelism, after resolving this issue. It also shows non-trivial speed-ups after freeze_steps are finished.

I had the same problem， it seems that 1-bit adam does not work with Pipeline Paralism, have you solved this problem?@conglongli @ShivanshuPurohit, h


Adding / Upgrading Crosswalk fails

First I tried updating crosswalk (was on ionic-cli=1.3.3 before), got this error:
✗ (node) warning: Recursive process.nextTick detected. This will break in the next version of node. Please use setImmediate for recursive deferral.

util.js:35
  var str = String(f).replace(formatRegExp, function(x) {
                      ^
RangeError: Maximum call stack size exceeded

then ran rm -rf engine and ionic browser add crosswalk; now I get:
Removed old Cordova Android platform
There was an error adding the Cordova Android library
Running command: /home/s0lll0s/dub-me/client/hooks/before_platform_add/init_directories.js /home/s0lll0s/dub-me/client
The provided path does not seem to contain a Cordova platform: ./engine/cordova-android-.5.5/
Cannot find module '/home/s0lll0s/dub-me/client/engine/cordova-android-c0.5.5/package'
 (CLI v1.3.8)

Luckily I didn't update my actual project and VM...

ionic browser add crosswalk also often gets stuck on
Adding crosswalk browser

Downloading: https://github.com/driftyco/cordova-crosswalk-engine/archive/c0.6.1.zip
[=============================]  100%  0.0s

I can confirm, it's a big issue. In other attempts I get a different message:
/Users/zarko/.nvm/v0.10.36/lib/node_modules/ionic/node_modules/colors/colors.js:125
return str + '';
^
RangeError: Maximum call stack size exceeded

Environment:
node 0.10.36, npm 2.5.0, cordova 4.2.0 (with android platform 3.7.1), ionic 1.3.8 on OSX 10.10.2

I also get the same errors, the current Ionic CLI build cannot be used for creating Crosswalk based projects.

I'm looking into this issue today. I've been running node 0.10.33 with no problems. I'm upgrading now to see if I run into the same issue.

@SidneyS That's interesting that 1.3.4 fixed it.
I just bumped to v0.12 and smoothed things out for CLI 1.3.9, I was previously on v0.10.33.
I'm wondering if some updates to some modules we've used have changed that? I'll look through the changelogs of some of those modules that have been updated between 1.3.4 and now.
@zarko-tg - I see you're using Node 0.10.36, would you mind bumping to node v0.12 and seeing if that fixes it? I'm just curious.
@SidneyS @S0lll0s - What version of node are you all running? Could you copy/paste the ionic info command to help me zero in on the issue?

Hey @jbavari ! I just did a quick check.
Adding Crosswalk when using ionic-cli 1.3.10 on top of Node 0.12.0 appears to be back to normal.

Awesome, thanks @zarko-tg! Waiting to hear back from @S0lll0s before closing the issue.

@jbavari seems to be working perfectly now!


MDN HTTP Observatory is launched, and Mozilla Observatory is now deprecated.

Are you experiencing an issue with...
shields.io
🐞 Description
Today I noticed our Mozilla Observatory Grade badge was no longer working.
After revieing the linked page I noticed that the service has been moved.

Please see link to the blog post: https://developer.mozilla.org/en-US/blog/mdn-http-observatory-launch/

Today, we are delighted to announce that the HTTP Observatory's new home is MDN! This blog post explains a little bit of the history behind the tool, how we got to where we are now, what the migration means for Observatory and MDN, and what Observatory tests.

🔗 Link to the badge
https://shields.io/badges/mozilla-http-observatory-grade
💡 Possible Solution
I think we will need to embrace the new MDN HTTP observatory and change the badge accordingly.
https://developer.mozilla.org/en-US/observatory

The badge not working could be unrelated as the project is apparently going to be sunset in September.

OK. If I go to https://observatory.mozilla.org/ and search for any domain, I get " Error: HTTP Observatory is down". Maybe it is an outage, maybe they've retired it earlier than documented.
There is now a new thing.
With the old Mozilla Observatory, the API we were using was actually a public documented API https://github.com/mozilla/http-observatory/blob/main/httpobs/docs/api.md
From reverse-engineering the site, it does appear to be possible to call a URL like https://observatory-api.mdn.mozilla.net/api/v2/analyze?host=shields.io but I can't see anywhere this API is actually documented as a public API we're allowed to use.
Before we do anything else, I've raised an issue upstream at https://github.com/mdn/mdn-http-observatory/issues/43 requesting some clarification. Lets see where that goes.

Thank you

Someone from MDN has confirmed we're good to use that endpoint, so I think this is good to work on if someone wants to pick it up. I've not really looked at how much difference there is between the new and old API response format.

actually, I will submit a PR for this one


Update devops helper script based on changes in azure-sdk repo

Sync devops helper script changes
Fix issue in SemVer script when using strict mode with Language not set

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow


feat: output unification v2

Outputs root hash merkle tree is now the same for all inputs, we stop resetting it for each input (v1). With that the tree will grow indefinetely and thus we also increased its height to 63 for up to 2^63 leafs.
Merkle tree tests also had to be updated because changing its height also changes its root hash.

Looks good for me, but let's wait @diegonehab review.

Sorry, jumped the gun and merged it before seeing this comment.


DSF DB

Merchant ID - Skull Reaction
This will make it easier to track the message ID's and add the reactions more consistently in case of a server outage from the host.
To Do

[ ] Add a new DB Field for DSF Server document
[x] Save last 10 or so message ID's and createdTimestamp
[x] Then re-use node-cron to check for the amount of time passed and add the skull reactions in order, checking the first item of the array/set.
[x] Remove the ones in the array/set that have had the reaction added.

Bypassed Database for a Discord Collection.


Maybe the gold_span is mixing with pred_span when evaluating in pred_span mode?

Hi, there. Thanks for your excellent work.
In the code here, you just use the gold span if there is no pred span. Considering the situation of evaluating and predicting, is there a risk leading an evaluation bias (since we still can get the right span predictions if there is no span prediction outputs)?
Thank you very much if you could comment and give more explanations.

Well, you are right. It indeed has the risk of leaking gold spans, but such risk is nearly negligible when the model is well trained.
The code here is just a hot fix to avoid the training crash at an early stage. Once the model is well trained, the NER module can always produce some candidate spans, so the length of span_token_tup_list will be greater than zero and the model will not use those gold spans. According to our experience on the ChFinAnn dataset, a well-trained model always produces some candidate spans, so the forementioned risk is limited.
We appreciate your findings, it should be better if we disable that statement for evaluation to be risk-free.

Very appreciate for your reply.


Ticking checkboxes doesn't work

Very cool plugin to upgrade files visually! Thank you. Unfortunately it's not possible to tick checkboxes inside admonition enviroment in preview.

Okay, this should work now.
Unfortunately Obsidian's MarkdownRenderer doesn't appear to handle this situation if it is not rendering the entire page, so I had to hack it in. This may not work completely the same as native checkboxes.

Thanks, but after this update only the first checkbox of Admonition blocks can be ticked.

Okay, I see why. Let me work on it, should have a fix soon.

Can you verify if 3.1.2 fixes it?

Yes, it did fix it. Thank you very much for your fast support.

Great, thank you. Let me know if you run into other issues -- like I said, I had to hack in a way to force Obsidian to do this, and it may be a little wonky on edge cases.

Hi Valentine, it's broken again but I don't know since which release.
Thank you for fixing.


Use the camel-kamelets-catalog jar for parsing Kamelets catalog

This is more a suggestion then an issue.
The camel-kamelets-catalog jar https://search.maven.org/artifact/org.apache.camel.kamelets/camel-kamelets-catalog, already contains the kamelets for the particular version released, so it could be better to use the jar contents instead of parsing from the repository by pointing a tag.

That makes sense.
The idea is to add more ways of adding kamelets/steps so people can decide how to store their custom kamelets, so adding this to the TODO :)

+1
If we can add the other ways to add kameletes/steps in the camel-kameletes-catalog it will allow to factorize a bit more the code. It will be also easier to be reused by Camel Language Server.


[stable/jaeger-operator] Update version jaeger-operator

Signed-off-by: Login Victor<EMAIL_ADDRESS>
What this PR does / why we need it:
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/ok-to-test

/assign @cpanato

/ok-to-test

/lgtm

/test pull-charts-e2e


Sockets not closed when body is not consumed

Description
it looks like the sockets are not closed when the body is not consumed; despite a call to disconnect all.
To Reproduce
const fetchAPI = require('@adobe/helix-fetch');

async function request(context, error) {
  const resp = await context.fetch('https://adobeioruntime.net/api/v1/web/helix/helix-services/word2md@v1?path=%2Fdefault.md&shareLink=https%3A%2F%2Fadobe.sharepoint.com%2Fsites%2FTheBlog%2FShared%2520Documents%2Ftheblog&rid=VqNCOOblZXzBlnpLTvgG39uWoAIrGDWF&src=adobe%2Ftheblog%2Fdd25127aa92f65fda6a0927ed3fb00bf5dcea069', {
    headers: {
      accept: 'application/json',
    }
  });
  if (error) {
    throw error;
  }
  const text = await resp.text();
  console.log(resp.status, text);
}

async function run() {
  const context = fetchAPI.context({});
  try {
    await request(context);
    await request(context, new Error('user error'));
  } finally {
    await context.disconnectAll();
  }
}
run().catch(console.error);

$ node src/simple.js
404 Error while converting document
Error: user error
    at run (src/simple.js:36:28)
    at processTicksAndRejections (internal/process/task_queues.js:97:5)
...(process hangs)...

Note
adding keepAlive: false seems to solve the problem:
  const context = fetchAPI.context({
    http1: {
      keepAlive: false,
    },
  });

Strange, I've reported this issue a while ago: https://github.com/grantila/fetch-h2/issues/77
Supposedly it had been fix. Apparently that's not the case.

I have verified that the problem described in grantila/fetch-h2#77 does indeed still exist. I have created a new issue: https://github.com/grantila/fetch-h2/issues/101
It is specific to HTTP1(.1) connections, i.e. with HTTP2 the problem doesn't occur. It is also independent of the response status code.
The problem is somehow hidden/mitigated by helix-fetch due to caching. The body of cached responses are always consumed.  Non-cachable responses however exhibit this problem.
The following code reproduces the issue with helix-fetch:
const { fetch, disconnectAll } = require('@adobe/helix-fetch');

(async function () {
  const resp = await fetch('http://httpbin.org/status/200', { cache: 'no-store' });
  await disconnectAll();
}());

It leaves an open socket despite calling disconnectAll() which prevents the process from exiting.

fixed in v2 (75cbcbe)

fixed in v2 (75cbcbe)


Add aliases to DeviceArray that are used by Colab notebooks.

Progress on https://github.com/google/iree/issues/8253 (I believe the notebooks will run correct again after we push a nightly release including these changes).

Nice, OOC, would having a print that copies/overloading print to copy work too? (Just wondering if seeing the devicearray would be the expectation vs the value, print is debug feature so using on critical path and incurring overhead seems less surprising than reshape doing a transfer would, but getting these working is good step and then optimizing off critical path)

Nice, OOC, would having a print that copies/overloading print to copy work too? (Just wondering if seeing the devicearray would be the expectation vs the value, print is debug feature so using on critical path and incurring overhead seems less surprising than reshape doing a transfer would, but getting these working is good step and then optimizing off critical path)

If we continue down this "implicitly read back to the host" path then I'd expect __str__ to also read back to the host and print the concrete values. Doing that for too many methods will add performance cliffs though...

Nice, OOC, would having a print that copies/overloading print to copy work too? (Just wondering if seeing the devicearray would be the expectation vs the value, print is debug feature so using on critical path and incurring overhead seems less surprising than reshape doing a transfer would, but getting these working is good step and then optimizing off critical path)

If we continue down this "implicitly read back to the host" path then I'd expect __str__ to also read back to the host and print the concrete values. Doing that for too many methods will add performance cliffs though...

Let's not take that step yet. I'm fairly wary of getting down a path we can't come back from, and would like to do this incrementally.

Ping, @stellaraccident ? I'd like to either land this or changes to the Colab notebooks so we can get the https://buildkite.com/iree/iree-samples build passing again.


sandbox support

to avoid duplicate work, i closed the PR https://github.com/ringo/ringojs/pull/150
but i'm keeping the link to code & text here:
https://github.com/mrdon/ringojs/commit/28b611672e8dfad8ae5c292c7e882335ada5d9a8

Sandbox mode shutters all Java classes out but key ones
Only a handful of modules ported so far
Jsgi works for me, though I did need to subclass servlet classes and let them in via the shutter
File upload likely broken as 'fs' module can't be done easily

I will close this for now. If there is demand for a full sandbox support, we can rethink how this can be done in a more maintainable way.


Measurement results are not stable against changes in Measurement class

There seems to be some strange behavior of our Measurement class as for example adding via-routing measurements here: 9eb65586ebb55aca603e1aba4788d16581a678ec changed the measured time for routingCH(_edge)_full even though this measurement runs before the new via-routing measurement. Ideally the performance / JVM optimizations (?) should not alter the measurement results like this when adding or skipping parts of the measurements.
A similar problem arose when I tried adding some more edge-traversal tests, see the potential_jdk_issue branch...
Another thing I was not able to explain was that in this commit: 6f07e5ce7ed0b72d35e3f38a182cdd1e2ef0788a the routingCH_alt.mean time increased by about 15% (I was not able to reproduce this consistently and I was wondering if the reason I could not reproduce it was that the slow down depends on some other code running before the test...).

Some discussion of this issue here: https://bugs.java.com/bugdatabase/view_bug.do?bug_id=JDK-8247618
Maybe the first thing we can do would be changing MiniPerfTest such that we can pass in the method that shall be tested instead of subclassing and overriding it. I think this would be cleaner anyway.

Ah, this is a good idea. Will do that.


Update webapi-7x.md for release notes update for 7.5.13

Update webapi-7x.md for release notes update for 7.5.13

Docs Build status updates of commit 3698355:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Odata-docs/changelog/webapi-7x.md
:bulb:Suggestion
View
Details

Odata-docs/changelog/webapi-7x.md

Line 3, Column 14: [Suggestion: duplicate-descriptions - See documentation] Attribute 'description' with value 'OData WebApi 7.x changelog' is duplicated in 'changelog/aspnetcoreodata-8x.md(3,14)', 'changelog/webapi-7x.md(3,14)'.
Line 5, Column 12: [Suggestion: ms-author-invalid - See documentation] Invalid value for 'ms.author', 'madansr7' is not a valid Microsoft alias.
Line 32, Column 101: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'T' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 293, Column 68: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'TStructuralType' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 295, Column 110: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'T' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


How does this work?

How does this software work? Like not how to use it but the in depth of how it works and how it does what it does. Unless that is classified.

https://arxiv.org/pdf/1811.09393.pdf

Here you go: https://arxiv.org/pdf/1811.09393.pdf

Thanks so much!


Support new headless mode of Chrome

There is a new headless model in chrome, would be nice if that would be supported:
https://developer.chrome.com/articles/new-headless/

This would be great. Currently, I forked the project to enable this.

@errantmind -- have you tried extensions with your fork? It would appear that upstream (this repo) always explicitly sets --disable-extensions here https://github.com/rust-headless-chrome/rust-headless-chrome/blob/main/src/browser/process.rs#L213.

@errantmind -- have you tried extensions with your fork? It would appear that upstream (this repo) always explicitly sets --disable-extensions here https://github.com/rust-headless-chrome/rust-headless-chrome/blob/main/src/browser/process.rs#L213.

That said, even puppeteer seems to retain this option (despite supporting the "new" headless mode)
https://github.com/puppeteer/puppeteer/blob/main/packages/puppeteer-core/src/node/ChromeLauncher.ts#L233
https://github.com/puppeteer/puppeteer/blob/main/packages/puppeteer-core/src/node/ChromeLauncher.ts#L187
^ this also forms the basis of a potential implementation (similar to your fork) which suggests that --headless=new would be set if headless is specified to be true.

I haven't tried extensions yet. I don't know why this repo disables them by default, I assume there is a reason.
This repo should probably allow the new headless though similar to puppeteer

Any updates on this?
I tried to load extensions in headless mode myself by manually adding --headless=new, but extensions are still not loaded.

@netram2 https://github.com/rust-headless-chrome/rust-headless-chrome/issues/388#issuecomment-1773930556


Upgrade Cypress and Python for github actions

Codecov Report

Merging #13 (a9289a4) into main (5bde7ea) will increase coverage by 0.34%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #13      +/-   ##
==========================================
+ Coverage   83.17%   83.51%   +0.34%
==========================================
  Files          16       17       +1
  Lines         719      734      +15
==========================================
+ Hits          598      613      +15
  Misses        121      121

Flag
Coverage Δ

unittests
83.51% <ø> (+0.34%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

app/__init__.py
100.00% <0.00%> (ø)

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 5bde7ea...a9289a4. Read the comment docs.


Link occur forcibly if exists

If the target file already exists, :PlugInstall command occurs error.

Thanks!


Docs: Add multi-dimension partitioning doc; refactor native batch and separate into smaller topics.

Adds documentation for multi-dimension partitioning. cc: @kfaraz
Refactors the native batch partitioning topic as follows:

Native batch ingestion covers parallel-index
Native batch simple task indexing covers index
Native batch input sources covers ioSource
Native batch ingestion with firehose covers deprecated firehose

This PR has:

[ x] been self-reviewed.

LGTM


Add text to Profile page when not signed in and provide sign in dialog

Text for Profile page before signing:
"Sign in to Profile to see the pages you've already tagged"
Username [username entry field]
Password [password entry field]
[sign in button and forgot password button]
[Sign up link]

zooniverse.org doesn't allow direct login from other domains, but we can hide the profile unless you are logged in to Zooniverse.

I think hiding the profile is a good idea in that case.


Show instance for GenericSpine

Hi,
It seems convenient for debugging and prototyping to have a Show instance for each type.
If we do not add it to Data.Generic, it could not be added at all, due to orphan instance policy.

Ok, Phil.
Can fix that and you merge?
--
CONFIDENTIAL NOTE:
The information contained in this email is intended only for the use of the
individual or entity named above and may contain information that is
privileged, confidential and exempt from disclosure under applicable law.
If you are not the intended recipient, you are hereby notified that any
dissemination, distribution or copying of this communication is strictly
prohibited. If you have received this message in error, please immediately
notify the sender and delete the email.

I fixed that and remade commit as 2e3f5808e181f5734e73414a62312009a2ebaf8b.
See separate pull request coming.


stat/distuv: Improve test coverage for Categorical distribution + add documentation to interfaces.go

This change modifies the stat/distuv package in Gonum to:

improve test coverage for Categorical distribution
add benchmarks for Rand function on Categorical
add documentation comments to interfaces.go

Please take a look.

Please start work off origin/master so that we don't have so many commits in the PR.

Please start work off origin/master so that we don't have so many commits in the PR.

Can you recommend a workflow? I am not terribly familiar with pull requests.

When I'm working on repos that I'm not an owner of, I keep two remotes, one is the origin and the other is my fork. Before I do some work, I make sure I pull from the origin/master and then branch from there. This way, when I push to my kortschak/feature-branch remote branch, I only have the changes that I am actually making in that branch.
After the PR has been merged, I delete my feature branch, and the remote branch and pull the origin/master again, which will now contain the added feature.
With repos that I own, it's a bit easier, since I only need to have one remote, but the process is essentially still the same.

Thanks.

I think I messed up this branch, I'll make two new separate PRs.


feat(history plugin): add public api for ignoreNext

PR Checklist
Please check if your PR fulfills the following requirements:

[ ] The commit message follows our guidelines: https://github.com/datorama/akita/blob/master/CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[X] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

No api for ignoreNext form entityStateHistory
What is the new behavior?
Does this PR introduce a breaking change?
[ ] Yes
[X] No

Other information

:tada: This PR is included in version 1.11.0 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:


There are still broken documentation links

E.g. http://druid.io/docs/latest/ingestion/ingestion-faq.html
@himanshug

hmm... some of the filenames were changed in https://github.com/himanshug/druid/commit/d2346b68344ad56e05da85519f4572ef7ff00384 but it link to them were not updated.
let me fix those

actually, links were updated too
@fjy is there something linking to http://druid.io/docs/latest/ingestion/ingestion-faq.html ?

@fjy ok I see now, filename changed made in  https://github.com/druid-io/druid/commit/d2346b68344ad56e05da85519f4572ef7ff00384 are not done to the toc.textile file. let me fix that

@fjy https://github.com/druid-io/druid/pull/1483

PR to update website https://github.com/druid-io/druid-io.github.io/pull/142/

@fjy PR to update website https://github.com/druid-io/druid-io.github.io/pull/142/


[Monitor] cannot create diagnostics-settings

Installed via apt.
In my command below, $group_name, $diagnostics_name and $eventhub_name are all short (single-segment) names. $eh_authzrule_id and $storage_account_id are full resource paths (aka ids). I tried alternate combinations of short and long names too and still get the same error, I think it's something intrinsic to building the request payload.
Here's the command and trimmed debug output:
az monitor diagnostic-settings create \
    --resource-group $group_name \
    --name $diagnostics_name \
    --event-hub $eventhub_name \
    --event-hub-rule $eh_authzrule_id \
    --resource $storage_account_id

Event: CommandInvoker.OnPostCommandTableCreate [<function add_id_parameters at 0x7fad87aa5620>]
Event: CommandInvoker.OnCommandTableLoaded []
Event: CommandInvoker.OnPreParseArgs [<function _documentdb_deprecate at 0x7fad878e90d0>]
Event: CommandInvoker.OnPostParseArgs [<function OutputProducer.handle_output_argument at 0x7fad88a929d8>, <function CLIQuery.handle_query_parameter at 0x7fad88ab1510>, <function handler at 0x7fad87820598>]
usage: az monitor diagnostic-settings create [-h] [--verbose] [--debug]
                                             [--output {json,jsonc,table,tsv}]
                                             [--query JMESPATH] --name NAME
                                             <snip>
az monitor diagnostic-settings create: error: 'Namespace' object has no attribute 'event_hub_name'

related: https://github.com/Azure/azure-cli/issues/5698

@tjprescott looks like a bug. Is it a regression?

Same issue here trying to activate redis diagnostics -> eventhub
azure-cli (2.0.31)
monitor (0.1.5)

Are there any known workarounds like switching to older CLI versions? (except powershell 😬)

@troydai it's not technically a regression. A review of PR #5424 makes it clear that it never worked. The custom command signature and validator don't match, and this scenario was not covered by a test.

I'll take a look today.

Roll out as in release a new version or just bring it in the codebase? 👀

In codebase. It will show up in the next release (check: https://github.com/Azure/azure-cli/milestone/38). If you want to use it right now, you can use the edge build (https://github.com/azure/azure-cli/#edge-builds).

Fixed in https://github.com/Azure/azure-cli/pull/6216#event-1594817368.


Schemas v3.15.0

What is the context of this PR?
Latest schemas
How to review
Ensure v3.15.0 is loaded as expected.
Checklist
New static content marked up for translation
Newly defined schema content included in eq-translations repo

 Release: 4.9.2


Feature Request: Add Support for Listing Enterprise Applications in Get-EntraApplication

Describe the feature
I would like to extend the functionality of the Get-EntraApplication cmdlet to list either App Registrations or Enterprise Applications within an Entra Tenant. To achieve this, a new switch parameter (e.g., -EnterpriseApps) could be introduced, allowing users to specify whether they want to retrieve Enterprise Applications instead of the default App Registrations. This approach would eliminate the need for a separate command while enhancing versatility.
How will this feature enhance your project and further the project’s overall goals? Who will benefit from this feature (i.e. all users; the project team)?
This feature will significantly streamline workflows for companies that often need to list Enterprise Applications instead of just App Registrations. It will help users who frequently switch between managing these two types of applications. Adding this parameter will benefit not only users but also the project team by reducing the need for workarounds and custom scripts, making the cmdlet more comprehensive and user-friendly.

All users of the Get-EntraApplication cmdlet, especially those in large organizations or enterprise environments that need to frequently query both App Registrations and Enterprise Applications.
Project maintainers who can avoid creating a separate command for this purpose, keeping the command set more streamlined and maintainable.

Describe alternatives you've considered
Currently, the alternative is applying a filter to the Get-EntraApplication command to list Enterprise Applications by filtering for the WindowsAzureActiveDirectoryIntegratedApp tag. However, this filter is not intuitive, and users often need to reference external documentation or scripts to apply it correctly. An example filter:
Get-EntraApplication -Filter "tags/Any(x: x eq 'WindowsAzureActiveDirectoryIntegratedApp')"

While functional, this approach lacks the simplicity and ease of use that a dedicated switch parameter would provide.
Additional context
I have written a small PowerShell script to simplify listing Enterprise Applications. However, it would be beneficial to have this feature built directly into the cmdlet. You can find the script in this Gist.
Additionally, @stevemutungi suggested reaching out here for further discussion and to potentially get this feature implemented.
I have written a little script to list them because at least for me it's a tough thing to remember the filter.

I’ve successfully implemented the switch in the psm1 file on my local machine. Additionally, I ensured that any pre-existing filters are properly extended to accommodate the new switch without overriding previous settings.
Param Section
    param (
    [Parameter(ParameterSetName = "GetVague", ValueFromPipeline = $true, ValueFromPipelineByPropertyName = $true)]
    [System.String] $SearchString,
    [Parameter(ParameterSetName = "GetQuery", ValueFromPipeline = $true, ValueFromPipelineByPropertyName = $true)]
    [System.Nullable`1[System.Int32]] $Top,
    [Parameter(ParameterSetName = "GetById", Mandatory = $true, ValueFromPipeline = $true, ValueFromPipelineByPropertyName = $true)]
    [System.String] $ObjectId,
    [Parameter(ValueFromPipeline = $true, ValueFromPipelineByPropertyName = $true)]
    [switch] $All,
    [Parameter(ParameterSetName = "GetQuery", ValueFromPipeline = $true, ValueFromPipelineByPropertyName = $true)]
    [System.String] $Filter,
    [Parameter(Mandatory = $false, ValueFromPipeline = $false, ValueFromPipelineByPropertyName = $true)]
    [System.String[]] $Property,
    [switch]$EnterpriseApp
    )

** Process Section **
    if($EnterpriseApp)
    {
        if($Params.ContainsKey("Filter")){
            $Params["Filter"] = $Params["Filter"] + " and tags/Any(x: x eq 'WindowsAzureActiveDirectoryIntegratedApp')"
        }
        else{
            $Params["Filter"] = "tags/Any(x: x eq 'WindowsAzureActiveDirectoryIntegratedApp')"
        }
        $response = Get-MgServicePrincipal @params -Headers $customHeaders
    }
    else
    {
        $response = Get-MgApplication @params -Headers $customHeaders
    }

@SteveMutungi254 You're very welcome! I would have loved to create a PR, but I'm unsure how to commit my changes to the repo since the parameter section seems to be generated during the build process. Instead, I applied the changes to my local module and shared the details via an issue here for your review.

@HCRitter: We apologize for the delay and appreciate your patience. We haven’t forgotten about this—we’re conducting further analysis to add more options (cmdlet switches) to additional commands for improved usability.


no lower bound is 0 not *

You have around 1000 files with "version": "*" exactly 7 lines after a CPE like:
https://github.com/cisagov/vulnrichment/blob/c31752fa4a9da78d1b6af62037ea7112a6b7d584/2024/36xxx/CVE-2024-36076.json#L90-L97
but I think you meant 0 (I guess for all 1000) because * means infinity in CVEs and it's nonsensical to start ranges at infinity. https://github.com/CVEProject/cve-schema/blob/master/schema/docs/versions.md In any version range, the details of the version syntax and semantics depend on the version type, but by convention, "version": "0" means that the range has no lower bound, and a * in an upper bound denotes “infinity”,

@rivkasegan thank you for the report. This is a process issue/question that I am referring to our analysts.

Taking a pass at this to see about fixing these (637 entries) lower-bound "*" characters. Stay tuned!

These have all been addressed as of now. The last fixes will be pushing out everywhere within an hour.


🐛 [Flexget] Not getting updated

Description
Auto updater didn't updated Flexget in last 5 months.
Reproduction steps
The updater updated it to a strange version number, but I don't see why It is a problem for it.

Addon Logs
I don't have those :D

Architecture
No response
OS
No response

It looks like fixed, thank you

Thanks, I corrected the updater json file


What is the difference between an Event and an Activity?

This is from the README:

Activities : what people have to propose for an event, can be a talk, a workshop, a run, a game...
Events : organize an event that will propose activities and that people can join to share this activity...

Is an Event a collection of Activities? Like a conference with many talks.
Is an Activity just a type of Event? A talk Event, a workshop Event, etc.
Is Event a suggestion generater? If you liked that running Activity you might likes these other running Activities too.
Are the two actually the same thing?

I guess (I'll let others correct me if I'm wrong) that an Event is is a talk/workshop/run/game that will happen or has happened, while an activity is more generic, does'nt have a fixed date and can happen several times.  For example if a group of people like to play tennis, tennis would be the activity while "this saturday afternoon tennis at the park" would be the event.
This means that people can suggest activities they would like to happen.  Someone can then make an event out of an activity.

I guess (I'll let others correct me if I'm wrong) that an Event is is a talk/workshop/run/game that will happen or has happened,

I agree.

while an activity is more generic, does'nt have a fixed date and can happen several times.

However, this sounds a lot like an Event too. An Event without a fixed date is a suggestion. An Event that happens several times is a recurring Event.

For example if a group of people like to play tennis, tennis would be the activity

This seems to make Activity an attribute of User.

while "this saturday afternoon tennis at the park" would be the event.

I agree.

This means that people can suggest activities they would like to happen. Someone can then make an event out of an activity.

It would allow an Organizer (Role played by User) of an Event to find a potential Participant and/or Sponsor (Roles played by User) based on their Activity (or other attibutes, where they live for instance) and message them.

I went back to the Event Storming pictures and the Activity does not seem to be in any of the photos...it seems to have entered the when cards were added to the Trello board. On the Trello board there is a lot of overlap with cards related to Event and Activity. I believe they are the same thing and we just need to settle on the word we want to use...and then update Trello, the README and our directory structure.

I think that distinction occured at the 3rd meeting, when categories changed.  Since I wasn't there, maybe someone else could explain exactly what the difference is ?

While we were adding the trello cards we split into 3 groups : spectators, organizers, speakers.
Spectators can attend an event.
Event are organized by organizers.
Organizers pick in a pool of talks suggested by speakers.
At first we called it talks and speakers but we thought, why restrain this to only talks ? What if we want another kind of activity ?
That's why we ended up with :
Organizers pick in a pool of activities suggested by other people.

@Dayde @thebignet Thank you for your feedback.

Spectators can attend an event.

Agree. However, I would suggest Participant rather than Spectator to describe this Role.
User (Participant) attends an Event.

Event are organized by organizers.

No arguement here either. I think Organizer is a good name for the Role too.
User (Organizer) creates an Event.

At first we called it talks and speakers but we thought, why restrain this to only talks ? What if we want another kind of activity ?
That's why we ended up with :
Organizers pick in a pool of activities suggested by other people.

I also agree that generalizing talks to something else is a good idea. I would suggest Interest rather than Activity.
Now to the part that needs clarification...

Organizers pick in a pool of activities suggested by other people.

This is search functionality.
For example, An User (Organizer) wants to create an Event based on an Interest and they need to find a User (Presenter) who knows the Interest to lead other Users (Participants) with the same Interest.
The same applies to an User (Organizer) who wants to create an Event based on an Interest, but who does not need a User (Presenter). The User (Organizer) still needs to find other Users (Participants) with the same Interest.
It also applies to an User (Participant) who wants to find an Event based on an Interest.
To me Organizer, Presenter and Participant are all Roles played by an User on a Event. The thing that all these Users and Event share is an Interest.
I opened an another issue to capture changes to the domain model based on the information from this discussion and potentially other discussions as they provide deeper insight and evole the domain model.


Add multiple trustedCertEntries to truststores.

Is your feature request related to a problem? Please describe.
We are currently working on external issuer for cert manager, and we would like to be able to add multiple trustedCertEntries to JKS truststore.
We are able to set the certificateRequest.Status.CA with a PEM containing multiple entries, but for the JKS truststore we end up with only first entry from the PEM, while for the ca.crt file (from target certificate secret) all entries are present.
Describe the solution you'd like
Read all certificate entries from certificateRequest.Status.CA and add them to JKS truststore. First certificate should be treated as CA used to sign the keystore while rest of them as extra trusted certificate authorities.
Describe alternatives you've considered
Adding new field in certificateRequest.Status like "trustedCertificates"
Additional context
We could probably support this feature, but we'd like to hear some feedback about the proposition first.
Environment details (if applicable):

Kubernetes version (e.g. v1.10.2): 1.17.2
Cloud-provider/provisioner (e.g. GKE, kops AWS, etc): N/A
cert-manager version (e.g. v0.4.0): v1.0.3
Install method (e.g. helm or static manifests): N/A

/kind feature

if ca.crt doesn't match with the content of JKS that feels like a bug to me,
On the proper root store distribution (going further than issuer provided CA) there is some discssions going around in both the Kubernetes as cert-manager community which expands this further. I think for this ticket it is out of scope and a fix for the JKS code to properly convert the ca.pem to the JKS store would work.
/remove-kind feature
/kind bug


Analyse and fix 'awaitEvent: invalid argument (Bad filedescriptor)' output

The cause is probably hidden in the GHC RTS.
socket-<IP_ADDRESS>: test (suite: default)

Progress: 1/2socket
  System.Socket
    accept
      Inet/Strem/TCP
        cancel operation:                                       OK (1.00s)
    connect
      Inet/Stream/TCP
        connect to closed port on inetLoopback:                 default: awaitEvent: invalid argument (Bad file descriptor)
OK
        connect to closed port on inetNone:                     OK

The error is thrown from within Haskell here: https://github.com/ghc/ghc/blob/876b00ba25a615423f48b0cf9d443a9fd5dbd6f4/libraries/base/GHC/Event/Thread.hs#L126


Why train model on smaller and smaller set of features recursively

Hi,
I have some doubts on recursive xgboost model process. In the source code, it seems the models were trained on smaller and smaller set of features that were selected by their column index in an order.
   for i in range(0,train_p.shape[1],iter_limit):
        start_time2 = time.time()
        imp_feats = []
        if train_p.shape[1]-i < iter_limit:
            X_train = train_p.iloc[:,i:]
            cols_sel = X_train.columns.tolist()
        else:
            X_train = train_p[list(train_p.columns.values)[i:train_p.shape[1]]]
            cols_sel = X_train.columns.tolist()

Is there a reason to select the subset of features by column order? And why train models on shrinking set of features repeatedly?
Thank you

Great question @MySciencePlayground 👍
The reason I chose this method is that I found permutation based methods where:

Multiple permutations of features are selected sequentially is very time consuming and for large datasets, can be explosive in terms of cost
Random selection of baskets of features may leave out certain features and that could be costly.
The best compromise between the two methods is this sequential selection method where smaller baskets of features are selected and it scales well even when 100's or 1000's of features are in a dataset. That's why it works well for large datasets as well as for small datasets.
Hope this clarifies
AutoVimal

Thanks for the quick respond, @AutoViML! So with the current approach, the column order can have some effect on the final selection. Out of curiosity, is there any test/evaluation on how different the results could be ? (applying same dataset but changing the column order)

Hi @MySciencePlayground
That's a good question but I was hoping you would try it yourself out of curiosity by changing the column order and testing it. I have done my tests and I will leave it to you to try it and report back your findings and questions if any 👍
This is the tryout medal for your efforts!🥇
AutoVimal


[SPARK-7853] [SQL] Fix bug of class loader issue in Spark SQL

bin/spark-sql --jars ./sql/hive/src/test/resources/hive-hcatalog-core-0.13.1.jar
CREATE TABLE t1(a string, b string) ROW FORMAT SERDE 'org.apache.hive.hcatalog.data.JsonSerDe';

Throws exception like
15/05/26 00:16:33 ERROR SparkSQLDriver: Failed in [CREATE TABLE t1(a string, b string) ROW FORMAT SERDE 'org.apache.hive.hcatalog.data.JsonSerDe']
org.apache.spark.sql.execution.QueryExecutionException: FAILED: Execution Error, return code 1 from org.apache.hadoop.hive.ql.exec.DDLTask. Cannot validate serde: org.apache.hive.hcatalog.data.JsonSerDe
at org.apache.spark.sql.hive.client.ClientWrapper$$anonfun$runHive$1.apply(ClientWrapper.scala:333)
at org.apache.spark.sql.hive.client.ClientWrapper$$anonfun$runHive$1.apply(ClientWrapper.scala:310)
at org.apache.spark.sql.hive.client.ClientWrapper.withHiveState(ClientWrapper.scala:139)
at org.apache.spark.sql.hive.client.ClientWrapper.runHive(ClientWrapper.scala:310)
at org.apache.spark.sql.hive.client.ClientWrapper.runSqlHive(ClientWrapper.scala:300)
at org.apache.spark.sql.hive.HiveContext.runSqlHive(HiveContext.scala:457)
at org.apache.spark.sql.hive.execution.HiveNativeCommand.run(HiveNativeCommand.scala:33)
at org.apache.spark.sql.execution.ExecutedCommand.sideEffectResult$lzycompute(commands.scala:57)
at org.apache.spark.sql.execution.ExecutedCommand.sideEffectResult(commands.scala:57)
at org.apache.spark.sql.execution.ExecutedCommand.doExecute(commands.scala:68)
at org.apache.spark.sql.execution.SparkPlan$$anonfun$execute$1.apply(SparkPlan.scala:88)
at org.apache.spark.sql.execution.SparkPlan$$anonfun$execute$1.apply(SparkPlan.scala:88)
at org.apache.spark.rdd.RDDOperationScope$.withScope(RDDOperationScope.scala:148)
at org.apache.spark.sql.execution.SparkPlan.execute(SparkPlan.scala:87)
at org.apache.spark.sql.SQLContext$QueryExecution.toRdd$lzycompute(SQLContext.scala:922)
at org.apache.spark.sql.SQLContext$QueryExecution.toRdd(SQLContext.scala:922)
at org.apache.spark.sql.DataFrame.<init>(DataFrame.scala:147)
at org.apache.spark.sql.DataFrame.<init>(DataFrame.scala:131)
at org.apache.spark.sql.DataFrame$.apply(DataFrame.scala:51)
at org.apache.spark.sql.SQLContext.sql(SQLContext.scala:727)
at org.apache.spark.sql.hive.thriftserver.AbstractSparkSQLDriver.run(AbstractSparkSQLDriver.scala:57)
at org.apache.spark.sql.hive.thriftserver.SparkSQLCLIDriver.processCmd(SparkSQLC

Merged build triggered.

Merged build started.

Test build #33472 has started for   PR 6396 at commit 7bc8502.

Test build #33472 has finished for   PR 6396 at commit 7bc8502.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33472/
Test FAILed.

Merged build finished. Test FAILed.

Merged build triggered.

Merged build started.

Test build #33516 has started for   PR 6396 at commit dd78775.

Test build #33516 has finished for   PR 6396 at commit dd78775.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33516/
Test FAILed.

Merged build triggered.

Merged build started.

Test build #33520 has started for   PR 6396 at commit fd26533.

Test build #33520 has finished for   PR 6396 at commit fd26533.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33520/
Test FAILed.

@marmbrus @yhuai @liancheng

@chenghao-intel We will submit a pr based on yours later.

sure, I am also working on this now, hopefully we can solve that asap.

This has been fixed right? If yes, can we close this?

oh, yes, closing it.


Add travis configuration

Related to #23

Changes Unknown when pulling ef49657698eb8e56390cd3c50cb466cb3c897aed on i23 into ** on master**.

Changes Unknown when pulling 81557d04d1bd8f095f932de4832aaf1fe331cf71 on i23 into ** on master**.

Changes Unknown when pulling 0ab63211c70953281ee79445f82c7f626f3acba1 on i23 into ** on master**.


Connlib/forward dns

With this we implement DNS forwarding that's specified in  #2043
This also solve the DNS story in Android.
For the headless client in Linux we still need to implement split dns, but we can make do with this, specially, we can read from resolvconf and use the forward DNS (not ideal but can work if we want a beta headless client).
For the resolver I used trusted-proto-resolver.
The other options were:

Using domain's resolver but while it could work for now, it's no ideal for this since it doesn't support DoH or DoT and doesn't provide us with a DNS cache.
Using trusted-proto-client, it doesn't provide us with a DNS cache, though we could eventually replace it since it provides a way to access the underlying buffer which could make our code a bit simpler.
Writing our own. While we could make the API ideal, this is too much work for beta.

@pratikvelani I did some refactor in the kotlin side so we can return an array of bytearrays so that we don't require parsing on connlib side, I also tried to make the dns server detector a bit simpler please take a look it's my first time doing kotlin
@thomaseizinger please take a look specially at the first commit, I tried to integrate with the poll_events and the ClientState.

@conectado will need to pull in main to get the right checks to run


[newrelic-logging] Omit cluster domain from Kube_URL

Is this a new chart
No.
What this PR does / why we need it:
Removes domain name from the fluent-bit kubernetes filter parameter Kube_API.
Also fixes some typos in the Readme and cleans it up a bit.
Which issue this PR fixes

fixes #147

Special notes for your reviewer:
The new value for Kube_API is the default in fluent-bit since version 1.1.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] Chart Version bumped
[x] Title of the PR starts with chart name (e.g. [mychartname])

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

Thank you @vrlo for the PR. looks good


python310Packages.json-rpc: 1.13.0 -> 1.14.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.json-rpc is: "JSON-RPC 1/2 transport implementation"
meta.homepage for python310Packages.json-rpc is: "https://github.com/pavlov99/json-rpc"
meta.changelog for python310Packages.json-rpc is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
Warning: a test defined in passthru.tests did not pass
found 1.14.0 with grep in /nix/store/bk6946lg1rspfbj92jm5wdxji5k5wwyi-python3.10-json-rpc-1.14.0
found 1.14.0 in filename of file in /nix/store/bk6946lg1rspfbj92jm5wdxji5k5wwyi-python3.10-json-rpc-1.14.0

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

4 total rebuild path(s)

4 package rebuild(s)

First fifty rebuilds by attrpath
python310Packages.dragonfly
python310Packages.json-rpc
python39Packages.dragonfly
python39Packages.json-rpc

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/bk6946lg1rspfbj92jm5wdxji5k5wwyi-python3.10-json-rpc-1.14.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/bk6946lg1rspfbj92jm5wdxji5k5wwyi-python3.10-json-rpc-1.14.0
ls -la /nix/store/bk6946lg1rspfbj92jm5wdxji5k5wwyi-python3.10-json-rpc-1.14.0/bin

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

4 packages built:

python310Packages.dragonfly
python310Packages.json-rpc
python39Packages.dragonfly
python39Packages.json-rpc

Maintainer pings
cc @oxzi for testing.

Result of nixpkgs-review pr 207415 run on x86_64-linux 1

1 package marked as broken and skipped:

darwin.builder

4 packages built:

python310Packages.dragonfly
python310Packages.json-rpc
python39Packages.dragonfly
python39Packages.json-rpc


Rename broken

Selected a variable, F2 to rename:

And nothing gets renamed

This is Windows 10 Pro, VS Professional 15.7.1

We've fixed refactor rename for 15.7.2, which will be out very soon.
https://github.com/Microsoft/PTVS/issues/4169
It's likely to fix your scenario, but I can't say for sure without seeing your code. Let us know if the issue persists once you've updated.