[stable/nextcloud] connect() failed (111: Connection refused) while connecting to upstream, client: x.x.x.x, server: , request: "GET /status.php HTTP/1.1", upstream: "fastcgi://<IP_ADDRESS>:9000", host: "nc.xxxx.com"

Describe the bug
I try to deploy nexcloud with mariadb but this fails with
kubectl -n nextcloud logs nextcloud-74b56fb9dd-c4smn nextcloud-nginx
2020/04/15 19:03:54 [error] 9#9: *3 connect() failed (111: Connection refused) while connecting to upstream, client: x.x.x.x, server: , request: "GET /status.php HTTP/1.1", upstream: "fastcgi://<IP_ADDRESS>:9000", host: "nc.xxxx.com"
2020/04/15 19:03:54 [error] 7#7: *2 connect() failed (111: Connection refused) while connecting to upstream, client: x.x.x.x, server: , request: "GET /status.php HTTP/1.1", upstream: "fastcgi://<IP_ADDRESS>:9000", host: "nc.xxxx.com"
x.x.x.x - - [15/Apr/2020:19:03:54 +0000] "GET /status.php HTTP/1.1" 502 157 "-" "kube-probe/1.17" "-"
x.x.x.x - - [15/Apr/2020:19:03:54 +0000] "GET /status.php HTTP/1.1" 502 157 "-" "kube-probe/1.17" "-"
x.x.x.x - - [15/Apr/2020:19:03:54 +0000] "GET /status.php HTTP/1.1" 499 0 "-" "kube-probe/1.17" "-"

This is my values.yaml
ingress:
  enabled: true
  annotations:
    nginx.ingress.kubernetes.io/proxy-body-size: 4G
    # nginx.ingress.kubernetes.io/ssl-redirect: "true"
    nginx.ingress.kubernetes.io/secure-backends: "true"
    nginx.ingress.kubernetes.io/backend-protocol: "HTTPS"
    cert-manager.io/cluster-issuer: "letsencrypt-prd"
  tls:
    - secretName: nextcloud-tls
      hosts:
        - nc.xxx.com
  labels: {}
nextcloud:
  host: xxxx
  username: xxxx
  password: xxx
  update: 0
  datadir: /var/www/html/data
  tableprefix:
  phpConfigs: {}
  defaultConfigs:
    .htaccess: true
    redis.config.php: true
    apache-pretty-urls.config.php: true
    apcu.config.php: true
    apps.config.php: true
    autoconfig.php: true
    smtp.config.php: true
  configs: {}

nginx:
  enabled: true
  image:
    repository: nginx
    tag: alpine
    pullPolicy: IfNotPresent
  config:
    default: true
  resources: {}

internalDatabase:
  enabled: false
  name: nextcloud

externalDatabase:
  enabled: false

mariadb:
  enabled: true
  volumePermissions:
    enabled: true
  securityContext:
    fsGroup: 82
    runAsUser: 33
  db:
    name: nextcloud
    user: nextcloud
    password: xxxx
  persistence:
    enabled: false
    accessMode: ReadWriteMany
    size: 8Gi
  master:
    persistence:
      accessModes:
        - ReadWriteMany
  slave:
    persistence:
      accessModes:
        - ReadWriteMany
redis:
  enabled: false
  usePassword: false
cronjob:
  enabled: false

service:
  type: ClusterIP
  port: 8080
  loadBalancerIP: nil

persistence:
  enabled: true
  annotations: {}
  storageClass: "kadalu.replica3"
  accessMode: ReadWriteMany
  size: 500Gi

Version of Helm and Kubernetes:
helm version
version.BuildInfo{Version:"v3.1.2", GitCommit:"d878d4d45863e42fd5cff6743294a11d28a9abce", GitTreeState:"clean", GoVersion:"go1.13.8"}

Client Version: version.Info{Major:"1", Minor:"17", GitVersion:"v1.17.3", GitCommit:"06ad960bfd03b39c8310aaf92d1e7c12ce618213", GitTreeState:"archive", BuildDate:"2020-02-29T16:37:45Z", GoVersion:"go1.14", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"17", GitVersion:"v1.17.4", GitCommit:"8d8aa39598534325ad77120c120a22b3a990b5ea", GitTreeState:"clean", BuildDate:"2020-03-12T20:55:23Z", GoVersion:"go1.13.8", Compiler:"gc", Platform:"linux/amd64"}

Which chart:
stable/nextcloud
What happened:
pod/nextcloud-nginx logs
kubectl -n nextcloud logs nextcloud-74b56fb9dd-c4smn nextcloud-nginx
2020/04/15 19:03:54 [error] 9#9: *3 connect() failed (111: Connection refused) while connecting to upstream, client: x.x.x.x, server: , request: "GET /status.php HTTP/1.1", upstream: "fastcgi://<IP_ADDRESS>:9000", host: "nc.xxxx.com"
2020/04/15 19:03:54 [error] 7#7: *2 connect() failed (111: Connection refused) while connecting to upstream, client: x.x.x.x, server: , request: "GET /status.php HTTP/1.1", upstream: "fastcgi://<IP_ADDRESS>:9000", host: "nc.xxxx.com"
x.x.x.x - - [15/Apr/2020:19:03:54 +0000] "GET /status.php HTTP/1.1" 502 157 "-" "kube-probe/1.17" "-"
x.x.x.x - - [15/Apr/2020:19:03:54 +0000] "GET /status.php HTTP/1.1" 502 157 "-" "kube-probe/1.17" "-"
x.x.x.x - - [15/Apr/2020:19:03:54 +0000] "GET /status.php HTTP/1.1" 499 0 "-" "kube-probe/1.17" "-"

What you expected to happen:
No error
How to reproduce it (as minimally and precisely as possible):
Run
helm install nextcloud stable/nextcloud --namespace nextcloud -f nextcloud.values.yml

Anything else we need to know:
n/a

Anyone who can assist?

Hi @papanito ,
One first mention is that the ingress annotations are not correct. There is not HTTPS connection between nginx ingress and nextcloud-nginx. So I would remove this two annotations as a first step.
nginx.ingress.kubernetes.io/secure-backends: "true"
nginx.ingress.kubernetes.io/backend-protocol: "HTTPS"

But this might not be the problem visible in your nextcloud logs. I have a question. Are you sure that the database was up and running at the time this logs messages showed?
regards

I removed the two lines in question and ran helm upgrade, still I see the same error
I just checked and the master is running, whereas the slave is not
kubectl -n nextcloud logs nextcloud-mariadb-master-0
 16:07:38.33
 16:07:38.34 Welcome to the Bitnami mariadb container
 16:07:38.34 Subscribe to project updates by watching https://github.com/bitnami/bitnami-docker-mariadb
 16:07:38.34 Submit issues and feature requests at https://github.com/bitnami/bitnami-docker-mariadb/issues
 16:07:38.34 Send us your feedback at<EMAIL_ADDRESS> 16:07:38.34
 16:07:38.34 INFO  ==> ** Starting MariaDB setup **
 16:07:38.38 INFO  ==> Validating settings in MYSQL_*/MARIADB_* env vars
 16:07:38.38 INFO  ==> Initializing mariadb database
 16:07:38.41 INFO  ==> Using persisted data
 16:07:38.42 INFO  ==> Running mysql_upgrade
 16:07:38.43 INFO  ==> Starting mariadb in background
 16:08:35.50 INFO  ==> Stopping mariadb

 16:08:45.52 INFO  ==> ** MariaDB setup finished! **
 16:08:45.57 INFO  ==> ** Starting MariaDB **
2020-04-15 16:09:18 0 [Note] /opt/bitnami/mariadb/sbin/mysqld (mysqld 10.3.20-MariaDB-log) starting as process 1 ...
2020-04-15 16:09:18 0 [Note] InnoDB: Using Linux native AIO
2020-04-15 16:09:18 0 [Note] InnoDB: Mutexes and rw_locks use GCC atomic builtins
2020-04-15 16:09:18 0 [Note] InnoDB: Uses event mutexes
2020-04-15 16:09:18 0 [Note] InnoDB: Compressed tables use zlib 1.2.8
2020-04-15 16:09:18 0 [Note] InnoDB: Number of pools: 1
2020-04-15 16:09:18 0 [Note] InnoDB: Using SSE2 crc32 instructions
2020-04-15 16:09:18 0 [Note] InnoDB: Initializing buffer pool, total size = 128M, instances = 1, chunk size = 128M
2020-04-15 16:09:18 0 [Note] InnoDB: Completed initialization of buffer pool
2020-04-15 16:09:18 0 [Note] InnoDB: If the mysqld execution user is authorized, page cleaner thread priority can be changed. See the man page of setpriority().
2020-04-15 16:09:18 0 [ERROR] InnoDB: Unable to lock ./ibdata1 error: 11
2020-04-15 16:09:18 0 [Note] InnoDB: Check that you do not already have another mysqld process using the same InnoDB data or log files.
2020-04-15 16:09:18 0 [Note] InnoDB: Retrying to lock the first data file
2020-04-15 16:09:19 0 [Note] InnoDB: 128 out of 128 rollback segments are active.
2020-04-15 16:09:19 0 [Note] InnoDB: Removed temporary tablespace data file: "ibtmp1"
2020-04-15 16:09:19 0 [Note] InnoDB: Creating shared tablespace for temporary tables
2020-04-15 16:09:19 0 [Note] InnoDB: Setting file './ibtmp1' size to 12 MB. Physically writing the file full; Please wait ...
2020-04-15 16:09:19 0 [Note] InnoDB: File './ibtmp1' size is now 12 MB.
2020-04-15 16:09:19 0 [Note] InnoDB: 10.3.20 started; log sequence number 1630851; transaction id 21
2020-04-15 16:09:19 0 [Note] InnoDB: Loading buffer pool(s) from /bitnami/mariadb/data/ib_buffer_pool
2020-04-15 16:09:19 0 [Note] InnoDB: Buffer pool(s) load completed at 200415 16:09:19
2020-04-15 16:09:19 0 [Note] Plugin 'FEEDBACK' is disabled.
2020-04-15 16:09:19 0 [Note] Server socket created on IP: '<IP_ADDRESS>'.
2020-04-15 16:09:19 0 [Warning] 'proxies_priv' entry '@% root@nextcloud-mariadb-master-0' ignored in --skip-name-resolve mode.
2020-04-15 16:09:19 0 [Note] Reading of all Master_info entries succeeded
2020-04-15 16:09:19 0 [Note] Added new Master_info '' to hash table
2020-04-15 16:09:19 0 [Note] /opt/bitnami/mariadb/sbin/mysqld: ready for connections.
Version: '10.3.20-MariaDB-log'  socket: '/opt/bitnami/mariadb/tmp/mysql.sock'  port: 3306  Source distribution

kubectl -n nextcloud logs nextcloud-mariadb-slave-0
 17:58:09.32
 17:58:09.32 Welcome to the Bitnami mariadb container
 17:58:09.32 Subscribe to project updates by watching https://github.com/bitnami/bitnami-docker-mariadb
 17:58:09.33 Submit issues and feature requests at https://github.com/bitnami/bitnami-docker-mariadb/issues
 17:58:09.33 Send us your feedback at<EMAIL_ADDRESS> 17:58:09.33
 17:58:09.33 INFO  ==> ** Starting MariaDB setup **
 17:58:09.38 INFO  ==> Validating settings in MYSQL_*/MARIADB_* env vars
 17:58:09.38 INFO  ==> Initializing mariadb database
 17:58:09.40 INFO  ==> Using persisted data
 17:58:09.41 INFO  ==> Running mysql_upgrade
 17:58:09.41 INFO  ==> Starting mariadb in background
 17:58:56.63 INFO  ==> Stopping mariadb

Still no further, not sure what is the issue here :-(

I've ensured to have an fpm-image
image:
  repository: nextcloud
  tag: 19.0.0-beta-fpm

and here my nginx config from the values.yaml

nginx:
  ## You need to set an fpm version of the image for nextcloud if you want to use nginx!
  enabled: true
  image:
    repository: nginx
    tag: alpine
    pullPolicy: IfNotPresent

  config:
    # This generates the default nginx config as per the nextcloud documentation
    default: true
    # custom: |-
    #     worker_processes  1;..

  Can you please raise the issue over there?  https://github.com/nextcloud/helm

https://github.com/nextcloud/helm/issues/11


Missing files for training on CIFAR

Hi
I am trying to run the provided code on CIFAR as instructed in the Readme file. However, there are some errors when trying to import some modules. It seems they are missing from the code folder
Is the full code for CIFAR experiments also not available yet, and is going to be released later with the ImageNet code?

You should run train.py at the project root dir.
Try
export PYTHONPATH=./exp:./:./BigGAN_PyTorch_1_lib
python exp/omni_loss/train.py ...


Support for listening to scale changes

I am currently using https://github.com/Baseflow/PhotoView in our app and we have a requirement to log analytics event if an user interacted with the image (zoom/pan etc.)
For this, we use the provided setOnScaleChangeListener method which gets a callback from https://github.com/Baseflow/PhotoView/blob/565505d5cb84f5977771b5d2ccb7726338e77224/photoview/src/main/java/com/github/chrisbanes/photoview/PhotoViewAttacher.java#LL151C18-L151C18
It would be great if telephoto could support something similar.

Is ZoomableState#zoomFraction what you're looking for?
https://saket.github.io/telephoto/zoomable/recipes/#observing-pan-zoom

Yes, this works perfectly. I was looking at the wrong place in the code.

Hi @saket,
I'm unable to reproduce expected behavior of the snippet in https://saket.github.io/telephoto/zoomable/recipes/#observing-pan-zoom
val zoomableState = rememberZoomableState()

            ZoomableAsyncImage(
                model = capturedItem.uri,
                contentDescription = null,
                modifier = Modifier
                    .zoomable(
                        state = zoomableState,
                    )
                    .fillMaxSize()
            )

            LaunchedEffect(zoomableState.contentTransformation) {
                println("Pan = ${zoomableState.contentTransformation.offset}")
                println("Zoom = ${zoomableState.contentTransformation.scale}")
                println("Zoom fraction = ${zoomableState.zoomFraction}")
            }

(This code is within a pager's composable callback for each page)
The zoom works as expected but I'm really unable to listen to the zoom updates unfortunately. I'm doing this to get my gallery in focus mode when the user has zoomed in and out of that mode when the user has completely zoomed out.
By any chance did any compose update cause the behavior of LaunchedEffect to change or am I doing something wrong Any help would be highly appreciated
PS: Thanks a lot for building out this great library with your valuable time and efforts!

Using Modifier.zoomable on a parent container/composable and using AsyncImage seemed to resolve the issue over applying Modifier.zoomable on the main ZoomableAsyncImage


Lag with Grid turned on at higher resolutions

At higher resolutions, with the grid turned on and zoomed in far enough to render, there is an immense amount of lag when doing any action (very noticeable at 256x256). From what I checked, the grid width had nothing to do with the issue. The issue can be replicated on the desktop and chrome version (didn't check firefox).

Indeed, it's much slower.
When the grid is activated, Piskel is no longer using the native canvas scale method but uses a custom scale implementation instead.
cf. ImageResizer : https://github.com/juliandescottes/piskel/blob/master/src/js/utils/ImageResizer.js#L35
cf. FrameRenderer : https://github.com/juliandescottes/piskel/blob/master/src/js/rendering/frame/FrameRenderer.js#L267
Leads to investigate :

draw the grid separately from the main sprite canvas
draw the grid only after a throttling delay

In the end, simply using clearRect on the already rendered frame gives a good visual result and pretty good performances.
The grid can now be used at higher resolutions without any issue.
I uploaded a video showing the new performance on a 640x640 sprite: https://www.youtube.com/watch?v=Uu9Tz4UDCsg


Binary analogous to Json

I've been wondering if you have considered adding a Binary type analogous to types::Json<T> using to serialize objects to BLOB and BYTEA using bincode.

I've been prototyping a migration tool for sqlx, and I think it would be more useful for this Binary/Blob type  to be at sqlx::types rather that my own crate.

I don't hate this idea, but I have some reservations.
For one, I'm not sure how useful this is going to be outside of very specific use-cases like yours. I'm not even sure what your exact use-case is. I'd like to see a more compelling argument for why this should live in SQLx.
Encoded binary data is going to be very difficult to introspect and debug using DB admin tools, which I end up using a lot. Even if a database flavor has no native support for JSON, it's still just a human-readable string.
I also don't know about using bincode. It may just be the default configuration but in the past I've had issues where even adding a field to a struct made existing data un-decodeable, which is problematic for long-term storage of data whose structure wants to evolve over time. Though less efficient, CBOR might be the better choice here, being a standard format and relatively portable/flexible. Cap'N'Proto feels a little overkill since it requires a separate schema definition but it would let us recover some speed I think.
At the end of the day though, you generally want to fan out a complex type into table columns rather than shove it into a single column in some arbitrary encoding. We only really use Json<T> for when we're building up complex return types in SQL; we try to use JSON for storage in the database only as a last resort.

You know, after thinking about the API for the crate I was working on, I realized that it didn't need to be sqlx specific. So in that case it would make more sense for Bytes<T> to live at my crate. Then I'd use features to enable further functionality.
Also, I do agree with you in that is usually not a great idea to use jsons unless they are used as collections. But as I see it, some user may want a table with  columns like these:
   foo JSON NOT NULL,
   bar BLOB NOT NULL,

And I want to provide an API that allows that. Either way, as you said, this is probably not the best place for Binary<T>.

I will take into consideration the alternatives to bincode you mentioned, as I wasn't aware of this behavior.

Closing this since we basically concluded that SQLx wasn't the right place for this type.


[Mention] won't update

I am trying to push new values to  "items" but after it changed. The drop down menu becomes empty.

See #41. You need to use the most recent version and set items to new array.

Thanks @dmacfarlane  I have fixed this one already.
Instead of "[string]", I used  "Observable<any[]>" instead.
and seting items to new array do the trick ;).


Fluids don't update below y=0

Just what the title says. Fluid blocks (both water and lava) don't update (i.e. flow) when they are below y=0.

Fluids update, just very slowly.

Thanks! I just didn't wait for the very slow world ticking.


Typekit 403

I'm seeing a 403 for Typekit CSS on live:
esn8rgw.css:1 Failed to load resource: the server responded with a status of 403 ()
I think you need to check the domain is listed correctly in your Typekit account.
Here's a similar report and some support that will help:
https://forums.adobe.com/thread/1554778

Moved to the PL repo: https://github.com/IATI/IATI-Pattern-Library/issues/88


Is it possible to opt out of storing data in userStore, ie. sessionStorage/localStorage?

According to the docs, there is a way to specify the userStore, ie. a place where user data is stored.
See this link for reference.
Is it possible to opt out of this?

You can inject your own userStore (https://github.com/authts/oidc-client-ts/blob/main/src/UserManagerSettings.ts#L84), where you can do whatever you need to...


fix bugs

I fix bugs. This is my final upload before we two start to modify united routines.

Thank you for merging.


Large scale focing

Hello, I would like to ask if the UCLALES-SALSA can set time-varying sea surface temperatures and divergence in the same way as the UCLALES model.

Hello. These are currently not included in UCLALES-SALSA.

You could just copy relevant codes from UCLALES and then modify those if needed.


[BUG] Unable to start docker stack: No module named common.tasks

Describe the bug
Docker container start but it fails with this error log:
/home/inventree/data/config.yaml exists - skipping
Waiting for database...
Database connection sucessful!
Base currency changed from USD to EUR
Updating exchange rates from https://api.exchangerate.host/latest
Using base currency 'EUR'
00:12:46 [Q] INFO Q Cluster hydrogen-pennsylvania-tennessee-berlin starting.
00:12:46 [Q] INFO Process-1:1 ready for work at 17
00:12:46 [Q] INFO Process-1:2 ready for work at 18
00:12:46 [Q] INFO Process-1:3 ready for work at 19
00:12:46 [Q] INFO Process-1:4 ready for work at 20
00:12:46 [Q] INFO Process-1:5 monitoring at 21
00:12:46 [Q] INFO Process-1 guarding cluster hydrogen-pennsylvania-tennessee-berlin
00:12:46 [Q] INFO Process-1:6 pushing tasks at 22
00:12:46 [Q] INFO Q Cluster hydrogen-pennsylvania-tennessee-berlin running.
00:13:11 [Q] INFO Process-1:1 processing [sink-hotel-hydrogen-october]
00:13:11 [Q] INFO Process-1:2 processing [avocado-london-spaghetti-arkansas]
00:13:11 [Q] INFO Process-1:3 processing [fifteen-angel-colorado-cola]
00:13:11 [Q] ERROR Failed [sink-hotel-hydrogen-october] - No module named 'common.tasks'
00:13:11 [Q] ERROR Failed [avocado-london-spaghetti-arkansas] - module 'InvenTree.tasks' has no attribute 'delete_old_error_logs'
00:13:11 [Q] ERROR Failed [fifteen-angel-colorado-cola] - module 'InvenTree.tasks' has no attribute 'delete_old_error_logs'

Deployment Method

[ x] Docker
[ ] Bare Metal

Version Information
Docker 0.5.4

Have you recently updated the docker instance? It might be that the "server" and "worker" containers are running different images?
Are you running docker-production or docker-development?

Docker-production, server and worker are running the same version of the
image.
I can access the web interface and all appears to work but the container
log is full of the above error at every web page request.
Il Gio 18 Nov 2021, 01:33 Oliver @.***> ha scritto:

Have you recently updated the docker instance? It might be that the
"server" and "worker" containers are running different images?
Are you running docker-production or docker-development?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/inventree/InvenTree/issues/2321#issuecomment-972373343,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIJLFEYKWMZ4576S4KRP2LTUMRCUJANCNFSM5IIJ6E6A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Ok, so version 0.5.4 does not have common.tasks.py and delete_old_error_logs was not yet implemented.
Did you at some stage install a newer version and then go back to 0.5.4?

There's no newer tag on Docker Hub, but before I used the latest tag, and
after I saw it was not working I went back to 0.5.4. The mounted directory
is fresh new, just the database is the same.
Il giorno gio 18 nov 2021 alle ore 21:56 Oliver @.***>
ha scritto:

Ok, so version 0.5.4 does not have common.tasks.py and
delete_old_error_logs was not yet implemented.
Did you at some stage install a newer version and then go back to 0.5.4?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/inventree/InvenTree/issues/2321#issuecomment-973263974,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIJLFE4V7KUWXZ63NWF3O7DUMVR75ANCNFSM5IIJ6E6A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Ok, that makes sense. It has added some scheduled tasks from the newer version.
It is fine, it will keep running OK (just with error messages in the logs)
If you want you can delete the scheduled tasks from the admin view:


PornHub users:brittney_jones failed with error 0

Error message: Unable to execute statement: database is locked
Query string: action=display&bridge=Pornhub&q=brittney_jones&type=users&sort=%3F&show_images=on&format=Atom
Version: dev.2020-11-10

Duplicate of https://github.com/RSS-Bridge/rss-bridge/issues/1981


Add unittests

Proposal: Run tests using local versions of run (elocal) and cd (lcd), against a mock git repro. Start with running CI using Travis.

Added a basic set of tests that covers setup and deploy (with git). bb747c67fee74b731bf5dbc2ad65b969e78bab19
Tests can be run using python runtests.py. Should in the feature run through python -m unittest tests.


Add all the informations that we have in the Span to the SpanData.

Codecov Report

Merging #270 into master will decrease coverage by 0.18%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master     #270      +/-   ##
============================================
- Coverage     94.21%   94.03%   -0.19%
+ Complexity      426      416      -10
============================================
  Files            71       70       -1
  Lines          1626     1627       +1
  Branches         95       93       -2
============================================
- Hits           1532     1530       -2
- Misses           68       72       +4
+ Partials         26       25       -1

Impacted Files
Coverage Δ
Complexity Δ

...ava/com/google/instrumentation/trace/SpanImpl.java
76.47% <100%> (+0.47%)
10 <2> (ø)
:arrow_down:

...ava/com/google/instrumentation/trace/SpanData.java
100% <100%> (ø)
2 <0> (ø)
:arrow_down:

...a/com/google/instrumentation/common/Timestamp.java
82.85% <0%> (-11.43%)
17% <0%> (-1%)

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 0c9fcff...37095f3. Read the comment docs.

Is the dropped events something for noop/unsampled? Or are you anticipating
tracking errors parsing events? Or is it something else?
On 2 May 2017 07:32, "Bogdan Drutu"<EMAIL_ADDRESS>wrote:

You can view, comment on, or merge this pull request online at:
https://github.com/google/instrumentation-java/pull/270
Commit Summary

Add all the informations that we have in the Span to the SpanData.

File Changes

M core_impl/src/main/java/com/google/instrumentation/trace/
SpanData.java
https://github.com/google/instrumentation-java/pull/270/files#diff-0
(177)
M core_impl/src/main/java/com/google/instrumentation/trace/
SpanImpl.java
https://github.com/google/instrumentation-java/pull/270/files#diff-1
(21)
D core_impl/src/main/java/com/google/instrumentation/trace/
TimedEvent.java
https://github.com/google/instrumentation-java/pull/270/files#diff-2
(76)
M core_impl/src/test/java/com/google/instrumentation/trace/
SpanDataTest.java
https://github.com/google/instrumentation-java/pull/270/files#diff-3
(64)
D core_impl/src/test/java/com/google/instrumentation/trace/
TimedEventTest.java
https://github.com/google/instrumentation-java/pull/270/files#diff-4
(62)

Patch Links:

https://github.com/google/instrumentation-java/pull/270.patch
https://github.com/google/instrumentation-java/pull/270.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/instrumentation-java/pull/270, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAD611KHUpFed0jU6aukDoXyYu013Ke1ks5r1muOgaJpZM4NNjXf
.

The library will allow to configure the maximum number of events (of each type) to be kept in memory for each Span. If the number of recorded events is larger than the maximum set in the config then events will be dropped (probably we will keep the latest N events initially).


ic-put-to on form element does not work

I am try to submit a html form via ic-put-to, but the form is still submitted via POST method, here is my code:
<form class="form-inline" ic-put-to="/example" ic-target="#refresh_target">
input text etc.....
<button type="submit" class="btn btn-info">Submit</button>
</form>

Do I use it wrong?

Hi Tony,
Intercooler will always use a POST when submitting the request, due to cross browser issues with AJAX requests for non-GET or POST requests.
We include two pieces of information to indicate the actual HTTP method:

The _method parameter, which is compatible with Rails)
The X-HTTP-Method-Override, which is a google standard

Details can be found here:
http://intercoolerjs.org/docs.html#requests

Very clear explanation, thank you very much.


Changed up command built image tag name

Changed az containerapp up command built image tag name to match its run image.
Merged various changes from yesterday night from branch daniv/variousMinorUpdates and resolved conflicts on new telemetry changes in main branch

Thank you for your contribution! We will review the pull request and get back to you soon.


LITE-23950: remove openpyxl warning about Data Validation

The warning UserWarning: Data Validation extension is not supported and will be removed is shown when a xlsx file with unsupported data validation is opened (typically after saving the file from Excel, which uses some custom extension). To avoid losing the data validation, it should be re-created each time the file is saved by the cli.
So instead of ignoring this warning globally, opted for a more conservative solution. The warning can be explicitly disabled by calling load_workbook(..., no_warn=True), after it has been verified that the data validation is recreated correctly when the file is saved.
For now it is ignored in the known cases where it is handled correctly.

as discussed offline, let's do it global. CLI shall not use them and that's only for excel manipulation....hence better globally since nobody can expect to use it when library in fact does not support it.

done, now the warning is ignored gobally


PC Holographic Remoting Tutorials

Docs Build status updates of commit a5eff5e:
:warning: Validation status: warnings

File
Status
Preview URL
Details

mixed-reality-docs/holograms-100.md
:warning:Warning
View
Details

mixed-reality-docs/mr-learning-pc-holographic-remoting-01.md
:warning:Warning
View
Details

mixed-reality-docs/advanced-hololens-emulator-and-mixed-reality-simulator-input.md
:bulb:Suggestion
View
Details

mixed-reality-docs/performance-recommendations-for-unity.md
:bulb:Suggestion
View
Details

.DS_Store
:white_check_mark:Succeeded

.gitignore
:white_check_mark:Succeeded

.openpublishing.publish.config.json
:white_check_mark:Succeeded

mixed-reality-docs/TOC.md
:white_check_mark:Succeeded
View

mixed-reality-docs/app-patterns-landingpage.md
:white_check_mark:Succeeded
View

mixed-reality-docs/comfort.md
:white_check_mark:Succeeded
View

mixed-reality-docs/community.md
:white_check_mark:Succeeded
View

mixed-reality-docs/design.md
:white_check_mark:Succeeded
View

mixed-reality-docs/development.md
:white_check_mark:Succeeded
View

mixed-reality-docs/device-portal-api-reference.md
:white_check_mark:Succeeded
View

mixed-reality-docs/dialog-ui.md
:white_check_mark:Succeeded

mixed-reality-docs/direct-manipulation.md
:white_check_mark:Succeeded
View

mixed-reality-docs/eye-gaze-interaction.md
:white_check_mark:Succeeded
View

mixed-reality-docs/gaze-and-commit.md
:white_check_mark:Succeeded
View

mixed-reality-docs/gaze-and-dwell-eyes.md
:white_check_mark:Succeeded
View

mixed-reality-docs/hand-coach.md
:white_check_mark:Succeeded
View

mixed-reality-docs/hardware-accessories.md
:white_check_mark:Succeeded
View

mixed-reality-docs/holographic-frame.md
:white_check_mark:Succeeded
View

mixed-reality-docs/holographic-remoting-create-host.md
:white_check_mark:Succeeded
View

mixed-reality-docs/holographic-remoting-version-history.md
:white_check_mark:Succeeded
View

mixed-reality-docs/hololens-emulator-archive.md
:white_check_mark:Succeeded
View

mixed-reality-docs/images/.DS_Store
:white_check_mark:Succeeded

mixed-reality-docs/images/BiographyImages/BronwenZande_270x270.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/BiographyImages/EricProvencher_270x270.jpg
:white_check_mark:Succeeded
View

mixed-reality-docs/images/BiographyImages/april-speight_270x270.png
:white_check_mark:Succeeded

mixed-reality-docs/images/BiographyImages/aysegul-yonet_270x270.jpg
:white_check_mark:Succeeded

mixed-reality-docs/images/BiographyImages/em-lazerwalker_270x270.png
:white_check_mark:Succeeded

mixed-reality-docs/images/BiographyImages/madoka-chiyoda_270x270.jpg
:white_check_mark:Succeeded

mixed-reality-docs/images/BiographyImages/thomas-lewis_270x270.png
:white_check_mark:Succeeded

mixed-reality-docs/images/HandCoach/MRTK_handCoach.jpg
:white_check_mark:Succeeded
View

mixed-reality-docs/images/MRDD/MRDevDaysBanner.png
:white_check_mark:Succeeded

mixed-reality-docs/images/MRDL_Surfaces_1.jpg
:white_check_mark:Succeeded

mixed-reality-docs/images/UX/MRTK_PulseShader_SpatialMesh.gif
:white_check_mark:Succeeded

mixed-reality-docs/images/UX/MRTK_SRMesh_Pulse.jpg
:white_check_mark:Succeeded

mixed-reality-docs/images/UX/MRTK_UX_Dialog.jpg
:white_check_mark:Succeeded

mixed-reality-docs/images/UX/MRTK_UX_Dialog_Types.jpg
:white_check_mark:Succeeded

mixed-reality-docs/images/mixed-reality-openxr-developer-portal.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/mixed-reality-openxr-developer-tools-status.png
:white_check_mark:Succeeded

mixed-reality-docs/images/mixed-reality-openxr-developer-tools.png
:white_check_mark:Succeeded

mixed-reality-docs/images/module3chapter1step12im.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/mrlearning-asa/tutorial4-section1-step1-1.png
:white_check_mark:Succeeded

mixed-reality-docs/images/mrlearning-asa/tutorial4-section1-step1-2.png
:white_check_mark:Succeeded

mixed-reality-docs/images/mrlearning-asa/tutorial4-section1-step1-3.png
:white_check_mark:Succeeded

mixed-reality-docs/images/mrlearning-asa/tutorial4-section1-step1-4.png
:white_check_mark:Succeeded

mixed-reality-docs/images/mrlearning-asa/tutorial4-section1-step1-5.png
:white_check_mark:Succeeded

mixed-reality-docs/images/mrlearning-asa/tutorial4-section2-step1-1.png
:white_check_mark:Succeeded

This comment lists only the first 50 files in the pull request.

mixed-reality-docs/holograms-100.md

Line 10, Column 0: [Warning-h1-not-first] Markdown content is not allowed before H1.

mixed-reality-docs/mr-learning-pc-holographic-remoting-01.md

Line 44, Column 28: [Warning-file-not-found] Invalid file link: 'mr-learning-base-02.md'.
Line 44, Column 117: [Warning-file-not-found] Invalid file link: 'mr-learning-base-02.md'.
Line 46, Column 4: [Warning-file-not-found] Invalid file link: 'mr-learning-base-02.md'.
Line 48, Column 4: [Warning-file-not-found] Invalid file link: 'mr-learning-base-02.md'.
Line 50, Column 4: [Warning-file-not-found] Invalid file link: 'mr-learning-base-02.md'.
Line 52, Column 4: [Warning-file-not-found] Invalid file link: 'mr-learning-base-02.md'.
Line 54, Column 4: [Warning-file-not-found] Invalid file link: 'mr-learning-base-02.md'.
Line 56, Column 4: [Warning-file-not-found] Invalid file link: 'mr-learning-base-02.md'.
Line 58, Column 17: [Warning-file-not-found] Invalid file link: 'mr-learning-base-03.md'.
Line 65, Column 80: [Warning-file-not-found] Invalid file link: 'mr-learning-base-02.md'.
Line 187, Column 81: [Warning-file-not-found] Invalid file link: 'mr-learning-base-08.md'.

mixed-reality-docs/advanced-hololens-emulator-and-mixed-reality-simulator-input.md

Line 51, Column 0: [Suggestion-column-header-missing] Column headers are required for tables. Add appropriate header text. NOTE: This Suggestion will become a Warning in late September, 2020.
Line 94, Column 0: [Suggestion-column-header-missing] Column headers are required for tables. Add appropriate header text. NOTE: This Suggestion will become a Warning in late September, 2020.

mixed-reality-docs/performance-recommendations-for-unity.md

Line 102, Column 0: [Suggestion-code-block-indented] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```). NOTE: This Suggestion will become a Warning in late September, 2020.
Line 166, Column 0: [Suggestion-code-block-indented] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```). NOTE: This Suggestion will become a Warning in late September, 2020.
Line 171, Column 0: [Suggestion-code-block-indented] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```). NOTE: This Suggestion will become a Warning in late September, 2020.
Line 175, Column 0: [Suggestion-code-block-indented] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```). NOTE: This Suggestion will become a Warning in late September, 2020.
Line 195, Column 0: [Suggestion-code-block-indented] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```). NOTE: This Suggestion will become a Warning in late September, 2020.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


feat(back): #1101 globbed attrs

Collect attr paths using python globs for initialization performance
Pass them as attrPaths to evaluator
Redefine evaluator logic for building outputs from attrs
Initialization for universe takes less than 3 seconds now

@kamadorueda Any architectural comments regarding this?

Instead of concatenating them, I will dump all paths into a json string and then parse them in nix. I don't want to use files to avoid IO

@kamadorueda Any architectural comments regarding this?

Not really, I understand the reason for the change. One failure mode to be aware of is that the OS restricts the length of command (between 32kB to 2MB depending on the OS). But at that point, we can switch to a file
Another way to optimize with a more aggressive trade-off is using the fact that people use the build system more often than they modify the build system: so we could add a "generate manifests" step after modifications that produces the list of commands (and everything else needed for the CLI), and which gets versioned in the repo. This way the CLI can consume a "manifest" or "lock" file that already contains the list of outputs, instead of computing it every time. It adds one step to modifications, but saves one step for the average user and also enables faster command auto-completion (as stuff is always ready to consume). The other option is making consumers "initialize the project": Something like "makes install", where before using the project you need to install it, and in this installation step we generate some manifests for the CLI that are reused on all future invocations
in general, if this PR makes it fast enough, I think that's good enough


Fix lowering of single-element numpy array

Fix #1850.
Workaround numpy issue 3147.

[Current coverage][cc-pull] is 73.17%

Merging [#1852][cc-pull] into [master][cc-base-branch] will increase coverage by +<.01%

@@             master      #1852   diff @@
==========================================
  Files           461        461
  Lines         62639      62646     +7
  Methods           0          0
  Messages          0          0
  Branches       6237       6237
==========================================
+ Hits          45835      45841     +6
- Misses        15887      15888     +1
  Partials        917        917

[][cc-pull]

Powered by Codecov. Last updated by 68587f7
[cc-base-branch]: https://codecov.io/gh/numba/numba/branch/master?src=pr
[cc-pull]: https://codecov.io/gh/numba/numba/pull/1852?src=pr


Add early "claim fee" to farms

We will need to create a configurable fee mechanism where user forfeits some amount of his tokens on claim if claiming early.
We will use plannedYieldingPeriods to measure how long the farm is supposed to run and add when your entry date is.
Then on claim we will check your lock with id:*farmId-positionId. The fee is based on the farm settings i.e. after 50% of planned yielding periods after your entry date there is no fee.

Updated... the locks will not work / too expensive and problematic. We are back to fee.

Tested and killed


Not generate columns generate when there is no column prune requirement for join

What type of PR is this：

[x] bug
[ ] feature
[ ] enhancement
[ ] others

Which issues of this PR fixes ：

Fixes #3174
The issue introduces a low cardinality optimization bug. When there is low cardinality optimization, the output slot id of scan will be modified. If you continue to use getLogicalProperty().getOutputColumns(), it will cause an error corresponding to the id.
Problem Summary(Required) ：

run starrocks_fe_unittest


Refactor GET /campaigns index

What's this PR do?
Moves the route definition of GET /campaigns to a new /app/routes/campaign-index, and refactors code from /app/routes/campaign and lib/contentful into middleware.

Adds a Phoenix getCampaigns function to make a  GET /campaigns?ids= to query Phoenix once instead making multiple GET /campaigns/:id requests

Fixes #980: If a Contentful keyword is linked to a Contentful campaign that doesn't have a corresponding Phoenix Campaign, the keyword isn't included in the response (and the response is no longer an error)

Adds a exclude query parameter. If this is set to true, don't query the Mobile Commons API to set the mobilecommons_group_doing and mobilecommons_group_completed properties.

How should this be reviewed?

GET /campaigns - Returns Mobile Commons Groups in response
GET /campaigns?exclude=true - Does not return Mobile Commons Groups in response
Confirm GET /campaigns/:id and POST /campaigns/:id/message work as expected

Any background context you want to provide?
For N campaigns, this endpoint was previously making:

1 request to Contentful for keywords
N requests to Phoenix
N requests to the Messaging Groups API

a single request to this service makes 2 requests to the Mobile Commons API

So 10 campaigns, that's a super sweet 41 HTTP requests?
A query to GET /campaigns?exclude=true with 10 campaigns now makes 2 requests:

1 request to Contentful
1 request to Phoenix

This should help out with regularly querying this endpoint in https://github.com/DoSomething/gambit-conversations/pull/184
Relevant tickets
#983
Checklist

[ ] Documentation added for new features/changed endpoints.
[ ] Tested on staging.

Codecov Report

Merging #986 into develop will decrease coverage by 1.29%.
The diff coverage is 45.63%.

@@            Coverage Diff             @@
##           develop     #986     +/-   ##
==========================================
- Coverage    59.37%   58.08%   -1.3%
==========================================
  Files           71       77      +6
  Lines         1578     1627     +49
==========================================
+ Hits           937      945      +8
- Misses         641      682     +41

Impacted Files
Coverage Δ

lib/contentful.js
93.33% <ø> (-5.18%)
:arrow_down:

app/routes/index.js
100% <100%> (ø)
:arrow_up:

app/routes/campaigns-index.js
100% <100%> (ø)

lib/middleware/campaigns-index/format-campaigns.js
17.64% <17.64%> (ø)

lib/middleware/campaigns-index/parse-keywords.js
20% <20%> (ø)

...middleware/campaigns-index/mobilecommons-groups.js
30% <30%> (ø)

...ib/middleware/campaigns-index/phoenix-campaigns.js
44.44% <44.44%> (ø)

lib/phoenix.js
81.57% <50%> (-15.3%)
:arrow_down:

.../middleware/campaigns-index/contentful-keywords.js
50% <50%> (ø)

app/routes/campaigns.js
25% <53.84%> (+3.3%)
:arrow_up:

... and 8 more

Continue to review full report at Codecov.

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


Feature request: color diff

Hello, first of all thank you for the extension you created, I use it very often!
Do you know if it could be possible to use it while comparing file contents? Indeed, I mostly use it to compare log files in order to detect regressions.

Glad you like the extension! I will look into adding support for diff editors when I have time.


Rename functional interfaces to conform with ferma 3.1.0 changes

Codecov Report

Merging #26 into v2.3.x will decrease coverage by 0.14%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             v2.3.x      #26      +/-   ##
============================================
- Coverage     46.81%   46.67%   -0.15%
  Complexity      465      465
============================================
  Files            48       48
  Lines          2121     2121
  Branches        242      242
============================================
- Hits            993      990       -3
- Misses          981      984       +3
  Partials        147      147

Impacted Files
Coverage Δ
Complexity Δ

src/main/java/com/syncleus/ferma/tx/TxFactory.java
100% <ø> (ø)
6 <0> (ø)
:arrow_down:

...amefactories/annotation/InVertexMethodHandler.java
54.54% <0%> (-9.1%)
4% <0%> (ø)

...mefactories/annotation/OutVertexMethodHandler.java
54.54% <0%> (-9.1%)
4% <0%> (ø)

...mefactories/annotation/IncidenceMethodHandler.java
31.33% <0%> (-0.67%)
16% <0%> (ø)

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 f1e6737...67da4ec. Read the comment docs.


Added a missing translation

Summary by CodeRabbit

New Features

Introduced the ability to link an existing ABHA number by scanning the ABHA QR code, enhancing user experience and accessibility.

LGTM


add a new Lab 04 and re-order/rename subsequent labs

see

https://github.com/Qiskit/platypus/pull/295
https://github.com/Qiskit/platypus/pull/232

new lab has been added/merged


Where can I see etcd data for k3s. I can't see etcd-related processes, and I don't know where the data is stored.

OS：ubuntu 18.04
k3s version: v1.21.4+k3s1 (3e250fdb)
I use the following command to initialize k3s:
curl -sfL http://rancher-mirror.cnrancher.com/k3s/k3s-install.sh | K3S_DATASTORE_ENDPOINT="etcd" INSTALL_K3S_VERSION=v1.21.4+k3s1   INSTALL_K3S_MIRROR=cn sh -s - --service-node-port-range=1-65535  --no-flannel --write-kubeconfig ~/.kube/config
But check the configuration and find that there is no etcd related configuration:
cat /etc/systemd/system/k3s.service
[Unit]
Description=Lightweight Kubernetes
Documentation=https://k3s.io
Wants=network-online.target
After=network-online.target

[Install]
WantedBy=multi-user.target

[Service]
Type=notify
EnvironmentFile=/etc/systemd/system/k3s.service.env
KillMode=process
Delegate=yes
# Having non-zero Limit*s causes performance problems due to accounting overhead
# in the kernel. We recommend using cgroups to do container-local accounting.
LimitNOFILE=1048576
LimitNPROC=infinity
LimitCORE=infinity
TasksMax=infinity
TimeoutStartSec=0
Restart=always
RestartSec=5s
ExecStartPre=-/sbin/modprobe br_netfilter
ExecStartPre=-/sbin/modprobe overlay
ExecStart=/usr/local/bin/k3s \
    server \
	'--flannel-backend=none' \
  	'--service-node-port-range=1-65535' \

Where can I see etcd persistent data of k3s and etcd related startup item configuration?
Thank You!

I believe the files you're looking for are at /var/lib/rancher/k3s/server/db/etcd/

I believe the files you're looking for are at /var/lib/rancher/k3s/server/db/etcd/. Since etcd run embedded within k3s, like all the other components, there isn't a separate systemd unit. If you want to change the configuration of the embedded etcd you should use the --etcd-arg flag to do so.

K3S_DATASTORE_ENDPOINT="etcd" is this parameter useless?

That parameter is only useful if you're going to use an external SQL datastore or etcd cluster. If you're using the single-node sqlite database, or managed etcd via --cluster-init, it should not be set.

I will also note that, given the systemd unit you showed above, you are not using etcd at all. You are using the single-server kine sqlite backend.

I will also note that, given the systemd unit you showed above, you are not using etcd at all. You are using the single-server sqlite backend.
https://rancher.com/docs/k3s/latest/en/installation/datastore/
Now I want to open etcd as k3s storage, no parameters have been added before(K3S_DATASTORE_ENDPOINT="etcd")
Will doing k3s initialization again rebuild k3s?

Thanks

If you want etcd, get rid of the datastore environment variable and add the --cluster-init flag.

If you want etcd, get rid of the datastore environment variable and add --cluster-init to the CLI arguments.

Wouldn't that affect the current cluster?

https://rancher.com/docs/k3s/latest/en/installation/ha-embedded/#existing-clusters

k3s version: v1.21.4+k3s1 (3e250fd)

k3s version: v1.21.4+k3s1
Seeing that 1.22 is supported, is there any way for v1.21.4+k3s1?

No, you would need to upgrade.

/var/lib/rancher/k3s/server/db/etcd/

Could you suggest how to change flags for etcd embedded into k3s? I added desired options to the /var/lib/rancher/k3s/server/db/etcd/config. How to apply it safely? I have a cluster with three nodes. Also, I have configured etcdctl, if it helps.

I added desired options to the /var/lib/rancher/k3s/server/db/etcd/config

don't do that

Could you suggest how to change flags for etcd embedded into k3s?

use the --etcd-arg CLI flag or config file key.
This is a quite old issue, if you have further questions please open a discussion thread.


i18n(fr): update api-reference.mdx and guides/routing.mdx

Description (required)

Updates French translations for api-reference.mdx and guides/routing.mdx (see #9488)
Rephrases a comment in a code snippet of guides/routing.mdx that looks strange

Related issues & labels (optional)

Suggested label: i18n

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

fr
guides/routing.mdx
Localization changed, will be marked as complete.

fr
reference/api-reference.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


Allow running tests using sqlite

Problem
Distro build workers do not want to install postgres in order to run the tests.
Solution
Added capability to use sqlite instead using env flag SQLITE
Commandments

[x] Write PEP8 compliant code.
[ ] Cover it with tests.
[ ] Update CHANGES.rst file to describe the changes, and quote according issue with GH-<issue_number>.
[x] Pay attention to backward compatibility, or if it breaks it, explain why.
[x] Update documentation (if relevant).

ping @auvipy. I can merge but would be bad practise IMO.


ui: Too small window on high-DPI Windows

Maybe I misunderstand the usage of glfwGetFramebufferSize.

https://github.com/hajimehoshi/ebiten/commit/4d53cf20da3366f5e9d1f10e37fcd6ebcf0837b6 Fixed


Add support for python 3

Ok, the PR is in. Thanks for the feedback!

Observation #1 -- different people have different opinions on this, but I prefer to squash commits to keep the commit history clean, unless there are important reasons otherwise. Are you familiar with commit squashing?

General meta-comment. Keep in mind the caveat that every project has different standards, and I'll be coming from a more "corporate" mindset (ie, fairly paranoid about a lot of things) which might be overkill for a casual open source project. Hence, not everything I suggest might be appropriate to go back into upstream project.
For example, most of our projects have either an implicit or explicit requirement that every function have a docblock describing what it does. This is usually good practice if not taken to a crazy extreme. BTW, PyCharm will automagically fill in a docblock with params if you open a triple double quote under the function name and hit return.

Another meta-comment -- before doing any further refactoring, I'd advocate writing some unit tests. Luckily many of these functions are pure functions without external dependencies, so that should be  fairly easy. Using the built-in unittest is probably the best place to start (easiest, doesn't require external tools.) Then you can continue to refactor fearlessly... ;)


Difference in FAMD eigenvalues by prince and PCAmixdata/FactoMineR?

Hello!
First of all, great job on the package! :)
I'm just starting to learn about FAMD, and have been trying to do the analysis in both R and Python. Strangely, while I am getting identical results on a dataset using the two R packages available for FAMD, PCAmixdata and FactoMineR, I am getting quite different results in terms of the eigenvalues with prince. I think I must be accessing the wrong attribute to get the eigenvalues, as more downstream analyses done using prince does give the same results as the two R packages.
For example, this is code that I am using with FactoMineR:
## Import libraries
library(FactoMineR)
library(factoextra)

## Import data
df <- read.csv('https://github.com/nchelaru/data-prep/raw/master/telco_cleaned_renamed.csv')

## PCA
res.famd <- FAMD(df,
                 sup.var = 19,  ## Set the target variable "Churn" as a supplementary variable, so it is not included in the analysis for now
                 graph = FALSE,
                 ncp=25)

## Inspect principal components
get_eigenvalue(res.famd)

And these are the results I am getting:

eigenvalue
variance.percent
cumulative.variance.percent

Dim.1
4.50988153612814
19.6081805918615

Dim.2
3.12384033884342
13.5819145167105

Dim.3
1.82860777021443
7.95046856614967

Dim.4
1.17979883732575
5.12956016228587

Dim.5
1.04993667220825
4.56494205307936

Dim.6
1.01660230193411
4.42001000840917

Dim.7
1.00407432481249
4.36554054266298

Dim.8
0.985272969616078
4.2837955200699

Dim.9
0.90429878988165
3.93173386905065

Dim.10
0.84681410468683
3.68180045516013

With prince:
## Import libraries
import pandas as pd
import prince
import pprint

## Import data
df = pd.read_csv('https://github.com/nchelaru/data-prep/raw/master/telco_cleaned_renamed.csv')

## Instantiate FAMD object
famd = prince.FAMD(
     n_components=25,
     n_iter=10,
     copy=True,
     check_input=True,
     engine='auto',       ## Can be "auto", 'sklearn', 'fbpca'
     random_state=42)

## Fit FAMD object to data
famd = famd.fit(df.drop('Churn', axis=1)) ## Exclude target variable "Churn"

## Inspect principal dimensions
pp = pprint.PrettyPrinter()
pp.pprint(famd.explained_inertia_)

I am getting very different numbers:
[0.5374656498067355,
 0.08801861905276565,
 0.057284992015226376,
 0.03937333799033976,
 0.03127878671244274,
 0.027912264693630378,
 0.02470305613207065,
 0.020807289598498833,
 0.018937227436470073,
 0.018005390670320004,
 0.01656022218673026,
 0.015976762563960006,
 0.014945073037668155,
 0.013999462067402076,
 0.013763382061419126,
 0.013589921877364007,
 0.012208282136007768,
 0.011979370577465807,
 0.011339881479543259,
 0.0070026572256823745,
 0.004847835526330159,
 4.80377794662604e-07,
 5.374980569743592e-08,
 1.0243263922726394e-09,
 5.366068243995469e-33]

I'm sure that I am just calling the wrong thing, but I can't seem to find what I should be using to get the same results as FactoMineR.
Any help will be greatly appreciated! :)

Hey! I'll look into this as soon as I can.

Difference in results on IRIS dataset (including the target variable as part of PCA):
With R:
Call:
FAMD(base = iris, ncp = 3)

Eigenvalues
                      Dim.1  Dim.2  Dim.3
Variance              3.870  1.342  0.592
% of var.            64.503 22.370  9.862
Cumulative % of var. 64.503 86.873 96.735

I tried the same with Prince without normalization:
X,y = load_iris(return_X_y=True)
X = pd.DataFrame(np.hstack([X,y.reshape(-1,1)]))
X.iloc[:,-1] = X.iloc[:,-1].astype("str")
famd = prince.FAMD(n_components=3,n_iter=10)
famd.fit(X)
print(famd.explained_inertia_)

[0.33736997081392395, 0.3314854899776377, 0.33114453920843834]

This looks like it is a data normalization/scaling issue with all principal components essentially depicting  the three levels of the categorical target variable ("species"), i.e. no proportion of variance is explained by the continuous features!
To confirm this, if one sets the arguments rescale_with_mean=True and rescale_with_std=True in the super().__init__ method of mfa.py, i.e. in the global pca - the results look better:
print(famd.explained_inertia_)

[0.6412568596845379, 0.2592472748780183, 0.09949586543744376]

What do you think is going on @MaxHalford ?

Hi,
Could it be because in factominer, by default, the continuous variable will be scaled?
From the documentation https://cran.r-project.org/web/packages/FactoMineR/FactoMineR.pdf

More precisely, the continuous variables are scaled to unit variance and the categorical variables are transformed into a
disjunctive data table (crisp coding) and then scaled using the specific scaling of MCA

In Prince, see: https://github.com/MaxHalford/prince/blob/master/prince/mfa.py#L58
it is not.
Would make sense to change de default, or to get a parameter maybe?

Hi,
Could it be because in factominer, by default, the continuous variable will be scaled?
From the documentation https://cran.r-project.org/web/packages/FactoMineR/FactoMineR.pdf

More precisely, the continuous variables are scaled to unit variance and the categorical variables are transformed into a
disjunctive data table (crisp coding) and then scaled using the specific scaling of MCA

In Prince, see: https://github.com/MaxHalford/prince/blob/master/prince/mfa.py#L58
it is not.
Would make sense to change de default, or to get a parameter maybe?

Hello there 👋
I apologise for not answering earlier. I was not maintaining Prince anymore. However, I have just refactored the entire codebase. This refactoring should have fixed many bugs.
I don’t have time and energy to check if this fixes your issue, but there is a good chance it does. Feel free to reopen this issue if the problem persists after installing the new version — that is, version 0.8.0 and onwards.


ErrorException in YoutubeDownloader.php line 483: fopen(videos/Kavinsky_-_Protovision.mp4_temp_1478159094): failed to open stream: No such file or directory

I have this in controller and when open in browser getting "ErrorException in YoutubeDownloader.php line 483: fopen(videos/Kavinsky_-_Protovision.mp4_temp_1478159094): failed to open stream: No such file or directory" error,
I'm using laravel 5.3
public function getInfo(){
$youtube = new YoutubeDownloader('41_svUt5_e0');
//$result = $youtube->getInfo();
$result=$youtube->download(18);
//dump($result);
}

I find the reason, beacuse i do not create the directory of  videos  ..........................., so i add the code to adding directory.
thanks

Thanks. I'll make it more clear in the README file

I have a problem with returned $filePath from onFinalized()
C:\xampp\htdocs\youtuder\public\videos\FC_Barcelona_vs_FC_Bayern_Munich_2015_（Champions_League_semi-final_Himno,Anthem ）.3gp
can you see spaces after ')' ?
original video:
https://www.youtube.com/watch?v=sMq_A0ETURQ


removes orange line and makes account logo a link to account route

Description
removes orange line and makes account logo a link to account route
Type of change
Styling
Please delete options that are not relevant.

[ x ] New feature (non-breaking change which adds functionality)

Checklist:

[ x ] My code follows the style guidelines of this project
[ x ] I have performed a self-review of my own code
[ x ] I have removed unnecessary comments/console logs from my code
[ x ] I have made corresponding changes to the documentation if necessary (optional)
[ ] My changes generate no new warnings
[ x ] I have checked my code and corrected any misspellings
[ x ] No duplicate code left within changed files
[ x ] Size of pull request kept to a minimum
[ x ] Pull request description clearly describes changes made & motivations for said changes

I think Ryan had just left feedback to ensure that all the related styles were also removed. It is not a code breaking issue but a suggestion to remove potentially unnecessary code. Once that is checked it can be merged.


toascii - Give Masks a More Useful Output Format

There are masks in the API not associated with enum flags (e.g. https://registry.khronos.org/vulkan/specs/1.3-extensions/man/html/VkRenderPassMultiviewCreateInfo.html).
They are currently output as the decimal value of their underlying type such as uint32_t. A useful representation would be a list of the bits in them that are lit up, such as:
[1,3,8,31]

Alternative representations:

C++'s binary literal format in a string: "0b0101010101111"
A hex value in a string: "0xabc9484"

In the Python generators, these generally fall under the else at the end of if elifs. e.g.:
elif value.is_pointer:
                if value.is_array:
                    if self.is_handle(value.base_type):
                    ...
                    else:
                        toString = 'ArrayToString ... # Mask handling currently generated here.

See Also
There is a related issue for when named flags bits are ORed together:
Flags should be printed as expanded flag values #620

I am just dropping by to say that this is something very useful.
For example the following pipeline barrier:
{
    "index": 14288,
    "vkFunc": {
        "name": "vkCmdPipelineBarrier",
        "args": {
            "commandBuffer": 2600,
            "srcStageMask": 33554432,
            "dstStageMask": 33554432,
            "dependencyFlags": 0,
            "memoryBarrierCount": 1,
            "pMemoryBarriers": [
                {
                    "sType": "VK_STRUCTURE_TYPE_MEMORY_BARRIER",
                    "pNext": null,
                    "srcAccessMask": 4194304,
                    "dstAccessMask": 6291456
                }
            ],
...

It takes some effort to decipher the access masks and pipeline stages, especially when there are many different ones.
Imo these should be printed with their enum names, like #620 is describing.


bold current lesson in menu

visual indication of current page in the lessons menu to improve navigation 🙂 (the old layout changed the color of the lesson listing)

@thth do you mind running mix format against the code and then pushing up the changes to appease the CI action?


Angular 2 Upgrade?

Hello,
Ive been looking at this  project recently trying to keep updated with whats going on. I must say, it looks really really nice now.
I just want to ask if there is plans to upgrade to Angular 2.0?
Its still in beta so i don't see it happening any time soon but is it planned?

Hi @ChadTaljaardt
Thanks :smile:
Angular 2.0 is definitely on the list. But I'm not sure if it would be suitable to do it before at least having a release candidate of Angular 2..
Updating vendor library usually depends on the availability of plugins that this library is using.. For example, I had to wait for jwt-auth and dingo/api to be compatible with Laravel 5.2 so I can start the upgrade ( #121 )..
But I'm guessing we won't be facing this issue here (mainly with ui-router, angular material, etc..) because of the ng-upgrade which allows including angular 1 components within an angular 2 app.
I'm going to keep this opening, if anyone would like to work on this, let's collaborate

I think it'll be better to smoothly update the code based on the 1.x's incrementations because Angular's developers tries to force users to migrate every incr to Angular 2 syntax. But non open source porject, companies will still use Angular1 as reference as long as ther will be LTS on It (about ~5y)

@rooferz I think you misunderstood me
1.x will be definitely upgraded (when 1.5 is released, we'll definitely upgrade to it) since this is a starter project that encourages best practices (angular's .component)
However including angular 2 when in beta might not be very wise, so we'll probably wait for a release candidate or official release before upgrading to angular 2
Let me know what you think :smile:

@jadjoubran this is exactly what I want tot tell you x)
Sorry, hangover time

haha awesome!

Just released 3.2
I will close this issue as the next release will definitely be about Angular 2
so the planning will be on the trello board


Simple(r) 2D detection debuggin overlay

Add a C++ node to collect images and render received 2D detections on the corresponding specific source image if it is in our history.

Added fixup commits. Let me know if they're OK by you and I can rebase + merge.

Added fixup commits. Let me know if they're OK by you and I can rebase + merge.

Looks good to me!


revise styles for title in index page

so it's within the cols

 [TARIFF16] Elsewhere on gov.uk right column on tariff startpage


Basic equlaity and inequality test is failing on 0.5.6 release

The below one is giving me wrong result.
template  bool testInEqualityEWAHBoolArray() {
cout << "[testing testInEqualityEWAHBoolArray] sizeof(uword)=" << sizeof(uword) << endl;
EWAHBoolArray b1;
EWAHBoolArray b = EWAHBoolArray::bitmapOf(3, 1, 10, 11);
return !(b1 == b);
}
I am expecting this should return true, where as this returns false.

Thanks. Fixed in release 0.5.7.


Email points to Email page,  but description points to SMS

Under common scenarios heading, check first table for errant description.
Page uses phone (mostly) but also telephone. Different meaning?

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: d5c2b60c-dd58-86ec-9186-2e18720f66c5
Version Independent ID: eda0832c-3c21-2c4d-6364-25a77556fb3c
Content: What is Azure Communication Services?
Content Source: articles/communication-services/overview.md
Service: azure-communication-services
GitHub Login: @tophpalmer
Microsoft Alias: chpalm

@jenellea
Thanks for your feedback! We will investigate and update as appropriate.

@jenellea Thanks for reaching here! Please see detailed document here on Phone number types in Azure Communication Services

@jenellea We will close this out but if you feel you need more information please just let us know.


Reach Out to Microsoft Word Team

Reach out to the Microsoft word team to get more info on their inclusion linter

There were two docs shared (that are internal so I will not share the links here. Unfortunately, they were more to do with Accessibility than inclusion so I have followed up again with Brock. This story is blocked until I hear back!

@sbohlen commented Brok is OOF and does not have a date of return.


Taligram

I am using this telegram account for my business purpose but may account banned please for give me if I did any mistake again please unban my account I will thanks full you 🙏

If you think that an account was banned by a mistake, please try to log in to the account from an official mobile app and send an email to recover the account by following instructions provided by the app.

My mistakes

Tarif223311


FIXED: Rotation works improperly on retina display #683

Hi,
I made some changes that work for me. But changes are pretty serious and I am afraid they may affect other parts of the application. Could you review my changes, please? If that solution is good enough I would be glad if you will merge it into the main branch.
Thank you.

Sorry, this pull request was incorrect. Had to close it.


test PR, do not upvote

Rob Browning<EMAIL_ADDRESS>writes:

rbrw requested changes on this pull request.

@@ -40,6 +40,9 @@
:character-encoding "UTF-8"
:accept :json}))))

+;; And a change here
+
This is a test.

This is a reply.

@@ -21,6 +21,8 @@
(metrics/keyword->metric-name
[:global] :message-persistence-time))}))

+;; And a change here
+
This is only a test.

This is only a reply.

@@ -10,6 +10,8 @@
(defn transfer-meta [old-node new-node]
(with-meta new-node (meta old-node)))

+;; In another commit
In the event of an actual emergency...

In the event of an actual reply...

--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/puppetlabs/puppetdb/pull/2169#pullrequestreview-18697838

Test PASSed


Bug/landing page font

I resized the font's so that they would not overlap and become muddy when screen is resized. I also implemented flexbox and media queries so that the font would remain readable and in its correct place when viewed on alternate devices and screen sizes.
Fixes # (font overlapping and images bleeding over)
Loom Video
https://www.loom.com/share/76b5140f98054644850ca6d8b8bc212b
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have removed unnecessary comments/console logs from my code
[x] I have made corresponding changes to the documentation if necessary (optional)
[x] My changes generate no new warnings
[x] I have checked my code and corrected any misspellings
[x] No duplicate code left within changed files
[x] Size of pull request kept to a minimum
[x] Pull request description clearly describes changes made & motivations for said changes

I went ahead and resolved the merge conflict on this PR, just had to remove the "parentDashboard" import from index.js

Hi, Always perfect job, This is our pair programming, I have double check here, A great job.


Set up HTTPS

Note that this isn't possible with GitHub Pages, unless you put a CDN in front of it.

HTTPS is not supported for GitHub Pages using custom domains.

https://help.github.com/articles/securing-your-github-pages-site-with-https/

We're actually going to be hosting the new site on Federalist.

Closing out, will not fix here as we are retiring the site.


Not capturing during night

If I enable daytime capture I can get a live preview. If I turn daytime capture off, it never captures at night. I get the yellow text screen in live view saying daytime capture is off. Says it all night.

@OrionA51 Please follow the instructions in the Reporting Issues Wiki page and attach the log file.  Start Allsky about 10 minutes before your nighttime begins, and let it runs for about 10 minutes after nighttime begins.  I will need to see what happens at the transition.

I’ve been leaving it run and it is now imaging. Daytime is set to false. So it looks to me like it’s a time issue. Or time zone issue. Under camera settings - other settings, I do have the coordinates input. The Rpi time is correct too. I’ll look into getting the log file and also see if it stops imaging at or near sunset.
Ryan C.

On Apr 2, 2022, at 12:14 PM, EricClaeys @.***> wrote:

@OrionA51 Please follow the instructions in the Reporting Issues Wiki page and attach the log file. Start Allsky about 10 minutes before your nighttime begins, and let it runs for about 10 minutes after nighttime begins. I will need to see what happens at the transition.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.

Ok so right at sunset the capture stopped and I get the screen that says capture is off during the day. I assume that should be easy to fix? Just a matter a reversing something…
Ryan C.

On Apr 2, 2022, at 12:14 PM, EricClaeys @.***> wrote:

@OrionA51 Please follow the instructions in the Reporting Issues Wiki page and attach the log file. Start Allsky about 10 minutes before your nighttime begins, and let it runs for about 10 minutes after nighttime begins. I will need to see what happens at the transition.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.

 I had my longitude set as -81W. Which is correct, but the program as I understand cannot use a "-" sign. I understand now that "W' or "E" is the determining factor for "-" or "+". Using a -+ sign is mainly for GPS systems.  Sorry to about opening an issue too soon.

@OrionA51 Glad you figured it out.
I will add a check for a minus sign with a N, S, E, or W, which should be illegal.
I modified a test version of the website to allow either the +/- or N,S,E,W version.  I think the WebUI should allow either form.
If your problem is resolved,  please close the Issue.


Workbook

Initial work to solve #447

Thanks @jacek-kurlit !
I'll take a look here in the next few days. Don't worry about the failing tests. Its because of forks not having access to GitHub Secrets. I'll run the tests locally. I'll let you know soon. There may be a few things needing to change but from what I saw while glancing it looked decent.

#448 will be finished in #450
Thanks for the contribution @jacek-kurlit @JK-Kitopi
See #450 for my explanation on why I had to move it to another branch. Your commits are in that branch as well.


Exchanged 'chalk' for 'colorette' because it's almost 2x faster :)

Instead of upgrading 'chalk' I thought about exchanging for 'colorette' cause it's faster and I saw the code could be easily exchanged.
All tests passed.

LGTM，and I'll fix GitHub pages later.

It's included in 2.3.1 now.


Migrate emmetio package to monorepo

Our CI is failing because it appears the @emmetio/expand-abbreviation package has been dropped from the NPM registry. This package has since been amalgamated into a new monorepo by emmetio here: https://github.com/emmetio/emmet/tree/master/packages/abbreviation
This PR updates to the new package. Jest (npm run js:test) is green locally.

@Sjeiti would you be able to cut a new release to npm? (Apologies if you've done this already and NPM is just not showing the newest version yet)


Updating some messages in the pt-BR locale file - [2]

Description
While browsing today I checked out some new updates.
On the left as it is on the black social network.

I've made some translations/corrections for pt-BR that consist mainly in:
Changing 'kit' to 'pacote' (per official use in the website and use in Brazilian news);
Some minor grammatical corrections;
Correction of minor consistency errors;
Translation of a few (very few) strings.
Test

Change to PT - BR language.
And test the information I provided.

@gleydson
I put  all the changes in this new committee @Garccez #5090

@gleydson I did a thorough search and adjusted them all in this new commit.

Please just tag me whenever you all are ready for this one to get merged in!


Incorrect file permissions set of volumes

I am trying to use the mqadvanced image. As part of the setup, when this container first runs it creates a number of files on a exposed volume. When the files are created they are created with a specific UID but when you look at the UID that the files have been given it matches the host OS logged in user.
Within the container, the files are created with the with the the UID & GID of 1414 but then on my machine the files have been created with the UID matching my locally logged in user.
Strangely when the container restarts and it looks at the files, it sees a UID of 1000 on the files.
Found on v 0.5.12

Thanks for the issue,
Are the files in the /var/mqm directory? They may have UID of 1000 because of the VirtualBox shared folders feature used to host-mount volumes on Mac OS X.

Yes the files are created in the local /var/mqm directory

For now, would this work? Re-create the container, but make sure not to:

Click on the "edit files" directory icons in Kitematic (that will host mount the volumes)
Choose a directory to host mount

The UI/UX around mounting volumes needs some work, thanks for posting the issue.

It looks like the problem is down to boot2docker issue #587  which has been around for months so doesn't looks like there is an outlook on the fix.
Is it possible to enable the ability to use the storage on the boot2docker image instead of a local folder i.e. something like /mnt/sda1/mqadvanced/. Running with that from the docker command line seems to work.

Why not use NFS?
https://github.com/boot2docker/boot2docker/issues/581#issuecomment-74535277

We could also add a guide telling this helps:
https://github.com/mybethel/docker-drupal-kitematic/compare/master...timbrandin:master


which JDK version

The config.sh has the default/example JDK set to 18, but Android build script expects a file rt.jar which was apparently removed in JDK 14?
EDIT: Apparently it was removed in JDK 9? Does that mean JDK 8 is required? (raylib itself links to JDK 13)

I'm fairly sure I used JDK 8 with my projects, but it's been a while. That was the JDK version I had installed via my package manager, so I might not have tested it on anything newer.


Integrate new core events

Description
I decided to move the four main events (test case / step starting / finished) down to the core, so that we don't need a special "report" API anymore. The EventBus has moved down to the core too. I extended it a bit so you can give it multiple namespaces for events, so we can use a mix of Cucumber::Core::Events and Cucumber::Events.
Motivation and Context
This simplifies the code, and allows more freedom / flexibility for subscribing to events from the runner.
Type of change
This is a breaking change to the events API, and will need to go into a major release (slated for 3.0)
TODO:

[ ] @tooky @brasmusson @danascheider review please
[ ] improve the docs?
[ ] Merge and release cucumber/cucumber-ruby-core#106

Now when #920 has been merged, the file lib/cucumber/filters/retry.rb needs to be updated for the moving of the event bus.

Thanks for merging @brasmusson. Brave move!
I think this constitutes a breaking API change - the events API has changed, so we should release it as 3.0. Shall we have the master branch target 3.0 now? I had hoped to get all the formatters replaced and deprecate the old formatter API with a 3.0 release, but we could do that in a 4.0 release.

If this should be released as 3.0, then we should have the master branch targeting 3.0. As there are a few bugfixes and upgrade to Gherkin 4.0 unreleased on the master before this, we should branch or maintenance branches before this merge and release a cucumber-core v1.5.0 gem and a cucumber v2.4.0 gem, I think (and then not intend to do anything else on those maintenance branches, but aiming that v3.0 will be the next release).

Yes, I agree @brasmusson.

If anyone has time, tidying up the milestones (and assignment of issues to them) to reflect this decision would be a useful exercise.

Now are 'cucumber' v2.4.0 and 'cucumber-core' v1.5.0 released (from the new branches v2.x-bugfix and v1.x-bugfix.

I should be able to do that @mattwynne

Fantastic, thanks @brasmusson!
Protocol-wise, do you think we should blog an announcement (e.g. https://cucumber.io/blog/2015/09/11/cucumber-2.1) and link to that from the google group, or just post on the google group?
I've not been very consistent in the past - it would be good to just make a rule and stick with it.


Add CRUD operations for categories

This commit adds a new "categories" section in the data management tab
such that categories can be created, deleted and updated.
Changes:

Add category management to data management tab.
Retrain bayes classifier when a category is deleted or changes for a
transaction.
Refactor reducers into separate files for each separate reducer.

Codecov Report

Merging #10 into master will decrease coverage by 0.48%.
The diff coverage is 50%.

@@            Coverage Diff             @@
##           master      #10      +/-   ##
==========================================
- Coverage   43.46%   42.98%   -0.49%
==========================================
  Files          17       22       +5
  Lines         352      463     +111
==========================================
+ Hits          153      199      +46
- Misses        199      264      +65

Impacted Files
Coverage Δ

src/configureStore.js
0% <ø> (ø)
:arrow_up:

src/components/Intro.js
0% <ø> (ø)
:arrow_up:

src/components/RestoreData.js
0% <ø> (ø)
:arrow_up:

src/components/modals/SaveData.js
0% <ø> (ø)
:arrow_up:

src/components/Data.js
0% <0%> (ø)
:arrow_up:

src/index.js
0% <0%> (ø)
:arrow_up:

src/components/modals/Confirm.js
0% <0%> (ø)

src/components/Categories.js
0% <0%> (ø)

src/reducers/categories.js
100% <100%> (ø)

src/actions.js
100% <100%> (ø)
:arrow_up:

... and 9 more

Continue to review full report at Codecov.

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


New install does not have Hass.IO on sidebar

New installation using the shell command does not show the hass.io on the side like the 3.8 version I had previously installed. Trying to do a new install with 3.13 but no hass.io option

Apparently it's "Supervisor" now... weird.


HOS-400: adding support for config

All Submissions:

[ ] Have you followed the guidelines stated in CONTRIBUTING.md file?
[ ] Have you checked to ensure there aren't any other open Pull Requests for the desired changed?

Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

New Feature Submission:

[ ] Does your submission pass the tests?
[ ] Have you linted your code locally prior to submission?

Changes To Core Features:

[ ] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your core changes, as applicable?
[ ] Have you successfully ran tests with your changes locally?

After these steps, you're ready to open a pull request.
a. Give a descriptive title to your PR.

b. Describe your changes.

c. Put `closes #XXXX` in your comment to auto-close the issue that your PR fixes (if such).

Hi  I want to contribute but I don’t know where to start
If anyone is seeing this I have knowledge 1.5/10 in python but I have web
development experience. I am eager to learn and learn rapidly using axiom
understanding.
I’m open to any suggestions for helping in coding/research .
On Tue, 24 Dec 2024 at 9:51 PM, jajajagg12 @.***> wrote:

@.**** approved this pull request.
—
Reply to this email directly, view it on GitHub
https://github.com/reflex-dev/reflex/pull/4540#pullrequestreview-2522193095,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A7J5JJV4GC7DIHH5G6QIHF32HG3M7AVCNFSM6AAAAABTTBNI2OVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDKMRSGE4TGMBZGU
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Running functions through CLI throws error compared to when ran in VS 2017

When running in VS 2017 15.3 Preview, I am able to run the function.
However, when I try to run the function through CLI after building the project, then I get an error.
A simple function method in my project is the following:
public static async Task RunAsync([BlobTrigger("profile-pictures/{name}")] CloudBlockBlob myBlob,  string name, TraceWriter log)

The function.json is:
{
  "bindings": [
    {
      "type": "blobTrigger",
      "path": "profile-pictures/{name}",
      "direction": "in",
      "name": "myBlob"
    }
  ],
  "disabled": false,
  "scriptFile": "..\\Facebook.Functions.dll",
  "entryPoint": "Facebook.Functions.ResizeImage.RunAsync"
}

I get the following running func run {FunctionName}

mscorlib: Exception while executing function: Functions.ResizeImage. Microsoft.Azure.WebJobs.Host: One or more errors occurred. Exception binding parameter 'myBlob'. Microsoft.Azure.WebJobs.Host: Blob identifiers must be in the format 'container/blob'.

You need to pass a container and blob name to func run, via the -c flag.


Non existing secret catch

Updating strings and making compatible with translation
https://github.com/stolostron/backlog/issues/17841

It appears that setCreationStatus is not expecting a translation key. We need to translate the string first. If there is no straightforward way to do this, that part can be left for a future update, as I see we have many cases in the code that are using this function with hard-coded strings.

I would also suggest that you do not try to re-use your branch for subsequent PRs. If you look above, it looks like you are trying to merge 20 commits. It also cause the dco check to fail because the merge commits are not signed-off. To clean this up, you could do the following.

Make sure your main branch is up-to-date
Switch to your branch.
Make a copy of your branch to try this out: git checkout -b secret-catch-strings (This is optional -  you can do this in place if you are comfortable then force push this branch to update this PR: git push -f)
Start an interactive rebase: git rebase -i main

At this point an editor opens. Here you can select which commits to keep. You should see the following:
pick cf407292 Adding error catch for non existing ansible secret
pick c3952af8 Filtering template dropdown when creating new cluster to avoid showing templates with non existing ansible secrets
pick db172e19 Adding tooltip for template dropdown
pick 09a4780d Reformatting to pass tests
pick 6222e6f4 Update plugin__acm-plugin.json to include "Add credential" (#1126)
pick 30071a25 Add provider connection changed to add credential (#1121)
pick 1046c3a4 Changing "common:learn.more" to "Learn more" on import cluster page (#1127)
pick c50c61fc Fix appearance of the Missing InnfraEnv warning (#1109)
pick cb7050c5 Do not show Unbind action for BMHs or AI flow (#1095)
pick 821f3bd9 add emptystates for each grc tab (#1124)
pick 6a7f6984 feature + testing (#1116)
pick 51f5dfe8 Update proxy path (#1111)
pick ab144d85 Update builder image (#1129)
pick f04af078 Removing unread variable
pick dde56991 Removing unread variable
pick 9ef15732 Adding deleted cred error status to translation
pick 05b55756 Updating template tooltip string

# Rebase 7c18d782..b6ed456f onto 7c18d782 (17 commands)
#
# Commands:
# p, pick <commit> = use commit
# r, reword <commit> = use commit, but edit the commit message
# e, edit <commit> = use commit, but stop for amending
# s, squash <commit> = use commit, but meld into previous commit
# f, fixup [-C | -c] <commit> = like "squash" but keep only the previous
#                    commit's log message, unless -C is used, in which case
#                    keep only this commit's message; -c is same as -C but
#                    opens the editor
# x, exec <command> = run command (the rest of the line) using shell
# b, break = stop here (continue rebase later with 'git rebase --continue')
# d, drop <commit> = remove commit
# l, label <label> = label current HEAD with a name
# t, reset <label> = reset HEAD to a label

Edit this to drop all but the last 2 commits:
drop cf407292 Adding error catch for non existing ansible secret
drop c3952af8 Filtering template dropdown when creating new cluster to avoid showing templates with non existing ansible secrets
drop db172e19 Adding tooltip for template dropdown
drop 09a4780d Reformatting to pass tests
drop 6222e6f4 Update plugin__acm-plugin.json to include "Add credential" (#1126)
drop 30071a25 Add provider connection changed to add credential (#1121)
drop 1046c3a4 Changing "common:learn.more" to "Learn more" on import cluster page (#1127)
drop c50c61fc Fix appearance of the Missing InnfraEnv warning (#1109)
drop cb7050c5 Do not show Unbind action for BMHs or AI flow (#1095)
drop 821f3bd9 add emptystates for each grc tab (#1124)
drop 6a7f6984 feature + testing (#1116)
drop 51f5dfe8 Update proxy path (#1111)
drop ab144d85 Update builder image (#1129)
drop f04af078 Removing unread variable
drop dde56991 Removing unread variable
pick 9ef15732 Adding deleted cred error status to translation
pick 05b55756 Updating template tooltip string

# Rebase 7c18d782..b6ed456f onto 7c18d782 (17 commands)
#
# Commands:
# p, pick <commit> = use commit
# r, reword <commit> = use commit, but edit the commit message
# e, edit <commit> = use commit, but stop for amending
# s, squash <commit> = use commit, but meld into previous commit
# f, fixup [-C | -c] <commit> = like "squash" but keep only the previous
#                    commit's log message, unless -C is used, in which case
#                    keep only this commit's message; -c is same as -C but
#                    opens the editor
# x, exec <command> = run command (the rest of the line) using shell
# b, break = stop here (continue rebase later with 'git rebase --continue')
# d, drop <commit> = remove commit
# l, label <label> = label current HEAD with a name
# t, reset <label> = reset HEAD to a label

Save your changes. The rebase will complete. Using git log, you can see that now you have only 2 commits on your branch beyond what is on the main branch, and both have the signed-off-by.
% git log
commit ee7a2d9b510588b62bafd5a51f1ab09a297b3eaa (HEAD -> secret-catch-strings)
Author: Ella Juengst<EMAIL_ADDRESS>Date:   Fri Jan 28 13:34:54 2022 -0500

    Updating template tooltip string

    Signed-off-by: Ella Juengst<EMAIL_ADDRESS>
commit 2993517ee48d1dfd01f8bb4afd1f15f82adb20dc
Author: Ella Juengst<EMAIL_ADDRESS>Date:   Fri Jan 28 13:34:07 2022 -0500

    Adding deleted cred error status to translation

    Signed-off-by: Ella Juengst<EMAIL_ADDRESS>
commit 7c18d782b61f15887c2d83252edda46e7b80cfef (origin/release-2.5, origin/main, main)
Author: Kevin Cormier<EMAIL_ADDRESS>Date:   Fri Jan 28 16:28:07 2022 -0500

    Restore cluster status popover (#1134)

Then you can push this branch to GitHub and do a new PR, or force push to update this PR if you worked on your existing branch.

@KevinFCormier On it! By "If there is no straightforward way to do this, that part can be left for a future update," do you mean that I should hardcode the string in the error message for now (if no straightforward way to translate first)?


Update installation.rst

people get permission errors with the "git@github.com:" address.

Could you say more about the errors? On Linux the SSH clone can be more convenient than the HTTPS clone.

@dwfncar do you want to weigh in?

I had permission errors installing melodies-monet on cheyenne with the<EMAIL_ADDRESS>command.
Another user, using MacOS Catalina got these errors:
(melodies-monet) [Xeon:~/melodies] cbardeen% git clone git@github.com:noaa-oar-arl/monetio.git
Cloning into 'monetio'...
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository.
Please make sure you have the correct access rights
and the repository exists.
(melodies-monet) [Xeon:~/melodies] cbardeen% git clone git@github.com:noaa-oar-arl/monet.git
Cloning into 'monet'...
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository.
Please make sure you have the correct access rights
and the repository exists.

If you create a public/private RSA key pair on Cheyenne/Casper and then add the public one to your https://github.com/settings/keys it should work.

We could include both - say use what works - and maybe you could add an explanation of what the ssh clone is more convenient.

@rschwant anything to add about your experience?

I didn't have to enter a password using the weblink - maybe that changed
when melodies became public?
On Tue, Mar 8, 2022 at 4:07 PM Zachary Moon @.***>
wrote:

maybe you could add an explanation of what the ssh clone is more convenient
The main benefit in my experience is that you don't have to enter your
GitHub password/token for everything. On Git for Windows, you get this
automatically with an HTTPS clone because it caches your password in the
background. But not on Linux without doing extra work.
—
Reply to this email directly, view it on GitHub
https://github.com/NOAA-CSL/MELODIES-MONET/pull/74#issuecomment-1062338863,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AH5BH7IIHDF7VP2RANWHWZLU67MSRANCNFSM5QHLRADA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

I didn't have to enter a password using the weblink

You don't need to enter the password/token to clone the repo with HTTPS. But in my experience on Linux you do need it if you want to push/pull/whatnot.

Setting up an ssh account is needed to do the<EMAIL_ADDRESS>Before the repository was public you had to set up an ssh key in order to clone the repository because it was private.
Maybe we can just recommend people set up an ssh account. This is definitely the easiest way to use github and recommended by github to setup. Maybe we can add something like the following:
If you are new to GitHub, check out this Github tutorial. We recommend adding an ssh key. If you do not add an ssh key, you will need to clone the repository with the following command instead of the one recommended below: git clone https://github.com/NOAA-CSL/MELODIES-MONET
What do you think about this @zmoon and @lkemmons

Yeah I think that sounds good. I can add that info as a footnote on this this docs page and reference it in the relevant section(s)?

Yes thanks Zach that would be great.

I added some notes about this to both install docs pages as part of #70. Thanks @lkemmons and @rschwant for the discussion.


Is there a way to remove remove aspects?

Loving this gem. Is there a standard way to remove applied aspects? When I'm testing I want to reset my application to a pure state between tests and I'd like to remove all applied aspects as part of that reset.
Thoughts?
Thanks for your work on this!

Hey, @thedeeno, thanks for the support! :)
You cannot unbind the whole aspector, however you can always disable all the aspects that you don't need / don't want:
aspect.apply(ExampleClass)
element = ExampleClass.new
element.test
aspect.disable!
element.test

# Expected output
# do_this
# do_that
# test
# test

and you can reenable it later. Aspector heavily integrates with given methods but if disabled - won't run at all (no aspector logic except a single check will be performed).

Do you think it's dangerous to blow away the self.class.storage for target's? I'm imagining an implementation that saves a list of constants with advice applied and a Aspector#reset! method that iterates over this list and reset's each constant's storage.
Would that work? Or are we hooking even lower than that?

I mention that because in my case disabling isn't enough. If possible we need to remove all advice so we effectively have a "pre-aspector" runtime.

@thedeeno I'm really curious about the case - could you explain why do you need that? :)
Blowing up storage won't help. Storage is used to store information where (and how) a given aspect was applied. It was introduced in order to support multi applying same aspect with different options and to support applying to not yet existing methods (once they are defined, we need to know how to apply aspects to them)
Applied aspect heavily (really heavily) integrates with methods to which it is being applied (it deattaches them, wraps the new logic around (or after/before/etc) and then puts the old logic in place (kinda like prepend in 2.2.0 but with logging logic and some other things that are done different). Resetting is possible, however it won't be as easy as you think due to supporting of resetting multiaspect applied methods/objects.
I can implement this feature, however it will require some time.
Note that resetting will require a given aspect to be applied again (there won't re unreset or anything like that)

When testing we want to apply different sets of advice to simulate different configured runtimes. Some environments will have logging, some will have metrics, etc. The way to solve this now is to apply in separate test runs (in their own processes) so we don't pollute the object model cross test. Maybe there's a better way to deal with this?
I think it'd be awesome if this library supported at least global reset or perhaps even target-level resets. Applying/removing advice wouldn't feel so permanent if we could reason about it like that.
Unreset isn't necessary. Cool!

@thedeeno  I've been looking into this recently and it creates some problems with ordering. Let's say that I have 2 aspects (AA, AB) that I apply. Aspector stores the unbind method (original base for adding aspects) and it could be restored, however when I apply the AA and after that AB and I reset! from AA, the AB aspect will go away as well (because for us the native method is the method from the beginning). But then if I reset the AB, it will restore the AA aspect.
The only solution that I see (at least now) is to make aspects aware of each other and that is a really bad idea (memory consumption, performance, etc). However if you still want to do what you are doing, you could unbind the methods that will be replaced with aspected versions and the overwrite them back again the way you want.
I'm not saying now right now - but it seems that this feature requires a lot of planning and it might turn out that it won't be possible to implement due to memory consumption and performance.

Deprecated and no longer maintained.


refactor: remove non translated english locales

Yes it makes sense to remove it. But it should also be good to have that config at only one place?

@Jisay Is there any internal ticket reference related to that change?

Yes it makes sense to remove it. But it should also be good to have that config at only one place?

@Jisay I agree, we should have this list at one place and reuse it everywhere its needed

Yes it makes sense to remove it. But it should also be good to have that config at only one place?

@Jisay I agree, we should have this list at one place and reuse it everywhere its needed

@marie-j I have centralized the available languages in @ovh-ux/manager-core as @ovh-ux/manager-navbar has this module as peerDependency.


Unit tests configuration issues (rely on relative path and mix testng and junit)

Currently, the unit tests:

are using relative path to load test resources (could be fixed by using classpath)
mix testng and a bit of junit (could be fixed by using only testng)

=> on my env, using intellij, some tests fail due to relative path issue and some tests are ignored because we have to choose between testng and junit framework
PR coming.

PR: #447


Food recipe website

Adding a project called foodRecs which lets a user to post his/her food recipes and also review them

@Adbnemesis Add more details about your project .

It is a web based application for posting food recipes online, getting reviews and viewing other user's recipes. It doesnt have the authentication yet . It includes implementation of node.js and mongoDB along with elementary frontend.

@Adbnemesis Okey cool Go ahead !!