Significant performance degradation during testing compared to pyethereum

py-evm Version: 0.2.0a26
OS: osx
Python Version (python --version): 3.6.5
Environment (output of pip freeze):

aiohttp==2.3.10
amqp==2.3.2
aniso8601==3.0.2
appnope==0.1.0
asn1crypto==0.24.0
async-lru==0.1.0
async-timeout==3.0.0
attrdict==2.0.0
backcall==0.1.0
billiard==<IP_ADDRESS>
bitcoin==1.1.42
celery==4.2.0
certifi==2018.4.16
cffi==1.11.5
chardet==3.0.4
click==6.7
coincurve==7.1.0
coverage==4.5.1
cryptography==2.3.1
cytoolz==<IP_ADDRESS>
decorator==4.3.0
dj-database-url==0.5.0
Django==2.0.6
django-celery-beat==1.1.1
django-celery-results==1.0.1
django-cors-headers==2.3.0
eth-abi==1.1.1
eth-account==0.2.3
eth-bloom==1.0.0
eth-hash==0.1.4
eth-keyfile==0.5.1
eth-keys==0.2.0b3
eth-rlp==0.1.2
eth-tester==0.1.0b27
eth-utils==1.0.3
Flask==1.0.2
Flask-GraphQL==1.4.1
graphene==2.1
graphene-django==2.0.0
graphql-core==2.0
graphql-relay==0.4.5
hexbytes==0.1.0
idna==2.7
idna-ssl==1.1.0
ipython==6.5.0
ipython-genutils==0.2.0
iso8601==0.1.12
itsdangerous==0.24
jedi==0.12.1
Jinja2==2.10
kombu==4.2.1
lru-dict==1.1.6
lxml==4.2.4
MarkupSafe==1.0
multidict==4.3.1
netdisco==2.0.0
netifaces==0.10.7
parsimonious==0.8.0
parso==0.3.1
pbkdf2==1.3
pexpect==4.6.0
pickleshare==0.7.4
plyvel==1.0.4
promise==2.1
prompt-toolkit==1.0.15
psycopg2==2.7.5
psycopg2-binary==2.7.5
ptyprocess==0.6.0
py-ecc==1.4.3
py-evm==0.2.0a26
py-solc==3.1.0
pycparser==2.18
pycryptodome==3.6.1
pyethash==0.1.27
PyGithub==1.40
Pygments==2.2.0
PyJWT==1.6.4
pysha3==1.0.2
python-dateutil==2.7.3
pytz==2018.5
PyYAML==3.12
redis==2.10.6
repoze.lru==0.7
requests==2.19.1
rlp==1.0.2
Rx==1.6.1
scrypt==0.8.6
secp256k1==0.13.2
semantic-version==2.6.0
simplegeneric==0.8.1
singledispatch==<IP_ADDRESS>
six==1.11.0
SQLAlchemy==1.2.9
toolz==0.9.0
traitlets==4.3.2
trie==1.3.8
typing==3.6.4
uPnPClient==0.0.8
urllib3==1.23
vine==1.1.4
wcwidth==0.1.7
web3==4.3.0
websockets==4.0.1
Werkzeug==0.14.1
yarl==1.2.6
zeroconf==0.20.0

What is wrong?
I'm moving my tests over from pyethereum to py-evm and I'm seeing a significant performance degradation. On pyethreum my test suite would take ~23 seconds. With py-evm it takes ~75 seconds. I've profiled my tests and the profile can be found at https://www.filehosting.org/file/details/753367/mycro.pstat.

This is about inline with the speed I'd expect.  Py-EVM was written first for readability and easy to understand code/architecture.  We're only now getting to the phase where we're starting to focus on performance.
Can you potentially share your test suite?  FYI, gists are really good places to dump files like your pstats file.  Depending on your test setup, you should be able to get some performance by using something like pytest-xdist to spread the test runs across multiple cores.
I'm closing as this isn't a bug, but please feel free to continue the coversation here.  We can re-open if a specific bottleneck is identified.

Ah, understood. Hopefully Py-EVM is eventually faster than pyethereum.
Unfortunately I don't think I can share the test suite just yet but once I open-source the code I'm happy to provide a link.
I would definitely see a performance improvement if I ran my tests across multiple cores but that's not a priority just yet. Looks like I'd have to move from unittest to pytest first.

@paymog it's up to you.  I'd be happy to look over your test suite if you want to give me access.  Contact me out-of-band (gitter/email) if you want to try that.
FYI, pytest can run unittest based tests so you might just try switching test harnesses (btw, pytest is awesome and you should absolutely switch because it is soooooooo much better)


AttributeError: 'QWenTokenizer' object has no attribute 'IMAGE_ST'

Traceback (most recent call last):
File "/scratch2/mas/jiangkailin/jkl/MMKE/VLKEB_F/multimodal_edit_2.py", line 591, in

globals()function_name
File "/scratch2/mas/jiangkailin/jkl/MMKE/VLKEB_F/multimodal_edit_2.py", line 295, in
train_SERAC_Blip2OPT
trainer.run()
File "/scratch2/mas/jiangkailin/jkl/MMKE/VLKEB_F/easyeditor/trainer/BaseTrainer.py",
line 219, in run
archive = torch.load(self.save_path, map_location="cpu")
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/jiangkailin/miniconda3/envs/vlkeb_1/lib/python3.11/site-packages/torch/serialization.py", line 791, in load
with _open_file_like(f, 'rb') as opened_file:
^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/jiangkailin/miniconda3/envs/vlkeb_1/lib/python3.11/site-packages/torch/serialization.py", line 271, in _open_file_like
return _open_file(name_or_buffer, mode)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/jiangkailin/miniconda3/envs/vlkeb_1/lib/python3.11/site-packages/torch/serialization.py", line 252, in init
super().init(open(name, mode))
^^^^^^^^^^^^^^^^
FileNotFoundError: [Errno 2] No such file or directory: './results/models/SERAC_MULTI/blip2_240912_192846'

Hi Kailin,
I have updated the necessary environment for running the Qwen model code. The new environment is at vlkeb_qwenvl. Thank you for your kind reminder!

Thank you for your work!

Hi Kailin, I have updated the necessary environment for running the Qwen model code. The new environment is at vlkeb_qwenvl. Thank you for your kind reminder!

Could you please update the KE code of qwen-vl
Have urgent use, thank you!!

Hi Kailin, I have updated the necessary environment for running the Qwen model code. The new environment is at vlkeb_qwenvl. Thank you for your kind reminder!

Could you please update the KE code of qwen-vl Have urgent use, thank you!!

Hi, Kailin. We have updated the multi-hop code for both the qwen-vl and owl-2 models.


Check if ECR repo exists

Ticket:  Automate ECR repository creation
TEST REPORT:
The function create_ecr_repo_if_needed has been created to check if the image is pushed to an existing ECR repository. It has been tested in the following ways:

Pushing an image with an existing repository:
The image is pushed to the existing repository with the given tag. No further action is needed.
Pushing an image with a non-existing repository name:
Throws the exception RepositoryNotFoundException, this causes a new repository to be created and the function ecr_lifecycle_policy adds a lifecycle policy to the repository. Then the image gets pushed to the repository with its tag.
An new project named 1-job is pushed to the registry:
Throws an exception, as the name convection is not accepted by AWS.

The test function test_ecr_repository_path checks if ecr_repository_path returns the correct repository name.

⚠ Could not find ticket corresponding to `feature/ECR_REPO_CHECK. Does your branch name follow the correct pattern?

@Jorg88 @koeves I think it's ready to be merged

Maybe you can include a quick test report to show how you tested it?

@koeves Yes that sounds good. Just need to fix the linting, then it's all done :)

All done! Can I merge it already or wait til Monday? @koeves

Yes, you can merge it :)


[Bug]: error when starting BONEWORK with MelonLoader latest version.  thank you

All of the following criteria must be met

[X] All Requirements must be installed.
[X] Full Latest.log file included. If no file exists then leave this unchecked and state so.

All of the following are optional to answer

[ ] Tried reinstalling the Game.
[X] Tried reinstalling MelonLoader.
[ ] Tried restarting PC.
[ ] Was able to see the Start Screen.

Describe the issue.
Latest.log
Did you attach your log file?

[X] Yes, I attached my log file to the text box above.
[ ] No, I could not find a log file at {Game_Directory}\MelonLoader\Latest.log

In Windows, click the Start menu and type in "Proxy" and click the result "Change Proxy". Disable all 3 toggles in the image below:

All of the following criteria must be met

[x]  All Requirements must be installed.
[x]  Full Latest.log file included. If no file exists then leave this unchecked and state so.

All of the following are optional to answer

[ ]  Tried reinstalling the Game.
[x]  Tried reinstalling MelonLoader.
[ ]  Tried restarting PC.
[ ]  Was able to see the Start Screen.

Describe the issue.
Latest.log
Did you attach your log file?

[x]  Yes, I attached my log file to the text box above.
[ ]  No, I could not find a log file at {Game_Directory}\MelonLoader\Latest.log

Is this solved?


Question about address of msipte when the offset may be out of 4K range

From the Chapter 2.3.3 of spec, in the process to translate addresses of MSIs,

Extract an interrupt file number I from A as I = extract(A >> 12, DC.msi_addr_mask) ...
Let m be (DC.msiptp.PPN x 2^12).
Let msipte be the value of sixteen bytes at address (m | (I x 16))...

The model uses m + ( I * 16 )  as the address of msipte. However they are not equivalent when  I * 16 is out of 4K range. I could not see any guarantee of that, neither from the extract function nor restriction of msi_addr_mask configuration. Is it expected that I * 16 may be out of 4K range? What should be the outcome when it is allowed and triggered?

An MSI page table is a flat array of MSI page table entries , each 16 bytes. MSI page tables have no multi-level hierarchy like regular RISC-V page tables do.
The definition of the table is provided by the AIA specification:

The number of entries in an MSI page table is 2^k where k is the number of bits that are ones in the MSI address mask used to extract the interrupt file number from the destination guest physical address. If an MSI page table has 256 or fewer entries, the start of the table is aligned to a 4-KiB page address in real physical memory. If an MSI page table has 2k > 256 entries, the table must be naturally aligned to a 2k × 16-byte address boundary.  If an MSI page table is not aligned as required, all entries in the table appear to an IOMMU as unspecified, and any address an IOMMU may compute and use for reading an individual MSI PTE from the table is also unspecified.


Document adding custom dtoverlays

Users might want to add their custom device tree overlays on RPi. Document how to do that (compiling from source, enabling from the dashboard, etc).
 Front conversations

Closing this as was added here: https://www.balena.io/docs/reference/OS/advanced/#setting-device-tree-overlays-dtoverlay-and-parameters-dtparam


Need a function that returns the JSON generated by WriteJson()

There are cases like caching of the response which needs the generated JSON. Is there a way to access this at the moment?
The way I am doing this write now is to marshal the response object manually and then pass the response object to WriteJson for rendering, which essentially does the same thing. This seems to be redundant.

You can do what WriteJson does:
	b, err := w.EncodeJson(v)
	_, err = w.(http.ResponseWriter).Write(b)

Sorry I was away for a few days. That would do. Thanks


If two files are opened only first is displayed in wintabs

If I open two files with nvim only the first one is shown in wintabs. After executing :bnext the second file is also shown in wintabs.
I would expect that both files would be displayed from the beginning.
I use neovim 0.4.2

Both are also shown after executing :WintabsAllBuffers. Maybe I don't understand how wintabs is supposed to work properly. Why does wintabs behave this way? Whats the deal with :WintabsAllBuffers?

I solved it by add the following line to my vimrc
au bufenter * WintabsAllBuffers

but I am still wondering why this doesn't work out of the box.

I think I got the idea.


Content-Length header lost in HEAD request

Hi there, I want to return Content-Length field in the response headers in HEAD requests, but I always get Content-Length: 0. I read the codes and found that Symfony\Component\HttpFoundation\Response@perpare handles HEAD request and cleans body and keeps Content-Length, but the framework doesn't set that.
if ($request->isMethod('HEAD')) {
    // cf. RFC2616 14.13
    $length = $headers->get('Content-Length');
    $this->setContent(null);

    if ($length) {
        $headers->set('Content-Length', $length);
    }
}

Hi there,
Thanks for reporting but it looks like this is a question which can be asked on a support channel. Please only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repo you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.

Hi @driesvints , thanks for your advice, but I do think this is a bug.
For example, this is what I had in my routes/web.php:
Route::get('test', function () {
    return 'hello';
});

when I made a HEAD request to this, I should have had Content-Length: 5 field in the headers.

From this post from stackoverflow  and HTTP 1.1 RFC

The Content-Length entity-header field indicates the size of the entity-body, in decimal number of OCTETs, sent to the recipient or, in the case of the HEAD method, the size of the entity-body that would have been sent had the request been a GET.

In my opinion there should be a Content-Length field with content length in the response headers.

And HTTP RFC sec9.4

The HEAD method is identical to GET except that the server MUST NOT return a message-body in the response. The metainformation contained in the HTTP headers in response to a HEAD request SHOULD be identical to the information sent in response to a GET request.

@yueyuzhao if this is a bug then please fill out the issue template. Please submit again. Thanks.


Caching Extensions using Django Cache

Implementations of Strawberry's ParserCache and ValidationCache, but using Django's cache instead of the functools.lru_cache
This is blocked by upgrading strawberry-graphql to >=0.98.0, since that will get us access to<EMAIL_ADDRESS>which fixes this issue https://github.com/graphql-python/graphql-core/issues/112 which stops unpickling DocumentNode

Hey @hiporox .
I can see how the validation extension being cached using django's cache would be useful, but I wonder in which scenario the parsing would would be? It seems to me that parsing should be faster than requesting an external resource for it (unless it is a very complicated query?)

@bellini666 I just copied over the functionality of the two types of caches that base strawberry already supports. It is very possible that the parsing cache will just be slower. I have not tested the speed of it in comparison to a regular query lookup. I am very happy to drop that chunk of the code and just leave the validation cache

@bellini666 I just copied over the functionality of the two types of caches that base strawberry already supports. It is very possible that the parsing cache will just be slower. I have not tested the speed of it in comparison to a regular query lookup. I am very happy to drop that chunk of the code and just leave the validation cache

Yeah, I think it is better to only have the validation cache here :)

@bellini666 Removed the parser cache and fixed the tests. How does it look now?

@hiporox the PR is fine, just need to make the tests pass and I'll merge it!

@bellini666 Tests pass now, not sure what is up with precommit

Thanks for the contribution @hiporox :)


gb fetch fail with when package name len == 1

diff --git a/cmd/gb-vendor/vendor/repo.go b/cmd/gb-vendor/vendor/repo.go
index b416c07..13d4d1d 100644
--- a/cmd/gb-vendor/vendor/repo.go
+++ b/cmd/gb-vendor/vendor/repo.go
@@ -52,7 +52,7 @@ var (
 // DeduceRemoteRepo takes a potential import path and returns a RemoteRepo
 // representing the remote location of the source of an import path.
 func DeduceRemoteRepo(path string) (RemoteRepo, string, error) {
-       validimport := regexp.MustCompile(`^([A-Za-z0-9-]+)(.[A-Za-z0-9-]+)+(/.[A-Za-z0-9-_.]+)+$`)
+       validimport := regexp.MustCompile(`^([A-Za-z0-9-]+)(.[A-Za-z0-9-]+)+(/.[A-Za-z0-9-_.]*)+$`)
        if !validimport.MatchString(path) {
                return nil, "", fmt.Errorf("%q is not a valid import path", path)
        }
diff --git a/cmd/gb-vendor/vendor/repo_test.go b/cmd/gb-vendor/vendor/repo_test.go
index 1828ed1..ddc5664 100644
--- a/cmd/gb-vendor/vendor/repo_test.go
+++ b/cmd/gb-vendor/vendor/repo_test.go
@@ -19,6 +19,11 @@ func TestDeduceRemoteRepo(t *testing.T) {
                path: "corporate",
                err:  fmt.Errorf(`"corporate" is not a valid import path`),
        }, {
+               path: "github.com/pkg/x",
+               want: &gitrepo{
+                       url: "https://github.com/pkg/x",
+               },
+       }, {
                path: "github.com/pkg/sftp",
                want: &gitrepo{
                        url: "https://github.com/pkg/sftp",

Nice catch. Do you want to provide PR?

I can provide one, but which one character package name should I use , there is no pkg/x

I guess we have to wait for Dave, since he can create one on his account to make sure it will exist in the future.

You can use https://github.com/cznic/b
The fact that the unit tests for that package require net access and an
actual repo to exist at the other end is terrible, and entirely my fault.
On Thu, Jun 11, 2015 at 3:07 AM, Dávid Kaya<EMAIL_ADDRESS>wrote:

I guess we have to wait for Dave, since he can create one on his account
to make sure it will exist in the future.
—
Reply to this email directly or view it on GitHub
https://github.com/constabulary/gb/issues/203#issuecomment-110837475.


Update best_practices.md

This PR resolves []
Description

PR Checklist

[ ] I tested my changes on iOS / Android.
[ ] I added screenshots or videos to illustrate my changes.
[ ] I added Tests and Stories for my changes.
[ ] I added an app state migration.
[ ] I hid my changes behind a feature flag.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

Need help with something? Have a look on our docs, or get in touch with us.

can you update the PR title to follow our best practices @patrinoua ?


Left and Right blocks aren't sorted

Describe the bug
Bug in the python code while sorting layout blocks.
Checklist

I used python "sorted"  function to solve that issue.

To Reproduce
Steps to reproduce the behavior:

What command or script did you run?

h, w = image.shape[:2]

left_interval = lp.Interval(0, w/2*1.05, axis='x').put_on_canvas(image)

left_blocks = text_blocks.filter_by(left_interval, center=True)
left_blocks.sort(key = lambda b:b.coordinates[1])

right_blocks = [b for b in text_blocks if b not in left_blocks]
right_blocks.sort(key = lambda b:b.coordinates[1])

# And finally combine the two list and add the index
# according to the order
text_blocks = lp.Layout([b.set(id = idx) for idx, b in enumerate(left_blocks + right_blocks)])

THIS CODE IS PASTED FROM DOCUMENTATION PAGE: https://layout-parser.readthedocs.io/en/latest/example/deep_layout_parsing/index.html

Environment

Just run on google colab
layoutparser version: 0.3.2
I installed layoutparser using pip and the installed detectron2 from source.

Error traceback
N/A
Screenshots

Additional context
Because I am short in time I've created an issue for that instead of PR.
"sorted(left_blocks, key=lambda x:x.coordinates[1]))" use such sorting to fix bug.

Ahhh -- you might want to add inplace=True for the sort function, which should be resolved by #104 .


add tests

i'd be much more comfortable using this if there were at least some basic ones :)
btw, thanks for making this!

Point taken. Will do my best to write some test when time permits.
For now you can feel comfortable with this as it's been successfully used in production for some time now.

Added basic tests, hopefully should suffice for now.

@fujifish awesome, thanks!


Add support for TruffleRuby

Hello,
This PR adds support for TruffleRuby in Bundler.
I searched for rbx and jruby (case-insensitive) through the codebase to find all places which need changes and where tests use specific Ruby implementations. So hopefully this PR is complete, but please tell me if I missed something.
I'd also like to test TruffleRuby in Bundler's CI, but will do this as a separate PR, as we first need the next release of TruffleRuby so that it includes the fix for https://github.com/oracle/truffleruby/issues/1413.

This should also be backported to 1.16.x.

@bundlerbot r+

@eregon This is great, thanks!
This PR I opened to rubygems a while ago is somehow related. Maybe you want to have a look since you have experience accross several implementations and the constants they define.

@bundlerbot timed out, should it be retried or is there an issue with the PR?

@bundlerbot retry

Thanks for review and merging!

This should also be backported to 1.16.x.

This pull-request is not bugfix. I think we need to bump 1.17.x maybe.
Can we merge non-bugfix feature with tiny versionup?

@hsbt We have allowed non-bugfixes to go into minor releases before if it made sense. And this isn't a huge change - so i'm happy to backport this into the next patch release.

1.16.x or 1.17.x is fine, what TruffleRuby needs is a (non-pre) release of Bundler so the patches in TruffleRuby can be removed and TruffleRuby can use vanilla Bundler.
Is there a schedule for the next Bundler release?
The next release of TruffleRuby has feature freeze on September 21.
That might be quite close, but it would be ideal so there is no overlap between patches in TruffleRuby and support for TruffleRuby in Bundler.

There is no scheduled time for a release, but i will try and get your changes released before the 21st.


XDPLWF: Add support for forwarding RSC

Add support for forwarding RSC in generic XDP. This involves:

Tracking the offload configuration of the NIC
Converting RSC to LSO as efficiently as possible, given the variety of written and unwritten rules for RSC and LSO in Windows, and the fact the XDP/eBPF program may also rewrite the packet data, further violating the rules.
Add diagnostics (perf counters, logging, etc.)
Add functional tests
Add fuzz tests for GSO. do this within functional tests as a stopgap until we complete a more general test #579
Add limited RSC support to XDPMP for perf testing (not automated)

Resolves #456

@csujedihy @maolson-msft @nigriMSFT I'd appreciate if you could look over the RSC and LSO code and pick any holes based on experience with weird RSC/LSO behavior or any contract gotchas that I missed.


Helm: Insert Dependency not working

I can't get the Helm: Insert Dependency command to work.
I've tried to run the command via the commands windows (Ctrl+Shift+P) and via the codelense popup (when applicable) at the following moments:

No existing requirements.yaml file
Existing but empty requirements.yaml file
Existing requirements.yaml file with empty dependencies: array.
Existing requirements.yaml file with other working dependency
Existing requirements.yaml file with other working dependency and existing requirements.lock file.

All of the above resulted in no output in the output window.
I've also tried the update dependencies command, which worked great and showed output in the output window.
I'm on version 0.1.13 of the extension.

cc @technosophos

@nphmuller What operating system are you on?

This command looks like it works in a slightly odd way (it dates back to the Before Time and may not have been polished) - it inserts the dependency information into the current document like a snippet.  So I think you would need to open requirements.yaml, position the cursor where you want to add the dependency, and then issue the command.
However, there is also a bug on Windows where it won't work if neither HOME nor HELM_HOME is defined.  If you're on Windows, you can work around this by setting the HOME environment variable to be the same as your USERPROFILE.
I've got a fix for the Windows bug, and will work with @technosophos to figure out a smarter and more obvious behaviour in terms of inferring the requirements.yaml file etc.  Thanks for flagging this!

@itowlson Thanks for the quick response!
I'm on Windows, so the problem was indeed what you explained. Setting the HOME env var made the command work.
Although the command certainly could be smarter, I'm more than fine with the current behaviour. What threw me off was that no output was printed to the output window and nothing happened, so I wasn't sure if I was using the command wrong, or if it simply was not working.
Do you want me to keep this issue open? If so, should I change the issue to refer to the Windows bug, or to make the command smarter?

Let's raise a separate issue for the command behaviour (I can do that and bring in the Helm folks to advise on UX), and I'll ship a PR with the Windows fix to close this one - that way we won't delay the fix while noodling on gold-plating the behaviour.  Thanks for being understanding!

Okay! I've updated the title.

Latest release should fix the Windows issue (but leaves behaviour unchanged) - thanks for reporting it!


local conditioning

is local conditioning a work in progress? Is there any interest in adding it?

There are several issues that discuss it (#112, #164, #189, #198, #233, #189, #235, and others). We are just taking care of bugs, upgrades and issues; pull requests are welcome, though. It seems some people have partially working implementations, but they either never finished them or didn't open a PR to merge them back.


Demo page - link google

Hi,
First i want to say, cool menu, like it.
I have an issue with the demo version.
I see there is a link to the googlepage but it does not work for me (No browser no mac or pc).
When i press open in new window it goes but not just clicking like that.
Any solution for that? Or is it just me?
Thanks in advance

Looks like all the links are broken ...

It's a demo page, of course if i am going to disable leaving from it ;p
I have to remove that bit from the demo page.


The installation doesn't work

Please provide us with the following information:

OS?

Windows 10

Versions.

Please run ng --version. If there's nothing outputted, please run in a Terminal: node --version and paste the result here:
node --version
v6.8.0

I have seen some open and closed isuues about that but I don't see any active conversation and I'm facing that error for a week+-.
What are your recommendations?
npm ERR! git clone --template=C:\Users\eliorsastiel\AppData\Roaming\npm-cache\_git-remotes\_templates --mirror git@github.com:webpack/compression-webpack-plugin.git C:\Users\eliorsastiel\AppData\Roaming\npm-cache\_git-remotes\git-github-com-webpack-compression-webpack-plugin-git-7e55907cd54a2e91b96d25a660acc6a2a6453f54-e286d0b5: undefined
npm ERR! git clone --template=C:\Users\eliorsastiel\AppData\Roaming\npm-cache\_git-remotes\_templates --mirror git@github.com:webpack/compression-webpack-plugin.git C:\Users\eliorsastiel\AppData\Roaming\npm-cache\_git-remotes\git-github-com-webpack-compression-webpack-plugin-git-7e55907cd54a2e91b96d25a660acc6a2a6453f54-e286d0b5: undefined

I have same problem with the following information:
npm ERR! Windows_NT 10.0.10586 npm ERR! argv "C:\\Program Files\\nodejs\\node.exe" "C:\\Users\\XXX\\AppData\\Roaming\\npm\\node_modules\\npm\\bin\\npm-cli.js" "install" "-g" "angular-cli" npm ERR! node v6.9.1 npm ERR! npm  v3.10.8 npm ERR! code 128

I have the same issue in the node docker image (node:latest)

Fixed in https://github.com/angular/angular-cli/pull/2709, out in beta.18.

After my test, I confirm


Add temperature to show aux

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
This feature is currently unavailable
Expected behaviour
1.Please add a function to the sensors aux_load1 and aux_load2 to insert temperature sensors, if possible.

Fix a graphical problem with energy_cost_buy and energy_cost_sell in the card in the full version. see my screenshot

Thank you very much and I wish you and your family all the best.
Possible Solutions

Mode
full
Context / Reason
Just an idea. This function would be used a lot

Your card is looking great 😄
I'll take a look at both of those

Perfect job. You could move the spot price entity a little to the left so that it does not cover the entities from the grid

I thought there was enough space when I tested it. Can you share a screenshot?

A little higher and it will be perfect :)
Thank you very much.

Should be fixed in 3.9.0


Explain what gets automatically imported as part of Prelude

I found out that BitString just works without importing so I was looking for resources as to what types or functions should work out of the box and I couldn't find any documentation. I only found the source code

Great shout, thank you!

Closing in favor of https://github.com/gleam-lang/gleam/issues/1022

Ah nice one, thank you


Separating OLH & Bravo scripts

QA-507
What?
Separated OLH and VC storage scripts.
Changes:

Renamed account script to OLH and moved VC storage script into a separate Bravo folder.

Included the Python environment to allow me to run version 3.11

Why?
To separate the OLH scripts and bravo scripts in different directories

Delete the ./accounts directory as the scripts are in ./bravo and ./olh folders.

Also, I don't think the data fromchangePhoneNumber_TestData.csv is being used in the OLH script (Change Phone scenario). Please validate this and remove the file and directory from OLH.


benchmark against best practices (reach)

benchmark hal-x against Seurat for single cell data, and FlowSOM/ConsensusCluster for CyTOF/spectral cytometry

FDC


Eyeriss configuration

Thanks for your kindness and great works.
At cfg file,
IfmapSramSzkB: 6144
FilterSramSzkB: 6144
OfmapSramSzkB: 2048
means that the processor SRAM size is (6144 + 6144 + 2048)?
At cfg of eyeriss, the sram size of (IA, W, OA) are 108, 108, 108 but eyeriss has only 108KB in total.
Moreover eyeriss has its register files in PE each. How the simulator handles it? (RF size, SRAM bandwidth)

Good question. expect answers.

Hi @gksruf, For your first question, yes. Processor SRAM size is the summation of IfmapSramSzkB, FilterSramSzkB and OfmapSramSzkB.
For your second question, scalesim models eyeriss-like systolic array. This means it does not support register files in each PE. You might have noticed we don't support row stationary dataflow, which is used in eyeriss. The goal is to support IS, OS and WS data flows in a eyeriss-like systolic array (14x12). Regarding memory, you are correct. It has been updated now to be 36kB each.


Can you close the results pane of the .http file editor in Visual Studio?

Description
When you run a request from the .http editor, it opens a new pane in the tab for the results. Is there any way to close this pane?
Page URL
https://learn.microsoft.com/en-us/aspnet/core/test/http-files?view=aspnetcore-8.0
Content source URL
https://github.com/dotnet/AspNetCore.Docs/blob/main/aspnetcore/test/http-files.md
Document ID
26f1374f-556a-35fc-cc42-b8911746db80
Article author
@tdykstra

If you close the file and reopen it, it opens without the results pane showing. The pane appears only after you send a request. If you want a more direct route to close the results pane, I recommend submitting a suggestion to the Visual Studio Developer Community. From the menu select Help > Send Feedback > Suggest a Feature.


Make sure ID is ASCII

Please only review the 2nd commit.
ping @jskeet

Approved on the 4/27 call


Implement Neural Cleanse Defense

Description
This PR adds the implementation of the methods from Neural Cleanse: Identifying and Mitigating Backdoor Attacks in Neural Networks. Wang et al. (2019).
Fixes #503
Type of change
Please check all relevant options.

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

Testing
Please describe the tests that you ran to verify your changes. Consider listing any relevant details of your test configuration.

[x] test_keras tests that this config works for Keras classifiers

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Codecov Report

Merging #604 into dev_1.4.0 will decrease coverage by 7.44%.
The diff coverage is 36.38%.


"Go To Definition" times out on a complex metadata file

Issue Description
Cannot "Go To Definition" with anything that was imported from a package that isn't from the default nuget source.
Steps to Reproduce
Try to "Go To Definition" on a method from an Object that was defined in a package that came from a source other that the default nuget's.
Expected Behavior
To go to the definition
Actual Behavior
Claims "no definition found"
Logs
OmniSharp log
    ************  Response ************

{
"Request_seq": 101,
"Command": "/gotoDefinition",
"Running": true,
"Success": false,
"Message": ""System.OperationCanceledException: The operation was canceled.\n  at System.Threading.CancellationToken.ThrowOperationCanceledException () [0x00010] in <9f0df102fe6e4cfea29d2e46f585d8a5>:0 \n  at System.Threading.CancellationToken.ThrowIfCancellationRequested () [0x00008] in <9f0df102fe6e4cfea29d2e46f585d8a5>:0 \n  at Microsoft.CodeAnalysis.Formatting.AbstractFormatEngine.ApplyTriviaOperations (Microsoft.CodeAnalysis.Formatting.FormattingContext context, System.Threading.CancellationToken cancellationToken) [0x00004] in <98b6908531d840228bf235ca8d22fd88>:0 \n  at Microsoft.CodeAnalysis.Formatting.AbstractFormatEngine.Format (System.Threading.CancellationToken cancellationToken) [0x0008b] in <98b6908531d840228bf235ca8d22fd88>:0 \n  at Microsoft.CodeAnalysis.CSharp.Formatting.CSharpSyntaxFormattingService.Format (Microsoft.CodeAnalysis.SyntaxNode node, Microsoft.CodeAnalysis.Diagnostics.AnalyzerConfigOptions options, System.Collections.Generic.IEnumerable1[T] formattingRules, Microsoft.CodeAnalysis.SyntaxToken token1, Microsoft.CodeAnalysis.SyntaxToken token2, System.Threading.CancellationToken cancellationToken) [0x0000c] in <faea2fbe354e4ba6bda2599ef8169680>:0 \\n  at Microsoft.CodeAnalysis.Formatting.AbstractSyntaxFormattingService.FormatIndividually (Microsoft.CodeAnalysis.SyntaxNode node, Microsoft.CodeAnalysis.Diagnostics.AnalyzerConfigOptions options, System.Collections.Generic.IEnumerable1[T] rules, System.Collections.Generic.IList1[T] spansToFormat, System.Threading.CancellationToken cancellationToken) [0x00036] in <98b6908531d840228bf235ca8d22fd88>:0 \\n  at Microsoft.CodeAnalysis.Formatting.AbstractSyntaxFormattingService.Format (Microsoft.CodeAnalysis.SyntaxNode node, System.Collections.Generic.IEnumerable1[T] spans, System.Boolean shouldUseFormattingSpanCollapse, Microsoft.CodeAnalysis.Diagnostics.AnalyzerConfigOptions options, System.Collections.Generic.IEnumerable1[T] rules, System.Threading.CancellationToken cancellationToken) [0x0004a] in <98b6908531d840228bf235ca8d22fd88>:0 \\n  at Microsoft.CodeAnalysis.Formatting.Formatter.GetFormattingResult (Microsoft.CodeAnalysis.SyntaxNode node, System.Collections.Generic.IEnumerable1[T] spans, Microsoft.CodeAnalysis.Workspace workspace, Microsoft.CodeAnalysis.Options.OptionSet options, System.Collections.Generic.IEnumerable1[T] rules, System.Threading.CancellationToken cancellationToken) [0x0008e] in <98b6908531d840228bf235ca8d22fd88>:0 \\n  at Microsoft.CodeAnalysis.Formatting.Formatter.Format (Microsoft.CodeAnalysis.SyntaxNode node, System.Collections.Generic.IEnumerable1[T] spans, Microsoft.CodeAnalysis.Workspace workspace, Microsoft.CodeAnalysis.Options.OptionSet options, System.Collections.Generic.IEnumerable1[T] rules, System.Threading.CancellationToken cancellationToken) [0x00000] in <98b6908531d840228bf235ca8d22fd88>:0 \\n  at Microsoft.CodeAnalysis.Formatting.Formatter.FormatAsync (Microsoft.CodeAnalysis.Document document, System.Collections.Generic.IEnumerable1[T] spans, Microsoft.CodeAnalysis.Options.OptionSet options, System.Collections.Generic.IEnumerable1[T] rules, System.Threading.CancellationToken cancellationToken) [0x0014a] in <98b6908531d840228bf235ca8d22fd88>:0 \\n  at Microsoft.CodeAnalysis.MetadataAsSource.AbstractMetadataAsSourceService.AddSourceToAsync (Microsoft.CodeAnalysis.Document document, Microsoft.CodeAnalysis.Compilation symbolCompilation, Microsoft.CodeAnalysis.ISymbol symbol, System.Threading.CancellationToken cancellationToken) [0x0049a] in <3e93b7a6b0cb4d6c978ff97c83568be8>:0 \\n  at OmniSharp.Roslyn.MetadataExternalSourceService.GetAndAddExternalSymbolDocument (Microsoft.CodeAnalysis.Project project, Microsoft.CodeAnalysis.ISymbol symbol, System.Threading.CancellationToken cancellationToken) [0x00246] in <b86a8f7ee5524070a682f0426faddb57>:0 \\n  at OmniSharp.Roslyn.CSharp.Services.Navigation.GotoDefinitionService.Handle (OmniSharp.Models.GotoDefinition.GotoDefinitionRequest request) [0x0037e] in <8ca5e445b15640a0986680e5a0180b25>:0 \\n  at OmniSharp.Endpoint.EndpointHandler2[TRequest,TResponse].GetFirstNotEmptyResponseFromHandlers (OmniSharp.Endpoint.Exports.ExportHandler2[TRequest,TResponse][] handlers, TRequest request) [0x00099] in <9e06207949b04d41ac60c7c691c8ecbb>:0 \\n  at OmniSharp.Endpoint.EndpointHandler2[TRequest,TResponse].HandleRequestForLanguage (System.String language, TRequest request, OmniSharp.Protocol.RequestPacket packet) [0x00163] in <9e06207949b04d41ac60c7c691c8ecbb>:0 \n  at OmniSharp.Endpoint.EndpointHandler`2[TRequest,TResponse].Process (OmniSharp.Protocol.RequestPacket packet, OmniSharp.Endpoint.LanguageModel model, Newtonsoft.Json.Linq.JToken requestObject) [0x0024b] in <9e06207949b04d41ac60c7c691c8ecbb>:0 \n  at OmniSharp.Stdio.Host.HandleRequest (System.String json, Microsoft.Extensions.Logging.ILogger logger) [0x000e7] in :0 "",
"Body": null,
"Seq": 5854,
"Type": "response"
}
Environment information
VSCode version: 1.50.1
C# Extension: 1.23.5
Mono Information
        OmniSharp using built-in mono
Dotnet Information
.NET Core SDK (reflecting any global.json):
 Version:   3.1.403
 Commit:    9e895200cd
Runtime Environment:
OS Name:     ubuntu
OS Version:  20.04
OS Platform: Linux
RID:         ubuntu.20.04-x64
Base Path:   /usr/share/dotnet/sdk/3.1.403/
Host (useful for support):
Version: 3.1.9
Commit:  774fc3d6a9
.NET Core SDKs installed:
3.1.403 [/usr/share/dotnet/sdk]
.NET Core runtimes installed:
Microsoft.AspNetCore.App 3.1.9 [/usr/share/dotnet/shared/Microsoft.AspNetCore.App]
Microsoft.NETCore.App 3.1.9 [/usr/share/dotnet/shared/Microsoft.NETCore.App]
To install additional .NET Core runtimes or SDKs:
https://aka.ms/dotnet-download

Visual Studio Code Extensions

Extension
Author
Version

csharp
ms-dotnettools
1.23.5

mongodb-vscode
mongodb
0.2.1

vscode-docker
ms-azuretools
1.7.0

vscode-kubernetes-tools
ms-kubernetes-tools
1.2.1

vscode-yaml
redhat
0.12.0

vscodeintellicode
VisualStudioExptTeam
1.2.10

I should add that this works fine on Visual Studio. I'm not sure why it's not working here on VS Code with OmniSharp.
I've tried all solutions here: https://stackoverflow.com/questions/47995468/vscode-c-sharp-go-to-definition-f12-not-working/51215346
None worked for me.
This is a fresh install too.

I've also tried on Windows 10 to no avail
    ************  Response ************

{
"Request_seq": 16,
"Command": "/gotoDefinition",
"Running": true,
"Success": false,
"Message": ""System.OperationCanceledException: The operation was canceled.\r\n   at System.Threading.CancellationToken.ThrowOperationCanceledException()\r\n   at Microsoft.CodeAnalysis.CSharp.Simplification.AbstractCSharpReducer.AbstractReductionRewriter.SimplifyNode[TNode](TNode node, SyntaxNode newNode, SyntaxNode parentNode, Func5 simplifier)\\r\\n   at Microsoft.CodeAnalysis.CSharp.Simplification.AbstractCSharpReducer.AbstractReductionRewriter.SimplifyExpression[TExpression](TExpression expression, SyntaxNode newNode, Func5 simplifier)\r\n   at Microsoft.CodeAnalysis.CSharp.Simplification.CSharpNameReducer.Rewriter.VisitIdentifierName(IdentifierNameSyntax node)\r\n   at Microsoft.CodeAnalysis.CSharp.CSharpSyntaxRewriter.Visit(SyntaxNode node)\r\n   at Microsoft.CodeAnalysis.CSharp.CSharpSyntaxRewriter.VisitQualifiedName(QualifiedNameSyntax node)\r\n   at Microsoft.CodeAnalysis.CSharp.Simplification.CSharpNameReducer.Rewriter.VisitQualifiedName(QualifiedNameSyntax node)\r\n   at Microsoft.CodeAnalysis.CSharp.CSharpSyntaxRewriter.Visit(SyntaxNode node)\r\n   at Microsoft.CodeAnalysis.CSharp.CSharpSyntaxRewriter.VisitQualifiedName(QualifiedNameSyntax node)\r\n   at Microsoft.CodeAnalysis.CSharp.Simplification.CSharpNameReducer.Rewriter.VisitQualifiedName(QualifiedNameSyntax node)\r\n   at Microsoft.CodeAnalysis.CSharp.CSharpSyntaxRewriter.Visit(SyntaxNode node)\r\n   at Microsoft.CodeAnalysis.CSharp.CSharpSyntaxRewriter.VisitAttribute(AttributeSyntax node)\r\n   at Microsoft.CodeAnalysis.CSharp.CSharpSyntaxRewriter.Visit(SyntaxNode node)\r\n   at Microsoft.CodeAnalysis.CSharp.Simplification.AbstractCSharpReducer.AbstractReductionRewriter.VisitNodeOrToken(SyntaxNodeOrToken nodeOrToken, SemanticModel semanticModel, Boolean simplifyAllDescendants)\r\n   at Microsoft.CodeAnalysis.Simplification.AbstractSimplificationService3.<>c__DisplayClass10_1.<<ReduceAsync>b__0>d.MoveNext()\\r\\n--- End of stack trace from previous location where exception was thrown ---\\r\\n   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\\r\\n   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)\\r\\n   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\\r\\n   at Microsoft.CodeAnalysis.Simplification.AbstractSimplificationService3.d__9.MoveNext()\r\n--- End of stack trace from previous location where exception was thrown ---\r\n   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\r\n   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)\r\n   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\r\n   at Microsoft.CodeAnalysis.Simplification.AbstractSimplificationService3.<ReduceAsync>d__8.MoveNext()\\r\\n--- End of stack trace from previous location where exception was thrown ---\\r\\n   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\\r\\n   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)\\r\\n   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\\r\\n   at Microsoft.CodeAnalysis.Simplification.Simplifier.<ReduceAsync>d__17.MoveNext()\\r\\n--- End of stack trace from previous location where exception was thrown ---\\r\\n   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\\r\\n   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)\\r\\n   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\\r\\n   at Microsoft.CodeAnalysis.MetadataAsSource.AbstractMetadataAsSourceService.<AddSourceToAsync>d__4.MoveNext()\\r\\n--- End of stack trace from previous location where exception was thrown ---\\r\\n   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\\r\\n   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)\\r\\n   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\\r\\n   at OmniSharp.Roslyn.MetadataExternalSourceService.<GetAndAddExternalSymbolDocument>d__4.MoveNext() in D:\\\\a\\\\1\\\\s\\\\src\\\\OmniSharp.Roslyn\\\\MetadataExternalSourceService.cs:line 63\\r\\n--- End of stack trace from previous location where exception was thrown ---\\r\\n   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\\r\\n   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)\\r\\n   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\\r\\n   at OmniSharp.Roslyn.CSharp.Services.Navigation.GotoDefinitionService.<Handle>d__4.MoveNext() in D:\\\\a\\\\1\\\\s\\\\src\\\\OmniSharp.Roslyn.CSharp\\\\Services\\\\Navigation\\\\GotoDefinitionService.cs:line 73\\r\\n--- End of stack trace from previous location where exception was thrown ---\\r\\n   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\\r\\n   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)\\r\\n   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\\r\\n   at OmniSharp.Endpoint.EndpointHandler2.d__19.MoveNext()\r\n--- End of stack trace from previous location where exception was thrown ---\r\n   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\r\n   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)\r\n   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\r\n   at OmniSharp.Endpoint.EndpointHandler2.<HandleRequestForLanguage>d__20.MoveNext() in D:\\\\a\\\\1\\\\s\\\\src\\\\OmniSharp.Host\\\\Endpoint\\\\EndpointHandler.cs:line 230\\r\\n--- End of stack trace from previous location where exception was thrown ---\\r\\n   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\\r\\n   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)\\r\\n   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\\r\\n   at OmniSharp.Endpoint.EndpointHandler2.d__16.MoveNext() in D:\\a\\1\\s\\src\\OmniSharp.Host\\Endpoint\\EndpointHandler.cs:line 131\r\n--- End of stack trace from previous location where exception was thrown ---\r\n   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\r\n   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)\r\n   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\r\n   at OmniSharp.Stdio.Host.d__13.MoveNext() in D:\\a\\1\\s\\src\\OmniSharp.Stdio\\Host.cs:line 215"",
"Body": null,
"Seq": 114,
"Type": "response"
}

this should definitely be working. could you share a repro project with this error?

Hey @filipw,
Unfortunately I personally only know of one external source for nuget packages, and it's my work one so I'm not allowed to share. I was hoping maybe someone else had one so I could test.

Unfortunately without a repro it is impossible to provide any help.
What you are seeing is not really any real error yet - it's only a timeout. At the moment go to definition when definition is contained in a metadata is limited to 5 seconds, so if it takes longer (e.g. slow machine or a very complex metadata source text) then it might get canceled in the process and not complete.

Ok I see. I'll invite you to a private repo.

@filipw invite sent for repo.

thanks, I tried it out, it's very helpful.
As I suspected the functionality works - it just runs into a timeout (by default 2s) because the metadata file it needs to go to is really large. Unfortunately this timeout at the moment is not configurable.
What you can do, is you can enable decompilation support (you can find instructions here https://www.strathweb.com/2020/05/decompilation-support-in-omnisharp-and-c-extension-for-vs-code/). This has a higher timeout and should easily complete. As an added bonus you will get better quality behavior - instead of seeing just metadata, you will see decompiled code.
To avoid these problems for the future, we will increase or add an option to increase the go to definition timeout.

Awesome, thank you so much @filipw !

Hey @filipw FYI I tried enabling decompilation support and it's still not working out for me.
I was also wondering if we could swap the label from "Needs More Info"

what do you see when you enabled decompilation? It works without issues on my machine

Nevermind it works fine I wasn't creating omnisharp.json in the right place. Sorry for the bother.

I have this issue on an nuget package: OpenTK
OpenTk generates a file for OpenGl Bindings and is around ~150.000 lines.
Intellisense works in an instant (hover with mouse), but F12 yields 'No definitions found for ...' after a couple of seconds.
For smaller files (example PixelFormat, ~45.000 lines ) it shows the generated metadata after around 1 second, so it works in general on this project, but not for larger files.
I already added decompilation support, as mentioned in an above post.
Here is a simple project:
using System;
using OpenTK.Graphics.OpenGL;

namespace Test
{
    class Program
    {
        static void Main(string[] args)
        {
            var x = PixelFormat.AbgrExt; // loads ~1 sec
            GL.TexCoord1(0.0); // fails after ~20 secs 'No definition found for 'TexCoord1''
        }
    }
}

<PackageReference Include="OpenTk" Version="4.*" />

OpenTk generates the methods the following way:
        [AutoGenerated(Category = "VERSION_1_0", Version = "1.0", EntryPoint = "glTexCoord1d")]
        public static void TexCoord1(Double s) { throw new BindingsNotRewrittenException(); }

And then rewrites each and emits IL for it.
Could this be the issue here?

A lot has changed with O# go to metadata since this issue was opened. If you are still having problems, please open a new issue at https://github.com/omnisharp/omnisharp-roslyn/issues


Systemd template path not correct for sentinel on Ubuntu 16.04

The systemd service file for redis was updated via https://github.com/brianbianco/redisio/pull/302 to work correctly under Ubuntu 16.04 but the sentinel side still needs to be updated
* Parent directory /usr/lib/systemd/system does not exist.
================================================================================
Error executing action `create` on resource<EMAIL_ADDRESS>================================================================================

Chef::Exceptions::EnclosingDirectoryDoesNotExist
------------------------------------------------
Parent directory /usr/lib/systemd/system does not exist.

Resource Declaration:
---------------------
# In /tmp/kitchen/cache/cookbooks/redisio/recipes/sentinel.rb

49: template<EMAIL_ADDRESS>do
50:   source<EMAIL_ADDRESS>51:   variables(
52:     bin_path: node['redisio']['bin_path'],
53:     limit_nofile: redis['default_settings']['maxclients'] + 32
54:   )
55:   only_if { node['redisio']['job_control'] == 'systemd' }
56: end
57:

@chewi you want to take a crack at it since you did the redis side?

Another reason to use the systemd cookbook. I'm not going to do that today though.


Docs section: Automation & DevOps - Storage account access (public endpoint)

Page section: Design Recommendations - Creation of AVD Images

This section details using an Azure Storage Account with Storage Account Keys.
The ESLZ reference implementations will put in place a policy to "Prevent usage of Public Endpoints for PaaS services in the corp connected landing zones", which this workload would meet.

Fix:

Update the recommendation on this page to accommodate the default ESLZ policy applications to ensure compliance with ESLZ. E.g. Private Link
Also if a Storage Account Key can be avoided from using and changed to Managed Identities etc. Then this should also be changed.

Migrated to ADO


ci: add prerelease

I'm merging this in master since in theory it shouldn't affect it. Changes from https://github.com/coveo/ui-kit/pull/2548 were copied here.
New prerelease process
To start a major prerelease (e.g., 1.2.3 → 2.0.0-pre.0):

Make sure that HEAD refers to a version bump on master.
Create a new branch prefixed with prerelease/ (e.g., prerelease/headless_atomic_v2)
Run npm run bump:version:major-prerelease -- @coveo/package1 @coveo/package2 @coveo/package3.
Push the new commit and its tags to the branch.

Updating a prerelease branch:

Wait for master to reference a version bump.
Pull changes from master into the prerelease branch.

Adding new changes to the prerelease branch:

Open pull requests towards the release branch
Squash as usual.

Officially releasing (e.g., 2.0.0-pre.15 → 2.0.0):

Create a pull request from the prerelease branch to “master”.

Associate with a JIRA which QA will use to validate.
Update the changelogs manually
Update all dependents to use the prerelease version (include the -pre. suffix)

Squash after a version bump on master.

Changes in this PR

Added a bump:version:major-prerelease script to the root package.json

This script will create a major prerelease version bump on the current branch, and will give it all the appropriate tags, but won't push it.
The commit will contain the same message that lerna gives version bumps, including [skip-ci].
Changelogs aren't affected.

Non-bump commits on a prerelease/** branch now trigger the same CI as master, except:

Only packages matching /pre\.[0-9]+$/ are upgraded.

If a dependent is out of scope (doesn't match the above version pattern), its dependencies won't be upgraded.

Changelogs aren't updated.

Bump commits on a prerelease/** branch now trigger the same CI as master, except:

Only packages matching /pre\.[0-9]+$/ are published to NPM.
The alpha tag on NPM isn't updated.
The package rollout stops at the dev environment.

Some scripts were converted from CommonJS (.js) to ESM (.cjs)
Various calls to execSync were replaced with a utility execute function

execute, in addition to returning its output, copies stdout & stderr to the console to make debugging easier. I caught some issues early thanks to this.

Other details

Packages are going to be published to the CDN based on their major/minor/patch version, the prerelease version will be ignored. Therefor, 2.0.0 will already be available on the dev CDN, and will be overwritten every time we update the prerelease. Out of scope packages will also update their CDN, but in theory the files should be the same anyway.

https://coveord.atlassian.net/browse/KIT-2114

Nice work!
It would be nice to update the readme for upcoming pre-releases.
Some nitpick: doing the refactor to modules in the same PR as adding the pre-release specific code will make it hard to discern changes when looking back. These kind of things were a bit annoying when fixing something in the dep. process in the past.
I'd wait for Olivier's feedback too before merging

Nice work!
It would be nice to update the readme for upcoming pre-releases.
Some nitpick: doing the refactor to modules in the same PR as adding the pre-release specific code & the migration from Lerna... it will make it hard to discern changes when looking back. These kind of things were a bit annoying when fixing something in the dep. process in the past.
I'd wait for Olivier's feedback too before merging

Agreed, I ideally would've liked to do it separately, but unfortunately I wasn't able to use the semantic monorepo tools with commonJS.

What if it was

PR1: switch to monorepo tools for our current release process + mjs conversion
Test that the main release process still works properly & iterate
PR2: add prerelease
Test, iterate...

What if it was

PR1: switch to monorepo tools for our current release process + mjs conversion
Test that the main release process still works properly & iterate
PR2: add prerelease
Test, iterate...

We can do that, but in this case PR1 may be as big as this PR (possibly bigger since we'd need to add changelogs and conventional graduating). It would be more consistent across both releases though. Is this what you prefer?

PR1: cjs -> mje
PR2: monorepo tools
PR3: prerelease
Etc?


Added preview of application to README.md

This PR Fixes #62 by adding a preview image of the running application (more specifically, the Solar Site Map) to README.md.
If anything doesn't look like what you'd like it to, just add a comment here and I'll try to fix it asap!

Thanks again for your contribution, @Jogius!  If you'd like a shout out on Twitter / for us to give you the Hacktoberfest role on Discord, please send us your handles :)

No worries, happy to help! I actually answered on the commit where you asked this before :D I'd be happy to get a shoutout on Twitter (@JuliusMakowski) and the role on Discord (@Jogius)


Fix broken links in vscode namespace API

Here is where the current page can be viewed: https://code.visualstudio.com/docs/extensionAPI/vscode-api
Here's a list of the links on the page that were broken; they were all due to missing hashtags, so they didn't properly point to the anchors:

https://code.visualstudio.com/docs/extensionAPI/TreeView.visible
https://code.visualstudio.com/docs/extensionAPI/EventEmitter
https://code.visualstudio.com/docs/extensionAPI/CodeAction
https://code.visualstudio.com/docs/extensionAPI/registerDocumentFormattingEditProvider
https://code.visualstudio.com/docs/extensionAPI/CompletionItem.additionalTextEdits

@mjclemente Thanks! 👏


Selfhost installation fails...

Expected Behavior
Should run after the instructions given
Actual Behavior
rails api:start
Fails - looks like IP of API_HOST given in application.yml was not taken...
| => Rails 5.1.6 application starting in production on http://<IP_ADDRESS>:3000
shortly after "unable to start"
The entry API_HOST: is set correctly in the application.yml but it looks like the environment variables are not set at all
Steps to Reproduce
I installed a fresh new Ubuntu 18.04 (Latest Ubuntu) on a VM with 4 Cores, 4GB Ram and 32GB HD
Followed all steps in the "ubuntu_example.sh" but...
---There is an issue in that document:
LIne 57 and line 58 dont make sense in that order
57 opens the PostGres command prompt and 58 is a bash command that will not execute in PostGres.
Is 58 required BEFORE 57 or AFTER 61?
Also line 76 and 77 say open new Tab and enter command there
But with a new tab the terminal does not run RAIL anymore:

chris@ubuFarmbot:~/Farmbot-Web-App$ rails mqtt:start
Command 'rails' not found, but can be installed with:
sudo apt install ruby-railties

It does work fine (rails mqtt:start) in the first tab where all the other setup calls went (but rails api:start with the error above). It does fail too with

{"init terminating in do_boot",{could_not_start,rabbit_auth_backend_jwt,{{shutdown,{failed_to_start_child,rabbit_auth_backend_jwt_pub_key_fetcher,{error,{failed_connect,[{to_address,{"<IP_ADDRESS>",3000}},{inet,[inet],econnrefused}]}}}},{rabbit_auth_backend_jwt_app,start,[normal,[]]}}}}
init terminating in do_boot ()

My FarmBot API Version
whatever is current on your Github
Run git log -1 and paste the results here

@muenchris

57 opens the PostGres command prompt and 58 is a bash command that will not execute in PostGres.

Yes, this is a typo. Thanks for bringing it to our attention. I will update the file shortly.

Command 'rails' not found

Did you log in / out after installing RVM? One workaround might be to run rvm use 2.5.1 every time you open a new tab.
Please let me know if that doesn't work and I can investigate further.

If I logout-login (of Ubuntu) “rails” is not even found in the main terminal.
If I “rvm use 2.5.1” and then “rails api:start” I still get the not that “Command ‘rails’ not found, but can be installed with ‘sudo apt install ruby-railities’”. I have not done this (as its not in your ubuntu_example.sh) – should I do this?
I tried to manuall set the Environment variable “API_HOST=” but that did not help either
On your main page under “Config Settings (important)” it says “Your server wont run without setting ENV variables first”.
But all the variables in my application.yml are not visible in the ENV. Is the file not loaded correctly?
The MQTT starter seems to read it just fine as the MQTT Host is found correctly and the RMQ is trying to connect to the API_HOST (same Ubuntu VM).
My Application.yml has only these entries:
API_HOST: "<IP_ADDRESS>"  <- this is the IP of my UbuntuVM and I can reach this from the Farmbot and other PCs
API_PORT: "3000"
DEVISE_SECRET: "485282..some Key I greated with rake secret”
OS_UPDATE_SERVER: https://api.github.com/repos/farmbot/farmbot_os/releases/latest
MQTT_HOST: "<IP_ADDRESS>"
SMTP_HOST: "..my email server.."
SMTP_PORT: "..my email port.."
NO_EMAILS: "TRUE"
MQTT_VHOST: "/"
RUN_CAPYBARA: "false"
RAILS_ENV: "production"
Right now both “Rails” do not start…MQTT looks better but requires that the api is running, correct?
Thanks for your help
Cheers
Chris
From: Rick Carlino<EMAIL_ADDRESS>Sent: Sunday, May 27, 2018 11:40 AM
To: FarmBot/Farmbot-Web-App<EMAIL_ADDRESS>Cc: Chris M. Muench<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [FarmBot/Farmbot-Web-App] Selfhost installation fails... (#868)
@muenchrishttps://github.com/muenchris
57 opens the PostGres command prompt and 58 is a bash command that will not execute in PostGres.
Yes, this is a typo. Thanks for bringing it to our attention. I will update the file shortly.
Command 'rails' not found
Did you log in / out after installing RVM? One workaround might be to run rvm use 2.5.1 every time you open a new tab.
Please let me know if that doesn't work and I can investigate further.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/FarmBot/Farmbot-Web-App/issues/868#issuecomment-392353688, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AA6totSofbuLxENuLVCa3YFGPFhWbaQjks5t2vMUgaJpZM4UPJnw.

@muenchris

I have not done this (as its not in your ubuntu_example.sh) – should I do this?

Correct, this should be handled by RVM. Installing Ruby components via apt-get can sometimes cause problems.

MQTT looks better but requires that the api is running, correct?

Correct, you will need to load the API before the MQTT process.
Let's try this:

cd wherever_the_app_is_installed
rvm use 2.5.1 (again)
gem install bundler
bundle install
rvm list
(if nothing went wrong in the steps above) rails api:start

Please paste your error logs and the results of rvm list. It is possible that RVM is installed fine but the Rails deps did not install correctly for whatever reason.

I don’t see anything wrong after executing the lines below…except:
<EMAIL_ADDRESS>After I “rails api:start” it still fails
<EMAIL_ADDRESS>Then lots of log output…in the end:
<EMAIL_ADDRESS>Chris
From: Rick Carlino<EMAIL_ADDRESS>Sent: Tuesday, May 29, 2018 5:32 AM
To: FarmBot/Farmbot-Web-App<EMAIL_ADDRESS>Cc: Chris M. Muench<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [FarmBot/Farmbot-Web-App] Selfhost installation fails... (#868)
@muenchrishttps://github.com/muenchris
I have not done this (as its not in your ubuntu_example.sh) – should I do this?
Correct, this should be handled by RVM. Installing Ruby components via apt-get can sometimes cause problems.
MQTT looks better but requires that the api is running, correct?
Correct, you will need to load the API before the MQTT process.
Let's try this:

cd wherever_the_app_is_installed
rvm use 2.5.1 (again)
gem install bundler
bundle install
rvm list
(if nothing went wrong in the steps above) rails api:start

Please paste your error logs and the results of rvm list. It is possible that RVM is installed fine but the Rails deps did not install correctly for whatever reason.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/FarmBot/Farmbot-Web-App/issues/868#issuecomment-392759077, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AA6tospRFcQo_D8h70a8jhAGCU38eNbfks5t3T-9gaJpZM4UPJnw.

@muenchris It seems like your email client is not sending image attachments correctly. Can you please post the screenshots using the Github link.


Questions on AAI verus skaai?

Hello Jim,
Average Amino acid identity (so called AAI), similar to that of ANI but based on amino acid sequences is another way to measure genomic distance smaller than 70~80% ANI. I noticed that skani does support faa (amino acid format as input, predicted via gene prediction software like prodigal/FragGeneScan) kmer processing (and AAI output) but I do not see benchmarks against blastp-based AAI (so alignment based) in the paper. Does it correlate well with blastp based AAI? Since AAI search/alignemnt is much more expensive than ANI, skani will then be another faster tool to approximate blastp-based AAI.
Thanks,
Jianshu

Hi Jianshu,
Thanks for bringing this interesting point up. I have actually never tried using .faa files with skani.
skani, with it's default parameters, would not work well for .faa files because the k-mers are too long (k = 15) and the subsampling rate is too high (c = 125). k = 15 for amino acids would correspond to k = 45 for the nucleotide case, similarly c = 125 corresponds to c = 375 for amino acids.
However, I believe that if one were to change the k and c parameters to something more reasonable, it may work. Currently, you can actually set -k as a hidden parameter (I made it hidden because I don't see why one would want to change it for nucleotide inputs), so feel free to try it.  Probably something like k = 6-9, c = 10-30 would work decently well in theory. However, I give no guarantees.
On a related note, originally skani had a method to take a DNA fasta and do a 6-frame translation to amino acids. We found this worked decently, but it fails on some edge cases, so we have not advertised this feature.
I'll keep this issue open for now, so if others are curious, they can play around with it.

Hi Jianshu,
skani does not work with .faa files right now. Thanks for bringing up that .faa files are actually supported, I never considered this case. I will patch this in the future.
In theory, skani's algorithm could work with .faa files, but I would have to change the implementation to recognize .faa files and do testing on default parameters, since skani's default parameter would work poorly. I think this is interesting though, and if I investigate this in the future I'll let you know.
If you're curious, we originally implemented a 6-frame translation with skani to calculate AAI. We found that this worked decently, but failed for some genomes, so we did not include it in the final release. We never tested with gene prediction software output .faa files, though.

Hello Jim,
Thanks for letting me know because I saw AAI in the triangle subcommand (-a option).  Therefore this option is just a dummy one right.
Thanks,
Jianshu

Hi Jianshu,
Yes that is my bad, I meant to remove it. I have updated the repo again. I checked and there should be no more AAI mentions in the help commands.
Thanks very much for your help!

Hi jianshu,
In my experience, fastani will give slightly better rank correlation than skani. There is slightly more variance associated to skani's ANI calculation due to its sparse seeding scheme when the ANIs are very very similar.

this is consistent with what i observed for the testing dataset.
Thanks for quick response!
Jianshu


Fix minimum and maximum input value settings not localizable

This allow implementer of select2 to provide a different translation for typeaheadMinimumValueText and typeaheadMaximumValueText.

LGTM!

:+1:


A (slightly more complex) PseudoPositioner problem case

Hello,
I'm looking for some pointers and hints as to how to solve a problem I have with Ophyd. If this is the wrong forum to ask, please advise as to where to turn to.
I'm implementing a simple Bluesky setup for an experiment which involves a signal generator. In a nutshell, every measurement point requires that a unique function is generated and uploaded to the signal generator. The "unique functions" are generated by parameters. For the sake of simplicity, let's assume that the function (signal) has two square peaks and it looks like this:
                  +-------------+
                  |             |
                  |             |
------------------+      A      +-----------+     B      +----------->
                                            |            |
                                            |            |
                                            +------------+

There are two peaks there, labeled "A" and "B".
Now I want the RunEngine in Bluesky to iterate over, say different starting points, amplitudes, and/or widths of any of the peaks. Say "A".
The underlying hardware has an EPICS-IOC which offers the basic capabilites one would expect (related to "management", e.g. triggering, start/stop, frequency and scaling of the signal in its entirety), as well as the ability to upload an arbitrary waveform shape for the signal generator to use.
There are obviously several ways of how one could be doing this, including modifying the IOC to actually generate the waveform on the fly and export parameters like FUNC:PEAKA:START, FUNC:PEAKA:WIDTH, ... FUNC:PEAKB:WIDTH... as EPICS variables for Ophyd to use.
But this would take away flexibility: needing to operate a different shape of function (say, with 3 peaks instead of 2, or with Gaussian peaks instead of squares) would require modification of the IOC, or at least restarts. Ugly.
My best idea so far was to implement all of this in Ophyd. The resulting API should look something like this:
# General function-generator control
class MyFunctionGenertor(Device):
    trigger = Component(...)
    freq = Component(...)
    ...

class Peak(MagicSubclassOfDevice): ...

# Specific shape of a square-peak
class SquarePeak(Peak):
    start = Component(...)
    duration = Component(...)
    amplitude = Component(...)

# ...or different type of peaks
class GaussPeak(Peak):
    fwhm = Component(...)
    ...

# Specific User-defined function
class MySpecificFunction(SubclassOfOphydDevice):
    peak_a = Component(SquarePeak, ...)
    peak_b = Component(SquarePeak, ...)

And then to use it like this:
detectors = [...]
dev = MyFunctionGenertor(...)
func = MySpecificFunction(...)

RE = RunEngine({})

RE( scan(
         [detectors],
         func.peak_a.amplitude, 0.0, 1.0, 10,
         func.peak_a.duration, 0.1, 0.5, 10, ...
) )

So this way I'd effectively use the peak parameters as motors. I don't care much for readback values at this point.
The essential part would be that the user can easily build their own function from building blocks like SquarePeak and GaussPeak:
class AnotherFunction(...):
    a = Component(SquarePeak, ...)
    b = Component(GaussPeak, ...)
    c = Component(SquarePeask, ...)

func2 = AnotherFunction(...)

RE( scan([...], func2.b.fwhm, 0, 1, 10, ... ))

My first attempt to do this was to make use of the PseudoPositioner class, along the lines of:
class SquarePeak(PseudoPositioner):
    start = Component(PseudoSingle)
    dur = Component(PseudoSingle)
    ampl = Component(PseudoSingle)

    single_peak_array = Component(SoftPositioner)

    @pseudo_position_argument
    def forward(self, pos):
        a = numpy.zeros(signal_length)[start:start+dur]=ampl
        return self.RealPosition(single_peak_array=a)

    @real_position_argument
    def inverse(self, pos):
        # This doesn't need to make sense. It not generally possible to reconstruct
        # original parameters from an array
        return self.PseudoPosition(start=0, duration=0, ampl=0)

The results are... mixed. It kind of does what it's supposed to in the sense that I can access fields:
In [1]: peak.start.set(3.14)
Out [1]: MoveStatus(done=False, pos=peak, elapsed=0.0, success=False, settle_time=0.0)

...and if I access other fields subsequently (start, dur, ampl) the results are as expected. The ugly side is that I'm forced to provide an inverse() even if it doesn't make sense. The inverse() can even be nonsensical, like here, and it still does the right ting, because apparently the forward() transformation receives all its parameters not from the current inverse(), but from previous setpoints. Still, this is a bit ugly.
But then it gets worse.
The actual data that gets written to single_peak_array is not an array at all, it's apparently only a 0 -- presumably the first array element. If I replace the definition by this, i.e. add the init_pos parameter:
class SquarePeak(PseudoPositioner):
    ...
    single_peak_array = Component(SoftPositioner, init_pos=np.zeros(25))
    ...

Then things fall apart. First, it's even uglier to need to know the size of the array at writing-time. If anything, I should specify it when defining something very specific, like MySpecificFunction, not when defining a general component like SquarePeak.
But even if we ignore that for a moment, what happens is this:
In [6]: s = SquarePeak(name="peak")
peak_partial_peak: < (array([0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0.]),) {}
peak_partial_peak: move (array([0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0.]),) {'wait': False, 'moved_cb': None, 'timeout': None}
Subscription readback callback exception (PartialPeak(name='peak_partial_peak', parent='peak', settle_time=0.0, timeout=None, egu='', limits=(0, 0), source='computed'))
Traceback (most recent call last):
  File "/var/home/florin/.local/lib/python3.12/site-packages/ophyd/ophydobj.py", line 492, in inner
    cb(*args, **kwargs)
  File "/var/home/florin/.local/lib/python3.12/site-packages/ophyd/pseudopos.py", line 734, in _real_pos_update
    self._update_position()
  File "/var/home/florin/.local/lib/python3.12/site-packages/ophyd/pseudopos.py", line 714, in _update_position
    if None in real_cur_pos:
       ^^^^^^^^^^^^^^^^^^^^
ValueError: The truth value of an array with more than one element is ambiguous. Use a.any() or a.all()

So apparently in pseudopos.py somewhere there's an assumption that the position can never be an array. (Right?)
At this point I'm already wondering am I just "holding it wrong"?
And then, even if I overcome these obstacles, I don't know how to continue. Once I have the SquarePeak working, with its partial function / sampling array, I need to bundle an arbitrary number of those into an actual MyFunction object. The obvious PseudoPositioner approach will fail, because SquarePeak is not a PseudoSingle
class MyFunction(PseudoPositionerSubclass):
    peak_a = Component(SquarePeak) # remember that this is a PseudoPositioner itself, not a PseudoSingle!
    peak_b = Component(SquarePeak)

    # This is where we eventually want to end up...
    device_function_upload = Component(EpicsSignal, ...)

So.
I can't shake the feeling that I'm awfully off-course here, but I have no idea where to go.
Do I:

just need to find "the right documentation"? (I've read pretty much everything Google turned up, but it's very limited "official" Ophyd docs only)
scrap through the Ophyd sources and try to puzzle it together until it makes sense?
forget PseudoPositioner, but reimplement what feels like 80-90% of parallel functionality, with the particular caveats in mind for my problem?
forget implementing this as an Ophyd device? (What else? How to I make use of the Ophyd-Device-ness when feeding this to RunEngine?)

The core of the issue seems to be that I need to access sub-components that go more than 1level deep (2 or 3, e.g. func.peak.param ...), but which need to directly influence the top-level data (func), and the data is an array.
Any pointers?
Thanks & Cheers,
F.

The core of the issue seems to be that I need to access sub-components that go more than 1level deep (2 or 3, e.g. func.peak.param ...), but which need to directly influence the top-level data (func), and the data is an array.

...oh, and one particular aspect is that I need to pass down specify information like array lengths, which is available to the "user", at "terminal time", down to intermediate components; and the current Ophyd API seems all about specifying everything at "library editing time", i.e. when defining those intermediate components (which I don't want my users to have to ever touch).

On further thought, is there a sanctioned way to programatically add positioners to  Device? Something along the lines of:
class Function(Device):
    def addSquare(self, start, duration, amplitude): ...

pulse = Function(...)
pulse.addSquare("a", ...)
pulse.addSquare("b", ...)

RE( scan([...], pulse.a_amplitude, ..., pulse.a_duration, ...) )

Because technically, my intermediate namespaces from above (i.e. the "peaks") aren't any kind of components or positioners that will ever be used independently. It's really just the individual peak parameters that I'm ever going to want to modify. But I'll want my users to be able to define a number of functions with wildly different number of peaks. Having them exist flatly, directly as children of the Function device is actually closer to the reality of my problem.
The preferred way of adding peaks directly to the definition of Function by use of Component(...) surely seems elegant, but is overengineered and actually hampering for my particular use case.


Pie Chart hide some labels using labelInterpolationFnc

Hi,
Is there a way to hide some labels on pie charts?
I have labels overlapping on each other and trying to use labelInterpolationFnc and hide some labels based on their % but I don't have access to the original dataset to do the calculation.
Thanks

Hello @arashkay 👋
Chartist has recently been updated to version 1.0 with some new exciting changes like ESM and TypeScript support. The migration guide can be found right here.
Please let me know if the issue is still relevant to you so we can further look into it!


[spark][core] fix when a table have no snapshot or no statistics

Purpose

Linked issue: close #xxx

Tests

API and Format

Documentation

+1


[ate_dll] add function for registering an OpenTitan device

The current ATE DLL contains example functions for registering a TPM or BMC device. These should be removed and replaced with a function to register a generic OpenTitan device.

Functions to register TPM/BMC device have been removed


Crud-fields page in official documentations have error

Bug report
Crud-fields page in official documentations have error.

Backpack, Laravel, PHP, DB version:
Backpack version above 3.0

Thank you Arto!
It looks like readme.io, where our docs are hosted, have pushed a bug in production, very sorry for that. We're investigating the problem and have reached out to them.
In the meantime here's a zip of the current docs: https://cl.ly/3O3T1F090W2O
Hope i'll have a much better solution for you soon.
Cheers!

FIXED! Apparently they pushed a bug to the "code snippet" block. It breaks some pages, some it does not. I've changed that content type to regular markdown everywhere I saw it was broken, so the docs content is now visible again.
Thank you for bringing this up @amovsisyan !
Cheers!

@tabacitu
Amazing work !!
Thanks.


Conjugation typeclasses. Orthogonal reqs for Complex/Quaternion.

Added a Conjugation typeclass, with instances for Complex/Quaternion,
and trivial instances for types having IsReal instances.
Made Complex require (and implement) commutative instances. Made the
Complex requirements stricter, as Complex operations now require at
least a CRing.
Made Quaternion require CRing as well.

Codecov Report

Merging #723 into master will decrease coverage by 0.17%.
The diff coverage is 55.64%.

@@            Coverage Diff             @@
##           master     #723      +/-   ##
==========================================
- Coverage   50.36%   50.18%   -0.18%
==========================================
  Files         192      196       +4
  Lines       11735    11698      -37
  Branches     1602     1567      -35
==========================================
- Hits         5910     5871      -39
- Misses       5825     5827       +2

Impacted Files
Coverage Δ

...ws/src/main/scala/spire/laws/CombinationLaws.scala
94.73% <ø> (ø)
:arrow_up:

core/src/main/scala/spire/syntax/Ops.scala
18.18% <ø> (ø)
:arrow_up:

...aws/src/main/scala/spire/laws/shadows/Shadow.scala
24.28% <0%> (-1.09%)
:arrow_down:

...n/scala/spire/laws/shadows/ShadowConjugation.scala
0% <0%> (ø)

core/src/main/scala/spire/syntax/Syntax.scala
20% <0%> (-0.28%)
:arrow_down:

...aws/src/main/scala/spire/laws/ConjuationLaws.scala
100% <100%> (ø)

...ore/src/main/scala/spire/algebra/Conjugation.scala
100% <100%> (ø)

core/src/main/scala/spire/math/Quaternion.scala
48.11% <40%> (+0.49%)
:arrow_up:

core/src/main/scala/spire/math/FastComplex.scala
47.12% <47.12%> (ø)

core/src/main/scala/spire/math/Complex.scala
55.23% <88.88%> (+3.68%)
:arrow_up:

... and 15 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 e8837e0...e0a7fdd. Read the comment docs.

This looks good, I just had one comment about a missing law in the documentation. Otherwise 👍 from me.

@non: thinking of it, we should not call this typeclass Conjugation, as conjugation from the conjugate elements of field extensions. Our typeclass only really applies to the extension of real numbers to complex numbers, where each number has only one conjugate element. Other extensions are not so (for example cyclotomic numbers over rational numbers).
What we are looking is something that would apply as well to other rings, such as complex matrices. It's closer to Involutive rings. Indeed, the missing law you mention should be present.
I'm going to propose a rename of the typeclass to Involution, with the conjugate operation named star.
BTW, we discussed splitting unprincipled stuff into the legacy package before deprecating it. We should have the reverse: an incubator package where stuff would be promoted to core after we got sufficient feedback. The package extras could play that role.

I like using extras as an incubator, although for stuff we want to eventually delete I'm not sure I'd like to put it back in extras (it doesn't communicate legacy/deprecation).
The Involution name change is good, but I'm a bit wary of using star. I've been using Kleene algebras in a few places and they also have a star operation, which has different laws. Could we perhaps call it adjoin or something? Any other ideas?

OK. I'm good to go for adjoint.

👍

Waiting for the CI to finish and will merge afterwards.

@non Was it the verb adjoin or the adjective/noun adjoint that you intended?


Fixed quiz questions as per feedback

Summary of changes

Checklist

[ ] I have read and followed the contribution guidelines.
[ ] I have read through the Code of Conduct and agree to abide by the rules.
[ ] This PR is for one of the available issues and is not a PR for an issue already assigned to someone else.
[ ] My PR title has a short descriptive name so the maintainers can get an idea of what the PR is about.
[ ] I have provided a summary of my changes.

closes #XXXXX

Is this a continuation of #1145? You should make new commits to the same branch in response to feedback, not open a new PR.

@darshancn
As mentioned, it is best to apply feedback to the same PR you already had opened and not open a new one.
But it looks like you already deleted that branch for the old PR.
But do keep that in mind for future review changes, so you understand how to best contribute.
Also, it is really important that you complete the checklist so you follow the guidelines for making contributions

Hi @darshancn !
I am going to close this PR due to inactivity.
If you would like to contribute, you are free to reopen this PR and apply the necessary changes mentioned in the reivews 👍


Ensure that gRPC PHP implementation conforms to standard status codes in gRPC

Error code specification here - https://github.com/grpc/grpc/blob/master/doc/statuscodes.md

Verified with @a11r in person that PHP is confirming to the standard.


Error 80004005 in classic ASP

When I try to read a non empty request parameter with method POST, in my ASP application, I always have the error:
erreur '80004005'
/test.asp, ligne 4

I can avoid this error by setting the SecStreamInBodyInspection to On, but it is not a very useful solution as it slow down a lot the uploads.
Here is a test page
<html>
    <head></head>
    <body>
        Read : <%=Request("test")%>
        <form method="post">
            <input type="text" name="test">
        </form>
    </body>
</html>

If you load this page it works fine.
If you enter no value in the input and press enter, it still works fine.
If you enter a value in the input and press enter, you will have the error.
If you modify the form method from POST to GET, no more error.
Environment :
Windows 8, IIS8, ASP 32 bits

Hello Victor,
I would like to CONFIRM with you that this issue still exists exactly the same as described above, even whilst using v2.9.2.  When I uninstall ModSecurity, the issue goes away.  I will try v3 next, and let you know what happens by coming back here and adding another note.  Please let me know when this will be fixed, as I desperately need to block Chinese IP addresses from accessing my website.  Thank you!

Hello Victor,
Actually I just found that v3 is not available as an MSI for installation on Windows (at: http://www.modsecurity.org/download.html).  Please let me know when you think there might be a fix in place.  I really need a way to stop IP addresses from China from accessing my servers.  I suppose I could add the PeerBlock firewall, but don't really want to go that route.  Thank you very much for your time!

Still could not reproduce the issue on a recent IIS build for 2.9.x.

2018-11-19 22:28:02 <IP_ADDRESS> POST /issue-613.asp - 80 - <IP_ADDRESS> Mozilla/5.0+(Windows+NT+6.1;+WOW64)+AppleWebKit/537.36+(KHTML,+like+Gecko)+Chrome/70.0.3538.102+Safari/537.36 200 0 0 916

Please bear in mind that the for the IIS build, we currently recommend setting SecStreamInBodyInspection to On. (See #1299).
The problem with the slowdown with the uploads is not present anymore since https://github.com/SpiderLabs/ModSecurity/commit/afae6906552fa5852ab295920e25da35392c8443 (issue #1366), so it should be safe to add this for most environments.

There's a chance that the behaviour @beatinger is experiencing on the newer version could be related with a false positive. To be certain, check the EventViewer / audit_logs and debug_logs.
If certain is not a false positive, please provide a detailed description of the running environment (IIS plugins, versions, customization, ruleset) and preferably with reproducible steps leading to the issue alongside the logs so we investigate properly and reopen the issue if still a bug.
Thanks.

I had similar issues.
Windows Web Server 2008 R2, IIS 7.5 and ModSecurity 2.9.1.
I found the ASP request object had TotalBytes that made sense, but BinaryRead(request.TotalBytes) returned 8004005. What was occurring was that ModSecurity was reading (and emptying the return buffer) but leaving all other properties intact.
Therefore it may be that the SecStreamInBodyInspection change will fix this.

Where is that SecStreamInBodyInspection setting at?

@beatinger
This issue was already fix on the 2.9.3 release. You want to consider upgrading.
If you want to change it manually on your local installation, search for a file named modsecurity.conf on your disk. If I remember correctly, by default it will be located on the folder where your ModSecurityIIS package is installed.
https://github.com/SpiderLabs/ModSecurity/commit/45337265f1df8f45908390caf896158d241aae72

I installed PeerBlock and paid for the yearly subscription service to obtain daily updates to block lists.  It seems to be working well, but there are so many blocks going on, that it is slowing down my server.  I really need to block all of this nonsense in my firewall, which is a Cisco ASA 5450, but I do not know how.


feat/edit-delete-subcontract-view

Please review before merging.

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


QMRI notebooks broken

At some points the results of our QMRI notebooks must have taken a turn for the worse:
https://ptb-mr.github.io/mrpro/notebook_qmri_sg_challenge_2024_t1.html
https://ptb-mr.github.io/mrpro/notebook_qmri_sg_challenge_2024_t2_star.html
Both the values of the T1 and T2* map are incorrect.

Probably because we are now using (s) instead of (ms) in IHeader for ti and te.


Split modules and rework types

Pull Request Test Coverage Report for Build c9f0d5822d01b6c29b77fe97022987a1524bd946-PR-4

66 of 66 (100.0%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 37a32bac4199d5cf0a6f81713f2853dd624ea2d9:
0.0%

Covered Lines:
177

Relevant Lines:
177

💛  - Coveralls


Pathing has been changed in gvm-libs

Hey @Jarthianur ,
Just wanted to drop in and say your code is great. I has helped me bootstrap myself in what I needed for myself and I want to help out if an issue occurs. Recently GVM released a push of PATHing that was changed. When running the script it fails for me saying /run/gvm Permission denied.
Here is the link: https://github.com/greenbone/gvm-libs/pull/574.
Let me know what happens. I am currently fixing the issue on my side.

Thanks for reporting this. I'm investigating...

It seems like these recent changes have broken a few things...

I would add this into gvmd and openvas systemd service @Jarthianur.
RuntimeDirectory=gvm
RuntimeDirectoryMode=0750
PermissionsStartOnly=True

This will run directories of gvm and ospd in /run. Hope this helps out.

Thanks for this hint. I've tried the directory entry already with at least some success. But currently facing the issue that there is no scanner created anymore upon installation- or at least gvmd cannot communicate to it.

What are the errors you are receiving in your logs @Jarthianur? For example, in /var/log/gvm/?

When I run gvm-cli like so: gvm-cli --gmp-username admin --gmp-password admin socket --socketpath /run/gvm/gvmd.sock --xml "<get_tasks/>"
It's able to get a status 200 response. Would like to help.

Honestly I do not know what exactly has gone wrong. I have pushed a few changes based on the official greenbone service config. Could you please confirm that it's working for you.

So it is giving an error on the changes with Greenbone's documentation. (I tried the same thing and was not playing nice with me.) For my build, I ironed out some of the pathing and added RuntimeDirectoryMode into my systemd settings for gvmd and openvas since my script will be run at bootup. My script is here: https://github.com/romelBen/gvm_bash_script. Check it out. It draws a lot of inspiration from your code. I cut the systemd services of running the updates for SCAP, GVMD, NVTS, etc and gsa, where I use gvm-cli instead for doing the scans. But what I could find is the pathing in openvas and gvmd can be a big tricky if you follow Greenbone's documentation on installation.
Let me know if you can get it to work. Would like to help out as much as I can.

I've tested the current state of master multiple times now - works good.
What exact errors are you facing?

Let me recreate the build and see if it will build correctly.


Access current control inside ea.addMethod

Hello,
In order to write a custom RequiredIf client-side method, I need a way to access the current control (input) on which the validator has been triggered.
The reason behind my question is pretty simple - instead of saving a value into a hidden field, I would like to set it as data-x custom attribute on the control level.
Can you please tell me how can I do this?
Thank you

Hi, there is no such an information injected to the methods.
The simplest solution would be to pass the field name to the method, e.g.
[AssertThat("Method('SomeField')")]
...

ea.addMethod('Method', function(name) {
    var field = $(':input[name="' + name+ '"]')
    ...
});

Is it acceptable workaround?

Hi,
This is was my original intention also - to pass the fieldname from the backend - the issue is a bit more complex then this and I will tell you why: I have a user-defined form which is generated at runtime based on some fields defined in the database.
So, my model for each column of this form looks like this:
public bool IsRequired { get; set; }

public string FormColumnRegexValidation { get; set; }

[RequiredIf("IsStringValueRequiredAndEmpty(StringValue, IsRequired, EncryptedFormColumnTypeId) == true")]
[AssertThat("IsStringValueRegexMatch(StringValue, FormColumnRegexValidation)")]
public string StringValue { get; set; }

[RequiredIf("IsIntValueRequiredAndEmpty(IntValue, IsRequired, EncryptedFormColumnTypeId) == true")]
public int? IntValue { get; set; }

In order to have the front-end validation working, I need to save the IsRequired and FormColumnRegexValidation (among others) as hidden fields so the javascript validation methods recognize them and be able to use them; my ideal situation from which I started this thread, was to save these 2 (and others) properties as data-x attributes on the input field and in the validation methods to use them to check for the validity.
Being a run-time generated form, I'm not able to send from the backend the field name in order to use it on the front-end and this is why I was looking for a way to "catch" a reference to the input field.
I hope I was much clear this time.
Many thanks for your wonderful work!!!
Evdin

You could fork the repo and proceed with the following few changes of the expressive.annotations.js script (new lines are denoted by // (1-5) <<---- ! comment):

add new currentElement variable to store the element for which expression is being computed:

(function($, window) {
    'use strict';
var
    backup = window.ea, // map over the ea in case of overwrite
    buffer = window.console,
    currentElement,   // (1) <<---- !

set it when assertion and requirement is being computed:

computeAssertThat = function(method, value, element, params) {
    currentElement = element; // (2) <<---- !

...

computeRequiredIf = function(method, value, element, params) {
    currentElement = element; // (3) <<---- !

modify the add method - inject currentElement as a last argument:

toolchain = {
    methods: {},
    addMethod: function(name, func) { // add multiple function signatures to methods object (methods overloading, based only on numbers of arguments)
        var old = this.methods[name];
        this.methods[name] = function() {
            if (func.length === arguments.length) {
                [].push.call(arguments, currentElement); // (4) <<---- !
                return func.apply(this, arguments);
            }
            if (typeof old === 'function') {
                return old.apply(this, arguments);
            }
            [].push.call(arguments, currentElement); // (5) <<---- !
            return func.apply(this, arguments); // no exact signature match, most likely variable number of arguments is accepted
        };
    },

After these changes you should be able to access the current element given as a last argument inside each method:
ea.addMethod('Method', function(arg1, arg2) {
    var element = arguments[arguments.length - 1];
    ...
});

I've not tested it, it's just an ad-hoc idea. Give it a try please.

MANY THANKS!!!!
I will try it and let you know!
Again, thank you!
Evdin

Hi,
Have you found some spare time to check whether it behaves as expected? If yes, do you think it is beneficial to have such a feature (or something similar) built-in into EA e.g. as a configurable option?
Thanks.

Hi,
Sorry - didn't had the time to do it yet but most probably I will take care of it this week because I really need it.
On the other hand - I don't see why it should be configurable - it should be there - if you need it, you can just use it :-)
Thank you again and sorry for the delay.

Due to no response I've provided different implementation, i.e. the current element can be now accessed through the __meta__ property of this scope, e.g
<script>
    ea.addMethod('Method', function() {
        var elem = this.__meta__.element;
        ...
    });

I think it is more straightforward than the solution presented earlier in this thread. __meta__ is a restricted identifier which is used to associate additional information with the model context. It is created to minimize eventual naming conflicts. Nevertheless it must be unique, so usage of the same name for e.g. fields names is forbidden, and enforced by relevant exception.


S390x changes

@jkandasa will do the fix for picking up image names from env. variable.
I will delete the Pull request because I think there are no changes required from our side. Thanks!