Missing code key in url query

Triggered by this code
this is the code in Imperial/FederatedServiceRouter that throws:
`    public func fetchToken(from request: Request) throws -> EventLoopFuture {
let code: String
if let queryCode: String = try request.query.get(at: codeKey) {
code = queryCode
} else if let error: String = try request.query.get(at: errorKey) {
throw Abort(.badRequest, reason: error)
} else {
throw Abort(.badRequest, reason: "Missing 'code' key in URL query")
}
    let body = callbackBody(with: code)
    let url = URI(string: accessTokenURL)

    return body.encodeResponse(for: request)
        .map { $0.body.buffer }
        .flatMap { buffer in
            return request.client.post(url, headers: self.callbackHeaders) { $0.body = buffer }
        }.flatMapThrowing { response in
            return try response.content.get(String.self, at: ["access_token"])
        }
}`

Known issue people had already before:
Post here how sb fixed it
Alright. I think I figured it out...
What's the chance the data is being serialized as JSON even though I set static var defaultContentType: HTTPMediaType = .urlEncodedForm in LinkedInCallbackBody? The chance ended up being pretty high, given that adding a client.headers = HTTPHeaders(dictionaryLiteral: ("Content-Type", "application/x-www-form-urlencoded")) to the beforeSend closure eliminated the issue I was having.
That, or Vapor doesn't automatically set that header... which would make more sense.
public func fetchToken(from request: Request) throws -> EventLoopFuture<String> {
    let code: String
    if let queryCode: String = try request.query.get(at: "code") {
        code = queryCode
    } else if let error: String = try request.query.get(at: "error") {
        throw Abort(.badRequest, reason: error)
    } else {
        throw Abort(.badRequest, reason: "Missing 'code' key in URL query")
    }

    let body = LinkedInCallbackBody(code: code, clientId: self.tokens.clientID, clientSecret: self.tokens.clientSecret, redirectURI: self.callbackURL)

    let url = URI(string: self.accessTokenURL)
    return body.encodeResponse(for: request).map {
        $0.body
    }.flatMap { body in
        return request.client.post(url, beforeSend: { client in
            client.body = body.buffer
            client.headers = HTTPHeaders(dictionaryLiteral: ("Content-Type", "application/x-www-form-urlencoded"))
        })
    }.flatMapThrowing { response in
        return try response.content.get(String.self, at: ["access_token"])
    }
}

<@!432065887202181142> This is what the (working) method ended up being. I'm willing to share my Imperial+LinkedIn code, if you think that'd be of any use.
I'm not quite good/confident enough with this to feel 100% comfortable submitting a PR, but maybe I should anyway...
public func fetchToken(from request: Request) throws -> EventLoopFuture<String> {
    let code: String
    if let queryCode: String = try request.query.get(at: "code") {
        code = queryCode
    } else if let error: String = try request.query.get(at: "error") {
        throw Abort(.badRequest, reason: error)
    } else {
        throw Abort(.badRequest, reason: "Missing 'code' key in URL query")
    }

    let body = LinkedInCallbackBody(code: code, clientId: self.tokens.clientID, clientSecret: self.tokens.clientSecret, redirectURI: self.callbackURL)

    let url = URI(string: self.accessTokenURL)
    return body.encodeResponse(for: request).map {
        $0.body
    }.flatMap { body in
        return request.client.post(url, beforeSend: { client in
            client.body = body.buffer
            client.headers = HTTPHeaders(dictionaryLiteral: ("Content-Type", "application/x-www-form-urlencoded"))
        })
    }.flatMapThrowing { response in
        return try response.content.get(String.self, at: ["access_token"])
    }
}

<@!432065887202181142> This is what the (working) method ended up being. I'm willing to share my Imperial+LinkedIn code, if you think that'd be of any use.
I'm not quite good/confident enough with this to feel 100% comfortable submitting a PR, but maybe I should anyway...
public func fetchToken(from request: Request) throws -> EventLoopFuture<String> {
    let code: String
    if let queryCode: String = try request.query.get(at: "code") {
        code = queryCode
    } else if let error: String = try request.query.get(at: "error") {
        throw Abort(.badRequest, reason: error)
    } else {
        throw Abort(.badRequest, reason: "Missing 'code' key in URL query")
    }

    let body = LinkedInCallbackBody(code: code, clientId: self.tokens.clientID, clientSecret: self.tokens.clientSecret, redirectURI: self.callbackURL)

    let url = URI(string: self.accessTokenURL)
    return body.encodeResponse(for: request).map {
        $0.body
    }.flatMap { body in
        return request.client.post(url, beforeSend: { client in
            client.body = body.buffer
            client.headers = HTTPHeaders(dictionaryLiteral: ("Content-Type", "application/x-www-form-urlencoded"))
        })
    }.flatMapThrowing { response in
        return try response.content.get(String.self, at: ["access_token"])
    }
}

<@!432065887202181142> This is what the (working) method ended up being. I'm willing to share my Imperial+LinkedIn code, if you think that'd be of any use.
I'm not quite good/confident enough with this to feel 100% comfortable submitting a PR, but maybe I should anyway...
I cannot fix this in the source code but as I saw other people are having the same issue, maybe this bug could be fixed.

What version of Imperial do you have in your Package.swift and what does your setup code look like? I've just built an app using Sign in With Google and it's working for me

Hi Tim,
thank you for having a look into this.
I am basically following exactly the code in the book but the processGoogleLogin func is never reached.
Code attached with configuration file.
Kr, Andreas
Secured Email (ProtonMail).
------- Original Message -------
Tim Condon @.***> schrieb am Montag, 4. April 2022 um 19:16:

What version of Imperial do you have in your Package.swift and what does your setup code look like? I've just built an app using Sign in With Google and it's working for me
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

configure.swift.zip
ImperialRouter+Controller.swift.zip

Package
// swift-tools-version:5.5
import PackageDescription
.package(url: "https://github.com/vapor-community/Imperial.git", from: "1.1.0"),
.product(name: "ImperialGoogle", package: "Imperial"),
fetched version: 1.1.0

So my guess is that the app isn't configured correctly in the Google console. Do you get any information in the browser logs etc?

And have you run a package update to make sure you're on the latest code?

And have you run a package update to make sure you're on the latest code?

Sure

Tim, I was able to catch the following errors. What does it mean?
[Error] Unrecognized Content-Security-Policy directive 'require-trusted-types-for'.
[Error] Unrecognized Content-Security-Policy directive 'require-trusted-types-for'. (x2)
[Error] Unrecognized Content-Security-Policy directive 'worker-src'.
[Error] Unrecognized Content-Security-Policy directive 'require-trusted-types-for'.
[Error] Unrecognized Content-Security-Policy directive 'require-trusted-types-for'.
[Error] Unrecognized Content-Security-Policy directive 'require-trusted-types-for'. (x2)

But this error is only shown when I try it for the second time and I am already logged in to google in this browser window

I re-checked all data in the google account and it is exactly done as described in the vapor book

but maybe I miss something.

Hmm so I can't reproduce. What I suggest you do is debug the request/responses that happen during the OAuth handshake and see what the responses returned from Google are etc

how can i see the google responses?
Sent from ProtonMail for iOS
Am Sa., Apr. 9, 2022 um 20:19, Tim Condon @.***> schrieb:

Hmm so I can't reproduce. What I suggest you do is debug the request/responses that happen during the OAuth handshake and see what the responses returned from Google are etc
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

Put breakpoints in the code in Imperial and use the debugger to inspect what requests or responses Google makes

Tim, am I missing something regarding Cors. Does this have an effect on Imperial?
https://developer.mozilla.org/en-US/docs/Web/HTTP/CORS#requests_with_credentials

I've implemented Imperial in a few projects recently and not hit this issue. Is it still an issue?
CORS is not an issue for this library

@0xTim  I tried it again and posted it also in the Vapor discord channel.
Everything works well when I run it locally.
As soon as I run it on it in the production environment I get this error.
[ WARNING ] Abort.400: Missing 'code' key in URL query [request-id: 9EA82C34-50CB-4293-82D5-91312438C7CC] (ImperialCore/Routing/FederatedServiceRouter.swift:100)
https://discord.com/channels/431917998102675485/443878065227956226/1069207958724038807
I spend a few days in January last year, then tried it again from March to June as you can see. Now I picked this issue up again and the success is that it runs locally but I cannot make it work on the server due to this issue.
The environment variables are correct.

Found the issue:
Following the request initiator chain in google
https://www.mywebsite.com/auth/google/
-> https://accounts.google.com/o/oauth2/auth?client_id=EVERYTHING_CORRECT_HERE
-> https://www.mywebsite.com/oauth/google?code=PROPER_CODE_HERE_AS_WELL
-> https://www.mywebsite.com/oauth/google/
I discovered that there is a moment where the code gets lost.
The problem was caused by this middleware that extends url paths with "/"
struct ExtendPathMiddleware: AsyncMiddleware {
func respond(to request: Request, chainingTo next: AsyncResponder) async throws -> Response {
if !request.url.path.hasSuffix("/") && !request.url.path.contains(".") {
return request.redirect(to: request.url.path + "/", type: .permanent)
}
return try await next.respond(to: request)
}
}
Put it here in case someone else searches for this and close the issue.
@0xTim thanks again for your ongoing support. Really hard to find the root cause sometimes.

ISSUE CAN BE CLOSED


The subset filter is slow

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
redhit
Modin installed from (source or binary):
github
Modin version:
2.0.5
Python version:
3.6.2
Exact command to reproduce:

Describe the problem

The filter is very slow,
old pandas
 %time data = data[data['FILTER']=="PASS"]
CPU times: user 24 s, sys: 2.92 s, total: 26.9 s
Wall time: 26.4 s

modin pandas
%time data = data[data['FILTER'].str.contains("PASS")]
W0102 13:37:16.832880 29990 node_manager.cc:243] Last heartbeat was sent 4481 ms ago

the modin pandas can't finished the command.

This part of the command data['FILTER'].str.contains("PASS") is going to take some time. Because we are still working on a distributed Series, this object will be collected and run on a single-core. This is what is taking time.
The error message you see is coming from Ray:
W0102 13:37:16.832880 29990 node_manager.cc:243] Last heartbeat was sent 4481 ms ago

This may mean that you do not have enough memory for the workload you are trying. If you have a lot of intermediate dataframes or objects, this could be the solution. We have some solutions planned for this problem, but for now they are not fully implemented yet.

Closing this. Feel free to reopen if the discussion should continue or if issue was not resolved.


Test MPI

Solution for issue #44

There are some features of phaseflow that have worked well in serial but will not work with MPI:

1D problems
Custom Newton iterations (since I wrote a simple Newton iteration routine from scratch with a serial algorithm)
fenics.AdaptiveNonlinearVariationalSolver (since DOLFIN reports that this simply is not supported yet)


get all grants associated to an account instead of delegators then grants

https://github.com/cosmos/cosmos-sdk/pull/11523
After this is released it would be good to migrate to getting grants associated to a grantee instead of getting delegators and so on

Ah man finally!! This is the biggest challenge with REStake right now, it has to hit the nodes so many times to get the info it needs. I figured an endpoint like this was coming but wasn't tracking anything specific.
Thank you, will implement as soon as it's available

ill try to get it released in the coming days, if not today

This was recently merged and some chains should start seeing it


Issues migrating from catkin_make run_tests to catkin run_tests

Hello,
I've recently switched from using catkin_make to using catkin build and, despite a few minor bumps, my packages are compiling fine. Switching from catkin_make run_tests to catkin run_tests has been a bit more difficult, however, as there are a number of test failures which I believe stem from a certain package "motoman_gp8_support" not being found. Below is an excerpt of the error, which is repeated throughout the tests
-- run_tests.py: execute commands
  /usr/bin/python3 /opt/ros/noetic/share/rostest/cmake/../../../bin/rostest --pkgdir=/root/checkout/internal/src/aa_lab --package=aa_lab --results-filename tests_integration_test_move_group_sequence.xml --results-base-dir "/root/checkout/internal/build/aa_lab/test_results" /root/checkout/internal/src/aa_lab/tests/integration/test_move_group_sequence.test
WARNING: Could not change permissions for folder [/root/checkout/logs], make sure that the parent folder has correct permissions.
... logging to /root/checkout/logs/rostest-fv-az90-360-433.log
resource not found: motoman_gp8_support
ROS path [0]=/opt/ros/noetic/share/ros
ROS path [1]=/root/checkout/internal/src/aa_lab
ROS path [2]=/root/checkout/internal/src/aa_cell_amp01_drivers
ROS path [3]=/root/checkout/internal/src/aa_cell_amp01_moveit
ROS path [4]=/root/checkout/internal/src/aa_cell_amp01_support
ROS path [5]=/root/checkout/internal/src/aa_component_support
ROS path [6]=/root/checkout/internal/src/aa_force_control
ROS path [7]=/root/checkout/internal/src/aa_init
ROS path [8]=/root/checkout/internal/src/aa_lab
ROS path [9]=/root/checkout/internal/src/aa_end_effector_support
ROS path [10]=/opt/ros/noetic/share
when processing file: /root/checkout/internal/src/aa_cell_amp01_support/urdf/world.urdf.xacro

CMakeLists.txt and package.xml for the aa_cell_amp01_support:
cmake_minimum_required(VERSION <IP_ADDRESS>)
project(aa_cell_amp01_support)

find_package(catkin REQUIRED)

catkin_package()

<?xml version="1.0"?>
<package>
  <name>aa_cell_amp01_support</name>
  <version>0.0.1</version>
  <description>redacted</description>
  <maintainer email="redacted">redacted</maintainer>
  <license>Commercial</license>
  <buildtool_depend>catkin</buildtool_depend>
  <run_depend>xacro</run_depend>
  <run_depend>aa_end_effector_support</run_depend>
  <run_depend>aa_component_support</run_depend>
  <run_depend>motoman_gp8_support</run_depend>
</package>

CMakeLists.txt and package.xml for the motoman_gp8_support package:
cmake_minimum_required(VERSION 2.8.3)

project(motoman_gp8_support)

find_package(catkin REQUIRED)

catkin_package()

if (CATKIN_ENABLE_TESTING)
  find_package(roslaunch REQUIRED)
  roslaunch_add_file_check(test/launch_test.xml)
endif()

install(DIRECTORY config launch meshes urdf
  DESTINATION ${CATKIN_PACKAGE_SHARE_DESTINATION})

<?xml version="1.0"?>
<?xml-model href="http://download.ros.org/schema/package_format2.xsd" schematypens="http://www.w3.org/2001/XMLSchema"?>
<package format="2">
  <name>motoman_gp8_support</name>
  <version>0.3.5</version>
  <description>

ROS-Industrial support for the Motoman GP8 (and variants).

      This package contains configuration data, 3D models and launch files
      for Motoman GP8 manipulators.

      Specifications

GP8 - Default

      Joint limits and maximum joint velocities are based on the information
      found in the online
      https://www.motoman.com/hubfs/Robots/gp8.pdf
      All urdfs are based on the default motion and joint velocity limits,
      unless noted otherwise.

      Before using any of the configuration files and / or meshes included
      in this package, be sure to check they are correct for the particular
      robot model and configuration you intend to use them with.

  </description>
  <author>Eric Marcil</author>
  <maintainer<EMAIL_ADDRESS>Edwards</maintainer>
  <maintainer<EMAIL_ADDRESS>vd. Hoorn (TU Delft Robotics Institute)</maintainer>
  <license>BSD</license>

  <url type="website">http://wiki.ros.org/motoman_gp8_support</url>
  <url type="bugtracker">https://github.com/ros-industrial/motoman/issues</url>
  <url type="repository">https://github.com/ros-industrial/motoman</url>

  <buildtool_depend>catkin</buildtool_depend>

  <test_depend>roslaunch</test_depend>

  <exec_depend>joint_state_publisher</exec_depend>
  <exec_depend>motoman_driver</exec_depend>
  <exec_depend>motoman_resources</exec_depend>
  <exec_depend>robot_state_publisher</exec_depend>
  <exec_depend>rviz</exec_depend>
  <exec_depend>xacro</exec_depend>

  <export>
    <architecture_independent/>
  </export>
</package>

I know that catkin build builds each package in isolation, so I'm wondering if there are some missing packages which were being shared across packages when using catkin_make...
The following warning also appears at the top of the test run. Unsure if it is related:
------------------------------------------------------------
WARNING: Your current environment's CMAKE_PREFIX_PATH is
different from the cached CMAKE_PREFIX_PATH used the last
time this workspace was built.

If you want to use a different CMAKE_PREFIX_PATH you should
call `catkin clean` to remove all references to the
previous CMAKE_PREFIX_PATH.

Cached CMAKE_PREFIX_PATH:
	/opt/ros/noetic
Current CMAKE_PREFIX_PATH:
	/root/checkout/external/devel:/root/checkout/internal/devel:/opt/ros/noetic
------------------------------------------------------------

It's also worth noting that the project consists of two workspaces, external and internal. aa_cell_amp01_support is a package in the  internal workspace, the motoman_gp8_support package is in the internal workspace, so wondering if that may be an issue

The xacro file mentioned in the error contains the following line, which is I believe where the failing import is happening:
<xacro:include filename="$(find motoman_gp8_support)/urdf/gp8_macro.xacro"/>

Verified that the tests work using catkin_make_isolated, so I don't think it's the isolated builds causing the issue. I'm wondering if it could be the last bullet point, taken from the migration guide, causing the issue:
"Additional Differences with catkin build
In addition to the differences due to isolation, catkin build is also different from catkin_make_isolated in the following ways:
It builds packages in parallel, using an internal job server to distribute load.
It puts products into hidden directories, and then symbolically links them into the devel space (by default).
It stores persistent configuration options in a .catkin_tools directory at the root of your workspace.
It passes --no-warn-unused-cli to the cmake command since not all packages accept the same CMake arguments.
It generates .catkin files where each source package is listed, individually, instead of just listing the source space for the workspace. This leads to similar ROS_PACKAGE_PATH variables which list each package source space."

This has been resolved now. The issue was that we're building two workspaces -- external and internal -- with dependencies between the two. Had to change the build script to build the external dependencies before the internal ones. Still a little unsure how it was working with catkin_make though...


Configurable transformer, plugins and presets.

Hi
Thanks for the plugin.
I am using Angular 2 Final and can't use all of the plugins listed in babel-preset-babili so it would be nice to have the option to disable presets and provide an array of plugins.

Will

Pr #4

This will be done on babili's side as Minifier Options - babel/babili#162
Adding custom plugins apart from babili provided options doesn't sound right IMO. Also, if the user wants selective optimizations / disable one plugin from the preset, then with this approach, s/he has to pass in all other plugins that babili uses - this is just bad.

babel/babili#162 is merged.


How do I give permission to use location services?

Hi,
My website have a page which requests for location and shows it on a google map. But I am not able to get this working once I build it. Any Idea?

Hi George, sorry I missed this issue.  Can you share the platform your building for?  Android would need you to turn location on in the package, Windows should prompt the user.


Add a readme file

The project is note taking app for iOS written in swift. Update the readme with the info about the project, technologies used

can you upload screenshot and feature of this app. It will help to create better README


Add JWT support for danubetech.

It built and ran on my system. The report shows Ed25519 and Secp256k1 VC-JWT and VP-JWT passing now. RSA does not appear.

@clehner I agree with your fix, thanks.
However I don't think we properly support RSA at the moment, so I suggest we merge as-is for now and maybe I'll do another PR later.

Sounds good to me


Handling MDNS request from localhost

Currently, the code throws an exception and exits if MDNS request is received from localhost (<IP_ADDRESS>)
Here is the message and my workaround
/home/me/tmp/node-matter/build/util/Network.js:42
    throw new Error(`Cannot find the device IP on the subnet containing ${remoteAddress}`);
    ^

Error: Cannot find the device IP on the subnet containing <IP_ADDRESS>
    at getIpMacOnInterface (/home/me/tmp/node-matter/build/util/Network.js:42:11)
    at MdnsServer.handleDnsMessage (/home/me/tmp/node-matter/build/mdns/MdnsServer.js:33:63)
    at /home/me/tmp/node-matter/build/mdns/MdnsServer.js:24:63
    at Socket.<anonymous> (/home/me/tmp/node-matter/build/io/udp/UdpMulticastServer.js:28:77)
    at Socket.emit (node:events:513:28)
    at UDP.onMessage [as onmessage] (node:dgram:922:8)

Node.js v18.7.0
$:~/tmp/node-matter$ diff node-matter/build/mdns/MdnsServer.js node-matter.x/build/mdns/MdnsServer.js
24c24,27
<         multicastServer.onMessage((message, remoteIp) => this.handleDnsMessage(message, remoteIp));
---
>         multicastServer.onMessage((message, remoteIp) => { try {
>             this.handleDnsMessage(message, remoteIp);
>         }
>         catch (e) { } });

Thanks! Here is a fix to handle MDNS requests from localhost: https://github.com/mfucci/node-matter/pull/19
Let me know if this fixes the issue!

I have tested the fix - it works.


Improving PR Title and Description Generation

What feature do you want to see added?
The generic title "Automatic PR" is not sufficiently descriptive. We need to create a more meaningful title that accurately reflects the purpose of the pull request (PR).
Recommendations:

Generic Title: Develop a default title that is both informative and broadly applicable to the modernization process.

Customization Option: If the generic title doesn't meet the user's needs, we should implement one of the following:

Add a command-line option to set a custom PR title
Create a variable that allows users to define their preferred title

Recipe Descriptions: When recipe descriptions are available, we should incorporate them into the PR body. This can be done as follows:

Create a separate section for each recipe

Use proper markdown formatting for improved readability

Example structure:
# Modernization Changes

## Recipe: [Recipe Name]
[Recipe description]

## Recipe: [Another Recipe Name]
[Another recipe description]

...

By implementing these changes, we can enhance the clarity and usefulness of our automatically generated PRs, making them more informative for reviewers and maintainers.
Upstream changes
No response
Are you interested in contributing this feature?
No response

I think this is quiet important for the adoption and the sake of release notes of plugins

Fixed by https://github.com/jenkinsci/plugin-modernizer-tool/pull/223


Initial CosmosDb Connector #99

Adds enough plumbing to use the Core SQL API with CosmosDB via the (pre-release) V4 client via ConnectionStringManager

Does not include extensions for adding to DI
Should probably (also) support the v3 client prior to adding to 2.x if desired

/azp

/azp run steeltoe


Implemented smooth scrolling for mouse in editor.

Description of the Change
Since the program relies on web technologies to build its user interface, we came up with a simple solution of introducing a smooth transition when the scrolling action occurs.
The end goal was to make the scrolling transition last 0.2 seconds instead of being instant, leading to a smother user experience.
To achieve this goal, we wrote a css transition code in the style.less file.
Alternate Designs
The proposed version was selected because the issue can be solved with a simple css script.
Why Should This Be In Core?
Smooth scrolling makes scrolling feel more natural.
Benefits
Scrolling is more natural for mouse users.
Possible Drawbacks
No possible drawbacks were found.
Applicable Issues
https://github.com/atom/atom/issues/12848

I tested it and it does look good, but it doesn't sit well with other types of scrolling (dragging the minimap, using page up/ page down, etc). Is there a way to only apply the effect when scrolling with the mouse wheel?

What is the status of that pr? I know it's old. Trying to find a way to enable smooth scrolling in Atom and this is the only thing that popped up.


Add has and get methods back to TaggedCache #12272

Add back in removed get and has methods to Tagged Cache. This should fix issue #12272  and allow all keys to be flushed from cache when only calling one tag.

This breaks the cache events.
It's also doing pretty much exactly the same thing that the get and has methods are doing on the Repository class (which it inherits from), so I'm not sure how this would fix the issue.

Please send this to 5.1, in such a way that the events still work.


Server does not start up: Wait for another process to build up the cache...

We just tried to deploy openslides with docker-compose. Building the services works fine, and all services start running, but the server does not complete its startup. It hangs with the message
[...] Wait for another process to build up the cache...

This message looks like it is supposed to only show up if there are several server instances running, but we do only start one instance (to the best of our knowledge).
For reference, this is our docker-compose file. We have slightly modified it to suit our environment and deployment process.

 docker-compose.yml
# This configuration was created from a template file.  Before making changes,
# please make sure that you do not have a process in place that would override
# your changes in the future.  The accompanying .env file might be the correct
# place for customizations instead.
version: '3.4'

x-osserver:
  &default-osserver
  image: $DOCKER_REPO/openslides-server:latest
  networks:
    - front
    - back
    - mail
  volumes:
    - openslides_secrets:/run/secrets
  restart: always
x-osserver-env: &default-osserver-env
    AMOUNT_REPLICAS: 1
    AUTOUPDATE_DELAY: 1
    DEMO_USERS: ""
    CONNECTION_POOL_LIMIT: 100
    DATABASE_HOST: "pgbouncer"
    DATABASE_PASSWORD: $DATABASE_PASSWORD
    DATABASE_PORT: 5432
    DATABASE_USER: "openslides"
    DEFAULT_FROM_EMAIL<EMAIL_ADDRESS>    DJANGO_LOG_LEVEL: "INFO"
    EMAIL_HOST: "mailserver"
    EMAIL_HOST_PASSWORD: ""
    EMAIL_HOST_USER: ""
    EMAIL_PORT: 25
    ENABLE_ELECTRONIC_VOTING: "False"
    ENABLE_SAML: "False"
    INSTANCE_DOMAIN: "${HOSTNAME}"
    JITSI_DOMAIN: ""
    JITSI_ROOM_PASSWORD: ""
    JITSI_ROOM_NAME: ""
    OPENSLIDES_LOG_LEVEL: "INFO"
    REDIS_CHANNLES_HOST: "redis-channels"
    REDIS_CHANNLES_PORT: 6379
    REDIS_HOST: "redis"
    REDIS_PORT: 6379
    REDIS_SLAVE_HOST: "redis-slave"
    REDIS_SLAVE_PORT: 6379
    REDIS_SLAVE_WAIT_TIMEOUT: 10000
    RESET_PASSWORD_VERBOSE_ERRORS: "False"
x-pgnode: &default-pgnode
  image: $DOCKER_REPO/openslides-repmgr:latest
  networks:
    - dbnet
  labels:
    org.openslides.role: "postgres"
  restart: always
x-pgnode-env: &default-pgnode-env
  REPMGR_RECONNECT_ATTEMPTS: 30
  REPMGR_RECONNECT_INTERVAL: 10
  REPMGR_WAL_ARCHIVE: "on"

services:
  server:
    << : *default-osserver
    # Below is the default command.  You can uncomment it to override the
    # number of workers, for example:
    # command: "gunicorn -w 8 --preload -b <IP_ADDRESS>:8000
    #   -k uvicorn.workers.UvicornWorker openslides.asgi:application"
    #
    # Uncomment the following line to use daphne instead of gunicorn:
    # command: "daphne -b <IP_ADDRESS> -p 8000 openslides.asgi:application"
    depends_on:
      - server-setup
    environment:
      << : *default-osserver-env

  server-setup:
    << : *default-osserver
    entrypoint: /usr/local/sbin/entrypoint-db-setup
    environment:
      << : *default-osserver-env

    depends_on:
      - pgbouncer
      - redis
      - redis-slave
      - redis-channels

  client:
    image: $DOCKER_REPO/openslides-client:latest
    restart: always
    depends_on:
      - server
    networks:
      - front
      - web
    labels:
      - "traefik.enable=true"
      - "traefik.backend=${HOSTNAME}"
      - "traefik.docker.network=web"
      - "traefik.main.frontend.rule=Host:${HOSTNAME}"
      - "traefik.main.port=80"

  pgnode1:
    << : *default-pgnode
    environment:
      << : *default-pgnode-env
      REPMGR_NODE_ID: 1
      REPMGR_PRIMARY: # This is the primary
    volumes:
      - "dbdata1:/var/lib/postgresql"

  pgbouncer:
    environment:
      - PG_NODE_LIST=pgnode1
    image: $DOCKER_REPO/openslides-pgbouncer:latest
    restart: always
    networks:
      back:
        aliases:
          - db
          - postgres
      dbnet:
  postfix:
    image: $DOCKER_REPO/openslides-postfix:latest
    restart: always
    environment:
      MYHOSTNAME: "localhost"
      RELAYHOST: "localhost"
    networks:
      - back
  redis:
    image: redis:alpine
    restart: always
    networks:
      back:
        aliases:
          - rediscache
  redis-slave:
    image: redis:alpine
    restart: always
    command: ["redis-server", "--save", "", "--slaveof", "redis", "6379"]
    depends_on:
      - redis
    networks:
      back:
        aliases:
          - rediscache-slave

  redis-channels:
    image: redis:alpine
    restart: always
    networks:
      back:
  media:
    image: $DOCKER_REPO/openslides-media-service:latest
    environment:
      - CHECK_REQUEST_URL=server:8000/check-media/
    restart: always
    networks:
      front:
      back:
    # Override command to run more workers per task
    # command: ["gunicorn", "-w", "4", "--preload", "-b",
    #   "<IP_ADDRESS>:8000", "src.mediaserver:app"]

volumes:
  dbdata1:
  openslides_secrets:
    external: true

networks:
  front:
  back:
  dbnet:
  web:
    external: true
  mail:
    external: true

secrets:
  django:
    file: ./secrets/django.env

# vim: set: sw=2 et:

Hi,
First I wonder, taht the depends_on is too less populated. E.g. the server should depend on redis. This may cause problems. Regarding the message: Something set a special key in redis. Maybe you aborted another startup and redis was not cleared. Please try the following: Stop the server and redis (incl. slaves, ..) so that redis is empty when starting up. This can solve the problem. Or do a docker-compose exec redis redis-cli flushall and docker-compose restart server. Maybe this helps.

can't reproduce it by myself anymore.


Supports avail-change Notification

Hawkular-services now listen for the AvailChange Notification.
It will react on this notification for Server types, creating an event to inform the avail change (e.g. to track Loss of connectivity).
Still needs to plug something onto BackfillCacheManager to close a hole, as the server won't notify when is down because the agent will also be down.

Can one of the admins verify this patch?

@jshaughn I added support on the backfill mechanism for the server avail.
I moved logic to do specific processing of the Notifications on a MIQEventUtil class that can be accessed from the Backfill code without needing to know about Notifications.
I still need to figure a way to send an Avail UP. I suppose services would need to do that, because the agent can't do that as the current availchange mechanism won't detect any aviail change on a local server from down to up.
Please let me know your thoughts on that.

still need to figure a way to send an Avail UP...

Perhaps @jmazzitelli can offer his opinion on what would be the easiest approach.  I was thinking that in the same way you don't report the initial avail for all other resources (the cached current avail=null) that you do report UP avail for the host server avail metric, although I'm not totally sure how you would know which metric represents the host server.

I tested workflow and is working well for services + java agent.
@jshaughn  Can I have another review round?
This depends on https://github.com/hawkular/hawkular-commons/pull/108 and https://github.com/hawkular/hawkular-agent/pull/347

Updated commons with src-dep, not sure why that default is failing.

I also don't know why the "default" build is failing and I can't visit the details site.  You did create a hawkular jira for this, if I remember.  Can you update the PR title to mention the jira and/or make sure this PR is linked to the Jira?  If you are confident I will merge the PR despite that off "default" build failure.

Just linked.
I'm not sure if the src-dep made it fail.
It works well on my local box and passes the tests, so we can go ahead and merge.

OK, merging despite the weird "default" build failure for which we can't even see the details.  If there is an issue we'll see it soon enough and then fix it.


Possible fix for #914 compactCells failing when passed all res 1 cells

Incorporates #914

coverage: 98.715% (-0.1%) from 98.825%
when pulling 663e02a305e8adf211ef4f0a312e43235e5570c0 on isaacbrodsky:compact_all_res1_tes-proposed-fix
into b355c9d825428b8f08636ffa7250422f52c2f6e0 on uber:master.


CORE-8226: Prevent blank client request ID by adding regex validation

Flow Endpoint - Cordapp can call the endpoint with a blank ClientRequestId, when it should throw an error. This PR adds regex validation to prevent blank ClientRequestId from being accepted and instead throw an exception.
Testing
Manual testing by running the combined worker and smoketests. Replicated QA's issue on swagger and got the 202 response.
Tested solution in same way.

When manually testing in Swagger I get:
Error: Server Error
Response body
Download
{
"title": "Illegal argument occurred.",
"status": 500,
"details": {
"cause": "IllegalArgumentException",
"reason": "Supplied Client Request ID "" is invalid, it must conform to the pattern "[-._A-Za-z0-9]{1,250}"."
}
}
-let me know if you're happy with the Exception I've gone with, and likely the message could be more user-friendly?


📎  ImplementuseNewForBuiltins - unicorn/new-for-builtins

Description
Implement unicorn/new-for-builtins

I will try to work on it myself.

Note that new Symbol and new BigInt are already forbidden by noInvalidNewBuiltin.
Thus, this rule should not cover these two cases.
Also, I propose a more consistent name with our existing rules: useConsistentNewBuiltin.

Why "Consistent" @Conaclos ? I think useNewBuiltin is enough.

Taking a closer look at the rule, the following constructors cannot be instantiated without new.

ArrayBuffer
BigInt64Array
BigUint64Array
DataView
FinalizationRegistry
Float32Array
Float64Array
Int16Array
Int32Array
Int8Array
Promise
Proxy
Set
SharedArrayBuffer
Uint16Array
Uint32Array
Uint8Array
Uint8ClampedArray
WeakMap
WeakRef
WeakSet

Since we have noInvalidNewBuiltin, we should certainly have something like noInvalidBuiltinInstatiation.
However, I doubt the usefulness of having three rules.
I think we could deprecate noInvalidNewBuiltin and simply handle Symbol and BigInt in useConsistentNewBuiltin.
Any opinion?

Why "Consistent" @Conaclos ? I think useNewBuiltin is enough.

Became I was thinking about useConsistentBuiltinInstatiation.
You are right we should certainly use useConsistentBuiltinInstatiation or useNewBuiltin.
Any preference?

Alternatively we could deprecate noInvalidNewBuiltin and create two rules: useConsistentBuiltinInstatiation and noInvalidBuiltinInstatiation.

I slightly prefer useConsistentBuiltinInstatiation even if it is bit long, since rule as it currently is dissallows use of new keyword in some cases, so it is bit more clear. noInvalidNewBuiltin deprecation also makes sense to me since they effetively do the same thing, but I will wait for confirmation before renaming things.


⚠ Site ANTS has degraded performance

In 09b4d0e, Site ANTS (https://ants.gouv.fr) experienced degraded performance:

HTTP code: 200
Response time: 4615 ms

Resolved: Site ANTS performance has improved in c866761.


Update issue templates

Let's have them upload a openapi spec file as well.

Let's have them upload a openapi spec file as well.

@seriouslag I have updated it.


using widthOfSiblingSlidePreview property not work properly

version 2.15.0
Hi i have an issue when i using widthOfSiblingSlidePreview, the expect behaviour is that i will see the next slide (element) but instead of it i see the last slide.
i attached image of the issue
The arrow in the image is point to the last image in the container the issue is that all image get the same translate3d x parameter...

@voronianski I'm having the same issue. the widthOfSiblingSlidePreview config option does not work as advertised. I'll open a PR if I can come up with the solution.

@Cyberlane I noticed you created the original PR maybe you have some advice for us.

I have the same issue,. Has someone solved?

I created the original functionality for the widthOfSiblingSlidePreview, and it was to attempt at quickly add a feature I wanted/needed at the time. It worked initially, but I quickly found a number of edge cases which caused further problems, much like the ones mentioned in this issue.
Ultimately I wrote something new from scratch for what I originally needed.
I can take a look and see if there's a quick fix for this specific problem.

@adambatso could you please provide an example of how to achieve your problem?
Currently I am trying this, and it works:

      var elem = document.getElementById('mySwipe');
      window.mySwipe = Swipe(elem, {
        widthOfSiblingSlidePreview: '50px',
        continuous: false,
      });

I am however aware that if you remove continuous or set it to true, that everything looks awful. Is that perhaps the bug you have right now?

I'm facing the same issue. Has anyone found the solution?

@gauravAtProptiger see my previous comment. Is that the same problem you face? If not, could you please provide an reproducible example of the issue, and I would be happy to look into it for you. :)

@Cyberlane  Yes, I am facing the same issue.
Setting the above mentioned configuration, the preview for the next slide is shown as the last slide that exist in list.
`

Swipe 2

@gauravAtProptiger I am working on a fix, if I pushed it to another branch, would you be willing to test it? There are a number of things I would ultimately like to change in the code for this library, but since I am not the owner, I will just do some small patches for now.

I have created a PR which should solve this issue here => #33
If somebody is willing to test it to confirm, I would be happy to merge it into master.


usemtl doesn't parse string correctly

Hi,
I discovered a problem with the usemtl parsing when there's whitespace before or after the material string name.
Example .OBJ:
.....
usemtl  1
.....
Example .MTL:
.....
newmtl 1
.....
This was easy to solve by using the parseString() function instead of the stringstream approach.
Thanks!
Ben
tinyobjloader.patch.txt

I guess that I should open a PR but hopefully this patch is simple enough...

Thanks! Fixed!
Also added a regression test : https://github.com/syoyo/tinyobjloader/blob/master/tests/tester.cc#L1255


BADLAND: Game of the Year Edition [PCSB00669]

Game summary

Game name: BADLAND: Game of the Year Edition
Game serial: PCSB00669

Vita3K summary

Version: v0.1
Build number: 239-3137d1f
Commit hash: https://github.com/vita3k/vita3k/commit/3137d1f

Test environment summary

Tested by: HorrorTroll
OS: Windows 8.1
CPU: Intel Core i5-4200H
GPU: Nvidia Geforce GTX 950M
RAM: 6 GB DDR3

Issues

Softlock
Screenshots

Log
vita3k.log
Recommended labels

Softlock bug

Vita3K summary

Version: v0.1
Build number: 1737
Commit hash: https://github.com/vita3k/vita3k/commit/ec3eaab4

Log
vita3k.log

Vita3K v0.1.5 2609-653b0fa1
show the placeholder screen
then nothing but a yellow screen
unicorn error
vita3k.log

Vita3K summary

Version: v0.1.6
Build number: 2842
Commit hash: https://github.com/vita3k/vita3k/commit/0a6ef4d9
CPU backend: Dynarmic

Test environment summary

Tested by: othmanead
OS: Windows 11
CPU: Intel i5-10300H
GPU: RTX 3060 mobile
RAM: 16 GB

Issue:
Game plays with minor graphical bugs
Screenshots:

Log
vita3k.log
Recommended labels

Playable


COPR package

Thanks for the good work. Kinda saved my arse when I found out Namesilo does not support NS records on a subdomain, which precludes spinning up a local dnsmasq instance for the challenge which is what I normally do.
Anyways, personally I do not like installing software unmanaged, so I packaged this thing on Fedora COPR for Fedora and RHEL over here. Makes life a bit easier for me and other people on these OSes.
This issue is just to let you know; feel free to put this info in README (or not, it's your project 😁).

@cyqsimon That is very cool. Thanks.
Would you mind submitting a PR with the info added to the README?


bind deconz to specific address instead <IP_ADDRESS>

I dont know if this may be a deconz issue so dont judge me if I'm wrong posting here.
Is it possible to tell the container in host mode to only bind to a specific ip address instead of <IP_ADDRESS>:80 ? I need another container running in host mode with the same port.
Thought I can add a secondary ip to the host and then assign the ips to each container but this fails due to the fact deconz consumes every ip available on the host.
netstat -tulpn | grep :80
tcp        0      0 <IP_ADDRESS>:80              <IP_ADDRESS>:*               LISTEN      119287/deCONZ

You should be able to use the env vars DECONZ_WEB_PORT and DECONZ_WS_PORT to change the ports that deconz binds to, which may indirectly solve your problem.


Support columnar processing for mapInArrow[databricks]

closes https://github.com/NVIDIA/spark-rapids/issues/6313
This PR adds the columnar support for the new API mapInArrow which is introduced in Spark 3.3.0.
Performance

About 6.8 GB Parquet data in local files.
CPU 12 cores, and one GPU (Titan V, with 12GB memory)

CPU Read + CPU mapInArrow
GPU Read + CPU mapInArrow
GPU Read + GPU mapInArrow

97.20
91.36
81.67

Signed-off-by: Liangcai Li<EMAIL_ADDRESS>

build

build

build

build

build

build

The latest failure is related to https://github.com/NVIDIA/spark-rapids/issues/6869

build


feat: pass accessibility props to MotiPressable

This adds support for Pressable accessibility props on MotiPressable,
It passes the props directly instead of spreading all props from Pressable.
Not sure if selectively excluding specific props from pressable would be a viable/cleaner approach

I think this looks good to me. I'd prefer to do it this way and not spread all of the props.

Can you run yarn prepare to make sure the types pass?

Published to canary:
yarn add @motify/interactions@canary moti@canary

thanks! sorry just checked messages, will keep that in mind for next time cheers


Python and dependencies update

I have an Apple M1 MacBook Pro, which cannot natively run Python 3.7. The lowest supported version is 3.9.1. However, TensorFlow 2.3 will not work with Python 3.9 (and possibly some other dependencies neither). Could you update the Python and dependencies requirements to higher versions if possible?

Hi Andreas, I am on vacation now, and I will update the versions later.

Please see the PT1.2 https://github.com/DUTIR-BioNLP/PhenoTagger-Updates
It can run with Python 3.10


Switching tabs does not update the main view properly

Screencast from 2023-12-13 16-59-43.webm

Maybe it is related to the STEP file since it does not happen with 2 JCAD files
https://github.com/jupytercad/jupytercad/assets/4451292/c1f7b396-11ab-4c05-b19d-248e6ff352bb

My first guess is that whenever a DISPLAY_SHAPE message is sent from the worker, all viewers are receiving the message

ah yes, I was too lazy to switch to main. Maybe we can wait for #230, I'm finishing it soon, and hopefully, this issue can be fixed.

it was done already :D


v0.16.1

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

thanks so much.

A small number of builds didn't go through:

https://app.travis-ci.com/github/conda-forge/watchfiles-feedstock/builds/253577518

I guess the builds don't have to work every time on every platform/python, but they need to work at least once for e.g. https://github.com/conda-forge/uvicorn-feedstock/issues/91

Thanks for reporting, they now all passed: https://app.travis-ci.com/github/conda-forge/watchfiles-feedstock/builds/253577518.


Provide option to require leading space for Layout/LineContinuationLeadingSpace

Is your feature request related to a problem? Please describe.
When using line continuations, I put space at the beginning of the next line on purpose, to make the spaces more visible. This conflicts with the new Layout/LineContinuationLeadingSpace cop.
Describe the solution you'd like
I'd like to configure Layout/LineContinuationLeadingSpace to require spaces at the beginning of the line instead of at the end of the previous one.
Describe alternatives you've considered
I can disable Layout/LineContinuationLeadingSpace. I can also let Layout/LineContinuationLeadingSpace autocorrect these offenses, but it will make the presence of spaces less salient. It may also push some lines over the maximum line length.
Additional context
N/A

This seems to be in the works. See #10762.


Remove overwriting a default rule from .prettierrc

According to http://json.schemastore.org/prettierrc, printWidth default is already 80.

Let's keep it to be explicit. The examples folder rc file use a different width.


[Feature] Add linter for commit convention

Is your feature request related to a problem? Please describe.

Now we have our commit convention, we should add a linter and a ci step for it.
ref: https://github.com/conventional-changelog/commitlint
Describe the solution you'd like

Describe alternatives you've considered

Additional context

/assign


Filenames that differ only by case cause generated projects to not load

So Linux can have files that differ only by case in the same directory. NTFS supports this as well, but if you generate a project file with these scripts that has files like that VS will not like it.
One way to fix the output generated by the scripts is with uniq since the generated lines are in sequence.
uniq -i linux.vcxproj linux-dedupe.vcxproj
uniq -i linux.vcxproj.filters linux-dedupe.vcxproj.filters
I may update the script to simply generate a temporary file then run this as the last step and delete the temp file. Not sure yet.

After generating the .filters file, get this error message:
The item metadata "%(FullPath)" cannot be applied to the path "Debug|ARM".Illegal characters in path.
Works fine without the filters.


[BUG] SharedBaseline on_dim default value should be -1

Describe the bug
I think in the SharedBaseline, on_dim here default value should be -1
If not, it raises 👇

To Reproduce
Simply change the baseline and run
Checklist

[x] I have checked that there is no similar issue in the repo (required)
[x] I have provided a minimal working example to reproduce the bug (required)

How are you changing the baseline?
If you use SharedBaseline, you should also use some inference techniques (e.g. multistarts for POMO or augmentation for SymNCO), so if you just use the default model with baseline="shared", it will not work. This is why the reward is always converted to [batch, num_pomo/num_aug]

Okay I see. But actually I wanted to use just the mean of reward as a naive baseline.
So in this case, should I implement this locally?

I see! Then feel free to submit a PR ;)


Allow User Adjustable Viewport Size

Can the user adjust the viewport size themselves (by dragging the viewport edges) when using Croppie?  Do we have to provide a fixed value like so:
croppieInstance = $('.conainer', $('div.e')).croppie({
	url: src + '?' + d.getTime(),
	enableOrientation: true,
	boundary: { width: 600, height: 450},
	viewport: { width: 400, height: 300, type: 'square' },
});

I have the zoom feature enabled, but there are many times when I as a user want to adjust the viewport area myself by enlarging or decreasing the edges of the crop area.  Is this possible?

Nope.  It's not possible.  Although I'll keep this issue open because I can see this being useful.

We are also interested in something like this, although one would have to be able to constrain which dimension can be adjusted (eg. locking width, but let height be adjustable). In this case for us the boundary size would be updated proportionally.
We will be putting some effort into getting this feature implemented in Croppie itself rather than find a way around it, hopefully it'll be interesting to maintainers to merge into core once done.


Is it possible to add support for LittleVGL？

Hi，
Is it possible to add support for LittleVGL？
"LittleVGL" is also written in C language.

It's a great GUI library, though more geared towards higher end MCUs. I assume that what you are asking for is support within the GUIslice Builder. I will transfer your issue to the Builder repository so that the author can comment, though my expectation is that it would require a fundamental rewrite.


App Transport Security blocking loading images

Even though this is an https connection to itunes, I am still getting this error. And the images doesn't load. However, if I set "NSAllowsArbitraryLoads" to "YES" it works. I don't want to set it "YES" because of obvious security issues.
How do I get around this?
thanks

Hi,
you are right, all the images in the iTunes API have http url:
  "results": [
    {
      "screenshotUrls": [
        "http://a1.mzstatic.com/us/r30/Purple5/v4/5f/1f/6f/5f1f6f60-822f-eada-8378-62e0a1000909/screen1136x1136.jpeg",
        "http://a4.mzstatic.com/us/r30/Purple3/v4/d6/59/d8/d659d88c-ca02-1599-6284-f985184fb08c/screen1136x1136.jpeg",
        "http://a1.mzstatic.com/us/r30/Purple3/v4/56/39/41/5639412f-ab11-144d-e519-57ecf33c37c0/screen1136x1136.jpeg",
        "http://a1.mzstatic.com/us/r30/Purple3/v4/d6/4e/1d/d64e1d95-f61f-b97b-6c2a-ab10e9b5f2dc/screen1136x1136.jpeg"
      ],

Unfortunately, their HTTPs version do not have a valid certificate so it would fail as well
https://a1.mzstatic.com/us/r30/Purple5/v4/5f/1f/6f/5f1f6f60-822f-eada-8378-62e0a1000909/screen1136x1136.jpeg
You could disable ATS only for this domain mzstatic.com but it is not guaranteed that those images will be hosted there all the time
    <key>NSAppTransportSecurity</key>
    <dict>
        <key>NSExceptionDomains</key>
        <dict>
            <key>mzstatic.com</key>
            <dict>
                <key>NSExceptionAllowsInsecureHTTPLoads</key>
                <true/>
                <key>NSIncludesSubdomains</key>
                <true/>
            </dict>
        </dict>
    </dict>

So I think the only way is to enable NSAllowsArbitraryLoads
cheers

cool... I tried my own implementation and had same issue... Now I am loading it through my app website so I still have control over my security front... :)

cool


Contacts without prefecture data from LOTW are skipped

Hello and thank you for this tool!  When I started working on WAJA nothing like this existed, so I resorted to using excel documents. Dumping my data from LOTW and inputting it into this tool resulted in me missing 12 prefectures.  However, my Excel document I've been tracking this in says I'm missing 9 prefectures.  I believe the problem lies within the LOTW data and if it has prefecture listed or not.  I think rather than skipping over data that doesn't have prefecture listed the tool should output an "error log" or notify the user that some contacts could not be mapped to a prefecture and they should manually investigate - otherwise some people might have contacts in prefectures they already have and not know it.
I'll attach screenshots that hopefully clarify this problem.

Looking into this a bit more, the adi file from LOTW on contacts that are not mapping correctly have no "State" data.  Screenshot attached.  I'm not sure what other peoples logs look like, but my ADI file from LOTW has about 1/3 of the contacts with this column blank.

The issue here is that the WAJA award requires proof of contact.  If the prefecture is not listed in LoTW then it cannot be verified.  The best solution here is to reach out to the JA station(s) and ask them to re-upload their log with the prefecture.  A list of JA QSL's without prefectures could be generated by this script for this purpose.
The JARL rules (#3) state, "Each list must be accompanied by a statement of the applicant himself/herself making an oath that the QSL cards of the contacts listed are in the possession of the applicant and that the items of the cards are correctly listed."   It is not a stretch to believe that the same would apply to LoTW QSL records, so having a record w/the necessary data (prefecture) listed seems imperative.

I agree then the script might make a list of contacts without prefectures so the user is aware they might want to contact those stations and have the contact reuploaded (or manually investigated at least to see if they need their listed prefecture).  In my case it makes a big difference as I have 3 contacts without prefectures that I was (incorrectly) thinking I could use their home prefecture listed on their address as proof of contact of that prefecture.

Will add a listing of QSL records that are missing prefecture; callsign and date of QSO will be saved to missing.csv.


Security warnings on macOS

Hi!  Thanks for making ruff, it's a great tool.  I also appreciate the regular development and updates, and I've configured a hook to automatically keep some of my repositories on the latest versions of ruff and related tools.
However, I've found that Apple's "Gatekeeper" security is getting more and more onerous, requiring me to re-authorize ruff on every repo, every time it gets an update.  They recently took away the click one button way to do this, now I have to go dig in the system settings panels every time.  Is there a way to use some kind of Astral-signed version of Ruff with pre-commit that won't give me all these security warnings?  Or is this inconvenience just the price I pay to live in Apple's world?

Thanks for the kind words.
Hmm interesting. I've never seen a security warning when running ruff as part of pre-commit. Can you tell me more on how you run ruff?

I created a small placebo repository to replicate my problem:  https://github.com/jpn--/placebo
I can check out this repo, run "pre-commit install" on it to activate the hooks, then when I try to commit a change to Python code in the repo, either using a GUI or even just git commit on the command line I get this system popup:

I actually see the pop-up twice, I presume once for the ruff and a second time for the ruff-format in my .pre-commit-config.yaml.
After triggering this warning, if I go to  > System Preferences > Privacy & Security, I can see this:

If I click here on "Allow Anyway" and re-try the commit, it will work, but only until I update the version of ruff listed in the .pre-commit-config.yaml, or reset my pre-commit cache, at which time the whole thing starts again.
I am running macOS Sequoia Version 15.1.1 (24B91).

Thanks for creating a repository! I'm currently on 15.1.1 macOS version and it seems to be running fine on my machine.
Can you tell us how did you install pre-commit? Does it occur only when running Ruff via pre-commit or does invoking ruff from the command-line also gives you this popup?

I have installed pre-commit (and ruff, for manual use) via homebrew.  I just updated homebrew itself, and both ruff and pre-commit to the latest versions, which has not fixed my issue.  I don't get the security warning message when I invoke the regular ruff from the command line myself, which is /opt/homebrew/bin/ruff.  I do get the security warning message when I explicitly run the copy of ruff that pre-commit has cached: ~/.cache/pre-commit/repoxlgredg0/py_env-python3.13/bin/ruff.

when I explicitly run the copy of ruff that pre-commit has cached: ~/.cache/pre-commit/repoxlgredg0/py_env-python3.13/bin/ruff.

Do I understand this correctly that you manually run ~/.cache/pre-commit/repoxlgredg0/py_env-python3.13/bin/ruff? You don't run ruff using pre-commit?

I definitely do not normally manually run ~/.cache/pre-commit/repoxlgredg0/py_env-python3.13/bin/ruff, I let pre-commit do that for me, which is kind of the point of pre-commit, right?  An hour ago just before my previous message was the first time I ever tried to do that manual command.  I was just noting that if I manually run that copy of Ruff, I get the same error.


feature: generate docker image for other architecture

Is your feature request related to a problem? Please describe
Currently, we are only generating production docker image for amd64, we need to build for other platforms
Describe the solution you'd like
Modify workflow to generate images for multiple platform
Describe alternatives you've considered
No response
Additional context
No response
Are you working on this?
Yes

The current issue has been fixed. But the docker publish has been slowed down too much after adding arm64 support. Issue #50


robot_state_publisher: 1.10.4-0 in 'jade/distribution.yaml' [bloom]

Increasing version of package(s) in repository robot_state_publisher to 1.10.4-0:

upstream repository: https://github.com/ros/robot_state_publisher.git
release repository: https://github.com/ros-gbp/robot_state_publisher-release.git
distro file: jade/distribution.yaml
bloom version: 0.5.19
previous version for package: null

robot_state_publisher
* Merge pull request #21 <https://github.com/ros/robot_state_publisher/issues/21> from rcodddow/patch-1
* Fix for joint transforms not being published anymore after a clock reset (e.g. when playing a bagfile and looping)
* Contributors: Ioan A Sucan, Robert Codd-Downey, Timm Linder

The PR reverts rviz. Please either fix the branch of this PR or create a new PR.

Updated.


Messages sent to self have incorrect header

For example, if a user sends a message to him or herself, the header in the stream view is blank (instead of displaying the user's name)

Can I work on this?

Should a user even be able to send a message to himself?

I think so. Could be useful for notes and testing. In any case, the server
and web app support it so we should as well
On Thu, Jan 26, 2017 at 12:07 PM Boris Yankov<EMAIL_ADDRESS>wrote:

Should a user even be able to send a message to himself?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/zulip/zulip-mobile/issues/231#issuecomment-275498596,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEczZYYCkpz8aUXQJNTg8oHWHZ9QCTHOks5rWPz9gaJpZM4LvA-n
.

Yes, we definitely support this.  Slack supports it as well for the same reasons -- it's super nice for testing out how the features work without bothering anyone.

I'll work on it, because we are missing to put in in the store correctly. Didn't think of that case.


Added Information to space-complexity.md

Added information to space-complexity.md.
Description of work:

Explanation of concept & need
Process of calculation
Explanation of process with examples
Standard space complexities of common algorithms

References used :

Sartaj & Sahani DSA book
geekforgeeks
stackoverflow
baeldong.com
studytonight.com

I've made the changes. Will you please review it once again?

Thank you for your contribution! 🎈🎉✨

Thank you so much @Utkarsh1504 And also thank you for guiding me throughout this process and giving me chance to contribute even when I faced issues.


Add Rotating Empty State Messages to ShellMate

Mary's text about it:

Sam and I had some fun with this but we want to add an empty state message to ShellMate. This would be anytime a new ShellMate window is opened, before any suggestions have been generated.
We came up with some funny/entertaining quotes that highlight creative ways to use terminal and shellmate. Each time shellmate is re-opened we'd rotate through these quotes
You can contribute here :sweat_smile:

FYI - this is the same ticket as #115

Closing duplicate


Fix breaking spec tests

These were never runnable on OSX. Adding osfamily to specs means you can test on a mac.

:+1: looks fine! merge away


Add Starlark Language Server

Language server
Starlark Language Server
Requested feature
It would be great to also have the starlark language server available for bazel files.
https://docs.stack.build/docs/vscode/starlark-language-server
There also is an cli to start the server with
bzl lsp serve

If someone can give me hints on how to do this, I would try it myself.
Other clients which have this feature
vscode

I just created a lua/lspconfig/server_configurations/starlark.lua
local util = require 'lspconfig.util'

return {
  default_config = {
    cmd = { 'bzl', 'lsp', 'serve' },
    filetypes = { 'bzl'},
    root_dir = util.find_git_ancestor,
  },
  docs = {
    description = [[
https://docs.stack.build/docs/cli/installation

https://docs.stack.build/docs/vscode/starlark-language-server
]],
    default_config = {
      root_dir = [[root_pattern(".git")]],
    },
  },
}

The server only starts once I do :LspStop and then some features like go to definition works. Unfortunately many of the lsp functionalities are behind a paywall... bummer

closing due to server restricting most functionality behind a subscription fee.


Error: yasm-tool[core] is only supported on 'native & !uwp'

host:
win10 x64; vs 2019 （MSVC140,  141）
Because of a bug with gmp in vcpkg for windows, i need to install yasm-tool in 32 bits to be able to correctly build gmp 64bits
but it shows the error logs as the title says.

@szhllyyl, what triplets do you install?

@szhllyyl, what triplets do you install?
yasm-tool:x86-windows

@szhllyyl, you can use overlay ports and remove 'native&' from https://github.com/microsoft/vcpkg/blob/master/ports/yasm-tool/vcpkg.json#L6, then try again and see if it works for you.

I removed ’native&‘ and it works! thanks for your reply!

Note that you can build "yasm-tool:x86-windows" when the host triplet is x86-windows. Native means target == host.
However the default vcpkg triplets on x64 machines are host x64-windows, target x86-windows...


Assigning multiple IP addresses the same node causes Kubeshark to be confused

Not quite sure if this is a bug or a feature request, but I am running into an issue where Kubeshark thinks that I have multiple nodes where in fact I have only one. This is a single MicroK8s node running on an Ubuntu machine.
The reason for this is that I have assigned multiple IPv4 addresses to the same network interface of the machine. This is being done for an unrelated reason. However having done this, Kubeshark thinks I have more than one node. See the image below:

I then get this license message, and cannot use it any more:

Is there anything that I can do to fix this easily? I want to specify to only use a single IP address on a node and not all of them. I can't find anything in the docs that jumps out at me.
I am using:
MicroK8s v1.30.3 revision 7040
Kubeshark: v52.3.69

@Smith334 thanks for reporting this. This is an interesting edge use/edge case. To get you going, please sign up for an account and we'l provision a license for you to use as if you were using a multi-node cluster.


Brief description of change

Added my info

Thank you for your contribution! Merging the pull request now.


Fix timestamp of flows

Describe the bug
Currently the timestamp of Packetparser and Dropreason generated flows are wrong, and not current. They need to be set in userspace and not parsed from bpf events.
Ref: https://github.com/microsoft/retina/blob/29863599d5fa295fd59ae1baec21d108c5b7c907/pkg/plugin/packetparser/packetparser_linux.go#L537
To Reproduce
Steps to reproduce the behavior:

Run Retina in advanced-local-context mode
Check flows generated by Packetparser plugin

Expected behavior
Timestamp should be current.
Platform (please complete the following information):

OS: Linux
Kubernetes Version: N/A
Host: Any Linux hosts
Retina Version: v0.0.4

This might be the wrong function to use for current timestamp: per the docs:
"Return the time elapsed since system boot, in nanoseconds"
Ref:
https://man7.org/linux/man-pages/man7/bpf-helpers.7.html

@anubhabMajumdar can you clarify - these timestamps should be in wall-time? Should all of our BPF event timestamps be in wall-time? AFAICT they are all currently relative to boot-time, because that's all you can get in eBPF (unless the code was written for 6.2+ kernels, which seems unlikely, and TAI has problems still)...
Assuming the answer to that is yes -
Indeed, bpf_ktime_get_ns() doesn't seem right. Emphasis mine:

Return the time elapsed since system boot, in
nanoseconds.  Does not include time the system was
suspended.

This means we're always counting up in relative time-since-boot, BUT we stop counting if the system is suspended. Consistently adjusting from this value to wall-time would be very challenging.
I think the immediate fix here could be to change over to bpf_ktime_get_boot_ns:

Return the time elapsed since system boot, in
nanoseconds.  Does include the time the system was
suspended.

Then, the first time we process an event through the plugin, we can calculate the UTC boot time, and on all subsequent events we can offset the timestamp by this value to locate the event in wall-time.
Researching this, I also found bpf_ktime_get_tai_ns:

A nonsettable system-wide clock derived from
wall-clock time but ignoring leap seconds.  This
clock does not experience discontinuities and
backwards jumps caused by NTP inserting leap
seconds as CLOCK_REALTIME does.

But I don't think it is usable here for a couple of reasons:

This is only available in Linux 6.2+, which would be a huge jump up from our current back-compat expectation of 5.4+.
TAI is skewed from UTC by 37 seconds and will continue to drift.
Although it would be easy to convert back to UTC (for now), TAI isn't in the TZDB so we would need to do it manually and the skew would depend on the date. I don't want to manage that.
We have dependencies on other eBPF plugins like inspektor-gadget which generate events in boot-time, so our data pipeline would need to understand which plugins are TAI and only need UTC offsetting, and  which are boot-time and need to be replaced.

We might already have a minimum kernel requirement of 5.11 based on https://github.com/microsoft/retina/blob/2a7418eee1b5533d99a72d8a416e1dda45a403f8/pkg/plugin/lib/common/libbpf/_src/bpf.c#L103


keep state on zustand first, then push to prisma every ~30s

too many api calls to server especially after hire manager.
dont need to push to server on every updated!
instead of pushing to db for every single action (which is currently 10++ actions / second).

fixed with latest push.
some minor bugs caused by this,
e.g. state coins could > manager cost, allowing user to buy the manager, but db have not update new coins and ends up with negative coins for a while.
however, with OfflineProfits auto added everytime user goes to /game page, most of these small discrepancies between the db sync (30s) are fixed on update, and did not seem to cause any notable UX issues during first 5 mins of gameplay.


[libs/movex] Fix linting Errors in File MemoryMovexStore.spec.ts

Where
libs/movex/src/lib/movex-store/MemoryMovexStore.spec.ts
What
When you run npx nx lint movex --quiet you get some errors as below:

Why is it important
Because in the close future, the linting command will be part of the Github Actions
Definition of done

running npx nx lint movex --quiet will not return any more errors for this file
tsc passes npx nx tsc movex
all tests pass yarn test
no unnecessary code is introduced

Could you please assign this to me


Suggestion

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
Hii, your portfolio is great , but , if you add a toggle button. I will look so nice.

Hi Sajana, May you please explain toggle button of what?

Hii Vinay,
Your Portfolio is great. If you add toggle button to your website. It will
look so nice.
Toggle button:- It allows the user to change a setting between two states.
If you want to change your website into dark and whatever mode you want to
use just by clicking one button,this toggle button help you.
Don't worry, this button  will also keep your current background theme as
it look.
If you want I will do this for you.
I will generate a pull -request for this.
Thanks a lot for your response.
Thanking you.
Regards,
Sanjana Singh.
On Wed, 22 Sep, 2021, 9:46 am Vinay Somawat, @.***>
wrote:

Hi Sajana, May you please explain toggle button of what?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/vinaysomawat/vinaysomawat.github.io/issues/31#issuecomment-924568981,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AS5XTWTFLIAO26NUG4YTHWLUDFKCXANCNFSM5DXQXYVQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Hi Sanjana,
Please go ahead! That would be a great enhancement to this.
Thanks.

Hii Vinay,
Thank you so much for this.
Soon I will do this work.
Soon I will generate a pull request for you.
Thanking you.
Regrads
Sanjana Singh.
On Wed, 22 Sep, 2021, 2:01 pm Vinay Somawat, @.***>
wrote:

Hi Sanjana,
Please go ahead! That would be a great enhancement to this.
Thanks.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/vinaysomawat/vinaysomawat.github.io/issues/31#issuecomment-924705022,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AS5XTWWLKTHJRQSFOTNPURDUDGH5BANCNFSM5DXQXYVQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Added dark mode enhancement. Closing the issue for the same.


[Fix #9771] Fix a false positive for Style/TrivialAccessors

Fixes #9771.
This PR fixes a false positive for Style/TrivialAccessors when defining a method that is not a writer.

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

I have the vague memory that this was by design, so people would uncover "accidental" accessors, regardless of the name. I also thought we had some configuration about the exactness of the name match, but I don't see it. I wonder if I'm thinking of some similar cop. :-)

Ah, actually it seems we didn't include examples of the config options in the docs, which is why I didn't see them at first. I was thinking of ExactNameMatch.


Access to another modules test classes with idea-Plugin in multi-module build

We generate our IntellJ project configuration using the idea Plugin in our multi-module build. The example for the following description is provided at https://github.com/acetous/gradle-idea-plugin-multi-module-example.
We're having two modules api and shared where api has a compile dependency on shared.
Expected Behavior
Using IntelliJ testclasses in api can see the production classes from shared but have no access to the test classes in shared.
Current Behavior
Using IntelliJ testclasses in api can see the production and test classes from shared.
Context
The example provided is very generic. PersonTest imports HelperTest which should not be possible. In our large project with ~150 modules and a lot of dependencies in between the modules there are always situations in which someone uses a class from the test sources of another dependent project. IntelliJ can compile the project and run the tests. Gradle cannot compile the project because the dependency between those projects does not include classes from test sources.
Apart from that, when trying to write tests with Spring, IntelliJ suddenly finds other configuration classes and imports them thus failing the tests.
Steps to Reproduce (for bugs)

Clone https://github.com/acetous/gradle-idea-plugin-multi-module-example
Run gradlew check
The build fails with PersonTest.java:8: error: cannot find symbol HelperTest ◀ That is correct!
Run gradlew idea
Open the project in IntelliJ
Press Build project ◀ This should fail but doesn't.
Run PersonTest in IntelliJ. The test passes. ◀ The test should fail.

Your Environment

Build scan URL: https://scans.gradle.com/s/gf5tm3lol6tss
IntelliJ IDEA 2018.2 (Ultimate Edition) Build #IU-182.3684.101, built on July 24, 2018

Please use Idea's own importer instead of running gradle idea. Select the "Module per SourceSet" option. This will give you the separation you want.


Allow CMAKE_INSTALL_PREFIX to be a relative path

As requested by @albestro.

This is also a small step towards https://github.com/STEllAR-GROUP/hpx/issues/4228.


WebCrawl.java:33-35: Design and implement a way of extending ...

The puzzle 79-3c7202f7 in src/main/java/com/amihaiemil/charles/WebCrawl.java (lines 33-35) has to be resolved: Design and implement a way of extending crawling logic. Users might want to fetch something else from a page besides the raw content (e.g. page category as value of a hidden field)
The puzzle was created by amihaiemil on 24-Dec-16.
Estimate: 30 minutes, role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code.

@charlesmike hello

@charlesmike hello

Hi @amihaiemil! I can help you index your Github-hosted website, provided that the repository name respects the format owner.github.io (or has a gh-pages branch) and the command is given by the repo's owner.
More on commands here.

@charlesmike hello

@charlesmike hello

Hi @amihaiemil! I can help you index your Github-hosted website, provided that the repository name respects the format owner.github.io (or has a gh-pages branch) and the command is given by the repo's owner.
More on commands here.
Also, this blog post might be useful.


Feature/GitHub pages

Migrating to GitHub pages...

Seems to be working - has a deployment here: http://dls-controls.github.io/pymalcolm/merge/index.html (I assume master will be built after merging, I copied the docs CI from scanspec)


refactor(cherow): simplify lexer code

Is this just adding tests?

Nope. There are more to come.

Coverage remained the same at 90.734% when pulling 5461919e461cebbc73e0a076e4705c844e3a65a9 on cleanup_lexer into ded9ae71c6f41b9a59efe6a7ebe23ffd78a5cf0e on master.

@KFlash Have you noticed that I have change the PR title?

I noticed your : yes

Yeah this will conform the conventional commit message and you may do it like this next time.

Merging this as fixed.


helm add email config

Description
helm provides email smtp-secret，easy help people use email notification
Only helm configuration changes are included
Checklist

[ ] I have reviewed the contributing guidelines.
[x] I have included unit tests for my changes or additions.
[  ] I have successfully run make test-docker with my changes.
[x] I have manually tested all relevant modes of the change in this PR.
[ ] I have updated the documentation.
[ ] I have updated the changelog.

Questions or Comments

What's the benefit of using this vs the suggested method of an extra volume mount (commented out at the bottom of values.yaml)? While this would work it means storing credentials in the values.yaml, vs the suggested method of creating the credentials as a secret separately from the chart's general config.
If we proceed with this PR then the documentation needs to be updated in README.md, and an explanation should be included that notes the difference between using this method vs the other method. Also the contributing guidelines must be reviewed and the changelog needs to be updated.

It is the practice of the multi-volume mount method. Before smtp-auth Secret was created manually.
This commit is a configuration that automatically creates a secret based on the values configuration and uses it with mount.
After this pr, users use email alerts instead of manually creating secrets themselves, just release the annotations

proceed with
Do you think this pr will continue?

Do you think this pr will continue?

My personal belief is that secrets should not be stored in chart config files, but rather should be managed separately. However, if you (and others) prefer to mix config data with secrets, that is your choice. We can proceed with the PR but you will need to finish the contribution guideline requirements. Specifically:

Review and agree to the contribution guidelines
Add the documentation for the new config parameters to chart/elastalert2/README.md
Update the CHANGELOG.md with this change, following the existing patterns used in that file.

ok, let me continue with the following

ok, i have done the above


chore(workflow): cleanup cached on pull request close

Issue
Refs: https://docs.github.com/en/actions/using-workflows/caching-dependencies-to-speed-up-workflows#force-deleting-cache-entries
Description
Cleans up caches when pull request is closed
Testing
Will be done after PR is merged

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

The github.ref had the destination branch main on pull_request close activity, although the documentation says it should be source branch.
Because of this issue, the caches from main branch were deleted in workflow run https://github.com/awslabs/aws-sdk-js-codemod/actions/runs/3709173361/jobs/6287515634

Fix posted in https://github.com/awslabs/aws-sdk-js-codemod/pull/174


[BUG] Crypto assets wrong color/arrow

Describe the bug
I've added a crypto asset to the app, on coin market I can see that the price is falling, but the app shows it as rising (arrow up and green colour).
To Reproduce
Steps to reproduce the behavior:

Add item
Set tiker with TA-DA (https://coinmarketcap.com/currencies/ta-da/)
Set type Crytpo
See error

Expected behavior
I'd really like the right colour and the right arrow to be displayed.
Screenshots

Desktop (please complete the following information):

OS: Windows 10
Version 1.2.0

Additional context
It's the only crypto I have a problem with. The others work perfectly, whether you're in a rising or a falling market.

Hey @Lerwox ! It seems CoinMarketCap is caching the response for the percentage value (You can check it when you reload the page, you'll see first a number is loaded and then the new one is updated). As JS cannot be loaded to get this data, we need to wait for it to be refreshed and according to my testing, it happens every 5 minutes or so.

Hey @matextrem,
the fact is that your app currently has a bug. Because the color and direction of the arrow don't correlate with the percentage displayed.
A super simple fix is in the drawQuoteImage method to base it on the percentage variable (which from what I understand corresponds to the percentage change), rather than the change variable.
In fact, you can see in the method that it's only the percentage variable that's displayed (I do not see any display of the real price change), so it would make sense to correlate the direction of the arrow and the color with the percentage variable.
But I didn't analyze all the code, just the method for drawing the whole thing and the CoinMarketCap provider. So if I've missed anything, don't hesitate to let me know 😉

Hey @Lerwox! I could do that but I don't think it's gonna fix the issue cause the change variable has the price change difference rather than the percentage but in terms of change it is the same (it is negative when the active is decreasing and positive when it's increasing).
The actual issue is what I mentioned above. CoinMarketCap is caching the percentage change when JS is not enabled so if it's not updated if you have it disabled. You can easily check this going to the CoinMarketCap asset page, disable JS and reload the page, and then in another window with JS enable, go to the same asset page and compare the values. You'll see that in the page with JS disabled, the value will not reflect the updated one (but a previous one)
After some minutes (it seems it's about 5/6 minutes), it is updated.

Thanks for the quick reply @matextrem.
I understand what you mean, the token price (and its price evolution) are updated correctly. Where as the percentage change is cached. Is this correct?
So if I understand correctly, it's either we have the most up-to-date values, but the percentage change is likely to be decorrelated. Or we use the cached value, no decorrelation but arrow direction and color not up to date.
Am I mistaken?
In terms of pure UX/UI, I'd find it logical to have a correlation of the data, too bad for the actuality of the data.
Or maybe you need to find another site that doesn't use this caching method.

@Lerwox Yeah, you got it right.
We can either use another site for fetching crypto data without values being cached or use the cached value in order for the data to be correlated even though they are not the up to date values. In that case, I could do what you suggest about using the percentage instead of the change variable.
WDYT?

Okay cool, I'm happy to help.
Above all, having correlated data is the most important thing in my opinion.
After that, to solve the problem of the most up-to-date data, maybe using coingecko.com (which doesn't seem to have a cache) could be a solution.

@Lerwox I've created a new version with the Fix we discussed https://github.com/matextrem/streamdeck-tickertap/releases/tag/v1.2.1
I'll be analysing the option of using CoinGecko instead for future releases.

@matextrem Nice !
It works like a charm, thank you !


Decide on how to convert docx to xslt

May be you guys can start by reading this question on SO:
http://stackoverflow.com/questions/30090592/convert-openxml-document-to-pdf-in-javascript
answer is not posted, but i guess that's a start!. :)


Weird expansion bug with cmp, that repeats trigger at first insert point :thinking:

I'm not sure which plugin is doing this behavior exactly
Using the following snippet:
snip("do", {desc = "do ... end"}, U.myfmt {
  [[
    do
      <body>
    end
  ]],
  { body = i(0) },
})

Here I try to use the snippet 4 times:

with no text before, direct expansion via ls.expand() ok
with no text before, with expansion through cmp (select then <C-y>) ok
with text before, direct expansion via ls.expand() BUG
with text before, with expansion through cmp (select then <C-y>) ok

I'm using cmp with cmp_luasnip of course

Can't reproduce this one either :/
https://github.com/L3MON4D3/LuaSnip/assets/41961280/4089c07e-06eb-41bc-864f-938d9cf96cb8

Yeah that's what I thought, I'll try to repro with a minimal config later, probably this weekend.

It might be because I'm still using cmp's native menu.
Here is a reproduction script:
https://gist.github.com/bew/76f2d512ec42136f61b3802335a7b620


Sending multiple emails causing connection timeout with tcp://smtp.office365.com:587

Q
A

Bug report?
yes

Feature request?
no

RFC?
?

How used?
laravel 5.3

Swiftmailer version
x.y.z

PHP version
5.6.4

Im using laravel and swiftmailer to send out emails via cron. I know microsoft has a limitation of 30 emails per minute so i have a sleep in between for every 30 emails to wait 60 seconds before continuing to do the rest. All seems to work fine for about 400 emails or so then suddenly i get the following error:
exception 'Swift_TransportException' with message 'Connection to tcp://smtp.office365.com:587 Timed Out'
Does anyone know what may cause this or if there is any limitation in swiftmailer that i am unaware of. I contacted microsoft and it says it looks fine on their end and they dont see any issues.
My settings are as follows:
MAIL_DRIVER=smtp
MAIL_HOST=smtp.office365.com
MAIL_PORT=587
MAIL_ENCRYPTION=tls
I know settings are ok because it does send it for about 400 of it or so.

I brought mine down to 13 a minute and that seemed to work for a while, but it still times out occasionally. I switched to using mailgun for large volumes.

Ya im thinking of switching as well


Fix examples e.g. u not u_1 in some places, to match VS Code

At least in today's Lean, #check List shows List.{u} (α : Type u) : Type u, so it's something different again!
I'll update these examples again, then merge.
Thank you!


Create devdb recipe

#313
Couple main changes

create devdb recipe - relatively straightforward
dcpy.utils.versions - different version types as data classes, with logic around parsing, bumping, labels, etc. Definition of "VersionStrategy" as well, which is currently either a "SimpleVersionStrategy" (bump_latest_release or first_of_month at the moment) or some other more complex structure (right now BumpLatestRelease which has a value of int for how much to bump).
dcpy.build.plan - logic around new version strategy stuff
dcpy.connectors.edm.publishing - new previous function to lookup previous version

A couple potential big outstanding questions.

One is the long comment below.
Another is the handling of strings vs Version objects. I go back and forth a bit, but in general outside of versions leave them as strings. Seems a little easier in some ways, and just use the objects for logic like bumping. But maybe we should have the objects themselves in the recipe objects? Probably. Presumably can override how they get serialized (override __str__ or __repr__?)

@fvankrieken
looks like this PR dropped developments/bash/01_dataloading.sh but didn't add a call like python3 -m dcpy.builds.load, so DevDB is failing on main here. I can fix it while adding a KPDB recipe.yml?

Oops. I would prefer to handle it in my devdb branch since I'm going to be touching a lot of those files. If it'll be ready to be merged tomorrow that's fine though, tomorrow is a crazy meeting day so I might not get to much


Highlight required approvers

I guess it should be some kind of "*" or something with the approver's image to identify them as required.

Maybe something like this. Note that some approvers have circle in the color of the state.

It's in progress:

I will try to add other improvements on the same new version to be released soon.

What to do if the person doesn't have any photo?

For the status circle, it might some kind of shadow, maybe.


Investigate why LSP is slow

Problem
Compiler currently compiles the whole program and does not allow one to compile a single file without compiling the crate root first.
This may be one of the reasons why the LSP is slow to compile programs. It could also be that it is always generating ACIR, whereas we could make the LSP go only upto typechecking/ssa.
Happy Case
.
Alternatives Considered
No response
Additional Context
No response
Would you like to submit a PR for this Issue?
No
Support Needs
No response

It could also be that it is always generating ACIR, whereas we could make the LSP go only upto typechecking/ssa.

We only go up to typechecking currently.

We've gone through a bunch of optimisations e.g. caching of parsing layers, etc. so I'm gonna close this.


Deploy Error: Initialization failed

Any idea?
mup deploy

Meteor Up: Production Quality Meteor Deployments
------------------------------------------------

Building Started: /srv/www/meteor/london
.write(string, encoding, offset, length) is deprecated. Use write(string, offset, length, encoding) instead.

Started TaskList: Deploy app 'ufg' (linux)
[placetee.us] - Uploading bundle

crypto.js:431
    this._binding = new binding.DiffieHellman(sizeOrKey);
                    ^
Error: Initialization failed
    at new DiffieHellman (crypto.js:431:21)
    at Object.DiffieHellman (crypto.js:424:12)
    at onKEXDH_GEX_GROUP (/usr/local/lib/node_modules/mup/node_modules/nodemiral/node_modules/ssh2/node_modules/ssh2-streams/lib/ssh.js:2349:22)
    at SSH2Stream.<anonymous> (/usr/local/lib/node_modules/mup/node_modules/nodemiral/node_modules/ssh2/node_modules/ssh2-streams/lib/ssh.js:191:36)
    at SSH2Stream.EventEmitter.emit (events.js:106:17)
    at parse_KEX (/usr/local/lib/node_modules/mup/node_modules/nodemiral/node_modules/ssh2/node_modules/ssh2-streams/lib/ssh.js:3856:14)
    at parsePacket (/usr/local/lib/node_modules/mup/node_modules/nodemiral/node_modules/ssh2/node_modules/ssh2-streams/lib/ssh.js:3731:12)
    at SSH2Stream._transform (/usr/local/lib/node_modules/mup/node_modules/nodemiral/node_modules/ssh2/node_modules/ssh2-streams/lib/ssh.js:555:13)
    at SSH2Stream.Transform._read [as __read] (_stream_transform.js:179:10)
    at SSH2Stream._read (/usr/local/lib/node_modules/mup/node_modules/nodemiral/node_modules/ssh2/node_modules/ssh2-streams/lib/ssh.js:213:15)

First fix the depreciated write, although it shouldn't be a problem.
second try mupx.

not sure where that 'write' comes from, but the error is about ssl/ssh
and mupx gives the exact same error

Did you try following the instructions to upgrade meaning you'very done a clean mupx install?

Yes, I have created the new folder and run mupx init, setup and tried deploy
The only difference as I can see is the mup.json. And the only difference in mup.json is
<   "enableUploadProgressBar": true,
<
<       "buildOptions": {
<               // build with the debug mode on
<               "debug": true
<       }
< }

It looks like there is timeout error with ssh, the strange thing sometimes it goes OK, but mostly it fails before I see the progress bar.

How about running 'npm updates'?
Maybe node needs updating. What version of node is installed?
Can you test ssh is working as expected?

npm version
{ http_parser: '1.0',
  node: '0.11.8-pre',
  v8: '3.20.17',
  uv: '0.11.13',
  zlib: '1.2.3',
  modules: '13',
  openssl: '1.0.1e',
  npm: '1.3.11' }

I have tried npm update. It just does nothing
I have tried ssh as the first possible issue. No problem, ssh connects instantly

Okay maybe this link will help...
https://github.com/arunoda/meteor-up/issues/427
it says they reverted back a version.

Thanks! I think that was it.
npm install -g n
n 0.12.7

0.12.7 - the latest ad greatest, I had 0.11 before
And now it flies


New signin: ignore spaces in front and after email address

Description:
User inputs email address and adds space at the front and after email address.
Actual: invalid email address error
Expected: ignore spaces in front and after address to make sign in smoother.
https://user-images.githubusercontent.com/50437378/219255024-b66d1c86-883e-418a-82ac-839c75549a2d.mov
Main card: https://github.com/ruuvi/com.ruuvi.station/issues/916

QA passed


These are minor release to fix a security issue.
Following our policy at https://golang.org/security, this is the pre-announcement of those releases.
Because we are so late in the release cycle for Go 1.7, we will not issue a minor release of Go 1.5.
Additionally, we plan to issue Go 1.7rc3 later next week, which will include any changes between 1.7rc1 and tip.
Cheers,
Chris on behalf of the Go team

Note:
the man/Dockerfile is not yet updated, because
the official image for Go 1.6.2 has not yet
been updated.

/cc @tiborvass @cpuguy83 :smile:

Removed from milestone until proven it's required.

why you gotta do this to the CI, it has a hard life as it is. poor ci ;)
closing until its actually released and we know everything contained in the release.

LGTM

@thaJeztah the 1.6.3 golang image is up on hub.

@tiborvass bumped man/Dockerfile to 1.6.3 as well

LGTM

LGTM 👼

this too appears to have been cherry-picked into bump_v1.12.0 already


Add epochCeiling function to Primitive.Types.

Issue Number
#1086
Overview
This PR:

[x] Adds the functions epoch{Ceiling,Floor}

epochCeiling calculates the earliest EpochNo whose start time is greater than or equal to the specified time. (Required by issue #1086.)
epochFloor  calculates the latest EpochNo whose start time is less than or equal to the specified time. (Added for completeness.)

[x] Adds the epochStartTime and epoch{Pred,Succ} helper functions.
[x] Adds a set of property tests for the above.

Comments
The functions added by this PR follow the same general pattern as the following existing functions:

slot{Ceiling,Floor}
slot{Pred,Succ}
slotStartTime

bors r+

bors r+

@paweljakubas
Thanks for your review, and very helpful feedback! I've added replies to your comments.
I have some further changes to make in response to your feedback, and will add those changes in PR #1121 (so they can be reviewed separately).


Snow EKS-A admin AMI packer build

Issue #, if available:
https://github.com/aws/eks-anywhere/issues/1166
Description of changes:
TODO
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/hold until I add a readme

/unhold