DynDNS: Check IP Services - plugin support

Use the Check IP Services plugin if it is installed.
No functional change to the core default.

Optional function variable (service name, default null).
Capture regex as variable.
Only execute if the Check IP plugin is installed.

For Check IP Services - Plugin (opnsense/plugins#1731)

It's probably easier to write a new dyndns plugin, get_dyndns_ip() is a remnant of two old plugins which aren't very well supported anymore. In core we don't accept dependancies on other plugins, which defuses separation of concerns.

Designed to work with the two plugins, DynDNS and RFC2136, as they currently exist.  To also be completely ignored if the CheckIP plugin is not installed.  Leaving the behavior unchanged.
Being that those two plugins are the only thing that use the get_dyndns_ip function and nothing in core uses it this is pretty insignificant.  None at all for those who do not use the CheckIP plugin.
Creating a new DynDNS plugin that also takes into account RFC2136 and CheckIP would be more work than it may seem.
Would it suffice to separate out the get_dyndns_ip function from etc/inc/util.inc into a  plugins.inc.d include?

I'm afraid not, eventually it should be moved out of core. The code quality of both legacy plugins is quite poor, which isn't a huge issue if it more or less works and no one wants to work on a new and better structured version.
Hard (optional) dependencies in core are definitely a no go area, if hooks are missing, we should consider adding them, in this case the function shouldn't be in core at all.
In theory this doesn't seem like the most complicated software to build, but as core team we don't intend working on this at the moment.
There might even be other external tools which are being maintained and could be implemented in OPNsense, the looser coupled the better.

Update: Removed in core dependency on other plugins.

as explained in https://github.com/opnsense/core/pull/3971#issuecomment-599967614 , currently we're not planning to pull this in.


Thank you

Thanks to your script, I made this image for the new year. I sent it by email.

Hah, that's awesome! Somehow I missed this ticket until today, 3+ years later!


cc.org/publicdomain redirects to random blog post

See https://creativecommons.org/publicdomain redirecting..

@janeatcc should it redirect here: https://creativecommons.org/share-your-work/public-domain/cc0/

No. There was previously a page that explained both tools (CC0 and PDM)
that used to exist at that URL. Not sure where it went.
On Thu, Sep 1, 2016 at 8:28 AM, Jennie Rose Halperin <
<EMAIL_ADDRESS>wrote:

@janeatcc https://github.com/janeatcc should it redirect here:
https://creativecommons.org/share-your-work/public-domain/cc0/
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/creativecommons/creativecommons.org/issues/448#issuecomment-244116224,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGGTmcdejb0HMECDDb71RRsLdoZsEgqoks5qlu8ogaJpZM4JtU-N
.

--
Jane Park
Director of Platforms and Partnerships
Creative Commons
@janedaily
Get updates: http://bit.ly/commonsnews
Invest in the commons: https://donate.creativecommons.org/

@robmyers @ericsteuer does anyone know the answer to this?

There are a couple of redirects that need fixing.

https://creativecommons.org/publicdomain/
note the trailing slash
On Thu, Sep 1, 2016 at 9:13 AM, Jennie Rose Halperin <
<EMAIL_ADDRESS>wrote:

@robmyers https://github.com/robmyers @ericsteuer
https://github.com/ericsteuer does anyone know the answer to this?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/creativecommons/creativecommons.org/issues/448#issuecomment-244130614,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOI8VjcyH_1lBGPqiRKfFxIvmCi_YYsks5qlvmzgaJpZM4JtU-N
.

--
Invest in an open future. Support Creative Commons today.
https://creativecommons.org/donate/

Similar to this issue: https://github.com/creativecommons/creativecommons.org/issues/434

Yes this and at least one other page need the web server config changing to cope with missing trailing slashes. I will do this soon.

The trailing slash is now added if missing.
Thank you everyone for reporting and working through this.
(Anyone looking to edit /publicdomain/: it is not in WordPress, it is in python_env.)


Changed int to SQLLEN to avoid infinite loop

This change is

Codecov Report

Merging #806 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev     #806   +/-   ##
=======================================
  Coverage   80.06%   80.06%
=======================================
  Files          25       25
  Lines        7325     7325
=======================================
  Hits         5865     5865
  Misses       1460     1460

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 eeea787...d1f95de. Read the comment docs.


class without superclass and super() call produces TypeError in IE9,IE10

In IE9, IE10, the following code produces a TypeError because of an undefined 'call'
(  $traceurRuntime.superConstructor($Test).call(this); )
class Test {
  constructor() {
    super();
  }
}

let test=new Test();

This is actually supposed to throw in the new final ES6 semantics.
#1675


which latest version of io.dropwizard.metrics is compatible with java 7

i'm using metrics in my application running on java 7. Need to know the version so that i could add the dependency in my pom.xml

The 3.2.* releases are Java6-compatible. 4.0.* releases will target Java8.

Yes, Metrics 3.2.3 should be compatible with Java 7.


Double clicking a heading makes the preview go to the table of contents

Hello !
If you have a document with a table of contents created with the outline function and you double click the heading in the document, it will make the preview jump to where the heading is in the table of contents instead of showing you where it actually is.
If you create a typst doc with
#outline()
#pagebreak()
= Title
#pagebreak();
hello
and you go to the page with "hello" written in the preview then you double click the heading in the .typ doc it will do it.
I believe it should show you the actual heading instead of the table of contents.
I use vs code version 1.86.0 (up to date) with the version 0.10.7 of the typst preview extension.
The extension is great btw, loving it !

The jump destination is provided by typst compiler and perhaps typst-preview cannot make a more smarter jump over it. You can check webapp, when you click a heading in the document, webapp also takes you to the table of contents.
The root cause is that both the text in the table of contents and the text in the heading are from the same source location: your clicking the heading in the document. When there are multiple jump destinations, the compiler will take you to the first occurrence of the text -- so the text in the table of contents.
We can transfer this issue to typst/typst, like the similar issue, Jump to code does not work within a code block.

Thanks for your answer ! I didn't know it was handled by typst and I will be transfering to the right github, have a great day !


Limiting results after X characters entered

Struggling with one thing, If I wanted to not show the autocomplete options until a user has entered X number of characters how would I do it, I suspect it has to do with filter but I'm having zero luck

Try this:
var horse = horsey(el, filter: function (q, suggestion) {
  if (q.length < min) {
    return false;
  }
  return horse.defaultFilter(q, suggestion);
});

I guess I'm trying to use it incorrectly
horsey(document.querySelector('#s'), {
	suggestions: ['banana', 'apple', 'orange'],
	filter: function (q, suggestion) { if (q.length < 3) { return false; } return horse.defaultFilter(q, suggestion);}
	}
});

Does that not work for you?

Am I using it correctly? Doesn't seem too

You have an extra } in line 4, that may be your problem

This works
<script type="text/javascript" src="https://cdnjs.cloudflare.com/ajax/libs/horsey/2.6.1/horsey.min.js"></script>
<script>
var horse = horsey(document.querySelector('#s'), {
suggestions: ['banana', 'apple', 'orange'],
filter: function (q, suggestion) { if (q.length < 2) {
    console.log(q);
    return false;
    }
return horse.defaultFilter(q, suggestion);
}

});

Cool!

Hi @bevacqua ,
It works, but does not prevents the suggestions method to be executed. Ajax inside there will be executed but keep hidden. Any Ideia in how to prevent suggestions? Duplicated the code inside there does not looks good to me. Than you.


AppRTCDemo link error "missing required architecture i386"

I am using the following:

OS X Yosemite version 10.10.3
Xcode version 6.3 (6D570)
MacBook Pro 2.7 GHz Intel Core i7
Ignoring file /../../libWebRTC-LATEST-Universal-Debug.a, missing required architecture i386 in file /../../libWebRTC-LATEST-Universal-Debug.a (3 slices)
"OBJC_CLASS$_RTCEAGLVideoView", referenced from:
"OBJC_CLASS$_RTCICECandidate", referenced from:
"OBJC_CLASS$_RTCICEServer":, referenced from:
"OBJC_CLASS$_RTCMediaConstraints":, referenced from:
"OBJC_CLASS$_RTCPair", referenced from: ........

Please advise how to resolve the problem.

The simulator builds are currently messed up. webrtc has recently fixed the issues with the x86 builds that mistakenly included x64 symbols. Try the x64 simulator. No promises on timeline but will be fixed soon

@thebehera Is this still an issue? I just pulled the latest code and added both simulator builds back and am not having compile time issues.

Im rebuilding now

Confirmed that the latest build runs on both i386 and x64.

I had tried the x64 simulator and ipad mini and they both working fine. I will tried the latest build later. Thanks

Hi:
How to force adding i386 to libWebRTC-LATEST-Universal-Debug.a via dance?
$ dance (no issues)
...
...
===> Notice i386 is missing
$ file ../*.a
../libWebRTC-LATEST-Universal-Debug.a:   Mach-O universal binary with 3 architectures
../libWebRTC-LATEST-Universal-Debug.a (for architecture armv7):	current ar archive random library
../libWebRTC-LATEST-Universal-Debug.a (for architecture x86_64):	current ar archive random library
../libWebRTC-LATEST-Universal-Debug.a (for architecture arm64):	current ar archive random library
../libWebRTC-LATEST-Universal-Profile.a: Mach-O universal binary with 3 architectures
../libWebRTC-LATEST-Universal-Profile.a (for architecture armv7):	current ar archive random library
../libWebRTC-LATEST-Universal-Profile.a (for architecture x86_64):	current ar archive random library
../libWebRTC-LATEST-Universal-Profile.a (for architecture arm64):	current ar archive random library
===> So Xcode hits a brick wall as follows:
...
...
ld: warning: ignoring file /Users/bdurant/work/att/webrtc-build-scripts/ios/webrtc/libWebRTC-LATEST-Universal-Debug.a, missing required architecture i386 in file /Users/bdurant/work/att/webrtc-build-scripts/ios/webrtc/libWebRTC-LATEST-Universal-Debug.a (3 slices)
Undefined symbols for architecture i386:
"OBJC_CLASS$_RTCEAGLVideoView", referenced from:
objc-class-ref in ARDVideoCallView.o
...
...
What to do?
Thanks!
Bill

@bdurant Have you pulled the latest set of scripts from this repo? This was fixed in the past few days. When you do dance now you will get all 4 architectures: i386, x86_64, armv7 and arm64.


Include signing entity in version metadata

Clients who are asking for metadata prior to download an archive from the registry may still want to display information from the signature which is non-trivial to extract. This includes it if possible.

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci smoke test

@swift-ci smoke test

@swift-ci smoke test

@swift-ci smoke test linux

@swift-ci smoke test windows

@swift-ci smoke test windows

@swift-ci smoke test windows

/Applications/Xcode-beta.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX12.0.sdk/usr/lib/swift/Foundation.swiftmodule/x86_64-apple-macos.swiftinterface:4:8: error: no such module 'Combine'
import Combine
       ^
/private/var/folders/pf/p5g2vb2j0yx1x6vlmxdppc700000gn/T/Miscellaneous_AtMainSupport.60lkRg/Miscellaneous_AtMainSupport/Package.swift:1:8: error: failed to build module 'Foundation'; this SDK is not supported by the compiler (the SDK is built with 'Apple Swift version 5.5 (swiftlang-13<IP_ADDRESS> clang-13<IP_ADDRESS>)', while this compiler is 'Apple Swift version 5.9-dev (LLVM f5c0e1516308d32, Swift 756fe15bdd053a8)'). Please select a toolchain which matches the SDK.
import Foundation
       ^

@swift-ci smoke test windows

@swift-ci smoke test macOS

@swift-ci smoke test windows


[google_maps_flutter_web] bring the version into line with the [google_maps_flutter] version

Hello!
I'd like to raise the [google_maps_flutter] version to 2.0.0 but there is a conflict with the current [google_maps_flutter_web] version:
[flutt] flutter pub get
Running "flutter pub get" in flutt...
Because google_maps_flutter_web >=0.1.1 depends on google_maps_flutter_platform_interface ^1.1.0 and google_maps_flutter 2.0.0 depends on google_maps_flutter_platform_interface ^2.0.0, google_maps_flutter_web >=0.1.1 is incompatible with google_maps_flutter 2.0.0.
And because no versions of google_maps_flutter match >2.0.0 <3.0.0, google_maps_flutter_web >=0.1.1 is incompatible with google_maps_flutter ^2.0.0.
So, because App depends on both google_maps_flutter ^2.0.0 and google_maps_flutter_web ^0.1.1, version solving failed.

pub get failed (1; So, because App depends on both google_maps_flutter ^2.0.0 and google_maps_flutter_web ^0.1.1, version solving failed.)
exit code 1

Is it possible to fix it somehow?
/cc @ditman

Duplicate of https://github.com/flutter/flutter/issues/75236


Documentation

A clear reference helps so please. The examples are alright but they don't give me a picture of everything I can do.

I agree, docs would be very helpful. I hope to write them at some point, but not really sure when I'll have the time/dedication to.

I could write documentation assuming everything that you can do is shown in the examples?

I would prefer to write the docs myself as I'm not sure how complete the examples are, especially for some little nuances in the language.

Okay! I could help with formatting though.

I just wrote some basic documentation if you want to take a look. It lives in the docs folder.

Okay the documentation is well written but there is just one type, this is kinda a nit but immutible should be immutable.

Thank you for the heads up! I wrote the docs in my code editor without spellcheck, and we all know how that turns out. :P


V4

Coverage increased (+19.3%) to 95.708% when pulling 232dbfd247ef449f5dcb316bd52a186e31762825 on v4 into 113788e4de039d9e40c42302648272a56a6d2541 on master.

Coverage increased (+19.1%) to 95.494% when pulling 232dbfd247ef449f5dcb316bd52a186e31762825 on v4 into 113788e4de039d9e40c42302648272a56a6d2541 on master.


Upgrading/Editing a LB scale does not reflect change for inactive LBs.

Rancher Version:
v1.2.0-pre4-rc14
Environment Type: (Cattle/Kubernetes/Swarm/Mesos)
Cattle
Steps to Reproduce:

Create a LB with scale 2. Let it become active and then stop it so it's inactive
Upgrade/Edit in the UI to a new scale of 1.

Results:
UI still shows 2 containers for the inactive service.
Expected:
UI to show that the LB will be 1 container instead of 2.

I doubt this applies in 2.0


budder, duckpond

This pull request was generated by the 'mq' tool
[test]
flake rate: 0.4
logical conflict every: 100
sleep for: 300s
[pullrequest]
requests per hour: 30
deps=[b,d]

/trunk merge


Sort the tags in the documentation version selector using semver

We also probably need to symlink 2.1.0 to latest and tell google/other robots to only index latest. The top result for my etcd 2.X search was 0.4.6.

This is an obvious one but unfortunately this is quite hard in Jekyll which is why it hasn't been done yet :/
Instead of blocking an index, I'd rather figure out how to rank the docs via the sitemap that is submitted to google. Just need to find some time to do that...

This is live.


2.37 failed to build

(let me know if there is anything else you need)
rbucker@puf:~/hg/metaventures/gwtwo$ perl -v
This is perl 5, version 12, subversion 2 (v5.12.2 (*)) built for i386-openbsd
(with 6 registered patches, see perl -V for more detail)
on an OpenNSD 5.0 install (VMware)
The errors are:
PERL_DL_NONLAZY=1 /usr/bin/perl "-MExtUtils::Command::MM" "-e" "test_harness(0, 'blib/lib', 'blib/arch')" t/.t t//*.t
t/mojo/app.t ............................... ok
. . . . . . snip . . . . .
t/mojo/delay.t ............................. ok
t/mojo/dom.t ...............................

Test Summary Report
t/mojo/dom.t                             (Wstat: 139 Tests: 230 Failed: 0)
Non-zero wait status: 139
Parse errors: Bad plan.  You planned 683 tests but ran 230.
Files=77, Tests=7182, 77 wallclock secs ( 0.46 usr  3.48 sys + 25.07 cusr 16.35 csys = 45.36 CPU)
Result: FAIL
Failed 1/77 test programs. 0/7182 subtests failed.
*** Error code 255
Stop in /home/rbucker/.cpanm/work/1323574147.9364/Mojolicious-2.37 (line 1626 of Makefile).
-> FAIL Installing Mojolicious failed. See /home/rbucker/.cpanm/build.log for details.

We appreciate the bug report, but I'm not quite sure what to do about it.  We don't have any OpenBSD installs, so more information is needed to track down the bug.
At this point, I'm more inclined to trust the cpan testers report, which indicates green on OpenBSD with all 5.12 perl versions on Mojolicious 2.37:
http://static.cpantesters.org/distro/M/Mojolicious.html
If you still believe there's a Mojolicious specific-issue, please look into dom.t and tell us exactly which tests are failing.
Thanks!

I'm upgrading all of the deps as I write this email.  Would it be better to
write a new issue that points to the install program needing to update the
deps or at least halt the build when the deps are not present? There is a
huge issue with so many deps when you expect customers to deploy in
production. This could be a very dangerous upgrade path.
*
Richard Bucker
*
*
*
*
*
On Sun, Dec 11, 2011 at 1:44 PM, Glen Hinkle <
<EMAIL_ADDRESS>
wrote:

We appreciate the bug report, but I'm not quite sure what to do about it.
We don't have any OpenBSD installs, so more information is needed to track
down the bug.
At this point, I'm more inclined to trust the cpan testers report, which
indicates green on OpenBSD with all 5.12 perl versions on Mojolicious 2.37:
http://static.cpantesters.org/distro/M/Mojolicious.html
If you still believe there's a Mojolicious specific-issue, please look
into dom.t and tell us exactly which tests are failing.
Thanks!

Reply to this email directly or view it on GitHub:
https://github.com/kraih/mojo/issues/238#issuecomment-3098841


commonLabels not applied on releases

Operating system
Ubuntu 22.04.3 LTS
Helmfile Version
0.158.1
Helm Version
3.13.2
Bug description
After running helmfile apply on the helmfile below, running helm list -l helmfile=test should list the installed release as if it was installed with helm install wiremock deliveryhero/wiremock -l helmfile=test, but this is not the case.
Example helmfile.yaml
commonLabels:
  helmfile: test

repositories:
  - name: deliveryhero
    url: https://charts.deliveryhero.io/

releases:
  - name: wiremock
    chart: deliveryhero/wiremock

Error message you've seen (if any)
N/A
Steps to reproduce
See Bug description and exemple helmfile.yaml
Working Helmfile Version
I havn't tried this feature prior.
Relevant discussion
No response

@mgalliou maybe you want to this: https://github.com/helmfile/helmfile/pull/1046

@yxxhero Indeed that should do it, thanks !


Allow a CompoundIdentification to have list of compounds

Currently CompoundIdentification can have compound be none, or valid metatlas compound.  To handle ambiguity, users will want to specify a list of potential compounds.

How about another field called potential_compounds?

compound = [] or None, means you have a peak, but no idea what it is
compound = [compound_id] means you believe it is exactly that compound
compound = [compound_id, compound_id, compound_id, etc] means you can't
distinguish among the listed compounds
On Sat, Oct 24, 2015 at 1:52 PM, Steven Silvester<EMAIL_ADDRESS>wrote:

How about another field called potential_compounds?
—
Reply to this email directly or view it on GitHub
https://github.com/metabolite-atlas/metatlas/issues/48#issuecomment-150849832
.

Fixed in 606510a, and updated in database.


fix(request-transformer) an empty JSON array after being decoded, can…

…'t be properly re-encoded as an array. Instead, it becomes an empty json object.
This is sister PR to PR:
#4901
It contains a similar fix, but now to request-transformer (this plugin).

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

done


Design model

Goal
The model is the foundation of AppArray.
Usages
It will be used to:

capture the concepts
build visualisation layer
as an exchange mechanism, either between clients and servers or between stored and in-memory format

Constraints

The model must be easy to serialize and to represent in a host language
The model should allow to represent a graph of dependencies / reverse dependencies
The model should be extensible
The model should "work" with Cloud-Native applications and platforms (Cloud Foundry,  k8s)

Open Questions

Language should not be enforced: Typescript and F# may provide interesting building blocks.
Seaparation between model and instantiation for e.g. an environment looks similar to class / object separation. Is this similarity worth exploring?

Success Criteria

The model must be expressive enough to use on two existing systems

Current model has a separation between pure, abstract model, and instantiated, environment dependant model.
This will have to be proven valid with a real world application.
Style and views should probable have separate model.


[BUG] @Listeners doesn't work for interfaces

TestNG Version - 7.3.0
@Listeners(IntellijIdeaTestNgPluginListener.class)
public interface IITest {
    void setTestCaseId();
}

Expected behavior
The listener IntellijIdeaTestNgPluginListener works (enable) for all IITest interface implementations.
Actual behavior
Lister does not work.
Is the issue reproductible on runner?

[X] Shell
[X] Maven
[ ] Gradle
[ ] Ant
[ ] Eclipse
[X] IntelliJ
[ ] NetBeans

Could you please show all you test code which can be reproduce your problem, thanks.

@kobebryant2007 I will gladly attach again! With explanations of course...
@Listeners(IntellijIdeaTestNgPluginListener.class)
public interface IITest {
    void setTestCaseId();
}

This code contains the interface IITest from which all test classes inherit
The IITest interface contains the @Listeners annotation
The @Listeners annotation contains IntellijIdeaTestNgPluginListener.class
The IntellijIdeaTestNgPluginListener.class implemented IInvokedMethodListener and contains constructor with:

public final class IntellijIdeaTestNgPluginListener implements IInvokedMethodListener {
    public IntellijIdeaTestNgPluginListener() {
        System.out.println("Stop mocking and turn on your brain!!!");
        System.exit(1);
    }
}

When running tests of classes that inherit from the IITest interface, the listener is not created and the message from the constructor is not displayed.

@shaburov Thanks for your explanation.
I tried your test case and I found that the listeners only works when you add them in implementation classes. I think the @Listeners annotation is not @Inherited, so the annotation can not be inherited in test implementation class.
@krmahadevan should we support this feature?

@krmahadevan @kobebryant2007 This is a request for fix.
If you use annotations, then you are obliged to find the annotations for both classes and interfaces. And I doubt this is just a feature. This is a clear implementation bug.

@shaburov

If you use annotations, then you are obliged to find the annotations for both classes and interfaces. And I doubt this is just a feature. This is a clear implementation bug.

I don't remember seeing any such statement anywhere within the TestNG documentation that states that @Listeners should/would work when annotated on listeners. There are some gaps in how this should work.
A user may simply add the annotation to an interface which never gets implemented by any class. in that situation there's nothing that TestNG can do in terms of figuring out if this listener should be wired in.
So IMO its not a bug, but can be treated as a feature request.
Since the following is not clear, I would want to get inputs from both @cbeust and @juherr

Is this a realistic use-case that a regular user would do ? Listeners are at the end of the day "global" in nature and are never scoped at "class" level. So why should a user have to resort to annotating an interface with this annotation ? The listener can always be built such that it recognizes an object type (if your idea of having an interface is to define a common type for a test class) and then inspect the test class instance type to find out if the listener is to be applied or not.
If this is a realistic use-case then I would like to understand what would/should be the expected behavior, when the annotation is added on top of a listener which never gets applied.

In a nutshell, I would need to have a bit more clarity on the expectations here.
In the meantime, if you have an implementation or an approach in mind, please feel free to submit a PR and we can help you get that vetted out as well.
@cbeust @juherr - WDYT ?

My feeling is that most of the frameworks have their annotation @Inherited and it is more natural for me too.
I don't see any reason to not implement it and I think it will just add some other listeners.

OK, I will do some investigation and maybe raise a PR later.
One thing I should reminder again is that @Inherited only works when annotate a class, and the interface doesn't support inherited.
so we should use other way.

Hi @krmahadevan
Based on your comments, could you please specify the expect behavior of 5 and 6? Listenners in different class or interface should be merged or override?
Please also notice the comments in findAnnotationInSuperClasses , Listenners annotation would be handled specially for ignore aggregation.

@kobebryant2007

Based on your comments, could you please specify the expect behavior of 5 and 6? Listenners in different class or interface should be merged or override?

5. Can you please add a test that checks the behaviour when there is inheritance of interfaces involved (multi level and multiple inheritance both) and every interface attempts at wiring in the same listener.

TestNG already gives a guarantee that duplicate instances of the same listener will not be wired in (They may get instantiated but they wont get hooked in). So this needs to be met for this case.
6. Can you please add a test that checks the behaviour when there is inheritance of interfaces involved (multi level and multiple inheritance both) and every interface attempts at wiring unique listeners.

Here the expectation is that all listeners get wired in.

Please also notice the comments in findAnnotationInSuperClasses , Listenners annotation would be handled specially for ignore aggregation.

The comment here shouldn't have an impact here, because all we are doing is, we are aggregating listeners while walking up the inheritance hierarchy. We just need to ensure that there are no duplicate instances of the same listener. As long as that expectation is met, I guess we should be fine.

This issue stands closed via https://github.com/cbeust/testng/pull/2398


PointLight2D don't animate AnimatedTexture

Godot version
Godot Engine v4.0.beta3.official.01ae26d31
System information
Xubuntu 22.04, Vulkan API 1.2.0 - Using Vulkan Device #0: NVIDIA - NVIDIA GeForce GTX 1060 6GB
Issue description
If I use a AnimatedTexture resource for a PointLight2D it only shows the first frame of the animation.
The same AnimatedTexture on a Sprite2D works.
Steps to reproduce

create a AnimatedTexture
set this AnimatedTexture as texture of a PointLight2D

Minimal reproduction project
No response

Related to https://github.com/godotengine/godot/issues/54506.

I'm not sure supporting animated textures in PointLight2Ds is a good idea. In 4.0 2D lighting is done in a single pass which means that all light textures have to be copied to an atlas. Supporting animated textures would require either:

updating the atlas every single frame by blitting the animated texture to it or,
copying every frame of the animated texture into the atlas and then animating the rect of the PointLight2D.

Both options will be harmful for performance. I think number 2 may not be terrible, but I guess it depends on how big the AnimatedTexture is and how many frames it contains.

Closing in favor of:

https://github.com/godotengine/godot-proposals/issues/9855


See if and how Swiftdoc comments can help in autocompletion

Just tried it, it's quite nice to document stuff, really helps in autocompletion and inline help. Good thing to do wherever we can.

First version is merged, we can now expand where handy. :)


Different training and testing conditionals in Seq2SeqAE

In the LSTM autoencder that you have the training and testing inputs are different. That is during training for the decoder you try to predict p(x[t+1] | x[<=t]) but in testing you rather condition it on the output of previous timesteps i.e. p(x[t+1] | y[<=t]).
This seems a bit off to me. Is it expected to be like this? Is there some reference somewhere for doing it this way?

Ahh I see why is it done this way!!


Version 2.0.1: Fatal Error! xsl:import elements must come first

After installing a XHTML extension plugin to the stock DITA-OT 2.0.1, I got the following error:
/opt/DITA-OT2.0.1/plugins/org.dita.xhtml/xsl/map2html-coverImpl.xsl:82:103: Fatal Error! xsl:import elements must come first
After looking around and bisecting the map2html-coverImpl(_template).xsl, it turns out that
the extension id for " dita.xsl.html.cover" is at the end of the template.
Thus, when installing a plugin that creates an extension to dita.xsl.html.cover, it adds a <xsl:import /> line after the named templates.

Closed by Bug #1937


[YBUG.IO] Greetings!

Greetings!
I just clicked on the info that appeared to update the app and install it and now I only get the message: Sovryn is loading ... and can't connect to Metamask, can't stake, trade, ...
I am using Chrome v.90.
Please can you help me resolve this problem so that I can continue to stake and trade.
Source url: https://live.sovryn.app/
Reported by<EMAIL_ADDRESS>Reported at: 30 Apr at 10:20 UTC
Location: HR, City of Zagreb, Sesvete
Browser: Chrome 90.0.4430.93
OS: Windows 10.0
Screen: 1920x1080
Viewport: 1920x947
Screenshot:

For more details please visit report page on Ybug:
https://ybug.io/dashboard/reports/detail/jrsyqhsg8mrtyc7b3q0q

Issue should be resolved now


4chan "Options" field support

Hey,
Could you please add support for the "Options" field? It's useful now.

And what does this field do now?

Typing "fortune" in /s4s/ will add a fun little footer to your post, typing "since4pass" on any board will add a footer showing how long you've had a 4chan pass for. Don't know of others.

I can implement options field as email field, ok?

Yep. It's the same thing, just a different name.

Added in new update.


Translation: "Base artifact crafting success rate" property

The proposed wiki translation: Divine Relic Crafting Base Rate sounds good enough

Currently, if you fail that related roll for the property, the end result is a Divine Relic tier Item. Does the following sentence make sense?

Considering the possible usage, calling it a Sacred Relic Crafting Base Rate would be more accurate, but it would overlap with Shendao Artifacts/Sacred Relics. To avoid overlapping, calling it T12 Artifact crafting success rate would be another possibility.


Issue with BSON decoder and ObjectId

I get an error when the BSON decoder tries to decode the machine identifier bytes of an ObjectId.
** (MatchError) no match of right hand side value: <<111, 230, 80, 74, 154, 74, 223, 8>>
stacktrace:
  (mongodb) lib/bson/decoder.ex:98: BSON.Decoder.document/1
  (mongodb) lib/bson/decoder.ex:6: BSON.Decoder.decode/1
  test/controllers/user_controller_test.exs:27

The original binary was <<86, 206, 224, 104, 111, 230, 80, 74, 154, 74, 223, 8>>

I changed to using mongodb directly from github, and then I use BSON.ObjectId.encode! and BSON.ObjectId.decode! instead of the functions found in the BSON module, then it works.

The functions in the BSON module encode to and from the BSON format, you should usually not call these functions yourself because the driver does it for you. BSON.ObjectId converts an ObjectId struct to and from its hex representation.

Is there a better way of doing that, or is that just how it is at the moment?

What do you mean?

Closing this due to no response.


Seeding DB from JSON files - add completion handler

Hi,
It could be useful to call some code just after your DB was seeded with data form JSON files. Then seeding function would be like this:
// SynchronizationManager+SeedDB.swift

public func seedDBFromJSONFiles(in directory: String,
                                in bundle: Bundle,
                                completion: @escaping(() -> ()))

What do you think?

Hi @loudmouth
I achieved the same thing using OperationQueue:
let seedOperation = BlockOperation {
    try? syncManager.seedDBFromJSONFiles(in: dirName, in: bundle)
}

let otherOperation = BlockOperation {
    self.otherOperation()
}

otherOperation.addDependency(seedOperation)

queue.addOperation(seedOperation)
queue.addOperation(otherOperation)

So I'm closing the issue.


@no-redirect directive does not seem to be honoured

Doing the # @no-redirect sample call from the guide
# @no-redirect

GET https://httpbin.org/absolute-redirect/2

For me displays only the final exchange:
GET http://httpbin.org/get
accept-encoding: gzip, deflate, br
accept: */*
user-agent: httpyac

HTTP/1.1 200 OK
access-control-allow-credentials: true
access-control-allow-origin: *
connection: close
content-length: 313
content-type: application/json
date: Wed, 01 Feb 2023 09:34:08 GMT
server: gunicorn/19.9.0

[...]

Doing CURL shows it actually goes through two 302:
curl --location --head --silent https://httpbin.org/absolute-redirect/2 | grep --extended-regexp --ignore-case "^(HTTP|Location|$)"
HTTP/1.1 200 Connection established

HTTP/1.1 302 FOUND
Location: http://httpbin.org/absolute-redirect/1

HTTP/1.1 302 FOUND
Location: http://httpbin.org/get

HTTP/1.1 200 OK

So I'd expect to see not the final exchange with
HTTP/1.1 200 OK
content-length: 313

but the first response with
HTTP/1.1 302 FOUND
Location: http://httpbin.org/absolute-redirect/1

in VSCode HTTPYac.
Am I correct or is there something else to set to see the first "redirecting" response?

A bug has crept in during refactoring. I have fixed it and added a test for it. thx.


bugfix/483: better error message when validating mismatch index

Fixes #483

hey @jekwatt looks like there are a few CI errors:

code style: https://github.com/pandera-dev/pandera/pull/500/checks?check_run_id=2656701766. Following the pre-commit stuff in the contributor guide helps make sure all the linting/code formatting stuff passes locally
unit test error: https://github.com/pandera-dev/pandera/pull/500/checks?check_run_id=2656703106. looks like this is only happening with pandas 0.25.3, not sure what's going on here, will take a closer look

@cosmicBboy Code style passed.  Let me know if anything I need to on my end.


Be able to change skipLibCheck flag in command line

Suggestion
Add a new --skipLibCheck [bool] flag to tsc when using --build. This flag would override the skipLibCheck value in all tsconfig.json files for all projects checked in the build.
🔍 Search Terms
performance
skipLibCheck
✅ Viability Checklist

My suggestion meets these guidelines:

[x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
[x] This feature would agree with the rest of TypeScript's Design Goals.

⭐ Suggestion
We are interested in leveraging skipLibCheck when using the typescript checker in order to avoid expensive checks on large .d.ts files from external libraries that we include as node modules. That said, we recognize that there is some danger in skipLibCheck. For example, a third party module may have been checked with a different version of tsc, or with different levels of strictness. We would feel more comfortable with a workflow that allows us to check libs when we import a new version of a node module, upgrade the tsc compiler, or change compiler flags.
In order to do this, we request that tsc support a --skipLibCheck (bool) flag when using --build. This flag would override the skipLibCheck compiler option in the tsconfig.json file (and the tsconfig.json file for all project references).
📃 Motivating Example
As a motivating example<EMAIL_ADDRESS>a dependency we take in all of our projects, takes 1.25 seconds to check on my dev machine. Without skipLibCheck, the overhead of splitting our monorepo into smaller projects is too high to justify the work.
💻 Use Cases
We generally want to set "skipLibCheck": true in our tsconfig.json files, but we want a workflow that allows us to check libs when we import a new version of a node module, upgrade the tsc compiler, or change compiler flags.

In the meantime you could create a second tsconfig file that extends your existing one, and just changes the skipLibCheck value. Then when executing the compiler point to your second config file.

We have a lot of tsconfig files, kind of annoying to keep them all in sync.
On Thu, Sep 2, 2021 at 2:57 AM Martin Johns @.***>
wrote:

In the meantime you could create a second tsconfig file that extends your
existing one, and just changes the skipLibCheck value. Then when
executing the compiler point to your second config file.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/TypeScript/issues/45691#issuecomment-911342734,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AU2KXHHBJK673GCWGBFMLLLT74U7PANCNFSM5DH7AKBA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

--
[image: photo]
Marc Celani (he / him)
Software Engineer
—
650.997.5975
airtable.com
https://www.linkedin.com/company/airtable/mycompany/
https://twitter.com/airtable https://www.youtube.com/c/AirtableApp
https://www.instagram.com/airtable/?hl=en
https://www.facebook.com/airtableapp


Install gke-gcloud-auth-plugin to manage authentication between kubectl and GKE

Changes:

Install gke-gcloud-auth-plugin to manage authentication between kubectl and GKE
Note: the gcp auth plugin is deprecated in v1.22+, unavailable in v1.25+

References

Important changes to Kubectl authentication are coming in GKE v1.25

LGTM


The pretrain.csv file is empty, failed to download.

Hi, we are facing some downloading issues right now. We will fix it ASAP and notify you. 😊


_PrefetchDataFetcher ignores prefetch_factor

Bug description
No matter what prefetch_factor is set for the DataLoader in a LightningDataModule wrapper, when the _PrefetchDataFetcher is initialized, the value is always reset to 1.
What version are you seeing the problem on?
v2.0
cc @justusschock @awaelchli

@botcs The prefetching done in the trainer is independent of the prefetching in the DataLoader. The trainer prefetches 1 batch just to know in advance whether the data loader is exhausted or not, that's all. But of course, you can set any value for DataLoader(prefetch_factor=N) and this will be handled by PyTorch. Let me know if you have any questions.

A fair! thanks for the explanation :)


Use ShortName for Title

When the grid figures out the title for a column, it should check for ShortName via GetShortName and fallback to GetName if it's not specified.

I agree that short name could be used. But shouldn't it be the other way around? Use GetName and then fallback to GetShortName? I would want to use GetName first, because people are more familiar with GetName notation from all the examples ASP.NET is providing. Unless there is deeper meaning to that attribute?

Oh, I think I get it, using short name would be better for the grid column, so that they don't span across half the table on longer names right?

Yeah. I think ShortName is specifically for grid headers wheres Name is for everything else.

The short name is used as the grid column label for a UI element that is bound to the property that is annotated with this attribute.


feat: add CTimeInput

@johannes-huther I reviewed the storybook screenshots. The text inside the input field seems to be not aligned correctly. However this seems to be a general problem, when reviewing the other screenshots of other inputs, there is the same problem. When using the component (even in storybook) it's alright.

@johannes-huther I reviewed the storybook screenshots. The text inside the input field seems to be not aligned correctly. However this seems to be a general problem, when reviewing the other screenshots of other inputs, there is the same problem. When using the component (even in storybook) it's alright.

Interesting, I'll take a look how to fix that, when we have more time :)


Update agent.rst

Updated infos how to mount the GuestAdditions.iso with cmd-line online virtualbox for installation of the Guest Additions in order to be able to use transient shares and other features.

We actively pursue people not to install the VirtualBox Additions in the VM, as malware is quite good at detecting it (and doing an early exit). So this documentation is probably not so useful :-)
Thanks for the contribution but afraid I'm going to have to close it.

Hi
I 100% understand the problem, yet I had to install them to be able to
update the agent.py file in my very specific setup, but you are right, I
guess I should uninstall them again after updating the agent.py file :)
Br
On 19/10/15 01:38, Jurriaan Bremer wrote:

We actively pursue people not to install the VirtualBox Additions in the VM, as malware is quite good at detecting it (and doing an early exit). So this documentation is probably not so useful :-)
Thanks for the contribution but afraid I'm going to have to close it.

Reply to this email directly or view it on GitHub:
https://github.com/cuckoobox/cuckoo/pull/662#issuecomment-149064699

I have the feeling when you're uninstalling you're already too late. If your VM has (temporary) access to the internet, you can just download it from github.com :-)
Jurriaan


Handle empty policies when traversing the policy tree in discovery policy analysis

Discovery policy analysis engine assumed that the inner policy passed to it is of the correct type and cannot be empty.
Added a nil check and a test.
Change-Id: Ia6eaf0211d8d9f4b7f2881547934dc32c725a458
Signed-off-by: Yacov Manevich<EMAIL_ADDRESS>

https://github.com/Mergifyio backport release-2.4

@Mergifyio backport release-2.4

@Mergifyio backport release-2.2


Create new helper function for "archive" storage class.

We have a number of helper functions for storage class:
https://github.com/googleapis/google-cloud-cpp/blob/e9ccd39d9f12066f43d77f621fdb560a6a0a178b/google/cloud/storage/storage_class.h#L26-L54
We need to add another one of these for ARCHIVE.

We also need to fix the helpers to create lifecycle policies:
https://github.com/googleapis/google-cloud-cpp/blob/1cc95570aaf877f9e86f64f50b7de246d5dad604/google/cloud/storage/lifecycle_rule.cc#L49

Please assign, @tritone as the reviewer.


Create site extension

Would allow for easier deployment (see https://github.com/ohadschn/letsencrypt-webapp-renewer/issues/9).
Basically the extension will simply install the WebJob.

I gave it some thought and I don't think I'll implement this.
Uploading a zip file isn't really harder than installing an extension from the SCM site.
Automated deployment should be easy enough too, whether via Azure Scheduler or CRON (see https://github.com/ohadschn/letsencrypt-webapp-renewer/issues/9).
Let me know if you disagree in the comments.


Generate Logging external service types

Updates https://github.com/GoogleCloudPlatform/google-cloud-python/issues/2572
Updates https://github.com/googleapis/googleapis/issues/187

@lukesneeringer This is an example of our Python gRPC generated package (the lowest level in the client library stack). This contains extra types supported by our Logging service, which we just released to Beta a couple weeks ago.


better documentation on what qr.fqm_mopac if used for

A clear and concise description of what the purpose of this script is for.
It has mopac in the name, but seems to use orca and not mopac.
Describe the solution you'd like
A clear and concise name and docstring would be helpful.
Lots of legacy blocks of code are left in, need to clean up when we get the chance.

To be deprecated.


test PR with Github API

Please pull this in!

Docs Build status updates of commit ad8de1d:
:warning: Validation status: warnings

File
Status
Preview URL
Details
Dependent Files

sdp-test/page-model-documents/Tutorial.yml
:warning:Warning

Details

sdp-test/page-model-documents/Tutorial.yml

[Warning] Unable to find uid "challenge-uid".

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 19adbf6:
:warning: Validation status: warnings

File
Status
Preview URL
Details
Dependent Files

sdp-test/page-model-documents/Tutorial.yml
:warning:Warning

Details

sdp-test/page-model-documents/Tutorial.yml

[Warning] Unable to find uid "challenge-uid".

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 18d3c91:
:warning: Validation status: warnings

File
Status
Preview URL
Details
Dependent Files

sdp-test/page-model-documents/Tutorial.yml
:warning:Warning

Details

sdp-test/page-model-documents/Tutorial.yml

[Warning] Unable to find uid "challenge-uid".

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 27d2bd2:
:warning: Validation status: warnings

File
Status
Preview URL
Details
Dependent Files

sdp-test/page-model-documents/Tutorial.yml
:warning:Warning

Details

sdp-test/page-model-documents/Tutorial.yml

[Warning] Unable to find uid "challenge-uid".

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 895ee53:
:warning: Validation status: warnings

File
Status
Preview URL
Details
Dependent Files

sdp-test/page-model-documents/Tutorial.yml
:warning:Warning

Details

sdp-test/page-model-documents/Tutorial.yml

[Warning] Unable to find uid "challenge-uid".

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Can't render anything on top of UnityWidget for iOS (in the example too)

To reproduce this issue you can use example from this repo or this one https://github.com/juicycleff/flutter-unity-arkit-demo and the latest version of iOS
For example in this repo I CAN interact with the slider but I can't see it
iOS:

Android:

Here is an example from https://github.com/juicycleff/flutter-unity-arkit-demo repo for iOS that uses the same widget:

Any thoughts?

I am experiencing the same issue. I have months of development on my app where this was working before. This is a Major Issue! Please help!!

Solved!...ish. After trying to figure out what they heck I had changed to make my Flutter UI elements disappear, I realized that I have updated Flutter without thinking about it. This looks to be and issue using Flutter version 1.17.1. After downgrading back to 1.16.3 everything worked again!

Downgrading worked for me too.Not too happy about this. I am at the beginning of my  project and i would feel more comfortable to work with latest stable version :(

@cobainmo @mtwillet you can use version 1.17.1 + this fix of the lib https://github.com/snowballdigital/flutter-unity-view-widget/issues/138. Works well in my case but I had to fork the lib and make described in the issue changes

@ilyaviache That's great. Will try that. Thank you.


Generate mock representations of proxies, products, and traffic

This restructures the generate-animal-apis helper tool into a CLI that has three subcommands:

enrolled generates YAML files containing the "enrolled" API definitions
traffic generates YAML files containing mock "traffic" signals
runtime generates YAML files containing mock "proxy" and "product" signals

Output from the two new subcommands is checked into the traffic and runtime directories.
The goal of this mock YAML is to help us decide on an intermediate point in the analysis process. Importers like registry-connect can be adjusted to generate the formats we settle on and downstream tools like viewers and classifiers can be written to expect these formats.
I'd like to treat this PR as a draft and not necessarily defining our final format, but all review suggestions are welcome.

@theganyo Thanks for all the discussion! I'll merge this to use for downstream tooling experiments, but won't mind if we revisit any of this if needed. I also made a couple of other small changes: removing the "updated" timestamps because they destabilized the output and didn't seem to be helpful and removing the confusing  animal attribute-related labeling.


Namespace discrepancy in kube-lego migration guide

In the kube-lego migration guide, step 2 points you to the cert-manager installation instructions, which start with creating a cert-manager namespace. When you go back to the migration guide, steps 3-5 assume you have installed cert-manager into kube-system, e.g.:

Once done, we need to create this new resource in the kube-system namespace. By default, cert-manager stores supporting resources for ClusterIssuers in the namespace that it is running in, and we used kube-system when deploying cert-manager above. You should change this if you have deployed cert-manager into a different namespace.

It would be good to have both the installation guide and migration guide use the same namespace, or to change the namespace in the migration guide to a where-you-installed-cert-manager variable.

I think this guide got outdated, oops.... a PR to change kube-system to cert-manager is more than welcome!

PR by @meyskens fixed it.
/close


at_template - Refactoring and template modifications

Describe the solution you'd like

[x] Update to mason:0.1.0-dev.7

done in #89

[x] Set default vars at the template level

done in #90

[x] Remove Var model verify function and use mason default values instead

done in #90
[x] Update the vars to reflect this change

done in #90

[x] Set default kotlin version to 1.5.32

done in #85

[x] Allow additional files to be gitignored programmatically

done in #92

[x] Rename at_template before first major release to something more appropriate?

at_app_create seems like a more suitable name for it
done in #93

[x] #91
[x] Migrate logic to at_app_create

done in #97
[x] AtAppTemplate model
[x] create command

No longer necessary:

[x] Merge the individual templates into a single template

Not sure this is necessary or possible (after investigating further)

[x] Dynamic kotlin version support - This is minor, since we can now easily update kotlin version in 1 place.
[x] Remove kotlin version overrides from each template.yaml

closed via #97


Handling Errors

This might already be possible with one of webpacks many plugins. But it seems like errors (right now testing with glslify, but presumably brfs would have same problem) are going unhandled and causing the webpack-dev-server to crash.
The following will fix the problem, but I'm sure there's a better way.
        multipipe.apply(this, transforms)
          .once('error', function (err) { // <-- handle errors in the pipe
            console.error(err.message)
          })

The above still emits an error to webpack since the bl module is calling done with an error in the first parameter.

Ah, my bad! Fixed :)


Remove some perl 5.14 use from rsaz-avx512.pl

The non-destructive substitution syntax (s///r), was introduced in perl
5.14. We need to support 5.10 and above.
Fixes #15378

This pull request is ready to merge

Apparently perl 5.14.0 was released on May 14, 2011, which is now over a decade ago.  Not that we need to for this specific case, of course, but we might consider sliding our minimum version dependency forward if we like the features that would bring in.

Solaris is rather weird system in the sense of open source software

I personally wouldn't mind raising the minimum perl version requirement

Note that SLES11 used Perl 5.10.0 and RHEL 6 uses Perl 5.10.1.
Not saying you must support those for recent OpenSSL, just giving some more data points.

Merged.


.IIV::-webkit-media-controls-play-button doesn't successfully hide the button

Hi, I tried .IIV::-webkit-media-controls-play-button to hide the play button, but failed;
Then I changed .IIV into video, it finally worked.
I'm a junior developer, I'm not quite sure what .IIV is, but I'm here just explaining video::-webkit-media-controls-play-button works.
Thanks for offering such a handy tool!

The IIV class is applied when iphone-inline-video (in short "IIV") is able to act on the element (generally on the iPhone on iOS 8-9)
I assume that you are seeing that on iOS 10, where IIV is not enabled and therefore the button is not hidden.
I hide the buttons because they cannot be fixed with IIV (i.e. They go fullscreen) but I suppose that it makes sense to document this so people know to make that change strictly for stylistic reasons.

Comment and documentation updated


Adding the file depthlevels.in.ARCTIC to the branch issue22_tp5_opera…

Added the depth file in a new branch which should merge with develop.

Hi, can you replace the old 12-levels depthlevels.in under MSCprogs/Input with the 40-levels so that it is easier to find?


Home page have to many dispatches, components are not at there proper positions

Reproduction steps
While visiting the website, user can see that at home page, many of the components are not present at there proper positions.
Expected behavior
They should be present perfectly, so that Website looks Beautiful and UI fits the content.
Actual Behavior

As you can see here, Components are not separated properly
therefore, at some places, margin is necessary, somewhere padding, somewhere content need some styling.
@SauravMukherjee44 I'm a GSSOC'22 participant, please assign this task to me with GSSOC'22 label.
I'll create a PR on this.
👀 Have you spent some time to check if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

@MadhuSaini22 You can start working on it.


Simplify the ready state of active clients

Details
This was a simplification I noticed in the code
Fixed Issues
None
Tests & QA

Open a chat
Go offline
Add a new comment
Open the dev console > network tab and clear out all requests
Go back online
Verify that a network request happens for adding the new comment

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by @techievivek in version: 1.1.64-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by @AndrewGable in version: 1.1.64-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


added bird animation

正直もう限界だ...
お望み通りびっくりする程構ってやる
~ presented by Miyuki Shirogane ~

I have a pen, I have a pen.
Wow, I have two pens!


Edge Postprocessing

LocalPostTrigger

Edge strength near 0, find peak.

Disable subsampling:

Sample small data near peak, Hann window
Normalize it
Return argmax(Corr(small, step) - parabola)

a *= stickiness
slope prop= 1/period
Sliding step sum is like integral: ig(slope) = (slope/2)x²
so parabola's a *= 1/period
and parabola's height = a*period^2 = period
Which matches summation of a signal. (todo rewrite this)

!CorrTC

period_width: float = .25
period_bonus: float ≥ 1 = 1.1

use a hann window i think

after_nsamp: 256
after: !LocalPTC

Correlation trigger: Discard edges entirely, rely on correlation only.
Postprocess results using a correlation/edge detector with diameter.

i swear i had some notes on this. the optimization cost is either parabolic, or bounded to 0.25 periods.
either store period in a PerFrameCache, or pass period into postprocessing trigger.

Problem: The stronger the data is, the larger correlation it has with the edge trigger. The parabolic cost function is more effective at low signal strengths.

zmm-final: Seems to jump around more than a well-optimized earlier edge_strength trigger.

it should give up if only exposed to negative correlation. maybe throw abs() in there

Problem? CT.update_buffer() is passed its own preference, not the PP'd preference. In the absence of periodic score boosting, postprocessing has zero feedback to CT.
 IDK


Fix the “hanging” issue

When the API is hit from a browser or a dashboard, regardless of whether it's containerized or not, it often "hangs" until the browser is refreshed. After repeated requests the service is left in a state with many (500+) hung processes and requires rebooting. Logs show that the request never makes it into the function of the requested endpoint.

@jdr0887 quick ping - can you fix and update the ticket?

I suspect this issue is due to the copious "assert" statements in the code.  For example, when a requested_object_id is not given, then the assert statement in the following throws an Exception and is not handled properly, causing the thread to stop/hang.
https://github.com/RENCI/fuse-agent/blob/8dec59999e3e92f6ed02969190afe023766bb062/main.py#L540
Per the following article, "assert" statements should be used for debugging purposes:
https://www.programiz.com/python-programming/assert-statement


cherrytree: 0.38.4 -> 0.38.5

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools.
This update was made based on information from https://repology.org/metapackage/cherrytree/versions.
These checks were done:

built on NixOS
/nix/store/ldddgcpac0rvfnn1f06dgscaqws2rv9k-cherrytree-0.38.5/bin/.cherrytree-wrapped passed the binary check.
/nix/store/ldddgcpac0rvfnn1f06dgscaqws2rv9k-cherrytree-0.38.5/bin/cherrytree passed the binary check.
2 of 2 passed binary check by having a zero exit code.
0 of 2 passed binary check by having the new version present in output.
found 0.38.5 with grep in /nix/store/ldddgcpac0rvfnn1f06dgscaqws2rv9k-cherrytree-0.38.5
directory tree listing: https://gist.github.com/21bb973739375865396284be25e432a6
du listing: https://gist.github.com/da57acfe574b3ee8ff342306c396f8b7

meta.description for cherrytree is: '"An hierarchical note taking application"'.
cc @AndersonTorres for testing.
This build is cached with Cachix. To use the cache follow these instructions:
One time setup in nixpkgs Git checkout:
cachix use r-ryantm
git remote add r-ryantm https://github.com/r-ryantm/nixpkgs.git

Test this build:
git fetch r-ryantm && git checkout b5a7d3940562b8c6380c6c831f4bb07489ff3223 && nix-shell --pure -I nixpkgs=. -p cherrytree

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: cherrytree
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: cherrytree
Partial log (click to expand)
Writing /nix/store/7yzhj0nalknbsal136nvcb7czcaqnhz4-cherrytree-0.38.5/lib/python2.7/site-packages/CherryTree-0.38.5-py2.7.egg-info
glibPreFixupPhase
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/7yzhj0nalknbsal136nvcb7czcaqnhz4-cherrytree-0.38.5
gzipping man pages under /nix/store/7yzhj0nalknbsal136nvcb7czcaqnhz4-cherrytree-0.38.5/share/man/
strip is /nix/store/4qvrxzxa535y8304mk195x50b6p9607d-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/7yzhj0nalknbsal136nvcb7czcaqnhz4-cherrytree-0.38.5/lib  /nix/store/7yzhj0nalknbsal136nvcb7czcaqnhz4-cherrytree-0.38.5/bin
patching script interpreter paths in /nix/store/7yzhj0nalknbsal136nvcb7czcaqnhz4-cherrytree-0.38.5
/nix/store/7yzhj0nalknbsal136nvcb7czcaqnhz4-cherrytree-0.38.5/share/cherrytree/glade/svg_to_png.py: interpreter directive changed from "/usr/bin/env python2" to "/nix/store/d3aa78jf7pm9lk6ad67x4f21a068dxn6-python-2.7.15/bin/python2"
checking for references to /build in /nix/store/7yzhj0nalknbsal136nvcb7czcaqnhz4-cherrytree-0.38.5...

Success on aarch64-linux (full log)
Attempted: cherrytree
Partial log (click to expand)
glibPreFixupPhase
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/51w4b475n17x3cgczr06x5kczdz21aqw-cherrytree-0.38.5
gzipping man pages under /nix/store/51w4b475n17x3cgczr06x5kczdz21aqw-cherrytree-0.38.5/share/man/
strip is /nix/store/0pjsgkxz0rp5baycq5sp2s72lrr5q9sg-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/51w4b475n17x3cgczr06x5kczdz21aqw-cherrytree-0.38.5/lib  /nix/store/51w4b475n17x3cgczr06x5kczdz21aqw-cherrytree-0.38.5/bin
patching script interpreter paths in /nix/store/51w4b475n17x3cgczr06x5kczdz21aqw-cherrytree-0.38.5
/nix/store/51w4b475n17x3cgczr06x5kczdz21aqw-cherrytree-0.38.5/share/cherrytree/glade/svg_to_png.py: interpreter directive changed from "/usr/bin/env python2" to "/nix/store/fh8z4sniyv18b4jf32f3gnn862bvwc3f-python-2.7.15/bin/python2"
checking for references to /build in /nix/store/51w4b475n17x3cgczr06x5kczdz21aqw-cherrytree-0.38.5...
/nix/store/51w4b475n17x3cgczr06x5kczdz21aqw-cherrytree-0.38.5

Fine! It runs perfectly here.
If nothing happens until tomorrow at night, I will merge it.


Lockscreen

When the screen turns off after a few minutes away or when your lockscreen appears the virtual touchbar stop working, you have to reset the app, for me its pretty useful because my real touchbar lcd doesnt works, but the bar actually does, so the virtual one gives me a visual guide to use the real one, but its annoying when i have to reset the app lot of times per day

Closing as this app is now discontinued.


Update AzureCI matrix for Python 3.11/NumPy 1.21..1.24

This updates the AzureCI test matrix. It aims to ensure that:

The latest supported NumPy and Python are tested on all OS.
The oldest supported NumPy and Python are tested on all OS.
That the rest of the "matrix" formed from the outer product of supported NumPys x supported Pythons has at least one build if packages are available.

Closes #8834

Close/Open to trigger new CI build against fixed main.

CI is failing because there is no scipy for numpy=1.22 and python=3.11

The window llvm error is tracked at https://github.com/numba/numba/issues/8840

CI is failing because there is no scipy for numpy=1.22 and python=3.11

Thanks, 5424212 fixes.


Everquest crashes when loading some zones

GPU: GeForce GT 710
Driver Version: 435.24.02
Wine version: 4.17.r7.g042a633c ( TkG Staging Esync Fsync )
Processor: Intel(R) Core(TM) i5-2310 CPU @ 2.90GHz
When loading some zones, the game crashes with d9vk installed.  It works fine (if glacially slowly) without it.
I couldn't upload the apitrace (too big), so it's here:
eqgame.trace.gz
eqgame_d3d9.log

Can you upload this somewhere else?
Its currently downloading at 30kb/s

eqgame.trace.gz
Apologies for the slow link

Trace doesn't crash here. Are you going to the zones that crash for you?

Yes, the character I'm logging in to is in a zone that crashes.

Would you be able to create a trace made with D9VK at the same time?

That was with d9vk, I looked at the trace with qapitrace and it shows the second to last call as "465312 @0 IDirect3DDevice9::CreateTexture(this = 0x20b10cc0, Width = 512, Height = 512, Levels = 10, Usage = 0x0, Format = D3DFMT_R8G8B8, Pool = D3DPOOL_SCRATCH, ppTexture = ?, pSharedHandle = ?) = D3DERR_INVALIDCALL" So I'm wondering if the crash is an attempt to access a texture that doesn't actually exist.

Can you try spoofing AMD using d3d9.customVendorId?

Setting d3d9.customVendorId to 1002 does the same thing.

Interesting, it's wanting a D3DPOOL_SCRATCH of R8G8B8.
I think I see what I have to do, although it's mildly annoying. :P

Can you please try with this?
d3d9.zip

This does allow me to load, but the ground textures are missing/strange/black.

Trace?

HI that posted d3d9.dll allows me to login on zones that previously crashed

This has been in master forever

I'll try to get apitrace of the texture issue this night, I'll install the
latest version first though.
On Sun, Dec 1, 2019 at 12:41 AM Joshie<EMAIL_ADDRESS>wrote:

This has been in master forever
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Joshua-Ashton/d9vk/issues/417?email_source=notifications&email_token=AABIBJ35UPWTFOPVFBOA3YTQWNMDBA5CNFSM4I57MJXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFQ6WDQ#issuecomment-560065294,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABIBJ3564BE4A6QNHJ4AQTQWNMDBANCNFSM4I57MJXA
.


Static class usage

This PR is breaking.
To alleviate some of the boilerplate (making new variables for every transition), I have abstracted the class creation process into the state-machine itself. Behaviors and nested machines alike are provided to the state machine in static form. Access to the individual classes are provided only while the class is active, usually through shouldTransition.
The examples are not yet complete, so consider this branch W.I.P.

@GenerelSchwerz I see are you modifing the core of package, maybe you are developing your own package and this pr must be close?

@GenerelSchwerz I see are you modifing the core of package, maybe you are developing your own package and this pr must be close?

Probably, I decided to open a PR in case the Prismarine devs wanted to switch. Probably  not though.

Typings within this version are vebose (to say the least) but are completely functional.
Added functionality:
Backend/functionality
======================
1. immediate chained state switching.
-  re-iterate over transitions after change in case other transitions are now true.

2. multiple parent states for transitions.
-  provides strongly typed union for shouldTransition.

3. Wildcard implementation.
-  provided wildcard transitions will be checked regardless of currently active state.

# Frontend/utility
======================
1. Positions for states can be set per state machine instead of globally.
-  Positions are not provided on the behaviors themselves, instead they are provided to a handler.

3. Exit states will be colored.
-  Wildcard states may also become colored in the future.

As Frej mentioned on the Discord you can apply these changes on sefiros' change_nested_layout_bar branch to avoid a conflict later on after #172 gets merged

As Frej mentioned on the Discord you can apply these changes on sefiros' change_nested_layout_bar branch to avoid a conflict later on after #172 gets merged

Sure, I can do that. :+1:

@GenerelSchwerz this seems to be changing everything
could be considered to merge but only if you then want to maintain it. Is it the case?

I moved this PR to a new repository: https://github.com/nxg-org/mineflayer-static-statemachine
There are a lot of additions/improvements I have added to that project that I will try to move here in time.


Simplified directory structure

Now the files are directly in the zip. If you don't like it, just close the PR without merging.
Here's the output: https://github.com/sztrzask/ModFinder/releases/tag/v1.2.1

Can you update so the zip contains the ModFinder folder which contains everything?

Should be doable, lemme try

Done - output is this: https://github.com/sztrzask/ModFinder/releases/tag/v1.2.6


Added and modified AudioStream descriptions

Added description for the get_length() function, and amended the get_playback_position() function to mention the measurement.

Tip for next time: add meaningful commit messages. "Added description" doesn't mean anything, it added description of what? When looking at the Git history with over 15,500 commit we want to understand what was done just from the title without opening the diff.

Can do, @vnen. I want to write a better description in the Git commit, but I may end up making them too long. ;)

Cherry-picked for 3.0.7.


upd(android-studio): 2<IP_ADDRESS> -> 2<IP_ADDRESS>

Also as a heads up, use conventional commits for commit messages

conventionalcommits.org/

Issues with installing so plan to fix.

Is this ready for review? Why it a draft still?

Android-studio does not install properly and if I were to update GitHub there would be a lot of commits. The current state on my end is it can't find $STUDIO_JDK

Still once more, same issue, Android-studio can't find JDK and other needed files.

:

Because there doesn't appear to be a solution linking files this PR will be closed.


Select from values with dynamic number of values

I want a query like this
SELECT
  ts_from,
  ts_to
FROM (VALUES
  ('2024-08-10 21:45:00'::TIMESTAMP, '2024-08-11 21:45:00'::TIMESTAMP),
  ('2024-08-11 21:45:00'::TIMESTAMP, '2024-08-12 21:45:00'::TIMESTAMP),
  ('2024-08-12 21:45:00'::TIMESTAMP, '2024-08-13 21:45:00'::TIMESTAMP)
) AS t(ts_from, ts_to)

but with a dynamic number of values coming from an array of arrays: [['2024-08-10 21:45:00', '2024-08-11 21:45:00'], ...] or array of objects.
I found in the docs here how this can be done statically

Here is what i tried
import { sql } from "kysely";

const intervals: [from: string, to: string][] = [
  ["one", "two"],
  ["three", "four"],
];

const intervalsStr = intervals
  .map(([from, to]) => `(${from}, ${to})`)
  .join(",");

const values = sql<{
  a: string;
  b: string;
}>`(values ${sql`${intervalsStr}`})`.as<"t">(sql`t(a,b)`);

const rows = await db
  .selectFrom(values)
  .select(["t.a", "t.b"])
  .execute();

but it escapes the array as one value
SELECT "t"."a", "t"."b"
FROM (VALUES $1) AS t (a, b)

and not sure how to make produce this instead:
SELECT "t"."a", "t"."b"
FROM (VALUES ($1, $2), ($3, $4), ...) AS t (a, b)

How could i achieve this?

See this example https://kysely.dev/docs/recipes/extending-kysely#a-more-complex-example


Sum the second value returned from pyglowmarkt if it exist.

Description
A change in how pyglowmarkt handles timezones causes two values to be returned most of the times instead of a single value.
Motivation and Context
The above change in pyglowmarkt was not reflected and only the first value was used, which only contained data from the first our of the day.  This is described in issue #330 and this PR fixes the issue.
How Has This Been Tested?
The fix has been tested on my own live system.
Types of changes

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

This is obviously not the "right" way to fix this (the right way would be to handle daylight savings properly and only ask for today's data) but in the interest of getting this integration working again, I am happy to merge this and release a new version.
Thank you so much for the PR @eknirk!

The “right” way would probably get the change in pyglowmarkt reversed as I cannot think of a way that this can be done in ha-hildebrandglow-dcc alone.

On 19 Apr 2023, at 20:58, Aniket @.***> wrote:
This is obviously not the "right" way to fix this (the right way would be to handle daylight savings properly and only ask for today's data) but in the interest of getting this integration working again, I am happy to merge this and release a new version.
Thank you so much for the PR @eknirk https://github.com/eknirk!
—
Reply to this email directly, view it on GitHub https://github.com/HandyHat/ha-hildebrandglow-dcc/pull/331#issuecomment-1515300751, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQOYHDXD3SFTHAXDCHONO3LXCA7WFANCNFSM6AAAAAAXCQGT2Y.
You are receiving this because you were mentioned.

Can I just ask, do I just copy the code under the heading changed filed and paste it over the top of what is in my config file on HA?
I've not dealt with a pull request before.


🐛Bug: Select builder multi-select defaultSelected does not allow for default value to be deselected

Describe the bug
When using defaultSelected on a multiselect field (multiple: true), the default selected value is not unselectable.  This comes from CreateSelectProps.
To Reproduce
Please check Minimal Repro
Expected behavior
The default value should be unselectable.
Screenshots
https://github.com/melt-ui/melt-ui/assets/91398645/b6e0d8ad-bd77-43b0-b698-1192942ced06
Additional context
This is using melt 0.51.0
@wesharper may be able to take a look.

This can be marked as resolved (possibly might want a doc update), but it is a bit of an awkward API that is related to some of my feelings expressed here. I think if the select builder had less reliance on deep equality and instead preferred matching on the value attribute, some of these gotchas may come up less.
The issue is that the toggle helper does a deep equality check for whatever is in the array. When melt adds an option object, it implicitly includes all 3 attributes of the SelectOption object (i.e. disabled, label, and value). However, if any of these is omitted from the defaultSelected array, the deep equality check fails, which prevents the toggle from working as expected.
The API that feels most natural to me in this case would be to just pass an array of simple values and let melt manage option objects behind the scenes. The use case here is the following:

User uses a multi-select input in a form to create a new object.
Database stores an array of values (not options objects)
User loads the form to edit the object.
Client passes in values array from DB directly and melt handles accordingly.

As is, this experience is not possible, and the consumer has to manually convert the array of values to an array of options objects in order to get things working reliably. Once the consuming party knows all of the details, it's a relatively minor annoyance, but the ideal for me would be for melt to create and accept values that can directly interface with the back-end without any "massaging".

Here's some debug logs using the minimal repro as a starting point.

I have some ideas for a PR that I can try to get to on Friday, but ultimately it would probably impact lots of folks.

A lightweight alternative could be to modify the defaultSelected array items into full option values with defaults similar to what happens when using the $option function.

but the ideal for me would be for melt to create and accept values that can directly interface with the back-end without any "massaging".

I'm not against an alternative! Would be interesting to see.


Expose helpers as shared module and usable while writing custom code

I am working on a private extension at the moment and I noticed that the core extension already provides specific helpers methods that I would like to reuse as sharedModules and as helpers when I wrote custom code in rule and data elements.
I am interested in the following helpers:

getObjectProperty https://github.com/adobe/reactor-extension-core/blob/master/src/lib/helpers/getObjectProperty.js
textMatch https://github.com/adobe/reactor-extension-core/blob/master/src/lib/helpers/textMatch.js
valueComparison https://github.com/adobe/reactor-extension-core/blob/master/src/lib/conditions/valueComparison.js
variable https://github.com/adobe/reactor-extension-core/blob/master/src/lib/conditions/variable.js

I am aware that variables and valueComparison are both linked to specific conditions but it would be ideal to move them to helpers.
Ideally once exposed the use will be:

when writing private extensions use turbine.getSharedModule('core', 'getObjectProperty'); to reuse in code to avoid writing similar code for same purpose
in configuration add an option to expose helpers to window._satellite.helpers.getObjectProperty to reuse in custom code section in Adobe Launch rules and data elements

At the moment I copied the helpers inside my private extension build, this leads to redundant code that is why I would like to use sharedModules capabilities and expose to window

Thanks @alcazes! I understand your desire to have these as shared modules. There are some downsides to having them as shared modules at the moment that we'll need to take into consideration. If the code were exposed as a shared module, it will always be included in all Launch libraries for all Launch properties that have Core installed, regardless of whether the code is used in the library or not. This is because the system that builds the Launch library isn't smart enough to know if another extension in the library is actually using the shared module (e.g., makes a call to turbine.getSharedModule('core', 'getObjectProperty');), so it always includes the shared module code just in case. On the other hand, if the code is not exposed as a shared module, it can be trimmed based on the features of the Core extension that a particular library might be using (see https://medium.com/adobetech/build-optimization-through-minimal-inclusion-87ea637e174f for more info).
We want to eventually improve the build process in such a way that we could determine whether a shared module is actually being used by another extension in the same library and, if not, exclude the shared module code from the library builds. Until then, I think this issue will need to stay open because the cost to everyone's library size would be too great for the potential benefit received.

@adobe export issue to Jira project PDCL

Checking in to make sure this request has not been forgotten.
Some of our platform have raised that Adobe launch seems to be one of the biggest library to load initially so at the moment every kb counts.

I would like to add load-script to the list of shared module usable in custom code.

I would like to add load-script to the list of shared module usable in custom code. I.E. something could look like _satellite.loadScript(), like we have _satellite.cookie.set() for cookies.

At present I was able to achieve this using a private extension:

But it would be great if it was available out of the box.


[FEATURE REQUEST] Add CodeJam event

Motivation
Why are you requesting this feature? Is it related to a problem?
CodeJam registrations open on March 3, 11AM PST. Ends April 4, 7PM PST.
Qualifiers on April 3, 4PM PST. Ends April 4, 7PM PST.
https://codingcompetitions.withgoogle.com/codejam/schedule
Proposed Change
What are you proposing?
Add event to Events page, Home page, and announcements on Puzzle page.

If this is difficult, there should probably be a way to make it easier :p


Added submission successful page

Issue Number
fixes #316
Describe the changes you've made
Added a page to indicate successful  submission of the form and also provided a link to form correction page in case any one want to edit.
Describe if there is any unusual behaviour of your code(Write NA if there isn't)
NA
Additional context (OPTIONAL)

Test plan (OPTIONAL)
A good test plan should give instructions that someone else can easily follow.
How someone can test your code?
Checklist

[x] My code follows the code style of this project.
[x] I have updated the documentation accordingly.
[x] All new and existing tests passed.
[x] The title of my pull request is a short description of the requested changes.
[x] Open Source Program names(OPTIONAL: If you participated in any open-source program then please mention the program name here)LGMSOC21

@menakinako nice.
Just change the name from thank you page to the submitted page.
Also, rename 'Edit Here' to "View Status of Application".

okayy.
I have made the changes


Added Rundeck Chart

What this PR does / why we need it:
k8s chart for Rundeck.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Followup to earlier PR
https://github.com/kubernetes/charts/pull/6201

/assign @sstarcher @dcwangmit01 @josdotso

/assign @sstarcher

@Deepak-Routray Please close this duplicate PR, and resubmit the changes in your old PR.
Your old PR code has not been lost.  It lives in the branch of the repository from which you've created your original PR.
The original PR lives here: https://github.com/kubernetes/charts/pull/6201


Make events parameter required for list_events

Description
Make events parameter required for list_events.
Documentation
Does this require changes to the WorkOS Docs? E.g. the API Reference or code snippets need updates.
[ ] Yes

If yes, link a related docs PR and add a docs maintainer as a reviewer. Their approval is required.

I'm not much of a rubyist. Do I need to update the method signature or comment at all on this options hash to reflect this parameter requirement?
https://github.com/workos/workos-ruby/blob/2a90a0bee142c31b11cfe3d85017f5d505f3f0ec/lib/workos/events.rb#L16-L24

This change doesn't look to be doing what you want. The InvalidRequestError would come from making the request with no parameters, not the SDK telling the user they need to add a parameter. If event is a required parameter I would probably change the signature of list_event to thus:
def list_event(event:, options = {})
end

This way the user will get a ArgumentError when they forget event and this should also be backwards compatible to current callers to this function.

Ah, right, that syntax isn't 2.7 compatible. You might need to add extra logic looking for options[:event] and throw an ArgumentError if it doesn't exist.

Ah, right, that syntax isn't 2.7 compatible. You might need to add extra logic looking for options[:event] and throw an ArgumentError if it doesn't exist.

Ah okay. Let me move make some changes.


codecov broken

Meta/Bug Report
Current Behavior
The codecov package fails to
a) fetch the endpoint or
b) find the token
Example a): link to run
==> Uploading reports
    FetchError: request to https://codecov.io/upload/v2?commit=ed7a5134beaa23855720f5c03dc4efab1d7f1d9f&branch=master&yaml=%2Fhome%2Frunner%2Fwork%2Fprobot%2Fprobot%2Fcodecov.yml&package=node-v3.7.2 failed, reason: connect ETIMEDOUT <IP_ADDRESS>:443

Example b): link to run
==> Uploading reports
    HTTP 400
Please provide the repository token to upload reports via `-t :repository-token`

Expected behavior/code
Github actions should send coverage data to codecov
Environment
N/A
Possible Solution
a) No Idea
b) Maybe somebody removed the token secret? Add it back.
Additional context/Screenshots

Thanks for flagging this @FoseFx! The secret is still set, but it looks like its not available in the workflow:
https://github.com/probot/probot/blob/ed7a5134beaa23855720f5c03dc4efab1d7f1d9f/.github/workflows/test.yml#L47-L48
We'd need to add the following to this step:
env:
  CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}

I'm not sure how it ever worked before this 🤔

#1350 seems to fix the issue, I'll monitor the next couple of commits and close it when nothing bad happens

Update: This run did not submit a coverage report.
Run npx codecov
npx: installed 24 in 2.014s
  _____          _
 / ____|        | |
| |     ___   __| | ___  ___ _____   __
| |    / _ \ / _` |/ _ \/ __/ _ \ \ / /
| |___| (_) | (_| |  __/ (_| (_) \ V /
 \_____\___/ \__,_|\___|\___\___/ \_/
                                v3.7.2
==> Detecting CI Provider
    No CI Detected. Using git/mercurial
==> Configuration:
    Endpoint: https://codecov.io
{ commit: '235e308e357d29184d353c12fb643e868db96fb6',
  branch: 'master',
  package: 'node-v3.7.2' }
==> Building file structure
==> Generating gcov reports (skip via --disable=gcov)
    Failed to run gcov command.
==> Scanning for reports
    + /home/runner/work/probot/probot/coverage/lcov.info
    + /home/runner/work/probot/probot/coverage/clover.xml
==> Uploading reports
    FetchError: request to https://codecov.io/upload/v2?commit=235e308e357d29184d353c12fb643e868db96fb6&branch=master&yaml=%2Fhome%2Frunner%2Fwork%2Fprobot%2Fprobot%2Fcodecov.yml&token=***&package=node-v3.7.2 failed, reason: connect ETIMEDOUT <IP_ADDRESS>:443

🤔 Maybe let's observe if this issue continues. If it does, can you ping Codecov about it?
Longer term, the best solution is to reach 100% test coverage and just fail tests on our side if coverage falls below. I've suffered my fair share of Codecov / Coveralls unreliability 😞

Only happened once in the last 7 commits, so I'm going to close this. In case anybody think this is a significant enough issue, feel free to re-open.


Consider renaming the package

First line of your README file is: "A data type for untagged unions." I think that purescript-untagged would be more descriptive name for this package ;-) Like in my previous comment - it is just a matter of preference for sure so just close the issue if you don't want to discuss naming issues.

Still very open to renaming the project. And changing the name of everything else inside.
I initially named it as OneOf since my original idea was simply to use rowlist so it reads like:
type ISB = OneOf (i :: Int, s :: String, b :: Boolean)

Incidentally, I found the same name used in: https://github.com/doolse/purescript-tscompat
Problem was it turns out, the labels in the rowlist aren't useful, and that the order matters. So I ended up with the current |+| operator.
So you're right, untagged as a name might be better, or perhaps untagged-unions to be more precise? If we choose the latter though, should we name our pacakge as UntaggedUnions or Untagged.Unions? Naming is hard 😢
I'm also considering changing |+| to |: so it resembles typescript more.

I like untagged-unions :-) Maybe even untagged-union as it introduces single abstraction. I'm somehow trying to follow @natefaubion module naming suggestion:

if a module introduces single abstract entity, a class or a type it can be called in singular

If a module contains multiple instances or values of a given class / type I name it in plural form

Regarding operator I much prefer this one: |+| or maybe this :+: (I'm following similar convention :*: from selda in my ntuples module).
....naming is hard ;-)


Project not marked as affected if dependency upgraded

There seems to be an issue where rust projects are not affected if a dependency in the versions file gets updated.
We're using dependabot to update our dependencies and if we update a javascript library the projects get marked as affected. This doesn't happen for the gradle projects if we update a gradle dependency.

Hi @paulhobbel Thanks for raising this issue and this discussion.
Do you speak about the root gradle files?
Do you use gradleRootDirectory to isolate nx root project from gradle root project?
Can you fix this with the Named Inputs https://nx.dev/recipes/running-tasks/configure-inputs?
Can you produce this with https://github.com/khalilou88/jnxplus-examples repo so we can have a common understanding of the issue?

Thanks for the quick response.

Do you speak about the root gradle files?
If we update a version in {workspaceRoot}/libs.versions.toml the gradle projects are not affected

Do you use gradleRootDirectory to isolate nx root project from gradle root project?
What exactly do you mean by this? This is currently set to .

Can you fix this with the Named Inputs https://nx.dev/recipes/running-tasks/configure-inputs?
I've looked into this as well, that would mean we would have to do something like the following correct?

{
  "namedInputs": {
    "default": [
      "{projectRoot}/**/*",
      "sharedGlobals",
      "!{projectRoot}/**/*.md"
    ],
    "production": ["default"],
    "gradle": [
      "production",
      "{workspaceRoot}/gradle/libs.versions.toml"
    ],
    "sharedGlobals": ["{workspaceRoot}/.github/workflows/pr.yml"]
  },
}

I will try to reproduce this with the example project of yours later today

If you set gradleRootDirectory  to . or if you don't use it. It's mean you are mixing the nx root project and the gradle root project. So if you change gradle configs you will affect angular or react project for example and I think you don't want that.
I create a PR to explain my idea about using inputs here : https://github.com/khalilou88/jnxplus-examples/pulls
I like your idea but I think is more suitable for a 100% java project.

I tried to reproduce the issue with https://github.com/khalilou88/jnxplus-examples and everything works fine:

clone repo
run npm i
run nx run-many --target=build
run nx graph --affected: No project should be marked as affected
update {workspaceRoot}/nx-gradle/gradle/libs.versions.toml by adding a new variable for example
run nx graph --affected again: All projects should be marked as affected

It works, adding those files to the inputs fixes the issue we had, thank you very much!


Contrast ratio of main navigation is too low

The contrast ratio, between the magenta and the light grey in the main menu, is too low at 4.21:1
With magenta isles and buttons, the contrast is against white which is fine.
With islands and islets that have the light grey background — #f4f4f4 — the magenta link colour is adjusted to use the more saturated version of magenta, #BE3778 — this is a named colour in the SCSS (I forget the name).
The menu bar should either use this version of magenta, or revert to white where there is now light grey.

Ready for testing in test3

Light grey has been replaced with white so now passes AA.
Thank you, @Vino-S


Add test_whole_domain plus minor misc. improvements

Follow-up from https://github.com/mysto/python-fpe/pull/5#issuecomment-923329154

Hi, I had to back out test_whole_domain as the test was failing on the import statement when built and run by travis and on my local Mac.
$ python3 ff3/ff3_test.py
Traceback (most recent call last):
File "ff3/ff3_test.py", line 25, in
from ff3.ff3 import reverse_string
ModuleNotFoundError: No module named 'ff3.ff3'; 'ff3' is not a package
The command "python3 ff3/ff3_test.py" exited with 1.
https://app.travis-ci.com/github/mysto/python-fpe/jobs/539306809


Same as before ordinal-mini-doges not correct on website

Hello guys, same old issue still not fixed.
ordinal-mini-doges is not correct on website. it seems the inscriptions.json was automatically rebuilt 3 months ago.
as mentioned as previously raised issues, we updated ordinal-mini-doges because 15 ordinals were transferred to the second collection pixel-doges.
I see now that you have listed our second collection pixel-doges which is good.
But the 15 ordinals that we wanted to remove from ordinals-mini-doges are now duplicated and existing in both collection.
Please help, all whats left to do is to remove the 15 from ordinal-mini-doges and its all good :
this is their inscription ids, they should only exist in pixel-doges collection, not ordinal-mini-doges collection.
c165a359a7ee43417156839d6f67d8c2552b6c8fadb6b20136375a900bcf4a28i0
2c3f52510b540333eeeac1c52e4bc100a3c04828b1983b6533318df969779639i0
e5922f7d2315d3ed2be3719c504314aa6a0cd21afeaca48443dfa39a29497a45i0
47fe89800be9445970579582dac2c59b064b819b07f7a20fda2fe7d410d3ff7fi0
4a25149cd270f1a600ca43aa13d98913d549bd31a1de515874db047dfed30985i0
966e3040c523890be6ee1532793b10e907b8b05877a55886f1260fa852afcfb1i0
2b18270a705e0ea449439a91763d374a88115f50630ae4dfdfe53b3767312ecai0
34ae08ab644fcccbab70f239e04fa023b94effcfbc8674d17f598a6759e90a1ci0
9720c2db0adbfdb87df9b8988af4b4517fe41cc51b3715dc365d36caab170448i0
dac39159be93067416ed3f1bf35f66f74d9fc802c73e91bc30c876ef0475b073i0
b5749d747e8f48545ece82c0b431266faffbc0cf6dc79687e178833784a17d84i0
6943b3328d77146df4c818ae411ec6c32160fa255f729201fb5deaa6718ec8dbi0
423e2f9dc7e517ec07a51ea02c6d6031980fd9e80c851e1c1b10cae70e8f5fe8i0
17f20a69b3f0a53bd087ef8de0c308c7abd9eb7774847fa04f3b094672af4ebfi0
8b7f88d9a726da3c1336f009c61272a54318e52637b9c3ded9c5549d93e6b3bfi0
Please help as it has been so many month we have tried to seek help here.
thank you !

Guys we also own "Doginal Mini Doges" inscription 14578-24913. We will launch soon, but it would be great to fix the issue above first. I would gladly submit another PR to once again fix the inscriptions.json, but as stated in our previous raised issues (now closed without fix), the website was not reflecting the changes. Please, help.


Unexpected Behavior of return in Switch Block

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
Executing the following script in PowerShell 7.4.0 results in an unexpected behavior which also differs from PowerShell 5.1
function foo($x) {
    $val = switch ($x){
        1 { 5 }
        default {
            return 10
        }
    }
    Write-Host "Val is $val"
    $val + 1
}

Write-Host "foo 1: $(foo 1)"
Write-Host "foo 2: $(foo 2)"

PS 5.1 output:
Val is 5
foo 1: 6
foo 2: 10

PS 7.4.0 output:
Val is 5
foo 1: 6
foo 2:

It's not clear if this is a bug or simply a documente change from classic PowerShell, but it certainly behaves in a way I don't find intuitive.
Expected behavior
PS> <execute script>
Val is 5
foo 1: 6
foo 2: 10

Actual behavior
PS> <execute script>
Val is 5
foo 1: 6
foo 2:

Error details
No response
Environment data
Name                           Value
----                           -----
PSVersion                      7.4.0
PSEdition                      Core
GitCommitId                    7.4.0
OS                             Microsoft Windows 10.0.19045
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Name                           Value
----                           -----
PSVersion                      5.1.19041.3570
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.19041.3570
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

Visuals
No response

I have a feeling this isn't specific to switch statements - it appears the return expression ends up in the original assignment target when you return from any dot-sourced block:
function f {
  try {
    $value = . { return 10 }
  }
  finally {
    Write-Host "`$Value is: '${Value}'"
  }
}

$returnValue = f
Write-Host "Return value is: '${returnValue}'"

The different behaviors can be demonstrated with the following simple command:
$a = if ($true) { return 10 }

Windows PowerShell and PowerShell (Core) 7.3.x (presumably earlier too):

No assignment, outputs 10.

PowerShell 7.4.0:

No assignment, no output.

That is, what the behaviors share is that no assignment takes place, but that there is no output in 7.4.0 is clearly a bug.
The WinPS / PS 7.3- behavior may be surprising, but is defensible:

It is important to note that return does not exit just a { ... }-enclosed part of a language statement such as if or switch, but its enclosing function/script file/script block.

That is, the { ... } parts of such statements do not function like stand-alone script blocks such as the ones passed to ForEach-Object, in which return truly only exits that block.

Therefore, in the original code the return immediately exits the whole function, with 7.4.0 inexplicably discarding the output.

@mklement0 The behavior of 7.3.x and previous is what I'd intuitively expect to happen when reading the code.