Problem on Building Simulation with RV32E

I am trying to build simulation, but I got this error:
ERROR: ../src/lowrisc_ibex_ibex_simple_system_core_0/ibex_simple_system.cc: In member function ‘std::__cxx11::string SimpleSystem::GetIsaString() const’:
../src/lowrisc_ibex_ibex_simple_system_core_0/ibex_simple_system.cc:40:46: error: ‘class Vibex_simple_system_ibex_simple_system’ has no member named ‘RV32E’; did you mean ‘RV32M’?
std::string base = top.ibex_simple_system->RV32E ? "rv32e" : "rv32i";
I am using Ubuntu Linux 18.04

I fixed it. The problem was about not properly cloning them.

I fixed it. The problem was about not properly cloning them.

Hi,
How did you fix it? I am suffering from the same problem when I build the simulation.
I have tried to change "RV32E" to "RV32M" in the ibex_simple_system.cc file.
Then it can be built without errors and produce the expected "hello_test" result.
But I don't think this is a proper solution as it ignores the E extension.

Follow the instruction explained in this website:
https://blog.csdn.net/qq_33766313/article/details/112780529
In my case, I needed to start over again the setting up.

I fixed it. The problem was about not properly cloning them.

What files are not properly cloned?
I am suffer from it too. And I followed the instructions in the website you gave. But the error still exist.

Hi,
My temporary solution is
In this file ../ibex/example/simple_system/ibex_simple_system.cc:40
Modify  "std::string base = top.ibex_simple_system->RV32E ? "rv32e" : "rv32i";"
To "std::string base = "rv32i";"
which means just to skip the setting of RV32E
Then the building will be successful without errors.
But I am still looking for better solutions to reserve the setting of RV32E

I fixed it. The problem was about not properly cloning them.

What files are not properly cloned? I am suffer from it too. And I followed the instructions in the website you gave. But the error still exist.

Hi,
My temporary solution is
In this file ../ibex/example/simple_system/ibex_simple_system.cc:40
Modify "std::string base = top.ibex_simple_system->RV32E ? "rv32e" : "rv32i";"
To "std::string base = "rv32i";"
which means just to skip the setting of RV32E
Then the building will be successful without errors.
But I am still looking for better solutions to reserve the setting of RV32E

I found that the problem is in the verilator installation flow.
I typed "apt-get install verilater" to install verilator before, now I follow this instructions to install it.
Maybe using apt-get to do a simple installation cannot build verilator very well.
# Prerequisites:
#sudo apt-get install git perl python3 make autoconf g++ flex bison ccache
#sudo apt-get install libgoogle-perftools-dev numactl perl-doc
#sudo apt-get install libfl2  # Ubuntu only (ignore if gives error)
#sudo apt-get install libfl-dev  # Ubuntu only (ignore if gives error)
#sudo apt-get install zlibc zlib1g zlib1g-dev  # Ubuntu only (ignore if gives error)

git clone https://github.com/verilator/verilator   # Only first time

# Every time you need to build:
unsetenv VERILATOR_ROOT  # For csh; ignore error if on bash
unset VERILATOR_ROOT  # For bash
cd verilator
git pull         # Make sure git repository is up-to-date
git tag          # See what versions exist
#git checkout master      # Use development branch (e.g. recent bug fixes)
#git checkout stable      # Use most recent stable release
#git checkout v{version}  # Switch to specified release version

autoconf         # Create ./configure script
./configure      # Configure and create Makefile
make -j `nproc`  # Build Verilator itself (if error, try just 'make')
sudo make install

And the rest of environment building instructions you can follow the instructions in this website.https://blog.csdn.net/qq_33766313/article/details/112780529
Now I can do a simulation well.

We encountered the same problem when using Verilator provided by apt-get in Ubuntu 20.04.5: Verilator 4.038 2020-07-11 rev v4.036-114-g0cd4a57ad. Fixed the problem by compiling and using Verilator version 4.210


Build error on react example

> yarn build
yarn run v1.22.17
$ react-scripts build && inliner -m --videos --preserve-comments dist/index.html > dist/inlined.html
Creating an optimized production build...
Failed to compile.

Module not found: Error: Can't resolve 'jquery/dist/jquery.js' in '/home/gavr/nimView/nimview/examples/react/src'

error Command failed with exit code 1

And there really is no jquery, and why would it be here?
And also nim c -r -d:debug src/App.nim failed, because it cant find file index in /dist/index.html
As I undestand - npm build creating that.

And there really is no jquery, and why would it be here?

For bootstrap css, as in any other example. Bootstrap has some effects which rely on jQuery.
I never tried "yarn", so this might be related to "yarn".
I just added jQuery to the package.json and run "npm run build". Looks good.

And there really is no jquery, and why would it be here?

For bootstrap css, as in any other example. Bootstrap has some effects which rely on jQuery.
I never tried "yarn", so this might be related to "yarn".
I just added jQuery to the package.json and run "npm run build". Looks good.

nice, now it works(with yarn too).  How to run dev build now? There is no dev script like in others examples.

Right... I didn't modify the default react package.json yet.
I guess you can use npm start for development and npm run build for release. I think there was no auto-refresh yet for react.


Support for 5Q0035840c

Hi,
Is it possible to patch the 5q0035840c (tehnisat, no navi, 1 sd slot). GEM enabled.

Without usb2hd or emmc options...

You need to find your own way then.


Language gets reverted to system default when app is restarted

Hello Heitor,
Thank you for the code concerning the new Android 13 per-app language preferences. I've tested it and have some questions:
When I select a language in the language picker of your app, I can see that the language also properly changes in the Android app language settings page. However, when I then restart your app, the language is set back to the system's default language (same for the Android app language settings page).
So, the changes from the language picker are persisted, but are obviously not automatically taken into account when the app gets restarted.
Do we maybe have to implement a getApplicationLocales() method in onCreate(), followed by a setApplicationLocales()? Or is this a bug in Android 13 Beta 2? What do you think?
Kind regards,
Patrick

Hi Patrick,
Thanks for reporting the issue. The code of initLocalePicker does not take into consideration the current locale, so it is always reset to the system one when the app is launched. Once this code was used only to present the feature in the article https://proandroiddev.com/exploring-the-new-android-13-per-app-language-preferences-8d99b971b578 , I kept it as simple as possible and didn't handle that as it would be expected in a real life application. But it is not an Android 13 bug.
Thanks,
Heitor


Doubt - Automatic reload ca certificates

Environment

VerneMQ Version: 1.10.0
OS: alpine
Erlang/OTP version (if building from source): 21.3
VerneMQ configuration (vernemq.conf) or the changes from the default

metadata_plugin = vmq_swc
plugins.vmq_passwd = off
plugins.vmq_acl = off
leveldb.maximum_memory.percent = 20
log.console = console
listener.max_connections = 200000
listener.nr_of_acceptors = 100
max_inflight_messages = 20
max_online_messages = 10000
listener.ssl.default = <IP_ADDRESS>:8883
listener.ssl.default.cafile =  /vernemq/cert/cabundle.crt
listener.ssl.default.certfile = /vernemq/cert/iotagent-mqtt.crt
listener.ssl.default.keyfile = /vernemq/cert/iotagent-mqtt.key
listener.ssl.default.crlfile =  /vernemq/cert/ca.crl
listener.ssl.default.use_identity_as_username = on
listener.ssl.default.require_certificate = on

Cluster size/standalone: 1

Expected behavior

When the file pointed to in listener.ssl.default.cafile is changed, this should be reflected in  vernemq.
This file is on a docker/k8s volume and I change it when a new ca is accepted, add it to the chain, as a ca bundle, without restarting vernemq.

Note: When I updated the crl this works fine...
Actual behaviour
The expected behavior only works when I restart the vernemq pod/container.

@mprevide Thanks, your assessment is correct. I'm not going to look into this but open to suggestions/PR's.

Thank you for supporting VerneMQ: https://github.com/sponsors/vernemq

@mprevide Thanks, your assessment is correct. I'm not going to look into this but open to suggestions/PR's.
Thank you for supporting VerneMQ: https://github.com/sponsors/vernemq

Thank you very much!

Do you have any tips for implementing this? Thanks
@ioolkos


MINOR: added inspection query docs

Describe your changes:
Fixes

I worked on ... because ...

Type of change:

[ ] Bug fix
[ ] Improvement
[ ] New feature
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] My PR title is Fixes <issue-number>: <short explanation>
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] For JSON Schema changes: I updated the migration scripts or explained why it is not needed.

handled in https://github.com/open-metadata/OpenMetadata/pull/17544


Initial block with $readmemh ineffective with Lattice ECP5-25F FPGA

With Lattice ECP5-25F FPGA used by OrangeCrab, $readmemh does not actually initialize an array element.
Also, attempting to manually initialize an array element beyond its number of elements, passes synthesis instead of failing.
Steps to reproduce the issue
parameter SZ = 256;
parameter DW = 64;
parameter SRCFILE = "mem.hex";
reg [DW-1:0] u [0:SZ-1];
initial begin
if (SRCFILE != "") begin
$readmemh (SRCFILE, u);
end
end

You're going to need to give quite a bit more information here (in particular, the design and contents of mem.hex) in order for anyone to be able to help.

Certainly; I will provide fast response; find attached mem.hex that was used; I had to append '.txt' to the filename for github to accept it as attachment.
mem.hex.txt

I use something very similar for synthesis in Cyclone FPGAs (for example, here). Although the $readmemh may look like something for simulation, it is certainly vital for synthesis too.

@jotego I agree that it is vital.

It should be supported, but you still haven't provided enough information for someone to investigate what the actual problem is.

It would help to know which additional information to provide.
To speak more on the issue, $readmemh is simply not initializing the memory it receives as argument.
I tried to debug further replacing $readmemh with manually initializing memory as such:
initial begin
	u[0]    <= 64'00add0391034d0ad;
	u[1024] <= 64'6e69206d6172206e; // Indexing element beyond size of array which is 256.
end

Not only was the initialization not taking place, synthesis would succeed despite indexing an element of the array beyond its size.
Please let me know any additional information you would like me to provide.

Please provide a complete design that can be passed to synth_ecp5 to produce the faulty netlist so that we can take a look at what is happening. There is no module in your example at all, and without the surrounding context the memory would just be optimized away.

Apologies for the long delay in responding. I finally found time to write code to reproduce the issue.
Find attached in orangecrab02.tar.gz following files:
orangecrab02
 |-orangecrab02.pcf
 |-orangecrab02.v
 |-yosys
 | |-makefile
 | |-mem.hex

Build using:
cd orangecrab02/yosys/
make

Program the orangecrab using:
make prog

The logic in orangecrab02.v initializes an array with bit sequences that are played on an LEDs every 1/4 seconds.
A different bit sequence in the array gets selected when pressing the reset button.
I find that the array gets initialized differently when manually initializing it versus using $readmemh: snippet from attached code:
initial begin
   //u[0] <= 128'haaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa;
   //u[1] <= 128'h0a0a0a0a0a0a0a0a0a0a0a0a0a0a0a0a;
   $readmemh (SRCFILE, u);
end

Initialization of the array does happen, but I see that with $readmemh, the blinking of the LED does not always reflect the initialization value.

I tried the latest version of the tools from https://github.com/YosysHQ/oss-cad-suite-build which is oss-cad-suite-windows-x64-20211010.
I see a much better behavior; the older version of the tool I was using definitely had issues.
To see the issue even better, I have modified the logic to AND corresponding bits between a manually initialized array u1 and an array u2 initialized using readmemh, as shown below:
module orangecrab02 (

	 usr_btn_n

	,clk48mhz_i

	,led_red_n
	,led_green_n
	,led_blue_n
);

input wire clk48mhz_i;

input wire usr_btn_n;

output wire led_red_n;
output wire led_green_n = 1'b1;
output wire led_blue_n = 1'b1;

localparam CLKFREQ = 48000000/4 /* 48 MHz */;

localparam SZ = 2;
localparam DW = 128;
parameter SRCFILE = "mem.hex";

reg [DW-1:0] u1 [0:SZ-1];
initial begin
	u1[0] <= 128'haaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa;
	u1[1] <= 128'h0a0a0a0a0a0a0a0a0a0a0a0a0a0a0a0a;
end

reg [DW-1:0] u2 [0:SZ-1];
initial begin
	$readmemh (SRCFILE, u2);
end

reg [1-1:0]  idx;
reg [32-1:0] cntr;
reg [DW-1:0] r1;
reg [DW-1:0] r2;

assign led_red_n = (~r1[0] && ~r2[0]);

reg saved_usr_btn_n; // Used for edge detection.

always @ (posedge clk48mhz_i) begin
	saved_usr_btn_n <= usr_btn_n;
	if (!usr_btn_n && usr_btn_n != saved_usr_btn_n) begin
		r1 <= u1[idx];
		r2 <= u2[idx];
		idx <= idx + 1'b1;
	end else if (cntr >= CLKFREQ) begin
		r1 <= {r1[0], r1[DW-1:1]};
		r2 <= {r2[0], r2[DW-1:1]};
		cntr <= 0;
	end else begin
		cntr <= cntr + 1'b1;
	end
end

endmodule

With the latest version of the tools, u1 and u2 are correctly identical, while with the older version of the tool I was using, u1 and u2 are not identical as they should.
I am continuing to run tests, as I am still seeing issues with a larger design that uses readmemh; it could be another issue with the design itself.


Improve PDF Stream/Buffer handling

Hi,
I need to create a custom DeviceN ColorSpace with a FuntionType 0 function which requires to set a custom stream.
For this I use a Uint8Buffer and ArrayBuffer as data argument of ref.end(data) , but it ends up corrupted by the appended '\n'.
Unfortunately I can't use directly Buffer objects as I am in a browser.
It would be useful, to have ref.end(data) being able to manage unchanged (no '\n') data when used with Array or to have something like a 'doc.toBuffer(data)' to be able to create a Buffer without dependency.
Thanks

Unfortunately I can't use directly Buffer objects as I am in a browser.

Are you using webpack?
If so, you can use Buffer

Thank you for the help,
I tried with a modified 'demo' bundle and standalone version.

Browserify will also work

I tried with provided browserify, it works if I force inclusion of Buffer (it is not by default).
Maybe a rerun of the browserify script may work, didn't try.

Can you post an example so i can update the demo?


Create HUDCamera Class

2D Orthographic projection to render overlays on top of scenes

In reference to https://github.com/jpaoneMines/csci441/issues/17


Suppress autoload option for scope policy is obsolete

Hello.
After PR #136 was merged the suppress autoload option now does nothing, because only functions can be autoload and autoload functions are always global. Thus scope of autoload functions is always ignored now.
Maybe suppress autoload and related code should be removed.

This issue was resolved by #165 .
Thank you, @Coacher !


Merge account and billing

Similar to classic

This task is only for moving Network Helper in with the billing stuff, and handling any renaming that may be needed.


Improve ini

Small improvements and fixes for the ini files.
The most important change is how to associate ini file with singleton projects (no dir, only one nit file).
The proposal, from @xymus, is just to add a .ini file named like the .nit. so lib/counter.ini for the project lib/counter.nit.
I'm not sure that this feature should be official, consider it just as a easy path to update existing code.

:shipit:

:shipit:


Show upcoming trainings chronologically

Context
When visiting the trainings page, users are forced to scroll down in order to find the next upcoming training session.
Probably sorting them chronologically (what this change does) is a more convenient way to present the info to the users.
Screenshots
Before:

After:

Note: Dead link to Eric's pic has been already handled @ #150.

Thanks @dcarral, great improvement!


[IOS] [EAS] app created with "eas build -p ios --auto-submit" are rejected from submission with an app thinning error

Build/Submit details page URL
No response
Summary
Two days ago i got my latest build release rejected by Apple. I made minor changes and tried to submit a new version yesterday and received this from Apple:

The issue cannot be reproduces locally because everything works fine there.. i tried but could not find any errors.
I asked about it on discord and notices others have the same issue.
Managed or bare?
Managed
Environment
expo-env-info 1.0.2 environment info:
System:
OS: Windows 10 10.0.22000
Binaries:
Node: 16.14.0 - C:\Program Files\nodejs\node.EXE
npm: 8.3.1 - C:\Program Files\nodejs\npm.CMD
IDEs:
Android Studio: Version     2<IP_ADDRESS> AI-203.7717.56.2031.7583922
npmPackages:
expo: ^44.0.6 => 44.0.6
react: 17.0.1 => 17.0.1
react-dom: 17.0.1 => 17.0.1
react-native: ^0.64.3 => 0.64.3
react-native-web: 0.17.1 => 0.17.1
Expo Workflow: managed
Error output
No error output.
Reproducible demo or steps to reproduce from a blank project
No reproducible error.

Related to this issue

Is this still an issue? Did disabling bitcode help?


add support for merge_vars

see #17
header[:merge_vars] = [ { rcpt: …, vars: [ { name: …, content: … } ] } ]

see also:
https://mandrill.zendesk.com/hc/en-us/articles/205582487-How-to-Use-Merge-Tags-to-Add-Dynamic-Content

Sorry for delay.  Just getting around to reviewing open PR's.  Thanks for this!  Just wondering if we can cleanup the reference in merge_vars, otherwise, looks good to me.  @jlberglund or @kshnurov thoughts?

Hi @tomasc, @jlberglund has transferred maintenance of this gem to me.  I've made a few small updates to reflect those changes.  Please rebase this PR, and then it should be good to merge.  Thanks!

Hi @spovich rebased and ready, looking forward, thank you!


Moment Edit Screen does not shows correct person

The dropdown of the person is not showing the correct person in edit screen of Moment.

Fixed in #25


Misc fixes

Adds space between coupon applied text and remove link.
Closes #669
Make everything in the pageNav bar disabled when not navigable
Closes #663
Take unneeded flex gutterH classes off of divs that only have text in them in the order details summary, so they don't overflow their containers.
Closes #661
If usecache or async in the options is undefined, make it false
Closes #647
Add left align to variants on listing details.
Closes #640
Change modal gutters to 12px.
Closes #518

👍


Feature Request - Bungeecord&Velocity support

Can you add support for BungeeCord and Velocity? Because i need using in my proxy server. At the same time, if I can see on which server it is reported in the report, I would greatly appreciate it.
And more things, whitelist servers.

Hey, thanks for creating an issue. We'll work towards getting BR on BungeeCord.

If you want tester, i can.

@Timmy109 any updates?

Also i need Velocity support.

It's in progress. I only just saw this now but I get working on it more.

It's in progress. I only just saw this now but I get working on it more.

What is the latest status?


Webstorm doesn't recognize step definitions

I don't really know if I should ask this here, but Webstorm doesn't recoginize the step definitions of this plugin because the steps are given, when, then. Webstorm does recognize the official cucumber package's Given, When an Then (capitalized).  Webstorm recognizing step definitions is extremely helpful in preventing duplicate step definitions.
// Recognized (like the official cucumber package)
Given(`I am on the webpage`, () => {
  cy.visit(url);
});

// Not recognized (like this plugin)
given('I am on the webpage', () => {
  cy.visit(url);
});

I did find a workaround, but I'd rather not have people forget to include this:
// Just add this above the step definitions
const Given = given;
const When = when;
const Then = then;

Is there a way this could be solved without a workaround?

I was thinking about this as well...
I don't really like the fact that the cucumber functions are called with capitalized letter (suggesting a class),
but if the existing tooling relies on that - let's add it. For the backward compability we would have to export given/when/then as both Capitalized and lowerCase.
@kayvanbree Would you like to send a PR for this?
Adding a simple test - where you define the steps with Capitalized letter would be appreciated as well.

@lgandecki I don't like Cucumbers way either, but I think they maybe did it to have capitalized letters in Gherkin files. Anyway, I'll send over a PR when I have time!

I got Webstorm to recognize the steps, but now only ScenarioOutline.feature passes. The rest of the tests fail:
Oops...we found an error preparing this test file:

  cypress\integration\DataTable.feature

The error was:

Error: Cannot find module '..supportstep_definitions/requiringFilesData' from 'C:\Projects\cypress-cucumber-preprocessor\cypress\integration'

This occurred while Cypress was compiling and bundling your test code. This is usually caused by:

- A missing file or dependency
- A syntax error in the file or one of its dependencies

I'll have to get back to this later.

very weird, the changes you made shouldn't affect anything. Can you open a PR here, circle should catch it and run tests on it - maybe the code is correct but it's some weirdness in your local setup.

Created #64, let's see what Circle thinks

I can't get this error on my machine... I still get the parsing error with the incorrect directory.

I don''t really get how stuff works, but I see there's a .toLowerCase in resolveStepDefinition.js.
function resolveStepDefinition(step) {
  const stepDefinition = stepDefinitionRegistry.resolve(
    step.keyword.toLowerCase().trim(),
    step.text
  );

  return stepDefinition || {};
}

the reason for different errors might be some difference between windows and unix machine. I will try to fix this up today (having your PR helps). It shouldn't be too hard now.
Thanks for your help :-) And will update you here in a few hours

I got this to work, but your PR is weird. Please DO NOT change anything in package.json and don't commit package-lock.json.
you've just added the exports, but you don't import them anywhere. Actually, there is no need to export them, the only place that you have to change is index.js, by adding:

  const Given = given;
  const When = when;
  const Then = then;

here:

we could export only given and alias everything to given anyway, but I left the duplicated given/when/then exports for backward compability.
Interesting thing though, that didn't make it work for me in webstorm.. when I tried this initially, it didn't work neither (it also did not work for a normal cucumber 2+ project)
Is there a chance you could push somwhere the code that you were showcasing this problem in the first post? I'd take a look if this is about a folder structure, or what's different..

@lgandecki Webstorm doesn't recognize step definition this way:
Given('user navigated to the Start page', () => { });
But if we replace quotes with /^...?/ then it works:
Given(/^user navigated to the Start page?/, () => { });

@lgandecki
I will change my PR and check if the build works when I do the const Given = given stuff.
@vadimromanyak You can also use ticks:
Given(\`user navigated to the Start page\`, () => {});

If you check out the test steps I created you can see I use either the ticks or /^..?/:
When(/^I write my steps using caps$/, () => {
  counter += 1;
});

@kayvanbree Bingo! backticks work!

@vadimromanyak replacing with /^ etc is not preferable as then we lose the prefered / new / cleaner syntax:
https://github.com/cucumber/cucumber-js/blob/master/docs/nodejs_example.md
But I'm excited that the backticks work, not sure why webstorm wouldn't work with " or ' but hey, not a huge deal once you know it :)

@lgandecki My local tests are failing again. I'll commit to see what Circle thinks.
Also, the /^ stuff may not be preferable, but it's the only way I got (\d+) and (.*?) working to use numbers and strings in my step definitions.

@kayvanbree well, yeah, the /^ gives you the regexp functionality.. but the point is, instead of using regexp, you can just use the newer syntax...
to give you an example, instead of:
then(/(.*?) is chosen/, choice => {})
then(/my lucky number is (\d+)/, number => {})

then(`{word} is chosen`, choice => {})
then(`my lucky number is {int}`, number => {})

Regexps are powerful, but unreadable to human, and very rarely u need such a complex processing anyway, usually it's a word, or a number, etc.. so the cucumber guys introduced the new syntax :)

@lgandecki The tests for #64 are passing!

I don't think webstorm plays nice with the new syntax...

Indeed

closed in #65 , it works with us as much as it works with pure cucumber, not much more we can't do at this time.
Thanks @kayvanbree @vadimromanyak for your help here :)

https://youtrack.jetbrains.com/issue/WEB-32819 for follow-up. You can "thumb-up" to make it gain interest, and/or watch the issues for updates.


feat: add program and device type registry

✍ Description
This PR improves the usability of Tranqu's transpile() method by adding automatic type detection capabilities.
Key Changes
Added Program Format Auto-detection

Implemented new ProgramTypeManager class to automatically detect library from program type
Made program_lib argument optional
Example: When passing a Qiskit QuantumCircuit object, it's automatically recognized as "qiskit"

Added Device Format Auto-detection

Implemented new DeviceTypeManager class to automatically detect library from device type
Made device_lib argument optional
Example: When passing a Qiskit BackendV2 object, it's automatically recognized as "qiskit"

before:
result = tranqu.transpile(
    program=circuit,
    program_lib="qiskit",  # explicit specification required
    transpiler_lib="qiskit",
    device=backend,
    device_lib="qiskit"  # explicit specification required
)

after:
result = tranqu.transpile(
    program=circuit,  # QiskitCircuit object
    transpiler_lib="qiskit",
    device=backend  # BackendV2 object
)

@snuffkin  I have addressed all the points raised in your review comments. Could you please review the changes again?


chore(e2e): add an e2e environment to styleguidist

update all baseline images
now spinner will stop spinning when running e2e tests

yarn prepr output:

No components have been changed, nothing to do. Exiting.
🤔 If this is not what you expected, ensure that your commit messages follow the Conventional Commits specification (https://conventionalcommits.org) and try again.

Please rebase and merge. After merging, please:

[ ] update our internal deployment documentation to include use of new script yarn dev:e2e
[ ] duplicate this fix to tds-community

Why did you have to re-generate all the screenshots? None of them should have changed right? Except for spinner...
yarn test:e2e @tds/core-spinner -u

To the merger: see Enrico's checklist at the top for the steps to follow after merging this in.


:wrench: Enable update trigger on Image Webhook

Decision Record
When the agent fails to pull an image (could be because of a temporary network issue), we have to delete it and recreate it to trigger the webhook again.
Let's enable the webhook for updates as well, so that saving the failed image triggers the webhook, enabling us to retry pulling.
NB: If the size of the image is not 0, the agent will ignore it, so successfully pulled image will still be a "NO-OP".
Changes

[x] :recycle: Update webhooks after migrations
[x] :wrench: Enable update trigger on Image Webhook

@linkdd Many thanks!
Why changing the webhook creation logic (we don't update webhook if it's already exists)?

update_or_create was easier and clearer than:

filter
if found create
else filter+update

Is not easer or cleaner, you change the logic. We don’t update the webhook if it exists.

Le 17 mai 2024 à 10:50, David Delassus @.***> a écrit :
update_or_create was easier and clearer than:
filter
if found create
else filter+update
—
Reply to this email directly, view it on GitHub https://github.com/SaaShup/netbox-docker-plugin/pull/129#issuecomment-2117065317, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAPE3XYHXHU2IQAELZYDE7LZCXAGLAVCNFSM6AAAAABH2P7LLWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMJXGA3DKMZRG4.
You are receiving this because your review was requested.


Update to rspec3 syntax

This switches to rspec3 syntax.  I think this will need a major version bump.

We should probably also update the gemspec to require RSpec 3 or higher, right?

Or at least the version that the new syntax showed up.

After looking into it, this is supported since at least rspec 2.14 http://rspec.info/blog/2013/07/rspec-2-14-is-released/http://rspec.info/blog/2013/07/rspec-2-14-is-released/
I was wrong in thinking we might need a major version bump.  We do not need a major version bump or to change dependencies

I think the minimum is v2.8, so we do need to up that to v2.14+.

2.8 > 2.1, we already depend on 2.8 or later, which is later than 2.14

It's not, actually. Ruby gems is smart enough to figure that out.


remove redundant paas manifest.yml that is no longer required

Why
The GOV.UK PaaS container hosting platform is now decommissioned
What
Remove the manifest.yml file that was required for the paas deployment but is not redundant.
Technical writer support
Run pipeline to deploy
How to review
ensure it deploys

Addressed in PR-158.


defaultGroupOrder throws warning

Guidelines

Please include a demo of the issue/behavior/question you have

Please try to be as detailed as possible

You may fork one of the following starter templates if you would like:

CodeSandbox

StackBlitz

Describe the bug
I have a simple MT. By default upon display I want to group the data based on the first column. When I use the defaultGroupOrder: 0 option in my first column definition, I get the below warning ..
Warning: Failed prop type: Invalid prop children of type array supplied to MTableCell, expected a single ReactElement.
To Reproduce
Steps to reproduce the behavior:

Visit codeSandBox for more details
In Chrome right mouse click and select "inspect" option then in the inspect, chose "Console" option to see the warning

Expected behavior
Having defaultGroupOrder shouldn't throw warning as there are no arrays supplied to any table cells
Screenshots
N/A
Desktop (please complete the following information):

OS: [e.g. iOS] MacOs
Browser [e.g. chrome, safari] Chrome
Version [e.g. 22]

Smartphone (please complete the following information):
N/A
Additional context
N/A

Thank you very much for this!

I honestly don't understand why children is being validated by prop types.
 Please correct me if I'm wrong, but we should be able to just remove it, right?

It looks like the fix is to check if props.children is an array, and if it is, to just wrap it in a Fragment.
Like this: (tested and appears to work)
https://github.com/material-table-core/core/blob/190c58bb7eaeb393edca1d7906cff740a3fe8fa9/src/components/MTableCell/index.js#L77-L86
@Domino987 thoughts?

Thanks @oze4 . does that mean I have to manually fix it? or will it be released as a patch?

It should be patched and released once @Domino987 wakes up and takes a look (he may know of a better way to fix it).
As far as I can tell, it's just a warning and isn't breaking anything. Or is it breaking something on your end? Is it urgent?

@oze4 correct, it's just a warning. Haven't seen page crashing even while reloading.

Hi, looks like this originates from the group row. Could we just wrap the children there?

Ya @Domino987 fixing at the source is prob a better idea.

I'm not sure what I was thinking last night, the fix is so obvious now. Just had to add a fragment around the icon (lines 137 and 153)
https://github.com/material-table-core/core/blob/7fed6f476d5eb1265865a3608ba882382ba84784/src/components/MTableGroupRow/index.js#L137-L153

This fix has been published in v2.3.37.
On another note, @Domino987 I had to change some npm commands - the git push at the end of the new release:x command was causing issues on my end.
Now just have to do:
npm run release:patch && npm run git:push:tags
Just something minor to take note of.

Thanks @oze4 . Much appreciated.

@oze4 great. If this works now, it will be great

Thanks @oze4 @Domino987 just upgraded the package to 2.3.37 and warning is gone.


--wallet not working

5:47PM INF Listening for peers. addr: <IP_ADDRESS>:3000
panic: the private key specified is invalid: 67724480df7c62815c9891fd50492ea1b589df130a072db09afc664431a76f6e12eaa2f2688b8eb9cd64aa293378b6adce7d63bae070f8995e84e73da6ed745c

goroutine 1 [running]:
main.start(0xc0002bd0a8, 0x1b25580, 0xc0000a2000, 0x1b1d3c0, 0xc00000e010)
	/home/rkeene/devel/perlin-dev/wavelet/cmd/wavelet/main.go:334 +0x17d3
main.Run.func3(0xc0002a0160, 0x0, 0x0)
	/home/rkeene/devel/perlin-dev/wavelet/cmd/wavelet/main.go:274 +0x779
gopkg.in/urfave/cli%2ev1.HandleAction(0x18757c0, 0xc0002728d0, 0xc0002a0160, 0x0, 0x0)
	/home/rkeene/go/pkg/mod/gopkg.in/urfave/cli.v1@v1.20.0/app.go:490 +0xc8
gopkg.in/urfave/cli%2ev1.(*App).Run(0xc00029e000, 0xc000030080, 0x8, 0x8, 0x0, 0x0)
	/home/rkeene/go/pkg/mod/gopkg.in/urfave/cli.v1@v1.20.0/app.go:264 +0x590
main.Run(0xc000030080, 0x8, 0x8, 0x1b25580, 0xc0000a2000, 0x1b1d3c0, 0xc00000e010, 0xc00007e000)
	/home/rkeene/devel/perlin-dev/wavelet/cmd/wavelet/main.go:282 +0x1859
main.main()
	/home/rkeene/devel/perlin-dev/wavelet/cmd/wavelet/main.go:75 +0x7b

It doesn't work when using the path for the private-key either.

The error is because the key has invalid prefixed zero bits.
If you want to generate new key, don't pass the --wallet flag.
A new key will be generated for you.

Thanks Ahmad,
Is this able to be connected to the key generated from lens.perlin.net? That's where I sourced the private key used from.

Yes, you can use key to login on lens.
When you don't pass the --wallet flag, a new key will be generated and logged on the output.
e.g.

The privateKey value is your private key.
You can copy the key and paste it on lens.

Hmm, weird, happening to me as well. I can't use the key generated from lens on master branch. This is the key I'm using: 02c1501edb7c1590024121d488463e70cbbfb6e435e94a3f6915d465f1ca318e2f5022f962070be13d7989c741e758d649a0ae7634cb100055fef271e7fde368.
I'm guessing it's something to do with different config?

Using PR #246, I'm getting the following error: keys are invalid: failed to pass static puzzle as prefix length of checksum is 0, yet c1 is 1.

Issue #252.


added documentation for Database util

Fixes #3579

Thanks for the addition to docs. Let’s add a disclaimer at the beginning of the page that the DB Util should only be used in emergency cases when regular abapGit functionality doesn’t work. Also users should report such cases as new issues on github so we can fix the code that caused such situation.

@mbtools I will change the current disclaimer at the end and improve it a bit in the next days. On the other hand I'm not sure if that disclaimer is not a contradiction to the issue/use case why the documentation was added at all (see comment no. 1 of this PR). So maybe we should have that disclaimer + make sure deletion works even when the defined package doesn't exist anymore.

thanks 👍


Display chunks of audio file

Is your feature request related to a problem? Please describe.
I work on active learning with long audio files (~1h-2h),  I need to present chunks of these audio files to the user for the annotation. I can't really pre-cut the files as they can change during the process (it won't always be the same chunks). I don't really know if we can do this with Label Studio or how to do it.
Describe the solution you'd like
A start/end option for audio file that display only this segment to the user. A task will be a audio file with a start/end timestamp.
Describe alternatives you've considered
Create a server that create urls on the fly to serve Label Studio and import data during the process, but it doesn't seem very practical. Or re-upload the 2h file for each iteration...
Additional context
I have not yet finished setting up the model with Label Studio ML backend, so maybe there is a solution I didn't see in the guide.

@J-Petiot Nice idea! I think we can pass start/end as task data, e.g.:
Labeling config:
<AudioPlus value="$path" start="$start" end="$end">

Task data
{
 "path": "/1.mp3",
 "start": 10,
 "end": 20
}

As I remember LS can load only needed parts of audio, but your serving server must support streaming, LS uploaded files support it.

Thanks @J-Petiot for initialize the issue. We have the same challenge with long audio that cause OOM in the client's browser.
@makseq Just to make sure I understand, you were saying that we can already do that or did you mean to discuss as a future feature?
I would like to add another aspect it that we often need to annotate chunk of audio with corresponding transcription under the form of dialogue and it would benefit from this feature of loading a chunk of audio also.
For example:
  <Paragraphs
          audioUrl="$audio"
          name="dialogue"
          value="$dialogue"
          layout="dialogue"
          savetextresult="yes"
          nameKey="speaker"
          textKey="phrase"
          showPlayer="true"
  />

Hi! We have this functionality already and the config is exactly like that one you made.

Hi! We have this functionality already and the config is exactly like that one you made.

What do you mean? I tried this in LS:
<AudioPlus value="$path" start="$start" end="$end">

It didn't work and is not documented anywhere.
It'd be great to have it though!

<View>
  <Paragraphs audioUrl="$audio" value="$para" name="paragraphs"
              layout="dialogue" textKey="text" nameKey="author"
              showPlayer="true"
              />

  <Choices name="choices" toName="paragraphs" choice="multiple">
      <Choice value="Good quality"/>
      <Choice value="Fast speech"/>
  </Choices>
</View>

<!-- {"data": {
  "para": [
    {"text": "test 1", "author": "A", "start": 0.0, "end": 1.0},
    {"text": "test 2", "author": "B", "start": 1.0, "end": 2.0},
    {"text": "test 3", "author": "A", "start": 2.0, "end": 3.0}
  ],
  "audio": "/static/samples/game.wav"
}}
-->


Tabs to Spaces screws up middle indentation

Consider this line, in Atom 1.4.3 with a tabLength of 8:
(E100_C		(dP -0.1)	(dT 100))

Using the Tabs to Spaces command should produce 9 spaces for the two consecutive tabs. However, Atom produces 16 spaces, pretending the line isn't indented in the middle. The choice of 8 is unimportant: Atom incorrectly converts tabs to spaces for all tabLengths.

I'm not sure I'm understanding what you're saying is the problem. If you tell Atom that the tabLength is 8 then when you press Tab, it will either insert a tab character or eight spaces. Otherwise, they would be tab stops, which isn't a feature that Atom provides.

Atom clearly does provide tab stops, because pressing tab indents to the next multiple of tabLength, and tab characters extend to multiples of tabLength. Atom's implementation may be broken, but it's still there. The problems is that even though they are indented correctly, they are not treated correctly when converting between tabs and spaces.


Broken build?

I tried using latest shader_compiler which uses this commit: https://github.com/Games-by-Mason/glslang-zig/commit/af4f26fcb2abb3e43f8794eb58df3289ab3b842b
(Latest commit of this repo also doesnt fix the issue; it only modifies the readme)
Unfortunately, the build.zig seems to use old build syntax (idk I started using zig a week ago):
tringle_zig on  main [!] via ↯ v0.14.0-dev.2424+7cd2c1ce8 via ❄  impure (nix-shell-env) took 8s
➜ zig build
/home/bigsmarty/.cache/zig/p/1220d8a973a82dc73994c611172c007155aa1938be78f672f003a30c7dca057e3fff/build.zig:864:9: error: expected type 'Build.Step.Compile.HeaderInstallation.Directory.Options', found 'build.install_headers_options__struct_117660'
install_headers_options,
^~~~~~~~~~~~~~~~~~~~~~~
/home/bigsmarty/.cache/zig/p/1220d8a973a82dc73994c611172c007155aa1938be78f672f003a30c7dca057e3fff/build.zig:8:34: note: struct declared here
const install_headers_options = .{
~^
/nix/store/fr22vi0kf59smmnqdf7dpgiq4pn9kjp7-zig-0.14.0-dev.2424+7cd2c1ce8/lib/std/Build/Step/Compile.zig:311:29: note: struct declared here
pub const Options = struct {
^~~~~~
/nix/store/fr22vi0kf59smmnqdf7dpgiq4pn9kjp7-zig-0.14.0-dev.2424+7cd2c1ce8/lib/std/Build/Step/Compile.zig:509:42: note: parameter type declared here
options: HeaderInstallation.Directory.Options,
~~~~~~~~~~~~~~~~~~~~~~^~
referenced by:
build: /home/bigsmarty/.cache/zig/p/1220d8a973a82dc73994c611172c007155aa1938be78f672f003a30c7dca057e3fff/build.zig:597:28
runBuild__anon_72751: /nix/store/fr22vi0kf59smmnqdf7dpgiq4pn9kjp7-zig-0.14.0-dev.2424+7cd2c1ce8/lib/std/Build.zig:2296:44
22 reference(s) hidden; use '-freference-trace=24' to see all references
tringle_zig on  main [!] via ↯ v0.14.0-dev.2424+7cd2c1ce8 via ❄  impure (nix-shell-env) took 2s
➜
Or am I doing something wrong? Please help me out with this because I would really love to use shader_compiler

It does work on 0.13.0, I think I will just stay on that version for now

Hey! Welcome to Zig, and thanks for the report. :)
Looks like both glslang-zig and shader compiler were relying on a feature of Zig's type system that recently changed. I just pushed a commit to both repos fixing this, so you should be able to update to the latest Zig build if you want to. I tested on 0.14.0-dev.2435+7575f2121.
I think I'll tag a release on the next major Zig release so that it's easier for people to figure out which version of the library is compatible with their current Zig version.
Let me know if you run into any other issues or you have trouble with the upgrade!


fixed pom.xml files so that unit and integration tests are run.

Fix pom.xml files to run all tests.
junit 4 and junit 5 tests are now run when mvn test is executed.
resolves #536

Updated to reference #556


Update errdefs to 0.3.0

Uses the new github.com/containerd/errdefs/pkg module which is intended to hold less stable utility functions separately from the stable github.com/containerd/errdefs error types.
Includes temporary update to hcsshim until a release is cut there. This will be updated before 2.0 after hcsshim releases 0.12.8 with the errdefs fix.
Closes #10478

/retest


omniverse-sim.py CPU pipeline argument error -- fix

Hello everyone,
Running environment:

Ubuntu 22.04
ROS2 Humble
IsaacSim 4.1.0
IsaacLab commit: 0b4f7bd

When running the repository for the GO2 script, the following happens:
james@workstation:~/unitree_resources/go2_omniverse$ ./run_sim.sh
#All required rosdeps installed successfully
Starting >>> carter_navigation
Starting >>> custom_message
Starting >>> isaac_ros2_messages
Starting >>> isaac_ros_navigation_goal
Starting >>> isaac_tutorials
Finished <<< carter_navigation [0.18s]
Finished <<< isaac_tutorials [0.18s]
Finished <<< custom_message [0.29s]
Finished <<< isaac_ros2_messages [0.29s]
Finished <<< isaac_ros_navigation_goal [0.41s]

Summary: 5 packages finished [0.53s]
#All required rosdeps installed successfully
Starting >>> go2_interfaces
Finished <<< go2_interfaces [0.38s]

Summary: 1 package finished [0.50s]
Traceback (most recent call last):
  File "/home/james/unitree_resources/go2_omniverse/main.py", line 25, in <module>
    from omniverse_sim import run_sim
  File "/home/james/unitree_resources/go2_omniverse/omniverse_sim.py", line 35, in <module>
    AppLauncher.add_app_launcher_args(parser)
  File "/home/james/IsaacLab/source/extensions/omni.isaac.lab/omni/isaac/lab/app/app_launcher.py", line 253, in add_app_launcher_args
    arg_group.add_argument("--cpu", action="store_true", help=argparse.SUPPRESS)
  File "/home/james/miniconda3/envs/isaaclab/lib/python3.10/argparse.py", line 1448, in add_argument
    return self._add_action(action)
  File "/home/james/miniconda3/envs/isaaclab/lib/python3.10/argparse.py", line 1650, in _add_action
    action = super(_ArgumentGroup, self)._add_action(action)
  File "/home/james/miniconda3/envs/isaaclab/lib/python3.10/argparse.py", line 1462, in _add_action
    self._check_conflict(action)
  File "/home/james/miniconda3/envs/isaaclab/lib/python3.10/argparse.py", line 1599, in _check_conflict
    conflict_handler(action, confl_optionals)
  File "/home/james/miniconda3/envs/isaaclab/lib/python3.10/argparse.py", line 1608, in _handle_conflict_error
    raise ArgumentError(action, message % conflict_string)
argparse.ArgumentError: argument --cpu: conflicting option string: --cpu

I am aware that the repo works for IsaacSim 4.0, and me using 4.1 might be the cause for this.
Nevertheless, there is a simple fix for anyone encountering it: LN18 in omniverse_sim.py change --cpu to something like '--cpu-pipeline'.
This will fix the argument conflict.

@PetriJF Hello! Thanks for your help, I tested it with 4.1 and saw a significant drop in simulation speed (the dog is like a Neo from the Matrix), What about you? Also, I cannot start more than 1 dog in the sim with 4.1, looks tile 4.0 and 4.1 has many difference..

@PetriJF Hello! I have a solution for the speed drop, can you please check it? https://github.com/abizovnuralem/go2_omniverse/issues/30

Hey @abizovnuralem
I will take a look tomorrow when I get into the office!
Will let you know


ELMo needs update

There are unaddressed comments from https://github.com/dmlc/gluon-nlp/pull/227

Looking more into the code, the following needs update:

model block should not be adding to the input data such as bos and eos. it's the job of transform functions.
This looks wrong. https://github.com/dmlc/gluon-nlp/blob/master/src/gluonnlp/model/elmo.py#L245. After the transpose, the data is no longer in the (N, T, C) layout.
Need a proper codec-based vocab. current ELMoCharacterVocab is not a vocab, and thus the preprocessing logic is leaking into many places.
ELMo blocks should be made into hybrid blocks.


Backport maximum allowed sessions to v7.0

This already exists in v7.1.

@mmoayyed can you take this one into account as the change exists in the master branch: https://github.com/apereo/cas/blob/master/api/cas-server-core-api-configuration-model/src/main/java/org/apereo/cas/configuration/model/core/authentication/policy/UniquePrincipalAuthenticationPolicyProperties.java#L27

Hello. The change is not a fix, and therefor does not belong in this release. Features go into feature releases.

Copy that. No problem.


Fix stubs for URL.createObjectURL and URL.revokeObjectURL in engine

Overview
The URL.createObjectURL and URL.revokeObjectURL functions are currently stubbed in Babylon.js nativeEngine.ts.
Implement these functions so their behavior matches current browser implementations.
Notes
This task was originally capture in issue https://github.com/BabylonJS/BabylonNative/issues/62.

Try to remove the stub and see if everything is okay.


GEODE-9924: Avoid merging repeat test logs

PROBLEM
Our repeat test tasks merge the output from all executions of a given
test class, making it very difficult to diagnose failures in repeat
tests.
CAUSE
In order to run tests repeatedly, our repeat test tasks override Gradle
code to allow a test class to execute more than once. Gradle's test
reporting code does not expect this. It directs the output from each
test to a log associated with the test class name, not with the specific
execution of the test class. This results in merging the outputs from
different executions of the same class.
SOLUTION
Change Gradle to distinguish separate executions of a test class, and to
log the output from each execution separately.
Components:

New InstanceNamingTestResultProcessor class: Wraps a given test
result processor to append an instance identifier (a simple counter)
to the name of the test class when reporting results. Gradle
associates the output with this instance-specific name instead of with
class name.
New InstanceNamingForkingTestClassProcessor class: Wraps a
ForkingTestClassProcessor to report via an
InstanceNamingTestResultProcessor.
Change RepeatTestExecuter to use an
InstanceNamingForkingTestClassProcessor.

This commit also reverts GEODE-9912, which partially addressed the same
problem by adding an execution ID to each log line from a DUnit ChildVM.
The execution ID made it possible to identify the source of each merged
log line. GEODE-9924 (this commit) makes the changes from GEODE-9912
unnecessary.
NOTES

Gradle now creates distinct XML and HTML report files for each
execution of a given test class.
The test summary HTML page created by Gradle lists each test class
execution separately. If a class named FooTest executes 50 times (as
in a stress test run), the summary will list FooTest-01,
FooTest-02, ..., FooTest-50 as if they were separate test classes.

I changed the names of the new classes after I wrote the commit message. Sorry about that.


Migrating to Kotlinter gradle plugin.

Summary

Migrating off of previous plugin.
How It Was Tested

CI passing.

Changes Unknown when pulling c482bd985acbceaddf7b14f0e4a107e70130ecc3 on TOA-180/kotlinter_plugin into ** on development**.


Add patch http method to OAuthClient

What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[x] Feature
[ ] Code style update
[ ] Refactor
[ ] Other, please describe:

[x] You consent that the copyright of your pull request source code belongs to Authlib's author.

Thanks


Can you provide a preview of this module?

Can you provide a preview of this module? You could use netlify it lets you easily synch github repositories with a free web hosting.
Netlify
Or at least provide some screenshot of this module at work, if you want I might help to do those things if you add me as collaborator, does it sounds good?
And I could update the README too.

Oops! Sorry, I had it linked on the A-Frame projects page and on my page but not on here yet.
http://code-panda.com/pages/projects/a_sky_forge_example
That should do the trick though. I should say that it's a bit heavy on the GPU usage. I still need to find out how to better remedy that, but my efforts toying around with the GLSL just haven't proven useful (I'm still such a noob to GLSL). I'll add the link on the page next time I push an update.

Ok, I've added this into the README.md file. So I'm going to close out this issue. Thiank you so much for your feedback ^_^.


Duplicate

Is this project redundant to this one? Their API seems almost identical.
https://github.com/step-up-labs/firebase-database-dotnet

This repo is a combination of multiple repos that use Firebase for C#. On the bottom of the ReadMe you will see @rlamasb giving credit to all the repos they used.

So is this a kind of "best of"? I'm thinking of switching as I'm struggling to get the other one to work with Xamarin. A few samples would be really helpful. Does anyone have any, specifically Offline and Realtime?

Good luck. I switched to Azure because of limited support from Firebase. A Google employee told me no one there is currently working on a c# library.

I am in the process of switching from Azure. :( There is so much ceremony in building a WebAPI layer (even with Azure Mobile App SDK) and it doesn't do realtime sync, which is the big deficiency for me. Cosmos DB doesn't do offline. So the only two hosted solutions I have found that do realtime and offline are Firebase and AWS AppSync, and neither have .Net SDKs. I'm also looking at Couchbase and Realm for an enterprise deployment, and both are great, but too costly for a hobbyist project (mainly because you have to pay for a VM to host them which can add up fast). Would love to hear from anyone willing to share lessons learned in this space.

I want to add another issue I have with the Azure Mobile App SDK is the backend has not been ported to .Net Core even though the dependencies have been ready for a year, and there doesn't seem to be any development being invested in it. This also feels like abandonware to me.


if candidate recordings are not temporal, exit early in Kar2019

evaluating models on Kar2019 takes notoriously long and often fails with out-of-memory errors (see e.g. http://braintree.mit.edu:8080/job/run_benchmarks/813 for a recent example) -- but ultimately most models (feedforward) get a nan score (cannot predict) anyway.
The out-of-memory errors occur especially for models with large activations when repeating their activations over time (https://github.com/brain-score/model-tools/blob/87e0a0fb5aec46e6c2c4aae06a3eb2040752e52c/model_tools/brain_transformation/temporal.py#L39-L47).
To solve the unnecessary use of compute resources for feedforward models, this PR runs a single image on the model candidate and checks whether any of the neuroids vary over time. If not, it exits early with a nan score.

jenkins test timed out, but the Kar2019 tests passed. Merging...


Mock rrdcached via tests

Currently test suite looks for a specific existing socket where rrdcached is running.
Better: Start & stop rrdcached via tests. (See issue #1)
Best: Mock rrdcached entirely.
Reference: http://godoc.org/github.com/stvp/tcptest

Semi-reference: https://github.com/alecthomas/gozmq/blob/master/zmq_test.go


Update the sign up form & success message

Currently, Users do not see any guidance about what to expect in the onboarding process if they go straight from the homepage to the access request form. We should update the form page and success message to better guide users through the process.
Links seen in the mockups:
Learn more about the onboarding process
Watch training videos
Read how-to guides

also tagging @vz3

@hmyers-cms what's the link to the figma designs? (doing this for all designs would be great)

I see it now in the design files, so I'm good. Here's the link.


Editor Tool - Chat: Add support for multiple chat sessions

Just like ChatGPT, add multiple chats instead just one.


All venv in one dir

Hi, given the scenario where all venvs are in $HOME/.venv how should we configure the plugin?
I tried to set path as $HOME/.venv, parents to 0.But nothing is found.
I tried to set path to $HOME, parents to 0 and name to .venv but then the list will include every subdir of $HOME/.venv.
Is it possible to only list the "single" venvs and avoid "duplicates" ?
The structure is like:
$HOME/.venv/:

venv1
venv2
venv3
etc.

Thanks!

I have a similar situation, whereby all my venv folders begin with the following pattern:
venv_
as such I've tried using a regex pattern in the plugin config file, but to no avail - I'm not a Lua/nvim expert, and yet I have tried many different regex styles and none worked.
Any suggestion on how to make the plugin recognise each venv_ folder as one venv, and listing only each main folder?
Thanks in advance

Hey,
The plugin cant do this right now actually. It always wants to know the name of the venv to search for, and in this scenario you need it to be more dynamic (every folder under a certain path).
Since the plugin was created, it has changed so much to try and support more and more scenarios, and the code is getting a bit messy now and hard to understand.
Im experimenting with a new version of the plugin where the entire search is much more configurable by the user. I will write more about it if Im happy with how its working out. :)

I just tried and the regexp branch of the plugin can easily do this.
This fd command will list your venvs: fd 'python$' ~/.venv
/home/cado/.venv/venv1/bin/python
/home/cado/.venv/venv2/bin/python
/home/cado/.venv/venv3/bin/python

Add the fd command to your plugin configuration:

      require("venv-selector").setup {
        settings = {
          search = {
            my_venvs = {
              command = "fd 'python$' ~/.venv",
            },
          },
        },
      }

And they will show up in the telescope viewer.

Please note that in order to actually activate the venv, a python file must be opened so the LSP is running in neovim.

Closing this since it should work fine in regexp version of the plugin. Reopen if it doesn't or if you have questions. :)


Crash in Dao constructor

Hi, I am using your library in one of our production apps and we have discovered one weird bug in Xperia phones with android 4.0.
The crash looks like this
	at android.app.ActivityThread.handleBindApplication(ActivityThread.java:3957)
	at android.app.ActivityThread.access$1300(ActivityThread.java:123)
	at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1185)
	at android.os.Handler.dispatchMessage(Handler.java:99)
	at android.os.Looper.loop(Looper.java:137)
	at android.app.ActivityThread.main(ActivityThread.java:4424)
	at java.lang.reflect.Method.invokeNative(Native Method)
	at java.lang.reflect.Method.invoke(Method.java:511)
	at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:817)
	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:584)
	at dalvik.system.NativeStart.main(Native Method)
Caused by: java.lang.IllegalArgumentException: DatabaseErrorHandler param value can't be null.
	at android.database.sqlite.SQLiteOpenHelper.<init>(SQLiteOpenHelper.java:95)
	at com.hannesdorfmann.sqlbrite.dao.DaoManager$OpenHelper.<init>(SourceFile:102)
	at com.hannesdorfmann.sqlbrite.dao.DaoManager.<init>(SourceFile:39)
	at com.hannesdorfmann.sqlbrite.dao.DaoManager.<init>(SourceFile:33)```

I`ve found that problem is that in one of your DaoManager constructor you are sending DatabaseHandler as null, I fixed this by passing instance of DefaultDatabaseErrorHandler() to the more expressive constructor. I think that its worth to fix this. Thanks!

Hm, I'm not entirely sure what actually the problem is. I have a custom SQLiteOpenHelper called OpenHelper. Yes I pass null as default error handler in DaoManager costructor
Passing null as error handler means using the default error handler according to the [android documentation](https://developer.android.com/reference/android/database/sqlite/SQLiteOpenHelper.html#SQLiteOpenHelper(android.content.Context, java.lang.String, android.database.sqlite.SQLiteDatabase.CursorFactory, int, android.database.DatabaseErrorHandler)):

the DatabaseErrorHandler to be used when sqlite reports database corruption, or null to use the default error handler.

If you want to have a look source code of android SQLiteDatabase implementation you will see
mErrorHandler = errorHandler != null ? errorHandler : new DefaultDatabaseErrorHandler();

So not sure where your issue comes from, but it seems to be a sony ericsson ROM problem

Yes, I know that its problem just with Xperias (or maybe android version), but we had 14 of this crashes in week, so it was critical for us to fix this somehow. If you look at this file https://android.googlesource.com/platform/frameworks/base/+/1c423b8/core/java/android/database/sqlite/SQLiteOpenHelper.java on line 93, you can see that earlier this crash was happening if DatabaseHandler was null.


Fix new Sonar smells

Here is a report, though: https://sonar.spring.io/project/issues?id=org.springframework.kafka%3Aspring-kafka-dist%3Asonar&resolved=false&sinceLeakPeriod=true&types=CODE_SMELL

I haven't found a way to disable that rule on Sonar.


Update article.md

What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

complete the --signoff. See  https://github.com/cubefs/cubefs/pull/2362/checks?check_run_id=15415074889


AST: Don't mangle weak/unowned as part of a property's type

LibraryEvolution.rst stipulates that adding or removing 'weak',
'unowned' and 'unowned(unsafe)' is a resilient change if the type
is resilient. This means we should not mangle these attributes as
part of the accessor declaration.

@swift-ci Please smoke test


REPL crashes with elm.json without "elm/json" dependency

When running REPL in a project with elm.json that has type: "package", a simple evaluation fails with message elm: Map.!: given key is not an element in the map.
See also comment about package case in #1747.
This is unfortunate, REPL is useful when writing packages for quick testing.
Specs: Elm 0.19.0 on Mac 10.13.6

Running 0 < 1 in REPL right after doing elm init works, i.e. with the following elm.json:
{
    "type": "application",
    "source-directories": [
        "src"
    ],
    "elm-version": "0.19.0",
    "dependencies": {
        "direct": {
            "elm/browser": "1.0.0",
            "elm/core": "1.0.0",
            "elm/html": "1.0.0"
        },
        "indirect": {
            "elm/json": "1.0.0",
            "elm/time": "1.0.0",
            "elm/url": "1.0.0",
            "elm/virtual-dom": "1.0.0"
        }
    },
    "test-dependencies": {
        "direct": {},
        "indirect": {}
    }
}

Running 0 < 1 in REPL after writing a package type elm.json resulted in the aforementioned failure, i.e. with the following elm.json:
{
    "type": "package",
    "name": "user/repository",
    "summary": "A summary.",
    "license": "BSD-3-Clause",
    "version": "1.0.0",
    "exposed-modules": [
    ],
    "elm-version": "0.19.0 <= v < 0.20.0",
    "dependencies": {
        "elm/core": "1.0.0 <= v < 2.0.0"
    },
    "test-dependencies": {}
}

The root issue is that functions like Platform.worker need some logic from elm/json to handle flags. As a result, it is mandatory to have an elm/json dependency in any "application" right now.
It is not a true requirement for a package though, since packages may not use it at all. So as it is right now, packages have the minimal dependencies and users always have the necessary elm/json dependency because of the "application" requirement.
So this should only appear in very specific circumstances. I'm not certain what would be a nice fix that keeps packages from needing superfluous dependencies.


Use new teleport tube API if available

Fixes #124
Try to find out Pipeworks API version and use API if it is available.
Also adds support for new Pipeworks API while attempting to keep support for old API attempting to detect points where public function signatures in Pipeworks API were changed.
Should allow setting receive configuration again with rewritten Pipeworks API.

Didn't test but I think it is good now.


Emtpy balance and history

Got a "Connect v8 is deprecated. Please ask your DeFi platform/wallet developers (not Trezor) to update to Connect v9.", On Edge, Version 121.0.2277.83 (Official build) (64-bit).
Also, getting blank/empty history and balance on wallet. Transfers are failing. This happens also on Google Chrome, latest stable build 121.0.6167.86 64 bits.
TzStats API Status: OFFLINE

Yep, the empty balance is due to the #24 which is the API token getting rate limited.
It should be fixed by tomorrow, but I will add a feature that lets users add their own API tokens soon.

As for the issue with Edge I'll have to test that. Trezor Connect is already at v9, so it's likely something else.

I'm unable to replicate the issue for Microsoft Edge. If you're able to provide screenshots or a video I'm willing to look into this again.


Grimlite Rev v1.3 (Skymeister)

Grimoire is always shown ON-TOP, Any tool from TOOLS, PACKETS isn't shown on top anymore but behind the client which is inconvenient. thank you for fixing the Accept quest issue regardless <3

I have reverted the On-top Priority update on every panels and changed the Grim main panel's behavior to a different one, on the newer update. Hope you're satisfied with it!


multicast packets are sent to a single interface only

Which version are you using?
v0.23.7
Which operating system are you using?

[ ] Linux amd64 standard
[ ] Linux amd64 Docker
[ ] Linux arm64 standard
[ ] Linux arm64 Docker
[ ] Linux arm7 standard
[ ] Linux arm7 Docker
[ ] Linux arm6 standard
[ ] Linux arm6 Docker
[x] Windows amd64 standard
[ ] Windows amd64 Docker (WSL backend)
[ ] macOS amd64 standard
[ ] macOS amd64 Docker
[ ] Other (please describe)

Describe the issue
discussed in #1963
https://stackoverflow.com/questions/71636874/send-udp-datagrams-to-multiple-interfaces-with-multicast-address
Describe how to replicate the issue

have two interfaces with multicast enabled on both
start the server
publish
read with multicast

Did you attach the server logs?

no
Did you attach a network dump?

no

fixed in v1.1.0


kubernetes-controller-tools: set version

Description of changes
controller-gen likes to embed its internal version in generated files. As such, while the generated code is compile time equivalent to official releases, there is some thrashing if some users use nix and others us upstream artefacts. This change patches the version package and runtime value via ldflags.
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/1552


Question: how does PhantomJS gets installed?

The docs say that PhantomJS is a dependency but I don't see how it "automagically" gets installed. It's not listed on the package.json. So how does it get installed?

pageres => screenshot-stream => phantom-bridge => phantomjs
https://github.com/sindresorhus/phantom-bridge/blob/23cd7e6f7596b680f990d300d1f0cc22fc2706c5/package.json#L31

@sindresorhus Thanks :)


Revert "Merge pull request #745 from mattermost/mm9922"

Summary
Unfortunately, the actual behaviour that we want where hovered links in posts always show the URL isn't possible with this behaviour since I haven't found a way to identify what in the WebView has been hovered from these events, so this will have to be handled from the web app.
Issue link
https://mattermost.atlassian.net/browse/MM-9922

@yuya-oc I believe this is in release-4.1 branch?

Thanks!
@hmhealey Just curious, is there something like class which we can know whether we should show URL? Possibly we might be able to try again by combining other APIs.

@yuya-oc I couldn't find anything on the update-target-url event that allows us to determine what element on the page triggered the event. The event's target field just points to the webview.
To get the functionality we want, I'm looking at reusing the component we have in the channel sidebar that renders an <a> tag in the web app and a <button> with a custom context menu in the desktop app since that works how we want. That just means we'll need to apply it manually when we want to get rid of the hovered URL

@hmhealey Yeah, the event only emits URL. So I had thought possibly we could get the target element by injecting JavaScript into webview. But your approach is already working in the LHS, so it would be better result.

Actually, if that's possible by injecting JavaScript to look where the link is, that would be preferable, so I'll take a look at doing that. The approach in the LHS requires some more complicated logic and an extra library that adds custom right click menus that I'd like to get rid of if possible


Checkbox Group expects stable object as modelValue

Describe the bug
This watch https://github.com/bootstrap-vue-next/bootstrap-vue-next/blob/48d25fa75d90cbe76745b2ce0a1e8a6bd542e1a2/packages/bootstrap-vue-next/src/components/BFormCheckbox/BFormCheckboxGroup.vue#L163C2-L163C2 will fire recursively if modelValue array is not the same object. In my reproduction I've tried to show real life example where user would need to modify modelValue array.
I investigated Checkbox Group only but other components may also be affected.
Reproduction
https://stackblitz.com/edit/github-jfyohs-ef8cfh?file=src%2FPages%2FIndex.vue
Used Package Manager
pnpm
Validations

[X] Have tested with the latest version. This is still alpha version and sometime things change rapidly.
[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A, please open a GitHub Discussion instead.
[X] The provided reproduction is a minimal reproducible of the bug.

Updated reproduction link with the correct one.

@VividLemon potentially radio group is also affected,  modelValue typings imply it can be object or array but I haven't tested this case https://github.com/bootstrap-vue-next/bootstrap-vue-next/blob/8f31d37a43389fff40743b0c25f4720419cdf2a4/packages/bootstrap-vue-next/src/components/BFormRadio/BFormRadio.vue#L132-L142

@VividLemon potentially radio group is also affected, modelValue typings imply it can be object or array but I haven't tested this case
https://github.com/bootstrap-vue-next/bootstrap-vue-next/blob/8f31d37a43389fff40743b0c25f4720419cdf2a4/packages/bootstrap-vue-next/src/components/BFormRadio/BFormRadio.vue#L132-L142

https://github.com/bootstrap-vue-next/bootstrap-vue-next/pull/1626

With this fix checkbox group has internal sync that's not in sync with modelValue prop, try click "no" in repro https://stackblitz.com/edit/github-jfyohs-yazvho?file=src%2FPages%2FIndex.vue it will check while modelValue array is empty

That's going to be the expected behavior


annotation files

Could you share the annotation files?Thank you!

Could you share the annotation files?Thank you!

@trugle I was previously in contact with waiyulam but he couldn't share the files.
If you are interested to get ORCNN working with your own dataset, please have a look at my git repository: https://git.wur.nl/blok012/sizecnn. This repository also includes an instruction on how to annotate the images with LabelMe and how to convert the annotations to COCO-ORCNN format.
Hope this works!

Thank you very much for your guidance. I want to ask if my target edge is not suitable for circular fitting, can it still work well? I have tried to train with a small amount of data, but the results are not ideal. Is there a problem with the annotation method? Or the network does not work well for non-circular targets. Looking forward to your reply.
------------------ 原始邮件 ------------------
发件人:                                                                                "waiyulam/ORCNN"                                                                                @.>;
发送时间: 2022年7月19日(星期二) 凌晨5:36
@.>;
<EMAIL_ADDRESS>主题: Re: [waiyulam/ORCNN] annotation files (Issue #7)
@trugle I was previously in contact with waiyulam but he couldn't share the files.
If you are interested to get ORCNN working with your own dataset, please have a look at my git repository: https://git.wur.nl/blok012/sizecnn. This repository also includes an instruction on how to annotate the images with LabelMe and how to convert the annotations to COCO-ORCNN format.
Hope this works!
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

@trugle I was previously in contact with waiyulam but he couldn't share the files.
If you are interested to get ORCNN working with your own dataset, please have a look at my git repository: https://git.wur.nl/blok012/sizecnn. This repository also includes an instruction on how to annotate the images with LabelMe and how to convert the annotations to COCO-ORCNN format.
Hope this works!

Sorry to disturb you，I want to ask if the target edge is not approximately circular can it still work well?

Sorry to disturb you，I want to ask if the target edge is not approximately circular can it still work well?
------------------ 原始邮件 ------------------
发件人:                                                                                "waiyulam/ORCNN"                                                                                @.>;
发送时间: 2022年7月19日(星期二) 凌晨5:36
@.>;
<EMAIL_ADDRESS>主题: Re: [waiyulam/ORCNN] annotation files (Issue #7)
@trugle I was previously in contact with waiyulam but he couldn't share the files.
If you are interested to get ORCNN working with your own dataset, please have a look at my git repository: https://git.wur.nl/blok012/sizecnn. This repository also includes an instruction on how to annotate the images with LabelMe and how to convert the annotations to COCO-ORCNN format.
Hope this works!
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Sorry to disturb you，I want to ask if the target edge is not approximately circular can it still work well?
------------------ 原始邮件 ------------------
发件人:                                                                                "waiyulam/ORCNN"                                                                                @.***>;
发送时间: 2022年7月19日(星期二) 凌晨5:36
@.***>;
<EMAIL_ADDRESS>主题: Re: [waiyulam/ORCNN] annotation files (Issue #7)
@trugle I was previously in contact with waiyulam but he couldn't share the files.
If you are interested to get ORCNN working with your own dataset, please have a look at my git repository: https://git.wur.nl/blok012/sizecnn. This repository also includes an instruction on how to annotate the images with LabelMe and how to convert the annotations to COCO-ORCNN format.
Hope this works!
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Both the amodal and the visible mask can be drawn as polygons, rectangles and circles.


[AIRFLOW-528] Add missing parameter in webserver systemd file

Dear Airflow maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[x] My PR addresses the following AIRFLOW-528 issues and references them in the PR title.

Description
Without "--daemon" flag I cannot stop systemd service and it end up in eternal deactivating state, with "--daemon" flag start, stop, and restart works.
Example:
● airflow-webserver.service - Airflow webserver daemon
   Loaded: loaded (/etc/systemd/system/airflow-webserver.service; enabled; vendor preset: enabled)
   Active: deactivating (stop-sigterm) since Tue 2017-11-21 16:43:06 UTC; 1min 36s ago
  Process: 13481 ExecStop=/bin/kill -s TERM $MAINPID (code=exited, status=0/SUCCESS)
 Main PID: 13300 (airflow)
    Tasks: 2
   Memory: 84.8M
      CPU: 56.188s
   CGroup: /system.slice/airflow-webserver.service
           └─13300 /usr/bin/python3 /usr/local/bin/airflow webserver -p 8082 --pid /run/airflow/webserver.pid

Nov 21 16:43:31 airflow airflow[13300]: /usr/local/lib/python3.5/dist-packages/flask/exthook.py:71: ExtDeprecationWarning: Importing flask.ext.cache is deprecated, use flask_cache instead.
Nov 21 16:43:31 airflow airflow[13300]:   .format(x=modname), ExtDeprecationWarning
Nov 21 16:43:31 airflow airflow[13300]: [2017-11-21 16:43:31 +0000] [13391] [INFO] Worker exiting (pid: 13391)
Nov 21 16:43:31 airflow airflow[13300]: /usr/local/lib/python3.5/dist-packages/flask/exthook.py:71: ExtDeprecationWarning: Importing flask.ext.cache is deprecated, use flask_cache instead.
Nov 21 16:43:31 airflow airflow[13300]:   .format(x=modname), ExtDeprecationWarning
Nov 21 16:43:31 airflow airflow[13300]: [2017-11-21 16:43:31 +0000] [13389] [INFO] Worker exiting (pid: 13389)
Nov 21 16:43:31 airflow airflow[13300]: /usr/local/lib/python3.5/dist-packages/flask/exthook.py:71: ExtDeprecationWarning: Importing flask.ext.cache is deprecated, use flask_cache instead.
Nov 21 16:43:31 airflow airflow[13300]:   .format(x=modname), ExtDeprecationWarning
Nov 21 16:43:31 airflow airflow[13300]: [2017-11-21 16:43:31 +0000] [13390] [INFO] Worker exiting (pid: 13390)
Nov 21 16:43:35 airflow airflow[13300]: [2017-11-21 16:43:35 +0000] [13370] [INFO] Shutting down: Master

Tests

[x]  does not need testing for this extremely good reason:
Tested on Ubuntu:

Distributor ID:	Ubuntu
Description:	Ubuntu 16.04.3 LTS
Release:	16.04
Codename:	xenial

With "--daemon" flag service successfully stops:
● airflow-webserver.service - Airflow webserver daemon
   Loaded: loaded (/etc/systemd/system/airflow-webserver.service; enabled; vendor preset: enabled)
   Active: inactive (dead) since Tue 2017-11-21 16:46:38 UTC; 24min ago
  Process: 14500 ExecStop=/bin/kill -s TERM $MAINPID (code=exited, status=0/SUCCESS)
  Process: 14476 ExecStart=/usr/local/bin/airflow webserver -p 8082 --daemon --pid /run/airflow/webserver.pid (code=exited, status=0/SUCCESS)
 Main PID: 14476 (code=exited, status=0/SUCCESS)

Nov 21 16:46:29 airflow systemd[1]: Stopping Airflow webserver daemon...
Nov 21 16:46:29 airflow airflow[14398]: /usr/local/lib/python3.5/dist-packages/flask/exthook.py:71: ExtDeprecationWarning: Importing flask.ext.cache is deprecated, use flask_cache instead.
Nov 21 16:46:29 airflow airflow[14398]:   .format(x=modname), ExtDeprecationWarning
Nov 21 16:46:29 airflow airflow[14398]: Running the Gunicorn Server with:
Nov 21 16:46:29 airflow airflow[14398]: Workers: 4 sync
Nov 21 16:46:29 airflow airflow[14398]: Host: <IP_ADDRESS>:8082
Nov 21 16:46:29 airflow airflow[14398]: Timeout: 120
Nov 21 16:46:29 airflow airflow[14398]: Logfiles: - -
Nov 21 16:46:29 airflow airflow[14398]: =================================================================
Nov 21 16:46:38 airflow systemd[1]: Stopped Airflow webserver daemon.

Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Codecov Report

Merging #2804 into master will increase coverage by 0.06%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2804      +/-   ##
==========================================
+ Coverage   73.43%   73.49%   +0.06%
==========================================
  Files         158      158
  Lines       11958    11958
==========================================
+ Hits         8781     8789       +8
+ Misses       3177     3169       -8

Impacted Files
Coverage Δ

airflow/models.py
87.25% <0%> (-0.05%)
:arrow_down:

airflow/jobs.py
79.49% <0%> (+0.26%)
:arrow_up:

airflow/utils/helpers.py
56.32% <0%> (+2.87%)
:arrow_up:

airflow/task_runner/bash_task_runner.py
100% <0%> (+6.66%)
:arrow_up:

Continue to review full report at Codecov.

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

Simple indicates keep running in foreground. We don’t have issues with the as is situation?  (Rh 7)

I also don't have any issues on Ubuntu 16.04.


Trouble pushing code

Hi Shinichi,
I'm struggling to push my code with questions/issues. I have a feeling it's because I clicked on a file that I wasn't sure if I should include or not (it must have been very large!). Is there a way to delete the committed code and then re-commit?

I tried to do it again with just the .rmd file but it must still be getting stuck on the first one so now it says I have 2 waiting to be pushed.

@madshowie - you cannot push your AMPHBIANS.shp  it is too big - just read your warning message.
What you can do is push everything else - and send me that file via one drive link

I'm not sure how I can only push the others because they've already been committed and have disappeared from the Git window. And I can't seem to get rid of the ones already committed - is there a special way to do so?

@madshowie - OK - what you can do it.

Save the new files somewhere
Delete the current file
Create a folder via R studio
Then, you can move new files to this new folder
Push new things apart from the big data file

Does it make sense?

That makes sense, but unfortunately didn't work :(
I've saved them into a new folder Maddi2 and it is just showing the files as deleted in the git tab.

Ok I fixed it! I'm just going to let it update for a little bit before I try push again. I'm going to tidy up my graphs this weekend and then hopefully finish off the bibliometric coupling graphs on Monday.

The files weren't showing in my git for some reason. Here is a link to the files. Maybe on Tuesday you can take a look at my screen and see what's going on.
https://1drv.ms/u/s!AlnYtVGXEhWBgWLmsnwK8x6jdS2Q?e=8HIX9z

Hi @madshowie https://github.com/madshowie,
The file is huge - it seems like you downloaded data for all Amphibians?
If this is the case, would you be able to download only data for the
species you have in your labelled references?
This should result in a much smaller and more manageable file.
On Fri, Jul 15, 2022 at 7:43 PM madshowie @.***> wrote:

The files weren't showing in my git for some reason. Here is a link to the
files. Maybe on Tuesday you can take a look at my screen and see what's
going on.
https://1drv.ms/u/s!AlnYtVGXEhWBgWLmsnwK8x6jdS2Q?e=8HIX9z
—
Reply to this email directly, view it on GitHub
https://github.com/itchyshin/bibliometrics_2022/issues/16#issuecomment-1185370894,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADDIDL3K34FQEFRGC6PNORTVUEXC7ANCNFSM53UY6QAA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Hi Losia! No I don't think I can because it's straight from IUCN. However, that shapefile is for species distribution mapping which perhaps it might be easier to source externally. My main plan was to compare country productivity with species distribution so if I can't do the map myself I can always download the map from IUCN and compare with country productivity. Do you think this would be ok?

hi, yes, the  easier way is better - if you can download individual maps
and compare manually, this will be good.
On Fri, Jul 15, 2022 at 8:19 PM madshowie @.***> wrote:

Hi Losia! No I don't think I can because it's straight from IUCN. However,
that shapefile is for species distribution mapping which perhaps it might
be easier to source externally. My main plan was to compare country
productivity with species distribution so if I can't do the map myself I
can always download the map from IUCN and compare with country
productivity. Do you think this would be ok?
—
Reply to this email directly, view it on GitHub
https://github.com/itchyshin/bibliometrics_2022/issues/16#issuecomment-1185400963,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADDIDL5LV27PPKJP4RT67SLVUE3LBANCNFSM53UY6QAA
.
You are receiving this because you commented.Message ID:
@.***>


Remove @Beta annotation from ConfigurationAwareModule

Remove @Beta annotation from ConfigurationAwareModule

I am not sure the purpose of @Beta here. This interface is widely used and is does not seem like something that would be dropped.

@Beta causes a warning in Intellij.

@Beta causes a warning in Intellij.

is the warning emitted by default?
at least in Presto, we are accepting code using eg guava @Beta, so if this is a default inspection, we should recommend disabling it in https://github.com/prestosql/presto/blob/master/DEVELOPMENT.md#additional-ide-configuration
(i am not saying this change should or should not be merged)


Conda deprecation and General Terms of Use to Usage policy page

https://csc-guide-preview.rahtiapp.fi/origin/usage-policy-conda/computing/usage-policy/

Clarified a bit and took the liberty to fix some typos on associated pages. Split also quite a few lines for improved readability.


RHODS: rhods_wait_ods: default: also wait for tensorflow and pytorch images

Without this, the automation may try to spawn notebooks before these
images are built, and RHODS dashboard refuses to go to JupyterLab when
these images are missing.

I'll push the update when the test completes

Test passed, good for merging @fcami
/approve
we'll adjust the timeouts if we see some failures


Jailbreak IPhone 6 12.1.3 Doesn’t work

Hello ! I don’t understand, I can’t Jailbreak my iPhone 6 on 12.1.3(16D39) with Unc0ver or Chimera, can you help me  ?

Same here.

It's cause A7~A8 and A12X devices are not supported with Unc0ver yet... They are supported only till iOS 12.1.2.. Chimera 1.2.4 should be supported, but sock_pupped exploit (used for iOS 12.1.3-12.2 jb) sucks on 4K devices yet...

So I have to wait for them to fix it ?

https://github.com/pwn20wndstuff/Undecimus/issues/1180#issuecomment-512263553


Diagonal operator vs batched diagonal operator

Is there a reason why both of these exist in the code base? It doesn't seem easy to construct a diagonal operator as the default is a batched diagonal operator.
Also, is the below line dangerous? It looks like it would drop the update_func.
https://github.com/SciML/SciMLOperators.jl/blob/31275d4e2b2f3671fb4c2811656dd555c4e1f24a/src/matrix.jl#L116

that should create a Diagonal matrix. Can you create a PR to fix that?

Yes there are two different use cases.
(1) DiagonalOperator can be used for the very common case where L.A is a diagonal matrix.
N, K = 3, 4
d = Diagonal(N)
u = rand(N, K)
A = DiagonalOperator(d) # MatrixOperator
v = A * u

will pointwise multiply each column on u with d.
(2) Consider the following case: you are evolving an ODE with size(u) = (N,K), ie you have K trajectories of size N. A very common operation you want to perform is pointwise multiplication, i.e. for every trajectory k, you want to do d[:,k] .* u[:,k]. This is common when you want to form a nonlinear term like u^2 in a Navier-Stokes or a Burgers solver. That's what BatchedDiagonalOperator allows you to do:
N, K = 3, 4
d = rand(N, K)
u = rand(N, K)
A = DiagonalOperator(d) # BatchedDiagonalOperator
v = A * u

If you look at the methods, DiagonalOperator(d::AbstractArray) is a convenience function that dispatches to MatrixOperator when d isa AbstractVector, and BatchedDiagonalOperator otherwise. We have tests to make sure the correct type is instantiated (MatrixOperator, or BatchedDiagonalOperator). Feel free to add more tests if you think there are corner cases. There is no type called DiagonalOperator - it is just a convenience.

I'm sure there are other operations that we'd want to implement in a Batched way, ie when you want to apply unique data to each trajectory/batch. I just haven't come across yet.

If Diagonal should create a regular diagonal matrix, not a SciMLOperator, then is the following line from tests also wrong?
https://github.com/SciML/SciMLOperators.jl/blob/31275d4e2b2f3671fb4c2811656dd555c4e1f24a/test/total.jl#L92
It seems you were trying to create a lazy diagonal operator here?

D1, D2 are SciMLOperators, [D1, D2] isa Vector{<AbstractSciMLOperator}, and Diagonal([D1, D2]) isa Diagonal matrix whose entries are SciMLOperators

I'm just doing matrix algebra to form my operators.
    TT = [T1, T2]
    DD = Diagonal([D1, D2])

    op = TT' * DD * TT

Hm, I see. Note that DD behaves a bit unintuitively, e.g. size(DD) = (2,2)

I'll need to modify the test to make all of these SciMLOperators because otherwise update_coefficients! isn't going to work. Although, it doesn't seem possible to make a lazy diagonal SciMLOperator currently?

No. All you need to do is pass an update_func to T1, T2, D1, D2. Leave the operator forming, matrix algebra as it is. When you call update_coeffs!(op,...; kw...) or op(du, u, p, t; kw...), everything should just work

Right, my bad, on the same page now

excellent. :D can you close this if all is good?


Add local_port and remote_port to VPN: IPsec: Connections [new]

#6818
I created new dropdown menus in the IPsec Connections Form.
Users can select between port 500 (default) and port 4500 (NAT-T). I didn't create a free input field since other values have to be actually supported by the backend, and I didn't see the usecase for free choice.
I didn't upgrade the model version since I used the elseif ((string)$attr == '') in the Swanctl.php in order to skip the default value. I chose the default value as empty, because I didn't want to add local_port = 500 and remote_port = 500 explicitly to the configuration file, it's not needed since its the default even if its not specified. Thus I could maintain a non required field that won't need a migration since no config changes will be made when its implemented.
When a user enables advanced mode and chooses "4500 (NAT-T), the port gets added in the swanctl.conf.

@Monviech the Swanctl.php file seems to be missing in your PR

@AdSchellevis
I didnt need to change it. The php file already polled for local and skipped empty strings so it all worked without a change to the code.

@Monviech my mistake :) needed coffee probably ;)

@Monviech looks good, thanks!

Hello,
please make an empty input field / possibility to change port for IPSec.
Because there are facis firewalls that block the default IPsec ports.
Custom ports are supported: https://docs.strongswan.org/docs/5.9/features/natTraversal.html#_custom_server_ports
Need a feature request for IPSec Custom Port?
grEEtZ

Wouldn't it make more sense to use OpenVPN on TCP 443 or something?

Maybe an example: The Android app stops receiving packets from the gateway

Wouldn't it make more sense to use OpenVPN on TCP 443 or something?

II don't think so, because nothing comes close to IPSec (strongSwan). Integrated almost everywhere in the OS, stable, very good AES performance.
End-to-end military grade encryption with e.g. Curve448 or Curve25519

Please create a new feature request.
I just think that changing ports here will not give the desired benefit since UDP encapsulated ESP packets can be easily detected in their pattern by a traffic matcher and blocked.
Changing ports these days is no guarantee. Thats why OpenVPN on TCP/443 with additional protocol scrambling is the best choice to get through any firewall. And if security is an issue, IPsec can be established through the OpenVPN tunnel additionally.