Unit tests fail for pip-installed gpfit

Unit tests currently fail on a fresh pip installation for two reasons:

It uses some pre-release GPkit features (initsolwarnings).

This will be resolved by GPkit release 1.0
Current solution is to pip install a local version of latest GPkit master

4 tests write output to a file that doesn't exist

======================================================================
ERROR: test_hoburgabbeel_ex6_3_mosek_cli (gpfit.tests.t_examples.TestExamples)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/philippe.kirschen/Optimization/gpkit/gpkit/tests/helpers.py", line 55, in test
    testfn(name, import_dict, path)(self)
  File "/Users/philippe.kirschen/Optimization/gpkit/gpkit/tests/helpers.py", line 86, in test
    with StdoutCaptured(logfilepath=filepath):
  File "/Users/philippe.kirschen/Optimization/gpkit/gpkit/tests/helpers.py", line 155, in __enter__
    sys.stdout = (open(self.logfilepath, mode="w")
FileNotFoundError: [Errno 2] No such file or directory: '/opt/anaconda3/envs/test_gpfit/lib/python3.9/site-packages/docs/source/examples/hoburgabbeel_ex6_3_output.txt'

Does #105 fix this?

No, I don't think it will. Thank you for the reminder.

@bqpd do you have suggestions for unit tests that use gpkit.tests.helpers.generate_example_tests when the example/test files live in a subdirectory of docs which is not included as a package inside GPfit. Is the only appropriate solution to move the docs dir into gpfit/gpfit/?

I might just skip the docs/example tests when it's a PyPi install (see t_examples.py) :grimacing: bad practice, I know

There might be the alternate option of including docs as a data directory?

Thanks @bqpd this was the nudge I needed. I used include_package_data=True added a MANIFEST.in.

ahaha glad to hear you're doing it the right way!

Although doesn't seem to have worked... (Thanks @galbramc for retesting)

I tested locally when I made the change and the tar-zip included the directories it was supposed to, so I'm kinda confused.

@galbramc sorry to ask more of you, but I have a few questions about Jenkins to try to understand what's going on. My problem seems to be that the directory tests/artifacts/ doesn't exist in the version installed on Macys and Reynolds, despite being included in my locally generated dist/ tarballs.

What is the significance of '--omit=gpfit/tests/*' in the line below? Is it relevant to the issue I am describing?

++ python /Users/jenkins/workspace/CE_gpfit_PR_unit_tests/buildnode/macys/venv2_gpfit/bin/coverage run --source=gpfit '--omit=gpfit/tests/*' test.py

I re-tested a locally pip-installed version of the code and realized I wasn't including a package gpfit.maths in setup.py. This caused my tests to fail locally and that's why I made the change in 11013ad, but I'm confused why those tests weren't failing on Jenkins. Do you know why that would be the case?
Similarly, prior to that last commit (which didn't change any core code) I was failing two tests that are no longer failing. The tests should be deterministic now so I'm confused why this would be happening.
It looks like pylint tests were passing prior to this last commit but now I see the following in the console output:

FileNotFoundError: [Errno 2] No such file or directory: '/Users/jenkins/Library/Caches/pylint'

The --omit=gpfit/tests/* simply means don't include the testing files when generating coverage information. The coverage information tells you how much of your code is actually getting tested, and the testing files them selves artifcially inflate the numbers.
I have no idea why the testes were not failing due to the missing gpfit.maths...
I've noticed there are a couple of tests in gpfit that do not seem to be deterministic. I'm guessing they use random numbers and some random numbers give you passing tests, while other random numbers give you failing tests.
I don't see that pylint error in the console output. Could you point me to the specific build? Jenkins was previously not properly catching the pylint errors. Make sure you have this line in the gpfit pylintrc file: msg-template={path}:{module}:{line}: [{msg_id}({symbol}), {obj}] {msg}

It looks like something is wrong with pylint.sh in the gpfit repo. Can you run it locally on your machine?

Can you also update README.md with:

Well I didn't know github would replace the text with strings. You can grab the "unprotected Markdown" code from here:
https://acdl.mit.edu/csi/view/convex engineering/job/CE_gpfit_Push_pylint/badge/
https://acdl.mit.edu/csi/view/convex engineering/job/CE_gpfit_Push_unit_tests/badge/

Thanks @galbramc.

Ah I see
Is there something in the logs (or that I could add to gpfit) that would help me understand what is being installed on the VMs? It makes me think it was using a cached build or something, but maybe that's not possible and I'm just fundamentally misunderstanding something. This would also explain why it can't find artifacts/.
Hmm I thought I had made all of them deterministic now by specifying the seed.
This latest build: https://acdl.mit.edu/csi/job/CE_gpfit_PR_pylint/118/
Line 89 of the pylintrc is
msg-template={path}:{line}: [{msg_id}({symbol}), {obj}] {msg}
which seems to be missing the {module} part -- I will update this.

pylint.sh seems to run fine for me locally:

--------------------------------------------------------------------
Your code has been rated at 10.00/10 (previous run: 10.00/10, +0.00)

Updated the README, thanks!

To expand upon my response to 2 above

Is there something in the logs (or that I could add to gpfit) that would help me understand what is being installed on the VMs?

I am seeing things like:
Checking out Revision 03b40d235eb9a32ccd05b8e69d038a5bcf45ebf6 (origin/pr/105/merge, origin/master)
 > git config core.sparsecheckout # timeout=10
 > git checkout -f 03b40d235eb9a32ccd05b8e69d038a5bcf45ebf6 # timeout=3
Commit message: "Merge 11013ade4147129f04d2cc5ec885cd2beb8f5cf0 into 930b2dc4bdabea0f44edcf02fc418806c41fb96a"

(where 11013 was latest commit of v1 and 930b2 is the latest commit of master)
Successfully installed gpfit-0.2.0

that make me believe the right thing is being built but I'm still very confused why some of the behaviour seems to suggest outdated code is being tested.

It turns out that pylint.sh was getting called in the "unit_tests", which it should not have been. That is fixed.
I don't know what the problem is with the artifacts directiry, but gpkit and gpfit are installed with the commands:
#Install GPkit
git clone --depth 1 https://github.com/convexengineering/gpkit.git $WORKSPACE/gpkit
python $PIP install -v --no-cache-dir -e $WORKSPACE/gpkit

# Install gpfit to run the documentation examples
python $PIP install -e $WORKSPACE/

gpfit is checked out in the $WORKSPACE directory. Does this help?

I added a print statment to show you the current working directory in the script to hopefully help with the artifacts problem


ghc 7.10.2: Fix for linuxbrew

Fix the error:
error while loading shared libraries: libgmp.so.3: cannot open shared object file: No such file or directory

I didn't find this patch was necessary to compile ghc 7.10.2 for me. I had previously created a symlink in $HOMEBREW_PREFIX/lib from libgmp.so.10.2.0 to libgmp.so.3. Perhaps that's why?

Yes, the old symlink is why this patch is not needed.

I'd be curious to know why linking statically against gmp doesn't work on Linux, as it seems to work on Mac.

Linking statically against gmp works fine on linux.
It is the step after making ghc fails because of missing shared library
==> Downloading http://downloads.haskell.org/~ghc/7.6.3/ghc-7.6.3-x86_64-unknown-linux.tar.bz2
==> ./configure --prefix=/tmp/ghc20151102-21968-1q007is/ghc-7.10.2/binary --with-gmp-includes=/home/paulhybryant/.linuxbrew/Cellar/ghc/7.10.2/libexec/integer-gmp/include --with-gmp-libraries=/home/paulhybryant/.linuxbrew/Cellar/ghc/7.10.2/libexec/integer-gmp
Last 15 lines from /home/paulhybryant/.cache/Homebrew/Logs/ghc/05.configure:
2015-11-02 16:49:02 +0800
./configure
--prefix=/tmp/ghc20151102-21968-1q007is/ghc-7.10.2/binary
--with-gmp-includes=/home/paulhybryant/.linuxbrew/Cellar/ghc/7.10.2/libexec/integer-gmp/include
--with-gmp-libraries=/home/paulhybryant/.linuxbrew/Cellar/ghc/7.10.2/libexec/integer-gmp/lib
--with-ld=ld
--with-gcc=/home/paulhybryant/.linuxbrew/bin/ygcc-5
configure: WARNING: unrecognized options: --with-ld
checking for path to top of build tree... utils/ghc-pwd/dist-install/build/tmp/ghc-pwd: error while loading shared libraries: libgmp.so.3: cannot open shared object file: No such file or directory
configure: error: cannot determine current directory
/home/paulhybryant/.linuxbrew/Library/Homebrew/debrew.rb:11:in `raise'
BuildError: Failed executing: ./configure --prefix=/tmp/ghc20151102-21968-1q007is/ghc-7.10.2/binary --with-gmp-includes=/home/paulhybryant/.linuxbrew/Cellar/ghc/7.10.2/libexec/integer-gmp/include --with-gmp-libraries=/home/paulhybryant/.linuxbrew/Cellar/ghc/7.10.2/libexec/integer-gmp/lib --with-ld=ld --with-gcc=/home/paulhybryant/.linuxbrew/bin/gcc-5

I'm on vacation until the new year. Please bump this thread after then if you haven't heard from me.

This PR no longer works for some reason. Now it complains about not being able to link to gmp

I just pushed ghc 7.10.3b. Any luck with that?

ghc install would still fail without the change here, complaining about not being able to find libgmp.so.3
With the change in this PR it now works.
Thanks,

And the change for linking the dynamic library should be
ln_sf "#{gmp}/lib/libgmp.so.10.3.0", "#{gmp}/lib/libgmp.so.3"
I will update the PR.

How about
ln_s Dir["#{gmp}/lib/libgmp.so.*"], "#{gmp}/lib/libgmp.so.3"

Thanks, I am not very familiar with ruby.
What does Dir["#{gmp}/lib/libgmp.so.*"] mean? Which file will libgmp.so.3 point to?

Dir[] is the globbing operator, just like shell globbing. http://ruby-doc.org/core-2.2.3/Dir.html#method-c-5B-5D

Thanks, updated the PR.

This patch would have the side effect of making ghc dynamically linked against libgmp, which is different than it's done on Mac. I'm hesitant to make this change for this reason.
To run the command ghc-pwd we need a gmp shared library named libgmp.so.3 somewhere in the library search path. We can get the shared library by installing the formula gmp, so I suggest adding the following line instead of removing "--disable-shared".
depends_on "gmp" => :build unless OS.mac?

In def install
if OS.linux?
  # Fix error while loading shared libraries: libgmp.so.3
  ln_s Formula["gmp"].lib/"libgmp.so", "#{gmp}/lib/libgmp.so.3"
  ENV.prepend_path "LD_LIBRARY_PATH", "#{gmp}/lib"
end

An alternative solution would be for the gmp formula to provide the symlink from libgmp.so.10 to libgmp.so.3.

Can you confirm that the dependency on libgmp.so.3 is only a build time dependency? That is, none of the executables installed by ghc depend on it?

Fixed. Thanks for your contribution, Paul. Sorry for the extended delay.

The fix that I went with is:
diff --git a/Library/Formula/ghc.rb b/Library/Formula/ghc.rb
index 2eec23b..b01e24b 100644
--- a/Library/Formula/ghc.rb
+++ b/Library/Formula/ghc.rb
@@ -14,6 +14,8 @@ class Ghc < Formula
   deprecated_option "tests" => "with-test"
   deprecated_option "with-tests" => "with-test"

+  depends_on "gmp" => :build if OS.linux?
+
   resource "gmp" do
     url "http://ftpmirror.gnu.org/gmp/gmp-6.1.0.tar.bz2"
     mirror "https://gmplib.org/download/gmp/gmp-6.1.0.tar.bz2"
@@ -81,14 +83,23 @@ def install
       args << "--with-gcc-4.2=#{ENV.cc}"
     end

+    if OS.linux?
+      # Fix error while loading shared libraries: libgmp.so.3
+      ln_s Formula["gmp"].lib/"libgmp.so", gmp/"lib/libgmp.so.3"
+      ENV.prepend_path "LD_LIBRARY_PATH", gmp/"lib"
+      # Fix /usr/bin/ld: cannot find -lgmp
+      ENV.prepend_path "LIBRARY_PATH", gmp/"lib"
+    end
+
     resource("binary").stage do
       # Change the dynamic linker and RPATH of the binary executables.
       if OS.linux? && Formula["glibc"].installed?

Thank you for the fix and maintaining linuxbrew!

No worries. I'm glad that it's useful.


Register zai.is-not-a.dev

Requirements

[x] You have completed your website.
[x] The website is reachable.
[x] The CNAME record doesn't contain https:// or /.
[x] There is sufficient information at the owner field.
[x] There is no NS Records (Enforced as of Sepetember 4th, 2024)

Description
personal portfolio website
Link to Website
https://zai-kun.vercel.app

I have read the CLA Document and I hereby sign the CLA


Unclear how to use Node sockets to speak to the language server

I have the TypeScript code below that tries to connect and speak to the example's language server. However, nothing happens after I send the initialization message over the wire. Am I doing something wrong? I think it would be very helpful if this kind of sample code could be included in the repository.
import * as net from "net"
import * as child_process from "child_process"

let socket = new net.Socket();
socket.on("data", (message) => {
    console.log(message.toString());
});

socket.on("connect", () => {
	console.log("connected");
	initialize();
});

child_process.spawn("node", [ "out/server.js", "--socket=3000" ]);
socket.connect(3000);

let messageId = 1;

function send(method: string, params: object) {
    let message = {
        jsonrpc: "2.0",
        id: messageId++,
        method: method,
        params: params
    };
    let json = JSON.stringify(message);
    let headers = "Content-Length: " + json.length + "\r\n\r\n";
    socket.write(headers, "ASCII");
    socket.write(json, "UTF-8");
}

function initialize() {
    send("initialize", {
        rootPath: process.cwd(),
        processId: process.pid,
        capabilities: {
            /* ... */
        }
    });
}

This got fixed with 3.5.0-next.1

There are now functions to create a socket transport for both the client and server side.


Usability/compatibility changes (more info inside)

Hi!
Thanks for an excellent library!  I've made a few usability tweaks to make this more friendly for others:

Makefile -- this makes it more clearly defined how to do thing like build and test.  It also references NPM binaries that are installed in the project so that globally installed modules are not required (mocha, gulp)
component.json -- this allows use of the module within Duo.
Additionally, I tweaked the browserify build options in the gulpfile so that the module is exported UMD style -- this was another requirement for Duo.  The changes should be backwards compatible.

Good improvements!
The only thing that is confuse for me is the Makefile. I don't know If I want to mixin Make with the project, because I identify Make command more with C/C++ projects, and, on the other hand, npm support some userful scripts for do it.
What do you think?

npm scripts could be good.  I lean to Makefiles for everything (including node projects) so that habit tends to bleed over into other things.  If you don't want to introduce a Makefile, that's totally fine -- I can update the PR to use npm scripts instead.

I think that extending package.json is enough. Putting another file for do the same just cause confusion.
If you can refactor the PR I will merged :-)

PR updated.  While I understand why you have a shell script to run your tests, I would strongly suggest moving that command inside the package.json and referencing the local mocha binary that is installed with npm i.  This would allow someone to run your tests without having mocha installed globally.

Good point, I updated the test file :+1:
I tried locally dist/acho.js and doesn't work. Browserify doesn't create correctly the package if you use standalone flag. Can you check it? If I delete the flags works fine, but I don't know if you has put the flag for something...

My apologies, example.html needs to change.  The standalone flag tells browserify to create a UMD compatible package.  When loaded in an environment that has require (like Node), the module can be required, as expected.  When loaded in a browser (where require doesn't natively exist), the module is leaked to the page as Acho.  Therefore, all you need to do in a browser is to create a new Acho().  No require is needed.
More info can be found here: http://www.forbeslindesay.co.uk/post/46324645400/standalone-browserify-builds.
I've updated the build script (to properly name the module) and example.html in the PR.

so now I don't understand why you want to expose as global object :sa:
It's more secure juse instance when you need, and later use it. The problem that I'm trying to avoid using require declaration is have too many global objects in the frontend.

Typically you don't want a global object.  I completely agree with you.  But that's not how example.html is written.  It's written to directly use the module from a global standpoint.  And that totally works.  If you were to use browserify to package another system that depends on Acho, it would not be leaked globally.  It would be self-contained within the scope of that application.
Take a look at this example I just whipped up:
http://embed.plnkr.co/wjKwxApGtHQpKrzcX5yX/preview
Notice that when using Acho as a component of larger application, an Acho global **does not ** get leaked to the page.  The global scope does not get dirtied up.  That's why a UMD module is option -- it automatically does the correct thing based on how it is used.  If you can require it, it can be required.  If you cannot, it leaks a global.  This allows for the most compatibility with existing systems.
More UMD info can be found:
http://davidbcalhoun.com/2014/what-is-amd-commonjs-and-umd/

Any thoughts here?

Really I don't use UMD or AMD because packaged the modules using browserify and require in the browser is extremely simple, but I understand you.
The only thing is that I compiled again the module using gulp and is expose as global object, can you check?
If you need to capitalize the name use titleize, is more clear for maintain the package in the future.
The only thing is that in the example is a global object,

Compiling the module and requiring it directly in the browser will always result in a global object.
If you require the module in a builder (webpack, duo, browserify), no global object will exist.
I'm sorry these changes go against the spirit of what you're trying to accomplish.  If this doesn't work for you, I can fork the project and make my own version.

No, I prefer adapt my style and merge the PR, but I need to understand all workflows.
Please, Can you write a little of documentation in README.md about how to load the module in the browser for the different module loaders? to make it all clear

Absolutely.  I'll get the README.md updated ASAP.

README.mdupdated and switched to titleize in the gulp file.

thanks! :-)

@clocklear do you think that standalone setting in gulpfile is just because Acho is a class?
I have a generator for git repositories and I'm thinking to update the code for the next time, but I don't sure if is a good point uses always titleize for prefix the module shortcut.

Yes.  I titleized the module name since it gets used as a class -- by convention.


[Bug] Creating vector indexes for MongoDB Atlas assumes data is stored in pascal casing

Context / Scenario
If CamelCaseElementNameConvention is enabled, data fields are stored in the collections in camel case despite what the property names are. However, the current implementation has index field references targeting field names in Pascal casing. This breaks the vector definition because it's targeting fields that do not exist.
What happened?
Our data is stored as:
{
  "index": "default",
  "embedding": [],
  "tags": [
    {
      "key": "",
      "values": ""
    }
  ]
}

But the index was created as:
"fields": [
  {
    "numDimensions": 1536,
    "path": "Embedding",
    "similarity": "cosine",
    "type": "vector"
  },
  {
    "path": "Index",
    "type": "filter"
  },
 {
    "path": "Tags.Key",
    "type": "filter"
  },
 {
    "path": "Tags.Values",
    "type": "filter"
  }
]

Importance
a fix would make my life easier
Platform, Language, Versions
C#
NET 8.0
<EMAIL_ADDRESS>MongoDB Atlas
Relevant log output
Command aggregate failed: PlainExecutor error during aggregation :: caused by :: embedding is not indexed as vector.

help is welcome if someone can look into this. Perhaps this is one of those problems that will be addressed once we've migrated to the new SK memory classes (end of 2024).

I can address the issue. I found out from the code that the names of the
targeted fields are hard-coded. I can change that.
How can I submit a PR?
Thanks,
Wale
On Wed, Aug 28, 2024, 12:43 AM Devis Lucato @.***>
wrote:

help is welcome if someone can look into this. Perhaps this is one of
those problems that will be addressed once we've migrated to the new SK
memory classes (end of 2024).
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/kernel-memory/issues/758#issuecomment-2314296296,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADCMMZX4ASEHQRJAZGEYISLZTVIPFAVCNFSM6AAAAABNEYRZ76VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMJUGI4TMMRZGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

How can I submit a PR? Thanks, Wale

@walexee the process is described here https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/creating-a-pull-request - usually that involves:

fork KM repo, which will generate a new repo under your user, e.g. https://github.com/walexee/kernel-memory
check out the repo on your local machine
create a new branch, e.g. "git checkout -b fix758"
make the code changes locally on your machine, plus tests
commit changes
push changes to GitHub (ie push the new branch to your fork)
create pull request - this link might work https://github.com/microsoft/kernel-memory/compare/main...walexee:kernel-memory:fix758 - otherwise see the docs

Thanks Devis. I'll submit a PR ASAP.
On Wed, Aug 28, 2024, 3:34 PM Devis Lucato @.***> wrote:

How can I submit a PR? Thanks, Wale
@walexee https://github.com/walexee the process is described here
https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/creating-a-pull-request

usually that involves:

fork KM repo, which will generate a new repo under your user, e.g.
https://github.com/walexee/kernel-memory
check out the repo on your local machine
create a new branch, e.g. "git checkout -b fix758"
make the code changes locally on your machine, plus tests
commit changes
push changes to GitHub (ie push the new branch to your fork)
create pull request - this link might work
main...walexee:kernel-memory:fix758
https://github.com/microsoft/kernel-memory/compare/main...walexee:kernel-memory:fix758
otherwise see the docs

—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/kernel-memory/issues/758#issuecomment-2316107892,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADCMMZTG7NJYCWZ4VMSSEM3ZTYQ57AVCNFSM6AAAAABNEYRZ76VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMJWGEYDOOBZGI
.
You are receiving this because you were mentioned.Message ID:
@.***>

Fixed - https://github.com/microsoft/kernel-memory/pull/762


Rework docs

This is the current WIP on the docs work. cc @sashavor.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Awesome! I'll keep working on things on my end as well :rocket:


Progressive Import/Export

Allows importing and exporting object graphs larger than can be stored in the shared buffer.

Needs rebase after merge of #14.

Have improved the import function along the lines we discussed so that it now reads as much as it can from the buffer before returning, rather than returning after every object (as my initial approach did).


Make release of examples

It would be useful to have a release, including each of the models in separate zip files, so that we have versions of the models that will work with older versions of Health-GPS.
Depends on #24.

Done: https://github.com/imperialCHEPI/healthgps-examples/releases/tag/20240907


Removed URL versioning.

This PR removes the URL based versioning mechanism, and is the first step towards a content-type negotiation schema discussed in #526.
Since URL versioning is not part of the schema, and we are planning on replacing this with a header-based approach, it is a good idea to remove this URL based approach now rather than later. Since this is the reference implementation, other implemetors might assume that URL based API versioning is required.

+1

+1 :cry:

Thanks @dcolligan --- sorry to nuke all that work!


Warning Level 4 Issue

Hi, your sources doesn't compile with Warning Level4 (/W4) (Visual Studio 2015). And warnings are treated as errors here what shouldn't be so unusal. Would like to see this fixed. It's only that line:
Traits::assign(*this->pptr(), ch);

1>...\beast\include\beast/core/detail/static_ostream.hpp(60): warning C4244: 'argument': conversion from 'beast::detail::static_ostream_buffer::int_type' to 'const std::char_traits::_Elem', possible loss of data

willfix

Ok, I was a bit to optimistic with that one line. There are dozens of additional warnings now because I had not referenced any of beasts classes so far :). Like e.g.:

...\beast\include\beast/zlib/detail/deflate_stream.hpp(814): warning C4244: '=': conversion from 'int' to 'uint8_t', possible loss of data
...\beast\include\beast/zlib/detail/deflate_stream.hpp(2629): warning C4244: 'argument': conversion from 'beast::zlib::uInt' to 'uint8_t', possible loss of data
...\beast\include\beast/zlib/detail/deflate_stream.hpp(1637): warning C4244: '+=': conversion from 'int' to 'uint16_t', possible loss of data
and many others...

Maybe you can check it with Warning Level4 (/W4) activated on your setup.

I will try to fix it. Note that the build scripts already enable /W4 but I had to disable some of the integer warnings because of the zlib code.

I think all the warnings are fixed in this branch, can you please confirm? https://github.com/vinniefalco/Beast/pull/527

Should be fixed in v65

Thanks for the update. No warnings had been shown so far.


Logs by year/month/day

What steps will reproduce the problem?
Not really a problem, mostly feature request, as I did not find any requests for it.
What is the expected result?
Having common logs folder in root of the web-application with the following structure:
common
backend
frontend
logs
  - 2017
  - - 07
  - - - 21
  - - - - categoryName.log

Maybe it could be another option in logs config such as the following:
'log' => [
    'traceLevel' => 0,
    'dateTimeTreeView' => true,

Or maybe dateTimeTreeView can be only attached to specific targets.
I started something already, but it is pretty raw:
<?php
/**
     * Possibility to ovverride file name from other controllers
     *
     * @var string
     */
    public $logFileName = 'backend';

    /**
     * Log starting sequence
     *
     * @var string
     */
    public $logPre = '----> ';

    /**
     * Array list of restricted actions.
     * For these actions, data in the _POST would not be recorded
     * @var array
     */
    public $logRestrictedActions = [];

    /**
     * Before controller runs
     *
     * @param \yii\base\Action $action
     * @return bool
     */
    public function beforeAction($action)
    {

        /**
         * Generate path
         *
         * @app by default is the root of the project according to the side (backend/frontend).
         * If controller is placed within backend, it will recognize it
         */
        $path = \Yii::getAlias('@app') . '/../log/';
        $year = date('Y') . '/';
        $month = date('m') . '/';
        $day = date('d');

        /**
         * Make the path. It consists of year, month and day
         */
        $path = $path . $year . $month . $day;

        /**
         * Create derictories
         *
         * http://www.yiiframework.com/doc-2.0/yii-helpers-basefilehelper.html#createDirectory()-detail
         */
        BaseFileHelper::createDirectory($path, 0777, true);

        /**
         * Save the file in the directory of $path + name of the log file + name of the controller ($this->id)
         *
         * @var $path
         */

        Yii::$app->log->targets[0]->logFile = \Yii::getAlias($path . '/' . $this->logFileName . '_' . $this->id . '.log');

        ...

        return parent::beforeAction($action);
    }

This is custom controller that extends yii\web\Controller.
Now I can create logs in thefollowing format "frontend_{controllerName}.log".
This is not the best solution, but I believe it will be useful to have for some projects.

Q
A

Yii version
2.0.12

PHP version
5.5.9

Operating system
Ubuntu 14.04

Interesting idea and could be actually handy if you read logs without any aggregation tool. Not a priority though.

Did you try:
        'log' => [
            'targets' => [
                [
                    'class' => 'yii\log\FileTarget',
                    'logFile' => '@runtime/logs/' . date('Y') . '/' . date('m') . '/' . date('d') . '/app.log',
                ],
            ],
        ],

?

@rob006 , yeah, but in this case you need to now for sure that this folder stucture exists, othertwise it will fail.
This is first thing. And second is that this is not handy solution if you have multiple targets, etc.

yeah, but in this case you need to now for sure that this folder stucture exists, othertwise it will fail.

Are you sure about that? Directory structure should be created by target: https://github.com/yiisoft/yii2/blob/master/framework/log/FileTarget.php#L88-L91

This is first thing. And second is that this is not handy solution if you have multiple targets, etc.

You could create own target which extends FileTarget and only adjusts logFile property.

@rob006 well, I thought that it does not create new directories, but as I noticed now, it does work.
I tried to extend FileTarget class last time and it did not work well for me. I wil try to do it again and will write here the reuslt.
Thanks for suggestion.

Closing since it works well.


Add provedState field to graphQL account

Enables retrieving the provedState flag from an account via graphql. This is needed to support the currently unimplemented isProved precondition in snarkyjs.
note: this is stacked on top of https://github.com/MinaProtocol/mina/pull/12166 (bringing rampup up to date with develop)

!approved-for-mainnet

!ci-build-me

!approved-for-mainnet


Automated cherry pick of #2317: Fix ovs group id conflict when dual stack is enabled

Cherry pick of #2317 on release-1.1.
#2317: Fix ovs group id conflict when dual stack is enabled
For details on the cherry pick process, see the cherry pick requests page.

/test-all

Codecov Report

Merging #2323 (94e4bc4) into release-1.1 (9af36f5) will decrease coverage by 20.15%.
The diff coverage is 0.00%.

@@               Coverage Diff                @@
##           release-1.1    #2323       +/-   ##
================================================
- Coverage        61.52%   41.37%   -20.16%
================================================
  Files              278      139      -139
  Lines            21189    17240     -3949
================================================
- Hits             13037     7133     -5904
- Misses            6794     9479     +2685
+ Partials          1358      628      -730

Flag
Coverage Δ

kind-e2e-tests
?

unit-tests
41.37% <0.00%> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/agent/proxy/proxier.go
45.68% <0.00%> (-18.22%)
:arrow_down:

pkg/util/k8s/name.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/agent/agent_linux.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/ovs/openflow/default.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/agent/config/node_config.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/apis/controlplane/helper.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/agent/cniserver/pod_configuration_linux.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/apis/controlplane/v1beta1/register.go
0.00% <0.00%> (-92.86%)
:arrow_down:

pkg/ovs/openflow/logs.go
9.52% <0.00%> (-90.48%)
:arrow_down:

pkg/apis/controlplane/register.go
0.00% <0.00%> (-90.00%)
:arrow_down:

... and 236 more

/skip-networkpolicy
https://jenkins.antrea-ci.rocks/job/antrea-networkpolicy-for-pull-request/2319/ actually succeeded but failed to update result to Github


[BUG]: Drizzle-kit Introspection Failed to Declare Default Value in SQL

What version of drizzle-orm are you using?
0.29.4
What version of drizzle-kit are you using?
0.20.14
Describe the Bug
Table introspection fails to find a type declaration for a default value.
 ref_icon character varying(255) COLLATE pg_catalog."default" DEFAULT 'activity'::character varying,

Automatic SQL import does not occur and it imports without declaring it to be raw SQL.
import { sql } from 'drizzle-orm';
As a result several errors are created in the imported schema.
refIcon: varchar("ref_icon", { length: 255}).default('activity'::character varying`),

Expected behavior
Introspection must import SQL and make the statement as evidenced in the Documentation:
refIcon: varchar("ref_icon", { length: 255 }).default(
sql'activity'::character varying,
),
https://orm.drizzle.team/docs/indexes-constraints
Environment & setup
Node.js v20.10.0

Exactly the issue I got today too.

I have the same issue as well


Zuliprc cannot be read from other directory.

This will be a problem since we allow them to specify path.

Taking a shot at it @zulipbot claim.

to specify the path of the .zuliprc is only possible in the sample app. It's not a feature of the API library as such.


Errors on Installation

Getting two errors trying to deploy Apex Trigger for CampaignMember

dlrs_CampaignMemberTest.testTrigger System.DmlException: Insert failed. First exception on row 0; first error: MALFORMED_ID, Invalid campaign id: null: [CampaignId] Class.dlrs.RollupService.testHandler: line 282, column 1 Class.dlrs_CampaignMemberTest.testTrigger: line 11, column 1

dlrs_CampaignMemberTrigger:Test coverage of selected Apex Trigger is 0%, at least 75% test coverage is required

Hiya, take a look through this issue for test code to use to avoid this. https://github.com/afawcett/declarative-lookup-rollup-summaries/issues/197


Support bybit v5 api

https://bybit-exchange.github.io/docs/v5/guide

V1 , as implemented now in the XCHANGE lib is not supported anymore.
Message of Bybit on Telegram 4th July (https://t.me/s/Bybit_API_Announcements?q=V1)
[IMPORTANT] Dear Bybit trader,
We offically inform you that we will suspend the use of our old version openAPI service. This decision was made in order to reduce unnecessary maintenance and cost, and also to ensure the stability our platform. You can migrate to V5 API, which has the full ability.

Can do that if there is still need for it

V5 will be great ;-)  or bybit DEX :-) even better

Feel free to contact me to discuss details

your  abraxas.ch address ?

linkedin

Hi, i'm interested. Are you working on it? Can I help you?

I'm looking into it yes :-) I have some succes so far (I'm now in the process to adapt The trade logic)  Not sure how far it will tale me inside the code... Help is always good :-) --> Discord ? Telegram?
+++ OPEN TRADE >Trade{type=BUY, index=60008, price=29415.64, amount=100.0}
++++ CLOSE TRADE >Trade{type=SELL, index=60039, price=29407.95, amount=100.0}
++++ OPEN TRADE >Trade{type=BUY, index=60101, price=29436.49, amount=100.0}
++++ CLOSE TRADE >Trade{type=SELL, index=60130, price=29462.0, amount=100.0}
++++ OPEN TRADE >Trade{type=BUY, index=60162, price=29436.51, amount=100.0}
++++ OPEN TRADE >Trade{type=SELL, index=60175, price=29431.88, amount=100.0}
++++ OPEN TRADE >Trade{type=BUY, index=60221, price=29430.07, amount=100.0}
++++ OPEN TRADE >Trade{type=SELL, index=60232, price=29428.48, amount=100.0}
++++ OPEN TRADE >Trade{type=BUY, index=60253, price=29413.0, amount=100.0}
++++ OPEN TRADE >Trade{type=SELL, index=60263, price=29412.2, amount=100.0}
++++ OPEN TRADE >Trade{type=BUY, index=60319, price=29381.15, amount=100.0

Hi I'm was looking into this ... :-) But seen the changes on EU rules I'm looking now to de DEX solution of Bybit ...

Ah, ok. I need the classic version. Did you fork it? Can I resume your work? Otherwise, I can create a new fork and start looking into it.

I forked and I'm actively working on it. Almost done. Give me few days to do end-to-end tests.

Oh wow, happy to help on tests
On Sun, Sep 10, 2023, 9:09 PM Damiano Derin @.***>
wrote:

I forked and I'm actively working on it. Almost done. Give me few days to
do end-to-end tests.
—
Reply to this email directly, view it on GitHub
https://github.com/knowm/XChange/issues/4731#issuecomment-1712925988,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABBUO2RC3E6XSIQ2R5HLWFTXZYM7ZANCNFSM6AAAAAAZOSCBME
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Hi, I opened the PR #4771 , but I have problems with placing orders and UNIFIED account mode. This evening I'll go on with end-to-end tests, but in the meanwhile, if you would like to try with your APIs, it could be great. Thanks in advance.

Hi, now it works for me. Please, can you review it? PR: #4771

It has been implemented, and I confirm that the Spot functionality is operational.
There is an issue with executing Futures, I've filed a new bug report here: https://github.com/knowm/XChange/issues/4822

close please


Download rate is steady, then stalls.

extension of previous issue.

I've tried spamming keys and enter and it doesn't resume it. Pressed ctrl-c to cancel. Here is the trace back: https://pastebin.com/FRqx2HsG
(Many errors related to multiprocessing library. Error most likely related to multi-threading stuff)
EDIT: This also occurs earlier if there are multiple terminals open running spot-dl. OS related issue?

EDIT: Mostly works when i spam enter and run it in admin mode. Still might wanna fix this somehow idk.


Cache access tokens in account momento to prevent throttling

This PR fixes #22403
When working with MSAL.js, access token always needs to be force refreshed due to issue in library, which leads to MSAL.NET throwing errors related to throttling of access token requests.
This PR associates active access tokens with account momento which will be available in memory.
Benefits of this PR:

Only 1 token request is made to AAD for a resource during the period it stays active.
As soon as a token is determined to get expired in next 2 mins, it will be refreshed and added back to momento.

[New] Tokens are encrypted before saving to momento

Tokens are encrypted with same encryption algorithm and keys as used by MSAL Cache Plugin, before saving to momento.
A new API 'encryptAccountSecurityToken' is exposed from azurecore provider to be able to encrypt access tokens before saving them with account information.
Decryption of access tokens from account momento is only done within the azure core extension,

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 40 (0.0%)  changed or added relevant lines in 5 files are covered.
6 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.04%) to 41.621%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

extensions/azurecore/src/account-provider/auths/azureDeviceCode.ts
0
1
0.0%

extensions/azurecore/src/account-provider/auths/azureAuth.ts
0
8
0.0%

extensions/azurecore/src/account-provider/azureAccountProvider.ts
0
9
0.0%

src/sql/workbench/api/common/extHostAccountManagement.ts
0
11
0.0%

src/sql/workbench/services/accountManagement/browser/accountManagementService.ts
0
11
0.0%

Files with Coverage Reduction
New Missed Lines
%

extensions/azurecore/src/account-provider/auths/azureAuth.ts
1
29.07%

extensions/azurecore/src/account-provider/azureAccountProvider.ts
1
0%

extensions/notebook/src/jupyter/serverInstance.ts
4
75.5%

Totals

Change from base Build<PHONE_NUMBER>:
-0.04%

Covered Lines:
30346

Relevant Lines:
67482

💛  - Coveralls

although this whole thing is really starting to feel like it should be removed from the extensibility API and just made as a contract directly between ADS core and azurecore.

The reason why we need to expose the API is to be able to encrypt the token before adding to momento - and so will also be done by other extensions that work with Azure accounts and get tokens from Azure core extensions to introduce caching. Since account object reference is not shared between azurecore extension and callers of account provider APIs - we cannot do this all within Azure core extension, and calls of Azure core APIs when passing accounts, need to keep their account updated with encrypted token to take benefit of cache design.

@cheenamalhotra  what is the lifespan of a token by default?

@cheenamalhotra  what is the lifespan of a token by default?

Token lifetime can range from 10 mins to 24 hours, by default it's 1 hour. We do receive 24 hr long tokens when acquiring for Managed Service Identity against Sql resources (something that's in plan to be supported in near future).
Configurable Token Lifetimes

Will proceed to merge as we're not introducing new extensibility APIs now. @alanrenmsft


Dockerize app and possibly add k8s support

After these are completed:

[ ] #23
[x] #27
[x] #85
[ ] #96
[x] #97
[x] #99
[ ] #100

Dockerize project and if wanted add k8s support
Note: check before refactoring for the latest changes and future developments and review what is still needed

Please follow the README.md instructions for pushing the image in github container registry. The purpose is that the image will be available from the 'Packages' section of repository

Planned Actions:

[ ] Waiting for #101 related actions
[ ] Adding k8s configuration files and instructions for a microK8S installation

PS: Nice work with the available container package

Proposal:
I suggest to close this issue for now as kubernetes configuration could also be done in next release

Done


ARROW-10154: [C++][Python][Gandiva][Docs] Start Python / C++ Gandiva Docs

This PR adds Gandiva user guides and API docs for C++ and Python. This includes a list of all available functions, shown within the C++ user guide.
You can view rendered previews at:

C++ User Guide
C++ API docs
Python User Guide
Python API docs

@amol- It would be nice if you could give this a review so as to move it forward.
@wjones127 Sorry for the delay. Can you please rebase from master and fix the (small) conflict?

I have rebased to the new docs theme. The links in the description have been updated as well.
@projjal, @jvictorhuguenin, @jpedroantunes, @anthonylouisbsb Any feedback you have on the docs would be much appreciated. (See the links in PR description.)

@amol- I started refactoring into a custom directive, but I found myself thinking the function list really wouldn't be that useful without clarifications of most of the functions. There's a lot of small details that are only described in unit tests or code comments. So I instead created hand-written documentation similar to the Available Functions section in the Compute documentation.

@wjones127 Hi Will, may I ask why is this PR closed? I think it's good enough for new users to start using Gandiva, at least it was for me.

I was having a hard time getting reviews on my Gandiva PR, plus I had changed jobs and wasn't planning on using Gandiva anymore. There aren't very many active Gandiva maintainers (most activity I've seen if a few people adding functions) and almost none of them are familiar with PyArrow, so it was difficult to get reviews and merged. After a while, I was reticent to document a submodule that doesn't have people committed to maintain it. So I closed it.
If the situation changes and there's more maintainers on Gandiva and the PyArrow bindings specifically, we should definitely reopen this.

@wjones127

I was having a hard time getting reviews on my Gandiva PR, plus I had changed jobs and wasn't planning on using Gandiva anymore. There aren't very many active Gandiva maintainers (most activity I've seen if a few people adding functions) and almost none of them are familiar with PyArrow, so it was difficult to get reviews and merged. After a while, I was reticent to document a submodule that doesn't have people committed to maintain it. So I closed it.
If the situation changes and there's more maintainers on Gandiva and the PyArrow bindings specifically, we should definitely reopen this.

From what you've said (aligned with my impression), I think it might make sense to spin off Gandiva from the Apache Arrow.

From what you've said (aligned with my impression), I think it might make sense to spin off Gandiva from the Apache Arrow.

I'm not sure I understand how spinning off would help. Spinning off would require more maintenance work than keeping and working within the repo.
If there are people / companies actively using Gandiva in PyArrow, hopefully some of them can come forward and take on active maintenance.

From what you've said (aligned with my impression), I think it might make sense to spin off Gandiva from the Apache Arrow.

I'm not sure I understand how spinning off would help. Spinning off would require more maintenance work than keeping and working within the repo.
If there are people / companies actively using Gandiva in PyArrow, hopefully some of them can come forward and take on active maintenance.

Hi Will, would you mind merging the C++ docs to master first? They seem complete already except for the newly added functions. I am currently maintaining Gandiva and plan to improve its document based on your work.
As for Gandiva in PyArrow, my company has long term plans to use it but not in the near future. Meanwhile I'll try my best to maintain it if needed.

Sure @js8544, I'll reopen a PR later today with the C++ user guide.


stopping MM

Hi,
I have a question regarding stopping MagicMirror via shell or terminal command. So if I have started MM with "npm start" is there also a possibility to stop it with a command (not only with shortcut).
Rational: I want to stop working MM with a Python code.
Thx in advance.
Gürkan

i wrote a script to do that..
needs the moreutils package installed
sudo apt-get install moreutils
I call this
clear_node
have to make it executable
chmod +x clear_node
#!/bin/bash

ps -ef | grep pocket | grep -v "grep"  | awk '{print $2}'  | ifne xargs sudo kill -9
ps -ef | grep node | grep -v "grep"  | awk '{print $2}' | ifne xargs sudo kill -9
ps -ef | grep chromium-browser | grep -v "grep"  | awk '{print $2}' | ifne xargs sudo kill -9
sudo killall  chromium-browser
sudo killall  /usr/lib/chromium-browser/chromium-browser
sudo killall electron

thanks a lot MichMich. And thanks for your amazing MagicMirror Project!
BlueMail for Android herunterladen
Am 16. Feb. 2020, 21:47, um 21:47, sam detweiler<EMAIL_ADDRESS>schrieb:

i wrote a script to do that..
needs the moreutils package installed
sudo apt-get install moreutils
I call this
clear_node
have to make it executable
chmod +x clear_node
#!/bin/bash

ps -ef | grep pocket | grep -v "grep"  | awk '{print $2}'  | ifne xargs
sudo kill -9
ps -ef | grep node | grep -v "grep"  | awk '{print $2}' | ifne xargs
sudo kill -9
ps -ef | grep chromium-browser | grep -v "grep"  | awk '{print $2}' |
ifne xargs sudo kill -9
sudo killall  chromium-browser
sudo killall  /usr/lib/chromium-browser/chromium-browser
sudo killall electron

--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/MichMich/MagicMirror/issues/1931#issuecomment-586750891

this can be closed


prometheus-bind-exporter: 0.4.0 -> 0.5.0

Motivation for this change
I need this fix from upstream.
Things done
I tested the new package version on my own infra. Works as expected.

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 154457 run on x86_64-linux 1

1 package built:

prometheus-bind-exporter


[HotFix][zeta]: Fix loss job (#6759)

Purpose of this pull request
fix loss job #6759

Does this PR introduce any user-facing change?
no

How was this patch tested?
test case

Check list

[x] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[x] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs
[x] If you are contributing the connector code, please check that the following files are updated:

Update change log that in connector document. For more details you can refer to connector-v2
Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist

[x] Update the release-note.

Thanks @drgonroot for fix this, could you follow the guide to open ci on your fork repository? https://github.com/apache/seatunnel/pull/6807/checks?check_run_id=24679528654


"Import from current" works with with old file for Stellaris 2.4+

The Import from current function imports from the unused settings.txt file for Stellaris 2.4+ instead of the current json based file. I noticed this when I tried to convert my list from the 0.7.1 of Paradoxos Mod Manager using the Stellaris load order as a go between.

Thanks
I will look that too
As "workaround" you can launch 0.7.1 and 0.8.0 PMM for legacy launcher (since it use settings.txt) and apply with 0.7.1 and import from current with 0.8.0
PS: Good idea to migrate lists 👌


PDK kong.response.exit bug, message body not properly set

Summary
https://github.com/Optum/kong-cluster-drain
https://github.com/Optum/kong-cluster-drain/commit/ce6b8e6dd112e7323e971c87d6b7a9827b716ba8
To install the 0.14.x working version of plugin of it just do luarocks install kong-cluster-drain
To install it from src(for 1.0 latest changes not on luarocks yet) I have been doing this for alpine linux kong:
RUN git clone https://github.com/Optum/kong-cluster-drain.git /usr/local/share/lua/5.1/kong/plugins/kong-cluster-drain &&\
    find /usr/local/share/lua/5.1/kong/plugins/kong-cluster-drain -type f -maxdepth 1 -delete &&\
    cd /usr/local/share/lua/5.1/kong/plugins/kong-cluster-drain &&\
    mv src/* . &&\
rm -r src

So what happens is that for:
kong.response.exit(503, { message = "Scheduled Maintenance" })

Which should throw the 503 status I am used to, but also my own custom HTTP Body message as per:
https://docs.konghq.com/1.0.x/pdk/kong.response/#kongresponseexitstatus-body-headers
In my testing of the plugin I see this is what my route that has the plugin enabled on it throws:
"The upstream server is currently unavailable" , which is a standard of Kong, so somewhere the message body I set is getting overridden I imagine with the default that is generally set by a 503 status.
Additional Details & Logs

Kong version 1.0

@jeremyjpj0916 I think you should remove this:
https://github.com/Optum/kong-cluster-drain/blob/ce6b8e6dd112e7323e971c87d6b7a9827b716ba8/src/handler.lua#L27-L36
What do you think?

@bungle Ohhhh, very likely is the culprit. I see you refactored out all that bulk from the  original request termination plugin I mirrored this off of too:
https://github.com/Kong/kong/commit/03d2e98a7f8116424be8f037985a72b9d18445b4#diff-3df61ddd3aa9d40123806ea9feb03143
Will give that a go and close up the Github issue if its that legacy bit causing the issue.
Thanks!

Validated and confirmed that was my problem, should have caught that before raising issue. Sorry about that. Thanks!


Add connector metadata generator util

Purpose
Add connector metadata generation package to diagram-util package
Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[x] Added necessary tests

[x] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

All connector API changes added to the new PR https://github.com/ballerina-platform/ballerina-lang/pull/33220


New/update endpoint to get the category tags in organizations workflows & tools

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like

The languages and other additional tags need to be returned and displayed on the UI
Number of stars also needs to be returned
Current UI: https://dev.dockstore.net/organizations/bdcatalyst/collections/GWAS
Mock up: https://app.zeplin.io/project/5fc690a2717ee9144814f190/screen/60073f22c824b7b8dfde055c

Describe alternatives you've considered
N/A
Additional context
Similar issues: https://github.com/dockstore/dockstore/issues/4074 https://github.com/dockstore/dockstore/issues/4038
e.g.

┆Issue is synchronized with this Jira Story
┆Issue Number: DOCK-1725
┆Sprint: Backlog
┆Issue Type: Story

One thought is that we could one single endpoint, off /metadata, that returns all the information needed. May not be very very RESTful, but it seems like it would work and we could make it very fast with the right queries.

Closing this since it's a duplicate of https://github.com/dockstore/dockstore/issues/4053 and https://github.com/dockstore/dockstore/issues/4052.
To address Charles' comment, we had a private chat and decided modifying the endpoints are the easiest, and the best for performance.


Azure Blob connection string containing key with + chars fails

If the key in a connection string for an Azure Blob Storage account contains a + character, the method AzureBlobStorageWithSharedKey fails with the message "Invalid length for a Base-64 char array or string."
I'm using Storage.Net 9.2.6
My rough guess is that the + characters are replaced with spaces somewhere (maybe there is some URL decoding).
Workaround (until fixed): Regenerate the key until you get one without a +.
Workaround/Fix: always URL-encode your storage key.
Stack Trace:
System.FormatException: Invalid length for a Base-64 char array or string.
   at System.Convert.FromBase64_Decode(Char* startInputPtr, Int32 inputLength, Byte* startDestPtr, Int32 destLength)
   at System.Convert.FromBase64CharPtr(Char* inputPtr, Int32 inputLength)
   at System.Convert.FromBase64String(String s)
   at Storage.Net.Factory.AzureBlobStorageWithSharedKey(IBlobStorageFactory factory, String accountName, String key, Uri serviceUri)
   at Storage.Net.Microsoft.Azure.Storage.Blobs.Module.CreateBlobStorage(StorageConnectionString connectionString)
   at System.Linq.Enumerable.WhereSelectListIterator`2.MoveNext()
   at System.Linq.Enumerable.FirstOrDefault[TSource](IEnumerable`1 source, Func`2 predicate)
   at Storage.Net.ConnectionString.ConnectionStringFactory.Create[TInstance](String connectionString, Func`3 createAction)

As I expected the + is replaced by a space in StorageConnectionString.Parse() with the line
_parts[key] = value.UrlDecode();

which leaves me uncertain if this is a bug or a feature.
On one hand, it is formatted like a URI, starting with the storage type as "protocol". So one could argue UrlDecodeing the arguments is a valid thing to do.
On the other hand it is really unexpected for me as a user that I cannot just copy&paste the storage key into the connection string but have to check for characters which need URL encoding, even if its just a connection string, not an actual URL.

I'm facing the same issue.
The workaround is like playing roulette...

This is unfortunate, yes plus sign in URL has a special meaning - space. We think of all the parameters as URL-encoded, because in some cases like drive path there are special characters that cause issues with a parser. The safest way to pass parameters is to url-encode them first though.

Yes, this is a bit unfortunate. It is inconsistent with how DB connection strings work, for example.
But I understand, why it is necessary to url-encode the parameters.
It would be great though if this is mentioned in the documentation.
I would add this as a hint to every occurrence of a connection string in the documentation, but I know this seems a bit excessive.

Workaround (until fixed): Regenerate the key until you get one without a +.

The workaround is like playing roulette...

Maybe instead of "key roulette" you could (strictly a workaround) use %2B instead of +

Yeah, should have updated the workaround after finding out the reason it doesn't work with a +. Have done it now.

@aloneguid should the connection string be encoded within the StorageNet library, or encoding it before initializing the storage would be the right way to do it?


Update to version 1.2.5

https://github.com/tailscale/tailscale/releases/tag/v1.2.5

1.2.6 is out
https://github.com/tailscale/tailscale/releases/tag/v1.2.6


Readme CLA

Add note to README regarding CLA for PR.
Note: I find many repos pull this out into a CONTRIBUTING.MD. Not sure if you're interested in that or not - add code of conduct, yada.

Thank you @marknadig!


Feature Request: allow to specify DSCP/TOS of network connection

Please allow to specify DSCP/TOS of network connection so that they can be detected more easily for better QOS management

https://godoc.org/golang.org/x/net/ipv4#Conn.SetTOS
https://godoc.org/golang.org/x/net/ipv6#Conn.SetTrafficClass

Great idea. It should be reasonably easy to implement in rclone's dial functions (fs/http_new.go and fs/http_old.go).
If you fancy giving it a go I'll write up exactly how?

Here is what the SSH manual says
IPQoS   Specifies the IPv4 type-of-service or DSCP class for connections.  Accepted values are “af11”, “af12”, “af13”, “af21”, “af22”, “af23”, “af31”, “af32”, “af33”, “af41”,
             “af42”, “af43”, “cs0”, “cs1”, “cs2”, “cs3”, “cs4”, “cs5”, “cs6”, “cs7”, “ef”, “lowdelay”, “throughput”, “reliability”, or a numeric value.  This option may take one or two
             arguments, separated by whitespace.  If one argument is specified, it is used as the packet class unconditionally.  If two values are specified, the first is automatically
             selected for interactive sessions and the second for non-interactive sessions.  The default is “lowdelay” for interactive sessions and “throughput” for non-interactive ses‐
             sions

And they are defined thus in the source code
/*
 * Definitions for IP type of service (ip_tos)
 */
#include <netinet/in_systm.h>
#include <netinet/ip.h>
#ifndef IPTOS_LOWDELAY
# define IPTOS_LOWDELAY          0x10
# define IPTOS_THROUGHPUT        0x08
# define IPTOS_RELIABILITY       0x04
# define IPTOS_LOWCOST           0x02
# define IPTOS_MINCOST           IPTOS_LOWCOST
#endif /* IPTOS_LOWDELAY */

/*
 * Definitions for DiffServ Codepoints as per RFC2474
 */
#ifndef IPTOS_DSCP_AF11
# define        IPTOS_DSCP_AF11         0x28
# define        IPTOS_DSCP_AF12         0x30
# define        IPTOS_DSCP_AF13         0x38
# define        IPTOS_DSCP_AF21         0x48
# define        IPTOS_DSCP_AF22         0x50
# define        IPTOS_DSCP_AF23         0x58
# define        IPTOS_DSCP_AF31         0x68
# define        IPTOS_DSCP_AF32         0x70
# define        IPTOS_DSCP_AF33         0x78
# define        IPTOS_DSCP_AF41         0x88
# define        IPTOS_DSCP_AF42         0x90
# define        IPTOS_DSCP_AF43         0x98
# define        IPTOS_DSCP_EF           0xb8
#endif /* IPTOS_DSCP_AF11 */
#ifndef IPTOS_DSCP_CS0
# define        IPTOS_DSCP_CS0          0x00
# define        IPTOS_DSCP_CS1          0x20
# define        IPTOS_DSCP_CS2          0x40
# define        IPTOS_DSCP_CS3          0x60
# define        IPTOS_DSCP_CS4          0x80
# define        IPTOS_DSCP_CS5          0xa0
# define        IPTOS_DSCP_CS6          0xc0
# define        IPTOS_DSCP_CS7          0xe0
#endif /* IPTOS_DSCP_CS0 */

thank you very much. This is very useful to identify rclone's connection at router level and deprioritize related traffic


Typecode drift

Reporting a bug

[x] I am using the latest released version of Numba (most recent is visible in
the change log (https://github.com/numba/numba/blob/master/CHANGE_LOG).
[x] I have included below a minimal working reproducer (if you are unsure how
to write one see http://matthewrocklin.com/blog/work/2018/02/28/minimal-bug-reports).

From https://github.com/numba/numba/pull/5579#issuecomment-617371499
Reproducer:
from numba import njit, types, typeof

@njit
def foo1(x):
    return x + 1

@njit
def foo2(x):
    return x + 2

c1 = typeof((foo1, foo2))._code
c2 = typeof((foo1, foo2))._code

assert c1 == c2

Here is a smaller reproducer:
>>> from numba import types, typeof
>>> t = (types.int32, )
>>> typeof(t)._code
748
>>> typeof(t)._code
749

Or even simpler:
>>> from numba import typeof
>>> typeof((2, ))._code
773
>>> typeof((2, ))._code
774

Do there exist cases where the typeof should return a new type for the same input? I guess one instance is when the typeof is used in different typeof context.
Currently, we have:
>>> typeof((2, 1.2)) == typeof((3, 3.4))
True
>>> typeof((2, 1.2))._code == typeof((3, 3.4))._code
False

Do there exists cases where the following property would not be desired in general:
(typeof(obj1) == typeof(obj2)) == (typeof(obj1)._code == typeof(obj2)._code)

?


How to distinguish between drag and click events

I want the mouse down to 1 second or 2, and then began to drag, in order to distinguish the click event . or any other good ideas ?

How can this be acvhieved in the angular version?


Validate with frictionless v5

The automated tests for camtrap-dp currently use frictionless v4:
https://github.com/tdwg/camtrap-dp/blob/aace2ee526c2b5e6b55325dea6173406762a96f5/tests/requirements.txt#L9
We should try to update this to frictionless v5 (see migration info):

[x] 1. Fix cannot import name 'validate_package' from 'frictionless'
[x] 2. Wait until license validation is fixed: https://github.com/frictionlessdata/framework/issues/1467
[x] 3. Wait until resource profile validation is fixed: https://github.com/frictionlessdata/framework/issues/1468

│ None │ None  │ package-error │ The data package has an error: {'name': 'deployments', 'path': 'deployments.csv', 'format': 'csv',           │
│      │       │               │ 'mediatype': 'text/csv', 'encoding': 'utf-8', 'schema':                                                      │
│      │       │               │ 'https://raw.githubusercontent.com/tdwg/camtrap-dp/0.6/deployments-table-schema.json', 'type': 'table'} is   │
│      │       │               │ not valid under any of the given schemas at property 'resources/0'

This issue is resolved with #320


[CI] Fix pylint.

Fix error with the latest pylint: https://github.com/dmlc/xgboost/actions/runs/9111497690/job/25048748146 .

LGTM.


[Event Request] page 510 "Blanket Purchase Order Subform" - Event for procedure DeltaUpdateTotals

Can we get the following event for the "Blanket Purchase Order Subform" page: OnDeltaUpdateTotals
    local procedure DeltaUpdateTotals()
    var
        IsHandled: Boolean;
    begin
        DocumentTotals.PurchaseDeltaUpdateTotals(Rec, xRec, TotalPurchaseLine, VATAmount, InvoiceDiscountAmount, InvoiceDiscountPct);
        //New Start
        IsHandled := false;
        OnDeltaUpdateTotals(IsHandled);
        if not IsHandled then begin
            if "Line Amount" <> xRec."Line Amount" then
                SendLineInvoiceDiscountResetNotification;
        end;
        //New End
    end;

    [IntegrationEvent(true, false)]
    local procedure OnDeltaUpdateTotals(var IsHandled: Boolean)
    begin
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Server often crashes under load. How can we fix it?

Good day!
Unfortunately, we have some kind of problem on our server.
The server periodically get crash error, and the more players login on it -we got this problrm more often.
Here are the screenshots of the error.
Is there any way to fix it?

I've had this issue too on my server when it gets a lot of activity.  Unfortunately, since I use Restart on Crash to keep the server running all the time, I could never get a screenshot of the console before the server restarted.  I think it would be a good idea to have the server log exception errors to a .txt file whenever they pop up so they don't go unnoticed.

Hi guys ( @DXMtt , @BlueCloudPony ). @Ignis34Rus had merged few stability fixes recently that were really helpful in my case. Perhaps your installations should also look better with the new code. Hope this helps.

@dimeona It works quite well.  Haven't noticed any out of the blue crashes since updating.  Combat works even better than ever.   The new way of adding spawnable items is really useful.  It's been a real timesaver with adding in all the flowers, mushrooms, and rocks I had planned to add in.  Picking up items works flawlessly now with the stock DB, but things do tend to start acting up when new stuff is added.


this don't work - build docs error

https://github.com/XRPLF/rippled/blob/2d1854f354ff8bb2b5671fd51252c5acd837c433/docs/README.md?plain=1#L59
Cmake gives an error lz4 package,
for the solution you propose to use Conan,
so write how to use conan to build the documentation.

duplicate of #4973


Update sample-type.ttl

removed hyperspectral scan as this is better modelled as an observation type on a core or cuttings sample type. Happy to present my rationale and evidence of the approved business procedure to a group if needed.

Hi all. Checking on status of this.  Ready to merge? @Steven-Bowden do you need to review?

@Steven-Bowden is there a final step that id missing between committing the cose to mainin github and picking up the new vocabs in prod?  Is this an SRA task, or a GSQ task?

GSQ task is being resolved as per GSD-480

resolved https://vocabs.gsq.digital/v/vocab/def:sample-type


Fix progressbar.ProgressBar initialization

Initializing progressbar.ProgressBar with the keyword argument max_value fails because ProgressBar.init only takes a maxval argument. That and a ProgressBar object needs to be further initialized before calling update() by first calling start().
I am not sure how this broke. Looking at the pypi page for progressbar, I don't see a version 2.4 to even download, which makes me suspect that the progressbar maintainers uploaded a different 2.4 version that breaks save-instapaper-highlights.
I also wanted to take this chance to express my thanks to you for creating this script. Getting my instapaper highlights exported into simple MD files was a huge boon to my productivity.

Ah well, I should have pulled the HEAD of master before making my PR. I see this is already fixed.
Closing the PR, but my thanks still stands :)


chore: change requests to flat encoding

Motivation
In devnet 4, the current request encoding scheme is replaced with a flat encoding scheme, i.e. no rlp involved.
The request payloads (i.e. bytes following the request type byte) are now encoded as follows:

EIP-7251: source_address ++ source_pubkey ++ target_pubkey
EIP-7002: source_address ++ validator_pubkey ++ amount
EIP-6110: pubkey ++ withdrawal_credentials ++ amount ++ signature ++ index

Solution
I decided to simply implement Encodable7685 and Decodable7685 for each request type, and for the Request enum wrapper in alloy-consensus, I defer to these implementations.
Closes https://github.com/alloy-rs/alloy/issues/1338
PR Checklist

[ ] Added Tests
[ ] Added Documentation
[ ] Breaking changes

Please do not merge this upon approval, I'd like to merge the devnet 4 PRs in one go when necessary.


Runtime Input Remapping

Describe the problem
Currently, it's very hard to rebind inputs at runtime without writing a separate controller class that sits in between actions and the actual code.
Describe the solution you'd like
It would be nice to have a script or GameObject that can serve as an active Input Action handler, or at least support editing the ControllerMappingConfigurationProfile at runtime.
Describe alternatives you've considered
The only viable solution I can think of is to map all the inputs to an input handler that can then dynamically call the respective parts of the code. If others have made remapping screens in their MRTK projects, it would be really helpful to see what you did!

We appreciate your feedback and thank you for reporting this issue.
Microsoft Mixed Reality Toolkit version 2 (MRTK2) is currently in limited support. This means that Microsoft is only fixing high priority security issues. Unfortunately, this issue does not meet the necessary priority and will be closed. If you strongly feel that this issue deserves more attention, please open a new issue and explain why it is important.
Microsoft recommends that all new HoloLens 2 Unity applications use MRTK3 instead of MRTK2.
Please note that MRTK3 was released in August 2023. It features an all-new architecture for developing rich mixed reality experiences and has a minimum requirement of Unity 2021.3 LTS. For more information about MRTK3, please visit https://www.mixedrealitytoolkit.org.
Thank you for your continued support of the Mixed Reality Toolkit!


rax 组件demo 支持引入 css 和 js

Do you want to request a feature or report a bug?
What is the current behavior?
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem.

Iceworks CLI version：
Node verson：
Platform：

What is the expected behavior?

同类问题 #157


Should allow an equalTo query with a 'null' value so that the client …

…is able to query for data that does not have a specific key.
I believe there should a more elegant solution but this works and we have a test in place.

@johnlim
How do you use this equal-to attribute? I pass a value and i get undefined

@JGSolutions like this:
   <firebase-query
      id="query"
      path="[[path]]"
      data="{{data}}"
      app-name="appName"
      equal-to="null"
      equal-to-null>
    </firebase-query>

This might have bad side effects where a particular value in the database would have a string "null" (which I don't know why they would do that but it will happen)
But having another flag called equal-to-null might be hackish in a sense that it is not really part of the firebase doc spec.
I don't know if it is possible to have it this way
<firebase-query
      id="query"
      path="[[path]]"
      data="{{data}}"
      app-name="appName"
      equal-to="[[equalTo]]">
    </firebase-query>

and have it equalTo = null then wrap around a checker that checks if it is undefined to not set equalTo

This would be helpful. Can this be merged now?


Update strings.i18n.json Tray -> Menu Bar

Small request:
On MacOS, the bar that runs along the top of the computer is referred to by Apple as the Menu Bar.1 I was slightly confused upon seeing "tray" and thought is was the dock, like this guy.2
It seems that updating this might impact all settings, if so, please tell me, and I'll see if I can make it Mac specific.

Yeah, I think we need to make this OS specific. On Windows it is a tray, on Linux it depends. So we can add a translation for macOS only.

Sounds good. Anything I can do to make it easier?

As someone on both Linux and macOS every day, can i suggest, "Quit: Minimize to Tray/Menu Bar",

Ok, sounds good


Update tap to ^16.0.0; small additional changes

Best to review this PR per commit. As f7950a0e12da85351ae2e27330c8a47c6d7820ee pollutes the diff enormously.
Fixes #10

@vladikoff, thanks for the fast response. Could you maybe create a new release?

In v0.16.0


Textual representation of values doesn't work when nested

Looks like there's a problem with how the environment is captured — eg. writing show 'a displays the input as 'a, but show ('a) displays ('<value>) instead of ('a).

Looks like this is the culprit:
https://github.com/wipplelang/wipple/blob/95627dd892ef2c2a3f9d656fe973728729f36f31/core/src/fundamentals/traits.rs#L91-L96
The conformance for Text derives the Text values for inner items (ie. those in quoted values and lists). We need to at least separate the validation and the derivation, and ideally refactor the conformance resolution system altogether.

Fixed in c83e009


Update package pins now that we're building again.

Pretty straightforward.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Example how to use data bags in recipes?

I checked the knife-solo_data_bag documentation but I cannot for the life of me find an example how to access the data in those data bags in my recipes?

@hackeron , did you read the docs?
1.[] ; ; ; ; ; X.Knife-solo data bag by thbishop ; ; http://thbishop.com/knife-solo_data_bag/


Scan chains in parallel on startup

Summary
Would be nice if Hermes would scan chains in parallel on startup.

Problem Definition
On startup, Hermes scans chains for open channels, and apparently it's done sequentially:
       let mut scans = ChainsScan {
            chains: Vec::with_capacity(self.config.chains.len()),
        };

        for chain in self.config.chains.clone() {
            scans.chains.push(self.scan_chain(&chain));
        }

And as far as I know, this process kinda blocks the whole Hermes startup.
Would be really cool if this process is done in parallel.
This would especially benefit those who relay a lot of chains at once, as instead of having to wait for each chain to be scanned, the process would take the time of the longest chain to scan at max.

Proposal

Scan chains in parallel, that's it.
Acceptance Criteria

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate milestone (priority) applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

Would gladly accept a PR for this, otherwise it will have to wait a bit until we have capacity to work on this.

@romac I'm all in for making a PR on that, but I am not sure about how to implement it in the best way. Are there any similar places which I can look into to copy-paste stuff from there?

I'm all in for making a PR on that

Awesome!

Are there any similar places which I can look into to copy-paste stuff from there?

Not really, we don't much of this kind of parallelism in Hermes, most of the relayer is concurrent but not parallel.
I would start by looking into rayon, it's probably the easiest way to get started.
Ideally we would do something like:
pub fn scan_chains(mut self) -> ChainsScan {
    let scanned_chains = self.config.chains.par_iter().map(|chain| {
         self.scan_chain(&chain)
    }).collect::<Vec<_>>();

  ChainsScan {
      chains: scanned_chains,
  }
}

but that's not going to work, because scan_chain takes &mut self, so this needs a wider refactoring of the scanning code, which is why we haven't gotten to it yet.

@romac any idea on what I can take as a reference to do this properly?
My Rust knowledge is quite limited so I get a lot of these:
    |         ^^^^^^^^^^^^^^^^^^^^^ `self` is a `&` reference, so the data it refers to cannot be borrowed as mutable

and I am not sure if there are any similar places in the project, or anything, that I can take as a reference to see how it's working there.


[Trivial] fixing a bug to grab photon vm syslog incase of test failure

dump_log.sh fails to collect syslog incase of test failure on photon vm.
This PR is going to fix the above problem and make sure not generating empty syslog while truncating the logs.
/CC @pdhamdhere @govint

https://github.com/vmware/docker-volume-vsphere/pull/1427#pullrequestreview-44201438

Thanks @msterin! I will sync up with you offline to more idea on the above comment.

https://github.com/vmware/docker-volume-vsphere/pull/1427#pullrequestreview-44198104

It was a regression at TOT. Photon VMs are not having syslog (we need to capture using journalctl) and following line (log truncation) is creating an empty syslog files.
$SSH $USER@$1 "echo > $SYS_LOGFILE"
Ideal way could be to find out VM type (photon vs ubuntu) and invoke the command to capture syslog (not sure whether it is more time consuming to find out OS type)

Got it. LGTM.


[Feature Request] Resource Widgets

Probably more of a power user type of thing, and probably related to #40 . I think it would be nice to have widgets that can be popped out from the party resource list, so you can show that singular resource as a bar/gauge or something (with the option to move and hide it), as opposed to showing all resources at once with the status bar.
Double points if you can replicate BarBrawl's function to use custom images for the bars, using a background image and foreground image to make something like this:

I believe bar brawl does this by hiding the foreground image by width.

Huh. 🤔
I'm not familiar with Bar Brawl, but I could check out if I can leverage it somehow for Party Resources, with regards to #40.

Bar brawl is for token bars (for hp and more), but it's super flexible.
They recently added "advanced settings" which allowed the picture based gauges, like this

My only worry with this is that it might be (a lot) easier to write an accommodation for Party Resources in Bar Brawl, than it would be the other way around.
I have a couple of other feature requests I need to spend some time on first, but after that I'll give this a closer look.

It would be a bit different to make it something that goes alongside bar brawl as that would just be another token resource bar, as opposed to a resource widget :P


Lua snip for macro autocompletion

Change from vsnip to Lua snip and reuse VSCode macro completion data:
https://github.com/L3MON4D3/LuaSnip/blob/master/DOC.md#vscode

https://alpha2phi.medium.com/neovim-for-beginners-snippets-450a855584f8

Luasnip gets slow when using it for m4's, so only used for other (b/d/i/l/m/x-file) snippets and other solution for m4s https://github.com/TomDeneire/nvim/commit/c462482f2677b33b240a21232b55d2ce15d2c9a2


handle buggy os api events with null payment token

this was a new os api event response issue that began happening in May 2022. Roughly 20 or so OS sales list payment token as null instead of ETH or WETH.
This fix assumes payment token was ETH, and logs a warning telling the user that a null payment token was encountered, that ETH was assumed to be the payment token, and logs the tx hash of the tx containing the sale.
Because OS API is a centralized third-party API, there isn't much we can do other than handle, assume, and warn when an invalid null payment token is encountered.

@jakerockland yeah, I'll toss something over to Kartik & Michael so they are aware


Update readme.md

Hi Travis, thanks for this awesome API. So I wrote a Dart API and a Flutter app using your repo. If you accept I will be very happy. :)

Looks awesome @coskuncay 🙏


cannot login after install via docker

whats the default user and password??

@zfirered login instructions: https://github.com/DefectDojo/django-DefectDojo/blob/master/DOCKER.md#access-the-application


Only fail validation of cash letter control for settlementDate field if it is not a return type

Here is the original discussion on this matter.
According to different specs, the SettlementDate is a conditional field. Currently, the parser requires this field to exist. Having this field present for Return bundles does not make sense hence why they are defined to be conditional.
There were two approaches to take:

To remove this check completely.
To not have it required for Return documents which are indicated by CollectionTypeIndicator.

I went with the second approach.

I like the idea of passing contextual knowledge over to Validate. It looks like we'll need to do more of this to support all X9 specs.


defaultViewport option does nothing

Steps to reproduce
Tell us about your environment:

Puppeteer version: 1.6.2
Platform / OS version: Windows 7
URLs (if applicable):
Node.js version: 8.11.3

What steps will reproduce the problem?
Launch a browser with defaultViewport size option
Please include code that reproduces the issue.
browser = puppeteer.launch({ 	headless: false, 	defaultViewport: { 		width: 600, 		height: 600 	} });
What is the expected result?
That the browser have the size configured in the options
What happens instead?
The browser viewport doesn't have the configured size

Well, then, the documentation linked in the repository home is wrong. It's not pointing to 1.6.2 API

Well, then, the documentation linked in the repository home is wrong. It's not pointing to 1.6.2 API

Fair.

@vsemozhetbyt I am using 1.7.0 and it is not working
How to set viewport? So simple question have no answer.

Can you provide the details (code, screenshots)?

@vsemozhetbyt thanks for reaction. I noticed that the issue is window size. Viewport changes but window is not. I guess I need to set it separately?

Seems so)

hey, any update on this?! I basically want to have a specific window size when it starts up.


fix: Fix Tabbable preventing click after enabling it

Closes #480
By the way, something like eslint-plugin-react-hooks would catch this.

Should enabling eslint-plugin-react-hooks be done in a separate PR?

@tom-sherman Yes, but I'm not sure that is a straight forward addition. I've tried to add it before, but it wasn't working well. I don't remember exactly why, but I guess it was complaining about the way we define the deps (like [options.disabled] instead of [disabled]). Feel free to give it a try if you want! :)

@diegohaz there have been some recent updates to the plugin that make the detection much more intuitive, I'll take a look!


Upgrade scc-state and scc-broker to SC v8

SC v8 introduces client subscription batching which should improve the efficiency of scc-state and scc-broker.

+1 Alright!!!

Upgraded scc-broker to v9.

@jondubois thanks so much man. Now I can update our broker/state servers.


hits and lin regr doc updates

minor updates to graph HITS algo docs
fixed order of params in lin regr prediction docs

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/287/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/288/

Commit daf67f81b merges this PR. This can be closed.


Sponge Exploit Check causing (some)entity related flags to no longer work

Wasn't sure if this was something Sponge would(or could) fix, or something GP will have to compensate for, so I figured might as well start reporting it here.
SpongeForge 1.12.2-2705-7.1.0-BETA-3477
GP <IP_ADDRESS>8
Sponge config for exploits update-tracked-chunk-on-entity-move=true causes a flag such as enter-claim minecraft:monster false to not work, and monsters are able to enter claims.
Setting update-tracked-chunk-on-entity-move=false and the behaviour works as it should.

Fixed in https://github.com/SpongePowered/SpongeCommon/commit/af82eca3132dc8756b218db2bdaac65c5a7adca2


Apply mysql connection pool

Connection pools help reduce the time spent connecting to the MySQL server by reusing a previous connection,
leaving them open instead of closing when you are done with them.
This improves the latency of queries as you avoid all of the overhead
that comes with establishing a new connection.
The connection can now be pulled out of the pool and used to
create a new connection or to use the created connection.

@MichaelKim20 Thank you for the reviews.
I applied the suggestions now.


ci: switch to using github actions

This PR switch to using GitHub Actions from Travis.

TODO: look into if it's possible to prevent running both "push" and "pull_request" test.

Nice work! We should definitely switch to github actions. I did it a while back  for some of my other repos but at that point in time codecov required a token even for public repos. Seems like that is not required anymore :-).
I think we can go ahead and merge this?


Detect changes and only prompt saving if changes are present

Currently the plugin always prompts the user to save regardless of whether the JSON is changed or not. If would be nice if it can detect changes, and only prompt saving if changes are present.

It doesn't seem like you can listen for changes to the items in a Tree. Help is welcomed!

Actually there is an item_edited signal, apparently I'm blind.


Connecting to all Databases of the same instance of Redis OM connection

Why I can not connect to all Databases of the same instance of Redis OM connection and just only the database with index zero, and if it is possible another way how can I implement it?

Search and Query only work on db0. Using other numbered databases in Redis is considered an anti-pattern, hence the restriction.

I have multiple instances of the same application each instance dedicated to a different client, what's your recommendation regarding the Redis connection? Should I create a different Redis server for each client or make them connect to the same Redis Server?
and if I should make them connect to the same Redis Server, Is there a way to make the filtration in the Redis server not in the memory ?


fix: turn off in-cluster discovery of vault URL in boot-vault

boot-vault will fail until https://github.com/jenkins-x/jx/pull/5613
is merged/in the builders we're testing, but hey. =)
Signed-off-by: Andrew Bayer<EMAIL_ADDRESS>

/lgtm

/override boot-vault

🎉


Restore Sensor name

Trying to address #204 by restoring more explicit sensor names, without affecting the improvements from #116.

Yeaaa @fmartinou please merge this <3


Roll Engine from fcf275410eee to 43c292b41cc9 (3 revisions)

https://github.com/flutter/engine/compare/fcf275410eee...43c292b41cc9
2022-01-19<EMAIL_ADDRESS>Roll Skia from 98c09e51f33e to d0c5af652318 (6 revisions) (flutter/engine#30930)
2022-01-19<EMAIL_ADDRESS>Roll Dart SDK from f4a696c76682 to b3f3c7dbd422 (1 revision) (flutter/engine#30929)
2022-01-19<EMAIL_ADDRESS>Allow builder to skip building examples. (flutter/engine#30917)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Add authentication

To make sure we have a secure platform, lets add some authentication.
Fixes: #4
/cc @brambokdam

:shipit:


Update to the mime package due to NPM audit fail.

https://nodesecurity.io/advisories/535

@andrewrk @eXist-FraGGer - just wondering - is this going to be merged? Its been a few months now.


Debug on cluster fails when plotting due to MATLAB version

Problem: When trying to run a batch job on the cluster, if one passes 'dsPlot' into 'plot_options' for in-batch plot generation, the dsPlot function call fails due to "Undefined function 'gcp'..." in dsAnalyze. This occurs even when parfor_flag is explicitly set to 0. This error is due to gcp only being introduced in MATLAB version 2013b, while the default cluster version seems to still be 2013a.
Suggested solution:
- implement a way for the user to control the MATLAB version on the cluster, in addition to
- add a parfor_flag check to the line 529 in dsAnalyze.
Sidenote: Does this mean no one has run cluster simulations with the debug branch that does plotting in months?? debug has not changed substantially for several months...
Below is a script that, when run from a login node, should reproduce the issue (which you can see in the sim_job1.out):
%{
# Simulation Runscript file for testing DynaSim code changes

Code mostly taken from `demos/dsDemos.m`
%}

% %%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
%% BUILDING LARGE MODELS WITH MULTIPLE POPULATIONS AND CONNECTIONS
% %%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%

study_dir = strcat(mfilename);

debug = 0; % Only use 1 if cleaning
if debug == 1
    clf
    close all
    % Note: the below may be dangerous!
    if 7==exist(study_dir, 'dir')
        rmdir(study_dir, 's')
    end
end

%% Setup simulation parameters and flags
time_end = 100;   % in milliseconds
dt =        0.01; % in milliseconds

save_data_flag = 0;
save_results_flag = 1;
verbose_flag =   1;
overwrite_flag = 1;
parfor_flag =    0;

% % local
% cluster_flag =   0;
% memory_limit =   '8G';
% num_cores =      1;

% cluster
cluster_flag =   1;
memory_limit =   '8G';
num_cores =      1;

vary={
'E', 'Iapp', 3;
% 'PYdr', 'appliedStim', [0,1,2];
};

%% Setup Sparse Pyramidal-Interneuron-Network-Gamma (sPING)

% define equations of cell model (same for E and I populations)
eqns={
  'dv/dt=Iapp+@current+noise*randn(1,N_pop); Iapp=0; noise=0'
  'monitor iGABAa.functions, iAMPA.functions'
};
% Tip: monitor all functions of a mechanism using: monitor MECHANISM.functions

% create DynaSim specification structure
s=[];
s.populations(1).name='E';
s.populations(1).size=80;
s.populations(1).equations=eqns;
s.populations(1).mechanism_list={'iNa','iK'};
s.populations(1).parameters={'Iapp',5,'gNa',120,'gK',36,'noise',40};
s.populations(2).name='I';
s.populations(2).size=20;
s.populations(2).equations=eqns;
s.populations(2).mechanism_list={'iNa','iK'};
s.populations(2).parameters={'Iapp',0,'gNa',120,'gK',36,'noise',10};
s.connections(1).direction='I->E';
s.connections(1).mechanism_list={'iGABAa'};
s.connections(1).parameters={'tauD',10,'gGABAa',.1,'netcon','ones(N_pre,N_post)'}; % connectivity matrix defined using a string that evalutes to a numeric matrix
s.connections(2).direction='E->I';
s.connections(2).mechanism_list={'iAMPA'};
s.connections(2).parameters={'tauD',2,'gAMPA',.1,'netcon',ones(80,20)}; % connectivity set using a numeric matrix defined in script

%% Simulate Sparse Pyramidal-Interneuron-Network-Gamma (sPING)
data=dsSimulate(s,...
    'save_data_flag',save_data_flag,'study_dir',study_dir,...
    'save_results_flag',save_results_flag,...
    'vary',vary,'cluster_flag',cluster_flag,...
    'verbose_flag',verbose_flag,'overwrite_flag',overwrite_flag,...
    'tspan',[0 time_end],'solver','euler','dt',dt,...
    'parfor_flag',parfor_flag,'random_seed','shuffle',...
    'memory_limit',memory_limit,'num_cores',num_cores,...
    'plot_functions',{@dsPlot},...
    'plot_options',{{'plot_type','waveform','format','png'}});

The quickfix was added in 11a5ba633

gcp has been removed.

also cluster version is now 2014a by default.