snakemake exit with MissingOutputException when using --immediate-submit

Snakemake version
8.16.0

Describe the bug

I am currently attempting to execute Snakemake with the --immediate-submit flag in order to submit all jobs to a SLURM cluster simultaneously, without waiting for the presence of input files for subsequent jobs. Previously, this flag worked seamlessly for me on the same SLURM cluster using Snakemake v5.31. However, after migrating to Snakemake v8.16, I noticed that the immediate submit functionality no longer functions as expected. Despite successfully adapting my Snakefile and configuration to be compatible with the changes in version 8, I can only run a simple workflow without the --immediate-submit flag. When immediate-submit is enabled, Snakemake appears to continually check for input files for the next step, even while a submitted job is still running. Consequently, job submission terminates prematurely with a MissingOutputException. I wonder whether this behaviour is a bug or if there’s a specific configuration in version 8 that I need to set? Any advice on this would be much appreciated. Thanks.
Logs
Execute snakemake without immediate-submit flag work fine
Command executed: snakemake -s snakefile --profile slurm
Using profile slurm for setting default command line arguments.
Building DAG of jobs...
Using shell: /usr/bin/bash
Provided remote nodes: 999
Job stats:
job      count
-----  -------
all          1
step1        1
step2        1
step3        1
total        4

Select jobs to execute...
Execute 1 jobs...

[Wed Jul 31 10:37:52 2024]
rule step1:
    input: input.txt
    output: output1.txt
    jobid: 3
    reason: Missing output files: output1.txt
    resources: mem_mb=2000, mem_mib=1908, disk_mb=1000, disk_mib=954, tmpdir=<TBD>, cpus=1

cat input.txt > output1.txt
Submitted job 3 with external jobid '1089'.
[Wed Jul 31 10:38:02 2024]
Finished job 3.
1 of 4 steps (25%) done
Select jobs to execute...
Execute 1 jobs...

[Wed Jul 31 10:38:02 2024]
rule step2:
    input: output1.txt
    output: output2.txt
    jobid: 2
    reason: Missing output files: output2.txt; Input files updated by another job: output1.txt
    resources: mem_mb=2000, mem_mib=1908, disk_mb=1000, disk_mib=954, tmpdir=<TBD>, cpus=1

sleep 20; head -n2 output1.txt > output2.txt
Submitted job 2 with external jobid '1090'.
[Wed Jul 31 10:38:32 2024]
Finished job 2.
2 of 4 steps (50%) done
Select jobs to execute...
Execute 1 jobs...

[Wed Jul 31 10:38:32 2024]
rule step3:
    input: output2.txt
    output: output3.txt
    jobid: 1
    reason: Missing output files: output3.txt; Input files updated by another job: output2.txt
    resources: mem_mb=2000, mem_mib=1908, disk_mb=1000, disk_mib=954, tmpdir=<TBD>, cpus=1

head -n1 output2.txt > output3.txt
Submitted job 1 with external jobid '1091'.
[Wed Jul 31 10:38:42 2024]
Finished job 1.
3 of 4 steps (75%) done
Select jobs to execute...
Execute 1 jobs...

[Wed Jul 31 10:38:42 2024]
localrule all:
    input: output3.txt
    jobid: 0
    reason: Input files updated by another job: output3.txt
    resources: mem_mb=2000, mem_mib=1908, disk_mb=1000, disk_mib=954, tmpdir=/tmp, cpus=1

[Wed Jul 31 10:38:42 2024]
Finished job 0.
4 of 4 steps (100%) done
Complete log: .snakemake/log/2024-07-31T103751.980121.snakemake.log`

Execute snakemake with immediate-submit flag failed to submit all jobs and exit prematurely with the following error:
Command executed: snakemake -s snakefile --profile slurm --immediate-submit --notemp
Using profile slurm for setting default command line arguments.
Building DAG of jobs...
Using shell: /usr/bin/bash
Provided remote nodes: 999
Job stats:
job      count
-----  -------
all          1
step1        1
step2        1
step3        1
total        4

Select jobs to execute...
Execute 1 jobs...

[Wed Jul 31 10:47:31 2024]
rule step1:
    input: input.txt
    output: output1.txt
    jobid: 3
    reason: Missing output files: output1.txt
    resources: mem_mb=2000, mem_mib=1908, disk_mb=1000, disk_mib=954, tmpdir=<TBD>, cpus=1

cat input.txt > output1.txt
Submitted job 3 with external jobid '1092'.
Waiting at most 5 seconds for missing files.
[Wed Jul 31 10:47:33 2024]
Finished job 3.
1 of 4 steps (25%) done
Select jobs to execute...
Execute 1 jobs...

[Wed Jul 31 10:47:33 2024]
rule step2:
    input: output1.txt
    output: output2.txt
    jobid: 2
    reason: Missing output files: output2.txt; Input files updated by another job: output1.txt
    resources: mem_mb=2000, mem_mib=1908, disk_mb=1000, disk_mib=954, tmpdir=<TBD>, cpus=1

sleep 20; head -n2 output1.txt > output2.txt
Submitted job 2 with external jobid '1093'.
Waiting at most 5 seconds for missing files.
MissingOutputException in rule step2 in file /mnt/scratch2/users/3056021/sm_centos8/immSub/snakefile, line 11:
Job 2  completed successfully, but some output files are missing. Missing files after 5 seconds. This might be due to filesystem latency. If that is the case, consider to increase the wait time with --latency-wait:
output2.txt (missing locally, parent dir not present)
Exiting because a job execution failed. Look above for error message
Complete log: .snakemake/log/2024-07-31T104731.768237.snakemake.log
WorkflowError:
At least one job did not complete successfully.

Minimal example

Content of the snakefile:
rule all:
  input:
      'output3.txt'
rule step1:
  input:
      'input.txt'
  output:
      'output1.txt'
  shell:
      'cat {input} > {output}'
rule step2:
  input:
      'output1.txt'
  output:
      'output2.txt'
  shell:
      'sleep 20; head -n2 {input} > {output}'
rule step3:
  input:
      'output2.txt'
  output:
      'output3.txt'
  shell:
      'head -n1 {input} > {output}'

Additional context

config.yaml file:
executor: cluster-generic
jobs: 999
default-resources: [cpus=1, mem_mb=2000]
cluster-generic-submit-cmd: "./slurm/sbatch.py {resources.cpus} {resources.mem_mb} {rule} {dependencies}"
max-status-checks-per-second: 10
rerun-incomplete: True
scheduler: greedy
keep-going: True
printshellcmds: True
show-failed-logs: True

sbatch.py
#!/usr/bin/env python3

import os
import sys
import subprocess

from snakemake.utils import read_job_properties

# last command-line argument is the job script -- required to submit jobs to slurm
jobscript = sys.argv[-1]

cpu = sys.argv[1]
mem = sys.argv[2]
rulename = sys.argv[3]
dependencies = set(sys.argv[4:-1])

cmdline = ["sbatch --chdir ./log_hpc --output=%j.out --error=%j.err --nodes=1 --parsable"]
cmdline.append("--job-name="+rulename)
cmdline.append("--ntasks="+cpu)
cmdline.append("--mem="+str(mem))
cmdline.append("--partition=k2-hipri --time=2:59:00")

if dependencies:
    cmdline.append("--dependency")
    cmdline.append( "afterok:" + ",".join(dependencies))

cmdline.append(jobscript)

# Constructs and submits
cmdline = " ".join(cmdline)
print (cmdline)
os.system(cmdline)

With Snakemake v8, we changed to a plugin scheme. There are plenty of plugins, incl. an executor plugin for SLURM. See the catalogue. The SLURM plugin has extensive documentation - although I admit, it merits improvement (as always).
Please report potential bugs to the executor repo if they occur.


Error Handling in Views / Failing Gracefully / Coding Defensively

JavaScript errors in views currently can break an app irrecoverably (see https://jsfiddle.net/4u6x45fe/3/).
Admittedly, the ideal solution is to only write error-free code. But the reality is that there will at some point be an error that makes its way to the user.
As a non-expert JavaScript developer I'm wondering: what is a good way to code defensively in order to avoid breaking the whole app with a failing view?
Is wrapping my views in try/catch blocks a good idea or a bad idea?
I'm not suggesting that domvm itself should try/catch the view functions - I think that wouldn't be a good idea, unless the error can be re-raised in a way that doesn't break the app (after redraw is complete?).
I'm just looking at how to set up a single-page-application around domvm that fails in a reasonably graceful way when I shoot myself in the foot.
Anyone willing to share how they deal with minimizing the effect of errors in their apps?
ETA: Ok, what is written at https://www.sitepoint.com/proper-error-handling-javascript/ tells me that plastering try/catch all over my own view code is probably not a good idea. If I reraise the error in the view's catch block, execution stops and the app breaks just as before. If I don't reraise the error, I'm introducing a bad code smell that makes debugging unnecessarily hard.

I'm not sure why this example screws up as it does while this seems to do ok:
https://jsfiddle.net/4u6x45fe/7/
In general, typed and compiled JS variants like TypeScript [1], Flow [2], Google Closure annotations [2] help prevent various errors like these. At runtime having most of these checks end up degrading performance (definitely in V8 where try/catch will not be JIT-optimized) and instrumenting every place with checks leads to code which is difficult to read.
[1] https://www.typescriptlang.org/
[2] https://flowtype.org/
[3] https://developers.google.com/closure/compiler/docs/js-for-compiler

Fair enough.  I'm personally leaning towards TypeScript since that seems least invasive/opinionated.
I can still accidentally mess things up, though. E.g. if I don't verify my API actually delivers data in the format expected by the SPA. Admittedly, for my own API, I could figure out how to automatically generate properly type-annotated code and then get great value out of type checking.

I'm not sure why this example screws up as it does while this seems to do ok:

Here, this makes it fail in what I believe to be the same way mine failed: https://jsfiddle.net/4u6x45fe/10/
I seem to get additional DOM nodes every second iteration.

interesting, thanks. i'll need to look into this.

so there's a fairly elegant solution here which would require an extra layer, but would work.
render() can return false instead of a template to indicate no changes, so you can wrap the template generation in a try/catch that console.logs and returns false: https://jsfiddle.net/4u6x45fe/11/
this is the simplest user-space solution. i can technically do the same thing within the lib as per https://github.com/leeoniya/domvm/commit/1b96a14e3a1bf510f839169e386e69059f692953, and it seems from initial benchmarks that there's no penalty, though I'm very hesitant to bake this into the lib as default behavior and discover otherwise in the future.

gonna close this, but feel free to discuss further.

though I'm very hesitant to bake this into the lib as default behavior and discover otherwise in the future

Given that's it's trivially easy to simply wrap every render-function in a generic try/catch/log function, I wouldn't.
return function() {
    view...
    };

vs.
return catchAll(function() {
    view...
    });

I see. Yes, I've been playing around in domvm/src/view.js as well. Wrapping render() seemed to be the most elegant/simple thing to do in order to stop the DOM from exploding after an error occurs in a view function.
I agree that it's not a good idea to bake this naive solution into domvm itself since catching and logging errors like that is too specific/opinionated. There might be other patterns we haven't thought of that would work better, or be more appropriate in particular situations.
@lawrence-dol It looks like we've been thinking the same thing. Though, having an explicit wrapper means that I need to never forget to wrap things which might blow up.
Maybe there is some way that allows to provide a wrapper that is automatically applied to all render() functions? (I suppose an application could "monkey-patch" domvm.view to automatically apply the wrapper.)
Maybe it should be possible to register a renderErrorHandler() with domvm that will be called from the catch block?
Maybe it's just a topic for the documentation? Saying that domvm requires error-free view functions really isn't unreasonable. That leaves taking measures to ensure error-free views is up to the application.
I'm just a clueless person with a talent to break things. :) Since the goal of domvm is to stay flexible, please never feel rushed to change things because I found another way to wreak havoc out of ignorance.

Attempt to do error-handling by monkey-patching domvm.view at https://jsfiddle.net/4u6x45fe/15/.

@grumpi : Putting something like this in your setup code should work:
vm.view=function() {
    vm._viewFunction=vm.view;
    vm.view=function() {
        try { vm._viewFunction.apply(Array.prototype.slice.call(arguments)); }
        catch(thr) { console.error("View exception:",thr); }
    }
};

@grumpi that monkey patch passes most but not all the tests. if you'd like this functionality baked in, just do a custom domvm build (which won't break anything).

I just realized that my monkey patch in the jsfiddle only seems to apply the wrapper function to the top-level views.
Turns out I needed to patch createView() instead of domvm.view() in order to end up wrapping each and every single subview. That appears to work but isn't a monkey patch anymore since I need to change domvm/src/view.js.
Yes, looks like this is a case for a custom build at this point (either that, or I'm missing another obvious point where I can try monkey-patching). If I'm doing a custom domvm build, I suppose the most reasonable place to patch is right around the render() call. :)
When I change domvm to internally call domvm.view() instead of createView(), I can apply the monkey patch successfully. This feels quite reasonable to me, since it lets me keep the app-specific error-handling out of domvm.

I suppose the most reasonable place to patch is right around the render() call. :)

it's already in the code, just uncomment: https://github.com/leeoniya/domvm/commit/1b96a14e3a1bf510f839169e386e69059f692953

Would it not make more sense to leave this as the developer's choice, using something like:
domvm.view.extend(function(vm) {
    let renderFnc=vm.render;
    vm.render=function(api,vm,model,key) {
        try { return renderFnc(api,vm,model,key); }
        catch(exc) {
            console.log(exc);
            return false;
            }
        }
    });

Which keeps the actual error handling in application space?


Use of Joda Time

Wonderful writeup, I really appreciate the effort you've put into this example and the blog post. I was wondering though, why are you using Joda Time library instead of the built-in LocalDate and LocalTime classes through the java.time package that comes with Java SE 8? I am refactoring some of your classes and integrating them into my project, and Intellij is throwing a fit because I haven't added the dependency to my build.gradle. I don't have a problem doing so, but I was wondering if there is a specific reason for this? Is the Joda Time API easier to use? Thank you very much!

@jsbUSMC
Thank you for the kind words.
Good point. I will remove Joda Time dependency and refactor code to use Java 8 time package instead.
Note: Main reason behind using Joda Time is my familiarity with that library.

Thank you once again for your effort with the blog and the code samples


datagram socket was larger than the internal message buffer

Check CONTRIBUTING guideline first and here is the list to help us investigate the problem.
Describe the bug
Im new to fluentd and we have setup this up to collect logs for 300+ servers but the aggregator is now reporting an error regarding UDP stocket message limit. I assume this is due to the large number of servers. Is there a way to increase size limit for UDP this in the config?
Your Environment
Fluentd or td-agent version: 3.7.1
Operating system: Windows Server 2016
Your Configuration
This config is intended for use as a site log-aggregator.
No authentication is performed, though this can be configured using TLS
or shared secrets. Do not expose to untrusted networks.
http://localhost:9880/<tag>?json=<json>
<source>
  @type http
  port 9880
  bind <IP_ADDRESS>

  @label @FORWARD_INFOSEC
</source>

$ echo <json> | fluent-cat <tag>
<source>
  @type forward
  bind 24224
  bind <IP_ADDRESS>

  source_address_key source_address
  source_hostname_key source_host

  @label @FORWARD_INFOSEC
</source>

syslog - udp
<source>
  @type udp
  port 5140
  bind <IP_ADDRESS>

  <parse>
    @type none
  </parse>

  source_address_key source_address
  source_hostname_key source_host
  tag syslog

  @label @FORWARD_INFOSEC
</source>

syslog - tcp
<source>
  @type tcp
  port 5140
  bind <IP_ADDRESS>

  <parse>
    @type none
  </parse>

  source_address_key source_address
  source_hostname_key source_host
  tag syslog

  @label @FORWARD_INFOSEC
</source>

fortigate syslog - udp
<source>
  @type udp
  port 5141
  bind <IP_ADDRESS>

  <parse>
    @type none
  </parse>

  source_address_key source_address
  source_hostname_key source_host
  tag syslog:fortigate

  @label @FORWARD_INFOSEC
</source>

fortigate syslog - tcp
<source>
  @type tcp
  port 5141
  bind <IP_ADDRESS>

  <parse>
    @type none
  </parse>

  source_address_key source_address
  source_hostname_key source_host
  tag syslog:fortigate

  @label @FORWARD_INFOSEC
</source>

<label @FORWARD_INFOSEC>
  <match **>
    @type forward
    heartbeat_type transport

  compress gzip

    transport tls
    tls_ca_cert_path C:\opt\td-agent\tls\ca.pem
    tls_client_cert_path C:\opt\td-agent\tls\*************.pem
    tls_client_private_key_path C:\opt\td-agent\tls\*************-key-enc.pem
    tls_client_private_key_passphrase "**************"

    <server>
      Server
      port 24224
    </server>
  </match>
</label>

Don't forward fluent logs
<label @FLUENT_LOG>
  <match fluent.**>
    @type stdout
  </match>
</label>

Don't forward unknown logs
<match **>
  @type null
</match>

Your Error Log
2020-06-03 09:49:46 +0200 [error]: #0 unexpected error in processing UDP data error_class=Errno::EMSGSIZE error="A message sent on a datagram socket was larger than the internal message buffer or some other network limit, or the buffer used to receive a datagram into was smaller than the datagram itself. - recvfrom(2)"
2020-06-03 09:49:46.695531000 +0200 fluent.error: {"error":"#<Errno::EMSGSIZE: A message sent on a datagram socket was larger than the internal message buffer or some other network limit, or the buffer used to receive a datagram into was smaller than the datagram itself. - recvfrom(2)>","message":"unexpected error in processing UDP data error_class=Errno::EMSGSIZE error=\"A message sent on a datagram socket was larger than the internal message buffer or some other network limit, or the buffer used to receive a datagram into was smaller than the datagram itself. - recvfrom(2)\""}
  2020-06-03 09:49:46 +0200 [error]: #0 suppressed same stacktrace

@jessey316
Maybe increasing the value of  message_length_limit parameter will fix this issue.
ref. https://docs.fluentd.org/input/udp#message_length_limit
td-agent version: 3.7.1(fluentd 1.10.2), the value of this parameter is 2048.
https://github.com/fluent/fluentd/blob/v1.10.2/lib/fluent/plugin/in_syslog.rb#L100

@jessey316 I'll close this issue after a while.

I'll close this issue without a response for a while.


Add a short demo code snippet in the README.md

Ideally with a png image visible in the README.md file

The snippet we currently have is not enough?
from jaxparrow import cyclogeostrophy, geostrophy

u_geos, v_geos = geostrophy(ssh_t=ssh,
                            lat_t=lat, lon_t=lon,
                            mask=mask)
u_cyclo, v_cyclo = cyclogeostrophy(ssh_t=ssh,
                                   lat_t=lat, lon_t=lon,
                                   mask=mask)


NexSlider online doc is not available

http://docs.iteadstudio.com/ITEADLIB_Arduino_Nextion/class_nex_slider.html

404 Not Found


Read a local XML config file

Hi,
this is actually not a real issue (maybe with the docs or not) but a question:
Is it possible to read a local config file and extend it as if I would do it with a Solr schema in a running Solr instance?
Imagine you would like to copy a local xml file, add a new field to it and use that new config file for a new collection within SolrCloud (unfortunately, the file needs to be existent before I can create a collection).
Is this possible somehow?

Hi David, I think this is possible in Solr using https://lucene.apache.org/solr/6_6_0//solr-solrj/org/apache/solr/client/solrj/impl/ZkClientClusterStateProvider.html#uploadConfig-java.nio.file.Path-java.lang.String-
I don't think SolrNet currently implements the equivalent of this.

Hi David, I think this is possible in Solr using https://lucene.apache.org/solr/6_6_0//solr-solrj/org/apache/solr/client/solrj/impl/ZkClientClusterStateProvider.html#uploadConfig-java.nio.file.Path-java.lang.String-
I don't think SolrNet currently implements the equivalent of this.


Fix [BUG] Icon copy notification text is not readable as the underlying element is visible.

Closes #9618
Fixes Issue
This PR fixes #9618
Changes proposed
[X] -Removed the unnecessary z index from navbar and now the Icon copy notification shows over the navbar as it should and text in the icon notification is readable properly.
Check List (Check all the applicable boxes)

[X] My code follows the code style of this project.

Screenshots
Before my changes

After my changes:

We are glad that you want to contribute the issue #9618  isn't labeled status:ready for dev so we can't accept the PR I would recommend you to read our contributing guidelines https://github.com/EddieHubCommunity/BioDrop/blob/main/CONTRIBUTING.md
I'm closing this Pull Request.

Thanks Sital for the updates, once this issue becomes ready-for-dev I'll submit a new PR again.


PLOT/GRATICULE misses the vertical graticules

Reported by Ansley B Manke on 29 Jun 2004 21:48 UTC
yes? use coads_climatology
yes? plot/grat sst[ ! horizontal but no vertical lines...
! This works correctly:
yes? plot/grat=(color=black) sstl=3
! or, this
yes? set mode grat
yes? plot sst[l=3]
MODE GRAT is new in v5.70 so this must be where
this got broken.
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/910

Comment by<EMAIL_ADDRESS>on 30 Jun 2004 21:04 UTC
fixed in routine set_graticule, checked in on
bug-fix branch FER_v_5_7_0_bug_fix_1


PKI: Properly reconcile certificates and keys

Currently, we only check if a secret has the right entries for a
certificate and key but we never check on the content of those entries.
This means that for example if a certificate expires or its config
changes in any other way, we will not re-create it.
This change:

Adds some code to verify a certificate and key against a config
Uses this when reconciling certificates
Cleans up the support/certs and
hontrol-plane-operator/controllers/hostedcontrolplane/pki packages to
not export methods that are not used in other packages

Ref https://issues.redhat.com/browse/HOSTEDCP-99

This is really cool. Thx! Love the tests.
One thing I wonder is whether/how to restart server processes that have their certificates renewed. But we should discuss that separately.
/lgtm


moved @click in radioBtn from input to label, same for checkboxBtn

Since Bootstrap hides the input, the target of clicks is the label. This results in the value not updating in some bootstrap themes. Should solve this issue:
https://github.com/yuche/vue-strap/issues/38

I can't reproduce this issue... I understand where you are coming from but since the input is wrapped by a  the browser takes care of sending the click event to the input (in case label was clicked).
I would love to see if you cna produce this error in jsfiddle (or let me know if this affects a particular browser, since it works fine for me on Chrome/Firefox).

I was also experiencing this problem, so I created a jsfiddle here: https://jsfiddle.net/3165Lh3y/
...and, of course, that works for me (Chrome Version 51.0.2704.103 (64-bit), Mac OS X 10.11.4), so I'm forced to back @amanpatel here.  Maybe there's some weird interaction with bootstrap's JS, which I'm also loading?  If I figure this out, I'll reply here with more details.

Sorry for not responding sooner. I had the issue in the bootstrap theme I was using (FlatLAB). Something in the theme's CSS or JS was preventing clicks on labels to bubble down to the inputs. Moving the click event to the input helped in the theme, so I thought I would share.
Thank you, @candu. I tried adding bootstrap's js into your jsfiddle, I tried messing with css, but can't replicate the issue outside of the theme, as I couldn't when I submitted it. What other js/css are you using that may have caused inputs to ignore clicks? Any particular theme package?

No theme package, though I do have a couple of third-party components (datetimepicker, tagsinput) installed.  I don't have that much time to investigate right now - I ended up applying your same fix (moving the click handler to the label), which addressed my problem.
I suppose this would break any code that (for whatever reason) also listens for the click event higher up the DOM tree with a handler that relies on knowing the original event target.  (It might be possible to listen on both the label and the input, and to handle that such that this use case doesn't break...)

A lot of changes from that, the radioBtn / checkboxBtn doesn't exist anymore, and added a new radio / checkbox elements.


Attributes not set in included files

When defining attributes in files included using include::, the attributes are not set. If the same attribute is defined in the parent file instead, it works as expected (and resolved in the included child file, too). For example:
parent.adoc:
:attribute: value

include::child.adoc[]

child.adoc:
{attribute}

This works as expected, i.e. the attrib. is resolved, and value is rendered in the resulting doc.
However:
parent.adoc:
include::child.adoc[]

child.adoc:
:attribute: value
{attribute}

Doesn't work. In this case, {attribute} is rendered.

This test provides this works: https://github.com/asciidoctor/jekyll-asciidoc/blob/9bb7aae945302c5e3e2eea978951f95bb600db40/spec/jekyll-asciidoc_spec.rb#L1091-L1096
Feel free to reopen if you can provide a test case that does not work.


Server Error Error: No matching bindings found for serviceIdentifier: Symbol(CmsEventBusService)

I believe the issue is related to the optional binding for CmsEventBusService (bind(CmsEventBusService).toDynamicValue(() => new Typed()).inSingletonScope().when(() => typeof window !== "undefined")). If that is the case, we won't be able to use initialize on the server side. Could you please take a look at it and provide an explanation?

Hey @joej0e!
You can call the initialize function from SPA SDK on the server side. Here is an example of using it inside getServerSideProps in the Next.js framework.
Could you please provide more information on how to reproduce the issue?

@hachok  How to go about it in angular 16?

@hachok Unfortunately, this is still an issue with the latest version the SDK, we are unable to render brXM content server-side in our mostly standard React applications. We are not using Next.js or any other similar framework.
We recently upgraded to the latest version from v17.x to resolve some security vulnerabilities stemming from that version of the SDK. In v17.x we had no server-side issues, however after the upgrade we now get the following error server-side:
Error: No matching bindings found for serviceIdentifier: Symbol(CmsEventBusService)
    at _validateActiveBindingCount (/Users/sforsberg/dev/mfe/homepage-mfe/node_modules/inversify/lib/planning/planner.js:63:23)
    at _getActiveBindings (/Users/sforsberg/dev/mfe/homepage-mfe/node_modules/inversify/lib/planning/planner.js:49:5)
    at _createSubRequests (/Users/sforsberg/dev/mfe/homepage-mfe/node_modules/inversify/lib/planning/planner.js:92:26)
    at /Users/sforsberg/dev/mfe/homepage-mfe/node_modules/inversify/lib/planning/planner.js:116:17
    at Array.forEach (<anonymous>)
    at /Users/sforsberg/dev/mfe/homepage-mfe/node_modules/inversify/lib/planning/planner.js:115:26
    at Array.forEach (<anonymous>)
    at _createSubRequests (/Users/sforsberg/dev/mfe/homepage-mfe/node_modules/inversify/lib/planning/planner.js:95:20)
    at Object.plan (/Users/sforsberg/dev/mfe/homepage-mfe/node_modules/inversify/lib/planning/planner.js:137:9)
    at /Users/sforsberg/dev/mfe/homepage-mfe/node_modules/inversify/lib/container/container.js:333:37

Once the app hydrates client-side all is good, but we load the impacted pages initially in a bad state (blank sections or entire pages are blank).
The issue seems to occurs soon after initialize gets called successfully. I confirmed that the page data is received from the brXM API through logging, but as soon as React starts to render the page we quickly run into the above error.
One workaround I have been able to do to get a partial render of the page (at least to render components that are not brXM-derived) is to use the NBRMode option described here: https://github.com/bloomreach/spa-sdk/blob/main/packages/react-sdk/README.md#non-blocking-render-mode-nbrmode in addition to removing some of the logic to fetch brXM data server-side. This is less than ideal as a lot of our content is dynamic, leading to other issues with CLS and things of that nature.
I see there is also this recently opened and somewhat related issue: #38
Do you have any suggestions on how to resolve this issue?
Thanks!


Promote start node and stop node methods to standard Libcloud compute API

This pull request adds new start_node(self, node) and stop_node(self, node) methods to the base compute NodeDriver class.
This means those two methods have been "promoted" and are now part of the standard Libcloud compute API.
Background, Reasoning
A big chunk of the existing compute drivers (18) already implemented this functionality as part of ex_start_node and ex_stop_node method so it only makes sense to promote those methods to be part of the standard compute API.
For backward compatibility reasons, existing extension methods are left in place and they continue to behave as they did in the previous releases. Those methods will be removed as part of a cleanup in a future major release.
TODO

[x] Changelog entry
[x] Upgrade notes entry

Codecov Report

Merging #1375 into trunk will decrease coverage by <.01%.
The diff coverage is 82.16%.

@@            Coverage Diff             @@
##            trunk    #1375      +/-   ##
==========================================
- Coverage   86.33%   86.32%   -0.01%
==========================================
  Files         372      372
  Lines       76232    76318      +86
  Branches     6979     6979
==========================================
+ Hits        65817    65885      +68
- Misses       7614     7633      +19
+ Partials     2801     2800       -1

Impacted Files
Coverage Δ

libcloud/compute/drivers/vsphere.py
0% <0%> (ø)
:arrow_up:

libcloud/compute/drivers/profitbricks.py
69.71% <100%> (+0.13%)
:arrow_up:

libcloud/compute/drivers/joyent.py
91.42% <100%> (+0.33%)
:arrow_up:

libcloud/compute/drivers/gridscale.py
87.17% <100%> (+0.09%)
:arrow_up:

libcloud/compute/drivers/nttcis.py
70.75% <100%> (+0.07%)
:arrow_up:

libcloud/compute/drivers/hostvirtual.py
81.81% <100%> (+0.42%)
:arrow_up:

libcloud/compute/drivers/ecs.py
84.9% <100%> (+0.1%)
:arrow_up:

libcloud/compute/drivers/openstack.py
84.3% <100%> (+0.05%)
:arrow_up:

libcloud/compute/drivers/cloudsigma.py
84.63% <100%> (+0.12%)
:arrow_up:

libcloud/compute/drivers/nephoscale.py
73.45% <100%> (+0.67%)
:arrow_up:

... and 12 more

Continue to review full report at Codecov.

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

Great idea! I skimmed through and it looks good to me @Kami.


token-cli: finish client conversion

this will be the last pr for converting the token cli to use the token client for everything. we will convert:

~transfer~
~create multisig~
~enable/disable transfer memo~
~set interest rate~
~gc~

closes #3482

the most recent push fixes the comment thread i marked resolved re: close authority in gc. unfortunately the behavior is somewhat more restricted than before. previously, it would attempt to transfer tokens out of accounts with separate close authority, but leave them open. now, it leaves them untouched
i drafted the logic for this and it made me somewhat queasy, so rather than wandering into the weeds and implementing it, ill wait for your review
            match (close_authority == owner, is_associated, amount == 0) {
                // owner authority, associated or auxiliary, empty -> close
                // owner authority, auxiliary, nonempty -> empty and close
                // separate authority, auxiliary, nonempty -> transfer
                // separate authority, associated or auxiliary, empty -> print warning
                // anything else, including a nonempty associated account -> unreachable
            }

because, see, i could do this. but what i might like better is to have it revoke the close authority and close the damn account. personally i find it very strange that setting close authority makes the owner unable to close the account, because having no close authority doesnt make the account uncloseable. but, on the other hand, this makes the logic even more complicated. and thats not even getting into the thought i had to gate authority revocation behind another cli flag...
anyway, the options are:

leave gc as-is, ignore noncloseable accounts entirely
impl the logic in the code block, replicate the previous transfer behavior
remove close authority and close the account, either unconditionally or behind a flag, either as its own transaction or by modifying the client more

i think i want to do 2 and let the user mess around with close authority on their own time, i dont think its that commonly used a feature

nevermind, writing the previous comment was enough to convince me i was right, so i implemented 2. i hope when youre back this saga is more amusing than confusing

re: close authority once again, today i learned an account owner cant even remove close authority from it! this feels wrong. nevertheless it means 3 was impossible anyway so 2 is the correct flow

alright i think i addressed everything! lmk if theres anything i missed or anything else


Switch to nanosockets to minimize allocations

I propose switching to nanosockets (https://github.com/nxrighthere/NanoSockets) for minimizing allocations. Currently riptide allocates quite bit of memory at least when data is received.
I might be able to implement this myself if I find the time.

Since I'm no longer actively working on Riptide I won't be able to do this, but if you do find the time, feel free to create a pull request 👍


The service example focuses on the least used google API

I do not understand why Google Books would be used as the example API when google calendar or gmail would be the more useful choice.
I have used several of these examples to different degrees and just get 404 errors.  The google documentation and this are inconsistent.

I have started a repo with examples at https://github.com/eRadical/google-api-php-client-examples
It's not much yet but I hope it will grow over time.

I think they use the books API because it's pretty simple and doesn't require much setup. To be fair, they also have examples for the Drive API which is pretty popular. You can also find some examples for the reseller API here: https://github.com/google/enterprise-deployments/tree/master/apps/php

Yeah, that's the main idea. For a little background, older versions of the lib (back when it was on subversion!) had lots of examples, which were mostly out of date - they gave a false sense of security. We (silvolu & I mainly) figured we weren't going to have time to keep all of them up to date, and we didn't have familiarity with every library, so we tried to create examples which show library features. From that point, it should be possible to figure out how to use a given service in PHP based on its general documentation and the API explorer. If you have seen a library feature which is missing an example do file a FR though - we want to make sure there is coverage for the types of task, if not the specific APIs.


Fix small typo in libraries.md

@monteiro-renato Please sign the CLA so we can review the PR. Thanks!

Done!

/fix:format


CMS-363: Add snowplow track to onRouteUpdate

Jira Ticket:
CMS-363
Description:

Add snowplow track to onRouteUpdate
Revert the initial idea since it was needed for single-page applications like Gatsby, where navigation between pages does not trigger a full-page reload

If you open a fresh tab and navigate to a page, will that fire onRouteUpdate? If not, do we need to also add any tracking to a hook like onInitialClientRender?

If you open a fresh tab and navigate to a page, will that fire onRouteUpdate? If not, do we need to also add any tracking to a hook like onInitialClientRender?

In that case useSnowplowTracking will do the job since it will run on mount and unmount.


Update plugin.json

marking , alert-true . To enable setting alert using this plugin

This will help resolve the issue-
https://github.com/trinodb/grafana-trino/issues/246
I have tested in my local and it works fine

@Amit-Hora can you submit the CLA? The list was last updated 2 days ago, so the next batch of updates is due in about 2 weeks: https://github.com/trinodb/cla/commits/master/contributors

@nineinchnick  i signed it and mailed it on 7th Sept , not sure why if it will take some more time to process ?

@cla-bot check

@cla-bot check


Add support for the vim-airline statusline/tabline.

Add theme support based off of the lightline theme.

Update README.md to document how to use the feature.

Fixes #2

awesome! thanks


use post.author vars in post header

What does this pull request aim to achieve? 💯
Uses correct variables from post.author for post header in feed. Resolves #566
Can't be reviewed currently in review environment quite yet but the results are:
Has photo:

No photo:

There is still the backend issue of these not being updated when the user updates their own or org logo but this will be dealt with separately.
🚨Before submitting this pull request🚨:
Please do NOT submit this PR if you have not done the following:

I have checked that no one else is working on similar changes.
I have read the latest README and understand the development workflow.
The name of this branch is: feature/<branch_name> (need to have cloned the repo not forked).
This branch is rebased/merged with the latest master.
There are no merge conflicts.
There are no console warnings and errors.
On the right hand side of this PR, I have linked the appropriate issue(s) under "Issues" and added a project progress under "Projects".

Closing to push to feature branch this actually will be reviewable.


AwaitingFundingProof to Active state transition

moving from signer setup to active breaks previous state tests

closing, was just pushed as reference


Linkify doesnt work in some cases

I am using it in my project. I found that it doesn't add 'className' option so I created a reproduction code. I used 'linkify-html' cuz it's html string but doesn't work in some cases. I don't know the specific cause.
reproduction
https://codesandbox.io/p/sandbox/epic-meninsky-w645vr?file=%2Fsrc%2Findex.js%3A10%2C80

Hi @donghoon-song, I'm getting a "Sandbox not found" message. Could you please check the permissions in codesandbox to make sure it has public access?

@nfrasser Sorry. I've updated it :)
https://codesandbox.io/p/devbox/58rq5l?migrateFrom=w645vr

Okay, I can see it now. So, to be clear, given the following input:
linkifyHtml('
hello google.com
Visit W3Schools.com!', {className: 'aaa'})

Do you expect the following output?

hello google.com
Visit W3Schools.com!

@nfrasser Yes I do!

Okay understood, unfortunately this kind of functionality is outside the scope of Linkify's functionality: Linkify is meant to find new links and wrap them with  tags. It is not meant to adjust existing HTML or links.
I assume you need to add a class to all  tags to apply CSS styling? Could you instead make a CSS rule that applies to all anchor tags? For example:
function App() {
  const html =' ...'
  return (

  )
}

.linkified a {
    /* insert CSS rules here */
}

@nfrasser Okay I see it is outside the scope. Thank you for your quick answer :)
I'd rather leave only links in html. I have to show the html from admin DB and it is also better to allow for the people who use an admin to write just links rather than whole a tags.


Circular Motion dynamics modifier for 3d particles

created a new DynamicsModifier and added it to the Flame tool and the GdxTests. The CircularMotion influencer adds a velocity to a particle in the activateParticles method and applies a force toward an axis specified by theta and phi, which is the axis of rotation for uniform circular motion.

A couple of people have asked about Circular Motion for 3d particles in the irc channel, so this is probably useful to people.


HTML formatter adding extra space between some elements

An instructor is running into an issue where she is adding subscripts using the ,, tags.  She adds a period immediately following one of those tags and an extra space is seen on the screen.
Here is an example of the html that is saved:

  You would be ripped apart exactly at the Schwarzschild radius when the mass of
  a black hole is about 2,000 Msun
  .  For lower mass black holes
  rrip  > rsch
  and one is torn before reaching the event horizon. If you want to see what it
  than 2,000 Msun
  . However, this delays your tidal disruption by only a small amount.

I think this is related to #305

is this the latest hax that's on hax.psu?

Yeah it is.

we'll need to see the actual steps to reproduce this as I haven't been able to in many months though I don't doubt it being possible. HAXcms repo is a bit ahead of when we had things in.

I wonder if it's because these register as valid HTML tags / children in a "slot" and therefor go through a different type of processing. it's not all tags that will cause this effect.

(I just witnessed this with strong tag)


Notes from mtg with SP, SC, RR

We went over the current synoptic site data workflow (Powerpoint attached), looking at how data flows between levels (Raw, L0, L0_normalize, L1a).

Minor note: include original units in L0_normalize

Talked about the idea of L1a using ‘output templates’ to generate its outputs

Concern from Roy: sin of omission, want to be able to QAQC all outputs in one place. Ben: we will pass all through in a single table, too #36
Do we like this enough to merge and make the default way forward? #30

A lot of concern about what explicit guarantees are made for each data level, in particular L1a, which currently has ONLY out of bounds checking but nothing else.

This is complicated, tricky problem; philosophical differences
Tension between more QAQC and more features versus making standardized data—even if imperfect and limited—available sooner for users
Opportunity cost of data not being available!

How do we handle versioning?

Ben showed one way based on hash, Roy uses dates which is more intelligible for users
Do this?

We could assign a globally unique row identifier to everything, letting QAQC be done later

Do this?

Need to resolve how we handle metadata. Complicated but also straightforward, hopefully #31
Idea of a paper down the line; there’s also the ESS-DIVE community workshop in November

I think can be closed.


certipy auth: Output LM hash

In this PR, the LM hash is extracted from the PAC_INFO_BUFFER. If it is zeroed out, the empty LM hash is printed. This is helpful, because Impacket-based tools usually require the hash in the LM:NT format.

Hello @rtpt-erikgeiser Sorry for the late reply. I really like this idea, even though the LM hash shouldn't matter in most cases, as you can do :NT for all the Impacket scripts, I've seen, i.e. leaving out the LM part. Certipy also allows you to just specify NT. But if you'd like to recreate this pull request for the new structure of Certipy (merge conflicts), I'll be happy to merge it.

Good point, I didn't know that it works with a colon prepended. However, I think this is still worthwhile because disclosing the LM hash allows us to reliably recover the plaintext password. I also didn't see any merge conflicts, I rebased the branch to main without issues. Should I just open a new PR or am I missing something?

Hello @rtpt-erikgeiser That's a good point. And yes, there were no merge conflicts at the time of you r PR, but I had restructured the project locally and added a bunch of features. So I could not merge and pull it without having too many conflicts. Maybe there are some smart tricks. So this is my fault. Apologies. If you'd like to recreate the PR and the new main branch, I'll be happy to merge it.
Best regards


Problem with Load_Empty_Texture

I'm trying to implement
glTexImage2D(GL_TEXTURE_2D, 0, GL_RGB, 1024, 768, 0, GL_RGB, GL_UNSIGNED_BYTE, 0);
in which GL_UNSIGNED_BYTE specifies the data type of the pixel data (GL_RGB).
Texture_2D.Load_Empty_Texture
(Level => 0, Internal_Format => GL.Pixels.RGB,
Width => 1024, Height => 768);
raised GL.ERRORS.INTERNAL_ERROR : gl-raise_exception_on_opengl_error.adb:30
gl-raise_exception_on_opengl_error.adb:30 indicates that INTERNAL_ERROR is due to an Invalid_Enum:
when Errors.Invalid_Enum => raise Errors.Internal_Error;
The pixel data type specified by API.Tex_Image_2D is Pixels.Data_Type'First which appears to me to be Byte not Unsigned_Byte (Pixels.Data_Type second).
My investigations indicate that GL_RGB can take that values 0..255 indicating that pixel data type for RGB in this context should be Unsigned_Byte.
Is this the problem?
Probably not; I tried changing  Pixels.Data_Type'First to Pixels.Unsigned_Byte in GL.Objects.Textures.With_2D_Loader.Load_Empty_Texture's call to API.Tex_Image_2D  but the same error occurred.
As usual, am I misunderstanding something?

I was going to try glTextureStorage2D (or glTexStorage2D) but it doesn't seem to be implemented.
Neither is glTextureSubImage2D (or glTexSubImage2D );
Both of these are used in a OpenGL SuperBible texture example.

Yes, glTexStorage2D is indeed missing. I will add it after migrating to new runtime loading branch.

Great to hear that the new runtime loading branch is going well, I look forward to trying it out.

I added glTexStorage2D to master. It is located right besides Load_Empty_Texture. As for your error, can you post a minimal working example?

Thanks Flyx.
As for the error, I've been off trying other tutorials but now I'm back at the error producing one  and am working on producing a minimal example.

I can reproduce the problem. However, I cannot figure out what goes wrong.
The documentation says that GL_INVALID_ENUM can be generated in two cases:

if target is one of the six cube map 2D image targets and the width and height parameters are not equal.

This is obviously not the case because you're using the Texture_2D target.

if type is not a type constant.

This also is not the case, because the type parameter is GL_BYTE.
You could help me figuring this out by writing corresponding C code and checking whether that works. Unfortunately, I currently do not have much time for looking into this.

OK. I'll try but it might take me some time.
For how much of it should I write the  corresponding C code?

It would suffice to have code until where you call glTexImage2D (and check the resulting error).

Thanks.
I think I'll try copying some C code from the SuperBible book or example code as this is what my program is "translated" from.

Attached is my attempt at building the C example requested.
I hope, as a novice, that this is OK.
What I noticed in doing this is that:
Load_Empty_Texture calls API.Tex_Image_2D with the Format parameter set to
Format : Pixels.Format = Pixels.Format'First = Color_Index => 16#1900#;
The 16#1900# DISAGREES with 0 which is used in the example code from which I based my test routine on. So, perhaps the might be the problem?
Minimal Framebuffer Object.zip

I fixed the wrong enum types; the enum for framebuffer pixel formats was used instead of an own one for data formats. However, this does not solve the issue.
You C code lacks the generation of a window with a properly versioned OpenGL context. I can only check the difference between C and Ada when the code setup is otherwise equivalent.

Here's one that generates a window and runs to completion without error.
I'm not sure what you mean by "properly versioned OpenGL context".
I currently have
Glfw.Windows.Hints.Set_Minimum_OpenGL_Version (3, 3);
in accordance with the problem described  above in
https://github.com/flyx/OpenGLAda/issues/25#issuecomment-271596964
Minimal Framebuffer Object C.zip

Here's another one in which I have parameterized Test.c:
void test (GLenum target, GLint Mipmap_Level, GLint internalformat,
GLsizei width, GLsizei height, GLenum format, GLenum type)
which calls:
glTexImage2D (target, Mipmap_Level, internalformat, width, height, 0,
format, type, NULL);
This displays a window and runs to completion without error.
Is there anything else that I need to try?
Minimal Framebuffer Object C.zip

Hum, the Ada code in your recent zips works for me after my fix. Perhaps I actually solved the issue with my commit. Can you try recompiling everything with latest master and see if the error goes away?

Thats's a bother. I still get the error.
After updating with GitHub Desktop I checked that I'm fully updated by
:/Ada_Source/OpenGLAda Roger$git pull
Already up-to-date.
I first cleaned the project then rebuilt but Run still responded with:
/OpenGL_Projects/SuperBible/Minimal Framebuffer Object/minimal_framebuffer
OpenGL Version:  3 3 0
Renderer Version: Intel HD Graphics 3000 OpenGL Engine
Set_Up, Filters set.
An exceptiom occurred in Set_Up.
raised GL.ERRORS.INTERNAL_ERROR : gl-raise_exception_on_opengl_error.adb:30

I made a clean copy and compiled it, it still works. Which operating system are you on again?

OSX 10.11.6
XCode 8.0
GNAT 6.1.0
:~ Roger$/usr/local/bin/g++ -v
Using built-in specs.
COLLECT_GCC=/usr/local/bin/g++
COLLECT_LTO_WRAPPER=/usr/local/libexec/gcc/x86_64-apple-darwin14.0.0/4.9.2/lto-wrapper
Target: x86_64-apple-darwin14.0.0
Configured with: ../gcc-4.9-20141029/configure --enable-languages=c++,fortran
Thread model: posix
gcc version 4.9.2 20141029 (prerelease) (GCC)

Here is a program that uses a thin binding (I think) of glTexImage2D and runs to completion without error on my system.
Minimal Framebuffer Object Thin.zip

I've started on a thick binding based on yours.
so far:
package pLoader is new GL.Objects.Textures.With_2D_Loader (GL.Objects.Textures.Texture_Proxy);
procedure Thin_Tex_Image_2D (Target : GL.Low_Level.Enums.Texture_Kind;
Level           : GL.Objects.Textures.Mipmap_Level;
Internal_Format : GL.Pixels.Internal_Format;
Width, Height   : GL.Types.Size;
Format          : GL.Pixels.Data_Format;
Data_Type       : GL.Pixels.Data_Type;
Data            : System.Address);
pragma Import (StdCall, Thin_Tex_Image_2D, "glTexImage2D");
--  procedure Thick_Tex_Image_2D (Target : GL.Low_Level.Enums.Texture_Kind;
procedure Thick_Tex_Image_2D (Object : pLoader.Target;
Level           : GL.Objects.Textures.Mipmap_Level;
Internal_Format : GL.Pixels.Internal_Format;
Width, Height   : GL.Types.Size) is
begin
Thin_Tex_Image_2D (GL.Objects.Textures.Texture_Proxy (Object).Kind, Level, Internal_Format, Width, Height,
GL.Pixels.Data_Format'First,
GL.Pixels.Data_Type'First, System.Null_Address);
end Thick_Tex_Image_2D;
which works.
I've yet to work out how to do the Object : Target.
All part of thee learning experience.

First success??
The attached code runs to successful completion on my system.
It seems that glTexImage2D doesn't like Data_Format'First (=> Stencil_Index).
I tracked this down using the C code:
glTexImage2D (target, Mipmap_Level, internalformat, width, height, 0, format, type, NULL);
with the aid of https://www.opengl.org/sdk/docs/man/html/glTexImage2D.xhtml to provide correct parameters according to which format = GL_STENCIL_INDEX should work but, on my system, it produces an error where as GL_RED does not.
Minimal Framebuffer Object C.zip

However, this similar program which does not involve an intermediate C interface and calls glTexImage2D via pragma Import fails with an INVALID_VALUE_ERROR.
Minimal Framebuffer Object Thin.zip

GL_STENCIL_INDEX may indeed be the culprit, since the documentation says

GL_STENCIL_INDEX may be used for format only if the GL version is 4.4 or higher.

You could try to change the value to RGB or something in gl-objects-textures-with_2d_loader.adb. If that works, please also update the corresponding *_1d_* and *_3d_* files and make a PR.

Seems to work.
PR submitted.
Attached is a program that invokes Texture_2D.Load_Empty_Texture and runs to completion without error on my system.
Minimal Framebuffer Object.zip


  Should contain user data of appropriate anonymity.
AuthenticationResource {
level: Enums.AuthenticationLevel,
message: String,
editor: Editor | PartialEditor
}
A partial editor can contain as little as an email.

No longer required.


making the size of the calculator smaller

when i open the calculator on desktop there is space below on the calculator that needs to scroll down to see, would be awesome to have the full calculator in sight when entering the webpage

when i open the calculator on desktop there is space below on the calculator that needs to scroll down to see, would be awesome to have the full calculator in sight when entering the webpage

yeah right

can you assign this to me?

?

can I get assigned to this issue please @Arman-ali-khan-786. I would love to work on it.

can I get assigned to this issue please @Arman-ali-khan-786. I would love to work on it.

sorry this issue has been resolved


Allow to make middleware in a sync way

Allow to make middleware in a sync way by manually providing required values that are fetched asyncronously: env and project name.
Async middleware complexes existing application source code and it seems like that data that is obtained asyncronously can be provided manually instead.

This is obviously a good idea as most express apps are currently sync. (The async main pattern used in the example makes error handling fragile.) It doesn't look like google are interested in providing this, but I point out that the middleware in https://github.com/googleapis/nodejs-logging-winston/blob/main/src/middleware/express.ts is simple enough that you can easily adapt it yourself.
For example, I quickly drafted this adaptation for use in App Engine (not yet fully tested):
// copy of https://github.com/googleapis/nodejs-logging-winston/blob/main/src/middleware/express.ts
// with async converted to sync

const { Log, middleware } = require('@google-cloud/logging');
const commonMiddleware = middleware

const REQUEST_LOG_SUFFIX = '_reqlog';
const LOGGING_TRACE_KEY = 'logging.googleapis.com/trace';
const LOGGING_SPAN_KEY = 'logging.googleapis.com/spanId';
const LOGGING_SAMPLED_KEY = 'logging.googleapis.com/trace_sampled';

function makeChildLogger(logger, trace, span, sampled) {
    return logger.child({
        [LOGGING_TRACE_KEY]: trace,
        [LOGGING_SPAN_KEY]: span,
        [LOGGING_SAMPLED_KEY]: sampled,
    });
}

const makeMiddleware = function (logger, projectId, logName) {
    // Unless we are running on Google App Engine or Cloud Functions, generate a
    // parent request log entry that all the request specific logs ("app logs")
    // will nest under. GAE and GCF generate the parent request logs
    // automatically.
    let emitRequestLogEntry;
    if (!process.env.GAE_APPLICATION) {
        // The request logs must have a log name distinct from the app logs
        // for log correlation to work.
        const requestLogName = Log.formatName_(
            projectId,
            `${logName}${REQUEST_LOG_SUFFIX}`
        );
        emitRequestLogEntry = (httpRequest, trace, span, sampled) => logger.info({
            logName: requestLogName,
            [LOGGING_TRACE_KEY]: trace,
            [LOGGING_SPAN_KEY]: span,
            [LOGGING_SAMPLED_KEY]: sampled,
            httpRequest,
            message: httpRequest.requestUrl || 'http request',
        })
    }

    return commonMiddleware.express.makeMiddleware(
        projectId,
        (trace, span, sampled) =>
            makeChildLogger(logger, trace, span, sampled),
        emitRequestLogEntry
    );
}

module.exports = makeMiddleware;

Assuming you already have a non middleware Cloud logger set up already you can use it like
const loggerMiddleware = require("./logger-middleware");
app.use(loggerMiddleware(logger, 'projectname', 'logname'))

@losalex would you merge the code above if I opened a PR or is it missing something? Removing the async component will make this much easier to work with.

Thank you @rileytomasek for reaching out!
I must admit that I am hesitant to make such a change since it might impact existing customers using async functionality...
Looking into a middleware usage sample, it seems that code could be slightly changed so it can be used in non-async way by leveraging then() like in example below:
  const lw = require('@google-cloud/logging-winston');
  const winston = require('winston');

  // Import express module and create an http server.
  const express = require('express');
  const logger = winston.createLogger();

  const app = express();
  lw.express.makeMiddleware(logger).then((mw: any) => app.use(mw));

Please let me know if thats works for you or comment - would really appreciate a feedback.

@bogdan / @rileytomasek  / @yaakovfeldman , closing this issue for now - please let me know if my suggestion above works for you and feel free to reopen an issue or comment if you have more suggestions


Library instrumentation guidance with DiagnosticSource

We are looking for help from the tracing community to review out library instrumentation guidance
https://github.com/Microsoft/ApplicationInsights-dotnet-server/blob/develop/LibraryInstrumentationGuidance.md
Start of the discussion:
https://github.com/StackExchange/StackExchange.Redis/issues/833
We will be happy to hear your feedback about anything, some exiting concerns:

no configuration
no contract
Tags vs payload usage
lack of detail
efficiency/performance
concrete vs anonymous types

We hope to come up with the guidance that would work well with library owners and tracing systems.
Our philosophy for tracing still has to follow some principles like

automagical with (almost) zero configuration
(almost) zero perf impact when nobody listens to, and runtime sampling and verbosity control when there is a listener
no additional dependencies, so .NET is traceable itself
ability to build rich UI on top of it and run our important customer scenarios

We'll publish the result in more official place.
/cc @cwe1ss, @NickCraver, @stebet,  @vancem
Update: I will submit the first PR later this week emphasizing concrete payload types, proposing PascalCase and would provide mode details about sampling/verbosity

With the repo migration, the new link to the document is:
https://github.com/microsoft/ApplicationInsights-dotnet/blob/develop/WEB/LibraryInstrumentationGuidance.md


Force sources attrib of FilterSources to be a tuple

AbstractDataStream requires that sources be a tuple. When sources are a list one get errors on concatenation with tuples, e.g. when a FilterSources transformer is chained with a Mapping one (there is then a list to tuple concatenation at https://github.com/mila-udem/fuel/blob/master/fuel/transformers/init.py#L107-L108 )

LGTM after square brackers are removed.

OK, done

Thanks for the fix!


Removing and replacing swagger-core with POROs

@HawaiianSpork please review?
swagger-core did not handle the full Swagger specification. We weren't really benefiting from a parser (it did some massaging of operations but otherwise was a just enabling dot notation for hash access), so this replaces a 3rd-party parser with POROs. swagger-diff should now accept 100% of valid Swagger specifications.
Adding support for:

parameter refs
shared parameters
response refs

Fixes #20
Fixes #30
Fixes #31
Closes #24

LGTM


The plugin is not working

When i start my server with Tshock, the program can't read or use the plugin

Are you using tshock 5.1.2 and the crossplay.dll

it didnt work for me either. it says the plugin is outdated

I am using tshock 5.1.2 and your crossplay plugin but it says yhe plugin is outdated

did you have the outdated plugin installed

And can anyone send the error logs I haven't been able to make it fail

Money lover has updated their cross play plugin https://github.com/Moneylover3246/Crossplay use that it should work


Feature Planning & Suggestions

Base64 encoding for channel & data

Base64 encoding for channel & data, would solve #1

encodeURIComponent might be easier to implement, since Base64 encoding / decoding isn't natively supported

URI encoding implemented in d7cf0f9


Break up main.c

Creating this as a draft PR based on the fixes/cmake_sanity_checks branch so CI can check things early.

I successfully tested all three supported command endpoints - HA, openHAB, and generic REST.
Well done!


💎  Updates for Android 14 (🙃 🍰)

Found that app-sdk needs to be updated to handle https://github.com/S-HealthStack/starter-app/issues/3

Update health connect to 1.1.0-alpha02 with relevant changes in HealthConnectAdapter, HealthConnectUtils and AndroidManifest of sample app.
Remove deprecated health data dependencies
Add forced rules to resolve class duplications with kotlin-stdlib and also update dagger version to 2.47 to prevent missing classes by forced rules
Quick fixes to remove type mismatches in AlarmUtils

Welcomes any feedbacks!

Remove deprecated health data dependencies

On my second review, I found that above is used by android health platform codes, which is deprecated.
I'd like to know about whether this project keeps to support android health platform or not.
I'll follow your decision.

Remove deprecated health data dependencies

On my second review, I found that above is used by android health platform codes, which is deprecated. I'd like to know about whether this project keeps to support android health platform or not. According to it, I will restore dependencies or resting codes of android health platform.

Well, now it's time to let them go. :)

Hello @naondol .
I apologize for the delayed review.
Before merging the features, we reflected the PR that changes the structure to import the sample app as a submodule.
Can you please reflect the PR related to the app again here: S-HealthStack/starter-app please?
We would like to deploy the SNAPSHOT by merging your PR as well.
We apologize for the inconvenience and thank you for your effort.

@jjyun-do OK. I split the original PR(https://github.com/S-HealthStack/starter-app/pull/6) according to the revised structure.


Thank you!


Junit test case addition

added CallableStatementTest and PrecisionScaleTest. Removed RandomData inside jdbc.datatypes beacuse it's redundant

Codecov Report

Merging #4 into dev will increase coverage by 0.36%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev       #4      +/-   ##
============================================
+ Coverage     45.98%   46.35%   +0.36%
+ Complexity     2203     2200       -3
============================================
  Files           108      108
  Lines         25210    25210
  Branches       4164     4164
============================================
+ Hits          11594    11686      +92
+ Misses        11697    11495     -202
- Partials       1919     2029     +110

Flag
Coverage Δ
Complexity Δ

#JDBC41
46.22% <ø> (+0.38%)
2192 <ø> (-3)
:arrow_down:

#JDBC42
46.24% <ø> (+0.42%)
2198 <ø> (+4)
:arrow_up:

Impacted Files
Coverage Δ
Complexity Δ

...om/microsoft/sqlserver/jdbc/SimpleInputStream.java
51.11% <0%> (-5.19%)
11% <0%> (-5%)

...ava/com/microsoft/sqlserver/jdbc/ScrollWindow.java
73.17% <0%> (-2.44%)
17% <0%> (-2%)

...in/java/com/microsoft/sqlserver/jdbc/IOBuffer.java
54.11% <0%> (-1.17%)
0% <0%> (ø)

...n/java/com/microsoft/sqlserver/jdbc/tdsparser.java
68.46% <0%> (-0.91%)
0% <0%> (ø)

...c/main/java/com/microsoft/sqlserver/jdbc/Util.java
59.17% <0%> (-0.87%)
88% <0%> (-3%)

...oft/sqlserver/jdbc/SQLServerResultSetMetaData.java
40.97% <0%> (-0.7%)
23% <0%> (-1%)

...m/microsoft/sqlserver/jdbc/SQLServerStatement.java
59.06% <0%> (-0.51%)
129% <0%> (-9%)

.../microsoft/sqlserver/jdbc/SQLServerConnection.java
45.71% <0%> (-0.24%)
273% <0%> (-15%)

...om/microsoft/sqlserver/jdbc/SQLServerBulkCopy.java
52.67% <0%> (-0.2%)
240% <0%> (-1%)

...m/microsoft/sqlserver/jdbc/SQLServerResultSet.java
33.38% <0%> (-0.13%)
245% <0%> (-10%)

... and 10 more

Continue to review full report at Codecov.

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

Codecov Report

Merging #4 into dev will increase coverage by 0.36%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev       #4      +/-   ##
============================================
+ Coverage     45.98%   46.35%   +0.36%
+ Complexity     2203     2200       -3
============================================
  Files           108      108
  Lines         25210    25210
  Branches       4164     4164
============================================
+ Hits          11594    11686      +92
+ Misses        11697    11495     -202
- Partials       1919     2029     +110

Flag
Coverage Δ
Complexity Δ

#JDBC41
46.22% <ø> (+0.38%)
2192 <ø> (-3)
:arrow_down:

#JDBC42
46.24% <ø> (+0.42%)
2198 <ø> (+4)
:arrow_up:

Impacted Files
Coverage Δ
Complexity Δ

...om/microsoft/sqlserver/jdbc/SimpleInputStream.java
51.11% <0%> (-5.19%)
11% <0%> (-5%)

...ava/com/microsoft/sqlserver/jdbc/ScrollWindow.java
73.17% <0%> (-2.44%)
17% <0%> (-2%)

...in/java/com/microsoft/sqlserver/jdbc/IOBuffer.java
54.11% <0%> (-1.17%)
0% <0%> (ø)

...n/java/com/microsoft/sqlserver/jdbc/tdsparser.java
68.46% <0%> (-0.91%)
0% <0%> (ø)

...c/main/java/com/microsoft/sqlserver/jdbc/Util.java
59.17% <0%> (-0.87%)
88% <0%> (-3%)

...oft/sqlserver/jdbc/SQLServerResultSetMetaData.java
40.97% <0%> (-0.7%)
23% <0%> (-1%)

...m/microsoft/sqlserver/jdbc/SQLServerStatement.java
59.06% <0%> (-0.51%)
129% <0%> (-9%)

.../microsoft/sqlserver/jdbc/SQLServerConnection.java
45.71% <0%> (-0.24%)
273% <0%> (-15%)

...om/microsoft/sqlserver/jdbc/SQLServerBulkCopy.java
52.67% <0%> (-0.2%)
240% <0%> (-1%)

...m/microsoft/sqlserver/jdbc/SQLServerResultSet.java
33.38% <0%> (-0.13%)
245% <0%> (-10%)

... and 10 more

Continue to review full report at Codecov.

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


--other=merge prepare_env.sh into dataproxy-start.sh，as `sh prepare_e…

[INLONG-2126][DataProxy] prepare_env.sh can be merged into dataproxy-start.sh，as a InLong beginner maybe forgot this step
where XYZ should be replaced by the actual issue number.
Fixes #2126
Motivation
Explain here the context, and why you're making that change. What is the problem you're trying to solve.
Modifications
Describe the modifications you've done.
Verifying this change

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

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

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

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
If a feature is not applicable for documentation, explain why?
If a feature is not documented yet in this PR, please create a followup issue for adding the documentation

Codecov Report

Merging #2127 (a3ec825) into master (c5c6db4) will increase coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #2127      +/-   ##
============================================
+ Coverage     12.26%   12.28%   +0.01%
- Complexity     1155     1157       +2
============================================
  Files           413      413
  Lines         35215    35215
  Branches       5542     5542
============================================
+ Hits           4320     4325       +5
+ Misses        30128    30122       -6
- Partials        767      768       +1

Impacted Files
Coverage Δ

.../producer/qltystats/DefaultBrokerRcvQltyStats.java
45.31% <0.00%> (+1.17%)
:arrow_up:

.../client/producer/qltystats/BrokerStatsItemSet.java
72.09% <0.00%> (+4.65%)
:arrow_up:

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 c5c6db4...a3ec825. Read the comment docs.


Add section on threading restrictions

Currently Awa doesn't support multiple threads. In particular, a session should not be shared between two or more threads. This needs to be explained in the documentation.

Affects 0.1.9.


Container Tests

I know we talked about this in passing in other PR's but I'm keen to start writing tests for my container components.
Not sure where to start, but mocking out redux might be a good place. (?)
Once we have some general direction I can create a template for the ever growing list of generators.
Advice appreciated.

Nice!
You can test a lot like you do with existing components... the redux portion.. well.  You can test that it's wrapped in redux.  But you won't be able to access the wrapped component without setting the flag in redux connect.  Interesting problem.

Yep. I started to implement component-like tests but once I threw in
import { connect } from react-redux things fell apart spectacularly.
I've been doing some reading and theoretically you should be able to mount the component using enzyme and test through there, but that doesn't seem to work either.
I think it needs some more thinking on and reading - its late here so parking until... 🌤

So have been looking at this a bit further.
Exercising the google-fu has given me plenty of reading, but I found this the best:
http://redux.js.org/docs/recipes/WritingTests.html#connected-components
There are two types of test, one to test the rendering of the component, and the other to test with redux connected (you pass a custom store in that type of test). I'm just looking at the first.
The docs suggest exporting the class as well as the default, which I have done for the test under question, which is:
import React from 'react'
import { SelectChildren } from '../../App/Containers/SelectChildren'
import { shallow } from 'enzyme'

// Mock out libs here, just to keep things in one file.
jest.mock('ramda', () => {})
jest.mock('ramdasauce', () => {})
jest.mock('reduxsauce', () => {})

const wrapper = shallow(<SelectChildren />)

xit('component exists', () => {
  expect(wrapper.length).toBe(1) // exists
})

I get this sad panda:

I tried to mock the createActions out (several ways), and no dice.
Suggestions welcome! 😖

Well mocking out createActions isn't the issue.  It's whatever createActions is supposed to be a property of.  So some other import seems to be null.  Def a difficult test.  I'll def take a shot at it, but not today.  Today I hope to get back around to enabling maps 🗺

Hmm that's reduxsauce I think. Happy to wait for a push in the right direction. ✅

https://blog.hellojs.org/testing-react-native-and-redux-e5a71b99e178#.v91z4gp4k

Closing this issue. Turns out the mocking wasn't the problem - the mocks to ramda et al were not needed. I had a call to R.map (ie ramdas version of map), jest confused it with ES6's new map function, and things exploded as documented above.
I replaced the ramda map function with the ES6 map function (eg object.map) and all fixed.
Thinking about how I'll submit a container test for v2.0 of ignite.


[BUG]: Error 24003 on Android 12

Version
6.5.0
What platforms are you having the problem on?
Android
System Version
Android 12
On what device are you experiencing the issue?
Real device
Architecture
Old architecture
What happened?
LOG  video error: {
  "error": {
    "errorCode": "24003",
    "errorException": "androidx.media3.exoplayer.ExoPlaybackException: MediaCodecVideoRenderer error, index=0, format=Format(1, null, null, video/avc, avc1.4D4029, -1, null, [1920, 1080, 24.999998, ColorInfo(BT709, Limited range, SDR SMPTE 170M, false, 8bit Luma, 8bit Chroma)], [-1, -1]), format_supported=YES",
    "errorStackTrace": "androidx.media3.exoplayer.ExoPlaybackException: MediaCodecVideoRenderer error, index=0, format=Format(1, null, null, video/avc, avc1.4D4029, -1, null, [1920, 1080, 24.999998, ColorInfo(BT709, Limited range, SDR SMPTE 170M, false, 8bit Luma, 8bit Chroma)], [-1, -1]), format_supported=YES
    at androidx.media3.exoplayer.ExoPlayerImplInternal.handleMessage(ExoPlayerImplInternal.java:623)
    at android.os.Handler.dispatchMessage(Handler.java:102)
    at android.os.Looper.loopOnce(Looper.java:201)
    at android.os.Looper.loop(Looper.java:288)
    at android.os.HandlerThread.run(HandlerThread.java:67)
    Caused by: androidx.media3.exoplayer.video.MediaCodecVideoDecoderException: Decoder failed: c2.rk.avc.decoder
    at androidx.media3.exoplayer.video.MediaCodecVideoRenderer.createDecoderException(MediaCodecVideoRenderer.java:1915)
    at androidx.media3.exoplayer.mediacodec.MediaCodecRenderer.render(MediaCodecRenderer.java:848)
    at androidx.media3.exoplayer.video.MediaCodecVideoRenderer.render(MediaCodecVideoRenderer.java:940)
    at androidx.media3.exoplayer.ExoPlayerImplInternal.doSomeWork(ExoPlayerImplInternal.java:1112)
    at androidx.media3.exoplayer.ExoPlayerImplInternal.handleMessage(ExoPlayerImplInternal.java:544)
    ... 4 more
    Caused by: java.lang.IllegalStateException
    at android.media.MediaCodec.native_queueInputBuffer(Native Method)
    at android.media.MediaCodec.queueInputBuffer(MediaCodec.java:2769)
    at androidx.media3.exoplayer.mediacodec.AsynchronousMediaCodecBufferEnqueuer.doQueueInputBuffer(AsynchronousMediaCodecBufferEnqueuer.java:220)
    at androidx.media3.exoplayer.mediacodec.AsynchronousMediaCodecBufferEnqueuer.doHandleMessage(AsynchronousMediaCodecBufferEnqueuer.java:189)
    at androidx.media3.exoplayer.mediacodec.AsynchronousMediaCodecBufferEnqueuer.access$000(AsynchronousMediaCodecBufferEnqueuer.java:45)
    at androidx.media3.exoplayer.mediacodec.AsynchronousMediaCodecBufferEnqueuer$1.handleMessage(AsynchronousMediaCodecBufferEnqueuer.java:91)
    at android.os.Handler.dispatchMessage(Handler.java:106)
    ... 3 more
    ",
    "errorString": "ExoPlaybackException: ERROR_CODE_DECODING_FAILED"
  }
}

Expected behaviour
Consistently play MP4s after rendering a new screen without issue.
Reproduction Link
https://github.com/m-cartmell/react-native-video-test
Reproduction
With the minimal repo, reload the application or tap next until an error is reported. The erroring is inconsistent.

same problem here!

Looks like a DECODER issue, It means that playing the 2 videos in the same time causes issues on your device.
I try to reproduce with the emulator, I have no issue.
You should try to stop (set source to undefined) or pause the video when the screen is not visible with useIsFocused

Looks like a DECODER issue, It means that playing the 2 videos in the same time causes issues on your device. I try to reproduce with the emulator, I have no issue. You should try to stop (set source to undefined) or pause the video when the screen is not visible with useIsFocused

If you look at the minimal repo I supplied, I am using the useIsFocused hook to unmount the previous video. I am experiencing the issue on a real device, not an emulator.

unfortunatelly, I don't reproduce on my device ... Did you try to add a delay before starting the video ?
I am afraid this is a device specific issue while there are 2 videos in the same time (unmount <Video is not an instant action) A delay can improve that.
You can also try to change the viewType

maybe it's also be related to this ticket ? https://github.com/TheWidlarzGroup/react-native-video/issues/3491

unfortunatelly, I don't reproduce on my device ... Did you try to add a delay before starting the video ? I am afraid this is a device specific issue while there are 2 videos in the same time (unmount <Video is not an instant action) A delay can improve that.
You can also try to change the viewType

I will explore trying to dispose of the instance sooner. As it's triggering an error for me sometimes on initial render or

unfortunatelly, I don't reproduce on my device ... Did you try to add a delay before starting the video ? I am afraid this is a device specific issue while there are 2 videos in the same time (unmount <Video is not an instant action) A delay can improve that.
You can also try to change the viewType

Thank you for your advice. Setting the video source to null when navigating away from a screen and using useIsFocused and a timeout to select the source when on a screen seems to have resolved the issue. I am closing the report.


refactor(time): enhanced type definitions for MockTimeSource#diagram

diagram method has now three signatures

the first with one argument, returning Stream<number | string>
the second with two arguments, with the latest an Array<T> of values, returning
Stream<T>
the third with two arguments, with the latest an ObjectDictionary<T> of values,
returning Stream<T>

I didn't add tests since this addition aims to reinforce an existing contract.

[x] I ran make test FOO for the package FOO I'm modifying
[x] I used make commit instead of git commit

Hey @sveinburne,
Thanks for this contribution! And for moving your pull request over.
Sorry for my delay in getting back to you.
Would you be able to take a look at the CI failure and see if you can determine the cause?

Will merge this after the fix for #672 lands.

Hi @Widdershin, sorry for my late answer ! Was in holiday. Will take a look this week-end,
cheers

@sveinburne no worries! 😄 @jvanbruegge and I are still sorting out our approach to #672 so we wouldn't have been able to merge this in any case.

All right ! That is why I passed the tests locally but not on CI server I guess !


Configurable plus/minus sign

Could plus/minus sign be configurable?
What if I want to use other element? Carret, for example.

Take a look at issue #31


youtube-viewer does not pick up options from (mpv.conf)?

Hello ,
i have an mpv.conf that i would like to use
to limit videos in 720p and to upconvert them with help of svp4linux to 60/75fps videos.
it seems that youtube-viewer does not pick up the mpv.conf in .config/mpv/mpv.conf
the inserted options are
--hwdec=auto-copy --input-ipc-server=/tmp/mpvsocket  ytdl-format="[height<720]/best"
do i need to change something in the youtube-viewer.conf?
thank you very much!

Hi,
By default, youtube-viewer extracts the YouTube URLs on its own, the youtube-dl options from mpv.conf do not apply.
To use mpv + youtube-dl to play the videos, set in the configuration file (inside the video_players section):
 mpv    => {
        arg => "*URL*",
        cmd => "/usr/bin/mpv",
        fs => "--fullscreen",
        novideo => "--no-video",
        srt => "--sub-file=*SUB*",
      },


referencing errors

I copied the project solution code for the todo app into project folder for VS2015 which has the cordova tools installed etc ... the project builds without errors but there are runtime errors (running in the default ripple emulator ) ...
scripts/frameworks/angular.min.js  .... but the frameworks folder is not in the project???
also the config.xml was not found even though it is in the project

What do you mean with copying the todo app into the project folder? Can you try  to open solution (.sln) file with VS? It will load the appropriate VS Project.


Port Postgres database to new data model

Reimplements the postgres interface for the new data model (#77)

lgtm, but just had a quick look through. fine to merge if it passes all related tests

The tests need to be modified in a few places, and the in-memory implementation still needs updating if we want to maintain that. I don't think it'll be too difficult to do


GitHub's page building failing

Due to my previously merged fork, GitHub fails to build the page (the ":" in the title is a special character and should be quoted)... Should be fixed with this pull request.

Merged, thank you LeChat!


Problem with jQuery

I'm using jQuery 1.9.1 with bower and when I compile with amdclean, I'm getting undefined error: "jquery" (note: lowercase).
If I tweak my build script to find and replace "jquery" -> "jQuery" everything works fine.
This is my build config:
({
  baseUrl: '../src',

  out: '../lego.min.js',
  // optimize: 'uglify2',

  include: [ 'namespace' ],
  // Remove license comments for almond, jquery, etc
  preserveLicenseComments: false,

  paths: {
    'jquery': '../bower_components/jquery/jquery'
  },

  onBuildWrite: function (moduleName, path, contents) {
    return module.require('amdclean').clean(contents);
  }
})

Hi, I am trying to build libraries that contain third-party dependencies using the hack described by @gfranko above in one of the comments,  I am a little confused over to where to Set the third-party dependency local variable values inside  source code. Has anybody implemented this way?


Fix integer signedness warning in metadata.cc.

@nicolasnoble

Looks like someone else had already sent in this change in #25027, which I just merged, so this PR is no longer needed.

Looks like someone else had already sent in this change in #25027, which I just merged, so this PR is no longer needed.


Missing tag for 0.1.7 and Cocoapods release

You have changed the version in Readme to 0.1.7, but have not created tag (release) and have not pushed the release to Cocoapods trunk.

Yes sure. We are working in some small improvement to our lib, I think today it will be done.
Before this please use 0.1.6 version. Sorry for misunderstanding.

Got it, thanks.
Also we are having issues with BigInt dependency version (the latest version is 5, but you are using version 3). I have created an issue to EthereumABI repo for updating its BigInt dependency:
https://github.com/shamatar/EthereumABI/issues/2


Add new modules (multi stft loss, vocoder and random resampling)

internalization of Multi band melgan (https://github.com/AppleHolic/multiband_melgan)

[ ] vocoder encode(wav to mel) decode(mel to wav)
[x] multi stft loss

[ ] Vocoder denoiser (from NVIDIA waveglow github)

#28 merged multi-stft loss. Handle vocoder functions on another issue


How do I download multiple files in the background using AWSS3TransferUtilityDownload?

It is not obvious from the sample or the source code that each download task will be assigned a single identifier.

You can call the download method for each file you want to download. Each task has a taskIdentifier per API Reference.

Looks like the taskIdentifier is automatically generated to be unique. In short, just download them without any special handling besides the standard background downloading and whatever is required by the app. Thanks.


ExportDocuments Not Exporting All Documents

ExportDocuments doesn't export all documents 100% of the time.
    var cards = await typesenseClient.ExportDocuments<DbVariant>("cards").ConfigureAwait(false);
    logger.LogInformation($"Cards: {cards.Count}");

Expecting 7326 records. Periodically this returns only around 300 records. Sometimes all 7326 are returned. Seems to be completely random.

@runeanielsen turns out this was an issue where data was not being duplicated to one of the nodes in typesense cloud high availability setup. The export was sometimes exporting from a node that didn't have all data for some reason. Closing this.


Add a job to build the image when a PR is created

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


feat: add E2B code interpreter

This is a plug-and-play alternative of the Docker code interpreter that uses E2B cloud sandboxes.
The inputs and outputs are slightly different compared to the default code interpreter:
Input: Dependencies are specified in the code via "!pip install"
Output: Output is a JSON structure separating standard output, errors, and rich data.
However these don't necessarily affect the end user as they are specified for the agent in the schema.
It also requires an E2B API key.

Example agent: https://gist.github.com/jamesmurdza/956021e18c9576560c01b25f3e9d5b6b

Hi @lorenzejay, I've just improved the text and added comments to the README to explain why we have a close() function. I also added arguments to the tool, including timeout.

Hi @lorenzejay, I've just improved the text and added comments to the README to explain why we have a close() function. I also added arguments to the tool, including timeout.

Awesome. Again, should it just automatically use the close method once it finishes the response? Don't need to keep the process running when its done right. Instead of having users set close manually. not saying remove the method just include that or even include that as a flag users can set ie: close_when_finished: bool

Hi @lorenzejay, I've just improved the text and added comments to the README to explain why we have a close() function. I also added arguments to the tool, including timeout.

Awesome. Again, should it just automatically use the close method once it finishes the response? Don't need to keep the process running when its done right. Instead of having users set close manually. not saying remove the method just include that or even include that as a flag users can set ie: close_when_finished: bool

@lorenzejay If I understand correctly that would close the sandbox after the first time the tool is called. Since we don't know how many times the tool will be called (and it's frequently used more than once) that would cause workflows to break.

@jamesmurdza  getting close please add E2BCodeInterpreterTool in init.py to properly import it from crew_tools

@jamesmurdza when running the tool on my crew i get this:
File ".../pypoetry/virtualenvs/crewai-Nmm_nbE4-py3.11/lib/python3.11/site-packages/e2b_code_interpreter/main.py", line 40, in init
super().init(
TypeError: Sandbox.init() got an unexpected keyword argument 'envs'


Re-investigate OSGi-based service binding

A lot of these specs have always or now have hooks for loading implementations via the HK2 ServiceLoader bridge, and using that would potentially save some hassle with all the impl-binding fragment bundles. However, in the past, I haven't had luck using that bridge on Domino. Perhaps it would work better when the bundles are pre-prepared for it and don't have to be woven at runtime.

This is more-or-less done now, with a specialized implementation of the ServiceLoader bridge, and each spec that needs special attention gets it.


Deploy And Usage on shared hosting with SSH Access

i can access my shared hosting with ssh but i have this error after running php artisan websockets:serve

WebSocket connection to 'ws://test.codeixi.com:6001/app/websocketkey?protocol=7&client=js&version=4.3.1&flash=false' failed: WebSocket is closed before the connection is established.

app.js:37780 WebSocket connection to 'wss://test.codeixi.com/app/websocketkey?protocol=7&client=js&version=4.3.1&flash=false' failed: Error in connection establishment: net::ERR_CERT_AUTHORITY_INVALID

Are you sure that you can open ports on shared hosting?

@masterix21 is right, normally you can't port forward on shared hosting


libretro.pcsx-rearmed: unstable-2024-02-07 -> unstable-2024-02-14

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for libretro.pcsx-rearmed is: Port of PCSX ReARMed with GNU lightning to libretro
meta.homepage for libretro.pcsx-rearmed is: https://www.libretro.com/
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found unstable-2024-02-14 in filename of file in /nix/store/ldn0sly1q66wzfqmwm0aqfxihzilz9hj-libretro-pcsx_rearmed-unstable-2024-02-14

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath
libretro.pcsx-rearmed
retroarchFull

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/ldn0sly1q66wzfqmwm0aqfxihzilz9hj-libretro-pcsx_rearmed-unstable-2024-02-14 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A libretro.pcsx-rearmed https://github.com/r-ryantm/nixpkgs/archive/440a0b533bc2dda234c8ba6120aa3546d1e1c6f8.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/440a0b533bc2dda234c8ba6120aa3546d1e1c6f8#libretro.pcsx-rearmed

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/ldn0sly1q66wzfqmwm0aqfxihzilz9hj-libretro-pcsx_rearmed-unstable-2024-02-14
ls -la /nix/store/ldn0sly1q66wzfqmwm0aqfxihzilz9hj-libretro-pcsx_rearmed-unstable-2024-02-14/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

libretro.pcsx-rearmed
retroarchFull

Maintainer pings
cc @aanderse @edwtjo @MP2E @thiagokokada @hrdinka for testing.

@NixOS/nixpkgs-merge-bot merge


Extract value from JSON

I would like to move the extract_value part from the REST sensor to util.

I don't want to add this because processing values from platforms like MQTT, rest, aRest will move to use the new template support from #720.
The config could look something like this:
sensor:
  platform: rest
  resource: http://example.com/api/bla
  value_template: '{{ value.some_attribute }}'

Your PR #720 makes this one obsolete.