Command line args should completely override corresponding yaml entries - globally

Example:
Plan has a script, like in plans/helps.fmf.
And when I run
tmt run -a prepare -h ansible -p ...... plan --name helps

At this point, I'll also get script in data (in Prepare step). I can ignore it (which would be unfortunate); or I will throw an error, or resolve it somehow...
Currently this has to be solved for all plugins and steps individually, which I do not thing is a good, as it can be solved globally. When user specifies --how on commandline, I expect not to get (in the step data) the corresponding entries present in L2 metadata, same as I do not get the --how  from L2 metadata.

@happz, for reference, this might be related to #2188.

I'm not sure I follow. Can we get a more verbose reproducer, please?

Sorry, the bug is so old the context is all lost. What I could recover was:
summary:
    Check help messages
discover:
    how: shell
    tests:
    - name: /help/main
      test: tmt --help
    - name: /help/test
      test: tmt test --help
    - name: /help/plan
      test: tmt plan --help
    - name: /help/smoke
      test: ./smoke.sh
      path: /tests/shell
prepare:
    how: shell
    script:
      - sudo dnf copr enable -y psss/tmt
      - sudo dnf install -y tmt
execute:

Without overrides:
tmt run -a -n -vvv
/var/tmp/tmt/run-005
    warn: /plans/helps:execute - None is not valid under any of the given schemas
Found 1 plan.

/plans/helps
summary: Check help messages
    discover
        how: shell
        order: 50
        summary: 4 tests selected
            /help/main
            /help/test
            /help/plan
            /help/smoke
    provision
        how: virtual
        order: 50
        user: root
        image: fedora
        memory: 2048 MB
        disk: 40 GB
        connection: session
        arch: x86_64
        list_local_images: False
        multihost name: default-0
        summary: 1 guest provisioned
    prepare
        queued push task #1: push on default-0

        push task #1: push on default-0

        queued prepare task #1: default-0 on default-0

        prepare task #1: default-0 on default-0
        how: shell
        order: 50
        overview: 2 scripts found
        script: sudo dnf copr enable -y psss/tmt
            cmd: set -eo pipefail; sudo dnf copr enable -y psss/tmt
        script: sudo dnf install -y tmt
            cmd: set -eo pipefail; sudo dnf install -y tmt

        queued pull task #1: pull on default-0

        pull task #1: pull on default-0

        summary: 1 preparation applied
    execute
        queued execute task #1: default-0 on default-0

        execute task #1: default-0 on default-0
        how: tmt
        order: 50
        exit-first: False

        summary: 0 tests executed
    report
        how: display
        order: 50
        summary: no results found
    finish

        summary: 0 tasks completed

total: no results found

With the override:
$ tmt run -a -n -vvv prepare -h ansible -p ansible.yml
/var/tmp/tmt/run-003
    warn: /plans/helps:execute - None is not valid under any of the given schemas
Found 1 plan.

/plans/helps
summary: Check help messages
    discover
        how: shell
        order: 50
        summary: 4 tests selected
            /help/main
            /help/test
            /help/plan
            /help/smoke
    provision
        how: virtual
        order: 50
        user: root
        image: fedora
        memory: 2048 MB
        disk: 40 GB
        connection: session
        arch: x86_64
        list_local_images: False
        multihost name: default-0
        summary: 1 guest provisioned
    prepare
        queued push task #1: push on default-0

        push task #1: push on default-0

        queued prepare task #1: default-0 on default-0

        prepare task #1: default-0 on default-0
        how: ansible
        order: 50
        playbook: ansible.yml

        queued pull task #1: pull on default-0

        pull task #1: pull on default-0

        summary: 1 preparation applied
    execute
        queued execute task #1: default-0 on default-0

        execute task #1: default-0 on default-0
        how: tmt
        order: 50
        exit-first: False

        summary: 0 tests executed
    report
        how: display
        order: 50
        summary: no results found
    finish

        summary: 0 tasks completed

total: no results found

Now prepare has only has one preparation - the ansible one, dropping the original one from the plan.
I think the problem is fixed, closing. Thanks!


addworker notification for preventing goroutines leak

Hi, came across the fact that many of our applications have started to notice goroutine leaks. Found out that
the reason is that we initialize the HTTPSyncTransport object many times in so-called handlers:
github.com/getsentry/sentry-go.(*HTTPTransport).worker(0xc0000000)

goroutine t.worker leaks in our case.
Using blocking HTTPSyncTransport is not beneficial for us and spoils latency.
You can take a look at the solution, thanks to which we managed to reduce the number of running goroutines by orders of magnitude!
Seems like it could be a solution for 731 issue
Thanks for your product anyway.

I'm testing this locally and tried a couple of my own solutions as well, but there's still a leaked goroutine when HTTPTransport is flushed.
func TestHTTPTransportDoesntLeakGoroutines(t *testing.T) {
defer goleak.VerifyNone(t)
fmt.Println("NumGoroutine before sending events:", runtime.NumGoroutine())

transport := NewHTTPTransport()
transport.Configure(ClientOptions{
	Dsn:        "https://test@foobar/1",
	HTTPClient: http.DefaultClient,
})

// Add logging to monitor worker termination
fmt.Println("Starting Flush")
if !transport.Flush(testutils.FlushTimeout()) {
	t.Error("Flush failed")
}
fmt.Println("Flush completed")

fmt.Println("NumGoroutine after sending events:", runtime.NumGoroutine())

}
Both goleak detect this (worker remains running) and the number of goroutines (2 before and 3 after Flush).

I'm testing this locally and tried a couple of my own solutions as well, but there's still a leaked goroutine when HTTPTransport is flushed.
...

Both goleak detect this (worker remains running) and the number of goroutines (2 before and 3 after Flush).

Yes, I assumed that when calling the Flush() method we should not interrupt the worker, because it should continue to do its own things. And we can give an opportunity to notify transport from outside the package that we don't need it anymore. something like that:
func TestHTTPTransportDoesntLeakGoroutines(t *testing.T) {
	defer goleak.VerifyNone(t)

	fmt.Println("NumGoroutine before sending events:", runtime.NumGoroutine())

	doneCh := make(chan struct{})
	transport := NewHTTPTransport()
	transport.Configure(ClientOptions{
		Dsn:        "https://test@foobar/1",
		HTTPClient: http.DefaultClient,
		Done: doneCh,
	})

	doneCh <- struct{}{} //comment this line to catch a leak

	fmt.Println("NumGoroutine after sending events:", runtime.NumGoroutine())
}

Another option is to hide this logic behind a new method HTTPTransport.Close() but it seems unnecessary.

This makes sense. The only thing I'm unsure about is whether we should pre-set the done channel and expose a Close() or End() method on HTTPTransport. That seems more readable than users having to know that they need to create, pass and write to channel to stop the worker?

That seems more readable than users having to know that they need to create, pass and write to channel to stop the worker?

yeah, looks a robust and easy to understand with Close method. thats ill gonna do.

I added changes to changelog and the leak detection test. Locally passes fine, here it reports there is a leak.

I think we should also add a method to the client, similar to other SDKs that provide a close method.

Add Close method for client and fix tests, there are problems in other tests where transport was not closed properly. And there was not only worker leaks btw.

@manc88 Anything else you'd like to add? If not ,we can merge this.

I'm merging this now and will follow up adding support for a timeout similar to flush. Thanks for your work @manc88!

@cleptric @ribice  thank you


adds cache entry point to strapi.middleware

Hello everyone,
Since I have had a similar issue, I thought I would help out and implement the idea discussed in #27 (closes #27)
If you have any suggestions or would like to change/rename something let me know.
Thanks for the great work @patrixr

@patrixr quick question. should I have a look at the dependencies and see if there are new versions available?

@patrixr quick question. should I have a look at the dependencies and see if there are new versions available?

@meck93 We can probably do that separately. I'll merge this PR as it is

@patrixr sounds good to me. I'll have a look at the non-breaking dependency changes.


TypeError AsyncChromiumLoader.init() got an unexpected keyword argument 'headless'

Describe the bug
I get a TypeError when trying to run the script examples indicating that AsyncChromiumLoader.init() got an unexpected keyword argument 'headless'
To Reproduce
from scrapegraphai.graphs import SmartScraperGraph
OPENAI_API_KEY = "YOUR_API_KEY"
graph_config = {
"llm": {
"api_key": OPENAI_API_KEY,
"model": "gpt-3.5-turbo",
},
}
smart_scraper_graph = SmartScraperGraph(
prompt="List me all the articles",
# also accepts a string with the already downloaded HTML code
source="https://perinim.github.io/projects",
config=graph_config
)
result = smart_scraper_graph.run()
print(result)
Desktop (please complete the following information):

OS: Windows
Browser Chrome (using Jupyter Notebook)

Additional context
I have very little programming skills

try with this config:
graph_config = {
"llm": {
"api_key": OPENAI_API_KEY,
"model": "gpt-3.5-turbo",
},
"headless": "false",

}

and if you are using a Jupyter notebook take reference from here:
https://colab.research.google.com/drive/1sEZBonBMGP44CtO6GQTwAlL0BGJXjtfd?usp=sharing


Add expressions for the rogue-like brogue and libtcod

Hi,
I added nix expressions for brogue and its dependency libtcod.
Best,
Sven

Good point, I will take a look at it.

The obvious approach is to copy the stuff into $out/share/brogue instead and then generate a wrapper that cd's into that dir and executes the binary.

I moved the fonts directory to $out/share/brogue and it works. Brogue does not need the other files to run.

@joachifm, how about that?

Thank you, pushed to master at 98bef7e0bde3b7b9cc4deff282c8863fc5b504a5


VCF annotation question

Hi there,
I would like to use ADAM to annotate VCF file. I have not found any example how to do so in the docs, it's why I'm writing here. I'd like to read a VCF file, add some annotations (for example new INFO fields with according header lines) and save it back to VCF.
Is it possible?
Thank you in advance,
Piotr

Hello @pstawinski!
Adding new header lines is straightforward
import org.bdgenomics.adam.rdd.ADAMContext._
import htsjdk.variant.vcf.VCFHeaderLineType

val vcs = sc.loadVcf("sample.vcf")

val vcsWithHeaderLine = vcs.addScalarInfoHeaderLine(
  "key", "Custom key INFO field", VCFHeaderLineType.String)

vcsWithHeaderLine.saveAsVcf("sample-info-key.vcf")

See
http://static.javadoc.io/org.bdgenomics.adam/adam-core-spark2_2.11/0.24.0/index.html#org.bdgenomics.adam.rdd.variant.VariantContextRDD
The code for adding new attributes looks different if you're using RDDs or Datasets, which are you working with?

Support of ADAM is really amazing! Thank you heuermh!
I prefer to work with Datasets.

ok, with Datasets it is a little bit verbose, because currently the classes generated to the org.bdgenomics.adam.sql package are Scala Products but not case class (see https://github.com/bigdatagenomics/adam/pull/1972)
import htsjdk.variant.vcf.VCFHeaderLineType

import org.bdgenomics.adam.rdd.ADAMContext._
import org.bdgenomics.adam.sql.Variant
import org.bdgenomics.adam.sql.VariantAnnotation
import org.bdgenomics.adam.sql.VariantContext

val vcs = sc.loadVcf("sample.vcf")

val vcsWithHeaderLine = vcs.addScalarInfoHeaderLine(
  "key", "Custom key INFO field", VCFHeaderLineType.String)

def addInfo(vc: VariantContext): VariantContext = {
  vc.variant.annotation.fold(vc)(annotation => {
    // add new attribute
    val attributes = annotation.attributes + ("key" -> "value")

    // copy to new annotation
    val annotationCopy = new VariantAnnotation(
      alleleCount = annotation.alleleCount,
      alleleFrequency = annotation.alleleFrequency,
      ancestralAllele = annotation.ancestralAllele,
      attributes = attributes,
      cigar = annotation.cigar,
      dbSnp = annotation.dbSnp,
      forwardReadDepth = annotation.forwardReadDepth,
      hapMap2 = annotation.hapMap2,
      hapMap3 = annotation.hapMap3,
      readDepth = annotation.readDepth,
      referenceForwardReadDepth = annotation.referenceForwardReadDepth,
      referenceReadDepth = annotation.referenceReadDepth,
      referenceReverseReadDepth = annotation.referenceReverseReadDepth,
      reverseReadDepth = annotation.reverseReadDepth,
      somatic = annotation.somatic,
      thousandGenomes = annotation.thousandGenomes,
      transcriptEffects = annotation.transcriptEffects,
      validated = annotation.validated
    )

    // copy to new variant
    val variantCopy = new Variant(
      alternateAllele = vc.variant.alternateAllele,
      annotation = Some(annotationCopy),
      contigName = vc.variant.contigName,
      end = vc.variant.end,
      filtersApplied = vc.variant.filtersApplied,
      filtersFailed = vc.variant.filtersFailed,
      filtersPassed = vc.variant.filtersPassed,
      names = vc.variant.names,
      quality = vc.variant.quality,
      referenceAllele = vc.variant.referenceAllele,
      splitFromMultiAllelic = vc.variant.splitFromMultiAllelic,
      start = vc.variant.start
    )

    // copy to new variant context
    vc.copy(variant = variantCopy)
  })
}

val vcsWithAttribute = vcsWithHeaderLine.transformDataset(dataset => dataset.map(addInfo))

vcsWithAttribute.saveAsVcf("small-info-key.vcf")

Thanks a million, Michael!

FYI, now that #1972 has been merged, on git head this is a lot shorter
import htsjdk.variant.vcf.VCFHeaderLineType

import org.bdgenomics.adam.rdd.ADAMContext._
import org.bdgenomics.adam.sql.Variant
import org.bdgenomics.adam.sql.VariantAnnotation
import org.bdgenomics.adam.sql.VariantContext

val vcs = sc.loadVcf("adam-core/src/test/resources/small.vcf")

val vcsWithHeaderLine = vcs.addScalarInfoHeaderLine(
  "key", "Custom key INFO field", VCFHeaderLineType.String)

def addInfo(vc: VariantContext): VariantContext = {
  vc.variant.annotation.fold(vc)(annotation => {
    val attributes = annotation.attributes + ("key" -> "value")
    val annotationCopy = annotation.copy(attributes = attributes)
    val variantCopy = vc.variant.copy(annotation = Some(annotationCopy))
    vc.copy(variant = variantCopy)
  })
}

val vcsWithAttribute = vcsWithHeaderLine.transformDataset(dataset => dataset.map(addInfo))

vcsWithAttribute.saveAsVcf("small-info-key.vcf")


Control where Kubevirt pulls disk iamges from

Is this a BUG REPORT or FEATURE REQUEST?:
/kind enhancement
Question:
Is there a recommended way of restricting/controlling the images that can be used? In my organization we would want to limit images to be deployed from a certain registry only. I noticed there is an admission control webhook. Could this be used or extended? Would Open Policy Agent be preferred?

@cjreyn a pod webhook would probably work. It may look a little bit strange for users, since they would see a successful VMI creation and then see warning events that the image is not accepted when the VMI is scheduled.
What exactly is k8s doing in that area?

I installed Open Policy Agent to test this. It provides a framework to programmatically define policy based upon the admission control webhooks. I tested it rejecting pods based upon the image name which worked fine. From what I can tell, it would work fine with Custom Resources too.
I believe that OPA wouldn't interfere with the existing admission control webhook that Kubevirt provides, since all webhooks are called so OPA will definitely run.
For my use case, OPA looks the best bet.

Great. Let us know if there is anything which you would needed changed in kubevirt to make it work seamlessly with Open Policy Agent.

/close
So to summarize, we don't have any preferred ways on how admins block unwanted content. Open Policy Agent is definitely fine from a KubeVirt perspective. @cjreyn if that answer is not good enough for you, feel free to reopen the issue.


chore(deps): update helm release actions-runner-controller to v0.22.0

This PR contains the following updates:

Package
Update
Change

actions-runner-controller
minor
0.21.0 -> 0.22.0

Release Notes

actions/actions-runner-controller
v0.22.0
Compare Source
A Kubernetes controller that operates self-hosted runners for GitHub Actions on your Kubernetes cluster.
v0.21.1
Compare Source
A Kubernetes controller that operates self-hosted runners for GitHub Actions on your Kubernetes cluster.

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Hibernate 5.3.7 compatibility

Hi,
I'm currently trying to migrate a Spring Boot App to SB 2.1.0.RELEASE using hibernate 5.3.7.
I'm using org.jadira.usertype:usertype.core:7.0.0.CR1
with spring.jpa.properties.jadira.usertype.autoRegisterUserTypes=true
My DateTime fields are not auto mapped to jadira anymore
(adding @Type(type="org.jadira.usertype.dateandtime.joda.PersistentDateTime") manually still works though)
Am I missing something or is this version not compatible with the latest hibernate version?

After some investigations, the registration of custom types seems OK.
I see it in the SessionFactory.
The problem occurs on an Embedded type
It is maybe related to https://hibernate.atlassian.net/browse/HHH-7310 ?

Issue created here https://hibernate.atlassian.net/browse/HHH-13088 with test case.
Thx!

@Xyaren The issue was only in embedded class in my case and I raised an issue in hibernate mentioned above. This one is still open...


grpcutil: Convert Resolver into concrete type

What this PR does:
Convert grpcutil.Resolver into concrete type instead of interface. Requested by @pstibrany.
Which issue(s) this PR fixes:
Checklist

[na] Tests updated
[x] CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

This makes sense to me. I don't think we should define interface for single-implementation. Projects needing such interface can easily do that. But let's see what others think.

@pstibrany I agree that we shouldn't expose interfaces in this library unless for good reason.


update: filters.md

Its confusing on quick glances.

The reason it is worded that way is to clarify what the default values are.

Thanks for the clarification :sweat_smile:


panic: runtime error: invalid memory address or nil pointer dereference

Getting issue while using xk6-mqtt with influxdb:
panic: runtime error: invalid memory address or nil pointer dereference [signal SIGSEGV: segmentation violation code=0x1 addr=0x28 pc=0x9bcbc0] goroutine 97 [running]: github.com/mstoykov/atlas.(*Node).Path(0xc001174100)         github.com/mstoykov/atlas@v0.0.0-20220811071828-388f114305dd/atlas.go:77 +0x20 go.k6.io/k6/metrics.(*TagSet).Map(...)         go.k6.io/k6@v0.41.0/metrics/tags.go:74 go.k6.io/k6/output/influxdb.(*Output).batchFromSamples(0xc00230a100, {0xc001518200, 0x10, 0x11a1720})         go.k6.io/k6@v0.41.0/output/influxdb/output.go:129 +0x2fc go.k6.io/k6/output/influxdb.(*Output).flushMetrics.func1()         go.k6.io/k6@v0.41.0/output/influxdb/output.go:203 +0x12a created by go.k6.io/k6/output/influxdb.(*Output).flushMetrics         go.k6.io/k6@v0.41.0/output/influxdb/output.go:195 +0x12f
Code:
Scenario file:
import {Counter,  Trend } from 'k6/metrics';
const mqtt = require('k6/x/mqtt');
let connectPub_count = new Counter('connectPub_count');
let publish_count = new Counter('publish_count');
let publish_trend = new Trend('publish_time', true);
.
.
.
export function connectPub(k6PubId ,vus_connections) {
let pub_client, err_pub_client
if (k6PubId in vus_connections) {
pub_client = vus_connections[k6PubId];
} else {
try {
let pub_client = mqtt.Client([MQTThost], user, password, cleanSessionSetting, k6PubId, timeout,)
pub_client.connect()
connectPub_count .add(1)
vus_connections[k6PubId] = pub_client;
} catch (error) {
err_pub_client = error;
console.log("connect error:", error)
}
}
check(err_pub_client, { "Connect to topic": err => err === undefined });
return pub_client
}
.
.
.
export function publish(pub_client, k6Message) {
let err_publish;
let startTime = new Date().getTime();
try {
pub_client.publish(k6Topic, QoS, k6Message, retainPolicy, timeout);
publish_trend.add(new Date().getTime() - startTime);
} catch (error) { err_publish = error }
publish_count.add(1)
check(err_publish, { "Sending through MQTT": err => err === undefined });
}
Main load file:
import { randomPayload } from "./common/helper.js";
import {Gauge} from 'k6/metrics';
.
.
.
let vus_connections = {};
let mqtt_count = new Gauge('mqttCount');
.
.
.
export function mqttFlow() {
let k6Message = randomPayload();
const k6PubId = pub-${__VU};
let pub_client = mqtt.connectPub(k6PubId, vus_connections)
vus_connections[k6PubId] = pub_client
mqtt.publish(pub_client, k6Message)
mqtt_count.add(ENV.DEVICE_COUNT_MQTT_1);
}
When I disable INFLUX everything works fine

Thanks for the report do you have any chance a simple script linux or macos that I Can run to reproduce thé issue ?

i run my script with docker-compose
docker-compose -f .\docker-compose-test.yml up

I pushed a fix you can try it I'll push a tag if it works for you


Improved logic for events fetching, reworked provider init, fixed logic for storing blockchain events in operational db

I don't see any logic for handling last checked block number when contract addresses change maybe it's in the old code. Would it make sense to expand last checked block logic to be tied to contract rather than whole chain so we can handle last checked block easier?


Make dynamic_decode() TFLite-convertible

This PR adds some logics to make dynamic_decoder() TFLite-convertible.
Hi. I am Jae from TensorFlow Lite team.
Because TFLite doesn't support dynamic shape of TensorArray yet, it changes the TensorArray's into static shape.
Also, 2-D TensorArray is not supported, so some reshaping at the input & output tensors is required.
Because TFLite assumes batch_size = 1 at the inference, we can simply implement the above.
Thank you.

Is this documented somewhere in tflite?

Is this documented somewhere in tflite?

The documentation is in google internal issue tracker. We are preparing for public documentation as a TF blog post. Please stay tuned. Thank you for your interest.

Thanks for the change. Is there a way to add a unit test running the TF Lite converter on this function?

Thanks for the change. Is there a way to add a unit test running the TF Lite converter on this function?

I think the unit tests can be added after the new tf is released. Currently, this function is only working with tf-nightly. In this case, what should I do? May I open a new PR to push a colab notebook to show how to convert a model with dynamic_decode() into a TFLite file?

You can add an unit test like https://github.com/seanpmorgan/addons/pull/15 then you can share a colab.

@qlzh727 Do you have any comment on this one?

@qlzh727 Do you have any comment on this one?

Sorry for the late reply. Functional wise, I think it is fine to add this feature. On the other hand, the additional if-else branch makes code less readable, but I don't have a good alternative, unless we want to breakdown this function and/or create another decode function just for tflite.

Sorry, due to the limited bandwidth, preparing unittest is begin delayed now. May I push this code & example colab first, and then put the unittest later? In fact, the colab will be the modified version of this:
https://github.com/TensorSpeech/TensorflowTTS/blob/master/notebooks/TensorFlowTTS_Tacotron2_with_TFLite.ipynb
And I am about to remove TensorflowTTS dependency, and use our TensorFlow addons only.

Can it work with the condition on decoder.batch_size?

@jaeyoo To move forward on this, I added a test that runs the conversion on your branch. Can you take a look and comment?

@jaeyoo To move forward on this, I added a test that runs the conversion on your branch. Can you take a look and comment?

Yes it looks good to me. Thank you Guillaume!

@lc0 Your code suggestions were applied and generated a co-authored commit. We need you to confirm that you are ok with that as explained in https://github.com/tensorflow/addons/pull/1964#issuecomment-664847840.


Error importing pypsg

When I import pyspsg I get the following error:
Exception has occurred: SyntaxError
invalid syntax (request.py, line 210)

Hi @LordFurno! Thanks for submitting this issue. It seems I forgot to update the python version requirements when I put in that match statement. I'll fix this in the next release. For now you should upgrade your python environment to use python>=3.10. Hope that helps!

Thank you for the clarification!

I updated my python version, to 3.10, and now I get a different issue
---------------------------------------------------------------------------
FileNotFoundError                         Traceback (most recent call last)
[<ipython-input-3-fd95ada1908a>](https://localhost:8080/#) in <cell line: 1>()
----> 1 import pypsg

8 frames
[/usr/lib/python3.10/subprocess.py](https://localhost:8080/#) in _execute_child(self, args, executable, preexec_fn, close_fds, pass_fds, cwd, env, startupinfo, creationflags, shell, p2cread, p2cwrite, c2pread, c2pwrite, errread, errwrite, restore_signals, gid, gids, uid, umask, start_new_session)
   1861                     if errno_num != 0:
   1862                         err_msg = os.strerror(errno_num)
-> 1863                     raise child_exception_type(errno_num, err_msg, err_filename)
   1864                 raise child_exception_type(err_msg)
   1865

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

Could you include the full error output? It might be difficult to get IPython to output that so you could go to the terminal and type python -c "import pypsg". Did you install the latest release via pip?

I installed pypsg with the command pip install pypsg
This is the full error output:
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "C:\Users\LordFurno\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.12_qbz5n2kfra8p0\LocalCache\local-packages\Python312\site-packages\pypsg\__init__.py", line 6, in <module>
    from .request import APICall, PSGResponse
  File "C:\Users\LordFurno\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.12_qbz5n2kfra8p0\LocalCache\local-packages\Python312\site-packages\pypsg\request.py", line 19, in <module>
    docker.set_url_and_run()
  File "C:\Users\LordFurno\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.12_qbz5n2kfra8p0\LocalCache\local-packages\Python312\site-packages\pypsg\docker.py", line 175, in set_url_and_run
    if is_psg_installed():
       ^^^^^^^^^^^^^^^^^^
  File "C:\Users\LordFurno\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.12_qbz5n2kfra8p0\LocalCache\local-packages\Python312\site-packages\pypsg\docker.py", line 47, in is_psg_installed
    containers_info = _get_containers_json()
                      ^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\LordFurno\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.12_qbz5n2kfra8p0\LocalCache\local-packages\Python312\site-packages\pypsg\docker.py", line 14, in _get_containers_json
    raw_output = subprocess.check_output(
                 ^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.12_3.12.1264.0_x64__qbz5n2kfra8p0\Lib\subprocess.py", line 466, in check_output
    return run(*popenargs, stdout=PIPE, timeout=timeout, check=True,
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.12_3.12.1264.0_x64__qbz5n2kfra8p0\Lib\subprocess.py", line 548, in run
    with Popen(*popenargs, **kwargs) as process:
         ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.12_3.12.1264.0_x64__qbz5n2kfra8p0\Lib\subprocess.py", line 1026, in __init__
    self._execute_child(args, executable, preexec_fn, close_fds,
  File "C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.12_3.12.1264.0_x64__qbz5n2kfra8p0\Lib\subprocess.py", line 1538, in _execute_child
    hp, ht, pid, tid = _winapi.CreateProcess(executable, args,
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
FileNotFoundError: [WinError 2] The system cannot find the file specified

Can you try installing from the branch tedjohnson12/issue32? I've made some changes there that are Windows-specific. I have never run any of this on a Windows machine before so there are some issues that I have not yet encountered. You can
git clone https://github.com/tedjohnson12/pypsg.git
cd pypsg
git checkout tedjohnson12/issue32
pip install -e .
python3 -c "import pypsg"

I believe this should import correctly, but there might be other errors down the line. Specifically it seems like Windows does not like waiting for PSG API calls to finish, which causes a backup that PSG does not like.

Thank you this works well. Just a quick question, what are possible parameters for output types when calling pypsg.APICall()? Can you get it to return transmittance data?

Right now only rad, cfg, noi, lyr, set, and upd are fully supported, while all is semi-supported (will only parse the rad, noi, lyr, and cfg portions of the reply). trn outputs are not that complicated and I could include support for it in the next release. If you are up for the challenge you are more than welcome to add the functionality yourself. A PyTrn class would probably be very similar to pypsg.rad.PyRad (i.e. inherit from Astropy QTable) and not that difficult to implement.

It turns out that its super easy to implement. Try doing a git pull on that same issue branch and set the output type to trn. You can do something like
caller = APICall(...)
response = caller()
trn = response.trn
wl = trn.wl
o3 = trn['O3']
plt.plot(wl,o3)

When I git pull the updated branch I get the following error:
Exception has occurred: IndexError
list index out of range
  File "C:\Users\LordFurno\Downloads\getting_pypsg_toWork\pypsg\pypsg\trn\trn.py", line 28, in _get_metadata
    re.findall(r'Spectral unit:.+\[(.+)\]\n',header)[0]
    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^
  File "C:\Users\LordFurno\Downloads\getting_pypsg_toWork\pypsg\pypsg\trn\trn.py", line 46, in from_bytes
    metadata = cls._get_metadata('\n'.join(header))
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\LordFurno\Downloads\getting_pypsg_toWork\pypsg\pypsg\request.py", line 340, in __call__
    return PSGResponse(**{self._type:returntype.from_bytes(reply.content)})
                                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\LordFurno\Downloads\getting_pypsg_toWork\pypsg\test2.py", line 16, in <module>
    response = caller()
               ^^^^^^^^
IndexError: list index out of range

When running the following code:
import pypsg

cfg = pypsg.cfg.PyConfig(
    target=pypsg.cfg.Target(object='Exoplanet',name='Proxima Cen b')
    )

caller = pypsg.APICall(
        cfg=cfg,
        output_type='trn',
    )
response = caller()
trn = response.trn

This happens because there is no atmospheric information for PSG/PUMAS to work with, so the API call returns an empty string ''. Try downloading a config from the PSG GUI and reading it into python with cfg = pypsg.cfg.PyConfig.from_file('path/to/config')

Thank you for your help, I appreciate it greatly. Just one last question, does this package require a PSG API key to make more calls?


Bug/2511

What's this PR do/fix?
Bug #2511
Are there unit tests? If not how should this be manually tested?
No tests as this is to slove backwards compatibility
Any background context you want to provide?
Attribute "name" and "required" in PathVariable annotation were added in Spring 4.3.3. Using the version of the code causes an exception.
What are the relevant issues?
#2511

@dbaje thanks for the pr!!

Made some changes and used the spring capabilities utility. I needed to move it so it would be available for the rest of the code. Also made some changes to be compatible with mocking and version checking in tests.


Fully Distributed Version for Apache Spark

Todo

[ ] distributed build, should transform to Annoy compatible binary.
[ ] distributed query, for all-pair similarity (OLAP)
[ ] persistence with like HBase for query server (OLTP)
[ ] CRUD for prebuilt index

It's feasible. stay tuned. https://github.com/mskimm/ann4s/projects/1


Some blank pages are rendered when scroll in page before generating the pdf

Describe the bug
When I scroll in the page and then generate the pdf, there is some blank pages/offset applied before the content is rendered in the pdf.
cf. this issue: https://github.com/kempsteven/vue-html2pdf/issues/52
Package Version
1.8

I can confirm this, same happens here.
Using with Nuxt.

Hmm there is an easy fix for this but kind of adds redundancy on it.
You can wrap your pdf content on a component like this, and also use it to show your content on the page,
something like this:
<template>
    <vue-html2pdf>
         <pdf-content slot="pdf-content" />
    </vue-html2pdf>

    // show content on the view
    <pdf-content />
</template>

Another solution is add a loading overlay & prevent the user to scroll when the pdf is generating.
This issue is caused by the html2canvas package which is being used by html2pdf.js
https://github.com/eKoopmans/html2pdf.js/issues/175#issuecomment-634832966
as per him, try adding scrollX: 0, scrollY: 0 to the html2canvas options to get past this (html2canvas docs).
you see a sample here, how to add specific options:
https://github.com/kempsteven/vue-html2pdf#sample-value-of-html-to-pdf-options

I confirm that adding scrollX: 0, scrollY: 0 to the html2canvas option works great to fix the blank pages filling when scroll and then generating the PDF.
Thanks @kempsteven ! Great work 👌

Thanks, works here as well !
Though this really should be the default behaviour, or at least consider a top level prop.
It almost made me throw in the towel with this package, but I'm happy that there is a workaround.
I might publish a draft pull request for the top level prop, if I have the time this week.

Hmm there is an easy fix for this but kind of adds redundancy on it.
You can wrap your pdf content on a component like this, and also use it to show your content on the page, something like this:
<template>
    <vue-html2pdf>
         <pdf-content slot="pdf-content" />
    </vue-html2pdf>

    // show content on the view
    <pdf-content />
</template>

Another solution is add a loading overlay & prevent the user to scroll when the pdf is generating.
This issue is caused by the html2canvas package which is being used by html2pdf.js eKoopmans/html2pdf.js#175 (comment)
as per him, the solution for this is adding scrollX: 0, scrollY: 0 to the html2canvas options to get past this (html2canvas docs).
you see a sample here, how to add specific options: https://github.com/kempsteven/vue-html2pdf#sample-value-of-html-to-pdf-options
I haven't tested it, but it seems it fixes the issue.

it's not work to me and overlay too

can anyone help ??
using scaleX:0 scaleY: 0  and  using overlay already but not work


Added MacOutlookEnabled to Set-CASMailbox.md

Closes https://github.com/MicrosoftDocs/office-docs-powershell/issues/6165.

Docs Build status updates of commit 9f46aaf:
:clock10: Full build: the average full build time is 17 min(s) 42 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 9f46aaf:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Set-CASMailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 341ddd5:
:clock10: Full build: the average full build time is 17 min(s) 41 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 341ddd5:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Set-CASMailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


podman play assigns wrong environment vars.

Is this a BUG REPORT or FEATURE REQUEST? (leave only one on its own line)
/kind bug
Description

podman play assigns environment vars to the wrong containers.
Steps to reproduce the issue:
podman run -dt --rm --name gitea_gitea  --pod new:gitea -p 3000:3000 -p 2222:22 gitea/gitea:latest
podman run -dt --rm --name gitea_postgres --pod gitea postgres:9.6
podman generate kube gitea -f gitea.yml
podman pod rm -f gitea
podman play kube gitea gitea.yml
podman generate kube gitea -f gitea_played.yml
diff -y gitea.yml gitea_played.yml

Describe the results you received:
There is an unexpected difference between the two yaml files
Differences are regarding the env variables.
Describe the results you expected:
I expected both files to be identical or with minor differences
Additional information you deem important (e.g. issue happens only occasionally):
always
Output of podman version:
[root@localhost ~]# podman version
Version:            1.6.4
RemoteAPI Version:  1
Go Version:         go1.12.12
OS/Arch:            linux/amd64

Output of podman info --debug:
debug:
  compiler: gc
  git commit: ""
  go version: go1.12.12
  podman version: 1.6.4
host:
  BuildahVersion: 1.12.0-dev
  CgroupVersion: v1
  Conmon:
    package: conmon-2.0.6-1.module_el8.1.0+272+3e64ee36.x86_64
    path: /usr/bin/conmon
    version: 'conmon version 2.0.6, commit: 7a4f0dd7b20a3d4bf9ef3e5cbfac05606b08eac0'
  Distribution:
    distribution: '"centos"'
    version: "8"
  IDMappings:
    gidmap:
    - container_id: 0
      host_id: 1000
      size: 1
    - container_id: 1
      host_id: 100000
      size: 65536
    uidmap:
    - container_id: 0
      host_id: 1000
      size: 1
    - container_id: 1
      host_id: 100000
      size: 65536
  MemFree: 614768640
  MemTotal:<PHONE_NUMBER>
  OCIRuntime:
    name: runc
    package: runc-1.0.0-64.rc9.module_el8.1.0+272+3e64ee36.x86_64
    path: /usr/bin/runc
    version: 'runc version spec: 1.0.1-dev'
  SwapFree:<PHONE_NUMBER>
  SwapTotal:<PHONE_NUMBER>
  arch: amd64
  cpus: 2
  eventlogger: journald
  hostname: localhost.localdomain
  kernel: 4.18.0-147.5.1.el8_1.x86_64
  os: linux
  rootless: true
  slirp4netns:
    Executable: /usr/bin/slirp4netns
    Package: slirp4netns-0.4.2-2.git21fdece.module_el8.1.0+272+3e64ee36.x86_64
    Version: |-
      slirp4netns version 0.4.2+dev
      commit: 21fdece2737dc24ffa3f01a341b8a6854f8b13b4
  uptime: 1h 21m 29.73s (Approximately 0.04 days)
registries:
  blocked: null
  insecure: null
  search:
  - registry.access.redhat.com
  - registry.fedoraproject.org
  - registry.centos.org
  - docker.io
store:
  ConfigFile: /home/sysops/.config/containers/storage.conf
  ContainerStore:
    number: 0
  GraphDriverName: overlay
  GraphOptions:
    overlay.mount_program:
      Executable: /usr/bin/fuse-overlayfs
      Package: fuse-overlayfs-0.7.2-1.module_el8.1.0+272+3e64ee36.x86_64
      Version: |-
        fuse-overlayfs: version 0.7.2
        FUSE library version 3.2.1
        using FUSE kernel interface version 7.26
  GraphRoot: /home/sysops/.local/share/containers/storage
  GraphStatus:
    Backing Filesystem: xfs
    Native Overlay Diff: "false"
    Supports d_type: "true"
    Using metacopy: "false"
  ImageStore:
    number: 0
  RunRoot: /run/user/1000
  VolumePath: /home/sysops/.local/share/containers/storage/volumes

Package info (e.g. output of rpm -q podman or apt list podman):
podman-1.6.4-2.module_el8.1.0+272+3e64ee36.x86_64

Additional environment details (AWS, VirtualBox, physical, etc.):
see attached files
gitea.yml.txt
gitea_played.yml.txt

@haircommander Could you take a look at this one?

@sujil02 Did you ever get a chance to look at this, If yes could you hand it over to @ryanchpowell

@sujil02 Did you ever get a chance to look at this, If yes could you hand it over to @ryanchpowell

Yes, I did look into this a couple of times. But could not get a significant breakthrough @ryanchpowell we can connect and then probably knock this one out together?

@haircommander Any chance you could look at this?

I will try to cleanup these play/generate kube issues

You da man.

it turns out, when we do the initial generate kube we aren't taking the image's env into account, only the containers. Play kube does this, so on the second generate kube, we've picked up the images env. Unfortunately, I believe fixing this requires plumbing all of generate kube with the image.Runtime and ctx to actually be able to inspect the image and get the env out of it. as such, I believe this is more than I am able to take on right now.

I don't understand exactly where this problem is. . .

if the image has environment variables, play kube takes them and adds them to the container's env structure. However, generate kube does not take the image env into account. So first generate does not have image env, first play does have image env, second generate does have image env. does that clear it up @zhangguanzhang

if the image has environment variables, play kube takes them and adds them to the container's env structure. However, generate kube does not take the image env into account. So first generate does not have image env, first play does have image env, second generate does have image env. does that clear it up @zhangguanzhang

so need to get all env from ctr first, and remove all the env from the img？

Sure, except you need to be careful, they match exactly, IE Name and Value.  Otherwise there is a chance the user specified an environment that overroad the default.  There is also the corner case where the user created the pod/container with an exact match, but I think we can live with this, as a big corner case.

You might want to remove "container=", since Podman adds this itself.  And we don't want openshift setting container=podman when a user runs the container on CRI-O.

@zhangguanzhang Interested in taking this on?

I am busy these days, if the problem is not solved later, I will try

I think this problem maybe fixed by https://github.com/containers/podman/pull/8654, so could you try with the latest version?

Reopen if it does not.


Driver.connect should return null for unsupported urls

According to java.sql documentation, connect should return null for foreign urls, current behaviour is throwing "java.lang.IllegalArgumentException: Incorrect ClickHouse jdbc url: ..."
http://docs.oracle.com/javase/8/docs/api/java/sql/Driver.html#connect-java.lang.String-java.util.Properties-

Thanks.
I've already merged another PR on the problem, which introduced conflict with yours, but the test from yours is also useful. Could you update your PR? Then I'll merge it.

I will make another PR with test


service/share: Service interface and basic implementation for GetShare

Content

Fully defined interface for share.Service

(in future we may add more write operations ad-hoc)

GetShare implementation.

GetShares are not needed right now and will be implemented together with BlockSync logic
GetSharesByNamespaces will be implemented by #5

Reusable testing utilities
Alias for DAH in service/header
Multiple TODOs for possible data usage optimizations

Refs
Closes #23
Depends on #158

@renaynay, one more change in here. I moved DataAvailibilityHeader to be aliased in share/service as share.Root. Semantically and logically, the data availability header is Root commitment to all the shares and nothing else and the data availability notion is more like a theoretical, so code-wise it makes more sense to keep root share commitment definition in service/share. Another motivation here is to remove dependency on service/header from service/share.


[firebase_messaging]: FirebaseMessaging.onBackgroundMessage not working in terminated state for iOS

Is there an existing issue for this?

[X] I have searched the existing issues.

Are you aware of the differences between iOS and Android background message handling?

[X] I understand that iOS and Android background messages behave differently, and I've designed my application with that in mind.

Do you have an active Apple Developer account?

[X] I have an active Apple Developer account.

Are you using a physical iOS device to test background messages?

[X] I am using a physical iOS device to test background messages.

Have you enabled "Remote Notifications" & "Background Mode" (Checking options for "Background Processing" & "Remote Notifications") in your app's Xcode project?

Have you created an APNs key in your Apple Developer account & uploaded this APNs key to your Firebase console?

Have you disabled method swizzling for Firebase in your app?

Are you sending messages to your app from the Firebase Admin SDK?

fcm.send({
    data: data.payload,
    token: data.token,
    android: {
      data: data.payload,
    },
    apns: {
      headers: {
        "apns-priority": "5",
      },
      payload: {
        aps: {
          contentAvailable: true,
          mutableContent: true,
          alert: {
            title: "TITLE",
            subtitle: "SUBTITLE",
            body: "BODY",
          },
          category: "FLUTTER_NOTIFICATION_CLICK",
        },
      },
    },
  });

Have you requested permission from the user to receive notifications?

[X] I have the relevant permission to receive notifications.

Have you used the 'Console' application on your macOS device to check if the iOS device's system is throttling your background messages?

default	11:48:02.216264+0530	SpringBoard	Received incoming message on topic com.influentialbrands.ibapp at priority 5
default	11:48:02.232266+0530	SpringBoard	[com.influentialbrands.ibapp] Received remote notification request BA86-E6A6 [ waking: 0, hasAlertContent: 0, hasSound: 0 hasBadge: 0 hasContentAvailable: 1 hasMutableContent: 1 pushType: Background]
default	11:48:02.232435+0530	SpringBoard	[com.influentialbrands.ibapp] Process delivery of push notification BA86-E6A6
default	11:48:02.232489+0530	SpringBoard	[com.influentialbrands.ibapp] Request DUET delivers content-available push notification to application
default	11:48:02.232784+0530	SpringBoard	SUBMITTING: com.apple.pushLaunch.com.influentialbrands.ibapp:A70163
default	11:48:02.234155+0530	dasd	Submitted Activity: com.apple.pushLaunch.com.influentialbrands.ibapp:A70163 at priority 5 (Tue Apr 25 11:48:02 2023 - Wed Apr 26 11:48:02 2023)
default	11:48:02.239975+0530	dasd	Daemon Canceling Activities: {(
com.apple.pushLaunch.com.influentialbrands.ibapp:C83A41
)}
default	11:48:02.240175+0530	dasd	CANCELED: com.apple.pushLaunch.com.influentialbrands.ibapp:C83A41 at priority 10 !
default	11:48:02.247837+0530	dasd	com.apple.pushLaunch.com.influentialbrands.ibapp:A70163:[
{name: ApplicationPolicy, policyWeight: 50.000, response: {Decision: Absolutely Must Not Proceed, Score: 0.00, Rationale: [{[pushDisallowed]: Required:0.00, Observed:1.00},]}}
], FinalDecision: Absolutely Must Not Proceed}
Additional context and comments
The  FirebaseMessaging.onBackgroundMessage is not getting called only when app is terminated.
It works perfectly fine when app goes in the background.
I have tried sending notifications with different payloads, but still it doesn't run.
When I send both data and notifications in the payload, it shows the notification due to the notification in the payload, but the
onBackgroundMessage is still not getting run. I have tried adding "contentAvailable" and "mutableContent" as true like it is said in most of the answers and I have implemented all the settings above.

Thanks for the report @Vipulwaghmare
Check this comment to see if it helps to answer your query.
Also, see this issue for your reference: https://github.com/firebase/flutterfire/issues/7407

I/FLTFireMsgService(xxxxx): FlutterFirebaseMessagingBackgroundService started!

not appears when i run ios .

@Vipulwaghmare are you found any solution ?


RedHat support

The docker repo for RedHat7 lives at:
https://yum.dockerproject.org/repo/main/centos/7/
The changes in this PR accommodate for that inconsistency in dockers repository naming structure.

This has been adapted and merged


Datarella_membership_request

Hi Dana / Henry,
My name is Rebecca Johnson. I’m a blockchain architect working at Datarella GmbH.  We’re a provider of industrial blockchain solutions based in Munich.  We do a lot of work in the humanitarian sector including with our most famous project “Building Blocks” which is deployed to provide contactless financial transaction services to 500,000 refugees in Jordan and another 46000 refugees in Bangladesh through the World Food Program. https://insight.wfp.org/how-blockchain-is-helping-wfps-fight-against-covid-19-in-bangladesh-d2b466a8becf
We’re potentially going to be in a position in the near future to develop and deploy contact tracing tech for some of the humanitarian organizations which we work with. As a result, I thought it would be a great idea to reach out to you and join the coalition. We’re investigating using the TCN protocol within our projects and I’d like to stay abreast of the newest developments.
I already put in a pull request for the website logo grid - I hope that’s okay.
Best regards,
Rebecca Johnson
Datarella GmbH
Oskar-von-Miller-Ring 36
D-80333 München
Mobil: +49<PHONE_NUMBER>9
E-Mail<EMAIL_ADDRESS>Geschäftsführung: Michael Reuter
Sitz der Gesellschaft: München, HRB 207851

Hi Rebecca,
Great to hear from you! I'll reach out by email and we can find a time to chat. Would love a brief conversation before officially adding you.
Thanks,
Jenny


Reworking batching interpolation

Reworked the way interpolation works with batching, allowing the user (or user-application) to have greater flexibility in what to expect when both the SOC and the internal health variable are batched.
By defaults, we try to match the SOC and internal batches, as well as remove superfluous batch dimensions of cardinality 1, as to successfully reproduce the behavior chosen earlier.

I'm likely to merge as-is when I'm back to work tomorrow. Docs and tests look good, and this code is internal enough that I'm not worried about changing behavior. I'll mostly look for performance issues.
Until then, two quick and mutually-exclusive suggestions:

Would a table be better than prose for explaining the output shapes?
Had you already considered requiring specific shapes and always returning a 3D array from the function? Adding and removing dimensions is very easy with NumPy arrays, so the tradeoff of "additional work to use" versus "difficulty to understand" is favorable.

Thanks, Logan! Here's my followups:

Would a table be better than prose for explaining the output shapes?

I am not certain I understand. Are you referring to the documentation? If so, then maybe? I would have to look up how to do a proper table in markdown for rendering the docs, but otherwise, perhaps? I can try to write one and you can tell me what you think?

Had you already considered requiring specific shapes and always returning a 3D array from the function? Adding and removing dimensions is very easy with NumPy arrays, so the tradeoff of "additional work to use" versus "difficulty to understand" is favorable.

Yes, and perhaps simply always returning a 3D array is preferable. The reason I didn't do it was that this particular class is far-reaching inside moirae, and I tried to mimic previous behavior as to not break anything else. The balance in my mind is "how much work will it take to re-adjust everything that uses this" vs."how much does one need to understand the dimensionality concerns when using it" (which in my mind is a balance that is unclear)
Now, in terms of the

I'm likely to merge as-is when I'm back to work tomorrow.

my concern is the test_interface.py that is failing. I don't feel comfortable merging something that doesn't pass all the tests, but, at the same time, I don't believe I know how to fix it...

Yes, and perhaps simply always returning a 3D array is preferable.

If you think this is the better solution, then shall we implement it? Your IDE will help you track down all the calls to get_values and the unit tests are there to help catch the ones you miss it.

my concern is the test_interface.py that is failing.

I'm back from PTO and will tinker with that today. Ugh, conda

If you think this is the better solution, then shall we implement it?

I am not convinced it is, as I few there are very few cases in which we would not want to match the SOC batch with the health variable batch. However, I can try changing it to always being 3D, but I will say I am a bit wary of it; I don't think our tests are as thorough as coveralls would make us believe. It would also involve re-writing a significant amount of the tests I think

Based on additional conversations with @WardLT, we opted to have the get_value() method of the SOCInterpolatedHealth class to always return a 3D array, so it's behavior is more predictable and easier to understand. This require a number of changes on the way the ECM classes operate, as well on a few tests. I am now just re-running a few of our demonstration notebooks to make sure they still perform as expected.


doc: update readme

Ideas:

[ ] Move local development instructions to a HACKING.md file.
[ ] Link to the Colab example notebook near the top right after the pip install nibiru section
[ ] Add a few usage example code blocks to the README

The README has the above items, so we can close this


debian plugin 'ag' alias overrides silversearcher app

Many people, including myself, are using the silver searcher as a better grep/ack. debian plugin overrides it. You should probably disable this alias if the application is installed, or just find a different alias.
Is there a way to turn this alias off instead?

Duplicate of #3633.

Please test out that patch and let me know how it works for you @apjanke
Maybe my implementation isn't the cleanest/best idea for the project, but it's working just fine on my end.

closing because duplicate of #3633 , @KhasMek i've replied on the original issue


Spamming launch Steam button will overwrite old run_northstar.txt

Sspamming launch via Steam will basically overwrite backup with the 1 version. Tbh, not a really a big issue but just something to keep in mind.
Might gonna later add a function to repair window that allows easily removing the file again ^^
Originally posted by @GeckoEidechse in https://github.com/R2NorthstarTools/FlightCore/pull/178#discussion_r1158520234

No longer relevant since #392 got merged


CMS Connection Dialog allows username\password for Windows Auth

Azure Data Studio Version: master

Steps to Reproduce:

Click "Add Central Management Server" button
Switch to SQL Login auth and then cancel the Connection Dialog
Click "Add Central Management Server" button

Expected: Dialog is in same state after step 1 and step 3
Actual: Auth reset to Windows auth, but username & password are not reset to read-only.

Fixed.


Play state not respected anymore

In latest build, play-state (paused/playing) of sonos is not respected anymore.
When sonos is paused and a tts is played, sonos starts playing the queue after that (while it should just return to pause)

Github bot. No salutation detected. Issue automatically closed.

Please install an older version

And please say me more.
What starts again? Stream radio, playlist, queue…

Hi
can you unzip this file into your .node-red/node_modules/node-red-contrib-tts-ultimate/ttsultimate folder, restart node-red and tell me if it works?
ttsultimate.js.zip](https://github.com/Supergiovane/node-red-contrib-tts-ultimate/files/7104109/ttsultimate.js.zip)

Hi,
That worked, tested everything, sonos readio, playlist from spotify, sptoify radio, etc... all works perfect now and playstate and queue is fully respected. Thanks for fixing this super-fast !

Perfect.
New version will be out next week. In the meantime, you can live with this modified file.
Whenever you open an issue, please remember to salute ("Hello", "Hi", etc..) otherwise the issue will be automatically closed.
Thanks.

Same here, everything is working fine!
Thanks for the hotfix!


Advanced Order info doesnt clear out of order ticket after placing an order/trade

create an open order and use the advanced order to set a time for expiration. After order goes onto the book....those filters dont clear out

can wire up clear button to clear order form advanced section.  We'd need to refactor how trading form is organized to make this work. I don't want to put in a hack just to clear advanced section of order form.
Will talk over with other devs.

if this isnt easy to solve we can push this to post

not easy to solve, push to post. The clear button in order form will now clear Advanced section

closing - hitting clear resets it and also leaving the pg and coming back resets it....will require major refactor to do


fix: calling nested commands

Changes Unknown when pulling 4d7d3f9a113660e08c2ef95b2689f03e83955444 on nested-cmd into ** on master**.


Improve spelling of items relating to nls.localize()

#48171

@jsoref we do accept PRs for fixing wording issues in the actual words (e.g. the second thing in the nls.localize()) call, but not for the first one, which is up to the developer. Can you redo your PR to only include changes for words that have errors, but not the keys which are only used for translation purposes? Thanks!


STONEBLD-1577 Add tekton-result

So I tried testing this out on prod and there were some minor issues with the URI. Once I had fixed those though it was failing with an authentication issue.
I figured the simplest way to work around that was to just use the Kube Clients HTTP Client instead, so I came up with this simple example that can read from prod: https://github.com/redhat-appstudio/jvm-build-service/commit/376186f544b0a561fb191510d1f28418749348b3
What do you think about removing the open API stuff and just using the kube client directly. It's only a single very straightforward call so I don't think the generated code buys us much.

This works, I think there are some general improvements we can make to the log viewer (e.g. prompt for the build if none are specified), however this is a separate issue.


Examples not working

Hi there,
When trying to reproduce the example for huawei p8 I am getting the following error:
~/Development/bootstomp/BootStomp$ python taint_analysis/bootloadertaint.py config/config.huawei
WARNING | 2018-03-08 16:11:06,322 | angr.analyses.disassembly_utils | Your verison of capstone does not support MIPS instruction groups.
WARNING | 2018-03-08 16:11:06,344 | cle.blob | No custom_entry_point was specified for blob /home/santitox/Development/bootstomp/BootStomp/bootloaders/huawei_p8/fastboot.img, assuming 0
WARNING | 2018-03-08 16:11:06,344 | cle.blob | No custom_base_addr was specified for blob /home/santitox/Development/bootstomp/BootStomp/bootloaders/huawei_p8/fastboot.img, assuming 0
Deprecation warning: Use main_object instead of main_bin
Traceback (most recent call last):
  File "taint_analysis/bootloadertaint.py", line 330, in
    bt.run()
  File "taint_analysis/bootloadertaint.py", line 271, in run
    self._preprocess_taint_file()
  File "taint_analysis/bootloadertaint.py", line 199, in _preprocess_taint_file
    line = patch_fucking_idapython(line, base_addr)
  File "taint_analysis/bootloadertaint.py", line 155, in patch_fucking_idapython
    bl = self._p.factory.block(int(call_addr, 16) - base_addr, thumb=True)
  File "/usr/local/lib/python2.7/dist-packages/angr/factory.py", line 332, in block
    backup_state=backup_state, opt_level=opt_level, num_inst=num_inst, traceflags=traceflags
  File "/usr/local/lib/python2.7/dist-packages/angr/block.py", line 60, in __init__
    traceflags=traceflags)
  File "/usr/local/lib/python2.7/dist-packages/angr/engines/vex/engine.py", line 461, in lift
    raise SimEngineError("No bytes in memory for block starting at %#x." % addr)
angr.errors.SimEngineError: No bytes in memory for block starting at 0x7002ec8.

This issue does not happen when running within the docker container provided. Yet, in both scenarios, trying to manually load the bootloader with angr returns a different error message:
~/Development/bootstomp/BootStomp$ python
Python 2.7.13 (default, Nov 23 2017, 15:37:09)
[GCC 6.3.0 20170406] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import angr
WARNING | 2018-03-08 16:26:21,362 | angr.analyses.disassembly_utils | Your verison of capstone does not support MIPS instruction groups.
>>> p = angr.Project("bootloaders/huawei_p8/fastboot.img")
Traceback (most recent call last):
  File "", line 1, in
  File "/usr/local/lib/python2.7/dist-packages/angr/project.py", line 164, in __init__
    self.loader = cle.Loader(self.filename, **load_options)
  File "/usr/local/lib/python2.7/dist-packages/cle/loader.py", line 126, in __init__
    self.initial_load_objects = self._internal_load(main_binary, *force_load_libs)
  File "/usr/local/lib/python2.7/dist-packages/cle/loader.py", line 493, in _internal_load
    main_obj = self._load_object_isolated(main_spec)
  File "/usr/local/lib/python2.7/dist-packages/cle/loader.py", line 583, in _load_object_isolated
    raise CLECompatibilityError("Unable to find a loader backend for %s.  Perhaps try the 'blob' loader?" % spec)
cle.errors.CLECompatibilityError: Unable to find a loader backend for bootloaders/huawei_p8/fastboot.img.  Perhaps try the 'blob' loader?

We advise you to use docker-ed version of the tool. When loading the BLOB manually, you are not passing the appropriate switches as the tool uses here: https://github.com/ucsb-seclab/BootStomp/blob/master/taint_analysis/bootloadertaint.py#L44


Basic Multi folder support

Split SwiftContext in two: WorkspaceContext for whole workspace and FolderContext for each folder.
Catch when folders are added/removed from project. Create/delete FolderContext on add/remove
Add observers to be called on add/remove of project
Add logObserver that logs adding and removing of folders
Add observer that adds the dependency view for the root folder
Add observer that calls resolve and make debug config for the root folder
Edit TaskProvider to returns tasks based on contents of WorkspaceContext

In effect what we had before, but now we can extend this for multi root projects
I see two types of multi-folder project. One where you are editing dependencies of a project locally and one where the folders are separate projects. The first is fairly easy to setup. We can add an observer that calls swift package edit/unedit when folders are added and removed. The second is harder as currently Tasks run in the workspace root. We will a lot more work to get that working.

For workspaces that contain multiple projects maybe we can add a top level launch.json file the extension can check for and know if it needs to be initialised

When you save a multi-folder project it places the launch.json configs in the saved project. I haven't looked too much into this yet but that seems to be where they go.


no such file for index.js for the main entry in package.json

main in package.json points to "index.js"
But there is no such file. and running require('electron-forge')
gives the error "Error: Cannot find module 'electron-forge'"
I get that electron-forge is a CLI program. so not even sure if this bug is important
but maybe we can remove the entry for main from package.json, if it is not used ?

It's there as a placeholder and whether it is there or not it will still throw the "can not find module" error.
See https://github.com/electron-userland/electron-forge/pull/56 for how we plan to expose the CLI methods as JD API's so you can actually require this module.


Inconsistent Results when using Result Types in Search

Currently using a classic page with a Search Results Web Part on it. I have configured result types, so I can load different display templates based on the type of content being returned in search.
I am receiving inconsistent results from search when I turn on "use result types" inside my search result web part. When I do a hard refresh in my page, the result that comes back does load the correct display template, however the item is missing a lot of managed properties. These manage properties include custom  RefinableStrings and even the ContentType property. If I do another hard refresh, the properties will populate correctly.
If I turn off result types completely, my properties for documents are always returned correctly. This may  be related to #3261 where different behaviors are happening when refreshing a page. I believe I have configured the result types correctly, as sometimes it does load the right way.
Category

[x] Question
[x] Bug

Expected or Desired Behavior
Expecting my Search Results to contain all the managed properties defined on the items consistently. Refreshing the page should not change the behavior of the results.
Observed Behavior
When using search results and result types, I am receiving inconsistent data back from search. Refreshing the page, often fixes the issue temporarily.
Steps to Reproduce

Create a display template that pulls in the "ContentType" managed property.
Configure a result type for documents to use your display template where ContentTypeId starts with 0x01
Configure Search Result Web Part to pull in those documents, and is configured to use result types
Hard refresh the page and notice the ContentType managed property is empty. Refresh  a few more times, and notice the Content Type field comes back. You may also just search for documents and keep hitting enter. Every couple of searches, will result in different data being returned.

What I'm seeing
Load the Page, results are basically empty because ContentType is empty and my refinables are missing.

Hard Refresh on the page w/ no other changes. The ContentType is populated, the image is available and the Refinable Properties show up below the title of the document.

Here is my result type definition, very basic. If Document, load Generic Document Tile Display Template

I would be happy to share the actually JSON results from search if needed. Basically in the first picture, the item returns,  it has a ContentTypeId, but no ContentType or Refinable properties. Refresh/search and they show in the result.

Fix was to use "Optimize for Frequent use" in the result type definition.


Removed EnableWebhookEventLogging setting

Codecov Report

Merging #847 (99f78a3) into master (39352da) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #847   +/-   ##
=======================================
  Coverage   31.42%   31.42%
=======================================
  Files          49       49
  Lines        5982     5982
=======================================
  Hits         1880     1880
  Misses       3913     3913
  Partials      189      189

Continue to review full report at Codecov.

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

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.


Client-go 5.0.0 ?

When can we expect client-go 5.0.0 released?

Appriciate if plan for  client-go 5.0.0 release can be shared.
Are there any gaps in release-5.0 branch  or is it compatible with 1.8 release?

/cc @sttts

I asked @sttts on slack. I am sharing his response:

thanks @tamalsaha

How about the CustomResourceDefinition in 1.7 that was missed by 4.0.0? Will 5.0.0 have it and be compatible with Kubernetes 1.7 and 1.8?

How about the CustomResourceDefinition in 1.7 that was missed by 4.0.0?

What do you mean?

CustomResourceDefinition that replaced ThirdPartyResource in Kubernetes 1.7 is missing in client-go 4.0.0 and 5.0.0. TPR is still there although obsolete.
See #247 .

See https://github.com/kubernetes/client-go/issues/247#issuecomment-335231159 (from the same thread). You can get CRD working with client-go 4.0.0

I was expecting it to be available as a first class citizen, just like TPR was.

release-5.0 is now synced with the 1.8 branch. :tada:


Version chooser missing after upgrading to 0.34.0

Problem
After upgrading to the modernized theme, version choosers are missing at relevant spots.

https://cratedb.com/docs/jdbc/
https://cratedb.com/docs/reference/

Thoughts
That's weird, @msbt's most recent preview still displays it.

On the theme itself, the version chooser also is present, see https://crate-docs-theme.readthedocs.io/.

@msbt shared that it is only rendered conditionally.
{%- if display_version %}
{% include "components/version_chooser.html" %}
{%- endif %}

-- https://github.com/crate/crate-docs-theme/blob/main/src/crate/theme/rtd/crate/sections/sidebar-secondary.html#L13-L15
That's sweet, but why doesn't it work any longer like before?

@amotl I just compared and before we were using
{%- if versions %}
{% include "version_chooser.html" %}
{%- endif %}

Not sure why it became display_version, but seems like an easy fix - will take care of it tomorrow.

I can remember we introduced display_version the other day, in order to be more explicit than implicit.
It has 13 hits across the org, so it can't be that wrong, at least it hasn't been in the past. See https://github.com/search?q=org%3Acrate display_version&type=code.

@amotl this is the corresponding commit: https://github.com/crate/crate-docs-theme/commit/638599314a5f8d0788759895ed3581a976c15254
I'm guessing we did that to enable the verson chooser in the demo theme, right? Which one do you suggest now to bring it back, current_version as it was in that commit or just versions as above?

Yeah, I think we did it because of this, at least that was the beginning of the story. If we should revert that, I don't know. What do you recommend?
Maybe honor both flags? Wouldn't make too much sense, hm?

@amotl at least when rendering localhost it didn't change anything, same dropdown menus visible.

Maybe the announcement of this variable in some blueprint/master configuration file has been dropped?

html_theme_options are used here:
https://github.com/crate/crate-docs-theme/blob/134f2a863633f15d105e0e9bff7635e7f2068b04/src/crate/theme/rtd/conf/__init__.py#L67-L81
Defined, they are over there:
https://github.com/crate/crate-docs-theme/blob/134f2a863633f15d105e0e9bff7635e7f2068b04/src/crate/theme/rtd/crate/theme.conf
Maybe it will be enough to also define display_version here? I may be totally wrong on my assessment, just asking/suggesting without having done any research at all on this topic, yet.

Which one do you suggest now to bring it back, current_version as it was in that commit or just versions as above?

I still think using display_version is currently the designated way of enabling the version chooser explicitly per project. If you can convince me to leave this behind, let's please have a session about it.
Otherwise, I think we may want to find out why it stopped working?

It has 13 hits across the org, so it can't be that wrong, at least it hasn't been in the past. See https://github.com/search?q=org%3Acrate display_version&type=code.

@amotl ah yes, good call, but I guess we forgot/missed to add that "display_version": True, snippet to the repos which actually need to have versioned docs, like the reference?

That would be silly, but it's possible. Thanks!

@amotl So should we first try to update the reference repo to see if the version chooser appears then before changing the theme?

Yeah, I think this would be good. Thanks!


feat: add Dashboard Settings Page

This is the initial proposal for #867, I think more insights would be nice to know that I could improve in order to finish. Your attention @chrisbbreuer

Looks amazing! Sharing some feedback with you next 😍


Proposal: rename repository to window-picker.nvim

Because of the name of the repo being nvim-window-picker, in the lazy package manager we need to do name = "window-picker" to make it play nicely with the automatic setup with the opts table. If we rename the repository to window-picker.nvim then this would be handled automatically.
Totally understand if you don't want to do this, so feel free to close this! One upside with GitHub is it will automatically forward the original repository name/link to the new location as long as a new repository hasn't been made with the original owner/name

At least now, auto config works with lazy nvim so closing this.


Set correct score

Is it possible that Testcafe sets the right score in CBT? (Pass | fail | unset).
So far, there is only unset and that makes it all a bit more complicated.

This is preventing me from using TestCafe with CBT.


ods:taxonFormula: unclear definition and location of the term

Term Name
ods:taxonFormula
Digital Object Name
DigitalSpecimen: Identification
Feedback
What is meant with "full formular" in the context of the taxonomic identification.
Also if this term pertains to the taxonomic identification, shouldn't it be located within the class ods:TaxonomicIdentification and not at this class (ods:Identification)?

Agree, for now we are removing this term. Probably we could use ods:specimenName for the taxonformula as it is basically the combined taxonIdentificaitons (in cases where one specimen consists of multiple taxa). If the need requires us to add this are a similar term we can always add it. For more on taxonFormula also see what I answered in issue https://github.com/DiSSCo/openDS/issues/170


[AutoPR containerregistry/resource-manager] [ACR] Populate Task Timer Trigger in Run Response

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/6221

(message created by the CI based on PR content)

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/5650


Mark required elements inside group in mutually exclusive set as optional

Please update CHANGELOG, too. Thanks!

I'm merging this.
Do you see any reason not to use deep_dup, would make things a bit shorter?
        dup_params = params.deep_dup # duplicate the params as we are going to modify them
        dup_params.reject do |param, value|
          is_nested_param = /^#{ Regexp.quote param }\[.+\]$/
          0 < params.count do |p, _|
            match = p.match(is_nested_param)
            dup_params[p][:required] = false if match && !value[:required]
            match
          end
        end

deep_dup is a rails method. I am not sure if everyone who uses grape-swagger use it along with rails. So avoided using it.

Makes sense.


Add example for CursorPagination::make($this->id()) to changelog

It took us a while to figure out what you meant with "The schema's id field must now always be provided to the CursorPagination::make() method and/or constructor.". In https://github.com/laravel-json-api/laravel/blob/develop/CHANGELOG.md#300---2023-02-14 we eventually found an example, so I added it here too for future reference.

Thanks for the PR!


Runaway forever logs

Hiya,
Have a bridgechain running for a month and noticed the ~/.forever/xxxx.log growing into the GBs.
Is there a way to set up log rotation and a reasonable size target size?
The logs seem to log a lot of debug which may not be needed for a long running bridge chain.
How can we adjust that?
Let's say you fix the log rotation, what is the proper way to upgrade the code already running?
This brings up lots of questions as to how to best run bridgechains longterm.

Closing in favour of #29


Recipe as class with Interface "Recipe"

I think it would be good to have recipes as classes with interface "Recipe".

Hmm. May be as alternative syntax? We need simple API.

For namespaces we need to turn all functions into classes. That because I think it is good to have recipes  as class files.
You can would have two chooses:
a) Load a recipe (-> class file).
b) Do the commands directly (like the functions.php currently, just with a class around it).

What the interface?

$deployer = new Deployer();
$deployer->useRecipt(new SymfonyRecipt());
$deployer->run();

Something like this would be nice.

? What the interface?
interface Recipe {
  ???
}

The interface would needed to get all recipt information from the recipt object.
$deployer->useRecipt will require one parameter of type ReciptInterface.
The interface could be something like this:
interface ReciptInterface {

public function getHost();
public function getUser();
public function getEnv();

}

Another idea would be to have one function which returns an array with all data:
interface ReciptInterface {

public function getRecipt();

}

First example it’s looks more like Server.
Second, obviously bad idea.

Yes, the second one is a bad idea. Just to write down all ways how you could do it. :+1:


505 internal server error

I was testing the API but it showed an 505 error
Edit: it works on topics other than science

Okk I'll Check it
Thanks for Reporting

Since the API is down, I might as well close this 🤷 (And its been 2 years lol)


Fix link to unit tests for translation problems

Hi,
the data_generator readme file points to an already removed unit test (wmt_test.py), introduced with eacde9d9f5b4dede91bb95d4c38083bc70824b30. This PR updates the documentation for that.

Thanks!


Invite text message deeplinking SBAT work on iOS

Expected Behavior
deeplinking works with iOS and android devices
Current Behavior
deeplinking presently works with android only

Subset of #1512

Ok, so this one was stale for quite a while, so it's time to finish it! :)
I had a chat with @martinvol and it appeared, that the main blocker is that final invitation text is too long for an SMS - https://github.com/celo-org/celo-monorepo/pull/1983/files#r374251754. @nityas  can we get this text under 160 or what should we do?

Invite w/o escrowed payment
Welcome to the Celo Wallet! Get the app at https://celolink.page.link/SZ1EeUaCHdrDsd9x9 and enter your Celo Code: RDfQlvthTUOf1VtZIJawRBU+dWCKJyndP/M0emKwSjc= (158 chars)
Invite w/ escrowed payment
I've paid you with Celo! Redeem with the Celo Wallet at https://celolink.page.link/SZ1EeUaCHdrDsd9x9 and Celo Code: RDfQlvthTUOf1VtZIJawRBU+dWCKJyndP/M0emKwSjc= (160 chars)


fix(ocm): prevent error when ManagedCluster.status doesn't contain a 'clusterClaims' field

Error:
When a ManagerCluster API response lacks the status.clusterClaims then the error Cannot read properties of undefined (reading 'find') is displayed on the UI.
Fix: Just make sure that clusterClaims is checked before calling find ;

/lgtm
/approve


Mobile wallet Dialog pages, Close icon (X button) needs to be removed

Description:
In the mobile wallet-->Outpiut page, Dialog pages have both the "OK" button and also Close icon (X button).
Please refer to the below screenshot.
Expected Result:
In Mobile wallet Dialog pages, Close icon (X button) needs to be removed which looks awkward.

The close icon is needed definitely. In small device screen, there is a scroll and users can't see ok button. In this case, close button is useful. If not, they always need to scroll down and click ok to close the dialog, which is not user friendly.
@bayk @vinayaga07


Make scripts fail if any of bash cmds fail

Description of the Change

If one of internal commands inside any of bash scripts fail (exit with non-0 code), the whole script continues, giving a false hope that bindings were built.
Benefits

This change prevents this behavior, and make the whole script fail if any subcommand fails.
Possible Drawbacks

None.

please, check if CI succeded


Remove footer from source maps and eval

What kind of change does this PR introduce?
feature removing

Did you add tests for your changes?
Not added, but modified

If relevant, link to documentation update:
N/A

Summary
There wasn't good use case to include a footer in source maps, and it didn't work properly with some development tools (e.g. Chrome Devtools).

Does this PR introduce a breaking change?
As far as I know, there weren't tools which used this feature, so it's not a breaking change.

Other information

Thanks


fixup 'make liteeth work' PR

@enjoy-digital
Sorry,  but the commit 5e0fe1b4f27f ('dts: use feasable model and compatible strings, drop sifive mentions') "throws the baby out with the bathwater". It drops necessary "simple-bus" 'compatible' value together with SiFive-related values. As a result liteeth driver is never probed.
This PR fixes the problem.

Thanks!


API changes report for Config

The review of API changes for the Config library since 0.3.1 version: https://abi-laboratory.pro/java/tracker/timeline/config/
Hope it will be helpful for users and maintainers of the library. The report is generated by the https://github.com/lvc/japi-tracker tool for jars found at http://central.maven.org/maven2/com/typesafe/config/ according to https://wiki.eclipse.org/Evolving_Java-based_APIs_2.
Thank you.

Thanks for the report!


added {{reading_time}} for estimated reading time.

Hello, The default Ghost theme -Casper includes estimated reading time. Vapor does not and I find this feature useful so I added it. https://themes.ghost.org/docs/reading_time
Here's a preview: https://i.imgur.com/5bQEdIH.png

Jason, thanks for this PR! It's a cool feature and I'm glad you included it. Merging into master!


Bump build number

I accidentally pushed to upstream/master and triggered a build without a PR. Now I wonder whether this lead to some weird behaviors on conda-forge.

Checklist

[x] Used a fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Re-run stiff ODE benchmarks

It's having troubles with the ODEInterface.jl binaries. @christopher-dG might need to help with getting those weird Fortran binaries to work.

Can you link me a run where something blows up? I installed Julia + ODEInterface in a minimal container and it passes its own tests as long as gfortran is installed.

Oh sorry, this is where it was seen: https://github.com/SciML/DiffEqBenchmarks.jl/pull/118

!rebuild StiffODE
We were missing basic build tools like make causing some packages to not build. I added build-essential which will hopefully take care of that.

!rebuild StiffODE
I think this ought to at least mostly work now

!rebuild StiffODE
aaaaand again, since the package renaming mucked some things up

!rebuild StiffODE
I think I got it...

Never mind....

close 😆

!rebuild StiffODE/VanDerPol.jmd


Developersdevelopersdevelopers

I think Travis is going to throw up on Ruby 2.3 and/or 2.4. The problem isn't the library, it's the tests. Investigating when I can get a straight answer out of Travis.

Oh my, it seems like I'm going to be arrested from all of these offenses
17 files inspected, 719 offenses detected
RuboCop failed!

Haha....looks like I have some cleanup to do.
This all looks great to me though! 👍 once Travis decides to cooperate.

I have a Rubocop PR pending too.

LOOKS GOOD 👍


[BUG] Release pipeline is broken with PR #51

Ref: https://github.com/pixincreate/PixeneOS/actions/runs/11371490427/job/31633678166

I only have 24 hours a day 🥲


[stable/mongodb] Add missing metrics port for standalone svc

What this PR does / why we need it:
If the replicaSet is disabled, metrics port is not added to the default service. I am using prometheus operator that uses service labels and the default service does not expose the metrics ports.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[X] DCO signed
[X] Chart Version bumped

@victorboissiere yet patched there #9560, voila :-)

Closing the pull request as the feature has already been implemented in the other pull request. Thanks!


FlashDim Unavailable in QS after update.

Hello!
Let me first say, FlashDim is amazing! I love the app and the amount of control I can get.
Alright, so here's the issue:
After I updated my phone to Android 14 QPR 2 Beta 1.1, the FlashDim QS tile now just shows "Unavailable", but when I enter the app, it works fine.
Any suggestions on how to fix would be greatly appreciated!
SteadyStatus21

Nevermind, the toggle fixed itself. Sorry about that.


How can we manage when user has disabled cookie from browser?

Hello
How can we manage when user has disabled cookie from browser?

Unfortunately, that's user choice


Copy settings between environments

Description of the new feature / enhancement
I'd like to be able to copy complex Keyboard Manager definitions between PCs.  Currently, I have to manually define all the settings at each PC.
Even just copying settings files from one PC to another would work, but I don't see any file modification stamps change when I define shortcuts.
Scenario when this would be used?
Setting up machines for my trainers' environments.
Supporting information
No response

@KevinLeeBuchcn while we work on more permanent solution, you can copy over the file  %LocalAppData%\Microsoft\PowerToys\Keyboard Manager\default.json to your other PCs. That should sync your layouts between devices.

Tracked in @3381


Make window move to active space upon alt-tab, instead of switching spaces.

This makes it much faster to send off quick messages and get back to what you were doing because:

You don't have to wait for the Spaces animation if the window is in a different space.
Alt-tabbing again brings you back to the same window you came from (as opposed to previously, where you might end up in a different window of the same app that happens to be in the space where Messenger is... or, if this doesn't happen and a space switch occurs, you have to wait on the animation again).

Personally I like it as it is now, which is more consistent with how the "Messages" app behaves. Any chance that this could be a preference instead of a default?

+1 for making it an option

I'd recommend something like this:
  if ([[NSUserDefaults standardUserDefaults] boolForKey:@"moves-with-active-space"]) {
    _window.collectionBehavior |= NSWindowCollectionBehaviorMoveToActiveSpace;
  }

Then power-users can defaults write to change the behaviour.
Note that since this PR was created, the app now sets the FullscreenPrimary flag to collectionBehavior, so you'll need to merge the flag, rather than set it.

Sounds reasonable, will update the PR. If at some point there are enough configurable options to merit a Preferences dialog, this should probably go there.


CI/CD Cluster (AWS) Does NOT allow batch/v1beta1 cronjobs to be deployed

Describe the bug
Currently, all data jobs that are deployed in the CI/CD cluster (AWS) use the batch/v1 Cronjob API irrespective of what template is used. This can be verified using kubectl and issuing the kubectl explain cronjobs command. The result printed to the terminal is
KIND:     CronJob
VERSION:  batch/v1

DESCRIPTION:
     CronJob represents the configuration of a single cron job.

FIELDS:
   apiVersion	<string>
     APIVersion defines the versioned schema of this representation of an
     object. Servers should convert recognized schemas to the latest internal
     value, and may reject unrecognized values.  Servers may infer this from the endpoint the client submits
     requests to. Cannot be updated. In CamelCase. More info:
     https://git.k8s.io/community/contributors/devel/sig-architecture/api-conventions.md#types-kinds

   metadata	<Object>
     Standard object's metadata. More info:
     https://git.k8s.io/community/contributors/devel/sig-architecture/api-conventions.md#metadata

   spec	<Object>
     Specification of the desired behavior of a cron job, including the
     schedule.  More info:
     https://git.k8s.io/community/contributors/devel/sig-architecture/api-conventions.md#spec-and-status

As per the above, the k8s server converts every cronjob object to the batch/v1 API schema, irrespective of what API is used to create it. As a result, any Control Service integration test that relies on deploying a job with the batch/v1beta1 API, would fail.
It is also possible to verify it by running DataJobDeploymentCrudIT integration test from a local environment connected to the CI/CD cluster.
Steps To Reproduce
Steps to reproduce the behavior:

Open the DataJobDeploymentCrudIT integration test in an IDE
Configure the IDE to use the CI/CD cluster
Place a break point somewhere in the testDataJobDeploymentCrud test case, preferably before the part, where the deployment is deleted.
Run the integration test in debug mode.
Check the logs for the name of the test data job created and deployed by the integration test.
Use kubectl to check the cronjob of the data job in the CI/CD cluster. Command is: kubectl get cronjob <job-name> -oyaml
Check the output and compare it with the logs of the integration test.

Expected behavior
Both batch/v1beta1 and batch/v1 cron jobs can be deployed in the CI/CD cluster.
Additional context
To reproduce the behaviour, one needs access to the CI/CD cluster.

I see two approaches that can be taken here. Either to somehow enable beta API cronjobs to be created in the cluster, or do a reverse-switch, i.e., the default version to be batch/v1 and the flag to be set to enable batch/v1beta1 to unblock the CI/CD.
Any thoughts on this, @tozka, @murphp15?

Have you tested it on a local kind cluster, does the same issue happen there?

Yes, our CICD uses 1.22 version of kubernetes. At this point v1beta1 is officially deprecated in favour of v1 version of Cron. So it's storage version is set to v1. That means any Cron resource being created (or updated) will automatically become v1.
It's explained here:
https://kubernetes.io/docs/tasks/extend-kubernetes/custom-resources/custom-resource-definition-versioning/#writing-reading-and-updating-versioned-customresourcedefinition-objects
I think we should read carefully
https://kubernetes.io/docs/reference/using-api/deprecation-policy/  as well in order to see how to handle both Cron v1 and batch/v1beta1 properly.
In order to test we might need an older version of Kubernetes. You can do that locally with kind. For automated testing in the CICD I am not sure I want us to create another Kubernetes cluster just for this feature especially since v1beta1 would be removed in 3 minor versions (1.26. probably) of Kubernetes.. So we might forgo end to end test here unless someone comes with some cheap idea.

Judging by the documentation, the v1beta1 API was dropped in 1.25, https://kubernetes.io/docs/reference/using-api/deprecation-guide/#v1-25.

So, @tozka, @murphp15, can we then consider the comments in #1580, requesting a new integration test to be introduced, as irrelevant as long as the change is covered with unit tests and instructions how to test it locally are added to the PR?

Yeah makes sense!

Triaged. This is fixed.


[Bug] Compiling Llama-2 with q3f16 and failed with errors KeyError: IterSplit(IterMark(v1, extent=T.int64(4096)), lower_factor=T.int64(1), extent=T.int64(4096), scale=T.int64(1))

🐛 Bug
Compiling Llama-2 with q3f16 and failed with errors KeyError: IterSplit(IterMark(v1, extent=T.int64(4096)), lower_factor=T.int64(1), extent=T.int64(4096), scale=T.int64(1))
To Reproduce
Steps to reproduce the behavior:

Compile the model according to the documents.
git clone https://huggingface.co/meta-llama/Llama-2-7b-chat-hf
python3 -m mlc_llm.build --model Llama-2-7b-chat-hf --target metal --quantization q3f16_1

Error Messages:
Save a cached module to dist/Llama-2-7b-chat-hf-q3f16_1/mod_cache_before_build.pkl.
Traceback (most recent call last):
File "/Users/jimmy/anaconda3/envs/tvm/lib/python3.8/runpy.py", line 194, in _run_module_as_main
return _run_code(code, main_globals, None,
File "/Users/jimmy/anaconda3/envs/tvm/lib/python3.8/runpy.py", line 87, in _run_code
exec(code, run_globals)
File "/Users/jimmy/Code/mlc-llm/mlc_llm/build.py", line 13, in
main()
File "/Users/jimmy/Code/mlc-llm/mlc_llm/build.py", line 10, in main
core.build_model_from_args(parsed_args)
File "/Users/jimmy/Code/mlc-llm/mlc_llm/core.py", line 655, in build_model_from_args
build(mod, args)
File "/Users/jimmy/Code/mlc-llm/mlc_llm/core.py", line 514, in build
mod_deploy = dl.ApplyDefaultSchedule(  # pylint: disable=not-callable
File "/Users/jimmy/.local/lib/python3.8/site-packages/tvm-0.12.dev1610+gceaf7b015-py3.8-macosx-11.0-arm64.egg/tvm/ir/transform.py", line 238, in call
return _ffi_transform_api.RunPass(self, mod)
File "tvm/_ffi/_cython/./packed_func.pxi", line 332, in tvm._ffi._cy3.core.PackedFuncBase.call
File "tvm/_ffi/_cython/./packed_func.pxi", line 263, in tvm._ffi._cy3.core.FuncCall
File "tvm/_ffi/_cython/./packed_func.pxi", line 252, in tvm._ffi._cy3.core.FuncCall3
File "tvm/_ffi/_cython/./base.pxi", line 182, in tvm._ffi._cy3.core.CHECK_CALL
File "/Users/jimmy/.local/lib/python3.8/site-packages/tvm-0.12.dev1610+gceaf7b015-py3.8-macosx-11.0-arm64.egg/tvm/_ffi/base.py", line 476, in raise_last_ffi_error
raise py_err
File "tvm/_ffi/_cython/./packed_func.pxi", line 56, in tvm._ffi._cy3.core.tvm_callback
File "/Users/jimmy/.local/lib/python3.8/site-packages/tvm-0.12.dev1610+gceaf7b015-py3.8-macosx-11.0-arm64.egg/tvm/ir/transform.py", line 307, in _pass_func
return inst.transform_module(mod, ctx)
File "/Users/jimmy/.local/lib/python3.8/site-packages/tvm-0.12.dev1610+gceaf7b015-py3.8-macosx-11.0-arm64.egg/tvm/dlight/base/transform.py", line 64, in transform_module
sch = _apply_rules(func, target, self.rules, tunable=False)
File "/Users/jimmy/.local/lib/python3.8/site-packages/tvm-0.12.dev1610+gceaf7b015-py3.8-macosx-11.0-arm64.egg/tvm/dlight/base/transform.py", line 80, in _apply_rules
space = rule.apply(func, target, tunable)
File "/Users/jimmy/.local/lib/python3.8/site-packages/tvm-0.12.dev1610+gceaf7b015-py3.8-macosx-11.0-arm64.egg/tvm/dlight/gpu/gemv.py", line 191, in apply
is_inner_reduction = normalize(sch, block_info)
File "/Users/jimmy/.local/lib/python3.8/site-packages/tvm-0.12.dev1610+gceaf7b015-py3.8-macosx-11.0-arm64.egg/tvm/dlight/gpu/gemv.py", line 122, in normalize
is_inner_reduction = iter_to_info[inner_axis].kind == "R"
KeyError: IterSplit(IterMark(v1, extent=T.int64(4096)), lower_factor=T.int64(1), extent=T.int64(4096), scale=T.int64(1))
Expected behavior
Expected compilation is successful.

@JimmyLi-Network Could you compile the model successfully after the fix is merged?

Hi,
I have merged and updated the repo of TVM. But the problem remains.
KeyError: IterSplit(IterMark(v1, extent=T.int64(11008)), lower_factor=T.int64(1), extent=T.int64(11008), scale=T.int64(1))
Do I need to merge the mlc-llm as well?

The issue is reported as "fixed" in the other thread: https://github.com/mlc-ai/mlc-llm/issues/1005. To double check if you have installed the latest version of TVM Unity, @JimmyLi-Network could you please follow the "Step 3" in here and take a look at the git commit version: https://llm.mlc.ai/docs/install/tvm.html#validate-installation?

The issue is reported as "fixed" in the other thread: #1005. To double check if you have installed the latest version of TVM Unity, @JimmyLi-Network could you please follow the "Step 3" in here and take a look at the git commit version: https://llm.mlc.ai/docs/install/tvm.html#validate-installation?

Thanks! I git fetch from the Tvm repo and installed it again.
My GitHub commit time is GIT_COMMIT_TIME: 2023-09-18 20:10:22 -0400
Is that too old?
Thanks,

Yes. Please follow the steps here to reinstall the latest packages: https://llm.mlc.ai/docs/install/tvm.html#option-1-prebuilt-package

The issue is reported as "fixed" in the other thread: #1005. To double check if you have installed the latest version of TVM Unity, @JimmyLi-Network could you please follow the "Step 3" in here and take a look at the git commit version: https://llm.mlc.ai/docs/install/tvm.html#validate-installation?

Thanks! I git fetch from the Tvm repo and installed it again.
My GitHub commit time is GIT_COMMIT_TIME: 2023-09-18 20:10:22 -0400
Is that too old?
Thanks,

Hi,
I built from the source of relax/tvm. It works and no errors.
Thanks.

Thanks! Seems that this issue has been resolved


[Runtime] Fix premature snapshot destruction in StableAddressConcurrentReadableHashMap::find.

Mark the relevant snapshot methods as ref-qualified (adding a & after the parameter list) so the compiler will enforce this for us and forbid calling them on temporaries.
rdar://72997638

Good idea, added comments.

Good idea, added comments.

@swift-ci please test

@swift-ci please test


Copy layout/styles from main website

Cool, can you add a docs link to the top as well?

Added docs link

LGTM

@jvatic looks good :smile: