MediaPipe hands examples gives me errors, using WINDOWS

Hi there, i have a problem with the hands example and i'm stuck on it !!!!!
In the cmd using the command that you recommend:
" bazel build -c opt --define MEDIAPIPE_DISABLE_GPU=1 mediapipe/examples/desktop/hand_tracking:hand_tracking_cpu "
I get this error:
INFO: Repository local_execution_config_python instantiated at:
no stack (--record_rule_instantiation_callstack not enabled)
Repository rule local_python_configure defined at:
C:/users/diego/_bazel_diego/p2omsmjl/external/org_tensorflow/third_party/py/python_configure.bzl:275:41: in
ERROR: An error occurred during the fetch of repository 'local_execution_config_python':
Traceback (most recent call last):
File "C:/users/diego/_bazel_diego/p2omsmjl/external/org_tensorflow/third_party/py/python_configure.bzl", line 210
_get_python_lib(repository_ctx, <1 more arguments>)
File "C:/users/diego/_bazel_diego/p2omsmjl/external/org_tensorflow/third_party/py/python_configure.bzl", line 130, in _get_python_lib
execute(repository_ctx, <1 more arguments>)
File "C:/users/diego/_bazel_diego/p2omsmjl/external/org_tensorflow/third_party/remote_config/common.bzl", line 208, in execute
fail(<1 more arguments>)
Repository command failed
File "", line 1
SyntaxError: (unicode error) 'unicodeescape' codec can't decode bytes in position 2-3: truncated \UXXXXXXXX escape
ERROR: Analysis of target '//mediapipe/examples/desktop/hand_tracking:hand_tracking_cpu' failed; build aborted: Traceback (most recent call last):
File "C:/users/diego/_bazel_diego/p2omsmjl/external/org_tensorflow/third_party/py/python_configure.bzl", line 210
_get_python_lib(repository_ctx, <1 more arguments>)
File "C:/users/diego/_bazel_diego/p2omsmjl/external/org_tensorflow/third_party/py/python_configure.bzl", line 130, in _get_python_lib
execute(repository_ctx, <1 more arguments>)
File "C:/users/diego/_bazel_diego/p2omsmjl/external/org_tensorflow/third_party/remote_config/common.bzl", line 208, in execute
fail(<1 more arguments>)
Repository command failed
File "", line 1
SyntaxError: (unicode error) 'unicodeescape' codec can't decode bytes in position 2-3: truncated \UXXXXXXXX escape
INFO: Elapsed time: 0.635s
INFO: 0 processes.
FAILED: Build did NOT complete successfully (0 packages loaded, 0 targets configured)

Hi @jiuqiant ,
Do you have a solution for that problem?
I have already changed the version of bazel with the one that you said but still this problem
Many thanks.

@dgrnd4 is my same problem at #1035

Solved installing UBUNTU because window for these things is a problem


XIVY-6828 Add product related data

I like it all. But I'm just not sure about the folder name:
product vs. excel-connector-product
The artifactId is called excel-connector-product
I would prefer excel-connector-product. But will make templating a bit more harder ...
Resulting zip:

Repo layout:


Query IP Format error

I get this error in log. And I get the response when I execute 'curl'
`curl -Lv myip.ipip.net

GET / HTTP/1.1
Host: myip.ipip.net
User-Agent: curl/7.54.0
Accept: /
当前 IP： xx 来自于：中国 安徽 合肥  电信

full log?

Full log is here.

Rebuilt URL to: myip.ipip.net/
Trying <IP_ADDRESS>...
TCP_NODELAY set
Connected to myip.ipip.net (<IP_ADDRESS>) port 80 (#0)

GET / HTTP/1.1
Host: myip.ipip.net
User-Agent: curl/7.55.1
Accept: /

< HTTP/1.1 200 OK
< Date: Thu, 04 Feb 2021 14:52:38 GMT
< Content-Type: text/plain; charset=utf-8
< Content-Length: 69
< Connection: keep-alive
< X-Via-JSL: 1d9bd9a,-
< Set-Cookie: __jsluid_h=ca044f40e1c04da6c0c5103fa37c6db3; max-age=31536000; path=/; HttpOnly
< X-Cache: bypass
<
当前 IP：xx  来自于：中国 安徽 合肥  电信

full log of aliyun-ddns-cli

在容器里启动新的终端的时候，报错了
OCI runtime exec failed: exec failed: container_linux.go:344: starting container process caused “exec: “/bin/bash”: stat /bin/bash: no such file or directory”: unknown

use /bin/ash instead

I get the log here:

Could not resolve host

And I get the reason, because I set the wrong firewall rule .
Sorry about that.


Make probot useable with RunKit

Feature Request - probot in RunKit
Users want to use probot with RunKit instead of self-hosted NodeJS or platforms like Glitch
Background
RunKit notebooks are interactive javascript playgrounds connected to a complete node environment right in your browser. Every npm module pre-installed. RunKit can host any notebook as an endpoint if required.
Solution
Add a method for starting probot from within a RunKit Notebook (start probot from inside JavaScript code, instead of from the command line)
This might already be possible by using the .createProbot method or the .start, although I doubt it by inspecting the structure of the probot object/class.
I would propose a .createEndpoint method for probot. That method would create an importable endpoint for either NodeJS itself or even framework like express. The following code shows the proposed method in action.
A: (using pure NodeJS)
var probot = require("probot");

exports.endpoint = probot.createEndpoint(app => {
  // your `probot` code here
}, options)

B: (using ExpressJS)
var probot = require("probot");
var express = require("@runkit/runkit/express-endpoint/1.0.0");
var app = express(exports);

app.use(probot.createEndpoint(app => {
  // your `probot` code here
}, options));

The first argument would be the standard app function that is passed to the export for probot.
The second argument would be an options object containing the same options that the probot-run script can take.
The introduced method should ideally return an object with the same signature as the callback passed to server.listen in NodeJS directly. The only change here is removing the explicit server creation and port binding.
Teachability, Documentation, Adoption, Migration Strategy
The change should not be too complex for someone very versed with the project. No migration required as only functionality is added.

Not sure if the same reasoning for closing https://github.com/probot/friction/issues/8 can be applied here... I will close if this is already possible somehow!

@jannik-mohemian 👋 this is indeed very similar to https://github.com/probot/friction/issues/8 - by grabbing the code used in probot run, you can start it with a programmatic API. We'd still like to have this kind of setup as a first-class API, but in the meantime this might help get you started:
const express =<EMAIL_ADDRESS>const { createProbot } = require('probot')

const probot = createProbot({
  id: process.env.APP_ID,
  port: process.env.PORT || 3000,
  secret: process.env.WEBHOOK_SECRET,
  cert: process.env.PRIVATE_KEY
})

const myProbotApp = app => {
  app.on('...')
}

probot.load(myProbotApp)

// Register the sub-app against Runkit's special Express app
const runkitApp = express(exports)
runkitApp.use(probot.server)

Let me know if that helps! I'm not sure how you can get a Webhook URL for GitHub to send webhooks to your Runkit app so that might be a challenge - very curious to hear what you do!

@JasonEtco Awesome! This helps me a lot! <3
One more question: Is it possible to load a private key from a string variable? (instead of passing in the filepath)

Is it possible to load a private key from a string variable?

Yes. In fact the cert option expects a string. We load that in from the file on disk from the private-key.ts module, but ultimately pass it to createProbot as a string. One note: it does need to be a single line string with \n instead of line breaks.

Will this help? #751

@gr2m Yeh I think the API to start Probot will definitely help with Runkit. Once we get that in Beta, I'd like to test it out in several scenarios to validate, including in the serverless extensions and in a couple of places in the Jira integration.

@tcbyrd it's out now :) wanna give it a try?

I'm getting an error related to ioredis I think:
https://runkit.com/tcbyrd/5c0e9f3b753de00018097eef
I wonder if we should have a way to not load Redis?

Can we move the import Redis from 'ioredis' from https://github.com/probot/probot/blob/9cc6131ed211a0d4e686e0a81da1897b0ff2a0e4/src/index.ts#L7
to line 133, after the if (options.redisConfig || process.env.REDIS_URL) { check? Not sure if conditional imports are supported with module imports?

@gr2m ecmascript module imports must be static and therefore they should be at the top of the file: http://exploringjs.com/es6/ch_modules.html#_imports-and-exports-must-be-at-the-top-level

Thanks Chris, I thoughts so 🤔 maybe we can move ioredis into optionalDependencies and see if that works?

@gr2m Can we maybe use an old school require() here?

or that :)

Just checking in again 😉
Now that #751 is merged, will this work in a minimalistic way in RunKit?
Looks like @tcbyrd 's runkit results in an error (https://runkit.com/tcbyrd/5c0e9f3b753de00018097eef)
Is this something I could help out with?

I did a quick test to see if this works correctly now; using @tcbyrd's code example but with a newer version of Probot, I was able to see errors in actual Probot code:

This is great - it means that now Probot code is being run, it tries to start up but is missing environment variables. One could set those (e.g. process.env.APP_ID = 123) before running Probot.run.
I'll close this out 🎉


This page can't load Google Maps correctly.

For a month or so now, an error message has been overlaid on top of this  module. Looking into it as best I could, it appears that Google is depreciating some old SDK code and it seems as though that is breaking this module.  Has anyone else experienced this or come up with a solution/upgrade to resolve it?

same issue here. don't have a solution yet though.

same, i think i have my api code setup correctly.
i have lat and lng setup correctly... i can see the map correctly momentarily - and then it gets covered with "Oops! Something went wrong. This page didn't load Google Maps correctly. See the JavaScript console for...." and I can't read anymore.
Any help would be greatly appreciated!!!

same, i think i have my api code setup correctly.
i have lat and lng setup correctly... i can see the map correctly momentarily - and then it gets covered with "Oops! Something went wrong. This page didn't load Google Maps correctly. See the JavaScript console for...." and I can't read anymore.
Any help would be greatly appreciated!!!

This "Oops!" happen to me when I didn't activate the "Maps JavaScript API" in the Google Cloud. After that it was gone

For a month or so now, an error message has been overlaid on top of this module. Looking into it as best I could, it appears that Google is depreciating some old SDK code and it seems as though that is breaking this module. Has anyone else experienced this or come up with a solution/upgrade to resolve it?

As far as I understand you need to add you credit card data to your Google Cloud account. Check the link

Hello, I have the same error message. What do we have to activate on Google settings ? We have to activate "Maps Javascript API" and what else ?


Add row position argument is not working

Calling add_row with as position argument does not produce effects. Data is always added in the last row.
tabulator.add_row(data, False, 2)

It's not index in the sense of third row. It's the row who has an index of 3. It depends what you are using as the index column. If you're using the id column then it will try to find the row whose id column has a value 3.
If you need more, I'll need more too. A clone link that demonstrates your issue will help.
Here are the docs for other possible values for index:
https://tabulator.info/docs/5.5/components#lookup

Thank you! Setted index column and now works.


Various bits of build file Gardening

This PR mostly consists of version bumps in our dependencies in anticipation of the Lift 3.3.0-RC1 release in a few weeks. There are a few minor, unrelated changes as well.

Looks like the htmlparser bump broke template processing. I'm going to revert that change.


Orange line shutdown notice

Marks shutdowns on the chart to make it clear that the slow zones didn't just disappear

resolves #218


Jenkins node issue

I have been trying to add centos node to master and i'm facing "known host" issue. Couldn't resolve the issue with the solutions found from google. Please find the screenshot attached and help me resolve the same.

Hi harini,
Can you please follow below link
https://support.cloudbees.com/hc/en-us/articles/115000073552-Host-Key-Verification-for-SSH-Agents?mobile_site=true

please copy ssh keys
or please select method called no host verify keys

Follow This Below Approach:
Node configuration:

Install java on node
create a user named as jenkins (or any name)  in node
ser the password for jenkins user.
add the user to the /etc/sudoers file as root user using  visudo commad
enable password_authentication in /etc/ssh/sshd_config
restart the service sshd restart
Server configuration:
add the user to the /etc/sudoers file as root user using  visudo commad
enable password_authentication in /etc/ssh/sshd_config
restart the service sshd restart
switch as jenkins user
try to connect to node manually
ssh <username>@<privateip> it will ask a password then enter jenkins  user password.
Note: I Think you already knew above things but please cross-check.Above process is for password authentication only.
If you find the solution for above issue please close the issue.


data_dictionary now includes naics_classification

We also read data in a simpler way.

Check failed on windows but seems due to a problem in the underlying gh action.


west.yml: Update open_amp and libmetal modules

Updates OpenAMP lib modules to integrate build fixes for zephyr
Signed-off-by: Arnaud Pouliquen<EMAIL_ADDRESS>

re-run CI w/latest PR versions

@arnopo now that the module repo changes are merged, you can update west.yml to the git SHAs for those changes.

@arnopo now that the module repo changes are merged, you can update west.yml to the git SHAs for those changes.

done


backup-cron failed to fail

A week ago tarsnap emailed me a standard notice that our balance was running low and it was time to refill. I deliberately let this run below zero and tarsnap duly disabled the ability to create or list backups:
$ tarsnap --configfile tarsnap.conf --list-archives
tarsnap: Cannot read list of archive fragments: Account balance is not positive.
tarsnap: Please add more money to your tarsnap account
tarsnap: Error listing archives
tarsnap: Error exit delayed from previous errors.
$ echo $?
0

I manually ran backup-cron in prod and confirmed the prod behavior matches: the two calls to tarsnap both printed this error message.
I consider the zero exit code indicating success to be a bug in tarsnap and will file it there, but wanted to track here so I can't forget. It seems very, very unlikely but if tarsnap wontfixes the behavior we'll have to increase the complexity of the backup script (:frowning:) to detect this text (and should probably audit tarsnap for any other uncoded failures).

I've funded the account, manually run a backup, and expect our daily backups to function normally.


PostgreSQL errors when starting WSO2IS

When we are is using postgresql 10.6 with IS-KM-5.7.0. Here we have all the tables in a single database.
The error logs in postgres db are given below.
2019-05-07 15:26:19.163 GMT [673] ERROR:  relation "sp" does not exist
2019-05-07 15:26:19.163 GMT [673] STATEMENT:  SELECT pg_get_serial_sequence($1, $2)
2019-05-07 15:26:19.163 GMT [673] ERROR:  relation "sp_inbound" does not exist
2019-05-07 15:26:19.163 GMT [673] STATEMENT:  SELECT pg_get_serial_sequence($1, $2)
2019-05-07 15:26:19.163 GMT [673] ERROR:  relation "sp" does not exist
2019-05-07 15:26:19.163 GMT [673] STATEMENT:  SELECT pg_get_serial_sequence($1, $2)
2019-05-07 15:26:19.164 GMT [673] ERROR:  relation "sp" does not exist
2019-05-07 15:26:19.164 GMT [673] STATEMENT:  SELECT pg_get_serial_sequence($1, $2)
2019-05-07 15:26:19.164 GMT [673] ERROR:  relation "sp_auth" does not exist
2019-05-07 15:26:19.164 GMT [673] STATEMENT:  SELECT pg_get_serial_sequence($1, $2)
2019-05-07 15:26:19.164 GMT [673] ERROR:  relation "sp" does not exist
2019-05-07 15:26:19.164 GMT [673] STATEMENT:  SELECT pg_get_serial_sequence($1, $2)
2019-05-07 15:26:19.164 GMT [673] ERROR:  relation "sp_claim" does not exist
2019-05-07 15:26:19.164 GMT [673] STATEMENT:  SELECT pg_get_serial_sequence($1, $2)
2019-05-07 15:26:19.165 GMT [673] ERROR:  relation "sp" does not exist
2019-05-07 15:26:19.165 GMT [673] STATEMENT:  SELECT pg_get_serial_sequence($1, $2)
2019-05-07 15:26:19.165 GMT [673] ERROR:  relation "sp_claim" does not exist
We were able to reproduce this issue when the same database is pointed in bps-datasource.xml file and master-datasource.xml file. But when we configure two different dbs in the bps-datasource file and master-datasource file, we are not getting those error logs. When we analyze the JDBC calls from the backend server, we observed that when a DB sequence is getting created, it is trying to execute the prepared statement query with a table or column that is not existing.
TID: [-1234] [] [2019-05-14 20:29:37,238] ERROR {jdbc.sqlonly} -  1. PreparedStatement.execute() SELECT pg_get_serial_sequence('sp', 'claim_mapping')
org.postgresql.util.PSQLException: ERROR: relation "sp" does not exist
at org.postgresql.core.v3.QueryExecutorImpl.receiveErrorResponse(QueryExecutorImpl.java:2440)
at org.postgresql.core.v3.QueryExecutorImpl.processResults(QueryExecutorImpl.java:2183)
at org.postgresql.core.v3.QueryExecutorImpl.execute(QueryExecutorImpl.java:308)
at org.postgresql.jdbc.PgStatement.executeInternal(PgStatement.java:441)
at org.postgresql.jdbc.PgStatement.execute(PgStatement.java:365)
at org.postgresql.jdbc.PgPreparedStatement.executeWithFlags(PgPreparedStatement.java:143)
at org.postgresql.jdbc.PgPreparedStatement.execute(PgPreparedStatement.java:132)
at net.sf.log4jdbc.PreparedStatementSpy.execute(PreparedStatementSpy.java:413)
at sun.reflect.GeneratedMethodAccessor114.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:498)
at org.apache.tomcat.jdbc.pool.StatementFacade$StatementProxy.invoke(StatementFacade.java:114)
at com.sun.proxy.$Proxy20.execute(Unknown Source)
at org.apache.openjpa.lib.jdbc.DelegatingPreparedStatement.execute(DelegatingPreparedStatement.java:367)
at org.apache.openjpa.lib.jdbc.LoggingConnectionDecorator$LoggingConnection$LoggingPreparedStatement.execute(LoggingConnectionDecorator.java:1256)
at org.apache.openjpa.lib.jdbc.DelegatingPreparedStatement.execute(DelegatingPreparedStatement.java:367)
at org.apache.openjpa.jdbc.sql.PostgresDictionary.queryOwnership(PostgresDictionary.java:465)
at org.apache.openjpa.jdbc.sql.PostgresDictionary.isOwnedSequence(PostgresDictionary.java:430)
at org.apache.openjpa.jdbc.sql.PostgresDictionary.isSystemSequence(PostgresDictionary.java:396)
at org.apache.openjpa.jdbc.schema.SchemaGenerator.generateSequences(SchemaGenerator.java:960)
at org.apache.openjpa.jdbc.schema.SchemaGenerator.generateSchema(SchemaGenerator.java:367)
When we debug the code, we observed that the error comes from this PostgresDictionary.java class [1]. From this isOwnedSequence(..) method, it getting different permutations for the name of the sequence and trying to get names for table and column from the name of the sequence. For example,
if the sequence name is reg_log_pk_seq,
then it  takes table name as  and column name as <log_pk> and execute SELECT pg_get_serial_sequence. Since there is no table exist as reg, it throws an error. Similarly, it takes different combinations and executes the query.
We tried to change the sequence name to an existing table and column. In that case also, we are getting errors since it executes  SELECT pg_get_serial_sequence query for all the permutations which may not be a table or column in the DB. But we are not getting this error when two different databases are pointed in master-datasources.xml and bps-datasources.xml.
[1]https://github.com/wso2/wso2-openjpa/blob/master/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/PostgresDictionary.java
[2]https://github.com/wso2/wso2-openjpa/blob/master/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/PostgresDictionary.java#L409

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Preloader

Is your feature request related to a problem? Please describe.
Yes. I would be adding a preloader to the site matching with the UI to increase the interest of user
Describe the solution you'd like
The preloader will be in svg with added css and would be displayed until the site loads totally
Describe alternatives you've considered
It would be done using useEffect and setTimeout functions
Additional context
Kindly assign this task to me under GSSoC'24

@Jagpreet153 Issue assigned to you

@SUGAM-ARORA  Is this preloader OK ?
https://github.com/SUGAM-ARORA/UniCollab/assets/119691733/efc37c37-5388-4ccf-9747-c5f09736da41

Merged


O365 Phishlet ADFS

I did read the readme and are aware about the part no asking for phishlet fixes However, this smell like a bug, So I give it a try:
An O365 test was done on a domain that used its own ADFS.
this ADFS domain was like:   sts.somedomain.com. the. relevant sections where changed in the o365 phislet:
Example of the lines 13 until 21 are as follows:
  #    sts.somedomain.com = adfs.example.com
  - {phish_sub: 'adfs', orig_sub: 'sts', domain: 'somedomain.com', session: true, is_landing:false}
  - {phish_sub: 'adfs', orig_sub: 'sts', domain: 'somedomain.com:443', session: true, is_landing:false}
sub_filters:
  - {triggers_on: 'login.microsoftonline.com', orig_sub: 'login', domain: 'microsoftonline.com', search: 'href="https://{hostname}', replace: 'href="https://{hostname}', mimes: ['text/html', 'application/json', 'application/javascript']}
  - {triggers_on: 'login.microsoftonline.com', orig_sub: 'login', domain: 'microsoftonline.com', search: 'https://{hostname}', replace: 'https://{hostname}', mimes: ['text/html', 'application/json', 'application/javascript'], redirect_only: true}
  # Uncomment and fill in if your target organization utilizes ADFS
  - {triggers_on: 'sts.somedomain.com', orig_sub: 'login', domain: 'microsoftonline.com', search: 'https://{hostname}', replace: 'https://{hostname}', mimes: ['text/html', 'application/json', 'application/javascript']}
auth_tokens:

The EnginX server was executed and forwards to the ADFS site. creds like account and password are intercepted. However. The following error is the result after the MFA code is submitted:
https://somedomain.com/adfs/services/trust' does not exist.

Looks something goes wrong with redirecthing.
Any solutions?

I got the same issue too

I got the same issue too

happy New Month I got an issue when i tried to test run the o365 url link i Phished, after i typed the username and hit enter this error occured i havent even been able to get to the password section and the 2FA to get the credentials on my server. can someone help out. thanks

pls any fix to this yet?


LoggingAppender - removed DCL

Avoid using deprecated newInstance() method.
Removed double checked locking pattern - it doesn't work, and unnecessary, since start() is already synchronized.

@saturnism, all of the builds fail.

Can we remove the getLogging() method?

@saturnism, can you please address the issue that @ajaaym raised?


Chromium failed to start in 7.0.2

Gutenberg version: 7.0.2
docker-compose version 1.29.1, build c34c88b2
docker version 20.10.6, build 370c289

This project works locally but when I try to run this on a t3a.medium AWS instance running Docker I get this error:
2021-08-25T17:09:04 - {"level":"error","ts":1629907744.42355,"logger":"api","msg":"handle request: convert HTML to PDF: convert to PDF: chromium PDF: chrome failed to start:\n","trace":"9fec0f04-09a4-4785-ab8c-3337652a519b","remote_ip":"<IP_ADDRESS>","host":"gotenberg:7777","uri":"/forms/chromium/convert/html","method":"POST","path":"/forms/chromium/convert/html","referer":"","user_agent":"python-httpx/0.14.3","status":500,"latency":10003991770,"latency_human":"10.00399177s","bytes_in":361,"bytes_out":21}
I tried increasing mem limits in dc file like so but it doesn't seem to work. Any tips on how to debug this?
 gotenberg:
    image: gotenberg/gotenberg:7
    command: gotenberg --api-port=7777 --api-process-timeout=10s --libreoffice-disable-routes --log-level=debug
    ports:
      - 7777:7777
    deploy:
      resources:
        limits:
          memory: 2048M
        reservations:
          memory: 1024M

Hello @mkotsalainen,
It looks like it's related to the chromedp/chromedp library.
Is your AWS instance running on an arch64 architecture?


subscriber events: Change schema for peer_add/peer_remove.

We now can send an implied matrix of user/stream tuples
for peer_add and peer_remove events.
The client code basically does this:
for stream_id in event['stream_ids']:
    for user_id in event['user_ids']:
        update_sub(stream_id, user_id)

We used to send individual events, which gets real
expensive when you are creating new streams. For
the case of copy-to-stream case, we should see
events go from U to 1, where U is the number of users
added.
Note that we don't yet fully optimize the potential
of this schema.  For adding a new user with lots
of default streams, we still send S peer_add events.

Merged, thanks @showell!


Clicking an input throws error in event-mixin.js

It's not easy to throw a test page up for 2.0 stuff atm, but I have a simple input on the page. When I click it, event-mixin.js:71 throws an error.

cc @azakus so you're aware

This should be fixed in the latest commit (i stopped seeing it after a bower update)

+10

On Sep 20, 2016, at 3:50 PM, Monica Dinculescu<EMAIL_ADDRESS>wrote:
This should be fixed in the latest commit (i stopped seeing it after a bower update)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/Polymer/polymer/issues/3929#issuecomment-248413994, or mute the thread https://github.com/notifications/unsubscribe-auth/AHxnvgy90jT_yvDiaj45vIqMz9AzE307ks5qsDj1gaJpZM4J5kzn.

Latest shadydom commit?

@ebidel yup, i think https://github.com/webcomponents/shadydom/commit/5169cad1e216f83fd033f66e4799e019509df62d

Seems gtg: http://jsbin.com/vayebit/edit?html,console,output. Please re-open if the issue resurfaces.


[BUG] CheckAreasAvailableOnMenu test script is failing as element paths are changed

We are trying to execute CheckAreasAvailableOnMenu test script in EasyRepro, Which is failing due to few elements are not identifiable.
We verified these element’s CSS/XPATHs in the O365 app, unable to identify these elements. Is there a way to fix these elements to pass this test script.
We are using EasyRepro NuGet package in our custom framework, We could not override these elements or methods from our framework as these are part of static or internal methods in EasyRepro.
Could you please help, if there is another way to fix this issue or to update CSS/XPaths of these elements in CheckAreasAvailableOnMenu test script.

@gurunadhd Could you add mode details: code, screenshots, stacktrace?

@Shakevg
CheckAreasAvailableOnMenu  test script in the below snapshot is failing.

xrmBrowser.Navigation.GetAreas() is returning. When i debug further, Reference.Navigation.TopLevelItem, Reference.Navigation.ActionButtonContainer and other elements in OpenMenu functions are returning null and I could not search these elements in the Dynamic 365 UI.

Hi All, anyone else facing this issue, any help here is appreciated.

@gurunadhd Hi, did you get this issue resolved? I am facing the same. Thanks

@gurunadhd Hi, did you get this issue resolved? I am facing the same. Thanks

Update: Sorted this out. Issue was, I was trying to run Web tests using UCI app. When provided the correct URL for web it went okay. Likewise for UCI app too.


Install error from dependency

When attempting to install version 1.1 of this package on laravel/framework v6.12.0, the package is not installable. I'm not an expert in this, but it seems that illuminate/support requires v1.x.x of nesbot/carbon but laravel/framework needs v2.x.x of nesbot/carbon. Additionally, I believe that illuminate/support was removed and cannot be installed alongside with laravel/framework v6.12.0.
I can post a minor part of composer install logs of this but this seems to be reproducible by making a new laravel v6.12.0 project and trying to install the package. I've also tried to downgrade to a few earlier versions of laravel/framework, but it still does not install
Thanks!

@alvin8t You need to update nesbot/carbon to 2.x since it is not supported from Laravel 6.x
More info here: https://laravel.com/docs/6.x/upgrade#carbon-support

Hey,
Thanks a lot. It’ll be helpful if you can send a PR. I was at some point also working on an entirely newer version for Laravel 8.
Not sure how feasible it is now that I am so busy.
Regards,
Neo Ighodaro
SoftwareEngineer •Twitter •www.neoi.sh
On 19. Feb 2021, 08:37 +0100, Daniel S. Billing<EMAIL_ADDRESS>wrote:

@alvin8t You need to update nesbot/carbon to 2.x since it is not supported from Laravel 6.x
More info here: https://laravel.com/docs/6.x/upgrade#carbon-support
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Update AVS Client to 1.9

Can you update the client SDK to version 1.9?
Thank you

Hi @Knoppix1,
This pull request is merging in application specific files from another project. As this is not within the scope of the SDK, I will be closing this pull request. Please re-open if you have further questions or comments.


Update frontend

@bajtos , tt's ready to test and review. I added for tests commit, because travis switch to 0.11.14 version by default, and we have prebuild binaries only for uncompatible 0.11.13 (I can't publish binaries for 0.11.14 automatically, because there is no win x64 release version). Also we wait 0.10.36.

@3y3 What's the best way how to review this patch? What are the important things to check?
The last time I was upgrading front-end, I reviewed the diff of the old and the new version to catch any code changes that may affect the front-end. Have you done the same? If not, how did you verify that no regression is introduced by the upgrade?

The last time I was upgrading front-end, I reviewed the diff of the old and the new version to catch any code changes that may affect the front-end. Have you done the same?

This is impossible - big part of files removed, other part renamed and replaced. I only tested that existed interface works correctly.
So I was digging each problem, that I found.
One important thing in this path - I don't use overrides for web socket.
(Look at Use default web socket to connect to inspector backend (38e3b43)).
There used default socket connection, therefore old way to open console was deprecated.
(Look atExtend protocol by Console.showConsole(4fd2ad3))
Profiling API converted to async style (I also updated v8-debug API with registerCommandAsync)
Each other commit adds new implementation of existed overriding and removes old impl.
Last commit (Deprecate unuseful front-end changes) removes all changes that are not useful for new frontend, in other words I can't find how to use this changes.
Finally now we need to send main context info to frontend (now devTools works with some contexts in one window (main + workers))
(Look at Send page context to frontend (e231659))
Please check that SaveLiveEdit works correctly for you. This is one of very problematic places.

I reviewed all commits except the front-end update itself, you did a great job!
See my comments above, most of them are concerned about coding style.
I did not test the new version yet, I'll try to do that ASAP, ideally today/tomorrow.

Please check that SaveLiveEdit works correctly for you. This is one of very problematic places.

It does not work :(
Also the page title is wrong, I see Developer Tools - file://<path-to-app.js>

Rebased to master.
@jakub-g , can you please look at this pr. There exists bug for you.
Now our inspector link is http://examlpe.com:8080/debug?port=5858&ws=example.com:8080
This link doesn't work on Windows with node-debug command for reason of unescaped &.
Original issue in opener https://github.com/domenic/opener/issues/9 (It will be nice, if you fix also this issue)
But current issue for you - unhandled error event:
D:\GIT\node-inspector [frontend]> node bin/node-debug index
Node Inspector is now available from http://<IP_ADDRESS>:8080/debug?ws=<IP_ADDRESS>:80
80&port=5858
Debugging `index`

Command failed: cmd /c start "" http://<IP_ADDRESS>:8080/debug?ws=<IP_ADDRESS>:8080&po
rt=5858
"port" ?? ?????? ???????? ??? ?????
????????, ??????? ?????? ??? ???????? ????.

Please open the URL manually in Chrome/Chromium/Opera or similar browser
Debugger listening on port 5858
events.js:73
      throw er; // Unhandled 'error' event
            ^
Error: read ECONNRESET
    at exports._errnoException (util.js:732:11)
    at TCP.onread (net.js:530:26)

@bajtos , now I will work on saveLiveEdit. Will be nice if you provide me some failure test.
(But in current time I can live without it)
@marcominetti, @auchenberg , I see that you already rebased to this pr. Any thoughts about saveLiveEdit?

@3y3 what's wrong with saveLiveEdit? I've never tested it. I usually disable inspector's liveedit cause I'm implementing live coding at application server layer in my project.

@3y3 I haven't had a look at saveLiveEdit yet.

@3y3 I will try to find some time and have a look soon

Hi there, I've just spend a few time onbuilding awareness of the saveLiveEdit related stuff.
It seems to me that the live edit triggers (within the backend lib):

generate a request for the debugger client of type "changelive" through DebuggerAgent.setScriptSource
1.1. try to update call frames for breakpoints/frontend through DebuggerAgent._handleChangeLiveOrRestartFrameResponse
1.2. flush new source code to disk through DebuggerAgent._persistScriptChanges

But where is or should be implemented 'changelive' request handler in debugger? Is it "changelive" a V8 debugger feature?

Is it "changelive" a V8 debugger feature?

Yes. Look at debug-debugger

Cool. Great. I'll delve into it...

@3y3 I am not sure what do you need from me at this point. Do you need me to test the updated front-end and especially live edit? Review the modified code? Something else?

@3y3 could you also squash "fixup" commits? It will make my review easier.

Hi there, to start working on the plugin system #582 I'm reviewing the @3y3 pulled front-end branch. Is there anyone working on that or an updated fork? I started working on cleaning the UI from useless and not-working stuff (and fixed two leftovers causing exceptions).

Ok. I finish inspection of live edit. There is no new problems with this feature, i.e. it works for me on some scripts. But for some other (for example periodicConsoleLog) it doesn't work correctly (I can reproduce this behavior on 0.7.4), so this problem is out of scope of this pr (and maybe out of scope of node-inspector).
So, I rebased pr to master. I also added fix for repeatMessageCounter.
@bajtos , please review.

I started working on cleaning the UI from useless and not-working stuff

@marcominetti , you can prepare this as next pr after frontend will be merged.
So I think that frontend is ready to merge and we'll do it soon.

@3y3 new versions of opener and biased-opener were released, you will get them after npm install. I keep in mind unhandled error, will come back to it when I have a spare minute.

@jakub-g , good news thank you!

I reviewed again. Unfortunately my previous comments were lost, so I did only a quick check for obvious issues.
Note that I don't have enough time to test that the new front-end is working well, especially in edge cases, I'll leave that out to other volunteers (probably after the change has landed on master).
For posterity, I am cross-posting my new comments below.

front-end-node/NodeInspectorOverrides.js in 33b5709

Is it worth adding a new method for this? IMO it's ok to define these overrides directly in the constructor.
WebInspector.NodeInspectorOverrides = function() {
   this._overridenStrings = {
     '(no domain)' : '(core modules)'
   };
   // etc.
};

tools/generate-front-end.js in ea513f0

This is very weird. extedModulesBy is reading ./front-end/inspector.json and here you are overwriting that file.
Can we keep the auto-generated files in ./front-end-node please? It will also make it easier to compare our version with the original version (IMO).

lib/ConsoleAgent.js in 39329bf

AFAICT, this commit is making two changes:

Same messages are no longer grouped together. Is this because the front-end performs grouping now?
clearMessages is no longer cleaning the cache. I am wondering why we cannot clear the cache? Is there a way how to access a cleared message from the UI? My concern is that the cache can grow too much during a longer debugging session.

I see this message when I first load node-inspector in chrome. (I'm using iojs 1.6.2.)
 Your Node version (undefined) has a partial support of profiler.
 The stack frames tree doesn't show all stack frames due to low sampling rate.
 The profiling data is incomplete and may show misleading results.
 Update Node to 0.11.13 or newer to get full support.

Sometimes when I stop on a break point and "Resume execution with F8" it hops to the console tab but doesn't always go back when it hits another breakpoint, I then see this message
Runtime.getProperties failed.
Object #-3# not found

but I can continue to debug so its not major issue.
Finally, CPU profiling is recording data but you cant leave the profile tab because all tabs become disabled once recording has started.
Otherwise everything is working great, I can hover over arrays + strings and they are shown correctly in the popup and I can also work in the console.
I'm going to keep using this version and will report anything else I find here.
Thanks doing the work on this, so far so great ;)

hi @3y3, any news on latest feedback from @bajtos?

@bajtos , so I rebased lot of commits for

This is very weird. extedModulesBy is reading ./front-end/inspector.json and here you are overwriting that file.

All other changes I maked in fixups:

Same messages are no longer grouped together. Is this because the front-end performs grouping now?

clearMessages is no longer cleaning the cache. I am wondering why we cannot clear the cache? Is there a way how to access a cleared message from the UI? My concern is that the cache can grow too much during a longer debugging session.

The second case was a bug - fixed in 71cb859
Last two fixups is for

Is it worth adding a new method for this? IMO it's ok to define these overrides directly in the constructor.

I'll squash fixups after review (or, if you want, before it).

@3y3 great! This patch is good to be landed, please don't forget to squash the fixup commits before merging.

@3y3 Could you please also check why the build is failing? I cannot tell whether the failures are related to this pull request or not. I am fine with landing this patch as it is and investigating build failures later.

@bajtos , ok I'll fix test for console API and merge this pr.

Great! I'll pull request all fixes...
@3y3 my fix/xxx branches are not anymore in sync with master but they were synced with your frontend branch? how can i resync against master? (sorry but i'm still new in prs... oops)

@marcominetti , I merged this pr. You are free to open your prs now! Please start from smaller =)

@marcominetti , you can try next:
If node-inspector/node-inspector is your upstream remote
git fetch upstream
git checkout fix/...
git rebase -i upstream master

There you receive a list of your commits ahead of master. Comment all that in not a part of your changes
# This is changes from outdated frontend branch. Comment all they
#pick f69a4be Send page context to frontend
#pick 858f96d Extend noop commands list
#pick 6e4cf12 Added PageAgent canEmulate and canScreencast callbacks
#pick bf39568 Disable unusable tabs
#pick bb51b00 Allow to save modified files
#pick a85761f Reload frontend on detach
#pick ef1a754 Fix ProfilerAgent relative to new protocol.json
#pick b15f403 Fix HeapProfilerAgent relative to new protocol.json
#pick 60bc509 Deprecate unuseful front-end changes
#pick 6479407 Fix console messageRepeatCounter

pick 6479407 LOOK HERE.

So, you are synced with my outdated frontend and there can be some small conflicts during rebase that you'll need to fix manually.

Ok, quite standard way then... ;)


Add configuration setting UNIX_SOCKET_GROUP

Feature Description
Using a unix socket to listen on is nice because it grants additional granularity over access via file system permissions.
This is currently hampered by the inability of specifying the owning group for the unix socket file.
I currently have to use UNIX_SOCKET_PERMISSIONS set to 666 instead of 660 because of this – not much of an improvement when compared to a TCP socket on a loopback device.
Adding something like UNIX_SOCKET_GROUP would improve this situation.
Screenshots
No response

Couldn't you use one of the following:

set the group sticky bit on the directory that the unix socket files are created in.
adjust how you run gitea so you run newgrp <groupname> before you run gitea

So idea nr. 1 is basically:
(assuming socket is /run/gitea/gitea.sock)
(assuming webserver runs with group nginx)
chown gitea:nginx /run/gitea/
chmod g+s /run/gitea/

Now, when gitea starts, the socket file will be created with user=gitea and group=nginx.
While this is nice, I think it would still be better to have the proposed option UNIX_SOCKET_GROUP.

Ok I tried this and it doesn't work.
Seems to be a pretty new phenomenon that gitea overwrites all permissions and user/group on startup.
I made gitea depend on another service, this other service did the steps from my previous comment but then gitea starts and the directory is again owned by gitea:gitea and the socket file too.


Ellipsis property added for button in context-switcher

Files changed
ui/components/header.js

Component
k8sContextMenu

@leecalcote @harkiratsm Please review this PR and tell me any changes that I need to make to this
Result:

Signed-off-by: deepakdinesh1123<EMAIL_ADDRESS>Notes for Reviewers
This PR fixes #6662
Signed commits

[x] Yes, I signed my commits.

@deepakdinesh1123 Thank you for the PR, please add a screenshot of the change in the PR description.

Yay @deepakdinesh1123  :balloon:


Fix ParetoNBDModel Docstring Typos

While working on https://github.com/pymc-labs/pymc-marketing/issues/527, I noticed some docstring typos and syntax errors for ParetoNBDModel. The docstrings in BetaGeoModel follow an identical convention and are error-free, so this would be a very quick PR fix.

I took a look at the docstrings and I fixed a few typos but I'm not sure if I got everything
I'll open a PR anyway so that you can take a look

I took a look at the docstrings and I fixed a few typos but I'm not sure if I got everything I'll open a PR anyway so that you can take a look

Thanks for the PR! We can work on iterations :)


Cleanup output, fix/catch some errors

Cleaned/prettied up output and added more info
Added most catches needed with printouts of what the issue is so there won't be traceback errors
Remove comma check in final_url =  re.findall(r'file: "(.+?)"', video_webpage) as some videos don't have the trailing comma after the file url, causing the fetch to fail
Added some comments

Fixes issue #3

Thanks :smile: :+1:


Leverage MSI for OpenAI Access

AZD Terraform template is using OpenAI Keys
Leverage MSI if possible

Still seeing keys

still not working with keys

Give yourself "Cognitive Services OpenAI User" permission in the target OpenAI resource.

Typically those sort of roles are assigned in the IAC, is that not possible for this one?

Yes, but you need to run "azd up" first to assign those. This repo is still not ready with that, so the test locally the app the only possibility is to manually assign yourself that role.

Ah okay, could you provide a scrip then? https://github.com/Azure-Samples/rag-postgres-openai-python/blob/main/scripts/setup_openai_role.sh
Could make it slightly easier?

Can you please reopen until either myself, @kristenwomack or @codebytes have tested. Thank you!


fix: swap src dst port from flow description

TODO: test ip filter with port parameter, and validate the network traffic.


fix cookie share example in docs (and make a bit verbose)

What these changes does?
Fix sample code in client docs
How to test your changes?
copy-paste-run
Related issue number
I believe this is related to misunderstanding in #815
Checklist

[ ] Code is written and well
[ ] Tests for the changes are provided
[x] Documentation reflects the changes

+1

Cool!
Please merge

bump


Alerting on slack when there is no alerts to be send

Resolves #133

Send test message on the CLI when there is no other alert to send on slack

TBD

Add/Do tests to understand it works correctly

This was merged here.


Normal User cannot see sent invitations

While running the development environment I noticed that an admin can access his list of invitees while accessing zulip online caused me to notice that there is no such privilege for normal users.

@zulipbot claim

Any sort of help will be appreciated.

@zulipbot claim

@showell Please let me know if I can work on this issue as my other 2 PRs are getting reviewed.

But Why do normal user need to see All the Invitations ?

But Why do normal user need to see All the Invitations ?

The user should atleast be able to see all the invitations he sent.

yes Now it make sense

@arunikayadav42 I officially assigned you this issue.  It might help to post a screenshot of the current admin UI here for context (like the one I posted to chat).
It's also good to post a link to any relevant chat discussions when you create an issue.
Thanks again for raising this issue, and good luck with the fix!

I believe this is the admin UI. I'll like to help on this issue too. We'll create a similar one for the user

I am already halfway through this, thanks for the help. You can take up some other issue or open up your own issue.

I am working on this issue.

@zulipbot  claim


[TASK] Label for Signed and Submitted Checkbox

Estimated story points: 1
Description

The Submitted and the Signed Checkbox on the edit contract page should have a label text.

Acceptance criteria

The Submitted and the Signed Checkbox on the edit contract page have a label text and a click on the text toggles the checkbox.

If the label wraps around the checkbox the checkbox is getting checked on click of the label, therefore no for attribute is needed which would not work here

time: 1h


docs(@tanstack/lit-table): Add example for virtualized rows

Adds an example of virtualized rows using the @lit-labs/virtualizer package.

We'll want to use the TanStack Virtual integration when that's released, right? What's the status of that?

Yes, even though it could be useful anyway, as the lit virtualizer is the solution provided by Lit maintainers.
There is a small issue with the Lit virtualizer that is still WIP, but I will definitely create another example using it when it is merged

@kadoshms Only reason I help off merging is I saw TanStack Lit Virtual had shipped. We can merge this, but we should include a TanStack Virtual alternative example hopefully too. Maybe full row + column virtualization with TanStack?

Yeah sure, I will add those once I'm back from paternity leave


Add issue and contributing template

Add issue and contributing template

Issue template was added, I'll add the other ones.


Solidus client integration

There should also be a comparable method for .get, here's what I have to do if I want to use universe.js as an API wrapper, but don't have a template to render (doing DOM scripting instead):
  fanclub.render({
    resources: {
      links: fanclub.resource('/events?schedule=active&scope=upcoming')
    },
    preprocessor: function ( data ){
      ...
    }
  }, function(){
    // nada
  });

Done, you can now universe.get(endpoint, params, callback). And I added the params argument to .post too: universe.post(endpoint, params, body, callback).

Development continues on #5, but we don't delete this branch yet, some sites use it directly.


macos x86_64 nightly testing flake fix

All github hosted macos machines share the same IP and github does per-IP counting for rate limiting.
This is why some curl requests do not succeed.
See issue https://github.com/actions/runner-images/issues/602

Adding our GITHUB_TOKEN to authenticate the request solved the issue.

All github hosted macos machines share the same IP and github does per-IP counting for rate limiting.

Wow! That's really surprising!


Mac Compilation

How do I build the dot files (The readme text file in sync seems incorrect, or I can't find the dot files and sync next.py files

The m.py script builds. You'll need a specific version of sdcc as well. I haven't built on mac, but it should work..

Thanks


factory: fix time based resync to use queue instead of calling sync directly

This prevents factory controllers to run two sync()'s in parallel. Instead the time based resyncs are now handled via queue.

informer resyncs replay the current state of the objects.  This means

you cannot optimized a zero-diff to no update
you are required to have informers
even if you have informers, you must be sure that they have contents
informers with different resync times result in many notifications
you aren't guaranteed to avoid numerous queuing for every configmap for instance

The time based trigger resolves all of these issues.  It is a different mechanism for "single item queue" controllers as a sub-class of other controllers.

/approve
I'm ok with taking fixes later.


Feature gui

What does this PR do?
This PR adds a graphical user interface to the system. The first commits separate the network logic from the ui logic. Then the gui classes are added. The last commits fix the build scripts for the entire system
Where should the reviewer start?
The reviews should start with the CryptoClient class and should check the new method signatures. Than the ConsoleLauncher class should be checked to ensure that the console user interface still works properly. Finally the reviews should analyze the gui classes.
How should this be manually tested?

All methods should be tested from the console interface (use script ./start.sh to start the client)
All methods should be tested from graphical interface (use script ./startGui.sh to start the client)

Any background context you want to provide?
The graphical user interface uses JavaFX with fxml files
What are the relevant tickets?
Closes #16
Screenshots

Changes done

Amazing job!!! Thanks! Very well done. :D 👍


Merge remote changes

DO NOT MERGE THIS BRANCH UNTIL AFTER THE FOLLOWING APPS HAVE BEEN DEPLOYED WITH THE LATEST CHANGES:

nimbusprayer
nimbusprayer-production
papiadmin-staging
papiadmin-production
devotional-service-staging
devotional-service-production
notification-service-staging
notification-service-prod

@mattmueller It should be safe to merge this in now, right?

I would sure hope so.


I'm trying to encrypt and decrypt with rsautl using openssl. I'm geting this information : rsautl: Use -help for summary

Hello,
I'm trying to encrypt and decrypt with rsautl using openssl. I'm geting this information : rsautl: Use -help for summary
ubuntu@ubuntu:~/test$ openssl rsautl -encrypt -pubin -inkey publiclovely.pud -in alice_hash.txt -out alice_ hash_encrypted.txt
rsautl: Use -help for summary.
the file alice_hash_encrypted.txt was created
--- try also to decrypt the file :
ubuntu@ubuntu:~/test$ openssl rsautl -decrypt -inkey lovely.priv -in alice_hash_encrypted.txt -out alice_ hash_decrypted.txt
rsautl: Use -help for summary.
And no file created
please help"

Hi,
I did find out what was the problems
Thank you


AD Starter Template

Starter template using Next.js and Material UI along with Tailwind CSS for styling. Not a final decision made on this yet, so leaving this as a draft PR

Works great!


Inherit from OdinEditor if ODIN_INSPECTOR is defined

Odin is a fantastic serialiser/editor overhaul, but anything inheriting from a Mirror class (NetworkBehaviour, etc) loses all of the nice advantages it provides like better List/Array support, dropdowns, etc.
I'd like to propose doing the following on all the custom inspectors:
#if ODIN_INSPECTOR
    public class NetworkManagerEditor : Sirenix.OdinInspector.Editor.OdinEditor
#else
    public class NetworkManagerEditor : Editor
#endif

This doesn't break anything, it just allows Odin to still provide the custom inspector overhauls for some of Unity's build in property drawers, and allows use of Odin's attribute system for quick implementation of custom editor bits and pieces. This won't cause any issues for those without Odin as they won't have ODIN_INSPECTOR set in their defines.
From what I can see, this would need to be done for the following:

NetworkBehaviourInspector (bit of inconsistent naming there?)
NetworkIdentityEditor
NetworkManagerEditor

+1 But maybe for Naughty Attributes? Making the NetworkBehaviourInspector inherit from Naughty Attributes's Base Editor class didn't do anything. Would require more investigation.

Why not both? Odin is NA on crack. If NA has an ifdef we can do both basically.

Ah, it does take a little more for the NetworkBehaviourInspector as it doesn't use the standard system for drawing fields:
http://www.sirenix.net/odininspector/faq/48/can-i-write-my-own-custom-editor-code-using-odin

@paulpach I could yeah, but I figured it would be more sensible to suggest adding support for one of Unity's most popular editor assets into the library instead.

Popular or not, that's opening Pandora's box for us.  Once we do it for one or two, then every asset publisher in the store will expect us to do it for them too.  Odin will need to produce an add-on to their asset, that they get revenue from, for a Mirror integration, just as the other asset publishers have done that have Mirror integrations.

See my Addendum above.


Add a few messages for the bot

Just added a few messages that are hopefully fun.
Also trying out pull requests so that I might be able to help open source projects. Let me know if I should be doing something different in the workflow.
Ben

Hey @Sycha, those are fantastic!  And you did everything just perfectly 👍 Congratulations on your first PR! 🥳


change fo to for in line 9

@vacuumboots
Good catch, but I think I meant "of" and not "for." I'll go ahead and fix it a commit a change here in a minute.


Fix scale mode

Imagine mode is a string: https://github.com/avalanche123/Imagine/blob/develop/lib/Imagine/Image/ManipulatorInterface.php#L24
Tasks:

[ ] test coverage
[ ] gather feedback for my changes
[x] added changelog line

Informations:

Q
A

Tests pass?
none

Fixed tickets
none

BC Breaks
no

Doc
https://github.com/sulu-io/docs/blob/master/detail-specification/500-assets/DET-507-ImageConverter.md#user-content-scale

Because "mode" is a string?

@dantleech righty right :)

ack


Abort on Reply

I have a permanent interceptor running using .permanent(). My code looks something like this:
Nock('the_host_name_here', { allowUnmocked: true })
.persist()
.post('/foo', function (body) {
    var zip = body.address.postalCode;
    zip = zip.slice(0, 5);
    zip = parseInt(zip, 10);

    return validZips.indexOf(zip) === -1;
})
.reply(422, { message: 'zip invalid' })
.post('/foo', function (body) {
    var line1 = body.address.line1;
    return line1.search(/\d+\W+Story\W+Rd./gi) >= 0;
})
.reply(200, { message: 'line 1 invalid' });

I was expecting that if the first .post returned true, then the second .post wouldn't even be called because the incoming request had already been handled. Is there a built in way to manage this or is this a feature that can be added?
As you can see, I need to be able to mix different status code responses based on some validation rules and I didn't see a way to respond with a dynamic status code.

It all seams to work as expected to me. The following script works as expected:
var nock = require('nock');
var request = require('request');

var validZips = [1,2,3];

nock('http://zzzzz.abc')
// .persist()
.post('/foo', function (body) {
  var zip = body.address.postalCode;
  zip = zip.slice(0, 5);
  zip = parseInt(zip, 10);

  return validZips.indexOf(zip) === -1;
})
.reply(422, { message: 'zip invalid' })
.post('/foo', function (body) {
    return true;
})
.reply(200, { message: 'line 1 invalid' });

request.post({
  url: 'http://zzzzz.abc/foo',
  json: {
    address: {
      postalCode: '1',
      line1: 'line one'
    }
  }
}, function(err, res, body) {
  if (err) throw err;
  console.log('response status code was: ', res.statusCode);
});

If I run this, this yields:
response status code was:  200

And if I change the postal code to an invalid one, say "4", it yields:
response status code was:  422

If you put logs in each of the .post() handlers, you'll see that both of them are called. Is there a way to prevent that? I'd like to have the first block validate the payload and if that fails, don't run any of the other matching .post methods.

Hmmm... the way that it works is by trying to match all unused matchers. When a matcher matches, it gets executed. If not, it tries to match the next.
I'm not sure what I could do to help you...

Could you clear the list of possible matches after the first match? You don't want to reply more than once anyway. Looking through the code I notice that you keep a list of possible matches; could you clear that list? Just a broad suggestion; not sure if it's possible.

Oh, I think I understand what you want.
Is it possible for you to try using nock.restore() after you get a response on the client?

No because if I restore then the interception goes away, I'm using persist() because I want nock running as long as the server is alive.

@arb Hmmm...
I think I have 2 alternatives for solving this:
a) Instead of hard-coding the status code, defining a function that returns the proper status code
b) Instead of having 2 separate functions, one for the status code and one for the body, just one function that returns [status, body].
It all boils down into turning nock into a dynamic "server". Would this solve your case?

Yea that is what I was trying to do with nock, use it as a standing, dynamic mock server that stays active. Of those two options, B seems like it would give the most flexibility with the least amount of development effort.

Addressed in #296


Add Code Sponsor to Advertising case studies

🎉

Is there a case study or some kind of white or greenpaper shared here? Sorry if I missed something, but it seems it'd be infinitesimally more helpful to see how individuals used specific products at the stand. Here's an example case study I put together when I monetized a new website using AdSense as an example of what I'd be expecting to see as a user:
https://comfusion.io/clients/chicago-gang-history/


Invalid indentation in ner_manager.py

https://github.com/piisa/muliwai/blob/main/ner_manager.py#L364 leads to invalid indentation
def spacy_ner(...):
      # ...
      if  nlp is None:
      #init spacy pipeline      # <--- this indent (possibly below) needs to be fixed!
      if src_lang == 'en':
        nlp = self.en_spacy_nlp
      # ...

Thanks. Unfortunately I couldn't get detect_ner_with_hf_model() to work with GPU. It's definitely not anything on my end. Can you check what's going on?
I'm calling
apply_anonymization("Bob and Amy are eating apples in Jack's home.", device="cuda")["text"]

>> RuntimeError: Expected all tensors to be on the same device, but found at least two devices, cuda:0 and cpu! (when checking argument for argument index in method wrapper__index_select)

The apply_anonymization function is pretty much the same as the one in your demo code:
def apply_anonymization(
    sentence: str,
    lang_id: str = "en",
    context_window: int = 20,
    anonymize_condition = True,
    tag_type = {'IP_ADDRESS', 'KEY', 'ID', 'PHONE', 'USER', 'EMAIL', 'LICENSE_PLATE', 'PERSON'} ,
    device: str = "cpu",
) -> str:
    """
    Params:
    ==================
    sentence: str, the sentence to be anonymized
    lang_id: str, the language id of the sentence
    context_window: int, the context window size
    anonymize_condition: function, the anonymization condition
    tag_type: iterable, the tag types of the anonymization. By default: {'IP_ADDRESS', 'KEY', 'ID', 'PHONE', 'USER', 'EMAIL', 'LICENSE_PLATE', 'PERSON'}
    device: cpu or cuda:{device_id}

    """

    if tag_type == None:
        tag_type = regex_rulebase.keys()

    lang_id = lang_id.split("_")[0]

    ner_ids = detect_ner_with_regex_and_context(
        sentence=sentence,
        src_lang=lang_id,
        context_window=context_window,
        tag_type=tag_type,
    )

    ner_persons = detect_ner_with_hf_model(
        sentence=sentence,
        src_lang=lang_id,
        device=device,
    )

    ner = list(set(ner_ids + ner_persons))
    ner.sort(key=lambda a: a[1])

    if anonymize_condition:
        new_sentence, new_ner, _ = augment_anonymize(sentence, lang_id, ner, )
        doc = {'text': new_sentence, 'ner': new_ner, 'orig_text': sentence, 'orig_ner': ner}
    else:
        new_sentence = sentence
        doc = {'text': new_sentence, 'ner': ner}

    return doc

Try it now. It was exepcting "cuda:0" or some other device number. I fixed it so it can accept just "cuda".  This is in notebook:
https://colab.research.google.com/drive/1olv6IMEP5SkwJb8CFyR2aZV19_9XdlZp#scrollTo=bsf83d-WFoNX

Yep this works, thanks for making the update!


[vcpkg baseline] Fix libxmu dep

missed libxext as a dependency. Thought it would be added through libxt

Thanks!

Thanks!


kata-deploy: add PodOverhead to RuntimeClasses

As of 1.18 this feature is Beta. We will need this applied if we are
running utilizing sandbox cgroups only (this should really be default).
We'll want to iterate on the most appropriate values going forward, but this is a safe starting point.
Fixes: #995
Signed-off-by: Eric Ernst<EMAIL_ADDRESS>

/test
/cc @likebreath @amshinde @jcvenegas PTAL

LGTM. Thanks, @egernst . Can't ACK though. ;)

Also, I am wondering how will the CI cover the changes to kata-deploy? I thought it is now only validated manually (during each release).

/AzurePipelines run

/test kata-deploy

@jcvenegas You have any insights into the obs failure?

@egernst not really, @devimc  is having some similar issues with another PR, I think he is debugging what grep is making file this.

@jcvenegas @devimc @chavafg : any updates on the failing job? I'd like to get this PR landed.
If necessary, should we mark that failing job non-required?

seems that @devimc fixed in #1082, merged it and let's restart this one.

/test kata-deploy

/AzurePipelines run


[toggo] Fix _VALID_URL to support toggolino

adding support for "toggolino" videos

Before submitting a pull request make sure you have:

[x] At least skimmed through contributing guidelines including yt-dlp coding conventions
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8 and ran relevant tests

In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[x] Fix or improvement to an extractor (Make sure to add/update tests)
[ ] New extractor (Piracy websites will not be accepted)
[ ] Core bug fix/improvement
[ ] New feature (It is strongly recommended to open an issue first)

Description of your pull request and other information
Explanation of your pull request in arbitrary form goes here. Please make sure the description explains the purpose and effect of your pull request and is worded well enough to be understood. Provide as much context and examples as possible.

Add a test


Correct the spelling of CocoaPods in README

This pull requests corrects the spelling of CocoaPods 🤓
https://github.com/CocoaPods/shared_resources/tree/master/media
One day I’ll make a bot that looks through the READMEs of all Pods, looks to see if it uses “Cocoapods” and PRs “CocoaPods” :D— Ørta (@orta) February 10, 2016

Current coverage is 23.18%

Merging #15 into master will not affect coverage as of 6d60b1b

@@            master     #15   diff @@
======================================
  Files           12      12
  Stmts          440     440
  Branches         0       0
  Methods          0
======================================
  Hit            102     102
  Partial          0       0
  Missed         338     338

Review entire Coverage Diff as of 6d60b1b

Powered by Codecov. Updated on successful CI builds.


Gaana not working after installing Carstream

First of all... Thanks bro for a such a beautiful app....  my best wishes to you...
problem is : Gaanaa is not working after using CarStream app..

Whats the exact issue ?

After installing Carstream, everything is perfect except Gaana.  Once I start gaana, it is giving error message like " some internal issue with this application"...
Later, I uninstalled CarStream and androidauto, then again androidauto was installed.
Then gaana is working fine..
simultaniously Carstream and Gaana not working

Pls give me solution


Fix for iPad sizing

If shown in a popover, the view thinks it's width is 768.  The size is setup in the viewWillAppear so this fixes it.

Hooray!  I was hoping to see this come through.  I need this for a popover also.

:beers: :grimacing: @RyanBertrand


[ACCF] No villagers found.

I want to change a villager in my save : he's inbox and I want to make it back to normal. But when I load my save, ACSE does'nt find any villager in the villagers tab.

This already exists in code. I will be releasing a 0.5.0 version shortly that will add this feature.


[BUG] All VCS files included in sdist in case of uncommitted changes (WITHOUT setuptools-scm)

setuptools version
65.3.0
Python version
3.10.5
OS
openSUSE Leap 15.4 Linux
Additional environment information
No response
Description
We would like to include only a bare minimum of files required for installation in the source distribution.
Thus, we use a MANIFEST.in file and make sure setuptools-scm is not installed.
However, if the repository contains uncommitted changes, then all files under version control will be included, even if setuptools-scm is not present.
Expected behavior
No arbitrary VCS files should land in the sdist.
Only files explicitly included by user configuration should be packaged (MANIFEST.in, package_data, ...).
Of course, defaults should still be included, though (README, setup.cfg, ...).
How to Reproduce
git clone "https://github.com/pypdfium2-team/pypdfium2.git"
cd pypdfium2/
git checkout setupinfra
# first run without any uncommitted changes
PDFIUM_BINARY=none python3 -m build --sdist -n -x
# now, manually make some change to a file covered by VCS (e. g. src/pypdfium2/version.py) and then re-run
PDFIUM_BINARY=none python3 -m build --sdist -n -x

Inspect the sdists with an archive opener after the two build runs to see which files were included.
Output

The first sdist will only contain the files included via MANIFEST.in.
The second sdist, however, will contain all files under version control.

Hmm, now I can't reproduce this anymore. Perhaps it caused confusion that I have multiple pythons installed and one of them still had setuptools-scm, IDK... Sorry!

Yes, figured out what was the problem. I was sometimes running the commands manually and sometimes via Makefile.
The Makefile command used the wrong Python version, but also implied additional changes, so I ended up with a completely wrong conclusion...


Fix state for air conditioning

Tests added.
Fixes https://github.com/skodaconnect/homeassistant-myskoda/issues/38

Please check the linter messages

Please check the linter messages

Done.

Thank you


mount.cifs is not provided in CoreOS 1745

Currently all the cloud profile samples refer to CoreOS version: 1745.7.0. This version does not have mount.cifs installed. Related bug report and fix are below:
https://github.com/coreos/bugs/issues/571
https://github.com/coreos/coreos-overlay/pull/3344
So It's impossible to use AzureFiles or CIFSShares as persistent  volumes.
The latest stable coreos version on Azure is: 1911.5.0 . Is that version supported?

Not tried, but so far we had no issues with new CoreOS versions. Give it a try and report back your results, please! :-)
PS: We don't maintain every new CoreOS/K8s versions in the example CloudProfile resources necessarily (new versions appear too frequent).

We are testing the newest image now.

we've tried version 1911.5.0 and it seems to be woking fine.

Thank you for checking out and reporting back! Then I guess it's fine to close the issue?!

İt's in examples/cloudprofile files
On Sat 5 Jan 2019, 18:29 Luis Davim<EMAIL_ADDRESS>wrote:

I'm on a mobile now so I can't check but I remember grepping the code and
finding references to the coreos version in it.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gardener/gardener/issues/627#issuecomment-451664804,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAxDlv2zMzgtqgxpFbRP8Cf6KBtiNzyvks5vAMTkgaJpZM4ZoOGG
.

I have updated the example CloudProfile and Shoot resources in f1098ce1ef83e4698d4eb1af11c7f8c119bfadd6 (currently still on the feature/kubernetes-1.13 branch, not yet merged to master). Once 1.13 is supported all the values should be up-to-date.
There should not be an explicit version statement in the code itself, only in the example resource manifests. Please report the file once you found it again. Thank you!


How to specify folder where release artifacts are created?

I am in learning/testing mode, trying to create a release and output it to a shared folder on my local disk.
The following are run from a command window opened in the folder where MyApp.csproj lives.
Publish command:
dotnet publish -c Release --self-contained -r win-x64 -o .\bin\Release\net8\win-x64\publish

Pack command:
vpk pack -u MyApp -v 1.0.0 -p .\bin\Release\net8\win-x64\publish -e MyApp.exe -o c:\Shared\Releases\MyApp\Windows\

Publish appears to have run ok, however I ran into a problem with pack.  This path, c:\Shared\Releases\MyApp\Windows\, is valid and the folder exists.  However when I ran the pack command I got this error:

[10:51:03 FTL] Command WindowsPackCommand had an exception.   UnauthorizedAccessException: Access to the path 'c:\Shared\Releases\MyApp\Windows\RELEASES' is denied.

It appears all the files in the publish folder were copied to c:\Shared\Releases\MyApp\Windows\ and no deployment files were created.
I then created a Releases folder under c:\Shared\Releases\MyApp\Windows\ and ran the above pack command again.
I got the same error message however this time MyApp.nupkg, .zip, and ..Setup.exe were created in c:\Shared\Releases\MyApp\Windows\ and the Releases folder remains empty.
Desired functionality (unless I'm just doing it completely wrong) is:
App will be published to .\bin\Release\net8\win-x64\publish
Release artifacts will be created in c:\Shared\Releases\MyApp\Windows\

I can't reproduce this. I ran vpk with exactly the command line arguments you provided (and the folder didn't even exist) and Velopack correctly created all of the necessary folders and output the packages there.
I would guess that there is either a permissions issue or there is another process (eg. notepad.exe?) locking the RELEASES file preventing Velopack from writing to it.

Well I am sorry for bothering you.  I rebooted and sure enough it ran fine.


Generate a build for each commit

Codecov Report

Merging #121 (3441aa0) into main (14ba719) will decrease coverage by 0.28%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #121      +/-   ##
==========================================
- Coverage   81.30%   81.02%   -0.28%
==========================================
  Files           3        3
  Lines         369      369
==========================================
- Hits          300      299       -1
- Misses         69       70       +1

see 1 file with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Per conversation with @gpeacock, make sure version is distinct and shows the commit.

I think the idea behind this was to always have a built copy available that we could test before publishing. But I don't think we need that unless we have someone testing those builds.  Perhaps we should add this as an optional manual step so we can create a pre-published set of assets for testing only when we need them.

Closing. We have similar work in c2pa-rs, so not going to reopen this.


How to write value to Hocon?

There is a plenty of Get***** methods, but I can't see any method like WriteString("root.object", "value")

Hocon objects are read only, there is no way to write a value programmatically into a parsed Hocon object.


User feedback and comparison features associated with a user account/profile

Description
Placeholder for a broad review of proposed new features in this area which would be linked to the creation of user accounts as a prerequisite - for future consideration.
Those suggestions received include:
1.  provide a UI element to allow users to vote-up and vote-down features in the webapp along with providing comments.
2. Find out what's new for your target or T-D associations, with respect to the last time you visited the platform.
How?

Provide facets on the evidence page which allows sorting by date of addition to the platform and highlighting of new data since the last release.

emails/push notification about new data on your disease (for eg.)

3.  Automated statistical comparision of rankings between data releases and competitors
Why?

As an additional QC tool per release and a comparison across rankings generated by other platforms.

How?
Nice summary visualizations, like the one prototyped in https://github.com/opentargets/data_analytics
would be helpful.
4.  Allowing users (who we assume to be well-meaning and relatively expert in the gene they search for) to upvote/downvote targets (scoring) in the target list, which would allow to filter out poor targets.
How?
Would require:

a backend to persist these rankings
some sort of filter (eg. "back up your vote with a PMID") to prevent erroneous manipulation of the score
a manual curation step?
UI/UX to allow user voting and display its contribution to the final ranking

5.  Possible to showcase competitive intelligence data?
Note: There are not a lot of public sources with this kind of data. However, that would be of great interest to a Pharma scientist, since their decision to pursue a target is largely conditioned from the market opportunity and the unmet medical need.

[ ] WHAT would this look like?

Progress Tracker

Closing this issue as deemed out of scope in the context of the Open Source vision for platform.


Document requirements for a resource to graduate to beta

In the related working doc, the following was proposed as a starting point:

Covered by conformance tests
Implemented by several implementations
Users are exercising most of the API surface

/assign

requirements for a resource to graduate

Meaning graduation to the beta will be in on a resource by resource basis even for types that ship within the API? (not trying to say that it is bad, just new)

Exactly. In the first round of v1beta1 changes, we will be graduating Gateway, GatewayClass, and HTTPRoute to v1beta1.


database dirty flag set (likely due to unclean shutdown): replay required #4742

--hard-replay or --replay-blockchain does not work,how to solve this..?

when use --replay options,it delete state ,use --hard-replay,it rescontruct,but all this does not make use.is this a bug?

refer to https://github.com/EOSIO/eos/issues/4742,how to sove this?

Please refer to #4742

all opts does not work,I have try many ways.
From: DU Jiaen
Date: 2018-08-24 10:29
To: EOSIO/eos
CC: csquan; Author
Subject: Re: [EOSIO/eos] database dirty flag set (likely due to unclean shutdown): replay required #4742 (#5393)
Please refer to #4742
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

does not work...........................


⚠ Samsung has degraded performance

In 5ae9aed, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2752 ms

Resolved: Samsung performance has improved in 3992a9f.


feat: add asyncIterator & fix pubsub serialize/deserialize

I added asyncIterator to the subscription context so that we can process multiple topics at once.
And I added json serialize/deserialize to the pubsub sending/receiving process. If we don't do this, a problem occurs when we perform a field-based permission check in middleware such as graphql-shield.
I am not 100% sure that this PR will not be a problem. It would be good to apply after judging whether there is a problem with asynchronous processing or performance. @mcollina

First of all, it was wrong to think that the problem was caused by middleware.
In my case, the "Maximum call stack size exceeded" error occurs in the following part.
https://github.com/mcollina/mqemitter-redis/blob/master/mqemitter-redis.js#L162
These are the issues related to this issue. When encoding circular structures, msgpack-lite has a problem.
https://github.com/kawanet/msgpack-lite/issues/63
https://github.com/kawanet/msgpack-lite/issues/74
I experienced an error when publish a mongoose model object, which is an object like this:
CoreMongooseArray [
        'transaction',
        [Symbol(mongoose#Array#_atomics)]: {},
        [Symbol(mongoose#Array#_path)]: 'scopes',
        [Symbol(mongoose#Array#_schema)]: [SchemaArray],
        [Symbol(mongoose#Array#_parent)]: [Circular] ]

As in the answer to the above issue, we should remove the circular structure by JSON.stringify.
I don't know if this is a good solution, but it works anyway. @mcollina

https://github.com/davidyaha/graphql-redis-subscriptions/blob/master/src/redis-pubsub.ts#L78
Other pubsub implementations are also serializing json.

mqemitter is serializing as msgpack. There is no need to double serialize, just don't send Moongoose models as they are not serializable and they have a special thing for json.

To be precise, the circular structure of the parent object is created in a specific situation in mongoose, but since the custom toJSON is well implemented mongoose internally, there is no error with JSON.stringify. However, custom serializers such as msgpack.encode don't call toJSON, so it seems to get the error.
As you said, don't send the mongoose object as it is, just run toJSON() on the object and publish it and it works fine. I will remove the JSON processing part and post PR.

docs are needed


Sponsored issue: mui/material-ui theme does not support Nextjs 13 App router

Currently, setting up mui/material-ui theme provider supports Nextjs 12 with pages directory and the mui documentation describes the setup process, and there are many guides online about it. However, the introduction of the Nextjs 13 app router, made many changes to how Nextjs works thus making the old implementation of the theme setup not working anymore. To match the current architecture of Nextjs, as developers we need to make several workarounds to make it work as expected. I have tried it and worked for my previous project without any performance issues because of caching and I'm willing to share my knowledge with other developers.
Priority Support

@herberthk is using  Mintycode to fund this issue.
If you would like to accept  bounty for solving this issue join Mintycode.
Thank you in advance for helping.

@herberthk What is this issue about?
We support the app router and we have guides for it


Default Split

Add a feature to set defaults for split. I don't like to define the share each time I enter an expense.

I don't understand, is the current setting not enough? If not, could you elaborate on what your issue with the default split is?

Yeah, its already there. What you do need to do is to define it one per device (browse and mobile for example). This issue can be closed.


TLS handshake error

Hi,
I keep getting this line after one hour or so in logs:
TLS handshake error from <IP_ADDRESS>:17927: tls: first record does not look like a TLS handshake
The IP address and port in the line keeps changing time to tome.
and also what does this means? I can access my app on internet with SSL enabled.
Config:
Engine: Nodejs
Using Express as http server and Nginx as Proxy
SSL: Let's Encrypt

Update: The above error is coming from mesh.data.protocol

mesh.data.portal is your load balancer. This error just says something from <IP_ADDRESS> attempted to connect to your app on port 443, but wasn't using HTTPS to do it. This isn't an error within your application, and unless you own something at that source address, it's not an error you can even fix. It is, however, entirely harmless.

So, i figured out that one of the ip was my home's and other one was coming from actions on google server. I am using nginx to redirect all http to https requests.

This seems like a very general error. I will reopen the issue if it becomes serious.


Add configuration example for nginx webserver

resto has been tested with nginx webserver - so let's use it :)

Configuration added in README file


Refactor parallelization

Use prefect to map registration and warping&saving over sections.
Create chained slurm jobs for registration and warping&saving.

Addresses #2.

Codecov Report

Merging #20 (10e0007) into main (7a79d36) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #20   +/-   ##
=======================================
  Coverage   96.74%   96.74%
=======================================
  Files          11       11
  Lines         583      583
=======================================
  Hits          564      564
  Misses         19       19

Impacted Files
Coverage Δ

src/sbem/experiment/Experiment.py
92.95% <ø> (ø)

Continue to review full report at Codecov.

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


对中文的数据集有做过测试吗？表现如何？

Nope. I only tested on English datasets, those four listed in ReadMe.

Nope. I only tested on English datasets, those four listed in ReadMe.

thanks!


Enable applications restart and rebuild tests

These tests were disabled because they introduced random issues in the CI.
It should be good now as I've disabled the check on the instance availability percentage.
It's not ready in the UI yet but the check will be on the restart counter.

CI ok here: https://github.com/epinio/epinio-end-to-end-tests/actions/runs/2002982559


[avocado-130] Upgrade to latest ADAM/Spark and add timers to avocado.

Resolves #130. Adds timers and does various other upgrading things.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/avocado-prb/47/
Test PASSed.


Active control fix

In order for the active column preset selection to appear as an active control, the value for the column_preset property needs to match the icon name, or in this case, the ligature value. By changing the control keys to match the column_presets, and also by updating the iconmoon project to use the correct ligatures of column-* instead of *columns, everything comes into alignment.
So now, when we set the active control in applyColumns and onBlockRender, the value of the column_preset matches the ligature and the correct control is rendered active.

Oh, great! Thank you!


neovim: neomake plugin not building

Summary
The Neomake plugin for Neovim won't build on nixos-unstable.
Details
NixOS version:
16.03.git.885acea (Emu)

Included in configuration.nix:
imports = [
  ./myvim.nix
];

In myvim.nix (if I remove the { name = "neomake"; } line then everything builds fine:
{ config, pkgs, ... }:

{
  environment.systemPackages = with pkgs; [
    (neovim.override {
      configure = {
        vam = {

          pluginDictionaries = [

            { name = "neomake"; }

          ];
        };
      };
    })
  ];
}

Error message after nixos-rebuild switch:
building Nix...
building the system configuration...
these derivations will be built:
  /nix/store/k3yxl7xmp6mflc9lyh36sa4d2gppaw43-vimplugin-neomake-2016-01-21.drv
  /nix/store/3rhjjv3g5jw3jag59ak7gym7pi3915x0-vimrc.drv
  /nix/store/zq3mwsamly8mi2glypryfk3g42a2ld3l-neovim-0.1.1-configured.drv
  /nix/store/i2ji2i3zcqrkywscyi0c2m7m02y2ysrb-system-path.drv
  /nix/store/wh1cplmcy9618a1ir42cifmc3wgaasxs-system-crontab.drv
  /nix/store/1dnrx911h1ly93bybiinvsvhcmd66jms-crontabs.drv
  /nix/store/2qisdc51ww1ips05p68xivz1sz3j5ml1-dbus-conf.drv
  /nix/store/8s1385czmd2dqla2y0scdgyzs0jppgks-unit-polkit.service.drv
  /nix/store/rdkbgy1mh94bx3cc78q7bff0nqaxk2pq-unit-dbus.service.drv
  /nix/store/x5d9lzk63nqps0ycivwrg8i70v77lk3b-system-units.drv
  /nix/store/8k2zikmlz1q2w9a0y7ah605pivc18jm2-etc.drv
  /nix/store/jn9a0cl4jphps3skanmz2zyy5c2yjqsp-nixos-system-nivian-16.03.git.885acea.drv
building path(s) ‘/nix/store/siv5znbzqx4dia582ags61gga6irhih5-vimplugin-neomake-2016-01-21’
unpacking sources
unpacking source archive /nix/store/d308cx1nxq5avrwwvja1nbbm69dyyi2w-neomake-e0fa234
source root is neomake-e0fa234
setting SOURCE_DATE_EPOCH to timestamp<PHONE_NUMBER> of file neomake-e0fa234/plugin/neomake.vim
warning: file neomake-e0fa234/plugin/neomake.vim may be generated; SOURCE_DATE_EPOCH may be non-deterministic
patching sources
configuring
building
installing
builder for ‘/nix/store/k3yxl7xmp6mflc9lyh36sa4d2gppaw43-vimplugin-neomake-2016-01-21.drv’ failed with exit code 1
cannot build derivation ‘/nix/store/3rhjjv3g5jw3jag59ak7gym7pi3915x0-vimrc.drv’: 1 dependencies couldn't be built
cannot build derivation ‘/nix/store/zq3mwsamly8mi2glypryfk3g42a2ld3l-neovim-0.1.1-configured.drv’: 1 dependencies couldn't be built
cannot build derivation ‘/nix/store/i2ji2i3zcqrkywscyi0c2m7m02y2ysrb-system-path.drv’: 1 dependencies couldn't be built
cannot build derivation ‘/nix/store/jn9a0cl4jphps3skanmz2zyy5c2yjqsp-nixos-system-nivian-16.03.git.885acea.drv’: 1 dependencies couldn't be built
error: build of ‘/nix/store/jn9a0cl4jphps3skanmz2zyy5c2yjqsp-nixos-system-nivian-16.03.git.885acea.drv’ failed

PS Let me know if there's a way to get more verbose error output and I'll be happy to post it.

Hrrm, the latest neomake works:
neomakeCustom = buildVimPluginFrom2Nix {
  name = "neomake";
  src = fetchgit {
    url = "git://github.com/benekastah/neomake";
    rev = "e3f6fc538cecf478f826df58314a94b2da936b9f";
    sha256 = "0wzh7gf9iqa7vdh2j1h1lwjl05xz4r2ci9zz5b78bkwn02pigh29";
  };
  dependencies = [];
};

The above was made manually though. I can't build it automatically though using the suggested method (https://github.com/NixOS/nixpkgs/blob/master/pkgs/misc/vim-plugins/vim-utils.nix) When I attempt to do so I get this error (after :sourceing my temp file from a vim-scripts file containing the one line "neomake"):
failed : neomake repository neomake unkown!
neomake
{'neomake': 'repository neomake unkown!'}
Error detected while processing function nix#ExportPluginsForNix:
line   59:
E716: Key not present in Dictionary: neomake
E116: Invalid arguments for function split(cache[k].derivation,"\n"))
E116: Invalid arguments for function append
Press ENTER or type command to continue

neomakeCustom resolve the same issue for me when using just buildVimPlugin. @seagreen: perhaps an update to https://github.com/NixOS/nixpkgs/blob/master/pkgs/misc/vim-plugins/default.nix is in order?

Good to hear. I'd still like to figure out what's going on before I make a PR. If the automatic process is broken it would be better to fix that than have to keep looking up the latest git commit for different packages by hand.
As I understand it, a maintainer should be able to put this file in a vim buffer: https://github.com/NixOS/nixpkgs/blob/master/pkgs/misc/vim-plugins/vim-plugin-names
Then execute this process: https://github.com/NixOS/nixpkgs/blob/master/pkgs/misc/vim-plugins/vim-utils.nix#L87
And get a list of the derivations for the latest versions of packages that can be pasted in here: https://github.com/NixOS/nixpkgs/blob/master/pkgs/misc/vim-plugins/default.nix#L99
(Sidenote: is that actually right? I'm only guessing that's the plan for how derivations can be upgraded in bulk.)
Anyway, if someone more knowledgeable doesn't drop by I guess I can go ahead and make the PR for neomake, that will at least fix our current problem.

This commit fixed the problem: https://github.com/NixOS/nixpkgs/commit/d2d0f19b57e4a7abb9d88556f04a526473404aaf


Updates to demo game

Removes old code from demo game
Makes the spline tension editable in sprite animations
Adds constraint for rotating a node towards another node
Adds swarms of enemies to demo game

lgtm, please consider the above nits.


Mockito Unnecessary Stubbings Everywhere in tests

In Junit 5 mockito changed how they detect unnecessary stubbings.
Previously unecessary stubbings were declared when the stubbing was never used during the tests. Now unneccesary stubbings are declared on a test by test basis. So this now throws a bunch of unnecessary stubbing errors for each of our beforeEach methods as not all stubbings in those methods are used every time.
It would be nice to refactor the tests to remove this effect and turn back on strict stubbing guidelines.

Are there any advantages to using strict stubbing?

Mainly to make sure that the tests don't become cluttered with unecessary code or setup that could have unintended effects in the future

In JUnit 5 default, a mock method counts as verify already, so can just run verifyNoMoreInteractions as @AfterEach . I think that's the reason why it's strict by default.
You can disable this for certain mocks (= where it makes sense) by calling lenient().when(...).then...

Yeah it is more so that when we moved to junit 5 if a mock was made in the before each that wasn't actually used in the test then it would throw a strict stubbing error.
This caused a lot of errors as the before is used for general setup in a lot of tests and not every mock is used every test but might be used in a few of them.
This the refactor headache

This was resolved in #3070


Revert to Checking only ci.yml

Pull only artifacts from ci.yml to resolve issue with conflicting package names.

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow


Use OldestId setting for picking last message for fan-out

Perhaps it makes more sense to allow customisation in fan-out setting?
For example in my case I built a service which has to consume whole stream each time it restarts (it maintains an in-memory read model). I can do that with consumer groups but that does seem to be a good fit for my case
PS. Though it breaks a default behaviour, since by default OldestId is $ :/

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 79.54%. Comparing base (e1e2380) to head (62ea826).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #34      +/-   ##
==========================================
+ Coverage   78.77%   79.54%   +0.76%
==========================================
  Files           3        3
  Lines         523      523
==========================================
+ Hits          412      416       +4
+ Misses         82       79       -3
+ Partials       29       28       -1

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

I suggest also update the comment for SubscriberConfig.OldestId
Others LGTM

@minghsu0107 thanks. Introduced settings and added a test.


Add common Functor#as and Applicative#replicateA directly

Thanks for taking this on!

The more I think about it, replicateA is a really bad name. The A comes from the fact that replicate is already a method on collections, IIRC. So, the A stands for applicative. I think maybe .exactly or something might be a better name. I used rep, following fastparse, to mean repeat at least, which is the most common kind of repetition (and almost always you want to repeat 0 or more times, or 1 or more times)...
honestly, maybe we should just punt on the whole replicate question and remove that for now. I can't even imagine a use case at the moment, and if you want it, you can use replicateA from Applicative to get it.
What do you think?

@johnynek regarding replicateA from name to use case it's a bit weird. Most of the time what we want is really 0/1 or more times so I'm fine in let it rest for now.
I'll clean it up in a bit!

Thanks a ton!
I'm going to make one more PR than release a new version today.


The login page is not loading after successful installation

Hello,
I am trying to install ph7builder latest version(i.e. 17.2.0) in cPanel. I am able to install it successfully but when I try to load the login page it is not loafing showing This page isn’t working with HTTP ERROR 500 error. I have checked .htaccess also, but I was unable to resolve it. I am able to load the login page in the previous version i.e. 17.1.8 without any error and it's working fine. Can you please help to resolve it?
Thanks.

it is solved.

https://github.com/pH7Software/pH7-Social-Dating-CMS/issues/1033#issue-1301361041


Dyson Plugin causes Homebridge to shut down

I am using a Dyson Pure Cool Purifier Gen2018 which I assume is TP07.
When activating the plugin, I get the error log below and Homebridge shuts down and restarts, stuck in a loop.
If I remove my email and password from the config file, I get a "wrong serial number" log and still nothing.
I even logged into the app again to receive the login code, which I entered into the accessory password field to try out more. still nothing. So basically at the time being I can't get the plugin to work at all...
Any help is much appreciated!
Thx a ton!
Patrick
[12/1/2021, 9:50:17 AM] Homebridge v1.3.8 (HAP v0.9.7) (Homebridge 8368) is running on port 51423.
[12/1/2021, 9:50:17 AM] SyntaxError: Unexpected token e in JSON at position 0
    at JSON.parse (<anonymous>)
    at IncomingMessage.<anonymous> (/usr/local/lib/node_modules/homebridge-dyson-link/index.js:180:40)
    at IncomingMessage.emit (node:events:402:35)
    at endReadableNT (node:internal/streams/readable:1343:12)
    at processTicksAndRejections (node:internal/process/task_queues:83:21)
[12/1/2021, 9:50:17 AM] Got SIGTERM, shutting down Homebridge...

UPDATE:
https://github.com/lukasroegner/homebridge-dyson-pure-cool
just found this plugin – working perfectly.

UPDATE:
https://github.com/lukasroegner/homebridge-dyson-pure-cool
just found this plugin – working perfectly.

nope ... this one cannot control fan rotation
(i am using purifier hot+cool)