[discussion] Container responsibilty

I agree on having a separate container so that it could be changed with any other interop container interface. But i believe it's not the container's responsibility to create request/response/error handler instances. Those services / instances should be created by service providers, or just by the app instance acting as service provider for them.
If we want to use another container we would otherwise have to extend it and rewrite (copy and paste) the same ctor code to provide those initial services.

I agree that it's not the role of the container to create request and response objects (already voiced that in another comment but can't find the link). I don't think it's a big deal but having those factories inside the app (or another class) would help replacing containers more easily, so +1.

Hi @mnapoli
sorry I missed your comment (I just got back to this project after a long "eu-cookie-law websites fixing week")

No worries I can't even find where it was. It's good to have an issue to discuss that.

This one I guess https://github.com/slimphp/Slim/pull/1179#discussion_r30459907

PR #1278 moves the creation of the Request into a factory method, so that Container is simply:
        $this['request'] = $this->factory(function ($c) {
            return Http\Request::createFromEnvironment($c['environment']);
        });

Looking at the rest of the factories in Container now, they all seem really really trivial to copy/paste into your own container if you're using something else.

Hello @akrabat,
It's really not about the simplicity of it, but more about the correctness of doing it (and also redoing it) in a container.
Let's take for instance the private property $defaultSettings. This is about cookie, http message settings or more generally application level configuration/settings. In my opinion a container should only provide new objects or shared services. The actual Container class in slim seems to me more like a not so slim slice cut off from the application cake.
A different container should only need to implement the get() interop interface method to fit in and replace the current one.
A class name such as \Slim\AppServices composing a simpler replaceable container would seem more sound to me for what it does now.
These are just my first impressions, i will re-read and rethink about it next week, maybe after reading other opinions.
Of course, if it's already been decided for this implementation, you can close the discussion instantly.
kind regards
maks

In some ways, Slim is intentionally opinionated; you're supposed to use our Container (and our Router).
Slim 3's App was originally written to extend Pimple so that changing the DIC was impossible. However, we have done enough work to enable replacement of Container with your own one, but you have to do a small amount of integration work.
Personally, I think that the number of people what will change the DIC is small and would be able to provide an integration package on Composer for this DIC of choice.
I'm happy with this state of affairs as I particularly don't want any additional complexity such as two containers in play. I've done this with Slim 2's Set composing Zend\ServiceManager and it's a right old pain.

Hello @akrabat
Along with You and - I believe - many other developers, I also think that slim should stay as slim as possible (we already have zf2 if we want greater SOLID compliance and more granularity).
If we want it slim, then we cannot have too much separation of concerns.
For the same reason if I had to choose between having a forced opinionated container and the ability to switch to another one with the condition of having to rebuld all the basic application services i would prefer the first and older way, even if i hate getting "things" via array access.
A better solution to me would be having an internal Slim Container (Service Locator) interface with an added set()/factory() method so that a container  would be a simple has/get/set/factory custom implementation or a 3d-party container bridge. All the basic (lazy) services creation could be moved back to the application constructor and the default slim container implementation could be a simple Pimple\Container bridge or decorator.
Shortly: switching to a different container should involve only the way or the actor in service creation and retrieval, the container should not be responsible of initializing the application.
As of now implementing a different container basically means rewriting the whole application bootstraping process and that also allows a dangerous flexibility. As an example, let's say (we discussed something related in another issue/pr) I implement a container that create an instance of phly/zend-diactoros server request. All the slim request's extra public methods would not be available anymore, and a change of container may result in runtime errors for the same unchanged application code. To have all those extra methods always available, a slim app must always start with its server request implementation and only allow external middlewares returning modified clones of it. The simplest/slimmest way to do that is having the application be the factory of the request instance like it was done before.
Another issue: suppose we have parts of the application that use the Pimple container factory() method to store a factory service. If we change the container, the new one may (probably) not have the factory method or may have it with a different signature. Well, since App::getContainer() simply returns a ContainerInterface we shouldn't even think about using the factory() method, so must we register all our factories inside the Container constructor? Simply put: how do i safely add factories to the generic service locator?
If we want to allow using different containers we should define a clear slim internal container (or more precisely a service-locator) interface.
After all what popped in my mind I feel that the current implementation of DIC independence adds more complexity and at the same time is taking away useful features that we assumed available when we were sure we were using Pimple (btw, i am not a Pimple fanatic, I just use it in some projects).
I didn't follow the whole #1179 discussion or other related discussions. If this argument was already largely discussed and "voted", feel free to close this. I surely don't want to cause any delay for the milestone release. These are just doubts that came to my mind.
kind regards

I don't think your application should set anything new into the container after App is instantiated. If you choose to replace Slim's Request object with another one, then Slim will work, but your app needs to be written to target your new Request object or the PSR-7 baseline.
I'm happy with what we have at the moment.

Same. We can revise for 3.x or 4.x later.

Now, I've been at odds with this approach myself for some time, and I remember discussing it with you before @codeguy, so forgive me for jumping in again here.
My preference would be to see the container filled by a bootstrap.php file which can be included in a project or replaced. There are enough exceptions through the codebase and the included components to show a developer what is missing and a bootstrap file can act as a guide, so it would seem to me that Slim can afford to be a little less opinionated in it's structure.
Container is a component of the framework, as is App, and all the other parts. Building contents of a container for an app to function is bootstrapping the app, so logically, it shouldn't happen inside one of the components, it should happen outside. All the container should do is employ the Interop\Container\ContainerInterface on top of Pimple\Container, not define the contents.

Not for 3.0 Maybe 3.x.


Trailing / in moonraker url on command line fails to work

Compare the two
C:\Users\accou\OneDrive\Reprap\klipper_estimator>"C:\Users\accou\OneDrive\Reprap\klipper_estimator\klipper_estimator.exe" --config_moonraker_url http://voronpi.lan/ post-process "c:\Users\accou\OneDrive\Desktop\20mm-box-Doc 2.4-Voron eSun Abs+ Purple.gcode"
Failed to load printer configuration: reqwest::Error { kind: Decode, source: Error("missing field `result`", line: 1, column: 390) }

C:\Users\accou\OneDrive\Reprap\klipper_estimator>"C:\Users\accou\OneDrive\Reprap\klipper_estimator\klipper_estimator.exe" --config_moonraker_url http://voronpi.lan post-process "c:\Users\accou\OneDrive\Desktop\20mm-box-Doc 2.4-Voron eSun Abs+ Purple.gcode"

C:\Users\accou\OneDrive\Reprap\klipper_estimator>

Also typo of reqwest should be request

Hi!
I couldn't reproduce this, but expect it's somehow related to using/not using a reverse proxy in front of Moonraker. I improved the robustness here so it should be good in next release.
reqwest isn't a typo, it's the name of the library used for HTTP requests :-)
Best regards
Lasse


(multiple) fixed misleading text in advanced options ui

I tested the advanced option "Name Under Marine in Multiplayer" RD_AdvancedSettings_HUD_NameUnderMarine and found out the setting "No Team Colors" actually disables drawing the player- and the marine-name. So i fixed it. For languages which already had translated this string, i took their translated "Disabled" value from the key RD_AdvancedSettings_WarnAt_None.

Swedish is corect here too.


AstTypeStubPaths_* test failures

AstTypeStubPaths_MergeStubsPath
AstTypeStubPaths_NoStubs
AstTypeStubPaths_MergeStubsPath
AstTypeStubPaths_ExclusiveStubs

They all fail like this:
System.AggregateException: One or more errors occurred. ---> Microsoft.VisualStudio.TestTools.UnitTesting.AssertFailedException: Assert.Fail failed. Failed to get module we just imported

This is not just tests - some imports are actually broken.


HDDS-2719. Rename S3Token to S3AuthInfo

What changes were proposed in this pull request?
Renamed references of S3TOKEN to S3AUTHINFO
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-2719
How was this patch tested?
Ran project unit tests

@bharatviswa504 We prefer to use ozone tokens for hadoop delegation tokens such as OM token, block token, and SCM/DN tokens (To be done in Security phase III).
The current S3 token is more like a standard S3 access key/secret, which we prefer to use standard S3 terms.


Incorporate updates for digest updater workflow on downstream as well

Related to:

https://issues.redhat.com/browse/RHOAIENG-2928
https://issues.redhat.com/browse/RHOAIENG-2924

Upstream PR:

https://github.com/opendatahub-io/notebooks/pull/474

Description

This PR updates the digest updater workflow to the following points:

Now, considers also the new notebooks such as code-server
Updates runtimes (No more manual update!)
Update and clean up the overall code

The rest errors of the code-static-analysis will be fixed once we sync the repo with upstream changes

/override ci/prow/images


Reproduce the COCO result.

Hi,
May I have known your precision on COCO datasets (mAP) by using mobilenet-v2-ssd config file.
Best

@atranitell Hi, I haven't train the model in COCO datasets. I just retrain the model in my dataset with 8 classes. The mAP is 69.5.
I sorry for can't tell you the result

@urbaneman thx your reply, I have reproduced it as same as the reported result.

@urbaneman thx your reply, I have reproduced it as same as the reported result.

Hi, may I ask how high precision you can get on the coco data set? the map I have reproduced was 18，which is much lower than the reported result.
Best

@urbaneman thx your reply, I have reproduced it as same as the reported result.

Hi, may I ask how high precision you can get on the coco data set? the map I have reproduced was 18，which is much lower than the reported result.
Best

Hi, MobileNet-v1 SSD could achieve 18.x and v2 could gain 23/24.x precision.
Best,
KJ

@urbaneman thx your reply, I have reproduced it as same as the reported result.

Hi, may I ask how high precision you can get on the coco data set? the map I have reproduced was 18，which is much lower than the reported result.
Best

Hi, MobileNet-v1 SSD could achieve 18.x and v2 could gain 23/24.x precision.
Best,
KJ

Nice job！Would you show some config of the MV2SSD or the ssdhead, I have try many methods to improve the precesion.This problem has bothered me for a long time.
Best
zhaoxin


[3.0] Redstone blocks like buttons and pressure plates turn into different blocks when used

Per title. Buttons turn into lava and pressure plates into blackstone it seems.

Fixed


Re-add "run all" mode to performance tester

The run-all mode of the performance tester inadvertently got removed in
a previous patch. This adds it back.

@swift-nio-bot test perf please

@swift-nio-bot test perf please

@weissi A new NIO release knocked some allocs off, so I'm asking for a re-review from you.

@weissi A new NIO release knocked some allocs off, so I'm asking for a re-review from you.


Some tests do not pass correctly

The following are the issues:

click-sort-test: Nodes sort, but if you sort and then hit "double quantity", lots of nodes just disappear.
dynamic-root-node-test: add works, remove does not.
fn-value-test: nothing appears in grid cells.
select-child-test: grid values not shown; when click row the grid highlights/dehighlights correctly, but automatic (e.g. parent should de/select children, or all children should de/select parent) does not do grid part correctly.
select-prevent-test: rows 2 & 3 should not select because of preventDefault, but it still selects.

Wait for #232 to merge in before working on these.


Credit Transactions | Dual role Gov't user can see both the Pull Back and Return to Analyst buttons for a Reviewed PVR transaction |1|

https://trello.com/c/29F4xKuh/770-credit-transactions-dual-role-govt-user-can-see-both-the-pull-back-and-return-to-analyst-buttons-for-a-reviewed-pvr-transaction
Not sure what the solution is here. Going to leave it for now.
It will be a very rare occurrence that an IDIR user will be assigned both the Gov't Analyst and Director roles. This "defect" does not break the system and will almost never occur in reality, thus it may not be worth the effort to try and "fix" it. Creating this card just to document it as a potential issue.

 Credit Transactions | Dual role Gov't user can see both the Pull Back and Return to Analyst buttons for a Reviewed PVR transaction |1|


Moved the need for 'cypress/integration/config.js' into 'cypress.json…

…' as an environment variable. Thie removes the need to have to import the config.js file into every test.

Ooh yes, this is much better, thanks!


Problem setting up SSL connection - SSL routines:ssl3_get_record:wrong version

When SSL is activated I get src/Csocket.cpp:664 error:1408F10B:SSL routines:ssl3_get_record:wrong version number.
The output of openssl s_client -connect for znc server:
New, TLSv1/SSLv3, Cipher is ECDHE-RSA-CHACHA20-POLY1305
Server public key is 2048 bit
Secure Renegotiation IS supported
Compression: NONE
Expansion: NONE
No ALPN negotiated
SSL-Session:
    Protocol  : TLSv1.2
    Cipher    : ECDHE-RSA-CHACHA20-POLY1305

ZNC config:
SSLCiphers = EECDH+AESGCM:EDH+AESGCM:AES256+EECDH:AES256+EDH:ECDHE+RSA+AES256+GCM+SHA384
SSLProtocols = -SSLv2 +SSLv3 +TLSv1 +TLSv1.1 +TLSv1.2

I use ZNC 1.7.2+deb3.
Any ideas?

Are you trying to connect to znc or is this a problem when znc is trying to connect to some IRC server? I am just trying to make sure I understand correctly: You are providing the openssl output from trying to connect from your computer to znc, but in #1743 you say that you have the same problem as someone who could not connect from znc to an IRC network.
Also, what are you using to try to connect to znc? As in, are the error message from Csocket and the output from openssl that you provide above from the same connection attempt?

Sorry for confusion. I try to connect to ZNC by IRC client. I use Erc, but I tried one other client with similar effect. The ZNC connects to the networks just fine. I thought the problem is SSLCiphers related, that's why I posted output from the openssl client connecting to ZNC server.
Currently, I don't get src/Csocket.cpp:664 error:1408F10B:SSL routines:ssl3_get_record:wrong version number I mentioned before, it turned out I started ZNC with SSL but tried to connect without. Right now I don't get any message on the server side and the client just stays forever on "Opening Connection.." stage.
I put a bit more details in:
https://emacs.stackexchange.com/questions/63639/erc-znc-over-ssl

I solved SSL connection problems thanks to help from #znc users. The problem was caused by misconfigured network config in Emacs.


fix: use settings.xml when updating versions

@terrestris/devs Please review.
Fixes #11 and #12

is it possible to release a version with this PR please ?

is it possible to release a version with this PR please ?


Remove some old Analyzer codes 2

What type of PR is this：

[ ] bug
[ ] feature
[ ] enhancement
[x] refactor
[ ] others

The removed codes are all unused, could remove these codes safely.

[FE PR Coverage check]
:disappointed: fail : 0 / 1 (00.00%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/rewrite/ExprRewriter.java
0
1
00.00%


Gulp 3.9.1 minimatch and graceful-fs is depricated

[ ] Gulp 3.9.1 =><EMAIL_ADDRESS>=><EMAIL_ADDRESS>is depricated
[ ] Gulp 3.9.1 =><EMAIL_ADDRESS>=><EMAIL_ADDRESS>=><EMAIL_ADDRESS>=><EMAIL_ADDRESS>is depricated
[ ] Gulp 3.9.1 =><EMAIL_ADDRESS>=><EMAIL_ADDRESS>=><EMAIL_ADDRESS>=><EMAIL_ADDRESS>=><EMAIL_ADDRESS>=><EMAIL_ADDRESS>is depricated

Already documented here #1485


Autocomplete on imports doesn't respect python.analysis.extraPaths when we have symlinks

Environment data

Language Server version: 2020.7.2
OS and version: Ubuntu 18.04
Python version (& distribution if applicable, e.g. Anaconda): Python 2.7.17

Expected behaviour
Fill the configuration in settings.json:
"python.analysis.extraPaths": ["/home/user/lib"],

Packages inside /home/user/lib are symlinks to real path.
Type in the code:
from my

I hope the autocomplete will suggest all packages that start with my inside the /home/user/lib folder.
Actual behaviour
The autocomplete doesn't suggest packages inside the /home/user/lib folder.
Logs
Background analysis message: setFileOpened
Background analysis message: markFilesDirty
[FG] completion at /home/wesleybl/git/symbolic.package/src/symbolic/package/__init__.py:1:1 ...
[FG]   parsing: /home/wesleybl/git/symbolic.package/src/symbolic/package/__init__.py (1ms)
[FG]   binding: /home/wesleybl/git/symbolic.package/src/symbolic/package/__init__.py (0ms)
[FG] completion at /home/wesleybl/git/symbolic.package/src/symbolic/package/__init__.py:1:1 [found 23 items] (3ms)
Background analysis message: getDiagnosticsForRange
Background analysis message: getDiagnosticsForRange
Background analysis message: setFileOpened
Background analysis message: markFilesDirty
[FG] completion at /home/wesleybl/git/symbolic.package/src/symbolic/package/__init__.py:1:2 ...
[FG]   parsing: /home/wesleybl/git/symbolic.package/src/symbolic/package/__init__.py (0ms)
[FG]   binding: /home/wesleybl/git/symbolic.package/src/symbolic/package/__init__.py (0ms)
[FG] completion at /home/wesleybl/git/symbolic.package/src/symbolic/package/__init__.py:1:2 [found 15 items] (3ms)
Background analysis message: getSemanticTokens
[BG(1)] parsing: /home/wesleybl/git/symbolic.package/src/symbolic/package/__init__.py (1ms)
[BG(1)] binding: /home/wesleybl/git/symbolic.package/src/symbolic/package/__init__.py (0ms)
[FG] completion at /home/wesleybl/git/symbolic.package/src/symbolic/package/__init__.py:1:3 ...
[FG]   parsing: /home/wesleybl/git/symbolic.package/src/symbolic/package/__init__.py (0ms)
[FG]   binding: /home/wesleybl/git/symbolic.package/src/symbolic/package/__init__.py (1ms)
[FG] completion at /home/wesleybl/git/symbolic.package/src/symbolic/package/__init__.py:1:3 [found 11 items] (3ms)
Background analysis message: setFileOpened
Background analysis message: markFilesDirty
Background analysis message: setFileOpened
Background analysis message: markFilesDirty
Background analysis message: getSemanticTokens
[BG(1)] parsing: /home/wesleybl/git/symbolic.package/src/symbolic/package/__init__.py (1ms)
[BG(1)] binding: /home/wesleybl/git/symbolic.package/src/symbolic/package/__init__.py (0ms)
Background analysis message: setFileOpened
Background analysis message: markFilesDirty
Background analysis message: analyze
[BG(1)] analyzing: /home/wesleybl/git/symbolic.package/src/symbolic/package/__init__.py ...
[BG(1)]   parsing: /home/wesleybl/git/symbolic.package/src/symbolic/package/__init__.py (2ms)
[BG(1)]   binding: /home/wesleybl/git/symbolic.package/src/symbolic/package/__init__.py (0ms)
[BG(1)]   checking: /home/wesleybl/git/symbolic.package/src/symbolic/package/__init__.py (0ms)
[BG(1)] analyzing: /home/wesleybl/git/symbolic.package/src/symbolic/package/__init__.py (2ms)
Background analysis message: resumeAnalysis
Background analysis message: getSemanticTokens
[FG] parsing: /home/wesleybl/git/symbolic.package/src/symbolic/package/__init__.py (2ms)
[FG] binding: /home/wesleybl/git/symbolic.package/src/symbolic/package/__init__.py (0ms)
Background analysis message: setFileOpened
Background analysis message: markFilesDirty
[FG] completion at /home/wesleybl/git/symbolic.package/src/symbolic/package/__init__.py:1:6 ...
[FG]   parsing: /home/wesleybl/git/symbolic.package/src/symbolic/package/__init__.py (0ms)
[FG]   binding: /home/wesleybl/git/symbolic.package/src/symbolic/package/__init__.py (0ms)
[FG] completion at /home/wesleybl/git/symbolic.package/src/symbolic/package/__init__.py:1:6 [found 78 items] (8ms)
Background analysis message: analyze
[BG(1)] analyzing: /home/wesleybl/git/symbolic.package/src/symbolic/package/__init__.py ...
[BG(1)]   parsing: /home/wesleybl/git/symbolic.package/src/symbolic/package/__init__.py (1ms)
[BG(1)]   binding: /home/wesleybl/git/symbolic.package/src/symbolic/package/__init__.py (0ms)
[BG(1)]   checking: /home/wesleybl/git/symbolic.package/src/symbolic/package/__init__.py (0ms)
[BG(1)] analyzing: /home/wesleybl/git/symbolic.package/src/symbolic/package/__init__.py (1ms)
Background analysis message: resumeAnalysis
Background analysis message: getDiagnosticsForRange
Background analysis message: getDiagnosticsForRange
Background analysis message: getSemanticTokens
Background analysis message: getDiagnosticsForRange
Background analysis message: getDiagnosticsForRange
Background analysis message: setFileOpened
Background analysis message: markFilesDirty
[FG] completion at /home/wesleybl/git/symbolic.package/src/symbolic/package/__init__.py:1:7 ...
[FG]   parsing: /home/wesleybl/git/symbolic.package/src/symbolic/package/__init__.py (1ms)
[FG]   binding: /home/wesleybl/git/symbolic.package/src/symbolic/package/__init__.py (0ms)
[FG] completion at /home/wesleybl/git/symbolic.package/src/symbolic/package/__init__.py:1:7 [found 7 items] (12ms)
Background analysis message: getDiagnosticsForRange
Background analysis message: getDiagnosticsForRange
Background analysis message: getSemanticTokens
[BG(1)] parsing: /home/wesleybl/git/symbolic.package/src/symbolic/package/__init__.py (1ms)
[BG(1)] binding: /home/wesleybl/git/symbolic.package/src/symbolic/package/__init__.py (0ms)
Background analysis message: analyze
[BG(1)] analyzing: /home/wesleybl/git/symbolic.package/src/symbolic/package/__init__.py ...
[BG(1)]   checking: /home/wesleybl/git/symbolic.package/src/symbolic/package/__init__.py (1ms)
[BG(1)] analyzing: /home/wesleybl/git/symbolic.package/src/symbolic/package/__init__.py (1ms)
Background analysis message: resumeAnalysis
Background analysis message: getDiagnosticsForRange
Background analysis message: getDiagnosticsForRange

In this case, /home/wesleybl/lib/my is a symbolic link.

In fact, the problem only occurs if we have symlinks. I updated the title and description.

@judej @jakebailey I created a repository to simulate the problem. To see:
https://github.com/wesleybl/symbolic.package

Also running into this issue. On OSX/Mac

@judej @jakebailey added the log in the description.

Getting same issue in MacOS. Pylance isn't recognising the extraPaths property.

We need to add support for symlinks. Marked as enhancement

We need to add support for symlinks. Marked as enhancement

This issue has been fixed in version 2021.3.1, which we've just released. You can find the changelog here: https://github.com/microsoft/pylance-release/blob/main/CHANGELOG.md#202131-10-march-2021
Since symlinks are tricky, if you encounter some issues with this support we'd appreciate a new issue to pin things down. 🙂

@jakebailey unfortunately it did not solve the autocomplete problem: #1031
Would it be possible to prioritize this? Thanks for the great job!


Change name

I think with the inclusion of webpack we should rename the devstack. Gulp is just one of the used technologies so I think it shouldn’t be emphasized in the name anymore.
If you agree, add you suggestions to comments (one at a time) and we can vote for them using 👍 emojis.

frontend-devstack

fe-devstack

I think we can safely change the name, it has been two weeks since this request was posted.
Probably, this will require to update the remote point for all repository contributors, right?

@kettanaito Yep, I guess so (remote).

I have a lot of documents with the link to this repo. I'm afraid that this links will be broken, when we change then name of repo.

@DavidSlavik  @vbulant  @kettanaito  Not sure,
We renamed the ui-kit repo from https://github.com/actum/frontend-trends. if you visit the last link, it will redirect you to the actual repo.


Add a class to help with limiting rates of events

This adds a simple utility class that helps limit the rate of events. This can be used to detect if a pathological cycle of conflict is happening.

We can go ahead and merge this any time since it doesn't create any conflicts.


Fix build

Make some fixes to the build so that the CIs are not disabled and the recipe is somewhat modernized for conda-build 3.

Checklist

[ ] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe:

Using toolchain directly in this manner is deprecated.  Consider using the compilers outlined here.

@conda-forge-admin, please re-render.

Hi! This is the friendly automated conda-forge-webservice.
I tried to re-render for you, but it looks like there was nothing to do.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe:

Using toolchain directly in this manner is deprecated.  Consider using the compilers outlined here.

@conda-forge-admin, please re-render.


About network performance

I'm struggling to meet the reasonable performance of the text classifier with a self-implemented network. (Both FC and RNN)
If I've done things right, I'm having an accuracy of ~50%, which is almost the same as the random guess.
I set the hidden state dimension as 128.
Are there any other things that I can try to increase the accuracy?

Hi, please use Q&As for questions. I am closing this issue.


TypeError: Transformer is not a function

I'm running rc1 of preact-cli and whenever I create a preact.config.js file (empty or not) I get TypeError: transformer is not a function
Do you want to request a feature or report a bug?
Bug
What is the current behaviour?
When creating a preact.config.js file the build fails with TypeError: transformer is not a function
I think I have narrowed the issue down to this line
When the file exists and contains this:
export default {
    webpack() {}
};

The const transformer contains the config object itself: { webpack: [Function: webpack] }
Shouldn't it have the value of the function itself?
Plus, if I export an empty object export default {} transformer contains the empty object and not undefined

Node version
v12.1.0
npm version
6.9.0
Operating system
arch
CLI version
rc1

can you post a sample config of yours?

I did. The error occurs with an empty file (no default exports), an empty export (‘export default {}’) and even with the sample config from the Readme

On 8. May 2019, at 19:17, Prateek Bhatnagar<EMAIL_ADDRESS>wrote:
can you post a sample config of yours?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

If a file exists, it has to export a function

Makes sense. But even with an empty object or and object with a function on the webpack key the error occurs

On 8. May 2019, at 19:45, Luke Edwards<EMAIL_ADDRESS>wrote:
If a file exists, it has to export a function
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

it works with your config, are you sure you're on rc?

I have the same question, i only create a project from cli tool, then add preact.config.js

@ForsakenHarmony
Fresh install of preact-cli@rc,
preact create default test,
cat > preact.config.js << EOF
export default {};
EOF

npm start: TypeError: transformer is not a function
So yes, I'm on RC and this is my exact setup in which the error occurs, same as @hnlinzhi19 described

oh you have to upgrade preact cli in the project as well, it'S still the old version

Thanks for that! 👍
Should've caught it myself! 🙈

No worries, happens to all of us sometimes 🙃


Lerna hoist package lock

Testing install / build time with hoist and package.lock in root package

Build time on this seems to be pretty consistently around 29 minutes


CameraBuf::acquire : release on failure

I only see two instances of the cloudlog over the last two weeks, so this seems to be hit only very rarely.

I only see two instances of the cloudlog over the last two weeks, so this seems to be hit only very rarely.


App crashes in iOS at launch

I downloaded and compiled the latest build using Xcode 6.1 and loaded it onto an iPhone 5s and iPad mini to test. Both instances crash upon load.
After reviewing the error reports I found this to be the crash issue

Dyld Error Message:
Library not loaded: @rpath/libswiftCore.dylib
Referenced from: /private/var/mobile/Containers/Bundle/Application/0BD865E2-AD0C-4277-B3E7-5E5FB863EBCA/HelloCordova.app/HelloCordova
Reason: image not found
Dyld Version: 353.6

After some searching I came across this link https://www.airsignapp.com/ios-apps-using-swift-crash-when-signed-with-inhouse-certificate/
Which outlines this as a certificate issue and asks a user to regenerate certificates and a new code signing identity. I did this and the issue persists. Any one have any ideas?
Is there a non swift version of the code so I can try in an older version of Xcode?

same errors


As a user, I want to have information on how to connect with the running node

implemented in https://github.com/PhilippLgh/ethereum-gui


Add skip_if_exists to add a file only if it doesn't exist

TL;DR
This has come up a couple of times now. As a template developer, I want the ability to place a file in the destination, iff there is not already a destination file of that name.
Use cases:

Add terraform.tf files if they don't already exist

Some ways I can think of that this could work:
Option 1: an option on the "include" action
steps:
  - action: 'include'
    params:
      paths: ['mydir/my_file.txt']
      skip_if_exists: true

We'd prevent skip_if_exists: true from being used together with from: destination, because that would be silly: including a file from the destination only works if the file exists, and skip_if_exists:true only has an effect if the file doesn't exist.
How we'd implement it: when a file is included with skip_if_exists, we'd copy it to the scratch directory as usual, and remember to handle that filename specially later at the time we commit the scratch dir contents to the destination dir.
Upside: simple devex for template authors
Downside: it's a bit architecturally hacky because it mixes commit-time logic (skip_if_exists) with include-time logic (including a file into the scratch dir). If we ever offer the ability in the future to relocate files in the scratch directory, we'll have to track the filename changes so we know the final location.
Option 2: add commit options
steps:
  - action: 'include'
    params:
      paths: ['mydir/my_file.txt']
commit_options:
  skip_if_exists: ['mydir/my_file.txt']

Upside: more architecturally clean
Downside: requires repeating the filename. More confusing devex if the template author isn't thinking carefully about the scratch directory and how committing-to-destination works.
cc @verbanicm
Detailed design
No response
Alternatives considered
No response
Additional information
No response

I prefer Option 1 over Option 2, because in a large YAML file, the skipping settings are extremely distanced from the actions. That makes it hard to contextualize how a step will behave.
I do wonder if we want something slightly more flexible, like:
steps:
  - action: 'include'
    params:
      paths:
        - path: 'mydir/my_file.txt'
          on_conflict: 'skip' (or 'overwrite' (default) or 'error')

I think that gives us maximum flexibility.

Re Seth, that sounds good to me. Changed the example to use on_conflict.

xref with https://github.com/abcxyz/abc/issues/130


Extending Image block in 0.6

We are using v0.6, and would like to create/extend a custom image block to do things like float left/right, add a caption and selecting custom sizes. I know there are examples in the custom blocks repo, but these don't work with version 0.6.
Do you have any examples that work in 0.6, or pointers where to start?

I can do an update tomorrow at some point for you.
On Thu, Apr 14, 2016 at 4:14 PM, Daryl<EMAIL_ADDRESS>wrote:

We are using v0.6, and would like to create/extend a custom image block to
do things like float left/right, add a caption and selecting custom sizes.
I know there are examples in the custom blocks repo, but these don't work
with version 0.6.
Do you have any examples that work in 0.6, or pointers where to start?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/madebymany/sir-trevor-js/issues/457

Awesome, thanks a lot.

In the latest version you'll find we've removed jQuery and as a result the majority of existing blocks will require a little work to make them compatible with the latest version. Docs are coming soon, but here's an example.
Any problems let me know. This is using es6 templates and object.assign, so assuming you are using babel, but could be modified to not be using these functions.
import SirTrevor from 'sir-trevor';

module.exports = (function() {

  const templateHTML = (data) => `

Caption

          <input type="text" name="caption" class="js-caption" value="${data.caption}" />

  `;

  return SirTrevor.Blocks.Image.extend({

    loadData: function(data) {
      const defaultData = {
        file: {
          url: ""
        },
        caption: ""
      };

      data = Object.assign({}, defaultData, data);

      this.inputs.style.display = 'none';
      this.editor.innerHTML = templateHTML(data);
      this.editor.style.display = '';
    },

    onDrop: function(transferData) {
      var file = transferData.files[0],
          urlAPI = (typeof URL !== "undefined") ? URL : (typeof webkitURL !== "undefined") ? webkitURL : null;

      // Handle one upload at a time
      if (/image/.test(file.type)) {
        this.loading();

        this.loadData({
          file: {
            url: urlAPI.createObjectURL(file)
          }
        });

        this.uploader(
          file,
          function(data) {
            this.setData(data);
            this.ready();
          },
          function(error) {
            this.addMessage(i18n.t('blocks:image:upload_error'));
            this.ready();
          }
        );
      }
    }
  })
})();

You'll find because the caption has a name attribute this will be picked up by the data serialisation. However if you want to include a checkbox or selectbox then you'll need to do your own data serialisation for the block using a function such as:
_serializeData: function() {
  return {
    caption: this.el.querySelector('.js-caption').value
  };
},

Ahh that explains why I didn't notice my own version falling over. (I use jQuery elsewhere for other functions so it's not triggered any weirdness)
That said I've only run a test version of 0.6, I haven't moved off 0.5 for some of the bigger projects where I've been using it.

@SgtOddball i'm going to start updating the docs today.
Part of that will be creating and customising a block example.
The main change was to remove the $el as the convention of naming didn't make sense when we aren't using jQuery anymore.
I did however leave in the convenience method:
$: function(selector) {
  return this.el.querySelectorAll(selector);
}

I wrote some migration notes a while back:
https://github.com/madebymany/sir-trevor-js/blob/master/docs/migrations/0.5-0.6.md

Ooooh documentation!  But if the docs are up in the near future I'll look over this again and see what i need to do to fully migrate everything.

Will it be possible to add other blocks as image description? I would like to add text and list blocks.

Sorry don't understand your request.
You mean you want to add text and lists to a description field in an image
block?
On Fri, 6 May 2016 at 17:51, Mario Schubert<EMAIL_ADDRESS>wrote:

Will it be possible to add other blocks as image description? I would like
to add text and list blocks.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/madebymany/sir-trevor-js/issues/457#issuecomment-217496169

Please see the screenshot :-)

Ok. So you want blocks within blocks.
This isn't something possible with the core project, but I think others
have done something similar.
You might find you can get what you want by using the scribe list plugin
too. This will allow you to use the text block and have lists within it.
Search issues or closed pull requests for columns or inner blocks and that
might give you some answers.
On Fri, 6 May 2016 at 19:08, Mario Schubert<EMAIL_ADDRESS>wrote:

Please see the screenshot :-)
[image: ste_inside_image_block]
https://cloud.githubusercontent.com/assets/1107183/15082098/33d8df8e-13c6-11e6-8ddd-6d3e648fb895.png
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/madebymany/sir-trevor-js/issues/457#issuecomment-217517472

Yes, thank you.
It would be really great, if nesting would become a core functionality. At least as an optional feature.

@marioschubert we won't be supporting this functionality.
Someone that's done some work that might help you is the following.
https://github.com/Upplication/sir-trevor-blocks
Take a look at the columns feature for some inspiration


Unable to kots upstream upgrade --deploy-version-label

The admin console is showing me only the currently deployed version and the latest, but I am looking for an intermediate version.
I've tried running upstream upgrade, but I'm getting an error:
kubectl kots -n $ns upstream upgrade $appSlug --deploy-version-label $appVersion
  • Checking for application updates ✗
  • Unexpected response from the API: 500

It only seems to work with the versions available in the admin console (the currently deployed one and the latest).
Did I misread the docs and the command above doesn't fetch application versions from the Replicated vendor? If so, is there any way of getting a specific version?
Thanks.

Hi. Thank you for your reply.
After more troubleshooting, I realized that kots upstream upgrade (and its UI equivalent) are working as expected, but the Vendor's license was unpinned and so it would only pull the latest.
Sorry for the confusion and thanks again!

You're welcome. Glad things are working as expected!
/close


Add support for Hook(s) folder

Fixes #2468
Changes proposed:

[x] Add
[ ] Delete
[ ] Fix
[ ] Prepare

I added a Hook(s) folder icon to support for an idiosyncratic feature of the React^16.8 library. There could be other languages outside my sphere of knowledge where this could still be applicable, so I opted not to use the React logo in the icon file.

There's a few cleanup items I'm taking care of right now. I'll update this comment when it's finished.


Fetch only important time series with latest data

The time series that the fetch module gets should be listed in a JSON document, not as embedded Python code.
Alternatively this could also be stuck into a database.

We now have 1600+ series in the database with which we fetch. This is a lot of information and doing a complete fetch takes around 1700 seconds (which is actually pretty good if you think about it). As we update on a month to month basis, we should only update certain series and only get data for the last year.


Treat dash as dot in rpm assemble

Signed-off-by: Peter Zhu<EMAIL_ADDRESS>Description
Treat dash as dot in rpm assemble
Issues Resolved
https://github.com/opensearch-project/opensearch-build/issues/1549#issuecomment-1098429590
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #1966 (2e5fedd) into main (6264846) will increase coverage by 7.23%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##               main     #1966      +/-   ##
=============================================
+ Coverage     92.76%   100.00%   +7.23%
=============================================
  Files            45         6      -39
  Lines           235       105     -130
  Branches         29        19      -10
=============================================
- Hits            218       105     -113
+ Misses           11         0      -11
+ Partials          6         0       -6

Impacted Files
Coverage Δ

tests/jenkins/jobs/BuildFailureMessage_Jenkinsfile

tests/jenkins/jobs/AssembleUpload_Jenkinsfile

tests/jenkins/jobs/GetManifestSHA_Jenkinsfile

...nIntegTestScript_OpenSearch_Dashboards_Jenkinsfile

tests/jenkins/jobs/UploadTestResults_Jenkinsfile

tests/jenkins/jobs/BuildManifest_Jenkinsfile

...PromoteArtifacts_OpenSearch_Dashboards_Jenkinsfile

tests/jenkins/jobs/CreateReleaseTag_Jenkinsfile

tests/jenkins/jobs/PromoteArtifacts_Jenkinsfile

...ts/jenkins/jobs/uploadMinSnapshotsToS3_Jenkinsfile

... and 29 more

Continue to review full report at Codecov.

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


Convert AHB related errors into warnings

CI gfxreconstruct build queued with queue ID 207072.

CI gfxreconstruct build # 4314 running.

CI gfxreconstruct build # 4314 passed.

CI gfxreconstruct build queued with queue ID 207132.

CI gfxreconstruct build # 4316 running.

CI gfxreconstruct build # 4316 passed.


[MTS100H] - Feature request

Hi,
Thanks for your work.
Do you think you will add support to MTS100H Smart Thermostat Valve?

Hi @KiraPC
I can only add support for devices once someone owns one and either confirms it already works or work together to implement it.
Since the plugin already supports the MTS100V3, this device may already work

Hi, thanks for reply.
I think I'm doing something wrong because I cannot see the list of my devices.
Il 18 dic 2022, 15:44 +0100, Ben @.***>, ha scritto:

Hi @KiraPC
I can only add support for devices once someone owns one and either confirms it already works or work together to implement it.
Since the plugin already supports the MTS100V3, this device may already work
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

is it added to the same meross account as you have configured in the plugin settings?

Sure.
I cannot see any error in the startup logs
Il 18 dic 2022, 17:22 +0100, Ben @.***>, ha scritto:

is it added to the same meross account as you have configured in the plugin settings?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Can you show all your plugin startup logs here? feel free to obfuscate any device ids or names etc

HI,
These are my logs filtered by meross plugin
[12/18/2022, 9:33:48 PM] Loaded plugin<EMAIL_ADDRESS>[12/18/2022, 9:33:48 PM] Registering platform 'homebridge-meross.Meross'
[12/18/2022, 9:33:48 PM] [Meross] Initializing Meross platform...
[12/18/2022, 9:33:48 PM] [Meross] Initialising plugin v8.4.0 | System darwin | Node v16.15.0 | HB v1.6.0 | HAPNodeJS v0.11.0...
[12/18/2022, 9:33:48 PM] [Meross] Plugin initialised. Setting up accessories....
[12/18/2022, 9:33:51 PM] [Meross] ✓ Setup complete. Have a feature request? Visit https://bit.ly/hb-meross-issues to ask!
[12/18/2022, 9:35:51 PM] [Homebridge UI] [homebridge-meross] Terminating child process...
[12/18/2022, 9:35:51 PM] [Homebridge UI] [homebridge-meross] Child process ended

So there are no devices in your meross account that you configured in the plugin?

I've enabled the debug mod and now I have a new log:
[Smart Hub] not initialising cloud device as matched as HomeKit native model.
I think I figured out.
I have 2 new model of the SmartValve that are ready for homekit and an old model that it is not, but the HB plugin is fetching just the Hub (that is the new version) and it is skipping all the devices connected.

for now can you disable the “ignore native homekit” option?

Yes I did, and now they are displayed

and working?

Yes, they are correctly working.
Thank you so much for support and sorry for disturb :D

If you update to the latest version of the plugin you should be able to re-enable the "ignore native homekit" option with these MTS devices still appearing in your homebridge

@bwp91 I updated to latest version, I re-enabled the flag, now all the devices appear, also the 2 homekit native

what are the models of the homekit native devices?

The model shown by the plugin is the same for all the devices "MSH300HK" but it is the model number of the Hub the devices are connected.

Can you tell from the Meross app whether the existing HomeKit ones are MTS100V3 or MTS150? also in the meross app does it provide a hardware version for the valves rather than for the main hub?

Yes
Basically, MTS100V3 is an hub compatible with HomeKit if the smartValve connected are MTS150, but if I connect the oldest valve MTS100 that are not compatible with homekit.


Does this support ssr?

SSR is a software which implement shadowsocks and other protocols as far as I know. Shadowproxy should work with SSR using shadowsocks protocol. Shadowproxy also support http and tls1.2 obfs plugins like SSR does.


Correct acronym for ACI

ACI (in this context) refers to the Azure Container Instances service, not
the Cisco SDN solution
Fixes: #46223
Signed-off-by: Graham Hayes<EMAIL_ADDRESS>

@grahamhayes Thanks for submitting this fix. The update appears to be in master, so we'll close the pull request.


Break the dragging animation if the slide index is equal to 0 or {max}

I need this feature in the project I'm currently working on and I'm a little bit surprised that this super carousel does not have this feature out of the box.
Please check the README.md file for more info and a preview.

Hi @AdiMarianMutu 👋
Thanks for your work on this. One of Embla's primary goals is to be lightweight and extensible and it looks like your feature request is something that may already be possible with the existing API.
What if you listened to the select event and then called reInit() with draggable: false after reaching the last slide? Does that sound like it would work for your project's needs?
Hope that helps!

Hi @AdiMarianMutu 👋
Thanks for your work on this. One of Embla's primary goals is to be lightweight and extensible and it looks like your feature request is something that may already be possible with the existing API.
What if you listened to the select event and then called reInit() with draggable: false after reaching the last slide? Does that sound like it would work for your project's needs?
Hope that helps!

Hi @nikrowell.
Thanks for the feedback. I have already tried to do it with the API, but because of the transform: translate3d injected style, there is always a small gap before breaking the dragging.
That's why the condition must exists before injecting the style.
Please check the preview inside the README.md

Hello @AdiMarianMutu,
Thank you for your contribution. This is related to issue #42. You can use the undocumented feature embla.dangerouslyGetEngine() to achieve what you want. This method exposes the internal functionality of Embla and can be used to extend Embla to cover most of the use cases for carousels.
Something along these lines should work:
const preventEdgeScrolling = (embla) => {
  const { limit, target, location, scrollTo } = embla.dangerouslyGetEngine();

  return () => {
    if (limit.reachedMax(target.get())) {
      if (limit.reachedMax(location.get())) location.set(limit.max);
      target.set(limit.max);
      scrollTo.distance(0, false);
    }
    if (limit.reachedMin(target.get())) {
      if (limit.reachedMin(location.get())) location.set(limit.min);
      target.set(limit.min);
      scrollTo.distance(0, false);
    }
  };
};

embla.on("scroll", preventEdgeScrolling(embla));

I hope this helps.
Best,
David

Hi @davidcetinkaya,
Thanks for the solution, it's working perfectly!
I'm really sorry about this "abusive" pull request, I've completely missed this guide line.
However, I'm unable to find the documentation about dangerouslyGetEngine() in the official methods doc, this would have been really useful if it was present.

Hi @AdiMarianMutu,
Not a problem. As I mentioned in my comment dangerouslyGetEngine() not being documented is intentional. When I created the method, it was intended to aid plugin authors with maximum extensibility. Another purpose was that it enabled for extending the carousel with features that either wouldn't be added to the core itself, or features that would be released later on. But I didn't want people creating a lot of issues about it because you really have to know what you're doing when using it, and that's why I kept it undocumented.
Cheers,
David

Hi @AdiMarianMutu,
Not a problem. As I mentioned in my comment dangerouslyGetEngine() not being documented is intentional. When I created the method, it was intended to aid plugin authors with maximum extensibility. Another purpose was that it enabled for extending the carousel with features that either wouldn't be added to the core itself, or features that would be released later on. But I didn't want people creating a lot of issues about it because you really have to know what you're doing when using it, and that's why I kept it undocumented.
Cheers,
David


Problems in macOS Sierra

People, help me please. Today I updated my MacBook Pro to the new version of Mac. I got this error on started my application (passenger start, rails s, puma, ...):
[ 2016-09-23 14:54:29.3122 36954/0x70000e363000 age/Cor/App/Implementation.cpp:304 ]: Could not spawn process for application /Users/klein/Code/rails/artia-app: An error occurred while starting up the preloader.
  Error ID: 3e3de1d0
  Error details saved to: /var/folders/h0/vhs_g11j66z3b9t6bsg_cj_m0000gn/T//passenger-error-4Fw7Ey.html
  Message from application: There was an error while trying to load the gem 'omniauth'.
Gem Load Error is: uninitialized constant OmniAuth::Configuration::Form

Please check this Gist with the backtrace: https://gist.github.com/thiagok/5a2b0083057f74ba8cac471ed73fea06

Going to close this out but, if this is still an issue, please feel free to reopen and ping me.


Add GREEK docs folder and README.md

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Hi,
Is there anything more I need to be doing in re to the above (Travis CI build check) so the pull request is accepted and my branch merged with the original?
Hope I'm not doing something wrong here.
Thanks in advance.

@stefanoszisidis Thanks for your contribution. This is a great start! Do you have time to go back in and finish translating this?
Also, congratulations on your first merged pull request to this project!


Play way is not correct

In HikariCPConfig the maxPoolSize is incorrect. Should it not be the following in line 82:
val maxConnectionsPerPartition = config.getInt("maxPoolSize").get

The same goes for the minPoolSize in line 88.

Master branch already has dropped the translation from bonecp to hikaricp.


refactor: Remove use of Compress function in sample profile due to issues

Compress function will eventual panic the service.
Signed-off-by: Leonard Goodell<EMAIL_ADDRESS>

If your build fails  due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/app-service-configurable/blob/main/.github/CONTRIBUTING.md
PR Checklist
Please check if your PR fulfills the following requirements:

[x] I am not introducing a breaking change (if you are, flag in conventional commit message with BREAKING CHANGE: describing the break)
[x ] I am not introducing a new dependency (add notes below if you are)
[ ] I have added unit tests for the new feature or bug fix (if not, why?) N/A
[x] I have fully tested (add details below) this the new feature or bug fix (if not, why?)
[ ] I have opened a PR for the related docs change (if not, why?)

Testing Instructions
Run service with sample profile
Verify after 10 mins it does not panic.
New Dependency Instructions (If applicable)

@bnevis-i this will unblock using app-sample in the helm. We randomly selected the Compress function for one of the function pipelines.


Warning about overalapping time routine

When I create a first routine I can set every time I want. But with the creation of a second routine I always get notified that the time is overlapping with an other routine. Even if this is not the case.
I use version 3.0.0-pre-beta-06 (59)

Hi! Could you tell the times that you were setting for the first and the second routine?

Sure. I have one routine going from sunset to 23:59. The second routine I would like to add runs from 00:00 to sunrise. I can add the starting time, but when I enter the end time it states that the times are overlapping.

Very nice and important catch, even I was able to reproduce the same. This has been fixed in https://github.com/corphish/NightLight/commit/970580e457c1de4114679069e7532fbf4b6d7f01 ... https://github.com/corphish/NightLight/commit/4e1e8976c96138fbaa0ffdb2517136c6cb647470, and should now work as expected. The fixes will be available in the next beta.
Thanks!


Onion option

I disabled the config option for the onion not to produce sprouts, and the Pikmin left the bodies under the beam. Could the onion still take the bodies and still convert them into Pikmin without creating sprouts and adding them to the count, or is that option meant for something else?

I forgot to make it so Pikmin take things back to the ship instead of the onion when the config is turned off. I'll fix this in the next version

0.2.13


Added jspm/jspm config settings

This allows users of jspm to quickly use nano scroller.
I've also added it to the jspm registry here - jspm/registry#235
This means users only need to do: jspm.import(nanoscroller); to get everything (including the css) loaded.
Let me know if that all makes sense!

Thanks!


Documentation on how get full IP list #72

Alternative solution to #72.
Documented two ways to get IP address list without changing resources definition

@domiborges Can you review the changes when you get a chance? Thanks

Hi! I'll take a look today. I assume this works for k8s and also OpenShift?

Added commit with docs review. I created a dedicated module for this procedure. Thanks @rigazilla for the docs!

Thanks @rigazilla @domiborges


Add examples to aggregate/window functions

Also disable font ligatures for code blocks so != does not render as ≠

Thanks Mark!


Add a new collection that keeps track of which transformations modified a target

MODSEC-63: The TFN collection would be populated with target-key=modified-transformation-list
{noformat}
SecRule ARGS "bar" "t:none,t:trim,t:lowercase,t:replaceComments...
{noformat}
{noformat}
GET /?foo=+Foo/+bar+/&bar=Bar
{noformat}
Would yield:
{noformat}
TFN:ARGS:foo = t:trim,t:lowercase,t:replaceComments
TFN:ARGS:bar = t:lowercase
{noformat}
And then you could do an anomaly adjustment based off this data:
{noformat}
SecRule TFN "@contains t:replaceComments" "pass,nolog,setvar:tx.anomaly=+5"
{noformat}

The TFN collection will be always populated within the content specified in the RULE itself and in the default action. With that, it will be possible to know those by the time that the rule is written. I am not sure if there is the need for such variable.


Displaced table headers while not using responsive_table

Bug report
What I did:
'responsive_table'        => false,
What I expected to happen:
I can resize the window and table headers are moving correctly
What happened:
Table headers are displaced until reload
Backpack, Laravel, PHP, DB version:
Latest Backpack, latest Safari/Chrome

Hmm... @tswonke what do you mean by displaced? Can you provide a screenshot please?

Sure:
When I open the Crud it looks like this:

After resizing the window horizontally it looks like this:

Oh, you're right. Thank you @tswonke - I can confirm that happens.
Apparently when using AJAX Datatables there's not one table, there's multiple:

one for the headings
one for the content
one for the bottom headings

So when the window gets resized, HTML resizes the tables according to content, which is always different from content to headings. Only solution I see is to recalculate after window resizing has stopped.
Just pushed a PR for it here - https://github.com/Laravel-Backpack/CRUD/pull/1651 . Let me know if that fixes it for you too, with no side-effects. I'll merge it into master.
Cheers!

Hi @tabacitu
thanks, that seems to work! 👍

Just realized: this still happens when using the button to hide the sidebar ;-)

Great! Thank you @tswonke ! Also fixed it on sidebar change - let’s move the conversation to PR #1651 if anything else shows up.
Cheers!


9 filter sort pagination

Description of changes
Add sorting, filtering and pagination for the course service
How has this been tested?
Please describe the test strategy you followed.

[x] automated unit test
[x] automated integration test
[ ] automated acceptance test
[ ] manual, exploratory test

In case of manual test, please document the test well including a set of user instructions and prerequisites. Each including an action, it's result, and where appropriate a screenshot.
Checklist before requesting a review

[x] My code follows the coding guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My code fulfilles all acceptance criteria
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I have made corresponding changes to the documentation
[ ] I have added explanation of architectural decision and rationales to wiki/adr
[ ] I have updated the changes in the ticket description

Checklist for reviewer

[ ] The code works and does not throw errors
[ ] The code is easy to understand and there are no confusing parts
[ ] The code follows the coding guidelines of this project
[ ] The code change accomplishes what it is supposed to do
[ ] I cannot think of any use case in which the code does not behave as intended
[ ] The added and existing tests reasonably cover the code change
[ ] I cannot think of any test cases, input or edge cases that should be tested in addition
[ ] Description of the change is included in the documentation

Please squash on merge


Set up Continuous Integration Builds

With Travis? Or is there alternatives that I should look into?

fixed in #22


Do not use JSON for router configuration

Fixes #63
This is WIP.  There are more commits to come as part of this PR.

@krancour code LGTM, I haven't tested in a cluster though. Let me know if you'd like me to

@arschles fyi... I am still working on exporting error types as you've suggested.  Doing so actually is allowing me to make a number of optimizations to the tests as well.  Stay tuned.

sounds good @krancour. let me know if you'd like more reviews after your changes!

@arschles error types are exported and docstring'ed now.

@krancour awesome, thanks. made a few more comments on the new code


E-Mail only Issues

Feature Description
I'd like to  suggest that it be made possible to create issues on Gitea Repositories without a user account on that instance or on Gitea at all.
This would make it easier for non-developers to post issues to Gitea and would mean that project maintainers do not have to operate a separate issue tracker for non-developers.
Implementation
This feature should be an option for the repository maintainers and could allow anyone to simply enter their E-Mail and an issue text - without signing up to an account.
That E-Mail could then be used to contact the issue creator later on.
Screenshots
No response

That E-Mail could then be used to contact the issue creator later on.

This is, by definition, an account. A very slim one, but it is one. If an e-mail is saved somewhere to identify a user uniquely, then you have an account, even if you cannot "log in" or something.

Not necessarily I think. In YouTrack you have sort-of helpdesk feature, basically incoming email creates an issue with "helpdesk user" and add all relevant emails (from/cc) to "emails-addresses-list" (save for users that have account on the system) and each time there is a new comment then notifications is generated to all those associated email addresses. And if there is a replay the it's added as a comment (This is already done in #22056 I guess).
It helps with communication with people that use mainly email, and use long email threads (with people in CC)...


scopes.rst Grammar changes and link to openID site

Hi @grimley517, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, DNFBOT;

thanks!


7.3 compatibility

For example in file PaymentCreateResponse.php there is this error at line 13:
syntax error, unexpected 'int' (T_STRING), expecting function (T_FUNCTION) or const (T_CONST)
It is about properties declarations "protected int $code;" - type is not allowed in this version of PHP.
I suppose this error is also in other classes.

Yes, I have same problem. My php version is 7.3.26. Same error
PHP Parse error:  syntax error, unexpected 'int' (T_STRING), expecting function (T_FUNCTION) or const (T_CONST) in /comgate/sdk/src/Entity/Response/PaymentCreateResponse.php on line 13


⚠ ClamAV has degraded performance

In 36411bb, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 556 ms

Resolved: ClamAV performance has improved in 0adc494.


IDEA shows incorrect number of buildSrc dependencies

Expected Behavior
IDEA's Gradle view should show correct number of buildSrc default dependencies.
Current Behavior
Shows every default dependency twice (or even thrice in test Source Set).
Context
The issue is cosmetic and doesn't really bother me.
Also custom dependencies declared in buildSrc/build.gradle(.kts) are not affected.
Steps to Reproduce (for bugs)
Create a Gradle project with buildSrc dir.
Create a buildSrc/build.gradle(.kts) file, it may be empty.
Open Gradle view in IDEA.
Your Environment
Ubuntu 18.10, OpenJDK 8, Gradle 5.2.1, IntelliJ IDEA 2018.3.4 (snap).

@wolfs @blindpirate is this something you guys have noticed before?

I can confirm this issue with my IDEA 2018.3.1 + Gradle 5.0. Will investigate it.

This is an IDEA issue. Opened issue there: https://youtrack.jetbrains.com/issue/IDEA-207207
It was also reported by other people: https://youtrack.jetbrains.com/issue/IDEA-181666


Tests and fixes

Wrote in some new tests and fixed a misunderstanding I had that lead to some issues
Fixes abacus-network/abacus-monorepo#1026

btw I think this may auto-close https://github.com/abacus-network/abacus-monorepo/issues/1026 so just flagging that we should make sure to do the other bits mentioned in there too


Compile error in enums example

The enums example fails to compile with the following error: "Unexpected enum".

Thanks for reporting :)

Wow, that was fixed quickly! Thanks for maintaining try.haxe 🍰


Change placeholder in ESP_LOGD conditionally depending on FF_FS_EXFAT (IDFGH-7832)

In case of using EXFAT by setting in ffconf.h:
#define FF_FS_EXFAT    1
the type FSIZE_t is changing from 4 to 8 bytes.
As a result,  ESP_LOGD() in vfs_fat_lseek() does not compile:
error: format '%d' expects argument of type 'int', but argument 8 has type 'FSIZE_t' {aka 'long long unsigned int'} [-Werror=format=]
     ESP_LOGD(TAG, "%s: offset=%ld, filesize:=%d", __func__, new_pos, f_size(file));

To solve the problem we need to change %d with %lld conditionally, depending on FF_FS_EXFAT.

sha=01e5d0a59d919aacb8684e0d08eb8359a3f4d2e5


"The portal" is vague

Need a link to which portal page this is referring to, as there have been recent changes and it doesn't appear to exist anymore. Other articles discussing these changes are also using the old setup of Endpoint which is now Intune.
The only way I found to add tags was going to Intune, select Tenent Admin, Roles, then Scope to add it.
[Enter feedback here]

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

ID: 51c29286-b601-4dad-0e65-227c8d045ee4
Version Independent ID: 2af4acf9-4089-5dff-d628-16b8425e0a57
Content: Create and manage device tags
Content Source: microsoft-365/security/defender-endpoint/machine-tags.md
Service: microsoft-365-security
Sub-service: mde
GitHub Login: @mjcaparas
Microsoft Alias: macapara

@mjcaparas Please help us in resolving this issue. Thanks

The portal has been unified in Microsoft 365 Defender.


Baseline on c3 multiseries graph

Hello,
I'm using c3 to draw some multiseries graphs but i'm having some problems. I wanna draw a graph with baseline but c3 always adjust x axis to adapt to the all range of data. I'll give an example:
var chart = c3.generate({
    data: {
        xs: {
            'data1': 'x1',
            'data2': 'x2',
        },
        columns: [
            ['x1', '2013-01-01', '2013-01-02', '2013-01-03', '2013-01-04', '2013-01-05', '2013-01-06'],
            ['x2', '2012-01-01', '2012-01-02', '2012-01-03', '2012-01-04', '2012-01-05', '2012-01-06'],
            ['data1', 30, 200, 100, 400, 150, 250],
            ['data2', 130, 340, 200, 500, 250, 350]
        ]
    },
    axis: {
        x: {
            type: 'timeseries',
            tick: {
                format: '%Y-%m-%d'
            }
        }
    }
});

I want that above graphic draw 2 lines but the x axis only show the values of x1, but when i click in a point of data2 line i want that the legend show the values of x2. Is that possible?

Hi @misticini! This is more of a support question — would you mind please asking instead on the C3 Google Group or Gitter channel? Thanks!


SMP in ZFP

When rp2040_hal is used with a ZFP runtime, there is no support for using the second CPU core. We should add a wrapper around the SEV instruction and ensure that the startup code will work if executed by both cores. We should still push users toward Ravenscar's tasks and protected types for SMP applications and the documentation should point out the caveats of accessing hardware peripherals from two cores at the same time.

RP.Multicore is available in release >=1.4.0


New App: Git Commit Humor

Description:
Displays humorous Github Commits in a scrolling fashion using the whatthecommit (https://whatthecommit.com/index.txt) API. The app will rotate through fonts and color based on commit text length.

I have read the CLA Document and I hereby sign the CLA


Must set other color scheme before I can set mellifluous as my color scheme

require('impatient')
require('rathod-sahaab')

require 'mellifluous'.setup({
  -- default conf
})

-- setting this is required before mellifluous or else nothing works
vim.cmd("colorscheme gruvbox")
vim.cmd("colorscheme mellifluous")

Even manually calling :colorscheme mellifluous  does not work unless I set different color scheme first

Thanks for making this issue!
I think this happens due to vim.opt.termguicolors not being set (most people set this in their configs). This makes neovim think that your terminal doesn't support 24-bit true colors and this support is required for mellifluous.
It should be fixed now, but in the future I would recommend having vim.opt.termguicolors = true line in your config, otherwise a lot of colorschemes will show different colors.
Let me know if its working for you.

Hi, @ramojus that was exactly what the problem was I might have purged that line while changing themes, Thanks for the awesome theme!


Jupyter notebook dark mode matplotlib message text not visible

When changing to dark mode the message for the cursor location in the lower-right corner is not visible. By changing the color of the element in the CSS code, it is visible. Is there a way to automate this?
The span that needs to have this applied is: .mpl-message
@tacaswell Also saw this issue

There is, using JupyterLab CSS variables. I thought this was added already.

We don't see it on a newer version of ipympl (I forget the system we were working on is pinned on a very old version for .... reasons...).

We don't see it on a newer version of ipympl

don't see the text, or don't see the issue? i.e., are you saying this is solved in recent versions, and the issue can be closed?

Sorry, I mean we don't see the problem (that is we can see the text) with newer versions of ipympl and jlab.
This is a duplicate of https://github.com/matplotlib/jupyter-matplotlib/issues/77 which is fixed on master.
Sorry for the noise on our side (I was sitting next to @andrewmkiss and asked him to open this issue).


reorganize readme

I took a crack at tidying the readme, mainly by moving technical content to separate files, and providing a link to the forum post.
With the existing readme, I think the table of query parameters is somewhat distracting - so that was my primary motivation for separating things out.
I also reordered the author list since you two are taking the lead at this point. :)

Made a couple suggestions, but it looks like we merged already :)
Should I make a separate PR?


tiatracker: init at 1.3

Motivation for this change
#81815 cc @OPNA2608
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[x] (Package updates) Added a release notes entry if the change is major or breaking
[x] (Module updates) Added a release notes entry if the change is significant
[x] (Module addition) Added a release notes entry if adding a new NixOS module
[x] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Fixed all the Darwin stuff in https://github.com/fgaz/nixpkgs/pull/3, builds & works on my machine now.

Do you think we should package that instead, or do both?

I would be in favour of doing both just cuz "why not?". But I don't have much experience with using trackers or the TIA so I can't judge how crucial these changes are (except undo/redo, which is obviously an improvement).

I would be in favour of doing both just cuz "why not?"

That's pretty much in the spirit of nixpkgs, and I agree :D
I'll add it to the list

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/471


Added arrow, cla, margins, contour

Hi, I'm not sure these fit your formatting but they've been useful additions for me. Hope they are helpful for you!

Sorry for the long delay, thanks for contributing!

@JBPennington can you maybe provide an example how to use the plt::contour() function? I need to plot a compressor map and therefore a contour map would be perfect. Im able to plot the compressor map in 3D with plot_surface but if I use the same data structure(vector of vectors) to plot with the contour function I get the following error:
../include/matplotlibcpp.h: In instantiation of ‘bool matplotlibcpp::contour(const std::vector<Numeric>&, const std::vector<NumericY>&, const std::vector<XType>&, const std::map<std::__cxx11::basic_string<char>, std::__cxx11::basic_string<char> >&) [with NumericX = std::vector<double>; NumericY = std::vector<double>; NumericZ = std::vector<double>]’:
../main.cpp:211:37:   required from here
../include/matplotlibcpp.h:1142:33: error: ‘get_array’ was not declared in this scope
     PyObject* xarray = get_array(x);
                        ~~~~~~~~~^~~
../include/matplotlibcpp.h:1142:33: note: suggested alternative:
In file included from ../main.cpp:2:0:
../include/matplotlibcpp.h:403:19: note:   ‘matplotlibcpp::detail::get_array’
 inline PyObject * get_array(const std::vector<std::string>& strings)
                   ^~~~~~~~~
In file included from ../main.cpp:2:0:
../include/matplotlibcpp.h:1143:33: error: ‘get_array’ was not declared in this scope, and no declarations were found by argument-dependent lookup at the point of instantiation [-fpermissive]
     PyObject* yarray = get_array(y);
                        ~~~~~~~~~^~~
../include/matplotlibcpp.h:1142:33: note: ‘get_array’ declared here, later in the translation unit
     PyObject* xarray = get_array(x);
                        ~~~~~~~~~^~~
../include/matplotlibcpp.h:1144:33: error: ‘get_array’ was not declared in this scope, and no declarations were found by argument-dependent lookup at the point of instantiation [-fpermissive]
     PyObject* zarray = get_array(z);
                        ~~~~~~~~~^~~
../include/matplotlibcpp.h:1142:33: note: ‘get_array’ declared here, later in the translation unit
     PyObject* xarray = get_array(x);
                        ~~~~~~~~~^~~```


Support one continuous net_buf for TCP TX case

Is your enhancement proposal related to a problem? Please describe.
Currently in function tcp_out_ext, for TCP TX, it will use two separate net_buf, one contains ip and tcp header, another contains data payload.
It has two side effects:
(1) for running TCP throughput case, the num of net_buf should be twice of net_pkt, and consume more memory.
(2) non-continuous buffer will decrease the memcpy efficiency when copy to lower interface.
Describe the solution you'd like
Use single net_buf for TCP TX data when NET_BUF_DATA_SIZE is large enough.

If memory is an issue, it is actually better to set CONFIG_NET_BUF_DATA_SIZE to smaller value so that buffers are utilized better if the amount of data to be sent is small. Other option is to use CONFIG_NET_BUF_VARIABLE_DATA_SIZE which uses malloc to allocate just enough memory to the data to be sent.

Hi @rlubos  why configuring the tcp_rx_window/tcp_tx_window as (CONFIG_NET_BUF_TX_COUNT * CONFIG_NET_BUF_DATA_SIZE) / 3, not (CONFIG_NET_BUF_TX_COUNT * CONFIG_NET_BUF_DATA_SIZE) / 2 or other value?
https://github.com/zephyrproject-rtos/zephyr/blob/1b70bfa14e288d97b44d2446c8ef0b93226514e6/subsys/net/ip/tcp.c#L57


[Feature] Automate adsorbate_depositor database check with GitHub workflow

Automate adsorbate_depositor database check with GitHub workflow: recheck only upon database change

Maybe not necessary, should close.


Bicep extension not working on WSL

Does this occur consistently? Yes
Repro steps:

I create a bicep file, for instance main.bicep.
It appears an error (see below) reporting an issue related to sdk version.
All bicep edition features no longer wors.

I'm working on WSL2 and I have a dotnet sdk installed on Linux. It seems it tries to use the sdks installed in Windows.
Action: bicep.activate
Error type: string
Error Message: Launching server using runtime /mnt/c/Users/redacted:username/AppData/Local/Programs/Microsoft VS Code/C:\Program Files\dotnet\sdk\redacted:id\ failed. Error: spawn /mnt/c/Users/redacted:username/AppData/Local/Programs/Microsoft VS Code/C:\Program Files\dotnet\sdk\redacted:id\ ENOENT
Version: 0.12.1
OS: linux
OS Release: <IP_ADDRESS>-microsoft-standard-WSL2
Product: Visual Studio Code
Product Version: 1.73.0
Language: en

Can you please verify that you have the Linux version of Bicep installed in WSL and which version by running bicep --version?

Here you have
bicep --version
Bicep CLI version 0.12.1 (e43d137c2d)

And these are the messages that appears

Ah, I now I understand. You are starting a WSL session from VSCode on Windows.

Any idea how to fix it? I don't know if it is possible to configure the sdk path in VSCode to use the WSL version

I'm not able to repro this. Can you run the following command in WSL termnial to verify that you have  both the Bicep extension and its dependency .NET Install Tool for Extension Authors:
code --list-extensions --show-versions

Are you using the dotnetAcquisitionExtension.existingDotnetPath setting described here? https://learn.microsoft.com/en-us/azure/azure-resource-manager/bicep/installation-troubleshoot
If so, please try again after removing the setting.


additional servicemonitor helm values support

Close #383

Let's wait for green CI to merge this.


Add Current user to Tester filter

It would be nice to be able to specify "Current user" in Tester filter. Probably also in Writer filter. The same as it is in Assignee.

@simkam been a bit busy with other stuff, but I have started looking at this


Customizing Rule Set

I am having trouble modifying my project's rule set (custom .ruleset file). No matter what I configure, the full set of rules are applied.
I do have ReSharper installed (with no extensions). I've verified that the rule set is applied on the current build configuration, etc. I am successful with the custom stylecop.json.
Clicking "Open Active Rule Set" on the Analyzers node gives me the error "The rule set file could not be opened"...
Any ideas?

FYI, if you open the project properties and select "Code Analysis", you'll see what rule set is selected and it should show you the path it used to find the rules.
At one point, I didn't have my rule set saved, and I got nearly the same error. Looking at the properties might help you troubleshoot.

@MADCookie  Yes, thanks. I did verify the location of the rule set. It is indeed the one that I'm editing.
Unfortunately my problem is elsewhere.

The very same problem occurred in my project. My ruleset path was relative to current project in form of <CodeAnalysisRuleSet>..\..\Shared\Common.ruleset</CodeAnalysisRuleSet> and I was getting the very same error. Unload the project and reloading it again fixed the problem temporary.
I would like to know the reason why this is happening as I am getting the error quite frequently.

I would report this bug to the visual studio team. I don't think we can do anything about it. The settings are provided to us by visual studio/the compiler. We don't parse them ourself.

My issue was resolved only by fully reinstalling Visual Studio 2015. Even a repair was insufficient.
Thanks.


No Module Named when Referencing local packages (folders)

Just running the main.py locally works.
Referencing a sub folder as a module
tree:
- test
- - __init__.py
- - base.py
- main.py
in main.py:
from test import base
def some_function():
    base.testFunc()
if __name__ == "__main__":
    some_function()
When deploying the function, or when running emulators locally with firebase emulators:start
ModuleNotFoundError: No module named 'test'
When running python main.py everything works as expected

Learned something else, it seems it works when referencing the folders within functions.
I can do imports such as "from x import y" directly in the function itself, but not at the top of main.py

Adding this to the top of main.py works as a workaround, but it's not ideal:
import sys
from pathlib import Path

sys.path.insert(0, Path(__file__).parent.as_posix())

from test import base
...

Ran into the same issue. Thanks for posting the workaround!

I'm still seeing the issue even when using the main branch. I'm not a Python expert but looking #12,
it adds this:
sys.path.insert(0, os.getcwd())

But @gijswobben's fix, which does work for me, adds this:
sys.path.insert(0, Path(__file__).parent.as_posix())


[RFC] support for embedding settings in a library

I'm trying to right now but struggling to understand how to configure it
The list of options here is quite confusing (as well as: the table extends off the rhs of the page but Chrome does not show  a horizontal scrollbar).
in clitool/conf/__init__.py I have instantiated:
settings = LazySettings(
    ENVVAR_PREFIX_FOR_DYNACONF='CLITOOL',
    SETTINGS_FILE_FOR_DYNACONF="clitool.ini",
)
settings.update({
    "MAX_LEVEL": 10,
    "ALLOW_THINGS": False,
})

I can now:
from clitool.conf import settings
settings.MAX_LEVEL
>>> 10

I would then like users of clitool to be able to create a clitool.ini in the root of their own project and have values loaded from there override the defaults I set directly in clitool.conf.settings when they run clitool.
I tried creating such a file but it does not seem to be loaded.
I'm sure this must be possible with dynaconf but it's not clear how from the docs.
One problem I have is to know what the file paths for SETTINGS_FILE_FOR_DYNACONF are relative to. Are these files in my project (which users will install from pypi)?  Or relative to the CWD when the code is run?
How to specify the format of the file?  By file extension only? Or can I have a .clitool file in TOML format for example?
taken from #155

Dup of #294


Individual color spaces not exported

There is currently no mechanism for a consumer to pull in a single color space, as they're not exposed via /src/index-fn.js.
We can't expose spaces/index.js, since that depends on every color space and expects them all to load. As such, the only feasible option for now is to map them individually in package.json, e.g.:
"exports": {
  ".": {
    "import": "./dist/color.js",
    "require": "./dist/color.cjs"
  },
  "./fn": "./src/index-fn.js",
  "./spaces/srgb.js": "./src/spaces/srgb.js",
  "./spaces/p3.js": "./src/spaces/p3.js",
  [...]
}

Node also supports wildcard exports (which could be a good fit here), but popular bundlers like webpack apparently do not, which limits their use.
The alternative would be to work towards dropping side-effects in color space files, so that we could have an index file that would re-export them individually, together with a separate file that would load and register all of them.
I'm happy to contribute the work here, but I'm not sure which of the two options (if either) you think is best!

Yeah, right now color spaces cannot be tree-shaken, one would need to import spaces/index.js, which as you point out expects them all to load. So this is basically the same issue as #163, just a different facet of it.
I think probably the best way forwards is:

Color spaces drop side effects
spaces/index-fn.js imports and re-exports all of them
index-fn.js imports and re-exports spaces/index-fn
spaces/index.js imports everything from spaces/index-fn.js and registers it calling ColorSpace.register()

Wondering if there's a better suffix than fn for this sort of thing.

So this is basically the same issue as #163, just a different facet of it.

You're right, this is fundamentally the same issue!

I think probably the best way forwards is:

Color spaces drop side effects
spaces/index-fn.js imports and re-exports all of them
index-fn.js imports and re-exports spaces/index-fn
spaces/index.js imports everything from spaces/index-fn.js and registers it calling ColorSpace.register()

I'm happy to prototype that! I'll post a PR once I have something.

Wondering if there's a better suffix than fn for this sort of thing.

I'm coming up short of alternatives, but I think fn is okay, in the same vein as date-fns vs more monolithic libraries like moment.

Thanks! Since this is a pretty extensive change, you may want to start with a draft PR so you can get feedback early before you make repetitive changes for every color space.

Thanks! Since this is a pretty extensive change, you may want to start with a draft PR so you can get feedback early before you make repetitive changes for every color space.

Good idea, thank you! 👍

This should have been auto closed after #184 was merged, so I'm gonna manual close it.


Update SIPMOpSensorSim.fcl

Account for mesh transmission in VD using a lower x-arapuca detection efficiency.

Hi @lpaulucc, would this change affect also protodune-vd simulation? Since in pd-vd the mesh is already included in the geometry, maybe we should create two different fhicl tables?

Yes, we would need a pdvd config to get it right for protodune

I restored the original QE and created a VD specific config. Even when we go back to the efficiency without the meshes, it will be more helpful to have a VD, PDVD, PDHD, etc... config


bio_ethanol carrier should not be available for transport in 100% pure form. Instead a fuel mix called 'E85' should be a transport fuel

Vehicles do not actually use 100% bio-ethanol. What they buy at pumps is actually E85, a mixture of 85% ethanol and 15% gasoline.
Suggested changes to etengine
After some more research by me, I wil suggest to change this is in the graph as follows:

create gasoline_mixer converter
rename bio_ethanol demand converters to e85demand converters

Also we need to:

create the e85 carrier
replace the bio_ethanol edges with e85 edges left of the gasoline_mixer converter

Changes to etmodel:
We need to

rename bio_ethanol sliders to E85 or something
update explanations and texts accordingly so users will understand that his carrier does contain some gasoline.

@jorisberkhout and @ChaelKruip please be aware of this issue.
@jorisberkhout I think it would a be a good idea to ask Thais what the Brazilians do with all their bio_ethanol and whether they also use E85 rather than 100% pure ethanol. Can you do this?

E95 supposedly also exists. Placing issue on hold until I know more

I have updated bio_ethanol on etdataset in quintel/etdataset@ 6c061e1eb9c9e109de05009cc69d90c5cd44772f to have an E85 price based on average monthly gasoline prices.
I have included a debt in the carrier_source_analysis that we use the physical properties of ethanol, but the prices for E85.
Putting on hold for now


Add a container health check

From the AWS console, the Health status of the task is Unknown:

I found the docs on how to add an health check here.
We could define an health check to tell AWS when the app is ready to accept requests. E.g. the health check can be an http endpoint, so that we start receiving requests after the http server started.
What do you think?

Yeah, that makes sense, should be easy to implement.

the aws dashboard still shows "unknown" so I need to investigate

https://github.com/rust-lang/simpleinfra/pull/559
I'll merge this tomorrow and check if it works in staging 👍

Thanks! Bors is an important service (well, this specific bors instance hopefully will be important one day :laughing:), so if we can get a health check that could perhaps even alert us in some way if it goes offline, that would be nice.

this specific bors instance hopefully will be important one day 😆

It will 😁
This is now done. Closing this 🎉


Port few more methods from ZIO for optional values

This PR adds few more methods (equivalent to ZIO ones), for queries with optional values

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


fix decode selector when use zookeeper

fix for  https://github.com/aeraki-mesh/aeraki/issues/246

@zhaohuabing    can help  review?


MM-23014 Keep Android cache for vector icons

Summary
Vector icons (as in the close icon) on Android are saved in the cache folder. When we logout / clear cache, the cache folder was completely removed.
With this PR we create a custom vectorIcons folder inside the cache folder and when cleaning that folder we will keep the vectorIcons so any previous cached icon will remain accessible.
The reason why it was not being regenerated unless you closed and re-opened the app is cause react-native-vector-icons keeps a list of "cached" files in a SET to avoid the need to hit the Native side.
iOS is unaffected.
Ticket Link
https://mattermost.atlassian.net/browse/MM-23014

Building app in separate branch.

Successfully building:
https://circleci.com/gh/mattermost/mattermost-mobile/4069
https://circleci.com/gh/mattermost/mattermost-mobile/4068

Artifact links:
https://4069-70265724-gh.circle-artifacts.com/0/~/mattermost-mobile/Mattermost_Beta.ipa
https://4068-70265724-gh.circle-artifacts.com/0/~/mattermost-mobile/Mattermost_Beta.apk


Add Resource Generate and use in API Controller

I added the functionality to generate Resource for those who prefer using it with the option to enable or disable it from the config also if it's enabled the API controller will use it.
I hope it will be helpful for someone

Hi @Si-HaMaDa, thanks for your efforts. We will review it soon and will get it merged.

Thank you @Si-HaMaDa. we did several fixes and now it's ready to merge. We will be working on documentation and then will merge this PR.

Thank you @Si-HaMaDa. we did several fixes and now it's ready to merge. We will be working on documentation and then will merge this PR.

@mitulgolakiya
Not at all, any thing to help :100:

@mitulgolakiya
Not at all, any thing to help :100:


Proto specific comparison

Gomock is used with GRPC services. And sometimes there is an error when matching the proto messages.
Using proto.Equal if the arguments are proto messages will be helpful.

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

I believe gomock's default matcher should not take on technology specific dependencies outside of go's standard ecosystem.
If proto specific logic is added, there's an argument that thrift specific logic should be added and so on and so forth which increases the maintenance burden of this library. Looking at this change, a custom matcher for this use-case seems sufficient.

Wont fix. We can continue to discuss this in an issue or a Discussion but any change for gomock's default matchers should be proto agnostic.
Sample code for a custom proto matcher.
type protoMatcher struct{ msg proto.Message }

var _ gomock.Matcher = (*protoMatcher)(nil)

func (m *protoMatcher) String() string { / ... TODO .. */ }

func (m *protoMatcher) Matches(x any) bool {
  other, ok := x.(proto.Message)
  if !ok { return false }

  return proto.Equal(m.msg, other)
}


🧪QA Testing

Complete The Item Below

[X] I have reviewed and tested all of the issues in the linked milestone.

Relevant Information
Testing needs to be thorough.  Pay special attention to the rendering side of things.  Items such as glitchy rendering and layers are very important things to look out for this release.
For a complete list of everything released this release, go here
Release:
Velaptor-v1.0.0-preview.16.zip
QA Testing Dependencies
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

@CalvinWilkinson All tests passed.


If possible, it is hoped that a hollowing function is built in, like the Savor theme in the image and video below.

https://github.com/user-attachments/assets/d147d0e6-282d-405e-abd8-11d69e7417bd

Thank you for your feedback. Will consider implementing this feature in future dev process

Hey, looks like there's already a plugin that does exactly what you asked for, and it's pretty good! I'll go ahead and close this issue.


ET Moses standalone?

In the ETM interface there are instructions to deploy ETM on a standalone laptop or desktop. Will this also be possible for ET Moses? This could be a possible solution for usecase with privacy & security issues.

Assigned @dennisschoenmakers

@JeroenvdLogt, ETMoses can make private LESs. Is that not sufficient for foreseen applications?

Will this also be possible for ET Moses?

It is possible but still quite technically challenging (you need to be able to run some software that are not installed by default on a pc).

I believe this will be challenging Chael but still the only option in some usecases. Privacy & security requirements  and -procedures of Liander (and more parties) can be a really pain in the ass;) Any involvement of American companies for cases with sensitive data is seen as big risk. It must be completely clear where data is stored and transported to. Because of some major issues recently procedures will become more complex.
In these cases a stand alone version is seen as less riskier and permission will be given more easy. It is worth trying I think.

We will add general instructions to the README file on how to run etmoses on your local machine comparable to the instructions found on etmodel. See #386. Closing this one.


Fix: changed 'tag' table name to 'category' in POST category endpoint.

Description
Fixing table name for creating categories with POST category endpoint.
Fixes # Changed 'tag' to 'category' in POST category endpoint.
Type of change - one word name change correction.
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Change Status

[x] Complete, ready to review and merge

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] My code has been reviewed by at least one peer
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] There are no merge conflicts

 POST Category set to create to tag table instead of category table


delivery report callback do not return  acknowledgement for every message

I am using librdkakfa 0.9.3 in RHOS
i have set dr_msg_cb to get delivery report for each message
here is the function def:
static void dr_msg_cb (rd_kafka_t *rk,const rd_kafka_message_t *rkmessage, void opaque) {
if (rkmessage->err)
fprintf(stderr, "%% Message delivery failed: %s\n",rd_kafka_err2str(rkmessage->err));
else
fprintf(stderr,"%% Message delivered (%zd bytes, " "Partition %"PRId32")\n",
rkmessage->len, rkmessage->partition);
/ The rkmessage is destroyed automatically by librdkafka */
}
function call:
kafka_conf  = rd_kafka_conf_new();
rd_kafka_conf_set_dr_msg_cb(kafka_conf, dr_msg_cb);
i observe that when i push 1k events to kafka producer
I get '% Message delivered (32 bytes, Partition 6) " for 217 times..
How can i understand the number of messages actually received by consumer

Hi,
I think you meant to post in https://github.com/edenhill/librdkafka/issues
Can you repost your issue there?
If you can, try to use the latest version (0.9.5) to see if this solve problem. In all case, a code snippet of how you produce data would also help
Thanks!

ok thanks for the reply..i wl repost the issue with code snippet