DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target/pull/702 is merged into live branch.
Auto archive content to https://github.com/VSC-Service-Account/docs-archive-test-target.git

Docs Build status updates of commit 985e66a:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-GitBranchDeletedOrForcePushed] Cannot sync git repo to specified commit because branch Release_Archive_master_2021-01-20-12-52-46 has been deleted or has been force pushed

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 985e66a:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-GitBranchDeletedOrForcePushed] Cannot sync git repo to specified commit because branch Release_Archive_master_2021-01-20-12-52-46 has been deleted or has been force pushed

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Validator version numbers

In explorer provide:

the latest validator version number
the version number of individual validator nodes

Perhaps provide some visual indicator of validator nodes that are outdated.
This will allow stakers to clearly see if nodes are current or not.
Dependencies:
https://github.com/helium/blockchain-http/issues/197
https://github.com/helium/miner/issues/683

this is done


authentication handler

For authenticating frontend services this might be interesting.
https://www.authelia.com/docs/
Authelia is a user authentication frontend (IDP). We will use eduID or LTI for authentication most of the time.

internally we will use authomator


HL-434: Calculator error notifications

Description :sparkles:

Error notifications implemented
Fixed the bug with not showing error on first Calculate click (caused by onError)
Fixed fields namings for the notification error list

Issues :bug:
Testing :alembic:
Screenshots :camera_flash:
Additional notes :spiral_notepad:

Codecov Report

Merging #729 (0407e60) into develop (9c156fb) will decrease coverage by 24.21%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##           develop     #729       +/-   ##
============================================
- Coverage    63.29%   39.07%   -24.22%
============================================
  Files          776       67      -709
  Lines        19393     1190    -18203
  Branches      1878      236     -1642
============================================
- Hits         12274      465    -11809
+ Misses        6756      701     -6055
+ Partials       363       24      -339

Impacted Files
Coverage Δ

...cationReview/calculatorErrors/CalculatorErrors.tsx
0.00% <0.00%> (ø)

...ymentAppliedMoreView/EmploymentAppliedMoreView.tsx
0.00% <0.00%> (ø)

...entAppliedMoreView/useEmploymentAppliedMoreView.ts
0.00% <0.00%> (ø)

...efit/handler/src/hooks/useHandlerReviewActions.tsx
0.00% <0.00%> (ø)

...kend/shared/shared/oidc/migrations/0001_initial.py

...rations/0023_move_benefit_amount_to_calculation.py

frontend/kesaseteli/youth/src/pages/index.tsx

.../employer/src/hooks/backend/useApplicationQuery.ts

backend/shared/shared/oidc/views/hki_views.py

...rc/components/application/form/ActionButtons.sc.ts

... and 704 more

Continue to review full report at Codecov.

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


Can I compare two ranges with this library?

Question
Can I check if two SemanticRange::Range intersect with this library? What would be the method to call?

It doesn't support it directly but I think you can use SemanticRange.to_comparators(range) to break a range into its separate parts so they can be compared, for example:
SemanticRange.to_comparators('~> 1.0.3') #=> ['>=1.0.3', '<1.1.0']

More examples in the specs: https://github.com/librariesio/semantic_range/blob/master/spec/semantic_range_spec.rb#L527-L606

I actually checked that, but how can I then compare for example '>1.0.3' with '>=1.0.3'? I cannot find any method that always takes the comparison operators into consideration.

What's the use case for comparing ranges? Are you looking for the intersection or which one is highest?

I am looking for the intersection. Example: I know that '<1.0.3' has a problem and I know that someone is using '>=1.0.3' and then I want to know if the problem is present in the usage.

If there is nothing maybe it can be done with a simple implementation of Morgan's Law.
Essentially just compare if xmin <= ymax && ymin <= xmax.

I think the only thing missing is the code to calculate what the min and max for a range is, and ranges can be a collection of ranges, for example 1.2.x || 2.x -> ['>=1.2.0', '<1.3.0'], ['>=2.0.0', '<3.0.0']

The main "problem" would just be unbounded ranges like '<3.0.0' or '>3.0.0'.
For the first I think we can consider that 0.0.0 is the lowest minimum possible but for the second what would be the maximum?

2.99999.999999999 is the max if you ignore prereleases

When you say: 2.99999.999999999 is it<PHONE_NUMBER>99999.299999999999999.299999999999999 ? I looked into http://semver.org/ but I could not find the maximum. Maybe I need to read it more carefully.

As many 9s as you can type in theory, different package managers have different limits, usually on the total string length

After some testing I though that using min and max would be a real pain. Specially the min since I would need to have to find the max of the previous version ?!?
Before quitting I had an idea of basing the implementation on comparisons of comparators.
The initial code is here https://github.com/rtfpessoa/semantic_range/commit/dcf02d5b4fba409e59d98cd287c8c121eae53eb1
fyi, this was done after a long day so please read it with a big bowl of salt 😄
As a baseline I already "tested" it with some local ideas:
SemanticRange.ranges_intersect('>1.3.0', '>1.4.0') #=> true
SemanticRange.ranges_intersect('<1.3.0', '>1.3.0') #=> false
SemanticRange.ranges_intersect('<1.3.0', '>=1.3.0') #=> false
SemanticRange.ranges_intersect('<=1.3.0', '>1.3.0') #=> false
SemanticRange.ranges_intersect('<=1.3.0', '>=1.3.0') #=> true
SemanticRange.ranges_intersect('<=1.3.0 || >=1.2.0 <=1.3.0', '>1.3.0') #=> false
SemanticRange.ranges_intersect('<=1.3.0 || >=1.2.0 <=1.3.1', '>1.3.0') #=> true
SemanticRange.ranges_intersect('<=1.3.0 >=1.2.0 <=1.3.1', '>1.3.0') #=> false

Let me know if this makes sense to you and if you find any problem. If you think this makes sense I will then make a proper PR with some real tests. Also suggestions for names are always welcome.

LGTM 👍

Just did some tests but for some reason in for example https://github.com/rtfpessoa/semantic_range/commit/6d50d8f654b40239de11ceae9b919dfdaae01aee#diff-9919d87eda5f711afd5077e400f8fabcR658 on the Comparator initializer the line return comp if comp.loose == loose fails saying that in comp.loose the loose is not a member. How is this possible?

I was able to "fix" it. When you have some time take a look to the PR.


Provide org-wide access to the Material for MkDocs Insiders repository

Follow the approach described in https://github.com/squidfunk/mkdocs-material/discussions/3816 to provide organization-wide access to the insider's build and features.

Completed.
To use the Material for MkDocs Insiders repo:

Your GitHub account belongs to the Status GitHub organization.
The Material Insiders repo must be forked in the organization (done).
Your GitHub account has permission in the Insiders fork.
Install the insider's pip package locally with pip install git+https://github.com/status-im/mkdocs-material-insiders.git.
Preview the site locally using mkdocs serve -f config/en/mkdocs.yml.
Look for the Made with Material for MkDocs Insiders message in the site footer.


No longer merge config files by default, use priorities instead

Description
At the moment, the way Topiary configuration works is counterintuitive: the README suggests that configuration files are somehow prioritized, and that a higher-priority file trumps a lower-priority one.
This is not the case. Because of how Nickel merge works, all configuration files are actually equivalent, which has led to problems when a configuration passed with the -C option disagrees with, for instance, the built-in configuration. Although this behavior is documented in the README, I think it's highly counterintuitive.
This PR switches the default behavior to priority-based, meaning that all lower-priority files will be disregarded.
The newly added flag --merge-configuration reinstates the old behavior.
Closes #612 (sort of)
Checklist
Checklist before merging:

[x] CHANGELOG.md updated
[x] README.md up-to-date

I wonder if either is the right behaviour, though. Do you actually want options that you haven't specified on the local configuration file to override the ones from the global configuration file? This doesn't seem like the general case.
Your feeling is that you shouldn't have to write | force in the local configuration to override an option (if I'm reading you correctly). You're saying that there's a natural hierarchy between files and that you really want there to be a priority. But I assume the intuitive behaviour is per-option priority. Or do I get it wrong?
Could we document what ideal semantics we'd like to have (regardless of whether it's feasible with Nickel)?

I'm not sure either I understand what you mean by "per-option priority", could you please elaborate on that? It sort of makes sense to think about per-language priority, but at the moment a "language" isn't well defined in the configuration file.

It's not necessarily something very precise. But maybe you have foo.bar's value sourced from the global configuration, while foo.baz is defined locally, and so overrides the global configuration's definition.

You're correct regarding one of the goal of this PR: I don't want to have to specify | priority 1 every time I want to override something from the built-in configuration.

Maybe a better way would be to have the gobal configuration file to be only default. But it has its limitation: maybe you want a configuration in /etc and one in $XDG_CONFIG, and then one local. There's no good way to annotate priorities on the middle one so that 1/ they take precedence on the /etc configuration file 2/ you don't have to write explicit priorities in the local configuration file.

However, there is another benefit to the behavior introduced in this PR: for the sake of reproducibility, we want to be able to completely specify the configuration of Topiary at runtime, and make it independent from, say, system-wide or built-in configuration files

I agree, it's akin to the --pure mode of nix-shell. But I'm questioning whether it should be the default. Maybe it should, but it's not intuitive to me.


[EventHub] remove uamqp imports from EventData/EventDataBatch into plugin module

related to: #21246
fixes: #24239

API change check for azure-eventhub
API changes have been detected in azure-eventhub. You can review API changes here
API changes
-             partition_key: str = None
+             partition_key: str = None,
+             **kwargs


Cannot open Query Builder in Enterprise Edition

I have purchased and paid for EE, but Query Builder does not open up.
lic issued 15/02/20 6:41 PM
Version <IP_ADDRESS>001141657
On ubuntu 18.0x
Pres CTRL-B or select SQLEDITOR|QUERYBUILDER and nothing new.

I can't reproduce this issue. But it may be somehow related to your system configuration.
Could you check error view (or log file) for any errors? https://github.com/dbeaver/dbeaver/wiki/Log-files

Hi Serge,
figured out the gotcha. Requires an active SQL Editor window open before the Query Builder becomes active.  Is not a bug, just an obscured command.


chore(package): update npm-template-sync to version 7.14.2

:tada: This PR is included in version 1.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Confusing webpage for old Injection plugin

I ended up at http://injectionforxcode.com/ from a Google search for "injection for xcode". Turns out that's the old web page with no link to http://johnholdsworth.com/injection.html.

Thanks for letting me know. I’ve updated the download links and made it redirect.


Clean xsd template

I'm trying to simplify this XSD generation.
During this extraction, I found lots of code duplication (2d4380c).
I know it is a risk change, but it will be easier to future development.

I'll give the patch a spin but I agree - cleaning up the jinja template is sorrowly needed. Should I wait until #55  is merged?

I prefer you wait for #55.

@pope1ni retired soapbox today so a soapfish release would be nice (#74). As you mentioned the XSD refactoring is pretty invasive. Do you think I should merge your PR now or after the release? Due to time constraints a release is likely within the next two weeks or so.

Do a good release now, after that we merge this PR. It is just lots of refactoring.

I'm currently working on a number of fixes that need to be in before the release and I'm happy to check this over before then.

:+1:
I think this pull request will help most of code generation changes in the future.

@pope1ni @iurisilvio What's your feeling about merging this now? I think we can deal with any fallout as follow-up patches.

This code is good. I'm using it in some cases. I don't know if @pope1ni validated it.
:+1: for merge

merged, thank you very much @iurisilvio


Support PNPM v6+ Lockfile

A proposed fix for https://github.com/google/osv-scanner/issues/324 issue.

@oliverchang @G-Rath anything else you need from my end to be able to review this PR?


An uploader of an embeds_one fails during file cleanup

find_previous_model_for_<column> blows up with the following:
undefined method `find' for #<EmbeddedObject:0x007ff26caf0b48>`

It looks like the cleanup routine was only designed to handle embeds_many associations and cannot handle an embeds_one association.
What happens in an embeds_one scenario is that the previous version of the object tries to find itself using a find method that doesn't exist.

+1 Confirmed that @coderifous's pull-request fixes this.

+1 please really need this pull-request


Language for C++ is CPP

The article only tells Language attribute value for VB and C# (CSharp)...at least from what I saw. It would be nice to indicate that C++ is CPP.

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

ID: 77c7c36c-84d9-9237-3d78-689e76d080f9
Version Independent ID: 6584735a-e8db-429d-5427-687fc709e6f8
Content: Walkthrough: Create a code snippet - Visual Studio
Content Source: docs/ide/walkthrough-creating-a-code-snippet.md
Product: visual-studio-windows
Technology: vs-ide-general
GitHub Login: @TerryGLee
Microsoft Alias: tglee

Map of content types:
Visual Basic: Basic
C++: C/C++
C#: CSharp
CSS: CSS
HTML: HTML
JavaScript: JavaScript
SQL: SQL_SSDT
XML: XML

Thanks for the tip, @mseld2. We updated this doc and it should be live shortly.


Fix fast jit issues

And implement several opcodes

LOL, we both made same modifications today.

LOL, we both made same modifications today.

Yes, I tested several spec cases and made the changes :)

LGTM


update prettier to 192, use prettierconfig.

[x] Updates prettier to 1.9.2
[x] Uses .prettierrc/.prettierignore so we can keep config in one place
[x] Uses overrides in .prettierrc so we don't need per-file overrides in scripts
[x] Can use opaque types in devices types now

Note: Make sure your IDE hard coded prettier settings are removed and that it picks up the .prettierrc

@keybase/react-hackers this is ready


Flash of un-initialized content with dynamic data

I'm using init-onload="items" because my slides are inside an ng-repeater.
The trouble is that since I changed the slides from static to Angular/repeater I find that all of the slides flash in a column before being initialized into a row. This doesn't seem to happen when they are hard coded as static slides in the HTML. Does anybody know how this can be fixed?

For what it is worth, I am using angular-slick and do not have any issue. Here is part of my code:

I'm not sure if this will help you but I had a similar issue and fixed it somewhat this way. The elements still flash briefly and it causes elements still move around, but I don't end up with a giant list not in a carousel. If you want it 100%, then make sure to call your beforeSlick() before you initialize/update the carousel data.  My situation is a little unique too because my carousel data is dynamic (updates via socket.io) and the DOM wasn't updating, I found the ng-if worked to trigger angular to redraw the DOM. Anyway, you can give this a shot.
In the angular slick JS file:
 scope: {
   ....
    prevArrow: '@',
    nextArrow: '@',
    // I'm not 100% sure these need to be '=' but this is what worked for me
    beforeLoad: '=',
    afterLoad: '='
 }
 link: function (scope, element, attrs) {
   ....
   initializeSlick = function () {
        var currentIndex, customPaging, slider;
        slider = $(element);

        // Let the controller know we are about to slick it
        if(scope.beforeSlick) {
          scope.beforeSlick();
        }
      ....

      slider.on('afterChange', function (event, slick, currentSlide) {
          if (scope.onAfterChange) {
            scope.onAfterChange(slick, currentSlide);
          }

          if (currentIndex != null) {
            return scope.$apply(function () {
              currentIndex = currentSlide;
              return scope.currentIndex = currentSlide;
            });
          }
       });

        if(scope.afterSlick) {
          // We call the callback in timeout to make sure everything has been slicked
          $timeout(function() {
            scope.afterSlick();
          });
        }

Here's some Jade showing how I'm doing it:
 slick(ng-if="dataLoaded" before-slick="beforeSlick" after-slick="afterSlick" init-onload="true" infinite="true" data="entries" )
	div.listItem(ng-repeat="entry in entries")
		.row
			.col-sm-12
				img(ng-src="{{url}}")
		.row
			.col-sm-12
				span {{entry.text}}

In the controller.js that the carousel sits in:
 $scope.beforeSlick = function() {
     $(".listItem").hide();
 };
 $scope.afterSlick = function() {
     $(".listItem").show();
 };

@Kenishi I think your example code should be:
scope: {
	data: '=',
	initialSlide: '@',
	beforeSlick: '=',
	afterSlick: '='
	// ... options ...
},

Thanks for sharing.

@jon64digital Not sure if you used one of the solutions above, what I did was to make the wrapping element width span the 9 items Im initially loading, in the largest state available and floated them to the left before the slick dom element are generated and slick-initialized is added as a class.
/ Handles view before Slick Initializes
slick:not(.slick-initialized) {
  min-width: 2,048px;
  display: block;
  .video-wrap {
    float: left;
    margin-right: $line-height-computed/2;
  }
}

This mimics the position. of when slick is initialized. We also found that lazy loading is absolutely needed for our case, so we limited our initial load to 9 items per slider and have 3 displaying.
Would say that you can pair it with @Kenishi 's solution as well. So we all know that this repo is not very well maintained, you should checkout this issue slick/issues/1005 - regarding https://github.com/kenwheeler/slick/issues/1005 so that the on-load executes in the correct manner.
Also see https://github.com/vasyabigi/angular-slick/pull/100

@jon64digital hey, have you solved this problem, I got the same issue.
I have tried @Kenishi method, but it does not work for me

@Kenishi Thank you so much! This method really works for me.


Can't delete apps.  It's not listed even though the docs say it's there.

[Enter feedback here]

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

ID: 632a367f-0c30-fc21-5040-d2d80f184bdb
Version Independent ID: 6cab37af-6dcf-d2c1-3d68-14fc7375f1f5
Content: Upload your custom apps in the Microsoft Teams admin center - Microsoft Teams
Content Source: Teams/upload-custom-apps.md
Service: msteams
Sub-service: teams-apps
GitHub Login: @ashishguptaiitb
Microsoft Alias: guptaashish

Hello @VNJoe how are you?
Can you please detail what you are not seeing on your Teams admin center so I can try to reproduce? Thank you

We appreciate any feedback that improves the content in the Microsoft docs but we haven't had any response from you so we are going to close this issue.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.

Hi @VNJoe , I've mentioned how to delete custom apps here -> https://learn.microsoft.com/en-us/microsoftteams/teams-custom-app-policies-and-settings#delete-custom-apps-from-your-organizations-catalog
Another insert in the docs is about stopping usage of an app -> https://learn.microsoft.com/en-us/microsoftteams/manage-apps#stop-app-usage-and-remove-app
Not sure if you were looking for all this info but see if it helps.

Hi @VNJoe , I've mentioned how to delete custom apps here -> https://learn.microsoft.com/en-us/microsoftteams/teams-custom-app-policies-and-settings#delete-custom-apps-from-your-organizations-catalog Another insert in the docs is about stopping usage of an app -> https://learn.microsoft.com/en-us/microsoftteams/manage-apps#stop-app-usage-and-remove-app
Not sure if you were looking for all this info but see if it helps.

CC @get-itips and @scanum as FYI.


Wouldn't it be nicer to have at least some hint of "luigi" in the name, e.g. luigi.cfg?
Can be handy when you use the "use config from current working dir" from a directory with loads of other configs and files.

I guess this is for historical reasons.
But, I think, in the long run, that it should only contain client-side related config. So I'm not sure we should rename it as we might change our minds again. No strong opinions though.

As for a luigi hint in the name. It's by default stored in /etc/luigi/client.cfg no?

indeed, for the global config /etc/luigi/client.cfg, it is,
but like I noted, you can also use ./client.cfg (from working dir).
It's not that unusual to have your project's root as working dir, which is typically littered with all sorts of config files. Having a non descriptive name like client.cfg is not really helpful for overview :)

Current version of pull request above (#1030) is the easy part: just adding support for ./luigi.cfg
the harder part is deciding about:

Also add support /etc/luigi/luigi.cfg?

whether to choose one over the other and update documentation/code comments accordingly?

also, about:

in the long run, that it should only contain client-side related config

how to split client vs non-client, e.g. luigi.cfg, luigid.cfg? What with non-client stuff in luigi.cfg then?

Good points all over and I really appreciate the PR you sent.
I like the luigid.cfg idea. :)

I went a step further and added a commit that updates to "luigi.cfg" in various parts of documentation. comments, ...
Feel free to comment
I was also considering to also add support for /etc/luigi/luigi.cfg to make the documentation less confusing. But I thought to first await some feedback from the maintainers about all this

I find that both the client ( interface.py, cmdline.py, etc) and server (scheduler.py) will read the client.cfg.
scheduler.py read client.cfg for "resources".
So it is a bit confused. what will luigi do if the client and server are deployed in different machines?  we have to put client.cfg in both machines?

Yea, I guess that's why they renamed it from client.cfg, as it's not only read by the client.

Closing this issue. as it got properly renamed with backward compatibility (see the referenced commits in this issue).


METRON-1291: Kafka produce REST endpoint does not work in a Kerberized cluster

Contributor Comments
This PR adds Kerberos support to the http://node1:8082/swagger-ui.html#!/kafka-controller/produceUsingPOST endpoint.  This can be verified in full dev by enabling Kerberos and using that endpoint to produce a message.  The same message should be returned with a call to the http://node1:8082/swagger-ui.html#!/kafka-controller/getSampleUsingGET endpoint (using the same topic).
I also added a unit test for the KafkaConfig class.
Pull Request Checklist
Thank you for submitting a contribution to Apache Metron.
Please refer to our Development Guidelines for the complete guide to follow for contributions.
Please refer also to our Build Verification Guidelines for complete smoke testing guides.
In order to streamline the review of the contribution we ask you follow these guidelines and ask you to double check the following:
For all changes:

[x] Is there a JIRA ticket associated with this PR? If not one needs to be created at Metron Jira.
[x] Does your PR title start with METRON-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.
[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

For code changes:

[x] Have you included steps to reproduce the behavior or problem that is being changed or addressed?

[x] Have you included steps or a guide to how the change may be verified and tested manually?

[x] Have you ensured that the full suite of tests and checks have been executed in the root metron folder via:
mvn -q clean integration-test install && build_utils/verify_licenses.sh

[x] Have you written or updated unit tests and or integration tests to verify your changes?

[x] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

[x] Have you verified the basic functionality of the build by building and running locally with Vagrant full-dev environment or the equivalent?

For documentation related changes:

[x] Have you ensured that format looks appropriate for the output in which it is rendered by building and verifying the site-book? If not then run the following commands and the verify changes via site-book/target/site/index.html:
cd site-book
mvn site

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.
It is also recommended that travis-ci is set up for your personal repository such that your branches are built there before submitting a pull request.

The latest commit address the issue of making the Kafka security protocol configurable.

+1 by inspection, great job.


Potential Security Vulnerabilities Detected in Package

Version
29.6.1
Steps to reproduce
Dear Team,
I trust you're in good health. I wanted to bring to your notice a potential security risk that has come up in our recently incorporated package. A vulnerability scan I conducted using Vulert on the lock file flagged the presence of over 15 potentially vulnerable dependencies.
The potential threats these vulnerabilities could introduce to our project are concerning, and I'm not completely sure about the appropriate course of action for responsible disclosure. It's of paramount importance to mention that, although some of these dependencies are only for development, their presence in the lock file could lead to their appearance in the vendor folder, making their meticulous management necessary.
For a comprehensive understanding, please refer to the vulnerability scan report of the package-lock file via this link: https://vulert.com/vuln-scan/list/92223e08-1f0c-431e-90b0-880ef2ec6518
I urge that we address these vulnerabilities without delay to uphold the security of our project. Should you need further information or an in-depth explanation, I'm available for consultation.
To make things easy, you can find the scanned lock file here: https://github.com/jestjs/jest/blob/main/yarn.lock
I look forward to your immediate attention and response regarding this matter.
Best wishes.
Expected behavior
I urge that we address these vulnerabilities without delay to uphold the security of our project. Should you need further information or an in-depth explanation, I'm available for consultation.
Actual behavior
A vulnerability scan I conducted using Vulert on the lock file flagged the presence of over 15 potentially vulnerable dependencies.
Additional context
No response
Environment
npm packages

Thanks!
11 of the reports are false positives as the tool doens't understand we don't depend on angular.
https://github.com/jestjs/jest/blob/7628207fd2e11947e05757a3bf842631f46820b1/yarn.lock#L5760-L5762
I've updated all the other versions reported in cbc8d0cee711dfc512c084f0976e96a16f842440. trim and got come via docusaurus, and is fixed in their v3, which is currently on pre-release.


Add DISTRIBUTE BY to dask-sql grammar

Currently DISTRIBUTE BY is missing from the dask-sql grammar which is extended from Postgres. There are situations where it would be beneficial to allow for data to be distributed/shuffled across a Dask environment much like Spark (which has DISTRIBUTE BY) allows to ensure data locality.

Sorry, or seems I gave you a non-working commit :-/ I can fix the style issues and I think also the first test problem with the missing "ANALYZE" later - but I think the missing /int comes from the fact, that the newer calcite version you introduced in the pom.xml seems to handle some operations differently.
I have an implementation of /int in #226 (due to other reasons), so you can either wait for that or revert the switch to the newer calcite version here (which, I guess, is not strictly needed?)

Ahh yeah so I don’t need to introduce that newer version of calcite anymore longer. I was experimenting for a bit thinking I could include spark dialect in the RelationalAlgebraBuilder amd it “just work” but that spark dialect wasn’t in calcite-core until 1.27 so that is literally the only reason I version bumped. Now I don’t need it
Get Outlook for iOShttps://aka.ms/o0ukef

From: Nils Braun @.>
Sent: Saturday, August 28, 2021 10:57:58 AM
To: dask-contrib/dask-sql @.>
Cc: Jeremy Dyer @.>; Mention @.>
Subject: Re: [dask-contrib/dask-sql] Add DISTRIBUTE BY to dask-sql grammar (#228)
Sorry, or seems I gave you a non-working commit :-/ I can fix the style issues and I think also the first test problem with the missing "ANALYZE" later - but I think the missing /int comes from the fact, that the newer calcite version you introduced in the pom.xml seems to handle some operations differently.
I have an implementation of /int in #226https://github.com/dask-contrib/dask-sql/pull/226 (due to other reasons), so you can either wait for that or revert the switch to the newer calcite version here (which, I guess, is not strictly needed?)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/dask-contrib/dask-sql/pull/228#issuecomment-907638019, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAQHLA5TJ4P2WDK67QYFQH3T7D2PNANCNFSM5C37A3AQ.

Codecov Report

Merging #228 (8bebd03) into main (0c3a6a1) will decrease coverage by 0.38%.
The diff coverage is 95.37%.

@@             Coverage Diff             @@
##              main     #228      +/-   ##
===========================================
- Coverage   100.00%   99.61%   -0.39%
===========================================
  Files           64       65       +1
  Lines         2470     2602     +132
  Branches       340      362      +22
===========================================
+ Hits          2470     2592     +122
- Misses           0       10      +10

Impacted Files
Coverage Δ

dask_sql/physical/rel/logical/filter.py
100.00% <ø> (ø)

dask_sql/physical/rel/custom/distributeby.py
47.36% <47.36%> (ø)

dask_sql/context.py
100.00% <100.00%> (ø)

dask_sql/input_utils/hive.py
100.00% <100.00%> (ø)

dask_sql/physical/rel/custom/__init__.py
100.00% <100.00%> (ø)

dask_sql/physical/rel/custom/create_experiment.py
100.00% <100.00%> (ø)

dask_sql/physical/rel/custom/create_model.py
100.00% <100.00%> (ø)

dask_sql/physical/rel/custom/export_model.py
100.00% <100.00%> (ø)

dask_sql/physical/rel/logical/__init__.py
100.00% <100.00%> (ø)

dask_sql/physical/rel/logical/join.py
100.00% <100.00%> (ø)

... and 16 more

Continue to review full report at Codecov.

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

Thanks for the response @jdye64 - I did revert to the earlier version and also fixed the styling and the remaining test issue(s).
The only thing missing to do is to add a small test for the new "distributed" keyword. Would you like to do so or shall I do it?

I am happy to write the test. I wanted your opinion on the best way to perform the actual assertion in the pytest however? Seems like checking things like partition counts and where the data logically ended up might be platform independent? Therefore I am a little lost on what that test condition should actually look like / be testing? Do you have any ideas there?

@nils-braun I have discovered a situation where this grammar does not work as expected and was hoping you had some input. For Example if I have a query like
SELECT age, name FROM people DISTRIBUTE BY age ORDER BY name
the parsing fails since it encounters the ORDER token and is expecting the EOF token. This is one of those situations where I really wish we could modify the base calcite grammar but as we already mentioned that doesn't seem feasible. What I would like to do is add something like another OrderedQueryOrExpr call in distribute.ftl after the DISTRIBUTE BY but that doesn't seem possible either? If the logic in OrderedQueryOrExpr definition was more granular I think it would be possible but it isn't. Any ideas?


TradingView add data dynamically at Android

Good day everyone!
I started work with TradingView.
How can I add and update bars from an Android app programmatically? What me need send in webView.evaluateJavascript(...) ?

Are you using "Technical Analysis Charts" https://github.com/tradingview/lightweight-charts-android/issues/1#issue-673602854? Even if you aren't, how android's wrapper is related to this repo?


Updates to getting started page

Which problem is this PR solving?
Making the text more clear, concise, readable, and consistent.
Short description of the changes
Wording changes, capitalization, rewording third-party speech, "we" speech, and forward-looking statements

@dyladan @legendecas @markwolff @mayurkale22 @mwear @naseemkullah any updates on this?

@dyladan @legendecas @markwolff @mayurkale22 @mwear @naseemkullah any updates on this?

All maintainers and approvers receive notifications when PRs and issues are opened. Please do not spam mentions.

A lot of these changes are good! Noticed the switching from we to you is following https://developers.google.com/style/person 👍

Thank you so much @naseemkullah and @mayurkale22. Could one of you please merge this PR for me? I don't have write access to the repo.

Hi @snyder114 I've mentioned your doc improvements on twitter here in response to Honeycomb looking for talent: https://twitter.com/naseemkullah/status/1331597502437535746 please let me know if thats ok or if you would prefer that be removed.

@naseemkullah thanks a lot! No that's no problem at all.


Add "--listvolumes" flag to geconfigureassetroot utility (#288)

Fixes: #288
Does require running as root, but does not require fusion service to be stopped.
Will follow up with documentation PR if this meets needs: expect that only geedocs/answer/3481558.html would need to be updated.

@tst-ppenev has approved this so, I'll go ahead and merge it. Thanks, @bradh!


[Removal]:

Hotspot Name
Tart Shadow Barracuda
Hotspot b58 Address
11TQAM7c6aYDjhmjzm2q8QW52ojryGcjixfzyo9GxvM75uacG4B
Discord Handle
No response
Hotspot Manufacturer
Bobcat
Removal Reason
I bought this hotspot (total 3 hotspots)  but not notice it was included in the denylist since it is not posteded on https://blacklist.longap.community/. However, I deployed the hotspot in the actual place without any gaming or spoofing activities. Maybe it used to be a gaming hotspot previously but it has been deployed properly since I bought it. please run the test algrothim again and check it, thanks a lot.
Modifications
No
Extra forwarders
No
Extra antennas
No
Additional Information
Hi the helium team,
Thanks for all you have done to maintain a health and honest network. Maybe I'm the unfortunate man who bought banned hotspot, but I deployed it at the right place to privide services to local devices as part of the network.
Pleas check the authenticity of the location with your algrothim again and it will prove my innocence

We do not have sufficient evidence to remove these hotspots from the denylist. This is an automated comment due to the volume of addition and removal requests. A future tool is in the works to provide more insight into the analysis approach made by the Helium team and community members that maintain this list.


Fix stack overflow

Thanks for submitting this PR. Your change to PackageFile.GetBytes did prevent a stack overflow, but didn't fix the underlying issue and make that function work. I've committed a fix to this issue and added tests (ref 5dab2aa).
The header length issue you raised will be addressed in PR #12 instead.

Yep, Ryan, it's my bad, I have not created new branch and sent PR from #12 code.
Also I have not understood the logic, and thought that it should find tag in every header.
It's great that you fixed that, thank you.

Thanks for your help!


Keywords Browser Can Not, Selenium Can

Is your feature request related to a problem? Please describe.
Selenium library has a lot of Keywords we already support.
This should be a checklist which capabilities we can not replace, but should.
Describe the solution you'd like
The follow keywords of functionalities are not yet supported:

[x] ! Alerts: Get Alert  as assertion, accept alert, wait for alert... #202

[x] ! Get Elements Size/Position #204

! Upload and maybe Download a file

[x] Upload #203
[x] Download: #201

[x] ! capture Element Screenshot Done in cce79beefe3762f63f75a87b3834dd33ce4a9664

[x] 🐭 Mouse (Up Down Move Click)

[ ] Move Mouse Over #218

[ ] Drag&Drop (playwright's mouse class might solve this) #205

[ ] ? submit form ?

[ ] ? get Source / Get inner html

[ ] Keyword Events (in general, not related to element)

[ ] ? simulate Event

[ ] ? Open Context Menu ???

Probably not going to get implemented

[ ] ? Scroll Element Into View ( not sure if needed)
[ ] ?Table Handling Keywords.? ( not sure if needed)

That is basically it.
Elements with ! are imho important
? i have no idea if needed!

´´´
Element Should Be Disabled
Element Should Be Enabled
Element Should Be Visible
Element Should Not Be Visible
Drag And Drop
Wait Until Location ...
Mouse Over
Mouse Out
Choose File
Handle Alert
Get Element Size
According to data from SL users these keywords have significant amounts of use and are not yet supported in robotframework-browser as of 4.8.2020

Element Should Be improvements and Mouse Over and Out are quite possible.
Some questions:
Element.innerHTML should already work through Get Attribute. Is there a need to also be able to get whole page's innerHTML?
What exactly was meant by Keyword Events?
Could you or Niklas contribute some test templates where drag and drop is used?


Mobile: Principles page hyperlink formatting issue

https://drive.google.com/file/d/1gygsZILKNpPBt9XprjlEe1uCxmSTXzWG/view?usp=sharing
Where hyperlinks overlap on adjacent lines, the underline doesn't continue on the top hyperlink. Additionally, the period seems disconnected from the word it should be attached to

https://designsystem.webstandards.ca.gov/principles/
iOS and and MacOS Safari

@adam-little and @kkoryaka I want to get your opinion on this.

The reported issue only seems to affect webkit browsers (Safari on iOS and macOS, Firefox).
I can fix it by removing text-underline-position: under; from the  tag globally.
There's no way I know of to reliably/responsibly target webkit browsers across screen sizes.

with under (Webkit)

with under (Chrome)

without under (Webkit)

without under (Chrome)

Should we:
a) Leave as is (with under)
b) Remove the text-underline-position: under; declaration
c) Something else

I'll defer to @kkoryaka. I don't understand the implications of removing the declaration.

The declaration changes the position of the line under the link. With it, the line goes under letters like g and y. The  first three screen shots are with the declaration, the last three screenshots are without.

Got it! I don't see any problem with removing it.

Definitely remove text-underline-position: under;.

Code merged. Closing issue.

@jaredwyee Looks good in mac chrome

https://644-fix-changelog-layout.pr.designsystem.webstandards.ca.gov/principles/


iPhone XR unc0ver 4.0.2 ECID always 0x0

As the title says when I first jailbroke last week (I had a private beta build) my ECID always read the same 0x102something but was always the same. I didn’t update till 4.0.1 came out ever since 0x0 I mean it doesn’t seem to do anything it still jailbreaks just fine and a lot faster but I feel it should be there I could try to put it in myself I took a screenshot when I was trying to help another person here but I’m worried it will mess up my jailbreak and I have a lot of stuff installed
Is this something to worry about oh and if I reinstall the beta build it will say 0x0 also wtf

https://twitter.com/pwn20wnd/status/1230595794429890560?s=21
https://twitter.com/razmashat/status/1230959939885109253?s=21
This is a known issue and will likely be resolved in the next couple days.

Duplicate of #1701


TypeError in chai assertions while using CLI of locally installed testcafe

TypeError: 'caller', 'callee', end 'arguments' properties may not be accessed on strict mode
functions or the arguments objects for calls to them

import { expect } from 'chai';

fixture `TestCafe testing day`
    .page('https://www.wikipedia.org/');

test('Chai expect', async t => {
    expect(42).to.equal(42);
});

TypeError in chai assertions while using CLI of locally installed testcafe

Do you run it from npm scripts?

I run it from bin directory with node testcafe chrome testname.js.

@arubtsov We don't support this approach.


Teach Azure Filesystem to authenticate using DefaultAzureCredential in the Python SDK

Fixes #24210.
Partially implements #20511.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @Abacn

Thanks @creste. Left some comments below the issue page. For the tests, to fix Lint and Formatter error, one could do
# Run from root beam repo dir
pip install yapf==0.29.0
git diff master --name-only | grep "\.py$" | xargs yapf --in-place

or
# Run from sdks/python
tox -e py3-yapf

For RAT error, adding apache license at the top of new files (see other source files in the project)

# Run from root beam repo dir
pip install yapf==0.29.0
git diff master --name-only | grep "\.py$" | xargs yapf --in-place

Thank you for the tip! I ran those commands and fixed all linting errors in the files I modified.  One exception is sdks/python/setup.py, which appears to have many style issues outside of my changes.  I didn't fix those.
The PythonLint Jenkins job is failing with linting errors outside of the code I modified:

18:47:35 ************* Module apache_beam.typehints.batch
18:47:35 apache_beam/typehints/batch.py:32:0: W0611: Unused List imported from typing (unused-import)
18:47:35 ************* Module apache_beam.dataframe.frames
18:47:35 apache_beam/dataframe/frames.py:679:53: I1101: Module 'pandas._libs.lib' has no 'no_default' member, but source is unavailable. Consider adding this module to extension-pkg-allow-list if you want to perform analysis based on run-time introspection of living objects. (c-extension-no-member)

I'm not sure how to proceed.

# Run from root beam repo dir
pip install yapf==0.29.0
git diff master --name-only | grep "\.py$" | xargs yapf --in-place

Thank you for the tip! I ran those commands and fixed all linting errors in the files I modified. One exception is sdks/python/setup.py, which appears to have many style issues outside of my changes. I didn't fix those.
The PythonLint Jenkins job is failing with linting errors outside of the code I modified:

18:47:35 ************* Module apache_beam.typehints.batch
18:47:35 apache_beam/typehints/batch.py:32:0: W0611: Unused List imported from typing (unused-import)
18:47:35 ************* Module apache_beam.dataframe.frames
18:47:35 apache_beam/dataframe/frames.py:679:53: I1101: Module 'pandas._libs.lib' has no 'no_default' member, but source is unavailable. Consider adding this module to extension-pkg-allow-list if you want to perform analysis based on run-time introspection of living objects. (c-extension-no-member)

I'm not sure how to proceed.

The remaining linting error is
18:47:35 ************* Module apache_beam.typehints.batch
18:47:35 apache_beam/typehints/batch.py:32:0: W0611: Unused List imported from typing (unused-import)

not relevant to the change. It was introduced in #24022 but somehow not detected by linter. Never mind.

https://github.com/apache/beam/blob/b952b41788acc20edbe5b75b2196f30dbf8fdeb0/build.gradle.kts#L342
adding dependsOn(":sdks:python:test-suites:direct:py37:azureIntegrationTest") below to add it into postcommit test suite. Running with Py37 should suffice.

Run Python 3.7 PostCommit

integration test passed:
13:28:40 test_1     |   azure_integration_test: commands succeeded
13:28:40 test_1     |   congratulations :)
13:28:59 azure_it-python_3_7-jenkins-beam_postcommit_python37_pr-493_test_1 exited with code 0
13:28:59 Stopping azure_it-python_3_7-jenkins-beam_postcommit_python37_pr-493_azurite_1 ...
13:29:00 Stopping azure_it-python_3_7-jenkins-beam_postcommit_python37_pr-493_azurite_1 ... done
13:29:00 Aborting on container exit...
13:29:00
13:29:00 real	1m39.973s
13:29:00 user	0m1.198s
13:29:00 sys	0m0.143s
13:29:00 + finally
13:29:00 + docker-compose -p azure_IT-python_3_7-jenkins-beam_PostCommit_Python37_PR-493 --no-ansi down
13:29:01 Removing azure_it-python_3_7-jenkins-beam_postcommit_python37_pr-493_test_1    ...
13:29:01 Removing azure_it-python_3_7-jenkins-beam_postcommit_python37_pr-493_azurite_1 ...
13:29:02 Removing azure_it-python_3_7-jenkins-beam_postcommit_python37_pr-493_test_1    ... done
13:29:02 Removing azure_it-python_3_7-jenkins-beam_postcommit_python37_pr-493_azurite_1 ... done
13:29:02 Removing network azure_it-python_3_7-jenkins-beam_postcommit_python37_pr-493_azure_test_net
13:29:02
13:29:02 real	0m1.785s
13:29:02 user	0m0.900s
13:29:02 sys	0m0.248s

ah I see, CHANGES will go to upcoming 2.44.0 here: https://github.com/apache/beam/blob/026563483809edb7d7e9873b729830d7d857e41e/CHANGES.md?plain=1#L63. Need a rebase to latest master.

@Abacn - Oh, thanks! I fixed CHANGES.md and rebased onto the latest master.


ci: remove rustfmt for speed

Im trying to find a way to speed up ci as were currently failing due to 50min timeout
It appears its the freescale entry in the matrix, most likely because it has 133 svds that it tests, thats 1/5 of the total svds upstreamed there.
It doesnt actually look like our code is the issue. We seem to run in a second or so. It seems like
maybe cargo check and rustfmt split the time. Why rustfmt at all? its probably on average 15 seconds * 133 svds, that would be a dent
Can someone bors this and see what happens?
cc @therealprof

bors try

Doesn't look like a significant change over previous successful builds: https://travis-ci.org/rust-embedded/svd2rust/builds

There are a couple groups, such as Atmel, which are completely empty, and wasting CI time. Perhaps removing these would help overall build time?
Or we could pare down the Freescale test set, which takes > 40 minutes by itself.

Hrm, Disagree. Definitely disappointed, and well have to see on average how much less, but it could approach 5 minutes less which is enough to start passing the occasional over 50min fail.

@rust-embedded/tools Any objections to disabling rustfmt? Does anyone know why it was added in the first place (errors are ignored anyway and we don't keep the formatted files anyway)?

rustfmt was introduced in https://github.com/rust-embedded/svd2rust/pull/104/commits/027e79dcc19e544faa547c36e9a6a2ec4179d083 to make errors format nicely. I can see how it can be useful for determining what went wrong, but I personally think we should switch over to svd2rust-regress for ci which could handle this more easily than the sh script. My suggestion would be to only format crates that fail.


Configurable Containers patch

I've added code to inform ConfigurableContainers of part volume change, so that B9-PW could also use CC tanks. MM patch to add such tanks I will maintain myself as part of CC.
I've also removed the code that propagated volume change to MFT/RF, because both support the same method as CC -- via KSPEvent system. For that to work with MFT properly I've added corresponding conversion value to config of the ModuleFuelTanks of wing typeA (as described in MFT code).
As an eye-candy, the NextConfiguration button in part menu is automatically hidden when MFT/RF/CC is used.
Last, I had to explicitly declare out variables, because Monodevelop does not support C# 7+, so no declaration expressions for me.
With CC everything is tested and works as expected. To test this with MFT/RF we need to wait until they're upgraded to KSP-1.3. Though the volume changes via KSPEvent was introduced some time ago for ProceduralParts, so it also should work.

Will test later to night

Only 1 change made from a quick review before testing.
https://github.com/Crzyrndm/B9-PWings-Fork/blob/master/B9 PWings Fork/WingProcedural.cs#L779
Because CC is checked by part module, presence had to be checked per part.
Thanks for the updated volume notification.
I'll keep the issue with out vars in mind (They really are nice though...)


Initialization Cells in Jupyter Books

Is there a way when using Thebelab in a Jupyter Book to have certain hidden cells run first as initialization cells (e.g. cells that load dependencies). I usually like to keep these hidden in textbooks so this feature would be very useful!

Ah I just realized that I'd asked about this several months back already :-)
here's the relevant issue:
https://github.com/minrk/thebelab/issues/94
I think in Jupyter Book, we could play around with the fix suggested there (to add javascript that listens for the thebelab event, and when it hears it, runs any cell that matches some kind of tag query).
So I think this could be a JB fix and not require changes in thebelab...

Let's close this one since I think it's covered in #94 and will hopefully be closable soon!


Bug Report: Same-frame renderer dependencies

Describe the bug
Right now, there is no concept of a renderer dependency. In my mind, and documentation's mind, two renderers running in a single frame have no guarantees at all about which one runs first/order etc.
That is actually not the case.
https://github.com/harrand/Topaz/blob/71fe2485b8af632e9fe2741992e37da32c05cb5e/src/tz/gl/impl/vulkan/renderer.cpp#L1102
This will CPU block for any previously-submitted work that happened this frame. That means for two composite renderers A and B. if A runs first, B will block the thread until A finishes.
Without dependency semantics this is sensible -- if we can't specify a dependency then the only way to be safe is if all composite renderers are dependent on the previous having ran to completion. However, this is dreadful for perf if the renderers are doing unrelated work and do not need to run one-after-the-other serially.
This work is blocked by Feature Request: Renderer Dependencies #74 as to make the change beforehand now means there is no safety in the case of composite renderers in a single frame (i.e all debugui will break, for starters)
Reproduction

Repros On: anything on a vulkan build
Repro Type: unexpected behaviour
Repro Rate: every time
Repro Severity (how badly does it affect your bottom line): high
Repro Steps
Steps to reproduce the bug:

Invoke any composite renderers on any vulkan debug build.
See unexpected behaviour - dbgui renderer will CPU wait on any previously-submitted CPU work. dbgui renderer has render_wait which will mean it blocks the thread until its own work is finished. So right now, it: waits for previous work -> submits -> waits for its own work. This is a potentially huge amount of unnecessary waiting. Note that for safety reasons, dbgui renderers should be dependent on anything that renders into the window, so this change needs to be careful to add some system to ensure that still happens.

Expected behavior
If renderer A is invoked before unrelated renderer B, B should not wait on A to finish - unless B has specified a renderer dependency on A; something only possible using the feature provided by Feature Request: Renderer Dependencies #74
Screenshots
If applicable, add screenshots to help explain your problem.
Conditions (please complete the following information):

OS: windows
Render API: vulkan
Build Config: debug
Version: master
Compiler: gcc

Additional context
N/A

As of Topaz 4.0, you are expected to populate the render graph and timeline, telling the underlying renderer the necessary information about how your renderers should be scheduled.
As a result of doing this, there is no longer this weird implicit wait dependency on any and all composite renderers, unless explicitly specified.


TypeError: InfoBox is not a constructor

With the latest update I'm getting this error
TypeError: InfoBox is not a constructor

using the same code as before
const InfoBox = require('google-maps-infobox')
const infoBox = new InfoBox({
    content: name,
    pixelOffset: new google.maps.Size(16, -32)
})
infoBox.open(map, marker)

Confirmed. I receive the same "not a constructor" error with version 2.0.0

See #10, there's a breaking change in v2.

const { InfoBox } = require("google-maps-infobox");
Does not work for me. Better use previous version of 2.0.0.

Fixed with the new syntax
let InfoBox: any
if (process.env.VUE_ENV === 'client') {
    import('google-maps-infobox').then(imported => {
        InfoBox = imported.InfoBox
    })
}


admin interface add buttons to add and delete items

maybe the 'delete' button should really be a 'suggest removal' or something.
We could have a suggestedForDeletion key which hides the entry in the admin interface and sets published to false, so it's easy for admins to clean up the database when the time comes.

Done and works great.


Invalid code produced

Operating system: macOS 10.13.3
Python version: 3.6.2
Black version: 18.5b0
Does also happen on master: Yes
Full repro commands: https://gist.github.com/jivid/daac0d2fa478e8acf1a1980c88de5044
The file below produces an error saying Black produced invalid code:
#!/usr/bin/env python3

import math

# fmt: off

sql = (
    "SELECT * "
    "FROM ("
        "SELECT * "
        "FROM foo "
    ") AS bar"
)

# fmt: on

print(math.floor(1.5))

If I remove the fmt comments, it works just fine.

Sorry for your trouble. This is a duplicate of #224.

Thanks @ambv. I should have searched more thoroughly before reporting


Fix missing endfor error

I somehow broke the caches page during my last rebase (I haven't ever really done rebasing like this before so I guess I screwed it up). This fixes the fatal error currently displaying on the Caches page. Sorry about that!

No worries. I'm also merging without running, just reviewing the code. I'd rather keep things moving fwd and fix issues like that than let me be bottleneck at the moment while I'm not fully productive.


ceate oc.txt

Institution/School Name
{Onaizah Colleges}
Instiution/School Website
{ https://oc.edu.sa/en }
Email Users
Please place an x between the square brackets if yes

[x] Students
[x] Academic/Teaching Staff
[ ] Other/Support Staff
[ ] Alumni

Education Levels
Please place an x between the square brackets if yes

[ ] Post-graduate research
[ ] Graduate School
[x] College (University) or Undergraduate School or equivalent
[ ] Community College or equivalent
[ ] High School or equivalent
[ ] Elementary School or equivalent

@s4cript First stage of verification passed, official domain is oc.edu.sa as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

Pull request merged. Thank you!

Thank you so much


Add Replicate demo and API

Hey @xuebinqin! 👋
Thank you for uploading the code and trained networks. It works easily and is very usefull.
This pull request makes it possible to run your model inside a Docker environment, which makes it easier for other people to run it.  We're using an open source tool called Cog to make this process easier.
This also means we can make a web page where other people can run your model! View it here: https://replicate.com/arielreplicate/dichotomous_image_segmentation
Replicate also have an API, so people can easily run your model from their code:
import replicate
model = replicate.models.get("[arielreplicate/dichotomous_image_segmentation]")
model.predict(...)

Claim your page here so you can edit it, and we'll feature it on our website and tweet about it too.
We will also change the name of the the model from "arielreplicate/dichotomous_image_segmentation" to "xuebinqin/dichotomous_image_segmentation"
In case you're wondering who I am, I'm from Replicate, where we're trying to make machine learning reproducible. We got frustrated that we couldn't run all the really interesting ML work being done. So, we're going round implementing models we like. 😊

Hey @xuebinqin! 👋
Thank you for uploading the code and trained networks. It works easily and is very usefull.
This pull request makes it possible to run your model inside a Docker environment, which makes it easier for other people to run it. We're using an open source tool called Cog to make this process easier.
This also means we can make a web page where other people can run your model! View it here: https://replicate.com/arielreplicate/dichotomous_image_segmentation
Replicate also have an API, so people can easily run your model from their code:
import replicate
model = replicate.models.get("[arielreplicate/dichotomous_image_segmentation]")
model.predict(...)

Claim your page here so you can edit it, and we'll feature it on our website and tweet about it too. We will also change the name of the the model from "arielreplicate/dichotomous_image_segmentation" to "xuebinqin/dichotomous_image_segmentation"
In case you're wondering who I am, I'm from Replicate, where we're trying to make machine learning reproducible. We got frustrated that we couldn't run all the really interesting ML work being done. So, we're going round implementing models we like. 😊

Hi,
The output messages are:
Output
CUDA out of memory. Tried to allocate 16.00 MiB (GPU 0; 14.76 GiB total capacity; 13.48 GiB already allocated; 7.75 MiB free; 13.49 GiB reserved in total by PyTorch)
Maybe it needs to remove some files dynamically.
Best,

Hi @DengPingFan,
Sorry for the very late reply.
It seems to work now on Replicate.com.
Can you check again and tell me if you'r still getting this error?

The error occurs after multiple successful generations. Something is probably adding up over time.

@ArielReplicate https://github.com/ArielReplicate/DIS/pull/1 could optimize inference.

Yes I see the crash after few trials.
Thanks @radi-cho I think using no_grad will help but probably wont solve the memory leak right?
@DengPingFan can you help on this? The inference code in the commit is quite simple. Where could the memory leak come from?

There is another version of this model on Replicate, which doesn't seem to have a memory leak. (However, it is slower to cold start and inference because of the added support for multiple additional file formats.) https://replicate.com/pratos/dis-v1


Nomad: Drain nodes on shutdown

Recently, a new (and potential useful) feature got added with Nomad v.1.5.4: Automatically draining nodes on shutdown or allowing nodes to leave the cluster automatically.
I would suggest inspecting these two new features and implementing those if useful.

Drain nodes on shutdown (the feature I would consider as being more important). Here is the documentation.
As part of configuring a "drain on shutdown", we could also consider configuring auto-leave for some nodes. While agents are already automatically removed after a 24-hour period, this setting could be useful for server nodes (when / how are those deleted?).

Thanks for taking a look at the auto-leave for server nodes. I would argue that the important part is "that server nodes should only leave if they are never added again". In our use case, this is an action only performed manually. Otherwise, I thought we could leave the cluster on shutdown and re-join when starting up. While this mechanism might have multiple disadvantages already, I would argue it is not an option any longer when checking with the documentation.
Hence, I am closing this issue. We completed everything we wanted to do (active the feature for agents and investigating it for server nodes).


Disable hotkey

Hi Volkan,
Thank you for this amazing framework,
I Have a situation, sometimes users press Alt+A hotkeys to Update a Records, but when is first time (Creating a new Record) sometimes this record get duplicated.
Is there any way to Disable this Hotkey (Alt+A) ?
Thank you!

Hide the button.

Thank you!


Pretrained models

Hi, authors! Thank you for open-sourcing your codes!
I'm wondering if you could provide your pretrained models? Thank you.

Hello, we may release these models in next month.


How to assign fixed color to individual word?

I want to assign fixed color to the words with same frequency, and the words with higher frequency will also have deeper color. I have generated the color array for each word. Is there any way to turn off the random assignment of the color and assign each word with self-defined color from the options? Thank you!

As we have determined the color for each word, but the behaviour of the wordcloud seems to shuffle the color array and assign the color to each word randomly within the given colors.
What we are caring about is disabling the random mechanism and assign the color to each word along the sequence of the color in the color array.

you could use the callback getWordColor like this:
getWordColor: word => word.value > 50 ? "blue" : "red",


feat: [SDKCF-1991] add build option for the timing of applying fetched config.

Description

Added build time settings for timing of applying fetched config

Links
https://jira.rakuten-it.com/jira/browse/SDKCF-1991
Checklist

[x] I have read the contributing guidelines.
[x] I wrote/updated tests for new/changed code
[x] I ran ./gradlew check without errors

Codecov Report

Merging #36 into master will increase coverage by 0.63%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #36      +/-   ##
==========================================
+ Coverage   78.54%   79.18%   +0.63%
==========================================
  Files          13       13
  Lines         289      293       +4
  Branches       24       24
==========================================
+ Hits          227      232       +5
+ Misses         55       54       -1
  Partials        7        7

Impacted Files
Coverage Δ

...om/rakuten/tech/mobile/remoteconfig/ConfigCache.kt
100.00% <100.00%> (ø)

...om/rakuten/tech/mobile/remoteconfig/AsyncPoller.kt
57.14% <0.00%> (+14.28%)
:arrow_up:

Continue to review full report at Codecov.

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


Assembly failure on PacBio reads (Salmonella bax reads from workshop downloads)

Has anyone ever had the reads assemble successfully?  I am preparing for the workshop and this job failed...twice.  Could someone check out and see why it failed?

When I look at the error report for the job, its blank.

I'm putting the error output in another comment to make this easier to read. Given that very first error it sounds like the upload did not succeed; and workspace pokes concur:
gally-4:release-2016-0922 olson$ ws-cat -a<EMAIL_ADDRESS>https://p3.theseed.org/services/shock_api/node/806e6260-5407-4cb9-b65f-a74e98983dd6
gally-4:release-2016-0922 olson$ ws-cat --shock -a<EMAIL_ADDRESS>gally-4:release-2016-0922 olson$
Rebecca, do you remember if you had any issues during the upload of this file?

olson@beech:/disks/p3/task_status/7a9026f3-0683-449a-af9a-9f2a58731caa$ cat stderr
GET WS => /scratch/tmp/fDrr02t6v3 salmonella_1.bax.h5<EMAIL_ADDRESS>Error retrieving https://p3.theseed.org/services/shock_api/node/806e6260-5407-4cb9-b65f-a74e98983dd6: {"status":400,"data":null,"error":["Node has no file"]}
@submit_cmd = $VAR1 = [
'ar-run',
'-r',
'smart',
'--single',
'/scratch/tmp/fDrr02t6v3/salmonella_1.bax.h5'
];
Running ar-run -r smart --single /scratch/tmp/fDrr02t6v3/salmonella_1.bax.h5
Submission returns
Job ID: 18
Submitted job 18, waiting for results
+--------+---------+----------------------------+----------+-------------+
| Job ID | Data ID |           Status           | Run time | Description |
+--------+---------+----------------------------+----------+-------------+
|   9    |    9    |          Complete          | 1:01:32  |     None    |
|   10   |    10   |          Complete          | 1:00:58  |     None    |
|   11   |    11   | [FAIL] Data does not exist |          |     None    |
|   12   |    12   |          Complete          | 1:30:58  |     None    |
|   13   |    13   | [FAIL] Data does not exist |          |     None    |
|   14   |    14   | [FAIL] Data does not exist |          |     None    |
|   15   |    15   |          Complete          | 0:08:45  |     None    |
|   16   |    16   |          Complete          | 2:05:43  |     None    |
|   17   |    17   |           Queued           |          |     None    |
|   18   |    18   |           Queued           |          |     None    |
+--------+---------+----------------------------+----------+-------------+
Running pull: ar-get -w -p -j 18 | ar-filter -l 300 -c 5
HTTP error: Forbidden (HTTP status code 403)
Error downloading results from 18

Bob, this was the upload question I pinged you about a week or two ago.   I’m not sure why her uploads were falling.  When I tested it with Rebecca, I could see that the data was sent (the UI went to 100% complete), but the files never showed up with any size associated with them in the workspace itself.   I discussed this with you the day after I had tested with Rebecca and on that day I tested myself using the same data files but uploading into Andrew’s workspace from my computer.  These uploads succeeded (and showed file size in the workspace).  It is not clear to my why it was failing for Rebecca.  At the time I asked you about it, you said you didn’t see anything odd about failures in the shock or workspace logs.

On Oct 3, 2016, at 12:31 PM, olsonanl<EMAIL_ADDRESS>wrote:
olson@beech:/disks/p3/task_status/7a9026f3-0683-449a-af9a-9f2a58731caa$ cat stderr
GET WS => /scratch/tmp/fDrr02t6v3 salmonella_1.bax.h5<EMAIL_ADDRESS>Error retrieving https://p3.theseed.org/services/shock_api/node/806e6260-5407-4cb9-b65f-a74e98983dd6 https://p3.theseed.org/services/shock_api/node/806e6260-5407-4cb9-b65f-a74e98983dd6: {"status":400,"data":null,"error":["Node has no file"]}
@submit_cmd = $VAR1 = [
'ar-run',
'-r',
'smart',
'--single',
'/scratch/tmp/fDrr02t6v3/salmonella_1.bax.h5'
];
Running ar-run -r smart --single /scratch/tmp/fDrr02t6v3/salmonella_1.bax.h5
Submission returns
Job ID: 18
Submitted job 18, waiting for results
+--------+---------+----------------------------+----------+-------------+
| Job ID | Data ID | Status | Run time | Description |
+--------+---------+----------------------------+----------+-------------+
| 9 | 9 | Complete | 1:01:32 | None |
| 10 | 10 | Complete | 1:00:58 | None |
| 11 | 11 | [FAIL] Data does not exist | | None |
| 12 | 12 | Complete | 1:30:58 | None |
| 13 | 13 | [FAIL] Data does not exist | | None |
| 14 | 14 | [FAIL] Data does not exist | | None |
| 15 | 15 | Complete | 0:08:45 | None |
| 16 | 16 | Complete | 2:05:43 | None |
| 17 | 17 | Queued | | None |
| 18 | 18 | Queued | | None |
+--------+---------+----------------------------+----------+-------------+
Running pull: ar-get -w -p -j 18 | ar-filter -l 300 -c 5
HTTP error: Forbidden (HTTP status code 403)
Error downloading results from 18
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/PATRIC3/patric3_website/issues/937#issuecomment-251155244, or mute the thread https://github.com/notifications/unsubscribe-auth/AALobpM_gka1CtC1Mi3uGTnyhsyBBZE2ks5qwS3tgaJpZM4KCrxB.

I do note that the shock error at the top here shows status 400, Node has no file.  Below that however, we see HTTP error: Forbidden (HTTP status code 403).   The 403 generally means that authentication headers existed (or we’d see a 401 error), but that the request was still forbidden.  I’m not sure why that would be the case here.
Dustin

On Oct 3, 2016, at 12:31 PM, olsonanl<EMAIL_ADDRESS>wrote:
olson@beech:/disks/p3/task_status/7a9026f3-0683-449a-af9a-9f2a58731caa$ cat stderr
GET WS => /scratch/tmp/fDrr02t6v3 salmonella_1.bax.h5<EMAIL_ADDRESS>Error retrieving https://p3.theseed.org/services/shock_api/node/806e6260-5407-4cb9-b65f-a74e98983dd6 https://p3.theseed.org/services/shock_api/node/806e6260-5407-4cb9-b65f-a74e98983dd6: {"status":400,"data":null,"error":["Node has no file"]}
@submit_cmd = $VAR1 = [
'ar-run',
'-r',
'smart',
'--single',
'/scratch/tmp/fDrr02t6v3/salmonella_1.bax.h5'
];
Running ar-run -r smart --single /scratch/tmp/fDrr02t6v3/salmonella_1.bax.h5
Submission returns
Job ID: 18
Submitted job 18, waiting for results
+--------+---------+----------------------------+----------+-------------+
| Job ID | Data ID | Status | Run time | Description |
+--------+---------+----------------------------+----------+-------------+
| 9 | 9 | Complete | 1:01:32 | None |
| 10 | 10 | Complete | 1:00:58 | None |
| 11 | 11 | [FAIL] Data does not exist | | None |
| 12 | 12 | Complete | 1:30:58 | None |
| 13 | 13 | [FAIL] Data does not exist | | None |
| 14 | 14 | [FAIL] Data does not exist | | None |
| 15 | 15 | Complete | 0:08:45 | None |
| 16 | 16 | Complete | 2:05:43 | None |
| 17 | 17 | Queued | | None |
| 18 | 18 | Queued | | None |
+--------+---------+----------------------------+----------+-------------+
Running pull: ar-get -w -p -j 18 | ar-filter -l 300 -c 5
HTTP error: Forbidden (HTTP status code 403)
Error downloading results from 18
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/PATRIC3/patric3_website/issues/937#issuecomment-251155244, or mute the thread https://github.com/notifications/unsubscribe-auth/AALobpM_gka1CtC1Mi3uGTnyhsyBBZE2ks5qwS3tgaJpZM4KCrxB.

this issue is more than 1 year old, do we still have similar issue? @olsonanl

I think it was probably an upload problem that we can close.


Don't log errors when MaintenanceSteps fail due to the repo being deleted

Resolves #1447

LibGit2Repo(...) will now log failures of Native.Repo.Open as warnings instead of errors.  The callers who catch the InvalidDataException can choose the log the problem as an error.

GitMaintenanceStep.Execute() will only log exceptions as errors when EnlistmentRootReady is true.

/azp run PR - Windows - Functional Tests (Sparse Mode)


[SPARK-29062][SQL] Add V1_BATCH_WRITE to the TableCapabilityChecks

What changes were proposed in this pull request?
Currently the checks in the Analyzer require that V2 Tables have BATCH_WRITE defined for all tables that have V1 Write fallbacks. This is confusing as these tables may not have the V2 writer interface implemented yet. This PR adds this table capability to these checks.
In addition, this allows V2 tables to leverage the V1 APIs for DataFrameWriter.save if they do extend the V1_BATCH_WRITE capability. This way, these tables can continue to receive partitioning information and also perform checks for the existence of tables, and support all SaveModes.
Why are the changes needed?
Partitioned saves through DataFrame.write are otherwise broken for V2 tables that support the V1
write API.
Does this PR introduce any user-facing change?
No
How was this patch tested?
V1WriteFallbackSuite

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/testing-k8s-prb-make-spark-distribution-unified/15468/
Test PASSed.

Test build #110494 has started for PR 25767 at commit a3f87bf.

Test build #110494 has finished for PR 25767 at commit a3f87bf.

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/110494/
Test FAILed.

Test build #111008 has started for PR 25767 at commit 39cb1a2.

cc @cloud-fan @jose-torres @rdblue  Can you ptal? I re-opened this after yesterday's DSV2 sync.

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-K8s/16109/
Test PASSed.

Test build #111010 has started for PR 25767 at commit e8b5942.

Test build #111008 has finished for PR 25767 at commit 39cb1a2.

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/111008/
Test PASSed.

Test build #111010 has finished for PR 25767 at commit e8b5942.

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/111010/
Test PASSed.

Test build #111024 has started for PR 25767 at commit cb39676.

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-K8s/16119/
Test PASSed.

Test build #111024 has finished for PR 25767 at commit cb39676.

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/111024/
Test FAILed.

retest this please

Test build #111049 has started for PR 25767 at commit cb39676.

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-K8s/16137/
Test PASSed.

Test build #111049 has finished for PR 25767 at commit cb39676.

This patch fails due to an unknown error code, -9.
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/111049/
Test FAILed.

retest this please

Test build #111052 has started for PR 25767 at commit cb39676.

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-K8s/16139/
Test PASSed.

Test build #111052 has finished for PR 25767 at commit cb39676.

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/111052/
Test PASSed.

LGTM, merging to master!


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-1.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 1013

Questions ?
Please refer to the Backport tool documentation


Preserve PeerInfos discovered

As brought up here: https://github.com/libp2p/js-libp2p/pull/64/files#r108192190
With the DHT, we will want to preserve PeerInfos discovered. This is also the start of having a way to add info about each Peer we know for statistics.

I would suggest using the peerBook to keep a record of all peers and details we have seen. In addition keeping a simple map peers or connectedPeers which are only {[id: PeerIdString]:  Multiaddr} and record the information about current connections

Another feature to add is the ability to dump to disk the PeerBook, so that we can shutdown and boot the node and get back almost every connection

yes, that should be easy to do now that we have the datastore

we could actually move the peerbook to be a direct wrapper round reading and writing to the datastore instead of in memory that way we consume less memory and it's directly backed up

@dignifiedquire that would be quite nice! As long as there is some queueing for the writes, so that we don't stress with a 'good to have but not strictly necessary' operation that does lot of IO

I can easily write datastore wrapper that batches ops if many come in at once :) but those read and writes to leveldb/indexedb are pretty cheap as well.

Though warning, it will make the whole peerbook api async

@dignifiedquire we can keep it sync, with an interval that flushes it.

@dignifiedquire on repo open, it should just load all the peerBook into memory.

This has been done \o/


Interested in support for micropython on the ESP8266?

I have an ESP8266 running Micropython. I was able to use python-tsl2591 to read data from the Lux sensor with a few minor changes. Micropython provides a lower-level interface to I2C. I implemented an SMBus emulation layer using this lower level API. This allowed me to use your code pretty directly, only changing the references to the smbus package.
Is this something you would be willing to include in your python-tsl2591 distribution? I think auto-detection of the platform can be done via the sys module. If that approach works for you, I can clean up my code and prepare a patch request. Thanks!

Sounds interesting, but my main focus is to keep it as simple as possible
so that not so experienced users could understand it. But who about a fork?
On 18 June 2016 at 00:09, Jeff Fischer<EMAIL_ADDRESS>wrote:

I have an ESP8266 running Micropython
http://docs.micropython.org/en/latest/esp8266/esp8266/quickref.html. I
was able to use python-tsl2591 to read data from the Lux sensor with a
few minor changes. Micropython provides a lower-level interface to I2C. I
implemented an SMBus emulation layer using this lower level API. This
allowed me to use your code pretty directly, only changing the references
to the smbus package.
Is this something you would be willing to include in your python-tsl2591
distribution? I think auto-detection of the platform can be done via the
sys module. If that approach works for you, I can clean up my code and
prepare a patch request. Thanks!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/maxlklaxl/python-tsl2591/issues/7, or mute the thread
https://github.com/notifications/unsubscribe/AB91_1maAu8SqKaV-U2U5MsjSQ3h9Jw0ks5qMxsVgaJpZM4I4xS_
.

Max,
Sure, I can fork the repo. I just wanted to check with you first to see if
you would like to incorporate the changes. It sounds like our goals are
slightly different - my goal is to support multiple platforms with the same
code base, making it easier for me to write multi-platform code in the
higher layers. When I have released my fork, I will put a pointer here in
case others are looking for something similar or in case you change your
mind. Thanks!
On Sat, Jun 18, 2016 at 2:07 AM, Max Ho<EMAIL_ADDRESS>wrote:

Sounds interesting, but my main focus is to keep it as simple as possible
so that not so experienced users could understand it. But who about a fork?
On 18 June 2016 at 00:09, Jeff Fischer<EMAIL_ADDRESS>wrote:

I have an ESP8266 running Micropython
http://docs.micropython.org/en/latest/esp8266/esp8266/quickref.html. I
was able to use python-tsl2591 to read data from the Lux sensor with a
few minor changes. Micropython provides a lower-level interface to I2C. I
implemented an SMBus emulation layer using this lower level API. This
allowed me to use your code pretty directly, only changing the references
to the smbus package.
Is this something you would be willing to include in your python-tsl2591
distribution? I think auto-detection of the platform can be done via the
sys module. If that approach works for you, I can clean up my code and
prepare a patch request. Thanks!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/maxlklaxl/python-tsl2591/issues/7, or mute the
thread
<
https://github.com/notifications/unsubscribe/AB91_1maAu8SqKaV-U2U5MsjSQ3h9Jw0ks5qMxsVgaJpZM4I4xS_
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/maxlklaxl/python-tsl2591/issues/7#issuecomment-226930874,
or mute the thread
https://github.com/notifications/unsubscribe/AAFSEI9KbFSwXtzZ3CW4NP99cFGGZ0Nhks5qM7VcgaJpZM4I4xS_
.

My forked repo is at https://github.com/jfischer/micropython-tsl2591. It really does have to be a fork -- micropython on the ESP8266 is severely memory-limited. Just importing the tsl2591 module caused a memory error. I ended up having to condense the code (e.g. removing things like getter methods and comments). Thus, this is going to be a micropython-specific version. The good new is that it works great!
Thanks for your help and thanks for providing a nice base for me to work from in your python-tsl2591 module. I will try to port any changes you make to my version. Thanks!


Lots of new lines in the log output makes it hard to read.

No progressbar disabled because non-interactive terminal. message and lots of \n in the log output.
See log output at https://source.cloud.google.com/results/invocations/4afdd134-00d3-424f-9f8a-7b213a699f69/targets/foss-fpga-tools%2Fverilog-to-routing%2Fsymbiflow%2Fpresubmit%2Farch-defs/log

How does more line lines make it useless.  Just grep those lines out.  If anything else I want more lines.  You never know what you need until you need it.


Adds pre/post steps for merge and update aggregate

This PR is prequel/continuation of https://github.com/NVIDIA/spark-rapids/pull/3373.
The work here adds part of the work by @ttnghia, and I added it to make sense of the extra processing added to the aggregate function expressions, and to be able to test it all.

The refactor adds a pre/post step to updates and merges. An example of a "pre merge" step is casting, or creating a struct, as is needed by MERGE_M2. The "pre update" case is not overloaded, and so it's the attribute reference as is (a pass-through projection).  The "post update" step can be used to cast (as is done in GpuM2) in the update, and then later in the merge, the "post merge" where a struct is decomposed, and fields casted, as expected by Spark.

These steps allow 1 set of casting to be removed from the grouped aggregates in aggregates.scala. I did not mess with reduction aggregates in this PR, I can do that next. It was not required for the stddev_pop work.

An untested (other than some quick examples in a shell) impl of stddev_pop is adapted from https://github.com/NVIDIA/spark-rapids/pull/3373 to demonstrate how the buffers are put together to produce the final result (sqrt(M2/n)).

The code here really needs testing, as such there is no GpuOverrides node added in this PR. In other words, the code is there, but it is not being actively used. The two new projections for pre/post steps are getting executed by existing aggs.
I tested the diffs with the integration tests locally, and in databricks 8.2. I have not run in databricks 7.3 yet, but I wanted to get this up to get some :eyes:. Note that on databricks 8.2 I am noticing other issues with the tests (as did @revans2), especially when we run with the parallel setting. Tests were failing due to some unrelated bugs, so I'll re-run tests and comment here tomorrow, and we'll need some follow ups for that.

build

I see tests passing in 7.3 and 8.2 for databricks and locally. I found a leak, and addressed it with: https://github.com/NVIDIA/spark-rapids/pull/3417/commits/a626e65ecb6e4e613ee1e41cce283fd613465887.

build

It seems that we have enough material to merge this PR. @abellina please remove the standard deviation stuffs (CudfM2, GpuM2, GpuStddevPop) as they will be reworked and reviewed in a separately PR.

It seems that we have enough material to merge this PR. @abellina before merging this please remove the standard deviation stuffs (CudfM2, CudfMergeM2, GpuM2, GpuStddevPop) as they will be reworked and reviewed in a separately PR.

I started doing this here https://github.com/NVIDIA/spark-rapids/pull/3417/commits/d4807c100f8c62c28d8c19c7897d588551917651, but re-reviewing my code I had comments around this because some of the changes are only for the M2 aggregates. Do you want me to remove comments as well, or rework them to not refer to M2?
At this point, you could let this in with the M2 reference implementation and change what you need to change, or just take the branch and do something on your own.

build

It seems that we have enough material to merge this PR. @abellina before merging this please remove the standard deviation stuffs (CudfM2, CudfMergeM2, GpuM2, GpuStddevPop) as they will be reworked and reviewed in a separately PR.

I started doing this here d4807c1, but re-reviewing my code I had comments around this because some of the changes are only for the M2 aggregates. Do you want me to remove comments as well, or rework them to not refer to M2?
At this point, you could let this in with the M2 reference implementation and change what you need to change, or just take the branch and do something on your own.
Or merge with the comments that point to future aggregates, if you are going to put your patch up soon.

Thanks, I just merged the code you removed and will continue working on them.


Add #include  for isnan

#1412

LGTM

@kdt3rd @cary-ilm We still have a broken build with GCC: https://github.com/AcademySoftwareFoundation/openexr/issues/1412#issuecomment-1581598674


We need a new Desgin for using orbitDB (data protocol)

The new design should cover

Application and user spaces
How security using orbitDB

Are we having a public collection for identity
Who can write to it?

#203
#201
#202

for namespace, can we have the below?
ehsan
ehsan/work
ehsan/home


Use XCJobs instead cpp-coveralls

Coverage increased (+15.44%) to 88.54% when pulling 3d7ace5411759915bfe4bd3bd1d7abbe0877799f on coverage into c1ceba716642593cb60c7f91e7dca6fad995d266 on master.

Coverage increased (+0.18%) to 73.29% when pulling a7d614d96401cc1e9a5bbf26a4ad8195cd2bc995 on coverage into c1ceba716642593cb60c7f91e7dca6fad995d266 on master.

Coverage increased (+0.18%) to 73.29% when pulling a0a125feef84934688ac20c2a56f8bc6d7e6ced1 on coverage into ff719f53ab615b24ce795a5f638ea5a686833746 on master.


Language auto redirection disabled

Issue #62
Language auto redirections temporarily disabled to avoid infinite looping on certain languages.

Not the way to handle this.  Pushing an commit.


Added :timeoutable to user, and set it to time out after 2 hours

I don't know why Travis didn't run on this...


Dashboard setup

Really nice, I just wish you guys had a tutorial on how to setup the dashboard for beginners that are only coming into home assistant now.

Did you have a look at the README and Example Cards ?

Can someone assist, I have the basic card that is created with a new dash up and running but none of the others are working... it seems as if sensor names have been changed I noticed that in the example file it shows capetown yet the sensor is cape_town
The template-entity-row is not being found even thought these front end integrations have all been installed !!
Has then HACS code been updated without the examples being updated, if this is the case is there any source for updated examples
Hugely appreciated and apologies for the moan but I dont have the skill to create the great cards that are already developed

Please post your code and screenshots of your sensors and logs


Add search by tag bar in feed page

Hi Emma. Did you modify the Message class as well?

I'm so sorry! I closed it by accident!


Make UITextField.rex_text send on all editing events

This solves some issues around keyboard shortcuts and autocomplete.
See: https://github.com/ReactiveCocoa/ReactiveCocoa/pull/1767

Could you update UITextView as well?

The rex_controlEvents operator is only available for iOS, so the tvOS is failing:
#if os(iOS)

I refactored UITextField's rex_text in #119 to be a bindable property and to only use rex_controlEvents for iOS. I can include this fix in that PR and provide support for UITextView as well.


Translation might be wrong

[Enter feedback here]
Hi, Thank you always for your help. I guess the translation is wrong on this page.
English: Only custom domains of length up to 64 characters are supported.
Local Language: 最大 64 文字のカスタム doメイン のみがサポートされます。
“do メイン” should be “ドメイン”. Please correct it if it is valid.

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

ID: cc2ee874-df45-1de2-1b30-1fd75c7fd709
Version Independent ID: ee181722-8386-9842-407f-d0549012d2e9
Content: Add and manage TLS/SSL certificates - Azure App Service
Content Source: articles/app-service/configure-ssl-certificate.md
Service: app-service
GitHub Login: @msangapu-msft
Microsoft Alias: msangapu

@grdicewa
It looks like your issue is resolved since you closed this thread.
If there are further questions regarding this matter, please tag me in a comment. I will reopen it and we will gladly continue the discussion.


can we play music along with rtmp streaming?

I'm not sure what do you means but if you use the microphone. Microphone can take skeapers sound.
Other way you can take internal audio that will get all audio produced by your device.

Hello
Thank you for reply .
I mean can we play raw file from assets folder along with rtmp streaming?

Hello,
You can stream a file using fromfilebase. You have the example here:
https://github.com/pedroSG94/rtmp-rtsp-stream-client-java/tree/master/app/src/main/java/com/pedro/rtpstreamer/filestreamexample
You can try get you assest file path using this method:
https://stackoverflow.com/a/56455963
If it isn't working you can copy file to a folder and use it.


Update SQL plugin documentation

Signed-off-by: Yury Fridlyand<EMAIL_ADDRESS>Description
Update SQL plugin documentation.
Issues Resolved

Update all SQL plugin information
Update list of SQL and PPL functions
Add relevancy search (full-text search) functions:

Most of them are already released in 2.1.0
query_string and match_phrase_prefix would be released in 2.2.0

#442 - PPL language block moved from observability plugin to SQL plugin, because it is provided by the SQL plugin.
Update website cross-links

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

@Naarcha-AWS, do you want me to rebase and squash? This will fix the DCO failure as well, I think.

@Yury-Fridlyand: Please do!

@Naarcha-AWS, ready for review, thanks!

@Naarcha-AWS,
Do you have any news for me so far?

Dear @hdhalter,
May I ask you to have a look on this PR?

@Yury-Fridlyand: Are you if someone from the Docs team takes this PR over and adds edits and redirects?

@Yury-Fridlyand: Are you if someone from the Docs team takes this PR over and adds edits and redirects?

For sure, do whatever you need.

@Yury-Fridlyand: Closing this in favor of #1111


[Bug]:  As soon as I am attempting to rename the JS module instance or a JS object, some random instances are coming up

Is there an existing issue for this?

[X] I have searched the existing issues

Description

Steps To Reproduce

Add any JS module instance
rename the instance
OR
Add any JS module instance
Add JS OBject
rename the JS Object

Public Sample App
No response
Environment
Release
Severity
High (Blocker to building or releasing)
Issue video log
No response
Version
Cloud/Self-hosted

@brayn003  @nidhi-nair  Can we treat this as high priority ? Looks like quite a bad UX

This has been fixed


Workshop Lab 6 cannot complete the lab

Workshop
Lab 6 HPC
Creating and Using a SLURM Cluster
Exercise 1 - Create a SLURM cluster using an Azure QuickStart template
Step 4 fails with the following error message when you try to deploy
At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/arm-debug for usage details. (Code: DeploymentFailed)
The platform image 'Canonical:UbuntuServer:15.10:latest' is not available. Verify that all fields in the storage profile are correct. (Code: ImageNotFound)
The platform image 'Canonical:UbuntuServer:15.10:latest' is not available. Verify that all fields in the storage profile are correct. (Code: ImageNotFound)
The platform image 'Canonical:UbuntuServer:15.10:latest' is not available. Verify that all fields in the storage profile are correct. (Code: ImageNotFound)

In Exercise 1, Step 2, be sure to change ubuntuOSVersion to 16.04.0-LTS. I will change the screen shot to make that clearer.


The project is for bowser or Nodejs?

I go an error:

I see the code like  require('path') in source. So is this project only for nodejs?

Project is definitely for web browser but I guess require isn't available by default on browsers indeed. I use webpack which I guess automatically converts it.
I'll check if I can convert the require to an import

So webpack includes Path-Browserify by default that should make the path lib accessible.
You can add it yourself from npm https://github.com/browserify/path-browserify

Ok I fixed it by replacing const path = require('path') with import * as path from "path-browserify"


Release 0.5.0

Add color sensor support (close #4)
Add docking sensor support (close #11)
Fix logical error causing events to not save state without handlers
Use proper Python enums for most getter types and commands
Update and add examples
General syntax improvements
Add experimental and questionably-documented Turtle backend
Bump minimum Python version to 3.10

I had to to go Python 3.10 in order to use match-case in the Turtle backend (which provides a drawing Turtle using the standard library. I think it's worth it, but I could be convinced otherwise.