Project Level 3(?)

basically i just really dont get how I'm out of index range? This is after I implemented level 8, and it passes 8_4 test case, so originally it was already working for this, but i have no idea what i changed that would even remotely affect so early in the levels, if anyone got any insight it would be greatly appreciated!!

Since the error occurs in your ArriveEvent, maybe in there you could input a print statement to print out the "status" of the ImList to help with debugging. You can also try to trace out the code line by line since level 3 test cases are quite short. All the best!

since the error from a get could it be that you have accidentally removed an obj from the list too early such that at the next Arrive the list is empty?

Hi, I think the index out of range probably arised due to the loop conditions. First try to locate which loop is causing the condition first. U can try commenting to ur code or something along this line to check. Its likely that its a minor error like <= instead of < etc. Hope u are able to debug!

SOLVED! It was just an inequality error that arised when I was checking for checking queues in self checkout, where the if elses in my arrive event went to get a self checkout server when there was not any since the checking queues always returned true.


Whack the mole should specify that you might need to unmute the simulator

In some browsers (cough cough safari) the simulator is muted by default because the browser requires direct user interaction to play sounds. The instructions should mention that you might need to unmute the sim if you don't hear anything. particularly on this step:

Added!


Spelling

Found manually with some help from IntelliJ IDEA syntax highlighter :)

Hi, sorry for taking so long to address this, I've replaced the example "word" with a real word. I do appreciate the feedback, you were right in that my example wasn't a real word and thus was unlikely to be something that a user would actually produce.
The two easiest words that satisfied the requirements (confuse the lazy script, could be used by a human) are cherrypick and handpick, I've settled on the former, although it's possible that the latter was closer to my original simplistic example of "this could confuse things". It's also possible that and pick would confuse it too.


N+1 Query Failed for  "undefined method `preloaded_records' for :tag:Symbol"

env:

acts-as-taggable-on (3.4.2)
rails (4.1.5)

problem:
I have a "Recruitment" class which has "acts_as_taggable"
class Recruitment < ActiveRecord::Base
    acts_as_taggable
    ...
end

when i run the code jobs = Recruitment.where(xxxx)
it shows N+1 query detected:
 Recruitment Load (1.6ms)  SELECT `recruitments`.* FROM `recruitments`  WHERE `recruitments`.`product_id` = 36592
  ActsAsTaggableOn::Tag Load (1.8ms)  SELECT `tags`.* FROM `tags` INNER JOIN `taggings` ON `tags`.`id` = `taggings`.`tag_id` WHERE `taggings`.`taggable_id` = 112 AND `taggings`.`taggable_type` = 'Recruitment' AND `taggings`.`context` = 'tags'
  ActsAsTaggableOn::Tag Load (1.2ms)  SELECT `tags`.* FROM `tags` INNER JOIN `taggings` ON `tags`.`id` = `taggings`.`tag_id` WHERE `taggings`.`taggable_id` = 113 AND `taggings`.`taggable_type` = 'Recruitment' AND `taggings`.`context` = 'tags'
N+1 Query detected
 Recruitment => [:tags]
 Add to your finder: :include => [:tags]
N+1 Query method call stack
  /local/code/company/gurudigger/gurudigger/gurudigger_git4/app/controllers/api/helpers.rb:21:in `block in represent_each'

so I add includes method for query: jobs = Recruitment.where(xxxx).includes([:tags])
it show error below:
NoMethodError - undefined method `preloaded_records' for :tag:Symbol:
  activerecord (4.1.5) lib/active_record/associations/preloader/through_association.rb:32:in `associated_records_by_owner'
  activerecord (4.1.5) lib/active_record/associations/preloader/has_many_through.rb:8:in `associated_records_by_owner'
  activerecord (4.1.5) lib/active_record/associations/preloader/collection_association.rb:13:in `preload'
  activerecord (4.1.5) lib/active_record/associations/preloader/association.rb:20:in `run'
  activerecord (4.1.5) lib/active_record/associations/preloader.rb:137:in `block (2 levels) in preloaders_for_one'
  activerecord (4.1.5) lib/active_record/associations/preloader.rb:135:in `block in preloaders_for_one'
  activerecord (4.1.5) lib/active_record/associations/preloader.rb:134:in `preloaders_for_one'
  activerecord (4.1.5) lib/active_record/associations/preloader.rb:106:in `preloaders_on'
  bullet (4.14.0) lib/bullet/active_record41.rb:35:in `preloaders_on'
  squeel (1.2.1) lib/squeel/adapters/active_record/4.1/preloader_extensions.rb:21:in `block in preload_with_squeel'
  squeel (1.2.1) lib/squeel/adapters/active_record/4.1/preloader_extensions.rb:20:in `preload_with_squeel'
  activerecord (4.1.5) lib/active_record/relation.rb:610:in `block in exec_queries'
  activerecord (4.1.5) lib/active_record/relation.rb:609:in `exec_queries'
  activerecord (4.1.5) lib/active_record/relation.rb:486:in `load'
  activerecord (4.1.5) lib/active_record/relation.rb:231:in `to_a'
  bullet (4.14.0) lib/bullet/active_record41.rb:10:in `to_a'
  activerecord (4.1.5) lib/active_record/relation/delegation.rb:46:in `map'
  app/controllers/api/helpers.rb:21:in `represent_each'

I lost here, so can anybody help me? thx

I had a similar error using Chewy for ElasticSearch import in 4.1.9. I was able to solve the Symbol error by adding references to the includes, so for your example:
jobs = Recruitment.where(xxxx).includes(:tags).references(:tags)

I don't know if this will solve your issue, but I wanted to throw it out there just incase.


K8s deployment recommendation

I've dockerized a chainlit application and I would like to run it in a kubernetes environment to enable multiple concurrent users.  Is there any guidance on how best to do this?  Simply run some number of pods in a deployment behind either an Ingress or Load Balancer?

No special advice except that your load balancer should have sticky sessions since Chainlit uses web sockets. I would also not spawn lots of pods, one pod should handle hundreds of users.


Opening article reader from AR breaks Voyager

Tested on up-to-date Chrome on Android 10
https://3d.si.edu/object/3d/c9ea282c-4bf8-4e8d-9080-750b85f04571
Clicking on the "Read more..." link on an annotation while in AR mode brings up the article viewer at full width of the frame instead of fitting to the right of the UI buttons (so that the UI buttons overlap the article reader)
Closing the article reader then displays an empty Voyager scene and does not bring the user back to the AR view.

Fix in #69


Issue with min cost flow input

I'm receiving the following error when calling KMeansConstrained.predict():
"There was an issue with the min cost flow input."
This model contains 5 clusters and is running on 850 data points.  During training, I have set the min and max size to be 140 and 200 respectively.  If I relax the size constraint to 120, 220, predict() works fine.

Hi 👋,
Can you please provided a minimum workable example to demonstrate the issue. Stackoverflow have a good article on how to produce this. Importantly the example should contain the least amount of code and data to reproduce the error. Ideally you should be able to paste it as few lines of code and include only a few datapoints of data within a GitHub comment.
Could you also include the k-means-constrained version you are using.
Thanks,
Josh

I'm going to close this as I haven't received a response. Please reopen it if you wish


Add Quip to sites.json

Adds Quip to the list. They require that you contact customer support to delete your account, which is a bummer.

Hello @rosszurowski ,
This repo is not being maintained anymore, but we are trying to keep it alive on a detached fork here, would you kindly submit your PR there and close this one here? :)

@tupaschoal Done, thanks for the heads-up!


How use option path like socket.io-client for JavaScript?

With client-side JavaScript:
var socket = io('http://<IP_ADDRESS>:8080', { path: '/myapp/notify' })
But I can't find path option in this library. In API documents, I saw:
h.connect("http://<IP_ADDRESS>:8080") without path option.
Because my socket.io server deploy on a sub-domain with NginX. On server-side:
var io = require('socket.io')(http, { path: '/myapp/notify' });
So my question is: How use option path like socket.io-client for JavaScript?

Did we get an answer on this? Or is this not supported?

I can confirm that this is officially not supported by this plugin due to the following line:
https://github.com/socketio/socket.io-client-cpp/blob/master/src/internal/sio_client_impl.cpp#L218

I was hit by this today. Kinda annoying...


Error while sending email from outlook.com

I am writing some junit test cases using ews-java-api to send mails. I have setup a mail account in outlook.com and using it to send and receive mails. While sending mails, sometimes I get this below error:

microsoft.exchange.webservices.data.core.exception.service.remote.ServiceResponseException: An internal server error occurred. The operation failed., WASCL UserAction verdict is not None. Actual verdict is RefuseQuota, ShowTierUpgrade.
at microsoft.exchange.webservices.data.core.response.ServiceResponse.internalThrowIfNecessary(ServiceResponse.java:278)
at microsoft.exchange.webservices.data.core.response.ServiceResponse.throwIfNecessary(ServiceResponse.java:267)
at microsoft.exchange.webservices.data.core.request.MultiResponseServiceRequest.execute(MultiResponseServiceRequest.java:165)
at microsoft.exchange.webservices.data.core.ExchangeService.sendItem(ExchangeService.java:789)
at microsoft.exchange.webservices.data.core.service.item.EmailMessage.internalSend(EmailMessage.java:171)
at microsoft.exchange.webservices.data.core.service.item.EmailMessage.sendAndSaveCopy(EmailMessage.java:300)
at com.dizitart.orange.exchange.ExchangeConnection.sendMessage(ExchangeConnection.java:199)
... 34 more

What does this error message means?

Nope. I have not.

@anidotnet  Were you able to solve it?

@exp-amal Nope.

@anidotnet  Try logging in your mail and verify it

Is there any fix for this issue? I have long ago abandoned the project where I was using it.

It is just mail verification issue. You can simply login to the outlook account which you are using. They will ask for you to confirm your identity. Something like "Help us fight spam email
We're sorry to have to ask you to prove you're a person. Most spam comes from automated programs, which can't solve puzzles like this."
Below, there will be option to enter your phone number and confirm your identity. That's it.


Discussion: How do we implement multiple projects/instances?

Right now, when the user opens Sprout, they see the landing page for importing a dataset that was created from a specific study/research project. But how will it work another users, from another research/study project, wants to upload data into a "study-specific instance/project" that holds all data collected for that project (and not connected to other projects)? This is something I don't really know how it will look like.
I assume we might need an initial landing page of "create project" and "select existing project", from there they can create tables for that project and be owners for it and assign permissions as needed there.
Then the question expands into, where is the data for each "project instance" stored, and how is it separated?
@philter87 I've assigned you to this since you might have more ideas about it.

I think it is fairly easy to imagine a solution in the web app. Every table is linked to to a project and users are assigned to projects with certain permissions (write, read, etc )
What is unclear to me is how many should be allowed direct access to the database (and the server)? And whether this access should be handled by the web application.
Scenario 1: Direct access to database is not handled in the web application
If there is only one admin in a seedcase instance, then everything is nice and simple. This person have access to everything in the database  and we don't need to align with the users and roles in the web application (The in a Django web application is defined in the tables auth_user, auth_group, etc).
The admin can manually give access to other people in the database by creating users and roles in the database. But these are not automatically aligned/synchronized with the users and roles in the web application.
Every project could have their own database "schema" to easily divide the database in multiple parts
Scenario 2. Users and roles are synchronized between the app and the database
If there is an admin for each project, and they should have direct access to see certain parts of the database, then it becomes more complicated! The admin users in the web application would need to get "synchronized" with the users in the database (and potentially the server somehow).

We decided to start the web app from scratch after the major design/architecture decisions and changes we made. No longer relevant.


sol-network & sol-socket: move strings to sol-buffer

Both network and socket operations that do manipulate strings should be dealing with buffers (struct sol_buffer) instead of getting char *buf and size_t (in some functions) or doing strdup() in others.
Keep a single API that receives a buffer, grow the buffer as needed and give to the function (some functions allow to query the size) or call sol_buffer_append().
Remember to document if buffers must be initialized before or not. I prefer to have people to provide initialized buffers since they can put one that points to the stack and says "fixed_capacity" so no allocations are done.
It would be nice to review the whole API to to provide a single behavior across soletta (all functions outputting strings to buffers).
@cabelitos and @ceolin should know how to do this

@barbieri Hello,
Don't you think that this change will make the API more difficult to use? Imagine the case that I wan't to receive a message from the socket.
void handle_msg(sol_socket *socket, struct sol_network_link_addr cliaddr)
{
    uint8_t mem[DTLS_MAX_BUF];
    struct sol_buffer buf = SOL_BUFFER_INIT_FLAGS(mem, sizeof(mem),
        SOL_BUFFER_FLAGS_FIXED_CAPACITY);
    int len;

  len = sol_socket_recvmsg(socket, &buf, &cliaddr);

  //Do my stuff here
}

On Fri, 19 Feb 2016 12:57:46 -0800, Guilherme Íscaro wrote:

@barbieri Hello,
Don't you think that this change will make the API more difficult to use? Imagine the case that I wan't to receive a message from the socket.
void handle_msg(sol_socket *socket, struct sol_network_link_addr cliaddr)
{
    uint8_t mem[DTLS_MAX_BUF];
    struct sol_buffer buf = SOL_BUFFER_INIT_FLAGS(mem, sizeof(mem),
        SOL_BUFFER_FLAGS_FIXED_CAPACITY);
    int len;

  len = sol_socket_recvmsg(socket, &buf, &cliaddr);

  //Do my stuff here
}

But that's not a string.

@ibriano Indeed, however I assumed that char * meant bytes, not purely string.

Anyway, ignore my comments. Thank you

#1461

merged


Update base image to RH UBI 9

Use ubi9 as base image and include OpenStack repos
Depends-On: https://github.com/infrawatch/service-telemetry-operator/pull/625

Pretty curious if the error in the CI is just a timeout or any of the smoketests images we use is having some issue... rechecking just to see if we hit the same issue

recheck

recheck


rkt: stdin and stdout redirection

with docker i can do something like this
echo hello world | docker run --rm -i --entrypoint /bin/cat busybox | cat

can i do that in rkt with one container pod?, i have tried
echo hello world | rkt run docker://busybox --interactive --exec /bin/cat | cat

but it look like --interactive always allocate tty, is there flag like --notty?

Indeed, there is no flag --notty.
I'll close this and let's track this in https://github.com/coreos/rkt/issues/2414


Fix storybook issue introduced from vite environment variables

Need to resolve issue from latest Build & Deploy Action.
Details:
ERR! => Failed to build the preview
ERR! ./src/firebase-config.ts 2:16
ERR! Module parse failed: Unexpected token (2:16)
ERR! File was processed with these loaders:
ERR!  * ./node_modules/babel-loader/lib/index.js
ERR! You may need an additional loader to handle the result of these loaders.
ERR! | var firebaseConfig = ***
ERR! >   apiKey: import.meta.env.VITE_FIREBASE_API_KEY,
ERR! |   appId: import.meta.env.VITE_FIREBASE_APP_ID,
ERR! |   authDomain: import.meta.env.VITE_FIREBASE_AUTH_DOMAIN,
ERR!  @ ./src/firebase-init.ts 2:0-47 6:25-39
ERR!  @ ./src/pages/home/home-page.tsx
ERR!  @ ./src/pages/home/home-page.stories.tsx
ERR!  @ ./src sync ^\.(?:(?:^|\/|(?:(?:(?!(?:^|\/)\.).)*?)\/)(?!\.)(?=.)[^/]*?\.stories\.(js|jsx|ts|tsx))$
ERR!  @ ./.storybook/generated-stories-entry.js
ERR!  @ multi<EMAIL_ADDRESS><EMAIL_ADDRESS>./.storybook/storybook-init-framework-entry.js<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>./.storybook/preview.js-generated-config-entry.js ./.storybook/generated-stories-entry.js

After several attempts at correcting this bug, I've opted to instead to drop storybook in favor of Docz via issue #27 .

After attempting to use an alternative component library and documentation solution via #27, it appears that there isn't a great solution that plays well with Vite due to the lack of import.meta. The other issue is that the current Storybook release uses Webpack 4 which doesn't have support for import.meta, but there is a prerelease that is built with Webpack 5. I've attempted to target that version to see if I could get Vite and Storybook to work together, but it ended up producing the same issue. The good news is that the recent 6.2 release opens the door for custom JS bundlers like Vite. I am going to comment out storybook for the time being, and keep an eye out for the open issue that Storybook has to add a Vite Builder.
This is the error I'm currently running into:
ERROR in ./src/firebase-config.ts 2:16
Module parse failed: Unexpected token (2:16)
File was processed with these loaders:
 * ./node_modules/babel-loader/lib/index.js
You may need an additional loader to handle the result of these loaders.
| var firebaseConfig = {
>   apiKey: import.meta.env.VITE_FIREBASE_API_KEY,
|   appId: import.meta.env.VITE_FIREBASE_APP_ID,
|   authDomain: import.meta.env.VITE_FIREBASE_AUTH_DOMAIN,
 @ ./src/firebase-init.ts 2:0-47 6:25-39
 @ ./src/pages/home/home-page.tsx
 @ ./src/pages/home/home-page.stories.tsx
 @ ./src sync ^\.(?:(?:^|\/|(?:(?:(?!(?:^|\/)\.).)*?)\/)(?!\.)(?=.)[^/]*?\.stories\.(js|jsx|ts|tsx))$
 @ ./.storybook/generated-stories-entry.js
 @ multi<EMAIL_ADDRESS><EMAIL_ADDRESS>./.storybook/storybook-init-framework-entry.js<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>./.storybook/preview.js-generated-config-entry.js ./.storybook/generated-stories-entry.js (webpack)-hot-middleware/client.js?reload=true&quiet=false&noInfo=undefined

Hey @HeyKos, I also faced the same problem in my monorepo architecture where individual packages are using vitejs. I solved this by using webpack5 along with providing a path to the .env file from a package and extending the plugin in custom webpack configuration.
Here's the code of .storybook/main.js
const { resolve } = require('path');
const dotenv = require('dotenv');
const webpack = require('webpack');

dotenv.config({
  path: resolve(__dirname, '../packages/safety/.env.development'),
});

module.exports = {
  stories: [
    '../packages/**/*.stories.mdx',
    '../packages/**/*.stories.@(js|jsx|ts|tsx)',
  ],
  addons: [
    '@storybook/addon-links',
    '@storybook/addon-essentials',
    {
      name: '@storybook/addon-postcss',
      options: {
        postcssLoaderOptions: {
          implementation: require('postcss'),
        },
      },
    },
  ],
  core: { builder: 'webpack5' },
  webpackFinal: async (config) => {
    config.resolve.alias = {
      src: resolve(__dirname, '../packages/safety/src'),
    };
    config.plugins.push(
      new webpack.DefinePlugin({
        'import.meta.env': JSON.stringify(process.env),
      }),
    );

    return config;
  },
};

There has been some movement with the custom vite builder for Storybook. It might be work another attempt.

Probably after upgrading to node 18 this happens for me as well using 7.0.11 of @storybook/react-vite. Whenever import.meta is used in a js the build breaks. Any ideas how to fix it?


Harden 'create_bucket' systest against 429 responses.

Closes #5533.

This hardly needs review, and the updated system test passes.


ARROW-17254: [C++][Go][Java][FlightRPC] Implement and test Flight SQL GetSchema

Consistently implements and tests the GetSchema method in Flight SQL.
Builds on #13897.

I expect Go will fail, need to fix that…

@zeroshade I implemented the GetSchema methods for Go here as well (the original PRs didn't implement them fully, so this PR is to rectify that)
@jduo I noticed GetSchema is in the protos but not the implementations; this rectifies that (I expect these methods won't see much use, but might as well)

Thankfully Go is straightforward!
And yeah, they're in the spec and in Java, but got omitted from C++ (whoops)

CI failures are unrelated or already tracked, merging


Wrong fix for crossing:markings=zebra;dots

Description
crossing:markings=zebra;dots should be legal, but the editor suggests "upgrading the tags" by simply removing "dots", leaving it as crossing:markings=zebra
It is however a correct syntax, as stated in https://wiki.openstreetmap.org/wiki/Key:crossing:markings#Examples
Screenshots

Version
2.2.5
What browser are you seeing the problem on? What version are you running?
Firefox v115.0
The OS you're using
arch, btw
Steps to reproduce

make a crossing with this tag "crossing:markings=zebra;dots"
rapid will complain

The browser URL at the time you encountered the bug
https://rapideditor.org/edit#background=Bing&datasets=fbRoads,msBuildings&disable_features=buildings,landuse,boundaries&map=21.16/44.77937/10.78473
The auto-detected useragent string for your browser (leave blank if you're manually filling this form out)
Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:109.0) Gecko/20100101 Firefox/115.0

It is probably offering that suggestion because that node also has the crossing=zebra tag.  I’m not sure if we will be able to support the semicolon delimited values as well as the plain values.


more examples.

your work is great ! thank you.But,more example is needed for beginer. Could you give us a example about the function:
std::pair<cv::Mat, cv::Mat> render(Mesh mesh, cv::Mat model_view_matrix, cv::Mat projection_matrix, int viewport_width, int viewport_height, const boost::optional<Texture>& texture = boost::none, bool enable_backface_culling = false, bool enable_near_clipping = true, bool enable_far_clipping = true).
thanks again.

Thank you very much!
4dface is quite a good example, particularly around this line for the render() function.
Does that help you?

It help me a lot , than you.


Issue with "Stick recent mentions to top of chat"

As you can see in the images, the mentions feature permanently expands and has previous chat messages hidden underneath new ones. The messages go away once you cant scroll up to see them in chat anymore, but its kind of annoying since it would be a useful feature otherwise. Would be nice if I could just close the pinned mention.

I feel like it's still a useful feature regardless. I'll think about a close button, but no promises, it might be too obnoxious/ugly or have too much of a performance impact.
You can try the dev version of the script meanwhile, it separates the stacked mentions a bit better.


Don't wait cookie before form submit, put them in a hidden input and parse on the server

I'm working on this

Done in https://github.com/churkin/testcafe-hammerhead/commit/6be7cb02879c28c703a3bfe8632c535f76a7059d


Compiler warnings are not collected

mojo-pytest uses mojo run whereas as of Mojo 0.5.x, it appears that compiler warnings are only emitted by mojo build.  Workarounds: none known. Possibly this could be re-written to use mojo build.

Compiler warnings are emitted by mojo run but are not reliably output, but maybe in the future if this is fixed then mojo-pytest can capture+display them.
$ mojo run repro.mojo
.../repro.mojo:2:5: warning: 'x' was declared as a 'var' but never mutated, consider switching to a 'let'
    var x = 42
    ^
42
$ mojo run repro.mojo
42
$ mojo run repro.mojo
42

Closed by https://github.com/guidorice/mojo-pytest/commit/58d5086091279d12f3c2014677694a7b1ee5d574


Adding a note about third party JSON libraries to the readme

Addresses a comment on issue #116

Great stuff. Was just about to write all of this myself ;)
Thanks!


Claim BLD insurance award

Describe the bug
The txHash generated by this command, is not visible in explorer:  68978D4BC53EFD595E71DA862190C8B5AA51132937ABF39A17850E2E7A9BD883
To Reproduce
ag-cosmos-helper tx distribution withdraw-all-rewards --from=stake2earn --chain-id=agorictest-11 --gas=auto --keyring-dir=$HOME/.ag-cosmos-helper
Screenshots

Desktop (please complete the following information):

OS: Linux 18.04

@michaelfig is that tx missing in your node for running Big Dipper?

Nop, from my node is not missing ( I can 'query tx' that hash directly in chain and see the results). But in Big Dipper browser explorer, while searching for that tx, I found anything.

@michaelfig is that tx missing in your node for running Big Dipper?

I've confirmed that the tx is on Big Dipper, just that the withdraw_rewards rendering has a problem with the data model, and it's the same on current master.  I've traced it to events.withdraw_rewards is undefined:
root@explorer-testnet:/home/bigdipper/big-dipper# rg withdraw_rewards .
./imports/ui/components/Activities.jsx
84:            return
<Account address={msg.delegator_address}/> {(this.props.invalid)?<T>activities.failedTo</T>:''}<MsgType type={msg["@type"]} /> {(!this.props.invalid)?<T _purify={false} amount={new Coin(parseInt(events['withdraw_rewards'][0].value), events['withdraw_rewards'][0].value.replace(/[0-9]/g, '')).toString(6)}>activities.withAmount</T>:''} <T>activities.from</T> <Account address={msg.validator_address} /><T>common.fullStop</T>

root@explorer-testnet:/home/bigdipper/big-dipper#

You can reproduce it at: https://testnet-11.explorer.agoric.net/transactions/68978D4BC53EFD595E71DA862190C8B5AA51132937ABF39A17850E2E7A9BD883


feat(theme-default): enhance header dropdown experience

This pull request introduces different transition behaviors for header and aside navbars:

header: transform & opacity, just like https://vuejs.org
aside: collapse (DropdownTransition)

A serious problem, did you fully test it? Desktop mode is not working. Please make sure that mouseleave event is correct (I don't think that's correct) @Shigma

https://user-images.githubusercontent.com/33315834/160976786-ede136c6-872f-418d-bc7c-f42bbed160be.mp4
I am afraid that if the dropdown menu on the right is too long, it will prevent the left dropdown to work

I will revert this as this is a serious issue, the dropdown menu are clickable(but not visible)

You may check my theme: http://vuepress-theme-hope.github.io/v2/ to find a good way achieving this.


Godot badly projects 3D camera ray if stretch mode is 2d+"expand" (or "keep")

Godot 3.1.1
Windows 10 64 bits
I have a 3D game in which I use the following code to get a ray from the captured mouse (itself patched to half the size of viewport because of #29559):
	var mouse_pos = get_viewport().size / 2.0
	var ray_origin = _camera.project_ray_origin(mouse_pos)
	var ray_direction = _camera.project_ray_normal(mouse_pos)

Which works fine with the default stretch mode, even when I resize the window.
However, I have a HUD I want to scale nicely, but if I set the stretch mode to 2d and expand, the 3D ray becomes wrong, offset somehow. I believe camera project functions are broken.
Meanwhile, I can't make my UI to stretch properly because of that.
(sorry no time for a repro project yet, I'm in the middle of a game jam)

Can anyone still reproduce this bug in Godot 3.2.3 rc3 or any later release?
If yes, please ensure that an up-to-date Minimal Reproduction Project (MRP) is included in this report (a MRP is a zipped Godot project with the minimal elements necessary to reliably trigger the bug). You can upload ZIP files in an issue comment with a drag and drop.

I just made a project to reproduce it, and I realize Viewport.get_mouse_position() is returning values that follow the 2D stretching behavior, which I didn't know before. When the doc says "relative to viewport", I thought it meant "relative to viewport's output pixels", without including one of the many canvas transformations happening under the hood.
To workaround #29559, and knowing my mouse is locked, I had assumed I could then get the mouse's position from the viewport's center, in final viewport pixels, using this:
	var mouse_pos = get_viewport().size / 2.0
	var ray_origin = _camera.project_ray_origin(mouse_pos)
	var ray_direction = _camera.project_ray_normal(mouse_pos)

That worked without stretching, but 2D expand mode made it wrong because project_ray_normal expects the output of get_mouse_position(), which again, is using a different coordinate system. There is no indication of that in the doc, maybe it should be added?
(a side consequence is that I'm not sure what I should do to convert get_viewport().size / 2.0, to the same coordinate system as get_mouse_position()).
Otherwise it seems that this issue is fixed.

I spent a lot of time trying to figure this out and finally found a solution that worked for me. I still don't completely understand it, but I'll just leave the (C#) code here in case anyone stumbles upon this and wants to try it themselves.
Vector2 screenPosition = this.GetViewport().Size / 2;
Vector2 viewportRelativePosition = this.GetViewport().GetFinalTransform().AffineInverse().Xform(screenPosition);
Camera.ProjectRayNormal(viewportRelativePosition);


fix: Cannot click on some items in search bar

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.
PR type

[x] Bugfix
[ ] Feature
[ ] Refactoring

What's new?

[x] PR closes #3818

Fixed:

click on the newest listings
click on history item

Before submitting Pull Request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality
[x] I've posted a screenshot of demonstrated change in this PR

Optional

[ ] I've tested it at </rmrk/collection/26902bc2f7c20c546a-1FVG7>
[ ] I've tested PR on mobile and everything seems works
[ ] I found edge cases
[ ] I've written some unit tests 🧪

Had issue bounty label?

[x] Fill up your KSM address: Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot

[x] My fix has changed something on UI; a screenshot is best to understand changes for others.

pay 50

😍 Perfect, I’ve sent the payout
💵 $50 @ 48.96 USD/KSM ~ 1.021 $KSM
🧗 Caiv9TbPz68q5dC8EcHu5xKYPRnremimGzqmEejDFNpWWLG
🔗 0xcbf24d5fa01ada94eada2446e4305e05fed4ef7f7810fdb2a976ac6f29f2e447
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


Re-throw non-NSUndefinedKeyException exceptions when applying classes

This prevents the incorrect exception from being thrown when setting an Obj-C property of type MTFThemeClass that itself has properties that are unapplied. It fixes this by re-throwing the original exception, rather than always throwing.

Coverage decreased (-0.02%) to 80.54% when pulling 7cf2186c6fef417b70f96af8b7445eaccf1d7f4f on add-exception-for-unapplied-properties into 0109cc25c739f27fae7a4597fa190c6f41ce4c99 on master.

Coverage increased (+0.39%) to 80.95% when pulling 35d31c754badd1d3a5153ce60cd750afd303cd91 on add-exception-for-unapplied-properties into 0109cc25c739f27fae7a4597fa190c6f41ce4c99 on master.


Error: A runtime version is set in your AndroidManifest.xml, but is missing from your app.json/app.config.js

Sometimes when running expo prebuild in project with android directory, we get the following error:

So far it has only been observed in CNG projects with android directory left as junk after build.
Quick remedy could be to

evaluate whether project is CNG, like here: https://github.com/expo/expo/blob/ee198d023415c5d3e0f9ae9676f567badb48821d/packages/expo-updates/utils/src/workflow.ts
run expo prebuild with flag --clean if it is CNG

It would be nice to understand exactly what the issue is though

Steps to reproduce:

Set up expo project with app.json which has no runtimePolicy field and android/ directory in .gitignore.
Run setup-ci --preset --eas.
Run npx expo prebuild.
Run git reset --hard HEAD; git clean -df.
Run setup-ci --preset --eas again.
error occurs

Root cause
EAS Preview recipe patches app.json by adding
runtimeVersion: {
    policy: "fingerprint"
}

After that, npx expo prebuild (step 3) produces android/app/src/main/AndroidManifest.xml file with the following line:
<meta-data android:name="expo.modules.updates.EXPO_RUNTIME_VERSION" android:value="@string/expo_runtime_version"/>.
This line is added to AndroidManifest.xml by expo prebuild only if runtimeVersion is defined in app.json.
Then, if we reset app.json to original state (step 4.), and runtimeVersion field is removed), the android directory is not removed since it is listed in .gitignore and running npx expo prebuild again (as a subprocess of step 5) results in the error.

I will close this with #120 as this seems like a very rare edge case that should not be handled by us.


Add WASD Camera Plugin

WASD Camera Plugin
-Listed under "WASD Camera"
-Allows use of WASD keys to move camera
-Chatbox is disabled with an overlay that says "Press Enter to Chat..."
-Upon hitting Enter, Tab or clicking a friend to PM the chat entry enables
-Hitting Enter again disables the chat entry
-Does not affect/disable F keys, arrow keys, etc.

You know you don't need to make new Pull Requests to update it? Just commit to the branch the PR is on.

Thanks for letting me know :) Obviously still learning git.

It looks like you can just see if 162.0 has a keylistener or not to tell whether or not the chatbox input is active.

Added in #4874


Enforcing SSL?

Currently, http://nightcoders.net/ serving on http, https would make visitors feel more safe to use.

Yeah I actually made an attempt early on to enable SSL but I had trouble getting the boot-reload web socket connection to work (which is used to do hot code reloading). I plan on making another attempt pretty soon though.

On a totally unrelated note (I thought better to say this here than by adding noise by opening a new ticket)...
Great work, this!
I work full time with ClojureScript/Reagent and I keep rediscovering nightcoders.net every few months, whenever I feel a need to test something (reagent based, most times) out.
Thanks. Your work is appreciated.

That is so good to hear - - I don't even have traffic logs so I have no idea if anyone is using it! I haven't forgotten about the SSL issue by the way, just been distracted.


Netlify middleware

What is it?

[x] Feature / enhancement
[ ] Bug
[ ] Docs / tests

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Use cases and why

One use case

Another use case

Checklist:

[x] My code follows the developer guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] Added new tests to cover the fix / functionality

cc @nnelgxorz could you try to merge main and run yarn api.update ?

Done!


Benchmark v2

Description
In this PR was improving functional of ASVS pages.
What was improved:

Auto save checklist when we  check the boxes.
Add missing notes for checks.
Improvements in design.
Change design in benchmark block on product page.
Optimize code

@shipko this is terrific! Love the notes section.
I only have two nitpicky things that would be nice to haves:

Update the currently selected level in the header when the level is changed. It think this would have to be an Ajax thing to get updated counts and percentages, so maybe not worth the effort of adding a a new endpoint just for that)

The progress bar is a little misleading being totally colored in
It would be really neat to display progress bars/percentage for all levels (thinking from a ciso perspective in presenting numbers to c suite)

This is crudely thrown together via the dev tools, but this is sorta what I am talking about

@Maffooch I understanded you.
But not agree with your idea.
This is some point with my vision:

The currently UI is already overloaded. Your suggestion is adding more.
Currently progress bar is not representing currently status of asvs checking. We see "0.8% Complele", but what is complete? I changed it to three color bars. Once you will see, it will be enough to understand currently situation in project
Do you think that CISO will see product page? I think with this page will be working team leads, PMs, developers and AppSec. If they will want to see metrics by some levels they can change level on asvs page. Metrics will be loaded by ajax without reload page :)

@Maffooch I have problem with accesslint and checks. Can you help me?

Good counterpoints. Agreed my suggestion may not be the best approach for the project.

@Maffooch I have problem with accesslint and checks. Can you help me?

Just ricked the failing test. There a a handful of tests that fail erratically,  but are seldom due to code changes. I'll keep an eye on them until everything is passing

@damiencarol @mtesauro hi, guys!
Please, review my PR. I want to be sure that DD community is approving my vision and PR

I'm good with this - wanted to give @damiencarol a chance to review it.
Since we have the 2 needed approvals, if Damien doesn't provide feedback we can always merge later.


[HOLD #22884] Request Money - Error when requesting money from a user under no prior conversation history

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

User A:

Login as User A
Click on the green plus button >  New chat
Search for user B
Start a new conversation with User B, by sending a random message
Tap on the green plus button and request any amount of money

User B:

Login as User B
In user A's chat, request any amount of money

User A:

Repeat step 3

Expected Result:
No errors are returned when requesting money from a user with no previous conversation history
Actual Result:
Unexpected error when requesting money from a user under no prior conversation history
''Unexpected error requesting money, please try again later''
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

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

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/78819774/08f91f1f-a366-489c-9d29-511128460ce2

https://github.com/Expensify/App/assets/78819774/184ba18b-e9b6-4117-9cef-98a90c79c996
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

Dupe

Thanks @ShogunFire, putting on hold pending
https://github.com/Expensify/App/issues/22884

Coming from here, the issue that we were holding on.

Chatted 1:1 with @jasperhuangg and we decided to close this issue since it's no longer reproducible with the instructions in the OP. Please report a new bug if anything comes up.

Unable to reproduce.  @lanitochka17 can you try to see if you can? If you're unable to, can you close this please? Thx

Issue is not reproduced in Build on 1.3.62-4
https://github.com/Expensify/App/assets/78819774/487011a7-6dfd-43ea-8430-d72fd592b891


Contact card 3 made responsive

for issue #937

@Rajdip019  I have moved both the component of Book Appointment to Contact and Deleted Book Appointment folder

Great job man @RavindraP04


Increment Decrements option is not available in grouped product at shop.

Bug report
Title   Increment Decrements option is not available in grouped product at shop.
Issue Description
need to provide + - option to manage quantity
Preconditions
framework Version.-0.1.8
Steps to reproduce
step1- Create a grouped product
step2- check that product on shop
Expected result:
options should be available
Actual result:
not getting these options.

framework Version.-0.1.8
issue fixed


Implement testcases for archive

As mentioned in #44

Closing this as it is too difficult to implement testcases for this command.
This is because, we have to somehow know the new path of the archived file, but it is only available at runtime (and also the time is in 6dp which is too precise for us to hardcode it)


OpenBLAS-0.3.11 fails to compile on ppc64le

Updating the Fedora package of OpenBLAS to version 0.3.11 fails, since the package does not build on ppc64le. I'm getting a bunch of errors like
../kernel/power/sgemm_kernel_power10.c:131:11: error: '__builtin_mma_xvf32gerpp' requires the '-mmma' option
  131 |           __builtin_mma_xvf32gerpp (&acc2, rowB[i], rowA[j+1]); \
      |           ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

which means you should probably add this option to the makefiles.
Also, I noticed a bunch of macro redefinitions in the power8 kernels, e.g.
../kernel/power/dtrmm_kernel_16x4_power8.S:85: note: this is the location of the previous definition
   85 | #define STACKSIZE 320
      |
../kernel/power/dtrsm_kernel_LT_16x4_power8.S:51: warning: "STACKSIZE" redefined
   51 | #define STACKSIZE 520
      |
../kernel/power/dtrsm_kernel_LT_16x4_power8.S:50: note: this is the location of the previous definition
   50 | #define STACKSIZE 320
      |

which you may want to address.

Full build log
build.log.gz

Thanks, looks like 0.3.11 is going to set a new record for short-livedness. At least this mma error does not appear to be my fault (the redefinitions are, I'd left the old values in when I expanded the stack to save&restore additional registers). Good to know you have access to POWER10 though...

Ah, that is "only" a DYNAMIC_ARCH build on POWER8 ? Guess we may need to add some core-specific compiler options like it is already necessary for AVX512 on x86_64.

The POWER10 support is only included when GCC >10.2 , and unfortunately I cannot test my current PR on Fedora32 in the OpenPOWER cloud as even with gcc 10.2.1 and the latest binutils (2.34-5.fc32) I hit an assembler error invalid switch -mpower10
@RajalakshmiSR do we need to (test for) a specific as or binutils version for POWER10 as well ?

Answering my own question it appears binutils 2.35 would be required

Hmm, do not get the -mmma error on Debian11 ppc64le (which has gcc10.2 & binutils 2.35)

Suspect something in the way your patches modify the build options overrides or prevents the automatic addition of -mcpu=power10 -mtune=power10 -mvsx here https://github.com/xianyi/OpenBLAS/blob/5e8ddc9001d705473e106cf7ff285d170b7e8944/Makefile.power#L13

checking

We need gcc10.2 or gcc11 and binutils 2.35 to compile power10 files.

May need to add a version test for as then (not that I like the idea, but that is what anyone on fedora32 would probably encounter right now). Main problem seems to be that his build options are somehow removing/preventing the addition of the -mcpu=power10 -mvsx, but perhaps that line in Makefile.power just needs to be override COMMON_OPT +=... for good measure.

So are you suggesting to check for binutils version before we add POWER10 in the DYNAMIC_ARCH list in Makefile.system?

Yes, thinking about that at least, as we cannot be sure that gcc and binutils are updated together.

I will create a PR for this.

@susilehtola Is it possible to apply this patch on your fedora system and compile/check if it helps.
diff --git a/Makefile.system b/Makefile.system
index 461f7370..44a9bbf5 100644
--- a/Makefile.system
+++ b/Makefile.system
@@ -648,12 +648,17 @@ DYNAMIC_CORE += POWER9
else
$(info, OpenBLAS: Your gcc version is too old to build the POWER9 kernels.)
endif
+LDVERSIONGTEQ35  := $(shell expr ld --version | head -1 | cut -f2 -d "."  | cut -f1 -d "-" >= 35)
ifeq ($(GCCVERSIONGTEQ11), 1)
+ifeq ($(LDVERSIONGTEQ35), 1)
DYNAMIC_CORE += POWER10
+endif
else ifeq ($(GCCVERSIONGTEQ10), 1)
ifeq ($(GCCMINORVERSIONGTEQ2), 1)
+ifeq ($(LDVERSIONGTEQ35), 1)
DYNAMIC_CORE += POWER10
endif
+endif
else
$(info, OpenBLAS: Your gcc version is too old to build the POWER10 kernels.)
endif

@susilehtola Can you also confirm your gcc and binutils version ?

Looking at the log, it is clear that -mcpu=power10 is not passed here. So the actual error is not due to binutils/gcc versions
.
gcc -O2  -fexceptions -g -grecord-gcc-switches -pipe -Wall -Werror=format-security -Wp,-D_FORTIFY_SOURCE=2 -Wp,-D_GLIBCXX_ASSERTIONS -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -fstack-protector-strong -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1  -m64 -mcpu=power8 -mtune=power8 -fasynchronous-unwind-tables -fstack-clash-protection -fPIC -DMAX_STACK_ALLOC=2048 -DUSE_LOCKING -Wall -m64 -DF_INTERFACE_GFORT -fPIC -DDYNAMIC_ARCH -DDYNAMIC_OLDER -DNO_WARMUP -DMAX_CPU_NUMBER=128 -DMAX_PARALLEL_NUMBER=1 -DBUILD_SINGLE=1 -DBUILD_DOUBLE=1 -DBUILD_COMPLEX=1 -DBUILD_COMPLEX16=1 -DVERSION="0.3.11" -UASMNAME -UASMFNAME -UNAME -UCNAME -UCHAR_NAME -UCHAR_CNAME -DASMNAME=sgemm_kernel_POWER10 -DASMFNAME=sgemm_kernel_POWER10_ -DNAME=sgemm_kernel_POWER10_ -DCNAME=sgemm_kernel_POWER10 -DCHAR_NAME="sgemm_kernel_POWER10_" -DCHAR_CNAME="sgemm_kernel_POWER10" -DNO_AFFINITY -DTS=_POWER10 -I.. -DBUILD_KERNEL -DTABLE_NAME=gotoblas_POWER10 -UDOUBLE  -UCOMPLEX -c -UDOUBLE -UCOMPLEX ../kernel/power/sgemm_kernel_power10.c -o sgemm_kernel_POWER10.o

@RajalakshmiSR please post a proper patch, that one is broken.
Also, the flags are determined by Fedora policy as
COMMON="%{optflags} -fPIC"
FCOMMON="$COMMON -frecursive"
make -C serial     $TARGET USE_THREAD=0 USE_LOCKING=1 USE_OPENMP=0 FC=gfortran CC=gcc COMMON_OPT="$COMMON" FCOMMON_OPT="$FCOMMON" $NMAX LIBPREFIX="libopenblas"      $AVX $LAPACKE INTERFACE64=0

If some object files need special mcpu flags, they should be specified in the respective make rule.

Suspect something in the way your patches modify the build options overrides or prevents the automatic addition of -mcpu=power10 -mtune=power10 -mvsx here
https://github.com/xianyi/OpenBLAS/blob/5e8ddc9001d705473e106cf7ff285d170b7e8944/Makefile.power#L13

@martin-frbg The only difference in passing the flags between power9 and power10 in Makefile.power is Power9 is using  CCOMMON_OPT whereas power10 uses COMMON_OPT.

@susielethola can you try changing that line in Makefile.power that we both quoted above so that it starts with override before the CCOMMON_OPT ?

@susilehtola can you try changing COMMON_OPT to CCOMMON_OPT in OpenBLAS/Makefile.power line 13

@RajalakshmiSR of course you are right, I seem to have overlooked the missing "C". Updated my PR #2911 accordingly but have not added the binutils version check in Makefile.system yet.

@martin-frbg adding the override fixes that issue, but now I see there's a linking problem, see #2922

@RajalakshmiSR new build running

@susilehtola thanks, but actually Rajalakshmi spotted the actual problem and proposed the correct solution hours before I wrote mine without properly reading. Giving that variable an extra C is the way to go.
The link errors are another sad saga that has seen several partial fixes by now - in short I had introduced multiple typos and logic errors in the gensymbol script when I rearranged functions by argument type, and there was nothing missing (i.e. a missing blank in a crucial spot) in its invocation from the Makefile as well. #2909 and #2919 fix most of it but I am still fixing more.

@RajalakshmiSR of course you are right, I seem to have overlooked the missing "C". Updated my PR #2911 accordingly but have not added the binutils version check in Makefile.system yet.

@martin-frbg Is it possible to try (manually edit Makefile.system) this https://github.com/xianyi/OpenBLAS/issues/2911#issuecomment-711439168 in your openpower cloud system (fedora32) which has new gcc and older binutils..  (to check if it fixes that invalid switch error in https://github.com/xianyi/OpenBLAS/issues/2911#issuecomment-711412747).

Yes, thinking about that at least, as we cannot be sure that gcc and binutils are updated together.

This is a generic problem and I think we don't have to fix this in a hurry only for POWER10 for 0.3.11 release. We can create a separate issue and discuss this.

Certainly, and I expect it will fix the problem.

Meh, this leads to a new problem later as dynamic_power.c does not "know" that gotoblas_POWER10 did not get built (as it only has ifdefs for __GNUC__). I guess the tests for defining HAVE_P10_SUPPORT should be moved into c_check or getarch (or as a quick and dirty fix, that spot in Makefile.power should also add -DHAVE_P10_SUPPORT to CCOMMON_OPT)

@martin-frbg Yes.. (expected)... this needs more testing with different combinations of compiler versions. I can fix this post release as I do not want to introduce new bugs by adding this.

I'll test moving the define HAVE_P10_SUPPORT from dynamic_power.c to Makefile.power for now - this would at least be consistent and should not hurt non-DYNAMIC_ARCH builds (famous last words)

With the current master,

Non dynamic arch builds are not affected.
DYNAMIC_ARCH is affected only when binutils is old and gcc is new.

Yes, I'm only worried about (2) because that seems to be the current state of the ppc64le Fedora. Of course we can leave it at that for now, I certainly do not want two broken releases in a row.

@martin-frbg can you update binutils on your fedora32. ?

I could certainly build from source, the only issue is that there appears ot be no binary update package available (yet ?) via their normal update mechanism.

I'll test moving the define HAVE_P10_SUPPORT from dynamic_power.c to Makefile.power for now - this would at least be consistent and should not hurt non-DYNAMIC_ARCH builds (famous last words)
@martin-frbg
Are you suggesting this for now?
patch.txt

Yes, with an additional "define HAVE_P10_SUPPORT" a few lines above, in the ifneq ($(C_COMPILER), GCC) block so that it matches the previous (intended) behaviour for other compilers as well. Would you agree to this ?

Yes.

Thx. This is #2925 now (though I notice we'll need a proper check in the cmake part eventually)

@susilehtola can you try changing COMMON_OPT to CCOMMON_OPT in OpenBLAS/Makefile.power line 13

Doesn't this still need to be done though?
Will you be releasing 0.3.12 soon or should I backport the changes to 0.3.11?

Doesn't this still need to be done though?

That was the "fix power10 option passing" bit of #2916
I do not intend to sit on the steaming pile of ... that was 0.3.11 for longer than I have to, I am just currently trying to convince myself that what we have now has no glaring bugs left. I was prepared for a bit of fallout from the big makefile reshuffle, but I certainly did not expect anything like this. And besides hating myself for rushing the release after becoming target-fixated with getting all Travis checks passing again, I am a bit worried that none of the current Ci tests gave an unmistakable warning.


Add insert_batch and variations

Objective

insert_or_spawn_batch exists, but a version for just inserting doesn't
Closes #2693
Closes #8384
Adopts/supersedes #8600

Solution

Add insert_batch, as well as the variations that insert has: insert_batch_if_new, try_insert_batch, and try_insert_batch_if_new

Testing

Added tests, and added a benchmark for insert_batch. Performance seems to be identical to insert_or_spawn_batch when only inserting:

Showcase

Usage is the same as insert_or_spawn_batch:

use bevy_ecs::{entity::Entity, world::World, component::Component};
#[derive(Component)]
struct A(&'static str);
#[derive(Component, PartialEq, Debug)]
struct B(f32);

let mut world = World::new();
let entity_a = world.spawn_empty().id();
let entity_b = world.spawn_empty().id();
world.insert_batch(vec![
    (entity_a, (A("a"), B(0.0))),
    (entity_b, (A("b"), B(1.0))),
]);

assert_eq!(world.get::<B>(entity_a), Some(&B(0.0)));

This would be hugely helpful for deprecating insert_or_spawn_batch, which is part of #15459. I do however want to point to #15614 as an example of a way to add this functionality without doubling the API surface. Can you look if this may be possible for insert variants as well?

I'm far from an expert, but I'm not sure if it's doable in the same way.
The World::entity variations deal with Single vs Plural, Ref vs Mut, and Static vs Dynamic, all things that work nicely with Rust types.
The insert variations are Panic vs Silent and Replace vs Keep, which I can only think of ugly solutions for.
I'd imagine that we would want to include all the other insert functions, to keep things consistent. I've seen ideas about chaining commands to achieve a similar nice API, which would work for the insert Commands, but I don't know if the functions on the World can do that.
The best I've got is passing in an enum or something, but that's obvious so there's probably a reason that's not how it works already

Maybe this could be its own PR, but the original PR has some unhealthy usage of UnsafeWorldCell, so I think this is a worthwhile fix.
Here's the context:
The problem was that I needed access to both the World's Entities storage and a &mut World to make a new BundleInserter.
insert_or_spawn_batch snuck around this by making a BundleSpawner first, which internally made an UnsafeWorldCell that was then able to be used to access Entities while still using a &mut World.
insert_batch, of course, doesn't need a BundleSpawner. However, if you try to recreate the effect with a BundleInserter, you'll find that it needs an ArchetypeId, which you get from an EntityLocation... which you get from Entities.
I originally used an UnsafeWorldCell directly to get around this, which worked, but it turns out the documentation specifically says to not do that.
So, eventually I figured out a (hopefully) proper solution: deal with the first entity in the iterator on its own, so you can make a BundleInserter with its ArchetypeId without triggering the borrow checker, and without dealing in unsafe (well, unsafe unsafe) operations.
And as a bonus, it's slightly faster now!

Should I remove the try_ variants from the World? I've realized that they're only really useful for Commands

I think keeping multiple inserters allocated would run into borrow problems, since they all need a &mut World

Should I remove the try_ variants from the World? I've realized that they're only really useful for Commands

I'm not sure I follow. We do have a try_despawn on world. I think it's more unlikely that an entity might not exist when working with World, but not impossible. You could be referencing an entity that is manually updated in a resource.

I think keeping multiple inserters allocated would run into borrow problems, since they all need a &mut World

It would definitely require something different from BundleInserter and probably a bunch of unsafe.

I got the #15614 thing working, using this trait:

Trait implementation
use core::iter;
use crate::{bundle::Bundle, entity::Entity};

/// Types that can be used to insert [`Bundles`](Bundle) into [`Entities`](Entity) from a [`World`].
///
/// Provided implementations are:
/// - (`Entity`, `Bundle`): Insert into a single entity.
/// - [(`Entity`, `Bundle`); N]: Insert into a static array of entities.
/// - Vec<(`Entity`, `Bundle`)>: Insert into a vector of entities.
///
/// [`World`]: crate::world::World
pub trait WorldEntityInsert<B: Bundle>
{
    fn iterator(self) -> impl Iterator<Item = (Entity, B)>;
}

impl<B: Bundle> WorldEntityInsert<B> for (Entity, B)
{
    fn iterator(self) -> impl Iterator<Item = (Entity, B)> {
        iter::once(self).into_iter()
    }
}

impl<B: Bundle, const N: usize> WorldEntityInsert<B> for [(Entity, B); N]
{
    fn iterator(self) -> impl Iterator<Item = (Entity, B)> {
        self.into_iter()
    }
}

impl<B: Bundle> WorldEntityInsert<B> for Vec<(Entity, B)>
where
    B: Bundle
{
    fn iterator(self) -> impl Iterator<Item = (Entity, B)> {
        self.into_iter()
    }
}

It doesn't reduce the API, but it adds the ability to insert into a single entity or an array of entities, instead of just a Vec. Since insert_batch uses an iterator, it was pretty much a drop-in replacement, no changes to the actual function.
So, follow-up questions:

Where should the trait live? I made an entity_insert.rs to write it, but I don't know if it needs a whole file
What should the functions be called? insert_batch isn't quite accurate since you can insert just one, but it's still probably mostly used for batches
Are there any other types that should be able to work? Maybe slices, but those are references and I'm not sure could/should work


[BUG] Moving a Mount While Both Mount and Mounter are Selected Breaks Alignment

Describe the bug
A clear and concise description of what the bug is.
If you move a mount and the token mounting it at the same time by having both selected it breaks the alignment and the smaller token drifts around inside the larger one. In my experience it typically drifts to the upper left corner.
Environment Details
Foundry VTT Version: 0.6.1
Mount Up! Version: 3.0.1
To Reproduce
Steps to reproduce the behavior:
_1. Mount one token on another
2. With both still selected move the tokens
3. Move them several more times and they shift around on top of each other. Typically drifting to the top left. It's especially noticable with center alaignment.
Expected behavior
They should stay locked to the set alignment.
Screenshots
If applicable, add screenshots to help explain your problem.
Before moving:

After moving:

So after some investigation, this is not related to Mount Up, but actually the way Foundry handles token placement. If shift is not held during token movement, a token will snap into the closest grid spot.
Since you are 'controlling' a token that is selected, Foundry movement rules take over. At current, I can only suggest a 'workaround' of only selecting the mount token like normal mount movements.
I will continue to do some investigation and see if I can hack around it in the future.
See example with one token:

Here is two tokens stacked, but not mounted in any way:

This doesnt happen with the old version of the module, I have reinstalled it manually and it works perfect.
https://i.imgur.com/HCmC5NU.gif


generated python code not compatible with python3

The code generated by grpc_python_plugin uses the python2 __metaclass__ syntax which is not compatible with python3.

One possibility could be to use six.with_metaclass(abc.ABCMeta); however that would mean importing the external package six in the pb2 files. If that's not an option, we could write our own helper function that creates stubs and servicers with the ABCMeta metaclass.

I'm a fan of orthodoxy and staying on the beaten path; I think what's emerged as the de facto consensus for writing 2/3 code is importing six in modules that need it. That these particular modules that need it are generated doesn't strike me as a reason to change that.

Sounds good to me. I'll make the change in the morning.
Le 2016-03-10 22:27, "Nathaniel Manista"<EMAIL_ADDRESS>a
écrit :

I'm a fan of orthodoxy and staying on the beaten path; I think what's
emerged as the de facto consensus for writing 2/3 code is importing six
in modules that need it. That these particular modules that need it are
generated doesn't strike me as a reason to change that.
—
Reply to this email directly or view it on GitHub
https://github.com/grpc/grpc/issues/5654#issuecomment-195153404.

This should be resolved by #5657


wipeout can't cover line in blockview

see the picture below, the wipeout can't cover the line, how to make the wipeout show correct?

I try to set the render type to full, but it will make autocad crash.

By the way, I use Autocad2020 to test.

Thanks, can you attach the test drawing here. ?

Thanks, can you attach the test drawing here. ?
Drawing1.zip
Here is the test drawing, only a line and wipeout.
and the kFullRender does not crash every time, it seems cause by other reasons, just forget it.


mypy: Add annotations to zephyr_mirror_backend.py

Automated message from Dropbox CLA bot
@anirudhjain75, it looks like you've already signed the Dropbox CLA. Thanks!


Unable to properly uninstall on Redmine 3.0.4 | ruby 2.2.1 via RVM

Hi jfqd
This plugin seems not work for me (Redmine 3.0.4 | ruby 2.2.1 via RVM), so I tried to uninstall it by using the standard command: "rake redmine:plugins:migrate NAME=redmine_helpdesk VERSION=0 RAILS_ENV=production"
This action aborted caused by the following errors:
== 5 AddTreatAsSupportclientToAnonymous: reverting ============================
rake aborted!
StandardError: An error has occurred, all later migrations canceled:
Couldn't find Role with 'id'=all/home/caiex/.rvm/gems/ruby-2.2.1/gems/activerecord-4.2.3/lib/active_record/relation/finder_methods.rb:324:in raise_record_not_found_exception!' /home/caiex/.rvm/gems/ruby-2.2.1/gems/activerecord-4.2.3/lib/active_record/relation/finder_methods.rb:444:infind_one'
Correspondingly those text fields coming along with this plugin can not be removed, while I don't want to get hands dirty by tweak mysql for removing these.
So could you fix this?
Thanks,
Cross

problem should comes from 005_add_treat_as_supportclient_to_anonymous.rb:
def self.down
Role.find(:all).each do |r|
r.remove_permission!(:treat_user_as_supportclient)
end
end

solved by:

def self.down
#Role.find(:all).each do |r|
r.remove_permission!(:treat_user_as_supportclient)
#end
Role.find(2).remove_permission!(:treat_user_as_supportclient)
end

I don't think this solution applies if the :treat_user_as_supportclient permission has been set to other users than anonymous.
It looks like the root cause is Rails 4.2 incompatibility. The migrations should be part of the test set to ensure compatibility.

you are right, Vilp.
I just have only one id who was added this permission. So I manually remove this plugin by the above way.

Old issue, but I had this issue in Redmine 3.3.x. Thanks for the solution.


Set label color

Is it possible to set a color for the label of the row ? Currently only task labels are supported.
It would be nice to to do so instead of customizing the templates.

You can use a CSS class in a Row model, and then style the row as you want.
Even if color property seems handy, I'm not sure we should support them, because CSS styling is really more powerful and wide.
I'll vote to just deprecate "color" property support from Task and Row model. What's your opinion @Schweigi ?

Sorry Toilal. I meant to set the color of the label section (e.g. coloring a tree label). Is this also possible via CSS ?

Yes, you can setup one or many class names on Row model using classes property, and then this class will appear in the DOM on each DOM element representing this Row (label included).
The class will be applied on multiple DOM elements used to display the row (the background, the foreground, ...), so you may need to add more classes to select only the label, but it should be possible.
No problem @kschwidder, asking questions is just fine :)


Add logging for user permissions

Stores the current permissions of the current user on every request but only creates a new database entry when the permissions change.
Provides two queries which can be used for the adminstration panel.

COM-664

Query logins of users

Query permissions of user

@fraxachun I've set this PR to draft since we discussed this in last week's meeting and weren't sure if we even need the user-facing implementation.


python3.pkgs.plux: 1.3.1 -> 1.4.0

Description of changes
https://github.com/localstack/plux/compare/a412ab0a0d7d17c3b5e1f560b7b31dc1876598f7...v1.4.0
Also pull in a patch to remove pytest-runner from setup_requires and add some build dependencies.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

[x] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

@ofborg build python310Packages.plux python311Packages.plux

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

12 packages built:

localstack (python310Packages.localstack)
localstack.dist (python310Packages.localstack.dist)
python310Packages.localstack-ext
python310Packages.localstack-ext.dist
python310Packages.plux
python310Packages.plux.dist
python311Packages.localstack
python311Packages.localstack-ext
python311Packages.localstack-ext.dist
python311Packages.localstack.dist
python311Packages.plux
python311Packages.plux.dist


Bookmark for YouTube links not working

Issue Summary
Adding a bookmark card with a link to YouTube seems to have broken in v5.86.0.
Previously, adding a bookmark card with a valid link to a YouTube video resulted in this:

Response from XXX in v5.85.2:
{
    "version": "1.0",
    "type": "bookmark",
    "url": "https://www.youtube.com/watch?v=aqz-KE-bpKQ",
    "metadata": {
        "url": "https://www.youtube.com/watch?v=aqz-KE-bpKQ",
        "title": "Big Buck Bunny 60fps 4K - Official Blender Foundation Short Film",
        "description": "Enjoy this UHD High Frame rate version of one of the iconic short films produced by Blender Institute!Learn more about the project here http://bbb3d.renderfa…",
        "author": null,
        "publisher": "YouTube",
        "thumbnail": "https://i.ytimg.com/vi/aqz-KE-bpKQ/maxresdefault.jpg",
        "icon": "https://www.youtube.com/s/desktop/9f65bf98/img/favicon_144x144.png"
    }
}

From v5.86.0 onwards (tested in v5.86.1 as well), the same link in a bookmark card produces an error:

Response from XXX in v5.86.0 and v5.86.1:
{
    "errors": [
        {
            "message": "Validation error, cannot read oembed.",
            "context": "URL contains insufficient metadata. https://www.youtube.com/watch?v=aqz-KE-bpKQ",
            "type": "ValidationError",
            "details": null,
            "property": null,
            "help": null,
            "code": null,
            "id": "8c6cb290-3160-11ef-bc69-01c02bf2f945",
            "ghostErrorCode": null
        }
    ]
}

Steps to Reproduce

Create a new post in v5.86.0 or v5.86.1
Add a bookmark card
Enter any YouTube URL (for example https://www.youtube.com/watch?v=aqz-KE-bpKQ)
Error is shown

Ghost Version
v5.86.0
Node.js Version
18 LTS
How did you install Ghost?
Ubuntu 22.04 with Ghost CLI
Database type
MySQL 8
Browser & OS version
No response
Relevant log / error output
No response
Code of Conduct

[X] I agree to be friendly and polite to people in this repository

Closed via https://github.com/TryGhost/Ghost/pull/20446. Released in 5.68.2.
Thanks for the quick report!

Thanks for the quick fix!


[CDS-54172]: Support for org/account scopes service overrides in Environments

Describe your changes
This PR depends on new release of harness-go-sdk ( v0.3.21 ). The current available release is (v0.3.20). The dependant change which is needed is here: https://github.com/harness/harness-go-sdk/pull/299
Comment Triggers

PR Check triggers

Build: trigger build
Sub Category Field Check: trigger subcategoryfieldcheck

I guess some fields in schema have changed please generate docs and add changelog file corresponding to this pr's number

Done


Quotes should not be part of the enum's identity

That just looks wrong. An enum value is not "example". It's "example".

An enum value is always and forever a string, which is why I'm including the quotes currently.  Why do you think it should exclude the quotes?

I think quotes should be there, but not as part of the value. The syntax used to denote strings can vary, so what you want to copy and reuse is example, not "example".

I don't understand. In what way can the syntax to denote strings change?

Yeah, the problem is that " is syntax. I don't like including that in the term.

All I can say is that I strongly disagree. The other IDL terms are either terms of art, or correspond to idents in JS, both of which obviously get used "directly".  Enum values correspond to strings in JS; you'd never use them without quotes.
You can override the handling if you really want to - provide a <dfn enum-value for="MyEnum" lt='"foo" | foo'>foo</dfn> for all the enum values.  That'll give you the linking text you want, and make sure the IDL recognizes the dfn.
Or I guess I can have enum-values automatically provide both linking texts when they're parsed in IDL, so you can use either.

Maybe I should put this another way, can the markup look like "string"?

Yes, if you create that markup pattern yourself. ^_^
I'm just going to do what I said above - have enum-values define themselves both with and without quotes.

Done.


[No QA] Add React Native Firebase Performance monitoring

Details
This PR adds some new events to the Firebase dashboard for Expensify/App:

App start time - how long the app takes to launch
js_loaded custom event - how long the app takes to launch + for the main app component to mount

This should give us some valuable metrics for detecting material improvements or regressions to app start time.
Fixed Issues
$ https://github.com/Expensify/Expensify/issues/171518
Tests
The metrics are disabled when in DEBUG to avoid logging bad values for now. Dev builds take considerably longer to start up because the JS is bundled in metro and the app is technically "launched" while that's happening.
To get around this we need to create release builds. Instructions for how to do that are here:
How do I run iOS Native Chat release builds on the iOS Simulator?
How do I run Android Expensify.cash release builds in Android Studio?
iOS / Android

Log into firebase dashboard
Build the app
Wait til it loads and force quit
Repeat a few times.
Note that the Performance dashboard has updated with new times

To verify we are only running this in debug we can build the apps normally (debug) and grep for this log:
[StartupTimer] Metric tracing disabled in DEBUG

QA Steps
No QA
Tested On

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

Screenshots

Web

Mobile Web

Desktop

iOS

Android

I wonder if it might be worth migrating the Grafana timers to Firebase too later on -- as Firebase is much more closely tied to the app dashboards and perhaps more likely to be used.

Good thought @Julesssss! I think we can pretty easily move some of the Grafana stuff to Firebase. The only thing is Firebase will not tell us anything about web - so if we want to move some of the traces to Firebase then they will be split between the two platforms. Not necessarily a bad thing. I think it's more likely we'll be worried about native performance than web performance in the long run anyway.

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

🚀 Deployed to staging         in version: 1.0.79-5🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production         in version: 1.0.80-2🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Reenable tabularView page

Page: https://trss.adoptium.net/tabularView
Source: https://github.com/adoptium/aqa-test-tools/tree/master/test-result-summary-client/src/TabularView
tabularView Page was created for comparing two builds (test vs baseline). However, this page has been broken for a while.
A couple of thoughts:

debug the current code
debug after reverting the auto-generated code introduced by npm: upgrade several dependencies first, so we have a relatively stable state to work with.

https://github.com/adoptium/aqa-test-tools/commit/aaa9210f39f08bf7c5e8899bb1ee56875a11e332#diff-42bd09343fc1b89e7de03ba2f137e9dc901f3c852fe198b57afdb11a2ea1e44f

rewrite it completely

I am currently working on this issue with three other team members


Update docs and testing for Globus extract support

#173 was resolved through #154. However #154 doesn't include information on Globus-extract in docs/source/usage.rst nor is it tested in tests/test_globus.py. These should be resolved before officially supporting Globus on extract.

#191 addresses the documentation component of this issue.


how to disable particular tree node checkbox

I use treeTable with checkbox, but i want to disable all checkbox of child node.
How can I achieve this?
If do this as official doc says: 'Selection of a particular node can be disabled by setting the selectable property of the node to false.', how to do this in practice, in html or in typescript code?
thank you very much

Hi Anne, I'm assuming you are using the build-in checkbox mode, which unfortunately does not support disable particular node atm.
However, you can implement something like this:
<ay-column field="name" header="Name">
    <template let-col let-node="rowData" pTemplate="body">
        <input type="checkbox" [disabled]="node.parent">
    </template>
</ay-column>

Hi, thx for replying,
I just figure out how to disable build-in checkbox:
Assume my tree data are follows:
data=[     {         data:{name:'aa'},         children:[             {data:{name:'ab'}}         ]     } ]
I want to set my children node undelectable, do like this:
data[0].children[0].selectable=false
Then, children node cannot be selected in UI (it works for my project), but the style of the checkbox does not change. I have to add a styleClass to hide it (of course, you can set other CSS styles).

Glad you have worked it out:)
Of course you can reset the data model and add a new property 'selectable' to each node, but It would potentially slow down your app if the data amount is huge.
My advise did not require any changes on data model, thus it could be much faster.


Added Azure Blob Storage documentation

Created Azure Blob Storage documentation. Used existing S3 document as a starting point to maintain consistency

Thanks for the contribution, @yorek. Could you also create a JIRA and add it to the title of the PR?
@bbevens @arina-ielchiieva could you please review?

@priteshm Done


INC-860 Fix DPS incentive level details page when history includes system generated reviews

For unmatched records we now visually get "not entered" which we can iterate on by stops the screen crashing

might be worth having a unit test to show the page loading should the prison api return 404 on a user lookup?


Ian/report content

Change description
Added content to fairness and dataset reports.

Good point. I think I'd like to add a "development mode" and run all notebooks using a test script


Create timer error - segmentation fault

Hey there,
I've only had this issue on production and not locally or in a staging environment so I haven't been able to enable debug for this.
My application gets around 40k requests per second, memory usage was fine but the CPU usage was spiking frequently making the instance crash. (ECS fargate with 2vCPU and 4G of RAM).
With php-fpm I can have 3-4 instances to handle that much traffic and with FrankenPHP it needed more for some reason.

I have reverted to php-fpm for now but I just wanted to show you what I experienced.
As far as the app, it caches lists of images / videos to display. With frankenPHP I used array cache and with php-fpm I'm using APCu.
I'm not really sure how to debug segmentation faults in production without having to start a lot of ECS instances but I'm open to suggestions
Thanks!

Caddyfile
{
	frankenphp
	order php_server before file_server
}

:80, :{$PORT} {
	root * /app/{$PUBLIC_DIR}

	php_server

	header /assets/* {
		Access-Control-Allow-Origin "*"
		Cache-Control "public, max-age=31536000"
	}

	log {
		output stdout
		format console
		level error
	}

	push
	encode zstd gzip
}

This looks like a duplicate of #249.
Can you check if https://github.com/dunglas/frankenphp/issues/249#issuecomment-1752116059 fixes the issue? Thanks.

@dunglas I'll test that and let you know the results.
Are you aware of any issues with APCu? it seems the memory usage was a lot higher on my end, I wonder if the APCu cache is per process or shared across all of them.

I don't know for APCu.

APCu is per unique parent / pre-fork PHP process. The author talked about it @ https://github.com/krakjoe/apcu/issues/121#issuecomment-136091336

If your process manager works like conventional FCGI/CGI then you will not be able to share (memory), if it works like FPM, and initializes PHP in a parent and forks child interpreters then you will have no problem.

If I am reading the interaction between go-land and c-land correctly, we're maintaining a thread pool and then calling php_execute_script in each thread pool, which I think means APCu won't be sharing memory across different worker threads.

Do you know at which point APCu cache is initialized? We execute the PHP executor boot logic in the main PHP thread before creating children threads.

I do not, sadly, I did notice that and was unsure on exactly when you'd have to thread to make APCu share memory across different request workers. We could probably ask in APCu? I'm reading up in https://github.com/krakjoe/apcu/blob/master/TECHNOTES.txt  and that makes it sound like in the MINIT phase, but there's some things in there that don't quite fit with my understanding of APCu, so maybe I've been living a lie for years, or maybe I'm not quite understanding the implication of those notes.

I just tested, APCu definitely works for franken in both worker and non-worker mode. Test repo @ https://github.com/SpencerMalone/apcu-test-prefork-franken. I'm not sure I understand why, but APCu definitely do be working for franken

The segfault has been fixed by https://github.com/php/php-src/pull/12537

I'm having the same issue. After finally successfully building FrankenPHP with plugins, The only alteration I made to my Caddyfile was adding frankenphp and order php_server before file_server to my global options then commenting php_fastcgi and file_server in every site block and replacing them with php_server. I haven't configured it at all, sure, but this doesn't seem like normal/expected out of the box behavior.
Caddy starts normally, serves websites normally, but trying to run a page speed test makes it crash with the same error as this issue:
Jun 26 00:07:40 caddy[620113]:

Jun 26 00:07:40 caddy[620113]: Fatal error:  Could not create timer: Resource temporarily unavailable (11) in Unknown on line 0

Watching htop while running the test showed me spikes in CPU and memory usage caused by Caddy reaching over 95%. This has never happened on Apache or even Caddy with php_fastcgi even when running load tests with thousands of connects, let alone such a mundane single page load speed test.
Here's my version in case it's relevant:
v2.8.4 h1:q3pe0wpBj1OcHFZ3n/1nl4V4bxBrYoSoab7rL9BMYNk=
(I find it extremely odd that it's not saying FrankenPHP in the version like some of my other builds, but it's correctly accepting the frankenphp global option so what do I know)
This is the command I used to build it:
CGO_ENABLED=1 XCADDY_GO_BUILD_FLAGS="-ldflags '-w -s'" xcaddy build     --output frankenphp     --with github.com/dunglas/frankenphp/caddy     --with github.com/dunglas/vulcain/caddy           --with github.com/caddy-dns/cloudflare             --with github.com/caddyserver/replace-response             --with github.com/greenpau/caddy-security             --with github.com/hslatman/caddy-crowdsec-bouncer             --with github.com/ggicci/caddy-jwt             --with github.com/ueffel/caddy-basic-auth-filter             --with github.com/pberkel/caddy-storage-redis             --with github.com/teodorescuserban/caddy-cookieflag             --with github.com/ltgcgo/floaty             --with github.com/baldinof/caddy-supervisor             --with github.com/quix-labs/caddy-image-processor

And this is my systemd file:
[Unit]
Description=Caddy Web Server
Documentation=https://caddyserver.com/docs/
After=network.target

[Service]
User=apache
Group=apache
#ExecStart=/usr/local/bin/caddy run --config /etc/caddy/Caddyfile --adapter caddyfile
#ExecReload=/usr/local/bin/caddy run --config /etc/caddy/Caddyfile --adapter caddyfile
ExecStart=/etc/caddy/caddy run --config /etc/caddy/Caddyfile --adapter caddyfile
ExecReload=/etc/caddy/caddy run --config /etc/caddy/Caddyfile --adapter caddyfile
TimeoutStopSec=5s
LimitNOFILE=1048576
LimitNPROC=512
Environment="LD_LIBRARY_PATH=/etc/caddy/php/lib:$LD_LIBRARY_PATH" #custom built version of PHP as per documentation, version 8.3

[Install]
WantedBy=multi-user.target

Version of custom PHP build:
/etc/caddy/php/bin/php -v
PHP 8.3.0 (cli) (built: Jun 25 2024 00:59:46) (ZTS)
Copyright (c) The PHP Group
Zend Engine v4.3.0, Copyright (c) Zend Technologies

I'm not sure if this is relevant, but while troubleshooting I discovered that this custom build of PHP did not have an .ini file. I created one with the php.ini-production file from PHP's official Github repo and tried to point FrankenPHP at it (using PATH and environmental variables passed in the systemd service file) but it kept looking for it at an old directory (I built PHP in a directory then moved it to a different directory). So I created the directory it's looking for and placed the .ini file where it's looking for it and running php --ini shows that it's detecting it, but that hasn't fixed the problem.


HugePages Support [DO NOT MERGE]

Additional configuration changes made to utilise hugepages.

This is not needed necessarily, If cluster supports hugepages, they can update the existing Authentication CR with huge-pages configuration and that should work.
hence closing this PR


HDDS-9731. Fix false positive findbugs warnings in CodecBuffer

What changes were proposed in this pull request?
Fix Return value of ... ignored, but method has no side effect warnings in CodecBuffer.
Examples:
https://github.com/vtutrinov/ozone/actions/runs/6655766005/job/18086771096#step:7:11
https://github.com/xichen01/ozone/actions/runs/6925852588/job/18837148517#step:7:11
The methods being called do have side effects, they return the same buffer only for convenience.  CodecBuffer.buf is final, we cannot update it to the buffer returned even if it's another instance.
assertSame call is added to make use of the return value, and also to alert if the implementation is changed to return any other instance.
https://issues.apache.org/jira/browse/HDDS-9731
How was this patch tested?

Checked out https://github.com/xichen01/ozone/commit/1589e17d0ab64d12064050c73a75bfba52b62132
Ran findbugs.sh locally to reproduce the problem
Applied this patch
Ran findbugs.sh again to verify the warnings were no longer reported

CI:
https://github.com/adoroszlai/ozone/actions/runs/6935777427

Thanks @szetszwo for the review.


Inconsistent execution of custom script extensions on VMs within a scale set

I notice that when I provision a VM Scale Set with a powershell custom script extension (using an ARM template) that custom script extension will run each time I deallocate and then restart the entire VM scale set from the Azure portal.  But if I deallocate a single VM instance of the Scale Set and then restart that single VM the custom script extension does not run.  This seems inconsistent but not sure if it is by design.

A custom script extension should no run when you deallocate/restart a scale set. It should run only when you create new VMs or make a change to the extension definition. Do you have an example template which shows this problem?

Sure!  You can use this deploy template below, it is for a regular VM but you will see the same issue:
https://github.com/Azure/azure-quickstart-templates/tree/master/201-vm-custom-script-windows
Repro Steps:

Deploy the VM using the template here:
https://github.com/Azure/azure-quickstart-templates/tree/master/201-vm-custom-script-windows
Once the deployment is complete remote onto the VM and notice that the CSE script copied a file to this location:  C:\WindowsAzure\FileToBeCopied.txt
Delete the file:  C:\WindowsAzure\FileToBeCopied.txt
Stop the VM in the Azure Portal by clicking the Stop button
After the VM has stopped then click the Start button
Remote onto the VM, open windows explorer, and navigate to this location:
C:\WindowsAzure

Expected Results:  The FileToBeCopied.txt file should not be present since you deleted it and the CSE script extension should not have executed again.
Actual Results:  The FileToBeCopied.txt is present.

Is this fixed?


Getting an error on setup

I've restarted .
I've regerated API key .
Clearly the screen shot does not include my full key or  account number however been ultra careful to ensure they are complete.

Sorry you're having issues. Do you have any errors in your Home Assistant logs?

Dave,
Thank you for following this up personally.
Please see log
Many thanks Peter
This error originated from a custom integration.
Logger: aiohttp.server
Source: custom_components/octopus_energy/api_client.py:120
Integration: Octopus Energy (documentation, issues)
First occurred: 18:06:21 (2 occurrences)
Last logged: 18:06:27
Error handling request
Traceback (most recent call last):
File "/usr/local/lib/python3.10/site-packages/aiohttp/web_protocol.py",
line 435, in _handle_request
resp = await request_handler(request)
File "/usr/local/lib/python3.10/site-packages/aiohttp/web_app.py", line
504, in _handle
resp = await handler(request)
File
"/usr/local/lib/python3.10/site-packages/aiohttp/web_middlewares.py", line
117, in impl
return await handler(request)
File
"/usr/src/homeassistant/homeassistant/components/http/security_filter.py",
line 60, in security_filter_middleware
return await handler(request)
File "/usr/src/homeassistant/homeassistant/components/http/forwarded.py",
line 100, in forwarded_middleware
return await handler(request)
File
"/usr/src/homeassistant/homeassistant/components/http/request_context.py",
line 28, in request_context_middleware
return await handler(request)
File "/usr/src/homeassistant/homeassistant/components/http/ban.py", line
82, in ban_middleware
return await handler(request)
File "/usr/src/homeassistant/homeassistant/components/http/auth.py", line
236, in auth_middleware
return await handler(request)
File "/usr/src/homeassistant/homeassistant/components/http/view.py", line
136, in handle
result = await result
File
"/usr/src/homeassistant/homeassistant/components/config/config_entries.py",
line 178, in post
return await super().post(request, flow_id)
File
"/usr/src/homeassistant/homeassistant/components/http/data_validator.py",
line 73, in wrapper
result = await method(view, request, data, *args, **kwargs)
File "/usr/src/homeassistant/homeassistant/helpers/data_entry_flow.py",
line 110, in post
result = await self._flow_mgr.async_configure(flow_id, data)
File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 280,
in async_configure
result = await self._async_handle_step(
File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 367,
in _async_handle_step
result: FlowResult = await getattr(flow, method)(user_input)
File "/config/custom_components/octopus_energy/config_flow.py", line 156,
in async_step_user
return await self.async_setup_initial_account(user_input)
File "/config/custom_components/octopus_energy/config_flow.py", line 86,
in async_setup_initial_account
account_info = await
client.async_get_account(user_input[CONFIG_MAIN_ACCOUNT_ID])
File "/config/custom_components/octopus_energy/api_client.py", line 112,
in async_get_account
"gas_meter_points": list(map(lambda mp: {
File "/config/custom_components/octopus_energy/api_client.py", line 117,
in
"agreements": list(map(lambda a: {
File "/config/custom_components/octopus_energy/api_client.py", line 120,
in
"tariff_code": a["tariff"]["tariffCode"],
TypeError: 'NoneType' object is not subscriptable
On Sat, 10 Sept 2022, 09:36 David Kendall, @.***> wrote:

Sorry you're having issues. Do you have any errors in your Home Assistant
logs?
—
Reply to this email directly, view it on GitHub
https://github.com/BottlecapDave/HomeAssistant-OctopusEnergy/issues/62#issuecomment-1242669734,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVCTQASYUAHI3GVFROZIBSLV5RCBLANCNFSM6AAAAAAQJG6PVY
.
You are receiving this because you authored the thread.Message ID:
@.***
com>

Sorry for the late response. Do you have the ability to set your log level to 'debug' and try again? This should add the raw response to your home assistant logs when you attempt again. This will help guide why you're getting the error you're getting. My gut feeling is I'm attempting to read an old agreement that you once had which is no longer applicable.


Refactoring in OTB notebooks

Added comments, removed unnecessary import statements and some minor improvements in OTB notebooks.

Resolve pull request conflicts


Incompatible with meteor-starter

Steps to reproduce:

git clone https://github.com/yogiben/meteor-starter.git myapp
cd myapp
meteor
Site loads well at localhost:3000
meteor add meteor-collection-hooks

Then I get this error:
Your app is crashing. Here's the latest log.
Meteor._wrapAsync has been renamed to Meteor.wrapAsync
/home/femi/.meteor/packages/meteor-tool/.<IP_ADDRESS>sddkj++os.linux.x86_64+web.browser+web.cordova/mt-os.linux.x86_64/dev_bundle/server-lib/node_modules/fibers/future.js:245
throw(ex);
^
Error: Tabular.Table options must specify collection
at new Tabular.Table (packages/aldeed:tabular/common.js:20:1)
at packages/yogiben:admin/lib/both/startup.coffee:89:27
at Function..each..forEach (packages/underscore/underscore.js:113:1)
at adminCreateTables (packages/yogiben:admin/lib/both/startup.coffee:66:4)
at __coffeescriptShare (packages/yogiben:admin/lib/both/startup.coffee:114:2)
at /home/femi/Dropbox/IT Simply Works/Klanten/www/a/.meteor/local/build/programs/server/boot.js:229:5
Exited with code: 8
Your application is crashing. Waiting for file change.

Did you try with: meteor add matb33:collection-hooks? I'm actually not sure what will get loaded with meteor add meteor-collection-hooks to be honest...

This gives me exactly the same error.

Are you on an old browser?

No, latest chrome. But the error also appears several times in the console.

You'll need to edit .meteor/packages and move the reference for matb33:collection-hooks further up the top, as long as it's above the comments package. We haven't figured out a way around this issue, and is actually a main driver towards implementing collection hooks into core (or having a core collection extension API).

I can confirm this works perfectly. Thanks a lot.
The actual goal was socialize:friendships which depends on socialize:base-model, which depends on matb33:collection-hooks.
With this workaround, this works as well.


Doesn't work for the getting the signed document URL

This ruby gem did not work to get the signed document URL

It would be nice if we could get the pdf final version of a file using the gem's auth
sr = HelloSign.get_signature_requests(signature_request_id: "e72fcc55")
tempfile = sr.final_copy

Presently, I  receive the webhook event and have to manually use open-uri to auth and fetch down the file and attach it to my own s3 bucket.

for anyone else checking issues, it's just not found in the docs:
https://github.com/HelloFax/hellosign-ruby-sdk/blob/f8af9b3219e14222cf44172071339a8651e1f7ea/lib/hello_sign/api/signature_request.rb#L250
but the method is HelloSign.signature_request_files


Support setting config flags from the environment.

Makes it easy to have local settings without having to deal with config
files or temporary config changes escaping into builds.

@mikekatz please review

:+1:


xaiger failure during abc9 pass

Steps to reproduce the issue
See gls_mvce.tar.xz. Reproduce with:
tar xfJ gls_mvce.tar.xz
cd gls_mvce
bash RunMe.sh

Expected behavior
yosys successfully completes synthesis run, including abc9 techmapping and aiger stage
Actual behavior
yosys fails with:
<IP_ADDRESS>. Executing AIGER frontend.
/usr/include/c++/10/bits/stl_vector.h:1045: std::vector<_Tp, _Alloc>::reference std::vector<_Tp, _Alloc>::operator[](std::vector<_Tp, _Alloc>::size_type) [with _Tp = Yosys::hashlib::dict<Yosys::RTLIL::SigBit, Yosys::RTLIL::State>::entry_t; _Alloc = std::allocator<Yosys::hashlib::dict<Yosys::RTLIL::SigBit, Yosys::RTLIL::State>::entry_t>; std::vector<_Tp, _Alloc>::reference = Yosys::hashlib::dict<Yosys::RTLIL::SigBit, Yosys::RTLIL::State>::entry_t&; std::vector<_Tp, _Alloc>::size_type = long unsigned int]: Assertion '__builtin_expect(__n < this->size(), true)' failed.

Reproducer tarball includes a backtrace in yosys-abc-QNqgGU-backtrace.txt and the entire yosys-abc temp folder yosys-abc-QNqgGU.
I am using upstream abc at commit 448f263 with PR 89 applied on top (which gets rid of a problem whereby abc itself crashes during the abc9 techmapping stage, see issue 84. Given the recent commit version of abc I'm using, I believe these problems are also shared with the current "vendored" abc fork maintained by yosyshq.
For context, the verilog files in the reproducer are building a LiteX + RocketChip SoC (edit: for the ecp5-versa-5g board from Lattice).
Happy to provide any extra details to help further narrow down the problem. Thanks!

cc @eddiehung

No longer reproduces on vanilla b2e9717419e9a852f4e64f12891b8e9742900917 -- the attached testcase gets all the way through the flow. Closing for now.


V2 transaciton endpoints feature request compilation

TODO's:

[ ] correct serve camelCasing for all variants of SettingsInfo (verify other endpoints too)
[ ] Replace any address with an AddressInfo object
[ ] remove is_cancellation flag from Custom transactions and create a new variant of TransactionInfo::Cancellation

Consistent use of Uri and Url (currently we mix them)


Keep add button visible

Seems like a good idea to have an option to keep the Add button always visible, because when you start typing it disappears until you go to the next line.

@Katyadelarosa Just got a chance to look at this and it seems like your PR changes the default behavior and makes the (+) button always be visible by default. Your changes should not change the default behavior and should only be applied when enabled through prop. Let me know if you can take a look. Otherwise, I can merge your PR and apply the required changes after that and then publish a new version.


previewer: fix js-error of deleting pic when photoswipe is closed

Please makes sure the items are checked before submitting your PR, thank you!

[ ] Rebase before creating a PR to keep commit history clear.
[ ] Only One commit
[ ] No eslint errors

Codecov Report

Merging #3011 into v2 will increase coverage by 0.39%.
The diff coverage is 10%.

@@            Coverage Diff             @@
##               v2    #3011      +/-   ##
==========================================
+ Coverage   19.47%   19.86%   +0.39%
==========================================
  Files         226      226
  Lines        5603     5607       +4
  Branches      710      711       +1
==========================================
+ Hits         1091     1114      +23
+ Misses       4393     4357      -36
- Partials      119      136      +17

Impacted Files
Coverage Δ

src/components/previewer/index.vue
5.17% <10%> (+1.46%)
:arrow_up:

src/components/blur/blur.js
66.66% <0%> (+2.15%)
:arrow_up:

src/components/x-input/index.vue
13.69% <0%> (+10.11%)
:arrow_up:

src/libs/base.js
45.45% <0%> (+27.27%)
:arrow_up:

Continue to review full report at Codecov.

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


Lumen 5.5.x support ( removed mixed Laravel and Lumen dependencies )

Since laravels dependency is required in composer.json it lead to following error: https://github.com/laravel/lumen-framework/issues/655 that is caused by mixed dependencies.
Because package supports both: Lumen and Laravel I think it's good idea to move laravel requirement to dev section only for testing purposes.

Hello, thanks I was thinking about doing this as well.
Quick comment, it seems like you forgot to import the alias loader in the laravel/modules class.

There has been a pretty big change on the repository, specifically class names that have changed.
Could you re-submit your PR with the up to date upstream and the fix to the class import of the AliasLoader please?
Thanks!

Sure, let me check. Just strange that all tests passed.

Indeed, but the error was shown on sensiolabs insights


Multline editing prior lines not possible

I would like to edit the text pasted after the multiline delimiter editable such that I can go back on previous lines to edit.

Hi @pdevine,
If I understand the issue correctly, when I open the terminal on my Mac and paste two lines, I can use the arrow keys to move back to the first line and edit it. However, when I open the terminal and paste a prompt that is composed of two lines, I can use the arrow keys to move back and access the first character of the last line, but I cannot move up to the previous line.
If this can be fixed, it would be great. If not, using an app like Msty is a good workaround.


based on the feedback of the client

make the site more accessible for the user
add an interactive element so the user can add their initiatives
look at the other website for some general inspiration

Done


Update intlTelInput.js

Update autoCountryLoaded using cookie to save data and not do duplicate request for the same public ip.

Thanks for your helpful input, but I've realised I need to refactor this whole system (as per my comment in that issue).