[main] Update actions

Cron -knative-prow-robot
/cc knative-sandbox/networking-wg-leads
/assign knative-sandbox/networking-wg-leads
Produced by: knative-sandbox/knobots/actions/update-actions
Details:

Fixing the lint issue by https://github.com/knative-sandbox/net-istio/pull/642

Closing as https://github.com/knative-sandbox/net-istio/pull/642 include this fix.


Remove redundant len check around loop

This pull request is a minor code cleanup.
From the Go specification (https://go.dev/ref/spec#For_range):

"1. For a nil slice, the number of iterations is 0."
"3. If the map is nil, the number of iterations is 0."

len returns 0 if the slice or map is nil (https://pkg.go.dev/builtin#len). Therefore, checking len(v) > 0 before a loop is unnecessary.

At the time of writing this pull request, there wasn't a lint rule that catches these issues. The closest I could find is https://staticcheck.dev/docs/checks/#S103

for other reviewers: I found viewing with whitespace changes hidden helpful.

@techknowlogick Yup!


Esc should bring focus to edit area

It would be very nice if this issue be resolved. Every time that I use ctrl+E to switch I should manually click in the editor area to set the focus back to editor.


How to select and then remove n rows from collection

Hi there,
I am newbie here. I want to select and then remove n rows (random or by created time) from collection? How to do this?
Thank you!

Hi @takaaptech, delete command from collection can be used with document _id or any Query. There is no needs to select before delete.
var col = db.GetCollection<YourClass>("collection");

col.Delete(x => x.Created < DateTime.Now);

Hi @mbdavid, Thanks for your quickly answer! In my case, i have a collection with have 10.000 rows. Each time, i need to select exactly 20 rows for processing and then delete them from collection.
For now, i am using this:
        var data_return = new List<RowClass>();
        using (LiteDatabase db = new LiteDatabase(@"db"))
        {
            // Get a collection (or create, if doesn't exist)
            var col = db.GetCollection<RowClass>("Collection");
            var data_list = col.Find(Query.All(), 0, 20);

            foreach (var row in data_list)
            {
                col.Delete(row.Id);
                data_return.Add(row);
            }
            db.Dispose();
        }
        return data_return;

Is this ok or we can using something better?

I recommend to you transform your data_list in an List<T> before loop.

I recommend to you transform your data_list in an List<T> before loop.

@mbdavid , could you please explain the reason behind it?


Form values set to undefined

For some fields that (may) always contain default values as defined using the constructor, these fields get set to undefined while other fields of the form are being changed.
I was able to reproduce the bug both in Chrome and Firefox for 1.0.0-4, using https://github.com/sindresorhus/notifier-for-github/pull/189 as test.

Chrome
Firefox

Thanks! I'll look into it

Found the issue

It seems to be all right now in 1.0.0-8
npm i webext-options-sync@next


Create Button.html

Implemented button
Close #4

Components/Light Mode/textarea.html

Copy the Button.html inside this folder.

button moved to Components/Dark Mode/

Also, rename the main HTML file with index.html.

I had also implemented textarea and u had merged it but its not visible in
your master branch.
On Sun, 3 Oct 2021 at 14:42, Sobhan Bera @.***> wrote:

Also, rename the main HTML file with index.html.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/sobhanbera/web-templates/pull/12#issuecomment-932894536,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQU6KUY4UVVSZRIKVNEIO5TUFANATANCNFSM5FGV4MPA
.

Resolve #8

I had also implemented textarea and u had merged it but its not visible in your master branch.
…
On Sun, 3 Oct 2021 at 14:42, Sobhan Bera @.***> wrote: Also, rename the main HTML file with index.html. — You are receiving this because you were assigned. Reply to this email directly, view it on GitHub <#12 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQU6KUY4UVVSZRIKVNEIO5TUFANATANCNFSM5FGV4MPA .

That was previously on the wrong path so I have to revert the changes. But now It seems good. I will merge this now.

Thanks


add missing meta files

When installing this package via github (ie not from disk) it would throw errors about these missing files.

Thanks for noticing and fixing these issues.

No problem, thanks for making the tool in the first place!


Attack Group and Default Alliance Group do not change alien behaviour

Same with other object classes. Those settings do not make civilians friendlier.

Here's the mod file (changing the behaviour tree will crash the game):
test.zip

You loaded "Alien" as Object. So the civilians are not affected by those changes.


(C) \ not respected

Describe the issue

Which language seems to have the issue?

C
Are you using highlight or highlightAuto?
highlight
Sample Code to Reproduce

https://jsfiddle.net/gm16s8nf/
// this is a comment, but since I added a slash, \
this is part of it too

Expected behavior

In C, a '\' character extends the current lined (effectively negating the following \n )
Since the second line is also part of the comment, it should be highlighted as such.
Additional context

I would presume this also applies to C++?

Now we have the added problem that MANY languages use the common C_LINE_COMMENT_MODE rule... but I'm not sure how many of them support this \ mechanic... although it does seem like the kind of that that might not hurt anything if it's added to a few more languages than necessary...

@QH4OS Is this common though?  I wonder why someone wouldn't use a block comment in that case...

@klmr Probably an easy one if you're still wanting to help.

It’s more than that: the sequence \<newline is replaced by the empty string by the C preprocessor even inside individual tokens.
The following program is valid C and prints “a = "1"” when executed:
#include <stdio.h>

int main () {
  /\
* this is a comment *\
/
  /\
/ so is this \
and this
  int a = 0;
  a+\
+;
  prin\
tf("a = \"%d\\
"\\
n", a);
}

HAHAHAHA.

Handling single-line comments only should be sufficient.

Wholly agree - lets fix the reported problem. :-)  The rest is outside scope.  Do you have any thoughts on negative impact that might have on other languages using the C rule but NOT fully complying?  I'm guessing it would be minimal (or zero impact) and therefore ok to add this to our C_ modes (esp. since that is their name).

Hmmm. C_LINE_COMMENT_MODE is used in 65 language definitions. As far as I know (might be wrong), no languages other than C and C++ actually support line continuations though. And while rare I don’t think it’s unthinkable that somebody is using \ at the end of their line, e.g. as an ornamental border. I think it would be best if we kept C_LINE_COMMENT_MODE unchanged and used something different for c-like. The name is a bit odd then, but not really; these are C-style line comments, after all.


Express server example

Is there some boilerplate code on how to set this up with an Express server?
I see here documented the basic usage https://slash-create.js.org/#/docs/main/latest/examples/webserver, but is there a repo somewhere with a full example for this, just like there is for Vercel?

You can use https://github.com/Snazzah/slash-create-template (npx slash-up init) and just replace fastify with express.


Setup of 40.0 via pip on MacOS High Sierra is not successful

I'm trying to install using pip but Mac OS X, 10.13.4 is already shipped with an old version
pip --version
pip 10.0.1 from /Library/Python/2.7/site-packages/pip (python 2.7)

MacBook-Pro:KerasDemo user$ sudo pip install setuptools --upgrade
Password:
The directory '/Users/user/Library/Caches/pip/http' or its parent directory is not owned by the current user and the cache has been disabled. Please check the permissions and owner of that directory. If executing pip with sudo, you may want sudo's -H flag.
The directory '/Users/user/Library/Caches/pip' or its parent directory is not owned by the current user and caching wheels has been disabled. check the permissions and owner of that directory. If executing pip with sudo, you may want sudo's -H flag.
Collecting setuptools
  Downloading https://files.pythonhosted.org/packages/ff/f4/385715ccc461885f3cedf57a41ae3c12b5fec3f35cce4c8706b1a112a133/setuptools-40.0.0-py2.py3-none-any.whl (567kB)
    100% |████████████████████████████████| 573kB 3.0MB/s
Installing collected packages: setuptools
  Found existing installation: setuptools 18.5
    Uninstalling setuptools-18.5:
Could not install packages due to an EnvironmentError: [('/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/_markerlib/markers.pyc', '/private/tmp/pip-uninstall-2PS_oP/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/_markerlib/markers.pyc', "[Errno 1] Operation not permitted: '/private/tmp/pip-uninstall-2PS_oP/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/_markerlib/markers.pyc'"), ('/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/_markerlib/__init__.py', '/private/tmp/pip-uninstall-2PS_oP/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/_markerlib/__init__.py', "[Errno 1] Operation not permitted: '/private/tmp/pip-uninstall-2PS_oP/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/_markerlib/__init__.py'"), ('/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/_markerlib/markers.py', '/private/tmp/pip-uninstall-2PS_oP/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/_markerlib/markers.py', "[Errno 1] Operation not permitted: '/private/tmp/pip-uninstall-2PS_oP/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/_markerlib/markers.py'"), ('/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/_markerlib/__init__.pyc', '/private/tmp/pip-uninstall-2PS_oP/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/_markerlib/__init__.pyc', "[Errno 1] Operation not permitted: '/private/tmp/pip-uninstall-2PS_oP/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/_markerlib/__init__.pyc'"), ('/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/_markerlib', '/private/tmp/pip-uninstall-2PS_oP/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/_markerlib', "[Errno 1] Operation not permitted: '/private/tmp/pip-uninstall-2PS_oP/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/_markerlib'")]

OK, let's try a user install:
MacBook-Pro:KerasDemo user$ sudo pip install setuptools --upgrade --user
The directory '/Users/user/Library/Caches/pip/http' or its parent directory is not owned by the current user and the cache has been disabled. Please check the permissions and owner of that directory. If executing pip with sudo, you may want sudo's -H flag.
The directory '/Users/user/Library/Caches/pip' or its parent directory is not owned by the current user and caching wheels has been disabled. check the permissions and owner of that directory. If executing pip with sudo, you may want sudo's -H flag.
Collecting setuptools
  Downloading https://files.pythonhosted.org/packages/ff/f4/385715ccc461885f3cedf57a41ae3c12b5fec3f35cce4c8706b1a112a133/setuptools-40.0.0-py2.py3-none-any.whl (567kB)
    100% |████████████████████████████████| 573kB 3.4MB/s
Installing collected packages: setuptools
Successfully installed setuptools-40.0.0

But then:
MacBook-Pro:KerasDemo user$ which easy_install
/Users/user/Library/Python/2.7/bin/easy_install
MacBook-Pro:KerasDemo user$ easy_install
Traceback (most recent call last):
  File "/usr/bin/easy_install-2.7", line 11, in <module>
    load_entry_point('setuptools==18.5', 'console_scripts', 'easy_install')()
  File "/Users/user/Library/Python/2.7/lib/python/site-packages/pkg_resources/__init__.py", line 479, in load_entry_point
    return get_distribution(dist).load_entry_point(group, name)
  File "/Users/user/Library/Python/2.7/lib/python/site-packages/pkg_resources/__init__.py", line 2703, in load_entry_point
    return ep.load()
  File "/Users/user/Library/Python/2.7/lib/python/site-packages/pkg_resources/__init__.py", line 2321, in load
    return self.resolve()
  File "/Users/user/Library/Python/2.7/lib/python/site-packages/pkg_resources/__init__.py", line 2327, in resolve
    module = __import__(self.module_name, fromlist=['__name__'], level=0)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/setuptools/command/easy_install.py", line 46, in <module>
    from setuptools.archive_util import unpack_archive
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/setuptools/archive_util.py", line 15, in <module>
    from pkg_resources import ensure_directory, ContextualZipFile
ImportError: cannot import name ContextualZipFile

Or maybe just try to ignore and install it anyway:
MacBook-Pro:KerasDemo user$ sudo pip install setuptools --ignore-installed
The directory '/Users/user/Library/Caches/pip/http' or its parent directory is not owned by the current user and the cache has been disabled. Please check the permissions and owner of that directory. If executing pip with sudo, you may want sudo's -H flag.
The directory '/Users/user/Library/Caches/pip' or its parent directory is not owned by the current user and caching wheels has been disabled. check the permissions and owner of that directory. If executing pip with sudo, you may want sudo's -H flag.
Collecting setuptools
  Downloading https://files.pythonhosted.org/packages/ff/f4/385715ccc461885f3cedf57a41ae3c12b5fec3f35cce4c8706b1a112a133/setuptools-40.0.0-py2.py3-none-any.whl (567kB)
    100% |████████████████████████████████| 573kB 3.1MB/s
Installing collected packages: setuptools
Could not install packages due to an EnvironmentError: [Errno 2] No such file or directory: '/usr/local/bin/easy_install'

The bottom line is that I can't seem to install setuptools 40.0 so it has a valid and working binary in /usr/local/bin/easy_install
Any help would be appreciated.

@adam-ah Did you ever get this resolved? I suspect that this is an issue with your specific setup, because I haven't seen other HS users having trouble installing setuptools.
I don't have access to a mac laptop, but I suspect that anything involving sudo pip install is a bad idea. My understanding is that the system-shipped version of Python is in a bit of a sorry state and you're best off never using system python and instead using homebrew's python and/or pyenv. I suspect that if you do that it will fix your permissions issues.

I do run on a mac. I've had High Sierra and now have Mojave... but I avoid and strongly recommend you avoid using the Python provided by the system. It's almost always old often has its own peculiarities. I don't even have pip installed in the system Python:
$ /usr/bin/python2.7 -m pip -V
/System/Library/Frameworks/Python.framework/Versions/2.7/Resources/Python.app/Contents/MacOS/Python: No module named pip

I do see that I have setuptools installed, though I suspect that was installed by macOS and not by me:
$ /usr/bin/python2.7 -m easy_install --version
setuptools 18.5 from /System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python (Python 2.7)

I also strongly second pganssle's recommendation to use homebrew or pyenv. I personally use Homebrew, even for when I need Python 2.7. With Homebrew's python@2 installed, and with /usr/local/bin ahead on the path, my default Python 2.7 is the one from homebrew:
~ $ python2.7 -V
Python 2.7.15
~ $ python2.7 -c "import sys; print(sys.executable)"
<EMAIL_ADDRESS>
And with homebrew properly configured not to require sudo, setuptools installs and upgrades nicely without sudo:
~ $ python2.7 -m pip install -U setuptools
Collecting setuptools
  Using cached https://files.pythonhosted.org/packages/96/06/c8ee69628191285ddddffb277bd5abdf769166e7a14b867c2a172f0175b1/setuptools-40.4.3-py2.py3-none-any.whl
Installing collected packages: setuptools
  Found existing installation: setuptools 39.2.0
    Uninstalling setuptools-39.2.0:
      Successfully uninstalled setuptools-39.2.0
Successfully installed setuptools-40.4.3
You are using pip version 18.0, however version 18.1 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.

Also, I want to recommend that even if you have to use sudo for some operations, you should avoid using sudo with --user (for any application), as that creates files in ~/.local (or similar) that are owned by root and often not accessible by the user.
I hope that helps. Let us know what you find.

If anyone faces this problem, all my python issues went away once I installed the official version from python.org. The Mac's own has it's own permission system so it's just too hard to use.


Fixed crash happening due to schema does not exist during GRANT operation

Previously, If end user tries to grant select on table using schema qualified name and if schema does not exist then server will crash due to segmentation fault issue. This commit aims to fix that issue by appropriately checking existence of the jira.
Task: BABEL-5078
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is under the terms of the Apache 2.0 and PostgreSQL licenses, and grant any person obtaining a copy of the contribution permission to relicense all or a portion of my contribution to the PostgreSQL License solely to contribute all or a portion of my contribution to the PostgreSQL open source project.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 73.023%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
41897

Relevant Lines:
57375

💛  - Coveralls


Unify terminology in Create Release Plan view

We should use the same terminology across the UI. We should change the link wording from "SDK Intake Questionnarie" to "Onboard APEX product section":

Updated to:


Not touch friendly on tablets nor phones when into fancybox

Everything seemed to be perfect as soon as I was getting to the end of the development of this site http://www.destizons.com but unfortunately when I checked the twenty twenty on my Ipad I realized that I cant use it normally as I do on my computer. I 'am not able to drag the images. You can check it out on this site http://www.destizons.com/nos-services-plus/ by clicking on "Relooking": If anybody knows what I've done wrongly or can give me a solution I would be thankful.

Hi, same here, if you found any solution then please let me know.


Placeholder: Ministry- View Business Intelligence (BI) Reporting (NARS) (EDW)

Business Context:
Needs Assessment Results (NARs) Report is a Federal Request annual report (Sent Nov/ Dec)   that we send to the federal government as a part of our admin services for running their program. It includes all inputs and all outputs (assessment, award, withdrawal and a number of others) for every single application that comes in within a program year. Changes to the data and formatting of this report is (I think) included within the scope of Program Year changes, and comes in periodically (not necessarily every year). This is the foundational report from which I believe a number of other business reports are generated, including the reports that go to institutions.

Duplicate of ticket 2162


[jackson] Add jackson shade

jackson shade package name prefix is org/apache/paimon/shade/jackson2/com/fasterxml/jackson

Please resolve conflicts~


Remove the unused PDFDocument.mainXRefEntriesOffset method

Not only is this method completely unused now, looking through the history of the code it never appears to have been used for anything either.
Years ago mainXRefEntriesOffset was included when creating XRef instances, however it wasn't actually used for anything (the parameter was never checked, nor assigned to a property on XRef).
If this method ever becomes useful (again) it's easy enough to restore it thanks to version control, but including dead code in the builds just seems wasteful.

Thank you!


runtimeChunk is not inlined

Hey, you're using the InlineChunkHtmlPlugin  plugin to inline in the html the runtime generated by webpack, precisely here.
https://github.com/facebook/create-react-app/blob/7e25df150de3c781430979effffa41cec0ba929f/packages/react-scripts/config/webpack.config.js#L547-L551
However this is not working, doing a clean install and build, the runtime chunk is in the files
Is this a bug report?
Yes
Environment
Environment:
OS: macOS 10.14
Node: 10.11.0
Yarn: 1.10.1
npm: 6.4.1
Watchman: Not Found
Packages: (wanted => installed)
react: ^16.6.3 => 16.6.3
react-dom: ^16.6.3 => 16.6.3
react-scripts: 2.1.1 => 2.1.1
Steps to Reproduce

npx create-react-app test
cd test
yarn build

Expected Behavior
There should be no runtime chunk
Actual Behavior

It is actually inlined, if you take a look at the content of build/index.html.

I assume we can see the runtime chunk in the build output because webpack needs to generate this chunk anyway.
The only issue I can see here is the runtime chunk still being committed and written to build folder.
Is it an issue for, @marcofugaro ?

Oh so it's duplicated and it's not actually using the runtime~... file? I think that is misleading

Well. I'd expect InlineChunkHtmlPlugin to remove runtime chunk from being created in the build folder, since it's inlined.
Another thing, that it's totally fine to see the runtime chunk in the build command's console output, because webpack actually creates this chunk, regardless of it's usage.

This file was purposely put in the build/ folder for people who integrate with an existing app (or use CSP) and do not use our index.html file.


Add GitHub script to capture PR info

Capture info and dump to compare with existing method of generation
Update tests to create new mock per test for more reliable assertions on calls to mocks (i.e. avoid interference across tests)

Part of #1538
Future PR(s) will update the pipeline to start using the outputs from the GitHub script and remove some of the bash steps in the workflow.

/test-force-approve
(changes not covered by /test)


Add link to announcement blog post

Added link to here: https://blog.github.com/2018-10-17-action-demos/

Thank you!


Fixes Feature to not require a "processor" argument like Dependent and Datasource

Looks good and tests pass.


Get rid of notes section in pir activated cam 2 readme

I was taking notes, but forgot to delete it later 🤦♂


Invalid CLA for sign command 1 and 2 in APDU docs

Hey, I think there might be an invalid CLA in sign command 1 and 2 in APDU docs. There should be  0x5A instead of 0x80.

@yogh333?

@franciszekjob This has been fixed in the last release ;)


Would a Promise-base example be useful?

The demo is all well and good, but for an application, something based on Promises seems more suitable. I offer this bit of code that is working for me:
function findFileChunks (infile, average) {
  const fd = fs.openSync(infile, 'r')
  const minimum = Math.round(average / 2)
  const maximum = average * 2
  const source = Buffer.alloc(maximum * 2)
  const target = Buffer.alloc(dedupe.targetSize(minimum, source.length))

  return new Promise((resolve, reject) => {
    let flags = 0
    const close = (error) => {
      fs.closeSync(fd)
      if (error) {
        // force the loop to exit
        flags = 1
        reject(error)
      }
    }

    let chunks = []
    let fileOffset = 0
    let chunkOffset = 0
    let sourceStart = 0

    while (flags === 0) {
      const length = source.length - sourceStart
      const bytesRead = fs.readSync(fd, source, sourceStart, length, fileOffset)
      fileOffset += bytesRead
      flags = (bytesRead < length) ? 1 : 0
      const sourceSize = sourceStart + bytesRead
      try {
        dedupe.deduplicate(average, minimum, maximum, source, 0, sourceSize, target, 0, flags,
          (error, sourceOffset, targetOffset) => {
            // n.b. the library throws the error, so this is always undefined
            if (error) {
              close(error)
              return
            }
            let offset = 0
            while (offset < targetOffset) {
              const hash = target.slice(offset, offset + 32)
              offset += 32
              const size = target.readUInt32BE(offset)
              offset += 4
              chunks.push({ hash, offset: chunkOffset, size })
              chunkOffset += size
            }
            // Anything remaining in the source buffer should be moved to the
            // beginning of the source buffer, and become the sourceStart for the
            // next read so that we do not read data we have already read:
            sourceStart = sourceSize - sourceOffset
            if (sourceStart > 0) {
              source.copy(source, 0, sourceOffset, sourceOffset + sourceStart)
            }
            if (flags !== 0) {
              // the last block has finished processing
              close()
              resolve(chunks)
            }
          }
        )
      } catch (err) {
        close(err)
      }
    }
  })
}

For demo purposes, this would need a .then() and some console logging to match the current demo. Let me know if this is useful and I can create a proper pull request. Or not, and feel free to close this.

Thanks for the suggestion @nlfiedler but I would prefer the module to stick with classic vanilla Javascript.
I've never been a fan of Promises. I don't find the style as clean as pure callback error handling. Promises also add runtime overhead.


FEATURE(planner): new Matchmaker and Core

@perezp can you review this PR?
Probably also @mbarrientos can have a look on this.

I close this to update with the rest service


Changed input style for add_polygon!

Easier handling for add_polygon! function, now consistent with input style of add_box! function,
local test worked but GMT integration test failed, which is at the moment also the case for the normal version

@TatjanaWeiler could you trigger the CI again?


Always include Pod labels in FlowAggregator IPFIX template

We always include the Pod labels IEs (for source and destination Pods), regardless of the value of the recordContents.podLabels configuration parameter. This simplifies the logic and the IPFIXExporter no longer needs to be aware of this configuration. There will be a minor size increase to the IPFIX records exported by the FlowAggregator when recordContents.podLabels is false, as we will need to include empty strings in the records for the 2 IEs.
We use an empty string when recordContents.podLabels is false, or when the endpoint is not a Pod. We use an empty JSON dictionary ("{}"), when the Pod has no labels.
Fixes #6386

/test-all

/test-all


can't find folder ~/.config/amethyst/Metadata Cache/ on first start

Describe the bug
on first start after installation from yay -S amethyst-player got this msg
A JavaScript error occurred in the main process
Uncaught Exception:
Error: ENOENT: no such file or directory, stat '/home/XXXXXXXXXXX/.config/amethyst/Metadata Cache'
    at statSync (node:fs:1551:3)
    at e.statSync (node:electron/js2c/asar_bundle:5:4539)
    at Object.<anonymous> (/opt/amethyst/resources/app.asar/release/dist/main/mainWindow.js:42:14)
    at Module._compile (node:internal/modules/cjs/loader:1120:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1175:10)
    at Module.load (node:internal/modules/cjs/loader:988:32)
    at Module._load (node:internal/modules/cjs/loader:829:12)
    at c._load (node:electron/js2c/asar_bundle:5:13343)
    at Module.require (node:internal/modules/cjs/loader:1012:19)
    at require (node:internal/modules/cjs/helpers:102:18)

Amethyst Details (required):

Version [e.g. v1.8.22]

To Reproduce (optional)

os: Manjaro KDE linux
Steps to reproduce the behavior:
start from KDE App menu - got nothing
start ftom terminal /opt/amethyst/amethyst - got error

fixing by manually create dir ~/.config/amethyst/Metadata Cache/

i do believe this issue has been fixed but the version on yay is outdated before the fix was implemented


Test Issue #2

Testing Smart Sheet auto population and ISS_NUM increment

This issue solved the auto population to Smart Sheets for belinda's closet nextjs


refactor: Tokens state

[x] Add partial token info to the state
[x] Update the actions on UI
[ ] Apply the denomination on frontend fee calcs

Later the Portfolio manager will change it, and the amounts will carry all info required, for now adding extra data to state
Relates to YOMO-902

:warning: This PR will surely conflict with https://github.com/Emurgo/yoroi/pull/2826


Option to use Ollama with local models instead of OpenAI/ChatGPT

Hi team,
I was wondering if it would be possible to have an option to use Ollama with local models instead of OpenAI/ChatGPT with the OpenDevin project.
I was thinking it could provide some benefits, such as:

Cost savings: Ollama is an open-source platform, which could significantly reduce the cost compared to OpenAI/ChatGPT.

Flexibility & Security: Ollama allows the use of different models ( and can create your own ), which could be more suitable for certain scenarios where data privacy or security concerns might arise.

Also beneficial for those that prefer to keep their data on-premises/locally.

Just thought I would put it out there/ask :)  Thank you for your time and consideration.
-Jeffery

Just saw this, https://github.com/OpenDevin/OpenDevin/issues/141, after posting mine...

As an intermediate solution: openrouter.ai has an OpenAI-like API, many tools (LangChain, LiteLLM, etc) have integrations, and while it doesn't support local models, it does support many many models, some from Huggingface too. So I think that is a relatively easy integration. On the other hand, by suddenly supporting many models raises the question: should different agents have the option to use different models? E.g. planning by chatgpt 3.5, but implementing by Claude? (or adding code by expensive model, extending documentation by a cheap one.)

See here for how to select a different model: https://github.com/OpenDevin/OpenDevin?tab=readme-ov-file#picking-a-model
Thanks all!

This is a template. Run cp config.toml.template config.toml to use it.
LLM_API_KEY="ollama"
WORKSPACE_DIR="./workspace"
LLM_BASE_URL="http://<IP_ADDRESS>:11434"
LLM_MODEL= "openchat:7b-v3.5-1210-q5_K_M"
LLM_EMBEDDING_MODEL="ollama_chat/nomic-embed-text:latest"
LLM_EMBEDDING_MODEL= "nomic-embed-text:latest"

@hqnicolas is there an error message you're seeing?
Here's your problem: LLM_API_KEY="ollama"
That's supposed to be an API key, not a model name. You probably just want to remove it.

@rbren ollama is an open source project, you dont need API Key,
That API Key on ollama project is a dummy value by standart


sidebar won't show

New install of version 2024.7.1.  Sidebar doesn't show in Chrome or Safari.
Home Assistant info below:
Core 2024.9.0
Supervisor 2024.08.0
Operating System 13.1
Frontend 20240904.0

Maybe an obvious question but just in case
Have you added a sidebar?


[C++][Parquet] Add data rates to benchmark output

Instead of only showing ns, we could also inform GBenchmark about the amount of processed bytes. Then we would get MB/s throughput rates.
Reporter: Uwe Korn / @xhochy
Assignee: Uwe Korn](https://issues.apache.org/jira/secure/ViewProfile.jspa?name=uwe) / @xhochy
Note: This issue was originally created as PARQUET-597. Please see the migration documentation for further details.

Uwe Korn / @xhochy:
PR: https://github.com/apache/parquet-cpp/pull/94

Wes McKinney / @wesm:
Issue resolved by pull request 94
https://github.com/apache/parquet-cpp/pull/94


[10.9] netifaces.ifaddresses() on AF_NET6 returns CIDR with v6 netmask

Hi,
Can someone please confirm if there is a regression between python-netifaces-0.10.4 and python3-netifaces-0.10.9 where
netifaces.ifaddresses() on AF_NET6 returns the v6 netmask entry with a the CIDR.  Is this by design, and if so why is it
not consistent with AF_NET for the v4 netmasks?
Changed behavior seen for AF_INET6  in python3-netifaces-0.10.9
Python 3.6.8 (default, Dec  5 2019, 15:45:45)
[GCC 8.3.1 20191121 (Red Hat 8.3.1-5)] on linux
Type "help", "copyright", "credits" or "license" for more information.

import netifaces
netifaces.ifaddresses('br_mgmt')[netifaces.AF_INET6][0].get('netmask')
'ffff:ffff:ffff:ffff::/64'
netifaces.ifaddresses('br_mgmt')[netifaces.AF_INET][0].get('netmask')
'<IP_ADDRESS>'

Behavior seen for AF_INET6  in python-netifaces-0.10.4
Python 2.7.5 (default, Feb 26 2020, 04:21:27)
[GCC 4.8.5 20150623 (Red Hat 4.8.5-36)] on linux2
Type "help", "copyright", "credits" or "license" for more information.

import netifaces
netifaces.ifaddresses('br_mgmt')[netifaces.AF_INET6][0].get('netmask')
'ffff:ffff:ffff:ffff::'
netifaces.ifaddresses('br_mgmt')[netifaces.AF_INET][0].get('netmask')
'<IP_ADDRESS>'

Would appreciate if someone can please confirm if this is expected and the inconsistency between v4 and v6 in the netifaces 10.9 version.
Thanks,
Sharmin

Please ignore this, it's a duplicate of https://github.com/al45tair/netifaces/issues/65 - github error'd out with 404 but opened the issue anyways

Duplicate of https://github.com/al45tair/netifaces/issues/65


Some CSS is not loaded when direct-linked to a built page

🐛 Bug Report
Prerequisites

[X] I'm using the latest version of Docusaurus.
[X] I have tried the npm run clear or yarn clear command.
[X] I have tried rm -rf node_modules yarn.lock package-lock.json and re-installing packages.
[x] I have tried creating a repro with https://new.docusaurus.io - the bug presents itself only when building the site, not when running docusarus start, but I did create a minimal repro repo, linked below
[X] I have read the console error message carefully (if applicable)

Description
Hey Docusaurus team, I'm running into an odd issue starting in 2.0.0-beta.5 (still present on 2.0.0-beta.6) where some CSS that I generate in JSX components is not rendered properly when you link directly to a built docs page, but it does show up fine when running in docusaurus start or if you click out of the page with messed up CSS, and then back into it.
Here's an illustration: if you open this page directly you'll see some cards with no CSS applied to them https://happy-curie-b91cab.netlify.app/docs/changelog , but if you click into another docs page and then back to the "Illustrate Issue" page on the sidebar, the CSS renders fine.
Have you read the [Contributing Guidelines on issues]
Yes
Steps to reproduce

Initialize a new docusaurus site
yarn add @material-ui/core
Add a few components that utilize makeStyles() and a couple of pages that use those components - https://github.com/taylorreece/docusaurus-css-issue/commit/e258b2f8745bf909dcd6ab098ffdfdca0e0ee3d8 illustrates a diff between a cleanly initialized Docusaurus site using 2.0.0-beta.6, and one with a couple components and docs pages
Run yarn build and yarn serve; note that some pages with JSX components render fine, but if you refresh the page, or link to the built page directly, the CSS doesn't render right.

Expected behavior
Correctly rendered CSS on page load when linked directly to a docs page.
Actual behavior
If you link directly to https://happy-curie-b91cab.netlify.app/docs/changelog you don't get properly rendered CSS.

If you open another docs page and then click to the changelog page from the sidebar, CSS renders properly.

Note: if I bump Docusaurus back down to 2.0.0-beta.4, the problem goes away. It seems like it was introduced between 2.0.0-beta.4 and 2.0.0-beta.5.
Your environment

Public source code: https://github.com/taylorreece/docusaurus-css-issue
Public site URL: https://happy-curie-b91cab.netlify.app/docs/changelog
Docusaurus version used: 2.0.0-beta.6
Environment name and version (e.g. Chrome 78.0.3904.108, Node.js 10.17.0): Chrome, Firefox, Mobile Chrome
Operating system and version (e.g. Ubuntu 20.04.2 LTS): MacOS and Android

Reproducible demo

Use https://new.docusaurus.io to create a CodeSandbox reproducible demo of the bug.

Unfortuantely this bug only appears if you build your project, and doesn't occur in the sandbox.
Example repo to illustrate the issue is at https://github.com/taylorreece/docusaurus-css-issue, though, and is split into two commits: the first commit is a clean initialization of a Docusaurus 2.0.0-beta.6 site using npx @docusaurus/init@latest init docusaurus-css-issue classic, and the second commit highlights the issue I'm seeing.
Thanks for your insight!

We don't support CSS-in-JS properly yet, as Docusaurus does have not lifecycle hooks to inline critical CSS-in-JS in the statically generated HTML files: https://github.com/facebook/docusaurus/issues/3236
MaterialUI is using JSS, a CSS-in-JS framework.
In any case, I strongly recommend not using it until we have better support, because it would only lead to flash of unstyled content before hydration, and the framework may not handle the React hydration well due to the lack of SSR integration that they probably recommend.
I find it very suspicious that it worked better on older Docusaurus versions, can you deploy one on beta.4 with the exact same Mui version and double-check? We didn't change anything related, and I suspect you just not noticed that it didn't work before, or you updated Mui at the same time?

Hey @slorber - thanks for your feedback! I'll look into not using CSS-in-JS.
It does seem to work fine in beta.4. I first noticed the issue yesterday after a beta.5 update, and it seems to also have issues in beta.6. Here's a commit that reverts my POC repo back to beta.4 - https://github.com/taylorreece/docusaurus-css-issue/commit/8a4f61a56a441a0093451aa4ba98e105bede0b88 - and the corresponding Netlify deploy - https://happy-curie-b91cab.netlify.app/docs/changelog/ . The page has a flash of unstyled content, but then pulls in the correct CSS with beta.4.
What's weird to me is that the CSS doesn't just not load. With beta.5 and beyond, the wrong CSS class is applied. After direct-linking to https://613103da4ac20ffdfd0d68c8--happy-curie-b91cab.netlify.app/docs/changelog/ the gray part of my cards have generated class jss13:

If I move to a different page and back they have the correct class jss16:

Very odd.
Anyways, I'll keep an eye on #3236. Should I close this one as a duplicate of that?

We recently did a change that may produce this behavior: https://github.com/facebook/docusaurus/pull/5349
Basically, Docusaurus used to render uselessly twice on the client, leading to more resource usage on React hydration. I suspect on the 2nd render, JSS was able to "fix" the lack of SSR support in Docusaurus but is not able anymore since we optimized re-renders, so the fix only applies once you navigate.
Yes, I think we can't do anything here apart adding good CSS-in-JS support in Docusaurus, so let's track the other issue.
A possible workaround if you want to keep using Mui with bad integration/FOUC is to trigger this duplicate re-render yourself (as it was before).
Here's an example, no guarantee it works though:
export const ChangeLogEntry = ({ children, date }) => {
+  const isBrowser = useIsBrowser();
  const classes = useStyles();
  const formattedDate = dateFormat(
    new Date(date),
    "mmmm dd, yyyy",
    true // Account for timezones
  ).toUpperCase();

  return (
    <>
+      <Card key={isBrowser} className={classes.root}>

          {formattedDate}

        <CardContent className={classes.cardContent}>{children}</CardContent>
      </Card>
    </>
  );
};


porter mixin update command

A command option to update the mixins available with the current envrionment.
E.g , the below list not up to date with the ones available in the official doc : https://porter.sh/mixins/
porter mixin list
Name         Version          Author
aws          v0.2.0-beta.1    DeisLabs
az           v0.4.0-beta.1    DeisLabs
azure        v0.7.1-beta.1    DeisLabs
exec         v0.22.0-beta.1   DeisLabs
gcloud       v0.3.0-beta.1    DeisLabs
helm         v0.10.0-beta.1   DeisLabs
kubernetes   v0.22.0-beta.1   DeisLabs
terraform    v0.5.0-beta.1    DeisLabs
It would be great to add a command to porter mixin to update / upgrade all mixins or individual basis.
eg.
- porter mixin update --all (update all installed mixins to latest)
- porter mixin update -name az --version (to a specific version)
- porter mixin update -name az -(default to latest version)

This is great! I've added it to the backlog. 👍

One more thing it should be able to do it take into account the source from where the mixin was installed, so that it can update all mixins, not just official ones.

Good idea!  I like all of the proposed commands.  I'm thinking we should build to support --version latest and --version canary in addition to the actual semver options, e.g. --version v0.5.0-beta.1.

Oh one more thing I just noticed is that since the other mixin commands, like porter mixin install [NAME] doesn't have a -name flag, we shouldn't have it for this one either. So it sounds like the final syntax would be this:
porter mixin update NAME [--version|-v VERSION]
porter mixin update --all

--version defaults to latest, and also understands a semver and canary
See https://github.com/deislabs/porter/issues/30#issuecomment-531493471 for the original issue and hints on how to implement

Oops, since there was already an issue for porter mixin update, I'm going to merge your issue with that existing one and add your suggestion for --all to that one so we have everything in one place.
Thank you for the good idea and letting us know that you'd use the feature!
Duplicate of #30


Feature/population reports display%

Some changes made to reflect the percentages in some population reports.

Conflicts Resolved


Bug: website elements above fullscreen video

Is there an existing issue for this?

[X] I have searched the existing issues

Which Mux Elements/Packages does this apply to? Select all that apply

mux-video
mux-video-react

Which browsers are you using?
Safari on iOS
Which operating systems are you using?
iOS
Description
My colleague just showed me this screenshot from Safari on his iPad, sadly I can't reproduce it myself on my iPad, or any other device, he also said that on other devices it works as expected.
So the website elements, Logo, the whole navigation at top, the five circular avatars almost in the center are not supposed to be above the video. How can I ensure this doesn't happen? The

The video is on this page https://www.konduktum.com/k/nIhvyp72xozgu
Reduced test case
No response
Steps to reproduce
As mentioned I can't reproduce, though it just happens when entering fullscreen on his iPad's Safari, he's got the M1 Pro on IpadOS 16.2
Current Behavior
website elements, top navigation and avatars are showing above the video
Expected Behavior
fullscreen video should cover everything else and not allow anything to be on top
Errors
No response
What version of the package are you using?
1.11.4

Hey @CanRau also having trouble reproducing. My suspicion is it's a bug in WebKit/Safari related to "correct" vs. actual z-indexing. Not sure how viable this is, but could you try explicitly bumping up the z-index on mux-player to higher than the value applied to the other elements on the page using a >0 z-index?
Also, if you could confirm whether or not your colleague is on the latest version of iPadOS/Safari, that would be helpful.
Tangentially related: Since this is an issue with mux-player/mux-player-react, would you mind updating "Which Mux Elements/Packages does this apply to? Select all that apply" to one of those so we can better track the issue?

related issue https://github.com/muxinc/elements/issues/644
Safari is very sensitive to this stuff. The best way to find out what is causing this is removing all CSS styles in the page which should fix the bug. Then step by step add CSS blocks until it breaks, and then precisely find the CSS property that adds this glitch.

Hey thank you for your comments, still didn't have the time to look into this again, probably next week get the time.
I'll keep you posted

Adding a high z-index solved it 🥳 finally had the time 😅
closing then, thanks y'all


Add Dockerfile

Untested attempt to do something for #5
Can't test it, and it requires rebuild for every appsettings.json change, but should work

@signed-log Thanks for the PR! I wonder - why not use one of the official .NET docker images? Bitnami is a 3rd party AFAIK (and not sure if the image is any lightweight 🤔 ). Will try to test it out in the following week.

I didn't know they actually existed
Bitnami is a VMWare program that produces many software images and has been really reliable for me

The smallest official image (7.0-alpine-amd64) is 240 MB, and the bitnami/dotnet-sdk is 374.44 MB. Not a huge improvement, but I think worth trying.
I often worry about the weight as container registries are not free and you need to host it somewhere 😅
Anyways, I'll try to test it out in the next few days 😃

I tried to run the project with my dockerfile, but I'm getting an error when trying to retrieve the vioshield value. I think because of chrome driver. Do you know anything about this?

Yes,I had to use Selenium to bypass Cloudflare.

I had some time to work on this and created a separate PR: https://github.com/ilkerogutcu/keydrop-giveaway-bot/pull/9. It works, so I guess we can close this one :)


Crud epic factory

TODO: review action interfaces to pass them to epics

Codecov Report

Merging #8 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master     #8   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          38     38
  Lines         197    201    +4
  Branches       12     12
=====================================
+ Hits          197    201    +4

Impacted Files
Coverage Δ

src/actionsCreatorFactory/index.ts
100% <100%> (ø)
:white_check_mark:

Continue to review full report at Codecov.

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


allow user name to be customized

By setting
set :fiesta_slack_username, "New Cool Stuff!!"

in Capistrano.

:+1: 😉


Futures have inconsistent behavior in bash app fn body

Eg :
@App('bash', dfk)
def app1(inputs=[], outputs=[], stdout=None, stderr=None, mock=False ):
    cmd_line = '''echo 'test' > {outputs[0]}'''

@App('bash', dfk)
def app2(inputs=[], outputs=[], stdout=None, stderr=None, mock=False ):

    with open('somefile.txt', 'w') as f:
        f.write("%s\n" % inputs[0])          #<--------------- Here inputs[0] is a DataFuture
    cmd_line = '''echo '{inputs[0]}' > {outputs[0]}'''
app1_future = app1(inputs = [],
                       outputs = [ "simple-out.txt"])
#app1_future.result()

app2_future = app2(inputs=[app1_future.outputs[0]],
                       outputs = ["simple-out2.txt"])
app2_future.result()

One fix is to evaluate the fn body entirely at app execution time. This need some work.

This issue is addressed by issue #37 and fixes are included in branch restructuring_libsubmit by 98f8d40f68479f032aa1a0bcbe20969d3370fc27

Tests passing on branch:restructuring_libsubmit. Closing.


monospaced_display should be True by default

The default value for the monospaced_display parameter is False. I think there is a good case for it to be True by default, given that many command line applications are designed to run in a terminal in a monospaced font.

That's a decent argument for making it the default, but, alas, it is not the default, and I'm pretty committed to not introducing breaking changes, even if it's something like a font change!
So, if I had a time machine: this would be an accepted suggestion, but since I don't, it'll have to remain as is ^_^


remove the active_support dependencies

You need to complete

[ ] unit tests and/or feature tests
[ ] documentation is changed or added

Codecov Report

Merging #22 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #22   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           3      3
  Lines         174    174
=====================================
  Hits          174    174

Impacted Files
Coverage Δ

lib/aliyunsdkcore/roa_client.rb
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


[CARBONDATA-1295] Optimize test cases,merge three QueryTest class into one

1.Optimize test cases,merge three QueryTest class into one, and update related code.
2.Remove one invalid scalastyle.

Can one of the admins verify this patch?

Build Success with Spark 1.6, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/440/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3028/

Build Success with Spark 1.6, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/444/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3032/

LGTM


Fix README.md revalidation function setup example. This closes #14

Fix README.md revalidation function setup example. Instead of specifying the function on setting an item, it should be on creating the StorageLRU instance.
This closes #14

Thanks!


Images stop saving / error in image aftert ~ 1 hr..

This is from @efarq (in https://github.com/TinyCamML/Boron-and-OpenMV/issues/26#issuecomment-2243314197):

Well, I just ran it successfully for one hour, and then the same issue started occurring. I have attached the datalog.txt, I have also attached the images, which shows that the same thing was happening with the images becoming a 2in1  situation. So I think I'll try adding the sensor.reset information in the while loop to see if that helps.

images.zip
DataLog.txt

ok, the last column of the datalog is the memory, which seems to stay constant/low..

this isuggests to me its not a memory issue, but maybe something else... what are th options on the table right now @efarq , @SUPScientist ?

the glitches start at the pictures where the memory gets larger for a cycle or two...

Still curious about the info here (https://forums.openmv.io/t/sleep-mode-wakeup-with-external-interrupt/767/2), despite it being a bit old, as mentioned here (https://github.com/TinyCamML/Boron-and-OpenMV/issues/7#issuecomment-2236514945). The portion of the frame granted to the split image appears to grow over time which could be another symptom of lack of proper resetting?

Does sensor shutdown/sleep need to happen? I see those are commented out:
https://github.com/TinyCamML/Boron-and-OpenMV/blob/19091607839c727c075aa3a415274c732d4bae33/Firmware/OpenMV/PublishOpenMV.py#L82

So I added the sensor.reset() lines to the while loop, and that still doesn't seem to fix the issue. The sensor shutdown items that are commented out are the steps that the senior design team took before putting their camera into machine.deepsleep(). I can try them for machine.sleep() though and see if it helps. The one other thing I think could be happening is when the boron sends its datetime to the openmv, the timing could be off, so then when the openmv tries to write that information to the logs, it doesn't have the number to do so and glitches or something

The main glitch here occurred before the addition of the uart.read() so I doubt it's that, but worth looking under every stone now I suppose.
https://github.com/TinyCamML/Boron-and-OpenMV/blob/19091607839c727c075aa3a415274c732d4bae33/Firmware/OpenMV/PublishOpenMV.py#L47
I don't understand how it was decided what goes into the while loop vs. not in the entire OpenMV script, though. What if almost everything is in that loop? @ebgoldstein, have a ref for what OpenMV does upon wakeup that might help us diagnose?

Just tried adding the
sensor.sleep(True)
sensor.shutdown(True)
lines to the end before sensor.sleep() and that does not seem to work either.

i see what you mean @SUPScientist - the whole camera might need to re-initialize in the while loop... the PR i link to above makes this change... @efarq , can you try it?

Still does not work

can you drop the txt file here?

DataLog.txt
It looks like it did save the image and said "NoFLOOD" but did not send it over to the boron

Ahh okay so the issue is not quite resolved. So it was just running nicely for about 40 minutes and then started doing the thing again. The only difference from when Evan and I were looking at it successfully on zoom until now was that I put the sampling period back to 6 minutes and turned the publishing back on for the Boron. On the OpenMV, the images look fine, but a new ERROR.txt was created, which I've attached. Also, the OpenMV light started blinking nonstop, which is what caught my eye this time, which hasn't happened really much before
https://github.com/user-attachments/assets/c70c18a1-6da7-4265-9d93-b6eef2ce623c
DataLog.txt
ERROR.LOG

shoot.. ok, looking at forums now...

can you try something simple, and just replace the jpg to bmp on this line:
https://github.com/TinyCamML/Boron-and-OpenMV/blob/19091607839c727c075aa3a415274c732d4bae33/Firmware/OpenMV/PublishOpenMV.py#L66
so  img.save("./images/" + file_name + ".bmp")
(this error has popped up previously on the forums related to jpg stuff...

ok, @efarq - can you change back to jpg, run the code till it breaks, and post all of the relevant details here (the exact OpenMV code, the folder of images, datalog, and error log)? and also maybe any details (how long it has run for)?
i think its time we post on the OpenMV forums, and I can do that

also, please note the firmware, and openMV details (at the bottom of the IDE)

Exact openmv code:
folder of images:
datalog file:
error log:
Firmware:
Board:
Things we've tried: changing .jpg to .bmp, adding a gc.collect(), adding del img, making sure we have the sensor.shutdown and reinitializing the camera after every restart
Also, I have a question for @SUPScientist, the openmv issue seemed to be fixed until we turned on the cellular from the boron and entered the publish state. In the publish state, we have the following lines of code:
 // Clean out any residual junk in buffer and restart serial port     Serial1.end();     delay(1000);     Serial1.begin(9600);     delay(500);     Serial1.setTimeout(TIMEOUT_TINYCAM_MS);
My question is why do we begin the Serial1 again when we don't use it for the rest of the publish or sleep state? Could that be causing a mismatch between the boron and openmv?

just to clarify @efarq - if cellular is off, the OpenMV will run forever?

I only saw what we saw on zoom together yesterday but it looked like it was working great without cellular. I'll try it out again real quick

OK, interesting. Eager to hear results of next test with publishing off (#define PUBLISHING 0). No idea how/when/from whom that chunk
https://github.com/TinyCamML/Boron-and-OpenMV/blob/19091607839c727c075aa3a415274c732d4bae33/Firmware/Boron/src/TinyCamPublish.ino#L107-L112
crept into PUBLISH_STATE but you're correct that it doesn't make sense there. Good catch.

We need a very minimal reproducible example of the failure if we are going to post on OpenMV forum, so i am trying this (with firmware 4.5.5):
Note that there is no sleep (nor poll, nor uart.read), but each loop it re-initializes then shuts down the camera...

import sensor, image, time, utime, pyb, tf, machine, gc, os, uselect
from pyb import UART, Pin, ExtInt
from machine import LED

uart = UART(1, 9600) # UART1, adjust baudrate as needed

net = tf.load('MNv2Flood_cat_CG.tflite', load_to_fb=True)
labels = ['Flood', 'NoFlood']

#make directory to save images
if not "images" in os.listdir():
    os.mkdir("images")  # Make a directory

pin = Pin("P7", Pin.IN, Pin.PULL_UP)
ext = ExtInt(pin, ExtInt.IRQ_FALLING, Pin.PULL_UP, callback)

curr_time = 0

while(True):
    #Reinitialize the sensor after sleep
    sensor.reset() # Initialize the camera sensor.
    sensor.set_pixformat(sensor.RGB565)
    sensor.set_framesize(sensor.QVGA)
    sensor.skip_frames(time = 2000)

    img = sensor.snapshot()

    TF_objs = net.classify(img)

    Flood = TF_objs[0].output()[0]
    NoFlood = TF_objs[0].output()[1]

    if Flood > NoFlood:
        print('Flood')
        uart.write('Flood')
        pyb.delay(1000);
        file_name = str(curr_time) +"_FLOOD"
        floodstate = "Flood"

    else:
        print('No Flood')
        uart.write('No Flood')
        pyb.delay(1000);
        file_name = str(curr_time) +"_NOFLOOD"
        floodstate = "NoFlood"

    with open("./DataLog.txt", 'a') as file:
        file.write(str(curr_time) + "," + floodstate + "," + file_name + "," + str(gc.mem_alloc()) + "\n")

    print(curr_time)
    curr_time = curr_time +1

    #print(file_name)
    img.save("./images/" + file_name + ".jpg")

    # run garbarge collection to prevent memory growth
    gc.collect()

    # Begin power down ------------------------------------
    # Shutdown the sensor (pulls PWDN high).
    sensor.shutdown(True)

Okay cool I'm trying it out with publishing off right now and then if that continues to work, I'll try commenting out those 108-112 lines and turning publishing back on to see if that helps anything.

And then if that doesn't work, I'll get you all those files/details for the openmv forum, Evan

i do think there is a timing issue, only because the datalog.txt files have strange moments where memory grows and there seems to be a 'skipped' uart read/write cycle... so the lines you identified might atleast help with that 🤞

Update: commenting out lines 108-112 on boron's code still did not work. I'm going to continue to mess with the Boron's code, but I think it would be good to post on the openmv forum too.
Exact openmv code as a .txt:
maintxt.txt
folder of images:
images.zip
datalog file:
DataLog.txt
error log:
ERROR.LOG
Firmware: 4.5.5
Board: OpenMV Cam H7 Plus (STM32H743 w/ SDRam)
Sensor: OV5640
Things we've tried: changing .jpg to .bmp, adding a gc.collect(), adding del img, making sure we have the sensor.shutdown and reinitializing the camera after every restart
Symptoms: OpenMV cam starts blinking white

@efarq - I have read that sometimes long openMV runs where it's connected to the IDE have problems. Is it possible for you to try deploying this not connected to the computer and see what happens?

Currently, the only thing connected to my computer is the boron, which is powering the rest of it - so the OpenMV is not connected to the IDE.
Just writing down this idea - but maybe the next best thing to do would be make some sort code that runs like:

Boron wakes up OpenMV and waits to move on until it receives something from Serial1.readString() aka flood or no flood.
The openmv has a delay while the serial1 is resetting in the boron and then waits to move on until it receives a value for the datetime from the boron. Then it sends an "all clear" to boron
Then the Boron won't enter publish mode and subsequently publish mode until it receives an "all clear"

I guess that opens the possibility of either one getting stuck in infinite loops but just thinking about how to get their timing lined up. The other thing I am wondering about is if there is a way to turn off the vusb power entirely on the boron each cycle, as that would give the openmv a fresh start every time. Because we use the A0 pin on the boron to toggle the openmv on, it seems like you may have thought about that before. Perhaps machine.lightsleep() doesn't reset the camera enough? Just brainstorming at this point

i like this idea, especially if we pair it with writing to a text file on OpenMV to help debug...

and i think we need this sort of brainstorming... i firmly believe there must be a way forward, and i think it involves being more careful with all the comms (and having a way to track it for debugging)..

The other thing I am wondering about is if there is a way to turn off the vusb power entirely on the boron each cycle, as that would give the openmv a fresh start every time.

This merits more exploration. A FET switch tied to VUSB's output from Boron to OpenMV and another GPIO on Boron should do the trick (should have some in the lab, same as I think you used for SEACOWs). The "fresh start" idea is compelling.
Re: @efarq's other idea that slows down/makes more explicit the Boron<—>OpenMV handshake seems reasonable but I'm having a hard time guessing how that gets at the root problem here. Maybe serial.flush statements on both sides "clean out" any residual stuff in buffers if that's the hypothesized problem?

I will continue to investigate this week...

I saw @ebgoldstein's branch about cleaning up the uart communications and that it worked for you for at least one hour. Were you testing that with cellular on the boron? Your code is not quite working as nicely for me.
However, last week I did add the mosfet switch into the system and it seems to work great! I ran it for almost 12 hours with no issues except that it adds a � into the result, which I cannot figure out how to get rid of:

It does seem to publish the character with no issues though so I don't think the character is necessarily a huge issue but will just need to be fixed eventually. Here is the code within the branch that adds the on/off switch:
https://github.com/TinyCamML/Boron-and-OpenMV/blob/OpenMVFix/Firmware/Boron/src/TinyCamPublish.ino
I'll test it again today but I feel pretty good about the consistency with the mosfet switch.

this is great news @efarq !
Did your second test run ok?
if so, i guess the action item is to make sure the boron & OpenMV are exactly the same as one the decives.. ( i see the new circuit diagram w/ mosfet).. then close the issue and we all have a big party in your honor for figuring it out!

Yes the second test seemed to run quite nicely! So maybe the mosfet switch is the way to go. What do you mean by, "the action item is to make sure the boron & OpenMV are exactly the same as one the decives" before closing the issue?

Sorry, was not super clear.. All I mean is- 'If the code on GitHub the exact same as the code on the devices, close the issue. If not, update code and close the issue'

Perfect, just wanted to make sure before I changed anything! Here is the merge: commit b141d4c


Add trim functionality to preview seekbar

Should be integrated into the seekbar itself, not add a separate range slider.

Don't really think this is actually that useful anymore


added new keyword to get device date and time

Hope this will fix https://github.com/appium/appium/issues/5893

Hey awesome, thanks for the contribution!
I left a small comment, with a suggestion of how to simplify the code. Could you update it?
After this gets merged, we will have to add the route to the server, to activate this method.
You can submit a separate pull request to this repo: https://github.com/appium/node-mobile-json-wire-protocol
I think we can make the route something like /device/system_time
Do you want to implement this on android driver too? Probably easy with adb. http://stackoverflow.com/questions/18135966/how-to-get-android-system-uptime-and-realtime-from-command-line

Yes, ofcourse @Jonahss. Working on the changes suggested.Will update the PR in sometime. Will look into android and routes after this got merged.

awesome!
On Tue, Jan 5, 2016 at 11:38 AM, Srinivasan Sekar<EMAIL_ADDRESS>wrote:

Yes, ofcourse @Jonahss https://github.com/Jonahss. Working on the
changes suggested.Will update the PR in sometime. Will look into android
and routes after this got merged.
—
Reply to this email directly or view it on GitHub
https://github.com/appium/appium-ios-driver/pull/59#issuecomment-169107272
.

@Jonahss can you please review this?

Added a couple of style fixes. Otherwise, excellent!

Getting closer, sorry for all the corrections.
Rather than using util.detectUdid(), we can use this.opts.udid. The driver object already is running a session when this command is called, so we already have a reference to the udid for this test.
We should also add a check, so make sure this command is only run on real devices (idevice_id) will not work with simulators I'm pretty sure. You can call await this.isRealDevice() and throw an error if it's false.

oops, its just this.isRealDevice() without the await

@Jonahss Is it fine now?

@Jonahss Did get clear about the reason for CI job failure.

Ah, I've seen this error elsewhere, I don't think it's your fault. will rerun.

@Jonahss Build passed.

Along with the log warning, can you also throw a NotImplemented error when realDevice is false? That way the test writer knows that the command failed, otherwise it will seem like it succeeds but the time returned will be null. This could result in an annoying null pointer exception for the end user.

Yup Got it @Jonahss Included the changes

@Jonahss Did'nt get clear about the reason for CI job failure.can you trigger again pls

@imurchie Is this looks fine?

I'm looking into it. This could either be an issue related to travis, or to appium-ios-simulator.
Currently working on figuring out whats wrong and trying to fix it.

Any update on this @Jonahss

I'm going to merge this!
Will be reviewing mjsonwp and android-driver related pull requests.

Thanks @Jonahss Working on android-driver implementation.Will raise a PR for android-driver soon.

@Jonahss Can you please review my android-driver and mjsonwp PR related to this


Make this a registered Julia package

Closes #1.
Tasks:

[x] Add documentation
[x] Configure CI
[x] Configure helper workflows for Julia

All I need to do now is to register the package, but I will do this after I merge this into master.  It seems all of the CI/CD/docs stuff is good to go.


I like trailing commas in config

I searched the Extensions repo to see if there was any discussion about removing support for trailing commas in json config. I found this from @HaoK:

@ajcvickers tests now are all passing, the trailing comma was a big issue in the new common provider tests you added, smallest change was to run it thru JSON.NET to clean it up. Thoughts?

https://github.com/aspnet/Extensions/pull/1028#issuecomment-461177591
My thoughts are that allowing trailing commas in config is the obviously better choice. And even if it's a close call, shouldn't we lean towards not breaking people? This broke the Kestrel "SampleApp" for instance.
Was there any other discussion about removing support for trailing commas in json config when switching from JSON.NET to System.Json.Text?

Was there any other discussion about removing support for trailing commas in json config when switching from JSON.NET to System.Json.Text?

No. In fact I pushed to get a setting added to S.J.T for explicitly for config files.

We probably need to expose this on the AddJson level if we are going to stray from the defaults

Hopefully that can wait for another PR. We already stray from System.Json.Text defaults for config wrt comments.

Change the file extension to .json5 :trollface:
Nah, I love it. Let's do it. Trail all the commas!

Did you finally run the kestrel sample?

I'm more than fine with supporting trailing commas so we don't break people. When this was discussed in the past (along with the single quotes issue) the answer was no, because the new thing was supposed to be "cleaner" or something like that.

The new parser doesn't support them by-default, which I believe was part of the discussion around making it a more "spec-adherent" parser. However, it does have the option to turn support on, which we take advantage of. I generally agree with both approaches that have been taken here (parser should be spec-adherent by default but consumers should be as flexible as their scenario allows).

Did you finally run the kestrel sample?

Yes. You know you could have submitted this PR back when you saw this issue first 😉.


Release 1.14.0

Update the changelog to package version to 1.14.
Resolves: OLPEDGE-2658
Signed-off-by: Mykhailo Kuchma<EMAIL_ADDRESS>

Please also add a line regarding newly merged #1276


Missing lodash dependency

If you try to run this without lodash installed globally, it will fail.

This works fine for me on a fresh trusty install without any lodash? What problem do you have specifically?

Are you sure you don't have lodash installed globally?
webpack-dev-server --config examples/webpack.config.js --content-base ./public/ --host <IP_ADDRESS> --port 1234 --hot --inline
http://<IP_ADDRESS>:1234/
webpack result is served from /
content is served from /Users/alex/dev/reactcards-original/public
Hash: 220c3b612b91719a572a
Version: webpack 1.13.1
Time: 7590ms
 Asset     Size  Chunks             Chunk Names
app.js  1.84 MB       0  [emitted]  main
chunk    {0} app.js (main) 1.71 MB [rendered]
    [0] multi main 76 bytes {0} [built]
    ....

ERROR in ./src/Container.js
Module not found: Error: Cannot resolve module 'lodash' in /Users/alex/dev/reactcards-original/src
 @ ./src/Container.js 17:14-31

Look in ./src/Container.js - line 3 is import { find, map } from 'lodash' but you have no lodash mentioned in the package.json.

I didn't realize that we've added lodash as a dependency. Not sure why I didn't see any error though. Thanks!


Docker Community Edition docker daemon is not supported on Linux.

Encountered the following error on Ubuntu 16.04 running docker version 17.06.0-ce.
``docker daemonis not supported on Linux. Please rundockerd directly
The ExecStart= line of the 'service-overrides-debian.conf.erb' template needs to use 'dockerd' as the command instead of 'docker daemon' when using the Community Edition. Editing the value and reloading the systemd daemon confirmed. Puppet run will revert the change, but daemon will continue running until  restarted and require the same change to restore service.

@ocastle would you be willing to test out some changes that I have in a fork?  I've been having the same problem with CentOS, but I've hacked together a workaround.  Its in my fork https://github.com/grantholly/garethr-docker.

From my tests, the latest docker (docker-ce_17.06.0~ce-0~ubuntu) on Xenial does not use the /etc/systemd/system/docker.service.d/service-overrides.conf file. The workaround for me was to delete that file altogether, rather than modifying /usr/bin/docker daemon to /usr/bin/dockerd.
I'm not sure, but I think /lib/systemd/system/docker.service is used instead.

@grantholly yes i'd be willing to take a look at the changes and integrating into my own fork for testing.

Hi,
Any news on this issue ? :)

The changes from @grantholly were integrated into the working fork used for testing and were verified to solve the issue faced on Ubuntu:16.04

The workaround for me was to delete that file altogether, rather than modifying /usr/bin/docker daemon to /usr/bin/dockerd.

This is related to #703 & #698. You actually shouldn't have to manually perform any workarounds, and should instead be able to pass in dockerd as the docker_command, but it's not writing out to the correct file.

In addition to changing docker_command to dockerd, you also need to override daemon_subcommand to an empty string to get a correct service file override. See https://github.com/garethr/garethr-docker/blob/master/templates/etc/systemd/system/docker.service.d/service-overrides-debian.conf.erb

Yep, already got that one. Here's what I've done so far.
The challenge I'm working through as we speak is the dual nature of docker_command usage in the module.
In the manifests/run.pp, manifests/image.pp, manifests/exec.pp, etc... docker_command is used as a client library.  I can't just change up docker_command = 'dockerd'.
The approach I'm taking right now, is to have 3 parameters:
docker_command = 'docker'
daemon_command = 'dockerd'
`daemon_subcommand = undef'
Using daemon_subcommand will output a deprecation warning, and shim the correct usage of docker daemon vs dockerd.
This will allow compatibility with any existing downstream manifests while enabling the latest versions of docker to run.

Is there a preference for making $docker_command work over using facter to figure how to call the docker daemon?  I have a pull request open (https://github.com/garethr/garethr-docker/pull/704) that fixes this issue and has been checked to work with at least Ubuntu and CentOS.

I reviewed the PR, and the only flaw I see is in the assumption that you're always going to have dockerd available. We'd have to declare that this module only supports specific versions of docker, deprecating support for versions of docker > 1.12.
I'm also not certain that the facter technique will work with custom build versions of docker (example being docker-latest package that is released for RedHat family OS's).

@grantholly would you mind if I pulled your PR (#704) into my fork?  While I think it can't do the job on it's own, it will work for like 99% of cases, and will serve as a much more solid base than just assuming /usr/lib/... inside the template files, (which has been bothering me for a while now.)

@LongLiveCHIEF go for it.
In my PR, there is no assumption that dockerd is installed.  The way the fact works is you either get a nil value for $dockerd_binary, as you would in the case of running an older version of docker, or you would get a path to the dockerd binary.  Here's the details on Facter::Core::Execution#which (http://www.rubydoc.info/gems/facter/2.4.6/Facter/Core/Execution#which-class_method)
In the unit file templates, (https://github.com/garethr/garethr-docker/pull/704/files#diff-7dae3e5d9fd27c34844a4d6e0485e2e0L8) and (https://github.com/garethr/garethr-docker/pull/704/files#diff-1ae24078d52aa1e01fa570dd84a1efddR5) there's a boolean check for the binary.  If $dockerd_binary is nil then we'll use the older docker daemon command.

I like it.

That nil value seems to generate a broken systemd file though?
Specifically the first run facter will return no value. As it runs before the package is installed.

Hello, facing this issue too, easy solution (waiting for the PR to be merged), if you're using Hiera:
docker::service::docker_command: 'dockerd'
docker::service::daemon_subcommand: ''

This works perfectly with docker 17.06 under CentOS7

@vide this will only work for installing and starting the docker service. If you use any of the custom docker resources from this module, the above settings will break your puppet catalogs.
docker_command is used in several different ways at the moment, and for the custom resource types, it's used as the client command.  This will break image, run, etc...

I bypassed the issue by declaring FACTER_DOCKERD_BINARY=/usr/bin/dockerd in
the packer script that was failing. It's an imperfect solution but allowed
my image building process to be unblocked.
On Aug 2, 2017 8:45 AM, "Brian Vanderbusch"<EMAIL_ADDRESS>wrote:

@vide https://github.com/vide this will only work for installing and
starting the docker service. If you use any of the custom docker resources
from this module, the above settings will break your puppet catalogs.
docker_command is used in several different ways at the moment, and for
the custom resource types, it's used as the client command. This will break
image, run, etc...
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/garethr/garethr-docker/issues/701#issuecomment-319712771,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAxu2e2-GsGwlVGtQyO95mMzrHHYVzSPks5sUJmHgaJpZM4OLDoA
.

Nope, I'm overriding only the docker::service class, not the base docker
class, it works perfectly, I'm running container and pulling images through
puppet without any issue
Davide
On Aug 2, 2017 17:45, "Brian Vanderbusch"<EMAIL_ADDRESS>wrote:
@vide https://github.com/vide this will only work for installing and
starting the docker service. If you use any of the custom docker resources
from this module, the above settings will break your puppet catalogs.
docker_command is used in several different ways at the moment, and for the
custom resource types, it's used as the client command. This will break
image, run, etc...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/garethr/garethr-docker/issues/701#issuecomment-319712771,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAQfu1MKjLLUjF710C331bS5D0Ud72S-ks5sUJmHgaJpZM4OLDoA
.

At least it works on the latest released version, 2.3.0. Maybe in master is broken, IDK, on mobile right now, I will check tomorrow

@LongLiveCHIEF Still work on current master to. The line that makes it work is
https://github.com/garethr/garethr-docker/blob/master/manifests/init.pp#L530
where the docker::service class is contained without overriding any parameter, so if you set docker::service::docker_command and docker::service::daemon_subcommand in Hiera as I said in my first comment, it works. In fact, it's working for me in my production servers.

Yeah, it does make sense actually, I just never considered the service
manifest as an entry point.
Doing it that way would keep the docker command that's inherited from the
normal entry point  at it's default value, so even the custom resources
from this component will still work correctly.
On Aug 4, 2017 4:50 AM, "Davide Ferrari"<EMAIL_ADDRESS>wrote:

@LongLiveCHIEF https://github.com/longlivechief Still work on current
master to. The line that makes it work is
https://github.com/garethr/garethr-docker/blob/master/
manifests/init.pp#L530
where the docker::service class is contained without overriding any
parameter, so if you set docker::service::docker_command and
docker::service::daemon_subcommand in Hiera as I said in my first
comment, it works. In fact, it's working for me in my production servers.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/garethr/garethr-docker/issues/701#issuecomment-320207912,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADIVgdI6px7udm4CqcUUz75rK4FW0mqsks5sUulogaJpZM4OLDoA
.

I would prefer to do this without any code change as you suggest @vide.  I've tested your work and it works like a charm.  However, It doesn't make sense for me to add docker::service::docker_command to every single manifest where we have the docker class applied.  I would like to do something like
Class { 'docker::service':   docker_command         => 'dockerd',   daemon_subcommand => '', }
in one place (our docker wrapper class).  However, this causes a duplicate resource declaration.

I have this in the master of my fork.
On Aug 22, 2017 5:06 PM, "grantholly"<EMAIL_ADDRESS>wrote:

I would prefer to do this without any code change as you suggest @vide
https://github.com/vide. I've tested your work and it works like a
charm. However, It doesn't make sense for me to add
docker::service::docker_command to every single manifest where we have
the docker class applied. I would like to do something like
Class { 'docker::service': docker_command => 'dockerd', daemon_subcommand
=> '', }
in one place (our docker wrapper class). However, this causes a duplicate
resource declaration.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/garethr/garethr-docker/issues/701#issuecomment-324165199,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADIVgcAyhjXxpSPcG6i-o0JHea-wqc1tks5sa1DkgaJpZM4OLDoA
.

TL;DR : declare the storage_driver (overlay(2) )parameter specifically.
Hi everyone, this is my 2cents on this issue:
In my case, it seemed to "correct" itself by specifying the storage driver. I am running on a debian jessie with the latest kernel from backports. Doing so it does not complain anymore about the dockerd issue... still need to investigate why, but i'm happy that it runs smoothly. YMMV.
This is what my manifest declaration looks like:
`
apt::source { 'jessie-backports':
location => 'http://ftp.debian.org/debian',
release  => 'jessie-backports',
repos    => 'main',
}
$kernel_pkgs = ['dkms', 'linux-headers-amd64', 'linux-image-amd64']
package { $kernel_pkgs:
ensure          => latest,
install_options => ['-t', 'jessie-backports'],
before          => [Class['::docker'], Class['::docker::compose'], Package[$firmware_pkgs],],
require         => Apt::Source['jessie-backports'],
}
class { '::docker':
version        => 'latest',
docker_users   => ['johndoe', 'janedoe'],
storage_driver => 'overlay2',
require        => Package[$kernel_pkgs],
}
class { '::docker::compose':
ensure  => present,
version => '1.15.0',
}
`

As this thread is still getting commented on:  as an FYI it's not techniclaly a module issue.  It's becuase of the removal of a command in the latest 17.06 that isn't slated for removal until 17.12 and is impacting lots of people, not just those who deploy via puppet:
https://github.com/docker/for-linux/issues/11
In that light I'm not 100% certain that a fix via puppet module is the right answer.  Instead, the docker team should be releasing a "good" package that installs and "stays running".  As others have mentioned, falling back to 17.03 will bypass this issue, and I'm playing with the idea os simply modifying the offending file and setting it correclty to dockerd in the meantime.

Quick warning:  while setting docker::service::docker_command: in Hiera will work for new installs, it's not bullet proof.
Anyone who's run into this problem in frustration and thus doesn't have a running copy of docker wont be able to get back and running this way, as the module itself tries to run docker before applying the service-overrides.conf file and thus will never get to the step of updating the file.
Thus you will be forced to update it manually or with some in-file puppet magic.
Honestly I think the "process" is backwards in the module.  all settings should be set before starting/restarting the service, which removes this "the module sets it back but the service is still running" scenario.

@Justin-DynamicD Exactly.
We worked around this on already broken nodes by adding this ugly exec:
  contain docker

  exec { 'docker-remove-broken-systemd-overrideconf':
    command => '/bin/rm /etc/systemd/system/docker.service.d/service-overrides.conf',
    unless  => 'grep ExecStart=/usr/bin/dockerd /etc/systemd/system/docker.service.d/service-overrides.conf',
    onlyif  => 'test -f /etc/systemd/system/docker.service.d/service-overrides.conf',
    before  => Class['docker::service'],
  }

It deletes the service-overrides.conf file, if it exists and does not contain the correct ExecStart command yet. Afterward the docker::service class drops in the correct file and the daemon starts correctly.
I'll be removing this monster from our manifests as soon as all nodes are confirmed working.


Simplify the util module

This variant of try_collect might not be as general as the previous one, but it also avoids the into_iter().collect() roundtrip and reallocation for the C_ = Vec<_> case, which is also the only case used in the application.

Maybe you merge these, as you may be better able to resolve conflicts or do a rebase? Do you have the required permissions?

Maybe you merge these, as you may be better able to resolve conflicts or do a rebase? Do you have the required permissions?

Maybe you merge these, as you may be better able to resolve conflicts or do a rebase? Do you have the required permissions?

I can do and will do. Generally speaking, the maintainer usually merges in the order he prefers. If this creates conflicts in the remaining pull requests, it is the task of their authors to resolve those. Nothing you have to do in that case besides possibly reminding people to go resolve those conflicts in their code.

Maybe you merge these, as you may be better able to resolve conflicts or do a rebase? Do you have the required permissions?

I can do and will do. Generally speaking, the maintainer usually merges in the order he prefers. If this creates conflicts in the remaining pull requests, it is the task of their authors to resolve those. Nothing you have to do in that case besides possibly reminding people to go resolve those conflicts in their code.


Contact image not aligned in Circle

To replicate choose a contact which has a profile pic in the phone contacts.

-Now the outline here is in square form instead of circle.

Issue still replicated in on staging app v1.6.0(277)

@AliMeer I checked on the Android device (Redmi) and Issue seems to be still replicated.


Fix parent / child assignment  logic fix

Fix #1109
Change
The matching logic (string) was "reversed" for membership so that parent assignments would  override child assignments. That is the logic should be to check if the path of the assignment is completely contained within the path to the geometry being tested.
e.g.

The assignment string "/a/b" is contained in path to the geometry: "/a/b/c" so the material can be assigned to "/a/b/c"
The assignment string "/a/b" is contained in path to the geometry: "/a/b" so the material can be assigned to "/a/b"
The assignment string "/a/b/c" is not contained in path to the geometry: "/a/b" so the child's material cannot be assigned to the parent geometry.

Desktop

Web

Official PR https://github.com/AcademySoftwareFoundation/MaterialX/pull/1113
has been put up. Closing local PR.


Solidity Bridge Deployment Scripts

Summary
Holesky Deployments
AtomicBridgeInitiatorMove : 0xb33d9fA868054FD3F8F02ba9Fc04B2a1862Ab855
AtomicBridgeCounterpartyMove : 0xA6bB0d5eA03f3bEe3510EA3Ec39e85Cd7395Bc37
Adds the necessary scripts for deploying the .sol atomic bridge contracts. Follows the same governance procedures laid out in MOVETokenDeployer.
NOTE that the addresses set for movementLabsSafe and movementFoundationSafe are unused addresses for now and are to be verified for real deployment to testnet and mainnet.
Changelog

Adds AtomicBridgeInitiatorMOVEDeployer.sol
Adds AtomicBridgeCounterpartyMOVEDeployer.sol

Testing
You can test this by deploying against a local anvil network :
PRIVATE_KEY=<YOUR_PRIVATE_KEY> forge script AtomicBridgeInitiatorDeployer --for
k-url http://<IP_ADDRESS>:8545 --broadcast

Grab the contract address generated and use that as the Initiator arg for the Counterpary script
PRIVATE_KEY=<YOUR_PRIVATE_KEY> forge script AtomicBridgeCounterpartyDeployer --for
k-url http://<IP_ADDRESS>:8545 --broadcast

You might want to merge primata/contract-pipeline and inherit Helper so that you can have all methods in place.

these addresses are wrong, here is the ethInitiator 0xa8ce38EF26fB6e912954cf1309f98d5a753d367F
and this is the ethCounterparty 0x09Da59E3fe9Db7BEb26e55467583b1C50357DC85

@0xmovses What are the next steps you'd like to take here?

@l-monninger Let me rebase main and redeploy these contracts, I'll update the PR description with new addresses.

I'd say this looks good pending resolution of @0xPrimata's comments


Separate compilation and synthesis steps

Currently the compile command performs two major steps:

compilation from vamp-ir to three-address codes (3ac)
synthesis from 3ac to a circuit in zk-garage/plonk

The second step requires public parameters and so vamp-ir cannot be compiled without first running a setup or reading parameters from a file. A user should be able to test their circuit by compiling it, or inspect the 3ac form of the circuit, or export the 3ac representation to a file etc. without having to read public parameters which are only needed for synthesis.

Perhaps we can split this into two commands:
vamp-ir compile --source <SOURCE> --output <OUTPUT>
vamp-ir synth --universal-params <UNIVERSAL_PARAMS> --circuit <CIRCUIT> --output <OUTPUT>

where the output of compile is a circuit in 3ac representation


Remove organization table and reference in locations table

Resolves #134
Description
Removes the Organizations table, and the organization_id in the locations table.  The final step in getting rid of Organization from the app entirely.

Thanks @cattywampus! :)
All set @DamirSvrtan we good to go now?

I manually restarted the travis build and it passed

@cattywampus seems like there is something flaky with the specs and the fixtures, that's probably why it's something passing and sometimes not. I can take a look at it!
I'm okay with merging this in if you are!


Use variadic arguments for Allocate function

@tylerb please review
REF: https://github.com/Rhymond/go-money/issues/18

Coverage remained the same at 90.0% when pulling 1ccf7d03bc0d31175d3a857669406c949421bfa5 on AllocateUpdate into 8b9a5ea73c605701cef875e206c3df41ec2e6526 on master.

Coverage remained the same at 90.0% when pulling 1ccf7d03bc0d31175d3a857669406c949421bfa5 on AllocateUpdate into 8b9a5ea73c605701cef875e206c3df41ec2e6526 on master.

Coverage remained the same at 90.0% when pulling<PHONE_NUMBER>796111a2672c501660633babfd2724 on AllocateUpdate into 8b9a5ea73c605701cef875e206c3df41ec2e6526 on master.


NotWorking!?

So i setup a new account to try this out, i have all correct details etc, but it won't go past the login screen. I have entered details correctly and still cant go past the login. also an older version requires a pass code?

Fix will be released shortly.

cool beans man look forward to trying it.


add docco style parser for "unstructured" documents

yii2-apidoc
With docco it is very easy to document source code files like configurations, views or even javascript.
It would be nice if we could add a parser which does a similar thing.
Or include HTML-files generated by docco (a custom docco-template could be needed)
It's basically extracting // comments from the source...

Interesting idea.

Would make the application documentation pretty complete.
You can use Markdown syntax with docco, eg. for headlines - could come handy when linking to source code.

This could be helpful: https://github.com/rossriley/phrocco

its unclear in this topic how the framework could benefit from docco. Please describe a detailed usecase on the advantages & disadvantages of this tool for Yii.

It's not in the scope of Yii core I think. Maybe for Apidocs or as an extension.


Authenticating clients against individual services

It would be great to have a libS client to be able to authenticate via SSL as an authorized user of a service. Alternatively an authentication token within established SSL certs that are shared across services could also work here. This would pertain to a libStorage client talking to possibly many services as well.
Lots of work here to be done here.

Related to https://github.com/codedellemc/rexray/issues/584

From @vladimirvivien

Users should be able to securely connect the libStorage client to the libStorage server

authenticate per service an option

A user should have the ability to easily configure a secure connection between libStorage client and the service backend
Users should be able to configure secured connection using several configuration vectors including environment variables and the client configuration YAML file.

Token auth tracked via #475

Replaced by #475.


perf: BW6 pairing computation using non-native Eval

Description
This PR refactors the methods in the fields_bw6761 package to use the Field.Eval methods introduced in #1299. Depends on it being merged first.
Currently we use simple schoolbook multiplication formulas, but can have better performance with specialized formulas a la Toom-Cook etc. Currently didn't find a good way to give a multivariate representation to them.
Type of change

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

How has this been tested?

[x] TestSquareVariantsFp6
[x] TestMulVariantsFp6
[x] TestFp6Mul023By023Variants
[x] TestFp6MulBy02345Variants
[x] TestFp6CyclotomicSquareKarabina12345Variants

How has this been benchmarked?

Single Miller loop before: 6500708, now: 3841000
Single Miller loop fixed G2 before: 5344302, now: 2680076
FinalExp before: 5245872, now: 3362746
Full pairing before: 11486969, now: 6947630
Full pairing fixed G2 before: 10440385, now: 5888826

But keep in mind that I only measured single operations so they include range checks for the initial witness.
Checklist:

[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[x] I did not modify files generated from templates
[x] golangci-lint does not output errors locally
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

Actually squareDirect is a re-implementation of mulDirect. Last commit specialises it to the evaluation of equal operands and saves some more scs.

Also slight optimization of Karabina decompression Eval which saves an additional 11k scs. Full pairing is now 6,920,110 scs.

Full pairing is now 6919174 scs.