Instruction collision panics

There are several types of situations when collision check raises PanicException for physically meaningful pulses:

Back-to-back pulses

Final pulse end colliding with auto-calculated end_time

See this notebook to reproduce (link).

[back-to-back collision error] one could try relaxing overlap check to allow for +-1 tick overlap.

Resolved. add_instr now allows a flexible rigid shift of <1ms which makes sure not to lose the signal.  Did not get to reproduce the second final pulse bug. Tested using nistreamer/debug_scripts/flexible_tick.ipynb.

@nlyu1
Do not agree with this solution. It can lead to very serious problems.
These panics are fundamentally coming from rounding errors when the physical (f64) pulse edges are falling within one clock tick from each other.
The proposed solution does cover that. But in addition, it will also "fix" the cases of actual user mistakes when they indeed try to insert pulses with physical overlap. In such cases, moving pulses is not acceptable for a few reasons:

Duration of the intervals between pulses is just as critical as duration of pulses themselves - intervals are a part of the sequence and must be precise. Silently altering them can easily lead to confusion and nonsensical/misinterpreted experiment results.

Shifting the conflicting pulse by any amount (especially as huge as 1 ms) may lead to a collision with some other pulse inserted later, which was otherwise completely legitimate. If such "induced" collision happens, it will panic at adding this other legitimate pulse and not the original conflicting one making it harder to spot the actual mistake in the user script.

As additional notes, there are two mistakes here:

If shift is done, the pulse is shifted by the whole delta of 1ms and not by conflict_ticks
A typo in this line: start_pos + delta instead of next.start_pos + delta

Instead of shifting pulses, I would consider:

either trim whichever of the pulses to the nearest non-overlapping tick
or relaxing from strict to inclusive inequality (this should result in effective trimming when calculating samples - the sample for the overlap tick will be calculated according to one of the instructions but not the other).

It is acceptable because the fundamental issue is that neighboring pulse edges are requested to be too close by, split by an interval below physical time resolution of the digital hardware. The best hardware can do for back-to-back pulses - output the approximate-last value of the left instruction on one tick and then the approximate-first value of the right instruction on the next tick. So pulses will have to be trimmer/extended to match the rigid clock grid anyways. This approximation/rounding is unavoidable.
For analog outputs, there is an additional factor - analog output bandwidth is intentionally restricted such that voltage step rise time is commensurate with the clock period. It is done to make the output signal smooth. So skipping or adding a single non-zero point at the edge of the pulse will not affect the actual signal shape much.

Thanks for catching the bug fix bugs! The unintended behavior has been fixed
Still disagree with the trim solution.

Silently ignoring an intended one-tick go_high or go_low is much more serious than ignoring unspecified intervals. The user can explicitly specify interval value if they really matter.
A more fundamental reason: the trim solution cannot handle back-to-back go-* with conflicting rounds elegantly without altering intended behavior.


CodeExchange: Supporting Reformulation of Internet-Scale Code Queries in Context (T)

http://ieeexplore.ieee.org/xpl/articleDetails.jsp?arnumber=7371992&punumber%3D7371449%26filter%3DAND(p_IS_Number%3A7371976)

http://codeexchange.ics.uci.edu/

There are more various tools available in this paper


Auditing date is the wrong timezone

The date time logged is shown in the timezone of the Google App Engine server. This should be controlled in the config.ini file by setting a local timezone to use.

Fixed in v1.2.0


Unable to run complete document through the pipeline

Bug 💥
When i run pipeline here it runs on first page of the document instead of all the pages.
doc = iter(df)
page = next(doc)

Is there a way to run all the pages in the document?
Expected behavior  🧮
Run all the pages in the document instead of first page.

To process all pages simply run a for loop over your dataflow.

Can you provide a simple code example please?

Thanks this has solved my issue, thanks for your help.

@JaMe76 is there a way to run multiple pages via batching?

@Matheus-Garbelini  Unfortunately not yet: Inference runs are still sequential only and allowing image batch processing will require re-design of the pipeline component in conjunction with the predictor.
There is this discussion and this issue  with ideas about how things could be speed up.
But there has been not much work towards this direction...

Thanks @JaMe76. For now I'm using YoloV8x for document segmentation detection: https://huggingface.co/DILHTWD/documentlayoutsegmentation_YOLOv8_ondoclaynet
The scores on DocLayNet, IBM are a bit unbelievable, but I wonder if such weights can help this project:
Metrics:
Precision: 0.98
Recall: 0.97
F1: 0.97
mAP50: 0.99
mAP50-95: 0.95
Regards.

@Matheus-Garbelini : 😳 … this must be a typo.  I was just checking the doclaynet paper. They reported a mAP50-0.95:0.76, it is very unlikely that a v8 will give such a boost.
Lookin at the confusion matrix, it seems they grouped some labels as there are only 5 compared to 11 in the original annotations. This will give you a higher mAP, but even then this does not explain their results.
Anyway, this could help the project. The model isn’t huge, definitely something that could qualify for the general purpose. Thanks for pointing to the weights!


Exception when filtering on uint property

When trying to filter on a uint property using a linq query, I get an exception:
42883: operator does not exist: jsonb = integer ---> Npgsql.PostgresException: 42883: operator does not exist: jsonb = integer
   at Npgsql.NpgsqlConnector.<>c__DisplayClass161_0.<<ReadMessage>g__ReadMessageLong|0>d.MoveNext() in C:\projects\npgsql\src\Npgsql\NpgsqlConnector.cs:line 1012
--- End of stack trace from previous location where exception was thrown ---
   at Npgsql.NpgsqlConnector.<>c__DisplayClass161_0.<<ReadMessage>g__ReadMessageLong|0>d.MoveNext() in C:\projects\npgsql\src\Npgsql\NpgsqlConnector.cs:line 1032
--- End of stack trace from previous location where exception was thrown ---
   at System.Threading.Tasks.ValueTask`1.get_Result()
   at Npgsql.NpgsqlDataReader.NextResult(Boolean async, Boolean isConsuming) in C:\projects\npgsql\src\Npgsql\NpgsqlDataReader.cs:line 444
   at Npgsql.NpgsqlCommand.ExecuteDbDataReader(CommandBehavior behavior, Boolean async, CancellationToken cancellationToken) in C:\projects\npgsql\src\Npgsql\NpgsqlCommand.cs:line 1217
   at Marten.Services.CommandRunnerExtensions.<>c__DisplayClass3_0`1.<<FetchAsync>b__0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at Marten.Services.ManagedConnection.<>c__DisplayClass41_0`1.<<ExecuteAsync>b__0>d.MoveNext()

The linq query is standard .Where(x => x.MyUIntProperty == 123)
If I inspect the command text that Marten is generating for the linq query, it looks like:
CAST(d.data -> 'Message' ->> 'MyUIntProperty' as jsonb) = :arg3)

As a workaround, for now I've been able to use a combination of MatchesSql and a cast to int:
x => x.MatchesSql("(d.data ->> 'MyUIntProperty')::bigint = ?", (int)myUInt))

I found TypeMappings and tried various combinations of RegisterMapping(), but none seemed to work. Not sure if this is the right path tho.

I'm taking this one this morning.

And it looks like we need to take this up with the Npgsql team

I tried again this morning, and I was able to get a fix in.


Fix is_distinct from for float NaN values

Which issue does this PR close?

Closes #5392.
Rationale for this change
To fix wrong behavior when comparing NaN. Details in #5392

What changes are included in this PR?

Are these changes tested?
Yes

Are there any user-facing changes?
No

cc @viirya

@alamb @tustvold please check this whenever you have time

This looks good to me. Thank you @comphead -- I have some suggestions but I don't think they are required to merge this PR.
I wonder if it is time to start porting more of these kernels into arrow 🤔

Right, I think https://github.com/apache/arrow-rs/issues/960 is good place to do it.


Release Google.Cloud.Dialogflow.V2 version 4.1.0

Changes in this release:
New features

Deprecated the filter field and add resource_definition (commit 05d7b7b)

Documentation improvements

Add more meaningful comments (commit 05d7b7b)

Created release for Google.Cloud.Dialogflow.V2-4.1.0


Cleanup c2rust output in recon.rs

[ ] Cleanup state machine control flow (current_block and friends) in decode_coefs.
[ ] Cleanup helper functions to take safe arguments where possible (may take raw  pointers as arguments currently).
[ ] General cleanup of c2rust generated code in functions.
[ ] Restore comments from original C.

The following functions still need to be cleaned up:

[ ] decode_coefs
[ ] read_coef_tree
[ ] mc
[ ] obmc
[ ] warp_affine
[ ] rav1d_recon_b_intra
[ ] rav1d_recon_b_inter
[ ] rav1d_filter_sbrow_deblock_rows
[ ] rav1d_filter_sbrow_resize

@thedataking, not sure if you're working on this other than cleaning up the state machines, but I'm cleaning up a bunch in recon.rs since I need it to make refmvs stuff safe, which I need to make the mvs Rav1dRefs Arcs.

Fixed by #1189.


Upgrade VM and unpin webcrypto

Breakout of https://github.com/UnifiedPush/dart-webpush-encryption/pull/3

@p1gp1g - breaking out SDK upgrade and unpin.

Thanks !


Removing a bunch of unneeded internal dependencies

[don't 'r+' until @andll has had a look, 'delegate+' instead]
Removing failure from scratchpad
Removing config_builder from benchmark
Removing client from cluster_test
Removing grpc_helpers from state_synchronizer
Removing crypto from state_synchronizer
Removing types from config_builder
Removing ir_to_bytecode from vm_genesis
Removing failure from transaction_builder
Removing transaction_builder from functional_tests
[don't 'r+' until @andll has had a look, 'delegate+' instead]

Removing client from cluster_test

Yep, its not used anymore, thanks for catching!

@bors-libra delegate+

:v: @huitseeker can now approve this pull request

:umbrella: The latest upstream changes (presumably 6020c8f67f4785a66024aedf3f1340623ec192ed) made this pull request unmergeable. Please resolve the merge conflicts.

@bors-libra r+

:pushpin: Commit c0d11ab has been approved by huitseeker

:hourglass: Testing commit c0d11ab4e00a01811d5bb4b9b3a9d7f83f2b8811 with merge 77674b7016c4a06e6a900feee04799ba0716e4c1...

:sunny: Test successful - checks-circle_commit_workflow
Approved by: huitseeker
Pushing 77674b7016c4a06e6a900feee04799ba0716e4c1 to master...


utils.filter_models.full_name is broken

utils.filter_models.full_name uses model.__module__ and model.__name__, while everything in Django uses model._meta.app_label and model._meta.model_name to build the model's full name. As a result, refactoring an app's models into a package with submodules breaks filter_models (everything else in Django works perfectly fine after this change) when patterns based on app_label/model_name are passed to the module.

PR welcome.


Setup readthedocs

Good to start early with this :-)

I don't think readthedocs is needed given the scope of this package. All relevant information can be contained in the README.


Move oauth-proxy to stable

There appears to be usage and no major problems with the oauth-proxy chart. Therefore, am recommending graduation to stable repository.

@compleatang thanks for looking to promote this to stable. We recently documented the process to do so. Could you make sure this PR meets with the guidelines at https://github.com/kubernetes/charts/blob/master/PROCESSES.md#promoting-a-chart-from-incubator-to-stable

@mattfarina sorry I missed those guidelines. Should comply now.

/ok-to-test

No sure what the Tide Pool test is or when it would fire, but the other tests appear to be good and this should be ready to merge which would then unblock #3593

I opened a PR against the incubator chart to rename it to align to the upstream project's name (oauth-proxy -> oauth2-proxy). In my estimation, it is valuable to match the upstream name. I propose making the jump to stable as oauth2-proxy (bundling together the promotion and rename).
What do you think @compleatang and @mattfarina ?

@emmanuel that makes sense to me. Do want to PR your changes into this one?

I don't know how to 'PR [my] changes into this one' (or I don't understand the suggestion). If you mean adding my changes (rename) to this PR, I don't know how to do it. If you mean something else, I didn't understand.
Given that, if I don't hear anything today, I'll open a new PR that promotes the chart to stable with the name oauth2-proxy. That will be the content of this PR, with the name changed to oauth2-proxy.

Renaming means we have to copy the chart and deprecated the old one. We certainly don't want to do this twice. Thus, we should copy/rename to stable in one go in this PR.
@emmanuel: You could create a PR against @compleatang's branch or he just does it himself.

@emmanuel the process that you probably want to take is:

create a fork of @compleatang's charts repo (that he forked from here)
checkout the oauth-proxy-to-stable branch of your new fork
apply your renaming changes to it, and push to your fork
open a PR with @compleatang

another strategy you could do, but is less ideal, is to pull @compleatang's changes into your existing fork, then add your rename changes, and then create a new PR here that has both the rename and the move to stable, which would make this PR obsolete - as I said, less ideal, but you can find more info on how to do it here: https://stackoverflow.com/a/6022366

the last strategy, is hope that @compleatang likes the renaming so much that he does the work of applying the changes to this PR

@emmanuel sorry for the quick response as I was on the road the last couple of days. I've just manually applied your changes. This PR should now cover the same ground as #3755 . I've renamed the PR to reflect the additional renaming.

@jeff-knurek thanks for spelling that out; makes total sense and I'm a little disappointed that I didn't understand in the first go.
@compleatang thanks for your original suggestion, and sorry for being dense and not understanding. And thanks for applying the changes yourself.
Yay open source!

any update on this PR?

@unguiculus thanks for the review. I've made the requested port non-configurable, updated the helpers and generally put the chart inline with 2.8 create best practices (I hope I caught them all).

/lgtm


Change flow yaml dir

feat: [flow.yaml] change dir and convert comment to Chinese
CLOSE #11

done


Improve speed of Cypress tests

Description
Fixes #117 (WIP)

Why is this important?
It removes unnecessary calls in memory to .each() for single element assertions. Most of the test cases benefit from using .each() as they have multiple HTML elements in index.html.
Note: For these first two files that were modified, the time per test decreases "slightly" as we no longer are using .each() so it consumes less memory (e.g. less calls added to memory). This is still in progress but for the tests that only assert a single element this will yield small performance improvements.

Covered test cases

Did you test on all major browsers?
No tests were added, only slimmed down.

[x] Chrome
[x] Firefox
[x] Edge
[ ] Safari

T&Cs

[x] I confirm I have read and understand the contributing guidelines
[x] I understand the work in this pull request will not be released straight away and will appear in a future release (if approved)
[x] I confirm the work in this pull request is true and valid to the best of my knowledge
[x] I have updated the README, features.md and codes.md files where applicable

You're welcome @jackdomleo7! This was only two files updated so there was quite a bit left to slim down (e.g. remove unnecessary .each() calls). I tried to signify this PR was a WIP but maybe I should have converted it to a draft to be safe.
Oh well, I have about 4-5 other test files in another branch that have .each() calls removed so I will submit those to complete this in a separate PR.

@tannerdolby Haha sorry, I didn't realise!

No problem at all @jackdomleo7! 😅
I will have it submitted within the next couple days.


Cannot run it on windows

Hi,
I was trying to give try to this implementation after reading the paper. I installed all the dependencies in a Conda env on a Window PC. However, I am having the following error when I run the experiment:
$ python experiment.py -d tic-tac-toe -bs 32 -s 1@16 -e401 -lrde 200 -lr 0.002 -ki 0 -wd 0.0001 --print_rule -i 0
C:\Users\m00827298\AppData\Local\miniconda3\envs\rrl\Lib\site-packages\torch\distributed\distributed_c10d.py:608: UserWarning: Attempted
to get default timeout for nccl backend, but NCCL support is not compiled
  warnings.warn("Attempted to get default timeout for nccl backend, but NCCL support is not compiled")
[W socket.cpp:697] [c10d] The client socket has failed to connect to [A2207000547.china.huawei.com]:47339 (system error: 10049 - The requested address is not valid in its context.).
Traceback (most recent call last):
  File "C:\Users\m00827298\Codes\RRL\experiment.py", line 174, in <module>
    train_main(rrl_args)
  File "C:\Users\m00827298\Codes\RRL\experiment.py", line 167, in train_main
    mp.spawn(train_model, nprocs=args.gpus, args=(args,))
  File "C:\Users\m00827298\AppData\Local\miniconda3\envs\rrl\Lib\site-packages\torch\multiprocessing\spawn.py", line 241, in spawn
    return start_processes(fn, args, nprocs, join, daemon, start_method="spawn")
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\m00827298\AppData\Local\miniconda3\envs\rrl\Lib\site-packages\torch\multiprocessing\spawn.py", line 197, in start_processes
    while not context.join():
              ^^^^^^^^^^^^^^
  File "C:\Users\m00827298\AppData\Local\miniconda3\envs\rrl\Lib\site-packages\torch\multiprocessing\spawn.py", line 158, in join
    raise ProcessRaisedException(msg, error_index, failed_process.pid)
torch.multiprocessing.spawn.ProcessRaisedException:

-- Process 0 terminated with the following error:
Traceback (most recent call last):
  File "C:\Users\m00827298\AppData\Local\miniconda3\envs\rrl\Lib\site-packages\torch\multiprocessing\spawn.py", line 68, in _wrap
    fn(i, *args)
  File "C:\Users\m00827298\Codes\RRL\experiment.py", line 57, in train_model
    dist.init_process_group(backend='nccl', init_method='env://', world_size=args.world_size, rank=rank)
  File "C:\Users\m00827298\AppData\Local\miniconda3\envs\rrl\Lib\site-packages\torch\distributed\c10d_logger.py", line 86, in wrapper
    func_return = func(*args, **kwargs)
                  ^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\m00827298\AppData\Local\miniconda3\envs\rrl\Lib\site-packages\torch\distributed\distributed_c10d.py", line 1177, in init_process_group
    store, rank, world_size = next(rendezvous_iterator)
                              ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\m00827298\AppData\Local\miniconda3\envs\rrl\Lib\site-packages\torch\distributed\rendezvous.py", line 246, in _env_rendezvous_handler
    store = _create_c10d_store(master_addr, master_port, rank, world_size, timeout, use_libuv)
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\m00827298\AppData\Local\miniconda3\envs\rrl\Lib\site-packages\torch\distributed\rendezvous.py", line 174, in _create_c10d_store
    return TCPStore(
           ^^^^^^^^^
torch.distributed.DistNetworkError: Unknown error

I am not very familiar with running PyTorch in a Windows environment. Based on the error message "Attempted to get default timeout for nccl backend, but NCCL support is not compiled", I suspect the reason might be that NCCL support is not compiled into your PyTorch installation.

NCCL seems to be related to NVidia GPU and I don't NVidia on my PC so I guess this is the reason I have this warning. Isn't it possible to run the code using only the CPU?

At present, CPU is not supported. I will add a CPU version in the future. However, it is still recommended to run on a GPU, otherwise the speed may be slow.

"I would like to ask if your issue has been resolved?"

Thanks for asking. I will give it another try when I get a GPU


feat(select): Added ability to clear select

Before submitting a pull request, please make sure you have at least performed the following:

[x] read and followed the CONTRIBUTING.md guide.
[x] linted, built and tested the changes locally.
[x] added/updated any applicable API documentation.
[x] added/updated any applicable demos.

This PR adds a isClearable property to the SuiSelect to allow the ability to deselect an option.
It can be useful if you are using a select as a filter and want the ability to reset it.
Here the example:

Hi, do you have any estimation on when will this PR be merged ?


Update armstrong.c

basically easy to understand for the reader if divided into several functions. Makes the code readable,precise &simple.

Definitely helps to understand, it's nice to see there are people trying to contribute to the repository. I think on Chapter 3, in the exercise he uses the keyword extern but in none of the resources he provided on Chapter 3 talks about this specific keyword. This repository needs a lots of reviews it seems.

I'll be adding information on what is missing if you or anyone reading this (as I go through while reading and practicing), whom understand better what is being covered on the repository, can try contributing to new changes to .md files to cover missing informations. NOTES.md


Add --full-depth flag to 'bower install'

Added a --full-depth flag to 'bower install' which turns off the '--depth 1' flag when the GitRemoteResolver creates a 'git clone' command.
This is done by adding full-depth to the install command's list of options, with 'U' as the shorthand. The fullDepth options is then added to the config object. Alternatively, the options object could be passed all the way to GitRemoteResolver by way of the Project, Manager, and resolverFactory modules or the options object itself could be added to the config object.
As an enterprise user of Bower this was done to get around the issue of bower hanging at the checkout stage due to an issue with git's clone command when run using the --depth 1 flag on a https repo. Issue referenced here https://github.com/bower/bower/pull/886 and here https://github.com/bower/bower/issues/1257.

Unfortunately Bower is not developed anymore and we recommend switching to Yarn


Clean up unhelpful extra folder names in crate libs

Summary
The modules crate's exported features are in an unnecessary long path.
For example in the CLI acceptance.rs file, here's how module components are imported:
use relayer_modules::ics03_connection::connection::ConnectionEnd;
use relayer_modules::ics03_connection::exported::Connection;
use relayer_modules::ics03_connection::exported::State as ConnectionState;
use relayer_modules::ics04_channel::channel::ChannelEnd;
use relayer_modules::ics04_channel::exported::Channel;
use relayer_modules::ics04_channel::exported::Order;
use relayer_modules::ics04_channel::exported::State as ChannelState;
use relayer_modules::ics23_commitment::CommitmentPrefix;
use relayer_modules::ics24_host::identifier::{ChannelId, ClientId, ConnectionId, PortId};
use relayer_modules::ics24_host::Path::{ChannelEnds, ClientConnections, Connections};

Proposal:
Remove the extra path where unnecessary, like exported, connection, etc.
For example, updated list:
use relayer_modules::ics03_connection::ConnectionEnd;
use relayer_modules::ics03_connection::Connection;
use relayer_modules::ics03_connection::State as ConnectionState;
use relayer_modules::ics04_channel::ChannelEnd;
use relayer_modules::ics04_channel::Channel;
use relayer_modules::ics04_channel::Order;
use relayer_modules::ics04_channel::State as ChannelState;
use relayer_modules::ics23_commitment::CommitmentPrefix;
use relayer_modules::ics24_host::{ChannelId, ClientId, ConnectionId, PortId};
use relayer_modules::ics24_host::Path::{ChannelEnds, ClientConnections, Connections};

(Makes easier to couple them too.)
To do this, the mod.rs file has to be updated. For example, instead of
pub mod connection;

it will read:
mod connection;
pub use connection::ConnectionEnd;

It's a bit of extra work for us since all published components need to be listed in the mod.rs file (unless there's some globbing option I'm not aware of), but it's beneficial for the developers: it reads more easily.
Please comment your opinions here. This is an opinionated idea, we may or may not want to do it.

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

This is similar to what you did with ibc-proto. That was confusing to me initially, but I like the idea. Just a remark: This modification on its own can be confusing to newcomers. Specifically it is not intuitive what this trick achieves just by just reading mod.rs:
mod connection;
pub use connection::ConnectionEnd;

So I suggest at least a little comment in mod.rs to explain what's happening
The exported module brings up a related problem I believe. Parts of this module (and its intended functionality) may be unnecessary and we can remove it, but perhaps let's double check with @ancazamfir for confirmation. (Note that I removed parts of the ICS03 exported module already in separate work.)

The exported module brings up a related problem I believe. Parts of this module (and its intended functionality) may be unnecessary and we can remove it..

yes, we should remove the traits from the exported.rs files (in both connection and channel folders).

The exported namespace is not a thing anymore.


[QUESTION] 报错提醒不明显

您好，我是刚搭建的ds，然后我将租户，资源中心等等都创建完成了，按照文档，应该是创建一个工作流，然后我创建了一个脚本类型的工作流，运行了一个echo "hello world"，运行失败了，去看日志，只显示了我失败了，但是没有具体说为什么失败

我尽到instance 6这个文件夹里面，看到的日志是"sudo: Sorry, you must have a tty to run sudo"这样的提示

Hi ! Please first confirm whether the deployment manual 1.3
section has been executed，
by the way ,please give priority to using English description, thx!

comment this line "#Default requiretty"  in  the /etc/sudoers  file

I didn't find anything related to this in the file you mentioned, I found that when I made it password free, I wrote "user ALL=(root) NOPASSWD:ALL", then I changed it to "user ALL=(ALL) NOPASSWD:ALL", but it's still wrong.

This order "sed -i 's/Defaults    requirett/#Defaults    requirett/g' /etc/sudoers" was executed.


set listen Address <IP_ADDRESS>, use public ip to access

i set listen Address <IP_ADDRESS>, use public ip to access, but got 400 response, Bad Request (Invalid host)
use peaceful-pie  v1.4.0

400 means that your request is reaching the server, so the IP address is ok. 400 means there is something wrong with your request. It is malformed in some way. But it's not an IP address issue. https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/400
On Tue, Mar 26, 2024, at 2:17 AM, Bo Li wrote:

i set listen Address <IP_ADDRESS>, use public ip to access, but got 400 response, Bad Request (Invalid host)
use peaceful-pie https://github.com/hughperkins/peaceful-pie v1.4.0
image.png (view on web) https://github.com/hughperkins/peaceful-pie/assets/4560963/e15cbd10-cd8e-4630-a7d6-010f662e921e
image.png (view on web) https://github.com/hughperkins/peaceful-pie/assets/4560963/b3b1ec7d-dc15-4cf1-9899-54fac90d793e
—
Reply to this email directly, view it on GitHub https://github.com/hughperkins/peaceful-pie/issues/4, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAA6FKEDXP2K4UPBLGWDTBLY2EHF3AVCNFSM6AAAAABFIJJQVKVHI2DSMVQWIX3LMV43ASLTON2WKOZSGIYDOMZVGMZTSMI.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Oh, just looked at your second screenshot. Hmmm.

Oh right, I remember now. Unity is the server, not the client. Your python executable is the server.
So... you cannot use <IP_ADDRESS> as the ip address when calling a server. <IP_ADDRESS> is not a valid ip address. At least, it means "all ip addresses". And how can you send a message to "all ip addresses"?
The connection is from Unity to the python process. So you need to find a way for Unity to communicate to the python process.
The easiest way is likely to run the python process on the same instance that you are running the Unity processes. That's what I was doing.
Otherwise, you could use ssh tunneling, to create a tunnel from the instance to your local machine, where there python service is running. More complicatd though.
Do you have any reasons not to run the python service on the insance itself?

Thank you for your patient reply.
Unity is the server, so it bind <IP_ADDRESS>, this unity program will be run in a windows server, and python will be deploy to another linux server, this is why this agent must bind <IP_ADDRESS>.
I need a lot of unity  program server, they are the game environments, I will use MCTS algorithm to evaluating the game difficulty.

To be honest, I don't remember which is the server. Would need to check the code... You might be right that unity is the server, in which case <IP_ADDRESS> would make sense, as you say. Yeah, unity is probably the server. Hmmm.
On Tue, Mar 26, 2024, at 7:35 AM, Bo Li wrote:

Thank you for your patient reply.
Unity is the server, so it bind <IP_ADDRESS>, this unity program will be run in a windows server, and python will be deploy to another linux server, this is why this agent must bind <IP_ADDRESS>.
I need a lot of unity program server, they are the game environments, I will use MCTS https://en.wikipedia.org/wiki/Monte_Carlo_tree_search algorithm to evaluating the game difficulty.
—
Reply to this email directly, view it on GitHub https://github.com/hughperkins/peaceful-pie/issues/4#issuecomment-2020188914, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAA6FKEFICOSXDCUJIMWY43Y2FMRNAVCNFSM6AAAAABFIJJQVKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMRQGE4DQOJRGQ.
You are receiving this because you commented.Message ID: @.***>

But, if unity is the server, why am I adding a config for IP address on the server? Pondering.
On Tue, Mar 26, 2024, at 7:40 AM, Hugh Perkins wrote:

To be honest, I don't remember which is the server. Would need to check the code... You might be right that unity is the server, in which case <IP_ADDRESS> would make sense, as you say. Yeah, unity is probably the server. Hmmm.
On Tue, Mar 26, 2024, at 7:35 AM, Bo Li wrote:

Thank you for your patient reply.
Unity is the server, so it bind <IP_ADDRESS>, this unity program will be run in a windows server, and python will be deploy to another linux server, this is why this agent must bind <IP_ADDRESS>.
I need a lot of unity program server, they are the game environments, I will use MCTS https://en.wikipedia.org/wiki/Monte_Carlo_tree_search algorithm to evaluating the game difficulty.
—
Reply to this email directly, view it on GitHub https://github.com/hughperkins/peaceful-pie/issues/4#issuecomment-2020188914, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAA6FKEFICOSXDCUJIMWY43Y2FMRNAVCNFSM6AAAAABFIJJQVKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMRQGE4DQOJRGQ.
You are receiving this because you commented.Message ID: @.***>

Oh it does clearly say "listen address". So yeah, probably the server...
On Tue, Mar 26, 2024, at 7:41 AM, Hugh Perkins wrote:

But, if unity is the server, why am I adding a config for IP address on the server? Pondering.
On Tue, Mar 26, 2024, at 7:40 AM, Hugh Perkins wrote:

To be honest, I don't remember which is the server. Would need to check the code... You might be right that unity is the server, in which case <IP_ADDRESS> would make sense, as you say. Yeah, unity is probably the server. Hmmm.
On Tue, Mar 26, 2024, at 7:35 AM, Bo Li wrote:

Thank you for your patient reply.
Unity is the server, so it bind <IP_ADDRESS>, this unity program will be run in a windows server, and python will be deploy to another linux server, this is why this agent must bind <IP_ADDRESS>.
I need a lot of unity program server, they are the game environments, I will use MCTS https://en.wikipedia.org/wiki/Monte_Carlo_tree_search algorithm to evaluating the game difficulty.
—
Reply to this email directly, view it on GitHub https://github.com/hughperkins/peaceful-pie/issues/4#issuecomment-2020188914, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAA6FKEFICOSXDCUJIMWY43Y2FMRNAVCNFSM6AAAAABFIJJQVKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMRQGE4DQOJRGQ.
You are receiving this because you commented.Message ID: @.***>

The problem has been solved.

set Listen Address to +,so unity can accept connection from another device.
peaceful-pie/PeacefulPie
/NetManager.cs
the code use HttpListener
		listener = new HttpListener(); 		listener.Prefixes.Add($"http://{ListenAddress}:{ListenPort}/");
I found the solution at then end of this web page. https://stackoverflow.com/questions/27770730/400-bad-request-invalid-host-using-simple-webserver-in-c-sharp

Thank you very much :) 🎉

Thank you very much :) 🎉

You're welcome! 😊


Pulling in latest from DMPRoadmap

Includes recent bug fixes

PR seemed indefinitely stuck waiting for checks to start. Did the merge locally and pushed directly to repo


How many problem contracts can MAIAN detect today？

Hello, I am a graduate student from Hohai University. I read your papers on MAIAN, and you can tell me whether there is an increase in the number of contract types that MAIAN can detect today, or whether they are still suicide, profligacy and greedy.

Hi xf97 Can you please check your email. Thanks

Done.


CLI not matching streets that don't align to cardinal directions

Got this result using the CLI with default parameters. Common theme: streets that don't align north-south or east-west are not matching.

Could this be related to the default bearing tolerance? I can't tell what it's set to be but assume it's the minimum (0 or 1 degree). A north-south grid system would be easy to map at 90-degree angles, but anything with angles is less likely to match perfectly.
Posting here as a note to make sure this gets solved.

Example of angular streets that don't align:
sf_angles.geojson.txt

OSRM routing has resolved this


KEP 2837: draft PR for Pod Level Resources KEP

Description

Issue

Closes: #

@ndixita, this PR seems to have a lot of commits that seem to have different authors, please only push your changes to the PR.

Also add description of the changes you have made on this PR.

Hello @ndixita :wave: please take a look at Documenting for a release - PR Ready for Review to get your PR ready for review before Tuesday November 19th 2024 18:00 PST. Thank you!

Hello @ndixita :wave:, 1.32 Docs Shadow here,
A quick reminder to get your PR ready for review before the deadline, Tuesday November 19th at 18:00 PDT. For additional information, refer to: Documenting for a release - PR Ready for Review.
Thank you!

/remove-area blog localization release-eng web-development
/remove-language bn de es fr hi id it ja ko pl pt ru uk vi zh
/remove-sig release

/retitle [WIP] Document pod level resources

If this is reviewable, @ndixita, please update the PR to show that.

working on this today, and will also request an extension

/assign @haircommander

This is a straightforward example of the work, well done  on the time crunch @ndixita . in the future, I think we could add more detailed docs on the different interactions with OOM score/QoS class, but that's not in scope now I think
LGTM from tech perspective

This is a straightforward example of the work, well done on the time crunch @ndixita . in the future, I think we could add more detailed docs on the different interactions with OOM score/QoS class, but that's not in scope now I think
LGTM from tech perspective

I think I missed that we have to write the docs for alpha features. Noting it for next time to add the docs in time.

/sig node

Here's the key piece of feedback: https://github.com/kubernetes/website/pull/48471#pullrequestreview-2468161934
I think if that gets covered, we can merge the PR and then plan some post-merge fixups.

This is a straightforward example of the work, well done on the time crunch @ndixita . in the future, I think we could add more detailed docs on the different interactions with OOM score/QoS class, but that's not in scope now I think
LGTM from tech perspective

@sftim this is where I have approvals from sig-node

Here's the key piece of feedback: #48471 (review)
I think if that gets covered, we can merge the PR and then plan some post-merge fixups.

Tried to address this. PTAL

still LGTM!

Docs lgtm & tech lgtm noted
/approve


Fixing TreeView nodes to expand and selection issues

Q
A

Bug fix?
[x]

New feature?
[ ]

New sample?
[ ]

Related issues?
fixes #1182 #1170 #1062

What's in this Pull Request?

nodesToExpand from TreeItem now takes nodesToExpand + defaultExpandedKeys instead of just defaultExpandendKeys

handleTreeExpandCollapse now updating nodesToExpand content
pathTo function from TreeView wont add duplicates to nodesToExpand
When populating children on first expand wont open all closed section containing selected childrens

You can ignore the other participant for this PR. JoelM365 is my second account. Joel Lopes is my main account.
I'm available if you have any questions.
Have a nice day.
Joel

Thank you @Dixdjo for the fixes!


HelmetData interface incorrect

Do you want to request a feature or report a bug?
Bug
The export interface used with Helmet.peek() is incorrect. The current interface:
export class Helmet extends React.Component<HelmetProps> {
    static peek(): HelmetData;
    static rewind(): HelmetData;
    static renderStatic(): HelmetData;
    static canUseDOM: boolean;
}

export interface HelmetData {
    base: HelmetDatum;
    bodyAttributes: HelmetHTMLBodyDatum;
    htmlAttributes: HelmetHTMLElementDatum;
    link: HelmetDatum;
    meta: HelmetDatum;
    noscript: HelmetDatum;
    script: HelmetDatum;
    style: HelmetDatum;
    title: HelmetDatum;
    titleAttributes: HelmetDatum;
}

However when using this in a mock:
const helmet = Helmet.peek()
console.log(helmet)

The following is outputted:
{
      baseTag: [],
      bodyAttributes: {},
      defer: true,
      encode: true,
      htmlAttributes: { lang: 'en' },
      linkTags: [],
      metaTags: [
        { name: 'description', content: 'passed description' },
        { property: 'og:site_name', content: 'site title' },
        { property: 'og:title', content: 'passed title' },
        { property: 'og:description', content: 'passed description' },
        { property: 'og:type', content: 'website' },
        { name: 'twitter:card', content: 'summary' },
        { name: 'twitter:creator', content: 'author' },
        { name: 'twitter:title', content: 'site title' },
        { name: 'twitter:description', content: 'passed description' }
      ],
      noscriptTags: [],
      onChangeClientState: [Function (anonymous)],
      scriptTags: [],
      styleTags: [],
      title: 'passed title | site title',
      titleAttributes: {}
    }

It appears that metaTags needs to be added to the HelmData interface.

I assume this is about the typescript types, vs library usage. The types are not (yet) maintained by this project.
See https://github.com/DefinitelyTyped/DefinitelyTyped/tree/master/types/react-helmet/
Feel free to PR the types to this lib and remove them from DefinitelyTyped

ref: DefinitelyTyped/DefinitelyTyped#52306


Разместить readme.md на верхнем уровне CTAN архива

Для решения #133.
Таково требование CTAN: http://ctan.org/file/help/ctan/CTAN-upload-addendum#uploadpreparation

Every package shall be accompanied by a README file giving a short introduction to the package's purpose as well as significant data like author's name and contact, license etc.
Its name can be either README or README.txt or README.md , but no other variants are permitted at the moment.
Its contents must be easily readable using the UNIX more or less commands or an arbitrary editor.
It should be written in English or at least be headed by a short abstract in English.
The encoding should be ascii or utf-8.
It has to be placed at the top level of the unpacked package (but in the doc/…/ directory of a possible .tds.zip file).

Leave a comment


Chore: add support for uws

Adding support for uws should give a performance boost for websocket operations, and given that it's typically trivial to add support there's not much reason not to unless you use a feature that it doesn't support because it's inefficient.

Isn't uws a node-only module?

This feature has been added in 2.0.0-beta.1.


Nuget support

Hello,
I wish to have a nuget package for this amazing library.
Thank you.

Does not works in asp:Content, does something need to be done first?


Linux: agent-1.x.x.jar not found

Describe the bug

I am using Arch Linux, also I cloned the repository from GitHub. When I try to open the attach window it is disable. I track with logs the secuences and the problems is about the missing file agentPath: /home/wolf/.config/Recaf/agent/agent-1.3.4.jar. I wonder how to generate the agent-1.3.4.jar since I ran all the gradle options?
Arch Linux:
Linux MSIGT73EVR7RF 6.2.8-arch1-1 #1 SMP PREEMPT_DYNAMIC Wed, 22 Mar 2023 22:52:35 +0000 x86_64 GNU/Linux

To Reproduce
Steps to reproduce the behavior:

Navigate to 'File > Attach to remote'
The window with all the Java processes is disabled.

Exception
If applicable, add the exception/stacktrace.
04:39:46.194 [AttachPane/JavaFX Application Thread] INFO : private void extractAgent()
04:39:46.242 [AttachPane/JavaFX Application Thread] INFO : agentPath: /home/wolf/.config/Recaf/agent/agent-1.3.4.jar
04:39:46.258 [AttachPane/JavaFX Application Thread] INFO : Files.isRegularFile(agentPath): false
04:39:46.274 [AttachPane/JavaFX Application Thread] INFO : agentPath.getFileName(): agent-1.3.4.jar
04:39:46.288 [AttachPane/JavaFX Application Thread] INFO : Directories.getAgentDirectory(): /home/wolf/.config/Recaf/agent
04:39:46.297 [AttachPane/JavaFX Application Thread] DEBUG: Extracting agent jar to Recaf directory: agent-1.3.4.jar

Screenshots

Sure, let me check this project https://github.com/Col-E/InstrumentationServer, and try to set up the Col-E/Recaf with the Col-E/InstrumentationServer jars.

Problem in instrumentation server solved, I'll update the 3x snapshot in a little bit to include the changes 👍


Index Search not working

/Lucene/Admin/Query?IndexName=Search
Index Search is not working anymore, since changes in Identity.
/Modules/Lucene/Views/Admin/Query.cshtml

Argument 1: cannot convert from 'int' to 'string'
 29       var contentItemId = Convert.ToInt32(document.GetField("ContentItemId").StringValue);
 30       var contentItem = await ContentManager.GetAsync(contentItemId, VersionOptions.Latest);

Easy to fix by removing the Convert.ToInt32() but are there other changes to check, now ContentId is a string?

ContentItemId is a string now, right, there might be other parts of the code to change. I changed most of it, must have missed some.


Local planner PID controller

Description
We found out that the previous control was overly spiky.

The new parameters

Thanks to @ykarmesh for this finding and contributions.
The new agent is much easier to imitate through behavior cloning.
Where has this been tested?

Platform(s): Linux, CARLA 0.96
Python version(s): latest
Unreal Engine version(s): latest

Possible Drawbacks
I don't know the extension in which the basic agent is used but this might create problems.

This change is

thanks a lot that realy help


Nice work... I also suggest to...

Hi Bruno,
Nice work. Simple to use and works fine on Chrome. I'm not a developer but following your instructions I got this result http://www.wocmultimedia.biz/en/cilavegna-and-italy/
I don't know if it's simple to achieve but I suggest the options of having double title for going to Full Screen and exit from Full Screen, so far you have only one option to set using "title:...."
Thanks
Angelo

Here you go, you can now use titleCancel option for that ;)

thanks. Have a good day Bruno.
Angelo


Remove cytoolz as hard dependency

What was wrong?
Fixes #70
How was it fixed?
Replaced the cytoolz imports with eth_utils.toolz imports
Cute Animal Picture

Good to merge when CI turns green.


[Question] How do I run DataBaseConfig class?

I have been trying to run with rigth click on Android Studio but it just make gradle fail with SourceSets main not found

We are using ormlite as orm for the database. I am unsure what exactly your issue is exactly. Can you maybe post the detailed error message why it is failing? Maybe you can also check the ormlite documentation for help.

I have solved the problem using a grade task. No more problems.


Fix test metrics

@hklai: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/daily-perf-tests.sh
a4ecefabd25ba8aa6344dbd39518f254b4bfb8e2
link
/test daily-perf-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hklai: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/daily-perf-tests.sh
a4ecefabd25ba8aa6344dbd39518f254b4bfb8e2
link
/test daily-perf-tests

prow/daily-upgrade-tests-1.0.1.sh
a4ecefabd25ba8aa6344dbd39518f254b4bfb8e2
link
/test daily-upgrade-tests-1.0.1

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hklai: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/daily-perf-tests.sh
a4ecefabd25ba8aa6344dbd39518f254b4bfb8e2
link
/test daily-perf-tests

prow/daily-upgrade-tests-1.0.1.sh
a4ecefabd25ba8aa6344dbd39518f254b4bfb8e2
link
/test daily-upgrade-tests-1.0.1

prow/daily-upgrade-tests-1.0.2.sh
a4ecefabd25ba8aa6344dbd39518f254b4bfb8e2
link
/test daily-upgrade-tests-1.0.2

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hklai: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/daily-perf-tests.sh
a4ecefabd25ba8aa6344dbd39518f254b4bfb8e2
link
/test daily-perf-tests

prow/daily-upgrade-tests-1.0.1.sh
a4ecefabd25ba8aa6344dbd39518f254b4bfb8e2
link
/test daily-upgrade-tests-1.0.1

prow/daily-upgrade-tests-1.0.2.sh
a4ecefabd25ba8aa6344dbd39518f254b4bfb8e2
link
/test daily-upgrade-tests-1.0.2

prow/daily-e2e-pilot-no_auth.sh
a4ecefabd25ba8aa6344dbd39518f254b4bfb8e2
link
/test daily-e2e-pilot-no_auth

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hklai: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/daily-perf-tests.sh
a4ecefabd25ba8aa6344dbd39518f254b4bfb8e2
link
/test daily-perf-tests

prow/daily-upgrade-tests-1.0.1.sh
a4ecefabd25ba8aa6344dbd39518f254b4bfb8e2
link
/test daily-upgrade-tests-1.0.1

prow/daily-upgrade-tests-1.0.2.sh
a4ecefabd25ba8aa6344dbd39518f254b4bfb8e2
link
/test daily-upgrade-tests-1.0.2

prow/daily-e2e-pilot-no_auth.sh
a4ecefabd25ba8aa6344dbd39518f254b4bfb8e2
link
/test daily-e2e-pilot-no_auth

prow/daily-unit-tests.sh
a4ecefabd25ba8aa6344dbd39518f254b4bfb8e2
link
/test daily-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Metric is fixed according to prow output.
E.g. https://k8s-gubernator.appspot.com/build/istio-prow/pull/istio-releases_daily-release/1197/daily-unit-tests/863/


[BUG] British Grand Prix with wrong "IsAccurate" value for lap 3.

Describe the issue:
If we filter LapNumber = 3 for the British Grand Prix event's laps data, IsAccurate column is set to true for 18 of 20 columns. It should not happen since it was a safety car lap.
Reproduce the code example:
import fastf1
session = fastf1.get_session(2022, 'British', 'R')
session.load()
session.laps.loc[session.laps['LapNumber'] == 3, "IsAccurate"]

Error message:
No response

Hi, can you explain why you think that this was a safety car lap? IMO, the order of events was as follows:
LAP 1 - immediate red flag, no safety car
-- long pause --
LAP 2 - second formation lap from box to grid with safety car in front; seems it doesn't count as safety car lap in the data
LAP 3 - standing start, green flag
Is this correct? In this case, the data is as correct as it can be, given the information available from the API.


Fix get_device_status for newer cast firmwares

See: https://github.com/home-assistant/home-assistant/issues/9965
Newer cast firmwares respond to the http://CAST_IP:8008/ssdp/device-desc.xml request pychromecast uses for manual setup with a 404 Not Found error - and so pychromecast thinks the cast device is unavailable. This PR instead uses http://CAST_IP:8008/setup/eureka_info?options=detail (found here: http://fiquett.com/2013/07/chromecast-traffic-sniffing/), which should be available on all cast devices/firmwares (tested on Chromecast, Chromecast Audio and Google Home).
The only issue is that I can't seem to find the api_version attribute - though it doesn't really seem to be used anywhere, so this shouldn't be a big deal.

In retrospect, in some cases the api_version attribute might be required for some projects - and my current approach doesn't provide that.
So, I have prepared a commit that changes the current behaviour to checking the old way (/ssdp/device-desc.xml) first and then falling back to the JSON (/setup/eureka_info?options=detail) request. If you think that way is better I would be happy to include it.

I'm fine with going with the PR as it currently stands and do a major release to indicate a breaking change. PyChromecast implements the API so a user should not be concerned which API is being used.

Thanks! 🐬

I tested it with a Google Home and Android TV locally and it worked.

Great!


Adding Hello World program in scala

I have worked on Scala before.
I can add a program for Hello World in Scala

Ok fork and make a PR, you can add Hello World Program in any language

closing this issue, as it is resolved already


Lens must be restarted in order to load an extension for the first time

The your first extension docs, as well as the README in the hello world extension, suggest that merely reloading the Lens window is enough to see the extension loaded.
But that's not the case with 4.0 alpha 2: Lens only seems to look at the ~/.k8slens/extensions dir on start-up, and never again.
So when loading an extension for the first time, Lens must currently be restarted (quit the app, start it up again).
Once this is done, and the extension is updated, then it's only necessary to reload the Lens window (i.e. with CMD+R).
If it's a bug that Lens doesn't look for new extensions on window reload, then please fix that. Otherwise, please update the docs accordingly...

Thank you for submitting this issue! It looks like this was fixed in #1230

Thank you for submitting this issue! It looks like this was fixed in #1230


Glue To Shape

I have been looking for a way to connect two shapes together by using the glue feature on Visio.  I was wondering if there is anyway to do that in your library.    Any suggestions?
Thank you,
Andrew

Hi @Andrewjb0813 ,
Hard to say what the problem might be based on that - but you might want to look at test_add_connect_between_shapes() in test_page.py
When I look at output files from that test, the connectors are not perfectly positioned, but each end of the connector is  centered on the 'to' or 'from' shape.
Feel free to share your code and vsdx file here and I will take a look,
Dave


feat: add result page to devcon round branch

Description
Simple display of the tally result at the result page.
Notice
Still need to be e2e tested.
This branch is forked from the feat/devcon-round but rebase main, so it looks like so many commits comparing to the base branch... does anyone have a better way to avoid this? such as merge instead of rebase, or squash commits, or we rebase and force push the feat/devcon-round branch? (but since this feature actually changes lots of parts of the code, that's why I think it should be a separate PR instead of force rebasing it)

@kittybest thanks, just two improvements and I'm good with this PR. Just fix the types ci job (not sure if e2e can be fixed here).

For the types check, I discussed before with @ctrlc03, that this is a problem that might not be able to be solved now, the error message looks like this:

The KernelSmartAccount can not fit in the client type, but the code does work. Seems like they might not be using permissionless package afterward.


stars: Fix flaky test due to message list not yet rendered.

With combined feed no longer cached, this test became flaky as it took longer for combined feed to load.
Verified that other tests waiting on .message-list don't need this change.
discussion: https://chat.zulip.org/#narrow/stream/43-automated-testing/topic/flaky.20star.20message

Makes sense, thanks for tracking this down!


Add BuG_17's baoyan doc

Have run update.py and the project locally.

Thanks for your contributions!


target specific browser and give them specific rules ?

Hi guys,
I cant find a way to target for example Firefox and give it a specific rules.
My problem is this : I am using react and styled components for all the styling. I am using chrome as my browser and when I look at Firefox there are some stuffs that are not correct. So I need to add som rules, but I only want to add those rules if it is Firefox.
Any suggestions on how to approach this ?

This has nothing to do with styled-components, it's a CSS question! Maybe try something like https://stackoverflow.com/questions/952861/targeting-only-firefox-with-css?


Question: Patterns Matching seems misleading to me

just personal opinion. The Patterns matching seems misleading to me.

I agree, do you have an idea how else to present or explain it ?

Maybe something like that:
You can listen/peek a channel and at the same time pattern match only a given process structure
 for (@"hello" <- ch1; @("hello", "world!") <- ch2; @("hello", _) <- ch3) { ... }

You can also match processes directly
match P {
"hello" => { ... }
"world" => { ... }
_ => { ... }
}

Maybe something like that:
You can listen/peek a channel and at the same time pattern match only a given process structure
 for (@"hello" <- ch1; @("hello", "world!") <- ch2; @("hello", _) <- ch3) { ... }

You can also match processes directly
match P {
 "hello" => { ... }
 "world" => { ... }
 _ => { ... }
}```

This looks better to me.
I think the misleading point in the image is that
"Tries to match Process against each Pattern_t until it finds a match"
This sentence would give people wrong impression that the join consume would just iterate the channel with different pattern which is totally not  the case.


[release-1.8] cert template remove SignatureAlgorithm

This is an automated cherry-pick of #27239

cc @cocotyty We need your consent to put this in 1.7

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-multicluster-tests_istio_release-1.8
ce27e88c98a31eaf73259f629eea3fdec9815406
link
/test integ-pilot-multicluster-tests_istio_release-1.8

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-multicluster-tests_istio_release-1.8
ce27e88c98a31eaf73259f629eea3fdec9815406
link
/test integ-pilot-multicluster-tests_istio_release-1.8

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


[bug]: lnd not resigning from leader role when disconnected from cluster

Background
In an lnd cluster, when the leader lnd node and its etcd endpoint lose their network connection to the rest of the cluster, another node takes over. So far so good. When the disconnected lnd node and its etcd endpoint are reconnected to the network, the lnd node continues to run. This results in two lnd nodes running in a cluster at the same time, both accessing the same database simultaneously. This might potentially lead to a loss of funds.
Your environment

lnd version 0.18.1-beta
Operating system: Debian containers on Proxmox VE (Linux kernel 6.8.4-2-pve)
bitcoind version v25.0.0
Network: regtest
LXC containers:

bitcoind
lndetcd1
lndetcd2
lndetcd3

Steps to reproduce
Set up a bitcoind regtest node with the following configuration:
regtest=1

[regtest]
server=1
txindex=1
disablewallet=0
peerbloomfilters=0
rpcuser=user
rpcpassword=password
rpcport=8332
rpcallowip=<IP_ADDRESS>/0
rpcbind=<IP_ADDRESS>:8332
zmqpubhashblock=tcp://<IP_ADDRESS>:29000
zmqpubrawtx=tcp://<IP_ADDRESS>:29001
zmqpubrawblock=tcp://<IP_ADDRESS>:29002
listen=1

Create 3 Debian containers named "lndetcd1", "lndetcd2" and "lndetcd3".
Install etcd on each of the containers
apt install etcd-server etcd-client

Add the following to the /etc/default/etcd file on each container:
ETCD_NAME=etcd1
ETCD_INITIAL_CLUSTER="etcd1=http://${IP_OF_LNDETCD1}:2380,etcd2=http://${IP_OF_LNDETCD2}:2380,etcd3=http://${IP_OF_LNDETCD3}:2380"
ETCD_ADVERTISE_CLIENT_URLS=http://${OWN_IP}:2379
ETCD_LISTEN_CLIENT_URLS=http://<IP_ADDRESS>:2379
ETCD_LISTEN_PEER_URLS=http://<IP_ADDRESS>:2380
ETCD_INITIAL_ADVERTISE_PEER_URLS=http://${OWN_IP}:2380
ETCD_MAX_TXN_OPS=16384

Change ETCD_NAME according to the respective container name.
Then start etcd on each container
systemctl start etcd.service

Check that the cluster is healthy
etcdctl endpoint health --cluster

Now install lnd on each of the containers with the following configuration changing the cluster.id accordingly:
[Application Options]
listen=<IP_ADDRESS>:9735
alias=lndetcd

[Bitcoin]
bitcoin.regtest=1
bitcoin.node=bitcoind

[Bitcoind]
bitcoind.rpcuser=user
bitcoind.rpcpass=password
bitcoind.rpchost=${IP_OF_BITCOIND}:8332
bitcoind.zmqpubrawtx=tcp://${IP_OF_BITCOIND}:29001
bitcoind.zmqpubrawblock=tcp://${IP_OF_BITCOIND}:29002

[db]
db.backend=etcd

[etcd]
db.etcd.host=<IP_ADDRESS>:2379
db.etcd.disabletls=1

[cluster]
cluster.enable-leader-election=1
cluster.leader-elector=etcd
cluster.etcd-election-prefix=cluster-leader
cluster.id=lndetcd1

Start lnd on one of the containers first and setup a wallet, then start lnd on the other containers.
Now that everything is set up, disconnect the container running the current lnd leader node from the network.
Expected behaviour
The lnd node should lose its leader status and crash or shut down.
Actual behaviour
The leader will keep running unable to access the database, since it does not have access to the majority of the cluster.
After about 1 minute, a different lnd node in the cluster takes over.
When the disconnected container is reconnected to the network, there are two active lnd nodes accessing the same database at the same time. This might potentially lead to a loss of funds.

Currently the leader LND resigns only if it is shutting down. You can also follow this in the logs: https://github.com/lightningnetwork/lnd/blob/fdd28c8d888792ea8fde3c557ba9f2594e0a6ec8/lnd.go#L407
IIUC you're expecting that we detect network disconnect and resign? Could you simply just shut down the container?

IIUC you're expecting that we detect network disconnect and resign?

Yes, that would be the desired effect.

Could you simply just shut down the container?

Then I would need to have some script running in the background all the time, checking that the container still has a network connection. And even then this would not be 100% reliable since there could be some unexpected network problem. (e.g. duplicate IP address, misconfigured firewall, ARP cache issues, routing issues...) In such a case, where the container would be connected to the network, but still unable to communicate with the majority of the cluster, it could still result in two leaders.

I hacked together a script that works around this issue.
The following sh script has to be started before lnd is started:
#!/bin/sh

LND_CONF_FILE="${HOME}/.lnd/lnd.conf"

lnd_cluster_id=$(awk -F "=" '/cluster.id/ {print $2}' $LND_CONF_FILE)
election_prefix=$(awk -F "=" '/cluster.etcd-election-prefix/ {print $2}' $LND_CONF_FILE)

echo "cluster.id: $lnd_cluster_id"
echo "cluster.etcd-election-prefix: $election_prefix"

fifo=$(mktemp -u)
mkfifo "$fifo"

etcdctl watch --prefix "$election_prefix" --write-out=json > "$fifo" &
cmd_pid=$!

while IFS= read -r line <&3; do
    key_base64=$(echo "$line" | jq -re .Events[].kv.key)
    has_key=$?
    value_base64=$(echo "$line" | jq -re .Events[].kv.value)
    has_value=$?

    if [ "$has_key" -eq 0 ] && [ "$has_value" -eq 0 ]; then
        key=$(echo "$key_base64" | base64 --decode)
        value=$(echo "$value_base64" | base64 --decode)

        if [ "$value" = "$lnd_cluster_id" ]; then
            kill "$cmd_pid"
            lease_id="${key#*/}"
            echo "Found lease_id=$lease_id for $lnd_cluster_id"
        fi
    fi
done 3< "$fifo"

wait "$cmd_pid"

while true; do
    sleep 5

    etcdctl lease timetolive "$lease_id" --write-out=json > "$fifo" 2>&1 &
    cmd_pid=$!

    while IFS= read -r line <&3; do
        ttl=$(echo "$line" | jq -re .ttl)
        has_ttl=$?
        kill "$cmd_pid" 2> /dev/null
    done 3< "$fifo"

    if [ "$has_ttl" -ne 0 ]; then
        echo "Failed to get ttl"
        break
    fi

    if [ "$ttl" = "-1" ]; then
        echo "Lease expired"
        break
    fi

    echo "Lease ttl=$ttl"
done

rm "$fifo"

kill -9 $(systemctl show --property MainPID --value lnd.service)
echo "Killed lnd"

Every 5 seconds it checks whether the lease has expired. If it has, it kills the lnd process. This ensures that there is never more than one lnd leader node running.
To ensure that lnd is always started and stopped together with this script, I used a combination of Requisite=, After= and StopPropagatedFrom= when configuring the systemd services.
lnd.service:
[Unit]
Description=LND Lightning Daemon
Requisite=etcd.service
Requisite=lelem.service
After=etcd.service
After=lelem.service
StopPropagatedFrom=etcd.service
StopPropagatedFrom=lelem.service

[Service]
ExecStart=/usr/bin/lnd

User=bitcoin
Group=bitcoin

Restart=always
RestartSec=30

Type=notify

TimeoutStartSec=infinity
TimeoutStopSec=1800

ProtectSystem=full
NoNewPrivileges=true
PrivateDevices=true
MemoryDenyWriteExecute=true

[Install]
WantedBy=multi-user.target

lelem.service:
[Unit]
Description=LND ETCD Lease Expiry Monitor
StopPropagatedFrom=lnd.service
Upholds=lnd.service

[Service]
ExecStart=/usr/local/bin/lelem.sh

User=bitcoin
Group=bitcoin

Restart=always

Type=exec

ProtectSystem=full
NoNewPrivileges=true
PrivateDevices=true
MemoryDenyWriteExecute=true

[Install]
WantedBy=multi-user.target

This actually works. But ideally we shouldn't need this hack and lnd should handle this internally.

Discussed this issue earlier today with @bhandras and we have a solution in mind, thanks for bringing this to our attention!


Benchmark Tests for FNO and DeepONets

To fix #13

[x] comparison with neuraloperators
[x] comparison with deepxde
[x] comparison with Flux variant
[ ] GPU comparisons

I had some issues with CUDA and all when installing torch CUDA toolkit.

I guess a more appropriate place for this would be SciMLBenchmarks?

Yes the CPU ones can go to SciMLBenchmarks. How are the julia native ones so slow? Did you run the profiler to see where the bottlenecks are?

For DeepONet, most of the time goes to the layers, iiuc

For FNOs, ig ffts are a bit expensive

Couple of things to check:

Load MKL.jl before calling the neural networks
Is the number of threads consistent between julia and python?

Also that compile function might be compiling the torch model with dynamo. It is pretty much impossible to beat that with Lux running in eager mode. You could try and compile the DeepONet model with https://github.com/EnzymeAD/Reactant.jl/pull/55 and see the performance.

Also looking at the plots you are on quite an old version of LuxLib, update it, and it should address some performance

@ayushinav can you install https://github.com/LuxDL/LuxLib.jl/pull/111 and let me know how the performance is?

I checked the recent lux releases. The current problems are

__project isn't fast enough. This needs to be rewritten to use LoopVectorization on CPU
batched_mul -- I am looking into this in LuxLib, this is quite easy to fix
allocations are expensive. Nothing much we can do there honestly.

Can we make this into a SciMLBenchmarks script? That will be easier to maintain in the long run.
We can make that support GPU

Also for python setup a pyproject.toml file so that setting up the benchmarks is easier

Haven't looked into the FNO version much, but that will most likely need https://github.com/LuxDL/LuxLib.jl/issues/118 for performance. To summarize the issue there:

gelu and friends are surprisingly expensive operations, and fusing them into the fused_dense operations slows down the entire loop. But that is not true in general. For example, tanh/relu/abs, etc. can and should be fused into the main loop for performance. So we need to extend the current implementation with additional traits to see which activations can be fused for performance.
But the main perf gain will come from better gradient ops -- even for gelu. Currently, we fail to use LoopVectorization in that case but if we hardcode some of the common cases the performance will significantly improve.

@ayushinav can we get this finished?

@ayushinav bump

@avik-pal
Yes, I'm on this. I need to write the gradients for the new  __projects using LoopVectorization. The projection becomes faster then, but there isn't a significant speed up for the overall network.

Do that in a separate PR, let's get the benchmarks aligned first. The pytorch ones use a different size it seems.

The sizes are now aligned. The python variant of DeepONet only supports 1 eval point in a batch, and the Flux variant doesn't support batching. To have the same size for inputs, I made the batch size and the eval points same to compare with both the variants.
The Flux variant of FNO only supported a fixed length of kernels, which is fixed here.
The difference in size now is because python uses (batch_size, N) whereas julia uses (N, batch_size)

I am guessing the overhead in FNO is currently from fft?

Fair share of fft and matmuladd, though the share of later has increased

Can you also profile the backward pass for the FNO? I am surprised it is that bad

For mse loss, profiling the backward as gradient(ps -> loss(first(model(x, ps, st)), y), ps)

using the permuted formulation it is now all just fft time in forward and backward. It is quite surprising that our FFT is so much slower than pytorch.
@ayushinav might be worth giving https://github.com/Taaitaaiger/RustFFT.jl a shot and checking the  performance on CPUs


Flow is not checking condition

I'm running flow 0.18.1, and simple conditions aren't being checked. For example:
/* @flow */

type T = {prop: number};

const val: T = {prop: 5};

if (val.nonExistentProp) {
  console.log(val.prop);
}

I'd expect a type error on nonExistentProp. If I change it to if (val.nonExistentProp()) or if (val.nonExistentProp + 2), flow correctly notices that there is no such property.
Thanks for any help! Let me know if I can provide any other info.

This is actually intended behavior. See earlier discussion in #698.

Thanks for your quick response - that's always helpful!


Failed to start WebServer[Errno 98]

I have a problem with web server. I try my best and follow instruction. I have used free account on pythonanywhere, but I am on dead end with this message:
Failed to start WebServer[Errno 98] Address already in use
I appreciate help.

Basically it means the PORT is already in use, maybe you have some other app using the port (maybe the bot is already running?).  You can also change the Port in the config file.

Actually pythonanywhere_com_wsgi.py is running with "hello world application" because I have choosen manual configuration of web site. How to stop this python app, and how to make configuration for web on pythonanywhere_com because I have above picture on static page:
Thanks in advance

@sestoculo  I haven't used python anywhere, maybe ask on our Gitter? https://gitter.im/Mikadily/poloniexlendingbot

The wsgi is just a config file... You do not even need to use it.
If you are following the instructions from https://poloniexlendingbot.readthedocs.io/en/latest/installation.html#creating-the-web-app-optional then you do not need to use the startwebserver option in the bot config.

@sestoculo is the issue resolved?

no, I invest some time, but I did not resolve my problem. I have response of the web site like in picture above an I have message: "Failed to start WebServer[Errno 98] Address already in use". I am ok with that, I do not have web page about bot, but bot works good.

I actually noticed this too, but haven't spent any time on it.
It happens sometimes after you shut down the bot and restart it shortly after, it looks like it's not releasing the port properly. Though netstat -tnlp and lsof show no sign of it being used.
If you leave it for a minute and restart, the error doesn't appear.


codeCTRL expects u32 as json key, which is not supported in python.

For the code_snippet map codeCTRL seems to be looking for integers as keys. (I am not actually sure this is what is happening, but it seems like it from errors.) Python (and many other languages) unfortunately do not allow this, and thus it would be really hard to send this payload.
Would it be possible to either send strings as keys or send the code_snippet as an array of maps with a line_number and a code key.
Json that I was sending from python:
{
    "message": "3  ",
    "message_type": "<class 'int'>",
    "line_number": 4,
    "code_snippet": {
        "2": "",
        "3": "",
        "4": "codectrl.log( 3)",
        "5": ""
    },
    "file_name": "/home/antone/pwnCTRL/software/codectrl-python-logger.git/main/a.py",
    "stack": [
        {
            "name": "<module>",
            "code": "codectrl.log( 3)",
            "file_path": "/home/antone/pwnCTRL/software/codectrl-python-logger.git/main/a.py",
            "line_number": 4,
            "column_number": 0
        }
    ],
    "warnings": [],
    "address": "<IP_ADDRESS>"
}

Error:

This should have been fixed already? Can you confirm that you're on the latest commit?

oh, errm the default branch is main, I was using that hahah


Add rails to transitioning section

Adapts Lukas' blog post

Moved the content inside MT transitioning in subsection "App Migration (Ruby on Rails)"

@sumedhpathak this issue got moved back to "Please Revise" but I think the previous review had been addressed actually. Can you take a look?

@begriffs Could you trigger a Heroku deployment? Just wanted to review how the current placement looks.

Looks good! 🚢


[google_maps_flutter] Plugins Repo is 0.5.20+1 but NOT available from pub.dev

pubspec.yaml of the google_maps_flutter package clearly shows the new version, but pub.dev has NOT been updated with this new version and is therefore unavailable to me.
Expected behaviour:

Flutter plugin version s available on pub.dev match the version in the plugin repo
Latest versions are available from pub.dev

@iskakaushik

@workerbee22 Done. Sorry for the delay here.

Thanks for the quick fix @iskakaushi. We can always reference the repo from pubspec.yaml, but thought I'd better point it out for others.


Pinging @everyone?

Hi, some of the ways I would like to integrate this are to have it ping everyone on certain messages. However it seems that even though I send a message with @everyone it shows up with the highlight around the word but doesn't actually ping users and the entire message doesn't turn orange as if someone sent it.
Example: https://imgur.com/a/9NM2qzV

This is a Discord limitation, mentions in embeds are not counted as pings. There are ways to make invisible-everyone pings in webhooks which I could look into for a future release, but for now, I'll be closing this as the question does not have an immediate fix.

@jos-b What do you think about the fix?
This function is really needed.

@MrZillaGold Please read above. This is a discord limitation, not mine.

If you set the text of the message to [](@everyone) then the webhook will invisibly ping everyone.


Add support for enum

Fixes #281 .
Changes proposed in this pull request:

Add support for enum types

Attention: @prkumar

Hello @prkumar
Thanks for your response, i just check it and i don't know where do you want to add this conversion.
I see that the property converter_key on Query and QueryMap returns a Sequence class so i think  this is the better location to make the convertion.
Please let me know where do you think i could move it.


wavlm extract features failed in loop

Describe the bug
Model I am using WavLM, follow the scripts in the wavlm's README.txt, (I modified to gpu device, I have tried cpu mode, the same memory exception.)
import torch
from WavLM import WavLM, WavLMConfig
device = torch.device("cuda:0" if torch.cuda.is_available() else "cpu")

# load the pre-trained checkpoints
checkpoint = torch.load('/path/to/wavlm.pt')
cfg = WavLMConfig(checkpoint['cfg'])
model = WavLM(cfg).to(device)
model.load_state_dict(checkpoint['model'])
model.eval()

# extract the representation of last layer
wav_input_16khz = torch.randn(1,10000).to(device)
rep = model.extract_features(wav_input_16khz)[0]

works well, but while I run it in a loop, it raised memory error.
......
loop = []
for i in range(10):
    rep = model.extract_features(wav_input_16khz)[0]
    loop.append(rep)

I don't think it is a bug related to the model. Most of your GPU memory is already used and by appending all of the representations in a single list, it fails to allocate the memory.
My suggested solutions:
1- dump each output representation in your hard dist rather than appending them together in a list.
2- if you are using the large model, switch it to the base or base+ model. The less number of parameters leads to less memory occupation.
3- Switching the processor to CPU will make the inference slower but may fix the error.

I tried a lot, and find the answer.
Reference from https://github.com/microsoft/UniSpeech/tree/main/downstreams/speaker_verification.
We should add with torch.no_grad(): when extracting features.
For example:
with torch.no_grad():
    feature = <<extract_feature_model>>(audio)

In your case, fix it with:
......
loop = []
for i in range(10):
    with torch.no_grad()
         rep = model.extract_features(wav_input_16khz)[0]
    loop.append(rep)

hope this could work.

RobertBoganKang's answer seems to work. Thanks.


[BUG] CRASH WHEN I START THE GAME

When i start the game, LeagueBroadcast crashs

OS: [e.g. Windows 10]
Version 1909

AFAIK the Windows Version is too old. Could you attach a Crash Report?
Its located here LeagueBroadcast\Logs\CrashLogs, then the file with the timestamp of the crash

I have the same issue
Log-2022-02-15-14-42.log

[14-42-15] [WARNING] App Crash detected. Writing Crash report to CrashLogs\Crash-2022-02-15-14-42.log
Could you send the crashlog instead please?

here we g
Crash-2022-02-15-14-42.log
o

Looks like a corrupted config. Try deleting farsight.json


Expose API for DataChannel

is there any api yet for it?

@benoitc there is a C-level API in OpenWebRTC, but it needs to be exposed up to the iOS SDK (Obj-C).


cmd/go/internal/modfetch: re-evaluate the undocumented proxy behavior in proxyRepo.latest()

Forked from the discussion during the review of https://golang.org/cl/183402

undocumented /@v/list format

proxyRepo.latest() assumes /@v/list could return a list of version, commit timestamp pairs. This is not documented in the proxy protocol doc, and I am not sure if there is any proxy that's following this format. Please reject the undocumented format or document it.
One of the benefit of preserving this feature is, when this proxyRepo.latest() is invoked, the commit timestamp info here is more reliable than the timestamp parsed from the pseudo-version like version string and helps avoiding an extra proxyRepo.Stat() call.  On the other hand, the code path can be completely eliminated if proxies implement the /@latest endpoint, which is also not documented.

whether to include pseudo-versions or not in /@v/list

The proxyRepo.latest() itself is also relying on part of undocumented behavior of /@v/list, which is supposed to include pseudo-versions. Before https://golang.org/cl/183402 (pre-1.13), we assumed it's not a good idea to include the pseudo versions in the /@v/list output and planned to include that in the proxy protocol spec.
https://go-review.googlesource.com/c/mod/+/176540/5/proxy/server.go#40
// List returns a list of tagged versions of the module identified by path.
// The versions should all be canonical semantic versions
// and formatted in a text listing, one per line.
// Pseudo-versions derived from untagged commits should be omitted.
// The go command exposes this list in 'go list -m -versions' output
// and also uses it to resolve wildcards like 'go get<EMAIL_ADDRESS>
If the spec draft is valid, the proxyRepo.latest() implementation doesn't do anything but an extra network round trip for the proxies that follow the spec. If the go command can handle the list output including pseudo-versions, we may want to revisit the decision.
So, I propose either

tighten the spec to require the /@latest endpoint. Then, remove the code that leads to the proxyRepo.latest(), or
relax the /@v/list format and allow pseudo versions in its output, and document it.

Option 2) implies more code to maintain on cmd/go side. Option 1) implies breakage of proxies that doesn't implement the /@latest endpoint.
p.s. having the go command's proxyRepo.Versions ensure not to return pseudo versions is  a good fix though.

s/also not implemented/also not documented/?

whether to include pseudo-versions or not in /@v/list

Just as an observation, the last paragraph at https://golang.org/cmd/go/#hdr-Module_proxy_protocol says:

Even when downloading directly from version control systems, the go command synthesizes explicit info, mod, and zip files and stores them in its local cache, $GOPATH/pkg/mod/cache/download, the same as if it had downloaded them directly from a proxy. The cache layout is the same as the proxy URL space, so serving $GOPATH/pkg/mod/cache/download at (or copying it to) https://example.com/proxy would let other users access those cached module versions with GOPROXY=https://example.com/proxy.

If one looks inside the local module cache as produced by cmd/go, pseudo-versions are included in /@v/list output. E.g.:
$ cat $(go env GOPATH)/pkg/mod/cache/download/golang.org/x/text/@v/list
v0.3.0
v0.3.1-0.20180807135948-17ff2d5776d2
v0.3.1-0.20181227161524-e6919f6577db
v0.3.2

undocumented /@v/list format

I think we should confirm no proxies are using extra fields. If there are none, we should start rejecting them in 1.14.
Currently, it looks like this can be used to set timestamps. We only really need timestamps for ordering pseudo-versions, and the timestamp should match the timestamp encoded in the pseudo-version, so there's no need to specify it separately.

whether to include pseudo-versions or not in /@v/list

CL 190838 updates go help goproxy to include /@latest. It also points out that /@v/list may include pseudo-versions.
Note that go1.13 treats /@v/list differently from go1.12 and earlier.
In go1.13, if /@v/list contains tagged versions and pseudo-versions, the pseudo-versions will only be considered if all tagged versions (including pre-release) are excluded.
In go1.12 and earlier, if /@v/list contains tagged versions and pseudo-versions, the pseudo-versions will be considered together with tagged pre-release versions (non-pre-release versions still take precedence).
So if a proxy stopped serving /@latest and instead included that version in /@v/list (if not already present), it would change behavior of go1.12 and earlier if that version is a pseudo-version newer than pre-release versions in /@v/list. I don't think that change would affect proxy.golang.org since a pseudo-version is only returned from /@latest if /@v/list is empty.


new one stopped working

i cant open it

I can open the program and it runs just fine, but the fps is not unlocked in-game.


Issue #301. Serialize accessor min/vax value as int

This will fix #301
@AurL Could you please conform this PR fixes your issue?

I just did a quick test and it's ok now, the validator is happy 👍
Thanks a lot for the quick fix!

I just did a quick test and it's ok now, the validator is happy 👍
Thanks a lot for the quick fix!

@AurL Thanks for the test! Merged!

@AurL Thanks for the test! Merged!


Feature/ct topic

Associated Template/Command/Core

Create and config
PR Checklist

[x] This comment contains a description of changes (with reason)
[x] Referenced issue is linked

Description of changes

Projects can now be created with a cookietemple topic (opt-in, configured via the config command).

Gotta update WFs


+'s internal to compounds should be ignored for classification

Just as we disregard the number of +s at the end of a sign for purposes of classification, so we should disregard extra +s at the end of a part of a compound.
So these two are really instances of the same compound, COP+RESEARCH:

Also, they should be disregarded for distinguishing variants of a given compound - both of these should be
grouped under
 BOY+CORRECT

Fixed


Training & Genrate ROC curve

Hai @ekosman
python TrainingAnomalyDetector_public.py --features_path "path-to-dataset" --annotation_path "path-to-train-annos"
python generate_ROC.py --features_path "path-to-dataset" --annotation_path "path-to-annos" --model_path "path-to-model"
Can anyone explain how the training process works and give Example for above mentioned comment
Thank you

Hello @Chuttyboy
Thanks for your question, I'll add more information about these arguments in the README.md file.
Meanwhile:

path-to-dataset is the path to the directory containing the pre-computed features
annotation_path is the path to the annotations file (.txt extension. Train_annotations.txt for training, Test_annotations.txt for testing.
model_path is the path to the trained anomaly detection model (you can download a pre-trained model from exps/models.

Hope this helps.

Hai @ekosman
ModuleNotFoundError: No module named 'PyQt5.QtMultimedia'
While running Demo File I got this error how can I overcome this issue?

Can you copy and paste the full error log?

Can you copy and paste the full error log?

Same Error I too face

issue 2:

This issue happens even extracting all 1950 videos

I'll try to reproduce this error. Meanwhile, can you show me that command you are trying to run?

Issue 3: Training
In this part, I use pre-computed features
python TrainingAnomalyDetector_public.py --features_path feature --annotation_path ./Videos/Anomaly-Pytorch/Train_Annotation.txt
`Feature doesn't exist: feat/Training_Normal_Videos_Anomaly/Normal_Videos535_x264.txt
2021-07-27 15:33:43,694 VideoIter:: ERROR!! (Force using another index:
4189)

Feature doesn't exist: feat/Training_Normal_Videos_Anomaly/Normal_Videos386_x264.txt
2021-07-27 15:33:43,700 VideoIter:: ERROR!! (Force using another index:
8872)`

It is strange, because you pass feature as an argument and then the loader tries to load features from a directory called feat... What the the name of the directory which contains all the computed features for the UCF dataset?

Hai @ekosman
I downloaded pre-computed features as per the readme file and that issue was resolved but after running any command ( Feature extraction, Training, demo ) I faced illegal instruction (core dumped ) error
How to resolved it?
Thank you

Sorry, This error does not reproduce on my machine.