Cannot build on android after google services update on June 17, 2019

Since June 18, 2019 react-native android build is failing. The following error is given during gradle build:
AGPBI: {"kind":"error","text":"Program type already present: android.support.v4.app.INotificationSideChannel","sources":[{}],"tool":"D8"}
> Task :app:mergeExtDexDebug FAILED
java.lang.RuntimeException: java.lang.RuntimeException: com.android.builder.dexing.DexArchiveMergerException: Error while merging dex archives:
Learn how to resolve the issue at https://developer.android.com/studio/build/dependencies#duplicate_classes.

React Native version:
react-native": "^0.57.3
Some of the libraries we use:
 "react-native": "^0.57.3",
 "react-native-base64": "0.0.2",
 "react-native-camera": "^2.9.0",
 "react-native-contacts": "^3.1.5",
 "react-native-date-ranges": "^2.3.1",
 "react-native-device-info": "^2.1.2",
 "react-native-firebase": "^5.2.1",
 "react-native-keyboard-aware-scroll-view": "^0.8.0",
 "react-native-maps": "^0.23.0",
 "react-native-modal-datetime-picker": "^6.0.0",
 "react-native-modal-dropdown": "^0.6.2",
 "react-native-nfc-manager": "^1.2.2",
 "react-native-phone-input": "^0.2.1",
 "react-native-router-flux": "^4.0.6",
 "react-native-splash-screen": "^3.1.1",
 "react-native-svg": "^9.2.2",
 "react-native-svg-uri": "^1.2.3",
 "react-native-user-inactivity": "^0.1.3-next",
 "react-native-vector-icons": "^6.2.0",
 "react-navigation": "^2.18.2",

Our sdk versions in project level build.gradle:
buildscript {
    ext {
        buildToolsVersion = "27.0.3"
        minSdkVersion = 16
        compileSdkVersion = 28
        targetSdkVersion = 28
        supportLibVersion = "27.1.1"
        googlePlayServicesVersion = "11.8.0"
        androidMapsUtilsVersion = "0.5+"
    }
  ......
}

Dependencies in App level build.gradle:
dependencies {
    implementation(project(':react-native-firebase')) {
        transitive = false
    }
    implementation project(':react-native-device-info')
    implementation project(':react-native-svg')
    implementation(project(':react-native-maps')) {
        exclude group: 'com.google.android.gms', module: 'play-services-base'
        exclude group: 'com.google.android.gms', module: 'play-services-maps'
    }
    implementation fileTree(include: ['*.jar'], dir: 'libs')
    implementation "com.android.support:appcompat-v7:${rootProject.ext.supportLibVersion}"
    implementation 'com.facebook.react:react-native:+'
    implementation 'com.google.firebase:firebase-core:+'
    implementation 'com.google.firebase:firebase-messaging:17.0.0'
    implementation 'com.google.android.gms:play-services-base:16.0.0'
    implementation 'com.google.android.gms:play-services-maps:+'
    implementation 'com.google.android.gms:play-services-gcm:16.0.0'
    implementation project(':react-native-camera')
}

Please suggest how to resolve this issue. We are aware that this maybe caused by google services update, but it's not viable for us to migrate to AndroidX.

Please refer to: https://github.com/facebook/react-native/issues/25293#issuecomment-503045776

dependencies {
implementation "com.google.firebase:firebase-messaging:17.0.0"
implementation("com.google.android.gms:play-services-gcm:16.0.0") {
exclude group: "com.google.android.gms"
}
implementation project(':react-native-device-info'),  {
exclude group: "com.google.android.gms"
}
implementation project(':react-native-exception-handler')
implementation project(':react-native-image-picker')
implementation project(':react-native-image-crop-picker')
implementation project(':react-native-android-location-enabler')
implementation project(':react-native-svg')
implementation project(':react-native-push-notification'), {
exclude group: "com.google.android.gms"
exclude group: "com.google.firebase"
}
implementation project(':@react-native-community_async-storage')
implementation project(':react-native-vector-icons')
implementation project(':react-native-maps')
implementation project(':react-native-gesture-handler')
implementation fileTree(dir: "libs", include: [".jar"])
implementation "com.android.support:appcompat-v7:${rootProject.ext.supportLibVersion}"
implementation "com.facebook.react:react-native:+"  // From node_modules
implementation fileTree(dir: "libs", include: [".jar"])
implementation project(':react-native-navigation')
implementation project(':react-native-contacts')
implementation project(':react-native-linear-gradient')
implementation project(':react-native-geocoder')
implementation(project(':react-native-maps')){
exclude group: 'com.google.android.gms', module: 'play-services-base'
exclude group: 'com.google.android.gms', module: 'play-services-maps'
}
implementation 'com.facebook.fresco:animated-gif:1.10.0'
implementation project(':react-native-image-picker')
}
my dependencies and still have this issue


access: may-start-trybots

See https://golang.org/wiki/GerritAccess
Hi! Could I maybe have Gerrit access for may-start-trybots, https://go-review.googlesource.com/#/admin/groups/1030,members?
It would be very helpful to help manage and test mobile builders referring to discussions in #49048, https://groups.google.com/g/golang-dev/c/oiuIE7qrWp0, and more.
My Gerrit user ID there is 30903, with the Gmail address<EMAIL_ADDRESS>Thank you!
cc @golang/release @dmitshur @steeve

Thank you very much for your contributions to to the Go project, and especially for your help maintaining the mobile builders @changkun! This access is now granted to your Gerrit account.
As a reminder from https://go.dev/wiki/GerritAccess#running-trybots-may-start-trybots, please be sure to at minimum skim CLs before starting a trybot run.
Thank you again!


Add COLLAR logo to Polygon

Adding PolyPup Finance COLLAR token logo

There were a few errors on some other tokens that I fixed in order to make tests pass. 👍


Consider not use OpenSSL to base64 encode

BIO_f_base64 is deprecit in BoringSSL, should we consider use another implementation?

linearize the buffer and use base64 encoder from nghttp2 sounds reasonable

cc @PiotrSikora

From a quick scan I don't think the nghttp2 base64 implementation is exported from the main library. We could potentially copy that code. Not sure if there are better options. Definitely happy to use something other than BIO here.

@tatsuhiro-t any interest in exposing the base64 encoder in the nghttp2 library? :)

Or I can simply just implement one if you think it is fine.

I'm happy with whatever as long as it has tests. @lizan feel free to take this one.

@mattklein123 No plan to expose base64 encoder.  It is MIT licensed, and header only. It is relatively easy to import it to envoy.  Implementing it from scratch is also not hard.

Closed via #306


Proposed new syntax

The current syntax has problems to delete the session instance (aka, it is never really closed). Using with would solve many problems since the __enter__ and __exit__ functions are properly executed. Thus, I am proposing the following change to the scholarly base syntax:
from scholarly import scholarly

with scholarly() as sc:
    tor_sock_port = 9150
    tor_control_port = 9151
    sc.use_tor(tor_sock_port, tor_control_port, "")
    authors = [a for a in sc.search_author('')]

Thoughts?

Makes sense.
In terms of avoiding network issues, I think we should also try to take out from Author and Publication and network calls, and use Author and Publication purely as data containers.
Perhaps we can change the Author.fill() and Publication.fill() functions to be sc.fill_author(Author) and sc.fill_publication(Publication, sections=[...]).
That would make it more explicit in the code when the network connection is being used, and there is no need to keep the connections open when the users do not need to fetch more data, but rather use the downloaded Author and Publication objects.

@silvavn Here is a related proposal:
I propose the Author and Publication objects to be pure data containers, with no parsing logic within them. I gave a shot at https://github.com/scholarly-python-package/scholarly/blob/develop/scholarly/data_types.py to introduce such simple data types, with the main goal being to document the exact semantics of each field, so that we do not confuse the meaning of the various similarly sounding fields.
Then, we can also create two objects AuthorParser, PublicationsParser, with their goal being taking as input DOM object, and returning back the related Author and Publications objects. These classes should not have any network calls in them, just parsing logic. DOM in, Author/Publication out.
Then, the navigator class has two objectives. One is to successfully complete the "getPage(URL)" calls, and another is to provide abstractions for all the necessary URL calls that we need (search for authors, search for publication, search for paper citing a publication, and so on). If we want to be too strict, we can even split that layer in two (the "tor/proxy/getpage" stuff, and then all the abstractions for searching for various URLs)
Let me know what you think.

I agree with that. We should definitely have separate scripts that do the parsing. That way if anything changes its easier to work on just the parser and not the whole navigator script.

Many of these issues were fixed with the ProxyGenerator addition.


LAB_03L01_Configure_Conditional_Access_policies_for_AVD_ADDS requires Entra Trial

Module: 09 Manage security for Azure Virtual Desktop
Lab: LAB_03L01_Configure_Conditional_Access_policies_for_AVD_ADDS
Task: Task 1: Configure Microsoft Entra Premium P2 licensing
Step: 12 	On the Activate blade, click Free trial in the MICROSOFT ENTRA ID P2
Description of issue
Students must use an Entra 30-day trial for this lab.
Students cannot sign up for a Entra Trial without a credit card
We cannot have them using a personal credit card which will start being billed after the trial
Solution: Provision the students with an M365 Tennant with trail licensing as in the MD-102 then
have the students activate The Azure Pass against the Pre-Provisioned M365 Tennant

See #272

Correct - pls refer to #272
rgds
Marcin


Refactor vexml for release

This PR fixes #248.
WIP

@rvilarl and @ ronyeh, just an FYI I'm preparing this library for release. Feel free to any questions or concerns. Happy Holidays!

@rvilarl and @ronyeh, just an FYI I'm preparing this library for release. Feel free to any questions or concerns. Happy Holidays!

Awesome! Next month we will try to get VexFlow 5 shipped for you so you don't have to depend on a beta/alpha release.
Let us know if there are any remaining issues you are running into with VexFlow.
Happy New Year!

Happy New Year!


Implement stupidsort() and updateSortVal(). Closes #132, #122, #50, #123

Would appreciate some review from the other contributors if you have time. I'll leave this up for a couple days before I merge it in.

Going to write the issue numbers here again so people participating in those discussions are informed.
Closes #132, #122, #50, #123

Gonna go ahead and go for it. I've tested the changes fairly thoroughly to ensure they are backwards compatible.

:hamster: Awesome!


Add LeetCode Solution

It contains the solution of the LeetCode Question no.347.
I have also added the test under the tests folder.
Fixes #58

@adityaarakeri Can you please review my pull request so that I can make required changes if required.Thank You.


Is nc_inq_var_fill() returning the correct answer for parameter no_fill for classic file?

I run this code to create a netcdf file with 1 dim, 1 var, and I write 1 value:
#define NDIM1 1
#define DIM_LEN 4
#define DIM_NAME "my_dim"
#define VAR_NAME "my_var"

       int ncid;
        int dimid;
        int varid;
        int cmode = 0;
        char testfile[] = "test.nc";
        size_t index = 2;
        int test_val = 42;
        int no_fill;
        int ret;

        if ((ret = nc_create(testfile, cmode, &ncid)))
            return ret;
        if ((ret = nc_def_dim(ncid, DIM_NAME, DIM_LEN, &dimid)))
            return ret;
        if ((ret = nc_def_var(ncid, VAR_NAME, NC_INT, NDIM1, &dimid, &varid)))
            return ret;
        if ((ret = nc_enddef(ncid)))
            return ret;
        if ((ret = nc_put_var1_int(ncid, varid, &index, &test_val)))
            return ret;
        if ((ret = nc_inq_var_fill(ncid, varid, &no_fill, NULL)))
            return ret;
        printf("in netcdf test no_fill = %d\n", no_fill);
        if ((ret = nc_close(ncid)))
            return ret;

As expected I get a file like this:
netcdf test {
dimensions:
	my_dim = 4 ;
variables:
	int my_var(my_dim) ;
data:

 my_var = _, _, 42, _ ;
}

But my print statement is telling me that no_fill is set for this variable:
in netcdf test no_fill = 1
Since I have not changed the fill mode, I expect this value to be 0, indicating that fill mode is on, which is the default. And indeed it does seem to be on. But nc_inq_var_fill is telling me that it is not.
Am I missing something?
(Note I do get the correct answer when I change cmode to NC_NETCDF4.)

You don't seem to be missing anything; I'll take a look at this.

I guess nc_inq_var_fill() was initially developed for NetCDF-4 files only (based on an older netCDF C Interface Guide 4.1.3 doc for nc_inq_var_fill). Look like the latest C Interface Guide no longer requires netCDF-4 (latest doc for nc_inq_var_fill). I also see no reason not to make it available for classic files.
For classic files, I traced the cause to line 202 of file libsrc/nc3dispatch.c which always sets the returned fill mode to 1. However, the fix may not be simple, because per variable fill mode is not saved in NC_var, a C struct defined in nc3internal.h that stores a variable's metadata.
Similar setting is used in PnetCDF dispatcher, line 1163 of libsrcp/ncpdispatch.c. But because PnetCDF has implemented API ncmpi_inq_var_fill since 1.6.1, the fix can be just to change line 1163 to the followings.
status = ncmpi_inq_var_fill(nc->int_ncid, varid, no_fill, fill_valuep);
if (status != NC_NOERR) return status;

WRT the per-variable fill mode in classic files, I understand that it is not controllable.
But the no_fill value returned by nc_inq_var_fill() should reflect the correct fill setting for the file. So if it is not changed for the file, I would expect no_fill == 0 for any variable in the file.

I just tested this, and, as I expected, since @wkliao's changes, this now works properly. I have a test that confirms this, I will put up a PR and then this ticket can be closed.


LoadStoneNightly causes issues with LCSoundTool installed

Hello!
I noticed that LoadStoneNightly breaks the entire game with LCSoundTool installed.
I also figured out why:
LoadStoneNightly tries to load an audio file from the directory AdiBTW-Loadstone.
This fails as the directory is called AdiBTW-LoadstoneNightly.
This always breaks the game - What you configured in the config doesn't matter

Yeah, I realized this last night. The automatic renaming I set up was a bit bugged because it didn't run early enough. I've since fixed that, but I haven't uploaded the changes yet. I'll upload a new version soon

Fixed in v0.1.8


How to only trigger mention if there is a space before the "@" sign

Currently, the behaviour appears to be that if a user types @andrea - the package automatically turns it into a mention.
However, this means that if a user types<EMAIL_ADDRESS>then it also is displayed as a mention. I do not want it to be considered a mention if there is not a space before the @ sign.
How to make it so that a mention is triggered by " @" not simply "@"?
Thank you very much

@andrea-h10 This is a bug, I will add a fix in some time.

is this fixed?


[NBS, Filestore] Decrease min number of channels and support used channel count "shrinkage" for the existing disks

Right now we allocate and use not less than 4 data channels for each partition tablet: https://github.com/ydb-platform/nbs/blob/3d5451f61b1fa9ad902f17a90617052aafc2eef7/cloud/blockstore/libs/storage/core/config.cpp#L313
This value was set a long time ago when we didn't have the autoreassign mechanism and wanted to decrease the chance of consuming all available space on the bsgroups under the data channels of some tablet. So now we have a minimum of 7 channels for each disk:

system (0)
index (1)
data (3-6)
fresh (7)
// channel 2 is legacy and is not used
Suppose a random bsgroup breaks. Suppose all channels use different bsgroups. Then the probability of becoming unavailable is something like 1 - (1 - 1/bsgroup_count)^7
If we had only 1 data channel, this probability would be 1 - (1 - 1/bsgroup_count)^4. If we have, let's say, 100 bsgroups, these probabilities are 0.068 vs 0.039. So decreasing MinChannels to 1 seems reasonable. And we also want to stop using the channels that have become "unneeded" in the tablets of the existing disks.

The same thing is also applicable to filestore.

Significantly increasing allocation unit size would also make sense. Right now an 8TiB filesystem or disk occupies 255 blobstorage groups which span hundreds of nodes. A single group failure will cause failure of most of the large filesystems / disks in a 1000 node cluster. And if we talk about multitablet 100+TiB filesystems, such filesystems are almost guaranteed to become unavailable if a single blobstorage group fails.


Does it support reply from origin server like SRS hstrs

Does it support reply from origin server like SRS hstrs

Yes, nginx-rtmp-module supports similar feature via pull & push directive.

May i know how can use it ?

BTW, do you have QQ or wechat

You may refer to pull & push directives in nginx-rtmp-module's wiki and you can contact me via email.


Pin icon missing from pinned memos

Describe the bug
Pinned memos used to be denoted by a pin icon. That icon is missing in the latest update.
Steps to reproduce

Create a pinned memo

Screenshots or additional context
No response

It will be released in v0.16.0.


How to get data object on click ?

Hello thanks for great plugin. I am looking for the way how to get data objects when user clicked on each cycles.
example: onclick="myfunc(data)"
func myfunc(data) {
console.log('data here', data)
}
thanks for your helps
thanks

I think you're looking something like this (assuming the diagram is in a div with id=venn)
d3.selectAll("#venn .venn-circle")
    .on("click", function(data) {
        console.log(data);
});

Hi Thanks for your replied. I will try it and let's you know


Fix failing etcd volume attach when upgrading etcd instances

PROBLEM
Running a kube-aws upgrade on a working cluster fails to properly restart etcd instances, causing a (again failing on the same issue) UPDATE_ROLLBACK of the CloudFormation stack.
CAUSE
When replacing etcd instances, the mounted volume is detached from the old instance and attached to the new one. This however can take a few minutes. Systemd service cfn-etcd-environment.service fails when the volume is not yet available and restarts itself.
Unfortunately depending services are failing because of systemd not capable of waiting for restarting failed services! (see snippet below).
After a while the attaching of the volume succeeds, but the depending services are not executed and the instance is not working properly.
May 04 15:00:48 ip-10-55-21-187.eu-central-1.compute.internal systemd[1]: cfn-etcd-environment.service: Control process exited, code=exited status=1
May 04 15:00:48 ip-10-55-21-187.eu-central-1.compute.internal systemd[1]: cfn-etcd-environment.service: Failed with result 'exit-code'.
May 04 15:00:48 ip-10-55-21-187.eu-central-1.compute.internal systemd[1]: Failed to start Configures EBS volume and R53 record set for this node and derives env
May 04 15:00:48 ip-10-55-21-187.eu-central-1.compute.internal systemd[1]: Dependency failed for Formats etcd2 ebs volume.
May 04 15:00:48 ip-10-55-21-187.eu-central-1.compute.internal systemd[1]: Dependency failed for /var/lib/etcd2.
May 04 15:00:48 ip-10-55-21-187.eu-central-1.compute.internal systemd[1]: Dependency failed for etcd (System Application Container).
May 04 15:00:48 ip-10-55-21-187.eu-central-1.compute.internal systemd[1]: Dependency failed for etcdadm update status.
May 04 15:00:48 ip-10-55-21-187.eu-central-1.compute.internal systemd[1]: etcdadm-update-status.service: Job etcdadm-update-status.service/start failed with res
May 04 15:00:48 ip-10-55-21-187.eu-central-1.compute.internal systemd[1]: Dependency failed for etcdadm reconfigure runner.
May 04 15:00:48 ip-10-55-21-187.eu-central-1.compute.internal systemd[1]: etcdadm-reconfigure.service: Job etcdadm-reconfigure.service/start failed with result
May 04 15:00:48 ip-10-55-21-187.eu-central-1.compute.internal systemd[1]: etcd-member.service: Job etcd-member.service/start failed with result 'dependency'.
May 04 15:00:48 ip-10-55-21-187.eu-central-1.compute.internal systemd[1]: var-lib-etcd2.mount: Job var-lib-etcd2.mount/start failed with result 'dependency'.
May 04 15:00:48 ip-10-55-21-187.eu-central-1.compute.internal systemd[1]: format-etcd2-volume.service: Job format-etcd2-volume.service/start failed with result

SOLUTION
Don't fail service cfn-etcd-environment.service when a volume cannot be attached (yet), but retry in a loop.

@Confushion Hey, thanks for your contribution as always!
Unfortunately this has never happened to me, but the fix does make sense. Probably I was too lucky.
LGTM.


Questions in V4 ?

Environment:

Node.js Version: 12.14.0
Redis Server Version: <!-- e.g. "redis-server --version" -->
Node Redis Version: 4.0.0
Platform: Mac os

Question 1
Use typescript ,The type of the instance was not found ?
const client: any = redis.createClient({ ... })

Question 2
tsconfig target is set es5 , use  tsc compile.
However, there is a #private definition in the type definition that is not available for es5

To get the correct type on the client instance just remove the : any:
const client = redis.createClient({ ... });

Regarding the tsconfig.target - we are using @tsconfig/node12 as the base tsconfig, which set the target to es2019. Why are you using es5 with node 12.14.0?

Or if you really need a type, use this import and use the type for your client:
import { RedisClientType } from '@node-redis/client/dist/lib/client';


DDL generation's "drop" statement contains schema name when "Use fully qualified names" is unchecked

could be related to https://github.com/dbeaver/dbeaver/issues/3164
System information:

Operating system (distribution) and version: mac os
DBeaver version : 6.1.0 community
Java version
Additional extensions

Connection specification:

Database name and version: postgres 11.2
Driver name
Do you use tunnels or proxies (SSH, SOCKS, etc)?

Describe the problem you're observing:

Steps to reproduce, if exist:
In DB Navigator right click a table --> generate SQL --> DDL --> check/uncheck use fully qualified names - makes no difference on the commented drop statement - it's always with the schema name.

Thanks for the bug report

verified

I am experiencing this but only with the create portion. The drop portion appears to be working correctly.


[Android] Launch app from notification

Is it possible to have a notification launch the app if it's dead/killed? If it's possible what is the correct way to do it? I currently have the event callbacks setup and working when the app is in the foreground/background. I'm using the latest version react-native-notifications.
Thanks.

Just use firebase's messaging().onNotificationOpenedApp()

Just use firebase's messaging().onNotificationOpenedApp()


How To Use IPv6 in Postal ???

Hello Team,
I am trying to connect sending emails while using IPv6 but it cannot send mail using IPv6.
It continuously sending mail to my host IP and I am also bind the IPv6 in the network file but it cannot use IPv6 if i am trying to sending mails so it shows the mails in pending .
And i want to send Emails using IPv6 .
Is there possible or not??
thanks in advance

It should be possible if your server has a working IPv6 connection to the internet and the receiving servers are also IPv6 enabled. Can you share screenshots of what you're seeing?

@zeba121 Is your server on Digital Ocean? https://ideas.digitalocean.com/ideas/DO-I-1366

Hello @willpower232
Sorry For the late reply.
Please check the following link:-
https://pastebin.com/usa3h5XF
Thank You

Hello @timraybould
My server is not on a Digital Ocean.

Hello @willpower232
ipv6 is pinging but when i am sending a mail through postal but they landing in pending or not sent.
please see this:-
root@vpsXXXXX:~# ping6 -c5 XXXX:XX00:XXX:X000::XXX
PING XXXX:XX00:XXX:X000::XXX(XXXX:XX00:XXX:X000::XXX) 56 data bytes
64 bytes from XXXX:XX00:XXX:X000::XXX: icmp_seq=1 ttl=64 time=0.071 ms
64 bytes from XXXX:XX00:XXX:X000::XXX: icmp_seq=2 ttl=64 time=0.055 ms
64 bytes from XXXX:XX00:XXX:X000::XXX: icmp_seq=3 ttl=64 time=0.057 ms
64 bytes from XXXX:XX00:XXX:X000::XXX: icmp_seq=4 ttl=64 time=0.064 ms
64 bytes from XXXX:XX00:XXX:X000::XXX: icmp_seq=5 ttl=64 time=0.075 ms
--- 2607:5300:201:3000::391 ping statistics ---
5 packets transmitted, 5 received, 0% packet loss, time 3997ms
rtt min/avg/max/mdev = 0.055/0.064/0.075/0.010 ms
Please response me as soon as possible.

If your messages are still pending, have you checked postal status to guarantee that everything is running as it should? If you were experiencing network problems, the messages would have failed rather than still being pending.

Hello @willpower232
See my postal status:-
root@vps121528:~# postal status
Procodile Version   1.0.23
Application Root    /opt/postal/app
Supervisor PID      24737
Started             2019-07-15 05:35:45 -0400
|| web
|| Quantity            1
|| Command             bundle exec puma -C config/puma.rb
|| Respawning          5 every 3600 seconds
|| Restart mode        usr1
|| Log path            none specified
|| Address/Port        none
|| => web.1            Running      05:35        pid:24743    respawns:0      port:-        tag:-
|| worker
|| Quantity            4
|| Command             bundle exec ruby script/worker.rb
|| Respawning          5 every 3600 seconds
|| Restart mode        start-term
|| Log path            none specified
|| Address/Port        none
|| => worker.1         Running      05:35        pid:24745    respawns:0      port:-        tag:-
|| => worker.2         Running      05:35        pid:24747    respawns:0      port:-        tag:-
|| => worker.3         Running      05:35        pid:24750    respawns:0      port:-        tag:-
|| => worker.4         Running      05:35        pid:24753    respawns:0      port:-        tag:-
|| cron
|| Quantity            1
|| Command             bundle exec rake postal:cron
|| Respawning          5 every 3600 seconds
|| Restart mode        term-start
|| Log path            none specified
|| Address/Port        none
|| => cron.1           Running      05:35        pid:24756    respawns:0      port:-        tag:-
|| smtp
|| Quantity            1
|| Command             bundle exec rake postal:smtp_server
|| Respawning          5 every 3600 seconds
|| Restart mode        usr1
|| Log path            none specified
|| Address/Port        none
|| => smtp.1           Running      05:35        pid:24758    respawns:0      port:-        tag:-
|| requeuer
|| Quantity            1
|| Command             bundle exec rake postal:requeuer
|| Respawning          5 every 3600 seconds
|| Restart mode        term-start
|| Log path            none specified
|| Address/Port        none
|| => requeuer.1       Running      05:35        pid:24762    respawns:0      port:-        tag:-

Looks like everything is running fine so its possible you were previously having some problems and the messages got locked by another process which isn't around any more, have a look at https://github.com/atech/postal/issues/287#issuecomment-367293435

Hello @willpower232
when we assigned ipv6 address then emails are pending in a mail queue.
But we assigned Ipv4 address emails are sent properly.

hello @willpower232
its done thanks for help.

It might be worth googling that particular message, I know a couple of others have had similar messages (#468 and #847)


$rootScope.$apply in $timeout to prevent 'digest already in progress...

Sometimes i get 'digest already in progress' so i wrapped the call in $timeout().
(This is my first pull-request.)

tests doesn't run correctly, but this change should not be the cause.

Hi, thanks for the PR, anyhow there are a few things that are not clear:

1/ it is very important to have all tests passing
2/ With every bug fixed, there should be a new test created/modified that
shows the issue
3/ The PR is replacing the code for success with the code for rejection
4/ $timeout without a false as a 3rd parameter, already calls apply
5/ Unless someone fiddles with script.onload (or script.on*) then I cannot
see how the extra apply happens not how this PR solves this

Would it be possible to get an updated PR with these issues resolved? An
alternative would be to just create a plunker that shows the error (even if
this error only show in the console)

I completly failed with my first push request. Tests are not successfull because the copy n paste error (your point 3/) - when i fix that tests are running fine.
Your point 5/ pointed me to a problem that perhaps is the cause for my local (not in tmhdynamiclocale) problem with 'digest already in progress'.
Thanks for your time, i learned something!


⚠ ZV800.com has degraded performance

In 3a1fd01, ZV800.com (https://www.zv800.com//) experienced degraded performance:

HTTP code: 200
Response time: 1110 ms

Resolved: ZV800.com performance has improved in 81b3b9e after 11 minutes.


Fix Cluster Creation issue - AZ's mapped wrong

before:

# results in:
"", "eu-west-1a", "eu-west-1b"

after:
availabilityZones: ["${AZS[1]}", "${AZS[2]}", "${AZS[3]}"]

# actually should be:
"eu-west-1a", "eu-west-1b", "eu-west-1c"

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Now sure how you encounter this, but array index start at 0 and I see no problem with the current command :
Admin:~/environment $ export AZS=($(aws ec2 describe-availability-zones --query 'AvailabilityZones[].ZoneName' --output text --region $AWS_REGION))
Admin:~/environment $ echo ${AZS[0]} ${AZS[1]} ${AZS[2]}
eu-west-1a eu-west-1b eu-west-1c
Admin:~/environment $ echo "export AZS=(${AZS[@]})" | tee -a ~/.bash_profile
export AZS=(eu-west-1a eu-west-1b eu-west-1c)
-- Restart terminal --
Admin:~/environment $ echo ${AZS[0]} ${AZS[1]} ${AZS[2]}
eu-west-1a eu-west-1b eu-west-1c


List of commands

The documentation isn't explicit about not accepting a list of commands so I assumed it would work, as it does with non-Docker builds. Maybe a feature request? It should help with readability.
Tested with buildkite-agent 3.2.0, docker plugin 1.3.0
pipeline.yml:
steps:
  - command:
      - "apk add --no-cache python make gcc"
      - "npm install"
      - "npm test"
    plugins:
      docker:
        image: "node:8-alpine"
        always-pull: true

Error:
~~~ Setting up plugins
# Plugin "github.com/buildkite-plugins/docker-buildkite-plugin" already checked out (5773f6a)
~~~ Preparing working directory
$ cd /home/buildkite-agent-4/builds/hostname/fluid-project/kettle
$ git remote set-url origin https://github.com/fluid-project/kettle.git
$ git clean -fqd
# Fetch and checkout pull request head from GitHub
$ git fetch -v origin refs/pull/43/head
From https://github.com/fluid-project/kettle
 * branch            refs/pull/43/head -> FETCH_HEAD
# FETCH_HEAD is now `30cb8b81e13b5ae7ea936edb5f32d5475d9d8c51`
$ git checkout -f 30cb8b81e13b5ae7ea936edb5f32d5475d9d8c51
HEAD is now at 30cb8b8... KETTLE-70 - Switch to Alpine image
# Cleaning again to catch any post-checkout changes
$ git clean -fqd
# Checking to see if Git data needs to be sent to Buildkite
$ buildkite-agent meta-data exists buildkite:git:commit
~~~ Running plugin github.com/buildkite-plugins/docker-buildkite-plugin command hook
$ /etc/buildkite-agent/plugins/github-com-buildkite-plugins-docker-buildkite-plugin/hooks/command
--- :docker: Pulling node:8-alpine
8-alpine: Pulling from library/node
605ce1bd3f31: Already exists
45d835fcee17: Pull complete
f2b4bf87c277: Pull complete
Digest: sha256:01e564c3a2f1c29b62aeafd21611712f9973b953ad9f595b9e112260160e1ae7
Status: Downloaded newer image for node:8-alpine
--- :docker: Running apk add --no-cache python make gcc
npm install
npm test in node:8-alpine
docker: Error response from daemon: OCI runtime create failed: container_linux.go:348: starting container process caused "exec: \"bash\": executable file not found in $PATH": unknown.
🚨 Error: The command exited with status 127
^^^ +++
^^^ +++

Thank you!

@gtirloni sorry about this, we're planning to improve the error messages. The issue here is that we invoke bash to execute commands so that we can support multi-lines, your container doesn't have bash, which is the error you are getting:
docker: Error response from daemon: OCI runtime create failed: container_linux.go:348: starting container process caused "exec: \"bash\": executable file not found in $PATH": unknown.

Ideally we should be using /bin/sh, which would work in your alpine container.

@lox thank you!

You can use the new shell option to set it to /bin/sh -e -c if you like


Feature: TestNG Reporting, how to quickly identify Retry-d Tests?

TestNG Version

6.11

Expected behavior
Hello: When a @Test fails and a Retry happens, is there a direct way in TestNG to distinguish between a Test that has been Retried and a test that has not been retried - such as through a NG context method or variable?

Case 1: If a Test not been retried then the 1st test is a test that has not been retried.
Case 2: If a Test that has been retried 2 times then the 3rd test is a test that has not been retried.

An option to do this: http://www.seleniumeasy.com/testng-tutorials/retry-listener-failed-tests-count-update.
Actual behavior
Currently, there is currently no easy way to determine a retried test.
Reference: http://stackoverflow.com/questions/43069150
Is the issue reproductible on runner?

[ ] Shell
[ ] Maven
[ ] Gradle
[ ] Ant
[x] Eclipse
[ ] IntelliJ
[ ] NetBeans

Test case sample

Please, share the test case (as small as possible) which shows the issue

Thanks for the report! :)

@bipo1980 - You should be able to do this very easily today. The idea is to basically query the retry analyzer instance associated with a test and check its retry count. If you found a value greater than zero then that method was retried. Here's a sample that shows what I am talking about
package com.rationaleemotions.stackoverflow;

import org.testng.Assert;
import org.testng.IRetryAnalyzer;
import org.testng.ITestContext;
import org.testng.ITestResult;
import org.testng.TestListenerAdapter;
import org.testng.annotations.Listeners;
import org.testng.annotations.Test;

import java.util.HashSet;
import java.util.Set;
import java.util.concurrent.atomic.AtomicInteger;

@Listeners(IdentifyRetriedTests.IdentifyingListener.class)
public class IdentifyRetriedTests {
    private static AtomicInteger counter = new AtomicInteger(0);
    private static AtomicInteger anotherCounter = new AtomicInteger(0);

    @Test(retryAnalyzer = Retry.class)
    public void hello() {
        if (counter.incrementAndGet() == 3) {
            Assert.assertTrue(true);
        } else {
            Assert.fail();
        }
    }

    @Test(retryAnalyzer = Retry.class)
    public void anotherHello() {
        if (anotherCounter.incrementAndGet() == 5) {
            Assert.assertTrue(true);
        } else {
            Assert.fail();
        }
    }

    @Test(retryAnalyzer = Retry.class)
    public void bye() {
        Assert.assertTrue(true);
    }

    public static class IdentifyingListener extends TestListenerAdapter {

        @Override
        public void onTestSuccess(ITestResult tr) {
            IRetryAnalyzer retry = tr.getMethod().getRetryAnalyzer();
            if (retry instanceof Retry) {
                //Check if the retry analyser's retry count was greater than zero. If yes, then its a retried method
                if (((Retry) retry).getRetryCount() > 0) {
                    tr.setAttribute("retried", true);
                }
            }
        }

        @Override
        public void onFinish(ITestContext testContext) {
            Set<ITestResult> results = new HashSet<>();
            results.addAll(testContext.getFailedTests().getAllResults());
            results.addAll(testContext.getPassedTests().getAllResults());
            for (ITestResult result : results) {
                if (result.getAttribute("retried") != null) {
                    System.err.println(result.getMethod().getMethodName() + "() was retried");
                }
            }
        }
    }

    public static class Retry implements IRetryAnalyzer {
        private AtomicInteger retryCount = new AtomicInteger(0);
        private static int maxRetryCount = 10;

        public boolean retry(ITestResult result) {
            return retryCount.getAndIncrement() < maxRetryCount;
        }

        int getRetryCount() {
            return retryCount.get();
        }
    }
}

@krmahadevan while this would work, it would be better to have this in the emailable html report itself. Probably a different section for retried tests and an updated summary table that shows the pass and fail count based on final results.

@AJ-72 - Sure. Would you be willing to help submit a pull request for this change ? We should be able to help vet it out and get it merged for you.

@krmahadevan Yeah am checking that emailable report. Is it emailablereport2 that I should check?

@AJ-72 - Currently its based on a condition. See here
So you may have to check both.


Add API for auto-wrapping a module as an imperative board

I want to take a slightly different approach that I think will have the same benefit (and more!): augment our "node type as URL" machinery to allow specifying modules. Basically, if the node type is of the form "#module:[moduleName], the node behaves as runModule with the $module set to moduleName.


1.20.5/Component support

Are the developers planning to update this mod to 1.20.5 and add support for the new component system?
If you are reading this and not a developer, you should expect this issue to take longer to resolve than other updates because of the changes to data format and components in 1.20.5.

It is planned but it will require a complete rewrite of the mod, so don't expect anything soon

I am really looking forward to updating this mod to version 1.20.6! I hope it will be released soon

Should we consider changing the name by then? qwq

I really want this plz


Suggest to keep a LICENSE file with the repository.

I think we should at least suggest the maintainers to keep a LICENSE file with the repository, this way the repository is more portable - if anyone would copy the repo from outside of github, they would still have the license.
This is all to make it easier for customers to use the repositories.

Hej Johan,
Good idea, I completely agree. In fact, I'm completely for making it an explicit requirement, listed under the "Requirements on your project" header, say right before / after saying that a README file is required.

Yes @sstrollo , I would also like to have it required, how about:
### LICENSE
Add a LICENSE file, we suggest using Apache 2.0 license. This makes it clear for anyone how the code can be used, modified and distributed.


Legend for spike amplitude view

Would it be possible to show a legend for the units displayed in the spike amplitude view? I recall such a legend from qt-mountainview. Could simply display the colored dot and the unitID. A legend overlapping the plot could block view of the data, so perhaps a legend could fit underneath the x axis, to the right of the time range display - where I think there is currently some unused space. If more than a handful of units are displayed in the same spike amplitude view, the legend may become too large and may interfere with the plot itself - perhaps then it could just go away (that is, only display the legend when, say, <10 units are selected or something). Not especially high priority, but will help make the spike amplitude view more readable when plotting multiple units at a time. Will also help user identify times when two units are assigned the same color marker in the plot. Thanks!

Great suggestion. Putting as medium priority.


Make showCharacterPalette() public

Can we make showCharacterPalette() in KeyEventHelpers.desktop.kt public? We need to use it in or code but it's currently set to internal. Depending on org.jetbrains.skiko.orderEmojiAndSymbolsPopup from Skia directly would make our JARs OS-dependent, which we can't work with.

@igordmn WDYT?

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


[firestore-translate-text] Array elements in an input map are translated as null

Extension name: firestore-translate-text
Extension version: 0.1.6
Configuration values (redact info where appropriate):

target_languages: en,es,de,fr
collection_path: some_collection
input_field: input
output: translated

Steps to reproduce:
Create a document in some_collection and add the following property input:
"input": {
  "first": [
    "Tree",
    "House"
  ],
  "second": "Hello"
}

Expected result
"input": {
  "first": [
    "Tree",
    "House"
  ],
  "second": "Hello"
},
"translated": {
  "first": [
    {
      "en": "Tree",
      "de": "Baum",
    },
    {
      "en": "House",
      "de": "Haus",
    },
  ],
  "second": {
      "en": "Hello",
      "de": "Hallo",
  }
}

Actual result
"input": {
  "first": [
    "Tree",
    "House"
  ],
  "second": "Hello"
},
"translated": {
  "first": {
    "en": null,
    "de": null,
  },
  "second": {
      "en": "Hello",
      "de": "Hallo",
  }
}

Thanks @CiriousJoker. I've added this for investigation.

Nesting maps in this way doesn't work either (same result).
Thanks for considering it.

Hi @CiriousJoker, the extension doesn't support iteration of nested objects like this, it's either a single value:
"input": "hello",
"output: {
  "en": "Hello",
  "de": "Hallo",
},

or multiple
"input": {
  "first": "hello",
  "second: "...",
},
"output: {
  "first": {
    "en": "Hello",
    "de": "Hallo",
  },
  "second": {
    ...
  },
},


fixes typo on homepage

Check List

[ x] Read OSS Contribution Guide
[ x] All commits are signed

This repository enables Require signed commits, so all commits must be signed

[ x] Avoid force push
[ x] Do only one thing in one Pull Request

fixes #1142

Thank you for your contribution!


fix: generate declaration files d.ts

Related #968
Now that ioredis is in TypeScript, we can set allowJs to false, let tsc generates declaration files d.ts and publish it to npm.
after:

Thanks for the pull request! Declarations are not generated and published to npm is because there are many methods (like redis.get() / redis.mget()) are generated dynamically instead of being defined statically. So the automatically generated declarations will not contain those methods, which, I'm afraid, may mislead many developers.
Currently, they may keep using @types/ioredis before the above issue is solved.

Ah ok. Just 1 more question, did you mean "dynamically generated" as using in using https://www.npmjs.com/package/redis-commands to generate it?

the use of generics would help with this


Allow Users to Sync Older Data

Why do we need this ?

The current ETL mechanism uses DEFAULT_SYNC_DAYS var stored in the env of the image to sync data for the past DEFAULT_SYNC_DAYS at first time data sync.
The use can update the DEFAULT_SYNC_DAYS in env in dev mode and run the server to sync past data but there is no mechanism to do so in the prod image.

Acceptance Criteria

[ ] Add Settings for DEFAULT_SYNC_DAYS. Add enum, getter and setter.
[ ] Update DEFAULT_SYNC_DAYS usage throughout the code base to use setting instead of env variable.
[ ] Add BookmarkService: Add a core boomark service to fetch, set and reset all bookmarks
[ ] Update usage of BookmarkService throughout the codebase
[ ] Update BookmarkService.reset_bookmarks in DEFAULT_SYNC_DAYS setter to reset all bookmarks if default sync days is update or increased.
[ ] Add UI for updating DEFAULT_SYNC_DAYS Setting.

Further Comments / References
Work Breakdown:

Add BookmarkService
- get_bookmark(entity_id, entity_type, provider)->datetime
- set_bookmark(entity_id, entity_type, provider, bookmark_timestamp)->datetime
- reset_org_bookmarks(org_id)
- _get_repo_bookmark(repo_id)
- _set_repo_bookmark(repo_id, datetime)
- _get_workflow_bookmark(repo_id)
- _set_workflow_bookmark(repo_id, datetime)
- _get_incident_bookmark(service_id, provider, bookmark_type )
- _set_incident_bookmark(service_id, provider, bookmark_type, datetime)
- _get_merge_to_deploy_bookmark(repo_id)
- _set_merge_to_deploy_bookmark(repo_id, datetime)
- _reset_repo_bookmarks(repo_id, datetime)
- _reset_workflow_bookmarks(id, datetime)
- _reset_incident_bookmarks(service_id, provider, bookmark_type, datetime)
- _reset_merge_to_deploy_bookmarks(id, datetime
Update usages of all getters and setters for bookmarks across the codebase
Add a new setting for default sync days
Update sync days usage across the code base to use setting instead of env vars
Update setter for DEFAULT_SYNC_DAYS_SETTING to call reset_org_bookmarks
Update setter for DEFAULT_SYNC_DAYS_SETTING to call reset_org_bookmarks
Add UI for updating Default sync days setting

I want to work on this issue.

@adnanhashmi09 Are you working on this issue? I have also been working on this 🥲.

Hey @Kamlesh72  apologies, it would be ideal to get issue assigned before starting work. Been there, done that. Still remember I spent a whole week to open a PR to RocketChat that was closed by one of their engineers midway :")

We can place the "Update Sync Days" button inside "Settings" button of Dora Metrics page.
Is this ok? @samad-yar-khan

That could work and open a modal @jayantbh / @e-for-eshaan  can hint better at the UI

That could work and open a modal @jayantbh / @e-for-eshaan  can hint better at the UI

Because this isn't strictly a Dora related setting, I recommend creating a new page for this, perhaps as the /settings path.
It won't have much today, but eventually it can have more things.

As of this comment the backend work for this has been done. In above PRs.
Additional context for UI and BFF:
Our sync depends on the default sync days value and if someone increases the default sync days setting, we should sync older data respecting the new default sync days for existing and prev data. The original plan was to call bookmark service from settings to reset the bookmark if the default sync days setting is updated. This side-effect would have been convenient  but magic. So we decided to make an API in #514 and whenever the UI calls to update the default sync days setting, it should also sequentially call reset bookmark api. If the second api to reset bookmark fails, we need to reset the setting to previous value.

For what purpose bookmark_timestamp is used in bookmark reset API?

Is this ok? Any suggestions?
I will later add toaster on this.
The Sync Days and Bookmark reset are sequentially called.
https://github.com/user-attachments/assets/9fa88656-c71e-4f07-9eb1-9bc108d2c765

For what purpose bookmark_timestamp is used in bookmark reset API?

So reset_bookmark will just reset all bookmarks to this param, but if its not passed, it will just pick it from the settings. So you can ignore the param for now.

We'll also need a toast to show the update is done and now we'll resync the repos.

As of this comment the backend work for this has been done. In above PRs.
Additional context for UI and BFF: Our sync depends on the default sync days value and if someone increases the default sync days setting, we should sync older data respecting the new default sync days for existing and prev data. The original plan was to call bookmark service from settings to reset the bookmark if the default sync days setting is updated. This side-effect would have been convenient but magic. So we decided to make an API in #514 and whenever the UI calls to update the default sync days setting, it should also sequentially call reset bookmark api. If the second api to reset bookmark fails, we need to reset the setting to previous value.

Okay. change of plans. Doing this from bff was too much context sharing, so I opened a PR to do this in backend. #518

@samad-yar-khan should this issue be marked as done, and close?

Yes @e-for-eshaan it should be closed now


Using List.Count property in C# results in a strange error

List Count property in C# is supposed to return Int value, but when using myList.Count in Consulo I get a strange error

Looks like https://github.com/consulo/consulo-csharp/issues/313
But it was fixed. Please say C# plugin version

2170

Looks like this issue obsolete. Please update Consulo & plugins

Fixed only partially.
Calling
UnityEngine.Random.Range(0, emptyPositions.Count);

still marked as error on latest consulo

Hi. I created new issue https://github.com/consulo/consulo-csharp/issues/331
Its not simple issue - maybe fixed in a week, or not (i am busy in this month)


When I install version 21.2.0 or newer I get Error NETSDK1082 then the project crashes.

I found a bug to report

I created a project using a standard template BlazorWebAssembly. I use Raider.

`
<PropertyGroup>
    <TargetFramework>net7.0</TargetFramework>
    <Nullable>enable</Nullable>
    <ImplicitUsings>enable</ImplicitUsings>
</PropertyGroup>

<ItemGroup>
    <PackageReference Include="Microsoft.AspNetCore.Components.WebAssembly" Version="7.0.17"/>
    <PackageReference Include="Microsoft.AspNetCore.Components.WebAssembly.DevServer" Version="7.0.17" PrivateAssets="all"/>
    <PackageReference Include="Telegram.Bot" Version="21.10.1" />
</ItemGroup>

`

Installed Telegram.Bot Version="21.10.1".

The project is broken.

JetBrains Rider 2024.1.1
Build #RD-241.14494.325, built on April 16, 2024
Runtime version: 17.0.10+8-b1207.12 amd64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
.NET Core v8.0.1 x64 (Server GC)
Windows 11 Pro 23H2 29.12.
A quick Google search brings up this page:
https://stackoverflow.com/questions/70220739/blazor-client-error-netsdk1082-browser-wasm
which is exactly what you describe.
The answer gives you the explanation: Telegram.Bot package is likely incompatible with WebAssembly
And indeed, if you had looked at our open issues (it's not like we have hundred of these 😏) before opening this one, you would see we have currently an open issue for AOT/Trimming compatibility (#1322), which is a prerequisite for a package to be compatible with Blazor WebAssembly


SaveAs now changes the workbook title

This change improves the SaveAs behaviour for Xamarin workbooks addressing #189
I've tested this in Windows/WPF and it seems to work well. Sorry, I've not got a current Mac setup for Xamarin testing.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: slodge sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

As it mentiones my "employer", I need to check with some legals on https://cla.opensource.microsoft.com/Microsoft/workbooks?pullRequest=501 (will be back Monday or Tuesday!)

CLA now signed - so over to you - but enjoy your trip first 👍


Add trash option to all bot responses

Issue
When using a command like !rule [rules...] or !pep <pep_number>, there is no way for the user to delete the response sent by the bot.
This is would be relevant when entering the wrong argument (for example !rule 4 instead of !rule 5), you just want to check something real quick or you get the output wasn't what you expected.
Proposed solution
Add a "trash can" reaction to let the user delete the response from the bot. This would be the default for all commands, with the option to disable it for specific commands.
A possible way of specifying that a command should not have the trash option would be using a decorator. Something like @not_deletable or @no_trash would be used for the commands.
We already have the trash functionality implemented in many commands, so this is just a matter of making it more general.

The function used for messages such as the codeblock auto-responder is implemented here.
Its used in the auto-responder here,
We'd have to somehow apply that function to all messages, and then add a decorator for messages we don't want to be deleted.

The only way I see a decorator working is if the command returns the message it sends so that the decorator can retrieve it and call wait_for_deletion.

Maybe that could work.

Prematurely hit send and close issue.
I think another option would be to make the decorator add a variable or similar to the command function, that will be passed forward to the function that handles the trash option.
Would this be possible?

I see @MarkKoz's way working better, because then we could just add an @deleteable or @trashable decorator and would not have to completely modify the bot's message-sending functionality. It just seems simpler to implement this that way.

Using a decorator feels confusing (because it's less direct and relies on return values, which is kind of implicit). Furthermore, I think it's redundant. Is it really a worthwhile design to save a single function call?

It is possible to implement a custom context class, and overwrite the send method. We could keep all of the methods the same, but add a custom kwarg to send, and default it to be awaited deletable.
await ctx.send('hello user!', not_trashable=True)


Updated label helper to improve usability

Description
Change regarding feedback in https://github.com/openclarity/kubeclarity/pull/452
Type of Change
[x] Refactor
Checklist

[x] I have read the contributing guidelines
[x] Existing issues have been referenced (where applicable)
[x] I have verified this change is not present in other open pull requests

@jmueller42 LGTM


Change Content to IgnoreColors and await for getAllTestResults

Content is now updated to IgnoreColors in Applitools.
getAllTestResults method returns a promise. Hence an await will give consistent result

@kobenguyent could you give me access so that I can fix this issue

@kobenguyent Forked and created PR #13
please review and merge if everything looks fine


Choose markers to use in FlowSOM clustering

Description of feature
Hello!
Thanks again for the excellent package; I've been using it lately with great success (and fun!). My feature request is for the ability to choose a subset of var_names to cluster the observations on. This is available in the FlowSOM R and Python packages, and is a common and useful tool to control clustering.
There are a few use cases for this:
The first is when we can partition our antigens into those that define clear lineages of cells (e.g. CD3, CD14, CD11b), and those that describe the functional state of cells (e.g. cytokines, metabolic markers). Restricting clustering to only those lineage markers sometimes gives better resolution between populations, whose activation state can then studied using the functional markers.
Secondly, we have performed studies where the question was "can marker set A be used to independently identify the same cells as identified by marker set B" (it was whether metabolic antigens only can be used to identify leucocyte populations). In this case being able to select antigens for a particular clustering model was central to the experiment.
And finally, sometimes we might just have a dud marker that either wasn't expressed or the antibody didn't work, and it simply adds noise.
If there's a convenient way to do this already, please forgive me!
Best wishes
Hefin

Hi @hrj21
thank you for the praise!
I am grateful for your detailed feature enhancement description and the detailed examples. Those are very helpful to understand your feature request. I usually approach the subsetting of var_names as follows, which is certainly blowing up the memory when working with large objects:

Create a temporary copy of the object with the subset of features you are interested in
Run the FlowSOM clustering
Save the clustering result to the original object
Delete the temporary copy

However, I can put some time aside to implement a subsetting functionality that is similar to the use_highly_variable_genes parameter in various scanpy functions. For context, when we compute a PCA on single-cell RNAseq data, we can use either all 10,000+ genes or we can select a subset of informative genes whose variability exceeds the expected noise of the data. We usually don't need that for flow or mass cytometry data, but I imagine that we can create a similar implementation here. The information of which feature was used will be encoded in the .var part of the anndata object. This way, you should be able to track which subset of features you used.
I do have to mention that I have not checked whether the FlowSOM package has this functionality already.
Best,
Maren


Newer elixir versions

The readme says that Scenic only works with Elixir 1.7 and OTP 21.
I can’t find any issues here on github on what’s not working on later versions. Is there such a list somewhere?

I just installed scenic and generated a nerves app. Now I get this warning: warning: the dependency :scenic requires Elixir "~> 1.8" but you are running on v1.7.4. So I guess the Readme here is wrong then?

@Linuus If we take a look at `mix.exs] in scenic repository

@Linuus If we take a look at mix.exs in scenic repository then we would find:
defmodule Scenic.Mixfile do
  # …

  @elixir_version "~> 1.8"
  # …

  def project do
    [
      # …
      elixir: @elixir_version,
      # …
    ]
  end
  # …
end

Which means that scenic requires elixir in at least 1.8 version.
Then we can take a look at history of that file. and finally we would see:

…
back to 1.8
boydm committed on 10 Mar
turn off build_embedded
boydm committed on 10 Mar
try building under 1.7
boydm committed on 10 Mar
so to elixir 1.8
boydm committed on 10 Mar…
…

Which gives as a hint that scenic not longer compiles under elixir in version 1.7. Most probably it uses some new elixir API or there were a bug in elixir fixed in version 1.8.
Looks like that scenic would require elixir in version 1.8 and it won't be changed.
@boydm Looks like we would need to update some parts of code (like templates in examples) + documentation.

Of course... I don't exactly remember what the reason for going to 1.8 was...

Cool :)
Just to be clear, this is the part that confused me: https://github.com/boydm/scenic_new/blob/master/README.md#erlangelixir-versions


Problem with compilation

I cannot compile. What should I do?
Could you please help me
artemis@ubuntu:~/grobid/grobid-service$ mvn -Dmaven.test.skip=true jetty:run-war
Warning: JAVA_HOME environment variable is not set.
[INFO] Scanning for projects...
Downloading: http://repo.maven.apache.org/maven2/org/eluder/coveralls/coveralls-maven-plugin/3.1.0/coveralls-maven-plugin-3.1.0.pom
Downloaded: http://repo.maven.apache.org/maven2/org/eluder/coveralls/coveralls-maven-plugin/3.1.0/coveralls-maven-plugin-3.1.0.pom (10 KB at 46.0 KB/sec)
Downloading: http://repo.maven.apache.org/maven2/org/eluder/eluder-parent/6/eluder-parent-6.pom
Downloaded: http://repo.maven.apache.org/maven2/org/eluder/eluder-parent/6/eluder-parent-6.pom (15 KB at 293.9 KB/sec)
Downloading: http://repo.maven.apache.org/maven2/org/sonatype/oss/oss-parent/9/oss-parent-9.pom
Downloaded: http://repo.maven.apache.org/maven2/org/sonatype/oss/oss-parent/9/oss-parent-9.pom (7 KB at 164.5 KB/sec)
Downloading: http://repo.maven.apache.org/maven2/org/eluder/coveralls/coveralls-maven-plugin/3.1.0/coveralls-maven-plugin-3.1.0.jar
Downloaded: http://repo.maven.apache.org/maven2/org/eluder/coveralls/coveralls-maven-plugin/3.1.0/coveralls-maven-plugin-3.1.0.jar (77 KB at 738.6 KB/sec)
...
Downloaded: http://repo.maven.apache.org/maven2/log4j/log4j/1.2.17/log4j-1.2.17.jar (479 KB at 1523.6 KB/sec)
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 5.264s
[INFO] Finished at: Tue Oct 06 00:42:13 PDT 2015
[INFO] Final Memory: 8M/22M
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal on project grobid-service: Could not resolve dependencies for project org.grobid:grobid-service:war:0.3.9-SNAPSHOT: Could not find artifact org.grobid:grobid-core:jar:0.3.9-SNAPSHOT -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/DependencyResolutionException

Hello!
You need to build first Grobid from the root project grobid/ before starting the service from grobid/grobid-service.
See http://grobid.readthedocs.org/en/latest/Install-Grobid/

Thank you!
That helped me


[WIP] Initial distributions of permanent income and mNrm

#592 asks for "a plausible calibration of distributions at age of workforce entry of: -Permanent income -Assets". In discussions with @llorracc, we determined the best place to get such a calibration would be the SCF. This PR works on adding calibrations of these two distributions coming ---as directly and transparently as possible--- from the SCF.
An issue is that we clearly do not want to have the full SCF in the toolbox. The solution that I propose here is:

Keep only a relatively light table with all (most of?) the summary statistics we will possibly need: WealthIncomeStats.csv. It currently contains survey-weighted means and standard deviations at multiple age-year-education combinations. I can easily expand it to contain, e.g., deciles in case one wants to avoid lognormality.
Link to a public repo that produces the table directly from raw SCF files.
Have functions that parse the contents of the table to produce the exact parameters that HARK's models need.

This clearly need more documentation, which I will add. I first wanted to check whether you (@llorracc @sbenthall ) like this solution of keeping only the table and linking to the external repo that reproduces it.
Please ensure your pull request adheres to the following guidelines:

[ ] Tests for new functionality/models or Tests to reproduce the bug-fix in code.
[ ] Updated documentation of features that add new functionality.
[ ] Update CHANGELOG.md with major/minor changes.

@llorracc Chris, not sure if you merged this by accident, but it was work in progress. It's a bit far from complete. I just wanted initial feedback on how you all felt about the external repo solution. It needs at least:

Documentation.
A way to transform the moments to real dollars for any year one wants. I wanted to discuss the best way to do this in our call today.

@llorracc Chris, not sure if you merged this by accident, but it was work in progress. It's a bit far from complete. I just wanted initial feedback on how you all felt about the external repo solution. It needs at least:

Documentation.
A way to transform the moments to real dollars for any year one wants. I wanted to discuss the best way to do this in our call today.


Update 055-roses.md

Monique GOBRIAL WITH MUSIC : 20210920-MG-MaurerRose-D-Shifdman
https://editor.p5js.org/MGOBRIAL/sketches/9_4EGEnOA
https://youtu.be/JoDbCtVBJBI

Community Contribution
Link to live project, video, or image:

Sharing

[x] I would be happy to have my project shared on The Coding Train's social media!

Instagram:
Twitter:

Duplicate of #3348


🚸 Support Spanish

Thanks @leiyre for your awesome feedback.

Made some more suggestions after trying the Spanish version in the UI. Also, here's a screenshot with some elements that have not been included in the translation file:

@damianpumar Could we include these texts in the translation files? I think after we include those translations we can safely merge this 😃


1938 ErrorProne fix DefaultCharset

Apply error prone patches for DefaultCharset.
Jenkins build for this commit hash: https://multipass.revelc.net/jenkins/job/ldh-emissary/19/

The jenkins build for the rebased commit:  https://multipass.revelc.net/jenkins/me/my-views/view/Lisa's View/job/ldh-emissary/23/


nuclei: 2.5.0 -> 2.5.1

Motivation for this change
Update to latest upstream release 2.5.1
Change log: https://github.com/projectdiscovery/nuclei/releases/tag/v2.5.1
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)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.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.

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

1 package built:

nuclei


Error interpretation

Context is needed for interpreting RMSE. Is an RMSE of 0.7 actually a large error here given that the scale is out of 10? Also, RMSE != accuracy (see RMSE/RMSPE and accuracy definitions). RMSE is a suitable measure for regression, whereas accuracy is a measure for classification.

Also, how are the scores at the bottom of the report different from those at the top? And what are the scores at the bottom of the report?

Thanks for the clarification on accuracy. The score at the top of the report is on the test set, whereas the score at the bottom of the report is on the cross-validation set (validation set).


Run TranSalNet models locally

Moves fine-tuned TranSalNet from Colab to the repository.

[x] Add model

[ ] Add losses

[x] Add pre-trained weights from Drive

[ ] Add image transformations

[x] Record screen-capture of interacting with Desktop / web / etc

[ ] Run sampled frames (10 / sec?) through the model

Superseded by https://github.com/The-Jacob-Lopez/GazeMouse/pull/10. Closing PR


Add rake task to repopulate Monthly Agggregations

For the Content Data App (Data Warehouse).

 (2) Raise 2ndline alarms when the Monthly aggregations are not collected from Google

👍


Support for 2 or more dynamic route parameters?

Which scope/s are relevant/related to the feature request?
create-analog
Information
I wrote it up as a question here:
https://stackoverflow.com/questions/76086555/rendering-routes-with-2-or-more-route-parameters-with-analogjs
Describe any alternatives/workarounds you're currently using
It works in Scully, however Scully is current experiencing growing pains ...
I would be willing to submit a PR to fix this issue

[ ] Yes
[ ] No

This works:
products.[productId].[partid].page.ts


ERROR in Autosize is not an NgModule

With<EMAIL_ADDRESS>the autosize module stops working with the error in the description above.
It seems that *.metadata.json files are missing that are needed for the new AOT compilation approach used by angular-cli
See: https://angular.io/docs/ts/latest/cookbook/aot-compiler.html

Totally My bad. Turned out that I accidentally placed the directive in @NgModule imports instead of declarations, leading to this issue. I'll close the issue.


Suggestion: rethink polyfills approach in CLI 2.0

Bug Report or Feature Request (mark with an x)
- [ ] bug report -> please search issues before submitting
- [x] feature request

Desired functionality.
Following angular/devkit#212, and as AOT will be the default in Angular 5 and CLI 2.0, the only polyfill left for evergreen browsers will be zone.js, which is not really a polyfill (it's not native and at stage 0 from a long time in TC39), it is an Angular dependency (as the comment in polyfills.ts suggests).
So if zone.js is moved to a more appropriate place (main.ts ? directly in webpack config ?), it would mean there would be no polyfill bundle to build and load for many users.

But some the polyfills are required for IE10 and IE11. They're commented out by default anyways. Am I missing something?

@Matmo10  If zonejs was not in the polyfills bundle and all other polyfills were commented out than the polyfills bundle would not need to exist for that app and would allow the elimination of the additional asset fetch.
Zonejs was previously (quite some time ago) part of the main bundle and there were some peculiar runtime issues regarding change detection.  The issue is here: https://github.com/angular/angular-cli/issues/2752 .

Just a clarification: Zones is not actually required by Angular, since v5 Zones are optional


feature suggestion: filter page-property

Thank you for your efforts.
This plug-in is very useful.
Each page has page-property in logseq,such as :
title:: this is tile
tags:: english
type:: book
I have 10 pages like above, Now I want to see the pages only in the 10 pages scope.
I wish you can add the feature through the pannel setting
like this:
{
"disabled": false,
"randomNoteIncludeJournals": true
"page-tags": english
}
thank you very much

Thanks for the advice. I will take a look and make it happen.


Error (after a long time on the world)

OTG Plugin or Mod

[X] OTG Mod for Forge
[ ] OTG Plugin for Spigot

OTG Version

[X] 1.12.2_v8_r1
[ ] 1.12.2_v7_DEV_r26
[ ] 1.12.2-v6
[ ] 1.12.2-v5
[ ] 1.11.2-v8
[ ] 1.11.2-v9
[ ] 1.10.2-v21
[ ] Other - specify here

Forge or Spigot Versions
Forge Mod Loader version <IP_ADDRESS>47 for Minecraft 1.12.2 loading
Java is Java HotSpot(TM) 64-Bit Server VM, version 1.8.0_231, running on Linux:amd64:5.4.14-mtxserv, installed at /usr/lib/jvm/java-8-openjdk-amd64/jre
Server/Client Log
https://pastebin.com/S9XupK4C
What other plugins are you running?
https://pastebin.com/h5kFzDSX
Description
So everything worked fine for more than 3 month but at some time while exploring at more than 20 000 blocks, the server crashed and I have the same error every time I reload the server.
(P.S : I was in v6 but when I got this error I tried to update in v8, the problem is still the same)

Hey @uiytt , it appears that the StructureData.txt file in /OpenTerrainGenerator/ has been corrupted. I'm not sure how this could have happened, perhaps the server crashed while the file was being saved. If you're not using BO4's (you most likely aren't, most presets like BB don't), you may be able to just delete the file, or create a new world and copy its StructureData.txt to your broken world.
*Each dimension has its own StructureData.txt, make sure you delete/replace the right one.
If you need help with more things like this, please join us on the OTG Discord. If you've found a technical error you're sure is a bug, feel free to open a new git issue and provide logs+repro steps, thanks!


[Bug] [Source] Temporary files are not ignored

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
ignored Temporary files
SeaTunnel Version
2.3.1
SeaTunnel Config
env {
  # You can set spark configuration here
  # see available properties defined by spark: https://spark.apache.org/docs/latest/configuration.html#available-properties
  #job.mode = BATCH
  job.name = "SeaTunnel"
  spark.executor.instances = 1
  spark.executor.cores = 1
  spark.executor.memory = "1g"
  spark.master = local
}
source {
  Hive {
    table_name = "test.test_count"
    metastore_uri = "thrift://master1:9083"
    hdfs_site_path = "/data/conf/cdh/hadoop/hdfs-site.xml"
    read_partitions = ["dt=2023-01-01"]
    result_table_name = "hive_table"
  }
}
transform {
  #Sql {
  #  source_table_name = "hive_table"
   # result_table_name = "trans_table"
   # query = "select * from hive_table"
  #}
}

sink {
  console{
  }
}

Running Command
seatunnel.sh --config test.conf --deploy-mode local

Error Exception
.hive-staging_hive*  Contains data, will dirty read。

Flink or Spark Version
No response
Java or Scala Version
No response
Screenshots

Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

Check whether there is a problem PTAL: @EricJoy2048  ,I'll fix it if I can


Send message to a connected client

Hi im new to this and was able to successfully bind in transceiver mode and send messages from client to the smpp server towards a simulated SMSC. However, I am trying to send a message back to a connected client. I have a single client connected and store the session for use later when I receive a request to send a message to this client from smsc. However it does not work. It doesnt reach the client side.
The below is my code:
`SmppServerSession smppSession = smppSessions.get(deviceId);
		DeliverSm pdu = new DeliverSm();

		pdu.setSourceAddress(new Address((byte)0x03, (byte)0x00, packet.getMessage().getRecipientAddress()));
        pdu.setDestAddress(new Address((byte)0x01, (byte)0x01, packet.getMessage().getDestinationAddress()));
        byte[] textBytes = CharsetUtil.encode(packet.getMessage().getText(), CharsetUtil.CHARSET_GSM);
		pdu.setShortMessage(textBytes);

		 WindowFuture<Integer,PduRequest,PduResponse> future = smppSession.sendRequestPdu(pdu, 10000, false);
	        if (!future.await()) {
	            log.error("Failed to receive deliver_sm_resp within specified time");
	        } else if (future.isSuccess()) {
	            DeliverSmResp deliverSmResp = (DeliverSmResp)future.getResponse();
	            log.info("deliver_sm_resp: commandStatus [" + deliverSmResp.getCommandStatus() + "=" + deliverSmResp.getResultMessage() + "]");
	        } else {
	            log.error("Failed to properly receive deliver_sm_resp: " + future.getCause());
	        }`

Logs are below on smpp server side but the message isnt delivered to the connected smpp client.
2016-11-20 22:16:37,836  INFO SmsProtocolHandler:59  -  -  - Received command MESSASGE_RECEIVE_REQ from client<PHONE_NUMBER>8
2016-11-20 22:16:37,837  INFO DefaultSmppSession:525  -  -  - async send PDU: (deliver_sm: 0x00000055 0x00000005 0x00000000 0x00000013) (body: (serviceType [] sourceAddr [0x03 0x00 [18764707478]] destAddr [0x01 0x01 [18888707478]] esmCls [0x00] regDlvry [0x00] dcs [0x00] message [5468616E6B7320666F7220616C6C6F77696E67206D6520746F206A6F696E])) (opts: )
2016-11-20 22:16:37,844 TRACE DefaultSmppSession:628  -  -  - Found a future in the window for seqNum [19]
2016-11-20 22:16:37,845 TRACE DefaultSmppSession:635  -  -  - Caller waiting for request: (deliver_sm: 0x00000055 0x00000005 0x00000000 0x00000013) (body: (serviceType [] sourceAddr [0x03 0x00 [18888707478]] destAddr [0x01 0x01 [18888707478]] esmCls [0x00] regDlvry [0x00] dcs [0x00] message [5468616E6B7320666F7220616C6C6F77696E67206D6520746F206A6F696E])) (opts: )
2016-11-20 22:16:37,845  INFO MessageClientManager:151  -  -  - deliver_sm_resp: commandStatus [0=OK]
I appreciate any help you can give.
Thanks

I realized that the issue was due to my smpp client's bind source address not matching. It is working now.


refactor: OpenAPI type name trimming regex

Pull Request Checklist

[ ] New code has 100% test coverage
[ ] (If applicable) The prose documentation has been updated to reflect the changes introduced by this PR
[ ] (If applicable) The reference documentation has been updated to reflect the changes introduced by this PR
[ ] Pre-Commit Checks were ran and passed
[ ] Tests were ran and passed

Description

This regex is used to remove superfluous chars from the start and end of the schema keys that we generate for openapi.
Sonar reports two issues to do with the original implementation:

a reliability issue to do with operator precedence
a security issue to do with backtracking

This PR refactors the regex so that it has less parts and less scope for backtracking.
A follow up to #2706
Close Issue(s)

As you can see it didn't report anything on this pr either @provinzkraut - but still isn't happy with the pattern on main 🤷♂

That's quite annoying. Fixing something like this by just guessing isn't ideal 🙃
I was thinking of maybe using their IDE integration to figure this out. Ideally though I'd like to be able to just run the tests from the command line or something.


Not receiving http webhooks

I create a geofence with a webhook with the node-tile38 module like this:
    let meta = { field: 'val1', field2: 'val2'};
    let opts = {
        point: [33.5123, -112.2693],
        radius: 10000
    };
    tileClient.setHook('testhook0', 'http://localhost:5656/webhook', meta, 'nearby', 'fleet', opts).then((result) => {
        console.log(JSON.stringify(result));
    }).catch((err) => {
        console.lof(JSON.stringify(err));
    });

When I create a key with a location within the geofence like this
    tileClient.set('fleet', 'truck1', [33.5123, -112.2693]).then((result) => {
        console.log(JSON.stringify(result));
    }).catch((err) => {
        console.log(JSON.stringify(err));
    });

I don't receive a webhook POST request to the given webhook, as I assumed I would do... What could have gone wrong? Thanks!

I verified that both the geofence as well as the key are written to Tile38. Sorry to not mention that above.

Are you seeing any errors with the setHook call?

No, both calls are going through smoothly. Here's the output from Tile38:
<IP_ADDRESS>:9851> hooks *
{"ok":true,"hooks":[{"name":"testhook0","key":"fleet","endpoints":["http://localhost:5656/webhook"],"command":["NEARBY","fleet","FENCE","POINT","33.5123","-112.2693","10000"]}],"elapsed":"64.2µs"}
<IP_ADDRESS>:9851> get fleet truck1
{"ok":true,"object":{"type":"Point","coordinates":[-112.2693,33.5123]},"elapsed":"943.5µs"}

Is there a way to check on the server side (logs?) whether the webhook is triggered or not?

There isn't a log (though there probably should be). Messages generally get sent immediately. If there's a hiccup with delivery then the message gets queued to queue.db and Tile38 tries again asap.
There's a built-in http-webhook tester that I use for diagnostics:
$ tile38-server --webhook-http-consumer-port 5656

All this does is create a webhook listener and print the notification to the console.

Actually if start tile38-server with -vv you'll see notification delivery log messages such as:
2018/03/14 09:58:32 [DEBU] Endpoint send ok: 5: http://localhost:5656/webhook: <nil>

Thanks for the fast replies! I started the server with -vv and now I actually see the error:
2018/03/14 17:23:08 [DEBU] Endpoint connect/send error: 1: http://localhost:5656/webhook: Post localhost:5656/webhook: dial tcp localhost:5656: connect: connection refused

It seems to be the problem that I was running it in Docker Compose, and localhost is pointing to the container of Tile38 if it isn't run in host networking mode I guess. Strangely, I see connection refused problems when running it in host networking mode from my node app when I try to connect to it then via localhost:9851, which should work IMO. Seems to be a Docker for Mac problem though.

BTW, it seems there is no exponential backoff strategy in place, or am I wrong?

The server will eventually give up on a message after too long, around a minute or so.

@tidwall Is there any way to check on the logs whether the web-hook is triggered or not? since i am not able trigger the webhook.

Actually if start tile38-server with -vv you'll see notification delivery log messages such as:

2018/03/14 09:58:32 [DEBU] Endpoint send ok: 5: http://localhost:5656/webhook: <nil>

see above

Hi @iwpnd,
Yeah, I started tile38 server with tile38-server -vv but i am not able see any logs.
in my case i am using amazon SQS endpoint.  I am able to set the data in tile38 collection for that created hook but I want o see why hook is not triggered

What do you mean you're not able to see any logs? If you don't see anything, your hooks are not not triggering.
You can test your setup locally and eliminate the possibility that your hooks don't fire because of a falsely setup with the use of channels.

What do you mean you're not able to see any logs? If you don't see anything, your hooks are not not triggering.
I mean, yeah hooks are not triggering, I want to know what is the reason for that - is there any way to check the logs?
Using channels, its working fine. I have tested it

Your server is emitting logs of you start it with the -vv flag.

@iwpnd below are the logs listed when I start server with -vv, but hook is not triggering when breach happens.
2023/12/15 11:37:13 [INFO] Server started, Tile38 version 1.32.1, git c494fe54
2023/12/15 11:37:13 [DEBU] Geom indexing: QuadTree (64 points)
2023/12/15 11:37:13 [DEBU] Multi indexing: RTree (64 points)
2023/12/15 11:37:13 [INFO] Ready to accept connections at [::]:9851
2023/12/15 11:37:13 [INFO] AOF loaded 274 commands: 0.00s, 107075/s, 15 MB/s
2023/12/15 11:37:20 [DEBU] Opened connection: <IP_ADDRESS>:61283

I suggest you open another issue, tell us what you want to do and provide some example code for anybody to reproduce. If you do not see any log when entering a hook - from my experience - it always comes down to a wrong setup.
If it were anything else the hook manager would throw an error that it cannot send the message. Take the latter with a grain of salt as I have never used SQS with tile38 and do not have the time to look into the SQS client right now.

Ok, i will open issue and provide you some example to start with. Thank you @iwpnd


use FASTER_DYNAMIC_SHAPES_0805 not kFASTER_DYNAMIC_SHAPES_0805

02-API/builderconfig.py

Thank you for finding this problem! My bad spelling.

what was this moved to?

It is deprecated in TensorRT-8.6 and removed in newer version of TensorRT (https://docs.nvidia.com/deeplearning/tensorrt/archives/tensorrt-930/release-notes/index.html#rel-8-6-1).


Post release automated changes for tables releases

Post release automated changes for azure-data-tables

@joheredi Please review


404 Error on Windows build

OS : Windows 10
Polaris version : 0.13.0
As the title suggests, a 404 error is found when I navigate to http://localhost:5050/
where the previous polaris build would present a login page.

Thanks for the report. The 0.13.0 Windows installer does look very broken. I'm investigating.

Fixed in ea7edea79bd21989bcd8b0cf26c8a7638c101791 and published as version 0.13.1 😓. I'll open a follow-up ticket for a better long term solution.
Note that if you run Polaris as administrator, the 0.13.0 server may have created a C:\Users\runneradmin folder. This folder is safe to delete.
@stukkat  Please let me know if this fix works for you!

Tested and everything seems to be in order. Thank you @agersant :)

Tested and everything seems to be in order. Thank you @agersant :)


Game Crash when Factory is deconstructed without clearing Factory overlay controller first

Error while running event factorissimo-2-notnotmelon::on_player_mined_entity (ID 68)
factorissimo-2-notnotmelon/control.lua:494: attempt to concatenate field '?' (a nil value)
stack traceback:
factorissimo-2-notnotmelon/control.lua:494: in function 'generate_factory_item_description'
factorissimo-2-notnotmelon/control.lua:522: in function <factorissimo-2-notnotmelon/control.lua:509>

What sort of item did you put in the controller

Crude oil was one of the entities listed om the controller before the crash occurred.


Use design system date component for scheduled publication input

What
Replace the scheduled publication date selection component with the date input component from the GOV.UK publishing components
Why
We feel this is a better user experience because users are able to input a date without scrolling through a large number of options. This is also bringing us in line with other publishing applications.
One thing to note is that until now there was no validation of the scheduled publication input because it was assumed to be impossible to enter an invalid date using the select inputs. Rails also automatically attempted to use the input to instantiate a Date object. This is no longer safe to do, so we have had to add a check to ensure that the input can be safely parsed to a date and to add an error to the edition if it cannot.
We've also fixed a bug where the scheduled publication active checkbox was losing its state on a validation error.
Next Steps
We're going to gradually replace the other document date fields
Screenshots

Trello: https://trello.com/c/IFGDegsk

@ryanb-gds - Is the error message in x's? I'm not sure this matches the pattern https://design-system.service.gov.uk/components/error-message/


Icons broken by default, 'make doctor' not helpful

When I open the editor, it displays incorrect unicode symbols:

Then, if I enable unicode in init.el, the entire Emacs window is just blank, white.

it loads 100% CPU, and after about a minute displays the default screen, with unicode symbols being still broken:

Finally, when I try to run make doctor, I receive the following errors:
.emacs.d(master)*$ DEBUG=1 make doctor
DOOM Doctor
Running Emacs v26.0.50, commit 5fe3cfbc75ba25f8c25f2633d21f2c7f29281a1e

Compiled with:
  XPM JPEG TIFF GIF PNG RSVG IMAGEMAGICK SOUND GPM DBUS GSETTINGS NOTIFY
  LIBSELINUX GNUTLS LIBXML2 FREETYPE M17N_FLT LIBOTF XFT ZLIB
  TOOLKIT_SCROLL_BARS GTK3 X11 MODULES XWIDGETS LIBSYSTEMD
uname -a:
  Linux forlorn-ruins 4.10.0-30-generic #34~16.04.1-Ubuntu SMP Wed Aug 2
  02:13:56 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux

Attempt to load DOOM: success! Loaded v2.0.4
Revision 5fe3cfbc75ba25f8c25f2633d21f2c7f29281a1e

----

Verified elpa.gnu.org
Verified melpa.org
peculiar error
Makefile:75: recipe for target 'doctor' failed
make: *** [doctor] Error 255

The icons can be fixed with M-x all-the-icons-install-fonts, perhaps add it to the README? The doctor command still fails.

You're right. I'm in the process of rewriting my readme (and adding more documentation in general). This will be mentioned in the new quick start section.
In the meantime, I've added font detection to the doctor. It will complain if all-the-icons fonts are missing (and tell you to run M-x all-the-icons-install-fonts if you don't).
On another note: the ui/unicode module is unrelated to icon fonts. It's more for non-English speaking users who need better language support, but I realize that isn't obvious. I'll work on that!
I'll keep this issue open until the new readme + documentation comes up.

Following up on this:
@hlissner: please don't make all-the-icons mandatory in make install. It will break terminal users (especially remote ssh users). Have you considered using  this? https://github.com/sebastiencs/icons-in-terminal. It would be great to have either all-the-icons or icons-in-terminal support based on UI mode or not.
BTW, I know this config is not so much about terminal users, but it's mostly working fine, and I love it. Please, don't break it ❤

@amoriello Not to worry, I have no intention of making it mandatory. I'm very tempted to switch to icons-in-terminal, not for terminal support specifically, but because its API is simpler. Sadly, it is low priority and is non-trivial, so it'll get a tentative "whenever I get around to it".

In any case, I've included this problem in the FAQ under troubleshooting, and talk about it in the "Post-installation tips" section of the new installation guide, including a link to the wiki in project readme (albeit on the develop branch, for now).
It's still a WIP, but I'll consider this resolved.


Test stack upgrades

Hello!

Vote on this issue by adding a 👍 reaction
If you want to implement this feature, comment to let us know (we'll work with you on design, scheduling, etc.)

Issue details
We should consider testing for no-changes provider upgrades.
There is a growing number of issues with provider upgrades generating unexpected changes in the users's stacks:

https://github.com/pulumi/pulumi-random/issues/258 P1
https://github.com/pulumi/pulumi-eks/issues/851
https://github.com/pulumi/pulumi-fastly/issues/173

Some of these issues could have been detected by test automation checking specifically for the upgrade scenario. We currently do not know of an example of such automation.

[ ] add pulumi-random test for stack upgrades (post mortem follow-up)
[ ] discuss design and see anything can be upstreamed to ProgramTest
[ ] figure out a plan for introducing this testing across other providers

Affected area/feature

Provider testing.

Implemented for Random provider: https://github.com/pulumi/pulumi-random/pull/263

Plus one P1 on this, will be propagating the ad-hoc test to TLS.

Closing in favor of https://github.com/pulumi/ci-mgmt/issues/325


[stable/consul] Make cluster name a variable

People don't necessarily use "local" as a cluster name. Since this
value was hardcoded in the cluster bootstraping script, the Chart
couldn't be installed on such cluster.
This commit introduces a "ClusterName" variable defaulting to "local"
for that parameter.
Note
Since the different consul node are living in the same namespace, just getting
rid of the "cluster.xxx"part in the domain name might be even more relevant.
What do you think ?

@k8s-bot ok to test

Thanks @elafarge. I think this is good to go. Can you please bump the chart version?

Need to dig into why pods aren't making it to running state.

Mmmmh I don't know. Pods appear as expected (and join together to form a consul cluster as well). However there is a 30-ish second delay between each pod creation. On Jenkins, it seems that we're waiting for 10 seconds only. Could it be linked ?
Eventually, I went for the "don't specify any cluster name" solution, which doesn't add any "Value" and I don't think people would be willing to spread Consul clusters across Kubernetes clusters anyways :)
It also works on my cluster, same spawning time but the commit removes code instead of adding a template variable. Let me know if you have any objections :)

Mmmh, it really works like a charm on my end... After a dozen seconds (the time for the PVC to be bound) the first Pod's state transitions from "Pending" to "Running".
I'm running Kubernetes 1.6.1 on top of AWS. Maybe your tests are running on a 1.5 cluster ? From what I recall, I don't think there are any differences in the way DNS namespacing is handled (nor in the Pod's /etc/resolv.conf) between these versions.
Or could it be linked to the StatefulSet's PersistentVolumeClaims not being honored somehow ? I have no clue about the environment these tests are running in, I'm just trying to guess.

@k8s-bot ok to test

@k8s-bot e2e test this

@k8s-bot ok to test

Thanks for the info @elafarge, just checked and the PVCs are not getting bound because the chart is requesting a beta storage class named 'default' by default. We need to either:

conditionally switch between alpha and beta storage class annotation for 1.4 and 1.5 support (and default to alpha)
leave out the beta annotation/storageClassName option by default, adding it in if a storage class is defined for 1.6+ support

The former will make it pass in the CI, the latter will require us to update the test cluster to 1.6 - however if we can verify that the chart works well in 1.6 I would prefer to go for the latter as we should upgrade to 1.6 soon anyway.

Wouldn't it be possible to test both versions ? I think Jenkins now has a feature somewhat equivalent to Travis CI's build matrix (I never tried it though) that would allow us to run the same test suite against multiple Kubernetes versions.
This way, if we running on 1.5 we'll use a user-defined storage class as StorageClass's value in our chart and just use the default one provided by Kubernetes for 1.6+ versions (and yeah, we'll also have to handle the alpha/beta status in the annotation)p
Also I don't think it's relevant to test the 1.4 version: kind: StatefulSet is hardcoded in the Chart. In 1.4 StatefulSets were still called PetSets :)
Would all that make sense ?

We haven't really settled on what versions we want to officially support (though we're working towards deciding this and making it transparent), I would say that for now we can err on the side of supporting 1.5 and 1.6 by conditionally switching between alpha and beta storage class annotations. I think we should do this in a separate PR though. @lachie83 want to go ahead and merge this despite the test failure?

Let's get this merged first -- #973

@k8s-bot ok to test

LGTM


*: add Auth prefix to auth related requests and responses

This PR is based on the comment: https://github.com/coreos/etcd/pull/4845#issuecomment-201652908
/cc @xiang90

BTW, current auth service has a method Authenticate() and its request and response are AuthenticateRequest and AuthenticateResponse. Should I add the auth prefix to them, too? I thought AuthAuthenticate() seems to be a little bit odd, so this PR doesn't change the method and structs.

Authenticate() and its request/response are fine.

LGTM


Crash when launching Rednand after updating it

Hello,
I followed all the steps until part 4 where I am stuck. It is said to update RedNand in the system parameters what I have done but everytime I launch RedNand with Luma3ds, I have a message saying to wait until menu arrive and then it crashes.
Can you help me?
Thanks in advance.

Redo Part 3, something has gone wrong somewhere...

I actually had no problem to lauch RedNand before updating it. I was on EMU 9.2. Is there perhaps a problem with Luma3ds ?

Ask @AuroraWright


NonHeapMemoryMetricsCollector.java reports heap-memory pools

The NonHeapMemoryMetricsCollector class reports heap-memory pools, specifically G1 eden, survivor and the old pools under the non_heap metric namespace.
The issue seems to be the foreach loop here - NonHeapMemoryMetricsCollector.java#L46. Going by the MemoryPoolMXBean documentation, it seems the getType() method's return value should be checked before emitting the metrics.
I'd be glad to submit a PR but not sure how this should be fixed - creating a new collector for memory-pools and gradually deprecating these or checking for Objects.equals(MemoryType.NON_HEAPpool.getType()) in the class and removing the heap pools altogether.

The PR looks good to me, thanks for getting this fixed quick.


useRenderButton() needs to accept width as number

Seems like there was a change maybe 7/27 or 7/28 in the client code that now only accepts a number value for the width property to render a button vs a string. It doesn't render the button if you pass in a string value for width.

https://github.com/DefinitelyTyped/DefinitelyTyped/pull/66207
https://github.com/MomenSherif/react-oauth/issues/281

Thanks for the quick response. I was getting a work around in place when I noticed you updated the package. Awesome!


Allow edition=2018 on beta

Edition RC1 is live as the 1.30 beta! 🎉
Switching to the 2018 edition still forces the toolchain to nightly. Now that it's available on beta, this should be changed to (beta or nightly) instead.

Thank you for the reminder.