Problems to compile

Hello! We are facing problems compiling HIP-CPU and I am wondering if someone could help us out. Maybe we are missing something...
Trying to compile in a machine with Intel CPUs and AMD GPUs. We have HIP 4.4.21401-bedc5f61 (the latest one we could install), TBB 2018, GCC 7.2.0 and CMake 3.16.4.
When we run the following commands:

mkdir build
cd build
cmake .. -DCMAKE_CXX_FLAGS="$(hipconfig --cpp_config)"
make install

This is what we get:

Scanning dependencies of target bit_extract
[  1%] Building CXX object examples/bit_extract/CMakeFiles/bit_extract.dir/bit_extract.cpp.o
/u/cruzeiro/hip-cpu/examples/bit_extract/bit_extract.cpp: In function ‘void bit_extract_kernel(uint32_t*, const uint32_t*, size_t)’:
/u/cruzeiro/hip-cpu/examples/bit_extract/bit_extract.cpp:24:21: error: ‘blockIdx’ was not declared in this scope
size_t offset = blockIdx.x * blockDim.x + threadIdx.x;
^~~~~~~~
/u/cruzeiro/hip-cpu/examples/bit_extract/bit_extract.cpp:24:21: note: suggested alternative: ‘clock’
size_t offset = blockIdx.x * blockDim.x + threadIdx.x;
^~~~~~~~
clock
/u/cruzeiro/hip-cpu/examples/bit_extract/bit_extract.cpp:24:34: error: ‘blockDim’ was not declared in this scope
size_t offset = blockIdx.x * blockDim.x + threadIdx.x;
^~~~~~~~
/u/cruzeiro/hip-cpu/examples/bit_extract/bit_extract.cpp:24:34: note: suggested alternative: ‘clock’
size_t offset = blockIdx.x * blockDim.x + threadIdx.x;
^~~~~~~~
clock
/u/cruzeiro/hip-cpu/examples/bit_extract/bit_extract.cpp:24:47: error: ‘threadIdx’ was not declared in this scope
size_t offset = blockIdx.x * blockDim.x + threadIdx.x;
^~~~~~~~~
/u/cruzeiro/hip-cpu/examples/bit_extract/bit_extract.cpp:24:47: note: suggested alternative: ‘pthread_t’
size_t offset = blockIdx.x * blockDim.x + threadIdx.x;
^~~~~~~~~
pthread_t
/u/cruzeiro/hip-cpu/examples/bit_extract/bit_extract.cpp:25:34: error: ‘gridDim’ was not declared in this scope
size_t stride = blockDim.x * gridDim.x;
^~~~~~~
/u/cruzeiro/hip-cpu/examples/bit_extract/bit_extract.cpp:29:18: error: ‘__bitextract_u32’ was not declared in this scope
C_d[i] = __bitextract_u32(A_d[i], 8, 4);
^~~~~~~~~~~~~~~~
/u/cruzeiro/hip-cpu/examples/bit_extract/bit_extract.cpp:29:18: note: suggested alternative: ‘__restrict_arr’
C_d[i] = __bitextract_u32(A_d[i], 8, 4);
^~~~~~~~~~~~~~~~
__restrict_arr
/u/cruzeiro/hip-cpu/examples/bit_extract/bit_extract.cpp: In function ‘int main()’:
/u/cruzeiro/hip-cpu/examples/bit_extract/bit_extract.cpp:73:9: error: ‘hipLaunchKernelGGL’ was not declared in this scope
hipLaunchKernelGGL(bit_extract_kernel, dim3(blocks), dim3(threadsPerBlock), 0, 0, C_d, A_d, N);
^~~~~~~~~~~~~~~~~~
/u/cruzeiro/hip-cpu/examples/bit_extract/bit_extract.cpp:73:9: note: suggested alternative: ‘hipLaunchKernel’
hipLaunchKernelGGL(bit_extract_kernel, dim3(blocks), dim3(threadsPerBlock), 0, 0, C_d, A_d, N);
^~~~~~~~~~~~~~~~~~
hipLaunchKernel
make[2]: *** [examples/bit_extract/CMakeFiles/bit_extract.dir/bit_extract.cpp.o] Error 1
make[1]: *** [examples/bit_extract/CMakeFiles/bit_extract.dir/all] Error 2
make: *** [all] Error 2

We also tried using hipcc instead of GCC: CXX=hipcc cmake .. -DCMAKE_CXX_FLAGS="$(hipconfig --cpp_config)" . And this is what we get:

Scanning dependencies of target occupancy
[ 20%] Building CXX object examples/occupancy/CMakeFiles/occupancy.dir/occupancy.cpp.o
/u/cruzeiro/hip-cpu/examples/occupancy/occupancy.cpp:54:12: error: no matching function for call to 'hipOccupancyMaxPotentialBlockSize'
HIP_CHECK(hipOccupancyMaxPotentialBlockSize(&mingridSize, &blockSize, multiply, 0, 0));
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/u/cruzeiro/hip-cpu/examples/occupancy/occupancy.cpp:11:9: note: expanded from macro 'HIP_CHECK'
if (status != hipSuccess) {
^~~~~~
/opt/rocm-4.5.0/hip/include/hip/hip_runtime_api.h:4473:35: note: candidate function template not viable: no known conversion from 'uint32_t *' (aka 'unsigned int ') to 'int ' for 1st argument
static hipError_t host inline hipOccupancyMaxPotentialBlockSize(int gridSize, int blockSize,
^
/opt/rocm-4.5.0/hip/include/hip/hip_runtime_api.h:3722:12: note: candidate function not viable: no known conversion from 'uint32_t *' (aka 'unsigned int ') to 'int ' for 1st argument
hipError_t hipOccupancyMaxPotentialBlockSize(int gridSize, int blockSize,
^
/opt/rocm-4.5.0/hip/include/hip/hip_runtime_api.h:4526:19: note: candidate function template not viable: no known conversion from 'uint32_t *' (aka 'unsigned int ') to 'int ' for 1st argument
inline hipError_t hipOccupancyMaxPotentialBlockSize(int gridSize, int blockSize,
^
/u/cruzeiro/hip-cpu/examples/occupancy/occupancy.cpp:54:12: error: no matching function for call to 'hipOccupancyMaxPotentialBlockSize'
HIP_CHECK(hipOccupancyMaxPotentialBlockSize(&mingridSize, &blockSize, multiply, 0, 0));
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/u/cruzeiro/hip-cpu/examples/occupancy/occupancy.cpp:12:40: note: expanded from macro 'HIP_CHECK'
std::cout << "Got Status: " << status << " at Line: " << LINE << std::endl;
^~~~~~
/opt/rocm-4.5.0/hip/include/hip/hip_runtime_api.h:4473:35: note: candidate function template not viable: no known conversion from 'uint32_t *' (aka 'unsigned int ') to 'int ' for 1st argument
static hipError_t host inline hipOccupancyMaxPotentialBlockSize(int gridSize, int blockSize,
^
/opt/rocm-4.5.0/hip/include/hip/hip_runtime_api.h:3722:12: note: candidate function not viable: no known conversion from 'uint32_t *' (aka 'unsigned int ') to 'int ' for 1st argument
hipError_t hipOccupancyMaxPotentialBlockSize(int gridSize, int blockSize,
^
/opt/rocm-4.5.0/hip/include/hip/hip_runtime_api.h:4526:19: note: candidate function template not viable: no known conversion from 'uint32_t *' (aka 'unsigned int ') to 'int ' for 1st argument
inline hipError_t hipOccupancyMaxPotentialBlockSize(int gridSize, int blockSize,
^
2 errors generated when compiling for gfx803.
make[2]: *** [examples/occupancy/CMakeFiles/occupancy.dir/occupancy.cpp.o] Error 1
make[1]: *** [examples/occupancy/CMakeFiles/occupancy.dir/all] Error 2

Any ideas of what is wrong?
Thank you very much beforehand!
Vinicius Cruzeiro and Michael Miller

HIP-CPU is self-contained and does not require a HIP install (in fact, it will clash with it, so mixing them is unsupported). We export a CMake namespaced target that you should pass to target_link_libraries which'll take care of setting everything up correctly in what regards the build. Having said that, I'm afraid GCC 7.2.0 is too old, and won't quite work (it doesn't have C++ standard support). Is there any chance to move to GCC 10 or later @vwcruzeiro?


Never leaves configuration mode.

I booted into configuration mode, then insert mouses USB and the green light just stays on.    Any ideas?

hi, so if I understood correctly, you started the board, when there is the red light you hold the B button, the red led flashes and then you have the green led on. Now you insert the USB joystick/gamepad and nothing happens, am I right?

Thanks!  I was trying to configure the mouse.  BTW, the mouse work perfect.

perfect, thanks for the check!


syncer: remove feature guard when checking ns meta equality

Signed-off-by: zhuangqh<EMAIL_ADDRESS>

What this PR does / why we need it:
update namespace meta is a generic operation
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #

/approve
/lgtm


[RESOLVED] High ICMP ping response time

Problem started at 12:00:33 on 2023.05.05
Problem name: High ICMP ping response time
Host: S100-Santiago
Severity: Warning
Operational data: Value: 10ms
Original problem ID: 20349
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23518&eventid=20349

Problem has been resolved in 4m 0s at 12:04:33 on 2023.05.05
Problem name: High ICMP ping response time
Host: S100-Santiago
Severity: Warning
Original problem ID: 20349


Proper structuring of server side

All the folders and files on server side is unstructured

Will make correct folder and structure it.

To Reproduce

Steps to reproduce the behavior:
1.Separate folders for sources
2.Separate route, controller, and model folder in source folder
3.All the files and folders have proper naming convention
4.Will maintain all the functioning of routes

Expected behavior

A clear and concise structure that will be easy to debug and error handling

Environment (please complete the following information):

OS: Window
Browser: Chrome

Additional Context

Unstructured folder looks messy and ceases the motivation to work on project.

@jain-rishabh-21 @Gurdeep475 ,Please review the issue and assign me.

Can I work on this issue?

hey @SnehaJaiswal123 you can go ahead with it


Create new C7 diagrams with pre-filled process TTL

Problem you would like to solve
As a user I'm creating a new Camunda 7 platform diagram and the first thing I'm faced with is an error message because TTL is missing on the process:

What I'd expect is that a reasonable TTL is set initially, and I can customize from there (to suit my needs).
Proposed solution

TTL is pre-filled for all new diagrams with a reasonable default (6 months)
This way I'm not faced with an error on diagram creation

Alternatives considered
None.
Additional context
Reached out to C7 maintainers to figure what a reasonable TTL could be.

Hey there, thanks for bringing this up. We do agree that the current behavior is not good.
We want to suggest to:

Set the default value to 6 months
Extend Ensure historic data clean-up is considered early#1335 (Planned for 7.20) to:

If the default is used (6 months) - Log level:Info: "You are using the default TTL (Time To Live) of 6 months; your data will be deleted by the clean-up after 6 months. We recommend adjusting this to a TTL aligned with your specific requirements"
If the field is empty - Do not deploy the model and throw Exception "Model property Time To Live is mandatory for deployment."

Greets Tobias

@toco-cam We're ready to pick this issue up any time, as specified in the issue description.

Kickoff Analysis
Business Cases
👉  🟢 New Processes should have a default configuration for historyTimeToLive (happy path of the feature)

Deploying a new process with null HTTL will throw a business exception

👉  🟡  Existing deployed processes should remain unaffected by this change. We ignore the new behaviour for already deployed processes which are migrated to 7.20

The behavior is still questionable if it is desired

Probably, not many people start from scratch with new processes to get the validation exception of defining a HTTL

Pros & Cons
➕ Keeping the same HTTL behaviour might be considered as less intrusive by our customers
➕  There might be customers that don't want to cleanup their historic data for their business needs e.g custom ETLs, reports etc.
➖ We will transiently allow our users to continue having the problem that this feature solves
➖ Introduces another feature flag & relevant logic to alternate between legacy & the new validation logic
➖  Weakens the character of the feature from mandatory for all processes to optional.
➖  There might be customers who would favour an automatic cleanup. Ignoring the new behaviour for existing processes could result in further support cases that request the feature to be enabled by default for existing deployed processes

👉 🟢 Does this requirement break backwards compatibility?

No, the legacy behavior will be accessible by using a feature flag in any case. If customers want to stick to it despite the recommendations, they will be able to use it.

🟢  Documentation is needed to describe the behaviour of HTTL for new & existing deployed processes

🟢 Backporting: This is a new feature request, it does not require any backports to previous versions. It will be available only in 7.20


BUG: Email Validation not provided

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?

Email Validation hasn't been provided here which will lead to spam of invalid emails (the input type has been kept as text).
Hence  the email validation feature needs to be applied so that spam emails could be kept in check.
Kindly assign this issue to me under GSSOC'24!
Thanks..
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@Kalivarapubindusree
I've fixed the issue via pr #455 you may review it.
Thanks!


Cannot load pretrained model

It seems that I cannot load a pre-trained model with the code. I have used the anaconda from the repo. I guess the problem is the CUDA version. I use CUDA 10.0, however, your requirements say CUDA > 9.0. Did you also test with a newer version? Thanks a lot
 net.load_state_dict(torch.load(config.checkpoint)['state_dict'])   File "/home/anaconda3/envs/deepICN/lib/python3.6/site-packages/torch/nn/modules/module.py", line 769, in load_state_dict     self.__class__.__name__, "\n\t".join(error_msgs))) RuntimeError: Error(s) in loading state_dict for LeastSquareTracking: 	Missing key(s) in state_dict: "encoder.net0.0........

I did at some time with cuda 10.0 with no issue. Did you change the conda environment?

No, I didn't. However, I guess this requirement anyway won't work if you use it when you only have cuda 10.0.

pytorch=1.0.0=py3.6_cuda9.0.176_cudnn7.4.1_1

Maybe, can you give it a try to install the pytorch 1.0 with cuda 10.0, and let me know how it goes?

I have finally found the issue. It is not the pytorch version, but it is the logger function save_checkpoint. It checks if the number of GPUs is greater than 1, and saves the state dict differently. It still saves it that way even if you only train with 1 out of N GPUs.


Change Local/Custom to Local/Remote

Add button should be in both

Also make 'properties' button close editing

https://github.com/Microsoft/RTVS/commit/2e62ba12bb572cfadfafa58536ed607d89cc51f5


Custom getters and setters in C/C++

Is there any way to bind getter and setter for JS variable from C?
Something like that.
int myVariable = 0;

int myVariableGetter() { return myVariable; }
void myVariableSetter(int v) { myVariable = v; }

int main() {
  ...
  duk_bind_getter_for("myVariable", myVariableGetter);
  duk_bind_setter_for("myVariable", myVariableSetter);
  duk_eval_string(ctx, "myVariable = 5;");
}

Hmm. Thanks. But what about capturing all dynamic properties of JS object's setters/getters?

What dynamic properties do you mean?

Do you mean capturing all property accesses, regardless of property name? If so, getters/setters don't allow you to do that, but you can use a Proxy object for that.

Yes, I mean. I will look for Proxy object.

There are some Ecmascript-based Proxy examples here: https://wiki.duktape.org/HowtoVirtualProperties.html. The C API call for creating a Proxy is https://duktape.org/api.html#duk_push_proxy if you'd rather initialize the Proxy from C code.

 But, is there any example about using Proxy object for that?

Proxy is a generic Ecmascript concept so almost any Proxy tutorial should be usable. Duktape doesn't implement all traps yet, but get, set, has, delete are probably what you need and they are implemented.

@sazlang Any updates or ready to close?


Adding dark mode toggle button

The website needs an additional feature that allows users to switch the mode of their viewing experience based on their comfort.
The dark mode toggle is one such feature that creates a better user experience.
It can be added to the navbar section of the website

BENEFITS :
Enhanced Comfort: Dark mode reduces eye strain in low-light conditions.
Better Accessibility: Improved readability for users with visual impairments.
Personalized Experience: Users can switch modes based on their preferences.
Increased Engagement: Customizable features boost user retention.
Modern Appeal: Dark mode adds a modern look to your site.
These changes will collectively enhance user experience and satisfaction.

Hi @sahithinandikula! Thanks for opening this issue on Ratna Supermarket. We appreciate your contribution to this open-source project. We aim to respond or assign your issue as soon as possible.


Accept more operations for int values

https://github.com/p4lang/p4c/issues/3730

In the interest of tidying up the set of active issues on the P4 specification repository, I'm marking this as "stalled" and closing it. Of course, we can always re-open it in the future if there is interest in resurrecting it.


About sharing all your experiment dataset

Thank you for your great work and your contribution to the open source community.
I would like to ask if you could share all your experiment dataset, including IMU fixed at the two positions, as shown in your paper.
Thank you!

I will look for the second dataset and get back to you.


Update troubleshooter.js

extensions.blocklist.pingCountTotal is not used anymore in FF76+

all yours - go ahead, squash and merge


BLOCKER - Reflection doesn't work on classes

If you try and build and run the API template with the latest reflection stuff, it fails with:
[ INFO ] Migrating 'sqlite' database (FluentProvider.swift:28)
[ INFO ] Preparing migration 'Todo' (MigrationContainer.swift:50)
⚠ [FluentError.reflectProperty: No property reflected for Swift.WritableKeyPath<App.Todo, Swift.Optional<Swift.Int>>] [/Users/condot01/dev/Vapor/ReflectTest/.build/checkouts/fluent.git-6251908308727715749/Sources/Fluent/Migration/Migration.swift:28:124]

This is apparently due to computed properties on classes (the ID key) not having an offset

Fixed in rc.2.5.


Fix Kafka CI

Overview

Add librdkafka-dev package to container for confluent-kafka.
Reenable kafka testing on CI.
Remove special case packages from kafka CI process.
Adjust kafka db settings to force IPv4
Small cleanup to pip scripting to match devcontainer.

Codecov Report

Merging #863 (86cdd04) into main (3bdb013) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #863      +/-   ##
==========================================
- Coverage   79.91%   79.91%   -0.01%
==========================================
  Files         183      183
  Lines       18244    18244
  Branches     3160     3160
==========================================
- Hits        14580    14579       -1
  Misses       2804     2804
- Partials      860      861       +1

see 1 file with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Resource Spanish merge-only grammar

I should add an example grammar as a file in resources/ so that users have an example from which to build their knowledge of MGs in general and MinG in particular.

Done! This is resources/espa0.mg which can be accessed by the constant $ESPA0 (it's just a useful shortcut to %?RESORCES{"espa0.mg"}).


Suspected bug in new ganache-core istanbul version

Some of our regression tests on kyber repository revert only on new Istanbul version. This is after we did required changes in contracts for Istanbul.
After some investigation the reverting scenario is this.

tx1: user approves token to contract A
tx2:

contract A calls token.transferFrom(user, contractB, amount)
contract B calls token.transfer(user, amount - 1)

if tx is split to two transactions. the revert doesn't happen.
to reproduce the use this repo:
https://github.com/ilanDoron/showGanacheBug/blob/master/README.md
I created a test contract on ropsten and seems this issue isn't there.
Steps to Reproduce (for bugs)
to reproduce the use this repo, just clone and follow steps to run tests.
https://github.com/ilanDoron/showGanacheBug/blob/master/README.md
Your Environment

Ubuntu 16.04 LTS
node 12.13.0
npm 6.12.0
ganache cli - ^6.8.0-istanbul.0
you can find ganache and truffle versions in linked repo.

I've also encountered an error when working with ERC721 contracts and Istanbul. I have adapted OpenZeppelin's ERC721 implementation to work with Solidity 0.6.0 and Istanbul, but when sending using multiple transferFrom() in a single transaction, it fails in some situations.
I cannot provide showcase repository as @ilanDoron did, but I suppose it's the same problem.
My observation when using ERC721 with Istanbul so far:
Case I) working

party A sends token 1 to party B
party A sends token 2 to party B
party A sends token 3 to party B

Case II) working

party A sends token 1 to party B
party B send token 2 to party A

Case III) not working

party A sends token 1 to party B
party A sends token 2 to party B
party B sends token 3 to party A

I am not able to reproduce the problem in my current setup with the newest ganache-core version 2.10.2 right now - it works as expected.
@ilanDoron can you check if the problem still exists in your project after you upgrade ganache?

Closing for issue maintenance.


problem not show the menu

Hello
i have problem my code not work fine in my site but it work good on jsfiddle
same code same url
here it work fine http://jsfiddle.net/vh1oknf7/8/
but on my site not work
any help ??
is there any issue if i used https + http ??

is there any issue if i used https + http ??

when your site is running on https, and you are loading the scripts via http, most browser will block it, as unsafe script.

my site used http not https but the some java used https

the problem is in my pc it work fine but in real site not same code

i used css:
Bootstrap v3.1.1
Jasny Bootstrap v3.1.3
Video.js Default Styles 4.2.2
js :
jQuery v1.11.0
Video.js v4.2.2
Jasny Bootstrap v3.1.3
Bootstrap v3.1.1
its ok or i need to upgrade ???
thnx

any help ??

can you provide a link to your site?

www.pixelsolutin4it.com

Hello this test page
http://demo.q8hm.net/demo.html

This is the error you recieve:
Uncaught ReferenceError: $ is not defined
The code block using the datepicker is defined before jquery is included

	$("#date-daterange .input-daterange").datepicker({
		format: "yyyy-mm-dd",
		startDate: "2014-09-20",
		endDate: "+0d",
		forceParse: false,
		autoclose: true,
		todayHighlight: true
	});

Either you include jquery above this code block or include jquery in your head section

Thnx bro u help me alot i was thinks to remove it :)

Hi i have little problem it not work fine if bootstrap is rtl

LTR = http://demo.q8hm.net/demo-ltr.html
RTL = http://demo.q8hm.net/demo-rtl.html

when u click on white space it going biger

and when u click the input box is be biger

this links for original css with rtl and rtl
http://demo.q8hm.net/demo2-ltr.html
http://demo.q8hm.net/demo2-rtl.html

This is indeed a bug, can you please submit a separate issue for it? And please include the demo page + a jsfiddle example! (You can use this jsfiddle as a base, it includes the latest stable version + a rtl language)


#860: Migrate bin/improver- python to lib/improver/cli

Addresses #860.
Testing:

[ ] Ran tests and they passed OK

How come so many things changed after Codacy looked at the code? It this because Codacy wasn't looking at these files when they were in bin (under the old names)?

Since we're touching all the CLI files anyway, I think it would be nice if we could change the main definition to accept argv (i.e. main(argv=None)), and then pass argv to parse_args. This would keep the existing behaviour (deriving from sys.argv if None is passed in), but also allow us to pass in the command line args to the main function.
...This is still not as nice as having a properly defined interface, and doesn't separate out the IO from the processing, but it at least means we won't need to call the CLIs using subprocess or something equally as messy.
(Used in my prototypes, and the one by @abooton/@carolinejjones)

I notice there are some failures in the CLI tests (probably not related to the changes in this PR...):

How come so many things changed after Codacy looked at the code? It this because Codacy wasn't looking at these files when they were in bin (under the old names)?

Yep!


Route / request prefix support

I need to make a MEANJS-based application accessible through an existing reverse proxy which acts as a front-end for multiple other applications. In such a case, all client requests, and all server routes, need to be prefixed.
Adding route prefix support requires a number of improvements and fixes, and for some of those, I'm not sure which the best course of action is. I think that the route prefix feature belongs upstream anyway (because others might need it as well), hence this issue, for discussion and decision purposes at first :)
For now, on the server side, I have implemented a half-measure which works: the reverse proxy strips the prefix, which sidesteps changing every server route to take the prefix into account. Of course, server routes need prefix handling, but littering every server route file with a require of the config, and every route definition with explicit prefix handling, looks insane to me.
A better way, IMO, is to use a thin wrapper for the express app object, whose sole goal is to add the user-specified prefix in front of every route definition before delegating to the actual express app object. And there may be other, more idiomatic methods I haven't thought about.
In order to save a bit of time, here's a sketch of the changes for the client side (sorry, no proper diff right now - the local application is currently based on a 3-month-old version of the 0.4.0 branch, I need to update...):

in config/env/*.js, a user-specifiable prefix accessible at config.app.prefix;
in config/lib/express.js, initLocalVariables(), injecting config.app.prefix or '' into app.locals.prefix:

	app.locals.facebookAppId = config.facebook.clientID;
 	app.locals.jsFiles = config.files.client.js;
 	app.locals.cssFiles = config.files.client.css;
+	if (config.app.prefix && config.app.prefix !== '') {
+		app.locals.prefix = config.app.prefix + '/';
+	}
+	else {
+		app.locals.prefix = '';
+	}

in config/lib/express.js, initErrorRoutes(), redirecting to either /server-error, or /server-error, depending on config.app.prefix (upstream changed this hunk in the meantime);
in modules/core/server/views/layout.server.view.html, leveraging the  tag in the HTML

-	<base href="/">
+	<base href="/{{prefix}}">

in multiple files which can be found by grepping, removing the leading '/' from the routes and attributes of  tags. For instance, not limited to:

modules/users/client/controllers/*.js
modules/users/client/controllers/settings/*.js
modules/users/client/views/settings/manage-social-accounts.client.view.html
modules/users/tests/client/authentication.client.controller.tests.js

@debrouxl I thought I had a similar need. However, turns out mine isn't a need at all :) My little thought experiment reminded me of this issue you opened. When I first read it, I didn't have much perspective. I think you have something here. However, I'd like to understand your need a bit more.
Could you elaborate on the relationship between your proxy & the client applications? I'm a little fuzzy on that aspect of it. If you could provide some practical examples, that may help.
How would your front-end application react to these prefix routes? Would they just propagate it down to the back-end to determine what /api to hit? Would you send them to a separate front-end application depending on the prefix?
For the back-end handling of the prefix, I think a express middle-ware would suffice. Have you explored that?

The reverse proxy multiplexes access to completely separate applications (different physical servers, different implementation languages, different programmer teams) in the following way:
https://<domain>/app1/<application 1 static files + routes>
https://<domain>/app2/<application 2 static files + routes>

etc.
Clients, usually Web browsers, need to make requests below
https://<domain>/app1/
https://<domain>/app2/

for a given application to work. The applications running on the server behind the proxy need to either be able to deal with that "/app1" or "/app2" prefix by themselves, or have any prefix stripped from requests by some rewriting rule before they reach the application (which is what I did for now, it's a one-liner in the nginx config).
Since I created this issue, I found about StrongLoop's restApiRoot: http://docs.strongloop.com/display/public/LB/Versioning+your+API . This only deals with the API, AFAICS.
For meanjs, I'm thinking about a stronger form, which could use separate static route prefix scalar and REST API route prefix both defined into the config, for effortless (from the meanjs framework user's POV, that is ;) ) prefix handling.
I haven't looked into Express middleware, but integration thereof belongs to upstream meanjs :)

@debrouxl This seems to be a unique use case. If I were you, I would look into express middleware and see what options there out there. If you come up with a solution you can submit a PR for discussion to see if it's a direction the community would like to take.

@debrouxl See #313. You may be able to spark discussion in that issue. It seems like there was a solution with nginx just had to change absolute paths to relative.

Looks like this issue is mostly a duplicate of #313, which I missed when searching in the issues list.
We are both using a reverse proxy, a route prefix, and the client code uses absolute paths which fail to work when there's a route prefix, so we both patch the client code.
As outlined in my first post, the server-side code needs patches as well for server-side routing, if we want to make meanjs able to support non-empty route prefix without using a reverse proxy in front of it (whose goal is to strip the prefix, precisely). StrongLoop applications can already do that.
But I couldn't say which is the best course of action for patching server route definitions in meanjs. I think that a thin wrapper for the Express "app" object, which would transparently inject the user-specified route prefix before delegating route registration to the real Express middleware, should fit the purpose. However, I'm clearly less experienced wrt. Express than the meanjs maintainers and multiple meanjs contributors, so there may be better ways I don't know about - hence an issue for discussion :)
A possible resolution of this issue and #313 could be to fix the client-side code to use relative paths (needs to be done in any case), based on the injected base with prefix, and document how to configure nginx to strip the route prefix. But integrating direct, reverse proxy-less support for a route prefix would be easier for the users of meanjs :)

Since this is a specific case, I'd rather not keep this open on GitHub and instead refer you to Gitter or Stack Overflow or another Forum.

I'm not sure it's a specific case. But I see this case otherwise.
In many companies , you do not have an authentication service application, it would be crazy, but it's a global authentication service (play with header in general). So it is possible (and even in many cases) want to use meanjs , client and server , but not the management of authentication, or at least go through an intermediate step :).
For this we use mocks in development. Who are only making a step reverse proxy (probably in node) between the client Meanjs and the server Meanjs, it change header, for api request, and api check if header is ok. It's only an other step who make a simple redirection.
If everything is on the same server (general case), for doing something like this, you juste have to change port questioned by the client side. In development, with this, your request will do this :
MeanJs client -> Proxy (Mock) (localhost:ProxyPort/api/)
(Proxy (Mock) will change header)
Proxy (Mock) -> MeanJs Server (localhost:ServerPort/api/)
MeanJs Server -> Proxy (Mock)
Proxy (Mock) -> MeanJs client
I hope I have been understandable :) sorry for english !
So for me it' not a specific case, but something for all company use case.
:) But I would not say it's necessary to integrate this, but it is necessary to make this easier or not ? :)
i have a look actually for a personal test :)


"Go to definition" - Node.js - Not working with simple example

VSCode Version: Code 1.15.0 (8b95971d8cccd3afd86b35d4a0e098c189294ff2, 2017-08-09T20:16:18.034Z)
OS Version: Windows_NT x64 10.0.15063
Extensions:

Extension
Author (truncated)
Version

sublime-keybindings
ms-
2.9.1

My "project" has just 2 files: app.js and hey.js. In the example below, VSCode isn't able to find the hello function in the other file.

If I declare hello local by putting var before, then it works fine, but this is not what I want since I need to call the function from app.js:

This issue seems to happen with anything declared in the global scope, including variables.
Do I have to add something else in order to fix this? Am I missing some configuration step? Keep in mind that I'm not following any coding good practices here.

VSCode "About" (tested with both Stable and Insider builds):

Yes, our intellisense doesn't support mixing module and non-module files this way. In this case, app is considered a module file since is uses require, while hey is not since it is not since it does not use module or export or import or require.
If you enable checkJs, you'll see this error reported:

This behavior is as-designed. One possible workaround is to create a d.ts file that declares all the globals in your project: https://code.visualstudio.com/Docs/languages/javascript#_global-variables-and-type-checking

Thanks for clarifying the problem. I suppose this Intellisense design affects not only "Go to definition", but also "Go to Symbol in File" and "Go to Symbol in Workspace", correct? I thought I could at least use these 2 instead of "Go to definition", but it looks like the Intellisense still isn't able to find the symbol in that first example that I mentioned above.
Finally, are there any plans to make this design more flexible? I'm coming from Sublime Text and have also used Webstorm in the past. Both tools are able to find the symbols and definitions just fine. The Node.js projects that I work on rarely use module.exports, so the files aren't split in modules. require is used just to define everything that needs to run. So if I want to use VSCode, I'd need to refactor all files and my peers won't be happy to know that I'm doing this only because of a different editor. Even if I used the workaround you mentioned and created this d.ts file, it will be hard to keep track of all globals across all files in the projects.
For me, the main feature that VSCode has for Node.js development is the integrated debugger. Even the documentation says that:

But if I can't navigate my code efficiently, then it will be hard to switch to VSCode.

Yes, this limitation effects any operation that involves finding usages of a symbol in JavaScript code. Our JavaScript language support is powered by TypeScript, and TypeScript is not able to understand some JavaScript patterns. Using globals and require is valid Node, but it is also not a pattern we have prioritized supporting or, I suspect, really will ever get around to supporting. With limited resources, we have focused on supporting more modern JS patterns, which means properly modularized code.
While I don't know the situation with your codebase, I highly recommend that you consider refactoring the code to use a module pattern with proper dependencies, not because it will improve the VS Code editor experience but because it will make the code itself far more maintainable and easier to reason about.


Don't count events as collectives in host report

Fixes https://github.com/opencollective/opencollective/issues/2083

Merging to hopefully fix CI.


fix: tls serial number causes illegal padding error

This is a hack to work around https://github.com/PeculiarVentures/x509/issues/74 until it is addressed upstream.
It seems serial numbers starting with 80 cause @peculiar/x509 to generate invalid certifiates that Node's TLSSocket then fails to parse, throwing an ERR_OSSL_ASN1_ILLEGAL_PADDING error, so the hack is to generate serial numbers until we get one that doesn't start with 80.
This can be reverted when the upstream issue is fixed.
Change checklist

[x] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation if necessary (this includes comments as well)
[ ] I have added tests that prove my fix is effective or that my feature works

I have been running connection tests with this patch against go-libp2p for 24 hours without a single error.


Install the webhooks after installing the deployments and services

Proposed Changes

The webhooks rely on the services. The webhooks can be triggered during the installation of the manifests, because CMs are installed as well. We need to make sure that the services webhooks rely on, are installed before the installation of the webhooks.
It may lead to some issues during the upgrading, if the webhooks are installed before the services. There is a gap the webhook is updated, but the service is not available. During this time, a change on the CM can trigger the webhook, raising errors.
Update the manifests and dependencies for the main branch.

/test all
/retest

/retest all

/test all

/test pull-knative-operator-eventing-upgrade-tests

/test pull-knative-operator-eventing-upgrade-tests

Why are there so many manifest changes in this PR? Seems to do much more than the title implies.


Need to work out how to package a fat jar file

Want to automate the deployment of the library (including all dependencies).

Actually this is not what I wanted.  I wanted to create a jar that can be used as an external dependency.  This can be done using sbt publish-local or sbt-publish depending...


Admission webhook prevents configuring 'sync_rate' when 'strategy=local' causing rate-limiting-advanced plugin to fail requests

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
When configuring the  rate-limiting-advanced plugin with strategy=local, the Admission webhook prevents setting attribute sync_rate with error below:
admission webhook "validations.kong.konghq.com" denied the request: plugin failed schema validation: schema violation (sync_rate cannot be configured when using a local strategy

This prevents the attribute sync_rate from getting initialized properly which then causes the rate-limiting-advanced plugin to fail  during initialization and invocation with the errors below (all requests ultimately fail with 500):
Error after configuring the plugin:
err in creating new ratelimit namespace: ...l/share/lua/5.1/kong/tools/public/rate-limiting/init.lua:503: sync rate must be a number, context: ngx.timer

Error during requests:
[notice] 2066#0: *267455 [kong] handler.lua:124 [rate-limiting-advanced] [rate-limiting-advanced] using shared dictionary 'kong_rate_limiting_counters', client: <redacted>, server: kong, request: "GET /api/v1/bananas HTTP/2.0", host: "<redacted>"
[error] 2066#0: *267455 [kong] handler.lua:156 [rate-limiting-advanced] err in creating new ratelimit namespace: ...l/share/lua/5.1/kong/tools/public/rate-limiting/init.lua:503: sync rate must be a number, client: <redacted>, server: kong, request: "GET /api/v1/bananas HTTP/2.0", host: "<redacted>"
[error] 2066#0: *267455 [kong] init.lua:317 [rate-limiting-advanced] ...l/share/lua/5.1/kong/tools/public/rate-limiting/init.lua:368: attempt to index local 'cfg' (a nil value), client: <redacted>, server: kong, request: "GET /api/v1/bananas HTTP/2.0", host: "<redacted>"

Request response looks like this:
< HTTP/2 500
< date: Mon, 16 May 2022 18:16:06 GMT
< content-type: application/json; charset=utf-8
< content-length: 42
< x-kong-response-latency: 1
< server: kong/<IP_ADDRESS>-enterprise-edition
<
* Connection #0 to host <redacted> left intact
{"message":"An unexpected error occurred"}%

Expected Behavior
When configuring the  rate-limiting-advanced plugin with strategy=local, the attribute sync_rate is initialized properly and plugin is able to handle requests.
Steps To Reproduce

Create KongPlugin with sync_rate unset:

cat <<EOF | k apply -f -
apiVersion: configuration.konghq.com/v1
config:
  limit:
  - 5
  window_size:
  - 60
  window_type: sliding
  identifier: consumer
  strategy: local
  # sync_rate: -1
  hide_client_headers: false
  retry_after_jitter_max: 0
kind: KongPlugin
metadata:
  name: example-api-rate-limiting-advanced-plugin
  namespace: my-app
plugin: rate-limiting-advanced
EOF

Bind previously created KongPlugin to an Ingress.

Submit a request to the Ingress. Request fails with 500 error.

Kong Ingress Controller version
Controller: kong-ingress-controller:2.2
Proxy: kong-gateway:<IP_ADDRESS>-alpine

Kubernetes version
1.22.8

Anything else?
We run our Gateway in Db-less mode.

You need a newer Kong version. The change to allow using -1 or null with local was added in <IP_ADDRESS>: https://docs.konghq.com/gateway/changelog/#plugins-2

Rate Limiting Advanced (rate-limiting-advanced)
...
Fixed an issue where the local strategy was not working with DB-less and hybrid deployments. We now allow sync_rate = null and sync_rate = -1 when a local strategy is defined.

If you still see a problem on 2.8, please respond back and we'll reopen this to investigate further.


[Bug] Wireless Charging - Armour Inventories not charging.

Playing Sky factory 4
My jetpack does not charge until I either

Equip/Unequip the jetpack
Click on the Show/Hide baubles icon

Jetpack is from the "Iron Jetpacks" mod.

Fixed: This specific Jetpack charges and works perfectly in the upcoming version, so should be fixed when it comes out.


Question: Can someone explain me how it (sharing cache files across instances) works, so I can do cache for self hosted runner?

I'm working on implementing cache for self hosted runner. Can someone share any ideas on how can I share cache data across self hosted instances?

IMO this is hosted on GitHub, and the only way to self-host a cache is to buy GitHub Enterprise.

Thanks


Incompatibility with Alloy Forgery + Mythic Metals

Mod Name
Mythic Metals
Mod Homepage
https://www.curseforge.com/minecraft/mc-mods/alloy-forgery https://modrinth.com/mod/mythicmetals
Describe the bug
crash
Steps to reproduce

...
Logs
https://mclo.gs/LK1Hz6i
Additional context
Im not sure if Mythic Metals is causing the crash but i feel like its a dependency

literally the same as #46 ...

Alloy Forgey works, but when I exit the creative with the mod installed, it frezzes for a bit and crashes. This is an issue I had with minecells (1.20.1). I wonder if it's something to do with owo-lib
See #229


Small refactoring

Here is my small contribution to this wonderful project.
It seems to be a very simple contribution, but if we look at IL Generated, we realize that it makes a total difference when we use Lambda Expression in our methods.
Quik sample:
public class Program
{
    static void Main(string[] args)
    {
        FooBar.Multiply(2, 2);

        FooBar.MultiplyExpressionBody(2, 2);
    }
}

public class FooBar
{
    public static int Multiply(int a, int b)
    {
        return a * b;
    }

    public static int MultiplyExpressionBody(int a, int b) => a * b;
}

IL code generated:
First Sample
.method public hidebysig static int32  Multiply(int32 a,
                                                int32 b) cil managed
{
  // Code size       9 (0x9)
  .maxstack  2
  .locals init ([0] int32 V_0)
  IL_0000:  nop
  IL_0001:  ldarg.0
  IL_0002:  ldarg.1
  IL_0003:  mul
  IL_0004:  stloc.0
  IL_0005:  br.s       IL_0007
  IL_0007:  ldloc.0
  IL_0008:  ret
} // end of method FooBar::Multiply

Second example is much cleaner
.method public hidebysig static int32  MultiplyExpressionBody(int32 a,
                                                              int32 b) cil managed
{
  // Code size       4 (0x4)
  .maxstack  8
  IL_0000:  ldarg.0
  IL_0001:  ldarg.1
  IL_0002:  mul
  IL_0003:  ret
} // end of method FooBar::MultiplyExpressionBody

I see no difference in https://sharplab.io/#v2:C4LglgNgPgAgDAAhgRgNwFgBQMDMSBMCAwggN5YKW4EIBiA9vQEICGATluZpVXigGwIwAO2AIAsgFcIwMAAcIATwAUIsSwA0Q0QgBGASgo8uPU0gDsCFggBUejNx4BfLEd5JkgtROmyFigFEADzk2AFMAZwiwemEmegATFW9NbTEDBABeAD4rW3ssF0xCoA=

Me neither:
https://gist.github.com/jbogard/9479e77a18b2c6fc23161319aad0220a/revisions#diff-94ec98548a42c8691354944e91e75b1d

Difference exists in Debug mode, but not in Release:
https://sharplab.io/#v2:EYLgHgbALANALiAlgGxgExAagD4AEAMABLgIwDcAsAFC4DMxATIQMKEDe1hXdjhAYgHsBAIQCGAJ2ocqXbvVIRCiAHZxCAWQCuyOIgAOyAJ4AKFWtEwlqwsACUnWdNnPiAdkKjCAKhuUZsgF9qBzliEkUzDW1dA0MAUTA9cQBTAGdUxAFlYQE0E0iLKzU7QgBeAD4Pb19qIKpaoA

I see no difference in https://sharplab.io/#v2:C4LglgNgPgAgDAAhgRgNwFgBQMDMSBMCAwggN5YKW4EIBiA9vQEICGATluZpVXigGwIwAO2AIAsgFcIwMAAcIATwAUIsSwA0Q0QgBGASgo8uPU0gDsCFggBUejNx4BfLEd5JkgtROmyFigFEADzk2AFMAZwiwemEmegATFW9NbTEDBABeAD4rW3ssF0xCoA=

@Tornhoof
IL is very different on the site you used!

Use ILDASM and you'll see!

@ralmsdeveloper compile your source in Release Mode and try again :)

@ralmsdeveloper compile your source in Release Mode and try again :)

Difference between Debug and Release in this context only applies if I Optimize the Code!
Otherwise, it does not change anything I showed.

@jbogard  for me besides make the code cleaner, makes it more secure, because if the option of optimization of code is cleared, we guarantee a single IL.

Cleaner code maybe but unless you’re compiling the source yourself it wouldn’t matter would it? The dlls in nuget I assumed are built in release mode.

I gave my little bits here!
If you think this is not important, you can close the issue, I thought very generically, both in Debug/ Release (Optimized) we would have a small gain with it!

ping @jbogard.
For me it makes sense to force this optimization, if for you do not, inform me to close the PR.

It’s not an optimization AFAICT, it’s a style change.
I’ve been traveling but can review next week.
On Mon, May 13, 2019 at 4:04 PM Rafael Almeida<EMAIL_ADDRESS>wrote:

ping @jbogard https://github.com/jbogard.
For me it makes sense to force this optimization, if for you do not,
inform me to close the PR.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jbogard/MediatR/pull/401?email_source=notifications&email_token=AAAZQMSJ3AIP75YO57BD5ODPVHJWLA5CNFSM4HK4UEU2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVJR3BA#issuecomment-491986308,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAZQMXV7VAKF7E43BZPOEDPVHJWLANCNFSM4HK4UEUQ
.


moved shield docker compose from its own repo into the shield repo

A fully running shield setup inside docker containers, with a tutorial on how to use shield.

Thanks!


Minor Changes

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

Project coverage is 0.00%. Comparing base (a3b7cf9) to head (ea4c8f1).
Report is 1 commits behind head on stable.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@          Coverage Diff           @@
##           stable     #13   +/-   ##
======================================
  Coverage    0.00%   0.00%
======================================
  Files           1       1
  Lines           4       4
======================================
  Misses          4       4

Flag
Coverage Δ

unittests
0.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Move accelerator_images up one level to work better with concourse

/assign @jjerger

/hold
Sorry, found some bugs. One minute.

/unhold
OK @jjerger should be ready

Oops I had missed this prior to opening a PR to do this myself; we are safe to close this one


[iOS] SIGABRT GrMtlCommandBuffer Crash on production app

Hi,
Our production App encountered a SIGABRT GrMtlCommandBuffer... crash and happened many times, but we can't reproduce it.

Logs
0 libsystem_kernel.dylib ___pthread_kill + 8
8 AGXMetalA12 0x00000001f36e6000 + 208220
9 Flutter GrMtlCommandBuffer::getRenderCommandEncoder(MTLRenderPassDescriptor*, GrMtlPipelineState const*, GrMtlOpsRenderPass*) (GrMtlCommandBuffer.mm:88)
10 Flutter GrMtlOpsRenderPass::precreateCmdEncoder() (GrMtlOpsRenderPass.mm:40)
11 Flutter GrMtlOpsRenderPass::GrMtlOpsRenderPass(GrMtlGpu*, GrRenderTarget*, GrSurfaceOrigin, GrOpsRenderPass::LoadAndStoreInfo const&, GrOpsRenderPass::StencilLoadAndStoreInfo const&) (GrMtlOpsRenderPass.mm:244)
12 Flutter GrMtlGpu::onGetOpsRenderPass(GrRenderTarget*, GrAttachment*, GrSurfaceOrigin, SkIRect const&, GrOpsRenderPass::LoadAndStoreInfo const&, GrOpsRenderPass::StencilLoadAndStoreInfo const&, SkTArray<GrSurfaceProxy*, true> const&, GrXferBarrierFlags) (GrMtlGpu.mm:213)
13 Flutter GrOpsTask::onExecute(GrOpFlushState*) (GrOpsTask.cpp:625)
14 Flutter GrDrawingManager::flush(SkSpan<GrSurfaceProxy*>, SkSurface::BackendSurfaceAccess, GrFlushInfo const&, GrBackendSurfaceMutableState const*) (GrDrawingManager.cpp:308)
15 Flutter GrDrawingManager::flushSurfaces(SkSpan<GrSurfaceProxy*>, SkSurface::BackendSurfaceAccess, GrFlushInfo const&, GrBackendSurfaceMutableState const*) (GrDrawingManager.cpp:498)
16 Flutter SkCanvas::onFlush() (GrDirectContext.cpp:398)
17 Flutter _ZNSt3__110__function6__funcIZN7flutter15GPUSurfaceMetal28AcquireFrameFromCAMetalLayerERK7SkISizeE3$_0NS_9allocatorIS7_EEFbRKNS2_12SurfaceFrameEP8SkCanvasEEclESC_OSE_ (gpu_surface_metal.mm:93)
18 Flutter flutter::SurfaceFrame::Submit() (surface_frame.cc:43)
19 Flutter flutter::Rasterizer::DrawToSurface(flutter::LayerTree&) (functional:1902)
20 Flutter _ZNSt3__110__function6__funcIZN7flutter10Rasterizer4DrawEN3fml6RefPtrINS2_8PipelineINS2_9LayerTreeEEEEENS_8functionIFbRS7_EEEE3$_1NS_9allocatorISE_EEFvNS_10unique_ptrIS7_NS_14default_deleteIS7_EEEEEEclEOSK_ (rasterizer.cc:356)
Flutter flutter::Rasterizer::Draw(fml::RefPtr<flutter::Pipeline<flutter::LayerTree> >, std::__1::function<bool (flutter::LayerTree&)>) (memory:2648)
22 Flutter _ZNSt3__110__function6__funcIZN7flutter5Shell14OnAnimatorDrawEN3fml6RefPtrINS2_8PipelineINS2_9LayerTreeEEEEENS4_9TimePointEE4$_35NS_9allocatorISB_EEFvvEEclEv (functional:1818)
23 Flutter fml::MessageLoopImpl::FlushTasks(fml::FlushType) (ref_ptr.h:121)
24 Flutter fml::MessageLoopDarwin::OnTimerFire(__CFRunLoopTimer*, fml::MessageLoopDarwin*) (message_loop_darwin.mm:76)
25 CoreFoundation ___CFRUNLOOP_IS_CALLING_OUT_TO_A_TIMER_CALLBACK_FUNCTION__ + 32
29 CoreFoundation _CFRunLoopRunSpecific + 600
30 Flutter fml::MessageLoopDarwin::Run() (message_loop_darwin.mm:47)
31 Flutter _ZNSt3__114__thread_proxyINS_5tupleIJNS_10unique_ptrINS_15__thread_structENS_14default_deleteIS3_EEEEZN3fml6ThreadC1ERKNS_12basic_stringIcNS_11char_traitsIcEENS_9allocatorIcEEEEE3$_0EEEEEPvSJ_ (message_loop_impl.cc:0)
32 libsystem_pthread.dylib __pthread_start + 148

MacBook-Pro-2:~ venus$ flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 2.2.3, on macOS 11.6 20G165 darwin-x64, locale zh-Hans-CN)
[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
[✓] Xcode - develop for iOS and macOS
[✓] Chrome - develop for the web
[✓] Android Studio (version 2020.3)
[✓] VS Code (version 1.62.3)
[✓] Connected device (1 available)
    ! Error: iPhone is not connected. Xcode will continue when iPhone is connected. (code
      -13)
    ! Error: iPhone8😍 is not connected. Xcode will continue when iPhone8😍 is connected.
      (code -13)
    ! Error: iphoneX😂 is not connected. Xcode will continue when iphoneX😂 is connected.
      (code -13)

• No issues found!

@curls
From the log you provided, it seems to be similar to https://github.com/flutter/flutter/issues/90743 which has been fixed and should be available on recent / latest versions (e.g latest master). Since you are on 2.2.3, you'll need to upgrade to latest version to see the fix.https://github.com/flutter/flutter/issues/90743#issuecomment-940469390.
Closing this as already fixed. If after upgrade to latest master you still see same exception, write in comments and I'll reopen.

@darshankawar
We have upgraded to flutter v2.8.0, but the exception still exists.

it seems to be similar to #97049 #89171, but still open


Create a Development Project in SAP Business Application Studio

Tutorials: https://developers.sap.com/tutorials/hana-cloud-mission-trial-7.html
Step 2 - Point number 14: After providing all the information and once the Finish button is clicked, got the below errors.

I have then manually opened the project via open folder option in Business Application Studio.
Followed all the subsequent steps.
In Step 7 - Point number 5, after clicking on Deploy button, got the below error:

@chriskollhed @VijayKrishnanSR

Hi Mohamend,
can you please let me know which landscape you are using. Trial, Free Tier, US10, EU10,....
Also:
Have you tried the same step but before doing a "cf login" please.

Either with a terminal session and type cf login
or with the CF symbol on the left side (around the middel of the symbol) and do the cf login there.
Be sure you have a proper binding of you project
Let me know the hwo this works for you
BR
Volker

Hi Mohamend, can you please let me know which landscape you are using. Trial, Free Tier, US10, EU10,....
Also:
Have you tried the same step but before doing a "cf login" please.

Either with a terminal session and type cf login
or with the CF symbol on the left side (around the middel of the symbol) and do the cf login there.
Be sure you have a proper binding of you project
Let me know the hwo this works for you
BR
Volker

Hi Volker,
I am using Free Tier.
My subaccount is created in US East (VA) region, but my URL has eu10 in it.
Please find the below screenshot.

I have tried as suggested (by doing a cf login), still I am getting the same error.

Please refer to https://answers.sap.com/questions/13732450/unable-to-deploy-sap-hana-project.html

We did a fix by engineering and the issue should be solved


Small typo

https://github.com/uber/jaeger-client-go/blob/master/config/config.go#L163
 and sets is as global OpenTracing Tracer should be  and sets it as global OpenTracing Tracer

thx


[SPARK-48494][BUILD][3.5] Update airlift:aircompressor to 0.27

What changes were proposed in this pull request?
upgrade airlift:aircompressor from 0.26 to 0.27
For branch 3.5
Why are the changes needed?
CVE-2024-36114
Decompressors can crash the JVM and leak memory content
The fix https://github.com/airlift/aircompressor/commit/d01ecb779375a092d00e224abe7869cdf49ddc3e
Does this PR introduce any user-facing change?
No.
How was this patch tested?
pass GA
Was this patch authored or co-authored using generative AI tooling?
No.

@pan3793

@bjornjorgensen thanks for making this backport.
cc @HyukjinKwon can we get this in 3.5?

Merged to branch 3.5. Thank you @bjornjorgensen @pan3793


Can the file system import folders?

Could you please tell me how to import a local folder in the file system? For example, I want to import the local node_modules into the jsh terminal through mount. Is it possible? I see that the specific file content is written directly in the files object in the document.

Just like the files object. Does it support importing folders?
const files = {
// This is a directory - provide its name as a key
src: {
// Because it's a directory, add the "directory" key
directory: {
// This is a file - provide its path as a key:
'main.js': {
// Because it's a file, add the "file" key
file: {
contents:             console.log('Hello from WebContainers!')          ,
},
},
// This is another file inside the same folder
'main.css': {
// Because it's a file, add the "file" key
file: {
contents:             body {               margin: 0;             }          ,
},
},
},
},
// This is a file outside the folder
'package.json': {
/* Omitted for brevity /
},
// This is another file outside the folder
'index.html': {
/ Omitted for brevity */
},
};

I want to use local node_module, because some dependencies on the packet network environment


Allow PolyPIC for general scalar/vector fields

Generalizes the dimension handling field such that both scalar and space-vector dimensioned fields (e.g. 3-vectors in 3D sims) can be used. Technically, for the collocated grid case any dimensioned field can be used while to staggered grid case is really only appropriate for scalar and space-vector dimensioned fields.
A couple of notes:

PolyPIC is only for p2g/g2p in the advection phase of the problem
The field that you want to apply PolyPIC to needs to be structured as a matrix such that the number of rows is equal to the number of PolyPIC modes (e.g. 8 for linear) and the number of columns equal to the number of field components . So a single material temperature might be a matrix sized (8,1), two-material as (8,2), and velocity as (8,3) for the linear case. The grid fields to which these data are applied are still scalars or vectors - the expansion to a matrix, even in the scalar case, only applies to the multi-mode particle field.

Codecov Report

Merging #7 (41e6e8e) into master (ef49072) will decrease coverage by 0.28%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master       #7      +/-   ##
==========================================
- Coverage   91.75%   91.47%   -0.29%
==========================================
  Files          37       37
  Lines        3422     2534     -888
==========================================
- Hits         3140     2318     -822
+ Misses        282      216      -66

Impacted Files
Coverage Δ

src/Picasso_PolyPIC.hpp
100.00% <100.00%> (ø)

src/Picasso_ParticleList.hpp
54.54% <0.00%> (-19.53%)
:arrow_down:

src/Picasso_ParticleCommunication.hpp
12.00% <0.00%> (-13.00%)
:arrow_down:

unit_test/tstLevelSetRedistance.hpp
87.09% <0.00%> (-12.91%)
:arrow_down:

src/Picasso_UniformMesh.hpp
69.38% <0.00%> (-10.62%)
:arrow_down:

unit_test/tstParticleInit.hpp
93.93% <0.00%> (-6.07%)
:arrow_down:

src/Picasso_InputParser.cpp
89.47% <0.00%> (-2.84%)
:arrow_down:

src/Picasso_AdaptiveMesh.hpp
83.60% <0.00%> (-2.11%)
:arrow_down:

src/Picasso_FacetGeometry.hpp
92.12% <0.00%> (-1.78%)
:arrow_down:

src/Picasso_FieldManager.hpp
97.22% <0.00%> (-0.61%)
:arrow_down:

... and 27 more

Continue to review full report at Codecov.

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

To be clear - we could have used this before, but now this is more explicit about the generality of use cases?

It actually would not have worked before - it always assumed it would be given a 3-vector for velocity/momentum. Now the collocated grid version will work for any number of components while the staggered grid version will work for scalars or space-vectors.


MH-13513 Officially support URL signing keys that handle multiple URL prefixes

The original URL signing implementation allowed multiple URL prefixes to be handled by the same URL signing key. This was, however, not an officially supported feature and it was removed in Opencast 7.0.
The goal of this task is to officially allow URL signing keys to support multiple URL prefixes.
This is mainly a convenience function (you could use different keys using the same secret to achieve the same), but it also keeps the number of keys that need to be tracked by the verifiers smaller.

@JulianKniephoff That is a follow-up of #863. Would be happy to have you as reviewer...  ;-)

@staubesv already looking at it :wink:


Fixed zz doesn't center the due to wrapped line

When there's wrapped line, sometimes scroll_one_line scroll more than one graphical line, this causes relative_line to be recorded wrong.
This means that if you're dealing with files with a lot of very long lines, typing zz scrolls more than it should. Sometimes no matter how many times you type zz you never get the cursor to center, it simply oscillate as it overshoots every time.
https://user-images.githubusercontent.com/38484873/211141016-90bfed28-aad6-49f8-8df3-7fda85066b56.mp4
This PR fixes that:
https://user-images.githubusercontent.com/38484873/211141059-82d2530a-017b-44b9-b293-fafb739f2aaa.mp4

Thank you @chomosuke!


4:32 i want to add this issue

linux_notes.pdf

commit to #16

hhhujhuhuhh

gghhjjjjjjjjjj

Commit tosses


pre-commit: Update black to fix CI

the lint job still appears in the list, but it should not affect the state of the CI.
I already upgraded black in my PR #104, so I suggest that we close this PR to avoid conflicts. We can also merge and rebase the other PR, as you prefer.

the lint job still appears in the list, but it should not affect the state of the CI.

Yes, it takes PR #107 to fix that. It also proves hat lint is indeed a zombie GitHub Action.

I already upgraded black in my PR #104, so I suggest that we close this PR to avoid conflicts. We can also merge and rebase the other PR, as you prefer.

I don't expect any conflicts because the two PRs fully agree on that file (and the commit). I'm good either way, as long as we get black updated on master soon-ish.

Already included in 53aba0791be4c65ed70a9653a9bbff3b2f34a560 on master by now, closing as obsolete.


fix: lg editor validation error & intelligence

Description
fix lg editor validation error & intelligence, bug introduced by #3738 & #3337

Task Item
close #3815

Screenshots

Can you add some details about the cause and the fix to better understand the context

I can repro the issue without this fix and confirm this PR fixed it.

intelligence not work is because this line only return templates of current lg file, but we need is allTemplate(include imported file)
https://github.com/microsoft/BotFramework-Composer/pull/3337/files#r467668623

Coverage decreased (-0.006%) to 56.422% when pulling ffb2d4c2fc4c175049da90fd7be6dd2d4342d90e on zhixzhan/lg-resolve-fix into f4e5f301e7786d19251f056c4b26ecfaadcd2e9d on main.


⚠ Taitopia Cloud Rendering Service Status has degraded performance

In 6c1cde9, Taitopia Cloud Rendering Service Status (https://status-api.taitopia.design/health/taco) experienced degraded performance:

HTTP code: 200
Response time: 5 ms

Resolved: Taitopia Cloud Rendering Service Status performance has improved in 246badb after 5 hours, 54 minutes.


Require that mstatus.MPRV become 0 on transitions to lesser privilege modes

The hypervisor extension defines an SPRV field in hstatus that performs for HS mode a function similar to the mstatus.MPRV available to M mode.  In order both to greatly simplify the machine states associated with hstatus.SPRV and to narrow a possible security risk, when any MRET or SRET changes the operating mode to one less privileged than HS mode, SPRV is required to become 0.
For consistency and to obtain the same advantages, I propose that mstatus.MPRV be required to become 0 when an MRET changes the mode to one less privileged than M mode.
Although this is technically a change to hardware behavior, respectable software is already expected not to depend on MPRV = 1 outside of M mode.  Hence, no existing respectable software should be impacted.

How is this different in effect than specifying that *PRV is treated as zero when operating at a lower privilege mode?
Perhaps *PRV should be cleared when moving to the associated mode from lower?

I had forgotten that, per the debug spec, MPRV is ignored when in D-mode.  So if we were to make a change to this feature here, it might be more consistent to ignore MPRV when the mode is not M, rather than resetting MPRV when leaving M for a less-privileged mode.  But I guess either solution works.

If the choice is to ignore, then should there be a HW requirement to clear *PRV when entering the mode from below?  This would avoid concern about a certain class of SW bug.

If avoiding that class of bug is part of the concern, then JohnH's original proposal (zero the bit when leaving M-mode for below) seems simpler, since it's only one mechanism (zero the bit on exit) vs. two mechanisms (ignore; also zero the bit on entry).

I don't have an opinion on avoiding the SW bug I mentioned above.
I think ignoring the state is preferable to changing the state.
Is it made clear whether xPRV is intended to only apply when in mode 'x', or if it's additive/cascading (M uses S which uses U)?

If trap entry code takes a trap while saving state because MPRV=1, it takes a trap again, likely hitting the same issue again, in an infinite loop.  Unless trap entry code anticipates that somehow.  Clearing MPRV avoids this, if somehow M-code can be tricked into exiting with it set (unlikely as that may be).
As defined, MPRV and SPRV cascade:  "When MPRV=1, load and store memory addresses are translated and protected as though the current privilege mode were set to MPP." Lower privilege modes act upon SPRV.  Drawing out the combinations is convoluted as you might guess (or more).

Closed by 569d07195a8495460f04592d8455153f730a0f54


[bug] error when build app with bun

Describe the bug
I try to use tauri v1 with bun and I get this error:
error: Failed to run script build due to error:
EBADF: Bad file descriptor (posix_spawn())

Reproduction
Just create a new project with bun and run bun tauri build
Expected behavior
No response
Full tauri info output
[✔] Environment
    - OS: Mac OS 14.4.1 X64
    ✔ Xcode Command Line Tools: installed
    ✔ rustc: 1.76.0 (07dca489a 2024-02-04) (Homebrew)
    ✔ cargo: 1.76.0
    ✔ rustup: 1.27.0 (bbb9276d2 2024-03-08)
    ✔ Rust toolchain: stable-aarch64-apple-darwin (default)
    - node: 20.11.0
    - pnpm: 8.15.4
    - yarn: 1.22.22
    - npm: 10.2.4
    - bun: 1.0.35

[-] Packages
    - tauri [RUST]: 2.0.0-beta.13
    - tauri-build [RUST]: 2.0.0-beta.10
    - wry [RUST]: 0.37.0
    - tao [RUST]: 0.26.1
    - @tauri-apps/api [NPM]: 2.0.0-beta.6
    - @tauri-apps/cli [NPM]: 2.0.0-beta.11

[-] App
    - build-type: bundle
    - CSP: unset
    - frontendDist: ../dist
    - devUrl: http://localhost:1420/
    - framework: React
    - bundler: Vite

Stack trace
No response
Additional context
No response

Duplicated by #9283


PersistentCookieStore not working as expected

This happens when loading the app from Bitrise, then installing BandUp from Android Studio.
Unknown error! Contact Administrator message is displayed when trying to log in into the app.

I seem to have fixed this issue. 😃


Persisted state after page reload!

What is the standard way to persist a state in Pinia?
I'm using nuxt3, and I get  hydration warnings when I do this:
// app.vue
cartStore.$subscribe((mutation, state) => {
  // persist the whole state to the local storage whenever it changes
  localStorage.setItem('cart', JSON.stringify(state))
})

onbeforeMount(() => {
 cartStore.$state = JSON.parse(localStorage.getItem('cart'))
})

This seems to work with a bunch of hydration errors/warnings in the console after a full page reload.
Is there a better way to persist a state?

Duplicate of https://github.com/posva/pinia/issues/309

Please, search existing issues and pull requests before opening issues, it saves maintainers a lot of time 🙏


Not working

NOt sure what's wrong, maybe I did something wrong. idk. Not sure if this is still being worked on. Mod page link led me here, so figured I'd post the issue.
[Error  : Unity Log] NullReferenceException: Object reference not set to an instance of an object
Stack trace:
TimberbornQuadrupleTerrainHeight.QuadrupleTerrainHeightPlugin.Awake () (at <11ad93fcf6a84d3187e3b6a37bb10a6f>:0)
UnityEngine.GameObject:AddComponent(Type)
BepInEx.Bootstrap.Chainloader:Start()
UnityEngine.InputSystem.InputSystem:.cctor()
This is the error that comes up when trying to load this mod.

Thanks for noticing, They've split up "MaxTerrainHeight" into "MaxMapEditorTerrainHeight" and "MaxGameTerrainHeight"

Your welcome and thank you!! Been looking forward to using this mod! :) amazing work!


corrupted files - every 65536th character missing on download/upload

Expected Behavior
Expect 1:1 representation of file on download (and upload)
Actual Behavior
Every 64101st character or sometimes 65536th character is removed from the download (or only after upload--I can't tell. Maybe both?) when using ssh2/ssh2-streams
Steps to Reproduce the Problem
I can not reliably reproduce the issue but I could create a demo and screenshare with you if you like. I'm able to reproduce it within 5 minutes but not on demand with reliable replication steps.
I download/upload a file larger than 200k and experience character removal.
please see the ssh2 developer's request https://github.com/mscdex/ssh2-streams/issues/78#issuecomment-406392923
I have spent several hours this evening trying to figure out how to debug a vscode plugin, specifically the sftp plugin with no luck. (first attempt doing this)
Requisites (important to diagnose the problem!)
Extension Logs

Open User Settings.

On Windows/Linux - File > Preferences > Settings
On macOS - Code > Preferences > Settings

Set sftp.printDebugLog to true and reload vscode.

Reproduce the problem, get the logs from View > Output > sftp.

I followed these steps - the logs look 100% normal when this issue occurs. [trace] execute diff for /Users/filepath/etcetc... no errors.
Specifications

Version: vscode 1.25.1 / sftp 1.2.5
Platform: osx 10.13.6, node v8.11.3, npm 5.10.0, ssh2 ^0.6.1 (0.6.1 currently)

Were you able to reproduce this after reboots, with different servers, from different locations, and/or with different network connections? Do other SFTP-related addons behave the same (if they do, this is more likely a server settings problem than an extension)?
It's been years since I had last gone through missing bytes problems similar to yours, and none of them were editor-related. It was always a connection problem; either with my local connection or the internet provider.

Thanks for taking the time @necmettin. I've tested on Windows & Mac and from two separate networks and the issue persists. I did more testing 12 days ago and still lost every 65536th character

This is the same issue - https://github.com/mscdex/ssh2-streams/issues/43
A co-worker and I came to the same conclusion and he implemented a fix by changing the highWaterMark to 32x1024. I have yet to confirm this myself, but I'll report back once I do.

I can confirm the issue still persists. With fastGet it works for me everytime

Changing the high water mark did fix the issue for our team. It'd be great if this were fixed in the mscdex/ssh2-streams package but alternatively if you could convert the sftp package to fastGet that would be spectacular. This problem also occurs with https://github.com/liximomo/vscode-remote-fs

This happens to me with larger files, seems changing the highWaterMark to one character less fixes it at the extension level.


Of/fix creation validation cookbook artifacts

Update org creation validation to include cookbook_artifacts
Repeats work done on policies endpoint

:+1:


COOPR-632: Update CLI to support partial templates

One minor comment, otherwise LGTM

@jwang47 done

LGTM - please resolve merge conflicts


Prepare Detekt 1.16.0-RC2

Time to bake a new RC 🎉
This will hopefully be the last one before we release 1.16.0 stable.

Should we merge this asap?
We probably also need to revert https://github.com/detekt/detekt/commit/7cd69d21ed13f3109254100532509790db33e320 since I merged that PR in before finding out about RC2 release.

Should we merge this asap?

This has broken tests. I still haven't managed to investigate why.

We probably also need to revert https://github.com/detekt/detekt/commit/7cd69d21ed13f3109254100532509790db33e320 since I merged that PR in before finding out about RC2 release.

I don't think it's a major problem. We could still bake a RC3 after this.

Another problem I found is that the default-detekt-config.yml inside  https://search.maven.org/artifact/io.gitlab.arturbosch.detekt/detekt-core/1.16.0-RC2/jaris still referring to the rules in RC1.

@cortinico I have a fix. We need to fix the int tests after #3450. I will also look into why gradle testkit tests are not picking up the latest code (i assume it is mavenLocal() that may have an impact)
diff --git a/detekt-gradle-plugin/src/intTest/kotlin/io/gitlab/arturbosch/detekt/CreateBaselineTaskDslTest.kt b/detekt-gradle-plugin/src/intTest/kotlin/io/gitlab/arturbosch/detekt/CreateBaselineTaskDslTest.kt
index 8fd648d30..39ada1f06 100644
--- a/detekt-gradle-plugin/src/intTest/kotlin/io/gitlab/arturbosch/detekt/CreateBaselineTaskDslTest.kt
+++ b/detekt-gradle-plugin/src/intTest/kotlin/io/gitlab/arturbosch/detekt/CreateBaselineTaskDslTest.kt
@@ -1,6 +1,7 @@
 package io.gitlab.arturbosch.detekt

 import io.gitlab.arturbosch.detekt.testkit.DslTestBuilder
+import io.gitlab.arturbosch.detekt.testkit.ProjectLayout
 import org.assertj.core.api.Assertions.assertThat
 import org.gradle.testkit.runner.TaskOutcome
 import org.spekframework.spek2.Spek
@@ -14,9 +15,17 @@ internal class CreateBaselineTaskDslTest : Spek({
                     val detektConfig = """
                         |detekt {
                         |   baseline = file("build/baseline.xml")
+                        |   buildUponDefaultConfig = true
                         |}
                         """
+
+                    val projectLayout = ProjectLayout(
+                        numberOfSourceFilesInRootPerSourceDir = 15,
+                        numberOfCodeSmellsInRootPerSourceDir = 15
+                    )
+
                     val gradleRunner = builder
+                        .withProjectLayout(projectLayout)
                         .withDetektConfig(detektConfig)
                         .build()

diff --git a/detekt-gradle-plugin/src/intTest/kotlin/io/gitlab/arturbosch/detekt/testkit/DslGradleRunner.kt b/detekt-gradle-plugin/src/intTest/kotlin/io/gitlab/arturbosch/detekt/testkit/DslGradleRunner.kt
index 63bb874ba..9313d8648 100644
--- a/detekt-gradle-plugin/src/intTest/kotlin/io/gitlab/arturbosch/detekt/testkit/DslGradleRunner.kt
+++ b/detekt-gradle-plugin/src/intTest/kotlin/io/gitlab/arturbosch/detekt/testkit/DslGradleRunner.kt
@@ -17,7 +17,9 @@ class DslGradleRunner @Suppress("LongParameterList") constructor(
     val dryRun: Boolean = false
 ) {

-    private val rootDir: File = Files.createTempDirectory("applyPlugin").toFile().apply { deleteOnExit() }
+    private val rootDir: File = Files.createTempDirectory("applyPlugin")
+        .toFile()
+        .apply { deleteOnExit() } // Comment out when debugging Gradle testkit tests.
     private val randomString = UUID.randomUUID().toString()

     private val settingsContent = """

Another problem I found is that the default-detekt-config.yml inside https://search.maven.org/artifact/io.gitlab.arturbosch.detekt/detekt-core/1.16.0-RC2/jaris still referring to the rules in RC1.

Let's tackle this as a separate problem. I also believe this requires an RC3 before we can fully release a 1.16.0.

@cortinico I have a fix.

Thanks for that. I've actually applied a different fix that relies on creating a Baseline file. Given that we have a testing API for that, let's use it :)

We need to fix the int tests after #3450. I will also look into why gradle testkit tests are not picking up the latest code (i assume it is mavenLocal() that may have an impact)

Yeah that's annoying. Specifically as we notice those failures only once we publish a new version


PREFIX not working in latest version of Pubby

When delcaring own prefix in Pubby e.g. "@prefix dgi: http://localhost:8080/ontology/ ." -->
The prefix is not showed in the HTML output of classes, properties or individuals. Pubby shows a questionmark (?) before every ObjectProperty.
Is there any solution to this?

This is a bug in the latest published release. It is fixed in the development codebase. If you have Java development tooling (JDK and Maven), you can build from source.


How to optimize B, which is the all zero matrix in the Lora method?

Thanks for the question! You can optimize B as usual, e.g., with Adam, since it will get a non-zero gradient in general.

Isn't it the case that all rows of the B matrix are linearly dependent, making it effectively a rank 1 matrix. Could it be simply reduced to a product of two vectors?

I have the same problem. When I perform gradient backpropagation, the weight of A can be updated, but the weight of B is always 0. Please tell me how should I solve this problem? Thank you!

I have the same problem. When I perform gradient backpropagation, the weight of A can be updated, but the weight of B is always 0. Please tell me how should I solve this problem? Thank you!

I met the same problem, have u solved it?

This shouldn't happen. Can you elaborate on your setup?


Different file system problem while mounting

Output of docker version:
$ docker -v
Docker version 1.9.0, build 76d6bc9

$ uname -a
MINGW64_NT-10.0 DESKTOP-CRKO4SG 2.1.0(0.288/5/3) 2015-06-28 18:57 x86_64 Msys
Output of docker info:
$ docker info
Containers: 1
Images: 52
Server Version: 1.9.0
Storage Driver: aufs
 Root Dir: /mnt/sda1/var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 54
 Dirperm1 Supported: true
Execution Driver: native-0.2
Logging Driver: json-file
Kernel Version: 4.1.12-boot2docker
Operating System: Boot2Docker 1.9.0 (TCL 6.4); master : 16e4a2a - Tue Nov  3 19:49:22 UTC 2015
CPUs: 1
Total Memory: 996.2 MiB
Name: default
ID: M7MG:VAXL:ZVSJ:6TJB:UAU4:S4LT:X73U:LGBB:LZ4E:QSHR:5DSU:4AWW
Debug mode (server): true
 File Descriptors: 11
 Goroutines: 20
 System Time: 2016-03-13T13:51:48.019276676Z
 EventsListeners: 0
 Init SHA1:
 Init Path: /usr/local/bin/docker
 Docker Root Dir: /mnt/sda1/var/lib/docker
Labels:
 provider=virtualbox

No additional settings to VirtualBox.
Steps to reproduce the issue:
I use an example image here:

docker run -ti -v //c/Users/X/tmp/work:/work libfuzzer/boringssl

Describe the results you received:
The folder, that is shared by the guest and the host OS, is used by the guest for compiling.
The usage of the symlinks/file systems are different in the guest and the host (Windows NTFS), that causes a "CMake Error: cmake_symlink_library: System Error: Protocol error" termination.
Describe the results you expected:
I expected, that with the mounting (-v) or with other parameters it will be solved, but I could not find anything in the manuals/descriptions/issues which would helped me.
I know that /c/Users is mounted by default in the VirtualBox.
Question:
How can I mount the "work" folder in docker/VirtualBox as a 0. step to make the symlinks working? Is it actually possible?
Thanks for your help!

I have found the solution here:
https://www.virtualbox.org/ticket/10085


doc: Update spelling for Netlify

We need these words for Netlify build that happens outside the docker
container that we use.
Signed-off-by: Michi Mutsuzaki<EMAIL_ADDRESS>Please ensure your pull request adheres to the following guidelines:

[ ] For first time contributors, read Submitting a pull request
[ ] All code is covered by unit and/or runtime tests where feasible.
[ ] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[ ] All commits are signed off. See the section Developer’s Certificate of Origin
[ ] Provide a title or release-note blurb suitable for the release notes.
[ ] Thanks for contributing!

Fixes: #issue-number

Coverage decreased (-0.003%) to 36.879% when pulling b906a51870fff350ade22937419940b9f53993f2 on pr/michi/ddd into 13f8fcad12f2daa9ca11fd6ef44aede66002eed2 on master.


Issue: ImportError: cannot import name 'VectorStore' from 'langchain.vectorstores'

Issue you'd like to raise.
Since 0.0.324, "from langchain.vectorstores import VectorStore" doesn't work anymore, any idea how to fix this?
Thanks!
Suggestion:
No response

+1

Yeap, this import does not seem to be working in the newer versions but you can import it as follow:

from langchain.vectorstores.base import VectorStore

Langchain, PLEASE stop doing this.  PLEASE make all your updates backwards compatible.  As it is, if you keep making changes like this, existing scripts break with every new update of Langchain.

Apologies - this was an oversight on our end, was not intentional. It seemed to have slipped in with this PR.
I just opened a PR to restore this import, as well as add a test for the public facing API to make sure this doesn't happen again: https://github.com/langchain-ai/langchain/pull/12804
@baskaryan is working on adding same style of tests for other modules that dont already have them (we have this for some modules, just not vectorstores)
@tyler-suard-parker - apologies, and thank you for raising

@hwchase17 You rock Harrison!  Thank you!


Event Filtering

I wrote a function generator for event filters, because raise_event + game events doesn't get filtered.
Event filter table -> function.
It generates this function at runtime, which is sertainly not fast.
It could cache functions for filter tables, which adds a bit of usage complexity.
I wanted to get all your thoughts on this, if it's useful, and how it should be exposed to the programmer.
Maybe it's also good to just see how modules will be structured.
Idk, this is new territory for me :D
Here is my current source as a reference, but it would change of course.

Wow, that's pretty cool! But I don't personally think it's worth the performance penalty. Event filters are not supposed to be used for matching purposes, it's supposed to increase performance by telling the game not to pass the event to your mod unless you need it. You need to do matching anyway if you're using an event for more than one thing ever.
This is new territory for me as well, I've never been involved in a project with multiple people before :D

How is this useful/used in practice?
Event filters only usage is cutting down the amount of calls a mod receives.
Does this take care of checking for matching filters inside functions called from filtered events?

alright, i thought about it again, and here is how i would have implemented it.
something like it anyway:
local event_filter_gen = require("__flib__.control.event-filter-generator")

local on_built_entity_filter = {{filter = "crafting-machine"}}
local on_built_entity_filter_func =
  event_filter_gen.generate(on_built_entity_filter, "created_entity")
-- created_entity is the name of the entity field in the event table.
-- it is the most performant to define it here.
-- it is not an option to just have an 'or' over all possible names
-- as you can see when searching for "return" in
-- https://github.com/JanSharp/EventDataValidatorGenerator/blob/master/FactorioEventDataValidator/Lua/event-specific-definitions.txt
-- there are too many different names

-- but maybe it's best to actually give it the event names
-- and the generator takes care of finding the right fields

script.on_event(defines.events.on_built_entity, function(event)
  if event.mod_name then -- if it came from script.raise_event
    if not on_built_entity_filter_func(event) then return end
  end

  -- handle event
end, on_built_entity_filter)

compared to
local crafting_machine_types = {
  ["assembling-machine"] = true,
  ["rocket-silo"] = true,
  ["furnace"] = true,
}

script.on_event(defines.events.on_built_entity, function(event)
  if event.mod_name then -- if it came from script.raise_event
    if not crafting_machine_types[event.created_entity.type] then return end
  end

  -- handle event
end, {{filter = "crafting-machine"}})

it could also be setup so that it can be used to filter events which don't have filters such as script_raised_built.
generally i see it being most useful for all of these, but just in general it means you never have to think about what to check.
And it would always be as optimized as possible, which means even though it adds a function call, chances are it's still faster.

How useful is that really?
Most mods I've seen need a check for one specific crafting machine or a subset of machine names like
script.on_event(defines.events.on_built_entity, function(event)
  local entity = event.created_entity or event.entity
  if my_machines[entity.name] then

  end
end

In those cases any group check is just a waste of time.

Hm, yea maybe you're right

Well actually, jumped the gun a bit, that means most filters are pretty useless.
The only simple and useful ones are (ghost) type, (ghost) name, force and the ones for LuaEntityDamagedEventFilters.
All other ones are hardly ever used, but when using them it's actually super annoying.
Though for that case, this entire function generation system would be overkill. simply exposing that table i liked would help with that.

For those few cases where it's useful to resolve entity types from filter we should make that table directly available.
Perhaps create a module specifically for lookup tables. flib.lookup_tables.filter_definitions

This is no longer needed since 0.18.27.
So i'll close it.


Added jq lib support to jenkins base slave for JSON parsing

This PR has been superseded by #24
Note that #24 takes a different approach of adding a separate container altogether just for jq which
is in contrast with this PR which modifies the base image.


CORS stopped working

CORS headers for Hoodie APIs no longer get set, so cross origin requests fail.
Steps to reproduce:

git clone https://github.com/hoodiehq/hoodie-app-tracker.git
cd hoodie-app-tracker/
npm start -- --url="http://localhost:8080"
Open http://<IP_ADDRESS>:8080/account.html in an incognito / private tab.
Try to log in.

A message like this should appear in the console:

XMLHttpRequest cannot load http://localhost:8080/hoodie/account/api/session. Response to preflight request doesn't pass access control check: No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'http://<IP_ADDRESS>:8080' is therefore not allowed access. The response had HTTP status code 404.

only the /hoodie/account/api/* routes are affected, not the /hoodie/store/api/* routes

Hey @sdvg any update on the issue? @brunopedroso just run into the same thing
For the record: a temporary workaround is to remove sandbox: 'plugin' from https://github.com/hoodiehq/hoodie/blob/master/server/index.js#L55

got the same problem when integrating hoodie in angular 2 app https://github.com/simpulton/ngrx-rest-app. removing the sandbox did the job for me aswell :+1:

@gr2m Not really, I spent some time on this but didn't find out much new.
The Hoodie implementation looks got to me (but I didn't find much documentation on the sandbox feature).
Next steps could be to look for a solution in the hapi code or to build a reduced test case.
I can't tell when I'll find the time to do so :/

I looked into this again and the problem seems to be solved in either Hoodie or hapi :)
FYI: I had to test with two different servers (on different ports). When I start Hoodie with npm start -- --url="http://localhost:8080" and open http://<IP_ADDRESS>:8080 in the browser, all request go to <IP_ADDRESS> instead of localhost now.

I think <IP_ADDRESS>, localhost and <IP_ADDRESS> are aliases, they all point to your own computer. The only difference is that if you bind an app running on your computer to <IP_ADDRESS> it cannot be accessed from outside, while if you bind it to <IP_ADDRESS> it can

Also thanks fro checking :)


[benchmarks] Test that we can keep on building the benchmarks against…

… toolchains by using build-script instead of via the integration tests.
This will ensure that I can put this down for now and things do not break.
In a subsequent commit to the integration test suite I am going to remove that
test.

@swift-ci smoke test

@swift-ci smoke test linux platform


Bug in Middleware History Replay for DevTools for POP Action

Was playing with the current master version because it looked pretty cool and I noticed an issue with the redux devtool time time traveling. I spoke to @taion on Discord about it but wanted to make an issue because Im curious about how it can be ultimately solved and can track it easier this way.
Currently the syncHistoryToStore will update the location object if you reset actions via the redux-devtools. This works really well and is really awesome but has an issue when the the last action to be applied is a POP. The POP happens typically because the user navigated with the URL bar or opened the page for the first time. The POP can't be applied correctly though so the URL bar is not correctly updated even though the reducer data.
This results in the URL being different from what is actually rendering and is pretty confusing.
One thought I had is to dispatch an action instead of calling the history API in this case, resulting in a another action being added to devTools, "showing" the user that you can't replay that far back. This dispatch could also be a REPLACE instead of a POP which would ensure the URL will match the reducer state. Im eager to hear/see other solutions though.

Getting devtools to work is pretty complicated, and I think we've decided to veer towards not guaranteeing the URL will be in sync rather than messing with the action history. If you are time traveling, there are going to be things we can't do with the URL in the location field, and I think that's ok.
I'm going to release 2.0.0 today or tomorrow, so we should get more people using and we'll see how it goes. Maybe there's another fix for it, but we definitely don't want to fire more actions as you time travel.


Syncing from original

Update from original

Hi @edombowsky, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;


chore: update flamegraph script link

Updates the url in the flamegraph script.

#7329  👈
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @LHerskind and the rest of your teammates on  Graphite

Benchmark results
Metrics with a significant change:

avm_simulation_time_ms (Token:mint_public): 57.0 (-89%)
avm_simulation_time_ms (Token:assert_minter_and_mint): 179 (+191%)
app_circuit_proving_time_in_ms (AuthRegistry:set_authorized (avm)): 1,692 (+30%)
app_circuit_proving_time_in_ms (FPC:pay_refund (avm)): 18,314 (-27%)
protocol_circuit_witness_generation_time_in_ms (public-kernel-tail): 22,864 (-16%)
protocol_circuit_proving_time_in_ms (base-rollup): 76,604 (-17%)
protocol_circuit_proving_time_in_ms (public-kernel-setup): 36,496 (-17%)
protocol_circuit_proving_time_in_ms (public-kernel-app-logic): 37,530 (-19%)
protocol_circuit_proving_time_in_ms (public-kernel-tail): 151,052 (-18%)
protocol_circuit_proving_time_in_ms (root-rollup): 19,508 (-19%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Proof generation
Each column represents the number of threads used in proof generation.

Metric
1 threads
4 threads
16 threads
32 threads
64 threads

proof_construction_time_sha256_ms
5,706
1,547
718 (+1%)
749 (-2%)
773

proof_construction_time_sha256_30_ms
11,775 (+1%)
3,159 (+2%)
1,410 (-1%)
1,430
1,461

proof_construction_time_sha256_100_ms
43,795 (-4%)
11,777
5,445
5,423 (+1%)
5,399 (+1%)

proof_construction_time_poseidon_hash_ms
78.0
34.0
34.0
57.0
89.0 (+2%)

proof_construction_time_poseidon_hash_30_ms
1,517
414
200
232 (+3%)
265

proof_construction_time_poseidon_hash_100_ms
5,745
1,564
724
783 (-4%)
786 (-1%)

L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
4 txs
8 txs
16 txs

l1_rollup_calldata_size_in_bytes
1,412
1,412
1,412

l1_rollup_calldata_gas
9,452
9,470
9,464

l1_rollup_execution_gas
611,191
611,360
611,505

l2_block_processing_time_in_ms
759 (-10%)
1,418
2,723 (-2%)

l2_block_building_time_in_ms
20,854 (-2%)
41,825
81,638 (-1%)

l2_block_rollup_simulation_time_in_ms
20,854 (-2%)
41,824
81,638 (-1%)

l2_block_public_tx_process_time_in_ms
17,842 (-2%)
38,581
78,464 (-1%)

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 8 txs.

Metric
3 blocks
5 blocks

node_history_sync_time_in_ms
7,051
9,863

node_database_size_in_bytes
12,255,312
16,220,240

pxe_database_size_in_bytes
16,254
26,813

Circuits stats
Stats on running time and I/O sizes collected for every kernel circuit run across all benchmarks.

Function
time_ms
bytecode_size_in_bytes

GasToken:_increase_public_balance
70.0 (+3%)
13,790

GasToken:set_portal
16.0 (-18%)
3,339

Token:constructor
93.8 (+2%)
23,692

FPC:constructor
63.8 (+2%)
13,592

GasToken:mint_public
52.6 (+3%)
10,158

Token:mint_public
:warning: 57.0 (-89%)
19,034

Token:assert_minter_and_mint
:warning: 179 (+191%)
12,925

AuthRegistry:set_authorized
31.3 (+1%)
7,812

FPC:prepare_fee
114 (+9%)
15,062

Token:transfer_public
37.8 (-20%)
31,218

FPC:pay_refund
133 (-2%)
25,260

Benchmarking:increment_balance
2,196
15,267

Token:_increase_public_balance
58.0 (+4%)
15,006

FPC:pay_refund_with_shielded_rebate
114 (-1%)
26,347

Public DB Access
Time to access various public DBs.

Function
time_ms

get-nullifier-index
0.152 (-7%)

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
1 leaves
16 leaves
64 leaves
128 leaves
256 leaves
512 leaves
1024 leaves

batch_insert_into_append_only_tree_16_depth_ms
10.4 (-2%)
16.7 (-4%)
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_count
16.8
31.7
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_ms
0.602 (-2%)
0.515 (-4%)
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_32_depth_ms
N/A
N/A
48.5
75.7 (-10%)
131 (-26%)
244 (-2%)
471 (-11%)

batch_insert_into_append_only_tree_32_depth_hash_count
N/A
N/A
95.9
159
287
543
1,055

batch_insert_into_append_only_tree_32_depth_hash_ms
N/A
N/A
0.496
0.465 (-10%)
0.450 (-27%)
0.442 (-2%)
0.440 (-11%)

batch_insert_into_indexed_tree_20_depth_ms
N/A
N/A
59.9 (+1%)
111 (-2%)
182 (-7%)
351 (-1%)
691 (-1%)

batch_insert_into_indexed_tree_20_depth_hash_count
N/A
N/A
109
207
355
691
1,363

batch_insert_into_indexed_tree_20_depth_hash_ms
N/A
N/A
0.506 (+1%)
0.500 (-1%)
0.479 (-7%)
0.476 (-1%)
0.475 (-1%)

batch_insert_into_indexed_tree_40_depth_ms
N/A
N/A
72.9
N/A
N/A
N/A
N/A

batch_insert_into_indexed_tree_40_depth_hash_count
N/A
N/A
133
N/A
N/A
N/A
N/A

batch_insert_into_indexed_tree_40_depth_hash_ms
N/A
N/A
0.519
N/A
N/A
N/A
N/A

Miscellaneous
Transaction sizes based on how many contract classes are registered in the tx.

Metric
0 registered classes
1 registered classes

tx_size_in_bytes
74,057
667,850

Transaction size based on fee payment method
| Metric |  |
| - |  |


AI Message Compilation Error with Tool Call and List Content

There is a chance that the model returns an AI message with the content being a list, AIMessage(content=[], tool_calls=[...]).
When involving this message in the conversation, the tool calls gets trimmed out. It runs into the if block on line 83 because the content type is list and never gets into the elif block for the tool call on line 141.
https://github.com/langchain-ai/langchain-google/blob/af44de36cb82277bb5e535e6c2ba968c5b52a18d/libs/vertexai/langchain_google_vertexai/_anthropic_utils.py#L83-L141
This will eventually get into a bad request error.
Error code: 400 - {'type': 'error', 'error': {'type': 'invalid_request_error', 'message': 'messages.2: `tool_result` block(s) provided when previous message does not contain any `tool_use` blocks'}}

thanks for spotting this!
would you be open to send a PR with a fix (and ideally also add a unit test for this case)?


change the way, the overlay is updating the new track?

just a question for me: since i use an djm750mk2 which one has no lan port, the track will update everytime i hit the play button on the decks. i use the beat link trigger tool too and make a txt file of my current playing tracks. this txt file is updated, when i hit the cue button on the last played deck.
deck a > play > txt file update with track a
deck b > play > fade to deck a
deck a ends > cue > txt file update with track b metadata
perhaps this is possible with the pro link overlay? if i change something in the code? whould love to use this overlay, but its annoying to see the overlay updating everytime i hit play. even if i just search a track and quick search through it, the overlay will update.

In the future I think what I would like to do in the case where there's no On-Air indicator is to allow a hotkey to be configured to mark a track as live.
I like this solution as well. I just need to add a configuration so that the user can indicate they don't have on-air status.

This will now automatically work.
If you don't have a mixer on the network it will default to only reporting now-playing tracks when the 2nd deck has been cued / ends / paused / etc

awesome evan, thanks!!