KYLIN-4542 After downloading spark with bin/download-spark.sh , still…

… need set SPARK_HOME manually .
Proposed changes
Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request. If it fixes a bug or resolves a feature request, be sure to link to that issue.
Types of changes
What types of changes does your code introduce to Kylin?
Put an x in the boxes that apply

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have create an issue on Kylin's jira, and have described the bug/feature there in detail
[ ] Commit messages in my PR start with the related jira ID, like "KYLIN-0000 Make Kylin project open-source"
[ ] Compiling and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged

Further comments
If this is a relatively large or complex change, kick off the discussion at user@kylin or dev@kylin by explaining why you chose the solution you did and what alternatives you considered, etc...

Kylin on parquet use SparkClassLoader.java to load class. It only needs to know SPARK_HOME and does not need to export the jar package directory in find-spark-dependency.sh.


fix(@vtmn/preset-tailwind): Only tailwind 2 works

As the documentation says, I installed tailwind to use the preset.
In the documentation it is not specified that you have to be on a version of tailwind 2.
Actually, if we use tailwind 3 with 2, we miss a configuration in the theme.color part https://tailwindcss.com/docs/upgrade-guide#fill-and-stroke-use-color-palette
We can override it on our side to make it work
module.exports = {
  presets: [require('@vtmn/css-tailwind-preset')],
  content: ['./src/**/*.{svelte,js}'],
  theme: {
    colors: {
      current: 'currentColor',   // needed for tailwind 3
    },
  },
};

But is this a good practice? Is it better to stay in tailwind v2 and wait for vtmn v1 to switch to v3 ?
In this case, it should be specified in the doc that the preset is only available for tailwind v2.
Thanks :)

Thanks @Tlahey for your proposition.
What do you think if we add this line in our current preset? It will break something for Tailwind 2 users or not?
Thanks in advance :)

@lauthieb I have no idea. I don't think so, by default I think tailwind 2 will auto-complete the current to 'currentcolor'.
We need to check that

Hi @Tlahey,
Thanks for your answer. I just came back from vacation. Did you check that? Do you want that we check this together?
Thank you :)

Hey @lauthieb, we stay on v2 to be iso with vtmn and avoid any side effect.


Inserting text snippets via a plugin

I'd like to have a list of words which a user can select from, then have that word inserted at the cursor. I see there is a insertHTMl() which can be used in a Jodit plugin, but how do I define a plugin when using jodin-react?

I have made some progress with inserting the text, now to work out how to do a drop down list of text options: (I've got no icon in the toolbar but the tooltip and insert work)
buttons:[ 'source', 'fullsize', '|','bold','strikethrough',
{ iconURL: '',
tooltip: 'Insert My Name',
exec: function (editor) {
editor.selection.insertHTML("Hi this is Guy");
}
}]

I fixed some of the known issues of Jodit in this repository, including this one.
See if it helps:
https://github.com/ShahrozeAli/react-jodit-fixes


When using the ValueInterpreter class, is there functionality for parsing values of specific sizes?

In implementation of RxAndroidBle I am trying to get the vlue of out the System ID characteristic using the ValueInterpreter helper class.
The linked specification says that the value contained a 40-bit id and a 24-bit id, and I am trying to find a way to parse that value.

Hey,
ValueInterpreter is for extracting standard 8, 16 and 32 bit floats/integers from characteristics to provide the same functionality that is available using Android's BluetoothGattCharacteristic class.
The link you provided describes which bytes contain which IDs — you simply copy appropriate bytes of the characteristic to get the ID you want.
From my understanding these IDs are not really comparable with others but rather they are there so the user can check if they are equal to some other ID. If that is the case you can simply use Arrays.equals().
I hope my response helps you

Thanks!


Laziness in example programs confusing me...

the fun "sht4x_read()" in "sht4x.c" should read out 6bytes of temp and hum data but not 4bytes
int16_t sht4x_read(int32_t* temperature, int32_t* humidity) {
uint16_t words[2];
...
here is sames not correct according to the manual by the first view:
I2C bus operates with 8-bit data packages. Information from the sensor to the master has a checksum after
every second 8-bit data package.
Humidity and temperature data will always be transmitted in the following way: The first value is the
temperature signal (2 * 8-bit data + 8-bit CRC), the second is the humidity signal (2 * 8-bit data + 8-bit CRC).
The master may abort a read transfer after the 16-bit data if it does not require a checksum.
Laziness in example programs confusing people...
and the uint32 data is 1000 times, by divide 1000 is float / double actual num is kinda not appear in the manual...
i looked it for several times,,

The bytes are correct. If you check out the driver code you can see that it's abstracted in sensirion_i2c_read_words_as_bytes where the CRC is read, checked and then discarded since you don't need it further down. If the CRC doesn't match you get an error back.
https://github.com/Sensirion/embedded-common/blob/1ac7c72c895d230c6f1375865f3b7161ce6b665a/sensirion_common.c#L110C9-L110C42
The multiplied by 1000 result is also documented in the driver header since it's a driver feature that has nothing to do with the sensor. This is done because some architectures (e.g. 8 bit) are much slower at processing doubles.
https://github.com/Sensirion/embedded-sht/blob/76cb6e6c00d381706d93e56e19c3a7ec3462be0f/sht4x/sht4x.h#L98
I think this can be closed.

It is not friendly for quick integration and use. It is not just a pair of .c and .h files, but a bunch of files.
have to read and understand them. Then find that the method for calculating temperature and humidity is inconsistent with the device manual, then you found out its right after looking carefully.
and then, you know what, time is consumed...


Can we use angular js ng init directive for Controller initialize?

After developing an application, I had come across with great confusion.
can we use ngInit directive for Controller to initialize or get the data for the current view (controller)?
I am using ngInit for controller, since sometime the scripts are loading two times. If I call a server side method, in the controller function end, then two calls will go to the server.
If ngInit for controller is wrong to initialize data, can you please suggest a solution.
EXAMPLE
app.controller('testcontrol', ['$scope', '$http', function($scope, $http) {
    $scope.resultset = [];
    $scope.GetData =function(){
       $http.get('http://localhost:3000/testdata').then(function(data) {
          $scope.resultset = data.data.resultset;
          console.log($scope.resultset);
      },function(error) {
        $scope.error = error;
      })
    }

    $scope.init=function(){
        $scope.GetData();
    }
}]);

HTML:

            {{result.name}}

There is no need to use ng-init here. The controller's constructor function is called whenever the controller is instantiated, so you can just call the relevant initialization code there. This is one of the main goals of a controller...
app.controller('Testcontrol', ['$scope', '$http', function($scope, $http) {
    $scope.resultset = [];
    $scope.GetData =function(){
       $http.get('http://localhost:3000/testdata').then(function(data) {
          $scope.resultset = data.data.resultset;
          console.log($scope.resultset);
      },function(error) {
        $scope.error = error;
      })
    }
    $scope.GetData();
}]);

HTML:

            {{result.name}}

By the way, you should give your controller names that start with uppercase characters.

Closing as this is not an issue with Angular, but more of a support question. Hopefully the feedback I have given will be sufficient to help you.

Thanks. Even I didn't post it as bug. I want to know the reason, that's it.
And I've another question. Angular Team suggested that NgInit directive should be used only for ng-repeat. Not for other things.
Why we should not use for other things?
Is there any issue?

@Jeevanandanj - sorry to be confusing. The issue tracker should really only be used for bug reporting and feature requests.
Regarding ng-init, we should try to minimise the programmatic logic that is in the template. We can unit test JS code files but it is not so easy to test code in templates. The controllers are the place to put initialization logic. The only place where we felt that ng-init provided something that wasn't easily achievable elsewhere was the ability to set an alias for the current item in an ng-repeat iteration.

I appreciate that you said not to do the init logic in the template which sounds like good design. However, you never did address the OP's real concern, that his requests were being made twice. If the correct way to do init logic is in the controller, what is causing the init logic to be called twice?
I am also having the same issue, and this is not the first time. Thank you for your time in advance.

@MrOutput could you provide an example of a controller being called twice? It is difficult to diagnose your problem without a reproducible example.

I wish I could provide an example, but I can't purposely reproduce the error, if I could, I'd fix it. It happens in angular occasionally. Perhaps something wasn't setup correctly in a service or directive. I am not sure how it happens, or what causes it. It just does. Everything in controller space, gets called twice. Usually controller setup occurs in controller space (init, fetching db data etc). And that means that my XMLHttpRequests are also called twice, making the application 2x slower and doubling my database load for no reason. I cannot reproduce the error.

I'm afraid there is nothing we can do if you cannot reproduce the error.

Why does AngularJS include an empty option in select? and my default value is empty ?
blow is my code:
.directive('gettheme',function( ){
return {
restrict:"AE",
template:"",
link:function(scope, element, attrs){
scope.theme=['macarons', 'infographic'];
}
}
})

How about lifecycle hooks? Is it preferable to use, for instance, this.$onInit on the controller over a ng-init directive on the template?
Also it might solve the loading twice issue, right?


Resolving issue #193 as a part of GSSoC'22

Have you read the Contributing Guidelines?
Yes
Fixes#193
Description
We can use different Timeline models to get multiple model usage.
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
Covid-19 third wave analysis using multiple models to test which one suits the best among them all. Basically to find the best model.

@Kushal997-das Sir I am resenting the pull request please check it and if there is some correction please let me know.
Thank you

Make title of pr similar to your title of project


Support for comb v4 UUIDs

Hey there,
Wondering if you have any thoughts on supporting comb v4 UUIDs. Doesn't seem to be a ton of information out there about this, but there seems to be general agreement that it delivers potentially massive increases in database performance at the relatively minor cost of decreased uniqueness guarantees.
I'd be willing to take a shot at implementing this, but just want to know if it would be a welcome addition or not first.

For me, v1 UUIDs pose an issue because of their guessability. I know there's some back-and-forth right now about the benefits (or not) that you get in having randomized, unguessable keys, but at best, it looks like the jury's still out. Thus, for my purposes, the most important criteria are

near-guaranteed unique
unguessable
efficiently indexed
easy-to-generate

V1 UUIDs fail on the second requirement, but hit all the others. V4 UUIDs fail on the third requirement, but hit all the others. CombV4 UUIDs are, to me, the perfect blend.....
But again, I accept that you're not interested in incorporating this functionality. I'll likely extend your library to add it when I get time to do some research on how, but for now, I'll just use standard V4's.

FWIW, the node field in v1 uuids is (in node-uuid) randomly chosen when the module initializes. See here.  It's only set once, when the module is first required, so it's known to anyone who sees a previously generated v1 uuid from that process, but you could inject your own randomized node value thusly:
> const crypto = require('crypto');
> const uuidv1 = require('uuid/v1');
>
> uuidv1({node:crypto.randomBytes(6)});

'58cabe40-2672-11e9-9dff-aaffbcc2dc6a'

For anyone interested, an IETF draft for UUID version 6 has now been submitted.   We'll see what happens.
https://github.com/uuid6/uuid6-ietf-draft/blob/master/draft-peabody-dispatch-new-uuid-format-00.txt
https://datatracker.ietf.org/submit/status/109939/

@bradleypeabody When you submit, does it automatically open up a discussion thread on the IETF mailing list? If so, which mailing list (I'm subscribed to the general one, but I don't see a thread for it).

@ramsey I believe the discussion is separate from the submission.  I did a bit more discussion on the main IETF mailing list (ietf at ietf dot org) and the upshot of it was to submit a draft that lays out the proposal in detail - with particular attention to why it's needed.   From my understanding the IETF meets quarterly and their next meeting is in a few weeks, which is when it would get some official feedback (although I don't know what form that feedback takes - this is new to me).  I'll also reach out separately and see if I can get some more interest in the subject in general.  I did get some great responses on the mailing list but the case of why this is needed will definitely require some advocacy to make it happen.

@kael-shipman if you want something like the proposed uuid v6 I use https://github.com/tanglebones/pg_tuid which time prefixes the top bits of a UUIDv4.

Closing in preference to #580.  There's been some good discussion here, but it's getting a bit long in the tooth and I suspect most/all of it is stale compared to the IETF conversations that have been taking place.


Map (North) Orientation bug

Once the orientation icon is tapped inside the map, the screen adjusts to the correct orientation, flickers for a few seconds and disappears. The icon, reappears again after map orientation is manually adjusted.
Steps to reproduce the behavior:

Go to Maps
Click on North Icon Button
The button starts to flicker
Verbose error report can be found

Expected behavior
The button should change the orientation without flickering and disappearing.
Device/Emulator/Simulator (and architecture)
Pixel 5 x86_64 Emulator,
iPhone 14 Simulator,
Galaxy Tab Device,
iPhone 14 Device

@SaadmLMC
Don't know what you mean, the compass orientation is disabled. I tried to set compassEnable: true on the GoogleMap widget, but it won't show the needle icon. What "North Icon Button" are you talking about?


Configure a mapper from string to List and from List to string

I am trying to configure Napster of these two entities to pass from List to string but it gives me an error.
any ideas.
Thank you
public class  EntidadVM
     { public List<string> CCC}

public class  EntidadDto
     { public string CCC}

I'm doing them like this, but it gives me an error
TypeAdapterConfig<MailingDto, MailingVM>.NewConfig()
 .Map(dest => dest.CCC, src => new List<string>(src.CCC.Split(',')).ToList() );

TypeAdapterConfig<MailingDto, MailingVM>.NewConfig()
 .Map(dest => dest.CCC, src => src.CCC.Split(',').ToList() );

from Spanish to English

And is it possible to map from List<string> to string? (The idea being, converting a list of strings to a string that contains each item separated by a comma.)
With this configuration:
TypeAdapterConfig<List<string>, string>
    .NewConfig()
    .Map(dest => dest, src => string.Join(", ", src));

When running this:
var list = new List<string> {"one", "two", "three"};
var result = list.Adapt<string>();

Then result contains System.Collections.Generic.List`1[System.String], instead of the expected "one, two, three".
According to the documentation:

When Mapster maps other types to string, Mapster will use ToString method

And that's what's happening.
Is it possible to override that behavior or do you recommend using an extension method on the string type in this particular case?

@ignacioerrico In the documentation, it is mentioned that you need to use MapWith. Please refer to: https://github.com/MapsterMapper/Mapster/wiki/Custom-conversion-logic
For more complex situations, you can always resort to building a LambdaExpression
E.g.
var c = x.When(argument => argument.DestinationType == typeof(string) && argument.SourceType == typeof(List<string>));

c.Settings.ConverterFactory = argument =>
{
    var input = Expression.Parameter(typeof(List<string>), "list");

    var joinMethod = typeof(string).GetMethod("Join", new[] { typeof(string), typeof(IEnumerable<string>) })
        ?? throw new InvalidOperationException("Method 'Join' not found.");

    var delimiter = Expression.Constant(",", typeof(string));

    var call = Expression.Call(joinMethod, delimiter, input);

    var lambda = Expression.Lambda<Func<List<string>, string>>(call, input);

    return lambda;
};

Thanks a lot for your prompt reply, @devbased. You may close this issue, since both directions have been covered (mapping from List<string>  to string and vice versa.)

@ignacioerrico I don't have the right to manage the issue 😅 It just turned out that I'm subscribed to this topic, so I quickly replied to you 😊


unbound service broken

Describe the bug
Unbound fails to start. Or more accurately it starts fine and works for 2 minutes but then  it fails with  systemd[1]: unbound.service: start operation timed out. Terminating.
This probably has something to do with the systemd service type beeing Type=notify for communication between unbound and systemd. It probably has something to do with unbound not beeing able to access the systemd socket out of some containerization constraints used in the systemd file or because of the option 'chroot: ""' beeing set by default in the unbound.conf
To Reproduce
  services.unbound = {
    enable = true;
  };

Notify maintainers
@rissson
@joachifm
@ehmry
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
 - system: `"x86_64-linux"`
 - host os: `Linux 5.4.122, NixOS, 21.05.1205.b72bde7c4a1 (Okapi)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.3.12`
 - channels(root): `"nixos-21.05.1205.b72bde7c4a1, nixos-unstable-21.11pre297796.e1f8852faac"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
# a list of nixos modules affected by the problem
module: services.unbound

Type=notify is correct. Upstream uses the same: https://github.com/NLnetLabs/unbound/blob/04e9adc6109a62f527974c7d95310e713db5adac/contrib/unbound.service.in#L56

Does it ever get to ready state before it is killed?

systemd notify uses abstract sockets, which is not affected by chroot. Only ipc namespaces would be an issue: https://www.freedesktop.org/software/systemd/man/sd_notify.html

Looks pretty standard: https://github.com/NLnetLabs/unbound/blob/13e445d50b6e2a9441d9364d4213a57781d9e6f1/daemon/daemon.c#L693 Should be visible in strace.

Are you sure that services.unbound.package is pkgs.unbound-with-systemd? It should be the default on 21.05, but you may have changed it.

Are you sure that services.unbound.package is pkgs.unbound-with-systemd? It should be the default on 21.05, but you may have changed it.

Uh oh yeah I checked an I did set it I'm very sorry!


Refactor ARIMA/Prophet

Fixes: #3091 #3067 #2960 #3046
Perf tests here: https://alteryx.atlassian.net/wiki/spaces/PS/pages/1152516105/ARIMA+-+No+longer+relying+on+date+index
I've also added 4 new datasets to the S3 bucket:

Christmas_Daily_PageViews
Summer_Daily_PageViews
OlympicGames_Daily_PageViews
Pandemic_Daily_PageViews

I got these by downloading the Wikipedia page views from here for different articles over various time horizons.

After creating the pull request: in order to pass the release_notes_updated check you will need to update the "Future Release" section of docs/source/release_notes.rst to include this pull request by adding :pr:123.

Just for completeness, I filed this : https://github.com/alteryx/evalml/issues/3121


fix: update return type for @types/react

Issue
In @types/react version 2.14 (could have started earlier), the return type of PolymorphicForwardRefExoticComponent started being incorrect causing a TS error.
Example code
import type {
  PolymorphicForwardRefExoticComponent,
  PolymorphicPropsWithRef,
  PolymorphicPropsWithoutRef,
} from 'react-polymorphic-types'

type BaseButtonDefaultElement = 'button'

const PrimaryButton: PolymorphicForwardRefExoticComponent<
  PrimaryButtonProps,
  BaseButtonDefaultElement
> = forwardRef(function PrimaryButton<T extends React.ElementType = BaseButtonDefaultElement>(
  props: PolymorphicPropsWithoutRef<PrimaryButtonProps, T>,
  ref: React.ForwardedRef<Element>,
) {
  ...
}

Example error
Type 'ForwardRefExoticComponent<Omit<Omit<SVGProps<SVGSymbolElement>, "ref"> | Omit<DetailedHTMLProps<ObjectHTMLAttributes<HTMLObjectElement>, HTMLObjectElement>, "ref"> | ... 121 more ... | Omit<...>, keyof BaseButtonCommonProps | ... 3 more ... | "as"> & PrimaryButtonProps & { ...; } & RefAttributes<...>>' is not assignable to type 'PolymorphicForwardRefExoticComponent<PrimaryButtonProps, "button">'.
  Type 'ForwardRefExoticComponent<Omit<Omit<SVGProps<SVGSymbolElement>, "ref"> | Omit<DetailedHTMLProps<ObjectHTMLAttributes<HTMLObjectElement>, HTMLObjectElement>, "ref"> | ... 121 more ... | Omit<...>, keyof BaseButtonCommonProps | ... 3 more ... | "as"> & PrimaryButtonProps & { ...; } & RefAttributes<...>>' is not assignable to type '<InstanceT extends ElementType<any> = "button">(props: PolymorphicPropsWithRef<PrimaryButtonProps, InstanceT>) => ReactElement<any, string | JSXElementConstructor<...>> | null'.
    Type 'ReactNode' is not assignable to type 'ReactElement<any, string | JSXElementConstructor<any>> | null'.
      Type 'undefined' is not assignable to type 'ReactElement<any, string | JSXElementConstructor<any>> | null'.ts(2322)

Proposed solution
When changing it to return React.ReactNode instead of React.ReactElement, the type error goes away. I've tested this in our internal codebase and everything seems to work as expected.

The pull request may close the https://github.com/kripod/react-polymorphic-types/issues/12

@kripod

I wouldn’t recommend using this project anymore, as outlined over here: https://github.com/kripod/react-polymorphic-box/issues/25#issuecomment-1575558809
I’d suggest using a render prop instead of as, because the latter can’t have prop mappings.

@TheWashiba Please be respectful towards OSS contributors. Your behavior is the primary reason I’m tired of maintaining projects like this one. I’m calling this out loud – enough is enough. Feel free to fork the project and create a fix on your own if you want to.
People like you made me burn out. I’ve consciously decided to distance myself from OSS due to its adverse effects on mental health. Maintainers don’t own you a single thing, especially considering the fact that you don’t seem to sponsor any of them.

My apologies @kripod, no disrespect intended here. This finally solved my issue and I was just too hasty to get it merged.
Thank you for your contributions, sincerely

Note that this change breaks support for older TypeScript versions. To keep support for both, it is necessary to add a separate index.d.ts file for older versions and configure typesVersions in package.json: https://www.typescriptlang.org/docs/handbook/declaration-files/publishing.html#version-selection-with-typesversions

Closing because no review activity


An ampersand '&' is added on drag & drop from Explorer Pane to Terminal

This following is a full copy of https://github.com/microsoft/vscode/issues/98170 which was closed, although not solved.
An ampersand is added when dragging and dropping on the terminal a folder with a space in its name.

Version: 1.46.0-insider (user setup)
Commit: da3bf75
Date: 2020-05-19T10:07:59.927Z
Electron: 7.2.4
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Windows_NT x64 10.0.18362

This is by design as the & is required to run the file, some context: https://github.com/microsoft/vscode/pull/65994


Input with label is not showing properly ?

How to write code for input with left and right label or icon, It is not showing correct this time with react semantify, If I am doing something wrong in this below code then reply me to correct one:

            <Input placeholder="Loading..." type="text"/>
            <Icon className="search"></Icon>

       `

            Label

            <Input type="text" placeholder="Placeholder..."/>

`

@sahilpandya
My test is ok:
<Input className="left icon loading">
    <input placeholder="Search..." type="text" />
    <Icon className="search"></Icon>
</Input>

It would like:

Which semantic-ui and react-semantify version did you use?

Thanks, It is working for me.
I have one doubt related to semantic check-box input type, some time check-box input click is not working, Is there any specific reason for these,  ?

@sahilpandya I not sure but you can use <Checkbox /> component for check-box input.
like
<Checkbox init={true}>
	<input type="checkbox"/>

		The Simple Checkbox

</Checkbox>

ref:
http://jessy1092.github.io/react-semantify/#/Checkbox

Thanks, it is working correctly.


less is passed path as a regular option

It is safer to tell less that the path must not be accidentally parsed as an option, by using "--" as delimiter.
I.e. instead of "less +F somefile" it is safer to call it as "less +F -- somefile".

Thanks for the suggestion! I didn't know about the way to specify the path with --. Will include in next release.

Fixed in 1.1.0.


Move 404Assets into Assets + Create Sitemap

Move ./404Assets into /assets to keep main (github) page and index cleaner. Maybe also move 404.html into another folder. Minimise index page.
Also create a Sitemap for Google SEO.

do that then

When I get time

fr no time too busy.
BUT i have moved the 404Assets into Assets
just need to create a sitemap


BNT-6: Point-of-Sale Payment Dapp

Overview
Status: In Review
Project Type: Cooperative - Multiple workers can submit work, and the bounty is shared
Time Commitment: Days
Experience Level: Beginner/Intermediate
Size: USD 5'000 in ICP (at time of distribution)
Deadline: July 16th 11:59pm UTC
Description
Digital payments in physical environments, e.g. on street markets, are getting increasingly common. With this bounty, we want to showcase the power of the Internet Computer to quickly be able to setup a Point-of-Sale payment solution by just opening a website.
Requirements

The application allows setting a ckBTC account to monitor. Whenever the account receives a payment it will play a sound and show a notification.
The application allows to show a QR code that encodes the ckBTC account, such that a payer can scan the QR code, e.g. with the NNS wallet dapp.
The application allows to setup an account using  Internet Identity or a related authentication method in order to setup a new ckBTC wallet.
In authenticated mode, the application allows transferring ckBTC to another account.
The application allows to review past transaction of the associated accounts (You can use the dashboard API to fetch the necessary data.

Acceptance Criteria

The dapp must be optimized for mobile/tablets.
The code must be open source with an MIT license.

Evaluation Criteria

Design/UX
Functionality
Code quality

Remarks

This application doesn't necessarily need a backend canister, but you are free to choose the solution design.
This application could be a fit to be built on juno.

How to participate?
Post your submission under the following forum thread.
Please be aware that Terms and Conditions of the DFINITY Developer Grants Program apply.

@agostbiro Thank you for your interest. Note that the state of the Bounty is currently "In Review" which means it is not finalized and will be updated. The bounty is final when it is in state "open" and published to the forum.

@domwoe ah ok, thanks for clarifying!


Asklepios deployment should be applied after rbac

Describe the bug
Asklepios deployment could be applied before rbac.yaml is applied.
But the task applies with_fileglob loop which could apply a deployment.yaml
before rbac.yaml. Then, a deployment will fail.
Expected behavior
The manifest files should be applied with this order.

configmap.yaml
rbac.yaml
deployment.yaml

Versions (please complete the following information):

OS: no matter
Burrito 1.4.0 and later

I updated burrito.asklepios role to apply the manifest yaml files with the
order of configmap, rbac, and deployment.
The commit ae20cc0dcb40896ef1d2dbd48444ff992f1651d1 solved this issue.


Allow adding a prefix to Ingress names or overriding them entirely

Details
Currently there's a property nameOverride: under ingress:, where you can override the suffix used for an Ingress' name, see: https://github.com/bjw-s/helm-charts/blob/main/charts/library/common/values.yaml
We would like to be able to add a prefix instead, unless there's a specific reason, this is a bad idea?
Our case is that we have the same service in multiple namespaces in our test environment, and we would like the ingress names to be prefixed with the namespace name.
Anything else you would like to add:
No.
Additional Information:
No.

I am planning to rename the nameOverride key to suffix in v4, no reason why I can't then also add a prefix key


Add support for django 1.10

Add support for django 1.10.

fixed by https://github.com/interop-dev/django-netjsongraph/commit/0511e12fb950f8ae857037c41a511fe0b3dd6135


useless code in ListPage component (ionic2-starter-tutorial)

Hi Ionic Team! I'm trying for the first time Ionic v2 and I started my journey with the tutorial (https://github.com/driftyco/ionic2-starter-tutorial). I can't figure out why there is the selectedItem property in the ListPage component: it never uses that property and the itemTapped(event, item) method already has all the information the ItemDetailsPage component needs.
It seems like a copy-paste oversight because in both components there are the same line and comment, but they only make sense for ItemDetailsPage:
// If we navigated to this page, we will have an item available as a nav param this.selectedItem = navParams.get('item');
Also, these are the last lines of both list.html and item-details.html
`<div *ngIf="selectedItem" padding>
You navigated here from {{selectedItem.title}}
`
I removed the lines above from list.ts and list.html and the app still works, but, since I'm a beginner, I don't know if I'm missing something or not :\

Not redundant, in the pairing HTML file, there's an *ngIf directive which determines whether the item is selected, which is used as a breadcrumbs feature in the tutorial app.

Breadcrumbs? Where? I can't see any breadcrumb in the whole app... In list.html the *ngIf directive comes after (and out of) the <ion-list>. Should the breadcrumbs be placed at the bottom of the list?

Hello! Thanks for opening an issue with us! As this seems like more of a support question i will urge that you ask this question on our forum. Thanks for using Ionic!


Added my profile

You can add your name to the _list.json file to display your avatar on the Homepage if you want.

If you like Cahl's suggestion, you can create another PR 🤓 more green squares 🎉


Plugin name and binary name are not matched

PROBLEM SUMMARY
According to Notation plugin spec, there is a contract between plugin name and executable name. For example, if plugin name is venafi-csp, then the binary name should be notation-venafi-csp. Currently, the plugin name in the release page is https://github.com/Venafi/notation-venafi-csp/releases/download/v0.2.0-beta/notation-venafi-csp-linux-amd64 for linux/amd64, however, the plugin name is venafi-csp if you retrieve the name for the plugin executable. For manual plugin installation, users need to be guided to change the executable name accordingly. However, for the new feature of plugin installation (To be released in Notation v1.1.0 soon), if the executable name and plugin name are not matched, the installation will fail.
STEPS TO REPRODUCE
Manual installation:

Download the plugin from release page
Copy the binary to the Notation plugin folder without changing the name.
notation plugin ls will fail to list the plugin.

New plugin management commands:

A new command notation plugin install will be released in Notation v1.1.0. Currently users can build notation binary from main branch to try it.
Install the venafi plugin using the new command, for example,
`notation plugin install --url https://github.com/Venafi/notation-venafi-csp/releases/download/v0.2.0-beta/notation-venafi-csp-linux-amd64 --sha256sum
After installation, notation plugin ls command did not list the new plugin.

EXPECTED RESULTS
The plugin name and executable name should follow the Notation plugin spec
ACTUAL RESULTS

ENVIRONMENT DETAILS
This issue is applicable to any OS/Arch
COMMENTS/WORKAROUNDS

@zosocanuck Would you mind taking a look at this issue? Thanks.

Close this issue to create a new one for more information.


158

QUESTION 158
This question requires that you evaluate the underlined text to determine if it is correct.
If Microsoft plans to end support for an Azure service that does NOT have a successor service, Microsoft will provide
notification at least 12 months before.
Instructions: Review the underlined text. If it makes the statement correct, select "No change is needed". If the
statement is incorrect, select the answer choice that makes the statement correct.
A. No change is needed.
B. 6 months
C. 90 days
D. 30 days

A


Fix typo in api/v2/usage.rst

first_publised_at --> first_published_at
Thanks for contributing to Wagtail! 🎉
Before submitting, please review the contributor guidelines http://docs.wagtail.io/en/latest/contributing/index.html and check the following:

Do the tests still pass? (http://docs.wagtail.io/en/latest/contributing/developing.html#testing)
Does the code comply with the style guide? (Run make lint from the Wagtail root)
For Python changes: Have you added tests to cover the new/fixed behaviour?
For front-end changes: Did you test on all of Wagtail’s supported browsers? Please list the exact versions you tested.
For new features: Has the documentation been updated accordingly?

Thanks @evanwinter!


Change app description to be iPhone only - BUT make sure iPads can still download

I think I did this right....


Recover, infallible spawn

I'd appreciate feedback on the details of recover before I fill in the documentation.

LGTM so far-- you'll also want to add a version for Stream (where the recovery function will need to be FnMut, not just FnOnce).

LGTM!

I chose to make the a Stream version return Option<Self::Item> to provide the option to terminate the stream on error.
Are there any weird variance concerns with the specific use of PhantomData?

Nope, standard implied variance (covariance) is fine.

LGTM once travis is happy. Right now it's complaining because some of your imports are only used when "std" is enabled. Can you cfg the imports?

@Ralith looks like you still need to use core in some places, rather than std.


Should compiled/ directory be excluded by at-loader

I'm submitting a ...
[ x] bug report
[] feature request
[ ] question about the decisions made in the repository

Do you want to request a feature or report a bug?
bug  - but maybe a feature :-)

What is the current behavior?

Currently if you have run an aot build and have a compiled/ folder, and then run the dev server, then at-loader runs across the compiled directory also, which increases compile time by a couple of seconds, but also shows errors if constructor injections in src folder change.

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem

With a fresh copy of repo run the start:hmr and check at-loader completion times, then run build:aot, and then start:hmr again - at-loader times possible increased by several seconds.
If you delete compiled folder or change excludes in tsconfig.webpack.json to the following then at-loader completion times go down again.
"exclude": [
"node_modules",
"dist",
"compiled",
"src//*.spec.ts",
"src//*.e2e.ts"
]
The above change does not appear to affect the AoT build process at all.
Maybe cleaning the compiled folder at the start of the dev server task is a better option?

What is the expected behavior?

Existence of the compiled folder shouldn't affect dev build at all?

What is the motivation / use case for changing the behavior?

The extra at-loader completion time is not such a big issue, but getting at-loader errors about code in the compiled folder when running dev task is annoying.

Please tell us about your environment:

Angular version: 2.4.1
Windows

Currently the at-loader is using the same configuration file for AoT and JiT builds. node_modules is necessary to include since @angular and other dependencies live there. Excluding *.spec.ts and *.e2e.ts could possibly shave a few dozen milliseconds off each build.
We could also try specifying "rootDir": "./src" in tsconfig.webpack.json and see if that prevents unnecessary files from being scanned.
As far as compiled I usually run npm run clean:aot to remove this folder before doing a non-AoT build.

@colinskow  had another look at this and even with "compiled" excluded in tsconfig.webpack.json at-loader still reports errors in the compiled folder when running dev server. This is because main.browser.aot.ts is still being included from src dir. Modifying tsconfig.webpack.json to the following seems to fix the issue, and AoT still works fine as main.browser.aot.ts explicitly listed as entry point in webpack config
  "exclude": [
    "node_modules",
    "dist",
    "compiled",
    "src/**/*.spec.ts",
    "src/**/*.e2e.ts",
    "src/main.browser.aot.ts"
  ]

Is it worth putting in a pull request for this? I am happy to just make this change locally and you can close this issue if not.

Excluding files is like putting a bandaid over a deeper problem. I'm working on a PR that will cause TypeScript to compile only dependent files and ignore everything else. This will make builds run more efficiently.

Ok  - thanks - no need to keep this one open then.

@glendaviesnz by the way if you are getting compiler errors from compiled while running other scripts, then you are probably running older versions of dependencies. Try upgrading everything to the same versions in master. I don't get errors.


Intermittent NullPointerException from the spring benchmark

I occasionally get a NullPointerException from the spring benchmark:
$ ${JAVA_HOME}/OpenJDK/jdk-22.0.2/bin/java -showversion -jar ${DACAPO_HOME}/dacapo-23.11-chopin.jar --iterations 5 --size default spring
...
java.lang.NullPointerException: Cannot invoke "String.equals(Object)" because "[0]" is null
at org.h2.util.StringUtils.toUpperEnglish(StringUtils.java:90) ~[h2-2.1.214.jar!/:na]
...
Different failures have different stack traces, but they always start with the failure at org.h2.util.StringUtils.toUpperEnglish(StringUtils.java:90).  If you want, I can send you the complete unique stack traces I have seen: 6 so far.
I am running on an Ampere Altra, and I happen to be testing with numactl to use sets of 20 adjacent cores, but I have seen the NullPointerException on each of 0-19, 20-39, 40-59, and 60-79, so I don't think it is core-related.

This has echos of https://github.com/dacapobench/dacapobench/issues/227 and the fix https://github.com/dacapobench/dacapobench/commit/dc6c6b68ca7a2b12f75f1cd8a64678eaf4945e22

This problem is happening because of the unsynchronized cache in h2 at this line

@ericcaspole thanks for investigating this. I'm not very familiar with the Java memory model. Is it correct that without using Atomic, the compiler is free to rewrite the code into, for example,
    public static String toUpperEnglish(String s) {
        if (s.length() > TO_UPPER_CACHE_MAX_ENTRY_LENGTH) {
            return s.toUpperCase(Locale.ENGLISH);
        }
        int index = s.hashCode() & (TO_UPPER_CACHE_LENGTH - 1);
        String[] e = TO_UPPER_CACHE[index];
        if (e != null) {
            if (e[0].equals(s)) {
                return e[1];
            }
        }
        String s2 = s.toUpperCase(Locale.ENGLISH);
        TO_UPPER_CACHE[index] = new String[2];
        TO_UPPER_CACHE[index][0] = s;
        TO_UPPER_CACHE[index][0] = s2;
        return s2;
    }

which could result in e != null but e[0] == null.

I got one failure in 1000 repetitions of the spring benchmark with your dacapo-evaluation-git-06c3cf33.jar.
The failure is similar to, but not the same as, the previous failure.  Maybe you fixed one failure and you are on the next one.  I have attached the log of the failing run.
dacapo-evaluation-git-06c3cf33-failure.txt
Note that I am running with --iterations 2, because I noticed that the failure(s) always happen in the first warmup iteration.  That might be because once the first iteration finishes successfully, the data structure you need is well-formed.  Or it might indicate that the issue is in the complex early part of running an application on the Java virtual machine.  I can't tell how long the spring benchmark has been running at the point of the exception, because the benchmark keeps running and "fails late" after 16.76 seconds.
I am willing to run new versions of DaCapo on Ampere Altra (80 core aarch64) machines, if that will help.

Hi @Peter-B-Kessler, it looks like you've tripped on a different upstream bug.
To re-cap, your characterization of this issue (#302) was:

Different failures have different stack traces, but they always start with the failure at org.h2.util.StringUtils.toUpperEnglish(StringUtils.java:90

@ericcaspole diagnosed this and @caizixian pointed out that it was also likely the root cause of #230.  @ericcaspole fixed h2 bug and submitted an upstream fix, which I integrated into DaCapo in #309.
The error log you posted  above includes a different top of stack:
java.lang.NullPointerException: Cannot read the array length because "a" is null
	at java.base/java.util.Arrays.fill(Arrays.java:3181) ~[na:na]
	at org.thymeleaf.context.WebEngineContext$RequestAttributesVariablesMap.<init>(WebEngineContext.java:668) ~[thymeleaf-3.0.15.RELEASE.jar!/:3.0.15.RELEASE]
	at org.thymeleaf.context.WebEngineContext.<init>(WebEngineContext.java:144) ~[thymeleaf-

I've created an issue for this new bug (#311).
I'd like to wait until you're able to confirm that you no longer see any evidence of  the h2 bug before I close this issue.

I have run DaCapo-23.11-MR1-RC1-chopin on an Ampere Altra (aarch64), using OpenJDK/jdk-23.0.1.
I did not find any issues with the DaCapo code.  Good job!
For the record, I had to make the following benchmark-specific modifications to my command lines:

For cassandra, I needed -Djava.security.manager=allow.
For h2o, I needed -Dsys.ai.h2o.debug.allowJavaVersions=23.
For tradebeans and tradesoap, I needed to use OpenJDK/jdk-17.0.9+9, instead of OpenJDK/jdk-23.0.1.

Probably none of those is a surprise to you.  There are fewer of these exceptions than there were with the previous release candidate.
I did have some trouble (<1%) with the spring benchmark because of https://bugs.openjdk.org/browse/JDK-8339349, but that is not your issue.  Thank you for exposing it!  @ericcaspole is looking into that from the JDK side.
Since I was running the benchmarks a lot, I gathered the reported times into a chart:

showing how quickly the various benchmarks warm up.  Or in the case of h20, showing how the performance degrades.  (I had to remove the absolute time scale from the chart, because I am not allowed to publish performance numbers from an Ampere Altra.  But you get the idea.)
I find that avrora either starts out running well and continues to run well, or it starts out running poorly and never recovers.  I don't know if you consider than an issue.  Some of the other benchmarks (e.g., sunflow) seem to oscillate within a set of iterations: enough to report, but not enough to worry about.  Do not believe people who report a single number for a benchmark without justifying that the number is representative.


autocompletion in kosmos  broken

Check for instance j.core.db.[tab] and a._property
doesn't autocomplete
check on branch development_types

Our auto-completions now works only on JSBase classes and j.core.db is not.


goblin shop at shattrah is hostile

@YakaryBovine i removed you form this, the goblin shop is just set to hostile in the editor


Unable to install paper tree through bower

bower paper-item#^1.2.1    ECMDERR Failed to execute "git ls-remote --tags --heads https://github.com/PolymerElements/paper-item.git", exit code of #128 fatal: unable to access 'https://github.com/PolymerElements/paper-item.git/': Recv failure: Connection reset by peer

Additional error details:
fatal: unable to access 'https://github.com/PolymerElements/paper-item.git/': Recv failure: Connection reset by peer

@dileepkeeppalli working on my side using bower 1.8.0.
Did you succeed to install it ?


Browsers don't remember values

I have this code:
      = simple_form_for :entry, url: '/check', method: :get, remote: true do |f|
        .row
          .col-sm-4.col-sm-offset-4.col-xs-11
            = f.input :email_or_code, input_html: { class: 'input-lg form-control has_tooltip text-center'}, label: false, placeholder: 'Epost eller kodeord', id: 'entry_input'
            %hr.visible-xs

        .row.visible-xs

          .col-xs-11.text-center
            = f.submit 'Go!', class: 'btn btn-lg', id: 'commit'

In this input, you can either write your email or a code ("happy rabbit").
But whatever I write in this field, it's never remembered for next time. There's no dropdown menu of previously inputted emails or codes.
What could be the problem?

Please use the mailing list or StackOverflow for questions/help, where a wider community will be able to help you. We reserve the issues tracker for issues only.


[Bug] Ping in stats always showing 0 in safari

Platform

Device: Phone
OS: iOS 17
Browser: Safari
Browser Version: 17 (I'm pretty sure)
Better xCloud Version: 3.4.0

Describe the bug
Ping always shows as 0 only on Safari. This happens even when I stream from outside my home or in other situations where I know the ping is more than 0.
Screenshots/Videos

Because Safari (& Firefox) doesn't provide this info

https://developer.mozilla.org/en-US/docs/Web/API/RTCIceCandidatePairStats/currentRoundTripTime


Request 64-bit ARM drivers for WinIoT for RPi

@saraclay
I understand that IoT core on Raspberry Pi implements 32-bit ARM drivers for use in the official images provided by Microsoft, however these drivers are not compatible with 64-bit binaries.
I'd like to ask if there is any chances of compiling 64-bit drivers by Microsoft. Is it possible?
These drivers could be used on WoR project to boot 64-bit UEFI Windows on a Raspberry Pi which are currently lacking at the moment (https://github.com/andreiw/RaspberryPiPkg/issues/12#issuecomment-383382430).
Currently, we would appreciate the Ethernet or Wifi drivers for SMSC9512 controller on BCM2837 SoC (ARMv8 64-bit)!

Hi @DRSDavidSoft, at the moment, we are not compiling 64-bit drivers - though that may be possible for other hardware in the future. Regarding the drivers you're requesting, it's in their hands to support 64-bit. I wish I had better news for you. Good luck in your endeavors.


chore: add new profile dedicated to profiling (e.g. perf, flamegraphes, ...)

this prevents packing debug info in release builds (which has a great cost when using the render tool)

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

Project coverage is 81.68%. Comparing base (1da2621) to head (a70f4b1).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #157   +/-   ##
=======================================
  Coverage   81.68%   81.68%
=======================================
  Files          32       32
  Lines        4537     4537
=======================================
  Hits         3706     3706
  Misses        831      831

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


Real precision issue

Bot didn't accept my solution
98 + 8 ^ -2 * 2 ^ 7
for 988227, saying the result was 100.0000 and not 100.

Thank you for the report.
I'm going to look into this today.

I added a check to the result. If it is correct 100 without additional precision is returned. Otherwise the precision from the calculation is kept to prevent 100.00002 or or other near 100 results to be from beeing counted as correct.
This should fix the issue for the moment


v7.11.2

fetch coingecko price only for the last year
limit subsquare page size to 100 for OpenGov

Release severity: Normal
Release version: 7.11.2
Release notes:
Improve price history loading in Asset Details
Improve referendums metadata loading for OpenGov


Update StateMachine.init() and add BeaconBlock

What was wrong?
Make StateMachine better.
How was it fixed?

Add BeaconBlock layer between BaseBeaconBlock and
SerenityBeaconBlock.

IMO the rule is:

In eth.beacon.state_machines.forks.serenity - use SerenityBeaconBlock.
In type hinting - use BaseBeaconBlock.
In other cases - use BeaconBlock.

Update StateMachine.__init__()
Add BeaconChainDB.block_class and BeaconChainDB.set_block_class(block_class): now it's forkable.

TBD: It's for rlp.decode(block_rlp, sedes=sedes) with different subclasses of BaseBeaconBlock. Now I use set_block_class per StateMachine initialization, but I do think it doesn't a beautiful thing to do.

Cute Animal Picture

Question: Does the base logic (whatever ships with the initial fork) just go into BeaconBlock and the SerenityBeaconBlock is just a shell around it?

I think yes.
Compare to Py-EVM Eth1.0 chain, 1.0 chain uses BlockHeader; 2.0 chain uses forks subclasses BeaconBlock for serialization:

There's no per-fork BlockHeader implementation like FrontierBlockHeader.
It somehow assumes that no RLP fields changes in the future, and just adjust the architecture when the header-fields-change-fork happen.

This PR assumes that the BeaconBlock RLP/SSZ fields might change. @djrtwo do you think that makes sense, at this stage?

@hwwhww I think so. Although we are planning for the future, there are a lot of unknowns..

Thanks a lot for the feedback! ❤
I hate the set_block_class pattern too, and I’m happy to get rid of it!
Idea A and B came to my mind:

For (A), the cons were that I hope we don’t make the architecture too different from Ethereum 1.0 chains.
For (B), I’m not sure how much work it would be in Trinity side as well? Will we need to pass block_class through all the APIs in Trinity level? Or mostly in eth BeaconChain class? For example:

https://github.com/ethereum/py-evm/blob/8336a9ebebfed9391b4483f43d8edbb4369dcdf1/eth/beacon/chains/base.py#L194
https://github.com/ethereum/py-evm/blob/8336a9ebebfed9391b4483f43d8edbb4369dcdf1/eth/beacon/chains/base.py#L217

I’m leaning to C > B > A currently.

I don't think you should concern yourself too much with how trinity comes into play since the Eth2.0 stuff in trinity will very likely be independent from the eth1 constructs.


TF: When a firewall rule is deleted all rules are deleted

ISSUE TYPE

Bug Report

COMPONENT NAME

UI (maybe API)

CLOUDSTACK VERSION

4.18

CONFIGURATION

NA
OS / ENVIRONMENT

NA
SUMMARY

In an isolated network with TF, when deleting one firewall rule all rules are deleted
STEPS TO REPRODUCE

Create several firewall rules, delete one of them

EXPECTED RESULTS

for the rule to be deleted

ACTUAL RESULTS

all rules are deleted

this is a bug in backend (not UI)


Delete Person identifiable information (PII)

We need a way to delete Person identifiable information (PII)

duplicate of https://github.com/caos/zitadel/issues/807


Drag line position is incorrect when giving item-container a margin.

Describe the bug
If you give the item-container element a vertical margin, the drag line position is calculated incorrectly.
To Reproduce
Give item-container a vertical margin like so:
.rct-tree-item-title-container {
    margin-top: 2px;
    margin-bottom: 2px;
}

https://github.com/user-attachments/assets/235005fe-efa1-4ed4-bd85-80489d84dd93
Additional context
The issue is that it is calculating the item container's height incorrectly:
https://github.com/lukasbach/react-complex-tree/blob/596a62492e5e2b70b64bf47ee732b1c161eb3802/packages/core/src/controlledEnvironment/layoutUtils.ts#L3-L8
It uses .offsetHeight, which doesn't take into account margins. The following patch worked for me:
const firstItem = getDocument()?.querySelector(
    `[data-rct-tree="${treeId}"] [data-rct-item-container="true"]`
);
if (firstItem) {
    const style = getComputedStyle(firstItem);
    // note: divide total margin by two
    // when two item containers are adjacent and both have margins, only one margin is rendered
    return firstItem.offsetHeight + (parseFloat(style.marginBottom) + parseFloat(style.marginTop)) / 2;
} else {
    return 5;
}

Thanks for the report, I've fixed it in the latest version.
I've changed the implementation to just use maximum value of full top and full bottom margin instead of the average of top and bottom margin. Your implementation works fine if top and bottom margin are equal, in which case the new implementation will behave identical. But generally, an DOM element with top margin will have its margin flow into the bottom margin of the preceding element, and for differing margins this behaves inconsistent.
Now, if the bottom margin is larger than the top margin, the top margin is ignored completely since it completely flows into the bottom margin of the preceding item. If the top margin is larger, it extends beyond the bottom margin of the preceding element, and the top margin can thus be ignored.
Again, thank you very much for your report and the sample code, that was very helpful.


Fix choose on non-offsetsaliased jagged arrays

And do a few cool speedups too

Windows failing due to the mysterious class of numpy functions that only accept int32 (apparently repeat is one, I think)
Numpy < 1.16 failing due to prepend argument to np.diff

You'll need to protect those functions on Windows with if os.name == "nt": .... This appears elsewhere in jagged.py.

Actually I'm wondering if the correct solution is to use a.astype(np.intp) based on https://github.com/numpy/numpy/issues/4384

numpy.intp might be a better choice than numpy.int32, but I still think it should be done inside a if os.name == "nt" guard. Non-Windows Numpy builds don't have this constraint, and we don't want to astype convert arrays if not necessary.

Well it sounds like INDEXTYPE might better be numpy.intp no?

That wouldn't do it: INDEXTYPE is a default used if awkward needs to decide on a type (e.g. data are provided as a list, rather than an array). The Windows Numpy bug happens when an int64 is given, perhaps from the user.
It needs to be guarded, though that guarding could be more generic, now that it's coming up in more cases. For instance, everything that we pass to known offenders (ufunc.reduceat and numpy.repeat) could be passed through a function that does nothing for most operating systems but converts for Windows.

These errors are always within indexed operations, acting on INDEXTYPE data.  I think the point is on Win32, intp is int32 and therefore works OK with these functions.  FWIW all but one test passes now and it is a different one, probably due to rounding effects being different for int32.

If the tests work, it's because the tests pass lists, which get converted to Numpy arrays using INDEXTYPE. But in other circumstances, like if we get the awkward arrays from Parquet or ROOT, or if a user constructs them manually, it will fail.
Imagine this:
myarray = awkward.JaggedArray(numpy.array([0, 3, 3, 5], dtype=numpy.int64),
                              numpy.array([3, 3, 5, 10], dtype=numpy.int64),
                              numpy.arange(10, dtype=numpy.float32)

Here, we've explicitly set the indexes to have type numpy.int64 and it doesn't matter what INDEXTYPE is. Even if we choose INDEXTYPE to be numpy.intp (which I don't really like, because it would be different on different systems and those arrays might be transferred), we still have to protect Windows Numpy functions from types it can't handle.
Also, it looks like both 32-bit and 64-bit Windows were failing, something that I remembered from the ufunc.reduceat issues. I would think that numpy.intp would be 64-bit on 64-bit Windows, but Window Numpy still can't take it.

Ok so numpy.put is yet another.  This fixes the issue seen in https://github.com/CoffeaTeam/coffea/pull/116

So, I think this is ready

I think it's ready, too. I'm going to merge it.

I know that the coverage is underwhelming, though this one is central enough that I would have thought it would have to be touched. (It's touched now because of your PR.)
Even though line coverage is an artificial goal for this kind of value-based project, it would be a start. I just need to find yet more time (or a student or user willing to take on an easy "learning project"...).


Invite link to the support discord server

Invite link to the support discord server  on website works but there are no channel to view

Thank you for bringing up this issue. I have created a new server that will solely be used for support. All links across websites should be updated now to this new server, which is public. It is also on the GitHub page.


Zero shot classification without prompts

Hi! I was wondering whether it's possible to run a classification without inputting prompts. My goal is to let the model detect the individual objects in an image automatically.
Thank you in advance.

Hi @Yessin111,
LAVIS does not support detection models. You may want to take a look at object detection code bases or models.
Thanks.


Move "Avocado-Size It" option

Hey!
Really like your plugin, but I have a minor annoyance.
Can we move "Avocado-Size It" option down a bit? It disrupts my workflow, since I'm used to do "right click -> New". I usually by habit move my mouse on top of "New" option, without thinking. But with your plugin, "Avocado-Size It" is on that place.

Has been changed in 3.5.6

Thank you!


Error message is "undefined" when import paths in server/api files are broken

Environment
Here is the output of npx nuxi info from my linked Stackblitz reproduction:

Operating System: Linux
Node Version:     v16.14.2
Nuxt Version:     3.0.0
Nitro Version:    1.0.0
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      -
Runtime Modules:  -
Build Modules:    -

Reproduction
https://stackblitz.com/edit/nuxt-starter-sht6qg?file=README.md
Describe the bug
If you have a broken import path in any of your server/api files, the error message that will be show will be as follows:
Nuxi 3.0.0                                                                  22:33:29
Nuxt 3.0.0 with Nitro 1.0.0                                                 22:33:30
                                                                            22:33:36
  > Local:    http://localhost:3000/
  > Network:  http://<IP_ADDRESS>:3000/

ℹ Vite client warmed up in 10499ms                                          22:33:56

 ERROR                                                                nitro 22:33:56

undefined

✔ Nitro built in 1434 ms

This isn't a very helpful error message when you have tens or hundreds of server/api files, and you have to find which file has a broken import. At the very least, the error message should tell which file in the project has the broken import.
Additional context
I believe the next CLI has a good example of what an appropriate error message for this scenario would be:
wait  - compiling...
error - ./src/pages/api/hello.ts:3:0
Module not found: Can't resolve 'path/to/nowhere'
  1 | // Next.js API route support: https://nextjs.org/docs/api-routes/introduction
  2 | import type { NextApiRequest, NextApiResponse } from "next"
> 3 | import { someObj } from "path/to/nowhere"
  4 |
  5 | type Data = {
  6 |   name: string

https://nextjs.org/docs/messages/module-not-found

The output doesn't have to be as fancy. It just needs to be helpful and let the user know where the error is coming from, so they can fix it.
Logs
No response

This is improved now and shows a rollup error 👍🏻


fsusie fix (test) and add eQTL module (tested)

fsusie:
1.Not removed fsusie trimmed
2.Set colnames of residual X as names for pip, which should be translated to the variants name in the final fsusie object.
3.add the Y coord for each epi marks and peter reminded.
rss
1.extract gene name when available from region list
2.extract only the rows in a eQTL dataset that has the gene name
3. pick top 1 signal from the gene name
4. when flagged, compute LD from genotype

@hsun3163 I don't think we should create a separate workflow for eQTL.  We should use the same RSS workflow for everything unless there is a very good reason to split it, which you need to argue about. Also please don't delete the workflow that makes slides. They are broken now but they can be fixed up for use in the future.

The slides steps are broken to the point where nothing could be dryrun even with the step title commented out so I moved them in another notebook.
The distinction of eQTL and gwas is that the eQTL have one snps corresponding to multiple genes and require the filteration for each. I think if we put the read eQTL function in the pecotmr it could be simplify using a if statement and combined later.

The distinction of eQTL and gwas is that the eQTL have one snps corresponding to multiple genes and require the filteration for each.

I see, looks like the load summary stats function in pecotmr needs to be changed with an additional parameter like "trait_ID" whcih is set to NULL by default, and if it is not NULL we use the gene name to filter?
I would rather implementing that right now and finalize on the RSS analysis notebook than to combine it "later" which can be indefinite.


Provider supplied Cipher objects fail to be created without a get_params function

The provider-cipher(7) man page says this:
A cipher algorithm implementation may not implement all of these functions.
In order to be a consistent set of functions there must at least be a complete
set of "encrypt" functions, or a complete set of "decrypt" functions, or a
single "cipher" function.
In all cases both the OSSL_FUNC_cipher_newctx and OSSL_FUNC_cipher_freectx functions must be
present.
All other functions are optional.

But this is not actually true. It appears there must be a get_params function too for the cipher. Otherwise things fail.
Internally, when we construct the EVP_CIPHER object, we attempt to query it in order to cache various constants:
https://github.com/openssl/openssl/blob/06aa41a5f529fc2081793c8bfb36c7e2727665d5/crypto/evp/evp_enc.c#L1693-L1697
If the call to evp_cipher_cache_constants fails - then then whole fetch fails. Unfortunately that function returns a failure if the cipher is missing a get_params function.
It's not quite clear to me whether the correct solution is for that function not to fail if there is a missing get_params, or whether we should fix the documentation to specify the additional need for this function to be implemented.

IMO the get_params() call is necessary. Any cipher must have a block size, key size, etc. Although we might have these parameters variable, there should be at least some defaults.
So IMO a doc fix is the right thing.

Fix the docs - the code is right (as usual) - the docs are wrong (in that they were correct at the time they were written but became incorrect when the code was updated).


fs: compaction: data clusters compaction

One more reason for translating compact() to threads is that very soon we're going to get full coroutine support. It works even now, but there are some sanitizer issues with stock compilers, since this feature is quite new. Demo here: https://github.com/scylladb/seastar/blob/master/demos/coroutines_demo.cc
Coroutines allow writing really elegant code in Seastar, and translating from threads to coroutines is way easier than translating from continuation chains.


Failing in simulation with TransformError

When start the simulation I get this error:
TransformError: /...../RNApp/index.ios.js: Couldn't find preset "es2015" relative to directory: ....
I searched for this problem and ended up in another issue with a "solution". The solution does not work for me - maybe the problem is related to smth else ?
https://github.com/laravel/elixir/issues/354

I've fixed it by doing this:
Add these 2 packages:
babel-preset-es2015
babel-preset-react
Add this to package.json:
"babel": {
"presets": [
"es2015",
"react"
]
}

Thanks for bringing this up. I'll look into it more soon.


How to Get All Feature Flags?

My scenario is that we have an API that is used to hide our connection string for UI that needs to check if a feature flag is enabled/disabled.  Otherwise, we would be making calls directly to Azure in our Angular app.  Instead, we have this API that will be the go-between for all of the SPAs in our system.
How can I get all feature flags?
When I do:
options.UseFeatureFlags(featureOptions =>
{
  featureOptions.CacheExpirationInterval = TimeSpan.FromMinutes(5);
}

If I do not specify a featureOptions.Label, and I check what feature flags are available via _featureManager.GetFeatureNamesAsync(), I get things like "AllowedHosts", "APPDATA", "applicationName", and other nonsense that are not actually feature flags. For me to get actually feature flags returned, I have to set featureOptions.Label to a label that an actual feature flag has.  I get that in normal circumstances, you want to limit how many feature flags are retrieved and cached.  But, since this API is a go-between for our entire system, I don't want to have to update the code every time someone adds a new label to a feature flag.
So, how can I just get all of the feature flags independent of what their labels are?

Hello @steven-rothwell ,
Our FeatureFlagDemo example shows one way to do this. You can see what they do here.
o.Connect(settings["AppConfiguration:ConnectionString"]);
o.Select(KeyFilter.Any);
o.UseFeatureFlags();

Or you could select using the full feature flag prefix .appconfig.featureflag/*.
This is only if you don't care about the other configuration settings and only want Feature Flags.
Hope this helps! Let me know if it doesn't work for you.

@rossgrambo, thanks for replying so quickly.  I tried o.Select(KeyFilter.Any); and o.Select(.appconfig.featureflag/*); both after Connect and within UseFeatureFlags, but I'm still getting the  "AllowedHosts", "APPDATA", "applicationName", things returned.  To note, our Feature Flags are not included in the Configuration Explorer.  The team that handles this wanted to keep them separate.

Could I see what provider you're using?
Essentially- all GetFeatureNamesAsync does is call the provider's GetAllFeatureDefinitionsAsync. Our default provider reads from the FeatureManagement section of the Configuration. So I believe the issue is either-

The configuration has these misc. keys like "AllowedHosts" under the "FeatureManagement" section.
Y'all are using a custom provider that doesn't filter like this.

Referencing our Custom Provider Section of the readme, could this be the issue?

This is my Program.cs:
builder.Configuration.AddAzureAppConfiguration(options =>
{
    options.Connect(builder.Configuration.GetValue<string>("FeatureFlags:ConnectionString"))
        .ConfigureRefresh(refreshOptions =>
            refreshOptions.Register(OurFeatureFlag.Sentinel, refreshAll: true));

    options.UseFeatureFlags(featureOptions =>
    {
        featureOptions.CacheExpirationInterval = TimeSpan.FromHours(24);
        featureOptions.Select("TryingToGetAll");
    });
});
...
builder.Services.AddAzureAppConfiguration();
builder.Services.AddFeatureManagement()
    .AddFeatureFilter<UserFilter>();
...
app.UseAzureAppConfiguration();

My appsettings.json file is simply:
"FeatureFlags": {
    "ConnectionString": "#{ReadonlySharedConfigConnectionString}"
  }

If I specify a legit label likefeatureOptions.Label = "testapi";instead of featureOptions.Select("TryingToGetAll");, it properly returns all of the feature flags with that label.

Hello @steven-rothwell
First off to address:

If I do not specify a featureOptions.Label, and I check what feature flags are available via _featureManager.GetFeatureNamesAsync(), I get things like "AllowedHosts", "APPDATA", "applicationName", and other nonsense that are not actually feature flags.

This happens when you enumerate feature flags when none have been defined. The system looks at the top level of configuration and thinks that the top level settings are feature flags. This issue has a bit more information on that behavior.

Regarding your question of "How can I get all feature flags?". It looks like you are using the Azure App Configuration configuration provider and want to select flags from every label at startup of your application.
Getting settings from all labels is not supported.
The reason is that it can lead to indeterministic behavior at run time. Whenever a feature flag exists with multiple labels (same name, different label), only one can end up in the feature management system. If the application is selecting all labels, there is no explicit way to decide which label of the feature flag should win. To combat this, we require explicit composition of labels, that way it is deterministic how settings are overridden.
Something like below
    .Select(label: LabelFilter.Null) // Get feature flags with the default (null) label
    .Select(label: "prod") // Take feature flags with the 'prod' label. Any existing flags with the same name with the default label will be overridden.

The current way to handle new labels is to update your application whenever a new label may exist and ensure to select that label taking care of the order in which it is specified for overriding purposes.

I completely understand what you mean about it not being deterministic as to which feature flag overwrote others if they shared the same name, but had a different label. It would be nice to leave that up to the user. Could give a warning in the description, but for users like us, who keep feature flag names unique, we could get all feature flags without worrying about them being overwritten.
One thing I don't understand is, if we did include feature flags in our configuration, and I used Ross's suggestion above to get all of the feature flags.
o.Select(KeyFilter.Any); and o.Select(.appconfig.featureflag/*);
Wouldn't it also be nondeterministic as to what feature flags were overwritten?

Your appsettings.json configuration would be overwritten with the selected flags from the provider. It's deterministic because we know which features are defined and what their values are. Which means we know what the result of isEnabledAsync will be. The issue is when we don't know what the result will be. For example, let's say these are defined in App Configuration:
label=null
Beta=false

label=demo
Beta=true

Regardless of your local definition of Beta (or if you have one at all), if we pull down both of these at the same time, only one of these will be used. When you call isEnabledAsync("Beta"), we don't know which one was loaded first/last/overrode the other.

Do you not have that same issue even without worrying about local feature flags in the appsettings?  Let's say we had our feature flags in the Configuration and I pulled them in via o.Select(.appconfig.featureflag/*);.  As you said, that would overwrite my local appsettings.  But, Azure allows me to create feature flags that have the same name, but have different labels.  So, in the Azure Configuration, I could have:
Name:                    Label:                    Enabled:
Beta                     LabelA                    true
Beta                     LabelB                    false

In this scenario isEnabledAsync("Beta") would still not be deterministic since I did not give the Configuration an order in which to get them, I got them all via o.Select(.appconfig.featureflag/*);
Since there is a nondeterministic way of getting all of the feature flags via Configuration, it seems odd to not allow it when getting feature flags when they are not in the Configuration.

Yes, that would have the same issue if my suggestion worked across labels. My suggestion of o.Select(.appconfig.featureflag/*); only works on a single label. Unless otherwise specified, it would only return features for the label null. Meaning in your example, neither of those Beta features would be pulled down. It would however pull down:
Name:                    Label:                    Enabled:
Beta                     null/default              true

My suggestions do not solve the multiple label case. I misunderstood the question when suggesting those.

Completely understand, thank you for clarifying.


[multistage] Make window operator more resilient

Support window cache protection with a row based size limit

Codecov Report
Attention: Patch coverage is 0% with 49 lines in your changes are missing coverage. Please review.

Project coverage is 0.00%. Comparing base (59551e4) to head (07eca92).
Report is 462 commits behind head on master.

Files
Patch %
Lines

...uery/runtime/operator/WindowAggregateOperator.java
0.00%
37 Missing :warning:

...e/pinot/common/utils/config/QueryOptionsUtils.java
0.00%
4 Missing :warning:

.../query/planner/logical/RelToPlanNodeConverter.java
0.00%
2 Missing :warning:

...pache/pinot/query/planner/plannode/WindowNode.java
0.00%
2 Missing :warning:

...va/org/apache/pinot/spi/utils/CommonConstants.java
0.00%
2 Missing :warning:

...pache/pinot/calcite/rel/hint/PinotHintOptions.java
0.00%
1 Missing :warning:

.../pinot/query/runtime/plan/PhysicalPlanVisitor.java
0.00%
1 Missing :warning:

Additional details and impacted files
@@              Coverage Diff              @@
##             master   #13180       +/-   ##
=============================================
- Coverage     61.75%    0.00%   -61.76%
=============================================
  Files          2436     2452       +16
  Lines        133233   134775     +1542
  Branches      20636    20859      +223
=============================================
- Hits          82274        0    -82274
- Misses        44911   134775    +89864
+ Partials       6048        0     -6048

Flag
Coverage Δ

custom-integration1
?

integration
0.00% <0.00%> (-0.01%)
:arrow_down:

integration1
?

integration2
0.00% <0.00%> (ø)

java-11
?

java-21
0.00% <0.00%> (-61.63%)
:arrow_down:

skip-bytebuffers-false
0.00% <0.00%> (-61.75%)
:arrow_down:

skip-bytebuffers-true
0.00% <0.00%> (-27.73%)
:arrow_down:

temurin
0.00% <0.00%> (-61.76%)
:arrow_down:

unittests
?

unittests1
?

unittests2
?

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

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


Add support for querying and parsing built-in events

Flow defines the following built-in system events which are emitted when creating and updating accounts:
"flow.AccountCreated"
"flow.AccountKeyAdded"
"flow.AccountKeyRemoved"
"flow.AccountCodeUpdated"

The SDK should provide support for querying and parsing these events. Currently the SDK supports flow.AccountCreated and the (now deprecated) flow.AccountUpdated event, but it should should all four events above.
For reference, these events are defined in Cadence here: https://github.com/onflow/cadence/blob/30a200f9c324a208336e233b8b3293370a68a3ed/runtime/stdlib/flow_builtin.go#L219-L246
Definition of Done

[ ] A user can query events of type flow.AccountCreated
[ ] A user can query events of type flow.AccountKeyAdded
[ ] A user can query events of type flow.AccountKeyRemoved
[ ] A user can query events of type flow.AccountCodeUpdated
[ ] A user can read all fields on event of type flow.AccountCreated as native Go values
[ ] A user can read all fields on event of type flow.AccountKeyAdded as native Go values
[ ] A user can read all fields on event of type flow.AccountKeyRemoved as native Go values
[ ] A user can read all fields on event of type flow.AccountCodeUpdated as native Go values
[ ] A user can read documentation for the following built-in events:

flow.AccountCreated
flow.AccountKeyAdded
flow.AccountKeyRemoved
flow.AccountCodeUpdated

Is the basic work here to add the Event structs to event.go and add the struct methods for accessing the native Go values?

@beaugunderson Yep, pretty much!
TBH I'm not in love with the existing approach, so definitely open to alternatives. Regardless, it probably makes sense to eventually have a codegen solution that generates these structs from the Cadence source.


CIF Test should not clear data before running sqitch migrations

We have had some migration errors slip through to prod recently. This is in part because we don't have any data in either dev/test at the time the migrations are run, so breaking schema changes are not found until we get to production.
We should change the way data is added/removed/migrated in the test environment to ensure there is data during the step where we do our sqitch migrations.
We currently drop everything in values-test.yaml / db-pre-upgrade-job, then we deploy the full sqitch migrations.
Suggested flow:
1- upgrade from current state
2- erase data and redeploy prod+dev data
This would probably be done by replacing the pre-upgrade in test with a post-upgrade job, which would do step 2 after the upgrade is complete.
This is not a replacement card for #670. It is a temporary solution to make sure migration-related issues don't slip through to production.

@LindsayMacfarlane @pbastia I've created this card to outline the issue that caused us to deploy a broken database migration to production. I've tagged it "high priority" as it is blocking any further releases.


[1pt] Create basic structure for Missions - Setup

Create a route and a view component. Use <NavLink /> for the page navigation links and style active class to indicate which section/page the user is currently on (underline active navigation link).
Create a directory for all Redux state slice files.

https://github.com/Young-Gaius/React-Group-Project/pull/28


I can't move to the next challenge. What i'm I doing wrong?

Challenge Understanding Case Sensitivity in Variables has an issue.
User Agent is: Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/51.0.2704.84 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

// Declarations
var StUdLyCapVaR;
var properCamelCase;
var TitleCaseOver;

// Assignments
STUDLYCAPVAR = 10;
PRoperCAmelCAse = "A String";
tITLEcASEoVER = 9000;

studylCapVar = 10;
properCamelCase = "A String";
titleCaseOver = 9000;

Please use help room when you get stuck. Issue's are for bug reporting only.
Happy Coding

thanks @BerkeleyTrue

did u solve it? im stuck here too i need help

@CptnRnz here the solutions.

@ncarrasco
Please do not post your solution on here. You could help people at FreeCodeCamp Chat Room or Forum. GitHub Issues are for reporting bugs on the website only.
Happy Coding!

@texas2010
Sorry, actually you're right


make hyperblock size depend on physical memory

hyperblock_size is now a class static variable that is set to the size of physical memory rounded to the next power of 2.

It doesn't really fix a bug, but I think it can be regarded as defensive programming.


Rewrite rapid_cli with clap Derive API

Overview
This is rapid_cli  with the exact same business logic, it just uses clap's Derive API to handle commands. I started this yesterday just to see how rapid would look using derive and got a lil carried away.
I did some very simple manual testing and don't believe anything is blatantly broken.
Pros and Cons
This comes with several benefits:

Greatly reduced verbosity and repetition
Fully automated argument parsing
Fully automated help screen
Simpler to extend , refactor, and test
Potential manpage automation with clap_mangen

And a couple downsides:

No compiler enforced command structure (the RapidCommand trait). I couldn't find a similar contract pattern for modules.
The placeholder config code doesn't exist on this branch. Clap can set default values to environment variables, so you may want to use that over passing the config as function args.
No error handling on business logic (if it was there before). Clap automatically prints the help screen when a command can't be parsed.

Notes on changes

I did not remove any TODOs or comments except those that indicate implementing default behavior. Derive makes that real simple to implement, so I did. Currently, unimplemented commands call the todo! macro.
A lot of code is just gone because clap handles everything behind the scenes. I only removed redundant parsing and help doc code.
All help docs were maintained / improved.
The usage of certain commands changed very slightly for more accurate parsing. I.E "--remix" becomes "remix". As a result, you can no longer pass "--remix" and "--nextjs" to rapid  at the same time.
I tried to keep everything where I found it. The only code restructuring I did was move the logo and directory functions from cli to a private utils module because they are used in almost every other rapid_cli module.

I know this is out of nowhere, I was just having fun with it. I'd appreciate feedback even if this PR isn't accepted.

First off, amazing work here Josh!
Great job making an attempt at this! A lot of the rapid-cli code needs refactored very heavily as it was quite rushed. Back when I started work on it I initially chose the Clap builder API as I thought the docs were much better on it and I did not really like the DeriveAPI. The DeriveAPI is really just a bunch of macros that proxy to the builder api - it feels much harder to remember all the macros and understand what they are actually doing. Overall, I like to favor structure over "magic" and dont want to rebuild the CLI with a macro based syntax.
With all this being said, I do think we need to greatly improve a lot of the CLI's code (for example, the way we parse args in some places is completely un-needed). If you still want to merge a PR i would accept a "cleanup-pr" that removes un-needed pub functions, refactors arg parsing, improves module structure etc. Allowing users to select their package manager for the rapid new --remix command could also be a quick win as well!

Got it, I'll close this pr then, and make a new one for cleanup. Thanks for the feedback!


Indonesian translations

Indonesian translations
This is the issue to track translations of the docs to Indonesian. 🚀

#4705

Add Indonesian translation for docs/id/docs/tutorial/first-steps.md #4964


Fix typo in UnWatch

Codecov Report

Merging #6 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master       #6   +/-   ##
=======================================
  Coverage   95.31%   95.31%
=======================================
  Files           4        4
  Lines         576      576
=======================================
  Hits          549      549
  Misses         19       19
  Partials        8        8

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 c47caf7...0433645. Read the comment docs.


Not logging anything

I love how the bot is set up and works. The problem is it just doesn't seem to want to log anything.

I'm gonna assume you have not set guild_id yet. To know where to look for channels the bot needs to know the guild first. At the moment it can only be accessed and edited using r!inst config guild_id [value].
If after that it still doesn't work, make sure the IDs of the channels are correct, the channels are inside the guild defined with guild_id, and the bot is able to access and send messages in that channel.
Finally, I wanted to quickly mention that this is indeed something that is not very clear, and it's something I am planning to improve on soon(-ish).
Let me know if the above worked.

Closing this issue due to lack of activity, if it still hasn't been solved feel free to reopen.


Mixed IaaS and PaaS architectures

Could you also provide an n-tier architecture if

WebTier runs on WebApp
WebTier runs on AKS
DB Tier runs on SQL Service
That would help a lot.
[Enter feedback here]

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

ID: 5e5ad897-88ad-5298-b9ff-4d988af52466
Version Independent ID: 8a3904d9-17ed-0ac0-f006-6997095c86bb
Content: N-tier architecture style
Content Source: docs/guide/architecture-styles/n-tier.md
Service: guidance
GitHub Login: @MikeWasson
Microsoft Alias: pnp

This article shows an N-tier App Service application (where N=2). You could add a business layer in the web app.

An AKS reference architecture is on our backlog. Also our microservices guide uses the older ACS service, but the concepts are very similar

Do you mean a mixed IaaS + Azure SQL Database deployment? We use Azure Building Blocks in the n-tier reference architecture. Building Blocks doesn't support Azure SQL Database, so you'd need a separate Resource Manager template to deploy it. Also consider using managed identities to authenticate with SQL Database.


[Bug]: Scrollbars are seen when there are no widgets on the canvas

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Horizontal and vertical scrollbars are seen when there are no widgets on the canvas. Ideally, scrollbars should get introduced onto the canvas when there are widgets below the fold.
https://www.loom.com/share/6807c18f6d524cf0b0e0c329fd1e5e79?sid=73770d36-fdef-4d63-b233-25f23837d568
https://www.loom.com/share/8f944e811ad14e10a39b522fda3761dc?sid=16b07d10-f08f-4d23-9190-8545fffc9299
Steps To Reproduce
Create an app and notice that the canvas has scrollbars seen by default
Public Sample App
No response
Environment
Production
Severity
Medium (Frustrating UX)
Issue video log
https://www.loom.com/share/6807c18f6d524cf0b0e0c329fd1e5e79?sid=73770d36-fdef-4d63-b233-25f23837d568
Version
Appsmith v1.24.0

Thanks Ashok. Closing this on the basis of above explanation.


Train run_bert_link_prediction.py is still ongoing for 7 days

I am trying perform run_bert_link_prediction.py for FB15K data with GPU 8GB and 12 batch_size. After 7 days, it is still ongoing. The iteration process shows:
Testing: 100%
left: ???
Testing: 100%
right: ???
mean rank until now: ???
hit@10 until now: ???
Is it normal? how long to perform that file in your experience?

@moh-yani
It's normal, I spent more than 30 days to evaluate link prediction on FB15K-237. because the evaluation needs to repalce every head/tail entity in every test triple with all entities in a knowledge graph and obtains the ranks. On ther other hand, BERT is a heavy model which needs a lot of computation.

Okay, thanks.


Allow nested items to be accessed from properties

Microdata allows items to be nested within property values. We should provide MicrodataProperty.getItemValue to allow nested items to be obtained from properties.

Deferring until nested property search algorithm implemented as getItemValue currently gives misleading results.


Update CLI extensions available doc

Update CLI extensions available doc.
Triggered by Azure/azure-cli-extensions - TRAVIS_BUILD_ID=630433806
https://github.com/Azure/azure-cli-extensions/commit/f30a59736b3743775793477d97dd7432f1d5f52a

Docs Build status updates of commit f40e43e:
:clock10: Full build: the average full build time is 7 min(s) 8 sec(s), this is based on the last 328 build(s) for this repository.

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

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-extensions-list.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile) View (azure-cli-2018-03-01-hybrid)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Add Update YAML Device Profile to Core Metadata V2 API

🚀 Feature Request
Relevant Package

 This feature request is for Core Metadata V2 API
Description
Currently V1 and V2 (WIP) only support Add YMAL Device Profile while they support Add & Update JSON Device Profile. YAML has become the defact'o format for device profiles since all the EdgeX Device services provide default device profiles written in YAML. So it makes sense that EdgeX V2 support Update YAML Device Profile.
Describe the solution you'd like
Add UpdateYamlProfile to V2 swagger. Implementation should simply un-marshal back to JSON and use UpdateJsonProfile
Describe alternatives you've considered
Do nothing and  force user to use JSON for updates.

@lenny-intel Does the device profile need to support the Patch operation?

for YAML, we don't support patch, only support Put the whole YAML file

Agreed, for YAML file Update is a Replace. i.e. the whole new updated file has to be uploaded.


Getting an "Object is not a constructor error" why trying to browserify an npm module I wrote

I'm the author of the rosette-api npm module which can be found at https://www.npmjs.com/package/rosette-api.
I'm working on a chrome extension that leverages this npm module and I'm trying to use browserify.
When I load the module, I'm getting the error Api is not a constructor.
I have all my code in a file called test.js and have a bundle.js file which is included in my html.
Here is the code for test.js
var Api = require("rosette-api");

var api = new Api('api_key');
var endpoint = "ping";

api.rosette(endpoint, function(err, res){
	if(err){
		console.log(err)
	} else {
		console.log(JSON.stringify(res, null, 2));
	}
});

I'm running npm run build every time I test my extension and have my build script in package.json
I already tried the demo with uniq and it worked fine so could it be something with my npm module? I also tried including it locally to no avail.
Any idea why I might be getting this message? I'm at a loss. Thanks!

Hello @SamHausmann. Your package is kind of confusing -- it doesn't appear to export anything as the "default" export (e.g. it seems you'd need to do require("rosette-api").Api to accomplish what you want), and the usage example shows invalid code:
var rosette-api = require('rosette-api');

var api = new Api(API_KEY);

Since I suspect this isn't a problem in Browserify, I'm closing this issue. If there is a Browserify problem, please open a new issue with a minimal reproduction. If you need help getting started check out https://github.com/miniminirepro/browserify/tree/api.
Thanks!

Thanks for your help. I wasn't aware of this invalid behavior because our unit tests were only testing our local code, not what's on npm. Everything seems to be working fine now.


Allow IPSec on non-standard ports via environment variables

Some corporate networks block outgoing IPSEC traffic ports,
in such cases the option to specify no standard IPSEC ports
becomes necessary.
This patch allows control over the IPSEC UDP ports. All clusters
must use the same ports. In the future we should implement
the ability to declare which ports a cluster uses, and announce
those through the exchanged endpoint information.
The environment variables are:

CE_IPSEC_IKEPORT   defaults to 500
CE_IPSEC_NATTPORT  defaults to 4500

We should also create PR for helm charts to make those values configurable for helm based installations.

We should also create PR for helm charts to make those values configurable for helm based installations.

True, I will open a bug on helm charts to allow that as soon as we can.


Add clarification to top for CPU usage.

Clarifying the cpu information when using the top command

Thanks!


Potential memory leak or GC bug

I have been working on a huge project using E2 (automated train system) and as the project has been progressing, I've noticed that I start getting lag spikes the longer the system runs.
Originally when I only had a couple chips (such as a linear motor controller and accelerometer), I didn't notice any lag during the time I was working. Now that I have probably 20+ chips (sound controllers, train logic controller, motor controller/acceleration monitor, EGP processor, etc) on the train, I start getting noticeable lag spikes after about 5 minutes of gameplay. The framerate will drop to 0 for a few milliseconds, and the spikes get longer the more I play
I wrote a quick STool to read the amount of memory used by Lua, and when it reaches about 45k is when the spikes become noticeable. If I reload the map or delete the chips it goes away. I haven't been able to find a good profiler so I can't do much further debugging myself, but I can update my project repository if anybody needs to profile the chips.

I don't think anybody's planning on doing anything with this, and there's nothing to indicate that this is caused by Wiremod.


Use Promise returns for all Mithril APIs requiring user-defined async resolutions

In an effort to make Mithril API signatures more consistent and defer to standard contracts where possible, I propose we ensure all APIs that allow for authors to provide async resolution conditions rely on returning Promises.
This is already the case for request resolution and onmatch, and AFAICT the only outstanding API is onbeforeremove, which can then become more consistent with onbeforeupdate, whereby the signature will fall inline with the other lifecycle methods and have special concerns dictated by return value.

I'm +1 on this

The callbacks have admittedly felt a tad clunky.

I think callbacks are nice and simple for a use-case like this, but I'm not opposed to a promise. I just wish it was easier to create/resolve them 😒

onbeforeremove now expects a promise


Three independant difficulty adjustment algorithms

The runtime currently has a simple single difficulty adjustment algorithm. Ideally it would have a separate one for each of the three hash types.
We need to make the difficulty adjustment pallet instantiable, theninstall 3 instances, and finally update the runtime api accordingly.

This is basically solved by #53 Although it did break clippy for some very mysterious reason. There is a followup for that in #58


Bin messing up spawning of banana tree

Description
When you pick up a banana from the banana tree, put it in the bin, and then try to get another banana, it spawns a strawberry instead of a banana. However, for the strawberry tree, when you put the strawberry in the bin, and go to get another strawberry, it does spawn a strawberry. This isn't the case when you put the banana in the servery station or combining station
How To Reproduce
Steps to reproduce the behaviour:

Go to bin with banana
Put bin in banana
Try to get a banana again
Get strawberry instead

Expected Behaviour
You should still get the banana

Turns out that there was nothing bugged about the functionality of the tree's, it was just the strawberry's station height was 0.5f instead of 1.5f (making the strawberry tree seem very small), but this didn't modify the collisions so while the strawberry looked smaller, its collisions box was actually quite big so if you were near the banana tree (specifically on the side of the bin, not on other sides as increasing the size of the tree shifts it to the right), you would be interacting with the strawberry tree unknowingly.
All I did was just increase the height and it seems to work good now.


DAVE DISCORD 😀💀⚠🥺

surely this will be accepted

i may have pushed it too far with this one

DAVE DISCORD 😀💀⚠🥺

i wish I could buy rains right now as a thank you but whatever
tysm
there will be no more stupid buttons like this

I wonder when will someone get the 1 in more than 5 000 000

would be insane


Cluster setup article and more

Current coverage is 70.45% (diff: 100%)

Merging #214 into master will not change coverage

@@             master       #214   diff @@
==========================================
  Files            70         70
  Lines          4563       4563
  Methods           0          0
  Messages          0          0
  Branches        596        596
==========================================
  Hits           3215       3215
+ Misses         1208       1205     -3
- Partials        140        143     +3

Powered by Codecov. Last update 6eb8e06...1612102