Cannot read property 'length' of null

Issue Id: bd26075f-745a-ed92-db67-fc21bc52c453Versions - 1.6.0-insider (9/23/2016 6:06:48 AM)Stack TypeError: Cannot read property 'length' of null/vs/editor/common/model/textModel.ts#L456:21 (length)/vs/editor/common/model/textModel.ts#L636:23 (getLineCount)/vs/editor/common/model/textModel.ts#L328:31 (getFullModelRange)/vs/workbench/parts/git/browser/gitWorkbenchContributions.ts#L244:19 (setValueFromRawText)/vs/base/common/winjs.base.raw.js#L1170:0 (onComplete)/vs/base/common/winjs.base.raw.js#L852:0 (_notify)/vs/base/common/winjs.base.raw.js#L1068:0 (enter)/vs/base/common/winjs.base.raw.js#L1036:0 (_run)/vs/base/common/winjs.base.raw.js#L1170:0 (onComplete)/vs/base/common/winjs.base.raw.js#L852:0 (_notify)

usage of this._lines.length' when this._lines` is null is the cause of the error with the below call stack as well. Adding it here instead of logging a separate issue
/vs/editor/common/model/textModel.ts#608:36 (length)
/vs/editor/common/model/textModel.ts#626:19 (validatePosition)
/vs/editor/common/services/modelServiceImpl.ts#87:21 (validateRange)
/vs/editor/common/services/modelServiceImpl.ts#78:16 (_createDecorationRange)
/vs/editor/common/services/modelServiceImpl.ts#76:74 (map)
/vs/editor/common/services/modelServiceImpl.ts#339:22 (setMarkers)
/vs/editor/common/services/modelServiceImpl.ts#333:19 (forEach)
/vs/base/common/callbackList.ts#63:26 (apply)

Another one:
Issue Id: ad2858e8-52d8-4a8e-0b83-6ac382285918Versions - 1.6.0-insider (9/23/2016 6:06:48 AM)Stack TypeError: Cannot read property 'length' of null/vs/editor/common/model/textModel.ts#L520:36 (length)/vs/editor/common/model/textModel.ts#L283:15 (getLinesContent)/vs/editor/common/services/editorWorkerServiceImpl.ts#L185:16 (toRawText)/vs/editor/common/services/editorWorkerServiceImpl.ts#L149:9 (_beginModelSync)/vs/editor/common/services/editorWorkerServiceImpl.ts#L296:40 (esureSyncedResources)/vs/base/common/winjs.base.raw.js#L1566:0 (onComplete)/vs/editor/common/services/editorWorkerServiceImpl.ts#L295:26 (then)/vs/editor/common/services/editorWorkerServiceImpl.ts#L308:14 (_withSyncedResources)/vs/editor/common/services/editorWorkerServiceImpl.ts#L58:64 (computeDirtyDiff)/vs/base/common/winjs.base.raw.js#L1566:0 (onComplete)

The root cause is various folks holding on to and making calls on disposed models. Will add assertions to get the callers.


API: Images sample code for GET request

Sample code for the API documentation project.
You can view my sandbox here: https://www.mediawiki.org/wiki/User:Martyav/Sandbox/API:Images

Sweet! :-)


fix: Add fail-fast when node sharing is enabled in SolverConfig instead of application.properties

The code must be validated before SolverConfig becomes available as a SolverConfigBuildItem, which means we must do it here (which is where we construct the SolverConfigBuildItem)

The code must be validated before SolverConfig becomes available as a SolverConfigBuildItem, which means we must do it here (which is where we construct the SolverConfigBuildItem)

This is going to bite us one day. Is there no chance of making the enterprise quarkus extension, well, extend the community quarkus extension? Do they need to be two different processors?

Must be two separate processors due to different dependencies.
SolverConfig validation is done in community, see
https://github.com/TimefoldAI/timefold-solver/blob/bc2d5a24201bab3c7d659b726f0656c711849215/quarkus-integration/quarkus/deployment/src/main/java/ai/timefold/solver/quarkus/deployment/TimefoldProcessor.java#L696-L706
which is also done in community despite being an enterprise feature.


Speed improvements on code

Reduced WMI queries from 11 to 3
Used more efficient queries by using WQL
Used variable assignment instead of independent functions
Used -f parameter to join text to reduce number of Write-Host cmdlets

Wow that is awesome! I learned a bunch of new things just by looking at your changes!

No problem, glad to be able to help you out. I just noticed that is was
kinda slow so I looked at how you structured your WMI queries and then I
got carried away :)
Happy scripting!
On Tue, Aug 25, 2015 at 10:23 AM, Mika Millar<EMAIL_ADDRESS>wrote:

Wow that is awesome! I learned a bunch of new things just by looking at
your changes!
—
Reply to this email directly or view it on GitHub
https://github.com/michalmillar/ps-motd/pull/1#issuecomment-134520595.


ZIP

Anyone considering adding zip files and unzipping functionality ?

Do you need exactly zip files support or tar.gz support is fine?

both if possible, priority to zip at the moment
Met vriendelijk groeten,
Miel Hostens,
B**uitenpraktijk+
Department of Reproduction, Obstetrics and Herd Health
http://www.rohh.ugent.be/
Ambulatory Clinic
Faculty of Veterinary Medicine http://www.ugent.be/di
Ghent University  http://www.ugent.be/
Salisburylaan 133
9820 Merelbeke
Belgium
Tel:   +32 9 264 75 28
Fax:  +32 9 264 77 97
Mob: +32 478 59 37 03
e-mail<EMAIL_ADDRESS>On 11 August 2017 at 14:29, Girish Reddy<EMAIL_ADDRESS>wrote:

Do you need exactly zip files support or tar.gz support is fine?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/springml/spark-sftp/issues/15#issuecomment-321800117,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALaIocIgxpGYhB6rd4bzOx-8DXp-CQQDks5sXEkwgaJpZM4OyIcq
.

@springml HI all do we have any update on this issue is the library capable of handling zip files?
Or if there is any work around for this one.
Thanks,
Ankush Reddy.

@springml : We kind of have a similar use case for zip where we have a csv which is zipped. So anything that extends that would be helpful


Parse ExecutionProperty in queryParameters

Fixes gh-854. Query param isn't quoted if it's an ExecutionProperty, similarly to https://github.com/spring-cloud/spring-cloud-contract/blob/c323e3e105810498a2bb9869d87eb144d5c8d0d5/spring-cloud-contract-verifier/src/main/java/org/springframework/cloud/contract/verifier/builder/JaxRsRequestHeadersWhen.java#L46-L67

Great job @mariuszdb !


Upsert support for query builder

Currently there's no way to atomically upsert a record i.e. try to insert a record, if uniqueness constraint failed, update it.
Suggested signature:
public function upsert($table, $columns, &$params);

MySQL supports it via INSERT ... ON DUPLICATE KEY UPDATE: https://dev.mysql.com/doc/refman/5.7/en/insert-on-duplicate.html
In PostgreSQL it could be achieved multiple ways: https://dzone.com/articles/performing-upsert-update-or-insert-with-postgresql
SQLite has INSERT ON CONFLICT REPLACE: http://sqlite.org/lang_conflict.html
MSSQL, Oracle. Complicated but doable via MERGE + some safeguards: http://weblogs.sqlteam.com/dang/archive/2009/01/31/UPSERT-Race-Condition-With-MERGE.aspx and http://michaeljswart.com/2011/09/mythbusting-concurrent-updateinsert-solutions/.

More analysis from PostgreSQL: https://wiki.postgresql.org/wiki/UPSERT

Could help solving issues like https://github.com/yiisoft/yii2/issues/11401

Can and will it support things like MySQL's VALUES(`col`)?
INSERT INTO `table` (..., `col`)
VALUES (..., 1)
ON DUPLICATE KEY UPDATE `col` = `col` + VALUES(`col`);

Does it violate AR concept?
 One of underlying reasons is that it cannot lock phantom rows. I did try two scenarios to perform INSERT or UPDATE; with non-AI PK and both does hit deadlock due to this reason. I want again to throw MySQL into trash can.

What's VALUES(col)?

Something like this:
INSERT INTO table (a,b,c) VALUES (1,2,3),(4,5,6)     ON DUPLICATE KEY UPDATE c=VALUES(a)+VALUES(b);
https://mariadb.com/kb/en/mariadb/values/

Sorry, I want to ask about referencing to old value:
ON DUPLICATE KEY UPDATE `col` = `col` + ...

Without this ability the subject feature is not so cool. Without this I still need to combine SELECT with INSERT or UPDATE.
Usage of VALUES(`col`) for single row INSERT is not so necessary. Just copy insertable value literally.

Of course, expressions should be supported.

From the suggested signature:
public function upsert($table, $columns, &$params);

We will be doing basic queries like:
$queryBuilder->upsert('item', [
    'id' => 1
    'name' => 'some item name',
    'description' => 'lorem ipsum blablabla'
    // other columns
]);

Which will provide something like:
INSERT INTO `item`
(`id`, `name`, `description`, /* other columns */)
VALUES
(...)
ON DUPLICATE KEY UPDATE
`id` = VALUES(`id`),
`name` = VALUES(`name`),
`description` = VALUES(`description`),
/* other columns */

It would also be good to have, like the insert() and batchInsert(), a function batchUpsert(). But then some problem might arise. For example where in some cases there are null values which should not override a previously filled column. How do we apply expression on the UPDATE part:
INSERT INTO `item`
(`id`, `name`, `description`, /* other columns */)
VALUES
(...),
(...),
(...),
(...),
(...)
ON DUPLICATE KEY UPDATE
`id` = VALUES(`id`),
`name` = COALESCE(VALUES(`name`), `name`),
`description` = COALESCE(VALUES(`description`), `description`),
/* other columns */

@samdark  If you’re busy ATM, I would  implement the feature next week to resolve #13212.

@sergeymakinen would be great. Thank you.

 Get ready. ;)

Here we go: #15276


Bump Mermaid JS

Aquamarine uses Mermaid v9.1.5.
In this minimal repro we're using some mermaid syntax which only was made available in later versions (correctly renders on Mermaid Live)
Recommended Fix:
Update mermaid version in https://github.com/mersinvald/aquamarine/tree/master/doc/js to latest. https://github.com/mermaid-js/mermaid/releases

Hi @gakonst. Could you please test this branch on your case and tell me how it goes?

Yup confirming it worked, e.g. my repro now renders! Send it!


Support for multi-part tweets?

I couldn't find any information on whether this is already supported, but being able to detect multi-part tweets would be awesome. E.g. these two tweets:

We regret to inform that we will not be attending Starladder as we were unable to obtain our visas in time. (1/2)

The Ukrainian embassy was closed from the 1st to the 10th due to the new year (2/2)

https://twitter.com/CDECgaming/status/554940186832814080
https://twitter.com/CDECgaming/status/554940524163911680

It's not possible to figure out which tweet another is in response to unless they've actually replied to the tweet. Anything else is going to be error prone and probably not that useful anyway.
As it stands, the bot mirrors the linked tweet as well as the chain of tweets it's a reply to, like so, which covers most cases.

I wrote up a comment about using regex to detect the (1/2) at the end, and then getting the users recent tweets and checking if the next one had (2/2), but I realised that the tweets being linked to won't always be the user's most recent tweets.
I haven't done anything with the Twitter API (or used Python in a while), but I'll have a poke around and see if I get anywhere.

I've done some very early prototyping here: https://github.com/InfernoZeus/TweetPoster/commit/9a97f3eb6680340fdc6b6640b94391b56099769b
It works (http://www.reddit.com/r/InfernoZeus/comments/2soyk1/example_multipart_tweet/) but is very buggy - for example, I've no idea what will happen if subsequent parts are actually replies to the previous one. Probably post them in this order:

Tweet 1 (Part 1)
Part 2
Part 3
Tweet 2 (Part 2 in reply to Part 1)
Part 3
Tweet 3 (Part 3 in reply to Part 2)

I also had to add in some extremely basic filtering so I could run tests in my own private subreddit.
Let me know if you have any thoughts :)


Refactor Network class

Refactored the network module, moving the fetchUTXOsForNumberOfStampsNeeded(), validateSLPInputs(), and broadcastTransaction() functions inside of the Network class with proper unit and integration tests.

:tada: This PR is included in version 1.0.5 :tada:
Your semantic-release bot :package::rocket:


Feature Request: Make EcsConverter methods public

It might make sense to make the converter methods public. In my converter implementation, I have to completely duplicate the ConvertContainer and ConvertObject code to make an automatic monoconverter.
Here is what I wanna use:
public class AwakeConverter : MonoBehaviour
{
    [SerializeField]
    private bool _destroyObjectAfterConvertion;
    [Inject]
    private EcsWorld _world;

    private void Awake()
    {
        EcsConverter.ConvertObject(gameObject, _world);
        if (_destroyObjectAfterConvertion)
        {
            Destroy(gameObject);
        }
    }
}

But EcsConverter.ConvertObject is not available due to it's protection level:
private static void ConvertObject(GameObject obj, EcsWorld world)
Maybe makes sense making it public?

Also possibility of direct calling internal static void ConvertContainer(ComponentsContainer container, EcsWorld world) can be useful.


Libraries?

It does not say if it runs on python 2 or 3. If I run it on python 2 it says:
python main.py
Traceback (most recent call last):
File "main.py", line 1, in
from neural_network import NeuralNetwork
File "/home/al/neural-network-animation/neural_network.py", line 3, in
from matplotlib import pyplot
File "/home/al/neural-network-animation/matplotlib/pyplot.py", line 27, in
import matplotlib.colorbar
File "/home/al/neural-network-animation/matplotlib/colorbar.py", line 32, in
import matplotlib.artist as martist
File "/home/al/neural-network-animation/matplotlib/artist.py", line 12, in
from .transforms import Bbox, IdentityTransform, TransformedBbox,
File "/home/al/neural-network-animation/matplotlib/transforms.py", line 39, in
from matplotlib._path import (affine_transform, count_bboxes_overlapping_bbox,
ImportError: No module named _path
And if I run it under python 3 it says:
Traceback (most recent call last):
File "main.py", line 1, in
from neural_network import NeuralNetwork
File "/home/al/neural-network-animation/neural_network.py", line 3, in
from matplotlib import pyplot
File "/home/al/neural-network-animation/matplotlib/pyplot.py", line 27, in
import matplotlib.colorbar
File "/home/al/neural-network-animation/matplotlib/colorbar.py", line 32, in
import matplotlib.artist as martist
File "/home/al/neural-network-animation/matplotlib/artist.py", line 12, in
from .transforms import Bbox, IdentityTransform, TransformedBbox,
File "/home/al/neural-network-animation/matplotlib/transforms.py", line 39, in
from matplotlib._path import (affine_transform, count_bboxes_overlapping_bbox,
ImportError: No module named 'matplotlib._path'
Apparently it requires some modules? I am running it under lubuntu 14.2

Did you solve this problem?

Since it has been over half a year I will ask again. Does anyone have a solution to this? I have the same issue.

By the way, I'm getting this error when I run python main.py:
ImportError: No module named _path
Any help? Thank you very much!

you need to download matplotlib using pip to get rid of the _path error. the version here doesn't work.
you also need to install ffmpeg if you haven't.
However, after going through all of this, the MP4 file and images that this program generates are all just black screens as far as i can tell, so I'd stay away from this anyway.


HDDS-3826. Split Ozone FS acceptance tests

What changes were proposed in this pull request?

Split monolithic Ozone FS acceptance test case into several steps.  This improves feedback about test progress, and makes easier to find out what's wrong in case of a failure.  (Unfortunately most subsequent steps still depend on earlier ones.  Improving that would be nice, but would make the test slower due to the necessary extra setup steps.)
Run test setup in proper Setup instead of a test case.
Split test setup into reusable keywords.
Add functions for formatting Ozone FS URLs with both o3fs and ofs flavor.  Includes its own test cases.
Test execution for ofs and o3fs is controlled outside of the test script.  This is necessary to avoid duplication of test cases and/or keywords.
Pass SECURITY_ENABLED and similar variables as Robot variables (-v), not environment ones (-e).  This allows setting default value in Robot tests, which makes it easier to run the test from command-line.

Most of this is in preparation for bucket links, as the same test should be executed for regular buckets and links.  Submitting as a separate PR for easier review.
Note that this PR is based on top of the change for #1091, a minor improvement which is necessary to save output of multiple executions of the same test suite (ozonefs.robot in this case).
https://issues.apache.org/jira/browse/HDDS-3826
How was this patch tested?
Ran tests for lib locally (no docker-compose):
$ robot lib
...
Lib.Fs Tests                                                          | PASS |
...
Lib.String Tests                                                      | PASS |
...

Ran Ozone FS tests inside docker container bypassing test.sh:
$ docker-compose exec scm robot smoketest/ozonefs/ozonefs.robot
...
Ozonefs :: Ozone FS tests                                             | PASS |
20 critical tests, 20 passed, 0 failed

$ docker-compose exec scm robot -v SCHEME:o3fs smoketest/ozonefs/ozonefs.robot
...
Ozonefs :: Ozone FS tests                                             | PASS |
20 critical tests, 20 passed, 0 failed

Full acceptance test suite:
https://github.com/adoroszlai/hadoop-ozone/runs/783679213
https://github.com/adoroszlai/hadoop-ozone/runs/784247369

Pass SECURITY_ENABLED and similar variables as Robot variables (-v), not environment ones (-e). This allows setting default value in Robot tests, which makes it easier to run the test from command-line.

DANCING BANANA. Thank you very much,
The patch LGTM.
One comment: string_test.robot and fs_test.robot seems to be duplicate of robot.robot. (I guess you refactored the robot.robot to two separated files.) I think one of them can be removed.

One comment: string_test.robot and fs_test.robot seems to be duplicate of robot.robot.  I think one of them can be removed.

Nice catch.

(I guess you refactored the robot.robot to two separated files.)

Exactly.

Thanks @elek for reviewing and committing it.


Crash when using Posenet (TensorFlowJS)

I use ThreeJS and TensorFlowJS.
https://github.com/tensorflow/tfjs-models/tree/master/pose-detection
After initalizing Posnet webgl-memory crashes
webgl-memory.js:772 Uncaught (in promise) ReferenceError: obj is not defined
at webgl-memory.js:772
at WebGL2RenderingContext.ctx. [as fenceSync] (webgl-memory.js:921)
at t.e.createFence (tfjs.js:17)
at t.e.createAndWaitForFence (tfjs.js:17)
at e. (tfjs.js:17)
at tryCatch (runtime.js:63)
at Generator.invoke [as _invoke] (runtime.js:293)
at Generator.next (runtime.js:118)
at Wm (tfjs.js:17)
at o (tfjs.js:17)
at tfjs.js:17
at new Promise ()
at e. (tfjs.js:17)
at e.read (tfjs.js:17)
at t.e.read (tfjs.js:17)
at t. (tfjs.js:17)
at tryCatch (runtime.js:63)
at Generator.invoke [as _invoke] (runtime.js:293)
at Generator.next (runtime.js:118)
at Wm (tfjs.js:17)
at o (tfjs.js:17)
at tfjs.js:17
at new Promise ()
at t. (tfjs.js:17)
at t.data (tfjs.js:17)
at t. (tfjs.js:17)
at tryCatch (runtime.js:63)
at Generator.invoke [as _invoke] (runtime.js:293)
at Generator.next (runtime.js:118)
at Wm (tfjs.js:17)
at o (tfjs.js:17)
at tfjs.js:17
at new Promise ()
at t. (tfjs.js:17)
at t.buffer (tfjs.js:17)
at posenet.js:17
at Array.map ()
at posenet.js:17
at posenet.js:17
at Object.next (posenet.js:17)
at posenet.js:17
at new Promise ()
at u (posenet.js:17)
at posenet.js:17
at t. (posenet.js:17)
at posenet.js:17
at Object.next (posenet.js:17)
at posenet.js:17
at new Promise ()
at u (posenet.js:17)

Have you any hint for me. Thanks.

Sorry it was a bug. Try v1.0.7

Thanks! Now it works. 👍


[WIP] Use .NET Core SDK

Closes #136.
TODO:

[x] test user experience in VS
[x] update packaging instructions
[x] fix appveyor.yml (get rid of it completely? it seems we're satisfied by the defaults for now)

Fun fact: I've found that there was a TexExpressionVisitor class that wasn't included into project (the new SDK helped me to find that). I've deleted it because it obviously has no purpose.

Please ignore the commits that reference this PR, #137 in the commit log: these were pushed accidentally and already cleaned up by my following force-push. I couldn't remove them from this GitHub page though. They are absent from my PR branch.

Okay, ready for review!

Hereby I approve these changes. Please report any issues you have with the new project system, I'll keep monitoring them.


nit: Rename this repository, aes-circuits?

aes-proof is a bad name
will require changes to upstream dependencies

Completed


Wasmedge provider

Implement the backbone of Wasmedge provider

The failure of CI due to the conflict of Rust SDK and the version WasmEdge.
Related issue is opened in wasmedge.
https://github.com/WasmEdge/WasmEdge/issues/863

Dependent on https://github.com/WasmEdge/WasmEdge/issues/869


DADATA - added country and region ISO code properties

Just for your information. This was created from master and merged to master. This doesn't affect to v1.1.*. To have it there, changes should be applied to 1.1 branch.

I have prepared a request to merge the branches according to your recommendations.

Thanks!

v1.1.17


Request for guidance on using reuse spdx output with GitHub Attestations in spdx format

Hi actions/attest team,
First, thank you for your excellent work on actions/attest! I’m new to supply chain security and am trying to understand how to incorporate SPDX SBOMs and licensing information into GitHub attestations using your GitHub Action. Most of my repositories are REUSE-compliant, and I’m trying to integrate the output of the reuse spdx command with  attestations in GitHub.
If I’ve misunderstood anything, I apologize, and I’d greatly appreciate any guidance you can offer.
Questions:

How can I integrate the SPDX SBOM and license data generated by the reuse spdx command into GitHub attestations using actions/attest?
How can I generate and attach an SBOM using the npm sbom command and include it in a GitHub attestation using the in-toto format?
Is there a recommended workflow or example for integrating REUSE-generated SPDX data with GitHub's attestation framework?

I’d appreciate any examples or advice on how to use your tool in this context to ensure SPDX compliance and proper licensing documentation.
Thank you again for your support!

I don't have experience with the reuse CLI, but anything which outputs a standard SPDX or CycloneDX JSON-encoded SBOM should be fairly easy to integrate with the attest-sbom action. That action will take an SBOM, wrap it in the appropriate in-toto format, and then attest it.
Have a look at the examples in the readme and lemme know if you still have usage questions.


Integration Requests

~Command Line Tool (https://github.com/dawnlabs/carbon/issues/52, https://github.com/dawnlabs/carbon/issues/240)~

https://github.com/mixn/carbon-now-cli

~IFrame/Embeddable (https://github.com/dawnlabs/carbon/issues/179)~

Released with 3.5.0

Remark.js Integration (https://github.com/dawnlabs/carbon/issues/209)
Carbon API (https://github.com/dawnlabs/carbon/issues/210)
Chrome Extension (https://github.com/dawnlabs/carbon/issues/219)
~Gatsby Plugin (https://github.com/dawnlabs/carbon/issues/312)~

Source

Google Slides Integration (https://github.com/dawnlabs/carbon/issues/353)
~Imgur integration (https://github.com/carbon-app/carbon/pull/1158)~

@mfix22 If you'd like I could try to write the CLI up sometime this week. I'm not sure about the how much work the API would be, I think I can just call getCarbonImage() and pass in params as the options.
Let me know if either would be helpful :ok_hand:

@GavinRay97 i was always imagining that the initial CLI version would simply open a file up in the browser and allow flags to be set for theme and such (to start just just opening a file would be cool). This way no API is needed besides passing the correct query params.
If/when we open an API, the CLI could then later be updated to use it directly, but we shouldn't let that block the CLI effort.
Happy to accept a PR adding it to our community section of the README 😄
Couple notes:

[ ] Has to work cross-platform (here is an example our teammate made awhile back of how to open links in browser for the most popular OS's https://github.com/briandennis/WebLauncher)
[ ] Would love if it was a Node CLI so people can install with NPM

An instagram mode with resizable text based on the square dimensions would be nice too. Let me know where else I can help. I will go ahead and knock off some testing/easy issues.

@kaicianflone there is already a Square dimensions mode for Instagram (: look under "Advanced" in the settings

On my computer it's only exporting as square rather than sizing as square. Like some of my pictures will have the edges cutoff and some of the code missing.

Feature: Create Gist (#393 )

Just like Github gist, add support for GitLab snippets - @mittalyashu (#464)

@mfix22 if you planning to add all the features one by one, how will you structure your repository, will it be a mono-repo or multi-repo?

We don't plan on supporting all of the integrations, just the most popular/useful ones.
A mono-repo 😄

@mixn right, most of these integrations can be created/maintained separate from our project! (:

@mfix22 I have few insight how we can deal with this feature -> IFrame/Embedded
IFrame
Allow other users to embed this whole component on their website (more people will use and it will increase popularity of carbon)

Possible options:

Limitation the use of carbon component on third-party website.
Unlimited usage of carbon component for verified third-party websites (verification can be done via github, gitlab or bitbucket)

Where it can be embed?

Websites

Embed
The user can embed the carbon as an editable image, where the viewers will see as an a code snippet converted into beautiful code which is copy able and for the admin that code snippet settings can be configured and even the code snippet can be changed.
Where it can be embed?

Google Slides
Websites (All types of websites: Dynamic, Static, Blogs)
Medium
Stackoverflow

@mittalyashu What you propose is exactly what I have been thinking about for quite sometime after coming across Carbon. I'm not sure what it would involve but I think it could be a great step in a solid direction.

Embed support was just released with 3.5.0! Try pasting any Carbon URL into Medium, or click the "Copy Embed" button to copy the iframe you can use in your own site or blog.

Slack Integration (#533)

@mfix22 I have tried pasting the carbon url. But nothing is showing on the screen.
https://medium.com/@mittalyashu/omo-b71b2a7529e2

@mittalyashu I'm going to hide your comment, but just because I'd like you to open issues not related to new features as new issues 🙂 Please do so for your comment above

Cant see any API integration ...

Not sure if we are allowed to create an issue for new syntax

But can some add a syntax of Atom one dark theme?

But can some add a syntax of Atom one dark theme?

@mittalyashu this thread is only for integration requests — to integrate Carbon with other services and to integrate other services into Carbon.
As for you question about adding the "Atom One Dark" theme, we already have "One Dark". Is there something preventing you from using the "Create+" a theme feature to create one for yourself?

Hi, thanks a lot for the amazing job here.
Any integration to rst format? It would help a lot for documentations.
Thanks

@guthlStarkware could you open a new issue? I'm not sure what rst format is.

Will do. rst is RestructedText used in Sphinx: https://www.sphinx-doc.org/en/master/usage/restructuredtext/basics.html
I guess my request could be simplified to outputting an HTML file containing all the snippet+CSS+JS. I'll open an issue

@guthlStarkware no need. We can track this under #1042 (which is very similar)

I want to  Integrate to Remark.js, but use embed it will create iframe for every code block. so, is there a better way to accomplish it ?

I want to Integrate to Remark.js, but use embed it will create iframe for every code block. so, is there a better way to accomplish it ?

Something like this: https://github.com/carbon-app/gatsby-remark-embed-carbon @Swordword
Also please open a new issue if you want to discuss this any further. Thanks!

I see in in readme that it can be integrated with markdown sites right?
Is there nay integration with dev.to? It is similar to medium and I cannot make it work even in Medium

Chrome extension is better integration.

Integration request for Confluence macro: https://github.com/carbon-app/carbon/issues/1305


Prevent a crash that is caused by limited typefaces

What does the pull request do?

This PR prevents a crash caused by typefaces that do not contain any whitespace glyphs. Previously this caused an IndexOutOfRangeException.
What is the current behavior?

What is the updated/expected behavior with this PR?

How was the solution implemented (if it's not obvious)?

Checklist

[ ] Added unit tests (if possible)?
[ ] Added XML documentation to any related classes?
[ ] Consider submitting a PR to https://github.com/AvaloniaUI/Documentation with user documentation

Breaking changes

Obsoletions / Deprecations

Fixed issues

Fixes: #9283

You can test this PR using the following package version. 11.0.999-cibuild0025428-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]


reimplement 91X in libcst

This one is chonky
I might go over it tomorrow and add review comments to help reviewing it. Although there's not really much to do other than grind through it - and running it on existing code, I wouldn't be shocked if there's crashes or bugs that didn't exist before, so maybe do that before releasing.
I generally optimized for minimizing the diff, rather than writing the best / most logical code - might leave that for a second commit, or do it when implementing autofix. Esp any isinstances should generally not be there when working against libcst.
I rewrote the loop checker to not require two passes, by injecting a fake statement and doing some fancy logic. Quite happy I came up with this solution, much better than the previous super messy one.
Removing redundant checkpoints might be more doable with this approach as well.
Having to access metadata to get the linenos of nodes is quite ugly, with the redundant checkpoint I'll be looking to store nodes instead of Statements in uncheckpointed_statements
test_910_permutations is now extremely slow, previously being so fast I never noticed it, but now it takes 50 seconds on my machine. I did some rudimentary timing, and the parse_module call took 5 of the seconds, and plugin.run() taking 45 seconds. Now that is across ~4000 calls, so nothing too crazy, but I'll probably put it behind --runfuzz if I don't find anything responsible for the slowdown. 91X currently doesn't use any decorator markers, so changing it to be a simple transformer might do wonders.
A few changes in eval_files:

realized that comprehensions aren't handled, added regression tests.
I think the nested error test ... was wrong? I don't see why that should give two errors per line.
multiple yields in a single boolop is no longer handled as before. I maybe should've just done fmt: off instead of fighting over and over with black/shed

Oh, I didn't check coverage before pushing. Will look at that another day.

test_910_permutations is now extremely slow, previously being so fast I never noticed it, but now it takes 50 seconds on my machine. I did some rudimentary timing, and the parse_module call took 5 of the seconds, and plugin.run() taking 45 seconds. Now that is across ~4000 calls, so nothing too crazy, but I'll probably put it behind --runfuzz if I don't find anything responsible for the slowdown. 91X currently doesn't use any decorator markers, so changing it to be a simple transformer might do wonders.

Switched to using CSTTransformer instead of m.MatcherDecoratableTransformer and that slashed the runtime of 910_permutations from 41.82s to 14.35s - so I'll default to CSTTransformer unless the decorators are actually useful. (Sidenote: I looked into writing my own, and it's not that easy). 14s is still enough to be kind of a hassle though, so added it to fuzz only.
The existing fuzz tests takes ... literal forever now though.

I had to add a bunch of stuff to eval files for coverage, since multiple visitors using the same helper shared the burden of covering them - but now that some of them are converted to libcst all the cases have to be covered both by cst visitors and ast visitors.


pathfinder based initialisation of Bayesian Inference

Background on NUTS initialisation with Pathfinder
Pathfinder is an approach to variational inference, that was initially envisioned as a method for pre-heating NUTS-based Bayesian inference. Performance of NUTS is improved by:

Starting from an (approximate) draw from the posterior distribution of the Bayesian inference problem.
Starting KE matrix adaptation from (inverse) Hessian around the MAP point of the posterior distribution of the Bayesian inference problem.

Pathfinder supplies both of the above using its pathfinder multivariate normal approximation to the posterior distribution. Also, multipathfinder supplies improved initial draw approximation (its samples from a mixture distribution of multiple single pathfinder approximations passed though a Pareto importance sampling filter).
Proposed improvement
Implement the pathfinder initialisation for Turing described here.
Additional questions
Is it worth lowering the inference call to AdvancedHMC so we can use metric initialisation?

I think in the first instance wee should do the simple version and then open a new issue to consider using AdvancedHMC for metric initialisation after discussion.

How is work going on this? Anything I can do to unstick?


Make query randomization more flexible

Description
Controllable query randomization was added in https://github.com/opensearch-project/opensearch-benchmark/pull/455, and was limited to randomizing range queries with fields gte/gt, lte/lt, and optionally format. This is limiting since we might need to randomize other queries for some use cases, like benchmarking the query cache.
This PR lets the user register different query names and parameter names to be randomized for each operation. This is done by adding registry.register_query_randomization_info(...) in workload.py. See an example.
In this example we have the operation:
{
  "name": "bbox",
  "operation-type": "search",
  "index": "nyc_taxis",
  "body": {
    "size": 0,
    "query": {
      "geo_bounding_box": {
        "pickup_location": {
          "top_left": [-74.27, 40.92],
          "bottom_right": [-73.68, 40.49]
        }
      }
    }
  }
}

And in workload.py we run registry.register_query_randomization_info("bbox", "geo_bounding_box", [["top_left"], ["bottom_right"]], []).
The first argument, "bbox", is the operation name. The second argument, "geo_bounding_box", is the query type name.
The third argument is a list of lists: [["top_left"], ["bottom_right"]].  Each entry in the outer list represents one parameter name that will be randomized. It's a list because we may have multiple different versions of the same name that represent roughly the same thing. For example, "gte" or "gt". In this case there's just one option for each parameter name. At least one version of each parameter name must be present in the original query for it to be randomized.
The last argument is a list of optional parameters. If an optional parameter is present in the random standard value source, it will be put into the randomized version of the query. If it's not in the source, it's ignored. There are no optional parameters in this example, but the typical use case would be "format" in a range query.
If nothing is registered, it falls back to the default query randomization info object, which randomizes range queries as was done before this PR. This is equivalent to registering registry.register_query_randomization_info(<operation_name>, "range", [["gte", "gt"], ["lte", "lt"]], ["format"]).
The dict returned by the random standard value source should match the parameter names you are trying to randomize. For example the standard value source for the above example is:
def bounding_box_source():
    top_longitude = random.uniform(-74.27, -73.68)
    top_latitude = random.uniform(40.49, 40.92)

    bottom_longitude = random.uniform(top_longitude, -73.68)
    bottom_latitude = random.uniform(40.49, top_latitude)

    return {
        "top_left":[top_longitude, top_latitude],
        "bottom_right":[bottom_longitude, bottom_latitude]
    }

Issues Resolved
https://github.com/opensearch-project/opensearch-benchmark/issues/711
Testing

[x] New functionality includes testing

Adds UTs. Also tested manually with a workload designed for testing the query cache, which uses a non-default QueryRandomizationInfo.

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.

@peteralfonsi Can you address the lint failures?


Error 500 when creating a pull request

cat templates/.VERSION
<IP_ADDRESS>7
I have just suddenly got an error when creating a pull request,
log:
[...routers/repo/pull.go:632 CompareAndPullRequestPost()] [E] NewPullRequest: testPatch: UpdateLocalCopy: Pull: exit status 128 - fatal: Not a git repository (or any of the parent directories): .git

Try delete path/to/gogs/data/tmp and try again.

the same with #2490

Same issue we too face any solution


Create API endpoint for executions

This issue should probably be broken down one we settle on an architecture.


community[minor]: Implement addGraphDocuments method for neo4j graph

Porting over add_graph_documents method from python.

LGTM

Really sorry for the delay - see above.

Thank you for your patience!


Some ideas about the framework

cache := hot.NewHotCache[string, *User](hot.LRU, 100_000).     WithLoaders(func(keys []string) (found map[string]*User, err error) {         rows, err := db.Query("SELECT * FROM users WHERE id IN (?)", keys)         // ...         return users, err     }).     Build()
above code is from your example, from the point of engineering, may be we can leverage “inceptor” to make this automatically when cache miss the key instead of manually writing the sql. Due the fact of the limited of go reflection, may be we can use “coding by contractor”. it will use the parameter of the DAO method as the key. In fact that’s what spring framework of Java(via annotation) doing.  Here is the contractor:
1: method must end with “Cachable”
2: the first parameter of the method should be a bool(saying cachable)
so the method is something like below
findUserByLastNameCachable(cachable bool, lastName string)
Of course this method is a method of DAO  struct
Below is the implementation
1: define another method accept dao method as parameter
2: in the method it will first check the method naming standard to check it ends with “Cachable” or not
3: Check the passing method’s first parameter is a bool or not
4: if it’s bool and the value is true then it will query the cache, if it find the value the return. Otherwise it will go to database directly and save the result to the cache for later usage

If you are ok with it, I would like to draft the propose.

I'm not a big fan of reflection.
We can think about a higher-order abstraction, but using reflection is a no-go for me.


Investigate how to integrate with Exeter's Kriya

I need to understand how Exeter's Kriya tool works and what my options for integration are so that I can make some technical choices.
DoD:

[x] Technical Specification document updated to include proposal on how integration with Kriya will work.

Note, that some of the finer detail is captured in the Exeter Kriya Integration document, but I extracted I believe the relevant information from that into the Technical Spec and made it more generic to cover possible approaches for any 3rd party to integrate.


Add tests for BatteryReceiverPlugin

Codecov Report

Merging #9 (d0fa5b9) into main (e0f979b) will increase coverage by 30.35%.
The diff coverage is 34.48%.

@@            Coverage Diff             @@
##            main       #9       +/-   ##
==========================================
+ Coverage   7.40%   37.76%   +30.35%
==========================================
  Files         15       15
  Lines       1094     1099        +5
==========================================
+ Hits          81      415      +334
+ Misses      1013      684      -329

Impacted Files
Coverage Δ

pykdeconnect/protocols.py
32.14% <0.00%> (+32.14%)
:arrow_up:

pykdeconnect/devices.py
41.66% <19.04%> (+41.66%)
:arrow_up:

pykdeconnect/payloads.py
93.10% <100.00%> (+93.10%)
:arrow_up:

pykdeconnect/plugins/battery.py
97.80% <100.00%> (+97.80%)
:arrow_up:

pykdeconnect/plugin_registry.py
33.78% <0.00%> (+33.78%)
:arrow_up:

pykdeconnect/helpers.py
59.52% <0.00%> (+59.52%)
:arrow_up:

pykdeconnect/plugin.py
80.00% <0.00%> (+80.00%)
:arrow_up:

pykdeconnect/exceptions.py
80.00% <0.00%> (+80.00%)
:arrow_up:

... and 3 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 e0f979b...d0fa5b9. Read the comment docs.


AssertionError: Results do not correspond to current coco set

Error info:
loading annotations into memory...
Done (t=0.01s)
creating index...
index created!
Loading and preparing results...
Traceback (most recent call last):
  File "/home/amax/mmdetection/tools/coco_eval.py", line 30, in <module>
    main()
  File "/home/amax/mmdetection/tools/coco_eval.py", line 26, in main
    coco_eval(args.result, args.types, args.ann, args.max_dets, args.classwise)
  File "/home/amax/mmdetection/mmdet/core/evaluation/coco_utils.py", line 41, in coco_eval
    coco_dets = coco.loadRes(result_file)
  File "/home/amax/anaconda3/envs/open-mmlab-sfy/lib/python3.7/site-packages/pycocotools-2.0.0-py3.7-linux-x86_64.egg/pycocotools/coco.py", line 317, in loadRes
    'Results do not correspond to current coco set'
AssertionError: Results do not correspond to current coco set

It happens when I tried to run coco_eval.py. And I'm aware that the error report indicates me the result doesn't fit coco format. But the result is absolutely and directly generated by mmdetection/tools/test.py.
The circumstance is, I used test.py to generate results of a Cascade R-CNN, and used coco_eval.py to evaluate the results, and nothing went wrong before.
This time I trained the same Cascade R-CNN config with another train set of data, the training seems normal, as well as the in-training validating.
Then I used test.py to generate a result, which is a list of json like:
{"image_id": 2973, "bbox": [0.07157446444034576,<PHONE_NUMBER>105469, 0.9284255355596542, 49.182525634765625], "score": 0.013011305592954159, "category_id": 4}

When running coco_eval.py, the error mentioned at first happens.
I don't think this format differs from coco's, but as you can see, the width of bbox is less than 1. I visualized the results, noticing that there are always some bbox with 0 width on the border of image.
Is the 0-width bbox resiponsible to the error? However after I filtered the 0-width bbox alway, the same error happens.

Replenish: I'm using the same validation set for in-training validating and later testing. While only testing goes wrong with this error.

Fine. It's just because the wrong path of annotation file in coco_eval.py parameters.


Annotate unused exports

Hi @Codex-  , I was able to enable knip-reporter to annotate also unused exports directly in the source files by extending buildArraySection.
See a demo screenshot here:

Please let me know if you'd be interested to pull it into upstream. I would clean it up a bit and open a PR then.

Sounds great, I've been meaning to add this so a PR to add this would be welcomed :)

All right! I will prepare and open a PR until Friday EOD

Currently being implemented, along with types and duplicates support. See #16

Currently being implemented, along with types and duplicates support. See #16

Great news! I will test and integrate this as soon as you merge.

Completed, releasd with v2.1.0. v2 tag has been updated to the latest v2 version :)


Business Central 15.2 Webclient Does not work

BCContainerHelper 2.0.9
Script:
function Get-WindowsTag
{
$OS = (Get-CimInstance Win32_OperatingSystem)
$UBR = (Get-ItemProperty 'HKLM:\SOFTWARE\Microsoft\Windows NT\CurrentVersion' -Name UBR).UBR
$HostOSVersion = [System.Version]::Parse("$($OS.Version).$UBR")
return "$($HostOSVersion.Major).$($HostOSVersion.Minor).$($HostOSVersion.Build).$($HostOSVersion.Revision)"
}
#OnPrem
$Country = 'W1'
$GenericImage = Get-BestGenericImageName
$BCArtifactUrl = Get-BCArtifactUrl -type OnPrem -version 15.2 -select Latest -country $Country
if ($BCArtifactUrl.StartsWith('https://bcartifacts.azureedge.net')) {
$BCTag = $BCArtifactUrl.Substring(41)
$BCTag = $BCTag.Substring(0,($BCTag.Length - 3))
} else {
#Alternative url
}
$ImageName = "businesscentral/onprem:$($BCTag)-$($Country)-$(Get-WindowsTag)"
$LicenseFile = "C:\Users\Van Lyssebettens\Documents\Licenses\SONJA_7400987_PARTNER_BC15_2021 04 01.flf"
New-BcImage -artifactUrl $BCArtifactUrl -imageName $ImageName -baseImage $GenericImage -licenseFile $LicenseFile `
-includeTestToolkit -includeTestLibrariesOnly
New-BcContainer -accept_eula -accept_outdated -containerName BC150 -imageName $ImageName -auth NavUserPassword -updateHosts
Output:
PS C:\WINDOWS\system32> C:\Users\Van Lyssebettens\Documents\Powershell\CreateDockerContainer.ps1
businesscentral/onprem:15.2.39040.0-w1-10.0.18363.1377
Building image businesscentral/onprem:15.2.39040.0-w1-10.0.18363.1377 based on mcr.microsoft.com/businesscentral:10.0.18363.1377 with https://bcartifacts.azureedge.net/onprem/15.2.39040.0/w1
Pulling latest image mcr.microsoft.com/businesscentral:10.0.18363.1377
10.0.18363.1377: Pulling from businesscentral
Generic Tag: <IP_ADDRESS>
Container OS Version: 10.0.18363.1377 (1909)
Host OS Version: 10.0.18363.1377 (1909)
Using process isolation
Using license file C:\Users\Van Lyssebettens\Documents\Licenses\SONJA_7400987_PARTNER_BC15_2021 04 01.flf
Files in c:\bcartifacts.cache\r3sosud1.q0v\my:
Copying Platform Artifacts
Copying Database
Copying Licensefile
c:\bcartifacts.cache\r3sosud1.q0v
Sending build context to Docker daemon  1.049GB
Step 1/6 : FROM mcr.microsoft.com/businesscentral:10.0.18363.1377
---> d7a4e1370ed2
Step 2/6 : ENV DatabaseServer=localhost DatabaseInstance=SQLEXPRESS DatabaseName=CRONUS IsBcSandbox=N artifactUrl=https://bcartifacts.azureedge.net/onprem/15.2.39040.0/w1 filesOnly=False
---> Running in 2f24c2a74dbe
Removing intermediate container 2f24c2a74dbe
---> e5da8c8eb344
Step 3/6 : COPY my /run/
---> ffc2007fd977
Step 4/6 : COPY NAVDVD /NAVDVD/
---> a45730c24914
Step 5/6 : RUN \Run\start.ps1 -installOnly -includeTestToolkit -includeTestLibrariesOnly
---> Running in 4c6aa59514c2
Using installer from C:\Run\150-new
Installing Business Central
Installing from DVD
Starting Local SQL Server
Starting Internet Information Server
Copying Service Tier Files
Copying PowerShell Scripts
Copying dependencies
Copying ReportBuilder
Importing PowerShell Modules
Determining Database Collation
Restoring CRONUS Demo Database
Setting CompatibilityLevel for CRONUS on localhost\SQLEXPRESS
Modifying Business Central Service Tier Config File for Docker
Creating Business Central Service Tier
Installing SIP crypto provider: 'C:\Windows\System32\NavSip.dll'
Copying Web Client Files
Copying Client Files
Copying ModernDev Files
Copying additional files
Copying ConfigurationPackages
Copying Test Assemblies
Copying Applications
Starting Business Central Service Tier
Importing CRONUS license file
Publishing C:\Applications\TestFramework\TestLibraries\Any\Microsoft_Any.app
Synchronizing Any
Installing Any
Publishing C:\Applications\TestFramework\TestLibraries\Assert\Microsoft_Library Assert.app
Synchronizing Library Assert
Installing Library Assert
Publishing C:\Applications\TestFramework\TestRunner\Microsoft_Test Runner.app
Synchronizing Test Runner
Installing Test Runner
Publishing C:\Applications\System Application\Test\Microsoft_System Application Test Library.app
Synchronizing System Application Test Library
Installing System Application Test Library
Publishing C:\Applications\BaseApp\Test\Microsoft_Tests-TestLibraries.app
Synchronizing Tests-TestLibraries
Installing Tests-TestLibraries
Stopping Business Central Service Tier
Installation took 95 seconds
Installation complete
Removing intermediate container 4c6aa59514c2
---> eabd6c5af053
Step 6/6 : LABEL legal="http://go.microsoft.com/fwlink/?LinkId=837447"       created="202104011343"       nav="1910"       cu="cu2"       country="w1"       version="15.2.39040.0"       platform="15.0.38951.0"
---> Running in 87cdaaee42e7
Removing intermediate container 87cdaaee42e7
---> 8a681311218c
Successfully built 8a681311218c
Successfully tagged businesscentral/onprem:15.2.39040.0-w1-10.0.18363.1377
Building image took 231 seconds
BcContainerHelper is version 2.0.9
BcContainerHelper is running as administrator
Host is Microsoft Windows 10 Enterprise - 1909
Docker Client Version is 19.03.5
Docker Server Version is 19.03.5
Removing BC150 from container hosts file
Removing BC150-* from container hosts file
Removing C:\ProgramData\BcContainerHelper\Extensions\BC150
Fetching all docker images
Fetching all docker volumes
Using image businesscentral/onprem:15.2.39040.0-w1-10.0.18363.1377
Creating Container BC150
Version: 15.2.39040.0-w1
Style: onprem
Multitenant: No
Platform: 15.0.38951.0
Generic Tag: <IP_ADDRESS>
Container OS Version: 10.0.18363.1377 (1909)
Host OS Version: 10.0.18363.1377 (1909)
Using process isolation
Using locale en-US
Disabling the standard eventlog dump to container log every 2 seconds (use -dumpEventLog to enable)
Files in C:\ProgramData\BcContainerHelper\Extensions\BC150\my:

AdditionalOutput.ps1
MainLoop.ps1
SetupVariables.ps1
updatehosts.ps1
Creating container BC150 from image businesscentral/onprem:15.2.39040.0-w1-10.0.18363.1377
ea0e629b851f05125fe10a5af4adc80fbefae3c438e3333e8d64f362ac1bf35c
Waiting for container BC150 to be ready

Creating Self Signed Certificate
Self Signed Certificate Thumbprint 250F80073976A411F23992A96596B3A22BC99581
Modifying Service Tier Config File with Instance Specific Settings
Starting Service Tier
Registering event sources
Creating DotNetCore Web Server Instance
Creating http download site
Setting SA Password and enabling SA
Creating admin as SQL User and add to sysadmin
Creating SUPER user
Container IP Address: <IP_ADDRESS>
Container Hostname  : BC150
Container Dns Name  : BC150
Web Client          : http://BC150/BC/
Dev. Server         : http://BC150
Dev. ServerInstance : BC
Setting BC150 to <IP_ADDRESS> in host hosts file
Files:
http://BC150:8080/ALLanguage.vsix
Container Total Physical Memory is 31.6Gb
Container Free Physical Memory is 18.0Gb
Initialization took 128 seconds
Ready for connections!
Reading CustomSettings.config from BC150
Creating Desktop Shortcuts for BC150
Container BC150 successfully created
Use:
Get-BcContainerEventLog -containerName BC150 to retrieve a snapshot of the event log from the container
Get-BcContainerDebugInfo -containerName BC150 to get debug information about the container
Enter-BcContainer -containerName BC150 to open a PowerShell prompt inside the container
Remove-BcContainer -containerName BC150 to remove the container again
docker logs BC150 to retrieve information about URL's again
Everything is working except when you go to http://BC150/BC and try to login, you always stays on  the loginwindow
No error, nothing in the eventviewer of the docker.
I don't have any issues when I create a 15.17or 17.6 container. Something with the artifacts? Cashed images on my buildservers are still working. This is only on my local machine where I did a cleanup of my artifacts & cached images.

You could try to flush the artifacts by using flush-containerhelpercache
BTW - all this code:
function Get-WindowsTag
{
$OS = (Get-CimInstance Win32_OperatingSystem)
$UBR = (Get-ItemProperty 'HKLM:\SOFTWARE\Microsoft\Windows NT\CurrentVersion' -Name UBR).UBR
$HostOSVersion = [System.Version]::Parse("$($OS.Version).$UBR")
return "$($HostOSVersion.Major).$($HostOSVersion.Minor).$($HostOSVersion.Build).$($HostOSVersion.Revision)"
}
$ImageName = "businesscentral/onprem:$($BCTag)-$($Country)-$(Get-WindowsTag)"
New-BcImage -artifactUrl $BCArtifactUrl -imageName $ImageName -baseImage $GenericImage -licenseFile $LicenseFile `
-includeTestToolkit -includeTestLibrariesOnly
New-BcContainer -accept_eula -accept_outdated -containerName BC150 -imageName $ImageName -auth NavUserPassword -updateHosts
Seems to be almost the same as just doing:
New-BcContainer -accept_eula -accept_outdated -containerName BC150 -imageName "bconprem" -auth NavUserPassword -updateHosts
New-BcContainer with an imagename like this will cache the image until there is a new generic or a new OS - then it will override the old.

On my buildservers, Get-WindowsTag =fixed value in the custom scripts. If a server runs on a different build, the final image (generic + artifact) must always have the same name, since that imagename is filled in in my Azure Devops Tasks

Every server can be used for the build pipeline, but not all the buildservers are updated on the same upgrade window.
Sow it can happen dat server has 2019 build X & the rest already build Y. Or maybe I'm making it myself to hard :)

Are you publishing the image to a registry and sharing between servers?

That would be the next step. Now I'm building once in a time all the needed images on every server

as long as things are on each server - the method I describe will work.
When publishing to a registry you might need what you do here,

My real issue is not with the script, only a BC 15.2 BE container won't let me log in in the webclient.
After entering the user & password, it stays on the login screen. ATM I'm trying with the 15.2 W1 artifacts.

This I ran, after stopping all my containers
Flush-ContainerHelperCache
$BCArtifactUrl = Get-BCArtifactUrl -type OnPrem -version 15.2 -select Latest -country 'W1'
New-BcContainer -accept_eula -accept_outdated -artifactUrl $BCArtifactUrl -containerName BC150 -imageName "bconprem" -auth NavUserPassword -updateHosts
result:

You see that I tried to logon to the instance. Only 15.2 gives me this issue.

I can repro this, but I think this is a result of a hardened security in browsers.
If you add
-useSSL -installCertificateOnHost
when you create the container, then the container will be created with a self-signed certificate and that certificate will be installed on the host, This made it work on my machine.


Add support for query "include"

This PR adds support for includes fields while query()ing. These new parameters control if what data is returned (and queried in the db).
All changes:

Add includes_[documents,embeddings,metadatas,distances] to query and pass it through to client.
Add logic in db to support controlling which columns are selected
Wire up client to support column selection
Tests for includes fields

Discussed offline w/ @jeffchuber

It is odd that an includes flag still includes other fields since we default the fields
We cannot change the default to False since it breaks backwards compatibility w/ integrations and users
We are going to switch to an explicit allowlist that defaults to all, if you modify it than you only get that field. Which is more intuitive behavior.


Add Profile Image Upload on Signup

Description

Add Profile Image Upload on Signup

@NikhilSharma03 I will create multer option for it!

@jinwoo1225 That would be great :rocket:. Thank you.


Fix server not returning error code

Signed-off-by: Tim Rozet<EMAIL_ADDRESS>

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.02%) to 72.838%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

server/database.go
0
1
0.0%

Files with Coverage Reduction
New Missed Lines
%

client/client.go
1
64.21%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
4202

Relevant Lines:
5769

💛  - Coveralls

actually no it won't because i've fixed it now. hopefully github is smart enough to avoid merge conflicts this time around


Sorting Morphs

Any ideas on how to use sorting on morphedbymany relations?

Hi! Have you tried using the same many-to-many sorting solution?

MorphedByMany should work the same as belongsToMany.

MorphedByMany should work the same as belongsToMany.


Added start_maximized option in config

I would recommend opening an issue with glutin assuming that with_maximized still doesn't work after the latest glutin 0.17 update (It's possible to already test this with the #1403 PR).
I'm not going to open one for now because I'd like to look into this a bit more to make sure it's not Alacritty's fault. But if someone else has tested this and can say for sure that it doesn't work properly with glutin 0.17, feel free to open an issue.

I'd be interested to know if .with_maximized(true) works on any platform with the latest version of Alacritty. I wasn't able to make it work with i3, but that's not a huge surprise.
Maybe @zacps can test this on Windows?
If this does not work anywhere, opening an issue upstream is probably the way to go for this PR.

Well... something happens:

(Frozen permanently after window creation)

That looks like it's just not acting upon a resize event. I feel like it might actually work and it's just an Alacritty issue?
If it does work, I don't see why we wouldn't be able to use with_maximized(true), however I'm not sure if it actually 'just works' on any platform. Might be good to test with the glutin examples rather than Alacritty, just to make sure it works in theory.

@jedahan Have you tested the with_maximized(true) version too? It seems like that's more likely to get included, since setting the window to be maximized right at the start after it has already been created is a bit unfortunate.

@chrisduerr trying that now

Closed in favor of https://github.com/jwilm/alacritty/pull/1823 which is rebased on top of master and fixes some problems discussed with this PR. Credit to the author of the commits made here is still included. Thanks for the PR @doubleloop.


Update aks-engine to v0.41.2

https://github.com/Azure/aks-engine/releases/tag/v0.41.2

/assign @mboersma


EXE export

Is your feature request related to a problem? Please describe.
Distribution of your GB Studio games outside of GB flashcarts and web platforms such as itchio, Kongregate, etc., selling on Steam in particular is problematic. First of all, you have to hunt down an emulator that is legal to use in commercial applications, secondly, you need to figure out a way so that your game's rom would start upon launch of such emulator. Both tasks are time consuming and cumbersome. Not to mention that most of the time you want also disable most of the emulator's UI, save maybe for keyboard/gamepad mapping to not confuse users with emulation-specific settings.
Describe the solution you'd like
I would like to be able to export games as windows/linux/macos executables, with the ROM embedded in a simplified emulator that is close to how the game behaves when launching in game test from GB Studio. This also would make easier to distribute the game to people that aren't necessarily emulator-savvy such as youtubers/streamers that may not want to play browser version for variety of reasons (issues with capturing entire game, not stable internet connection, etc.)

This is problematic as most small ways of bundling an emulator with a game in a small app will be detected as a virus.
The larger method (that gbstudio runs on) is packaging electron (chromium web browser) with the JavaScript emulator and rom used for preview and web view, this makes games easily 50-100mb, and often less near than a simple .exe. i don't think you can do that with a pre packed electron app either, so gbstudio might get way bigger...
Exporting stand alone apps for each rom from gbstudio might be out of scope for the small dev team.

Most small ways = not all small ways. Plus, there are engines that do just that and they're not detected as viruses. The Games Factory comes to mind...

Don't know of any gameboy emulators that are multi platform, easy to combine with a rom to make standalone, and not html5 based requiring a browser bundled.
Someone got close combining bgb and a rom with batch script, then turning the whole thing into an exe, but it was constantly flagged as a virus.
Bit short on developer experience to do that sadly.

Didn't someone rewrite the testing emulator for 3.0? If so, maybe that same person could do this?

The new testing emulator is the same as the new html5 export emulator, binjgb, it is web assembly based and must run in a browser.

Then all you need to do is to port it to a native language or a language that can crosscompile to both native and web such as haxe or Dart.

Then all you need to do is to port it to a native language or a language that can crosscompile to both native and web such as haxe or Dart.

This is an open source project so your feature additions are welcome!


Quick insights does not require two tables.

I was able to run Quick Insights against a single table (Adventureworks.Sales,SalesOrderHeader) without any issues. It does not seem to require a fact table AND a dimension table.

@eugman : Thanks for your contribution! The author, @mihart, has been notified to review your proposed change.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

powerbi-docs/service-insights-optimize.md
:white_check_mark:Succeeded

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

Hi @maggiesMSFT
I believe this one goes to you?

#sign-off

@maggiesMSFT: I'm sorry - only the author of this article, @mihart, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

Microsoft authors of articles need to make updates through the private repository so the updates can be staged and validated by the current validation rules. After you review staging and fix all validation issues, you can add the sign-off comment to let the PR reviewers know the updates are ready to be merged.It's easy to access the private repo version of your articles in GitHub. Just add -pr to the URL, as follows:
Public repo URL: https://github.com/Microsoft/azure-docs/blob/master/articles/azure-portal-overview.md
Private repo URL: https://github.com/Microsoft/azure-docs-pr/blob/master/articles/azure-portal-overview.md
We have closed this pull request. Please resubmit the updates through the private repo. If you need perms to the private repo, see the contributor guide instructions

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

powerbi-docs/service-insights-optimize.md
:white_check_mark:Succeeded

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

#sign-off -- signing off on an article that will be mine.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

powerbi-docs/service-insights-optimize.md
:white_check_mark:Succeeded

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


Making partial item updates to existing documents?

I can't find a method or service within the SDK to achieve the following steps? (What am I missing?)

Create a file container in S3
Upload to the file container
Push the fieldValueReplace to the source and document specified.

Basically, the code equivalent of:
{
  "partialUpdate": [
    {
      "documentId": "https://sapcommerce/product/p/40035",
      "operator": "fieldValueReplace",
      "field": "9991_9993",
      "value": "true"
    }
  ]
}

Unfortunately, the PlatformSDK does not support partial updates. For large files, the SDK takes care of requesting the S3 container.


Whatsdue fix 050519

Fixes !whatsdue and !ecp. They broke because you now need to supply a user agent when making a request to the site.
I also added some more error handling to stop it from hanging as often.
Closes #473 and closes #472

Jenkins is failing due to linting errors, please rectify :)
uqcsbot/utils/uq_course_utils.py:77:62: E252 missing whitespace around parameter equals
uqcsbot/utils/uq_course_utils.py:77:63: E252 missing whitespace around parameter equals
uqcsbot/utils/uq_course_utils.py:86:70: W291 trailing whitespace
uqcsbot/utils/uq_course_utils.py:87:80: W291 trailing whitespace


Removal of Redundant Logger Setup

PR Type:
Refactoring

PR Description:
This PR removes the unnecessary call to setup_logger() in the serverless.py file. The logger setup is already being handled within github_app.py, making the additional call in serverless.py redundant.

PR Main Files Walkthrough:
 files:

pr_agent/servers/serverless.py: Removed the import and call to setup_logger().

/describe

/review


Make admissionOperations backward compatible with previously deployed ModRules

Upgrading to v0.18.0 introduces admissionOperation, which is an optional ModRule attribute defaulting to ["CREATE", "UPDATE"].
The issue is that the default set of operations is only set on ModRule deployment, essentially deactivating previously deployed ModRules.

Hi @orbatschow,
FYI, please note the above issue, now resolved in v0.18.1.


Observation dictionary

Smarts environment was ensured to organically produce nested observations which are type checked. The observations only consists of List, Tuple, and NamedTuple data.

When a class object is pickled, both its data and accompanying methods are pickled. This causes pickling to slow down and to bloat.

Wherever possible, the following was adhered to in order to make the code scalable, flexible, and faster.

Data is decoupled from methods
Inheritance is avoided, except for implementation of interfaces

An obs_config.yaml file is introduced. If provided, the dynamic observation lengths will either be truncated or padded to produce the fixed sizes denoted in the obs_config.yaml file. If obs_config.yaml file is not provided, Smarts will return observations with varying lengths at different time steps.

Closes #548 .

Since this will break the observations, the merge will need to be deferred until 0.5.

@Adaickalavan Instead of splitting the methods out into a different class, did you consider using a pattern like:
@dataclass(frozen=True)
class Waypoint:
    pos: np.ndarray
    heading: Heading
    lane_width: float
    speed_limit: float
    lane_id: str
    lane_index: int

    def dist_to(self, p):
        """Calculates straight line distance to the given 2D point"""
        return np.linalg.norm(self.pos - vec_2d(p))

    def relative_heading(self, h: Heading):
        etc

    def serialize(self):
        return cloudpickle.dumps(self.__dict__)

    @classmethod
    def deserialize(cls, serialized_obj):
       return cls(**pickle.loads(serialized_obj))

(Note: I didn't test this -- it might require some tweaking to get it to work.  I'm just trying to show the basic idea here.)
And then in remote_agent.py, you would do:
    def act(self, obs):
        # Run task asynchronously and return a Future.
        self._act_future = self._worker_stub.act.future(
            worker_pb2.Observation(payload=obs.serialize())
        )

@Adaickalavan Instead of splitting the methods out into a different class, did you consider using a pattern like:
@dataclass(frozen=True)
class Waypoint:
    pos: np.ndarray
    heading: Heading
    lane_width: float
    speed_limit: float
    lane_id: str
    lane_index: int

    def dist_to(self, p):
        """Calculates straight line distance to the given 2D point"""
        return np.linalg.norm(self.pos - vec_2d(p))

    def relative_heading(self, h: Heading):
        etc

    def serialize(self):
        return cloudpickle.dumps(self.__dict__)

    @classmethod
    def deserialize(cls, serialized_obj):
       return cls(**pickle.loads(serialized_obj))

(Note: I didn't test this -- it might require some tweaking to get it to work. I'm just trying to show the basic idea here.)
And then in remote_agent.py, you would do:
    def act(self, obs):
        # Run task asynchronously and return a Future.
        self._act_future = self._worker_stub.act.future(
            worker_pb2.Observation(payload=obs.serialize())
        )

I haven't yet gone through this review entirely, so there may be good reasons to do it the other way. I just wanted to get the suggestion/question out now, because I might not be able to get through the review for a little bit.

If the above works, you might then create a "Serializable"  base class that contains the serialize and deserialize methods, and have any data object that needs to be sent to the remote agent inherit from that.

after coming back to this, I guess you will need to include the object type in the serialized dict as well such that you can determine what type of object to create when deserializing.  I'm guessing this was the reason you opted for the other pattern, as it does add a little ugliness to the pattern.  However, I wonder whether it might still be worth doing, just for the sake of avoiding changing all of the observation code.  (This PR would be much shorter at least!)  :)  Something to consider...

On the issue of splitting methods and data:
I think to achieve good code scalability and flexibility (inspired by GOlang coding philosophy)

keep data (in struct like objects) and methods/functions separate
more functional programming and less OOP style code
OOP style is more difficult to scale and to parallelize as we have to be careful of statefulness/side-effects

On the issue of inheriting from a “Serializable” base class:
I prefer to avoid it here, as

multiple inheritance does not work well for type NamedTuple, e.g., def Observation(Serializable, NamedTuple):
inheritance brings difficulty in hierarchy/dependency tracking (inspired by GOlang coding philosophy)

On the issue of including class type in the serialized data:
Conceptually, I think we should only send the observation data to the remote agent and we should not be bound to recreate the corresponding class methods inside the remote agent.

we should make no representation of the corresponding methods when we send the data to the remote agent.

(inline)

1. On the issue of splitting methods and data:
   I think to achieve good code scalability and flexibility (inspired by GOlang coding philosophy)

   * keep data (in struct like objects) and methods/functions separate
   * more functional programming and less OOP style code
   * OOP style is more difficult to scale and to parallelize as we have to be careful of statefulness/side-effects

Except this isn't the pattern we've been following so far, so it's a fairly big change and makes our codebase inconsistent to do it partially, and probably not necessary yet. (And while I disagree about Parallelization and Scaling in OOP, it's not worth debating here.)  For better and worse, SMARTS is squarely OOP.

2. On the issue of inheriting from a “Serializable” base class:
   I prefer to avoid it here, as

   * multiple inheritance does not work well for type `NamedTuple`, e.g., `def Observation(Serializable, NamedTuple):`
   * inheritance brings difficulty in hierarchy/dependency tracking (inspired by GOlang coding philosophy)

Yeah, I agree about avoiding multiple inheritance.  Of course, it's easy enough to achieve the same effect (while avoiding touching so much of the code) by adding something like a Serializer class (not a base class) with static serialize() and deserialize() methods.

3. On the issue of including class type in the serialized data:
   Conceptually, I think we should only send the observation data to the remote agent and we should not be bound to recreate the corresponding class methods inside the remote agent.
   * we should make no representation of the corresponding methods when we send the data to the remote agent.

cloudpickle is already sending the object types to the remote agent.  That's how it's able to unpickle things there.  And there's no need to send any other methods.  And we're not doing any sort of monkey patching of our objects as far as I know.
Anyway, I'd be happy to discuss further in any case, but I suspect the above questions may be obsoleted anyway by your other (proto) PR, so I'll check that out first.


Consider relicensing

Options around this are somewhat complex, relicensing requires I either remove contributions from those who don't agree, or get agreement.
Supporting the cogs has become a burden again because of people taking a permissive license, making modifications that could break the cog, and not doing the bare minimum to ensure that this does not become my problem.
To prevent any future problems, I may also need to add a CLA with signing. Currently being considered, will be discussing with a lawyer if I decide it is worth pursuing.
The issue serves as a notice that this is the intent and the reason behind it. Whatever license is decided on will leave this open to use, but no longer open to modification. If you have made a contribution and are not okay with this, feel free to comment in advance to discuss options, but relicensing to something less permissive will not happen without explicit consent or another handling from contributors.

My commits may be relicensed if the new license still allows me to use the code, let me know if this requires me to sign anything.
I was also curious how much other people have actually contributed to anything substantial. @PredaaA and @jack1142 and I are the only people to have contributed new functionality, the remainder of PRs people made are bug fixes and probably aren't protected as being a separate idea (still talk to the lawyer, I don't want you getting in trouble over this)

Until this is settled in any fashion, to keep things simple I'm not going to be accepting new PRs.

#227 has the form of license currently under consideration.

My commits may be relicensed if the new license still allows me to use the code

Same for me here. As said in Discord, I want to suggest (if possible) having a license that doesn't require developers to disclose source (i.e. like (A)GPL).

The new license was merged after more talk about this


[Docs] Replace image in README file

Current  #State:
See the white background between the man and woman in the image below that needs to be updated to a better version in the README file.

Desired State:
This is the desired image.

Replace with the image below.

Contributor Resources

Meshery documentation site
Meshery documentation source
Instructions for contributing to documentation

Can I work on this issue?

Sure @d-cryptic Go ahead!

I have initiated the Pull Request. Will you please look into it.


feature: support toolbox.dataZoom brush component `dataZoom.slide…

feature: support toolbox.dataZoom brush component dataZoom.slider visualMap.continuous drag outside and release outside.
That enhances the user experience of dragging.
zrender should be updated to the PR:
https://github.com/ecomfe/zrender/pull/536
Check test/drag-out.html.

This apparently caused issue https://github.com/apache/incubator-echarts/issues/12146

This apparently caused issue https://github.com/apache/incubator-echarts/issues/12146


s-casing exception

[ERRO] runtime: Caught exception in "GameLoop Update"
System.Collections.Generic.KeyNotFoundException: The given key 's-casing' was not present in the dictionary.
at System.Collections.Generic.Dictionary`2.get_Item(TKey key)
at Content.Client.ClickMapManager.IsOccluding(RSI rsi, StateId state, Direction dir, Int32 frame, Vector2i pos) in /var/lib/jenkins/workspace/SS14 Content/Content.Client/ClickMapManager.cs:line 73
at Content.Client.GameObjects.Components.ClickableComponent.CheckClick(Vector2 worldPos, Int32& drawDepth, UInt32& renderOrder) in /var/lib/jenkins/workspace/SS14 Content/Content.Client/GameObjects/Components/ClickableComponent.cs:line 110
at Content.Client.State.GameScreenBase.GetEntitiesUnderPosition(MapCoordinates coordinates) in /var/lib/jenkins/workspace/SS14 Content/Content.Client/State/GameScreenBase.cs:line 128
at Content.Client.State.GameScreenBase.FrameUpdate(FrameEventArgs e) in /var/lib/jenkins/workspace/SS14 Content/Content.Client/State/GameScreenBase.cs:line 63
at Robust.Client.GameController.Update(FrameEventArgs frameEventArgs) in /var/lib/jenkins/workspace/SS14 Content/RobustToolbox/Robust.Client/GameController.cs:line 296
at Robust.Shared.Timing.GameLoop.Run() in /var/lib/jenkins/workspace/SS14 Content/RobustToolbox/Robust.Shared/Timing/GameLoop.cs:line 242

The exception was fixed in b168a7926ff14e6acd9d806185b378aac7b808d6, the bug causing the broken sprite in 1fa960d5d6af827237e31a634df37f62f171f86b


Exercise config compilation

Symbolic PR, compilation is working, now all usable boxes have to be implemented.

Coverage decreased (-0.002%) to 74.277% when pulling a02522aa2333a24a459450a0f1c189139aba44c2 on exercise-config-compilation into a6845b548885f4ffb09e03221a79553b646c7e13 on master.


go.mod bump to go1.20

Update github actions.

Codecov Report

Merging #58 (389e8ce) into main (8b22020) will decrease coverage by 0.1%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##            main     #58     +/-   ##
=======================================
- Coverage   62.5%   62.5%   -0.1%
=======================================
  Files         64      64
  Lines       7337    7337
=======================================
- Hits        4589    4583      -6
- Misses      2596    2600      +4
- Partials     152     154      +2

see 1 file with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


EthBundleApiClient trait clashes with MevApiClient

It's a common use-case to want to send a private transaction in conjunction with a bundle, similar to the backrun example in the TS reference implementation.
However, importing both methods under the same name can lead to compiler errors, and attempting to differentiate them without causing confusion can be verbose.
Proposal 1: Split the API client
One way to address this is by breaking up the EthBundleApiClient into two separate clients:

EthTxApiClient: this client would exclusively handle individual transactions. It would encompass methods such as:

send_private_transaction
send_raw_private_transaction
cancel_private_transaction

EthBundleApiClient: this client would retain methods specifically designed for bundle operations, including:

send_bundle
sim_bundle
cancel_bundle

Proposal 2: Prefix the 'eth' methods
An alternative solution would be to prefix the eth methods, for instance, renaming send_bundle to eth_send_bundle. However, this approach might introduce a new expectation for the mev_* methods to follow a similar naming convention, like mev_send_bundle.
This inconsistency can be confusing for users. Therefore, Proposal 1 might be more intuitive.

Importing both leads to compiler error and using it without compiler error is verbose

do you mean when calling the function?
I recommend using the fully qualified syntax in this case

OK, sorry for live editing, I mistakenly submitted while still drafting. I should probably get into the habit of entering the title last.
Anyhow, using the fully qualified syntax is what I'm currently doing in the recently added example:
let tx_to_backrun_hash =
      mev_share_rpc_api::EthBundleApiClient::send_private_transaction(&client, private_tx)
           .await?;

this could be:
let tx_to_backrun_hash = client.send_private_transaction(&private_tx).await?;

Seems worthwhile? Will draft PR.

sgtm!


null check before doing anything with api helper (could be null when …

…switching quickly between watch faces)

This one should be merged asap


config."opensearch.yml" value should be string type

Currently the default values.yaml uses a dict type for config."opensearch.yml". Any custom values defined under this key are merged with the default values. This means you need to carefully inspect the default values in order to know how custom values will be merged with this field and what the final applied values will be.
Instead if this field was a string type it would simply be replaced by any custom value instead of merged. This seems much safer and easier to use. It is also how securityConfig.config.data.* values work.
Any problem with me submitting a PR to fix this?

@smlx This was done purposely and kept as dict type to avoid mistakes while assigning values. @DandyDeveloper opinions?

@smlx This is intentional because we want the config to be read as a YAML to make it easier for all kinds of different reasons.
For example, maybe you have a test cluster, so you could define an additional test values with some minor adjustments more indicative of your smaller cluster or some different self-signed certificates for transport comms. Rather than having multiple copes of the same config.
A number of CI/CD tools would also rely on YAML paths to find certain values which can be used for a number of additional benefits. You wouldn't be able to read or set specific fields in the helm upgrade command as well, without defining the entire config.
Its consistent with how all other values are managed in helm.
I actually think the default in the values.yaml can be removed entirely because AFAIK its just a dupe of the config that is already baked into the OpenSearch image.
I do understand the justification for the other end of this. You'll probably find arguments on both sides for one or the other. My preference is to keep it how things currently are, but I'm just one voice.

Thanks for the explanations. So are you saying that those default values align with the configuration baked into the images already? In other words, should you be manually overriding each of those specific configuration options anyway for a production deployment, even if they weren't set in the chart's default values.yaml?

@smlx If I remember correctly, the demo opensearch.yml baked into the image is basically the exact same as what we include in the values.yaml.
So IMO we can remove this entirely / comment it out.
But yes, to me, all those available option should be replaced by your own input anyway, because those demo certs are available publicly. That CAN put your cluster at risk in transport/http. Those are most of the lines in that config.
The rest:
        audit.type: internal_opensearch
        enable_snapshot_restore_privilege: true
        check_snapshot_restore_write_privileges: true
        restapi:
          roles_enabled: ["all_access", "security_rest_api_access"]
        system_indices:
          enabled: true
          indices: [".opendistro-alerting-config", ".opendistro-alerting-alert*", ".opendistro-anomaly-results*", ".opendistro-anomaly-detector*", ".opendistro-anomaly-checkpoints", ".opendistro-anomaly-detection-state", ".opendistro-reports-*", ".opendistro-notifications-*", ".opendistro-notebooks", ".opendistro-asynchronous-search-response*"]

IMO are effectively mandatory anyway, except maybe enable_snapshot_restore_privilege and check_snapshot_restore_write_privileges which can just be disabled by a user easily through their own values.

@smlx if you are okay with the answer can we close this?

Hi, yes thanks please close this :+1:


Update index.html

@ujjwalgupta9125  is it cool if you can tell avinash if he can update mine then he can update yours I have put some significant changes to the code.

@ujjwalgupta9125 is it cool if you can tell avinash if he can update mine then he can update yours I have put some significant changes to the code.

How can I tell to avinash?

@ujjwalgupta9125 is it cool if you can tell avinash if he can update mine then he can update yours I have put some significant changes to the code.

How can I tell to avinash?

All you need to do is press @ then enter avinash and GitHub will tell you his GitHub name this means you are pinging him and tell him to update mine first srry for the discomfort in advance bro

I think he will do whenever he gets time. Isn't it?
On Sun, Oct 3, 2021, 11:44 AM Mohamed Ayaan @.***>
wrote:

@ujjwalgupta9125 https://github.com/ujjwalgupta9125 is it cool if you
can tell avinash if he can update mine then he can update yours I have put
some significant changes to the code.
How can I tell to avinash?
All you need to do is press @ then enter avinash and GitHub will tell you
his GitHub name this means you are pinging him and tell him to update mine
first srry for the discomfort in advance bro
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/avinash201199/stopwatch/pull/19#issuecomment-932871271,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AODWWGBF2PO35JU2RNJJXUTUE7YGBANCNFSM5FHGZ4UQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Sorry to say, one word changes are not accepted, officials of hacktoberfest will review every repo and pr so contribute genuine


sbt 1.0.3

Codecov Report

Merging #10 into master will decrease coverage by 1.92%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #10      +/-   ##
==========================================
- Coverage   85.67%   83.74%   -1.93%
==========================================
  Files          16       16
  Lines         363      363
  Branches       32       34       +2
==========================================
- Hits          311      304       -7
- Misses         52       59       +7

Impacted Files
Coverage Δ

.../workday/prometheus/akka/ForkJoinPoolMetrics.scala
81.08% <0%> (-18.92%)
:arrow_down:

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 f46b236...060f715. Read the comment docs.


Placeholder entry point for parent POM pipelines.

This PR adds a pipeline that does nothing so that I can generate the parent POM pipeline infrastructure using the pipeline generator. Another PR (incoming soon) will add the necessary changes to make this do something useful.

/cc @Azure/azure-sdk-eng


TopologyProposal usage

After a mention on #83 , I thought I would start a discussion here on potentially using TopologyProposal as a more general sampler-state container.

TopologyProposal contains a paired set of old and new sampler states.  What if you instead had a SamplerState and TopologyProposal contained an old and new sampler state?

Going to close this for now to avoid issue clutter. It can be reopened later if necessary.


remove event_prefix from handler identifiers

[x] fix compile
[x] fix plug dependency
[ ] fix tests

The test is failing because your handle_route/4 is failing because the span context is :undefined because your handle_start/4 wasn't called because nobody sent a [:plug_adapter, :call, :start] because there's no adapter present.

Ok, I'm going to need to loop around and submit a PR for Plug.Test because it doesn't call into telemetry.execute/3 as required by [reads notes] Plug.Conn.Adapter.

I can make the test pass by calling :telemetry.execute/4 before MyRouter.call/2, but any users who don't run around doing the same to their tests will get a noisy error as our telemetry handler disconnects:
12:41:18.908 [error] Handler {OpentelemetryPlug, :plug_tracer_router_start} has failed and has been detached. Class=:error
Reason=:function_clause
Stacktrace = [
  {:ot_span_ets, :update_name, [:undefined, "/hello/:foo/*glob"],
   [file: '$HOME/opentelemetry_plug/deps/opentelemetry/src/ot_span_ets.erl', line: 131]},
  {:telemetry, :"-execute/3-fun-0-", 4,
   [file: '$HOME/opentelemetry_plug/deps/telemetry/src/telemetry.erl', line: 123]},
  {:lists, :foreach, 2, [file: 'lists.erl', line: 1338]},
  {MyRouter, :dispatch, 2, [file: 'lib/plug/router.ex', line: 277]},
  {MyRouter, :plug_builder_call, 2, [file: 'test/opentelemetry_plug_test.exs', line: 39]},
  {OpentelemetryPlugTest, :"test creates span", 1,
   [file: 'test/opentelemetry_plug_test.exs', line: 9]},
  {ExUnit.Runner, :exec_test, 1, [file: 'lib/ex_unit/runner.ex', line: 355]},
  {:timer, :tc, 1, [file: 'timer.erl', line: 166]},
  {ExUnit.Runner, :"-spawn_test_monitor/4-fun-1-", 4, [file: 'lib/ex_unit/runner.ex', line: 306]}
]

We can compensate for that, having handle_route/4 call handle_start/4 if the context is :undefined, but there's no similarly easy way to get handle_stop/4 called with a finished conn, which brings us back to trying to write advice for our users on how to write a test that'll prove their OT integration works. What do you reckon? This getting messy enough you'd ponder switching back to @behaviour Plug for integration?

Reckon I'll apply José's hint from elixir-plug/plug#945 and take a test-only dependency on plug_cowboy, and have handle_route and handle_stop no-op if the span context is :undefined.

Incorporates #1 for one-click merge convenience.


Publish micro site is broken

e.g. https://github.com/optics-dev/Monocle/runs/713390144
GitHub returned an error: Failed invoking with status : 401 body :
 {"message":"Bad credentials","documentation_url":"https://developer.github.com/v3"}
	at sbtorgpolicies.github.syntax$GHResponseOps.execE(syntax.scala:53)

fixed


feat: use shared folder

Moves the validations from /app/utils/ to the new shared folder (and also updates Nuxt to support this new folder).
Closes #33

Thank you!


Option to use EffectTS for errors

Broken out of #18
Pinging @Suyashtnt as they showed interest.

If I start working on glowsquid again sometime soon I'll see if I can help on this issue since I do plan to use effect

Most of the work for this has been done on #120 so it would be really easy to implement if anyone see this and wants to give it a stab.


update jsfm to 0.19.6,contains strict model

          4 Errors

:no_entry_sign:
Objective-C file ‘ios/sdk/WeexSDK/dependency/SRWebSocket.m’ does not have the copyright header.

:no_entry_sign:
Objective-C file ‘ios/sdk/WeexSDKTests/WXFloatCompareTests.m’ does not have the copyright header.

:no_entry_sign:
Objective-C file ‘ios/sdk/WeexSDKTests/WXRootViewTests.m’ does not have the copyright header.

:no_entry_sign:
Objective-C file ‘ios/sdk/WeexSDKTests/WXStorageTests.m’ does not have the copyright header.

          1 Warning

:warning:
Big PR

  Generated by :no_entry_sign: danger


Question: Can this be used with existing jupyter notebooks?

Hi there. This project is awesome!
I understand the workflow of using python -m jupyter_ascending.scripts.make_pair --base <fname> to create a pair of synced jupyter files, and then starting the jupyter notebook. But I'm wondering, if I have a long notebook.ipynb file, is there a script to turn that into compatible .sync.py file? Thanks.

hey!
yes, you can use:
jupytext --to py:percent <file_name>

and then add the .sync suffix to both files

Thanks!


Upsample support

I would like to request the addition of native support for the Upsample operation in MLX. Currently, the absence of Upsample functionality limits the flexibility of certain tasks that require resizing or upsampling of data (like UNet definition).
In the meanwhile, does someone have alternative methods for emulating the upsample functionality?
Thank you!

See the upsample_nearest function the Stable Diffusion example:
def upsample_nearest(x, scale: int = 2):
    B, H, W, C = x.shape
    x = mx.broadcast_to(x[:, :, None, :, None, :], (B, H, scale, W, scale, C))
    x = x.reshape(B, H * scale, W * scale, C)

    return x

Thank you very much! If you do not plan to implement a standalone class, it is possible for me to close the issue.


npm install of 3.19.0 failing

With the recent release, I am not currently able to install curlconverter:
➜  testtest git:(master) ✗ npm -v
7.24.0
➜  testtest git:(master) ✗ node -v
v16.10.0
➜  testtest git:(master) ✗ npm install --save-dev curlconverter
npm ERR! code 1
npm ERR! git dep preparation failed
npm ERR! command /usr/local/Cellar/node/16.10.0/bin/node /usr/local/lib/node_modules/npm/bin/npm-cli.js install --force --cache=/Users/omitted/.npm --prefer-offline=false --prefer-online=false --offline=false --no-progress --no-save --no-audit --include=dev --include=peer --include=optional --no-package-lock-only --no-dry-run
npm ERR! ><EMAIL_ADDRESS>prepare
npm ERR! > npm run compile
npm ERR!
npm ERR!
npm ERR! ><EMAIL_ADDRESS>precompile
npm ERR! > rimraf build
npm ERR!
npm ERR!
npm ERR! ><EMAIL_ADDRESS>compile
npm ERR! > tsc
npm ERR!
npm ERR! lib/yargs-parser.ts:737:17 - error TS2571: Object is of type 'unknown'.
npm ERR!
npm ERR! 737             if (ex.name === 'PermissionDenied') error = ex
npm ERR!                     ~~
npm ERR!
npm ERR! lib/yargs-parser.ts:737:49 - error [... etc ... etc]

The recent releases (with the exception of 4.0.0-alpha.2, which I fixed after a day or two) have been pre-releases. When you npm install curlconverter you're installing  3.19.0. If you want to install the recent release, you have to do
npm install curlconverter@next

But this has something to do with
https://github.com/NickCarneiro/curlconverter/blob/d8c131a4e11b09ce57642bbff4efeb694461ac92/package-lock.json#L17
@NickCarneiro if you rebase that repo and https://github.com/NickCarneiro/yargs-parser-ansi when you get the chance to their upstreams it might solve it. Also, this time around you can skip adding my yargs-parser changes because they don't actually add ANSI-C strings to yargs (as we discussed). idonava's changes are the only changes actually doing anything.

npm install curlconverter@next
npm install --save-dev<EMAIL_ADDRESS>and even just: npm install --save-dev tree-sitter
all failing with same error:
npm ERR! code 1
npm ERR! path /Users/connor/code/testtest/node_modules/tree-sitter
npm ERR! command failed
npm ERR! command sh -c prebuild-install || node-gyp rebuild
npm ERR! CC(target) Release/obj.target/tree_sitter/vendor/tree-sitter/lib/src/lib.o
npm ERR!   LIBTOOL-STATIC Release/tree_sitter.a
npm ERR!   CXX(target) Release/obj.target/tree_sitter_runtime_binding/src/binding.o
npm ERR! gyp info it worked if it ends with ok
npm ERR! gyp info using<EMAIL_ADDRESS>npm ERR! gyp info using<EMAIL_ADDRESS>| darwin | x64
npm ERR! gyp info find Python using Python version 3.8.2 found at [... etc ... etc]

I wonder if this could have anything to do with me using zsh instead of bash...

@cf512 no, you're running into https://github.com/tree-sitter/node-tree-sitter/issues/82 . tree-sitter is currently broken on Node 16. The fix has been merged, it just needs to be released.

@verhovsky
Thanks for the answer.
Sorry, I do not see official contributors.

@dmvict I just published 4.0.0-alpha.7 which should have the fix.
Just to be clear, this is unrelated to the issue described in the first comment though. This only applies if you're installing curlconverter with npm install curlconverter@next because curlconverter 4 is still in development. curlconverter 3.19.0 (which is what npm tries to install when you don't include @next) is still broken.

@verhovsky Yes, it works now. Many thanks.


cGuard protocol decoding issue

I've got a pair of cGuard Atom devices,  one of the them is '2015' and another one - '2016'. I'm trying to install and configure the Traccar server (inside Docker) to acquire data from the devices.
Traccar version: 3.10,
cGuard Atom fw: 3.2.3 (the latest available)
The server successfully detects a new device (2017-04-09 03:06:43  WARN: Unknown device -<PHONE_NUMBER>0**** (***)) and after a couple of minutes begins to decode gps-data. However, the navi data seem to be being decoded incorrectly. All the testing time, coordinates are zeroes, the datetime mark correspond to Unix epoch time, etc.. (see below).
2017-04-09 03:14:57 DEBUG: [04509DA7: 5123 < <IP_ADDRESS>] HEX: 4e563a313730343039203033313435363a35362e3830383535333a36302e3539353437363a303a4e414e3a300a42433a313730343039203033313435363a435351313a36343a4e5351313a31373a505752313a300a
2017-04-09 03:14:57  INFO: [04509DA7] id:<PHONE_NUMBER>09961, time: 1970-01-01 00:00:00, lat: 0.00000, lon: 0.00000, speed: 0.0, course: 0.0
2017-04-09 03:16:57 DEBUG: [04509DA7: 5123 < <IP_ADDRESS>] HEX: 4e563a313730343039203033313635363a35362e3830383535333a36302e3539353437363a303a4e414e3a300a42433a313730343039203033313635363a435351313a36343a4e5351313a31363a505752313a300a

The hex-decoder on the Trackar site correctly decodes HEX-parts of the data:
4e563a313730343039203033313035363a35362e3830383535333a36302e3539353437363a303a4e414e3a300a42433a313730343039203033313035363a435351313a35383a4e5351313a373a4e5351323a313a505752313a300a

NV:170409 031056:56.123456:60.123456:0:NAN:0
BC:170409 031056:CSQ1:58:NSQ1:7:NSQ2:1:PWR1:0

I have fixed the problem.