Support for componentWillUnmount.

I use "setInterval" in "OnInit".
Hope to have a chance to "clearInterval" just befor the panel will be destroyed.
On the same time, I want to know how to share timerid  between "OnInit" and other code.

I'll see what can be done with the unmount function.
The share onInit with other code (I assume onRender) you can bind it to the htmlNode. Which is what I usually do.
https://developer.mozilla.org/en-US/docs/Learn/HTML/Howto/Use_data_attributes
onInit
const timerId = 2;
htmlNode.lastElementChild.dataset.timerId = timerId;

onRender
const timerId = htmlNode.lastElementChild.dataset.timerId;
console.log(timerId);

Here is an example
JSON Options

You can import this code in the import section of the htmlgraphics panel
{
    "add100Percentage": true,
    "centerAlignContent": true,
    "overflow": "visible",
    "SVGBaseFix": true,
    "codeData": "{\n  \"text\": \"Random text\"\n}",
    "css": "* {\n  font-family: Open Sans;\n}\n",
    "html": "<div id=\"test\" />\n",
    "renderOnMount": true,
    "onRender": "htmlNode.querySelector(\"div\").textContent = htmlNode.lastElementChild.dataset.timerId;",
    "dynamicData": false,
    "panelupdateOnMount": true,
    "onInit": "const timerId = 2;\nhtmlNode.lastElementChild.dataset.timerId = timerId;\n"
}

Shareing variables with data attribute wiorks fine! Thanks lot.
If you have a plan to add Unmount functutionalioty,
I hope to integrate OnInit, OnRender,WillUnmount to one text fileld.

let timer_id
function OnInit(){
timer_id= setInterval ..
}
function OnRendet(){
Rendering codes goes here.
}
function WillUnmount(){
clear_interval(timer)id)
}

This style allows

closure variables
One file Cut&Pase editing!!

Simplified implementation structure without error handling could be
// user define code from configuration panel
const user_code = `
let i;
const onInit = () => { i=1;}
const onRender = () => { console.log(i, htmlNode);}
const onWillUnmount = () => { console.log(i);}
return [onInit, onRender, onWillUnmount]

`

// htmlNode, data, codeData, codeData, options, theme, getTemplateSrv, getLocationSrv, htmlGraphics
const [onInit, onRender, onWillUnmount] = (new Function('htmlNode', user_code))('dummy_html_node')

// use generated code
onInit()
onRender()
onWillUnmount()

I would suggest checking out https://github.com/gapitio/htmlgraphics-svg-bundler-template
It generates the panel-options which makes it pretty easy to copy paste it in.
There is an event https://github.com/gapitio/gapit-htmlgraphics-panel#panelupdate-event if you want to make everything in the onInit code. panelupdate is pretty much the same as onRender.
The unMount will also be an event (if I'm able to xD) and can be used in the onInit as well.
Events are handy if you're working on something like Svelte, Solid, React, ETC. You can also use it to have all the variables available in the onRender code aswell as onInit code.

Thank you for  good suggestions.
I 'm looking forward to have  unMount  on htmlNode in near future.
For a while, I'll check   htmlgraphics-svelte-bundler-template


Compression/Decompression and custom Path

Hi Mark,
I've tried to compress and decompress a Digital Link URI with a custom path and I had a problem.
Here is what I'm doing :
const input = 'https://example.com/some/other/path/info/01/09780345418913/21/12345';
const compress = Utils.compressWebUri(input);
const uncompress = Utils.decompressWebUri(compress);
expect(input).to.equal(uncompress);

But uncompress equals https://example.com/01/09780345418913/21/12345
Here are my compress and decompress functions :
const decompressWebUri = (uri, useShortText = false) =>
  toolkit.decompressGS1DigitalLink(uri, useShortText, getUriStem(uri));

const compressWebUri = (uri, useOptimisations = true, compressOtherKeyValuePairs = true) => {
  const uncompressedPrimary = false;
  const useShortText = false;

  return toolkit.compressGS1DigitalLink(
    uri,
    useShortText, // Not used
    getUriStem(uri),
    uncompressedPrimary, // Not used
    useOptimisations,
    compressOtherKeyValuePairs,
  );
};

Do you know where the issue comes from?
Thanks

Hi Clément,
I think the problem is that the compression algorithm is not preserving the URI path info before the part that is specific to GS1 Digital Link, so /some/other/path/info is being lost in the round-trip.  You can verify this at https://gs1.github.io/GS1DigitalLinkCompressionPrototype/  where you'll see that
https://example.com/some/other/path/info/01/09780345418913/21/12345
is compressed to
http://example.org/ARHKVAdpQkIKMDk
rather than
http://example.org/some/other/path/info/ARHKVAdpQkIKMDk
The good news is that the decompression algorithm will currently decompress
http://example.org/some/other/path/info/ARHKVAdpQkIKMDk  to
http://example.org/gtin/09780345418913/ser/12345
so with some relatively minor adjustment of the decompression algorithm, we can arrange for it to decompress to
https://example.com/some/other/path/info/gtin/09780345418913/ser/12345
or
https://example.com/some/other/path/info/01/09780345418913/21/12345
The GS1 Digital Link compression algorithm currently starts with an associative array of GS1 Application Identifiers and their values - it does not currently start with an uncompressed GS1 Digital Link URI.
If you feel that this is a major problem, please mention this in the GS1 Digital Link v1.2 work group - but it would need some work to make the adjustment to the initial compression flowchart to start with an uncompressed GS1 Digital Link URI and a corresponding adjustment to the decompression flowcharts.

Thanks for your answer.
I'll correct this issue in digital-link.js by adding the custom path directly after the compression and compression.

I think this should indeed be discussed in the group @mgh128 as it might be something the current description of the algorithm is missing, the issue is not so much that the custom path is not compressed (I do not think it would make sense) but that the compression algorithm's output will truncate it and hence is lossy.

Yes, we should have work group discussion on this, possibly even today.
The current starting point for the compression algorithm is an associative
array / object / map of AIs and their values, rather than the uncompressed
GS1 DL URI, so it looks like we'd need one extra flowchart or a
modification to existing flowcharts to ensure that any preceding path info
is preserved without compression.
We'd also need to check the grammar for a compressed GS1 DL URI to ensure
that it can tolerate path info before the compressed string as the final
component of URI path information.
On Thu, 29 Oct 2020, 10:45 Dominique Guinard<EMAIL_ADDRESS>wrote:

I think this should indeed be discussed in the group @mgh128
https://github.com/mgh128 as it might be something the current
description of the algorithm is missing, the issue is not so much that the
custom path is not compressed (I do not think it would make sense) but that
the compression algorithm's output will truncate it and hence is lossy.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/gs1/GS1DigitalLinkCompressionPrototype/issues/2#issuecomment-718634311,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABSXRL3WQ3M6RIKASIZATKDSNFBS7ANCNFSM4TBBQVYA
.


Check for local binaries

Make sure to check for binaries used by the script, beforehand, if it's not present fail as early as possible. For example in our case we didn't know yq is supposed to be part of the container image:
e.g. https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/openshift_release/31857/rehearse-31857-pull-ci-redhat-appstudio-managed-gitops-main-managed-gitops-e2e-tests-on-kcp/1565006131774885888#1:build-log.txt%3A37

/test all


ARROW-17927: [C++] Avoid hitting thread limits in stress tests

Non-trivial use of SleepAsync() and SleepABitAsync() could end up running hundreds of threads at once, which could hit resource limits on some CI configurations.

Hit a sporadic arrow-dataset-discovery-test timeout in https://github.com/apache/arrow/actions/runs/3197629372/jobs/5221100057

@github-actions crossbow submit -g cpp

Hmm, std::condition_variable::wait_until takes a const ref to the deadline...

@github-actions crossbow submit -g cpp

@github-actions crossbow submit -g cpp

Revision: e9f0437ff3178087d7fe320a251eb112ffff5caf
Submitted crossbow builds: ursacomputing/crossbow @ thread-async-sleep2-1

Task
Status

test-alpine-linux-cpp

test-build-cpp-fuzz

test-conda-cpp

test-conda-cpp-valgrind

test-debian-10-cpp-amd64

test-debian-10-cpp-i386

test-debian-11-cpp-amd64

test-debian-11-cpp-i386

test-fedora-35-cpp

test-ubuntu-18.04-cpp

test-ubuntu-18.04-cpp-release

test-ubuntu-18.04-cpp-static

test-ubuntu-20.04-cpp

test-ubuntu-20.04-cpp-17

test-ubuntu-20.04-cpp-bundled

test-ubuntu-20.04-cpp-thread-sanitizer

test-ubuntu-22.04-cpp

@github-actions crossbow submit test-ubuntu-20.04-cpp-thread-sanitizer

This approach would cause all future callbacks to be serialized which would change the dynamics of the tests a little bit.  I currently rely on SleepABitAsync's thread-per-sleep behavior to ensure parallelism in my stress tests.  So this might reduce the coverage of the stress tests.  Can we use a ThreadPool instead? #14339 should be sufficient.
This would mean we still have sufficient parallelism in the tasks.  The only downside is that if you have something like 128 sleeps then you will end up waiting 4 * delta seconds (where delta is SleepABit).  However, I did some experiments and this doesn't seem to be a problem.  Plus, I think running the tasks in parallel is likely to make up for any time lost.

Closing in favor of #14339.


Can the Grove Vision AI V2 be programmed on the Arduino IDE

Can the Grove Vision AI V2 be programmed on the Arduino IDE
I just want to use the SD card, but assume I have to use a different platform.

Hi @hpssjellis ,
The following link describes the communication between Grove Vision AI V2 and XIAO(arduino).  Hope his helps.

https://wiki.seeedstudio.com/grove_vision_ai_v2_at/

Thanks @stevehuang82 I see that area has some information about the SD card and saving images
https://wiki.seeedstudio.com/grove_vision_ai_v2_at/#example-3-saving-jpeg-images-to-sd-card
We can close this.


Add Third-party cargo subcommands to the README

r? @alexcrichton
(rust_highfive has picked a reviewer for you, use r? to override)

I wrote up the page so that the community can list the subcommands they've developed for Cargo. Right now I could only find one that seems intended for general use but going forward I'm sure we'll see more!

@bors: r+ 1e5f706
cool!

:hourglass: Testing commit 1e5f706 with merge 5bb4d8d...

:sunny: Test successful - cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-32, cargo-win-64


Dialyzer issue with Tesla.Client typespec

After upgrading my application from Tesla 0.7.2 to 0.9.0, I started seeing a warning when running Dialyzer.
Given the following code:
  @spec client(String.t) :: Tesla.Env.client
  def client(api_token) do
    encoded_token = Base.encode64("#{api_token}:api_token")
    Tesla.build_client [
      {Tesla.Middleware.Headers, %{"Authorization" => "Basic #{encoded_token}"}}
    ]
  end

I began seeing this warning:
Invalid type specification for function 'Elixir.InvoiceTracker.TimeTracker':client/1. The success typing is (_) -> #{'__struct__':='Elixir.Tesla.Client', 'fun':='nil', 'post':=[any()], 'pre':=[any()]}

I also tried with Tesla.Client.t instead of Tesla.Env.client without success.
I was able to work around the error by changing my typespec to the less strict:
@spec client(String.t) :: %Tesla.Client{}

I'm not sure what the root cause of this issue is or how to fix it.  If someone can give me some idea of what might be wrong here, I'd be happy to submit a PR with a fix.

Probably because Client.t has fun:  (Tesla.Env.t, Tesla.Env.stack -> Tesla.Env.t), part, but the fun can be nil. I'll try to find a workaround.

Please check with current master (https://github.com/teamon/tesla/commit/8fb4acaa108990cdb962376fc60cc582f6b453b7)

@teamon Yes, this resolves the problem for me.  Thanks!  Please let me know when you've created a release with the fix.
Thanks again for a great project.  I love using Tesla!

Thanks! I plan to push a new release in next few days


Dynamic links received is not fired during cold-start on iOS

I am having a problem where DynamicLinksReceived is not fired during cold start on iOS 14.5+, i.e. first open. It is fired when the app is suspended and the link is launched.
The Dynamic Links was working fine before I updated AdMob from 5.4.0 to 6.0.1, so I am guessing something is causing a conflict there.
        private void HandleDynamicLinks()
        {
#if !UNITY_WEBGL
            DynamicLinks.DynamicLinkReceived += DynamicLinks_DynamicLinkReceived;
#endif
        }

#if !UNITY_WEBGL
        private void DynamicLinks_DynamicLinkReceived(object sender, ReceivedDynamicLinkEventArgs e)
        {
            Debug.Log("Router: Dynamic link");

            PlayerPrefs.SetInt("DLinkTest", 1);

            var dynamicLink = e.ReceivedDynamicLink;
            FirebaseLogger.Log("Router: Deep link event fired");
            FirebaseLogger.Log($"Router: {e.ReceivedDynamicLink.Url} IsNull: {e.ReceivedDynamicLink.Url==null}");
            FirebaseLogger.Log($"Router: Dynamic Link Report 1");
            if (dynamicLink.Url != null)
            {
                Application_deepLinkActivated(dynamicLink.Url.AbsoluteUri);
            }
        }
#endif

Handle Dynamic Links is called right after FirebaseApp dependencies are available, which in turn is called right when the app is started.
Firebase.FirebaseApp.CheckAndFixDependenciesAsync()
Dynamic Links always open the app, which tells me that the configuration regarding the domain name should be correct. Along with the Apple App Site association

Turned out issue was a conflict with AppFlyers SDK removing which fixed the isse


markings in MB should not have e.g. "typo" classes?

Should the class .typo only be applied to typo markings in the editor?

I believe it was fixed here, https://github.com/OpenCovenant/quill/pull/483, therefore closing it.


purescript: 0.14.6 -> 0.14.7

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for purescript is: "A strongly-typed functional programming language that compiles to JavaScript"
meta.homepage for purescript is: "https://www.purescript.org/"
meta.changelog for purescript is: "https://github.com/purescript/purescript/releases/tag/v0.14.7"
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
Warning: no invocation of /nix/store/pcf1mgj9wqn608dpjmn21q0mazcbh0pw-purescript-0.14.7/bin/purs had a zero exit code or showed the expected version
0 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 0.14.7 with grep in /nix/store/pcf1mgj9wqn608dpjmn21q0mazcbh0pw-purescript-0.14.7
found 0.14.7 in filename of file in /nix/store/pcf1mgj9wqn608dpjmn21q0mazcbh0pw-purescript-0.14.7

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
2 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
pulp
purescript

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/pcf1mgj9wqn608dpjmn21q0mazcbh0pw-purescript-0.14.7 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A purescript https://github.com/r-ryantm/nixpkgs/archive/64ba26f745354ace38e955ca0554b0825c901ce3.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/pcf1mgj9wqn608dpjmn21q0mazcbh0pw-purescript-0.14.7
ls -la /nix/store/pcf1mgj9wqn608dpjmn21q0mazcbh0pw-purescript-0.14.7/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

pulp
purescript

Maintainer pings
cc @justinwoo @mbbx6spp @cdepillabout for testing.

@GrahamcOfBorg eval


Feature Request:  Enhance compute_duration() to handle Interval and Duration

Feature Idea
lubridate::time_length(x, unit = "second") handles Interval and Duration differently when unit is months/years

When x is an Interval object and unit are years or months, time_length() takes into account the fact that all months and years don't have the same number of days.
When x is a Duration, Period or difftime() object, length in months or years is based on their most common lengths in seconds (see timespan()).

Example of difference between intervals and durations:
> int <- interval(ymd("2000-02-01"), ymd("2000-03-01"))
> time_length(int, "months")
[1] 1
> time_length(as.duration(int), "months")
[1] 0.9527721

Currently, compute_duration() only uses interval to calculate "duration", would it be okay to add a feature to choose between interval and duration?
https://github.com/pharmaverse/admiral/blob/6dead5ad9b9fe529d8b459ae718f8f35cb52296f/R/compute_duration.R#L158
Relevant Input
No response
Relevant Output
No response
Reproducible Example/Pseudo Code
No response

@pharmaverse/admiral what do people think about this one?

I like the idea of adding this, wondering if interval should still be retained as the default?

I think adding an argument to select duration vs. interval is a good idea.
I am not sure if we need an option to specify the number of days of a year or a month. The lubridate values (365,25 and 30,4375) are the standard values used at Roche.

@pharmaverse/admiral oohhh this issue is stale - tyty mr bot.
Any takers @pharmaverse/admiral??  I think this would be a nice feature to add for the upcoming release.

@bms63 i can take this one on

I am working on this update now. @zdz2101 you mentioned keeping interval as the default (the current implementation in the released version). My impression is that the duration rather than the interval is the most common method for time period calculations, however. Certainly, this is the case in the academic research I've done. Is it different in the pharma world?
Thoughts @pharmaverse/admiral

My instinct is to lean towards keeping the default as interval just to minimize impact to users - thinking Roche here.  However, if @bundfussr, @millerg23, @manciniedoardo  says there is no big impact to admiralroche or Roche's templates - maybe we could just switch to duration.  I'd be keen on duration as well.

Agree to keep as is to minimise impact on any users.


Intercept argv and remove mirrord's temp dir

When an executable script my_script with a shebang #!/usr/bin/env some_interpreter is executed, it resolves to
/usr/bin/env some_interpreter my_script

So currently if we patch it and save a patch version in mirrord-tmp/, running /mirrord-tmp/my_script resolves to
/usr/bin/env some_interpreter /mirrord-temp/my_script

so what then happens when we run my_script with the mirrord CLI (if some_interpreter is a binary executable that does not have SIP):

We read my_script and its shebang.
We create a version of my_script in /mirrrord-tmp/path/to/my_script with the shebang #!/mirrord-tmp/usr/bin/env some_interpreter, and create a SIP-free version of env in /mirrord-tmp/usr/bin/env.
We execute /mirrrord-tmp/path/to/my_script which resolves to /mirrord-tmp/usr/bin/env my_interpreter /mirrrord-tmp/path/to/my_script
patched env is executed with loaded layer.
env calls execve with path my_interpreter and argv: ["my_interpreter", "/mirrrord-tmp/path/to/my_script"].

This is a problem when running node as the interpreter (#!/usr/bin/env node), since node determines the location of modules relatively to the executed file, which it receives as an argument (here argv[1]). In our example node would search for modules relative to /mirrrord-tmp/path/to/my_script, instead of relative to /path/to/my_script.
This PR suggests a mitigation that problem:
In the execve hook: go over the arguments in argv and strip the path to our temp directory from them if it's there.
That way node reads the original script, not the patched one, and can find the required modules.

With this change, the user needs to make sure the script they are running is excluded from FS hooks.
Before this change node would read the patched version of the script, which was in a directory that the sip patch code added to excluded directories. So if you would put your script in an included path, you could still run it on mac because of this side effect of the sip patch. Now if you try to run a script that is in a path that is included in FS ops, it will fail both on linux and mac. This seems like correct behaviour to me. To ensure good user experience I suggest we add the path of the executable (binary or script) the user is running to the excluded files at the start of the run (in a separate PR).

No changelog entry because the bug was not yet released.

Need to add it to cli's exec as well.

Need to add it to cli's exec as well.

Now using the original executable as argv[0], but I wouldn't change the other args to execvp as they are supplied by the user. Anyway I think it does not affect the actual issue we're dealing with.
The theoretical remaining questions is what happens in an imaginary situation where node itself is SIP (I think it's usually not) and we run a script at /path/to/script.js that starts with:
#!/bin/node

We would patch node at /mirrord-temp/bin/node and create /mirrord-temp/path/to/script.js with:
#!/mirrord-temp/bin/node

Then call execvp with:
path: "/mirrord-temp/path/to/script.js"
argv: ["/path/to/script.js"]
Which would result in the execution of /mirrord-temp/bin/node with /mirrord-temp/path/to/script.js (even though we have /path/to/script.js in argv[0]). If there is an extra call to execve there, it's before we have any hooks.
Node would then read and interpret /mirrord-temp/path/to/script.js and would not find modules that are required with a relative path.
I don't know how important that remaining problem is, but if it is important, a possible solution I can think of is:
Instead of running the patched script and letting shebang resolving occur naturally, we parse the shebang and execute
interpreter script directly. In the example from before that would mean call execvp with:
path: "/mirrord-temp/bin/node" (patched binary interpreter)
argv: ["/bin/node", "/path/to/script.js"] (original script)
I'm not sure what happens with arguments if we take this approach. Like is everything guaranteed to go as we would hope if we run /usr/bin/env node script.js arg-a arg-b arg-c and more generally /usr/bin/env interpreter script arg-a arg-b arg-c or /my/interpreter script arg-a arg-b arg-c

bors r+


Typed Builder implementation part 13.2: Clean up generators and lifter methods

[x] Tests added for any new code
[x] Ran make fmt-fix (or had formatting run automatically on all files edited)
[ ] Documentation added for any new functionality
[ ] Entries added to ./unreleased/ for any new functionality

closes #1966
Major changes:

All Gen[TParam] instances removed from individual tests and collected in BuilderTest.Generators
Ad-hoc implementations of toSeq values passed to expectEqTyped replaced with standard implementations in BuilderTest.ToSeq
checkRun now takes explicit generators; all generators are defined as explicit

Minor changes:

Normalized builder methods/generators to use Int instead of BigInt (inputs come from users, not TLA sources)
Changed notSupportedByModelChecker tests to use the standard framework.

Codecov Report

Merging #1981 (967405f) into unstable (bd6ffe2) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##           unstable    #1981      +/-   ##
============================================
- Coverage     77.81%   77.79%   -0.02%
============================================
  Files           418      418
  Lines         12787    12787
  Branches        567      573       +6
============================================
- Hits           9950     9948       -2
- Misses         2837     2839       +2

Impacted Files
Coverage Δ

...la/typecomp/subbuilder/LiteralAndNameBuilder.scala
86.36% <100.00%> (ø)

...he/tla/typecomp/unsafe/UnsafeApalacheBuilder.scala
100.00% <100.00%> (ø)

.../typecomp/unsafe/UnsafeLiteralAndNameBuilder.scala
83.33% <100.00%> (ø)

...a/at/forsyte/apalache/tla/lir/TlaLevelFinder.scala
97.36% <0.00%> (-2.64%)
:arrow_down:

...he/io/annotations/parser/CommentPreprocessor.scala
95.60% <0.00%> (-1.10%)
:arrow_down:

Continue to review full report at Codecov.

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


Remove children from props and add name

Description for 18
Removed children from props in template to increase clarity of generated code.
📋 Acceptance Criteria

[x] Code for generator has been changed to include name in props instead of children

Checked off by the PR assignee(s)

🔎 Review Checklist

[ ] Merge destination is correct
[ ] Code is correct as understood and conforms to quality standards
[ ] Tests have been added where appropriate (unit, visual, end-to-end)
[ ] Acceptance Criteria have been met

Checked off by the PR reviewers

Codecov Report

Merging #24 into develop will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           develop    #24   +/-   ##
======================================
  Coverage      100%   100%
======================================
  Files            3      3
  Lines            5      5
======================================
  Hits             5      5

Continue to review full report at Codecov.

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

Good call, I'll take a look!

We main need to compare our travis.yml against this: https://docs.cypress.io/guides/guides/continuous-integration.html#Travis
Our error ERROR: Cannot copy to clipboard: Couldn't find the required xsel binary. On Debian/Ubuntu you can install it with: sudo apt install xsel—looks like it may be related to the # Ubuntu 16+ does not install this dependency by default, so we need to install it ourselves.

I added that into the travis.yml and that seems to have solved the travis issues. Let me know what you think. @roblafeve @blaketarter


Point to different author

@technicallyfeasible did change the author based on this  but it appears that it is failing circleci.


Use pion/randutil

Move common random generators to pion/randutil.
Reference issue
Fixes #203

:fire: glad to see this not duplicated across packages anymore!


gmsh backend + merge requires an export/import step  - better do to in memory

A low-key solution is to use a ram-disk. To really fix-it would require overriding the export routine of cadquery

Likely this is not very relevant due to modern system buffering anyway...

closing - this has been identified as not being a bottleneck


How to solve the maximum context length limit

I typed "refresh this page", and it reported error as follow
Error
This model's maximum context length is 4097 tokens. However, your messages resulted in 6543 tokens. Please reduce the length of the messages.

We have a lot of ideas to shrink the page context further. The most general one is the concept of a viewport/scrolling, which we'll likely implement soon. See #6

get it, thank you


add rwkv nlp score

Thank you for your work.
I don't think this will work due to model name differences between the score file and benchmark file.

Fixed it. Merging!


engiborgs should be insulated

Description
or have an insuls module so they can actually do anything

Borgs don't need insuls iirc

They already are insulated? You can cut wires and pulse doors just fine as a Borg idk what you’re on about

they arent insulated i had an engiborg cutting a wire and got shocked

Last night I cut the HV cable under the AME so I have no idea why it’s doing that

on master, happens on both tool module and advanced tool module @mirrorcult reopen
most likely introduced by borg tweaks

glub

actually got fixed


http request node should enable options for POST, PUT, PATCH request body

It seems the msg.payload is the only choice for the request body of the http request node. It would be nice if we were able to send the entire msg so we could consider adding other fields to it when calling a REST service.

Hi, thanks for raising an issue.
If you check out our contribution guidelines you'll see we prefer to discuss things on the mailing list or slack teamin the first instance, rather than raise an issue.
Nothing stopping you from setting msg.payload to be an object with whatever fields you want to send. There's no pressing reason to expose an 'entire message' option.
Thanks!

Thanks -- I tried setting msg.payload=msg.toString(); and that didn't work but I understand what you've suggested (e.g., being selective about which fields to include). Thanks.

The intended behaviour is you set msg.payload to the body of the request. More often than not, the complete msg will include properties you don't want going near the request. I don't think there's anything more to consider here. Please reopen if you think there's more to address.


Read-only text block for forms

Quite frequently there's a need to display some value in the form, it could be achieved by making an input read only (there was a proposal to add read-only prop for TextField: #9790), but it would still have the underline, also, the presented text can have formatting, links, images, etc., which is impossible to do with input.
So far I used the following code as temp solution:
import {styles as inputBaseStyles} from '@material-ui/core/InputBase/InputBase';

const ReadOnly = withStyles(inputBaseStyles)(({children, classes, className, ...props}) => (

{children}

));

So that it could be used as follows:
<FormControl>
    <InputLabel shrink>Label</InputLabel>
    <ReadOnly>Foo</ReadOnly>
</FormControl>

Does it make sense to add such component to the lib? Is it safe to use default styles like this?
PS. There is a special class in bootstrap for such case: http://getbootstrap.com/docs/4.1/components/forms/#readonly-plain-text

Ping

I'm not sure if I get this right:
You want to display read-only text field, with the Material styling?
Or you want to "display anything you want" but styled as if it was a Material styled text field?

@kirill-konshin What was https://github.com/mui-org/material-ui/issues/9790#issuecomment-356193517 lacking?
Displaying an image in TextField would be a different issue. Do you mean something like adornments? Could you showcase such a use case?.

I was not talking about adornments, I was looking for something that allows to display any content with an input label on top. The implementation that I've supplied to this ticket does exactly this.

But that's not really an input not even mentioning a textfield. It's just a generic container with a label. I was interested in a specific use case that relates to input provided by the user.

Case 1. Read-only phrase with edit button
Case 2. Custom control like an MP3 player
Case 3. A checkbox or set of checkboxes or radios under common label

Looks like this :)

Case 1. Read-only phrase with edit button

Sounds like https://github.com/mui-org/material-ui/issues/9790#issuecomment-356193517 would solve this. What is missing?

Case 2. Custom control like an MP3 player

That's very specific use case where an input is not the appropriate abstraction

Case 3. A checkbox or set of checkboxes or radios under common label

This should be covered by https://material-ui.com/demos/selection-controls/#radio-buttons
I feel like your example is perfectly fine. It's a generic container with a label.

https://github.com/mui-org/material-ui/issues/9790#issuecomment-356193517 can only have text, no hyperlinks, no images, no custom tags, so this does not work.
Fieldset+legend (https://material-ui.com/demos/selection-controls/#radio-buttons) could be an option as a generic container with a label, but it has different visual style than a textfield's label, that's my main concern...

https://codesandbox.io/s/l7vykxl57q

Actually, this issue is coupled with https://github.com/mui-org/material-ui/issues/13643, forgot to mention that )

Sorry for confusion.
This particular ticket is only about the ReadOnly container that looks like an Input but can have rich formatting, not just text.
The container discussion is a separate ticket: #13643.

This seems to be a duplicate of #9790. The readOnly prop is supposed to cover this problem.

No, readOnly prop just makes field to become read only, it does not allow to have HTML markup in the block. I commented about that earlier in this thread:

#9790 (comment) can only have text, no hyperlinks, no images, no custom tags, so this does not work.


Core BB automated testing

Hi @shurwit, @petyos,
These are some ideas I have for how we can efficiently test our storage adapter functions on the Core BB and other building blocks. Please take a look at this when you have time and let me know if you have any questions about this strategy or any additional suggestions. Thanks!
General Strategy

Have MongoDB connection, maintain directory of JSON documents to initialize database with standard testing data. Initialize DB using InsertMany on each collection.
Maintain another directory of JSON documents containing the expected results of the tests. Specify only changed fields in result JSON, apply these changes to original JSON, then compare with test result for correctness.
Allow default result checking to be overridden or combined with programmatic checks for each test.
Use DeleteMany, then InsertMany on modified collections to restore to original data between tests (unless there is a more efficient way to take snapshots and restore from them).
Tear down test DB.

Find

Check document(s) against documents in initial JSON by id(s)

Replace

Do replace, then find replaced doc(s) to verify correctness

Update, FindAndUpdate

Do update, then find updated doc(s) to verify correctness

Insert

Do insert, then find inserted doc(s) to verify correctness

Delete

Do delete, then find to make sure deleted doc(s) no longer exists

CountDocuments

Do count, verify against number of documents in JSON matching filter

Additional Considerations (probably to be done after implementing General Strategy)

How to make sure collection document data only changes when it should?

Watch all collections, throw error if updates occur to unexpected collection (Problem: may be hard to know which tests trigger which changes because change streams are asynchronous)
Track all collection-level operations that were performed for a test (How to get a list of these operations?)

Query/index optimization testing

Use db.RunCommand in Go driver to evaluate query performance with explain documents (maybe use something similar to collection.go that returns command documents for RunCommand instead of actually performing command)

Hi @roberlander2 , thanks for your ideas. I like them. I also think that point 1 from Additional Considerations is important and should be covered. The key moment when we updating something in the database is to ensure that we have not modified any additional information, not just to verify that we have updated what we expect.

Hi @roberlander2 , thanks for your ideas. I like them. I also think that point 1 from Additional Considerations is important and should be covered. The key moment when we updating something in the database is to ensure that we have not modified any additional information, not just to verify that we have updated what we expect.

Thanks @petyos, I agree. I'll think some more on finding a good solution for this before going too deep into the rest of the implementation.

Hi @petyos, @shurwit
I think I found a way to track collection-level operations that are used for each test, which we can use to make sure collections are modified only when they should be. Here is a link to the command:
https://www.mongodb.com/docs/v4.4/reference/command/serverStatus/#mongodb-serverstatus-serverstatus.metrics.document
If I understand correctly, we can use the metrics.document object in the output of the serverStatus database command to know the number of documents inserted, updated, and deleted during a test. If we add expected values for these fields to the JSON containing expected results for each test, I think it should be easy to validate for correctness.
What do you think of this plan?


Object.keys error in IE11

After getting latest version of grapesjs v0.15.3
Getting error in IE11
Object.keys: argument is not an object
Please give me a fix
Have a look on attached screenshot

@senchden Thanks

No problem.


Help wanted

Hi,
I very much interested to integrate fetch to in my app. My scenario is we have 4 activities extends from one base activity and all activity's data passed to one adapter. That media adapter row's have download button. I want media adapter row download when clicked and change that download icon to download current download item have pause with circular progress-bar and queued items have queued symbol. When completed it will update icon.
Is it possible with fetch? ow can do it

@smart222 Yes it is possible with Fetch. Have you tried the sample app?.  What you do is create a new instance of Fetch in the Application class of your app and reference it in the different activities. What you can also do is create a single global instance of fetch and a global fetch listener and share that global listener with the different activities. Hope this helps.

@tonyofrancis could you add some samples for global listener and how to enqueue and state update adapter rows it will help me.

@smart222 The sample app does have global instance of Fetch. in the App.java class. That is referenced in other Activities. Probably best to take the time and go through the code to get a better understanding of how Fetch works. Cannot do the work for you.


[SPARK-16125][YARN] Fix not test yarn cluster mode correctly in YarnClusterSuite

What changes were proposed in this pull request?
Since SPARK-13220(Deprecate "yarn-client" and "yarn-cluster"), YarnClusterSuite doesn't test "yarn cluster" mode correctly.
This pull request fixes it.
How was this patch tested?
Unit test
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)

Test build #61013 has started for PR 13836 at commit 7820382.

That's right; let's try to get the other instances where yarn-* is used incorrectly too.

Test build #61013 has finished for PR 13836 at commit 7820382.

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

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

Merged build finished. Test PASSed.

Test build #61021 has started for PR 13836 at commit 6c60d5a.

Test build #61021 has finished for PR 13836 at commit 6c60d5a.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Jenkins retest this please

Test build #61029 has started for PR 13836 at commit 6c60d5a.

Test build #61029 has finished for PR 13836 at commit 6c60d5a.

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

Merged build finished. Test PASSed.

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

Please put SPARK-16125 in the title

Test build #61101 has started for PR 13836 at commit 8d2dea7.

Test build #61101 has finished for PR 13836 at commit 8d2dea7.

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

Merged build finished. Test PASSed.

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

I'm merging into 2.0 as well as master because this is a clear and outright bug fix


Update README with ArchLinux install instructions.

Hi, I just created a package for Archlinux, and I'm using it (so this mean it should keep up to date)
Could you check if I miss some dependency?

I tested with archlinux inside a docker container and it looks fine to me, even though I cannot check with a proper archlinux setup.


corporate: Tighten coverage checks in decorator.py.

@mateuszmandera FYI, getting this to 100% can be a low-priority task on your TODO list.


OkEx heartbeat monitoring

OkEx.Stream.Connection sends pings but doesn't monitor if matching pongs are received.
This can sometimes lead to stale data, and it takes ~20 minutes before websockex quits. You can see this on the chart in the flat regions. At the end of flat period, you get a Tai.StreamDisconnect, and everything starts moving again.

This change means that if a pong hasn't been received in the last 30 seconds, terminate the socket.
Notes:

My own version of Tai has diverged from master, so I've done my best to port over my changes. I wasn't able to test it with master, but it works just fine with my version.
This is my first PR ever, so please let me know if I've done anything wrong or could do better.

Thanks!

@chrism2671 oh also, do you mind submitting future PR's from a feature branch? It looks like the CI status badge picks up the status from forks when they're submitted on the master branch

@chrism2671 updated the Github action so that your tests should no pass. Can you rebase with master. Also can you squash your 3 commits into 1?

Thanks! It's exciting to be able to contribute something!
I've made the changes, hopefully it's a bit better now :)

@chrism2671 💥. I think I posted this in your previous issue, but if you every publish something that's open source that uses tai, I'd love to add it to the built with tai section.


Nuget error upgrading to 1.4.30 - The "=" operator is missing after a named argument.

When using Visual Studio 2013 to update to the new Nerdbank.GitVersioning.1.4.30 package, I get the following PowerShell error printed in the NuGet Package Manager output window in VS.
The error appears to be benign for me, but that might be because i am doing package upgrade and so my project already has version.json file.
HTH.
The "=" operator is missing after a named argument.
At E:\Depot\GitHub\ServerHost\packages\Nerdbank.GitVersioning.1.4.30\tools\Create-VersionFile.ps1:15 char:38
+ [CmdletBinding(SupportsShouldProcess) <<<< ]
    + CategoryInfo          : ParserError: (:) [], ParseException
    + FullyQualifiedErrorId : MissingEqualsInNamedArgument

------- Updating...Nerdbank.GitVersioning 1.4.30 -------
Installing 'Nerdbank.GitVersioning 1.4.30'.
Added file 'AppVeyor.targets' to folder 'Nerdbank.GitVersioning.1.4.30\build'.
Added file 'CodeGenerationAttributes.Net40.dll' to folder 'Nerdbank.GitVersioning.1.4.30\build'.
Added file 'Nerdbank.GitVersioning.targets' to folder 'Nerdbank.GitVersioning.1.4.30\build\dotnet'.
Added file 'LibGit2Sharp.dll' to folder 'Nerdbank.GitVersioning.1.4.30\build'.
Added file 'git2-e0902fb.dll' to folder 'Nerdbank.GitVersioning.1.4.30\build\NativeBinaries\amd64'.
Added file 'git2-e0902fb.dll' to folder 'Nerdbank.GitVersioning.1.4.30\build\NativeBinaries\x86'.
Added file 'Nerdbank.GitVersioning.dll' to folder 'Nerdbank.GitVersioning.1.4.30\build'.
Added file 'NerdBank.GitVersioning.targets' to folder 'Nerdbank.GitVersioning.1.4.30\build'.
Added file 'Nerdbank.GitVersioning.Tasks.dll' to folder 'Nerdbank.GitVersioning.1.4.30\build'.
Added file 'Newtonsoft.Json.dll' to folder 'Nerdbank.GitVersioning.1.4.30\build'.
Added file 'PInvoke.MSCorEE.dll' to folder 'Nerdbank.GitVersioning.1.4.30\build'.
Added file 'Nerdbank.GitVersioning.targets' to folder 'Nerdbank.GitVersioning.1.4.30\build\portable-net+win+wpa+wp+sl+netmf+MonoAndroid+MonoTouch+Xamarin.iOS'.
Added file 'Validation.dll' to folder 'Nerdbank.GitVersioning.1.4.30\build'.
Added file 'VisualStudioTeamServices.targets' to folder 'Nerdbank.GitVersioning.1.4.30\build'.
Added file 'readme.txt' to folder 'Nerdbank.GitVersioning.1.4.30'.
Added file 'Create-VersionFile.ps1' to folder 'Nerdbank.GitVersioning.1.4.30\tools'.
Added file 'Get-CommitId.ps1' to folder 'Nerdbank.GitVersioning.1.4.30\tools'.
Added file 'Get-Version.ps1' to folder 'Nerdbank.GitVersioning.1.4.30\tools'.
Added file 'Install.ps1' to folder 'Nerdbank.GitVersioning.1.4.30\tools'.
Added file 'version.schema.json' to folder 'Nerdbank.GitVersioning.1.4.30\tools'.
Added file 'Nerdbank.GitVersioning.1.4.30.nupkg' to folder 'Nerdbank.GitVersioning.1.4.30'.
Successfully installed 'Nerdbank.GitVersioning 1.4.30'.
Looking for updates for 'Nerdbank.GitVersioning'...
Updating 'Nerdbank.GitVersioning' from version '1.4.19' to '1.4.30' in project 'ServerHost'.
Removing 'Nerdbank.GitVersioning 1.4.19' from ServerHost.
Added file 'packages.config'.
Successfully removed 'Nerdbank.GitVersioning 1.4.19' from ServerHost.
Adding 'Nerdbank.GitVersioning 1.4.30' to ServerHost.
For adding package 'Nerdbank.GitVersioning 1.4.30' to project 'ServerHost' that targets 'net45',
>> Build files are being added from 'build\portable-net+win+wpa+wp+sl+netmf+MonoAndroid+MonoTouch+Xamarin.iOS'
Added file 'packages.config'.
Successfully added 'Nerdbank.GitVersioning 1.4.30' to ServerHost.
For adding package 'Nerdbank.GitVersioning 1.4.30' to project 'ServerHost' that targets 'net45',
>> PowerShell scripts are being executed from 'tools' (not framework-specific)
Executing script file 'E:\Depot\GitHub\ServerHost\packages\Nerdbank.GitVersioning.1.4.30\tools\Install.ps1'.
Uninstalling 'Nerdbank.GitVersioning 1.4.19'.
Successfully uninstalled 'Nerdbank.GitVersioning 1.4.19'.
==============================

Closing this in favor of #66 which has more data.


isort behavior different on linux vs osx

OSX:
pipenv run isort server/portfolio_manager/management/commands/safe_migrate.py

from django.core.management import call_command
from django.core.management.base import BaseCommand, CommandError

from db_mutex import DBMutexError, DBMutexTimeoutError
from db_mutex.db_mutex import db_mutex

Ubuntu 16.04:
pipenv run isort server/portfolio_manager/management/commands/safe_migrate.py

from db_mutex import DBMutexError, DBMutexTimeoutError
from db_mutex.db_mutex import db_mutex
from django.core.management import call_command
from django.core.management.base import BaseCommand, CommandError

Perhaps there is a setting that would force OSX to behave like ubuntu that I'm missing?

I resolved it with
[settings]
known_django=django
sections=FUTURE,STDLIB,DJANGO,THIRDPARTY,FIRSTPARTY,LOCALFOLDER

Still seems like the defaults should work on both platforms.

This is a duplicate of #725. Can you please close it?


glide: update hash of glide.lock

The hash in glide.lock was out of date causing the following warning on doing glide install --strip-vendor
Lock file may be out of date. Hash check of YAML failed. You may need to run 'update'

Most probably it was caused by not updating the lock file properly in #1474
https://github.com/coreos/etcd-operator/pull/1474/files#diff-f16a80eae23d5b298c2652448ec420cfR1

lgtm


Trouble with Python Linting

Issue Type: Performance Issue
Hi there,
I have downloaded VSCode on my MacOS, I downloaded the Python extension, and I downloaded a Python 3.8.5 interpreter. The probelm is when I try to run linting on code that clearly has errors, nothing shows up.
When I created a .py file for the first time, there was an error message about the linting...something about the PATH, but I am new to coding and disregarded that, which I realize was a mistake. Can someone help me figure out how to fix this?
Thanks.
VS Code version: Code 1.47.2 (17299e413d5590b14ab0340ea477cdd86ff13daf, 2020-07-15T18:18:50.054Z)
OS version: Darwin x64 17.7.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-2415M CPU @ 2.30GHz (4 x 2300)

GPU Status
2d_canvas: unavailable_softwareflash_3d: disabled_softwareflash_stage3d: disabled_softwareflash_stage3d_baseline: disabled_softwaregpu_compositing: disabled_softwaremetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: disabled_offrasterization: disabled_softwareskia_renderer: disabled_off_okvideo_decode: disabled_softwareviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: unavailable_softwarewebgl2: unavailable_software

Load (avg)
4, 4, 3

Memory (System)
16.00GB (7.70GB free)

Process Argv
-psn_0_6288895

Screen Reader
no

VM
0%

Process Info
CPU %	Mem MB	   PID	Process
   13	    98	 12537	code main
    0	    82	 12622	   gpu-process
    0	    33	 12626	   utility
    1	   328	 12628	   window (settings.json — VSCode)
    0	   180	 12636	     extensionHost
    0	    82	 12643	       /private/var/folders/qm/5xvmw_ls2015fkvn370pmlf80000gn/T/AppTranslocation/D5647B77-0A20-40CF-A117-98D0A951B812/d/Visual Studio Code.app/Contents/Frameworks/Code Helper (Renderer).app/Contents/MacOS/Code Helper (Renderer) /private/var/folders/qm/5xvmw_ls2015fkvn370pmlf80000gn/T/AppTranslocation/D5647B77-0A20-40CF-A117-98D0A951B812/d/Visual Studio Code.app/Contents/Resources/app/extensions/json-language-features/server/dist/node/jsonServerMain --node-ipc --clientProcessId=12636
    0	    16	 12718	       /Library/Frameworks/Python.framework/Versions/3.8/Resources/Python.app/Contents/MacOS/Python /Users/benroy/.vscode/extensions/ms-python.python-2020.7.96456/pythonFiles/pyvsc-run-isolated.py /Users/benroy/.vscode/extensions/ms-python.python-2020.7.96456/pythonFiles/completion.py
    0	    16	 12720	         /Library/Frameworks/Python.framework/Versions/3.8/Resources/Python.app/Contents/MacOS/Python /Users/benroy/.vscode/extensions/ms-python.python-2020.7.96456/pythonFiles/lib/python/jedi/inference/compiled/subprocess/__main__.py /Users/benroy/.vscode/extensions/ms-python.python-2020.7.96456/pythonFiles/lib/python 3.8.5
    0	    33	 12641	     watcherService
    0	    49	 12681	     searchService
    0	   131	 12640	   shared-process
    0	     0	 12840	     /bin/ps -ax -o pid=,ppid=,pcpu=,pmem=,command=
    0	    82	 12830	   window (Issue Reporter)

Workspace Info
|  Window (settings.json — VSCode)
|    Folder (VSCode): 4 files
|      File types: json(1) py(1)
|      Conf files: settings.json(1);

Extensions (1)

Extension
Author (truncated)
Version

python
ms-
2020.7.96456

I figured it out with Homebrew.


Poltergeist times out on page.driver.*cookie

Hi I am using Poltergeist 1.6.0 and I cannot resolve a timeout issue. Specifically, when calling page.driver.cookies or page.driver.set_cookie, Poltergeist times out. I've extended the timeout option to 1000 seconds with no effect. I also ensure that I have visited a page prior to calling these methods. My exact usage (edited for brevity):
  before(:each) do
    visit '/lrm'

    expect(lrm.el).to have_content 'Inception Version 3.0.13'

    case Capybara.current_driver
    when :selenium
      manager = page.driver.browser.manage

      manager.add_cookie(name: 'inception_user',  value: username)
      manager.add_cookie(name: 'inception_lab',   value: 'lab')
      manager.add_cookie(name: 'inception_persb', value: 'lab')
    when :poltergeist
      page.driver.set_cookie('inception_user',  username)
      page.driver.set_cookie('inception_lab',   'lab')
      page.driver.set_cookie('inception_persb', 'lab')
    end

    visit '/lrm'

    find('#queryPortlet')
  end

Provide debug log, you can read about it in readme

So it seems I run into the issue in the Pry console as the responses are clearly successful. When I run page.driver.cookies in the debugger it hangs. Also, the cookies don't seem to be set but this could be an unrelated issue.
{"name":"set_js_errors","args":[false]}
{"response":true}
{"name":"set_debug","args":[true]}
{"response":true}
{"name":"visit","args":["http://<IP_ADDRESS>:63842/lrm"]}
{"response":{"status":"success"}}
{"name":"find","args":["xpath","/html"]}
{"response":{"page_id":1,"ids":[0]}}
{"name":"visible","args":[1,0]}
{"response":true}
{"name":"visible_text","args":[1,0]}
{"response":"\nUser Id:\n\n\nPassword: (Windows, not Unix)\n\n\nSelect your area:\n Lab (LRM)\n Execution (Mosaic)\n\nTest Technologies PortalWelcome\nInception Version 3.0.13. Copyright © 2010, Juniper Networks, Inc."}
{"name":"visit","args":["http://<IP_ADDRESS>:63842/lrm"]}
{"response":{"status":"success"}}
{"name":"current_url","args":[]}
{"response":"http://<IP_ADDRESS>:63842/portal/authenticate/login?url=/lrm"}
{"name":"set_cookie","args":[{"name":"inception_user","value":"jpfosi","domain":"<IP_ADDRESS>"}]}
{"response":true}
{"name":"current_url","args":[]}
{"response":"http://<IP_ADDRESS>:63842/portal/authenticate/login?url=/lrm"}
{"name":"set_cookie","args":[{"name":"inception_lab","value":"lab","domain":"<IP_ADDRESS>"}]}
{"response":true}
{"name":"current_url","args":[]}
{"response":"http://<IP_ADDRESS>:63842/portal/authenticate/login?url=/lrm"}
{"name":"set_cookie","args":[{"name":"inception_persb","value":"lab","domain":"<IP_ADDRESS>"}]}
{"response":true}
{"name":"visit","args":["http://<IP_ADDRESS>:63842/lrm"]}
{"response":{"status":"success"}}
{"name":"find","args":["css","#queryPortlet"]}

What are you expecting to see in the log you posted?

I'm not even sure now. I switched to Capybara Webkit. I'll update this if I encounter this bug again


Is there a way to validate input?

I have a username field which I want to validate on creating and editing to see if the username already exists in any other rows (i.e. usernames have to be unique). On edit, if the username already exists, I want to be able to alert that to the user and then save data with the default username.

+1

Hey Guys, please check out the (editConfirm) event https://akveo.github.io/ng2-smart-table/documentation in which you can confirm or reject the changes.
Here's an example https://github.com/akveo/ng2-smart-table/blob/master/demo/src/app/pages/demo/components/advanced-example-confirm.component.ts#L139

+1

Is it possible to get working links again ? I really want to implement cell validation :)

https://github.com/akveo/ng2-smart-table/blob/master/src/app/pages/examples/various/advanced-example-confirm.component.ts

https://github.com/akveo/ng2-smart-table/blob/master/src/app/pages/examples/various/advanced-example-confirm.component.ts

https://github.com/akveo/ng2-smart-table/blob/master/projects/demo/src/app/pages/examples/various/advanced-example-confirm.component.ts


Enhanced HTML-CSS Autocompletion Features for Neovim

Hello everyone,
I have been working on a fork of the HTML-CSS autocompletion plugin for Neovim, and I'm excited to share some enhancements that might be beneficial for the community. My fork can be found here: https://github.com/ESSO0428/nvim-html-css.

Enhancements include:

External Git Repository Support: The plugin now operates smoothly outside of a git repository, maintaining a performance that is robust and efficient.It supports autocompletion for both CSS classes and ids, enhancing coding efficiency.
Selective File Reading: Instead of loading the entire working directory, the plugin now only reads the style tags and linked CSS from the currently open HTML file. This significantly reduces performance overhead.
Asynchronous Operations and CSS File Monitoring: Implemented mechanisms for asynchronous operations and monitoring changes in CSS files to improve responsiveness and reduce lag.
Caching for Remote URLs: To enhance performance, the plugin now downloads and caches files from remote URLs, speeding up subsequent accesses to these files.
Enhanced Template Support for htmldjango: In addition to the above, for htmldjango files (*.html), the plugin will also read templates extended by the current file, including any href or style tags within those templates.
Completion Item Symbols:

Items from the style tags in the current HTML file are marked with a star symbol (★) to indicate their source.
Items from remote CSS files linked (link tags) via href in previously accessed HTML files are marked with a globe symbol (🌐) to denote their remote origin.
Completion items from local CSS files linked (link tags) via href in any HTML file do not have a symbol, maintaining a clean and simple identification.

I hope these enhancements will be helpful for your ongoing development and perhaps could be integrated into the original project. I'm looking forward to any feedback or suggestions you might have!
Best regards,
Thank you to the development team for your hard work and dedication.

Thank you for sharing these amazing enhancements. They will be very helpful for the upcoming development of the plugin. I have a bunch of ideas to improve the user experience and plugin performance, and these enhancements will definitely be beneficial in implementing those ideas. Once again, thank you for sharing!


Custom Domains Allowed in okta_org_url But Not app_url

Hey folks,
It seems _get_org_url_entry was updated to allow custom domains by validating /.well-known/okta-organization, but _get_appurl_entry is still looking for the specific allowlist domains (okta.com, oktapreview.com, okta-emea.com).
This appears to only affect --action-configure. If I manually configure app_url in my .okta_aws_login_config, it accepts the custom domain without complaint.
Expected Behavior

--action-configure should allow custom domains to be used in app_url.
Current Behavior

Receive an error from --action-configure:
Okta organization URL must be HTTPS URL for okta.com or oktapreview.com or okta-emea.com domain

which originates here: https://github.com/Nike-Inc/gimme-aws-creds/blob/094a3292e3b6bf63006f63043a86a757fba06ce5/gimme_aws_creds/config.py#L421
(It appears the warning is also referring to the wrong entry?)
Possible Solution

Update _get_appurl_entry with the same logic as _get_org_url_entry, or to validate the app_url against the okta_org_url instead of the static allowlist?
Steps to Reproduce (for bugs)

Attempt to provide a custom domain to --action-configure's "Application url" prompt.

Context

My team is currently configuring .okta_aws_login_config with our custom Okta domain, to work around the --action-configure limitation.
Your Environment

App Version used: 2.7.2
Environment name and version: MacBook Pro Apple M1 Max
Operating System and version: MacOS 13.5.2

The error you're getting is because the provided URL didn't start with https:// - if the /.well-known/okta-organization file isn't found, you'll see the following message: {okta_url} is not a valid Okta domain

Hi, I'm a developer working at Warner Bros. Discovery and we've come across the same bug.
I have seen the same behaviour with a proper corporate OKTA https URL and looking at the gimme-aws-creds code, I come to the same conclusion as @straub.
Can you please reopen this issue?
cc. @epierce


Screener questionnaire ui alignment issues.

IMPORTANT: All PRs must be linked to an issue (except for extremely trivial and straightforward changes).
Fixes #625
Description
Add fix for ui alignment issue for below questionnaire.
Symptoms, Medical History.
Include hidden extension to hide questionnaire items.
symptoms, medical history, vital signs.
Modify Risk factors questionnaire json structure  to support definition based resource extraction.
Alternative(s) considered
no
Type
Choose one: Feature
Screenshots (if applicable)
Checklist

[x] I have read and acknowledged the Code of conduct
[x] I have read How to Contribute
[x] I have read the Developer's guide
[x] I have signed the Google Individual CLA, or I am covered by my company's Corporate CLA
[x] I have discussed my proposed solution with code owners in the linked issue(s) and we have agreed upon the general approach
[x] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the style guide of this project
[x] I have run ./gradlew check and ./gradlew connectedCheck to test my changes locally
[x] I have built and run the reference app(s) to verify my change fixes the issue and/or does not break the reference app(s)

Codecov Report

Merging #785 (1b9d908) into master (14ab1d1) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 1b9d908 differs from pull request most recent head b390197. Consider uploading reports for the commit b390197 to get more accurate results

@@            Coverage Diff            @@
##             master     #785   +/-   ##
=========================================
  Coverage     52.96%   52.96%
  Complexity      197      197
=========================================
  Files            48       48
  Lines          1027     1027
  Branches        146      146
=========================================
  Hits            544      544
  Misses          396      396
  Partials         87       87

Continue to review full report at Codecov.

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

Can we please review it.


python3Packages.ocrmypdf: 13.0.0 -> 13.1.0

Motivation for this change
https://github.com/jbarlow83/OCRmyPDF/blob/v13.1.0/docs/release_notes.rst
Things done

Built on platform(s)

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

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

[x] Fits CONTRIBUTING.md.

The release notes mention a bunch of dependencies are not necessary anymore (the cffi stuff). Can you update the deps?

Oops, I missed the removal of cffi with the previous update. And the dependency on setuptools was also dropped a while back.


Symbol Paths Not Respected?

I'm having a lot of difficulties with symbol server support.

It looks like the checkbox for _NT_SYMBOL_PATH is not respected? The actual API call is pIDiaDataSource->loadDataForExe in pdb_file.cpp. The value of symStoreBuffer appears to contain the value of _NT_SYMBOL_PATH even if it's not checked.

I'm not sure if the string value in "public store" of symbol settings is used anywhere? Although the documentation is unclear.

I expect I should be able to specify a custom symbol server by either adding it to _NT_SYMBOL_STORE OR putting the URL in the "Public Store" field. But maybe that's wrong? It's not clear to me how public store / local store are actually used in code.

should be fixed in v4.8.3


[Question] Do you consider supporting spark?

Question
是否考虑支持spark？

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑🤝🧑👫🧑🏿🤝🧑🏻👩🏾🤝👨🏿👬🏿

Question
Are you considering supporting spark?

hi, the spark monitoring is in progress now. https://github.com/dromara/hertzbeat/issues/990


Django 5+Support

Hi everyone,
Hope you're all doing well!
I'm using the latest Django version, Django 5.0.6, and I've encountered an issue:

ERROR: Cannot install -r requirements.txt
Django==5.0.6 because these package versions have conflicting dependencies.
django4-inline-actions 1.7 depends on django<4.3 and >=3.0

@HiroshiFuu, do you plan to release a version compatible with Django 5+?
Thanks!

Actually, yes. And actually, I plan to upgrade it when Django 5.1 is released. But never mind, I will test it with Django 5.0 tomorrow or the day after tomorrow.

Any updates?

Any updates?

Sorry for the late replay, I have added Django 5.
https://pypi.org/project/django4-inline-actions/

Thanks @HiroshiFu 🦾


Should this live in Threat Dragon

I had originally created this as a separate repository as I was learning the deeper inner workings of Jekyll and doing a PoC for Threat Dragon's v2 docs.
I licensed it under the same license as Threat Dragon to avoid any conflicts if we wanted to migrate it to TD.  Threat Dragon's repo has become more of a mono repo in the past year, so to me, it'd probably make sense to have this all in one place.
Thoughts, @jgadsden ?

Sounds good to me either way. Perfectly fine with me if you want to keep this theme here, after all it is your creation
We can put a note in the dev notes for version 2.0 Threat Dragon that the theme lives here ... but if you want to combine them that is also perfectly good
I guess if they were in the same place then a push to the theme would trigger a refresh of https://www.threatdragon.com/docs/ , which at the moment (I think) waits for a push to v2-development/docs before updating?

I guess if they were in the same place then a push to the theme would trigger a refresh of https://www.threatdragon.com/docs/ , which at the moment (I think) waits for a push to v2-development/docs before updating?

That's a good enough reason in and of itself.  I honestly don't think I even have any github actions set up here to deploy the new gem, so it'd currently rely on me pushing a gem and then a rebuild of v2-development... Very fragile. :-1:
I'll migrate it over soon and get it well-integrated. :smile:

OK, thanks - much appreciated as always @lreading


Improve access to mysqldump when server isn't localhost

Improve access to mysqldump when server isn't localhost

I've done some (unsuccessful) research into this. Before wasting too much time on it, ping me and I'll share what I've tried.


Question: Proper way to make a WASM build

Hi, I was very excited to here this could run in wasm and wanted to test it out, first to see relative byte code sizes I could expect before embedding it in my program. However, I had a number of errors compiling (on Rust 1.62). The minimal version that finally worked was:
cargo build --release --target wasm32-unknown-unknown --no-default-features
But I think I really would want:
cargo build --release --target wasm32-unknown-unknown --no-default-features --features wasm-bindgen
The following variants all errored:

cargo build --release --target wasm32-unknown-unknown
cargo build --release --target wasm32-unknown-unknown --features wasm-bindgen

   Compiling getrandom v0.2.7
error: the wasm32-unknown-unknown target is not supported by default, you may need to enable the "js" feature. For more information see: https://docs.rs/getrandom/#webassembly-support

cargo build --release --target wasm32-unknown-unknown --features wasm-bindgen,no_std --no-default-features also errored:

error: `wasm-bindgen` cannot be used with `no-std`
   --> src/lib.rs:447:1
    |
447 | compile_error!("`wasm-bindgen` cannot be used with `no-std`");

It seems funny to disable both std and no_std to get it to compile. Is this the proper approach?
NB: I looked at the CI Scripts to see how you build wasm there, and it is not included. May be nice to add a small section there, to at least test it compiles to wasm properly (even if it doesn't spin up node server and run the wasm)

You can also consult: https://rhai.rs/book/start/builds/wasm.html

Ah, I get it now.  There is a problem in recent versions due to the ahash dependency.  ahash changed the way feature flags are used and caused all kinds of problems with builds.
The problem is, ahash/runtime-rng which is enabled by std does not seem compatible with wasm.  I'll have to think of a way to do this elegantly...
In the meantime, you're correct to find that you  must exclude std via --no-default-features.  You probably should also include ahash/compile-time-rng.

Thank you for this advice.
I made a PR that should address this as well as I could: https://github.com/rhaiscript/rhai/pull/653 (both CI tests as well as feature changes). I tried to unify std feature to enable runtime or compiletime ahash rng based on target, but hit the limits of Cargo.toml (or my knowledge and google ability).
We should add docs, but now the following two would work well for Wasm builds:
cargo build --target wasm32-unknown-unknown --features js (compiles std, with browser get random)
cargo build --target wasm32-unknown-unknown --no-default-features --features wasm-std (almost the same as std, but with compile-time-rng)
Happy for feedback on this approach and how to cleanup / improve it.

P.S. you can see the CI runs here: https://github.com/ethanfrey/rhai/pull/1

Great. With #656 and #659, this works. For the record, here is how.
If you can use wasm-bindgen, this will work (targeting web):
wasm build --target wasm32-unknown-unknown --features wasm-bindgen
If you want to make a wasm build with no imports (custom runtime), you need to disable access to png and system time. You can do like this:
wasm build --target wasm32-unknown-unknown --no-default-features --features std,no_time

If you can use wasm-bindgen, this will work (targeting web):
wasm build --target wasm32-unknown-unknown --features wasm-bindgen

Unfortunately you'd also need to pull in. getrandom/js to get it to work on browser.


MemoryPooling requires hash-able objects?

I was getting into messing with memory pools and noticed became curious about a method DespawnAll(), so I peeked in to see what it did. What I found was a HasSet<TContract> to hold active items and checks to ensure add and removal took place.
Does this mean that objects can only be pooled if they implement GetHashCode()? If so, this decision seems odd to me, and actually breaks what I was planning on using the pooling for (Well, not break. More force me to add an arbitrary incremental int to a base class to act as a hash. I suppose this is the intended solution. But I am still curious about the decision).

Silly reason to make an issue for something that wasn't really a problem. Was just curious was all.

oh?

I think you're right, we shouldn't use HashSet.  I think we should maybe just remove support for DespawnAll entirely to avoid needing to store the active set.

Related:  https://groups.google.com/forum/#!topic/zenject/iu1Iw1-Umeo

Fixed by removing HashSet entirely

Hmm, but how am i now supposed to despawn all?

You can't and that's intentional.  We want to be able to spawn items from a memory pool and then decide whether or not to return it to the pool.  If we don't return it to the pool, we want to be able to just stop referencing it and then let the garbage collector get rid of it (assuming it's a non monobehaviour).  If we added this feature to despawn all automatically then we couldn't also support making returning to the pool optional, and the latter seems like the better feature to support


Can not download torrent file

I'am trying to download a .torrent from butter.vodo.net API by passing the torrent URL to client.add but i'am getting CORS error, also it seems that neither instant.io nor βTorrent can handle this. Is there any workaround to get it working?

The webserver needs to be CORS-Enabled. There's no way to access resources from that domain through XHR from another without it. You could ask the domain owner to add CORS. Or if you want to have it on your own site have this small read: https://medium.com/@diegorbaquero/how-to-create-a-swarm-cdn-for-free-with-webtorrent-bfa09d193f71


Searchability Improvement: Delete a stack in Pulumi Cloud

The documentation for how to delete a stack in the Pulumi Cloud is not easily searchable on the Pulumi docs website.
If I enter 'delete stack" into the search, the following results are shown:

If I enter 'delete stack cloud', no results are shown.

If I enter just 'delete', it is included in these results but much further down the list.

If I enter "deleting a stack", it is the first option.

Thank you for writing this up @toriancrane! This is great feedback.

@interurban Assigning to you, probably needs to go to the Docs board


Add CORS

Hello again 👋🏻
I can't use fetch without 'Access-Control-Allow-Origin' header

Thanks! Fixed with https://github.com/rlindskog/covid19-graphql/pull/6

This issue seems to have cropped up again randomly. Not sure why as I can't see any recent changes in this repo or mine.


adding myissues feature and initial support for Issue#jql searching

This adds a new feature/route called myissues. It allows a user, if identified, to retrieve a list of assigned issues to them from JIRA that are 'active'. I take active to mean everything but tickets that are in the Closed status of a workflow.
It also adds initial support for Issue#jql from https://github.com/sumoheavy/jira-ruby/blob/master/lib/jira/resource/issue.rb#L47
This could eventually be leveraged to support more generic JIRA searching, but that seemed to be biting off too much for a single PR.

:+1: Thanks for this, looks awesome!


Add exitFunction capability to GracefulCluster

Our Express application has some processes that run in parallel to the worker processes that we need to gracefully shut down before exiting. This PR adds the ability to specify an exit function that is called whenever the cluster is ready to exit that can be used instead of the default of process.exit(0).
I also made the documentation look a bit prettier on GitHub by using file format tags and formatting the options as a table.

@ziggythehamster thanks for PR! Published in npm as 0.0.4.


Can't dump/restore with user and password

Using arangodb 2.6.8, i am unable to dump/restore db + collection using user and password
Here are the commands i use.  The sample dump can be downloaded as zip here http://dl.free.fr/n5nhqFNM2
arangodump --server.endpoint tcp://<IP_ADDRESS>:8529 --output-directory "dump" --server.username user --server.password password --server.database database --include-system-collections false --dump-data false --overwrite true
arangorestore --server.endpoint tcp://<IP_ADDRESS>:8529 --server.username user --server.password password --server.database database --create-database true --input-directory "dump"
Could not connect to endpoint tcp://<IP_ADDRESS>:8529
Error message: 'got error from server: HTTP 401 (Unauthorized)'

As far as I can see the dump is taken from server <IP_ADDRESS>:8529 with username user and password password from database database. I understood that this worked ok, because the error message above is printed after the arangorestore and not after the arangodump command invocation.
The restore however is done on the same server and same database and same user, but with a different password (qmcHwhHNPjfnOM1y). I don't think this is expected to work, as this user had a different password (password) when dumping, and I don't see the password being changed in between.
So either the example above is incorrect, or, if it is correct and two different passwords are used for dump and restore, then the failure is intentional. Or am I missing something?

Sorry example incorrect fixed

More infos : after restore cmd database is created (I can see it using the web interface), but i can't login using interface or nodejs.
So I can't say if collection exists

Ok, I was able to reproduce. The database needs to be dropped in between, and then re-creation failed. I have pushed a fix for the 2.6, 2.7 and devel branches. The fixes will be contained in the next releases (i.e. 2.6.10 and first 2.7.0 stable).

Sorry I forgot to mention the fact that database must be drop between the two commands.
Thank you for the fix ;)


BasicHistorySessions: Request Fine Location Permissions isn't shown

 Issue by cmunter
Monday Apr 18, 2016 at 12:24 GMT
Originally opened as https://github.com/googlesamples/android-fit/issues/16

Shouldn't BasicHistorySessions manifest include:
<uses-permission android:name="android.permission.ACCESS_FINE_LOCATION"/>
Otherwise Request Permissions isn't shown

 Comment by colinrtwhite
Friday Aug 05, 2016 at 22:14 GMT

The example doesn't have any permissions requested in its manifest. It should contain:
<uses-permission android:name="android.permission.ACCESS_NETWORK_STATE"/>
<uses-permission android:name="android.permission.ACCESS_FINE_LOCATION"/>
<uses-permission android:name="android.permission.INTERNET"/>


[Feature] Create a lookup operation

Create a lookup operation to read one or more resources from the cluster.

The purpose of this operation would be to read one (or more) resources from the cluster.
Then create outputs based on the retrieved resources.
I wonder if it would make sense to do that directly in assert and/or apply operations 🤔

The purpose of this operation would be to read one (or more) resources from the cluster.
Then create outputs based on the retrieved resources.
I wonder if it would make sense to do that directly in assert and/or apply operations 🤔

I'll give it a try and see how it works out.

@siddhikhapare i started working on this in #1089

@eddycharly  I used this to read resources from cluster as helm have lookup function
type Lookup struct {
    // Timeout for the operation. Overrides the global timeout set in the Configuration.
    // +optional
    Timeout *metav1.Duration `json:"timeout,omitempty"`
    // Namespace defines the namespace in which to search for the resources.
    // +optional
    Namespace string `json:"namespace,omitempty"`
    // Labels defines the labels to filter the resources.
    // +optional
    Labels map[string]string `json:"labels,omitempty"`
    // Kind defines the kind of resources to retrieve.
    // +optional
    Kind string `json:"kind,omitempty"`
    // Name defines the name of the resource to retrieve.
    // +optional
    Name string `json:"name,omitempty"`
    // APIVersion defines the API version of the resource to retrieve.
    // +optional
    APIVersion string `json:"apiVersion,omitempty"`
    // Template determines whether resources should be considered for templating.
    // +optional
    Cluster string `json:"cluster,omitempty"`
    // Cluster defines the target cluster (default cluster will be used if not specified and/or overridden).
    // +optional
    Template *bool `json:"template,omitempty"`
    // DryRun determines whether the operation should be performed in dry run mode.
    // +optional
    DryRun *bool `json:"dryRun,omitempty"`
    // Expect defines a list of matched checks to validate the operation outcome.
    // +optional
    Expect []Expectation `json:"expect,omitempty"`
}

@siddhikhapare the only thing i'm not sure about is how to deal with multiple resources (when working with labels for example)

@siddhikhapare the only thing i'm not sure about is how to deal with multiple resources (when working with labels for example)

@eddycharly can we create function to search resources of cluster by labels which specified in a Lookup struct using labelSelector?
It may be not needed here. You have completed lookup operation only test are remaining.

I'm still not sure how it will be useful, and we already have a get operation so 🤷
I think i will post pone to a later version until we find a use case for it.

I would like very much to be able to use outputs on the get operation. At present I use a command or script operator to run kubectl get.
Were this implemented, I would expect the resulting JMESPath context @ to differ between using get.name and get.selector:

get.name, found: @ is the object
get.name, not found: error? @ is null?
get.selector, found some: @ is an array of objects
get.selector, found none: @ is an empty array


NullPointerException when loading remote reference.

Following simple API example
openapi: 3.0.0
info:
  x-api-id: 10eab089-30d0-4dd1-9be7-2948c322b82b
  x-audience: company-internal
  version: "0.1.0"
  title: "My API"

paths:
  /test:
    get:
      responses:
        200:
          description: test
          content:
            application/json:
              schema:
                type: object
                properties:
                    id:
                      type: string
                      format: uuid
                    created_at:
                      type: string
                      format: date-time
      parameters:
        - $ref: 'https://infrastructure-api-repository.zalandoapis.com/apis/401d47fe-1628-4bea-8243-32f45ce9913d/versions/1.0.1/revisions/6#/parameters/Consents'

loads a valid external reference that should work but creates creates a 500 - Internal Server Error response:
{
    "title":"Internal Server Error",
    "status":500,
    "detail":"check invocation failed: id=Z001 title=Path parameters validation checkName=checkSchemaOrContentProperty reason=java.lang.NullPointerException: parameter.name must not be null"
}

The details of the stack trace are
java.lang.RuntimeException: check invocation failed: id=Z001 title=Path parameters validation checkName=checkSchemaOrContentProperty reason=java.lang.NullPointerException: parameter.name must not be null
        at org.zalando.zally.core.RulesValidator.invoke(RulesValidator.kt:63) ~[zally-core-2.1.1-SNAPSHOT.jar!/:na]
        at org.zalando.zally.core.RulesValidator.validate(RulesValidator.kt:40) ~[zally-core-2.1.1-SNAPSHOT.jar!/:na]
        at org.zalando.zally.core.CompositeRulesValidator.validate(CompositeRulesValidator.kt:9) ~[zally-core-2.1.1-SNAPSHOT.jar!/:na]
        at org.zalando.zally.apireview.ApiViolationsController.validate(ApiViolationsController.kt:47) ~[classes!/:na]
        at jdk.internal.reflect.GeneratedMethodAccessor262.invoke(Unknown Source) ~[na:na]
...
Caused by: java.lang.NullPointerException: parameter.name must not be null
        at org.zalando.zally.ruleset.zally.PathParameterRule.checkSchemaOrContentProperty(PathParameterRule.kt:47) ~[zally-ruleset-zally-2.1.1-SNAPSHOT.jar!/:na]
        ... 98 common frames omitted

The failure looks like an authorization problem, however, while on first sight the referenced location looks fine, it contains another local reference that may create problems:
in: body
name: body
description: "Consent object"
required: true
schema:
  $ref: "#/definitions/Consents"

The problem here is that in: body is not allowed in this parameter list. Consequently, the parser seems to filter resulting in an empty parameter array element, which results in a null-name as the NPE is telling. Note: referencing in: query parameter is working fine, e.g. Services.

@tkrop given that definitions field mentioned here
schema:
  $ref: "#/definitions/Consents"

doesn't exist in OpenAPI 3.0, but exist in Swagger 2.0 does this means that the referenced API uses Swagger 2.x?


[project-monitoring] Helm Chart crashs with disabled AlertManager

Describe the bug
The Helm Chart has the option to disable the AlertManager. If this value is set (enabled: false) Helm crashs due the broken yaml file for alertmanager.yaml

To Reproduce
disable AlertManager

Result
Helm crashs
Expected Result
AlertManager is completely disabled and will not installed

Screenshots

Additional context
In the deeper look and why this is not happen in the rancher-monitoring Chart:
This patch file sets the endtag of the first condition in line 73 instead at the end of the file. As a result you become a broken alertmanager.yaml which can't parse by Helm.
Just wondering why this patch logic is a good idea because it's complicated to debug and caused easy to an error.

retested on a local HA (3 nodes all roles)
2-7head
retest status- passed. see video
steps
1 installed monitoring
2. installed prom federator without alert manager enabled
3.  confirmed the application successfully installed, validated the alerts page 'alert manager config' page does not displayed and Alert Manager wasn't installed.
https://zube.io/files/rancher/4288faf85bd748336ddb7b71b42ccddc-2023-05-03_10-38-24.mp4

testing completed and passed. prom federator works as expected with alert manager disabled.


Is not compiling in Android Termux

the new secp256k1 is not compiling in termux
sorry guys i will work in that.

Hi Alberto, I sent you some messages in that group of puzzles, but I did not receive a reply.  Why we used to work for Android, but now it can not be compiled !!.  You mentioned that it works on Termux.  Please solve this problem, you will satisfy everyone.  thanks for everything

Hi Alberto, I sent you some messages in that group of puzzles, but I did not receive a reply. Why we used to work for Android, but now it can not be compiled !!. You mentioned that it works on Termux. Please solve this problem, you will satisfy everyone. thanks for everything

Sorry for not reply before, i left the channel in that momment. But as you see im back.
Im thinking in make an altertate keyhunt native, this is all the new features but with GMP, this will include all excepy the new speed :(

Compile with
make legacy
compilation with termux is back!


Unable to gain valid mTLS client certs

What version of the credhub server you are using?
Master, at commit 6eb9d8cbcda44744762e4760d02ac960bd0b6645
What version of the credhub cli you are using?
2.2.0
If you were attempting to accomplish a task, what was it you were attempting to do?
I was attempting to test the new mTLS client in the go-credhub repo by running CredHub locally and using its test certificates.
What did you expect to happen?
I was expecting to be able to test CredHub's mTLS auth with client certs.
What was the actual behavior?
I may be going about this test incorrectly. Here's what I was doing. I started CredHub locally using:
$ ./scripts/start_server.sh -Dspring.profiles.active=dev,dev-h2,dev-local-uaa

I attempted to use the linked client using various combinations of the certs generated in src/test/resources, such as client_ca_cert.pem and client_ca_private.pem, and server_ca_cert.pem and server_ca_private.pem. None worked. I noticed that perhaps I needed separate client certs that would be generated by running acceptance tests, as per the README:

Assuming it works, that will generate some test client certificates for testing mutual TLS (in certs/ in the acceptance test directory) and run the acceptance test suite against your locally running credhub server.

However, after attempting to run those tests using:
$ CREDENTIAL_ROOT=/home/tbex/go/src/github.com/cloudfoundry-incubator/credhub/src/test/resources UAA_CA=/home/tbex/go/src/github.com/cloudfoundry-incubator/credhub/src/test/resources/ca/dev_uaa.pem ./scripts/run_tests.sh

... the mTLS tests succeed, but the smoke tests fail with:
• Failure in Spec Teardown (AfterEach) [0.075 seconds]
UAA Lifecycle [AfterEach] lifecycle
/home/tbex/go/src/github.com/cloudfoundry-incubator/credhub-acceptance-tests/vendor/github.com/onsi/ginkgo/ginkgo_dsl.go:365

  Expected error:
      <*errors.errorString | 0xc0005b4590>: {
          s: "Received HTTP 403 error while revoking token from auth server: \"{\\\"error\\\":\\\"insufficient_scope\\\",\\\"error_description\\\":\\\"Insufficient scope for this resource\\\",\\\"scope\\\":\\\"uaa.admin tokens.revoke\\\"}\"",
      }
      Received HTTP 403 error while revoking token from auth server: "{\"error\":\"insufficient_scope\",\"error_description\":\"Insufficient scope for this resource\",\"scope\":\"uaa.admin tokens.revoke\"}"
  not to have occurred

  /home/tbex/go/src/github.com/cloudfoundry-incubator/credhub-acceptance-tests/api_client_test/uaa_lifecycle_test.go:15
------------------------------
•••••••••

Summarizing 1 Failure:

[Fail] UAA Lifecycle [AfterEach] lifecycle
/home/tbex/go/src/github.com/cloudfoundry-incubator/credhub-acceptance-tests/api_client_test/uaa_lifecycle_test.go:15

Ran 18 of 18 Specs in 1.497 seconds
FAIL! -- 17 Passed | 1 Failed | 0 Pending | 0 Skipped

And there is no resulting certs/ directory yielding test certs.

@tyrannosaurus-becks It seems like this is a problem with your local development UAA. The error message implies that your UAA admin doesn't have the proper scopes. it seems like the UAA admin client was not granted uaa.admin authority. There could be a number of reasons this happened: perhaps you are using an old version of UAA, or maybe you overwrote this in you manifest.


Problem Connecting to Replica Sets?

Seems that I can't get the config set correctly to connect to a replica set.
I can properly connect to a local non-replica MongoDB database, however when I put in replica set information it fails with not authorized for query on <db_set_in_config>.system.namespaces
Failing replica config:
{
  "database": {
    "default": {
      "host": "<db1_url>,<db2_url>",
      "port": 27017,
      "name": "<db_set>",
      "username": "<username>",
      "password": "<password>"
    }
  }
}

Working Non-Replica Config:
{
  "database": {
    "default": {
      "host": "localhost",
      "port": 27017,
      "name": "some_database",
      "username": "",
      "password": ""
    }
  }
}

Anyone else run into this issue?

Why don't just use the official mongodb driver?

@danielo515 by now probably we should... but if monk is still working, then there are 3 other priorities before refactors
@droppedoncaprica please see https://github.com/azat-co/mongoui/pull/47 if it solved the issue. closing for now.


Randomly failing string tests

On my windows dev setup (standard git bash+docker) i get random crashes in the string tests... always the same but only in 10% of the runs.
https://gist.github.com/PurHur/4d4bb8548d4adfd5afa093fffc6c8a4b
Is that a problem at all? Or a cygwin/windows thing? I never saw that on debian machines.

Nothing was changed between the runs.
Compiling /compiler/lib/JIT.pre
Compiling /compiler/lib/JIT/Helper.pre
Compiling /compiler/lib/JIT/Builtin/Internal.pre
Compiling /compiler/lib/JIT/Builtin/MemoryManager.pre
Compiling /compiler/lib/JIT/Builtin/Output.pre
Compiling /compiler/lib/JIT/Builtin/Refcount.pre
Compiling /compiler/lib/JIT/Builtin/MemoryManager/Native.pre
Compiling /compiler/lib/JIT/Builtin/MemoryManager/PHP.pre
Compiling /compiler/lib/JIT/Builtin/Type/String_.pre
Compiling /compiler/lib/JIT/Builtin/Type/Value.pre
Compiling /compiler/lib/JIT/Call/Native.pre
Updating demo files
docker run -v /c/Users/purhur/php-compiler:/compiler ircmaxell/php-compiler:16.0
4-dev vendor/bin/phpunit
PHPUnit 8.2-g21ee615 by Sebastian Bergmann and contributors.

....................F.......................................      60 / 60 (100%)

Time: 1.07 minutes, Memory: 6.00 MB

There was 1 failure:

1) PHPCompiler\JITTest::testCases with data set #20 ('Basic String Operations',
'<?php\r\n$a = "hello";\r\n$b ...";\r\n', array('Basic String Operations\r\n', '
<?php\r\n$a = "hello";\r\n$b ...";\r\n', 'hello\r\nworld\r\nhello world\r\n'))
Failed asserting that two strings are equal.
--- Expected
+++ Actual
@@ @@
-'hello\n
-world\n
-hello world'
+''

/compiler/test/BaseTest.php:164
/compiler/test/BaseTest.php:145

FAILURES!
Tests: 60, Assertions: 88, Failures: 1.
make: *** [Makefile:70: test] Error 1

purhur@purhur-PC MINGW64 ~/php-compiler (test/str_repeat)
$ make test
docker run -v /c/Users/purhur/php-compiler:/compiler ircmaxell/php-compiler:16.0
4-dev php script/rebuild.php onlyChanged
Compiling /compiler/ext/types/strlen.pre
Compiling /compiler/lib/JIT.pre
Compiling /compiler/lib/JIT/Helper.pre
Compiling /compiler/lib/JIT/Builtin/Internal.pre
Compiling /compiler/lib/JIT/Builtin/MemoryManager.pre
Compiling /compiler/lib/JIT/Builtin/Output.pre
Compiling /compiler/lib/JIT/Builtin/Refcount.pre
Compiling /compiler/lib/JIT/Builtin/MemoryManager/Native.pre
Compiling /compiler/lib/JIT/Builtin/MemoryManager/PHP.pre
Compiling /compiler/lib/JIT/Builtin/Type/String_.pre
Compiling /compiler/lib/JIT/Builtin/Type/Value.pre
Compiling /compiler/lib/JIT/Call/Native.pre
Updating demo files
docker run -v /c/Users/purhur/php-compiler:/compiler ircmaxell/php-compiler:16.0
4-dev vendor/bin/phpunit
PHPUnit 8.2-g21ee615 by Sebastian Bergmann and contributors.

............................................................      60 / 60 (100%)

Time: 1.05 minutes, Memory: 6.00 MB

OK (60 tests, 88 assertions)

There exists a memory corruption bug in the string code somewhere. What's happening is every once in a while a segfault is issued (though not often) and when it is the test fails with no output.

Thanks for explaining. I encountered this only on this machine... If a wonder comes by i fix it :P

Code like this should be able to reliably segfault (or at least it does for me):
<?php

$test = 'a';

for ($i = 0; $i < 1000; $i++) {
    $test = $test . 'a';
}
echo $test;

echo "\n";


Docs Page is not rendering

The docs page is not rendering.  I am getting a 404:  https://bashtage.github.io/linearmodels/doc

Thanks for pointing this out.  Dev docs are up at:
https://bashtage.github.io/linearmodels/devel/index.html
I'll try to get the main rebuilt.

It is back now.   Thanks.
On Thu, Nov 2, 2017, 20:31 mgcdanny<EMAIL_ADDRESS>wrote:

The docs page is not rendering. I am getting a 404:
https://bashtage.github.io/linearmodels/doc
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/bashtage/linearmodels/issues/102, or mute the thread
https://github.com/notifications/unsubscribe-auth/AFU5Rcet6B3YNva_okPQegV1eqg-J-NOks5syia1gaJpZM4QQTQb
.


Add Fuzzing test suites in Litmus

What happened:
What you expected to happen:
Where can this issue be corrected? (optional)

How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:

Isn't this issue included for Gsoc project (#4425)?

Hi @Saranya-jena , could you provide more context on this issue.

@Saranya-jena For adding fuzzing test suites in litmus ,we may add AFL, libFuzzer and some more tools that may be compatible with litmus.I don't much about the compatibility. Please guide me

@Saranya-jena I will take this.

@prithvi1307 @Saranya-jena I'll work on this

Hey @Baalekshan Are you still working on this?

@Akash-Singh04  Yeah, I do but If you want to work on this I think you can also add test suites if you want, just let me know what part of the codebase you are covering so both might not be working on the same thing.

@Baalekshan No worries I'll take up another issue, All the best!

@Baalekshan are you still working on this ?

No, @sagnik3788 feel free to take it !


Autocomplete doesn't delete first letter on input.

[x] The issue is present in the latest release.
[ ] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
Hello there, i wish you can
help me please, i have troubles trying to implement  autocomplete, it worked perfectly days before :(  that's why i writting because I don't know what happen if it's me, or if the new release mixed sth :(
I'm already using version<EMAIL_ADDRESS>When I delete letters inside the input the last one gets stucked and it doesnt allow me to use the autocomplete correctly, also the delete button doesnt work too.

I tried to create a sandbox closed to my app.
I really think the trouble could be on the event, but really it did work before :(
https://codesandbox.io/embed/patient-butterfly-9riyd?fontsize=14&hidenavigation=1&theme=dark
I'm sorry to bother, but i super struggling with this bug and I need to solve out this so much :(
I so pleased with your help and work. greetings <3

Tech
Version

Material-UI
v4.?.?

React

Browser

TypeScript

etc.

From what I understand, the example works in codesandbox but doesn't in your app. You need to make sure you are using the latest version of both core and lab. It should solve your problem.
However, if you manage to create a reproduction on Codesandbox, please share it. We will have a look at it. In the meantime, we can't action your problem. I'm closing.

From what I understand, the example works in codesandbox but doesn't in your app. You need to make sure you are using the latest version of both core and lab. It should solve your problem.
However, if you manage to create a reproduction on Codesandbox, please share it. We will have a look at it. In the meantime, we can't action your problem. I'm closing.

i did attach a sandcode  reproduction up 🥺

Finally it works if I don't use hooks  in both cases, my app y and example codesand, maybe I'm using i hooks wrongly, if you uncomment hook in the codesand example you will see  :o

i did sttach a sandcode example :(

@sueherrera30 You are not controlling the component correctly. They are two controllable props:

value/onChange
inputValue/onInputChange

Don't mix the two :)

@sueherrera30 Thanks for the codesandbox. You are not controlling the component correctly. They are two controllable props:

value => onChange
inputValue => onInputChange

Don't mix the two :)

What if i need to use both oh them?

What if i need to use both oh them?

Keep them isolated, use two different states.