About code of real-world data experiments

Hi Siyuan,
Thanks for your wonderful work. I'm quite interested in it and would like to conduct some experiments of your proposed method in industrial scenarios. I wonder whether you could share a copy of code for experiments on real-world datasets. My e-mail address is<EMAIL_ADDRESS>Thank you so much. Look forward to your reply.
Best

Hi, I have uploaded the codes for real-world experiments. Too busy recently.

Hi, I have uploaded the codes for real-world experiments. Too busy recently.

Thank you so much for your reply. I have tried to reproduce the results in your paper by strictly following the settings, but met some difficulties and would like to ack you for help.
First, the results of some baselines, such as Naive and IPS, show much better results than what are reported in the paper, especially DCG@6 and Recall@6 in yahoo (Naive and IPS achieve the best results).
Seond, the proposed double learning approach seems do not work well, and better results are obtained by not utilizing this optimization method.
Look forward to your reply. Thank you so much.

Hi, from my experience, Naive and DR-JL perform poorly and IPS is the relatively comparable baseline on two datasets. On Coat, MRDR-DL significantly outperforms IPS, and on Yahoo, MRDR-DL less significantly outperforms IPS.
As almost all of my codes and data were lost due to the server failure, the code on Github is a previous copy on my laptop. I think that maybe there are some differences between them. I’ll try to fix this problem after I return to the school.
I’m sorry that I didn’t check the codes fully before uploading. And thanks for your attention on our work.

Hi, from my experience, Naive and DR-JL perform poorly and IPS is the relatively comparable baseline on two datasets. On Coat, MRDR-DL significantly outperforms IPS, and on Yahoo, MRDR-DL less significantly outperforms IPS.
As almost all of my codes and data were lost due to the server failure, the code on Github is a previous copy on my laptop. I think that maybe there are some differences between them. I’ll try to fix this problem after I return to the school.
I’m sorry that I didn’t check the codes fully before uploading. And thanks for your attention on our work.

Thanks for your reply. I also have some questions about the expeiments on semi-synthetic data.
Specially, I don't quite understand how the imputed error of different approaches are derived, especially EIB, DR and MRDR. Could you help explain this in more details? It is strange for me that MRDR could be evaluated without training, since the error of MRDR is the same as DR. Particularly, could you provide the derivation of the estimation function above equation 17. Thank you so much.

Hi. For EIB, DR and MRDR, we follow the similar steps, i.e., we first estimate the global average CVR; then, we estimate the imputed errors with the cross entropy between the global average CVR and the predicted CVR. For EIB and DR, we compute the global average CVR with \hat{p} to consider the MNAR effect. For MRDR, we compute it with the proposed enhanced weight to approximate the MRDR estimator.
Actually, we argue that error imputation is a complicated process; hence, we didn’t introduce an extra imputation model in the semi-synthetic dataset. In contrast, we directly use the different estimated error to demonstrate the effectiveness of the MRDR estimator in a naive way.
In equation 18, L_ideal(\hat{R}) can be computed with equation 1 (in the semi-synthetic dataset, we already have the ground-truth CVR), and L_estimator(\hat{R}) can be computed with equation 2~5 (we have detailed how to estimate the propensity and the imputed error in Section 4.1.2).


[Question]: What is frequency of robot_get_joint_states receiving the latest joint state?

Question
I am trying to realize high frequency reaction control. So I wonder what is frequency of robot_get_joint_states to receive the latest joint state?
Robot Model
wx250s
Operating System
Ubuntu 20.04
ROS Version
ROS 1 Noetic
Additional Info
No response

robot_get_joint_states will return the last received joint states. You can control the joint state update rate of the xs_sdk with the update_rate parameter of the xs_sdk's joint_state_publisher in whatever motor config file you're using.

I actually want to know the joint state update transport rate about the servo motor. For example, thought the update rate of ros message is 100hz, the reaction rate of motor joint state update only 50 hz. It will get same joint state msg in the ros msg.
So how fast can the widows250 motor joint state update [limited by hardware] ?


[new config model] Setting up a 2 node cluster with toml missing j2 files

Suggested Labels
Affected/IS5.9.0 alpha
Priority/High
Severity/Major
Type/Improvement
When setting up a 2 node cluster with deployment pattern 1 as in doc [1] using new configuration model (toml based) theres no j2 files for following configurations.
Under setting up dashboard section for the configurations mentioned there j2 files cannot be found.
IS_HOME>/repository/conf/identity/sso-idp-config.xml
IS_HOME>/repository/deployment/server/jaggeryapps/dashboard/conf/site.json
WSO2IS_HOME>/repository/deployment/server/webapps/shindig/WEB-INF/web.xml
[1]. https://docs.wso2.com/display/IS580/Setting+Up+Deployment+Pattern+1

We are not using this configurations in 5.9.0 IS pack

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


[Dynamic Task Templates]:Change sql_statement param to sql

Describe the bug
get_value_list has a kwarg called sql_statement which is too verbose, change it to sql
Version

Astro: [e.g. 0.6.0]
OS: [eg. Debian]

To Reproduce
Steps to reproduce the behavior:

Write the DAG  '...'
Create connection '....'
Run using  '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

Duplicate of https://github.com/astronomer/astro-sdk/issues/865 . Hence closing


Laravel Analyzer

This pull request includes changes and recommendations for crafting your application "The Laravel Way". Feel free to commit any additional changes to the shift-8522 branch.
Before merging, you should:

Checkout the shift-8522 branch
Review all pull request comments for additional changes
Thoroughly test your application

If you want help with your upgrade, check out the Shift Human Services or join the Shifty Coders Slack room to get answers to all your Laravel questions.

ℹ Your Laravel application contains 35 classes and 273 lines of code, with an average of 4.72 lines of code per method.
Most of your classes are Middlewares. Your Models contain the most lines of code (11 loc) and your __ have the highest lines of code per method ( loc).

⚠ Shift detected direct Mail::send() calls. Laravel 5.4 introduced Mailables which better encapsulate the code for sending emails.

[ ] app/Http/Controllers/Api/ContactController.php

⚠ The following controllers contain actions outside of the 7 resource actions (index, create, store, show, edit, update, destroy). For more details, review the docs or watch Cruddy by Design to see if you can rework these into resource controllers.

[ ] app/Http/Controllers/Api/ContactController.php
[ ] app/Http/Controllers/Api/LoginController.php
[ ] app/Http/Controllers/Api/RegisterController.php
[ ] app/Http/Controllers/Api/UserController.php

⚠ Shift found the following config files differ from the defaults. While you are welcome to customize your configuration, you should leverage ENV variables rather than hardcode values. If you find you're adding a lot of configuration options, consider creating a domain specific config file, such as core.php. Both will make app upgrades and deployments easier.

[ ] config/app.php
[ ] config/auth.php
[ ] config/filesystems.php
[ ] config/hashing.php
[ ] config/logging.php
[ ] config/queue.php
[ ] config/services.php

⚠ Shift found inline validation in the following controllers. Unless you require this level of control, you should use a Form Request to encapsulate this validation logic and keep your controller clean.

[ ] app/Http/Controllers/Api/ContactController.php
[ ] app/Http/Controllers/Api/LoginController.php
[ ] app/Http/Controllers/Api/RegisterController.php

ℹ If you have manually upgraded your app from previous versions of Laravel, you should run the Laravel Linter to ensure your app is fully upgraded to the latest version of Laravel.


BrowseFacet for ranges

@NightOwl888
Hi, I do not understand why BrowseFacet has not a specialized version for ranges that replaces
public virtual string Value { get; set; }

with
public virtual string UpperBoundValue{ get; set; }
public virtual string LowerBoundValue{ get; set; }

It would be much easier to manage the formatting problem with this solution. Moreover, it seems to me that to compose the range string values (e.g. [a TO b]) is quite time consuming.
Do you agree?

Actually, there is a RangeFacet that works much like you described, but it is a private type. There is really no way to expose this functionality to the outside without an explicit cast. But also, it is an input. The facets that are generated at the output are done so at a level deep in the API (in the RangeFacetCountCollector and then again in the CombinedFacetCountCollector), not in the FacetHandler, and at that point there is no straightforward way to specify the type of BrowseFacet to generate (it applies to all of the facet handlers). The only thing tying the input to the output is the range string.
I also considered making a tool to manage the strings in a similar way, but it seemed like it would be even more difficult to understand and manage, especially when you consider that you would need to regenerate the strings on each round trip the same way that they are done at application startup. So it would need some way to match the output of the BrowseFacet up with a type that is pre-configured to be the same as the FacetHandler. Technically, the BoboIndexReader is supposed to be configured as a singleton (once during application startup), so that would make it necessary to somehow regenerate the same configuration at startup and on each round trip (or also make this configuration into a singleton).
When you consider how the strings are used, creating a unified tool like that is not straightforward.

The RangeFacetHandler inputs are (normally) generated at application startup. This can be done through Spring.NET XML configuration.
The BrowseFacet instances (for output) are generated deep within the API (in the RangeFacetCountCollector and CombinedFacetCountCollector) on each round trip. Only the facet values that match the current query are output unless FacetSpec.ExpandSelection is specified.
The BrowseSelections correspond to what the user selected on the API, and must exactly match the format specified in the RangeFacetHandler. This is problematic when you consider that the string that is displayed to the user may be formatted in a specific culture, so matching it upon return trip from the UI is a challenge.
RangeFacetHandler could potentially also be used as a runtime facet handler by calling BoboBrowser.SetFacetHandler to make dynamic ranges.

So, the primary factor to consider is where in the application lifecycle the configuration of the range facet generator would be stored so it could be used to generate the input strings, generate the output format strings for the UI, and then be matched on round trip from the UI again so it could produce the original strings for the BrowseSelections. In the end, it seemed more straightforward to generate a throw away user-friendly culture-specific range string at the end of each browse request than to try to work out a storage and matching mechanism for the range generator objects that could be used at each of the places in the application lifecycle. And leave it up to the UI to store the actual range string for the round trip.
That said, I agree with you that it should be simpler. There are a lot of design decisions that were made on this project that I personally don't agree with, especially when it comes to data types and dependency injection. But, it is much less time consuming to make a port of an existing design than to come up with a design of my own. Staying true to the original design means that the existing intentions of the application's usage are preserved and I believe it is the best course of action to make this project maintainable over time, even if I don't personally agree with all of the choices.
That said, this design is somewhat extensible. If you would like to contribute a new RangeFacetHandler that works more to your liking, I would be happy to wrap it up into a new NuGet package (something like BoboBrowse.Net.Extensions) so it can be downloaded as an add-on to this package. Perhaps there is a way to internalize all of the data necessary to make it work (including type info) into the range strings, and then make an extension method for BrowseFacet that can convert those strings into the formatted strings, or better yet, convert it into a RangeBrowseFacet instance. This doesn't get around the fact that the string will need to be the same on the round trip from the UI, but if the type info is embedded in the string it won't have to be supplied explicitly, nor will you need to instantiate an external tool to get the formatted string for the UI.

I am closing this as a wontfix for now. Feel free to reopen this issue if you decide you want to contribute a custom RangeFacetHandler that meets these goals.


[Feature] Support Tencent cos filesystem

CREATE EXTERNAL CATALOG `ice_hadoop_cos`
PROPERTIES (
	"type"  =  "iceberg",
	"iceberg.catalog.type"  =  "hadoop",
	"iceberg.catalog.warehouse" = "cosn://xxx/iceberg_catalog/",
	"tencent.cos.access_key"  =  "access_key",
	"tencent.cos.secret_key"  =  "secret_key",
	"tencent.cos.endpoint"  =  "cos.ap-shanghai.myqcloud.com"
)

What type of PR is this:

[ ] BugFix
[x] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[x] Yes, this PR will result in a change in behavior.
[ ] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[x] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[ ] 3.2
[ ] 3.1
[ ] 3.0
[ ] 2.5

[BE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

[FE Incremental Coverage Report]
:heart_eyes: pass : 1 / 1 (100.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/credential/CloudType.java
1
1
100.00%
[]

[FE Incremental Coverage Report]
:disappointed: fail : 18 / 49 (36.73%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/credential/tencent/TencentCloudConfiguration.java
0
17
00.00%
[31, 32, 33, 34, 39, 40, 41, 42, 43, 44, 48, 49, 50, 54, 59, 64, 65]

:large_blue_circle:
com/starrocks/credential/tencent/TencentCloudCredential.java
9
22
40.91%
[42, 43, 44, 45, 46, 47, 48, 58, 59, 60, 61, 65, 74]

:large_blue_circle:
com/starrocks/credential/tencent/TencentCloudConfigurationProvider.java
8
9
88.89%
[40]

:large_blue_circle:
com/starrocks/credential/CloudType.java
1
1
100.00%
[]

[BE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

[FE Incremental Coverage Report]
:disappointed: fail : 18 / 49 (36.73%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/credential/tencent/TencentCloudConfiguration.java
0
17
00.00%
[31, 32, 33, 34, 39, 40, 41, 42, 43, 44, 48, 49, 50, 54, 59, 64, 65]

:large_blue_circle:
com/starrocks/credential/tencent/TencentCloudCredential.java
9
22
40.91%
[42, 43, 44, 45, 46, 47, 48, 58, 59, 60, 61, 65, 74]

:large_blue_circle:
com/starrocks/credential/tencent/TencentCloudConfigurationProvider.java
8
9
88.89%
[40]

:large_blue_circle:
com/starrocks/credential/CloudType.java
1
1
100.00%
[]

[BE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

PTAL @kevincai @Smith-Cruise

[FE Incremental Coverage Report]
:heart_eyes: pass : 49 / 49 (100.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/credential/CloudType.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/credential/tencent/TencentCloudConfigurationProvider.java
9
9
100.00%
[]

:large_blue_circle:
com/starrocks/credential/tencent/TencentCloudCredential.java
22
22
100.00%
[]

:large_blue_circle:
com/starrocks/credential/tencent/TencentCloudConfiguration.java
17
17
100.00%
[]

[BE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

@mergify backport branch-3.2


Weatherflow integration not showing after HACS download

Downloaded the integration via HACS
It shows up correctly under HACS integrations:

Restarted HA

Settings > Devices and Services > Add Integration  > Weatherflow

No integration found.

Checked custom_components the directory exists as expected.

Restarted again and again, tried redownload via HACS.
HA core version: 2022.7.5
Note: I set this up without issue on someone else's HA a month ago.  I suspect a 2022.7.x issue
Note2: the README.MD Needs a slight update under "Configuration" . "Go to Configuration and Integrations" should be "Go to Settings > Devices and Services" due to HA Menu changes.

Try and go to Settings > Devices and Services and do a Hard Refresh of your browser. Then try and add it again. It's most likely a caching issue.
Thanks for the reminder on the READMME


Autoderef with visibility

Fixes https://github.com/rust-analyzer/rust-analyzer/issues/7841.
I am not sure about the general approach here. Right now this simply tries to check whether the autoderef candidate is reachable from the current module. However this doesn't exactly work with traits (see the tests::macros::infer_derive_clone_in_core test, which fails right now). see comment below
Refs:

rustc_typeck checking fields: https://github.com/rust-lang/rust/blob/66ec64ccf31883cd2c28d045912a76179c0c6ed2/compiler/rustc_typeck/src/check/expr.rs#L1610

r? @flodiebold

bors r+

changelog fix take visibility into account for autoderef


Copyright year

I believe there should be an automatic means to update the copyright year in the ForSyDe pages footer with every push. Right now it is hard-coded in every page, e.g. "©2018-2019". Should be something along the lines of ©2018-{{ thisyear  }}.

Maybe a "last updated" entry in the footer would help with citing material in the web pages...

Fixed with commit that goes in version 0.1.3.


Added tracking Gym Leaders beaten

Changes

Added the Gym Leaders as trackable items
Added the Gym Leader items as automatically being marked when the item is collected
Added a function to logic.lua to track if any Gym leader has been beaten

As a result:

Fixed a small issue with offline tracking (and maybe autotracking) if Rt 3 access is set to "beat any gym" or "beat brock"

I added full pngs of the gym leaders without realising you had the closeups. If you want me to use those images then I can change it if needed.

Thanks for the PR though, I'd been meaning to fix this but got real busy! (And tbh not too many people seem to use offline tracking so it was kinda low on the list.)

Sorry for not adding logic.lua, your suggestion is honestly a better solution and one I didn't think of until after PRing
I've made the changes, it should be ready to merge.
On my end it says I haven't made the required changes, but I have updated items.json, itemgrid.json, and locations.json
As for manual tracking, while I personally use autotracking as much as I can, adding the leaders would have fixed the issue and therefore also fixes things for the random manual tracking person.


Public bot appears down

I'm getting responses from karmabot using help or stats, but no response to ++, --, or query functions.

Should be fixed. Sorry about that. Let me know if you notice the issue again.


Support Client Directives in Prisma plugin (@skip and @include)

Related to https://github.com/hayes/pothos/issues/723

It would be great to add a test for these directives.  The prisma tests all verify the queries that are executed, so adding a test that skips some relation field should be pretty straight forward


XHR progress event not firing on iOS while downloading Blob/Arraybuffer response type

Description
Hello,
as title says I've noticed a strange behaviour while trying to subscribe to the "progress" event that should be fired by an XHR call.
NB: Links used below are just for testing purpose, I need to download a big file (about 100MB) and read how many bytes I've already downloaded from "progress" event. Sefram had some heavy file hosted online where I could test
export default function dlTest(dlCalled, nextTest, settings) {

  if (dlCalled) return;

  dlCalled = true;

  let xhr = new XMLHttpRequest();

  xhr.addEventListener("progress", (e) => console.log("progress", e.loaded));
  xhr.addEventListener("load", (e) => console.log("loaded", e.loaded));
  xhr.addEventListener("error", () => console.log("error"));

  let zipAddress = "https://www.sefram.com/images/products/photos/hi_res/BK1760A.zip";
  let jpgAddress = "https://www.sefram.com/images/products/photos/hi_res/207030301.jpg";
  let documentAddress = "https://www.sefram.com/en/download/high-resolution-images.html";

  xhr.open("GET", zipAddress, true);
  xhr.responseType = "blob";

  xhr.send();
}

Basically I need to read how much bytes I've downloaded since the call started.
To sum up my tests:

iOS:

Call documentAddress && Response-Type: document:
It does fire progress event but doesn't update event.loaded (it remains 0).
You can try this yourself with documentAddress.

Call documentAddress && Response-Type: blob/arraybuffer:
It doesn't fire progress event at all

Call zipAddress && Response-Type: document
It doesn't fire progress event at all

Call zipAddress && Response-Type: blob/arraybuffer
It doesn't fire progress event at all

I can't really understand why this is happening.
I've done many tests on Android, progress mostly work fine, but this isn't the issue to discuss Android's problems.
I've opened this issue because I couldn't find any other discussion about this kind of problems.
Thanks in advance for your work.
Version
0.64.3
Output of npx react-native info
System:
- OS: Windows 10 10.0.22000
- CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
- Memory: 4.21 GB / 15.85 GB

Binaries:
- Node: 14.17.6 - C:\Program Files\nodejs\node.EXE
- Yarn: Not Found
- npm: 6.14.15 - C:\Program Files\nodejs\npm.CMD
- Watchman: Not Found

SDKs:
- Android SDK: Not Found
- Windows SDK:
- AllowAllTrustedApps: Disabled

IDEs:
- Android Studio: Version     2<IP_ADDRESS> AI-211.7628.21.2111.8139111
- Visual Studio: 17.1.32210.238 (Visual Studio Community 2022)

Languages:
- Java: 11.0.12 - C:\Program Files\Microsoft\jdk-<IP_ADDRESS>-hotspot\bin\javac.EXE

npmPackages:
- @react-native-community/cli: Not Found
- react: 17.0.1 => 17.0.1
- react-native: 0.64.3 => 0.64.3
- react-native-windows: Not Found

npmGlobalPackages:
- *react-native*: Not Found

Steps to reproduce

Setup react-native project
Copy dlTest
Call it from App.js
enjoy

Snack, code example, screenshot, or link to a repository
Snack Example Download Test

Same

Same here!


Vulnerability review (and fix dependencies)

We need help with prioritization

Duplicates Security Cleanings for 1.5.4 #875


storage: Use write cf stats to decide load action for default cf instead of near seek

What is changed and how it works?

Issue Number: Ref #16245

What's Changed:
Use write cf stats to decide load action for default cf instead of near seek

It gains some performance improvement when value is not inlined.

As you can see, default next is smaller than write next with this PR because some unnecessary near_seek is avoided.

Related changes

[ ] PR to update pingcap/docs/pingcap/docs-cn:
[ ] Need to cherry-pick to the release branch

Check List
Tests

[ ] Unit test
[ ] Integration test
[x] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Release note

Use write cf stats to decide load action for default cf instead of near seek

@you06 PTAL

/merge


Registry server setup needs to run on all worker nodes and deployer.

As inventory file has changed and looks like we missed this to change.

the airship-workers have been renamed to "caasp-workers" in the inventory file

Yeah looks like I had old inventory file and made changes accordingly. fixed it.


Mac agent appears to be caching build.id

Have you tried trouble shooting?
Trouble shooting doc
Agent Version and Platform
v2.140.0
OS of the machine running the agent? OSX/Windows/Linux/...
OSX
VSTS Type and Version
VisualStudio.com
If VisualStudio.com, what is your account name? http://account.visualstudio.com
Avecto
What's not working?
We currently have an issue on our Mac VSTS agent where the Build id is being retained from a previous build. On the vsts side the build id is incrementing however this is not the case on the agent.  If we restart the agent the id resets.
We use build.number to set overall build number (i.e. <IP_ADDRESS>), however we have some components in our application which follow a different build number structure (i.e <IP_ADDRESS>), we are using the build.id to set the incremental build number. Our build number variable is build from a fixed version 2.0.(build.id), within vsts this is incrementing, however when we come to use the build.id on the agent for stamping some components this uses the first build.id on the agent

@avectoauto can you share some log? like run set to print out the build_id variables.

To update, this only appears to happen when building from XCode

@avectoauto is your XCode project change any environment variable?
can you print out build.id before your xcode task?

Output from TFS variable:
2018-10-08T15:09:27.2850790Z BUILD_BUILDID=28857
2018-10-08T15:09:27.2888250Z BUILD_BUILDNUMBER=5.2.28857.0
Output from Xcode build:
export AVECTO_BUILD_BUILDNUMBER=5.2.28856.0

@avectoauto what set AVECTO_BUILD_BUILDNUMBER?
the variable we set is BUILD_BUILDNUMBER

AVECTO_BUILD_BUILDNUMBER is set by BUILD_BUILD.ID with a static value appended in front
As mentioned in the description "Our build number variable is build from a fixed version 2.0.(build.id)"

is AVECTO_BUILD_BUILDNUMBER a variable in your build definition? or it's only set when during your xcode task?
i am trying to figure whether AVECTO_BUILD_BUILDNUMBER is a variable that VSTS set its value.

It is in the build def and passed into the xcode build step, the issue seems similar to this
https://developercommunity.visualstudio.com/content/problem/303198/the-buildid-variable-does-not-get-replaced-anymore.html


Add attachments to page publish call

Attachments now have an API (see also #3407), and the publish call in the Lepiter client should use it.

A part of the following commit belongs to this issue: https://github.com/feenkcom/lepiter-client/commit/a6bd8ca7b817b471e9be350bbd6d47dec045263d


No support of http_proxy https_proxy

The framework uses websockets module (here).
The websockets module does not support HTTP proxy (see https://github.com/python-websockets/websockets/issues/364, issue from 2018), while websocket-client module (here) supports it.
Are they plans to migrate websockets implementations though websocket-client instead?

Hi @antoinetran - no such plans at the moment.
We would be willing to review a PR, if you'd be interested in  contributing such a feature.
I would also recommend to look at adding an option of using these other websocket implementation and toggling it with a flag, rather than outright migrating

Ok, I tested a bit and it seems to work! Will do a PR, with a class parameter to choose which Websocket client implementation. By default, it should be the websockets, because it is natively coded as coroutine, but I will ship another class that implements SimpleWebSocket using websocket-client.
I am not sure if my implementation is optimal.
I will also add websocket-client as an dependency of this framework and modify the README a bit to show how to change the implementation to use proxy.

@antoinetran Hey, I've created a patch for websockets to use with proxy servers: https://github.com/racinette/websockets_proxy

@antoinetran Hey, I've created a patch for websockets to use with proxy servers: https://github.com/racinette/websockets_proxy

Nice! You are free to provide your own websocket client handler to fastapi websocket rpc, in for example your module. But for now, I would rather stick with the official websocket client, and in the future, only the websockets, when their team will fix support of http proxy.

Should it be closed now by #32?


pandas.stats.ols.MovingOLS._calc_betas incompatibility issue with newer numpy

numpy version 1.8.2
pandas version 0.14.1
In pandas/stats/ols.py:821, we have:
mask = -np.isnan(betas).any(axis=1)
This causes problem with newer numpy which deprecated negative unary operator (-) on boolean arrays. I got this following error when running into this line:
DeprecationWarning: numpy boolean negative (the unary - operator) is deprecated, use the bitwise_xor (the ^ operator) or the logical_xor function instead.
When I change "-" to "~", the problem goes away. It's minor, but hope we can incorporate this into next pandas update.  Thanks.

Just saw this in 0.16.2 .   I'll do a pull request for this shortly.

I believe this issue can be closed from this PR.
https://github.com/pandas-dev/pandas/commit/2113672040a884e8a3b18881e9608fed795e1129#diff-1e25b8d60d53005c3c7cb6612f2b41a6

yep


Allow explorer root to toggle

This lets the user expand/collapse the root folder in the explorer by clicking on it, the same way that non-root folders can be expanded/collapsed. Sure, this is not immensely useful, but it's less alarming than a ‘Not implemented yet’ alert.

@rhendric thanks for the PR definitely and improvement over the alert 😆 and for adding tests 😍

👍 , thanks a lot @rhendric ! That alert was really embarassing - glad it's working properly now. Code change looks great to me. Build issue on appveyor doesn't look related, I just kicked it off again to check though.


Seed file in nodejs mode failed.

How to seed file in node mode?
I created seed.js like this :
function runSeed (input) {
client = new WebTorrent()
function onTorrent (torrent) {
console.log('Client is seeding:', torrent.infoHash)
console.log(torrent.magnetURI)
}
client.seed(input , onTorrent)
}
runSeed('mp3.mp3')
open terminal and it output:
Client is seeding: 38178900b09f42319e8be46c22566222fdcede55
magnet:?xt=urn:btih:38178900b09f42319e8be46c22566222fdcede55&dn=mp3.mp3&tr=udp%3
A%2F%2Fopen.demonii.com%3A1337&tr=udp%3A%2F%2Ftracker.openbittorrent.com%3A80&tr
=udp%3A%2F%2Ftracker.publicbt.com%3A80&tr=udp%3A%2F%2Ftracker.webtorrent.io%3A80
&tr=wss%3A%2F%2Ftracker.webtorrent.io
after that I open an other terminal and input : webtorrent download magnet:?xt=urn:btih:38178900b09f42319e8be46c22566222fdcede55
and it only print one line:
fetching torrent metadata from 0 peers
I have not specified any trackers ,and  webtorrent  will  always use the same trackers ,
But the code doesn't work , Am i doing wrong ?

Could you please enable debug and comment the logs? #518 looks the same

My comment is too long ,so I attached the files:
log.zip

Are you currently seeding?
Hop on gitter and PM me, I'll help you :)

I'm not online , sleeping at  night at that time.
I have opened a vps and now running  webtorrent ,and it still seeding file.
magnet:?xt=urn:btih:38178900b09f42319e8be46c22566222fdcede55
I will email vps account and password to you .
the problem is:
Every things is ok , when I running the seeder and downloader at the same computer,
But can not download from  computer which located behind  NAT
My purpose is seeding files on server , and users can download  all of world  using browser ,

Checking this now, will report in a few minutes.

I can connect to internet and still failed , I don't know why ,
did it  needs NAT traversal service ?

Something must be blocking it, NAT or a firewall.


specify target for artwork button actions

@ashfurrow suggested that i set up delegates for the various buttons contained in the artwork view and ensure that there is a target specified for the button actions. I guess this could be structured a few different ways, but this made sense to me.

Ah, such a shame, but simple + works > elegant

Tests are passing locally.
@orta ready for review.

Alright, can we also get a CHANGELOG entry too please

:green_apple:

Good show you two :+1:


Marked Crossing Ways now default to crossing=uncontrolled instead of crossing=marked

Describe the bug
Creating Marked Crossing ways now defaults to crossing=uncontrolled instead of crossing=marked.
Expected behavior
Similar to Unmarked Crossings, I would expect when creating, that tag be crossing=marked, but instead is now crossing=uncontrolled.

See #568 and #408 for previous discussion.


Bertha Wifi network bug fixes.

Hi @namkedia, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Namrata Kedia). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;


feat: implement ctx.custom_id and ctx.label properties

About
This pull request is about adding custom_id and label properties to ComponentContext, which makes it easier to retrieve these commonly used values.
Checklist

[x] I've ran pre-commit to format and lint the change(s) made.
[x] I've checked to make sure the change(s) work on 3.8.6 and higher.
[x] This fixes/solves an Issue.

#480

[ ] I've made this pull request for/as: (check all that apply)

[ ] Documentation
[ ] Breaking change
[x] New feature/enhancement
[ ] Bugfix

Let me know if there are any other attributes anyone would want. I didn't include ctx.component since the idea is to have access to the info without needing it, or ctx.selected_options since that is passed to the callback already

I changed them to properties, but I don't know how to mark your change request as completed (if I even can)

I changed them to properties, but I don't know how to mark your change request as completed (if I even can)

Press the rerequest button or something like that

I changed them to properties, but I don't know how to mark your change request as completed (if I even can)

Press the rerequest button or something like that

Found it, thanks


Does not have mandatory providers: 'PyZProvider'

Hi just trying out these rules and running into an issue with local wheels. I am getting
(14:30:51) ERROR: /somepath/third_party/pypi/py3/BUILD.bazel:2:1: in deps attribute of pyz_library rule //third_party/pypi/py3:ipaddress: '//third_party/pypi/py3:py3_ipaddress' does not have mandatory providers: 'PyZProvider'. Since this rule was created by the macro 'pypi_libraries', the error might have been caused by the macro implementation in /somepath/third_party/pypi/py3/rules.bzl:241:14
(14:30:52) ERROR: Analysis of target '@workspace//:manage' failed; build aborted: Analysis of target '//third_party/pypi/py3:ipaddress' failed; build aborted

This error makes sense as the local wheels are a filegroup but are added as a dependency to pyz_library - but not sure what the intendend behaviour is. I suppose there should be something extracting the wheels and adding a pyz_library for it no?

Which bazel version are you using? This can happen when the WORKSPACE was not imported with the correct global name.

I am using 0.17 rc1. I'll take a look again in a few days.

Thanks! Figuring out how to reproduce the issue is probably the biggest issue. Make sure you are importing it as @com_bluecore_rules_pyz.

I've run into this also, and I'm sure I'm importing as @com_bluecore_rules_pyz.
I'm using bazel 0.16.1 on CentOS 7. I've tried going back to the initial change that added local wheel support and still run into the same issue, so it's not particularly clear how this is supposed to work. Is there a minimal example of a repo using local wheels available somewhere?

@snifi did you test that fix with any actual pyz_* targets that depend on the markupsafe library? I did what you suggested and got myself to the point where my library builds, but then I get import errors when I actually run my test. Taking a look at the test_exe_exedir, all third-party libraries that are local wheels are missing.
Does anyone have a working example with local wheels?


add etcd dependency

Codecov Report

Merging #8 (7b4e152) into master (89e35d9) will decrease coverage by 1.27%.
The diff coverage is 67.91%.

@@             Coverage Diff              @@
##             master       #8      +/-   ##
============================================
- Coverage     70.58%   69.30%   -1.28%
- Complexity       84      111      +27
============================================
  Files            14       15       +1
  Lines           204      391     +187
  Branches         12       32      +20
============================================
+ Hits            144      271     +127
- Misses           51      101      +50
- Partials          9       19      +10

Impacted Files
Coverage Δ
Complexity Δ

.../baidu/hugegraph/computer/core/bsp/EtcdClient.java
67.91% <67.91%> (ø)
27.00 <27.00> (?)

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 89e35d9...7b4e152. Read the comment docs.

Codecov Report

Merging #8 (7b4e152) into master (89e35d9) will decrease coverage by 1.27%.
The diff coverage is 67.91%.

@@             Coverage Diff              @@
##             master       #8      +/-   ##
============================================
- Coverage     70.58%   69.30%   -1.28%
- Complexity       84      111      +27
============================================
  Files            14       15       +1
  Lines           204      391     +187
  Branches         12       32      +20
============================================
+ Hits            144      271     +127
- Misses           51      101      +50
- Partials          9       19      +10

Impacted Files
Coverage Δ
Complexity Δ

.../baidu/hugegraph/computer/core/bsp/EtcdClient.java
67.91% <67.91%> (ø)
27.00 <27.00> (?)

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 89e35d9...7b4e152. Read the comment docs.


Issue #12: Wrong paths in recipe

https://github.com/chef-cookbooks/jetty/issues/12

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
GitHub Users Who Are Not Authorized To Contribute
The following GitHub users do not appear to have signed a CLA:

@alrf

Please sign the CLA here.

@alrf Thanks for this, but I think the real solution is to switch to the official jettty releases and do away with jetty6 support since it's horribly outdated and EoL.  I'm going to do this in https://github.com/chef-cookbooks/jetty/issues/16


Update AD-FS-FAQ.md

There is a mistake. It's not supported to place third party proxy in front of WAP.

@billmath  Please review and let me know if it's okay to publish. Thanks!!

BTW, the change makes good sense to me, but I don't know the topic well enough to confirm. Waiting for Bill Mathers to confirm that the change is correct.

@gaubert-ms I can work with you to merge this update, but as a Microsoft employee, please use the private repo, https://github.com/MicrosoftDocs/windowsserverdocs-pr in the future. You will have more options from the writing team by going through there.
@billmath  Please review this text and let me know if I can merge it. Thanks!

Looks good.
#sign-off

Hi Elizabeth,
Thank you for your answer. I tried the link you send me but I get a 404. Is it correct?
Regards,
Guillaume.
From: Elizabeth Ross<EMAIL_ADDRESS>Sent: vendredi 15 mai 2020 23:23
To: MicrosoftDocs/windowsserverdocs<EMAIL_ADDRESS>Cc: Guillaume Aubert<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/windowsserverdocs] Update AD-FS-FAQ.md (#4296)
@gaubert-mshttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fgaubert-ms&data=02|01|gaubert%40microsoft.com|d35c12f30dfe4d20a1b108d7f9161a71|72f988bf86f141af91ab2d7cd011db47|1|0|637251745637197140&sdata=Z6qE2a7WPkNfNKbhat4%2FnWnBnJON%2FiC60HQsChYBYjs%3D&reserved=0 I can work with you to merge this update, but as a Microsoft employee, please use the private repo, https://github.com/MicrosoftDocs/windowsserverdocs-prhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fwindowsserverdocs-pr&data=02|01|gaubert%40microsoft.com|d35c12f30dfe4d20a1b108d7f9161a71|72f988bf86f141af91ab2d7cd011db47|1|0|637251745637207126&sdata=ijN1%2F1CZLOcRJcyxkUOwmULEinAb6cZDEj%2FpzhsWseY%3D&reserved=0 in the future. You will have more options from the writing team by going through there.
@billmathhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fbillmath&data=02|01|gaubert%40microsoft.com|d35c12f30dfe4d20a1b108d7f9161a71|72f988bf86f141af91ab2d7cd011db47|1|0|637251745637207126&sdata=TNse31eOp8yok%2BhQltNQWzZXVLXYVSTX4fLpWDW540w%3D&reserved=0 Please review this text and let me know if I can merge it. Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fwindowsserverdocs%2Fpull%2F4296%23issuecomment-629494351&data=02|01|gaubert%40microsoft.com|d35c12f30dfe4d20a1b108d7f9161a71|72f988bf86f141af91ab2d7cd011db47|1|0|637251745637217126&sdata=xA5s%2BOaE8dlpPI3%2Bvl79KNQL7oA0VSUJ291Ko1Imj7E%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAKTVE3SLZOBXQU4FXGU4RJ3RRWXCDANCNFSM4M2IG65Q&data=02|01|gaubert%40microsoft.com|d35c12f30dfe4d20a1b108d7f9161a71|72f988bf86f141af91ab2d7cd011db47|1|0|637251745637217126&sdata=WFPiDU%2Bky%2FxSlOo%2FrLtoadRfuqV1FRCK1%2BCmSu0XMs4%3D&reserved=0.

@gaubert-ms : You probably need to access the internal Microsoft internal PR repository (https://github.com/MicrosoftDocs/windowsserverdocs-pr) directly from this repository by visiting pages where this link is posted. I don't know exactly what happens to link translation in the e-mails you read from GitHub, so maybe the links don't open correctly if you click a link in one of your e-mail messages.

@gaubert-ms  Yes, the 404 error is normal, I see it too :) It just means that you have to sign in to your GitHub account before you can access the repo. The repo needs to make sure you have the right credentials...after that, you can use the instructions here: https://github.com/MicrosoftDocs/windowsserverdocs-pr/blob/master/Contributor-guide/github-browser-updates.md (if you want to edit, but don't want to use a code editor) or here: https://github.com/MicrosoftDocs/windowsserverdocs-pr/blob/master/Contributor-guide/edit-existing-using-github.md (to edit using Visual Studio Code). Hope this helps! Thank you for all of your effort so far, it's greatly appreciated!


Bluesentinel mode?

After downloading the VC_redist.x64 file,
I ran loader, but a warning appears.
I'm not sure what the problem is.
I'll leave you a picture, so please take a look.

BlueSentinel is an anti-cheat mod for Ds3, it also fixes the RCE exploit that caused the official servers to be taken down.
The warning is just telling you that you don't have it installed. You can ignore it, but it's -heavily advised that you use it to fully protect yourself from the RCE.


Deployment Error on NOW

Next.js version: 6.0.2

Build a simple Next.js project by

mkdir next-test
cd next-test
npm init -y
npm install --save react react-dom next

Add deploy scripts to package.json

  "scripts": {
    "dev": "next",
    "build": "next build",
    "start": "next start"
  }

Deploy to NOW

now

Error shows up during the build log:
Deploying ~/projects/tmp/next-test under ###

Using Node.js 8.11.1 (default)
https://next-test-cqdudbobrb.now.sh [in clipboard] (sfo1) [5s]
Synced 9 files (306.47KB) [5s]
Building…
▲ npm install
✓ Using "package-lock.json"
⧗ Installing 3 main dependencies…
events.js:183
Error: Invalid tar header. Maybe the tar is corrupted or it needs to be gunzipped?
at Object.exports.decode (/snapshot/ace/node_modules/tar-stream/headers.js:270:43)
at Extract.onheader [as _onparse] (/snapshot/ace/node_modules/tar-stream/extract.js:120:39)
at Extract._write (/snapshot/ace/node_modules/tar-stream/extract.js:243:8)
at Extract._continue (/snapshot/ace/node_modules/tar-stream/extract.js:207:28)
at oncontinue (/snapshot/ace/node_modules/tar-stream/extract.js:64:10)
at Extract.onheader [as _onparse] (/snapshot/ace/node_modules/tar-stream/extract.js:128:7)
at Extract._write (/snapshot/ace/node_modules/tar-stream/extract.js:243:8)
at Extract._continue (/snapshot/ace/node_modules/tar-stream/extract.js:207:28)
at oncontinue (/snapshot/ace/node_modules/tar-stream/extract.js:64:10)
at Extract.ondrain (/snapshot/ace/node_modules/tar-stream/extract.js:84:5)
  throw er; // Unhandled 'error' event
  ^

▲ npm install
<EMAIL_ADDRESS>postinstall /home/nowuser/src/node_modules/webpack/node_modules/uglifyjs-webpack-plugin
node lib/post_install.js
npm WARN<EMAIL_ADDRESS>No description
npm WARN<EMAIL_ADDRESS>No repository field.
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})
added 682 packages in 7.522s
▲ npm run build
<EMAIL_ADDRESS>build /home/nowuser/src
next build
▲ Snapshotting deployment
▲ Saving deployment image (10.8M)
Build completed
Verifying instantiation in sfo1

App runs fine on the server. Seems some libraries are missing on the now for next.

I can't reproduce based on the steps provided 🤔

seems like you need to create a pages dir with index.js in it for it to start compiling / deploy.
add step 2.5
"
mkdir pages
echo "export default ()  =>  hello" >> index.js
"

Already did while trying, otherwise Next would error out.

This is what I get when I try to deploy to now.
The code is synced to server and started to build but with errors in tar header

@timneutkens Just tried it on a new server on Ubuntu.
NodeJS: v10.1.0 and v8.11.1
NPM: v6
OS: Ubuntu
NOW: v11.1.7
still seeing events.js:183 (unhandled 'error' event)
followed by Invalid tar header...

Hey there, I tweaked some knobs on the backend. Can you please try once again?

@TooTallNate not sure if related, (I haven't changed code in my project), but now npm packages I used randomly failed during the build stage in NOW.
the app still works and get deployed on the server.
Example 1: Error: Error parsing package.json for fsevents-1.2.3.tar

Building…
▲ npm install
✓ Using "package-lock.json"
⧗ Installing 10 main dependencies…
Building<EMAIL_ADDRESS>remotely
Building<EMAIL_ADDRESS>remotely
Building<EMAIL_ADDRESS>remotely
Building<EMAIL_ADDRESS>remotely
Building<EMAIL_ADDRESS>remotely
Building<EMAIL_ADDRESS>remotely
Building<EMAIL_ADDRESS>remotely
Error: Error parsing package.json for fsevents-1.2.3.tar
at extract (/snapshot/ace/lib/extract.js:36:11)
at
at process._tickCallback (internal/process/next_tick.js:188:7)
▲ npm install

Example 2: Error: Error parsing package.json for styled-components-3.2.6.tar

Synced 1 file (2.71KB) [5s]
Building…
▲ npm install
✓ Using "package-lock.json"
⧗ Installing 10 main dependencies…
Building<EMAIL_ADDRESS>remotely
Building<EMAIL_ADDRESS>remotely
Building<EMAIL_ADDRESS>remotely
Building<EMAIL_ADDRESS>remotely
Building<EMAIL_ADDRESS>remotely
Building<EMAIL_ADDRESS>remotely
Building<EMAIL_ADDRESS>remotely
Building<EMAIL_ADDRESS>remotely
Error: Error parsing package.json for styled-components-3.2.6.tar
at extract (/snapshot/ace/lib/extract.js:36:11)
at
at process._tickCallback (internal/process/next_tick.js:188:7)
Building<EMAIL_ADDRESS>remotely
▲ npm install

I'm also getting the Error: Error parsing package.json for fsevents on now 11.3.0 and next 6.1.1.


fingerprint

Describe the bug
i folowed the steps in https://github.com/commaai/openpilot/wiki/Fingerprinting#fingerprinting-20
to fingerprint my santafe but icant edit file on opendbc_repo/opendbc/car/ because the opendbc_repo is another repo can you help please

Closing as this isn't a bug - check out our Discord instead.


Request for 3.5.4 release to deploy #150

Hello! Would it be possible to publish a new release so that we can use #150 from the Maven repo at a pinned version? This recent PR solves a handful of issues across a few projects - this is summarized at https://github.com/becvert/cordova-plugin-zeroconf/issues/63#issuecomment-378483860
#150 Locally potentially related issues:

#143
#116
#107

We look forward to helping a bunch of cordova-plugin-zeroconf users with this update. Thanks for your help!

please update the dependency from slf4j-api-1.7.16.jar to slf4j-api-1.7.25.jar

@Abu-Abdullah please open a new issue with your request and an explanation for doing so.

please update the dependency from slf4j-api-1.7.16.jar to slf4j-api-1.7.25.jar

@Abu-Abdullah Done with https://github.com/jmdns/jmdns/pull/158

Done! https://github.com/jmdns/jmdns/releases/tag/jmdns-3.5.4


oclif dependencies are deprecated

Would be awesome with an upgraded package

@rhummelmose feel free to review https://github.com/contentful-userland/cf-content-types-generator/pull/278

merged!


Is optimization a blocking operation on parameter server?

Assuming that I have 1 parameter server node in dist_async mode and multiple worker nodes, I am trying to do most of the computation on the server using a custom updater function.
Essentially I am trying to implement Word2Vec using async SGD using the parameter server. My updater method receives some indices from the workers and then it performs several matrix operations on the server itself to update the weights.
Will these operations be blocking? If yes, then is it true that I am not achieving any parallelism out of it?

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


arcade kiosk raises error window (Type Error: Failed to fetch)

open private mode window in (edge) browser
navigate to https://arcade.makecode.com/kiosk
see the following problem ("Type Error: Failed to fetch"):

reload and the error goes away

This happened to me yesterday. A refresh fixed it, but I'm not sure what happened.

This is likely fixed in the new deployment. Will test once it makes it out to prod.
Side issue: we need to remove use of window.alert here.

I was able to get a repro and the errors!

Looks like there's an open issue for browsers to safelist this header in the context of CORS: https://github.com/whatwg/fetch/issues/911

Retry logic might be the best solution here. Seems like the browser doesn't always send this header, since it works on page refresh. :shrug:

I know that page refresh will get the kiosk back in a working state, but this makes me think there's a better way we can approach this. Other arcade applications don't have a problem with fetching assets, so if we can consolidate without retries, I think that would be my preferred path forward.
I guess I'm just curious why this is a problem for kiosk. If, with fully moving this over now, we can more mirror what happens with our other web apps, that would be nicer at least.

Agreed. It would be good to know why this is happening in kiosk and not to similar apps like skillmap and multiplayer. There is something peculiar going on here we're not seeing elsewhere.

The CORS check itself seems entirely unnecessary in this case. It's hitting CDN, all requests to it are cross origin.

I now cannot get this to repro very easily. I've restarted my computer many times to try to get the error but I only got it once this morning. The failure I got was not the same, either. I got a network error instead of the CORS error. Not sure how to proceed here.

From what I've gathered and chatted with Eric about, including traceparent and tracestate to our allowed headers will be the best route forward for the CORS error that I saw, but if the CORS error isn't the real error to look out for, then I wouldn't want to make that change.

Actually this request isn't hitting our backend so modifying CORS handling isn't an option here.
I can get a 90% repro in Firefox if I hit refresh a bunch. Here's what I see in the sent headers:

I highlighted the problematic headers. It doesn't always send them. Here's the error response:
Cross-Origin Request Blocked: The Same Origin Policy disallows reading the remote resource at https://pxt.azureedge.net/blob/9e5e9051f8f72796b42a1e3320fad605192cdbd1//kiosk-data/GameList.json. (Reason: header ‘request-id’ is not allowed according to header ‘Access-Control-Allow-Headers’ from CORS preflight response).

Retry logic is probably the way to go here.

Oh, I'm glad(ish) to see that you got a network error as well. Okay, I'll go with retry logic.

Still seeing this occasionally. Just a thought: I wonder if a small delay before the second fetch attempt would help.

Yeah, I can do that. When you see this, are you still getting the same errors?

I can replicate consistently on Safari by holding shift+reload

If you reload again will you get the error? I'm worried that just trying to delay the retry a bit won't really solve the problem.

Still happening in my logged-in window. Not happening at all in a private window.

I believe this is fixed now. Please reopen if you see it.

still happening :/

Should be fixed now. Please reopen if you see it again.


feat: Equivalence between Alexandrov-discrete spaces and preorders

Construct the category equivalence between Alexandrov-discrete spaces and preorders

[x] depends on: #6980
[x] depends on: #6981
[x] depends on: #6982
[x] depends on: #6985
[x] depends on: #6986
[x] depends on: #6988

Thanks :tada:
bors merge


随机播放重点列表歌曲是否应该再重新刷新生成一个队列

仅仅提交一个建议。Submit just one proposal.
在你提交此建议前请注意：Please note before you submit this proposal:

详细阅读椒盐音乐帮助内容（应用设置-帮助）。Read more about the SaltPlayer help content (app Settings - Help).

反馈时候请注意：When giving feedback, please note:

尽量详细的描述。Be as detailed as possible.
不接收任何 UI 样式调整相关的反馈，出现此类 issue 会被关闭（如 “我不喜欢此设计” 、“我觉得这样设计更好” 、“还没原来好看” 等等类似反馈）。Do not receive any feedback on UI style changes, and the issue will be closed (" I don't like this design ", "I think this design is better", "it's not as good as the original", etc.).

建议内容 The proposals
<在此输入你的建议 Enter your suggestions here>

7.0.7 版本完成开发


Windows: Installer or portable .exe?

On Windows, Julia has the option to download an installer (is this user or system level?), or a portable .exe. I remember that in my high school, I was able to run portable exes, but not installers, maybe this is still common for school computers.
VS Code offers all options: user installer, system installer and a "portable exe" (a zip containing the full installation and Code.exe).

Some questions:

Right now we have an installer, right?

Is this a user or system level installer?

Can a portable exe self-update? Might be interesting to download an old VS Code zip, run it, and ask it to update itself.
Are there situations (like in my old high school) where people would only be able to use Pluto if it was a portable exe?

In these situations, would a portable Julia exe with Pkg.add("Pluto") work?

Here are my thoughts:

The executable we currently package is an installer

By default it's installed on a system level, although I think the installer has a configuration option for making it user-level

Portables can probably self-update, although I've never heard of it being done
There are definitely situations where this is useful, my university doesn't allow customized system installs either

One issue might be figuring out where to locally install Julia. It would be great if we could somehow package the Julia executable entirely within the PlutoDesktop executable. In principle I think its possible, but I've never done anything like that before.

Ah then let's start by making it a user-level installer, right?
I think we already bundle Julia and the julia depot in the installer, take a look!


[Feature Request]: Add LMStudio support

Feature Description
Add a way to connect LMStudio with LlamaIndex.
Reason
Add direct implementation for LMStudio
Value of Feature
This features enables loading models on LMStudio and allows LlamaIndex index to send request to the API provided by LMStudio.

how to use embedding model from LMStudio?

@nielinjie Haven't implemented the logic for that yet. Perhaps you can create an issue explaining your use case.


DefaultLogSystem constructor makes it hard to log to stderr

For my testing program, I would like to log to stderr. The signature of DefaultLogSystem makes that quite hard though.
I would like to just plug in a pointer/reference to std::cerr, but according to the constructor it must be wrapped inside a shared_ptr.
For obvious reasons, I can't put std::cerr into a smart pointer or my program will segfault.
I tried making a wrapper object using Boost.Iostreams but for some reason that has a static_assert saying that I'm not allowed to wrap any of the built-in streams.
It would be good if there was a constructor where I could just pass in a naked pointer (or you supply an object that I can construct and pass that wraps another std::ostream).

What about making the shared_ptr deleter a noop?

Bret did something like you are trying to do in the android tests. I'm not
by a computer at the moment but you may want to have a look at that. I'll
have a look as soon as I make it to my machine.
On Oct 6, 2015 11:52 AM, "Christopher J. Hanks"<EMAIL_ADDRESS>wrote:

What about making the shared_ptr deleter a noop?
—
Reply to this email directly or view it on GitHub
https://github.com/awslabs/aws-sdk-cpp/issues/30#issuecomment-145963613.

Doh, that's a simple solution.
Still doesn't feel quite right but at least it gets me out of a slump right now :). Thanks


Unexpected Error Occurring

Hello,
I am experiencing an unexpected error since a few days with my SteamBot.
When I launch my SteamBot.exe, I get the following : (log copied from console)
[(System) 2016-05-17 12:47:39] INFO: Launching Bot Bot 1...
[(System) 2016-05-17 12:47:39] DEBUG: Starting bot with username rohan6516.
[Bot 1 2016-05-17 12:47:39] INFO: Connecting...
[Bot 1 2016-05-17 12:47:39] SUCCESS: Done Loading Bot!
Type help for bot manager commands.
botmgr > [Bot 1 2016-05-17 12:47:43] ERROR: Login Error: AccountLogonDeniedNeedT
woFactorCode
[Bot 1 2016-05-17 12:47:44] SUCCESS: Generated 2FA code.
[Bot 1 2016-05-17 12:47:48] SUCCESS: User Authenticated!
[Bot 1 2016-05-17 12:47:48] SUCCESS: Steam Bot Logged In Completely!
[Bot 1 2016-05-17 12:47:48] SUCCESS: Starting task loop

Unhandled Exception: Newtonsoft.Json.JsonReaderException: Unexpected character e
ncountered while parsing value: <. Path '', line 0, position 0.
   at Newtonsoft.Json.JsonTextReader.ParseValue()
   at Newtonsoft.Json.JsonTextReader.ReadInternal()
   at Newtonsoft.Json.JsonTextReader.Read()
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.ReadForType(Jso
nReader reader, JsonContract contract, Boolean hasConverter)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.Deserialize(Jso
nReader reader, Type objectType, Boolean checkAdditionalContent)
   at Newtonsoft.Json.JsonSerializer.DeserializeInternal(JsonReader reader, Type
 objectType)
   at Newtonsoft.Json.JsonConvert.DeserializeObject(String value, Type type, Jso
nSerializerSettings settings)
   at Newtonsoft.Json.JsonConvert.DeserializeObject[T](String value, JsonSeriali
zerSettings settings)
   at SteamAPI.TradeOffers.TradeOffers.GetTradeOffers(Boolean getActive) in d:\D
ocuments\SteamTradeOffersBot-SteamTradeOffersBot\SteamAPI\TradeOffers\TradeOffer
s.cs:line 208
   at SteamAPI.TradeOffers.TradeOffers.<CheckPendingTradeOffers>b__3d() in d:\Do
cuments\SteamTradeOffersBot-SteamTradeOffersBot\SteamAPI\TradeOffers\TradeOffers
.cs:line 377
   at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionCo
ntext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
   at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, C
ontextCallback callback, Object state, Boolean preserveSyncCtx)
   at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, C
ontextCallback callback, Object state)
   at System.Threading.ThreadHelper.ThreadStart()

My bot used to work perfectly like 2 weeks ago then I started getting this error, I posted on /r/Steamot, There somebody told me this is a steam issue as it is returniing HTML instead of JSON and should be fixed but this still occurs.
Help appreciated!
Thanks!

You are using https://github.com/waylaidwanderer/SteamTradeOffersBot.

This sounds like a legitimate bug report, but it's not for this repo.  Please post to the correct repo.

Just Steam being poop sending HTML instead of JSON over the wire. 😄  Could be handled differently though.


Add parallelism when performing multiple tasks at the same that can benefit from it

This PR adds parallelism using the ProcessPoolExecutor when and where we can benefit from executing multiple tasks on potentially multiple cores.  It will default to using number of processors on the machine for the number of workers.
Fixes #40
Introduces #47

There seems to be a lingering issue with this PR and the unit tests in #49 . Let me fix it before finishing the review.

The b59b0601cd159506106bf0fd60f9e942b3f30144 commit should fix #49. There seesm to be another unit test issue in https://github.com/eth-educators/ethstaker-deposit-cli/actions/runs/9273010996/job/25512131191?pr=48 which need more fixes.

It seems like all remaining issues are fixed. You can have another look @valefar-on-discord .


Use GitHub as docker registry

GitHub's registry is (as of today) not suitable for our case because it requires a personal access token to pull a public image.


Add/Remove dynamic children editors with Tune

Added ability to add children EditorJS layouts to current layout by Tune.
Added ability to remove children EditorJS with remove editor button
Added ability to update container class, and display on screen
Added ability to update children container classes and display on screen.
Added css parsing to style plugins, and load on screen.

I need to add the ability to update the ID for containers so don't merge yet. Thanks


Fix parallel repair

solveCrosswordRow sets cells for the crossword which are accessed in solveCrosswordColumn, which means that both being run in parallel has a race condition.
This PR resolves this by running all solveCrosswordRow in parallel and waiting for all these calls to finish before executing all solveCrosswordColumn calls in parallel

[x] Fixes #123

This PR also includes switching leopard to klauspost, the caching changes, amongst other things, so it cannot be reviewed in isolation.
Can you include only the changes that are made by the parallelisation_crosswordloop branch and this fix, or make a PR to the parallelisation_crosswordloop branch with this fix.
Furthermore, I had some questions:

This seems to imply the reason why parallelisation_crosswordloop didn't work was not because of the decoder, but because of rsmt2d. In that case how come it worked with infectious but not leopard or klauspost?

Blocking column repair until row repair is complete means that we aren't maximizing the parallelisation that we can do. Is there another way to fix this, eg add mutexes for setting cells?

Whoops, sorry. I thought I was making the PR parallelisation_crosswordloop, might've switched back on refresh. I'll close this and make a new PR.
I'll address the questions on the new PR @musalbas


docs(introduction): update content with more graphic explanation.

Add some more graphic introduction.

Deploy preview ready!
Built with commit f4a9fa4f92a083ad48e6dbb620bddf123f6eb5bb
https://deploy-preview-318--mcs-lite-mobile-web.netlify.com

Deploy preview ready!
Built with commit f4a9fa4f92a083ad48e6dbb620bddf123f6eb5bb
https://deploy-preview-318--mcs-lite-ui.netlify.com

Deploy preview ready!
Built with commit f4a9fa4f92a083ad48e6dbb620bddf123f6eb5bb
https://deploy-preview-318--mcs-lite-introduction.netlify.com


Error When Installing

When running mvn clean install in the project directory, I get this error. I am new to Maven and am generally unsure why this error is happening, any help with getting the build to work is appreciated.

That looks like an error internal to Maven. I would check to make sure you have it set up correctly.


Matching on StatefulSet via annotations

Problem Statement
In ClusterPolicy yml, I want to rule can filter out the eligible 'spec.template.metadata.annotations', about the StatefulSet.
Solution Description

If my StatefulSet 'spec.template.metadata.annotations' ‘app’ is not empty, I will for this StatefulSet ‘Mutate Rules’。
Example "Good" Resource
No response
Example "Bad" Resource
No response
Other Comments
No response
Slack discussion
No response
Troubleshooting

[X] I have read and followed the documentation AND the troubleshooting guide.
[X] I have verified the policy does not exist in the samples library.

Please avoid posting screenshots of code and terminal output. We cannot search it. You can use preconditions to match this.

Please complete the issue template correctly and completely or this will be closed.

No response, closing.


pepo webhook

added Pepo webhook. more information here:

https://www.notion.so/Pepo-Webhooks-6d1681c40d0d4699888a3dd09efb93a4

Pepo no longer exists. therefore this PR is useless.


[ROCKETMQ-355] Client asyncSend is not fully async

What is the purpose of the change
Client asyncSend is not fully async because of param validation, connection status check and get route info may cause exception,and these exceptions is called in user thread may cause biz stop.
Brief changelog
Make client asyncSend fully async.
Verifying this change
This is a trivial change.
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a JIRA issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a JIRA issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ROCKETMQ-XXX] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

@zhouxinyu,yeah, keep comment is a good idea for continuous develop. At first I want to limit the send request and using network semaphore as send signal,but after doing that I found it was a mistake and open another PR for concise and speed.


add delete button to /repl

Now that we have the ability to delete apps from /apps, it should be straightforward to add the ability to delete a single app from within /repl (albeit by requesting a DELETE from /repl/[id] rather than a POST to apps/destroy). Need to figure out what happens on the page after you've deleted though

Need to figure out what happens on the page after you've deleted though

Bringing you to the /apps page seems like a straightforward and sensible thing to do after deleting the current REPL.

closing since the site has been rewritten. please file at https://github.com/sveltejs/svelte.dev/issues/new if the issue persists


feat(object_store): Add buffered::BufUploader

Which issue does this PR close?
Closes https://github.com/apache/arrow-rs/issues/5834
Rationale for this change
Add buffered::BufUploader to provide buffered support via PayLoad native API.
What changes are included in this PR?
New struct: buffered::BufUploader
Are there any user-facing changes?
New API: buffered::BufUploader
I left tags and attributes support in following PRs. They should be easy to implement upon current API without breaking changes.

Kindly cc @alamb and @tustvold for a look, thanks!

I will try and review this PR later today or this weekend. Thanks @Xuanwo

I will try and review this PR later today or this weekend. Thanks @Xuanwo

Thanks a lot! There is no hurry for this PR to get merged since we won't release soon. I just want to know about your plan first.

Hi sorry, had a few too many things going on this week. I'll review this on Monday. This seems to have a lot in common with the existing BufWriter and so I wonder if we can combine them somehow

Hi, @alamb, I have addressed all comments. PTAL, thanks!

Hi apologies I have been ill the last few days, looking again at this I think we could simply achieve this by adding BufWriter::put with the same signature as BufUploader::put.
In my mind this would have a few key benefits:

Would avoid additional cognitive overhead for users wondering which abstraction to use
Is consistent with WriteMultipart which has both write and put methods
Allows mixing the write and put APIs, e.g. use write for thrift encoding and put for writing parquet pages
Already supports all the attributes, tags, etc... #5867
Avoids code duplication

What do you think?

Hi apologies I have been ill the last few days,

Wishing you a speedy recovery. Best wishes.

looking again at this I think we could simply achieve this by adding BufWriter::put with the same signature as BufUploader::put.

As long as we can avoid the extra Box::pin(fut), I'm ok with this design. I will take a look.

Hi, @tustvold, I have added put for BufWriter directly, please review again.

Thanks @Xuanwo

Thanks a lot for the quick review and nice design from @tustvold

I don't know if a week of reviews qualifies as quick -- I hope to do better ;)

Thanks again @Xuanwo


Use path segments parameter for Url.crossOrigin

It'd be great if the urls could be generated using Url.crossOrigin's path segments parameter. In my case, an openapi endpoint /example/{x} results in this Elm code
            Url.Builder.crossOrigin
                (String.replace
                    "{x}"
                    config.params.x
                    "http://example.com/example/{x}"
                )
                []
                ...

which then results in an url with a trailing slash http://example.com/example/x/. However, this then does not match the endpoint's url in the openapi spec which is without trailing slash.
Would it be possible to have the Elm code being generated like this:
            Url.Builder.crossOrigin
                "http://example.com"
                [ "example"
                , config.params.x
                ]
                ...

Then the resulting url would be without trailing slash and hence matching the openapi spec.

Moved to the correct repo,https://github.com/wolfadex/elm-open-api-cli/issues/72


Crash while reading the configuration file.

Hello again! Just updated to the last version.
Select Add-On (place a lowercase x for the relevant add-on)

[x] Xiaomi Mi Scale

Describe the bug
Xiaomi Mi Scale crashes while reading the config file with this error:
-------------------------------------
2020-09-09 09:42:40 - Starting Xiaomi mi Scale...
2020-09-09 09:42:40 - Loading Config From Add-On Options...
2020-09-09 09:42:40 - Config Loaded...
Traceback (most recent call last):
  File "/opt/miscale/Xiaomi_Scale.py", line 344, in <module>
    main()
  File "/opt/miscale/Xiaomi_Scale.py", line 312, in main
    if MQTT_DISCOVERY.lower() in ['true', '1', 'y', 'yes']:
AttributeError: 'bool' object has no attribute 'lower'

To Reproduce
Update to the last version and try to start the add-on.
Expected behaviour
I would expect it to load the configuration file without crashing. Also, seeing that you are treating the MQTT_DISCOVERY paramater as string I tried to put that parameter in the config file between quotation marks, but didn't help. It was like this:
MQTT_DISCOVERY: true

And tried like this:
MQTT_DISCOVERY: 'true'

None of them worked
Scale (please complete the following information):

Name Mi Body Composition Scale
Model XMTZC02HM

Device running Home Assistant (please complete the following information):

Raspberry pi 4 B 4GB
Built-in

same error on 0.1.9

Thanks both, I've located the issue and fixed it
Currently pushing 0.1.10...

fixed in 0.1.10

after 0.1.10 this error gone, but something still broken - cant see weight in lovelace
in log
------------------------------------- 2020-09-09 22:59:30 - Starting Xiaomi mi Scale... 2020-09-09 22:59:30 - Loading Config From Add-On Options... 2020-09-09 22:59:30 - Config Loaded... 2020-09-09 22:59:30 - Discovery Completed... 2020-09-09 23:00:06 - Measuring Unit: 82 2020-09-09 23:00:06 - Measured Raw Weight: 0.6 2020-09-09 23:00:41 - Measuring Unit: 82 2020-09-09 23:00:41 - Measured Raw Weight: 0.6 2020-09-09 23:01:16 - Measuring Unit: 82 2020-09-09 23:01:16 - Measured Raw Weight: 0.6 2020-09-09 23:01:51 - Measuring Unit: 82 2020-09-09 23:01:51 - Measured Raw Weight: 0.6
in entity

in lovelace

There was a breaking change in 0.1.8
All json attributes are now in lower snake_case
You need to adjust your HA config and replace Weight with weight


Add test for custom exception

Currently broken, passes with the fix from #21.

With #21 merged, do we need this explicit test?

I think, it prevents regression and, may be clearer in intention. Thanks!


Can't add/edit Product List Block after updating from <IP_ADDRESS> to <IP_ADDRESS>

The error is thrown twice (if that matters). Happens when adding new product list blocks or editing existing ones.
I've updated the package with git pull, upgraded it in the dashboard and refreshed the db.xml file for the product list block.
Any ideas how to fix that?

Same thing happened after installing community store on a different c5 installation (v.7.5.9).

Temporarily solved by setting  $args['relatedPID'] = 0; in the Block Types' save method.

This looks like a bug from MySQL strict mode (and not testing in it).
I've pushed up this commit as a fix:
https://github.com/concrete5-community-store/community_store/commit/495e10f076dab94f91a51e03b80b16d81c171c07


Migrated to ArcGIS Runtime 10.2.6-2 and Android Studio

Fixes #104
This is a big pull request because we had to move a lot of files around to migrate from Eclipse to Android Studio. We also removed Eclipse-specific files and added Android Studio-specific files.
@csmoore, if you have time in the next week, please take a look and merge.

Actually @csmoore, hold off please. I have a couple of things I need to do first.

Sure - if it can wait until Monday, I will have some time to actually try to set up the dev environment to test out the build

Part of what I need to do is document how to build it, so hopefully it will be relatively easy for you.

@csmoore, this is ready to check when you are ready. Thanks in advance.

Sure, I'll have a look - under the guise if 'if I can do it anyone can' - just a heads up I may not get to until next Monday (I am out one of the days)

Not part of these updates but the prebuilt Download link at the beginning of the readme http://www.arcgis.com/home/group.html?owner=Arcgisonline_defense&title=ArcGIS for Defense and Intelligence looks like it might be the old one vs. http://www.esri.com/apps/products/download/#ArcGIS_for_the_Military

Good catch. I updated it with a better link and text in 7e8bed7583e8214f204dc9c9d231186a25f242e5 .

I tried the gradle build first & got this error
> squad-leader-android\source\SquadLeader>gradlew.bat assemble

FAILURE: Build failed with an exception.

* What went wrong:
A problem occurred configuring project ':aFileChooser'.
> failed to find target android-19 : C:\Users\chri0020\AppData\Local\Android\sdk

BUILD FAILED

I have -23 installed - do I need to install older SDK versions and/or update some configuration file somewhere?:

I wasn't able to get the default Nexus 5 emulator to run - but I can confirm the build works fine. The only issues I had/noticed that you might want to address:

[ ] those 2 dependencies (android-19 & Build Tools 21)
[ ] add any additional steps needed to run the unit test
[ ] perhaps get someone to sanity check the apk since I wasn't able to run/test

It is ready to merge after you decide if/what to do with those issues

We actually only need Android API level 15 for this app, so I changed the requirement to that in<PHONE_NUMBER>59b7fcc6ba08abde31b884ce0c577f . I will document that and the build tools requirement, with a note that you can modify build.gradle if you want to use newer versions. Sorry that was unclear.

I clarified the dependencies in e74733f0c1ca67b38f13141de5e9d69f27aff652.

On the missing All Tests, that's my mistake. It's stored in .idea/workspace.xml, which is in .gitignore and which IntelliJ recommends be ignored by version control systems. So I will instead add documentation on how to run unit tests.

I added additional steps for unit tests in e854068508cd6a83d9cfc589f0ce534ef33aa3d4.
@csmoore, unless you object, I would suggest that we save the final user testing with an .apk until we're closer to an actual release. If that's agreeable, I think we're ready to merge unless I missed anything. Thanks again for your great review.

Sure, sounds good - I just wanted to let you know I hadn't actually run - merging now


[BUG] I put my FPS limit to 9999 (no limit), yet roblox is running on 30-60 FPS

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.

What problem did you encounter?
I did read the wiki, yet the solutions did nothing. I'm just wondering if there's another solution to this problem. Even if my graphics is on 1, it still stays on 30-60. Its pretty annoying now.

It can also possibly go down to 1-10.

Are you actually capable of exceeding 60? In which case, of course you won't be going past 60.
I just want to ensure that this isn't a performance issue before I suggest configuration changes, so please momentarily switch back to stock Roblox and check if you experience similar performance issues there. https://github.com/pizzaboxer/bloxstrap/wiki/Switching-between-Roblox-and-Bloxstrap

On roblox's bootstrapper, it's still the same, capped to 60 and it goes as low as 10. (I tested this on Arsenal)

It could be capped to 60 due to my pc being a laptop, but sharkblox (Who also used a laptop in reviewing bloxstrap) wasn't capped, and it ran smoothly. (Sorry for my bad english I'm asian.)

Okay, so you are being capped at 60 with Bloxstrap. Have you read through this? https://github.com/pizzaboxer/bloxstrap/wiki/Roblox-is-still-capped-to-60-FPS-after-raising-the-limiter

Bloxstrap_20240122T092907Z.log
this?

Bloxstrap_20240128T025327Z.log
this?

Yep, that's what I need, thanks.
It definitely is configuring the limiter. You haven't configured any other options in the menu, have you?

only changed the limiter to 0, as it ran more smoothly that 9999

Try setting it to 120, see what happens.

on it

Average is 34. Tested this on Arsenal

It does go over 60 though, reaching only 70 maximum.

Be quick please its 12 am in the morning and I'm gonna sleep.

It does go over 60 though, reaching only 70 maximum.

Alright, seems like it's working then.

thanks


Selective Collapse option to exclude comments getting collapsed or modifying default collapse behavior feature.

I'm mostly coding in JS or TS and using Collapse by levels while browsing quickly. ( CMD+K CMD+[n] )
When I collapse 1st level it also collapses comments (JSDoc, TS Definitions) which I would like to exclude.
If possible, it would be helpful to exclude option for default collapse behaviour.

duplicate of https://github.com/microsoft/vscode/issues/48556


Upgrade xgboost-spark to spark-3.0-preview2

Upgrade xgboost-spark to spark-3.0-preview2

Gently ping @CodingCat , Thanks!

thanks for the contribution, but we cannot support a preview version
and based on the current progress of 3.0, even it is formally released,  we may be in a place that support both 2.4.x and 3.x


Pattern match with variable sub-pattern doesn't create the match variable

Pattern match with variable sub-pattern doesn't create the match variable.  For example (from Date::Manip::TZ):
if ($val =~ /^now${dstrx}${zonrx}$/oi) {
   ($dstflag,$zone) = ($1,$2);
}

gives:
    [dstflag, zone] = (_m.group(1), _m.group(2))
NameError: name '_m' is not defined. Did you mean: 'm'?

Fixed in v1.030


Incompatibility with AGP 8.2.0

We are trying to update our project to Gradle and AGP 8.2.0 (from 8.1.4), but the build fails once update is performed.
Tried with SDK versions 4.1.4, 4.1.5 and 4.2.0.
Stacktrace
> Could not resolve all files for configuration ':payments:judo:debugCompileClasspath'.
   > Could not resolve com.squareup.okhttp3:okhttp-bom:4.9.3.
     Required by:
         project :payments:judo > com.judopay:judo3ds2:1.3.1
      > No matching variant of com.squareup.okhttp3:okhttp-bom:4.9.3 was found. The consumer was configured to find a library for use during compile-time, preferably optimized for Android, as well as attribute 'com.android.build.api.attributes.BuildTypeAttr' with value 'debug', attribute 'com.android.build.api.attributes.AgpVersionAttr' with value '8.2.0', attribute 'org.jetbrains.kotlin.platform.type' with value 'androidJvm' but:
          - Variant 'apiElements' capability com.squareup.okhttp3:okhttp-bom:4.9.3 declares a component for use during compile-time:
              - Incompatible because this component declares a platform and the consumer needed a library
              - Other compatible attributes:
                  - Doesn't say anything about com.android.build.api.attributes.AgpVersionAttr (required '8.2.0')
                  - Doesn't say anything about com.android.build.api.attributes.BuildTypeAttr (required 'debug')
                  - Doesn't say anything about its target Java environment (preferred optimized for Android)
                  - Doesn't say anything about org.jetbrains.kotlin.platform.type (required 'androidJvm')
          - Variant 'enforcedApiElements' capability com.squareup.okhttp3:okhttp-bom-derived-enforced-platform:4.9.3 declares a component for use during compile-time:
              - Incompatible because this component declares an enforced platform and the consumer needed a library
              - Other compatible attributes:
                  - Doesn't say anything about com.android.build.api.attributes.AgpVersionAttr (required '8.2.0')
                  - Doesn't say anything about com.android.build.api.attributes.BuildTypeAttr (required 'debug')
                  - Doesn't say anything about its target Java environment (preferred optimized for Android)
                  - Doesn't say anything about org.jetbrains.kotlin.platform.type (required 'androidJvm')
          - Variant 'enforcedRuntimeElements' capability com.squareup.okhttp3:okhttp-bom-derived-enforced-platform:4.9.3 declares a component for use during runtime:
              - Incompatible because this component declares an enforced platform and the consumer needed a library
              - Other compatible attributes:
                  - Doesn't say anything about com.android.build.api.attributes.AgpVersionAttr (required '8.2.0')
                  - Doesn't say anything about com.android.build.api.attributes.BuildTypeAttr (required 'debug')
                  - Doesn't say anything about its target Java environment (preferred optimized for Android)
                  - Doesn't say anything about org.jetbrains.kotlin.platform.type (required 'androidJvm')
          - Variant 'runtimeElements' capability com.squareup.okhttp3:okhttp-bom:4.9.3 declares a component for use during runtime:
              - Incompatible because this component declares a platform and the consumer needed a library
              - Other compatible attributes:
                  - Doesn't say anything about com.android.build.api.attributes.AgpVersionAttr (required '8.2.0')
                  - Doesn't say anything about com.android.build.api.attributes.BuildTypeAttr (required 'debug')
                  - Doesn't say anything about its target Java environment (preferred optimized for Android)
                  - Doesn't say anything about org.jetbrains.kotlin.platform.type (required 'androidJvm')

Module gradle config
plugins {
    id("com.android.library")
    id("org.jetbrains.kotlin.android")
    kotlin("kapt")
    id("dagger.hilt.android.plugin")
}

android {
    namespace = "[REDACTED].payments.judo"
    compileSdk = 34
    buildToolsVersion = "34.0.0"

    defaultConfig {
        minSdk = 24

        testInstrumentationRunner = "androidx.test.runner.AndroidJUnitRunner"
        consumerProguardFile("consumer-rules.pro")
    }

    packaging {
        resources {
            excludes += setOf(
                "META-INF/LICENSE.txt",
                "META-INF/NOTICE.txt",
                "META-INF/AL2.0",
                "META-INF/LGPL2.1",
                "META-INF/DEPENDENCIES"
            )
        }
    }

    buildTypes {
        release {
            isMinifyEnabled = false
            proguardFiles(getDefaultProguardFile("proguard-android-optimize.txt"))
        }
    }
    compileOptions {
        sourceCompatibility = JavaVersion.VERSION_17
        targetCompatibility = JavaVersion.VERSION_17
    }
    kotlinOptions {
        jvmTarget = JavaVersion.VERSION_17.toString()
    }
}

kapt {
    correctErrorTypes = true
}

hilt {
    enableAggregatingTask = true
}

dependencies {
    implementation(project(":base"))
    implementation(project(":payments:base"))

    implementation("io.reactivex.rxjava2:rxjava:2.2.21")

    implementation("com.judopay:judokit-android:4.2.0")

    implementation("com.google.dagger:hilt-android:2.48.1")
    kapt("com.google.dagger:hilt-android-compiler:2.48.1")
    kapt("androidx.hilt:hilt-compiler:1.0.0")

    implementation("com.squareup:otto:1.3.8")
}

Hey @Ekalips,
Thank you for raising this,
We're on it!

Hey @Ekalips,
Release 4.2.1 should fix the issue described here.


refactor:Command objects use memory pools

优化命令对象, 使用内存池管理命令对象 避免每次执行命令都需要复制一个命令,

Summary by CodeRabbit

New Features

Introduced a MemoryPool class for efficient memory management.
Added new Clone methods across various command classes to utilize the memory pool for object allocation.
New methods in PikaCmdTableManager for retrieving raw commands and calculating maximum command size.

Bug Fixes

Adjusted parameter types in several methods to improve memory management and ownership semantics.

Documentation

Enhanced error handling and logging in the ACL system.

Chores

Updated constructor signatures across multiple classes to include memory pool parameters.

void func1() {
    MemoryPool pool;
    std::vector<C *> vector;
    vector.reserve(20);
    for (int i = 0; i < 1000000; ++i) {
        vector.clear();
        for (int j = 0; j < 20; ++j) {
            C *c = pool.Allocate<C>();
            vector.emplace_back(c);
        }
        for (const auto &item: vector) {
            pool.Deallocate(item);
        }
    }
}

void func2() {
    std::vector<C *> vector;
    vector.reserve(20);
    for (int i = 0; i < 1000000; ++i) {
        vector.clear();
        for (int j = 0; j < 20; ++j) {
            C *c = new C();
            vector.emplace_back(c);
        }
        for (const auto &item: vector) {
            delete item;
        }
    }
}

int main() {
    auto start = std::chrono::high_resolution_clock::now();
    func1();
    auto end = std::chrono::high_resolution_clock::now();
    std::chrono::duration<double, std::milli> elapsed = end - start;
    std::cout << "Function1 took  ms:" << elapsed.count() << " s:" << elapsed.count() / 1000.0f << std::endl;

    start = std::chrono::high_resolution_clock::now();
    func2();
    end = std::chrono::high_resolution_clock::now();
    elapsed = end - start;
    std::cout << "Function2 took  ms:" << elapsed.count() << " s:" << elapsed.count() / 1000.0f << std::endl;
    return 0;
}

用这个例子测试下来, 性能提升还是非常明显的


tab scrollbar is possessed 😵

VSCode Version: Code 1.7.2 (7ba55c5860b152d999dda59393ca3ebeb1b5c85f, 2016-11-21T22:06:37.428Z)
OS Version: Windows_NT ia32 10.0.14931

See gif below - zoom level is moderately high, and many tabs exist in the left tab wells. After dragging a file from right to left tab well, the tab scrollbar moves all the way to the left. any further attempts to scroll back to the right will fail, as the scrollbar has taken on a life of its own.

@mousetraps hard to reproduce for me in insiders, I wonder if this got fixed with the Electron update. Can you try with insiders to reproduce?

@mousetraps ping

@bpasero sorry for the late reply - just validated that I can no longer repro, so looks like it got fixed


Connected! Waiting for guilds to become available...

How can I skip this

You don't. It's part of the bot's startup. If it is on that message for too long then your IDs are wrong or Discord is having an outage.


Question about removed_docs argument to query

Thanks for the library, I spent quite a bit of time researching the available libraries for my project and this one strikes the perfect balance for my needs, in particular the support for webassembly is critical!
It seems that if I vacuum the index then there is no need to pass removed_docs as the last argument to query?
Am i right in thinking that the removed_docs argument exists to support the case when documents have been removed but not yet vacuumed from the index and the query should ignore them?

Thanks for your question! Great that you find it useful!
You have understood it correctly. You only need to pass it if you have not vacuumed the index. The reason for this is that vacuum is a scanning operation, while passing the removed_docs HashSet acts like a filter. An easy condition for vacuuming could be to do it when the removed_docs set has grown too large. Or do it every X minute (depends on your use case)

Thanks for the prompt reply, much appreciated 🙏


Support for Swift 3.2 and Xcode 9

This converts Substring to String and explicitly constructs String from a subscripted result.

This PR solves Xcode 9 build problems. Thanks @Performat

Thank you very much <3


1.12 kick from the server

https://pastebin.com/raw/Mr7ZL77r
Latest viaversion, viarewind and viarewind legacy support.

Please follow the issue template and attach the viaversion dump link.

Will reopen when all info is provided, see issue template https://github.com/MylesIsCool/ViaVersion/blob/master/.github/ISSUE_TEMPLATE/bug_report.md

@krusic22 @MylesIsCool Join the server with 1.12, get kicked from the server? What else do you want?

c5c7d36bd11696f5414ecc5640f1e0dbe12cc5fa is the cause of this issue

@creeper123123321 so you added a function to kick 1.12 players with this commit?

It is fixed. https://ci.viaversion.com/job/ViaVersion/


Move has now been implemented

Hi @seesharprun,
Are you the right person to review the changes to this article?
If you are, can you review the proposed changes?

When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.

Otherwise, do you know who might be the right person to review?
Thanks.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Thanks, @seesharprun! When you get a chance, can you open a PR in the private repo to update the author and ms.author metadata fields?


Improve error handling and pipe flow for just ps

Fixes

Fixes #1899 by @sarayourfriend
Description

This PR makes two changes to the just ps recipe:

It changes the recipe itself so that it never reports a failing exit code, meaning it doesn't fail any subsequent jobs
It adds some additional error handling to the ps.py script so that errors are more clearly reported.

Here's an example of a failing run:
$ j catalog/up "s3 upstream_db"
env COMPOSE_PROFILES="catalog" just ../up s3 upstream_db
env COMPOSE_PROFILES="catalog" docker-compose -f docker-compose.yml -f docker-compose.overrides.yml up -d s3 upstream_db
[+] Running 3/3
 ⠿ Network openverse_default          Created                                                                                0.1s
 ⠿ Container openverse-upstream_db-1  Started                                                                                0.5s
 ⠿ Container openverse-s3-1           Started                                                                                0.6s

python3 utilities/ps.py || true
================================================================================
                                 Service Ports
================================================================================
Traceback (most recent call last):
  File "/home/madison/git-a8c/openverse/utilities/ps.py", line 104, in print_ps
    for service in parse_ps():
  File "/home/madison/git-a8c/openverse/utilities/ps.py", line 69, in parse_ps
    raise ValueError(f"Unexpected line format from `docker ps`: {line}")
ValueError: Unexpected line format from `docker ps`: openverse-upstream_db-1	openverse-upstream_db	5432/tcp

================================================================================
Failed to get service info (ValueError):
Unexpected line format from `docker ps`: openverse-upstream_db-1	openverse-upstream_db	5432/tcp
================================================================================

Testing Instructions

Make the following change to ps.py:
diff --git a/utilities/ps.py b/utilities/ps.py
index cdf1d7a66..30ca50f1c 100644
--- a/utilities/ps.py
+++ b/utilities/ps.py
@@ -64,7 +64,7 @@ def parse_ps() -> list[Service]:
     lines = get_ps()
     lines = [stripped for line in lines if (stripped := line.strip())]
     for line in lines:
-        parts = line.split("\t")
+        parts = line.split("\t")[:1]
         if len(parts) != 3:
             raise ValueError(f"Unexpected line format from `docker ps`: {line}")
         service, image, ports = parts

Then run just up and observe an error similar to the above.
Checklist

[x] My pull request has a descriptive title (not a vague title likeUpdate index.md).
[x] My pull request targets the default branch of the repository (main) or a parent feature branch.
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[x] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no visible errors.
[ ] I ran the DAG documentation generator (if applicable).

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

I don't quite understand what the problem with the example output (ValueError: Unexpected line format from docker ps: openverse-upstream_db-1 openverse-upstream_db 5432/tcp) is. It would be nice if we could make the parsing more flexible to fix it, too, so that we wouldn't need to handle errors.

This was just my example from forcing the error, it's not an organic issue itself! I think the issue Sara was seeing was that the ps was happening so quickly after up that there weren't actually any services to report, and so ps wasn't able to parse the output properly. You're right, that example line is correct and we do parse that correctly. I'm not sure what the exact output Sara was encountering which cause the initial reported error, so I wouldn't be certain how best to adjust the parsing to compensate.

This is failing because of #2055, I think it's safe to merge nonetheless