Polymer 1.x shared-styles are working in Chrome but not in Firefox

I have this in shared-styles.html:
<dom-module id="shared-styles">
  <template>
    <style>
    ...
    </style>
  </template>
</dom-module>

And then I use it like this:
<dom-module id="foo-view">
  <template>
    <style include="shared-styles">
      ...
    </style>
  </template>
  <script src="view.js"></script>
</dom-module>

In Chrome my shared styles work fine but in Firefox they seem to be completely ignored.
Why?
I'd really appreciate any help on this.
(Btw, I already tried googling and asking on Slack but found no answers.)

{
  "name": "foobar",
  "private": true,
  "dependencies": {
    "font-roboto": "StartPolymer/font-empty#1.1.0",
    "iron-flex-layout": "PolymerElements/iron-flex-layout#1.2.3",
    "iron-localstorage": "PolymerElements/iron-localstorage#1.0.5",
    "iron-pages": "PolymerElements/iron-pages#1.0.7",
    "iron-selector": "PolymerElements/iron-selector#1.2.2",
    "page": "visionmedia/page.js#1.6.4",
    "paper-drawer-panel": "PolymerElements/paper-drawer-panel#1.0.6",
    "paper-icon-button": "PolymerElements/paper-icon-button#^1.1.6",
    "paper-input": "PolymerElements/paper-input#1.1.6",
    "paper-item": "PolymerElements/paper-item#1.1.3",
    "paper-material": "PolymerElements/paper-material#1.0.6",
    "paper-menu": "PolymerElements/paper-menu#1.2.2",
    "paper-scroll-header-panel": "PolymerElements/paper-scroll-header-panel#1.0.15",
    "paper-styles": "PolymerElements/paper-styles#1.1.3",
    "paper-swatch-picker": "PolymerElements/paper-swatch-picker#master",
    "paper-toast": "PolymerElements/paper-toast#1.1.3",
    "paper-toggle-button": "PolymerElements/paper-toggle-button#1.1.1",
    "paper-toolbar": "PolymerElements/paper-toolbar#1.1.4",
    "platinum-sw": "PolymerElements/platinum-sw#1.2.3",
    "polymer": "Polymer/polymer#1.3.0",
    "start-google-analytics-tracker": "StartPolymer/start-google-analytics-tracker#1.1.0",
    "geo-location": "ebidel/geo-location#^1.0.6",
    "google-map": "GoogleWebComponents/google-map#^1.2.0",
    "paper-input-location": "Koada-os/paper-input-location#^2.0.0",
    "paper-button": "PolymerElements/paper-button#^1.0.15",
    "iron-ajax": "PolymerElements/iron-ajax#^1.4.4",
    "iron-form": "PolymerElements/iron-form#^1.1.4",
    "paper-dropdown-menu": "PolymerElements/paper-dropdown-menu#^1.5.1",
    "iron-dropdown": "PolymerElements/iron-dropdown#^1.5.6",
    "iron-image": "PolymerElements/iron-image#^1.2.6",
    "paper-checkbox": "PolymerElements/paper-checkbox#^2.0.0"
  },
  "devDependencies": {
    "web-component-tester": "4.2.0"
  },
  "ignore": [],
  "resolutions": {
    "polymer": "^1.5.0",
    "iron-selector": "^1.5.2",
    "paper-input": "^1.1.23",
    "paper-styles": "^1.1.4",
    "paper-icon-button": "^1.1.6"
  }
}

Could you make sure you are loading the latest version of Polymer (v1.10.1 https://github.com/Polymer/polymer/releases/tag/v1.10.1)? Your bower.json references 1.5.0 instead

How can I safely update one dependency so that it also works for all other dependencies?

You should be able to update Polymer to 1.10.1 without any issues as it is fully backwards-compatible.

I did bower update -S but it didn't update polymer.
This is the bower.json file afterwards.
It seems to have just added webcomponentsjs..
Does that mean no newer version was compatible with all deps?

Remove Polymer from the resolutions field and try again. That should force it to update

I did that and it asked me to choose a polymer version, I chose the highest one that resolves to 1.10.1 but it only updated to 1.7.1, why?
And why didn't it update any of the other deps?

It's still not working in FF, any idea why?

I have encountered this issue myself, but I have to get a solid reproduction case first. Could you create a JSBin that shows the issue?

Thanks for the issue.  Please see CONTRIBUTING.md for guidelines on filing issues.  Please fill out as many fields in the issue template as possible, and be sure to include a live reproduction so that the team can debug or understand your issue.  You can start from this jsBin template as a starting point and link it in the issue template under "Live Demo".
Feel free to file a new issue following these instructions.  Thank you!


Keep No Maximum limit

Sometimes there could be a limit but not for all objects like soldering reels , lights or probably remove mathf.clamp completely or make it a abysmally big number

That would create potential lag when there are too many objects existing in the game world. I could add a toggle to increase the maximum limit to the biggest number the game can handle, but it'll have be an optional setting.

yeah that would be great


Added package name

According to PyPI doc (https://packaging.python.org/tutorials/packaging-projects/)

Why would anyone add name = "package name" when you can just call __name__ off an import?
>>> bitcash.__name__
'bitcash'

Even if I import bitcash as foo, name is still bitcash.

Indeed, I agree this is not really useful.
I added this line following the official PyPI documentation (available here).
It states:

You should also edit example_pkg/init.py and put the following code in there:
name = "example_pkg"
This is just so that you can verify that it installed correctly later in this tutorial and is not used by PyPI.

And then:

import the module and print out the name property. This should be the same regardless of what you name you gave your distribution package in setup.py (in this case, example-pkg-your-username) because your import package is example_pkg.

This is what I've followed to deploy various apps to PyPI so far, but apparently this is not necessary and major packages don't seem to use it.
You may close this PR ☺

Ok, thank you for letting me know :).


List investigations OData options not complete

The OData's $filter query is also supported on the "id" field; however, that is not listed. Additionally, $top and $skip are supported but not listed.

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

ID: 3a3e1789-673f-3c48-0663-9f2ae8e2380c
Version Independent ID: 8c5c0bd0-2a2c-c167-199a-11a5584e8803
Content: List Investigations API
Content Source: microsoft-365/security/defender-endpoint/get-investigation-collection.md
Product: m365-security
GitHub Login: @mjcaparas
Microsoft Alias: macapara

@MastaDisasta Thank you for your feedback. We have made changes to the document via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/5978
Thanks
Sri

@msbemba Thank you very much for the contribution and updating the documentation with PR. @MastaDisasta Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Add builtin user account creds

I had a need to impersonate some builtin accounts like Network Service. Unfortunately all the constructors in UserCredentials prevented me from passing an empty password. So I created a private constructor and just used static properties to create builtin accounts. Seems to work well!
EDIT: Oh.... I just now saw the other pull request... No worries if you want to close this one.

I think I like this one better.  Thanks!


dsn_patch_5

dsn_class v.0.2.9 - fix 1:

Added Licorice_Black to the color lookup for load_levels

dsn_levels v.0.3.7 - fix 1:

Fixed a spelling mistake in a comment and fully fixed jumping. I forgot to change the timing for held jumps from 200 to 150 milliseconds.
The bug was related to jump boost where on respawn, the player square would still be jumping. The fix was that the jump boost wasn't reset to it's default value on death, causing the jump to continue when the player respawned.

main v.0.2.3 - fix 1:

Fixed the window caption where I accidentally added the file path to it's name 🤣

everything seems good for now, will keep you updated on any issues and testing


Add php build example

Signed-off-by: Richard Gee<EMAIL_ADDRESS>

Could you change the examples to use ttl.sh? It seems like it's less confusing as I saw a user try to publish to the docker.io/functions account - not realising that it wasn't their own login.


Registering and Displaying Errors

In a general dev cycle, you are likely to run into errors from one or more fuse plugins.  Whether it be a scss compilation error, TypeScript type error, or any other place where a developer can make a mistake.  The general strategy right now seems to be to log these errors directly to the console. Take SassPlugin for example:
https://github.com/fuse-box/fuse-box/blob/ea625759a6e7069197e75417f96677def335f1af/src/plugins/stylesheet/SassPlugin.ts#L103
I would like to propose an internal mechanism for plugins to register errors with fuse-box.  The errors could then be reported in the following places:

In the fuse-box output at the end of the output for a bundle, rather than at the top before the file list
In the result object that you get after bundling.  This could be used in CI if your production bundle has some errors in it.
In the console of the browser if using HMR (would be really useful if you don't often look at the fuse-box output in your terminal)
As some sort of overlay in the DOM (Nice to have in case you don't have terminal or browser console open)

Finally, it would be really nice if code outside of fuse-box could register errors as well.  Namely, if I could hook up fuse-box-typechecker and have it automagically report type errors in my browser, that would be amazing.

Thanks, @dgreif that would be great to have.
There is already an attempt like this:
https://github.com/fuse-box/fuse-box/blob/ea625759a6e7069197e75417f96677def335f1af/src/core/BundleProducer.ts#L69-L78
But there are a lot of places where it's just console.log. I absolutely agree with you, having a class where all the errors will be dumped is just great.
The best for such things is in BundleProducer
https://github.com/fuse-box/fuse-box/blob/ea625759a6e7069197e75417f96677def335f1af/src/core/BundleProducer.ts
Everything and everyone can access this object during the bundle time.
We could start sending errors from plugins at first, do you think @dgreif you could start implementing it?

@nchanged I'll start looking into it.  Thanks for being open to the idea!

See https://github.com/fuse-box/fuse-box/pull/718 for the first step.

@dgreif awesome!!
https://github.com/fuse-box/fuse-box/tree/error-logging

@dgreif are you going to continue improving it?

@nchanged sorry for the delay. I'll have another PR early next week adding styles and minimizing in the browser.

Let's close this one now

I am using TypeScript because I want compiler to find certain errors that I make as early as possible. Why would this not be enabled by default somehow? And what do I need to do to get those?


CircleCI doc downloading and showing fix

https://github.com/braindecode/braindecode/blob/8b2770a0c632ff267ede090ea93dca42d92a247b/.github/workflows/push-artifact-to-circleci.yml#L6
This should be changed to "docs" I guess, cannot do easily from train, can you try @bruAristimunha ? Otherwise I do tomorrow
That's why we cannot see docs in PRs anymore

sure @robintibor :)

Btw, you can still see the doc, I'll show you tomorrow how I was doing it.


[SPARK-17491] Close serialization stream to fix wrong answer bug in putIteratorAsBytes()

What changes were proposed in this pull request?
MemoryStore.putIteratorAsBytes() may silently lose values when used with KryoSerializer because it does not properly close the serialization stream before attempting to deserialize the already-serialized values, which may cause values buffered in Kryo's internal buffers to not be read.
This is the root cause behind a user-reported "wrong answer" bug in PySpark caching reported by @bennoleslie on the Spark user mailing list in a thread titled "pyspark persist MEMORY_ONLY vs MEMORY_AND_DISK". Due to Spark 2.0's automatic use of KryoSerializer for "safe" types (such as byte arrays, primitives, etc.) this misuse of serializers manifested itself as silent data corruption rather than a StreamCorrupted error (which you might get from JavaSerializer).
The minimal fix, implemented here, is to close the serialization stream before attempting to deserialize written values.
How was this patch tested?
The original bug was masked by an invalid assert in the memory store test cases: the old assert compared two results record-by-record with zip but didn't first check that the lengths of the two collections were equal, causing missing records to go unnoticed.
TODO: I also should add a dedicated test for PartiallySerializedBlock in its own suite.

Test build #65199 has started for PR 15043 at commit 35a32e7.

Test build #65199 has finished for PR 15043 at commit 35a32e7.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65199/
Test PASSed.

Did a simple test and it does fix the bug. One interesting thing is while records.count() returns a smaller number than the actual count, the spark UI still shows the correct records number, in my test case it's 2999808 v.s. 300000.

#15056 also touches this code and creates a new test suite for this component so I'd prefer to merge that PR first.

Test build #65341 has started for PR 15043 at commit 2f43e69.

Test build #65341 has finished for PR 15043 at commit 2f43e69.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65341/
Test FAILed.

Merged build finished. Test FAILed.

Test build #65390 has started for PR 15043 at commit c4e50e6.

Test build #65393 has started for PR 15043 at commit ccf929f.

Test build #65393 has finished for PR 15043 at commit ccf929f.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65393/
Test FAILed.

Jenkins, retest this please.

Test build #65397 has started for PR 15043 at commit ccf929f.

Test build #65390 has finished for PR 15043 at commit c4e50e6.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65390/
Test PASSed.

Test build #65397 has finished for PR 15043 at commit ccf929f.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65397/
Test FAILed.

Merged build finished. Test FAILed.

Test build #65407 has started for PR 15043 at commit daf447b.

Test build #65407 has finished for PR 15043 at commit daf447b.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65407/
Test PASSed.

Test build #65465 has started for PR 15043 at commit 0d70774.

Alright, I've updated this to address the latest round of review feedback. I did manage to spot a memory-accounting problem with off-heap memory because PartiallyUnrolledIterator had hardcoded the use of MemoryMode.OFF_HEAP.
If this change

Test build #65465 has finished for PR 15043 at commit 0d70774.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65465/
Test FAILed.

Jenkins, retest this please.

Test build #65541 has started for PR 15043 at commit 0d70774.

I agree that we should add more off-heap tests, but I'd like to do it in another patch so that we can get this one merged faster to unblock the 2.0.1 RC.
In terms of testing off-heap, I think that one of the best high-level tests / asserts would be to strengthen the releaseUnrollMemory() checks so that inappropriately releasing unroll memory during a task throws an exception during tests. Today there are some circumstances where unroll memory can only be released at the end of a task (such as an iterator backed by an unrolled block that is only partially consumed before the task ends), so the calls to release unroll memory have been tolerant of too much memory being released (it just releases min(actualMemory, requestedToRelease)). However, this is only appropriate to do at the end of the task so we should strengthen the asserts to only allow it there; this would have caught the memory mode mixup that I fixed here.
I'm going to retest this and if it passes tests then I'll merge to master and branch-2.0. I'll add the new tests described above in a followup.

Test build #65541 has finished for PR 15043 at commit 0d70774.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65541/
Test FAILed.

Merged build finished. Test FAILed.

I believe that this latest test failure is caused by a known flaky PySpark test, so I'm going to merge this now and will monitor tests afterwards.


Fallback to native entity if no revision found for properties

If an audited entity has relational properties, the reader will try to find them as the same version, or previous.
But if the targeted class does not has any revision at all, this produces a NoRevisionFound exception.
This should be catched and should fallback to a classic doctrine find query.
Concrete error with stack trace:
No revision of class 'AppBundle\Entity\User' (1) was found at revision 7 or before. The entity did not exist at the specified revision yet.

in vendor/simplethings/entity-audit-bundle/src/SimpleThings/EntityAudit/AuditReader.php at line 322   +
at AuditReader ->find ('AppBundle\Entity\User', array('id' => '1'), '7', array('threatDeletionsAsExceptions' => true))
in vendor/simplethings/entity-audit-bundle/src/SimpleThings/EntityAudit/AuditReader.php at line 443   +
at AuditReader ->createEntity ('AppBundle\Entity\Ticket', array('id' => 'id', 'title' => 'title', 'addedAt' => 'added_at', 'updatedAt' => 'updated_at', 'assignedAt' => 'assigned_at', 'closedAt' => 'closed_at', 'priority' => 'priority', 'state' => 'state', 'awaitingAnswerReminded' => 'awaiting_answer_reminded', 'feedbackScore' => 'feedback_score', 'feedbackComment' => 'feedback_comment', 'feedbackDoneAt' => 'feedback_done_at', 'awaitingFeedbackReminded' => 'awaiting_feedback_reminded', 'category_id' => 'category_id', 'author_id' => 'author_id', 'assigned_id' => 'assigned_id', 'organization_id' => 'organization_id', 'server_id' => 'server_id'), array('id' => '526', 'title' => 'Ticket de test Behat', 'addedAt' => '2016-10-01 14:45:00', 'updatedAt' => '2016-10-04 11:33:45', 'assignedAt' => null, 'closedAt' => null, 'priority' => '2', 'state' => '0', 'awaitingAnswerReminded' => '0', 'feedbackScore' => null, 'feedbackComment' => null, 'feedbackDoneAt' => null, 'awaitingFeedbackReminded' => '0', 'category_id' => '1', 'author_id' => '1', 'assigned_id' => null, 'organization_id' => null, 'server_id' => '1'), '7')
in vendor/simplethings/entity-audit-bundle/src/SimpleThings/EntityAudit/AuditReader.php at line 331   +
at AuditReader ->find ('AppBundle\Entity\Ticket', array('id' => '526'), '7')
in src/AppBundle/Manager/TicketManager.php at line 250   +
at TicketManager ->getTimeLine (object(Ticket))
in src/AppBundle/Controller/TicketController.php at line 259   +
at TicketController ->showAction (object(Request), object(Ticket))
at call_user_func_array (array(object(TicketController), 'showAction'), array(object(Request), object(Ticket)))
in vendor/symfony/symfony/src/Symfony/Component/HttpKernel/HttpKernel.php at line 144   +
at HttpKernel ->handleRaw (object(Request), '1')
in vendor/symfony/symfony/src/Symfony/Component/HttpKernel/HttpKernel.php at line 64   +
at HttpKernel ->handle (object(Request), '1', true)
in vendor/symfony/symfony/src/Symfony/Component/HttpKernel/DependencyInjection/ContainerAwareHttpKernel.php at line 69   +
at ContainerAwareHttpKernel ->handle (object(Request), '1', true)
in vendor/symfony/symfony/src/Symfony/Component/HttpKernel/Kernel.php at line 185   +
at Kernel ->handle (object(Request))
in web/app_dev.php at line 34   +

Basically, the reader try to find a User instance of a specified revision because it's a property of the Ticket instance.
But after a fresh deploy, existing User instances does not has any revision yet.
Could you please push a new patch release after merging this? This is quite critical.
Regards.

@Soullivaneuh would you mind writing a test that covers this case?
quick question: what happens, if the find-query doesn't succeed either? Wouldn't this throw a 'new' exception (which might need conversion to NoRevisionFoundException)?

would you mind writing a test that covers this case?

I'll look at it.

quick question: what happens, if the find-query doesn't succeed either? Wouldn't this throw a 'new' exception (which might need conversion to NoRevisionFoundException)?

You are right. I'll throw the exception again if no result is found.

@tolry After reflection, I'm not sure this way is the good way.
Indeed, this method is here to find an entity at a specific revision.
This make no sense to return another entity if the revision does not exist.
Two possible solution:

Do nothing. The end user must catch the NoRevisionFoundException and do the native query instead, or not.
Add a fallbackToNativeEntity parameter. If true, fallback to native find query like I did, if false, just throw the NoRevisionFoundException.

What do you think?

Ok now I remember why I did this. This is for relation.
Indeed, on my code, I list all revision of an entity before finding them:
$ticketRevisions = array_reverse($this->auditReader->findRevisions(Ticket::class, $ticket->getId()));
foreach ($ticketRevisions as $revision) {
    $currentTicketVersion = $this->auditReader->find(Ticket::class, $ticket->getId(), $revision->getRev());

So there is no way to have a NoRevisionFoundException.
But the reader also tries to find related entities at the same revision. And at this case, it fails.
So we should add a doctrineFallback flag, default to false and set it to true on internal calls.

Ok so just forget all my comments. The PR is right because the fallback is done only for properties here.
This happen when you didn't update a PR from a long time... :stuck_out_tongue:
I'm looking for testing.

@tolry Tests added. Failing Travis seems to not be related to this PR.
Something looks wrong with pgsql...

I have an issue with MySQL: https://travis-ci.org/simplethings/EntityAudit/jobs/188847686#L305
I can't delete the related object of article to test the exception re-throw.
Actually, the find method of the manager returns null if no entity is found. Maybe would it be better to let the doctrine behavior for this case?

Actually, the find method of the manager returns null if no entity is found. Maybe would it be better to let the doctrine behavior for this case?

:+1: yes, i think so too.

@DavidBadura will do.
I saw the 1.0.0 release was done with the related branch.
As it's a bugfix, should I change the target branch for 1.0?

done

:+1: LGTM

@DavidBadura I just fixed the tests, it should be OK now.

thanks!

Thanks you too! Any change to have a patch release for this? This is related to a quite critical issue.

ping @DavidBadura? :-)

I'll make it this week

@DavidBadura Please wait a little before make a new release, I found a bug with reverse relations.
I'm working on a patch.


Windows debugimage has .diz files instead of .pdb files

Despite the use of --disable-zip-debug-info in build.sh, debug images for Windows contain .diz files instead of the expected .pdb files.

It turns out that configureCommandParameters() doesn't call buildingTheRestOfTheConfigParameters() for Windows: #1736 fixes this problem.


Fix analytics.js initialization timing issue and refactor

use analytics.ready() to call analytics.track() etc.
reorganize files
remove Tracker.nonreactive in a place where I think it had no effect

Fixes #47 and #58

@pauldowman  :+1: . I also expanded the example apps to show analytics emit results. If it looks good to you then I can bag n' tag this PR and publish 1.0.1 to Atmosphere

Nice! I tweaked the example apps just a little bit to show all events in order, not just the latest, to help spot duplicates and reactive re-running problems.

awesome @pauldowman !


Add Section and Section times/dates/TAs

If you look up a major you see: Title Capacity Instructor	LectureDays Enroll Code
add in Section and the information for the Section (time, dates, TA(s)) using the scraper from the scrapers-ucsb-curriculum
F16- OK (200pts)

F16- OK (200pts)


Include .net-9.0 as Target Framework

In v1.64 .net8.0 is included as a target framework for this  package however, in v1.7.0 .net9.0 is not included as a target frame work (net9.0-ios & net9.0-android are included.
I have a unit test project for my maui project which uses nunit. This needs to be able to target the plain net9.0 framework.

I'll add it to the next version which should be online by mid-next week.

I've just released version 1.7.1 which includes it.
Closing as completed.


Support mqtt protocol

Close: #382

aicers/giganto-client#31

Codecov Report
Patch coverage: 67.93% and project coverage change: -0.03 :warning:

Comparison is base (02bd1ba) 69.55% compared to head (de8ea61) 69.53%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #386      +/-   ##
==========================================
- Coverage   69.55%   69.53%   -0.03%
==========================================
  Files          20       20
  Lines        2989     3115     +126
==========================================
+ Hits         2079     2166      +87
- Misses        910      949      +39

Impacted Files
Coverage Δ

src/ingest/tests.rs
91.37% <ø> (ø)

src/graphql/export.rs
16.45% <3.70%> (-1.22%)
:arrow_down:

src/publish.rs
66.30% <57.89%> (-0.36%)
:arrow_down:

src/graphql/network.rs
87.50% <83.33%> (+0.07%)
:arrow_up:

src/ingest.rs
84.03% <100.00%> (+0.46%)
:arrow_up:

src/ingest/implement.rs
96.85% <100.00%> (+0.21%)
:arrow_up:

src/publish/tests.rs
97.80% <100.00%> (+0.12%)
:arrow_up:

src/storage.rs
67.35% <100.00%> (+0.40%)
:arrow_up:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


[CLOSED] Zoom does not maximize fully if toolbar is hidden

 Issue by GoogleCodeExporter
Monday Mar 16, 2015 at 09:15 GMT
Originally opened as https://github.com/douglasdrumond/macvim/issues/27

What steps will reproduce the problem?
1. Start MacVim
2. Hide toolbar
3. Click on the green zoom button

What is the expected output? What do you see instead?
Expected: Maximize vertically
Instead: Does not maximize fully

What version of the product are you using? On what operating system?
snapshot-0710B

Please provide any additional information below.
Running on 10.5

Original issue reported on code.google.com by crazor on 6 Nov 2007 at 12:35

 Comment by GoogleCodeExporter
Monday Mar 16, 2015 at 09:15 GMT

Additionally, when a sheet appears on a manually maximized window, the window
height
is resized to exactly the same size as if I had maximized by clicking the zoom
button.

E.g.
* Click zoom
* resize window manually by dragging
* :browse split ~/

Original comment by crazor on 6 Nov 2007 at 12:50

 Comment by GoogleCodeExporter
Monday Mar 16, 2015 at 09:15 GMT

Thanks for the report.  I can confirm this bug.  I'll look into it.

Original comment by<EMAIL_ADDRESS>on 10 Nov 2007 at 10:40

Changed state: Accepted

 Comment by GoogleCodeExporter
Monday Mar 16, 2015 at 09:15 GMT

The problem was that Cocoa resizes windows that are too large automatically
when a dialog sheet is displayed.  I
have fixed this bug by making sure the window never is larger than what Cocoa
thinks will fit on the screen.  You
may notice the window "snapping back" if you try to make it "too large".

Original comment by<EMAIL_ADDRESS>on 19 Nov 2007 at 8:49

Changed state: Fixed


Merge latest develop branch to main.

Merging latest code from develop in preparation of deploying to slightly less alpha environments.


Working with bash arrays

NOTE: All 1st-time contributors should know Topic approval and PR submission do NOT guarantee your Topic/Article will be published. Our team of Peer Reviewers and Content Moderators will review all PRs that come in to make sure they adhere to the standard of quality we expect from the EngEd community.
We expect all community members to go through provided examples, resources, and previously published material before submitting content. As a rule of thumb, please only submit articles (pull requests) that are complete, formatted correctly, and include a fully polished article (ready to be published and error-free).
All revisions and edits should be completed on your forked repo (as to not take up room in the queue). Any PR that is submitted incomplete (meaning not ready to be published as is) will be closed. A PR being closed does NOT mean the article can not be published (or fixed) it simply means that the edits, revisions, and fixes will happen outside of the queue.
NOTE: Any topic suggestion copied/taken/modified from YouTube may Lead to a permanent ban from the community.
Topic Suggestion
Working with bash arrays
Writing sample(s):
https://www.section.io/engineering-education/concurrency-in-go/
https://www.section.io/engineering-education/integrating-graphql-api-in-a-django-application/
Proposal Submission
Proposed title of the article
Working with bash arrays
Use keyword research to improve your article's SEO.
Proposed article introduction
Bash scripting, just like with other programming languages and scripting tools, allows you to create variables, run for loops, apply conditional logic, and more importantly, you can work with arrays, which are our focus in the tutorial. Arrays in Bash scripting are quite powerful for storing multiple data and fundamental data structures.
In this tutorial, we will walk you through what bash arrays are, the distinction between bash arrays and other concepts, and how to perform key operations on them with detailed examples.
ONLY viable topics to pick from:

API
Artificial Intelligence
Blockchain
Containers
Edge Computing
Languages
Machine Learning
Networking
Node.js
Security

Key takeaways

Good understanding of bash arrays.
How to perform key operations using bash arrays with detailed examples.
How to differentiate bash arrays from other similar bash concepts.
The different types of bash arrays with examples and code snippets.

Article quality
This article provides a detailed view of what bash arrays are and how to perform key operations with them using code snippets and examples. There are other concepts which may seem similar to bash array this article differentiates these concepts with examples. Lastly, shows how to add, remove and print elements in a bash array.
References
N/A.
Conclusion
Finally, remove the Pre-Submission advice section and all our blockquote notes as you fill in the form before you submit.
We look forward to reviewing your topic suggestion.
Templates to use as guides

How To Guide Template
Software Review Template
Tutorial Template

Sounds like a helpful topic - lets please be sure it adds value beyond what is in any official docs and/or what is covered in other blog sites. (We're really looking for unique/original content. EngEd articles should go beyond a basic explanation - and it is always best to reference any EngEd article and build upon it).
Please be attentive to grammar/readability and make sure that you put your article through a thorough editing review prior to submitting for final approval. (There are some great free tools that we reference in EngEd resources.)

ANY ARTICLE SUBMITTED WITH GLARING ERRORS WILL BE IMMEDIATELY CLOSED.

Please be sure to double check that it does not overlap with any existing EngEd articles, articles on other blog sites, or any incoming EngEd topic suggestions (if you haven't already) to avoid any potential article closure.
Please reference any relevant EngEd articles in yours. - Approved
@Anitaachu


Fit Predict method

Currently, no fit predict is supported. Behaviour is desirable and probable would need to use Parametric UMAP base or other means as workaround.

Also see #10

For the RandomizedSearchCV I get TypeError: _BaseScorer.call() missing 1 required positional argument: 'y_true' for random_search.fit(coordinates) when I place error_score="raise".  Really like the code, but when I run it with GridSearchCV to see if it works it doesn't choose the optimal parameters.

@matthewmeadows81  This is the following the "Tuning with HDBSCAN" example which initializes `validity_scorer = make_scorer(hdbscan.validity.validity_index,greater_is_better=True)?
As for the issue with GridSearchCV not choosing the optimal parameters, it could be due to a number of reasons. It could be that the scoring metric you're using is not suitable for your data or problem, or it could be that the range of parameters you're searching over does not include the optimal parameters.
`


PHP Configuration hints

I had loads of trouble trying to get MailHog (with or without mhsendmail) to work with php-fpm (MacOS, homebrew, php5.6, mailhog 0.2)
I finally got it going with this in php.ini:
sendmail_path = "/usr/local/bin/mailhog sendmail<EMAIL_ADDRESS>I can do a PR to add that simple recipe to the docs if you'll direct me where to put it.
(I do see that hint in https://github.com/mailhog/mhsendmail/issues/3#issuecomment-127992203, which is where I got it, but that was closed.)

mailhog/mhsendmail#3 should have fixed it, assuming you have a To: header in the message
It could be worth adding some documentation to mention it - PHP config is mentioned in the (mhsendmail)[https://github.com/mailhog/mhsendmail]) readme, but currently without the e-mail address in the path

Hmm, maybe a new release for mhsendmail is the key fix?
Note that I couldn't get mailhog 0.2.0 to work without the gratuitous<EMAIL_ADDRESS>but it works fine with it.

It might be that (really need to fix #71!), but if you've tried the MailHog sendmail one then it should use the latest mhsendmail for whichever version of MailHog you have installed
I'll do a new release after I've got through some open issues, hopefully that'll fix it!

OK, thanks for your work on this. MailHog is awesome. A real step forward in my dev workflow. Love the "delete all".

Thanks @rfay, and no problem, glad its useful :smile:

A tip to anyone wondering how to use Mailhog if it's running on a separate instance (not the php server) - for example on a separate docker container.
All you need to do is modify the sendmail_path on php server to include -S option:
sendmail_path = /usr/sbin/sendmail -S mail:1025 -t

Where "mail" is the alias of my container running Mailhog.

:+1: thanks for the info, I've added the info to the MailHog README including a php.ini example

I would like to add that using sendmail in debian docker containers seem to have issues. A good replacement is ssmtp.
Install using apt install ssmtp
ssmtp can be configured to use mailhog by editing /etc/ssmtp/ssmtp.conf like so:
<EMAIL_ADDRESS># replace mailhog with the name of the mailhog service
mailhub=mailhog:1025
hostname=yourserver.example.com
FromLineOverride=YES

PHP can then be configured in php.ini like so:
sendmail_path = /usr/sbin/ssmtp -t
Some more details on ssmtp can be found here
This worked for me. I hope someone finds it useful


MeasureGroupComponent Codes Don't Reflect Multiple Populations

The odd numerator and denominator counts in my tests are due to a defect in the FhirMeasureEvaluator that does not properly code the MeasureReportGroupComponents to handle multiple numerators, denominators, etc. For example, DMS measure has 3 numerators and 3 denominators that aren't being coded correctly. Here's a snippet from a successful MeasureReport:
  "type": "individual",
  "measure": {
    "reference": "Measure/measure-dms"
  },
  "patient": {
    "reference": "Patient/Patient-2517"
  },
  "period": {
    "start": "2017-01-01T00:00:00-05:00",
    "end": "2017-12-31T00:00:00-05:00"
  },
  "group": [
    {
      "population": [
        {
          "code": {
            "coding": [
              {
                "code": "initial-population"
              }
            ]
          },
          "count": 1
        },
        {
          "code": {
            "coding": [
              {
                "code": "numerator"
              }
            ]
          },
          "count": 1
        },
        {
          "code": {
            "coding": [
              {
                "code": "denominator"
              }
            ]
          },
          "count": 1
        },
        {
          "code": {
            "coding": [
              {
                "code": "numerator"
              }
            ]
          },
          "count": 1
        },
        {
          "code": {
            "coding": [
              {
                "code": "denominator"
              }
            ]
          },
          "count": 1
        },
        {
          "code": {
            "coding": [
              {
                "code": "numerator"
              }
            ]
          },
          "count": 1
        },
        {
          "code": {
            "coding": [
              {
                "code": "denominator"
              }
            ]
          },
          "count": 1
        }
      ]
    }
  ],
  "evaluatedResources": {
    "reference": "#cd3c4385-e2bb-4218-89c0-d044c99c68e9"
  }
}
Aaron Kelly Peters [75] is in initial-population
Aaron Kelly Peters [75] is in numerator
Aaron Kelly Peters [75] is in denominator
Aaron Kelly Peters [75] is in numerator
Aaron Kelly Peters [75] is in denominator
Aaron Kelly Peters [75] is in numerator
Aaron Kelly Peters [75] is in denominator

I don't see the above patch in the latest checkout. Is there a better way included or is there a reason why it is not?

Closing, thanks


Unable to validate Twitter Protected account

I am not able to prove my protected Twitter account identity using Keybase. Please guide me with instructions. Thank you.

See this issue.


AttributeError

sorry for asking again this question because its closed or not answered.
so i am getting same error
" return image.copy(), label.copy(), np.array(size), name
AttributeError: 'NoneType' object has no attribute 'copy'' while i am running test.py.
Could you please guide me.

That means the image and the label are None, so maybe the path of these files is not correct


Move part of timing module out of the library and to test

Investigate options for removing the timing module.
Discussion on the list suggests that there are good arguments for moving at least parts of this out of the main library (although DTLS has uses for some of this, so we might want to keep some functionality in libmbedtls), but we still want the functionality in the example programs.
Mailing list discussion: https://lists.trustedfirmware.org/pipermail/mbed-tls/2020-April/000051.html
This is part of #4030

Moving this would probably involve splitting it, moving one bit to the SSL library (perhaps a new sub-module there), and for most of the rest, creating a new "Unix / Windows" platform layer, which would have to be designed and would probably require discussion before we even agree on what its goal should be.
I think this requires more design work that we can afford for 3.0, and since we agreed to limit 3.0 to simple changes, I'm inclined to post-pone that one to 4.0.

As I wrote on the mailing list, I think that the fate of functions that the library needs (for DTLS) and functions that only test/sample programs need should be distinct considerations.
For the functions that library needs, it wouldn't be much work to move them to platform.c and rename them to mbedtls_platform_xxx. We could then later in 3.x add the same configurability mechanism that other mbedtls_platform_xxx have.
I still think that we should move the benchmark-only functions to tests/src (potentially later in 3.x to be moved to programs/src or some such, but that's a separate discussion).

I have no objections to doing what you suggest in 3.0 if it we can agree on a design quickly enough (in particular, which functions should go where, and associated options to replace MBEDTLS_TIMING_C).

In this comment on net_sockets you suggest that in the future (probably 4.0) we should re-design the platform module in order to include things that are currently in some other modules, including timing. I agree with that, and based on that "do nothing for 3.0, revisit in 4.0" has my vote here too.

Team decision: keep what's needed for DTLS, move the rest (benchmarking etc) out of the library. Excat list TBD.

mbedtls_timing_hardclock: was used for HAVEGE and mbedtls_hardclock_poll. HAVEGE has been removed. mbedtls_hardclock_poll is a weak source of entropy, which is not really useful since there is no guarantee that it has any entropy, so it should be removed. Remove.
mbedtls_timing_get_timer: I'm on the fence about keeping it in the API.

used directly in udp_proxy, which is not meant to be portable beyond POSIX+Windows, so it can.
used as an internal function in timing.c.

mbedtls_set_alarm: only used in benchmark.c. Remove.
mbedtls_timing_set_delay, mbedtls_timing_get_delay: not used in the library, but necessary for DTLS. Keep.
mbedtls_timing_self_test: Pointless. Remove.

@gilles-peskine-arm , seems that the mbedtls_timing_hardclock() is still used in the macro TIME_AND_TSC( TITLE, CODE ) in benchmark.c. I guess that the former should be moved to the latter? And if so I assume that it can be removed from the test_suite_timing.function?

Yes, if something is only used in the benchmark program and not useful for DTLS, please move it to the benchmark program. If a function is no longer in the library then corresponding test code (in a test suite or in a self-test function) needs to be removed as well.


Output split between "Output" and "Terminal" panes

Preflight Checklist

[X] I have installed the latest version of Power Query SDK.
[X] I have checked existing resources, including the common issues and the release notes.
[X] I have searched for similar issues.

Power Query SDK
0.1.7
Regression From
No response
Platform
Windows
Architecture
x64
OS Version
No response
VSCode version
No response
PQSdkTool Path
No response
Bug Description
When VSCode builds an extension then evaluates the current file, the command line statements run + results produced are spliit between VSCode's "Output" and "Terminal" panes.
Steps to Reproduce
Edit extension file so that the SDK rebuilds it, then run "Evaluate current file."
Actual Experience
VSCode "Terminal" pane:
Executing task: c:\Users\ben\.vscode\extensions\powerquery.vscode-powerquery-sdk-0.1.7-win32-x64\.nuget\Microsoft.PowerQuery.SdkTools.2.110.3\tools\MakePQX.exe compile

{
  "TargetMezRootPath": "C:\\Users\\ben\\Desktop\\TestConnector\\BenTest\\bin\\AnyCPU\\Debug\\BenTest.mez",
  "RelativePathsOfContent": [... {output truncated for brevity}
 *  Terminal will be reused by tasks, press any key to close it.

VSCode "Output" pane:
[5:28:40 PM]	[Info]	[Task found] c:\Users\ben\.vscode\extensions\powerquery.vscode-powerquery-sdk-0.1.7-win32-x64\.nuget\Microsoft.PowerQuery.SdkTools.2.110.3\tools\PQTest.exe run-test
				--extension "c:\Users\ben\Desktop\TestConnector\BenTest\bin\AnyCPU\Debug\BenTest.mez"
				--queryFile "c:\Users\ben\Desktop\TestConnector\BenTest\BenTest.query.pq"
				--prettyPrint
[5:28:42 PM]	[Info]	RunTestBattery result [
    {
        "ActivityId": "5594a7b4-6688-4e53-9682-18fbf197ca5a",
{output truncated for brevity}

Expected Experience
Both sets of output to appear in the same pane (i.e. either both in "Terminal" or both in "Output"). That way, the user can see all commands that were used in one place.
Additional Context
No response

I'm not sure there is anything we can change here.
@albertli-msft can keep me honest, but the Build Task is triggered at the VS Code level rather than calling it directly from the SDK. This runs it as a Task, so its output shows up in a Terminal window. We do it this way because it allows the developer complete control over their default Build Task, allowing the use of their build utility of choice (ant, make, cake...).

That makes sense. Thanks, @mattmasson.


sqlite: Quoted tables generate invalid code

Version
v1.18.0
What happened?
Queries on tables with dashes (and other characters that require quotes) generates invalid Go code.
Relevant log output
sqlc generate failed.

# package db
error generating code: source error: 20:6: expected 'IDENT', found "Foo-Bar"

Database schema
CREATE TABLE "foo-bar" (
  name text      NOT NULL
);

SQL queries
-- name: ListFooBar :many
SELECT * FROM "foo-bar";

Configuration
{
  "version": "1",
  "packages": [
    {
      "path": "db",
      "engine": "sqlite",
      "schema": "query.sql",
      "queries": "query.sql"
    }
  ]
}

Playground URL
https://play.sqlc.dev/p/d99fe3b457f038006ef8c8c29b22c60edab178dc9e83e5cddac4408c9cbe696c
What operating system are you using?
macOS
What database engines are you using?
SQLite
What type of code are you generating?
Go

Duplicate of https://github.com/sqlc-dev/sqlc/issues/1817


docs: update Anaconda env create commands for latest version

Our Anaconda installation instructions are incorrect for the latest versions, as reported in #81 and #95. This pull request updates the instructions, which no longer distinguish between Anaconda and Miniconda.
This pull request will resolve #95.

Updating reviewers and assignees to the first likely person returning from holiday 🙂.

I saw it! :)

Thanks @codeallthethingz !


Can't connect ui to registry while building docker from example ssl with clair

I'm trying to build from the example for clair with SSL and getting errors while trying to link the registry to ui.
It's giving me SSL errors no matter what I try with self signed certificates.
Basically just to try things out I ran a "docker-compose -f docker-compose.clair-ssl.yml up -d"
I'm able to log into the UI afterwards once I accept the self sign SSL cert, but when I enter the FQDN for the registry, even with adding an entry to /etc/hosts, it still will not connect.
This is on release version 2.4.3

MAN!!! I spent no joke 2 weeks trying to figure out how to make the certs work within these in the docker-compose bridge network. I got a CA from our DC and used self signed cert and put htem EVERYWHERE that the base OS of each image uses and used update-ca-certificates in each of the places it looks via the Man page... Hoping for a resolution because long term use will be hard to use in docker....
I do wonder if "bare metal" works better.


BindingResolutionException on Connector if request is sent async

Hi there,
I got a connector, which resolves it's url this way:
    public function resolveBaseUrl(): string
    {
        $url = config('services.cloudservice.url');

        return "https://$url";
    }

If I use the connector using send($request) everything works like a charm.
But if I do this: instead
return $this->cloudConnector->sendAsync($request)

I get this error:

Illuminate\Contracts\Container\BindingResolutionException: Target class [config] does not exist

The trace points to:
->resolveBaseUrl(): string …
      › {
      ›     $url = config('services.cloudservice.url');
      ›

...

/vendor/saloonphp/saloon/src/Http/PendingRequest.php:93 {▶
      Saloon\Http\PendingRequest->__construct(Connector $connector, Request $request, MockClient $mockClient = null) …
      › $this->method = $request->getMethod();
      › $this->url = URLHelper::join($this->connector->resolveBaseUrl(), $this->request->resolveEndpoint());
      › $this->authenticator = $request->getAuthenticator() ?? $connector->getAuthenticator();
    }

Hi, not a problem, I am very thankful for your work.
I am using it within a Laravel app with PestPHP tests.

 composer show | grep saloon
saloonphp/laravel-plugin              v3.0.0    Laravel plugin for Saloon
saloonphp/saloon                      v3.0.2    Build beautiful API integra...

I wonder if this has to do with the promise wrapping that is happening inside of the sendAsync method. I'll have a look now.

I've been testing and I can't seem to replicate it - where is your 'cloudConnector' coming from inside of your job, does it come from the Laravel service container? If you are able to share some of your job class that would be great thank you

Sure:
This is a part of my model class:
public function hcloud(): ?HcloudService
{
    if (!$this->hcloudToken) {
        return null;
    }

    $service = new HcloudService(hcloudToken: $this->hcloudToken);

    if ($this->hcloudSshKey_id) {
        $service->setHcloudSshKeyId(hcloudSshKeyId: $this->hcloudSshKey_id);
    }

    if ($this->hcloudServer_id) {
        $service->setHcloudServerId(hcloudServerId: $this->hcloudServer_id);
    }

    return $service;
}

And this is an example coming from my job:
$this->server
    ->hcloud()
    ->getSshKey()
    ->then(function () {
        $this->server
            ->hcloud()
            ->deleteSshKey()
            ->then(function () {
               //...
            });
    });
    ```

And this is coming from the service class:

public function getSshKey(int $hcloudSshKeyId = null): PromiseInterface|Response
{
$hcloudSshKeyId = $hcloudSshKeyId ?? $this->hcloudSshKeyId;
throw_if(condition: !$hcloudSshKeyId, exception: new InvalidArgumentException("No SSH ID provided"));

$request = new GetSshKeyRequest($hcloudSshKeyId);

return $this->hcloudConnector->sendAsync($request);

}

Thank you for sharing - apologies for asking you for lots of code, but are you able to share with me how $this-. hcloudConnector is being set?

@ahoiroman Hi!
Some questions:

Which version of Laravel are you using?
Is it possible to share the whole stack trace?
You can omit parts like username, app name/path, if you want.
Does it fail during tests and/or when running the app?
If in tests:

Which version of PestPHP are you using?
Can you share your test case class, or at least the imports and extend of it?

I've seen things like these happen before, both with helper functions, but also classes.
It's usually related to Laravel (and not something like Saloon), like extending the wrong base test case.
Or like Sam said, code trying to do things before dependent code has booted.

Hey! Was running into this issue too when running Pest in parallel mode.
tl;dr I fixed it with the following snipped in a teardown of test cases:
public function tearDownSaloonTrait(): void
{
    Config::setSenderResolver(null);
    Config::clearGlobalMiddleware();
}

Why this works, or rather why it failed in the first place? The Saloon Laravel plugin uses the Saloon\Config to set global middleware and a sender resolver. These are static properties and will be shared in a parallel testing session between testcases. When you mix Laravel application tests with pure Unit tests, these config settings will persist. So in a Unit test, where you don't have the application booted, those middleware and sender resolver are still loaded and will try to access the application core.
In my opinion, Saloon should probably reset this Config class after each testcase, or give the tools to do this easily.
Cheers!

Ooh that's interesting @erikgaal thank you for sharing that. What version of Saloon and Laravel are you using? We're using L10 at work and Saloon v2 and not getting issues but it could be an issue with v3's service provider

▲ (erik.gaal) composer show | grep -E "saloon|laravel/framework|pestphp/pest"
laravel/framework                                v10.32.1            The Laravel Framework.
pestphp/pest                                     v2.23.2             The elegant PHP Testing Framework.
pestphp/pest-plugin                              v2.1.1              The Pest plugin manager
pestphp/pest-plugin-arch                         v2.4.0              The Arch plugin for Pest PHP.
pestphp/pest-plugin-laravel                      v2.2.0              The Pest Laravel Plugin
saloonphp/laravel-plugin                         v3.1.0              Laravel plugin for Saloon
saloonphp/saloon                                 v3.0.5              Build beautiful API integrations and SDKs with Saloon

Hey @erikgaal I've attempted to replicate this issue. I've created a brand new Laravel application with 60 individual tests (which are exact copies of each other) and I'm sending a Saloon request inside of each of them. It seems to work fine without the teardown in parallel and regular single-threaded tests.

This is the test which is run 60 times
Saloon::fake([
   MockResponse::fixture('user'),
]);

$connector = new TestConnector;
$request = new UserRequest;

$response = $connector->send($request);

expect($response->status())->toEqual(200);

Let me know if I can refine my tests!

Hey @ahoiroman @erikgaal I'm going to close this issue to due inactivity and I don't believe this is an issue with Saloon. I'm happy to discuss this further if you want, but I'm going to close this for now.

Sorry, missed this. Did you mix Laravel's TestCase (that have CreatesApplication) and PHPUnit's base TestCase?
You need to mix tests with the Laravel application booting and tests that don't.

I will take a look, I'm pretty sure it was tests that booted Laravel, but I didn't mix tests with both, thanks for the info.

I'm going to close this one for now as it's really difficult replicate. Thanks all for helping out so far, hopefully in the future we can figure out the issue.


[.NET SDK] Error: Invalid Cast Exception while casting "builtin.datetimeV2.daterange" type to Newtonsoft.Json.Linq.JArray after upgrading to Bot Builder 3.9.

SDK Platform: .NET
SDK Version: 3.9
Active Channels: Web Chat, DirectLine;
Deployment Environment: Azure App Service

Issue Description
Exception is raised during runtime whenever I try to cast "Resolution.Values" in "builtin.datetimeV2.daterange" type to Newtonsoft.Json.Linq.JArray after upgrading to Bot Builder 3.9.
Code Example
//Getting all the entities of type builtin.datetimeV2.daterange into list.

var years = result.Entities.Where(x => x.Type.Equals("builtin.datetimeV2.daterange"));

            foreach (var year in years)
            {
                 foreach (var vals in year.Resolution.Values)
                 {
                    //Exception is raised when I try to cast vals to JArray.
                          if (((JArray)vals).First.SelectToken("type").ToString() == "daterange")
                           {
                            _startYear = (DateTime)((JArray)vals).First.SelectToken("start");
                            _endYear = (DateTime)((JArray)vals).First.SelectToken("end");
                            }
                  //custom logic ....
                  }
         }

JSON
"entities": [
    {
      "entity": "attrition",
      "type": "attrition",
      "startIndex": 0,
      "endIndex": 8,
      "score": 0.74733305
    },
    {
      "entity": "this year",
      "type": "builtin.datetimeV2.daterange",
      "startIndex": 14,
      "endIndex": 22,
      "resolution": {
        "values": [
          {
            "timex": "2017",
            "type": "daterange",
            "start": "2017-01-01",
            "end": "2018-01-01"
          }
        ]
      }
    }
  ]

Reproduction Steps

Getting all the entities of type builtin.datetimeV2.daterange into years list.
Iterating over each year in years list and getting the JArray Object from "year.Resolution.Values".
Exception is raised when I am casting each "year.Resolution.Values" type to Newtonsoft.Json.Linq.JArray.

Expected Behavior
I need to get _startYear and _endYear values from "year.Resolution.Values" which contains list of JArray Objects in the form of <key,value> pair.
Actual Results
Invalid cast exception is raised when casting "EntityRecommendation.Resolution.Values" After upgrading to BotBuilder 3.9 .However, there were no issues when I was using BotBuilder 3.8.5.
Is there any way to read start and end values without casting it to Newtonsoft.Json.Linq.JArray type?

Are you able to cast to JObject in lieu of JArray?

@JasonSowers - It still throws Invalid Cast Exception when I tried casting to JObject too. Json posted in the post is not the actual json response but a part of it. The problem is with the "values" under "Resolution" key.

At the risk of sounding stupid, you tried
(((JObject)vals).SelectToken("type").ToString() == "daterange")

and not the following, right?
(((JObject)vals).First.SelectToken("type").ToString() == "daterange")

Are you able to post your complete code to a repo so we can take a look?

Did you ever resolve this?

I am also getting the same error. Any solution for that.

I got the alternate solution by converting JSON value to C# Class Properties and Deserialize using JSONConverter Class.

Would you mind posting an example of your workaround for others?

Here is the example code. Please follow the below mentioned steps:
1) Create a Class File with following structure. You can modify the structure based on JSON value. Similarly might be you have to create a class for TimeRange.
using System.Collections.Generic;
namespace BotXYZ
{
public class CustomDateRange
{
public string timex { get; set; }
public string type { get; set; }
public string start { get; set; }
public string end { get; set; }
}
public class RootObject
{
    public List<CustomDateRange> values { get; set; }
}

}
2) Place the code in RootDialog.cs as per below:
						if (entity.Type == "builtin.datetimeV2.daterange")
                        {
                            object jsonobject;
                            entity.Resolution.TryGetValue("values", out jsonobject);

                            string jsonstringvalue=  JsonConvert.SerializeObject(jsonobject);

                            List<CustomDateRange> lstDateRangeObj = (List<CustomDateRange>)Newtonsoft.Json.JsonConvert.DeserializeObject(jsonstringvalue, typeof(List<CustomDateRange>));

                            foreach (CustomDateRange objCustomDateRange in lstDateRangeObj)
                            {
                                if (objCustomDateRange.start != null)
                                {
                                    var start = objCustomDateRange.start;
                                }

                                if (objCustomDateRange.end != null)
                                {
                                    var end = objCustomDateRange.end;
                                }

                                if (objCustomDateRange.timex != null)
                                {
                                    var timex = objCustomDateRange.timex;
                                }
                            }

                        }

Hoping that issue has been resolved, closing it. Feel free to reopen if any questions.


FaissException when virtual table is empty

Currently there is a FaissException whenever the virtual table is empty. This causes a lot of problems as the system terminates without throwing an exception or any warning. Is there any way to check if the virtual table is empty before running the query? Or is there a more elegant solution?
libc++abi: terminating due to uncaught exception of type faiss::FaissException: Error in virtual void faiss::IndexFlat::search(idx_t, const float *, idx_t, float *, idx_t *, const SearchParameters *) const at /Users/alex/actions/sqlite-vss-runner/_work/sqlite-vss/sqlite-vss/vendor/faiss/faiss/IndexFlat.cpp:34: Error: 'k > 0' failed

I have same issue https://github.com/asg017/sqlite-vss/issues/126


⚠ ClamAV has degraded performance

In 05b2ae0, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 196 ms

Resolved: ClamAV performance has improved in e1a1dda.


Test #18 fails (cpp-netlib 0.11.2, Visual Studio 2015, boost 1.55)

I successfully built cpp-netlib v0.11.2 using Visual Studio 2015. However, when I run the tests an error is reported for Test #18: cpp-netlib-http-server_async_run_stop_concurrency.
The console output is as follows:
1>------ Build started: Project: RUN_TESTS, Configuration: Debug Win32 ------

1>        Start  1: cpp-netlib-message_test
1>   1/19 Test  #1: cpp-netlib-message_test .............................   Passed    0.48 sec
1>        Start  2: cpp-netlib-message_transform_test
1>   2/19 Test  #2: cpp-netlib-message_transform_test ...................   Passed    0.04 sec
1>        Start  3: cpp-netlib-utils_thread_pool
1>   3/19 Test  #3: cpp-netlib-utils_thread_pool ........................   Passed    0.04 sec
1>        Start  4: cpp-netlib-uri_test
1>   4/19 Test  #4: cpp-netlib-uri_test .................................   Passed    0.06 sec
1>        Start  5: cpp-netlib-uri_builder_test
1>   5/19 Test  #5: cpp-netlib-uri_builder_test .........................   Passed    0.06 sec
1>        Start  6: cpp-netlib-uri_builder_stream_test
1>   6/19 Test  #6: cpp-netlib-uri_builder_stream_test ..................   Passed    0.04 sec
1>        Start  7: cpp-netlib-uri_encoding_test
1>   7/19 Test  #7: cpp-netlib-uri_encoding_test ........................   Passed    0.02 sec
1>        Start  8: cpp-netlib-relative_uri_test
1>   8/19 Test  #8: cpp-netlib-relative_uri_test ........................   Passed    0.03 sec
1>        Start  9: cpp-netlib-http-response_incremental_parser_test
1>   9/19 Test  #9: cpp-netlib-http-response_incremental_parser_test ....   Passed    0.02 sec
1>        Start 10: cpp-netlib-http-request_incremental_parser_test
1>  10/19 Test #10: cpp-netlib-http-request_incremental_parser_test .....   Passed    0.02 sec
1>        Start 11: cpp-netlib-http-request_linearize_test
1>  11/19 Test #11: cpp-netlib-http-request_linearize_test ..............   Passed    0.05 sec
1>        Start 12: cpp-netlib-http-client_constructor_test
1>  12/19 Test #12: cpp-netlib-http-client_constructor_test .............   Passed    0.06 sec
1>        Start 13: cpp-netlib-http-client_get_test
1>  13/19 Test #13: cpp-netlib-http-client_get_test .....................   Passed    6.27 sec
1>        Start 14: cpp-netlib-http-client_get_different_port_test
1>  14/19 Test #14: cpp-netlib-http-client_get_different_port_test ......   Passed    4.81 sec
1>        Start 15: cpp-netlib-http-client_get_timeout_test
1>  15/19 Test #15: cpp-netlib-http-client_get_timeout_test .............   Passed   41.11 sec
1>        Start 16: cpp-netlib-http-client_get_streaming_test
1>  16/19 Test #16: cpp-netlib-http-client_get_streaming_test ...........   Passed    1.28 sec
1>        Start 17: cpp-netlib-http-server_constructor_test
1>  17/19 Test #17: cpp-netlib-http-server_constructor_test .............   Passed    0.08 sec
1>        Start 18: cpp-netlib-http-server_async_run_stop_concurrency
1>  18/19 Test #18: cpp-netlib-http-server_async_run_stop_concurrency ...***Failed   23.84 sec
1>        Start 19: cpp-netlib-http-server_header_parser_test
1>  19/19 Test #19: cpp-netlib-http-server_header_parser_test ...........   Passed    0.07 sec
1>
1>  95% tests passed, 1 tests failed out of 19
1>
1>  Total Test time (real) =  78.46 sec
1>
1>  The following tests FAILED:
1>  	 18 - cpp-netlib-http-server_async_run_stop_concurrency (Failed)
1>  Errors while running CTest
1>C:\Program Files (x86)\MSBuild\Microsoft.Cpp\v4.0\V140\Microsoft.CppCommon.targets(133,5): error MSB3073: The command "setlocal
1>C:\Program Files (x86)\MSBuild\Microsoft.Cpp\v4.0\V140\Microsoft.CppCommon.targets(133,5): error MSB3073: "d:\Server\Programs\CMake 3.3.2\bin\ctest.exe" --force-new-ctest-process -C Debug
1>C:\Program Files (x86)\MSBuild\Microsoft.Cpp\v4.0\V140\Microsoft.CppCommon.targets(133,5): error MSB3073: if %errorlevel% neq 0 goto :cmEnd
1>C:\Program Files (x86)\MSBuild\Microsoft.Cpp\v4.0\V140\Microsoft.CppCommon.targets(133,5): error MSB3073: :cmEnd
1>C:\Program Files (x86)\MSBuild\Microsoft.Cpp\v4.0\V140\Microsoft.CppCommon.targets(133,5): error MSB3073: endlocal & call :cmErrorLevel %errorlevel% & goto :cmDone
1>C:\Program Files (x86)\MSBuild\Microsoft.Cpp\v4.0\V140\Microsoft.CppCommon.targets(133,5): error MSB3073: :cmErrorLevel
1>C:\Program Files (x86)\MSBuild\Microsoft.Cpp\v4.0\V140\Microsoft.CppCommon.targets(133,5): error MSB3073: exit /b %1
1>C:\Program Files (x86)\MSBuild\Microsoft.Cpp\v4.0\V140\Microsoft.CppCommon.targets(133,5): error MSB3073: :cmDone
1>C:\Program Files (x86)\MSBuild\Microsoft.Cpp\v4.0\V140\Microsoft.CppCommon.targets(133,5): error MSB3073: if %errorlevel% neq 0 goto :VCEnd
1>C:\Program Files (x86)\MSBuild\Microsoft.Cpp\v4.0\V140\Microsoft.CppCommon.targets(133,5): error MSB3073: :VCEnd" exited with code 8.
========== Build: 0 succeeded, 1 failed, 0 up-to-date, 0 skipped ==========

This appears to have been a known issue in previous releases but was reported to have been fixed.

Huh. This one has something to do with the port number. I have no way of testing nor fixing this currently, but if this is the only issue then it should be benign. It would be great if we can make it more deterministic (probably involves something like choosing a random non-privileged port (>4096) for a number of times). Maybe you can try something like this @ksl67? I can probably try that too, but can't test with MSVC myself.

@deanberris If you are on linux, you can check out https://developer.microsoft.com/en-us/microsoft-edge/tools/vms/windows/ with the Visual Studio 2015 Community edition. It allows snapshots so you can revert.

@kiwidoggie Thanks, that might be useful at some point. But in my case it's not just access to the compiler but also access to Windows (which I don't have).

@deanberris The Modern.IE is a full Windows Operating system that is good for 90 days, more than enough time to resolve compiler issues. You are able to reactivate the VM 4 times also, take a snapshot after you install Visual Studio/Tools and you can revert to that snapshot.


add aquavanjaram942X_20CU HSS/BSS NN/NT/TN libraries

results of hipblaslt-test on my node (20CU):
[----------] Global test environment tear-down
[==========] 35886 tests from 13 test suites ran. (2104335 ms total)
[  PASSED  ] 35886 tests.
hipBLASLt version: 1000
command line: ./hipblaslt-test


Update from upstream

Internal Asana discussion: https://app.asana.com/0/1125583950823011/1125994356185124


Few new cards

Added cards like Oshawott , Zard-Y etc..

PSGO cards are now edited / added with commands.  Seek someone on the server who has access to these commands to implement these changes.


Disable Email Notification

I could not figure out how to disable sending the report in an email.
I excluded all parts regarding emails from the script. Maybe you could provide a switch for enabling/disabling email notification.

Provided option to disable 'send email' email feature as part of https://github.com/adiralashiva8/robotframework-metrics/commit/67219da1b73a4c9274ac300f6a7b52ec337e8615
How to disable email

By default robotmetrics.py will send email on executing .py file
Using -email false (or) -email False (or) -email 0 can disable send email feature

Example: python robotmetrics.py -email False


Advice for multi-window (and multi-swapchain) setups

This is not a bug! I'm just hoping to get some advice :bowing_man:
In order to create a multi-window application, one must create multiple swap chains. My current understanding is that in order to draw to each of these swapchains, I must call swapchain::acquire_next_image for each one, however each of these calls may block if an image is not immediately available.
Is it the case that image availability for each of these swapchains may occur at differing rates if those swapchains are running on different physical devices or are presenting to different displays?
If so, would you opt for spawning a new thread for each window's swapchain? Or perhaps use futures, tokio and use Future::select to let the tokio runtime handle each one as they become ready?
If anyone has worked with vulkano and multiple windows I'd love to hear their experience or perhaps get a link to some working code! Once I get my head around the best approach I wouldn't mind making an example to add to vulkano-examples for this to clarify things for future users.

An example would be great!
I have never worked with multiple swapchain or multiple windows.
Heres the specs for the underlying vulkan call: https://www.khronos.org/registry/vulkan/specs/1.1-extensions/html/vkspec.html#vkAcquireNextImageKHR
I don't see anything about differing rates across different devices or displays.
I know from experience that linux + nvidia gpu doesnt block on this call at all, but windows + nvidia gpu does. So across different devices sounds very likely to be different.
As for displays, if the windows + nvidia gpu blocking is waiting for the frame to be displayed, then monitors with different refresh rates would block for different lengths.

So a bit of context here, although not very relevant to your problem: This difference between drivers is  the cause of a gpu OoM in the examples when running on linux + nvidia because the examples are reliant on the timing used by other drivers e.g. windows + nvidia. https://github.com/vulkano-rs/vulkano/issues/923
I know the problem here now .. I just need to sit down and figure out the best way to fix it.
It was initially thought that the linux + nvidia driver was just misbehaving and vkAcquireNextImageKHR be blocking but the vulkan specs actually specifically says that it can return whenever it wants regardless of the presentation mode in use.
from the spec for vkAcquireNextImageKHR
Applications should not rely on vkAcquireNextImageKHR blocking in order to meter their rendering speed.
The implementation may return from this function immediately regardless of how many presentation requests are queued, and regardless of when queued presentation requests will complete relative to the call.
Instead, applications can use fence to meter their frame generation work to match the presentation rate.

Ahh thanks for the extra context!

Instead, applications can use fence to meter their frame generation work to match the presentation rate.

Thank you for highlighting this - this refers to the use of the previous_frame_end future within the triangle.rs example right? Specifically this section?

@rukai just for the record, I was testing timings of certain parts of the triangle.rs example and I noticed that on my linux + intel (integrated gpu) laptop swapchain::acquire_next_image does actually block. I thought this was interesting:

setup
does acquire_next_image block?

linux+nvidia
no

linux+intel
yes

windows+nvidia
yes

Just to clarify, this was with PresentMode::Fifo. PresentMode::Mailbox always returned immediately (using 2 or 3 buffers didn't seem to affect this) which makes sense to me as the idea is to just display the most recently presented image.

Applications should not rely on vkAcquireNextImageKHR blocking in order to meter their rendering speed.

Do you have any advice on how to do this with vulkano? Is it the assignment of previous_frame_end here (which results in dropping the current previous_frame_end GpuFuture) the part that blocks on a fence, in turn metering frame generation? It's tricky to test this on my machine by throwing in some timestamps as acquire_next_image is blocking on my machine. Perhaps it would be useful to know whether or not the triangle.rs example syncs to your refresh rate on your linux+nvidia setup or whether it just loops as fast as possible? If it does sync, then perhaps a useful test could be to time how long the previous_frame_end assignment (and in turn, the drop call) takes, considering we know the frame delay is not in the call to acquire_next_image on that setup.
Sorry for so many questions! Just trying to clarify my understanding on how (or whether) vulkano does this "fence"ing under the hood.
Ahh just noticed your edit in your last comment! No worries if you are unsure, I might have to do some digging through the source, unless maybe @tomaka can briefly clarify?

I'm running two windows in separate threads.  Very few resources are shared across threads, and this doesn't seem to be a problem.  I detect no dependence on timing between the two windows / threads.  Granted, I've only tested on Linux with Nvidia's ICD.  I have an issue with winit and input, but it seems surmountable with a correct use of the winit event loop.

Vulkano now has examples with multiple windows, so I think this can be closed.


Provide more useful error message if a non-compliant DAP tool (or user) sends bad input to DAP server

This fixes https://github.com/flutter/flutter/issues/99734 which was a crash if flutter debug-adapter is run and then invalid DAP protocol messages were sent (perhaps from a non-compliant tool, or a user that discovered this command and is running it interactively).
Invalid message now result in a message printed before the real exception:

Input could not be parsed as a Debug Adapter Protocol message.
The "flutter debug-adapter" command is intended for use by tooling that communicates using the Debug Adapter Protocol.

Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Thanks! Is the Google-testing bot stuck, or is it just slow/have a backlog?

Thanks! Is the Google-testing bot stuck, or is it just slow/have a backlog?

Must be the latter, it's running now


⚠ Bofrost.at has degraded performance

In 9564cee, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 893 ms

Resolved: Bofrost.at performance has improved in d043400 after 6 minutes.


Pin GitHub Actions packages using commit SHA

Issue #, if available:
N/A
Description of changes:
This pull request pins third-party GitHub Actions packages by commit SHA based on recommended best practices in Security Hardening for GitHub Actions.
Note dependabot will keep the human readable version tag up-to-date.
Testing performed:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Is dependabot going to automatically update these anyway? Do we really gain anything with this?

Is dependabot going to automatically update these anyway? Do we really gain anything with this?

Dependabot will keep these packages updated, but the approve and merge step is still manual. The benefit is partially build reproducibility but there also is a security aspect.

This practice seems to only apply for third-party actions based on the link you provided. I think we should keep most of the actions as is and only pin the non-GH Actions ones (aka create-pull-request and action-gh-release) to a specific commit.

Going to close this one for now and we can revisit it if needed. The original intent was to pin CI dependencies to for security concerns but the likelihood of backdoors being introduced into actions packages might not be likely.


ampinstmgr quick doesn't respect ip binding

Bug Report
System Information

Operating System (Including distribution name and version number)
Debian GNU/Linux 11 (bullseye)
AMP version and build date (Always use the version number, 'latest' is not valid!)
AMP Instance Manager v<IP_ADDRESS> built 26/07/2022 20:09
Which AMP release stream you're using (Mainline, Nightly or FastTrack)
Mainline

I confirm:

[x] that I have searched for an existing bug report for this issue.
[x] that I am using the latest available version of AMP.
[x] that my operating system is up-to-date.

Symptoms

What are you trying to do? Setup HTTPS via getamp.sh on first install or executing ampinstmgr quick with parameters for <IP_ADDRESS> binding
What are you expecting to happen? ADS instance should be bind to <IP_ADDRESS>
What is actually happening? ('Nothing' is not an acceptable answer!) ADS instance is bind to <IP_ADDRESS> instead of <IP_ADDRESS>

Reproduction

Delete ADS instance if already present
Execute ampinstmgr quick admin "123456Test" <IP_ADDRESS> 8080 +Core.Webserver.UsingReverseProxy True like the getamp.sh script would also do
Look at ampinstmgr status and notice the ADS instance is bind to <IP_ADDRESS>

The correct way to do this is as follows:
bash <(wget -qO- getamp.sh) postSetupHTTPS
This corrects all of the bindings for existing ADS instances. It's not necessary to delete the instance for HTTPS.
If you have deleted the ADS01 instance, you run bash <(wget -qO- getamp.sh) as normal and it'll skip the parts of the setup that don't need doing.

Yey, I know that, but running bash <(wget -qO- getamp.sh) will result in the ip binding <IP_ADDRESS> if I say yes to HTTPS because the script executes the quick command and the quick command doesn't set the ip binding correct. The postSetupHTTPS works as expected because it executes a rebind.


#207: PersistentBuffer feature for Akka Streams

Making PR for review. Please do not yet merge util the docs are provided.

PR is complete and ready for final review.

I have following comments already discussed with Akara, just to make note here so won't lose for future improvements possibly. I can merge this PR once the build is green.
(1) Does this pull(in) desirable during preStart? https://github.com/paypal/squbs/pull/208/commits/2746b290180146917da295b38d4aad426f0b6c34#diff-e2b6421114f5f2bfa387d245178189fcR42
(2) For the buffer model as FlowShape, would SinkShape + SourceShape possibly makes more usage. Where subscriber and publisher does not need to be in the same flow. But buffer usage is good here
(3) does unlimited buffer potentially violates ReactiveStream back-pressure requirement?
(4) There might be some room of improvement in encoding/decoding phase, where using ByteString as bridge to/from ByteBuffer

Let me address these one-by-one:

Is the pull(in) desirable during preStart? Yes. This is documented at: http://doc.akka.io/docs/akka/2.4.4/scala/stream/stream-customize.html#Rate_decoupled_graph_stages
Should we have a separate source and sink shape instead of flow shape? IMO, these two are complementary. The flow shape keeps the flow declaration simple and easy to understand. The separate source/sink shape is more flexible and can even be used for IPC between two flows in two different JVMs. I'll file an issue to create a source shape. As for sink shape, I'd prefer it to be a write stage (flow shape). This would allow easy building of a composite sink which materializes to a value of choice, rather than having a fixed sink that materializes to NotUsed.
No. This is also documented at the same place under "Rate decoupled graph stages": http://doc.akka.io/docs/akka/2.4.4/scala/stream/stream-customize.html#Rate_decoupled_graph_stages
I'll file an issue for this improvement. This one is of immediate use. I'll work on it as soon as I have the time.

Thanks a lot Akara!
BTW, I manually trigger the build one more time still not passing ...

Yes, I need to try a lower parallelism. I don't know whether travis is restricting resources more or what else. Let me do that.


Make pricing adaptable

Allow pricing fields to be set in the data
The fields the pricing module is made of can now be set in the template data.
The new keys are non longer in camelCase to better match the style the rest of our python code has.
Fix pricing spacing issue when validation fires
The pricing module has a negative margin-right on desktop to allow it to stretch out of the two-thirds column.
When the validation fired, it wrapped the module in a .validation-wrapper div which constrained the width to the parent column.
This adds the same negative margin to the .validation-wrapper div to allow its contents to stretch.
Allow price fields to be marked as optional

Checked out locally and :+1:


difference between distributed and decentralized and how to get Adjacency matrix

Thank you for your excellent environment. What is the difference between distributed and decentralized settings in each environment. how to define the neighbors of each agent, or how can I obtain the Adjacency matrix of the topology of the environment

Hi,
The difference between distributed and decentralized modes are written in ReadMe file. If you would like to get the adjacency matrix of the topology, you can convert the topology to a networkx model and additionally invoke the functions in networkx to achieve the goal.

It works for me, thank you very much!

It works for me, thank you very much!


Integrate with bash-completion, if available

It would be helpful if bash-language-server could utilize bash-completion to fetch the available switches and arguments for certain commands. I'm not familiar with Typescript myself, unfortunately, so I cannot determine the feasibility of implementing such a feature at this time.
Subcommand and switch completion
For example, if git and its associated Bash completions are installed on the machine, then typing the following into a buffer:
git <ctrl+space>

would yield the following completions from the language server:
add
am
archive
bisect
etc...

Similarly, typing the following:
git --<ctrl+space>

would yield the following completions:
--bare
--exec-path
--exec-path=
--git-dir=
--help
etc...

Optionally, the server could attempt to fetch results for both git , git -, and git -- (in that order) and show completions for all three possibilities such that they can be easily filtered and selected at a glance. I'm not sure about the performance implications of this behavior, however.
Argument completion
A similar process could allow for argument completion, if the completion function provides it. For example, if openssh and its associated Bash completions are installed, then typing the following:
ssh <ctrl+space>

would possibly yield the following completions:
<IP_ADDRESS>
<IP_ADDRESS>
::1
bar
baz.com
foo.local
github.com
localhost

Future work
If a manpage exists on the system for a specific command (the same one manpage currently used for documentation suggestions in bash-language-server), we could potentially search the manpage for the subcommands/switches provided to us by bash-completion, such that entering the following:
git --h<ctrl+space>

would allow the user to select --help from the list, and hovering over the --help would parse the git manpage for a section titled --help and display the following:
--help
           Prints the synopsis and a list of the most commonly used commands. If the option --all or -a is given then all available commands are printed. If a
           Git command is named this option will bring up the manual page for that command.

           Other options are available to control how the manual page is displayed. See git-help(1) for more information, because git --help ...  is converted
           internally into git help ....

This could be done using explainshell for help, as also suggested in #107.

Any contribution here would be much appreciated. This sounds like a great feature!


ipam: Warn when failing to update CN status

This commit attempts to improve the readability and debugability of
(*Node).syncToAPIServer(). The main motivation for this change is to fix
the swallowing of the error when UpdateStatus fails, and the subsequent
Get succeeds.
To that end, this commit adds a warning log msg and propagates the error
in case we never succeed, to surface any trouble with updating the
status of the CiliumNode (CN) resource. Failures in updating the status
can be indicative of general connectivity problems in ENI and Azure IPAM
modes. As seen in https://github.com/cilium/cilium/issues/13193, it can
also lead to Cilium never becoming ready because the agent was never
able to retrieve ENI information required to make allocations, etc. (The
former was caused by the Operator being blocked from "publishing" the
ENI information into the CN resource, which is why the agent was unable
to retrieve it.)
Warn when failing to update CiliumNode status

test-me-please


[FIX] Invalid date error

🤔

When does this happens?

I had the browser open in the background and when I clicked this appeared. If I reload this error appears again.

Should I clear browser history completely?

Hum... may be somehow related to the new whats new content that was published... Anyway, on my side, I checked even with your language, it works just fine. But I will check it asap where an invalid date could occur. If you delete the browser history, it will delete all your data as well. So not sure, if I should I recommend that.

I think I’ve found something that I could improve to catch the exception. Not knowing yet why it behaves differently on your side. Update should be ready after few minutes. Apologies for the inconvenience. 😊

All good! No problem :)

Sorted out! You always hit the target! I had to hit update fast 😂
There was even a notification, but it disappeared when I clicked on it. Maybe it's bugged, but after starting later I'll see if the notification appears.

Yeah, they are removed as soon as the next fetch was occurred. But it is on its way again. Sadly, I removed the one that makes it wait one hour again before appearing. But please, I am waiting for your feedback to know if it was solved on your side.

All very well!


PHP Dropbox API v2

Since Dropbox has deprecated the v1 API (https://blogs.dropbox.com/developers/2016/06/api-v1-deprecated/) and they won't release a v2 official PHP SDK (https://www.dropboxforum.com/hc/en-us/community/posts/207429906-PHP-SDK-for-API-v2), will this package implements some of the unofficial repos listed [here] (https://www.dropbox.com/developers/documentation/communitysdks)?

No plans for that at the moment. Waiting for Dropbox to update their own sdk.

Thanks, very quick!


Please look for filename LICENSE_1_0.txt

Projects using the Boost Software License (http://www.boost.org/LICENSE_1_0.txt) usually store their license in a file named LICENSE_1_0.txt. The BSL is a license which is widely used by the C++ community (https://github.com/search?utf8=✓&q=Boost+filename%3ALICENSE_1_0.txt&ref=simplesearch).
Would it be possible to add this file name to the list of recognized options for github to detect?
Thanks!

Interesting. Do you know more about why the license has a custom filename?
If we want to add support, it'd likely be a matter of adding another level of preference to the regex around https://github.com/benbalter/licensee/blob/master/lib/licensee/project_files/license_file.rb#L30-L31.

Interesting. Do you know more about why the license has a custom filename?

I don't know. It just happened, I guess... The website (http://www.boost.org/users/license.html) does not mention anything explaining the choice of file name.

Seems fine to me to add another level of preference for LICENSE_.
Out of curiosity I'll ask Diane Cabell (see acknowledgements at the link above) if she recalls a rationale for the filename.

ping?

ping?

@benbalter Cool, thanks a lot! When will that propagate to the main site?

@benbalter it now has propagated through the trenches and the BSL is recognized properly on the community page of our project (https://github.com/STEllAR-GROUP/hpx/community). It is not recognized as the project's license on the project homepage, though: https://github.com/STEllAR-GROUP/hpx. Shouldn't it say BSL 1.0 next to 'Contributers' in the top row?
Is there anything we have to do to make it show up there as well?


Enable docker images to be built from local content (esp. for egeria container)

Enable a developer working in their dev environment (macos, linux, windows) to test changes in a cloud based k8s service
For example

build egeria under intellij on macos
Deploy & test on IBM cloud

scenarios I've felt the need for this in

working on a low power system (ie microsoft surface) when travelling
deploying full stack / larger images than local machine allows
testing cloud specific function integration (ie external kafka, ingress etc)

For example

build egeria under intellij on macos
Deploy & test on IBM cloud

See also #810 which allows for local containers to directly use locally built artifacts

I have a reworked egeria dockerfile which builds from the latest maven repo version
By default this uses 1.1-SNAPSHOT & the nexus public repo (where our build snapshots go) but these are arguments which can be modified by the user.
This cuts the docker build time massively - to just over 30s for the build phase which was taking around 30 minutes. More importantly it means that
a) The artifacts used in the container environment are identical to that we build on the official build
b) We test the distribution archives that are intended for other consumers too.
For now I am using the open-metadata-assemblies maven artifact. This contains ALL of our .gz archives. The only one containing the user interface is the *distribution.tar.gz
This works but future changes might include

Considering whether we use one or more maven artifacts
Considering whether the .gz s should be pushed anywhere other than maven central
considering the set of archives contained within the packaging jar - for example a simpler 'runtime' archive.

So will leave this issue open for that consideration.
Now testing the new build approach....

The docker build fails following this change....
The reason is that the order the plugins executed in the pom is

Build the assemblies jar
package the files into .gz

This is a reasonable default as usually one would be building java code into a jar, then creating an external zip archive for download
However in our case we are trying the opposite - to take our assemblies, and package them into a special packaging jar for distribution via maven central.
In order to achieve this I have re-configured the maven assembly plugin to execute in the 'prepare-package' phase rather than 'package. Since the jar build is linked to the 'package' lifecycle phase this will correct the order. I have then added the 'default-jar' build of the jar which will take the archives and compose them into the module jar.
It likely worked locally due to not working in a from-0 build environment and testing steps in a unit test manner.

So we now have
➜  open-metadata-assemblies git:(dockbldfix2) ✗ jar -tf target/open-metadata-assemblies-1.1-SNAPSHOT.jar
META-INF/MANIFEST.MF
META-INF/
META-INF/maven/
META-INF/maven/org.odpi.egeria/
META-INF/maven/org.odpi.egeria/open-metadata-assemblies/
egeria-1.1-SNAPSHOT-distribution.tar.gz
egeria-1.1-SNAPSHOT-sources.tar.gz
egeria-1.1-SNAPSHOT-omag-server.tar.gz
META-INF/maven/org.odpi.egeria/open-metadata-assemblies/pom.xml
META-INF/maven/org.odpi.egeria/open-metadata-assemblies/pom.properties

Note: as we revise our packaging this may be revisited as per earlier comments, but it moves the docker build forward so is a prudent change for now.

Is the docker image build expected to be fixed now?  Docker Hub image hasn't been updated in ~3 days -- so it's missing various merged PRs now...
12:59:36 [INFO] gzip: egeria-1.1-SNAPSHOT-distribution.tar.gz: No such file or directory
12:59:36 [INFO]
12:59:36 [INFO] tar: This does not look like a tar archive
12:59:36 [INFO]
12:59:36 [INFO] tar: egeria-omag-1.1-SNAPSHOT/user-interface/access-services-user-interface-1.1-SNAPSHOT.jar: Not found in archive
12:59:36 [INFO] tar: egeria-omag-1.1-SNAPSHOT/server/server-chassis-spring-1.1-SNAPSHOT.jar: Not found in archive
12:59:36 [INFO] tar: Exiting with failure status due to previous errors
12:59:36 [INFO]
12:59:36 [ERROR] The command '/bin/sh -c mvn org.apache.maven.plugins:maven-dependency-plugin:2.5.1:get -DremoteRepositories=$repo     -Dartifact=org.odpi.egeria:open-metadata-assemblies:${version}:jar -Ddest=egeria.jar &&     jar -xf egeria.jar egeria-${version}-distribution.tar.gz &&     gunzip -c egeria-${version}-distribution.tar.gz | tar -xf -        egeria-omag-${version}/user-interface/access-services-user-interface-${version}.jar       egeria-omag-${version}/server/server-chassis-spring-${version}.jar &&     mv egeria-omag-${version}/user-interface/access-services-user-interface-${version}.jar /opt/egeria/binaries/ &&     mv egeria-omag-${version}/server/server-chassis-spring-${version}.jar /opt/egeria/binaries/' returned a non-zero code: 2
12:59:36 [INFO] ------------------------------------------------------------------------
12:59:36 [INFO] BUILD FAILURE
12:59:36 [INFO] ------------------------------------------------------------------------
12:59:36 [INFO] Total time: 01:24 min
12:59:36 [INFO] Finished at: 2019-09-19T11:59:36Z
12:59:36 [INFO] ------------------------------------------------------------------------
12:59:36 [ERROR] Failed to execute goal com.spotify:dockerfile-maven-plugin:1.4.10:build (build-and-tag-latest) on project open-metadata-docker-egeria: Could not build image: The command '/bin/sh -c mvn org.apache.maven.plugins:maven-dependency-plugin:2.5.1:get -DremoteRepositories=$repo     -Dartifact=org.odpi.egeria:open-metadata-assemblies:${version}:jar -Ddest=egeria.jar &&     jar -xf egeria.jar egeria-${version}-distribution.tar.gz &&     gunzip -c egeria-${version}-distribution.tar.gz | tar -xf -        egeria-omag-${version}/user-interface/access-services-user-interface-${version}.jar       egeria-omag-${version}/server/server-chassis-spring-${version}.jar &&     mv egeria-omag-${version}/user-interface/access-services-user-interface-${version}.jar /opt/egeria/binaries/ &&     mv egeria-omag-${version}/server/server-chassis-spring-${version}.jar /opt/egeria/binaries/' returned a non-zero code: 2 -> [Help 1]

I am aware of the issue. I've manually pushed an update to dockerhub with latest master. Will fix soon.

I can confirm the build is now fixed - it's building under jenkins (and also azure) and down to around 3-5 mins by reusing maven artifacts. I am considering whether we can trigger the container build automatically after, or together with a merge build (not verify which is on a PR). Ideally an optional PR trigger would work too - but publish to a diff repo. Will raise an issue on some of these ideas

See https://dev.azure.com/odpi/Egeria/_build/results?buildId=199 for the azure builds.
I'm inclined to close this issue and open new ones as the changes went slightly away from the original text in the issue (build from local) to improving the build performance/turnaround time to integrate fixes.


NOISSUE - Update dune-project dependencies

In reference to :-
https://github.com/pitag-ha/cardio-crumble/pull/7#discussion_r1140449279
The dependencies needs to be mentioned in dune-project, so that dune build works normally, and the .ocamlformat matches the dependencies used in the project.

@pitag-ha @Engil, can you please review this one (for the versions, and in general), so maybe I can rebase this on top of the other PR #7 !

Nice, thanks for fixing this for us, @AryanGodara! You got the syntax right! I only have some comments on the constraints you've chosen and on adding ocamlformat to the dependencies.

The versions don't have much reasoning behind them. I've used versions I found in other projects I checked (for syntax), and the versions I had currently installed on my local machine.
I've added <= max_version , in case there is some breaking change between these versions in the future.
I need your help in setting the right versions @pitag-ha !

Nice, looks perfect to me now. Thanks!

I need your help in setting the right versions @pitag-ha !

Sure! Just let me know what you need my help with. I pointed out the version constraints in the comments, and you've all adapted them now, so it all looks good to me. Maybe I can explain the constraints I've chosen:

dune: dune is the OCaml build system. In our dune-project, we've specified dune 3.3 as our language. dune is backward compatible. So any dune version greater or equal to 3.3 works.
portmidi: There's only one OCaml portmidi version: 0.1, so there wasn't a big choise.
cmdliner: This is a very specific topic. There was a breaking change in cmdliner.1.1.0. That's why we have to decide whether we want cmdliner less than 1.1.0 or greater equal to 1.1.0. We want the newest, so go for the latter.

Does that make things clearer?

Yes, this makes everything clear!! I just wanted to know the reasoning for the constraints chosen