Error:  sent 1002 (protocol error); no close frame received with websockets

Description
Getting the error Error:  sent 1002 (protocol error); no close frame received with websockets while running the camera feed from ESP32 to web.
Note

Not getting this error at all the time, but some times.
The above error was displayed in middlware's console.

Solutions found

StackOverflow - WebSocket Closes with Protocol Error 1002

Extracted conclusion: Need to change the ESP32 websockets from synchronous to asynchronous. But earlier when using AsyncWebServer as in HashIncludeElectronic's reference, got the issue in bridging. Try once again, with this setup.

reg.

Need to change the ESP32 websockets from synchronous to asynchronous.

No need to change the library for this. As the current library ArduinoWebSockets, also supports operating in Async mode too.
Look into this issue of its repo - https://github.com/Links2004/arduinoWebSockets/issues/119

as per above comment

To do: in WebSockets.h.

Guess, commenting out the L124 and uncommenting the L125.
Hoping that, this is what the extract of the messages mentioned in the issue.

Experiment out and report the status here.

Refer to the docs of websockets


Password should be of type Secret

Hi,
Thank for your module.
Currently password is handled as a string - dagger exposes a type dagger.Secret.
Using dagger.Secret would hide the password in logs.
Would it be possible to change it to type dagger.Secret?

Hello.
Thanks for your proposal!
I'll change it to a Dagger Secret.

@Jaxwood The new version v1.0.0 of the module now uses a secret for the password.
I also changed to the stdin method for helm registry login.
Moudle: https://daggerverse.dev/mod/github.com/puzzle/dagger-module-helm/helm@03b5661b7c1c49318f9546a93d349bccdb88894d


s3-credentials: 0.13 -> 0.14

Description of changes
https://github.com/simonw/s3-credentials/releases/tag/0.14
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

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

2 packages failed to build:

s3-credentials (python310Packages.s3-credentials)
python39Packages.s3-credentials

error: builder for '/nix/store/yizgc48718wwbll3adxbmszx7x0n9yv5-python3.10-s3-credentials-0.14.drv' failed with exit code 4;
       last 10 log lines:
       > pythonRemoveBinBytecodePhase
       > pythonImportsCheckPhase
       > Executing pythonImportsCheckPhase
       > Check whether the following modules can be imported: s3_credentials
       > pytestCheckPhase
       > Executing pytestCheckPhase
       > ImportError while loading conftest '/build/source/tests/conftest.py'.
       > tests/conftest.py:5: in <module>
       >     from moto import mock_s3
       > E   ModuleNotFoundError: No module named 'moto'

Result of nixpkgs-review pr 191420 run on aarch64-darwin 1

2 packages built:

s3-credentials (python310Packages.s3-credentials)
python39Packages.s3-credentials

Thanks @totoroot, I've updated with a resolution to the error you were getting.


Allow configuring stack test command

This PR allows overriding the test command for Haskell's stack. Specifically, it allows you to use test . as the default command, which works better in multi-package stack projects.

Whoops sorry about that, I'll take a look now.

I'm actually not sure why "test" was quoted only in the zero-argument case. I think this is incorrect. Are you ok with me updating this test to remove the quotes?

I've updated the test, let me know if this works for you.

I've updated the test, let me know if this works for you.

Thank you for quickly sorting this. Could you please  add a scenario to the spec where g:test#haskell#stacktest#test_command is overriden and then I think this is good to merge in.

@codeinabox done


[Benchmark] tutorial report-out seems not correct

The performance report shows too high matmul performance (in TFlops). The TFlops keep growing even when the problem sizes are already very big, which seems indicate the time being measured deosn't grow when the problem size grow.
The root cause seems that we are measuring the host time using  datetime, not the kernel.  It is likely the host doesnt' wait for the device kernel complete so we measure the host drive time, which is more or less constant comparing to kernel time,  so we got very high artificial TFlops when problem becomes bigger.   https://github.com/intel/intel-xpu-backend-for-triton/blob/llvm-target/python/triton/testing.py#L148
**  Below is an excerpts from CI report on the tutorial test. These are from the two matmul tutorial test cases.
Note that the cuBLAS perfomance is really the torch default performance using onDNN
https://github.com/intel/intel-xpu-backend-for-triton/actions/runs/7834541571/job/21377998423
matmul-performance:
M     N     K  TRANS_A  TRANS_B       cuBLAS      Triton
0  2048   512   512    False     True    51.177268    8.263486
1  2048  1024  1024    False    False   178.360381   34.643074
2  2048  2048  2048     True    False   637.677823  122.546929
3  2048  4096  4096     True     True  2853.766101  558.586000
matmul-performance:
M       N       K       cuBLAS       Triton
0  2048.0   512.0   512.0    42.891425     7.942856
1  2048.0  1024.0  1024.0   159.419456    32.283868
2  2048.0  2048.0  2048.0   571.885667   144.403996
3  2048.0  4096.0  4096.0  2745.051201   558.586000
4  2048.0  8192.0  8192.0  9450.176267  2033.371260
https://github.com/intel/intel-xpu-backend-for-triton/blob/llvm-target/python/tutorials/09-experimental-tma-matrix-multiplication.py
https://github.com/intel/intel-xpu-backend-for-triton/blob/llvm-target/python/tutorials/10-experimental-tma-store-matrix-multiplication.py

The use of datetime  in https://github.com/intel/intel-xpu-backend-for-triton/blob/llvm-target/python/triton/testing.py#L148 was introduced by https://github.com/intel/intel-xpu-backend-for-triton/pull/363/files#r1467799172

@etiotto  Is it right that we need to move the end_time after synchronize() ?
        start_times[i] = datetime.now()
        fn()
        //end_times[i] = datetime.now()
    # Record clocks
    if torch.cuda.is_available():
        torch.cuda.synchronize()
    elif torch.xpu.is_available():
        torch.xpu.synchronize()
    end_times[i] = datetime.now()

@etiotto Is it right that we need to move the end_time after synchronize() ?
        start_times[i] = datetime.now()
        fn()
        //end_times[i] = datetime.now()
    # Record clocks
    if torch.cuda.is_available():
        torch.cuda.synchronize()
    elif torch.xpu.is_available():
        torch.xpu.synchronize()
    end_times[i] = datetime.now()

@jinz2014 let me give that a try to see what effect that would have on the performance numbers computed.


Rework legend

[x] Logscale
[x] Color Description
[x] Show/ Hide Buttons
[x] Filtering on low volume

@theimo1221
Will take care of this and also integrate into the README asap.
I think only thing that would be helpful to have from your perspective on this is a description of how the low volume orders are filtered out for the final visualization. Then I can write up a plain english version for the readme.

All issues taken care of and integrated into the legend.


Or 1565 thanos cannon update module name

Update go module from "github.com/ethereum-optimism/optimism" to "github.com/tokamak-network/tokamak-thanos"

Thank you!

I am not sure but we should check with the LICENSE from the Optimism team
https://github.com/ethereum-optimism/optimism/blob/master/LICENSE

On my side, I think we can modify the module name. But I think we should keep this PR under review. After Theo's PR is completed and merged. We will take a look carefully and merge it.
Because of two PRs have a lot of changes, Theo's PR has changed more than this PR and is more complicated.

I am not sure but we should check with the LICENSE from the Optimism team https://github.com/ethereum-optimism/optimism/blob/master/LICENSE

Optimism uses MIT license. So, I think we can and use modify the source code freely but we should keep the LICENSE


Import MissForest / Typo in readme

importing MissForest with:
from missforest.missforest import
Leads to an Error:
`---------------------------------------------------------------------------
ModuleNotFoundError                       Traceback (most recent call last)
Cell In[2], line 23
15 from sklearn.preprocessing import StandardScaler, OrdinalEncoder, OneHotEncoder, LabelEncoder
17 # Small fix to make missingpy forward compatible
18 # import sklearn.neighbors._base
19 # sys.modules['sklearn.neighbors.base'] = sklearn.neighbors._base
20
21 # from missingpy import MissForest
---> 23 from missforest.missforest import MissForest
ModuleNotFoundError: No module named 'missforest.missforest'
`

The correct way to import MissForest is:
from missforest.miss_forest import

Below code shall solve the issue:
import sklearn.neighbors._base # this is from `scikit-learn` instead of `sklearn`
import sys
sys.modules['sklearn.neighbors.base'] = sklearn.neighbors._base
from missingpy import MissForest # remember to put this after sys.modules


Editing pass on section 1

I tried to just fix obvious typos and inconsistencies rather than suggesting any larger changes.

Thank you very much for the edits!


Feat: Automated Closure Comments

Describe the feature
Implement an automated comment system that triggers upon the closure of an issue. This system would post a standardized comment on the closed issue, providing closure to the reporter and potentially offering additional resources or guidance.
Add ScreenShots
NA
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC'24 contributor
[X] I want to work on this issue

please assign this issue to me sir


BUG: Jira StopWatch encountered an unhandled error.

What happened?
Jira StopWatch encountered an unhandled error. A logfile has been created. If the error continues to occur, please send the logfile content to<EMAIL_ADDRESS>See more details in the logfile: jirastopwatch.log
How can it be replicated?
Start the app
What did you expect instead?
Expected the app to start
Relevant log output
9/8/2021 9:51:16 AM Request: https://.../rest/api/2/configuration
9/8/2021 9:51:16 AM Response: Unauthorized -

- Replaced our domain with "..."
- Note that the date shown in the log is not correct.  I am entering this on 6/14/2023.

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Hi @mwoffenden, thanks for submitting this bug report!
Looking at the first log line, it looks like the domain for your Jira instance is not set. To update your settings, click on the little cog icon in bottom right corner and enter your Jira domain, your email (must be the same as your Jira account), and an API key.

I'm not sure why the logs would have the wrong timestamp, I'll have to look into that. In the meantime please try and set up those settings? If the problem continues, can you please provide more information about the system you are running; what operating system and which version of JiraStopWatch you are using, etc..
Thanks!

Thanks for the reply.
We can't even start the app, so we can't get to that screen.
Is there a way to clear stored data in the registry, or wherever it's kept?

@mwoffenden, it looks like the error you are seeing is coming from JiraApiRequester.cs on line 43. This would indicate that the exception is occurring due to an Unauthorised status code being returned from your Jira instance, possibly due to a bad API token?
https://github.com/jirastopwatch/jirastopwatch/blob/ca54b91e81caabbf5f0d2ecbe0be08386e28e157/source/StopWatch/Jira/JiraApiRequester.cs#L43
Can you please check if the API token shows as having been used in the Atlassian settings? If it hasn’t been used yet that probably indicates that the token isn’t being properly passed in the request, possibly because of an extra whitespace character for example. Otherwise if the token has been used recently and it’s use aligns with when you attempted to use JiraStopWatch, then that would indicate that the request is being denied due to insufficient permissions on your token or perhaps that the user it is attached to doesn’t have access to the Jia instance.
In either of these cases you might need to edit the settings manually. I don’t recall whether these settings are stored in the registry or in a config file, I’ll look in to that in the morning for you 👍

I need to reset the app, but it will not start.  Maybe you can advise on where the settings are stored and I can clear them out.

Hi @mwoffenden, The configuration file will be stored in C:\Users\<USERNAME>\AppData\Local\StopWatch\StopWatch.exe_<ID>\<VERSION>\, either in user.config or possibly in StopWatch.exe.config. The file is (mostly) human readable/modifiable so you can confirm that the settings are correct or delete the file and try re-installing.

Following the settings UI, created a PAT and entered the username and PAT.  The PAT did not work.  However using the username and password did work, and we can now use the tool again.
I think the UI could be clarified, to indicate that username+password is an option.

That's good to hear, I didn't even know that using the password was supported so I'll definitely add that to the documentation 👍
Thanks for again opening this issue and sharing what the fix was.


Move java runtime into it's own repo

Related to #2791
Repo located here now: https://github.com/apache/incubator-openwhisk-runtime-java
PG1 2109 👍

@ddragosd
I tried my best to leave everything identical and keep the git history.
For java9 we could use core/java9Action/Dockerfile
My second option  would be 9/Dockerfile
And then on the next opportunity to when we update java8 then make it consistent 8/Dockerfile

Java repo should not use “bx” either.

leave everything identical and keep the git history

👍 @csantanapr

@rabbah

Java repo should not use “bx” either.

My bad some copy/paste from another repo :-)
I will fix in all of them.
Also noticed that new repos don't have issues enabled, I will open an INFRA ticket

@csantanapr i think it would make sense after all these prs are merged to have a runtimes top level directory (peer with core) and put all the dockerfiles there. thoughts?


Default food item added to the cart by default

Description:
When accessing the cart, a default food item is automatically added without any user action. This behavior can lead to confusion or incorrect orders.
To Reproduce

Open the application.
Navigate to the cart section.
Observe the presence of a default food item already added to the cart.

Expected behavior
The cart should be empty by default unless the user explicitly adds an item.
Actual Behaviour
A default food item is pre-added to the cart automatically, even when the user has not selected anything.
Desktop (please complete the following information):

OS: Windows
Browser: Chrome

Smartphone (please complete the following information):

Device: iPhone15pro
OS: iOS17.6.1

Hi, Can you assign this issue to me? I  am new to open source and would like to work on this.


Please note that HTTP requests containing the same IP, the same date, and the same user agent are considered a unique visitor. In your case, I'm assuming you are only taking unique IPs.

Please note that HTTP requests containing the same IP, the same date, and the same user agent are considered a unique visitor. In your case, I'm assuming you are only taking unique IPs.
The unique IPs count should be displayed right above the data column in the Visitors Panel.
Originally posted by @allinurl in https://github.com/allinurl/goaccess/issues/1205#issuecomment-414336124

Hello,
I'm using GoAccess - 1.3 on Ubuntu 16.04 to parse Nginx logfile.
Goaccess returns "406" for "unique visitors"... and "42 063" in data for IP...
the command in shell :
cat /var/log/nginx/access.log | awk '{print $1}' | sort | uniq | wc -l
returns also :  "42 063"
I don't understand how you determine the unique key for the visitor in https://github.com/allinurl/goaccess/blob/master/src/parser.c
Thanks in advance

The code that keeps track of the unique visitors panel is here. Let me know if that answers your question.

i'm not shure if its the same problem but in my case the uniqe visitors are counted twice.
in a test setup uniqe vis. shows: 4
VISITOR HOSTNAMES AND IPS is listing two ips
cat /var/log/nginx/access.log | awk '{print $3}' | sort | uniq | wc -l  -> 2
(my log_format [%v] %T %h %^ %e [%d:%t %^] %s "%r" %b "%R" "%u" %^)

@njrl Please post the lines from your access log that are being counted twice so I can reproduce this on my end. also, are you using the on-disk storage, if yes, did you persist and restore the data?

the log file https://pastebin.com/JRPUEifp
its a simple on-disk log ... blank started

I get 2 unique visitors for the sample log you posted.

thats strange
with: goaccess --log-format='[%v] %T %h %^ %e [%d:%t %^] %s "%r" %b "%R" "%u" %^' --date-format='%d/%b/%Y' --time-format='%H:%M:%S'  -f /var/log/nginx/test.log i get
(https://user-images.githubusercontent.com/10773007/81261304-20be4780-903c-11ea-9b2e-785867402d94.png)
goaccess -V GoAccess - 1.3.

found MY fault.. got an extra scace in my logs
[%v] %T %h %^ %e [%d:%t %^] %s "%r" %b "%R" "%u" %^ --> counts 4 u.visitors
[%v] %T %h %^ %e [%d:%t %^] %s "%r"  %b "%R" "%u" %^ --> counts 2 u.visitors
sorry for bothering

Glad you figured out. Closing this, feel free to reopen it as needed.


Reverse mapping having a previous defined map?

I've been using ModelMapper in my project and this works like a charm. However, meanwhile i was finishing to map all my entities to DTO, i was thinking if it's possible to make the inverse process when you have a TypeMap already defined.
Assuming the relationship "source field to destiny field" defined by a property map, might this be used to fill a bean in a reverse way (DTO to Entity)?
Thank you guys. You rock!

I think it's a good proposal and nice feature to have, but it's hard implement. It's hard to define how we should map destination to source if we only know how to map source to destination.
For example, if we had a converter toUpperCase using to map entity.firstName to dto.firstName like this using(toUpperCase).map(entity.firstName, dto.firstName). How can I reverse-map this?


Well, sorry, I didn't know what I was really thinking when I posted the picture of the leak, other than that Collision's help is not related to that, but you can probably give it another chance because that's where you'd get the best instructions and help.

you literally tried getting help with a leaked / stolen MLO, no shit you are gonna get banned.

and the rules don't say anything about leaked stuff, because before I posted that picture I looked in the rules to see if there was, but there wasn't. should be added there.

The rules are pretty self-explanatory, and you don't need to be a registrant to understand that asking for help on products that have been stolen is not to be taken into account in any way.
It's pretty basic to understand.


Save the losses when creating embeddings

In the process of generating embeddings from a trained deep learning model, we perform inference on an input image and save the resulting embedding, which captures the semantic reconstructions of salient features as determined by the model. To enhance the utility of these embeddings, I propose saving the reconstruction loss alongside the embedding vector.
The reconstruction loss, calculated as the difference between the input image and the model's reconstructed output, provides valuable insights into the semantic content and anomalies present in the input image:

Images with expected semantics that align well with the model's training data will exhibit a smaller reconstruction loss, indicating that the model can effectively capture and reconstruct the salient features.
Images containing rare, unexpected, or anomalous semantics will result in a larger reconstruction loss, as the model may struggle to accurately reconstruct the input due to the presence of features outside its learned representation.

Real-world applications:

Monitoring changes in satellite imagery: By comparing embeddings and reconstruction losses of a region (e.g., Kiev) before and after significant events (war), we can detect and quantify the extent of semantic changes. Pre-event images will likely have smaller losses, while post-event images containing destruction, damage, and other anomalies will have higher losses.
Anomaly detection in various domains: The reconstruction loss can serve as a valuable metric for detecting anomalies, such as rare events (city floods, locust plagues), unusual semantics (algae blooms, green pools), or "noise" (fog, smog, ships in the ocean, image artifacts). By setting appropriate thresholds on the reconstruction loss, we can flag images containing such anomalies for further analysis.

Implementation:  Modify the embedding generation pipeline to calculate and save the reconstruction loss alongside the embedding vector. This can be achieved by comparing the input image with the model's reconstructed output using e.g. the same loss, or others (MSE, ...).
Action Items:

[ ] @yellowcap to add saving the reconstruction losses in the embedding generation pipeline.
[ ] @stephen-downs to assess how to surface this in the app.

I think this change no only makes our outputs much more useful, but also of measurable relative confidence, AND highlight of operational bias (making the loss a feature, not a thing to get rid of).
cc @MaceGrim @danhammer for the utility feedback.

+1 on this based on customer needs to lean on embedding shift for anomaly detection. Not having the losses messess the value of the anomalies.


Add query escape to query params in GetURI

This PR wraps query params in GetURI with url.QueryEscape to ensure query is well formed e.g if I provide VM Network it will be converted into VM+Network. This seems to be expected in the Avi 30.1.1 API.
Fixes #2902

Hi @Rohan-sss1 could you help merge this PR. Thank you

Hi @parimanur could you help merge this PR?

The changes for this PR were included in https://github.com/vmware/alb-sdk/pull/2933. Closing


get_lexer_for_filename performance issues

(Original issue 1537 created by cyrus-and on 2019-08-24T19:54:25.651180+00:00)
get_lexer_for_filename is so much slower than get_lexer_by_name even though AFAIK it only performs some string matching:
$ time python3 -c 'import pygments.lexers; pygments.lexers.get_lexer_for_filename("foo.c")'

real    0m0.938s
user    0m0.860s
sys     0m0.073s

$ time python3 -c 'import pygments.lexers; pygments.lexers.get_lexer_by_name("c");'

real    0m0.071s
user    0m0.054s
sys     0m0.016s

Since subsequent invocations do not exhibit this behavior, it hints (pure speculation) that maybe this is related to a Python import that gets cached after the first time:
$ time python3 -c 'import pygments.lexers; pygments.lexers.get_lexer_for_filename("foo.c"); pygments.lexers.get_lexer_for_filename("foo.c")'

real    0m0.907s
user    0m0.779s
sys     0m0.105s

(Original comment by cyrus-and on 2019-08-24T20:07:16.991674+00:00)
Something weird:

I tried in a clean Docker container on the same machine and the problem disappears (or it is fast enough);
I tried on another equivalent machine and the problem is less severe (300ms instead of 1s) but still there.

Well, this function needs to find all the plugin lexers provided by entry points. This can be an expensive operation depending on the number of installed packages and pkg_resources behavior.


Enhancement CNF-4565: Do not create empty fields for managed policies that do not have appropriate content

Only create the managed policies field for policies that meet the general requirements for policy content, namely;

Have kind subscription
Have a namespace
Have a name

This will prevent policies from appearing with a content of null in the cgu status

/retest

/cherry-pick release-4.11


java.net.ProtocolException: unexpected end of stream

I try download video. But I cannot complete download.Because error occur.The progress is 92%. I often fail download.Help me.

unexpected end of stream it's a server error and this a better question for StackOverFlow: https://stackoverflow.com/questions/45838774/java-io-ioexception-unexpected-end-of-stream-on-connection-in-android


[Maps Creator Service] API Review

New API Review meeting has been requested.
Service Name: Azure Maps Creator Service
Review Created By: Ismael Navarro-Fuentes
Review Date: 1/17/2023 1:00 PM PT
PR: https://github.com/Azure/azure-rest-api-specs/pull/19539
Hero Scenarios Link: Not Provided
Core Concepts Doc Link: Not Provided
Description: Features is a new service within Azure Maps Creator that conforms to the Open Geospatial Consortium - Features API Standard. Features allows consumers of the API to query and modify their geospatial data.
Detailed meeting information and documents provided can be accessed here
For more information that will help prepare you for this review, the requirements, and office hours, visit the documentation here

Meeting updated by Ismael Navarro-Fuentes
Service Name: Azure Maps Creator Service
Review Created By: Ismael Navarro-Fuentes
Review Date: 1/17/2023 1:00 PM PT
PR: https://github.com/Azure/azure-rest-api-specs/pull/19539
Hero Scenarios Link: here
Core Concepts Doc Link: here
Description: Features is a new service within Azure Maps Creator that conforms to the Open Geospatial Consortium - Features API Standard. Features allows consumers of the API to query and modify their geospatial data.
Detailed meeting information and documents provided can be accessed here

API Stewardship Board Review: 17-Jan-23
For the most part, this is tracking the proposed industry standard.
There is a question about incrementing the API version, which will be discussed on the Azure Breaking Changes review board.

API Stewardship Board Review: 17-Jan-23
For the most part, this is tracking the proposed industry standard. There is a question about incrementing the API version, which will be discussed on the Azure Breaking Changes review board.

Hello @markweitzel, I've created PR https://github.com/Azure/azure-rest-api-specs/pull/22590 to create a base line for the API version increment. Once that PR is merged into main, I will merge those changes to https://github.com/Azure/azure-rest-api-specs/pull/19539 (or create a new PR if it's easier to review) so that the Features APIs are a part of 2023-03-01-preview.
Could I kindly ask for a review of https://github.com/Azure/azure-rest-api-specs/pull/22590.

PR is signed off -- closing as complete.


"Relighting_preprocessing_tools" had been missing two files.

Dear authors,
I have found that I met problem with Relighting_preprocessing_tools in executing python create_dataset.py
I cannot find pretrained/BEST_checkpoint_r18.tar when i run arcface in /Arcface/inference.py
env =  "/home/ubuntu/anaconda3/envs/zyh_difareli/bin/python"
subprocess.call([env, './Arcface/inference.py',
print(f'{os.path.abspath(os.path.dirname(file))}/pretrained/BEST_checkpoint_r18.tar')
arcface_ckpt = f'{os.path.abspath(os.path.dirname(file))}/pretrained/BEST_checkpoint_r18.tar'

https://github.com/foamliu/InsightFace-v2/releases/tag/v1.0@zyhsweety


Move bazel flags to bazelrc.

~ +2 minutes.
I think Saleem is going to fold it into: https://github.com/tensorflow/swift-apis/pull/801


Add support for nodejs mysql2 package in TypeScript definition

Currently per the TypeScript definition only the mysql package may be used. According to the issue https://github.com/aws/aws-xray-sdk-node/issues/6 the mysql2 package is also supported.

@thomaschaaf thanks for raising this issue, you're right mysql2 types should be supported for instrumentation in our type definitions as well. We are open to you adding a PR for this support otherwise I'll add it soon.

Any updates on this? I seem to have run into this issue, my code is
import mysql2 from 'mysql2'
import { captureMySQL } from 'aws-xray-sdk'

const mysqlCaptured = captureMySQL(mysql2)

Which results in the following type check error:

Types of property 'createConnection' are incompatible.
Type '{ (connectionUri: string): Connection; (config: ConnectionOptions): Connection; }' is not assignable to type '(connectionUri: string | ConnectionConfig) => Connection'.
Types of parameters 'connectionUri' and 'connectionUri' are incompatible.
Type 'string | ConnectionConfig' is not assignable to type 'string'.
Type 'ConnectionConfig' is not assignable to type 'string'.

Hello, I've raised a PR to hopefully close this issue. Feedback welcome.


Upgrade the icon

The current dates from Windows XP
There's a really good candidate from imageres.dll

Unfortunately, I can't just rip Microsoft's copyrighted icons out of Windows and redistribute them, and I'd rather not rewrite HashCheck to directly load the icons from imageres.dll at runtime (I assume it'd be possible?) if I can avoid it--it's a lot of work for not much benefit.
However, Microsoft does offer a Visual Studio image pack with far more permissive redistribution rights. I'll take a look in there to see if this icon (or something similar) is available.

The file type icon's path is stored in HKEY_LOCAL_MACHINE\SOFTWARE\Classes\HashCheck\DefaultIcon. You can make it point to the imageres.dll's icon instead of the built-in icon.
You can change the key with the installer after the DLL is registered or change the following code so that it directly points the imageres.dll icon instead of the built-in when the DLL registers.
Please note that the icon is only available starting from Windows 8, so the DLL/installer should fallback on the built-in when the OS is older.

Thanks for the additional info.
There's also the icon that's displayed in the upper left of any HashCheck window that's running and on the task bar (search for IDI_FILETYPE in the code, it wouldn't be too hard to change, but is it worth it?), and even worse there's the one (two and counting if you consider this branch) in the context menu handler which needs to be in BMP format, not ICO format....

I personnaly prefer to have an empty context menu icon because currently it's the only one having an icon and stands a bit too much:

But if you want to still show an icon in the context menu, it's not hard to do (Maybe consider an toggle to hide the icon in the settings also)
I'm not sure if the window icon is worth the trouble, I only use the properties panel. Depends if it's used a lot.


In Admin Page button "Delete printer" not working

Describe the bug
When I'm trying to delete printer with "Delete printer" button in Configured Printers section, the printer appears again after a refresh.
Describe the device(s) you are using
I'm making test with FP-700X
How To Reproduce

The printer is connected to PC and working correctly.
Disconnect printer from PC
Click "Detect printers" in Admin Page
The printer is still there and it is appeared as Ready
Click button "Delete printer"
The page automatically detects printer again
The printer is still there and it is appeared as Ready

Expected behavior
I expect deleted printer to not showing as a printer in the Admin page

Fixed, but please wait for new binary releases.


Wat

Add support for the WebAssembly Text Format (Wat). This should be relatively straightforward with this library: https://github.com/AssemblyScript/wabt.js
Just convert the source Wat to a WasmModule, then call the toBinary function. Boom done

Added!


[infra upgrade] use Capistrano action to automate blue green deployments External

https://github.com/Pralish/violet_rails/blob/test-staging/.github/workflows/deploy-staging.yml
Hi @donrestarone , Here's the branch I am experimenting with. I haven't been able to make capistrano-deploy work without setup-ruby. The error says:

Error: Error: Unable to locate executable file: bundle.

https://github.com/Pralish/violet_rails/runs/5048056550?check_suite_focus=true
I will update you once I find something.

@Pralish is there a reason why we cant use setup-ruby?


Allow to save on local storage

Allow users to save on local storage.

Fixed on https://github.com/matheusgr/modeligado/commit/5d79e544d85670dfca05a095f64bf065e4a2ca6d


Question: What should the satisfiability-solver solve for?

Suppose I have the following definition for a single dimension line:
Line:: {
	P_a: (2.0*P_ab) - P_b
	P_ab: (P_a/2.0) + (P_b/2.0) // middle
	P_b: float
}

Technically there is enough constrains such that P_b could be solved (P_b: (2.0*P_ab) - P_a) but since it's only constrained to be a float, it is not solved for.
I could see this being kinda a rabbit hole if we we're to automatically solve for P_b, both computationally and with regard to just defining what cue solves for (much simpler to say it references something, that references something, ... that references something concrete).
Theoretically a solver of this nature could be idempotent. Although it would be difficult to say definitively something is the only solution which meets the provided constraints, it's not inconceivable to define a solver that would come up same solution given the provided constraints.
But again, likely very computationally expensive and probably not wise to include as a core feature of the language. Perhaps a package in tool?
A somewhat related project is to this issue is Barliman

Yeah, CUE explicitly does not solve this. There is no "satisfiability-solver" whatsoever, in fact.
CUE does allow cycles, though. So like in propagator networks, one can specify the formula for each three directions and CUE will verify that all are consistent as soon as there is enough information.
So if one wants to add a solver to CUE, the way to do this is to have a tool that reads CUE and generate more CUE that can be added to existing CUE to complete formula in other directions when possible.

This issue has been migrated to https://github.com/cue-lang/cue/issues/230.
For more details about CUE's migration to a new home, please see https://github.com/cue-lang/cue/issues/1078.


Use default Golang ciphersuites if none are specified

What this PR does / why we need it:
Since #2743 merged, testgrid has been completely red for all versions of Kubernetes <1.17.
The error:
2020-03-30T20:39:08.180691847Z stderr F 2020/03/30 20:39:08 http: TLS handshake error from <IP_ADDRESS>:38402: tls: no cipher suite supported by both client and server

I suspect #2562 is defaulting the list of ciphersuites that are enabled to a subset of those that would normally be enabled by default if we hadn't explicitly set our own defaults.
We also changed the private key algorithm for this certificate from RSA to ECDSA in #2743, so I suspect this may also be related to why we've just started seeing this issue come up.
Which issue this PR fixes: fixes #
Special notes for your reviewer:
This brings us in line with how other Kubernetes components (i.e. consumers of component-base) allow configuring TLS ciphersuites, and how they also allow disabling TLS 1.2 (with a dedicated flag to set the minimum version).
Release note:
webhook: add `--tls-min-version` to allow configuring the minimum allowed TLS version and fix default ciphers list.

/area webhook
/kind bug
/priority critical-urgent

/test pull-cert-manager-e2e-v1-11
/test pull-cert-manager-e2e-v1-11
/test pull-cert-manager-e2e-v1-11
/test pull-cert-manager-e2e-v1-11

/test pull-cert-manager-e2e-v1-11
/test pull-cert-manager-e2e-v1-12
/test pull-cert-manager-e2e-v1-13
/test pull-cert-manager-e2e-v1-14
/test pull-cert-manager-e2e-v1-15
/test pull-cert-manager-e2e-v1-16

/retest

cc @willthames

/kind bug
/priority critical-urgent
/area webhook

I also wonder if there's anything we could have done to pick up these regresssions before the breaking changes got merged.
Could we require both the min and max supported K8s versions be e2e tested and succeed before PRs are merged?

That's not a bad idea, but I'm not sure if it's actually that helpful given the additional overhead of doubling the number of e2e presubmits we run.
In this instance, we merged the faulty change and within a couple of hours I got notified by email (by testgrid) that our tests were failing. We purposely run it like this to avoid having to run multiple e2e presubmits (whilst also maintaining 'signal' on these versions).
If we start to see a lot more things slip through that do effect older versions, I think it becomes a bit more helpful to run them all the time. But right now at least we very rarely have changes the effect only a specific k8s version/specific set of them 🤷♂


Thread Safety Issues

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/510
Originally created by @jschatz1 on 2019-03-20 15:46:56

We have multiple threads running and we have an issue where Project.find() can get called multiple times from multiple threads. This sets the global current working directory which changes the pointer which messes up the cwd.
We should change find to not do a recursive search and only search the cwd
@classmethod
def find(self, from_dir: Union[Path, str] = None, activate=True):
    project = Project()

    if not project.meltanofile.exists():
        raise ProjectNotFound()

    if activate:
        project.activate()

    return project

cc @dmor

View 3 previous comments from the original issue on GitLab


Error: 'Badge' is imported from both 'package:badges/src/badge.dart' and 'package:flutter/src/material/badge.dart'. in side_menu_item.dart:148:14

I had a project with an old version and then figured the issue is fixed in the latest one 😅


Encryption at host for North Europe

Hi,
It seems the availability of Encryption at host feature is limited to some US regions. Are there any plans to bring this feature to North Europe region in near future?

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

ID: ade067f1-3f3e-73de-0321-bb8d07147cd3
Version Independent ID: e8dc38c4-0114-b7c2-887f-e1d2810206e8
Content: Server-side encryption of Azure Managed Disks - PowerShell - Azure Virtual Machines
Content Source: articles/virtual-machines/windows/disk-encryption.md
Service: virtual-machines
Sub-service: disks
GitHub Login: @roygara
Microsoft Alias: rogarana

Hi, @juho-hanhimaki. I am going to have an engineer who supports this service provide a direct response to your inquiry. Thank you for taking the time to reach out and inquire.

I have reached out to the internal team to check if it is on the roadmap and if it can be shared. Thanks.

Hi @juho-hanhimaki,
Yes, we do have plans of rolling the feature out in North Europe in the near future.


[BUG] When add spring-cloud-stream-binder-eventhubs or serviceubs dependency in pom but don't configure them will throw exception

Describe the bug
When adding azure-spring-cloud-eventhubs-stream-binder or azure-spring-cloud-servicebus-queue-stream-binder azure-spring-cloud-servicebus-topic-stream-binder dependency into pom file, without configuring them, will throw IllegalArgumentException  exceptions.
Exception or Stack Trace
Caused by: org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'spring.cloud.azure.eventhub-com.microsoft.azure.spring.cloud.autoconfigure.eventhub.AzureEventHubProperties': Invocation of init method failed; nested exception is java.lang.IllegalArgumentException: Either 'spring.cloud.azure.eventhub.namespace' or 'spring.cloud.azure.eventhub.connection-string' should be provided
	at org.springframework.beans.factory.annotation.InitDestroyAnnotationBeanPostProcessor.postProcessBeforeInitialization(InitDestroyAnnotationBeanPostProcessor.java:160) ~[spring-beans-5.2.9.RELEASE.jar:5.2.9.RELEASE]
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.applyBeanPostProcessorsBeforeInitialization(AbstractAutowireCapableBeanFactory.java:415) ~[spring-beans-5.2.9.RELEASE.jar:5.2.9.RELEASE]
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.initializeBean(AbstractAutowireCapableBeanFactory.java:1786) ~[spring-beans-5.2.9.RELEASE.jar:5.2.9.RELEASE]
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.doCreateBean(AbstractAutowireCapableBeanFactory.java:594) ~[spring-beans-5.2.9.RELEASE.jar:5.2.9.RELEASE]
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.createBean(AbstractAutowireCapableBeanFactory.java:516) ~[spring-beans-5.2.9.RELEASE.jar:5.2.9.RELEASE]
	at org.springframework.beans.factory.support.AbstractBeanFactory.lambda$doGetBean$0(AbstractBeanFactory.java:324) ~[spring-beans-5.2.9.RELEASE.jar:5.2.9.RELEASE]
	at org.springframework.beans.factory.support.DefaultSingletonBeanRegistry.getSingleton(DefaultSingletonBeanRegistry.java:234) ~[spring-beans-5.2.9.RELEASE.jar:5.2.9.RELEASE]
	at org.springframework.beans.factory.support.AbstractBeanFactory.doGetBean(AbstractBeanFactory.java:322) ~[spring-beans-5.2.9.RELEASE.jar:5.2.9.RELEASE]
	at org.springframework.beans.factory.support.AbstractBeanFactory.getBean(AbstractBeanFactory.java:202) ~[spring-beans-5.2.9.RELEASE.jar:5.2.9.RELEASE]
	at org.springframework.beans.factory.config.DependencyDescriptor.resolveCandidate(DependencyDescriptor.java:276) ~[spring-beans-5.2.9.RELEASE.jar:5.2.9.RELEASE]
	at org.springframework.beans.factory.support.DefaultListableBeanFactory.doResolveDependency(DefaultListableBeanFactory.java:1307) ~[spring-beans-5.2.9.RELEASE.jar:5.2.9.RELEASE]
	at org.springframework.beans.factory.support.DefaultListableBeanFactory.resolveDependency(DefaultListableBeanFactory.java:1227) ~[spring-beans-5.2.9.RELEASE.jar:5.2.9.RELEASE]
	at org.springframework.beans.factory.support.ConstructorResolver.resolveAutowiredArgument(ConstructorResolver.java:884) ~[spring-beans-5.2.9.RELEASE.jar:5.2.9.RELEASE]
	at org.springframework.beans.factory.support.ConstructorResolver.createArgumentArray(ConstructorResolver.java:788) ~[spring-beans-5.2.9.RELEASE.jar:5.2.9.RELEASE]
	... 48 common frames omitted
Caused by: java.lang.IllegalArgumentException: Either 'spring.cloud.azure.eventhub.namespace' or 'spring.cloud.azure.eventhub.connection-string' should be provided
	at com.microsoft.azure.spring.cloud.autoconfigure.eventhub.AzureEventHubProperties.validate(AzureEventHubProperties.java:75) ~[azure-spring-cloud-autoconfigure-2.0.0-alpha.20201014.0.jar:na]
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[na:na]
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) ~[na:na]
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) ~[na:na]
	at java.base/java.lang.reflect.Method.invoke(Method.java:566) ~[na:na]
	at org.springframework.beans.factory.annotation.InitDestroyAnnotationBeanPostProcessor$LifecycleElement.invoke(InitDestroyAnnotationBeanPostProcessor.java:389) ~[spring-beans-5.2.9.RELEASE.jar:5.2.9.RELEASE]
	at org.springframework.beans.factory.annotation.InitDestroyAnnotationBeanPostProcessor$LifecycleMetadata.invokeInitMethods(InitDestroyAnnotationBeanPostProcessor.java:333) ~[spring-beans-5.2.9.RELEASE.jar:5.2.9.RELEASE]
	at org.springframework.beans.factory.annotation.InitDestroyAnnotationBeanPostProcessor.postProcessBeforeInitialization(InitDestroyAnnotationBeanPostProcessor.java:157) ~[spring-beans-5.2.9.RELEASE.jar:5.2.9.RELEASE]
	... 61 common frames omitted

To Reproduce
Create a Spring application and add the dependency
      <dependency>
        <groupId>com.azure</groupId>
        <artifactId>azure-spring-cloud-eventhubs-stream-binder</artifactId>
      </dependency>

Code Snippet
Run the application without setting any properties will reproduce the exception.
Expected behavior
Don't throw exceptions when adding the binder to the pom file, but do when want to use the binder without setting such connection-string properties.
Setup (please complete the following information):

OS: Any OS
IDE : IntelliJ
Version of the Library used: Any version will do.

Additional context
N/A
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[x] Bug Description Added
[x] Repro Steps Added
[x] Setup information Added

This issue also prevents using multiple different binders in the same application, (e.g. using both the Event Hub and Service Bus Queue binders) since one of them will always treated not configured. This is a blocker for us for going to production, since we have such cases that we need two binders.

Hi @avpines , you can use spring.cloud.azure.eventhub.enabled=false or spring.cloud.azure.servicebus.enabled=false switch to suppress the bean creation,  please try to add these configuration items.

@moarychan thanks, but I'm not sure that I'm following - I need a single application that will use the two binders, e.g. consume events to an Event Hub and publish them to a Service Bus queue. How disabling the bean creation will help in this case?

Hi @avpines , I have passed this usage scenario, send message to event hubs, then transform the message to service bus queue, consume the message in service bus queue, please try the following configuration. Hope to help you!
spring:
  cloud:
    azure:
      eventhub:
        type: eventhub
        defaultCandidate: true
        connection-string: "event hubs connection string"
        checkpoint-access-key: "storage key"
        checkpoint-storage-account: "storage account"
        bindings:
          consumer:
      servicebus:
        enabled: false
    stream:
      eventhub:
        bindings:
          uppercase-in-0:
            consumer:
              checkpoint-mode: MANUAL
      poller:
        fixed-delay: 3000
      function:
        definition: eventhub;uppercase;servicebus
      defaultBinder: eventhub
      binders:
        servicebus-queue:
          type: servicebus-queue
          defaultCandidate: false
          environment:
            spring:
              cloud:
                azure:
                  servicebus:
                    enabled: true
                    connection-string: "service bus queue connection string"
                  eventhub:
                    enabled: false
      bindings:
#        eventhub-in-0:
#          destination: event-hub-aa
#          binder: eventhub
#          group: $Default
        eventhub-out-0:
          destination: event-hub-aa
          binder: eventhub
          group: $Default
        uppercase-in-0:
          destination: event-hub-aa
          group: $Default
#          checkpoint-mode: MANUAL
        uppercase-out-0:
          destination: space-1-queue-aa
          binder: servicebus-queue
        servicebus-in-0:
          destination: space-1-queue-aa
          binder: servicebus-queue

Hi @avpines, the solution my colleague @moarychan provided is a workaround to make the current version of binders work. But to totally get rid of such checking, we need to move even further. I created this PR to remove the validate() in @PostContruct. https://github.com/Azure/azure-sdk-for-java/pull/17066

Thank you very much for the workaround and detailed example.
Just to make sure that I understand it completely: In each binder you explicitly disable the other binder so it won't throw an exception in validate(), so since the default binder is using Event-Hub, you explicitly disable Service-Bus via spring.cloud.azure.servicebus.enabled=false, and in the servicebus-queue binder you explicitly disable the Event-Hub binder via spring.cloud.stream.binders.servicebus-queue.environment.spring.cloud.azure.eventhub.enabled=false.
Am I correct that this is the only change needed, and once https://github.com/Azure/azure-sdk-for-java/pull/17066 is complete, the explicit checking won't be needed?

Yes. #17066 is merged and we don't need such workaround anymore.


Use project locking to enable parallel by default

Introduces a project lock service that controls when tasks can execute
Project lock is released when tasks are running async work
Enable multiple task workers by default
Remove @ParallelizableTask

Note that the simplification proposed above can be a follow-up story, as long as the locking implemented so far works well. There were a few things that I think we should fix first though, like the duplication of lock tracking in two places.

@oehme I've changed the things we discussed and I think this is ready for another review.
With regards to the simplification you suggested, per our conversation, I like what you're thinking, although I don't know that we can do exactly what you're suggesting.  I think we're several steps closer to being able to do something like that, though, and I'm sure we can continue to simplify things.

🎉


"No JSON object could be decoded" when retrieving claim

Trying to get to https://spee.ch/@minutephysics:589276465a23c589801d874f484cc39f307d7ec7/bell-s-theorem-the-quantum-venn-diagram and you get a blank white page with "No JSON object could be decoded"
This content was originally attempted during spee.ch server issues on Thursday night

from the logs: Request Error: /@minutephysics:589276465a23c589801d874f484cc39f307d7ec7/bell-s-theorem-the-quantum-venn-diagram No JSON object could be decoded

@tzarebczan this error is coming from the daemon.  Should I file an issue?  From the lbrynet logs:
2017-11-15 14:06:24,374 ERROR    lbrynet.daemon.Daemon:688: Failed to get bell-s-theorem-the-quantum-venn-diagram (No JSON object could be decoded)
Also, I have not been able to reproduce on spee.ch.  Do you know of any other links that return this error?

Let's take a look at this closer later today
we should try running a file_list bell-s-theorem-the-quantum-venn-diagram to see if there was a previous failed download that could be causing this. Depending on what we find, an issue may be warranted on lbry

@billbitt - can you check if this is still happening on the latest daemon and if so, please open an issue on lbry


How to create an empty set?

# a is now an empty dict
a = {}

# tried this
b: set[int] = {}

but getting an error
ERROR: Error when compiling test_random module: Type error
 24:14-22
   |
24 |    results: set[int] = {}
   |             ^^^^^^^^^
Cannot infer __builtin__.dict[T_144, T_145] < __builtin__.set[__builtin__.int]

@nordlander how to create an empty set?

Ah right, or actually set() is set(None) barring default argument values (which is what I didn't think of and thus opened this issue).
I think it would be nice if we could say that {} is either empty dict or empty set and rely on type inference to determine which one it should be. I think it provides a more consistent UX. Low priority though...

I've updated AbE to explain this, so closing issue.


System.InvalidOperationException:“Sequence contains no elements” in windows sdk 10.0.17134

mnist_demo  this line    var maxIndex = ModelOutput.Plus214_Output_0.IndexOf(ModelOutput.Plus214_Output_0.Max());

Hello !   what are you trying to do ?      Can you try this with the latests SDK (17723) and insider build (17733) and send the details.   We will totally get you running !


fix: update readme

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

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

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, particularly in hard-to-understand areas
[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
[x] I have checked my code and corrected any misspellings

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


No Request Header with different token in images request - Critical

this is an awesome component! I have heavily used it in my app! but now I am facing a major issue, my webservices are now secured with request headers with different token every time & I find no way to pass them a different token in every image request based on user's username password generated token. This is very critical for me! And I don't want to use any othr plugin other than urs' wonderful plugin. Would be so glad if you can help me asap.
Thanks In Advance.

The problem is that @ankitherocker wants each SDWebImage request to specify its own headers.
As it is headers are not associated with requests, they're shared between all requests for a downloader.
Same issue:
https://github.com/rs/SDWebImage/issues/729
https://github.com/rs/SDWebImage/issues/166#issuecomment-8379698


add turbopy-dev conda environment

Should we have two different conda environments for developing apps within the turbopy development environment vs developing turbopy itself?  For example, livereload would be in turbopy-dev but not in turbopy.

Just to loop back to this issue. I guess I don't really understand why we would need two conda environments. Wouldn't it make sense to have one conda env just to sync development across apps and core?

That's a good question. I think the point is that if there are any packages needed only for development on the core framework, they should go in the turbopy-dev environment, but not the turbopy environment. E.g. packages like livereload and sphinx are needed for development, but not by someone who just wants to use the framework.
Perhaps another way to get at what you are asking is "do we even need the 'non-dev' version of the environment?"
That is an interesting question. If we get to the point where the framework is on pypi and can be installed via pip install turbopy or conda install turbopy, then I don't really see a need for a "non-dev" conda environment. All of the prerequisites would be installed by the package manager (pip or conda as the case may be).

Very good point @arichar6. I'd support just thinking of the turbopy conda environment as the conda environment required for developing turbopy and turbopy apps. (I'll use this approach with nepc, too.)

Ok. For now we will treat the turbopy conda environment as the "development version". At some point, we will have to figure out how package turbopy up for installation via a package manager. That will take care of the "non-dev" dependancies.


Not compatible with Pillow=10.0.0

Pillow just released a new version on July 1st, 10.0.0, which deprecates ImageDraw.textsize(), currently used by the render attribution function.
The quick fix is to add a version constraint to the requirements to this package to use version 9.5.0. The longer fix is to stop using textsize().

Fixed via #32


CLOUD-999 added new network ports

@keyki or @martonsereg

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/1027/


APS 1049 - link to premises and bed from OOS bed page

Context
Jira ticket: https://dsdmoj.atlassian.net/browse/APS-1049
Changes in this PR
Adds links to the out of service bed page heading, so that the user can navigate to the premises and bed pages.

Screenshots of UI changes
Before
After

Can we include a unit test to describe this and prevent regression? Or describe the navigation from OOSN to i) premises and ii) bed in an integration test?

It's not unit testable as is, but we could include an integration test. I wasn't sure how much value it delivered but preventing regression is a good point.


Define Semantics for Extended Protocol Types

What would you like to be added:
The Gateway API defines "Core" and "Implementation-specific" protocol types but does not address "Extended" protocols. Should "Extended" protocols contain a prefix, i.e. `extended/GRPC" to denote that implementations are not required to support this protocol type?
xref: https://github.com/kubernetes-sigs/service-apis/pull/398

/remove-lifecycle rotten

/remove-lifecyclt stale
/lifecycle frozen

Considering we document everything "Extended" within our API, should we add in documentation that says that Extended protocol must be defined within this package?
We could define an extended protocol as well. How about GRPC?

So, we currently have no Extended support protocol types, but this issue serves to record that it's fine for implementations to bring Custom support protocol types upstream to us, and we'll consider moving them to Extended support.
This will be subject to the usual Extended support rules:

the exact value will be specified
there has to be conformance tests that validate how it works, if an implementation chooses to implement it.

I'll close this for now, please feel free to comment if you want to reopen.


Web Bluetooth Notifications NotSupportedError

Hello!
I am trying to connect a TI SensorTag CC2650 to Web Bluetooth in a Chrome browsers running on Win10.
I have used this demo successfully with the following UUID parameters (Humidity Sensor):
Bluetooth Service: f000aa20-0451-4000-b000-000000000000
Bluetooth Characteristic: f000aa21-0451-4000-b000-000000000000
It outputs:
> Characteristic UUID:  f000aa21-0451-4000-b000-000000000000
> Broadcast:            false
> Read:                 true
> Write w/o response:   false
> Write:                false
> Notify:               true
> Indicate:             false
> Signed Write:         false
> Queued Write:         false
> Writable Auxiliaries: false

Everything in its right place so far :)
Here's what I don't understand:
In the notification example sample, using the same parameters as before i get the following Error:
Argh! NotSupportedError: GATT operation failed for unknown reason.
Why is this happening? :(

TLDR: Is it possible to fetch, say, the heartrate value from a mi band2 using web bluetooth on windows? Which example should I explore further?
Thanks in advance!!


Self-hosted etcd cannot survive pod deletion

Simply deleting the etcd pod in a single master setup (most common) seems to not reconcile and create a new pod, which causes a cluster outage. This is less resilient than the default on-host setup and other control plane pods, which can recover from pod failures. In the on-host case, one can repair any etcd issues (or systemd restarts them) as long as data is still present on disk.
Longer discussion: https://github.com/coreos/tectonic-installer/pull/601#issuecomment-300242512

There is only one etcd pod, right? If so, deleting it means losing the entire cluster.
We are working on disaster recovery at the moment.
For more info, please track on https://github.com/kubernetes-incubator/bootkube/issues/432 .
I think this is dup to that issue.

This would be more akin to deleting /etc/kubernetes/etcd-pod.yaml or deleting the etcd-member.service` (essentially you're saying "this should no longer be running on this host").
simple pod failure should recover (e.g. kubelet will keep trying to restart it). Deletion is a little bit more questionable.
I'll actually move this into https://github.com/kubernetes-incubator/bootkube/issues/432


Add fork acknowledgment to README and make MIT copyright notice compliant

This PR updates the README's "Acknowledgement" section to credit the base project for yarte, askama, and amends the copyright notice in LICENSE-MIT to comply with the license terms of the base project, following the advice of this answer on Stack Exchange.

I understand that it is reflected in https://github.com/botika/yarte/blob/master/README.md#acknowledgment and https://github.com/botika/yarte/search?q=%2F%2F+Based+on&unscoped_q=%2F%2F+Based+on these references in the code. I understand that it is sufficient for the licenses the project has.
To ensure these affirmations, a demonstration of the facts is necessary.
You can explain it in more detail and with references of the cases.

 You are so petty, it's amazing.

I'd just like to point out that this repo (master) still holds 9 files that are exact copies of files found in current askama master.
All tests/benchmarks, but it kind of shows that this is a fork.
Now if this can be seen as a substantial part of askama is difficult to say.
But considering the obvious contention and oddities of moving to a new repo etc, why not just add the attribution and be done with it?

Ok, but the benchmarks are somewhat public and it is normal to match empty files, "Hello world!", Literals or {{ inner.holder.a }} or {{ inner.a }}or simple texts for testing.
  But when it has nothing to do with it.


Autodelete old Data after x-Days or Months

How can I configure autodeletion of old data after e.g. 90 days / 3 months?

That functionality currently does not exist in Shynet; perhaps set
something up in Postgres?
On Mon, Sep 30, 2024 at 5:18 AM SebastianGode @.***>
wrote:

How can I configure autodeletion of old data after e.g. 90 days / 3 months?
—
Reply to this email directly, view it on GitHub
https://github.com/milesmcc/shynet/issues/333, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAXFS4ESGX53KMSL2XWQ63DZZE6Q5AVCNFSM6AAAAABPDE6TL2VHI2DSMVQWIX3LMV43ASLTON2WKOZSGU2TMNBYGA3DCNQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Connection through Android/iPhone app over traefik fails - connection not secure

I set up a vm with traefik enabled. The connection to the containers through the web client works perfectly fine, but fails using the android or IPhone app.
The failure message in the apps reads:
"The application enabled can only be accessed using a secure connection (HTTPS)"
This happens for each of the arm templates (getnav, getbc, getbcext) for bc and nav containers.
I am able to connect through the apps to a single container without traefik enabled.
When connecting with the web client, I am monitoring several http and https requests. For instance when the client requests the Header.png for the login page.
See also on this Yammer thread: https://www.yammer.com/dynamicsnavdev/#/Threads/show?threadId=396327303618560

Fixed with https://github.com/microsoft/navcontainerhelper/pull/758


Add valuesController to Web API conventions article /a

Fixes  #21655
Internal review URL

@pranavkm  1 minute review when you have time.


Need to know how to update the authorization header and also return using Promise

I have just started playing with this awesome library. I have started from scratch. Here, I am trying to implement the authorization. I want to follow the guideline from the OCPP guidelines. Can you guide me or send a sample code to implement this?
I have explained the problem I am facing in this video.
https://www.loom.com/share/9c39b4204b824458bd01ba1b27fb42b9
In short, first, I need to know if there is a way to return using the promise from the central system and secondly, how can I change the authorization key of the client object? Please watch the video above to understand clearly..
This is the flow that I am trying to implement.

Hi Ahsan, thanks for taking the time to put together this video.
I think I understand what you are trying to do.
Firstly, it appears that you are trying to implement HTTP Basic authentication (as per OCPP1.6-J and OCPP1.6 security whitepaper (security profile 1 & 2)). You'll be happy to know that this module supports this feature natively!
Note: The OCPP "auth key" is considered to be a "password" by the HTTP Basic Auth spec, so you'll see this library often referring to a "password" instead of "authorization key" or "authentication key" - but just know that these terms are all synonymous.
To take advantage of the module's support for this authentication scheme - don't set the Authorization header directly; Use the password option in the RPCClient constructor instead. The password option will ensure that the authorization header is set correctly (- even allowing you to use binary Buffer's as auth keys for maximum entropy if you want to; which seems to be encouraged by the OCPP spec, although discouraged by HTTP Basic Auth spec).
There are a few other quirks about how OCPP uses HTTP auth which you can read at some length at the bottom of the readme, but if you stick to using the password field, then you should not need to worry about any of them.  (I recommend reading the linked section on OCPP Security Profile 1 if you haven't already.)
So, setting the auth key should be as simple as adapting your code as follows:
// in ChargerPoint.ts (line 19)
this.cli = new RPCClient({
  endpoint: endpoint,
  identity: identity,
  protocols: ['ocpp1.6'],
  password: this.authKey
});

Then, when you are handling the authentication on the server side, you should compare your auth key against the client.handshake.password.  Note: The password will be received by your server as a Buffer - so if you want to perform a string comparison, you should do so like this:
// in server.ts (line 48)
if (client.handshake.password.toString('utf8') == 'golalu') {

Also keep in mind that the password may be undefined if the client does not specify a password, so you may want to use optional chaining for safety, like so:
if (client.handshake.password?.toString('utf8') == 'golalu') {

Finally, to change the RPCClient's password for the next time it connects, you can use the reconfigure() method like so:
// in ChargerPoint.ts (line 54)
this.cli.reconfigure({password: config.value});

I hope this helps.

Hi. Thanks for your detailed reply. I followed your steps and it worked. Really appreciate your detailed and fast response.
I had another question in the video.
OCPP specification states that the central system should return the boot notification status as "pending" for an unauthorized password and then initiate a configuration change request.
However, in the current implementation, I am required to return the boot notification message. Unfortunately, after the return line, no further execution takes place, making it impossible to trigger the config change message as per the documentation.
I would like to explore if there is any alternative approach or solution available that aligns with the specifications outlined in the documentation.
Here is my current code
const response: IBootNotificationResponse = {
            status: BootStatus.Pending,
            interval: 300,
            currentTime: new Date().toISOString(),
          };

          this.makeChangeConfigurationRequest(client, { key: 'AuthorizationKey', value: 'golalu' });
          return response;

Ah yes, I admit this is a bit clunky to handle.  I think this is something which may see a change in a future major revision.
You can work around this by using the alternative reply() callback which is provided as part of the object passed to the handle() callback.  This behaves just the same as returning the response.  Once the reply callback has been called, you no longer need to worry about your return value, and are free to do whatever else you need to do in your handler.
The general example is as follows:
client.handle('BootNotification', async ({reply}) => {
    reply({
        status: 'Pending',
        interval: 300,
        currentTime: new Date().toISOString(),
    });

    // The BootNotification response has now been handled.
    // It no longer matters what we return from this handler, as the response has already been queued.

    await client.call('ChangeConfiguration', { /* params */ });
});

Okay, it worked. But I had to add a delay to make sure that the boot notification is sent first.
here is my updated code.
const response: IBootNotificationResponse = {
            status: BootStatus.Pending,
            interval: 300,
            currentTime: new Date().toISOString(),
          };
          body.reply(response);
          setTimeout(() => {
            this.makeChangeConfigurationRequest(client, { key: 'AuthorizationKey', value: 'golalu' });
          }, 4000);

I will continue to explore and experiment with this further. If any specific issues arise, I will create separate issue.  Thanks for your help.


Notification Reminder in Python.

A script that will create pop-up notification after specified time.

can i work on this project? please assing me this

hi, @imsushant12 would you like to work on this ? or this time do you want to give chance to @mrvishalsharma ? sweat_smile

hi, @imsushant12 would you like to work on this ? or this time do you want to give chance to @mrvishalsharma ? sweat_smile

hi @viralvaghela , Please assign @mrvishalsharma  this time :)

hi, @imsushant12 would you like to work on this ? or this time do you want to give chance to @mrvishalsharma ? sweat_smile

hi @viralvaghela , Please assign @mrvishalsharma this time :)

can i work on this issue?

Hi @mrvishalsharma , @imsushant12 is already working on that ,you can try something new !


Add penalty if has no shields, and Skip next Turn

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
258

Relevant Lines:
258

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
258

Relevant Lines:
258

💛  - Coveralls


Enable SSL_MODE_AUTO_RETRY by default

Because TLS 1.3 sends more non-application data records some clients run
into problems because they don't expect SSL_read() to return and set
SSL_ERROR_WANT_READ after processing it.
This can cause problems for clients that use blocking I/O and use
select() to see if data is available. It can be cleared using
SSL_CTX_clear_mode().

The documentation update itself is in #6240

I'm totally confused currently. When I try to test this with s_client in master (so without this patch), it already behaves like what I expect the SSL_MODE_AUTO_RETRY to do. The SSL_write() does not return until the renegotiation is done.

I have to guess that not all non-application data is being treated the same. A TLS 1.2 renegotiation happens in a single SSL_read()/SSL_write() call when using blocking I/O, while a TLS 1.3 new session ticket or key update makes SSL_read() return and sets the error to SSL_ERROR_WANT_READ. An SSL_write() after an SSL_key_update() just sends it's own packet and returns successfully. I can never make SSL_write() return SSL_ERROR_WANT_READ using blocking I/O.

A TLS 1.2 renegotiation happens in a single SSL_read()/SSL_write() call when using blocking I/O

Is this the case even with a full (non resumption) renegotiation?

I can never make SSL_write() return SSL_ERROR_WANT_READ using blocking I/O.

I wonder what happens if SSL_write() is the first thing you call (i.e. before a handshake has occurred). That should initiate a handshake, and I would expect to see SSL_ERROR_WANT_READ.

I wonder what happens if SSL_write() is the first thing you call (i.e. before a handshake has occurred). That should initiate a handshake, and I would expect to see SSL_ERROR_WANT_READ.

Oh, that's a case I didn't try yet, I will try this later.


Removing Text Overflow

Hey! I noticed that the text inside the message box overflows in case of long messages. I'd like to work on this issue and remove the overflow. I'm a GSSOC'21 participant.

@AnjaliPatle Yes you can start working on the issue.

@AnjaliPatle What is the status ? Kindly keep me updated !

I've created a PR for this issue. Kindly checkout PR #75 .

Merged with PR #75


Export all depth map levels

Hi,
The line code "depth = chunk.depth_maps[camera].image()" gets the depth map level 1. How can I get all depth map levels?
Thank you in advance :)

Hi,
Just use level number as an argument to image (zero level is the most detailed):
level = 0
depth_level = chunk.depth_maps[camera].image(level)

Thank you for the rapid response @PolarNick239
I have tried it, but it pops up an error saying that image() takes no arguments (1 given). On which version of metashape it supossed to work? I am using V1.5.2
Best regards,
Jordi

Oh, yes, it was introduced in version 1.5.3. And if the depth map doesn't have a specified level - the image() will return None.

Please feel free to reopen the issue if you still have the problem.


querying by tx id fails

Describe the bug
querying by tx id fails
To Reproduce
Steps to reproduce the behavior:
let indexer_result = Indexer::new("https://algoindexer.testnet.algoexplorerapi.io/");
let indexer = indexer_result.unwrap();
let mut txn_query = QueryTransaction::default();
txn_query.txid = Some("Y5I4TYMKJI3MYJHBQ46DRHEXFW7BTURBKZ7V2NF2OU4QEHLYTKBQ".to_owned());
let txns_result = indexer.transactions(&txn_query).await;

gives an error trying to GET
https://algoindexer.testnet.algoexplorerapi.io/v2/transactions?tx-type=Y5I4TYMKJI3MYJHBQ46DRHEXFW7BTURBKZ7V2NF2OU4QEHLYTKBQ
which is asking for tx-type, whereas the code was filtering on txid.
Expected behavior
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]: macOS
Browser [e.g. chrome, safari]: chrome
Version [e.g. 22]:

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Closed by #208


Fixes #286 Allows non english character based hashtags to be correctly identified.

Sample string which passes the regex Strings previously used to break if àáâäæãåābçćčdèéêëēėęfghîïíīįìjkłmñńôöòóœøōõpqrßśštûüùúūvwxÿžźż were present.
Now it passes this too.

Mergng #289


Sync eng/common directory with azure-sdk-tools for PR 4713

Sync eng/common directory with azure-sdk-tools for PR https://github.com/Azure/azure-sdk-tools/pull/4713  See eng/common workflow

/check-enforcer override


Handle Korean with an extra 'span' layer under  XPath selector

The reason why wikipron didn't work for Korean is that Korean has an extra <span> layer (for specifying fonts, it looks like), which couldn't be handled by the existing XPath selector of //li[sup[a and would require something to the effect of //li[span[sup[a. This PR expands this <li> XPath selector to handle Korean. I wouldn't be surprised if CKJ or other languages that would need special font treatment would benefit from this change.
This PR also reactivates the language/wikipron/scrape script for Korean and commits the scraped data as well.

Keeping this PR as a draft, as scraping for Korean is in progress. 5500 pronunciations scraped for far; we're expecting ~12,000. 🤞  I'll also update the changelog.


Improve publishing docs to recommend passing tokens via an environment variable

The publishing docs suggest a command line like this:
rush publish -a -b targetBranch -p -r registryUrl -n npmToken

It's insecure to include security credentials on a command-line, because they are likely
to be included in a log file.  We should show how to use an environment variable instead.

I think we can actually improve Rush in this regard: https://github.com/microsoft/rushstack/issues/1588


Getting 'Exception in callback of async function: TypeError: Cannot call method 'toString' of undefined' with default test

I just did meteor add xolvio:cucumber and then added default test (by clicking on button in velocity panel). Now I'm getting this error:
I20150407-00:13:11.737(3)? [xolvio:cucumber] Cucumber is running
I20150407-00:13:11.881(3)? Exception in callback of async function: TypeError: Cannot call method 'toString' of undefined
I20150407-00:13:11.881(3)?     at Object.parse (/Users/jaro/.meteor/packages/xolvio_cucumber/.<IP_ADDRESS>vnxmns++os+web.browser+web.cordova/npm/node_modules/cucumber/lib/cucumber/parser.js:15:52)
I20150407-00:13:11.882(3)?     at Object.getFeatures (/Users/jaro/.meteor/packages/xolvio_cucumber/.<IP_ADDRESS>vnxmns++os+web.browser+web.cordova/npm/node_modules/cucumber/lib/cucumber/runtime.js:30:35)
I20150407-00:13:11.882(3)?     at Object.start (/Users/jaro/.meteor/packages/xolvio_cucumber/.<IP_ADDRESS>vnxmns++os+web.browser+web.cordova/npm/node_modules/cucumber/lib/cucumber/runtime.js:16:37)
I20150407-00:13:11.882(3)?     at _rerunCucumber (packages/xolvio:cucumber/server.js:132:1)
I20150407-00:13:11.882(3)?     at runWithEnvironment (packages/meteor/dynamics_nodejs.js:108:1)

Hi
Can you try 0.6.0-rc.1
That's the new version that will be published soon
Thanks
On Mon, 6 Apr 2015 14:17 Jaro<EMAIL_ADDRESS>wrote:

I just did meteor add xolvio:cucumber and then added default test (by
clicking on button in velocity panel). Now I'm getting this error:
I20150407-00:13:11.737(3)? [xolvio:cucumber] Cucumber is running
I20150407-00:13:11.881(3)? Exception in callback of async function: TypeError: Cannot call method 'toString' of undefined
I20150407-00:13:11.881(3)?     at Object.parse (/Users/jaro/.meteor/packages/xolvio_cucumber/.<IP_ADDRESS>vnxmns++os+web.browser+web.cordova/npm/node_modules/cucumber/lib/cucumber/parser.js:15:52)
I20150407-00:13:11.882(3)?     at Object.getFeatures (/Users/jaro/.meteor/packages/xolvio_cucumber/.<IP_ADDRESS>vnxmns++os+web.browser+web.cordova/npm/node_modules/cucumber/lib/cucumber/runtime.js:30:35)
I20150407-00:13:11.882(3)?     at Object.start (/Users/jaro/.meteor/packages/xolvio_cucumber/.<IP_ADDRESS>vnxmns++os+web.browser+web.cordova/npm/node_modules/cucumber/lib/cucumber/runtime.js:16:37)
I20150407-00:13:11.882(3)?     at _rerunCucumber (packages/xolvio:cucumber/server.js:132:1)
I20150407-00:13:11.882(3)?     at runWithEnvironment (packages/meteor/dynamics_nodejs.js:108:1)
—
Reply to this email directly or view it on GitHub
https://github.com/xolvio/meteor-cucumber/issues/67.

How to do that?

edit your .meteor/packages file to say<EMAIL_ADDRESS>

Unfortunately still getting error:
package-version-parser            removed from your project
sanjo:long-running-child-process  added, version 1.0.3
sanjo:meteor-version              removed from your project
velocity:chokidar                 added, version 0.12.6_1
velocity:core+                    upgraded from 0.4.5 to 0.6.0-rc.4
velocity:meteor-internals         added, version 1.1.0_5
xolvio:cucumber                   upgraded from 0.5.5 to 0.6.0-rc.1

+ In order to resolve constraints, we had to use experimental versions of these
packages.
W20150407-14:20:23.006(3)? (STDERR)
W20150407-14:20:23.007(3)? (STDERR) /Users/jaro/.meteor/packages/meteor-tool/.1.1.3.spsu4v++os.osx.x86_64+web.browser+web.cordova/mt-os.osx.x86_64/dev_bundle/server-lib/node_modules/fibers/future.js:245
W20150407-14:20:23.008(3)? (STDERR) 						throw(ex);
W20150407-14:20:23.008(3)? (STDERR) 						      ^
W20150407-14:20:23.008(3)? (STDERR) TypeError: Cannot set property 'start' of undefined
W20150407-14:20:23.008(3)? (STDERR)     at /Users/jaro/Dev/Meteor/toic/.meteor/local/build/programs/server/packages/underscore.js:882:21
W20150407-14:20:23.008(3)? (STDERR)     at Array.forEach (native)
W20150407-14:20:23.008(3)? (STDERR)     at _.each._.forEach (/Users/jaro/Dev/Meteor/toic/.meteor/local/build/programs/server/packages/underscore.js:139:11)
W20150407-14:20:23.008(3)? (STDERR)     at Function._.extend (/Users/jaro/Dev/Meteor/toic/.meteor/local/build/programs/server/packages/underscore.js:879:5)
W20150407-14:20:23.008(3)? (STDERR)     at DEBUG (/Users/jaro/Dev/Meteor/toic/.meteor/local/build/programs/server/packages/velocity_node-soft-mirror.js:449:5)
W20150407-14:20:23.008(3)? (STDERR)     at DEBUG (/Users/jaro/Dev/Meteor/toic/.meteor/local/build/programs/server/packages/velocity_node-soft-mirror.js:539:3)
W20150407-14:20:23.008(3)? (STDERR)     at /Users/jaro/Dev/Meteor/toic/.meteor/local/build/programs/server/packages/velocity_node-soft-mirror.js:543:4
W20150407-14:20:23.008(3)? (STDERR)     at /Users/jaro/Dev/Meteor/toic/.meteor/local/build/programs/server/packages/velocity_node-soft-mirror.js:768:3
W20150407-14:20:23.008(3)? (STDERR)     at /Users/jaro/Dev/Meteor/toic/.meteor/local/build/programs/server/boot.js:222:10
W20150407-14:20:23.008(3)? (STDERR)     at Array.forEach (native)

Now I'm getting errors because of upgraded velocity.core:
package-version-parser            removed from your project
sanjo:long-running-child-process  added, version 1.0.3
sanjo:meteor-version              removed from your project
velocity:chokidar                 added, version 0.12.6_1
velocity:core+                    upgraded from 0.4.5 to 0.6.0-rc.4
velocity:meteor-internals         added, version 1.1.0_5
xolvio:cucumber                   upgraded from 0.5.5 to 0.6.0-rc.1

+ In order to resolve constraints, we had to use experimental versions of these
packages.
W20150407-14:20:23.006(3)? (STDERR)
W20150407-14:20:23.007(3)? (STDERR) /Users/jaro/.meteor/packages/meteor-tool/.1.1.3.spsu4v++os.osx.x86_64+web.browser+web.cordova/mt-os.osx.x86_64/dev_bundle/server-lib/node_modules/fibers/future.js:245
W20150407-14:20:23.008(3)? (STDERR) 						throw(ex);
W20150407-14:20:23.008(3)? (STDERR) 						      ^
W20150407-14:20:23.008(3)? (STDERR) TypeError: Cannot set property 'start' of undefined
W20150407-14:20:23.008(3)? (STDERR)     at /Users/jaro/Dev/Meteor/toic/.meteor/local/build/programs/server/packages/underscore.js:882:21
W20150407-14:20:23.008(3)? (STDERR)     at Array.forEach (native)
W20150407-14:20:23.008(3)? (STDERR)     at _.each._.forEach (/Users/jaro/Dev/Meteor/toic/.meteor/local/build/programs/server/packages/underscore.js:139:11)
W20150407-14:20:23.008(3)? (STDERR)     at Function._.extend (/Users/jaro/Dev/Meteor/toic/.meteor/local/build/programs/server/packages/underscore.js:879:5)
W20150407-14:20:23.008(3)? (STDERR)     at DEBUG (/Users/jaro/Dev/Meteor/toic/.meteor/local/build/programs/server/packages/velocity_node-soft-mirror.js:449:5)
W20150407-14:20:23.008(3)? (STDERR)     at DEBUG (/Users/jaro/Dev/Meteor/toic/.meteor/local/build/programs/server/packages/velocity_node-soft-mirror.js:539:3)
W20150407-14:20:23.008(3)? (STDERR)     at /Users/jaro/Dev/Meteor/toic/.meteor/local/build/programs/server/packages/velocity_node-soft-mirror.js:543:4
W20150407-14:20:23.008(3)? (STDERR)     at /Users/jaro/Dev/Meteor/toic/.meteor/local/build/programs/server/packages/velocity_node-soft-mirror.js:768:3
W20150407-14:20:23.008(3)? (STDERR)     at /Users/jaro/Dev/Meteor/toic/.meteor/local/build/programs/server/boot.js:222:10
W20150407-14:20:23.008(3)? (STDERR)     at Array.forEach (native)

Do you use another testing framework besides xolvio:cucumber?

Yes, mike:mocha

Ok, right now the latest version of mike:mocha does not work with the new velocity:core. I've made a breaking change. I will inform him that he should release a new version.
For testing xolvio:cucumber you would need to remove mike:mocha temporarly.

Ok. Thanks.
After removing mike:mocha package, xolvio:cucumber started without errors but...
I'm getting an error after adding sample tests:
Exception while invoking method 'velocity/copySampleTests' TypeError: Arguments to path.join must be strings
I20150407-14:50:45.399(3)?     at path.js:360:15
I20150407-14:50:45.399(3)?     at Array.filter (native)
I20150407-14:50:45.399(3)?     at Object.exports.join (path.js:358:36)
I20150407-14:50:45.399(3)?     at Object.pathJoin (/Users/jaro/Dev/Meteor/toic/.meteor/local/build/programs/server/packages/velocity_meteor-internals.js:660:16)
I20150407-14:50:45.399(3)?     at /Users/jaro/Dev/Meteor/toic/.meteor/local/build/programs/server/packages/velocity_core.js:588:36
I20150407-14:50:45.400(3)?     at Array.forEach (native)
I20150407-14:50:45.400(3)?     at [object Object].Meteor.methods.velocity/copySampleTests (/Users/jaro/Dev/Meteor/toic/.meteor/local/build/programs/server/packages/velocity_core.js:587:21)
I20150407-14:50:45.400(3)?     at [object Object].methodMap.(anonymous function) (/Users/jaro/Dev/Meteor/toic/.meteor/local/build/programs/server/packages/meteorhacks_kadira.js:2495:30)
I20150407-14:50:45.400(3)?     at maybeAuditArgumentChecks (/Users/jaro/Dev/Meteor/toic/.meteor/local/build/programs/server/packages/ddp.js:2445:12)
I20150407-14:50:45.400(3)?     at /Users/jaro/Dev/Meteor/toic/.meteor/local/build/programs/server/packages/ddp.js:1476:20

Can you please try with meteor add<EMAIL_ADDRESS>please? Please be sure to see the breaking changes section.

@samhatoum now I have no errors with<EMAIL_ADDRESS>Closing the tasks.


Sequencediagram text with  BR tag with space fails

This works
working example
sequenceDiagram
    participant Joe
    participant Alice
    Joe->>Alice: Foo
    Note over Alice: Bar
Baz
    Note over Joe,Alice: That note above should have had a carriage return between Bar and Baz

but this, with a space between the r and / fails by rendering the
 as text not as a line break. One tricky thing is that if you look at it in the interactive view, and then type something on the line with the
 it displays correctly, but the view link continues to display incorrectly.
non working example
sequenceDiagram
    participant Joe
    participant Alice
    Joe->>Alice: Foo
    Note over Alice: Bar
Baz
    Note over Joe,Alice: That note above should have had a carriage return between Bar and Baz

This may relate to some challenges people had in #384

@watsoncj You'd said that PR #764 fixes this bug, but when I go and test the non working example above, it still manifests this bug.
Is this because when @tylerlong merged your PR it hasn't yet been manifested in the code used on mermaidjs.github.io? I'm assuming that your code would be live now considering it was merged back in November 2018

@watsoncj You'd said that PR #764 fixes this bug, but when I go and test the non working example above, it still manifests this bug.
@tylerlong merged your PR into 7.0.2 but the bug persists in the current version of 8.3.0.

This seems to be similar to #1088. Looking at the commit mentioned above, the regex
 is used to determine the lines. This takes care of
 and
 but does not find
.
If noone is already working at this issue, I can take a look at it.

PR coming up for the weekend I suppose.


Added code to reverse transforms for more flexibility

New attempt at improving the flexibility and use of "pre-made" transforms such as masses transforms etc.
Source code changes
I have added two utility functions:

One function to create a BijectiveTransform given a name_mapping and the forward and backward transformation functions which take arrays as input. That way, we can easily create these "pre-made" transforms by specifying these in a one-liner. At the same time, it allows users to create similar transforms themselves very easily, which therefore also makes the name_mapping no longer hardcoded for the masses since users can change it if they desire, as @thomasckng mentioned some time ago.
Another function reverses a BijectiveTransform object, as @kazewong suggested in my previous PR attempt.

These functions are then used to simplify the "pre-made" transforms in single_event/transforms.py to be shorter and clearer.
Test changes

Made sure the GW150914_D.py file in test/integration matches this new implementation.
Added a new toy example test file to play around with the transforms in a simplified setting.

@ThibeauWouters I think the test script in test/ are meant to be light-weighted, as they will run whenever there is a new commit. They meant to check if everything is working instead of generating meaningful result. A example with optimized parameters are meant to be put in example/. I am canceling the test runs now.

@thomasckng Thanks! I had local copies of the test file to try more agressive hyperparameters but indeed I seemed to have messed them up with the ones in the test directory. Thanks for canceling the runs.

@ThibeauWouters I am planning to merge my PR with your PR. There are some conflicts in the test directory. I'll be resolving these conflicts by overwriting your files in the test directory with my files. Please let me know if there is still some changes in your test directory that you wish to preserve.

@ThibeauWouters I have created a PR to merge my commits into your PR. Could you please take a look at it to see if there is any issue?

@kazewong I adapted the code to adapt to your previous comment.  The former require users to give name_mapping, which as I mentioned earlier on is a bit redundant since we would like the name mapping to be fixed for these classes, whereas the former doesn't. So the predefined transform and its reverse are not treated equally in the current implementation.

@kazewong I adapted the code to adapt to your previous comment. But now it feels a bit off in the sense that the "predefined" transforms are classes, that have to be instantiated in order to get the actual transform to be used in PE, and the reverse transforms that are created from the reverse function are objects/instances already. The former require users to give name_mapping, which as I mentioned earlier on is a bit redundant since we would like the name mapping to be fixed for these classes, whereas the former doesn't. So the predefined transform and its reverse are not treated equally in the current implementation.

There are a couple of things you mentioned here:

Re: predefined transform and its reverse are not treated equally: I don't think they should.  For example, reverse_bijective_transform(ScaleTransform((['x'], ['y']), 1.0)) should return a valid transform. As long as this is the interface the user will use, I think this is legit.
If we later establish predefined transform can be called with PredefinedTransform(), the interface makes sense to me still, which will be replacing PredefinedTransform() with reverse_bijective_transform(PredefinedTransform()) at the same spot.

I think this is more ergonomic for the user, instead of needing to use a function to call the predefined transform.
Going forward, I do think predefined transforms that enforce the naming strictly should mostly be constructed using the interface PredefinedTransform(). Only transform that does not enforce naming such as ScaleTransform should allow inputs other than the transform parameters.

@thomasckng @kazewong I have updated the PR a bit after the discussion yesterday. Let me know if this looks OK to you or if further changes are required.

@thomasckng The files have been deleted

@kazewong I have updated the source code as requested -- is this what you were referring to?


Transaction.Parse obsolete

In the latest version of NBitcoin (<IP_ADDRESS>), I receive a warning when I use Transaction.Parse(...):
Transaction.Parse(string)' is obsolete: 'You should instantiate Transaction from ConsensusFactory.CreateTransaction'
I looked in the unit test code (transaction_tests.cs) and see you're still using the Parse method:
Transaction tx = Transaction.Parse(raw);
Are the unit tests obsolete? Or should I ignore the warning?
I don't see any way to parse tx hex using the consensus factory.

I see your change. It's working now.
Transaction tx = Transaction.Parse(raw, ntwk);
Thanks.


Editable rows with editable-select, on adding new row, how to make the drop down appear automatically

In the example for this I notice that when you add a row, then immediately the drop down is shown as obviously clickable - basically that the rowform.show() function has fired somehow. But I don't see where that's firing in the example upon creating a new row, and therefore in the example's case, a new user object in the repeated array of users.
I solved this with a sort of hacky code where I passed rowform through the showGroup function and called rowform.show() in the case that the group's id is zero, as it is after being created and pushed into the array, awaiting selection in the drop down.
But how is it the example accomplishes this automatically? Here's the JS fiddle: http://jsfiddle.net/NfPcH/93/

When you click add row, it calls the addUser function which adds a new item to the array.
Is there a specific issue you are having with this?

@eugef Please close due to lack of response.


[FEATURE REQUEST] Add option to configure environment that agents will be assigned to

Hi,
I'm testing out this plugin, I'm having a hard time finding how to assign the elastic agents to an environment, otherwise my pipelines won't run on the elastic agents.
Having for example:
2 envs (stg/prd)
3 pipelines (pl1, pl2, pl3)

pl1 is assigned to stg
pl2 is assigned to prd
pl3 is not assigned to any env

Only pl3 will run on the ec2 elastic agents, because I have no way to tell them they are assigned to an environment.
According to the docs here: https://docs.gocd.org/current/advanced_usage/agent_auto_register.html you can set the agent.auto.register.environments so the agent get assigned, the problem is there is no way to do this using the EC2 Elasting plugin because whatever you set in the userData will be executed after the agent has started and registered.
As a workaround, I have patched your source so the starting of the agent is run after everything else, but this is ugly, is there any change you can implement another setting in the elasticAgentProfile so we can configure the environments to be assigned?
Thanks!

With the above patch and then setting in the profile config userData something like:
echo "agent.auto.register.environments=env1,env2,env3..." >> /var/lib/go-agent/config/autoregister.properties

I can get the agents to register with GoCD using environments.
The ideal way would be to have a separate configuration param that can be empty (nothing changes) or can be set (the agent.auto.register.environments param gets added).

I am also trying to do the same thing, I provided ami and other config details on elastic agent configuration and I have created a file /var/lib/go-agent/config/autoregister.properties and provided auto registration key in it as mentioned in https://docs.gocd.org/current/advanced_usage/agent_auto_register.html.
but I am getting such errors:
Error occurred when agent tried to ping server:
org.springframework.remoting.RemoteAccessException: Could not access HTTP invoker remote service at [https:///go/remoting/remoteBuildRepository]; nested exception is org.apache.http.client.ClientProtocolException: The server returned status code 403.
can you please help me with this

Hi @ShubhamAgarwal1616, I'm not sure it's the same problem. Sounds to me you are configuring the autoregister.properties manually anbd you don't need to, the EC2 plugin uses the ec2 userdata to automatically configure the agents for auto-registration.

Hi @joelsdc!
As you figured out yourself we didn't implement environments feature in this plugin and this is not something we have on our roadmap right now, but maybe in a feature we will. If you're interested I encourage you to implement it yourself and make a pull request.
The workaround you suggested makes sense to me, so I will merge it.
Thanks!

I've been using the workaround for a couple weeks now with no problems. I don't think it's worth either as this does in fact solve the problem. Thanks for merging!

Will you create a new release or not until you have more significant changes?

I think it would be great to test the plugin with the newest version of GoCD server first. What version you have been running?

I didn’t now there could be a version limitation, I started with v20.2.0, and updated recently to v20.3.0. So far no issues...


state.matches is undefined in 0.13.3

I tried to use this example code on server:
something.serve(function (req, res) {
  Router.run(routes, req.path, function (Root, state) {
    // could fetch data like in the previous example
    fetchData(state.matches).then(function (data) {
      var html = React.renderToString(<Root data={data} />);
      res.send(html);
    });
  });
});

But state.matches is always undefined.
This code is coming directly from doc for 0.13.3.

Thanks for pointing that out. Should be state.routes as far as I can tell.


[5.3] Add 'html' cast

This PR adds a 'html' cast to eloquent models so you can mark certain attributes as HTML-safe.
Useful for simplifying blade templating so you can always use {{ }} without worrying about marking attributes as html-safe.

Think I will hold off on including this in the core. My worry is that people will mark all displayable attributes this way and make themselves vulnerable to XSS.


env variable for nerdctl isn't set

after a fresh install via brew install lima today
asdf@MacBookPro ~ % lima XDG_RUNTIME_DIR=/run/user/501 nerdctl ps
CONTAINER ID    IMAGE    COMMAND    CREATED    STATUS    PORTS    NAMES

# works

asdf@MacBookPro ~ % lima nerdctl ps
WARN[0000] environment variable XDG_RUNTIME_DIR is not set, see https://rootlesscontaine.rs/getting-started/common/login/
WARN[0000] environment variable XDG_RUNTIME_DIR is not set, see https://rootlesscontaine.rs/getting-started/common/login/
FATA[0000] rootless containerd not running? (hint: use `containerd-rootless-setuptool.sh install` to start rootless containerd): environment variable XDG_RUNTIME_DIR is not set, see https://rootlesscontaine.rs/getting-started/common/login/

# doesn't work
asdf@MacBookPro ~ %

i edited nerdctl.lima for the time being
cat nerdctl.lima
#!/bin/sh
set -eu
exec lima XDG_RUNTIME_DIR=/run/user/501 nerdctl "$@"

Normally systemd sets these.

Confirmed, this happened to me as well

sudo systemctl status -l systemd-logind
loginctl


refactor ProxyPool->killProcessByPort to not have dependency onto the lsof utility

we have to make killProcessByPort os independent at all or upgrade to new iplementation of BrowserMob proxy where issue with stopping fixed...

closing as won't fix