How to setup a 404 page?

I've been looking on how to set this up to render a page when a template is not found. Can anyone help me with this? Thank you all.

Rendering a dynamic 404 page in rails is not necessarily related to high_voltage, if that's what you want.  Check out the rescue_from method - http://api.rubyonrails.org/classes/ActiveSupport/Rescuable/ClassMethods.html#method-i-rescue_from - that might help.
If you ARE trying to add 404 customization to high_voltage, can you be more specific about what you want to do?

@mjankowski, what i was trying to do was use high_voltage to render a dynamic 404.
As you say, this can be achieved without high_voltage, this is what I have in application_controller.rb:
unless config.consider_all_requests_local
  rescue_from Exception, :with => :render_500
  rescue_from ActiveRecord::RecordNotFound, :with => :render_404
  rescue_from ActionController::RoutingError, :with => :render_404
  rescue_from ActionController::UnknownController, :with => :render_404
  rescue_from ActionController::UnknownAction, :with => :render_404
end

protected

def render_500 exception
  logger.error exception.inspect
  ExceptionMailer.exception(exception).deliver
  render :template => "/pages/500.html.erb", :status => 500, :layout => 'static'
end

def render_404 exception
  logger.error exception.inspect
  render :template => "/pages/404.html.erb", :status => 404, :layout => 'static'
end

Thank you!

It is depend on particular use case, but in case someone looking to simply handle invalid pages:
class PagesController < ApplicationController
  include HighVoltage::StaticPage

  def invalid_page
    render('pages/404-error', :status => 404)
  end
end

Please see "Override" scection for more details about PagesController:
https://github.com/thoughtbot/high_voltage#override
p.s.
invalid_page is the method from HighVoltage::StaticPage module, which, by default, raises ActionController::RoutingError https://github.com/thoughtbot/high_voltage/blob/54217603f6745a1feecff6761bc600dc07631ba4/app/controllers/concerns/high_voltage/static_page.rb#L22-L24


Understand prepared statements and responses

The code so far may understand a little of this traffic, but it's quite likely that returns a binary response format for efficiency and we should aim to grok that.

We can now deal with prepare and execute to some degree.  Need to see what happens when we get results back and probably cursors too.

This mostly works.  Things left to do:

[ ] nulls
[ ] all the types (and testing) - missing time related types

Also need to deal with unsigned integers.

As far as I can tell this all works for now.


Docker remote API v1.24 Documentation discrepancy on GET /services

Description

Steps to reproduce the issue:

http://<IP_ADDRESS>:2375/services?filters={"id":["<node-id>"]}
http://<IP_ADDRESS>:2375/services?filters={"id":["<service-id>"]}

Describe the results you received:

[ ]
Service filtered by id.

Describe the results you expected:

Array of services filtered by node id.
[ ]

Additional information you deem important (e.g. issue happens only occasionally):
API Docs state   as a filter parameter whereas service method filters by  instead.
How can I get the list of services running on a particular node or with instances running on a specific node?
Output of docker version:
Docker v 1.12.3

Output of docker info:
{
  "ID": "TZFE:3XXK:PYKN:NIYU:BEQF:YNPU:4LKC:WFFG:LJ52:4UGQ:WJXH:SWDS",
  "Containers": 4,
  "ContainersRunning": 4,
  "ContainersPaused": 0,
  "ContainersStopped": 0,
  "Images": 23,
  "Driver": "devicemapper",
  "DriverStatus": [
    [
      "Pool Name",
      "docker-253:0-752710-pool"
    ],
    [
      "Pool Blocksize",
      "65.54 kB"
    ],
    [
      "Base Device Size",
      "10.74 GB"
    ],
    [
      "Backing Filesystem",
      "xfs"
    ],
    [
      "Data file",
      "/dev/loop0"
    ],
    [
      "Metadata file",
      "/dev/loop1"
    ],
    [
      "Data Space Used",
      "7.089 GB"
    ],
    [
      "Data Space Total",
      "107.4 GB"
    ],
    [
      "Data Space Available",
      "100.3 GB"
    ],
    [
      "Metadata Space Used",
      "9.732 MB"
    ],
    [
      "Metadata Space Total",
      "2.147 GB"
    ],
    [
      "Metadata Space Available",
      "2.138 GB"
    ],
    [
      "Thin Pool Minimum Free Space",
      "10.74 GB"
    ],
    [
      "Udev Sync Supported",
      "true"
    ],
    [
      "Deferred Removal Enabled",
      "false"
    ],
    [
      "Deferred Deletion Enabled",
      "false"
    ],
    [
      "Deferred Deleted Device Count",
      "0"
    ],
    [
      "Data loop file",
      "/var/lib/docker/devicemapper/devicemapper/data"
    ],
    [
      "Metadata loop file",
      "/var/lib/docker/devicemapper/devicemapper/metadata"
    ],
    [
      "Library Version",
      "1.02.107-RHEL7 (2016-06-09)"
    ]
  ],
  "SystemStatus": null,
  "Plugins": {
    "Volume": [
      "local"
    ],
    "Network": [
      "overlay",
      "bridge",
      "null",
      "host"
    ],
    "Authorization": null
  },
  "MemoryLimit": true,
  "SwapLimit": true,
  "KernelMemory": true,
  "CpuCfsPeriod": true,
  "CpuCfsQuota": true,
  "CPUShares": true,
  "CPUSet": true,
  "IPv4Forwarding": true,
  "BridgeNfIptables": true,
  "BridgeNfIp6tables": false,
  "Debug": false,
  "NFd": 55,
  "OomKillDisable": true,
  "NGoroutines": 194,
  "SystemTime": "2016-11-08T16:46:49.153754781+01:00",
  "ExecutionDriver": "",
  "LoggingDriver": "json-file",
  "CgroupDriver": "cgroupfs",
  "NEventsListener": 3,
  "KernelVersion": "3.10.0-327.36.3.el7.x86_64",
  "OperatingSystem": "CentOS Linux 7 (Core)",
  "OSType": "linux",
  "Architecture": "x86_64",
  "IndexServerAddress": "https://index.docker.io/v1/",
  "RegistryConfig": {
    "InsecureRegistryCIDRs": [
      "<IP_ADDRESS>/8"
    ],
    "IndexConfigs": {
      "docker.io": {
        "Name": "docker.io",
        "Mirrors": null,
        "Secure": true,
        "Official": true
      }
    },
    "Mirrors": null
  },
  "NCPU": 4,
  "MemTotal":<PHONE_NUMBER>,
  "DockerRootDir": "/var/lib/docker",
  "HttpProxy": "",
  "HttpsProxy": "",
  "NoProxy": "",
  "Name": "master",
  "Labels": null,
  "ExperimentalBuild": false,
  "ServerVersion": "1.12.3",
  "ClusterStore": "",
  "ClusterAdvertise": "",
  "SecurityOptions": [
    "seccomp"
  ],
  "Runtimes": {
    "runc": {
      "path": "docker-runc"
    }
  },
  "DefaultRuntime": "runc",
  "Swarm": {
    "NodeID": "6hftixtprlhop5addjmo7qnxe",
    "NodeAddr": "<IP_ADDRESS>",
    "LocalNodeState": "active",
    "ControlAvailable": true,
    "Error": "",
    "RemoteManagers": [
      {
        "NodeID": "6hftixtprlhop5addjmo7qnxe",
        "Addr": "<IP_ADDRESS>:2377"
      }
    ],
    "Nodes": 2,
    "Managers": 1,
    "Cluster": {
      "ID": "5lp9rqb6l7my0ek9jg21cwkf9",
      "Version": {
        "Index": 819
      },
      "CreatedAt": "2016-11-03T12:50:34.101987364Z",
      "UpdatedAt": "2016-11-08T12:50:34.44261306Z",
      "Spec": {
        "Name": "default",
        "Orchestration": {
          "TaskHistoryRetentionLimit": 5
        },
        "Raft": {
          "SnapshotInterval": 10000,
          "LogEntriesForSlowFollowers": 500,
          "HeartbeatTick": 1,
          "ElectionTick": 3
        },
        "Dispatcher": {
          "HeartbeatPeriod":<PHONE_NUMBER>
        },
        "CAConfig": {
          "NodeCertExpiry":<PHONE_NUMBER>000000
        },
        "TaskDefaults": {}
      }
    }
  },
  "LiveRestoreEnabled": false
}

Additional environment details (AWS, VirtualBox, physical, etc.):
Physical

Oh, Yeah.
I can reproduce this issue. And it is indeed an issue in docs. I will fix this ASAP.
#dibs


Drush based Drupal 8 install breaks on Linux

What happened (or feature request):
Drupal themes will not load by default when running a command line Drush install on Linux
What you expected to happen:
A nice shiny baby Drupal to show up
How to reproduce this:

drush dl drupal
cd drupal-8.4.3
composer install
ddev config
ddev start
ddev exec drush si standard --account-name=admin --account-pass=admin --db-url=mysql://db:db@db/db
Load Drupal from a browser

The theme does not load and it throws an error about not being able to write a temp dir.
This is fixed by opening up the file permissions on sites/default
Version: Please include the output of ddev version, docker version and the project's .ddev/config.yaml.
0.9.3
Anything else do we need to know:
Reproduced on Ubuntu, likely caused by root password prompt, which sets sites/default ownership to root when it needs to be www-data
Related source links or issues:
https://github.com/drud/ddev/issues/131
Please use a complexity rating of 1-5 (5 is high) for a feature request. (High complexity implies more PR planning)
3, theres a fix by changing the permissions but doesnt work OOTB on install

Thanks so much for this. On our hosting platform container we were able to solve this more effectively, so we'll be able to transplant that approach over here, probably have the user execute commands as the nginx/www-data user.

Manually? It would be nice if ddev could automate this

No, the container needs to be set up so that when you ddev exec you're not using the root user, but instead an appropriate user that www-data will be then able to manipulate files (like sites/default and sites/default/files)

This was resolved in v0.16.0 - Containers now run as the appropriate user on Linux.


incorrect routing of self-messages in slack direct chats

The following happens when connecting to slack with my irc client (weechat) via the matterircd gateway, which is owned by someone else.
In a direct chat with another person, self-messages (messages I send from the slack app or browser clients, not irc client) are routed to a direct chat with myself. This breaks the continuity of the conversation view in the irc client, as the messages I send from slack clients don't appear in the direct chat with the other person.
Also, the name of the direct chat with myself does not correspond with my slack handle, but rather another string that I am guessing comes from the part of my email address before '@'. Apparently, other users of the direct gateway always see my handle as this other string (not my slack handle). I have tried setting Full Name and Display Name as my desired slack handle in slack preferences, but without any success.

There is #203 to fix problem where messages go to yourself.
To fix you handle, you should set your username.
Go to Profile & account -> Account settings (click gear) -> Username is at bottom. Click expand, change your username and save.


Could we access position, rotation and scale from Node?

Hello,
I want to write a Shader helper where the user could give the name of a node (e.g. from a Light, Camera, Mesh, or TransformNode), and I would add a Vector3 for the position of that node in the shader.
However, when I use the function scene.findNodeByName(), then I couldn't access the position of the Node; I realised that position, rotation and scale are not accessible from Node.
It looks like that Camera, Light, Mesh and TransformNode all have a position, rotation and sometime a scale. Is there a specific reason for not making them accessible from the Node abstraction level?

I think because all nodes don’t have these properties. Typically a light doesn’t have a scale, same for camera etc.

I did say "[They have] sometimes a scale".
But already the ability to access the properties rotation and position from any node would be nice.
Although maybe it's already possible to access them? For instance, I was wondering, how do Animations access the properties of their targets?

My bad I missed the « sometimes » ^^
Are you sure that hemispheric light has a rotation ?

This is a question for the forum. The repo must only be used for bug and features.
If you are in TS, just cast the node: (node). position
In js, just call node.position :)

Thanks for your response :)
Sorry, I thought of it as a feature request as I was requesting the ability to access position and rotation of a Node without knowing what the Node is.
If I need to cast the Node to Camera, that means I need to know that the node is a camera. I just thought that it could be nice to be able to access the position of objects, without knowing too much of what they are.
And I thought Node might be the good level of abstraction for it.

... yeah; that's a shame. I got that idea because in Blender or Unity, cameras, lights, emptys and meshes all hold a position and a rotation (and even a scale, I think)...
For instance, a directional light does not need a position for its light to be rendered. However, as Blender is a visual editor, directional light still holds a position, because it is important for the user to be able to put the light somewhere that is easy to reach and not in the way.
Now, in the case of BabylonJS, that does not make much sense. So I guess I will just have to test if the object I am given holds a position :D
Thank you for your feedback anyway @julien-moreau  and @deltakosh


CT200: Remove dhw1 openWindowDetectionStatus

This PR removes openWindowDetectionStatus from all easycontrol water heater (dhw1) circuits.
This reduces unecessary logging in Home Assistant, as the id is not found on the gateway.
Fixes https://github.com/bosch-thermostat/home-assistant-bosch-custom-component/issues/323.
Above issue also includes a debug scan showing the status only available in /zones/zn1/openWindowDetection/status.

Are you sure that nobody has this sensor attached?
We'd need to collect some logs. If somebody has it then we would need to avoid unnecessary fetch in different way

Just updated the code because it doesn't make sense to remove and it should be in the zone circuit.
Also water heaters usually are not impacted by open windows - heating circuits or zones, yes. But a water heater that just boils for showering or warm water?

Thanks, I'll check it next week and let you know.
I don't know why I put it under water heater. Probably a typo


(WIP) Specify custom bind port through org.apache.brooklyn.osgilauncher.cfg

Much more complicated task than I expected.
The parameter should be passed to the server started by karaf (started by this module https://github.com/apache/brooklyn-server/tree/master/karaf/jetty-config)
I'd be happy if someone takes this. It is very useful feature when testing and developing.

@bostko As you noticed the port is managed by Karaf, not by Brooklyn. You can configure another port in org.ops4j.pax.web.cfg.

Thank you.
I added a note about that here https://github.com/apache/brooklyn-docs/pull/125


Guide "Using uv with marimo"

📝 Summary
This is a draft pull request intended for testing the online documentation build and is
not yet ready for review.
🔍 Description of Changes
Adding a guide "Using uv with marimo".
📋 Checklist

[x] I have read the contributor guidelines.
[x] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on Discord, or the community discussions (Please provide a link if applicable).

📜 Reviewers

I have read the CLA Document and I hereby sign the CLA


Create Backend

We need to add a node/express/typescript backend inside the packages directory.
Just make sure some basic API boilerplate is in place. Probably could copy the [eth-tech-tree-backend] and put it here.
The purpose of the backend will be to run recurring jobs that refresh the data from Open Source Observer and store it based on timeframes. This way when a user queries a certain time frame we won't have to recalculate everything from scratch.
Since we will have a backend we will make use of aggressive caching so that every request is as fast as possible.

Nice, I can get started on this today

@swellander Thanks! Use the version of eth-tech-tree-backend in this PR: https://github.com/BuidlGuidl/eth-tech-tree-backend/pull/13
It has moved the project to be more of a monorepo structure and you should be able to mostly just copy the packages/server and you can delete any ETT specific files.

@swellander Thanks! Use the version of eth-tech-tree-backend in this PR: BuidlGuidl/eth-tech-tree-backend#13
It has moved the project to be more of a monorepo structure and you should be able to mostly just copy the packages/server and you can delete any ETT specific files.

Great, will do. Thanks @escottalexander !

Per our conversation on Telegram we don't have a need of this any more. Thanks for your input @swellander!


added windows 11

after reading this article, i confirmed windows 11 is supported

Learn Build status updates of commit 39756cd:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

windows/security/threat-protection/security-policy-settings/generate-security-audits.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
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.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


:lang/python - Add lsp-python-ms

Many people prefer Microsoft's Python language server over pyls. This allows them to use it.

Hi! Thanks for the PR. I agree that lsp-python-ms is the better choice.
I have two requests:

Could you target the straight branch instead, and
Could you gate it behind +lsp instead of +lsp-ms? I'd rather it be installed by default if lsp is enabled.

@hlissner There are probably some people out there still using pyls, and lsp-python-ms sets itself as the default. Would you like me to do that anyway?
Also, targeting the straight branch has some adverse effects.

Also, targeting the straight branch has some adverse effects.

It appears you've merged onto #1588. Would you mind rebasing onto straight? Without the shceme module there shouldn't be any conflict.

There are probably some people out there still using pyls, and lsp-python-ms sets itself as the default

Having inspected lsp-python-ms's source, I find it disturbing that it would forcibly install itself if it isn't present, rather than gracefully yielding to the next available lsp client.
I don't like that behavior at all considering a) M-x lsp-python-ms-setup exists and b) LSP has a client fallback mechanism (that would allow it to use pyls in lsp-python-ms's absence). I think the better solution would be:
(defun +python--dont-auto-install-server-a (orig-fn)
  lsp-python-ms-executable)
(advice-add #'lsp-python-ms--command-string
            :override #'+python--dont-auto-install-server-a)

So lsp-mode do it's job and fall back to pyls.

Aaand I managed to do a weird thing with the branches. Give me a moment and I'll reopen this.


Translations update from Foundry Hub - Weblate

Translations update from Foundry Hub - Weblate for Simple Calendar/main.
Current translation status:

Codecov Report

Merging #366 (8aca43b) into main (4d2f7ad) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main      #366   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           73        73
  Lines         6698      6698
  Branches      1910      1910
=========================================
  Hits          6698      6698

Flag
Coverage Δ

unittests
100.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Users should not be allowed to change the location to a location they do not manage

When creating a party, users can only create events for the locations they own.
But in a second step, they can actually change the location to any desired location.
It would be nice if both in edit and in create mode, the user only can select from the locations he is allowed to manage

Fixed permission check in backend. Its not possible to change the location to or from an location the user does not have a permission to.


TW server

hey,
How can i works ravahn/BnS_ACT_Plugin on tw server

Download main application and install a plugin there as described in the readme.
http://advancedcombattracker.com/


This is a test reported by @sketchydroide

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:
Break down in numbered steps
Expected Result:
Describe what you think should've happened
Actual Result:
Describe what actually happened
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platform:

Where is this issue occurring?

Web
iOS
Android
Desktop App
Mobile Web

Version Number:
Reproducible in staging?:
Reproducible in production?:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:
Issue reported by:
Slack conversation:
View all open jobs on GitHub

no need to pay attention @laurenreidexpensify


enabling the PING service kills my current Taefik setup

Do you want to request a feature or report a bug?

Bug

What did you do?
Added the default ping config to my current Traefik setup

What did you expect to see?
The ping service working
What did you see instead?
My server emits errors and dies
Output of traefik version: (What version of Traefik are you using?)

Traefik 2.1.6

What is your environment & configuration (arguments, toml, provider, platform, ...)?
my docker-compose file
version: '3'

services:

  traefik:
    image: "traefik:v2.1.6"
    container_name: "traefik"
    command:
      - "--configFile=/etc/traefik/static.yaml"
    network_mode: "host"
    volumes:
      - "/var/run/docker.sock:/var/run/docker.sock:ro"
      - "${PWD}/static.yaml:/etc/traefik/static.yaml:ro"
      - "${PWD}/conf/:/etc/traefik/conf/:ro"
      - "${PWD}/cert/:/etc/traefik/cert/:ro"
    restart: always
    logging:
      driver: "json-file"
      options:
        max-file: "2"
        max-size: "2m"

my static.yaml
log:
  level: DEBUG

accessLog: {}

# this config works fine until I add the below line
ping: {}

api:
  dashboard: true

entryPoints:
  http:
    address: ":80"
  https:
    address: ":443"
  k8s:
    address: ":6443"
  dash:
    address: ":8080"
  metrics:
    address: ":8082"
  nipap:
    address: ":7000"

metrics:
  prometheus:
    addEntryPointsLabels: true
    addServicesLabels: true
    entryPoint: "metrics"
    buckets:
      - 0.1
      - 0.3
      - 0.5
      - 1.0
      - 5.0

providers:
  file:
    directory: "/etc/traefik/conf"
    watch: true

I have a several dynamic files also but I'm not sure they matter here as the server seem to die before reaching them.

If applicable, please paste the log output in DEBUG level (--log.level=DEBUG switch)
time="2020-03-12T23:32:55Z" level=debug msg="Static configuration loaded {\"global\":{\"checkNewVersion\":true},\"serversTransport\":{\"maxIdleConnsPerHost\":200},\"entryPoints\":{\"dash\":{\"address\":\":8080\",\"transport\":{\"lifeCycle\":{\"graceTimeOut\":10000000000},\"respondingTimeouts\":{\"idleTimeout\":180000000000}},\"forwardedHeaders\":{}},\"http\":{\"address\":\":80\",\"transport\":{\"lifeCycle\":{\"graceTimeOut\":10000000000},\"respondingTimeouts\":{\"idleTimeout\":180000000000}},\"forwardedHeaders\":{}},\"https\":{\"address\":\":443\",\"transport\":{\"lifeCycle\":{\"graceTimeOut\":10000000000},\"respondingTimeouts\":{\"idleTimeout\":180000000000}},\"forwardedHeaders\":{}},\"k8s\":{\"address\":\":6443\",\"transport\":{\"lifeCycle\":{\"graceTimeOut\":10000000000},\"respondingTimeouts\":{\"idleTimeout\":180000000000}},\"forwardedHeaders\":{}},\"metrics\":{\"address\":\":8082\",\"transport\":{\"lifeCycle\":{\"graceTimeOut\":10000000000},\"respondingTimeouts\":{\"idleTimeout\":180000000000}},\"forwardedHeaders\":{}},\"nipap\":{\"address\":\":7000\",\"transport\":{\"lifeCycle\":{\"graceTimeOut\":10000000000},\"respondingTimeouts\":{\"idleTimeout\":180000000000}},\"forwardedHeaders\":{}},\"traefik\":{\"address\":\":8080\",\"transport\":{\"lifeCycle\":{\"graceTimeOut\":10000000000},\"respondingTimeouts\":{\"idleTimeout\":180000000000}},\"forwardedHeaders\":{}}},\"providers\":{\"providersThrottleDuration\":2000000000,\"file\":{\"directory\":\"/etc/traefik/conf\",\"watch\":true}},\"api\":{\"dashboard\":true},\"metrics\":{\"prometheus\":{\"buckets\":[0.1,0.3,0.5,1,5],\"addEntryPointsLabels\":true,\"addServicesLabels\":true,\"entryPoint\":\"metrics\"}},\"ping\":{\"entryPoint\":\"traefik\"},\"log\":{\"level\":\"DEBUG\",\"format\":\"common\"},\"accessLog\":{\"format\":\"common\",\"filters\":{},\"fields\":{\"defaultMode\":\"keep\",\"headers\":{\"defaultMode\":\"drop\"}}}}"
time="2020-03-12T23:32:55Z" level=info msg="\nStats collection is disabled.\nHelp us improve Traefik by turning this feature on :)\nMore details on: https://docs.traefik.io/contributing/data-collection/\n"
2020/03/12 23:32:55 traefik.go:71: command traefik error: error while building entryPoint traefik: error preparing server: error opening listener: listen tcp :8080: bind: address already in use

Hello,
It's because you already have an entrypoint to 8080.
You can either set the entrypoint in the ping configuration https://docs.traefik.io/v2.1/operations/ping/#entrypoint
or
enable the manual routing https://docs.traefik.io/v2.1/operations/ping/#manualrouting

Hi, so the default port for ping is 8080? Where can I find this in the docs? Is there any way to see all the default config of Traefik?

https://community.containo.us/t/unable-to-get-the-ping-service-working-on-custom-router-entrypoint/4914/10


Subdomain support

Let's say, the domain in Plesk is domain.com, will there be an option to have the certificate include subdomains like www.domain.com?
Are you aware of a workaround to create a certificate for www.domain.com through this plugin?

I guess you are talking about SAN Support - I should have read this earlier before opening #19
Anyway, I guess this is the same need!

See #4.

Supporting multiple Subdomains in one Certificate would be a solution for the Error "Too many certificates already issued for...". Subdomains seems to get counted to the top-level Domain.
https://community.letsencrypt.org/t/public-beta-rate-limits/4772/2

Close as duplicate


throw err

/home/gzp/code/reactnative/react-native-nw-react-calculator/node_modules/.6.24.1@babel-core/lib/transformation/file/index.js:590
throw err;
^
SyntaxError<EMAIL_ADDRESS>Unexpected token, expected ; (15:5)
13 | const path = require('path');
14 |
15 | type ProjectInfo = {
\\\\\\\     |      ^
16 |   name: string;
17 |   isWorkspace: boolean;
18 | }
at Parser.pp$5.raise (/home/gzp/code/reactnative/react-native-nw-react-calculator/node_modules/.6.16.1@babylon/lib/index.js:4373:13)
at Parser.pp.unexpected (/home/gzp/code/reactnative/react-native-nw-react-calculator/node_modules/.6.16.1@babylon/lib/index.js:1716:8)
at Parser.pp.semicolon (/home/gzp/code/reactnative/react-native-nw-react-calculator/node_modules/.6.16.1@babylon/lib/index.js:1697:38)
at Parser.pp$1.parseExpressionStatement (/home/gzp/code/reactnative/react-native-nw-react-calculator/node_modules/.6.16.1@babylon/lib/index.js:2191:8)
at Parser.pp$1.parseStatement (/home/gzp/code/reactnative/react-native-nw-react-calculator/node_modules/.6.16.1@babylon/lib/index.js:1866:17)
at Parser.pp$1.parseBlockBody (/home/gzp/code/reactnative/react-native-nw-react-calculator/node_modules/.6.16.1@babylon/lib/index.js:2223:21)
at Parser.pp$1.parseTopLevel (/home/gzp/code/reactnative/react-native-nw-react-calculator/node_modules/.6.16.1@babylon/lib/index.js:1733:8)
at Parser.parse (/home/gzp/code/reactnative/react-native-nw-react-calculator/node_modules/.6.16.1@babylon/lib/index.js:1628:17)
at parse (/home/gzp/code/reactnative/react-native-nw-react-calculator/node_modules/.6.16.1@babylon/lib/index.js:7083:37)
at File.parse (/home/gzp/code/reactnative/react-native-nw-react-calculator/node_modules/.6.24.1@babel-core/lib/transformation/file/index.js:517:15)

I have the same error, did you find a solution?

I had the same error...
I have tried:
yarn upgrade

after that
yarn start

and
yarn react-native:ios

works, but I can't build it with Xcode

@ApolloTang you have another error. First problem in transpile code


Collect Codes of Conduct?

Perhaps we should have a page about CoCs?

I think this would make a great add: Putting a few that I came across in the reources

first on the list: https://github.com/EthicalSource/contributor_covenant

@RichardLitt added here https://sustainoss.org/dei-handbook/landscape.html#code-of-conduct-guides
Please let me know if this was what you were thinking. If so, we can close the issue :)

This is great, thanks!


Current state of entity locking overhaul?

@Tsuser1, @pop4959
I was reading the wiki and noticed entity locking is currently not supported and being redeveloped.
https://github.com/Tsuser1/Modern-LWC/wiki/Entity-Locking
Is entity locking currently functional at all? If not, perhaps you could look into merging some changes made to dmulloy's lwc fork which adds entity locking support? https://github.com/dmulloy2/LWC
It supports adding an entity section below blocks in the main config.
i.e.
    blocks:
        chest:
            enabled: true
    entities:
        armor_stand:
            enabled: true
        painting:
            enabled: true
        item_frame:
            enabled: true
        minecart_chest:
            enabled: true
        SKULL_ITEM:
            enabled: true

Except they have a few fix commits for it and separate it from blocks. There isn't anything to pull into Modern LWC?

No, because the way entities are handled in both versions is fundamentally flawed. The current version of Modern LWC branch is almost identical in functionality (except for those few fixes). The goal is to completely overhaul it so it works overall better than the current system that was created. Currently it's quite hacky in how it works, and there are several unfixable bugs/issues without redesigning it.

@mibby Sorry, I was in a bit of a rush yesterday because I had to get some sleep. So to fully answer your question, the current state of entity locking is that it can be used as-is, and we have no current plans to break compatibility with your existing entity locks. If/when we have time to overhaul entity locking, we want to automatically convert old protections to new ones. For now, there are no plans to make significant changes to how entity locking currently functions, since it would probably need to be rewritten later anyway as part of the overhaul (and thus it is wasted effort). We are only providing limited support for the current implementation.
Keep in mind this is not to say that we are not open to pull requests which do affect entity locking. You can feel free to suggest something, and if it is appropriate, I may merge it. Also feel free to reach out to me on Discord if you want to chat about this in more detail. I do not know much about this other fork.

To answer your other question: Yes, entity locking is currently functional in Modern LWC.


Add a WorkManifest-based MachineSet deployer

This is required to manage deployments through the hub.
Signed-off-by: Stephen Kitt<EMAIL_ADDRESS>

/cc @aswinsuryan you might need this for GCP too!


Enable User assigned managed identity for scalesets

Summary of the Pull Request
Create a User Assigned Managed Identity at deplyoment
Assign the idenity to scalesets deployed by the service
PR Checklist

[x] Applies to work item: #209
[x] CLA signed. If not, go over here and sign the CLI.
[ ] Tests added/passed
[x] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Info on Pull Request
What does this include?
Validation Steps Performed

Deployed
Enabled "User assignment required" on the entreprise application of the instance
Verified that job can run to completion

Running integration tests

Integration tests pass


Multiple style editing workflow

I currently maintain four styles, a number that will rise considerably in the foreseeable future. Some improvements that I make to one of the styles will also improve the other styles (or some of them).
Currently I use the following workflow:

I work on a style for a session of some hours (up to one day)
Then I fire up a file merge tool (I use DiffMerge) loading the style I worked on as the source, and one of the other three styles as the target. It is important that I do this as long as I still remember what the changes I made were meant to achieve.
I merge all changes that make sense in the target style into it.
Repeat with style number three and four.
I commit all styles to GitHub

This workflow works surprisingly well but I know I will run into problems once I have to maintain five, ten or 50 styles.
In an ideal world Maputnik (or a Maputnik version for advanced users) would help me with this workflow. I am not sure yet how but I would like to open this up for discussion. What do you guys think? How do you work with multiple styles, and what role could Maputnik play here?

We should use your merge strategy :+1:
But the problem for us stemmed more because we were iterating on the vector tile schema and field values and layers were changing.
I guess a common pattern is that you have a base style (with some layers) and then a lot of other styles are just layers on top of that or changed colors.
Perhaps one idea would be to specify a base style URL in the metadata (which would be displayed as immutable layers) and only the layers on top of it can be edited.
But that solves kind of a limited use case.
My goal is that we always operate on a valid JSON style as described by the Mapbox GL style spec. If we have additional needs it must be described via metadata.
That's also why I am not sure variables are a good idea.

There are various approaches that came to my mind, that could possibly be implemented in Maputnik (just some quick ideas):
A. Includes
The CSS way. Layers could be bundled into sub-stylesheets that could be included into styles. This would make sure that  , yet as (unlike with CSS) that's not part of the GL spec this would create a Maptnik-proprietary format that would have to be converted to standard GL files in the end.
Another downside would be that a setup of includes that really fits a specific use case well would require quite some brainwork, as we know from the CSS world. Not very novice user friendly.
B. Copy/Merge on file basis
Basically a replication of the workflow I described above: The possibility to display the difference between to styles in some way (map? comparison slider?), with the option to transfer selected differences from one style to another.
C. Copy/Merge on layer basis
A layer could have a function "Copy layer to another style". The user would select a target style and Maputnik would check if that layer existed (by layer name? or maybe by comparing filters). If not, it would be added (on the same position if possible). If yes, a JSON diff view could show differences and let the user accept or cancel.
If Maputnik knew all my styles (e.g. all the styles in my Github repo) than the function could be "Copy layer to [layer list dropdown]".

I think the discussion is really interesting - hence I'll keep it open.
However I don't think that I will provide a solution for this any time soon - since this is one of the more advanced features - while it is probably better to work on the lower hanging fruits that can still be improved.

I’ll start working on this issue once 13 people react positively to this comment. :)

Being able to have a "base" style and then add new layers or change properties of "base" layer (without changing the base style itself) would be very useful!
For example if we have a number of base layers (general, topo, light, dark, ortho etc.), it should be possible to create new derivative styles (thematic maps) and update those thematic styles easily when base style changes.
(do not know how much this is in line with Mapbox plans: https://github.com/mapbox/mapbox-gl-js/issues/4225)

I guess the complexity comes with the ordering of layers. For example with a base map in which you want to add a traffic layer this would sit in between the two layers in the original style.
road                   <--- base style
transit overlay
road label          <--- base style


ModuleNotFoundError: No module named 'undetected_chromedriver.v2'

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to 'quickstart.py'
Click on 'run'
Scroll down to 'error'
See error

Expected behavior
ScreenshotsTraceback (most recent call last):
File "c:\Users\lifea\Downloads\TinderBotz-master\TinderBotz-master\auto_swipe.py", line 8, in
from tinderbotz.session import Session
File "c:\Users\lifea\Downloads\TinderBotz-master\TinderBotz-master\tinderbotz_init_.py", line 1, in
from tinderbotz.session import Session
File "c:\Users\lifea\Downloads\TinderBotz-master\TinderBotz-master\tinderbotz\session.py", line 4, in
import undetected_chromedriver.v2 as uc
ModuleNotFoundError: No module named 'undetected_chromedriver.v2'
Any help how do i fix this pls I tried re installing programs as well

https://github.com/frederikme/TinderBotz/issues/142
try this


Global Exception handler seems to break on certain configs

v3.14.1 seems to include something that breaks existing flows.
In this particular CI pipeline, i have an earlier checkout with
      - uses: actions/checkout@v2
        with:
          fetch-depth: 0

so when it comes to this action, it will bail because the git config already exists.
Odd thing is, it looks like it actually does go through all the steps of changelog generation, but then finally decides that something went wrong in the init steps, and then returns an error as exit code.
Using "chore(release): {version} [skip ci]" as commit message
Using "Conventional Changelog Action" as git user.name
Using<EMAIL_ADDRESS>as git user.email
Using "1" release count
Using "./package.json" as version file
Using "version" as version path
Using "v" as tag prefix
Using "CHANGELOG.md" as output file
Using ".github/tools/changelog-config.js" as config file
Using "github.com" as gitUrl
Using "refs/pull/638/merge" as gitBranch
Skipping empty releases is "disabled"
Skipping the update of the version file is "enabled"
Loading "/home/runner/work/XXXXXX/.github/tools/changelog-config.js" script
/usr/bin/git config user.name Conventional Changelog Action
/usr/bin/git config user.email<EMAIL_ADDRESS>/usr/bin/git remote set-url origin ***github.com/XXXXX.git
error: could not lock config file .git/config: File exists
Unhandled Rejection occurred. Error: The process '/usr/bin/git' failed with exit code 255
    at ExecState._setResult (/home/runner/work/_actions/TriPSs/conventional-changelog-action/v3/node_modules/@actions/exec/lib/toolrunner.js:592:25)
    at ExecState.CheckComplete (/home/runner/work/_actions/TriPSs/conventional-changelog-action/v3/node_modules/@actions/exec/lib/toolrunner.js:575:18)
    at ChildProcess.<anonymous> (/home/runner/work/_actions/TriPSs/conventional-changelog-action/v3/node_modules/@actions/exec/lib/toolrunner.js:469:27)
    at ChildProcess.emit (events.js:[31](https://github.com/Solteq/xxxxx/actions/runs/3059089495/jobs/4936086371#step:6:32)4:20)
    at maybeClose (internal/child_process.js:1022:16)
    at Socket.<anonymous> (internal/child_process.js:[44](https://github.com/xxxxxactions/runs/3059089495/jobs/4936086371#step:6:45)4:11)
    at Socket.emit (events.js:314:20)
    at Pipe.<anonymous> (net.js:675:12)
Error: Unhandled Rejection occurred. Error: The process '/usr/bin/git' failed with exit code 255
    at ExecState._setResult (/home/runner/work/_actions/TriPSs/conventional-changelog-action/v3/node_modules/@actions/exec/lib/toolrunner.js:592:25)
    at ExecState.CheckComplete (/home/runner/work/_actions/TriPSs/conventional-changelog-action/v3/node_modules/@actions/exec/lib/toolrunner.js:575:18)
    at ChildProcess.<anonymous> (/home/runner/work/_actions/TriPSs/conventional-changelog-action/v3/node_modules/@actions/exec/lib/toolrunner.js:[46](https://github.com/xxxxx/actions/runs/3059089495/jobs/4936086371#step:6:47)9:27)
    at ChildProcess.emit (events.js:314:20)
    at maybeClose (internal/child_process.js:1022:16)
    at Socket.<anonymous> (internal/child_process.js:444:11)
    at Socket.emit (events.js:314:20)
    at Pipe.<anonymous> (net.js:675:12)
Recommended release type: patch
Because: static
Using GIT to determine the new version

I think this was always the case but in 3.14.1 we catch unhandled exceptions and markt he job failed.

dont-pull and dont-push are not valid optons? Could you try with skip-git-pull, this should prevent the initial pulling.

config used:

      - name: Version
        id: cleanchangelog
        uses: TriPSs/conventional-changelog-action@v3
        with:
          config-file-path: .github/tools/changelog-config.js
          github-token: ${{ secrets.ADMIN_TOKEN_GITHUB }}
          release-count: '1'
          skip-on-empty: 'false'
          skip-git-pull: 'true'
          skip-commit: 'true'
          skip-version-file: 'true'
          git-push: 'false'

Usecase here is a PR build, where i want to include the changelog in the build, but not actually push it back to the branch

Could it be something you do in .github/tools/changelog-config.js? As the error happens directly after that?

Unclear...
'use strict';
const config = require('conventional-changelog-angular');

config.then((x) => {
  x.recommendedBumpOpts.whatBump = (commits) => {
    // custom - we consider everything to be a patch
    return {
      level: 2,
      reason: 'static',
    };
  };
});

module.exports = config;

Hmm, then I think gitSemverTags is maybe causing this issue. Not quite sure how we can tackle this.
Is it possible that you share your whole workflow?

uhm, well, i can share the bit up to this point at least
on: pull_request
name: Pull Request Workflow

concurrency:
  group: pr-${{ github.event.number }}
  cancel-in-progress: true

jobs:
  install:
    runs-on: ubuntu-latest
    env:
      NPM_TOKEN_XX: ${{ secrets.XX }}
    steps:
      - uses: actions/checkout@v2
      - uses: ./.github/actions/node-prepare
        id: node-cache

      - name: install if needed
        if: ${{ steps.node-cache.outputs.cache-hit != 'true' }}
        run: |
          npm ci --prefer-offline --ignore-scripts true --audit false
          npm run postinstall

  build:
    needs: install
    runs-on: ubuntu-latest
    env:
      NPM_TOKEN_XX: ${{ secrets.NPM_TOKEN_XX }}
    outputs:
      versioninfo: ${{ steps.changelog.outputs.tag }}-${{ github.sha }}
      deployinfo: ${{ github.event.number }}
      changelog: ${{ steps.cleanchangelog.outputs.clean_changelog}}
    steps:
      - uses: actions/checkout@v2
        with:
          fetch-depth: 0

      - uses: ./.github/actions/node-prepare

      - name: Cache Build Artifacts for later jobs
        id: artifactscache
        uses: actions/cache@v2
        with:
          path: |
            ./dist/
          key: artifacts-${{ github.sha }}

      # pin version to whatever the PR says it is
      - name: Version
        id: changelog
        run: |
          currentversion=$(node -e "console.log(require('./package.json').version)")
          echo "::set-output name=tag::v${currentversion}"

      - name: Version
        id: cleanchangelog
        uses<EMAIL_ADDRESS>        with:
          config-file-path: .github/tools/changelog-config.js
          github-token: ${{ secrets.ADMIN_TOKEN_GITHUB }}
          release-count: '1'
          skip-on-empty: 'false'
          skip-git-pull: 'true'
          skip-commit: 'true'
          skip-version-file: 'true'
          git-push: 'false'

./.github/actions/node-prepare is just a composite action that runs
runs:
  using: "composite"
  steps:
      - uses: actions/setup-node@v3
        with:
          node-version: ${{ inputs.version }}

      - uses: actions/cache@v3
        id: node-cache
        with:
          path: '**/node_modules'
          key: ${{ runner.os }}-modules-${{ hashFiles('**/package-lock.json') }}

But yeah, i agree that this has most likely been happening all the time. It just didnt matter, because
a) i got the changelog i wanted
b) i didnt really want it to fiddle with the repo in any way
c) it did not fail the build action
I checked the code, assuming that --skip-git-pull /push would essentially just disable the whole git manipulation, but i see now, that it changes the logic from being based on the version file, to being based on the git tags.
So, maybe there isnt a good fix for this. Maybe this usecase is just crazy.

You could try one thing, but I don't think it will change anything: add:
        env:
          ENV: 'dont-use-git'

This will make sure that the git helper will never do anything.
If this does not work I would recommending pinning your version to v3.14.0 where the exception is still ignored. We could ofc also add a option that will let the action continue even though it fails.

Will try that. I can always just tell github actions to continue in case of errors, although that is a bit harsh.
I mostly brought it up, in case there was some unexpected behavior here, when using the actions on a repo that was already fully-fetched.


ENH: add figure title to morphed MxNE in example

LGTM. I killed all CIs except the relevant ones.
If you feel like making one more change, it's probably better to separate the original-subject and morphed-subject plots into to different sphinx-gallery sections so that the plots are grouped more clearly.

@adykstra long time no see :) Glad to have you back !
are you done here? if so +1 for merge

@Eric89GXL , where/how is that specified?

You'd just need to create a new section like
###############################################################################
 # View on a destination subject using morphing

... and put the plot commands after that heading

Need the
###############################################################################
#comment
be immediately prior to the plotting function? Or can it also be placed before the actual morphing?

It can be before the morphing. The #### separate sections (where plots are grouped).

Much better :)
https://4783-1301584-gh.circle-artifacts.com/0/tmp/circle-artifacts.cipiYTp/html/auto_examples/inverse/plot_mixed_norm_inverse.html

Ready to go from your end @adykstra ?

Codecov Report

Merging #4316 into master will decrease coverage by 34.21%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master    #4316       +/-   ##
===========================================
- Coverage   85.89%   51.67%   -34.22%
===========================================
  Files         346      346
  Lines       62897    62897
  Branches     9672     9672
===========================================
- Hits        54026    32503    -21523
- Misses       6212    28404    +22192
+ Partials     2659     1990      -669

Yep.

Thanks @adykstra


WIP: Data gatherers can fetch data from local file

Possible solution for #78, currently only implemented for the AKS data gatherer.

@j-fuentes @charlieegan3 What do you think of this approach?
By trying to run Preflight like this I was able to find and fix #81, so I think it's worth doing something to make it easier to just run locally with fake data. Not sure if this is the best way though.

I'd rather bypass the whole data-gathering process:

add a command-line option like --bypass-data-gathering-with-file that points to a json file containing the raw input for the Rego evaluation. That raw input is:

{
  "data-gatherer ID 1": {... data-gatherer 1 data ...},
  "data-gatherer ID 2": {... data-gatherer 2 data ...},
  ...
}

if that option is present, skip the execution of the data gatherers, and check that the enabled data-gatherers that are required by packages are present in the file.

then change the part that builds the input for the Rego evaluation so it uses the data from the JSON.

Additionally, we could make schema validation on the data from the JSON, but that would require more work on the DataGatherer interface to provide methods for data validation.

I think if there's a way of doing this without making changes to individual gatherers that'd be preferred.

Thanks for the feedback!
I'd agree having to do this for every data gatherer is not ideal so I'm going to look for a more general approach.
I went for loading the data from the data gatherers as it maximises how 'end-to-end' the test is and keeps as much in common with a 'normal' run as possible. Data gatherers also use the relevant struct for data so this provides some validation of what's being loaded from file too.
I'd prefer to be able to bypass individual data gatherers. I'd also prefer to have each data gatherer's data in separate files. I think this will be easier to work with and is more like the format we would get it in anyway, for example from intermediate type output.

@j-fuentes @charlieegan3 I've updated the implementation.
This now adds a local data gatherer that to read a local file. This currently has no validation on the data it reads, it is just assumed to be JSON and passed direct to OPA for evaluation.
If any of the data gatherer's config specify a data-path then while creating data gatherers a local data gatherer will be created which 'impersonates' that gatherer.

Closing in favor of #80 (since I don't have permissions from Wil for his branch)

/close


Gaurav : Can't Hurt Me

Book Title?
Can't Hurt Me
Author?
David Goggins
Genre of Book?
Non-Fiction

I would like to work for this project.

I would like to work for this project.

what will be your appoach to make this work?

I might need to follow how to contribute such as fork it ,install npm package ,npx add .
I think I can do this even there might be some challenges.

I might need to follow how to contribute such as fork it ,install npm package ,npx add .
I think I can do this even there might be some challenges.
 I've assigned this to you... you can proceed with this one 🚀

#26


C64 screen driver

All functions implemented except screen_setstyle. Kernal calls are used when available and a lot of the code is reused from the PET bios.
Screen_getchar ignores the delay argument and just performs a non-blocking read.
qe works, but it is a bit hard to use without being able to type lowercase characters ;)

Oops, I completely missed this --- sorry! Merged. Thanks!


Translations update from Mattermost Weblate

Translations update from Mattermost Weblate for mattermost-languages-shipped/mattermost-webapp.
Current translation status:

/release-note-none

@amyblais do we need to cherry pick this? if yes just to cloud branch?


[improper hover] - the hover effect on contributors page is improper and should be changed

Is your feature request related to a problem? Please describe.
Upon hovering the image of Contributors, the image becomes more dull which doesn't make sense. Upon hover the image should be more highlighted instead of dull. I can add a better hover effect for the contributors page.
Normal :

Upon Hover :

Describe the solution you'd like.
Solution is to create a better hover effect, I can do that
Describe alternatives you've considered.
nope
Add any other context or screenshots about the feature request here.
added above
Record

[X] I agree to follow this project's Code of Conduct
[X] I have checked the existing issues
[X] I'm a GSSoC'23 contributor
[X] I want to work on this issue

Please assign me this issue under GSSoC'23

Too easy issue


Fixed an issue with names in functions

Two functions within SNAP has a different name, causing SNAP to not to compile properly on YYC.
Mainly SnapBufferWriteBinary and SnapBufferWriteJSON.

Additional mistakes were noted in SnapBufferWriteBinary, and have been corrected. (There was more references to SnapToBinary)

Oops! Thank you Tabular


more updates for TS

Notice: This PR and the previous one are work towards making our JS files work with Typescript types. Yes, we could have simply marked them as "don't analyze" but that does NOT move us towards TS adoption, really. The upshot of this is that we need to do as full of a regression test as possible. I'm working on that, but you have much more intimate knowledge about all the variations of OTF and Item functionality, so I need help in this effort.
I recommend that we get this testing done before doing anything else, as in the worse case scenario we may want to back out these two merges, which will become harder if we put more code changes on top.


Support component TS type resolving in Volar context

Purpose
Make Vue Vine become usage compatible with Vue SFC
i.e. user can import the components that export from .vine.ts in .vue file, and no type issues.
This issue may need Volar 2.0 TS Plugin's help.
We need to implicitly convert the "Vine component function" type to a "Vue component" object's type, based on information like props, emits and so on ...
I'd be so appreciated if @johnsoncodehk could provide some information about it. Thanks ❤ !

Tracking https://github.com/volarjs/volar.js/issues/85
Waiting for stable API from Volar 2.0

We should collect props and slots definition from Vine SFC and convert them into pure TS files

We should collect props and slots definition from Vine function component and convert them into pure TS files

Sounds good! So what is the actual form/struct/content in those pure TS files?
// _VINE_VLS.virtual.ts ??

interface _Comp1_Props_ {
  // ...
}

interface _Comp_Slots {
  // ...
}

interface _Comp_Emits {
  // ...
}

You mean something like this ??

Yes! And then we can convert Vine SFC into such code:
export default defineComponent({
  props: {} as _Comp1_Props,
  slots: {} as _Comp1_Slots,
  emits: {} as _Comp1_Emits,
})

So you mean I need to write a complete process of this generation by myself manually or just provide these enough data for Volar new API?🥹🥰😜

Not sure if volar provides such function 🥹


[Feat] Java solutions Sprint-1

This PR contributes Java solutions for the following programming challenges:

Determining Even/Odd Numbers
Checking for Prime Numbers
Validating Leap Years
Calculating Armstrong Numbers
Generating the Fibonacci Series
Identifying Palindromes
Crafting Star Patterns

Notes:

Solutions are written in Java, following best practices.
More solutions will be added in future commits.

Future Contributions:

Additional Java solutions and improvements are planned.
Feedback is welcome to ensure alignment with repository standards.

@PrinceSinghhub
I forgot to create a separate branch. I will do it next time, for sure.


connect function will not call error handler as they don't get registered

$dbConnection = new React\MySQL\Connection($loop, array(
'dbname' => 'mydb',
'user'   => '',
'passwd' => '',
));
$dbConnection->connect(function ($err, $conn) use ($loop) {
    echo "abc";
    $this->assertEquals("Access denied for user 'test'@'localhost' (using password: YES)", $err->getMessage());
    $this->assertInstanceOf('React\MySQL\Connection', $conn);
    //$loop->stop();
});

Even though connection is rejected we are not notified with the above code. Therefore we have to use the following to catch the error. I don't think that this is the expected behaviour for the above code block. Or is it?
$dbConnection->on('error',function($error){echo "$error";});

Looking at the code, you're right. The connect callback is only called on authenticated. And, the error handler is called on error.

The current code should correctly notify you through the callback when the remote server rejects your authentication information (such as invalid username/password). This is also covered by the test suite.
However, the current code does not notify you through the callback if the underlying TCP/IP connection to the server is rejected (such as invalid hostname / port). However, it does notify you through the error event.
I agree that this inconsistency is rather annoying and should be fixed. I'll look into this :+1:


speed up read_ts

this is currently read.table. Is pretty slow (seems slower than the file download) and could be sped up by avoiding read.table. Something in tom eventually?

Just to clarify, you mean read.table from the file system is taking more time than download.file?

yeah, haven't provided it, but seems to be the case. They are close. can do both in sequence:
file_handle <- download_ts(site = 'nwis_01408500', variable = 'doobs')
dissolved_oxygen <- read_ts(file_handle)

Slow step is most likely this line
df[, 1] <- as.POSIXct(df[, 1])

for a well formatted file, there aren't much faster options than read.table so tom is unlikely to improve the situation. May want to check where the bottleneck really is and maybe look into the fasttime package.,

Odd, I always thought read.table was slower than alternatives. Got any sources to back up your claim? data.table has some zip to the fread: http://stackoverflow.com/questions/1727772/quickly-reading-very-large-tables-as-dataframes-in-r/15058684#15058684

that was just a "re:docker" comment. I don't really want to see sources

Here's the bigger issue here really.
http://programmers.stackexchange.com/questions/80084/is-premature-optimization-really-the-root-of-all-evil
always something to keep in my when you have a working function.

Thanks, sport

this is going to be in read_unitted()


Alternative start script for multicontainer apps

Our current start script is only suitable for applications running in the same container as WiFi Connect. Applications running in separate containers may need to start after WiFi Connect exits: similarly to how our start script currently works. We need to provide an alternative start script for this use case.
 Front conversations

It would be greatif resin-wifi-connect would run in separate container and if resin docker host loses internet connection this would be the fallback for it. Docker host would take care of internet connectivity for other containers.


Inline function calls with hotkeys using & are not converted correctly

V1:
; Turn off CapsLock
CapsLock::SetCapsLockState, AlwaysOff

; Turn on CapsLock
CapsLock & Tab::SetCapsLockState, AlwaysOn

V2 (Converted):
; Turn off CapsLock
CapsLock::SetCapsLockState("AlwaysOff")

; Turn on CapsLock
CapsLock & Tab::SetCapsLockState, AlwaysOn

V2 (Expected):
; Turn off CapsLock
CapsLock::SetCapsLockState("AlwaysOff")

; Turn on CapsLock
CapsLock & Tab::SetCapsLockState("AlwaysOn")

Inline function call with hotkeys not using & are converted correctly.
Code block with hotkeys using & are also converted correctly.
Only inline function calls with hotkeys using & are not converted correctly.

Fixed this
Just need to test some edge cases


[Snyk] Upgrade: , , , axios

Snyk has created this PR to upgrade multiple dependencies.
👯♂ The following dependencies are linked and will therefore be updated together.

:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

Name
Versions
Released on

@backstage/plugin-scaffolder-commonfrom 1.5.4 to 1.5.5 | 4 versions ahead of your current version | 25 days agoon 2024-08-20
@backstage/plugin-scaffolder-reactfrom 1.10.0 to 1.11.0 | 5 versions ahead of your current version | 25 days agoon 2024-08-20
@kubernetes/client-nodefrom 0.20.0 to 0.21.0 | 1 version ahead of your current version | 4 months agoon 2024-05-03
axiosfrom 1.7.4 to 1.7.5 | 1 version ahead of your current version | 22 days agoon 2024-08-23

Release notes

Package name: @kubernetes/client-node

0.21.0 - 2024-05-03What's Changed

Support for Kubernetes 1.29.x
fix: Fix TypeError of topNodes by @ koooge in #1445
fix: do not overwrite current context by @ carlossg in #1487
feat: custom path for service account token (supporting TokenVolume projection) by @ omerlh in #1549
fix: use correct types for websocket by @ mstruebing in #1665
Lots of dependency verison updates

Full Changelog: 0.20.0...0.21.0

0.20.0 - 2023-11-02What's Changed

Add node 20 to our test matrix, remove node 14, fix test for node 20. by @ brendandburns in #1269
Automated Generate from openapi release-1.28 by @ github-actions in #1278
Add an additional test for version synchronization by @ brendandburns in #1268
Update to version 0.20.0 and k8s 1.28 support by @ brendandburns in #1281
ci: pin node version to 20.7.0 to make unit tests pass by @ mstruebing in #1313
chore: Some cleanup by @ Ananya2001-an in #1320
Fix spelling in README.md by @ masonbrothers in #1345
Fixed typo in topPods command by @ kamadek-sys in #1369
feat(metrics): add single node metrics and query options by @ 0fatal in #1368
Update Kubernetes Version in README.md by @ masonbrothers in #1378
A lot of dependency updates

New Contributors

@ masonbrothers made their first contribution in #1345
@ kamadek-sys made their first contribution in #1369
@ 0fatal made their first contribution in #1368

Full Changelog: 0.19.0...0.20.0

    from @kubernetes/client-node GitHub release notes

Package name: axios

1.7.5 - 2024-08-23Release notes:
Bug Fixes

adapter: fix undefined reference to hasBrowserEnv (#6572) (7004707)
core: add the missed implementation of AxiosError#status property; (#6573) (6700a8a)
core: fix ReferenceError: navigator is not defined for custom environments; (#6567) (fed1a4b)
fetch: fix credentials handling in Cloudflare workers (#6533) (550d885)

Contributors to this release

 Dmitriy Mozgovoy
 Antonin Bas
 Hans Otto Wirtz

1.7.4 - 2024-08-13Release notes:
Bug Fixes

sec: CVE-2024-39338 (#6539) (#6543) (6b6b605)
sec: disregard protocol-relative URL to remediate SSRF (#6539) (07a661a)

Contributors to this release

 Lev Pachmanov
 Đỗ Trọng Hải

    from axios GitHub release notes

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:

🧐 View latest project report
📜 Customise PR templates
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Tests failed on https://issues.redhat.com/browse/RHTAP-3381. Merging.


Quickfix  breaks after a few uses

For some reason - I am finding that the <cr> key seems to break (by break i mean pressing enter just moves the cursor to the next line, does not select and jump to the target location) in the quickfix buffer after performing a few different searches.
It's hard for me to reproduce reliably, although I do wonder if it could be an issue related to vim?

When it happens again:

What does :map <cr> output?
Does it work again after :unmap <cr>?

:map <cr>
No mapping found

And unmap seems to fail as a result of that.

And it seems to say that both when <cr> seems broken, and when it works 😳

I wonder if it's related to this change: https://github.com/vim/vim/pull/3902

Yeah, I just wanted to make sure that no other plugin is doing anything weird.
No mapping is good. Then <cr> defaults to opening the file in the quickfix list (according to :h quickfix, /<enter>.

When <cr> stops working.. do :cnext/:cprev still work?
Does :set ft? report qf?
Does :set buftype? report quickfix?

You figured it out!
:set buftype? was returning empty, since :cnext and :cprev worked fine.
Manually calling :set buftype=quickfix fixed the problem.

Then you need to figure out which code is doing that. It's certainly not this plugin. :-)

Thanks for the help debugging! Much appreciated.
Does this look to you like something that could be related? Or do you think this is more in plugin land?
https://github.com/vim/vim/issues/516

If it was a plugin, :verbose set buftype? would print the Vim script which set the option last.

  buftype=
        Last set from ~/.vim/bundle/vim-grepper/plugin/grepper.vim line 884

Yeah, looks like this must be a vim issue

Huh, this is super weird. That line just does :copen which explicitly says to set buftype to quickfix.
The function executing that line runs on the exit handler, so it might be a race condition in Vim.
Would be interesting to know if you can reproduce that with Nvim.

Looks like it was a Vim bug after all:
https://github.com/vim/vim/pull/4056

Ah, good to know. Thanks for the link!


Hotfix/2194

Fixing #2194
There was a problem with pretty printing, where klabels wouldn't be properly escaped.

I tested it and it works on the other agent examples.
looks good to me

we don't need to be extra strict about this but the convention is for the reviewer to merge.
regardless, thank you for the fix!

Sorry, I did not know about the convention... Maybe I should rebecome
active in this project :)
În Vin, 12 aug. 2016, 01:52 Cosmin Radoi<EMAIL_ADDRESS>a
scris:

we don't need to be extra strict about this but the convention is for the
reviewer to merge.
regardless, thank you for the fix!
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/kframework/k/pull/2196#issuecomment-239317394, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACwZU_8ngQerC85HWrW2XSYQ-OO-VB8Tks5qe6ehgaJpZM4JiNzG
.


Passing other params to handleSubmit/onSubmit

I still have trouble understanding how handleSubmit and onSubmit works but i just read #190 and i think i begin to understand.
But now i would like to pass parameters to handleSubmit.
Why? Because i have a wizard form like this :
<Parent>
      {child === 1) && <Child1 }/>}
      {child === 2) && <Child2 }/>}
      {child === 3) && <Child3 }/>}
      {child === 4) && <Child4 }/>}
</Parent>

And inside each  i have a  controller with a previous and a next button.
When i click on previous or next i want to pass a parameters to my global handleSubmit function to indicate if it's need to update one of my field input to go next or prev. (The current step of the wizard is in my form state.
I would like to be able to do handleSubmit('next')
But i really don't understand where and how should i put this.
Thanks.
P.S : Kind of related to #2505
P.S : Tell me if i'm not clear...

Think of handleSubmit as the adaptor function between an html5 form and a reduxForm, while onSubmit is your function that states what you want to do when the form is submitted. For example handleSubmit prevents the default action and provides special behaviour if it receives a Promise.
Thus, for the most part, your onSubmit function is where most of your application logic should live. If it returns a promise, you can decide whether to resolve the previous question or next question change asynchronously, or reject with SubmissionError that will fail submit validation and mark errors on invalid fields.
While we often leave support requests like this open for a time, this tracker is primarily for bugs and feature requests (including docs enhancement requests) as stated in the template. Stack overflow is better suited for questions like these, but you can post issues with a Stack Overflow link as desired.


HTTP proxy for kube-controller-manager

Hi guys
We need to set http_proxy (and no_proxy) for the kube-controller-manager binary.
Use case: kube-controller-manager talks to external systems (i.e. to the Cloud API for storage provision) which are behind a proxy in our environment.
This is currently not possible with the BOSH release - we'd like to add this config item to the release.
This basically means to add export http_proxy=<%= p('http_proxy') %> (and no_proxy) before launching the binary in kubernetes_controller_manager_ctl.
Before I create a PR I have some questions:

Would you welcome it?
For which jobs shall we expose the proxy? We only need it for the kube-controller-manager job, however, for consistency, it should probably be added for all jobs that interact with external systems
Shall I also create a PR for kubo-deployment? If yes, should I reuse the already existing env.http_proxy that is used for BOSH?

hey @MatthiasWinzeler. Your use case and proposed PR sound very reasonable. Please proceed and we will be happy to review/merge.

hi @glestaris
created the PR #130
We figured out that not only the kubernetes-controller-manager interacts with the cloud provider, but also other jobs do.
Disclaimer: The PR is just a quick & dirty way to get it running for us, it probably needs some polishing - let's consider it a discussion basis :)
Also, these vars need to be exposed in kubo-deployment as well, but we can create a PR there if this is merged.
Thank you!

https://github.com/cloudfoundry-incubator/kubo-release/pull/130 is merged.


Captouch button only works when grounded to my PC

Resolved in #64

(I configured it to use the wrong pin, lol)


Cant re-run grunt tasks since v1.18

I just upgraded to v1.18 and can no longer re-run a grunt task. This problem did not exist prior to this version.
The issue only exists if you try to run the same grunt task again, i.e I can run 'task1' once and if I try to run again I cannot but I can run 'task2' but then can't run 'task2' again.

VSCode Version: 1.18
OS Version: Windows 10 Pro

Steps to Reproduce:

Run a grunt task, in this case I'm running one of my grunt tasks called 'cleanAll' which cleans up some folders. This problem exists for any of my grunt tasks though.


refactor: remove redundant validateBasic check

Description
ValidateBasic has already been called in the GenerateOrBroadcastTxCLI function and we no longer need to call it in the CLi
closes: #XXXX

Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...

[x] included the correct type prefix in the PR title
[ ] added ! to the type prefix if API or client breaking change
[ ] added appropriate labels to the PR
[x] targeted the correct branch (see PR Targeting)
[ ] provided a link to the relevant issue or specification
[x] added a changelog entry to CHANGELOG.md
[ ] included comments for documenting Go code
[ ] updated the relevant documentation or specification
[ ] reviewed "Files changed" and left comments if necessary
[ ] confirmed all CI checks have passed

Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
I have...

[x] confirmed the correct type prefix in the PR title
[ ] confirmed all author checklist items have been addressed
[ ] reviewed state machine logic
[ ] reviewed API design and naming
[ ] reviewed documentation is accurate
[ ] reviewed tests and test coverage
[ ] manually tested (if applicable)

Summary by CodeRabbit

Refactor

Streamlined transaction processing by removing preliminary message validation steps in various financial operations.

Bug Fixes

Ensured consistent transaction handling across different modules by standardizing control flow.

The checks are not redundant. We want to keep them, and avoid broadcasting a transactions with messages that are not correct.

As I said, the ValidateBasic function is called before the broadcast by performing the following steps, so you are essentially calling ValidateBasic twice before broadcasting the message.
return tx.GenerateOrBroadcastTxCLI(clientCtx, cmd.Flags(), msg)

func GenerateOrBroadcastTxCLI(clientCtx client.Context, flagSet *pflag.FlagSet, msgs ...sdk.Msg) error {
	txf, err := NewFactoryCLI(clientCtx, flagSet)
	if err != nil {
		return err
	}

	return GenerateOrBroadcastTxWithFactory(clientCtx, txf, msgs...)
}

func GenerateOrBroadcastTxWithFactory(clientCtx client.Context, txf Factory, msgs ...sdk.Msg) error {
	// Validate all msgs before generating or broadcasting the tx.
	// We were calling ValidateBasic separately in each CLI handler before.
	// Right now, we're factorizing that call inside this function.
	// ref: https://github.com/cosmos/cosmos-sdk/pull/9236#discussion_r623803504
	for _, msg := range msgs {
		if err := msg.ValidateBasic(); err != nil {
			return err
		}
	}
...
}

Relevant PRss: https://github.com/cosmos/cosmos-sdk/pull/9236#discussion_r623803504


Chunk support?

Does this library support Chunk? I'm not seeing either in the code or the docs.

no answer yet?

currently the library does'nt support chunks uploads

sorry. uploading chunks is related to the server side protocol and how it
accepts chunks and headers, i dont believe we can cover all of possible
implementations. do you have any nginx module or something in mind to
implement it for it?
On 24 November 2017 at 13:19:59, AndreasSchmid1<EMAIL_ADDRESS>wrote:

no answer yet?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/bleenco/ngx-uploader/issues/363#issuecomment-346816721,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABtntqixlskQcL18Br07thO-gu4OaaFmks5s5rRvgaJpZM4QDhrW
.


Add trace level log support in containerd.

Description
logrus doesn't support trace. But it is useful in general https://github.com/containerd/cri-containerd/issues/548.
In https://github.com/containerd/cri-containerd/pull/547/files#diff-1135f265d25e96e557b8114567653176 we added the library to support Trace in cri-containerd.
Can we have this library in containerd? Or can we at least accept trace in log-level flag?

Steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Output of containerd --version:
(paste your output here)

@crosbymichael Do we want to upstream the TraceLevel?

That would be ideal

@stevvooe This was proposed several times in logrus:

https://github.com/sirupsen/logrus/issues/663
https://github.com/sirupsen/logrus/issues/629
https://github.com/sirupsen/logrus/issues/463

Even several PRs:

https://github.com/sirupsen/logrus/pull/652
https://github.com/sirupsen/logrus/pull/516
https://github.com/sirupsen/logrus/pull/198
https://github.com/sirupsen/logrus/pull/284
https://github.com/sirupsen/logrus/pull/643

I just don't understand why they don't want it.

@Random-Liu I am a maintainer there...
I think the problem with sirupsen/logrus#652 is that it is not backwards compatible for those using the interface to define a logger type. I'll LGTM it and see if that is a concern.


BUG - Element Settings dialog opens false Element

Does the issue occur in the electron-app or the web-app?

[ ] midi-bricks-electron
[ ] midi-bricks-web-app
[x] both

Can you determine, which version was used?
Version: 1.3.10
Steps To Reproduce
Steps to reproduce the behavior:

Go to 'Settings Mode'
Open Settings dialog of an existing Element
Edit e.g. Element title
Close Settings dialog
Open Settings dialog of another Element
Edit e.g. Element title
Close Settings dialog
See error: the false Element has been edited, namely the Element which was opened and edited before.

Check:
When leaving Settings Mode between editing two Elements, this error doesn't occur.

Go to 'Settings Mode'
Open Settings dialog of an existing Element
Edit e.g. Element title
Close Settings dialog
Exit Settings Mode
Enter Settings Mode again
Open Settings dialog of another Element
Edit e.g. Element title
Close Settings dialog
See: Works like expected.

Additional context:
After editing Elements with these troubles mentioned above, I saved the Preset. Directly after saving the faulty Elements didn't exist anymore. Also not after reloading the Preset, which was just saved.

My assumption: when opening an element in Settings Mode, it automatically gets selected (for multiple-selecion + "Copy to last page" function). Without leaving the Settings Mode between editing several Elements' Settings, all these Elements are selected. This seems to confuse the Settings Dialog.


It replicates the metadata file

This PR Fixes DICL/EclipseMETA#58
Checklist

[x] Follows Google Coding Style.
[x] At least one reviewer approved (for non-trivial changes).

Not ready to merge, it needs refactor and peer review

This PR is ready to merge, please peer review it :+1:

The PR was closed not merged into the master branch. It has conflicts, Ill solve them


Introduces a CancellationPanic that forcibly kills a task after a grace period.

STERN. STERN BUT FAIR.
Side effect; we now have a function that checks for ShouldCancel as this will let us introduce more logic in here later (if we need it). This is a breaking change.

LGTM


Any particular reason to the protobuf version constraint >2.4.1?

Currently it requires protobuf > 2.4.1
Is there any particular reason? I'm asking this because we internally are using a customized protobuf, which is based on 2.4.1.

Not completely sure why that was. Can you try running against your custom version?

It works with our customized 2.4.1 version

I was getting error as below any idea?
++++
$ python
Python 2.6.6 (r266:84292, Jul 23 2015, 15:22:56)
[GCC 4.4.7 20120313 (Red Hat 4.4.7-11)] on linux2
Type "help", "copyright", "credits" or "license" for more information.

from snakebite.client import Client
Traceback (most recent call last):
File "", line 1, in
File "/usr/lib/python2.6/site-packages/snakebite-2.11.0-py2.6.egg/snakebite/client.py", line 16, in
import snakebite.protobuf.ClientNamenodeProtocol_pb2 as client_proto
File "/usr/lib/python2.6/site-packages/snakebite-2.11.0-py2.6.egg/snakebite/protobuf/ClientNamenodeProtocol_pb2.py", line 12, in
import Security_pb2
File "/usr/lib/python2.6/site-packages/snakebite-2.11.0-py2.6.egg/snakebite/protobuf/Security_pb2.py", line 231, in
DESCRIPTOR.message_types_by_name['TokenProto'] = _TOKENPROTO
AttributeError: 'FileDescriptor' object has no attribute 'message_types_by_name'

++++
python ./inject.py  host-namen-node 8030<EMAIL_ADDRESS>hdfs "/bin/ls"
Traceback (most recent call last):
File "./inject.py", line 3, in
from snakebite.client import Client
File "/home/hadoop/poc/snakebite/client.py", line 16, in
import snakebite.protobuf.ClientNamenodeProtocol_pb2 as client_proto
File "/home/hadoop/poc/snakebite/protobuf/ClientNamenodeProtocol_pb2.py", line 12, in
import Security_pb2
File "/home/hadoop/poc/snakebite/protobuf/Security_pb2.py", line 231, in
DESCRIPTOR.message_types_by_name['TokenProto'] = _TOKENPROTO
AttributeError: 'FileDescriptor' object has no attribute ‘message_types_by_name'
++


Fix receipt image modal logic

Details
Allows users to tap on the receipt image right away, without the need for a thumbnail url
Fixed Issues
$ https://github.com/Expensify/App/issues/26590
Tests

Log into NewDot and create a new Workspace
Navigate to a workspace chat
Tap "+" and select "Request money" > "Scan"
Scan a receipt and submit a request
Open the request in the chat and tap the scanned receipt image
Verify that the image modal opens and the receipt image loads
Go to Settings > Preferences and turn on Force offline
Repeat steps 3-6

[x] Verify that no errors appear in the JS console

Offline tests
Covered above
QA Steps
Same as test

[ ] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
https://github.com/Expensify/App/assets/22219519/71b02fd0-fdc7-44fc-93a9-99358c29dacd

Mobile Web - Chrome
https://github.com/Expensify/App/assets/22219519/fa5133b5-67a2-4f4b-84b4-a2909292bce6

Mobile Web - Safari
https://github.com/Expensify/App/assets/22219519/14ffe52c-c8e2-4a5f-90ce-27cec555836b

Desktop
https://github.com/Expensify/App/assets/22219519/6b9d67c1-97c3-4466-a137-fe3d6b022a02

iOS

https://github.com/Expensify/App/assets/22219519/29f3bfe2-fef8-449a-81df-0b321c3064ca

Android
https://github.com/Expensify/App/assets/22219519/324e9b26-8989-4c26-90a9-5c248880005f

Asked for a C+ review here

When image is replaced while previewing, attachment modal closes. I think that's fine but just to confirm
https://github.com/Expensify/App/assets/108292595/a388f750-d61c-4250-9152-1c6ae347cc7e

Local image in distance request is also being previewed. Hope that's fine too.
https://github.com/Expensify/App/assets/108292595/c4d827a7-a08b-418b-bce7-fd70da4309ce

It doesn't make sense to show download button for local image. Anyway not blocker
https://github.com/Expensify/App/assets/108292595/535b69ef-fb31-4578-85df-2203d26803b7

Yea, I think that's fine since it's an edge case and we wanna display the up to date image. Gonna merge this so we can get it out in time for saastr

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/luacmartins in version: 1.3.65-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/mountiny in version: 1.3.65-7 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/luacmartins in version: 1.3.66-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/Beamanator in version: 1.3.66-3 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


Fix i2c busy / bus busy handling

Restore previous logic where a transaction starts by waiting for the controler
to be ready and the bus available and then only wait for the controller to be ready
at intermediate step (not the bus to be available as it will be busy).
Modify the i2c_busy_wait to implement this behavior.
fix #34

To be clear, this basically reverts https://github.com/rust-embedded-community/tm4c-hal/pull/29/commits/4d50753d21331a667a97bc585f83fe6b5d7cf26c

Not true ! :) It keeps most of these changes, only it won't raise an error when busbsy is set everytime the macro is used, only when the extra argument is provided (eg. at beg of transaction).
It still spins on busy to be clear before reading other bits (which the patch you link fixes).

I appreciate the clarification. I find Github diff mode quite hard to grok :/

No problem ! And I may still have overlooked something, so feel free to question my choices :) I can still say that previous try to use i2c always failed with the BusBusy error since the @dtwood's changes and now it works correctly for hours. Not a clear proof it fixes something, but at least, it works here :)

Great, thanks ! I can say that before your changes, i2c was really unstable, after your fix it does not work at all, and the combination of your fix and these changes makes it work flawlessly on my app. It is simple, so can't say it's a very robust test, but it is testing basic things...


weird results

Hi, I am a totally new starter and my question could be too simple to you.
When I run the inference.py and I get this werid result, could you please tell me the reason?
My cmd line is:
python scripts/inference.py --exp_dir=exp_0308_001/ --checkpoint_path=pretrained_models/hyperstyle_ffhq.pt --data_path=./exp_data/ --test_batch_size=8 --test_workers=8 --n_iters_per_batch=5 --load_w_encoder
Input:

Output:

You need to make sure you align the image before running inference.

Thank you for your patient reply.


Add the Ferret compiler

Works for JVM (the YS command)
does unfortunately not compile with GRAAL

I am stuck on this PR, because I do not know how to GRAAL compile all this. For the record, here is a cc of the respective element chat from Jun 5, 2024:

it's just a bug to fix and if it approaches impossible then we can do it another way
we can always shell out or use a Pod (see bb pods)

I am not sure about the best way to "shell out" or use pods either
So before tidying up the above issues (i.e. make a cpp.clj namespace, rename to compile-ys-to-cpp) it would be nice to clarify on the usage of Ferret with GRAAL. Especially the use of Ferret within a YAMLScript-shared-library, i.e. how to generate C++ code out of a YAMLScript-file from within Python (not using the ys command line tool at all)

Ah, we can trigger this from the Python certainly, but why do we want to?
It would be helpful for you to describe as much as you know, what a Python programmer would be trying to do here and what the shape of her Python code would look like...

The idea is that a Python programmer can mix YAMLScript into Python code. Maybe by this transforming Python to YS over time, piece by piece.
Sure, that code-mixing can be done by subsequently shelling out to the ys command. Indeed I made an example ptutorial.py which creates a temporary file and calls ys and ferret and clang commands. The idea behind integrating ferret into the YAMLScript-Python library is mostly to beautify that example by reducing the shell command to one call to the clang C++ compiler.

Here's a way to use start using YS in Python now:
import os, yamlscript

def main(a, b):
    print("%s + %s = %s" % (a, b, a + b))
    print("%s + %s = %s" % (a, b, add1(a, b)))
    print("%s + %s = %s" % (a, b, add2(a, b)))

def add1(a, b):
    return a + b

def add2(a, b):
    return runYS(
    """
!yamlscript/v0

a =: ENV.a1.num()
b =: ENV.a2.num()

=>: a + b
"""
    , a, b)

def runYS(code, *args):
    i = 1
    for v in args:
        os.environ["a" + str(i)] = str(v)
        i = i + 1
    return yamlscript.YAMLScript().load(code)

main(3, 4)

We can make this better certainly, by expanding the load() function to take an argument providing more context data.

It's on my short list to expand/redo the libyamlscript API anyway.
We need load() to not be all-powerful by default.
It's powers need to be asked for explicitly by then caller.


Invalid code from logging-operator crds (properties containing periods)

The code generated from logging-operator crds has some syntax errors.
Source: banzaicloud/logging-operator
Some generated interface property names contain periods and are not quoted.
e.g. .../types/output.ts:
        export interface LoggingSpecFluentbitBufferStorage {
            storage.backlog.mem_limit?: string;
            storage.checksum?: string;
            storage.path?: string;
            storage.sync?: string;
        }

(crd2pulumi version v1.0.5)

This is a variant of https://github.com/pulumi/crd2pulumi/issues/43 or more generally https://github.com/pulumi/pulumi/issues/15874 since punctuation is not currently stripped.


My new favourite app!! - some feedback, suggestions & bugs

First of all, thanks to have made this app in first case, i have been thinking about making something out of monaco editor for ages but never got the time, so thanks from the deep of my heart to have made it, and made it so good!
I really love that it is a one time purchase, i messed around did some basic app and backend's with the free beta on TestFlight i fall in love with the app and instantly bought it! The price tag is also really accessible, i found the one time purchase houndres of times better than a monthly subscription so please never change the price model, a raise of the price would be better, there are app that do less than yours at 14.99€ (wich is still unexpensive compared to the price tag that software that i commonly use at work like Intellij Idea Ultimate, or Visual Studio IDE Pro, so a raise would be still extreamely fine and a better choice for the end user compared to a monthly subscription in my hopinion).
I will not ask for the problems that there are with touch controls because i know those depends on monaco editor itself, but you did an amazing job with the overral UI making a real VSCode native port with Swift UI.
One of my favourite feature is that python and node are built-in and work offline for me this is essential and a game changer. it is way better compared to the many online compiler that can be found in the store, can we expect other runtime coming to the app? I don't know like .net? Since i think it would be easier than Java, there is already an app called Continuous .NET that had success porting C# to iPad/iPhone even with the possability to use graphics api (Open GL) and UI frameworks like Xamarin.
(A fast question i would like to ask you is: when the 1.4.5 will be out on App Store, i see that the TestFlight version is already downloadable while the App Store version is still 1.4.4)
At the end i want to say thanks again and i hope the project will have a really long life since many promising project got abbandoned (like Continuous .NET itself), this is indeed the most promising and the best option there it is in this moment!

Thank you! This project started as a personal experiment and I'm glad people are enjoying it.
For .NET I will definitely take a look at the app you mentioned. It will provide interesting use cases if graphics is supported. Also thanks for reminding me of 1.4.5, I should be able to make a release later this week.


bump pydantic-core to 2.5.0

Change Summary
Bumps to pydantic_core==2.5.0. I pulled in the commit to enable the new decimal implementation to satisfy tests.
Related issue number
N/A
Checklist

[ ] The pull request title is a good summary of the changes - it will be used in the changelog
[ ] Unit tests for the changes exist
[ ] Tests pass on CI
[ ] Documentation reflects the changes where applicable
[ ] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

please review


Fix report icon logic for newly created policy rooms

@deetergp, please review when you get the chance.
CC: @neil-marcellini for help in testing this out on iOS/android and providing screenshots!
Details
Fixes bug that causes newly created policy rooms to crash when you click on their details. Since the rooms have
Fixed Issues
$ https://github.com/Expensify/App/issues/7182
$ https://github.com/Expensify/App/issues/7055
$ https://github.com/Expensify/App/issues/7262
Tests/ QA Steps

Log into NewDot with an account that has a workspace or create a workspace using the plus on the bottom left. Make sure you are the only person in the workspace.
Using the same plus on the bottom left, choose New Room. Fill in the options in the modal and after the new room is created, verify that you see the couch icon for the room after it is created:

Click on that icon and verify you can access the room details like in the screenshots below. The room's settings should also be accessible.

[ ] Verify that no errors appear in the JS console

Tested On

[x] Web
[x] Mobile Web
[ ] Desktop
[x] iOS
[x] Android

Screenshots

Web/ Mobile Web

iOS

Android

LGTM

@deetergp looks like this was merged without passing tests. Please add a note explaining why this was done and remove the Emergency label if this is not an emergency.

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

The tests were passing. Not sure why it got flagged. Removing the Emergency label.

🚀 Deployed to staging by @deetergp in version: 1.1.33-0 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
failure ❌

🍎 iOS 🍎
failure ❌

🕸 web 🕸
failure ❌

🚀 Deployed to production by @roryabraham in version: 1.1.33-3 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


purge-deleted should try to keep the cursor where it was

fixes #1728

Coverage decreased (-0.02%) to 59.195% when pulling c1a253964719e0db307b85fe1a83d99e0c23c64a on allanwind:fix1728 into cb4aced4f33ae1c5c19691fdd7a18ecc1532d443 on newsboat:master.