mdAutocomplete

Add autocomplete and allow for template usage.
Based on: https://material.angularjs.org/HEAD/#/demo/material.components.autocomplete
Needs to allow template via schema form attribute or reference to included template.

Added basic functionality in fbf7bfc0215f8c28c09c4cfb47caba978bca6436

Not sure if something changed, but I had to double escape the single quotes in md-items in the autocomplete.html file.
md-items="item in evalExpr('this[\\''+form.optionFilter+'\\'](\\''+searchText+'\\')')"

@brianpkelley it may have, which version of MD did you run with?

Angular 1.5.6
Angular Material 1.0.9
Angular Schema Form 0.8.13
Angular Schema Form Material current Develop branch
I also commented on #11 not sure if it notified you.

In the same vein as #18, md-autocomplete (and other fields that are of type object) pass null to the validator when empty.  I may be wrong, and this is intended behavior, but this should be set to undefined like on empty strings, numbers etc.
    // Object fields (material - autocomplete ) will give a null value when we're looking for an object or undefined
    if (schema.type === 'object' && value === null) {
       value = undefined;
    }

I know this is still a work in progress, but I needed to make a few changes to the md-autocopmlete template to work with errors.  Because the md-autocomplete directive creates it's own md-input-container sf-messages needs to be moved from the container div to the inner md-autocomplete as well as an option for md-required-match
    if ( args.form.schema.requireMatch ) {
        mdAutocompleteFrag.setAttribute('md-require-match', true);
    }

  <md-autocomplete flex
    sf-messages
    ng-class="{'has-error': hasError(), 'has-success': hasSuccess(), 'has-feedback': form.feedback !== false}"
    ng-disabled="form.readonly"
    ng-model="$$value$$"
    sf-autocomplete
    sf-field-model="replaceAll"
    sf-changed-auto-complete="form"
    schema-validate="form"
    md-selected-item="$$value$$"
    md-search-text="searchText"
    md-selected-item-change="'todo';"
    md-items="item in evalExpr('this[\\''+form.optionFilter+'\\'](\\''+searchText+'\\')')"
    md-item-text="item.name"
    md-floating-label="{{::form.title}}"
    md-menu-class="autocomplete-custom-template"
    >
    <md-item-template>
      {{item.name}}
    </md-item-template>
    <md-not-found>
      No matches found
    </md-not-found>
  </md-autocomplete>

Thanks @brianpkelley back from holiday so I'll try to sort it out soon, yes autocomplete was the least finished of all the components, really hacked together for my own project at the time.


MCT Adaptation query issues: Batching

The OpenMCT Adaptation does not support query by batching. Browsers limit queries to 6 at a time, so any plot or group of plots with 6+ channels will be blocked waiting for previous channel returns. Typical batching parameters look like this:
    domainObject.dict,
    options.strategy,
    options.size,
    options.start,
    options.end,
    options.domain

Each option represents a unique type of query in MCT that may be batched with other queries.

Original ticket reference: https://github.com/NASA-AMMOS/AIT-GUI/issues/209


Add swagger integration test

Fixes up static swagger generation and adds a basic integration test to ensure it doesn't revert again.

Codecov Report

Merging #83 into master will increase coverage by 3.5%.

@@             Coverage Diff             @@
##             master      #83     +/-   ##
===========================================
+ Coverage     40.27%   43.77%   +3.5%
- Complexity       89       95      +6
===========================================
  Files            63       63
  Lines           514      514
  Branches         44       44
===========================================
+ Hits            207      225     +18
+ Misses          286      268     -18
  Partials         21       21

Impacted Files
Coverage Δ
Complexity Δ

...main/java/com/redhat/ipaas/api/v1/rest/Getter.java
0% <ø> (ø)
0 <ø> (ø)
:x:

...ain/java/com/redhat/ipaas/runtime/Application.java
33.33% <ø> (+33.33%)
1% <ø> (+1%)
:white_check_mark:

...ain/java/com/redhat/ipaas/runtime/CorsFeature.java
77.77% <ø> (+77.77%)
3% <ø> (+3%)
:white_check_mark:

...at/ipaas/runtime/InfinispanCacheConfiguration.java
100% <ø> (+100%)
2% <ø> (+2%)
:white_check_mark:

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 07f2396...3382a2e. Read the comment docs.

[merge]

Thanks @rhuss!


Expose HostConfiguration type to Compose through Treehouse

This will hold more in the future such as the locale, pixel density, etc. For now we propagate whether the host is in dark mode to get things started.
https://user-images.githubusercontent.com/66577/192565428-61c9db20-764c-40bd-b46b-cdacb343dab3.mov
Refs #355

Really nice


Failed to read schema document 'dbchangelog-3.8.xsd', because 'file' access is not allowed due to restriction set by the accessExternalSchema property

Hi
I have faced an issue with reading  local schema file  in  liquibase  version 4.8 and 4.9
Liquibase Version: 4.9.0
Liquibase Community 4.9.0 by Liquibase

Unexpected error running Liquibase: Error parsing line 3 column 55 of ./db.changelog.xml: schema_reference: Failed to read schema document 'dbchangelog-3.8.xsd', because 'file' access is not allowed due to restriction set by the accessExternalSchema property.

For more information, please use the --log-level flag

We store schema file locally  (XMLSchema\dbchangelog-3.8.xsd) to be able run liquibase "update" without internet access
and in change log we reference to this location
Here is example of changeLog
<databaseChangeLog xmlns="http://www.liquibase.org/xml/ns/dbchangelog"
    xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
    xsi:schemaLocation="http://www.liquibase.org/xml/ns/dbchangelog XMLSchema/dbchangelog-3.8.xsd"
    logicalFilePath="db.changelog.xml">

    <include file="changelog_db.xml" relativeToChangelogFile="true"/>

</databaseChangeLog>

There is no such issue in version 4.7.1.
Could you have a look?

Hi @kirillka415,
In 4.8 we introduced a new liquibase.secureParsing parameter. You can read a bit more about it in in the docs, also a good description in the release notes for https://github.com/liquibase/liquibase/pull/2558.
Let me know if this doesn't resolve things on your end.
Thanks much!

Thank you, @kataggart
It works for me.
I added liquibase.secureParsing parameter to the property file
liquibase.secureParsing=false

Is there a way to disable it programmatically? (Java)

How do you set liquibase.secureParsing in a Spring Boot project?

Since this issue is closed, we are no longer actively monitoring it. I opened a thread on the forum so community folks can help you all out.

How do you set liquibase.secureParsing in a Spring Boot project?

Did you get where to set liquibase.secureParsing in a Spring Boot project?

Actually I tried it with 4.9.1 and it didn't pick up the parameter seemingly, didn't solve the issue. Increased liquibase version to 4.25.0 and it works now.
@mukundajvcs I guess I'm late, but you have to provide it as VM argument:
bash gradlew bootRun  -Dliquibase.secureParsing=false or in idea set it in "VM Options" or "Program arguments" in the run config.


Remove trailing whitespaces and a false claim in the JavaScript Style Guide

The claim that multi-line strings are not part of ECMAScript is
false. See the "LineContinuation" production in:
http://www.ecma-international.org/ecma-262/5.1/#sec-7.8.4.
Removed the false claim and a few trailing whitespaces.

@andybons Can you merge this? Thanks!


I need a freelance job.

Hello everyone.
I am android developer.
I need a freelance job.
My skype name is "live:.cid.63b9628cbd4ce2d4"
thank you.

邮件已收到


AIS 目标转视觉目标

请教一下， camera_param.txt里面的camera_para[7] camera_para[8]  camera_para[9]  camera_para[10],这几个参数的含义是什么？ 我看代码里也有计算target_x, target_y， 但是这部分代码屏蔽了，是什么原因呢？

Thank you for your question!
The 7th and 8th elements in camera_para are the focal length on the x-axis and y-axis, respectively. Similarly, the 9th and 10th elements represent the principal point index in the image domain. Since the error occurred during the camera production, the principal point is not the middle point of a picture. These parameters are all estimated by the zhengyou zhang's calibration. Regarding the coordinate transformation, please refer to the image coordinates to the world coordinates formula.

谢谢你的解答！


Duplicate Email Validation

Ensure emails are unique across all users to prevent conflicts and potential errors in user identification. Add a unique=True constraint to the email field in the database schema. Implement service-level checks to catch duplicate emails before saving or updating user profiles. Return a clear, user-friendly error message for duplicate email attempts.

Passed the tests in other tests


Explain a little more about Shell.JS

Hi!
Cash looks really cool, thanks for making it!
This PR just adds a little more explination to the Shell.JS section about the difference between ShellJS and Cash.

cc @nfisher.

cc @nfischer

For what it's worth, I like it.
Also, @dthree, feel free to open a PR on our project or modify the wiki pointing toward cash. I recently opened the wiki to point toward projects based off shelljs, and it might be good to have a page for other related projects, such as cash.

Thank you, this will be very helpful for people new to either project trying to figure out where they both fit and when they would want to use one or the other!

This looks great! Thanks!

@nfischer

feel free to open a PR on our project or modify the wiki pointing toward cash.

Thank you!

Added another link higher-up in the programmatic section:
https://github.com/dthree/cash/commit/63e1a727aac64742f2b4890e16b495970593b78d

:+1:


Peer Review 2

Hello Team,
I am starting with the peer review Assignment 2. I believe you have Jenkins setup done. Please update the documentation and provide credentials via email.
Thank you.

It is now updated.


No support for MongoDB 6.0 on Ubuntu 22.04 (jammy)

Affected Puppet, Ruby, OS and module versions/distributions

Puppet: 6.28
Ruby: ?
Distribution: Ubuntu 22.04 jammy
Module version: 4.2.0

How to reproduce (e.g Puppet code you use)
Try to install MongoDB 6.0
What are you seeing
Nothing is installed. Wrong APT-source with no key installed.
(Maybe need too look into new way to install keys)
What behaviour did you expect instead
Support for MongoDB 6.0

@rogermartensson https://github.com/voxpupuli/puppet-mongodb/pull/677
If possible can you give a hand to verify this PR on Ubuntu ?
(I only tested it on redhat based distro's)

Late-Reply-Reason: Vacation
I will take a look. My guess there will be some differences.

I've started to look into this and to begin with I started to look into using Ubuntu 20.04 and MongoDB 4.4 which is what we are running in production right now.
This to look into if I need to change any of our own Puppet code in this deployment.
I am having the same type problems setting up at replica set as reported in your PR.
Somehow it will not initialize the replicatset even if it says it does.
Notice: /Stage[main]/Profile::Mongodb::Node/Mongodb_replset[rs1]/ensure: created

and it will error out with:
Error: /Stage[main]/Profile::Mongodb::Node/Mongodb_replset[rs1]: Could not evaluate: Can't connect to any member of replicaset rs1.

Before this message and before the 'created' message I get this message three times.
Debug: Request failed: 'Execution of '/usr/bin/mongosh admin --quiet --host mongodbnode1.example.org --eval "load('/root/.mongoshrc.js'); EJSON.stringify(rs.status())"' returned 1: MongoServerError: no replset config has been received' Retry: '3'

  class { 'mongodb::globals':
    version             => '4.4.15'
    manage_package_repo => true,
    use_enterprise_repo => false,
  } -> class { 'mongodb::client':
  }

    class { 'mongodb::server':
      dbpath        => $db_path,
      bind_ip       => ['<IP_ADDRESS>'],
      port          => 27017,
      verbose       => true,
      replset       => 'rs1',
      # set_parameter => $c,  # set_parameter used to insert enableLocalhostAuthBypass: false
      require       => Class['mongodb::globals'],
    }

# To create the replica set I use
        mongodb_replset { 'rs1':
          ensure  => present,
          members => $nodenames,
        }

I have tried using MongoDB 6 and it seems to install it correctly but it has the same problem as described above so I didn't test it very deeply.

Did some more testing and it looks like it may work if I, after a new install, manually run rs.initiate() on the first node.
More testing needed.

There need some more work on this PR indeed.  Hadn't that much time recently, but picking this up again ...
Thanks for the feedback and testing.
Installing a new cluster right now on redhat.  Will pay close attention about the  rs.initate() ...

Hello,
Same issue with MongoDB 7x, isn't it ?

Error: Could not prefetch mongodb_database provider 'mongodb': 783: unexpected token at 'DeprecationWarning: .) is deprecated. Use .setReadPref("primaryPreferred" instead
Setting read preference from "primary" to "primaryPreferred"
{
  databases: [
    {
      name: 'admin',
      sizeOnDisk: '40960',
      empty: false
    },
    {
      name: 'config',
      sizeOnDisk: '12288',
      empty: false
    },
    {
      name: 'local',
      sizeOnDisk: '73728',
      empty: false
    }
  ],
  totalSize: '126976',
  totalSizeMb: '0',
  ok: 1
}
'
Warning: /Stage[main]/Mongodb::Server/Mongodb::Db[admin]/Mongodb_database[admin]: Skipping because provider prefetch failed
Notice: /Stage[main]/Mongodb::Server/Mongodb::Db[admin]/Mongodb_user[User admin on db admin]: Dependency Mongodb_database[admin] has failures: true
Warning: /Stage[main]/Mongodb::Server/Mongodb::Db[admin]/Mongodb_user[User admin on db admin]: Skipping because of failed dependencies
Warning: /Stage[main]/S_mongodb::Zabbix/S_mongodb::Users[zabbix]/Mongodb_user[Create user zabbix on database admin]: Skipping because of failed dependencies
Info: Stage[main]: Unscheduling all events on Stage[main]

I will be picking up https://github.com/voxpupuli/puppet-mongodb/pull/677 shortly since we need this working properly for work.  Feel free to add comments to the PR of any issue/wishes you have ...

I tried to set up a new monbodb cluster. Latest pull-request had some issues but I left a comment in the PR.
There is still problems with Ubuntu 22.04 (jammy) to set up a new cluster. It doesn't seem to initialize it. The workaround is to manually run rs.initiate() and everything after that seem to work quite ok.

Did some more testing when hitting a Nil-issue. With limited knowledge in puppet module development and ruby I tracked down the problematic code to be this in lib/provider/mongodb.rb.
    if $config['auth_mechanism'] && $config['auth_mechanism'] == 'x509'
      args.push("--authenticationDatabase '$external' --authenticationMechanism MONGODB-X509")
    end

This does not look like the rest of the code in the file.

Thank for testing.  Just have to finish some small projects @work, will look into this after that..  Your comments are very helpfull to get this working properly

Thanks for working on this.. Also need it. :)
I am trying to understand how the rs.initiate() is called in the module and the only way I got it try to run was to enable authentication.
I still haven't to manage to set up a running cluster this way yet. I got a few steps closer when I found the 'store_creds' parameter and set it to true.
Still the "simplest" way to do it is to use 'auth = false', pick one node as the main one and run the create_admin on that node. Then manually rs.initate() and after that do the rest of the stuff you need to do finishing off with enabling authentication.
So.. not easy. Have no idea at the moment how this could be streamlined. Maybe my lack of insight in MongoDB stops me to see the full picture.

I did had it working, on redhat ... but I guess during a rebase or cleaning up the commits (which were a bit of a mess I must admit) I reverted that ...  I hope to get more time next week to delve in it again ...

Found something that doesn't have anything with cluster setup but may be a security issue. When using the "store_creds = true" the diff will be shown while applying the catalogue. Ie.. the password will be shown in the logs. So that might be a good fix to have in your PR.

I noticed that you added tls_mode as a paramter to server.pp. Nice.
Although, since I do not use TLS at all, the default of requireTLS is a bit strict?
If not it should be documented, but I guess you already have that on your todo.

@rogermartensson could you verify if release 6.0.0 is solving this?
Regarding the issue with exposed password in logs, would be nice if you could report that as a separate issue.
When it comes to setting up replica sets, there is some integration tests in the git repo at https://github.com/voxpupuli/puppet-mongodb/blob/master/spec/acceptance/replset_spec.rb.

Sorry for being superlate. I haven't done any large testing around this and don't think I will have time to do any deep dives currently.
If I get some time to do some better testing I'll return in another issue if I notice anything.


Fix randomness of servers shuffling

@samuel please review.

Coverage increased (+0.1%) to 81.31% when pulling a260e372d2bb5ce6df14acaf09d4b25021e21ff1 on tevino:patch-1 into 1d7be4effb13d2d908342d349d71a284a7542693 on samuel:master.

Coverage increased (+0.01%) to 81.182% when pulling a260e372d2bb5ce6df14acaf09d4b25021e21ff1 on tevino:patch-1 into 1d7be4effb13d2d908342d349d71a284a7542693 on samuel:master.

The seed of global Rand object is not determined, the value set here could be overrode by anyone.
IMHO: Libraries should not rely on or even modify the global seed.


chore(cspell): add spell-check-all.yaml

Description
Currently, spell-check-differential is one of the CI executed by pull requests as a trigger.
However, since there are too many cspell errors, the error is almost ignored.
Also, spell-check-differential cannot provide the full picture of the spell error in the repository.
Therefore, in order to accelerate the efforts to fix typo, I propose this workflow that can continue to track the entire cspell errors.
Tests performed

Not applicable.
Effects on system behavior

Not applicable.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

Thanks!


Refactoring of the entire component and additional features

I just refactored the entire component. I'll appreciate if you'll accept this PR otherwise we can keep components separated and I'll create a new repo for my version ;)
This PR adds

Server-side rendering is now possible since the googleApi is initialized on componentDidMount
GoogleApi simple requests cache
Props validation
A better way of organize the code ( this is just my point of view )
Fix BEM classes in GeosuggestItem
Moved React as peerDependency

Actually I just gave a fast check if things are working. Would be nice if you could test it as well.
P.S. I just read later your guidelines for commit message style. Sorry for that

Hi,
you mixed up a lot of issues in one PR. That’s a bit hard to review and keep track of unexpected changes. Also, on a first view, there are a few breaking changes. I will not merge this PR as a whole, but we can add these things one by one. See my notes to your additions beneath. Also note that we have ES Lint and tests fail when it is not satisfied. (Not sure why Travis Checks don’t appear here.)
I don’t think we need or want a fork. Willing to get your changes into this component.

Server-side rendering is now possible since the googleApi is initialized on componentDidMount
--> Talking about that in #23. Will be implemented.
GoogleApi simple requests cache
--> Good idea, I’d like to add that.
Props validation
--> Sounds good, too.
A better way of organize the code ( this is just my point of view )
--> What purpose does a restructure satisfy? For the user of the component it doesn’t matter and it’s possible that it’ll introduce bugs. If we had tests, that are lacking in this project, it would be easier to do so. Also, we @ubilabs have our code style. Please respect that when contributing. (See ES Lint errors when running npm test.)
Fix BEM classes in GeosuggestItem
--> What’s the problem of the current structure? Kept them as individual modules, that’s why the items are individual.
Moved React as peerDependency
--> Like that.

Hi @ro-ka! I also noticed there are some other bugs on my code besides breaking changes.
I agree with you that this PR should not be merged but most of the changes that are should be implemented to make this component better. So I pushed this PR to see if you like my idea and if we can work together on this.
I'm gonna open an issue for each of those points so that we can track discussions in a better way

That’s great!  :)

Ok done! Let's discuss on those issues and take this PR as inspiration ;)

Closing this as there are separate issues.


Fixed Invoke-JenkinsJobReload to use invoke-jenkinscommand

This change is

Can you add a new entry to an ### Unreleased section in the readme.md for me?
Cheers @liambinnsconroy
Nice job as always though sir!

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Reviewed 1 of 1 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


[Feature][workflow list edit] add task update with upstream

#7018

Codecov Report

Merging #7829 (bdc5446) into dev (d2794be) will decrease coverage by 0.23%.
The diff coverage is 7.96%.

@@             Coverage Diff              @@
##                dev    #7829      +/-   ##
============================================
- Coverage     41.11%   40.88%   -0.24%
+ Complexity     3691     3684       -7
============================================
  Files           637      637
  Lines         26934    27020      +86
  Branches       3042     3063      +21
============================================
- Hits          11075    11048      -27
- Misses        14795    14906     +111
- Partials       1064     1066       +2

Impacted Files
Coverage Δ

...duler/api/controller/TaskDefinitionController.java
3.22% <0.00%> (-0.23%)
:arrow_down:

...he/dolphinscheduler/dao/entity/TaskDefinition.java
64.34% <0.00%> (-1.14%)
:arrow_down:

...ache/dolphinscheduler/dao/entity/TaskMainInfo.java
0.00% <0.00%> (ø)

...er/api/service/impl/TaskDefinitionServiceImpl.java
22.04% <8.08%> (-4.10%)
:arrow_down:

.../org/apache/dolphinscheduler/api/enums/Status.java
100.00% <100.00%> (ø)

...er/master/processor/queue/TaskResponseService.java
50.00% <0.00%> (-18.19%)
:arrow_down:

...r/plugin/registry/zookeeper/ZookeeperRegistry.java
48.21% <0.00%> (-8.04%)
:arrow_down:

...rver/master/processor/queue/TaskResponseEvent.java
93.54% <0.00%> (-6.46%)
:arrow_down:

...dolphinscheduler/remote/future/ResponseFuture.java
81.96% <0.00%> (-1.64%)
:arrow_down:

Continue to review full report at Codecov.

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


Next version of CollectionKit (v2.0.0), first PR

CollectionKit becomes my personal favourite framework for building iOS app.
It is definitely a huge time saver.
To make it even better, here are some more improvements that I would like to add to the framework.
Note this version won't be source compatible with 1.3.0. It is easy to migrate though. A migration guide will be written before releasing 2.0.0.
Focus of 2.0.0 will be towards API cleanliness & solves pain points with v1.0+
Main feature planned for 2.0.0:

Sticky Header Support
Better support for mixed data type
Layout cleanup
Internal architecture cleanup for future features

This PR introduces the first set of changes towards 2.0.0:

Layout becomes data independent.

previously, FlowLayout<Int> -> FlowLayout

Layout only receives an abstract object LayoutContent during layout, instead of receiving dataProvider & sizeProvider.
Move flattening step out of CollectionComposer so that it is easy to create other composer variants.
A new CollectionHeaderComposer to support header & sticky header
Add HeaderExampleViewController to demonstrate header & sticky header support
Add ViewProviderComposer to better support mixed data type for a single CollectionProvider

Not entirely sure this is the best solution. Advices welcome.

This PR is still WIP. Posting for review first.

closing this, work will be done to v2.0 branch

Codecov Report

Merging #66 into master will decrease coverage by 5.27%.
The diff coverage is 45.6%.

@@            Coverage Diff             @@
##           master      #66      +/-   ##
==========================================
- Coverage   55.11%   49.83%   -5.28%
==========================================
  Files          34       38       +4
  Lines        1056     1222     +166
==========================================
+ Hits          582      609      +27
- Misses        474      613     +139

Impacted Files
Coverage Δ

Sources/Addon/SpaceCollectionProvider.swift
0% <ø> (ø)
:arrow_up:

Sources/Addon/EmptyStateCollectionProvider.swift
0% <ø> (ø)
:arrow_up:

Sources/Layout/InsetLayout.swift
0% <0%> (ø)
:arrow_up:

Sources/Provider/EmptyCollectionProvider.swift
8.69% <0%> (ø)

Sources/ViewProvider/CollectionViewProvider.swift
50% <0%> (-33.34%)
:arrow_down:

Sources/Layout/WaterfallLayout.swift
0% <0%> (ø)
:arrow_up:

Sources/Layout/OverlayLayout.swift
0% <0%> (ø)
:arrow_up:

Sources/Provider/CollectionHeaderComposer.swift
0% <0%> (ø)

Sources/Layout/VisibleFrameInsetLayout.swift
0% <0%> (ø)
:arrow_up:

Sources/Layout/ClosureLayout.swift
0% <0%> (ø)
:arrow_up:

... and 19 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 54f1d95...41d6882. Read the comment docs.


Add semester planning meeting notes

@jonleung raised some valid concerns that we may not be looking at goals at the right level. I think it may be worth revisiting this in mid-November, but these are a great start.


Save Metadata automatically

It should save the metadata changes automatically if i see the checkmark. And take over the title to the Cutting window.

The metadata is intentionally not saved automatically. This is to reflect how it will work in the Admin UI in Opencast in the future, and to avoid any confusion on whether something does and does not get saved automatically. (Currently nothing is saved automatically).
We probably need to communicate the meaning of the checkmark better to the user (the input was recognized and is valid), or just leave it out altogether (only tell the user when they made a mistake).
Taking over changes to the title was done in #317.

We probably need to communicate the meaning of the checkmark better to the user (the input was recognized and is valid), or just leave it out altogether (only tell the user when they made a mistake).

As an idea for that: Just reverse this and add a red X if a field is not valid and nothing if it is.


Convert bestman2 overview to Markdown.

Mostly an automated conversion, with some updated links.

[ ] Add migration header to TWiki document

I think it's appropriate:

Obsolete = "we do not support this, you should have already retired it."
Deprecated = "this is on the path to obsolete in the future, probably not a good idea to start a new service"

So, anything that's not in the current release series (3.4) is de-facto deprecated.  We should just make that apparent.

Replaces this document: https://twiki.grid.iu.edu/bin/view/Documentation/Release3/BestmanOverview


how to change api-url from MyVuetable.vue

Hi,
I am trying to learn vuetable-2, I followed the tutorial and created the basic MyVuetable.vue.
In my page I have  . How can I set api-url from my index.html ?
I tried <my-vuetable api-url="something"></my-vuetable> and <my-vuetable :api-url="something"></my-vuetable> both didnt work, it only work if I set the api-url from MyVuetable.vue.
Thanks.

@rained23 You will have to define api-url as a prop in MyVuetable, then you could pass it in index.html, like so
// MyVuetable.vue
<template>
  //...
  <vuetable ref="vuetable"
      :api-url="apiUrl"
      //...
  ></vuetable>
</template>

<script>
//...
export default {
  //...
  props: ['apiUrl'],
  data () {
    //...
  }

  <my-vuetable
    api-url="...url here..."
  ></my-vuetable>

Thanks for your help


Change cell span dynamically

Hi,
I was wondering if it is possible to change the column span of a cell dynamically as you change the flow in the documentation: https://github.com/exyte/Grid#columns?

@codertimu Yes, it's possible. Cell spans are changed in the Calculator example.


[Help request] unable to show data in grid

I'm trying to have information via controller, everything looks fine, i also can access data from ondataLoaded but stil not visible in grid.
here is my code:
promise to get info
let myFirstPromise = new Promise((resolve, reject) => {      ipc.send('get');     ipc.on('get',(e, result)=>{             if(result.err){         reject(result.err)       } else {         resolve(result.result)       }     })   })
`$("#jsGrid").jsGrid({
height: "70%",
width: "100%",
filtering: true,
inserting: true,
editing: true,
sorting: true,
paging: true,
autoload: true,
pageSize: 10,
pageButtonCount: 5,
deleteConfirm: "Do you really want to delete client?",
fields: [
{ name: "name", type: "text", width: 150 },
{ type: "control" }
],
controller: {
loadData: function(filter) {
// ipc.send('data');
return myFirstPromise
    },
    insertItem: function(item) {
        return $.ajax({
            type: "POST",
            url: "/clients",
            data: item
        });
    },
    updateItem: function(item) {
        return $.ajax({
            type: "PUT",
            url: "/clients",
            data: item
        });
    },
    deleteItem: function(item) {
        return $.ajax({
            type: "DELETE",
            url: "/clients",
            data: item
        });
    }
}, onDataLoaded: function(data) {
  console.log(data);
}

});`
data logged in from onDataLoaded
Object {data: Array(2), grid: d}
Please help

Data object  should be {data:[],itemsCount:0}

@heiovsky  done but still issue persists

Put this on jsfiddle and i will take a look
pt., 25.05.2018, 18:04 użytkownik Imran Khan<EMAIL_ADDRESS>napisał:

@heiovsky https://github.com/heiovsky done but still issue persists
[image: image]
https://user-images.githubusercontent.com/17918895/40554436-21e9d156-605f-11e8-84b4-ea139aa7a19e.png
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tabalinas/jsgrid/issues/1126#issuecomment-392104966,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQbIfQdhy6p8DIc9JzcMUz_AQuO6s--6ks5t2Ct4gaJpZM4UOHB6
.

@heiovsky  thanks for your support here it is.
https://jsfiddle.net/f5fd11xn/

https://jsfiddle.net/f5fd11xn/3/

@heiovsky  Many thanks dear, you are an angel for me :) it's working now

Thanks to @heiovsky , great support


Publish extension methods to access MutableDocument and MutableComposer from Editor for convenience. (Resolves #2157)

Publish extension methods to access MutableDocument and MutableComposer from Editor for convenience. (Resolves #2157)
This PR doesn't contain any behavior changes - it just adds extension APIs to make it easier to access the MutableDocument and MutableComposer in the EditContext. The standard access lets developers query for any possible Editable, and there's no obvious place to learn about the existence of document or composer. So these methods now appear in intellisense.

@angelosilvestre I'm gonna merge this so I can keep moving with some refactors. Feel free to post any followup comments.


[DEVX-1046] Print browser version for docker

Proposed changes

Types of changes

Add browser version when reporting test results.

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)

Checklist

[x] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Further comments

The base testrunner-image Docker image already has labels defining the browser versions installed:
https://github.com/saucelabs/testrunner-image/blob/176727a000876bf4b3421f261014cd1508f83872/Dockerfile#L334
https://github.com/saucelabs/testrunner-image/blob/176727a000876bf4b3421f261014cd1508f83872/Dockerfile#L349
But those labels are not accurate for the playwright image since playwright bundles its own browsers. We'll need to include new labels for the playwright browsers if this is the implementation we want to move forward with.

@mhan83 The titel still reads excluding playwright, but I think this now includes playwright as well, right?

Correct! Fixed the title, for future posterity.


Resource local caches invalidation

Description
This PR adds the building blocks for local per-session-resource cache invalidation within the cluster and uses them to implement a hazelcast-topic based invalidator for session in memory catalogs.
Testing

[x] Unit

LGTM

+1


Doesn't render data from one collection but does from other

I am building a simple reporting tool to see data about Quotes and Jobs in both tabular and chart form.  I am using this package along with the maazalik:highcharts package.
I have two views, one to see jobs and filter by user and one to see customers and their totals.
The jobs one has all worked perfectly as expected but when I try to create a table reading from the 'Customers' collection it just says "No data available in table"  this is despite there being a chart drawn on the same view with data, that uses the same selector on the same collection.
I have done a lot of tweaking and reduced the table down to just one column and it works fine with the 'Quotes' collection. As soon as I switch to the 'Customers' collection, it's data disappears.
What's the best way I can debug to see what data is actually being received?

// the following displays no data:

TabularTables = {};

Meteor.isClient && Template.registerHelper('TabularTables', TabularTables);

TabularTables.Customer = new Tabular.Table({
    name: "Report",
    collection: Customers,
    columns: [{
        data: "company",
        title: "Customer"
    }]
});

Check if your data is really published in with the Meteor Method in Browser Console like
Customers.find().fetch()

I'm not sure what this would be without seeing a full reproduction.

Closing pending full reproduction


Versions 4.0.1 and 4.1.0 missing from GitHub releases

Versions 4.0.0 and below are on GitHub releases. Versions 4.0.1 and 4.1.0 are missing, however.

Git tags (e.g.: with source) are available here: https://github.com/pimutils/todoman/tags
Binary packages are also available at PyPI: https://pypi.org/project/todoman/#history
GitHub releases were just copies of the files in PyPI and provide no extra value. GH also auto-generates some broken zips and tars that are more of a confusion than anything else.

Git tags are signed, so for packaging and scripting are the recommended source.

I realize the files are available elsewhere. However, since all the previous releases are here on GitHub, I assumed newer ones would be too, and therefore that watching the project would be a good way to get an e-mail on a new release. Which didn’t work. Maybe it would make sense to announce somehow, perhaps in the README, that GitHub releases are not going to be used going forward? Sure, I’ve now pointed myself at the RSS on PyPI—it’s not that I couldn’t do that, it’s that I didn’t know I needed to because it looked like watching the GitHub project instead would provide the same information, and that was slightly easier (actually much easier because at the time, years ago, when I started using todoman, I didn’t have an RSS system set up).

People do watch for releases, you're quite right.
I'll try and push an extra one, and include a note of how to watch for new releases in future.


[Question] Are you planning to provide Firebase C++ for Mac Catalyst?

I found out you guys don't provide a library for Mac Catalyst.
But in apple.toolchain.cmake I see you still support building libraries for Mac Catalyst.
So, are you planning to provide the library for Mac Catalyst? Or do users have to build their own libraries when needed?
Thanks

Hi @ryuw0n
Firebase iOS SDK rolled out macOS Catalyst beta support from 8.9.0
https://firebase.google.com/support/release-notes/ios#version_890_-_october_28_2021
However, C++ SDK does not support this yet.
Let me turn this into FR.  If anyone is interested in this, please upvote.
Shawn


Validate make start performance

This issue is meant to collect benchmarks from running 'make start' with a clean cache.
Process:

Prune all images from docker cache
Set .env version to 'local'
Run make start

Ubuntu
Processor: 11th Gen Intel® Core™ i9-11950H @ 2.60GHz × 16
Memory: 64,0 GiB
I've done 2 tests:

Test 1
docker system prune -a
docker compose build

Total of 34m

Test 2 (changed version to local)
docker system prune -a
make start

Total of  32m

Huh, this seems like a weird outlier. My Windows build was on a 9th Gen Intel chip... I assume this is a laptop?

Yep, laptop

Hi folks! Here are some tests with my machine:
OS: MacOs Ventura 13.5.2
Computer: iMac 2017
Processor: 3,4 GHz Quad-Core Intel Core i5
Memory: 24,0 GiB
Internet speed: 218 Mbps
Ran docker system prune -a
Test 1 - time docker compose build
Total: 24m 36s
Test 2 - time make start
Total: 7m 44s
Remark: I'm not sure if I set the env as local, I'm using the following vars:

Ubuntu
Processor: 11th Gen Intel® Core™ i9-11950H @ 2.60GHz × 16
Memory: 64,0 GiB

docker system prune -a

docker compose build

Total of 34m

When using the internet at the office, I'm down to 8m30s, so it seems that my home internet just sucks 😅

Closing this as part of issue cleanup. If there's anything that needs to be done, please create a new issue.


publicsuffix-list: 2019-05-24 -> 2021-09-03

Motivation for this change
The publicsuffix-list data file in Nix is pretty out of date. Trying to fix this long-term with occasional updates is probably not all that great (vs. keeping it up-to-date on the target machine by refetching), but this is definitely the easiest way to get another update in.
Things done
I have tested building the package on Ubuntu. I have not done a full recompilation of all packages depending on this package because a) it's an update to a single data file and b) it affects a ton of reverse dependencies.

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Maybe target this to staging as this is a mass rebuild?

Good point, done!

Broke libpsl's tests. Looks like .sh changed domain status?
$ nix build -f. -L libpsl
...
libpsl> FAIL: test-is-public-builtin
...
libpsl> have 9184 suffixes and 8 exceptions
libpsl> psl_is_public_suffix(whatever.platform.sh)=0 (expected 1)
libpsl> psl_is_public_suffix(.platform.sh)=0 (expected 1)
libpsl> psl_is_public_suffix2(whatever.platform.sh, NO_STAR_RULE)=0 (expected 1)
libpsl> psl_is_public_suffix2(.platform.sh, NO_STAR_RULE)=0 (expected 1)
libpsl> psl_builtin_file_time()=1
libpsl> psl_builtin_sha1sum()=a596eaf9f72870f1e0c6ffa89eaa12252aab2ccc

Old https://github.com/publicsuffix/list/blob/a1db0e898956e126de65be1a5e977fbbbbeebe33/public_suffix_list.dat#L12480-L12483 :
// Platform.sh : https://platform.sh
// Submitted by Nikola Kotur<EMAIL_ADDRESS>*.platform.sh
*.platformsh.site

Now:
// Platform.sh : https://platform.sh
// Submitted by Nikola Kotur<EMAIL_ADDRESS>bc.platform.sh
ent.platform.sh
eu.platform.sh
us.platform.sh
*.platformsh.site
*.tst.site

I would patch the like *.platform.sh -> *.platformsh.site.

Oh, I also found https://github.com/rockdaboot/libpsl/commit/f364cea73e351ce62e0b337fd1fbc21e70b52d56 upstream fix. Should already be in most recent libpls.

Proposed bump as https://github.com/NixOS/nixpkgs/pull/146255


Fix Releases show requests where app name contains a request format

Rails magic was causing requests where the repository name included a valid format (e.g. 'reponame.js') to error.

:+1:


Dynamically set version label values as part of the CI

In https://github.com/redhat-openshift-ecosystem/operator-certification-operator/blob/main/Dockerfile the labels version and release need to get values dynamically as part of the GH Action main build and release build.

/assign


Error detected! Won't run

Attempting to run charl-e on my M1 macbook pro and it fails to move past initialization.
Logs show:
usage: txt2img [-h] [--prompt [PROMPT]] [--outdir [OUTDIR]] [--skip_grid] [--skip_save] [--ddim_steps DDIM_STEPS] [--plms] [--laion400m] [--fixed_code] [--ddim_eta DDIM_ETA] [--n_iter N_ITER] [--H H] [--W W] [--C C] [--f F] [--n_samples N_SAMPLES] [--n_rows N_ROWS] [--scale SCALE] [--from-file FROM_FILE] [--config CONFIG] [--ckpt CKPT] [--seed SEED] [--precision {full,autocast}] txt2img: error: unrecognized arguments: -B -S -E -s -c from multiprocessing.resource_tracker import main;main(9)
transformers/models/clip/feature_extraction_clip.py:67: DeprecationWarning: BICUBIC is deprecated and will be removed in Pillow 10 (2023-07-01). Use Resampling.BICUBIC instead. transformers/models/clip/feature_extraction_clip.py:204: DeprecationWarning: BICUBIC is deprecated and will be removed in Pillow 10 (2023-07-01). Use Resampling.BICUBIC instead.
transformers/image_utils.py:188: DeprecationWarning: BILINEAR is deprecated and will be removed in Pillow 10 (2023-07-01). Use Resampling.BILINEAR instead.
torch/_jit_internal.py:751: UserWarning: Unable to retrieve source for @torch.jit._overload function: <function _DenseLayer.forward at 0x168f6b520>. warnings.warn(f"Unable to retrieve source for @torch.jit._overload function: {func}.") torch/_jit_internal.py:751: UserWarning: Unable to retrieve source for @torch.jit._overload function: <function _DenseLayer.forward at 0x168f6b6d0>. warnings.warn(f"Unable to retrieve source for @torch.jit._overload function: {func}.")
torchvision/io/image.py:13: UserWarning: Failed to load image Python extension:
[2111] WARNING: file already exists but should not: /var/folders/nc/n0_cfz7n01bbqqj17x5g39xr0000gn/T/_MEItcVSBe/torch/_dl.cpython-310-darwin.so
[2111] WARNING: file already exists but should not: /var/folders/nc/n0_cfz7n01bbqqj17x5g39xr0000gn/T/_MEItcVSBe/torch/_C_flatbuffer.cpython-310-darwin.so
[2111] WARNING: file already exists but should not: /var/folders/nc/n0_cfz7n01bbqqj17x5g39xr0000gn/T/_MEItcVSBe/torch/_C.cpython-310-darwin.so
[2111] WARNING: file already exists but should not: /var/folders/nc/n0_cfz7n01bbqqj17x5g39xr0000gn/T/_MEItcVSBe/pyarrow/lib.cpython-310-darwin.so
multiprocessing/resource_tracker.py:104: UserWarning: resource_tracker: process died unexpectedly, relaunching. Some resources might leak.
RuntimeError: expected scalar type BFloat16 but found Float [2092] Failed to execute script 'txt2img' due to unhandled exception!
DDIM Sampler: 0%| | 0/5 [00:04<?, ?it/s] data: 0%| | 0/1 [01:19<?, ?it/s] Sampling: 0%| | 0/1 [01:19<?, ?it/s]
DDIM Sampler: 0%| | 0/5 [00:00<?, ?it/s]
usage: txt2img [-h] [--prompt [PROMPT]] [--outdir [OUTDIR]] [--skip_grid] [--skip_save] [--ddim_steps DDIM_STEPS] [--plms] [--laion400m] [--fixed_code] [--ddim_eta DDIM_ETA] [--n_iter N_ITER] [--H H] [--W W] [--C C] [--f F] [--n_samples N_SAMPLES] [--n_rows N_ROWS] [--scale SCALE] [--from-file FROM_FILE] [--config CONFIG] [--ckpt CKPT] [--seed SEED] [--precision {full,autocast}] txt2img: error: unrecognized arguments: -B -S -E -s -c from multiprocessing.resource_tracker import main;main(9)
transformers/models/clip/feature_extraction_clip.py:67: DeprecationWarning: BICUBIC is deprecated and will be removed in Pillow 10 (2023-07-01). Use Resampling.BICUBIC instead. transformers/models/clip/feature_extraction_clip.py:204: DeprecationWarning: BICUBIC is deprecated and will be removed in Pillow 10 (2023-07-01). Use Resampling.BICUBIC instead.
transformers/image_utils.py:188: DeprecationWarning: BILINEAR is deprecated and will be removed in Pillow 10 (2023-07-01). Use Resampling.BILINEAR instead.
torch/_jit_internal.py:751: UserWarning: Unable to retrieve source for @torch.jit._overload function: <function _DenseLayer.forward at 0x16b5fb520>. warnings.warn(f"Unable to retrieve source for @torch.jit._overload function: {func}.") torch/_jit_internal.py:751: UserWarning: Unable to retrieve source for @torch.jit._overload function: <function _DenseLayer.forward at 0x16b5fb6d0>. warnings.warn(f"Unable to retrieve source for @torch.jit._overload function: {func}.")
torchvision/io/image.py:13: UserWarning: Failed to load image Python extension:
[2094] WARNING: file already exists but should not: /var/folders/nc/n0_cfz7n01bbqqj17x5g39xr0000gn/T/_MEIBcxlVe/torch/_dl.cpython-310-darwin.so
[2094] WARNING: file already exists but should not: /var/folders/nc/n0_cfz7n01bbqqj17x5g39xr0000gn/T/_MEIBcxlVe/torch/_C_flatbuffer.cpython-310-darwin.so
[2094] WARNING: file already exists but should not: /var/folders/nc/n0_cfz7n01bbqqj17x5g39xr0000gn/T/_MEIBcxlVe/torch/_C.cpython-310-darwin.so
[2094] WARNING: file already exists but should not: /var/folders/nc/n0_cfz7n01bbqqj17x5g39xr0000gn/T/_MEIBcxlVe/pyarrow/lib.cpython-310-darwin.so

Not sure how to troubleshoot.  Any help would be appreciated.

Hmm, what version of MacOS are you on? It may not work unless you're on 12.0.6

Running MacOS 12.2.1
16inch M1 MacBook pro
16GB memory

Same issues here, however, I left it for a while and it started working.

Upgrading to macOS 12.6 resolved this issue for me

same issue for me on MacOS 13


Zulip URL encoding doesn't work for dots

Rustbot link
https://rust-lang.zulipchat.com/#narrow/stream/233931-xxx/topic/Use.20sane.20defaults.20in.20x.py.20compiler-team.23326
Actual link
https://rust-lang.zulipchat.com/#narrow/stream/233931-t-compiler.2Fmajor-changes/topic/Use.20sane.20defaults.20in.20x.2Epy.20compiler-team.23326
If we compare the Rustbot link (and elaborate the xxx to the full stream name) against the actual link, the only difference that I see is that the actual link encodes the . in x.py as .2E, whereas Rustbot encodes it as just ..
From what I see in the rest of the link, it looks like Zulip encodes special characters as a . and then the ASCII hex code. Because of that, it seems a . must be encoded as its hex code.
Cc https://github.com/rust-lang/triagebot/issues/701#issuecomment-660532531
Cc @Mark-Simulacrum

Relevant code? https://github.com/rust-lang/triagebot/blob/823fd7fe45ce3f86095d29e2a1b0294d69870b07/src/zulip.rs#L337-L359

Seems fixed since #653

I would like to add a test, then, just to make sure we don't regress this bit in the future.


Gracefully catch HAL push errors

hal push can throw the exceptions defined here: https://github.com/inspirehep/python-client-sword2/blob/python3/sword2/exceptions.py. The exceptions should be caught and logged gracefully here: https://github.com/inspirehep/inspirehep/blob/9d3ef0776defeed49edd0892c238606d78babd15/backend/inspirehep/hal/tasks.py#L46-L61

I'm not sure I understand what this issue is about, but it would be nice to log nice messages from the HAL server rather than the full exception, similarly to what is done currently using https://github.com/inspirehep/inspire-hal/blob/f8c8ccfdbedf14e0ba12521116c2c63ba88b2c54/inspire_hal/bulk_push.py#L108-L117


Update Docs: Reference “configuration-options”

Automatically generated by Netlify CMS

Hi @abraithwaite, looks like there is a similar clarification at the beginning of that file https://www.netlifycms.org/docs/configuration-options/

Closing for now, please comment re-open if you think the existing clarification is not enough

Hey Erez, thanks for the review!  I don't think the clarification is sufficient, in particular for the Hugo framework which has a root config.yaml/config.toml file.  When searching for a solution and scanning the docs for a match to an issue, it's easy to miss that minor note at the top.  Maybe the wording can be improved, but I don't think the clarification would hurt to add.
See more context here:
https://netlifycms.slack.com/archives/CPRR0PQ9E/p1588427744312500
Also, I'm not alone in this confusion:
https://github.com/search?q="editorial_workflow"+filename%3Aconfig.toml&type=Code

Thanks for clarifying, re-opened - I'll think a bit on the phrasing so it doesn't look like duplication


Make uncss:ignore survive CSS minifiers

Regex has been updated to allow for an optional, leading "!" which will prevent it from being stripped by CSS minifiers/tools such as grunt-contrib-concat and grunt-cssmin. This will increase the flexibility of using grunt-uncss within various points of the task order.

Coverage remained the same at 100.0% when pulling fb4ef00f40b96a62f50d32009a1fac2cdef2d7ad on levineuland:master into bfc95ad1a61f7b98b123d67ef1b30fa54fbc1fab on giakki:master.

Why would one want this?

In order to use the comment method for uncss to ignore certain selectors it has to be used prior to most concat/minification/preprocessors to ensure the comments are not stripped.
grunt-contrib-concat, clean-css, less, and sass offer option to ignore "loud comments", which are standard comments that start with a "!".
This would provide much more flexibility to use uncss at various points within a build process without having to hard-code css selectors to ignore, making front end frameworks much more portable and reusable from project to project.

@XhmikosR it's a simple change anyway, no harm in adding it ;)


add dtors_in_dtors_in_dtors

That's a pretty neat test from the standard library. Sadly not enough to check for https://github.com/rust-lang/miri/issues/2754, but still worth having here.

@bors r+

:pushpin: Commit 854f21f65b36954f850ad45d5dec760a6210ecfa has been approved by RalfJung
It is now in the queue for this repository.

:hourglass: Testing commit 854f21f65b36954f850ad45d5dec760a6210ecfa with merge 6a682a7da14103bc058768819aed97ad01436551...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing 6a682a7da14103bc058768819aed97ad01436551 to master...


Redux-saga

Have been digging into redux-saga a bit and it can be a little hard to wrap your mind around at first. I think a screencast on this would be pretty helpful.

Cool, thanks for the suggestion. Any reference materials you are aware of?

The redux-saga docs themselves are pretty good http://yelouafi.github.io/redux-saga/, I've also found a few scattered articles:

https://riadbenguella.com/2016/01/20/how-es6-generators-are-changing-the-way-we-write-javascript/
http://jaysoo.ca/2016/01/03/managing-processes-in-redux-using-sagas/
http://konkle.us/master-complex-redux-workflows-with-sagas/


Merged streams don't continue

I'm not sure if the title makes any sense, but I'll do my best to explain.
I've got a gulp task that loops through some folders and runs some processes against all the files in each folder. These get added together with merge-stream. Once each folder has been processed and merged back together in to one stream, I then want to emit a notification to indicate that the task ran and completed. However, it seems that nothing after return MERGED_STREAMS is firing.
My full scripts task can be found on my new-site repository, but this is the relevant bit:
return MERGED_STREAMS
    // prevent breaking on error
    .pipe(plugins.plumber({
        errorHandler: on_error,
    }))
    // notify that task is complete, if not part of default or watch
    .pipe(plugins.gulpif(gulp.seq.indexOf("scripts") > gulp.seq.indexOf("default"), plugins.notify({
        title:   "Success!",
        message: "Scripts task complete!",
        onLast:  true,
    })))
    // push task to ran_tasks array
    .on("data", () => {
        if (ran_tasks.indexOf("scripts") < 0) {
            ran_tasks.push("scripts");
        }
    })
    .on("end", () => {
        console.log("reached the end"); // never happens
        return resolve();
    });

Any idea what could be causing this? I'm at a complete loss.

I figured out a different approach to accomplish what I was trying to do, but would still like to know what the problem is.

I'm not familiar with what most of these gulp things you're using do, but I noticed this:
plugins.gulpif(gulp.seq.indexOf("scripts") > gulp.seq.indexOf("default"), ...)

It seems likely to me that doesn't do what you want, since the conditon is only evaluated once when your program starts.

So that stuff is to handle different notifications; I notify for each individual task, but combine them in to one notification when gulp or gulp watch is ran. Never had any issues with that in the past, still working well today.
I'm going to go ahead and close this issue, I think the problem was that a promise was returning too early and breaking things.


Add callback to check what part of home screen is occluded

Hi there, I hope this is the right place to ask a compatibility question about home screen apps. I'm one of the developers of the home screen app Niagara Launcher, and lately, we're trying to optimize our UX for foldable devices, including the Surface Duo.
The Problem
I noticed that when the Duo is unfolded, the home screen is displayed on both displays. If you launch an app, it occupies one of the two screens by default. Now half of our home screen is occluded by that app. I tried to find a callback method that notifies us about what part of the app is currently visible/occluded to the user but unfortunately to no avail. Microsoft Launcher, on the other hand, rearranges its layout after launching an app. I wish there was a public method for third-party launchers to check what parts of the home screen are visible to the user and what parts are occluded by other apps.
What callbacks I tried but did not fire on when the home screen is partially occluded:

ScreenInfoListener.onScreenInfoChanged(...) (from com.microsoft.device.dualscreen:screenmanager-windowmanager:1.0.0-beta4)
windowInfoRepository.windowLayoutInfo (from androidx.window:window:1.0.0-beta02)
OnApplyWindowInsetsListener.onApplyWindowInsets(...) (from the Android framework)

Illustrations
Niagara Launcher on both screens

Half of Niagara's home screen is occluded when an app is launched

Microsoft Launcher can readjust its layout (e.g. the dock at the bottom) when an app is launched

Additional details
I tested this on the Surface Duo Emulator (SurfaceDuoEmulator_2021.816.1_windows).

Hello, Peter!
At the moment the SDK does not offer this feature.
But I contacted multiple developers from our team and we are working to find a solution for you.
Thank you for creating the issue and detailed explanations!

Thank you very much, Christian 😀 Feel free to contact me if you and your team need any more details about the issue.

Hello, Peter!
Just wanted to give you a status.
We identified a solution but we are still working on it. It might take a couple of weeks to complete.
Thank you, Cristian!

Hello, Peter!
Just wanted to give you a status. We identified a possible solution but we are still working on it. It might take a couple of weeks to complete.
Thank you! Cristian

Any update from MSFT on this @CristianVerdes?

Hi wylew,
Thank you  for the kind reminder and sorry for the late reply. There are many parts involved and we are still working to get the best solution possible.
Sadly there is no ETA for it but I will update you as soon as we have a more meaningful progress.
Thank you,
Cristian

Any traction on this with MSFT @CristianVerdes ?

Hey wylew,
Thanks for reaching out! Sadly, priorities shifted for us, and we couldn't handle this work. If anything changes, I will let you know in this thread.
Thank you,
Cristian


Focus-within wrong browsers support

Hi data from caniuse are different than mdn:
https://caniuse.com/#search=focus-within
vs
https://developer.mozilla.org/en-US/docs/Web/CSS/:focus-within
I've tested it in Samsung Internet and it's definitely working.

Confirmed! Have updated the data, thanks.


out-of-the-box tensorboard support

Rasa version: Docker image, rasa/rasa:2.2.5-full
Feature request:
Support tensorboard out of the box. Currently running tensorboard fails due to missing packages.
See #5336.
See #7744
docker run --rm -it --entrypoint "tensorboard" rasa/rasa:2.2.5-full
2021-01-17 09:31:52.353959: W tensorflow/stream_executor/platform/default/dso_loader.cc:59] Could not load dynamic library 'libcudart.so.10.1'; dlerror: libcudart.so.10.1: cannot open shared object file: No such file or directory
2021-01-17 09:31:52.354114: I tensorflow/stream_executor/cuda/cudart_stub.cc:29] Ignore above cudart dlerror if you do not have a GPU set up on your machine.
Traceback (most recent call last):
  File "/opt/venv/bin/tensorboard", line 8, in <module>
    sys.exit(run_main())
  File "/opt/venv/lib/python3.7/site-packages/tensorboard/main.py", line 65, in run_main
    default.get_plugins(),
  File "/opt/venv/lib/python3.7/site-packages/tensorboard/default.py", line 108, in get_plugins
    return get_static_plugins() + get_dynamic_plugins()
  File "/opt/venv/lib/python3.7/site-packages/tensorboard/default.py", line 146, in get_dynamic_plugins
    "tensorboard_plugins"
  File "/opt/venv/lib/python3.7/site-packages/tensorboard/default.py", line 145, in <listcomp>
    for entry_point in pkg_resources.iter_entry_points(
  File "/opt/venv/lib/python3.7/site-packages/pkg_resources/__init__.py", line 2461, in load
    self.require(*args, **kwargs)
  File "/opt/venv/lib/python3.7/site-packages/pkg_resources/__init__.py", line 2484, in require
    items = working_set.resolve(reqs, env, installer, extras=self.extras)
  File "/opt/venv/lib/python3.7/site-packages/pkg_resources/__init__.py", line 787, in resolve
    raise DistributionNotFound(req, requirers)
pkg_resources.DistributionNotFound: The 'wheel>=0.26; python_version >= "3"' distribution was not found and is required by the application

Thanks for submitting this feature request 🚀 @rgstephens will get back to you about it soon! ✨

Thanks for submitting this feature request 🚀 @rgstephens will get back to you about it soon! ✨

@Mazyod I've updated your issue description since it is a feature request.
@koaning wdyt of this?

I'd like to double-check with @dakshvar22 but my feelings are somewhat mixed. Tensorboard doesn't occur to me as a feature that many of our users use and Rasa is already heavy with dependencies.

It was just strange to me because it used to work fine when using 1.10.16, and also the tensorboard command is available at:
# which tensorboard
/opt/venv/bin/tensorboard

But I'm all for reducing image sizes, if perhaps this can be documented as a breaking change and the tensorboard bin is removed?


Collision prevention velocity limitations also based on max accel/jerk

Describe problem solved by the proposed pull request
Until now the collision prevention assumed that the vehicle could follow the tuning of the velocity controller. However, it is possible to have a tightly tuned velocity controller, with a very limiting max jerk/accel on top.
Describe your preferred solution
The Collision prevention now takes the more conservative of the tuning parameters (both smooth and non-smooth) when limiting velocity for a given braking distance. It uses the same method as the smooth auto velocity controller to limit speeds given available braking distances.
@baumanta could you please SITL test
@bresch FYI
Todo:

[x] unit tests

SITL tests looked fine. But it seems we have some more general issues with the collision prevention right now. Even master breaks way too late. We should probably investigate what changed first

Issues fixed, they were on the avoidance side. We flew this PR today, it did not significantly improve the behavior but there was also no regression. I would be fine with merging it as it is the theoretically correct thing to do.


[Feature Request] Tests (using Molecule)

For the moment there are NO tests at all, which makes it more difficult to refactor the code, to test changes  and thus to contribute.
My proposal is to change that, using the great Molecule framework.
As a start, I got an initial version done, running in a docker container (with systemd), of course single node first  (multiple nodes are possible too)
You can check at my branch: https://github.com/scigility/cloudera-playbook/tree/molecule_tests
Plz note: This is not ready yet, but I wanted to inform the community about the initial work

@dbeech wdyt about molecule tests?

Hi @lhoss . I haven't had chance to check your molecule_tests branch in detail but I'm definitely in favour of automated testing. Changes are difficult to test today and usually need runs of the entire playbook, which is very time consuming. Anything to improve that would be very welcome.

A 1. iteration: https://github.com/cloudera/cloudera-playbook/pull/59 :)


Remove book from saved books list

User Story
AS AN avid reader
I WANT to search for new books to read,
SO THAT I can keep a list of books to purchase.

Acceptance Criteria
GIVEN a book search engine,
WHEN I click on the Remove button on a book,
THEN that book is deleted from my saved books list.

See PR https://github.com/nestibry/book-search-engine/pull/21


problem with VTK (OCCT 7.4.0 hd267c4f_8)

Issue:
I work on an C++ application using OCCT.
I've updated OCCT to 7.4.0 (hd267c4f_8) with conda and rebuild the project with CMake. It seems there are no errors in the CMakeLists.txt but when I compile I got the error : "LINK : fatal error LNK1181: cannot open input file 'vtkCommonCore-9.0.lib"
It is strange because I don't use vtk...
So I thought it was due to a vtk dependencies and I decided to change my OCCT build version to a version without vtk and finally install  OCCT 7.4.0 h823b557_3 and everything works fine.
For me it seems the vtk dependency is the problem, is there a way to use the last build of OCCT (hd267c4f_8) or to fix the problem with a CMake command ?

 Is it possible for you to share a minimal example?

I guess some of the occt->vtk linking shouldn't be linked public. Is it possible for you to share a minimal example?

Here is my CMakeLists
cmake_minimum_required(VERSION 3.0.0 FATAL_ERROR)

set(CMAKE_CXX_STANDARD 14)

#GTest

find_package(GTest REQUIRED)

message(STATUS " GTests library " ${GTEST_BOTH_LIBRARIES})
message(STATUS " GTests libraries " ${GTEST_LIBRARIES})
message(STATUS " GTests include " ${GTEST_INCLUDE_DIRS})

#nlohmann_json

find_package(nlohmann_json REQUIRED)

#OpenCASCADE

find_package(OpenCASCADE REQUIRED)

message(STATUS " OpenCASCADE library " ${OpenCASCADE_LIBRARIES})
message(STATUS " OpenCASCADE include dir " ${OpenCASCADE_INCLUDE_DIR})
message(STATUS "OpenCascade library dir " ${OpenCASCADE_LIBRARY_DIR})

#gsl

find_package(GSL REQUIRED)
message(STATUS "GSL ROOT DIR: " ${GSL_ROOT_DIR})
message(STATUS "gsl include dirs: " ${GSL_INCLUDE_DIRS})
message(STATUS "gsl lib dirs: " ${GSL_LIBRARIES})

#pybind11

find_package(pybind11 CONFIG REQUIRED)

message(STATUS "pybind11 include dirs: " ${pybind11_INCLUDE_DIRS})
message(STATUS "pybind11 lib dir: " ${pybind11_LIBRARIES})

###################################################
add_subdirectory(src)

I never call VTK because I don't need it.

As a workaround try not linking against TKIVtk. Your cmake file does not tell much due to the add_subdriectory call but I assume that you link against all OpenCASCADE_LIBRARIES.
It seems that the fix-private-linking.patch is not applied properly @looooo .

@adam-urbanczyk  you're right I link directly against OpenCASCADE_LIBRARIES (target_link_libraries(Airfoil PUBLIC Utils ${OpenCASCADE_LIBRARIES})).
I'm quite new to CMake. So how do you do to not link against TKIVtk ?
Could you tell me please the right command ?

To inform anyone who could be interested, I have found a solution to my problem by adding the following commands to the CMakeLists.txt :
find_package(VTK COMPONENTS CommonCore REQUIRED)

#Freetype
find_package(Freetype REQUIRED)

#GLEW
find_package(GLEW REQUIRED)


Payment did not went trough but funds deduced from my wallet

Just playing with lnbits on legend.lnbits.com with small amount of sats - but... see the screenshot

the first trascastion with memo 123 for 5000sats is invoice I generated but never paid, so I dont count this
then I funded the wallet with 500sats - test from wallet of satoshi
then I tried to pay lightning maneki neko (srv.dread.cz) and played with lnurl-w
then I funded another 1000sats, everything ok to this point
then I tried maneki again, but the 1st try did not came trough (that with fee 2500msats) - I got some error in lnbits but I did not catched it - it disappeared too fast... payment to maneki was not done
then I send it again immediatelly and it worked ok  (I own the maneki server so I can see logs and there is only one transaction visible)

But both payments were deduced from my LNbits account - so now Im 250sats short. Interesting is that both maneki payments at the top of the transaction list have the same payment hash...

This is what I get if I click the top maneki payment with fee 1016msats

... and I get exactly the same if I click that pending one with fee 2500msats

I have had quite a few of these <•••> Outgoing payment pending within the last month on a wallet generated on https://legend.lnbits.com/.
Sometimes the pending payment seems to be canceled out shortly after a failed forwarding and returned to the wallet, but a lot so far didn't return the failed forwarding amount. Oldest Outgoing payment pending is more than a month old.
Should the pending pending payments be considered to be lost or are they supposed to be returned at some point in time?
It's my impression that failing payments are caused by either bad internet connectivity or amount being to big to be routed.

Issue https://github.com/lnbits/lnbits/issues/118 got fixed, can you verify if updating solves it for you?

Closing for inactivity


[WFCORE-754] : WildFly fails to start with -bmanagement= option and with specific IP settings

Wrong usage of address.getHostName() before formatting address to IPv6.
Jira: https://issues.jboss.org/browse/WFCORE-754
Upstream: https://github.com/wildfly/wildfly-core/pull/820

Core - Full Integration Build 73 is now running using a merge of ac62c31794f619d4e351a2a3675364335f90f05b

Linux Build 86 is now running using a merge of ac62c31794f619d4e351a2a3675364335f90f05b

Windows Build 73 is now running using a merge of ac62c31794f619d4e351a2a3675364335f90f05b

Windows Build 73 outcome was SUCCESS using a merge of ac62c31794f619d4e351a2a3675364335f90f05b
Summary: Tests passed: 3102, ignored: 68 Build time: 0:20:17

Linux Build 86 outcome was SUCCESS using a merge of ac62c31794f619d4e351a2a3675364335f90f05b
Summary: Tests passed: 3102, ignored: 68 Build time: 0:22:30

Core - Full Integration Build 73 outcome was SUCCESS using a merge of ac62c31794f619d4e351a2a3675364335f90f05b
Summary: Tests passed: 2871, ignored: 375 Build time: 0:49:24


Directly assign any user (Preview)

"To do this, the access package must have a policy that allows users not yet in your directory to request access"
Is this a hard requirement, or can I also use "None (administrator direct assignments only)" in the policy?

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

ID: 270ab048-109b-7c3f-95a3-1d683d9433b6
Version Independent ID: 1ff429c1-6d62-f4e9-9244-074d0573abdc
Content: View, add, and remove assignments for an access package in entitlement management - Microsoft Entra
Content Source: articles/active-directory/governance/entitlement-management-access-package-assignments.md
Service: active-directory
Sub-service: compliance
GitHub Login: @OWinfreyATL
Microsoft Alias: owinfrey

Hey @BakkerJan
Thanks for your question. From the standing of this preview a policy where the guest user must request access is a requirement to use this feature.
Thanks for your question!

#please-close


Config file path should be type string

See bmi.initialize() definition,
def initialize( self, bmi_cfg_file=None ):
    #NJF ensure this is a Path type so the follow open works as expected
    #When used with NGen, the bmi_cfg_file is just a string...
    bmi_cfg_file = Path(bmi_cfg_file)

Per #NJF comment, we should maintain bmi_config_file as a string for ngen compliance.
Current behavior
bmi_config_file is of type path
Expected behavior
bmi_config_file is of type string

adjusted in PR #34 . closing


Wrong genre_ids filter

case 'genre': {
          const ids = queryObject[key] ? queryObject[key].split(',') : [];
          result.genre_ids = { $all: ids };
          break;
        }

Closed in https://github.com/IlonaMenkui/super-enigma-service/pull/22


41 decisiontree gpu

This version supports GPU benchmarking of
HummingbirdPytorchGPU
HummingbirdTorchScriptGPU
ONNXGPU
for:
xgboost
randomforests
TO DO

TVMGPU must be built from source
some rapids ai, CuML requirements are conflicting with the existing requirements

Great, Mahidhar!
Can you create an image of your AWS instance, which we can use to help test and review your work?

@mahidhar96
FYI... I've created an AMI image from your instance named decision-forest-bench-GPU.
If possible, let's discuss this later this afternoon.

@mahidhar96
I've met some errors at the end of the execution, where are these errors from?
(rapids-22.08) ubuntu@ip-172-31-24-156:~/netsdb/model-inference/decisionTree/experiments$ python test_model.py -d higgs -m xgboost -f NvidiaFILGPU
==============EXPERIMENT STARTING=========================
DATASET: higgs
MODEL: xgboost
FRAMEWORK: NvidiaFILGPU
Query Size: 100000
Batch Size: 100000
Trees 1600
Depth 8
Time Taken to load higgs as a dataframe is: 5554.696083068848
[08:14:09] WARNING: /home/conda/feedstock_root/build_artifacts/xgboost-split_1660208897656/work/src/learner.cc:627:
Parameters: { "verbose" } might not be used.
This could be a false alarm, with some parameters getting used by language bindings but
then being mistakenly passed down to XGBoost core, or some parameter actually being used
but getting flagged wrongly here. Please open an issue if you find any such cases.
Time Taken to load sklearn model:<PHONE_NUMBER>8098145
Time Taken to predict on NvidiaFILGPU is 1438.4582042694092
Time Taken to write results to a text file for NvidiaFILGPU is<PHONE_NUMBER>01831
Classification Report NvidiaFILGPU
precision    recall  f1-score   support
       0       0.85      0.59      0.70   1034836
       1       0.71      0.91      0.80   1165164

accuracy                           0.76   2200000

macro avg       0.78      0.75      0.75   2200000
weighted avg       0.78      0.76      0.75   2200000
################
Time Taken to convert NvidiaFILGPU model: 5861.56439781189
TOTAL Time Taken for NvidiaFILGPU: 7873.032569885254
==============EXPERIMENT ENDING=========================
Traceback (most recent call last):
File "fil.pyx", line 273, in cuml.fil.fil.ForestInference_impl.get_dtype
AttributeError: 'NoneType' object has no attribute 'float32'
Exception ignored in: 'cuml.fil.fil.ForestInference_impl.dealloc'
Traceback (most recent call last):
File "fil.pyx", line 273, in cuml.fil.fil.ForestInference_impl.get_dtype
AttributeError: 'NoneType' object has no attribute 'float32'

I'm not sure where these are coming from. I'm still investigating these errors. I thought it might be at the end of loading the model, where there is no data, but FIL is trying to stream it and convert it to float32 (because it doesn't support float64). I'm going through their code to investigate this.
https://github.com/rapidsai/cuml/blob/branch-22.10/python/cuml/fil/fil.pyx This is the file that error is referring to, but I didn't find anything it is not called float32 on that particular line. get_dtype function is on line 277, and it is accessing float32 from numpy, but it should do this every time it imports numpy, but it is only at that line. I'm still investigating this.

I got this message from @jiazou-bigdata
I just tried rapids-22.10 using the below installation.
conda create -n rapids-22.10 -c rapidsai-nightly -c nvidia -c conda-forge \
rapids=22.10 python=3.9 cudatoolkit=11.5

However, the error showed up again at the very end, probably when the model object gets freed.
Traceback (most recent call last):
File "fil.pyx", line 282, in cuml.fil.fil.ForestInference_impl.get_dtype
AttributeError: 'NoneType' object has no attribute 'float32'
Exception ignored in: 'cuml.fil.fil.ForestInference_impl.__dealloc__'
Traceback (most recent call last):
File "fil.pyx", line 282, in cuml.fil.fil.ForestInference_impl.get_dtype
AttributeError: 'NoneType' object has no attribute 'float32'

I got this message from @jiazou-bigdata
I just tried rapids-22.10 using the below installation.
conda create -n rapids-22.10 -c rapidsai-nightly -c nvidia -c conda-forge \
rapids=22.10 python=3.9 cudatoolkit=11.5

However, the error showed up again at the very end, probably when the model object gets freed.
Traceback (most recent call last):
File "fil.pyx", line 282, in cuml.fil.fil.ForestInference_impl.get_dtype
AttributeError: 'NoneType' object has no attribute 'float32'
Exception ignored in: 'cuml.fil.fil.ForestInference_impl.__dealloc__'
Traceback (most recent call last):
File "fil.pyx", line 282, in cuml.fil.fil.ForestInference_impl.get_dtype
AttributeError: 'NoneType' object has no attribute 'float32'

@nwstephens Thanks Nathan for getting back to us. Any suggestions?


Possibility to use selenide-appium along with selenide in 1 repo

Currently have a repo which uses selenide for both web and mobile automation for various project inside 1 repo.
Due to https://github.com/selenide/selenide/issues/1751 issue wanted to know if it's possible to use both dependencies in order to use fixed version for actions for mobile platform.
Is it somehow possible or it's better to setup separate project?

@adicusar-nm Sorry, I didn't understand how it's related to https://github.com/selenide/selenide/issues/1751
You can have a single project with dependency selenide-appium.
Plugin selenide-appium is smart enough: it understands if current webdriver is mobile or web, and behaves accordingly. At least I hope so. If some function doesn't check the "web/mobile" context - please report, we will fix it asap.

@asolntsev I mean that selenide doesn't have that fix (as I see from changelog) and wanted to know if I need to use both selenide and selenide-appium
But you've answered my question, so if you have mobile/web project you can just use selenide-appium


Add mruby-cli v0.0.4

If there’s a checkbox you can’t complete for any reason, that's okay, just explain in detail why you weren’t able to do so.
After making all changes to the cask:

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

Additionally, if updating a cask:

[ ] If the sha256 changed but the version didn’t,
provide public confirmation by the developer: {{link}}

Additionally, if adding a new cask:

[X] Named the cask according to the token reference.
[X] brew cask install {{cask_file}} worked successfully.
[X] brew cask uninstall {{cask_file}} worked successfully.
[X] Checked there are no open pull requests for the same cask.
[X] Checked the cask was not already refused in closed issues.
[X] Checked the cask is submitted to the correct repo.

Was previously rejected by homebrew because the formula submitted used a binary:
https://github.com/Homebrew/legacy-homebrew/pull/46085
The official instructions for building the CLI binary involve using a docker image which installs a cross compilation toolchain.  It will be easier to get this tool on user's machines by deploying a prebuilt MacOS binary.

This seems to be a CLI-only open-source tool. Please submit it first to Homebrew/core and only if they don’t accept it, submit here (linking back to the pull request where it was refused).
This policy is documented in faq/rejected_casks:

The app is both open-source and CLI-only (i.e. it only uses the binary artifact). In that case, and in the spirit of deduplication, submit it first to Homebrew/core. If it is rejected there, you may then try again in Homebrew-Cask (link us to the issue on Homebrew so we can see their reasoning for rejection).


load-extension=/path/to/extension with chromium-args Not Work Ruffle on Browservice

i use --chromium-args=load-extension=/path/to/extension Not Work

Trying Ruffle Extension on browservice is Not Work

This Ruffle Extension Chrome

on Browservice --chromium-args=load-extension=/path/to/extension is Not work

Extensions were previously discussed in https://github.com/ttalvitie/browservice/issues/18 and https://github.com/ttalvitie/browservice/issues/45, the summary is: For me extension support is not a priority and I haven't really investigated what is required for it. Hopefully someone else can produce a pull request or proof-of-concept on it.


Add a method to get Vec<Entity> from Entities

What problem does this solve or what need does it fill?
I have access to the world or an Entities system parameter, and want a list of Entity in a useful format.
This is a natural method, and would be useful for writing tests or editor-like functionality.
What solution would you like?
Add a method to get Vec<Entity> from Entities.
What alternative(s) have you considered?
Current workaround: use a query for Query<Entity>.

Entities doesnt actually store a Vec<Entity> so you'd probably want an iterator to avoid allocating the vec at which point using a Query<Entity> seems reasonable :sweat_smile:

Entities doesnt actually store a Vec so you'd probably want an iterator to avoid allocating the vec at which point using a Query seems reasonable 😅

Yeah, I'm fine if it returns a impl Iterator<Item = Entity>. Fundamentally, this is about aligning with user expectations: it's super weird that you can't get a list of the entities from the system parameter Entities.

I think in some cases a Query<Entity> could probably be more efficient than something from Entities since Entities hasnt got a fast way to only iterate "alive" entities. I think it'd have to iterate over Vec<EntityMeta> checking if each element is alive and then constructing an entity if it is. Query<Entity> can just iterate all tables/archetypes automatically skipping dead entities since they dont exist there :thinking:

Agreed there, but this isn't a perf-sensitive path. This would be almost entirely for debugging and tests.

What is actually the purpose of Entities (it's missing docs)? Isn't even more confusing to be able to query entities via a resource?
I always had the impression that ECS is designed to Query entities that represent the objects living in the world. Resources are for global state that are not related to an entity. Is this impression wrong? How does this relate to this issue? Isn't Entities kinda violating this separation?

What is actually the purpose of Entities (it's missing docs)?

This is an internal collection of "which entities are valid", that is exposed as a system parameter. You can access metadata about entities (such as whether or not they exist) and reserve specific entities for later use using it.

Isn't even more confusing to be able to query entities via a resource?

It's not a resource, it's part of the World's assorted internal-ish state. This wouldn't permit you to query for any of their data, merely get a list.
Overall, I think this is a questionable idea: I'm going to close this out and we can revisit it later once we have a better idea of what Entities is used for in real code.


Add README

Made changes to the REDAME. Please let me know what else is there that you would like to have in there.

@mridulg thanks for the formatting. It was definitely needed.

@varunj not an issue.


Cherry pick Fix 876, allow null array to be cast from and to other types to active_release

Automatic cherry-pick of 81ffa24

Originally appeared in https://github.com/apache/arrow-rs/pull/884: Fix 876, allow null array to be cast from and to other types

Codecov Report

Merging #917 (0337d48) into active_release (03d95e6) will decrease coverage by 0.12%.
The diff coverage is 100.00%.

:exclamation: Current head 0337d48 differs from pull request most recent head 863cc98. Consider uploading reports for the commit 863cc98 to get more accurate results

@@                Coverage Diff                 @@
##           active_release     #917      +/-   ##
==================================================
- Coverage           82.45%   82.33%   -0.13%
==================================================
  Files                 168      168
  Lines               48175    48092      -83
==================================================
- Hits                39725    39596     -129
- Misses               8450     8496      +46

Impacted Files
Coverage Δ

arrow/src/compute/kernels/cast.rs
94.48% <100.00%> (-0.15%)
:arrow_down:

arrow/src/array/transform/fixed_binary.rs
75.00% <0.00%> (-3.95%)
:arrow_down:

parquet/src/schema/visitor.rs
67.10% <0.00%> (-2.64%)
:arrow_down:

arrow/src/compute/kernels/cast_utils.rs
87.50% <0.00%> (-2.50%)
:arrow_down:

parquet/src/encodings/decoding.rs
90.45% <0.00%> (-2.02%)
:arrow_down:

arrow/src/datatypes/ffi.rs
70.85% <0.00%> (-2.02%)
:arrow_down:

parquet/src/arrow/arrow_reader.rs
89.32% <0.00%> (-1.95%)
:arrow_down:

arrow/src/ffi.rs
84.69% <0.00%> (-1.89%)
:arrow_down:

arrow/src/csv/reader.rs
90.00% <0.00%> (-1.78%)
:arrow_down:

arrow/src/util/data_gen.rs
75.44% <0.00%> (-1.66%)
:arrow_down:

... and 70 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 03d95e6...863cc98. Read the comment docs.


Create sql-server-compliance.md

Document the compliance of SQL Server to the SQL standards.
Fixes #4616

@vanillajonathan Thanks for your pull request. It will be assigned for review by the content team shortly.

@vanillajonathan Hello Jonathan, Thank you for creating this PR #4665 , as this might make for an excellent new documentation article.
But first, can you please provide some context for this?

From where did you obtain this info?

All articles need a paragraph of explanatory text at the top of the introduction. For instance, these table rows involve "compliance" with what?  (Maybe with ANSI-? standards for the SQL language, or exact what?)?

Can you please enter such a paragraph into either (a) An update of your draft article, or (b) Into another Comment box (one like this comment box, here in this PR)?

What does 'Unknown' mean in the third column with the label 'Microsoft SQL'?

Does 'Unknown' mean you do not know, or nobody knows, or it would be impossible to determine, or what does 'Unknown' mean?

How can our SQL Server documentation writing staff accurately reassess possible updates to this article when SQL Server vNext is eventually released?  What master source of accuracy did you reference when drafting this SQL 2019 article?

I suspect that the draft article should have a clickable link to the master source of accuracy for general Structure Query Language compliance requirements.

A minor point: for our markdown tables, we always include both an opening '|' and a closing '|' character on each row.
As a practical matter for easier scan-ability for our customers, we will likely modify your draft to divide the one enormous markdown table into several smaller tables. We will also insert several ## H2 title headers.
It will be very helpful if you can provide the above requested info.
Thanks a lot Jonathan.
cc: @rothja , @MikeRayMSFT , @ktoliver
. .

@vanillajonathan I see now that the earlier GitHub Issue #4616 gives some info about webpages hosted by other companies, that have some kind of SQL compliance requirements. But I do not see a link to the master source of truth about SQL standards.
Microsoft's "Docs" website has some articles that seem to be related to this draft:

https://docs.microsoft.com/sql/database-engine/configure-windows/common-criteria-compliance-enabled-server-configuration-option?view=sql-server-ver15

https://docs.microsoft.com/openspecs/sql_standards/ms-sqlstandlp/89fb00b1-4b9e-4296-92ce-a2b3f7ca01d2

@ktoliver Thanks, the article is more of a rough draft right, hence most features are specified as "Unknown", and even for the features that are specified as "Yes", I don't really don't know if they adhere 100% to the standard, nor which edition of the SQL standards. It needs lots of work to mark up which features are supported and to what extend. By doing the outline, I would hope that someone who knows their stuff (I surely don't) can fill in the information.
@MightyPen

From where did you obtain this info?

I gleaned at the links mentioned in the issue and wanted something similar for SQL Server so threw together a Markdown table. The features are documented in the SQL standard documents but I do not have access them, because they cost money.

All articles need a paragraph of explanatory text at the top of the introduction. For instance, these table rows involve "compliance" with what? (Maybe with ANSI-? standards for the SQL language, or exact what?)?

The compliance would be to the features defined in the SQL standards, such as SQL:2016, but some features in SQL Server might not have full support for the features as defined in SQL:2016 an might only be compliant to SQL:2011 or SQL:2008, then this could be mentioned.

Can you please enter such a paragraph into either (a) An update of your draft article, or (b) Into another Comment box (one like this comment box, here in this PR)?

The article have "Allow edits from maintainers" enabled, so feel free to improve the article.

What does 'Unknown' mean in the third column with the label 'Microsoft SQL'?

"Unknown" in the column means that I do not know, I am sure someone else knows. So someone with expertise in SQL Server needs to change this to "Yes", "No", "Partially" or something like "Yes, SQL:2008". Please fill in if you feel you know and can help.

How can our SQL Server documentation writing staff accurately reassess possible updates to this article when SQL Server vNext is eventually released? What master source of accuracy did you reference when drafting this SQL 2019 article?

They can add an additional column to the table, to have one column represent SQL Server 2019 and a column next to it representing SQL Server vNext.
The article was inspired by the Supported Features article in the PostgreSQL documentation as well as the SQL conformance article in the Firebird SQL documentation. So I created a similar table in Markdown syntax and did a very poor job at filling it in.

I suspect that the draft article should have a clickable link to the master source of accuracy for general Structure Query Language compliance requirements.

My understanding is that all the features are documented in the SQL:2016 or ISO/IEC 9075:2016 and related documents from previous editions and these are not public available on the web and have to be purchased.

A minor point: for our markdown tables, we always include both an opening '|' and a closing '|' character on each row.

Okay, I can do that adjustment. The article is open for edits, so anyone can chip in.

As a practical matter for easier scan-ability for our customers, we will likely modify your draft to divide the one enormous markdown table into several smaller tables. We will also insert several ## H2 title headers.

Yeah, that sounds good.
Thanks!

@vanillajonathan - thank you for this submission and for answering @MightyPen 's questions.
I'm going to engage with some members of the product group to evaluate your submission.
Thanks again.

@vanillajonathan I consulted with @pmasl who noted:

Standards support docs already exists in https://docs.microsoft.com/en-us/openspecs/sql_standards/ms-sqlstandlp/89fb00b1-4b9e-4296-92ce-a2b3f7ca01d2. Specifically, the PR refers to the current SQL standard (ISO/IEC 9075:2008), which is already documented here: https://docs.microsoft.com/en-us/openspecs/sql_standards/ms-tsqliso11/60cb661b-8fad-4d05-831f-1d5bc60bc1a2
A short URL is https://aka.ms/sqlstandards.

With this in mind, it seems that the best answer is to make sure appropriate links are placed in the documentation.
Is there a place you might recommend placing these links?
Thanks,
Mike

@MikeRayMSFT That documentation is rather difficult to comprehend, there are lots of links to PDFs and lots of sub-chapters. That documentation might be good when you want to dig deep into a particular function, but is less useful to get overview/glance of the implemented functionality in regards to the SQL standards. Looking at the PostgreSQL and the Firebird documentation is much easier. I still think we should have a presentation in a table format because it is just so much easier to comprehend and make it so much more user friendly.

@vanillajonathan - thank you for the follow up and I understand your point about presentation of the content. I appreciate the example table you propose. We're looking at options that meet our customer's requirements.

@vanillajonathan
Thank you very much for submitting this article. We will not be able to include it in the docs.
After reviewing your proposal and our business requirements, we have decided that the standards support documentation will remain as published at SQL Server Standards Support. This presentation meets compliance requirements, obligations, and is aligned with the standards documentation provided by other commercial database vendors.
SQL Server technical documentation is published in an open publishing system. This system supports broad community contribution to our documentation set. The documentation is subject to frequent changes as the product is updated, technology improves, and use cases evolve. However, the standards are relatively static, and our implementation of the standards in the design of the SQL Server relational database management system is also stable. To ensure that the way standards are implemented is documented, we provide the SQL Server Standards Support documentation in a system that is carefully vetted and updated only as required.
The request you submitted highlights a need to link the content in SQL Server technical documentation to the applicable standards and we plan to add additional links as appropriate. For the short term, we have already made that link available from the landing page for Transact-SQL Reference (Database Engine). We are also evaluating publishing the standards content in way that is more searchable.
Thank you, again for your proposal.

@MikeRayMSFT I am sure your extensive standard documents ticks every checkbox for compliance requirements and obligations, but it is not very useful. Other standard documents provided by other database vendors offers presentation formats that are much easier to to get an overview of which features are compliant to the standards, features deviate from the standard and which standard features are not implemented.
In a comment above in this thread are links to the documentation of PostgreSQL and Firebird which shows how other vendors make their information available in a much more user friendly way.
The way the documentation is presented by Microsoft is only good for fulfilling compliance requirements and obligations.
I would like to request that the team responsible for the SQL compliance documentation should append a feature table in addition to the existing documentation.

@vanillajonathan as mentioned, we are also evaluating publishing the standards content in way that is more searchable than downloading a file and using Word/PDF search functionality. To establish expectation, that is not a process with a short turnaround. Thank you for the feedback.

Yeah it would be good if it was more searchable, but it would also be good nice if it was presented in such a way that you quickly get an overview of compliance against all the features in the SQL standards, such overview could be presented in a table, if I may suggest.


Use pgrep instead of pidof in E2E test

pgrep is provided in Ubuntu and Debian by default. pidof is provided by
the sysvinit-utils package, which is not available in Debian. Use of
pgrep will allow downstream to repackage Antrea agent into various
distributions and still re-using the E2E test.

/test-all

/jenkins-windows-networkpolicy

/test-windows-networkpolicy


KeyError: "The name 'input_image:0' refers to a Tensor which does not exist. The operation, 'input_image', does not exist in the graph."?

However I run your code in CPU mode. but it gives the following error:

KeyError: "The name 'input_image:0' refers to a Tensor which does not exist. The operation, 'input_image', does not exist in the graph."

Would you help me please?
Thanks

The tensor input_image should be in the Graph, are you loading the right model? and have you tried loading the same config class which you used for training?

Hi, thanks for sharing this serving_ready
I can create and load the frozen graph now.
After I loaded the graph, I can get_tensor_by_name('input_image:0'), but what about the outputs?
I tried the following, and none of is worked.
tf_rois = tf_sess.graph.get_tensor_by_name('rois:0')
tf_masks = tf_sess.graph.get_tensor_by_name('mask:0')
tf_class_ids = tf_sess.graph.get_tensor_by_name('class_ids:0')
tf_scores = tf_sess.graph.get_tensor_by_name('scores:0')

@bendangnuksung @i-chaochen
problem is solved. The tensor name was not same as .pb file. thanks

problem is solved. The tensor name was not same as .pb file. thanks

so where do you know the tensor name from the frozen graph?

During creating the freeze_graph, just print model.inputs and model.outputs. In my case, the name was
input_image_7.  Thanks

How did you get the output for the model.inputs? I tried printing them but nothing appeared.


changed to API 16, allow optional module tags in jni

I have recently upgraded the minsdk to 9 and hence removed the native jni code


Added the function isIdempotent()

Pull Request Template
Description
This function checks if the given matrix is idempotent or not. It returns 1 in case the matrix is idempotent and 0, otherwise.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[ ] New feature (non-breaking change which adds functionality)

How Has This Been Tested?
I have run the code on compiler.

[ ] Test A
[ ] Test B

Test Configuration:

Firmware version:
Hardware:
Toolchain:
SDK:

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules
[ ] I have checked my code and corrected any misspellings

!bounty 30


Fix aggregation $group compat test with consistent sort order

Description
Closes #2616.

Readiness checklist

[ ] I added/updated unit tests.
[x] I added/updated integration/compatibility tests.
[ ] I added/updated comments and checked rendering.
[ ] I made spot refactorings.
[ ] I updated user documentation.
[ ] I ran task all, and it passed.
[x] I ensured that PR title is good enough for the changelog.
[x] (for maintainers only) I set Reviewers (@FerretDB/core), Labels, Project and project's Sprint fields.
[x] I marked all done items in this checklist.

Hmmm... I understand the idea but it looks to me like we try to hide a problem with $group and sorting by making our data simple enough to make the test work. So I'm not sure what is needed in the original issue, let's discuss.

@rumyantseva could you suggest if you have a thought as well please?

Closing because this solution is not optimal. We should use unique field for sorting instead