API for extension destructors

Following an interesting thread: https://sqlite.org/forum/forumpost/abf074d0d5ef87e1, we should consider extending the API in order to allow extension/user-defined function developers to provide destructors for their modules. Such destructors would automatically get called once the database connection to which they're bound gets closed.

Hey, curious if any progress has been made here. I need it for cr-sqlite to be able to fully clean itself up when enabled on Turso. Otherwise Turso ooms since each connection leaves around prepared statements :(

No progress so far, but I'll take another shot at it soon, thanks for bumping!

@tantaman I'm reading through https://www.sqlite.org/c3ref/create_function.html and I need clarification -- why isn't xDestroy from sqlite3_create_function_v2 not enough for that purpose? I also checked https://www.sqlite.org/c3ref/create_module.html and modules allow you to load a destructor as well. Do I understand the issue correctly that the dangling prepared statements are created per connection, so what we actually need is a handler that gets called each time a connection is closed?
If so, I have the following conclusions:

In case of https://github.com/libsql/sqld, we should be robust and use https://www.sqlite.org/c3ref/next_stmt.html to go over all dangling statements, force-finalizing them right before calling sqlite3_close. That guards us against all kinds of leaks in extensions we load. /cc @MarinPostma @penberg I'd appreciate your thoughts here
Maybe we want a generic sqlite3_close_hook instead, that allows users to register their own hook that gets called on close? Then the extensions can use the generic mechanism, and it won't be specific to creating functions or modules. It will also leave the existing APIs intact, so it sounds overall better for backward compatibility

We could also add sqlite3_close_v3 that auto-closes statements, it doesn't seem like a particularly costly operation

It seems like an ordering problem on SQLite's side.
From the comments in the thread (edited for clarity):

sqlite3_close checks for unfinalized statements (and aborts the close) before calling destructors. You can't finalize statements in the xDestroy method because the method won't be called.

And this has been my experience as well.
I have a few long-lived prepared statements that I stick into the extension's AuxData that I'd like to finalize only when the extension is unloaded. Re-preparing the statements every single transaction would be pretty expensive.

A sqlite3_close_hook close hook sounds like a good option. That way the extension can finalize the statements then have its xDestroy method called in the normal way.
The only issue with sqlite3_close_v3 is that it'd make it harder for developers to track down if they're leaking prepared statements. They'd have to understand which prepared statements are supposed to be there at close time vs which statements were actually leaked.

@tantaman just pushed https://github.com/libsql/libsql/pull/219, let me know if it solves your shutdown order issue. The close hook is fired as the first thing in sqlite3Close(), and in particular it happens before a call to connectionIsBusy which checks for dangling prepared statements, so it should be enough to perform a cleanup.


about training details

Hi!
Thanks for your great contribution. It is really a novel method that is differential and fast!
I have read the paper and the code.
My question is how to train the model?
The number of vertex of the mesh in the dataset is not the same.
Given a mesh, has N vertices, so is the batch size N?
Do you train one mesh in one epoch? This means the epoch loss is only about this mesh.
If I'm right, I wonder that how the model can generate to out-of-training data with the different number of vertex and different topology?
Look forward to your reply. I'm really want to learn from this work!

I am not associated with the team who wrote the paper and is not at all expert in 3D, meshes etc. - I found the paper, read it many times and decided to try to implement it.
I have not completed the implementation, I have written a yolo training loop but not tried to run it yet 🙈
From what I understand and my own intuition, the model may converge on out-of-distribution data from its training strategy of using Point Sampler, Edge Predictor and Face Classifier in a way that:

Uses local geometric features
Avoids global topology assumptions
Learns generic simplification operations
Validates on different mesh types

It would probably struggle on meshes that are very complex or highly irregular compared to the training data. For example, the ABC dataset I am using here is relatively simple compared to e.g. game character meshes. So the trained weights from the industrial CAD meshes would probably not do well in gaming, and thus needs to be re-trained on the domain it needs to be applied in.
On the training part, the meshes, epochs and batch size are not connected in that way. Training happens in a number of loops (epochs) which will traverse items in the training set using a batch size. Read more here: https://machinelearningmastery.com/difference-between-a-batch-and-an-epoch/

😮 I sincerely admire the diligent work and effort you have put in; it is truly inspiring.
I'm interested in mesh simplification in Mobile Game.
I have found out that I misunderstand the training process.
For the training loop, the basic data item is a mesh, output by self.__get__item__(). So the loss is computed within the mesh.
A Batch has batch_size meshes, and an epoch just iteratively processes ALL data within several Batches.
👍 Thanks again for your reply as well as the amazing work you have done. It really helps me a lot to understand this paper.


lib/types: addCheckDesc: init

It's used in only a couple places, but being able to communicate what
check was violated will help with cryptic error messages like:
error: A definition for option `programs._1password-gui.gid' is not of type `signed integer'. Definition values:
       - In `/home/vin/workspace/vcs/nixpkgs/master/test.nix': 998

Instead, add the ability to append a nice description of the check:
error: A definition for option `programs._1password-gui.gid' is not of type `signed integer (with check: greater than or equal to 1000)'. Definition values:
       - In `/home/vin/workspace/vcs/nixpkgs/master/test.nix': 998

Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

I added a comment above the declaration and replaced the manual's usage of addCheck with addCheckDesc. Is there anywhere else I should put some docs?

Ha ha :P
I'd prefer to keep the canonical type description but have an addendum so that it's easier to search for in e.g. the generated configuration.nix.5 manpage. Not a terribly strong reason, but...
I wouldn't be opposed to also adding an addCheckWith as you've described and implementing addCheckDesc (would addCheckDescription make you happier?) in terms of that, but it's difficult to communicate that the description is that of the attached check, and not of the type itself, otherwise -- i.e., I'd expect addCheckWith { description = "lol"; } to set the description, but I want addCheckDesc "lol" to augment the description.

Is there something I can do to help this getting merged?

I think I was waiting on a response from roberth on the approach, since they had some reservations, but there's nothing wrong with this as-is, IMHO.

The example is still an awful error message. Why would you want this?
signed integer (with check: greater than or equal to 1000)

... if you could have
integer of at least 1000

This way, you avoid

any references to the bit-wise representation of integers
the implementation detail that an extra check is performed
a conjunction (or)

I guess (1) and (3) could be fixed elsewhere, but (2) is introduced by this PR.
If the goal was to make type descriptions nicer, this is not the right solution.


Expand wildcard targets into individual jobs

Issue
Config looks roughly like this:
- job_name
  scheme
  basic_auth
  metrics_path
  static_configs:
    - targets:
        - host:port
      labels

The scrape URL components are split into two hierarcical levels of a scrape_config:

top level:

scheme
basic_auth (part of netloc)
metrics_path (the suffix of the path; the path prefix will be coming from per-unit relation data)

inner level:

static_configs->targets->host:port (part of netloc)

Currently, "static_configs": [{"targets": ["*:1234"]}] is autoexpanded into
  'static_configs': [{'labels': {'juju_unit': 'remote-app/0', ...},
                      'targets': ['<IP_ADDRESS>:1234']},
                     {'labels': {'juju_unit': 'remote-app/1', ...},
                      'targets': ['<IP_ADDRESS>:1234']}]}]

Which means we can't specify different metrics_path per unit, but it is needed when using ingress-per-unit.
Solution
Expand wildcard targets into individual jobs
Context
#349
Testing Instructions

Release Notes

I wonder whether it would be better to set a per-unit metrics path or to simply set a flag that ingress-per-unit is enabled. Passing in an IngressPerUnitRequirer to MetricsEndpointProvider and checking the status of that to determine the path is a much simpler UX, since the "actual" PathPrefix is reasonably predictable, and could be constructed in the library so authors don't need to worry about how to create an external_url property or similar.

I agree we construct & deconstruct url parts a bit too much. Rethinking this should probably be a v1 effort.
Introducing interlib dep. injection is worth exploring but also seems like a greater effort. For example adding a Protocol file to the lib interface repo for every relation. I'd like to stay away from this altogether under the current scope.


Boolean operation: Wrong result

Hi
I used a Boolean operation in the Grasshopper component, but the result looks weird. How can I fix this?

void BooleanOp(Curve curveA, Curve curveB,int cliptype)
        {
            resultCurve.Clear();
            Paths64 boolean;
            Paths64 subj = ConvertPolyline(curveA);
            Paths64 clip = ConvertPolyline(curveB);
            boolean = Clipper.BooleanOp(clipTypes[cliptype], subj,clip , FillRule.NonZero);
            foreach (var path in boolean)
            {
                Polyline polyline = new Polyline(path.Select(p => new Point3d(p.X , p.Y , 0)));
                polyline.Add(polyline[0]);
                resultCurve.Add(polyline.ToNurbsCurve());
            }
        }

Sorry, there is no Grasshopper component in this library.
I presume it's a third-party derivitive so you'll need to contact the author of that component for assistance.

I am the author :)
And i ask about this result, actually i use scaling to make the result better


A metric with the name X has already been registered

Receiving the following error

{"nativeError":{},"name":"Error","message":"A metric with the name trace_calls has already been registered.","stack":[{"fullFilePath":"/node_modules/prom-client/lib/registry.js:67:10","fileName":"registry.js","fileNameWithLine":"registry.js:67","fileColumn":"10","fileLine":"67","filePath":"node_modules/prom-client/lib/registry.js","filePathWithLine":"node_modules/prom-client/lib/registry.js:67","method":"Registry.registerMetric"},{"fullFilePath":"/node_modules/prom-client/lib/metric.js:48:13","fileName":"metric.js","fileNameWithLine":"metric.js:48","fileColumn":"13","fileLine":"48","filePath":"node_modules/prom-client/lib/metric.js","filePathWithLine":"node_modules/prom-client/lib/metric.js:48","method":"new Metric"},

Perhaps might need to check if the gauge/counter exists before registering a 2nd time.
if ( !prometheus.registry.getSingleMetric(name) ) {
    prometheus.registerGauge(name, "custom help", Object.keys(labels));
}

Seems to be fixed


1-click no longer generating tasks

Today 1-click stopped generating tasks for us.  It did work yesterday (before we received v 0.11.3).
Here is an example of a previously working task:

Version screenshot:

Hi @ahardin13 , somehow I managed to publish a development version of the extension.
Would you give it another go!

@figueiredorui That fixed it!  Thanks!


New page - CDN hosting

https://www.figma.com/file/iETaF6c8BJGKKvbFuy3NV9/ZESTY-PAGES-DESIGNS?node-id=0%3A1
https://8-aaeffee09b-7w6v22.manager.zesty.io/content/6-ca81d1fdf3-vf07cl/7-d8c2c59ff7-qcpmlc

https://www.zesty.io/features/cdn-hosting/


Update to Meshtastic 2.0.11

Updates to the latest Meshtastic (2.0.11)
Changes the "Disable writing to comm device" setting to "Plugin Manages Device", moves the setting up in the preferences menu and defaults it to false

Ive tested this and it does not fix #147


Breadcrumb for improved navigation

Breadcrumb for improved navigation

We will look into this, as it is something we have been interested in implementing sooner or later. Thanks for the feedback.
As a mental note, we should remember to add structured data to the website for the breadcrumbs.

Update for anyone interested: We will be delivering this feature sometime later in 2020 Q1, most likely around early March.
The expected behavior for this feature as well as related changes are:

Breadcrumbs will be added to each snippet page, possibly replacing the Back to ... link.
Breadcrumbs will be collapsed on mobile, probably reverting to the existing Back to ... link and possibly allowing a way to show appropriate breadcrumbs.
The Back to ... link functionality should be redesigned to allow for personalization (combined with the ordering from #23), but also allowing for a reasonably clear hierarchy between listings and snippets.
Structured data for breadcrumbs will be added. This will not be personalized, but rather match the hierarchy of Language > Primary Tag > Snippet.

Update
Due to the proposed changes from #57, implementation of this feature will be slightly different:

The need for category-level breadcrumbs has already been addressed in #71, by introducing category/tag-level anchors that allow navigation from a language's "All" listing to any of its tags and back.
Snippet-level breadcrumbs provide some value for users looking to navigate back to the previous page, but based on the current designs and proposals from #57, it is yet unclear if the Back to ... link will be replaced at this time, entirely removed or converted into a usable breadcrumbs component that will move to the top bar on Desktop and probably take the place of the sliding anchors on mobile. Therefore this feature will be implemented towards the end of Sprint 3 to account for any design changes before that.
Breadcrumb/Back to ... personalization will be automated as part of #23, most likely allowing users to go back to the previous page they were viewing, while having sensible defaults.
Structured data for breadcrumbs is a priority, as it will improve crawling significantly, so it might even break into its own PR soon.


replace const_string

fix #235, fix #223, fix #168, fix #164, fix #163, fix #159
The tasks from #235 are split into separate commits.

Root out const_string, ✅
Cut half of the segments/params containers out,
Cut out half of the observers,
Change the pct_encoded_rule to use the view, and ✅
Change the pct_encoding API per the issue (fixed in https://github.com/CPPAlliance/url/pull/234)
Adapt url and url member functions

add set_XXXXX(pct_encoded_view) ✅
XXXXX() can become noexcept: we need a private constructor that lets us bypass validation ✅

Review javadocs ✅
Review docs ✅
Check coverage

In the tests, we have a situation where
u.set_user(s);

works for string_view (encoding whatever we have to) but doesn't work for a pct_encoded_view representing/dereferencing to the same decoded string because s.encoded() might not have been created with the correct characters for that component.
One thing we will probably have to discuss is the semantics of set_XXXXXX(pct_encoded_view).

Getting inspiration from set_XXXXXX(string_view) seems to make sense because the iterators dereference to the decoded chars.
Getting inspiration from set_encoded_XXXXXX(string_view) also seems to make sense because it stores the encoded string so we just have set_encoded_XXXX(s.encoded()).

Another alternative is to make their behavior uniform in regards to invalid chars. That is, set_XXXXX(pct_encoded_view) and set_encoded_XXXXX(string_view) would encode anything that happens to be outside the charset instead of throwing. The main difference would be how they treat "%"s. This way, set_XXXXX(pct_encoded_view s) could just be defined as set_encoded_XXXXX(s.encoded()).

good work so far

Merged at 5c307a889af674bcacb8ba4e5f15e1c9a949b6f9


[ENH] Add Neurobagel vocab graph file

Closes #182
See also https://github.com/neurobagel/bagel-cli/blob/main/bagel/models.py

Checklist

[ ] PR has an interpretable title with a prefix ([ENH], [FIX], [REF], [TST], [CI], [MNT], [INF], [MODEL], [DOC]) (see https://neurobagel.org/contributing/pull_requests for more info)
[ ] PR links to GitHub issue with mention Closes #XXXX
[ ] Tests pass
[ ] Checks pass

For new features:

[ ] Tests have been added

For bug fixes:

[ ] There is at least one test that would fail under the original bug conditions.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 99.754%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
405

Relevant Lines:
406

💛  - Coveralls


GSoD'21 Volunteering Proposal

I am adding the GSoD'21 volunteering proposal on behalf of myself and Simin.

Could we please add the team presentation YouTube video in our blog post so that we can better explain our proposal to the community? Thanks!

Yes, I and Simin are collaborating to make a YT video.
I will request a review after committing the video, sorry for replying with only a thumbs-up emoji earlier 😅

That's alright. Looking forward to seeing the presentation with Simon and you!

@huan We have added the team presentation YouTube video in our blog post.
Special thanks to Simin for spending so much effort in creating the presentation and editing the video.

Thank you very much for the volunteer team proposal!


add improved error handling options

For example, if an optional property on an object happens to be a promise, do we fail the whole thing?  Can we add a skip_promises or skip_errors property?
Use case which is geotiff-tile(-web-worker) which returns additional info when debugging, which fails the postMessage from the worker

The particular issue is I think it is trying to close a geotiff.js image object (but I'm not sure).


Feat removedummies #587

Check:

post::info
matches::gridview
matches::*::sent-request

Fixes #587

Regression bug:
When posting, the behaviour described in #414 is back again: in FF, the cursor jumps to the beginning of the text field repeatedly while typing, in Chrome all is fine.

Yeah, contenteditable seems to be not very standardized. Every browser generates different html (e.g. FF generates a trailing <br>)
I'll change the textfield to use a micro-version of our architecture within itself:

Keypress events will be directly registered and swallowed before they reach the contenteditable
these update the internal model: an array of strings (=lines of text) plus the caret-position
the resulting model gets rendered as html and the caret position set

This should avoid any more regression bugs for the textfield. As an added bonus, should we ever allow richer formatting, this should be a lot simpler to maintain.
Also see a blog-post by a developer for medium.com on the topic: [Why ContentEditable is Terrible,
Or: How the Medium Editor Works](https://medium.engineering/why-contenteditable-is-terrible-122d8a40e480#.2ut5hxm1w]

Can we merge this? If so, does this version of dynamic-textfield.js supersede the one in master?

Now we can :)


jkanbanInstance.options.boards not updated to match board state

Opening a new issue since the other issue that reported it is closed
See: https://github.com/riktar/jkanban/issues/75#issuecomment-1092750774

This is a viable workaround for me by adding it to the dropEl & addElement/removeElement
https://github.com/riktar/jkanban/issues/64#issuecomment-1153336849


Begin to fill-out README.md

[ ] closes #xxxx
[ ] tests added / passed
[x] docs reflect changes
[x] entry in CHANGELOG.md

Our README was really bad before. This PR updates it with something a little bit better.
Expect more PRs to clean things up further.

Tagging @delgadom  for review, if you can spare the time. This is just to get docs started. I'm happy for any proof reading or quick suggestions other than "get good". 👍  Thank you!


Figure out the color palette

So far, I've used the previous color palette. Review and maybe change the colors. Everything is saved in SCSS colors, so it would be super easy to change an existing color.
Existing color palette

I guess this is all cool. Minor modifications will be added as needed.


Leaving a group is not possible

It's not possible to leave a group for both karrot.world and dev.karrot.world.
After selecting Yes having question Are you sure you want to leave this group?, nothing happens.
Swagger
Using swagger it throws an error message for missing or incorrect CSRF token.
It's the first time I am using Swagger, hope I checked it correctly:

I authenticated in swagger with my test account for dev.karrot.world
I tried to use api/groups/{id}/leave with my group id (which was 52)

Code: 403
Response body:
{
  "detail": "CSRF Failed: CSRF token missing or incorrect.",
  "error_code": "permission_denied"
}

Request URL:
https://dev.karrot.world/api/groups/52/leave/

Thanks for the report! I think the problem is with the frontend. The CSRF error looks unrelated, at least I managed to leave the group via the API successfully just now.


Support IPFamily for echo service [WIP]

Please provide a description of this PR:
Add support to configure IPFamily for echo service. Add "Addresses" for ClusterIPs. Add config function to get config dump.

Hi @ycai-aspen. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

LGTM

/ok-to-test

LGTM, but we should also mention why we need the Config dump with EDS in the description

@ycai-aspen: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
16b54325092c200b92ac4910f39bd16cafee9cd8
link
true
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ycai-aspen: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
16b54325092c200b92ac4910f39bd16cafee9cd8
link
true
/test lint_istio

gencheck_istio
16b54325092c200b92ac4910f39bd16cafee9cd8
link
true
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

LGTM


SMB v. 2 / 3 integration based on SMBJ

Hi,
based on this issue: https://issues.apache.org/jira/browse/VFS-635 I created an implementation of https://github.com/hierynomus/smbj into vfs 2.2
I named the provider SMB2, so the scheme I used is also smb2. I dont know if it's better to rename the existing smb to cifs and use smb for the smb v2+ implementation.
All the tests from the ProviderTests pass (except known issues with classLoader ans sealedClass).
Thanks for any feedback or further advice how to proceed with this pr,
André

I removed the test class because it needs to have a smb server running on either localhost or an accessible system. I used a Windows Server 2016 VM and copied the test-data to the share root.

@markt-asf , can I ask why did you close it ? was it integrated in some other way to this repository ?

It was closed automatically when the trunk branch was removed. I've re-opened the PR. Please re-base it against master.

@ahbonsu @markt-asf - can you please advise if this planned to be merged ?


svg input differs to pdf output

I thought you might be interested. I am currently creating .svg files using Poloto and was wondering why my plots suddenly become informationless when I save them as .pdf. You can have a look at the files from my gitlab project for comparison. The .svg were first translated with your library and then again with inkscape. So from my point of view you are just as good as inkscape at translating .svg to .pdf.

False alarm. Inkscape is better than you.

I'm not following, is there an SVG that is converted differently from what you expected? I tried the bar.svg and it looks fine to me:
test.pdf

Then please help me with which settings you did that. I have now set up the code like this and all the labels are now gone.
use std::fs::write;
use svg2pdf::{convert_str,Options};

fn main() {
    let data = [
        (2.327473685, "Rust-bio"),
        (38.339006423950195, "Python"),
        (0.258953, "Julia"),
    ];

    let svg = poloto::build::bar::gen_simple("", data, [0.00])
        .label(("Comparison of Programming Language Runtime", "Runtime in Seconds", "Languages"))
        .append_to(poloto::header().light_theme())
        .render_string().unwrap();
    let pdf = convert_str(
        &svg,
        Options::default()
        ).unwrap();

    write("bar.pdf", pdf).unwrap();
}

See the documentation for convert_str, it doesn't load any fonts, and thus cannot convert any text. You should do something like this instead:
use std::fs::write;
use svg2pdf::{Options, usvg};
use svg2pdf::usvg::{fontdb, PostProcessingSteps, TreeParsing, TreePostProc};

fn main() {
    let data = [
        (2.327473685, "Rust-bio"),
        (38.339006423950195, "Python"),
        (0.258953, "Julia"),
    ];

    let svg = poloto::build::bar::gen_simple("", data, [0.00])
        .label(("Comparison of Programming Language Runtime", "Runtime in Seconds", "Languages"))
        .append_to(poloto::header().light_theme())
        .render_string().unwrap();

    let mut tree = usvg::Tree::from_str(&svg, &usvg::Options::default()).unwrap();

    let mut db = fontdb::Database::new();
    db.load_system_fonts();
    tree.postprocess(PostProcessingSteps::default(), &db);

    let pdf = svg2pdf::convert_tree(&tree, Options::default());

    write("bar.pdf", pdf).unwrap();
}

That's weird. The labelling is still missing. Are you loading the library with a different version number?

Hmm, as I said, for me the text shows up fine... I suspect that it's a font fallback issue. Are you sure you have Roboto installed on your system? Usually it should fallback to another font in case that is not the case, but looks like it doesn't do that here and fails silently. But you can try the following:
Install resvg 0.38 (it uses the same underlying library as us for font loading, so if it fails there, it's probably an upstream issue):
cargo install resvg --version ^0.38
Then run resvg bar.svg bar.png.
If the text is missing there as well, it's most likely an upstream issue. If not, then it might be an issue on our side... But since I can't reproduce it, I also can't really investigate it.
You can also verify that svg2pdf can find your fonts by installing the CLI and running svg2pdf fonts.

What do you mean by Roboto? I couldn't find any reference to this in the documentation other than the readme.

That would be super weird, because the svg is created correctly and also has all the features and only the pdf is suddenly without labelling. I'll give it a try. Since I have a Nixos, I have to compile the operating system for new packages every time. So see you soon.

OK, I've installed it. Still not working. I mean, it's not that bad. I can still turn an .svg into a .pdf using inkscape.

Hmm, yeah, sorry then... It's hard for me to debug if I can't reproduce it locally. But I'm still 99% sure it's some kind of font issue, because apart from font loading, everything should work independently from the platform...

Since I'm 99% sure it's a font loading/fallback issue, I'll close this in favor of #65.


Roll Flutter Engine from c4396f9f602f to d2973619074e (6 revisions)

https://github.com/flutter/engine/compare/c4396f9f602f...d2973619074e
2023-04-18<EMAIL_ADDRESS>[macOS] Fix Ctrl+Tab is broken (flutter/engine#40706)
2023-04-18<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from K1LGtKXyxRlW3Q9O1... to qZHSvkpAU1-YYGvYc... (flutter/engine#41293)
2023-04-18<EMAIL_ADDRESS>Roll buildroot to 059d155b4d452efd9c4427c45cddfd9445144869 (flutter/engine#41288)
2023-04-18<EMAIL_ADDRESS>[Impeller] Remove glyph pixel rounding during text frame conversion (flutter/engine#41285)
2023-04-18<EMAIL_ADDRESS>Revert "Reland  "Migrate mac_host_engine to engine v2 builds."" (flutter/engine#41284)
2023-04-17<EMAIL_ADDRESS>focus SkiaGPUObject wrappers on only those objects that need the protection (flutter/engine#41237)
Also rolling transitive DEPS:
fuchsia/sdk/core/mac-amd64 from K1LGtKXyxRlW to qZHSvkpAU1-Y
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC chinmaygarde@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.

Landing manually to catch up rolls. This failed due to a transient infra issue and is safe to land.


provision-post.sh and playbook-post.yml in the ~/.vccw/conifg

Some people are hoping to customize every guest machine.
Allow to run additional provisioner when ~/.vccw/config/provision-post.sh exists.
But additional ansible based provisioner should be ansible not ansible_local.

fixed

@miya0001 There is an inconsistency between the documentation and the Vagrantfile. The documentation is mentioning provision-post.yml but Vagrantfile is looking for playbook-post.yml.
Which one should be correct?


Hand over development/Gem ownership

This gem seems to be widely used but also unmaintained, would you mind to handing this over to us: https://github.com/bsm?

I'd like to see this. Not sure if anyone else is having issues with newer versions of Grape but I'm guessing that some maintenance is in order.

Hi, I can help with the hand over are you still interested? @dim  or @subvertallchris?

@chytreg if you could transfer the ownership to bsm, we can take it from there.

Sure, how about rubygems?

Same pls, just transfer https://rubygems.org/gems/grape-kaminari to https://rubygems.org/profiles/bsm

With the code, it's not a problem, but for rubygems ownership have to ask former employees of the company, hope they will reply shortly.
cc @teamon & @Ostrzy

As per docs I'd need an email for the new owner.

You can use<EMAIL_ADDRESS>if that's OK.

Adding<EMAIL_ADDRESS>Owner could not be found.

It must be an email of existing rubygems account.

As for repo transfer,
You don’t have the permission to create public repositories on bsm

I've added @dim as a collaborator with Admin permission, but I don't think you would be able to transfer the repo since the only owner can do that ... but I can transfer to your GitHub account ... and then you can move to your organization. WDYT?

Sure, you can transfer it to my "private" GH handle, which is https://github.com/dim

Actually, I could do it myself, the GH transfer is complete.

Hi again,
Sorry, my bad, didn't check it properly, getting:
You do not have permission to push to this gem. Ask an owner to add you with: gem owner grape-kaminari --add<EMAIL_ADDRESS>

Done


new CRD for VirtualMachineImage

Is this a BUG REPORT or FEATURE REQUEST?:
/kind enhancement
What happened:
We can upload vm image using virtlctl image-upload with image path. However every time creating VM, we have to know exact image path or PVC to copy. If we have image repository with what image is included and its metadata, It can be very useful.
We have container registry disk and it works similarly, however the difference is VirtualMachineImage is admin by k8s and we can handle as kubernetes object.
What you expected to happen:
We can declare new CRD, like VirtualMachineImage.
kind: VirtualMachineImage
metadata:
  name: myvmi
  label:
  - os: ubuntu18.04
spec:
  fromUrl:
    url: <url of ubuntu 18.04 cloud image>
(or)
  fromPvc:
    pvcName: <pvc of ubuntu 18.04>
(or)
  fromRegistryDisk
    image: vmdisks/fedora25:latest

How to reproduce it (as minimally and precisely as possible):
None
Anything else we need to know?:
None
Environment:
None

Hello, I was thinking about working on this issue. Any tips on how to get started?

Hi @WoohyungHan I have a question, How do you image VirtualMachineImage would be used together with VirtualMachine? Thank you.

Hello, I was thinking about working on this issue. Any tips on how to get started?

Hi @WoohyungHan I have a question, How do you image VirtualMachineImage would be used together with VirtualMachine? Thank you.

@JacksonKrauss @petrkotas hi, i'm not sure 100% this is worth for now... however I think we can learn from the relationship of glance and cinder service in openstack. We can provision a VM with DataVolume and the DataVolume has a source of image. for now, this source is form of raw file or http url. I think we can change this as image service id (or name).
first user create VirtualMachineImageStore. VMIS has a pvc and it stores real image with metadata in etcd.
kind: VirtualMachineImageStore
spec:
  pvc:
    name: {name of image store}

user can upload their image by using VirtualMachineImage. our contoller accept this CRD and download from source and store to pvc.
kind: VirtualMachineImage
metadata:
  name: ubuntu 18.04
spec:
  source:
    http: "http..."

and then user can create a DataVolume with the VirtualMachineImage
kind: DataVolume
spec:
  source:
    VirtualMachineImage: ...
  pvc:
     ...

so what's diff with previous? We can see our vm images like this.
kubectl get VritualMachineImage

name                 source (...)
ubuntu18.04        ...

Thanks for sharing the idea @WoohyungHan, I am struggling to see the difference between our current CDI importer. Have you checked it out?

Thanks for sharing the idea @WoohyungHan, I am struggling to see the difference between our current CDI importer. Have you checked it out?

@petrkotas Maybe I'm wrong, but I think It's a bit different. If some user want to keep their image with read-only and clone it to create hundred of VMs, how the user can distinguish original image and clone image? he can create label for it, but I think we can create new CRD to get this. I think it is simillar with cinder-glance relation in openstack. Could you please check this link? :) https://ask.openstack.org/en/question/29907/what-is-the-difference-between-cinder-glance-and-swift/


[Bug]: AppName is not properly set when using "create" command

Severity
Trivial
What release version, tag or commit-hash did you use?
v0.6.3
Current Behavior
When calling velocitas create without an example, a project name has to be chosen. Unfortunately this name is not set as AppName in the Python code.
Steps to Reproduce

velocitas create -n testApp -l python
check main.py or vapp.py

Expected Behavior
AppName is properly set in python scripts
Possible Solution
No response
Additional Information
No response
Code of Conduct

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

will be solved with https://github.com/eclipse-velocitas/cli/pull/199

Fixed with #212


Docs: Add simple examples to Introspection > Object Types

Throughout the last couple of weeks, I have been spending some time going through the documentation and trying understand the fundamentals of SDL and EdgeQL. One part that was particularly elusive for me was figuring out how to perform a basic introspection to determine the names of properties and links that are associated with a specific object.
Here's the Movie and Person SDL migration from the tutorial (transaction omitted):
CREATE MIGRATION movies TO {
    type Movie {
        required property title -> str;
        # the year of release
        property year -> int64;
        required link director -> Person;
        multi link cast -> Person;
    },
    type Person {
        required property first_name -> str;
        required property last_name -> str;
    },
};

Now let's say the user wanted to query the data stored in any Movie object, but forgot the names of the links and properties. In a real-world scenario, the user could be using a preexisting database where they aren't familiar with the schema structure. In order to determine those names, it would be very useful to know how to use a simple introspect:
 SELECT INTROSPECT Movie {
     properties: {
         name,
     },
     links: {
         name,
     },
 }

I think something like this would definitely be worth mentioning in the EdgeQL section of the tutorial, perhaps just after the section starting with "The above query simply returned all the Movie objects...".
As someone primarily experienced with RDBMS with some minimal experience using document-object DBs, I had an unanswered question that bothered me for the remainder of the tutorial after seeing this example:
SELECT Movie;
{
    Object { id: <uuid>'4d0c8ddc-54d4-11e9-8c54-7776f6130e05' },
    Object { id: <uuid>'64d024dc-54d5-11e9-8c54-a3f59e1d995e' },
}

If I were entering a preexisting database or don't recall the schema definition for a given object, how am supposed to figure out the names? I couldn't exactly use SELECT * from table or a GUI dropdown menu to look at the schema structure. I knew that SELECT * wouldn't be at all be suitable for EdgeQL as a strongly static language and that a GUI wasn't a necessity, but my question was still unanswered.
As is apparent from the above examples, I later figured it out from the Introspection > Object Types section. But even the first example wasn't immediately obvious that it was showing me what I was looking for:
WITH MODULE schema
SELECT ObjectType {
    name,
    links: {
        name,
    },
    properties: {
        name,
    },
}
FILTER .name = 'schema::ObjectType';

This example is fairly abstract and uses a filter that wouldn't be needed if the user only wanted a specific object. This example and the one below it lead me to believe something like this would be the minimum way to figure out the properties and links for Movie:
WITH MODULE schema
SELECT ObjectType {
    name,
    links: {
        name,
    },
    properties: {
        name,
    },
}
FILTER .name = 'default::Movie';

As a result, I think it would be worthwhile to add something along the lines of my introspection example to the tutorial and perhaps a couple of additional simple examples to Introspection > Object Types. I'd be glad to open a couple of PRs to add (one for the tutorial and another for Introspection > Object Types) an example either or both sections.

Another useful example might also be something like this, to perform a basic inspection on all of the user created objects in the database:
> WITH MODULE schema
 SELECT ObjectType {
     name,
     links: {
         name,
     },
     properties: {
         name,
     }
 }
 FILTER .name LIKE 'default%';

Great points here.  There are plans for introspectability UX improvements in #672.  We also plan to add support for splats (equivalent of SELECT *) in #180.  Once those are implemented, we'll surely include their uses in the tutorial.
Direct schema introspection using the schema queries and INTROSPECT are a bit on an advanced side, and I'm concerned that including them in the initial tutorial might be a bit too much.

Once those are implemented, we'll surely include their uses in the tutorial.

Ah, good to know. I particularly like the SELECT User {...User} syntax. Something easy to at least expose the properties seems like it would be quite useful.

Direct schema introspection using the schema queries and INTROSPECT are a bit on an advanced side, and I'm concerned that including them in the initial tutorial might be a bit too much.

Hmm, do you think one of those examples (or something along those lines) might have a place in Introspection > Object Types? As mentioned in the mentioned in the topic, I think that section could potentially benefit from the one or two examples. Particularly the direct property and link introspection of a specific user-created object, without the usage of FITLER .name = 'default::User'.

Changed title of the PR to reflect the new proposal, since @elprans mentioned the introspection examples might be too advanced for the tutorial.

@ambv Please take a look.


replace form recognizer with doc intelligence

Purpose

...

Does this introduce a breaking change?

[ ] Yes
[ ] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

Get the code

git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install

Test the code

What to Check
Verify that the following are valid

...

Other Information

I have checked out this PR myself and am changing all the references since I think it's confusing to only change it in one place, but would like to know the plans for the package name/module import.


use material as theme，some components have black border

Are you tab to navigate to the Data in the Datapicker, if you are I think it is expected, Generally it is how tab focus works


New Version of Pandas (1.0.3) Attribute Error

Hi
Thank you for this very neat tool. I have installed the widget to use with Jupyterlab and I am using the latest version of Pandas (1.0.3). I get this error when I try to cluster using net.cluster():
AttributeError: 'DataFrame' object has no attribute 'ix'
I think this error might arise from depending on an older version of Pandas. If I am correct, will this be something that will be fixed soon.
Thanks,
Kenan

Hi @kjijakli, this was recently taken care of here https://github.com/MaayanLab/clustergrammer-widget/commit/3093d4901830afc9c82db0755b55db40b44a1933
but we have to make a new release.
We also took care of this in the new Clustergrammer2 widget https://github.com/ismms-himc/clustergrammer2. For reference, we are actively upgrading the new Clustergrammer2 so we reocmmend users switch to it.

Thanks

I'll reclose the issue once we've updated

@cornhundred any update on this? I am getting the same error.  Below are the versions I am on:
clustergrammer                       1.13.6
clustergrammer-widget                1.13.3
Python 3.6
pandas                               1.2.1
I installed clustergrammer_widget by :
pip install --upgrade clustergrammer_widget
When I run the iris example from here, I am getting below error:
AttributeError: 'DataFrame' object has no attribute 'ix'
When I checked github, it does show ix has been replaced with .loc but not sure why I am getting the error. How can I resolve this? Thanks.


undefined method `which' for #HomebrewUserWrapper:0x000000010b9f6a58

:speaking_head: Foreword
:ghost: Brief Description
undefined method `which' for #HomebrewUserWrapper:0x000000010b9f6a58
:pancakes: Cookbook version
5.4.5
:woman_cook:  Chef-Infra Version
18.3.0
:tophat: Platform details
platform=mac_os_x
platform_version=14.3
Steps To Reproduce
Steps to reproduce the behavior:
converge cookbook
:police_car: Expected behavior
execute normally
:heavy_plus_sign: Additional context
  NoMethodError
  -------------
  undefined method `which' for #<HomebrewUserWrapper:0x000000010b9f6a58>

  Cookbook Trace: (most recent call first)
  ----------------------------------------
    /var/folders/8r/pjq314jj7mzfr_fd98_5cvbr0000gq/T/cookbooks.PdJuE9l5Vh/site-cookbooks/homebrew/libraries/helpers.rb:64:in `owner'
    /var/folders/8r/pjq314jj7mzfr_fd98_5cvbr0000gq/T/cookbooks.PdJuE9l5Vh/site-cookbooks/homebrew/recipes/default.rb:63:in `block in from_file'
    /var/folders/8r/pjq314jj7mzfr_fd98_5cvbr0000gq/T/cookbooks.PdJuE9l5Vh/site-cookbooks/homebrew/recipes/default.rb:61:in `from_file'

  Relevant File Content:
  ----------------------
  /var/folders/8r/pjq314jj7mzfr_fd98_5cvbr0000gq/T/cookbooks.PdJuE9l5Vh/site-cookbooks/homebrew/libraries/helpers.rb:

   57:      ::File.exist?("#{HomebrewWrapper.new.install_path}/bin/brew")
   58:    end
   59:
   60:    def owner
   61:      @owner ||= begin
   62:        # once we only support 14.0 we can switch this to find_homebrew_username
   63:        require 'etc'
   64>>       ::Etc.getpwuid(HomebrewUserWrapper.new.find_homebrew_uid).name
   65:                 rescue Chef::Exceptions::CannotDetermineHomebrewOwner
   66:                   calculate_owner
   67:      end.tap do |owner|
   68:        Chef::Log.debug("Homebrew owner is #{owner}")
   69:      end
   70:    end
   71:
   72:    private
   73:

  System Info:
  ------------
  chef_version=18.3.0
  platform=mac_os_x
  platform_version=14.3
  ruby=ruby 3.1.2p20 (2022-04-12 revision 4491bb740a) [arm64-darwin20]
  program_name=/opt/chef-workstation/bin/chef-solo
  executable=/opt/chef-workstation/bin/chef-solo

Also seeing this.
$ sw_vers
ProductName:		macOS
ProductVersion:		14.3.1
BuildVersion:		23D60
$ chef --version
Chef Workstation version: 24.2.1058
Test Kitchen version: 3.6.0
Cookstyle version: 7.32.7
Chef Infra Client version: 18.3.0
Chef InSpec version: 5.22.40
Chef CLI version: 5.6.14
Chef Habitat version: 1.6.652


Initial work on eval-server

I meant to push this to a separate branch but that didn't work out. :/


Remove device_profile

It's unnecessary to read the current device profile at runtime.
The use of preprocessor macro is discouraged by the Google style guide, so it's
constexpr double kProfileFactor = 0.7;

rather than
#define PROFILE_FACTOR 0.7

Have you test the behavior on each device? or Do you need help testing on each device?

@bbrto21 No, I haven't. I'm out of the office until tomorrow. I'll be grateful if you run the test for me (if you think this change is reasonable).

I've test this on wearable, tv, mobile device. it works properly!


Make some delay/timeout option to browser automation for navigation timeout

Since I'm running on RPi 3B, Chromium itself is really struggles to load/proceed page. I think. it succeed to pass here on 1/10 times.
[2022-04-21 17:42:54.448 +0000] DEBUG: Could not refresh credentials. Logging in fresh.
[2022-04-21 17:42:54.451 +0000] DEBUG: hcaptchaAccessibilityUrl not configured, captchas are less likely to be bypassed. Follow this guide to set it up: https://github.com/claabs/epicgames-freegames-node#hcaptcha-accessibility-cookies
[2022-04-21 17:42:54.464 +0000] DEBUG: Logging in with puppeteer
[2022-04-21 17:43:11.019 +0000] TRACE: devtools://devtools/bundled/inspector.html?ws=<IP_ADDRESS>:39203/devtools/page/BC0733F20DA7A56188A267257A9BEEDC
[2022-04-21 17:43:11.539 +0000] TRACE: Navigating to Epic Games login page
[2022-04-21 17:43:46.503 +0000] INFO: Checking free games for<EMAIL_ADDRESS>[2022-04-21 17:43:46.556 +0000] DEBUG: Refreshing login session
[2022-04-21 17:43:46.558 +0000] TRACE: Request store homepage
    url: "https://store.epicgames.com/en-US/"
[2022-04-21 17:43:46.941 +0000] ERROR: Encountered an error during browser automation. Saved a screenshot for debugging purposes.
    errorFile: "error-2022-04-21T17:43:41.544Z.png"
[2022-04-21 17:43:48.219 +0000] ERROR: Navigation timeout of 30000 ms exceeded
    err: {
      "type": "TimeoutError",
      "message": "Navigation timeout of 30000 ms exceeded",
      "stack":
          TimeoutError: Navigation timeout of 30000 ms exceeded
              at /usr/app/node_modules/puppeteer/src/common/LifecycleWatcher.ts:205:18
              at FrameManager.waitForFrameNavigation (/usr/app/node_modules/puppeteer/src/common/FrameManager.ts:238:19)
              at Frame.waitForNavigation (/usr/app/node_modules/puppeteer/src/common/FrameManager.ts:790:12)
              at Page.waitForNavigation (/usr/app/node_modules/puppeteer/src/common/Page.ts:1833:12)
              at async Promise.all (index 1)
              at PuppetLogin.startLogin (/usr/app/src/puppet/login.ts:43:5)
              at PuppetLogin.login (/usr/app/src/puppet/login.ts:83:7)
              at Login.fullLogin (/usr/app/src/login.ts:102:7)
              at redeemAccount (/usr/app/src/index.ts:26:5)
              at async Promise.all (index 0)
      "name": "TimeoutError"
    }
[2022-04-21 17:43:48.605 +0000] WARN: Current version
    COMMIT_SHA: "fa805b4262c88fefa73da8942eab864bde3df183"
    BRANCH: "master"
    DISTRO: "alpine"

[2022-04-21 17:54:34.557 +0000] DEBUG: Could not refresh credentials. Logging in fresh.
[2022-04-21 17:54:34.557 +0000] DEBUG: hcaptchaAccessibilityUrl not configured, captchas are less likely to be bypassed. Follow this guide to set it up: https://github.com/claabs/epicgames-freegames-node#hcaptcha-accessibility-cookies
[2022-04-21 17:54:34.569 +0000] DEBUG: Logging in with puppeteer
[2022-04-21 17:54:35.445 +0000] TRACE: Waiting for MFA possibility
[2022-04-21 17:54:40.195 +0000] TRACE: devtools://devtools/bundled/inspector.html?ws=<IP_ADDRESS>:36359/devtools/page/231EB0C45CEF692B4737A83CAFB4B66F
[2022-04-21 17:54:54.070 +0000] TRACE: Navigating to Epic Games login page
[2022-04-21 17:55:18.853 +0000] TRACE: MFA detected
[2022-04-21 17:55:18.876 +0000] TRACE: Filling MFA field
[2022-04-21 17:55:23.020 +0000] TRACE: Waiting for continue button
[2022-04-21 17:55:23.072 +0000] TRACE: Clicking continue button
[2022-04-21 17:55:38.199 +0000] ERROR: Encountered an error during browser automation. Saved a screenshot for debugging purposes.
    errorFile: "error-2022-04-21T17:55:24.075Z.png"
[2022-04-21 17:55:39.518 +0000] ERROR: Navigation timeout of 30000 ms exceeded
    err: {
      "type": "TimeoutError",
      "message": "Navigation timeout of 30000 ms exceeded",
      "stack":
          TimeoutError: Navigation timeout of 30000 ms exceeded
              at /usr/app/node_modules/puppeteer/src/common/LifecycleWatcher.ts:205:18
              at FrameManager.waitForFrameNavigation (/usr/app/node_modules/puppeteer/src/common/FrameManager.ts:238:19)
              at Frame.waitForNavigation (/usr/app/node_modules/puppeteer/src/common/FrameManager.ts:790:12)
              at Page.waitForNavigation (/usr/app/node_modules/puppeteer/src/common/Page.ts:1833:12)
              at async Promise.all (index 1)
              at PuppetLogin.startLogin (/usr/app/src/puppet/login.ts:43:5)
              at PuppetLogin.login (/usr/app/src/puppet/login.ts:83:7)
              at Login.fullLogin (/usr/app/src/login.ts:102:7)
              at redeemAccount (/usr/app/src/index.ts:26:5)
              at async Promise.all (index 1)
      "name": "TimeoutError"
    }
[2022-04-21 17:55:40.248 +0000] WARN: Current version
    COMMIT_SHA: "fa805b4262c88fefa73da8942eab864bde3df183"
    BRANCH: "master"
    DISTRO: "alpine"

[2022-04-21 16:06:16.446 +0000] INFO: Checking free games for <REDACTED>
[2022-04-21 16:06:18.156 +0000] ERROR: Encountered an error during browser automation. Saved a screenshot for debugging purposes.
    errorFile: "error-2022-04-21T16:06:09.215Z.png"
[2022-04-21 16:07:01.517 +0000] ERROR: Encountered an error during browser automation. Saved a screenshot for debugging purposes.
    errorFile: "error-2022-04-21T16:06:54.679Z.png"
[2022-04-21 16:07:01.861 +0000] ERROR: Navigation timeout of 30000 ms exceeded
    err: {
      "type": "TimeoutError",
      "message": "Navigation timeout of 30000 ms exceeded",
      "stack":
          TimeoutError: Navigation timeout of 30000 ms exceeded
              at /usr/app/node_modules/puppeteer/src/common/LifecycleWatcher.ts:205:18
              at FrameManager.waitForFrameNavigation (/usr/app/node_modules/puppeteer/src/common/FrameManager.ts:238:19)
              at Frame.waitForNavigation (/usr/app/node_modules/puppeteer/src/common/FrameManager.ts:790:12)
              at Page.waitForNavigation (/usr/app/node_modules/puppeteer/src/common/Page.ts:1833:12)
              at async Promise.all (index 1)
              at PuppetLogin.startLogin (/usr/app/src/puppet/login.ts:43:5)
              at PuppetLogin.login (/usr/app/src/puppet/login.ts:83:7)
              at Login.fullLogin (/usr/app/src/login.ts:102:7)
              at redeemAccount (/usr/app/src/index.ts:26:5)
              at async Promise.all (index 1)
      "name": "TimeoutError"
    }
[2022-04-21 16:07:02.295 +0000] WARN: Current version
    COMMIT_SHA: "fa805b4262c88fefa73da8942eab864bde3df183"
    BRANCH: "master"
    DISTRO: "alpine"

latest log is hangs on here:

https://github.com/puppeteer/puppeteer/blob/main/docs/api.md#pagesetdefaultnavigationtimeouttimeout Maybe expose this option to solve this.

Added browserNavigationTimeout: https://claabs.github.io/epicgames-freegames-node/classes/AppConfig.html#browserNavigationTimeout


fixes #59    (the bug of the button , which was not being clicked)

Please mention which issue you resolved and provide some screenshots so that I can get an idea about what contribution you make.


Testing, please ignore. (jevqbu)

Bug bounty test - please ignore.... Please DO NOT APPROVE THIS! (jevqbu)

Hello @yanshulmatok,
I'm closing this PR. This repository has mechanisms in place which prevent the execution of CI on PR for non-contributors and will not execute these modifications to the build scripts automatically. If you're interested in working with us for the purposes of bug bounty research, please feel free to do so in a fork, and please report any suspected or confirmed security issues to AWS Security https://aws.amazon.com/security/vulnerability-reporting/
Thank you!


"Merged" and "Closed" buttons in extend-discussion-status-filters link to opposite filters

"Merged" button links to closed PRs and "Closed" button links to merged PRs.
Example URL: https://github.com/sindresorhus/refined-github/pulls
Screenshot:

Thank you for the report! This was because the SearchQuery parser was being created for the Closed link and then edited later thinking it was the Merged link.
Also I noticed that they made PR lists responsive so there are 2 pairs of Open/Closed buttons: one for mobile view and one for the regular view


backport: bitcoin#18885, #16625, #16665 - new maintainer repo

Issue being fixed or feature implemented
Moving some scripts from dashcore repo to dash-maintainer-repo.
That's better to merge rather than skip as DNM by 3 reasons:

these scripts are not part of dash core codebase, but some extra tools (bitcoin's motivation)
these scripts can be used not only for dashpay/dash repo but for other repos too (bitcoin's motivation)
bitcoin's original scripts continue to get improvements and fixes but we are not aware of them so far as they are removed from bitcoin/bitcoin repo

Also linter complains about this scripts if include contrib/devtools to list "to lint". It blocks bitcoin#20451
What was done?
backports: bitcoin#18885, bitcoin#16625, bitcoin#16665
Created new repo: dashpay/dash-maintainer-repo
How Has This Been Tested?
Compared our version of scripts to bitcoin's version at the moment when they are removed and dashified missing changes in the new repo.
Breaking Changes
N/A
Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have added or updated relevant unit/integration/functional/e2e tests
[x] I have made corresponding changes to the documentation
[x] I have assigned this pull request to a milestone

@PastaPastaPasta needs your help with transfer https://github.com/knst/dash-maintainer-tools to https://github.com/dashpay/dash-maintainer-tools once it is reviewed

I think I'd rather we keep these in dash repo; maybe in a git subtree if we want to "backport" from there repo

Closed as DNM. So far as these scripts are used only for dash-core and not for other repos, without better reasoning this PR is no go.


Update GitHub Actions versions

Thank you, I've fixed the mentioned syntax error and applied this patch upstream to the boilerplate so all future projects plus the ones I'm currently working through updating will get this change - see: https://github.com/pimoroni/boilerplate-python/pull/13/commits/39d73f7ed68e6ef03ed7343920f37c6e789346fd


[PROPOSAL] Neural Search field type

Can we memic this feature in OpenSearch https://www.elastic.co/search-labs/blog/semantic-search-simplified-semantic-text
I know that a lot has been done recently in OpenSearch projects to make things headache free. I think a neural-search field type in OpenSearch would be an interesting addition. However, it should account for synonyms to avoid any fine-tuning headache.

@asfoorial from Opensearch side we do this via a combination of ingestion processor and vector field. As there are multiple use-cases for semantic search including multi-model, this would be an interesting field to have.
But is there any specific reason you are looking for the field as compared to what is present currently. My main motive here is to know the advantages of a new field vs what is currently present in opensearch.

The main reason is simplifying the process and keep the focus on the business. In fact elasticsearch had the same reason when they introduced the field.
Another reason is alignment of new features across multiple OpenSearch projects. I have noticed over the past number of releases we get new features in ml-commons and kNN. But it takes a while until we see their benefits reflected in neural-search. If they become one component (neural-search field), then that would sort of guarantee that any new feature in ml-common or kNN must be reflected in the neural-search field type before their release.

If they become one component (neural-search field), then that would sort of guarantee that any new feature in ml-common or kNN must be reflected in the neural-search field type before their release.

@asfoorial thanks for providing the details. I want to know little bit more on what features added in ML/k-NN doesn't make into Neural. May be there is something missing.
But I really like the idea of having a field which can encapsulate the processor information.

One place where having the field will be useful is nested fields. I see putting this information in the processor is very painful and not intutive.

@minalsha please take a look into this and please add your thoughts

@heemin32 any reason for closing this gh issue?

@navneet1v No. I think it is closed automatically when I added them in NeuralSearch RoadMap.

One case where I feel this field type will be very useful is in cases of complex nested fields. Currently with TextEmbedding processor it is always feels like we are finding different cases where the processor is not working some GH issues:

https://github.com/opensearch-project/neural-search/issues/1042
https://github.com/opensearch-project/neural-search/issues/1024
#895
#798
#739
#678
#73

I believe having a field type will solve this problem, in the mappers only we will call the MLCommons inference APIs to convert the text to embeddings. I think we can use the concept of properties in the mapper to have a neural field handling both text and vectors.
cc: @minalsha , @heemin32 , @vibrantvarun , @martin-gaievski

This will also reduce the number of inference requests when multiple fields should be embedded.

Inference requests in semantic_text fields are also batched. If you have 10 documents in a bulk API request, and each document contains 2 semantic_text fields, then that request will perform a single inference request with 20 texts to your inference service in one go, instead of making 10 separate inference requests of 2 texts each.
(https://www.elastic.co/search-labs/blog/semantic-search-simplified-semantic-text)

I'll work on this item.


Add validation on remove category.

Fixed Issues
#21517, #21518
Files Changed
CategoryController.cs - called IsCheckCategoryInQuestionsById() method
ICategoryRepository.cs - declared IsCheckCategoryInQuestionsById() method
CategoryRepository.cs - implementations of IsCheckCategoryInQuestionsById() method
delete-category-dialog-component.ts - display appropriate message
Checks
-[x] Naming Conventions
-[x] Server side code comments (proper English, grammar and no spelling mistakes)
-[x]Client side code comments (proper English, grammar and no spelling mistakes)
-[x] Unused variables, methods, blank spaces and name spaces.
-[x] Optimal Code and code formatting
-[x] Commit History is proper, no merge is done.
-[x] Commit/pull request messages should be proper to justify your feature
-[x] All test cases are executed provided by tester.
Known Issues
No Issues.
Comments

This change is

SonarQube analysis reported 1 issue

 1 major

Watch the comments in this conversation to review them.

SonarQube analysis reported 7 issues

 7 major

Watch the comments in this conversation to review them.
6 extra issues
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 CategoryRepositoryTest.cs#L45: Rename 'category' which hides the field with the same name.
 CategoryRepositoryTest.cs#L57: Rename 'category' which hides the field with the same name.
 CategoryRepositoryTest.cs#L68: Rename 'category' which hides the field with the same name.
 CategoryRepositoryTest.cs#L79: Rename 'category' which hides the field with the same name.
 CategoryRepositoryTest.cs#L95: Rename 'category' which hides the field with the same name.
 CategoryRepositoryTest.cs#L129: Rename 'category' which hides the field with the same name.

Reviewed 10 of 10 files at r3.
Review status: all files reviewed at latest revision, 3 unresolved discussions.

Comments from Reviewable

verified

Comments from Reviewable

Review status: all files reviewed at latest revision, 3 unresolved discussions.

Trappist/src/Promact.Trappist.Test/Category/CategoryRepositoryTest.cs, line 160 at r1 (raw file):
Previously, Krunal-Promact wrote…
option values are not formatted properly

solved

Trappist/src/Promact.Trappist.Utility/Constants/StringConstants.cs, line 147 at r1 (raw file):
Previously, Krunal-Promact wrote…
This Category belongs to Questions. You can not delete this Category.

solved

Trappist/src/Promact.Trappist.Web/wwwroot/app/questions/questions-dashboard/delete-category-dialog.component.ts, line 30 at r1 (raw file):
Previously, Krunal-Promact wrote…
not better way to extract error

solved.

Comments from Reviewable


[storage] TreeCache supports bootstraping from PRE_GENESIS_VERSION

To support #2439 we need the state to be readable before geneiss version.
I'm defining PRE_GENESIS_VERSION = std::u64::max_value() and in progress of making our state tree capable of reaading from it.
This commit specifically tweeks the TreeCache so that it detects whether a pre genesis state is present when initialized with next_version == 0.

Motivation
(Write your motivation for proposed changes here.)
Have you read the Contributing Guidelines on pull requests?
Yes.
(Write your answer here.)
Test Plan
Existing coverage and new unit test.
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/libra/website, and link to your PR here.)

r=lightmark

r=lightmark

@bors-libra r+

:pushpin: Commit fefd2c6 has been approved by wqfish

:hourglass: Testing commit fefd2c60361de13106b326a0f9cf7104b54eda61 with merge 0132afa613b1dc1ad858beb65f0fbe53318dac8e...

:broken_heart: Test failed - checks-circle_commit_workflow

r=lightmark

r=lightmark

@bors-libra retry

@bors-libra  r=lightmark

:pushpin: Commit eb0ccdc has been approved by lightmark

:hourglass: Testing commit eb0ccdc5701cd571d3c9e819fbb8e69b40fbcbb6 with merge 58997bdbdf66f605a94c5d1ab18f081085a0f232...

:broken_heart: Test failed - checks-circle_commit_workflow

@bors-libra retry

:hourglass: Testing commit eb0ccdc5701cd571d3c9e819fbb8e69b40fbcbb6 with merge 82f49f7fc5990d9840f6fb2d6c15fc4f01cc0c1c...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: lightmark
Pushing 82f49f7fc5990d9840f6fb2d6c15fc4f01cc0c1c to master...


[Enhance] Support rotate points without bbox in GlobalRotScaleTrans

In the current implementation of GlobalRotScaleTrans, we will rotate points only when bbox is presented in input_dict (here). However, in PC Seg, we need to rotate points without bbox. In this PR, I add a if condition for this situation. I also add some input check & perform params transformation in __init__ to save future time.

I discover that in this unit test, someone has tested GlobalRotScaleTrans and the expected_points don't perform rotation. So, I wonder if 'don't rotate points when bbox is not presented' is the desired behavior of this function?

I discover that in this unit test, someone has tested GlobalRotScaleTrans and the expected_points don't perform rotation. So, I wonder if don't rotate points when bbox is not presented is the desired behavior of this function?

I guess it should be a legacy issue. We don't have a separate class for points before, so previously some transformations related to points are achieved with bounding boxes (a hack workaround). Similar problems exist here. Maybe we can gradually refactor these similar operations in the future.


[production] special characters do not render for .docx files

Steps:

View a .docx file that contains a special character (example: https://osf.io/7dybc/)

Expected:
Rendered version of the .docx file should reflect its contents exactly.
Actual:
Special characters (é, ö, etc.) do not render when viewing the document on the OSF.
The first name of the author listed below should read Belén, not Beln. His last name should read Fernández, not Fernndez.

To start with, perhaps a message stating that the rendering of docx files
is imperfect would suffice?
On Mon, Feb 9, 2015 at 11:54 AM, Mallory Kidwell<EMAIL_ADDRESS>wrote:

Steps:

View a .docx file that contains a special character (example:
https://osf.io/7dybc/)

Expected:
Rendered version of the .docx file should reflect its contents exactly.
Actual:
Special characters (é, ö, etc.) do not render when viewing the document on
the OSF.
The first name of the author listed below should read Belén, not Beln.
[image: screen shot 2015-02-09 at 11 46 03 am]
https://cloud.githubusercontent.com/assets/7945215/6110779/225d103c-b052-11e4-9234-f81848c3604a.png
—
Reply to this email directly or view it on GitHub
https://github.com/CenterForOpenScience/osf.io/issues/1739.

That's a sound approach -- Do we have any messaging like that currently? Around other file types?

I think a message might be nice, but I don't think that should be sufficient to close this issue. Special characters shouldn't be dropped; they should either be rendered properly (preferable) or converted to their closest ascii equivalent. There is potential for confusion and, in the case of scientific papers, entirely inaccurate data/results if characters are dropped.
Most of our rendering is an approximation of some sort or another. We don't really have a concept of paper size, we are missing a bunch of fonts, and so on. It could be useful in MFR to have some metadata for each plugin listing the known limitations, so people can see what they will be, but I think that would be a) a larger issue; and b) a completely separate issue.

I filed an issue for pydocx: https://github.com/CenterForOpenScience/pydocx/issues/109

Looks like it's MFR, not pydocx.

@mfraezz, since I suspect you have MFR all checked out and building and everything, could you make that change?

@nanchen47 -- Did this one deploy to Production as well? Thanks.

@lbanner this pr is merged into develop 28 days ago. I believe it should be on production.

@mckidwell Can you do me a favor and just double-check this one on Production with some/one of your projects? TY

@lbanner .docx files are timing out and failing to render; will let you know when I'm able to check.

@mckidwell Looking into a fix for docx rendering now.

.docx rendering fixed by @sloria now -- TY!

Thanks @sloria! Just checked a project - special characters in .docx files are rendered as expected.

Marvelous -- Thank you @mckidwell -- all is good and closed with this one.


Question: use "column_editable_list" like function in "list_row_actions"

i added 2 row action buttons in the list view by overriding "list_row_action". the buttons are meant to do a simple update in a db colummn. it would be very helpfull if the "x-editable" thing could be done throug the action buttons as it is shown in the node row.
is this possible?

stale...closing


First character of model items are now lower cased

The first character of model items are now lower cased since upgrading from 6.0.0-beta9 to the latest pre-release of 1.0.0-rc1.
Example:
SomeModel {
modelID (string, optional),
description (string, optional),
useJobPrices (integer, optional),
}
I'd rather have everything c# cased, where first initial is capitalized. I can't seem to find settings for this anywhere and I'm at a loss. Anything I can do here?

I figured it out. Looks like swagger now defaults to the camelCase contract resolver. I modified my code in ConfigureServices to use the DefaultContractResolver, which is pascal case.
services.AddMvc()                 .AddJsonOptions(                     options =>                     {                         options.SerializerSettings.Converters.Add(new StringEnumConverter { CamelCaseText = false });                         options.SerializerSettings.ContractResolver = new DefaultContractResolver();                     });


Document "webResources"

From the README, I can't figure out what to pass for "webResources", or really how to use remote resources at all. Is it recommended to just use https://www.npmjs.com/package/juice2 instead?

I will update the docs so that in the bullet about options (below) there is a link to https://www.npmjs.com/package/web-resource-inliner
webResources - An options object that will be passed through to web-resource-inliner for juice functions that will get remote resources (juiceResources and juiceFile). Defaults to {}.

I went to see the tests for web-resource-inliner, but I still don't understand what to passe to webResources to provide an url for an external file. Could you please provide an example?

@francoisromain
Hope this little example can help you : )
var options = {
  preserveMediaQueries: true,
  // this is passed to the web-resource-inliner package
  // docs are in https://www.npmjs.com/package/web-resource-inliner
  webResources: {
    scripts: false,    // don't fetch external scripts
    images: 500     // inline images with size smaller than this (kb)
  }
};
juice.juiceResources(htmlString, options, callback);


Dissenter refuses connection from trends.gab.com

a little while ago the dissenter comments connection in trends.gab.com stopped working ("dissenter.com refused to connect"). I used my vpn to come from another IP /country. I used another user. I used several browsers and computers. No luck.
I can log on gab.com, but if I try dissenter.com I get kicked over to trends.gab.com. What's going on???
Now that I think about it, it sounds like a circular reference issue - trends.gab.com>>dissenter.com>>trends.gab.com.

  I just wanna be able to swear in youtube comments and not get shadowbanned for it.

Maybe I'm misunderstanding how this works but it looks like 'someone' just needs to host the comment database and fork this code to point to it.
There's a hosting/bandwidth cost involved, but probably the main factor is the legal risk of hosting such comments (or a lot of work to moderate).

Maybe I'm misunderstanding how this works but it looks like 'someone' just needs to host the comment database and fork this code to point to it. There's a hosting/bandwidth cost involved, but probably the main factor is the legal risk of hosting such comments (or a lot of work to moderate).

Like with mastodon, use federated nodes and wash your hands. The legal risk is a non-factor on this thing, it's mostly that those people over at gab just lost interest because they weren't ready for it either.

Maybe I'm misunderstanding how this works but it looks like 'someone' just needs to host the comment database and fork this code to point to it. There's a hosting/bandwidth cost involved, but probably the main factor is the legal risk of hosting such comments (or a lot of work to moderate).

It's more than just hosting a database, but also a web site. There is also a web site to host. The extension would display a frame of the web site to log you into the extension.

Looking at an old snapshot of the site from 2019, it seems it was just a news aggregation site at first with the ability to comment on the story on the web site, and display the same story and comments in the extension.
https://web.archive.org/web/20190228031600/https://dissenter.com/
Doing a comment based on a URL seems to go to a different level of things.


changed some comments: typos and cosmetic

Hi, thank you for your pull requests!
It is best practice to submit a single pull request when the changes across different files are similar. In this case you opened six PRs (#3, #4, #5, #6, #7, and #8) that are all small typo changes to different files.
It is difficult for me to review these changes in whole. To better take advantage of GitHub's workflows please submit a single PR with all of these proposed changes. When you make the new PR, make sure the PR includes a well written description that briefly explains what was done since "typos and cosmetic" is vague.
As a result I'll close this and the others in the series, and wait for your update. Thanks!


Lcm Calculator

Description 📜
I have built up a Lcm Calculator using html, css, javascript with great UI.
##Built With :-
• Html, Css, Javascript

Checklist ✅

[ ] I follow Contributing Guidelines & Code of conduct of this project.
[ ] I have performed a self-review of my own code or work.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] My changes generates no new warnings.
[ ] I'm GSSOC'22 contributor
[ ] Have you added a working video of your calculator ?

Screenshots / Gif (Optional) 📸

Ok @Chayan-11


⚠ ClamAV has degraded performance

In efeea8a, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 70 ms

Resolved: ClamAV performance has improved in 4dc18c0.


Enable node encryption for all AKS nodes

This change will force all nodes to enable encryption at rest. Two current issues with enabling this. One is that it will require node recreation, the other is that it seems like the feature needs to be enabled per subscription.
More info can be found here.
https://docs.microsoft.com/en-us/azure/aks/enable-host-encryption

@simongottschlag @NissesSenap @jimgus
Considering to just close this because we have moved over to ephemeral nodes. We will never have any data at rest right?

Sounds reasonable

@phillebaba data will be stored on the nodes the VMs are running and I think it would we good to have the facts about how it's handled before closing.


cookie.domain not being set for use across all subdomains

After a user accepts the cookie usage alert on my site I am wanting cookieconsent to add a cookie to expire after 365 days and work across all subdomains but it is not working correctly.
If I only specify the expiry date it works fine but only on the subdomain it was set on (Host Only: true).
If I only specify the domain (.mydomain.com.au) it doesn't set a cookie at all.
If I specify the domain (.mydomain.com.au) and the expiry date it also doesn't set a cookie at all.
here is a code snippet..
window.addEventListener("load", function(){
  window.cookieconsent.initialise({
    cookie: {
      domain: ".mydomain.com.au",
      expiryDays: 365
    },
    palette:{
      popup: {background: "#999"},
      button: {background: "#aaa"},
    },
    content:{
      header: 'Cookies used on the website!',
      message: "my message...",
      dismiss: 'Accept',
      link: '',
      target: '_blank',
    }
  });
});

Any assistance would be much appreciated. I have read through other similar issues but none of the solutions work in my case

@forcefulhandshake With respect to RFC6265 most browsers should ignore that leading dot and allow top level domain access as well as subdomain access. This also holds true for cross sub-domains if you use the root domain without the dot. Give that a try and let me know if that was blocking it for you.
domain: "mydomain.com.au",

-- Update --
CookieConsent takes the value for the domain literally and applies it to the cookie, so how is handled becomes the responsibility of how your browser interprets cookies. I believe setting your domain without the . is the solution for you. See the following reverse stack trace to the options.cookie to see how your domain string is used directly:
---
48:  setCookie: function(name, value, expiryDays, domain, path, secure) {
...
59:   cookie.push('domain=' + domain);
---
645: var c = this.options.cookie;
...
651: util.setCookie(
...
655: c.domain,
---
671: var c = this.options.cookie;
672: util.setCookie(c.name, '', -1, c.domain, c.path);


[Enhancement] improve mv rewrite performance

Why I'm doing:

The MvUtils.canonizePredicateForRewrite may be pretty slow for the sql with a long predicates

What I'm doing:

Check optimizer timeout when rewriting MV
Improve performance of MvUtils.canonizePredicateForRewrite

Benchmark:
Benchmark                                                     (predicateSize)  Mode  Cnt   Score    Error  Units
-- Before
NormalizePredicateBench.bench_NormalizePredicate_Random                   160  avgt    5  53.448 ± 10.608  ms/op

-- After
NormalizePredicateBench.bench_NormalizePredicate_Random                   160  avgt    5  15.312 ± 0.094  ms/op

Fixes #issue
What type of PR is this:

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.2
[x] 3.1
[ ] 3.0
[ ] 2.5

@Mergifyio rebase


Update docker-multinode.md

Modified cd command on line 89 (Adding a worker node) to reflect proper path.

This change is

I signed it!

Needs a rebase but good otherwise


Suggestion: switch from "dep" to go modules

Currently the repo uses dep + Gopkg.toml. Golang now includes a native packaging system "go modules".
One benefit amongst any is that projects using go modules do not need to be installed + developed within $GOPATH maze.
References for go modules + how to switch:

https://blog.golang.org/using-go-modules
https://blog.golang.org/modules2019
https://starkandwayne.com/blog/switching-to-go-modules/

Currently the repo uses dep + Gopkg.toml. Golang now includes a native packaging system "go modules".
One benefit amongst any is that projects using go modules do not need to be installed + developed within $GOPATH maze.
References for go modules + how to switch:

https://blog.golang.org/using-go-modules
https://blog.golang.org/modules2019
https://starkandwayne.com/blog/switching-to-go-modules/

Thank you for your suggestion and the link you provided, we are planning to support go modules.
Thank you for your attention to Alibaba Cloud CLI Project!

Currently the repo uses dep + Gopkg.toml. Golang now includes a native packaging system "go modules".
One benefit amongst any is that projects using go modules do not need to be installed + developed within $GOPATH maze.
References for go modules + how to switch:

https://blog.golang.org/using-go-modules
https://blog.golang.org/modules2019
https://starkandwayne.com/blog/switching-to-go-modules/

The package manager has been changed to Go Modules in version 3.0.26.


ZXHN F680 read_payload_type = 6

Hi, I have a F680 with read_payload_type = 6. I'm out of luck...
To change header = struct.unpack('>28I', infile.read(112)) to header = struct.unpack('<28I', infile.read(112)).
Apparently is in little-edian.

Hi @ludufre, did you succeed in decryption?

Sadly, not.

Can you try again with the latest version of the repo? Also try the examples/auto.py script.


using vue

anyone can provide how to start using vue > astro 3.0 with astrowind template ?
i still getting error and confusing every first setup
i'm sorry i still beginner

We have received this question several times. Maybe your problem is something else, we will talk about it, but just in case, it seems to me that there is a problem when the Vue integration is installed in Astro automatically using this commands:
npx astro add vue

It seems that in this process all the dependencies are uninstalled and then when you want to run the project it doesn't work. I have noticed that if after the previous command you do an installation again, then everything works:
npm i

Another alternative may be to install vue manually as the guide explains: https://docs.astro.build/en/guides/integrations-guide/vue/#install-dependencies-manually
Please let me know if everything is mixed up with this or else to continue investigating. In my environment it works without problems. Thank you.


mipnerf360 dataset is broken

I was trying to reproduce results of BakedSDF and it seems like there's a problem with the ns-download-data mipnerf360 command.
Running it generates a file called nerfstudio-data-mipnerf360.zip, but the file itself is not a zip file but instead is an error message saying there is no file found:

Hi, the mipnerf360 data is preprocessed by nerfstudio team and seems like they deleted it. We now upload it to google drive and you can download it here: https://drive.google.com/file/d/17YW7S7oigh0hOYJBZ8iBNeplP-2Qh9p0/view?usp=sharing (Hope it's ok to distribute it).


Eliminating render blocking css files

In the default head layout partial, the css is just rendered after the html is downloaded, meaning that we will not be able to display any value until the css is downloaded and processed.  Instead, I propose we switch the header to use preload instead of stylesheet.  This has a 93.37% adoption rate for all users. caniuse.com
<link rel=stylesheet href="{{ $style.Permalink }}" integrity="{{ $style.Data.Integrity }}" crossorigin="anonymous">

vs
<link rel="preload" href="{{ $style.Permalink }}" integrity="{{ $style.Data.Integrity }}" crossorigin="anonymous" as="style" onload="this.onload=null;this.rel='stylesheet'" >
<noscript><link rel="stylesheet" href="{{ $style.Permalink }}"></noscript>

[Fixed by 7ba50cd018c8c0b9efe8893253e26ec9a1b5f23a.
Thank you very much!


Sdk docs

Description
A few sentences describing the overall effects and goals of the pull request's commits.
What is the current behavior, and what is the updated/expected behavior with this PR?
Tested
An explanation of how the changes were tested or an explanation as to why they don't need to be.
Other changes
Describe any minor or "drive-by" changes here.
Related issues

Fixes #[issue number here]

Backwards compatibility
Brief explanation of why these changes are/are not backwards compatible.

Sorry, created this by accident.


Iot-inspector windows not working.

After running the start_inspector.bat, The CLI displays there was an error downloaidng the latest Iot Inspector. The same error occurs across different windows machines.

As we're preparing for a relaunch, the current infrastructure is unstable. See https://iotinspector.org/


Move the call to reload ntp settings to the server only

Previously, when the ntp settings were changed every worker would enqueue a message for the server to reload the settings when they went through the Config::Activator.
Multiple calls to restart chronyd within a matter of seconds causes systemd to back off and fail the unit causing chronyd not to run until the appliance is rebooted or it is started manually.
This change makes the server reload the ntp settings every time its config is changed, but removes the call to reload from the activator which removes all the calls generated when the workers refresh their settings. This results in just one call to reload the settings when the config is changed rather than tens.
https://bugzilla.redhat.com/show_bug.cgi?id=1429714

Euwe backport details:
$ git log -1
commit fa7e8750e688ff73ca5863636924cf40131c9dcd
Author: Gregg Tanzillo<EMAIL_ADDRESS>Date:   Tue Mar 7 10:27:46 2017 -0500

    Merge pull request #14208 from carbonin/fix_chrony_start_issue

    Move the call to reload ntp settings to the server only
    (cherry picked from commit 515b4a0c9d769330e749fbc0a3cef48f98d8ac70)

    https://bugzilla.redhat.com/show_bug.cgi?id=1429999


Storage: When moving a file, allow keeping old ACL

From @SimenB on May 3, 2017 13:59
Environment details

OS: macOS
Node.js version: 6.10.2
npm version: yarn 0.23.3
google-cloud-node version<EMAIL_ADDRESS>
Steps to reproduce

require @google-cloud/storage
Instantiate it, and get a bucket
call const file = await bucket.upload(filePath, { public: true })
Verify file is publicly available from https://storage.googleapis.com/
call file.move(somewhere)
Try to access file from https://storage.googleapis.com/, get 403

What I want is either:

some way to pass equivalent of gsutil mv -p flag, which "Causes ACLs to be preserved when copying in the cloud."

file.move(newName, { keepAcl: true }) or something like that
https://cloud.google.com/storage/docs/gsutil/commands/cp#options

allow me to pass destinationPredefinedAcl to the move method

https://cloud.google.com/storage/docs/json_api/v1/objects/rewrite

Copied from original issue: GoogleCloudPlatform/google-cloud-node#2274

From @stephenplusplus on May 3, 2017 14:8
Adding the keepAcl option sounds good to me. For a quick fix, something like this should work:
file.interceptors.push({
  request: function(reqOpts) {
    if (reqOpts.uri.includes('rewriteTo')) {
      reqOpts.qs.destinationPredefinedAcl = '{{predefined acl value}}'
    }
    return reqOpts
  }
})

The incoming reqOpts that you'd be modifying are defined here in the this.request() call.

From @SimenB on May 3, 2017 14:8
I suppose a workaround is to change the ACL after moving the file. The code doing the rename lives in a project which tries to just pass through options to the storage api though. So passing some custom predefined ACL (which defeats the purpose of the nice public: true option) is something I'd really like to avoid. It also adds the extra ACL step to that lib.
The thing we do to avoid it is setting the default ACL of the entire bucket public. Which is fine (in our case), but not ideal.

This came up in Python GCS client library and after speaking with a GCS team member. I decided to close the feature request for the following two reasons: (ref)

Operation isn't supported by the API and follow-up operations to reset ACL may fail in between leaving the object in a bad state. No guarantees.
It's recommended that developers should use bucket-level IAM policies over ACL to prevent inadvertent data exposures.


Avro $ref, documentation and parser

The $ref in the Avro documentation is misleading:
https://www.asyncapi.com/docs/specifications/2.0.0/#message-object-example
Specifically:
"$ref": "path/to/user-create.avsc#/UserCreate"

An Avro schema doesn't have this kind of structure, instead the UserCreate record would be represented at this path (jsonpath syntax):
$.types.[?(@.name=='UserCreate')]
Example Avro schema for reference:
https://github.com/spring-cloud/spring-cloud-schema-registry/pull/41/files#diff-f84d47e6ddc1b7ccd0d9fb0bcbae8a3dR4
Expected behavior
The docs should be updated to remove this misleading reference and probably explain the state of the Avro support here (References to elements within an Avro schema aren't possible due to the difference in structure between Avro and asynapi)

@safetytrick you mean it should be "$ref": "path/to/user-create.avsc#/$.types.[?(@.name=='UserCreate')]"?

@safetytrick you mean it should be "$ref": "path/to/user-create.avsc#/$.types.[?(@.name=='UserCreate')]"?

@safetytrick is this still something you are interested with?

Hi @derberg
I'm having a similar issue while trying to parse an avro schema containing more objects definitions. What is the proper syntax to select one of the objects from the avsc file that has attributes of other object types defined in the same avsc file?
Thanks

@dalelane @M3lkior hi folks, did you try more complex references to Avro files? you have some examples?

My interpretation of this, and how I've been using it, was that the $ref specifies

location of the schema file
fully-qualified name of the record in the schema file
(separated by #/)

(And I only include the name of the record where the schema file contains more than one record - typically, I try to keep the avsc file containing a single definition, and in those cases I would just point the $ref at the avsc file without additional qualification).
But, in the rare occasions where I need to point at an avsc file containing multiple records, such as the example specified I would've written
"$ref": "path/to/user-create.avsc#/org.springframework.cloud.schema.avro.User1"

I hadn't considered using jsonpath to identify where in the avsc file the schema lives - I've just fully-qualified the name of the schema I want to use, as this is consistent with how I reference other Avro schemas within an Avro schema file.
But I've been using this for user-readable documentation purposes, and haven't yet tried running this through the parser to see what it would do.

FYI - There is also a related discussion about being more consistent with JSON schema in how we use $ref in https://github.com/asyncapi/spec/issues/649


Assertion failed: must be connected to a display to use UtteranceQueue features

This occurred while testing the PhET-iO instrumentation of PreferencesDialog.
Steps to reproduce:

State wrapper with Ratio and Proportion
open preferences dialog
Audio tab
Check "Extra Sounds"

The issue seems to be that this listener that announces description fires before the Dialog has been added to the scene graph and redrawn.
I feel like an easy way around this is to have alertDescriptionUtterance not speak during PhET-iO state, which I believe should be the case anyways. I believe this is an example of #1312 again, but we should be able to get around it.

Ok. I found this and blocked it when setting PhET-iO. I also moved the queue clearing to be in updateViews. @samreid can you please spotcheck this?

The change sets look reasonable, closing.


several improvements to DO content script and related items

Replace the workaround from #2437 with a Firefox-appropriate way of getting local asset URLs.
Introduce a new shouldHijackDO setting, which DO content script checks before proceeding. Set that to false once the user has successfully created a cloud server.
Actually check the activePromoId global setting to determine whether to include the promo code text input in our overlay. Previously we were always including it.
If the user toggles the "I have a promo code" checkbox, actually update activePromoId so that we include the promo code text input in the overlay accordingly.
Enable i18n in the overlay.
Lint our Firefox js in "grunt test" so that something like f4f00114a8737c239866ff45308194bc0f80f163 doesn't get past our CI again.

This change is

@bemasc I converted the code to use promises, and decided to make the remaining desired improvements to the DO content script while I was at it: 2d19292
Tested this in Chrome and it's working well. If this looks ok to continue working from, I'll make it work in FF and ping you for a final 👍 .

Thanks for this! Always good to follow up with better hackery :-)
What do you want to do with this, now that the event has passed and you worked around some other issues with website changes?

I ended up using this branch for several improvements that should still be
made for sure. If the latest changes on it look good so far, I'll make the
remaining changes to catch FF up with Chrome and send a ping for a final
thumbs up.
On Tue, May 24, 2016 at 11:15 trevj<EMAIL_ADDRESS>wrote:

Thanks for this! Always good to follow up with better hackery :-)
What do you want to do with this, now that the event has passed and you
worked around some other issues with website changes?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/uProxy/uproxy/pull/2439#issuecomment-221304081

Hey @trevj, started looking at getting this to work in FF but hit a snag when I realized port.emit's payload must be JSON-serializable, so I can't just pass a resolve function the way I can with chrome.runtime.sendMessage. Looks like I have to refactor a bit...

Ok @trevj, got this working in Firefox now too, but I'd like to ask you a question or two about how to clean this up before I have you review.

Haven't had a chance to look at the code yet but based on the description it sounds like @jpevarnek's #2452 could help here and maybe we should coordinate.

:+1:, following our discussion this morning. Thanks for following up and improving the hacks!!

@trevj Merging as per our discussion. Psyched to make further improvements in another PR hopefully soon. Thanks for reviewing!


Misbehaving search icon in nav

Search input's icon repositions and scales oddly when input's autocomplete suggestions are hovered over.

Materialize v1.0.0-rc.2
Chrome v67.0.3396.99 (Official Build) (64-bit)

Reproducible on official docs page here.

Example video: http://take.ms/WRuEr
Screenshot

Fixed in ca9beaf1


React-router-dom & browser refresh

Hello everybody
This topic may has been already posted but I haven't find yet something similar
I would like to have a help in react router and browsers
I got version 4 and when browser does a reload I get that page for ex /login didn't found
How this can be fixed in react-router-Dom version 4?
My route it's :
ReactDOM.render(

,
document.getElementById('ticketappinterface')
); //render

ReactDOM.render(

,
document.getElementById('ticketappinterface')
); //render

Sorry for multiple posts please remove the above
https://pastebin.com/zpfq4BBL

Are you using a server or hosting static content? The issue you described typically happens with static content. <BrowserHistory> only works when you have a server that can handle requests for any url. You will either need to use a server or switch from <BrowserHistory> to <HashHistory>.

Use three backticks on separate lines before and after your code to include multiline code snippets. (The js specifies to higlight the code as javascript).
```js
const variable = 'let it be known'
```
is formatted as
const variable = 'let it be known'

I currently use static localhost but I could make a server by my own in my home
Could you please give me an example in the code I gave you before how it could be used with browser history ?

You wouldn't modify the above code. You would just add a server to handle requests. Basically it would use a wildcard to catch all requests (besides the ones for static files) and return your index.html.
You'll have to search around for implementation details (probably check stackoverflow), but that is the idea.


WIP Control how many jobs of same name to retain

Current coverage is 46.04% (diff: 100%)

Merging #972 into master will not change coverage

@@             master       #972   diff @@
==========================================
  Files           115        115
  Lines          5206       5206
  Methods           0          0
  Messages          0          0
  Branches        864        864
==========================================
  Hits           2397       2397
  Misses         2588       2588
  Partials        221        221

Powered by Codecov. Last update 4eff510...5902c88