add Duplicate Post option to post action menu

Add a new menu item to the "three dot" menu in the PostHeader called "Duplicate".
This will open up the Create/Edit post modal (called the PostEditor component) and pre-populate it with all the data of the previous post, with the title changed to "Copy of [original post title truncated to fit into 80 characters with Copy Of at the front]", so that the user can edit the title, dates, content, etc. before they post it.
There's a few ways to accomplish this. All the data could be passed to the CreateModel and then down to the PostEditor via url parameters. We already have this option for lat/lng when a post is created by clicking on the map. But i don't think that would work for a very long description. Another option is to add a new "action" to the PosteEditor component. Right now we have 2 actions "create" and "edit". You would add a "copy" action. So then the URL you would open to duplicate a post with an id of 12345 would be /groups/xxxx/post/12345/copy. This would allow the PostEditor to pull the data from the store for the existing post. The main difference is that the Save button would change to say Duplicate, and when you click it it would call createPost instead of updatePost. That's probably the way to go.
One thing to test is whether this works ok for setting up the location object (make sure the new post appears on the map at the right location) and for the link preview (the link preview shows up in the post).
I will get you an icon for the Duplicate item soon!

Just so I know the terms, is this an example of a PostCard?

Yes!On Feb 8, 2024, at 7:13 PM, Kevin Triplett @.***> wrote:
Just so I know the terms, is this an example of a PostCard?
Screenshot.2024-02-08.at.9.12.38.PM.png (view on web)
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

That’s also the name of the component. You an inspect and see it in the DOM too. On Feb 8, 2024, at 7:13 PM, Kevin Triplett @.***> wrote:
Just so I know the terms, is this an example of a PostCard?
Screenshot.2024-02-08.at.9.12.38.PM.png (view on web)
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

I agree, I was going to suggest opening it for edit -- but I think you added a good point, that it not be created in the backend until the user submits it from this modal. That way, if the user changes their mind, no harm, the DB is not touched.

BTW, do I or will someone else add Spanish translation for "Duplicate Post"?

Ok I updated the description with ideas for how to do this!
It would be great for you to add the translation. This means adding the text to en.js and es.js and using the t() function to translate it as you will see all across the app

BTW, do I or will someone else add Spanish translation for "Duplicate Post"?

A google translate will work for now. We'll try to get the translations periodically reviewed, to ensure accuracy

My friend (from Mexico) suggested "Duplicar" for the translation.
I need to get on a zoom call with someone to discuss the process for this, I'm having difficulty understanding the store and how to trigger an event in React (editPost in this case)

Kevin, great to hear from you. I just got back to California to find my
campervan has been ransacked. So a bit distracted by that.
I'm somewhat assuming that one would need to navigate to the /create modal
for posts. There are numerous instances you can look at of useRouter, and
you can search for createPath to see how the path is created (which could
also be used for a regular anchor Link button). As for the duplicated data:
probably easiest to stuff it into localStorage (if you search
localStorage you can see other instances of how we have used
localStorage), and check that whenever the create component is opened. Of
course, one will want to avoid stale data in the localStorage, or the next
time they go and create a post it will be there as well. Perhaps they
should only check the local storage on a conditional query-string param in
the url?
-Tom
On Wed, 20 Mar 2024 at 06:59, Kevin Triplett @.***>
wrote:

My friend (from Mexico) suggested "Duplicar" for the translation.
I need to get on a zoom call with someone to discuss the process for
this, I'm having difficulty understanding the store and how to trigger an
event in React (editPost in this case)
—
Reply to this email directly, view it on GitHub
https://github.com/Hylozoic/hylo-evo/issues/1585#issuecomment-2009640427,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABHT4AU5W5EB7FD3O6HPH63YZGI3XAVCNFSM6AAAAABC6WWZYGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMBZGY2DANBSG4
.
You are receiving this because you commented.Message ID:
@.***>

Thanks for writing Tom - dag! Sorry to read about your van, that's my fear everytime I leave my truck camper somewhere.
I can respond, now that I have a pseudo plan to propose: can I co-opt the built-in edit action by adding this test: If "duplicate" then modify the title and spoof the action as create? Might that work, rather than create a new action?

The edit button (in the menu on the postDetail view) goes to the URL
prior-path/post/<postId>/edit
I'd add an action here (not sure if we are only allowing duplication of
posts by the authors or what but that is handled by the onClick being a
real function or not)
const dropdownItems = filter([
{ icon: 'Pin', label: pinned ? t('Unpin') : t('Pin'), onClick: pinPost },
{ icon: 'Edit', label: t('Edit'), onClick: editPost },
{ icon: 'Copy', label: t('Copy Link'), onClick: copyLink },
{ icon: 'Flag', label: t('Flag'), onClick: this.flagPostFunc() },
{ icon: 'Trash', label: t('Delete'), onClick: deletePost ? () => deletePost(
t('Are you sure you want to delete this post?')) : undefined, red: true },
{ icon: 'Trash', label: t('Remove From Group'), onClick: removePost, red:
true }
], item => isFunction(item.onClick))
And then creating an function in PostHeader.connector.js that uses
the createPostUrl
from navigation.js. This function has the arguments (opts = {},
querystringParams = {}) so you can add querystrings super easy.
Whatever we add to the queryString (probably something like
?duplicate=<postId>), it needs to change the behavior of the component
that renders from the createPostUrl (the component is PostEditor). Probably
in the PostEditor.connector.js, you'll want to check for whatever
queryString you put and use the postId from that to look up the data for
that postId, and then pass that into the PostEditor. Then you'll have the
desired endstate; the post editor open with a bunch of the fields
pre-filled with data from the prior post.
Does that make sense?
On Mon, 1 Apr 2024 at 18:09, Kevin Triplett @.***>
wrote:

Thanks for writing Tom - dag! Sorry to read about your van, that's my fear
everytime I leave my truck camper somewhere.
I can respond, now that I have a pseudo plan to propose: can I co-opt the
built-in edit action by adding this test: If "duplicate" then modify the
title and spoof the action as create? Might that work, rather than create a
new action?
—
Reply to this email directly, view it on GitHub
https://github.com/Hylozoic/hylo-evo/issues/1585#issuecomment-2030898162,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABHT4AUVBADUA4RLFFO2RXTY3IAOPAVCNFSM6AAAAABC6WWZYGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMZQHA4TQMJWGI
.
You are receiving this because you commented.Message ID:
@.***>

Thanks, Tom! That jives quite nicely with what Tibet and I independently crafted this afternoon, so it's good to get confirmation. It works (yay!), now I need to add a bunch of tests 😅.
@tibetsprague agreed to add the 'duplicate' icon to the master branch, so when my PR is merged it should seamlessly pick it up.
Regarding authorization for duplicating, does Hylo allow restriction of post creation? If not, then anyone should be able to duplicate a post, it seems. They could do this now, manually.

@tibetsprague and @thomasgwatson -- one side effect is that the Post button is not enabled until the title is changed.
Advantage: forces the user to change the title (which they likely will)
Disadvantage: it forces the user to figure out why the Post button is not enabled. They poke around until they discover that the title has to be changed, which might be frustrating / frictionfull
Thoughts? I haven't poked around the code to find out what enables the Post button but I suspect it's triggered by the title field change/keydown/press event.

Thanks Tom, that did the trick -- I'll issue the PR in a few minutes


CAMEL-20544 - Camel-AWS-Bedrock-Agent: Add a consumer for polling the…

… status of ingestion or more ingestions, don't log body and headers in integration tests
Description

Target

[x] I checked that the commit is targeting the correct branch (note that Camel 3 uses camel-3.x, whereas Camel 4 uses the main branch)

Tracking

[ ] If this is a large change, bug fix, or code improvement, I checked there is a JIRA issue filed for the change (usually before you start working on it).

Apache Camel coding standards and style

[x] I checked that each commit in the pull request has a meaningful subject line and body.

[x] I have run mvn clean install -DskipTests locally and I have committed all auto-generated changes

/component-test aws


Efficient Hive UDF calling framework

now we have ffi-based UDF failbacking supported in the latest versions. UDAF/UDTF should be supported in the future.


Backport/backport 177 to 2.0

Signed-off-by: Ankit Jain<EMAIL_ADDRESS>(cherry picked from commit 6ae2bf526d4df27f8232d001bf282136df48a681)
Is your feature request related to a problem? Please provide an existing Issue # , or describe.
#176 #177
Check List

[x] New functionality includes testing.

[x] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #181 (f58ed3c) into 2.0 (8a39c12) will decrease coverage by 0.05%.
The diff coverage is n/a.

:exclamation: Current head f58ed3c differs from pull request most recent head 2321dd1. Consider uploading reports for the commit 2321dd1 to get more accurate results

@@             Coverage Diff              @@
##                2.0     #181      +/-   ##
============================================
- Coverage     71.44%   71.38%   -0.06%
+ Complexity     2939     2937       -2
============================================
  Files           376      376
  Lines         18769    18769
  Branches       1445     1445
============================================
- Hits          13409    13398      -11
- Misses         4770     4779       +9
- Partials        590      592       +2

Impacted Files
Coverage Δ

...nalyzer/rca/net/handler/PublishRequestHandler.java
77.55% <0.00%> (-10.21%)
:arrow_down:

...nceanalyzer/rca/samplers/MetricsDBFileSampler.java
75.00% <0.00%> (-9.10%)
:arrow_down:

.../org/opensearch/performanceanalyzer/core/Util.java
62.50% <0.00%> (-8.34%)
:arrow_down:

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 8a39c12...2321dd1. Read the comment docs.


Multithreading Support on predict_many_sk and predict_many_v2

Added multithreading/pool support on the two main generation functions.  This allows for a linear speedup of x/NUM_THREADS as defined in the local_settings.py.
Updated readme.MD to reference the new required setting - May be worth adding as a default to the main settings.py?
Running this in a docker environment with PostgreSQL on a SSD.
predict_many_v2 is still running in the background right now, once it's done, I'll provide some time stats.
Machine Stats:
2x Intel(R) Xeon(R) CPU           L5639  @ 2.13GHz
72Gb ram
Runtimes:
Running with 24 threads:

predict_many_sk:

real 2m 7.850s
user 27m 25.829s
sys 1m 41.492s

Non-threaded:

predict_many_sk:

real 32m 2.854s
user 22m 5.785s
sys 0m 44.857s

merging tomorrow. speak now if comments, problems... etc etc

/Tip 0.1 BTC

Maybe NUM_THREADS could default to multiprocessing.cpu_count() or sth like this https://github.com/thefab/cpu_cores/blob/master/cpu_cores/linux.py

I rebased this on my Docker branhc and it works. I can see the two python threads spinning the two CPU on the crappy DO server.

I'd considered moving this to an automated number of cpus, but considering that this thing can and will thrash your entire system if you set it too high, and/or have other things running, my system was choking a bit at 24, because the DB and everything else was shared on those 24 cores.

Set the default worker thread count to 1, this will keep things from breaking for people who don't read the readme!

thanks, merging

Issue Status: 1. Open 2. Started 3. Submitted 4. Done

This issue now has a funding of 0.1 ETH (69.21 USD @ $692.1/ETH) attached to it.

If you would like to work on this issue you can 'start work' on the Gitcoin Issue Details page.
Questions? Checkout Gitcoin Help or the Gitcoin Slack
$0 more funded OSS Work available on the Gitcoin Issue Explorer

Issue Status: 1. Open 2. Started 3. Submitted 4. Done

This issue now has a funding of 0.2 ETH (138.42 USD @ $692.1/ETH) attached to it.

If you would like to work on this issue you can 'start work' on the Gitcoin Issue Details page.
Questions? Checkout Gitcoin Help or the Gitcoin Slack
$0 more funded OSS Work available on the Gitcoin Issue Explorer


kogito with resteasy reactive - java.lang.ClassNotFoundException: io.quarkus.resteasy.runtime.QuarkusRestPathTemplate

Describe the bug
While trying out the kogito quickstart with quarkus 2.8.0.Final and kogito 1.20.0.Final it throws error
java.lang.ClassNotFoundException: io.quarkus.resteasy.runtime.QuarkusRestPathTemplate
Attaching a sample code base need help to fix the bug.
Expected behavior
should work seamlessly as per the jira https://github.com/quarkusio/quarkus/issues/19868 and https://github.com/quarkusio/quarkus/issues/20846 but new error with quarkus 2.8.0.Final and kogito 1.20.0.Final
Actual behavior
java.lang.ClassNotFoundException: io.quarkus.resteasy.runtime.QuarkusRestPathTemplate
kogitoreactive.zip
e
How to Reproduce?
Download the attached zip and run the program same issue in Windows and Mac not tested in Linux.
Output of uname -a or ver
Darwin Debabratas-MacBook-Pro.local 21.3.0 Darwin Kernel Version 21.3.0: Wed Jan  5 21:37:58 PST 2022; root:xnu-8019.80.24~20/RELEASE_X86_64 x86_64
Output of java -version
openjdk version "17.0.2" 2022-01-18 OpenJDK Runtime Environment GraalVM CE <IP_ADDRESS> (build 17.0.2+8-jvmci-22.0-b05) OpenJDK 64-Bit Server VM GraalVM CE <IP_ADDRESS> (build 17.0.2+8-jvmci-22.0-b05, mixed mode, sharing)
GraalVM version (if different from Java)
No response
Kogito version or git rev (or at least Quarkus version if you are using Kogito via Quarkus platform BOM)
1.20.0.Final
Build tool (ie. output of mvnw --version or gradlew --version)
Maven home: /usr/local/Cellar/maven/3.8.5/libexec Java version: 18, vendor: Homebrew, runtime: /usr/local/Cellar/openjdk/18/libexec/openjdk.jdk/Contents/Home Default locale: en_IN, platform encoding: UTF-8 OS name: "mac os x", version: "12.2.1", arch: "x86_64", family: "mac"
Additional information
refer attached
kogitoreactive.zip
.

@evacchi Per previous changes reactive support was added to kogito in 1.17.0.Final but not sure if there is anything additional to add for the same in the program.

@evacchi It seems the issue is with the following svg generation library.
<dependency> <groupId>org.kie.kogito</groupId> <artifactId>kogito-addons-quarkus-process-svg</artifactId> <exclusions> <exclusion> <groupId>io.quarkus</groupId> <artifactId>quarkus-resteasy</artifactId> </exclusion> </exclusions> <dependency>

removing the svg extension works fine.

@nmirasch who can we request on svg module to support reactive rest easy. Quarkus have defaulted to resteasy reactive now. And kogito-svg addon does not support the request. @evacchi is there any traction for this ask.

any update on svg generation with reactive

@nmirasch @evacchi can you please have some traction. Its not yet working. We need full reactive app but cant do due to svg thing incompatibility.


Add write-up for CM09: Maria

 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.


Not working at all for me (Safari 11, macOS 10.13)

I downloaded the latest binary release and ran it. The extension didn't appear in Safari's prefs until I quit and relaunched Safari; then I enabled it, and now the app says the extension is enabled.
However, I've tried to use the Yubikey at Yubico, Google and Github, and all say "you need Chrome".
And in the Safari JS Console I've entered window.u2f and the result is undefined.
Is this a compatibility issue?

Yubico FIDO U2F Security Key
Safari 11.0.2 (136<IP_ADDRESS>.3)
macOS 10.13.2 Beta (17C76a)
MacBook Pro (2013, retina display)

Did you remember to switch Safari's User Agent "personality" over to that of "Google Chrome"? If not, try this:

Enable Safari's Develop menu.
Select Develop > User Agent > Google Chrome — macOS.
Navigate to the Yubico's Test your U2F device site and register your token. You can skip this step if you've already done this successfully in Google Chrome.
Switch to the Login tab and login with the credentials used above. You should see a Performing U2F action alert indicating that you should "touch the flashing U2F device now". Touch the blinking token, and then you should see the Authentication successful! page.

Many sites that support FIDO U2F make a blanket assumption that only Google Chrome has this capability, and simply don't provide the U2F option if your browser "doesn't smell like Chrome". So, for now, you just need to "lie" to the website to coerce its cooperation.
Interestingly, even GitHub doesn't quite work with the workaround above. Before posting this comment, I logged into GitHub with Safari, and after landing on the "Two-factor authentication" screen, the js-u2f-error area indicated that "Something went really wrong". Selecting Develop > User Agent > Google Chrome — Windows refreshed the page, and my token started blinking, and I was able to complete the 2FA part of my login. UPDATE: I am always getting the "Something went really wrong" indication after landing on the 2FA screen, but switching the UA from Google Chrome — macOS to Google Chrome — Windows (or vice versa) allows U2F to proceed, but this only seems to happen on GitHub. Huh, go figure…
And FWIW…

Safari-FIDO-U2F v1.1b3 extension
Safari 11.0.2
macOS 10.12.6
MacBook Pro Retina Mid-2015 (MacBookPro11,4)

I confirm that this plugin doesn't work at all with Safari on macOS High Sierra 10.13.1, Safari 11.0.1 (are you sure you're running Safari 11.0.2?), Xcode-9.1, current master Safari-FIDO-U2F.
I can install binary, or recompile on my own, no real difference.
Symptoms: after I set User Agent to "Chrome -> macOS", insert the token, and open the Yubico U2F demo page, I get the correct prompt from the page (and the remote sends the correct challenge block) - but the token does not blink. U2F authentication does not happen.
Doing the same (same token, same site, same system) with Firefox or Chrome works fine. Must be something incompatible between this plugin and High Sierra...

Also, no matter what I do, Safari FIDO U2F.app shows this:

I think I have a fix for the demo page, but there still seems to be an issue with Github, which I suspect might be partly at their end.
I hope to have a bit more time to work on this in the new year, so will attempt to get a new, working, build uploaded.


[Webcrawler] Allow urls up to 512 chars

Description
That's it. Following this error thread
Risk
na
Deploy

run migration
connectors

nice catch

:pray:


refactor: Refactor & Modernised QuadTree module.

Description:
Refactored files, Adding Obj-C uni tests file for reference:

GQTBounds
GQTPoint
GQTPointQuadTreeItem
GQTPointQuadTree
GQTPointQuadTreeChild
GQTPointQuadTreeTest

Refactor:

Refactor & Modernised QuadTree module.
Refactor & Modernised GQTPointQuadTreeTest unit test.

Validations:

All the tests of GQTPointQuadTreeTest are passing.

Opened a new PR


Explain project better in README #75

Fix #75
@IvoLimmen does this meet your expectations? Anything missing? Going too deep somewhere?

Lovely!


Adding a dev stream to  the generated YAML.

Fixes #103
Creates generated/installer/ibm-spectrum-scale-csi-operator-dev.yaml which points to the dev version of the release.

This doesn't affect any running operator code, it's just an alternate config file pointing to the dev stream.

Hi John, what is ETA for this PR to be merged in the 1.1.0,  Test team is blocked for this hence test cycle is getting delayed too?

And one more clarity, whether we will have this dev yaml file for every release going forward for testing only and for customer this will not be published ?

The Dev stream is  something  that any user consuming the dev stream can ingest. Right now  we're pointing at the 1.1.0 dev stream, when we bump versions  the version that this config  will consume will be bumped too.

This depends on #110 to be merged, because I'm not merging this change with a  breaking travis build.


chore(main): release 0.16.3

:robot: I have created a release beep boop
0.16.3 (2024-07-20)
Bug Fixes

ec2-postgresql-connect installing correctly (05d53d9)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v0.16.3
:sunflower:


Remove NSLog from FirebaseAuth

Description
Several instances of NSLog have been recently introduced in FirebaseAuth. These should be converted to FIRLogger so customers can filter the log messages as they like.
Reproducing the issue
FirebaseAuth/Sources/MultiFactor/TOTP/FIRTOTPSecret.m:        NSLog(@"Cannot access openURL:options:completionHandler: method");
FirebaseAuth/Sources/MultiFactor/TOTP/FIRTOTPSecret.m:      NSLog(@"URL cannot be opened");
FirebaseAuth/Sources/MultiFactor/TOTP/FIRTOTPSecret.m:    NSLog(@"sharedApplication cannot be accessed");
FirebaseAuth/Sources/Utilities/FIRAuthRecaptchaVerifier.m:                                       NSLog(@"reCAPTCHA verification faled because reCAPTCHA SDK "
FirebaseAuth/Sources/Utilities/FIRAuthRecaptchaVerifier.m:                                     NSLog(@"reCAPTCHA verification faled because reCAPTCHA SDK "
FirebaseAuth/Sources/Utilities/FIRAuthRecaptchaVerifier.m:                                   NSLog(@"reCAPTCHA verification succeeded.");
FirebaseAuth/Sources/Utilities/FIRAuthRecaptchaVerifier.m:                    NSLog(@"reCAPTCHA config retrieval failed.");
FirebaseAuth/Sources/Utilities/FIRAuthRecaptchaVerifier.m:                  NSLog(@"reCAPTCHA config retrieval succeeded.");
FirebaseAuth/Sources/Utilities/FIRAuthRecaptchaVerifier.m:                NSLog(@"reCAPTCHA token retrieval succeeded.");
FirebaseAuth/Sources/Utilities/FIRAuthRecaptchaVerifier.m:                NSLog(@"reCAPTCHA token retrieval failed. NO_RECAPTCHA sent as the fake code.");
Firebase SDK Version
10.16.0
Xcode Version
14.3
Installation Method
N/A
Firebase Product(s)
Authentication
Targeted Platforms
iOS
Relevant Log Output
No response
If using Swift Package Manager, the project's Package.resolved

Expand Package.resolved snippet

Replace this line with the contents of your Package.resolved.

If using CocoaPods, the project's Podfile.lock

Expand Podfile.lock snippet

Replace this line with the contents of your Podfile.lock!

Can be addressed earlier, but is staged in the auth-swift branch for Firebase 11.


Ghost ffmpeg instance

Hello, I m using NMS to relay more than 500 sources streams to 100 users (friends and family) and I have some issues.
I m using only relay functions defined with this conf:
`
var config = {
rtmp: {
port: 1935,
chunk_size: 60000,
gop_cache: true,
ping: 60,
ping_timeout: 30
},
http: {
port: 8000,
allow_origin: '*'
},
relay: {
ffmpeg: ffmpeg_path,
tasks: [
{
app: 'live',
mode: 'pull',
edge: 'http://<IP_ADDRESS>:8888'
}
]
}
};
`
It appears in some time ffmpeg doesn't close and publisher still alive with 0 subscribers.
How can I "kill" ghosted instances ?
Tahnx a lot for your job !

There will be such unexpected problems.
I will solve them in later versions.

Maybe you can try this one.
Comment and uncomment , in the node_relay_session.js
  end() {
    this.ffmpeg_exec.kill('SIGINT');
    //this.ffmpeg_exec.stdin.write('q');
  }

Ok i try it

I have this error in log maybe it can help you:
2018-4-9 12:01:58 [ERROR] Unknown command { cmd: 'FCSubscribe' }


test: fix race condition in multithreaded test

The main thread's epilog closed the file descriptor without waiting for
the worker to shut down, making it sporadically fail with a EBADF error.
Fixes: https://github.com/libuv/libuv/issues/248
R=@saghul

LGTM!


[Feature Request] Label Expressions - Traversing parents

Hello!
Is it possible to traverse parents using Label Expressions?
I'd like to add information about view name (Context), parent folder name (1 Preliminary) and its parent folder name (PROJ A) to get the project name into label.

$ ???
$vfolder{name}
$view{name}
Thank you!
David

Hi, not sure that's its possible but :
you can add a property "project" in the "1 Preliminary" which has value "PROJ A" and use it like
$vfolder{property:project}

For this, jArchi is your friend, a script which update a property on the selected elements : https://gist.github.com/davidsara/0bad09ad4d148b9cbef940b54f4df2e3

Thank you, I will try!


Evaluation on MTEB benchmark

The examples you show on the /examples folder add some instructions and also uses logistic regressionis this the way that gives the results in paper？ I try using the official evaluation by mteb package and the results are lower than that in the paper.

Hi @ylwangy,
Yes, we added instructions similar to that shown in the examples folder. We used Instructor's code for this. The entry point is evaluate_model.py We changed the instructions here.

okay, I'll check for this. It would be great if a full evaluation code (for all subsets w/ the full instructions) can be shared.

We are working on it, stay tuned! :)

Hi @vaibhavad, thanks for the great work! I am following this issue and curious whether there are any updates on the full evaluation code on MTEB?

It's impossible to reproduce score using McGill-NLP/LLM2Vec-Meta-Llama-3-8B-Instruct-mntp-supervised with Instruction on the paper and the code that you mentioned. How can I reproduce the results?

@bzantium @ShengYun-Peng  - I am working to integrate LLM2Vec model evaluation directly into the MTEB package - https://github.com/embeddings-benchmark/mteb/pull/926. As soon as it is merged, I'll update the LLM2Vec repository on how to evaluate using mteb package.

Thank you all for your patience. I have added the code and documentation to evaluate models on MTEB, you can find it here.

Feel free to re-open if you face any issues with MTEB evaluation.

Hi @vaibhavad,
I’ve been trying to reproduce your work using the recently published MTEB evaluation script. However, the results of the tested subtasks using the Mistral-7b-Instruct-v2-nmtp-unsupervised model show discrepancies compared to the results reported in Table 11 of your paper. Specifically, the performance on most tasks is lower than what was reported.
I have observed that when using bf16 for model inference, parameters like batch_size significantly affect the outcomes. Could you please provide a more detailed and feasible tutorial to help reproduce the results as reported in the paper?
Thank you!

hello thanks for awesome works and implementation!
I am trying to reproduce the experiments in the original paper. However, i am facing some errors with mteb evaluation process.
I run the exact code that you have added in here. But some error occured about task_to_instructions argument.

I find that the task_to_instructions argument is inserted into the sentence_transformers library using mteb by this his line.
However, I think sentence_transformers doesn't use the task_to_instructions argument which makes the error.
Do you know is there any possible way to solve this error?
The libraries I am using is as below:
llm2vec==0.2.2
sentence-transformers==3.0.1
transformers==4.40.2
torch==2.4.0
mteb==1.12.60


Add Rails 5 Auth0 Post

Hello
Based off the original post here. How do I include my post's images to this blog?
CC: @diegopoza

@amingilani Right now you can just upload your images to Github and include the URL provided by Github.com in the markdown

@unicodeveloper thanks. This pull request is a WIP, i just realized that my markdown wasn't kramdown compatible.

All done. Please review.


[BUG]

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
spotless-maven-plugin check error when run local maven build test, due we has run npm install package in home, but the installed package also has the md file.
                    <markdown>
                        <includes>
                            <include>home/**/*.md</include>
                        </includes>
                        <flexmark />
                        <replaceRegex>
                            <name>Markdown Formatter</name>
                            <searchRegex>(^-*\n$)([\s\S]*?)(-+$)</searchRegex>
                            <replacement>---$2---</replacement>
                        </replaceRegex>
                    </markdown>

[ERROR] Failed to execute goal com.diffplug.spotless:spotless-maven-plugin:2.40.0:check (spotless-check) on project hertzbeat: The following files had format violations:
[ERROR]     home/node_modules/is-plain-obj/readme.md
[ERROR]         @@ -4,14 +4,12 @@
[ERROR]
[ERROR]          An·object·is·plain·if·it's·created·by·either·`{}`,·`new·Object()`,·or·`Object.create(null)`.
[ERROR]
[ERROR]         -
[ERROR]          ##·Install
[ERROR]
[ERROR]          ```
[ERROR]          $·npm·install·is-plain-obj

how about we use the markdownlint in home to instead of  spotless ? , or the github action markdown lint check? https://github.com/DavidAnson/markdownlint @zhangshenghang
Expected Behavior
No response
Steps To Reproduce
No response
Environment
HertzBeat version(s): master

Debug logs
No response
Anything else?
No response

@tomsun28  Can we skip checking the home/node_modules/ directory and only check home/docs and home/blog ，How about this method

@tomsun28 Can we skip checking the home/node_modules/ directory and only check home/docs and home/blog ，How about this method

hi this is also OK.  Another benefit of markdownlint is that it supports more markdown standard checks and use markdownlint-cli2 to fixes.

@tomsun28 Can we skip checking the home/node_modules/ directory and only check home/docs and home/blog ，How about this method

hi this is also OK. Another benefit of markdownlint is that it supports more markdown standard checks and use markdownlint-cli2 to fixes.

@tomsun28 Thanks for your reminder, I use skip checking the home/node_modules/ to temporarily solve this problem. I will update to markdownlint for formatting in the near future.


🍻 Cancellable response stream

This is a prototype (yet) to resolve #2025.
New Pull Request Checklist

[ ] I have read the Documentation
[ ] I have searched for a similar pull request in the project and found none
[ ] I have updated this branch with the latest main branch to avoid conflicts (via merge from master or rebase)
[ ] I have added the required tests to prove the fix/feature I'm adding
[ ] I have updated the documentation (if necessary)
[ ] I have run the tests without failures
[ ] I have updated the CHANGELOG.md in the corresponding package

Additional context and info (if any)

Closing in favor of #2032


as const cast wrong colour when using typescript 4.5.2

TS Template added by @mjbvz
TypeScript Version:4.5.2
Search Terms

semantic coloring
encodedSemanticClassifications

Does this issue occur when all extensions are disabled?: Yes

Version: 1.62.3
Commit: ccbaa2d27e38e5afa3e5c21c1c7bef4657064247
Date: 2021-11-17T07:59:13.865Z
Electron: 13.5.2
Chrome: 91.0.4472.164
Node.js: 14.16.0
V8: 9.1.269.39-electron.0
OS: Darwin x64 19.6.0
Steps to Reproduce:

Create a typescript file with the line [1, "hello"] as const
Notice that const is green (as in a type) instead of blue since it is a keyword.

Thanks for testing
Moving to TS since this coloring is coming through semantic highlighting. const is getting identified as a type

I've just merged #47000, which fixes #46988; it doesn't seem to be exactly the same as it appears as though both the TS Playground and VS Code Insiders allow the hover there. That PR is about the hover being wrong, but the semantic token code shouldn't be taking that path as it's only involved in generating the tooltip's contents, not the actual symbol type.

There are two format options for encodedSemanticClassifications-full:

2020
original

For the 2020 TS server returns const as keyword.
For the original TS server returns const as identifier. Since there is no information about the format parameter, I suppose the problem is in the response related to original. I've changed that https://github.com/microsoft/TypeScript/pull/47066 to match that same response for both formats.


Search input inconsistent on iPhone

The -webkit-appearance: textfield is not enough to remove the border rounding on iPhone.
Strangely, the rounded borders are correctly set only when entering into the field (iPhone bug).
A working solution is to use the none appearance, which has no side effects as far as i tried:
[type="search"] {
	-webkit-appearance: none;
}

@Yapaslfeu, as you know, setting -webkit-appearance: none; will destroy all default styles, and that's not the goal of this project.
Is this happening on all versions of Safari Mobile? If this is a bug, is there a bug report that we can see? Thanks!

Closed via #587

This is not a patch to make forms looking the same on every browser, as explain in #587
This is a patch to correct a bug on iPhone not fully supporting your line 392 -webkit-appearance: textfield; patch!
But @battaglr is right: this is a display bug which should be fixed by Apple on iPhones

The search field once opening the web page on iPhone:

The search field once putting the cursor into:

Then, the search field remains with the same aspect even moving the cursor to another field, till the web page is refreshed…


Lua error in ModBlizzUI.lua

Seems to happen when the guild frame is open and someone logs on or off.
31x Guildbook\ModBlizzUI.lua:265: attempt to index local 'button' (a nil value)
[string "@Guildbook\ModBlizzUI.lua"]:265: in function <Guildbook\ModBlizzUI.lua:261>
[string "=[C]"]: in function GuildStatus_Update' [string "@Interface_TBC\FrameXML\FriendsFrame.lua"]:397: in function FriendsFrame_Update'
[string "@Interface_TBC\FrameXML\FriendsFrame.lua"]:348: in function `FriendsFrame_OnShow'
[string ":OnClick"]:2: in function <[string ":OnClick"]:1>
Locals:
numTotal = 132
numOnline = 9
numOnlineAndMobile = 9
(for index) = 14
(for limit) = 132
(for step) = 1
i = 14
button = nil
(*temporary) =  defined =[C]:-1
(*temporary) = "14"
(*temporary) = "Lil Sparkies"
(*temporary) = 6
(*temporary) = 8
(*temporary) = "Mage"
(*temporary) = "Silvermoon City"
(*temporary) = ""
(*temporary) = ""
(*temporary) = false
(*temporary) = 0
(*temporary) = "MAGE"
(*temporary) = 0
(*temporary) = 1
(*temporary) = false
(*temporary) = false
(*temporary) = 3
(*temporary) = "Player-4384-03B79AE8"
(*temporary) = "6 hours"
(*temporary) =  {
Profession1Level = 0
Talents =  {
}
CookingLevel = 0
Class = "MAGE"
Race = "BloodElf"
FirstAidLevel = 0
Inventory =  {
}
Alts =  {
}
Level = 8
PublicNote = ""
OffSpecIsPvP = false
MainSpecIsPvP = false
OffSpec = "-"
Profession1 = "-"
RankName = "Lil Sparkies"
Profession2Level = 0
Gender = "MALE"
OfficerNote = ""
Availability =  {
}
profile =  {
}
MainSpec = "-"
Name = "Yuper"
CalendarEvents =  {
}
Profession2 = "-"
MainCharacter = "-"
FishingLevel = 0
PaperDollStats =  {
}
}
(*temporary) =  defined =[C]:-1
(*temporary) = GuildFrameButton13 {
0 =
GuildbookColumnProfession2 = GuildFrameButton13GuildbookProfession2 {
}
guildIndex = 13
GuildbookColumnNote = GuildFrameButton13GuildbookNote {
}
GuildbookColumnProfession1 = GuildFrameButton13GuildbookProfession1 {
}
GuildbookColumnOnline = GuildFrameButton13GuildbookOnline {
}
GuildbookColumnRank = GuildFrameButton13GuildbookRank {
}
GuildbookColumnMainSpec = GuildFrameButton13GuildbookMainSpec {
}
}
(*temporary) =
(*temporary) =
(*temporary) = "-"
(*temporary) =  defined =[C]:-1
(*temporary) =  {
773 = "Inscription"
333 = "Enchanting"
185 = "Cooking"
755 = "Jewelcrafting"
171 = "Alchemy"
164 = "Blacksmithing"
202 = "Engineering"
165 = "Leatherworking"
393 = "Skinning"
356 = "Fishing"
129 = "First Aid"
182 = "Herbalism"
197 = "Tailoring"
186 = "Mining"
}
(*temporary) = "attempt to index local 'button' (a nil value)"
L =  {
OPT_COMBAT_COMMS_LOCK = "Block data comms during combat"
CONSUMABLES = "consumables"
MC = "Molten Core"
GUILD_BANK = "Guild bank"
OPT_TT_TRADESKILLS_SHOW = "Show tradeskills (professions)"
FEET = "feet"
Feral = "Feral"
Karazhan = "Karazhan"
Enhancement = "Enhancement"
UPDATE_NEWS = "What's New?
-More Bug Fixes
-More Translation support
-Fishing level in Profile now works properly
-First Aid skill added in Profile
"
SPELL_CRIT = "Crit"
IMPORT = "Import"
Ranged = "Ranged"
Note = "Note"
ADDON_LOADED = "initialising Guildbook!"
OPT_TT_DIALOG_DPL = "Display a list of tradeskills that use the current item. (Data is taken from Guildbook database)"
GUILDBANK_HEADER_COUNT = "Count"
SEPTEMBER = "September"
OPT_SH_MINIMAP_BUTTON = "Show / Hide Minimap Button"
Survival = "Survival"
OPT_CHAT_SMCO = "Show the characters main character name (if an alt) in the guild chat messages. Can only show data where the player has set this value"
Restoration = "Restoration"
MY_SACKS = "My containers"
RESET_AVATAR = "Reset avatar"
TRADESKILL_GUILD_RECIPES = "Guild Recipes"
OPT_RESET_GLOBAL_SETTINGS = "Reset global settings to default values?

SEND_TRADE_ENQUIRY = "Send message about item"
PRIVACY = "Privacy"
OPT_DELETE_GUILD_DATA = "Delete all data for %s"
Combat = "Combat"
MAGE = "Mage"
SPELL_DMG_HOLY = "Holy"
RANGED_DMG = "Damage"
GUILDBANKHELPTEXT = "Guild Bank
|cffffffffGuildbook provides an in-game guild bank for guild
to share bank character's inventory.
|r
|cff00BFF3To use the Guild Bank, add the word 'Guildbank'
to the Public Note of the character being used as a bank
(this will add them to the dropdown menu).
Said character must then open his bank
to sync his inventory with connected Guild Members.
The guild bank sends/receives data when a player logs in,
only the most recent data is being used, thus bank characters
should sync t

That's very odd, someone reported this bug and I added some defence code around the button variable.
Are you using the latest version?

Says 4.99212 at the bottom of the options frame.

Well I didn't get any bugs show up but that's not always been a shining light of the add-on being ready!
But I think push it, and we can swiftly tackle anything that pops up, I'm a bit burned out from playing so maybe some Dev time will be good.


Add root assembly for graph generation

If there are multiple root assemblies or if specific tree needs to be isolated for graph generation, have ability to specify root assembly to reduce clutter in rendering.

https://github.com/AAATechGuy/NetCoreMigrationUtils/commit/5ea98fa42d148285a39d3f14043f6c39c7bdcb6d


merge package template from Kronos-Tools/npm-package-template

package.json

chore(devDependencies): add<EMAIL_ADDRESS>from template
chore(devDependencies): add<EMAIL_ADDRESS>from template
chore(devDependencies): update<EMAIL_ADDRESS>from template

Codecov Report

Merging #94 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #94   +/-   ##
=======================================
  Coverage   83.33%   83.33%
=======================================
  Files           1        1
  Lines          12       12
=======================================
  Hits           10       10
  Misses          2        2

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 8a0f27b...e9ac882. Read the comment docs.


[build] Update dependency graph in build.yml

Is your feature request related to a problem? Please describe.
The GitHub workflow template for Maven workflows includes a task Update dependency graph by default in order to improve the quality of Dependabot alerts. This task using the maven-dependency-submission-action should be added.
Describe the solution you'd like
Add to build.yml:
# Uploads the full dependency graph to GitHub to improve the quality of Dependabot alerts this repository can receive
- name: Update dependency graph
  uses: advanced-security/maven-dependency-submission-action@v4

Note that this action requires that the workflow is allowed to upload files. Otherwise the action will fail with:
Error: Failed to submit snapshot: HttpError: Resource not accessible by integration - https://docs.github.com/rest/dependency-graph/dependency-submission#create-a-snapshot-of-dependencies-for-a-repository
    at L (/home/runner/work/_actions/advanced-security/maven-dependency-submission-action/v4/node_modules/@github/dependency-submission-toolkit/dist/index.cjs:30:1)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
Error: HTTP Status 403 for request POST https://api.github.com/repos/kthoms/operaton/dependency-graph/snapshots
Error: Response body:
{
  "message": "Resource not accessible by integration",
  "documentation_url": "https://docs.github.com/rest/dependency-graph/dependency-submission#create-a-snapshot-of-dependencies-for-a-repository",
  "status": "403"
}

Permissions do not match, yet, although they have been configured on the repo level. Probably needs a separate Github Token with special permissions, according to the docs

I activated a GitHub feature for this, which automatically adds a pipeline step. The explicitly added pipeline step can be removed, now (will do it later this day)


vsphere driver cant find boot2docker.iso

The vspheredriver looks for boot2docker.iso in ~/.docker/machine/ by default. Creating machine without iso already downloaded gives an error about the iso is missing. It seems to complete the download to the cache folder. Copying the iso from ~/.docker/machine/cache/ to ../machine/ solves this temporarily.

cc @frapposelli

The current plugin is being rewritten from scratch (see #1522) once that is merged this will be fixed.

:+1:

Fixed in https://github.com/docker/machine/pull/2487


Energy Consumption going negative exactly 3 months after start

Original issue post, I probably put in the wrong project;
https://github.com/ablyler/bradford_white_connect_client/issues/45
First off, thank you for your contributions, I love this integration.
We installed the Bradford White Connect on April 17th. On July 17th, the total energy values being reported were lower than the previous day.

When capturing the delta for daily consumption, this shows negative energy being consumed.

I'm thinking that the energy value being reported is only the last 90days or 3months worth.
Is it possible to publish daily values instead of rolling 90 day totals?

I see the same problem.
I started collecting heat_pump_energy_usage data on 2024-05-20.
Data values increased as expected until 2024-07-16.  At that time, data collection stopped because my Bradford White account wasn't able to fetch any data.
Data collection resumed on 2024-08-13.  (Thus my energy usage between 2024-07-16 and 2024-08-13 is unknown.)
My heat_pump_energy_usage values have been decreasing since 2024-08-13.
(I don't know the exact date when the values began decreasing because I don't have data during the time window from 2024-07-16 to 2024-08-13.)
It's interesting that my Bradford White data collection failed on the same day that httr-1991's data began going negative.  I don't know whether this is significant or a coincidence.

@jeffstearns - The date is just a conincidence because I was still getting data in the BW Connect app and my delta consumption can be positive when that days consumption is greater than the consumption of 90 days ago.

Thank you for the report! It seems the API only provides energy information for the past three months. I’m pushing an update now that adjusts the add-on to report the current month’s data to Home Assistant. As a result, at the beginning of each month, you’ll see the energy information reset. The Energy Dashboard in Home Assistant should handle this transition smoothly and continue to provide accurate insights.


arduino library

Hi. Thank you for your original research about the DS18B20 temperature sensor.
I'd like to submit the following for your consideration: your software, repackaged as an Arduino library.
ClassifyDS18B20.zip
If your project uses a DB18B20, this library may help you if suddenly a batch of boards acts strange.

Thank you for your effort!
Since I won't have time to maintain this as a library in the Arduino ecosystem you would need to find a way to clarify that you are the maintainer of the submitted library dealing with issues that may arise. I have no experience in how this should be done best.
Besides, the code is mildly out of date since Family E (new on the page as of 2022) is not explicitly accounted for.

The NS18B20 is not in classify_fake_DS18B20.ino yet.
If you do the NS18B20, I'll do the library. OK?

OK, done. library

Thank you for the notice.
Please read the license, specifically point 4b, and edit the source files accordingly.

OK. Like this?
koen

File:    classify_fake_DS18B20.ino
Author:  Chris Petrich
Version: 25 May 2020

Source:  https://github.com/cpetrich/counterfeit_DS18B20/
Documentation:  https://github.com/cpetrich/counterfeit_DS18B20/

Arduino library:

Author:  Koen De Vleeschauwer
Source:  https://github.com/koendv/CheckDS18B20


[Bug]: Total Commander returns broken links

What happened?
Invoke-WebRequest -Uri (Get-EvergreenApp -Name 'GhislerTotalCommander' | where {$_.Architecture -eq 'x86'} | select -ExpandProperty Uri)
404 Not Found
Invoke-WebRequest -Uri (Get-EvergreenApp -Name 'GhislerTotalCommander' | where {$_.Architecture -eq 'x'64} | select -ExpandProperty Uri)
404 Not Found
Version
2211.697
What PowerShell edition/s are you running Evergreen on?
Windows PowerShell
Which operating system/s are you running Evergreen on?
Windows Server 2016+
Have you reviewed the documentation?

[ ] Troubleshooting at: https://stealthpuppy.com/evergreen/troubleshoot/
[X] Known issues at: https://stealthpuppy.com/evergreen/issues/

Verbose output
Invoke-WebRequest -Uri (Get-EvergreenApp -Name 'GhislerTotalCommander' | where {$_.Architecture -eq 'x86'} | select -ExpandProperty Uri -Verbose) -Verbose
VERBOSE: GET https://totalcommander.ch/win/tcmd1010520x32.exe with 0-byte payload
Invoke-WebRequest :

<TITLE>404 Not Found</TITLE>
<BASE href="/error_docs/">

Not Found

The requested document was not found on this server.

<ADDRESS>
Web Server at totalcommander.ch
</ADDRESS>

This uses a DNS TXT lookup, and the record value doesn't match the filename.
id 64732, opcode QUERY, rcode NOERROR, flags QR RD RA
;QUESTION
releaseversion.ghisler.com. IN TXT
;ANSWER
releaseversion.ghisler.com. 714 IN TXT "<IP_ADDRESS>;1"
;AUTHORITY
;ADDITIONAL

From the author:
The TXT record does not report the file name, it reports the version
like an IP address with the following meaning:
"<IP_ADDRESS>;1"
10 always 10, to use the private address range 10.x.x.x
10 major version
52 minor version
0 flags: 0=nothing, 1=a, 2=b or 0x81=beta 1, 0x82=beta 2 etc;0x80+101=RC1 etc, 0x80+121=aRC1..aRC6
;1 re-check every 1 day


Kaleidoscape units do not use maxed skill/ability levels

Kaleidoscape units should use maximum skill/ability levels rather than the levels that the player has upgraded them to.
Should amend StartDungeon in DmodeDungeonService to use the maximum levels instead of those of the player's character.

fixed by #359


bin/configure fails with error: "You don't have Redis installed." Even though Redis is installed and running

On Ubuntu 22.04 LTS following the Quickstart Guide:
git clone git@github.com:bullet-train-co/bullet_train.git foo && cd foo
❯ bin/configure

Successfully installed colorize-1.1.0
1 gem installed

Successfully installed activesupport-7.1.3
1 gem installed

Bullet Train requires Ruby 3.3.0 and `ruby -v` returns ruby 3.3.0 (2023-12-25 revision 5124f9ac75) [x86_64-linux].
You don't have Homebrew installed. This isn't necessarily a problem, you might not even be on macOS, but we can't check your dependencies without it.
Try proceeding without Homebrew? [y/n]
y
You have PostgreSQL 14 installed.
You don't have Redis installed. Please see the installation instructions at https://redis.io/docs/getting-started/installation/install-redis-on-linux/ .

systemclt shows redis as active (running) and redis-cli is working:
❯ redis-cli ping
PONG

But Bullet Train insists that redis is not installed. How to fix that?

Interesting, do you have redis-server installed? As far as I can tell I believe we require it for Bullet Train, and if redis-server is not on your system we should update bin/configure to be more specific.

Feb 14 08:30:54 pop-os systemd[1]: Started Advanced key-value store.

❯ bin/configure

Successfully installed colorize-1.1.0
1 gem installed

Successfully installed activesupport-7.1.3
1 gem installed

Bullet Train requires Ruby 3.3.0 and `ruby -v` returns ruby 3.3.0 (2023-12-25 revision 5124f9ac75) [x86_64-linux].
You don't have Homebrew installed. This isn't necessarily a problem, you might not even be on macOS, but we can't check your dependencies without it.
Try proceeding without Homebrew? [y/n]
y
You have PostgreSQL 14 installed.
You don't have Redis installed. Please see the installation instructions at https://redis.io/docs/getting-started/installation/install-redis-on-linux/ .

@StandardNerd Got it, thank you for the detailed reply. I'm thinking it has to do with how we gather the system package information, I'll try to submit a PR for this one soon.

@gazayas Thank you very much for your commitment and the work you put into the project. That is great, keep on rocking!


Add ability to specify custom Formatter for Pygments code highlighting

In our use of the Python-Markdown library, we discovered the need to be able to use a custom Pygments Formatter class. Pygments supports creating custom formatters, but there wasn't a way to have the CodeHiliteExtension utlize them.
I've added pygments_formatter as a config option for the CodeHiliteExtension class (which is also used by FencedCodeExtension). It can take a string, which will be passed to the call to Pygments' get_formatter_by_name function. It defaults to 'html', which is what was hard-coded before.
Also, the option can take a reference to a class. This makes it convenient to specify a custom class in code without having to create a separate package for your custom formatter. Usage is something like this:
def get_html_from_markdown(value):
    # Use the extra extensions
    from code_samples.code_formatters import MarkdownHtmlFormatter
    return markdown.markdown(value, extensions=[
        'markdown.extensions.extra',
        TocExtension(anchorlink=True),
        CodeHiliteExtension(pygments_formatter=MarkdownHtmlFormatter)])

The last line of code demonstrates the usage.
All tests pass (aside from those marked as Skip), including the additional test I added. Let me know if you'd like any modifications.

I work with @dprothero. As front end design evolves, I'm sure we're not the only ones who could benefit from specifying custom formatters.
Thanks for your maintenance of the project and for giving this PR a look!

Cool! I like it. And I would merge this if we weren't in a feature freeze right now. We are only accepting bug fixes at this time. As we move toward version 3.0 (see the roadmap) the extensions will be broken out into separate packages and will no longer be tied to the development/release cycle of the main package. At that time these sorts of things won't hold up a good contribution like this. Unfortunately, until then, this will have to wait. Of course, in the meantime, you are welcome to fork the extension as long as you honor the licence.

Oh, and before you ask, the timeline is "when it's done."

That sounds totally fine to me! We'll just keep my fork in our requirements.txt until this all comes together.


feat!: add feegranter option to txsim

Overview
This PR adds feegrant back to txsim. Except this time instead of only using feegrant, we now have feegrant as an option. This way we can test without it enabled, which keeps our testing environments slightly more realistic while still being able to enable it when convient. This is particularly useful when we don't want to send gobs of testnet tokens around or have a very long running sequence.
closes #2289
blocking #2197

Q: Do we want feegrant on a per sequence basis or to just set it globally?

Do we want feegrant on a per sequence basis or to just set it globally?

yeah we can refactor to use globally that's a good idea


Screen blanking out issue

I've noticed this for a while, but thought I would finally report it. Not sure if anyone else has experienced this or not.
I had several terminal sessions open earlier today. I left home for a few hours, and came back after a few hours. The entire app was "all black", with no tabs at the top, etc. Alt-F4 wouldn't close it either. Had to kill via it task manager in Win10.
I've also seen this behavior occasionally when I have multiple tabs open, and I CTRL-W out of one of them.
This is with v1.0.91, but as I said, I've been experiencing it for several versions.
I will let you know if I can provide more data on this.

I have a repro too, on v1.0.91, on Win10 x64.

Just a ping to keep this issue alive. Still experiencing this issue periodically.

Here is another clue that may be of help. I went away this weekend and when I got back, I saw all of the terminal screens blank. Tried to close one, and got the same problem (and Terminus hung on me, had to kill with Task Manager). I saw from other apps that my Internet connection had gone down for a bit, then came back online). So perhaps the lack of connectivity played a role here. Of course, I would expect that the Terminus sessions would eventually close due to the lack of connection, but perhaps it isn't handling it gracefully or something?

Any update on this? This issue makes it pretty difficult to work with, especially when you have a dozen tabs open and all of a sudden they go away. :(

The same issue here https://github.com/Eugeny/terminus/issues/1340, still have no idea.

Unfortunately no progress on this yet - I could get a reliable repro yet and these are Electron renderer process crashes, which are not easy to debug either.

same issue, 1.0.117


Responsive layout delayed due to document never being ready

The audio play button covers some of the text on mobile (iPhone 7). micro.blog defaults to putting the audio player at the end, but I think it's more user friendly having it at the beginning, which causes the issue.

It looks fine when I zoom in a bit.

@jakel1828 Thank you for reporting an issue! I appreciate the screenshots as well. That helps a lot.
It appears as though the mobile view isn't loading at all on some Micro.blogs - perhaps due to the number of posts being loaded before the script fires. I'll be taking a close look at this because it affects many sites.

@jakel1828 I'm made an attempt at a fix for this. I'm no longer seeing the issue in the next build. Lets see if it works for you before we close this issue.

It looks the same on my end. I had the same problem when I also tried on Chrome and Firefox.
On Wed, Mar 24, 2021, at 7:47 AM, Colin Devroe wrote:

@jakel1828 https://github.com/jakel1828 I'm made an attempt at a fix for this. I'm no longer seeing the issue in the next build. Lets see if it works for you before we close this issue.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/cdevroe/cypress-microblog/issues/2#issuecomment-805791604, or unsubscribe https://github.com/notifications/unsubscribe-auth/AP3R2DOGCM6GNEYUU7CLK6DTFHNN3ANCNFSM4ZV4SKJA.

@jakel1828 Yes, sorry Jake. The latest version hasn't been deployed to Micro.blog yet. That isn't something I'm able to do. Hopefully the team there will release the latest version soon!

I'm ignorant of how these things work, so I misunderstood. Thanks!
On Wed, Mar 24, 2021, at 8:56 AM, Colin Devroe wrote:

@jakel1828 https://github.com/jakel1828 Yes, sorry Jake. The latest version hasn't been deployed to Micro.blog yet. That isn't something I'm able to do. Hopefully the team there will release the latest version soon!

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/cdevroe/cypress-microblog/issues/2#issuecomment-805842539, or unsubscribe https://github.com/notifications/unsubscribe-auth/AP3R2DLK4GO5SGYXZONWTIDTFHVP5ANCNFSM4ZV4SKJA.

I'm reopening this issue because it appears that Cypress on Micro.blog has a wider viewport, which causes the media queries to not kick in. I'm trying to get to the bottom of it.

Got to the bottom of it. 2021.1.3 will be released on Micro.blog very soon and will fix this issue. Thanks to all for their patience. @jakel1828


Weird output and error after transpilation

inline-svg.service.js:56 Uncaught TypeError: i0.ɵɵdefineInjectable is not a function
    at inline-svg.service.js:56
    at Object../node_modules/ng-inline-svg/lib/inline-svg.service.js (inline-svg.service.js:64)
    at __webpack_require__ (bootstrap:78)
    at Object../node_modules/ng-inline-svg/lib/inline-svg.component.js (inline-svg.component.js:14)
    at __webpack_require__ (bootstrap:78)
    at Object../node_modules/ng-inline-svg/lib/inline-svg.directive.js (inline-svg.directive.js:17)
    at __webpack_require__ (bootstrap:78)
    at Object../node_modules/ng-inline-svg/lib/index.js (index.js:5)
    at __webpack_require__ (bootstrap:78)
    at Module../src/app/app.module.ts (app.component.ts:9)

We ran into this issue as well on Angular 7 with the latest version 8.6.1, reverting to 8.6.0 fixed it.

Same here. Rollback to 8.6.0 works.
Upgrade to 9.2 not.
Any stable solution ?

The same issue - it does not seem to work with ivy.

Currently on Angular 7. Not sure if there is a later version that works as well but I'm using an older version for now until this gets resolved.
npm install<EMAIL_ADDRESS>--save-exact

I've released v8.6.2 and v9.2.1 which are the same as the previous versions but vend both commonjs and esmodule versions. Can you try those?

Hi
I am using angular 6 and same issue occurred.
npm install<EMAIL_ADDRESS>--save-exact
I tried above solution but didn't work.
Is there anyone who solved this issue on angular 6 before?


ENH: Use micromamba for speed and set anaconda-client version

High level summary:

Use micromamba for speed.
Configure Docker image environment for shell environment stability.
Explicitly set anaconda-client version for reproducibility and security. (This should ideally be a lock file but one thing at a time).

I'm going to merge this as we have multiple people approving (thanks for fast reviews!). Comments from others are always welcome.
Anyone object to making a v0.2.0 release from this?


Team Meeting 2/13/24

Agenda: Revision 0

TODO:
Task Management: Filter between incomplete and complete tasks @DangJustin @Lazarus-XD
Bill Management: UI and Payoff Bill @HamidHarris @FardeenAfsar
Account: Edit info and Delete user @fadymorcos21
Bug Fixes: Things don't show up properly on account refresh
Scheduling: Delete event @fadymorcos21
CI/CD stuff: Create tests (on server side)
Work on completing issues from other groups
Next Meeting: Thursday


iproute2: enforce --prefix/usr and fix build on clang only systems

iproute2's makefile uses hardcoded gcc in one place even if it should use at least $CC. if gcc is installed you get a frankenbuild with mixed $CC ( clang ) and gcc. This change fixes this.
Existing package

[x] I am the maintainer of this package.

Didn't notice the force push. :)


Add autocomplete feature

I think the code completion suggestions could be addeded as a plugin.
I think the easiest way could be rewrite keyword property in language definition files y transforming
Prism.languages.actionscript = Prism.languages.extend('javascript',  {
'keyword': /\b(?:as|break|case|catch|class|const|default|delete|do|else|extends|finally)\b/,
'operator': /\+\+|--|(?:[+\-*\/%^]|&&?|\|\|?|<<?|>>?>?|[!=]=?)=?|[~?@]/
});

to something like
keywords=['as','break','case','catch','class','const','default','delete','do','else',''extends,'finally'];
keywordstring="";

for( var i=0;i<keywords.length;i++ ){
 keywordstring+="|"+keywords[i];
}
Prism.languages.actionscript = Prism.languages.extend('javascript',  {
'keyword': /\b(?:keywordstring)\b/,
'operator': /\+\+|--|(?:[+\-*\/%^]|&&?|\|\|?|<<?|>>?>?|[!=]=?)=?|[~?@]/
});

this way the ´.token.keyword´ could remain intact.

Apart from the keywords being provided as arrays, there is still the huge work of creating an actual editor with real-time highlighting.

Well... concerning that please check that [Fiddle] (http://jsfiddle.net/javatlacati/nu4ywjd8/5/).
is there a function for highlight without replacing everything?
As you can see as you type DOM gets regenerated and cursor moves position.

You're right. And Dabblet uses Prism for the highlighting, so I have no doubt this is doable. But should it really be included in Prism itself? I feel like it should be another project. Prism focuses on highlighting only, and that other project could use it to provide highlighting in an editor.

FWIW I always wanted to make PrismEdit, as a separate project. I was planning to clean up & release Dabblet's editor code, but recently someone made a really cool demo that overlays a textarea on top of a pre, which simplifies the code quite a lot, so ideally I would adapt that and then use it in Dabblet too.

Please keep us updated :D

We agreed that it would be done as a different project, so I'm closing the issue.


Cm 273 return localized cards from api

Adding isLocal key:value pair to score_nodes.py to select cards for feed relevant to user's zip code.

Need to remake, this branch was based on Master.


Custom Example Text in Web Components

Custom example text was added in the 1.3.0 web components release, so I need to know what text to put in the web components on the site (gene and trait search).
For gene search, I need the identifier, description, and gene family ID examples.
For trait search, I need traits, pub ID, and an example author.

Adding the bug label because the components don't have default examples, so they're left blank.

I'll propose the following:
Gene search:
identifier:  Glyma.16G044100
description:  proteasome subunit alpha
gene family ID: L_6BFHQX

Trait search:
traits:  Seed weight
pub ID:  10.1371/journal.pone.0054985
example author:  Specht

Notes:

Glyma.16G044100 is involved in flowering time control.
proteasome subunit family: generally strongly conserved; interesting in terms of its evolution.
Gene family L_6BFHQX: a nicely conserved gene family, and has important agronomic traits.
Seed weight: lots of studies report this trait.
Pub ID: an important publication in this field.
Author: A prolific and respected soybean researcher (now retired).

I could not get your branch to load up, so I can't review what you have done.  I need to know how it looks.  I open a code space on your branch and it does not load the html.

I could not get your branch to load up, so I can't review what you have done. I need to know how it looks. I open a code space on your branch and it does not load the html.

Do you mean the web components branch or the recent one updating components to version 1.3.0?

I went to issue #76 which you asked for review. It was about web components so I tried to bring up your web-components-1.3 branch and it failed.  Which branch should I have looked at?
Rex Nelson
@.***
On Jan 3, 2024, at 8:22 PM, Simon @.***> wrote:

I could not get your branch to load up, so I can't review what you have done. I need to know how it looks. I open a code space on your branch and it does not load the html.
Do you mean the web components branch or the recent one updating components to version 1.3.0?
—
Reply to this email directly, view it on GitHubhttps://github.com/soybase/jekyll-soybase/issues/76#issuecomment-1876220387, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADLL6226UJ4QO2M2NWYHXUTYMYG4XAVCNFSM6AAAAABBKV7WHOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZWGIZDAMZYG4.
You are receiving this because you were assigned.Message ID: @.***>

I went to issue #76 which you asked for review. It was about web components so I tried to bring up your web-components-1.3 branch and it failed. Which branch should I have looked at? Rex Nelson @.*** On Jan 3, 2024, at 8:22 PM, Simon @.> wrote:  I could not get your branch to load up, so I can't review what you have done. I need to know how it looks. I open a code space on your branch and it does not load the html. Do you mean the web components branch or the recent one updating components to version 1.3.0? — Reply to this email directly, view it on GitHub<#76 (comment)>, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADLL6226UJ4QO2M2NWYHXUTYMYG4XAVCNFSM6AAAAABBKV7WHOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZWGIZDAMZYG4. You are receiving this because you were assigned.Message ID: @.>

Do you get any errors when trying to start the web-components-1.3 branch? I can't replicate the issue on my computer.

I went to webcomponents-1.3.  I made a code space on it.  I did not see a "Trait-Search".  There was only a Gene_search option.  This is what it looks like to me. I was looking for something like what Steven sent, and of course, that was not in the Gene_Search  had.  I see now that what he said was put on the interface in a way that I was not expecting. It looks ok to me.  I assume the Trait_Search will have a similar look.

Yes, the new change to the web components allows custom examples to be set for each component.
The web-components-1.3 branch just updates the component version in _config.yml and adds the examples to the already in-place gene search.

Handled in #77; closing.


documentation examples

in doc/getting_started.md: The Ising example is probably missing a config file, and also the project_config.sh file is not in MCMCSimulationLib/bash_scripts/ - probably since it is ignored in your .gitignore.
Build your own project
Navigate to MCMCSimulationLib/bash_scripts/ and copy the project_config.sh file
cp project_config_template.sh project_config.sh
You can leave the defined parameters for the moment. More details on the meaning of the parameters is explained in the README.md file.
Next, build a new (template) project by running:
bash build_project.sh

I think the issue is that you got multiple folders with bash scripts, I had navigated to examples but actually the necessary folder is in the main directory.

Hey iulusoy, thanks for the comment! I rearranged the project so that config files are no longer necessary.


Make the volume control remember if muted

PR Checklist

Please check all that apply to this PR using "x":

[ x] I have checked that this PR is not a duplicate of an existing PR (open, closed or merged)
[ x] I have checked that this PR does not introduce a breaking change
[ ] This PR introduces breaking changes and I have provided a detailed explanation below

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ x] Feature
[ ] Code style update (formatting)
[ ] Refactoring (no functional changes)
[ ] Documentation changes
[ ] Other - Please describe:

Fixes
Issue Number: https://github.com/lbryio/lbry-desktop/issues/2361
What is the current behavior?
At this moment when you mute some video and go to other video the muted option is losing
What is the new behavior?
with this new pr, when you set mute in some video and go to other video, this new video will be settled with older muted option
Other information

Thanks for the PR!! We are going to be switching out this video player soon, but we'll reuse this code. Will leave open for now. It should just remember the overall volume, not just muted state - want to add that?

I see that the overall volume state is saved, but I can't change to a new volume

That's fine, it's broken in this player. So do we really need a separate state for muted or is 0 for volume good enough?

We need the two state one for volume and other for muted, since in the player it is taken as two different properties

The properties for volume is a number and for muted is boolean

Thanks for the PR! Tested and working good.


refactor(store): add StoreWrite, rename StoreData to Store

working on https://github.com/latticexyz/mud/issues/2369

docs need to be regenerated

docs need to be regenerated (and prob an issue of https://github.com/latticexyz/mud/issues/2351 again)

It wasn't actually, it was because the docs tried to read StoreData

I am now wondering if my protocol version tests are incorrect and should use IStore and IBaseWorld instead of the current approach (per our chat in Discord about "incomplete" implementations due to "diamond")
Interesting, so the tests should read different interfaces? As opposed to what?


Issues with readonly and enumreable

I'm currently testing these decorators in a typescript boilerplate project, but I've run into some issues. I might totally miss understand how they work, in that case please enlighten me.
When using read-only and nonenumerable on my props I can't get any values at all to stick.
I also get nog warnings about the prop being read only when trying to set it.
Consider the following code:
export class Test {
  prop: string = 'Initial'

  @readonly
  propReadOnly: string = 'Initial'

  @nonenumerable
  propNonEnumerable: string = 'Initial'

  constructor() {
    console.log('Constructor: ', this);
    console.log('Log nonenumerable prop from within constructor: ', this.propNonEnumerable);
    console.log('Log read-only prop from within constructor: ', this.propReadOnly);
  }

  getProp(): string {
      return this.prop;
  }
  getNonEnumerableProp(): string {
      return this.propNonEnumerable;
  }
  getReadOnlyProp(): string {
      return this.propReadOnly;
  }

  setProp(value: string): void {
    this.prop = value;
  }
  setNonEnumerableProp(value: string): void {
    this.propNonEnumerable = value;
  }
  setReadOnlyProp(value: string): void {
    this.propReadOnly = value;
  }
}

Test
import { Test } from './test'

const test = new Test();

console.group('Class members:');
console.log(Object.keys(test));

console.groupEnd();

console.group('Prop');
console.log('test.prop (initial): ', test.prop);
console.log('test.getProp() (initial):', test.getProp());

test.prop = 'New value';
console.log('test.prop (assign): ', test.prop);
console.log('test.getProp() (assign): ', test.getProp());

test.setProp('Final value');
console.log('test.prop (set):', test.prop);
console.log('test.getProp() (set):', test.getProp());
console.groupEnd();

console.group('Prop Read-only');
console.log('test.propReadOnly (initial):', test.propReadOnly);
console.log('test.getReadOnlyProp() (initial):', test.getReadOnlyProp());

test.propReadOnly = 'New value';
console.log('test.propReadOnly (assign):', test.propReadOnly);
console.log('test.getReadOnlyProp() (assign):', test.getReadOnlyProp());

test.setReadOnlyProp('Final value');
console.log('test.propReadOnly (set):', test.propReadOnly);
console.log('test.getReadOnlyProp() (set):', test.getReadOnlyProp());
console.groupEnd();

console.group('Prop NonEnumerable');
console.log('test.propNonEnumerable: (initial)', test.propNonEnumerable);
console.log('test.getNonEnumerableProp(): (initial)', test.getNonEnumerableProp());

test.propNonEnumerable = 'New value';
console.log('test.propNonEnumerable (assign):', test.propNonEnumerable);
console.log('test.getNonEnumerableProp() (assign):', test.getNonEnumerableProp());

test.setNonEnumerableProp('Final value');
console.log('test.propNonEnumerable (set):', test.propNonEnumerable);
console.log('test.getNonEnumerableProp() (set):', test.getNonEnumerableProp());
console.groupEnd();

Output
Constructor:
Test {prop: "Initial"} (when inspecting: prop: "Final value")
 Log nonenumerable prop from within constructor:  undefined
 Log read-only prop from within constructor:  undefined

Class members: Prop
 ["prop"]
 0: "prop"
 length: 1
 __proto__: Array(0)

Prop
 test.prop (initial):  Initial
 test.getProp() (initial): Initial
 test.prop (assign):  New value
 test.getProp() (assign):  New value
 test.prop (set): Final value
 test.getProp() (set): Final value

Prop Read-only
 test.propReadOnly (initial): undefined
 test.getReadOnlyProp() (initial): undefined
 test.propReadOnly (assign): undefined
 test.getReadOnlyProp() (assign): undefined
 test.propReadOnly (set): undefined
 test.getReadOnlyProp() (set): undefined

Prop NonEnumerable
 test.propNonEnumerable: (initial) undefined
 test.getNonEnumerableProp(): (initial) undefined
 test.propNonEnumerable (assign): undefined
 test.getNonEnumerableProp() (assign): undefined
 test.propNonEnumerable (set): undefined
 test.getNonEnumerableProp() (set): undefined

There are some known issues and non-standard ways that TS compiles decorators unfortunately.

core-decorators does not officially support TypeScript. There are known incompatibilities with the way it transpiles the output. PRs certainly welcome to fix that! - https://github.com/jayphelps/core-decorators#get-it

I haven't wanted to spend a bunch of effort on working around them because the decorator proposal in stage-2 changed the spec in very major ways that are incompatible and neither Babel nor TypeScript have implemented it yet.

That said, I'm not sure if that's related to your specific issues or not.

I also get nog warnings about the prop being read only when trying to set it.

If you're using the @readonly decorator, that should be expected. It doesn't make it readonly only for external accesses, it makes it readonly for everyone.
Perhaps you can elaborate?

Sorry, we can close this. I got my answer from the official Typescript repo.
https://github.com/Microsoft/TypeScript/issues/22048

@JonWallsten oh interesting. Thanks for following up!

@JonWallsten just fyi, I've found a way to decorate the instance property:
/**
 * @enumerable decorator that sets the enumerable property of a class field to false.
 * @param value true|false
 */
const nonenumerable = (target: any, propertyKey: string) => {
    let descriptor = Object.getOwnPropertyDescriptor(target, propertyKey) || {};
    if (descriptor.enumerable !== false) {
        Object.defineProperty(target, propertyKey, {
            enumerable: false,
            set(value: any) {
                Object.defineProperty(this, propertyKey, {
                    enumerable: false,
                    writable: true,
                    value
                });
            }
        })
    }
}

class Employee {
    @nonenumerable
    public id: number;
    public name: string;

    constructor() {
        this.id = 1;
        console.log("this.id | Expected: 1 | Actual: ", this.id);
    }
}

var user = new Employee();
user.id = 2;
user.name = 'John Doe';
console.log("user.id | Expected: 2 | Actual: ", user.id);
console.log("Object.keys(user).length | Expected: 1 | Actual: ", Object.keys(user).length);

console.log("Object.keys(user): ", Object.keys(user));

(playground)


Way to stop server with node API?

Is there a way to stop reqlite server after creating it using node API?
Running server keeps my tests up forever (due to network event listeners).
I would want something like this:
// init dummy server
const server = new Reqlite({silent: true});
const connection = server.createConnection();
// tests go here
....
// terminate server once done
server.close();

If it does not exist, I would be willing to implement and create a PR if you are OK with that addition.

It I'm not mistaken, it doesn't exist. A PR would be welcome :)
You have to stop listening on the server, and also close all the connections.

@neumino alright, cool. then I'll prepare the PR

@neumino is there a way to tell that all operations are done? closing during the operations kinda breaks things :)

Hum, I don't think so. In the first iteration, you don't have to immediately abort the queries.
You can just close the connection, and if we try to write on them after the server is closed, you can just skip the operation.

@neumino just create a PR, please have a look and let me know if something needs to be changed


testing omgsvg/Illustrator svg precision

{{short description}}
Changelog
Changed
adds precision to 20/app-switcher.svg (places h/v centered elements @ 8.75 vs 8.8)

Center elements now equally antialiased (properly centered)

Opening this with Illustrator un-centers all of them!


NextButtonShieldEnabled should throw if turned on and CommonControls version is less than 6.0

A typical .Net WinForms app gets a default manifest that does not specify version 6.0 of CommonControls, which is required for showing shield icon on buttons. If this is not done, the property fails silently when set to True - everything appears to works fine, but the call to SendMessage does nothing.
My testing suggests that it's not possible to fix this situation within the library itself. One can add a Win32 manifest to the DLL and specify CommonControls v6, but this doesn't fix the issue unless the primary EXE specifies this as well (both versions of the DLL get loaded into the process space, but the EXE's version probably takes precedence for WinForm bindings to CommCtl controls).
I suggest the following approach: check the CommonControls version sometime during AeroWizard startup (a static class initializer is probably best). If the check fails and user code attempts to set the above property to 'True', it should throw PlatformNotSupportedException (and detail the situation).

For the record, this is the snippet that must be embedded in the manifest:
<dependency>
    <dependentAssembly>
      <assemblyIdentity
          type="win32"
          name="Microsoft.Windows.Common-Controls"
          version="<IP_ADDRESS>"
          processorArchitecture="*"
          publicKeyToken="6595b64144ccf1df"
          language="*"
        />
    </dependentAssembly>
  </dependency>

Also, looking through my posts, I actually removed a check for Application.RenderWithVisualStyles that threw an exception as some customers wanted a graceful failure if visual styles were disabled. Sorry. If this is causing you problems, you can handle this condition in your host form.


refactor make forbid_unknown_col_stats check more accurate

Proposed changes
ignore unknown col stats check if:
colunm not used in query
column is Array/Json/Map/Struct type
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall

run buildall


Make the Solus logo a bit smaller

This is a fairly little change. Sorry for bothering again :/

No worries. :)


Primary naming

Will it use primary when it is released?
IMO it is a generic name which can lead to confusion (I don't know it is a standard in another frameworks). Is it planned to add a secondary?
I think something like Injectable(...) is more descriptive.

It comes from @Primary annotation from Spring Framework.
However the naming conventions are not aligned between Dagger/Hilt and Spring (e.g. Component means something completely different). So @Primary might be confusing to developers used to Dagger/Hilt.
I think the name should refer to @Binds annotation in some way, e.g.:

@Bound
@BoundTo
@BindTo

Anvil from Square is using @ContributesBinding it can be a candidate too

I decided to go with @BoundTo.

it's understandable to Dagger and Hilt users,
it's grammatically correct,
it's not too long,
it's different than other annotations in Dagger/Hilt/Anvil.

I like it, but that to can be strange when you add support to detect the current supertype and if the components is not indicated no?

There is a small obstacle with autodetecting supertype.
The annotation is:
annotation class BoundTo(
    val supertype: KClass<*>,
    val component: KClass<*>
)

I can provide a default type, but not null:
annotation class BoundTo(
    val supertype: KClass<*>? = null, // <-- this is not possible (nullable is forbidden)
    val component: KClass<*> = SingletonComponent::class // <-- this is fine
)

So I can either set the supertype to some general type (like Nothing or Any), which seems a bit dodgy, or I can implement another annotation:
annotation class Bound(
    val component: KClass<*> = SingletonComponent::class
)

Do you think when both annotations exists, the one which doesn't infer the supertype will be used? If yes, then having two different annotations fit with that naming very well

Yes, although rarely, for classes with 2 or more direct supertypes.


Filter options do not work when searching for ports.

hello. First of all, due to personal circumstances, I need to use the web serial API in a firefox environment. Thank you for providing such a beautiful program!
However, I have a question regarding filter options.
I am using the filter option like this in my code, but in the latest Linux release, this filter option does not work and all ports are searched.
< my code in Chrome web serial api >
port = await navigator.serial.requestPort({ filters: [{ usbVendorId: myVendorId }] });

This is not implemented yet.
There is an incoming PR (#4), but the author didn't follow up with the review.

Thank you for your reply. By any chance, can I customize this and use it? Of course it will take a long time...


Add support for Elixir arrays

This PR adds support for array splitting and joining in Elixir.
Works with the following examples:
[a, b, c]

[
  a,
  b,
  c
]

[a, b, c | d]

[
  a,
  b,
  c
  | d
]

[a: 1, b: 2, c: %{a: 1, b: 2}]

[
  a: 1,
  b: 2,
  c: %{a: 1, b: 2}
]

[a: 1, b: 2 | [c: %{d | e: 1}]]

[
  a: 1,
  b: 2
  | [c: %{d | e: 1}]
]

[1, 2, 2 |> Kernel.+(1) | [4]]

[
  1,
  2,
  2 |> Kernel.+(1)
  | [4]
]

I did manage to find an edge case where this doesn't work:
[a: 1, b: 2, c: %{d | e: 1}]

[
  a: 1,
  b: 2,
  c: %{d
    | e: 1}
]

However, to handle this it's necessary to build a parser and given it's such an uncommon edge case (updating a map when declaring the last element of an array), I assumed it wouldn't be worth the effort.

On a sidenote: as a byproduct of this, I managed to add some missing tests for Elixir and I'm currently working my way through maps and arrays via sigils.

Thanks for reviewing! I'll be handling these later today.
I did have some issues running the tests with indentation (especially adding tests to existing do-blocks on a different branch) because macvim also has some issues with indentation. It'd really help if I could add a submodule for vim-elixir.
In fact, it already depends on it since the delimiters used in sp#elixir#SplitArray() are in fact provided by vim-elixir. I actually forgot to mention in the PR description, but I assumed that dependency since I'm 99.99% certain someone working with Elixir in Vim to the point they'd need to use splitjoin, would be using vim-elixir. I think a similar dependency already exists with the Ruby parser and vim-ruby.

I assumed that dependency since I'm 99.99% certain someone working with Elixir in Vim to the point they'd need to use splitjoin, would be using vim-elixir. I think a similar dependency already exists with the Ruby parser and vim-ruby.

Yes, there's a lot of dependencies, to Rust as well. A couple of days ago, I decided to vendor these in specs, and this seems to be working fine. Here's the lines that load the plugins: https://github.com/AndrewRadev/splitjoin.vim/blob/99d12007112b63b8e3fd0fcf435471ac63ccf030/spec/spec_helper.rb#L14-L16
And here's the .gitmodules file: https://github.com/AndrewRadev/splitjoin.vim/blob/master/.gitmodules
So, I'd do the same for vim-elixir, but I can do that for you as well -- if you'd like, focus on your fix and I'll get the tests running afterwards.


Provide a way to pass flags to chromium

I need to pass a few cli flags to the Chromium part of Carbonyl. I'm currently trying to set the user-agent string and proxy configuration but I may need to set a few additional parameters.
Is currently this possible?

See https://github.com/fathyb/carbonyl/issues/37 .  If you can do it with chromium command line flags, you should be able to do it with carbonyl.


raftstore: an efficient workflow for localreader

What have you changed? (mandatory)
This PR optimizes local reader's workflow that saves unnecessary lease check(makes a syscall), and reuse snapshots for requests in the same batch.
What are the type of the changes? (mandatory)

Improvement (non-breaking change which is an improvement to an existing feature)

How has this PR been tested? (mandatory)
unit tests and integration tests.
Does this PR affect documentation (docs/docs-cn) update? (mandatory)
No.
Does this PR affect tidb-ansible update? (mandatory)
No.
Refer to a related PR or issue link (optional)
#2639
Benchmark result if necessary (optional)
QPS improved about 16.5%, from 207211 to 241526.9.
Add a few positive/negative examples (optional)
No.

any update? @overvenus

A friendly ping.

rest LGTM.

PTAL, thanks!

LGTM.

@BusyJay PTAL


AppSync Resolver multiline string

** Which Category is your question related to? **
API
** What AWS Services are you utilizing? **
AppSync
** Provide additional details e.g. code snippets **
I have a simple AppSync request resolver like this, I am finding it hard to maintain long MySQL queries in a single line. I wasn't able to find any solutions for a multiline string in VTL. Is there a standard for handling multiline strings in VTL?
{
  "version": "2018-05-29",
  "statements": ["SELECT this.Post as that, that.Post as this, this.Tag as thatTag, that.Tag as thisTag FROM POST JOIN TAG ON TAG.Post_id = POST.id"]
}

@oste I think your best option is to move the select statement out to a variable, since with VTL you can have multiline strings in #set.
#set ($statement = "
SELECT
    this.Post as that
FROM
    POST
")

and then reference the variable from your JSON construct at the second half of the template.

I'm closing the issue, but feel free to reopen it if something new comes up.

@attilah that was one of the first things I tried that I hoped would work. It is giving an error.
{
  "data": {
    "listPosts": null
  },
  "errors": [
    {
      "path": [
        "listPosts"
      ],
      "data": null,
      "errorType": "MappingTemplate",
      "errorInfo": null,
      "locations": [
        {
          "line": 2,
          "column": 3,
          "sourceName": null
        }
      ],
      "message": "Unable to parse the JSON document: 'Illegal unquoted character ((CTRL-CHAR, code 10)): has to be escaped using backslash to be included in string value\n at [Source: (String)\"\n{\n  \"version\": \"2018-05-29\",\n  \"statements\": [\"\n  SELECT\n

I tried that. I was missing $util.escapeJavascript.
This works:
#set ($statement = "
SELECT
    this.Post as that
FROM
    POST
")

{
  "version": "2018-05-29",
  "statements": ["$util.escapeJavaScript($statement)"]
}

Thanks.

This doesn't seem to work for me, even when i use escapeJavasScript any ideas? I get unrecognized escape character

Agreed, even with util.escapeJavaScript, I still get unrecognized escape character

Dellybro - this worked for me:
{
"version": "2018-05-29",
"statements": [
$util.toJson($multiline_sql)
]
}

Dellybro - this worked for me:
{
    "version": "2018-05-29",
    "statements": [
        $util.toJson($multiline_sql)
    ]
}

Thank you thank you thank you


Non-platform specific canvas clearing

The goal of this PR is to move the canvas clearing logic from platform specific code to non platform specific code.
It also removes the clearing done using the OpenGL API

Can you elaborate on what you would like to be tested and if it is possible to do with the current testing infrastructure?

I want a test that shows the example above - where a combination of RN Views and Skia views verifies that we’re handling opacity correctly now that we’re setting everything up from within js. So basically just a simple absolutely positioned skia view with opacity above a react native view would be enough :)

here the feature is tested in the sense that subsequent drawings in the e2e suite are correct (the previous drawing has been cleared). We are only testing for the canvas images right? It's not clear to me how we would test for a result that includes React Native views.

Agree, that's not possible to do.
The current tests only test for clearing the screen - not for opacity effects between RN and Skia Views.
Maybe we could add an example screen showing this instead? What do you think?

Don't we have many screens showcasing Skia view transparency?

Yes - within the Skia World - but not between RN Views and Skia I think. I remember spending time on getting the configuration right to support this early on (remember the cat demo). So I think it would be nice to have either a test or an example verifying this after this change.

I was thinking about a screen such as this one:

Yes! Sorry :) didn’t think about those.


Stix2.1 wd05

Major changes include:

Updates to all SDO classes for STIX 2.1 WD05
Improved the exception class hierarchy
Improved how property "cleaning" errors are handled
Improved how the workbench monkeypatching works

Codecov Report

Merging #283 into stix2.1 will increase coverage by 0.31%.
The diff coverage is 98.98%.

@@             Coverage Diff             @@
##           stix2.1     #283      +/-   ##
===========================================
+ Coverage    98.66%   98.97%   +0.31%
===========================================
  Files          122      121       -1
  Lines        13060    12889     -171
===========================================
- Hits         12885    12757     -128
+ Misses         175      132      -43

Impacted Files
Coverage Δ

stix2/test/v21/constants.py
100% <ø> (ø)
:arrow_up:

stix2/test/v21/test_malware_analysis.py
100% <ø> (ø)
:arrow_up:

stix2/properties.py
100% <ø> (ø)
:arrow_up:

stix2/v21/sro.py
94.59% <ø> (ø)
:arrow_up:

stix2/core.py
100% <ø> (ø)
:arrow_up:

stix2/test/v21/test_workbench.py
100% <ø> (+27.27%)
:arrow_up:

stix2/test/v21/test_datastore_filters.py
100% <ø> (ø)
:arrow_up:

stix2/test/v21/test_opinion.py
100% <ø> (ø)
:arrow_up:

stix2/test/v21/conftest.py
100% <ø> (ø)
:arrow_up:

stix2/test/v21/test_datastore_memory.py
100% <ø> (ø)
:arrow_up:

... and 25 more

Continue to review full report at Codecov.

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

Looks good to me...

This looks great! Can we add some deprecation warnings for Observed Data and for its objects property? I think warnings.warn() is the preferred way to do this.
Can we add a test for threat-actor.last_seen < first_seen? Looks like it's the only thing not covered from the coverage report.
Also, (though I would merge the PR without this one) for the testing that was removed because it looked at extension properties, could we get back at least a little of it by putting 'extensions' in the InvalidValueError.prop_name and checking that? I think this would mean adding an except AtLeastOnePropertyError() to _check_property. The tests in question were the ones changed in test_observed_data.py for both stix versions.

Can we add some deprecation warnings for Observed Data and for its objects property? I think warnings.warn() is the preferred way to do this.

Seems to me, the warnings module is primarily for the interpreter and perhaps built-in libraries.  It interacts with the interpreter's -W commandline option.  The provision for "user" warnings is via a special exception type, UserWarning.  This gives the impression that ordinary users may use the module, but they shouldn't use the other warning types.  Sounds like your idea was to use DeprecationWarning in particular, but I'm not sure that's how it's intended to be used.

Also, for the testing that was removed because it looked at extension properties

You may have misread the commit comment.  It said: A small amount of testing was removed, since it was looking at custom exception properties...
So this isn't specifically about extensions.  I think those changes were regarding exceptions which had previously been TypeError's, and therefore hadn't been caught and replaced with InvalidValueError.  Most of the property presence exception classes fall into that category (at least one, extra, missing, etc).

For warnings, I was reading this. Looks like you can set a filter so they show up without the interpreter option, but I'm not sure yet what best practice is for library authors like us.
Yes I misspoke (and probably misread too) about the removed tests. I was trying to get at stuff like the removed code above this: https://github.com/oasis-open/cti-python-stix2/pull/283/files#diff-5d546fb7e9a4fb90af64f4ae52b50106R799 I was hoping to be able to check a little more precisely that the correct error was encountered.

... I'm not sure yet what best practice is for library authors like us.

Hmmm... I couldn't find anything that described an intention for users (of Python interpreters and core libraries, i.e. normal developers) to only use UserWarning.  I found a Deprecation library, which includes a white paper describing ways other Python libraries deal with deprecation warnings, including well-known libraries like Flask and Django.  All signs suggest they use all the warning categories.  Ok then, guess we can too.  I guess that Deprecation library wouldn't really work for us, since we aren't deprecating a function or class (something we can decorate); we're deprecating a virtual "property" whose definition is buried in a data structure... it's more complicated for us.
As far as best practices, I think our libraries should emit the warnings but never set up warning filters.  Any commandline tools we create should be free to set filters or give commandline options to allow user control over them.

I was hoping to be able to check a little more precisely that the correct error was encountered.

If you throw out the exception object, you can't test it; that's inherent in the original design.  In Python 3, you can chain the original and the new one via a special attribute (__cause__), which is what I chose to do.  So on Python 3, you could get the original exception and run some tests.  But they would be python version specific.
Another idea is to not test cleaning via object creation, as that triggers the whole property checking, exception replacing, mechanism.  If you want to test cleaning, directly call clean() on a property.  That will raise the exception directly to you and then you can check whatever you want.

Hi, is there a plan to allow setting deterministic uuidv5 ids for stix sdo objects?
It seems this is mentioned in the wd05 draft but there are no changes here to address that.
See: https://github.com/oasis-open/cti-python-stix2/blob/582ba2be2c539ec2ef1f8bb384f5c1deb5e23831/stix2/properties.py#L19
Or is this something that will be included in future PRs?

@cyvs Yes, this is something we are working on and will be in a future PR.

The beginnings are already in the stix2.1 branch here.  uuid5's are allowed, but logic for automatically computing one for an SCO from "ID Contributing Properties" is not implemented yet.