Add support for multiple and/or floating point labels

Currently DIGITS supports only a single integer label per image. For many applications, like regression or multiclass classification, this is not enough. I would like to propose adding support for both of these features.
There are several problems with this.

The interface for feeding in a new dataset. Currently, DIGITS supports parsing a folder with a bunch of subfolders, each of which contains images from a single class, and also loading from a pregenerated text file. While the former approach cannot be extended to any of the proposed enhancements easily, the latter can, although not quite straightforwardly. Each line in such text file is currently matched against (.+)\s+(\d+)\s*$ (path/to/image 123). This could be replaced with (.+)((?:\s+\d+(?:\.\d*))+)\s*$ to check for a list of ints or floats (path/to/image 123 4. 5.67).
The format for storing a dataset. Currently, that's an LMDB (I think it's always an LMDB and never LevelDB, although the code seems to support both; correct me if I'm wrong) which stores Caffe's Datum structs. The problem is that a Datum has a field for a label, and that's a single int (proof). Currenlty DIGITS dumps the class label into that field. There are at least three solutions to this, but none seem particularly easy:

Split all databases DIGITS creates in two: one for images and one for labels. Breaks compatibility with previous versions, no reason to do this for single-class classification which most people use.
Add support for two kinds of databases to DIGITS: the old-style consolidated ones and the new-style split ones. Hard to implement and maintain.
Patch Caffe, adding something like float_label or int_labels or float_labels to Datum. Increases memory usage (not much) and changes a widely-used structure (very bad).

I am currently working on patching create_db.py to support split databases, although I'm not sure that this is the best approach. Comments would be very much appreciated.

+1 very needed feature

I'm starting on this now. Do you guys see a need for arbitrary numbers of LMDBs per phase (train/val/test)? I can hardcode it to two options per phase: images and labels, but I'm thinking people might have more complicated use cases. Maybe they would want some complicated mix of LMDBs including images, coarse_classifications, fine_classifications, segmentations, etc.
Are there any use cases you know of that would make this a worthwhile feature?

Bounding box estimation, for example. 4 integers to encode one box.

Thanks for the response! After reading back through these comments, maybe I should explain how I'm implementing this:

The data database contains unlabeled images (datum.label isn't used).
The labels database contains an N-dimensional label for the image. It can be Kx1 for a classification, or 4x1 for a single bounding box, or HxWxK for a per-pixel segmentation, etc.
Or, you can leave your data unlabeled and do unsupervised learning, in which case you only need the data database.

These are the types of problems which may require more than two databases:

Producing "coarse" and "fine" classifications with the same model (see CIFAR)
Mixing tasks, like classification AND segmentation, with the same model

My question is about whether anyone is using models which perform these more complex types of tasks.

How can I train train for this? is it possible now with caffe-DIGITS?
Regards,
Saneesh.

After reading through BVLC/caffe#523, BVLC/caffe#1414 and BVLC/caffe#1698, I think this quote best sums up what is possible with Data layers.

Caffe is perfectly happy with models that make matrix outputs and learn from matrix ground truths for problems where the output and truth have spatial dimensions e.g. reconstruction / de-noising, pixelwise semantic segmentation, sliding window detection, and so forth. The forward and backward passes for these models follow directly from the definitions and Caffe has always been capable of computing these.
https://github.com/BVLC/caffe/issues/1698#issue-53768814

So I'll revise my previous statement and say that the labels database contains 1-, 2- or 3-dimensional labels. Not "N-dimensional". That's because you have to use Datum in LMDBs, which is restricted to 3 dimensions. It may be easier to work with arbitrary blobs using HDF5Data layers.

It has come to my attention that HDF5DataLayer can now produce multiple blob outputs
https://github.com/BVLC/caffe/pull/1414#issuecomment-69285965

For the current iteration, I'm just going to assume that you've created your LMDBs manually and want to use DIGITS for running caffe. In the next iteration, I'll tackle picking a standard input data format so that DIGITS can create your LMDBs or HDF5 files or whatever.

I'm discussing a much more general solution which solves many more problems. It will be a little overkill for the simple multi-labeled example you gave, but will definitely be sufficient.

How can I train train for this? is it possible now with caffe-DIGITS?

It's possible with Caffe, but see BVLC/caffe#1698 (it's hard). I'm currently working on adding it to DIGITS.

@lukeyeager
Thank you very much!
Can we expect this feature in the version DIGIT 3?
How long it will take to finish?
and How will be informed about to the DIGIT users?
Regards,
Saneesh

Hi, @lukeyeager. How is this issue evolving? Is the work in progress in a specific branch?

I should have something pushed to master within a week or two.
My in-progress branch is at lukeyeager/generic-inference. You can take a look at it if you want but beware - it could do bad things like corrupt your data from previous jobs.

#189 has been merged. If you can create your own LMDBs, now you can run your Caffe models in DIGITS. Please see #197 for a discussion about data formats.

Has the multiclass classification feature discussed here been implemented?


Update bt.json

attempt to make the persistent works.

Closes #XXXX

Relates to #XXXX

[x] I have read the Contributing Guide.

/verify

Is this PR abandoned? I can create a new PR for this change if appropriate. It saves people the trouble for any manual post fixes.

Is this PR abandoned? I can create a new PR for this change if appropriate. It saves people the trouble for any manual post fixes.

Apologies for the confusion, but I need guidance on why this PR interests you. It fixes a bug, but I've forgotten the exact details of what it does.

Oh now I recall, yeah I guess you came here after you've lost the config after the new update.
Please do what you think is more appropriate to fix this issue once for all for everyone!

Hi @neoOpus , nice to have you here. Looks @HUMORCE just need you to add a comma by the end of array close. If you still have the repo around, maybe you can still fix it, and have the PR merged as your change.

@neoOpus I used your change and created a new PR #12733. 😊
@HUMORCE , please check it out, and if it works, it closes both.

Fixed in #12733, closing this.


about the shadows of black hole

🐞 **I am trying to plot the shadows of black hole by using the code:
Shadow cast by a thin emission disk around a Schwarzschild black hole.ipynb - einsteinpy [GitHub] - Visual Studio Code - GitHub
I am trying to run this code on the jupyter notebook but not getting any plot due to some error related to "shadowplotter, although,
I have installed the package einsteinpy-0.2.1"Can anyone please guide me about this issue? **
🎯 Goal

💡 Possible solutions

📋  Steps to solve the problem

Comment below about what you've started working on.
Add, commit, push your changes
Submit a pull request.
Add this in the comments:

Addresses #<put issue number here> if you are partially fixing the issue.
Fixes #<put issue number here> if you are completely fixing the issue.

Ask for a review in comments section of pull request
Celebrate your contribution to this project 🎉

@Drsehrish-iftikhar ShadowPlotter was added in EPy 0.3.0. Can you update on your end using pip install --upgrade einsteinpy? That should install v0.4.1 and fix the issue.

Dear
Dr. Jyotrimaya Shivottam
Thank you so much for your response. I will try this, and let you know.
Best regards
Sehrish
On Tue, Nov 28, 2023 at 10:11 PM Jyotirmaya Shivottam <
@.***> wrote:

@Drsehrish-iftikhar https://github.com/Drsehrish-iftikhar ShadowPlotter
was added in EPy 0.3.0. Can you update on your end using pip install
--upgrade einsteinpy? That should install v0.4.1 and fix the issue.
—
Reply to this email directly, view it on GitHub
https://github.com/einsteinpy/einsteinpy/issues/653#issuecomment-1830320917,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BD264VPHOXLXXS6TDZPSEMTYGYLLRAVCNFSM6AAAAAA7EPURW2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMZQGMZDAOJRG4
.
You are receiving this because you were mentioned.Message ID:
@.***>

@Drsehrish-iftikhar I am closing this issue for now. I hope the issue was resolved. If not, please reopen.


feat: git delete branch | FE

Description
This PR cleans up git branch popup, and introduces a new feature: delete branch.
Fixes #12612, #12677,  #12074
Type of change

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
This change requires a documentation update

How Has This Been Tested?
Tests will be provided.
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Test coverage results :test_tube:
:green_circle: Total coverage has increased
// Code coverage diff between base branch:release and head branch: feat-git-delete-branch

Status
File
% Stmts
% Branch
% Funcs
% Lines

:green_circle:
total
56.35 (0.02)
37.76 (0.03)
35.95 (0.01)
56.58 (0.02)

:green_circle:
app/client/src/actions/gitSyncActions.ts
62.29 (0.09)
100 (0)
3.77 (-0.31)
64.29 (1.33)

:red_circle:
app/client/src/api/GitSyncAPI.tsx
45.9 (0.14)
16.67 (0)
10.53 (-0.58)
49.09 (0.03)

:red_circle:
app/client/src/ce/constants/messages.ts
77.25 (-0.01)
100 (0)
32.73 (-0.04)
82.08 (-0.09)

:green_circle:
app/client/src/components/ads/Button.tsx
90.97 (0)
78.26 (2.61)
95.56 (0)
91.55 (0)

:red_circle:
app/client/src/pages/Editor/gitSync/utils.ts
95.45 (-4.55)
66.67 (-33.33)
83.33 (-16.67)
93.33 (-6.67)

:sparkles: :new:
app/client/src/pages/Editor/gitSync/components/BetaTag.tsx
100
100
100
100

:green_circle:
app/client/src/pages/Editor/gitSync/components/BranchList.tsx
29.91 (6.42)
19.51 (9.38)
0 (-2.5)
32.71 (7.21)

:sparkles: :new:
app/client/src/pages/Editor/gitSync/components/BranchListItem.tsx
40
0
0
41.67

:sparkles: :new:
app/client/src/pages/Editor/gitSync/components/BranchListItemContainer.tsx
100
83.33
100
100

:sparkles: :new:
app/client/src/pages/Editor/gitSync/components/DefaultTag.tsx
100
100
100
100

:sparkles: :new:
app/client/src/pages/Editor/gitSync/components/LocalBranchList.tsx
58.33
66.67
25
70

:sparkles: :new:
app/client/src/pages/Editor/gitSync/components/RemoteBranchList.tsx
87.5
83.33
66.67
87.5

:sparkles: :new:
app/client/src/pages/Editor/gitSync/components/RemoteBranchListItem.tsx
100
100
100
100

:sparkles: :new:
app/client/src/pages/Editor/gitSync/hooks/useActiveHoverIndex.ts
15.79
0
0
14.29

:sparkles: :new:
app/client/src/pages/Editor/gitSync/hooks/useFilteredBranches.ts
18.75
0
0
15.38

:sparkles: :new:
app/client/src/pages/Editor/gitSync/hooks/useHover.ts
11.11
0
0
14.29

:red_circle:
app/client/src/reducers/uiReducers/gitSyncReducer.ts
10.94 (-0.73)
0 (0)
1.75 (-0.14)
10.94 (-0.73)

:red_circle:
app/client/src/sagas/GitSyncSagas.ts
10.03 (-0.38)
0.92 (-0.06)
8.33 (-0.37)
12.27 (-0.38)

/ok-to-test sha=258ccf9

Issue while installing packages

/ok-to-test sha=258ccf9

/ok-to-test sha=1e95214

https://user-images.githubusercontent.com/1573771/163066371-a8e7fb9d-eaf3-4e37-be6e-22e600fdfe9f.mov

Will this file be re-written? app/client/src/pages/Editor/gitSync/components/BranchList.test.tsx

Check utils test file. The function moved and so did the tests.

/ok-to-test sha=cb5a188


Points of Path not Accessible

I am using the Path class from the painting.dart file to draw lines with a CustomPainter.
There is no way for me to access the contained points.
This is critical as the amount of points stored in the Path object can get huge (around 100k) which will lead to bad performance or I simply cannot work with these points.
As I cant access the points, I cannot do anything about it.
On the other hand storing these points to another list will unecessarily increase the used memory.
Please make a simple attribute or getter to access the Points stored to a Path object.
Thanks!

I attempted to add this feature previously but it was rejected out of concerns that it may hinder our ability to further optimize path handling.  The list of points you pass in may not be the list of points the path uses, and relying on them from a fairly opaque object may end up placing constraints around how we can further optimize paths.
Can you speak more about what you're trying to achieve?

https://github.com/flutter/flutter/issues/18458

My goal is to implement a writing app.
It includes normal writing - I use a CustomPainter and add the Points to a Path and draw the Path by canvas.draw(). This is alot faster than drawing lines manually.
It should also delete lines - for this I need to access the points of the lines directly (coloring the parts shite will not work).
And I need a bucket fill option. For this my plan is to convert my lines to a png byte array and use a flood fill algorithm.

Drawing paths is definitely the way to go there.  However, if you want to delete lines you should be able to either keep a copy of the path and redraw it using the background color, or just use something like path.computeMetrics() to find the segment you want to delete.
You can convert your paths to images by using Picture.toImage and do your flood filling that way - although you might be better off just keeping it as a vector and re-drawing your stroked path as a filled path and then rasterize later.

There are a few more features where I also need the points.

None of the features you're mentioning would actually benefit from having the points of the path available though.  Part of the philosophy of Flutter's API design is to avoid making it easy to do things that would hurt performance, and having the points here would likely hurt you more than help - at least for what you're mentioning so far.

Oh, and there's more discussion here https://github.com/flutter/engine/pull/5632 than the issue I originally linked.

I am aware (and using) the toByteData() feature of Image to manipulate the bytes directly.
I cant use the background color as the background might be a gradient.
I do understand the concept behind flutter, still I think exposing "everything" but making clear that you should avoid using these features in favor of performance is better than just locking things completely away.
I am used to edit the source code often to achieve better results (performance or just different features) so I think that flutter is really lockinh things away from me.
I dont want to make an uneducated statement and I dont know how flutter handles things internally, but if I store the Point to a list will it copy the point or will it just keep a reference to it?

You'll end up with duplicated memory usage even if they are references - even if the path had references it'd still be an 8 byte pointer (instead of the 8 byte double).  And even then, it's entirely possible that now or in some future version the points Skia is using to draw the path will be different from the points you have in your array because of some optimization - for instnace, if you have a list that's literally describing every point in the path at a specific resolution (e.g. a rasterized version of the path), it will be many times larger than the actual path which may end up being described as moves, lines, and beziers with control points.  A bezier in particular might take 1000s of points to describe when rasterized, but only 6 doubles to describe as the start, end, and control points.
The problem with exposing everything is it constrains what we can do - we're trying really hard to make sure we avoid breaking changes to our users, and exposing something like this makes it harder to not break someone down the line. That said it'd be nice to allow people to more easily extend the engine API without needing to fork it.

I thought a Point was more than just the 8 byte double. Okay thanks for background info!
This will leave me with an even bigger issue. I do understand why the Path is a lot faster and more memory efficient then. I can understand that you dont want to break stuff (which is good!). Storing the points then will have a huge impact on memory and is no option for me. I have tested storying points (necessary ponts). I even created some easy algorithms that remove unnecessary points from my list - which still was a lot slower than using a Path.

We should move to the original issue as the discussion isnt spread out to far then .flutter/engine#5632

A point will be two 8 byte doubles.
Skia stores paths internally as vectors of verbs and control points (and for certain types of curves another number as well).
I'm not sure what exactly your use case is, but I'm pretty confident you'd be better off storing paths and manipulating them rather than trying to store all the points you want to draw.
I'd prefer to continue discussion here as this will be more focused on your use case - if Hixie wants to respond to that one that's fine, but right now it's a closed pull request and won't have as much visibility to others who might be interested.

To make things clear:
The feature I am actually needing is to get a certain SubSequence of a Path and the Points that describe that Path. Even an optimized List of Points will work fine for me as I can edit the sequence by removing or adding Points to the Path.
If I understand correctly, the path is optimized in a way that is not changing the outcome (the look) of a Path. If I change up the given list of points and let the backend compute that Path again I should have the expected result without breaking anything. Even only getting the list of points will work as I can destroy the old path and replace it with a new one.

You can convert your paths to images by using Picture.toImage and do your flood filling that way - although you might be better off just keeping it as a vector and re-drawing your stroked path as a filled path and then rasterize later

I am storing a list of paths as every path can have different stroke widths and colors.
Can I combine these paths to one ? I tried but I only got it to work with one color and one stroke width.
I cant use the fill option between multiple paths.

So you could combine them but then you would lose the different color/stroke widths.  But if you were just trying to do fills that should be ok right?

Most of the source code you'd be interested in would be in https://github.com/flutter/engine/tree/master/lib/ui/painting

So you could combine them but then you would lose the different color/stroke widths. But if you were just trying to do fills that should be ok right?

Oh right of course that will work. Why wasnt I thinking of that. Thanks alot! I now just need to figure a way to delete or split up a path. And I am pretty sure that thats not possible by drawing paths.
But it will not be to hard. I will edit the byte data directly.
It would be convenient to have a isIntersecting(Path other) method.

There are definitely ways to do that :)
Path p1 = Path()..lineTo(10, 10);
Path p2 = Path()..moveTo(10, 0)..lineTo(0, 10);
final Path intersection = Path.combine(
        PathOperation.intersect,
        p1,
        p2,
      );

       // Check if the intersection created a path. If so, the
      // paths overlap.
      if (intersection != null && intersection.computeMetrics().any((ui.PathMetric metric) => metric.length > 0)) {
  // They overlap or intersect
}

You can use the return value of computeMetrics to also extract specific segments.

There are definitely ways to do that :)

Okay thanks alot! I wasnt aware of that.
I think that thats "a lot" to do for that simple task. I will use this. Thanks!

Just be aware that it's not cheap.  Computing the metrics is expensive, and even doing the path combination is potentially expensive - although probably not as expensive as if you were doing it on a whole rasterized image.
If you're doing it for specific effects/tasks that should be fine - but doing it every frame (doing it in a widget's build method) is probably a bad idea.

I will figure out on how to do it just once. At the moment I am just creating the layout for each screen and researching about the functionality.
I got most of it working, thats nice!
Thanks again for the help :)

Hi @dnfield is there a way to check the point is on unclosed path? Or calculate the cross point of 2 unclosed path?

PathMetric access first and last point should be without any calculations involved

What's not working?

Is there a way to know lines intersecting? Your sample code I think it didn't work

I was also surprised that it is not possible to access the points on a path. As a result, I resampled the path using getTangentForOffset() on the metrics of the path and extracted the coordinates from the returned offset. Are there drawbacks to this approach? I am not 100% sure what the original question was about, but this seems like a way to let Flutter do it's optimization while still being able to sample points on the path with arbitrary resolution.

Hmm.  Based on conversation with @deakjahn and how a Path.combine with paths with curves would be difficult to track, I think we should reconsider adding access to Path::iter. See specifically https://github.com/flutter/flutter/issues/62449#issuecomment-671572751
@Hixie - any thoughts on this? You originally had some objections to exposing a path verb iterator.

I wouldn't mind exposing something on PathMetric to get something like this, but it would be a very large burden on us to maintain in the future (e.g. imagine if we move from Skia to another graphics backend that stores path data differently), and it would probably end up being a very expensive API.

PathMetric is more expensive than iterating the verbs/points of a path. If we force people to compute the metrics every time they want to iterate the paths, we may be doing work that just gets thrown away - i.e.  we're forcing them to measure the length of a contour when really they just want its points.
It shouldn't be a large burden if we have to maintain it ourselves - the worst case scenario would be that the methods that add verbs/points on a path should update some internal storage we keep for them, which is really all Skia does anyway.

Sorry, Hixie, but I have to disagree. Everything, really everything in the whole industry and beyond uses the same curve segments to describe paths since PostScript times and maybe even earlier. I never saw anybody reinventing that wheel and coming up with something completely new. I really think that the chances of lines and Bézier and similar curves going away, no matter what engine you might end up using later, are just about zero.
And I don't believe in it being a large burden, either. If you just add two fields to the metric, an enum with the verb and a double array with the coordinates, the impact for you is minimal because both should only expose underlying, readily available data, and doesn't need to produce anything new that might be theoretically get out of sync any time in the distant future.

I'm really a bit baffled, Dan, because we have SkParsePath::ToSVGString() right now. We seem to prefer the iterative way rather than SVG, fully agreed, I just mention it to provide the complexity we speak about. A simple for that iterates over the path and has six or so switch cases. All that's needed would be to make a version that instead of putting it into a string, returns a list that has a verb enum and a co-ords array in each entry. Both are there for the asking, needs to be copied directly and if ever there will be another verb (which I really doubt, actually), adding a single new case to this switch would be the very least of your worries.

Skia has an SkPath::Iter interface already that does the iterating for us. The linked patch (https://github.com/flutter/engine/pull/5632) provides a way of exposing that. It needs some cleanup/updating for style changes.
SkParsePath::ToSVGString uses the SkPath::Iter under the hood anyway (or uses the same things Iter does - it's been a while since I checked).
My comment above is to indicate that I do not think we should try to bolt this on to PathMetrics, which is a interface exposing most of SkPathMeasure, which involves also measuring the length of a path contour, which is more expensive than just listing its points/verbs.  For example, getting the length of a cubic vs. just saying it's a cubic and listing its control points.

Completely agree on all counts, PathMetrics included. Your patch was a full-blown solution that's the completely elegant way of solving the issue with the introduction of a real PathSegment and the iterating mechanism. I couldn't get better than that. My repeated attempts at simpler substitutes (both SVG path in the beginning and mentioning a simple list of a simple struct [a very simplified version of that PathSegment] were just surrogate solutions to make the change simpler, the added complexity smaller, so all-in-all, the whole thing more palatable).
My reference to SkParsePath was actually a link to check it out, so yes, completely, it uses a simple Iter.

It shouldn't be a large burden if we have to maintain it ourselves - the worst case scenario would be that the methods that add verbs/points on a path should update some internal storage we keep for them, which is really all Skia does anyway.

That's by far not the worst case scenario. Consider a graphics system that can get Path objects out of a data format that doesn't express paths as verbs, but as parallel batches of identical components (e.g. "here's 32 arcs and 32 lines that form the outline of the path"), where the logic to convert that data into pixels is a shader that never constructs the path (it just determines for each pixel how many of the lines are crossed when you draw a semi-infinite straight line from that pixel). We'd have to implement the Path canonicalization logic ourselves, order the segments, figure out how they map to the kinds of segments we have today, etc.

We could have a computePoints method or some such that returns something similar to PathMetrics. An iterator directly on Path would be difficult because Paths are mutable and iterators are garbage collected so they interact poorly.
I'm not really sure I understand the use case though. The purpose of dart:ui is to draw pixels on a screen, not to generate SVGs. If you want a rendering engine for generating files based on drawings or text from fonts or whatnot, I recommend linking in a library to do that (e.g. Skia, or Cairo, or creating one in Dart), not using dart:ui.

For your proposed rendering system, it seems like we'd either never have a Path at all, or we'd have something with points/verbs we could iterate (even if they'd be different than what paths are).
I agree that the Path object itself should not be iterable, but we could certainly make a PathSegment iterator similar to the PathMetrics, which already has the caveats around if you mutate the path after this it's invalid.
The goal of iterating the points is not necessarily to generate SVGs, but to discover what points were used after an operation like intersecting bezier curves. The math to do that is not trivial. Linking in Skia or Cairo yourself means you lose interoperability with Flutter, because we can't share Skia objects with someone else's Skia (and forget Cairo).
Creating one in Dart would be fine, but that's a large undertaking for someone who needs to inspect the shape of a path that has been manipulated by API dart:ui exposes.

We have a Path already, I'm not sure what you mean by "never have a Path". We couldn't just drop Path...

...but to discover what points were used after an operation like intersecting bezier curves

For what purpose?

We have a Path already, I'm not sure what you mean by "never have a Path". We couldn't just drop Path...

If I understand your example, we have some binary format where what we think of as an array of points and verbs (e.g. "move to 1.0 1.0") are stored in some other manner that is less optimal for iteration. In that case, someone either built it using a ui.Path object, in which case we store the commands from the interface we have, or someone built it from a stored binary format, in which case why are we even calling it a Path?

For what purpose?

Serialization without rasterization.

@dnfield I dusted off and ported my C# code that reads and interprets fonts (I happened to write a couple of font converters in my life :-) ) and while it would be theoretically possible to use it to help my use case, but not on Flutter Web. The font file can't be accessed in a browser sandbox (Android and iOS can be handled with a platform plugin). So, we're in a position where even bundling a complete extra engine (not even considering how insane is to bundle a full implementation of something that's already there in the engine) is not enough to solve the problem.

In my example, consider a Path constructor that takes an efficient representation of a path that cannot be (easily or cheaply) represented using the existing Path constructs, but that can be used in all the ways you would normally expect to use a path (except stroking probably). (This is not an academic example, it's likely we'll do something like this one day.)

Except stroking seems like a big exception.  Why not just say except stroke and except iterating (control) points efficiently?

Right, that's why I proposed having a method (like computePoints, similar to computeMetrics) to get the object from which we can iterate the path components.

(In general though I am very reluctant to add more API surface, as it just means more bugs for us to fix, at a time where we already have a lot of bugs to fix.)

I'm trying to add some shapes that would follow a cubic bezier path segment. With the points I could query a X position and retrieve a Y position (like a math function). With no points, I need to build the function myself.

I'm currently working on a "Drawing app" where the user should be able to save drawings and come back to these later on.
so I need to store my paths locally, and since I can't serialize them at the moment does anyone have any other ideas of how I can achieve this?

@Hampusheglert use lines segments

@Hampusheglert use lines segments

What do you mean by line segments? I'm making a pixel art app and because you can't iterate through Paths, I find myself having to save thousands of points on big canvasses when multiple layers are involved.
The ability to save and come back to a project later is the only remaining feature among the ones I originally planned.
How would you propose I save the paths so that I can serialize and deserialize them easily?

After many years, nothing has been done. The required knowledge is right inside the engine, it just isn't exposed. @Hixie made a vague remark back then about "what if the underlying stucture ever changes". Well, it hasn't changed and it probably never will. Still, there's no single function that would allow us to use the data that's already there.

As it happens we are currently doing a significant rewrite of our graphics backend (Impeller) that replaces Skia. As part of that it is quite possible that we will change how Paths are internally represented.

OK. And then, will it be still impossible to get back what you put into it? And why?

I peeked into https://github.com/flutter/engine/blob/main/impeller/geometry/path_builder.cc and I can't see any fundamental difference to what we have described paths since antiquity. :-)

The point is that we should not lock ourselves into a specific API. For example, if we do something like the spinel path backend one day, we may not have points to report.
Generally speaking, it's best to view the Path API as a write-only API, in the same way that when you call print you can't later find out what you printed to the screen, or when you set an integer to 1 + 2 you can't later find out what those components were that led to the 3. When path manipulation is the goal, it's typically better to use a library that allows for path manipulation as a first class citizen, and then convert that to a Path if you need to render it. (I'm not an expert at this so I don't have specific libraries to recommend unfortunately.)
Could you elaborate on your use case?

I can't add much to what I faced already two years ago in https://github.com/flutter/flutter/issues/62449. Yes, it's obvious that most apps don't need this functionality. Still, as mobile moved from just fancy browser-like operation to full fledged software, these happen to include graphics-capable software like

any app that uses the outline of a text to create an effect, eg. a clipping path or a fancy border;
apps that allow the user to create graphical content and need to store that content for later re-use or upload it to create either digital or traditional products using the graphical content (print, press, etc).

Editors, creators, anything that lets the user create artworks for later or external use. Fonts are the next, even more problematic issue because you can't get the path from your created text back at all (actually, this was the main reason for me to even start this issue). While what you say can be done with very simple curves (any intersection of Bézier-curves is already very far from trivial), I don't think it's very feasible to expect every single app that needs something like this to bundle a complete, parallel font loader and rendering subsystem. No, the usual suggestion of "note the fonts and steps and let the other side recreate the artwork from scratch" doesn't work, you can never count on the recreated copy to be exactly equivalent.

Those use cases, especially editors and the like, are precisely the kinds of use cases for which Path is not well suited (and never will be). Path is an output mechanism. Editors need to be able to mutate paths, introspect them in detail, do maths on the paths, etc. A dedicated library is the way to go.

Again: handling fonts is built into the system. From a Flutter app, you can't access the font. You can't get the font file itself, even if you would be ready to bundle a whole font renderer. I really can't understand the animosity towards getting a path. :-) I did quite a lot a graphics work in my career in all the various platforms and I've yet to see one but Flutter that obviously uses the very same approach inside, has paths, code to render, intersect, merge, manipulate in whatever way possible, has obvious access to the path internally, and is not willing to expose it. Back then I was working on an app that allowed the user to create an artwork, upload it to a service that printed it to a variety of materials, T-shirts, mugs, mouse pads, whatever. We did bundle quite a lot of parallel work to cater for cases that Flutter didn't allow us to do (including crafting a complete WebGL based replacement image filtering engine because Skia engineers decided to remove functionality from the engine they decided nobody had used, you might not remember it but we discussed the details back then :-) ). But really, bringing your own code for many tasks missing from the system is a very good approach but very frustrating when you have to reproduce large parts of the underlying engine just because it's there, just closed, unavailable.

I really can't understand the animosity towards getting a path.

@deakjahn I'm sorry I haven't explained it well enough.
There's several factors:

The wider the API surface we provide, the more we are constrained in how we can improve Flutter in the future. We want to have a lot of latitude for improving Flutter especially around aspects of the engine that are performance sensitive since that directly affects Flutter's performance. To put it a different way, we have to make a trade-off between enabling some edge cases and enabling fast performance in the future and we chose fast performance in the future.

API design is driven by use cases. Different solutions are better at different things. The specific use case that dart:ui is intended for is painting UIs as fast as possible. It's not intended to be used for creating editors. These two use cases have very different needs. To put it a different way, we have to make a trade-off between being good for one thing, and being mediocre for two things, and we chose being good for one thing.

The quality of a library is driven in part by the expertise of its developers. Our expertise lies in creating APIs for UI and implementations for rendering. Editor APIs require expertise in very different areas, areas that we are not experts at. To put it a different way, we have to make a trade-off between a high-quality library and a mediocre library, and we chose high-quality.

(Whether we succeeded at being fast, good, and high-quality is debatable; however, to the extent that we have failed, we would have certainly failed even more if we'd chosen the other trade-offs.)
If you want to create an editor, there are many libraries you can use that offer you this kind of thing. You can wrap Cairo or Skia for example, or you could pay Freya Holmér to port her Shapes library from Unity to C or Dart. Trying to use Flutter's output library as an editor backend is not a winning strategy.
It's like, if I have a bicycle and want to go to the moon, I don't strap large solid fuel boosters on the side of my bicycle. I put the bicycle aside and build a dedicated rocket.

Any update based on the latest impeller engine? For most of cases I can save the original points and verbs and replay to path, but when I use path operation (union, xor, etc), then it's hard for me to calculate the final points from path. Look I have to do it by calculating the intersection and points? On iOS, they have CGPath.applyWithBlock, this is exactly what I want. Or do you guys have any suggestion on this function?


🚀 GitHub Issues: Additional sort options added to GitHub Issues Plugin

Plugin Name
GitHub Issues
🔖 Feature description
Add additional sorts to the github issue plugin to allow sorting by created date and updated date.
🎤 Context
It would be nice to review the oldest github issues with my team, but everything is sorted by newest updated date first. It would be helpful if I had the functionality to sort by created date and updated date. This would allow us to identify and work the oldest issues first.
✌ Possible Implementation
No response
👀 Have you spent some time to check if this feature request has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
None

Looking for contributions to implement this request! 🚀


fix: parent fork repo context is not accessible from forked repo

What
What does this PR add or remove?
Why
Why are these changes needed?
How
How do these changes achieve the goal?
Test
How has this been tested? How can a reviewer test it?
Checklist

[ ] I have self-reviewed this PR
[ ] I have added tests that prove the feature works or the fix is effective

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: maggie98choy:x: git commit and pushgit commit and push seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


[hi] Add content/hi/docs/contribute/_index.md

localized the file from website/content/en/docs/contribute/_index.md
@anubha-v-ardhan please have a look

fixes #31012

noted, thanks @sftim @divya-mohan0209

/label tide/merge-method-squash
(it'd be better to have the commits squashed by the author - that's you @Nawaz027 - but if that's not feasible then Tide can do it instead)

@sftim @divya-mohan0209 Please have a look.

@AvineshTripathi I am not sure. @anubha-v-ardhan @divya-mohan0209 are the right people to answer

@anubha-v-ardhan @Nawaz027 should we keep सिग as it is or should we change it to SIG

I think we should go with SIG it being short for "special interest group"

ok, thanks @anubha-v-ardhan . I will change सिग to SIG in the above file.

/assign
/label tide/merge-method-squash

/lgtm
/approve
Thank you @Nawaz027 :)


Not able to commit and push

While pushing the code from vs code I'm getting the below error

@neubig can u please help me out

Please use nvm.

This is not a bug. The action triggered by commit in vscode does not use the node version switched by nvm in the vscode terminal.
A viable approach is to commit code in the terminal using command line. If you want to use the nvm version when committing code in the vscode GUI, please refer to https://stackoverflow.com/questions/44700432/visual-studio-code-to-use-node-version-specified-by-nvm.

Yeah this is not related to this repo. I think xcodebuild explained it. I'll just give a tag and close it. If you find it related to this project you can reopen it.


Load Image from String

Resolves #1723. The element function to load svg (and other images) from a string is prototypically implemented. To avoid redundant code for Layout, LocalName and Figurable I used #[duplicate_item()]. Let's talk about it and finalize it.
Example:
SVG from string:
#let svgstr = ```<svg xmlns="http://www.w3.org/2000/svg" height="140" width="500"><ellipse cx="200" cy="80" rx="100" ry="50" style="fill:yellow;stroke:purple;stroke-width:2" /></svg>```.text
#image.decode(svgstr, "svg")

With modification:
#image.decode(svgstr.replace("yellow", "green").replace("purple", "cyan"), "svg")

Regular image:
#image("demo/xxx.svg")

I have some changes that fix an intricate issue with set rules not applying on decoded images. If it's alright with you, I would just push to this PR.

Thank you!


Selecting a particular pronunciation of a word?

I'm using Home Assistant with Larynx to read a daily weather forecast as part of an automation. It pronounces "southwest winds 0 to 5mph" with a long "I" sound, like "eye".
Is there a preferred way to have it pronounce the weather related phenomenon rather than what one does with a rope around a post?
If this support request should be posted in Larynx, I'll close it and open one there.
Thanks!

SSML will work for this, something like:
<speak>southwest <w role="gruut:NN">winds</w> 0 to 5 MPH</speak>

Since you use the MaryTTS API for Larynx, I've pushed an update that will assume the input text is SSML if it starts with a <.

Super helpful! I'll install the update and give it a try.

Works great! Thanks again @synesthesiam !


[Improvement] Add enable/disable toggle to all auth schemes

Insomnia Version: 5.14.7
Operating System: Windows 10

Details
When setting an Auth method with parameters, and then when changing to another Auth method or when disabling Auth, the current Auth settings are lost. Why this behavior? It's very annoying when switching from an environment using an Auth of type "foo" to another using an Auth of type "bar" (for the same API / workspace). Postman is persisting those settings.

Can you give a concrete example of this? Auth fields are preserved when no information will be lost. For example, both Basic and NTLM have username/password fields, so switching between them preserves the settings. However, switching between something like OAuth 2 and Basic don't have any shared fields, so settings are lost.
I'm curious, why is the situation you are in where you need to use the same request with multiple authentication types? Can you describe your setup and use case so I can better understand your problem?

However, switching between something like OAuth 2 and Basic don't have any shared fields, so settings are lost.

Yes, that's the case I'm talking about.
My use case is when requests in the same API (workspace) can either don't use any Auth method (typically dev or beta environments) or use OAuth (typically production environments).

When I'm developing on my local (dev) environment, I don't set any Auth method (because we have a special mechanism that bypass OAuth using a special query string parameter)
When I'm debugging a production environment, OAuth is required. So in Insomnia I switch the Auth method to OAuth, but as settings aren't preserved because of what you wrote above, I have to re-enter OAuth credentials (actually I have to re-enter Insomnia environment variables that contains the needed OAuth credentials, crednetials that different for each Insomnia environment)

I'm not sure if I'm explaining correctly, don't hesitate to ask for clarification.

I'm pretty sure I understand now. Thanks for clarifying your use case.
If you are only switching between OAuth2 and No Auth, maybe a toggle to enable/disable auth would work for you? In the last release, I added this toggle to a few of the simpler authentication schemes (Basic, NTLM, Digest, etc) but not OAuth.
If there were an enable/disable checkbox in the OAuth 2 settings, would that solve your problem? Or, do you have cases where you are mixing other auth schemes as well?

In my use case I'm only using either No Auth or OAuth 1.0 (remember, I'm the annoying OAuth 1 guy on Twitter and in #197 😄). So yes, this will do the trick if you add this toggle for OAuth 1.0.
Out of curiosity; why not adding this toggle for all Auth methods?

Awesome. I'll update this issue name to reflect the task.

Out of curiosity; why not adding this toggle for all Auth methods?
I don't really have a good reason, other than I was feeling lazy at the time 😛

Ah, this was done a while ago


Zoom in built-in browser

Hi,
Great peace of work, installer for v0.11 works like a charm...
My "issue":
I'm using web-gui in both external browser an in built-in one. In external (mostly chrome, sometimes firefox) I have set up zoom level (75%) because I have more shared folder then fits on screen and I don't want to scroll down. Font in web-gui is "ugly" big anyway so this suits me just fine. But I can't do it in built-in browser. I went through couple forum post on how to set zoom level in cefsharp but I didn't even find any config file in synctrayzor folder which I could modify. Can you add "zoom level" to settings? Or could you implement it to native UI you mentioned some time before (if it is still actual)?
Thanks

This shouldn't be too hard. The native UI is still a work-in-progress (fixing things is taking priority)


Test Issue created at Tue, 01 Jun 2021 10:00:46 GMT

Test issue body  Tue, 01 Jun 2021 10:00:46 GMT

Comment on issue at Tue, 01 Jun 2021 10:00:49 GMT


Seg fault when focusing away from floating window

I recently got a seg fault when using the the focus next command from a floating window.
2018-04-10 10:54:40 DEBUG: chunkwm: processing event of type 'ChunkWM_PluginCommand'
2018-04-10 10:54:40 DEBUG:     command: 'd', arg: 'id'
2018-04-10 10:54:40 DEBUG: chunkwm: processing event of type 'ChunkWM_PluginCommand'
2018-04-10 10:54:40 DEBUG:     command: 'd', arg: 'mode'
2018-04-10 10:54:41 DEBUG: chunkwm: processing event of type 'ChunkWM_PluginCommand'
2018-04-10 10:54:41 DEBUG:     command: 'f', arg: 'next'
[1]    42916 segmentation fault  bin/chunkwm

Thread 6 Crashed:
0   tiling.so                     	0x0000000107991fa0 FindWindowUndirected(macos_space*, virtual_space*, node*, macos_window**, char*, bool) + 304
1   tiling.so                     	0x0000000107986ac5 FocusWindow(char*) + 1109
2   tiling.so                     	0x000000010798ac04 CommandCallback(int, char const*, char const*) + 1028
3   tiling.so                     	0x0000000107996f0e PluginMain(char const*, void*) + 3950
4   chunkwm                       	0x0000000107796cf2 Callback_ChunkWM_PluginCommand(chunk_event*) + 178 (callback.cpp:169)
5   chunkwm                       	0x00000001077932a9 ProcessEventQueue(void*) + 473 (event.cpp:30)
6   libsystem_pthread.dylib       	0x00007fff52ea26c1 _pthread_body + 340
7   libsystem_pthread.dylib       	0x00007fff52ea256d _pthread_start + 377
8   libsystem_pthread.dylib       	0x00007fff52ea1c5d thread_start + 13

This is a follow up on #367

fixed on master.


Access to IPCAM-3000

Hi Stijn,
have you been able to access the IPCAM-3000 from KlikAanKlikUit?
Best regards,
Wessel

It seems like the camera stuff is a bit different. I can't really reverse that because I don't own a camera from kaku :(.

It seems like the camera stuff is a bit different. I can't really reverse that because I don't own a camera from kaku :(.


http: adding a default idle timeout of 1 hour

Adding a default 1 hour idle timeout - having none set is a recipe for connection leaks.
Note this only applies for idle connections (no streams exist on the connection and none have been created or destroyed for a full hour)
Risk Level: High (changes behavior for many Envoy users)
Testing: new unit tests
Docs Changes: inline
Release Notes: included
I think this must also include an envoy-announce email for those who want connections doing absolutely nothing for longer than 1 hour.
@envoyproxy/senior-maintainers
@jmarantz

qq: does this cover both client-proxy idle timeouts, and also proxy-upstream idle timeouts?

@jmarantz both - was in relnotes but added to PR description too


CKEditor4 vs CKEditor5

I see you use CKEditor4. Why do you not use CKEditor5.
Because of the license of CKEditor5 (GPL, not LGPL)?

To be honest, I haven't given this much thought. The main reason is that https://github.com/django-ckeditor/django-ckeditor hasn't added CKEditor 5 support yet.
https://github.com/hvlads/django-ckeditor-5 seems to be maintained a bit better than https://github.com/pancodia/django-ckeditor5, maybe this would be an alternative? Anyway, I only use a minimal CKEditor configuration most of the time and switching to a completely different rich text editor should be possible without too much effort.
(I'm definitely interested in upgrading CKEditor but it isn't a priority to me.)

CKEditor 5 is GPL, not LPGL.
This means, if you create a commercial site with it, you might get legal troubles.
Are you aware if this?
See: https://ckeditor.com/docs/ckeditor5/latest/builds/guides/support/license-and-legal.html

Thanks! I think I have been aware of this in the past but I have forgotten it (despite you mentioning it in the issue description already).
Yes, CKEditor 5 being GPL'd is a good reason for NOT using it in feincms3. It would probably be possible to create a GPL'd feincms3-ckeditor5 package and move https://github.com/matthiask/feincms3/blob/main/feincms3/static/feincms3/plugin_ckeditor.js there but it's still a legal risk according to my understanding of the GPL (https://greendrake.info/#nfy0)
Anyway, the point above regarding switching rich text editors still stands. Since CKEditor 4 is well maintained this isn't an issue now (I think) and will be relatively painless to solve in the future.

Thank you for the link to https://greendrake.info/#nfy0


pedantic package: analysis options not recognized

When I open the folder for a package I'm working on, I always get a warning on the pedantic package:

I have it declared in pubspec.yaml per the instructions on the pedantic package:

And this file shows up in the Dependencies list in VSCode:

If I delete and re-add it, the error message sometimes disappears, but it always comes back again on a subsequent folder open.
@DanTup also seems to be able to reproduce this. His comments:

It seems like a server bug to me - I don't know why it's generating that warning (I also witnessed it vanish when I was making changes in that file, but I reloaded the project and it came back.
Strangely, the file even shows up on the completion list in a .dart file confirming it's on disk (whether we should list non-Dart files here, I'm not sure!)..

I believe @scheglov fixed this and similar bugs recently by re-analyzing on a larger set of files (like the YAML files) etc.

Not sure why I hadn't previously replied here, but in case others come across this there was some additional discussion in https://github.com/Dart-Code/Dart-Code/issues/3259 about another instance of this (or perhaps the same) which seemed to be triggered by a nested project that didn't reference the pedantic package.
If the same thing happens now, the error message was improved to include the root of the package so it's clear which pubspec/.packages file was being used to resolve the package URI that failed.


Prevent memory leak

This PR resolves a memory leak.
While a browser instance is open, chances are an exception throws. If we close the browser instance in try block, then we'll miss the chance to close it gracefully and it'll stay open forever. finally is a good place to close instances whether the job was successful or not. For this purpose, we can keep a reference out of the try block and use it in the finally block. (I used ES6 let keyword which creates block-scoped variables)
This is a good article about the topic, It's in Java:
Java: How to use the finally keyword to avoid resource leaks

@amirhoseinsalimi thank you! I think you should close xvfb too.

Glad to help. Just did what you said. Let me know if anything's wrong.

Thank you so much!

Hi,
There was a mistake in your code. You have put page as reference variable but removed const pages. Correct way will be:
let browser, page;
...
const pages = await browser.pages()
page = pages[0]

Please check it from your side.

My bad! See #66 for thefix.


Not detecting existing clusters

For some strange reason the kubectl plugin shows no info neither about kubectl/helm/cloud etc...
While in debug tool, it detects the cluster but doesn't show anything in extension screen.

Hi @rverma-nsl , sorry for the late response. Have you been able to make it work or you still face this issue? Any info about the cluster you are connected to? It's weird that you don't see any message in the heml/clouds section. usually if it doesn't fetch any repo/providers a message is displayed.

I'm having the same problem.


In the implementation of shapenet, is there only poinext-s, no poinext-B, poinext-L implementation code???

In the implementation of ShapeNet , is there only poinext-s, no poinext-B, poinext-L implementation code???

Hi, there, because of the characteristics of shapenetpart, it is a considerably small dataset and has fewer sampling points in the dataset. In the experiment, we found it is beneficial to use pointnext-s as the backbone and with higher width, the result has got better

Thanks for your interest. We only trained PointNext-S on ShapeNet


Potential resolution of Issue #463 --replacements-only set, but no Replacement key in the ResourceChange back from CFN due to addition only

I dug around this a bit, and it seems like this fix would work.  I don't see how this code path would ever not be executed when the --replacmenets-only flag is set.  Perhaps I missed something and this seemed way too easy to resolve (and I don't know what side effects this change will make)
If you were curious, this is what we got back from CFN when I printed it out to the shell in my test:
[{u'ResourceChange': {u'Action': 'Add', u'ResourceType': 'AWS::EC2::SecurityGroupIngress', u'Scope': [], u'Details': [], u'LogicalResourceId': 'RuleNumber1'}, u'Type': 'Resource'}]

No "Replacements" key existed, so yeah... this nich issue would always happen in this specific scenario.
Is there another way you would prefer doing this?

Resolves issue 463

@troyready, @jkalscheuer-sturdy and I spoke about options for this today at Sturdy. I will be working on a better/safer way to implement this.
Also going to add a test so we don't regress on this.  I suspect that not many people use the --replacements_only option, which is why this went unnoticed for so long.

ok - no longer a try/catch... negative side effects of that was if you have any non "replacement" keys, it would just pass. That could be bad.
writing tests now.

ok, functional test written.  I Think this is ready to merge, unless you guys find something else wrong with it.

@phobologic - changed.  Of course that works as well and the tests passed.

Looks great, thanks @GarisonLotus!


Kitware apt failing in build container

The build container uses the kitware apt mirrors, which seem to be pretty error-prone and not so well maintained
Currently they are failing due to an OCSP error, and this is causing our CI to fail
probably we can workaround, but it would be good if we could find a more reliable source for cmake

not quite sure why but running the example locally passes
bashing into the build image and trying to apt-get update fails tho, and this is what is causing ci to fail

re kitware apt mirror - not sure why but we are explicitly not using the bazel toolchain provided by rules_foreign_cc
https://github.com/envoyproxy/envoy/blob/75c31d7af31c751d0e82530b9bd96f23d7b1b239/bazel/dependency_imports.bzl#L26-L27
not sure exactly how we are enforcing use of the version of cmake installed in the build-tools container
im wondering if we can just remove cmake from the container and use the rules_foreign_cc toolchains
cc @lizan

this has been resolve (mostly) by removing this from the container and shifting any builds that expect host cmake to hermetic toolchains


Update RunOnceApplicationCustomizer.ts

added comments to clarify what to do if action is web-scoped

Q
A

Bug fix?
no - yes?

New feature?
no - yes?

New sample?
no - yes?

Related issues?
fixes #X, partially #Y, mentioned in #Z

What's in this Pull Request?
Please describe the changes in this PR. Sample description or details around bugs which are being fixed.
Guidance
You can delete this section when you are submitting the pull request.

Please update this PR information accordingly. We'll use this as part of our release notes in monthly communications.
Please target your PR to 'dev' branch.

Thanks for your contribution! Sharing is caring.

added comments to clarify what to change if action is web-scoped

thx Russel, appreciated.


[FEATURE-REQUEST] Hide action buttons in show template

Describe the problem you're proposing to solve
Certain scenarios require hiding actions within show templates. For instance, when the category is populated via integration or ETL, users should be able to view the categories list and details. However, modifying or deleting should not be enable

Describe the solution you'd like
Depending on the default_actions configure in resource hide the actions
Describe alternatives you've considered
Additional context
I fork the repo and add the condition to the show.html.eex. Here is the diff
I tested it out and this is how it looks when
  def default_actions(_schema) do
    []
  end

@JessikaCastellano Thank you. This has been implemented and will be available in the upcoming release.


Use cutie for menus

found this really cool menu/user input package cutie and would like to use this for use for things like managing playlists and other things.

Added cutie to the pipfile and updated some of the menu
pull request here


service call to get dart / sdk version

Related to https://github.com/dart-lang/dart-pad/issues/403, we need a service call that will tell us what version of dart2js and the analyzer we're running (a single version for both).

Laned (not yet available in the production server).


Incompatibility with other editor commands

Currently gopen works with terminal based editors by changing to the directory and opening a blank buffer. It would be nice to have a configurable setting to either open the editor in the netrw buffer or pass in other editor commands, like 'rider' or 'goland' etc which require the path as part of the argument to open the project. Having the option to configure the cmd execution to take in the target path as an argument would, A); allow the use of other editors, and B); allow one to open projects in nvim in netrw.

Having the option to configure the cmd execution to take in the target path as an argument

Totally agree on that part. However, you got me thinking about something. You could be using rider for one project and nvim for another. This means that it might be more favorable to make the editor command configured per alias instead of a single global one... 🤔

Could have a default behaviour that could be toggleable, then also perhaps flag to overwrite it. I think having a editor for each alias could be nice, but trying to change the behaviour for each editor could be a lot of overhead

Good point. That might be a good lead for me to start something.


add Kiki program (SPIKE Prime)

Dear @dlech and @laurensvalk,
I am Ariana and I am 5 this month. My dad @TheVinhLuong102 and my brother @AntoniLuongPham teach me typing and coding.
I love SPIKE Prime lego and this is my first python program for your review.
Thank you!
Ariana

Hi Ariana,
This is a really nice program. :cloud: :sun_with_face: :national_park:  Thank you very much!

Dear @dlech,
Thank you very much for accepting my program!
I am building the grabbers and will submit the program.
Ariana


LogHook is not usable

In registry/handlers/app.go
    for _, configHook := range configuration.Log.Hooks {
            if !configHook.Disabled {
                    switch configHook.Type {
                    case "mail":
                            hook := &logHook{}
                            hook.LevelsParam = configHook.Levels
                            hook.Mail = &mailer{
                                    Addr:     configHook.MailOptions.SMTP.Addr,
                                    Username: configHook.MailOptions.SMTP.Username,
                                    Password: configHook.MailOptions.SMTP.Password,
                                    Insecure: configHook.MailOptions.SMTP.Insecure,
                                    From:     configHook.MailOptions.From,
                                    To:       configHook.MailOptions.To,
                            }
                            logger.Hooks.Add(hook)
                    default:
                    }
            }
    }

So there is no other way to catch log than mail, despite using logrus. That's very very annoying

@fbacchella :  Panic notification by email was added as it was required by a community member who contributed this change.  Supporting other capture methods is not too difficult and all contributions are accepted.


MWPW-146209 [MILO][MEP] MEP param override does not work if manifest sent by Target and pzn and chosen column not in Target

Steps to reproduce:

Create a manifest
Create a page that references that manifest in its personalization metadata
Create a Target activity that serves the same manifest to the page
Update the manifest to have a new column but don't update the Target offer
Try to use the MEP button to spoof the new column

Solution: don't check if the selected variant should be overridden until the manifests have been consolidated in cleanAndSortManifestList function
Example page setup that is sending an old version of manifest via Target and is trying to spoof a new column not sent by Target:

Before: https://main--cc--adobecom.hlx.page/drafts/vgoodrich/fragments/2024/q2/mepparamtarget/?at_preview_token=vq2HGezS8xc9gv4--gb5Ug&at_preview_index=1_1&at_preview_listed_activities_only=true&mep=%2Fdrafts%2Fvgoodrich%2Ffragments%2F2024%2Fq2%2Fmepparamtarget%2Fmanifest.json--photoshop-any
After: https://main--cc--adobecom.hlx.page/drafts/vgoodrich/fragments/2024/q2/mepparamtarget/?milolibs=mepparamtarget&at_preview_token=vq2HGezS8xc9gv4--gb5Ug&at_preview_index=1_1&at_preview_listed_activities_only=true&mep=%2Fdrafts%2Fvgoodrich%2Ffragments%2F2024%2Fq2%2Fmepparamtarget%2Fmanifest.json--photoshop-any

Resolves: MWPW-145303
Test URLs:

Before: https://main--milo--adobecom.hlx.page/?martech=off
After: https://mepparamtarget--milo--adobecom.hlx.page/?martech=off

Also included a tiny CSS fix for "instant" links

@vgoodric these changes are breaking preview functionality (and our promo nala tests, thats why it was not passing. We have added those tests recently so please do not skip run-nala label). If you go to the url below and try to set to default any of the manifests and preview, it will not go through. It reloads again with all the manifests set to all.
https://mepparamtarget--milo--adobecom.hlx.live/drafts/nala/features/promotions/promo-default

It works on: https://main--milo--adobecom.hlx.live/drafts/nala/features/promotions/promo-default
Could you please review?
cc: @nateekar

@afmicka I turned off nala because there was something broken on stage as well.  So it wasn't instructive.  But I think everything is working now.  Can you confirm?

@afmicka I turned off nala because there was something broken on stage as well. So it wasn't instructive. But I think everything is working now. Can you confirm?

Thanks @vgoodric. Confirming it is ok now.

@vgoodric no verified label. Please add


Improve user interface for better usability

Summarized Feedback from<EMAIL_ADDRESS>
Users are struggling with template integration due to lack of clear instructions or guidance, because they are new to the tool and find the existing resources (like YouTube tutorials) insufficient or hard to follow.


Replace classnames with clsx in example

related #1409
The example is also modified.
thanks!
Because it was deleted.

Coverage remained the same at 72.375% when pulling 856dabbc4ef2bcc0742abbe9398c76f97733bcf1 on wdh2100:example/replace-clsx into eb36e3aa428032664aa3244b2904f09afdf6b8f9 on gregnb:v3_2_0.

Good catch!


⚠ Bofrost.fr has degraded performance

In 59476c1, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 3547 ms

Resolved: Bofrost.fr performance has improved in 8919bdf after 6 minutes.


ROCm 2.10: rocm-smi --setsclk  does not stick to passed value

setting sclk does not stick to passed value, but frequency varies over time.

setting sclk does not stick to passed value, but frequency varies over time.

setting sclk 6 never sticks to 1801 MHz but to a lower frequency. Suggestions?

No luck with setting gpu by hand:
echo "vc 2 1801 950" > /sys/class/drm/card0/device/pp_od_clk_voltage
bash: /sys/class/drm/card0/device/pp_od_clk_voltage: Permission denied

Conversely --setmclk always sticks to the passed value.

TL;DR, it seems driver accounts for power dissipation and safe temperatures regardless of user setting.
I'm running different cards at manual with gpu and memory overrides. In a workroom with HVAC set to 18 degrees C. They do stick to override values. However when HVAC is off the gpu clock starts to fluctuate and if i want to work in comfortable 25 degree C, the gpu downclocks significantly, as low as level 3 or 4.
Memory doesn't seem affected as much, so it sticks to settings in all conditions.

TL;DR, it seems driver accounts for power dissipation and safe temperatures regardless of user setting.
I'm running different cards at manual with gpu and memory overrides. In a workroom with HVAC set to 18 degrees C. They do stick to override values. However when HVAC is off the gpu clock starts to fluctuate and if i want to work in comfortable 25 degree C, the gpu downclocks significantly, as low as level 3 or 4.
Memory doesn't seem affected as much, so it sticks to settings in all conditions.

Thank you for the explanation.

TL;DR, it seems driver accounts for power dissipation and safe temperatures regardless of user setting.
I'm running different cards at manual with gpu and memory overrides. In a workroom with HVAC set to 18 degrees C. They do stick to override values. However when HVAC is off the gpu clock starts to fluctuate and if i want to work in comfortable 25 degree C, the gpu downclocks significantly, as low as level 3 or 4.
Memory doesn't seem affected as much, so it sticks to settings in all conditions.

Thank you for the explanation.

however it is too early for me to turn on the HVAC, I have to continue without for some months.
Thank you again :-)


CAMEL-18450: camel-jpa - use transaction strategy in JpaMessageIdRepo…

…sitory

merging as we need to be ready for 3.18.2 release


Broken image reference in explanation Manage secrets not coming from a provider

Original report in https://github.com/projectsyn/component-hub/issues/68 by @arska:

https://hub.syn.tools/appcat/explanations/crossplane-secrets-non-provider.html references an image at https://hub.syn.tools/appcat/_images/crossplane-secrets.png which does not exist. The original https://github.com/vshn/component-appcat/blob/master/docs/modules/ROOT/pages/explanations/crossplane-secrets-non-provider.adoc references the image at https://github.com/vshn/component-appcat/raw/master/docs/modules/ROOT/pages/explanations/crossplane-secrets.png where it does exist.

Reproduce

Run the local preview with make docs-serve
Navigate to http://localhost:2020/appcat/explanations/crossplane-secrets-non-provider.html

Actual Behavior
Image reference in the rendered documentation page (https://hub.syn.tools/appcat/explanations/crossplane-secrets-non-provider.html) is broken.
Expected Behavior
Image is displayed in the rendered documentation page. Please note that Antora expects images to be present in docs/modules/ROOT/assets/images and not alongside the Asciidoc pages, see e.g. https://github.com/vshn/billing-collector-cloudservices/

https://github.com/vshn/component-appcat/pull/209

fixed with #209


Local config sanity

Currently changing our local configs for testing can be a little annoying because they are in git too. Let's rename them to *.sample, so we don't have to worry about that anymore.

@enusbaum

I think we should add LaunchSettings to GitIgnore.
If we're going to rename the appconfig.json, we might want to add some verbiage to the CLI default help screen about the file containing some default settings that they can rename.
A-OK with renaming modules.json to sample.

Added the launchsettings.json to .gitignore, if we can see a way to rename appsettings.json.sample for packaging, then we should be golden.

Just adding this so we don't lose track:
This PR needs to also enhance the build YAML to perform a rename on the appsettings.json.sample to appsettings.json so we have a file available to run by default for folks who download the release packages.

Closing -- this might cause more confusion.


Don't spawn villages (setting)

I think there should be a world setting to not spawn villages. Thanks :D

I totally agree. there is no command for that. There should be a setting not to spawn villages.
Or at least to chose how many you want. I have like 100 villages in my multiverse created worlds and I hate it.

Even though you posted that link, and this issue is old and closed, I still can't find the setting to turn off villages. Please update your manuals to include this.


AWS: not all routes work

When deploying this: https://github.com/asalkeld/test-app
The /store/ endpoint seems to work, but the /history endpoint is "not found"
Note it works on a local "run".
To reproduce, clone the repo and run
nitric deployment apply -t aws

fixed by https://github.com/nitrictech/cli/commit/32c49f56499cd2cd4d5ce083af193251ee6795c9


[W6.1d][F09-B4] Liu Hang

@liuhang0213 . Some comments added. Please look at them and close this PR once you are done :D


Python 3.5  32/64 bit

Hi,
i am getting a build error concerning the python version.
Is it supposed to be 32 or 64 bit?
Regards,
Lars

Hi, currently only 64bit is supported

Hi, thanks for the quick reply!
Maybe this also explains the large number of linking errors that I get.

Are you on windows or mac ?

On windows; changing to 64bit got rid of all the linking errors.
Though I got a warning when opening the editor:
"UE4Editor-UnrealEnginePython.dll" was either missing or compiled with a different version of the engine - not sure if it was the ".dll" as I can't reproduce the warning.
Anyway, the editor did some compiling/build on startup. and everything seemed to work.
The only issue now is that my python module can't be found.
Is there anything that needs to be done to add the "Content/Scripts"-folder as a "lookup" location - similar to the PYTHONPATH?!

I strongly suggest you to pull latest code and work with the python console too: https://github.com/20tab/UnrealEnginePython/blob/master/PythonConsole.md

Thanks for the suggestion! I will definately do that!
Scripting the editor is actually something I was looking forward to!
As I stated in my edit above, the module detection now works!
Thanks for your support!


fromURLs gives no output when 1+ wrong URL

When using the fromURLs method to generate shapes, where you give a list of URLs, rather than 1 by 1, if any of the URLs does not contain a valid ontology, an error occurs.
This prevents the creation of any SHACL file, not even from the other valid URLs in the list.
You could add a Try/Catch to still parse the valid URLs and generate a SHACL file from the valid URLs only, skipping the non-valid ones, but notifying the user. In this way, you at least have some output, rather than nothing.
I believe the API already does this somehow, (handled by code 206), but it would be nice to implement this in this code too.
I made these small changes and will create a pull request if you are interested in implementing this little fool-proofing enhancement.

Yes, this was a feature it should be there maybe we thought about it and miss the implementation in the code. Thanks for contributing we definitely are merging the PR.


Is there a restriction on permission names,  regarding cacheability?

is there a restriction on naming permissions that should be documented? https://github.com/maticzav/graphql-shield/blob/0a9c4726d7675db801f87bd5da6a5ff85ac9be69/src/index.ts#L133

@nickluger no, there’s no restrictions to permission name - as far as they don’t have the same name as field they are restricting.
I’ll add this to the docs, thanks for pointing it out! 🙂


Filter allowed values

Hi. Here I have added a button to filter the list of allowed values. This is because often not all terms are actually needed by the end user, so a filtered list is therefore nice to have.
The way I have approached this is by adding a terms variable to the EntitySelectionModel. This contains a filtered list of terms, or null when no filter is applied. As OntologyTermValidationDescriptor already contains a terms variable that is automatically populated by all terms on creation, I have now extended the constructor to allow a filtered list to be passed.
In terms of UI, I simpy added a button to the ValidationValuesPanel that opens a pop-up where the filtering can take place. At first I had added a checkbox to turn the filtering on or off (when off, terms=null in EntitySelectionModel). However there is no straightforward way to check if a range has been filtered. So the initial state of the checkbox is ambiguous.
One way to solve this is by adding this info somewhere in the exported file. Otherwise, on cell selection, I could load in all terms of the selected cell and do a comparison with the terms in the validation descriptor. If it is not equal, the checkbox should be on. Is that an idea?
This is my first time working on a Java project and contributing to another project, so feel free to tell me what can be improved and I will try my best to update it. :-)

Hi @mochar , sorry I hadn't seen this until recently, I don't think i am getting notifications from this repository which I need to look into.
I think this pull request may conflict and/or duplicate some other work which allows selection of individual items from the list of terms. This can be seen by building directly off the master branch. Would you be able to take a look and see if there is a way of consolidating ? If not I will try and take a look over the next week.


{packaging} Update argcomplete to 2.0.0

Description
Version 2.0.0 of argcomplete was recently released and many Linux
distributions are updating to that version now.
This patch replaces azure/azure-cli#20259
Testing Guide
No changes to existing tests.

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

packaging

Thanks @major, but we currently don't have time to test if argparse is compatible with the currently CLI because it is a major version change. I am afraid I need to put this PR on hold.

@major Oops, you probably shouldn't have closed #20259 that fast, sorry about that!

Thanks @major, but we currently don't have time to test if argparse is compatible with the currently CLI because it is a major version change. I am afraid I need to put this PR on hold.

Understood. This is working fine in Fedora 36 (rawhide) currently, but that's a sample size of one. 😉
I'm able to build the RPM package and run the same tests that I run for all of the other releases.

@major Oops, you probably shouldn't have closed #20259 that fast, sorry about that!

It's okay, @pdecat. 👍🏻


Move plugins to contrib

As discussed in SIG meetings:
All plugins moved to https://github.com/open-telemetry/opentelemetry-js-contrib except http/s, grpc, and xhr.

I still have 2 PRs on http and mongodb plugin, i'm guessing that i'll have to re-open them over the contrib repo ?

I still have 2 PRs on http and mongodb plugin, i'm guessing that i'll have to re-open them over the contrib repo ?

not for http

If that is not too much trouble, yes. If it is a pain, we can merge them here and then mirror the changes over.

@obecny Please review when you get a chance.

Should be able to do the same thing in the new repo

But I want to debug 2 repos at the same time

You will still be able to step into the built js, but that would likely be it. In what case would you want to step into the SDK from a plugin just out of curiosity? It seems like if the internals of the SDK are working, you should not need to step inside it. Similar to when stepping past calls to libraries that aren't ours, you step over them not into them.


Add python testing tool

I signed it!
пт, 1 марта 2019 г. в 22:45, googlebot<EMAIL_ADDRESS>
Thanks for your pull request. It looks like this may be your first
contribution to a Google open source project (if not, look below for help).
Before we can look at your pull request, you'll need to sign a Contributor
License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/
https://cla.developers.google.com/ to sign.
Once you've signed (or fixed any issues), please reply here (e.g. I
signed it!) and we'll verify it.
What to do if you already signed the CLA Individual signers

It's possible we don't have your GitHub username or you're using a
different email address on your commit. Check your existing CLA data
https://cla.developers.google.com/clas and verify that your email is
set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.

Corporate signers

Your company has a Point of Contact who decides which employees are
authorized to participate. Ask your POC to be added to the group of
authorized contributors. If you don't know who your Point of Contact is,
direct the Google project maintainer to go/cla#troubleshoot (Public
version https://opensource.google.com/docs/cla/#troubleshoot).
The email used to register you as an authorized contributor must be
the email used for the Git commit. Check your existing CLA data
https://cla.developers.google.com/clas and verify that your email is
set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.
The email used to register you as an authorized contributor must
also be attached to your GitHub account
https://github.com/settings/emails.

ℹ Googlers: Go here
https://goto.google.com/prinfo/https%3A%2F%2Fgithub.com%2Fgrpc-ecosystem%2Fawesome-grpc%2Fpull%2F43
for more info.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/grpc-ecosystem/awesome-grpc/pull/43#issuecomment-468749670,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AApWZFeFURXRCanu-x8lU99uVgp9ex_pks5vSWc3gaJpZM4bZgTd
.


Parameter of type Map with some symbols (dot ., slash / and etc) in key is not supported

Property key with square bracket is not support. Config server responses with HTTP error code 400, "error": "Bad Request".
But such properties is supported by Spring Boot.
Spring Boot documentation https://github.com/spring-projects/spring-boot/wiki/Spring-Boot-Configuration-Binding
For YAML files, the brackets need to be surrounded by quotes for the key to be parsed properly.
foo:
  nested:
    "[bar.baz]":
      bling: 2

Or another documentation:
https://docs.spring.io/spring-boot/docs/2.7.6/reference/pdf/spring-boot-reference.pdf s. 96
For example, consider binding the following properties to a Map<String,String>: Yaml
my:
  map:
  "[/key1]": "value1"
  "[/key2]": "value2"
  "/key3": "value3"

NOTE For YAML files, the brackets need to be surrounded by quotes for the keys to be parsed properly. The properties above will bind to a Map with /key1, /key2 and key3 as the keys in the map. The slash has been removed from key3 because it was not surrounded by square brackets.
Sample
Config file: example.yml
foo:
  nested:
    "[bar.baz]":
      bling: 2
my:
  map:
  "[/key1]": "value1"
  "[/key2]": "value2"
  "/key3": "value3"

curl localhost:8888/example-default.yml
400 error code is returned

As in my test with config file:
my:
  map:
  "[/key1]": "value1"
  "[/key2]": "value2"
  "/key3": "value3"

curl localhost:8888/example/default              works fine
curl localhost:8888/example-default.yml       500 errors (class java.util.LinkedHashMap cannot be cast to class java.util.List)


Support integer vertex id.

Close #190

I see you replace VertexID with Value. Is this different from we discussed before? I thought we use VertexID, use Integer Vid as string. And storage will return Value with correct vid type (int or string).

I see you replace VertexID with Value. Is this different from we discussed before? I thought we use VertexID, use Integer Vid as string. And storage will return Value with correct vid type (int or string).
PS: the perf between int id and string id is not trivial according to benchmark.

The storage client will convert all int vid to string vid, there are not much change.

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


Maintain: Appendix A and B

#570

This change is

@enitrat  do you think built-in functions is relevant? For now , we should quote assert and panic corelib func  but there are plenty of others.  Should i add more? leave it like that? or remove examples with just a short explanation that we should not use the name of a corelib func for another item


Feature: Add type check (validation) for username during signup

Feature Request
Is your feature request related to a problem you're trying to solve with Devhub? Please describe.
Add a type check during signup for username (should be alphanumeric, must be greater than 7 and less than 12 characters)
Describe the solution you'd like
When a user Signup then add a check whether the username is valid or not. (should be in the client side)
validation criteria:

must be alphanumeric
greater than 7 characters
less than 12 characters

If not valid then show the error below the input field what's missing or going wrong.

@deepraj21 Can you assign this issue to me? I would like to work on this

Ok assigning.


Hidingchestname

Xanth3r is going to test some more for me but im pretty sure this is well flushed out now, give it a look and let me know if you have questions.
Pull Request Etiquette
There are several guidelines you should follow in order for your
Pull Request to be merged.

[x] I have checked the PRs for upcoming features/bug fixes.
[x] I have read the contributing guidelines.

It is sometimes better to include more changes in a single commit.
If you find yourself having an overwhelming amount of commits, you
can rebase your branch.

Template
Pull Request
Replace this sentence with general description of what your Pull Request does.

this should fix #70

Everything should be tested and good now


animation watermark

sometime Add animation watermark can not look animation;

It is not clear what you are asking.


Send E-Mails with PDF attachments in a background batch (REPORT.SAVEASPDF)

The functions SaveAsPDF, SaveASExcel, SaveAsWord, SaveAsHtml are no longer available in extensions V2.
We need a functionallity to send mails with attachments (NAV Reports as PDF) via a background batch job.
How can we make it work?
Perhaps you can provide a function to send a mail via SMTP with an attached outstream?

We are currently looking into the area of sending email file attachements.

Can you tell me when this "future update" will be there?  printing-in-code is quite necessary for many extensions...

We are aiming at getting the update out before the end of the year.

@StanislawStempin I still have this issue on AL Language version 0.14.17461?

It this yet to be delivered?

We should have been more clear when closing this issue. The solution provided was the second part of he original suggestion: "Perhaps you can provide a function to send a mail via SMTP with an attached outstream?"
We did exactly that, have a look at the new function COD260.EmailFileFromStream

Will look into it. Thanks!


WSL - non-http connection problem (111: Connection refused)

WSL version:
Linux version 4.4.0-17134-Microsoft<EMAIL_ADDRESS>(gcc version 5.4.0 (GCC) ) #1130-Microsoft Thu Nov 07 15:21:00 PST 2019
Microsoft Windows [Version 10.0.17134.1130]
I cannot download from repositories with our firewall/proxy restrictions.
After this operation, 381 kB of additional disk space will be used.
Err:1 https://archive.ubuntu.com/ubuntu bionic/main amd64 make amd64 4.1-9.1ubuntu1
  Could not connect to archive.ubuntu.com:443 (<IP_ADDRESS>). - connect (111: Connection refused) Could not connect to archive.ubuntu.com:443 (<IP_ADDRESS>).

I suppose the cause is that the repository is only to be accessed via http.
https://archive.ubuntu.com/ubuntu/pool/main/m/make-dfsg/make_4.1-9.1ubuntu1_amd64.deb is not reachable.
What I can do:
ping archive.ubuntu.com
 curl https://google.com but not
curl http://google.com
And also I can only do that after setting our proxy server via export HTTP_PROXY=...
The question would be then, how to access ressources HTTP?

If http is blocked, there are several official mirrors that support https:
https://ftp.halifax.rwth-aachen.de/ubuntu/
https://ftp.lysator.liu.se/ubuntu/
https://mirrors.cs.wmich.edu/ubuntu/
Just search and replace the ubuntu.com links with one of the above. I tested the .de mirror above with https and it worked for me.

Thanks for your help, but seeemingly, the problem lies somewhere else:
 0% [Connecting to ftp.halifax.rwth-aachen.de (<IP_ADDRESS>)] Err:1 https://ftp.halifax.rwth-aachen.de/ubuntu bionic InRelease   Could not connect to ftp.halifax.rwth-aachen.de:443 (<IP_ADDRESS>). - connect (111: Connection refused)


mcu-interface: isotp: set block size to 8

when the MCU tries to send payloads that are
larger than 7 bytes, ISO-TP chunks the data.
The block size is set by the receiver and
indicates to the sender when a flow control
packed must be awaited.
8 CF (consecutive frames) are now sent
by the sender before one FC (flow control)
frame is sent back.

still relevant or should we close?

adding to backlog


private #variable in js class

Editor says that define a private field inside a javascript class is a syntax error, but it works fine, can you fix this?
because that annoys me to find erros in the code.

Seems linter issue


"Huh?" showing up when i shouldnt

so i saw that "Huh?" shows up in some cases when it wasnt before. not a big deal, but thought i would run it by you:
        --=( You are in the Intersection )=--

A boring intersection. There is a passageway to the
north that leads to the shop. To the east is the elevator
landing, to the west is the guest lounge, and to the
south is the reception desk. There is nothing to do here.

> go east
It's locked! You will need a pebble and a key.
Bonk! you can't seem to go that way.
Huh?

that extra huh? gets tacked on if you cant go into a location or you type in an invalid direction
> go northe
Don't know how to go 'northe'.
Huh?

again, not a big deal, but wondering if there is a condition that was getting caught before that now isnt and is letting the checks in run() slip to the very end, past all the conditionals with 'continue'

I think mostly these result from forgetting to return True from verb handlers.  Some of these should be fixed now that the Verb classes do this, but the also sometimes just pass through the return of random functions passed into their constructors.

ah, i see, that makes sense. i'm heading out of town this week, but might look at this if no one else gets to it


Add Automated Greeting Workflow

Have designed a workflow which will greet user on their email and in the issue raising chat for raising a issue or PR
Please approve my PR @jinx-vi-0 with labels assigned

Have designed a workflow which will greet user on their email and in the issue raising chat for raising a issue or PR Please approve my PR @jinx-vi-0 with labels assigned

Mention issue number in your comment. like this: Fixes #issue-number

Have designed a workflow which will greet user on their email and in the issue raising chat for raising a issue or PR Please approve my PR @jinx-vi-0 with labels assigned

Mention issue number in your comment. like this: Fixes #issue-number

i have added it in the PR description
@jindalpriyanshu101  @jinx-vi-0 kindly approve my PR


Incorect error message when building for MacOS, iOS is mentionned

Steps to reproduce
...
Error: The pod "Firebase/Analytics" required by the plugin "firebase_analytics" requires a higher minimum iOS deployment version than the plugin's reported minimum version.
To build, remove the plugin "firebase_analytics", or contact the plugin's developers for assistance.
Error running pod install

The error message is incorrect MacOS is not iOS.
Expected results
Error: The pod "Firebase/Analytics" required by the plugin "firebase_analytics" requires a higher minimum **MacOS** deployment version than the plugin's reported minimum version.
To build, remove the plugin "firebase_analytics", or contact the plugin's developers for assistance.
Error running pod install

Actual results

Error: The pod "Firebase/Analytics" required by the plugin "firebase_analytics" requires a higher minimum iOS deployment version than the plugin's reported minimum version.
To build, remove the plugin "firebase_analytics", or contact the plugin's developers for assistance.
Error running pod install

Code sample
Code sample
no code

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[Paste your output here]

Hi @stephane-archer,
Please provide your flutter doctor -v and flutter run --verbose logs, and a minimal complete reproducible code sample along with steps to reproduce so that we can investigate this issue.
Thank you

deleting Pods and pod.lock solves the issue on my side but your error message is wrong, search for "requires a higher minimum iOS deployment version than the plugin's" in your codebase to find the issue and make a platform-dependent string.

maybe look at  packages/flutter_tools/test/general.shard/macos/cocoapods_test.dart line 505 ?

This was a mistake in https://github.com/flutter/flutter/pull/138097, I adjusted two of the error messages to handle this correctly, but missed the third one.


Cross References previews should be 'view' sensitive

Version and Platform (required):

Binary Ninja Version: 3.2.3813 (stable and dev)
OS: macOS
OS Version: 12.6
CPU Architecture: M1

Bug Description:
When a Pseudo C view is active, the Cross References pane shows no previews for local variables, neither Pseudo C nor disassembly.
Steps To Reproduce:
Please provide all steps required to reproduce the behavior:

Open any binary
Switch to Pseudo C view (F5)
Select any local variable (stack, register, or parameter)
See no preview in the Cross References pane

Expected Behavior:
Pseudo C previews should be shown in the Cross References pane.

Implemented on dev as of last week, it makes a best effort attempt to find the corresponding IL instruction in the active view. Failure preserves the existing behavior of falling back to disassembly.


GitHub Board Video Walkthrough

YouTube Video of Stephen's session here - https://youtu.be/SWIu-zHoycg

List of things that should be covered in the walkthru https://docs.google.com/document/d/1syNtf-xH-CtyrWgnpZZv3cb37wk4MV4WoJpDUwEKOiM/edit?usp=sharing

Duke aim to complete by next meeting 15th May 2024

This is complete but needs to be shared on Archives Discord

Still waiting on Duke's response

Duke wasn't able to complete the task due to pressure of other work, so he passed it on to Ayo.
(As a note for future reference, this is possibly not the best approach; it might have been better to offer it out to everyone; nevertheless, thanks to Ayo for stepping up.
Here is the video https://drive.google.com/file/d/1sg3R9-mnP1blXhlQSNfywNtV7XmyhUCP/view?usp=drivesdk

At 22 minutes, is it a bit too long?
Also there are some problems with it, and it perhaps doesn't fit the brief very well. It's more of a generic "how to use a GitHub board" rather than an explanation of how WE use OUR board; and the voiceover is often not really explaining what is being demonstrated.

Add Ayo as a GitHub member - Vani or Stephen

Ayo has said he's happy for it to be reassigned; so, reassigned to Lord_Kizzy
Deadline of 2nd Oct
Aiming for a length of c 7 mins

Finished video is here https://youtu.be/abaUWUBOU28


Load data from disk into memory in batches

Instead of loading all training and test data, can we load the data in memory in batches, i.e. on the fly during training and evaluation?

Thank you for the suggestion. This is not easy to implement for training (where shuffling between epochs requires having access to all the data, or at least some way of addressing smaller groups of samples at a time), but could be an easy enhancement for evaluation.

One way is to have a single training example in a single file, shuffle the list of filenames before every epoch (as you already do), prefetch batches and preprocess them using a CPU while the model trains on GPUs, and maybe use tf.data in the entire process.

Yes, that's roughly how to do it. Two notes on that:

One example per file has surprising side effects at this scale (e.g., many filesystems become very slow when millions of files are in one directory, so you need subfolders; reading one file from the can lead to horrible disk access patterns; shuffling + length of an epoch make sure that caches don't work well). These things can be mitigated by using K samples per file, and then reading from several files at once (so each minibatch of N samples is drawn from M << N files).
The minibatching routine is a bit complicated to handle joint training on many languages, optional random switching between docstring and function name embeddings, etc. Pushing this into tf.data is possible, but painful.

So, overall: It's a substantial amount of work that we are most likely not going to do. The released baselines are really just meant as a "here's a simple straightforward approach, beat that" - we are happy for others to improve on this, either by entirely rewriting things or improving our codebase.

Got your point :)
This is an example, although on the data pipeline side, of general problems faced with system requirements when trying to beat SOTA language models these days.
Anyway, closing the issue now. Thank you for your consideration.


Logging Noise Seen from Resource Processor

When running an AppInsights query such as this...
traces
| where cloud_RoleName == "runner.py"
| where message !startswith "Retrying due to transient client"
| where message !startswith "No sessions for this process"
| where not (message startswith "Unable to find image" and strlen(message)<170)
| where severityLevel >1
| order by timestamp desc

... a number of messages that are simply [stderr] are reported...

Specifically, this query will return all such logs:
traces
| where cloud_RoleName == "runner.py"
| where message == "[stderr]"

This appears to be related to code in runner.py and logging.py within the resource processor.
We need to avoid this noise and, at the same time, make sure this noise doesn't mean we're dropping other more important logs.

Is shell_output_logger ever supposed to log to AppInsights?
The name suggests it shouldn't.

When addressing this just need to also account for local development and us using direct console view (like docker logs).

At present:
    logger.log(logging_level, prefix_item)
    logger.log(logging_level, console_output)

This means [stderr] prefix appears as its own entry.
Will look at changing to:
    logger.log(logging_level, f"{prefix_item} {console_output}")

Also default log level is INFO, some packages are outputting too much info at the INFO level, such as service bus, going to look at changing the default to `WARN' when not debugging. Ideally this would be configurable.


Add help messages to errors related to the Cedar schemas

Description of changes
Issue #, if available
#1125
Checklist for requesting a review
The change in this PR is (choose one, and delete the other options):

[x] A change "invisible" to users (e.g., documentation, changes to "internal" crates like cedar-policy-core, cedar-validator, etc.)

I confirm that this PR (choose one, and delete the other options):

[x] Does not update the CHANGELOG because my change does not significantly impact released code.

I confirm that cedar-spec (choose one, and delete the other options):

[x] Does not require updates because my change does not impact the Cedar formal model or DRT infrastructure.

Superseded by #1129