2022-09-12 - KDB Project Call Meeting Agenda

Date
Monday, September 12, 2022 - 9:30am ET / 2:30pm UK
// Second Monday of every month
Attendees

Fullname
Affiliation
GitHub Username

Meeting notices

FINOS Project leads are responsible for observing the FINOS guidelines for running project meetings. Project maintainers can find additional resources in the FINOS Maintainers Cheatsheet.

All participants in FINOS project meetings are subject to the LF Antitrust Policy, the FINOS Community Code of Conduct and all other FINOS policies.

FINOS meetings involve participation by industry competitors, and it is the intention of FINOS and the Linux Foundation to conduct all of its activities in accordance with applicable antitrust and competition laws. It is therefore extremely important that attendees adhere to meeting agendas, and be aware of, and not participate in, any activities that are prohibited under applicable US state, federal or foreign antitrust and competition laws. Please contact<EMAIL_ADDRESS>with any questions.

FINOS project meetings may be recorded for use solely by the FINOS team for administration purposes. In very limited instances, and with explicit approval, recordings may be made more widely available.

Agenda

(5 min) Convene, roll call, welcome new people.
[x] Display FINOS Antitrust Policy summary slide
[x] Review Meeting Notices (see above)
(5 min) Approve previous meeting minutes.
(10 min) Demo of vscode-q integration with VS Code Notebook API.

https://code.visualstudio.com/api/extension-guides/notebook

(5 min) Kdb Studio customization sound reasonable?

https://github.com/finos/kdb-studio/issues/136

(10 min) Configuring GitHub CVE scanning instead of Whitesource.

Will be closing the Whitesource issue:

https://github.com/finos/qPython/issues/8

Maoo's updated docs:

https://github.com/maoo/security-scanning

https://github.com/finos/qPython/blob/main/.github/workflows/cve-scanning.yml

Added workflow_dispatch.

Found a vulnerability!

https://github.com/finos/qPython/runs/8164123578?check_suite_focus=true

(10 min) Discussion of CCLA / ICLA status.
(5 min) Even on Thursday : KX LIVE: Breakthrough to Real-time Data Analytics in the Cloud

https://www.tickettailor.com/events/kx1/729530

(5 min) AOB, Q&A, Adjourn.

Decisions Made

Decision 1
Decision 2
...

Action Items

Action 1
Action 2
...

WebEx info
Webex

https://finos.webex.com/finos/j.php?MTID=me61719ca3c3295a7cc76762d78392bc3

Dial-in and Project Materials

US: +1-415-655-0003 US Toll
UK: +44-20319-88141 UK Toll
Access code: 2704 417 0390
GitHub Repos:

https://github.com/finos/kdb
https://github.com/finos/kdb-studio

Wiki Page:

https://finosfoundation.atlassian.net/wiki/spaces/DT/pages/329383945/kdb+Project

KDB Metrics on FINOS

https://metrics.finos.org/goto/033b15c80f0d08cab70f40f4358932f6

Hi!

hi

Hi :fox_face:

Hi

Howdy!


Documentation for operatorUI and operatorAPI

Hi ,
Where can I find usage documentation for operatorUI and operatorAPI. This is my first attempt to use operator.
Regards,

@vipulsha22
https://hyperledger.github.io/bevel-operator-fabric/docs/operator-ui/getting-started

Thank you. It's coming for me now. Got confused with port.


Fails to install under FreeBSD

zef install NCurses
===> Searching for: NCurses
===> Searching for missing dependencies: LibraryCheck
===> Testing: LibraryCheck:ver<0.0.6>:authgithub:jonathanstowe
===> Testing [OK] for LibraryCheck:ver<0.0.6>:authgithub:jonathanstowe
===> Testing: NCurses:ver<0.6.3>:authgithub:azawawi
===> Testing [FAIL]: NCurses:ver<0.6.3>:authgithub:azawawi
Aborting due to test failure: NCurses:ver<0.6.3>:authgithub:azawawi (use --force-test to override)

perl6 --version
This is Rakudo Star version 2018.06 built on MoarVM version 2018.06
implementing Perl 6.c.

zef uninstall NCurses
===> Uninstalled from /opt/rakudo-2018.06/share/perl6/site
NCurses:ver<0.6.3>:authgithub:azawawi
root@FreeBSD11:~/jtt # zef install --verbose NCurses
===> Searching for: NCurses
===> Updated cpan mirror: https://raw.githubusercontent.com/ugexe/Perl6-ecosystems/master/cpan1.json
===> Updated p6c mirror: http://ecosystem-api.p6c.org/projects1.json
===> Found: NCurses:ver<0.6.3>:authgithub:azawawi [via Zef::Repository::Ecosystems]
===> Fetching [OK]: NCurses:ver<0.6.3>:authgithub:azawawi to /root/.zef/tmp/1541610549.1200.3072/perl6-ncurses.git
===> Extraction [OK]: NCurses to /root/.zef/store/perl6-ncurses.git
===> Testing: NCurses:ver<0.6.3>:authgithub:azawawi
t/01-sanity.t ....... ok
t/02-basic.t ........ All 1 subtests passed
t/99-author-meta.t .. ok
All tests successful.
Test Summary Report
t/02-basic.t  (Wstat: 0 Tests: 0 Failed: 0)
Parse errors: Bad plan.  You planned 1 tests but ran 0.
Files=3, Tests=2,  15 wallclock secs
Result: FAILED
===> Testing [FAIL]: NCurses:ver<0.6.3>:authgithub:azawawi
Aborting due to test failure: NCurses:ver<0.6.3>:authgithub:azawawi (use --force-test to override)

I'm going to close this as I had a fresh install that didn't show any problems.
if it comes up again, I'll report it.


[docs] Add a post about generating random data

@testcafe-build-bot /retest
oh twitter

@testcafe-build-bot \retest

@testcafe-build-bot \retest

what on earth is wrong with Twitter...

I guess we can ignore these errors, they don't make sense, we'll wait and see


allow nil values in record

I'm not sure that's a good default. Maybe we can make it explicit? Something like
record Foo, required_field, optional_field?

I'm not sure about this either. What's your use case?

brainfuc interepter:
record Op, op, subtree
...
  iterator.each do |c|
    op = case c
      when '+'; Op.new(:inc, nil)
      when '-'; Op.new(:dec, nil)
      when '>'; Op.new(:movef, nil)
      when '<'; Op.new(:moveb, nil)
      when '.'; Op.new(:print, nil)
      when '['; Op.new(:loop, parse(iterator))
      when ']'; break;
    end

    res << op if op
  end```

In that case I'd prefer to do this:
record Record1, a, b do
  def initialize(@a)
  end
end

p Record1.new(1, 2)
p Record1.new(3)

The problem with nil is that if you forget to pass an argument you'll get an error in the place where you access that property. I find the above snippet, or an explicit nil, preferable.
I also think the use case for record is like a tuple, but with names instead of indices. Since a tuple has a fixed size, I see record the same way. Allowing to create a record with less elements seems odd.


False positive for FastAPI return types.

As mentioned in #52
According to this test, CustomType should be moved to a TYPE_CHECKING block.
@pytest.mark.parametrize('fdef', ['def', 'async def'])
def test_api_router_decorated_function_return_type(fdef):
    """
    We don't care about return types. To my knowledge,
    these are not evaluated by FastAPI/pydantic.
    """
    example = textwrap.dedent(f'''
        from fastapi import APIRouter
        from fastapi import Request

        from app.types import CustomType

        some_router = APIRouter(prefix='/some-path')

        @some_router.get('/{{resource_id}}')
        {fdef} list_something(request: Request) -> CustomType:
            return None
        ''')
    assert _get_error(example, error_code_filter='TC001,TC002,TC003', **defaults) == {
        '5:0 ' + TC002.format(module='app.types.CustomType')
    }

However, doing so, even with a pretty basic route gives the following error in FastAPI:
     def test() -> Test:
                   ^^^^
 NameError: name 'Test' is not defined

Agree that this seems unexpected. Looking at this quickly, I would expect:
from fastapi import APIRouter
from fastapi import Request

from app.types import CustomType

some_router = APIRouter(prefix='/some-path')

@some_router.get('/{{resource_id}}')
def list_something(request: Request) -> "CustomType":
    return None

or
from __future__ import annotations

from fastapi import APIRouter
from fastapi import Request

from app.types import CustomType

some_router = APIRouter(prefix='/some-path')

@some_router.get('/{{resource_id}}')
def list_something(request: Request) -> CustomType:
    return None

To be correct, wrt. to the return type.
Would you be interested in creating a fix for this @danpker? I'm on holiday at the moment, so won't have time to look at this, but will make time to push a fix if one is created 👍


_HOT%%LINK__Engonga Ebang Baltasar Viral video ʟᴇᴀᴋᴇᴅ Video ᴠɪʀᴀʟ On Social Media

𝖮𝗋𝗂𝗀𝗂𝗇𝖺𝗅 𝖵𝗂𝖽𝖾𝗈 𝗏𝗂𝖽𝖾𝗈 𝗍𝗈𝗈𝗄 𝗍𝗁𝖾 𝗂𝗇𝗍𝖾𝗋𝗇𝖾𝗍 𝖻𝗒 𝗌𝗍𝗈𝗋𝗆 𝖺𝗇𝖽 𝖺𝗆𝖺𝗓𝖾𝖽 𝗏𝗂𝖾𝗐𝖾𝗋𝗌 𝗈𝗇 𝗏𝖺𝗋𝗂𝗈𝗎𝗌 𝗌𝗈𝖼𝗂𝖺𝗅 𝗆𝖾𝖽𝗂𝖺 𝗉𝗅𝖺𝗍𝖿𝗈𝗋𝗆𝗌 All 𝗏𝗂𝗋𝖺𝗅 𝖿𝗎𝗅𝗅 𝗏𝗂𝖽𝖾𝗈 𝗅𝖾𝖺𝗄𝖾𝖽 𝗌𝗈𝖼𝗂𝖺𝗅 𝗆𝖾𝖽𝗂𝖺 𝗍𝗐𝗂𝗍𝗍𝖾𝗋
🌐 𝖢𝖫𝖨𝖢𝖪 𝖧𝖤𝖱𝖤 🟢==►► 𝖶𝖠𝖳𝖢𝖧 𝖭𝖮𝖶
🔴 𝖢𝖫𝖨𝖢𝖪 𝖧𝖤𝖱𝖤 🌐==►► 𝖣𝗈𝗐𝗇𝗅𝗈𝖺𝖽 𝖭𝗈𝗐
🔴 𝖢𝖫𝖨𝖢𝖪 𝖧𝖤𝖱𝖤 🌐==►► 𝖣𝗈𝗐𝗇𝗅𝗈𝖺𝖽 𝖭𝗈𝗐
Civil servants caught having sex in their offices will face "severe measures", the vice-president of Equatorial Guinea has warned after hundreds of allegedly pornographic videos were leaked to the public.
The videos reportedly involve high-ranking civil servant Baltasar Ebang Engonga having sex with various women - including the wives of prominent officials - in his office.
Mr Engonga is head of the country's National Financial Investigation Agency and also a relative of the country's president.
He has not yet commented.
Vice-President Teodoro Obiang Mangue said any officials found engaging in sex acts at work would be suspended as this was a "flagrant violation of the code of conduct".
He has also ordered the installation of surveillance cameras in courts and ministries to combat "indecent and illicit acts".
A statement from the vice-president's office said the decision had been taken in the wake of the widely circulated videos which had "denigrated the image of the country".
It added that he had recommended opening an investigation.
The videos in question were leaked after Mr Engonga was arrested on separate corruption charges, state television TVGE said.
Last week, Vice-President Obiang said "pornographic videos" had been "flooding" social media and ordered telecoms companies to curb the spread.
Since then, the flow of internet traffic - particularly the downloading of images - has been severely disrupted in the country, people in Equatorial Guinea told the AFP news agency.
As head of the National Financial Investigation Agency, Mr Engonga works on tackling financial crimes such as money laundering.
Following the furore over the videos, Equatorial Guinea's chief prosecutor Anatolio Nzang Nguema told state TV that if Mr Engonga was found to have been "infected with a sexually transmitted disease" he would be prosecuted for an offence against "public health".


query.graph typing issue

Hi,
I'm trying out this template and encountering an error when copying the REST API.
In the src/api/admin/companies/[id]/employees/route.ts API route, this repository has the following definition:
  const { data: employees, metadata } = await query.graph(
    {
      entity: 'employee',
      fields: req.remoteQueryConfig.fields,
      filters: {
        company_id: id,
        ...req.filterableFields,
      },
    },
    { throwIfKeyNotFound: true }
  );

However, the generated types do not include a company_id field for some reason.

I'm still new here, and any help is appreciated. In the documentation, I can't find anything related to this auto-generated type.

Please refer to these docs on how to filter links: https://docs.medusajs.com/learn/fundamentals/module-links/query#apply-filters
You can also see an example of a different approach here: https://github.com/medusajs/medusa/blob/dc5e73af4d2fe4eed8239660d74af6cd2a0994cd/packages/medusa/src/api/store/products/middlewares.ts#L33

Hi @sradevski, thanks for the quick response! The reference code is helpful. However, I have a concern about the generated typing for the custom model.
Here’s the code defining the Employee model:
export const Employee = model.define('employee', {
  id: model.id({ prefix: 'emp' }).primaryKey(),
  spending_limit: model.bigNumber().default(0),
  is_admin: model.boolean().default(false),

  company: model.belongsTo(() => Company, {
    mappedBy: 'employees',
  }),
});

And the corresponding database DML:
CREATE TABLE public.employee
(
    id                 text                                   NOT NULL
        PRIMARY KEY,
    spending_limit     numeric                  DEFAULT 0     NOT NULL,
    is_admin           boolean                  DEFAULT FALSE NOT NULL,
    company_id         text                                   NOT NULL
        CONSTRAINT employee_company_id_foreign
            REFERENCES public.company
            ON UPDATE CASCADE,
    raw_spending_limit jsonb                                  NOT NULL,
    created_at         timestamp WITH TIME ZONE DEFAULT NOW() NOT NULL,
    updated_at         timestamp WITH TIME ZONE DEFAULT NOW() NOT NULL,
    deleted_at         timestamp WITH TIME ZONE
);

Here’s the auto-generated typing for this model (from .medusa/types/remote-query-entry-points.d.ts):
export type Employee = {
  __typename?: 'Employee';
  id: Scalars['ID']['output'];
  spending_limit: Scalars['String']['output'];
  is_admin: Scalars['Boolean']['output'];
  company: Company;
  raw_spending_limit: Scalars['JSON']['output'];
  created_at: Scalars['DateTime']['output'];
  updated_at: Scalars['DateTime']['output'];
  deleted_at: Maybe<Scalars['DateTime']['output']>;
  customer_link: Maybe<LinkCompanyEmployeeCustomerCustomer>;
};

As you can see, the generated type includes only company but not company_id. How can I adjust the model definition, so the generated typing to include a company_id field?
In the core model example with MikroORM, e.g. in the shipping-option model, https://github.com/medusajs/medusa/blob/376b9e85bcce7923f702f822a4e4b36302eb4a42/packages/modules/fulfillment/src/models/shipping-option.ts#L100. Both shipping_profile_id and shipping_profile were generated in type. I’d like something similar for the company and company_id.


how to add comment to the transaction in z-nomp

i want to add a comment to the transaction,etc domain name.

it is done in the node module "node-stratum-pool" Not in z-nomp

i just want to add my pool web site domain name to the transaction. can you tell me the details?

in bitcoinjs-lib-zcash?

i think its in "transactions.js" under node_modules/stratum-pool/lib on line 60:

tx.addInput(new Buffer('0000000000000000000000000000000000000000000000000000000000000000', 'hex'),
4294967295,
4294967295,
new Buffer.concat([serializedBlockHeight,
Buffer('5a2d4e4f4d50212068747470733a2f2f6769746875622e636f6d2f6a6f7368756179616275742f7a2d6e6f6d70', 'hex')]) //Z-NOMP! https://github.com/joshuayabut/z-nomp
);

what you want is to change the hex part to what you want it to say:

Buffer('5a2d4e4f4d50212068747470733a2f2f6769746875622e636f6d2f6a6f7368756179616275742f7a2d6e6f6d70', 'hex')])

i tried this before ... but it never worked for me :)))

yes, it does not work by default!

anybody show me the code details?


Explore doesn' work for me.

Hello,
I just copy zabapgit from http://www.abapgit.org and try the button Explore but i only see blank page.
it's an issue of my install ?
thanks for help.

could you please help ?

Same issue here. (You can explore on https://dotabap.org/ for now.)
Seems to me like dotabap.org was recently changed to enforce https. The json-request gets redirected from http to https and fails with a CORS error.

yeah, I moved everything to HTTPS
@jmoutreux try again, latest version, fix by @flaiker merged

Thanks it work now.
Best regards
Jérôme Moutreux
SAP Development consultant
Mobile: +33 6 43 47 92 09
[LOGO-TEAMWORK-SIGNATURE-version Outlook]
FRANCE : 1, Rue Jean Carmet - CS 80004 - 69808 Saint-Priest
Phone: +33 (0)4 78 63 63 25
www.teamwork.nethttp://www.teamwork.net/
Print this message only if this is essential.
De : Lars Hvam<EMAIL_ADDRESS>Envoyé : vendredi 9 novembre 2018 04:15
À : larshp/abapGit<EMAIL_ADDRESS>Cc : MOUTREUX Jerome<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Objet : Re: [larshp/abapGit] Explore doesn' work for me. (#2069)
yeah, I moved everything to HTTPS
@jmoutreuxhttps://github.com/jmoutreux try again, latest version, fix by @flaikerhttps://github.com/flaiker merged
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/larshp/abapGit/issues/2069#issuecomment-437236214, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AqzLmZTKN4umRvq94mE8oAg7ysS1WJ35ks5utPMlgaJpZM4YVVZL.


Upload a schematic of the database

A schematic showing all the relationships would be a useful reference. We have an almost entirely complete one available https://github.com/MIT-LCP/mimic-code/issues/183

The one you have here is nice but it would be much more clear if you could rearrange your tables to create something like this

Certainly that view of the schema is clearer. This task is fairly low priority for us though so we will leave it as is - if you are able to create a prettier diagram we'd welcome a PR!

Schema spy availability solves this issue. https://mimic.physionet.org/mimicdata/schema/


[UI] french translation too formal :(

taking a closer look at latest french translation of rancher.
i've spotted many translations which are too formal or even worse google translating.
that's somehow the weak spot of crowdin i think.
How can we update the french translation to make it more accurate and less formal and make these changes persistent ?
i'd be happy to give another hand on this matter :)

Since we don't generally speak the languages and they come from a smallish number of people we more or less just accept anything put in.  Changes made on CrowdIn are periodically synched with the UI repo.  With enough people watching a particular language I think there are voting and trusted maintainers and other sorts of tools that become useful.
So you can make changes and they will make their way in, usually at the beginning and end of a release cycle (though the current one is particularly long...).

@xinity closing this out, if you have any more questions please do reopen.


Is it possible to generate ed_update::handle_new_char from a grammer representation of roc?

Inspiration: https://emacsconf.org/2021/talks/structural/

Editor is no longer being worked on, and has been removed from the repo. Closing.


Add option to reveal hidden widget while scrolling back

I tried to use the widget on AppBar to implement a common behavior, when the appbar hides when you scroll down, and shows again when you scroll up. But as I can see widget reveal triggers only when you are close to zero scroll offset. It would be nice to have an option when revealing also triggers while scrolling back.

@theiskaa
I'm attaching wanted behavior compared to your widget behavior

So .. you want to an option to customise the scrolling visibility algorithm? Am I right?

If, so. I'll implement asap and assign you as reviewer. @radibobovich

So .. you want to an option to customise the scrolling visibility algorithm? Am I right?

That's right, thank you for quick response! Would love to review it once you get it done.

@radibobovich good evening/morning, I'm finished implementation, and let's see if we understood each other correctly. I'll write example implementation, in the PR itself -> #27.

@radibobovich I'll wait for your comment of approval, and then merge to develop.

Hey @theiskaa, I checked the feature and made use of it, thank you. This is the function I passed to the widget to implement the desired behavior:
double customHidableVisibility(ScrollPosition position, double previousOffset,
    double currentOffset, double currentVisibility) {
  const double deltaFactor = 0.08;

  /// When user scrolls down
  if (position.userScrollDirection == ScrollDirection.reverse) {
    return (currentVisibility - deltaFactor).clamp(0, 1);
  }

  /// When user scrolls up
  if (position.userScrollDirection == ScrollDirection.forward) {
    return (currentVisibility + deltaFactor).clamp(0, 1);
  }
  return currentVisibility;
}

I also checked the code you provided in your first reply more carefully:

/// We set visibility to zero (which probably could be optional, able to customize from widget)
if (position.axisDirection == AxisDirection.down && position.extentAfter == 0.0) {
  if (visibilityNotifier.value == 0.0) return;
  visibilityNotifier.value = 0.0;
  return;
}

/// We set visibility to one (which probably could be optional, able to customize from widget)
if (position.axisDirection == AxisDirection.up && position.extentBefore == 0.0) {
  if (visibilityNotifier.value == 1.0) return;
  visibilityNotifier.value = 1.0;
  return;
}

And I found out that the last condition probably doesn't work as intended (it is never true), because position.axisDirection doesn't represent the direction user scrolls in (check it here), so it will always say AxisDirection.down, you probably want to use position.userScrollDirection instead.
Also, while I was trying to implement the "reveal-on-scroll-back" feature, I was confused by the previousOffset because it actually represents the offset only while scroll offset is less than preferredSize, then it just clamps to it. Looks like intended though, but I couldn't make use of it while implementing back scroll revealing logic and thus used naive substraction instead.

I'm happy to hear that!
regarding the original algorithm, not sure actually. I'll test and let you know. I loved your customised implementation, much simpler than ours. Also will think about replacement, since it will make no difference, after making it able to customise the algorithm itself.
Thanks again!

I finally got time to test and clear up things.
I really appreciate your suggestions, and yeah we're at here. I replaced the algorithm with yours, now it has default implementation like that, plus I've provided the extra deltaFactor field to Hidable widget itself. So you can decide the strength of functioning without re-writing the algorithm itself.
Tho, I also removed some unused fields and variables from controller, I've removed the size, current/previous offsets from the HidableVisibility function type and also from controller. We don't need 'em.
I'll merge now, If you think that suits your needs well.
Check the PR, for updated example.


Particular Font Gets Rendered Incorrectly

VSCode Version: 1.23.1
OS Version: Ubuntu 16.04 LTS

The font Monaco gets rendered incorrectly. It is very thin in VS Code, but normal in other text editors. Even changing the fontWeight doesn't fix the problem. This problem doesn't occur with other fonts.
This is not the same as #35675, because the font isn't distorted, it looks like it's lower resolution somehow.
Here's a picture of VS Code with Sublime Text side-by-side for comparison.

Steps to Reproduce:

Open user settings
Set "editor.fontFamily": "Monaco"

Does this issue occur when all extensions are disabled?: Yes

Font rendering in the editor is handled by Chromium. Please try to reproduce in Chromium and file an issue against them.

I'm not sure how to do that.
Should I just install chromium and change system font to Monaco to see if the problem exists there too?

@krtk6160 Here is a website that uses Monaco:
http://jsbin.com/saquhuyeti/edit?html,js,output

Since we use the same rendering engine, I expect Chromium to render the same text in the same buggy way. If you find, for example, that Firefox renders Monaco correctly, you can create an issue against Chromium...

@alexandrudima I tried to reproduce the problem in chromium, but was unable to do so.
I installed chromium version 58.0.3029.110-0ubuntu<IP_ADDRESS>1 and it rendered the font correctly. I also tried it on the latest stable chrome and it works perfectly.

@krtk6160 Can you please try a recent insiders version where we have upgraded to Chromium 61.0.3163.100 ?
https://code.visualstudio.com/insiders/

Indeed, it looks like today's linux x64 build didn't succeed. Most likely it will be out tomorrow.

I tried the latest 1.25 Code - Insiders build, and the problem remains. I even reinstalled the font, but even that didn't help. Does this rule out chromium as the source of the problem?

@alexandrudima Something interesting just happened. I changed the font size after reinstalling the font, and it works fine now. The problem occurs when the size is set to 14, but not when it's set to 14.5, 15, or even 13.

I think I'll just keep the size to 14.5 for now and hope the problem gets fixed somehow in future builds.
Thank you for your time.

That is absolutely crazy! Perhaps there is some persistent font cache somewhere for that font size that needs to be reset?

@alexandrudima I was able to reproduce the problem in Firefox, Chrome (version 66), and Chromium (versio 58) as well now, and it only exists when the font size is 14px. I only know the fc-cache -fv command, which I've already run and it doesn't seem to fix the problem.

@alexandrudima I was finally able to fix the issue. I had to disable bitmap fonts for Monaco by editing the ~/.fonts.conf file.
This is what I added to the file that fixed the issue:
<match target="font">   <test qual="any" name="family">     <string>Monaco</string>   </test>   <edit name="embeddedbitmap">     <bool>false</bool>   </edit> </match>
I found the solution here.

Thanks for following up and documenting the fix! :heart:


ion-input with ion-label; keyboard closes

When you have an ion-input directive with an ion-label inside and you touch the ion-label the software keyboard will appear but disappear promptly after. This only happens if you touch the area with the input-label. This bug is reproducible on both Android and iOS.
I'm not sure precisely when this problem was introduced. I am on 'Copenhagen' right now. Sometime after 'Amsterdam'.

Is this with Ionic v1 or Ionic v2? What does your markup look like?

Ionic v1.
Markup:
<ion-input class="item item-input">
					<ion-label>
						<translate>PIN-kode</translate>
					</ion-label>
					<input name="pin" type="tel" ng-model="pin.pin">
				</ion-input>

same here, tested on Android

Found what might be the issue: https://github.com/rubenv/angular-gettext/issues/191

So ion-label does not allow html at all, for focus to work, only text, that seems like an error

Yes I'm using https://angular-gettext.rocketeer.be for translations. Here the translate-directive is used.
That does seem like an bug.

easy fix is just to put "pointer-events: none;" on the inner elements.
https://css-tricks.com/almanac/properties/p/pointer-events/

OK. That is great! Thanks!
But of course this needs to be fix within ionic.

Just to close this off. Using the {{ 'text-to-translate' | translate }} I was able to overcome this problem. I'm not classifying this as a bug in ionic.


Seamless gradual upgrade from react-redux

Related to #3
Edit: This will not work because react-redux injects state but redux-react-hooks injects store into Context
If redux is same, then reusing context should be as easy as injecting foreign context object:
https://github.com/facebookincubator/redux-react-hook/blob/3a86bc02ca81def12e0240c986d56ee1da5fc792/src/create.ts#L21-L30

new code (will not work)
export function create<
  TState,
  TAction extends Action,
  TStore extends Store<TState, TAction>
>(customContext?: React.ContextType): {
  StoreContext: React.Context<TStore | null>;
  useMappedState: <TResult>(mapState: (state: TState) => TResult) => TResult;
  useDispatch: () => Dispatch<TAction>;
} {
  const StoreContext = customContext || createContext<TStore | null>(null);

  // ...
}

This may break if context shapes from react-redux and redux-react-hook are different — they are different

In our app we made a StoreContext.react.js component that applies both the react-redux provider and the redux-react-hook provider, and just use that everywhere. As you pointed out, the shapes are not the same, so this won't really work.


SigNoz Integration

Add an integration with SigNoz. The integration, in the initial phase, should do the following:

Read alerts information via the API
Read logs via the API

Logs API - https://signoz.io/docs/logs-management/logs-api/overview/#api-endpoint
Alerts API - maybe instead of Signoz directly, we can integrate with one of the integrations it has like pagerduty?

Also, for Logs API since community edition does not have apikey support, if we can get support via user credentials that would make integration more widely used.

Thanks for sharing the references to SigNoz docs :) @nitin302
About the alerts API - since we're already integrated with PagerDuty/Opsgenie, it'd indeed be simpler to stick with that. Namely, rely on SigNoz integration with them and integrate with PD/Opsgenie.
About the logs API - can you please elaborate on the API key v.s basic credentials? Are you referring to SigNoz community edition?
BTW - If you want to take a stab at that and implement and initial version of SigNoz integration, that would be amazing :) I can help you on that.

Yes, the SigNoz community edition does not have API key support - https://signoz.io/pricing/, but typically user credentials (basic) works.
I might be interested in contributing, if you can share steps... I can create a new PR.

@nitin302 sure, I can provide some guidance :) Thanks for showing interest.
For the SigNoz integration, I recommend following the structure we used for Coralogix:
Database: Add a new record in the vendors table by updating packages/db/src/models/vendor.ts and including SigNoz in the vendorsData array.
UI Card: Copy the Coralogix UI card in services/dashboard/src/components/Connection/integrations/basic, adjust it for SigNoz, and configure the necessary fields like username, password, and host.
Sub-Agent: Create a sub-agent in Langchain based on the Coralogix structure, modifying it for SigNoz. Currently, the Coralogix agent fetches logs, so we’ll need to decide what SigNoz should do.
(Optional) Processor: If needed, create a processor for Opsgenie incidents to handle SigNoz-related root-cause-analysis flows.
The main work is to create the sub-agent. This is the trickiest part, and requires some prompt engineering.
The agent needs to understand the underlying topology, log structures, service names, etc. We did some work on Coralogix and now the agent is in a pretty decent place, but it's still not perfect.
Lemme know what you think about it. I'd be happy to hop on a call with you if you want to discuss further :)

@david1542 @nitin302 I am going to attempt Signoz integration. We will see how it turns out.

@vikash-fourkites Gotcha, thanks for good spirit of contributing 😊
I know @nitin302 has started working on that. @nitin302 Do you mind sharing your progress maybe?

I wasn't able to make much progress but i've provided context to @vikash-fourkites, thanks.

@nitin302 awesome, thanks a lot.
@vikash-fourkites if you need any help, feel free to reach out to me here or on Slack :)

Hi @david1542, I noticed there haven't been any updates on the issue since September 18. I'm interested in contributing and would love to take it on if possible. Could you also send me an invitation to join the Slack? Thanks!

Hi @Subash-Mohan :) Thanks for reaching out! We're currently working on it ourselves. You can join the Slack community using this link.
If you want, there are some other issues that you can take.


Only enable the mempool crawler after synchronization reaches the chain tip

Motivation

The mempool crawler should only run after the chain is synchronized.
Solution

Update the mempool::Crawler to use the SyncStatus helper type so that the crawler only runs if the synchronizer is likely at the chain tip (i.e., it has finished catching up to the current network status).
Review

@teor2345 has been following the development of the crawler.
Reviewer Checklist

[ ] Code implements Specs and Designs
[ ] Tests for Expected Behaviour
[ ] Tests for Errors

Follow Up Work

I updated the original mempool crawler test to now be a proptest so that it can consider scenarios where the crawler pauses and resumes.


Route doesn't exist but cf cli thinks it does

High five for submitting a report!
To avoid additional back and forth, please include the following details as relevant:
Command
 2016-07-08 11:45:02 ☆  |2.3.0| lawry in ~/workspace/metrics-data/src/metrics-data/logqueue/build
±  |master U:2 ?:4 ✗| → cf check-route elasticsearch-logqueue metrics12.pcf-metrics.com
Checking for route...
OK
Route elasticsearch-logqueue.metrics12.pcf-metrics.com does exist

 2016-07-08 11:45:06 ☆  |2.3.0| lawry in ~/workspace/metrics-data/src/metrics-data/logqueue/build
±  |master U:2 ?:4 ✗| → cf unmap-route elasticsearch-logqueue metrics12.pcf-metrics.com
FAILED
Route  not found

e.g. cf org-users
CLI Version
 2016-07-08 11:45:09 ☆  |2.3.0| lawry in ~/workspace/metrics-data/src/metrics-data/logqueue/build
±  |master U:2 ?:4 ✗| → cf -v
cf version 6.17.1+2ca614d-2016-05-02

e.g. 6.13.0-dfba612 (visit the releases page and confirm it's reproducible on the latest version)
Error
See above
Stack Trace
See above
CC API Endpoint Version
 2016-07-08 11:46:06 ☆  |2.3.0| lawry in ~/workspace/metrics-data/src/metrics-data/logqueue/build
±  |master U:2 ?:4 ✗| → cf api
API endpoint: https://api.acceptance.pcf-metrics.com (API version: 2.56.0)

see cf api output
CF Trace
See above
For bugs related to HTTP requests, please run the command with env var CF_TRACE=true
Platform Details
e.g. Mac OS X 10.11, Windows 8.1 64-bit, Ubuntu 14.04.3 64-bit
Shell
iTerm
e.g. Terminal, iTerm, Powershell, Cygwin, gnome-terminal, terminator

Hi @jadekler
Can you share with us the trace output when running those two commands?
Cheers,
Dies Koper
CF CLI PM

Closing. Feel free to reopen or submit a new issue if it reoccurs and you can share a trace with us.

For the record, this looks like a user error.
The syntax for cf check-route is:
cf check-route HOST DOMAIN [--path PATH]

whereas that for cf unmap-route is:
cf unmap-route APP_NAME DOMAIN [--hostname HOSTNAME] [--path PATH]

So, in this issue, cf check-route was issued with:
HOST=elasticsearch-logqueue
DOMAIN=metrics12.pcf-metrics.com
(PATH not specified)

whereas cf unmap-route was issued with:
APP_NAME=elasticsearch-logqueue
DOMAIN=metrics12.pcf-metrics.com
(HOSTNAME and PATH not specified)

but HOSTNAME should have been specified as elasticsearch-logqueue.
Note the extra space in the error message Route  not found, which presumably means that HOSTNAME is not defaulted. (It might have been helpful for cf to default HOSTNAME to APP_NAME to avoid this kind of mistake, but I'm not sure that could be retrofitted without breaking backwards compatibility.)

Thanks @glyn, for setting the record straight :)
I'm thinking of changing all route commands to take one argument ROUTE, like how we support it in app manifest files, but it is indeed the retro-fitting that makes it complicated.


Phantomjs crashed after >10 seconds

Hi,
It seems like phantomjs crashing after 10 second. Is it possible to increase the timeout limit ?
Thanks & Regards,
Kamal Patel.

This is actually configurable within the Karma Configuration: browserNoActivityTimeout.

Thanks for you answer.


On premise deployment of QHub

This is an issue to track progress on making QHub deployable on existing kubernetes clusters and on-prem deployments.

@costrouc I am a little bit confused about this ticket.
Since this is on-prem shouldn't it be part of QHub-HPC? Or is the Epic describing two different things?
Can you elaborate, please ❔

@costrouc I am a little bit confused about this ticket.
Since this is on-prem shouldn't it be part of QHub-HPC? Or is the Epic describing two different things?
Can you elaborate, please ❔

In the on-prem hpc faq, it says

We have eventual plans for supporting existing on-prem kubernetes deployments in issue #216. HPC still has many advantages over kubernetes:

simplicity as it following a more traditional linux and systemd making it easier to diagnose errors
can deployed to take advantage of pre-provisioned existing infrastructure
TODO still possible to support containerized workflows
can fully take advantage of HPC infrastructure in on-prem datacenter

Resolved by PR https://github.com/Quansight/qhub-cloud/pull/287


Update datacache.ts

Description

A clear and concise summary of the change and which issue (if any) it fixes. Should also include relevant motivation and context.
Resolved or fixed issue:
Affirmation

[ ] My code follows the CONTRIBUTING.md guidelines

Checkmarx One – Scan Summary & Details – 98901a81-5722-4c91-9026-76fa4767bd1b
New Issues

Severity
Issue
Source File / Package
Checkmarx Insight

CVE-2023-30547
Npm-vm2-3.9.11
Vulnerable Package

Deserialization_of_Untrusted_Data
/routes/b2bOrder.ts: 18
Attack Vector

Unsafe_Object_Binding
/routes/order.ts: 155
Attack Vector

Unsafe_Object_Binding
/routes/login.ts: 36
Attack Vector

Unsafe_Object_Binding
/routes/createProductReviews.ts: 19
Attack Vector

Information_Exposure_Through_an_Error_Message
/routes/login.ts: 80
Attack Vector

Information_Exposure_Through_an_Error_Message
/routes/b2bOrder.ts: 24
Attack Vector

Information_Exposure_Through_an_Error_Message
/routes/profileImageFileUpload.ts: 40
Attack Vector

Information_Exposure_Through_an_Error_Message
/routes/profileImageUrlUpload.ts: 25
Attack Vector

Information_Exposure_Through_an_Error_Message
/routes/resetPassword.ts: 24
Attack Vector

Information_Exposure_Through_an_Error_Message
/routes/resetPassword.ts: 50
Attack Vector

Information_Exposure_Through_an_Error_Message
/routes/order.ts: 74
Attack Vector

Information_Exposure_Through_an_Error_Message
/routes/order.ts: 173
Attack Vector

Information_Exposure_Through_an_Error_Message
/routes/order.ts: 77
Attack Vector

Information_Exposure_Through_an_Error_Message
/routes/profileImageUrlUpload.ts: 33
Attack Vector

Information_Exposure_Through_an_Error_Message
/routes/order.ts: 171
Attack Vector

Information_Exposure_Through_an_Error_Message
/routes/b2bOrder.ts: 28
Attack Vector

Information_Exposure_Through_an_Error_Message
/routes/fileServer.ts: 22
Attack Vector

Information_Exposure_Through_an_Error_Message
/routes/securityQuestion.ts: 23
Attack Vector

Information_Exposure_Through_an_Error_Message
/routes/profileImageFileUpload.ts: 46
Attack Vector

Information_Exposure_Through_an_Error_Message
/routes/changePassword.ts: 45
Attack Vector

Information_Exposure_Through_an_Error_Message
/routes/order.ts: 147
Attack Vector

Information_Exposure_Through_an_Error_Message
/routes/order.ts: 143
Attack Vector

Information_Exposure_Through_an_Error_Message
/routes/coupon.ts: 31
Attack Vector

Information_Exposure_Through_an_Error_Message
/routes/coupon.ts: 29
Attack Vector

Information_Exposure_Through_an_Error_Message
/routes/login.ts: 54
Attack Vector

Information_Exposure_Through_an_Error_Message
/routes/search.ts: 69
Attack Vector

Information_Exposure_Through_an_Error_Message
/routes/order.ts: 150
Attack Vector

Information_Exposure_Through_an_Error_Message
/routes/changePassword.ts: 40
Attack Vector

Information_Exposure_Through_an_Error_Message
/routes/profileImageUrlUpload.ts: 32
Attack Vector

Information_Exposure_Through_an_Error_Message
/routes/resetPassword.ts: 41
Attack Vector

Information_Exposure_Through_an_Error_Message
/routes/redirect.ts: 22
Attack Vector

Information_Exposure_Through_an_Error_Message
/routes/securityQuestion.ts: 29
Attack Vector

Information_Exposure_Through_an_Error_Message
/routes/resetPassword.ts: 44
Attack Vector

Information_Exposure_Through_an_Error_Message
/routes/profileImageFileUpload.ts: 50
Attack Vector

Information_Exposure_Through_an_Error_Message
/routes/profileImageUrlUpload.ts: 36
Attack Vector

Information_Exposure_Through_an_Error_Message
/routes/coupon.ts: 25
Attack Vector

Information_Exposure_Through_an_Error_Message
/routes/profileImageFileUpload.ts: 23
Attack Vector

Information_Exposure_Through_an_Error_Message
/routes/authenticatedUsers.ts: 25
Attack Vector

Information_Exposure_Through_an_Error_Message
/routes/search.ts: 41
Attack Vector

Information_Exposure_Through_an_Error_Message
/routes/changePassword.ts: 36
Attack Vector

Information_Exposure_Through_an_Error_Message
/routes/basket.ts: 32
Attack Vector

Information_Exposure_Through_an_Error_Message
/routes/quarantineServer.ts: 17
Attack Vector

Information_Exposure_Through_an_Error_Message
/routes/keyServer.ts: 17
Attack Vector

Information_Exposure_Through_an_Error_Message
/routes/logfileServer.ts: 17
Attack Vector


Make business and industry field optional when creating client account

Adding this to backlog.
From PO comment, we should consider client account as your everyday customer. So business and industry field needs to be optional.
https://github.com/kevin-ghannoum/soen490/issues/72
#72

This issue is a duplicate of #117. Hence, it is being closed.


Security radio does not work when close to a handheld radio

Description

https://github.com/user-attachments/assets/df671b26-0b41-4f69-80a2-20d241277822
Reproduction

Screenshots

Additional context

is it only with security radio?


.Net Core Api List Not Include System.Http.Net

I found in the api list page https://docs.microsoft.com/en-us/dotnet/api/?view=netcore-3.1, it does not include System.Net.Http Namespace within the list, however if I search a class in that namespace like HttpClient using the search bar at the top of the page, I am able to reach the namespace's document page https://docs.microsoft.com/en-us/dotnet/api/system.net.http.httpclient?view=netcore-3.1. Is this a bug ?

Thanks for the feedback. This is related to #3169 and I've opened an internal bug to track it: https://dev.azure.com/ceapex/Engineering/_queries/edit/325235/?triage=true.


process crashes while getting room statistics

The mapping field is missing stream

{
    "type": "stream",
    "id": "DEPRECATED_SPA_Gr9TnRmPoMcx|TR_AMRvi64HtRvnuT",
    "timestamp":<PHONE_NUMBER>215000,
    "streamIdentifier": "PA_Gr9TnRmPoMcx|TR_AMRvi64HtRvnuT",
    "trackIds": [
        "DEPRECATED_TI1"
    ]
}

I submitted a PR to fix this


libpng doesn't work after update

Hi.
After ps2dev update it turned that libpng doesnt work in my project.
To be 100% sure that is not my fault, I tried to reproduce this bug in gsKit and problem is also there.
I was wondering where to put this PR, but I think the problem is more with the toolchain than with libpng or gsKit itself.
Which line cause the problem:
FILE *file = fopen("host:files/test.png", "rb");
png_structp png_ptr = png_create_read_struct(PNG_LIBPNG_VER_STRING, NULL, NULL, NULL); // <-- Problem is here
png_infop info_ptr = png_create_info_struct(png_ptr);
...

Repro steps:

GSKITSRC=cloned-ps2dev/build/gskit
cd $GSKITSRC
Uncomment LIBPNG and ZLIB in Makefile.pref
make
Open ./examples/png-texture/textures.elf in PCSX2

Code snippet:
#include <png.h>

// CFlags: -DHAVE_LIBPNG -DHAVE_ZLIB
// LDFlags: -L$(PS2SDK)/ports/lib
// EELibs: -lpng -lz

FILE *file = fopen("host:files/test.png", "rb");
assertMsg(file != NULL, "Failed to open .png file!");
unsigned char header[8];
fread(header, 1, 8, file);
assertMsg(!png_sig_cmp(header, 0, 8), "File was not recognized as a PNG file");
png_structp png_ptr = png_create_read_struct(PNG_LIBPNG_VER_STRING, NULL, NULL, NULL);
assertMsg(png_ptr, "PNG read struct init failed!");
png_infop info_ptr = png_create_info_struct(png_ptr);
assertMsg(info_ptr, "PNG info struct init failed!");
assertMsg(!setjmp(png_jmpbuf(png_ptr)), "PNG reader fatal error!");
png_read_end(png_ptr, NULL);
png_destroy_read_struct(&png_ptr, &info_ptr, (png_infopp)NULL);
fclose(file);

Thanks

Hello
I think you're wrong, there is a cyclic dependency between gskit a ps2sdk-ports
If you want to have gskit with png support, you need:

To compile first gskit without png-support
Then ps2sdk-ports (which compiles libpng)
Finally you need to compile again gskit enabling the png support

Actually to demonstrate that you're wrong
You can download artifacts from gsKit
https://github.com/ps2dev/gsKit/actions/runs/854668541
Here you have it, running locally:

Quite important note: I saw you're using PCSX2, for this example is fine, but host unit doesn't contain all the I/O functions implemented in PCSX2, as readdir, stat.... so I have done a PR which is waiting to be merged
https://github.com/PCSX2/pcsx2/pull/4372
Please, test it and close the issue.
Thanks

By the way, the same issue was already replied
https://github.com/ps2dev/ps2dev/issues/28

Worked. Thanks man!


Eventhubs: fix SpotBugs P2

This PR containes the fix for P2 errors introduced by Event Hubs.
Issue: Clean Up EventHubs P2 and P3 SpotBugs #3236

Can one of the admins verify this patch?


[mobile] Add HTTP/3 related API in starting_envoy instruction

Commit Message: Add HTTP/3 related API in starting_envoy instruction
Additional Description: enableHttp3() has long been available. addQuicHint() was recently added in https://github.com/envoyproxy/envoy/pull/29224
Risk Level: Low
Testing: n/a
Docs Changes: Envoy Mobile api doc change
Release Notes: n/a
Platform Specific Features: mobile only

/assign @alyssawilk

/retest

is CI wedged?  maybe a main merge would help?

/retest


Intent.reset_context returns False

Hello,
It seems that there is a bug on dialogflow python client api dialogflow_v2.IntentsClient.Intent.reset_context parameter. Python client api returns always false for reset_contexts. Please find the details below. I need to get the end of conversation value for Intents.
Environment details

MacOS Catalina v10.15.7 and Docker Container
Python version and virtual environment information: Python 3.8.5
dialogflow version:
Name: dialogflow
Version: 0.7.2
Summary: UNKNOWN
Home-page: https://github.com/dialogflow/dialogflow-python-client-v2
Author: Google LLC
Author-email<EMAIL_ADDRESS>License: Apache 2.0
Location: /Users/mac/Desktop/Profilo Product Mix Optimization/chatbot_platform/venv/lib/python3.8/site-packages
Requires: google-api-core
Required-by:

Steps to reproduce

Initialize your dialogflow agent, GCP Keys and local dev environment(ref)
2.Create a Virtual Environment and install dialogflow python client library
3.Create 2 intents on dialogflow UI(ES) and set "end of conversation" switch to True under responses section for one of the Intent
4.Send an API request to list the intents using the code example below and you will see "reset_contexts" parameter is false for both Intents

Code example
import dialogflow
import json
project_id = "ENTER_YOUR_PROJECT_ID"  # You can find your project ID on your GCP Key JSON File or on GCP Console

def intents_list(project_id, language_code):

    intents_client = dialogflow.IntentsClient()
    intents = intents_client.list_intents(parent='projects/' + project_id + '/agent',
                                          language_code=language_code,
                                          intent_view='INTENT_VIEW_FULL')

    intent_dictlist = []
    for idx, intent in enumerate(intents):

        intent_dictlist.append({
            "name": intent.name,
            "display_name": intent.display_name,
            "reset_contexts": intent.reset_contexts
        })

    return intent_dictlist

print(
        json.dumps(
            intents_list(project_id, "en"),
            indent=4
        )
     )

Stack trace
[
    {
        "name": "projects/PROJECT_ID/agent/intents/UNIQUE_INTENT_NAME",
        "display_name": "Intent1",
        "reset_contexts": false
    },
    {
        "name": "projects/PROJECT_ID/agent/intents/UNIQUE_INTENT_NAME",
        "display_name": "Intent2",
        "reset_contexts": false  # true expected here
    }
]

Note: This code has been tested on multiple environments and multiple Python3 versions including a Docker container.
Thanks!

Any workarounds?

@gitefeb

I was able to create intent with reset_contexts set to true, list_intents it came out to be true.

@munkhuushmgl Did you explicitly follow step 3 in "steps to reproduce" section? How did you set the reset contexts to true, using programmatic access or on the Dialogflow ES UI? I just deleted all the intents inside one of my agent and tried this again . I was able reproduce the bug.

@gitefeb I did not use the Dialogflow UI, I used programmatic approach using create_intent from client library.
If thats the problem with Dialogflow UI, I would create a bug on here
https://b.corp.google.com/issues/new?component=311383&description=https:%2F%2Fcloud.google.com%2Fdialogflow%2Fcx%2Fdocs%3Fhl%3Den

@munkhuushmgl Thanks for quick response. Unfortunately I am not a google employee so I dont think I have permission to visit the link you just shared :) I would appreciate if you could inform the relevant departments or share an email to get in touch.
We have also tried updating the intents using update intents method on client library which did not work, but will give it an another try and let you know.
Thanks.

@gitefeb If you go to documentation page, at the right bottom there is bug icon button (using that you can report a bug)


Is it possible to reconstruct the Cartesian sampled image instead of gold-angle radial sampling?

what i think is using this TDDIP methods for simulated sampling pattern (possible not for clinical application), such as the pseudo-radial sampling (Cartesian version of radom radial sampling) as in k-t SLR method. If it is, it will be very fast to train the model and test the efficacy of the model due to the fact that Cartesian FFT can replace the time-consuming NUFFT.

I implement this idea by myself, github link
hope it is right


update create_hf_reader to correctly pad sequences of variable lengths

What does this PR do? Please describe:
This PR extends create_hf_reader to pad sequences of variable lengths.
Changes:

Changed data = dataset.to_list() to data = dataset because the collater doesn't pad python lists.
Updated builder.map(Collater()) to builder.map(Collater(pad_value=pad_value)) because the collater will only stack sequences unless a pad_value is specified (default is None).
Add pad_value argument to the function

Does your PR introduce any breaking changes? If yes, please list them:
None.
Check list:

[X] Was the content of this PR discussed and approved via a GitHub issue? (no need for typos or documentation improvements)
[X] Did you read the contributor guideline?
[X] Did you make sure that your PR does only one thing instead of bundling different changes together?
[X] Did you make sure to update the documentation with your changes? (if necessary)
[X] Did you write any new necessary tests?
[X] Did you verify new and existing tests pass locally with your changes?
[x] Did you update the CHANGELOG? (no need for typos, documentation, or minor internal changes)

Also LGTM, thanks Ahmed !


Displaying progress percentages, and not just a progress bar

Describe the feature
I can't clearly understand what the progress is from the bar, so I prefer percentages

Added in latest release


download page timeout

I wanted to get BLT to load in HUDs, but every time I try to download it the page times out.

Please post the download link you are trying to use, or the name of the mod you are trying to update. Otherwise we cannot look into this.

I just tried it again and it works now.  It was the download link for BLT, but it work now so yay.


Add support for negating only non-alpha channels

This is based on comment by @lovell in https://github.com/lovell/sharp/issues/1035#issuecomment-877003420 - it changes the signature of the negate method to take either a boolean or a plain object with an alpha boolean property. Setting it to false will negate all channels but the alpha, which fixes #1035

Coverage remained the same at 100.0% when pulling 35bf4072ed327075367cde3f0eb01b1b5b6e6da2 on rexxars:feat/negate-non-alpha into 7467fa8b50d280780dfa9fe9a7248deaa499c7a1 on lovell:master.

Landed via commit https://github.com/lovell/sharp/commit/b7ddbe71f724bf16aa2a11f30a9abba9c8b3c0b0, takk!


The fully-managed rewrite

Note: this is a meta tracking issue. Features will be divided into smaller issues.
Here's a diagram of the architecture we plan to rewrite HaaS to use:

Tasks:

[x] Freeze the current (old) production instance so that customers keep uptime #202

Further discussion in the hack-as-a-service org :)


Support for Github actions artifacts?

Hi,
just wanting to test latest code from master supporting latest cuda runtime on Windows (https://github.com/vosen/ZLUDA/commit/5cd9a5fbc4b92c58a1db6f205559e690af30481b)..
Lazy to build project just to do a quick test and seems github actions support currently lacks support for artifacts..
https://github.com/vosen/ZLUDA/actions/runs/472515955
I'm mainly interested on seeing support for Windows builds artifacts (so having the resulting zluda.dll but better if Linux zluda.so is built also)..
thanks..

Yes, this is important. First, I want to have a more robust way to run Windows applications with ZLUDA (currently it requires too much manual intervention) and finish up GeekBench support. Then I'll be looking into improving CI to have a Windows build and to upload artifacts.

Yes, this is important. First, I want to have a more robust way to run Windows applications with ZLUDA (currently it requires too much manual intervention) and finish up GeekBench support. Then I'll be looking into improving CI to have a Windows build and to upload artifacts.

nice to hear! thanks..

nice to hear! thanks..

Should work with #39

thanks!
will test soon and report..
but upon quick test using zluda_with tool on Windows I get:
zluda_with -- CUDA-Z-0.11.259-SVN-64bit.exe
I get error missing zluda_redirect.dll in the Windows artifact..

EDIT: in addition to zluda_with zluda_redirect.dll issue I also see no 32bit Windows nvcuda.dll present in artifact..
I was going to test and old CUDA-Z:
CUDA-Z-0.10.251-32bit.exe

Ooops, zluda_redirect.dll is now included.
As for 32bits there's no simple way forward. ZLUDA relies on Level 0 API and it's 64bit only with no plans for 32bit. There's a potential path to 32bit, but it's not simple and has some limitations (more discussion here: #31).

thanks..
yes now CUDA-Z works.. so closing issue..
didn't know L0 was 64 bit only..


Remove source-built dir from tarball in favor of prev source-built archive

Remove copying of assets to source-built dir in tarball
On initial tarball build, lookup boostrap versions of arcade, sourcelink
and DotNetHost from previously source-built archive and use those
versions when building arcade, sourcelink and corefx.

Fixes #1358

This work is in 3.1 with #1397.  Closing this for 3.0.


Reload background service worker without reloading entire extension

From Chrome's debugging article: https://developer.chrome.com/docs/extensions/mv3/tut_debugging/

Also, if you have made changes to the service worker code, you can use the Update button and skipWaiting to apply the changes immediately.

If there's a way to do this programatically, that would be great. It means extension pages wouldn't have to be closed when changing the background.
Maybe we have access to these ServiceWorker APIs inside the background?

https://developer.mozilla.org/en-US/docs/Web/API/ServiceWorkerRegistration/update
https://developer.mozilla.org/en-US/docs/Web/API/ServiceWorkerGlobalScope/skipWaiting

By running the following code in the top-level scope of the background service worker, I was able to access the two functions necessary to update the service worker without reloading the entire extension:
addEventListener('install', (event) => {
  const sw = event.target as ServiceWorkerGlobalScope;
  const { registration, skipWaiting } = sw;
  const update = registration.update;
  console.log({ skipWaiting, update });
  globalThis.reloadBackground = async () => {
    await update();
    await skipWaiting();
  };
});

Extension service workers can listen to the standard web worker install event, from which we're able to access the ServiceWorkerGlobalScope and ServiceWorkerRegistration.
Unfortunately, calling the reloadBackground function from the dev console results in an error:

Here's a minimal extension demoing this behavior: minimal-example.zip

Conversation on the chrome google groups: https://groups.google.com/a/chromium.org/g/chromium-extensions/c/Z-H3IkRODpQ

Alright, working example:
// demo/src/entrypoints/background.ts
export default defineBackground(() => {
  // ...

  const serviceWorker = globalThis as unknown as ServiceWorkerGlobalScope;
  globalThis.addEventListener('install', () => {
    void serviceWorker.skipWaiting();
  });
  // @ts-expect-error
  globalThis.updateSelf = async () => {
    await serviceWorker.registration.update();
  };
});

Then you can call updateSelf from the service worker's dev console after making a change to demo/.output/chrome-mv3/background.js

And nevermind, will break in Chrome 117...
https://chromiumdash.appspot.com/commit/b8b06b6b8071b96e3f76858d662d8991ba3b3374

Can confirm that you can no longer update the service worker to get the latest code.


Fix on send_message

Fixed ordering on lines of code for setting dummy draft when submitting file attachments. This fixed problem that no message was sent when sending attachments.

thanks


See if there's a way to detect if fonts are done loading and have loaded successfully

Not sure if this is possible, but would be really nice for better debugging and confidence in reliability.

@g-a-v-i-n  This lib + blog post about it

pursuing this because it explicitly says that it supports canvas whereas the other one didn't

@g-a-v-i-n getting this error while trying to test my code on branch feat/88

it looks like you were messing around with the sigil stuff but unsure if this is a me problem (is there no require necessary for the sigil lib?)

what is git .sigil?

ugh i know what this is. we can fix it tmr.

@g-a-v-i-n
other options:

fontfaceset --> this looks super close to what we want?
fontfaceobserver

going to implement this because it seems like it can wrap around the current structure w/o making changes to drawing syntax like with opentype.js


Scan only one file

Is your feature request related to a problem? Please describe.
First, thanks to this fantastic initiative, I was looking for a tool like this one.
I'd like to scan only one file instead of a directory.
Describe the solution you'd like
It could be great to have an option to scan only a file instead of a whole directory
Optional: Describe alternatives you've considered
I'm attempting to scan all the module, but it's big.
Optional: Additional context
I'd like to see the depencies of a file in python, but I got this error:
NotADirectoryError(f'error in analysis {analysis.analysis_name}: source directory not found/ accessible:

@israteneda Thanks for the feedback, I've included a new config option only_permit_files_matching_absolute_path in the current release 1.2.2 that should solve your problem (https://github.com/glato/emerge/releases/tag/1.2.2). You can now simply specify one or more source code absolute paths in only_permit_files_matching_absolute_path, which are only permitted within the source code scan.

Hey @glato thanks for your effort put in this solution, I appreciate it 🙌


How long is the registration information retained on the client side after eureka server closed

Application(A) as a service was registered in the eureka server, so another application(B) was able to call the service (A) by eureka client. Then I shutdown the eureka server, What I'm confused about is that the app(B) still was able to call the service(A) during about 30 seconds.
Could you please explain how long is the registration information retained on the client side after eureka server closed and which property affect the duration.
Thank you,

Hi, if you are using the provided java eureka-client, then the registry cache on the client side will be forever and does not expire. The data will update if (at least one) eureka server exist and is reachable, and if no eureka servers exist, then the cache will essentially be frozen in time. Note that the default DiscoveryClient.java implementation offers the method getLastSuccessfulRegistryFetchTimePeriod() for users to monitor for staleness of the local registry cache.
If by closing the server, you mean the shutdown of the server for Application(A), then the entries for Application(A) will be purged from the local cache of the client on Application(B). The time delays are:
A shutdown gracefully -> eureka server removes entries for A (0s delay)
A has a dirty shutdown -> eureka server evicts entries for A (90s delay)
+
B cache update -> B see cache update for removal of A (0-30s cache refresh cycle)
Also please note that if you run with a small number if instances, you may be subject to self-preservation as described here (https://github.com/Netflix/eureka/wiki/Understanding-Eureka-Peer-to-Peer-Communication), and may want to tune the self-preservation thresholds as necessary.


Updated java examples to use the Gateway

Updates the Java examples to use the Gateway, see https://github.com/openmicroscopy/openmicroscopy/pull/3899

To be considered only when the decoupling work is merged.

Assuming this can be reviewed now unless you have further work to push before it is ready for someone else to look at @dominikl ?

I added some methods for ROI handling to the Gateway, but not tested yet. So I still might have to update the ROI example in the documentation.

Ok, I'll leave you to stick this on stand-up when you are ready then. Thanks!

The documentation should now comply with the Java code examples in Training/Java

Something funny going on with Travis here, any ideas @sbesson ?

Likely related to maintenance activity in the afternoon. Relaunched.

Reference to Requests.java in line 136 of developers/Server/GraphMigration.txt will also need updating here.

Gateway not going in until 5.1.4 now.

Staged at https://www.openmicroscopy.org/site/support/omero5.1-staging/developers/Java.html

Few comments:

I agree in general with what's going on here; comparing to the Python pages, there's no "raw" use of omero.client() that we document, so this brings us in line.
I do wonder if the code here is or can be tested in any automated way.
I missed something from https://github.com/openmicroscopy/openmicroscopy/pull/3952 -- I thought we were going to get rid of the "pojo" package. If so, we should also try to remove use of "pojo" here completely. (That may just be a follow-up PR if we're stating clearly "This is an ALPHA release of this API" etc.)

The code is basically copied from the Java training examples, but I'm not sure if/where they are run automatically.
Pojos are going to be removed? Means always working with (slightly annoying, e. g. getXYZ().getValue() etc.) ICE objects directly? I wasn't aware of that...

Pojos are going to be removed? Means always working with (slightly annoying, e. g. getXYZ().getValue() etc.) ICE objects directly? I wasn't aware of that...

Sorry, I meant renaming to something more OME-y.

code is run as part of training job.

@jburel /gateway/Gateway.java is still showing as a broken link on https://ci.openmicroscopy.org/view/Docs/job/OMERO-DEV-merge-docs/13/warnings3Result/ - is this PR still waiting on further code changes?
Dom is back tomorrow so if you have time to look over this today, he can make any further changes needed and we can get this cleared.

@hflynn: I will have a look at it

@hflynn: I have opened a PR against @dominikl's PR so he can merged tomorrow when he returns.

Sorry @dominikl one slight tweak needed that i missed yesterday

Finally all green thanks! Is this good to merge now @jburel ? Is a card needed on the 5.2 board for any follow on docs re: Josh's comments above?

@hflynn: renaming will be another breaking changes. It has been discussed before. note that 3rd party projects already use them.

--no-rebase

renaming will be another breaking changes. It has been discussed before. note that 3rd party projects already use them.

Unless it's possible to inject the new renamed classes as superclasses of the existing pojo.* classes.

Unless it's possible to inject the new renamed classes as superclasses ...

Sorry, this would require multiple inheritance. Sorry for the noise. But I do think this is a question that we should decide very soon since we don't want to be supporting pojos.* for any significant amount of time.


Support for HotChocolate.Execution > 12.4.0?

What about support for e.g. 12.6.0 or 12.5.0 ?
According to https://github.com/benmccallum/fairybread#compatibility only 12.4.0 is supported? Or is only the readme outdated?

This is a minimum version, so anything above 12.4 is supported, no worries :)


Block calls from sidecar to app when app is not healthy - fix 4883

Description
This PR adds a new apphealth module which is used to probe for the health of the app. The feature is disabled by default.
As described in #4883, when the feature is enabled, Dapr periodically polls the app at a given endpoint (HTTP or gRPC, depending on app-protocol). When probes fail for more than a certain threshold in a row, Dapr stops all subscriptions (pubsub topics and input bindings) and short-circuits service invocation calls (that are rejected before attempting to invoke the app).
This PR adds a few new CLI options for daprd:

enable-app-health-check (annotation: dapr.io/enable-app-health-check): Boolean that enables the health-checks (it's false by default, so no health checks are performed)
app-health-check-path (annotation: dapr.io/app-health-check-path): String, used only when app-channel is HTTP. This is the path that Dapr invokes for health probes (default: /health).
app-health-probe-interval (annotation: dapr.io/app-health-probe-interval): Integer with number of seconds between each health probe (default: 5)
app-health-probe-timeout (annotation: dapr.io/app-health-probe-timeout): Integer with number of milliseconds for timing out health probe requests (default: 500)
app-health-threshold (annotation: dapr.io/app-health-threshold"): Integer with max number of consecutive probe failures before the app is considered unhealthy

Almost everything in #4883 was included. The missing thing was being able to use successful requests from Dapr to the app as passed health probes (e.g. a network failure there would be considered the same as a failed health probe). Support for that is included in the apphealth module, but I have not (yet) implemented it because this PR was already large enough, and after initial investigation, that feature requires a lot more work and care (because signals would have to be sent in multiple packages, and with resiliency still optional there are lots of code paths to consider). If that is a feature that's strongly desired, I can revise that in the future.
This PR also includes E2E tests to validate the behavior.
Issue reference
Fixes #4883
Checklist
Please make sure you've  completed the relevant tasks for this PR, out of the following list:

[X] Code compiles correctly
[X] Created/updated tests
[X] Unit tests passing
[X] End-to-end tests passing
[ ] Extended the documentation @ItalyPaleAle  TODO

@artursouza added a feature flag AppHealthCheck which needs to be enabled for this to work (will need to be put in docs).
If the feature flag is not set and the user adds --enable-app-health-check, health checks are disabled and this is shown in the logs:

App health checks are a preview feature and require the AppHealthCheck feature flag to be enabled. See https://docs.dapr.io/operations/configuration/preview-features/ on how to enable preview features.


Collection Edit: Support Permissions Update

Context
Add functionality to the collection edit modal allowing users to update collection permissions. Ensure that updated permissions are accurately enforced and reflected on-chain (if applicable).
Task
Your task is to call nfts.updateMintSettings(collection, mintSettings) where collection  parameter is obvious so I expand the mintSettings

mintSettings has few fields:

type - the most important part
price - the price that is set to mint
startBlock - starting period of minting
endBlock - ending period of minting.

For the task I would hide start and endblock
So we have just type and price
we have three types:

Issuer - the one that Koda uses classic minting (create at our platform)
Public - one we use with drops (along with price :)) - anyone can
HolderOf(collection) - we used that In the past at one drop

I will expand after MetaHours

@vikiival could you provide more input about the permission? Which field or metadata are related to the permission of collection?

@Jarsen136 I have updated the description, please check, I will expand the rest


Gitea 1.7 and 1.8 just segfault: Ubuntu 14.04

Gitea version (or commit ref): 1.7.6 and 1.8.3
Git version:  2.22.0
Operating system: Ubuntu 14.04, amd64
Database (use [x]):

[ ] PostgreSQL
[x] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[x] Not relevant

Log gist: N/A

Description
I'm running Gitea 1.6.3 just fine on an Ubuntu 14.04 amd64 VM. I just wanted to upgrade to Gitea 1.8.3 but the only thing I get is this:

Segmentation fault

Nothing more, no stack trace.
The same goes for Gitea 1.7.6 which I just tried afterwards.
Anything I could do here to at least get a stack trace or any more information?

I think golang compiler does not support it anymore

Oh, that would be a little unfortunate. I'm a bit stuck with this machine on Ubuntu 14.04 and want to upgrade it to Ubuntu 20.04 as soon as it's released. This would mean that I'm stuck with Gitea 1.6 until then.

You can compile Gitea yourself from source using older golang compiler version (golang 1.9 should probably work). Check docs.gitea.io on how to do that

Thank you for the answer. I successfully compiled from source with Go 1.11 and that binary worked.
Then I found my actual problem: wget failed a few times yesterday and my gitea-1.8.3-linux-amd64 was incomplete. I overlooked the successful download gitea-1.8.3-linux-amd64.3. So I can report, that the actual release binary just works fine as well with Ubuntu 14.04. m(


FHIR Server ARM Template deployments fails

Describe the bug
Running the following fails:
New-AzResourceGroupDeployment -TemplateUri https://raw.githubusercontent.com/Microsoft/fhir-server/master/samples/templates/default-azuredeploy-sql.json -ResourceGroupName $rg.ResourceGroupName -serviceName $fhirServiceName
I'm getting this error:
New-AzResourceGroupDeployment : The template deployment 'default-azuredeploy' is not valid according to the validation
procedure. The tracking id is '1cbe52c4-3b5c-446c-9926-c4b95bcacf2f'. See inner errors for details.
At line:1 char:1

New-AzResourceGroupDeployment -TemplateUri https://raw.githubusercont ...

  + CategoryInfo          : CloseError: (:) [New-AzResourceGroupDeployment], CloudException
  + FullyQualifiedErrorId : Microsoft.Azure.Commands.ResourceManager.Cmdlets.Implementation.NewAzureResourceGroupDeploymentCmd     let

Getting the extended error using the correlation Id using Get-AzLog, I'm getting this error
statusCode     : BadRequest
serviceRequestId:
statusMessage  : {"error":{"code":"InvalidTemplateDeployment","message":"The template deployment
'default-azuredeploy' is not valid according to the validation procedure. The tracking id is
'1cbe52c4-3b5c-446c-9926-c4b95bcacf2f'. See inner errors for
details.","details":[{"code":"ValidationForResourceFailed","message":"Validation failed for a resource.                           Check 'Error.Details[0]' for more information.","details":[{"code":"ServerFarmNotFound","message":"The                            specified app service plan was not found."}]}]}}
Note:
If I copy this template and deploy via the Azure portal, it works fine

I have confirmed this bug, however, it is a bit puzzling since we have been running with this for a while, so it could be something that changed on the ARM side of things. We will have to investigate.
@brendankowitz we made some changes to use the zipdeploy and there was a quick fix for the dependencies there to avoid that small problem we had, but as far as I can see, we did not touch the dependencies between appservice plan and app service.

Don't think its anything we did, I get the same error using the version before any changes. I recall seeing an issue like this for ARM in the past
New-AzResourceGroupDeployment -TemplateUri https://raw.githubusercontent.com/microsoft/fhir-server/6d9ed8bcaf9f5ba18c683da8841c31ac9b11374d/samples/templates/default-azuredeploy.json

I agree, something changed in ARM. Last time we had this issue, I think @jackliums was involved in our workaround. Maybe some learnings from that?

For now, the workaround in this issue (https://github.com/microsoft/botframework-solutions/issues/2762) also seems to work here.
So changing this https://github.com/microsoft/fhir-server/blob/master/samples/templates/default-azuredeploy-sql.json#L204 to
"serverFarmId": "[concat('/subscriptions/', Subscription().SubscriptionId,'/resourcegroups/', resourceGroup().name, '/providers/Microsoft.Web/serverfarms/', variables('appServicePlanName'))]"

Portal deployment worked, thanks :-)
I noticed that with the Msdeploy Package Url , we can specific a forked repo /branch. Previously in the KUDU version if the src ref. was left blank only the PaaS components were created (no app deployment). It was then possible to do an independent deployment.
Is there an equivalent in this new version?

@uxbridgeroad right now the options are to pass in a webdeploy file to use instead or it will use the latest build from master. After the deploy succeeds you can always push a deployment over the top.

This morning I had to come back to one of my old demos.  It deploys using a DevOps pipeline, so I can delete it when I finish to save money.  I have deployed 161 times over the last couple of years and it has always worked.  This morning it failed.  Luckily I stumbled across this.  Changing as suggested above from subscription().id to Subscription().SubscriptionId in serverFarmId in my ARM template fixed it for me.

Fixed in #903


Newrelic readme note

Could you please provide more info about this note in the readme: "If you're using newrelic make sure you import newrelic first as it patches global.Promise before zone,js does"
What is the issue exactly?
Can we do something so that zone.js can simply be loaded before any other script?
It is really hard (and in some cases impossible) to get the order of execution right, when there are special cases. Specially because we need to look for any library that patches global.Promise.

This is just how to the two libraries are designed. They both money-patch globals for their own reason. Meteor also did this in the past so it's just something everyone has to deal with until Zone becomes a standard or if there is a flag introduced into Node that will support Zone.js natively. Newrelic can introduce a zone.js version that takes advantage of zone.js rather than their own system for tracking errors but that's also a lot of development work for no real reason.

@gdi2290 ,
Thanks for your reply.
So the issue only exist on Node and not on browsers?
Also, Could you please provide more detail why loading Zone.js after newrelic is resolving the issue?

the same thing happens in browsers. If there is anything patching Promises/setTimeout etc you have to include it before zone.js that way zone.js is able to wrap them for angular to track async calls.
for newrelic they also patch Promises and you can see it here.
https://github.com/newrelic/node-newrelic/blob/master/lib/instrumentation/promise.js
you can also create an issue with newrelic and ask them more about their lib


Resetting formKey won't clear the file preview

When resetting formKey, the value of field is being resetted successfully, but the preview is not cleared at all.

Hi!
Please fill the template with info and screenrecorder to improve this issue. Without this info, will be hard solve this
## Environment

**Package version:**

  Flutter doctor

  Code sample

```dart

Description
Expected behavior:
Current behavior:
Steps to reproduce

This
Than that
Then

Images
Stacktrace/Logcat


BANP: Reuse port unpacking logic

Description
Use unpackANPPorts function in combinePortsWithANPEgressPeers

Related issues/PRs
Related to https://github.com/projectcalico/calico/pull/9464

Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

TBD

Reminder for the reviewer
Make sure that this PR has the correct labels and milestone set.
Every PR needs one docs-* label.

docs-pr-required: This change requires a change to the documentation that has not been completed yet.
docs-completed: This change has all necessary documentation completed.
docs-not-required: This change has no user-facing impact and requires no docs.

Every PR needs one release-note-* label.

release-note-required: This PR has user-facing changes. Most PRs should have this label.
release-note-not-required: This PR has no user-facing changes.

Other optional labels:

cherry-pick-candidate: This PR should be cherry-picked to an earlier release. For bug fixes only.
needs-operator-pr: This PR is related to install and requires a corresponding change to the operator.

Thanks!


Issuing credits to Wes

Issue 15000 credits to @wesc.

 Follow up
On Sunday, 10-Mar-24 02:17:00 UTC by Gov4Git dev
Could not issue 15000 credits to member @wesc. Reopen the issue to retry.
Because: file does not exist


chore(DSE-291): va monitoring db backup

Resolves DSE-291
Context
Adding va monitoring database to aws backup plan
Summary of Changes

standardization of tags to align with tagging guidelines
addition of customer managed multi region kms key
addition of backup-plan tag to optionally opt-in to backup plan

Progress Checklist
A break down of the steps. Useful when the PR is in the draft stage

[ ] step 1
[ ] step n

Additional Considerations
Any additional consequences, side effects, uncertainties stemming from the changes in this PR.
Instructions for the Reviewers
What do you expect from the reviewers? E.g. What code should be run to reproduce the results? Is there anything that needs attention?
Housekeeping Checklist

[ ] Linked the PR to a Jira ticket?
[ ] Linked the Jira ticket to the PR?
[ ] Checked Draft/vs Ready to Review?
[ ] Tagged the reviewers if Ready for Review?
[ ] Assigned yourself as Assignee?

Code Review Checklist
The following items shall be verified prior to merging any new code into the production environment.
Please refer to the Code Review Checklist to evaluate each category
Correctness

[ ] Yes
[ ] N/A

Style and Clarity

[ ] Yes
[ ] N/A

Performance

[ ] Yes
[ ] N/A

Security

[ ] Yes
[ ] N/A

closing, meant to open against my fork 🤦


Persist the last POST

Description
Currently neither the initial nor the subsequent proofs are persisted. Thus it might happen that the node is shut down between the proof is created and it is used. That proof would be lost and would have to be recreated, which is hard.
Affected code

especially the initial POST,
every post built by nipostBuilder,

Proposed solution
The last proof should be persisted (in a DB or in a file, like the nipost builder state).

i think for ephemeral data it is better to stick with files


Add azure custom script extension

What does this PR do?

New attack technique: Azure Custom Script Extension

Motivation

Related to !116

Checklist

[X] The attack technique emulates a single attack step, not a full attack chain
[X] We have factual evidence & references that the attack technique was used by real malware, pentesters, or attackers
[X] The attack technique makes no assumption about the state of the environment prior to warming it up

Rebased on latest main and added some minor fixes, let me know what you think @rcobb-scwx!

LGTM! Ship it! Thank you for the great work.


Attachment saving to specific Directory

Hi zbateson,
I just tried this library everything works well, really appreciating your effort and time!
I would like to know more about attachment handling, in single word how I can save all of attachments to specific directory in my server?
Here I am standing now
$att = $message->getAttachmentPart(0);
$atts = $message->getAllAttachmentParts();
Could please help me on this? it will be very helpful, I already spend lot-of time to figure out  :(
Again Thanks for this great library!

Hi @channelriyas
Sure -- perhaps some extra documentation is needed there :) you should be able to:
$atts = $message->getAllAttachmentParts();
$ind = 0;
foreach ($atts as $part) {
    $filename = $part->getHeaderParameter(
        'Content-Type',
        'name',
        $part->getHeaderParameter(
             'Content-Disposition',
             'filename',
             '__unknown_file_name_' . $ind
        )
    );
    $out = fopen('/path/to/dir/' . $filename, 'w');
    stream_copy_to_stream($part->getContentResourceHandle(), $out);
}

I didn't run this, but that's the basic idea.  Let me know if it works out or if some corrections are needed and I'll add it to the usage guide.

Hi Zaahid,
Thanks for quick reply!
I just run this code but got error for this line
$atts = $message->getAllAttachmentParts();
PHP Fatal error:  Call to a member function getAllAttachmentParts() on a non-object in /../../
is this familiar?
Thanks!

Hi @channelriyas -
That probably means you haven't assigned anything to $message.  If you look at the example on the project's front page you can see how $message is assigned:
// ...
$message = $mailParser->parse($handle);
// ...

unless something went wrong in the call to parse().

Hi Zaahid,
Yes, figure out the issue, my mistake, accidentally $message replaced with body content, thats why error ;)
Anyway now attachments works well, I can move to specific dir as-well, here is my code some one will help.
#!/usr/bin/php -q
<?php
require_once 'vendor/autoload.php';
$mailParser = new ZBateson\MailMimeParser\MailMimeParser();

$handle = fopen ("php://stdin", 'r');
$message = $mailParser->parse($handle);         // returns a ZBateson\MailMimeParser\Message
fclose($handle);

$to  = $message->getHeaderValue('to');
$from  = $message->getHeaderValue('from');          //<EMAIL_ADDRESS>$from_name  = $message->getHeader('from')->getPersonName();    // Person Name
$subject  = $message->getHeaderValue('subject');       // The email's subject

$res = $message->getHtmlStream(); // getTextStream
$body_content = stream_get_contents($res);

$atts = $message->getAllAttachmentParts(); //get all attachments
if($atts){
	$ind = 0;
	foreach ($atts as $part) {
	    $filename = $part->getHeaderParameter(
	        'Content-Type',
	        'name',
	        $part->getHeaderParameter(
	             'Content-Disposition',
	             'filename',
	             '__unknown_file_name_' . $ind
	        )
	    );
	    $out = fopen('/path/to/dir/' . $filename, 'w');
	    stream_copy_to_stream($part->getContentResourceHandle(), $out);
	}
}
?>

Zaahid, Thank you very much, this is really awesome library!!!!!!!! :+1:

Excellent, happy to hear that Riyas :)

Hello, I am trying to get the script working using the code example from Riyas above but every time I send a email I receive: "Mail delivery failed: returning message to sender" and no error log is written.
Library is installed: zbateson/mail-mime-parser 0.4.9   MIME email message parser
Can you help me?
#!/usr/bin/php -q
parse($handle);         // returns a ZBateson\MailMimeParser\Message
fclose($handle);

$to  = $message->getHeaderValue('to');
$from  = $message->getHeaderValue('from');          //<EMAIL_ADDRESS>$from_name  = $message->getHeader('from')->getPersonName();    // Person Name
$subject  = $message->getHeaderValue('subject');       // The email's subject

$res = $message->getHtmlStream(); // getTextStream
$body_content = stream_get_contents($res);

$atts = $message->getAllAttachmentParts(); //get all attachments
if($atts){
    $ind = 0;
    foreach ($atts as $part) {
        $filename = $part->getHeaderParameter(
            'Content-Type',
            'name',
            $part->getHeaderParameter(
                 'Content-Disposition',
                 'filename',
                 '__unknown_file_name_' . $ind
            )
        );
        $out = fopen('attachments/' . $filename, 'w');
        stream_copy_to_stream($part->getContentResourceHandle(), $out);
    }
}
?>


"Segmentation fault: 11" when using iojs 3.1.0

Hello
Simply running a file with only the below contents with iojs 3.1.0,
require('watchify');

will cause the error,
Segmentation fault: 11

Previous versions of iojs don't seem to have the same issue ...
Thanks!

Ah shoot, don't mind me ... I just needed to recompile the native addons under iojs 3.1.0. Thanks!


Could you tag a release

It would be great with a tagged release, so when downloading zips you can include a specific version in your builds.

@lsolesen v1.0.0 is tagged now, you can probably close this issue. It's also composerised: https://packagist.org/packages/ptcinc/solr-php-client .


Fix *nix to match Winsock's separation of nonblocking/blocking and overlapped I/O.

Winsock considers the {non,}blocking option of a socket separately from the overlapped option for the same. As a result, a blocking socket can be used for nonblocking I/O on Windows, provided the overlapped APIs are used. As these APIs are what is used under the covers by the asynchronous Socket APIs, this behavior leaks through (intentionally or otherwise) to Socket: a blocking Socket can be used for async I/O without any special consideration.
Unfortunately, no other platform allows non-blocking I/O on a blocking socket. This change moves all socket I/O on non-Windows platforms onto non-blocking sockets in order to support the Winsock-style semantics. Blocking I/O is emulated using async I/O and events.

@stephentoub PTAL at the cross-platform changes. @CIPop: there is a very small set of changes to Socket.cs; PTAL.

@stephentoub I should point out that I have additional changes on top of these; I'll hold off on submitting a corresponding PR until these are in.

A few minor comments, but LGTM.

@CIPop there are also a few minor changes to SocketPal.Windows.cs.

Ping: I'd like to get this merged ASAP as there are other changes (yet to be posted) that depend on this PR.

@davidsh can you take a look at these changes as @CIPop is on vacation?

@pgavlin I need to abstain from this PR.  I have not been involved in this particular effort and don't really have context to give it justice reviewing it. Given the approaching deadline, I'm fine with pushing this if you think it is ready for the next step.  When @CIPop is back on Wednesday, he can give additional feedback.  You can incorporate that feedback later into future PR's..

SGTM. Thanks!


Disable inlining for x86 when there is try

Curious- what is the motivation behind this change?

https://github.com/Microsoft/ChakraCore/pull/3832 - enabled inlining into functions with try. On x86, additional analysis is needed to compute arg offsets and stack offsets when there is a bailout in EH region, due to x86 specific calling conventions. Disabling till all corner cases are fixed.


Custom skip_cache

Is it possible to create your own custom skip_cache? For instance, there was recently a commit to add woocommerce_items_in_cart in the http_cookie conditional. But there are a number of other ecommerce solutions for WordPress (EDD, Leaky Paywall, etc). It would be nice if there was a way to add a conf in the site's directory to add their own conditional to skip the cache.
Perhaps just the appropriate conf files could include a skip-cache.conf file in common/ that can be customized and not overwritten on ee update. Of course, this isn't entirely ideal, because it'd be a global for every site on a server, but I haven't found any other way to achieve this.

This is how I do in my .conf file
`# Don't skip by default
set $skip_cache 0;

Woocommerce Pages

if ($request_uri ~* "/(apoio|carrinho|finalizar-apoio/minha-conta)/*$") {
set $skip_cache 1;
}
if ( $cookie_woocommerce_items_in_cart = "1" ){
set $skip_cache 1;
}
BuddyPress // bbPress Pages
if ($request_uri ~* "/(sala|estatistica|perguntas/ativacao/turmas)/*$") {
set $skip_cache 1;
}
`
You could create a file and put your custom skip-cache statements and load it. Save your version for ee updates.
I agree that a custom localization or skip-cache file loading with support for updates would be a good addition to ee, because custom skip-cache are a specific scenario, one should know first how many people actually changes the .conf file to add this.

@lewayotte  you can follow the solution provided by @renatonascalves . We do not replace whole .conf file on ee update unless there is a major change on file.
If you have any other queries please open a topic at http://community.rtcamp.com/c/easyengine.

Just one more thing, the only time I saw/noticed the .conf being updated and removing all my custom skip-cache was when I updated PHP version, from 5.6 to 7. Easyengine updates the whole .conf with the new information and back the old one up.


Fix for echo() function

This adds back/ fixes the 'echo' feature and adds a simple cli test (for now) to test it

The changes for echo() look fine.
FYI, if echo() is decommissioned then some of the examples need to be updated.

The changes for echo() look fine.
FYI, if echo() is decommissioned then some of the examples need to be updated.


Update secp256k1 dependency, remove rand_legacy.

Paves the way to upgrade to rand:0.8 by updating the secp256k1 dependency to latest and removing the obsolete rand_legacy dependency.

Note that we can't update from 0.7 to 0.8 of rand unless we can get ff-zeroize updated, see: https://github.com/algorand/ff-zeroize/pull/2


Bug in React about where it wants to insert its script elements

On building the Docker image locally, I noticed that React was inserting script elements incorrectly, which caused Cesium to break.  As a workaround, I was able to move the external script reference earlier in the html.

FYI -- going to merge this one so I can test it with the Docker build.  Seems silly but I was able to reproduce malformed html that appeared to be as a result of React trying to insert things.


feat(Share): install react-native-share

Codecov Report

Merging #2 into develop will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff            @@
##           develop       #2   +/-   ##
========================================
  Coverage    98.25%   98.25%
========================================
  Files           20       20
  Lines          401      401
  Branches        61       61
========================================
  Hits           394      394
  Misses           7        7

Impacted Files
Coverage Δ

src/viewModels/home.ts
98.73% <100%> (ø)
:arrow_up:

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 27f7230...ed9197b. Read the comment docs.


[cli] Add docker to rancher/cli docker image

Rancher versions:
rancher/cli: 0.6.0
Steps to Reproduce:
$ docker run -it --rm \
-e RANCHER_URL=${myHost} \
-e RANCHER_ACCESS_KEY=${myKey} \
-e RANCHER_SECRET_KEY=${mySecret} \
-e RANCHER_ENVIRONMENT=${myEnv} \
rancher/cli exec ${myContainer} exec  uname
FATA[0000] exec: "docker": executable file not found in $PATH

Reason
The dockerimage for rancher/cli is currently built from alpine with rancher and ssh added.
"rancher exec" relies on "docker exec" inside it's container
Possible resolution
Add docker packages to rancher's cli image

I've created an image, where I installed docker and rancher clie-s https://hub.docker.com/r/skoba/rancher-cli/
I will update it with Dockerfile and readme soon.
rancher exec works

@aemneina Why label question? It seems more like a bug - rancher exec does not work in official Docker image.

@sergey-koba-mobidev, would you be willing to share your Dockerfile for skoba/rancher-cli?

With the release of Rancher 2.0, development on v1.6 is only limited to critical bug fixes and security patches.


plot3d with select_atoms does not work anymore

If you try to plot3d() from a generated structure with select_atoms showing subset of atoms:
from pyiron import Project
pr = Project('test')
struc = pr.create.structure.bulk('Fe', cubic=True)
struc.plot3d(select_atoms=[0])

you will end up with the error:
File /u/system/SLES12/soft/pyiron/dev/anaconda3/lib/python3.10/site-packages/structuretoolkit/visualize.py:310, in _plot3d(structure, show_cell, show_axes, camera, spacefill, particle_size, select_atoms, background, color_scheme, colors, scalar_field, scalar_start, scalar_end, scalar_cmap, vector_field, vector_color, magnetic_moments, view_plane, distance_from_camera)
    308 if select_atoms is not None:
    309     select_atoms = np.array(select_atoms, dtype=int)
--> 310     elements = elements[select_atoms]
    311     atomic_numbers = atomic_numbers[select_atoms]
    312     positions = positions[select_atoms]

TypeError: only integer scalar arrays can be converted to a scalar index

The current structure.plot3d() currently works as follows:

plot3d() Inside pyiron_atomistics/atomistics/structure/atoms.py calls and passes pyiron_to_ase(self): self=structure to
plot3d() inside /structuretoolkit/visualize.py
Which passes the element list as a list and not numpy array
However plot3d() inside visualize.py explicitly changes the select_atoms into a numpy: select_atoms = np.array(select_atoms, dtype=int)
You end up trying to index a list with a numpy array which does not work:

s = ['A', 'B', 'C', 'D']
ind = np.array([1])
s[ind]

If you try to use plot3d() directly from visual.py and pass the structure to it, it works because elements is an array no pyiron_to_ase was imposed.
from structuretoolkit.visualize import _plot3d
_plot3d(struc, select_atoms=[0])

Proposed solutions:

remove pyiron_to_ase(self) and just pass (self) in plot3d() in atoms.py
explicitly change elements list to numpy similar to select_atoms in plot3d() in visual.py: elements= np.array(elements)

Both are tested and work locally. However, I am not sure if they affect other functionalities.

This is now fixed by #104


Update Readme

Fix errors in the readme, add on-demand import examples, and synchronize some content to readme.zh.

can you also please translate the example scripts on the ZH translation
Many thanks.

can you also please translate the example scripts on the ZH translation
Many thanks.

I can, please merge this first, and I will submit a PR after the ZH is modified.

Sure, no problem.
On Thu, Aug 22, 2024, 7:37 AM 薛定谔の狗狗 @.***> wrote:

can you also please translate the example scripts on the ZH translation
Many thanks.
I can, please merge this first, and I will submit a PR after the ZH is
modified.
—
Reply to this email directly, view it on GitHub
https://github.com/RaSan147/pixi-live2d-display/pull/10#issuecomment-2303446842,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIDNL2ZSZN5ZABGH4WRMP4LZSU6DZAVCNFSM6AAAAABM3LPO7GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMBTGQ2DMOBUGI
.
You are receiving this because you commented.Message ID:
@.***>


Policy and workspace trust can switch to using Setting indicators

Currently in the GUI Settings editor, settings affected by policies or untrusted workspaces display large alerts that also disallow the user to change the setting value.
Instead, we can consider moving those alerts into the indicators area, which is the area around each setting where indicators such as "Also modified in" can show up.
Done:

[x] Change the configurationService to return the values for all scopes even for policy and workspace restricted settings, while keeping the actual inspected value the same as before. @sandy081
[x] Change the UI to use indicators for policies and workspace trust. @rzhao271

Todo:

[x] Get some rounds of feedback
[x] Write TPIs for policy and workspace trust settings in the Settings editor

\closedWith 740d837

Adding a link to the epic https://github.com/microsoft/vscode/issues/147756


Load Search with a facade

This branch will merge into #431
This PR replaces the initial entry point to the Search component with a facade. Once the users clicks or focuses on the facade, the Search component will be loaded.
How to test

QA checklist

[ ] No errors logged to console.
[ ] Accessibility score in Chrome DevTools Audit/Lighthouse is 100% with Lighthouse version: #.#.#.
[ ] Component is accessible at mobile-size viewport.
[ ] Component is accessible at tablet-size viewport.
[ ] Component is accessible at laptop-size viewport.
[ ] Component is accessible at big-monitor-size viewport.

Open the test cases app locally on:

[ ] Chrome, no errors logged to console.
[ ] Firefox, no errors logged to console.
[ ] Safari, no errors logged to console.
[ ] Edge, no errors logged to console.
[ ] Mobile Safari, no errors logged to console.
[ ] Android Chrome, no errors logged to console.

Before merge

[ ] Add entry to CHANGELOG.md to describe changes.
[ ] If updating dependencies or creating a release, run npx browserslist@latest --update-db to update the browser data and commit any changes to package-lock.json.

Performance testing in docs-lighthouse looks good. I want to wait for #414 to merge so that I can test these together.


Support rails 7.2 and update gemfiles

Fixes #178

Could anyone review this? It is blocking updating to the latest Rails. I've tested @npezza93 branch for couple for weeks in production and seems stable 😄

Seems stable to me also, been using it for a couple of weeks as well.

Also works for me


Allow nodemon to continue after npm run serve errors

This PR ensures Node.js errors (from Express) don't interrupt gulp watch
Follows the same approach as:

https://github.com/alphagov/govuk-frontend/pull/3034
https://github.com/alphagov/govuk-frontend/pull/3035

Do I undestand correctly that what was happening is nodemon logging to stderr and us turning that into an error event on the stream that closed everything? If that's the case, I'm wondering if that doesn't highlight that we're using output in stderr to be more than what they actually are. Maybe we should only pass them through like we do for stdout rather than interpret them as errors and leave it to non-zero exit codes to decide that something errored?

Do I undestand correctly that what was happening is nodemon logging to stderr and us turning that into an error event on the stream that closed everything? If that's the case, I'm wondering if that doesn't highlight that we're using output in stderr to be more than what they actually are. Maybe we should only pass them through like we do for stdout rather than interpret them as errors and leave it to non-zero exit codes to decide that something errored?

Yeah it was a "no risk allowed" decision 😬
I felt better knowing that every write to stderr will break the build unless:

Stream or task listens via .on('error', handler)
Stream has gulp-plumber handling errors

I'm undecided though, maybe get a second opinion before we remove the fail-safe?