Native environment not loaded with custom environment repository

Given the spring cloud implementation below, only properties from the custom repository implementation are loaded to the config client.
Only test.key property from the custom repository is available for the config clients.
Spring cloud - Greenwich-RELEASE
Spring boot - 2.1.2-RELEASE
Config server application.yml
server:
    port: 8888
spring:
    cloud:
        config:
            server:
                accept-empty: false
                native:
                    search-locations: classpath:/config
                    addLabelLocations: false
                    fail-on-error: true
                    order: 1
    profiles:
        active: native

Custom repository bean implementation
public class CustomEnvironmentRepository implements EnvironmentRepository, Ordered {

    @Override
    public Environment findOne(String application, String profile, String label) {
        String[] profiles = StringUtils.commaDelimitedListToStringArray(profile);
        Environment environment = new Environment(application,  profiles);

        Properties props = new Properties();
        props.setProperty("test.key", "value");

        environment.add(new PropertySource("application-dev", props));

        return environment;
    }

    @Override
    public int getOrder() {
        return Ordered.LOWEST_PRECEDENCE;
    }
}

The Workaround
Using a local git repository with 'basedir' and the custom repository implementation works fine and serves intended purpose to have config properties for all client applications in a single folder location and have a custom repository that loads and serves config properties from a SQL based datasource
See related StackOverflow issue also.
Using a composite environment did not solve this either. Please advise if this is a non-issue and recommend an appropriate configuration for native+custom environment repositories.

Why would you extend EnvironmentRepositoryConfiguration?

@spencergibb You're right, The EnvironmentRepositoryConfiguration extension should be removed. Edited snippet to reflect that. But issue is still same without it.

Can you provide a complete, minimal, verifiable sample that reproduces the problem?  It should be available as a GitHub (or similar) project or attached to this issue as a zip file.

Hi @spencergibb, please see attached mcve code for this issue.  config-client is unable to startup.

That's the intended behaviour. If there is already a bean of type EnvironmentRepository the auto configuration will step down so there won't be any NativeEnvironmentRepository bean.
Adding following to your custom configuration will resolve this:
@Bean
public NativeEnvironmentRepository nativeEnvironmentRepository(
		NativeEnvironmentRepositoryFactory factory,
		NativeEnvironmentProperties environmentProperties) {
	return factory.build(environmentProperties);
}

I tested this with Spring Boot 2.1.6.RELEASE and Spring Cloud Greenwich.SR2, so please use these versions too.


Analyzer hasReferenceToSuper is not always computed before referenced

Consider this code:
// test1.dart
abstract class Super {
  void foo();
}

abstract class Sub extends Super {
  void foo () {
    super.foo();
  }
}

and the following in another file:
import 'test1.dart';

abstract class C extends Super with Sub {}

void main() {

}

Without super mixins enabled, _checkForMixinInheritsNotFromObject in error_verifier.dart will report an error, but _checkForMixinReferencesSuper does not (even though the check is done).
With debug printing in place, I see that hasReferenceToSuper is checked on Sub in _checkForMixinReferencesSuper before Sub is visited, and is still false.  Later, Sub is visited by the TypeResolver, and hasReferenceToSuper is set.

Could you please clarify, which errors you expect when super mixins feature is: (1) enabled; (2) disabled? My current understanding of your comment is that you expect two errors when the feature is disabled; and zero when it is enabled. I tried to search the language specification PDF, but it does not talk anything about "super mixin".

If by the super mixins feature you mean the old class based super mixins, I have no expectations: it is being deleted.  :)
For the new feature, the relevant code is to replace the definition of Sub above with the following:
mixin Sub on Super {
  void foo () {
    super.foo();
  }
}

If I do that and test it with a recent analyzer build, I get a crash.  If I do that and test it with the CFE I get the following error (which is what I expect):
file:///Users/leafp/tmp/test1.dart:7:11: Error: Superclass has no method named 'foo'.
    super.foo();

OK, then I will stop looking at the original error, as I understand, it does not matter anymore.
What crash do you get? I see a problem that we don't report an error (and will fix it), but not a crash.

Haven't tried bleeding edge.  This is the crash I see from a recent (yesterday?) build:
Unhandled exception:
NoSuchMethodError: The getter 'element' was called on null.

With https://dart-review.googlesource.com/c/sdk/+/75783 we will produce the requested error.


Removed unused insertBefore overload

This PR removes the unused Prism.languages.insertBefore overload mentioned in #1505.
This PR resolves #1505.

Looks good; needs conflicts resolved :/


Bug fixes and extensibility enhancements

[UIImage imageNamed:] loads images from the main bundle, so when NMRangeSlider is used as a pod, its images fail to load. 312173b fixes this issue by loading images from the bundle NMRangeSlider is contained in instead of loading from the main bundle.
60f5c49 provides subclasses of NMRangeSlider with a couple more customization options.
32ec15b does as its title suggests

Awesome. Thanks


Update Faucet API Routes to Edge Runtime

This is because Cloudflare require the API routes
must be run on the edge runtime to be able to use
Read more: https://developers.cloudflare.com/pages/framework-guides/deploy-a-nextjs-site/#configure-the-application-to-use-the-edge-runtime
Summary of changes
Provide a detailed description of proposed changes.

Update faucet API routes to edge runtime.

Proposed area of change
Put an x in the boxes that apply.

[ ] apps/bridge-dapp
[ ] apps/hubble-stats
[ ] apps/stats-dapp
[ ] apps/webbsite
[x] apps/faucet
[ ] apps/tangle-website
[ ] libs/webb-ui-components

Reference issue to close (if applicable)
Specify any issues that can be closed from these changes (e.g. Closes #233).

Closes NaN

Deploy failed :/

The error was discovered to originate from the backend itself, not due to any issue with Cloudflare.
Downgraded to 13.4.19 because the netlify/next runtime does not support Next 13.5 very well. Read more here.


Worse accuracy with ONNX version (inferenced in C++ by OpenCV)

Hello,
I used your script "make_deploy_model.py" to create ONNX version of person segmentation models (for all three versions).
After I inferenced it using OpenCV:
cv::dnn::Net net = cv::dnn::readNetFromONNX("my_path/erf/model.onnx"); cv::Mat inpBlob = cv::dnn::blobFromImage(image, ...) //necessary blob image net.setInput(inpBlob); cv::Mat output = net.forward(); cv::Mat mask(H, W, CV_32F, output.ptr(0, 1));
and after some processes on obtained mask from inference otput based on your source code (necessary information like image_means for extract proper pixels I taked from cfg.yaml).
I would like to ask you if ONNX version can have worse accuracy or if I'm doing something wrong in postprocess step on obtained output from network? I was also thinking if maybe blob image step can cause some information loss.
Please let me know your opinion
sample results comparison:

Hi,
The network is not the only step taken to segment the image. Before the network you need to put the images in RGB (opencv uses BGR), and normalize the images. You can check the python script and check every step taken before the model inference happens. This happens in the User class if I am not mistaken. I am on my phone and can't check no

I suggest you use our C++ library in your project instead of opencv. It takes care of all this for you, and it uses tensor RT which makes this model run at 200fps when infering in int8

Thank you for your answer. I was trying to build your library follow instructions (by catkin) but I've got errors which I wasn't able to fix :(
I would like to exploit TensorRT library but I'm forced to run it on PC without GPU. Furthermore, I have to work on windows...
I will try to apply changes :)

I have another question... I'm also working on keypoint detection (for human pose estimation) and I've read that you're going add this algorithm to your library. I would like to ask you how is it going :) ?

Unfortunately, since we started working with lidar, this project became stagnant (some people in the lab are still adding tasks though). I will remove those placeholders now, since it probably won't happen in the near future


[Doc] Add site2/website/versioned_docs/version-2.3.1/getting-started-docker.md

Motivation
I've corrected the command in this file in #4121, the change does not effect in version 2.3.1 document, since I didn't move the updated file to the site2/website/versioned_docs/version-2.3.1 directory.
Modifications
Copy the updated file getting-started-docker.md to the site2/website/versioned_docs/version-2.3.1 directory.

run integration tests

run java8 tests

Run CPP Tests

@murong00 Thank you very much for your kinder reminder, I've update the file as per your advice.

run integration tests


/api/v1/apikeys not found return 500 instead of 404

st2 apikey get foobar return 500. Should it return 404 instead?

I'll dig into it, but it would be helpful if you provide a traceback when you encounter similar issue next time.

Here's the client side error.  vagrant@arkham:~/st2$ st2 apikey get foobar Traceback (most recent call last):   File "/home/vagrant/projects/stackstorm/st2/st2client/st2client/commands/resource.py", line 162, in get_resource_by_pk     instance = self.manager.get_by_id(pk, **kwargs)   File "/home/vagrant/projects/stackstorm/st2/st2client/st2client/models/core.py", line 39, in decorate     return func(*args, **kwargs)   File "/home/vagrant/projects/stackstorm/st2/st2client/st2client/models/core.py", line 204, in get_by_id     self.handle_error(response)   File "/home/vagrant/projects/stackstorm/st2/st2client/st2client/models/core.py", line 165, in handle_error     response.raise_for_status()   File "/usr/local/lib/python2.7/dist-packages/requests/models.py", line 862, in raise_for_status     raise HTTPError(http_error_msg, response=self) HTTPError: 500 Server Error: Internal Server Error Unable to retrieve detailed message from the HTTP response. Expecting value: line 1 column 1 (char 0)  for url: http://<IP_ADDRESS>:9101/v1/apikeys/foobar API Key "foobar" is not found.

and here's what the st2api.log says...
2017-04-20 19:06:52,743<PHONE_NUMBER>61744 INFO logging [-] 6757d1ac-b3fa-417f-9e1f-03d20a0a74d0 - GET /v1/apikeys/fooba with query={} (remote_addr='<IP_ADDRESS>',method='GET',request_id='6757d1ac-b3fa-417f-9e1f-03d20a0a74d0',query={},path='/v1/apikeys/foobar')
2017-04-20 19:06:52,749<PHONE_NUMBER>61744 ERROR auth [-] ('ApiKey matching %s for reference and id not found.', u'fooba')
ApiKeyNotFoundError: ApiKey with key or id=foobar not found.

Not able to reproduce:
st2 2.2.1
ubuntu@ansible-st2-ubuntu16:~$ st2 --debug apikey get foobar
2017-06-14 19:55:19,214  DEBUG - Using cached token from file "/home/ubuntu/.st2/token-testu"
# -------- begin<PHONE_NUMBER>86672 request ----------
curl -X GET -H  'Connection: keep-alive' -H  'Accept-Encoding: gzip, deflate' -H  'Accept: */*' -H  'User-Agent: python-requests/2.11.1' -H  'X-Auth-Token: 92fb784a9bf0462db72b097bbb9f7b81' http://<IP_ADDRESS>:9101/v1/apikeys/foobar
# -------- begin<PHONE_NUMBER>86672 response ----------
{
    "faultstring": "Unauthorized - ApiKey with key or id=foobar not found."
}
# -------- end<PHONE_NUMBER>86672 response ------------

Traceback (most recent call last):
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/st2client/commands/resource.py", line 162, in get_resource_by_pk
    instance = self.manager.get_by_id(pk, **kwargs)
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/st2client/models/core.py", line 39, in decorate
    return func(*args, **kwargs)
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/st2client/models/core.py", line 204, in get_by_id
    self.handle_error(response)
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/st2client/models/core.py", line 165, in handle_error
    response.raise_for_status()
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/requests/models.py", line 862, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
HTTPError: 401 Client Error: Unauthorized
MESSAGE: Unauthorized - ApiKey with key or id=foobar not found. for url: http://<IP_ADDRESS>:9101/v1/apikeys/foobar
ERROR: 401 Client Error: Unauthorized
MESSAGE: Unauthorized - ApiKey with key or id=foobar not found. for url: http://<IP_ADDRESS>:9101/v1/apikeys/foobar

CLI settings:
----------------
Config file path: /home/ubuntu/.st2/config
Client settings:
----------------
ST2_BASE_URL: http://<IP_ADDRESS>
ST2_AUTH_URL: http://<IP_ADDRESS>:9100
ST2_API_URL: http://<IP_ADDRESS>:9101/v1
ST2_AUTH_TOKEN: 92fb784a9bf0462db72b097bbb9f7b81

Proxy settings:
---------------
HTTP_PROXY:
HTTPS_PROXY:

Traceback (most recent call last):
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/st2client/shell.py", line 289, in run
    args.func(args)
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/st2client/commands/resource.py", line 324, in run_and_print
    instance = self.run(args, **kwargs)
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/st2client/commands/resource.py", line 44, in decorate
    return func(*args, **kwargs)
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/st2client/commands/resource.py", line 320, in run
    return self.get_resource_by_id(resource_id, **kwargs)
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/st2client/commands/resource.py", line 176, in get_resource_by_id
    instance = self.get_resource_by_pk(pk=id, **kwargs)
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/st2client/commands/resource.py", line 169, in get_resource_by_pk
    raise e
HTTPError: 401 Client Error: Unauthorized
MESSAGE: Unauthorized - ApiKey with key or id=foobar not found. for url: http://<IP_ADDRESS>:9101/v1/apikeys/foobar

ubuntu@ansible-st2-ubuntu16:~$ st2 apikey get foobar
Traceback (most recent call last):
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/st2client/commands/resource.py", line 162, in get_resource_by_pk
    instance = self.manager.get_by_id(pk, **kwargs)
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/st2client/models/core.py", line 39, in decorate
    return func(*args, **kwargs)
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/st2client/models/core.py", line 204, in get_by_id
    self.handle_error(response)
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/st2client/models/core.py", line 165, in handle_error
    response.raise_for_status()
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/requests/models.py", line 862, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
HTTPError: 401 Client Error: Unauthorized
MESSAGE: Unauthorized - ApiKey with key or id=foobar not found. for url: http://<IP_ADDRESS>:9101/v1/apikeys/foobar
ERROR: 401 Client Error: Unauthorized
MESSAGE: Unauthorized - ApiKey with key or id=foobar not found. for url: http://<IP_ADDRESS>:9101/v1/apikeys/foobar

ubuntu@ansible-st2-ubuntu16:~$ st2 --version
st2 2.2.1

st2 2.3dev
vagrant@u16:~$ st2 --debug apikey get foobar
2017-06-14 19:54:09,881  DEBUG - Using cached token from file "/home/vagrant/.st2/token-st2admin"
# -------- begin<PHONE_NUMBER>62768 request ----------
curl -X GET -H  'Connection: keep-alive' -H  'Accept-Encoding: gzip, deflate' -H  'Accept: */*' -H  'User-Agent: python-requests/2.14.2' -H  'X-Auth-Token: b8c7dd37c60744bdac6ec37e3df0cb51' http://<IP_ADDRESS>:9101/v1/apikeys/foobar
# -------- begin<PHONE_NUMBER>62768 response ----------
{
    "faultstring": "ApiKey matching foobar for reference and id not found."
}
# -------- end<PHONE_NUMBER>62768 response ------------

API Key "foobar" is not found.

CLI settings:
----------------
Config file path: /home/vagrant/.st2/config
Client settings:
----------------
ST2_BASE_URL: http://<IP_ADDRESS>
ST2_AUTH_URL: http://<IP_ADDRESS>:9100
ST2_API_URL: http://<IP_ADDRESS>:9101/v1
ST2_AUTH_TOKEN: b8c7dd37c60744bdac6ec37e3df0cb51

Proxy settings:
---------------
HTTP_PROXY:
HTTPS_PROXY:

Traceback (most recent call last):
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/st2client/shell.py", line 298, in run
    args.func(args)
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/st2client/commands/resource.py", line 331, in run_and_print
    raise OperationFailureException('Resource %s not found.' % resource_id)
OperationFailureException: Resource foobar not found.

vagrant@u16:~$ st2 apikey get foobar
API Key "foobar" is not found.

vagrant@u16:~$ st2 --version
st2 2.3dev

Closing for now, please open if reproducible.


Bot only gets latest 100 messages in each channel

Due to this little line using discord.client.logs_from standalone, it defaults to only fetching 100 messages. We need every message, so maybe iterate over history with the oldest read date in some form of temp variable?

Has this been fixed yet? Trying to find a way to download all of the chat for some channels. :/

Has this been fixed yet? Trying to find a way to download all of the chat for some channels. :/

@hook54321a I've been having some hardware issues these past few months. I managed to get everything up and running last week, but the holidays have left me with no time to work on code. x_x
I've been taking advantage of the peace and quiet of everyone being hung over from new year's celebrations, so I'll most likely push commits within the next 24 hours. ^_^

@hook54321a I've been having some hardware issues these past few months. I managed to get everything up and running last week, but the holidays have left me with no time to work on code. x_x
I've been taking advantage of the peace and quiet of everyone being hung over from new year's celebrations, so I'll most likely push commits within the next 24 hours. ^_^

The whole codebase has been rewritten from the ground up, thus this issue is irrelevant.


Wallet tab should display symbol name instead of token name for ENS and aUSDC

In hweeboon.eth. Seems like it should, according to #3184.

ENS
aUSDC

Can you help add test cases?

Does it need to be applied only for ens, and aUSDC tokens ?

Not only ENS definitely. Could you look at #3184 and see what's the criteria? The gist of it should be: if a token has a long name and short symbol, the symbol should be used instead. Maybe #3184 is lost?


batctl: 2018.0 -> 2018.1

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools.
This update was made based on information from https://repology.org/metapackage/batctl/versions.
These checks were done:

built on NixOS
ran ‘/nix/store/8djn6psqrxjc66chnig215c5z8il9zaw-batctl-2018.1/bin/batctl -h’ got 0 exit code
found 2018.1 with grep in /nix/store/8djn6psqrxjc66chnig215c5z8il9zaw-batctl-2018.1
directory tree listing: https://gist.github.com/13f600b1fb0404e8b5b78d0e2e32f5c5

cc @viric @fpletz for review

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: batctl
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on aarch64-linux (full log)
Attempted: batctl
Partial log (click to expand)
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/khmir7ln0j0n25i1nlwrzcrxb0g2als4-batctl-2018.1
shrinking /nix/store/khmir7ln0j0n25i1nlwrzcrxb0g2als4-batctl-2018.1/sbin/batctl
gzipping man pages under /nix/store/khmir7ln0j0n25i1nlwrzcrxb0g2als4-batctl-2018.1/share/man/
strip is /nix/store/j7d4mr0ikv974ig7yzhknpsq288js4bs-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/khmir7ln0j0n25i1nlwrzcrxb0g2als4-batctl-2018.1/sbin
patching script interpreter paths in /nix/store/khmir7ln0j0n25i1nlwrzcrxb0g2als4-batctl-2018.1
checking for references to /build in /nix/store/khmir7ln0j0n25i1nlwrzcrxb0g2als4-batctl-2018.1...
moving /nix/store/khmir7ln0j0n25i1nlwrzcrxb0g2als4-batctl-2018.1/sbin/* to /nix/store/khmir7ln0j0n25i1nlwrzcrxb0g2als4-batctl-2018.1/bin
/nix/store/khmir7ln0j0n25i1nlwrzcrxb0g2als4-batctl-2018.1

Success on x86_64-linux (full log)
Attempted: batctl
Partial log (click to expand)
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/8djn6psqrxjc66chnig215c5z8il9zaw-batctl-2018.1
shrinking /nix/store/8djn6psqrxjc66chnig215c5z8il9zaw-batctl-2018.1/sbin/batctl
gzipping man pages under /nix/store/8djn6psqrxjc66chnig215c5z8il9zaw-batctl-2018.1/share/man/
strip is /nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/8djn6psqrxjc66chnig215c5z8il9zaw-batctl-2018.1/sbin
patching script interpreter paths in /nix/store/8djn6psqrxjc66chnig215c5z8il9zaw-batctl-2018.1
checking for references to /build in /nix/store/8djn6psqrxjc66chnig215c5z8il9zaw-batctl-2018.1...
moving /nix/store/8djn6psqrxjc66chnig215c5z8il9zaw-batctl-2018.1/sbin/* to /nix/store/8djn6psqrxjc66chnig215c5z8il9zaw-batctl-2018.1/bin
/nix/store/8djn6psqrxjc66chnig215c5z8il9zaw-batctl-2018.1


Move JavaScript constants into their own file

This should allow us to rename main.js.erb to main.js in the future.

Coverage remained the same at 98.618% when pulling d22c6f28649b12c5d2eb9594d23aa40a67586448 on EvanHahn:constants-file into 3db99fe9cdd53f25fa4dcb167c621ca2e5b38a29 on sfbrigade:production.

Awesome, thank you! One step closer to sane JS.


fix: avatar url wrong url using external api

When using an external API service (like Gravatar) the avatar URL returned from /api/compat/wakatime/v1/users/current is wrong: http://localhost:3000/https://gravatar.com/avatar/3c90a086090c74081892efea2f523658.
This PR checks if the returned URL from User.AvatarURL() is an URL or a path.

In my previous PR (#553) I didn't know that using an external avatar service was a thing. Sorry 😅


open rows totalRow property

PR Details
Rows struct 开放私有属性totalRow
Description
Rows.Next()  totalRow>TotalRow
Rows() totalRow>TotalRow
Motivation and Context
因为我需要将读出来的数据写入到数据库，如果单条写入，速度太慢，所以需要批量写入。而批量写入的拆分条件就需要根据总量计算出来。
How Has This Been Tested

Types of changes

[ x] Docs change / refactoring / dependency upgrade
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[ x] My code follows the code style of this project.
[ x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Codecov Report

Merging #1054 (d101532) into master (e64775f) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #1054   +/-   ##
=======================================
  Coverage   97.70%   97.70%
=======================================
  Files          29       29
  Lines       17011    17019    +8
=======================================
+ Hits        16621    16629    +8
  Misses        267      267
  Partials      123      123

Flag
Coverage Δ

unittests
97.70% <100.00%> (+<0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

col.go
99.31% <100.00%> (+<0.01%)
:arrow_up:

rows.go
94.58% <100.00%> (+0.04%)
:arrow_up:

Continue to review full report at Codecov.

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

Hi @oneweek20169902, I've accepted this PR and introduced TotalCols, CurrentCol, TotalRows, and CurrentRow for the column and row's iterator in version v2.5.0. Since the performance optimization commit 4daa6ed, the internal variable totalRows was no longer available in the row's iterator, I refactor the row's iterator drop the pre-streamingly parsing for the worksheet for reducing the memory usage and time cost, so I made a breaking change: removed the exported function TotalRows. For consistent reason and following the minimum usable design principle, I decided to also remove another 3 functions: TotalCols, CurrentCol, CurrentRow, because developers could be easy to get total columns/rows by iterator without fetching every single column/row's cell value, and in the iterator progress, developers also could get the current column/row number by a simple counter, and the performance also similar with TotalCols/TotalRows in the same data scale. This breaking change will be released in the next version. What do you think about this, if you have any suggestions please let me know.


No default value parsed for optional positional argument

In the following Usage doc:
Usage:
    script cmd1 [<arg3>] [<arg4>] [--argument=<arg>]
Options:
  <arg4>        First argument [default: fubar]
  -h --help     Show this screen.
  --version     Show version.

When you parse: cmd1 C --argument=foo, the argument <arg4> does not get the default value, it stays at null.

Options: section does not support Argument (<arg4>). Thus, the [default: ] won't work.
docopt does not support default value for Argument either.

Hm… it looks like it's there but commented out, I guess I should look at older issues/PR to understand why this has been commented out:
https://github.com/docopt/docopt/blob/master/docopt.py#L564-568
which through a git blame led me to that commit fa0b81dd and as it has a reference to it, led me to the following issue's thread:
https://github.com/docopt/docopt/issues/64
I'm not sure how my use case fits in that picture, because I still get that Argument defaulting to null, so it's definitely recognized as optional, and whether it gets a default value or not, would still makes it greedy as shown on that issue.
I would love to get this feature straight, so I can keep my code simple with a single doc parser, until I change the design to make it use properly subparsers.


[BUG] [CRASH] [Bromite] Uncaught ReferenceError: Atomics is not defined

I got this error while browsing in Bromite version 85.0.4183.114 in Android 9 (Lineage OS + microG)

Browser Version: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/85.0.4183.114 Safari/537.36

Error Message : Uncaught ReferenceError: Atomics is not defined

Call Stack : undefined

Thanks for opening this issue, but we don't fully support mobile browsers yet. To support some of the underlying technologies need browser flags to be turned on manually.

One question tho did you manually go to app.modfy.video or by chance was this a tablet? because on mobile devices modfy.video(the landing page) shouldn't give you access to the application

I manually went there yes, by ticking the "desktop view" in the browser; I opened this issue as it was an error that I did not find anywhere, so that it was registered somewhere.

@Danoloan10 Yeah that makes sense, I will try to explore how to fix the Atomics error but wouldn't except a resolution anytime soon


Feature extraction for point clouds

Hi, thanks for your great work. I am trying to use your PonderV2 as a frozen feature extractor for new 3D point clouds, like how people use CLIP/DINOv2 for images.
Could you share some simple scripts or ideas about how we can achieve this with your codebase? This can be very helpful.

Hi, thanks for your interest!
You can refer to this function in Ponder model. It will return a sparse_backbone_feat tensor in the data dict. See here.
Alternatively, you can also refer to our downstream finetuning codes, where we load the backbone's weights to the SparseUNet.


Where is the log saved?

I can see that
            fabric.log_dict(
                {"iter": iter_num, "train_loss": loss, "step": step_count, "lr": lr}
            )
            fabric.print(
                    f"iter {iter_num}: loss {loss.item():.4f}, time: {dt*1000:.2f}ms, speed: {tokens_sec_str} toks/s/device"
            )

But I can't find the log. Should I set the direction the log will be saved? How can I set it?

Hi @ForcewithMe66 Right now we are mostly only printing things because the scripts are quite simple. For adding more detailed logging via TensorBoard for exmaple, you could add a logger to Fabric. Here is the docs for logging: https://lightning.ai/docs/fabric/stable/guide/logging.html
So basically, if you add
from lightning.fabric.loggers import TensorBoardLogger

logger = TensorBoardLogger(root_dir="logs")
fabric = Fabric(loggers=logger)

to the scripts you will find some extra files in the logs dir and can run tensorboard --logdir logs.

Thank you


HttpContext.User.Identity.IsAuthenticated is false

I have added a breakpoint in the line 122 at Login.cshtml.cs file right after the
await HttpContext.SignInAsync(
CookieAuthenticationDefaults.AuthenticationScheme,
new ClaimsPrincipal(claimsIdentity),
authProperties);
statement. I have also added the watch  HttpContext.User.Identity.IsAuthenticated and displays false. Is that normal?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: d2229fc6-f8c8-952e-e567-b1d5a463055a
Version Independent ID: b18a015b-914d-f014-f711-120d208904d9
Content: Use cookie authentication without ASP.NET Core Identity
Content Source: aspnetcore/security/authentication/cookie.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Hello! Thanks for contacting us. This sounds like a general question about using ASP.NET Core. While we try to look at and respond to all issues, for questions like this we recommend posting to a community support group like Stack Overflow with the asp.net-core tag.

I am having this exact issue too. Seems like the correct venue when your documentation projects are not functioning correctly.

If you think this is a product bug, create the simplest possible sample that reproduces the problem and open an issue at GitHub ASP.NET Core
Include steps to reproduce the bug and the version you're using dotnet --version

i have blazor server app on net8 with indetity
   builder.Services.AddScoped<AuthenticationStateProvider, IdentityRevalidatingAuthenticationStateProvider>();    builder.Services.AddAuthentication(options =>        {            options.DefaultScheme = IdentityConstants.ApplicationScheme;            options.DefaultSignInScheme = IdentityConstants.ExternalScheme;        })        .AddIdentityCookies();
i need custom authorize to controller, i add     builder.Services.AddAuthentication(CookieAuthenticationDefaults.AuthenticationScheme).AddCookie();
and do              await HttpContext.SignInAsync(           CookieAuthenticationDefaults.AuthenticationScheme,           new ClaimsPrincipal(claimsIdentity),           authProperties);
and cookie .AspNetCore.Cookies is set without any problem but "isAuthenticated":false.


Execution Time not working - kernel idle event not captured

I just put in a PR for this, but I don't know if it's considered nice to have a corresponding issue?
Here's what I changed:

CSS wasn't loading unless you had a very specific directory structure
The amount of time for cell to execute was never getting recorded, because it was listening for the 'status_idle.Kernel' event, which is probably an old version - doesn't exist in the current kernel.js
I made some slight simplifications by removing an event notification, and re-ordering the function call that records the beginning of execution to happen before the cell execution is triggered rather than after.  I believe in situations where execution is nearly immediate this could help with possible race conditions

Please let me know if there's something I can change - and thanks to those who provided these nice extensions!

Ah...I see that all of those changes are already in 3.0 branch!  I thought master followed 3.0 - my bad!  Closing...


Load gmaps lib before cartodb.js for public pages

Fixes #3074, using same gmaps lib include as on old public pages, https://github.com/CartoDB/cartodb/blob/master/app/views/admin/pages/public_dashboard.html.erb#L27

@xavijam ok?

Frontend tests were OK :+1: (details)

I wouldn't load gmaps script unless highlight map uses it. As we do in embed page:
...
<% if @visualization.map.provider == 'googlemaps' %>
  <script type="text/javascript" src="//maps.google.com/maps/api/js?sensor=false&v=3.12"></script>
<% end %>
...

Good point, fixed in f0f5930

:+1:

Frontend tests were OK :+1: (details)


Add Hiro Platform documentation

This issue is to create new documentation for the Hiro Platform. There are a number of different topics that need to be covered. These include:

How to deploy a contract in 5 minutes or less.
How to build, unit-test, debug, and deploy all from a web browser.
How to use and leverage pre-installed developer tools and a persistent environment.

Specific docs content:
Create

Contract examples
Contract cloning
Contract importing

Build

Single-sign-on authentication
Hosted and persistent development environment
Pre-installed developer tools: VS Code, Clarity Extension for VS Code, Git & Clarinet

Deploy

An intuitive UI for single and multi-contract deployments

Done


chore(): add typings, sourceMap (resolve #305, #335)

I added typings in previous commits, does not work well with webpack or @ngtools/webpack.

@jkuri Thank you for reviewing this pull request.
Please check again with sourceMap: true option in your project.
#335 ERROR in Unexpected value 'NgUploaderModule in ???/node_modules/ngx-uploader/lib/src/ngx-uploader/module/ngx-uploader.module.d.ts' imported by the module 'AppModule in XXX/src/app/app.module.ts'. Please add a @NgModule annotation.
In order for the compiler to know the location of the ngx-uploader.module.ts file, the ts file must be either in the declaration file or it must be declared via sourceMap.


squiggly-java not recognized after deploy war file tomcat

Hi,
When I use squiggly on development environment with embebed tomcat work as expected filtering fields. but when I build the war-file and add to external tomcat its not recognized and throw infinite recursion. Anyone know how to fix this?

squiggly version: 1.3.14
spring boot: 1.5.2

Tomcat 7 Java 8
Thank you

@damascenol Please take at look at examples/servlet.  Run mvn package and then unjar the squiggly.war to the ROOT webapp directory of your Tomcat instance .  Then follow the README.md in that directory to make curl requests to test it out.  You can also test in the browser, but remember to url encode the square brackets in the fields query parameter.
If you are still having trouble, please attach a war exhibiting the problem with instructions how to replicate.
Thanks!
~Ryan

Thanks, I can make it work now!
This is a great library.


How to set Heap size

my Extension is a debugger which implement DAP protocal. It loads a lot of data into memory at startup, which triggers javascript heap out of memory error .
For this kind of error, we can usually solve it by adjusting the heap size --max_old_space_size=4096. But I didn't find any way to set this parameter for extension.
I found a type question but no fresh answer  #10149.
So how to set Heap size for my extension to avoid javascript heap out of memory error .

If your DAP implementation runs directly in an extension, then you share the extension host (node.js) with all other extensions und you cannot control things like heap size.
In order to work around this, I recommend to implement DAP in a separate process that you launch from the extension itself. In this case you in control of the node.js process and can pass whatever arguments you want.


Update os-ot-favor-small-code-favor-fast-code.md

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

check minor format fix


Add CI Static Analsysis: CPP Check, Inclusion Guards

Codecov Report

Merging #121 (996e697) into master (8500e7f) will not change coverage.
The diff coverage is 75.000%.

:exclamation: Current head 996e697 differs from pull request most recent head d8df4f5. Consider uploading reports for the commit d8df4f5 to get more accurate results

@@            Coverage Diff            @@
##            master      #121   +/-   ##
=========================================
  Coverage   89.041%   89.041%
=========================================
  Files           26        26
  Lines         1752      1752
=========================================
  Hits          1560      1560
  Misses         192       192

Impacted Files
Coverage Δ

src/adiar/adiar.cpp
91.667% <ø> (ø)

src/adiar/bdd/apply.cpp
91.781% <ø> (ø)

src/adiar/bdd/assignment.cpp
100.000% <ø> (ø)

src/adiar/bdd/build.cpp
100.000% <ø> (ø)

src/adiar/bdd/count.cpp
100.000% <ø> (ø)

src/adiar/bdd/evaluate.cpp
100.000% <ø> (ø)

src/adiar/bdd/if_then_else.cpp
96.748% <ø> (ø)

src/adiar/bdd/negate.cpp
100.000% <ø> (ø)

src/adiar/bdd/quantify.cpp
100.000% <ø> (ø)

src/adiar/bdd/restrict.cpp
96.667% <ø> (ø)

... and 11 more

Continue to review full report at Codecov.

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


Limit length of repo description and repo url input fields

Both allow only 255 characters. If you input more, you will get a error message. So it make sense to limit the characters of the input fields to 255.

That doesn't look to me as if we limit it to 255 chars.
If we do throw an error after 256 chars, then we should probably investigate and fix that error beforehand.

Would be nice to add a screenshot on how it looks like if you put to much chars into it

That doesn't look to me as if we limit it to 255 chars. If we do throw an error after 256 chars, then we should probably investigate and fix that error beforehand.

IIRC, XORM has a "feature" that it implies to use VARCHAR(255) if there is no data-type in the xorm: tag.
So, adding maxlength could be good here.
(off-topic: is varchar(255) too short for the description field?)

(off-topic: is varchar(255) too short for the description field?)

I would say yes

According to my test, 8-line description is around 1024 chars, it should satisfy most cases. (see the screenshot)

@JakobDev
Could you also try this patch? It relaxes the repo Description MaxSize to 2048, Website MaxSize to 1024.
Then the form can have maxlength=2048/1024 accordingly.
These limitations should be good for most cases.
diff --git a/modules/structs/repo.go b/modules/structs/repo.go
index 6a5736898..c7358372c 100644
--- a/modules/structs/repo.go
+++ b/modules/structs/repo.go
@@ -111,7 +111,7 @@ type CreateRepoOption struct {
        // unique: true
        Name string `json:"name" binding:"Required;AlphaDashDot;MaxSize(100)"`
        // Description of the repository to create
-       Description string `json:"description" binding:"MaxSize(255)"`
+       Description string `json:"description" binding:"MaxSize(4096)"`
        // Whether the repository is private
        Private bool `json:"private"`
        // Label-Set to use
@@ -140,9 +140,9 @@ type EditRepoOption struct {
        // unique: true
        Name *string `json:"name,omitempty" binding:"OmitEmpty;AlphaDashDot;MaxSize(100);"`
        // a short description of the repository.
-       Description *string `json:"description,omitempty" binding:"MaxSize(255)"`
+       Description *string `json:"description,omitempty" binding:"MaxSize(4096)"`
        // a URL with more information about the repository.
-       Website *string `json:"website,omitempty" binding:"MaxSize(255)"`
+       Website *string `json:"website,omitempty" binding:"MaxSize(2048)"`
        // either `true` to make the repository private or `false` to make it public.
        // Note: you will get a 422 error if the organization restricts changing repository visibility to organization
        // owners and a non-owner tries to change the value of private.
@@ -208,7 +208,7 @@ type GenerateRepoOption struct {
        // Default branch of the new repository
        DefaultBranch string `json:"default_branch"`
        // Description of the repository to create
-       Description string `json:"description" binding:"MaxSize(255)"`
+       Description string `json:"description" binding:"MaxSize(4096)"`
        // Whether the repository is private
        Private bool `json:"private"`
        // include git content of default branch in template repo
@@ -316,7 +316,7 @@ type MigrateRepoOptions struct {
        LFS            bool   `json:"lfs"`
        LFSEndpoint    string `json:"lfs_endpoint"`
        Private        bool   `json:"private"`
-       Description    string `json:"description" binding:"MaxSize(255)"`
+       Description    string `json:"description" binding:"MaxSize(4096)"`
        Wiki           bool   `json:"wiki"`
        Milestones     bool   `json:"milestones"`
        Labels         bool   `json:"labels"`
diff --git a/services/forms/repo_form.go b/services/forms/repo_form.go
index 4eb20d297..d8e6294bb 100644
--- a/services/forms/repo_form.go
+++ b/services/forms/repo_form.go
@@ -34,7 +34,7 @@ type CreateRepoForm struct {
        UID           int64  `binding:"Required"`
        RepoName      string `binding:"Required;AlphaDashDot;MaxSize(100)"`
        Private       bool
-       Description   string `binding:"MaxSize(255)"`
+       Description   string `binding:"MaxSize(4096)"`
        DefaultBranch string `binding:"GitRefName;MaxSize(100)"`
        AutoInit      bool
        Gitignores    string
@@ -76,7 +76,7 @@ type MigrateRepoForm struct {
        LFS            bool   `json:"lfs"`
        LFSEndpoint    string `json:"lfs_endpoint"`
        Private        bool   `json:"private"`
-       Description    string `json:"description" binding:"MaxSize(255)"`
+       Description    string `json:"description" binding:"MaxSize(4096)"`
        Wiki           bool   `json:"wiki"`
        Milestones     bool   `json:"milestones"`
        Labels         bool   `json:"labels"`
@@ -116,8 +116,8 @@ func ParseRemoteAddr(remoteAddr, authUsername, authPassword string) (string, err
 // RepoSettingForm form for changing repository settings
 type RepoSettingForm struct {
        RepoName               string `binding:"Required;AlphaDashDot;MaxSize(100)"`
-       Description            string `binding:"MaxSize(255)"`
-       Website                string `binding:"ValidUrl;MaxSize(255)"`
+       Description            string `binding:"MaxSize(4096)"`
+       Website                string `binding:"ValidUrl;MaxSize(2048)"`
        Interval               string
        MirrorAddress          string
        MirrorUsername         string

@wxiaoguang I have now applied the patch

Let's try if it works well ~


default for TypeVar is not working consistently with type inference for generic class

 Environment data

Language Server version: 2024.9.2
OS and version: win32 x64
Python version (and distribution if applicable, e.g. Anaconda):
python.analysis.indexing: true
python.analysis.typeCheckingMode: basic

 Code Snippet

from dataclasses import dataclass, field
from typing import Generic, Literal, assert_type, overload, reveal_type

from typing_extensions import TypeVar

FlagA = TypeVar("FlagA", Literal[True], Literal[False], default=Literal[False])

@dataclass
class Config(Generic[FlagA]):
    flag_a: FlagA = field(default=False)

class HasFeatureA(Config[FlagA], Generic[FlagA]):
    pass

T = TypeVar("T", Config[Literal[True]], Config[Literal[False]], contravariant=True)

@dataclass
class A(Generic[T]):
    config: T

    @overload
    def f(self: "A[HasFeatureA[Literal[True]]]") -> Literal[1]: ...
    @overload
    def f(self: "A[HasFeatureA[Literal[False]]]") -> Literal[2]: ...

    def f(self) -> int: ...

a1 = A(Config(flag_a=True))
assert_type(a1.f(), Literal[1])

a2 = A(config=Config(flag_a=False))
assert_type(a2.f(), Literal[2])

default_config = Config()
a3 = A(config=default_config)
assert_type(a3.f(), Literal[2])

a4 = A(config=Config())
assert_type(a4.f(), Literal[2])  # "assert_type" mismatch: expected "Literal[2]" but received "Literal[1]"
reveal_type(a4)  # A[Config[Literal[True]]]

 Expected behavior
All four calls to assert_type should not raise an error. Specifically, a3 and a4 should both be of type A[Config[Literal[False]]].
 Actual behavior
a4 is inferred as A[Config[Literal[True]]], causing assert_type to fail.
Workarounds
Remove constraints of T
This way all the assert_type works, but we lose the type hint for A.config.
Use bound instead of constraints on T
Does not work because bounds cannot be generic, which means the upper bound of T becomes Config[Literal[False]].
Please let me know if there are other working workarounds, thanks.

Could someone from the pylance team please transfer this to the pyright project, since it's a core type checker issue? Thanks!

@kc0506, there is a pyright bug here, but it's different than what you're reporting.
The issue here is that your code has a bug that is not being reported by pyright currently (a false negative). The statement flag_a: FlagA = field(default=False) should result in a type error because a value of False is not assignable to a value with type FlagA. This assignment will lead to unsoundness if you attempt to do something like Config[Literal[True]](). It also leads to the other seemingly-inconsistent behavior that you're seeing here.
It's important to keep in mind that a TypeVar's default value is used only in cases where a TypeVar is otherwise unsolved. If you have the expression x: Config[Literal[True]] = Config(), the type variable is solved (because it's provided by the context of the type annotation), so the TypeVar will not receive its default value in this case.
This false negative will be addressed in the next release of pyright.
You'll may need to rethink your approach here. I don't think that value-constrained type variables and TypeVar defaults are the appropriate mechanisms for what you're trying to accomplish here.


[Enhancement] Make it not spam channels

Checks

[X] I confirm that I have searched for existing issues before reporting to avoid duplicate reporting.

Description
When you run a command like /carpet, the command will output a big multiline response. When you have the console logging on, it will spam that channel with messages line by line, which will result in the bot getting rate limited by discord. It would be better if it would put all the lines into one message so the bot wouldn't get rate limited. This would also be nice for chat messages in general since a player can get the bot rate limited by spamming the chat a few times.

Duplicate of #3
In fact, MCDC already has anti-spam, and can only send 20 messages every 20 seconds (including console log and command execution, excluding chat).
When you use a mod such as Litematica that executes thousands of commands per second, you'll find the existence of anti-rate limit. MCDC should not be able to reach the rate limit?

Oops! I'm sorry for the misunderstanding.
You meant execution in the console, while #3 meant execution in the game. Strictly speaking, this issue is not a duplicate. However, anti-spam was also implemented for the console execution at that time, as I mentioned in the previous comment.
It's worth noting that DMCC (MCDC) has the implementation of the console log channel re-written in #72. Now, console log messages are no longer sent individually, but in groups.


The model is in the train mode in demo

Hello!
Why is the model put into train mode in the demo?
https://github.com/dennisbappert/u-2-net-portrait/blob/master/demo.py#L78
The difference between train() and eval() is huge actually.

Good catch, this is actually a bug. I will fix it, thanks for raising.

Fixed.


When doing suggestions for symbols, it would be useful to show the path relative to the workspace

In this situation, it would be a lot easier to show the path relative to the workspace root instead of the relative path because I often move from file to file and I never really know what folder I'm in. The relative paths don't help me with picking the correct symbol. This could be a setting if not everyone would find it useful.

This is what I'm seeing in the server log - fileName is already there alongside moduleSpecifier
<EMAIL_ADDRESS>

@mjbvz Given Ryan's latest answer, is this something that could be implemented in the vscode extension?


Support decompiling enums inside of namespaces

Fixes the issue where the "stop all animations" block in arcade doesn't decompile. I couldn't find an issue for it.

Many extensions in micro:bit have nested enums, this is great!

This fix should go to microbit branch @abchatra

Only point fixes with clear bugs should go to stable branches. Otherwise chance of regression are high. Why should this go to microbit?


Then i add --action_env PYTHON_BIN_PATH=/usr/local/bin/python   according other issue's solution.
bazel build -c opt --strip=ALWAYS \
--host_crosstool_top=@bazel_tools//tools/cpp:toolchain \
--fat_apk_cpu=arm64-v8a \
--spawn_strategy=standalone \
--action_env PYTHON_BIN_PATH=/usr/local/bin/python \
//mediapipe/examples/android/src/java/com/google/mediapipe/apps/faceeffect:faceeffect

Another, macOS 12.3 not support python2.7 officially, is it relative?

Reference:
https://github.com/tensorflow/tensorflow/issues/15618#issuecomment-376009820
https://github.com/tensorflow/tensorflow/issues/10289#issuecomment-868247016
macOS 12.3 not support add link to /usr/bin/ ,  even if diable System Integrity Protection(SIP).

Solved , because bazel use python3, not python2.

Still build failed .
ERROR: /Users/zhengying/Downloads/mediapipe-0.8.9/mediapipe/java/com/google/mediapipe/framework/BUILD:34:16: Validating proguard configuration failed: (Exit 127): proguard_whitelister failed: error executing command bazel-out/darwin-opt-exec-50AE0418/bin/external/bazel_tools/tools/jdk/proguard_whitelister --path mediapipe/java/com/google/mediapipe/framework/proguard.pgcfg --output ... (remaining 1 argument(s) skipped)
env: python: No such file or directory
Target //mediapipe/examples/android/src/java/com/google/mediapipe/apps/faceeffect:faceeffect failed to build
Use --verbose_failures to see the command lines of failed build steps.
INFO: Elapsed time: 820.192s, Critical Path: 359.98s
INFO: 3732 processes: 434 internal, 3298 local.
FAILED: Build did NOT complete successfully

Before update macOS to version 12.3, there is no this problem. Is this  because /usr/bin/ don't have python, just have python3?
Build command:
zhengying@wingdideMacBook-Pro mediapipe-0.8.9 % bazel build -c opt --strip=ALWAYS \
--host_crosstool_top=@bazel_tools//tools/cpp:toolchain \
--fat_apk_cpu=arm64-v8a --spawn_strategy=standalone \
//mediapipe/examples/android/src/java/com/google/mediapipe/apps/faceeffect:faceeffect

Hi @wingdi ,
Could you provide steps to reproduce this issue.

I'm experiencing the same issue trying to build the handtracking example in ios, which was working fine before upgrading to macOS 12.3
Platform: macOS 12.3
Bazel version: 3.7.2 (also tried 5.0.0 with no luck)
Mediapipe Version: 0.8.9
I agree the issue relates to removal of python 2 in macOS 12.3, and the fact that there's no python in /usr/bin. The following two attempted build commands resulting in the following failure, which is identical for both commands:
bazel build -c opt --config=ios_arm64 mediapipe/examples/ios/handtrackinggpu:HandTrackingGpuApp

bazel build -c opt --config=ios_arm64 --action_env PYTHON_BIN_PATH=$(which python3) mediapipe/examples/ios/handtrackinggpu:HandTrackingGpuApp

ERROR: An error occurred during the fetch of repository 'local_execution_config_python':
   Traceback (most recent call last):
	File "/private/var/tmp/_bazel_jory/f9f0f6bca00c121e1c40b0977e1c780a/external/org_tensorflow/third_party/py/python_configure.bzl", line 212, column 41, in _create_local_python_repository
		python_include = _get_python_include(repository_ctx, python_bin)
	File "/private/var/tmp/_bazel_jory/f9f0f6bca00c121e1c40b0977e1c780a/external/org_tensorflow/third_party/py/python_configure.bzl", line 152, column 21, in _get_python_include
		result = execute(
	File "/private/var/tmp/_bazel_jory/f9f0f6bca00c121e1c40b0977e1c780a/external/org_tensorflow/third_party/remote_config/common.bzl", line 230, column 13, in execute
		fail(
Error in fail: Problem getting python include path.
<string>:1: DeprecationWarning: The distutils package is deprecated and slated for removal in Python 3.12. Use setuptools or check PEP 632 for potential alternatives
<string>:1: DeprecationWarning: The distutils.sysconfig module is deprecated, use sysconfig instead
Is the Python binary path set up right? (See ./configure or PYTHON_BIN_PATH.) Is distutils installed?
ERROR: Analysis of target '//mediapipe/examples/ios/handtrackinggpu:HandTrackingGpuApp' failed; build aborted: Problem getting python include path.
<string>:1: DeprecationWarning: The distutils package is deprecated and slated for removal in Python 3.12. Use setuptools or check PEP 632 for potential alternatives
<string>:1: DeprecationWarning: The distutils.sysconfig module is deprecated, use sysconfig instead
Is the Python binary path set up right? (See ./configure or PYTHON_BIN_PATH.) Is distutils installed?
INFO: Elapsed time: 3.406s
INFO: 0 processes.
FAILED: Build did NOT complete successfully (39 packages loaded, 179 targets configured)

As an attempted fix, I installed python 2 manually from https://www.python.org/downloads/release/python-2718/. Separately, I tried setting action-env to the python3 that's in /usr/bin. In those cases, the following two build commands result in the following identical failure:
bazel build -c opt --config=ios_arm64 --action_env PYTHON_BIN_PATH=/usr/local/bin/python --verbose_failures mediapipe/examples/ios/handtrackinggpu:HandTrackingGpuApp

bazel build -c opt --config=ios_arm64 --action_env PYTHON_BIN_PATH=/usr/bin/python3 --verbose_failures mediapipe/examples/ios/handtrackinggpu:HandTrackingGpuApp

INFO: Analyzed target //mediapipe/examples/ios/handtrackinggpu:HandTrackingGpuApp (168 packages loaded, 10586 targets configured).
INFO: Found 1 target...
ERROR: /Users/jory/mediapipe/mediapipe/examples/ios/handtrackinggpu/BUILD:34:16: StoryboardCompile mediapipe/examples/ios/handtrackinggpu/HandTrackingGpuApp-intermediates/Base.lproj_storyboardc/Main.storyboardc failed (Exit 127): xctoolrunner failed: error executing command
  (cd /private/var/tmp/_bazel_jory/f9f0f6bca00c121e1c40b0977e1c780a/execroot/mediapipe && \
  exec env - \
    APPLE_SDK_PLATFORM=iPhoneOS \
    APPLE_SDK_VERSION_OVERRIDE=15.4 \
    XCODE_VERSION_OVERRIDE=<IP_ADDRESS>E5104i \
  bazel-out/host/bin/external/build_bazel_rules_apple/tools/xctoolrunner/xctoolrunner ibtool --compilation-directory '[ABSOLUTE]bazel-out/ios_arm64-opt/bin/mediapipe/examples/ios/handtrackinggpu/HandTrackingGpuApp-intermediates/Base.lproj_storyboardc' --minimum-deployment-target 10.0 --target-device iphone --target-device ipad --module HandTrackingGpuApp '[ABSOLUTE]mediapipe/examples/ios/common/Base.lproj/Main.storyboard')
Execution platform: @local_execution_config_platform//:platform
env: python: No such file or directory
Target //mediapipe/examples/ios/handtrackinggpu:HandTrackingGpuApp failed to build
INFO: Elapsed time: 4.168s, Critical Path: 0.04s
INFO: 6 processes: 6 internal.
FAILED: Build did NOT complete successfully

it's a bazel problem .
In bazel-out/darwin-opt-exec-50AE0418/bin/external/bazel_tools/tools/jdk/proguard_whitelister file, the first line is #!/usr/bin/env python , this error come from here.
Then i found this in bazel :

So maybe bazel 6.0.0-pre.20220310.1 work.
I need go to sleep now.

Problem solved.
Use bazel 5.0.0,  don't use bazel 4.2.1


Dd 1302

Fixes DD-1302
Description of changes
How to test
Related PRs
(Add links)

Notify
@DANS-KNAW/dataversedans

Just need to add some tests to verify behaviours


Update sql-database-designing-cloud-solutions-for-disaster-recovery.md

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

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

TTYL, AZPRBOT;


Optimize printing of integers

i = 0
println i

becomes
i=0
printf("%s", i)
printf("%s", \"\n\")

where it should be
i=0
printf("%s\n", i)

Similar for all other primitives.


deployment based on api stages implemented

updated PR of https://github.com/Miserlou/Zappa/pull/663
fixed #628 #254

I'd really like to see this functionality!

@anush0247 I support your effort here. I'll be reviewing your changes. Alas I am new to this codebase. Please get in contact if I can be of assistance.

@PeteW I would like to help you with your queries, in a week I will resolve all the conflicts and update pr


0.1.2 missing dist folder

The 0.1.2 version published on npm is missing a dist folder. Have to pin to 0.1.1 for the moment.

ping @diragb, can you release a new version with the dist folder?

sorry for the delay @staaky & @dan-lee, this issue has been fixed along with a TS inference issue


Link to echo guide is broken

The link to the Echo guide in README.md is broken. Right now, it points to https://docs.sentry.io/platforms/go/echo/, but that URL 404s. Instead, it should point to https://docs.sentry.io/platforms/go/guides/echo/.

This was fixed in #290.


[jnigen] Implement void interface methods as listeners

Closes #1568.
I converted all the void callbacks to instead be FutureOr<void>, then I check if they're Future<void> and if so, implement them as listeners.
There are some other considerations when checking the types that is explained in a comment in dart_generator.dart / _InterfaceIfAsyncMethod class.

coverage: 90.87% (-0.02%) from 90.886%
when pulling a2a5dc4df36390c15638cb4da89e96e4fef65848 on implement-as-listener
into 117cc75553e1c597b0b8c19a4d8bd9d94560e645 on main.

Drastically changing the behavior of the method based on the return type of the function like this might be too much black-box magic.

+1 on this.

Ended up using booleans. Adding more types of callback like temporary isolate will require a breaking change anyways as in that case I prefer JCallable.xyz constructors instead of enum-like values as they're no longer limited to void returning methods and they might require special lints (maybe using a pragma) to warn users against certain patterns due to the limitations of temporary isolate mode.
PTAL @liamappelbe @dcharkes


New release?

👋
Latest release happened some time and some fixed issues ago (I'm looking at you, https://github.com/buildkite/buildkite-agent-metrics/pull/94). Any chance of cutting a new one from master?
Thanks! 🙇

🎉 Thanks!


use UART2 on wemos lolin D32 pro

Hi Jan,
any chance we can use your lib with another UART ? i've tried playing with console.begin
//void Console::begin(int baud, int rxPin, int txPin, uint8_t channel)
console.begin(115200, RX2, TX2, 0);
but it's not working so far. At best, the console answer on uart0 via USB cable. Or my board reboot every 1 s when I'm change channel to anything else than 0
I saw that rxPin or txPin are not use in begin(...) function. So I wonder if it's possible to use another UART

You are right, the txPin and rxPin option were never used (and actually there were another bug which prevented the use of another UART channel).
With Version 1.2.1 you should be able to configure the rxPin and txPin properly and even using other UART channels, which allows you to use ESP32Console parallel to Serial.println on its own UART channel (see other_uart_channel example).


Add CPPCheck

Knowing you probably are using Jenkins you could easily add static code analysis with CPPCheck to your code.
Visualizations:
https://wiki.jenkins-ci.org/display/JENKINS/Cppcheck+Plugin
CPP:
http://cppcheck.sourceforge.net/
With all checks enabled I got the following performance results:

We will look into this, thank you. After getting virtual box up and running.


[Datagrid] Manually updating clrDgPageSize no longer results in clrDgRefresh being triggered

[X] bug
[ ] feature request
[ ] enhancement

Expected behavior
Manually updating the clrDgPageSize should trigger a clrDgRefresh event to be fired on the table (this was how it worked in Clarity v0.9.x).
Actual behavior
Manually updating clrDgPageSize no longer triggers a clrDgRefresh event.
Reproduction of behavior
Here is a working version (v0.9.x): https://plnkr.co/edit/7PBBUqnPSopd21NCyfxU?p=preview
And here is the same code running in v0.10.x where the event is no longer fired: https://plnkr.co/edit/cl9WfX76lYlbVTh6nm2u?p=preview
Environment details

Angular version: 4.x.x

Clarity version:

OS and version: MacOS Sierra

Browser: Chrome 62.0.3202.94 (Official Build) (64-bit)

@jbgarr Have you fixed it?
I met the same issue and use "clrDgStateChange" as mentioned in #1099 but do not work.

We have the same problem. The problem seems to be that the Page service does not trigger the StateDebouncer inside the size setter. It only does this for changes of the current page.

@m-tomczyk
This is on our radar but we do not currently have it assigned to anyone to be fixed. If it is a potential blocker for your team, we are certainly open to contributions.

Btw, plunkr reproduction no longer working.
Attaching the stackblitz one I did while researching the issue, if needed for future reference:
https://stackblitz.com/edit/clarity-issue-1752

Fixed in 0.13 and 1.0
Thanks, @stefnoten and @hippee-lee for the issue research. Very helpful!


Mocking: Return List

As a user I want the API to return a list of examples so I can mock a list endpoint.
List will take the example array and return it to the user.
  x-kusk:
    mocking:
      enabled: true
      method: list

Note this would need to be used with example as an array and not examples.
components:
  schemas:
    ArrayOfUsers:
      type: array
      items:
        type: object
        properties:
          id:
            type: integer
          name:
            type: string
      example:
        - id: 10
          name: Jessica Smith
        - id: 20
          name: Ron Stewart

Would be solved by https://github.com/kubeshop/kusk-gateway/issues/324

Duplicate


No module named 'sklearn.linear_model.logistic'

C:\Users\ADMIN\Desktop\Fake News\Fake_News_Detection\prediction.py:19: DeprecationWarning: Please use `csr_matrix` from the `scipy.sparse` namespace, the `scipy.sparse.csr` namespace is deprecated.
  load_model = pickle.load(open('final_model.sav', 'rb'))
C:\Users\ADMIN\Desktop\Fake News\fake\lib\site-packages\sklearn\base.py:329: UserWarning: Trying to unpickle estimator TfidfTransformer from version 0.18.1 when using version 1.0.2. This might lead to breaking code or invalid results. Use at your own risk. For more info please refer to:
https://scikit-learn.org/stable/modules/model_persistence.html#security-maintainability-limitations
  warnings.warn(
C:\Users\ADMIN\Desktop\Fake News\fake\lib\site-packages\sklearn\base.py:329: UserWarning: Trying to unpickle estimator TfidfVectorizer from version 0.18.1 when using version 1.0.2. This might lead to breaking code or invalid results. Use at your own risk. For more info please refer to:
https://scikit-learn.org/stable/modules/model_persistence.html#security-maintainability-limitations
  warnings.warn(
Traceback (most recent call last):
  File "C:\Users\ADMIN\Desktop\Fake News\Fake_News_Detection\prediction.py", line 28, in <module>
    detecting_fake_news(var)
  File "C:\Users\ADMIN\Desktop\Fake News\Fake_News_Detection\prediction.py", line 19, in detecting_fake_news
    load_model = pickle.load(open('final_model.sav', 'rb'))
ModuleNotFoundError: No module named 'sklearn.linear_model.logistic'

I also get the same error. If the version of scikit-learn is higher than 0.24.0, this err would occur. In my test, if using scikit-learn=0.24.0, there would be no err. Then I turned to scikit-learn=0.24.1, the err happened.

yeah version issue.

Yeah version issue


Chapter 10, Exercise 09 + restatement of Exercise 02

changes pushed

Can you fix the conflict?

Thanks!


Rename devenv processes stop to devenv processes down

For clarity / compatibility with simliar usage in Docker Compose.

Leave devenv processes stop as an alias.
Add devenv down as an alias (maybe devenv stop too?)

I'm totally down for this.

I'd be happy to help, but for the life of me can't find where the Python CLI bits live in the repo.

You need the rewrite branch: https://github.com/cachix/devenv/blob/python-rewrite/src/devenv/cli.py

In 1.0.2
❯ devenv processes
Start or stop processes.

Usage: devenv processes <COMMAND>

Commands:
  up
  down
  help  Print this message or the help of the given subcommand(s)

Options:
  -h, --help  Print help


[GURPS] 2.23.1

Submission Checklist
Required

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

Changes / Description

BUGFIX: A couple of translation updates and fixing a page reference for Aim. Suggested by Spriteanon and Uncle Mojo.
BUGFIX: When adding a brand new spell, not all of the modifiers were correctly added. This has now been fixed.

Character Sheet Info Roll20 Internal Use only.


Can't add myself user to issue.

There is noassigne yourself button. What should I do?
the link: https://github.com/RespiraWorks/Ventilator/issues/1198

If member is not yet invited, they cannot assign themselves to a ticket.  New users will need to request to an admin to be invited first.  Is this the intended behavior for the repo?

checked with Martin, this is the desired setting.


configure automated deployment

Do deployment on every commit (snapshot) using travis infrastructure
https://docs.travis-ci.com/user/deployment/

done


gamescope: 3.14.24 -> 3.14.26

Description of changes
https://github.com/ValveSoftware/gamescope/compare/3.14.24...3.14.26

added new dependency lcms

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

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

4 packages built:

bottles
bottles-unwrapped
gamescope
gamescope-wsi


[REVIEW]: GSAreport: Easy to Use Global Sensitivity Reporting

Hi @Basvanstein and sorry for the late review,
Here are few suggestions and requests,

[x] Numpy version conflict between gsareport and shap (Numpy version should be < = 1.20), may need to update requirements.txt at the src.
[x] Readme is too informative. The "installation instructions" and the " usage" sections are the same in github readme and in the documentation. It would be more convenient to follow the readme if it contains the highly recommended installation/usage and hyperlinks(documentation) for the others, including the 'Building binaries (for developers)' section.
[x] How about some example analysis images/gifs (e.g. headmaps, correlation analysis) in the introduction section of the readme.
[x] References in the paper and in the readme have different citation styles.  Citation syntax
[x] It is not required, but I suggest test coverage in the actions. E.g. the plot functions are not included in the automated test.

Let me know what you think of these suggestions.

Thank you for the comments!
I will check and update the numpy conflict, we can just take the shap version and remove it in the requirements.
Good point about the readme, will clean it up a bit and will add some example images.
I used the citation syntax that the packages themselves provided, the paper uses a bibtex file, so I guess the style there is standardized no? Will check and update.
Plot functions are not that useful to automatically test I think, running the demo and checking the output basically shows that they are all working. In the tests I could check if the files exist but I don't think that it adds much. The rest of the functions is covered.

I have updated the readme and made it more clear for developers, pointing to the documentation to avoid duplication.
I have fixed the paper bibtex and citation style and resolved the numpy version conflict.
Hope you like it :)

Indeed, well spotted. Moved the References section to the end and called that section Related software.


bootstrapping Aurelia app routing from within an ASP.NET MVC route

I've setup ASP.NET 5 MVC to route to http://localhost:12345**/app** that serves as my Aurelia app. From here, I bootstrap the router:
        config.map([
            { route: 'link-a', name: 'link-a', moduleId: 'features/link-a', nav: true, title: 'Link A', settings: { auth: true } },
            { route: 'link-b', name: 'link-b', moduleId: 'features/link-b', nav: true, title: 'Link B', settings: { auth: true } }
        ]);

It seems that the Aurelia router doesn't seem to work properly when trying to generate the route from within another VM (BTW route.href is undefined, so I use route.relativeHref, also seems strange...):

 ${route.title}

When accessing that generated link, the browser does a full server request and ignores the Aurelia router completely. Is this a bug or am I missing something?

@bryanrsmith I think this is the same issue we discussed the other day related to config.options.root. Can you confirm?

Is /app the root of the aurelia app, or a page of it? You need a route entry for the page you're starting on. Probably either /, or app. The router can't activate without a matching route for the current page. The navModel's href is not set until after the router activates with a valid current route, which is probably why your nav links are not working.

/app is the result of a razor view being served. That view is an HTML page (..), with the body tag containing aurelia-app="main" es5. Inside main (static content served from wwwroot) it resolves to aurelia.start().then(a => a.setRoot('shell/shell'));. Inside shell, I have this:
    configureRouter(config: RouterConfiguration, router: Router)
    {
        config.map([
            { route: 'link-a', name: 'link-a', moduleId: 'features/link-a', nav: true, title: 'Link A', settings: { auth: true } },
            { route: 'link-b', name: 'link-b', moduleId: 'features/link-b', nav: true, title: 'Link B', settings: { auth: true } },
        ]);

        this.router = router;
    }

Inside the html template for shell, I perform this:
   <compose view-model="./menu-tree.js"
                         model.bind="router"></compose>

Inside menu-tree, the markup is dynamically build up for the InlineViewStrategy. The reason for using menu-tree is to sub-select from the router's routes, i.e. dynamic menu build up depending on what the user has access to. In hindsight, I guess its better to try and  configure the router dynamically inside shell based on the user's privs?

And is there a <router-view></router-view> in your shell template? You will need to add a default route at '' so the router has something to activate.

@bryanrsmith yeah, <router-view></router-view> inside shell template. Changed my first route to include the default route and that renders.
I'm still having no luck with this, so I decided to bypass my own shell root component and go back to basics and created a new root component called bla. The contents are copied from app from the Aurelia docs, with my own routes applied as per earlier comment:
When setting an MVC route to [Route("abc")] (http://localhost:60751/abc), it renders the default route, but the nav fails for all Aurelia routes. Generated nav URL sample: http://localhost:60751/app#/link-a. The same thing happens when removing the route [Route("")](http://localhost:60751/).
The puzzling part is the /app in the generated URLs. I can't figure out where this comes from? I have no physical app component or route path both on MVC and Aurelia's routing. Its a plain vanilla Aurelia component.

Well that's progress :-)
@EisenbergEffect Does the <compose> use look right to you? I think you can drop the .js extension, but I'm not sure if that would break it.

When dropping the .js extension, the URL ends up being http://localhost:60751/shell/menu-tree. Don't know if this is because I'm using RequireJS, will have to step through Aurelia's code, just haven't had much time.
I've replaced the InlineViewStrategy based menu-tree with helper functions on the shell to achieve my menu requirements. I take two points away:

Utilising the router in an InlineViewStrategy call as part of the "bootstrapping" phase is a bad idea, since the router hasn't finished "activating" itself yet.
We need documentation illustrating more complex router uses including, templating, view parts, etc. especially in terms of do's and dont's. I've spent many an hour figuring out where Aurelia's router limitations are that are not documented.

A couple of notes:
I don't understand your first point above. You don't show  how you are using that in any of the preceding examples. Using InlineViewStrategy should work in any scenario. If it doesn't, there's a bug...but I can't exactly tell what you are doing from the details above. There's a lot of missing information in the descriptions.
While we do need much more documentation, I don't see what your issue above has to do with any of the things you are asking about for templating in point 2. The router doesn't have much to do with templating, view-parts, etc.
Please try to put together a simplest possible solution that reproduces your issues. The above information isn't thorough enough for me to understand what you are doing.
Note: You should not have a .js file extension in your compose usage.
@bryanrsmith Should he be setting config.root ?

Setting router.config.root = 'app'; when the root router is configured would tell the router to handle everything under /app. So navigating to '/' would map to /app, '/about' would map to '/app/about', etc. I'm not sure if that's the desired behavior or not, but from what I can tell none of the problems described are symptomatic of a misconfigured router root.
@stefan505 Does this help? As Rob notes, we can certainly help work through some of this, but will probably need to see a complete code sample.
We do need more and better documentation and samples. Is there a specific use case that you've found lacking? I think the docs do have basic info on viewports and templating.


Bump jgit to <IP_ADDRESS>309021850-r to address GHSA-3p86-9955-h393.

It's not clear that this affects gradle, but it at least seems plausible from reading the CVE description. It also makes CVE scanners happy.

Context

Contributor Checklist

[ ] Review Contribution Guidelines
[ ] Make sure that all commits are signed off to indicate that you agree to the terms of Developer Certificate of Origin.
[ ] Make sure all contributed code can be distributed under the terms of the Apache License 2.0, e.g. the code was written by yourself or the original code is licensed under a license compatible to Apache License 2.0.
[ ] Check "Allow edit from maintainers" option in pull request so that additional changes can be pushed by Gradle team
[ ] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective
[ ] Provide unit tests (under <subproject>/src/test) to verify logic
[ ] Update User Guide, DSL Reference, and Javadoc for public-facing changes
[ ] Ensure that tests pass sanity check: ./gradlew sanityCheck
[ ] Ensure that tests pass locally: ./gradlew <changed-subproject>:quickTest

Reviewing cheatsheet
Before merging the PR, comments starting with

❌ ❓must be fixed
🤔 💅 should be fixed
💭 may be fixed
🎉 celebrate happy things

Thank you for your interest in Gradle!
Unfortunately, simply bumping the version breaks code and features in Gradle. I have been working on a proper update but it is currently blocked on a nasty impact on Spock testing.

#27291

I will close this PR given it cannot be accepted as-is.

Oh and I missed your change is only about Gradle's own build logic so it does not impact the Gradle distribution.
Please in the future, follow the security policy as well.


Question about the Hopfield network for TSP problem

Hi! I'm really appreciate for your code and it helps me so much to understanding the paper. I have a question about the initial state of the u_xi matrix, as your code in follow:
    # (i) All initial output potentials were set
    self.input_potential = np.zeros(self.input_potential_shape) # refers to variable u
    self.output_potential = self.output_potential_function(self.input_potential) # refers to variable v
    if strategy == "A":
        self.output_potential = np.random.uniform(0, 0.03, self.input_potential_shape)
    elif ......
    ......

we can see that the u_xi begins as a zero matrix, and by using equation 1 we can get the v_xi matrix as a 10*10 matrix which full of 0.5. But according to the strategy, we should use a random number for v_xi. So in the next step, v_xi changed again accordingly, which makes our first calculation meaningless. My question is, do we really need to calculate the v_xi from u_xi at first? or we can just use random number directly?
Looking forward to your reply.

You can set the initial output_potential to random values directly, based on the stochastic strategies you used (i.e. A, B, C, D). In the case where you prefer not to use these preset strategies recommended by the original paper, we fall back to the output potential calculated by output_potential_function() as mentioned in equation 1.
Hope this answers your question.


Small corrections

Many thanks for submitting your Pull Request :heart:!
What this PR does / why we need it:
requires  #56 to be merged
Special notes for reviewers:
Additional information (if needed):

@antmendoza should we exclude package-lock in gitignore? just wondering if that would be ok
needs rebase

@antmendoza ok after reading stuff around it it seems that we should not exclude package-lock, but use npm ci
instead of npm install. i'm not an expert on this so not sure

rebased for you :)

@tsurdilo I was about to write more or less the same
regarding to npm ci vs npm install, I though  that they have different propose. I am going to read around and come back to you.


Further build issues after latest bundler release

Similar to #44 (fixed by #46), rubygems-await is not able to detect successful releases in Github actions after the latest release of rubygems.
The await appears to start successfully, then prints a warning about a missing rename in fileutils.
After this, the build continues to run but never detects the successful release.
2024-06-28 07:16:28 +0000 W [  versions] [Errno::ENOENT] /opt/hostedtoolcache/Ruby/3.3.3/x64/lib/ruby/gems/3.3.0/gem_exec/gems/bundler-2.5.14/lib/bundler/vendor/fileutils/lib/fileutils.rb:1169:in `rename': No such file or directory @ rb_file_s_rename - (/home/runner/.bundle/cache/compact_index/rubygems.org.443.29b0360b937aa4d161703e6160654e47/versions.1920.tmp, /home/runner/.bundle/cache/compact_index/rubygems.org.443.29b0360b937aa4d161703e6160654e47/versions) (Errno::ENOENT)
	from /opt/hostedtoolcache/Ruby/3.3.3/x64/lib/ruby/gems/3.3.0/gem_exec/gems/bundler-2.5.14/lib/bundler/vendor/fileutils/lib/fileutils.rb:1169:in `block in mv'
	from /opt/hostedtoolcache/Ruby/3.3.3/x64/lib/ruby/gems/3.3.0/gem_exec/gems/bundler-2.5.14/lib/bundler/vendor/fileutils/lib/fileutils.rb:2469:in `block in fu_each_src_dest'
	from /opt/hostedtoolcache/Ruby/3.3.3/x64/lib/ruby/gems/3.3.0/gem_exec/gems/bundler-2.5.14/lib/bundler/vendor/fileutils/lib/fileutils.rb:2485:in `fu_each_src_dest0'
	from /opt/hostedtoolcache/Ruby/3.3.3/x64/lib/ruby/gems/3.3.0/gem_exec/gems/bundler-2.5.14/lib/bundler/vendor/fileutils/lib/fileutils.rb:2467:in `fu_each_src_dest'
	from /opt/hostedtoolcache/Ruby/3.3.3/x64/lib/ruby/gems/3.3.0/gem_exec/gems/bundler-2.5.14/lib/bundler/vendor/fileutils/lib/fileutils.rb:1160:in `mv'
	from /opt/hostedtoolcache/Ruby/3.3.3/x64/lib/ruby/gems/3.3.0/gem_exec/gems/bundler-2.5.14/lib/bundler/compact_index_client/cache_file.rb:139:in `block in commit'
	from /opt/hostedtoolcache/Ruby/3.3.3/x64/lib/ruby/gems/3.3.0/gem_exec/gems/bundler-2.5.14/lib/bundler/shared_helpers.rb:105:in `filesystem_access'
	from /opt/hostedtoolcache/Ruby/3.3.3/x64/lib/ruby/gems/3.3.0/gem_exec/gems/bundler-2.5.14/lib/bundler/compact_index_client/cache_file.rb:138:in `commit'
	from /opt/hostedtoolcache/Ruby/3.3.3/x64/lib/ruby/gems/3.3.0/gem_exec/gems/bundler-2.5.14/lib/bundler/compact_index_client/cache_file.rb:123:in `commit!'
	from /opt/hostedtoolcache/Ruby/3.3.3/x64/lib/ruby/gems/3.3.0/gem_exec/gems/bundler-2.5.14/lib/bundler/compact_index_client/cache_file.rb:118:in `write'
	from /opt/hostedtoolcache/Ruby/3.3.3/x64/lib/ruby/gems/3.3.0/gem_exec/gems/bundler-2.5.14/lib/bundler/compact_index_client/cache_file.rb:43:in `block in write'
	from /opt/hostedtoolcache/Ruby/3.3.3/x64/lib/ruby/gems/3.3.0/gem_exec/gems/bundler-2.5.14/lib/bundler/compact_index_client/cache_file.rb:55:in `initialize'
	from /opt/hostedtoolcache/Ruby/3.3.3/x64/lib/ruby/gems/3.3.0/gem_exec/gems/bundler-2.5.14/lib/bundler/compact_index_client/cache_file.rb:41:in `new'
	from /opt/hostedtoolcache/Ruby/3.3.3/x64/lib/ruby/gems/3.3.0/gem_exec/gems/bundler-2.5.14/lib/bundler/compact_index_client/cache_file.rb:41:in `write'
	from /opt/hostedtoolcache/Ruby/3.3.3/x64/lib/ruby/gems/3.3.0/gem_exec/gems/bundler-2.5.14/lib/bundler/compact_index_client/updater.rb:55:in `replace'
	from /opt/hostedtoolcache/Ruby/3.3.3/x64/lib/ruby/gems/3.3.0/gem_exec/gems/bundler-2.5.14/lib/bundler/compact_index_client/updater.rb:17:in `update'
	from /opt/hostedtoolcache/Ruby/3.3.3/x64/lib/ruby/gems/3.3.0/gem_exec/gems/bundler-2.5.14/lib/bundler/compact_index_client/cache.rb:80:in `fetch'
	from /opt/hostedtoolcache/Ruby/3.3.3/x64/lib/ruby/gems/3.3.0/gem_exec/gems/bundler-2.5.14/lib/bundler/compact_index_client/cache.rb:26:in `versions'
	from /opt/hostedtoolcache/Ruby/3.3.3/x64/lib/ruby/gems/3.3.0/gem_exec/gems/bundler-2.5.14/lib/bundler/compact_index_client/parser.rb:24:in `versions'
	from /opt/hostedtoolcache/Ruby/3.3.3/x64/lib/ruby/gems/3.3.0/gem_exec/gems/bundler-2.5.14/lib/bundler/compact_index_client.rb:69:in `versions'
	from /opt/hostedtoolcache/Ruby/3.3.3/x64/lib/ruby/gems/3.3.0/gem_exec/gems/rubygems-await-0.5.3/lib/rubygems/await.rb:211:in `process_collection'
	from /opt/hostedtoolcache/Ruby/3.3.3/x64/lib/ruby/gems/3.3.0/gem_exec/gems/rubygems-await-0.5.3/lib/rubygems/await.rb:60:in `block in call'
	from <internal:kernel>:187:in `loop'
	from /opt/hostedtoolcache/Ruby/3.3.3/x64/lib/ruby/gems/3.3.0/gem_exec/gems/rubygems-await-0.5.3/lib/rubygems/await.rb:53:in `call'
	from /opt/hostedtoolcache/Ruby/3.3.3/x64/lib/ruby/gems/3.3.0/gem_exec/gems/rubygems-await-0.5.3/lib/rubygems/await.rb:42:in `block in call'
2024-06-28 07:16:29 +0000 I [  versions] missing: katalyst-tables (katalyst-tables-3.3.2)
2024-06-28 07:16:33 +0000 I [      info] missing: katalyst-tables (katalyst-tables-3.3.2)
2024-06-28 07:16:37 +0000 I [  versions] found katalyst-tables-3.3.2
2024-06-28 07:16:37 +0000 I [  versions] all found!
2024-06-28 07:16:38 +0000 I [full index] missing: katalyst-tables-3.3.2
2024-06-28 07:16:39 +0000 I [      info] missing: katalyst-tables (katalyst-tables-3.3.2)
2024-06-28 07:16:48 +0000 I [      info] missing: katalyst-tables (katalyst-tables-3.3.2)
2024-06-28 07:16:48 +0000 I [full index] missing: katalyst-tables-3.3.2
2024-06-28 07:16:52 +0000 I [      info] missing: katalyst-tables (katalyst-tables-3.3.2)
2024-06-28 07:16:55 +0000 I [full index] found katalyst-tables-3.3.2
2024-06-28 07:16:56 +0000 I [full index] all found!
2024-06-28 07:17:01 +0000 I [      info] missing: katalyst-tables (katalyst-tables-3.3.2)
2024-06-28 07:17:07 +0000 I [      info] missing: katalyst-tables (katalyst-tables-3.3.2)
2024-06-28 07:17:14 +0000 I [      info] missing: katalyst-tables (katalyst-tables-3.3.2)
2024-06-28 07:17:22 +0000 I [      info] missing: katalyst-tables (katalyst-tables-3.3.2)
2024-06-28 07:17:31 +0000 I [      info] missing: katalyst-tables (katalyst-tables-3.3.2)
2024-06-28 07:17:41 +0000 I [      info] missing: katalyst-tables (katalyst-tables-3.3.2)
2024-06-28 07:17:52 +0000 I [      info] missing: katalyst-tables (katalyst-tables-3.3.2)
2024-06-28 07:18:04 +0000 I [      info] missing: katalyst-tables (katalyst-tables-3.3.2)
2024-06-28 07:18:17 +0000 I [      info] missing: katalyst-tables (katalyst-tables-3.3.2)
2024-06-28 07:18:31 +0000 I [      info] missing: katalyst-tables (katalyst-tables-3.3.2)
2024-06-28 07:18:46 +0000 I [      info] missing: katalyst-tables (katalyst-tables-3.3.2)
2024-06-28 07:19:02 +0000 I [      info] missing: katalyst-tables (katalyst-tables-3.3.2)
2024-06-28 07:19:19 +0000 I [      info] missing: katalyst-tables (katalyst-tables-3.3.2)
2024-06-28 07:19:37 +0000 I [      info] missing: katalyst-tables (katalyst-tables-3.3.2)
2024-06-28 07:19:56 +0000 I [      info] missing: katalyst-tables (katalyst-tables-3.3.2)
2024-06-28 07:20:16 +0000 I [      info] missing: katalyst-tables (katalyst-tables-3.3.2)
2024-06-28 07:20:37 +0000 I [      info] missing: katalyst-tables (katalyst-tables-3.3.2)
2024-06-28 07:20:59 +0000 I [      info] missing: katalyst-tables (katalyst-tables-3.3.2)

See https://github.com/katalyst/tables/actions/runs/9708970325/job/26796749546 for an example.

It might be we managed to get past the issue in #44 and now we hit a new issue, but this might have been around for a while and the code just never managed to get here. I'll try taking a quick look again and see if I can track this down as well 😄

Give https://github.com/segiddins/rubygems-await/pull/52 a try?

@segiddins happy to test, how do I do so? The action fails reliably.

Thank you for opening #52 @segiddins, I totally missed the checksum was also moved into the parser 🤦
Will definitely give it a try on my next Faraday release.
@sfnelson you can test the fix in #52 using the commit sha

@iMacTia I'm not clear how to use that on GitHub Actions, but I've tested locally and this change does resolve my issue.

Sorry @sfnelson I forgot rubygems-await is used as a dependency of the rubygems/publish-gem action, the link I shared was not really helpful.
@segiddins I managed to give this a go just now by explicitly installing rubygems-await pointing to the last commit in #52. You can see the action run here: https://github.com/lostisland/faraday/actions/runs/9837249299/job/27154646226 (note the fetching from your GitHub repo in the setup-ruby step, I'm pointing to it from our repo Gemfile).
The release worked and I received the email correctly this time, but the action is not completing successfully and still showing the error message in the description of this issue

Given it's running gem exec, I think it might still be using the published version. Let me try cutting a new release

mmmh it might be. When I read the documentation, it seemed like it would only install the gem "if needed", but there's no clear way to tell just by looking at the logs unfortunately.
Thank you for the new release, I'll let you know if I still bump into issues on the next Faraday release 👍


helm upgrade error

Output of helm version:
3.11.2
Output of kubectl version:
1.26.3
Cloud Provider/Platform (AKS, GKE, Minikube etc.):
I reported the following error while executing the upgrade
helm upgrade -nacos acos .
WARNING: Kubernetes configuration file is group-readable. This is insecure. Location: /root/.kube/config
WARNING: Kubernetes configuration file is world-readable. This is insecure. Location: /root/.kube/config
Error: UPGRADE FAILED: unable to build kubernetes objects from current release manifest: [resource mapping not found for name: "acos-victoria-metrics-cluster" namespace: "acos" from "": no matches for kind "PodSecurityPolicy" in version "policy/v1beta1"
ensure CRDs are installed first, resource mapping not found for name: "zk-pdb-clickhouse" namespace: "acos" from "": no matches for kind "PodDisruptionBudget" in version "policy/v1beta1"
ensure CRDs are installed first]
But there is no PodDisruptionBudget in my chart. I have deleted all of these and still report these errors

see https://github.com/helm/helm-mapkubeapis for a description of the problem and solution

see https://github.com/helm/helm-mapkubeapis for a description of the problem and solution

Hi I'm able to fix no matches for kind "PodDisruptionBudget" in version "policy/v1beta1
But the error no matches for kind "PodSecurityPolicy" in version "policy/v1beta1 is still occurred. Any idea why?

Hi @donnykrn!
Have you managed to solve this issue?


request: Slash Commands in Command framework

Is your feature request related to a problem? Please describe.
Easy integration of slash commands
Describe the solution you'd like
An simple option to enable slash commands, e.g:
const client = new SapphireClient({
  defaultPrefix: '!',
  id: '123',
  slashCommands: true,
});

Probably a little oversimplified but should hopefully explain it :smile:
Describe alternatives you've considered
There is an example here, but it doesn't integrate well with non interaction commands imo
Additional context
N/A

And syncing with normal commands will be helpful

This will be made available through https://github.com/sapphiredev/interactions, however that plugin is currently WIP and waiting for @vladfrangu to finish it.

Looks like that repo got archived, any other plans to implement this?

It will be in the framework itself. There is a Q&A stage event tomorrow at 18:00 CEST which you can join on the Discord server: https://sapphirejs.dev/discord

We forgot to merge this with #293. Slash command support has been merged for a while now. For now it can be installed through npm i @sapphire/framework@next until we publish v3 to latest.


Should protocol be at a higher level than ports?

In the current API, port numbers/ranges are given priority over protocol:
ports:
- portNumber:
    port: 443
    protocol: TCP

This makes it tricky to express rules for port-less IP protocols such as ICMP, OSPF, etc. This is probably fine for IPv4, but I think this is a problem for IPv6 due it's heavy reliance on ICMP. I expect a use-case such as "allow (TCP, 443) and ICMP traffic" to become increasingly common as IPv6 adoption improves. Maybe ICMP should be special-cased? But that seems a bit awkward.
I'm wondering whether this use-case warrants making protocol a top-level field in AdminNetworkPolicyPort. E.g:
ports:
- protocol: TCP
  portNumber:
    port: 443
- protocol: ICMP

Or perhaps making protocols the one-of variants. E.g:
protocols:
- tcp:
    portNumber:
      port: 443
- udp:
    portRange:
      start: 0
      end: 1024
- icmp:
    type: 3 # Destination unreachable

This is a much larger change, but it enables custom selectors for each protocol. Looks like the CNI-specific NetworkPolicy APIs have gone in this direction as well:

https://docs.tigera.io/calico/3.25/network-policy/policy-rules/icmp-ping
https://docs.cilium.io/en/latest/security/policy/language/#limit-icmp-icmpv6-types

Apologies if this has already been discussed elsewhere, I wasn't able to find any previous threads on this topic.

Thanks! I'll try to attend the meeting.

Kubernetes networking is mostly only defined for TCP, UDP, and SCTP. It is not defined how/whether NetworkPolicy affects other protocols. (Eg, if a pod is "isolated", does that mean it can't answer ARP requests?) For that matter, it is not defined how/whether other protocols even work on the pod network. (Eg, it's possible that the network plugin intercepts ARP requests and unicasts them rather than broadcasting them, or even just forges replies to them without ever sending them to the expected destination.)
Administrators may have more use cases for lower-level protocols than app developers, but this still runs into the problem that it's not well-defined whether such protocols can traverse the pod network at all...

We decided to start with documenting the fact that the behaviour for some protocols is undefined. Then we are going to get a list of use cases when this behaviour needs to be defined and update the docs for the corresponding cases.

/assign @npinaeva

https://github.com/kubernetes/website/pull/39875

/remove-lifecycle stale

This effort will be tracked here https://github.com/kubernetes-sigs/network-policy-api/issues/187

/close


importing a scdoc

Trying to import a .scdoc (or .prt) but receive the response:
PrimeRuntimeError: JSON Parsing Error - Escape sequence used in the string was not recogonized
import os
import tempfile
from ansys.meshing import prime
from ansys.meshing.prime.graphics import Graphics
modelFile = 'D:\TRRs\2023\05 SGRE Meshing'
model = 'bob.scdoc'
myModel = os.path.join(modelFile,model)
prime_client = prime.launch_prime()
model = prime_client.model
file_io = prime.FileIO(model)
file_io.import_cad(
file_name=myModel,
params=prime.ImportCadParams(
model=model,
length_unit=prime.LengthUnit.MM,
part_creation_type=prime.PartCreationType.MODEL,
),
)
params = prime.ImportCadParams(
model=model, cad_reader_route=prime.CadReaderRoute.SPACECLAIM
)
prime.FileIO(model).import_cad(file_name=myModel, params=params)
💻 Which operating system are you using?
Windows
📀 Which ANSYS version are you using?
23r1
🐍 Which Python version are you using?
3.10
📦 Installed packages
PrimeRuntimeError: JSON Parsing Error - Escape sequence used in the string was not recogonized

@mcMunich The issue seems to be that you are using regular string to describe the file path which contains \ character. I would suggest trying either of the following options.
modelFile = 'D:\\TRRs\\2023\\05 SGRE Meshing'
model = 'bob.scdoc'
myModel = os.path.join(modelFile,model)

OR with raw strings in python as
modelFile = r'D:\TRRs\2023\05 SGRE Meshing'
model = 'bob.scdoc'
myModel = os.path.join(modelFile,model)

@ninad-kamat Thanks. I thought I checked that but that was the issue.

I guess we do need to have better error messaging for this. I will create a PR to fix this so that we throw a better error message during import.


Pop ups like Intellisense are driving me nuts! How can I disable them all at once and but use them again when I want to?

RedCMD,
Please tell me more!  I've spent an entire day fiddling around with
settings and gotten nowhere.
The only way I have found to disable Intellisense pop ups is to uninstall
the "Intellicode" and "Intellicode API usage examples" extensions entirely.
Of course, this prevents me from using Intellicode when I really do need
it.
The only popups that I've been able to turn off by changing settings are
the suggestions for function and variables.names which can, indeed, be
controlled with ctrl + space.
What am I missing?
...Bob
On Mon, Jan 22, 2024 at 1:20 PM RedCMD @.***> wrote:

you should be able to just disable it in settings
can use ctrl+space to bring intellisense suggestions
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/203072#issuecomment-1904838383,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQVEH3L45324JETL6QNWPTLYP3J3PAVCNFSM6AAAAABCFZGBXCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMBUHAZTQMZYGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

We closed this issue because it is a question about using VS Code rather than an issue or feature request. Please search for help on StackOverflow, where the community has already answered thousands of similar questions. You may find their guide on asking a new question helpful if your question has not already been asked. See also our issue reporting guidelines.
Happy Coding!


2 points are  less in output.obj compare to input.obj file

I have tested code on my test file. Code run without error, but the number of points in output file is less compare to input file by 2 points. I had fixed 2 points in input file as per you suggested in repository.Is there any thing going wrong? I had attached the test file below.
test_file = test_simple_sample.obj.zip
Number of points in input file = 63
Number of points in output file = 61

Vertex 15 and 59 are not in some triangles. Could you verify the mesh?

@icemiliang  Yes this unassigned points are missing form output mesh. So basically input mesh should not have any unassigned points in it. Is it correct?

@rupeshchandgude After loading the vertices and faces, MeshLib will build a half-edge connection of all the vertices and faces. Isolated vertices will then be removed, so the output will not have those vertices.

@icemiliang  Thanks for clarification.