Feature/dockerized app

Really love this project! I thought it could benefit from a simple Docker build/run setup.
Eventually worth considering pushing that on the registry, so people could pull the image and run it without installing any dependencies!

The check that failed will be solved by merging https://github.com/pinterest/snappass/pull/11

Hi @samueldg , thanks for the PR! Great idea to dockerize this.
I have one question: I'm not very proficient in docker, and I'm wondering if it's a better idea to have REDIS run on the same docker, or if it's better to have it running in a separate one (that is my understanding of what your docker-compose file is for.

Thanks for the quick replies @nichochar!
It is generally advised to run, as much as possible, only one process per container (ref. Docker best practices)
A few advantages off the top of my head:

Simplicity - Using python + redis base images, instead of doing manual steps in another base image;
Easier to upgrade - If we want to upgrade the Redis image, we can simply update the tag (now I'm using "latest" which will result in that behaviour);
Scalability - We can scale the python app container if we need to without running more Redis instances. For example, if someone really wanted to have more servers running, it would be as easy as running docker-compose scale and load balancing. All servers would talk to the same Redis store;
Integration in existing tools - orchestration and monitoring tools will be simpler to use as the processes can be interacted with separately.

@nichochar Rebase done.

looks good. agree that the redis is better separated from snappass's dockerfile. Thanks for the dockerizing snappass.

Great! Thanks @samueldg and @yongwen !


fix(flat-stores): fix wrong target

PNPM seems using a different key packages (where npm and yarn uses workspaces) -- https://github.com/JasonMan34/pnpm/commit/6114bcef10d1291bfd1c6e09871598fc2ae9502f


Broken link in Excel sheet

The downloadable Excel workbook (in the green box) that generates the information barriers PowerShell has a broken documentation link for "Attributes for information barrier policies".
Thanks for the spreadsheet - it's very helpful!

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

ID: 2baec804-4a76-95ac-3595-985629448fd9
Version Independent ID: c0a5fb9d-93be-b448-5bf2-4c3bdd787b8e
Content: Define information barrier policies - Microsoft 365 Compliance
Content Source: microsoft-365/compliance/information-barriers-policies.md
Service: o365-seccomp
GitHub Login: @chrfox
Microsoft Alias: chrfox

@EricaToelle Thank you for your feedback. We have created fix this excel file and waiting for it to go live once the author approves. Related PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/2532
Thanks
Sri

@msbemba Thank you very much for the contribution and updating the documentation with PR. @EricaToelle Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


IDENTITY-6962 Client side validator for Charon 3

Moved from https://wso2.org/jira/browse/IDENTITY-6962
currently, there is no single method to validate SCIM 2 object received from the server side.

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


Avoid clipboard history to store passwords

I've been using the clipboard history very frequently and I noticed that whenever I copy a password from 1Password on my personal computer the password is ignored but when I copy a password that is stored on Safari's password manager or KeePassX, which I use at my work's computer for compliance, the password is stored in the clipboard history and this should not happen, is there a way to ignore those?

Hey @dimitriacosta ! This repository is for issues related to the ray.so ecosystem. Support questions are better asked in the community Slack or by emailing<EMAIL_ADDRESS>To give a quick answer to your question, you can disable applications from being included in the clipboard history settings.

Thank you @samuelkraft I didn't know where to post this so sorry if this wasn't supposed to be here.
Anyways thanks for the information, I'll add those to the disabled applications list.


Import "pox.core" could not be resolved

I'm trying to modify the L2_learning script, and re-implement a portion of L2_learning myself using Pox.
"When controler sees a packet with destination address Y, source address X and source port P, controller will lean that a packet from address X arrived at port P, then if a packet arrives with destination X, it will be sent through port P."
I digress, when I try opening the example L2_learning script in Visual studio code, the IDE does not see the libraries being imported.
I have ran
pip install pox
pip3 install pox

and both result in
Defaulting to user installation because normal site-packages is not writeable
Requirement already satisfied: pox in ./.local/lib/python3.10/site-packages (0.3.2)

However Pylance is not seeing the packages existing, even after a machine restart. Is there something I've missed in getting these libraries? I have pox installed.

I have resolved this issue, for reference I am using Python 3.10.7, on Ubuntu.
The problem here was that the pox installation directory wasn't added to my PYTHONPATH. What you need to do is

Figure out where your python installation looks for site packages if not obvious.
python -m site --user-site

Go to the site packages directory (yours might be different)
cd /home/user.local/lib/python3.10/site-packages

Create a .pth file here and add your pox directory. This does it in one line.
echo "/home/user/Downloads/pox" > pox.pth

This resolves the problem of missing libraries.


Modal, Open choice view does not show fly-over text.

Describe the bug
Modal and open choice view does not show fly over text on inclusion of flyover code to questionnaire-itemControl extension.
e.g
"extension": [
{
"url": "http://hl7.org/fhir/StructureDefinition/questionnaire-itemControl",
"valueCodeableConcept": {
"coding": [
{
"system": "http://hl7.org/fhir/questionnaire-item-control",
"code": "flyover",
"display": "Fly-over"
}
],
"text": "Flyover"
}
}
]
Component
SDC library
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
flyover text should be shown.
Screenshots
If applicable, add screenshots to help explain your problem.
Smartphone (please complete the following information):

Device: [e.g. Pixel4a emulator]
Android version: [e.g. Settings -> About phone -> Android version]
Build number: [e.g. Settings -> About phone -> Build number]
Version [e.g. 22]

Additional context
Add any other context about the problem here.
Would you like to work on the issue?
Please state if this issue should be assigned to you or who you think could help to solve this issue.

@shelaghm is it okay to have this change in sdc module?

@santosh-pingle yes, updating in SDC module/library sounds good to me.

@PallaviGanorkar
Flyover text is missing from below screenshot.

You can refer Dropdown component workflow to get expected workflow, where flyover text is shown.
Just attaching screenshot for reference.

@shelaghm I have added "Health conditions" as flyover text for open choice component in Catalog app. Please let me know if you have other thoughts. Thanks!

@PallaviGanorkar Looks good to me


Question about xcode build property name

Is this name correct?
Isn't should be 'FRAMEWORK_SEARCH_PATHS'?
https://github.com/DavidPeicho/unity-swiftui-example/blob/a149dbecf72d8d4c24a18e1fb05d24701296c7dd/unityapp/Assets/Editor/AutoBuilder.cs#L64

I guess it should yes! Thanks for noticing. Then it means that it must be automatically set properly by Unity or isn't needed anymore because it works...


Beacon markers on linear compass don't follow the compass exactly

https://nextcloud.swedneck.xyz/s/rytpwRn3bYMjaSm

That's really weird, I'll look into it


Add jest/supertest integration tests to server

All endpoints are tested with an in-memory mongodb database so no external dependencies are needed for passing the tests.
Babel now transpiles the server code so ES6 features can be used also in tests.
closes #11

nastaa


Model params for GPU setup

Hello @nexus! How do I choose the parameters for the models for my GPU setup? I'm manipulate different params, as example n_ctx or n_batch, but is so long, and not effectivity. Maybe do you know formula or method for calculate optimal params for GPU.
I have 2 1080Ti with 11 gb memory, please give me advice ))

Hi,
Sorry, I am not aware of a formula that allows calculating it easily. Would start with monitoring the GPU usage (on Linux, you can use nvidia-smi tool) and optimise the parameters one by one.


sync

describe your changes...

[ ] Read and sign the CLA. PRs that haven't signed it won't be accepted.
[ ] Make sure your PR complies with the writer's guide.
[ ] Review the diff carefully as sometimes this can reveal issues.
[ ] Do not abandon your Pull Request: Stale Pull Requests.
Remove these instructions from your PR as they are for your eyes only.

@.***的自动回复邮件。已收到您的邮件！


Plugin error reported by IntelliJ IDEA 2024.2 (Ultimate Edition)

IntelliJ is now reporting the following exception in the IvyIDEA (1.0.17) plugin:
com.intellij.diagnostic.PluginException: `ActionUpdateThread.OLD_EDT` is deprecated and going to be removed soon. 'org.clarent.ivyidea.ResolveForActiveModuleAction' must override `getActionUpdateThread()` and chose EDT or BGT. See ActionUpdateThread javadoc. [Plugin: IvyIDEA]
	at com.intellij.diagnostic.PluginProblemReporterImpl.createPluginExceptionByClass(PluginProblemReporterImpl.java:23)
	at com.intellij.diagnostic.PluginException.createByClass(PluginException.java:90)
	at com.intellij.diagnostic.PluginException.reportDeprecatedUsage(PluginException.java:125)
	at com.intellij.openapi.actionSystem.ActionUpdateThreadAware.getActionUpdateThread(ActionUpdateThreadAware.java:21)
	at com.intellij.openapi.actionSystem.AnAction.getActionUpdateThread(AnAction.java:201)

Build #IU-242.20224.300, built on August 6, 2024

Thanks for the report.
This will be fixed in the next release (1.0.18).

Is there a target date for 1.0.18?  Or should I downgrade my IntelliJ to get Ivy resolving back?  Thanks for the open-source work!

I'll try to release it somewhere next week.
It should work however, IntelliJ just shows a warning, but everything should still work. There is no need to downgrade.


Potential race condition bug inside choices_to_shell_cmd_previewer

I just solved a bug in fzf-lua that uses similar code to choices_to_shell_cmd_previewer, it might affect nvim-fzf so you may want to look into it.
While implementing a "remembering" live_grep (i.e. every keystroke gets saved so that you can continue your live search exactly where you left off), I had implemented a raw_async_function with code borrowed from choices_to_shell_cmd_previewer that runs rg in the background and populates the results in fzf.
While testing this I noticed that the result set would vary randomly in size and the function was returning a partial result set.
Troubleshooting the issue I discovered that read_cb could be called multiple times before even a single uv.write is complete, imagine a  print at the start of read_cb and at the completed callback of uv_write, the printout would more often than not look like this:
read_cb
uv.write
read_cb
uv.write
read_cb
read_cb
read_cb    <-- end of data
uv.write
uv.write
uv.write

Now let's assume that the marked read_cb is be the last call and would be sent with data = nil - that would trigger a cleanup() call, close the fzf pipe and the all the following uv.write calls would err.
Hope this makes sense over text.
Now for the solution, I implemented a simple read_cb_count variable and made sure that the pipe wouldn't be closed unless the last uv.write call is complete (i.e. read_cb_count = 0)
You can take a look at the code here, hopefully this would make sense after you read the code, lmk your thoughts?
https://github.com/ibhagwan/fzf-lua/blob/57ce1bff55a287ce4f66b90e8448f32b860d22cd/lua/fzf-lua/core.lua#L305-L363

Thanks for the report! I've been a bit busy lately, I will get to this soon!

No worries, take your time, I’m not getting any reports on this being broken.

Finally got it, and I believe I have fixed it. Thank you so much for the well written issue, must have been pretty difficult to diagnose. Concurrency is always difficult haha.

Finally got it, and I believe I have fixed it. Thank you so much for the well written issue, must have been pretty difficult to diagnose. Concurrency is always difficult haha.

As long as it’s not kernel level multi thread debugging I’m happy :)
Interesting implementation, you prefer the cleanup_request_sent to uv.is_closing check? any specific reason? Does it have higher overhead than what is most likely  an internal library boolean test?

Handles that wrap file descriptors are closed immediately but callback will still be deferred to the next iteration of the event loop. It gives you a chance to free up any resources associated with the handle.

My implementation was designed under the possibility of long running processes, where people might try to close the pipe at a time where no write callback is scheduled. I think my implementation handles that edge case.
I'm not really sure about the utility of uv.is_closing(), I feel like one only needs it when they already close a pipe (using uv.close()), and then need to query whether they did that before the close callback. My implementation only calls uv.close() once and then never does anything after that, so I didn't think I needed it?

Ah yes, that is correct, which I why I had the boolean to prevent closing twice. Yep I agree, just different implementations of the same bugfix :)


add missing strings to translate files

Coverage remained the same at 52.786% when pulling 38072b48fd01ea037770ede8a928896fbb536949 on nazaninreihani:fix-translate into 54a1f57f54b9faed2c9babff3734f1331b0200e4 on binary-com:dev.

@nazaninreihani I figured out that we are not supposed to change the translation files under _constants/po/*.js
If we need new translation, we have to checkout translation branch and add all our requirements into *.po and *.pot files. Once you add those translation requirement in those files, weblate system will translate them and we have to run sh fetch-translation.sh command in dev branch to generate the new .json and .js file out of those translated files.
Please do not worry about the changes you submitted in those .js files. They will get overidden by the command mentioned above. For now, could you please add those empty translation strings in those .po files and then submit the PR to translation branch? I will take care of all other tasks from there.

sure I will do this


Change to notification hubs v1.0.9

Fixes #397
Switched from Microsoft.Azure.NotificationHubs v<IP_ADDRESS> to v1.0.9 after input from Notification Hubs product group. This is in the issue #397.
There are changes in this that come from the two previous PRs I just created.

:x: Build ServiceBusExplorer 4.2.0-ChangeToNotificationHubs109.67.build.699 failed (commit https://github.com/paolosalvatori/ServiceBusExplorer/commit/d62ffc49e9 by @ErikMogensen)

:white_check_mark: Build ServiceBusExplorer 4.2.0-ChangeToNotificationHubs109.68.build.700 completed (commit https://github.com/paolosalvatori/ServiceBusExplorer/commit/a438b22148 by @ErikMogensen)


how to get database-encryption-key

how to get database-encryption-key

You should come up with your own secret string. It will be your secret. See also #23


Update read.response.html

Had a complaint that "id" is ambiguous and need to clarify that this is an ODS code, not an ASID.

Change to reflect that the "id" is ambiguous and need to clarify that this is an ODS code.


Question:it automatically resigns me

Why does it automatically resign me when i dont play one of the moves it suggests.

actually it automatically resigns me after 7 moves is played in a game

Is it on chess.com or lichess?

Lichess

Thanks. I will see.

No problem


chore: bdd-spring-1585887717 to 0.0.1

chore: Promote bdd-spring-1585887717 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Improving UI of Form Creation Page of Web-App

The files are rcreation.js and its respective stylesheet in styles folder

on it !!


Could not gem install github-linguist on Mac OS Catalina

The issue
I tried to install linguist by executing gem install github-linguist which failed with  fatal error: 'ruby/config.h' file not found #include "ruby/config.h"  in the error log.
Project repo
Ruby version: 2.6.3p62
maja@Majas-MacBook-Pro Photography_portfolio_V2 % gem install github-linguist
Fetching github-linguist-7.12.2.gem
Fetching escape_utils-1.2.1.gem
Fetching mini_mime-1.0.2.gem
Fetching rugged-1.1.0.gem
Fetching charlock_holmes-0.7.7.gem
ERROR:  While executing gem ... (Gem::FilePermissionError)
    You don't have write permissions for the /Library/Ruby/Gems/2.6.0 directory.
maja@Majas-MacBook-Pro Photography_portfolio_V2 % sudo gem install github-linguist
Password:
Fetching mini_mime-1.0.2.gem
Fetching github-linguist-7.12.2.gem
Fetching charlock_holmes-0.7.7.gem
Fetching escape_utils-1.2.1.gem
Fetching rugged-1.1.0.gem
Building native extensions. This could take a while...
ERROR:  Error installing github-linguist:
        ERROR: Failed to build gem native extension.

    current directory: /Library/Ruby/Gems/2.6.0/gems/charlock_holmes-0.7.7/ext/charlock_holmes
/System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/bin/ruby -I /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0 -r ./siteconf20210207-85253-95znya.rb extconf.rb
checking for -licui18n... *** extconf.rb failed ***
Could not create Makefile due to some reason, probably lack of necessary
libraries and/or headers.  Check the mkmf.log file for more details.  You may
need configuration options.

Provided configuration options:
        --with-opt-dir
        --without-opt-dir
        --with-opt-include
        --without-opt-include=${opt-dir}/include
        --with-opt-lib
        --without-opt-lib=${opt-dir}/lib
        --with-make-prog
        --without-make-prog
        --srcdir=.
        --curdir
        --ruby=/System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/bin/$(RUBY_BASE_NAME)
        --with-icu-dir
        --without-icu-dir
        --with-icu-include
        --without-icu-include=${icu-dir}/include
        --with-icu-lib
        --without-icu-lib=${icu-dir}/lib
        --with-icu-i18n-config
        --without-icu-i18n-config
        --with-pkg-config
        --without-pkg-config
        --with-icu-io-config
        --without-icu-io-config
        --with-pkg-config
        --without-pkg-config
        --with-icu-uc-config
        --without-icu-uc-config
        --with-pkg-config
        --without-pkg-config
        --with-icui18nlib
        --without-icui18nlib
/System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/mkmf.rb:467:in `try_do': The compiler failed to generate an executable file. (RuntimeError)
You have to install development tools first.
        from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/mkmf.rb:546:in `block in try_link0'
        from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/tmpdir.rb:93:in `mktmpdir'
        from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/mkmf.rb:543:in `try_link0'
        from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/mkmf.rb:570:in `try_link'
        from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/mkmf.rb:789:in `try_func'
        from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/mkmf.rb:1016:in `block in have_library'
        from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/mkmf.rb:959:in `block in checking_for'
        from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/mkmf.rb:361:in `block (2 levels) in postpone'
        from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/mkmf.rb:331:in `open'
        from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/mkmf.rb:361:in `block in postpone'
        from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/mkmf.rb:331:in `open'
        from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/mkmf.rb:357:in `postpone'
        from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/mkmf.rb:958:in `checking_for'
        from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/mkmf.rb:1011:in `have_library'
        from extconf.rb:39:in `<main>'

To see why this extension failed to compile, please check the mkmf.log which can be found here:

  /Library/Ruby/Gems/2.6.0/extensions/universal-darwin-19/2.6.0/charlock_holmes-0.7.7/mkmf.log

extconf failed, exit code 1

Gem files will remain installed in /Library/Ruby/Gems/2.6.0/gems/charlock_holmes-0.7.7 for inspection.
Results logged to /Library/Ruby/Gems/2.6.0/extensions/universal-darwin-19/2.6.0/charlock_holmes-0.7.7/gem_make.out

Attempted solutions
I ran brew install cmake pkg-config icu4c and tried installing again with no luck.
From this issue I then ran
sudo gem install charlock_holmes -- --with-icu-dir=/usr/local/opt/icu4c --with-cxxflags='-Wno-reserved-user-defined-literal -std=c++11'  which failed with the same error, as well as
sudo gem install github-linguist --version "7.12.2" --source "https://rubygems.pkg.github.com/github"
I did notice the solutions that suggest  modifying certain lines but they are in the file I'm missing.

Unfortunately, this isn't a bug in Linguist, nor charlock-holmes (which is the gem that's struggling here) but rather a problem with the Ruby as shipped with macOS.
See my comments at https://github.com/github/linguist/pull/4918#issuecomment-679028679 and https://github.com/github/linguist/issues/5147#issuecomment-761051470
I'm in the process of updating the docs in which I'll be adding a note discouraging the use of the macOS Ruby because of this precise reason, and the fact that Apple has already announced they'll stop shipping Ruby so really have no incentive to fix this or any other issues.


bug: bootstrapper mode refuses incoming connections

This PR fixes the issue of Bootstrapped nodes not able to accept incoming connections by always keeping one spot available by disconnected the longest lived connection.
It addresses issue #287

Hi @elielnfinic, thanks for the PR.
It looks good: it's addressing the problem with a good fix. There is one thing I'm confused about though: why doesn't the error message match the one mentioned in the issue? For reference, the suggested change works around this bail! statement:
            bail!("Attempted to connect to more peers than allowed. Aborting connection.");

whereas the observed error message was "neptune_cash::main_loop: Got error: Refusing incoming connection. Reason: MaxPeerNumberExceeded".
Can you shed light on this discrepancy?
If not, maybe you fixed another, independent way to trigger the same error :D

Hello @aszepieniec, thanks for pointing that problem. I might have not understood the issue as it should be sorted out. My guess was to remove the longest lived peer before MAX is reached.
I think another issue is on replicating the error. I am not able to replicate that issue. Could you please share how to replicate the error? I was trying spoofing the node with multiple connections but I might be wrong. As I could not replicate the issue, I went ahead and examined the code to say where I can insert the verification if the node is a bootstrap node.
Here is my script to try to connect multiple nodes to boostrap for them to discover one another :
#!/bin/bash

BASE_DIR="$MY_PROJECT_PATH/neptune-core"
BOOTSTRAP_PEER="<IP_ADDRESS>:9797"
NODE_COUNT=30

BASE_PEER_PORT=8800
BASE_RPC_PORT=8900

launch_node() {
    NODE_ID=$1
    DATA_DIR="$BASE_DIR/data/node_$NODE_ID"
    PEER_PORT=$((BASE_PEER_PORT + NODE_ID))
    RPC_PORT=$((BASE_RPC_PORT + NODE_ID))

    echo "Launching node $NODE_ID..."
    mkdir -p "$DATA_DIR"

    cargo run -- \
        --data-dir "$DATA_DIR" \
        --peer-port "$PEER_PORT" \
        --rpc-port "$RPC_PORT" \
        --peers "$BOOTSTRAP_PEER" \
        --max-num-peers 30  > "$DATA_DIR/node_$NODE_ID.log" 2>&1 &

    echo "Node $NODE_ID running with peer port $PEER_PORT and RPC port $RPC_PORT."
}

echo "Starting $NODE_COUNT nodes to connect to bootstrap peer $BOOTSTRAP_PEER..."

cd "$BASE_DIR" || { echo "Error: Directory $BASE_DIR does not exist!"; exit 1; }

for i in $(seq 1 $NODE_COUNT); do
    launch_node $i
    sleep 0.5
done

echo "All nodes launched. Check logs in individual data directories under $BASE_DIR/data."

Ah, I see what you mean now. I wasn’t aware that answer_peer_inner also included a check, which could indeed result in two peers being disconnected. I’m currently reviewing the process to better understand the test changes you’re suggesting.

Per the the sequence you provided, I think I was able to fix the issue. By the way, my script had an error on the port which had to be set to the value 9798 but was 9797 and that's why I wasn't able to replicate the issue. After setting it to 9798, I could replicate the issue. With the new commit, the initial issue appears to be fixed, please check it out.
Thank you.

I increased the sleep time in my script to 1 sec and this issue disappears so I believe its a concurrency issue.

This information is invaluable. Based on this analysis, the following scenario seems plausible / likely:

Two incoming connection requests are received simultaneously.
They are processed by different tasks, so they do not know about each other.
They both observe that the new connection would set the peer count to max. So they both generate a AcceptedMaxReached status code followed by a DisconnectFromLongestLivedPeer message sent to the main loop.
Before the first DisconnectFromLongestLivedPeer message is processed, the first peer is added to the peer map, setting the number of peers to max.
Before the first DisconnectFromLongestLivedPeer message is processed, the second connection handler attempts to add the second peer to the peer map, but as the max has already been reached this attempt fails and this failure results in the observed error message.

The root of the problem, as I see it, is that the peer map is read and then later modified within one thread/task, but another thread/task modifies it in the mean time.
Brainstorming for solutions:

Hold a write lock on the peer map for the entire peer loop setup. This prevents the second incoming request handler from starting until the first one is finished. The downside would be holding the write lock longer, preventing all other threads/tasks from progressing.
Stall before entering into the peer loop for however long it takes the main loop to process the DisconnectFromLongestLivedPeer message. This would require extra communication with extra .await steps.
Plow ahead, temporarily exceeding the max number of peers.

I invite @Sword-Smith and @dan-da to comment; both have much more experience than I do when it comes to concurrency.

I invite @Sword-Smith and @dan-da to comment; both have much more experience than I do when it comes to concurrency.

I haven't been following this issue much, so just going from the analysis in https://github.com/Neptune-Crypto/neptune-core/pull/291#issuecomment-2558422487...

Brainstorming for solutions:

Hold a write lock on the peer map for the entire peer loop setup. This prevents the second incoming request handler from starting until the first one is finished. The downside would be holding the write lock longer, preventing all other threads/tasks from progressing.

bad for concurrency, my least favorite.  holding write-lock for longer (anywhere) is going the wrong direction.

Plow ahead, temporarily exceeding the max number of peers.

this seems best for concurrency.  I guess we could possibly exceed max by the number of simultaneous processing tasks.  So long as that doesn't pose a problem...


Cursor doesn't move between text and adornments with paragraph editor

https://user-images.githubusercontent.com/15268625/195593830-3529ab4d-c639-467f-9e41-8bcc5983d540.mov

text := 'Hello World' asRopedText
		attributes: {BrGlamorousTriangleExpanderTextAttribute new}
		from: 5
		to: 5.

editor := BrEditor new
		aptitude: BrEditorAptitude;
		text: text.

editor navigator addAt: 5; apply.
editor


crash report

Hi thanks for the good plugin.
Is it possible to setup the logger to catch all unhandled exceptions and to send these ones as a crash report?
Nicky

If you're using 2.6 it should automatically catch unhandled ones...
tracker.enableExceptionReporting(true);

Not sure if this is what you mean?

Thanks and it is also possible to listen to the application uncaughtErrorEvent
application.on(application.uncaughtErrorEvent, function (args) {
	//do something with the tracker
	tracker....
});


HA 2024.6.0: multiple warnings

HA 2024.6.0
Bosch component 0.25.1
Getting the following warnings:

Logger: homeassistant.util.loop
Source: util/loop.py:84
First occurred: 12:48:37 (5 occurrences)
Last logged: 12:48:52

Detected blocking call to open inside the event loop by custom integration 'bosch' at custom_components/bosch/__init__.py, line 299: await self.gateway.check_connection() (offender: /usr/local/lib/python3.12/site-packages/bosch_thermostat_client/db/__init__.py, line 35: with open(file, "r") as db_file:), please create a bug report at https://github.com/bosch-thermostat/home-assistant-bosch-custom-component/issues Traceback (most recent call last): File "<frozen runpy>", line 198, in _run_module_as_main File "<frozen runpy>", line 88, in _run_code File "/usr/src/homeassistant/homeassistant/__main__.py", line 223, in <module> sys.exit(main()) File "/usr/src/homeassistant/homeassistant/__main__.py", line 209, in main exit_code = runner.run(runtime_conf) File "/usr/src/homeassistant/homeassistant/runner.py", line 190, in run return loop.run_until_complete(setup_and_run_hass(runtime_config)) File "/usr/local/lib/python3.12/asyncio/base_events.py", line 672, in run_until_complete self.run_forever() File "/usr/local/lib/python3.12/asyncio/base_events.py", line 639, in run_forever self._run_once() File "/usr/local/lib/python3.12/asyncio/base_events.py", line 1988, in _run_once handle._run() File "/usr/local/lib/python3.12/asyncio/events.py", line 88, in _run self._context.run(self._callback, *self._args) File "/usr/src/homeassistant/homeassistant/setup.py", line 165, in async_setup_component result = await _async_setup_component(hass, domain, config) File "/usr/src/homeassistant/homeassistant/setup.py", line 447, in _async_setup_component await asyncio.gather( File "/usr/src/homeassistant/homeassistant/setup.py", line 449, in <genexpr> create_eager_task( File "/usr/src/homeassistant/homeassistant/util/async_.py", line 37, in create_eager_task return Task(coro, loop=loop, name=name, eager_start=True) File "/usr/src/homeassistant/homeassistant/config_entries.py", line 742, in async_setup_locked await self.async_setup(hass, integration=integration) File "/usr/src/homeassistant/homeassistant/config_entries.py", line 594, in async_setup result = await component.async_setup_entry(hass, self) File "/config/custom_components/bosch/__init__.py", line 148, in async_setup_entry _init_status: bool = await gateway_entry.async_init() File "/config/custom_components/bosch/__init__.py", line 245, in async_init if await self.async_init_bosch(): File "/config/custom_components/bosch/__init__.py", line 299, in async_init_bosch await self.gateway.check_connection()

\
Detected blocking call to open inside the event loop by custom integration 'bosch' at custom_components/bosch/__init__.py, line 299: await self.gateway.check_connection() (offender: /usr/local/lib/python3.12/site-packages/bosch_thermostat_client/db/__init__.py, line 35: with open(file, "r") as db_file:), please create a bug report at https://github.com/bosch-thermostat/home-assistant-bosch-custom-component/issues Traceback (most recent call last): File "<frozen runpy>", line 198, in _run_module_as_main File "<frozen runpy>", line 88, in _run_code File "/usr/src/homeassistant/homeassistant/__main__.py", line 223, in <module> sys.exit(main()) File "/usr/src/homeassistant/homeassistant/__main__.py", line 209, in main exit_code = runner.run(runtime_conf) File "/usr/src/homeassistant/homeassistant/runner.py", line 190, in run return loop.run_until_complete(setup_and_run_hass(runtime_config)) File "/usr/local/lib/python3.12/asyncio/base_events.py", line 672, in run_until_complete self.run_forever() File "/usr/local/lib/python3.12/asyncio/base_events.py", line 639, in run_forever self._run_once() File "/usr/local/lib/python3.12/asyncio/base_events.py", line 1988, in _run_once handle._run() File "/usr/local/lib/python3.12/asyncio/events.py", line 88, in _run self._context.run(self._callback, *self._args) File "/usr/src/homeassistant/homeassistant/config_entries.py", line 742, in async_setup_locked await self.async_setup(hass, integration=integration) File "/usr/src/homeassistant/homeassistant/config_entries.py", line 594, in async_setup result = await component.async_setup_entry(hass, self) File "/config/custom_components/bosch/__init__.py", line 148, in async_setup_entry _init_status: bool = await gateway_entry.async_init() File "/config/custom_components/bosch/__init__.py", line 245, in async_init if await self.async_init_bosch(): File "/config/custom_components/bosch/__init__.py", line 299, in async_init_bosch await self.gateway.check_connection()

Detected blocking call to open inside the event loop by custom integration 'bosch' at custom_components/bosch/__init__.py, line 325: supported_bosch = await self.gateway.get_capabilities() (offender: /usr/local/lib/python3.12/site-packages/bosch_thermostat_client/db/__init__.py, line 35: with open(file, "r") as db_file:), please create a bug report at https://github.com/bosch-thermostat/home-assistant-bosch-custom-component/issues Traceback (most recent call last): File "<frozen runpy>", line 198, in _run_module_as_main File "<frozen runpy>", line 88, in _run_code File "/usr/src/homeassistant/homeassistant/__main__.py", line 223, in <module> sys.exit(main()) File "/usr/src/homeassistant/homeassistant/__main__.py", line 209, in main exit_code = runner.run(runtime_conf) File "/usr/src/homeassistant/homeassistant/runner.py", line 190, in run return loop.run_until_complete(setup_and_run_hass(runtime_config)) File "/usr/local/lib/python3.12/asyncio/base_events.py", line 672, in run_until_complete self.run_forever() File "/usr/local/lib/python3.12/asyncio/base_events.py", line 639, in run_forever self._run_once() File "/usr/local/lib/python3.12/asyncio/base_events.py", line 1988, in _run_once handle._run() File "/usr/local/lib/python3.12/asyncio/events.py", line 88, in _run self._context.run(self._callback, *self._args) File "/usr/src/homeassistant/homeassistant/config_entries.py", line 742, in async_setup_locked await self.async_setup(hass, integration=integration) File "/usr/src/homeassistant/homeassistant/config_entries.py", line 594, in async_setup result = await component.async_setup_entry(hass, self) File "/config/custom_components/bosch/__init__.py", line 148, in async_setup_entry _init_status: bool = await gateway_entry.async_init() File "/config/custom_components/bosch/__init__.py", line 245, in async_init if await self.async_init_bosch(): File "/config/custom_components/bosch/__init__.py", line 325, in async_init_bosch supported_bosch = await self.gateway.get_capabilities()

I started looking into this in #425 and https://github.com/bosch-thermostat/bosch-thermostat-client-python/pull/43, feel free to try it out if you want to. I barely know what I'm doing.. 😁

That gives a lot of hope 😂😂😂😉😉😉
I'm not that technical, so how can I try?

I think you need to know a bit about Python and Home Assistant internals to test it right now. Basically, you need to find both the files from the bosch-thermostat-client Python library and from the custom integration, and apply the changes to them, and restart Home Assistant after that. There's also a small risk that you need other changes that have been made to the custom integration since the last release.
If/when the changes are merged you can manually install the latest development version via HACS in your Home Assistant.

Perhaps it will be better to just wait until the next release 😉

@haaino 0.26.1 is now available via HACS.


[Holder App] Last row is cut on the Device ID Claims Screen

same issue as in other views / apps

fixed


openstack: update CentOS 6 cloud image

Signed-off-by: Loic Dachary<EMAIL_ADDRESS>

Refer to this link for build results (access rights to CI server needed):
http://jenkins.ceph.com//job/teuthology-pull-requests/1883/
Test PASSed.


Improve Sorting of Events

Is your feature request related to a problem? Please describe.
the current sorting system for events could cause long-term (ongoing) events to appear prominently at the top of the list in the future. This could reduce the visibility of single (one-time) events, which may be less noticeable to users. As a result, users may miss out on important individual events, leading to a less optimal user experience
Describe the solution you'd like
An improved sorting mechanism where single (one-time) events are prioritized and displayed at the top of the feed, followed by long-term (ongoing) events. This adjustment would ensure better visibility for time-specific events, making it easier for users to discover and engage with them
Describe alternatives you've considered
An alternative solution could involve visually differentiating long-term events with distinct labels, making them easier to distinguish. However, this approach might not fully resolve the issue of reduced visibility for single events
Additional context
The implementation is dependent on the backend release of the option to categorize events in single events and ongoing events (📆 #3070)

An improved sorting mechanism where single (one-time) events are prioritized and displayed at the top of the feed, followed by long-term (ongoing) events. This adjustment would ensure better visibility for time-specific events, making it easier for users to discover and engage with them

@hauf-toni do you mean by this that all single events are shown first? And all long terms are shown below?
Should we add some visual separation and/or explanation why there are two different list (as it won't follow chronological sorting anymore)?

I would suggest we discuss this together with @osmers in our upcoming meeting next week @steffenkleinle


5 problems with capybara (Production mode, Mega menu, Wishlist, Filtration, Refresh product page)

Hello dear developers. Thanks for your great digital products. I really like that you make the world a better place, like your ideas. But please take me out of the dense forest, show me the way:
I have just 4 questions. I'll start with the most important.

1.) The most important thing that concerns me today is how to switch to the production mode in the Capybara theme, for example, in the default theme, I just start yarn start in the VSF folder, at that time the dispatcher (PM2) starts.
And in Capybara I know only one lerna command bootstrap && yarn dev. The question is if I now edit the Capybara theme for my needs, how visitors to my site in vue storefront will interact with the Capybara theme if the Capybara only works in development mode. What is the point of this topic then?
Below I show two pictures first after the yarn start command, where default theme starts. And the second picture is capybara. So, in the case of the theme "Capybara" the application does not start.

2.) Where did the child menu items disappear in the Capybara theme.That's what I meant

At the same time, I noticed that you have this correct extended menu for magento 2. It is only present on your demo site, but it is not in the repository.

3.) I think it would be advisable to place the wishlist button on the product page in Capybara, since this is an e-commerce format, especially since it is in your repository, but it’s turned off, I tried to turn it on but it didn’t work
Here's what the work button looks like in the default theme.

4.) I also noticed that the product is not filtered by price, the filter displays "nothing was found". This problem is observed in two topics, the default and Capybara. I am sure that you know about this problem, but it would be nice if you suggested how to solve it
There are several issues that still prevent me from releasing my site on Vue Storefront. I like some things in the default theme, such as: “wish list button”, “add to comparison button”, “full menu from magento 2”.
Filtered by price, only on your demo sites, but not from the repositories https://github.com/DivanteLtd/vsf-capybara and https://github.com/DivanteLtd/vsf-default

5.) When I refresh the page in the product card of my site (Ctrl + R), I get redirected to the error page (I get an error on the page). But when I do Ctrl + R on your product site in the product card, it works correctly, that is, I continue to remain on the page. For example, on this page everything works, but it doesn’t work on my site
https://capybara.storefrontcloud.io/women/tops-women/tees-women/tees-25/gabrielle-micro-sleeve-top-1417.html?childSku=WS02-XS-Blue

How to beat this?
But I do not see these things in the Capybara theme, even in version v1.11.2. I understand that the atomic approach in Capybara is a great solution. but lack of buttons: "wish list", "add to comparison" and limited menu. How to solve these problems in Capybara?
Thanks again for what you are doing. We will be very happy if you help us / tell us
I would really appreciate if there were tips at the copy / paste level since I can't write code

lerna bootstrap && yarn dev triggers yarn dev in vsf directory. https://github.com/DivanteLtd/vsf-capybara/blob/develop/package.json#L9 Thats because capybara can't work without vsf. If you want to have capybara in production mode then just use same building process as for vsf. yarn build && yarn start from vsf directory

1.) What you mean. Can you specify the commands sequentially, since if you mean first, from vsf directory I run
yarn build && yarn start
then
lerna bootstrap && yarn dev
A button appears in the address bar it really works and I get the "Install app?". Then, after I install it on the desktop, then I edited the capybara theme and no longer need the developer mode. I exit it developer mode, after this application stops working.

From vsf directory:

yarn build => creates dist where are your app files
yarn start => starts pm2 process which serves files from dist folder
yarn dev => starts app in dev mode (webpack)

From capybara theme directory there is only:

lerna bootstrap && yarn dev => which starts yarn dev from vsf directory :point_up:

2,3,4,5 => it's not possible to help you if there is no error message or at least some information what did you changed in vsf or in capybara theme :)
2 => did you applied this https://github.com/DivanteLtd/vsf-capybara/blob/develop/local.json config changes into your local.json file?

Thank you for the ability to help, today I will check everything.

I have the same issue, Can any one help ??

Thank you for the ability to help, today I will check everything.

Did you over come the issues ?


Copy edits to /about-us/

Doesn't need to be merged this sprint.
Changes

Made copy edits to /about-us/ per @sarahsimpson09's email

Testing

Visit localhost:7000/about-us/

:+1:

The removal of the link to the org chart under Bureau Leadership was an oversight, there should be a link under Cordray and Antonakes/Fuchs that should say "See how we're organized"
Thanks for catching that, @KimberlyMunoz!

@sarahsimpson09

:+1:


Add endIcon textfield

Is it possible to add endIcon?

@wuilmerj24 yes it is possible but it has to be implemented. PR are welcome


pack not referenced in any index

When running restic check I get multiple errors of "pack xxx not referenced in any index"
So I ran a prune as suggested elsewhere...get multiple "unable to list pack xxx header is larger than file"
So I ran a rebuild-index and also get the header is larger than file errors...
Is there any way to fix this or is the repo hopelessly corrupted?

Sadly, you did not fill out the issue template :cry:
restic prune takes care of these files. When it is finished, the files have been removed.

Sorry about the template thingy, but as noted in my description, restic prune does NOT take care of this.
I ran prune multiple times and the error persists.  Also ran restic rebuild-index several times.  Still get errors running restic check.
Running restic 0.5.0 on ubuntu 14
From: Alexander Neumann<EMAIL_ADDRESS>Sent: September-07-18 1:53 AM
To: restic/restic
Cc: durcom; Author
Subject: Re: [restic/restic] pack not referenced in any index (#1986)
Sadly, you did not fill out the issue template 😢
restic prune takes care of these files. When it is finished, the files have been removed.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/restic/restic/issues/1986#issuecomment-419330854 , or mute the thread https://github.com/notifications/unsubscribe-auth/AbxymqQ9ibLjWtU0eC9660KppbHWH3cEks5uYgm9gaJpZM4WdX4b .  https://github.com/notifications/beacon/AbxymrezIL4P4emGHuJAS-kYhJxt3ru8ks5uYgm9gaJpZM4WdX4b.gif

Just realized I’m running very old version of restic.  Will try latest version and hopefully it works better.
From: Alexander Neumann<EMAIL_ADDRESS>Sent: September-07-18 1:53 AM
To: restic/restic
Cc: durcom; Author
Subject: Re: [restic/restic] pack not referenced in any index (#1986)
Closed #1986 https://github.com/restic/restic/issues/1986 .
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/restic/restic/issues/1986#event-1832045120 , or mute the thread https://github.com/notifications/unsubscribe-auth/Abxymra-4kehmIEg8j-MWSC6i2BLc4V_ks5uYgm_gaJpZM4WdX4b .  https://github.com/notifications/beacon/AbxymlkgFOrRlSQeAqa0XFKGLpVT4IR1ks5uYgm_gaJpZM4WdX4b.gif

hello,maybe it doesnot work better;
restice 0.12.0 centos 7.5
pack fe0cb3dd: not referenced in any index
pack 4389dc1c: not referenced in any index
pack de6d2ff1: not referenced in any index
316 additional files were found in the repo, which likely contain duplicate data.
You can run `restic prune` to correct this.
check snapshots, trees and blobs
no errors were found
[0:00] 100.00%  40 / 40 snapshots

i encounter this problem frequently；i dont know what lead to this problem i just backup a file.
i have no idea how to reslove it.
hi,bro,do you have some idea? or you already gave up.

@saberya There is no error in the output you show above. The fact that there are paths "not referenced in any index" is not a problem, your repository's integrity is healthy according to the "no errors were found". You can however add --read-data to your check command to make sure that the actual data is intact as well, in case of bit rot, etc.
If you are having another problem, please open a new issue about it, and make sure to fill out the issue template when asked to (including the actual commands you run, and all of their output, where applicable).

OK,thanks for your advice.i'll open a new issue.
i have looked up many issues,and I found this problem a couple of times in some issues.
so i just want to confirm if this problem was resloved.
but i'd better to discuss it in a new issue.
thank you @rawtaz


Make 'Name' field public

The Problem
In the Action interface, both attached functions receive an Actor and a Rule. While this is useful for things like contextual logging, there's no way to extract the actor name since it's not publicly exported.
The Solution
In this PR, the Actor.name field is renamed to Name. That way it becomes gettable for structs that implement Action interface.
If we really want to keep the name private, we can instead add a GetName getter function to the struct that just returns a copy of the name. However, that seems like overkill.
Related Issues
Closes #6

Hi @terrabitz, thanks for this! It was super thoughtful. But this opens up BadActor's API to race conditions. All of an Actor's functions are private because it expects the instance of the Director to manage all Actors concurrently and correctly. The Director uses mutexes to facilitate this.
I think the best way  at the moment, to get the private name value from an Actor, is to add a public function to an Actor called Name(). Every Actor stores a pointer to its Director so I believe a public function must be added to the Director called ActorsName(a *Actor) which accepts the pointer of the Actor. This function will lock the list that the Director manages and return the value of the Actor's name safely.
Functionally this would manifest itself using the interfaces.
func (ma *MyAction) WhenJailed(a *badactor.Actor, r *badactor.Rule) error {
  go func() {
    sendNameToLogServerUsingHTTP(a.Name())
  }()
  return nil
}

func (ma *MyAction) WhenTimeServed(a *badactor.Actor, r *badactor.Rule) error {
  go func() {
    sendNameToLogServerUsingHTTP(a.Name())
  }()
  return nil
}

To reiterate.

The above examples would call out to the Director.
The Director would lock the list of Actors.
And you could make a multi-threaded function call, without concern of a race condition, that would copy the Actor.name returning a value.

Does that make sense?
If you'd like to take a crack at this, we can collaborate.
Please let me know.

Hey @jaredfolkins, I'd definitely be happy to take a look at implementing this myself! However, I'm a little confused by the description. Is the race condition we're trying to avoid from something else updating the actor name at the same moment I'm trying to retrieve it? If so, I can definitely understand putting a lock on the Actor list first. However, I was under the impression that name was an unchanging identifier for a particular actor, in which case it seems like we could simply use the actor's name property without the worry of it changing out from under us.

Hey @terrabitz, yeah the public API has always been exposed using the Director. You can only set the name on the creation of the Actor. I hesitate changing that part of the API and allowing Name to be set publicly. I've just suffered those types of design decisions over the years.
By calling out to the Director we also remove the edge case where the expired TTY of an Actor forces the Director to delete it from the cache/list and potentially making another go routine call a nil pointer. The order of operation is still maintained.

Accidental close.

@jaredfolkins Just reverted my previous commit and updated the PR to lock the director before returning the name field.

Solved. Closing.
https://github.com/jaredfolkins/badactor/commit/1d1f4105ddd60d654b006be03839da20c7933e6d


Generating unique pages to each of the listings according to a template?

Hi again,
Would you have a suggestion on how to generate unique pages to each of the listings (saas products in your case) according to a template?
(i.e. not linking out to the different SaaS products (in your case), but to unique pages in saasforcovid.com that would offer more details about each saas product.)

That would be great for the website. Not sure how to get all the content right, you have an idea?

No idea at all right now.
Just really from top of head:
Guess a template and a static site generator would be needed.
i.e.

When using static site generators (like Jekyll), there would be layout templates, and styling files and each page then would be coded with markdown.

Then when Jekyll would do its thing, it would generate the the final html files for each of the "listings" (saas products in the case of saasforcovid).

The step to write the different markdown files for each of the "listings" should be automated so one does not need to code each page in markdown.

Having said that, there should be a way of doing this more easily without bringing a parsing engine to complicate things further here :)
Should any other idea come to you or you end up adding pages to saasforcovid, do let me know :)


is it possible to integrate sdk with react native?

There are some libraries, but they are deprecated, I implemented them directly in the Android project using Native Modules, it works great.

could you share how you did?

using expo to development ?


Localization

Detailed Description

To provide users options to switch display language, or even better, display language becomes a configurable option in their user profile.
Context

In Asia (China, Taiwan, Japan, Korea, for example), operators prefer to use localized UI and read localized documentation. Localization can gain wider adoption in this market as most operators do have limited English proficiency.
Possible Implementation

Linking upstream issue - https://github.com/cloudfoundry-incubator/stratos/issues/1534


How to debug a program which has some paramters with delve?

Please answer the following before submitting your issue:
Note: Please include any substantial examples (debug session output,
stacktraces, etc) as linked gists.

What version of Delve are you using (dlv version)?
Version: 0.12.1
What version of Go are you using? (go version)?
go version go1.6.2 linux/amd64
What operating system and processor architecture are you using?
Ubuntu 16.04.1 LTS \n \l
What did you do?
I just want to debug a program which has some parameters with delve. The command is like that
./foo --bzzaccount $BZZKEY --datadir $DATADIR --bzznetworkid 322
I debug it like dlv exec ./foo--build-flags="--bzzaccount $BZZKEY --datadir $DATADIR --bzznetworkid 322"
but it does not work. So would you please tell me what shouid I do? thank you.
What did you expect to see?
What did you see instead?

dlv exec ./foo -- --bzzaccount $BZZKEY --datadir $DATADIR --bzznetworkid 322


fix: remove dependencies required for the library

Splits the crate in two: a binary and a library. This reduces the number of packages required for the library because we can move those to the binary crate.

Clippy errors are fixed in #14


[go_router] Impossible to specify custom format for debug logs

Since go_router writes debug logs using developer.log (https://github.com/flutter/packages/blob/main/packages/go_router/lib/src/logging.dart#L37), it's impossible to override the logged message formatting, which may lead to ugly console log messages whenever go_router writes its logs.
For example, we have debug logging set up in a way, that each log message will be prepended with a constant-width section. Enabling the debugLogDiagnostics on go_router, though, breaks all our formatting, like here:

It would've been great if there was an option to override the format of messages written to the debug console by go_router.

Thanks for the report. Treating this as a proposal to provide an option to override format of messges.
/cc @chunhtai @johnpryan

In general I prefer the event emitter pattern instead of directly logging, letting the user choose however he was to print event.

superseded https://github.com/flutter/flutter/issues/109263


[Bug]: CRLF not respected in snaps

Description
I'm writing a formatter and a testing suite of files alongside it that also needs to validate line endings (LF, CRLF) of string outputs or file reads. From my understanding, I think the snaps are only written in LF which leads to my snaps not matching my bytes output as they sometimes use CRLF.
I understand this is a bit of a complicated messy edge case, so I'm fine with no solution. Just wanted to let you know that line endings are inconsistently handled which does break tests. My only workaround is to disable CRLF testing in the meantime.
Steps to Reproduce
Read any CRLF file into a snap and it will replace all line endings into LF. Read the CRLF file again to match with the snap and it will fail. Note my snaps actually mix both LF and CRLF in the same file so that might be another complicated factor to handle.
Expected Behavior
No response

hmm 🤔  I think this is a "known" issue. Not sure if there is a way in order to handle this. Do you happen to know how jest behaves in the above scenarios?

Not particularly, but to be fair this issue is pretty dependent on the machine you are using or Git config you have set. e.g. git config --global core.eol lf will also automatically convert your checked in content as LF.
I'm not really sure either how to approach this. One solution could be to strip CRLF on incoming data since it will never match snaps anyways, but that sort of invisible behaviour is pretty bad when you have bugs that might depend on line ending behaviour. Perhaps just clearer documentation so developers know about the limitation is the best approach.

That makes sense, will update the documentation for this case.

Took me sometime 😅  Sorry and thank you for opening this issue.


Question : Printing to plotter T7200

Hello,
I am trying to print to plotter T7200 from CUPs using ipp library : https://github.com/williamkapke/ipp. This plotter does not have postscript upgrade kit, hence doesn't support postscript language.
Supported languages include : HP-GL/2, HP-RTL, CALS G4. Is there any way which can work as a workaround to print to this plotter. Any driver or change in ipp attributes  ?
Thank you in advance.
Regards,
Sonia

Hey @michaelrsweet , I am not sure If I understand what changes I will have to do to support this model.
Right now it looks like I will have to add this printer to CUPS with an external PPD file.
In this case will i also need rastertopclx driver ? Maybe you could point me to some documentation where I can first read about these. ?
Thank you very much.
Regards,
Sonia

Hello Sonia,
when you are on linux install the cups-filter package. I think this is always installed on Linux, when your are doing something with printing.
Configure the HP DesignJet T1100 pcl printer model. This should do the need work.
regards
Franz

Hey @frazhome
Thankyou for providing information.
Is it possible for you to give me some pointers on what parameters i should be sending to the printer setup with HP Designjet T1100 pcl model which will use cups-filter package ?
I am assuming some settings has to be made also ? Otherwise T7200 plotter will error out saying - PCL Language not supported.
Regards,
Sonia

Hello Sonia,
does the plotter support pclx? Is Graphics language setting on automatic?
I take a look on our company plotters via snmp and the all show
MFG:Hewlett-Packard;CMD:PJL,DW-PCL,POSTSCRIPT,HP-GL/2,RTL,JPEG,TIFF,PDF,CALSRASTER,ASCII
regards
Franz

Hey @frazhome ,
Standard language supported are : HP-GL/2, HP-RTL, CALS G4.
Graphic language is set to HP-GL/2. Do you know any way to print after setting graphic language to automatic ? I just tried with a postscript driver and plotter gave error that the format is not supported because it doesnt have Postscript upgrade kit installed, so i am looking for alternatives.

regards,
Sonia

please set to automatic

hp rtl is a flavour fo pclx

ok.. to make it clear

I should set Graphics language to automatic.
Add printer to cups with HP Designjet T1100 pcl driver

Do I also need cups-filter package now ?

Its Ubuntu 16.04.
I will check the option you suggested. Basically the print request will have an image to be printed, with 'job-attributes-tag' - 'orientation-requested', 'copies','media'.
document-format is image/png.
I will set up cups with T1100 pcl driver and will send print after changing graphics language to Automatic.

Hi,
It worked fine. Thank you very much. closing this issue.
Regards,
Sonia


🦜just do a lint. the kind install invariably fails 🦜

Kind install is always failing for charts that require OCP features, or need PV/PVC's as two obvious examples.
This holds up the automatic releasing of minor chart changes.
We need a better way to test chart install.
Ideally this step just does what it says on the tin "linting"

This is fine with me - @ckavili what do you think?

Yeah until we decide where to run chart testing, only linting is fine 🙃


[NTR/SOX3]

Preferred gene-related syndrome label
SOX3-related X-linked pituitary hormone deficiency with or without intellectual developmental disorder
Synonyms
N/A
Parent term (use OLS, or your favorite ontology browser)
X-linked syndromic intellectual disability
Definition
This disease is caused by alterations to the SOX3 gene which leads to hypopituitarism with variable deficiency of hormones in the anterior lobe of the pituitary gland. In some cases there is also intellectual disability.
Definition source (Please give PubMed ID, if applicable, in format PMID:#######)
PMID: 25402377, PMID: 21289259, PMID: 15800844
Children terms (if applicable) Should any existing terms be re-classified as children underneath this new proposed term?
N/A
Attribution
This is a request from the ClinGen Syndromic Disorders GCEP.

Hi @elliebroeren, thanks for reporting this. We are focused on a mass obsoletion effort this month, and will look at this more closely in November or later. We appreciate your patience.


consensus: exit if wrong password is provided in configuration

Fix #1312 .

And we also need this for master branch.


Sequences refactorings and removed unused code (part of #3798) [WIP]

This PR includes several related fixes/refactorings around Sequences subsystem, in the importance order:

Make Sequences.withBaggage() and Sequences.withEffect() to consistently close the baggage and execute the effect after base sequence is closed. Before, it was so in accumulate() path but was the opposite in Yielder path. This in mentioned in #3563.
Fixed potential defect in SpecificSegmentQueryRunner, which catches MissingSegmentsException in toYielder() but doesn't catch exception in Yielder.next().  This fix develops the idea of #3617. Probably it could never be an issue in practice, because if the segment is missing, MissingSegmentsException should always be thrown from toYielder(). But I'm not sure.
Add Sequences.wrap() which allows to simplify and remove repetitive code from MetricsEmittingQueryRunner,  CPUTimeMetricQueryRunner and SpecificSegmentQueryRunner.
Review exception handling in Sequences subsystem
Remove unused classes from io.druid.java.util.common.guava package.
Try to create sequences via factory methods from Sequences class rather than constructing XxxSequence classes directory; reduce visibility of the latter somewhere.

Marking Discuss as confusion was evident on the dev sync about what this is actually fixing or improving in practice. @leventov could you please elaborate on whether this is fixing/improving actual observed issues, actual issues that were unobserved, or things that are not known to be actual issues?

@gianm

Negative effects vary from inconsistent CPUTime/Query time measurement (including/not "closing time" of the wrapped sequence) to memory leaks and who knows what effects, because some "sub-resources" are closed after "super-resources", i. e. the resources the former are "derived" from. Taking accumulate() or Yielder path is hardly trackable and undetermined in some sense, because applying e. g. Sequences.limit() on upper level turns accumulate() into Yielder iteration underneath (because it's a YieldingSequenceBase)

It's just breaks general Yielder implementation contract, when "the same thing" should be done in toYielder() and next(). It's hard to say if it is a bug or not right now, (if MissingSegmentsException is always thrown in toYielder(), it is not), but it could become a bug at any point in future.

At the moment it removes some repetitive code. I originally did this because I added more to those code as part of my project of increasing amount and precision of query metrics. And repeating 30 lines of code in three places felt much more dangerous and boilerplate than repeating 10 lines. (This change is not yet in the PR itself, because it's blocked by this PR).

This addresses #3563. Attempt to match try-with-resources safety in Sequences. Try-with-resources always catch Throwables and use addSuppressed() when needed. Current code catches either Exception or RuntimeException and logs exceptions which should better be suppressed. Suppression allows to see more actual exceptions in query response, for example. Generally it guarantees that two exceptions ("main" and suppressed) are logged at the same place, wherever it is done. Logging one exception and propagating the another may "unlink" those two in the unified log.

Just remove unused code (merging java-util, extendedset, bytebuffer-collections, etc. opens a lot of such opportunities)

Eases Sequences refactoring in future, and generally static factory methods are considered better than constructors, but really a minor thing (and it's only a couple of lines in this PR)

@fjy @gianm this is a part of #3798, let it be the justification for this PR

Other than those 3 comments, the rest looks good to me.

@gianm thanks, addressed comments.

@gianm I want also resolve the question with LimitedSequence and isDone() as part of this PR, so marked as WIP

ok @leventov. @cheddar do you have insight into this thread: https://github.com/druid-io/druid/pull/3693#discussion_r95703215, & if there's a good reason for why things are the way they are?

Gian, which things specifically?  Just reading the description of the PR it
all sounds legit (and I would agree that the yielder closing thing is a
bug).  Are there specific things you are concerned about?
On Wed, Jan 11, 2017 at 7:14 PM Gian Merlino<EMAIL_ADDRESS>wrote:

ok @leventov https://github.com/leventov. @cheddar
https://github.com/cheddar do you have insight into this thread: #3693
(comment)
https://github.com/druid-io/druid/pull/3693#discussion_r95703215, & if
there's a good reason for why things are the way they are?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/druid-io/druid/pull/3693#issuecomment-272064861, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AANOyr5STYgjeR-J3ocMVATUQLffPYz1ks5rRZqngaJpZM4Kx7lg
.

@cheddar Just wondering if there's a good reason that limited sequences aren't done when they finish, or if that's a bug. It seems like a bug but maybe there's a reason that they behave the way they do.

@gianm I'm not sure why you think it wouldn't be done.  I see Yielders.done being called, which should be done.  Also, remember that the call to close is the user of the Yielder's responsibility, so once the final Yielder is yielded, close() should get called and resources cleaned up.
That said, I also realize you might be talking abotu the isDone() call.  A cursory glance at that makes me wonder if it is implemented correctly.  I'm not quite sure.  It looks like at some point in time interruptYield and the other check were added on top of just "within bounds".  I'm not sure those belong in isDone.  That said, though, there's nothing like a unit test or two to show the current behavior and see if it lines up or doesn't line up with what we expect.  Fwiw, I would expect that the yielder returned on the next() call after the final element to be done().
Also, all of the "baggage" related stuff I would expect to be done in the close() method.  I'm not quite sure what the semantic difference is between withEffect and withBaggage, but I would question the semantics of anything that is not done in the close() method or on each iteration.

Ahhh, I think I see what's going on. I misunderstood the behavior that exists now. I thought that @leventov was saying that limited sequences don't act as done when they hit their limit. They do and that's good. What does happen is that if you do this:
Sequences.limit(
  Sequences.withEffect(baseSequence, runnable, exec),
  threshold
)

Then the "runnable" will not run if the "threshold" is reached before "baseSequence" is exhausted. This is good behavior for what we use withEffect for in production, which is populating the cache. We don't want the cache to populate with an un-exhausted result set.
So I think the behavior is fine as-is and just needs to be documented (since it's not obvious what is going on and why).

@gianm @cheddar the problem is #3849. I don't see how it could be easily fixed, so I'm not going to fix this in this PR. At least, this PR shouldn't make the things worse than they used to be. So I'm removing [WIP]

I think the "fix" to #3849 is to just rename "withEffect" to
"withEffectOnCompletion" or "withEffectOnExhaustion".
Sounds like that is by design (I'll admit, I also find it
counter-intuitive, but if those are the semantics, they are the semantics),
so it's the naming that is making it confusing.
--Eric
On Fri, Jan 13, 2017 at 2:48 PM Roman Leventov<EMAIL_ADDRESS>wrote:

@gianm https://github.com/gianm @cheddar https://github.com/cheddar
the problem is #3849 https://github.com/druid-io/druid/issues/3849. I
don't see how it could be easily fixed, so I'm not going to fix this in
this PR. At least, this PR shouldn't make the things worse than they used
to be. So I'm removing [WIP]
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/druid-io/druid/pull/3693#issuecomment-272569085, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AANOyrCwvKoXnALxkVDbeBqrPXUjOKzpks5rR_8mgaJpZM4Kx7lg
.

Could someone please review this? @cheddar @himanshug @jon-wei

This patch looks good to me.

LGTM, :+1:


boards: arm: nrf52840dongle_nrf52840: Fix Bluetooth USB CDC

Fixes an issue when building Bluetooth HCI USB applications whereby there would be a conflict with USB CDC.

@fabiobaltieri give this a try and see if it resolves your issue

It does, good stuff.


Multiverse doesn't know about world

Hi, I get this message when I'm trying to remove a world from my server:
14.01 21:20:30 [Server] INFO [Multiverse-Core] Multiverse does not know about 'Seasons' and it's not loaded by Bukkit.
But the world exists in the config file. The world is called Enchanted Seasons and it lists the alias as "Seasons".
Do I need to use the full name to remove it, and if so how do I do that when the name is two words with a space in between?

Going to assume this is solved - closing the ticket. If you're still having issues, please don't hesitate to make another ticket or for a faster response, you can also receive support on the Multiverse Discord.


How to. change default effects

Hi, I just tried to preset some effects.  But the preference remains on the effect "rainbow runner", although the alteration I made was saved in the json-config.  What do I have to do to change that? Another point: There is no possibilty/menue in the home-app to select any of these effects. So, where can I find a presslection of the effects in homekit?
Could someone help? Best regards

Hi,
the config is:
{
"accessory": "WLED",
"name": "WLED Garage 2",
"host": "192.168.xxx.xx",
"effects": [
"Solid",
"Spots"
],
"log": true
}
The version seems to be the newest v0.0.1
Best regards, Ron

Okay please update to the newest homebridge-simple-wled Plugin version (https://www.npmjs.com/package/homebridge-simple-wled) v.1.2.3.
Then you also have to change your config.json to something like:
https://github.com/jstrausd/homebridge-simple-wled#-adding-the-accessory-to-the-configjson
Do you also get any errors in the homebridge-console or in the logs?

Thanks for Your prompt help. I'll test that.

Now I had problems with the update. So I deinstalled and insralled again the latest version as You've recommended. But now WLED doesn't appear any,ore in homekit after restart of homebridge. The protocol tells me
"The dynamic platform WLED from the plugin homebridge-simple-wled seems to be configured multiple times in your config.json. This behaviour was deprecated in homebridge v1.0.0 and will be removed in v2.0.0!"
What can I do now?
My config is this:
    {
        "platform": "WLED",
        "name": "WLED Garage 1",
        "host": "<IP_ADDRESS>",
        "effects": [
            "Solid",
            "Spots"
        ]
    },
    {
        "platform": "WLED",
        "name": "WLED Garage 2",
        "host": "<IP_ADDRESS>",
        "effects": [
            "Solid",
            "Spots"
        ]
    }
],

Best regards, Ron

Please read through the README.md of the plugin. And look at the link posted above. There is an example of adding a WLED-instance in the README!

Sorry, again no result. I configured this:
    {
        "platform": "WLED",
        "name": "WLED Garage 1",
        "host": "192.168.xxx.xx",
        "effects": [
            "Solid",
            "Spots"
        ]
    },
    {
        "platform": "WLED",
        "wleds": [
            {
                "name": "WLED Garage 1",
                "host": "192.168.xxx.yy",
                "effects": [
                    "Solid",
                    "Spots"
                ],
                "turnOffWledWithEffect": true
            },
            {
                "name": "WLED Garage 2",
                "host": "<IP_ADDRESS>",
                "effects": [
                    "Solid",
                    "Spots"
                ],
                "turnOffWledWithEffect": true
            }
        ]
    }
],

But no WLED-button occurs in homekit. And again there is this red hint in the protocol:
"The dynamic platform WLED from the plugin homebridge-simple-wled seems to be configured multiple times in your config.json. This behaviour was deprecated in homebridge v1.0.0 and will be removed in v2.0.0!"

Are you on the latest homebridge version?
Do it show the device, if you click on the plus (+) in the HomeKit App and select "Add a new device" --> and then select you do not have a Code... ?

Thank You very much for Your help. Now the devices appear properly in homekit. I think we can close that issue. Best regards Ron


Create a separate x25519 private key for staging

This is a core team task because netlify access is required
Right now when the bot generates a payment permit then:

The bot takes an encrypted partner wallet's private key
Decrypt patner's private key via the X25519_PRIVATE_KEY env variable set in netlify
Generates a permit using the decrypted partner's wallet private key

The thing is that we use the same X25519_PRIVATE_KEY env variable value both for staging and production bot instances which is not really secure. We should use different X25519_PRIVATE_KEY env variables values for staging and production.
What should be done:

Generate a new X25519_PRIVATE_KEY here
Generate a new ethereum wallet private key
Encrypt the generated wallet private key with X25519_PRIVATE_KEY from step 1
Set the encrypted wallet value in the staging org config
Set wallet allowances with permit2 for the gnosis chain
Set the new X25519_PRIVATE_KEY env variable in netlify staging environment
At this point permits should be ready. Make sure that permits are generated and that they are claimable on staging.

@rndquu feel free to set up the priority level as well for every repo (in case this is not a draft task.)
I notice I have a tendency to set "low priority" tasks as "normal" and "normal priority" tasks as "medium"
I am not keen on changing the labels though because it would make the task seem insignificant. However all tasks are significant

looks like this task needs to be update as 4th step it's not available anymore

Set the encrypted wallet value in the staging org config

Points to 404


pull request

Just trying to make a pull request to get the source code to put it into my android thank you

I don't quite get what this PR is supposed to do?
We've already merged https://github.com/AndreMiras/QrScan/pull/34 that was updating the REAME.md

Thank you


Bumped zookeeper dependency version to work with latest release

@f1yers thanks for your contribution!
I’d ask that you please not bump the version; version bumps are part of the release process & should be handled by a project’s maintainer. Esp. in this case where a major dependency has incremented by a major (aka breaking) version. As such, this cookbook’s version number will also be incremented to reflect a potentially breaking change (since a major change in a dep like this could be breaking).

Merged in as 9d64abe7c1f7fc466fab9eff01fbc3aad82d3c00 & shipped as exhibitor v0.9.0


Apply filters to input of terminology functions.

Resolves #387

I think this change has been superseded, closing.


[BUG] Can not see input typed by users while registering/login

Description of the bug
the fields of all the inputs fields i.e. name, email, pass and others can not be seen by user while entering them
Please add screenshots (if applicable)

Add any other context about the problem here
No response
Code of Conduct

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

I have figured out the problem to this solution:
this is happening because entered text has color same as input bg, so we need some styling changes and that's it(hope so)
I have checked from my side that this is neither exist nor assigned, so assign it to me so I can start working on it.

I have already setup the project, one of my PR has merged too :)


Execution Order of Events

For example, I need to handle an event as last plugin after every plugin has handled it.
There is currently no reliable way to do this.
Yes, I can wait for PluginLoaded events etc but again this is not a 100% reliable way.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/36778438-execution-order-of-events?utm_campaign=plugin&utm_content=tracker%2F10719061&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F10719061&utm_medium=issues&utm_source=github).

Yep, will be in the next version of Rocket :)


Added support for AutoPublishAlias to AWS::Serverless::Function

Hi there -
Thanks for troposphere!
I was trying to use it to build a Serverless Application Model (SAM) definition and discovered that the AutoPublishAlias attribute was not supported.
Based on the SAM spec, AutoPublishAlias should be allowed as an optional attribute.
I've added support for it and a test.  LMK if all looks right as this is my first PR here.
Best,
-Donnie

Nice job...thanks!


feat: add river gauges AB#19381

This is the original PR with merge conflicts. Keep this around until we're sure nothing went wrong in solving these merge conflicts in the new PR.


How can dependent_on by empty in area dependencies?

including @RobTerwel @ChaelKruip @jorisberkhout: who knows? :smile:

The forms for these models will submit an empty area dependency if you don't select one:

{
"input_element": {
  "area_dependency_attributes": {
    "dependent_on": null
  }
}

accepts_nested_attributes_for will then save a blank AreaDependency. We should probably add a reject_if value to prevent the creation of empty records; although they don't have any negative effect, they are a bit pointless.

Thanks!


Fix typo on homepage

CI is failing because pnpm fails to download. Is it possible to restart the jobs to try again?


missing "scripts/start.js" in packages/builder ?

How would you tag this issue?

Question

Describe your issue:
when I try to start builder, find missing start.js .

How would you tag this issue?

Question

Describe your issue:
when I try to start builder, find missing start.js .

Just start the builder command in the root directory : npm start, and


Is it possible to remove the restriction such that SlidingUpPanelLayout can be a non-root View element?

I have a real world requirement that at the bottom of the activity layout there must be a Toolbar-like View that doesn't slide, and a sliding panel is above that View.

OK. I figured out that it is allowable to treat SlidingUpPanelLayout as a non-root View element in xml.


The new website document language switch did not work

Bug report 🐞
The new website document language switch did not work.I tried to clear the website cache and switch browsers, but the problem still exists.
Version & Environment
The website document is now in English by default. I tried to click to switch the language. The route was changed, but the language was not switched.
Expection
Language can be switched normally
Actual results (or Errors)

Sorry, I seem to be mistaken. It turned out that the new domain name file supports switching between Chinese and English. It is normal now. Now I will bookmark the new document address.

@2775668948  : ) .  Thanks for you attention,  Now, home page is only with English . I am busy now so i can't set Chinese version . But It will be a good suggestion in feature , And in tips  is that vue-rc domain  only work for canary version .


fix!: Remove pointerId from Draggable callbacks

Description
Since pointerId is already handled by the handle* methods we don't have to expose it in the onDrag* methods, this conforms with the standard set in Tappable.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the
relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] The title of my PR starts with a Conventional Commit prefix (fix:, feat:, docs: etc).
[x] I have read the Contributor Guide and followed the process outlined for submitting PRs.
[x] I have updated/added tests for ALL new/updated/fixed functionality.
[x] I have updated/added relevant documentation in docs and added dartdoc comments with ///.
[x] I have updated/added relevant examples in examples.

Breaking Change
Does your PR require Flame users to manually update their apps to accommodate your change?

[x] Yes, this is a breaking change. (Indicate it in the Conventional Commit prefix with a !,
e.g. feat!:, fix!:).
[ ] No, this is not a breaking change.

This is a positive change, it makes the API cleaner. However, several things need to be addressed:

This is a big breaking change ("big" meaning that it will definitely break the code for whoever uses Draggable components). Shouldn't we go via a more cautious deprecation route here?

The documentation is missing (I know it was missing before, but still). It's hard to figure out why the onDrag* methods return booleans, or what's the difference between onDragEnd vs onDragCancel. I think it's a great opportunity to add the missing doc-strings.

With pointerId parameter gone, how can a user access this information if they really need it? Say, could it be stored inside the info parameter?

We'll release it together with other breaking changes at a later point.

Good idea, will add it!

They can override the handle* methods to access the pointerId if they really need it.

Hmm, now that I took another look at Draggable, I think I understand what the problem is: it doesn't really make a Component draggable, it only enables drag events on it. In particular, the default implementation does not move the component in response to the drag event.
Also, I don't see any mechanism for "capturing" the drag event by the component that started it. In particular, how do we know that the component that had captured onDragStart will also be getting all onDragUpdates and the eventual onDragEnd?

Hmm, now that I took another look at Draggable, I think I understand what the problem is: it doesn't really make a Component draggable, it only enables drag events on it. In particular, the default implementation does not move the component in response to the drag event.

Correct, this isn't a problem a problem though, this is just how it is designed since it isn't a mixin on a PositionComponent but on a Component, and also users might want to react differently to such events on top of their components. We could create another mixin that is more "high level" specifically for the PositionComponent that does what you are describing.

Also, I don't see any mechanism for "capturing" the drag event by the component that started it. In particular, how do we know that the component that had captured onDragStart will also be getting all onDragUpdates and the eventual onDragEnd?

Because it has stored the pointer id, check the handleDrag* methods.

this isn't a problem a problem though, this is just how it is designed

Fair; but then it underscores the need for clear documentation, since the name of the class doesn't reflect its actual functionality. We could consider renaming it, or extend its functionality to match the current name, or both.

Because it has stored the pointer id, check the handleDrag* methods.

Ah ok, makes sense. But then what's the point in bool return value in onDragUpdate, onDragEnd and onDragCancel, since those can't be handled by any component other than the one that captured onDragStart?

Fair; but then it underscores the need for clear documentation, since the name of the class doesn't reflect its actual functionality. We could consider renaming it, or extend its functionality to match the current name, or both.

It could of course be clearer, but we do go through it here:
https://docs.flame-engine.org/1.0.0/gesture-input.html?highlight=draggable#draggable-components
I have actually never heard anyone that have thought that it would move the component directly by adding the mixin, which is a bit strange since it does have a bit of a confusing name like you say, or it could be because most come from the Flutter world and know how the detectors work.

Ah ok, makes sense. But then what's the point in bool return value in onDragUpdate, onDragEnd and onDragCancel, since those can't be handled by any component other than the one that captured onDragStart?

It can be "captured" by more than one component.

It can be "captured" by more than one component.

Seems to me that dragging multiple components with a single gesture is an even more rare use case than multi-finger drag.

Seems to me that dragging multiple components with a single gesture is an even more rare use case than multi-finger drag.

They are not mutally exclusive, you can do both. The id is only removed from the callbacks, not the handle methods. I don't get your point here I think.

The point of this PR is to simplify the API of Draggable by removing the pointerId arguments that are very rarely needed (but the users who need them can still access via the handle* methods).
At the same time, boolean return values are also very rarely needed (perhaps even more rarely), and removing them would provide an even better improvement to the API (improvement because the developer doesn't need to worry whether to return true or false or what does it all mean).

At the same time, boolean return values are also very rarely needed (perhaps even more rarely), and removing them would provide an even better improvement to the API (improvement because the developer doesn't need to worry whether to return true or false or what does it all mean)

These booleans are used all the time, or do you mean that the default should be false and that the events should always only affect the top most component? The feature to let an event propagate until you tell it to stop is not something that is unique to Flame.
The boolean is perhaps used more in other events, like in the Tappable, but we want to have the API consistent.

The feature to let an event propagate until you tell it to stop is not something that is unique to Flame.

I know, this paradigm comes all the way from HTML. But, in HTML there is no explicit marking of components as tappable / draggable / etc -- all elements respond to all events; and also they don't support 2 levels of APIs "simple" and "advanced" as we do.
So, yes, if a component is Tappable and is visually on top, then it should receive the tap event. If it is Draggable, it should receive the drag event, etc. At least in the "simple API" case. The cases where you'd need to pass the event through are so rare that I can't imagine a single example of a game where that could be useful. Can you?

I know, this paradigm comes all the way from HTML. But, in HTML there is no explicit marking of components as tappable / draggable / etc -- all elements respond to all events; and also they don't support 2 levels of APIs "simple" and "advanced" as we do.

The second, "advanced" level, is mostly for internal use, but can be overridden if there really is a need for it, almost no use-cases need to do that.

So, yes, if a component is Tappable and is visually on top, then it should receive the tap event. If it is Draggable, it should receive the drag event, etc. At least in the "simple API" case. The cases where you'd need to pass the event through are so rare that I can't imagine a single example of a game where that could be useful. Can you?

Yes, I have seen it used in several games. One game for example was falling fruits where you should tap the correct fruits to gain points and these fruits could overlap, and one tap should give all the plus and minus points from all the fruits that were tapped. We can discuss this more in the chat, because this is not related to this PR.

If we are going to add breaking changes for the pointerId, might it be an idea to expose it on the info event classes and create such classes for the cancel events as well?
Or does the dragCancel not give a pointer?

If we are going to add breaking changes for the pointerId, might it be an idea to expose it on the info event classes and create such classes for the cancel events as well?

It is possible, a little bit of a hassle in the gestures file though.

Or does the dragCancel not give a pointer?

Yes, you can get the pointer in dragCancel, but you can't get the other information that is needed to create an event.
Do you want to try to do it? :smile:

class HasDraggables still contains int pointerId parameter

class HasDraggables still contains int pointerId parameter

It does, and it should.


Docker Pull not Working

I was trying to install from docker using the suggested command:
 docker pull seahorn/clam_5.0:latest
But I keep getting the following error
Error response from daemon: manifest for seahorn/clam_5.0:latest not found: manifest unknown: manifest unknown
I am using Docker version 19.03.4, build 9013bf583a on Debian GNU/Linux 9 (stretch)

There is no latest tag.
If you look at https://cloud.docker.com/u/seahorn/repository/docker/seahorn/clam_5.0
you will see the only tag is xenial.

Thanks.


Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated since Python 3.3,and in 3.9 it will stop working

__init__.py:16
class frozendict(collections.Mapping):

Already a PR: #30

Oh, and a duplicate: #25


Roll Flutter from 3c3c9a1bd98f to f2e89755e54f (19 revisions)

https://github.com/flutter/flutter/compare/3c3c9a1bd98f...f2e89755e54f
2023-02-07<EMAIL_ADDRESS>b67690f69 Roll Skia from 6babb6a1afe6 to 3b1401c4870d (1 revision) (flutter/engine#39455) (flutter/flutter#120198)
2023-02-07<EMAIL_ADDRESS>[web] Move JS content to its own .js files (flutter/flutter#117691)
2023-02-07<EMAIL_ADDRESS>Roll Plugins from d065e4e0a82a to 6f985d57b04b (10 revisions) (flutter/flutter#120193)
2023-02-07<EMAIL_ADDRESS>Roll Flutter Engine from 5aadda2f40b1 to e432b82f49f3 (3 revisions) (flutter/flutter#120191)
2023-02-07<EMAIL_ADDRESS>Fix cut button creation in 'buttonItemsForToolbarOptions' (flutter/flutter#119822)
2023-02-07<EMAIL_ADDRESS>5aadda2f4 Roll Skia from 419bb63e733d to 83da27e4cd3a (1 revision) (flutter/engine#39448) (flutter/flutter#120172)
2023-02-07<EMAIL_ADDRESS>M3 segmented buttons token fixes (flutter/flutter#120095)
2023-02-07<EMAIL_ADDRESS>39f5e4cba Roll Skia from 7e2c9f54c0fd to 419bb63e733d (1 revision) (flutter/engine#39447) (flutter/flutter#120159)
2023-02-07<EMAIL_ADDRESS>fix a [SelectableRegion] crash bug (flutter/flutter#120076)
2023-02-07<EMAIL_ADDRESS>remove deprecated accentTextTheme and accentIconTheme from ThemeData (flutter/flutter#119360)
2023-02-07<EMAIL_ADDRESS>Roll Flutter Engine from 2a104cdfcdf8 to 165126e7034c (13 revisions) (flutter/flutter#120150)
2023-02-07<EMAIL_ADDRESS>Run verify_binaries_codesigned task on release branches (flutter/flutter#120141)
2023-02-07<EMAIL_ADDRESS>Remove unreachable_from_main linter rule (flutter/flutter#120110)
2023-02-06<EMAIL_ADDRESS>Force web_tool_tests to run on x64 builders (flutter/flutter#120128)
2023-02-06<EMAIL_ADDRESS>Fix context menu web examples (flutter/flutter#120104)
2023-02-06<EMAIL_ADDRESS>Fix flutter_tools stuck when using custom LLDB prompt (flutter/flutter#119443)
2023-02-06<EMAIL_ADDRESS>Added "insertAll" and "removeAll" methods to AnimatedList (flutter/flutter#115545)
2023-02-06<EMAIL_ADDRESS>Roll Plugins from 9302d87ee545 to d065e4e0a82a (6 revisions) (flutter/flutter#120084)
2023-02-06<EMAIL_ADDRESS>Update Android TESTOWNERS (flutter/flutter#119960)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-plugins
Please CC rmistry@google.com,stuartmorgan<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Plugins: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


How to enable polling after a producer is created with EventsPollingOption::Manual

Hi,
It's kinda related to setErrorCallback/statsCallback, so if we create a producer with EventsPollingOption::Manual(because I want to set callbacks afterwards), it seems there isn't a public interface I can call to enable polling?

Hi, @YaoDing Here's a quite simple example for pollEvents(),  https://github.com/morganstanley/modern-cpp-kafka/blob/main/doc/KafkaProducerQuickStart.md#kafkaproducer-with-kafkaclientskafkaclienteventspollingoption
Generally speaking, you have to periodically call producer.pollEvents(), no matter whether there's any message just been sent or not.

Yeah, I was looking for something like startBackgroundPollingIfNecessary in Producer class so that I don't need to create thread/take care of it by myself :-). Given we allow users to create Producer with EventsPollingOption::Manual option, maybe you could consider make startBackgroundPollingIfNecessary public? But it's another topic, I will close this one now. Thank you for answering all my questions!

@YaoDing It doesn't make much sense to expose the startBackgroundPollingIfNecessary for users, -- it does nothing else except triggering the internal polling thread to start, then why not using the default producer constructor (with no EventsPollingOption::Manual specified)? While for a sophisticated user, the simple pollEvents() interface for a manual polling producer would be enough, -- we're able to create some "helper" code, but probably not what they want.


MM-34324 Migrate 'components/admin_console/permission_schemes_settings/permission_team_scheme_settings' and associated tests to TypeScript

Summary
Migrate 'components/admin_console/permission_schemes_settings/permission_team_scheme_settings' and associated tests to TypeScript
Ticket Link
Fixes https://github.com/mattermost/mattermost-server/issues/17251
Release Note
NONE

Hello @pikami,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

/update-branch

Thanks for participating in Hacktoberfest! You can claim your sticker set here: https://get.printfection.com/hacktober21/4144583267 @pikami


Demo site doesn't work properly

Except for the home screen, no images load on any pages. Like this:

After clicking on a tadpole to view, this view came up:

Thanks @garettarrowood as always for raising the issue. We'll put this one on icebox for now and "thaw" out when we get to reviewing the Rails portion of our track.

going to remove the link to the demo until we can get it working properly


Add feature to read gaussian_prior from setting file

Thanks for the all the changes, looks mostly good! One question I have is that, if two light profiles are provided with two filters, then number of light profiles is changed to 4 by dolphin, then will the provided prior be appropriately passed for the corresponding light profile in the second filter?

Thanks for the all the changes, looks mostly good! One question I have is that, if two light profiles are provided with two filters, then number of light profiles is changed to 4 by dolphin, then will the provided prior be appropriately passed for the corresponding light profile in the second filter?

Currently, you would have to manually specify the priors for each light profile for each filter. Should I automatically apply the prior to the same light profile for different filters?

I think, the convention should be consistent for the users. It is currently not consistent as the user has to provide the light profile set for a single band, but then have to specify priors for duplicated light profiles. With future complications in mind, I think it would be wiser to actually make the user to provide the full list of light profiles (without the need for dolphin to duplicate them) and a separate keyword to specify which light profile corresponds to which band.

Actually I was thinking what we can do is if dolphin only detects indices from the first filter, it will automatically add the profiles and priors to the other profiles , but if  information  from other filters are found it will manually add the information