fix: rename argument parameters

fixes https://github.com/adoptium/ci-jenkins-pipelines/issues/433
Signed-off-by: sakshi1215<EMAIL_ADDRESS>

Hi @zdtsw can you please review this?

Hi @karianna @zdtsw can you please take a look again?

Ping @AdamBrousseau to double check if these two parameters  need to be updated for your Jenkins job

Is there a manual update needed to job(s) or will they automatically regenerate with the updated names?

Is there a manual update needed to job(s) or will they automatically regenerate with the updated names?
the job should be re-generated accordingly with new parameter's name.
but if you have anything else calls such job, e.g remote trigger with parameters, that need to be updated manually.

@karianna @zdtsw @AdamBrousseau Is this good to be merged?

Can I tru that change too? Which repo stored the buildConfig?
On Wed, 12 Oct 2022 at 10:19 PM, Wen Zhou @.***> wrote:

@.**** commented on this pull request.
my bad, should have consider this change would require the change in the
class IndividualBuildConfig, which is not in the scope of this git repo.
—
Reply to this email directly, view it on GitHub
https://github.com/adoptium/ci-jenkins-pipelines/pull/440#pullrequestreview-1139429537,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANF3CEDX2LQ2Y5ZWAD4MC4LWC3TYNANCNFSM6AAAAAARBIOUII
.
You are receiving this because you authored the thread.Message ID:
@.***>

@sakshi1215 It's in the jenkins-helper repo: https://github.com/adoptium/jenkins-helper/blob/9fd063790c64a7cf2f9ee38c4bf4655f430d8be6/src/common/IndividualBuildConfig.groovy

Actually we'd need to change this as well: https://github.com/adoptium/jenkins-helper/blob/9fd063790c64a7cf2f9ee38c4bf4655f430d8be6/src/common/IndividualBuildConfig.groovy#L58

Its already done @andrew-m-leonard https://github.com/adoptium/jenkins-helper/pull/43

Actually we'd need to change this as well: https://github.com/adoptium/jenkins-helper/blob/9fd063790c64a7cf2f9ee38c4bf4655f430d8be6/src/common/IndividualBuildConfig.groovy#L58

Its already done @andrew-m-leonard adoptium/jenkins-helper#43

excellent

Will merge with the jenkins helper fix post release.


Unable to verify: Unable to reach http://amankapoor.me/.well-known/acme-challenge/

Hey,
Below are the logs
Debugging tips: -v improves output verbosity. Help is available under --help.
Sleep for 3600s
2017/11/05 20:21:24 Received event die for container 2f82ed7595b0
2017/11/05 20:21:24 Received event stop for container 2f82ed7595b0
2017/11/05 20:21:39 Debounce minTimer fired
2017/11/05 20:21:39 Generated '/app/letsencrypt_service_data' from 1 containers
2017/11/05 20:21:39 Running '/app/update_certs'
Sleep for 3600s
2017/11/05 20:27:20 Received event start for container c673b7addb40
2017/11/05 20:27:20 Received event die for container c673b7addb40
2017/11/05 20:27:20 Received event start for container 569e379d1e58
2017/11/05 20:27:21 Received event die for container 569e379d1e58
2017/11/05 20:27:22 Received event start for container 7039252acf68
2017/11/05 20:27:37 Debounce minTimer fired
2017/11/05 20:27:37 Generated '/app/letsencrypt_service_data' from 2 containers
2017/11/05 20:27:37 Running '/app/update_certs'
Creating/renewal amankapoor.me certificates... (amankapoor.me)
2017-11-05 20:27:37,525:INFO:simp_le:1213: Generating new account key
2017-11-05 20:27:42,602:ERROR:acme.challenges:311: Unable to reach http://amankapoor.me/.well-known/acme-challenge/AuWKk8DDRFgqqKyL8EdrarI5nJo074uEZ_WFPT1J5AQ: HTTPConnectionPool(host='amankapoor.me', port=80): Max retries exceeded with url: /.well-known/acme-challenge/AuWKk8DDRFgqqKyL8EdrarI5nJo074uEZ_WFPT1J5AQ (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7f38bed38090>: Failed to establish a new connection: [Errno 111] Connection refused',))
2017-11-05 20:27:42,602:WARNING:simp_le:1304: amankapoor.me was not successfully self-verified. CA is likely to fail as well!
2017-11-05 20:27:42,913:INFO:simp_le:1314: Generating new certificate private key
2017-11-05 20:27:44,512:ERROR:simp_le:1272: CA marked some of the authorizations as invalid, which likely means it could not access http://example.com/.well-known/acme-challenge/X. Did you set correct path in -d example.com:path or --default_root? Is there a warning log entry about unsuccessful self-verification? Are all your domains accessible from the internet? Failing authorizations: https://acme-v01.api.letsencrypt.org/acme/authz/3lSkp7D0A8tgpYIx64-BAfuUlnc2S8D9sQ8-7FsWpP8
Challenge validation has failed, see error log.

My golang application is very basic, see this
package main

import (
	"net/http"
)

func main() {
	r := http.NewServeMux()

	r.HandleFunc("/", func(w http.ResponseWriter, r *http.Request) {
		w.Write([]byte("Hello Aman"))
	})

	http.ListenAndServe(":8080", r)
}

The associated Dockerfile is:
FROM golang:onbuild
LABEL Name=ws Version=0.0.1
EXPOSE 8080

And the docker-compose.yml, looks like following:
version: '3'

services:
  ws:
    container_name: ws
    image: ws
    build: .
    environment:
      VIRTUAL_HOST: amankapoor.me
      LETSENCRYPT_HOST: amankapoor.me
      LETSENCRYPT_EMAIL<EMAIL_ADDRESS>      ACME_CA_URI: https://acme-staging.api.letsencrypt.org/directory
    expose:
      - "80"
      - "443"

networks:
  default:
    external:
      name: nginx-proxy

I have followed this link https://medium.com/@francoisromain/host-multiple-websites-with-https-inside-docker-containers-on-a-single-server-18467484ab95 as setup guide.
Where am I missing? What is the problem and what do I do to make it work?
Also, if you are familiar with Go, then could you please tell me if I need to use http.ListenAndServeTLS?
And, the ACME_CA_URI env var is for testing purpose right? So, when I need to push it for production, I will remove this env var.
Also FYI, I use AWS and Cloudflare, so I have put an A record of my site above with the public IP provided by AWS, in the DNS settings of Cloudflare.

Additionally, this is how docker ps looks,
/srv/www/vhost.d$ docker ps
CONTAINER ID        IMAGE                                    COMMAND                  CREATED             STATUS                             PORTS                                      NAMES
7039252acf68        ws                                       "go-wrapper run"         28 minutes ago      Up 28 minutes                      80/tcp, 443/tcp, 8080/tcp                  ws
9786bed0572b        jwilder/docker-gen                       "/usr/local/bin/do..."   2 hours ago         Restarting (1) About an hour ago                                              nginx-gen
e957ddb6c52f        jrcs/letsencrypt-nginx-proxy-companion   "/bin/bash /app/en..."   2 hours ago         Up 2 hours                                                                    nginx-letsencrypt
dce85311d0b6        nginx                                    "nginx -g 'daemon ..."   2 hours ago         Up 2 hours                         <IP_ADDRESS>:80->80/tcp, <IP_ADDRESS>:443->443/tcp   nginx

ws is the simple go web app. And docker-gen remains closed, I don't know why, but it has been like this since the creation of these containers. And it restarted once, probably when ws container was created or doing something related to certificates.

If docker-gen keep restarting something is misconfigured and the stack won't work. Most probable cause is you did not grab the nginx.tmpl template file from jwilder/nginx-proxy.
Try the two containers setup with a compose file version 2 before going for the unnecessary complex (for a first try) three container setup on compose file version 3. If you need a starting point you can grab this one and replace duch with jrcs.
I know this repo's doc says the three container setup is recommended but truth is it's not.
Also your golang application container expose only port 8080 (on which something is listening) but your docker-compose file also expose ports 80 and 433 on which nothing is listening. When a proxyed container has multiple ports exposed including port 80, nginx-proxy will default to port 80.
Remove this expose: directive from your app compose file or you app won't work even if the cert get generated.

Oh and right now your domain appears to resolve to cloudflare, I have no idea if the le-companion container will work behind cloudflare.

Hey, thank you for your comments. I followed what you wrote above and I was able to generate a certificate for my site. But the problem is that I can't get Cloudflare to work with it when using it as DNS+HTTP Proxy. If I only set it as DNS, then it works fine.
Where can I learn more to make it work in sync with Cloudflare? Do we have any settings available to make any changes required for Cloudflare? Or, any trusted GitHub repo of the same? From what I have researched till now, there are ways to make it work with CF when using http-01 and dns-01 as challenges.

Strange observation:
You already know I was saying that my site doesn't run when with CF. But now if you see the ssl certificate on my site, you will find that its from Cloudflare. And how I got it working? My observation is, when I first got the LE certificate, Cloudflare DNS+HTTP Proxy (Orange Cloud) was on. As far as I remember, I did not change it and I got a certificate using your methods above. Then I observed that turning the cloud to grey (DNS only), resolved to my IP, showed LE cert and I could see the content. What I did just now, changed the grey cloud back to orange and my site's content is accessible and cert is of Cloudflare (COMODO). The SSL setting which I have is Full (strict) which mean, ssl between my origin (aws) server and cloudflare and between cloudflare and client is fully https, which is true. So, this way I got it working. The steps will be fully clear to me as I will be deploying more sites in the coming days.
Btw, before closing this issue, I wish to have clarity on two more things:

I want certificates for all the subdomains that I will ever create in future, so this time I did LETSENCRYPT_HOST=amankapoor.me only, but should I be doing like this -> LETSENCRYPT_HOST=amankapoor.me, *.amankapoor.me

And, regarding test certificates, this time I did not add LETSENCRYPT_TEST=true, but only ACME_CA_URI=https://acme-staging.api.letsencrypt.org/directory. So which type of certificate do I have now? A test certificate?

Let's Encrypt does not support wildcard certificates yet : https://letsencrypt.org/2017/07/06/wildcard-certificates-coming-jan-2018.html

Yes, you'll get a test certificate too. What LETSENCRYPT_TEST=true does behind the scene is telling simp_le to use the staging api endpoint.


Don't store the agda path in the config

When switching between projects that use different versions of Agda, I have to clear the agda path and restart vscode every time, which is annoying.
If there is a use case for saving the path in the config, we could expose this as a setting. Or maybe only save when the path is not empty.
I couldn't build rescript on NixOS so I didn't regenerate the javascript.

My current workaround for this is to set the Agda path to a shell script that just does exec agda "$@".

Thanks!

Don't forget to rebuild the JS!


[BUG] build error

This is my first time using this library, and when i build it it threw an error
Im using termux on android 11
Logs :

Its fixed now


Event priority for example pointertap event before mousedown

Hi
I need to handle mouse click and mousedown/mousemove/mouseup to move Sprite from A to B (user can do it by drag and drop or click Sprite to select it and then click on location B to move)
but problem is that click event fire after mousedown and mouseup.
How to handle this issue?

On mousedown -> save mouse down position, start listening for mouse up & mouse move
On mousemove -> if mouse has moved from the starting position more than a specified threshold, activate dragging state, and mouseup will end the drag.
On mouseup -> if mouse has not moved more than the threshold yet and thus isn't dragging, activate sticky-click. Otherwise, complete your drag.


coming soon added to readme.md

I have made some changes in your readme file please consider them in your main project
I have aded the coming soon section to the readme

Please consider testing branch first.


Import @react-mf/api

I don't understand how are you compiling the application. You are importing @react-mf/api but it is not declared anywhere. I'm having some problems compiling my application.

Anything starting with @react-mf means it's a module that's resolved at runtime, in the browser and not during compile time. See https://single-spa.js.org/docs/recommended-setup#import-maps

OK cool, but I've been looking at the webpack config of the planet's and people's repo and there's nothing that let the compiler know that the api module should be imported at runtime.

They’re not connected at compile time, so there’s nothing for the compiler to know about. Everything happens at runtime
A comparison would be: do native ES modules know about each other at compile time? No. They just export things that other modules can import.

I saw you import people, navbar , planets, styleguid, root-config here
https://github.com/react-microfrontends/root-config/blob/master/src/index.ejs
but when and where you import api ? I know it happened at runtime, but at least it should imported api ? dont know what I'm missing.  @frehner

OK cool, but I've been looking at the webpack config of the planet's and people's repo and there's nothing that let the compiler know that the api module should be imported at runtime. I dunno what I'm missing

Hi there, do you understand what happened?

https://github.com/react-microfrontends/root-config/blob/master/src/index.ejs#L9
find the reason here.

@react-mf/api is an "in-browser module." Import maps determine the URL that the browser uses to download the module from. Webpack externals instruct webpack to omit @react-mf/api from the bundle so that it can be provided as an in-browser module.
Here are documentation links, which include links to tutorial videos:

https://single-spa.js.org/docs/recommended-setup#in-browser-versus-build-time-modules
https://single-spa.js.org/docs/recommended-setup#import-maps
https://webpack.js.org/configuration/externals/#root


[boost-core] Build error on x64-windows-static

Package<EMAIL_ADDRESS>Host Environment

Host: x64-windows
Compiler: MSVC 19.39.33523.0
vcpkg-tool version: 2024-04-23-d6945642ee5c3076addd1a42c331bbf4cfc97457
vcpkg-scripts version: dee924de7 2024-05-27 (8 hours ago)

To Reproduce
vcpkg install boost
Failure logs
-- Downloading https://github.com/boostorg/core/archive/boost-1.85.0.tar.gz -> boostorg-core-boost-1.85.0.tar.gz...
[DEBUG] To include the environment variables in debug output, pass --debug-env
[DEBUG] Trying to load bundleconfig from C:\vcpkg\vcpkg-bundle.json
[DEBUG] Failed to open: C:\vcpkg\vcpkg-bundle.json
[DEBUG] Bundle config: readonly=false, usegitregistry=false, embeddedsha=nullopt, deployment=Git, vsversion=nullopt
[DEBUG] Metrics enabled.
[DEBUG] Feature flag 'binarycaching' unset
[DEBUG] Feature flag 'compilertracking' unset
[DEBUG] Feature flag 'registries' unset
[DEBUG] Feature flag 'versions' unset
[DEBUG] Feature flag 'dependencygraph' unset
Downloading https://github.com/boostorg/core/archive/boost-1.85.0.tar.gz
[DEBUG] Trying to hash C:\vcpkg\downloads\boostorg-core-boost-1.85.0.tar.gz.18744.part
[DEBUG] C:\vcpkg\downloads\boostorg-core-boost-1.85.0.tar.gz.18744.part has hash a1e774c98b537dd42018742af68f5889af1afd4e6c9809399987124e56a3b9ae2f20e8173170bc6fc0ad76fb95e2da7b9d385a4037e9ee12011fc02bf8f7e009
error: Failed to download from mirror set
error: File does not have the expected hash:
url: https://github.com/boostorg/core/archive/boost-1.85.0.tar.gz
File: C:\vcpkg\downloads\boostorg-core-boost-1.85.0.tar.gz.18744.part
Expected hash: 3a7be75e52f5c20830fccb9e7391a1e4556ebb072e6324df95b1ba38bed46e24f4c9f27a62a1099eddc90f2ac1ede083f0c850e2dc27fd42375d028516f675b3
Actual hash: a1e774c98b537dd42018742af68f5889af1afd4e6c9809399987124e56a3b9ae2f20e8173170bc6fc0ad76fb95e2da7b9d385a4037e9ee12011fc02bf8f7e009
[DEBUG] D:\a\_work\1\s\src\vcpkg\base\downloads.cpp(997):
[DEBUG] Time in subprocesses: 0us
[DEBUG] Time in parsing JSON: 6us
[DEBUG] Time in JSON reader: 0us
[DEBUG] Time in filesystem: 1022us
[DEBUG] Time in loading ports: 0us
[DEBUG] Exiting after 425 ms (404912us)

CMake Error at scripts/cmake/vcpkg_download_distfile.cmake:32 (message):

      Failed to download file with error: 1
      If you are using a proxy, please check your proxy setting. Possible causes are:

      1. You are actually using an HTTP proxy, but setting HTTPS_PROXY variable
         to `https://address:port`. This is not correct, because `https://` prefix
         claims the proxy is an HTTPS proxy, while your proxy (v2ray, shadowsocksr
         , etc..) is an HTTP proxy. Try setting `http://address:port` to both
         HTTP_PROXY and HTTPS_PROXY instead.

      2. If you are using Windows, vcpkg will automatically use your Windows IE Proxy Settings
         set by your proxy software. See https://github.com/microsoft/vcpkg-tool/pull/77
         The value set by your proxy might be wrong, or have same `https://` prefix issue.

      3. Your proxy's remote server is out of service.

      If you've tried directly download the link, and believe this is not a temporary
      download server failure, please submit an issue at https://github.com/Microsoft/vcpkg/issues
      to report this upstream download server failure.

Call Stack (most recent call first):
  scripts/cmake/vcpkg_download_distfile.cmake:270 (z_vcpkg_download_distfile_show_proxy_and_fail)
  scripts/cmake/vcpkg_from_github.cmake:106 (vcpkg_download_distfile)
  ports/boost-core/portfile.cmake:3 (vcpkg_from_github)
  scripts/ports.cmake:175 (include)

duplicate of #38974

Having the same issue on MacOS Monterey with an M1 chip


[SUPPORT][HELP] SparkSQL can not read the latest change data without execute "refresh table xxx"

SparkSQL can not read the latest change data without execute "refresh table xxx" after write the data in datasource mode
To Reproduce
Steps to reproduce the behavior:

run spark-shell and import class
import org.apache.spark.sql.SaveMode
create like this:

spark.sql(
    s"""|CREATE TABLE IF NOT EXISTS `malawi`.`hudi_0_12_1_spark_test` (
        |     `id` INT
        |    ,`name` STRING
        |    ,`age` INT
        |    ,`sync_time` TIMESTAMP
        |) USING HUDI
        |TBLPROPERTIES (
        |     type = 'mor'
        |    ,primaryKey = 'id'
        |    ,preCombineField = 'sync_time'
        |    ,`hoodie.bucket.index.hash.field` = ''
        |    ,`hoodie.datasource.write.hive_style_partitioning` = 'false'
        |    ,`hoodie.table.keygenerator.class`='org.apache.hudi.keygen.ComplexKeyGenerator'
        |)
        |COMMENT 'hudi_0.12.1_test'""".stripMargin
)

spark.sql(
    s"""|create table `malawi`.`hudi_0_12_1_spark_test_rt`
        |using hudi
        |options(`hoodie.query.as.ro.table` = 'false')
        |location 'hdfs:/xxx/malawi/hudi_0_12_1_spark_test';
        |""".stripMargin
)

make test data

var dfData = spark.sql(
    s"""|select 1 as id,'name1' as name, 18 as age, now() as sync_time
        | union all
        |select 2 as id,'name2' as name, 22 as age, now() as sync_time
        | union all
        |select 3 as id,'name3' as name, 23 as age, now() as sync_time
        |""".stripMargin
)

var dfData2 = spark.sql(
    s"""|select 4 as id,'name1' as name, 18 as age, now() as sync_time
        |""".stripMargin
)

make hudi datasource options

var hoodieProp = Map("hoodie.table.name" -> "hudi_0_12_1_spark_test", "hoodie.datasource.write.operation"->"upsert", "hoodie.datasource.write.recordkey.field" -> "id", "hoodie.datasource.write.keygenerator.class"->"org.apache.hudi.keygen.ComplexKeyGenerator", "hoodie.datasource.write.partitionpath.field"->"", "hoodie.datasource.write.precombine.field"->"sync_time", "hoodie.metadata.enable"->"true", "hoodie.upsert.shuffle.parallelism"->"10", "hoodie.embed.timeline.server"->"false")

write data fist time

dfData.write.format("org.apache.hudi").options(hoodieProp).mode(SaveMode.Append).save("hdfs://xxx/malawi/hudi_0_12_1_spark_test")

query in spark sql

spark.sql(s"""|select *
              | from (
              |         select 'ori' as flag,a.* from `malawi`.`hudi_0_12_1_spark_test` a
              |         union all
              |         select '_rt' as flag,b.* from `malawi`.`hudi_0_12_1_spark_test_rt` b
              |      ) t
              |order by t.id asc, t.flag asc""".stripMargin
).show(false)

write data second time

dfData2.write.format("org.apache.hudi").options(hoodieProp).mode(SaveMode.Append).save("hdfs://xxx/malawi/hudi_0_12_1_spark_test")

repeat step 6
the data(id = 4) should be queried out in _rt table but it is not

refresh table

spark.sql("REFRESH TABLE `malawi`.`hudi_0_12_1_spark_test`")

repeat step 6

Environment Description

Hudi version : 0.12.1

Spark version : 3.1.3

Hive version : 3.1.1

Hadoop version : 3.1.0

Storage (HDFS/S3/GCS..) : HDFS

Running on Docker? (yes/no) : no

@alexeykudinkin Can you take a look, is this related with the FileIndex cache ?

Yes, I think @danny0405 is right. Like Spark file listing, hudi also cache file status to avoid call filesystem repeatedly when listing files. And the cache will be update when you call refresh explicitly.
But I'm thinking whether we can hold the number of complete commits as a variable, and decide whether to use the cache or not by comparing if that num is changed or not.[This cannot work if the relation changed to HadoopRelation, ToHadoopRelation use Spark InmemoryFileIndex]

@alexeykudinkin  Hi，is there any way to solve this problem, such as closing some parameter settings? In addition to manually refreshing the table (because this means that a large number of business codes need to be modified), there is no temporary closure measure. The bug that must be solved by modifying the business code is a destructive defect, which directly affects the upgrade and use. This problem should be repaired through the patch version.

@JoshuaZhuCN the reason why it's not refreshing is b/c you're writing into a table by path and not by table identifier (ie db.table) -- therefore it bypasses the Catalog and goes straight to Hudi's Data Source which is not going to be refreshing the Spark caches.
This is not an issue and is an expected behavior. To address try writing into the table by its id (which in turn will involve the catalog refresh)

@JoshuaZhuCN the reason why it's not refreshing is b/c you're writing into a table by path and not by table identifier (ie db.table) -- therefore it bypasses the Catalog and goes straight to Hudi's Data Source which is not going to be refreshing the Spark caches.
This is not an issue and is an expected behavior. To address try writing into the table by its id (which in turn will involve the catalog refresh)

@alexeykudinkin i  dont understand what "write into the table by its id" means, just using sql like insert into/update/delete from db.table` to write date?

@alexeykudinkin i don't understand what "write into the table by its id" means, just using sql like insert into/update/delete from db.table to write data?

Correct. You can do the same from Spark DS.

@alexeykudinkin At present, the problem I encounter is not only that the Spark datasource cannot be read after it is written, but also that the Spark sql cannot be read after it is written by Flink using hive sync. In other words, the SparkSQL query can not immediately read new data in any other way except by writing data in SQL. Therefore, I think this is a problem that needs to be solved

Interesting. Can you please create another issue specifically for this one as this hardly could be related?

@alexeykudinkin Let's resolve the related issues all in this one. Re-open it.

Can you elaborate why you think these issues are related?

@alexeykudinkin I think the query engine should not limit the writing way for querying data. Even for the tables created by Spakrsql, the query engine should be able to query new data regardless of the way in which the data is written by spark datasource, spark sql, java client, flink sql, and flink stream api, without requiring users to do additional operations for different writing methods when using the query engine.

This is not a limitation of the query engine this is a limitation of how you're using the query engine -- when writing to a table specified as a path following issues are at play

Spark SQL will cache the Relation w/in the session cache when queried
When writing to a table identified by a full path, rather than a name and Spark has no way to invalidate the SQL session cache (since it doesn't have the table identifier)

@alexeykudinkin At present, the problem I encounter is not only that the Spark datasource cannot be read after it is written, but also that the Spark sql cannot be read after it is written by Flink using hive sync. In other words, the SparkSQL query can not immediately read new data in any other way except by writing data in SQL. Therefore, I think this is a problem that needs to be solved

Interesting. Can you please create another issue specifically for this one as this hardly could be related?

@alexeykudinkin here https://github.com/apache/hudi/issues/7452

@JoshuaZhuCN thanks for creating #7452.
Closing this now, as this is not really a Hudi issue, but rather the way Spark is used

@JoshuaZhuCN you can try this issues around by actually switching off Spark's file relation cache by setting spark.sql.filesourceTableRelationCacheSize=0


Bridge's System.Collections.Generic.List implements Count() and Slice(int) incorrectly

The following code, although working good in C# intellisense, generates broken JavaScript:
    public class App
    {
        [Ready]
        public static void Mainz()
        {
            HeadingElement h1 = new HeadingElement(HeadingType.H1);
            h1.InnerHTML = "Look in js console";

            Document.Body.AppendChild(h1);

            List<string> items = new List<string>() { "bus", "plane", "tram", "train", "sub" };

            Console.Log(items.Count);

            List<string> result = items.Slice(2);

            Console.Log(result.Count); // error because result = undefined
        }
    }

This is the output:
        mainz: function () {
            var h1 = document.createElement("h1");
            h1.innerHTML = "Look in js console";

            document.body.appendChild(h1);

            var items = Bridge.merge(new Bridge.List$1(String)(), [
                ["bus"],
                ["plane"],
                ["tram"],
                ["train"],
                ["sub"]
            ] );

            console.log(items.getCount());

            var result = items.slice(2);

            console.log(result.getCount()); // error because result = undefined
        }
    }

In JavaScript, items itself does not have a slice() or getCount() methods. But rather items.items has, respectively, slice() and length.
In live bridge, you can rewrite the code with its last three lines with:
            Console.Log(items.items.Length);
            List<string> result = items.items.Slice(2);
            Console.Log(result.Length);

And you will have a working equivalent code.
This code though, is not functional on Visual Studio or intellisense-aware IDEs as List does not implement a public property/field named items.

I believe this situation will no longer be reproducible after the fix above. Now JavaScript List's slice() method returns a new List with the slice.

PR #3119 removed Slice methods from List class as non-standard.


[Console] fix console deinit: add ability to unblock linenoise, to fix deadlock (IDFGH-8529)

This solves: https://github.com/espressif/esp-idf/issues/9974, and https://github.com/espressif/esp-idf/issues/9877
Main Changes (to solve the deadlock):

Implements: usb_serial_jtag_unblock_reads(); & uart_unblock_reads(); which causes any pending read() to return -1 with EWOULDBLOCK.
Implement vRingbufferUnblockRx which is what the above functions are wait on
callingesp_console_repl_X_delete() unblocks and safely exits the linenoise thread, before we stop the console. This lets the linenoise thread let go of the _readLock, so that we can deinit.
linenoise.c, read operations now return early if EWOULDBLOCK is returned by the read operation. This is necessary for linenoise to exit when we deinit.

The crux of the deadlock fix is here:
esp_console_replc.
static esp_err_t esp_console_repl_usb_serial_jtag_delete(esp_console_repl_t *repl) {
    ....
    // wait for repl thread to stop, if it is running
    if (repl_com->state == CONSOLE_REPL_STATE_START) {
        repl_com->state = CONSOLE_REPL_STATE_REPL_STOP_REQUESTED;
        while (repl_com->state != CONSOLE_REPL_STATE_REPL_TASK_ENDED) {
            usb_serial_jtag_unblock_reads();
            vTaskDelay(20 / portTICK_PERIOD_MS);
        }
    }
    ...
}

Do you want me to implement this change for Uart and Usb CDC as well? Let me know what you think about this change.

Let me know if I can make this less "risky" to merge. The only changes I need now are these 2 functions to be available:

esp_vfs_usb_serial_jtag_use_driver_for_rx();
usb_serial_jtag_has_ever_received_bytes_since_boot();

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.2 out of 4 committers have signed the CLA.:white_check_mark: chipweinberger:white_check_mark: sio13:x: gspatankar:x: david-cermakYou have signed the CLA already but the status is still pending? Let us recheck it.

@igrr Any thoughts on these changes?
And specifically these 2 changes:
(low risk & important) [Driver] add non-blocking for tx & rx individually
(low risk & important) [VFS] add flag if driver has ever received bytes

@igrr hope you have time to respond to the new comments at some point.

@igrr my mistake, my comments were still "pending"

@igrr I'm okay denying this Pull Request. Seems like we are loosing steam on it.
The last idea I had was this approach:
void xxx_console_deinit(){
   ....
  while(linenoiseIsWaitingForNextCommand()){
        vTaskAbortDelay(consoleThread);
   }
   ....
}

I wont have time to implement a select based solution.
Let me know, otherwise I'll close the PR. I'll open a new PR regardless if we go forward.

Closing this PR. Still Unfixed. De-initializing consoles is not possible until this is fixed.
If anyone ever fixes this deadlock, they should open a new PR. Please see my previous comment for how this could be solved. Ivan does not want to alter FreeRTOS.


On dnd-effect-allowed="copy", items cannot be stopped from duplicating themselves withing their container.

dnd-droppable-in, dnd-type and dnd-allowed type are not working.

Would love to know how this was resolved.

I made sure the splice is occuring
On 12 Aug 2016 6:10 pm, "Sarah Dulaney"<EMAIL_ADDRESS>wrote:

Would love to know how this was resolved.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/marceljuenemann/angular-drag-and-drop-lists/issues/252#issuecomment-239488944,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIvfo0pfZ79H1t7e_NIBj0y1hz3XGhUaks5qfJr8gaJpZM4IPBG_
.


fix: SegFault in remove(x) DLL #765

Description of Change
Segmentation fault occurred while deleting elements. Fixing required to check for the NULL before checking the value of the node, as well as many corner cases which I have handled. reverseShow() failed as well because of NULL node.

Checklist

[x] Added description of change
[x ] PR title follows semantic commit guidelines
[x] Search previous suggestions before making a new one, as yours may be a duplicate.
[x] Sort by alphabetical order
[x] I acknowledge that all my contributions will be made under the project's license.

This looks good but the cpplint is failing.  Please click Details to the right of the ❌ below.  TL;DR: Please start by replacing tab indentation with 4 spaces or run clang-format to auto fix.

@kheealam Can you resolve the conflict?

#765 Was fixed in a different PR


Expecting value: line 1 column 1 (char 0)

Thanks for your work! But I met an error when I ran the python app.py. It said "Error Expecting value: line 1 column 1 (char 0)". Can you help me fix this problem?

can you share the whole command you ran?

can you share the whole command you ran?

I enter the command 'python app.py --share'. After opening the web, I uploaded the image and gave a prompt and it gave an error.

can you share the whole command you ran?

I enter the command 'python app.py --share'. After opening the web, I uploaded the image and gave a prompt and it gave an error.

Hello, I got the same problem with u, do u know how to solve it? thx!

can you share the whole command you ran?

I enter the command 'python app.py --share'. After opening the web, I uploaded the image and gave a prompt and it gave an error.

Hello, I got the same problem with u, do u know how to solve it? thx!

sry, i don't know

can you share the whole command you ran?

I enter the command 'python app.py --share'. After opening the web, I uploaded the image and gave a prompt and it gave an error.

Hello, I got the same problem with u, do u know how to solve it? thx!

sry, i don't know

Actually, I write some codes in this work, i.e. “result.show(), or result.save()” to get the inpainting result. It works for me.

Hi @XiruiTeng , @Chinafsh,
I'm sorry I can't reproduce this problem :(
I guess it can be raised when you are using a VPN or proxy. You can have a try after turning off VPN.


IRC links to jenkins.kubernetes.io don't work

IRC gets status reports from builds, but the included links don't work.

[12:25]  Project kubernetes-e2e-gce build #1609: FIXED in 40 min: http://jenkins.kubernetes.io/job/kubernetes-e2e-gce/1609/

And my browser:

The server at jenkins.kubernetes.io can't be found, because the DNS lookup failed.

Zach, I think you've been doing jenkins work?  Or perhaps this is @brendandburns ?

Yeah, this is because the Jenkins server is still not public at all, and we'd rather not spam the private URL. I'm happy to own it but it's going to sit for a while.

Okay, thanks Zach -- P3.


feat: add borrowed mutable and referenced runtime

For the evm tests we require the ability to produce blocks from the account, which wraps the runtime. It would be fantastic if it is possible to get a borrowed Ref or MutRef so that we can do such.
What this does is exactly that, with the bonus of fixing a potential issue with Runtime::current_block which had returned a mutable Block and not a referenced Block. I'm sure it is ideal for there not to be mutable access to it as it is generated.

Looks like @mhalambek fixed the CI. So we can merge this branch now.


chore(ContractOffer): remove field assetId

What this PR changes/adds
Removes assetId from ContractOffer
Why it does that
Cleanup, since assetId is not used  by anyone
Linked Issue(s)
Closes #2030
Checklist

[ ] added appropriate tests?
[x] performed checkstyle check locally?
[ ] added/updated copyright headers?
[ ] documented public classes/methods?
[ ] added/updated relevant documentation?
[x] assigned appropriate label? (exclude from changelog with label no-changelog)
[x] formatted title correctly? (take a look at the CONTRIBUTING and styleguide for details)

Hi @ndr-brt
in this way the field will be removed also from the API which could cause a breaking change, i guess it could be an issue right?

Even if it was used in downstream projects, e.g. MVD, we'll just have to adapt there.


Enable GitHub discussions feature to enable easier chat between users

Is your feature request related to a problem? Please describe.
I am sure people have lots of good ideas, questions to ask etc.
These sort of things aren't always issues, but the new per-repository discussions feature of GitHub are great for this kind talk. Easy to enable, just a checkbox away in the repo config :).

@mountaindude you have now the chance to be the first who starts a discussion.

@mountaindude you have now the chance to be the first who starts a discussion.


Fix failing test cases and add new test cases for bal persist integration with bal build command

Description:
Part of https://github.com/ballerina-platform/ballerina-library/issues/5784

Fixed in https://github.com/ballerina-platform/persist-tools/pull/331


the dimensions of the action

Can I understand it like this: In real-world deployment, instead of a simulated environment, for a self-created dataset, the dimensions of the action should include [x, y, z, roll, pitch, yaw] rather than [joint_1_angle, joint_2_angle, ..., joint_6_angle], and it should not include the gripper. The dimensions of the state can be defined according to our own requirements.

Right. It also can be joint angles or including grippers, as the pre-training datasets have those examples.


Conformence tests should be well-documented.

As interchaintest as a library user
I want to be able to discover the conformance package
Because its the standard way to test my ibc integration
Acceptance Criteria

[x] a link from the interchaintest README
[x] a .md file w/in docs explaining how to use conformance from the interchaintest binary
[x] a .md file w/in docs explaining how to use conformance as a library

I have just opened a PR that will add a doc to interchaintest explaining how to import into other repos as a library to make use of the conformance testing suite here


fix(choice): process value change is new !== old

Here is a fix for ux-choice-container attribute. When we optimized the code in valueChanged() we removed an important part... this valueChanged() must also request to process the value if the newValue is different thant oldValue.
Currently the attribute works well for one-way binding because the right choice is selected anyway. But for two-way binding the valueChanged() needs to process the value.
Ready for review @EisenbergEffect @bigopon

By default, valueChanged wont be called unless newValue !== oldValue. Can you give an example where this would not be the case?

I don't think it would happen. We can remove this if

kk thanks for the fix @ben-girardet
I will try to setup test infra for us to start testing and be able to solidify our logic around tricky parts soon.


Why does make try to build tools/runner?

With #1861, it is almost possible to checkout weaveworks/weave in an arbitrary location (i.e. not under your GOPATH) and do make.
Almost, but not quite, because it fails while doing make -C tools/runner.  But it's not obvious to me why the all target needs to do that anyway (if one considers the point of the default target to be to satisfy the instructions in site/building.md).  tools/runner looks like something for circleci, and surely circleci can invoke make with some other target to get what it needs?
If I remove $(RUNNER_EXE) as a dependency of all, then a build succeeds when not under GOPATH.

CC @tomwilkie

Yup we can remove that, but we we surely get complaints for people who try to run the tests manually (@rade).  I could add it to the run-smoketests target?

I could add it to the run-smoketests target?

Fine, but I don't normally run that; I tend to run ./setup.sh and individual or groups of tests manually. So perhaps also add a pseudo-target for building all that is required to run the smoketests, so I get something more memorable than make tools/runner/runner.

So perhaps also add a pseudo-target for building all that is required to run the smoketests

And make run-smoketests depend on that, as I frequently use make all tests run-smoketests.

And make run-smoketests depend on that

Exactly. that was the idea.


Laboratory work #2, Garanina Maria - 19FPL1

Well done, We will discuss styling and technical issues later


python39Packages.tablib: 3.2.1 -> 3.3.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python39Packages.tablib is: "Format-agnostic tabular dataset library"
meta.homepage for python39Packages.tablib is: "https://tablib.readthedocs.io/"
meta.changelog for python39Packages.tablib is: "https://github.com/jazzband/tablib/raw/v3.3.0/HISTORY.md"
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
Warning: a test defined in passthru.tests did not pass
found 3.3.0 with grep in /nix/store/qc4iajfn4ssn8cn6f9s4f9wzinq5xsbv-python3.9-tablib-3.3.0
found 3.3.0 in filename of file in /nix/store/qc4iajfn4ssn8cn6f9s4f9wzinq5xsbv-python3.9-tablib-3.3.0

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

7 total rebuild path(s)

7 package rebuild(s)

First fifty rebuilds by attrpath
netbox
peering-manager
python310Packages.django-tables2
python310Packages.tablib
python39Packages.django-tables2
python39Packages.tablib
tandoor-recipes

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/qc4iajfn4ssn8cn6f9s4f9wzinq5xsbv-python3.9-tablib-3.3.0 \
  --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 python39Packages.tablib https://github.com/r-ryantm/nixpkgs/archive/dd28110140e39552eb11eeef37e0afcbd9d64bb2.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/qc4iajfn4ssn8cn6f9s4f9wzinq5xsbv-python3.9-tablib-3.3.0
ls -la /nix/store/qc4iajfn4ssn8cn6f9s4f9wzinq5xsbv-python3.9-tablib-3.3.0/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

7 packages built:

netbox
peering-manager
python310Packages.django-tables2
python310Packages.tablib
python39Packages.django-tables2
python39Packages.tablib
tandoor-recipes

Maintainer pings

Result of nixpkgs-review pr 206735 run on x86_64-linux 1

7 packages built:

netbox
peering-manager
python310Packages.django-tables2
python310Packages.tablib
python39Packages.django-tables2
python39Packages.tablib
tandoor-recipes


feat(core): add CMapBuilder structure

Implement the CMapBuilder struct with the needed features to replace existing constructor items.
The builder currently does not support generic attributes; See #57 for details
Scope

[x] Code: honeycomb-core

Type of change

[x] New feature(s)

Necessary follow-up

[x] Other: Previous utils / io / constructor items need to be deprecated & redirect to the new struct

Codecov Report
Attention: Patch coverage is 95.47170% with 24 lines in your changes are missing coverage. Please review.

Project coverage is 78.52%. Comparing base (c128760) to head (41b6145).

Files
Patch %
Lines

honeycomb-core/src/cmapbuilder/structure.rs
48.93%
24 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #87      +/-   ##
==========================================
- Coverage   86.67%   78.52%   -8.16%
==========================================
  Files          19       24       +5
  Lines        2469     2738     +269
==========================================
+ Hits         2140     2150      +10
- Misses        329      588     +259

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Fixed null safety issue in icon

Related Issue: #7
Previous Code::
icon initialization  final Widget icon;
icon usage icon: icon ?? Icon(             Icons.expand_more,           ),
according to this case icon is optional, so I updated icon as optional
initialization final Widget? icon;
icon usage icon: widget.icon ?? Icon(             Icons.expand_more,           ),

Yes, I noticed this earlier and since you have fixed, merged yours. Thanks @Rashid-Khabeer for your contribution!


Cannot call method 'slugify' of undefined

Hi,
When I run the generator, in the step "Will you be using heroku?" I have this error:
? Will you be using heroku? No
(!) #mkdir() is deprecated. Use mkdirp module instead. https://www.npmjs.com/package/mkdirp
(!) #mkdir() is deprecated. Use mkdirp module instead. https://www.npmjs.com/package/mkdirp
(!) #_ is deprecated. Require your own version of Lodash or underscore.string

events.js:72
        throw er; // Unhandled 'error' event
              ^
TypeError: Cannot call method 'slugify' of undefined
    at eval (eval at <anonymous> (/usr/local/lib/node_modules/generator-mongoose/node_modules/yeoman-generator/node_modules/lodash/index.js:10693:16), <anonymous>:8:13)
    at underscore (/usr/local/lib/node_modules/generator-mongoose/node_modules/yeoman-generator/lib/util/engines.js:23:45)
    at MongooseGenerator.actions.engine (/usr/local/lib/node_modules/generator-mongoose/node_modules/yeoman-generator/lib/actions/actions.js:224:32)
    at MongooseGenerator.template (/usr/local/lib/node_modules/generator-mongoose/node_modules/yeoman-generator/lib/actions/actions.js:198:19)
    at MongooseGenerator.app (/usr/local/lib/node_modules/generator-mongoose/app/index.js:76:8)
    at /usr/local/lib/node_modules/generator-mongoose/node_modules/yeoman-generator/lib/base.js:421:16
    at processImmediate [as _immediateCallback] (timers.js:345:15)

I think that is related with this issue: https://github.com/yeoman/generator/issues/766

I will look into this, will have a fix for this tonight.
Thanks!

While installing the generator
$  npm install -g generator-mongoose
npm WARN peerDependencies The peer dependency<EMAIL_ADDRESS>included from generator-mongoose will no
npm WARN peerDependencies longer be automatically installed to fulfill the peerDependency
npm WARN peerDependencies in npm 3+. Your application will need to depend on it explicitly.
<EMAIL_ADDRESS>/usr/local/lib/node_modules/generator-mongoose
├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>└──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
error stack when running the installer with a No for heroku (genmon is the name of folder I am scaffolding in)
 $ genmon  yo mongoose
(!) this.readFileAsString() is deprecated. Use require("html-wiring").readFileAsString() instead.

 .--------------------------.
 |      mongoose:app        |
 .__________________________.

? Database Name: myDb
? Database Host: localhost
? Database User:
? Database Password:
? Database Port: 27017
? Will you be using heroku? No
(!) #mkdir() is deprecated. Use mkdirp module instead. https://www.npmjs.com/package/mkdirp
(!) #mkdir() is deprecated. Use mkdirp module instead. https://www.npmjs.com/package/mkdirp
(!) #_ is deprecated. Require your own version of Lodash or underscore.string
events.js:85
      throw er; // Unhandled 'error' event
            ^
TypeError: Cannot read property 'slugify' of undefined
    at eval (lodash.templateSources[1]:9:12)
    at underscore (/usr/local/lib/node_modules/generator-mongoose/node_modules/yeoman-generator/lib/util/engines.js:23:45)
    at MongooseGenerator.actions.engine (/usr/local/lib/node_modules/generator-mongoose/node_modules/yeoman-generator/lib/actions/actions.js:224:32)
    at MongooseGenerator.template (/usr/local/lib/node_modules/generator-mongoose/node_modules/yeoman-generator/lib/actions/actions.js:198:19)
    at MongooseGenerator.app (/usr/local/lib/node_modules/generator-mongoose/app/index.js:76:8)
    at /usr/local/lib/node_modules/generator-mongoose/node_modules/yeoman-generator/lib/base.js:421:16
    at processImmediate [as _immediateCallback] (timers.js:358:17)

$ yo --version
1.4.6

Just wanted to keep you up to date, found the changes I need to make, will be making them later today.

@leandono & @arvindr21 Thanks for the help, this is resolved.


[experimental] Add presubmit check for ESP32 target

Problem
<<<<<FILL ME IN - the issue this PR is intended to address>>>>>

Summary of Changes
<<<<<FILL ME IN - what's in this PR>>>>>

Fixes #<<<<<FILL ME IN  - issue number(s). If no issue, please create one>>>>>

Please fill in the "fill me in" bits so it's clear what the goal of this PR is.

This PR will be replaced by #5945 since we have resolved the compatibility of im and ember library.


Adjusting silence models

Thanks for a great github project. I'm trying to adjust the silence models and haven't been able to come up with much except this:
pmsseg=./$datadir/$show.pms.seg
java -Xmx$mem -classpath "$LOCALCLASSPATH" fr.lium.spkDiarization.programs.MDecode  --help  --fInputDesc=$fDescD --fInputMask=$features --sInputMask=./$datadir/%s.i.seg --sOutputMask=$pmsseg --dPenality=10,10,50 --tInputMask=$pmsgmm $show

where
Penality set penalties for the i to j states
10 for the first and second model corresponding to boad/narrowband silence
50 for the other jingle speech (f0 f2 f3 fx), jingle and music
So I have 2 questions:

How to determine these parameters
Are there any other options for adjusting the silence/non speech models

Hi, I am very sorry, I have not clear about how to determine those parameters.
Maybe you can contact the LIUM team
Do let me know if you get the answers.
Thanks.

Will do thanks

i wasn't able to find any answer even grubbing among source in the .jar
however i have another (quite related) question:
let's say that i record 10 seconds, 3 of speech and the rest of light noisy background. pms analysis is telling me that silence occoured in a certain range with label ''iS''
How come in the next step (see below) silence gets clustered as a new and unique speaker?
Any tips?
echo filter spk segmentation according pms segmentation
fltseg=$datadir/$show.flt.$h.seg
java -Xmx$mem -classpath "$LOCALCLASSPATH" fr.lium.spkDiarization.tools.SFilter  --fInputDesc=$fDescD --fInputMask=$features --fltSegMinLenSpeech=150 --fltSegMinLenSil=25 --sFilterClusterName=j --fltSegPadding=25 --sFilterMask=$pmsseg --sInputMask=$datadir/%s.d.$h.seg --sOutputMask=$fltseg $show

That's not what happens to me, but thanks for pointing out the iS code -- I hadn't noticed that.
For my data, using this github repo, the iS disappears at the flt stage.


train_and_evaluate.ipynb ValueError: Input contains NaN, infinity or a value too large for dtype('float32').

In tutorial: https://github.com/tensortrade-org/tensortrade/blob/master/examples/train_and_evaluate.ipynb
Implement basic feature engineering
sel.fit(X_train, precalculate_ground_truths(X_train, column='close'))
we got:
ValueError: Input contains NaN, infinity or a value too large for dtype('float32').

Maybe your data has NAN values. Try X_train.dropna()

I also encountered this problem and tried X_train.dropna(), but problem still there, any more suggestions?

Hey, also encountered this when returned to my code after some time off.
In my case it seemed to be an issue with Bitcoin volume at certain exchanges when using CryptoDataDownload. Change to BTC-USDT on Binance Exchange resolved this issue.
Hope it might help!

I haven't tried Binance data yet. But after taking a quick glance, I found a value is too large to convert to float32.
Before the fit function, try adding the below line.
X_train = X_train.drop(labels='compsum', axis=1)

Hey, also encountered this when returned to my code after some time off.
In my case it seemed to be an issue with Bitcoin volume at certain exchanges when using CryptoDataDownload. Change to BTC-USDT on Binance Exchange resolved this issue.
Hope it might help!

Would it be possible to share the modification? Below code returns HTTPError: HTTP Error 404: Not Found
bitfinex_data = cdd.fetch("Binance", "BTC", "USDT", "1h")

Thanks

Hey, also encountered this when returned to my code after some time off.
In my case it seemed to be an issue with Bitcoin volume at certain exchanges when using CryptoDataDownload. Change to BTC-USDT on Binance Exchange resolved this issue.
Hope it might help!

Would it be possible to share the modification? Below code returns HTTPError: HTTP Error 404: Not Found
bitfinex_data = cdd.fetch("Binance", "BTC", "USDT", "1h")

Thanks

done:
Modified the line as below:
bitfinex_data = cdd.fetch("Binance", "USDT", "BTC", "1h")

Also added one line in the fetch_default function in the cdd.py file:
    df = pd.read_csv(self.url + filename, skiprows=1)
    **df.columns = [x[0].lower() + x[1:] if x[0]!='V' else x for x in df.columns]**
    df = df[::-1]

Now it seems working.

This issue is related to the datetime stamp of UNIX. I had to revise the cdd.py file.


Add an option to directly update a state if the current configuration is loaded from a state

also address the snack bar for annotation form not functioning properly issue? (double check as it was an issue from previous PR)


Styling not responsive

I'm opening my modal which looks fine in larger browser windows, but overflows smaller viewports and makes them become unusable as large parts of the modal are outside of the screen and is not scrollable. These modals work properly with the original SweetAlert library

Okay, after viewing the source, it seems there is a different style when using a mobile useragent, so after a refresh, the styling is already much better. However, with a large modal the bottom is still outside of the screen and not scrollable. Possibly this can be fixed by changing the styling to flexbox so it is also responsive?

I made some changes to the styling that use the flexbox system like the original sweetalert library https://github.com/djorg83/react-bootstrap-sweetalert/pull/47. I am however unable to run the project on my machine as it keeps giving warnings when trying to build. So if you have some spare time and would be able to build and test these changes? Or just build and give the file so I can test.
Regards :)

Are you working from the latest in master?  I've been pushing a bunch of changes recently.

You'll need to install yarn, typescript, and tslint to build.  Use command yarn demo to build and run a demo server locally.  This also updates the bundle js file for the live demo.

Hey, I forked the latest from master when I opened this issue. I installed yarn, node, typescript and tslint and am now able to demo and build. However, I get the same issue you have on your own demo site where the console gives an error that d is not a constructor. Which seems to be the Example class which seems fine, so it's kinda weird.
I did manage to build however and use this in my own project to test. I adjusted some more of the styling to be responsive and always stay in the centre of the screen (so it also fixes my other issue). I got most of the changes from the original SweetAlert library. These changes can also be found in my pull request #47.
Normal Alert:

Smaller alert on mobile but still with nice padding and no overflow:

Really large alert where the top starts at the top of the browser and is able to scroll so the entire alert is readable:

Thanks for this update.

I'll fix the d not a constructor issue.

@djorg83 Any chance you could push a new release with this in it to npm, please?

Sorry, I have been really short on free time recently.  I'll test the latest version now, and if everything is good I'll push a release to npm.

I've published version 5.0.1


Simulation sometimes results differ from other tools

This was noticed by @freiburgermsu.
Example model (Elowitz repressilator model)

SBML
CellML

Example results
PySCeS
PySCeS' results are internally consistent between CVODE and LSODA, but different from that of other tools that also use LSODA and CVODE

COPASI (LSODA)

OpenCOR (CVODE)

Interactive versions of the same results

SBML/PySCeS/CVODE
SBML/COPASI/LSODA
CellML/OpenCOR/CVODE

@jonrkarr thanks for the report, I will look into it.

BIOMD0000000012_url.xml.psc.txt
The bug is in the SBML parser. The <ln/> MathML operator is translated to math.log10 in the PSC file. Corrected PSC file attached above. This gives the correct results. @bgoli  you wrote the SBML conversion code, would you be able to pinpoint this quickly?

Maybe Frank Bergmann could shed light on how COPASI does it.
Thanks for the explanation. GillesPy2 has similar issue with this model. I only realized this from your explanation about ln. In their case, they raise an exception which indicates they don't know how to interpret ln.

@jmrohwer thanks for finding the problem this, I will look into this in more detail. We probably notice the problem as we use the L2 infix translator while COPASI would use the ASTnodes directly.

This #45 fixes it, but I am not sure if you would want to make more significant changes in terms of using the AST nodes vs infix translator. Also, SBML export from the PSC file does not work 100% (assignment rules are not properly translated), this most probably has to do with PySCeS still using SBML L2 internally.

Fixed in #53


Fix #1217 - converted some properties from implicitly unwrapped optionals to address crash

This PR changes JTACMonthLayout.delegate and JTACMonthDelegateProtocol._cachedConfiguration from implicitly unwrapped options to optionals. It addresses a crash reported in #1217 where the following line crashes in rare circumstances due to a force unwrapped optional with a nil value.
JTACMonthLayout.Swift:168:
strictBoundaryRulesShouldApply = thereAreHeaders || delegate._cachedConfiguration.hasStrictBoundaries

Although I don't know under which circumstances the implicitly unwrapped optional(s) are nil, this PR should at least prevent the crash. It should have no impact on functionality.

@patchthecode  Is there any plans to merge this PR ? I'm facing the exact same crash in my code.

I'll review this on the weekend.
in the mean time @mohanvydehi does this fix your issue?

Yes, it worked for me @patchthecode

Our team has the same issue. It would be great to see this PR merged.

Just following up here, this should really get merged

Awesome. Thank you


Missing build instructions

Hi There!
This is a really nice project. I would love it if you would provide the build instructions

Hi @ZeroCoolHacker
Actually I heaved stopped development in the master branch which has nodejs/expressjs/reactjs.
To check the current code so far, please follow the following

After cloning the repository navigate to pos-api and pos-ui folders invidually and run yarn  in both the folders.

The run yarn start in both the folders. For username and password kindly check the https://github.com/shanmugharajk/react-point-of-sale/blob/master/pos-api/database.sqlite file please.

I didn't add instructions because I stopped the development in the current master branch.
But I am actively working the backend migration to elixir/phoenix and react with hooks/material ui latest version along with typescript.
Once I am done with that I will add a demo link along with all clear instructions.
Thanks!

Thanks for the instructions!

Hi @shanmugharajk , are you working on this project?


disabled.from and to are the wrong way around?

Hello,
I am confused with the from and to aspect of disabling dates. I would expect that from is the date you are starting on and to is the date you are ending on. From being lowest, to being highest.
from: 2017-01-01
to: 2017-01-15
but with datepicker its the other way around which is a little confusing.
Is there any logic behind this decision that maybe I am missing?
Many Thanks! :)
~G

Disabled to is to disable all dates to or up to the date and disabled from is disabling all dates from or after the date.
Hope that reasoning makes more sense now?

@charliekassel thanks for the quick response.
What you saying makes sense but myself and my team mates find it confusing to think that way round.
This makes more sense:
from - meaning that anything before is disabled, going from a certain date forward.
'to' - up until a certain date, anything beyond that date disabled.
Similar logic for comparison would be: minDate/startDate and maxDate/endDate.
minDate: The lowest date you can pick.
maxDate: The highest date you can pick.
from: The lowest date you can pick.
to: The highest date you can pick.
Its your component and we are grateful to be using it. Just a suggestion.


Problem installing on Linux Mint

I try to install mermaid. clli on Linux mint, this fails because puppeteer install fails due apparently to an error in install.js (line 78):
><EMAIL_ADDRESS>install /usr/local/lib/node_modules/mermaid.cli/node_modules/puppeteer
> node install.js

/usr/local/lib/node_modules/mermaid.cli/node_modules/puppeteer/install.js:78
  return Promise.all([...cleanupOldVersions, generateProtocolTypesIfNecessary(true /* updated */)]);
                      ^^^

SyntaxError: Unexpected token ...
    at exports.runInThisContext (vm.js:53:16)
    at Module._compile (module.js:374:25)
    at Object.Module._extensions..js (module.js:417:10)
    at Module.load (module.js:344:32)
    at Function.Module._load (module.js:301:12)
    at Function.Module.runMain (module.js:442:10)
    at startup (node.js:136:18)
    at node.js:966:3
/usr/local/lib
└── (empty)

npm ERR! Linux 4.15.0-47-generic
npm ERR! argv "/usr/bin/nodejs" "/usr/bin/npm" "install" "-g" "mermaid.cli"
npm ERR! node v4.2.6
npm ERR! npm  v3.5.2
npm ERR! code ELIFECYCLE

npm ERR<EMAIL_ADDRESS>install: `node install.js`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node install.js'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the puppeteer package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node install.js
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs puppeteer
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls puppeteer
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /home/openeyedev/npm-debug.log
npm ERR! code 1

In your NPM log I see this:

npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! node v4.2.6
npm ERR! npm  v3.5.2

Puppeteer requires at least Node v6.4.0.
Please run this code to upgrade your NPM & NodeJS package:
sudo apt-get update && sudo apt-get install -y nodejs npm

Best;

@Yokutto Thanks for your reply! I was finally able to get everything working after updating as you suggested the packages.
I had to use this command first to get a newer version then in my default OS packages:
curl -sL https://deb.nodesource.com/setup_10.x | sudo -E bash -

Updating the node source via the above worked when nothing else did.   I'm on Ubuntu 16.04.


adding latest changes

update ADT_Backend to the lastest implementation.
every further PRs will be smaller and more function specific

I have a few comments, but I guess we are more or less forced to merge this once it compiles. So I will open separate issues

I've approved the review, but not checked the details, guess we should just merge the PR and then move from there

I'm not that huge fan of try and error, but hopefully, within the next PRs we should be on a clean track


Revert "Use Docker Image For Documentation"

Reverts byte-physics/igor-unit-testing-framework#71

Needs more work.


feat(stepper): Support additional properties for step

Add support for editable, optional, and completed properties of each step.

I had to close and reopen the PR for additional properties of step b/c of rebase issues 😢
Previous comments addressed; ready for review.

Ready for review again. I also left some follow-up comments :+1:

Addressed comment regarding shared code between horizontal and vertical stepper. Ready for review :smiley:

Refactoring and optional step logic change done. Ready for review again :+1:

Changes made based on discussion. Ready for another review

Ready for review again :smiley:


allow optional appendment to shell configurations

I'm using zinit to install nvm, zinit allows things to be run lazily and can execute scripts, and load/unload completions, as well as keep track of them. I am also using yadm, yadm manages dotfiles, so the zshrc that's being modified is actually just a copy of a template that may later be overwritten. It'd be nice if I could run the installer without writing to my ~/.zshrc

I'm not familiar with any of those tools, and please note that nvm is 100% unsupported unless you use the install script in the readme.
You can set PROFILE=/dev/null when running the install script, and it won't attempt to write to a profile file, but then you'll have to source nvm.sh on your own.

I'm not familiar with any of those tools, and please note that nvm is 100% unsupported unless you use the install script in the readme.
You can set PROFILE=/dev/null when running the install script, and it won't attempt to write to a profile file, but then you'll have to source nvm.sh on your own.

I'm just suggesting that you allow us to configure the installer to disable
the appending, why can't that be possible, it's certainly an easy change
On Wed, Jan 13, 2021, 5:33 PM Jordan Harband<EMAIL_ADDRESS>wrote:

Closed #2410 https://github.com/nvm-sh/nvm/issues/2410.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nvm-sh/nvm/issues/2410#event-4203703556, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAABLDMSE4DFWP2BWBWLPFLSZYUWLANCNFSM4WBOQIRQ
.

I'm just suggesting that you allow us to configure the installer to disable
the appending, why can't that be possible, it's certainly an easy change
On Wed, Jan 13, 2021, 5:33 PM Jordan Harband<EMAIL_ADDRESS>wrote:

Closed #2410 https://github.com/nvm-sh/nvm/issues/2410.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nvm-sh/nvm/issues/2410#event-4203703556, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAABLDMSE4DFWP2BWBWLPFLSZYUWLANCNFSM4WBOQIRQ
.

It is already configurable, by setting the PROFILE env var to /dev/null, as I explained.

It is already configurable, by setting the PROFILE env var to /dev/null, as I explained.

sorry, I missed that... somehow... mah bad. But now I'm using that and it's not adding the lines, I'm only executing nvm.sh, but somehow I'm still getting completions, not that I don't want completions, but I'm rather perplexed. deleted my nvm dir, deleted ~/.zinit, and exited my terminal, just to be sure all of the in memory stuff and cached configs and whatever were cleared out. still there.
oh I see, I have a plugin already installed that's already loading some completions for nvm, not sure if they're a direct copy from here or not, but kinda good that I'm no longer loading both.

sorry, I missed that... somehow... mah bad. But now I'm using that and it's not adding the lines, I'm only executing nvm.sh, but somehow I'm still getting completions, not that I don't want completions, but I'm rather perplexed. deleted my nvm dir, deleted ~/.zinit, and exited my terminal, just to be sure all of the in memory stuff and cached configs and whatever were cleared out. still there.
oh I see, I have a plugin already installed that's already loading some completions for nvm, not sure if they're a direct copy from here or not, but kinda good that I'm no longer loading both.

@xenoterracide can you paste your solution to work with zinit?

For reference here's a one-liner for installing nvm without appending to shell config:
PROFILE=/dev/null bash -c 'curl -o- https://raw.githubusercontent.com/nvm-sh/nvm/v0.39.3/install.sh | bash'

Created #3100 to add this to readme.


Constr param: Is there a way to preserve the regex annotation?

Is there a way to preserve the regex annotation?
when i add constr the docs show just str
    use_filters: Optional[List[constr(regex=models.SEGMENT_PARAM_REGEX)]] = Query(
        None,
        alias="useFilters",
        title="Use Filters",
        # regex=models.SEGMENT_PARAM_REGEX,
        description="Filter results using Field:Operator:Value",
    ),

but with the regex on Query it shows up in the docs
    use_filters: Optional[List[str]] = Query(
        None,
        alias="useFilters",
        title="Use Filters",
        regex=models.SEGMENT_PARAM_REGEX,
        description="Filter results using Field:Operator:Value",
    ),

Originally posted by @dwbelliston in https://github.com/tiangolo/fastapi/issues/1021#issuecomment-786443902

Yep, exactly that, you should put the regex in the parameter passed to Query instead of using the const function. Both achieve the same.

Sorry for the long delay! 🙈  I wanted to personally address each issue/PR and they piled up through time, but now I'm checking each one in order.


corner case of simplifyContour

I played with some obj file, and found that the generated navmesh has folding triangles.

The raw contours

The contours

The simplified region edge goes to the left side of the vertex.

Hmm!  This does indeed seem to create some very small folded triangles along the overlapping contour that passes though the polygon.  This could either be an issue with the contour simplification process, a float inaccuracy issue, or both.

There's an assumption that the internal edges between regions can be represented straight line. This is not true in big open areas. I recommend tiles and monotone partitioning for most use cases, that combo behaves better.


qcal: init at 0.9.1

Description of changes
This adds a new package: qcal, a CLI tool for CalDAV Calendars.
Also adds myself as a maintainer.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

should the package contain the default config.json ? program wont start without a config and withotu default it's tough to come up with one.
 Copy config-sample.json from /usr/share/doc/qcal/ to ~/.config/qcal/config.json and modify accordingly

@teto might I ask you to look at this PR again?

I managed to make it work.
Note to upstream: the default config doesn't mention "PasswordCmd" and I had a weird html error with qcal -l when using a wrong caldav error.
I dont know if you are using home-manager (HM) but since recently, HM added accounts for caldav so it would be possible to have hm generate the config for qcal (only khal is supported for now).


⚠ Slack has degraded performance

In df99607, Slack (https://status.slack.com/api/v2.0.0/current) experienced degraded performance:

HTTP code: 200
Response time: 138 ms

Resolved: Slack performance has improved in 4d06aca.


Unsafe pointer from string

Looks like JSIL doesn't support resolving pointer from string. Here is test case:
using System;

static class Program {
    public static void Main () {
        string str = "qww";
        unsafe
        {
            fixed (char* sPtr = str)
            {
                *sPtr = 'a';
            }
        }

        Console.WriteLine(str);
    }
}

Translation result:
    var sPtr = JSIL.UntranslatableInstruction("Implicit conversion to pointer: Untranslatable Expression Conversion of expression '@<var System.String str>' to pointer");
    sPtr.set(("a").charCodeAt(0));
    $T00().WriteLine("qww");

It would be very good, if at least read operations will work.

Looks like it will be pretty easy to implement for read operations. Will do it.
In any case, mutate string through a pointer is not best idea even in .Net.

Yeah, PinAndGetPointer just needs to copy the string's chars into a temporary utf-16 buffer.

Will it be OK if we create Pointer with null as MemoryRange and source string as view (I've already done it)? Or there will be problems if we'd like to cast pointer afterward?

No, that's no good.

What should we do on write through such pointer attempts? Right now I throw NotImplementedException. Or should we allow write to such temporary array (waithout writing it back to source string)?

I would allow the write and not represent it in the source string. It's not possible to mutate a string instance in JS anyway.
While mutating a pinned string in C# might work it's a terrible idea so I'm okay with not implementing it. If it's needed, we could make it work by implementing the unpin portion of fixed blocks and using that to flush the change to the string somehow... but the string's object reference would change, so it's still not gonna work.

Additional notes: get pointer to char[] element doesn't work also.
One more thing to test - cast pointer to byte* or int* (as sizeof(char)=2).
using System;

static class Program {
    public unsafe static void Main(string[] args)
    {
        string str = "qww";
        unsafe
        {
            fixed (char* sPtr = str.ToCharArray())
            {
                RunMe(sPtr, str.Length);
            }
        }

        Console.WriteLine(str);
    }

    private unsafe static void RunMe(char* sPtr, int length)
    {
        char* endPtr = sPtr + length;
        byte* bPtr = (byte*)sPtr;
        while (sPtr < endPtr)
        {
            Console.WriteLine(*sPtr);
            Console.WriteLine(*bPtr);
            Console.WriteLine(*(bPtr + 1));

            sPtr++;
            bPtr += 2;
        }
    }
}

char is tricky since it's a string instance right now. at some point we
should switch them over to ints but the lack of boxing prevents it
currently. same goes for bool.
On 5 October 2015 at 11:34, Igor Kiselev<EMAIL_ADDRESS>wrote:

Additional notes: get pointer to char[] element doesn't work also.
One more thing to test - cast pointer to byte* or int* (as sizeof(char)=2).
using System;
static class Program {
public unsafe static void Main(string[] args)
{
string str = "qww";
unsafe
{
fixed (char* sPtr = str.ToCharArray())
{
RunMe(sPtr, str.Length);
}
}
    Console.WriteLine(str);
}

private unsafe static void RunMe(char* sPtr, int length)
{
    char* endPtr = sPtr + length;
    byte* bPtr = (byte*)sPtr;
    while (sPtr < endPtr)
    {
        Console.WriteLine(*sPtr);
        Console.WriteLine(*bPtr);
        Console.WriteLine(*(bPtr + 1));

        sPtr++;
        bPtr += 2;
    }
}

}
—
Reply to this email directly or view it on GitHub
https://github.com/sq/JSIL/issues/860#issuecomment-145624914.

Could you explain why lack of boxing prevents us from moving chars to Numbers? Right now I don't see why it is so.

char and boolean would lose all type information when cast if unboxed, which poses some difficulties. there's also some other stuff that needs to happen when indexing into strings, etc. i think once boxing/wrapping works the rest can be sorted out.

That is what I do in cil.js and it works great. The type is implicitly known after the unboxing and always provided when boxing occurs.


Urban Jungle - New Character Sheet

Changes / Comments
Use Permission Granted for this.
This is a new character sheet
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[x] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[x] Does this add functional enhancements (new features or extending existing features) ?
[x] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Yes, I'm a discord server with the creator the game.  He gave explicit use to create this char sheet.  If you would like some verbiage from him to this effect, just let me know.

On November 25, 2019 at 8:05 AM "Andreas J."<EMAIL_ADDRESS>wrote:
@Anduh approved this pull request.

    > >

    Use Permission Granted for this.
    What does this mean? Did you reach out to the publisher for a explicit permission to create the sheet?

>

Anyway, a solid submission, with no apparent problems on a quick view.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/Roll20/roll20-character-sheets/pull/5703?email_source=notifications&email_token=AJVRTLAC34YLHY6ZXOC4XEDQVPSTDA5CNFSM4JRJN6W2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCM3JTII#pullrequestreview-322345377 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AJVRTLATR43QETRZ7L4ARE3QVPSTDANCNFSM4JRJN6WQ .

Is a preview a requirement?

The actual jpg file is the background of the char sheet.  Is there a better place to put this?  Is a preview pic mandatory?

The background image is fine where it is. However, in the sheet code the link will not work, you need to point it to the full url of th image where it will be on github.
Example of functional link to background image of sheet.
After you changed the url for the background image, and then go into your test campaign and take a screenshot of the sheet, you place the preview image in the same folder.
Follow the example of any existing sheet

URL is fixed.  Anything else needed?

Now make a preview image of the sheet in Roll20, and place it in the same folder, and make the sheet.json point to it, instead of the background image.

preview.jpg added and css changed to point at it.


[Bug]: mariaDB not supported in newer Ghost versions

Is there an existing issue for this?

[X] I have searched the existing issues

Description
As per their documentation, Ghost 5+ requires MySQL 8+ to run.
The config when setting Ghost up as a service in Coolify now deploys and sets up MariaDB. This means that it would require a migration to MySQL for already deployed services.
The steps to do a migration are written here: https://forum.ghost.org/t/how-to-migrate-from-mariadb-10-to-mysql-8/29575
The interface also shows a warning:

Steps To Reproduce

Deploy Ghost from services
Open the Ghost admin panel
Observe the warning under Settings > About Ghost

Version
3.10.16

It is fixed with the new community based templates.


fix(apps/prod/tekton/configs/triggers/templates): split the tiup global locks between staging and prod

Signed-off-by: wuhuizuo<EMAIL_ADDRESS>

/approve


error after move my project of computer: object http is not a member of package com.twitter.finatra

Hi, I've a weird (for me) error, I had a project in other machine and it was working fine but now I've migrate that code to other machine, I resolve the dependencies but when I try run this I get

 sbt run                                                                                  master
[info] Loading global plugins from /home/yo/.sbt/0.13/plugins
[info] Updating {file:/home/yo/.sbt/0.13/plugins/}global-plugins...
[info] Resolving org.fusesource.jansi#jansi;1.4 ...
[info] Done updating.
[info] Loading project definition from /home/yo/Documentos/proyectos/scala/job_new/project
[info] Updating {file:/home/yo/Documentos/proyectos/scala/job_new/project/}job_new-build...
[info] Resolving org.scala-lang#scalap;2.10.4 ...
[info] Done updating.
[info] Set current project to finatra-seed (in build file:/home/yo/Documentos/proyectos/scala/job_new/)
[warn] Multiple resolvers having different access mechanism configured with same name 'Twitter Maven'. To avoid conflict, Remove duplicate project resolvers (`resolvers`) or rename publishing resolver (`publishTo`).
[info] Updating {file:/home/yo/Documentos/proyectos/scala/job_new/}job_new...
[warn] Multiple dependencies with the same organization/name but different versions. To avoid conflict, pick one version:
[warn]  * com.twitter.finatra:finatra-http_2.11:(2.0.0.M2, 2.1.2)
[info] Resolving org.scala-lang#scalap;2.11.6 ...
[info] Done updating.
[warn] Scala version was updated by one of library dependencies:
[warn] 	* org.scala-lang:scala-library:(2.11.4, 2.11.6, 2.11.2, 2.11.0, 2.11.1) -> 2.11.7
[warn] To force scalaVersion, add the following:
[warn] 	ivyScala := ivyScala.value map { _.copy(overrideScalaVersion = true) }
[warn] There may be incompatibilities among your library dependencies.
[warn] Here are some of the libraries that were evicted:
[warn] 	* com.twitter.finatra:finatra-http_2.11:2.0.0.M2 -> 2.1.2
[warn] 	* com.twitter.inject:inject-core_2.11:2.0.0.M2 -> 2.1.2
[warn] Run 'evicted' to see detailed eviction warnings
[info] Compiling 32 Scala sources to /home/yo/Documentos/proyectos/scala/job_new/target/scala-2.11/classes...
[info] 'compiler-interface' not yet compiled for Scala 2.11.6. Compiling...
[info]   Compilation completed in 36.973 s
[error] /home/yo/Documentos/proyectos/scala/job_new/src/main/scala/com/example/AdminsControllers.scala:4: object http is not a member of package com.twitter.finatra
[error] import com.twitter.finatra.http.Controller
[error]                            ^
[error] /home/yo/Documentos/proyectos/scala/job_new/src/main/scala/com/example/AdminsControllers.scala:17: not found: type Controller
[error] class AdminsControllers extends Controller {
[error]                                 ^
[error] /home/yo/Documentos/proyectos/scala/job_new/src/main/scala/com/example/ExampleServer.scala:8: object http is not a member of package com.twitter.finatra
[error] import com.twitter.finatra.http.response.ResponseBuilder
[error]                            ^
[error] /home/yo/Documentos/proyectos/scala/job_new/src/main/scala/com/example/ExampleServer.scala:20: not found: type ResponseBuilder
[error]   implicit class ResponseWithSession(r: ResponseBuilder#EnrichedResponse) {
[error]                                         ^
[error] /home/yo/Documentos/proyectos/scala/job_new/src/main/scala/com/example/AdminsControllers.scala:18: not found: value get
[error]   get("/wt/hola") { req: Request => response.ok.plain("hola") }
[error]   ^
[error] /home/yo/Documentos/proyectos/scala/job_new/src/main/scala/com/example/AdminsControllers.scala:18: not found: value response
[error]   get("/wt/hola") { req: Request => response.ok.plain("hola") }
[error]                                     ^
[error] /home/yo/Documentos/proyectos/scala/job_new/src/main/scala/com/example/AdminsControllers.scala:20: not found: value get
[error]   get("/wt/login") { req: Request =>
[error]   ^
[error] /home/yo/Documentos/proyectos/scala/job_new/src/main/scala/com/example/AdminsControllers.scala:21: not found: value response
[error]     response.temporaryRedirect.location(Twt.getUrlAdmin)
[error]     ^
[error] /home/yo/Documentos/proyectos/scala/job_new/src/main/scala/com/example/AdminsControllers.scala:25: not found: value get
[error]   get("/mylists") { req: Request =>
[error]   ^

this is my build.sbt

name := """finatra-seed"""
organization := "com.example"

version := "1.0.0-SNAPSHOT"

scalaVersion := "2.11.6"

fork in run := true

resolvers ++= Seq(
  Resolver.sonatypeRepo("releases"),
  "Twitter Maven" at "https://maven.twttr.com",
  "Finatra Repo" at "http://twitter.github.com/finatra"
)

lazy val versions = new {
  val finagle = "6.31.0"
  val finatra = "2.1.2"
  val logback = "1.0.13"
  val mockito = "1.9.5"
  val scalatest = "2.2.3"
  val specs2 = "2.3.12"
}

libraryDependencies += "org.apache.oltu.oauth2" % "org.apache.oltu.oauth2.client" % "latest.milestone"

libraryDependencies += "org.twitter4j" % "twitter4j-core" % "latest.milestone"

libraryDependencies += "org.apache.httpcomponents" % "httpclient" % "4.5"

libraryDependencies += "org.slf4j" % "slf4j-log4j12" % "latest.milestone"

libraryDependencies ++= Seq(
  "ch.qos.logback" % "logback-classic" % versions.logback,
  "ch.qos.logback" % "logback-classic" % versions.logback % "test",

  "com.twitter.finatra" %% "finatra-http" % versions.finatra,
  "com.twitter.finatra" %% "finatra-slf4j" % versions.finatra,

/*  "com.twitter.finatra" %% "finatra-http" % versions.finatra % "test",
  "com.twitter.inject" %% "inject-server" % versions.finatra % "test",
  "com.twitter.inject" %% "inject-app" % versions.finatra % "test",
  "com.twitter.inject" %% "inject-core" % versions.finatra % "test",
  "com.twitter.inject" %% "inject-modules" % versions.finatra % "test",
  "com.twitter.finatra" %% "finatra-http" % versions.finatra % "test" classifier "tests",
  "com.twitter.inject" %% "inject-server" % versions.finatra % "test" classifier "tests",
  "com.twitter.inject" %% "inject-app" % versions.finatra % "test" classifier "tests",
  "com.twitter.inject" %% "inject-core" % versions.finatra % "test" classifier "tests",
  "com.twitter.inject" %% "inject-modules" % versions.finatra % "test" classifier "tests",*/

  "org.mockito" % "mockito-core" % versions.mockito % "test",
  "org.scalatest" %% "scalatest" % versions.scalatest % "test",
  "org.specs2" %% "specs2" % versions.specs2 % "test")

what could be the problem here?..thank you and congratulation for the nice work did with finatra

sorry guys, I didnt notice than running sbt clean doesn't remove the files inside _target folder and after run ack I notice than many of this xml was pointing to an old scala version, I remove this folder, run sbt update and now it works nicely
every day I learn a bit more about scala and sbt :ok_hand: :smile:


replace the underline naming in the code with the hump naming method

What this PR does:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Codecov Report

Merging #235 (e201742) into master (0cb9e30) will not change coverage.
The diff coverage is 93.24%.

:exclamation: Current head e201742 differs from pull request most recent head 31a7a9a. Consider uploading reports for the commit 31a7a9a to get more accurate results

@@           Coverage Diff           @@
##           master     #235   +/-   ##
=======================================
  Coverage   47.57%   47.57%
=======================================
  Files          56       56
  Lines        2806     2806
=======================================
  Hits         1335     1335
  Misses       1407     1407
  Partials       64       64

Impacted Files
Coverage Δ

pkg/protocol/codec/global_report_request_codec.go
90.00% <0.00%> (ø)

pkg/remoting/getty/getty_remoting.go
33.82% <ø> (ø)

pkg/remoting/getty/listener.go
0.00% <0.00%> (ø)

pkg/remoting/getty/readwriter.go
76.61% <66.66%> (ø)

pkg/protocol/message/request_message.go
92.30% <92.30%> (ø)

pkg/protocol/codec/branch_commit_req_codec.go
100.00% <100.00%> (ø)

pkg/protocol/codec/branch_commit_response_codec.go
91.42% <100.00%> (ø)

pkg/protocol/codec/branch_register_req_codec.go
100.00% <100.00%> (ø)

...g/protocol/codec/branch_register_response_codec.go
90.32% <100.00%> (ø)

pkg/protocol/codec/branch_rollback_req_codec.go
100.00% <100.00%> (ø)

... and 22 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


apparent bug in kg_json_to_tsv.py

In kg_json_to_tsv.py, this line of code doesn't seem right:
https://github.com/RTXteam/RTX-KG2/blob/1514437f5b81f1b457458a16d1775e8d081222a5/kg_json_to_tsv.py#L297
The construction not "None", i.e., the predicate for the if statement, always evaluates to False. I think maybe what was intended was:
value = (edge['qualified_predicate'] if edge['qualified_predicate'] is not "None" else edge['source_predicate'])

Line no longer used as of 153748b


Added missing shared Schemes for Carthage

The main workpace didn't have any available scheme that could be compiled by Carthage. I've added them. Now Carthage works! :rocket:

thanks @pepibumur ! :smile: good catch!
there were also some other project / travis-ci issues. i've pushed updates!


arch/x86_64:g_current_regs is only used to determine if we are in irq,

with other functionalities removed.
Change-Id: Ic4bdf0e28c2cc0ec462a7c941f7a7cf2b1b53313
Note: Please adhere to Contributing Guidelines.
Summary
g_current_regs is only used to determine if we are in irq,
with other functionalities removed.
Update this section with information on why change is necessary,
what it exactly does and how, if new feature shows up, provide
references (dependencies, similar problems and solutions), etc.
Impact
Update this section, where applicable, on how change affects users,
build process, hardware, documentation, security, compatibility, etc.
no
Testing
Update this section with details on how did you verify the change,
what Host was used for build (OS, CPU, compiler, ..), what Target was
used for verification (arch, board:config, ..), etc. Providing build
and runtime logs from before and after change is highly appreciated.
ostest

[Experimental Bot, please feedback here]
This PR does not fully meet the NuttX requirements.  While it provides some information, it lacks crucial details and context.  Here's a breakdown:
Missing/Insufficient Information:

Summary: The summary is extremely brief and doesn't adequately explain the why behind the change.  Simply stating that g_current_regs is only used to determine if we're in an IRQ and that other functionalities are removed isn't enough.  What were the other functionalities? Why were they removed? What problem did this solve or what improvement did it create?  What are the benefits of this change?
Impact:  The impact section is almost entirely empty.  While it says "no" seemingly in reference to adding a new feature, it needs to explicitly address all the points: impact on user, build, hardware, documentation, security, compatibility.  Even if the answer is "NO" for each, it should be stated explicitly.  If there is an impact, it needs to be described.
Testing: The testing section is severely lacking. It only mentions "ostest".  This doesn't provide any useful information. What is "ostest"? What are the test results?  It's crucial to provide:

Build Host Details:  Operating System (e.g., Linux Ubuntu 20.04), CPU architecture (e.g., x86_64), Compiler (e.g., GCC 10.2.0).
Target Details:  Architecture (e.g., ARM Cortex-M4), Board and Configuration (e.g., STM32F4 Discovery).
Logs: "Before" and "After" logs demonstrating the change's effect. These logs should be relevant to the functionality being modified and show that the intended behavior is achieved.

Recommendations for Improvement:

Expand the Summary: Explain the rationale behind the change.  Justify the removal of functionalities.  Highlight the benefits (e.g., reduced code size, improved performance, bug fix).  Provide context.
Complete the Impact Section: Explicitly address all impact points, even if the answer is "NO".  If there is an impact, describe it in detail.
Provide Detailed Testing Information:  Specify the build host and target environments completely.  Include relevant "Before" and "After" logs that demonstrate the change's effect and verify that it works as intended.
Consider adding a section on how to reproduce the bug if this is a bug fix.

Example of an Improved Testing Section:
## Testing

I confirm that changes are verified on my local setup and work as intended:

* **Build Host:** Linux Ubuntu 22.04, x86_64, GCC 11.2.0
* **Target:** ARM Cortex-M4, STM32F4 Discovery, `nsh` configuration

**Testing logs before change:**

nsh> some_command  # Command that uses the removed functionality
Segmentation fault

**Testing logs after change:**

nsh> some_command
... (expected output showing the correct behavior without the removed functionality) ...
nsh> irqinfo # showing the irq state is still accessible
... (output showing interrupt information) ...

By providing more detail and following the NuttX PR requirements, the review process will be much smoother, and the changes will be more likely to be accepted.


Crowdsourcing - Annotation

Description
As a user, I would like to contribute to the PIA project by annotating the (digital) objects.
Annotation scenarios:

Annotations can be distinguished by user (human or machine) - for instance colours/shapes/etc - but all annotations are kept, even if it means that they are duplicated.
Annotations can be distinguished by their validation status and can be modified by others (with a log journal), for instance with different colours: red for the ones that are unchecked, green for those that have been assessed.

Wish list:

[ ] Import/export annotations
[ ] Save annotations
[ ] Ability fo search annotations
[ ] Ability to filter annotations
[ ] Various shapes/colours to highlight annotations
[ ] Different annotation formats (textual, audio, video)
[ ] Ability to link annotations between them

Variation(s)
Related to #5 (Crowdsourcing - Enriching the metadata)
Proposed Solutions
Process

Cf. https://miro.com/app/board/o9J_l_66_DQ=/?moveToWidget=3074457361936106910&cot=14 for the updated version
Components/things to keep in mind

[ ] Leveraging Mirador 3 with specific plug-ins
[ ] Deploying an Annotation Store
[ ] Updating IIIF Manifests with associated Annotations
[ ] Deploying a Search Service
[ ] Deploying an Assessment module

Additional Background

Awesome IIIF - Annotations
Documentation Database - Typologie der Partizipation

Mirador 3

Default instance: https://projectmirador.org/ + https://projectmirador.org/embed/?iiif-content=<manifest_url>
With annotations plug-in: https://mirador-annotations.netlify.app/
With layers plug-in: https://mirador-dev.netlify.app/tests/integration/mirador/layers.html
Image Cropper (repository): https://github.com/dbmdz/mirador-imagecropper
Text Overlay: https://mirador-textoverlay.netlify.app/
Physical Ruler (repository): https://github.com/ubleipzig/mirador-ruler-plugin

Mirador 2

DBMZ (Open Source at the Bayerische Staatsbibliothek) Mirador 2 plug-ins: https://github.com/dbmdz/mirador-plugins
Image Cropper (for example from the Bodmer Lab): https://bodmerlab.unige.ch/fr/constellations/autographes/mirador/1072068450?page=002
Châteauroux Demo (virtual reconstruction): https://demos.biblissima.fr/chateauroux/demo/

Per @glenrobson: Madoc, which is based on Omeka-S, could be a suitable approval workflow tool. For versioning of annotations, Memento's Robustifying Links could be leveraged (more information here: http://mementoweb.org/guide/howto/).


Cannot serialized higher order type when type is boxed

This code throws System.ArgumentException: 'Could not determine JSON object type for type Program+MyRecord.'
type MyRecord = {
    Field1 : string
    Field2 : int
    Field3 : bool
}
type Wrapper<'a> = {
    Id:     string
    Config: 'a
}
let myObject = {
    Id = "test test test";
    Config = (box {Field1 = "hi"; Field2 = 2; Field3 = true })
}
let result = JsonConvert.SerializeObject(myObject, Converters.Converter())

If we remove the box, the serialization works but sadly in my case I cannot avoid it.

Hello,
at first, I would have say that boxing the type would prevent us to access the "real" type info. But the error seems to know that it's the type Program+MyRecord that need to be accessed.
Will have to dig more in it.

Thanks!
Line 416 of Encode.fs it call autoEncoder with fi.PropertyType which returns System.Object, which ultimately create this encoder: boxEncoder(fun (v: obj) -> JValue(v) :> JsonValue)
I think this happen because the type of the Config field when we box it is object.
I might have a fix

This indeed looks like the right direction


Remove anyconfig dependency

The primary goal of this PR is to get rid of the anyconfig package dependency. This is what the first commit in this PR does.
In order to make merge function that we created a bit safer to use, we also included the second commit in this PR. In that commit, we made sure merge_dicts function operates on a copy of the first input parameter. If that change is not desired, we can remove the second commit from the PR.
Fixes: #2435

I fixed the pre-commit errors (and properly installed it locally while I was at it).


Initialise separation of core and app

Build new module and move core function to it.

Service
Notifications
UtilClasses

done in merge #60