IMAGING-266 read numeric data from GeoTIFFs

Updates to the TIFF API to support access to numeric data from TIFF files.  This change addresses JIRA item IMAGING-266.

Sorry, I forgot about the Spotbugs test (did checkstyle, pmd, and findbugs, but not Spotbugs).  So I just submitted changes for Spotbugs.
For reasons of efficiency, the TiffRasterData classes expose arrays directly through the API. This approach is reasonable because the raster data classes are really just simple containers for transferring image read results to the calling application.  Also, since many TIFF rasters consist of literally 100s of millions of raster cells, we wish to avoid making redundant copies of data in memory.  This rationale is described in the Javadoc for the relevant methods with appropriate caveats for developers.

Coverage decreased (-0.04%) to 76.689% when pulling f122b890fb36f55d12c19238e8cac293346dd21d on gwlucastrig:IMAGING-266a into 32e1a3b839eb75b38c47531cb2dd6e0012bb3bf5 on apache:master.

Found a bug in TiffRasterInt, getDataType returns FLOAT instead of INTEGER.  I will fix it when I address the other review comments.

Made changes for your review comments and added a few more JUnit tests to address the Coverage issues.
Unfortunately, I accidentally picked up two files that had tabs in them when I replaced tabs with spaces.  Other than replacing tabs with spaces, these files did not change.
\test\java\org\apache\commons\imaging\TestImageReadException.java
\test\java\org\apache\commons\imaging\TestImageWriteException.java

I think that's everything...   Thanks for your excellent review.
Github is reporting that more changes are requested, but searching through this discussion, I don't see anything that's unresolved.
Please let me know if you find additional items that need attention.

Thanks @gwlucastrig !

Github is reporting that more changes are requested, but searching through this discussion, I don't see anything that's unresolved.

It's because when I submitted my review, I had marked it as request changes. It will remain with that status until I review it again. We are in lockdown in Auckland due to the delta variant, so quite sure I'll be able to finish reviewing it over this weekend.

On Fri, Sep 10, 2021, 5:38 PM Bruno P. Kinoshita @.***>
wrote:

Thanks @gwlucastrig https://github.com/gwlucastrig !
Github is reporting that more changes are requested, but searching through
this discussion, I don't see anything that's unresolved.
It's because when I submitted my review, I had marked it as request
changes. It will remain with that status until I review it again. We are in
lockdown in Auckland due to the delta variant, so quite sure I'll be able
to finish reviewing it over this weekend.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/commons-imaging/pull/165#issuecomment-917227883,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEWJDYKARSB6OHMRODCUNKTUBJ3FZANCNFSM5DWCCJSA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I looked up rebasing and squashing commits and I wanted to check with you before proceeding.
I have 6 commits that I just pushed as part of this PR.  So would my command be the following?

git rebase -i HEAD~6

But as I was researching this, I found a comment:

If you have previously pushed your code to a remote branch, you will need to force push.
git push origin branchName --force

Do I need to do the force when I push?   And, if so, would my command be the following?

git push origin IMAGING-266a --force

where IMAGING-266a is the branch name for these changes?

Hi @gwlucastrig !
Great job fixing the code comments and trying to rebase.
You have done everything correctly. And you will have indeed to push force.
I always take a look at the remote branch in GitHub, and compare with my local branch (using gitk).
If I'm happy that my local branch is the way I want my remote to look, then I push force.
Note that you should avoid push-force to certain repos in case of existijg etiquette. E.g. linux project forbids push force to their main branches. Apache projects normally discourage too.
But you are pushing force to your branch, to your fork. So it doesn't matter in this case. It's only to make merging the code easier :)
I will take a look at the code latert oday gain, if nothing wrong will include one commit later with changes.xml and merge it! 👏
p.s.: a git rebase squash is an advanced git topic, great job! It is also part of the process for fixing conflicts, so you got most of what's needed to fix conflicts in the future
p.p.s: I forgot to add a comment about scansize. Is it OK if I edit your commit when merging, and rename it to scanSize?

I just pushed the squashed version of the code.
Please go ahead and rename scansize to scanSize.  I definitely like that name better.

@gwlucastrig great job! GitHUb UI is already showing 1 commit, so you have successfully rebased and squashed your commits :-)
I had marked during the previous review all files reviewed as "Viewed". Now, when I go to Files Changed, @gwlucastrig , GitHub keeps the files that did not change closed. And it expands the ones that suffered modifications since I left reviewed it.

This saves a lot of time since I don't need to re-read the code to compare with the previous version.

I am really looking forward to going onto the JIRA page and posting a notice about our progress on ISSUE 266 (including the picture of Auckland). Commons Imaging is shaping up to be one of the most useful TIFF solutions out there... Certainly, it's my favorite Java solution.

And props go to you for improving the API and code so much. Really appreciate all your work in reviewing issues, sending new code (with the extra work of chasing test images), and in educating other devs like myself :)

@gwlucastrig here's what I've done in the last commit.

 This file is used to generate the changelog and in the site and in the RELEASE-NOTES.txt file (I think, although can't recall if we use JIRA export tool in release process?). I added an entry with myself as dev, and you as due-to. The dev is always an ASF committer, and due-to the person responsible for work (it's not always present). I didn't put your user as dev as you are still not a committer (only pending you accept & submit the form :)
Pushed force a commit with the squashed commit and the changes.xml (I can push to your branch as you allowed it when creating the PR)
Waited for CI to confirm I didn't mess it up, then merged the PR
Updated the JIRA (fix version, status, etc).

This is the whole process for a ticket in Imaging, but applies to pretty much every Apache Commons projects too.
Cheers
Bruno


SAK-32534 - SAK-30153 NPE Fix

This needs a proper JIRA filled out and used in the commit please DON'T USE NOJIRA.

JIRA made, https://jira.sakaiproject.org/browse/SAK-32534


tasty-head with template-url doesn't work correctly

When I use template-url inside tasty-head as specified in the examples, the resulting table does not include <thead> tag.
I think this is because of this piece of code in table.js:
if (scope.templateUrl) {
$http.get(scope.templateUrl, { cache: $templateCache })
.success(function(templateContent) {
element.replaceWith($compile(templateContent)(scope));
});
}
as you see the element is replaced with the template content and therefore the <thead> element is completely replaced with template which starts with <tr> tag

Good to know, I will improve the docs http://zizzamia.com/ng-tasty/directive/table/custom-sorting by adding the <thead> tag inside the text/ng-template.


Add support and Runtime API implementation for eth_getLogs

Add support and Runtime API implementation for eth_getLogs
Rel #83
This PR adds support for etg_getLogs.

Iterative process in pallet-ethereum to filter stored receipt logs.
Runtime API implementation.
Removed limit field in Filter struct, as not specified in https://eth.wiki/json-rpc/API#eth_getlogs.

A couple details:

I included the removed field in the Log output hardcoded as false. Is this expected to be like this or we want to change it?
Still, this PR just adds the necessary to do the Runtime API implementation. The Receipt stored in PendingTransactionsAndReceipts on pallet-ethereum::execute() appears to be TODO.


[google_maps_flutter][android] suppress unchecked cast warning

Suppress a unchecked cast warning which shows up during builds of the google_maps_flutter plugin. Since generic types cannot be checked with instanceof and a manual check would introduce too much overhead, I think these warnings can be suppressed.
List which issues are fixed by this PR. You must list at least one issue.
flutter/flutter#90969
Pre-launch Checklist

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[x] I read and followed the [relevant style guides] and ran [the auto-formatter]. (Note that unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
[x] I signed the [CLA].
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated pubspec.yaml with an appropriate new version according to the [pub versioning philosophy].
[x] I updated CHANGELOG.md to add a description of the change.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test exempt.
[x] All existing and new tests are passing.

@bparrishMines Looks like this is ready for another review.

how to update on Maps SDK for Android. An upgraded map renderer is available as of version 18.0.0 of the Maps SDK for Android.

@blasten @bparrishMines this is ready for a final review from one of you.


Increase homepage content width on mobile to match docs content width

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/2302
Originally created by @DouweM on 2020-08-21 19:38:23

View 1 previous comment from the original issue on GitLab


Fix cabal new-build and stack build

Now it can compiles with cabal new-build.

bors r+

bors r+


Bug: Cannot add an insight page name in create contributor insight

Describe the bug
The input for adding an insight name inorder to create the insight page does not work.
This blocks the creation of the insight page.
This issue is similar to #2960 but not the same.
I'd like to fix this once triaged :)
Steps to reproduce

Open the workspace sidebar from the workspaces page
Click on create a contributor insight
Try naming your contributor insight
Notice that it does not accept any input

Do you have any images or screen captures?
https://github.com/open-sauced/app/assets/67555014/357cf6d4-af36-4d77-a407-e9f0840b3cd1
Browsers
Chrome
Additional context (Is this in dev or production?)
localhost
Code of Conduct

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

Contributing Docs

[X] I agree to follow this project's Contribution Docs

I can reproduce this on prod:

Worse yet, it seems that just the contib insight name doesn't work at all blocking creating contirb insights completely. In my gif ^ you can see me mashing the keyboard to try to enter anything.

Can I do the fix, i had the PR ready

Fix is ready for review https://github.com/open-sauced/app/pull/3044

Also noticed this for more inputs like this one.

Also noticed this for more inputs like this one.

The inputs might need a closer look 🤔


Emit an event when we move to FLUSHCOMPLETE

Emit an event when we move to FLUSHCOMPLETE here and here.

@chatton hey can i work on this ?

of course @sontrinh16

Closed by #5245


Edit form

fix sorting issues of the fields when delete / styling

@raphaeldieu please check

http://media.giphy.com/media/Jr7YlhsCbwZDG/giphy-facebook_s.jpg

on the merge conflicts now :)

Done ;)

@ggharry @f3r  Is it ok ?

I replied https://github.com/WDI-HK-8/Formidable-web/pull/24#discussion_r39335986


Ignore other node children when fetching IO containers. Fixes #2000

Other resources can be mistaken for the input/output containers resulting in the node snapshot being created incorrectly.

Failing test is due to flaky sciserver integration test


bugfix: after executing statement table lock was not cleared

When we execute a statement, we step once and return. If the query returned more than one row, we would not have consumed everything so the underlying table would still be locked even when the execution had finished.
This change resets the statement after execution so that the locks can be cleared.
Apart from the included test, the following go program serves to reproduce the issue:
package main

import (
	"context"
	"database/sql"

	_ "github.com/mattn/go-sqlite3"
)

func main() {
	ctx := context.Background()
	sqldb, err := sql.Open("sqlite3", "file:test.db?cache=shared&mode=memory")

	stmt, err := sqldb.PrepareContext(ctx, "CREATE TABLE person (id number, name text);")
	stmt.Exec()
	stmt, err = sqldb.PrepareContext(ctx, "INSERT INTO person VALUES (1, 'Fred'), (2, 'Susan');")
	stmt.Exec()

	stmt, err = sqldb.PrepareContext(ctx, "SELECT name FROM person;")
	_, err = stmt.ExecContext(context.Background())

	stmt, err = sqldb.Prepare("DROP TABLE person;")
	_, err = stmt.ExecContext(ctx)
	if err != nil {
		panic(err) // panic: database table is locked
	}
}

@mattn Can you take a look to see if this PR makes sense? Should I create an issue along with it?

As far as I look this changes, looks good to me. @rittneje could you please take a look?

@rittneje can you take a look?


Design web interface

This step will include designing the web interface:

[ ] Sketch it in paper
[ ] Write HTML and CSS for it.

First dashboard sketch


dts: edtlib: handle child-binding or child-child-binding as 'normal' binding with compatible

Is your enhancement proposal related to a problem? Please describe.
I want to define the bindings for the pinctrl state nodes of pin controllers and the pin configuration nodes of pin controllers as child-binding within the pin-controller binding (see additional context). Edtlib does not recognize the compatible in a child-binding or a child-child-binding.
Describe the solution you'd like
Handle child-bindings the same way as 'normal' bindings. Allow at least for one level of child-binding
within a child-binding.
Edtlib should find three compatibles ("level_0", "level_1", "level_2) and three bindings in the following binding file example:
description: Level 0

compatible: "level_0"

child-binding:
    description: Level 1

    compatible: "level_1"

    child-binding:
        description: Level 2

        compatible: "level_2"

Describe alternatives you've considered
Define the "pinctrl-state" binding and the "pincfg" binding in separate binding files.
Additional context
The pin controller binding looks like this:
# Copyright (c) 2018..2020 Bobby Noelte
# SPDX-License-Identifier: Apache-2.0

description: Pin controller

include: base.yaml

properties:
    pin-controller:
      type: boolean
      required: true
      description: device controller identification

    "#pinctrl-cells":
      type: int
      required: false
      description: >
        Number of pin control cells in addition to the index within the
        pin controller device instance

child-binding:
    description: >
      Pinctrl state. A node that bundles a set of pin configuration
      nodes that shall be applied in a single pin control operation.

    compatible: "pinctrl-state"

    child-binding:
        description: Pin configuration

        compatible: "pincfg"

        groups:
          descripion: >
            The list of names of the groups that properties in the node apply to
            (either this, "pins", "pinmux" or "pinctrl-pin-array" have to be
            specified).
          required: false
          type: string-array

        pins:
          descripion: >
            The list of pin identifiers that properties in the node apply to
            (either this, "groups", "pinmux" or "pinctrl-pin-array" have to be
            specified).
            The binding for the specific pin controller hardware defines whether
            the entries are of type int-array or string-array.
          required: false

        function:
          required: false
          type: string

        pinmux:
          descripion: >
            The list of numeric pin ids and their mux settings that properties
            in the node apply to (either this, "pins", "groups" or
            "pinctrl-pin-array" have to be specified).
          required: false
          type: int-array

        pinctrl-pin-array:
          descripion: >
            The list of pin controller register index and values (either this,
            "pins", "pinmux" or "groups" have to be specified). pinctrl-cells
            specifies the number of value cells in addition to the index of the
            registers. No other properties shall be in the node.
          required: false
          type: int-array

        bias-disable:
          description: Disable any pin bias.
          required: false
          type: boolean

        bias-high-impedance:
          required: false
          type: boolean

        bias-bus-hold:
          required: false
          type: boolean

        bias-pull-up:
          description: Pull up strength in Ohm. 0 to disable pull up.
          required: false
          type: int

        bias-pull-down:
          description: Pull down strength in Ohm. 0 to disable pull down.
          required: false
          type: int

        bias-pull-pin-default:
          required: false
          type: int

        drive-push-pull:
           required: false
           type: boolean

        drive-open-drain:
           required: false
           type: boolean

        drive-open-source:
            required: false
            type: boolean

        drive-strength:
           description: >
             Sink or source at most X mA. 0 to disable drive strength control.
           required: false
           type: int

        input-enable:
            required: false
            type: boolean

        input-debounce:
            description: Debounce time in usec. 0 to disable debouncing.
            required: false
            type: int

        input-disable:
            required: false
            type: boolean

        input-schmitt-enable:
            required: false
            type: boolean

        input-schmitt-disable:
            required: false
            type: boolean

        low-power-enable:
            description: Enable low power mode.
            required: false
            type: boolean

        low-power-disable:
            description: Disable low power mode.
            required: false
            type: boolean

        output-disable:
            required: false
            type: boolean

        output-enable:
            required: false
            type: boolean

        output-low:
            required: false
            type: boolean

        output-high:
            required: false
            type: boolean

        power-source:
            description: Numerical ID of power source to select.
            required: false
            type: int

        slew-rate:
            description: Slew rate
            required: false
            type: int

        skew-delay:
            description: >
              Expected clock skew on input pins and the delay before latching
              a value to an output pin.
              Typically indicated by the number of double-inverters used to
              delay the signal.
            required: false
            type: int

The pinctrl-state and pincfg bindings are currently attached to the respective nodes by a patched version of edtlib.Node._init_binding.
# @brief Initialize Node.matching_compat, Node._binding, and Node.binding_path
#
# Specially treat the /chosen node, /aliases node and the pin controller
# pinctrl state and pin configuration nodes to attach a default binding.
#
# Monkey patch for edtlib.Node._init_binding.
# To be used as long as the extended functionality is not part of edtlib.
#
# @param self refers to edtlib.Node object
def _edtlib_Node_init_binding(self):
    # Initializes Node.matching_compat, Node._binding, and
    # Node.binding_path.
    #
    # Node._binding holds the data from the node's binding file, in the
    # format returned by PyYAML (plain Python lists, dicts, etc.), or None
    # if the node has no binding.

    # This relies on the parent of the node having already been
    # initialized, which is guaranteed by going through the nodes in
    # node_iter() order.
    print("Monkey patch '{}' called."
          .format(inspect.currentframe().f_code.co_name))

    if "compatible" in self._node.props:
        self.compats = self._node.props["compatible"].to_strings()
    elif self._node.name == "chosen":
        # use default compatible for chosen node
        self.compats = ["chosen"]
    elif self._node.name == "aliases":
        # use default compatible for aliases node
        self.compats = ["aliases"]
    elif self._node.parent and \
         self._node.parent.props.get("pin-controller", False):
        # child of pin controller node
        if self._node.nodes:
            # the pin controller sub nodes has sub nodes in itself
            # Assume it is a pinctrl state node
            self.compats = ["pinctrl-state"]
        else:
            # No sub nodes - assume a pin configuration node
            self.compats = ["pincfg"]
    elif self._node.parent and self._node.parent.parent and \
         self._node.parent.parent.props.get("pin-controller", False):
        # Grand child of pin controller node
        # Assume  a pin configuration node
        self.compats = ["pincfg"]
    else:
        self.compats = []

    if self.compats:
        on_bus = self.on_bus

        for compat in self.compats:
            if (compat, on_bus) in self.edt._compat2binding:
                # Binding found
                self.matching_compat = compat
                self._binding, self.binding_path = \
                    self.edt._compat2binding[compat, on_bus]
                return
    else:
        # No 'compatible' property. See if the parent binding has a
        # 'child-binding:' key that gives the binding (or a legacy
        # 'sub-node:' key).
        binding_from_parent = self._binding_from_parent()
        if binding_from_parent:
            self._binding = binding_from_parent
            self.binding_path = self.parent.binding_path
            self.matching_compat = self.parent.matching_compat
            return

    # No binding found
    self._binding = self.binding_path = self.matching_compat = None

What should happen if some node has compatible = "level-1" but isn't the child of a node with compatible = "level-0"? Seems a bit confusing to allow a compatible in a mechanism that says "give this binding to child nodes".
I'll look at it some more later, but I wonder if there might be a simpler way.

Is this all about being able to give a custom Node.matching_compat to nodes that get their binding from child-binding? Might be able to work something out in that case.

Is this all about being able to give a custom Node.matching_compat to nodes that get their binding from child-binding?

The evident problem in my use case is that  _init_compat2binding() does not extract child bindings from a binding file. Therefor the child bindings are also NOT associated to a compatible if there is any in the child binding.
In a broader scope the problem may be that there is no mechanism to associate a binding to a node that is a child-node or child-child-node of a node that has a binding with recursive child-bindings.
I don't use a general mechanism for that in the pinctrl use case because the child-node relation may be

pin-controller -> pincfg OR
pin-controller -> pinctrl-state -> pincfg.

These pinctrl child-node relation variants seem also an issue in #20398 and may make a general mechanism more complex.
The pin-controller binding is an approximation of child-node relation type 2 (the one "somehow" preferred by Linux). The specific binding to node association does some heuristics and therefor also covers the type 1 child-node relation.
    if "compatible" in self._node.props:
        self.compats = self._node.props["compatible"].to_strings()
    ...
    elif self._node.parent and \
         self._node.parent.props.get("pin-controller", False):
        # child of pin controller node
        if self._node.nodes:
            # the pin controller sub nodes has sub nodes in itself
            # Assume it is a pinctr state node
            self.compats = ["pinctrl-state"]
        else:
            # No sub nodes - assume a pin configuration node
            self.compats = ["pincfg"]
    elif self._node.parent and self._node.parent.parent and \
         self._node.parent.parent.props.get("pin-controller", False):
        # Grand child of pin controller node
        # Assume  a pin configuration node
        self.compats = ["pincfg"]
    else:
        self.compats = []

Seems a bit confusing to allow a compatible in a mechanism that says "give this binding to child nodes".

I though this is the most simple, take the compatible mechanism that is well known.

What should happen if some node has compatible = "level-1" but isn't the child of a node with compatible = "level-0"?

Nothing, just take the compatible as it is given. I would not construct a deep DTS error analysis for such a specific case.

I stumbled across this issue while looking at something else.

In a broader scope the problem may be that there is no mechanism to associate a binding to a node that is a child-node or child-child-node of a node that has a binding with recursive child-bindings.

We have this now. Multiple levels of child binding have been supported for a while. I'm therefore going to close this issue as completed. If I've missed something and today's edtlib is still missing a feature, please reopen. Thanks.


kubelet: Disable cgroupsv2 for kubelet hosts (bug/workaround)

Motivation for this change
NixOS enables cgroupsv2 by default (systemd.enableUnifiedCgroupHierarchy=true). This breaks node memory stats for kubelet and metrics-server. Without access to node memory info, kube-scheduler cannot balance loads based on memory usage. This can cause severe instabilities on systems running under load.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested 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

[ ] Fits CONTRIBUTING.md.

@juselius got this in the meantime fixed in another way?

Yup @SuperSandro2000! The solution is simple: Just set systemd.enableUnifiedCgroupHierarchy = false; and all is well again.


uv tree --quiet doesn't show the tree

Just a thing I noticed, that,
uv tree -q

has no output. I think it's expected it shows the tree (and no superfluous status output like resolving etc)
uv 0.4.24

Yeah uv tree and uv pip list should both show output.


multiple issues with tools after migration to ts

working on it

Can't reproduce
Still working

Fixed in v0.2.2


Windows Preview: tf.exe

This is the next highest priority.  We also have to get started soon before dev15 gets too far.


Header for templates

Fixes #84
CC @Zimmi48

Since there is a new field meta_file, it should be documented in ref.yml.

Since meta.yml is fixed by ./generate.sh there is no reason to make it a field/variable.

This looks good to me, but unfortunately needs a rebase after the Nix changes.

I've had to revert the header comment for dune-project because it's apparently not allowed by Dune. I don't know if we can add comments at some other place in the file.


feat: adding retry_limit to http-outputs

What this PR does / why we need it:
This PR adds the functionality to add a Retry_Limit to http-outputs. This functionality of fluent-bit is described here: https://docs.fluentbit.io/manual/v/1.4/administration/scheduling-and-retries
This PR simply adds support for this existing functionality of fluent-bit.
Which issue(s) this PR fixes:

Fixes #
Does this PR introduced a user-facing change?

This version adds the option to specify Retry_Limits to http-outputs. The good news is if you do not want to set Retry_Limits you do not have to do anything. If you want to introduce a Retry_Limit, just add "retryLimit: <N>" where N is an integer >=1 to the specific http-output.
For more information on Retry_Limits in fluent-bit please refer top the official documentation at: https://docs.fluentbit.io/manual/v/1.4/administration/scheduling-and-retries

Additional documentation, usage docs, etc.:

@wanjunlei please review again, I think it should be good now :)
The only thing missing is a sort of check if the supplied value for Retry_Limit is an acceptable value. Right now users could supply values like retryLimit: "I am Batman" and fluent-operator would accept it and write this broken config. I could add a check that checks if "false" is supplied or any valid integer. However I wouldn't know what to do in case the supplied value is illegal. Just log it so users can see the error when watching the operators logs and not write it to any output's config or what should happen in case an illegal value is found?

@wanjunlei please review again, I think it should be good now :) The only thing missing is a sort of check if the supplied value for Retry_Limit is an acceptable value. Right now users could supply values like retryLimit: "I am Batman" and fluent-operator would accept it and write this broken config. I could add a check that checks if "false" is supplied or any valid integer. However I wouldn't know what to do in case the supplied value is illegal. Just log it so users can see the error when watching the operators logs and not write it to any output's config or what should happen in case an illegal value is found?

That is good. A simple way to check the value is used // +kubebuilder:validation:Pattern annotation, you can try it.

@wanjunlei are you ok with the new changes?

@momoXD007 Thanks very much for this PR!


getting 404 on request-docs

In my dev server, everything works great, but I'm running php 7.4.   In my production servicer, we have php 8.1.3 and I'm getting 404 on /request-docs/
Could it be the php version?

@dpreiss did you solve the issue? I have same problem in docker environment locally works and 404 on prod.

By default I don’t suggest it to enable it on prod without proper middleware.
but if it is showing 404, it could be because it is installed only as a dev dependency
composer require rakutentech/laravel-request-docs --dev
you could confirm your composer.json

I have installed it under require but still get 404.
It's development vm.
I have tried clearing cache restarting server and php-fpm but nothing works.
I can see the request-docs end point in the routes list.
Regards,


MaterialComboBox might need property UseLightCursor

Test out how the combo box reacts on a dark background, when it is set as editable.

Not required anymore. See Foundation library.


Object reference not set to an instance of an object

Repro steps

Create a default Fsharp project with mono-develop

Build this project

Expected behavior
Build should be success without any warning nor error.
Actual behavior
/usr/lib/mono/fsharp/Microsoft.FSharp.Targets: error : Error executing task Fsc: Object reference not set to an instance of an object
Related information

This bug happens with master version.
The version <IP_ADDRESS> can compile this project without any error.

What version of monodevelop please?
Also, it seems this bug should be submitted in http://bugzilla.xamarin.com/, not here.

I am not sure whether it is a bug of MonoDevelop or not. Someone has a similar issue here. I found also that the file Microsoft.FSharp.Targets and the corresponding dll of the task Fsc are generated when building and installing fsharp.

I am not sure whether it is a bug

To figure that out, we need to reproduce your exact problem. And to do that, we need more information from you. You still haven't told us what operating system are you using, what version of it, and what version of monodevelop (and how did you install it).

Closing as this seems to be external

@knocte and @dsyme : Sorry for this retard but I think this is not an external bug (i.e. coming from MonoDevelop) since I can compile successfully the same projects with fsharp-<IP_ADDRESS>, whereas the master and <IP_ADDRESS>.
I am using now MonoDevelop version 5.10 and Fedora 24 (kernel 4.7.6-200.fc24.x86_64).

Check if you can reproduce the issue with MonoDevelop 6.1 please.

@tathanhdinh Thanks the update

This looks similar to the failing homebrew PR (log output). Is there an incompatibility between mono <IP_ADDRESS> and fsharp > <IP_ADDRESS>?

@samritchie Not aware of any specific problems besides the logs linked here
CI is passing OK for <IP_ADDRESS>, e.g. see the history here: https://travis-ci.org/fsharp/fsharp/
Mono JIT compiler version 4.6.1 (Stable <IP_ADDRESS>/ef43c15 Wed Oct 12 09:10:37 UTC 2016)
Copyright (C) 2002-2014 Novell, Inc, Xamarin Inc and Contributors. www.mono-project.com

Thanks @dsyme, as far as I can tell, this was a homebrew sandboxing problem (https://github.com/Homebrew/homebrew-core/issues/5302). A PR for the latest mono/fsharp has been merged now.

Hmm, after I updated via mono I also started getting the error. It looks like it’s triggered when you have an fsproj with no <DefineConstants> set, meaning it will break on some templates & existing projects.
I’m unsure whether this is the correct behaviour or not?

Ah, nm - I’ve just seen #628

@samritchie @dsyme : many thanks, I believe that #628 is also the problem reported here.

@tathanhdinh Great


Initial Kubernetes support

Basic support is going to be released in version 1.2.0.
Verify if the following features work for Kubernetes. Provide error message and GitHub issue if not (either in fablo-k8s.sh, fablo.sh, _validateEngineSpecificSettings):

[ ] up, down, reset, start stop
[ ] prune, recreate
[ ] chaincode install/upgrade
[ ] dev mode?
[ ] TLS (won't be supported) - serve validation error @Hejwo
[ ] snapshot, restore
[ ] Orderer - Solo or RAFT
[ ] CA db - embedded or postgres
[ ] Peer db - embedded or couch
[ ] Custom endorsement policy
[ ] Post generate hook
[ ] Channel query scripts
[ ] Orderer sharding
[ ] Fabric in versions lower than 2.0
[ ] Blockchain explorer (won't be supported)
[ ] Monitoring - log level
[ ] Fablo REST (won't be supported?)
[ ] Number of peer instances
[ ] Number of anchor peer instances
[ ] Number of orderer instances
[ ] Private data collections
[ ] Setting versions of CA, HLF etc on Kubernetes Operators @dzikowski https://github.com/hyperledger-labs/fablo/pull/373

Additional tasks:

[ ] Update default CA to 1.5.5 @dzikowski https://github.com/hyperledger-labs/fablo/pull/373
[ ] Create script for building Fabric images for Apple M1 (./fablo build-images <arm64>) @dzikowski

Hey @dzikowski , I want to work on this aspect and improve Kubernetes operators support for Fablo. I have applied LFX mentorship program for the term June-Nov 2023.
Hope you look into my application and get accepted!
I want guidance on what I have to do.

Basic, experimental support added in Fablo 1.2.0. This is a subject of further testing and improvements in separate issues


Selected row unset on click with Ctrl(Cmd)+click in editableCell === false mode

https://github.com/GigaTables/reactables/releases/tag/2.9.5


Kuberctld stopped with no error logs

Rancher versions: 1.6.10
Docker version: (docker version,docker info preferred) 1.12.6
Operating system and kernel: (cat /etc/os-release, uname -r preferred) Ubuntu16.04
Type/provider of hosts: (VirtualBox/Bare-metal/AWS/GCE/DO) Bare-metal
Setup details: (single node rancher vs. HA rancher, internal DB vs. external DB) single
Environment Template: (Cattle/Kubernetes/Swarm/Mesos) Kubernetes
Hi, all
I have a strange issue on kubectld, it restarts everyday, and one day it stopped, but no error logs there, please help, thanks.
Container kubernetes-kubectld-1 stopped without any errors:
11/15/2017 10:23:39 PMtime="2017-11-15T14:23:39Z" level=info msg="Connection established"
11/15/2017 10:23:39 PMtime="2017-11-15T14:23:39Z" level=info msg="Starting websocket pings"
11/16/2017 10:24:43 AM$HELM_HOME has been configured at /root/.helm.
11/16/2017 10:24:43 AMNot installing Tiller due to 'client-only' flag having been set
11/16/2017 10:24:43 AMHappy Helming!
11/16/2017 10:24:43 AMtime="2017-11-16T02:24:43Z" level=info msg="Listening for health checks on <IP_ADDRESS>:10240/healthcheck"
11/16/2017 10:24:43 AMtime="2017-11-16T02:24:43Z" level=info msg="Initializing event router" workerCount=50
11/16/2017 10:24:43 AMtime="2017-11-16T02:24:43Z" level=info msg="Connection established"
11/16/2017 10:24:43 AMtime="2017-11-16T02:24:43Z" level=info msg="Starting websocket pings"
11/16/2017 10:26:27 PM$HELM_HOME has been configured at /root/.helm.
11/16/2017 10:26:27 PMNot installing Tiller due to 'client-only' flag having been set
11/16/2017 10:26:27 PMHappy Helming!
11/16/2017 10:26:27 PMtime="2017-11-16T14:26:27Z" level=info msg="Listening for health checks on <IP_ADDRESS>:10240/healthcheck"
11/16/2017 10:26:27 PMtime="2017-11-16T14:26:27Z" level=info msg="Initializing event router" workerCount=50
11/16/2017 10:26:27 PMtime="2017-11-16T14:26:27Z" level=info msg="Connection established"
11/16/2017 10:26:27 PMtime="2017-11-16T14:26:27Z" level=info msg="Starting websocket pings"
11/17/2017 10:27:10 AM$HELM_HOME has been configured at /root/.helm.
11/17/2017 10:27:10 AMNot installing Tiller due to 'client-only' flag having been set
11/17/2017 10:27:10 AMHappy Helming!
11/17/2017 10:27:10 AMtime="2017-11-17T02:27:10Z" level=info msg="Listening for health checks on <IP_ADDRESS>:10240/healthcheck"
11/17/2017 10:27:10 AMtime="2017-11-17T02:27:10Z" level=info msg="Initializing event router" workerCount=50
11/17/2017 10:27:10 AMtime="2017-11-17T02:27:10Z" level=info msg="Connection established"
11/17/2017 10:27:10 AMtime="2017-11-17T02:27:10Z" level=info msg="Starting websocket pings"
11/17/2017 10:28:03 PM$HELM_HOME has been configured at /root/.helm.
11/17/2017 10:28:03 PMNot installing Tiller due to 'client-only' flag having been set
11/17/2017 10:28:03 PMHappy Helming!
11/17/2017 10:28:03 PMtime="2017-11-17T14:28:03Z" level=info msg="Listening for health checks on <IP_ADDRESS>:10240/healthcheck"
11/17/2017 10:28:03 PMtime="2017-11-17T14:28:03Z" level=info msg="Initializing event router" workerCount=50
11/17/2017 10:28:03 PMtime="2017-11-17T14:28:03Z" level=info msg="Connection established"
11/17/2017 10:28:03 PMtime="2017-11-17T14:28:03Z" level=info msg="Starting websocket pings"
11/18/2017 10:29:21 AM$HELM_HOME has been configured at /root/.helm.
11/18/2017 10:29:21 AMNot installing Tiller due to 'client-only' flag having been set
11/18/2017 10:29:21 AMHappy Helming!
11/18/2017 10:29:21 AMtime="2017-11-18T02:29:21Z" level=info msg="Listening for health checks on <IP_ADDRESS>:10240/healthcheck"
11/18/2017 10:29:21 AMtime="2017-11-18T02:29:21Z" level=info msg="Initializing event router" workerCount=50
11/18/2017 10:29:21 AMtime="2017-11-18T02:29:21Z" level=info msg="Connection established"
11/18/2017 10:29:21 AMtime="2017-11-18T02:29:21Z" level=info msg="Starting websocket pings"
11/18/2017 10:30:19 PM$HELM_HOME has been configured at /root/.helm.
11/18/2017 10:30:19 PMNot installing Tiller due to 'client-only' flag having been set
11/18/2017 10:30:19 PMHappy Helming!
11/18/2017 10:30:19 PMtime="2017-11-18T14:30:19Z" level=info msg="Listening for health checks on <IP_ADDRESS>:10240/healthcheck"
11/18/2017 10:30:19 PMtime="2017-11-18T14:30:19Z" level=info msg="Initializing event router" workerCount=50
11/18/2017 10:30:19 PMtime="2017-11-18T14:30:19Z" level=info msg="Connection established"
11/18/2017 10:30:19 PMtime="2017-11-18T14:30:19Z" level=info msg="Starting websocket pings"
11/19/2017 10:31:19 AM$HELM_HOME has been configured at /root/.helm.
11/19/2017 10:31:19 AMNot installing Tiller due to 'client-only' flag having been set
11/19/2017 10:31:19 AMHappy Helming!
11/19/2017 10:31:19 AMtime="2017-11-19T02:31:19Z" level=info msg="Listening for health checks on <IP_ADDRESS>:10240/healthcheck"
11/19/2017 10:31:19 AMtime="2017-11-19T02:31:19Z" level=info msg="Initializing event router" workerCount=50
11/19/2017 10:31:19 AMtime="2017-11-19T02:31:19Z" level=info msg="Connection established"
11/19/2017 10:31:19 AMtime="2017-11-19T02:31:19Z" level=info msg="Starting websocket pings"
11/19/2017 10:32:17 PM$HELM_HOME has been configured at /root/.helm.
11/19/2017 10:32:17 PMNot installing Tiller due to 'client-only' flag having been set
11/19/2017 10:32:17 PMHappy Helming!
11/19/2017 10:32:17 PMtime="2017-11-19T14:32:17Z" level=info msg="Listening for health checks on <IP_ADDRESS>:10240/healthcheck"
11/19/2017 10:32:17 PMtime="2017-11-19T14:32:17Z" level=info msg="Initializing event router" workerCount=50
11/19/2017 10:32:17 PMtime="2017-11-19T14:32:17Z" level=info msg="Connection established"
11/19/2017 10:32:17 PMtime="2017-11-19T14:32:17Z" level=info msg="Starting websocket pings"
11/20/2017 10:33:05 AM$HELM_HOME has been configured at /root/.helm.
11/20/2017 10:33:05 AMNot installing Tiller due to 'client-only' flag having been set
11/20/2017 10:33:05 AMHappy Helming!
11/20/2017 10:33:05 AMtime="2017-11-20T02:33:05Z" level=info msg="Listening for health checks on <IP_ADDRESS>:10240/healthcheck"
11/20/2017 10:33:05 AMtime="2017-11-20T02:33:05Z" level=info msg="Initializing event router" workerCount=50
11/20/2017 10:33:05 AMtime="2017-11-20T02:33:05Z" level=info msg="Connection established"
11/20/2017 10:33:05 AMtime="2017-11-20T02:33:05Z" level=info msg="Starting websocket pings"
11/20/2017 10:33:47 PM$HELM_HOME has been configured at /root/.helm.
11/20/2017 10:33:47 PMNot installing Tiller due to 'client-only' flag having been set
11/20/2017 10:33:47 PMHappy Helming!
11/20/2017 10:33:47 PMtime="2017-11-20T14:33:47Z" level=info msg="Listening for health checks on <IP_ADDRESS>:10240/healthcheck"
11/20/2017 10:33:47 PMtime="2017-11-20T14:33:47Z" level=info msg="Initializing event router" workerCount=50
11/20/2017 10:33:47 PMtime="2017-11-20T14:33:47Z" level=info msg="Connection established"
11/20/2017 10:33:47 PMtime="2017-11-20T14:33:47Z" level=info msg="Starting websocket pings"
11/21/2017 10:35:02 AM$HELM_HOME has been configured at /root/.helm.
11/21/2017 10:35:02 AMNot installing Tiller due to 'client-only' flag having been set
11/21/2017 10:35:02 AMHappy Helming!
11/21/2017 10:35:02 AMtime="2017-11-21T02:35:02Z" level=info msg="Listening for health checks on <IP_ADDRESS>:10240/healthcheck"
11/21/2017 10:35:03 AMtime="2017-11-21T02:35:03Z" level=info msg="Initializing event router" workerCount=50
11/21/2017 10:35:03 AMtime="2017-11-21T02:35:03Z" level=info msg="Connection established"
11/21/2017 10:35:03 AMtime="2017-11-21T02:35:03Z" level=info msg="Starting websocket pings"
11/21/2017 10:36:13 PM$HELM_HOME has been configured at /root/.helm.
11/21/2017 10:36:13 PMNot installing Tiller due to 'client-only' flag having been set
11/21/2017 10:36:13 PMHappy Helming!
11/21/2017 10:36:13 PMtime="2017-11-21T14:36:13Z" level=info msg="Listening for health checks on <IP_ADDRESS>:10240/healthcheck"
11/21/2017 10:36:13 PMtime="2017-11-21T14:36:13Z" level=info msg="Initializing event router" workerCount=50
11/21/2017 10:36:13 PMtime="2017-11-21T14:36:13Z" level=info msg="Connection established"
11/21/2017 10:36:13 PMtime="2017-11-21T14:36:13Z" level=info msg="Starting websocket pings"
11/22/2017 10:37:50 AM$HELM_HOME has been configured at /root/.helm.
11/22/2017 10:37:50 AMNot installing Tiller due to 'client-only' flag having been set
11/22/2017 10:37:50 AMHappy Helming!
11/22/2017 10:37:50 AMtime="2017-11-22T02:37:50Z" level=info msg="Listening for health checks on <IP_ADDRESS>:10240/healthcheck"
11/22/2017 10:37:50 AMtime="2017-11-22T02:37:50Z" level=info msg="Initializing event router" workerCount=50
11/22/2017 10:37:50 AMtime="2017-11-22T02:37:50Z" level=info msg="Connection established"
11/22/2017 10:37:50 AMtime="2017-11-22T02:37:50Z" level=info msg="Starting websocket pings"
11/22/2017 10:39:06 PM$HELM_HOME has been configured at /root/.helm.
11/22/2017 10:39:06 PMNot installing Tiller due to 'client-only' flag having been set
11/22/2017 10:39:06 PMHappy Helming!
11/22/2017 10:39:06 PMtime="2017-11-22T14:39:06Z" level=info msg="Listening for health checks on <IP_ADDRESS>:10240/healthcheck"
11/22/2017 10:39:06 PMtime="2017-11-22T14:39:06Z" level=info msg="Initializing event router" workerCount=50
11/22/2017 10:39:06 PMtime="2017-11-22T14:39:06Z" level=info msg="Connection established"
11/22/2017 10:39:06 PMtime="2017-11-22T14:39:06Z" level=info msg="Starting websocket pings"

https://github.com/rancher/rancher/issues/9484


中文路径下调试有问题|a bug when debug cpp in Chinese dir

VSCode Version:1.11.2
OS Version:Windows 7 x64

Steps to Reproduce:
我在“D:\2017编程”下放了Simple Tree.cpp，设置好lauch.json和tasks.json后进行调试（使用gdb），能成功运行生成的exe并进入调试，但断点无效，调试中按F10后会出现错误提示：

无法打开“Simple Tree.cpp”: 找不到文件(D:\2017261340263314\Simple Tree.cpp)

在英文路径下则调试正常

I put my src("Simple Tree.cpp") in "D:\2017编程", then use gdb to debug. Although i can run "Simple Tree.exe", breakpoints don't work. When i press F10, it throw an error message:

can not open "Simple Tree.cpp": can't find the file (D:\2017261340263314\Simple Tree.cpp)

However, all works normally when I put my file in English dir.

Please file against the CPP debugger extension

when I debug react-native, I've had this problem, too.

How should I solve this problem?


Add back ML type fixes

This reverts commit d2680ae3b660ef6b7a5c7e799f13e232183ea157.
Most of these changes are ok, but there are some of them which are not, please take a look and test in the Kibana codebase to validate if the generated types are what you expect, thanks!
cc @elastic/ml-core & @elastic/ml-ui

It might be good to think up a naming convention for pre and post saved jobs and their related types.
We have request and response versions for some of the types, e.g. MlPutJobRequest  which is very similar to MlJob. but both contain the same MlAnalysisConfig
Adding request to the name of the version of MlAnalysisConfig that is used in MlPutJobRequest doesn't seem right as it isn't used on its own as a request payload.
I'm not suggesting replacing all request and response types. But in special cases like MlPutJobRequest the body could be something like MlJobPreSave (terrible name) which contains MlAnalysisConfigPreSave and the response MlJobPostSave which contains MlAnalysisConfigPostSave
Also, I just notices that MlPutJobResponse has its own type which is separate from MLJob. These should be the same.


Update conda-forge to release 0.4.3?

Thanks @ResidentMario  for a great package, and for adding NearviewPerspective to 0.4.3 :) I was precisely looking for it yesterday night!
Any idea how to help conda-forge get up to speed on that new release? It's still shipping 0.4.1, not 0.4.3 yet. Thanks!

It's unclear (to me) why the bot that usually handles this sort of thing hasn't minted a new PR to the geoplot-feedstock pulling in the update. It's been 24 hours so I've asked about it in the conda-forge gitter.

Thanks Aleksey!
On Mon, 5 Jul 2021, 17:05 Aleksey Bilogur, @.***> wrote:

It's unclear (to me) why the bot that usually handles this sort of thing
hasn't minted a new PR to the geoplot-feedstock pulling in the update.
It's been 24 hours so I've asked about it in the conda-forge gitter.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ResidentMario/geoplot/issues/246#issuecomment-874214536,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFBERJ5CI2AD4PXUAV2WVDTWHJ5XANCNFSM472NPEMQ
.

Neglected to close this issue out. It turned to be because conda requires a source distribution, and the method I used to push to PyPI initially only pushed a wheel. Comment there here in gitter, for posterity.

Awesome, thanks @ResidentMario !


improve CPU utilisation by adding timeout to main thread loop

So, why does this improve CPU utilization? Presumably this would only work if there was a busy wait somewhere downstream of this... where is the busy wait?
Does this add any latency?
I don't have time to go through the whole PSMoveService logic right now, but I'm really hesitant to think that simply adding a sleep is a good idea. If you can isolate where the busy wait is then we should fix it there.

@cboulay It's not actually adding a new sleep but instead adding a configurable value to an existing sleep in the main loop. The original value in that sleep was 1ms, which I think I might have left in as experimently low value and forgot to take out (it probably should be like 5ms at a minimum). I agree that we should understand where the busy waiting is occurring (maybe in the camera frame polling?), but seeing as how this PR doesn't actually change any existing functionality and it can help some people who were having CPU usage issues I think it's a useful knob to add in for the time being.
When I tested an earlier version of the change I got nearly the same update frame rate on the console test client when this sleep was 20ms instead of 1ms and the CPU usage dropped from 25-30% down to 15%.
@gb2111 I think we should keep the default at 1ms just to keep the behavior the same for now until we understand more deeply where busy-waiting is occurring.

60 Hz = 16.6666 msec per frame, so sleeping 20 msec necessarily misses every other frame and therefore I would hope that would get you a significant speedup.
If there's no difference between 1 and X (X = 16 msec - frame processing time) then this is probably not doing anything other than missing frames.
Also boost::this_thread::sleep is deprecated and should be replaced with boost::this_thread::sleep_for, or better yet, std::this_thread::sleep_for.

60 Hz = 16.6666 msec per frame, so sleeping 20 msec necessarily misses every other frame and therefore I would hope that would get you a significant speedup.

That... is an excellent point. I am embarrassed I missed that. Though that does make me wonder why I was getting around 30fps when testing on my machine with the 1ms sleep (maybe I tested with a debug build?).
I any event, I still think it's still worth having a configurable value there. If a user really wants to run at a slower frame rate to get a game running I think that's a reasonable knob. It's also helpful for debugging. The filtering code takes in the time between updates and can handle variable frame rates.

Also boost::this_thread::sleep is deprecated and should be replaced with boost::this_thread::sleep_for, or better yet, std::this_thread::sleep_for.

Also a good point. My vote is for std::this_thread::sleep_for. I'd like to slowly keep working toward removing dependence on boost libraries, particularly for functionally that already exist in the standard library.

@gb2111 ,
Please switch to using std::this_thread::sleep_for (which will also require converting the time argument to a std::chrono::duration).
Please add a comment to the sleep_for line like //TODO: Find out if there is a downstream busy wait.
Please change the parameter name from thread_timeout_ms to something like tracker_sleep_ms, which indicates that it is being used by the tracker exclusively and that it isn't a timeout. A "timeout", to me, is the amount of time a process is allowed to work on something before it must return, even if that means returning without a complete result. Here you aren't limiting the tracker processing time at all, and might even be lengthening it.
@HipsterSloth , does the #include "TrackerManager.h" to PSMoveService.cpp add any awkward dependencies that we'd like to avoid, maybe creating a circular dependency in the future when we add in other features we're interested in? I haven't thought about this at all, but I'm always going to ask this question whenever I happen to notice a PR with more includes.

Does the #include "TrackerManager.h" to PSMoveService.cpp add any awkward dependencies

Good question. I didn't see any circular dependencies, but I have mixed feeling about "PSMoveProtocol.pb.h" getting pulled in (ultimately due to DeviceTypeManager::getListUpdatedResponseType). I think that could come as part of another refactor. I actually have another task for cleaning up some unwanted dependencies in PSMoveService.cpp in this issue: https://github.com/cboulay/PSMoveService/issues/39


Your Discord invitation expired

As I said.
Your Discord invitation expired.
Hopefully, you'll resend your discord invitation.

Oh, Yes, I will Send Here, Thanks.
Discord Link


Fix shape inference in MobileNet v3

Replace self-written hard_sigmoid with activation layer to solve https://github.com/keras-team/keras/issues/15282

Hi @shkarupa-alex Can you please check @nkovela1's comments and keep us posted ? Thank you!

@nkovela1 , could you please share the description and stack trace of test failure you got?
I've made a local dev container as described here https://github.com/keras-team/keras/blob/master/CONTRIBUTING.md
and MobileNetV3-related tests are passed:
root ➜ /home/keras (patch-2) $ bazel test --test_output=errors //keras/applications:applications_load_weight_test_mobilenet_v3_small
WARNING: The following configs were expanded more than once: [v2]. For repeatable flags, repeats are counted twice and may lead to unexpected behavior.
INFO: Options provided by the client:
  Inherited 'common' options: --isatty=1 --terminal_columns=243
INFO: Reading rc options for 'test' from /home/keras/.bazelrc:
  Inherited 'build' options: --apple_platform_type=macos --define open_source_build=true --define=use_fast_cpp_protos=false --define=tensorflow_enable_mlir_generated_gpu_kernels=0 --define=allow_oversize_protos=true --spawn_strategy=standalone -c opt --announce_rc --define=grpc_no_ares=true --config=short_logs --config=v2
INFO: Reading rc options for 'test' from /home/keras/.bazelrc:
  'test' options: --define open_source_build=true --define=use_fast_cpp_protos=false --config=v2
INFO: Found applicable config definition build:short_logs in file /home/keras/.bazelrc: --output_filter=DONT_MATCH_ANYTHING
INFO: Found applicable config definition build:v2 in file /home/keras/.bazelrc: --define=tf_api_version=2 --action_env=TF2_BEHAVIOR=1
INFO: Found applicable config definition build:v2 in file /home/keras/.bazelrc: --define=tf_api_version=2 --action_env=TF2_BEHAVIOR=1
INFO: Analyzed target //keras/applications:applications_load_weight_test_mobilenet_v3_small (0 packages loaded, 0 targets configured).
INFO: Found 1 test target...
Target //keras/applications:applications_load_weight_test_mobilenet_v3_small up-to-date:
  bazel-bin/keras/applications/applications_load_weight_test_mobilenet_v3_small
INFO: Elapsed time: 6.156s, Critical Path: 5.80s
INFO: 2 processes: 1 internal, 1 local.
INFO: Build completed successfully, 2 total actions
//keras/applications:applications_load_weight_test_mobilenet_v3_small    PASSED in 5.8s

Executed 1 out of 1 test: 1 test passes.
INFO: Build completed successfully, 2 total actions

root ➜ /home/keras (patch-2) $ bazel test --test_output=errors //keras/applications:applications_load_weight_test_mobilenet_v3_large
WARNING: The following configs were expanded more than once: [v2]. For repeatable flags, repeats are counted twice and may lead to unexpected behavior.
INFO: Options provided by the client:
  Inherited 'common' options: --isatty=1 --terminal_columns=243
INFO: Reading rc options for 'test' from /home/keras/.bazelrc:
  Inherited 'build' options: --apple_platform_type=macos --define open_source_build=true --define=use_fast_cpp_protos=false --define=tensorflow_enable_mlir_generated_gpu_kernels=0 --define=allow_oversize_protos=true --spawn_strategy=standalone -c opt --announce_rc --define=grpc_no_ares=true --config=short_logs --config=v2
INFO: Reading rc options for 'test' from /home/keras/.bazelrc:
  'test' options: --define open_source_build=true --define=use_fast_cpp_protos=false --config=v2
INFO: Found applicable config definition build:short_logs in file /home/keras/.bazelrc: --output_filter=DONT_MATCH_ANYTHING
INFO: Found applicable config definition build:v2 in file /home/keras/.bazelrc: --define=tf_api_version=2 --action_env=TF2_BEHAVIOR=1
INFO: Found applicable config definition build:v2 in file /home/keras/.bazelrc: --define=tf_api_version=2 --action_env=TF2_BEHAVIOR=1
INFO: Analyzed target //keras/applications:applications_load_weight_test_mobilenet_v3_large (0 packages loaded, 0 targets configured).
INFO: Found 1 test target...
Target //keras/applications:applications_load_weight_test_mobilenet_v3_large up-to-date:
  bazel-bin/keras/applications/applications_load_weight_test_mobilenet_v3_large
INFO: Elapsed time: 6.792s, Critical Path: 6.45s
INFO: 2 processes: 1 internal, 1 local.
INFO: Build completed successfully, 2 total actions
//keras/applications:applications_load_weight_test_mobilenet_v3_large    PASSED in 6.4s

Executed 1 out of 1 test: 1 test passes.
INFO: Build completed successfully, 2 total actions

HardSigmoid used in MobileNetV3 does not match keras hard_sigmoid (different slopes).
Fixed this with layers.Add-Multiply to prevent Lambda wrapper.
Test pointed by @nkovela1 now passes.


[Turbo] Update @hotwired/turbo - v7.1.0

@hotwired/turbo 7.1.0 is released:
https://github.com/hotwired/turbo/releases/tag/v7.1.0

I think we should keep it at 7.0 because we don't "need" the user to be using 7.1

I agree with Ryan, 7.0 is the minimal version, we always recommend using the latest minor if possible.


Products tags

I need a way to set tags in a product. Those tags will be useful to organize products in sections, for searching, to display related products in their pages and also to better categorize somethings for the users.
An example for clothing would be: I have a product named T-Shirt Foo Bar. I can tag it with summer-2017, beach, casual, etc.
At first I though to create an attribute for it. The first problem is that there's no way to define multi-valued attributes atm. Other problem is that it would be hard to integrate with the search/product page/etc since it's something custom and created by the user.
The other solution that I thought was to define it in Product class. Just like a product have a name and a price I think it can also have tags.
What do you think for the proposal?
I'll try to implement this (unless someone is already working on something like that) and would really appreciate your feedback since I would like to contribute it back to you.

Hey, we're planning to change current categories to taxonomies (tree structure, many-to-many) while introducing a strict one-to-many relation to a category. Would this solve your case?

@patrys that probably would solve my case! Is that already being worked on? I might need something like that for mid december max.
I can help with the development of the feature also unless there's already enough people working on it.

Feel free to help! 👍 I don't think anyone is working on this at the moment.

@patrys great! I'll assign this issue to me then.
Btw, do you have a public dev chat place where I can talk to you if I need to ask something quickly? Like an irc channel, a slack room, etc?

@hackedbellini
There is gitter, but it seems pretty dead.
https://gitter.im/mirumee/saleor

@patrys

Hey, we're planning to change current categories to taxonomies (tree structure, many-to-many) while introducing a strict one-to-many relation to a category. Would this solve your case?

Hey, I am currently being concern to product taxonomy,  I just realized that this case was already discussed. Would you update me how is going on on now with this plan?
I am interested to help in case no body working on it yet. I came to this case as I need to get the taxonomy in my language. I think many user will happy if it available in their language.
So I would propose you this scenario:

Would you please consider to use Google Product Categories shown in Product Taxonomy - Google. I think it is a right way to start from that point.
I would very happy if you include this as a project in your Transifex - Mirumee. I just checked that there are no similar project exist there for either product, nor category nor taxonomy.

Thanks in advance your your consideration.

Hello @patrys ,
Thanks for this awesome project. I was wondering has this product taxonomy feature implemented or in active development?

Hey, we're planning to change current categories to taxonomies (tree structure, many-to-many) while introducing a strict one-to-many relation to a category. Would this solve your case?


Install both eaxefx_bin_win32 AND eaxefx_bin_win64?

Do I need to install both eaxefx_bin_win32_v1_0_6.zip AND eaxefx_bin_win64_v1_0_6.zip binary files? The original Host OpenAL installs OpenAL32 files in both %systemroot%\system32 and %systemroot%\SysWOW64 folders so I am assuming both eaxefx 32 and 64 bit files need copying, right?
Please correct me if I am wrong. System32 is for 64 bit files and SysWOW64 is for 32 bit files, right? So the OpenAL32.dll from eaxefx_bin_win64_v1_0_6.zip goes in %systemroot%\system32 \ folder
and OpenAL32.dll from eaxefx_bin_win32_v1_0_6.zip goes in %systemroot%\SysWOW64\ folder, right?
Thanks!

Yes.
But take note that they are independent of each other.
So unless you have 64-bit openal games, you'd usually just need the one in SysWOW64.
(also remember that if the game ships with its own version of openal32, it will override the system one)

Understood. Thank you!


SQLite: decimal literals formatted incorrectly

When I insert a decimal value into a Sqlite database using EntityFramework Core, it will save it in a Text column.  This is fine and prevents loss of data, but sometimes makes you unable to find a value you added before
Steps to reproduce

Create Sqlite DbContext
Insert Entity with a decimal value without decimal places
SaveChanges
Try to find by inserted value

using(var db = new MyDbContext()) {
    db.Database.EnsureCreated();
    db.Add( new MyEntity() { Value = 3m } );
    db.SaveChanges();

    // Creates error
    db.Entities.First( x => x.Value == 3m );
}

It adds the value as "3" into the database, but uses "3.0" in the query. It seems to not use the same ToString()
Further technical details
EF Core version: 1.1.1
Database Provider:  Microsoft.EntityFrameworkCore.Sqlite
Operating system: Win7 x86
IDE: Visual Studio 2017
Edit:
This works for some reason:
using(var db = new MyDbContext()) {
    db.Database.EnsureCreated();
    decimal value = 3m;
    db.Add( new MyEntity() { Value = value } );
    db.SaveChanges();

    // Works as expected
    var y = db.Entities.First( x => x.Value == value );
}

Looks like we need to add the following.
partial class SqliteSqlGenerationHelper
{
    protected overrride string DecimalFormat => "G";
}

BUT this is only the tip of the iceberg. Any arithmetic operation will coerce the values into doubles before performing the operation resulting in lossy results. In general, using decimal values (and any non-SQLite-primitive type like DateTime, DateTimeOffset, and TimeSpan) in an expression that gets evaluated on the server should be carefully reviewed for potential errors due to coercion.

Oh wait, the above code probably isn't sufficient. They need to be TEXT literals.

We should review all literals too--we had anther issue with Guid in 1.0.0.

It looks like double and float literals are also wrong:
SQLite Error 1: 'unrecognized token: "1.79769313486232E+308E0"'.

@smitpatel the logic used by Microsoft.Data.Sqlite is value.ToString(CultureInfo.InvariantCulture) quoted as a string literal.

This should be fixed in SQLite to allow lossless arithmetic operations.

lossless less lossy

Fixed by aspnet/Microsoft.Data.Sqlite#381

This issue was moved to aspnet/Microsoft.Data.Sqlite#382


fix: Str literal in code/math mode to be preserved

fixes #116.
Maybe some tests should be added (I didn't know where to add it).

Amazing thanks!
You could add one or two test_eq! in lisb/src/tests/snippets.rs with the name string_literal
I'd suggest this $ a "        x" $

Thank you very much! Awesome !


Can not perform multiple sync() on morphToMany relationship

Laravel Version: 6.18.17
PHP Version: 7.2.24
Database Driver & Version: MariaDB 10.0

Description:
Can not perform multiple sync() on single model instrance for morphToMany relationship. On second sync is thrown BadMethodCallException : Call to undefined method Illuminate\Database\Eloquent\Relations\Pivot::setMorphType()
The problem has occurred in the version 6.18.17. The version 6.18.16 works fine.
Steps To Reproduce:

create model with morphToMany relationship

    public function settings()
    {
        return $this->morphToMany(Setting::class, 'has_setting');
    }

Sync multiple times in one request

$model = factory(Model::class)->create();
// Works
$model->tags()->sync([1]);
// Throws BadMethodCallException : Call to undefined method Illuminate\Database\Eloquent\Relations\Pivot::setMorphType()
$model->tags()->sync([2]);

Can confirm this is an issue. This appears to be the commit that broke:
https://github.com/laravel/framework/commit/684208b10460b49fa34354cc42f33b9b7135814f#diff-3c8f4b63b093b84a3e3d4e862743dec0
When $record is an instance of Illuminate\Database\Eloquent\Relations\Pivot, setMorphType is not an available method.

I have the same problem, however in version 7.14. In version 7.13. * Did not present this problem.

Also experiencing this with 7.14.0, 7.13.0 works fine. Same commit to blame. as reported by @ralphschindler
For me the bug is showing when using spatie laravel-permission. In a test I ssign roles in two separate calls and I get this error.
This line in spatie laravel-permission triggers the issue:
https://github.com/spatie/laravel-permission/blob/e25986c298b921e884a0a93a6f09f81c0c9cdb5a/src/Traits/HasRoles.php#L116

Can confirm this issue in 7.14.

Will release a patch.


"CIDR6" terminology is bad

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
Files and docs use "CIDR6" as a term.
CIDR means "classless-interdomain routing" which term is a legacy IPv4-only term and makes no sense for IPv6 since the concept of "class" does not exist (and the concept of "class" in IPv4 is deprcated).  The non-deprecated term in IPv4 is just "prefix" which is what I think Cilium means.   It would be more consistent with modern standards to just say "IPv4 prefix" and "IPv6 prefix" than trying to imply that "CIDR" applies to IPv6.
Cilium Version
Current
Kernel Version
N/A
Kubernetes Version
N/A
Sysdump
No response
Relevant log output
No response
Anything else?
https://kubernetes.io/docs/concepts/services-networking/dual-stack/ is similarly confused and should really be fixed.
Code of Conduct

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

Looks like wikipedia uses it consistently to apply to IPv6, so closing for now.


Add information about hddimg and installer

Signed-off-by: Simon A. Eugster<EMAIL_ADDRESS>Description
Adds example on installing an image on a target system which does not have (easily) removable storage.

@Granjow: Thanks! I don't think we can include the part about hddimg in the official documentation, because this is not something we test, and I would not expect this to work in all scenarios. I have modified the text a little bit and resubmitted in https://github.com/mendersoftware/mender-docs/pull/1606.

Superseded by #1606.

@kacf I understand – thanks for merging!


Add .editorconfig file

"helps developers define and maintain consistent coding styles between different editors and IDEs"
http://editorconfig.org/

Why close?

Oops, an errant mouse click! Meant to edit and say: creating a pull request

How do yo

Yeah those two buttons are confusing


Discuss large tables and how to deal with those

#682 ?

Some text written in Figjam on this subject

Link to Figma: https://www.figma.com/file/xm1aNey7RrP4llwou1t8OX/Envis-content-Figjam?version-id=1641021455&node-id=1205%3A16069


Keep document titles in database updated

The refs table has a title column that is set (to "Untitled") when the ref is created, but it is never updated. Thus, it is not possible to get the title of a document from a database query without parsing the full JSON content. The title column should be updated by the backend whenever the document is (auto)saved.

On second thought, perhaps we should just store the document content as JSON using Postgres' native support for such and query against that. Then we don't have to store things like titles redundantly and worry about keeping them in sync.

Will be resolved in #211 by the means described in the previous comment.


Git not available.

Running CentOS7, simple-php-git-deploy 1.3.1
# git --version
git version <IP_ADDRESS>

However, github webhook log shows
Checking the environment ...

Running as nginx.

git not available. It needs to be installed on the server for this script to work.

@mxxcon Quickest fix probably use your git full path
which git
/usr/local/bin/git
Then change every git call in deploy.php line : 197, 235, 244, 250, 258, 264
into something like : /usr/local/bin/git clone --depth=1 --branch %s %s %s

@hdytsgt  That’s a good idea. I have save problem with centos 5.11. I change every git call in deploy.php

git clone --depth=1 --branch %s %s %s
to
/usr/local/bin/git clone --depth=1 --branch %s %s %s
And important change

$requiredBinaries = array('git', 'rsync');

to
$requiredBinaries = array();

or you can delete
foreach ($requiredBinaries as $command) {
  $path = trim(shell_exec('which '.$command));
  if ($path == '') {
    die(sprintf('
%s not available. It needs to be installed on the server for this script to work.
', $command));
  } else {
    $version = explode("\n", shell_exec($command.' --version'));
    printf('%s : %s'."\n"
      , $path
      , $version[0]
    );
  }
}

I know this is an old issue, but using the solution in this ticket fixed running this script in NameCheap's shared hosting after enabling shell_exec in your php.ini using these links: https://www.namecheap.com/support/knowledgebase/article.aspx/9396/2219/how-to-enable-exec and https://www.namecheap.com/support/knowledgebase/article.aspx/9314
Just wanted to share in case anyone else runs across the issue.


BUG: Add backwards compatibility for position lookup by int.

Coverage decreased (-0.05%) to 87.654% when pulling b55d4bd4238ca89b4108783cc996e9c0cd4c95f9 on shim-portfolio-access-by-int into 3c4f6e69f367c945106e61b074633bbce0c5055b on master.


(PUP-4495) Fix tag filtering regression

Prior to this commit a change made to the way tags are split when
filtering was causing the agent to create more resources than
nessecary.
Update the way a tag is recoreded when filtering to fix this issue.

:+1:
Nit: can you amend the commit message to reference the commit that introduced the regression - https://github.com/puppetlabs/puppet/commit/0412515e2 - and also fix the spelling of 'necessary' and 'recorded'?
And btw 'necessary' is one of my least favorite words to spell ... :)

Looks good (for PUP-4495). Mentioning the SHA for the regression commit is also good. A test would be good. "Tags given in a transaction searches for the complete tag, not the individual parts"

:+1:
I'll merge once travis/appveyor go green.


Create my_change.txt

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@cbf15de). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #1   +/-   ##
=========================================
  Coverage          ?   94.07%
=========================================
  Files             ?       54
  Lines             ?     2784
  Branches          ?        0
=========================================
  Hits              ?     2619
  Misses            ?      165
  Partials          ?        0

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 cbf15de...d5f72a6. Read the comment docs.


Handled the presence & proxy case

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


Rust: update everest-framework

Clang format is complaining, but I doubt we care about this at this point in time. Merging anyways.


Machavity: Watch bowtricks.com

Machavity requests the watch of the watch_keyword bowtricks\.com. See the MS search here and the Stack Exchange search here.

bowtricks\.com has been seen in 2 true positives, 0 false positives, and 0 NAAs.

Approved by CalvT in Charcoal HQ


Feature/multiple raw query strings

Hi
there, as discussed here is the PR.
Thanks for the lib its my favourite :)

Thanks for the PR! It looks sane at first glance -- I'll try and look through it properly later (although it might take me a couple of days), and I'll probably have a few comments.
Things also need to be based off and targeted develop rather than master, but you can't do that once you've opened a PR. I'll rebase the branch before merging manually though, so that's not a problem.
The build failure isn't you -- it's because this is based off master rather than off develop (and I tweaked some CI settings, so it'll only build develop currently).

Ive pushed my changes. I wasnt sure if you ll resolve or i have to 😄

Thanks! I'll take a look when I get a minute
On Mon, 20 Apr 2020, at 8:02 AM, TraderMoe wrote:

Ive pushed my changes. I wasnt sure if you ll resolve or i have to 😄

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/canton7/RestEase/pull/153#issuecomment-616352193, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAEKWKDAUKRW5Z2PWE4GAUDRNPXR5ANCNFSM4MKR4NIQ.

Yeah go for it. its not a Problem 😅👍🏼

Merged as 2c2adcd692bf243dc4df1335832499be3de286cd (I had to rebase it, which means the PR annoyingly loses track of it. You're still down as the commit's author).
Thanks a lot for your work on this! I'll get a release out soon.


[Document] v0.1 Document tasks

[x] #212
[x] #213
[x] #214
[ ] #218

Website about documents https://github.com/rooch-network/rooch-network.github.io/issues/192

Migrate #218 to v0.2


Include allcontributors.org to acknowledge the work of everyone

Reviewing translations is a valuable work. I'd like to be able to acknowledge and enhance this type of contribution.
https://allcontributors.org/
https://github.com/all-contributors/all-contributors

Hi! AllContributors is now available as a bot :)
https://allcontributors.org/docs/en/bot/installation
That said, it appears to only add entries to a single file, so you might struggle with adding acknowledgements to multiple language files.


PTRACE_O_TRACEEXIT, PTRACE_O_TRACEEXEC

I am testing Swift on WSL.
Swift REPL uses LLVM's LLDB which calls ptrace(PTRACE_SETOPTIONS,...).
ptrace on WSL accepts PTRACE_O_TRACECLONE, but not PTRACE_O_TRACEEXIT nor PTRACE_O_TRACEEXEC.
The test code is as follows:
#include <unistd.h>
#include <stdlib.h>
#include <sys/ptrace.h>

int main()
{
    long data = PTRACE_O_TRACECLONE|PTRACE_O_TRACEEXEC|PTRACE_O_TRACEEXIT;

    pid_t child = fork();
    if (child == 0) {
        ptrace(PTRACE_TRACEME, 0, NULL, NULL);
        execl("/bin/echo", "echo", NULL);
    }
    wait(NULL);
    if (ptrace(PTRACE_SETOPTIONS, child, NULL, (void*)data) == -1)
        perror("ptrace error");
}

On Ubuntu LTS 14.04, the test code does not generate errors.
On WSL Build 14367, the test code produces an "Invalid argument" error with PTRACE_O_TRACEEXEC or PTRACE_O_TRACEEXIT suboptions.

I met another similar case with PTRACE_GETREGSET option.
#include <unistd.h>
#include <stdlib.h>
#include <sys/ptrace.h>
#include <linux/uio.h>

int main()
{
    long regs[16];
    struct iovec iov = { regs, sizeof(regs) };
    pid_t child = fork();
    if (child == 0) {
        ptrace(PTRACE_TRACEME, 0, NULL, NULL);
        execl("/bin/echo", "echo", NULL);
    }
    wait(NULL);
    if (ptrace(PTRACE_GETREGSET, child, 1, &iov) == -1)
        perror("ptrace error");
}

I met yet another case with PTRACE_TRACEME reporting EPERM on Creator's Update.  This blocks lldb from functioning.

@SvenGroot recently added support for PTRACE_O_TRACEEXIT and PTRACE_GETREGSET. Support for PTRACE_O_TRACEEXEC exists in Creators Update (build 15063). So, I am going to mark this as "fixinbound".

💯 Sven.

PTRACE_SYSEMU for the win, BTW. It is being tracked in awkwardly titled #1692.

Support for  PTRACE_O_TRACEEXIT, TRACE_GETFPREGS and PTRACE_GETREGSET has come in the Insider build 16251


Fixed hard returns

They were breaking the table formatting in docs/extensions.md.

Thanks for spotting that @raygesualdo !


bugs in "bias" mode

When we're in "bias" mode the system is doing several things wrong. Start a fresh instance of Neuron Sandbox and switch from "threshold" mode to "bias" mode. Set the bias value to 4. Click on "Show details". Hover over the last row of the table.

The first equation should now read Sigma = (Have Peanut Butter x w1) + (Have Helly x w2) + Bias
The second equation shold now read Sigma = (1 x 1) + (1 x 1) + 4 = 6
The "Activation Sigma" column values should be 4, 5, 5, and 6.
The Current Output table header should read "Current Output Sigma > 0".

Please fix this in the angela branch so I can merge it into master right away.

These are fixed.


test: Keep browser test bundle separate

Turned out that if I ran ./go test --coverage before ./go serve --test, the in-browser Mocha test page would be broken because public/tests/generated/index.js would be the incorrect version.

Coverage remained the same at 98.115% when pulling ea1150c0e051129e9a70bef1da3de3671e433044 on separate-bundles into c1eb3442226a55814d36daf9d76e76dc62f2eda9 on master.

Coverage remained the same at 98.115% when pulling ea1150c0e051129e9a70bef1da3de3671e433044 on separate-bundles into c1eb3442226a55814d36daf9d76e76dc62f2eda9 on master.


Renaming file seems to break addon.

I think I've noticed a small issue. If you change the filename of your .blend file after you've already used the addon, the addon can't seem to function anymore. Instead, I get this error:
Traceback (most recent call last):
  File "/Users/ewoods/Library/Application Support/Steam/steamapps/common/Blender/blender.app/Contents/Resources/2.79/scripts/addons/io_scene_gltf2/__init__.py", line 282, in invoke
    setattr(self, k, v)
  File "/Users/ewoods/Library/Application Support/Steam/steamapps/common/Blender/blender.app/Contents/Resources/2.79/scripts/modules/bpy_types.py", line 611, in __setattr__
    return setattr(properties, attr, value)
TypeError: bpy_struct: item.attr = val: expected a string enum, not int

location: <unknown location>:-1

For anyone else experiencing this issue: just append your stuff from the "broken" file to a new file and then run Make Local on all the things. Then you can save over the original file and it seems to work fine.
There may be other factors that contribute to the problem I saw. I will update if I can more consistently reproduce the error and learn something new.

Which file has been renamed? The .blend, or some texture in the model?

The .blend. I'll edit for clarification.

Resolved in new importer/exporter, glTF-Blender-IO.


new regex list

Description:
It contains new regex list for many tokens/secrets/keys to detect

@raadsec I recently added some contributing guidelines for adding new rules, see https://github.com/zricethezav/gitleaks/blob/master/CONTRIBUTING.md
The rules you are proposing that I'm willing to add as the default are the following:

jdbc assuming a password capture is added
odbc regex needs to be tightened up to reduce false positives
CSR
ftp assuming username/password is part of the capture. ex: ftp://user:password@host:port/path
JWT
authorization-bearer assuming a minimum entropy value is added. This rule might need some tweaking.
google-api-key
google-cloud-api
google-cloud-oauth
google-drive-oauth
google-oauth

Closing this PR due to inactivity. If someone wants to follow the contributing guidelines and add some new rules, I'd be happy to review


[SPARK-7578] [ml] [doc] User guide for spark.ml Normalizer, IDF, StandardScaler

Added user guide sections with code examples.
Also added small Java unit tests to test Java example in guide.
CC: @mengxr  I may wait until some other guides are merged to avoid other people from having to deal with conflicts.

Merged build triggered.

Merged build started.

Test build #32635 has started for   PR 6127 at commit 8a2b5cd.

Test build #32635 has finished for   PR 6127 at commit 8a2b5cd.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
public class JavaNormalizerSuite
public class JavaStandardScalerSuite

Merged build finished. Test PASSed.

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

Build triggered.

Build started.

Test build #33085 has started for   PR 6127 at commit 45245e8.

Test build #33085 has finished for   PR 6127 at commit 45245e8.

This patch passes all tests.
This patch does not merge cleanly.
This patch adds the following public classes (experimental):
public class JavaNormalizerSuite
public class JavaStandardScalerSuite

Build finished. Test PASSed.

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

@mengxr  Just rebased off of master, so this should be ready to review (with fewer PRs for the ml-features.md file now).

Merged build triggered.

Merged build started.

Test build #33274 has started for   PR 6127 at commit f16bcec.

Test build #33274 has finished for   PR 6127 at commit f16bcec.

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

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

Merged build finished. Test FAILed.

Test build #848 has started for   PR 6127 at commit f16bcec.

Test build #848 has finished for   PR 6127 at commit f16bcec.

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

@mengxr  Updated.  Thanks!

Merged build triggered.

Merged build started.

Test build #33310 has started for   PR 6127 at commit cd47f4b.

Test build #33310 has finished for   PR 6127 at commit cd47f4b.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
public class JavaNormalizerSuite
public class JavaStandardScalerSuite

Merged build finished. Test PASSed.

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

LGTM. Merged into master and branch-1.4. Thanks!


Cannot find module 'gaston-tester' from '/Users/youzi/dev/gaston/lib/browser'

Cannot find module 'gaston-tester' from '/Users/youzi/dev/gaston/lib/browser' in 3c444773c8314edc10db7586dda0dc71640f02b1

this problem has been fixed


Text in statblocks is hard to read in Light mode

In SMF 2.1, statblocks is used in Forum stats page, Memberlist, and in the polls.
In Light mode, text in statblocks is hard to read (white font on grey background).
I will attach a screnshot from my forum.

I would suggest, if it's possible, to use separate statbars, like in SMF 2.0. With this variant, changing the secondary theme color will never affect the text in stats page. This variant is also used by additional mods - SMF Gallery, Download System, and Tinyportal in their stats pages.
I will attach a screenshot from my forum, showing Download System stats page.

I would rather avoid editing the statbar structures for a different style, mods tend to add their own stats there so it could get messy.
I think a text shadow would solve the issue, making the white text stand out.
Either an outline style like this:

or a simple drop shadow:

I thin I'll go with this after all:

Fixed in https://github.com/SychO9/smf-potato-theme/commit/48774d6f4b65e8597fb4b78571b938aa820e05ed


In Multires mode, iOS's Safari browser has a problem with not rendering after jumping over multiple scenes

I used generate.py to generate the multires file and sent it to Safari browser of iOS. After jumping the page several times (using loadScence()), the renderer will not render the panorama and the page will only show the backgroundColor
I'm sure I have no problem with the path I referenced to the file, because all the scenarios show that just a few more jumps can cause problems
I didn't encounter this problem in Chorme on PC. My mobile phone is iPhone11 version 14.6

What messages are shown in the developer console?

What messages are shown in the developer console?
It doesn't show any message, which also confused me

What about network activity? Does it actually load the image? I've never seen such behavior, and I don't have a device that runs a recent version of iOS to test with.

I linked the phone to the computer and debug it with Safari. I found that there was no network requested image. I don't understand why this happened, because it was accidental
It is possible to switch the scenic spot for four times in a row, and the next scenic spot will not load the image. Could you please help me solve this problem? Thank you very much

When the image was not loaded, I switched back to other scenic spots that had been loaded and found that they did not download the image either. There is a base64 image in the network request (according to my observation, it should be the first image to be displayed after entering, and it also becomes the backgroundcolor).
But if I switch back to the Equirectangular mode of the scenic area, there will be no problem. But my image is too big, I still want to use multires mode
This problem really bothers me for a long time, thank you again for all your contribution and support to Pannellum, wish you all the best, my friend

I linked the phone to the computer and debug it with Safari

This is what I asked you to do originally. When you're remotely debugging, are there any warnings in the console?

I don't understand why this happened, because it was accidental

What was accidental?

I switched back to other scenic spots that had been loaded and found that they did not download the image either

This is to be expected, since the browser caches the images.
You've given me very little to work with. As I have no way of reproducing the issue, I'm afraid I can't be of much help.


ValueError in train.py

Hello,
I am using the control task of DSO. I recently reinstall everything and ran an ancient program that used to work with my previous installation. My goal is to find control law for simple chaotic dynamical systems (Dadras, Rössler, Lorenz etc...). I created gym environements for them, but now I get this error when launch DSO on the Dadras system (not the Lorenz):
MY COMMAND:
python -m dso.run ~/Bureau/Scripts/Python/DSO/config/DS.json
THE OUTPUT:
WARNING:tensorflow:
The TensorFlow contrib module will not be included in TensorFlow 2.0.
For more information, please see:
  * https://github.com/tensorflow/community/blob/master/rfcs/20180907-contrib-sunset.md
  * https://github.com/tensorflow/addons
  * https://github.com/tensorflow/io (for I/O related ops)
If you depend on functionality not listed there, please file an issue.

== EXPERIMENT SETUP START ===========
Task type            : control
Environment          : DS-v0
Starting seed        : 0
Runs                 : 1
== EXPERIMENT SETUP END =============

== TRAINING SEED 0 START ============
/home/remi/deep-symbolic-optimization-master/dso/dso/prior.py:757: RuntimeWarning: divide by zero encountered in log
  self.logit_adjust[tokens] -= np.log(len(tokens))
-- BUILDING PRIOR START -------------
WARNING: Skipping invalid 'RelationalConstraint' with arguments {'targets': [], 'effectors': [], 'relationship': None}. Reason: Prior disabled.
WARNING: Skipping invalid 'RepeatConstraint' with arguments {'tokens': 'const', 'min_': None, 'max_': 3}. Reason: Prior disabled.
WARNING: Skipping invalid 'InverseUnaryConstraint' with arguments {}. Reason: There are no inverse unary Token pairs in the Library.
WARNING: Skipping invalid 'TrigConstraint' with arguments {}. Reason: There are no target Tokens.
WARNING: Skipping invalid 'ConstConstraint' with arguments {}. Reason: Uses Tokens not in the Library.
WARNING: Skipping invalid 'LanguageModelPrior' with arguments {'weight': None}. Reason: Prior disabled.
LengthConstraint: Sequences have minimum length 4.
                  Sequences have maximum length 30.
NoInputsConstraint: Sequences contain at least one input variable Token.
UniformArityPrior: Activated.
SoftLengthPrior: No description available.
-- BUILDING PRIOR END ---------------

WARNING: max_length (256) will be overridden by value from LengthConstraint (30).
-- RUNNING EPOCHS START -------------
/home/remi/anaconda3/envs/DSO_env_6/lib/python3.6/site-packages/numpy/core/fromnumeric.py:3373: RuntimeWarning: Mean of empty slice.
  out=out, **kwargs)
/home/remi/anaconda3/envs/DSO_env_6/lib/python3.6/site-packages/numpy/core/_methods.py:170: RuntimeWarning: invalid value encountered in double_scalars
  ret = ret.dtype.type(ret / rcount)
WOLA []
Traceback (most recent call last):
  File "/home/remi/anaconda3/envs/DSO_env_6/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/home/remi/anaconda3/envs/DSO_env_6/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/home/remi/deep-symbolic-optimization-master/dso/dso/run.py", line 156, in <module>
    main()
  File "/home/remi/anaconda3/envs/DSO_env_6/lib/python3.6/site-packages/click/core.py", line 1128, in __call__
    return self.main(*args, **kwargs)
  File "/home/remi/anaconda3/envs/DSO_env_6/lib/python3.6/site-packages/click/core.py", line 1053, in main
    rv = self.invoke(ctx)
  File "/home/remi/anaconda3/envs/DSO_env_6/lib/python3.6/site-packages/click/core.py", line 1395, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/remi/anaconda3/envs/DSO_env_6/lib/python3.6/site-packages/click/core.py", line 754, in invoke
    return __callback(*args, **kwargs)
  File "/home/remi/deep-symbolic-optimization-master/dso/dso/run.py", line 139, in main
    result, summary_path = train_dso(config)
  File "/home/remi/deep-symbolic-optimization-master/dso/dso/run.py", line 33, in train_dso
    result = model.train()
  File "/home/remi/deep-symbolic-optimization-master/dso/dso/core.py", line 83, in train
    **self.config_training))
  File "/home/remi/deep-symbolic-optimization-master/dso/dso/train.py", line 472, in learn
    p_r_best = programs[np.argmax(r)]
  File "<__array_function__ internals>", line 6, in argmax
  File "/home/remi/anaconda3/envs/DSO_env_6/lib/python3.6/site-packages/numpy/core/fromnumeric.py", line 1188, in argmax
    return _wrapfunc(a, 'argmax', axis=axis, out=out)
  File "/home/remi/anaconda3/envs/DSO_env_6/lib/python3.6/site-packages/numpy/core/fromnumeric.py", line 58, in _wrapfunc
    return bound(*args, **kwds)
ValueError: attempt to get argmax of an empty sequence

As stated by the error message, the problem comes from train.py at line 471, where the variable r is empty, causing the error. I just started to study the code, but I don't think I will have time to understand everything. Maybe an expert could tell me if that error is normal or not.
I don't get this error when working with another dynamical system (Lorenz system for example). My gym environment is the same for Lorenz and Dadras (apart from its behavior, that is to say the equations  of the dynamical system).
Looking forward to an answer. Thanks and regards!
Rémi

Hi, if you post your config I should be able to help!

Feel free to reopen if you want to engage.

Hi Brended, I have the same issue.
Here is my code:
        # Initializing and fitting DSR model
        dso = None
        import dso
        modelreg = dso.DeepSymbolicRegressor( {   "experiment": {
            "logdir": None
                    },
                    "task" : {
                    "task_type" : "regression",
                    "metric": "inv_nmse", # "r2_shift"
                    "metric_params": (0,),
                    "function_set" : ["add", "sub", "mul", "div", "exp", "log", "sqrt","const" ] # ,
                } ,
                    "training" : {     "n_cores_batch" : -1
                        },
                        "prior": {
                        "length": {
                        "min_": 3,
                        "max_": 15,
                        "on": True
                    }}})

        X = df.iloc[:i+years_start][["x1","x2"]].values.copy()
        y = df.iloc[:i+years_start][["y"]].values[:,0].copy()
        modelreg.fit(X, y)

Should I provide anything else like the full code?
It works with a some simulated data bit fails on the real data although from a programming point of view there does not seem to be any difference.
I can also provide sample datasets if needed.

Here is a full config.json and a sample data file.
{
"experiment": {
"logdir": "./log",
"seed": 0,
"cmd": "path_to_file/dso/dso/run.py path_to_file/config.json",
"timestamp": "2023-02-21-115723",
"task_name": "Users_name",
"save_path": "./log/Users_name"
},
"task": {
"task_type": "regression",
"dataset": "path_to_data/data/DSR_real_test.csv",
"function_set": [
"add",
"sub",
"mul",
"div",
"exp",
"log",
"sqrt",
"const"
],
"metric": "r2_shift",
"metric_params": [
0
],
"extra_metric_test": null,
"extra_metric_test_params": [],
"threshold": 1e-12,
"protected": false,
"reward_noise": 0.0,
"reward_noise_type": "y_hat",
"normalize_variance": false,
"decision_tree_threshold_set": []
},
"training": {
"n_epochs": null,
"n_samples": 2000000,
"batch_size": 500,
"epsilon": 0.02,
"baseline": "R_e",
"alpha": 0.5,
"b_jumpstart": false,
"n_cores_batch": -1,
"complexity": "token",
"const_optimizer": "scipy",
"const_params": {},
"verbose": true,
"debug": 0,
"early_stopping": true,
"hof": 100,
"use_memory": false,
"memory_capacity": 1000.0,
"warm_start": null,
"memory_threshold": null,
"save_all_epoch": false,
"save_summary": false,
"save_positional_entropy": false,
"save_pareto_front": false,
"save_cache": false,
"save_cache_r_min": 0.9,
"save_freq": 1,
"save_token_count": false
},
"state_manager": {
"type": "hierarchical",
"observe_action": false,
"observe_parent": true,
"observe_sibling": true,
"observe_dangling": false,
"embedding": false,
"embedding_size": 8
},
"controller": {
"max_length": 256,
"cell": "lstm",
"num_layers": 1,
"num_units": 32,
"initializer": "zeros",
"learning_rate": 0.0025,
"optimizer": "adam",
"entropy_weight": 0.03,
"entropy_gamma": 0.7,
"pqt": true,
"pqt_k": 10,
"pqt_batch_size": 1,
"pqt_weight": 200.0,
"pqt_use_pg": false,
"summary": false
},
"gp_meld": {
"run_gp_meld": true,
"verbose": false,
"generations": 20,
"p_crossover": 0.5,
"p_mutate": 0.5,
"tournament_size": 5,
"train_n": 50,
"mutate_tree_max": 3
},
"prior": {
"count_constraints": false,
"relational": {
"targets": [],
"effectors": [],
"relationship": null,
"on": false
},
"length": {
"min": 3,
"max": 10,
"on": true
},
"repeat": {
"tokens": "const",
"min_": null,
"max_": 10,
"on": true
},
"inverse": {
"on": true
},
"trig": {
"on": true
},
"const": {
"on": true
},
"no_inputs": {
"on": true
},
"uniform_arity": {
"on": false
},
"soft_length": {
"loc": 10,
"scale": 5,
"on": true
},
"language_model": {
"weight": null,
"on": false
}
},
"postprocess": {
DSR_real_test.csv
  "show_count": 5,
  "save_plots": true

}
}


MP/QA Import not working

Issue:
When a user uploads a file to import for MP/QA and clicks Import, then a spinner is displayed and it keeps spinning without any success.

MP

The QA import does work, but the UI indicates there are some errors. This will be investigated in a different ticket. MP does not import as the UI is indicating there is an error. Will be investigated in a future ticket.


Redundant Assignments and Unread Variables

One common anti-pattern in cFS is to call functions that return
potential error codes, store the return values in variables, then
never bother to check the value of the variable.
cppcheck reports this as a redundant assignment.
Current cases in CFE:
(redundantAssignment)
{{{
cfe/fsw/cfe-core/src/evs/cfe_evs_task.c:1395: performance: Variable 'Status' is reassigned a value before the old one has been used.
cfe/fsw/cfe-core/src/evs/cfe_evs_task.c:1385: performance: Variable 'Status' is reassigned a value before the old one has been used.
}}}
(unreadVariable)
{{{
cfe/fsw/cfe-core/src/es/cfe_es_start.c:486: style: Variable 'status' is assigned a value that is never used.
cfe/fsw/cfe-core/src/es/cfe_es_api.c:469: style: Variable 'ReturnCode' is assigned a value that is never used.
cfe/fsw/cfe-core/src/es/cfe_es_api.c:1964: style: Variable 'Status' is assigned a value that is never used.
cfe/fsw/cfe-core/src/es/cfe_es_apps.c:740: style: Variable 'ReturnCode' is assigned a value that is never used.
cfe/fsw/cfe-core/src/es/cfe_es_apps.c:971: style: Variable 'ReturnCode' is assigned a value that is never used.
}}}
Not sure how we missed this in the 2016-04-12 sweep.

Imported from trac issue 152.  Created by glimes on 2016-04-18T12:20:34, last modified: 2019-04-01T16:46:19

#425 documents and #432 resolves cppcheck issues in the current code base.  Closing this as duplicate.