Release of foodon should always have asserted superclasses

melon seed has no superclass http://purl.obolibrary.org/obo/FOODON_03307879
Consider using robot relax

OK, multiple problems here:
melon fruit food product and (has part only seed)

dubious. The definition is inclusive of mereological sums of seeds
transitive properties can't be used in combination with universal restrictions. Consider: nothing has-part only seeds. If X has-part seed, then X has-part endoderm, endoderm is not a seed, therefore X is unsatisfiable. I think you need hermit/fact to see this

In general, I think FOODON needs to be a bit more cautious with the automated/sweeping assertions. @Public-Health-Bioinformatics we should consider having an extended call to go through some of the top-levels.

Ok, fixed 'melon seed'. Superclass must have accidentally been deleted.
I'd welcome a call on the axiomatization, I appreciate the advice... I'll connect to find a good time.
About "melon fruit food product and (has part only seed)" , that should have been:
'melon fruit food product' and ('has part' some seed and 'has part' only seed)

Now fixed. Now the way I was expecting that to semantically ring true is that 'melon seed' is a descendant of 'melon food product', defined as:
'cucurbit fruit food product' and ('derives from' some 'melon plant (food source)')

So 'melon seed' is a food product deriving from some 'melon plant (food source)' and only having part seed.  Can that be improved some how?  'melon plant (food source)' has subclasses, some of which point to specific plants via NCBITaxon. The remainder will eventually have NCBITaxon info.

By the way my approach is to have at least one explicit superclass for any class. As well, if there is an equivalency axiom for the class, the explicit superclass should be mentioned as the leading term in it (as a conjunction term). This guarantees a hierarchic structure for users who aren't using a reasoner.  But I see now what your "robot relax" transformation does to this end too, @cmungall .

Better!
The only+transitive issue remains. I'd recommend thinking very carefully about  universal restrictions in general. As well as being outside the scope of what can be reasoned in Elk, they are a minefield for modeling errors. I think there should be a feature in Protege where a paperclip pops up whenever you use an only where it says "are you SURE you need this?"
Some other issues - your axiom means that wild melon seeds are food products. I thought the modeling pattern in foodon is that a material entity must have a food role to be food products.
What is wrong with 'melon seed = seed and part-of some NCBITaxon:melon'?

I'll look at the transitivity issue next week to understand what you mean.
'melon seed' inherits "'environment material' and 'has role' some food" via "food material" -> "plant food product"... so wild melon seeds can be food products if fulfilling the food role. (I suppose 'has role' some food" could be added to top level "food product type" too, although chemical additives may preclude this move.)
In FoodOn we have a separate "food source" branch that currently attempts to enumerate all plants and animal organisms as a whole that are used as food sources by humans (and domesticated/agri animals), but this this is not directly a biological taxonomy hierarchy for a few reasons. I've made the deliberate move to separate out linguistic reference to food organism from reference to taxonomy.  We aspire to cross reference each food source item with an NCBITaxon or other taxonomy identifier, but since NCBITaxon is not the only taxonomy in town, I set it up that each "food source" branch entity becomes the focus on which cross reference to taxonomic identifiers like NCBITaxon occur. This allows food source reference to organism in the face of changing taxonomies, or lack of taxon info.
Now there is some debate about whether 'food source' and 'food product type' should be merged, but it is clear to see that references within 'food product type' to 'food source' organisms indicate one has hit "rock bottom" in the food processing chain.
Earlier I considered
'melon seed = seed and 'derives from' some 'melon plant (food source)'

with 'melon plant (food source)' referencing NCBITaxon. I would like to talk about this on a call. One thing I don't like is that the PO plant ontology hierarchy suddenly starts getting food products placed under its classes like "seed" (even without using a reasoner in protege). So what was a PO hierarchy of anatomy parts/features suddenly gets product variations under it, essentially a shift in paradigm of the hierarchy. But if that is acceptable to the community, then I can run with it.

@cmungall I see point that a universe containing 'melon seed: equivalent to 'melon food product' and 'has part' only seed' can't coexist with a set of claims that infer a whole (normal) organism from the existence of one of its parts, as an anatomy ontology might bundle up.  " X has-part seed -> X has-part endoderm" etc.  So are we talking here about how one ontology or rule engine's use of 'part of' would clash with FoodOn's? A solution then would be to switch to using a less-general purpose object relation like 'has ingredient' ?
To understand none and transitivity, I tried to express your problem model in protege and ran against hermit/fact, but didn't run up against contradiction, so can you see what I'm missing? I presume I don't need instance data to see it? Is problem more about rule engine implication than it is about object relation transitivity?
classes
seed: disjoint with endoderm
endoderm: disjoint with seed
plant: equivalent to: 'has part' some seed and 'has part' some endoderm
melon seed: equivalent to 'melon product' and 'has part' only seed  // + 'has part' some seed
melon product: disjoint with 'non-melon product'
non-melon product: disjoint with 'melon product'

object relations
'has part': transitive

On 23 May 2018, at 0:57, Damion Dooley wrote:

@cmungall I see point that a universe containing 'melon seed:
equivalent to 'melon food product' and 'has part' only seed' can't
coexist with a set of claims that infer a whole (normal) organism from
the existence of one of its parts, as an anatomy ontology might bundle
up.

I think you're saying the incoherency is only revealed when combined
with an anatomy ontology - in this instance yes, but it's not hard to
come up with trivial examples where the mix of universal+transitive is
incoherent in a single ontology.

" X has-part seed -> X has-part endoderm" etc.  So are we talking here
about how one ontology or rule engine's use of 'part of' would clash
with FoodOn's?

No I think it's more basic

A solution then would be to switch to using a less-general purpose
object relation like 'has ingredient' ?

a more specific property can work

To understand none and transitivity, I tried to express your problem
model in protege and ran against hermit/fact, but didn't run up
against contradiction, so can you see what I'm missing? I presume I
don't need instance data to see it?

No

Is problem more about rule engine implication than it is about object
relation transitivity?

Not sure what you mean by rule engine implication, it's not about any
particular rule engine

classes
seed: disjoint with endoderm
endoderm: disjoint with seed
plant: equivalent to: 'has part' some seed and 'has part' some

endoderm
melon seed: equivalent to 'melon product' and 'has part' only seed
// + 'has part' some seed
melon product: disjoint with 'non-melon product'
non-melon product: disjoint with 'melon product'
object relations
'has part': transitive

you need to express the relationship between seed and endoderm

--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/FoodOntology/foodon/issues/47#issuecomment-391220386

Ok, what was throwing me was that I couldn't find "endoderm" in relation to the seed itself (my botany knowledge is minimal so I'm looking at https://en.wikipedia.org/wiki/Seed). So I think "endosperm" is what the test calls for, so I'm rejigging it for that.  Thanks.

I get it crystal clear now. In using the "only" constraint in conjunction with a transitive relation 'has part', e.g. "X has part only A", transitivity allows an axiom "A has part some B" to yield "X has part some B", a contradiction.  So all uses of "only" cannot involve transitive relations. Which is what you said! Oy.

I've dropped the "only" part and made it a subclass axiom.  The equivalency was experimental - i.e. the only food product using an "has part only ..." axiom - so fortunately no other changes to FoodOn required in this respect.


[10.x] Batch remove transaction for sync

Follow up on: https://github.com/laravel/framework/pull/48699
I kept the transaction for all other drivers/connections.
For sync connection should not use a transaction since there is no need for one and its causing a bug that when a job fails it rolls back any insert/update queries.

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.


In GSSoC'24, GitHub Topics will help the discoverability of your project.

I see that you already have great topics on your repository!
I would recommend adding the name of the company like the software you use to build like "vs-code, ghdesktop" to improve your discoverability.
If you are happy with the topics you have, feel free to close this issue. 👍

@Princegupta101 Thank you for contributing to the repo but this is not needed as of now.


MCO-392: Start using rhel-coreos image rather than machine-os-content.

The machine-os-content is deprecated and we'd like to stop shipping it entirely in the release payload. Point the DTK imagestream to the rhel-coreos image instead.

Closes: https://github.com/openshift/driver-toolkit/issues/101
/cc @jlebon

Thanks for opening this!

/hold
Should be merged together with openshift/release#48526

It's not necessary they go together. https://github.com/openshift/release/pull/48526 should be able to go in alone. Once we confirm everything still works fine, and we're confident there's nothing else relying on it, then we can coordinate on flipping over again DTK, RHCOS and ART together.

It's not necessary they go together. https://github.com/openshift/release/pull/48526 should be able to go in alone. Once we confirm everything still works fine, and we're confident there's nothing else relying on it, then we can coordinate on flipping over again DTK, RHCOS and ART together.

Ack. I am not sure what https://github.com/openshift/release/pull/48526 really does but in DTK we are referencing to the machine-os-content labels for the driver-toolkit imagestream in the cluster (which is used by KMM to build kmods in cluster).
Anyway, I will keep it on hold until we have the ART+RHCOS+DTK parts together.
Thanks for the update.

OK, we're looking good now. openshift/kubernetes references have been removed.
I've re-staged the openshift/os change in https://github.com/openshift/os/pull/1457.
Let's target early next week to get these in? Will reach out more directly for coordinating.

/retest

/override ci/prow/images
/override ci/prow/e2e-aws
/override ci/prow/e2e-aws-driver-toolkit-presubmit
/override ci/prow/e2e-upgrade

/hold

/override ci/prow/images

/lgtm

/override ci/prow/e2e-aws

/override ci/prow/e2e-upgrade

/override ci/prow/e2e-aws

/override ci/prow/e2e-aws-driver-toolkit-presubmit ci/prow/e2e-upgrade

/override ci/prow/e2e-aws-driver-toolkit-presubmit

/override ci/prow/e2e-upgrade

/override ci/prow/images

/label acknowledge-critical-fixes-only


Handle dash-number interpolations in SCSS. Fix Microsoft/vscode#56743

Fix is good, thanks @octref !


Staging - After adding a manual entry, a new session is started for the currently active session project.

Repro steps

Start a session
Add a manual entry

Expected Result
The session is still continued, it shouldn't be interrupted to add a manual.
Actual Result

System Info

Manual session types are no longer considered valid 'last session' types


The return value from Function.prototype.bind.bind is not marked as pure

Function.prototype.bind.bind is effectively marked as pure as seen here. However, the problem is that the return value of Function.prototype.bind.bind is not marked as pure. As seen here (which should compile down to zero bytes), Closure Compiler does not consider it to be pure because it does not compile down to zero bytes.

This issue has been solved and fixed. Let us close this issue.

Done, thanks!


[PYPI] Flesh out readme and add long_descrition

I guess its time to up the ante.
At least the "organs" of Ovos should get the special treatment
(should referencing each other)

let's do a final pass over the readmes before closing this issue
linking relevant docs sections  (tech manual + community + docker where applicable) and companion repos

bus + bus-client
core
gui
listener
audio
media (soon)
workshop
opm
PHAL
utils

I'm pretty sure I got all the missing sections in setup.py for all those repos, but as we review the READMEs please double check 🙂

Is this good? I see a lot of PRs

closing for now


node-tap strange behaviour cause CircleCI & Travis-CI keep failing

Issue detail is posted here: https://github.com/tapjs/node-tap/issues/272
Stackflow: http://stackoverflow.com/questions/38239735/strange-bug-in-circleci-travie-ci-docker-env-node-tap-js-pass-but-tap-tap-js
CircleCI Forum: https://discuss.circleci.com/t/strange-bug-in-circleci-docker-env-node-tap-js-pass-but-tap-tap-js-will-fail/5005

switched tap to tape had already solved this problem.
now wechaty use ava as test runner.


dynamically configure Redirects in Routes?

Some details
Hello, my question is this:
I realize that YARP is meant for proxying, not redirecting, but YARP does provide an elegant dynamic configuration system that is not default to .Net Core's Redirect setup... ie dynamically configuring Redirects in .Net Core is hard 😢
I looked at the YARP route configuration and didn't see any notion of Redirects.
My current application has a table of Redirects that get configured on startup, but they are not dynamic -- ie if they change in the database we have to restart the site.
As a part of a rewrite of our app we're using YARP to proxy some pages to the new site and some to the old.
However, given that my YARP project now effectively provides a nice "pinch point" for all things route(ish), it seems like a good place to also handle general redirects in a more dynamic way.
Is there anything built-in that I'm missing, or can you point me to any examples that I could use to come to a solution for dynamically configurable redirects.
For some background, an example redirect is a feature of the old site that we're deprecating now should be redirected to the homepage.
How many backends are in your application?

[x] 1-2

How do you host your application?

[x] Azure App Service
[x] Azure Container Apps

You're right that this isn't natively supported, but you could add it with custom transforms keyed from config.
https://microsoft.github.io/reverse-proxy/articles/transforms.html#extensibility
https://github.com/microsoft/reverse-proxy/pull/1923

@burtonrodman did you look at the above? Is it sufficient for you?

sorry, haven't had a chance to look back at this.  will do asap.

@burtonrodman do you have rough ETA when it might be good time to check out the answer?
Or should we close it as resolved and let you reopen if you find more problems / questions later?

Closing as this is not actionable by us at the momemnt. Feel free to reopen if/when you have more info/questions.

@Tratcher  RequestTransforms did the trick.  Since I feel this could be a general(ish) solution, I'm sharing here:
in Program.cs
  .AddTransformFactory<MyTransformFactory>();

in my backgroundservice that updates in-memory proxy config regularly:
    // about 100 "dead" sub-sites pulled from db
    routes.Add(new RouteConfig() {
      RouteId = "deadsite",
      Match = new RouteMatch() { Path = "deadsite/{**catch-all}" },
      ClusterId = "franchiseCluster"
    }.WithTransformRedirect("/new-route-from-db", permanent: true));

    _provider.Update(new ReadOnlyCollection<RouteConfig>(routes), Clusters);

in RedirectRequestTransform.cs

using Yarp.ReverseProxy.Transforms;
using Yarp.ReverseProxy.Transforms.Builder;

namespace MyReverseProxy.Transforms;

public class RedirectRequestTransform(string location, bool permanent) : RequestTransform
{
  public const string RedirectKey = "redirect";
  public const string PermanentKey = "permanent";

  public override ValueTask ApplyAsync(RequestTransformContext context)
  {
    context.HttpContext.Response.Redirect(location, permanent);
    return default;
  }
}

public class MyTransformFactory : ITransformFactory
{
  public bool Validate(TransformRouteValidationContext context, IReadOnlyDictionary<string, string> transformValues)
  {
    if (transformValues.TryGetValue(RedirectRequestTransform.RedirectKey, out var location) &&
        transformValues.TryGetValue(RedirectRequestTransform.PermanentKey, out var permanentString))
    {
      if (string.IsNullOrWhiteSpace(location)) context.Errors.Add(new ArgumentException("a non-empty redirect value (location) is required."));
      if (!bool.TryParse(permanentString, out bool permanent)) context.Errors.Add(new ArgumentException("permanent must be a valid bool"));
      return true;
    }
    return false;
  }

  public bool Build(TransformBuilderContext context, IReadOnlyDictionary<string, string> transformValues)
  {
    if (transformValues.TryGetValue(RedirectRequestTransform.RedirectKey, out var location) &&
        transformValues.TryGetValue(RedirectRequestTransform.PermanentKey, out var permanentString))
    {
      if (string.IsNullOrWhiteSpace(location)) throw new ArgumentException("a non-empty redirect value (location) is required.");
      if (!bool.TryParse(permanentString, out bool permanent)) throw new ArgumentException("permanent must be a valid bool");
      context.RequestTransforms.Add(new RedirectRequestTransform(location, permanent));
      return true;
    }
    return false;
  }
}

public static class RouteConfigExtensions
{
  public static RouteConfig WithTransformRedirect(
    this RouteConfig route, string location, bool permanent
  )
  {
    if (location is null) throw new ArgumentNullException(nameof(location));
    if (string.IsNullOrWhiteSpace(location)) throw new InvalidOperationException("location must be at lease 1 character.");

    return route.WithTransform(transform =>
    {
      transform[RedirectRequestTransform.RedirectKey] = location;
      transform[RedirectRequestTransform.PermanentKey] = permanent.ToString();
    });
  }
}


Qt iOS project fails to link when using Declarative / QtQuick modules

Looks like we need to make sure to link in the "Security" framework for iOS if using the declarative modules with QtQuick
hunter_add_package(Qt COMPONENTS qtbase qtdeclarative qtsvg)
find_package(Qt5Widgets REQUIRED)
find_package(Qt5Qml REQUIRED)
find_package(Qt5Quick REQUIRED)

set(CMAKE_AUTOMOC ON)
set(CMAKE_AUTOUIC ON)
set(CMAKE_AUTORCC ON)

add_executable(example MACOSX_BUNDLE main.mm)
target_link_libraries(example Qt5::Widgets Qt5::Quick)

Ld _builds/ios-9-0/touch-events/qt-ios-examples.build/Debug-iphonesimulator/touch_events.build/Objects-normal/i386/touch_events normal i386
    cd /Users/jneidlinger/qt-ios-examples-master
    export IPHONEOS_DEPLOYMENT_TARGET=9.0
    export PATH="/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/usr/bin:/Applications/Xcode.app/Contents/Developer/usr/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin"
    /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/clang++ -arch i386 -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator9.0.sdk -L/Users/jneidlinger/qt-ios-examples-master/_builds/ios-9-0/touch-events/Debug-iphonesimulator -F/Users/jneidlinger/qt-ios-examples-master/_builds/ios-9-0/touch-events/Debug-iphonesimulator -filelist /Users/jneidlinger/qt-ios-examples-master/_builds/ios-9-0/touch-events/qt-ios-examples.build/Debug-iphonesimulator/touch_events.build/Objects-normal/i386/touch_events.LinkFileList -mios-simulator-version-min=9.0 -Xlinker -objc_abi_version -Xlinker 2 -Wl,-search_paths_first -Wl,-headerpad_max_install_names /Users/jneidlinger/.hunter/_Base/868408f/d527dd3/2d207c4/Install/lib/libQt5Widgets_debug.a /Users/jneidlinger/.hunter/_Base/868408f/d527dd3/2d207c4/Install/lib/libQt5Quick_debug.a -force_load /Users/jneidlinger/.hunter/_Base/868408f/d527dd3/2d207c4/Install/plugins/platforms/libqios_debug.a /Users/jneidlinger/.hunter/_Base/868408f/d527dd3/2d207c4/Install/lib/libQt5PlatformSupport_debug.a /Users/jneidlinger/.hunter/_Base/868408f/d527dd3/2d207c4/Install/lib/libqtpcre_debug.a /Users/jneidlinger/.hunter/_Base/868408f/d527dd3/2d207c4/Install/lib/libqtharfbuzzng_debug.a -lz -framework CoreGraphics -framework CoreText -framework CoreFoundation -framework QuartzCore -framework Foundation -framework OpenGLES -framework AssetsLibrary -framework UIKit /Users/jneidlinger/.hunter/_Base/868408f/d527dd3/2d207c4/Install/lib/libQt5Gui_debug.a /Users/jneidlinger/.hunter/_Base/868408f/d527dd3/2d207c4/Install/lib/libQt5Qml_debug.a /Users/jneidlinger/.hunter/_Base/868408f/d527dd3/2d207c4/Install/lib/libQt5Network_debug.a /Users/jneidlinger/.hunter/_Base/868408f/d527dd3/2d207c4/Install/lib/libQt5Core_debug.a -fobjc-link-runtime -Xlinker -dependency_info -Xlinker /Users/jneidlinger/qt-ios-examples-master/_builds/ios-9-0/touch-events/qt-ios-examples.build/Debug-iphonesimulator/touch_events.build/Objects-normal/i386/touch_events_dependency_info.dat -o /Users/jneidlinger/qt-ios-examples-master/_builds/ios-9-0/touch-events/qt-ios-examples.build/Debug-iphonesimulator/touch_events.build/Objects-normal/i386/touch_events

Undefined symbols for architecture i386:
  "_SSLClose", referenced from:
      QSslSocketBackendPrivate::disconnectFromHost() in libQt5Network_debug.a(qsslsocket_mac.o)
  "_SSLCopyPeerTrust", referenced from:
      QSslSocketBackendPrivate::verifyPeerTrust() in libQt5Network_debug.a(qsslsocket_mac.o)
  "_SSLCreateContext", referenced from:
      QSslSocketPrivate::ensureInitialized() in libQt5Network_debug.a(qsslsocket_mac.o)
      QSslSocketBackendPrivate::initSslContext() in libQt5Network_debug.a(qsslsocket_mac.o)
  "_SSLGetNegotiatedCipher", referenced from:
      QSslSocketBackendPrivate::sessionCipher() const in libQt5Network_debug.a(qsslsocket_mac.o)
  "_SSLGetNegotiatedProtocolVersion", referenced from:
      QSslSocketBackendPrivate::sessionProtocol() const in libQt5Network_debug.a(qsslsocket_mac.o)
  "_SSLGetNumberSupportedCiphers", referenced from:
      QSslSocketPrivate::ensureInitialized() in libQt5Network_debug.a(qsslsocket_mac.o)
  "_SSLGetSupportedCiphers", referenced from:
      QSslSocketPrivate::ensureInitialized() in libQt5Network_debug.a(qsslsocket_mac.o)
  "_SSLHandshake", referenced from:
      QSslSocketBackendPrivate::startHandshake() in libQt5Network_debug.a(qsslsocket_mac.o)
  "_SSLRead", referenced from:
      QSslSocketBackendPrivate::transmit() in libQt5Network_debug.a(qsslsocket_mac.o)
  "_SSLSetCertificate", referenced from:
      QSslSocketBackendPrivate::setSessionCertificate(QString&, QAbstractSocket::SocketError&) in libQt5Network_debug.a(qsslsocket_mac.o)
  "_SSLSetClientSideAuthenticate", referenced from:
      QSslSocketBackendPrivate::initSslContext() in libQt5Network_debug.a(qsslsocket_mac.o)
  "_SSLSetConnection", referenced from:
      QSslSocketBackendPrivate::initSslContext() in libQt5Network_debug.a(qsslsocket_mac.o)
  "_SSLSetIOFuncs", referenced from:
      QSslSocketBackendPrivate::initSslContext() in libQt5Network_debug.a(qsslsocket_mac.o)
  "_SSLSetPeerDomainName", referenced from:
      QSslSocketBackendPrivate::initSslContext() in libQt5Network_debug.a(qsslsocket_mac.o)
  "_SSLSetProtocolVersionMax", referenced from:
      QSslSocketBackendPrivate::setSessionProtocol() in libQt5Network_debug.a(qsslsocket_mac.o)
  "_SSLSetProtocolVersionMin", referenced from:
      QSslSocketBackendPrivate::setSessionProtocol() in libQt5Network_debug.a(qsslsocket_mac.o)
  "_SSLSetSessionOption", referenced from:
      QSslSocketBackendPrivate::initSslContext() in libQt5Network_debug.a(qsslsocket_mac.o)
  "_SSLWrite", referenced from:
      QSslSocketBackendPrivate::transmit() in libQt5Network_debug.a(qsslsocket_mac.o)
  "_SecCertificateCopyData", referenced from:
      QSslSocketBackendPrivate::verifyPeerTrust() in libQt5Network_debug.a(qsslsocket_mac.o)
  "_SecCertificateCreateWithData", referenced from:
      QSslSocketBackendPrivate::verifyPeerTrust() in libQt5Network_debug.a(qsslsocket_mac.o)
  "_SecPKCS12Import", referenced from:
      QSslSocketBackendPrivate::setSessionCertificate(QString&, QAbstractSocket::SocketError&) in libQt5Network_debug.a(qsslsocket_mac.o)
  "_SecTrustEvaluate", referenced from:
      QSslSocketBackendPrivate::verifyPeerTrust() in libQt5Network_debug.a(qsslsocket_mac.o)
  "_SecTrustGetCertificateAtIndex", referenced from:
      QSslSocketBackendPrivate::verifyPeerTrust() in libQt5Network_debug.a(qsslsocket_mac.o)
  "_SecTrustGetCertificateCount", referenced from:
      QSslSocketBackendPrivate::verifyPeerTrust() in libQt5Network_debug.a(qsslsocket_mac.o)
  "_SecTrustSetAnchorCertificates", referenced from:
      QSslSocketBackendPrivate::verifyPeerTrust() in libQt5Network_debug.a(qsslsocket_mac.o)
  "_SecTrustSetAnchorCertificatesOnly", referenced from:
      QSslSocketBackendPrivate::verifyPeerTrust() in libQt5Network_debug.a(qsslsocket_mac.o)
  "QTcpServerConnection::QTcpServerConnection()", referenced from:
      QQmlDebugServerThread::run() in libQt5Qml_debug.a(qqmldebugserver.o)
  "_kSecImportExportPassphrase", referenced from:
      QSslSocketBackendPrivate::setSessionCertificate(QString&, QAbstractSocket::SocketError&) in libQt5Network_debug.a(qsslsocket_mac.o)
  "_kSecImportItemCertChain", referenced from:
      QSslSocketBackendPrivate::setSessionCertificate(QString&, QAbstractSocket::SocketError&) in libQt5Network_debug.a(qsslsocket_mac.o)
  "_kSecImportItemIdentity", referenced from:
      QSslSocketBackendPrivate::setSessionCertificate(QString&, QAbstractSocket::SocketError&) in libQt5Network_debug.a(qsslsocket_mac.o)
ld: symbol(s) not found for architecture i386
clang: error: linker command failed with exit code 1 (use -v to see invocation)

Manually adding "-framework Security" resolves the issue. Do we need to have a special script like the Qt5Widgets_HunterPlugin.cmake to do this?

FYI I believe this all stems from the fact that we default QT to build statically - see comments.gmane.org/gmane.comp.lib.qt.user/17036
There is an issue with the QML debugger plugin not being automatically built into QtQml during static builds - really an upstream bug.
The work around for that will probably be to make sure to link in the qmldbg_tcp plugin if qt is built statically

Just for your information next cleanups can be applied:
hunter_add_package(Qt COMPONENTS qtbase qtdeclarative qtsvg)

qtbase is used by all components and not needed
qtsvg depends on qtdeclarative and hence not needed too

so this should be enough:
hunter_add_package(Qt COMPONENTS qtdeclarative)

Qt5::Qml target is not used explicitly. If some module need it Qt5::Qml will be linked automatically => can be removed:
find_package(Qt5Widgets REQUIRED) # find Qt5::Widgets
find_package(Qt5Quick REQUIRED) # find Qt5::Quick

Do we need to have a special script like the Qt5Widgets_HunterPlugin.cmake to do this

Yes, but not for Qt5Widgets.
"_SSLClose", referenced from:
      QSslSocketBackendPrivate::disconnectFromHost() in libQt5Network_debug.a(qsslsocket_mac.o)

mean we need to add lib/cmake/Qt5Network/Qt5Network_HunterPlugin.cmake

The work around for that will probably be to make sure to link in the qmldbg_tcp plugin if qt is built statically

I see Qt5::QTcpServerConnection target defined. I think since plugins are an optional requirement you have to add this manually. Also probably for the static build we need to have Q_IMPORT_PLUGIN defined. This is also possible to solve using Hunter, example: https://github.com/ruslo/hunter/blob/master/scripts/static_qt_DSServicePlugin.cpp
@jneidlinger Can you please create SSCCE example so I can test it?

@ruslo See here https://copy.com/Abbn88OHjkxkr2Ps
Should be very very simple.
Build with
cmake -H. -B_builds/ios-9-0 -GXcode -DCMAKE_TOOLCHAIN_FILE="${POLLY_ROOT}/ios-9-0-wo-armv7s.cmake" -DCMAKE_IOS_INSTALL_UNIVERSAL_LIBS=YES -DCMAKE_XCODE_ATTRIBUTE_ONLY_ACTIVE_ARCH=NO -DHUNTER_STATUS_DEBUG=ON

Open XCode proj, and build. You'll see the linker errors I described.
Manual workaround is to link in the "Security" framework, and to link in "plugins/qmltooling/qmldbg_tcp_debug.a" library

See here https://copy.com/Abbn88OHjkxkr2Ps

Thanks, this is perfect

Fixed: https://github.com/ruslo/hunter/releases/tag/v0.12.24
New example: https://github.com/ruslo/hunter/tree/develop/examples/qt-qml

Woohoo! Thanks @ruslo

:+1:


Update ldap.py for parse_result_attributes

This PR was opened in ldap.py to handle all parse_result_attributes operations, and PR 441 was opened to avoid overlapping some changes. Relevant changes will occur slowly and with frequent checks.

Only gmsa left. Can't test it yet. It can be review for now.

The errors I've encountered on ldap_results.py.
Important Note: Same error in ldap_results.py in the current main iirc and https://github.com/Pennyw0rth/NetExec/pull/441's ldap_results.py . This bug was fixed in https://github.com/Pennyw0rth/NetExec/pull/381/. After merging https://github.com/Pennyw0rth/NetExec/pull/441/and https://github.com/Pennyw0rth/NetExec/pull/381/, this PR should be checke and review with the new ldap_results.py.

check-if-admin (not only for this, also o other flags.), got utf-8 byte error.
"This fix addresses UnicodeDecodeError encountered while parsing LDAP attribute values that contain non-UTF-8 compatible byte sequences. The code first attempts to decode each attribute value as UTF-8, and if it fails (triggering UnicodeDecodeError or AttributeError), it falls back to hexadecimal encoding for unsupported byte sequences. This approach improves reliability and flexibility when handling mixed or non-UTF-8 data in LDAP responses."

After fix ldap_results.py, other bug-fixes on ldap.py

check-if-admin.
After parsing (with newest ldap_results.py), An error was occurring due to SID conversion.
"This fix resolves decoding issues encountered when processing the objectSid attribute in LDAP parsing. By encoding the SID data as Latin-1 (ISO-8859-1), we handle the byte-to-character conversion without altering byte values. This works because Latin-1 is a single-byte encoding that maps byte values directly to corresponding code points, preserving the original byte structure of non-UTF-8 data. Using latin1 ensures compatibility with the self.sid_to_str function, which requires a byte-accurate input, making it suitable for decoding SIDs that contain non-UTF-8-compatible sequences."

[Before fix]. Top SID value was coming from  self.sid_to_str (which is wrong), The second one comes from the currently running code.

[Fixed code]

query
After parsing (with newest ldap_results.py), some values come with non-display format.

[Before fix]. To display non-printable characters, such as ï3o®w{\F¥òLßö, in a readable format, need to convert these bytes to their hexadecimal representation (latin1). This approach preserves the data while avoiding unreadable symbols.

[Fixed code]

admin-count
After parsing (with newest ldap_results.py), The new output is not same with old one.

[Before Fix] All admin accounts and groups are displaying with the currently running code. But After parsing it does display only accounts.

[After Parsing]

[After Fix] In the old code, if there was no attributes value, an empty value was assigned and append was performed. Relevant sections for this were added to the new code. This correction was then made to all similar flags https://github.com/Pennyw0rth/NetExec/pull/471/commits/26391a3370c07340a9cc8f643585c4e02013266a.
sAMAccountName = item.get("sAMAccountName", "")
mustCommit = sAMAccountName is not None
userAccountControl = f"0x{int(item.get('userAccountControl', 0)):x}"
memberOf = str(item.get("memberOf", " "))
pwdLastSet = "<never>" if str(item.get("pwdLastSet", 0)) == "0" else str(datetime.fromtimestamp(self.getUnixTime(int(str(item.get("pwdLastSet", 0))))))
lastLogon = "<never>" if str(item.get("lastLogon", 0)) == "0" else str(datetime.fromtimestamp(self.getUnixTime(int(str(item.get("lastLogon", 0))))))


When reaching the bottom of the emoji list user is taken back to the top of the Flags section

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Expected Result:
When scrolling to the bottom, emoji list should stay on the same place.
Actual Result:
After reaching the bottom of the page if the user keeps scrolling the list is redirected back to the top of the flags section.
Action Performed:

Navigate to https://staging.expensify.cash/
Select any user and initiate chat
In the text input field select emoji
Search for "flag" in search field
Scroll to the end

Workaround:
No need, visual issue.
Platform:
Where is this issue occurring?
Web
iOS
Android
Desktop App
Mobile Web ✔
Version Number: 1.0.49-0
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/44479856/118901476-606f6e80-b8e1-11eb-8936-22c29239237c.mp4
Expensify/Expensify Issue URL:
View all open jobs on Upwork

Triggered auto assignment to @muttmuure (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

reproduced following steps adding engineering label

Triggered auto assignment to @chiragsalian (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

Issue reproducible during today's KI retests..

Triggered auto assignment to @NicMendonca (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

Posted to Upwork: https://www.upwork.com/jobs/~017cbd904460423141

Issue reproducible today during KI retests

Raised the price for this job to $500.

Proposal
What/where is the issue?

It's not a logical bug instead of a performance(didn't know any other term :rofl: ) issue. Here https://github.com/Expensify/Expensify.cash/blob/2345457326b7c35101b2704b9b9f135325bb7d7c/src/pages/home/report/EmojiPickerMenu/index.js#L338.
This onScroll updates the state on each frame. There will be many in the case of the scroll. Which is the culprit, it somehow causes the scroll bug.

Solution

I didn't see any action where we want to react to Scroll event instead we just want to save the scroll offset. Which could be simplified as

onScroll={e => this.currentScrollOffset = e.nativeEvent.contentOffset.y}

and then we can replace all the this.state.currentScrollOffset instances to this.currentScrollOffset. It will have the same effect(I tested it) as before but with state change.

**This fixes this bug. **

Posting Video Shortly.
Test performed

Using console check the scroll offset.
Performed arrow navigation multiple times in all directions.

Nice, good catch. Feel free to create a PR.
@NicMendonca feel free to accept @parasharrajat proposal on upwork.

@parasharrajat just sent the offer in Upwork - thanks!

@NicMendonca Just sent the PR in Github - thanks! :sweat_smile:

Issue reproducible today during KI retests

I will test it today.

@isagoico I am not able to reproduce it anymore. Could you please tell me :
the version of the app?
Platform?
any video? if possible.
Thanks.

@isagoico bumping @parasharrajat request ☝ Thanks!

Issue is not reproducible anymore, seems like it was fixed above 🎉

Thank you for letting us know.


fix changed state bug

if someone removes the dependencies instead of resolving, state is "stuck"
@alvarocavalcanti - please review
Not sure this is the best fix.
Scenario:

Create Dependency from PR 1 on PR 2
Check runs & fails as PR 2 isn't merged yet
Delete the comment that created the dependency from PR 1
Check will NOT update the status of PR, as there are no dependencies now...

This is related to #22


Refactor Use pydantic-settings for env vars

@ryansurf
Sorry for bombarding you with pull requests one after another!
I refactored the environment variable loading process (os.getenv()) and used pydantic-settings to make it type-safe!
Maybe you know, The benefits of pydantic-settings are:

You can define the expected type for each environment variable using pydantic-settings.  When the Python script runs, it automatically validates the loaded values against the specified types.  This helps catch errors in the environment variable values early on, preventing the process from proceeding with incorrect settings.

If the type is wrong, it'll throw an error at runtime.

With pydantic-settings, you can define all the environment variables in a centralized configuration class(settings.py). This makes it easier to manage and maintain the configuration settings in one place, improving code organization and readability.

Thanks to this, send_email.py now throws an error if the email address isn't set, for example.
This changed commit: https://github.com/ryansurf/cli-surf/pull/28/commits/c2f5bf51c8545c8ac5ae3ff51be9c55b16d725e0
I'd be stoked if you could merge this in!!

Also, I know this article is in Japanese, but I wanted to share a piece I wrote about pydantic-settings:)
https://qiita.com/inetcpl/items/b4146b9e8e1adad239d8

This is awesome. I didn't know about pydantic-settings but it looks great.
For send_email.py, will an error be thrown but everything runs as expected? I think that's what will happen, the email settings are optional
Nice job on this PR! Article was interesting too :D
Merging

Just to make sure, we can get rid of the .env file, right? Redoing the docs

@ryansurf
Thanks for merging this PR!
You're right, the email settings are optional, and when send_email.py runs, it will check if the values in .env conform to the expected types.
As long as send_email.py isn't executed, other processes like server.py and cli.py won't be affected.
I'm planning to add some test code later to verify that the settings in settings.py are correct.
Also, thanks for checking out my article! :D

Just to make sure, we can get rid of the .env file, right? Redoing the docs

Thanks for the follow-up question.
If I understand correctly, you're asking about the behavior when running the code without the .env file (i.e., when the .env file doesn't exist), right?

Exactly, the from my understanding the settings.py file replaces the .env file. Is that correct?

Exactly, from my understanding the settings.py file replaces the .env file. Is that correct?

No, that's not quite right!
The settings.py file is responsible for loading and validating the types of values from the .env file.
This means that the .env file is still necessary, and users will need to continue writing configuration values in the .env file, just like before.

# settings.py
class CommonSettings(BaseSettings):
    """
    Base class for defining common settings.
    model_config (SettingsConfigDict)Configuration dictionary
    for specifying the settings file and encoding.
    """

    model_config = SettingsConfigDict(
        env_file=f"{Path(__file__).parent.parent}/.env",
        env_file_encoding="utf-8",
        extra="ignore",
    )

The .env file is being loaded by the settings.py file through the line env_file=f"{Path(__file__).parent.parent}/.env".

Apologies for the lack of clarity in my explanation!
To summarize:

The .env file is necessary, and users will input their configuration information into the .env file.
The settings.py file reads the .env file and validates whether the values conform to the specified types. If there are any issues, it will output an error.
If there are no problems with the input values, the application can import settings.py to access the loaded environment variable values that have been "type-validated".

Sorry for the confusion😵💫

Please point out if I'm missing the point of your response, as I may not have understood the exact intent of your question.🙂↕

Ah, that makes sense. Sorry, I was being a bit lame, you answered what I was asking :thumbsup:


Adaptive color theme "Copy theme URL" function results in a broken link

Description
When using the Adaptive color theme tool, by opening the Share dropdown and clicking Copy theme URL, a URL is copied to the clipboard. That URL contains all of the necessary information to load the color theme into the Leonardo Web Tool in a new tab or browser. However, the result of pasting that URL into a new tab is an indefinite loading screen.
Steps to reproduce

Go to http://leonardocolor.io
Click on Adaptive color theme
Click on the Share button at the top right
Click Copy theme URL
Open a new tab and paste the URL into the address bar
Observe an indefinite loading screen

Expected behavior
When I paste the copied URL into a new tab or browser window, I expect the resulting web page to be the same as the adaptive color theme I copied from.
Screenshots

Environment

Chrome version 111.0.5563.146 on MacOS Ventura 13.2.1
Safari version 16.3 (186<IP_ADDRESS>.6) on MacOS Ventura 13.2.1

Additional context
This isn't happening on every color set, but it has happened to me a handful of times. I have a feeling that some of the URL variables are causing this. Whether it's how they're getting translated into the URL, or read back into the app after pasting the URL.
Here is one that fails for reference:
https://leonardocolor.io/theme.html?name=Untitled&config=%7B%22baseScale%22%3A%22Sky%22%2C%22colorScales%22%3A%5B%7B%22name%22%3A%22Sky%22%2C%22colorKeys%22%3A%5B%22%23bbd2ec%22%2C%22%23837afa%22%5D%2C%22colorspace%22%3A%22OKLCH%22%2C%22ratios%22%3A%5B%221.025%22%2C%221.125%22%2C%221.25%22%2C%221.75%22%2C%222.5%22%2C%223.25%22%2C%224.5%22%2C%227%22%2C%229.25%22%2C%2213%22%2C%2218%22%5D%2C%22smooth%22%3Afalse%7D%5D%2C%22lightness%22%3A100%2C%22contrast%22%3A1%2C%22saturation%22%3A100%2C%22formula%22%3A%22wcag2%22%7D

I've also encountered this issue.
Here is a link that doesn't load for me:
https://leonardocolor.io/theme.html?name=Prism+-+Periwinkle&config={"baseScale"%3A"blue"%2C"colorScales"%3A[{"name"%3A"grey"%2C"colorKeys"%3A["%23333333"]%2C"colorspace"%3A"HSLuv"%2C"ratios"%3A["9"%2C"35"%2C"70"]%2C"smooth"%3Atrue}%2C{"name"%3A"red"%2C"colorKeys"%3A["%23df2020"]%2C"colorspace"%3A"OKLCH"%2C"ratios"%3A["9"%2C"35"%2C"70"]%2C"smooth"%3Afalse}%2C{"name"%3A"orange"%2C"colorKeys"%3A["%23ec8713"]%2C"colorspace"%3A"OKLCH"%2C"ratios"%3A["9"%2C"35"%2C"70"]%2C"smooth"%3Afalse}%2C{"name"%3A"yellow"%2C"colorKeys"%3A["%23f0c742"%2C"%23805100"]%2C"colorspace"%3A"HSLuv"%2C"ratios"%3A["9"%2C"35"%2C"70"]%2C"smooth"%3Afalse}%2C{"name"%3A"green"%2C"colorKeys"%3A["%2371c639"]%2C"colorspace"%3A"OKLCH"%2C"ratios"%3A["9"%2C"35"%2C"70"]%2C"smooth"%3Afalse}%2C{"name"%3A"mint"%2C"colorKeys"%3A["%233cdda7"]%2C"colorspace"%3A"OKLCH"%2C"ratios"%3A["9"%2C"35"%2C"70"]%2C"smooth"%3Afalse}%2C{"name"%3A"cyan"%2C"colorKeys"%3A["%233cd2dd"]%2C"colorspace"%3A"OKLCH"%2C"ratios"%3A["9"%2C"35"%2C"70"]%2C"smooth"%3Afalse}%2C{"name"%3A"blue"%2C"colorKeys"%3A["%232059df"]%2C"colorspace"%3A"OKLCH"%2C"ratios"%3A["9"%2C"35"%2C"70"]%2C"smooth"%3Afalse}%2C{"name"%3A"purple"%2C"colorKeys"%3A["%237513ec"]%2C"colorspace"%3A"OKLCH"%2C"ratios"%3A["9"%2C"35"%2C"70"]%2C"smooth"%3Afalse}%2C{"name"%3A"pink"%2C"colorKeys"%3A["%23d926a0"]%2C"colorspace"%3A"OKLCH"%2C"ratios"%3A["9"%2C"35"%2C"70"]%2C"smooth"%3Afalse}]%2C"lightness"%3A96%2C"contrast"%3A0.95%2C"saturation"%3A95%2C"formula"%3A"wcag3"}

@jesselcampbell I think it's the OKLCH colorspace that's causing the issue. If I change your URL data to RGB, it loads:
This URL uses this data:
config = {
  baseScale: "Sky",
  colorScales: [
    {
      name: "Sky",
      colorKeys: ["#bbd2ec", "#837afa"],
      colorspace: "RGB",
      ratios: [
        "1.025",
        "1.125",
        "1.25",
        "1.75",
        "2.5",
        "3.25",
        "4.5",
        "7",
        "9.25",
        "13",
        "18",
      ],
      smooth: false,
    },
  ],
  lightness: 100,
  contrast: 1,
  saturation: 100,
  formula: "wcag2",
};

@damiankorcz same issue with your url - leonardo is converting the color to an RGB colorspace in the config (as a hex color), but it isn't changing the colorspace to RGB, so it's failing to trying to load an RGB color in a different color space. If I convert your data to use the RGB colorspace, it loads.
Here's that updated URL using this data:
config = {
  baseScale: "blue",
  colorScales: [
    {
      name: "grey",
      colorKeys: ["#333333"],
      colorspace: "RGB",
      ratios: ["9", "35", "70"],
      smooth: true,
    },
    {
      name: "red",
      colorKeys: ["#df2020"],
      colorspace: "RGB",
      ratios: ["9", "35", "70"],
      smooth: false,
    },
    {
      name: "orange",
      colorKeys: ["#ec8713"],
      colorspace: "RGB",
      ratios: ["9", "35", "70"],
      smooth: false,
    },
    {
      name: "yellow",
      colorKeys: ["#f0c742", "#805100"],
      colorspace: "RGB",
      ratios: ["9", "35", "70"],
      smooth: false,
    },
    {
      name: "green",
      colorKeys: ["#71c639"],
      colorspace: "RGB",
      ratios: ["9", "35", "70"],
      smooth: false,
    },
    {
      name: "mint",
      colorKeys: ["#3cdda7"],
      colorspace: "RGB",
      ratios: ["9", "35", "70"],
      smooth: false,
    },
    {
      name: "cyan",
      colorKeys: ["#3cd2dd"],
      colorspace: "RGB",
      ratios: ["9", "35", "70"],
      smooth: false,
    },
    {
      name: "blue",
      colorKeys: ["#2059df"],
      colorspace: "RGB",
      ratios: ["9", "35", "70"],
      smooth: false,
    },
    {
      name: "purple",
      colorKeys: ["#7513ec"],
      colorspace: "RGB",
      ratios: ["9", "35", "70"],
      smooth: false,
    },
    {
      name: "pink",
      colorKeys: ["#d926a0"],
      colorspace: "RGB",
      ratios: ["9", "35", "70"],
      smooth: false,
    },
  ],
  lightness: 96,
  contrast: 0.95,
  saturation: 95,
  formula: "wcag3",
};

Hello. I have a similar problem, this is a link to my Leonardo theme
https://leonardocolor.io/theme.html?name=Untitled&config={"baseScale"%3A"Gray"%2C"colorScales"%3A[{"name"%3A"Gray"%2C"colorKeys"%3A["%231e1e3c"]%2C"colorspace"%3A"HSV"%2C"ratios"%3A["1.13"%2C"1.31"%2C"1.5"%2C"2.31"%2C"3.59"%2C"8.15"%2C"9.7"%2C"16.08"]%2C"smooth"%3Atrue}%2C{"name"%3A"Primary"%2C"colorKeys"%3A["%234a44fd"]%2C"colorspace"%3A"OKLCH"%2C"ratios"%3A["1.13"%2C"1.31"%2C"1.5"%2C"2.31"%2C"3.59"%2C"8.15"%2C"9.7"%2C"16.08"]%2C"smooth"%3Afalse}]%2C"lightness"%3A100%2C"contrast"%3A1%2C"saturation"%3A100%2C"formula"%3A"wcag2"}

@dmthrill, that might have been related to a separate issue, #242 which I fixed this morning. Could you recheck it? Trying your URL now, I'm seeing this:


src-sheet attribute has confusing value when Drive folder is grabbed.

Steps to Recreate
I have been following the Documentation.
I have successfully grabbed a folder of GSheets.
See my notes here.
What occurs?
In the resulting Xml that is streamed to the STD_OUT
I note that the Inventory src-sheet attribute is actually defining a Google Folder and not a sheet.
What is expected
Either

the src-sheet attribute should be renamed to reflect that it is the 'target url' for the command that got the xml (maybe src-url ), or
the src-sheet should actually specify a Google Sheet url and not a Folder url. Thus making the value match the name.

We will have to have a think about this to see what is better.

@zoka123
I have been testing this and I am incredibly pleased. I followed your technical writing of the documentation very easily and got it working almost immediately. So, thanks.
I have spotted one bug, quite minor.

I've added PR with updates
https://github.com/forikal-uk/gsheet-to-xml/pull/9


Error fetching scenes since upgrading to v4

When calling api.scenes.getAll() on version 4.0.4 (have also tested downgrading to all other v4 versions) I get the following error message:

Error: The number of entries for the list, "0" is less than required minimum of 1
at ListType.getValue (C:\users\jack\desktop\hue-project\node_modules\node-hue-api\lib\types\ListType.js:48:13)
at LightScene.setAttributeValue (C:\users\jack\desktop\hue-project\node_modules\node-hue-api\lib\model\BridgeObject.js:46:48)
at C:\users\jack\desktop\hue-project\node_modules\node-hue-api\lib\model\BridgeObject.js:133:16
at Array.forEach ()
at LightScene._populate (C:\users\jack\desktop\hue-project\node_modules\node-hue-api\lib\model\BridgeObject.js:131:25)
at Object.module.exports.createFromBridge (C:\users\jack\desktop\hue-project\node_modules\node-hue-api\lib\model\index.js:306:12)
at C:\users\jack\desktop\hue-project\node_modules\node-hue-api\lib\api\http\endpoints\scenes.js:76:25
at Array.forEach ()
at buildScenesResult (C:\users\jack\desktop\hue-project\node_modules\node-hue-api\lib\api\http\endpoints\scenes.js:71:23)
at C:\users\jack\desktop\hue-project\node_modules\node-hue-api\lib\api\http\Transport.js:81:39

I thought that it could mean there aren't any scenes available but on v3 everything was working perfectly. Any suggestions?
Cheers
Jack

I just came across this (still happens in the v5 beta), and I'm pretty sure I found the root cause - this happens when fetching scenes, when there's a room that has no scenes.


[SPARK-5003][SQL]cast support date data type

enable cast to support date data type
such as :
select * from tableX where dateFiled > cast('2014-12-30' as date)

lgtm

Can one of the admins verify this patch?

i think this issue have be resolved, could you test it and close this?

yeah，thank you for reminding me.


add downstream tests in upstream packages

https://github.com/CliMA/ClimaCoupler.jl/pull/1033 introduces a test/runtest.jl file that can be run from upstream packages to determine downstream compatibility. Now we need to use it from various upstream packages.
Packages to update

ClimaAtmos https://github.com/CliMA/ClimaAtmos.jl/pull/3393
ClimaLand https://github.com/CliMA/ClimaLand.jl/pull/875
ClimaUtilities https://github.com/CliMA/ClimaUtilities.jl/pull/121
ClimaDiagnostics https://github.com/CliMA/ClimaDiagnostics.jl/pull/91
SurfaceFluxes https://github.com/CliMA/SurfaceFluxes.jl/pull/179
Thermodynamics https://github.com/CliMA/Thermodynamics.jl/pull/231
ClimaCore https://github.com/CliMA/ClimaCore.jl/pull/2056
RRTMGP https://github.com/CliMA/RRTMGP.jl/pull/558

the downstream AMIP test has been added to all of the listed packages


Fixed crashing on discord restart

There was a bug that caused discord to crash on restart if you were already in a voice channel.
This was caused if you have the "ShowHiddenChannels" plugin enabled and you are already in a voice channel on restart.
I have fixed that bug.

Thanks for catching that!


update MACOSX_DEPLOYMENT_TARGET to 10.16 to support building Python chip-device-ctrl on Bigsur

Building Python chip-device-ctrl on Bigsur generates whl file with old OS version and might throw error ERROR: chip-0.0-cp38-cp38-macosx_11_0_x86_64.whl is not a supported wheel on this platform.
Proposed Solution
update MACOSX_DEPLOYMENT_TARGET to 10.16 to support building Python chip-device-ctrl on Bigsur until the https://github.com/pypa/packaging/pull/319 resolved

Looks like the referenced PR is already in. Closing.


Rename GitHub test reporter to GitHub interactor

Since the module does more than just reporting test result, we should have a name that correctly represents the module :)

@FWDekker Yes, it was a test.


Migrate the macOS app away from the legacy Fastlane certificate management

Implemented in #244, and documentation updated in #259.


Remove dependency on tools in release bundles.

This also changes swfm player to use closure compiled versions of the bundles.
While doing the refactoring, I noticed that the space twits example was stale and I removed it. Sorry for the sloppy PR, I could clean it up.
r? @yurydelendik

/botio test

Thank you.


Right Rotation on Array

No description provided.

Thank you so much for your contribution it was good to go and do provide a detailed description of what type of contribution you've done and put your GitHub username either at the top or end of the code


Making regulate relationship transitive

Perhaps because this was out of sight on JIRA, it never received a response.
@cmungall I am resurrecting for your reading and discussing pleasure.  :bowtie:
The original issue and comments can be found at http://jira.geneontology.org/browse/GO-904
If there is no urgency or need to address at this time, please comment and close.
=====
_Submitted on Sunday, July 26, 2015 - 13:25
Submitted by anonymous user: --IP goes here--
Submitted values are:
Name: E. Tapia
E-mail: --email-goes-here--
Subject: Ontology (e.g. structure, relations, subsets/slims...)
Comment or query:
Dear all,
I have a doubt with the regulate (R) relationship. May it be possible that a
chain R.R exist within the BP ontology or any other?
In such a case, being R non transitive, may it be correct to transverse de
DAG to the root node and propagate annotations using the True Path Rule?
If not, does it mean that DAGs in GO never contain non-transitive paths to
the root?
Best regards,
--signature--
The results of this submission may be viewed at:
http://geneontology.org/node/899/submission/635_

The regulates relation is transitive. The characteristics of different relations can be determined unambiguously by looking at the OWL edition of GO. See http://geneontology.org/page/download-ontology
To find all genes that are involved in or regulate a process P or function F, you can traverse over any number of or combination of part-of, regulates (incl pos and negative) relations

Brilliant! Thanks for adding a response here. :bowtie:

Hi Chris,
Is that true? I thought it was transitive over BP, but not MF
For example you could not propagate  annotations to
"protein kinase regulator activity" to "protein kinase activity"
http://www.ebi.ac.uk/QuickGO/GTerm?id=GO:0019887#term=ancchart
?

Nothing to do with annotation propagation.  A relation is transitive if you can collapse down a chain and still get a true statement  In this case.
A regulates B regulates C regulates D  => A regulates D
There are no limits to the length of a chain.
On 21 Jun 2016, at 08:34, Val Wood<EMAIL_ADDRESS>wrote:

Hi Chris,
Is that true? I thought it was transitive over BP, but not MF
For example you could not propagate annotations to
"protein kinase regulator activity" to "protein kinase activity"
http://www.ebi.ac.uk/QuickGO/GTerm?id=GO:0019887#term=ancchart
?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

OK, I always get confused between this and "transitivity".  Ignore.

Actually though, in the original ticket, the user is talking about "propagating annotation",
http://jira.geneontology.org/browse/GO-904
and Chris seems to be referring to this above?

There are two separate issue:

When is it safe to propagate annotations over regulates relationships (the answer should be the same whether or not regulates is transitive).
Is regulates transitive?  The relation in RO never used to be set to transitive, but it is now.

Perhaps the confusion comes from assuming that it is safe to propagate annotations over all relationships in GO.  This is definitely not the case, although by convention we've tended to keep the relationship types it definitely is not safe to propagate over in go-plus.owl.  I've never been entirely comfortable with propagation over regulates, but it seems to have become standard practise (at least for BP?).  Strictly, it breaks the declared relationship between Gene product and GO term:

GP-X involved in BP-Y  = GP enables an MF that is part_of (some) BP-Y
If we propagate over regulates then the relationship for propagated annotations becomes:

GP enables an MF  that regulates (some) BP-Y


Set the result according to the number of decimals

This PR is related to https://github.com/swiss-knife-xyz/swiss-knife/issues/5

Thanks for taking the initiative and deciding to contribute to this project!

Made an update, kindly have a look.

Made an update, kindly have a look.

Had to resort to the previous method, because (1 / token1PerToken0InDecimals) had to be limited to the desired decimal values.

Had to resort to the previous method, because (1 / token1PerToken0InDecimals) had to be limited to the desired decimal values.

I think that's better, that way you can only ensure that it is only changed on the display instead of actual calculations.


Check for Java and Go compilation errors when building a project, for further compile error code repairing task

Is there a specific reason why we use the wording "mistakes"? To me it looks like more about "compilation errors".

Is there a specific reason why we use the wording "mistakes"? To me it looks like more about "compilation errors".

@ahumenberger Markus suggested the name "mistakes" for the testdata repository, so I used that name for consistency.


Code compatible with Tensorflow 1.0.0

I've updated the Tensorflow code so that it works on Tensorflow 1.0.0.
I also fixed some typos, added a fresh requirements.txt and updated deprecated Keras and Tensorflow code. I committed these changes separately so you could look through the git history.

@lstmemery Thank you for the contribution!
I've merged everything, except the commit with "requirements.txt". Some of the fields seems to be redundant there.


fix-new-chat

<If necessary, assign reviewers that know the area or changes well. Feel free to tag any additional reviewers you see fit.>
Details

Fixed Issues

Fixes GH_LINK
Tests

Tested On

[ ] Web
[ ] Mobile Web
[ ] Desktop
[ ] iOS
[ ] Android

Screenshots

Web

Mobile Web

Desktop

iOS

Android

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

Hey @johndev0711 can you please fill in the PR template? You need to link the GH issue that your PR is for as well as provide instructions on how to test your PR and screenshots from each of the platforms listed to show that it works on each platform. Thanks!

Hey @johndev0711 can you please fill in the PR template? You need to link the GH issue that your PR is for as well as provide instructions on how to test your PR and screenshots from each of the platforms listed to show that it works on each platform. Thanks!


”Fața Motrului”, ”Poșta Veche”, ”Satu Mare”, ”Târsa” are listed in the drop-down menu

Description:
When user writes ”Fața Motrului”, ”Poșta Veche”, ”Satu Mare”, ”Târsa” the locations appear in the drop-down menu next to the comune and the county which is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Fața Motrului"

Expected results
The location appears in the drop-down menu as "Sat Fața Motrului, MEHEDINȚI (Stângăceaua)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Poșta Veche"

Expected results
The location appears in the drop-down menu as "Sat Poșta Veche, MEHEDINȚI (Stângăceaua)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Write in the search bar "Satu Mare"

Expected results
The location appears in the drop-down menu as "Sat Satu Mare, MEHEDINȚI (Stângăceaua)".

Step 6
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 7
Write in the search bar "Târsa"

Expected results
The location appears in the drop-down menu as "Sat Târsa, MEHEDINȚI (Stângăceaua)".


(BKR-940) Update package testing to puppet 4.8.0

Wouldn't this be nicer if we had these values parameter-ized so that we could control them from the job rather than needing code changes? Below is one simple idea of how we might do this..
:version        => ENV['BEAKER_PUPPET_VERSION'] || '4.8.0',

What do you think?

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/3369/

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/3370/

@ferglor if you merge this and rebase your PR, you should be good to go.


Typo fix (README.md file)

Hi,
In the bottom part of the README.md file, the section "Contributing", the typos are suggested to be fixed in this sentence:

"If someone has a problem or suggestions, you can submit new issues or new pull requests"

add a word 'submit' (or 'open')
fix a typo in a word 'requests' in the link to pull requests
Thank you

Thank you Lev. New release v0.1.10 fixed it


Changes and fixes for time series classification

Fixes #1681 #3095 #3125
I'd recommend starting with evalml/utils/gen_utils.py

@ParthivNaresh FYI you have to add "Fixes" before each issue for github to automatically close them. So "Fixes #1681, Fixes #3095, ..." I saw the same thing happened with #3104


Define legend key not work with pieChart

When is used legend.key, he ought to set which my "key" for the legend. But this is not working with pieChart
    $scope.data = [
         {
            key: "One",
             y: 5
         }
    ];

    $scope.chart_options = {
        chart: {
              type: 'pieChart',
              height: 500,
              x: function(d){return d.key;},
              y: function(d){return d.y;},
              showLabels: true,
              legend: {
                 key: function(d) {return d.y; }
             }
        }
    }

I also tested it with other data
    $scope.data = [
        {
            key: "Long",
            y: 10,
            label: "teste"
        },
        {
            key: "Medium",
            y: 20,
            label: "teste - 2"
        },
        {
            key: "Clean",
            y: 20,
            label: "teste - 3"
        }
    ];
    $scope.chart_options = {
        chart: {
        ...
                legend: {
                        key: function(d) {return d.y; }
                }
        ...
        }
    }

Also not working :)
This is not working with pieChart, but with cumulativeLineChart, scatterChart and others, him works fine

I too am experiencing this issue.

Any update on this please?

for those who is still looking for quick solutions for pie chart legend, i found in the source code that the legend for the pie chart is simply using the pie.x(), so for now I  am just doing
x: function(d){return d.whatever;} and then the legend will just show the value for x


rexray volumes with no name are shown on get-volume

Would it be a good idea to not retrieve volumes that do not have names? For instance "".
Other idea is to have a flag that retrieves everything, including volumes with "" no names.

I can see this related to https://github.com/emccode/rexray/issues/28.  Yes, since we rely on the name of a volume as its uniqueness, something like a blank name shouldn't be valid.  Different underlying storage platforms may or may not allow blank names.  In the case of EC2 it defaults to blank, which misaligns to the uniqueness RR relies on in the name.  I think the EC2 driver can address this.


Fix: Possible function name confusion in gonum/stat/roc_example_test.go #176

Thanks for merging it in.

Thanks.
Two things:

the commit message should start with the path to the package being modified: 'stat: blah blah'
could you rebase on top of 'origin/master' so as to get a nicer, linear, git history? (If you don't know how to do that let me know and I'll guide you)

Thanks again.

To add to @sbinet's comment. When adding a person to A+C, please say the name in the commit message: "A+C: add Sam Zaydel"

It seems that you are already in A+C, so when you rebase onto master, this can also be interactively rebased to include only the final commit here.

Sounds good, will take care of it shortly.
On Sat, Aug 5, 2017 at 4:04 PM, Dan Kortschak<EMAIL_ADDRESS>wrote:

It seems that you are already in A+C, so when you rebase onto master, this
can also be interactively rebased to include only the final commit here.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gonum/gonum/pull/177#issuecomment-320475265, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAPOUBrXpCQQPlCFnDsoJG-elUzhnP-dks5sVPT4gaJpZM4OueBR
.

--
We are prone to overestimate how much we understand about the world and to
underestimate the role of chance in events.
Please feel free to connect with me on LinkedIn.
http://www.linkedin.com/in/samzaydel

Abandoned in favour of #177.


[5.5] Allow graceful handling of SIGTERM in queue workers

I would like to be able to gracefully shutdown my application when sending a SIGTERM to a running queue daemon.
The current behavior traps SIGTERM signals and waits for the queue worker to process the current message, however it then treats the signal as a SIGKILL and does not gracefully shutdown.

What else would you be doing when catching this event?

Closing connections to my queueing server.

Can you show an example of that though?

I've got some jobs that override the pcntl_signal stuff to add custom timeout handling. (I kill off external processes that my job started which I presume has hung when the timeout occurs.)
It's reasonable that we get a WorkerStopping event when the worker stops, but is it possible to also add a JobTimedOut event with a reference to the job, just as the JobProcessing/JobProcessed/JobFailed events?
On the other hand, that posix_kill(getmypid(), SIGKILL); is meant to really really hard kill our process without executing any registered shutdown handlers... yet we're now talking about adding functionality to run things on timeouts and the following shutdown...

@taylorotwell thanks for merging!  Here is an example use case (pseudo code):
Queue::stopping(function () {
        Queue::connection('queue-connection')->close();
});

@sisve I agree that we shouldn't be adding logic to a method intended to SIGKILL, however the current behavior traps SIGTERM and runs posix_kill(getmypid(), SIGKILL);, which is improper handling of SIGTERM.  It would be ideal if sending a SIGTERM resulted in a call to $this->stop() as opposed to $this->kill(), but this would require a small refactor of this class.


Auto download

Auto-download transcriptions will be a great feature.

Currently, transcripts auto download at the end of each meeting. Please explain what do you actually mean by "Auto download".

it wasnt downloading , i check again and now it works. Apparently was aconflicting with one of the many puglins i was trying. This is just what i need it . Thanks!


Runtime config is not used for  in nuxt.config.js

Versions

nuxt: v2.14.0
node: 12.18.0

Reproduction

Create a new nuxt.config.js
Update nuxt.config.js to have head attribute that is trying to use runtime config

export default {
  mode: "universal",
  target: "static",
  publicRuntimeConfig: {
    GOOGLE_MAPS_API_KEY: process.env.GOOGLE_MAPS_API_KEY,
  },
  head: {
    script: [
      {
        src: `https://maps.googleapis.com/maps/api/js?key=${process.env.GOOGLE_MAPS_API_KEY}&libraries=places`,
        defer: true,
      },
    ],
  },

Additional Details

We're encountering this issue as we're trying to adopt the new runtime configuration. There is a long discussion in #5100 about how many `nuxt.config.js` should be able to be built once as a context.
There is also a similar problem in the auth module which can be found at nuxt-community/auth-module#713, but I felt that <head> should be supported by Nuxt out of the box.

Steps to reproduce

Set GOOGLE_MAPS_API_KEY environment variable to ABC (from .env file, or by bash export)
nuxt generate
Set GOOGLE_MAPS_API_KEY environment variable to XYZ (from .env file, or by bash export)
nuxt generate

What is Expected?
In step number 4:

Nuxt will detect that the build has been created before, so it will proceed with static pages generation
Generated static page should have XYZ API key

What is actually happening?
In step number 4:

Generated static page stays as ABC

+1

+1

+1

+1 Same use case here

+1

+1

+1
is there any workaround to load runtime config env to head property?

+1
no workaround yet?

+1
This would make builds much easier as many different assets are used across different deployments.


tour: [REPLACE WITH SHORT DESCRIPTION]

Context: https://tour.golang.org/basics/1
Change the title above to describe your issue and add your feedback here, including code if necessary

第一个提交

Empty issue, closing.


Allow https redirect on reading remote preferences

When I configure our reverse proxy to forward all http traffic to https, there is a JSON parsing error. (see here https://github.com/FAForever/downlords-faf-client/issues/1368).
The client needs to be able to follow a redirect.

Right now it's broken by design:
https://bugs.openjdk.java.net/browse/JDK-4620571?focusedCommentId=12159233&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-12159233

Ehm in the issue it say resolved 2002

Fixed in java 4

Ah and they agreed to not redirect between protocols...

So we have an issue... We should just change the url to https and once we have a way to enforce updates we can one day disable http completely


response: fix connect tunneling bug - v3

The response was emitting partial body data depending on how you fed the parser with inbound and outbound data chunks. It seems the intended behavior is to not emit body data if HTP_STREAM_TUNNEL will eventually be entered (please correct if mistaken).
The fix was to allow htp_connp_REQ_CONNECT_WAIT_RESPONSE to resume in order to enter the HTP_STREAM_TUNNEL or complete the request.
The tunneling transaction was also incomplete because the request side wasn't being finalized after entering HTP_STREAM_TUNNEL.
See test case for example.

Thanks @cccs-sadugas
Would you have a redmine ticket for this ? Suricata-verify test ?

Running suricata on the pcap created by the test, I see this diff
{                                                                       {
  "event_type": "http",                                                   "event_type": "http",
  "tx_id": 0,                                                             "tx_id": 0,
  "http": {                                                               "http": {
    "hostname": "www.ssllabs.com",                                          "hostname": "www.ssllabs.com",
    "http_port": 443,                                                       "http_port": 443,
    "url": "www.ssllabs.com:443",                                           "url": "www.ssllabs.com:443",
    "http_method": "CONNECT",                                               "http_method": "CONNECT",
    "protocol": "HTTP/1.0",                                                 "protocol": "HTTP/1.0",
    "status": 200,                                                          "status": 200,
    "length": 0                                                 |           "length": 15
  }                                                                       }
}                                                                       }
{                                                                       {
  "event_type": "flow",                                                   "event_type": "flow",
  "src_ip": "<IP_ADDRESS>",                                                  "src_ip": "<IP_ADDRESS>",
  "src_port": 57207,                                                      "src_port": 57207,
  "dest_ip": "<IP_ADDRESS>",                                                 "dest_ip": "<IP_ADDRESS>",
  "dest_port": 8001,                                                      "dest_port": 8001,
  "proto": "TCP",                                                         "proto": "TCP",
  "app_proto": "http",                                          |         "app_proto": "failed",
                                                                >         "app_proto_orig": "http",

I am not sure the fix is correct, "app_proto": "failed", looks better to me...
As for the length, the PR looks to improve it...
So, I guess I need to dig into this app_proto diff

This delays the callback HTPCallbackResponseComplete

Thanks for the initial feedback! I will try to create a redmine ticket, look at comments, and do more testing with Suricata in the next week or two.

Hello @simdugas will you work on this again ?

Hello @simdugas will you work on this again ?

Yes, as soon as I find spare time to do so. Sorry for the delay!


Repository mountpoint shows only one folder of two folders contained in backup

Output of restic version
restic 0.3.3
compiled with go1.7.4 on linux/amd64
Expected behavior
I run restic two times to backup two different folders and then mount the restic repository. What I expect is to see both folders in the mountpoint with the latest date (backup of folders foo and bar):
mnt/snapshots/2017-01-28T17:03:52+01:00/ contains foo (new repository assumed, folder foo was the first backuped folder)
mnt/snapshots/2017-01-28T17:03:58+01:00/ contains foo and bar
Actual behavior
mnt/snapshots/2017-01-28T17:03:52+01:00/ contains only foo
mnt/snapshots/2017-01-28T17:03:58+01:00/ contains only bar
Steps to reproduce the behavior
restic -r resticRepo/ init
restic -r resticRepo/ backup foo
restic -r resticRepo/ backup bar
restic -r resticRepo/ mount mnt/
ls mnt/snapshots/2017-01-28T17:03:52+01:00/
foo
ls mnt/snapshots/2017-01-28T17:03:58+01:00/
bar

Without much investigation, I'd say that this is completely expected. When you do backup foo, you can't expect that it contains any bars, under no circumstances.
If you want both folders to be contained in the same snapshot, why not backup them both at the same time, like this: restic backup foo bar?

Thanks for your fast reply! You are completely right I got my fallacy.


Where to place static images not related to a component?

Hello,
I have a large image library, and I want it to be accessible by all the components and then to link to them dynamically.
I tried adding test.jpg to the images folder in the root, and then inside a component trying to load an image from ../images/test.jpg but it wasnt accessible.
What is the correct method to link to static images/assets - things which don't need to become components, and as it might be hundreds/thousands of assets, I think an import for them would be inefficient.
Thoughts? Again, might be something for the documentation.

Nobody knows?

Maybe use Cloudinary? ---> https://cloudinary.com/

I dont mean hosting, I mean when the static site is built, where do I put the static images that are not part of a component, so that they can be referred by multiple components or views?
So I want something like
static/images/
js/components
js/services
js/containers
js/app.js
index.html
So my components can refer to images using something like  whether they are in a component, or a view etc.

If you do not want them hosted using a service like cloudinary and would rather have them stored within your project, I would maybe place them in an img folder inside your src folder. The img folder cannot be outside the src folder.

*app folder

Closing bc this isn't an issue with the boilerplate. Please ask help questions on StackOverflow or other help forums.


[Help] Need to add on folder in production build

There is a folder example in root need to add it in the production build how can I do that?
Tried using
  "files": ["dist-electron", "dist", "example"], in electron-builder.json but not working

"files": ["dist-electron", "dist", "example/**/*"]


Support for MySQL: PKMySQLSequence

[x] Do only one thing
[x] Non breaking API changes
[x] Tested

What did this pull request do?
Adds support for sharding using a MySQL sequence for the primary keys.
User Case Description
When using MySQL as DB this allows to shard a table.

LGTM


image-tests: don't use random names for CI artifacts

When using random names for artifacts like AWS snapshots, or Azure
images, it becomes hard to clean them up in case of CI failure. See this
issue for more details:
https://github.com/osbuild/osbuild-composer/issues/942
This PR introduces predictable names so that we can easily determine
which artifact belongs to which PR and therefore we can decide to wipe
all resources that are not needed any more.

@larskarlitski the PR is again ready for review. RHEL8.3 stages fail due to corrupted mock repo and I don't know what is wrong with codecov, but I can't see any errors in the UI.

The corrupt repo error seems to be gone from the latest log, but vhd-boot is failing. Is that unrelated?

This seems related, I'll check it.

The corrupt repo error seems to be gone from the latest log, but vhd-boot is failing. Is that unrelated?

This seems related, I'll check it.

I haven't found anything that could cause this yet. So I at least tried restarting the job. It passed this time, but failed with VMWare this time :(

For the record, VMWare was broken yesterday, this was an unrelated failure.


Multi-container pod tailing, duplicate tails fixes

Despite PR #17, the current main can still duplicate container tails. That PR also introduced a regression that causes only one container to be tailed from a pod with multiple containers.
Repro - note that you may need to update the stateful set multiple times as the duplicate tailing doesn't necessarily occur every time:

Start ktail:

[ktail (main-)]$ ktail -lapp ktail-test --timestamps

Apply the following stateful set:

[ktail (main-)]$ kubectl apply -f -
apiVersion: apps/v1
kind: StatefulSet
metadata:
  name: ktail-test
spec:
  selector:
    matchLabels:
      app: ktail-test
  serviceName: ktail-test
  replicas: 1
  template:
    metadata:
      labels:
        app: ktail-test
      annotations:
        foo: a
    spec:
      containers:
      - name: c1
        image: bash:latest
        command:
          - bash
          - -c
          - "x=1; while true; do echo $x; x=$((x+1)); sleep 1; done"
      - name: c2
        image: bash:latest
        command:
          - bash
          - -c
          - "x=1; while true; do echo $x; x=$((x+1)); sleep 1; done"
statefulset.apps/ktail-test created

Observe that only one of the containers is tailed:

[ktail (main-)]$ ktail -lapp ktail-test --timestamps
==> New container [ktail-test-0:c1]
==> New container [ktail-test-0:c2]
==> New container [ktail-test-0:c1]
2022-07-31 18:44:56.137848218 +0000 UTC ktail-test-0:c1 1
2022-07-31 18:44:57.139114374 +0000 UTC ktail-test-0:c1 2
2022-07-31 18:44:58.140316102 +0000 UTC ktail-test-0:c1 3
2022-07-31 18:44:59.141517492 +0000 UTC ktail-test-0:c1 4
2022-07-31 18:45:00.142660184 +0000 UTC ktail-test-0:c1 5
2022-07-31 18:45:01.143851672 +0000 UTC ktail-test-0:c1 6
2022-07-31 18:45:02.144969138 +0000 UTC ktail-test-0:c1 7
2022-07-31 18:45:03.146063988 +0000 UTC ktail-test-0:c1 8
2022-07-31 18:45:04.147207492 +0000 UTC ktail-test-0:c1 9
2022-07-31 18:45:05.148722608 +0000 UTC ktail-test-0:c1 10

Apply the stateful set again with an altered the foo annotation value:

[ktail (main-)]$ kubectl apply -f -
apiVersion: apps/v1
kind: StatefulSet
metadata:
  name: ktail-test
spec:
  selector:
    matchLabels:
      app: ktail-test
  serviceName: ktail-test
  replicas: 1
  template:
    metadata:
      labels:
        app: ktail-test
      annotations:
        foo: b
    spec:
      containers:
      - name: c1
        image: bash:latest
        command:
          - bash
          - -c
          - "x=1; while true; do echo $x; x=$((x+1)); sleep 1; done"
      - name: c2
        image: bash:latest
        command:
          - bash
          - -c
          - "x=1; while true; do echo $x; x=$((x+1)); sleep 1; done"
statefulset.apps/ktail-test configured

Observe two tails occurring for the one container that is being tailed.

2022-07-31 18:50:18.37143432 +0000 UTC ktail-test-0:c1 158
2022-07-31 18:50:19.3726387 +0000 UTC ktail-test-0:c1 159
2022-07-31 18:50:20.373825931 +0000 UTC ktail-test-0:c1 160
2022-07-31 18:50:21.375127732 +0000 UTC ktail-test-0:c1 161
==> Container left (waiting) [ktail-test-0:c1]
==> New container [ktail-test-0:c1]
==> New container [ktail-test-0:c2]
==> New container [ktail-test-0:c1]
2022-07-31 18:50:25.302927917 +0000 UTC ktail-test-0:c1 1
2022-07-31 18:50:26.304258933 +0000 UTC ktail-test-0:c1 2
2022-07-31 18:50:25.302927917 +0000 UTC ktail-test-0:c1 1
2022-07-31 18:50:26.304258933 +0000 UTC ktail-test-0:c1 2
2022-07-31 18:50:27.305345594 +0000 UTC ktail-test-0:c1 3
2022-07-31 18:50:27.305345594 +0000 UTC ktail-test-0:c1 3
2022-07-31 18:50:28.306654222 +0000 UTC ktail-test-0:c1 4
2022-07-31 18:50:28.306654222 +0000 UTC ktail-test-0:c1 4
2022-07-31 18:50:29.307982535 +0000 UTC ktail-test-0:c1 5
2022-07-31 18:50:29.307982535 +0000 UTC ktail-test-0:c1 5
2022-07-31 18:50:30.309096542 +0000 UTC ktail-test-0:c1 6
2022-07-31 18:50:30.309096542 +0000 UTC ktail-test-0:c1 6
2022-07-31 18:50:31.310262842 +0000 UTC ktail-test-0:c1 7
2022-07-31 18:50:31.310262842 +0000 UTC ktail-test-0:c1 7
2022-07-31 18:50:32.311563671 +0000 UTC ktail-test-0:c1 8
2022-07-31 18:50:32.311563671 +0000 UTC ktail-test-0:c1 8
2022-07-31 18:50:33.312661348 +0000 UTC ktail-test-0:c1 9
2022-07-31 18:50:33.312661348 +0000 UTC ktail-test-0:c1 9
2022-07-31 18:50:34.313957002 +0000 UTC ktail-test-0:c1 10
2022-07-31 18:50:34.313957002 +0000 UTC ktail-test-0:c1 10

I believe this PR fixes both of the above issues, as well as issue #18.
Context for the changes:

Prior to #17 the tailer key differentiated by container ID, but that is not always populated, and thus could result in duplicate container tails. As such, the container ID was removed from the tailer key. However, that meant that only one container per pod could be tailed.
https://github.com/atombender/ktail/commit/8623cabc1acb4b0ed52e51e0bd6390ab0d03a80d updates the tail key to differentiate by container name to allow for multiple containers per pod to be tailed without the issue of container ID sometimes not being populated. I believe using container name should be sufficient.

In my debugging, I observed that in the tailer runStream loop, EOF may be read before Stop is called and before current Run loop calls runStream. In that case we may get a duplicate tail as the controller may have initiated a new tail, but the existing runStream loop would continue indefinitely.
https://github.com/atombender/ktail/commit/f917806999776a1bccf6141a45d6ece60364a8e1 adds a check to the runStream loop to see if the controller has attempted to stop the given tailer.

Awesome, thank you! I'm on vacation right now, but will look as soon as I have a chance.


Request credentials for an os-climate bucket itr

Requesting credentials for github user: dunstanm
Requesting credentials for the bucket: itr
Email address to send a credential download link<EMAIL_ADDRESS>Request checklist:

[ ] User can establish connection with the bucket
[ ] User can read files from the bucket
[ ] User can write files to the bucket

hi,  I meant to apply as dunstanm,   but accidentally used an old sign-in.  I have deleted to avoid further confusion, and will re-raise for clarity


@subset should have implicit skipmissing = true, or provide an option to specify

This is an awesome package :) - I really like the 'rowwise by default' operations.
Currently @subset will fail for a simple :col == X predicate if :col contains missing values. I think that for most users it might be preferable to have skipmissing set to true by default. Or failing that to allow a skipmissing option. There is of course the third option of the user writing something like ismissing(:col) ? false : :col == X, but that makes things more verbose

you can use keywords with all macros, so @subset(df, :col == X; skipmissing = true) should work. It currently only works with the parentheses macro syntax because of the way parsing of keyword arguments works. And I've noticed that it doesn't currently work with Chain.jl when leaving out the first parameter, because the keyword arguments technically come before the positional arguments and I hadn't considered that at the time. I still want to add that to Chain.jl, though.

Ahh I see, thanks for the quick response! Yeah I was getting caught out when using the Chain.jl workflow. The following seems to work:
@subset(_, predicates...; kwargs...)

Macro keyword arguments work now in Chain.jl https://github.com/jkrumbiegel/Chain.jl/pull/31

Oh wow awesome thanks!


Finish AFR-0 part about The Aml Standard Runtime Library

There are many more chapters that need finishing, not just this particular one. Closing.


WP 5.1.1 available

There is a new WP version out there: 5.1.1 can we update the Wordpress Dockerfile?
We don't need it for every Patch, but for Minor versions we should be up to date.

@soophie Cheers for bringing this up! We'll take care of it as soon as there's a little bit of time (probably next Friday).

Might be enough to bump the version here, and test the result: https://github.com/sevenval/dockerfiles/blob/master/wordpress/Dockerfile#L1

I'm not sure if we want to update this file every time a new version is released.

I'm not sure if we want to update this file every time a new version is released.
@soophie What else do you propose?
I'll update to 5.1.1-php7.3-apache. S. https://hub.docker.com/_/wordpress

Is in PR #26 .

Why not use wordpress:5-php7.3-apache? So we get all Minor and Patches?

@soophie If that works for you, I'll change the PR.

I changed the tag to 5-php7.3-apache. Waiting for review...

Merged. Images should be available soon.


no std support?

I am really interested in using zbox as the default file system for my OS, but can it be used in no std environments? I took a look at a couple pieces of your code and saw nowhere a no std indicator. Your project claims to be embeddable, but it isn't truly embeddable if you can't use it in those environments. If this is a project GOAL, great, and I would be willing to help to the best of my ability. If not, oh well. Great work so far though! :)

You're right, no std hasn't been support yet, although I wish it could be implemented in the future but that is not my priority at this moment.As regarding the embeddable meaning, it is more on application side, that is, running inside application memory space to minimize data leak. It can be used in the true "embeddable" environment, but like you said it must enable no std first. If you could help on that, it will be really appreciated.Thanks.-------- Original Message --------Subject: [zboxfs/zbox] no std support? (#28)From: hydra007 To: zboxfs/zbox CC: Subscribed I am really interested in using zbox as the default file system for my OS, but can it be used in no std environments? I took a look at a couple pieces of your code and saw nowhere a no std indicator. Your project claims to be embeddable, but it isn't truly embeddable if you can't use it in those environments. If this is a project GOAL, great, and I would be willing to help to the best of my ability. If not, oh well. Great work so far though! :)
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or mute the thread.
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/zboxfs/zbox","title":"zboxfs/zbox","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/zboxfs/zbox"}},"updates":{"snippets":[{"icon":"DESCRIPTION","message":"no std support? (#28)"}],"action":{"name":"View Issue","url":"https://github.com/zboxfs/zbox/issues/28"}}}
{"@type":"MessageCard","@context":"http://schema.org/extensions","hideOriginalBody":"false","originator":"37567f93-e2a7-4e2a-ad37-a9160fc62647","title":"no std support? (#28)","sections":[{"text":"","activityTitle":"hydra007","activityImage":"https://avatars2.githubusercontent.com/u/33503692?s=160\u0026v=4","activitySubtitle":"@hydra007","facts":[{"name":"Repository: ","value":"zboxfs/zbox"},{"name":"Issue #: ","value":28}]}],"potentialAction":[{"name":"Add a comment","@type":"ActionCard","inputs":[{"isMultiLine":true,"@type":"TextInput","id":"IssueComment","isRequired":false}],"actions":[{"name":"Comment","@type":"HttpPOST","target":"https://api.github.com","body":"{"commandName":"IssueComment","repositoryFullName":"zboxfs/zbox","issueId":28,"IssueComment":"{{IssueComment.value}}"}"}]},{"name":"Close issue","@type":"HttpPOST","target":"https://api.github.com","body":"{"commandName":"IssueClose","repositoryFullName":"zboxfs/zbox","issueId":28}"},{"targets":[{"os":"default","uri":"https://github.com/zboxfs/zbox/issues/28"}],"@type":"OpenUri","name":"View on GitHub"},{"name":"Unsubscribe","@type":"HttpPOST","target":"https://api.github.com","body":"{"commandName":"MuteNotification","threadId":334374501}"}],"themeColor":"26292E"}


Use 's' audio track layout for mobile

Fixes

Fixes #668 by @obulat
Description

This PR uses 's' layout for screens smaller than md. This layout doesn't show the audio track, which is shown at the bottom of the screen instead.

Testing Instructions

Run the app and search for audio on a narrow screen. You should not see a large play button.
Checklist

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the default branch of the repository (main) or a parent feature branch.
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no visible errors.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

It works great and smoothly. I just noticed three things.

I'm not sure if we set 130% for the line-height of Description bold style, converted to rem value. It seems bigger, as the red area in the image below shows.
The spacing between title, author, and other meta-data info is inconsistent; blue area in the image attached. The mockup has a 8px spacing separating those three rows.
The spacing between CC icons should be 2px.

Thanks @panchovm I've made a new issue for those spacing issues here


JBPM-4646 - Case management - simplified use for case role assignments

@krisv here it comes...

+1


Check for format in specs.

We can add a check in specs for crystal tool format output. We can so:

spot bugs in formatter (e. g. #2325, #2326, #2327).
have a repository in a clean state while coding (it's annoying to revert  unrelated formatter fixes).
check PRs format easily.

It's a really good idea. Maybe we can add crystal tool format --check, which basically formats everything and returns 0 if no changes were made, and 1 otherwise (and prints an error). We can then be sure that our commits are always formatted, otherwise the build will break. And PR will also fail if they are not properly formatted.
What do you think?

I think we can implement this, but returns 2 when there is a internal formatter bug.


Provide custom tag instead of bumping automatically from file

Hello,
We are quite happy with your action, but would like to provide custom tags lower than current version to the action inputs. In our case we have two different versions for staging and prod and this is what we use to deploy hotfix on prod. Are you open for PR for this feature?

Hi, yes that would be awesome!
If I understand correctly you want to ability to modify the new version? If so maybe it's an idea to do it similar to pre-commit , that users can provide a pre-changelog-generation where they get the same info as pre-commit but where they are able to modify and return tag and version which will then be used for the changelog and tag.

Yeah, we want the ability to provide our custom tag instead of what preset has generated and use it later on as a tag and in changelog.
Sound reasonable with pre-changelog-generation. Thanks for quick reply

Thanks for the PR! This has been released in v3.5.0 and v3 has been updated


Code highlighting plugin

how to add syntax highlighting? In the assembly there is no plugin Code Snippet (https://ckeditor.com/cke4/addon/codesnippet). Is it possible to add it yourself?

+1

Closing because of inactivity


[Style] Fixing the UI of the homepage

What's the style idea?
We need to improve . as we enter the webpage it looks simple and not at all attractive . I want to add various hover effects new texts as well as icon and make same colours changes to make homepage attractive and user friendly .
please assign this issue to me @CodeXsubham

Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[X] I am a GSSoC'24 contributor.

Code of Conduct

[X] I agree to follow this project's "Code of Conduct".

I'm not  project admin @HarshitGupta1511  tag @CSEdgeOff

@CSEdgeOff please assign this task to me

@CSEdgeOff  I would like to work on this issue. It would be great if you assign this to me. I will do this in 1 or 2 day.
Thank you
Your sincerely,
Sarthak Sharma


Add Export Sarif Button to GUI

It would be nice to be able export the entire analysis report to a single HTML file, or other browser format, so that it can be passed to anyone and they can open it in any web browser and see the results without having to look through the json file or create a tool to parse the json file or have access to the original instance of the ASA where the report was created.
Option to select to export ASA report to an HTML
A clear and concise description of what you want to happen. E.g. I want to be able to run a command to see what runs are in the database.
Describe alternatives you've considered
The alternative is creating a tool to parse the json and display it in a useful visual format or zipping up the entire instance of the ASA where the report was created and attaching it where others can download it to see the report. Another alternative is to use the browser's Save As feature to save the current data in the page to an html file, the issue with this is you only get the data for the current page of the report that you are on so you would have to save a file for every page which is not very useful when there are 30+ pages of data in the report.
System Configuration (please complete the following information):

OS: Windows 11
OS Version: Windows 11 Enterprise version 21H2
Application Version: 2.3.299
CLI or GUI: both

Additional context
N/A

Hmm...I guess this can actually be handled by the web browser's Save As feature on the Report page of the MS ASA. It creates an Attack Surface Analyzer.html file and an associated Attack Surface Analyzer_files folder with the necessary data to re-open the report in any web browser. I think this will suffice for what I need.

ASA supports exporting the report as Sarif which has a number of view clients available (including the sarif web component: https://microsoft.github.io/sarif-web-component/)
Have you tried with the sarif format report?

I have not tried the sarif export, I wasn't aware of that. So is it only available from the command line?

Yes, I believe you can only export sarif from the command line currently. I can take a look at adding a sarif export button for the GUI as well. Thanks for the feedback.

Keeping this issue open to track potentially adding a sarif export button to the GUI.