Karpenter subnet selection preference

Version
Karpenter: v0.13.2
Kubernetes: v1.22.0
Expected Behavior
Karpenter should pick subnets in a round-robin balanced fashion.
Actual Behavior
We recently had to migrate our development VPC to bigger subnets since we were running out of IPs. We have our development EKS cluster and its nodes spread across all of 3 private subnets of the development VPC and for some reason Karpenter seems to be preferring 1b in order to spawn nodes, which creates a huge gap in IP allocation between subnets (1subnet per AZ).
The numbers look like these:
1a -> 3974
1b -> 3436
1c -> 3904

As you can understand there is a diff of ~500 IPs between subnets.
We have checked Karpenter controller logs and it indeed shows that it finds all of subnets but for some reason it chooses 1b most of the time, creating an unbalanced situation.
Is there a way to find if there is a bug behind this situation?
Let me know if you need me to provide you with more insights around our setup!
Steps to Reproduce the Problem
Check the specs below.
Resource Specs and Logs
apiVersion: karpenter.sh/v1alpha5
  kind: Provisioner
  metadata:
    name: spot-workers
  spec:
    requirements:
      - key: "node.kubernetes.io/instance-type"
        operator: In
        values: ["t2.xlarge", "t3.xlarge", "m5.xlarge"]
      - key: karpenter.sh/capacity-type
        operator: In
        values: ["spot"]
      - key: "kubernetes.io/arch"
        operator: In
        values: ["amd64"]
      - key: "topology.kubernetes.io/zone"
        operator: In
        values: ["eu-west-1a", "eu-west-1b", "eu-west-1c"]
    labels:
      managed-by: karpenter
      purpose: spot-workers
    provider:
      launchTemplate: spot-workers-lt
      subnetSelector:
        karpenter.sh/discovery: <our_cluster_name>
      tags:
        karpenter.sh/discovery: <our_cluster_name>
    ttlSecondsAfterEmpty: 30

related https://github.com/aws/karpenter/issues/1854

Karpenter uses constraints to figure out which nodes to launch. If AZs are not constrained, we will choose the cheapest instance by using the capacity optimized prioritized (by cost) strategy within the AWS fleet API. To add zonal constraints, you'll need to use Kubernetes scheduling mechanisms like topology spread constraints or node affinity https://karpenter.sh/v0.13.2/tasks/scheduling/#topology-spread

I went ahead and fetched all the pricing history for the last 2 months from AWS EC2 Spot Price History API, for the instance types we use.
My understanding is that in rare to no cases, eu-west-1b is cheaper than eu-west-1a, so I would expect Karpenter to be launching there based on your comment @ellistarn .
Please find a spreadsheet with all the info available here: https://docs.google.com/spreadsheets/d/1QpyQOcSUq_DhXZJnfxtEscGEmM0nAS6jYWB9bzvQFoM/edit?usp=sharing

Cost is secondary for Spot when the EC2 Fleet API chooses an instance pool. The first consideration is a proxy for the interruption rate of the instance. It's generally not valuable to run a spot instance at the absolutely lowest-price because it could be taken away very soon (maybe even a few minutes). That is why Karpenter uses the capacity-optimized-prioritized allocation strategy for Spot. First, it looks for the deepest capacity pools to find the instance pools with the lowest interruption rates, and then it will try to optimize for the cost between the lowest interruption rate pools.  The spot discount is usually deep enough for this to be the best trade-off.


Bug in Transformer.get_epoch_iterator

Here's a minimal example reproducing the bug:
import numpy

from fuel.datasets import IndexableDataset
from fuel.streams import DataStream
from fuel.schemes import SequentialScheme
from fuel.transformers import Mapping

def run(bug=True):
    constructor = DataStream.default_stream if bug else DataStream
    dataset = IndexableDataset(numpy.arange(100))
    dataset.default_transformers = ((Mapping, [lambda d: (2 * d[0],)], dict()),)
    data_stream = constructor(dataset, iteration_scheme=SequentialScheme(100, 10))
    iterator_1 = data_stream.get_epoch_iterator()
    iterator_2 = data_stream.get_epoch_iterator()
    for batch in iterator_2:
        pass
    next(iterator_1)

The problem lies in the fact that Transformer sets its child_epoch_iterator attribute when get_epoch_iterator is called. Even though the two iterators are different, the data stream they point to has the same child epoch iterator.
This is an issue that can arise e.g. in Blocks when someone re-uses the same data stream for monitoring and for training. The obvious solution is "don't ever use a data stream for more than one purpose", but it took me quite some time to figure out what the problem was when I was made aware of the crash. It would be good if we could find a better long-term solution to that.

The datastreams in Fuel are indeed stateful, but this is so by design. Perhaps we should raise a warning when get_epoch_iterator overrides a non-empty child_epoch_iterator?

How would you define non-empty? If next(child_epoch_iterator) does not return StopIteration?

This seems to be the only possible solution, although it looks sort of ugly.
In fact a data stream is something like a file, so maybe we should emulate the file behaviour at this point:
In [3]: s = StringIO('a\nb\nc\n\d')

In [4]: s
Out[4]: <StringIO.StringIO instance at 0x7efdeab5ac68>

In [5]: it = iter(s)

In [6]: next(it)
Out[6]: 'a\n'

In [7]: it2 = iter(s)

In [8]: next(it2)
Out[8]: 'b\n'

In [9]: next(it)
Out[9]: 'c\n'

that is not to reset the child_epoch_iterator. What do you think? This still makes the user take care of not reusing a data stream, though.

Sounds good. I'd document that thoroughly, though. How would you go about implementing that in practice?

After sleeping over it, it sounds not so good anymore. The main loop relies on the fact the get_epoch_iterator does reset the data stream. More specifically, that it switches the data stream to the next epoch, which so far for all available datasets is equivalent to calling reset (there is some capacity for different epochs in the core).
That being said, I think that this is a documentation issue, and no changes in the code should be done.


[Question]: Example of get_activities()

Question form pre-submit checklist.

[X] I have searched the existing issues to ensure there isn't already an issue about this question.
[X] My question has to do with the Python SDK and isn't a general question about the API. (If it is please open your issue here)
[X] My question isn't about how to do a specific algorithm or asking for trade advice (answers to these are outside the scope of this repo).

Question
There is no example in documentation about how to get stock activities in the new official SDK
In alpaca-trade-api version: 2.3.0 is as simple as: get_activities()
Please add an example in the docs and close this issue with the reference URL.
Thanks in advance.
Regards,

Agreed. Would be nice to have alpaca-py support account activities.


Trails don't update properly when changing indicators

Trails don't update properly when changing indicators
old data remains cached because it caches on selection

when unselect-select it's fine again

improvement possible: keep the trail in the new indicator space


⚠ Bofrost.be has degraded performance

In 40a4b18, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1820 ms

Resolved: Bofrost.be performance has improved in 6b06d07.


update README

Codecov Report

Merging #35 (3040de0) into main (69d81ac) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #35   +/-   ##
=======================================
  Coverage   86.71%   86.71%
=======================================
  Files          15       15
  Lines        2010     2010
=======================================
  Hits         1743     1743
  Misses        267      267

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Bug: Using Tab selection gets stuck resizing the sidebar

Description
If I use tab to navigate in Files it will select the sidebar width divider and get stuck there. Tab, Shift + Tab will resize it to be bigger.
Steps To Reproduce

Use Tab
Wait until sidebar is highlighted
See controls only make sidebar bigger

Requirements

Tab and shift tab should switch between elements and not resize.

Files Version
<IP_ADDRESS>
Windows Version
10.0.22621.2428
Log File
debug.log

Should we make the sidebar resizer not selectable using tab?

Should we make the sidebar resizer not selectable using tab?

It should be selectable so that you can resize the sidebar via keyboard, but it should also be possible to use the tab key to move to the next item.

@chingucoding fyi

e selectable so that you can resize the sidebar via keyboard, but it should also be possible to use the tab key to move to the next ite

Ok, so we should change the resize command to not be tab


missing args to main

Original issue: https://charm.cs.illinois.edu/redmine/issues/62

Add argument message to main in test and examples

Original date: 2013-03-03 08:10:03

All fixed - there were only 8 of them, but the scripting approach was more fun. They were conveniently regular - every single one was written precisely "entry main();", so the final sed once I found them was really easy. The finding them distinctly was a bit trickier.
Eric- please build for yourself to confirm, then close.

Original date: 2013-03-04 19:06:49

I think you missed a few.  These don't show up in a make test, but are part of the examples distribution.
grep -ri 'main(\s*)' examples
./charm++/hello/hello.ci:    entry main();
./charm++/barnes-charm/barnes.ci:    entry Main();
./charm++/state_space_searchengine/3SAT/soln-checker.py:def main():
./charm++/state_space_searchengine/3SAT/soln-checker.py:    main()
./charm++/cell/md/main.h:    ~Main();
./charm++/cell/md/main.C:Main::~Main() {
./charm++/f90charm/hello2D/hellof.f90:      SUBROUTINE f90charmmain()
./charm++/f90charm/hello/hellof.f90:      SUBROUTINE f90charmmain()
./charm++/f90charm/ldbDemo/ldbDemof.f90:      SUBROUTINE f90charmmain()
./multiphaseSharedArrays/simpleTestVarsize/testV.C:main()
./multiphaseSharedArrays/matmul/mm_sequential.C:int main()
./multiphaseSharedArrays/matmul/mm_sequential.C:main()
./multiphaseSharedArrays/matmul/sequential/mm2mkn.C:main()
./multiphaseSharedArrays/matmul/sequential/maxflops.c:main()
./multiphaseSharedArrays/matmul/sequential/mm2.C:main()
./multiphaseSharedArrays/matmul/sequential/mm1mkn.C:main()
./multiphaseSharedArrays/matmul/sequential/mm1.C:main()
./amr/cfd/cfdAMR.ci:   entry main();
./amr/jacobi2D/jacobi2DAMR.ci:   entry main();
./amr/jacobi2Dthesis/jacobi2DAMR.ci:   entry main();
./bigsim/tools/model_based_rewrite/interpolatelog.C:int main()
./bigsim/tools/rewritelog/interpolatelog.C:int main()
./bigsim/tools/changelog/changelog.C:int main()
./bigsim/tools/loadlog/loadlog.C:int main()
./bigsim/tools/buildlog/buildlog.C:int main() {
./bigsim/tools/rewritelog-old-format/interpolatelog.C:int main()
./pose/HiSim/Main/pgm.ci:    entry main();
./pose/SimBenchmark/pgm.ci:    entry main();
./pose/PHOLD/phold.ci:    entry main();
./pose/MiniNet/mininet.ci:    entry main();
./pose/LBSim/sched.ci:    entry main();
./pose/ASIM/pgm.ci:    entry main();
./pose/Ring/pgm.ci:    entry main();
./pose/PatternGen/pgm.ci:    entry main();
./pose/ChunkSim/pgm.ci:    entry main();
./converse/pingpong/pingpong.C:CmiStartFn mymain()
./converse/pingpong/pingpong_multipairs.C:CmiStartFn mymain()
./converse/pingpong_multi/pingpong.C:CmiStartFn mymain()
./collide/collidethread/hello.ci:    entry main();
./collide/collidecharm/hello.ci:    entry main();

Original date: 2013-03-04 23:26:27

You were a bit generous, since only the .ci files are relevant, but there are many good catches. Here's my result set:
$ grep -P -r -nH --include '*.ci' -i 'entry.*main\(\s*\);' examples/ tests/ | column -s : -t
examples/charm++/barnes-charm/barnes.ci                        28      entry Main();
examples/charm++/hello/hello.ci                                20      entry main();
examples/amr/jacobi2Dthesis/jacobi2DAMR.ci                     7      entry main();
examples/amr/cfd/cfdAMR.ci                                     9      entry main();
examples/amr/jacobi2D/jacobi2DAMR.ci                           5      entry main();
examples/pose/Ring/pgm.ci                                      6       entry main();
examples/pose/LBSim/sched.ci                                   5       entry main();
examples/pose/ASIM/pgm.ci                                      6       entry main();
examples/pose/SimBenchmark/pgm.ci                              6       entry main();
examples/pose/HiSim/Main/pgm.ci                                11      entry main();
examples/pose/MiniNet/mininet.ci                               6       entry main();
examples/pose/PHOLD/phold.ci                                   6       entry main();
examples/pose/PatternGen/pgm.ci                                6       entry main();
examples/pose/ChunkSim/pgm.ci                                  6       entry main();
examples/collide/collidethread/hello.ci                        6       entry main();
examples/collide/collidecharm/hello.ci                         6       entry main();
tests/charm++/delegation/pipelined-section-reduction/hello.ci  18      entry main();
tests/charm++/commSpeed/pgm.ci                                 6       entry main();
tests/charm++/load_balancing/lb_test/sim/sim.ci                19      entry main();
tests/charm++/broadcast/broadcast.ci                           6       entry main();

Original date: 2013-03-04 23:58:26

Everything turned up is fixed now.

Original date: 2013-03-05 00:12:01

A few more stragglers.
bohm`intellect:~/work/git/charm/net-linux-x86_64-prod$ grep -ri 'main\s*(\s*)' --include *.ci *
tests/charm++/penciltest/testpencil.ci:	entry main ();
tests/charm++/pmetest/testpme.ci:	entry main ();
tmp/libs/ck-libs/pmaf/pgm.ci:    entry main();
tmp/libs/ck-libs/TMRC2D/old_pgm.ci:    entry main();

Original date: 2013-03-05 23:12:08

The results of net-linux-x86_64 (charm *$)$ find . -name '*.ci' | while read i; do echo $i; bin/charmc $i; done now show nothing in this vein. In addition to the four you cite, I also covered examples/multiphaseSharedArrays/matmul/test.ci, which had an attribute that threw us both off:
     mainchare Test
     {
-        entry [threaded] Test();
+        entry [threaded] Test(CkArgMsg *);
         entry void done();
         entry void doneg();
     }


Add camera to vehicle (emulator)

As a user I want my vehicle in the emulator to have a camera attached to it so that I can get a video feed from the vehicle.
Acceptance Criteria:

[ ] The emulator vehicle has a working camera device attached to it.
[ ] The implementation is tested.

No longer a valid issue as there is no real actions to be taken in order to complete this task.


Circuits follow the cursor when pasting

Fixes #971
Describe the changes you have made in this PR -
When a circuit is copied, the circuit will follow the cursor until the user left clicks on the simulator area or when the user clicks Ctrl+Z.
Screenshots of the changes (If any) -

Coverage remained the same at 84.266% when pulling 0e29d1c1984a2d1359f569abb2d3e55bed744619 on KaustubhDamania:circuit-copy-cursor-follow#971 into afd537df5f0111e598647e7e617bdd841c70e50f on CircuitVerse:master.

@satu0king can you review this?

@satu0king can you review this please?

@satu0king can you review this please?

Nice Attempt. Please check the comment I have posted in the original issue.
Your implementation though smart hack, it falls short. It doesn't work correctly when multiple elements are selected. You will have to make changes to paste function in logix.js


Alwys display the marketing modal for the Miami Fair Week on /miami-fair-week

It seems like it is ok to show the default modal when the login button in the header is clicked, but we still need to show the specialized marketing modal for Miami Fair Week for every request. In order to make sure to always show the modal, the request without the m-id key will be redirected to the same path but with the m-id=ca12 pair (ca12 is already available both in staging and production). Not sure if there's a better way to achieve this but open to suggestions.

I'll go ahead and merge this to send this to QA.

👍


Nuxt3 import component globally

Hi, is it possible to install component globally in nuxt3.
i am using nuxt 3.6.5
and latest version of twelements
const { Ripple, Modal, initTE } = await import("tw-elements")
initTE({ Ripple, Modal })
if i do something like this in component it works but i want to ask you this.
i will use Ripple in a lot of component
if i write  const { Ripple} = await import("tw-elements") in every component is it bad for performance or is it possible for me to install ripple globally

Hi! Maybe try creating a new component that would receive a slot between an element with ripple? Something like that:
<template>
  <div data-te-ripple-init>
    <slot></slot>
  </div>
</template>

... and try using it with wherever you need.

I tried to solve this issue via latest version of Nuxt 3; to import all components of tw-elements via plugins(like we've used to do in nuxt 2 on most 3rd party UI kits, packages, etc.) but sadly no luck. Its so tiring and making your OCD triggered when always declaring those needed components/functions from tw-elements every components who have a form fields/elements.
plugins/tw-elements.js (i also declare the plugin in nuxt.config.ts)
export default defineNuxtPlugin({
  name: 'tw-elements',
  enforce: 'pre', // or 'post'
  async setup (nuxtApp) {
    // this is the equivalent of a normal functional plugin
  },
  hooks: {
    // You can directly register Nuxt app hooks here
    async 'app:mounted'() {
      // const nuxtApp = useNuxtApp()
      const {
        Input,
        Ripple,
        initTE,
      } = await import("tw-elements")

      initTE({ Input, Ripple }, { allowReinits: true })
    }
  }
})

i tried this approach too but still no luck (same plugin file):
export default defineNuxtPlugin({
  hooks: {
    'app:mounted': async () => {
      const {
        Input,
        Ripple,
        initTE,
      } = await import("tw-elements")

      initTE({ Input, Ripple }, { allowReinits: true })
    }
  }
})

It would be nice if the creator found a way to implement this way if ever possible. Since I discovered this UI kit, everything seems so easy to put UI components to the app I've been working on, just like I've been using Vuetify, but sadly I thought my experience with this UI kit would be similar to Vuetify.
Hoping for a response from the creators on this great UI kit.

Hi, you are using a vanilla JS package for framework and sure, there will be struggles. We are recomending to create a custom component and initialize it via JavaScript, as I was said in example above. Ensure that proper routing integration.

@Trochonovitz hello, thank you for your response.
Can you able to provide some sample/s on initialization of the said codeblock via JavaScript?
It would be a great guide to those of us that have no idea how to resolve the said problem.
Thanks.

@iprzybysz hi, do you have an idea about this issue?
Thanks

@noelc10 try our integration tutorial: it should answer to all of you questions. :) https://tw-elements.com/docs/standard/integrations/nuxt-integration/

@noelc10 try our integration tutorial: it should answer to all of you questions. :) https://tw-elements.com/docs/standard/integrations/nuxt-integration/

@Trochonovitz I already follow those tutorials and resolve the routing issue on my end. But regarding this issue about initializing/calling/importing all the needed components from TW-Elements in a plugin or in a one file only would be helpful so that we can declare those components in a single file and do direct creation of a desired component codebase from TW-Elements. Or something a proper coding pattern to understand and educate us on how to properly use the TW-Elements in a case-to-case basis is one of the helpful ways IMO.
Thanks.


Multiple accounts in solvers, step 2: driver honors solver's account

Part of #827.
Refactor driver to support different accounts for solvers. We drop solver names from most places, and replace them with unique solver IDs. Driver can look up solver by its ID and get its name, address, and other info.
Test Plan
Unittests are in progress. Proper testing suite will require contract mocking, I will add it later.

This PR is outdated, I'll send another (several of them, actually).


Add ignoring PyCharm IDE project settings directory to Python.gitigno…

Add ignoring PyCharm IDE project settings directory to Python.gitignore file.
Reasons for making this change:
PyCharm IDE creates own project settings directory. Ignoring it is a good practice

Thanks, but see Global\JetBrains.gitignore


Base EPrints Importer script

Create a base script that can be reused across eprints repos with minimal work. Based on JSON export.

https://github.com/ULCC/dart_hyku/tree/kfpub_shib/lib/importer/eprints

Remaining work:

[ ] Move KFSPECIFIC into a separate script, leaving the generic script, generic

Still needs a resource types lookup, but we don't have a standard list for that yet.


[fix] [broker] [branch-2.10] Upgrade rocksDB version to 6.16.4 to keep sync with BookKeeper 4.14.7

Fixes #xyz

Master Issue: #xyz

PIP: #xyz

Motivation
Pulsar 2.10 uses BookKeeper 4.14.7, and BookKeeper 4.14.7 uses RocksDB 6.16.4.
However, Pulsar also introduced RocksDB dependency and uses 6.10.2. When packaging all the dependencies into the Pulsar release package, the RocksDB 6.10.2 in Pulsar will override the RocksDB 6.16.4 in BookKeeper. The Pulsar release package will only contain RocksDB 6.10.2 in the end.
We encountered a RocksDB 6.10.2 memory leak issue when upgrading from Pulsar 2.9.x to 2.10.x, but still have not found any related issues in RocksDB. Pulsar 2.9.x uses RocksDB 6.16.4. One related issue is https://github.com/apache/bookkeeper/issues/3507
The reason why we revert https://github.com/apache/pulsar/pull/20287 is that we found the bookie throws method not found exception when upgrading RocksDb to <IP_ADDRESS> https://github.com/apache/pulsar/pull/20243#issuecomment-1540505700
The root cause of the exception is

RocksDB ABI broken : https://github.com/facebook/rocksdb/commit/1001bc01c9ea06785c2d5261ca9fe7ac0eac0625#r69967694 This commit was included in RocksDB 6.17.3. Code compiled with RocksDB <6.17.3 isn't compatible with a newer RocksDB version at runtime. https://github.com/apache/pulsar/pull/14962#issuecomment-1084192562
The BookKeeper 4.14.7 complies RocksDB 6.16.4 and uses RocksDB <IP_ADDRESS> at runtime

So we should upgrade the RocksDB version in Pulsar with the following steps.

Upgrade the RocksDB version to 6.16.4 to keep sync with BookKeeper 4.14.7 first to fix the RocksDB memory leak issue.
Upgrade BookKeeper branch-4.14 RocksDB version to <IP_ADDRESS> and trigger a new release 4.14.8
Upgrade Pulsar branch-2.10's BookKeeper dependency to 4.14.8
Upgrade Pulsar's RocksDB version to <IP_ADDRESS>

The reason why we should upgrade Pulsar's RocksDB version to <IP_ADDRESS> is https://github.com/apache/bookkeeper/issues/3734#issuecomment-1407626941. Pulsar 3.0 uses BookKeeper 4.16.0 which uses RocksDB 7.9.2, and we should make sure Pulsar can roll back from 3.0 to 2.10.x
Modifications

Verifying this change

[ ] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[ ] The REST endpoints
[ ] The admin CLI options
[ ] The metrics
[ ] Anything that affects deployment

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository:

could you please fill in the description ?
why this patch is better than https://github.com/apache/pulsar/pull/20287 ?
how did you choose 6.16.4 ?

could you please fill in the description ?
why this patch is better than #20287 ? how did you choose 6.16.4 ?

@eolivelli Thanks for your reminder. I updated the description, please help take a look, thanks.

/pulsarbot run-failure-checks

hi, @hangc0276, no offence. i have a naive idea that if we shade the rocksdb dependency on the pulsar broker side, can this problem be fixed？I have a worry that this problem will come again when the dependency update.

I created a separate issue #20314 for dealing with the broken Python tests in branch-2.10


calendar module infinit loading

like this.
How can i resolve that?
i already founded data about calendar problems in magicmirror forum.
i use google calendar private, public ical all have problem.

bonjour j ai le meme souci


Bounty: Demo Bounty for Max

Reward Size in USD
100 USD
Bounty
Test bounty
Acceptance Criteria

Sign up to Mintycode

Go to Mintycode to approve the Bounty.

/accept #25


add content loader + plus adaptive + bugs fix

Pull request template
Checklist

[x] I have read the CONTRIBUTING guide
[x] branch rebased on develop
[ ] styleguide check npm run validate
[x] good naming
[x] keep simple
[ ] video or screenshot attached
[x] testing instruction attached
[x] check your PR once again

Description

Video or screenshot

What and how to test

https://drive.google.com/file/d/1Y0SeOCLdClBlJFDhQRNFDaSDuDGGMIMY/view


appsody test fails for quarkus stack

Describe the bug
Running appsody test on the experimental/quarkus stack and it fails. Logs here:
[Container] [INFO] -------------------------------------------------------
[Container] [INFO]  T E S T S
[Container] [INFO] -------------------------------------------------------
[Container] [INFO] Running org.acme.quickstart.GreetingResourceTest
[Container] [ERROR] Tests run: 2, Failures: 0, Errors: 2, Skipped: 0, Time elapsed: 4.215 s <<< FAILURE! - in org.acme.quickstart.GreetingResourceTest
[Container] [ERROR] testHelloEndpoint  Time elapsed: 0.024 s  <<< ERROR!
[Container] org.junit.jupiter.api.extension.TestInstantiationException: TestInstanceFactory [io.quarkus.test.junit.QuarkusTestExtension] failed to instantiate test class [org.acme.quickstart.GreetingResourceTest]: Failed to create the boostrap class loader
[Container] Caused by: java.lang.IllegalStateException: Failed to create the boostrap class loader
[Container] Caused by: io.quarkus.bootstrap.BootstrapException: Failed to create the deployment classloader for org.acme:getting-started::jar:1.0-SNAPSHOT
[Container] Caused by: io.quarkus.bootstrap.resolver.AppModelResolverException: Failed to read descriptor of org.acme:getting-started:jar:1.0-SNAPSHOT
[Container] Caused by: org.eclipse.aether.resolution.ArtifactDescriptorException: Failed to read artifact descriptor for org.acme:getting-started:jar:1.0-SNAPSHOT
[Container] Caused by: org.apache.maven.model.resolution.UnresolvableModelException: No versions matched the requested parent version range '[0.2,0.3)'
[Container]
[Container] [ERROR] testGreetingEndpoint  Time elapsed: 1.602 s  <<< ERROR!
[Container] java.net.ConnectException: Connection refused (Connection refused)
[Container] 	at org.acme.quickstart.GreetingResourceTest.testGreetingEndpoint(GreetingResourceTest.java:28)
[Container]
[Container] [INFO]
[Container] [INFO] Results:
[Container] [INFO]
[Container] [ERROR] Errors:
[Container] [ERROR]   GreetingResourceTest.testGreetingEndpoint:28 ? Connect Connection refused (Con...
[Container] [ERROR]   GreetingResourceTest.testHelloEndpoint ? TestInstantiation TestInstanceFactory...
[Container] [INFO]
[Container] [ERROR] Tests run: 2, Failures: 0, Errors: 2, Skipped: 0
[Container] [INFO]
[Container] [INFO] ------------------------------------------------------------------------
[Container] [INFO] BUILD FAILURE
[Container] [INFO] ------------------------------------------------------------------------
[Container] [INFO] Total time:  01:44 min
[Container] [INFO] Finished at: 2019-11-25T13:37:56Z
[Container] [INFO] ------------------------------------------------------------------------
[Container] [ERROR] Failed to execute goal org.apache.maven.plugins:maven-surefire-plugin:2.22.1:test (default-test) on project getting-started: There are test failures.
[Container] [ERROR]
[Container] [ERROR] Please refer to /project/user-app/target/surefire-reports for the individual test results.
[Container] [ERROR] Please refer to dump files (if any exist) [date].dump, [date]-jvmRun[N].dump and [date].dumpstream.
[Container] [ERROR] -> [Help 1]
[Container] [ERROR]
[Container] [ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[Container] [ERROR] Re-run Maven using the -X switch to enable full debug logging.
[Container] [ERROR]
[Container] [ERROR] For more information about the errors and possible solutions, please read the following articles:
[Container] [ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException
[Container] [Warning] Wait Received error starting process of type APPSODY_RUN/DEBUG/TEST_ON_CHANGE while running command: ./mvnw -B test error received was: exit status 1

To Reproduce
Steps to reproduce the behavior:

Initialise a quarkus project
Run appsody test
See error

Expected behavior
The base tests should pass
Actual behaviour
The tests fails
Environment Details (please complete the following information):

OS: macOS

If applicable please specify:

CLI version: 0.5.0
Stack you are using (including the version): Quarkus

This was resolved in https://github.com/appsody/stacks/pull/525 - Closing.


Add Isolyzer output to METS file

Wrap in premis:objectCharacteristicsExtension elt, similar to MediaInfo's EBUCore output.
Before doing this make below change to isolyzer:
https://github.com/KBNLresearch/isolyzer/issues/17
Then use cd-info output to determine SECTOROFFSET for Isolyzer call.

Done: https://github.com/KBNLresearch/omSipCreator/commit/24a8c8be747a3667192d4680ec36d833dc76cb75
Add namespace: https://github.com/KBNLresearch/omSipCreator/commit/c0f5e03b83a773e7d1e1d16c1aa40e399f2f2008
TODO:

Include SECTOROFFSET

TODO: when calling Isolyzer, use sector offset as reported by cd-info! Now set to 0 (default), which goes wrong with enhanced CDs.

Sector offset now taken from cd-info output https://github.com/KBNLresearch/omSipCreator/commit/c57d7fbb2e0bc27b99cbc9800138b8f6e034d4dc


documentation: license

The license block has to be present at the top of every PHP file, before the namespace
recorded in #69
a sniff a day keeps the doctor away... 😎
as mentioned in #69, this one just raises a warning as it appears to be specific to contributing to any symfony repository so shouldn't raise an error as far as i'm concerned. also no check whether the comment actual contains a license reference as placing any other comment prior to the namespace doesn't really make sense to me.
👍 for all the quick merges today @djoos, very much appreciated

Thanks for your contribution @wickedOne!


Allow selecting the Falco version we want to run the playground with

Motivation
While not every version of Falco can be executed with every playground version, it would be nice to allow users to select what they would like to have a drop-down menu where to select the version of Falco they want to test in the playground. Now that we have introduced releases, I believe this became a possibility
Alternatives
Run only with the latest released version, like it is today

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle rotten

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

@poiana: Closing this issue.

In response to this:

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.


fix: use logger instead of an exception

Even if there is an unusable filter, doesn't mean the remaining
interface could not still be used. Using logging we can inform the admin
of the misconfiguration, but still show the form.

No, this is a dedicated exception which tells you what went wrong.

I'm confused, does this mean you've changed your opinion? Two months ago, you agreed with what I said...

Or, to be more precise, at least acknowledged the argument/didn't disagree.

I agreed with you that Django breaking by itself is not helpful often, as it might not tell you what broke.
But this exception here is a custom one which tells you exactly what broke. That is helpful.

Ok, that wasn't obvious to me at all as I specifically singled out this very exception as being unhelpful; it's what the bulk of my comment was about. My reasoning remains the same, so I guess we'll have to agree to disagree on what is (or isn't) helpful.

But this exception here is a custom one which tells you exactly what broke. That is helpful.

I'not sure how this exception is helpful, because, ehm, a string is a string. But YMMV:

Filters for individual entities need to be of type string or dictionary.
Got instead: <class 'str'>

But whatever the Exception or the Logger says: with a9eb86831c1f180a52b633668b06e09a9afe1c7d we fixed the entities settings, so that they are read from the settings file again (as discussed). If projects have a settings.py containing entity settings (like for example the example project) and their filter definitions contain strings that does not adhere to the expectations of the set_sort_filters method (like for example in the example project) the projects would now get an exception. Just by updating. In the current setup, where we have no changelog and no release management, the most sensible thing to do is to try not to break existing installations.

the projects would now get an exception. Just by updating.

Ah, I missed that. How so? And what does updating mean in this context? Pushing a new commit to apis-ontologies?


unregister widgets

When running the widgets cache we got a failure in severe_fatal_count_on_small_motor_by_accident_year
Unregistering for further tests before adding to production.
@shaysw @ziv17 FYI


Added input type 'Hidden'

Enhacement: https://github.com/the-control-group/voyager/issues/822

Thanks!


Is this a typo in the conf file

https://github.com/dockur/samba/blob/4f88c26e9c4b92b832305cf235915c2508ab33cf/smb.conf#L16
It looks like a typo in the config file. Should it not be
writeable = yes

Thanks for the fast answer. I was quite confused by it :-)


[Toolbar] Height should be 48px for Mobile Landscape

Problem description
https://material.io/guidelines/layout/structure.html#structure-app-bar
Currently, the toolbar uses the 'sm' width breakpoint, so the toolbar becomes 64px in landscape on mobile devices. According to the guidelines, it should be 48px.
Link to minimal working code that reproduces the issue
https://material-ui-1dab0.firebaseapp.com/component-demos/app-bar

Use Chrome DevTools to emulate mobile device.
Click the rotate button in DevTools.

Versions

Material-UI: 1.0.0-alpha.21
React: 15.6.1
Browser: Chrome 59.0.3071.115 Windows x64

The issue lies in the stylesheet, which makes no determination of mobile/desktop, but relies on the defined breakpoints.  Below small, the device is considered mobile.  Small and higher, the device is considered Tablet/Desktop:
export const styleSheet = createStyleSheet('MuiToolbar', theme => ({
  root: {
    position: 'relative',
    display: 'flex',
    alignItems: 'center',
    height: 56,
    [theme.breakpoints.up('sm')]: {
      height: 64,
    },
  },
  gutters: theme.mixins.gutters({}),
}));

In order to properly detect mobile landscape, one of the dimensions fall below the small breakpoint, so we'd have to consider the device height before specifying a toolbar height of 64.

For reference, this is how material-components-web currently implements it:
    min-height: 64px;

    @media (max-width: 959px)
      and (orientation: landscape) {
      min-height: $mdc-toolbar-mobile-landscape-row-height;
    }

    @media (max-width: 599px) {
      min-height: $mdc-toolbar-mobile-row-height;
    }

I wish we could rely on material-components-web at some point. But given how different their styling solution is from ours, it might be simpler to take inspiration from their style.
I like the and (orientation: landscape) solution. We can use that.

Sure, we could create a new abstraction. But that's gonna require some thought:

We need to better understand the specification. I find the responsive docs confusing.
How do we know the device type? User Agent based solution would quite increase the payload. Do we need that much information? I would say no.
Is orientation and width enough to implement the logic? Maybe
Do we need an abstraction on top of orientation? Given how straightforward it is, I would say no.


Fix for bug in inactive set calculation.

@a-latyshev Can you check this corresponds with what we discussed earlier?

These changes seem to be in line with our last meeting.

@jhale I think you can put
    # Construct inactive set on displacement.
    u_inactive_set = np.arange(
        0, V_u.dofmap.index_map_bs * V_u.dofmap.index_map.size_local, dtype=np.int32
    )
    u_restrict = np.setdiff1d(u_inactive_set, bc_dofs_u_all)

outside of the main loop for a better performance as well as the assembling of the matrix B, I suppose.
    B.zeroEntries()
    fem.petsc.assemble_matrix_block(B, B_form)
    B.assemble()

Thanks - I will do this in another branch, and also the fixes for snes.solve.


add bg-repeat = norepeat and bg-size = cover

Fix background for smaller background images so it does not tile.

I won't merge this, sorry. Because if i merge this PR, this time people won't be able to use tile patterns.  So people can add this according to their personal usages. Thank you.

That's up to you just keep in mind that your repo uses a very large background image not a tile pattern so any time a user tries to customize it with a similar background image that isn't quite as large it tiles.  In my opinion when a user replaces a large bg image with a tile pattern they understand they need to make changes to the css.  Your code had me scratching my head for a few seconds because I was using it in the same use case as the repo (large bg image just "only" 1280px wide instead of your 1920px) which is why I opened the PR.  As it is now it may work for a case you are not illustrating (tile patterns) but for the situation that is displayed (a large bg image) it has a bug under 1920px.
Just something to keep in mind.

Here tiled pattern on my website: http://taylantatli.me/
But now with your proper explanation, it makes more sense. Yes, this will be better for background images. I will merge it now. Thank you.
BTW, i'm just curious. Are you selling this theme under sitecoxyz?

Thanks Taylan, I'm not selling it but I plan to use it on at least one blog.  For now sitecoxyz is just an account with a collection of soon-to-be blog sites for cheap domains I grabbed.

I should add I really like the style of this which is why I said "at least" one blog :)
Keep up the good work!

Thank you. :smile:


Shopping cart - Mainly UI Changes

Description of changes

Added Shopping Cart functionality
Shopping Cart is saved to session
Added Shopping Cart view page
All values update correctly when quantity is incremented or decremented.

Screenshots (If Applicable)

Checklist

[ ] I have unit tested my code.
[ ] I have integration tested my code.
[X] All new and existing tests passed.
[X] I have updated the diagrams ModelsUMLClassDiagram and EntityRelationshipDiagram (if necessary).

Add to cart button is not refreshing the shopping cart icon in the top bar of the page so it seems like nothing is happening when you click on "Add to cart" in the table.

Remove button on cart screen is not working, you should be able to remove the items from your cart and see that no items are currently in your cart.

there's no test cases unit or integration included with this PR, I think at a minimum we should unit test the ShoppingCart, ShoppingCartController, and ShoppingCartRestController classes, maybe testing ShoppingCartService and integration tests of the controllers too

When removing an item from a shopping cart and another item exists, we are taken to a page that says shopping cart is empty which is not true.

the add to cart button automatically increases the cart item count by 1 without first ensuring that the item exists and was added to the cart

the decrement item button automatically decrements the number of items before ensuing this is actually done on the back-end (i changed the book id to a fake book and it still decremented the number of items on the front-end)


ASSERT FAILED: space space.txt should not exist.

Not sure what gone wrong and how to fix that.
MAC OS 10.12.3

I make the change you suggested (thanks!).  Sadly I can't get "make confidence" working on macOS 10.12.2 working because I upgraded GnuPG which seems to have broken it.  Meanwhile
Meanwhile I can't get it working on Linux (CENTOS 7) (batch creation of keys spins forever).
I committed some fixes but I don't have time to work on this today. Maybe that's enough for you to continue.

@tlimoncelli I've tried repository with last commit with latest ubuntu and centos clean install but it breaks at different places.

Sorry I dropped the ball on this. Would you please check to see if the changes I've made in the last few months covered this situation? If not, please rebase and I'll take a look at the PR again.


On touch devices scroll closes popup

Describe the bug
On touch devices scroll closes popup
To Reproduce
Add click trigger and closeOnClickOutside attribute to popup. Open on a mobile device and scroll with touch.
Expected behavior
Popup should remain.
Additional context
I checked old repo and to get click events on some mobile devices this._globalEventListeners.push(this._renderer.listen("document", "touchend", this.hideOnClickOutsideHandler.bind(this)));code added to popper.directive.ts. But this cause a scroll as click outside.
Similar bug issue on old repo
The issue that caused the error on old repo

Absolutely not true. Just checked  Try on the demo page.

There is not an example that use hideOnClickOutside on demo page.
Can you check this example https://angular-ivy-awfugd.stackblitz.io? Open the poppover and scroll with touch. Poppover should not close when user just scroll, it should close on click outside.

@berkayyildiz it seems that to fix what you say we should revert this commit.
https://github.com/MrFrankel/ngx-popper/commit/96edf1af751e79de2c13888bd3dd935c14d4b30f
I'll do some tests. I'm looking comments anyway. I will tag you when I have news.
Cheers and thanks for reporting.

Fixed in v 13.0.1


Asking to take a picture using my camera, not working. Using the button "take a shot from webcam" work fine.

Expected Behavior
Expect the app to use my camera to take a picture without using the button "take a shot from webcam"
Current Behavior
The answer asked me to provide the image. Using the button "take a shot from webcam" to provide the image work just find
Steps to Reproduce
Ask the app to take a picture with the camera. Not working. I try the same queries as in a video you provide that do just that and it still not work
Possible Solution
Make sure the app understands the queries.
Context
Please provide any additional context about the issue.
Screenshots

ah ok, well this works only with 3rd gen personalities. For example, if you use lollmz (with z) which can be found in the catagory generic, you'll be able to do many many things like taking screenshots or camera shot or open another webpage, or generating an image etc...


Fix bug in computing values for solvated molecules over a range of temperatures

The main reason for the pull request is that when iterating through temperatures, the concentration would always be calculated as if the molecule is in the gas phase, even if the user defined a (liquid) concentration (e.g. "-c 1"). Thus, I added a new Boolean, gas_phase, that would be assigned based on whether or not the user used the "-c" option. Then, iterating through the temperature range, the specified concentration is kept at the user-input value (assuming the liquid is reasonably described as incompressible), and otherwise recalculating the concentration based on an ideal gas assumption (as was previously done in all cases).
Other changes are to move closer to PEP8 standards (https://www.python.org/dev/peps/pep-0008/). For example, variables in all caps should correspond to globals only. Additionally, "line" was redefined within multiple loops, which can lead to unexpected behavior, so the outer-loop "line" was renamed in those cases.
The GoodVibes tests were checked on my local machine and passed (hopefully ensuring that the PEP8 changes caused no problems). Thus, the calculation for solvated molecules over a range of temperatures is likely not currently tested.

Thanks for the contribution


fix: Replace Karpenter SQS policy dynamic service princpal DNS suffixes with static amazonaws.com

Description

Replace Karpenter SQS policy dynamic service princpal DNS suffixes with static amazonaws.com

Motivation and Context

Resolves #2932

Breaking Changes

No

How Has This Been Tested?

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

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

This PR is included in version 20.2.2 :tada:


chore(File): support weapp

https://github.com/leancloud/weapp-polyfill/commit/ca6dffd61ef6a0475f826112b256aa53bd01dcf1

Current coverage is 65.55% (diff: 100%)

Merging #381 into next will not change coverage

@@               next       #381   diff @@
==========================================
  Files            30         30
  Lines          2619       2619
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           1717       1717
  Misses          902        902
  Partials          0          0

Powered by Codecov. Last update 26fd0a9...89b3e49

LGTM


[NOTICK] Fix broken links to the api docs on the docsite

Changes

Fix broken links to the api docs on the docsite

I think there's been a misunderstanding. There's no need to update the doc links. Just revert the config which controls the links to be conclave again. What I actually want is Conclave Core to appear in the title, like it does here: https://r3-conclave.artifacts.teamcity.com/presignedTokenAuth/d7b1bd54-87e9-43f7-b474-7da8aa407dd8/repository/download/ConclaveCore_SdkBuild/3649:id/docs.zip!/api/index.html

I tried that, it doesn't work. The module name controls both the links and the title. Displayname doesn't seem to do anything that I can see :man_shrugging:

The escaped space character is really ugly. No way to fix that is there? Perhaps rename to Conclave-Core to fix that. WDYT?

The escaped space character is really ugly. No way to fix that is there? Perhaps rename to Conclave-Core to fix that. WDYT?

The links work fine without the escaped space, but unfortunately autocomplete of the URL in intellij doesn't

Let's just get rid of the space. I know it's supported in browsers, but I just can't help thinking this will come back to bite us!


Website front page enhancements v0.5

Acceptance

[ ] fixed navbar content responsiveness

[ ] nice looking "Features" section, that matches the rest of the page #247
[ ] a bit smaller images in the "How it works" section for small screens

[ ] fixed footer navigation links responsiveness

Bonus (added 02/02/2023):

[ ] prettier nav dropdown styling

@PhilippeduPreez if you have time, please also look into dropdown styling. I've added it to the acceptance criteria above.

@olayway i want us to remove navbar dropdown - see #256. I suggest we do this asap and stop supporting this.

@olayway Does navbar content responsiveness and footer navigation links responsiveness still give you problems? Because they seem to work for me on all screen sizes

Does navbar content responsiveness and footer navigation links responsiveness still give you problems? Because they seem to work for me on all screen sizes

@PhilippeduPreez it seems it works just fine now

@PhilippeduPreez have you already removed the support for menu dropdowns? I'm asking because this task is ticked off. And if so, we can close this issue as I've moved the reference to feature section improvements to issue #410 as it has been mentioned there as well.


Q: parallel execution for each resolver in Query not supported

Query has multiple resolvers. How can I make these resolvers excute in parallel

@abhaylangoti Use the ExecutorServiceExecutionStrategy as the queryExecutionStrategy.


Support for default gateway for containers

If a containers endpoints doesn't have the GW set by the driver libnetwork creates an endpoint on a
bridge network which acts as default gateway for external connectivity.

@sanimej Can you please squash the Merge commit? It should not show up in the PR

LGTM

ping @mrjana

LGTM


region_with_plus_1_translational_frameshift [sf#74]

Reported by batchelorc on 2008-01-15 12:23 UTC
Stacia Engel writes:
> we would like to use the term 'plus 1 translational frameshift'
> as a locatable sequence feature, but we see that it's an 'attribute'
> instead of a 'feature', which has left us nonplussed
I'm just creating this as a new tracker item so it doesn't get buried in the plasmid discussion.
I've entitled it "region_with_plus_1_translational_frameshift"---I'm sure there must be a better name for the new term.
Colin.

Commented by eilbeck on 2008-01-15 22:15 UTC
Logged In: YES
user_id=742851
Originator: NO
Thanks Colin - I was on my to do list - I had talked to Stacia separately about it.
I think the term(s) we are trying to describe here are the 1 or  bases that are skipped when a frame shift occurs, not the whole transcript. I would call it plus_1_frameshift_region (or base). We should also be able to add cardinality to this at some point - if we include base in the ontology.
I think we need to keep that attributes as are in the ontology.
--K

Commented by srengel on 2008-01-17 18:00 UTC
Logged In: YES
user_id=785556
Originator: NO
hi,
could it just be plus_1_frameshift?  or plus_1_translational_frameshift?  do we have to have 'region' or 'base' as part of the term name?
stacia

Commented by eilbeck on 2008-01-18 18:43 UTC
Logged In: YES
user_id=742851
Originator: NO
Hi Stacia et al
translational_frameshift SO:0001214 is a mRNA region and thus a sibling of recoding_stimulatory_region. (This makes me feel happy that it is in the right place.)
It has the child plus_1_translational_frameshift. SO:0001213
--Karen

Updated by eilbeck on 2008-01-18 18:43 UTC

status: open --> closed-accepted


Cache assets similar to how the normal compilation does it

Previously extract-text-webpack-plugin emitted css assets every time anything else changed.
Injected assets were also not included in compilation hash calculation and caused some rebuilds to be hidden.
There are no tests included in this PR, since this is watch-related and tests will go in a separate PR in the main repo that has great facilities for that.
This continues fixes started with https://github.com/webpack/webpack/pull/3737 and https://github.com/webpack/webpack/pull/3744 and should hopefully address a lot of watch related issues with extract-text-webpack-plugin.

Current coverage is 85.92% (diff: 88.88%)

Merging #325 into master will increase coverage by 0.08%

@@             master       #325   diff @@
==========================================
  Files             4          4
  Lines           332        341     +9
  Methods          68         68
  Messages          0          0
  Branches         71         74     +3
==========================================
+ Hits            285        293     +8
- Misses           47         48     +1
  Partials          0          0

Powered by Codecov. Last update cbd4690...580b19a

@d3viant0ne Any thoughts on this? Post final merge?

@uncleyo Please close and reopen the PR to trigger the CLA Bot again, sign the CLA and rebase against current master 😛

I've rebased this PR just to refresh it to latest master. I Don't think it should be merged.
For anyone interested, I've been using this plugin https://gist.github.com/uncleyo/106c3cb407bf5dd38ebd6be0710afd4a as a workaround for a while now. If anyone else is missing this feature, feel free to use it.
I don't like the code in this PR here myself, honestly, as I believe this is something webpack should handle automatically., either by exposing something like compilation.addAsset() or with plugin, something like the one in my gist.
I'll not close the PR just yet, hoping to hear some comments

@uncleyo Thx :) appeciated, well I'm not 💯 suited to give a meaningful answer about this I will try to bring @sokra in :D

@uncleyo Would you be interested in make the gist into a plugin cache-webpack-plugin or the like :)  ?  It's caching assets 'only' atm right ?

We are getting close to a new major release and therefore I will close this for now, feel free to open a new PR with these changes, when #540 landed on master, but as you commented this maybe better solved at core level in general 😛. Thx


VisLeafletFlowMap: 'layer.getMapLibreMap is not a function' when using renderer="raster"

VisLeafletFlowMap: 'layer.getMapLibreMap is not a function' when using renderer="raster"
This only seems to happen when the page first loads:
Error in /turbo_modules/<EMAIL_ADDRESS>(172:39)
layer.getMaplibreMap is not a function

Using Firefox on Mac
See stackblitz:
https://stackblitz.com/edit/react-ts-ht4tcq?file=App.tsx

@ereid7 Thanks for the stackblitz!
@reb-dev Can you please look into it?

@ereid7 Fixed in version 1.0.3-beta.6


Can hand tracking be used with black glove or reflective markers?

Hi! I would like to use media pipe hand tracking for tracking a hand inside a black glove. Is there any way to do this? I'm a developer so if there's any part of the open source code I should change, I'd be glad if you point me there. If this is not possible, would it help using reflective markers? It's very important for me, thanks!

 For example: Python, JavaScript, Android or through Other solutions available in the mediapipe. Thank you!

Hi @kuaashish , I was hoping to make it work on Android but it doesn't really matter. The issue is more transversal and about the ML model & the positioning of the dots. I would like to now if there's any possible workaround to make the program work with a black glove or with reflective markers, and what part of the open source code should I be looking at in order to make the changes, regardless the platform but preferably for Android.

Hi @crpistillo,
The hand tracking model has been trained specifically with exposed hands.  I believe other conditions will require training a similar model.  Thank you!

Hi @kuaashish , yeah I suppossed another training will be required (I'm not intending to do that). Can you point me at least to the part where the markers in the hand are being set? So I can reuse that part and set the points with another logic while reusing your skeleton? Otherwise nevermind, I guess I can find that part eventually

Hello, unfortunatelly I did not get your question. What do you mean by markers in the hand? If you are asking about hand tracking model inference, it happens here: https://github.com/google/mediapipe/blob/6cdc6443b6a7ed662744e2a2ce2d58d9c83e6d6f/mediapipe/modules/hand_landmark/hand_landmark_cpu.pbtxt#L69

Hi @crpistillo,
Could you please go through the above comment and let us know the required info to understand the issue better. Thank you!


[stable/grafana] Update to the recommended pvc patterns.

This is a split-off of PR #408 to just contain the chart grafana.
If there's any commonality in changes nessary between this PR and the others that are being submitted at once, please mention them in #408.
Some things to note:

This initial PR will not be updating the Chart.yaml's version string, as I feel it should be done manually per-pr once each chart's otherwise acceptable.
An automated script created the splits and the PRs, on a per-folder basis under stable, after copying the relevent files from a backup of my local copy of #408

@k8s-bot ok to test

@k8s-bot ok to test


movie card content fix

##483
Closes: #483
Description of Changes
Updated the movie card module to fix the bug.
Checklist:

[x] My code adheres to the established style guidelines of this project.
[x] I have conducted a self-review of my code.
[x] I have included comments in areas that may be difficult to understand.
[] I have made corresponding updates to the project documentation.
[x] My changes have not introduced any new warnings.

Screenshots

Original
Updated

Please provide any necessary screenshots to illustrate the changes made.

@sourabhsikarwar This is my new PR please check it is there any error.


Windows path error

...miniconda3\envs\pysnowpark\lib\site-packages\snowcli\cli\snowpark_shared.py:84 in snowpark_create
is resolving a windows temp_dir to:
'C:\Users\JDECAR~1\AppData\Local\Temp\tmpd52masc9'
and then erroring out due to the presence of the tilde:
ProgrammingError: 001003 (42000): SQL compilation error: syntax error line 1 at position 26 unexpected "'~'"

Same as:
https://github.com/Snowflake-Labs/snowcli/issues/124

Closing as duplicate of https://github.com/Snowflake-Labs/snowcli/issues/124


Error when trying to run demo.jl

I am getting the an error after the following:
cd Nautilus.jl
julia --project=.
julia> include("test/demo.jl")

I get the following error
ERROR: LoadError: MethodError: no method matching main(::Menu)
Closest candidates are:
  main() at ~/Dropbox/inbox/Nautilus.jl/test/demo.jl:92
Stacktrace:
 [1] (::var"#9#10")(arg1#291::Bool, arg2#292::Bool, arg3#293::Bool, arg4#294::Bool, arg5#295::Bool, arg6#296::Bool, arg7#297::Bool, arg8#298::Nothing, arg9#299::Bool, arg10#300::Bool)
   @ Main ~/Dropbox/inbox/Nautilus.jl/test/demo.jl:144
 [2] lift(::Function, ::Observable{Any}, ::Observable{Any}, ::Vararg{Any}; kw::Base.Pairs{Symbol, Union{}, Tuple{}, NamedTuple{(), Tuple{}}})
   @ Makie ~/.julia/packages/Makie/vwpRo/src/interaction/observables.jl:30
 [3] lift(::Function, ::Observable{Any}, ::Observable{Any}, ::Observable{Any}, ::Vararg{Any})
   @ Makie ~/.julia/packages/Makie/vwpRo/src/interaction/observables.jl:27
 [4] top-level scope
   @ ~/Dropbox/inbox/Nautilus.jl/test/demo.jl:133
 [5] include(fname::String)
   @ Base.MainInclude ./client.jl:451
 [6] top-level scope
   @ REPL[1]:1
in expression starting at /Users/jakevossen/Dropbox/inbox/Nautilus.jl/test/demo.jl:133

Am I missing a step - or is there a bug somewhere?

Nope, there's a bug, and the bug's my fault. I should upload the fix later today.
Thanks for keeping up with the newsletter 😅

No worries 🙂


[BUG]micro v4 can't register service

i use v4, and run command: micro run . --name helloworld, but micro services not list the helloworld service here, when run go run ., the service can start success, but also can not list with micro services command,
os: mac os m1
go version: go version go1.20.11 darwin/amd64
protobuf: libprotoc 25.1
etcd version: etcd Version: 3.5.9
micro version:  micro version v4.3.1
micro status output:
knight@knightdeMacBook-Pro helloworld % micro status

2023/11/23 12:09:25 /home/runner/go/pkg/mod/gorm.io/gorm@v1.24.1-0.20221019064659-5dd2bb482755/gorm.go:206
[error] failed to initialize database, got error Binary was compiled with 'CGO_ENABLED=0', go-sqlite3 requires cgo to work. This is a stub
NAME		VERSION	SOURCE					STATUS	BUILD	UPDATED		METADATA
helloworld	latest	/Users/knight/Projects/y/helloworld	error	n/a	1h2m31s ago	owner=admin, group=micro, error=n/a

code :
package main

import (
	"github.com/go-micro/plugins/v4/registry/etcd"
	"go-micro.dev/v4"
	"go-micro.dev/v4/logger"
	"helloworld/handler"
	pb "helloworld/proto"
)

func main() {
	// Create service
	reg := etcd.NewRegistry()
	srv := micro.NewService(
		micro.Name("helloworld"),
		micro.Address(":9090"),
		micro.Registry(reg),
	)

	// Register handler
	pb.RegisterHelloworldHandler(srv.Server(), handler.New())

	// Run service
	if err := srv.Run(); err != nil {
		logger.Fatal(err)
	}
}

Seems you aren't using the etcd registry in the services call.
micro --registry etcd services

You can add etcd with a noop import and then run the service with "--registry etcd", this allows easy switching of dependencies.
package main

import (
	"go-micro.dev/v4"
	"go-micro.dev/v4/logger"
	"helloworld/handler"
	pb "helloworld/proto"

	_ "github.com/go-micro/plugins/v4/registry/etcd"
)

func main() {
	// Create service
	srv := micro.NewService(
           micro.Name("helloworld"),
        )

	// Register handler
	pb.RegisterHelloworldHandler(srv.Server(), handler.New())

	// Run service
	if err := srv.Run(); err != nil {
		logger.Fatal(err)
	}
}

Then:
micro run . --registry etcd

Same as:
go run main.go --registry etcd

micro --registry etcd services

when i run  , error occurred, flag provided but not defined: -registry, the error detail below:
Incorrect Usage. flag provided but not defined: -registry

NAME:
   micro - API first development platform

USAGE:
   micro [global options] command [command options] [arguments...]

VERSION:
   v4.3.1

COMMANDS:
   auth      Manage authentication, accounts and rules
   call      Call a service e.g micro call greeter Say.Hello '{"name": "John"}'
   config    Manage configuration values
   env       Get/set micro cli environment
   gen       Generate a micro related dependencies e.g protobuf
   get       Get resources from micro
   init      Generate a profile for micro plugins
   kill      Kill a service: micro kill [source]
   login     Interactive login flow.
   logout    Logout.
   logs      Get logs for a service e.g. micro logs helloworld
   new       Create a service template
   run       Run a service: micro run [source]
   server    Run the micro server
   service
   services  List services in the registry
   shutdown  Shutdown a micro server
   status    Get the status of services
   store     Commands for accessing the store
   stream    Create a service stream e.g. micro stream foo Bar.Baz '{"key": "value"}'
   update    Update a service: micro update [source]
   user      Print the current logged in user
   web       Run the micro web UI
   help, h   Shows a list of commands or help for one command

GLOBAL OPTIONS:
   -c value               Set the config file: Defaults to ~/.micro/config.json [$MICRO_CONFIG_FILE]
   --env value, -e value  Set the environment to operate in [$MICRO_ENV]
   --namespace value      Namespace the service is operating in (default: "micro") [$MICRO_NAMESPACE]
   --client_id value      Account ID used for client authentication [$MICRO_CLIENT_ID]
   --client_secret value  Account secret used for client authentication [$MICRO_CLIENT_SECRET]
   --public_key value     Public key for JWT auth (base64 encoded PEM) [$MICRO_PUBLIC_KEY]
   --private_key value    Private key for JWT auth (base64 encoded PEM) [$MICRO_PRIVATE_KEY]
   --name value           Set the service name [$MICRO_SERVICE_NAME]
   --profile value        Set the micro server profile: e.g. local or kubernetes [$MICRO_SERVICE_PROFILE]
   --network value        Service network address [$MICRO_SERVICE_NETWORK]
   --help, -h             show help (default: false)
   --version, -v          print the version (default: false)
Flag provided but not defined: -registry

Micro v4 is not compatible with go-micro

Micro v4 is not compatible with go-micro

how can i registry a service to the micro server? any docs about that?

Go to micro.dev and read the docs there. Effectively import micro.dev/v4/service and use "micro run [repo]"

That's correct they are now two separate projects

That's correct they are now two separate projects

what the different between these and which one i should use?

One is a platform. The other is a framework. Use go-micro if you want to piece everything together yourself.

Use go-micro if you want to piece everything together yourself.

thanks! it's clear.


[Cherry-pick] Api version NumberFormatException fix from master to maint

This is just pr cherry-pick from master branch, so will merge as ci passed.


need to separate benchmarks that need to get data files from those that dont

In nc_perf we have some benchmarks which work independently, and some which fetch files from a Unidata FTP server.
We need to separate those. It's hard for NOAA HPC systems to handle fetching files. THe compute nodes don't have internet connectivity.
So we need to introduce a new configure option --enable-download-benchmarks, which will be like --enable-benchmarks, but will fetch and download files first. Then --enable-benchmarks will only run benchmarks which don't require file downloads.

Turns out to be just one file that fetches from ftp: run_knmi_bm.sh.
I will put up a PR to take that out of the list of tests run by make check. It will remain as a stand-alone benchmark script if anyone wants to run it.


Add TypeScript support

Adding TypeScript support and a tip to format on save in the README.md.

Hi,
Thanks for your contribution, I will publish later for technical reason.


401 response deletes satellizer token in local storage

This is more of a question.. I've noticed that sometimes when the server returns a 401, the local token is deleted and I have to login again. It usually happens when something has gone wrong, and I fix it in my server-side code, but I've noticed I can sometimes return 401 with a proper 'unauthorized' request and the local token remains valid.
For example, with Express and Node, this response reset my local token:
} else {
      res.status(401).send({ message: 'You do not have permission to change this role.'});
    }

Should I reserve 401 for the case where I want to blow away their token, and use a different response code, or is there something I'm missing?
Thanks!

@sahat, is there any discussion why have this done and possible workaround? It caused a lot of problem to users, for example here: http://stackoverflow.com/questions/35672707/how-to-delete-token-on-api-error-401


Graceful Failure for no iOS Push Information

With no APN info, the app behaves poorly.
Rather, this should gracefully degrade. Tokens can still be saved with devices, but if the APN is not configured, it's simply not used.

This is now working, and allowed Travis-CI to build.


No Line Breaks When Pasting

When I paste the contents of code into a misbehave component the resulting code is displayed in single line instead of many lines the code was copied from.
Is there any way to show the line breaks of code that has has been pasted in?

I have this problem when when I copy code from vs-code and paste it into the demo.

Yes. @orbitbot Thanks so much for looking into this.


Support for built-in PostgreSQL ranges

Users can create arbitrary range types, but Npgsql's EF Core type mapper currently doesn't look at the database to construct the mappings (singleton), making it impossible to know the datatype name for a given NpgsqlRange. What we can do is support the built-in range types, which will probably be enough.
Includes scaffolding support.

Scaffolding depends on https://github.com/aspnet/EntityFramework/pull/7269

https://github.com/aspnet/EntityFramework/pull/7269 is done now - is there anything else blocking this?
Thanks for all the hard work, @roji. If it weren't for npgsql I probably wouldn't be using .NET Core!

https://github.com/aspnet/EntityFramework/pull/7269 was only blocking for scaffolding support, which isn't very critical. If I remember correctly I already added support for built-in range types (e.g. int4range, daterange...) to the current dev branch, so if you're feeling brave you can try to compile the provider yourself and use ranges (would love to hear about it).
Non-built-in ranges (which you can create with CREATE TYPE) aren't supported yet and would require a more significant refactor in how type mapping works.

Is it shipped with 2.0? Trying to add migration with a field
[Column("length")] public NpgsqlRange<int> Length { get; set; }
but package manager only says "Build failed." and nothing else :S

bump The issue is still open but has milestone 2.0.0. Will it be shipped with the next release or is it going to be delayed?

I didn't manage to release this for 2.0.0, but I do hope to do this for the upcoming 2.1.0 release (for built-in types only at this stage). Anyone who's interested should upvote to let me know you need it, in case I don't have time to do everything on the plan.

That's great news to hear! Looking forward to see 2.1.0 happen!

This is done and will be probably be released with 2.1.0-preview1.
Note that this only includes the 6 built-in range types, and not range types created by the user. It also doesn't map any operations yet - it only allows you to have NpgsqlRange<T> on your entities and have that correctly mapped to the database. For mapping range operations I've opened #311 - it shouldn't be too hard and may even make it for 2.1.

Is it possible to include mappings for NpgsqlRange<float> and NpgsqlRange<double> to numrange?
https://github.com/npgsql/Npgsql.EntityFrameworkCore.PostgreSQL/blob/a67d76659594ae45cc0d9d95459a793e06b2bd45/src/EFCore.PG/Storage/Internal/NpgsqlTypeMappingSource.cs#L97-L105

@austindrenski, PostgreSQL actually supports defining your own range types, and it's very easy to define on over floating-point types (see the example in the PostgreSQL docs).
The only problem is that currently the EF Core provider only supports the built-in types, which don't including the floating-point types (see #329 for adding user-defined ranges and why it's not trivial). In the meantime, I don't think it makes sense to automatically map CLR float/double to PostgreSQL numeric, since these are quite different types, and since one day we may actually support user-defined ranges at which point things will probably break. The conversion between range and range would have to be supported at the Npgsql level, which I really don't want to do.
Besides, with the new value converters coming in 2.1 it should be pretty trivial to write a conversion between the two types...

That makes sense. Thanks for providing some context, much appreciated!

@roji It appears to me that the mapping of DateTimeOffset was forgotten in the implementation, am I right?
I ask because we chose to explicitly map any timestamp to DateTimeOffset in an attempt to avoid timestamp skews caused by typos of DateTime.Now where it should read as DateTime.UtcNow because DateTimeOffset.Now behaves like DateTimeOffset.UtcNow. DateTimeOffset also provides better synergies for integration with grpc / protobuf. With the implementation given we are forced to use DateTime over DateTimeOffset and thus mix concepts in the application code.

@dpsenner first, regarding your decision to use DateTimeOffset in order to avoid timezone issues, I'm not a big fan :) See the following long rants if you're interested: https://github.com/npgsql/Npgsql.EntityFrameworkCore.PostgreSQL/issues/303, https://github.com/npgsql/Npgsql.EntityFrameworkCore.PostgreSQL/issues/473.
Regarding support of ranges over timestamptz, the discussion took place in #561.

Future readers, please ignore my comment above. If you are reading this you should consider to use timestamp over timestamptz along with NodaTime Instants in your model. This makes your application layer responsible for dealing with timezoned information and that's the best place to have that logic. Note that reporting tools deliberately choose to discard any interpretations of timestamptz columns, effectively making them utc timestamps. People should think about how to store and interpret application data that comes from and goes to multiple timezones.


docker run error

why this cmd cannot successfully run

and this problem when I create claim.

Hi @andrewzhang97, please share issuer-API container logs.

Hi @andrewzhang97, please share issuer-API container logs.

here it is for the second problem.

Hi @andrewzhang97 , we tried the latest version of the issuer node with this payload and worked perfectly, let’s make sure that you have the latest version.
{    "credentialSchema":"https://raw.githubusercontent.com/iden3/claim-schema-vocab/main/schemas/json/KYCAgeCredential-v3.json",
    "type": "KYCAgeCredential",
    "credentialSubject": {
        "id": "did:polygonid:polygon:mumbai:2qMZrfBsXuGFTwSqkqYki78zF3pe1vtXoqH4yRLsfs",
        "birthday": 19960424,
        "documentType": 2
    },
    "expiration":<PHONE_NUMBER>
}

Hi @andrewzhang97 , we tried the latest version of the issuer node with this payload and worked perfectly, let’s make sure that you have the latest version.
{    "credentialSchema":"https://raw.githubusercontent.com/iden3/claim-schema-vocab/main/schemas/json/KYCAgeCredential-v3.json",
    "type": "KYCAgeCredential",
    "credentialSubject": {
        "id": "did:polygonid:polygon:mumbai:2qMZrfBsXuGFTwSqkqYki78zF3pe1vtXoqH4yRLsfs",
        "birthday": 19960424,
        "documentType": 2
    },
    "expiration":<PHONE_NUMBER>
}

The problem is solved. I restarted and problem was gone. Thanks for your help. Appreciate.

Hi @andrewzhang97 , Since your problem is solved can you please close this issue?


Getters cannot work as before that in case of multi level nested structs with optional fileds.

Getter cannot be parsed correctly with error Not a tuple.
import "@stdlib/deploy";

struct Point {
    x: Int as int64;
    y: Int as int64;
}

struct Line {
    start: Point;
    end: Point;
}

struct Location {
     idx: Int;
     line: Line?;
}

contract Test with Deployable {

    get fun location(): Location {
        let start: Point = Point{ x: 0, y: 0};
        let end: Point = Point{ x: 1, y: 1};

        return Location {
            idx: 1,
            line: Line {start: start, end: end}
        }
    }
}

import { Blockchain } from "@ton/sandbox";
import { Test } from "../build/Test/tact_Test";
import { toNano } from "@ton/core";

describe('TEST', () => {

    it('1', async () => {
        let blockchain = await Blockchain.create({});

        let deployer = await blockchain.treasury('deployer');
        let test = blockchain.openContract(await Test.fromInit());

        await test.send(
            deployer.getSender(),
            {
                value: toNano('0.1'),
            },
            {
                $$type: 'Deploy',
                queryId: 0n,
            }
        );
        await test.getLocation(); //Not a tuple
    });
});

Can you provide more info please? Are you experiencing this in TypeScript tests? What is your test code then?

If the field line of Location is non optional, the getter will run correctly.

and it works in v1.4.0

I see the problem, turns out that the 1.4.2 fix is incomplete.

@verichenn Thanks a lot for all your bug reports! Really valuable. Btw, we just released Tact v1.4.3 on NPM. Would be great if you could try the new version and share your feedback

@verichenn Thanks a lot for all your bug reports! Really valuable. Btw, we just released Tact v1.4.3 on NPM. Would be great if you could try the new version and share your feedback

Thanks for the update! I'll try it and let you know if I encounter any issues. Appreciate the continued improvements!

Awesome! Thanks for trying it out, really appreciate it


Task Error

I'm getting the following error in the logs because all Apple News tasks are failing.
Any ideas what could be causing this? I can download the content from the articles in the task and preview it in the News Preview app (Just to double check there were no errors with the way the article was formatted)
2017/11/22 10:14:47 [error] [application] Encountered an error running task 63856 (AppleNews_PostQueuedArticles), step 1 of 4: Array
(
    [code] => 404
    [message] => HTTP/1.1 404 Not Found
    [response] => stdClass Object
        (
            [errors] => Array
                (
                    [0] => stdClass Object
                        (
                            [code] => NOT_FOUND
                            [keyPath] => Array
                                (
                                    [0] => articleId
                                )

                            [value] => 206516d9-4401-4b23-8338-5e5e8cf637fa
                        )

                )

        )

)
 in /home/website/craft/plugins/applenews/vendor/chapter-three/apple-news-api/src/PublisherAPI/Base.php on line 197 on line 289 of /home/website/craft/plugins/applenews/vendor/chapter-three/apple-news-api/src/PublisherAPI/Base.php

Is there a stack trace associated with the error in your logs?

This is the full output of the log for this error:

2017/11/29 10:08:50 [info] [application] [Forced] Starting task AppleNews_PostQueuedArticles that has a total of 23 steps.
2017/11/29 10:08:50 [info] [application] [Forced] Starting step 1 of 23 total steps.
2017/11/29 10:08:51 [error] [php] Array
(
[code] => 404
[message] => HTTP/1.1 404 Not Found
[response] => stdClass Object
(
[errors] => Array
(
[0] => stdClass Object
(
[code] => NOT_FOUND
[keyPath] => Array
(
[0] => articleId
)
                        [value] => 206516d9-4401-4b23-8338-5e5e8cf637fa
                    )

            )

    )

)
in /home/website/craft/plugins/applenews/vendor/chapter-three/apple-news-api/src/PublisherAPI/Base.php on line 197 (/home/website/craft/plugins/applenews/vendor/chapter-three/apple-news-api/src/PublisherAPI/Base.php:289)
Stack trace:
#0 /home/website/craft/plugins/applenews/vendor/chapter-three/apple-news-api/src/PublisherAPI/Base.php(197): ChapterThree\AppleNewsAPI\PublisherAPI->triggerError()
#1 /home/website/craft/plugins/applenews/vendor/chapter-three/apple-news-api/src/PublisherAPI/Curl.php(104): ChapterThree\AppleNewsAPI\PublisherAPI->onErrorResponse()
#2 /home/website/craft/plugins/applenews/vendor/chapter-three/apple-news-api/src/PublisherAPI/Curl.php(88): ChapterThree\AppleNewsAPI\PublisherAPI->response()
#3 /home/website/craft/plugins/applenews/vendor/chapter-three/apple-news-api/src/PublisherAPI/Curl.php(242): ChapterThree\AppleNewsAPI\PublisherAPI->request()
#4 /home/website/craft/plugins/applenews/services/AppleNews_ApiService.php(153): ChapterThree\AppleNewsAPI\PublisherAPI->post()
#5 /home/website/craft/plugins/applenews/services/AppleNews_ApiService.php(103): Craft\AppleNews_ApiService->post()
#6 /home/website/craft/plugins/applenews/services/AppleNewsService.php(339): Craft\AppleNews_ApiService->updateArticle()
#7 /home/website/craft/plugins/applenews/tasks/AppleNews_PostQueuedArticlesTask.php(90): Craft\AppleNewsService->postArticle()
#8 /home/website/craft/app/services/TasksService.php(239): Craft\AppleNews_PostQueuedArticlesTask->runStep()
#9 /home/website/craft/app/services/TasksService.php(200): Craft\TasksService->runTask()
#10 /home/website/craft/app/controllers/TasksController.php(63): Craft\TasksService->runPendingTasks()
#11 /home/website/craft/app/framework/web/actions/CInlineAction.php(49): Craft\TasksController->actionRunPendingTasks()
#12 /home/website/craft/app/framework/web/CController.php(308): CInlineAction->runWithParams()
#13 /home/website/craft/app/framework/web/CController.php(286): Craft\TasksController->runAction()
#14 /home/website/craft/app/framework/web/CController.php(265): Craft\TasksController->runActionWithFilters()
#15 /home/website/craft/app/framework/web/CWebApplication.php(282): Craft\TasksController->run()
#16 /home/website/craft/app/etc/web/WebApp.php(823): Craft\WebApp->runController()
#17 /home/website/craft/app/etc/web/WebApp.php(287): Craft\WebApp->_processActionRequest()
#18 /home/website/craft/app/framework/base/CApplication.php(185): Craft\WebApp->processRequest()
#19 /home/website/craft/app/index.php(62): Craft\WebApp->run()
#20 /home/website/public_html/index.php(19): require_once()
REQUEST_URI=/index.php/admin/actions/tasks/runPendingTasks
2017/11/29 10:08:51 [error] [application] Encountered an error running task 64229 (AppleNews_PostQueuedArticles), step 1 of 23: Array
(
[code] => 404
[message] => HTTP/1.1 404 Not Found
[response] => stdClass Object
(
[errors] => Array
(
[0] => stdClass Object
(
[code] => NOT_FOUND
[keyPath] => Array
(
[0] => articleId
)
                        [value] => 206516d9-4401-4b23-8338-5e5e8cf637fa
                    )

            )

    )

)
in /home/website/craft/plugins/applenews/vendor/chapter-three/apple-news-api/src/PublisherAPI/Base.php on line 197 on line 289 of /home/website/craft/plugins/applenews/vendor/chapter-three/apple-news-api/src/PublisherAPI/Base.php

I managed to get this working locally, but putting it on production produced this error. Which I'm unsure why.

@brandonkelly Sorry to nudge you on this (I hate it when people do this to me!) But I've got my client on my back as to why this still isn't working :'(

@bymayo That error is saying that your article ID is invalid. Have you double-checked that?

I'm having a similar issue with failed tasks and craft.log entries that look nearly identical to @bymayo's. Only difference is that my logs showed 22 total steps. After step 3 of 22, the log stops with the following error at the top:
Message Encountered an error running task 9 (AppleNews_PostQueuedArticles), step 3 of 22: Array

I did some digging around in the craft_applenews_articles table and found that the article referenced in step 3 DOES have a record with a status of LIVE, but it is not currently in Apple News (if you try to visit the shareUrl). In the craft_applenews_articlequeue table, that article is the first record in the table.
I'm wondering if deleting the above db records would allow the plugin to continue through the queue?

Additionally, the articles referenced in steps 1 and 2 show a status of PROCESSING in their db record, even though the shareURL value is good. Not sure if that means anything or not.


logging tests fails to build on a number of platforms

./scripts/twister -s tests/subsys/logging/log_benchmark/logging.log_benchmark_v2 -s tests/subsys/logging/log_benchmark/logging.log_benchmark_v2_speed -l
Fails to build on:
ERROR   - segger_trb_stm32f407      tests/subsys/logging/log_benchmark/logging.log_benchmark_v2_speed FAILED: Build failure
ERROR   - see: /home/galak/git/zephyr/twister-out/segger_trb_stm32f407/tests/subsys/logging/log_benchmark/logging.log_benchmark_v2_speed/build.log
INFO    - Total complete:  162/ 658  24%  skipped:   42, failed:    1
ERROR   - ip_k66f                   tests/subsys/logging/log_benchmark/logging.log_benchmark_v2_speed FAILED: Build failure
ERROR   - see: /home/galak/git/zephyr/twister-out/ip_k66f/tests/subsys/logging/log_benchmark/logging.log_benchmark_v2_speed/build.log
INFO    - Total complete:  187/ 658  28%  skipped:   42, failed:    2
ERROR   - decawave_dwm1001_dev      tests/subsys/logging/log_benchmark/logging.log_benchmark_v2_speed FAILED: Build failure
ERROR   - see: /home/galak/git/zephyr/twister-out/decawave_dwm1001_dev/tests/subsys/logging/log_benchmark/logging.log_benchmark_v2_speed/build.log
INFO    - Total complete:  194/ 658  29%  skipped:   42, failed:    3
ERROR   - bt510                     tests/subsys/logging/log_benchmark/logging.log_benchmark_v2_speed FAILED: Build failure
ERROR   - see: /home/galak/git/zephyr/twister-out/bt510/tests/subsys/logging/log_benchmark/logging.log_benchmark_v2_speed/build.log
INFO    - Total complete:  197/ 658  29%  skipped:   42, failed:    4
ERROR   - bl654_dvk                 tests/subsys/logging/log_benchmark/logging.log_benchmark_v2_speed FAILED: Build failure
ERROR   - see: /home/galak/git/zephyr/twister-out/bl654_dvk/tests/subsys/logging/log_benchmark/logging.log_benchmark_v2_speed/build.log
INFO    - Total complete:  202/ 658  30%  skipped:   42, failed:    5
ERROR   - bl652_dvk                 tests/subsys/logging/log_benchmark/logging.log_benchmark_v2_speed FAILED: Build failure
ERROR   - see: /home/galak/git/zephyr/twister-out/bl652_dvk/tests/subsys/logging/log_benchmark/logging.log_benchmark_v2_speed/build.log
INFO    - Total complete:  237/ 658  36%  skipped:   42, failed:    6
ERROR   - usb_kw24d512              tests/subsys/logging/log_benchmark/logging.log_benchmark_v2_speed FAILED: Build failure
ERROR   - see: /home/galak/git/zephyr/twister-out/usb_kw24d512/tests/subsys/logging/log_benchmark/logging.log_benchmark_v2_speed/build.log
INFO    - Total complete:  264/ 658  40%  skipped:   42, failed:    7
ERROR   - hexiwear_kw40z            tests/subsys/logging/log_benchmark/logging.log_benchmark_v2_speed FAILED: Build failure
ERROR   - see: /home/galak/git/zephyr/twister-out/hexiwear_kw40z/tests/subsys/logging/log_benchmark/logging.log_benchmark_v2_speed/build.log
INFO    - Total complete:  385/ 658  58%  skipped:   42, failed:    8
ERROR   - segger_trb_stm32f407      tests/subsys/logging/log_benchmark/logging.log_benchmark_v2 FAILED: Build failure
ERROR   - see: /home/galak/git/zephyr/twister-out/segger_trb_stm32f407/tests/subsys/logging/log_benchmark/logging.log_benchmark_v2/build.log
INFO    - Total complete:  495/ 658  75%  skipped:   42, failed:    9
ERROR   - ip_k66f                   tests/subsys/logging/log_benchmark/logging.log_benchmark_v2 FAILED: Build failure
ERROR   - see: /home/galak/git/zephyr/twister-out/ip_k66f/tests/subsys/logging/log_benchmark/logging.log_benchmark_v2/build.log
INFO    - Total complete:  513/ 658  77%  skipped:   42, failed:   10
ERROR   - decawave_dwm1001_dev      tests/subsys/logging/log_benchmark/logging.log_benchmark_v2 FAILED: Build failure
ERROR   - see: /home/galak/git/zephyr/twister-out/decawave_dwm1001_dev/tests/subsys/logging/log_benchmark/logging.log_benchmark_v2/build.log
INFO    - Total complete:  522/ 658  79%  skipped:   42, failed:   11
ERROR   - bt510                     tests/subsys/logging/log_benchmark/logging.log_benchmark_v2 FAILED: Build failure
ERROR   - see: /home/galak/git/zephyr/twister-out/bt510/tests/subsys/logging/log_benchmark/logging.log_benchmark_v2/build.log
INFO    - Total complete:  531/ 658  80%  skipped:   42, failed:   12
ERROR   - bl654_dvk                 tests/subsys/logging/log_benchmark/logging.log_benchmark_v2 FAILED: Build failure
ERROR   - see: /home/galak/git/zephyr/twister-out/bl654_dvk/tests/subsys/logging/log_benchmark/logging.log_benchmark_v2/build.log
INFO    - Total complete:  535/ 658  81%  skipped:   42, failed:   13
ERROR   - bl652_dvk                 tests/subsys/logging/log_benchmark/logging.log_benchmark_v2 FAILED: Build failure
ERROR   - see: /home/galak/git/zephyr/twister-out/bl652_dvk/tests/subsys/logging/log_benchmark/logging.log_benchmark_v2/build.log
INFO    - Total complete:  565/ 658  85%  skipped:   42, failed:   14
ERROR   - usb_kw24d512              tests/subsys/logging/log_benchmark/logging.log_benchmark_v2 FAILED: Build failure
ERROR   - see: /home/galak/git/zephyr/twister-out/usb_kw24d512/tests/subsys/logging/log_benchmark/logging.log_benchmark_v2/build.log
INFO    - Total complete:  596/ 658  90%  skipped:   42, failed:   15
ERROR   - hexiwear_kw40z            tests/subsys/logging/log_benchmark/logging.log_benchmark_v2 FAILED: Build failure
ERROR   - see: /home/galak/git/zephyr/twister-out/hexiwear_kw40z/tests/subsys/logging/log_benchmark/logging.log_benchmark_v2/build.log
INFO    - Total complete:  654/ 658  99%  skipped:   42, failed:   16

Guessing these might all be platforms that utilize CONFIG_USE_SEGGER_RTT=y / CONFIG_RTT_CONSOLE=y by default.


Improve performance of brew/installed

Currently brew/installed is written in Python, and it's called about a hundred times in brew/init, and by extension provision. Of course Python isn't known for startup speed, especially not if the re module is being imported. I need to rewrite this in shell.

Turns out I'm wrongly blaming Python. The culprit is brew.


Prototype Update - Processing Team Queue

Title of ticket: Prototype Update - Processing Team Queue
Description
Prototype Update - Processing Team Queue
DOD

[x] Update UI for Processing Team Queue based on Sessions and Jamies Feedback
[x] Have SME & PO review
[ ] Circle back with Jamie for feedback

@m-prodan @arielleandrews Could you have a look at this updated design when you have a chance and let me know if you have any changes: https://jacklyn808742.invisionapp.com/console/share/3W2O64AP5C/846624085

Looks great to me - I'll get started on the AC. Number of pages may be dependent on AXIS ETL, so I will mention in the AC that it should read as N/A for now. We could also look at adding a field for IAO to manually enter the info into the review request page if there's an issue with pulling that number from AXIS

Good catch @m-prodan I don't think we have that built into the system yet, we should add a ticket for how this will be flagged.
We will also have to circle back to the creating multiple requests at once in intact which would be a flagged for XGOV along with request coming into the system with multiple ministries selected

Late to the party but noting for teams heavy on personal requests, we may hear a want to see the "on behalf of" field displayed. An example is some lawyers/law firms will submit requests on behalf of multiple clients. In that case the applicant name would display as the lawyer's name and you would not have an easy way to differentiate the requests in your queue aside from the ID number

Good one to flag... I wonder if we can do something similar to what we were thinking for request description, for example: when an analyst hovers over a law firm/OBO app name, it shows the OBO name in a little pop up over the mouse pointer.
Alternatively we could just set up a different queue for personals heavy teams (Social Tech, Justice Health, and CFD)

@arielleandrews good one to flag, do we think we would want to just replace the request type column with on behalf of? (for personal focused teams) Or add it?

@JHarrietha-AOT I would say add that column. The Social Tech and Justice Health teams both have a pretty even split of generals vs personals so they may still find the request type column beneficial. CFD Personals is the only team that wouldn't need it since all their requests are personal

Sounds good, I'll add. Thanks Arielle. Will share a link her shortly with the update

@arielleandrews Updated: https://jacklyn808742.invisionapp.com/console/share/3W2O64AP5C/846624085

@arielleandrews  Thanks for reviewing, I have updated columns accordingly https://jacklyn808742.invisionapp.com/console/share/3W2O64AP5C/846624085


Update validators.py

Recaptcha not always returns error-codes and flask-wtf might crash with such responses.
Here's sample response that I got today:
[console ready]

json_resp
{u'success': False}

KeyError: 'error-codes'

You should raise a validation error if {u'success': False}

I followed current logic - if error is not in the list, it'll skip to return False. Updated pull request so it'll raise ValidationError instead.

@mrjoes Sorry. My bad. It has a return False at the end.

I have cherry picked your commit: https://github.com/lepture/flask-wtf/commit/8879e9827b0160776031bd06ef083b81282904e5


Add mark_step only for inference

Add mark_step only for inference to split the graphs so that it fits in memory

Unmatched PR


Retirement of ISE for Logic Apps

[Enter feedback here]
The documentation references setting up Logic Apps using ISE as a security baseline best practice, however it has an upcoming retirement as detailed here https://learn.microsoft.com/en-us/azure/logic-apps/ise-manage-integration-service-environment.
Suggesting that updates are made to take into account this factor.

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

ID: 90f74d10-6e09-34ca-c20e-7dd9098558a3
Version Independent ID: 10c11873-f94b-bc9e-78e8-1668137fd691
Content: Azure security baseline for Logic Apps
Content Source: benchmark/azure/baselines/logic-apps-security-baseline.md
Service: logic-apps
GitHub Login: @msmbaldwin
Microsoft Alias: mbaldwin

@RossCavanagh
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.


Fix opencover-nunit when nunit not cached

Does that work with parallel builds ? (seems to be goind a bit backwards from #24 no ?) cc @reeflog

Depending of where is called "getNunitFolder()". For instance, if it's used in a closure in the plugin configuration like it was the case before. Looking at the code I don't expect that anymore.
That being said, I'm not feeling confident with having a simple get with such side effect (downloading NUnit). A alternative could be to download explictly NUnit from OpenCover. I'm not sure it's better.
So, it looks good enough for me.

maybe renaming getNunitFolder to getCachedOrDownloadNUnit or something like that would make the trick ? (because changing opencover doesn't seem good either)

Well, isn't it easier/better to just have something like opencover.dependsOn nunit?

opencover runs instead-of nunit (no need to run UTs twice), so it can't depend I guess

Maybe we could do something like the jacoco plugin does to the standard
test task. IIRC it customizes it to specify jacoco specific stuff on the
jvm args
Le mar. 17 mai 2016 16:57, Francois Valdy<EMAIL_ADDRESS>a
écrit :

opencover runs instead-of nunit (no need to run UTs twice), so it can't
depend I guess
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/Ullink/gradle-nunit-plugin/pull/28#issuecomment-219744452

Something like that: https://github.com/gradle/gradle/blob/master/subprojects/jacoco/src/main/java/org/gradle/testing/jacoco/plugins/JacocoPluginExtension.java#L100


Series of Prompts

Series of Prompts
User Story
AS AN employee with access to sensitive data,
I WANT to randomly generate a password that meets certain criteria,
SO THAT I can create a strong password that provides greater security.
Acceptance Criteria
GIVEN I need a new, secure password,
WHEN I click the button to generate a password,
THEN I am presented with a series of prompts for password criteria.

Complete. See https://nestibry.github.io/umn-bcc-mod03-password-generator/


[Bug]: When choosing a scene as an entity, the toggle action will fail since there is no on/off available

Current Behavior
Failed calling the service scene/turn_off. Service not found. [Might be different. Using HA in another language]
Expected Behavior
The Scene should turn on [off is not available inside of HA]
Steps To Reproduce

create Entity Chip
use a scene as entity
select toggle as action

Environment
- OS: 			Home Assistant OS, aarch64 5.10.103-v8
- Version:		2022.5.1
- Python-Version:	3.9.9
- mushroom-version:	1.7.3

Anything else?
No response

Hello !
Scenes can't be turned off but design. You need to call the turn_on service in the actions instead of the toggle action 🙂


support raw buffers with {encoding: null}

Overview
Allow the use of raw Buffer objects with node-nats, by passing {encoding: null} as an explicit option. Note that the default encoding, UTF8, is not changed.

Whoops - this is supposed to be on a fork, not the original project. Closing.

Coverage decreased (-0.2%) to 95.05% when pulling 315a6dec382d6637821eae2422f6dbb127f33799 on autopilothq:direct-buffer-access into d277cab2b7a9ec00a8ca9075bfa53d700ed572db on nats-io:master.

Coverage decreased (-0.2%) to 95.05% when pulling 315a6dec382d6637821eae2422f6dbb127f33799 on autopilothq:direct-buffer-access into d277cab2b7a9ec00a8ca9075bfa53d700ed572db on nats-io:master.

Coverage decreased (-0.2%) to 95.05% when pulling 315a6dec382d6637821eae2422f6dbb127f33799 on autopilothq:direct-buffer-access into d277cab2b7a9ec00a8ca9075bfa53d700ed572db on nats-io:master.


URL detection bug

URLs that are not actually valid protocols (let's say owoWhatsThis://) get marked as URLs.
Perhaps use a JSON file of valid schemes?

Although it might not be a URL, it certainly is a URI. And according to the W3C or RFC3986 spec, this might as well be a valid protocol in the future.

this might as well be a valid protocol in the future.

Would be funny...

We can only hope


Refactor logging functionality

Need to verify the changes aftect the logging function in Jenkins plugin.

Need to verify the changes aftect the logging function in Jenkins plugin.

Need to verify the changes aftect the logging function in Jenkins plugin.

After the change, it works as usual in the Jenkins plugin


Updated Alert.js

Added Margin
Closes #8277
Check List (Check all the applicable boxes)

[X] My code follows the code style of this project.
[X] This PR does not contain plagiarized content.
[X] The title of my pull request is a short description of the requested changes.

Screenshots

Thank you for your contribution, but this issue is not assigned to you? https://github.com/EddieHubCommunity/LinkFree/issues/8277
Please read our contributing guide to learn how to contribute

@eddiejaoude Sorry for that , I'll not repeat this next time .

Thank you for understanding @parth-ranalkar 👍 you can filter issues by "not assigned"


[#11] [Feature][BackEnd] As a User, I can submit the login form and see the status of my login request

#11

What happened 👀

[x] Configure Client ID & Secret (on API + in app secrets + local env setup) -- move deploy to GitHub Actions
[x] Create oauth API endpoint adapter
[ ] Expose the adapter & handle the response (check where to store the token!)
[ ] Add unit tests
[ ] Add UI tests

Insight 📝
I moved the Netlify configure to run with GitHub Actions

More freedom in setting different ENV variables based on the branch building the package
Only 1 place to maintain ENV variables (as GitHub also needs these variables for tests)

Proof Of Work 📹
WIP

Closing as I would start a new Repository if I take this IC again.