Small update to use more inclusive industry terminology

A tiny edit to use 'blocklist' as it is commonly accepted as a more inclusive term and costs nothing in terms of readability or formatting to use.

Can you reapply on top of main? It looks like there is  merge conflict. Thanks!

In this particular example to me looks ambiguous since a block is an object and blocklisting might be confused.
there is no "industry standard". perhaps denylist can be used

Closing as inactive


Update benchmark result in README.MD

Bindings have improved since the last 2 years, i think it should be re-run to show REAL numbers
Thanks

I no longer have the time or motivation to maintain this project. I'm now working full time on my game engine Bevy.
If anyone wants to handle updates, they're welcome to fork. I'll mark this as "archived" and update the readme to make it clear that the numbers are old and the project unmaintained.


fix: Correct wait_until_stable_timeout variable type

Description

Correct wait_until_stable_timeout variable type

Motivation and Context

Resolves #104

Breaking Changes

No

How Has This Been Tested?

[ ] I have updated at least one of the examples/* to demonstrate and validate my change(s)
[ ] I have tested and validated these changes using one or more of the provided examples/* projects

[x] I have executed pre-commit run -a on my pull request

This PR is included in version 5.2.2 :tada:


Manage Lookup Rollup Summaries Edit Issue

When in lightning experience, when an existing summary is selected, the page does not change from adding a new rollup to editing an existing rollup.

@ddatawitch i was not able to reproduce this just now. Are you still able to?

@ddatawitch we can re-open this if you want to continue to discuss. Thanks.


Disable Metronome

Jitsi requires Prosody and if Metronome is running then Jitsi cannot be used. Prosody and Metronome conflict. You would have to stop Metronome and restart Prosody in order for Jitsi to work. On my system Metronome always keep enabling itself...
systemctl disable metronome.service doesn't seem to help.
metronome.service is not a native service, redirecting to systemd-sysv-install.
Executing: /lib/systemd/systemd-sysv-install disable metronome

Many have complained about this in the past, they see a "rejoin" message that keeps on looping. This is the reason why....

Hi!
The packages’s docs now asks to uninstall metronome before install jitsi :)


Can you tell me about Casdoor as a RADIUS server?

Casdoor used as a RADIUS server do not know whether it can realize the WIFI Portal SMS authentication function

@dacongda

according the code , WIFI Portal SMS auth is unsupported

@dacongda Support WIFI Portal LDAP authentication?

In face, I don't really understand what you what to do, can you post some picture or doc to describe it?

Closed for stale


WarpAffine strange behaviour on NVIDIA

Summary of your issue
Trying to apply WarpAffine transform - using UMats - to each frame of a video file and write it back to a new file. I see very strange behaviour using an NVidia RTX 3060, as a lot of frames are become the same as the previous one.
Displaying the source frame with ImShow shows even the source frames are corrupted.
Environment
Laptop - Corei7 with Iris-XE, Windows 11 22H2
Desktop - Ryzen 5 (no gpu) with RTX3060, Windows 11 22H2
What did you do when you faced the problem?
My code works perfectly on my laptop with an Intel CPU's built-in Iris-XE gpu. When I move to an other machine with an NVidia 3060, I can successfully run the code, but a lot of frames are become the same as the previous one. Displaying the source frame with ImShow shows even the source frames are corrupted.
I've tried to investigate a bit, if I does not dispose the original UMat - by removing the using statement - I can get perfect result on both machines, however its leaking gpu memory.
I've tried to replace the WarpAffine with a simple copy for test purposes, and it was working perfectly on both machines.
Example code:

using OpenCvSharp;
using OpenCvSharp.XImgProc;

string inputFile  = "test.mp4";
string outputFile = "result.mp4";

using (VideoCapture videocapture = new VideoCapture(inputFile, VideoCaptureAPIs.FFMPEG))
{
    if (!videocapture.IsOpened()) { return; }

    VideoWriter videoWriter = new VideoWriter(outputFile,
        VideoCaptureAPIs.FFMPEG,
        VideoWriter.FourCC("mp4v"),
        videocapture.Fps,
        new OpenCvSharp.Size(videocapture.FrameWidth, videocapture.FrameHeight));

    int idx = 1;
    int frameCount = videocapture.FrameCount;
    int frameNumLength = frameCount.ToString("D").Length;

    Mat appliedTranslation = new(2, 3, MatType.CV_64FC1);
    appliedTranslation.At<double>(0, 0) = Math.Cos(0.1);
    appliedTranslation.At<double>(0, 1) = -Math.Sin(0.1);
    appliedTranslation.At<double>(1, 0) = Math.Sin(0.1);
    appliedTranslation.At<double>(1, 1) = Math.Cos(0.1);
    appliedTranslation.At<double>(0, 2) = 100;
    appliedTranslation.At<double>(1, 2) = -50;
    UMat translated= new UMat();

    using (Mat srcFrame = new Mat())
    {
        Mat frame;
        while (videocapture.Read(srcFrame))
        {
            Console.WriteLine($"Processing {idx} from {videocapture.FrameCount}...");
            Cv2.ImShow("Result", srcFrame);

            //Doesn't work on NVidia, works perfectly on Intel
            using (UMat uImg = srcFrame.GetUMat(AccessFlag.READ, UMatUsageFlags.DeviceMemory))
            {
                Cv2.WarpAffine(uImg, translated, appliedTranslation, uImg.Size());
                frame = translated.GetMat(AccessFlag.RW);
            }

            //Works on both, but leaks gpu memory
            //UMat uImg = srcFrame.GetUMat(AccessFlag.READ, UMatUsageFlags.DeviceMemory);
            //Cv2.WarpAffine(uImg, translated, appliedTranslation, uImg.Size());
            //frame = translated.GetMat(AccessFlag.RW);

            //Works perfectly on both as expected
            //using (UMat uImg = srcFrame.GetUMat(AccessFlag.READ, UMatUsageFlags.DeviceMemory))
            //{
            //    uImg.CopyTo(translated);
            //    frame = translated.GetMat(AccessFlag.RW);
            //}

            videoWriter.Write(frame);
            Cv2.ImShow("Result", frame);
            Cv2.WaitKey(1);
            frame.Dispose();
            ++idx;
        }
        videocapture.Release();
        videoWriter.Release();
    }
}

Output:

What did you intend to be?

After playing a bit more, I've found a workaround for the issue. It is working nicely both on intel and nvidia gpu's however it needs and extra, theoretically unnecessary copy of the image. This way I copy the source frame's UMat to a temp UMat, so I can dispose the source frame's UMat. Now I think the issue is maybe with the videocapture somehow.
UMat temp = new();
UMat translated= new UMat();

using (Mat srcFrame = new Mat())
{
    Mat frame;
    while (videocapture.Read(srcFrame))
    {
        Console.WriteLine($"Processing {idx} from {videocapture.FrameCount}...");
        Cv2.ImShow("Result", srcFrame);

        using (UMat uImg = srcFrame.GetUMat(AccessFlag.READ, UMatUsageFlags.DeviceMemory))
            uImg.CopyTo(temp);
        Cv2.WarpAffine(temp, translated, appliedTranslation, temp.Size());
        frame = translated.GetMat(AccessFlag.RW);

        videoWriter.Write(frame);
        Cv2.ImShow("Result", frame);
        Cv2.WaitKey(1);
        frame.Dispose();
        ++idx;
    }
    videocapture.Release();
    videoWriter.Release();
}


Update gcloud-python/datastore to use the v1 API

The v1 API is now available.
The protos are identical (just a different package name). The only behavioral change is captured in the release notes:
https://cloud.google.com/datastore/release-notes

On it


Allow Installation Without Jupyter

I'm running Julia kernels in a dockerized environment (running kernels are connected to from frontends that are completely separate) where having Jupyter itself installed is completely superfluous (it adds a LOT of dependencies to install Jupyter + Conda when it won't even be used). Is there a way to just skip this step (e.g. environment variable)?
user@ef06f0d39e8b:/$ du -sh /usr/local/julia/share/julia/site/v0.6/Conda/
1.5G	/usr/local/julia/share/julia/site/v0.6/Conda/

How can we install the kernel configuration file without jupyter?

I guess we can install in the “standard” location (is this documented?) and only install Jupyter if notebook() is called...

@stevengj Do you mean the kernel.json file in .../share/jupyter/kernels or the deps.jl file in the IJulia package directory?
If the former, I vote "just don't.", and if the latter, I think that the check for that file would move away from module initialization and into the notebook() command. I'm not sure if the constants defined in deps.jl are used anywhere else but I was able to comment out the include and have the kernel itself run just fine.

I mean the kernel.json file, so that it works with any existing Jupyter installation.  Why not install this, since the installation directory is documented: http://jupyter.readthedocs.io/en/latest/projects/jupyter-directories.html, and this way it will work with any jupyter installation regardless of PATH.
I don't like the idea of making notebook() issue an error and forcing you to install Jupyter manually.  Why not just install Jupyter automatically if it is not found?

Those directories might not exist (e.g. they wouldn't in my scenario), so I suppose just make sure that that's handled gracefully.
As for notebook(), I don't have strong opinions, it just seems that installing the notebook is a relatively large operation that might not be expected or desired (principle of least astonishment) when running notebook(). For example, if my PATH is misconfigured I don't want to install Conda + Jupyter. Maybe a prompt before installing (if in a terminal, otherwise default to yes)? But I'm not a consumer of that API so I certainly won't be upset if you decide to just have it download and install everything.

Those directories might not exist (e.g. they wouldn't in my scenario), so I suppose just make sure that that's handled gracefully.

You'd call mkpath.

I think a prompt would be nice, if running interactively (and default to just downloading if running non-interactively).
Jupyter wasn't found in your PATH. Download it? [Y/n]:


added selectable text property for web

it would be cool to also be able to set this via a property, there might be cases where you want this in non-web

Could anyone help me to understand how to make text selectable in flutter web? thanks.

Selectionarea doesnt let you select text by double tapping it.

+1


Add code to link and unlink Cloud accounts to a NewRelic account

This pull requests adds functionality to link a Cloud account to a NewRelic infrastructure account

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Arno Geurts seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Hey @arnogeurts, thanks so much for your contribution, we really appreciate this kind of support from the community.  We believe this functionality already exists in the cloud package here and here, can you give it a try and let us know if it works for you?

@arnogeurts, my message was premature.  I'm not sure this functionality is working yet as intended.  Our team had to pivot in the middle of the implementation.  We'll decide how to proceed and get back to you here.

Duplicated by #535.
This functionality is now available in v0.48.0.


fix: wrapped keywords with the  tag

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[ ] I have tested these changes either locally on my machine, or GitPod.

Closes #56033

@CodiumAI-Agent /describe

Title
fix: wrapped keywords with the  tag

User description
Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[ ] I have tested these changes either locally on my machine, or GitPod.

Closes #56033

PR Type
enhancement, documentation

Description

Enhanced the description of the "Build a Cat Blog Page" workshop by wrapping HTML semantic element names (main, nav, article, footer) with <code> tags to improve readability and clarity.
This change helps users easily identify code elements within the instructional text.

Changes walkthrough 📝
Relevant filesEnhancement

intro.jsonEnhance workshop description with code formatting

client/i18n/locales/english/intro.json
Wrapped HTML semantic element names with  tags in the text.  Improved readability and clarity of the workshop description.

+1/-1

💡 PR-Agent usage:
Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

@CodiumAI-Agent /describe


clean-conversation-sidebar: Cannot read property 'remove' of undefined

Happening on private repo running GitHub Enterprise Server 2.20.1

Going to close this as we have not got any response and its an unsupported GHE

@yakov116
I am also getting this error on a private normal GitHub repo:
refined-github.js:1923 ❌ Refined GitHub → clean-conversation-sidebar → TypeError: Cannot read property 'remove' of undefined
    at clean (refined-github.js:6568)
    at runFeature (refined-github.js:1942)
    at setupPageLoad (refined-github.js:1955)
    at refined-github.js:1988
Search issue: https://github.com/sindresorhus/refined-github/issues?q=is%3Aissue+Cannot%20read%20property%20'remove'%20of%20undefined

Open an issue: https://github.com/sindresorhus/refined-github/issues/new?labels=bug&template=bug_report.md&title=%60clean-conversation-sidebar%60%3A%20Cannot%20read%20property%20'remove'%20of%20undefined

It does not only seem to be GHE.
Refined GitHub

Version 21.4.13

Yes that’s https://github.com/sindresorhus/refined-github/issues/4258


Add "Edit this page" on each documentation page

I was planning to add this when undertaking the doc. I'll open a PR as early as possible so we can discuss around it.


Can LibSVMClassificationTrainer and LIMEExplanation be used to extract meaningful features from SVM?

I would like to use SVM for classification, then later use LIME to explain what the model thinks in a human friendly way.
Unfortunately, it looks like LIMEText expects a class implementing SparseTrainer which LibSVMClassificationTrainer does not do. I'm wondering if there is a chance to make it work somehow? I'm new to the API (started exploring it yesterday), so maybe I'm missing a concept which would come with more experience.
Is your question about a specific Tribuo class?
LIMEText, SparseTrainer, LibSVMClassificationTrainer
System details

Tribuo version: 4.1.0
Java version: 11

After getting more acquainted with unit tests, I came to a realization that I made a wrong assumption that LIME needs to use the same trainer as used for building the original model. It's actually the opposite, there is no requirement as such. Therefore it's completely fine to use SVM to create a model and later use any sparse trainer to explain it, like this:
// ...
var trainer = new LibSVMClassificationTrainer(params);
Model<Label> model = trainer.train(trainData);
// ...
var sparseTrainer = new CARTJointRegressionTrainer(4, true);
var lime = new LIMEColumnar(new SplittableRandom(1), model, sparseTrainer, 5000, rowProcessor, tokenizer);
LIMEExplanation explain = lime.explain(trainData.getExample(0));
explain.getModel().getActiveFeatures();

Yep the LIME requirement is that the explaining model is sparse not the model that's being explained. This is actually more expressive than the original LIME paper which only used l1 regularised linear models as the explaining model, in Tribuo you can use trees too.


Question: UrlCreator

Hi,
how would I annotate a custom UrlCreator? To get something like:
        <public>
            <name>MyUrlCreator</name>
            <class>de.espirit.firstspirit.generate.UrlCreatorSpecification</class>
            <configuration>
                <LegacyClass>de.my.urlservice.creator.MyUrlCreator</LegacyClass>
                <UseRegistry>no</UseRegistry>
            </configuration>
        </public>

Is it possible?
Thanks, Heiko

Played around a bit:
@UrlFactoryComponent(name = "MyUrlCreator", useRegistry = false, parameters = {
        @Parameter(name = "LegacyClass", value = "de.my.urlservice.creator.MyUrlCreator") })

I end up with:
		<public>
			<name>MyUrlCreator</name>
			<displayname/>
			<description/>
			<class>de.espirit.firstspirit.generate.UrlCreatorSpecification</class>
			<configuration>
				<UrlFactory>de.my.urlservice.creator.MyUrlCreator</UrlFactory>
				<UseRegistry>false</UseRegistry>
				<LegacyClass>de.my.urlservice.creator.MyUrlCreator</LegacyClass>
			</configuration>
		</public>

I guess "UrlFactory" breaks it?

I guess "UrlFactory" breaks it?

What do you mean by that? Is the generated configuration not working?

Without testing it I was confused by:
<UrlFactory>de.my.urlservice.creator.MyUrlCreator</UrlFactory>
<LegacyClass>de.my.urlservice.creator.MyUrlCreator</LegacyClass>

Now I tested and the old UrlCreator seems to work. Unfortunately we cannot use the newer approach UrlFactory because we want to access the generation task variables.

I tested it now in the correct environment and it works even with both set: UrlFactory and LegacyClass.


Feature request: autoscaling for ticks and spines

Seems to me this is the one thing missing right now. Thank you for doing this, it's a pretty elegant solution to a widespread problem, love it !

Good catch! I'll fix this -- should be straightforward.
And I'm glad you find it useful.

btw, not directly related, but I see that the GitHub repo and the PyPI page don't reference each other. I'm happy to contribute a PR to make them more discoverable if you'd like/

Sure, that would be great, thanks!

This should be fixed now -- thanks again for bringing this up


add just_error, just_done, done_as_optional and done_as_exception

This gives me pause. But don't wait for me to figure out why.


speed of the device and distance travelled in realtime

is it possible to get realtime speed and distance travelled in realtime.

can we get speed and distance realtime in free drive

Stale issue message


Concurrent runs still failing.

We're still struggling with concurrent Danger runs, even after the changes in https://github.com/danger/swift/pull/530
We get a little further this time, but somehow either Danger JS or Danger Swift is trying to write to the same location. I've run a test with two pipelines on the mac, with debug enabled.
The first fails:
2022-09-16T13:26:35.491Z danger:process_runner Preparing to run: .build/debug/danger-swift,runner,/snapshot/danger-js/distribution/commands/danger-ci.js,--process,.build/debug/danger-swift,--passURLForDSL,-i,danger_ci,-f,--cwd,../
2022-09-16T13:26:35.491Z danger:runDangerSubprocess Running sub-process: danger-swift - runner,/snapshot/danger-js/distribution/commands/danger-ci.js,--process,.build/debug/danger-swift,--passURLForDSL,-i,danger_ci,-f,--cwd,../
2022-09-16T13:26:35.500Z danger:runDangerSubprocess Started passing in STDIN via the URL: danger://dsl//var/folders/0h/c4gthkj13sz3hb1z4fwpnd3c0000gp/T/danger-dsl.json
2022-09-16T13:26:35.500Z danger:runDangerSubprocess Passed DSL in via STDIN
Ran with: /var/folders/0h/c4gthkj13sz3hb1z4fwpnd3c0000gp/T/danger/8B52C17F-7357-4BF2-BC85-5FB55F2448AC/_tmp_dangerfile.swift runner /snapshot/danger-js/distribution/commands/danger-ci.js --process .build/debug/danger-swift --passURLForDSL -i danger_ci -f --cwd ../ /var/folders/0h/c4gthkj13sz3hb1z4fwpnd3c0000gp/T/danger-dsl.json /var/folders/0h/c4gthkj13sz3hb1z4fwpnd3c0000gp/T/danger/8B52C17F-7357-4BF2-BC85-5FB55F2448AC/danger-response.json
ERROR: could not find DSL JSON at path: /var/folders/0h/c4gthkj13sz3hb1z4fwpnd3c0000gp/T/danger-dsl.json
Launching Danger Swift runner (v3.14.0)
Got URL for JSON: /var/folders/0h/c4gthkj13sz3hb1z4fwpnd3c0000gp/T/danger-dsl.json
Created a temporary file for the Dangerfile DSL at: /var/folders/0h/c4gthkj13sz3hb1z4fwpnd3c0000gp/T/danger-dsl.json
Running Dangerfile at: Dangerfile.swift
Preparing to compile
Running: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/swift --driver-mode=swift -L /private/tmp/gitlab-runner-builds/245930/Danger/.build/debug -I /private/tmp/gitlab-runner-builds/245930/Danger/.build/debug -lDangerDeps /var/folders/0h/c4gthkj13sz3hb1z4fwpnd3c0000gp/T/danger/8B52C17F-7357-4BF2-BC85-5FB55F2448AC/_tmp_dangerfile.swift runner /snapshot/danger-js/distribution/commands/danger-ci.js --process .build/debug/danger-swift --passURLForDSL -i danger_ci -f --cwd ../ /var/folders/0h/c4gthkj13sz3hb1z4fwpnd3c0000gp/T/danger-dsl.json /var/folders/0h/c4gthkj13sz3hb1z4fwpnd3c0000gp/T/danger/8B52C17F-7357-4BF2-BC85-5FB55F2448AC/danger-response.json
Completed evaluation
ERROR: Dangerfile eval failed at Dangerfile.swift
ERROR: Could not get the results JSON file at /var/folders/0h/c4gthkj13sz3hb1z4fwpnd3c0000gp/T/danger/8B52C17F-7357-4BF2-BC85-5FB55F2448AC/danger-response.json

While the second completes fine:
2022-09-16T13:26:35.135Z danger:process_runner Preparing to run: .build/debug/danger-swift,runner,/snapshot/danger-js/distribution/commands/danger-ci.js,--process,.build/debug/danger-swift,--passURLForDSL,-i,danger_ci,-f,--cwd,../
2022-09-16T13:26:35.135Z danger:runDangerSubprocess Running sub-process: danger-swift - runner,/snapshot/danger-js/distribution/commands/danger-ci.js,--process,.build/debug/danger-swift,--passURLForDSL,-i,danger_ci,-f,--cwd,../
2022-09-16T13:26:35.162Z danger:runDangerSubprocess Started passing in STDIN via the URL: danger://dsl//var/folders/0h/c4gthkj13sz3hb1z4fwpnd3c0000gp/T/danger-dsl.json
2022-09-16T13:26:35.162Z danger:runDangerSubprocess Passed DSL in via STDIN
Ran with: /var/folders/0h/c4gthkj13sz3hb1z4fwpnd3c0000gp/T/danger/C31A4E15-B2D0-4622-AA41-EC29822D0386/_tmp_dangerfile.swift runner /snapshot/danger-js/distribution/commands/danger-ci.js --process .build/debug/danger-swift --passURLForDSL -i danger_ci -f --cwd ../ /var/folders/0h/c4gthkj13sz3hb1z4fwpnd3c0000gp/T/danger-dsl.json /var/folders/0h/c4gthkj13sz3hb1z4fwpnd3c0000gp/T/danger/C31A4E15-B2D0-4622-AA41-EC29822D0386/danger-response.json
Decoding the DSL into Swift types
Setting up to dump results
Sending results back to Danger
Sending results back to Danger
2022-09-16T13:26:59.330Z danger:runDangerSubprocess Got JSON URL from STDOUT, results are at:
danger-results://var/folders/0h/c4gthkj13sz3hb1z4fwpnd3c0000gp/T/danger/C31A4E15-B2D0-4622-AA41-EC29822D0386/danger-response.json
Launching Danger Swift runner (v3.14.0)

Notice theres a URL which is identical in both runs: danger://dsl//var/folders/0h/c4gthkj13sz3hb1z4fwpnd3c0000gp/T/danger-dsl.json
I'm happy to look into this, but I'm not entirely sure if this a problem with Danger JS or Swift. If you wouldn't mind taking a look and giving me a clue, I'd get cracking.

Actually, I think I see the issue on DangerJS:
  const sendDSLToSubprocess = () => {
    if (exec.options.passURLForDSL) {
      const resultsPath = join(tmpdir(), "danger-dsl.json")
      writeFileSync(resultsPath, dslJSONString, "utf8")
      const url = `danger://dsl/${resultsPath}`
      d(`Started passing in STDIN via the URL: ${url}`)
      child.stdin.write(url)
      child.stdin.end()
    } else {
      d(`Started passing in STDIN`)
      child.stdin.write(dslJSONString)
      child.stdin.end()
    }
    d(`Passed DSL in via STDIN`)
  }

https://github.com/danger/danger-js/blob/a6b40e953ff318c6230bf0f09152e4ba58cd567f/source/commands/utils/runDangerSubprocess.ts#L55-L69

After more research, it's clear this is an issue in danger-js so I've opened an issue there instead.


[HOTFIX] Disabling GPU fused relu & maxpooling operator because of bug

Fixed the timer that counts the number of times memory chunks are
zero-ed out
Some minor code refactoring

@niketanpansare

LGTM. Please feel free to merge. I can add relu_maxpooling fix later. Thanks :+1:

Thanks @niketanpansare, the bug to fix is in JIRA 1403
I shall merge.


fix: remove deprecated folder_offset and preview_offset, add prepend_rules and append_rules to [icon]

This PR adds prepend_rules and append_rules introduced in https://github.com/sxyazi/yazi/commit/b027487d12213b3653f33265b291dd41c707d1ab at Yazi 0.2.4; while removing the deprecated folder_offset and preview_offset from Yazi 0.2.0.

Ah I keep missing these because I usually scan for changes to the preset TOML files and this didn't cause a change.


Write action for removing participant from contest

Route: contests/{contestId}/participants/remove/{username}

Done!


Fix native event name collisions

Fixes issue https://github.com/paramaggarwal/react-native-youtube/issues/6, caused by the event name onError colliding with an identically named event published by React Native core RCTImageViewManager.
Renamed the other events as well to safeguard from incompatibilities with other libraries, or future changes in React Native.

Thanks! Published to npm as v0.1.3.


Appstudio update test-component-pac-mhzz

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI/test-component-pac-mhzz-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
23 seconds
init

✅ Succeeded
24 seconds
clone-repository

✅ Succeeded
3 minutes
build-container

✅ Succeeded
39 seconds
deprecated-base-image-check

✅ Succeeded
38 seconds
inspect-image

✅ Succeeded
3 minutes
clair-scan

✅ Succeeded
1 minute
clamav-scan

✅ Succeeded
37 seconds
sbom-json-check

✅ Succeeded
10 seconds
label-check

✅ Succeeded
6 seconds
show-summary

✅ Succeeded
7 seconds
show-sbom

Pipelines as Code CI/test-component-pac-mhzz-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
23 seconds
init

✅ Succeeded
24 seconds
clone-repository

✅ Succeeded
3 minutes
build-container

✅ Succeeded
39 seconds
deprecated-base-image-check

✅ Succeeded
38 seconds
inspect-image

✅ Succeeded
3 minutes
clair-scan

✅ Succeeded
1 minute
clamav-scan

✅ Succeeded
37 seconds
sbom-json-check

✅ Succeeded
10 seconds
label-check

✅ Succeeded
7 seconds
show-sbom

✅ Succeeded
6 seconds
show-summary

Pipelines as Code CI/test-component-pac-mhzz-on-pull-request has successfully validated your commit.


Add caching to middleware

Added runtime caching to middleware.
Definitions cached separately by the isBackOffice flag so update to one does not need to fetch the other.

Hi @Digipex, I can see you have merged both PR branches, not sure which one is the most up to date.
Seen as they are both merged together now, Ill just review one of them as a fix for both issues :)
Can you let me know which one I should be looking at.
Matt

@Matthew-Wise Thanks. I'll get those changes made. Should have time to do it

Thanks for the great work @Digipex  :) Hopefully it wont take me to long to get it tested and released :)


Incorrect use of OAuth

I mentioned this to one of you at SOTM-US but looking at it some more I think you guys are using OAuth in a way that it isn't really intended to be used. The intended use of getting an OAuth token from osm.org is to make authenticated requests back to the OSM API on the user's behalf without needing to know the user's password. You are using it as an authentication mechanism for your own site. What you are looking for is OpenID or something like that. I don't think OSM provides this.
The result is this misuse of OAuth can be seen on my osm.org account settings page:

All other applications in that list have a single token issued per installation of that application and persist the token locally to keep issuing requests to the API as me.

Well authenticating with OAuth is pretty much accepted these days, even if it wasn't what OAuth was originally intended for, but people that do that should hang on to the token rather than getting a new one each time like this.

Well in that case, it might be good to set up a different application keys for the mobile app and the website.  It still uploaded pictures but they vanished into /dev/null and never showed up in my profile. If you had a separate key named OpenStreetView - Android or something like that it would help in keeping things groomed.

Well ideally yes they would use different application keys, but even so it would be OK to have one token for each, just not to generate a new one on each use.

Agreed we should do this more cleanly @alexandrui-telenav can we track this internally?

I was going to enter the same bug, very annoying having to login every 3-4 hours

Sorry for the late reply! We started a new sprint today and this issue is included. Will keep you guys posted, thanks for raising this.

Hello, I have same bug

Great to see progress in action! :-P

@ToeBee is this issue still valid?

Kind of. You are still using the same kind of authentication. But at least now the session on your website seems to last a LOT longer so I am no longer being asked to authorize it every time I visit the website.
It would still be nice to use a separate developer key for the website and the app to distinguish them in my OSM settings.


Correction for raw affiliation University of Lyon, CNRS, University of Lyon 3 Jean Moulin, Environnement Ville Société UMR 5600, F-69007, Lyon, France

Correction needed for raw affiliation University of Lyon, CNRS, University of Lyon 3 Jean Moulin, Environnement Ville Société UMR 5600, F-69007, Lyon, France
raw_affiliation_name: University of Lyon, CNRS, University of Lyon 3 Jean Moulin, Environnement Ville Société UMR 5600, F-69007, Lyon, France
new_rors: 05b5c0584;02feahw73;029brtt94;0256r1e04
previous_rors: 05b5c0584;02feahw73;029brtt94
works_examples: W2516625973
contact: 95c3849f35c40e139fd56b567ee7e543:20512a49ccddc85c358054e2786c @ ens-lyon.fr

This issue was accepted and ingested by the OpenAlex team on 2024-11-01. The new affiliations should be visible within the next 7 days.


Controller cache invalidation (cont.)

Description

Related issue and scope

[ ] I opened an issue to propose and discuss this change (#????)

My changes affect the following components

[ ] API
[x] Controller
[ ] Message Bus (e.g., Kafka)
[ ] Loadbalancer
[ ] Invoker
[ ] Intrinsic actions (e.g., sequences, conductors)
[x] Data stores (e.g., CouchDB)
[x] Tests
[ ] Deployment
[ ] CLI
[ ] General tooling
[ ] Documentation

Types of changes

[ ] Bug fix (generally a non-breaking change which closes an issue).
[ ] Enhancement or new feature (adds new functionality).
[ ] Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

[ ] I signed an Apache CLA.
[ ] I reviewed the style guides and followed the recommendations (Travis CI will check :).
[ ] I added tests to cover my changes.
[ ] My changes require further changes to the documentation.
[ ] I updated the documentation where necessary.

Codecov Report

Merging #106 (92ed658) into master (bdf67e5) will decrease coverage by 7.79%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #106      +/-   ##
==========================================
- Coverage   77.51%   69.71%   -7.80%
==========================================
  Files         194      194
  Lines        9478     9478
  Branches      583      593      +10
==========================================
- Hits         7347     6608     -739
- Misses       2131     2870     +739

Impacted Files
Coverage Δ

...org/apache/openwhisk/core/entity/WhiskAction.scala
89.07% <100.00%> (-2.19%)
:arrow_down:

...rg/apache/openwhisk/core/entity/WhiskPackage.scala
96.77% <100.00%> (ø)

...rg/apache/openwhisk/core/entity/WhiskTrigger.scala
100.00% <100.00%> (ø)

...la/org/apache/openwhisk/http/BasicRasService.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...che/openwhisk/core/invoker/LogStoreCollector.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...a/org/apache/openwhisk/http/BasicHttpService.scala
0.00% <0.00%> (-91.55%)
:arrow_down:

.../apache/openwhisk/core/controller/Controller.scala
0.00% <0.00%> (-81.74%)
:arrow_down:

...pache/openwhisk/core/invoker/InvokerReactive.scala
0.73% <0.00%> (-81.62%)
:arrow_down:

...la/org/apache/openwhisk/core/invoker/Invoker.scala
0.00% <0.00%> (-70.50%)
:arrow_down:

...che/openwhisk/core/loadBalancer/LoadBalancer.scala
16.66% <0.00%> (-66.67%)
:arrow_down:

... and 48 more

Continue to review full report at Codecov.

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

Overruled by #107


[Feature Request] Send tokens to a different address than the sender address

Problem
The bridge seems to only send tokens to the same address as the sender address, only on the opposite network. For example if the user is sending USDT from 0xd8dA6BF26964aF9D7eEd9e03E53415D37aA96045 on Ethereum then the bridge will send RUSDT to 0xd8dA6BF26964aF9D7eEd9e03E53415D37aA96045 on the RSK network.
However, some RSK wallets (such as Liquality) do not generate the same RSK address as the Ethereum address. This results in difficulty for users of these wallets when using the bridge, who must then figure out a series of complicated steps to export their seed, import in another wallet, find the correct derivation path the addresses were generated on, etc.
Solution
Allow users to input a custom receiving address for the transaction they are sending across the bridge.

Hi John we are aware of this issue, we are working to allow sending funds to other addresses on version 2 that will be released in this quarter. It's taking us a while as there are security concerns in case you send the funds to a contract on the other network so we are in the process of auditing the code.
Thanks for contributing! :)


Portainer worker container button missing

Hi, i have a problem with portainer i can't see the button on worker.
To begin with, we're working with a swarm with portainer-ce, and portainer-agent: 1 manager and 2 workers.
We ran docker swarm init, docker swarm join to link the nodes. They can ping each other and they are listed when running "docker nodes ls".
But on portainer services tab, when clicking on portainer agent & dev-api, the worker nodes are missing some buttons.

`
---- ---- ---- ----
networks
---- ---- ---- ----
networks:
private:
external:
name: private
web:
external:
name: web
portainer_agent:
driver: overlay
attachable: true
services:
---- ---- ---- ----
traefik
---- ---- ---- ----
traefik:
    image: traefik:v2.4.2
    ports:
        - 80:80
        - 443:443
    volumes:
        - /var/run/docker.sock:/var/run/docker.sock:ro # readonly
        - /var/docker/letsencrypt:/letsencrypt
    networks:
        - web
    command:
        - --api=true
        - --api.dashboard=true
        - --api.insecure=true
        - --entrypoints.web.address=:80
        - --providers.docker=true
        - --providers.docker.watch=true
        - --providers.docker.network=web
        - --providers.docker.exposedbydefault=false
        - --log.level=DEBUG
        - --providers.docker.swarmMode=true
        - --entrypoints.websecure.address=:443
        - --certificatesresolvers.letsencrypt.acme.httpchallenge=true
        - --certificatesresolvers.letsencrypt.acme.httpchallenge.entrypoint=web
        -<EMAIL_ADDRESS>        - --certificatesresolvers.letsencrypt.acme.storage=/letsencrypt/acme.json
        - --entrypoints.web.http.redirections.entryPoint.to=websecure
        - --entrypoints.web.http.redirections.entryPoint.scheme=https
        - --entrypoints.web.http.redirections.entrypoint.permanent=true
    deploy:
        placement:
            constraints: [node.role == manager]
        labels:
            - traefik.enable=true
            - traefik.http.routers.traefik.tls=true
            - traefik.http.routers.traefik.rule=Host(`traefik.dsp-archiwebo20-db-od-sn-bm.fr`)
            - traefik.http.routers.traefik.entrypoints=websecure
            - traefik.http.services.traefik.loadbalancer.server.port=8080
            - traefik.http.routers.traefik.tls.certresolver=letsencrypt

---- ---- ---- ----
portainer
---- ---- ---- ----
portainer:
    image: portainer/portainer-ce:latest
    command: -H tcp://tasks.agent:9001 --tlsskipverify
    # ports:
    #     - "8000:8000"
    #     - "9000:9000"
    volumes:
        # - /var/run/docker.sock:/var/run/docker.sock
        # - portainer-data:/data
        - /var/docker/data/portainer:/data
    networks:
        - portainer_agent
        - web
        - private
    deploy:
        placement:
            constraints: [node.role == manager]
        labels:
            - traefik.enable=true
            - traefik.http.routers.portainer.tls=true
            - traefik.http.routers.portainer.rule=Host(`portainer.dsp-archiwebo20-db-od-sn-bm.fr`)
            - traefik.http.routers.portainer.entrypoints=websecure
            - traefik.http.services.portainer.loadbalancer.server.port=9000
            - traefik.http.routers.portainer.tls.certresolver=letsencrypt

---- ---- ---- ----
portainer agent
---- ---- ---- ----
agent:
    image: portainer/agent:latest
    environment:
        AGENT_CLUSTER_ADDR: tasks.agent
        # CAP_HOST_MANAGEMENT: 1
        LOG_LEVEL: DEBUG
    volumes:
        - /var/run/docker.sock:/var/run/docker.sock
        - /var/lib/docker/volumes:/var/lib/docker/volumes
        # - /:/host
    # ports:
    #     - target: 9001
    #       published: 9001
    #       protocol: tcp
    #       mode: host
    networks:
        - portainer_agent
    deploy:
        mode: global
        placement:
            constraints: [node.platform.os == linux]`

please check the overlay network ports are open on your hosts...

Port
Protocols
Description

2377
TCP
Cluster management communications

7946
TCP, UDP
Inter-node communication

4789
UDP
Overlay network traffic

@ncresswell i think they are open ? (iptables)
Manager :

Worker1:

Worker2:

One thing I have found in the past that helps with agent related issues, specifically missing buttons and/or resources is force updating the agent service. An example of this command if docker service update portainer_agent --force

@itsconquest i tried that yesterday same problem

@samdulam
portainer_agent on manager:

portainer_agent on worker1:

portainer_agent on worker2:

portainer_web on manager:

5th line on each of the agent logs show you the results for tasks.agent, it should be 3 one for each agent in the network. Looks like an issue with overlay network or docker dns.
Try restarting docker service (keep in mind this will stop all running containers)
Also check this

@Strawbang the Portainer people are reasonably certain that this is a Docker network/dns issue, and expect that one diagnostic is to deploy 3 busybox containers as swarm service, and ping each of them by hostname from each of them - if it works, then we're wrong :)
I'm hoping that you've not updated the issue because it solved itself - but please, ping us if we're wrong?


Codeunit 5760 new event on procedure InitSourceDocumentLines

Hi,
We need a new event on codeunit 5760  procedure  InitSourceDocumentLines :
.....
 SetRange("Source Line No.", TransLine."Line No.");
                                if FindFirst then begin
                                    OnAfterFindWhseRcptLineForTransLine(WhseRcptLine2, TransLine);
                                    ModifyLine := TransLine."Qty. to Receive" <> "Qty. to Receive";
                                    if ModifyLine then begin
                                        // > New Event
                                        OnBeforeModifyTransferLine(TransLine, WhseRcptLine2, IsHandled);
                                        if not IsHandled then
                                            // < New Event
                                        TransLine.Validate("Qty. to Receive", "Qty. to Receive");
                                    end;
.....

[IntegrationEvent(false, false)]
    local procedure OnBeforeModifyTransferLine(var TransferLine: Record "Transfer Line"; WhseRcptLine: Record "Warehouse Receipt Line"; var IsHandled: Boolean)
    begin
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Missing top.html for building doc

Hi,
I could not make html doc, looks it's because of missing top.html, this issue happens in 4.3.1 and 4.3.2, also in current git repo:
Exception occurred:
  File "/usr/lib64/python2.7/site-packages/sphinx/jinja2glue.py", line 159, in get_source
    raise TemplateNotFound(template)
TemplateNotFound: top.html

Hopefully, fixed in master. Please, try and report back in case of issues.

Seems I can also apply b0d77a9 to 4.3.1 and 4.3.2, I tried and all master/4.3.1/4.3.2 looks OK, thanks.


[NTR] enteroendocrine cell of colon

Note - please check that the term does not already exist (check OLS: https://www.ebi.ac.uk/ols/ontologies/cl)
Preferred term label:
enteroendocrine cell of colon
Synonyms
Definition (free text, please give PubMed ID in format PMID:XXXXXX)
Enteroendocrine cell that is located in the colon.
Parent cell type term (check the hierarchy here https://www.ebi.ac.uk/ols/ontologies/cl):
intestinal enteroendocrine cell http://purl.obolibrary.org/obo/CL_1001516
What is the anatomical structure that the cell is a part of? Please check Uberon: https://www.ebi.ac.uk/ols/ontologies/uberon
colon http://purl.obolibrary.org/obo/UBERON_0001155
Your nano-attribution (ORCID)
0000-0003-3440-1876
Any additional notes or concerns

[ ] 'enteroendocrine cell of appendix' should no longer reason under 'enteroendocrine cell of colon' after Uberon release and ingest

@sfexova
CL_0009042 (enteroendocrine cell of colon)


Capture and replay v0.1.0 - Removed Datastash, FetchMigration, and Plugins

Description

Removed functionality that will not be included in Capture and replay v0.1 release:

Datastash
FetchMigration
Plugins

Issues Resolved
[List any issues this PR will resolve]
Testing
Tested end-to-end solution.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Not merging to main.


fix(input): allow pointer events on placeholder

Removes the pointer-events: none from the input placeholder in order to allow for users to bind events to it. Clicking through isn't necessary anyway, because it automatically redirects focus to the underlying input already.

LGTM

Fixes #3823

This somehow made one project's e2e tests start to fail, needs investigation by caretaker.

Hi @jelbourn do you have any link where to inspect the failed tests? The bot just mentioned all Travis build passed, including e2e
https://travis-ci.org/angular/material2/jobs/217799152

@aldo-roman before merging on PRs we run them against the tests for all Google applications that use the components.
The team that owns these particular tests have made a change that should resolve the issue, I'll be able to confirm when the next set of commits are synced.


Copy-Paste doesn't work as root under macos

Description of the problem or steps to reproduce
Run micro as root under mac os.
Wrote some text
Mark some text
Copy text
Paste-Text ->> Exitcode1
Specifications
Version: 2.0.9
Commit hash: 6bc498e6
Compiled on May 16, 2021
OS: Mac OS 11.4 M1
Terminal: iTerm

Workaround: Set the clipboard to internal.


Wong rendering via GLES2/Angle on Windows

Hi, something wrong...

Do you have any idea?
PS: Code working properly with emscripten.

Looks like stencil buffer is not set up correctly. You should create render buffer with a stencil buffer.

My mistake, thanks.


dims not working on 32 bit versions

Relevant info: https://github.com/arrayfire/arrayfire/issues/1365

I took a shot in the dark here and it appears the issue resides in ct.c_longlong as it is a 64 bit value.  I changed util.py's dim4 method on line 15 to
def dim4(d0=1, d1=1, d2=1, d3=1):
    c_dim4 = ct.c_long * 4      '''32 bit value
    out = c_dim4(1, 1, 1, 1)

    for i, dim in enumerate((d0, d1, d2, d3)):
        if (dim is not None): out[i] = dim

    return out

from
def dim4(d0=1, d1=1, d2=1, d3=1):
    c_dim4 = ct.c_longlong * 4  #64bit value
    out = c_dim4(1, 1, 1, 1)

    for i, dim in enumerate((d0, d1, d2, d3)):
        if (dim is not None): out[i] = dim

    return out

and then the hello world program executed properly.
Im going to find and replace all instances with a 32bit long in the code if the only impact will be performance and memory allocation due to the 32bit map.

@colbyshores I knew what the issue was when you reported the bug. But simply changing longlong to long will break on windows. I need to fix it in a more proper manner.

Since in AF, dim4 is defined as an aggregate of four dim_t and dim_t is arch-dependent then the proper fix may be to define a c_dim_t in the Python API and use it in the utility function?

@ghisvail That's the plan. I am just a bit surprised that I did not do that yet.

Let me know when you have a fix committed and I'll apply it to the packaging. Unless you plan to make a new release soon with the fix.

This is fixed in master.

@colbyshores This fix is now both in devel and master. Can you check if this works for you ?

Looks great on the dims!  Many examples work now.  The examples that segfault are those in graphics: cos, sin, maxof, constant.  Should I create a new bug report for these?

Hmm I ran the tests and all of them passed. Forgot to run the tests though..

@colbyshores looks like the graphics examples themselves are a bit buggy. Is there any stand alone code that will reproduce the segfaults ? Can you also print the output of af.info() ?

You're right, it looks like all the methods are working except the close() method for window handling at a glance, though since you mentioned the buggy graphics I am sure you already aware of it.
ArrayFire v3.3.1 (CUDA, 32-bit Linux, build f53efc3)
Platform: CUDA Toolkit 6.5, Driver: CUDA Driver Version: 6050
[0] GK20A, 4018 MB, CUDA Compute 3.2
import arrayfire as af
af.info()
win = af.Window(800,800,"window")
a = win.close() #close method is buggy, segfauts here
print(a)
exit(0)

@colbyshores I'll look into fixing this. Thanks for taking time to reproduce the issues.

No problem, thanks for your help too.


Build and generate website with java tools

Use the appropriate java classes to create the website
Demonstrate what each java class does and create a short document that details what they do
Work with programmers to update wiki article regarding website generation
Demonstrate that you have the website up and running

https://docs.google.com/document/d/1TcbdG4jRfgQ3UGW40xKMFsqiEpggR00JCWChbIU0XkQ/edit?usp=sharing
BioGears_Java_Build.pdf
@ajbaird  @LucasJMO  kindly look at that
Tasks done while building the biogears website using java:-

[ ]  Used Java Classes to build the website

[ ] Created and uploaded the short documentation of how to build the website and what those classes are doing respectively

[ ] Website is up and running some screenshots are attached in that docx file which demonstrate the process and the website is running

Task remaining now is:-

To update a wiki article

I think this issue is ready to close out.  @LucasJMO If you bring over the wiki articles from the branch close this issue and we will start using CMD2MD.java to track the python work.

-- Pull Request Accepted and Ready for Merge


cfn-lint fails to handle resource based meta-data for ignore_checks

cfn-lint version: (cfn-lint --version)  cfn-lint 0.19.1
Description of issue.
Resource based meta-data to ignore_checks for errors and warnings doesn't work
Please provide as much information as possible:

Template linting issues:

Please provide a CloudFormation sample that generated the issue.
If present, please add links to the (official) documentation for clarification.
Validate if the issue still exists with the latest version of cfn-lint and/or the latest Spec files

  rInstanceRole:
    Metadata:
      cfn-lint:
        config:
          ignore_checks:
            - W3005
    Type: AWS::IAM::Role
    DependsOn: rInstancePolicy
    ManagedPolicyArns:
      - !Ref rInstancePolicy

another case:
  rMyRole:
    Type: "AWS::IAM::Role"
    Metadata:
      cfn-lint:
        config:
          ignore_checks:
            - E3001
    Description: "Role description"

Cfn-lint uses the CloudFormation Resource Specifications as the base to do validation. These files are included as part of the application version. Please update to the latest version of cfn-lint or update the spec files manually (cfn-lint -u)

Sorry but this feature has been coded but not released.  It will be included in the 0.20.0 release which I hope to get out this week.

I should add I tested this off of master and the ignores worked in both scenarios you provided.

I should add I tested this off of master and the ignores worked in both scenarios you provided.

Do you mean it works in 0.19.1 ?

No, this feature was merged after 0.19.1 was released.

Thanks, will wait for 0.20.0, meanwhile will clone the master and try it out

Fixed with release v0.20.0


Add postorder traversal for new atomics API

I'm trying to implement Sam's feedback in an email chain.
So, I think your detectDeletedElementsInChannel function needs two things:
-	detect elements that are sub-modeled, and recursively delete the submodel.
-	Detect elements that have children and recursively delete the children.

The prior implementation deletes elements that are scoped to their repository link, but that has to be done explicitly by the connector author? updateIModel takes a mandatory scope.
 We can't merge that because I broke the runner API to get the tooling to work, so I'll have to manually integrate any changes into main. I want you all to be able to run the (very simple) tests though.
TODO

[x] Fold status from deletion of parent (don't arbitrarily return IModelStatus.Success at the last line). It's probably best to just let this throw, and fold the statuses from the API functions that return them.

[ ] As long as throwing doesn't leave the iModel in an inconsistent state. I'm not sure if the runner is designed to intercept errors from native.

[ ] Definition elements are going to cause problems. The documentation says their deletion is O(scanning the GeometryStreams of all GeometricElement). If I'm doing a depth-first traversal of the iModel to avoid deleting elements that other elements reference, then that's very expensive, because an upper bound is O(dsl) where d is the number of definition elements, l the length of the longest geometry stream, and s the number of geometry streams.
[ ] What about relationships that point across the iModel tree and not to a descendent, like ElementRefersToElements? Needs testing to determine if it will throw. (edit: It appears referencing relationships to not throw.)
[x] The deletion should be smart because the author is responsible for synchronizing elements. If it sees that a child element has been updated but a parent above it has not, then it should not attempt to delete the parent.

Closing, will reopen with the commits from main so it's not as difficult to follow.


bug: Toggle call ionChange on view init

Type: bug
Ionic Version: 2.x
Platform: all
This is my toggle:
<ion-toggle [(ngModel)]="workMode" (ionChange)="workModeChanged()">
if this.workMode == true so workModeChanged callback will called on page init, even without click on the toggle. that is the bug.
if this.workMode == false there is no bug.
Thanks

Hello! Thanks for opening an issue with us! As this seems like more of a support question i will urge that you ask this question on our forum. Thanks for using Ionic!

@jgw96 That is not abouy support its a bug...

Hello @MatanYed sorry for the misunderstanding! Would you mind sharing a code sample that i can use to reproduce this issue? Thanks!

@jgw96 i am sorry i am from the phone so i cant put here code
But just create 2 pages
In the second page put ion toggle with ng model and ion change event
When you enter the second page and ngModel is true, ionChange event will fire
It shouldnt fire because the page only initialized

No problem, thanks!

Hi,
same issue for me. ionChange is fired on init if the ngModel evaluation is true.
In my case for the toggle component there is two-way data binding like that:
<ion-toggle primary [(ngModel)]="currentSite.usersNotifications" (ionChange)="preferenceChanged()"></ion-toggle>
The model currentSite is initialized in the constructor using a parameter of the view like that:
private currentSite: SiteModel = null;
constructor(public nav: NavController, public navParams: NavParams) {     this.currentSite = navParams.get('site');   }
Hope this help.
Thanks!

I can't reproduce this issue, in fact it was already fixed a long time ago:
https://github.com/driftyco/ionic/issues/6144
What version of ionic are you using?
Closing issue until addition info is provided. Maybe a repo?

I can reproduce the issue again on Ionic 3. I have fetched the value from backend and set the field in both constructor and ngOnInit. Both ways cause event to fire.
Ionic version:
"ionic-angular": "3.0.1"
Constructor code:
constructor(

private navCtrl: NavController,
private popoverCtrl: PopoverController,
private userService: UserService,
private toastHelper: ToastHelper) {
  this.userService.getUser().subscribe(
  user => {
    this.disableAccount = user.disableAccount;
  },
  err => {
    this.toastHelper.showTranslatedToast('connection.serverUnavailable');
    console.log(err);
  }
);

}
HTML:
<ion-item>
    <ion-label>Disable account</ion-label>
    <ion-toggle [(ngModel)]="disableAccount" (ionChange)="toggleDisableAccount()" checked="false"></ion-toggle>
</ion-item>

I can reproduce the issue again on Ionic 3. I have fetched the value from backend and set the field in both constructor and ngOnInit

but this is not a sync task... the value is changed before the component is initialized

I have just refactored code so that I don't have async HTTP request anymore.
ngOnInit() {
      this.user = this.userService.user;
  }

still fires up the event though...

@VsMaX can you update to 3.1?

Sure, I will let you know soon if the issue still persists.

Upgraded to Ionic 3.1, packages:
"dependencies": {
    "@angular/common": "4.0.2",
    "@angular/compiler": "4.0.2",
    "@angular/compiler-cli": "4.0.2",
    "@angular/core": "4.0.2",
    "@angular/forms": "4.0.2",
    "@angular/http": "4.0.2",
    "@angular/platform-browser": "4.0.2",
    "@angular/platform-browser-dynamic": "4.0.2",
    "@ionic-native/core": "3.4.4",
    "@ionic-native/splash-screen": "3.4.4",
    "@ionic-native/status-bar": "3.4.4",
    "@ionic-native/app-version": "3.4.4",
    "@ionic-native/camera": "3.4.4",
    "@ionic-native/network": "3.4.4",
    "@ionic/storage": "2.0.1",
    "ionic-angular": "3.1.0",
    "ionicons": "3.0.0",
    "@angular/animations": "4.0.2",
    "@angular/platform-server": "4.0.2",
    "@ngx-translate/core": "^6.0.1",
    "@ngx-translate/http-loader": "0.0.3",
    "@types/cordova-plugin-globalization": "0.0.3",
    "decimal.js": "^7.1.1",
    "moment": "2.18.1",
    "rxjs": "5.3.0",
    "sockjs": "^0.3.15",
    "sockjs-client": "^1.1.1",
    "stompjs": "^2.3.3",
    "sw-toolbox": "3.6.0",
    "zone.js": "0.8.5"
  },
  "devDependencies": {
    "@ionic/app-scripts": "1.3.4",
    "gulp": "^3.9.1",
    "typescript": "2.2.2"
  },

but the problem still exists

After upgrading to 3.1.1 issue is fixed.

@mcwienczek I am now on Ionic 3.2.0 and this is still an issue for me.


ui-select dropdown options not coming out of ng-grid

@Debanjan92 Try using the append-to-body attribute. The grid does not allow it's contents to overflow.

Hi,
We are using "ui-select" within "ng-grid", which is populated dynamically using cell template.
Sample cell-template:
eval("htmlDropdown = '" +
"" +
"<ui-select ng-hide=moDisabled style="width:75%;float:left;" ng-model="COL_FIELD" theme=selectize ng-disabled=disabled>" +
"<ui-select-match placeholder="Select">{{COL_FIELD}}" +
"<ui-select-choices repeat="it in opts" + gridId + " | filter: $select.search">" +
"" +
"" +
"" +
"" );
Implementing this code within an ng-grid, the options stay within the grid, as shown in the image above.
Will be highly grateful if you could kindly help to sort out the problem. :)

Hi,
We are using "ui-select" within "ng-grid", which is populated dynamically using cell template.
Sample cell-template:
eval("htmlDropdown = '" +
"" +
"<ui-select ng-hide=moDisabled style="width:75%;float:left;" ng-model="COL_FIELD" theme=selectize ng-disabled=disabled>" +
"<ui-select-match placeholder="Select">{{COL_FIELD}}" +
"<ui-select-choices repeat="it in opts" + gridId + " | filter: $select.search">" +
"" +
"" +
"" +
"" );
Implementing this code within an ng-grid, the options stay within the grid, as shown in the image above.
Will be highly grateful if you could kindly help to sort out the problem. :)

@Debanjan92 You can find an explanation of how to do that here: http://brianhann.com/ui-grid-and-dropdowns/

it worked.. Thanks :)

function setAllowClear(allow) {
$select.allowClear = (angular.isDefined(allow)) ? (allow === '') ? true : (allow.toLowerCase() === 'true') : false;
}

ng-options="item.ProfileID as item.ProfileName for item in availableProfilesProductionHistory"

How can we represent this kind of options in ui-select repeat?

Closing as a support issue.


Bug: change in get_country_data behaviour?

The notebook template is failing in cell [6] (see https://oscovida.github.io/html/Germany.html) and screenshot:

A change in behaviour would be fine I think - we just need to update the notebook template that uses it.

Associated (I think), we need updates of https://oscovida.github.io/tutorial-accessing-cases-and-deaths.nbdata.html where the code has been updated but the narrative needs changes. I'd remove the following cells and the markdown explaining the Italy example:

You're right, the number of returned values has been changed. I thought I fixed all occurrences of this function call, but that one I missed. My bad. I will rework this one as well.

Fixed the tutorial notebook in https://github.com/oscovida/oscovida/pull/136


optimise IP multicast

Currently weave implements IP multicast in terms of broadcast, i.e. multicast packets are always sent to all peers. That is sub-optimal.
Weave does observe IGMP and hence could build up knowledge about which peers contain receivers for specific multicast groups, and then use that knowledge to route packets to just those peers.

@inercia :+1:

Do you have any update on getting the multicast optimized ?

@erandu no, this has not changed

Currently weave implements IP multicast in terms of broadcast, i.e. multicast packets are always sent to all peers.

I spent some time recently understanding how multicast works in Weave. Elaborating this a bit to give perspective on the problem if some one is interested. For e.g. lets take a cluster with 5 nodes running application container accessing  <IP_ADDRESS>.  L3 multicast traffic from the container is mapped to L2 multicast and sent out. In this case <IP_ADDRESS> maps to L2 multicast IP 01:00:5e:01:02:03.
L2 multicast traffic from the container through weave bridge, and veth pairs vethwe-bridge <-> vethwe-datapath reach the OVS datapath with below ports
root@ip-172-20-62-75:/home/admin# ovs-dpctl show
system@datapath:
    lookups: hit:270 missed:269 lost:3
    flows: 15
    masks: hit:646 total:2 hit/pkt:1.20
    port 0: datapath (internal)
    port 1: vethwe-datapath
    port 2: vxlan-6784 (vxlan: df_default=false, ttl=0)

Weave programs OVS datapath datapath with below rule. Which will basically result in packet getting broadcasted to all the peers irrespective of fact that there is any container interested in receiving packets to multicast IP  <IP_ADDRESS>
in_port(1),eth(src=0a:78:b1:2c:11:ae,dst=01:00:5e:01:02:03),eth_type(0/0x0000), packets:103, bytes:12669, used:0.204s, actions:set(tunnel(tun_id=0x51b960,src=<IP_ADDRESS>,dst=<IP_ADDRESS>,tos=0x0,ttl=64,flags(df,key))),2,set(tunnel(tun_id=0x5f8960,src=<IP_ADDRESS>,dst=<IP_ADDRESS>,tos=0x0,ttl=64,flags(df,key))),2,set(tunnel(tun_id=0x732960,src=<IP_ADDRESS>,dst=<IP_ADDRESS>,tos=0x0,ttl=64,flags(df,key))),2,set(tunnel(tun_id=0x5ae960,src=<IP_ADDRESS>,dst=<IP_ADDRESS>,tos=0x0,ttl=64,flags(df,key))),2,set(tunnel(tun_id=0x44b960,src=<IP_ADDRESS>,dst=<IP_ADDRESS>,tos=0x0,ttl=64,flags(df,key))),2,0

Similarly each peer is configured to receive packets to <IP_ADDRESS> irrespective of any local container interested in receiving the traffic.
tunnel(tun_id=0x96051b/0xffffffffffffffff,src=<IP_ADDRESS>/<IP_ADDRESS>,dst=<IP_ADDRESS>/<IP_ADDRESS>,tos=0/0,ttl=0/0,flags(key)),in_port(2),eth(src=62:ae:1b:34:4e:76,dst=01:00:5e:01:02:03),eth_type(0/0x0000), packets:102, bytes:12546, used:0.552s, actions:1,0
tunnel(tun_id=0x96044b/0xffffffffffffffff,src=<IP_ADDRESS>/<IP_ADDRESS>,dst=<IP_ADDRESS>/<IP_ADDRESS>,tos=0/0,ttl=0/0,flags(key)),in_port(2),eth(src=ea:f2:dd:f4:f2:c1,dst=01:00:5e:01:02:03),eth_type(0/0x0000), packets:103, bytes:12669, used:0.016s, actions:1,0
tunnel(tun_id=0x9605f8/0xffffffffffffffff,src=<IP_ADDRESS>/<IP_ADDRESS>,dst=<IP_ADDRESS>/<IP_ADDRESS>,tos=0/0,ttl=0/0,flags(key)),in_port(2),eth(src=a2:a9:d0:e9:99:c0,dst=01:00:5e:01:02:03),eth_type(0/0x0000), packets:103, bytes:12669, used:0.116s, actions:1,0
tunnel(tun_id=0x9605ae/0xffffffffffffffff,src=<IP_ADDRESS>/<IP_ADDRESS>,dst=<IP_ADDRESS>/<IP_ADDRESS>,tos=0/0,ttl=0/0,flags(key)),in_port(2),eth(src=1a:1f:83:fa:c1:fb,dst=01:00:5e:01:02:03),eth_type(0/0x0000), packets:106, bytes:13038, used:0.245s, actions:1,0

This is sub-optimal as reported in this issue.

It should be easy to incorporate semantics of IGMP protocol in the Weave and based on the membership populate the OVS datapath to broadcast only to the intended group members.

@murali-reddy , having more specific rules may be disadvantageous, because rule processing within OVS consumes resources, requires careful table design, etc. Having a single generic rule speeds things up in terms on flow processing. It is a trade-off.

Out of curiosity, is weave still implements IP multicast using broadcast? Thanks

The implementation has not changed, however I don’t think the original wording conveys the correct idea.
If you have a cluster of 3 machines and are running 10 containers that receive multicast, Weave Net will do 2 unicast sends to convey the packets machine-to-machine, then inject the packets as multicast to be received by the 10 containers.
Matthias meant that the machine-to-machine part always reaches all machines, not that it is literally implemented using broadcast.

So like this?


Doesn't work with PDF

docquery scan "What is the invoice number?" https://slicedinvoices.com/pdf/wordpress-pdf-invoice-plugin-sample.pdf
Errors:

FileNotFoundError: [Errno 2] No such file or directory: 'pdfinfo'

pdf2image.exceptions.PDFInfoNotInstalledError: Unable to get page count. Is poppler installed and in PATH?

Hi Vanilize, do you have poppler installed on your system? The error seems to indicate that it's missing. I ran it on my machine and saw this output:
$ docquery scan "What is the invoice number?" https://slicedinvoices.com/pdf/wordpress-pdf-invoice-plugin-sample.pdf

image-classification is already registered. Overwriting pipeline for task image-classification...
2022-09-18 09:10:00,114 INFO: Loading https://slicedinvoices.com/pdf/wordpress-pdf-invoice-plugin-sample.pdf
2022-09-18 09:10:00,114 INFO: Done loading files. Loading pipeline...
2022-09-18 09:10:01,397 INFO: Ready to start evaluating!
https://slicedinvoices.com/pdf/wordpress-pdf-invoice-plugin-sample.pdf What is the invoice number?: INV-3337

you were right! Thanks

Thanks for following up -- please let me know if you have any other questions. We hope to address this confusing UX in #20 since for the PDF you linked, Poppler is technically unnecessary.


As a user I want a link to this issue tracker from the SWAGGER documention

When I come to the API, and i do some testing, or have a question, then I want a quick way to raise it here (if this is the canonical spot).

added to https://app.swaggerhub.com/apis-docs/dimus/gnames/1.0.0
If you found a bug or want to request a feature for API, create an issue at
https://github.com/gnames/gnames/issues

If you use a web-site at https://verifier.globalnames.org or command line
client from https://github.com/gnames/gnverifier and want to report a
bug or request a feature, create an issue at
https://github.com/gnames/gnverifier/issues


License?

Cool library! What license is this released under? Can you add a LICENSE file?
https://help.github.com/articles/open-source-licensing/

If you already have a project on GitHub, and it doesn't have a license file or other text indicating what rights you are granting to others, you retain all rights to the software, except those outlined in the GitHub Terms of Service.

I'm pretty sure that means nobody else can use your code at all :disappointed:
http://choosealicense.com/

@lukeyeager Thanks! Added MIT license.


Buildkite cache improvements

Quick note on what we have in https://github.com/wellcomecollection/platform-infrastructure/pull/64
Instances are reused between jobs, so they share the Docker and sbt cache. But when instances restart, the cache is lost.


Missing Output for Some Positions

Appreciate this is an old repository but just as a heads-up for future users: this library seems to miss positions. For example:
Some examples using hg19ToHg38.over.chain.gz (md5sum 35887f73fe5e2231656504d1f6430900):

chr6:915652 (UCSC liftover returns chr6:915402-915402), this library returns no matches
chr17:296626 (UCSC liftover returns chr17:446835-446835), this library returns no matches
chr12:153058 (UCSC liftover returns chr12_GL877875v1_alt:31786-31786), this library returns no matches

I'll create a PR when I've found the issue!

There also seem to be a few issues that give an output which is incorrect:
('chr2', 845936) (expected[i]: ('chr2', 850281), pyliftover[i]: ('chr2', 850329, 1, '+', 3910))
('chr2', 843218) (expected[i]: ('chr2', 847216), pyliftover[i]: ('chr2', 846736, 1, '+', 1043))

Please note that this library uses 0-based coordinate indexing while the web-based tool uses 1-based indexing.
In particular, the examples you brought in your first comment should be converted as follows:
from pyliftover import LiftOver
lo = LiftOver('hg19', 'hg38')
[('chr6', 915151, '+',<PHONE_NUMBER>1)]
> [('chr6', 915401, '+',<PHONE_NUMBER>1)]  # Corresponds to chr6:915402 in 1-based indexing
lo.convert_coordinate('chr17', 296625)
> [('chr17', 446834, '+',<PHONE_NUMBER>)]  # Corresponds to chr17:446835 in 1-based indexing
lo.convert_coordinate('chr12', 153057)
> [('chr12_GL877875v1_alt', 31785, '+', 127260)]  # Corresponds to chr12_GL877875v1_alt:31786

Quite a number of issues claiming to detect mismatches with UCSC Liftover have been reported ever since the algorithm was first implemented in 2013, yet not a single one of them turned out to be a valid bug requiring any change to the algorithm - it is  always related to confusion around 0-based and 1-based indexing.


Indent after numbers is not consistent with reference commonmark implementation

The following text:
1. foo

     this should not be code

Renders the last line as code, rather than text. I think this is a bug.
This does not match the reference commonmark implementation: http://spec.commonmark.org/dingus/
The majority of other implementations do not show this behaviour: http://johnmacfarlane.net/babelmark2/?normalize=1&text=1.+foo

++++bar
I believe the relevant part of the commonmark spec is 5.3 Lists, which says:

A list is a sequence of one or more list items of the same type. The list items may be separated by any number of blank lines.

Note that this was confusingly worded in earlier versions of the spec: https://talk.commonmark.org/t/multiple-blank-lines-inside-a-list/2289

What version of commonmark-java are you using? Just tried with master, and the following string:
"1. foo\n\n\n     this should not be code"

renders like this:

foo

this should not be code

This has been changed in commonmark-java 0.6.0 when we updated to spec 0.26, see release notes: https://github.com/atlassian/commonmark-java/releases/tag/commonmark-parent-0.6.0

Aha, excellent news!
I'm using commonmark-java on Bitbucket server v4.12.0. I don't suppose you know which commonmark-java versions are used in later versions of bitbucket server?

Just checked, 4.12.0 shipped with 0.5.1. Would you mind filing an issue for updating to a newer version here?: https://jira.atlassian.com/projects/BSERV
Thanks! :)


kubectl leaks tokens for configs created with the azure command line tools

When connecting to an Azure AKS cluster, you run the command az aks get-credentials to generate a local ~/.kube/config file. For whatever reason, this config file has both client-certificate-data and a token. When you run kubectl config view, the certificate data is redacted, but the token is displayed in plain text.
For example, this is an example of the result of running kubectl config view after running az aks get-credentials. Note the token is not redacted:
apiVersion: v1
clusters:
- cluster:
    certificate-authority-data: REDACTED
    server: https://urlgoeshere:443
  name: k8scluster
contexts:
- context:
    cluster: k8scluster
    namespace: default
    user: clusterUser_k8scluster_k8scluster
  name: k8scluster
current-context: k8scluster
kind: Config
preferences: {}
users:
- name: clusterUser_k8scluster_k8scluster
  user:
    client-certificate-data: REDACTED
    client-key-data: REDACTED
    token: 12345678ABCDEFG

/sig cli
/area kubectl
/kind bug
/remove-lifecycle stale

/remove-lifecycle stale

When you run kubectl config view, the certificate data is redacted, but the token is displayed in plain text.

Is something running kubectl config view and logging the results automatically?

If you have access to run kubectl config view, you could also just run cat $KUBECONFIG and view the config file directly. kubectl config view just echoes the contents of the config file it is using.

@liggit is right, the REDACTED text is really just a mechanism to hide the text from plain sight, but there is nothing stopping you from reading the file directly and seeing everything unredacted.
Anyway, i submitted a PR which makes token behave like the other fields and show REDACTED, but I’m not sure why this issue was not auto closed when it merged.
I’ll see if I can close the issue myself.
/close


Limit the google-cloud packages used

We currently bring in the umbrella google-cloud package, which has a ton of packages. We should just use the packages we need.

This is fixed


LEP-241 MTR phase 2 mandatory income disregards

https://dsdmoj.atlassian.net/browse/LEP-241

Checklists
Author: (before you ask people to review this PR)

[ ] Diff - review it, ensuring it contains only expected changes
[ ] Whitespace changes - avoid if possible, because they make diffs harder to read and conflicts more likely
[ ] Changelog - add a line, if it meets the criteria
[ ] Secrets - no secrets should be added
[ ] Commit messages - say why the change was made
[ ] PR description - summarizes what changed and why, with a JIRA ticket ID
[ ] Tests pass - on CircleCI
[ ] Conflicts - resolve if Github reports them. e.g. with git rebase main

Reviewers remember:

[ ] Jira ticket criteria are met
[ ] Migrations - test migration and rollback: rake db:migrate && rake db:rollback

Should have created branch from main


XQuery source formatting error

Extension Version
0.1.7
Description
Formatting src\test\xq-data\ast-to-xqdoc.xqm  changes text around line 156 when formating doc or selection
Screenshots
Before

After

VS Code Version
What version of VS Code are you using?
Operating System
What OS (and version) are you using?

Fixed in 0.1.14


Better props example

In the README you say:

props doesn't need ${...props} or ...${props} //  because any value can be passed as interpolation

That doesn't seem to me to be literally true. I'd expect that in the case of <tag attr=${value} /> any value would be legal. In the case of <tag ${value} />, it would seem that not all values would be legal though. <tag ${2} /> seems like nonsense.
So what is legal? Just {[attr]: value} objects? ESX fragments? If it's ESX fragments you'd have to be sure that the fragment is not itself a tag, because <tag <tag />/> is also nonsense.

It seems to me that there may well be reason to implement <tag ...${props} /> as it creates a useful and unambiguous declaration of intent and doesn't open the door to invalid constructions.

Tokens as attributes make no sense, but you could pass it and deal with it.
…${any} is ambiguous, as it can be also part of content where those dots are just part of the text, meaning nothing. I think not needing spread is ok.

P.S. if you pass a value without being an attribute assignment the assign operation happens, and that’s fine, as that’s exactly what React and others do too. Don’t pass non props or attributes related content, but feel free to.

Also P.S. this has been discussed in the TC39 proposal. Please read it through to understand the choice made here.

<tag ${2} /> seems like nonsense.

as mentioned in the README, it’s not a goal of this proposal to limit developers intent. That is an interpolation as part of attributes with value 2. It’s not on me or you to decide if that makes sense or not, as ESX purpose is to tokenize and don’t impose any opinionated choice behind. The only opinion, because there wouldn’t be ESX otherwise, is to provide tokens in a JSX like syntax, not to tell developers what to do with tokens. When it comes to the current known React influenced world, 2 as props makes no sense, but props here are just a backward compatible helper to satisfy current expectations, although attributes are completely free from expectations or caveats, allowing developers to explore better alternative to the current JSX de-facto standard, as that’s also going nowhere.

I don't understand. Ensuring your tokens make up a basic kind of syntax isn't the same as telling developers how to use the syntax. XML has clear syntactic rules: <a 2 /> is not valid XML. So what would be the point in ensuring that it's possible for XML like this to be produced specifically when XML-like syntax is written in a javascript template tag?

If your parser rejects that static syntax, and it should reject any dynamic interpolation that would lead to a syntactically incorrect structure that could not be understood as a collection of nodes, children, and attributes.

With this reasoning I see no problem with  <a ...${props} /> because <a ... /> is not valid XML, therefore there is no ambiguity.

It's particularly important because if you don't forbid invalid syntax you'll never be able to extend your syntax at all!

ESX is not XML … it’s JSX, which is also neither XML nor JS. ESX doesn’t sanitize anything, it leverages developers intent and it represents tokens. If a developer wants to use 2 statically that’s ignored. If a developer wants to use 2 as explicit interpolation intent, that’s represented by the tokenizer.

This is also described in the README in the about section, so I hope the goal scope of this project is clear. ESX is a tokenizer, not HTML, not XML, it tokenizes JSX like syntax. As this is discussed in TC39 proposal and clarified in the README, I might improve that part to clarify further, but I won’t change ESX goals. Accordingly, I think there’s no action for me to take in here.

I see, that does clarify a bit.

To close this: ESX in here will never sanitize as a whole parser would make it unusable, slow, and bloated for no real-world need or use case. Anything else can be discussed as TC39 proposal, and this library will adjust when, or if ever, needed, just like any transformer or tokenizer. It won’t happen in here at this point though, but I might relax the RegExp around attributes to ignore … that’s still not sanitizing anything, it would just accomodate people believing … in attributes are needed. To me that’s just ambiguous syntax out of template literals, and it brings nothing to the plate. FYI in JSX you can spread even a function without errors, and props will fail your expectations. That’s it, spreading limits ESX, and I don’t like it.

To whom it might concern, it's important to realize that <element ...${spread} /> can be written but there won't be any extra information that it was a spread operation. Anything in attributes that's not an assignment gets discarded.


Added site playoff bracket

Added a new table: playoff_series
This table is similar to the series table.
Added a bunch of stuff to create, view, edit, list these new playoff_series. These are all very similar to their respective series.
Added a new page called bracket. This displays all of the playoff_series that exist for a defined season. It uses round and match_number to determine how to place the matches/series in the bracket.

Round determines which column the match goes into
Match number determines which slot the match goes into within that column.
Using incorrect numbers will horribly disfigure the layout of the bracket, but if done correctly it should look something like this.

In order for the bracket to resemble the challonge bracket someone has to make all the series. All admins have the ability to do so. Admins also see an edit tag on the bracket.

I'm not a big fan of having a new series table. It would probably make more sense to have seasons have a specified number of rounds, after which everything is assumed to be post-season games.
I'll take a closer look at the PR tomorrow to see if there's anything that would conflict with what I've suggested.

the biggest difference between the playoff_series table that I added and the existing series table is that playoff_series have a home_division_id and an away_division_id since teams can be playing eachother from different divisions.
The main reason I thought that a separate table was best was because adding a home and away division to the existing series table would invalidate all the existing records and we would need to make a migration task to populate the new columns.
You guys know the code and processes better than I do so if you think modifying the series table to handle playoff games then it can be done moderately easily

You can derive the division_id for each team by joining the team table, so I don't believe it would be necessary to create any new columns, actually. Should largely eliminate the benefit of a separate table.
However, I don't have any suggestions for deriving match_number yet, but I suspect it might be possible to do something using SoS. Need to look a bit more at the bracket function.

@tristan-gill another thing you can do is just add a t/f flag on series if it is playoffs or not

yea ill probably get around to redoing this pr later in the week


Styling and Dark Mode

Potential Issues
When I switch between dark and light mode, I have to change the colors in the settings to increase contrast to make the text legible.
Feature Description
An option to display the stats just like they are on the home screen. So it could say:
Studied X cards in Y hours this week (Zs/card)
Studied A cards in B hours total (Cs/card)

This would be right below the normal text to maintain consistency and location of information. Here is a photo of the home screen with the relevant text:

Alternatives Considered
Color picker for dark mode and light mode separately.

Should be added in the most recent release (v2.0.0)!
Thanks, again, for the request!


Delegator not looking for aliases

Good morning,
I've the following configuration:
    // Container configuration
    'dependencies'      => [
        'invokables' => [
            CommandBus::class => SimpleCommandBus::class,
        ],
        'factories'  => [
            EventBus::class                  => EventBusFactory::class,
            EventStore::class                => EventStoreFactory::class,
            PhpVersionRepository::class      => PhpVersionEventStoreRepositoryFactory::class,
            PhpVersionCommandHandlers::class => PhpVersionCommandHandlersFactory::class,
        ],
        'delegators' => [
            CommandBus::class => [
                PhpVersionCommandBusDelegatorFactory::class
            ]
        ]
    ]

Problem is that when I get the Command::Bus service, the PhpVersionCommandBusDelegatorFactory isn't invoked.
However, the following works:
    // Container configuration
    'dependencies'      => [
        'invokables' => [
            CommandBus::class => SimpleCommandBus::class
        ],
        'factories'  => [
            EventBus::class                  => EventBusFactory::class,
            EventStore::class                => EventStoreFactory::class,
            PhpVersionRepository::class      => PhpVersionEventStoreRepositoryFactory::class,
            PhpVersionCommandHandlers::class => PhpVersionCommandHandlersFactory::class,
        ],
        'delegators' => [
            SimpleCommandBus::class => [
                PhpVersionCommandBusDelegatorFactory::class
            ]
        ]
    ]

Why there is no alias matching there?

This repository has been closed and moved to laminas/laminas-pimple-config; a new issue has been opened at https://github.com/laminas/laminas-pimple-config/issues/1.


| over multiple lines

Breaking up a regex with a | over multiple lines messes up the line numbering of following lines:
<LOOP>{
\bwhile\b|
(?^^\h+/while\b)	return yy::parser::make_WHILE(location());
}

Doesn't appear to mess up:
%%
abc     |
def     ECHO;
%%

lex.yy.cpp:
          case 1: // rule at line 2: abc
          case 2: // rule at line 3: def
#line 3 "test2.l"
ECHO;
            break;

The | should be placed after the pattern, otherwise reflex throws a syntax error.

Yes, I thought it helped to solve the issue.


Improve speed and memory usage with simple integer indexes

http://stackoverflow.com/questions/13692418/does-pandas-cache-values-on-ix-call
I believe this will be mostly remedied when the equivalent of the range-index branch is merged.
import pandas as pd
import numpy as np
import datetime as dt
import matplotlib.cbook as mc
import inspect

print 'pandas', pd.__version__

li_list = []
for i in range(50):
    li_list.append(pd.DataFrame(data=np.random.randn(50, 17000)))

def first_element():
    a = 0
    for i in range(50):
        b = li_list[0] #Only access first element
        for j in b.columns:
            a += b.ix[i, j]

def all_in_list():
    a = 0
    for i in range(50):
        b = li_list[i] #Access all in list
        for j in b.columns:
            a += b.ix[i, j]
            print(inspect.getabsfile(b.__class__))
            break

if __name__ == '__main__':
    # first_element()
    all_in_list()
    print(mc.report_memory())

closed by https://github.com/pydata/pandas/pull/11892


Profiles do not work as documented

I took this command from the manual:
nix-env -p /nix/var/nix/profiles/other-profile -i subversion
It results in
error: opening lock file ‘/nix/var/nix/profiles/other-profile.lock’: Permission denied
See chapter 9 of the Nix manual for the expected behaviour.
If, however, this is a user-error (can't imagine how), then I would like the error message to say what permissions were expected.

Does your user have write permission into the requested profile files? Normal user would not have it in your particular command.
With -p /nix/var/nix/profiles/per-user/$USER/other-profile it works for me fine. Let's keep #17017 open instead, as the documentation isn't good about this.


Make description an Option<DocValue> and implement markdown rendering.

It seems like this is due the the markdown transition in nixpkgs: option descriptions may now look like {"_type":"mdDoc","text":"..."}, which does not deserialise to Option<String>.
This was not supposed to be a problem until flakes started using lib.mdDoc, but, since we index _module.args for all flakes and that was given a markdown description in https://github.com/NixOS/nixpkgs/commit/e04a09082e33d444c245e1d5859b37ace25f54a9, things break for all flakes.
So, amusingly, https://github.com/NixOS/nixos-search/pull/528 fixes this issue, although that was not intended at all.
The proper fix, of course, is to make description an Option<DocValue> and implement markdown rendering.
~Also we should probably fail the CI job when no flakes are successfully imported.~ That wouldn't have caught this since flakes with no modules are still succeeding.
Originally posted by @ncfavier in https://github.com/NixOS/nixos-search/issues/534#issuecomment-1240357880

Why do we need _reverse fields?

That was some elasticsearch optimization if i recall correctly


[RFE] Add Support for "resourceNames" in Rancher's RBAC

This is literally a continuation of issue #31329 from @asemen2608 which was closed due to stale bot, but still relevant in Rancher 2.6.6. Part of the old issue's text pasted.
Feature request:
The resourceNames field should be exposed and usable when creating custom roles in Rancher.
Our specific usecase of using resourceNames in RBAC is giving 'get' rights to specific clusterroles and also to specific crd's.
Rancher role templates are just Kubernetes roles. As such, whilst the Rancher UI does not expose the resourceNames field of rules, this field exists on the API object (schema can be seen at https://<rancher_url>/v3/schemas/policyRule).
Therefore, a user can add or edit resourceNames for a role, within the API Explorer, by clicking 'View in API' for a role from the Rancher UI.
Other usecase in old issue
Limit at Rancher's level RBAC to prevent people from deleting specific namespaces with a given name.
In Kubernetes, that would be translated to:
rules:
apiGroups: [""]
resources: ["namespaces"]
resourceNames: ["cattle-system"]
verbs: ["list", "get", "watch"]
However, the particular field "resourceNames" isn't visible for end-users in Rancher.

I totally agree on this. It is now not possible to give cluster members more permissions on their own resources like crd's, without opening access to all other crd's. This just one example we experience too.


[Robot] Added tests for PMM homepage

W-8956949
Validates the following are displayed on homepage:

Clients with missing phone number report
All Active Program Engagements report
Services delivered (year to date) report
Recent Items component
Today's Tasks component

Critical Changes
Changes
Issues Closed
New Metadata
Deleted Metadata
Definition of Done
Refer to Asteroids DoD document to see any additional details for the items below

[ ] Any net new LWC work has Sa11y tests & 50% or above lines JEST test coverage
[ ] CRUD/FLS is enforced in Apex
[ ] Permission sets are updated to account for CRUD|FLS|Tab|Classes
[ ] Field sets are updated to account for new fields
[ ] UX approval or UX not necessary
[ ] Link the pull request and work item by PR comment and Chatter post respectively, e.g. GUS: W-0000000: Work Name (https://github.com/SalesforceFoundation/PMM/pull/303)
[ ] All acceptance criteria have been met

[ ] Developer
[ ] Code Reviewer
[ ] QA

[ ] PR contains draft release notes
[ ] QE story level testing completed

@boakley would you mind giving this a once over?


⚠ Bofrost.fr has degraded performance

In c0ad887, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1312 ms

Resolved: Bofrost.fr performance has improved in fa9e45c after 6 minutes.


Upgrade to V10 of Lighthouse as it has updated weights on score calculation

Is your feature request related to a problem? Please describe.
Inaccurate scoring when comparing external lighthouse tools (Chrome Dev tools, Pagespeed Insights),
Describe the solution you'd like
Upgrade the lighthouse package to latest version

Hi @BrentGEverlaw! Apologies for the delayed response, but please be assured this is on our radar. The upgrade to v10 requires some manual intervention, and the team will be looking into a fix.

Any updates?


added file for vibrant school

@miradnan Pull request merged. Thank you!
Please notice that we recommend students who are younger than 13 of age to use Community Editions of JetBrains tools, which do not require a license/subscription at all, even free one.


Model diagrams for the GNN examples

❓ Questions and Help
This repo presents couple of nice examples for the GNN.
I am particularly interested about:

Semantic Parsing
NMT

Do you have the model architecture described somewhere as part of the tutorial or documentation?
Alternately do you have a canonical architecture described somewhere for these Graph2Seq based models?
Is the model same as the Graph2Seq: A Generalized Seq2Seq Model for Graph Inputs?

@AlanSwift @hugochan can you please help?

Currently, we don't provide the architecture graph about the specific applications. But we have visualized specific graph types such as dependency and etc. in our survey paper.

There are some differences.
We apply RNN or bert encoding before GNN to initialize the node embedding. And we use separate attention: 1. attention on node embedding, 2. attention on node initial embedding. This is just an example. For more details, please refer to our docs.

@AlanSwift thanks for your response. But I can't find that much details in the document.
I see at first you generate initial node embedding using word2ve or BERT.
But your statement about the separate attention: 1. attention on node embedding, 2. attention on node initial embedding is not clear. Can you please elaborate? 🙏

@AlanSwift also a bit confused here you said, we use separate attention:

attention on node embedding
attention on node initial embedding.

But the example for NMT is with a GCN. GCN does not use attention. So lost here. Please elaborate so that I can understand the model a little better.

Just an example:
encoder pipeline: RNNencoder --> GNN encoder
decoder pipeline: 1. attention on RNNencoder results 2. attention on GNN encoder results. 3. fuse them

@AlanSwift this RNNencoder comes after the word2vec or bert embedding. The document states:
For instance, for single-token item, w2v_bilstm strategy means we first use word2vec embeddings to initialize each item, and then apply a BiLSTM encoder to encode the whole graph (assuming the node order reserves the sequential order in raw text).

I do not understand why/how the BiLSTM encoder is used to encode the whole graph. Can you please explain?

@AlanSwift this part is quite confusing. Why/how do you encode the whole graph with the BiLSTM?
Also for the decoder pipeline, you mentioned:

attention on RNNencoder results
attention on GNN encoder results.
fuse them

Any other paper used this approach? Can you please provide me any reference paper?
Also would appreciate if you provide me with pointers how is it done in the code?

The BiLSTM, Bert are used to initialize the node embedding, which can enrich contextural information. This trick is widely used in NLP&GNN researches https://arxiv.org/pdf/1908.04942.pdf (only an example).
For technique details, please refer to the implementations.

@AlanSwift I not asking about word2vec or BERT to initialize the node embedding. I am asking why the BiLSTM is used after learning word2vec  or BERT embedding.

Considering the bidirectional sequential information is beneficial for most NLP tasks.

@AlanSwift got it. But why the BiLSTM encoder to encode the whole graph? I would be thinking that is used to update the embedding for the node. Isn't it?
Is the description incorrect?

@AlanSwift the BiLSTM encoder is used to update the embedding for the node embedding. So it appears to me:

initialize node embedding with word2vec or BERT
update using BiLSTM

Now feed it to the GCN encoder.
Is this understanding correct?

@AlanSwift I understand bidirectional sequential information is beneficial for NLP tasks. But the BiLSTM encoder updates the initial word2vec/bert word embedding before feeding it to the GCN encoder.
So I am confused when you state that BiLSTM encoder to encode the whole graph.
Would you please assist me with this question?

This issue will be closed. Feel free to reopen it if needed.


Clean up acceptance spec helper

Tests should be fixed with https://github.com/voxpupuli/puppet-python/pull/519. Could you please rebase your branch? :)

Rebased on master.


Synchronise feature

Scope:

implementation of the button in the top bar + menu to select the morphology viewers (3d viewers) to select and sync together.

implement logic to handle all the cameras together across different morphology viewers.

@lrebscher I increased the estimate from the user stories by 8 hours to address inside this card also the button + menu implementation, plus whatever additional logic is required for the synchronisation but this is just a rough estimate so feel free to change this number.

estimate is fine for me

modified during the call with Noelle and the team.
Scope of the implementation, have a single button to trigger the 360 rotation of all the morphology viewers on focus on neuroscan.


Pineapple pro is now Pineapple

changed name to reflect change on the website
upgraded to latest stable version

Thank you for the contribution. It needed some fixes, so they were made in c6d6c8cd11989fdb0e68cc27070af986c5d1c912. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.

ok, thanks and sorry for the mess!


[FLINK-17889][hive] Hive can not work with filesystem connector

What is the purpose of the change & Brief change log

Hive jar should not include format java SPI file. Modify hive pom to exclude SPI file.
relocation flink-parquet, Because we have already relocated the dependency of parquet, if we use hive and FS parquet at the same time, no relocation will cause class conflict.

Verifying this change
Manually testing.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no

One minor suggestion: we should use the same package prefix for all the shaded dependencies.


⚠ HopeAlley Ecards has degraded performance

In debec6b, HopeAlley Ecards (https://hopealley.com/ecards) experienced degraded performance:

HTTP code: 200
Response time: 6039 ms

Resolved: HopeAlley Ecards performance has improved in fa2140a after 8 minutes.


Add Q_ID to ElemwiseBinary/ELEMWISE_BINARY_EQ

Please have two api calls.
The original api(args...), without the queue_id, that calls the one that requires the queue_id by setting queue_id to 0?
api(args...)
api(queue_id, args...)
This is better than having a default argument for queue_id.

Working on this now.

This PR adds support for queue_id argument in ttnn C++/py.
Please read PR description for details/usage example:
https://github.com/tenstorrent/tt-metal/pull/9071

I tested the EQ output in int and here are the results.
https://docs.google.com/spreadsheets/d/1us0n4F3Sm90IhB07rWP2j-x4ANu8ssehnVGfSUzjMN8/edit?usp=sharing
#9177 adds cq_id support  Binary EQ in tt_lib.

@KalaivaniMCW , thank you! Looks good. Merged now.


Trying to implement Jaccard similarity comparison

from chatterbot import ChatBot
from chatterbot.comparisons import JaccardSimilarity

chatbot = ChatBot(
    'Test bot',
    statement_comparison_function=JaccardSimilarity
)

I'm trying to implement the Jaccard similarity comparison, but even just using a basic setup with or without any logic adapters renders the error TypeError: get_initialization_functions() missing 1 required positional argument": 'self'
I can provide the entire exception thrown if necessary

You could write something like this
chatbot = ChatBot(
    "My ChatterBot",
    logic_adapters=[
        {
            "import_path": "chatterbot.logic.BestMatch",
            "statement_comparison_function": "chatterbot.comparisons.jaccard_similarity",
            "response_selection_method": "chatterbot.response_selection.get_first_response"
        }
    ]
)

or
from chatterbot import ChatBot
from chatterbot.comparisons import jaccard_similarity

chatbot = ChatBot(
    'Test bot',
    statement_comparison_function=jaccard_similarity
)

The comparison class are converted to calable names at line number 325
More information about comparison functions http://chatterbot.readthedocs.io/en/stable/conversations.html#statement-comparison

Ah, okay. Everything appears to be functioning as normal now after using the correct class. Thank you


Wrong path

The path under "Install USB Drivers" isn't present on my machine:
C:\Users[username]\AppData\Local\Android\android-sdk
This does exist
C:\Users[username]\AppData\Local\Android\sdk
But there's no android.bat file in that location. There is an android.bat file in:
C:\Users[username]\AppData\Local\Android\sdk\Tools
Is this the file I need to run?

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

ID: 0fe8d6a6-7a42-810a-d5d4-e3bd162ae09a
Version Independent ID: c7851826-27c9-9126-cb46-9300c57c440b
Content: Set Up Device for Development - Xamarin
Content Source: docs/android/get-started/installation/set-up-device-for-development.md
Product: xamarin
GitHub Login: @mgmclemore
Microsoft Alias: mamcle

That is the file you need to run.
Sorry about the confusion. Sometimes the exact path to where the Android SDK is located (and how the Android SDK organizes its files) changes over time.  We'll add this to our backlog to fixup .  Thanks for reporting this.