Typescript-file

I created a Typescript-Definition file .. so maybe you can add it in your lib (https://stackoverflow.com/questions/37210059/how-to-add-a-typescript-definition-file-to-a-npm-package).
declare module "ipaddr.js" {

    interface IParsedIp {
        kind(): "ipv4" | "ipv6"
        toString(): string
        match<T extends IParsedIp>(ipAddress: T, bits: number): boolean
        match<T extends IParsedIp>(parseIP: T): boolean
        toByteArray(): Array<number>
        isIPv4MappedAddress(): boolean
        toNormalizedString(): string
    }

    interface IParsedIPv4 extends IParsedIp {
        kind(): "ipv4"
        octets(): Array<number>
        prefixLengthFromSubnetMask(): number | null
    }

    interface IParsedIPv6 extends IParsedIp {
        kind(): "ipv6"
        toIPv4Address(): boolean
        parts(): Array<number>
    }

    // https://github.com/whitequark/ipaddr.js#global-methods
    function isValid(ipAddress: string): boolean
    function parse(ipAddress: string): IParsedIPv4 | IParsedIPv6
    function process(ipAddress: string): IParsedIPv4 | IParsedIPv6

    function parseCIDR(ipAddress: string): IParsedIPv4 | IParsedIPv6
    function fromByteArray(list: Array<number>): IParsedIp

    interface IRangeList {
        documentationOnly: Array<IParsedIPv4 | IParsedIPv6 | number>,
        tunnelProviders: Array<Array<IParsedIPv4 | IParsedIPv6 | number>>
    }
    function subnetMatch(parsedIP: IParsedIPv4 | IParsedIPv6, rangeList: IRangeList, defaultName: string): string

    let IPv4: IIpV4;
    let IPv6: IIp;

    interface IIp {
        isValid(ipAddress: string): boolean
        parse(ipAddress: string): IParsedIp
        process(ipAddress: string): IParsedIp
    }

    interface IIpV4 extends IIp {
        isValidFourPartDecimal(ipAddress: string): boolean
        subnetMaskFromPrefixLength(prefixLength:string): string
        broadcastAddressFromCIDR(CIDR:string):string
        networkAddressFromCIDR(CIDR:string):string
    }
}

Send a pull request?

3rd of June and still not merged in? Is this library being maintained or should I look for alternatives?

It's not a pull request and it doesn't cover the entire interface of the library, that's why it's not merged. You're welcome to contribute a proper TypeScript interface yourself.

Will merge as a PR that covers the entire surface area of the library and handles NPM. I don't have time to figure out TypeScript, sorry.


Display Options -> Adjust Display Limits

When we select the Data Min and Data Max the artview changes the color bar, but it is not masking the data outside the interval (Data Min, Data Max). For example, if the data is under 10, when we set Data Min = 15, everything becomes blue, but it is still plotted. I realized it while using plot_ppi(), and in this case the solution is very simple. One of the options of plot_ppi() is mask_outside=False (this is the default), if we set mask_outside=True in /artview/artview/components/plot_radar.py, in line 741:
Create Plot
        self.plot = self.display.plot_ppi(
            self.Vfield.value, self.Vtilt.value,
            vmin=cmap['vmin'], vmax=cmap['vmax'],
            colorbar_flag=False, cmap=cmap['cmap'], mask_outside=True,     #<------------------------
            gatefilter=gatefilter, ax=self.ax, fig=self.fig, title=title)

then data outside (Data Min, Data Max) will be masked and artview will display only the data between Data Min and Data Max.
This option, mask_outside, also exists in plot_rhi(), and in plot_sweep_grid().

pyart recently changed it default from mask_outside=True to mask_outside=False, we are still some what behind. Thank you for pointing this out @andrevf. Can you implement @nguy? I have to take my plane now!

Thank you @andrevf The change has been made, just pull down the most recent version of the code. I am going to keep this issue open to make sure there are not unintended consequences before closing

Ok. Thank you.


Illegal value for [object Object] of type string: boolean (not a string)

Error: Illegal value for [object Object] of type string: boolean (not a string)
    at Error (native)
    at null.<anonymous> (/Users/kylesmith/Documents/github/node-steam-chat-bot/node_modules/steam/node_modules/protobufjs/dist/ProtoBuf.js:1868:114)
    at ProtoBuf.Reflect.ElementPrototype.verifyValue (/Users/kylesmith/Documents/github/node-steam-chat-bot/node_modules/steam/node_modules/protobufjs/dist/ProtoBuf.js:1930:25)
    at ProtoBuf.Reflect.FieldPrototype.verifyValue (/Users/kylesmith/Documents/github/node-steam-chat-bot/node_modules/steam/node_modules/protobufjs/dist/ProtoBuf.js:3536:33)
    at MessagePrototype.set (/Users/kylesmith/Documents/github/node-steam-chat-bot/node_modules/steam/node_modules/protobufjs/dist/ProtoBuf.js:2542:59)
    at MessagePrototype.set (/Users/kylesmith/Documents/github/node-steam-chat-bot/node_modules/steam/node_modules/protobufjs/dist/ProtoBuf.js:2533:38)
    at Message (/Users/kylesmith/Documents/github/node-steam-chat-bot/node_modules/steam/node_modules/protobufjs/dist/ProtoBuf.js:2462:34)
    at SteamUser.logOn (/Users/kylesmith/Documents/github/node-steam-chat-bot/node_modules/steam/lib/handlers/user/index.js:33:6)
    at ChatBot.logOn (/Users/kylesmith/Documents/github/node-steam-chat-bot/lib/chatBot.js:253:19)
    at ChatBot._onConnected (/Users/kylesmith/Documents/github/node-steam-chat-bot/lib/chatBot.js:670:7)

Getting that error every single time I try to login to Steam after I'm connected.

Show how you're calling SteamUser#logOn.

try {
			var sha = '';
			if(fs.existsSync(this.sentryFile)) {
				var file = fs.readFileSync(this.sentryFile);
				sha = crypto
					.createHash('sha1')
					.update(file)
				.digest();
			}
			this.steamUser.logOn({ account_name: that.username, password: that.password,
				auth_code: that.guardCode, sha_sentryfile: sha });
		} catch (err) {
			this.winston.error(this.name+"/chatBot.js: Exception trying to connect", err);
		}

I am calling this after the 'connected' event is received. That error is thrown in the catch block error.

What are the types of that.username, that.password and that.guardCode?

username and password are both strings, while guardCode is either a string or undefined, depending on whether someone specified it.

Can you confirm that? Add console.log(typeof that.username) and so on before the logOn call.

string
string
string
object

For username, password, guardCode, and sha.
The issue does seem to be gone though, even though I didn't change anything. Could have been a Steam issue.

@dragonbanshee Probably was. Steam was pretty funky on Monday.

I don't think so, look at the stack trace. It clearly failed to serialize the protobuf message because one of the values was typeof boolean.

@scholtzm You're right. I just can't see another way the issue would just "fix itself" unless it was some funny Steam connection issue.


Immersive Portals Core (imm_ptl_core) has failed to load correctly

According to your testing, does the issue occur with ONLY Immersive Portals mod installed?
Yes
Forge Version
forge-1.19.3-44.1.23
ImmersivePortals Version
2.5.4-mc1.19.3-forge
Latest Log
https://gist.github.com/Pichu6/4a843f4f03630e4153bd08a38937e471
Crash Report (if applicable)
https://gist.github.com/Pichu6/ed6759a9d11e22ecfdb2fbc4661ea036
Steps to Reproduce
game crash on start.

What You Expected
the game to start. I believe that forge and game versions where correct.
What Happened Instead
game crash at startup
Additional Details
No response
Please Read and Confirm The Following

[X] I have confirmed this bug can be replicated without the use of Optifine.
[X] I have confirmed the details provided in this report are concise as possible and does not contained vague information (ie. Versions are properly recorded, answers to questions are clear).
[X] I have confirmed this does not happen on the fabric version (If it does please report to here instead).
[X] I have confirmed I'm not using a hybrid server (Magma, Cauldron) or a different (non official) Forge Server release.
[X] I have confirmed this issue is unique and has not been reported already.

so whats the solution

ITS BEEN LIKE 4 MONTHS COME ON

How about not shouting at me?

ITS BEEN LIKE 4 MONTHS COME ON

You are missing Cloth Config, which is a required dependency for this mod.


Fail-over support

When I start 2 instances of ClientsServiceApplication on my desktop then I stop the first started one, the second instance is not seen by Eureka and the app does not work. Any idea?

It looks like default Eureka settings don't allow to register multiple instances of the same application with server.port=0. I am preparing a fix for that and learning a new thing :).

I'm learning too.
@mszarlinski I don't see your commit. Do you have push it?

Commit was merged to config repo, no changes needed here. You can see a pull request: https://github.com/spring-petclinic/spring-petclinic-microservices-config/pull/1

Ok thanks


ULTRA Add simple level to curriculum learning

Side tasks:
Extract traffic distr. from distributions.py to be used inside of ultra-env.py, instead of hard coding all the distr.
Also I do not know why it shows there is a difference of 45 commit between ultra-curriculum-learning and ultra-cl-add-simple-level? Nevertheless, the file difference is correct though

Would it be possible to merge #857 into ultra-develop and then merge ultra-develop into ultra-curriculum-learning? I think most of the reviews/comments of this PR will be the same as in #857. If there would be too many conflicts when merging ultra-develop into ultra-curriculum-learning though, I can review this one as well.

@christianjans I had a small debate with myself yesterday regarding to create this PR or not. I decided to go forward and merge the changes of #857 into this PR because I wanted to have a sort of a "reminder" for myself that I have merged the simple level into the curriculum learning branch. I also wanted to train/test on the grade-based structure using the new simple level and I felt keeping all changes on local branches would create some confusion. Thus, this PR advocates as a token that I have merged the simple level branch into curriculum learning.
In terms of reviewing, there is no major change from the #857 PR. All changes are identical except the changes to ultra-env.py. That change basically takes care of the hard coding of the traffic distribution as in the previous version.


More matrix operation support

While writing the GRO parser, I've come to realize there should be two features relating to Matrix3D.

[x] Support for scalar operations such as * and /.

[ ] Support for compound operators *=, +=, etc.

[ ] Support for setting the unit cell via a matrix.

Yes, these would be nice!
Also, while working on cfiles, I found myself missing assignment operators on both Vector3D and Matrix3D:
Vector3D += Vector3D
Vector3D -= Vector3D
Vector3D /= scalar
Vector3D *= scalar

# Do we want this one?
Vector3D *= Matrix3D

Matrix3D += Matrix3D
Matrix3D -= Matrix3D
Matrix3D *= Matrix3D
Matrix3D /= scalar
Matrix3D *= scalar

Those do not look too bad to add!

What is your opinion on stream operators ( << in particular)?

Honestly, I don't like them, mainly because to define them you have to pull the whole <iostream> header in =)
What would be your use case with them ? For debug purposes I sometimes add an implementation of operator<< where needed and then remove it.
I think operator>> does not make sense here, as there is no single way to parse a Matrix/Vector from a stream (is it (a, b, c)? [a, b, c]? a, b, c? a; b; c? ...)

Mostly for debugging as I use them quite a bit in my own projects for this purpose! I won't implement them then!

Yeah, it is not that hard to do whenever we need this. If a lot of people start asking for it, we could have it hidden under an un-documented  preprocessor flag.

I love un-documented preprocessor flags! My current project is filled with them!


Adds not_starts_with validation

This PR adds the ability to check if the string does not start with the given values, and I think this validation is important and makes the validation easier and we will not resort to using something else such as Regex.
$request->validate([
    'name' => ['required', 'not_starts_with:mahmoud'],
]);

@taylorotwell The PR sent to 9.x branch as you told :)

Change the title to [9.x] Add not_starts_with in to validation rules


SimpleRequest is empty

I use Laravel and AngularJS. I tested so much and nothing worked... first i startet with flow.js now ended with resumable ... i think i only have a last problem before it will work. I have this code:
...
use Dilab\Network\SimpleRequest as DilabSimpleRequest;
use Dilab\Network\SimpleResponse as DilabSimpleResponse;
use Dilab\Resumable as DilabResumable;
...
// Setup resumeable
$request = new DilabSimpleRequest();
$response = new DilabSimpleResponse();
$resumable = new DilabResumable($request, $response);
$resumable->tempFolder = $chunkDir;
$resumable->uploadFolder = $basicDir;
...
// Upload
$resumable->process();
...
What i found is that $request seems to be empty. If i do the following code, i dont get an object...
return json_encode([
'success' => $request
]);
Hope you can help me? :(

Now it works... just needed to set testChunks: false in my settings for resumable


CAMEL-20093: cleanup code duplications in camel-catalog

Just creating the first bits to cleanup the catalog code ... there's huge amount of duplicated code there.
Waiting for my own CI to test the whole set of changes.


Hide Keypad when EditText out of focus

Hey!
To enhance the user experience, the soft keyboard is now being hidden whenever the EditText loses focus. The changes have been made to the following:

acitivity_login
fragment_categorization
fragment_multiple_upload_lists

This PR references the issue #1109
Edit: I've added a new PR as a few issues were coming up with the previous one.
Please review :-)
Thanks!

Codecov Report

Merging #1168 into master will decrease coverage by 0.02%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master   #1168      +/-   ##
=========================================
- Coverage    3.85%   3.83%   -0.03%
=========================================
  Files         125     125
  Lines        5757    5793      +36
  Branches      567     568       +1
=========================================
  Hits          222     222
- Misses       5520    5556      +36
  Partials       15      15

Impacted Files
Coverage Δ

...e/nrw/commons/category/CategorizationFragment.java
0% <0%> (ø)
:arrow_up:

...n/java/fr/free/nrw/commons/auth/LoginActivity.java
0% <0%> (ø)
:arrow_up:

...nrw/commons/upload/MultipleUploadListFragment.java
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

It'd be great if someone could review this PR :-)

Hi @gupta-meghna64 , sorry, there is a bit of a backlog going on, and we generally have to take some time to test every PR before we merge it. Please feel free to work on other issues in the meantime, if you would like. :)

Looks good to me. Have also tested it on a real device and it works perfectly for me. Going ahead and merging this.
@gupta-meghna64 Congrats for your first contribution. :)


Split out tests

Issue #, if available:
Description of changes:
Testing (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/retest

/retest

/lgtm

/approve

/cherry-pick release-0.16


Fix javadoc errors

Doing ./gradlew install fails with javadoc errors resulting in a failed build.
This PR fixes the javadoc errors.
That make me think about changing github actions to not only do a run: ./gradlew check but a full run: ./gradlew install - or do I miss something @blackwinter ?

Ok, yes we could do this. But help me understand this:
 public void omitXmlDeclaration(final boolean currentOmitXmlDeclaration) has no javadoc and no check error nonetheless (although this is a public method) while  private void writeRaw(final String str) errors (although that's just a private method).
However, I propose to stick to scope public:
diff --git a/config/checkstyle/checkstyle.xml b/config/checkstyle/checkstyle.xml
index a5415c3d..c7b63a98 100644
--- a/config/checkstyle/checkstyle.xml
+++ b/config/checkstyle/checkstyle.xml
@@ -68,6 +68,9 @@
         <module name="InterfaceIsType"/>
         <module name="InterfaceTypeParameterName"/>
         <module name="JavaNCSS"/>
+        <module name="JavadocMethod">
+            <property name="scope" value="public"/>
+        </module>
         <module name="JavadocType"/>

What your opinion?

public void omitXmlDeclaration(final boolean currentOmitXmlDeclaration) has no javadoc for @param and no check error nonetheless (although this is a public method)

This method doesn't have any Javadoc at all; the JavadocMethod rule doesn't verify its presence, that would be MissingJavadocMethod.

However, I propose to stick to scope public:

Since these are the only two violations for private methods we currently have, I'd say we just fix them and keep the default scope. But I'm fine either way.

Right, thx for the hint!
ACK to keep the default scope of JavadocMethod.
So I like to have MissingJavadocMethod to ensure having the API documented. But using this results in a lot of missing java doc not only for methods but also for public constructors. Many (all?) of the latter just define a default constructor, e.g.:

public PicaXmlHandler() {}

Are these default constructors necessary ?
From http://www.java2s.com/Tutorials/Java/OCA_Java_SE_8_Class_Design/0030__Java_Constructor_Inheritance.htm (section "Compiler Enhancements"):

... [these] class and constructor definitions are equivalent

Couldn't we just get rid of them? (If so,  this implies also removing the module MissingCtor in checkstyle.xml because it forces the appearance of a constructor (and be it only a default constructor)).

It probably boils down to what we actually want, then we can adjust the config accordingly (see also #389):

Do we want explicit constructors that don't necessarily require Javadoc? Then enable MissingJavadocMethod but remove CTOR_DEF from tokens.
Do we want explicit constructors but don't require Javadoc for empty constructors? Then enable MissingJavadocMethod but set minLineCount to 1 (?).
Do we want Javadoc everywhere but don't insist on explicit constructors? Then enable MissingJavadocMethod and remove MissingCtor (along with all the empty constructors that were only introduced to satisfy that rule).

I'm leaning towards 2. But I'm not particularly invested in Javadoc, anyway, so I'll leave it up to you ;)

You boiled it down quiet nicely :) And I also like 2 the most (see also the edit in https://github.com/metafacture/metafacture-core/pull/396#issuecomment-931385139). Will go with that.

Awesome! :tada:
I still get some Javadoc reference errors, though:
metafacture-biblio/src/main/java/org/metafacture/biblio/marc21/MarcXmlEncoder.java:136: warning - #NAMESPACE_NAME (referenced by @value tag) is an unknown reference.
metafacture-biblio/src/main/java/org/metafacture/biblio/marc21/MarcXmlEncoder.java:136: warning - #NAMESPACE_NAME (referenced by @value tag) is an unknown reference.
metafacture-biblio/src/main/java/org/metafacture/biblio/marc21/MarcXmlEncoder.java:136: warning - #NAMESPACE_NAME (referenced by @value tag) is an unknown reference.
metafacture-commons/src/main/java/org/metafacture/commons/StringUtil.java:63: warning - #DEFAULT_VARSTART (referenced by @value tag) is an unknown reference.
metafacture-commons/src/main/java/org/metafacture/commons/StringUtil.java:63: warning - #DEFAULT_VAREND (referenced by @value tag) is an unknown reference.
metafacture-commons/src/main/java/org/metafacture/commons/StringUtil.java:94: warning - #DEFAULT_VARSTART (referenced by @value tag) is an unknown reference.
metafacture-commons/src/main/java/org/metafacture/commons/StringUtil.java:94: warning - #DEFAULT_VAREND (referenced by @value tag) is an unknown reference.
metafacture-strings/src/main/java/org/metafacture/strings/StreamUnicodeNormalizer.java:157: warning - Tag @link: reference not found: Normalizer.Form
metafacture-strings/src/main/java/org/metafacture/strings/UnicodeNormalizer.java:76: warning - Tag @link: reference not found: Normalizer.Form

Fixed in 3b8289e. But I wonder why they weren't picked up by the Github actions build...

I wonder why they weren't picked up by the Github actions build...

Probably a bugfix in 1.8? I use java version 1.8.0_292.

Probably a bugfix in 1.8? I use java version 1.8.0_292.

8u302 here. Github actions doesn't show the patch version.


Queue exchange params types

Makes usage a bit easier so I can pass around these options in my application before passing them to the client. Also added a ExchangeType type, so it will autocomplete in IDE (but you can still pass in other values)

Test action on node12 timed out, but that shouldn't be affected by my changes

Thank you! Great addition!


Is there a way to check the ServiceUrl is valid for AmazonS3Config?

Now if i enter custom service url with some relative path like 'http://<IP_ADDRESS>/app/' (with ForcePathStyle = true) and try to use GetBucketLocationRequest it will hang and eat CPU. So far I'm doing check that url should not have relative path, but question is there an API method to allow me validate service url before using AWS? Thank you

There is no API in the .NET SDK to do this check.

Ok, then take a look at this case, i think it should be fixed in sdk, because it hang at all

I don't believe adding this functionality to the SDK would serve most users of the SDK.
For your specific case, you can force the SDK to fail faster with a configuration similar to one below. Currently, you're seeing the "hang" because the SDK is taking 120 seconds waiting for a response from a server that will never send one. The below approach simply cuts that time down to 5 seconds, though you can set an even shorter time when testing if a url works. (Keep in mind, that this config should only be used to determine if an endpoint is valid, not for actual service calls.)
var config = new AmazonS3Config
{
    ServiceURL = "http://<IP_ADDRESS>/app/",
    ForcePathStyle = true,
    Timeout = TimeSpan.FromSeconds(5)
};


Add missing language identifier to code block

Can one of the admins verify this patch?

Can one of the admins verify this patch?

@kivlor, thanks for completing the CA and for the patch. Merging...

no probs


Revert "[Fix Diagnose] Remove netpath EP config (#23686)"

What does this PR do?
This reverts commit 08c78aefcff0ab8fac9e888173270e40c012a27f. We've worked it out with EP so they'll now return 200s in prod.

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


remove dependencies on nodejs:6 runtime kind

NodeJS 6 is long past EOL.  We've long since deprecated the nodejs:6 runtime kind.
We need to remove all dependencies throughout the entire system on the nodejs:6 runtime kind, then remove it from the runtime manifest.
Some known lingering uses:

https://github.com/apache/openwhisk-package-kafka/issues/367
https://github.com/apache/openwhisk-package-cloudant/issues/222
https://github.com/apache/openwhisk-wskdeploy/pull/1089

@dgrove-oss
there is a dependancy in openwhisk-composer
https://github.com/apache/openwhisk-composer/blob/1e1297eb9e8119854b35ebb0b0630f7f8aee39c6/travis/runtimes.json#L14

Thanks @emishas -- will take a look

submitted https://github.com/apache/openwhisk-composer/pull/72 for composer; waiting to see what travis thinks :)

@dgrove-oss there are also dependancies inside packages, i.e.:
Alarms package was updated
https://github.com/apache/openwhisk-package-alarms/commit/a2e108e3537b6015f4e437ed43a9450eef90fa2e
but Kafka not:
https://github.com/apache/openwhisk-package-kafka/search?q=nodejs&unscoped_q=nodejs
Haven't checked the rest.

We've dropped node 6 now.


Stroke color

is there a way to change the stroke (border) color with the divider?
and thanks for your work, it's amazing

Hi again, thank you for positive feat back,
Sure there is some methods and attributes in order to change the stock color, but I'm not clearly understand what you meant by saying color with the divider?
This is what you can archive with this attributes
setSelectedStockColor = "#Purple"
setUnSelectedStockColor ="#Grey"
setSelectedBackgroundColor ="#Blue"
If this is will not help, please provide more details for clarification

Will be changed to strockColor, it's typo

no problem, thank you for your effort man

Fixed typo stock -> stroke
compile 'com.github.RobertApikyan:SegmentedControl:release_1.0.2'


Totally Lost .... Help !

Hi peeps,
Tryna dowload PDF from Pearson E-Text.
Not sure if I am doing the right thing.
When I open up > https://nomod-programming.github.io/PearsonEbookDownloader/
Enter Book ID ,

Do I wait for the PDF to completely download ?
I would appreciate some help. Thanks

I have the same issue ! The web version does stay stuck at Downloading pages, but it's due to a 404 Not Found error if I'm not mistaking.
Would love some help as well.
Coco

same issue

Hi friends, It is this issue agian:
https://github.com/NoMod-Programming/PearsonEbookDownloader/issues/21

Hi friends, It is this issue agian:
#21

Sure seems like it. It's been nearly 3 months, figured maybe there would be some changes by now. Thanks for the response.


Responsive styles fixing

Shot Type
Count
Status

Added
0
🟡

Deleted
0
❌

Regression
0
❗

Shot Type
Count
Status

Added
6
🟡

Deleted
0
❌

Regression
19
❗

🟡 custom/widgets-filters--filter-button-items-not-selected__[w390px] (click to expand)

Image

🟡 custom/widgets-filters--more-button-items-selected__[w390px] (click to expand)

Image

🟡 storybook/widgets-filters--all-items-enabled__[w1440px] (click to expand)

Image

🟡 storybook/widgets-filters--all-items-enabled__[w1920px] (click to expand)

Image

🟡 storybook/widgets-filters--all-items-enabled__[w390px] (click to expand)

Image

🟡 storybook/widgets-filters--all-items-enabled__[w900px] (click to expand)

Image

❗ custom/widgets-collapsiblenavbar-opened__[w1439px]_statistics_submenu_popover (click to expand)

Before
After

❗ custom/widgets-collapsiblenavbar-opened__[w1920px]_statistics_submenu_popover (click to expand)

Before
After

❗ custom/widgets-collapsiblenavbar-opened__[w899px]_statistics_submenu (click to expand)

Before
After

❗ custom/widgets-filters--default__[w1920px] (click to expand)

Before
After

❗ custom/widgets-filters--more-button__[w900px] (click to expand)

Before
After

❗ custom/widgets-searchbox--cross-icon__[w900px] (click to expand)

Before
After

❗ custom/widgets-searchbox--search-icon__[w900px] (click to expand)

Before
After

❗ storybook/widgets-bots-table--bots-table__[w390px] (click to expand)

Before
After

❗ storybook/widgets-bots-table--bots-table__[w900px] (click to expand)

Before
After

❗ storybook/widgets-bots-table--no-control-orders__[w390px] (click to expand)

Before
After

❗ storybook/widgets-bots-table--no-control-orders__[w900px] (click to expand)

Before
After

❗ storybook/widgets-filters--default__[w1440px] (click to expand)

Before
After

❗ storybook/widgets-filters--default__[w1920px] (click to expand)

Before
After

❗ storybook/widgets-filters--default__[w390px] (click to expand)

Before
After

❗ storybook/widgets-filters--default__[w900px] (click to expand)

Before
After

❗ storybook/widgets-heart-rate--default__[w1440px] (click to expand)

Before
After

❗ storybook/widgets-search-box--default__[w390px] (click to expand)

Before
After

❗ storybook/widgets-search-box--default__[w900px] (click to expand)

Before
After

❗ storybook/widgets-table-filters--default__[w390px] (click to expand)

Before
After


[WC-2680] onChange not trigger on empty or not empty filter

Pull request type

Description

I think it should be fixed on useBasicSync level. Just change line 26 from
return reaction(() => [store.arg1.value, store.arg2.value], createPusher(pbox));

To this:
return reaction(() => [store.arg1.value, store.arg2.value, store.filterFunction], createPusher(pbox));

I think it should be fixed on useBasicSync level. Just change line 26 from
return reaction(() => [store.arg1.value, store.arg2.value], createPusher(pbox));

To this:
return reaction(() => [store.arg1.value, store.arg2.value, store.filterFunction], createPusher(pbox));

but this will trigger onchange for every filter change, of which we don't need.


update data tutorial

Admonitions for important info
Better formatting
Some content changes

@jjhong922 after this gets merged can you update it further to include a transfer fields section before the dataloading stuff? Just to explain that anndatamanager can be used to identify equivalency of objects


best_params_ missing on GridSearchCV

The best_params_ dict seems to be missing from GridSearchCV, even if refitting is enabled.
grid_search.py#L195 refers to that parameter, it is determined in grid_search.py#L371 but never actually exposed after fitting. This contradicts both, your own docs and the Scikit Learn 0.19.1 and 0.20.0 docs. Was this attribute purposefully not exposed or is this a bug? Depending on your answer, I'd be happy to provide a PR for a change of the docs or the code to resolve this.
See also #37 which uncovered this a year ago.

Ah, no that doesn't work. Looks like we can't set this attribute as it's inherited from BaseSearchCV. Maybe someone with better knowledge of Python could propose a better way to do it, but this won't work. Reverting ...

I'm reopening this as there is clearly an issue here, though the 'quick fixes' don't work:
https://github.com/databricks/spark-sklearn/issues/37
https://github.com/databricks/spark-sklearn/pull/79
https://github.com/databricks/spark-sklearn/pull/84
https://github.com/databricks/spark-sklearn/pull/85
See in particular https://github.com/databricks/spark-sklearn/pull/79
It seems like the intent is to set these attributes for parity with scikit, but, they can't be set directly. There may be a simple answer; I don't know it yet. Ideas welcome here!

I'm on 19.2 of scikit learn and can't reproduce this issue when trying to add in best_params_ manually, at least not when running run_tests.sh.
Is there another way to reproduce the bug?
I tried by adding this to line 164 in base_search.py, then ran tests.
 self.best_params_ = results["params"][self.best_index_]


Fix up Style/Semicolon

rubocop -a --only Style/Semicolon to find all occurrences.

Hello! I would like to help with this

Cool, @nesteves when you make a pull request, be sure to include a link to this issue in the description and we'll take a look.  Thanks!


update kotlin to v1.2.0

https://github.com/JetBrains/kotlin/releases/tag/v1.2.0

From 1.1 to 1.2 there are many changes inside the compiler.
I don't feel safe to ship it with RC5-6, will release this version first.

You're right, I was hoping this would help solve the issue reported in #566. But trying it locally that doesn't seem to be the case.

Merged manually.


Not Working?

Hi
Did something change or is there a known problem? I am currently using this in a program, and even just the simple code that is on the wiki is not working. I've been using this for a while, but suddenly on friday it stopped working. (exchange "my chat id here" with the youtube chat id.
For example
from pytchat import LiveChat
import time
chat = LiveChat(video_id = "my chat id here")
while chat.is_alive():
try:
data = chat.get()
items = data.items
for c in items:
print(f"{c.datetime} [{c.author.name}]- {c.message}")
time.sleep(3)
except KeyboardInterrupt:
chat.terminate()
break
when i run this, it just finishes, and it doesnt actually pull anything.

Could you please tell me the concrete video_id ?
Also, is your version of pytchat up to date?
In my current environment, it works just fine: e.g.
from pytchat import LiveChat
import time
chat = LiveChat(video_id = "eJ7ZkQ5TC08")

while chat.is_alive():
  try:
    data = chat.get()
    items = data.items
    for c in items:
        print(f"{c.datetime} [{c.author.name}]- {c.message}")
    time.sleep(3)
  except KeyboardInterrupt:
    chat.terminate()
    break

Could you please tell me the concrete video_id ?
Also, is your version of pytchat up to date?
In my current environment, it works just fine: e.g.
from pytchat import LiveChat
import time
chat = LiveChat(video_id = "eJ7ZkQ5TC08")

while chat.is_alive():
  try:
    data = chat.get()
    items = data.items
    for c in items:
        print(f"{c.datetime} [{c.author.name}]- {c.message}")
    time.sleep(3)
  except KeyboardInterrupt:
    chat.terminate()
    break

Could you please tell me the concrete video_id ?
Also, is your version of pytchat up to date?
In my current environment, it works just fine: e.g.
from pytchat import LiveChat
import time
chat = LiveChat(video_id = "eJ7ZkQ5TC08")

while chat.is_alive():
  try:
    data = chat.get()
    items = data.items
    for c in items:
        print(f"{c.datetime} [{c.author.name}]- {c.message}")
    time.sleep(3)
  except KeyboardInterrupt:
    chat.terminate()
    break

ah.. i apologize. i thought i set this to auto update... updating fixed the issue.
sorry about that.

Could you please tell me the concrete video_id ?
Also, is your version of pytchat up to date?
In my current environment, it works just fine: e.g.
from pytchat import LiveChat
import time
chat = LiveChat(video_id = "eJ7ZkQ5TC08")

while chat.is_alive():
  try:
    data = chat.get()
    items = data.items
    for c in items:
        print(f"{c.datetime} [{c.author.name}]- {c.message}")
    time.sleep(3)
  except KeyboardInterrupt:
    chat.terminate()
    break

ah.. i apologize. i thought i set this to auto update... updating fixed the issue.
sorry about that.


Add config option to add alternative python packages

Sometimes python package name is not the same as it's module name (eg sklearn vs scikit-learn). For that we have MODULE_PACKAGE_MAPPING in mlem.core.requirements that you can update from code. But it would be nice to be able to do it from config like
config.yaml
core:
  package_mapping:
    box: python-box

Would this approach require the user to manually specify the mapping?

Yes, there is no way to know package name after it was installed

@mike0sv, have you seen this approach? I've tried it and it seems to work
>>> from importlib_metadata import packages_distributions
>>> packages_distributions()['box']
['python-box']

Got to this again, while trying to use pip install chatgptpy --upgrade, which is imported like from pychatgpt import Chat. We need to fix this 🔧


Question about replicaCount

Hi,
There were recently two issues opened (#186 and #193) where it was said that replicaCount has to be always set to 1. However, the explanations of "why" were rather short. I checked the setup with SQ deployed as StatefulSet with three pods running and an external database and everything was running just ok, with no problems whatsoever. So, before rolling this up to production (other teams within my company), I would like to know the risk, like what exactly can go wrong and under which circumstances (e.g. what does it mean "database corruption").
Thanks,

Hi @blazej-budzynski
Here is already a discussion on the topic : https://community.sonarsource.com/t/sonarqube-in-aks-with-more-than-one-replica/51892
If you still need more information about it, I invite you to comment on the community forum as this topic is more related to SQ itself than the helm chart


Update D1-Inception.md

@Major-Lag98 I went ahead and created a rough draft. If you could expand a bit on:

what problem your software is solving and the context of the solution
look at value proposition and change/add things
look at customer targets and change/add things


updating to create the bdio file

@akamen @jrichardBD @psantos1113 @stevebillings Please review at your leisure.

Looks good,

looks good

Looks good


Rename all mention of bundleExclude to publishExclude in project.json

There are several places in samples and screenshorts mention 'bundleExclude':
https://github.com/aspnet/Docs/search?utf8=✓&q=bundleExclude
See
https://github.com/aspnet/Home/issues/610
https://github.com/aspnet/dnx/issues/1474
for details.

@dimaaan thanks for reporting.
@Erikre  can you fix these and then close this issue?

I'll work on this now.

submitted pr #613


The version 7.2.0 of the Nagios cookbook breaks Apache

The following line was added to the nagios::apache recipe which breaks apache on restart
include_recipe 'apache2::mpm_prefork' # mod_php5 is not threadsafe as packaged

The error is the following:
Starting httpd: httpd: Syntax error on line 106 of /etc/httpd/conf/httpd.conf: Syntax error on line 1 of /etc/httpd/mods-enabled/mpm_prefork.load: Cannot load /usr/lib64/httpd/modules/mod_mpm_prefork.so into server: /usr/lib64/httpd/modules/mod_mpm_prefork.so: cannot open shared object file: No such file or directory

I personally don't use the apache configuration and use nginx, but it seems to me that you are missing the mpm module on disk. So I hope that you have a suggestion on how to fix this or maybe @ChrisLundquist can help with this as he suggested the change in PR #342

Yeah, I agree the module doesn't seem to be installed. At the time I ran it apache2::mpm_prefork installed the needed packages.
I'm running Ubuntu 14.04.2, here are the relevant files
$ locate mpm
/etc/apache2/mods-available/mpm_event.conf
/etc/apache2/mods-available/mpm_event.load
/etc/apache2/mods-available/mpm_itk.load
/etc/apache2/mods-available/mpm_prefork.conf
/etc/apache2/mods-available/mpm_prefork.load
/etc/apache2/mods-available/mpm_worker.conf
/etc/apache2/mods-available/mpm_worker.load
/etc/apache2/mods-enabled/mpm_prefork.conf
/etc/apache2/mods-enabled/mpm_prefork.load
/usr/lib/apache2/module/disabled_by_admin/mpm_event
/usr/lib/apache2/module/enabled_by_admin/mpm_prefork
/usr/lib/apache2/modules/mod_mpm_event.so
/usr/lib/apache2/modules/mod_mpm_prefork.so
/usr/lib/apache2/modules/mod_mpm_worker.so

Here are the apache packages I have.
$ dpkg -l | grep apache
ii  apache2                          2.4.7-1ubuntu4.5                     amd64        Apache HTTP Server
ii  apache2-bin                      2.4.7-1ubuntu4.5                     amd64        Apache HTTP Server (binary files and modules)
ii  apache2-data                     2.4.7-1ubuntu4.5                     all          Apache HTTP Server (common files)
ii  libapache2-mod-php5              5.5.9+dfsg-1ubuntu4.11               amd64        server-side, HTML-embedded scripting language (Apache 2 module)

On RHEL, at least, the mpm modules do not need to be installed separately.

The prefork module is compiled in as can be seen below:
# httpd -l
Compiled in modules:
  core.c
  prefork.c
  http_core.c
  mod_so.c

# rpm -qi httpd
Name        : httpd                        Relocations: (not relocatable)
Version     : 2.2.15                            Vendor: CentOS
Release     : 29.el6.centos                 Build Date: Tue 13 Aug 2013 10:30:33 AM PDT
Install Date: Fri 14 Aug 2015 02:35:08 PM PDT      Build Host: c6b8.bsys.dev.centos.org
Group       : System Environment/Daemons    Source RPM: httpd-2.2.15-29.el6.centos.src.rpm
Size        : 3076447                          License: ASL 2.0
Signature   : RSA/SHA1, Tue 13 Aug 2013 10:32:39 AM PDT, Key ID 0946fca2c105b9de
Packager    : CentOS BuildSystem <http://bugs.centos.org>
URL         : http://httpd.apache.org/
Summary     : Apache HTTP Server
Description :
The Apache HTTP Server is a powerful, efficient, and extensible
web server.

It sounds like the apache2 cookbook doesn't take this into account when it generates /etc/apache2/mods-enabled/mpm_prefork.load which sets the path to load the module

It looks like there is an open issue https://github.com/svanzoest-cookbooks/apache2/issues/364

you do not want load a new MPM via the recipe. You want to include_recipe 'apache2::mpm_prefork', you will want to change the MPM via node['apache']['mpm'] = 'prefork'

I have followed the logic of the apache2 cookbook and I agree that we do not want to include the mpm_prefork recipe, instead we should set the mpm attribute. I will file a PR.

Thank you all for your support on this.... very much appreciated...

@sbotman when is the next release of the cookbook expected?

I you want one, I will create one...

@sbotman it would be nice to get a hot fix release for this particular fix, as the cookbook may be broken for users running it on CentOS/RHEL.

agreed, will do..

Thanks I appreciate it.

Deploy is done and you're welcome, thanks for helping out...


GCB: need design/plan for which builds will hit production vs use emulator

Some integration tests need to hit production. For some services we have emulators. Tests that hit production are (generally) slower, flakier, and may run into production quota issues.
Sanitizier builds need to execute code in order to be effective. Running unit tests is good, but running integration tests gives us more coverage. So possibly all the sanitizers need to run integration tests too.
The code coverage build needs to run integration tests in order to cover as much code as possible.
Auto-generated libraries will likely not have any emulators. Their integration tests and samples would run against production.
We need a mini-design doc outlining what our testing strategy should be for integration tests
This doc should answer:

Which builds will run integration tests against production?
Which builds will run integration tests against emulators?

[x] The mini-design doc is at go/cloud-cxx:integration-tests-2022q1, discussed and approved today
[x] We need to create a GCB build testing against production (#8323)
[x] We need to remove the integration tests from the Kokoro builds (#8339)
[x] We need to add a GCB builds that runs only the quickstarts using production (#8342)
[x] We need to run two quickstarts (google/cloud/{storage,spanner}/quickstart) in each of the Kokoro builds (#8344)
[x] We need to stop running quickstarts when compiling against older versions of the libraries (#8345)

This is now done.  There are (as always) opportunities for improvement, e.g., maybe more refactoring of the powershell scripts, and maybe parallelizing the quickstart builds a bit more.  We will leave that for the future.


Use "SHOULD" instead of "recommended"

To minimise ambiguity, CSIP should say "It SHOULD be ..." instead of "It is recommended that it be ..."
A specific example is Ch 5.3.1 CSIP2 mets/@OBJID, which currently reads:

It is recommended that it be the same as the name or ID of the package (the name of the root folder)

Referenced CSIP version: a84d77a, 2018-09-11

Text update will come when the METS profile is ready.

There are two requirements "hidden" in the text:

In addition to the attributes the METS root element mets MUST define all relevant namespaces and locations of XML schemas using the @xmlns and @xsi:schemaLocation attributes.
In case XML schemas have been included into the package (i.e. placed into the schemas folder) it is recommended to link to the schemas using the relative path of the schema file (i.e. schemas/mets.xsd).

Information regarding @OBJID have been updated. Its now CSIP1.


Console error on loading / displaying the plan

Hello,
I setup for my father the same foorplan3d-card as I made it for me. But on his home assistant I keep receiving this error when the card 100% finish loading the plan:

Any ideea?
Thanks in advance!
Dan Barbu

I remember how to solve it. By default is choosing some entities on the setup. I delete them and now working. I will close the bug. Thank you.


Would it be possible to use Esp32 or ESP8266 as the main MCU?

It would be possible to use one of these devices to replace the arduino. With this we would get wifi / bluetooth in a simpler way

There's a project with ESP32 already. Check recent posts in the Facebook group.

thank you very much I can help you in that!


MapProxyDomain does not work with InPlaceResourceOptimization

I think this should be straightforward to support, as the MapProxyDomain prefixes are established at configuration time and can be statically checked in the handler at request-time.
Note that there are a variety of ipro/MapProxyDomain tests around https://github.com/pagespeed/mod_pagespeed/blob/master/pagespeed/apache/system_test.sh#L988 but those are not shared with nginx.
In the shared system_test script there is a test for MapProxyDomain from the HTML flow: https://github.com/pagespeed/mod_pagespeed/blob/master/pagespeed/system/system_test.sh#L241
but there is no evidence of testing in that shared file for MapProxyDomain of in-place resources.

In normal usage this isn't a problem, because any resources PageSpeed moves onto the mapped proxied domain it will be able to create .pagespeed. resources for, but if you want to set up a server to do IPRO optimization of another server this would be a useful feature.
(This does work in Apache.)

Also, in general it's safer if we accept these, in case a CSS file we proxy onto our domain references something relative and we don't manage to rewrite it's url for some reason.

See https://groups.google.com/forum/#!topic/ngx-pagespeed-discuss/IuMfDTtz8sg for a related problem

I created pull requests with changes to make this work for nps (note there's also an MPS side to this).
If the new 'trusted_input' flag in ProxyFetch is not a good idea and another approach would be better, I'd love to hear that.

LGTM.  I'll merge this along with the apache side of the change once I finish running internal tests.

Fixed with becc2f6


Plans for adding platform-specific standard keyboard shortcuts?

Things like <command+q> to quit on macOS, which is implemented in some other windowing libraries (sdl even have <command+ctrl+f> to toggle fullscreen by default). Wondering if winit is planning to implement these or let users handle?

Hi, and thanks for the question!
There are no plans to implement this on our end. Winit's goal is to provide the minimal API necessary for users to create windows and desktop applications in a consistent way across platforms, and defining platform-specific keyboard shortcuts falls outside that scope. There's definitely room for another crate to expose definitions for platform-native keyboard shortcuts, though!

I see, thanks!

So... I'm going to backtrack a bit here. Looks like there's some ongoing discussion about this in #41 which has broad support for implementing this in some form.


Copy Paste Syntax Highlighting does not paste installed font

VSCode Version: Code 1.10.1 (653f8733dd5a5c43d66d7168b4701f94d72b62e5, 2017-03-02T00:47:58.633Z)
OS Version: Linux x64 4.9.13-200.fc25.x86_64
Extensions:

Extension
Author
Version

material-icon-theme
PKief
1.3.4

vscode-chester-atom
chriseckenrode
0.1.7

hopscotch
idleberg
0.1.4

The copy paste of syntax highlighting works half-way since it does not paste the font used in VS Code, even when installed in the system.
Steps to Reproduce:

Select text in VS Code with a programming font such as Source Code Pro.
Paste in Outlook New Mail window with "Keep Formatting" option.

Sorry I can't reproduce this issue. Outlook is telling me the pasted content is Source Code Pro.


Unable to get data set

The data link has expired, can you provide an accessible link? thank you!!!
https://cnrsc-my.sharepoint.com/personal/nicola_messina_cnr_it/_layouts/15/onedrive.aspx?id=%2Fpersonal%2Fnicola_messina_cnr_it%2FDocuments%2Fweb%2Ftern_teran&ga=1
I also can't access this link

@hongsong-wang @LCFractal can you help me? thank you very much

i can open your link and download the file.


Overhaul the Curve Editor

Currently has issues. I made this PR as though the problem outlined here - https://github.com/godotengine/godot/pull/74927#issuecomment-1469917500 - has been solved. As it stands, the PR can still be merged, but it would have a few irritating issues:

After using Add Point / Remove Point operations, the selected point would get deselected.
Adding points would corrupt the Undo.

I disabled the "property_list_changed" notifications of curve to make the recordings below and show how the PR would ideally behave.
Anyway, here's what the PR does:
Refactoring
Splits CurveEditor in two: CurveEditorRect (the area where you edit a curve) and CurveEditor (includes the buttons on top).

Context menu removed
The context menu really slows down curve editing. I went for an intuitive control scheme:

Left click to add a point or drag the point/tangent you're hovering
Right/Middle click to remove a point or make a tangent linear
Shift + Left click to drag a tangent individually, like before.

As shown, the presets are moved as a button above the curve editor.
(The recordings below don't show the context menus, probably because the engine treats them as different windows. Use your imagination.)
Old:
https://user-images.githubusercontent.com/85438892/225438826-571133fa-3b59-4409-bbb3-3e787622c98f.mp4
New:
https://user-images.githubusercontent.com/85438892/225436981-12491925-af99-4948-b714-cdf73aaa073b.mp4
Utilities
Added a snap button with an adjustable number of divisions. Ctrl + Drag tricks were removed.
Shift + Drag on points locks them on one axis.
https://user-images.githubusercontent.com/85438892/225436694-bd31f21c-0eeb-42b2-9c43-64078e139056.mp4
And as you saw in the previews, a bit of text at the top shows the position/angle of the currently selected point or tangent.
Enhancements + Bugfixes

Visual tweaks
Adds hover and selection indicators to tangents.
Makes it harder for tangents to slip outside of the editable area.

No longer forces new points inside the [0, 1] range (Fixes #41083).
Fixes bug where pressing Delete also attempts to delete nodes.
"Hold Shift to edit tangents individually" help text is only shown when relevant.
Presets now use the full range.
Hover areas are now rectangular and stretch out a little further than their indicators to reduce the number of misclicks.
When clicking in a place with multiple elements, selects the closest one. Tangents get priority.
Improves the logic for keeping handles selected during operations.
Stops operations from being committed if they don't change anything.
Tells which tangent was modified when committing actions (left / right / both).
Prevents points from sharing an X position (important for snapping).
Improves the preview a little.
Fixes "Remove Curve Point" message when adding a point.

Misc

Adds Curve.get_range() (seems generally very useful, used in multiple places throughout the code)
Code style improvements (i.e. "p_" before parameter names, sentence-like comments)

Simplified the curve preview to use Brezenham's line algorithm. Fixed inconsistent behaviors when two points have the same offset (mostly happens around x=0 and x=1).

Until the mechanism for updating the property tree is modified, a temporary solution is to pass the necessary state data by calling the set_meta()/get_meta() methods on the curve object.

I'll look into it, thanks. I haven't really tinkered with meta before.
I'm also trying to somehow make an indicator for the snapping positions.

I provided a method in #75030 to predict the index of a point after it has been added.

Sorry for force-pushing so much, I just keep noticing little oversights!

I looked at the animations, and noticed something. When adding a new point, tangents should be placed at a third of the distance to the nearest points, not at constant distances. Otherwise, too often, the tangents will go past the next point, which creates a curve that is non-continuous in x. It was actually a common complaint I heard about the animation bezier curve editor from an animator with experience in Blender and Maya.

The situation you're describing should be rarer now because tangents are a bit smaller. I don't think I can implement this because I don't understand it, for example, what if two points are very close to each other? Would the tangent be shown on top of the selected point? Wouldn't it be confusing to make all tangents differently-sized?

GDquest’s team has an pr with those stated features. We can unify them.On Mar 24, 2023, at 8:46 AM, Mew Pur Pur @.***> wrote:
The situation you're describing should be rarer now because tangents are a bit smaller. I don't think I can implement this because I don't understand it, for example, what if two points are very close to each other? Would the tangent be shown on top of the selected point? Wouldn't it be confusing to make all tangents differently-sized?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because your review was requested.Message ID: @.***>

https://github.com/godotengine/godot/pull/64647On Mar 24, 2023, at 9:31 AM, Ernest Lee @.> wrote:GDquest’s team has an pr with those stated features. We can unify them.On Mar 24, 2023, at 8:46 AM, Mew Pur Pur @.> wrote:
The situation you're describing should be rarer now because tangents are a bit smaller. I don't think I can implement this because I don't understand it, for example, what if two points are very close to each other? Would the tangent be shown on top of the selected point? Wouldn't it be confusing to make all tangents differently-sized?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because your review was requested.Message ID: @.***>

But this is for editing the Curve resource, which that editor doesn't seem to be tied to... the curve resource also only has a Free mode and a Linear mode... :thinking:

I don't think I can implement this because I don't understand it, for example, what if two points are very close to each other? Would the tangent be shown on top of the selected point?

You can start up Blender and see how they do it. It's free (and open source). But generally, you don't want tangents to go past the point. And if you have to go looking for the tangent, and adjust it after each point, you're going to have a bad time.
An alternative would be to somehow let the user set the default tangent length. So, if they have their own expectation on how tight their points are going to be, they can set it accordingly.

Hey @SlugFiller , please don't tell contributors "go look it up". If you want development to go in the direction you wish, it's up to you to provide clear specifications and actionable items. I think that auto placing point is an interesting feature but possibly outside of the scope of this PR. The curve resource editor, differently to the animation curve editor, does not offer length adjustment on the tangents.
@fire Is there a spec somewhere with a list of controls and features? Can you elaborate on "unifying them"? Do you mean that we should use the same curve editor for both animations and curve resources? If that's your proposal I think it's an interesting idea but outside of the scope of this PR

I am not able to give good design until I arrive back home. But this photo is from the post I linked.

It's actually from my PR :P
Tangents are all 36px wide, but can be shrunk down to 18px if they would slide outside of the view area. This prevents you from being unable to grab them.

Until the mechanism for updating the property tree is modified, a temporary solution is to pass the necessary state data by calling the set_meta()/get_meta() methods on the curve object.

It's better to save this stuff on a per-curve basis anyway, so I implemented metadata.

@Geometror Thanks a lot for the feedback! Time to iterate, I suppose. Just a few comments on where I plan to take it.

Good point about Ctrl, perhaps I'll replace the constraint thingy with quick snapping instead. I'd rather not use Alt because it's better to avoid making Alt+Shift meaningful.
I'll increase the height a little more.
Right-clicking is essential for people on laptops to be able to delete points quickly, would rather not remove it. Right-clicking on tangents to show a context menu makes sense to me as they can't be deleted, I'll have to play around with it.
Adding a point would eventually allow you to directly adjust it, but it's not up to my PR to fix this. Also this is why I increased the size of the hoverable areas so much.

On the additional points:

Current rendering code is a hot mess because of the transform applied, so I'll stick to primitives unless I think up of something
(look what happened when I put a quad lmao
I really want to have better grid alignment, but have been unable to think of anything that doesn't make the editor too busy. There's the main grid, possibly a secondary one, and the numbers on the bottom which shouldn't overlap...
Good point, I'll try to improve the usability of tangents with Shift.

While left-click for adding a new point is great, I noticed that it can become a bit annoying when you quickly want to select a point but accidently create a new one because you didn't click precisely.

This is arguably an accessibility consideration

A snapping grid just makes this too busy IMO

So I didn't implement it.

When the tangents are edited individually, grabbing one tangent handle without Shift pressed will make them collinear again, which is quite annoying to me.

Now if you press Shift it restores the old angle of the opposite tangent. I also removed the hover indicator for the opposite tangent in case it's linear.

QUAD LINES YES

Actually added one more thing to address feedback I got over Discord that getting accustomed to the new control scheme can be confusing the first time. Now there's a + that shows where new points would be added. Also clamps the value and works with snap, as shown below. I think it's nice.

Now there's a + that shows where new points would be added, making it obvious what LMB does.

Either I'm misunderstanding something, or this last change feels like an overkill, because mouse pointer should point to an exact point on the screen, otherwise it just doesn't function properly.
... That's my point, I guess.

@Iniquitatis
It's pretty standard for editors to show something that will be placed when you use LMB (i.e. tilemap), but I believe it's better to judge on a case by case basis. So if you think it's bothersome, please do tell me.

because mouse pointer itself should point to an exact point on the screen

It does, in the above example the cross is a bit away because snapping is enabled.

Vector2i grid_steps = Vector2i(0, 2);
if (snap_enabled || Input::get_singleton()->is_key_pressed(Key::CTRL)) {
	// If snapping, find appropriate grid size.
	if (snap_count == 2) {
		grid_steps.x = 2;
	} else if (snap_count == 3 || snap_count == 6 || snap_count == 9) {
		grid_steps.x = 3;
	} else if (snap_count % 5 == 0) {
		grid_steps.x = 5;
	}
}

if (grid_steps.x == 0) {
	// Use a default/fallback value.
	grid_steps.x = 4;
}

I don't this code would look very nice for the "adjust grid based on snap divisions" suggestion, but I'm just leaving it here for reference.

What's the status on this? Can we review&merge soon?

I removed the green cross because a bug unrelated to my PR made it jump around erratically. It would also be really nice to solve the problems outlined in https://github.com/godotengine/godot/pull/74927#issuecomment-1469917500 first so that UndoRedo doesn't get bugged.
Other than that, I think it's ready for someone to take another look :)

The really bad occurrences of the "inspector rebuilding on inopportune moments" bug have been worked around, I've added a FIXME on the hack inside Curve.

Thanks to the review! Here's my review on the main points of your review:
get_offset_without_collision()
Not sure what to do with this function, while I like its utility, its implementation is very sus.
Scaling parameter instead of transforms
This is only done for one of the transforms, the one used for drawing the curve, and I document why this transform is so weird. In a nutshell, the curve itself must be transformed, because set_draw_transform() breaks quad lines when the scaling is non-uniform.
This is fundamental, not a bug. Picture a transform that scales something by making it 2 times wider. Such a transform would make horizontal lines 2 times longer, and vertical lines two times fatter. So I don't think there's a better solution if we want antialiased curves.
The hover rect doesn't appear centered
No idea how to address this.
Shift after adding a point is weird
Fixed.

I'd also prefer we merge this since it's definitely an improvement overall, refinement can be done later.

Just a little note about Geometror's request for editing tangents easily: I wasn't sure the idea would be approved, so I didn't implement it here, but I wanted to make a popup for tangent editing that shows up when double-clicking a point. This would be similar to the gradient editor, which allows to edit a color when double-clicking a handle.

I'm testing this PR locally (rebased against latest master https://github.com/godotengine/godot/commit/06ccbfe8eec6e840db3df2360dc2804386f940b8) and can notice the points are offset from the drawn line:

Thank you for catching that! I don't believe I tested the change in range a lot. Was there anything specific you did (other than min != 0 && max != 0), or does it always happen? Does the same thing happen when editing the curve directly vs editing an "inline" curve on some other resource?
Rebasing everything and will try to do some testing also!

Nope, you're right, can confirm it also happens without the curve being inline:

This might be because of those scaling parameters used to place points or draw lines (I forget which, probably one but not the other). I think this speaks a little to the need to make changes in the readability/maintainability of rendering. Though as @MewPurPur suggested, due to availability/capacity concerns, the best way to address this might be to fix the bug in this PR and then do a further PR refactoring the refactor :)

Never experienced this even though I did test various ranges. Might be an unintended side effect of some recent tweak. Onto it. And TY a lot for catching this.

Ah ok, I hadn't tested with negative values in a while, this happens when the min value is negative.

Phew it's a super simple fix.

Oh, and while testing negative values, I encountered a bug that seems to also be present in the old CTRL snapping, which I hadn't fixed... but now have! So that's cool! This PR fixed like 10 bugs at this point lmao

After this is merged, the toolbar at the top can easily have its Presets button changed into a dropdown menu with common functionalities.

This is exactly the kind of UI improvement the Godot Editor needs! Thank You!


Multi-Label Classification with Scikit-MultiLearn

Proposed title of article
Multi-Label Classification with Scikit-MultiLearn
Introduction paragraph (2-3 paragraphs):
Multi-label classification methods allow us to classify data sets with more than 1 target variable, a prediction can belong to various labels. There are various methods that should be used depending on the dataset on hand.
Methods for solving Multi-label Classification Problems

Problem Transformation

Adapted Algorithm

Ensemble approaches

Problem Transformation
It refers to transforming the multi-label problem into a single-label problem(s) by using

Binary Relevance: treats each label as a separate single class classification
Classifier Chains: In this, the first classifier is trained just on the input data and then each next classifier is trained on the input space and all the previous classifiers in the chain.
Label Powerset: we transform the problem into a multi-class problem with one multi-class classifier is trained on all unique label combinations found in the training data.

Adapted Algorithm
adapting the algorithm to directly perform multi-label classification, rather than transforming the problem into different subsets of problems.
Ensemble methods.
Incorporate the merits of two previous approaches.

In this tutorial, we shall use all three methods in building our multi-label model using Scikit-Multilearn.
Scikit-multilearn is a BSD-licensed library for multi-label classification that is built on top of the well-known scikit-learn ecosystem.
Key takeaways:

Difference between Multi-class classification and Multi-label classification.
Installing and exploring Scikit-Multilearn.
Using all the three approaches in building and training our model.
Building a text classification model.
Finding the optimal method of the three.

References:
Please list links to any published content/research that you intend to use to support/guide this article.
Templates to use as guides

How To Guide Template
Software Review Template
Tutorial Template

Hello @lalith1403 and @WanjaMIKE, what is the status of this topic? Kindly advise.

Great topic 🚀 , make sure it matches the following:

Adds value beyond what is in the official docs and what is covered in other blog sites.
To avoid potential article closure, it should not overlap with existing EngEd articles or incoming topic suggestions.

Please reference any relevant EngEd articles in yours and build a unique project - Approved. @charles721


Why is the layouts folder empty?

No files in there to match the demo

Hi, I didn't get it, could you please explain it in detail?

Hmm, how do you install this theme?
There is a file inside the layouts, I don't understand what does the empty mean.

Someone asked this question and I visited this repo to see the theme. But in the layouts folder, you don't have any files except the sidebar end one.

Thanks for the details, this repo is a template of the theme, that's why there aren't files in the layouts folder.
Btw, if I understand correctly, the question is about switching from HBS  theme to others, and how to add the sidebar with recent posts and featured posts on the new theme.


Add our school mission and vision to a suitable page

Is your feature request related to a problem? Please describe.
Currently, in our school website our school's mission and vision is not included
Describe the solution you'd like
Add our school mission and vision to a suitable page
Are you working on it?
Yes

Shall I work on this ?

My idea is to add it to the home page

Tell me your suggestions @Piumal1999 @jayasanka-sack @Hasthikagirihagama @sathsarabandaraj @anjula-sack


Non-starter

Hi  there! Very interesting idea here. Unfortunately, it fails pretty early for
me:
The config file is saved at ~/.config/configstore/downloadstation-cli.json, but
the token is not generated at first.
if (!url || !username || !password || !token) {
  return null;
}

And unfortunately, this block starting on line 16 of config-store.js prevents the token from being generated in
subsequent calls, even though line 22 of init.js would do this (if not for the
entirely null config object returned from config-store).
An other note - the trailing slash should be stripped from the url when saving
the config - otherwise the request urls will have a double slash. I fixed this
by manually editing the config file.
Re: the token issue - no matter - I tried to push through by getting my session ID out of the browser
cookie. With this set properly in the config, I still got either:
"ECONNREFUSED" if using my quickconnect URL
or
"Hostname/IP does not match certificate's altnames: IP: {REDACTED} is not in the cert's list:" if using the local IP address as the url.
These last two errors may be the fault of my personal settings, but I wasn't able to figure it out.
I was able to successfully get a list of Download Station tasks by copying entry.cgi the request as curl from the network tab of my browser. There I noted that the copied request uses v2 of the Download Station API, and this cli appears to use V1. That's probably not a problem, but I found it curious.

Hello @AdamWagner, thanks for your comments.
The first 2 errors are fixed on https://github.com/skydiver/downloadstation-cli/pull/4

The Hostname/IP does not match certificate's altnames it's because you are trying to connect to an ip or hostname which the certificate is not valid.
I think you have 2 options:

connect without https, just http://<your_ip>:5000 (port 5000 is for http)
use NODE_TLS_REJECT_UNAUTHORIZED=0 environment variable before running the app

Thank you! I'll give it another shot!


Select Grant Application direct in Editor tool #44

OVERVIEW
Choose a Grant application to work on in the editor. Feature to switch existing Grants directly in same page
ACCEPTANCE CRITERIA
User can see the list of available Grants and can select a grant application in the editor, switching active Grant.
OUT OF SCOPE
BACKGROUND/REFERENCE
Simple solution may be to adda dropdown list on Editor page with available grants and let user choose and switch grants. This maybe bad UX if ORG has many grants list will be large and require some filters. Search as Modal dialog where to search and filter what grant to use.

I'd like to revist this - it may make more sense to move the Grant Editor tab on the other side of the Grant Application tab to reinforce the process priority on moving from Application to application.

completed as requested


Support all value types

WIP.

Just for the record, unmarshalValues is also part of the production shard, and is called while iterating over the shard (I had assumed after the discussion above that is was actually just by ReadSeries). So right now all type "conversion" is being done at the low-level, and no reference is made to the Metatstore when performing the conversion, that I can see.

Obviously using a single source of truth for field types would be best, of course.

@pauldix @benbjohnson -- please review these latest changes and let me know what you think. This is still WIP, but I think it's close. I'd like some feedback at this point. Points to note:

I basically just lifted the "marshal" and "unmarshal" code from shard.go. There are some potential optimizations there -- and we should do them -- but wanted to get this working first.
I just added a new interface to the cursor, such that it expects a field decoder. I think this is fine, but for now I just slammed the Measurement object in there. Doesn't feel great.
@benbjohnson suggested a dedicated field codec, with a static of set of fields, presumably to avoid races. But we seem to have the same race with mapValues and there is no locking there. Is this code special, or we just haven't fixed up mapValues yet?
All integers are still stored internally as floats.
While the build is green, I haven't actually added any tests for booleans and strings yet.

I just noticed there is one more place that needs change:
https://github.com/influxdb/influxdb/blob/master/server.go#L1682

We need to also support raw bytes as a type.
On the mapValues part of things, almost all the aggregate functions look at only one field. It would be nice to have a function that takes this field ID and returns the raw value without marshaling it into a map. We should be able to just cast the raw bytes from the DB into the type.

How would a user insert raw bytes?

Materializing into a map was just to get it working. It definitely need to get optimized. As for the mapValues(), the lack of locking is a bug. Queries can potentially run for a while so we need to be able to copy the field metadata to the codec to be used during query execution.

I'm looking at some other stuff on the query side in terms of raw data queries. I think it would be useful to have the iterators yield the raw bytes to the map functions. Then have the iterator have a method that will pull a field value from those bytes.
This will be useful for queries that need to access multiple fields in the map portion of the query.

Field codec split into its own type, which allows the locking to be decoupled from the server lock. I still need to add support for other types, and have questions about interactions with the shard iterators.

Unit tests added for string and boolean types.

LGTM. Drop it like it's hot. Where's the Snoop Dogg emoji when you need it?

Thanks for the review @pauldix -- merging now.


Generated sourcemaps differ from LESSC and fail to resolve.

Im struggling to get sourcemaps to resolve correctly.
The following is a generated sourcemap from CLESC which chrome can not resolve:
{"version" : 3,"sourceRoot": "/static/","file": "static/apps/_/less/ie-fixes.css","sources": ["static/apps/_/less/ie-fixes.less"],"names": [],"mappings": "0TAME,qCACE,iEACI,SAAU,CARlB,CAQoB,CARpB,CAQsB,EACd,KAAM,CATd,CASgB,CAThB,CASkB,EACV,aAAc,CAVtB,CAUwB,CAVxB,CAU0B,"}
The below is a generated sourcemap from LESSC with the same invocation options which chrome can resolve:
{"version":3,"sources":["/static/apps/_/less/ie-fixes.less"],"names":[],"mappings":"AAMsC,oCACA,iEAC9B,SAAA,EAAA,EAAA,EACA,KAAA,EAAA,EAAA,EACA,aAAA,EAAA,EAAA"}

Thank you this is now fixed!
This may not be the appropriate place to comment - but this now means I can use CLESSC in our dev eng. Would you consider making this a 1.0 release? If helpful I could provide binaries for Ubuntu and OSX


Fix sourcetree appname on case-sensitive filesystem

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

The Appname is Sourcetree.app not SourceTree.app

Thanks for the PR!


Restore windows position when app is restarted

Closes issue #181.
I am not sure about the cleanness of reading saved state in the main window module, feedback/alternative approaches are more than welcome :)

Hi, I added some comments to your PR.
However an alternative to the current approach could be to always load the window with default settings, and then dispatch the bounds back from renderer/main.js during initialization. This way we could avoid adding State in main/window/main.js since it currently belongs in renderer/lib/.
Probably need more opinions on this though! 😅

Thanks for the feedback! I agree the line comments you shared.
Maybe not including State in main/window/main.js would be more appropriate. I tried something along those lines, but I could see the window moving from the default position to the saved one, not really a nice effect. I probably messed up somewhere.
If we decide to go with this PR, I can fix the minor issues you pointed out in your comments.

If the movement is visible I think loading State in main/window/main.js is better 👍
We could also load the state in main/index.js and pass it to windows.main.init().
Either way the state module should probably be moved then, but maybe someone has a better idea?

Closing this PR as this was fixed in #827 👍


Blocks without transactions return array with empty string inside

GET http://localhost:1984/block/height/0
{
  "indep_hash": "4eqrcbdscydag0th2mgs4wn2d7q0w9k2x23k3mzh221xcwf6kwleakdi4nrb2v50",
  "timestamp":<PHONE_NUMBER>,
  "previous_block": "",
  "height": 0,
  "txs": [
    ""
  ]
}

In the SQLite blocks table, this block has no transactions.  I think the fix here is to just return an empty array.

As block txIds are stored as a comma delimited string, the call to split is the source of the error in both block routes.
''.split(',') // results in ['']

Thanks @jim-toth for noticing this. Fix has been released on version 1.1.58.


[BUG] Themes don't work if Foundry uses routePrefix

Describe the bug
Themes don't work when Foundry uses routePrefix.
Simple calendar uses URL without routeprefix, for example:
https://hostname.domain/systems/wfrp4e/ui/sidebar_button-background_grey.webp
but is should use:
https://hostname.domain/routeprefixhere/systems/wfrp4e/ui/sidebar_button-background_grey.webp
To Reproduce
Steps to reproduce the behavior:

set routePrefix in Foundry config
set a theme in SC config, for example Warhammer theme.

Expected behavior
Simple calendar should work when routePrefix is set.
Foundry Information

Foundry Version: 6.3.1
Game System: warhammer
Simple Calendar Version: 2.1.80

Hey sorry about the long delay in getting back to you about this (had a couple months of medical issues).
This issue should be fixed in version 2.2.0!


How to deal with framework attribute bindings, updates to custom icon component?

Currently I'm wrapping using span tags in an Ember component

which is working great for initial loading & replacement.
Unfortunately, if I need to replace classNames or attributes on the icon element, those changes aren't reflected on the now-replaced SVG (the attribute bindings provided by the framework are lost once Iconify takes over and replaces the SPAN).
My use case is similar to:
<IconComponent @name="fa-solid:chevron-down" class={{if item.open 'rotate-180' 'rotate-0'}} />

What's the best way to approach this? Potentially, if there was a way to leave the initial SPAN/I tag, I could keep the binding and observe that element for className changes, and manually add it to the SVG.
I could handle rendering each and every icon myself with the SVG functions and leverage the above strategy, but it seemed a bit more straightforward to let the SVG framework do it's thing and replace all in a batch update, rather than on a per-component basis, seeing as there could be several dozen icons on one page.

Best way to approach this is to create a native component for Ember. Recently I've streamlined all code, making it easy to reuse for various components and created 4 components with it (React, Svelte, Vue 2, Vue 3), so it should not be hard to create Ember component. I'll take a look at that.
Another solution is deal with node directly if you can get parent node. I'm not familiar with Ember, so I'll use React as example. In React you can get reference to any element, so if you get reference to parent element, you can use it to find child element and change its attribute. This is quirky, but doable.

@cyberalien thanks for the suggestion. I'm not as hopeful as you that it's as easy to create an Ember component as React, Svelte, and Vue, but hopefully I'm wrong! 😅

Component is in attachment, everything seems to be working except for 2 things:

Inline attribute: <IconifyIcon @icon="mdi:home" @inline={{true}} />
Color attribute: <IconifyIcon @icon="mdi:home" @color="red" />

Problem is both require adding inline style to SVG, but there are some weird shenanigans with adding style, so it doesn't support it yet.
Both can be set by style instead: <IconifyIcon @icon='mdi:home' style='color: red; vertical-align: -0.125em' />
Unpack archive, add component to app/components/.
As soon as I solve issue with style and figure out why it doesn't work as addon (my primary suspect is Ember doesn't like monorepos), it will be published as @iconify/ember package.
Archive.zip

Finally got it working.
Install @iconify/ember as dependency.
Use <IconifyIcon /> in templates. All component properties start with @ and @icon property is required.
All functions that are available in other components, such as addIcon(), addCollection() and so on are available in Ember component too. You need to import them from @iconify/ember/components/iconify-icon:
import { disableCache, addIcon } from '@iconify/ember/components/iconify-icon';

disableCache('all');

const iconData = {
  body: '<path d="M4 19h16v2H4zm5-4h11v2H9zm-5-4h16v2H4zm0-8h16v2H4zm5 4h11v2H9z" fill="currentColor"/>',
  width: 24,
  height: 24,
};

addIcon('demo', iconData);

I'll do more tests, will write documentation and will publish 1.0.0 when it is ready.

@cyberalien Great! Sorry, I'm bouncing around different things at the moment, but will leave feedback here once I get a chance to implement this component (hopefully by end of week).

@cyberalien I'm way behind on this, I apologize, but I finally found the time to test this out. All seems to be working great!

Thank you!


Map & Character hero in different languages

Hi how do you manage the map & character name on hotslogs as they are different for every localization?
Do you use a a map id / character id from replay.details or any other way?
thank you

I keep a dictionary of every possible hero name in different languages :)
In the replay attributes file, there is an attribute for selected Hero, and that uses a consistent 4 character string that is the same across all languages.
The only problem is this will show as 'Rand' when picking Auto Select, so you won't know what the actual Hero is when picking Auto Select.
There may be other places where you can find the selected Hero and Map in the replay file, and not have to worry about the above issues.  I haven't found it yet though


Improved external user discovery / Added TOC

Category

[x] Content fix
[ ] New article
[x] Example checked item (delete this line)

Added additional details about external user discovery in channel settings and Teams Admin Center.
cc: @JimmyHang

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

File
Status
Preview URL
Details

Community/TOC.md
:white_check_mark:Succeeded
View

Community/glossary.md
:white_check_mark:Succeeded
View

Community/index.yml
:white_check_mark:Succeeded
View

Community/media/teams-shared-channel-for-admins/tsc08.png
:white_check_mark:Succeeded
View

Community/media/teams-shared-channel-for-admins/tsc09.png
:white_check_mark:Succeeded
View

Community/teams-shared-channel-for-admins.md
:white_check_mark:Succeeded
View

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


Home and End sometimes do not work to scroll to the top and end of the page

Description
Sometimes, pressing the Home and End keys do not work as intended to scroll to the top and the bottom of the page. Although Ctrl+Home and Ctrl+End do work when this happens.
Refreshing the page seems to fix the issue.
How to replicate the issue + URL
I have not been able to reproduce it consistently.
There is no specific place where it happens. The page just has to be scrollable.
Extension version
22.11.6
Browser(s) used
Firefox 106.x-107.x

This isn't something related to Refined GitHub. Also a demo or screenshot of what you're experiencing is always useful, as suggested by the issue template.

This isn't something related to Refined GitHub. Also a demo or screenshot of what you're experiencing is always useful, as suggested by the issue template.

It never happened before I installed Refined GitHub.
And a video of it happening will require me to be recording all the time since I can't know when it's going to happen, and can't reliably reproduce it.

If pressing a key doesn't work, you can start recording on that moment, not all the time. Mostly I'm interested to see how the page is scrollable with the mouse wheel when the home key doesn't work.
In any case, we don't listen to that key specifically; if it works with ctrl, there's not even the chance we're involuntarily blocking the key.
At most, RG is causing additional scroll somehow, but without a demo it's impossible to determine.

Thanks for the info. I'll record it the next time it happens.


Cache Improvements

Problem
Currently caching is done by collector/cacheStakingReward by accessing every endpoints periodically, which is not horizontally scalable.
To Be
We need to find a cache method that fits the blockchain. Every GET queries can be cached until next block comes. For Dashboard endpoints, we should change queries to calculate yesterday data and add it to another table, now we are querying the whole data from the beginning every single day.
To Do

[x] ~Move memoizee to higher level~
[x] Remove cacheStakingReward
[x] Cache for /block/{height}, /tx/{hash}
[ ] Find / Implement solution for Dashboard

Not sure how relevant to this issue it is, but caching will be implemented on the api gateway layer level. current block data will be cached for 1sec only while other endpoints will have longer retention.
This will provide more insight via the api gateway stats on what cache is hitted/missed the most, etc and will help determine if a more advanced service side caching is necessary (or not)


Define routes available in App

If we check the Design (#2) we can note these routes:
home or root (/)

[ ] #42

Related issues:

#12
#22
#23
#29
#32

check

[ ] #48

Related issues

#20
#24
#25
#26
#32
#41

map

[ ] #39

Related issues

#3
#4
#27
#28
#36
#38
#44

chat

[ ] #40

Related issues

#31
#34

confirm

[ ] #43

Related issues

#21
#30
#45
#46

error 404 and error 5XX

[ ] #49

[ ] app error 5XX

TBD
Notice:

Should include id and timestamp of error
Should include developer/technical staff contact information
Should have link to Home or other TBD

Define routes is different that create the views, or I'm wrong?. I think the "related issues" should be in each issue and not here, since if this issue is not about complete the views to 100%, would be more useful if the information is in each issue that yes is dedicated to complete the view.
Confirm what is the objective of this issue:

Define and create the route links to access to the views (This issue will be closed when are defined the URL and view files are created)
Create the views (This issue will be closed when all views look and work how is expected)

All the user interface seems covered with those routes, i like the name of the sections, but i would change map for menu or something that includes more of the whole functionality of that section.

What about delivery for the view related to show the map and chat, confirm options? @AlejandroSalcido

What about delivery for the view related to show the map and chat, confirm options? @AlejandroSalcido

delivery would be better, it is a more accurate name!

When we manage internal errors we can close this


do not raise an IncompatibleOptionValues For a couple of potentially valid use cases

I believe the following two params blocks should be valid:
# The default is the entire range of allowed values
params { optional :numbers, type: Array[Integer], values: 0..2, default: 0..2 }

# The default is a subset of the entire range of allowed values
subject.params { optional :numbers, type: Array[Integer], values: [0, 1, 2], default: [1, 0] }

The current implementation doesn't account for these use cases and raises Grape::Exceptions::IncompatibleOptionValues.  This PR will allow the two cases above.

Looks good! :+1:


feat(API): rename existing auth client to authorizer, add new endpoints

Renamed the auth client directory to authorizer
Added new endpoints to the authorizer client:

GET /authorizer/api/v1/cas
GET /authorizer/api/v1/cas/{id}
GET /authorizer/api/v1/cas/{id}/crl
POST /authorizer/api/v1/ca/authorize
GET /authorizer/api/v1/principals
GET /authorizer/api/v1/principals/{group_id}
DELETE /authorizer/api/v1/principals/{group_id}
POST /authorizer/api/v1/principals/{group_id}/create
POST /authorizer/api/v1/principals/{group_id}/import
POST /authorizer/api/v1/principals/{group_id}/sign

Renamed the functions.


Missing dotnet client library for identitytoolkit.googleapis.com service

Hi. Trying to manage Identity Toolkit API for users and tenants required to create either a custom library or build a generated library based on the discovery documents.
Is there a reason why the dotnet client library missing?
I'd expect to have nugets, something like Google.Cloud.IdentityToolkit.V1 and Google.Cloud.IdentityToolkit.V2, in nuget.org repository and maintained by Google Team.
Additional context
Some observations

There are two nugets mentioned because of v1 is responsible for user management and v2 for tenants. Both APIs are officially referred and documented.
Google APIs Discovery service, that seems to be used for generating client libraries, does not define those two services but points to identitytoolkit v3 (No identitytoolkit v3 nuget, even if it were, it is undocumented and at least does not cover tenants functionality). Should identitytoolkit v1 and v2 be mentioned in discovery document either?

Thanks

Closing this one in favor of googleapis/google-api-dotnet-client#1605. The Identity API is REST and so should live in googleapis/google-api-dotnet if anything.


Error updating nix store on NFS: removing extended attribute ‘system.nfs4_acl’

Issue description
Was about to update my nix-store a Fedora 25 workstation:
$ nix-channel --update
downloading Nix expressions from ‘https://d3g5gsiof5omrk.cloudfront.net/nixpkgs/nixpkgs-18.03pre116369.14cbeaa892/nixexprs.tar.xz’...
downloading ‘https://d3g5gsiof5omrk.cloudfront.net/nixpkgs/nixpkgs-18.03pre116369.14cbeaa892/nixexprs.tar.xz’... [8991/9145 KiB, 640.3 KiB/s]
$ nix-env -u
error: removing extended attribute ‘system.nfs4_acl’ from ‘/nix/store/ijxblasjbl4l53gd3cg3q49ci8nh6wj1-env-manifest.nix’: Input/output error

This used to work for years.
The store is located on an NFSv4 mount served over network. Its location is
~/.nix-store, bind-mounted at login to /nix.
Steps to reproduce
# set up nix and nixpkgs; then
$ nix-channel --update
$ nix-env -u

Technical details

System: (NixOS: nixos-version, Ubuntu/Fedora: lsb_release -a, ...)
$ uname -a
Linux drift.m.i2n 4.12.11-200.fc25.x86_64 #1 SMP Fri Sep 8 11:44:51 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux

Nix version: (run nix-env --version)
$ nix-env --version
nix-env (Nix) 1.11.14

Nixpkgs version: (run nix-instantiate --eval '<nixpkgs>' -A lib.nixpkgsVersion)
$ nix-instantiate --eval '' -A lib.nixpkgsVersion
"18.03pre114739.d0d905668c"

Sandboxing enabled: (run grep build-use-sandbox /etc/nix/nix.conf)

Does not apply: Not NixOS.

Probably caused by NixOS/nix@d798349ede3d6eb6e92a2e4f95f6b2179407ceb9.

Looks like. Anything I can do?

We can probably ignore system.nfs4_acl, as with security.selinux, but we'd have to be sure that this leaves no ACLs behind...

Could this be made configurable? I imagine other situations may arise where users need to whitelist attributes because of an exotic setup.

Just having the same problem on a Lustre file system:
error: removing extended attribute ‘lustre.lov’ from ‘/nix/store/y2y67fng7drrqlgfwf92mlf62lmgg7zj-perl-5.24.3/bin/perl5.24.3’: Permission denied

Hello,
Any news? Because of that error I can't even install nix.

@knedlsepp : Thank you for the tip, but do you have any idea why this fix is not included upstream?

I'm having this issue in nix 2.0.2, which I just built manually from source. I'm confused as to how to apply the workaround @knedlsepp mentioned (since I'm just learning how to use nix); it looks like after dropping that file (in my home directory? Or somewhere in my nix store or state directories?) I need to use nix to rebuild nix. Is that likely to work if I can't update the channels? And how would I go about doing that?

I'm having this error in Nix 2.1.3 with a CephFS filesystem:
error: removing extended attribute 'ceph.dir.entries' from ...: Operation not supported

I'd also appreciate if this were made configurable, since indiscriminately removing xattrs from files can cause problems in unusual setups.

Is there an easier work around like granting the chattr rights to the user, who does the single-user-installation of /nix in ubuntu?
sudo nix-env ... does not work.

Still important to me. I have the same issue but with btrfs.compression.

Still important to me. I can't install nix because of this error. Is there any workaround I can do when installing?

Still important to me. I can't install nix because of this error. Is there any workaround I can do when installing?

Note the simple workaround of using NFSv3.

Linux kernel NFS client code disables removing system.nfs4_acl xattr
https://github.com/torvalds/linux/blob/9b76d71fa8be8c52dbc855ab516754f0c93e2980/fs/nfs/nfs4proc.c#L5947


Make alerter not use Order struct from model

The motivation is that if we add a new mandatory field the staging
alerter would fail to read the prod api. This also makes the alerter
more independent of the rest of the code and allows it to detect if we
accidentally make an incompatible change.
Downside is that it is more work if we do want to breaking change the api but I feel it is reasonable to do it by hand in that case.
Test Plan
manually ran to see it works with prod

Codecov Report

Merging #1217 (8a59150) into main (2d219da) will increase coverage by 0.00%.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##             main    #1217   +/-   ##
=======================================
  Coverage   67.18%   67.18%
=======================================
  Files         135      135
  Lines       27503    27500    -3
=======================================
  Hits        18477    18477
+ Misses       9026     9023    -3

Impacted Files
Coverage Δ

alerter/src/main.rs
0.57% <0.00%> (+<0.01%)
:arrow_up:

Continue to review full report at Codecov.

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


python/python3-modules: added pip + wheel, small fixes

small fixes for setuptools
added pip
added wheel

if not needed, we could close this one. all this is also included in the pipython3 layer repo here:
https://github.com/mahaase/pipython3

I've just merged #166 where I added you as a co-author. Thanks for putting this forward...


[ASK] how to free cam?

Acknowledgement of preliminary instructions

[X] I have read and acknowledged the preliminary instructions.

What idea do you have?
idk how to free cam

This came out of nowhere but doing the SHIFT + P does not work

The freecam option won't work, just ignore it.

Is the others apart of bloxstrap?

They come from you enabling this option.

The freecam option isn't supposed to work, just ignore it.

Can u make an freecam option ?


Create user login and signup pages

Use express-passport-local to allow users to create accounts.
There are some guides here:
https://scotch.io/tutorials/easy-node-authentication-setup-and-local
But anything works.
The urls can be:
/user/login
/user/signup
Steps are roughly like:

Create a User model
Setup express passport
Create the views for login and signup
Take the users when they're logged in to users/profile
Restrict the profile page to users who are not admins
Add buttons for login and signup to the homepage
Include roles in the user model, possible roles are [standard, admin, superadmin, chef]
Create the serializers for the users, so that the role can be accessed in controllers

@jideokus will be working on this.


Dev/clone task

Functionality:
Clonning a Task
Solution:
Copy a Task which is already created
Risk level:

[ ] high
[x] medium
[ ] low

How to test:
firebase emulators : start
ng serve

Now @shivamgoyal7 Can You Check

Demo is done and changes are requested.


Fix console warnings when running examples.

Removes console warning due to mixed use of margin shorthand with specific marginX properties.

Coverage remained the same at 94.892% when pulling 12a4228f6531fe5dde468b163ae1defb7e144d99 on stevenmusumeche:fix/example-warnings into 9f7af1676a46048d6c39270b4898b486e1ebdde2 on FormidableLabs:master.

Coverage remained the same at 94.892% when pulling 12a4228f6531fe5dde468b163ae1defb7e144d99 on stevenmusumeche:fix/example-warnings into 9f7af1676a46048d6c39270b4898b486e1ebdde2 on FormidableLabs:master.


decapitalized json in late materialization

Decapitalized json in late materialization. They were capitalized by mistake.
http://jenkins01.arangodb.biz:8080/job/arangodb-matrix-pr/7520/

http://jenkins01.arangodb.biz:8080/job/arangodb-matrix-pr-windows/1494/

All blue except windows which can not start.


Use lower TCP ports for unit tests

This avoids conflicting with kernel-allocated high ports.

LGTM

LGTM


Rework Server Rules

This is an open consultation on updates to the Server Rules, on which we would appreciate feedback.
Whilst Discord feedback will be accepted, GitHub feedback is far easier to track, so we'd prefer it here if possible.
This pull request will not be merged until we feel that all appropriate feedback has been enacted, and the old rules remain in place until this PR is merged.
Resolves #312

Also, Dyno has not had the invite exceptions added yet, so don't get annoyed if it deletes them.

not bad

Looks like there's an error on line 54 of RULES.md.
https://discord.cyberdiscommunity.uk is not a valid URL.
Should be https://discord.cyberdiscoverycommunity.uk

seems pretty good.
some of the punishments seem a bit extreme to me but thats the mods decision to make and at least theyre now documented.

@Little-Green-Guy what changes would you propose?

Another typo on line 57.
Should be https://discord.gg/python
not https://discord/python

@lightspeedana none really. straight from a warning to a 1hr mute seems a bit extreme, but it is after a warning, and no doubt there would be a reminder that a 1hr mute is next so its kinda fair
also as to the harassment point, as sometimes people may not realise what they are doing counts as harassment, a straight up 14 day ban (or even permanent) seems extreme. maybe a few hour/day mute first to establish that it isnt allowed, then a temp ban as soon as it is repeated.

Already said this in discord, but it would be nice to see some clarity on whether the punishment length is decided by the severity of the action or by if they have already been punished for the same action.

uh why is there a mention of sexual harassment in this? Think that's a police issue, not a 14-day ban issue

Actually, although it may not need to be included specifically in the rules, can there be some clarification on copypastas?
Of course when copypastas are sent multiple times its spam and should be removed and treated as such, however if its only sent once, its not spam, and I was under the impression it didn't break the rules, and imo it shouldn't be removed. Could this be clarified, one way or the other?

This is pretty rad and a really awesome step in the right direction. Perhaps, an example of what is and is not acceptable would be useful - also, clarification on what's allowed in overflow, memes, etc.