Add AccessToken to UserCredentials constructors.

3-legged-auth has access token with optional refresh token. Breaking change to UserCredentials constructors.

@garrettjonesgoogle and @ejona86 can you review?

Addressed code review feedback. One behavior change was to change the exception from refreshing the credentials with no refresh token into IllegalStateException which seems like a better fit. I can squash commits we are ready to submit.

LGTM.

@anthmgoogle, FYI, the UserCredentials constructor change is no problem for grpc, as we aren't using the class directly.

@anthmgoogle LGTM


Create an Image-to-Text action kamelet

Create an action kamelet based on the Camel AI example: https://github.com/megacamelus/camel-ai-examples/tree/main/image-to-text

The YAKS tests have been failing for a while. We should look at the failures but for the time being, let me go ahead to merge it.


Documentation link is broken

Link in https://github.com/pyrra-dev/pyrra#documentation is broken :)

Yes, good find. So far all the documentation is in the README, but I'm planning on adding a lot more documentation with examples to the docs/ folder soon. Until then it might be better to comment it out.

Removed it until we have a proper site with docs.


Tlspatch

Following a conversation with the team on Slack, this is a patch users can run (either through node or bash) that modifies the milvus-sdk-node after it has been installed to node_modules. I have attached a readme so that it is easy to understand and use.

An example of use is with Attu, as shown in https://github.com/zilliztech/attu/pull/191

Hi, agahEbrahimi, thank you for the contribution.  😀
I have a question, can we add tls parameters in this file, other than using a patch to modify compiled package.
https://github.com/milvus-io/milvus-sdk-node/blob/218ad4c2763442b3b25beff6dfed8f4a20f481d7/milvus/grpc/GrpcClient.ts#L60
If you don't have time, I will do this week.

For sure @shanghaikid ! I can do it within the next couple of days if that's alright.

For sure @shanghaikid ! I can do it within the next couple of days if that's alright.

Yeah, that's totally fine. Thanks a lot.

Closed this pr, since we have implemented tls in the sdk.


HISENSE-193 - Exit fatory app on turning off and on TV

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: madanagopaltcomcast:x: kannanmadanagopaltYou have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: madanagopaltcomcast:x: kannanmadanagopaltYou have signed the CLA already but the status is still pending? Let us recheck it.


Patch API sub-array object find support

Would it be reasonable to add sub-array object find support to the patching array?
I'd like to be able to find an object in a sub-list before patching it. It doesn't seem to be working for me. There is no error, but the object is not updated in the database.
For example:
theSession.Patch<Target>(target.Id).Set(x => x.subList.anotherSubList.First(o => o.prop == "someValue").Number, 10);

I don't see that happening very easily. You're just talking about way, way too many permutations to account for. We don't have anything quite like what you're talking about, but I think it'd be way easier to use the Javascript transform feature for these kinds of unusual patches.
It's not just a "sub-array" object, it's understanding the First() within the expression that's the killer there.

This probably isn't going to get done. I'm recommend using a javascript function transform here. We could do better JS transforms that allow for inputs. Maybe an extensible patching model I guess.


remove funds from vesting contracts

[x] EGIO still outstanding (uses older contract, so need to contribute to buy rounds to get tokens)
will need funds for this; can work with EGIO team on this
[x] Swamp Audio (currently in progress)
[X] EXLE
[X] Ergolend

Marty wants to do this on his own

potentially use JSON to do this
there is a tool that may help that Noob has made

One more with ergolend still (Lui)

EGIO still outstanding.
Ergolend need to be sent to Chris & team
Need to figure out EXLE burn and EGIO burn
Swamp Audio vesting next

converting task into new one to cleanup EGIO


LIVD Table Update Utility

Background
Currently the process for updating the LIVD table is a manual one, which involves downloading the file and then doing some processing around extracting a single tab and saving it as a new CSV file. We can build a tool that will take a LIVD file downloaded off of the website, extract the data that we want, and then replace the existing file with the new data.
At some point the LIVD table will end up in the database as part of @carlosfelix2 work, but for now, this will ease our efforts around keeping up-to-date with LIVD changes.
Acceptance Criteria
NOTE that these acceptance criterias are a bit old.  See epic acceptance criteria for epic issues.

The current LIVD table to no longer have the date in the name, but instead the "current" file will have the name LIVD-SARS-CoV-2.csv.
All references to the most recent file will be updated in the project as well.
A script (python is fine, or we can build it into the prime executable if you prefer) will be created that will migrate the existing file to an archive with the following name format: LIVD-SARS-CoV-2-Archive-yyyymmdd-00.csv where the date format is the date of archiving, and if the 00 is the current count of archives, the first being 00 and then incremented each time

The script will accept a path to a downloaded XLSX LIVD table from https://www.cdc.gov/csels/dls/sars-cov-2-livd-codes.html
The script will create an extract of LOINC Mapping tab from that spreadsheet that matches our current files and create a temporary CSV file from it
The script will then do a hash to compare the existing CSV file in the tables folder against the temp CSV file, and if the hashes match, it exits without replacing anything
If the hashes don't match it copies the existing file into an archive with the name formatted as above and then replaces the current file with the temporary file we created

Archiving and replacing the existing file should be the last step of the process, so we do not inadvertently delete the existing file, and if there are any errors, the existing file should not be replaced either

Additional Notes
There is obviously no provision above for rotation or clean up of archived files above. This is intended to be a tool that makes the process of updating the LIVD table more repeatable and safer, but it is not intended to be run without human oversight. In fact, after the file is replaced by running this utility, the application still needs to be thoroughly tested, and the update needs to be submitted as a PR for review.
Eventually we can expand on and improve this tool to automate the process of saving the new LIVD information to the DB.

This is already done in the ./prime livd-table-download command.


HTTP/3 extended CONNECT is not normalised to H1 format

Title: HTTP/3 extended CONNECT is not normalised to H1 format
Description:

As part of our CONNECT-UDP implementation for envoy, we found that presently, HTTP/3 Extended Connect requests stay in HTTP/3 format, and this is a bug because they are therefore not recognized in any of the HTTP Upgrade routing logic. Therefore, we must normalise H3 requests, in a similar way to how we normalise HTTP/2 requests

(assign this to me, since I am already half way with the fix)

I am already working on a fix for the issue as previously discussed with @alyssawilk .

There is also a problem inside the QUICHE codec that results in the "unexpected DATA frame" error on the response to the extended CONNECT.


Unable to compile: RequestRouterBase.cs(102,29): error CS0019: Operator '==' cannot be applied to operands of type 'TDescriptor' and 'default'

Looks like the issue is here:
https://github.com/OmniSharp/csharp-language-server-protocol/blob/09fc4f347b259ffe736cb3fb679e0c41ffd5b1cb/src/JsonRpc/RequestRouterBase.cs#L102
Can open a PR to fix

Same issue elsewhere:
Models\BooleanOr.cs(18,32): error CS0019: Operator '!=' cannot be applied to operands of type 'T' and 'default' [C:\Users\roholt\Documents\Dev\csharp-language-server-protocol\src\Protocol\Protocol.csproj]

These appear to be fixed thanks @rjmholt


"Didn't find class "com.appsflyer.AppsFlyerLib" error was reported after upgraded to version 6.2.3

I need to upgraded appsflyer sdk to version 6.2.3 and got this error:
AndroidRuntime: Caused by: java.lang.ClassNotFoundException: Didn't find class "com.appsflyer.AppsFlyerLib" on path: DexPathList[[zip file "/data/app/~~FvFWLvnrVtldkS2_TMEW8g==/com.uxin.live.overseas-FBcwFccULnUTK2Px3ww7kw==/base.apk"],nativeLibraryDirectories=...
my code is below.
build.gradle:
dependencies {
implementation 'com.appsflyer:af-android-sdk:6.2.3'
implementation 'com.android.installreferrer:installreferrer:1.0'
}
proguard.cfg:
-dontwarn com.android.installreferrer
-keep class com.appsflyer.** { *; }
and the initialization part is the same as the official recommendation.Now I have no solution to the problem,so I will be very grateful for any help!

@zengjie-android thank you for opening this issue. did you upgrade from earlier sdk version? if yes, which version?


Add equals to most Monoids (#230)

This PR makes most of the monoids setoids as well, by implementing the equals function.

Coverage remained the same at 100.0% when pulling 8078d6bb46768399bacb28d2b1e5812e3ba85e0d on monoids-setoids into 747311034dc05da301bfaad4a7ff1492e5989e78 on master.

No docs updates?

@karthikiyengar: if you wanna take @dalefrancis88 suggestion, you could make a helper in core like:
const isSameType = require('./isSameType')
const equals = require('./equals')

const mequals = (x, y) =>
  isSameType(x, y)
    && equals(x.valueOf(), y.valueOf())

@evilsoft Do you think it's a worthwhile change? I personally like the reduction in repeated tests but curious what you think

@dalefrancis88 I like it.I do no think it would reduce the number of specs though as we would want to test each in context. Which specs do you think could be omitted?

@dalefrancis88, @evilsoft - I do not see the value of adding the helper, especially in the cases of First and Last where the mequals won't work since the underlying construct is a Maybe. Also, we are getting rid of neither tests nor additional parameters with the helper. But, if it's unanimous, I can add the helper.
@evilsoft, @bennypowers - With respect to the docs, can we extract it to a separate issue? I am not sure how the release process works, but do we need to have complete documentation around issues? If not, then this would help us push out many improvements in with a new release that would be of great help, given that there are no more code related issues in the release board.
@evilsoft - I will reorder the testcases and push a commit soon.

@evilsoft I guess when it comes down to what can/should be removed it's more a question on what you believe should be tested.
@karthikiyengar The value in having the method is that it communicates intent. It also means that when anything deviates from that intent it should be for a notable reason. Having all the tests around it will let you define it as an aspect as well, equality is not unique to each monoid. Now when it comes to the Maybe, the question i guess is how often the equal with the Maybe will show or is already in the code.  You could have the func check and support maybe or have another func that expects maybes?
Either way if you decide not to do it, let me know and i'll do it later this weekend. I'm finally going to get some time to work on this in more than 15 min blocks this weekend

@karthikiyenga my concern is that docs will stay out of sync for a significant amount of time, and users like me won't even be aware of awesome features. OTOH I certainly appreciate the desire to ship features quickly, especially as we pursue f/l coverage.
Is there a middle ground that will allow us to gain ground at pace while still keeping docs in the picture? In my view, even stubs are better than nothing. Perhaps a stub pragma which renders both a link to edit docs and a link to relevant source code? This is a larger issue.

@dalefrancis88 - I cannot see how in this particular case the words isSameType and equals in 2 lines of code fail to communicate intent.
@bennypowers - Agreed - I will spend some time on the docs on the long weekend ahead. Please feel free to contribute in case you some spare time.

Its all good, I was more pointing it out because its a pattern thta matches other parts of the code-base, like isArray, or isFunction, these are wrappers around 1 line of code but are still good.
I believe this may be because if there was ever a reason to change how we determine those, we're only changing the in one place rather than everywhere.
But like I said it's not a major concern for this PR, if you get the changes done by @evilsoft suggested i'll but a :shipit: on it.

No problem at all, I'll take care of it with the next line of commits :-)
On Thu, Apr 5, 2018, 8:53 AM Ian Hofmann-Hicks<EMAIL_ADDRESS>wrote:

@evilsoft requested changes on this pull request.
I am so sorry @karthikiyengar https://github.com/karthikiyengar, I
forgot to bring up one other thing (it is so new, I always forget about
it). As we are changing the api for the datatypes, we will need to
increment the VERSION constant at the top of each ADT file.
This is used for sanctuary-def integration which allows for type
versioning to make sure that becomes part of the type checking.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/evilsoft/crocks/pull/250#pullrequestreview-109593343,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AH4KbwUt9ClO5evyviogYxiJ2sIYLlmRks5tlb9MgaJpZM4S-DJ-
.

@karthikiyengar how would you feel about merging this one in with the VERSION updates.
Then we can do the docs on another PR, and just wait to ship a new version after that is merged in.
I ask because I have a big change PR that is going to 🌽flict with this one in every file. Would be easier for me to rebase these changes into mine.

Consider it done. I will send in another PR in a few hours.
On Mon, Apr 9, 2018, 8:27 PM Ian Hofmann-Hicks<EMAIL_ADDRESS>wrote:

@karthikiyengar https://github.com/karthikiyengar how would you feel
about merging this one in with the VERSION updates.
Then we can do the docs on another PR, and just wait to ship a new version
after that is merged in.
I ask because I have a big change PR that is going to 🌽flict with this
one in every file. Would be easier for me to rebase these changes into mine.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/evilsoft/crocks/pull/250#issuecomment-379848553, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AH4KbxrfHxyYpgeMO3sHAZAFm5-xeaj4ks5tm6gogaJpZM4S-DJ-
.

Closing in favour of https://github.com/evilsoft/crocks/pull/258


Categories for posts

So I am trying to add a category model for the posts by doing the following:
class Post(db.Model):
    __tablename__ = 'posts'
    id = db.Column(db.Integer, primary_key=True)
    title = db.Column(db.String(80))
    body = db.Column(db.Text)
    timestamp = db.Column(db.DateTime, index=True, default=datetime.utcnow)
    author_id = db.Column(db.Integer, db.ForeignKey('users.id'))
    body_html = db.Column(db.Text)
    comments = db.relationship('Comment', backref='post', lazy='dynamic')

    category_id =db.Column(db.Integer, db.ForeignKey('category.category_id'))
    category = db.relationship('Category')

    def __init__(self, title, category):
        self.title = title
        self.category = category

    @staticmethod
    def on_changed_body(target, value, oldvalue, initiator):
        allowed_tags = ['a', 'abbr', 'acronym', 'b', 'blackquote', 'code', 'em', 'i', 'li', 'ol', 'pre', 'strong',
                        'ul', 'h1', 'h2', 'h3', 'p']
        target.body_html = bleach.linkify(bleach.clean(
            markdown(value, output_form='html'),
            tags=allowed_tags, strip=True))

    def __repr__(self):
        return '<Category %r>' % self.id

db.event.listen(Post.body, 'set', Post.on_changed_body)

First question am I doing it right? To my guess it's one-to-many since it's one category for many posts. And if I do the following I get errors on main/views.py
 post = Post(body=form.body.data,
                    author=current_user._get_current_object())

The error (using Pycharm IDE) for "body=form.body.data,
author=current_user._get_current_object()" is Unexpected argument.
Any ideas what's the right way?

The problem is that you defined a constructor for class Post, which overrides the constructor that SQLAlchemy provides. When you create your object you are passing body and author, but your redefined constructor takes only title and category.
Take a look at how authors are assigned to posts, categories works more or less in the same way. I don't think you need to overload the constructor, but if you do, you have to preserve the arguments and call the base class using super.

Okay so I took your suggestion and I ended up with the following result which magically worked (first step to becoming a programmer, code works but have no idea why lol)
class Post(db.Model):
    __tablename__ = 'posts'
    id = db.Column(db.Integer, primary_key=True)
    title = db.Column(db.String(80))
    body = db.Column(db.Text)
    timestamp = db.Column(db.DateTime, index=True, default=datetime.utcnow)
    author_id = db.Column(db.Integer, db.ForeignKey('users.id'))
    body_html = db.Column(db.Text)
    comments = db.relationship('Comment', backref='post', lazy='dynamic')
    category_id = db.Column(db.Integer, db.ForeignKey('categories.id'))
    category = db.relationship('Category',
                               backref=db.backref('posts', lazy='dynamic'))

    @staticmethod
    def on_changed_body(target, value, oldvalue, initiator):
        allowed_tags = ['a', 'abbr', 'acronym', 'b', 'blackquote', 'code', 'em', 'i', 'li', 'ol', 'pre', 'strong',
                        'ul', 'h1', 'h2', 'h3', 'p']
        target.body_html = bleach.linkify(bleach.clean(
            markdown(value, output_form='html'),
            tags=allowed_tags, strip=True))

db.event.listen(Post.body, 'set', Post.on_changed_body)

class Category(db.Model):
    __tablename__ = 'categories'
    id = db.Column(db.Integer, primary_key=True)
    name = db.Column(db.String(50))

    def __repr__(self):
        return '<Category {}>'.format(self.name)

I created a CategoryForm
class CategoryForm(Form):
    name = StringField('Category name', validators=[DataRequired()])
    submit = SubmitField('Submit')

Category.html
{% extends "base.html" %}
{% import "bootstrap/wtf.html" as wtf %}
{% import "_macros.html" as macros %}

{% block title %}Flasky{% endblock %}

{% block page_content %}

{{ wtf.quick_form(form) }}

{% endblock %}

Category view
@main.route('/category', methods=['GET', 'POST'])
def register():
    form = CategoryForm()
    if form.validate_on_submit():
        category = Category(name=form.name.data)
        db.session.add(category)
        flash('A confirmation email jas been sent you.')
        return redirect(url_for('main.index'))
    return render_template('category.html', form=form)

and then after adding categories I edited the PostForm with
category = QuerySelectField('Category', query_factory=enabled_categories,
                                allow_blank=True)

And then index view category=form.category.data and then rendered the category with the posts and vuala! It worked. Is this the best way to do it or is there a better and simpler way? So I guess the most difficult part is done? Now how do I try to get www.example.com/category/Flask and all the posts that have been tagged with Flask category to show on that url? Any tips where I should look at?

I answered on your StackOverflow question here.

Thank you Miguel :).


is setInterval to listen to the ticker mandatory?

Hi,
Thanks for this great tool.
I was just wandering if I need to listen to the ticker changes with new setInterval like:
const TradingViewAPI = require('tradingview-scraper');
const tv = new TradingViewAPI();

setInterval(() => tv.getTicker('EURUSD').then(result => {
  console.log(result)
}), 1000);

Or I'm doing it wrong? because firing the tv.getTicker(ticker) is returning only one response.

Hey,
Thanks for your interest in my project.
As this tool was made because I needed just the current ticker data, there's no event support, but actually, since I was going to update the library soon (the codebase is really dirty), I may implement it in next version. I'll leave this issue open as a feature request. If anyone wants to implement this feature and submit a PR, you're more than welcome.
So answering your question: yes, that's the only way at the moment.
Best regards,
Piotr Adamczyk

Ok thanks!
On Saturday, November 30, 2019, Piotr Adamczyk<EMAIL_ADDRESS>wrote:

Hey,
Thanks for your interest in my project.
As this tool was made because I needed just the current ticker data,
there's no event support, but actually, since I was going to update the
library soon (the codebase is really dirty), I may implement it in next
version. I'll leave this issue open as a feature request. If anyone wants
to implement this feature and submit a PR, you're more than welcome.
So answering your question: yes, that's the only way at the moment.
Best regards,
Piotr Adamczyk
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/imxeno/tradingview-scraper/issues/7?email_source=notifications&email_token=AGSW6E3C7FNHVBEIZ3HUU3LQWKAY3A5CNFSM4JTGNE32YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFQKRLI#issuecomment-559982765,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGSW6EYCQCRD2VZNBGLBSUTQWKAY3ANCNFSM4JTGNE3Q
.

@chang96 a new, experimental version that implements event emitter functionality is available on develop branch - cb82a5327a72010e8038eb684ae1b42dea3daa80. If you've got some time check it out and let me know what do you think. It is not available on NPM at the moment.

The new version was made available on NPM, so I'm closing this as it's no longer relevant.


Nuxt frontend (more of a complete re-write)

can probably have some checklists here or smth
edit: check this for todo list: https://github.com/PaperMC/Hangar/issues/314

manually merged, woooo


Update deployment-stacks.md - fixing a capitalization issue

Fixed a capitalization issue which causes ambiguity among the translators in "Use the new command" -> "Use the New command". Translators assume "new" is used as an adjective defining the noun "command".

@mumian
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@QueryCoordinator - thank you for your contribution.
#sign-off


Rust / Rocket

https://github.com/TatriX/realworld-starter-kit

Ok, it's currently passes all the tests and mostly works. Sure it's still need some polish and such. For example I need to get rid of all the expects and rename things to make them consistent.
It would be good if someone could review the code. Thanks!

I think all the required functionality is done. What is the next step?

@TatriX typically we look for one other member of the Rust community to take a look and give it a stamp of approval. If we can't make that happen then we will clone the repo and test locally and verify everything is working as expected.
One thing I did notice was the blank Demo link. If there won't be a hosted Demo you might just remove that link, not like that would hold us up from merging but just an observation.
I'll make a note to take a look this weekend if no one from the Rust community can chime in.
FYI, this looks really cool, excited to dig in a little bit 😄

Awesome, thanks!

@TatriX I started looking at this and ran into an issue. I've never used rust so maybe I'm doing something wrong but I followed the instructions on your repo and got all the way to cargo run but then I hit some errors.
I was going to file an issue in your repo but it looks like Issues are turned off?
This is what I see:

Running on a Mac:
System Version:	macOS 10.13.3 (17D102)
Kernel Version:	Darwin 17.4.0

Oh, I don't know why issues was disabled.
The error is strange, maybe nightly is broken. I'm going to update and check it.

Yes, it's a mainstream problem
I'll find rustc version which works.

rustup toolchain install nightly-2018-04-01
rustup default nightly-2018-04-01
cargo run

This seems to work.

Hmm, seems like this was fine 3 days ago. Maybe stable pinned version of rust should be used in README and for TravisCI? Also, turning on daily cron on TravisCI may not be a bad idea. I too am a total rust noob btw 😄

It works fine now. Do cargo update if you want to fix an existing installation.

@TatriX this looks good to me.
I only have a couple comments:

The tests still fail for me. Seems to be around authorization:

failures:
    a_auth::register
    b_auth::login
    b_auth::login_and_save_credentials
    c_auth::current_user

With the addition of the /api/ on the routes the Postman collections provided won't work. Adding that to the route makes sense to me but I just wanted to mention that the postman collections won't work out of the box.

I'm good with adding this to the main page as is though. @EricSimons I'd say this one is ready to go 😄
Nice job @TatriX!

bump?


Fix content file load error

Proposed changes
Fix content-script.ts load error
Types of changes

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update or tests (if none of the other choices apply)

Checklist

[x] I have read the CONTRIBUTING doc
[x] I have added necessary documentation (if appropriate)
[x] Any dependent changes have been merged and published in downstream modules

@arndom


Faster utf8 validation

Which issue does this PR close?

Closes https://github.com/apache/arrow-rs/issues/6667
Rationale for this change
Improves performance for about 4-5% (on M1 Pro) on strings (plain encoding):
arrow_array_reader/StringArray/plain encoded, mandatory, no NULLs
                        time:   [740.81 µs 746.51 µs 752.11 µs]
                        change: [-5.8127% -5.2637% -4.6414%] (p = 0.00 < 0.05)
                        Performance has improved.
arrow_array_reader/StringArray/plain encoded, optional, no NULLs
                        time:   [743.62 µs 748.70 µs 754.14 µs]
                        change: [-4.2825% -3.6551% -3.0212%] (p = 0.00 < 0.05)
                        Performance has improved.
arrow_array_reader/StringArray/plain encoded, optional, half NULLs
                        time:   [633.43 µs 638.47 µs 643.71 µs]
                        change: [-5.1930% -4.5414% -3.8189%] (p = 0.00 < 0.05)

What changes are included in this PR?

Are there any user-facing changes?

I'm not sure that 5% really justifies an additional dependency, especially one that uses so much unsafe...

I'm not sure that 5% really justifies an additional dependency, especially one that uses so much unsafe...

Hm yeah wondering about that.
I think that 5% speed up for Parquet might be quite valuable though, given that it often translates in close to 5% faster query execution for queries where Parquet scan is a bottleneck (quite some DF benchmarks actually involving string data).

FWIW some other projects are using simdutf8 as well, like polars https://github.com/pola-rs/polars/blob/main/Cargo.toml#L77

I am not sure exactly the usecase here, but what about simply disabling utf8 validation for known good data?

https://github.com/apache/arrow-rs/issues/6701

I am not sure exactly the usecase here, but what about simply disabling utf8 validation for known good data?

Proposal: Add unsafe option to disable UTF8 validation on parquet read #6701

The "use case" of this PR is just that utf8 validation takes time, this PR improves the performance.

So what shall we do with this PR? Make it an optional opt-in feature of the parquet crate that people can enable if they want more performance?

I believe this PR is solely for performance enhancement. Introducing an optional opt-in feature deserves a separate PR.


[kimchi] choosing messagepack as our serialization format

Quickly went through the different serialization format that we can use, and msgpack seems to be one of the smallest/fastest/most well-supported format.
The only other contestant is cbor, which seems to be originated from msgpack, and has an RFC. People have mix feelings, and the two standards seem close enough, that I chose the one with the best Rust support.

Codecov Report

Merging #205 (5f38397) into master (be94fe4) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #205      +/-   ##
==========================================
- Coverage   88.91%   88.90%   -0.02%
==========================================
  Files          74       74
  Lines       18248    18249       +1
==========================================
- Hits        16226    16224       -2
- Misses       2022     2025       +3

Impacted Files
Coverage Δ

dlog/plonk-15-wires/src/index.rs
97.50% <ø> (ø)

circuits/plonk-15-wires/src/gate.rs
87.35% <100.00%> (+0.04%)
:arrow_up:

dlog/commitment/src/commitment.rs
91.90% <0.00%> (-0.36%)
: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 be94fe4...5f38397. Read the comment docs.

rebasing

@mrmr1993 isn't there an option in the settings to allow people to rebase through github when merging?

isn't there an option in the settings to allow people to rebase through github when merging?

There is, yes. We deliberately disabled it because rebases were causing us hell, we always prefer merges

ah interesting, what was the issue :o ? I always saw rebasing as cleaner as it doesn't pollute the commit history


New tiler service to show source based on positional selection

Implementing the following, with the purpose of calling from a client callback function on mouse button press
/**
 * @brief Shows a single source based on positional selection when the Tiler is currently showing all sources
 * If the Tiler is currently showing a single source, the show source timeout will be updated if set.
 * @param[in] name unique name of the Tiler to update
 * @param[in] x_pos relative to given window_width.
 * @param[in] y_pos relative to given window_height
 * @param[in] window_width width of the window the x and y positional coordinates are relative to
 * @param[in] window_height height of the window the x and y positional coordinates are relative to
 * @param[in] timeout time to show the source in units of seconds, before showing all-sources again...
 * A value of 0 indicates no timeout.
 * @return DSL_RESULT_SUCCESS on success, DSL_RESULT_TILER_RESULT
 */
DslReturnType dsl_tiler_source_show_select(const wchar_t* name,
    uint x_pos, uint y_pos, uint window_width, uint window_height, uint timeout);

This is implemented as above, except that calling dsl_tiler_source_show_select when a single source is currently shown, returns the tiler back to show all.  Toggling with the mouse seems to work more intuitively than extending the timeout for the same source on click.

reopening as this is incomplete.  Need to be able to query the window for current dimensions on select. Update example and documentation.

this is now addressed with https://github.com/canammex-tech/deepstream-services-library/issues/354


InternalServerError: operator is not unique: date - date_duration_t (from range)

EdgeDB Version: 4.5

Steps to Reproduce:
with
  period := range(
    <cal::local_date>"2024-01-01",
    <cal::local_date>"2025-01-05",
    inc_upper := true
  ),
  upper := range_get_upper(period)

select upper - <cal::date_duration>"1 day"

InternalServerError: operator is not unique: date `-` date_duration_t

FWIW I think the range_get_upper should give me the 5th, not the 6th anyways.
select range_get_upper(range(1.5, 7.5, inc_upper := true)); # => 7.5

select range_get_upper(range(
    <cal::local_date>"2024-01-01",
    <cal::local_date>"2025-01-05",
    inc_upper := true
)) # => "2025-01-06"

"The 6th? I just put in the 5th."
I know this differs from datetime logic, and I stand beside that.
I also know that this would be a breaking change, so I doubt it will be changed.

I was able to work around this by casting to str and back.
<cal::local_date><str>range_get_upper(period) - <cal::date_duration>"1 day"

Thank you @dnwpark! And thank you for the trailing comma change as well :heart:

will the fix work when the - operator is called with date and date? Like in:
test:edgedb> select Reservation { days:= range_get_upper(.dateRange) - range_get_lower(.dateRange) };
edgedb error: InternalServerError: operator is not unique: date `-` date
  Server traceback:
      edb.errors.InternalServerError: operator is not unique: date `-` date
test:edgedb> select Reservation { days:= <cal::local_date><str>range_get_upper(.dateRange) - <cal::local_date><str>range_get_lower(.dateRange) } ;
{
  default::Reservation {days: {}},
  default::Reservation {days: <cal::date_duration>'P2D'},
  default::Reservation {days: {}},
  default::Reservation {days: <cal::date_duration>'P10D'},
  default::Reservation {days: {}},
  default::Reservation {days: {}},
  default::Reservation {days: <cal::date_duration>'P1D'},
}
test:edgedb>

@MiroslavPetrik
Using this schema:
module default{
  type Reservation {
    single dateRange -> range<cal::local_date>
  };
}

After doing
insert Reservation {
  dateRange := range(
    <cal::local_date>"2024-01-01",
    <cal::local_date>"2025-01-05",
    inc_upper := true
  )
};

The query select Reservation { days:= range_get_upper(.dateRange) - range_get_lower(.dateRange) }; produces
{default::Reservation {days: <cal::date_duration>'P5D'}}

Having a similar error
operator is not unique: timestamp with time zone+ duration_t
in schema default := (.updatedAt ?? .createdAt) + <cal::date_duration>'60 days'
fixed similarly by default := <datetime><str>(.updatedAt ?? .createdAt) + <cal::date_duration>'60 days'
(also <cal::date_duration>'1440 hours' wrongly migrates an throws in runtime that smaller units than days are not allowed)


Request for debug version of VS Code to test widget

Could you provide a debug version of vs code that can pull from a custom location for the widget source, to help with debugging our custom widget for VS Code dark theme issues.
Note to team:

This is a request from AML team

We could improve the logging so that the hash of where the directory is going to be is logged. This would make it possible to predict where to put your index.js

What do you mean by where the directory is going to be? Is this going to be a local path?

@ansaxena1 I'm going to take a look at this quick (though it might be early next week) but we think that we can just log where we are looking for the widget sources locally. Then you can just drop your widget bits there to test them without a debug version of vs code.

Following up - has there been any progress with this?

@ansaxena1 Sorry again about the delay. I'm adding in a bit of tracing to our extension to surface the path that we are loading local files from.
https://github.com/microsoft/vscode-jupyter/pull/7085
When this PR is in then in tomorrow's vscode-insiders build of our extension this new logging should show up. This will point to the exact location on disk that we are looking for the name / version combo of the widget. If you drop the widget files at that path you should be able to test out a new version locally.

@ansaxena1, has this issue been resolved? If not, please reopen with additional details.


Simple SV normalization algorithm

Describe your algorithm in as few sentences as possible
Normalize the SV of a chart based on multiple BPM points. This is a common algorithm that could be useful for people who map multi bpm charts or for NSV implementation(?)

It was originally in my previously deprecated sv package. I found it too messy to maintain so I had to drop it.
I'll see if I have capacity to implement this though, but I can't guarantee it'll be soon.


can't pull modules from http://repository.jboss.org/nexus/content/repositories/thirdparty-releases/

I'm not 100% sure if this is a bug but it sure feels like one.
After adding the repo http://repository.jboss.org/nexus/content/repositories/thirdparty-releases/ to a maven settings.xml file, I can't for the life of me convince Ceylon to accept:
import maven:"junit:junit" "3.8.2-brew";

Or any other module in that repo. Now, perhaps I'm using settings.xml or --rep=aether:... wrong, but I have read all the docs I can find, and I don't see how.
@quintesse would you take a look at this please?

Don't have time to look at it today, but could you perhaps show the settings.xml you're using?

@quintesse I've tries lots of variations on:
<settings xmlns="http://maven.apache.org/SETTINGS/1.0.0"
    xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
    xsi:schemaLocation="http://maven.apache.org/SETTINGS/1.0.0
                        http://maven.apache.org/xsd/settings-1.0.0.xsd">
    <profile>
        <id>myprofile</id>
    <repositories>
        <repository>
            <id>mavenCentral</id>
            <name>Maven Central</name>
            <url>http://repo1.maven.org/maven2/</url>
        </repository>
        <repository>
            <id>jcenter</id>
            <name>JCenter</name>
            <url>http://jcenter.bintray.com/</url>
        </repository>
        <repository>
            <id>jboss</id>
            <name>JBoss</name>
            <url>http://repository.jboss.org/nexus/content/repositories/thirdparty-releases/</url>
        </repository>
    </repositories>
    </profile>

    <activeProfiles>
        <activeProfile>myprofile</activeProfile>
    </activeProfiles>

</settings>

Your settings.xml is wrong.. here this is working for in maven at least (did not try in ceylon):
<settings xmlns="http://maven.apache.org/SETTINGS/1.0.0"
    xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
    xsi:schemaLocation="http://maven.apache.org/SETTINGS/1.0.0
                        http://maven.apache.org/xsd/settings-1.0.0.xsd">
    <profiles>
    <profile>
        <id>myprofile</id>
    <repositories>
        <repository>
            <id>mavenCentral</id>
            <name>Maven Central</name>
            <url>http://repo1.maven.org/maven2/</url>
        </repository>
        <repository>
            <id>jcenter</id>
            <name>JCenter</name>
            <url>http://jcenter.bintray.com/</url>
        </repository>
        <repository>
            <id>jboss</id>
            <name>JBoss</name>
            <url>https://repository.jboss.org/nexus/content/repositories/thirdparty-releases/</url>
        </repository>
    </repositories>
    </profile>
</profiles>
    <activeProfiles>
        <activeProfile>myprofile</activeProfile>
    </activeProfiles>

</settings>

It worked in ceylon: remote=http://repository.jboss.org/nexus/content/repositories/thirdparty-releases/

It worked in ceylon: remote=http://repository.jboss.org/nexus/content/repositories/thirdparty-releases/

huum, i think it worked because my settings.xml and not because this conf

@DiegoCoronel Right, the Ceylon gradle plugin is able to pull dependencies from that repo. And I assume your settings.xml is pointing to the repo that the gradle plugin creates.

Yes, i can confirm that ceylon worked because my settings.xml

@gavinking i created a new project without gradle, with the settings.xml i fixed above its working for me in ceylon.   But its not working if i try remote=... in ceylon conf

@DiegoCoronel AFAIK, -repo=remote... is not expected to work with Maven repos. You're supposed to use -repo=aether:path/to/maven.setting.xml.

Huum, right, i did it using eclipse IDE Add Remote Repository .. let me fix that and try again

@gavinking huum, so i cant tell the URL to a maven repository ? Its required the maven settings.xml ? If thats true then its working for me as i told above about the fixed settings.xml you posted.

@DiegoCoronel you're telling me it works for you with the settings.xml I'm using??

If so, could you send me your project please?

mysample.zip

Damnit! I was missing the <profiles> element. And it failed silently! Grrrrr!

Thanks @DiegoCoronel.

Alright, so now, my next problem. Given this module.ceylon and settings.xml:
native ("jvm")
module test.delete "1.0.0" {
    import maven:"junit:junit" "3.8.2-brew";
    import maven:"org.wildfly:wildfly-servlet-feature-pack" "10.1.0.Final";
}

<settings xmlns="http://maven.apache.org/SETTINGS/1.0.0"
    xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
    xsi:schemaLocation="http://maven.apache.org/SETTINGS/1.0.0
                        http://maven.apache.org/xsd/settings-1.0.0.xsd">
    <profiles>
    <profile>
        <id>myprofile</id>
    <repositories>
        <repository>
            <id>mavenCentral</id>
            <name>Maven Central</name>
            <url>http://repo1.maven.org/maven2/</url>
        </repository>
        <repository>
            <id>jcenter</id>
            <name>JCenter</name>
            <url>http://jcenter.bintray.com/</url>
        </repository>
        <repository>
            <id>jboss</id>
            <name>JBoss</name>
            <url>http://repository.jboss.org/nexus/content/repositories/releases/</url>
        </repository>
        <repository>
            <id>jbossthird</id>
            <name>JBossthirdparty</name>
            <url>http://repository.jboss.org/nexus/content/repositories/thirdparty-releases/</url>
        </repository>
    </repositories>
    </profile>
    </profiles>

    <activeProfiles>
        <activeProfile>myprofile</activeProfile>
    </activeProfiles>

    <!--localRepository>build/maven-repository</localRepository>
    <offline>true</offline-->
</settings>

I get this error:

Error:(5, 5) ceylon: cannot find module artifact maven:org.wildfly:wildfly-feature-pack-10.1.0.Final(.car|.jar)
- dependency tree: test.delete/1.0.0 -> org.wildfly:wildfly-feature-pack/10.1.0.Final

I suppose that is because there is no .jar, only a zip. Is this one of these "POM-only" modules that #6485 refers to, or is it something else?

It seems so yes. The pom.xml contains a long list of dependencies and the .zip only seems to contain non-code files. Many JBoss Modules module.xml files for example.

Yes it's a "POM-only" or yes it's a "something else"? :-)

Anyway, the bug here is that the compiler shouldn't just give up with an error if it encounters a dependency that doesn't include any .jar. Maven modules can have dependencies on stuff that doesn't contain code.

That's indeed the issue I reported.

Note that to use swarm you don't need to import any of their modules. The ee APIs are enough.

@FroMage Alright, you're right, that makes it way simpler than what @DiegoCoronel was using. Closing this issue, since an issue for POM-only artifacts already exists.


⚠ Jenkins Plugins Artifactory has degraded performance

In fd34f67, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1259 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 6f0b21f after 26 minutes.


Great work! But I can't understand something

Several times I read the instructions, but I can not understand how to work in X64 mode. With I X86 had no problems. Everything works just fine. However, where are the classes X64Compiler, X64GpVar...?? I apologize for my stupid question and would be very grateful if you answer me.

@dial-up The X86 classes use 64-bit registers and such when on 64-bit platforms.

There is no such concept to make a difference between 32-bit and 64-bit anymore.
X86Assembler and X86Compiler both can generate code for x86 and x64 architectures. When using X86Assembler you need to take care of registers yourself, so you can use asmjit::x86::regs directly or architecture dependent  registers accessible through the code generator itself - X86Assembler.zax would result in eax or rax depending on the architecture.
Asmjit can also generate 64-bit assembly when running on 32-bit and vice versa.

Thanks!!!


Your Provider Contract Is Not Compatible With The Provider Codebase - Error unclear

After performing all steps to publish and verify contracts for consumer and provider dotnet repos, the final result is failure. Its a bit unclear to understand what is the mismatch here on provider side

.
Repos in Question
https://github.com/pactflow/example-bi-directional-provider-dotnet
https://github.com/pactflow/example-bi-directional-consumer-dotnet

@tl-madhulika-mitra should be sorted now https://github.com/pactflow/example-bi-directional-provider-dotnet/commit/cf01b00e167a80bbfe1993be87346b756c8ea579
you will need to checkout master of the repo - https://github.com/pactflow/example-bi-directional-provider-dotnet


Wind and related issues

What is needed for the integration of the wind simulation

[ ] File upload system

[ ] and input at which position what data is available

[ ] Accesibility to the database for positions or verticlespeed (.txt , .csv or other format)

[ ] Maybe console to access the matlab-connector

I have multiple questions:

[ ] File upload system

Do you want to do this via the front-end? Or do you want to specify a direct path to the simulation via the config? This might be easier to begin with, because for letting the user import the simulation via the frontend we need to communicate with the front end first obviously. This might need time to be implemented.

[ ] and input at which position what data is available

Didn't you want to import the data? Isn't the geographic data automatically included in some kind of header?

[ ] Accesibility to the database for positions or verticlespeed (.txt , .csv or other format)

What do you mean with that? We have the "standard" mongo db like stated here

[ ] Maybe console to access the matlab-connector

This will be a challenge because the whole concept is "input-free" up to this point. I think a solution could be a "remote" client which connects via TCP to the GS and there you can type something in a console which will be executed in a designated verticle.

What do you mean with that? We have the "standard" mongo db like stated here

this issue is old and at that point the database was not available yet

Do you want to do this via the front-end? Or do you want to specify a direct path to the simulation via the config? This might be easier to begin with, because for letting the user import the simulation via the frontend we need to communicate with the front end first obviously. This might need time to be implemented.

I think here is the first misunderstanding the file upload system is for the wind-data file only with a fixed path.
It should/has to be at the frontend because it will be needed in Kiruna and the person operating is maybe someone who doesn't really know how everything works. With the frontend it will be less likely to make an error.

[ ]  and input at which position what data is available

Didn't you want to import the data? Isn't the geographic data automatically included in some kind of header?

That is not what was meant by that. The previous wind-data files I have, they all have the data at different positon for example RE23 had a windspeed at column 5 and the heading at column 6 but RE25&26 had a Wind speed North at column 1 and wind speed East at column 3. maybe you see the problem

[ ]  Maybe console to access the matlab-connector

This was just wishful thinking


fix: use scratch buffer and lhs for keymap.normalize()

Fix https://github.com/neovim/neovim/issues/18303
This can be slower, I don't know how frequently this function is called as I don't use this plugin.

@arsham Can you test if this fixes https://github.com/neovim/neovim/issues/18303?

@arsham Can you test if this fixes neovim/neovim#18303?

It works, thank you!

LGTM! It looks reasonable. Thank you!


psi-plus: 1.5.1556-2 -> 1.5.1576

Motivation for this change
https://github.com/psi-plus/psi-plus-snapshots/releases/tag/1.5.1576
There should be properly tagged releases now (for example: https://github.com/psi-plus/psi-plus-snapshots/releases/tag/1.5.1556.1), so no need for suffixes like -2 anymore.
Things done

Built on platform(s)

[X] x86_64-linux
[x] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[X] Fits CONTRIBUTING.md.

@GrahamcOfBorg build psi-plus

@misuzu

There should be properly tagged releases now (for example: https://github.com/psi-plus/psi-plus-snapshots/releases/tag/1.5.1556.1), so no need for suffixes like -2 anymore.

Actually there was no such promise from the devs. And recent release showed that this is not true. At least not always. You can check latest releases:

There was no 1.5.1575.1 or 1.5.1575.2.

Actually the devs explicitly said that there will be no such sub-patch tags. 1.5.1556.1 is more like an accident.

You can check latest releases:

If it's not tagged - it's not a release. Doing raw revisions makes it difficult to automatically update packages via tools like nixpkgs-update (for example, https://github.com/NixOS/nixpkgs/pull/95557).

If it's not tagged - it's not a release.

This is true because the devs kind of have said that. Though I wouldn’t take this as a default for everything. I mean some projects may have weird versioning strategies or just not following them. Or they can have some occasional release made few years ago but being in active development. That’s not good for the project but still can be a valid cases for packaging concerns.


Fix crash when using Duration::MAX

Objective
If you set the ReactiveLowPower max wait to Duration::MAX, stuff panics. Fix that.
Solution
Wait forever if addition failed.

Ah my bad. No, I meant as a comment on the variants in UpdateMode.

Gotya, done.

bors r+


Skipfish doesnt find vulns in DVWA

I am trying to run Skipfish against DVWA, which I installed on a VM. But
Skipfish doesn't find any of the vulnerabilites DVWA provides. I can't
understand why, it reports two warnings "Limits exceeded, suppressed" and "Node
should be a directory, detection error?". So i put a maximum to the requests.
This is the command I am using:

skipfish -W /dev/null -o /root/Skipfish_DVWA_scan10 -C "security=low" -C
"PHPSESSID=*ZAPCookieInformation*" -l 3 -X /logout.php -X /setup.php -X
/security.php -r 4000 -m 5 -A admin:password
http://*ipv6ofVM*/dvwa/vulnerabilities/login.php

I also tried to point directly to the vulnerabilities, but neither it helped. I
think Skipfish should find some vulnerabilities, at least SQLinjection. But I
don't know what I am doing wrong. Does someone know? Thank you in advance...

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 17 Jul 2015 at 8:16

new to skipfish and i am having the same issue. i tried to change the URLs but no change.


M07-LAB16: Link Broken from MS Learn

Contact Details
No response
What happened?
Module: Implement Security and Compliance in an Azure Pipeline
Unit: 9 of 11
Description of issue:
Launch Exercise from the learning module is broken.
Repro steps:

From the learning module - unit 9 of 11 - Implement Security and Compliance in an Azure Pipeline, click the button Launch Exercise.
404 Not Found is seen on page with link https://microsoftlearning.github.io/AZ400-DesigningandImplementingMicrosoftDevOpsSolutions/Instructions/Labs/AZ400_M07_L15_Implement_Security_and_Compliance_in_an_Azure_Pipeline.html

Correction being requested:
Change the link to:
https://microsoftlearning.github.io/AZ400-DesigningandImplementingMicrosoftDevOpsSolutions/Instructions/Labs/AZ400_M07_L16_Implement_Security_and_Compliance_in_an_Azure_Pipeline.html
(lab number had changed from L15 to L16)
Lab
Lab 15 Implement security and compliance in Azure DevOps Pipelines
Relevant screenshots
paste here 😉
Do you want to help us? 👏

[ ] Create a Pull Request by following this project's Contribution Guide

Thank you @k-byun! We are working on updating all lab pages with the new Lab links. It should be fixed soon.

I confirmed the issue is now resolved. Thank you!


Router-Link doesn't support v-on:click

This doesn't fire the click event:
<router-link v-on:click="doSomethingCool" to="/">Home</router-link>

Is this intentional?
If yes, what is the recommended approach to have Vue click bindings in Router Links?

Home

http://vuejs.org/guide/components.html#Using-v-on-with-Custom-Events

@LinusBorg  What is the meaning of “a native event”? Thanks

A real Browser event, not a Vue event (which is created with $emit().
Further information available in our guide on the website.

For future reference, if you really need to use it as a Vue-event (not using the .native-approach), then you can do this instead:
<router-link to="/">
     <span v-on:click="doSomethingCool">Home</span>
</router-link>

...that would also react to a native click event, though :-P

@LinusBorg - just making sure I understand. The most recent example is reacting to a native event because the v-on directive  is attached to a native HTML element. In the OP's example, the event is a Vue event because the v-on directive is attached to a Vue component.

@rpkilby exactly right.

Olá, como que faço para deixar os links desabilitados e só ativá-los mediante alguma condição?

@jonecir esses pontos da documentação podem te ajudar!
att,
https://vuejs.org/v2/api/#v-cloak
https://vuejs.org/v2/guide/conditional.html

Hi ! So, if I do a @event.native=... with an event that I created, it won't trigger the event ?
And thanks a lot for the solution

<router-link v-on:click.native="doSomethingCool" to="/">Home</router-link>

http://vuejs.org/guide/components.html#Using-v-on-with-Custom-Events

Since v-on:click.native is deprecated now, is there any other approach to handle this without creating an wrapper div?

Since v-on:click.native is deprecated now, is there any other approach to handle this without creating an wrapper div?

Using @click.native works:
https://stackoverflow.com/a/42092519/3068344

Home

Would this not have problems with accessibility, e.g. keyboard user's tabbing through links?


[DRAFT] Typescript - Add BlockOptions and TextOptions

This will need a review from somebody who knows the library better.
Options found in documentation

Hey @mrchantey thanks for the contribution !
I wonder if we should make textContent,width,height optional too.
This would allows us to say BlockOptions extends TextOptions as we can define text propreties on the parent
What do you think ?

I agree with @swingingtom, all block options should be optional given the set API accepts the BlockOptions object as a parameter, and that param can be changing things unrelated to with or height entirely. Otherwise this is looking great.

Awesome, this is a change from the current implementation, but I agree it more accurately reflects the javascript.


proposition to add Mayaks 💚

hey,
I am Maya, a trans girl, living in Berlin. I love experimenting with code and spending time in nature. I have some projects like my own window manager (mostly dependent on wasm-based emulators, added some roms running on uxn), a custom 3D engine (still in an early phase but I am working on basing shaders on monochromatic dithering), and I am writing monitoring system for plants with raspberry pi. I also started doing some 3d art but I am not satisfied with it enough to share it yet 😅 I am also a vegan and happy labor union member.
My website is generally about all these things, my projects, my thoughts, and my lists of things I want to share.
I love your community and I would love to be a part of it, looks very cozy <3

Lovely website!
I read your notes on Uxn5, I didn't realize anyone cared for it, so I've only implemented enough of it to run Varvara's Unicycle, but if you'd like I could implement the screen, mouse and keyboard to work on canvas.

@neauoire
thank you :) and yeah I will be happy to add more implementation for uxn5


Content-Type and Accept headers not really under domain validation

https://github.com/woohoolabs/yin/blob/eae175718f66ff1c1e77d69847449a310541bd8e/src/JsonApi/Request/JsonApiRequest.php#L136
I've just found that this line above is not working as it should.
According to the official documentation:

preg_match() returns 1 if the pattern matches given subject, 0 if it does not, or FALSE if an error occurred.

You are testing that the returned value is equal to 0 but it should be equal to 1.
This mistake makes my API not returning HTTP code 415 and 406 on bad headers as the following images show you:

On the first picture, $jsonApi is just a wrapper of your JsonApi object:

EDIT:
Your regex is also broken because it should be able to capture application/vnd.api+json only but it does not.
 Quoting the related parts of the specification:

Servers MUST respond with a 415 Unsupported Media Type status code if a request specifies the header Content-Type: application/vnd.api+json with any media type parameters other than profile.

Servers MUST respond with a 406 Not Acceptable status code if a request’s Accept header contains the JSON:API media type and all instances of that media type are modified with a media type parameter other than profile.

All this means is that servers are not obliged to return an error, unless the application/vnd.api+json is present in one of headers and it is modified with a parameter other than profile.

Hi @kocsismate
Thanks for taking time.
First of all from my understanding of the spec, it's about Client Side Responsabilities to send Content-Type with the JSON API media type.

Clients MUST send all JSON:API data in request documents with the header Content-Type: application/vnd.api+json without any media type parameters.

I think it should be cool to get at least a 415 HTTP code in case you don't give application/vnd.api+json like in the screenshots above from my postman about the client request headers.
What do you think?

I'm very confused because actually your regex does not let me just put Content-Type: application/vnd.api+json without specifying a profile.
I have done that in your method to get negotiate work with or without a profile.
protected function isValidMediaTypeHeader(string $headerName): bool
    {
        $header = $this->getHeaderLine($headerName);

        // The media type is modified by media type parameters
        $matches = [];
        $isMatching = preg_match("/^.*application\/vnd\.api\+json\s*(;\s*([A-Za-z0-9]+)\s*=.*?)?$/i", $header, $matches);

        return $isMatching === 1 || (isset($matches[1]) && strtolower($matches[1]) === "profile");
    }

The spec does not treat about returning a 415 http code in case you don't give any profile and so, your regexp does not allow me to just give the json api media type from the client side

I think it should be cool to get at least a 415 HTTP code in case you don't give application/vnd.api+json like in the screenshots above from my postman about the client request headers.

You can implement this in your application, but doing so at the framework level would cause problems for anybody who wants to strictly follow the specification.

The spec does not treat about returning a 415 http code in case you don't give any profile and so, your regexp does not allow me to just give the json api media type from the client side

Unfortuantely, I don't exactly get what you wrote. Could you please elaborate? Anyway, what I can tell for sure is that application/vnd.api+json is a valid JSON:API content type. Just have a look at it in the tests: https://github.com/woohoolabs/yin/blob/master/tests/JsonApi/Negotiation/RequestValidatorTest.php#L275

@kocsismate my point is if I call negotiate, I'm expecting an exception to be raised or a 415 response code if I don't give application/vnd.api+json for Content-Type like on my first screenshots

my point is if I call negotiate, I'm expecting an exception to be raised or a 415 response code if I don't give application/vnd.api+json for Content-Type like on my first screenshots

Yeah, I see what you want, however this is against the specification. I remember that there used to be issues in the past when people wanted to explore a JSON:API directly from browsers which by default send the Accept: text/html. If I implemented your feature request, this would be impossible to do from now on.
However, as a workaround, you could provide your custom implementation for RequestInterface::validateAcceptHeader and RequestInterface::validateContentTypeHeader(), or it's also possible to even extend the RequestValidator class, and override its negotiate() method. But on the framework level, this change shouldn't be implemented in my opinion.
That being said, I'm closing this PR.

It's good for me. I don't wanna break the official spec :) :+1:


FreshRSS: Subscribe to feed always gives "Already Subscribed" error

ok, well I know that worked at some point. Sorry :/

Do you remember what URL you used? I think I know what is going on here, but I would like to test some more. (I don't believe that FreshRSS has the ability to detect an RSS feed from the metadata of a base domain, so you have to give the direct link to the feed)

Try this one: https://vincode.io/blog/feed
If FreshRSS doesn't support finding a feed from a page, we might want to look at using our Feed Finder to supplement FreshRSS in the account delegate.


Allow adjusting devspace config on run_default_pipeline

Is your feature request related to a problem?
As discussed in the Slack channel, with devspace 6.0.0-alpha.13 there is currently only create_deployment [name] --from-file file.yaml to adjust the helm values on a pipeline run. This works and is generally fine, however it forces a user to overwrite the default (in my case dev) pipeline to achieve a relatively simple goal.
I personally think it would be great if we could apply config patches on a devspace dev run without overwriting a default pipeline.
Which solution do you suggest?
@LukasGentele suggested a --set flag for run_default_pipeline which could adjust the config without duplicating the default pipeline definition. With this approach it would probably be a good idea to also allow --set-from-file (or similar) as to not have huge pipeline definitions when there are more things to adjust.
Which alternative solutions exist?
Alternatively allowing dev to define patches for a devspace dev run, similar to how profile patches work could work. I however I could see both options as being worthwhile to implement.
Additional context
https://loft-sh.slack.com/archives/CDSV29U85/p1650347170419019
/kind feature

@CiiDyR thanks for the issue! After internal discussions we have decided to add the --set flag to run_default_pipeline as well as run_pipelines that modifies the current config that will be passed to the pipeline execution.


keptn trigger sequence invalid helper text / unnecessary --sequence flag

@afzal442 @odubajDT FYI
The keptn trigger sequence help text is invalid. It leads me to believe that keptn trigger sequence delivery --project fulltour --service helloservice --stage qa is valid whereas in fact the command needs to be keptn trigger sequence delivery --sequence delivery --project fulltour --service helloservice --stage qa
Is the flag even required if we're passing the sequence name after keptn trigger sequence foo it seems like pointless duplication to have that flag.
Affects keptn 0.15.0 and higher.

Thank you @agardnerIT for the bug report! The --sequence flag should be considered optional


Get “insert_id” for one to one relationship in Flask, SqlAlchemy

Hey guys,
I tried seeking help on SO, but couldn't get any feedback.  Thought I'd reach out to the Flask community.
I'm trying to insert a new row through sqlalchemy. The parent table (Milestone) has a child table called Funding. Both tables share a relationship through a column called milestone_id. It is a one to one relationship.
I have looked it up, but I can't figure out how to reference the milestone_id when INSERTING a new row in the Funding table. The parent ID is an autoincrement. I am using Flask and SqlAlchemy.
models:
class Milestone(db.Model):
    __tablename__ = "**************"

   milestone_id = db.Column(db.Integer, primary_key=True)
   company_id = db.Column(db.Integer, db.ForeignKey('stlines_startups.company_id'))
   milestone_date = db.Column(db.Integer)
   snapshots = db.relationship('Snapshot', uselist=False, primaryjoin='Milestone.milestone_id==Snapshot.milestone_id', backref='milestone')
   fundraising = db.relationship('Funding', uselist=False, primaryjoin='Milestone.milestone_id==Funding.milestone_id', backref='milestone')

   def __init__(self, milestone_id, company_id, milestone_date, snapshots = [], fundraising = []):
    self.milestone_id = milestone_id
    self.company_id = company_id
    self.milestone_date = milestone_date
    self.snapshots = snapshots
    self.fundraising = fundraising

class Funding(db.Model):
    __tablename__ = "**************************"

   funding_id = db.Column(db.Integer, primary_key=True)
   funding_type = db.Column(db.Text)
   funding_message = db.Column(db.Text)
   funding_amount = db.Column(db.Integer)
   milestone_source = db.Column(db.Text)
   company_id = db.Column(db.Integer, db.ForeignKey('stlines_milestones.company_id'))
   milestone_id = db.Column(db.Integer, db.ForeignKey('stlines_milestones.milestone_id'))
   user_id = db.Column(db.Integer)
   funding_timestamp = db.Column(db.Integer)

def __init__(self, funding_id, funding_type, funding_message, funding_amount, milestone_source, milestone_id, company_id, user_id, funding_timestamp):
    self.funding_id = funding_id
    self.funding_type = funding_type
    self.funding_message = funding_message
    self.funding_amount = funding_amount
    self.milestone_source = milestone_source
    self.milestone_id = milestone_id
    self.company_id = company_id
    self.user_id = user_id
    self.funding_timestamp = funding_timestamp

Alchemy Query:
@app.route('/_add_funding')
def add_funding():
    funding_type = request.args.get('funding_stage', '', type=str)
    funding_message = request.args.get('funding_message', '', type=str)
    funding_amount = request.args.get('funding_amount', 0, type=int)
    milestone_source = request.args.get('milestone_source', '', type=str)
    milestone_date = request.args.get('milestone_date', '', type=str)
    company_id = request.args.get('company_id', '', type=int)

    milestone_date_final = datetime.datetime.strptime(milestone_date, '%B %d, %Y')

    ''' In this line, I try to reference milestone_id with new_milestone.milestone_id, but nothing shows up in the database '''
    new_funding = Funding('', funding_type=funding_type, funding_message=funding_message, funding_amount=funding_amount, milestone_source=milestone_source, company_id=company_id, milestone_id=new_milestone.milestone_id, user_id='1', funding_timestamp=milestone_date_final)
    new_milestone = Milestone('', company_id=company_id, milestone_date=milestone_date_final, snapshots=None, fundraising=new_funding)
    db.session.add(new_milestone)
    output = new_milestone.milestone_id
    db.session.commit()

    return jsonify(result=output)

How can I tell SqlAlchemy to use the auto generated milestone_id from the milestone table when insert the funding information in the funding table? Should these be two separate queries?  When I do them as two separate queries I get the error: sqlalchemy.orm.exc.ObjectDeletedError: Instance '<Milestone at 0x39ebeb0>' has been deleted, or its row is otherwise not present.
Any help would be greatly appreciated, thanks guys :)

you need to db.session.flush() before you can access the ID
also, this is not the proper place for support questions.

Hey,
thanks for getting back to me ... sorry for posting in the wrong spot.  I took up your advice, but I'm still getting an error:
UnboundLocalError: local variable 'new_milestone' referenced before assignment
here's the updated code:
@app.route('/_add_funding')
def add_funding():
    funding_type = request.args.get('funding_stage', '', type=str)
    funding_message = request.args.get('funding_message', '', type=str)
    funding_amount = request.args.get('funding_amount', 0, type=int)
    milestone_source = request.args.get('milestone_source', '', type=str)
    milestone_date = request.args.get('milestone_date', '', type=str)
    company_id = request.args.get('company_id', '', type=int)

    milestone_date_final = datetime.datetime.strptime(milestone_date, '%B %d, %Y')

    new_funding = Funding('', funding_type=funding_type, funding_message=funding_message, funding_amount=funding_amount, milestone_source=milestone_source, company_id=company_id, milestone_id=new_milestone.milestone_id, user_id='1', funding_timestamp=milestone_date_final)
    new_milestone = Milestone('', company_id=company_id, milestone_date=milestone_date_final, snapshots=None, fundraising=new_funding)
    db.session.add(new_milestone)
    db.session.commit()
    db.session.flush()
    output = new_milestone.milestone_id

    return jsonify(result=output)

any ideas?  Thank you!

Remove the milestone_id=new_milestone.milestone_id and link your two objects via the relationship instead. That's basic sqlalchemy; you can probably find an example for this in the SA docs

is there a section in the docs, you could at least refer me to?  I'm seeing a lot of stuff on building the model, which I have already done, but nothing on inserting with a relationship.

e.g. http://docs.sqlalchemy.org/en/latest/orm/basic_relationships.html
# create parent, append a child via association
p = Parent()
a = Association(extra_data="some data")
a.child = Child()
p.children.append(a)

@JeanClaude3 While I understand your situation that you're not getting any help on SO, posting your question on Flask's bug tracker, especially one that has nothing to do with Flask itself, is doing the vastly understaffed maintainers of Flask (e.g. me) a strong disservice. The tracker has already a lot of open, unanswered and unresolved tickets, and this one is just adding fuel to what is already a huge bonfire.
In other words, please don't ask support questions on the issue tracker.


COPY FROM returns a SQLActionException with no details

While attempting to re-import previously backed up data, I got this error. Would help if there was at least a descriptive error message.
cr> copy posts from 's3://(valid path...)/posts_*' with (compression='gzip', overwrite_duplicates=true);

SQLActionException[null]

hi @jeffnappi
looks like there is an Exception without a message thrown. We'll have to take a look at this.
additionally it seems like the verbose mode of crash is broken :(
I've opened an issue in the project repo: https://github.com/crate/crash/issues/55
you could try to use the REST API to get more information
https://crate.io/docs/en/latest/sql/rest.html?highlight=error_trace#error-handling
best, christian

@chaudum thanks - I'll keep you posted if I encounter it again.

ok, will close it then.

@jeffnappi the crash bug has been fixed in 0.11.4 https://github.com/crate/crash/issues/55


Observation refactor to bootstrap-4

Merges recent changes on master into bootstrap-4.

Coverage increased (+0.1%) to 84.921% when pulling 5de1d922bb0c40639e63b570690039115594a4eb on observation-refactor-to-bs4 into 526da3afa742595c737432e57032db15bb22770e on bootstrap-4.

This merge looks reasonable to me, but I'd love to have either @JoeCohen or @nimmolo look it over and merge if it looks good to them.

My apologies if you received multiple notices about my suggested change; I haven't wuite figured out how to use GitHub Review.
I'd like to get this merged soon so that it doesn't interfere with bootstrap-4 development. Because I'll be tied up or away most of the week, I might merge it tonite or tomorrow a.m. if there are no additional comments.


Update RBAC page for other backends

The RBAC page at https://pulumi.io/reference/service/roles-and-access-controls/ RBAC only works for Team Edition and GitHub. The page should be updated to include details about Team and Enterprise editions and about each backend (including SAML) works with regards to user permissions

This is fixed as part of https://github.com/pulumi/docs/pull/1505.


Update bosh-cli go module to v7

bosh-cli recently updated bosh-agent to v2 and to get that bosh-cli needs to be updated to v7 in this repo.

@wayneadams I'm not sure if I'm supposed to create a new depnames-*.txt file since this PR is changing dependency. Are you able to advise what to do?


Allow specifying cmake setup directory

Description
Please include a brief summary of the changes, relevant motivation and context.
Allow specifying which directory to store cmake build cache, which makes user defined docker buildx workflows more flexible.
Fixes # (issue)
Type of change

[ ] Documentation change (change only to the documentation, either a fix or a new content)
[ ] Bug fix (non-breaking change which fixes an issue)
[x ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Infra/Build change
[ ] Code refractor

Changes
Please list the changes introduced in this PR:

Change A
Change B

Checklist:

[x ] I have read and followed the contributing guidelines
[x ] The functionality is complete
[na] I have commented my code, particularly in hard-to-understand areas
[na ] I have made corresponding changes to the documentation
[x ] My changes generate no new warnings
[na] I have added tests that prove my fix is effective or that my feature works
[na] New and existing unit tests pass locally with my changes

Hello @ryxli, could you please sign your commits (https://github.com/NVIDIA/TransformerEngine/blob/main/CONTRIBUTING.rst#sign-your-work)?

/te-ci


Controlled DropdownMenuSub is not working

Current Behavior
I'm trying to control the DropdownMenuSub component, so that the secondary dropdown is opened on click, not on hover. Setting the controlled props open and onOpenChange does not seem to have any effect.
Expected Behavior
I'd expect either <DropdownMenuSub onOpenChange={() => false}> and/or <DropdownMenuSub open={false}> to prevent the secondary menu from opening.
(As a next step, I'd like to make the secondary menu open on click events, not on hover events. Guidance on the best way to do that would also be appreciated.)
Steps To Reproduce
The following does not prevent the secondary dropdown menu from opening:
<DropdownMenu>
    <DropdownMenuTrigger>Open menu</DropdownMenuTrigger>
    <DropdownMenuContent>
      <DropdownMenuSub open={false} onOpenChange={() => false}>
        <DropdownMenuSubTrigger>Option 1</DropdownMenuSubTrigger>
        <DropdownMenuSubContent>
          <DropdownMenuItem>Submenu option 1</DropdownMenuItem>
          <DropdownMenuItem>Submenu option 2</DropdownMenuItem>
        </DropdownMenuSubContent>
      </DropdownMenuSub>
      <DropdownMenuItem>Option 2</DropdownMenuItem>
    </DropdownMenuContent>
  </DropdownMenu>

Link to Reproduction / Stackblitz (reproduction template)
No response
More Information
No response

So with controlled in this case, it will still update internally via internal state changes (triggered by the event on the trigger). You can prevent this default behavior by calling e.preventDefault on the different mouse events of the DropdownMenuSubTrigger!

Gotcha, thanks! The following disabled={true} (which feels like a bit of a hack) also worked for me:
<DropdownMenu>
  <DropdownMenuTrigger>Open menu</DropdownMenuTrigger>
  <DropdownMenuContent>
    <DropdownMenuSub bind:open={subMenuOpen}>
      <DropdownMenuSubTrigger
        disabled={true}
        on:click={() => (subMenuOpen = !subMenuOpen)}
      >
        Option 1
      </DropdownMenuSubTrigger>
      <DropdownMenuSubContent>
        <DropdownMenuItem>Submenu option 1</DropdownMenuItem>
        <DropdownMenuItem>Submenu option 2</DropdownMenuItem>
      </DropdownMenuSubContent>
    </DropdownMenuSub>
    <DropdownMenuItem>Option 2</DropdownMenuItem>
  </DropdownMenuContent>
</DropdownMenu>


DOC: Improve README sections

Improve description wordings
Clarify installation steps to account for availability of the extension in the extensions manager
Move "Installing the extension" to the top
Add "Interface" section
Add "Maintainers" section

To make the extension more useful and easier to use, where some additional suggestion:

while the collage include the example of analysis, user interface and input dataset structure is relevant in the context of the extensions manager. I don't think it should be included in the README, instead I suggest you include each independently.
an example of input dataset could be provided (including the annotation file) could be provided.

Hi @jcfr, thank you for your suggestions. I just added those changes to the readme and added an example dataset (I used a Pixar Brain MRI dataset w brain mask segmentations from openFMRI and generated by hdbet, as it might be tricky for us to release the original one w brain malignancies)


Is this project abandoned?

Does anyone know if this project has been abandoned?

I'm working on another framework and I have no idea to make this framework better. So, maybe yes.
I'm not recommend you to use this framework for production. But if you want to use this, you can submit any question, bug, or idea, I will respond it ASAP.

Let me just highlight what i loved about this framework

the way file uploads are handled
not too many dependencies on third party packages
lightweight micro framework

However i wished Psr standards were considered during development
Like for the http layer loading loging etc
I would love to be notified when your new framework is ready or maybe a link to the github repo so i can follow.
Thanks

Thank you for loving this framework. Yes, that was my purposes when I create this framework. :smile:
No, I'm not about creating another framework. I mean, I'm working fulltime on app that using another (existing) framework.
I will enhance this framework sometime when I have project that not require me to use another framework. But I don't know exactly when. Maybe about few months ahead.
For your suggestion, I will learn onto it. I want implement them as simple as possible and as small as possible.
If you have any idea to implement them, issues and PR are welcome :+1:


[libjuice] update to 1.3.1

[X] Changes comply with the maintainer guide
[X] SHA512s are updated for each updated download
[ ] ~The "supports" clause reflects platforms that may be fixed by this new version~
[ ] ~Any fixed CI baseline entries are removed from that file.~
[ ] ~Any patches that are no longer applied are deleted from the port's directory.~
[X] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[X] Only one version is added to each modified port's versions file.

All features are tested successfully in the following triplet:
x86-windows
x64-windows
x64-windows-static

The usage test passed on x64-windows (header files found):
libjuice provides CMake targets:

  # this is heuristically generated, and may not be correct
  find_package(LibJuice CONFIG REQUIRED)
  target_link_libraries(main PRIVATE LibJuice::LibJuice)


Denylist https://github.com/gemini-testing/png-img for indexer

https://github.com/gemini-testing/png-img seems to cause matches against it instead of libpng.

(from https://github.com/google/osv-scanner/pull/621)

Hmm... there's probably a more general approach here where we ignore these "third_party" and similar folders from the initial indexing, could reduce a lot of false positives.
Also detecting a non C/C++ package manifest file and avoiding these repositories when indexing will help.

Hmm... there's probably a more general approach here where we ignore these "third_party" and similar folders from the initial indexing, could reduce a lot of false positives.

+1 that's a great idea! In the meantime though, does it seem reasonable ot just remove the config for this repo and delete all relevant index entities?

Yep can do

png-img has now been removed from the indexer config and datastore hashes.


history import imports lines it shouldn't

My ~/.bash_history file looks like this:
#1650679383
git co master
#1650679389
git branch -D exclude-take-3
#1650679393
git pull
#1650985607
ls testing

And atuin imported all of the command times (#1650985607, for example) as commands. I'm honestly not exactly sure what part of my config tells bash to write those comments - I'm guessing it's HISTTIMEFORMAT? Here's the possibly-relevant section of my bashrc:

# append to history instead of overwriting
shopt -s histappend
# Save multi-line commands as one command
shopt -s cmdhist

# Before each bash prompt, write to history and read from it. This
# makes multiple terminals sync to history
# PROMPT_COMMAND='history -a; history -n'

# Huge history. Doesn't appear to slow things down, so why not?
HISTSIZE=500000
HISTFILESIZE=100000

# Avoid duplicate entries
HISTCONTROL="erasedups:ignoreboth"

# Don't record some commands
export HISTIGNORE="&:[ ]*:exit:ls:bg:fg:history:clear"

# Use standard ISO 8601 timestamp
# %F equivalent to %Y-%m-%d
# %T equivalent to %H:%M:%S (24-hours format)
HISTTIMEFORMAT='%F %T '

So there's lots of imported commands like:
sqlite> select * from history where command regexp '#\d+$' limit 5;
76b773158aff4b4aaff2be7c4316976b|1650985004731089000|-1|-1|#1644242519|unknown|98099ebaa5c247cda923bba31c286cf5|lexeme:llimllib
12b1228a89d4441baaff77f8297b99c7|1650985002731116000|-1|-1|#1644242523|unknown|2106ff48b631471fa254ec9caafbdce8|lexeme:llimllib
139f80d3333a4678bb112e24c7a0bfc8|1650985000731131000|-1|-1|#1644242526|unknown|9d338cc1c9e94114843304fe4f8a1445|lexeme:llimllib
287b5f41f68a4a538a878a1e10edf7f3|1650984998731152000|-1|-1|#1644242816|unknown|73ddb792ea7349f3b30940db761d2c0d|lexeme:llimllib
cf5ad019bb59485989c59a2be289c10a|1650984996731168000|-1|-1|#1644243276|unknown|ff651a6299d247df9b863a20f9a2e14a|lexeme:llimllib

I deleted these rows with this command:  delete from history where command regexp '^#\d+$';
However, atuin also imported any command that ended with a backslash incorrectly, including the next lines' timestamp. My ~/.bash_history has lines like:
dig +trace example.com\
#1644269003

which atuin imported as a line continuation. I'm not actually sure why my bash history has lines like that, but it does. Fortunately, there were only 4 of them in a 45,000 command history and none were vital, so I just deleted them with delete from history where command regexp '#\d+$' limit 5;. It's possible that there's no sensible way for atuin to avoid importing these lines this way, just reporting it since I was cleaning up the db.

Yeah, we've seen similar issues occur in ZSH(#100).
For bash, we assume that the user does not have any timestamp support (as far as I understand, out of the box bash does not support timestamps?).
I guess we can theoretically support parsing some HISTTIMEFORMAT environment variables and processing accordingly (we do similar in zsh). In the general case it's impossible though

The bash manual says:

If the HISTTIMEFORMAT is set, the time stamp information associated with each history entry is written to the history file, marked with the history comment character. When the history file is read, lines beginning with the history comment character followed immediately by a digit are interpreted as timestamps for the following history entry.

Should the import process ignore lines beginning with the history comment character? (Is this configurable or always #? no idea!)
Also if this is not helpful and you just don't plan to deal with this, no worries and feel free to close the issue.
And finally, I'm really excited to have my history in sqlite, so thanks for the software!

Should the import process ignore lines beginning with the history comment character? (Is this configurable or always #? no idea!)

I think in an ideal world, we would parse them. Although I can see this being difficult 😅

Also if this is not helpful and you just don't plan to deal with this, no worries and feel free to close the issue.

I'd like us to deal with this at some point, but I'm not sure we'll get to it any time soon. I'll leave it open for now!

And finally, I'm really excited to have my history in sqlite, so thanks for the software!

Aww yay, hope you like it!

'histchars'
     Up to three characters which control history expansion, quick
     substitution, and tokenization (*note History Interaction::).  The
     first character is the HISTORY EXPANSION character, that is, the
     character which signifies the start of a history expansion,
     normally '!'.  The second character is the character which
     signifies 'quick substitution' when seen as the first character on
     a line, normally '^'.  The optional third character is the
     character which indicates that the remainder of the line is a
     comment when found as the first character of a word, usually '#'.
     The history comment character causes history substitution to be
     skipped for the remaining words on the line.  It does not
     necessarily cause the shell parser to treat the rest of the line as
     a comment.

so yes, the stupid history comment character can be changed, computers are fractally awful

I'll see if I can write a parser for this. If bash can do it, I'm sure we can too (even if it's a lil bit lossy)

I'm going to mention #167 here as a dupe, and also provide a sqlite one-liner to try and update commands with their timestamps. It seems to have worked for me, but your mileage may vary.
I also suspect that I've sort of broken things for myself as far as syncing is concerned until deletes are implemented...
UPDATE history
SET `timestamp` = COALESCE(
    (SELECT
        (CAST(LTRIM(b.command, "#") as INTEGER) *<PHONE_NUMBER>) as newtimestamp
    FROM history b
    WHERE
        b.rowid = (history.rowid - 1)
        AND SUBSTR(b.command, 1, 1) == "#"
    ORDER BY b.rowid ASC LIMIT 1
    ),
    `timestamp`
);

DELETE FROM history WHERE substr(command, 1, 1) == "#"

This is fixed by #747

is this really related to the format of .bash_history?

I think that if the third character of histchars were something other than #, then .bash_history would record comments starting with something other than #.
But agreed that if you're crazy enough to do that, atuin doesn't have to support you. Really just... don't do that to yourself.

was the \ a literal?

I'm not sure, but I don't think that they always are. Here's an example from my ~/.bash_history of continuation characters in a subshell:
#1652445647
ENI=$(aws ecs describe-tasks --task $TASK --cluster $CLUSTER | \
  jq -r '.tasks[0].attachments[0].details[] | \
         select(.name == "networkInterfaceId").value')

here's an example of a simple command taht definitely just has a continuation character:
#1656510510
sh -c 'curl -fLo "${XDG_DATA_HOME:-$HOME/.local/share}"/nvim/site/autoload/plug.vim --create-dirs \
       https://raw.githubusercontent.com/junegunn/vim-plug/master/plug.vim'

here's an example of continuation characters, but inside a quoted multiline filter string:
#1664588032
ffmpeg -ss 1:57 -to 2:00   -copyts   -i /tmp/ytgif_cache/video_httpswwwyoutubecomwatchvnrqxmQruto.webm   -filter_complex "\
  [0:v] fps=10, \
  scale=640:-1, \
  split [a][b], \
  [a] palettegen [p], \
  [b][p] paletteuse, \
  drawtext=borderw=1:bordercolor=black:fontcolor=white:fontsize=30:x=(w-text_w)/2:y=(h-text_h)-10:text=🔥 flames 🔥"

Maybe those are all literals - a subshell $( is a type of quote as far as bash is concerned, I guess?

See https://github.com/ellie/atuin/pull/747#discussion_r1158660221.


PAM Self-Hosted and Creating Accounts

Our organization is using PAM self-hosted and I'm currently looking at this provider as an option to writing secrets to CyberArk safes via terraform.  The support for PAM self-hosted looks promising, but the only resources I see so far as far as account creation goes looks to be specific to aws, db etc..where as our organization supports multiple platform ids etc.
Our workflow is basically provisioning Confluent Cloud resources such as API Keys and Secrets via terraform and we need those secrets written to the CyberArk safe for future retrieval.
We would like a way (a resource) that just creates a basic account where I can set a username and password.  Using @infamousjoeg example https://github.com/infamousjoeg/cyberark-examples/blob/master/python_example.py would be a good baseline as we've been able to replicate this in Postman for writing secrets successfully.
As this terraform project seems to be the only terraform support for what we need, we've also considered using example https://github.com/infamousjoeg/cyberark-examples/blob/master/python_example.py and figuring out how to get the secrets created via terraform into this script to write to CyberArk, which seems like a burden.  It would be much easier to write the secrets via this provider to keep the state files up to date.
@infamousjoeg @szh

CC @imheresamir

@imheresamir, I want to confirm from you if the request is viable or not.  If it is, would you happen to have a timeline to when it could be available? We just want to decide if we need to go a different direction, try to make the update ourselves etc...

Hi @karand1979,
You are correct that this provider currently only supports creating AWS, Azure, and DB account types. The issue is that different account types have different sets of required and optional attributes, so there would need to be code changes to add support for the specific account types, of which there are many.
Could you let us know which specific account types you are interested in managing with Terraform? Unfortunately I can't provide a timeline for adding these at the moment, but I have reached out to our Product team and will update here if/when there is a timeline for this request.
Best regards,
Samir

@imheresamir, thank you for your reply.  We are basically just looking for a basic account type that allows basic information like username and password with really no additional account properties.  This would pretty much replicate the add_cyberarkAccount method's accountBody structure in this example.  The secretType would always be password.  I've attached a screenshot of what it looks like when I add through the UI.

This would basically be just like the Pam Self Hosted API call to create an account
https://docs.cyberark.com/pam-self-hosted/14.0/en/content/webservices/add account v10.htm?tocpath=Developer|REST APIs|Accounts|_____5


Add option to load previously-saved model and continue training

Great project! One feature I think would add a lot to it--it would be valuable to be able to load a previously-saved model and continue training.

It would be helpful! I guess we need a little change in train.py.

Solved with #27.


failed to replace processor successfully

It's caused by the former refactor of Task and Executor

Interesting. We have already test case for changing a processor (description, paralism, etc.). Better add a test case along with the fix

But we don't check for clock advance

Will create a test case individually https://github.com/gearpump/gearpump/issues/1655


Issue plotting gx1v7 grid with "spstere" projection

Issue:
When plotting regional masks on a gx1v7 grid (POP) using proplot, the regions do not show up when using a polar projection. The same issue persists when just using matplotlib and when the masks are converted to a 180X360 grid. If I start using a 180x360 grid the problem does not arise.
Code sample: see below
https://gist.github.com/ZZephyrS/0a76c0fa934b25c5c3fcb0fac7d3ef98
Actual result vs. expected result
Actual result: masked regions are not plotted when using a polar projection but they are plotted in a platte carree projection
Expected results: the regions are plotted regardless of the projection.

Thanks for the complete example -- unfortunately this wasn't a proplot/matplotlib issue but an issue with your dataset.  If you change cmap to something that is non-white bottom color (e.g. cmap='viridis') you can see this even in the Plate Carrée example:

I think it just so happened that the matplotlib pcolormesh algorithm drew the white grid cell before (underneath) the colored ones in the Plate Carrée example, and after (on top of) the colored ones in the polar projection example.
Luckily, it looks like if you filter out the comparatively small values in your masks, e.g. with:
threshold = 1.0
atl_mask_filtered = atl_mask.values.copy()
atl_mask_filtered[atl_mask_filtered < threshold] = np.nan

then this issue disappears:

Actually, it looks like a similar issue persists in proplot but not matplotlib:

You can fix it with e.g.:
mask = atl_mask.values[:-1, :-1]
ax.pcolormesh(lon, lat, mask, ...)

When matplotlib's pcolormesh encounters pcolormesh coordinates with N x-coordinates, M y-coordinates, and an M x N data array, it simply trims the last row and column in your data array to make an M - 1 x N - 1 array -- because the coordinates accepted by pcolormesh are supposed to represent the grid-cell edges, not centers.
Instead of trimming the last row and column, in this situation proplot tries to guess the correct graticule by taking averages of the coordinate vectors (see the last bullet point on this page, and edges2d, edges). For your dataset, this also seems to mess up the pcolormesh algorithm.
However, when you have really high resolution data, this distinction doesn't really matter -- you can safely trim the mask edges and "pretend" your longitude-latitude coordinates are for the grid cell edges. In the future I might make this a configurable option, e.g. trim_edges =True.


Amoled theme

can you please add amoled theme in the app?

This has been implemented in the latest release v0.9.0: https://github.com/sakethpathike/Linkora/releases/tag/release-v0.9.0


Annotations are missing

I was comparing the results of instrumentation and on Stackdriver Tracing I can see Annotations but on AWS X-Ray unfortunately I can't. The docs for the X-Ray SDK seem to imply that it supports annotations though

Never mind, I was hung up on "Description" which I don't see mentioned in the AWS SDK docs. My comparison was Stackdriver vs AWS
Stackdriver with the annotation descriptions

AWS X-Ray without them


Scanning to find the R10

Hey!
Love the work and excited about the progress. I recently got the R10 (which the quick guide talks about QRing) and tried using the scan functionality and it doesn't seem to find it.
I verified that gadgetbride can find the ring - so I am not sure why this client can't.
Let me know what I should try or test.
Tested with: Macbook Air M2:
me@MacBook-Air % colmi_r02_util scan
Found device(s)
                Name  | Address
--------------------------------------------

Known issue, you can pass the --all flag to the scan command and you should see a COLMI R10

Fixed now, I can find my R10


E_INVALIDARG 0x80070057 at VS2022 with .NET 6.0

hi guys,
i installed the tool V1.2.0 into VS 2022 on Win10 Pro, the tool was installed succesfully.
i have more projecst into the solution i.e WPF, C# Libraries, WPF Libraries, i'm using .NET 6.0
after press the menu item Tools > Protected Using Obfuscar propmt the Window Error Message Box, i tried with all projects, prompt the saem error at all projects.
Wrong parameter Exception HResult E_INVALIDARG 0x80070057
could you help me

There is no V1.2.0 from this repo. You probably use another tool and should report to its authors.


Add support for Microsoft.Extensions.Caching.Memory

I just copy from CacheManager.SystemRuntimeCaching and change some things.
Microsoft.Extensions.Caching.Memory
my code
If you want to add it, I can submit a PR.

Sure, submit a PR! Looks very good :+1: ;)
Please, just make sure it does pass the unit tests, so at least add it to https://github.com/MichaCo/CacheManager/blob/dev/test/CacheManager.Tests/BaseCacheManagerTest.cs#L297 which is used in many tests.

will work on your PR #68 next, whenever I find some more time ~~

PR #68 has now finally been merged. Sorry for the delay and thanks a lot again!
I also changed the deps to use the released dotnet core bits


Run without NVIDIA Driver?

Would there be any way to run the program without a NVIDIA Driver and NVIDIA GPU?

Hi @ultramyslixs, thanks for your question. We strongly recommend running on a GPU machine since the computation would be heavy for CPUs. But anyway, I will be fixing this problem and make it compatible with a CPU machine tonight. I will let you know when it's ready to use.

Ok, thank you!

Hi @ultramyslixs, PyTorch-CPU mode is now supported! You can try out on your local machine without any GPU cards. But please note that you still need PyTorch installed if you run on CPU mode. You can simply follow the instructions in the README file and the code will automatically detect whether there will be any GPU cards available. If there is no GPU, the code will run on the CPU automaticlly.

Thanks! Running this program currently and it looks really good! Is there any way to maximize the image while the image is being generated? At the moment the image is about 1 inch wide and 1 inch long.

Hi @ultramyslixs, I have made a minor update on the adjustable display window size. You can try again and adjust the window size as you want.

thank you so much!