Enhancement request - make the control points easier to grab with mouse/stylus

As suggested by David AKA "the,quass" in this conversation ......
http://skibuilders.com/phpBB2/viewtopic.php?t=4779
"My only issue is the pick points are pretty tiny at high res so they are really difficult to drag with the stylus. I could change red and it would sort this I think"
I agree with him they are hard to grab with the mouse and in a tablet he finds it tricky with stylus. Can they be locked on to easier and more visible?
cheers
Rich

As suggested by David AKA "the,quass".
"My only issue is the pick points are pretty tiny at high res so they are really difficult to drag with the stylus. I could change red and it would sort this I think"
I agree with him they are hard to grab with the mouse too I find. Can they be locked on to easier and more visible?
cheers
Rich

I've never used git before so I thought I'd give this simple one a bash. I've doubled the size of the control handles when activated, increased the activation area, and increased the size of the handles when inactive. @Rich if you're able to compile the project and take a look, let me know if it's what you had in mind and close or comment again as you see fit. I'm gonna get us to use git at work, this was much more pleasant than SVN !

haha @danielgraf I'll leave compile for the experts mate, I once used an assembler compiler for Intel 8031 but my code was a mess honestly a total mess! So letting me loose on this compiling is like a machine gun in a childs hands - dangerous, I'll leave that to @mikemag to do the releases!!!!!

Released: https://github.com/mikemag/snoCAD-X/releases
Personally, I found the control points much easier to grab, and not too big. Nicely done.


[bug] Edge 12 or 13 targets don't transpile destructuring.

According to MDN, destructuring is supported on Edge 14+, but setting the target to edge:12 or edge:13 doesn't transpile the following:
this._elementManager.setClasses(...properties.classes);

I was expecting it to use Function.prototype.apply.

If MDN is true, then here's a PR: https://github.com/Rich-Harris/buble/pull/17

That's actually a spread operator applied to an array in a function call, which is supported in Edge 12.

Ah, confusing. I feel like the MDN docs could unify restructuring and spread features in a better way. It's hard to know exactly what to search for.


Fix dot in single page

Fix dot showing when is single page.

@eulerlobo Does this handle adding back the dot if you add more elements to the carousel via add.owl.carousel  or is there additional work to be done to handle that situation?


sidebar width

i'm trying to find the width of sidebar and would like to change it because the text does not show as a whole..

Personally I like to set it to auto


feat: global screen wrapper

Summary of Changes
This is a first step into having a global screen wrapper to allow more customization over pages layout
Included

Creating the wrapper with minimal settings
Applying to some screens as a means of initial implementation
Applying to terms screen with flags to allow custom headers
Extend injectable screen options to allow injecting custom header

Replace this text with a high-level summary of the changes included in this PR.
Related Issues
Please reference here any issue #'s that are relevant to this PR, or simply enter "N/A" if this PR does not relate to any existing issues.
Pull Request Checklist
Tick all boxes below to demonstrate that you have completed the respective task. If the item does not apply to your this PR check it anyway to make it apparent that there's nothing to do.

[ ] All commits contain a DCO Signed-off-by line (we use the DCO GitHub app to enforce this);
[ ] Updated LICENSE-3RD-PARTY.md for any added dependencies or vendored components;
[ ] Updated documentation as needed for changed code and new or modified features;
[ ] Added sufficient tests so that overall code coverage is not reduced.

If you have any questions to any of the points above, just submit and ask! This checklist is here to help you, not to deter you from contributing!
Pro Tip 🤓

Read our contribution guide at least once; it will save you a few review cycles!
Your PR will likely not be reviewed until all the above boxes are checked and all automated tests have passed.

PR template adapted from the Python attrs project.

@MosCD3 Able to up so we can merge this one in?


Add banner showing MultiMatrix load erros

Currently, there is no banner to show errors in table loading.

Let's wait until Pranav's changes are in for this. It'll work better when I've added vuex

Closed via #237


quarkus-devtools-testing to version.io.quarkus.quarkus-test-maven

The kogito-runtimes prod nightly build is failing due to io.quarkus:quarkus-devtools-testing is not productized by throwing
[ERROR] Failed to execute goal on project kogito-quarkus-serverless-workflow-integration-test: Could not resolve dependencies for project org.kie.kogito:kogito-quarkus-serverless-workflow-integration-test:jar:2.0.0.redhat-2022080923: Failure to find io.quarkus:quarkus-devtools-testing:jar:2.7.5.Final-redhat-00011 in REPO_URL was cached in the local repository, resolution will not be reattempted until the update interval of qe-repository has elapsed or updates are forced -> [Help 1]

I suggest to use version.io.quarkus.quarkus-test-maven property for io.quarkus:quarkus-devtools-testing dependency, this way community libraries for this particular dependency will be taken.

How to replicate CI configuration locally?

Build Chain tool does "simple" maven build(s), the builds are just Maven commands, but because the repositories relates and depends on each other and any change in API or class method could affect several of those repositories there is a need to use build-chain tool to handle cross repository builds and be sure that we always use latest version of the code for each repository.
build-chain tool is a build tool which can be used on command line locally or in Github Actions workflow(s), in case you need to change multiple repositories and send multiple dependent pull requests related with a change you can easily reproduce the same build by executing it on Github hosted environment or locally in your development environment. See local execution details to get more information about it.

How to retest this PR or trigger a specific build:

for pull request checks
Please add comment: Jenkins retest this

for a specific pull request check
Please add comment: Jenkins (re)run [kogito-runtimes|kogito-apps|kogito-examples] tests

for quarkus branch checks
Run checks against Quarkus current used branch
Please add comment: Jenkins run quarkus-branch

for a quarkus branch specific check
Run checks against Quarkus current used branch
Please add comment: Jenkins (re)run [kogito-runtimes|kogito-apps|kogito-examples] quarkus-branch

for quarkus main checks
Run checks against Quarkus main branch
Please add comment: Jenkins run quarkus-main

for a specific quarkus main check
Run checks against Quarkus main branch
Please add comment: Jenkins (re)run [kogito-runtimes|kogito-apps|kogito-examples] quarkus-branch

for native checks
Run native checks
Please add comment: Jenkins run native

for a specific native check
Run native checks
Please add comment: Jenkins (re)run [kogito-runtimes|kogito-apps|kogito-examples] native

for mandrel checks
Run native checks against Mandrel image
Please add comment: Jenkins run mandrel

for a specific mandrel check
Run native checks against Mandrel image
Please add comment: Jenkins (re)run [kogito-runtimes|kogito-apps|kogito-examples] mandrel

Full Kogito testing (with cloud images and operator BDD testing)
Please add comment: Jenkins run BDD
This check should be used only if a big change is done as it takes time to run, need resources and one full BDD tests check can be done at a time ...

(tests) - kogito-apps job #748 was: UNSTABLE
Possible explanation: This should be test failures
Please look here: https://eng-jenkins-csb-business-automation.apps.ocp-c1.prod.psi.redhat.com/job/KIE/job/kogito/job/main/job/pullrequest/job/kogito-runtimes.tests.downstream.kogito-apps/748/display/redirect
Test results:

PASSED: 1892
FAILED: 1

Those are the test failures:

org.kie.kogito.index.ProcessDataIndexMongoDBIT.testProcessInstanceEvents
java.lang.IllegalStateException: Failed to load ApplicationContextCaused by: org.springframework.context.ApplicationContextException: Failed to start bean 'webServerStartStop'; nested exception is org.springframework.boot.web.server.PortInUseException: Port 53504 is already in useCaused by: org.springframework.boot.web.server.PortInUseException: Port 53504 is already in useCaused by: java.lang.IllegalArgumentException: standardService.connector.startFailedCaused by: org.apache.catalina.LifecycleException: Protocol handler start failedCaused by: java.net.BindException: Address already in use

Jenkins run kogito-apps

kogito-apps unstable due to ProcessDataIndexMongoDBIT.testProcessInstanceEvents failture
[2022-08-10T06:15:29.321Z] Caused by: java.net.BindException: Address already in use
[2022-08-10T06:15:29.321Z] 	at java.base/sun.nio.ch.Net.bind0(Native Method) ~[na:na]
[2022-08-10T06:15:29.321Z] 	at java.base/sun.nio.ch.Net.bind(Net.java:459) ~[na:na]
[2022-08-10T06:15:29.321Z] 	at java.base/sun.nio.ch.Net.bind(Net.java:448) ~[na:na]
[2022-08-10T06:15:29.321Z] 	at java.base/sun.nio.ch.ServerSocketChannelImpl.bind(ServerSocketChannelImpl.java:227) ~[na:na]
[2022-08-10T06:15:29.321Z] 	at org.apache.tomcat.util.net.NioEndpoint.initServerSocket(NioEndpoint.java:274) ~[tomcat-embed-core-9.0.60.jar:9.0.60]
[2022-08-10T06:15:29.321Z] 	at org.apache.tomcat.util.net.NioEndpoint.bind(NioEndpoint.java:229) ~[tomcat-embed-core-9.0.60.jar:9.0.60]
[2022-08-10T06:15:29.321Z] 	at org.apache.tomcat.util.net.AbstractEndpoint.bindWithCleanup(AbstractEndpoint.java:1227) ~[tomcat-embed-core-9.0.60.jar:9.0.60]
[2022-08-10T06:15:29.321Z] 	at org.apache.tomcat.util.net.AbstractEndpoint.start(AbstractEndpoint.java:1313) ~[tomcat-embed-core-9.0.60.jar:9.0.60]
[2022-08-10T06:15:29.321Z] 	at org.apache.coyote.AbstractProtocol.start(AbstractProtocol.java:614) ~[tomcat-embed-core-9.0.60.jar:9.0.60]
[2022-08-10T06:15:29.321Z] 	at org.apache.catalina.connector.Connector.startInternal(Connector.java:1072) ~[tomcat-embed-core-9.0.60.jar:9.0.60]
[2022-08-10T06:15:29.321Z] 	... 93 common frames omitted

I guess not related with this change

still it seems my comment about retest is not triggering the new build?

Jenkins run kogito-apps tests

still it seems my comment about retest is not triggering the new build?
according to documentation tests is required, see my previous comment @mareknovotny (it launched the test)

(tests) - kogito-examples job #686 was: UNSTABLE
Possible explanation: This should be test failures
Please look here: https://eng-jenkins-csb-business-automation.apps.ocp-c1.prod.psi.redhat.com/job/KIE/job/kogito/job/main/job/pullrequest/job/kogito-runtimes.tests.downstream.kogito-examples/686/display/redirect
Test results:

PASSED: 550
FAILED: 1

Those are the test failures:

org.kie.kogito.quarkus.outbox.OutboxIT.(?)
java.lang.RuntimeException: org.testcontainers.containers.ContainerLaunchException: Timed out waiting for container port to open (localhost ports: [8080, 49183] should be listening)	at org.kie.kogito.quarkus.outbox.OutboxIT.before(OutboxIT.java:86)Caused by: org.testcontainers.containers.ContainerLaunchException: Timed out waiting for container port to open (localhost ports: [8080, 49183] should be listening)


Library release removal request

Library name
YoupiLabEsp32
Version to remove
1.0.0

Bonsoir

Hi @josiascarnot. You must delete the tag from the library's GitHub repository before we can remove the release. I see that it has not yet been removed.
I'll provide instructions you can follow to do that:

Click the following link to open the GitHub page of the tag:
https://github.com/YoupiLab/YoupiLabEsp32/releases/tag/1.0.0
Click the Delete button on that page.
Click the I understand, delete this tag button.
Comment here to let me know that the tag has been deleted so that I can proceed with removing the release from Arduino Library Manager.

Hello i remove already
Le mer. 5 juil. 2023 à 18:57, per1234 @.***> a écrit :

Hi @josiascarnot https://github.com/josiascarnot. You must delete the
tag https://git-scm.com/book/en/v2/Git-Basics-Tagging from the
library's GitHub repository before we can remove the release. I see that it
has not yet been removed.
I'll provide instructions you can follow to do that:

Click the following link to open the GitHub page of the tag:
https://github.com/YoupiLab/YoupiLabEsp32/releases/tag/1.0.0
Click the Delete button on that page.
Click the I understand, delete this tag button.
Comment here to let me know that the tag has been deleted so that I
can proceed with removing the release from Arduino Library Manager.

—
Reply to this email directly, view it on GitHub
https://github.com/arduino/library-registry/issues/3037#issuecomment-1622224314,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3DILBBKYJS4QKNN7IMZKFLXOWTJDANCNFSM6AAAAAAZ7HOYHQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

hello i remove release already

@josiascarnot the tag is still there in the library's GitHub repository:
https://github.com/YoupiLab/YoupiLabEsp32/releases/tag/1.0.0
Please carefully follow the instructions I provided in my previous reply.

Sorry i remove now
Le jeu. 6 juil. 2023 à 22:56, per1234 @.***> a écrit :

@josiascarnot https://github.com/josiascarnot the tag is still there in
the library's GitHub repository:
https://github.com/YoupiLab/YoupiLabEsp32/releases/tag/1.0.0
Please carefully follow the instructions I provided in my previous reply.
—
Reply to this email directly, view it on GitHub
https://github.com/arduino/library-registry/issues/3037#issuecomment-1624357390,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3DILBDYUDN2C4DJLQ7W5RLXO4YBNANCNFSM6AAAAAAZ7HOYHQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi @josiascarnot. I am confirming your request.
Unlike submissions, this type of request is still handled manually. The Library Manager index maintainer occasionally comes through to clear all the outstanding requests. They will close this issue when it is done. You will get a notification from GitHub at that time.
After that, you can monitor the indexing of the library via its dedicated logs web page.
Information about that here:
https://github.com/arduino/library-registry/blob/main/FAQ.md#can-i-check-on-library-releases-being-added-to-library-manager

I see now that there is another tag named ylesp in the repository that the Library Manager indexer will use as the 1.0.0 release even after you deleted the 1.0.0 tag (it is the version field of the library.properties file that sets the version number for the release, not the tag name). That tag must be removed also:
I'll provide instructions you can follow to do that:

Click the following link to open the GitHub page of the tag:
https://github.com/YoupiLab/YoupiLabEsp32/releases/tag/ylesp
Click the Delete button on that page.
Click the I understand, delete this tag button.
Comment here to let me know that the tag has been deleted so that I can proceed with removing the release from Arduino Library Manager.

it is done
Le ven. 7 juil. 2023 à 09:07, per1234 @.***> a écrit :

I see now that there is another tag named ylesp in the repository that
the Library Manager indexer will use as the 1.0.0 release even after you
deleted the 1.0.0 tag (it is the version field of the library.properties
file that sets the version number for the release, not the tag name). That
tag must be removed also:
I'll provide instructions you can follow to do that:

Click the following link to open the GitHub page of the tag:
https://github.com/YoupiLab/YoupiLabEsp32/releases/tag/ylesp
Click the Delete button on that page.
Click the I understand, delete this tag button.
Comment here to let me know that the tag has been deleted so that I
can proceed with removing the release from Arduino Library Manager.

—
Reply to this email directly, view it on GitHub
https://github.com/arduino/library-registry/issues/3037#issuecomment-1624978254,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3DILBCMS2RLYIIKC3JZDUTXO67UTANCNFSM6AAAAAAZ7HOYHQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi @josiascarnot. I am confirming your request.
Unlike submissions, this type of request is still handled manually. The Library Manager index maintainer occasionally comes through to clear all the outstanding requests. They will close this issue when it is done. You will get a notification from GitHub at that time.
After that, you can monitor the indexing of the library via its dedicated logs web page.
Information about that here:
https://github.com/arduino/library-registry/blob/main/FAQ.md#can-i-check-on-library-releases-being-added-to-library-manager

Hello
Le ven. 7 juil. 2023 à 13:42, per1234 @.***> a écrit :

Hi @josiascarnot https://github.com/josiascarnot. I am confirming your
request.
Unlike submissions, this type of request is still handled manually. The
Library Manager index maintainer occasionally comes through to clear all
the outstanding requests. They will close this issue when it is done. You
will get a notification from GitHub at that time.
After that, you can monitor the indexing of the library via its dedicated
logs web page.
Information about that here:
https://github.com/arduino/library-registry/blob/main/FAQ.md#can-i-check-on-library-releases-being-added-to-library-manager
—
Reply to this email directly, view it on GitHub
https://github.com/arduino/library-registry/issues/3037#issuecomment-1625359546,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3DILBF5JWEMFLVSE5TWA43XO772BANCNFSM6AAAAAAZ7HOYHQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hello

Supression takes quite a long time, so I'd like to know how long I have to wait for my request to be executed.

@josiascarnot I don't have any information for you about when your request will be accommodated. Please be patient. I can assure you that spamming this thread with messages is not productive.
But really, removing a release from Library Manager is something that should only be necessary in very rare cases where the release contains something that absolutely must not be distributed such as a bug that might cause permanent damage to the Arduino board or peripherals. In all other cases, the professional development practice is to simply release a new version (e.g., 1.0.1) with whatever improvements or fixes you want to distribute to the users.
We allow requests for removal of releases because we understand that there might be some exceptional circumstances where this is truly needed, but you should understand that if the maintainer of every one of the thousands of libraries requested removals of releases for trivial reasons and nagged us about it then we would need to withdraw the offer to accommodate such requests so you behaving irresponsibly here.

Done:
2023/07/18 12:35:25 Loaded 5988 libraries from DB
Removing<EMAIL_ADDRESS>Success!


tmpfs mount

Most of the Display packages need to write tmp files for every render, so it's important that there be a proper tmpfs mount.
https://docs.docker.com/storage/tmpfs/

I tried docker run --rm --tmpfs /tmp -p 8888:8888 -v $PWD/notebook:/home/jovyan/work --name learn-you-a-haskell ghcr.io/jamesdbrock/ihaskell-notebook:master jupyter lab --LabApp.token='' and confirmed that the ihaskell-diagrams package is using the /tmp directory. But it didn't improve refresh speed perceptibly.


[Feature] Local SQLite DB

Contact Details
No response
Describe the Feature
Use a local SQLite DB to store user feed data
Requirements

[ ] Create an initial DB and relevant data tables
[ ] Insert article/category info, etc. into relevant data
[ ] Allow for update for new articles/categories
[ ] Update read, priority, and other tags, etc.

@AndiRen
I'm spit balling here but I think the DB needs the following tables / columns

feeds

feed_id: Primary key because why not
url: the access url for feed
alias [Optional]: the display name of the feed (instead of just displaying the url
poll_interval: the time to wait before pulling / checking for new items in the feed
category: Foreign key references categories table to sort on in the front end

categories

category_id: for reference
name: the name to display back to the user
sort_order: I don’t know about this one but I think we could leverage it to sort the front end in some way (we may want to do this with feeds too again I don’t know)
parent_category: Another maybe but I think we might be able to nest the categories ❓

I'm probably missing some things here but I think with these tables we would have a solid foundation

Oh we should also probably add

feed_items

feed_item_id: PK
content: Blob the stuff of a single element
feed_id: FK to feeds table

Totally! Looks like a good set of tables. Articles could also maybe be in there. May need a connector table between feeds/categories and articles so there could be many categories an article belonged to.


feat: improve sdk behaviour with custom auth and spinner loading

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#2  👈
#1
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Shedna and the rest of your teammates on  Graphite


The Makefile does not work on the Mac

Description of the issue
I downloaded the latest release of OEO to my Mac and ran “make”, but seem to be experiencing a compatibility problem with the “sed” command.

$ make
mkdir -p build/oeo/1.13.0/imports
mkdir -p build/oeo/1.13.0/modules
cp src/ontology/catalog-v001.xml build/oeo/1.13.0/catalog-v001.xml
sed -i -E "s/http:\/\/openenergy-platform\.org\/ontology\/oeo\/dev\/([a-zA-Z/\.\-]+)/http:\/\/openenergy-platform\.org\/ontology\/oeo\/releases\/1.13.0\/\1/m" build/oeo/1.13.0/catalog-v001.xml
sed: 1: "s/http:\/\/openenergy-p ...": \1 not defined in the RE
make: *** [build/oeo/1.13.0/catalog-v001.xml] Error 1

(I think I worked around the markdown syntax, by doubling all the backslashes, so that it looks right here.)
It turns out that the Makefile depends on the GNU version of the “sed” command, which is not available on OS X.
Ideas of solution
The regular expressions could be adjusted to work with the traditional BSD version of the “sed” command, which I believe is available on every variant of *nix. Moreover, it is also possible to rewrite them more simply, without escaping all those slashes. For example:

$ echo "http://openenergy-platform.org/ontology/oeo/dev/test.xxx" |
sed "s,http://openenergy-platform.org/ontology/oeo/dev/([a-zA-Z/.-]*),http://openenergy-platform.org/ontology/oeo/releases/1.13.0/\1,"

Which outputs:

http://openenergy-platform.org/ontology/oeo/releases/1.13.0/test.xxx

(Again. I think I worked around the markdown syntax so that it looks right here.)
Workflow checklist

[X] I am aware of the workflow for this repository

Taking a look at this

Taking a look at this

@areleu : Any update on this?


No outer  founded. on UMD

TuSimple/naive-ui version (版本)
2.34.3
Vue version (Vue 版本)
3.3.0-beta.1
Browser and its version (浏览器及其版本)
112.0.5615.121
System and its version (系统及其版本)
Windows 10
Node version (Node 版本)
Reappearance link (重现链接)
https://jsbin.com/saxubitaki/1/edit?html,output
Reappearance steps (重现步骤)
In the case of UMD
excute this code "naive.useDialog()"
Unable to get dialog object,
throw exception：No outer  founded
Expected results (期望的结果)
dialog available.
Actual results (实际的结果)
No outer  founded
Remarks (补充说明)
in JS Bin and local，Can not be used

you need to wrap the component where you call related methods inside n-dialog-provider and use useDialog to get the API.

To access the API for calling related methods, you should wrap the component in n-dialog-provider.

BAD Design! HARD to use.

This issue does not have any recent activity. If you are still experiencing similar problems, open a new error, including a minimal copy of the problem


Add rosdep for python-catkin-sphinx

see: ros-infrastructure/catkin-sphinx#6

+1
Fedora packages: https://admin.fedoraproject.org/pkgdb/package/rpms/python-catkin-sphinx/
Ubuntu packages from OSRF


Use RAMDISK for etcd storage in Ansible install jobs

Make the generate script work from new repo basedir
Signed-off-by: Steve Kuznetsov<EMAIL_ADDRESS>
Added a debugging job
Signed-off-by: Steve Kuznetsov<EMAIL_ADDRESS>
Use RAMDISK for etcd storage in Ansible install jobs
Signed-off-by: Steve Kuznetsov<EMAIL_ADDRESS>

test running here: https://ci.openshift.redhat.com/jenkins/job/skuznets_test_etcd_data_dir/3/

OK, reproducing the common failure...
TASK [etcd : Enable etcd] ******************************************************
task path: /usr/share/ansible/openshift-ansible/roles/etcd/tasks/main.yml:122
fatal: [localhost]: FAILED! => {
    "changed": false,
    "failed": true,
    "generated_timestamp": "2017-04-20 08:44:31.956237",
    "msg": "Unable to start service etcd: Job for etcd.service failed because the control process exited with error code. See \"systemctl status etcd.service\" and \"journalctl -xe\" for details.\n"
}

The config file is:
ETCD_NAME=default
ETCD_DATA_DIR=/tmp/etcd
#ETCD_SNAPSHOT_COUNTER=10000
ETCD_HEARTBEAT_INTERVAL=500
ETCD_ELECTION_TIMEOUT=2500
ETCD_LISTEN_CLIENT_URLS=https://<IP_ADDRESS>:2379
#ETCD_MAX_SNAPSHOTS=5
#ETCD_MAX_WALS=5
#ETCD_CORS=

ETCD_ADVERTISE_CLIENT_URLS=https://<IP_ADDRESS>:2379

#[proxy]
#ETCD_PROXY=off

#[security]
ETCD_CA_FILE=/etc/etcd/ca.crt
ETCD_CERT_FILE=/etc/etcd/server.crt
ETCD_KEY_FILE=/etc/etcd/server.key
ETCD_PEER_CA_FILE=/etc/etcd/ca.crt
ETCD_PEER_CERT_FILE=/etc/etcd/peer.crt
ETCD_PEER_KEY_FILE=/etc/etcd/peer.key
#[logging]
ETCD_DEBUG="False"

So we are seeing the data dir being set, but NOT the debugging... the etcd journal has nothing at all interesting:
Apr 20 08:44:31 ip-172-18-8-7.ec2.internal systemd[1]: Starting Etcd Server...
Apr 20 08:44:31 ip-172-18-8-7.ec2.internal etcd[23921]: recognized and used environment variable ETCD_ADVERTISE_CLIENT_URLS=https://<IP_ADDRESS>:2379
Apr 20 08:44:31 ip-172-18-8-7.ec2.internal etcd[23921]: recognized and used environment variable ETCD_CA_FILE=/etc/etcd/ca.crt
Apr 20 08:44:31 ip-172-18-8-7.ec2.internal etcd[23921]: recognized and used environment variable ETCD_CERT_FILE=/etc/etcd/server.crt
Apr 20 08:44:31 ip-172-18-8-7.ec2.internal etcd[23921]: recognized and used environment variable ETCD_DEBUG=False
Apr 20 08:44:31 ip-172-18-8-7.ec2.internal etcd[23921]: recognized and used environment variable ETCD_ELECTION_TIMEOUT=2500
Apr 20 08:44:31 ip-172-18-8-7.ec2.internal etcd[23921]: recognized and used environment variable ETCD_HEARTBEAT_INTERVAL=500
Apr 20 08:44:31 ip-172-18-8-7.ec2.internal etcd[23921]: recognized and used environment variable ETCD_KEY_FILE=/etc/etcd/server.key
Apr 20 08:44:31 ip-172-18-8-7.ec2.internal etcd[23921]: recognized and used environment variable ETCD_PEER_CA_FILE=/etc/etcd/ca.crt
Apr 20 08:44:31 ip-172-18-8-7.ec2.internal etcd[23921]: recognized and used environment variable ETCD_PEER_CERT_FILE=/etc/etcd/peer.crt
Apr 20 08:44:31 ip-172-18-8-7.ec2.internal systemd[1]: etcd.service: main process exited, code=exited, status=1/FAILURE
Apr 20 08:44:31 ip-172-18-8-7.ec2.internal systemd[1]: Failed to start Etcd Server.
Apr 20 08:44:31 ip-172-18-8-7.ec2.internal systemd[1]: Unit etcd.service entered failed state.
Apr 20 08:44:31 ip-172-18-8-7.ec2.internal systemd[1]: etcd.service failed.

https://ci.openshift.redhat.com/jenkins/job/skuznets_test_etcd_data_dir/4/

woo https://ci.openshift.redhat.com/jenkins/job/skuznets_test_etcd_data_dir/8/


DiffProvider for byte[]

Hi,
I'm testing your library and found out that if a bean has byte[] it throws IllegalStateException.
How can add a proper differ  for this object type? I couldn't figure out a way of doing that reading your API. My differ will only base64 and compare the values to check whether has changed or not.
Thanks in advance.

sorry, duplicated.

Don't worry about it. Did the other issue give you enough pointers to solve your problem?

Yes it indeed did.
ObjectDifferBuilder.startBuilding().differs().register((differDispatcher, nodeQueryService) -> new ByteArrayDiffer()).build();
private static final class ByteArrayDiffer implements Differ {
@Override
public boolean accepts(Class<?> aClass) {
    return byte[].class.equals(aClass);
}

@Override
public DiffNode compare(DiffNode diffNode, Instances instances) {
    final DiffNode beanNode = new DiffNode(diffNode, instances.getSourceAccessor(), instances.getType());

    if(instances.areNull() || instances.areSame())
        beanNode.setState(DiffNode.State.UNTOUCHED);
    else if(instances.hasBeenAdded())
        beanNode.setState(DiffNode.State.ADDED);
    else if(instances.hasBeenRemoved())
        beanNode.setState(DiffNode.State.REMOVED);
    else
        beanNode.setState(DiffNode.State.CHANGED);

    return beanNode;
}

}
I just need to check whether it has changed so the code worked like a charm!
Thanks!

On Jan 27, 2016, at 18:17, Daniel Bechler<EMAIL_ADDRESS>wrote:
Don't worry about it. Did the other issue give you enough pointers to solve your problem?
—
Reply to this email directly or view it on GitHub https://github.com/SQiShER/java-object-diff/issues/152#issuecomment-175834502.

Awesome! :smiley:


Apply the 'name' as the initial MOTD and normalize into legal container name

...otherwise the allowed characters for the name is limited to identifier characters

Maybe the default MOTD could be something like "A Minecraft server running ${name}"
The original name should probably be stored in a container label, such as mccy.name, for later reference (labels are useful)

Just a remind to document the label at https://github.com/itzg/minecraft-container-yard/wiki/Container-labels

#40 took care of most of this. Let's open a new issue for MOTD aspects.


Prune/Clean not working

The backups and the prune/clean are running with an hour+ of interval in-between, so I know for a fact that it's not actually a concurrency problem.
However, I noticed that for months, the prune/clean steps haven't been running:
Starting prune at 2023-02-08 04:00:00
unable to create lock in backend: repository is already locked by PID 56 on docker by root (UID 0, GID 0)
lock was created at 2023-02-05 19:58:26 (56h1m37.11065948s ago)
storage ID 7b519053
the `unlock` command can be used to remove stale locks

I thought maybe there's a stale lock? So I dropped a shell onto the prune container, and typed restic unlock. It worked:
# restic unlock
repository e3497749 opened successfully, password is correct
successfully removed locks

except when I try to run prune after, it complains about the exact same lock issue.
Something weird is going on with the backup/prune/clean containers and locks - it spans hours after a backup has finished, so it's not a matter of "race condition". Thoughts?

So did I get you right, you run restic unlock and then prune one after another inside the prune container and it complains that the repository is locked?

Yep - all in one shell session, too.

Which type of repository do you use? Sounds like a problem in the backend.

It is on backblaze b2

I don't have an idea about this right now.

No worries - hopefully someone else who has run into this eventually finds this thread and are able to provide some insight into this.

Chiming in.  I set up resticker to replace my former directly invoked restric process with b2 and am still proving it out. Today I noticed the three restic pods were in a restart loop because of a stale lock from 59 hours prior.
After clearing the lock I was unable to reproduce the full restart loop.  I did however observed the same as @JaneJeon reports in my prune logs.  And check my backup logs - this behaviour happens everytime.
Sharing here if it helps or if any further knowledge is out there.  Will report if I find anything more useful.
restic-backup  | Load(<lock/1f4932b5ef>, 0, 0) returned error, retrying after 552.330144ms: b2_download_file_by_name: 404: File with such name does not exist.
restic-backup  | Load(<lock/d890ce19ef>, 0, 0) returned error, retrying after 720.254544ms: b2_download_file_by_name: 404: File with such name does not exist.
restic-backup  | Load(<lock/1f4932b5ef>, 0, 0) returned error, retrying after 873.42004ms: b2_download_file_by_name: 404: File with such name does not exist.
restic-backup  | Load(<lock/d890ce19ef>, 0, 0) returned error, retrying after 703.28564ms: b2_download_file_by_name: 404: File with such name does not exist.
restic-backup  | Load(<lock/1f4932b5ef>, 0, 0) returned error, retrying after 1.040217184s: b2_download_file_by_name: 404: File with such name does not exist.
restic-backup  | Load(<lock/d890ce19ef>, 0, 0) returned error, retrying after 1.335175957s: b2_download_file_by_name: 404: File with such name does not exist.
restic-backup  | Load(<lock/1f4932b5ef>, 0, 0) returned error, retrying after 954.512469ms: b2_download_file_by_name: 404: File with such name does not exist.
restic-backup  | Load(<lock/d890ce19ef>, 0, 0) returned error, retrying after 1.107876242s: b2_download_file_by_name: 404: File with such name does not exist.
restic-backup  | Load(<lock/1f4932b5ef>, 0, 0) returned error, retrying after 1.511079094s: b2_download_file_by_name: 404: File with such name does not exist.
restic-backup  | Load(<lock/d890ce19ef>, 0, 0) returned error, retrying after 2.027308147s: b2_download_file_by_name: 404: File with such name does not exist.
restic-backup  | Load(<lock/1f4932b5ef>, 0, 0) returned error, retrying after 3.854635449s: b2_download_file_by_name: 404: File with such name does not exist.
restic-backup  | Load(<lock/d890ce19ef>, 0, 0) returned error, retrying after 4.987726727s: b2_download_file_by_name: 404: File with such name does not exist.
restic-backup  | Load(<lock/1f4932b5ef>, 0, 0) returned error, retrying after 4.067955962s: b2_download_file_by_name: 404: File with such name does not exist.
restic-backup  | Load(<lock/d890ce19ef>, 0, 0) returned error, retrying after 5.015617898s: b2_download_file_by_name: 404: File with such name does not exist.
restic-backup  | Load(<lock/1f4932b5ef>, 0, 0) returned error, retrying after 6.988645419s: b2_download_file_by_name: 404: File with such name does not exist.
restic-backup  | Load(<lock/d890ce19ef>, 0, 0) returned error, retrying after 8.277195667s: b2_download_file_by_name: 404: File with such name does not exist.
restic-backup  | Load(<lock/1f4932b5ef>, 0, 0) returned error, retrying after 10.034154425s: b2_download_file_by_name: 404: File with such name does not exist.
restic-backup  | Load(<lock/d890ce19ef>, 0, 0) returned error, retrying after 10.163166574s: b2_download_file_by_name: 404: File with such name does not exist.
restic-backup  | Load(<lock/1f4932b5ef>, 0, 0) returned error, retrying after 22.663987965s: b2_download_file_by_name: 404: File with such name does not exist.
restic-backup  | Load(<lock/d890ce19ef>, 0, 0) returned error, retrying after 13.811796615s: b2_download_file_by_name: 404: File with such name does not exist.
restic-backup  | Repository found.

restic-check  | Checking configured repository 'b2:namename:/restic' ...
restic-check  | Repository found.
restic-check  |
restic-check  |
restic-check  | Scheduling check job according to cron expression.
restic-check  | new cron: 0 0 6 * * *

restic-prune  | Checking configured repository 'b2:namename:/restic' ...
restic-prune  | Repository found.
restic-prune  |
restic-prune  |
restic-prune  | Executing prune job on startup ...
restic-prune  |
restic-prune  | Starting prune at 2023-04-30 16:53:00
restic-prune  | Fatal: unable to create lock in backend: repository is already locked by PID 8 on docker by root (UID 0, GID 0)
restic-prune  | lock was created at 2023-04-30 16:52:57 (12.004408814s ago)
restic-prune  | storage ID e25eb87e
restic-prune  |
restic-prune  |
restic-prune  | Scheduling prune job according to cron expression.
restic-prune  | new cron: 0 30 4 * * *


Project Maintained?

This project is really useful for a project of mine, but it seems like this isn't maintained anymore, and it doesn't seem to have much support for gifs or webp.
Any updates?

@CleverSource consider phrasing your request for a specific feature. e.g: GIF Support?

Apologies for not responding. You're ight, this project serves as an example worker and has not seen any activity in a long time. That said, we can certainly look at adding some features.
Image formats should be fairly easy to add, as they appear to be supported by the library used to do the resizing, i.e. https://docs.rs/image/latest/image/enum.ImageFormat.html. I cannot recall a specific reason for not adding these formats.

Hey @jupiter, 👋 Hope you're well. ☀
We are keen to deploy this ourselves 🚀
and extend it to cover a few other image formats including GIF and webp. 🖼
Would you accept a PR with the update? 🧑💻
Thanks. ❤

A PR would indeed be welcome.


Fixed bug where comments were getting added to the request

Found this bug when trying to create data to test the original sample processing views with

Has someone tried this out locally and confirmed that it now works?

Has someone tried this out locally and confirmed that it now works?

@khelwood I am going to merge this along with others and going to test all of it together. Is it ok?

Has someone tried this out locally and confirmed that it now works?

@khelwood I am going to merge this along with others and going to test all of it together. Is it ok?

Since it appears to be addressing a live bug, I am going to merge it into UAT instead of devel, so we can get it tested and released immediately.

Has someone tried this out locally and confirmed that it now works?

@khelwood I am going to merge this along with others and going to test all of it together. Is it ok?

Since it appears to be addressing a live bug, I am going to merge it into UAT instead of devel, so we can get it tested and released immediately.

Ok, I will test it and let you know

I think Ben's already tested it, so I am going to merge into uat and deploy to uat.

I think Ben's already tested it, so I am going to merge into uat and deploy to uat.

Ok  👍


Change event not fired when selecting hours

When clicking on dates the change event is fired, but when clicking on hours or am/pm, no event is fired. Can you suggest a solution to this?

Released in 2.0.2


Javascript

I guess start with an array variable of questions?

events listeners for buttons and the quiz choice

display none to keep only one box visible at a time

create elements for question and answer

math.random to choose a question to append child question and answer to the element

if statements for matching answer choice and correct answer

a time interval that subtracts time if the answer is wrong

counters for score

create element and get and set and apend to make highscore ranking li for ul

sort it from highest to lowest

added a array with objects {question, answer, and an options array} for javascript to later math random and pull from

add most of the event listeners, added variables, the buttons should work now. biggest issue i can't understand how to solve is the actual quiz part, clicking on the options doesn't produce a "correct" or "wrong" in the h3 elements, neither does it display the next randomly selected question. timer also doesn't reset if i click on another other button unless i let the timer run out go to the results box.

need to figure out how to make it a requirement to add initial before you can submit you score, how to display the scores sorted by highest, how to clear highscores

fixed issues in 2.2 the basic flow of the application now works mostly fine. still need to figure out how to solve issues in 2.3.

okay... i think i actually cheated. I got help from a friend who is a programmer. but honestly pair programing FTW.
solved problems listed in 2.3
i think because i added prevent default in order to make the submit button work. i had to make an if statement for an alert if users didn't enter their initials to prevent it to still submit.


The state of the project (spoiler: everything is fine)

Hi, I'm the maintainer of the program.
The project is alive and will be alive. It got much more attention for the last half a year which is great. But it has its downsides, I'm not really able to handle all the issues and pull requests on my own, because basically people create new issues like every single day and I have other things to do including my job/family.
I'd be grateful if someone helped people in issues because there are some generic questions/problems which don't require a lot of knowledge of existing codebase or internal logic of the program.
We could also think about updating README and addressing frequent questions/problems.
I could also grant some rights to push code, close issues, assign labels to someone who is willing to help me with the project.
Thanks!

Hi, we find this an incredibly useful tool at my employer - I'm more than happy to start helping a hand answering stuff.
I'll add it to my list of things check in on :)

I have the same sentiment as ITHedgeHog.  I'm ramping up usage of Mark in my team and it's very needed.  Confluence is awful for all the reasons you say in your blog post - I cannot believe we ask engineers to use Confluence, it is unuseable (by engineers, if you love WYSIWYG, enjoy) for the thing it was designed to do, but people still use it for some reason.
But, until I can convince my company to get off of Confluence, I have to use it.  Definitely want to keep this tool alive and growing, and happy to contribute engineering time to it!

I would sponsor this for a few dollars a month.
awesome project!


GoogleDefaultCreds: support new-style xDS cluster names

Why doesn't this change match against the authority? Isn't that the point of federation?

We could do that if there's a reason to.  My thought was that it wouldn't really buy us much, because it doesn't seem likely that this will accidentally be used with some other authority.  Also, hard-coding it here would make it harder to switch to some other authority in the future if we decided that we wanted to do so, although that's admittedly also not very likely.  So I just erred on the side of doing less work.
If you can think of a compelling reason to add this, I'm certainly open to discussion.

@markdroth, @dfawley, and I discussed this today and discovered the "/envoy.config.cluster.v3.Cluster/google_cfe_" path in the control plane was specified for one specific authority. This had been my assumption and Mark had the opposite assumption/memory (this specific path is independent of the authority, within the context of this one control plane). Mark was closer to this work and had read some documents I didn't know existed, so there were surprises for everyone.
With the knowledge the path was specified for one authority by the control plane, then we all were basically on the same page of being fine both ways but leaning toward checking the authority. So we're going to check the authority.

I've sent #29901 to add the authority check.


Master to Live for SharePoint Migration Tool and a number of skype updates.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/docs-conceptual/sharepoint-pnp/sharepoint-pnp-cmdlets.md
:white_check_mark:Succeeded
View (sharepoint-ps)

skype/skype-ps/skype/Get-CsImConfiguration.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/New-CsImConfiguration.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Remove-CsImConfiguration.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Set-CsImConfiguration.md
:white_check_mark:Succeeded
View (skype-ps)

spmt/docfx.json
:white_check_mark:Succeeded

spmt/docs-conceptual/index.md
:white_check_mark:Succeeded

spmt/docs-conceptual/intro.md
:white_check_mark:Succeeded

spmt/docs-conceptual/overview.md
:white_check_mark:Succeeded

spmt/docs-conceptual/toc.yml
:white_check_mark:Succeeded

spmt/mapping/monikerMapping.json
:white_check_mark:Succeeded

spmt/spmt-ps/spmt/Add-SPMTTask.md
:white_check_mark:Succeeded

spmt/spmt-ps/spmt/Get-SPMTMigration.md
:white_check_mark:Succeeded

spmt/spmt-ps/spmt/Register-SPMTMigration.md
:white_check_mark:Succeeded

spmt/spmt-ps/spmt/Remove-SPMTTask.md
:white_check_mark:Succeeded

spmt/spmt-ps/spmt/Show-SPMTMigration.md
:white_check_mark:Succeeded

spmt/spmt-ps/spmt/Start-SPMTMigration.md
:white_check_mark:Succeeded

spmt/spmt-ps/spmt/Stop-SPMTMigration.md
:white_check_mark:Succeeded

spmt/spmt-ps/spmt/Unregister-SPMTMigration.md
:white_check_mark:Succeeded

spmt/spmt-ps/spmt/spmt.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/docs-conceptual/index.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/docs-conceptual/sharepoint-pnp/sharepoint-pnp-cmdlets.md
:white_check_mark:Succeeded
View (sharepoint-ps)

skype/skype-ps/skype/Get-CsImConfiguration.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/New-CsImConfiguration.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Remove-CsImConfiguration.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Set-CsImConfiguration.md
:white_check_mark:Succeeded
View (skype-ps)

spmt/docfx.json
:white_check_mark:Succeeded

spmt/docs-conceptual/index.md
:white_check_mark:Succeeded

spmt/docs-conceptual/intro.md
:white_check_mark:Succeeded

spmt/docs-conceptual/overview.md
:white_check_mark:Succeeded

spmt/docs-conceptual/toc.yml
:white_check_mark:Succeeded

spmt/mapping/monikerMapping.json
:white_check_mark:Succeeded

spmt/spmt-ps/spmt/Add-SPMTTask.md
:white_check_mark:Succeeded

spmt/spmt-ps/spmt/Get-SPMTMigration.md
:white_check_mark:Succeeded

spmt/spmt-ps/spmt/Register-SPMTMigration.md
:white_check_mark:Succeeded

spmt/spmt-ps/spmt/Remove-SPMTTask.md
:white_check_mark:Succeeded

spmt/spmt-ps/spmt/Show-SPMTMigration.md
:white_check_mark:Succeeded

spmt/spmt-ps/spmt/Start-SPMTMigration.md
:white_check_mark:Succeeded

spmt/spmt-ps/spmt/Stop-SPMTMigration.md
:white_check_mark:Succeeded

spmt/spmt-ps/spmt/Unregister-SPMTMigration.md
:white_check_mark:Succeeded

spmt/spmt-ps/spmt/spmt.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/advanced-threat-protection/Get-MailDetailATPReport.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/advanced-threat-protection/Get-MailTrafficATPReport.md
:white_check_mark:Succeeded
View (exchange-ps)

sharepoint/docs-conceptual/index.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/docs-conceptual/sharepoint-pnp/sharepoint-pnp-cmdlets.md
:white_check_mark:Succeeded
View (sharepoint-ps)

skype/skype-ps/skype/Get-CsImConfiguration.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/New-CsImConfiguration.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Remove-CsImConfiguration.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Set-CsImConfiguration.md
:white_check_mark:Succeeded
View (skype-ps)

spmt/docfx.json
:white_check_mark:Succeeded

spmt/docs-conceptual/index.md
:white_check_mark:Succeeded

spmt/docs-conceptual/intro.md
:white_check_mark:Succeeded

spmt/docs-conceptual/overview.md
:white_check_mark:Succeeded

spmt/docs-conceptual/toc.yml
:white_check_mark:Succeeded

spmt/mapping/monikerMapping.json
:white_check_mark:Succeeded

spmt/spmt-ps/spmt/Add-SPMTTask.md
:white_check_mark:Succeeded

spmt/spmt-ps/spmt/Get-SPMTMigration.md
:white_check_mark:Succeeded

spmt/spmt-ps/spmt/Register-SPMTMigration.md
:white_check_mark:Succeeded

spmt/spmt-ps/spmt/Remove-SPMTTask.md
:white_check_mark:Succeeded

spmt/spmt-ps/spmt/Show-SPMTMigration.md
:white_check_mark:Succeeded

spmt/spmt-ps/spmt/Start-SPMTMigration.md
:white_check_mark:Succeeded

spmt/spmt-ps/spmt/Stop-SPMTMigration.md
:white_check_mark:Succeeded

spmt/spmt-ps/spmt/Unregister-SPMTMigration.md
:white_check_mark:Succeeded

spmt/spmt-ps/spmt/spmt.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


CLOUDSTACK-8499: Fixuireload

This PR is for issue https://issues.apache.org/jira/i#browse/CLOUDSTACK-8499
This affects all versions of cloudstack and can be cleanly applied to all supported branches.

I think it was added to burst caching in browsers when developing the UI. The fix looks good to me and definitely fixes the issue in a production deployment but we still need to figure out a way (perhaps in maven client-ui packaging) to insert ?t=timestamp.

Hi bhaisaab,
A UI developer should be able to just press CTRL+F5 and always get the fresh version of files.. no need for something complicated... I just did that a few times yesterday with no issues while testing changes in .css

:D merging


Move bootstrap test to its own configuration.

org.scalajs.testsuite.library.LinkingInfoTest appears to fail in FullOptStage.
Otherwise looks good :)


fix(version): update fabric8-stack-analysis-ui to 0.0.12

Superseded by new version 0.0.12 #558

There were the following issues with your Pull Request

Commit:<PHONE_NUMBER>43478f289e18b37f42edf98c685401
Your subject line is longer than 50 characters

Guidelines are available at https://github.com/fabric8io/fabric8-ui

This message was auto-generated by https://gitcop.com

:tada: This issue has been resolved in version 4.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Dynamically start DHCP services on networks

Rather than starting DHCP services on a network when it is created, they
should be started dynamically when domains are created with interfaces
on the network with requested addresses (or hostnames).
Because domains can be created in parallel, some read/write access to
the libvirt network definition must be synced to prevent an error that
occurs when a DHCP range is added twice to a network.

Coverage decreased (-0.3%) to 21.649% when pulling fd8f73a2b2b451b653689636cc4e5af1c57ea44a on rawlinp:dynamic_dhcp into 447994ba9e20be752a5edeb5e35c7546157eef28 on dmacvicar:master.

Coverage decreased (-0.6%) to 46.003% when pulling 9ba92c0ef3e63f41c599d952fb2ca61254674cfe on rawlinp:dynamic_dhcp into 68630335a6c23ab544661b8ef9c6692df886fd7d on dmacvicar:master.

@inercia, in regards to your earlier comment about making the simplest case harder, how does the user find the IP that would be allocated to their VM via DHCP? I think it might fly by in the output of terraform apply, but isn't it easier to just request a static IP from the CIDR they already configured for the network rather than read through the command output? Is there an easier way to find the IP to use in that case?
Would it be too complex to detect the case where the user hasn't specified any addresses on any network_interface and enable DHCP on the first interface's network just to make the VM usable?

Coverage increased (+0.7%) to 47.285% when pulling 0c11247cc629c4c89cf35307be0cb6fb2483b504 on rawlinp:dynamic_dhcp into 68630335a6c23ab544661b8ef9c6692df886fd7d on dmacvicar:master.

@rawlinp So in my view the best behaviour from a user point of view would be:

if the user creates a network with no addresses at all, the network is created with a DHCP serving in a default range.
if the user creates a network with addresses = [something], the network is created with a DHCP serving in that range.
if the user creates a network with no addresses = [], the network is created with no DHCP.

The first case is the default case I was talking about, the simplest case that most users will use with DHCP and NAT. What do you think? Not sure this is what we currently have...

@rawlinp Regarding getting the IP address for a VM in your cluster, all the IPs are saved in the tfstate file, but you can also print it from your .tf file with something like:
output "ip_something" {
  value = "${libvirt_domain.something.network_interface.0.addresses.0}"
}

if !provided { enableDHCP(DEFAULT_RANGES) }
What would DEFAULT_RANGES be? How can we be sure it doesn't clash with any other private CIDR in use on the user's system?
if provided && len(addresses) > 0 { enableDHCP(addresses) }
This is what I'm trying to avoid by only enabling DHCP when a VM has specified addresses on one of its NICs. With multiple NICs on networks with DHCP enabled, a VM can end up with multiple default routes (a default route for every NIC on a DHCP-enabled network). This way, a VM can specify addresses on only a single NIC, and the rest can be configured manually if they so choose.
if provided && len(addresses) == 0 { disableDHCP() }
This kind of ties in with Case 1 above, because without any provided addresses on a network, the user cannot use NAT functionality or DHCP because we won't know what IP to use on the virtual bridge (we'd have to access the user's networking stack and compute a non-clashing CIDR to use).

Being able to bring up a VM without specifying its network's addresses seems like a nice idea for simplicity's sake, but I don't see it being able to use NAT/DHCP unless it's at least provided its own CIDR for the network. If no CIDR is provided, I don't think it's within the scope of the terraform libvirt provider to fully access the user's networking stack to attempt to compute a non-clashing default CIDR to use.

@rawlinp Ok, maybe you are right: to try to compute a non-clashing range would require some work (and some assumptions). So I think we could require addresses from users, but please update the documentation clearly stating that, including some examples on how to create a ready-to-use network as well. Most users will start with the DHCP & NAT case (and will probably not go further than that), so that should be well documented.
I also think you have some pending changes regarding where to put the Locks when enabling DHCP and so...

@rawlinp This needs to be rebased and resolve some conflicts...

@inercia I don't think I will have the time to finish working on this patch. If you or someone else still find it valuable, please feel free to take it over.

What's the problem with starting DHCP when you create the network? It seems to me the right moment.
Also, I need to be able to create networks without DHCP and with fixed IP addresses. If I'm not wrong, this patch would make it impossible. So I would vote very strongly against this patch.
Sorry in advance if I misunderstood something.

Closing as I can no longer work on this PR, and it's vastly out of date.


Upgrade to Vue v3

The output bundle size is 174KB, compared to 153KB of master (with webpack v5).


Remove all empty translations

Some of the challenges in the seed files have translations with no text. These will cause errors when campers have selected these languages. These look like:
      "translations": {
        "de": {
          "description": [],
          "title": ""
        }
      }

In this case, we would need to delete
        "de": {
          "description": [],
          "title": ""
        }

We need to go through and delete all of these empty translations from seed files, and leave the empty "translation": {}

@QuincyLarson I would like to take on this task.

Can I help this seems easy, I'm a beginner.

@envy7 Sure, be sure to follow the CONTRIBUTING.md. Happy coding.

@tmashuang thanks a lot, I'll get to work on it right away.

@QuincyLarson there are a lot of seed files should I do all the changes in a single commit, or in multiple commits and then squash them? Does it make a difference.

@envy7 you can do it in anyway as it suits you, but make sure when you have opened the pull request it has one commit.
Also, refer the contributing guide, most things have been answered there to guide you!
Thanks and happy fixing!


issue-131: write log messages to stderr instead of stdout

Log messages should go to stderr. This way others can use safety inside their own scripts without getting stdout all mixed up.

Also partially solves #133

A partial fix is better than none! ;-)


Document query

Add documents to explain how the v0.6 query interface will work, especially covering issues on raw vs. custom queries, and how to handle inter-contract queries without violating the actor model.
This could use some review, not just on the explanation, but the design in general. Is this the proper model? Can we improve this?
This will close https://github.com/confio/cosmwasm/issues/72 and it is good to read up on that discuss as well (after this text) when debating pros and cons of the architecure.

Good questions.
I havent thought too much of non contract modules, but the address section on did was to go the way of providing a unified naming system for both. So the dispatcher could send it either direction. I should specify this more.
Also, raw queries refer to state on the blockchain and can come with proofs. Custom queries process the data so the answer is a transformed version of the proven data. Proving proofs for that is much more complex and out of scope for a long time. Rough idea is to provide merkle proofs for all keys read by the computation and have the client run the same wasm with same input to validate result. Clearly not light weight. For any mid term design, proofs are only raw queires and if you want more, do the processing in the client


Replace isatty with atty

isatty  is deprecated, the author suggests to use atty. And isatty::stdin_isatty method not implemented on windows.
Tested on windows.

@lewiszlw
Hi, thanks for the contribution.
There are many features that arrow_cli may lack compared with bendsql.
If you are willing to use and improve arrow_cli, you can take a look at bendsql  and port some useful features here.

I tried bendsql. It needs server supporting version function and system table.

I tried bendsql. It needs server supporting version function and system table. I just want a simple interactive flight sql client.

Yes，I mean that some pure features  may be introduced into this crate.


Claude Prompt Caching!

Issue
More info here:
https://x.com/alexalbert__/status/1823751966893465630
hope you guys implement it fast.
Version and model info
No response

Came here to ask about this same feature, but I'm 2 hours late.

and i am 3 hours late!

I was thinking we could have an /add-docs or /add-static or some other way to add information that would work for being cacheable (that would have no effect on models that don't support caching).
@paul-gauthier do you want any help building this? I'm interested in contributing but unsure about how you like to go about "feature" contribs vs bug fixes.

So it looks like the prompt caching implementation is done by adding a cache_control parameter.

Prompt Caching references the entire prompt - tools, system, and messages up to and including the block designated with cache_control.

It would be really useful if Aider had the concept of "forking" or creating a "checkpoint" in the conversation, so the conversation could be restored to that point in time, with the same exact context. There's plenty of reasons why this is useful, but now with prompt caching it seems like a no-brainer. I think this would even be useful for models that don't have prompt caching.
Maybe you could tag a checkpoint to enable prompt caching on it. Something along the lines of (being extra verbose for clarity) /cache-from-next-message or /create-checkpoint {regular message}.
Prompt caching is 25% more expensive on the first call, but 90% cheaper for followup calls, so being able to reuse different "checkpoints" would be very valuable.

Aider leverages litellm, which has support for Prompt Caching in the newest update https://github.com/BerriAI/litellm/releases/tag/v1.43.13-stable
Hope this makes Aider support prompt caching easier

How we are going with this? Spending like $10/day is getting expansive.
https://x.com/mckaywrigley/status/1824148683711189033

Also, Gemini has Context Caching too - https://ai.google.dev/gemini-api/docs/caching

The main branch has experimental support for prompt caching with Claude and DeepSeek models. It has not yet been well tested, so try it at your own risk!
You can get it by installing the latest version from github:
python -m pip install --upgrade git+https://github.com/paul-gauthier/aider.git

To try prompt caching, run:
aider --cache-prompts

Currently, it will only print out cache hit statistics if you also do --no-stream. It should be caching even during streaming, but without showing you stats.
Tokens: 755 sent, 2,103 cached, 460 received. Cost: $0.0092 message, $0.01 session.

AFAICT, if we just naively send updated queries to to llm, we'll bust the cache every time the query changes.
Am I missing something? I see there are some arguments passed to the Coder constructor but was wondering if we could be a little more specific with caching things intentionally like docs, etc.

Aider orders the chat that gets sent to the LLM to try and maximize caching. Read only files from /read and the repo-map will be cached along with the system prompt.

This is really amazing thanks so much!

I recommend installing with
pipx install --suffix=@dev git+https://github.com/paul-gauthier/aider.git
then you can run this dev version with
aider@dev
but still have access to the regular aider in case of any problems.

This is hot, we can use a dev now, going to auto setup it on my daily pull
now. 😁
Enviado do Gmail para celular
On Tue, Aug 20, 2024 at 5:25 PM lockmeister @.***>
wrote:

I recommend installing with
pipx install @.*** git+https://github.com/paul-gauthier/aider.git
then you can run this dev version with
@.***
but still have access to the regular aider in case of any problems.
—
Reply to this email directly, view it on GitHub
https://github.com/paul-gauthier/aider/issues/1086#issuecomment-2299703320,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AET26Z76UCCEU6L6PMXH5GDZSOQ5LAVCNFSM6AAAAABMQWZVVKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOJZG4YDGMZSGA
.
You are receiving this because you authored the thread.Message ID:
@.***>

what does the 5 minute timer for the cache mean for users? Do we need some kind of "mouse jiggler" keep-alive to ensure that we don't pay more after a coffee break?

I'm going to close this issue for now, but feel free to add a comment here and I will re-open or file a new issue any time.

what does the 5 minute timer for the cache mean for users? Do we need some kind of "mouse jiggler" keep-alive to ensure that we don't pay more after a coffee break?

i'm too dumb to do the calculations but i think you'd end up eating up all the savings you get from prompt caching by forcing a dummy request automatically after 4 minutes of inactivity... right?

This doc may be helpful:
https://aider.chat/docs/usage/caching.html


added configurable order numbers

Bagisto currently exposes actual orders table ID column to the client and there is no option to configure a custom order number for a specific store.
This is a neccessary feature since many inventory/backend systems require order numbers in a certain format. This is feature is available in WooCommerce and Magento to name a few.
Current change adds an option to configure a custom order number in the backend that consists of

a prefix,

order number that contains the order id and has user-set length

a suffix.

If none of the options are set, then the number is set as previously.
The order number is then visible in admin order index, order single-template view and for customer in success page and order history.

I amended the generateIncrementId() function to include configuration defaults, that will let the increment_id generation to fallback on the original order id increment

Fixed the typo


Rename Focus.js and related references

From https://github.com/phetsims/scenery/issues/1223 @zepumph said

REVIEW: I think that "focus" in terms of non HTML DOM focus is very confusing, can we talk about this more?

I can see this. Focus.js is now being used for more than DOM focus as we have multiple types of states we want to signify as "active" with the HighlightOverlay. These currently include

DOM focus
Which Node is currently under the Mouse/Pointer
Which Node is being spoken about through Voicing

I think for each of these "focus" is a good word, but it is overloaded with "DOM" terminology.
The class manages the Trail and Display containing an active Node. Brainstorming names:
ActiveContainer
Active
Activated
ActivatedInfo
ActiveDetails
Attention
Attended
"Focus" is still the best name I can think of. Maybe it should just be more clear that in scenery "Focus" is for more than DOM. Over to @zepumph to review or discuss further.
EDIT: More words:
FocusInfo
FocalData
FocalInfo
Focal
Fixation
Observation (overlap with Observable, bad)

This is a month out, and I feel like it would be fastest and easiest to discuss in person with @jessegreenberg.

@jessegreenberg and I are ready to close this. "Focus" is not just about DOM Focus, but instead focus of a variety of inputs onto the Display. See https://github.com/phetsims/scenery/issues/1236 for a refactor that helps make this more clear. Closing

eopenning. While discussing other issues related to focus with @zepumph it became clear that the overloaded term for focus was very confusing. We brainstormed the following terms:
ActiveNode
ActiveNodeData
ActiveTrailData
ActiveTrailEssence
HighlightedElement
ActivatedTrail
ActiveTrailData
ActiveDetails
ActiveData
ActivatedData
UserActivatedData
FocalPoint
FocalData
TrailDetails
We like ActivatedData best. We want to keep "Trail" and "Node" and "Display" out of the term because they are specific to the implementation. We are concerned about how generic this term is, but can't think of anything better. A name that is perhaps too general is better than having a name that is confusing.
I will make this change.
Reminder to self: This will be a PhET-iO API change so after this is done we will need to go through and update everything. @zepumph gave me this command to do so: grunt generate-phet-io-api --simList=../perennial/data/phet-io-api-stable.

Coming back to this I think Focus is still the best name, and renaming to ActivatedData is not an improvement. I would rather not rename now but expand our definition of focus. @zepumph do you feel strongly?

I certainly don't feel like ActivatedData is the way forward :).
Let's close this issue, but work toward thinking of focus as not just related to keyboard/browser focus. As we talk about it and write code about it.


Can I use Mediapipe on IOS with Java

I have already designed an app on Android with mediapipe and am wondering if I can use the code structure in Java for iOS as well using multi-os engine.
If yes, can someone provide details how to I can achieve that?
Thank you!

Hi @V-m1r, You can Start building IOS applications following this steps, We don't have java examples of IOS applications . If you are looking for Multiplatform support you can try using Javascript solutions to render on browsers from our JS solutions.  There are also unity plugin where you can build applications of MediaPipe in many platform which is shared by the community members.


ci: scope github token for actions

Signed-off-by: Anish Ramasekar<EMAIL_ADDRESS>
Reason for Change:

By default GITHUB_TOKEN has write-all access. Scoping permissions to least required access.
https://docs.github.com/en/actions/security-guides/automatic-token-authentication#modifying-the-permissions-for-the-github_token
Once this PR is merged, we should also update the action > workflow permissions to read only (Read repository contents permission).

Requirements

[x] squashed commits
[ ] included documentation
[ ] added unit tests and e2e tests (if applicable).

Issue Fixed:

Does this change contain code from or inspired by another project?

[ ] Yes
[ ] No

If "Yes," did you notify that project's maintainers and provide attribution?
Special Notes for Reviewers:

WIP


Color picker & live preview in Config page

Description
provide a color picker in config page so users can pick a color visually
display of a live preview of chosen colors on a small mockup of AWS navigation and account menu would be even great
Other information
No response
Acknowledge

[ ] I may be able to implement this feature request
[ ] This feature might incur a breaking change

Thank you for your request!
In future, this feature will be enable to import config from remote. No rich GUI editor feature is intentioned.
https://github.com/xhiroga/aws-peacock-management-console/issues/44


When moving mail a second time, the folder list scroll position is off by 1

Expected behavior
If you're going to move messages to specific folders, the last used destination folder should always be displayed as first entry.
Actual behavior
If you're going to move messages to specific folders, always the folder which follows the last used destination folder is displayed as first entry.
Steps to reproduce

Beside the Inbox you've create several destination folders in which you want to archive your email.
Lets assume that these folders are called a-folder, b-folder, c-folder ...
Now you mark one or more messages, touch the folder icon and select the folder named a-folder as destination folder.
Again you mark one or more messages, touch the folder icon and want to select the previously used destination folder named a-folder. Unfortunately the first displayed folder on the selection list is now b-folder instead of a-folder. You have to scroll on folder up to select a-folder again.
This behavior can be reproduced every time you want to move messages to folders.

Environment
K-9 Mail version: 5.0.10
Android version: 5.1.1
Account type (IMAP, POP3, WebDAV/Exchange): IMAP

I can confirm this.  I've seen this behavior for about 2 years now.

I can can take a stab at this.

I got Android Studio installed, compiled K-9, and ran it on an AVD.  I'll take a look at ChhoseFolder.java tonight.  I suspect the position counter is adding one for the current folder, say inbox, which is not displayed because it's not a valid destination folder.


added a working html CSS and JavaScript site

added the fully working site with the required files in the folder , the site contains the listed features

site contains beautiful and efficient CSS implementation
used JavaScript for the smooth scrolling feature
used the google map api for the map at the footer for locating the office.

Kindly give deployed link
https://github.com/shubham7668/WebDev#rules
& make sure to follow rules

ok will do it

hello there i have uploaded the working link and have followed all the rules for contribution thank you for your help

is it yours?

I am marking this pr as invalid as i am unable to find your proof of work.

If I am mistaken then feel free to comment


Missing functions in the UDF overview page

Describe the bug
The UDF overview page is missing certain functions that are available in the sidebar. I was trying to find time_to_int64 and couldn't use cmd+f on the page because it literally was not there.
https://docs.px.dev/reference/pxl/udf/
However the page does exist:
https://docs.px.dev/reference/pxl/udf/int64_to_time/
To Reproduce

Open https://docs.px.dev/reference/pxl/udf/
cmd+f time_to_int64 and see that you don't get any results on the page.
Expected behavior
the time_to_int64 function should be rendered on the page. Any updates to UDFs should be rendered on this page.

Screenshots

**Additional context**
Add any other context about the problem here.

@philkuz - looks like you fixed this in https://phab.corp.pixielabs.ai/D11600?

This was fixed by this action.


Read The Docs contains un-sercure configuration example

I found and error within http://graphite-api.readthedocs.org/en/latest/deployment.html#gunicorn-nginx nginx example that contains critical vulnerability.
Example states that nginx should try finding local file, if not - pass to the proxy. It should be other way around. Everything goes to proxy unless it specifies for specific data. Following this suggestion anyone could just go to http://graphite/graphite/local_settings.py and thus expose possibly critical settings. Sure you should have quite a few things wrong with the installation for this to be exploitable, but still it spreads insecure deployment practice.
Original:
server {
    server_name graph;
    listen 80 default;
    root /srv/www/graphite;

    location / {
        try_files $uri @graphite;
    }

    location @graphite {
        proxy_pass http://graphite;
    }
}

It Should be:
server {
    server_name graph;
    listen 80 default;
    root /srv/www/graphite;

    location /content {
        alias /srv/www/content;
    }

    location /  {
        proxy_pass http://graphite;
    }
}

I've tried to look for docs to edit but the only thing that I found was on https://github.com/graphite-project/graphite-web/blob/d3c889b235e81635858b573c38725a28b7c3e144/docs/config-local-settings.rst witch is correct. So maybe just the read the docs should be updated?

https://github.com/brutasse/graphite-api is a separate project.

And in this installation guide /srv/www/graphite contains no exploitable Python / config file.

Ah yes. My Bad. Too much time on this had been spend so everything mixed up together at some point for me. Ignore and continue on.


[config] Enables swagger UI by default

CATEGORY

[ ] Bug Fix
[X] Enhancement (new features, refinement)
[ ] Refactor
[ ] Add tests
[ ] Build / Development Environment
[ ] Documentation

SUMMARY
This is a proposal to enable FAB Swagger UI by default. To access the UI, user's need to have a specific permission can show on SwaggerView, if set by default superset init will create the permission for swagger and associate it to the Admin role.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[X] Introduces new feature or API
[ ] Removes existing feature or API

REVIEWERS

Codecov Report

Merging #8732 into master will increase coverage by 0.04%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #8732      +/-   ##
==========================================
+ Coverage   65.54%   65.58%   +0.04%
==========================================
  Files         482      482
  Lines       24090    24090
  Branches     2763     2763
==========================================
+ Hits        15789    15799      +10
+ Misses       8121     8111      -10
  Partials      180      180

Impacted Files
Coverage Δ

superset/config.py
89.86% <100%> (ø)
:arrow_up:

superset/db_engine_specs/postgres.py
83.33% <0%> (+27.77%)
:arrow_up:

Continue to review full report at Codecov.

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

I'm OK with this, given that it's behind a permission.


SWC minifier broken due to a commit

https://github.com/privatenumber/minification-benchmarks/commit/4e48512ae24fc6f55db8ae73988153586fee367a

I checked out the parent commit and updated @swc/core to v1.2.189 and it works

Fixed in https://github.com/privatenumber/minification-benchmarks/commit/042d2168dc9a4054b9a52efe5466df553e63f682
Thanks for the heads up


Watcher::store_triggered_appointment should be called as an async task

The current codebase has an edge case where, if bitcoind is unreachable while an already triggered appointment is added, and store_triggered_appointment is called as result, the request will hang and end up timing out but be accepted. However, the user will not be handed the receipt.
This could be fixed making store_triggered_appointment be called as an async task, so the receipt can be handed straightaway. This is surprisingly not as straightforward as it sounds, and implies dealing with lifetimes in a way I don't fully understand (at least with my current knowledge of Rust 😅).

Hey @sr-gi, I have a fix which turns store_triggered_appointment into an async function like you suggested but I am running into a design issue that I'd like your input on.  As you know, store_triggered_appointment ends up calling the carrier's hang_until_bitcoind_reachable which blocks the current thread until bitcoind_reachable is true.  This is problematic if store_triggered_appointment is an async function because the std::sync::Condvar used to notify the carrier can only be used between threads, not asynchronous calls.  This problem is solvable but would require changing the carrier's bitcoind_reachable type to something thats async compatible and including either async_std or using tokio::sync::Notify to signal the carrier when bitcoind is reachable.
So from here I see two immediate options:

Continue down the async path
Spawn a thread from within add_appointment which calls store_triggered_appointment.  This option might raise some questions pertaining to thread resource management to limit thread creation overhead (think thread pools).

I have some other thoughts rolling around but wanted to start a discussion about how to move forward.  Certainly open to criticism and other alternatives!

Hey @carterian8, happy to see someone is up to help with this.
My approach when I tried to solve this was going the async route, but I couldn't find a proper way to do so why my knowledge of Rust async. I don't think it'll be a big deal having to replace std::sync::Condvar for something that works with async as long as the functionality is preserved (I'd lean towards using something from tokio to prevent having to add an additional dependency).
Have in mind that bitcoind_reachable is used by multiple components of the tower, so it may require a bigger change that just update the Carrier. I'm curious what's the issue with async and Condvar. Mainly asking since currently it is being use in some async methods (like here or here), so that may be wrong.

Hey @sr-gi, happy to help!
First regarding std::sync::Condvar, I'm basing my conclusions off of tests I've written within the watcher.rs test module and this open tokio issue.  What I've found is that the std::sync::Condvar for bitcoind_reachable does not get notified after calling notify_all() or notify_one() from the tests.  Just want to open up my testing/thought process to others to ensure I've not made any errors myself.
Assuming std::sync::Condvar is not compatible with tokio, and since tokio does not implement a Condvar of its own, I think tokio::sync::Notify could be a suitable replacement.  The idea being any component finding bitcoind_reachable = false will wait to be notified of a change in state.  I'm in the process of implementing this but have found async bubbling up to things like the responder's chain::Listen trait.  This is problematic because async functions are not allowed in traits.  My point is, there could be some non-trivial restructuring to turn the Condvar.wait()s into something async.  I'm certainly up for taking it on but wanted to make you aware and continue the discussion.
In the meantime I'll charge forward and continue to brainstorm cleaner solutions.

Replacing std::sync::Condvar for something else that works with async is certainly an option. As long as we can achieve the same functionality (and the design does not become too intricate), I'm happy with it.

Can't we make store_triggered_appointment add to the responder trackers and let the responder handle that tracker on the next block feed?
This has a downside though: we are skipping the opportunity to publish this tx (if it's valid) in the very next block.

Can't we make store_triggered_appointment add to the responder trackers and let the responder handle that tracker on the next block feed?
This has a downside though: we are skipping the opportunity to publish this tx (if it's valid) in the very next block.

Or if we can make handle_breach an aysnc method, we can make it a timeout based logic:
If we don't get a response from the backend before timeout T (say 100ms) we add a tracker to the responder trackers and hand the user the receipt early and let the handle breach method retry on the next connected block.


nixos/nslcd: pull in network-online.target

Motivation for this change
ldap queries require a network connection. It might be reasonable to backport this change to 20.03 as well...
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

How does the system behave while nslcd isn't yet active, but lookups happen?

In manual cases (like user logins) there isn't a problem: a query fails and then tries again. I ran into issues when having services (managed by systemd) depend on ldap users, though.

Yeah, I'd assume these things to be somewhat scary, but probably nothing we can fix here (at least as long as we don't have socket activation on such things ;-) )

I don't have an LDAP installation, so I'll leave it to @Mic92 to test and merge this.

We discussed on IRC that it might be better to test if users.ldap.bind.policy = "hard_init"; might not solve network issues in a more general way.


Fix brittleness issue, resolves #4 and #5

@devinrader that brittleness issue was a really nice catch, I really appreciate your feedback. IMO after the changes made this solution looks way more better.
Would you mind having a look on this one, if it is worthy left a LGTM or merge it please.

@acamino - Unfortunately, after merging the first two PRs, this one has a merge conflict now. Can you fix this real quick?


Fix averaging phasor coordinates and related issues

Description
This PR proposes several new features and breaking changes to fix issues raised in #99 and #122:

Add normalize parameter to phasor_from_signal function.
This is a trivial change but might be useful for experimenting with filter algorithms.

Add phasor_normalize function.
Use to normalize phasor coordinates returned by phasor_from_signal(..., normalize=False) or provided by user.
Do we need a phasor_unnormalize function, maybe for internal use? It's probably a good idea that no other public function should require unnormalized coordinates. Un/re-normalization can be transparently handled internally if necessary.

Rename phasor_filter function to phasor_filter_median:

Remove method parameter.
Add use_scipy parameter to be used instead of method='median_scipy'.
Add mean parameter to detect presence of harmonic axis in real and imag and make the API like other functions.
Remove 'axes' from kwargs. It's redundant to skip_axis.
Return mean input unchanged to make API match other functions.
Remove private _median_filter_scipy and _median_filter functions. Their implementation moved into phasor_filter_median.

Support multiple harmonics in phasor_threshold:

Propagate NaNs in mean or any real and imag harmonics to mean and all harmonics in real and imag.
Add detect_harmonics option to skip detecting multiple harmonics and revert to pure ufunc behavior.

Fix phasor_center function:

Add mean parameter to fix averaging phasor coordinates, detect presence of harmonic axis in real and imag, and make API similar with other functions.
Use phasor_threshold for method='mean' to propagate NaNs among arrays to ensure the averages are calculated over the same elements. Should this be done also for method='median'?
Return filtered mean array, required to correctly interpret averaged phasor coordinates and make the API like other functions.
Change skip_axis refer to axes in mean, not real. The harmonic axis, if present, is automatically excluded.

Fix phasor_calibrate function:

Add reference_mean parameter, which is required by the phasor_center function and used to automatically detect presence of harmonics.
Change skip_axis refer to axes in reference_mean, not reference_real. The harmonic axis, if present, is automatically excluded.

Remove formal docstrings from private _mean and _median functions.
They are repetitive, distracting, and not really needed to understand the functions.

Adjust tests and tutorials for the above changes and features:
Multi-harmonic support in phasor_calibrate and phasor_filter_median make the tutorials cleaner.

Closes #122.
Checklist

[x] The pull request title and description are concise.
[x] Related issues are linked in the description.
[x] New dependencies are explained.
[x] The source code and documentation can be distributed under the MIT license.
[x] The source code adheres to code standards.
[x] New classes, functions, and features are thoroughly tested.
[x] New, user-facing classes, functions, and features are documented.
[x] New features are covered in tutorials.
[x] No files other than source code, documentation, and project settings are added to the repository.

Hi @cgohlke, sorry I wasn't able to respond before, it has been crazy this past week with the workshop.
I can review this in detail tomorrow or Friday since today is the last day of the course.

Hi @cgohlke! I think all of this are great improvements to the workflow. Particularly all the tutorials are much clear now that all functions can handle harmonics. Regarding this, I also see that in the basic workflow for phasor analysis, the plot_phasor is the only one that doesn't support multiple harmonics and each harmonic has to be plotter separately. Do you think at some point it would make sense to modify the implementation of plot_phasor, or better yet the plot methods in PhasorPlot, to detect and plot harmonics separately? Plotting multiple harmonics in the same plot doesn't make much sense by default and it would be nice if it would support this detection of harmonics and plotting all separately. But I don't know how much work needs to be put into that.
I tested all the modifications and the results for multiple-harmonic analysis are the same as before, so everything seems to work as expected. As I said before, now the workflow is much easier not having to keep track of the harmonic axis and all related.

Do we need a phasor_unnormalize function, maybe for internal use? It's probably a good idea that no other public function should require unnormalized coordinates. Un/re-normalization can be transparently handled internally if necessary.

I was going to propose this exactly. If it's not much trouble, it would be very nice to have this also because we are going to need this for other functions. I think it's a good idea to keep it private, at least for now.

Do you think at some point it would make sense to modify the implementation of plot_phasor, or better yet the plot methods in PhasorPlot, to detect and plot harmonics separately?

PhasorPlot is intrinsically single harmonic. plot_phasor could be changed to plot a grid of single-harmonic phasor plots. Currently it can't automatically detect the presence of harmonics. We would need to add a parameter and/or pass mean. Anyway, that would go into a separate PR.

If it's not much trouble, it would be very nice to have this also because we are going to need this for other functions. I think it's a good idea to keep it private, at least for now.

I'll add a private function. We'll see how useful it is. I would expect the un/normalization to be done more efficiently "inline", like in the _mean function, or directly in a Cython/numpy ufunc.

Use phasor_threshold for method='mean' to propagate NaNs among arrays to ensure the averages are calculated over the same elements. Should this be done also for method='median'?

Any thoughts on this?

Use phasor_threshold for method='mean' to propagate NaNs among arrays to ensure the averages are calculated over the same elements. Should this be done also for method='median'?

I think using phasor_threshold for propagating NaNs is a clever way to do this. Also I think this is not computationally too heavy so it should add too much to the function. I liked how this is done. I think it can be useful also for the median method, but maybe I am missing something on why it should be only used with the mean method.

I think it can be useful also for the median method, but maybe I am missing something on why it should be only used with the mean method.

How about adding a threshold : bool | None = None parameter such that the user can explicitly decide if necessary (or not, for example, if the input is already thresholded)? By default, it would be enabled for method='mean'.

How about adding a threshold : bool | None = None parameter such that the user can explicitly decide

This can be a good idea to optimize for cases not needed. Although maybe there is a more intuitive name for the parameter? Because although it uses the phasor_threshold the idea is to propagate the NaNs, so maybe something more related to the use of the parameter? But threshold can work as long as the description fits.

By default, it would be enabled for method='mean'

I think it's a good idea to keep it by default, but I am missing something on why it should not be default for all methods but only for mean.

Although maybe there is a more intuitive name for the parameter?

Agreed. How about:
nan_safe: bool = True,

"""
nan_safe : bool, optional
    Ensure `method` is applied to same elements of input arrays.
    By default, distribute NaNs among input arrays before applying
    `method`.
"""

Other suggestions?

Why it should not be default for all methods but only for mean.

Because it is not strictly necessary for median? And in another issue you argued not using thresholds before median filters (let's get back to that in Monday's meeting). Anyway, I'm in favor of applying nan_safe=True as a default.

nan_safe: bool = True,
"""
nan_safe : bool, optional
Ensure method is applied to same elements of input arrays.
By default, distribute NaNs among input arrays before applying
method.
"""

I like this very much! This is clear that is NaN related and the description matches.

Because it is not strictly necessary for median? And in another issue you argued not using thresholds before median filters (let's get back to that in Monday's meeting).

 I think I am also inclined to using it by default. I don't know the reason behind applying the threshold before filtering, it's a good idea to discuss and try to find the reason.

If it's not much trouble, it would be very nice to have this also because we are going to need this for other functions.

I'll leave this out of this PR. Normalization is not completely reversible. If one really needs "un-normalized" coordinates, calculate them from the signal.


⚠ Bofrost.at has degraded performance

In ba6ea6c, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2829 ms

Resolved: Bofrost.at performance has improved in a0333a1.


Added start:dev command to npm scripts

I noticed there was the option to start electron with the dev command line argument. So I figured it would make sense to add an entry for that in the package.json scripts section for this.

Codecov Report

Merging #2423 (052f0b8) into develop (1fd506f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #2423   +/-   ##
========================================
  Coverage    39.42%   39.42%
========================================
  Files           20       20
  Lines         1608     1608
========================================
  Hits           634      634
  Misses         974      974

Continue to review full report at Codecov.

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

Codecov Report

Merging #2423 (052f0b8) into develop (1fd506f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #2423   +/-   ##
========================================
  Coverage    39.42%   39.42%
========================================
  Files           20       20
  Lines         1608     1608
========================================
  Hits           634      634
  Misses         974      974

Continue to review full report at Codecov.

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

Thanks! Would you mind sending a PR to the docs repo as well?

Thanks! Would you mind sending a PR to the docs repo as well?

Thanks! Would you mind sending a PR to the docs repo as well?

Will do. Which section do you think would it fit in @MichMich ?

Thanks! Would you mind sending a PR to the docs repo as well?

Will do. Which section do you think would it fit in @MichMich ?

No idea. 😂

No idea. 😂

I just found out it's already there:
https://docs.magicmirror.builders/getting-started/installation.html#usage

I just found out it's already there:
https://docs.magicmirror.builders/getting-started/installation.html#usage


[Enhancement] Rename "get:code-challenges" to just "get:code-challenge"

Is your enhancement request related to a problem? Please describe.
The current naming convention is confusing, as this script/api only returns 1 code-challenge.
Describe the solution you'd like
Rename the script and docs from get:code-challenges to just get:code-challenge
Additional context
This technically is a breaking change and will need to be tied with a new major version.

Closed with above pr, in v1


Fixed channel order RGB bug in the inference Code

Motivation
The inference section of the tutorial jupyter notebook takes the input image as RGB. This affects the performance of the detector as the detector is trained with BGR format. This effect is not apparent in the provided ballon dataset but can have a huge impact depending on the dataset.
Modification
RGB bug in the inference Code is fixed by creating another variable for image. The model is then sent the image in BGR format and the visualizer is sent the image in RGB format.
BC-breaking (Optional)
Does the modification introduce changes that break the backward-compatibility of the downstream repos?
No sure
Checklist

Pre-commit or other linting tools are used to fix the potential lint issues.
The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
If the modification has potential influence on downstream projects, this PR should be tested with downstream projects, like MMDet or MMPreTrain.
The documentation has been modified accordingly, like docstring or example tutorials.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


[Feature]: Deep link Release Notes and show in Package Manager

Related Problem
Release Notes seems like a good place to put important information about changes in the current release but this isn't very visible.  It doesn't appear when the package is selected in the VS Package Explorer, and I can't deeplink to it.
There's already a hash link for the tab, but it doesn't activate the tab: https://www.nuget.org/packages/SerratedSharp.SerratedJQ.Lite#releasenotes-tab
The Elevator Pitch
In documentation I find myself putting things like "For current compatibility information visit Nuget link and click the right Release Notes tab" which has unnecessary exposition rather than what would be more straightforward "For current compatibility information see Nuget link".
To accomplish this, make the hash link deep link directly to Release Notes tab, so that we can link directly to the tab from other places:
https://www.nuget.org/packages/SerratedSharp.SerratedJQ.Lite#releasenotes-tab
Also consider showing the Release Notes as a section in the VS Package Manager when the package is selected.  I.e. in the panel where Description, Version, License, etc. is shown.
Typically Release Notes would contain information about breaking changes which would warrant it being much more visible.
Additional Context and Details
No response

Hello, thank you for opening this issue! I believe this issue is a duplicate of https://github.com/NuGet/NuGetGallery/issues/8822 and will addressed by this fix: https://github.com/NuGet/NuGetGallery/pull/8860.
I'll go ahead and close this one off for now but please let us know if you have follow up questions or concerns!


Writing a docx file results in text repeating four times

Following the examples in the readme to write a simple docx file, the text is repeated four times.
Sample main file:
use docx::Docx;
use docx::document::Para;

fn main() {
    let mut docx = Docx::default();
    let mut para = Para::default();
    para.text("Lorem Ipsum\n");
    docx.insert_para(para);
    docx.write_file("demo.docx").unwrap();
}

With a Cargo.toml file with nothing other than docx as a dependency, this creates a demo.docx file containing

Lorem Ipsum
Lorem Ipsum
Lorem Ipsum
Lorem Ipsum

Running with rust 1.38 on both RHEL 7 and Ubuntu 18.04.

Sorry for keeping you waiting!
I just released a new version 0.2 which rewrites the whole proc macro and replaces quick-xml with xmlparser. I'm been working on it for a while and it's finally done.
You can upgrade to 0.2 now and I've verified that this bug is fixed on 0.2. Feel free to re-open this issue if the bug still exists.


Wrong class is reported as required data class in Memory Trainer Project

The Memory Trainer Project (Part 6) requires the user to implement a Stat class. This Stat class needs to be a data class Stat. However, if I make it a class Stat the error after hitting Check says: The StatService should be a data class!. But it is the Stat that should be a data class. Not the StatService.
I fixed it here https://github.com/VeMike/kotlin-onboarding-object-oriented-programming/commit/c1d1d349f9886bfdd35146e9c29a35371658aaa8 and can do a PR for it.

@VeMike thank you a lot!