OS update progress

Coverage increased (+0.6%) to 32.112% when pulling 717ec518b1db56dffb3d872796d385eff58b7169 on gabrielburnworth:master into 22739263f47efef04d95b1807cacbea9d8e12c65 on FarmBot:master.


How to exit fully?

I close the application but it lingers on in the drop-down notification area. Only exit I find is by force closing it from Android settings. Is there a way or can it be?

If not I fear it consumes RAM/battery and takes up space in the drop down notification area of Android.

Newer versions don't fix this.

His can I get a response from dev? By adding device info or smth?

BUMP

What if the could be a "Close"-button in the notification? Like DNS66 has.
Do I have to learn to code and make a pull request to get a response?

Maybe you are to busy to answer...

Kind of.  This wasn't ever intended to be a full-fledged audio recording app, it was a proof of concept for Web Audio testing.  I'm not sure what kind of hosting problem you're running in to.


Updated sample-app link

LGTM

not-LGTM ;-) I think we should present cleanest examples possible and I'm not considering the ugly long command line as 'clean'.

reverting my not-LGTM to LGTM as we discussed this issue and created issue: https://github.com/openshift/source-to-image/issues/158

lgtm but merge build is failing.

Let's re-[test]

@soltysh I've already sent a PR in vagrant-openshift repo ro fix this issue, waiting for merge

[test]

LGTM, fixed the Jekins job.
Merging


GPU docker with authentication

Hi,
How to run TFserving GPU image with http basic auth enabled?
https://github.com/tobegit3hub/simple_tensorflow_serving/blob/master/Dockerfile.latest-gpu
uses simple_tensorflow_serving:
CMD ["simple_tensorflow_serving", "--port=8500", "--model_config_file=./examples/model_config_file.json"]
which don't have any auth related settings:
https://github.com/tobegit3hub/simple_tensorflow_serving/blob/master/simple_tensorflow_serving/command.py

The auth settings are in https://github.com/tobegit3hub/simple_tensorflow_serving/blob/master/simple_tensorflow_serving/server.py#L89 .
I'm not sure if it is available in WSGI and any contribution is welcome.


Codegen should treat properly [In | Out] and [Optional] void* pointers

An example is:
[DllImport(api_ms_win_service_management_l2_1_0, SetLastError = true, CharSet = CharSet.Unicode)]
[return: MarshalAs(UnmanagedType.Bool)]
public static unsafe extern bool ChangeServiceConfig2(
   SafeServiceHandle hService,
   ServiceInfoLevel dwInfoLevel,
   [Friendly(FriendlyFlags.Out | FriendlyFlags.Optional)] void* lpInfo);

The friendly code generated on these cases is:
public static unsafe bool ChangeServiceConfig2(SafeServiceHandle hService, ServiceInfoLevel dwInfoLevel, ref void? lpInfo)
{
   void lpInfoLocal = lpInfo.HasValue ? lpInfo.Value : default(void);
   bool result = ChangeServiceConfig2(hService, dwInfoLevel, lpInfo.HasValue ? &lpInfoLocal : null);

   if (lpInfo.HasValue)
      lpInfo = lpInfoLocal;

   return result;
}

which gives us errors because there is no void type to use without a pointer. I propose we either skip this friendly code generation or we generate this instead on these cases:
public static unsafe bool ChangeServiceConfig2<T>(SafeServiceHandle hService, ServiceInfoLevel dwInfoLevel, ref T? lpInfo)
   where T : struct
{
   IntPtr lpInfoLocalPtr = IntPtr.Zero;
   GCHandle? gch = null;
   bool isStruct = false;

   try
   {
      T lpInfoLocal = lpInfo.HasValue ? lpInfo.Value : default(T);
      var type = typeof(T);

      if (type.IsEnum || type.IsPrimitive)
      {
         gch = GCHandle.Alloc(lpInfoLocal, GCHandleType.Pinned);
         lpInfoLocalPtr = gch.Value.AddrOfPinnedObject();
      }
      else
      {
         var layoutKind = type.StructLayoutAttribute?.Value ?? LayoutKind.Auto;
         isStruct = layoutKind == LayoutKind.Explicit || layoutKind == LayoutKind.Sequential;

         if (isStruct)
         {
            var iSizeOfTestStructSimple = Marshal.SizeOf(typeof(T));
            lpInfoLocalPtr = Marshal.AllocHGlobal(iSizeOfTestStructSimple);
            Marshal.StructureToPtr(gch, lpInfoLocalPtr, false);
         }
         else
            throw new ArgumentException("struct should have either LayoutKind.Explicit or LayoutKind.Sequential");
      }

      bool result = ChangeServiceConfig2(hService, dwInfoLevel, lpInfo.HasValue ? (void*)lpInfoLocalPtr : null);

      if (lpInfo.HasValue)
         lpInfo = lpInfoLocal;

      return result;
   }
   finally
   {
      if (gch.HasValue)
         gch.Value.Free();

      if (isStruct)
         Marshal.FreeHGlobal(lpInfoLocalPtr);

      lpInfoLocalPtr = IntPtr.Zero;
   }
}

Thanks for the proposed fixes. I would be inclined to go with the first option for now (not generating anything). The second option would need a thorough review that I don't have time for at the moment and may not even be qualified to say for sure works without memory leaks, etc.
Now regardin the first option, does that even really need a fix? If the desired behavior is to no-op, why add (or allow) the Friendly attribute on that parameter in the first place? At least, we could remove the attribute from the method that fails to be regenerated and it's done. At most, we could cause the generator to emit an error telling the user the attribute isn't supported on void* parameters instead of generating an invalid method.

Well I think someone might put it not knowing the expected end-result, as I did.  ;-)
Anyway, this way we can map all the pointers matching their proper declaration and when (and if) we want to offer new overloads, we just change the CodeGenerator. I think it is best for us to have a straight policy to map stuff and we decide what and how to offer friendly overloads when we feel it's necessary.

Closing due to a workaround being available or lack of popular demand. This allows us to better focus on high impact issues. If you feel this is blocking you, you may add a comment here to explain why this should be prioritized.
Thank you for your understanding.


replace_in_file didn't find pattern '/MDd ' in './openssl-1.0.2l/ms/ntdll.mak' file

I am getting the following error when trying to build OpenSSL from this recipe:
ERROR: OpenSSL/1.0.2l@conan/stable: Error in build() method, line 103
        self.visual_build(config_options_string)
while calling 'visual_build', line 212
        tools.replace_in_file("./openssl-%s/ms/ntdll.mak" % self.version, "/MDd ", "/%s " % runtime)
        ConanException: replace_in_file didn't find pattern '/MDd ' in './openssl-1.0.2l/ms/ntdll.mak' file.

Environment:
Conan Version 0.27 (windows installer)
Visual Studio 2017 Enterprise Preview - Version 15.4.27004.2000
Perl/Nasm provided by conan repositories.
I fixed this locally by adding , strict=False :
        tools.replace_in_file("./openssl-%s/ms/ntdll.mak" % self.version, "/MD ", "/%s " % runtime, strict=False)
        tools.replace_in_file("./openssl-%s/ms/nt.mak" % self.version, "/MT ", "/%s " % runtime, strict=False)
        tools.replace_in_file("./openssl-%s/ms/ntdll.mak" % self.version, "/MDd ", "/%s " % runtime, strict=False)
        tools.replace_in_file("./openssl-%s/ms/nt.mak" % self.version, "/MTd ", "/%s " % runtime, strict=False)

Thanks for telling, the strict=False is the good approach. I'll patch.


Why the new Google analytics code was reverted?

The is no description in d2f951c as to why the gtag Google analytics code was reverted. Could you, please, clarify?

@sjparkinson Looks like this was part of https://github.com/pages-themes/cayman/pull/43. Can you explain your reasoning?

https://github.com/pages-themes/cayman/pull/44

It was separated out into a new PR.

I think we've hit a bad merge by doing #43 after #44. Reverting the revert should correct it.
I should have cleaned up #43 better after the suggestion to split the work, my mistake.

@secumod if you'd like to submit a pull request to add it back, I'd be glad to merge it.


Adds toggle button to switch between estimated and actual execution plans

This PR fixes #19621
This PR includes the following changes:

The "Explain" button is renamed to "Estimated Plan"

A new toggle like button was added that allows a user to switch into actual execution plan creation mode by clicking on "Enable Actual Plan".  Additionally, the CTRL/CMD + M shortcut will save the user from needing to click on the toggle button, so they accomplish the same thing without needing to click a button.

NOTE: Clicking on the "Enable Actual Plan" button won't generate an execution plan until the "Run" button or the F5 key is pressed to execute one or more queries.

Conversely, clicking on "Disable Actual Plan" will stop generating actual query plans when one or more queries needs to be executed.

Telemetry has been added around running queries when the toggle is enabled to display actual execution plans.

New Shortcuts/Commands:

CTRL/CMD + L: Will create and display estimated execution plans for all or only selected queries. Additionally, the command palette can also be used to accomplish this.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

4 of 29 (13.79%)  changed or added relevant lines in 3 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.02%) to 41.891%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/sql/workbench/contrib/query/browser/queryEditor.ts
0
4
0.0%

src/sql/workbench/common/editor/query/queryEditorInput.ts
1
6
16.67%

src/sql/workbench/contrib/query/browser/queryActions.ts
3
19
15.79%

Files with Coverage Reduction
New Missed Lines
%

extensions/notebook/src/jupyter/serverInstance.ts
4
75.13%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
27878

Relevant Lines:
62091

💛  - Coveralls

@lewis-sanchez, please make sure we are firing the telemetry events for this new action as well.

I didn't have a separate icon, and used the same one. I'll request a new icon for the next release.

I'd suggest copying the icon file and then giving it a more appropriate name - then when you get the updated icon you can just replace the file with the new one.

could you please update the screenshots? @lewis-sanchez

@alanrenmsft, updated the screenshots


yi-home 1080 outdoor future support?

You plan to support the outdoor version of yi-home? The-Crypt0 is doing nothing in his project and I regrets donating my money to him. I prefer donating money to people that deserve it.

It depends on the platform.
If it's based on hisilicon platform, no.

I also have two outdoor cameras of that type. I think these cameras are based on the processor, but how do I find out if it's real?

Open it and take a picture of cpu.
Or make a dump of the flash.

It depends on the platform.
This is the procedure for MStar: https://github.com/roleoroleo/yi-hack-MStar/wiki/Dump-your-backup-firmware
But, 1st step, take a picture.

Oh, thanks for the description, but this is a bit too much for me. Are there different types of processors in the YI Outdoor? I am using the yi-hack-v4 today, does that perhaps help?

I will try to take a picture of the chip in a near future

I forgot to mention that I'm using yi-hack-v4 like @Sprocket02. @roleoroleo this would help?

Oh, thanks for the description, but this is a bit too much for me. Are there different types of processors in the YI Outdoor? I am using the yi-hack-v4 today, does that perhaps help?

If you are using yi-hack-v4, your cam is hisilicon based.
This hack doesn't support this plaform.


Enable hpu_graph support for wav2vec2-asr + custom ctc_loss only for Gaudi2

What does this PR do?

Fixes # (issue)
Enable hpu_graph support for wav2vec2-asr (add custom ctc_loss usage only for Gaudi2). Motivation to add hpu graphs is two-fold, (1) to improve performance, (2) to reduce run to run performance variations.
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

The change caused regression on 1.14 docker run. Can you please check it?

The change caused regression on 1.14 docker run. Can you please check it?

I think this happens because the CTC loss kernel is introduced in Synapse 1.15 right?
If it cannot work withg 1.14, we can merge this PR into the synapse_1.15 branch.

@regisss updated all review comments. @yeonsily @libinta updated code to ensure backward compatibility with Synapse 1.14.0.


Fixing Issue #2239

This PR is to fix issue #2239
The parser needed some encoding changes (Python2 to Python3)
Current accepted labels for PRs:

bugfix

Closing because I did PR to master by mistake. Re creating another PR to dev branch


[Bug]: Email verification on signup

Contact Details
No response
What happened?
There is no verification when user signs up using email

Use email verification with firebase - info.
Also implement a check for all authorised page to redirect to login page with message as "Verify your email address first"

What browsers are you seeing the problem on?
No response
Relevant log output
No response
Are you interested in contributing?

[ ] I'd like to work on this issue

Code of Conduct

[X] I have used the search function to check if an issue already exists
[X] I agree to follow this project's Code of Conduct

@pranshugupta54 Can you assign me this issue?

@pranshugupta54  can i work on this issue ?

Hey, @Ajen07 sure! Did you try locally setting up?

@pranshugupta54  Yes I have set up the project locally

Hey @Ajen07, are you still working on it? We'll re-assign in 24hrs

Hey @Ajen07, are you still working on it? We'll re-assign in 24hrs

You can re-assign me , I will work on this issue .

@Ajen07, you are already assigned. You can work on this, I remember you had most of the things working properly - so it's going to be easy for you to do minor changes.

If it is possible can you  add the label of gssoc


Tic Tac Toe JavaScript created(#675)

Tic Tac Toe JavaScript #675
Close #675
Checklist:

[X] I have mentioned the issue number in my Pull Request.
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have created a helpful and easy to understand README.md

Okayy
On Sun, 9 Oct 2022, 12:57 am Pranjay Poddar, @.***>
wrote:

@.**** requested changes on this pull request.
Move your project files to js game folder
—
Reply to this email directly, view it on GitHub
https://github.com/pranjay-poddar/Dev-Geeks/pull/724#pullrequestreview-1135256112,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWAEG7NGEIB2JDPUPA4TBCTWCHDIJANCNFSM6AAAAAARAHB72M
.
You are receiving this because you authored the thread.Message ID:
@.***>


Add heading option for a billboard

When I have a billboard with an image that is facing north, I can tilt the camera so the image is flipped. This is not what I would've expected, and there doesn't seem to be a way to keep the billboard's direction locked. Please fix.
Expected behavior - image pointing north:

After a slight camera tilt, it then faces more southward:

At a more extreme tilt, it seems to be pointing into the earth:

I've been able to get it to happen in this workspace:
https://cesiumjs.org/Cesium/Apps/Sandcastle/?src=Hello World.html&label=Showcases&gist=6f6710474236e3ffb99d41673e7b3632

There is a related problem where the "aligned axis" does not work as expected, and billboards that are near the edges of the screen behave as if their rotations are calculated from the center of the screen.
For example, we have a grid of billboards whose alignedAxes are tangent to the Earth's surface, pointing north, with 0 rotation:

While it looks fine or "close enough" at some angles, at other angles the billboards are not rendered correctly.  The camera is pointed south in the following picture:

One would expect the billboards to the right (west) to point right, and the billboards to the left (east) to point left.

Also requested by @hieeyh in #6825

Bumping this one. This feature would be really nice, I'm currently working on an app that shows ships moving around in different directions, and as a workaround I have created polygons instead. but as pointed out these don't scale. So to come around that problem i use PointGraphics instead when zooming out. Billboards with heading seems perfect for the job. I've tried to modify BillboardCollectionVS.glsl to force billboards to lay flat on the ground, but this shader is a bit complex and I'm currently struggling to adjust it to how I want. Could someone else give it a try please?

Hi! Have same needs, but maybe it is better to have a new separated graphics for oriented SVGs using Cesium.HeadingPitchRoll?

now do you have any solution to solve it？？

I faced this exactly problem and I come to this sollution, see if that suites you too. It's based on updating the rotation of the billoboard acording to camera's actual heading.
const self = this;

style.billboard.rotation = new Cesium.CallbackProperty(function (time, result) {
    return self.map.scene.camera.heading;
}, false);

I faced the same issue recently and came up with the following approach to solve this problem in sandcastle.
Sandcastle

I faced the same issue recently and came up with the following approach to solve this problem in sandcastle. Sandcastle

when refPos get undefined，the direction is error, do you have any solution to solve it？？


Updated UG & Test for FindCommand

This is a follow up from #31.
Documentation

[x] User Guide
[x] Developer Guide
[x] Javadocs

Sections updated under UG

Find command

Sections updated under DG

NIL

Classes with Javadocs update

NIL

Testing
Sections updated

NameContainsKeywordsPredicateTest - Tests the functionality of the partial keyword matching
FindCommandTest - Unit tests the find command as integrated into the model

Codecov Report

Merging #71 (7411c57) into master (587df69) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #71   +/-   ##
=========================================
  Coverage     67.34%   67.34%
  Complexity      404      404
=========================================
  Files            76       76
  Lines          1326     1326
  Branches        153      153
=========================================
  Hits            893      893
  Misses          399      399
  Partials         34       34

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


node 4.x is  end of service

current appmetrics has  following in package.json
  "engines": {
    "node": ">=4"
  },

Now that node 4.x is end of service
Should this be changed to
  "engines": {
    "node": ">=6.x"
  },

Question will be:
Does appmetrics works with node 8.x (LTS)?
What about node 10.x?

Yes we do work with node 8x (LTS).  Node 10 support is currently not available (although it will work on some platforms). We will add full support for node 10 later in the year

Hello Sir @tobespc, i am so sorry, but i still got this error when tried to install it, here some error snippet:
Thu, 04 Oct 2018 23:38:20 GMT

You are installing the Node Application Metrics monitoring and profiling module.
Licensed under the Apache License, Version 2.0 (the "License")
you may not use this file except in compliance with the License.
You may obtain a copy of the License at
http://www.apache.org/licenses/LICENSE-2.0
Unless required by applicable law or agreed to in writing, software
distributed under the License is distributed on an "AS IS" BASIS,
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
See the License for the specific language governing permissions and
limitations under the License.

Unsupported version v8.9.4. Exiting.
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: node extract_all_binaries.js
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      /Users/tonifirnandes/.npm/_logs/2018-10-04T23_38_21_045Z-debug.log
and also i attached the debug log files here.
Thank you very much.
appmetrics-elk-error-debug.log

@smartmouse I think the package.json has now been updated https://github.com/RuntimeTools/appmetrics/blame/master/package.json#L5


Not working with imagehandler.ashx

I tried adding an imagehandler.ashx link to an image as the large image but easyzoom can't seem to load this image.
http://www.swedoor.dk/Handlers/ImageHandler.ashx?type=product&typeId=290

I was expecting it to work since the other zoom plugins like this uses 'data-' atrribute for the large image and they are working with imageHandler.ashx links.

Could you please setup a demo displaying the error?
The following code (the same as used by the plugin) works as expected for me:
var img = new Image();
img.onload = function(e) { console.log(e) };
img.onerror = function(e) { console.error(e) };
img.src = 'http://www.swedoor.dk/Handlers/ImageHandler.ashx?type=product&typeId=290'

Yes, it does work. My bad, I might have mistaken the error with something else as I must have probably replicated this issue https://github.com/i-like-robots/EasyZoom/issues/88.


fix(client): merge and update the learn map

Closes #36820
Todo:

[x] Update tests for welcome component
[x] Add few tests for some of the learn page components component We already have tests for the map component

Progress so far:

Signed in view:

Non-Signed in view

@raisedadead, I don't think we should be nesting Grid elements.  learn.js wraps everything in one, so we don't need them in Map or Welcome.  It ends up creating too much padding on small screens.

You are correct, I think I missed it. I never checked the mobile views. I'll update the PR.

@raisedadead I will take a look at this later today and give my review.

Thanks @RandellDawson.
Please note I am not doing the restructuring of sections that we talked about yet on this PR. This lays the groundwork for it though.
@ahmadabdolsaheb is going to be implementing some changes in his landing page updates that depends on this PR.
The section restructuring will come after his changes have been merged.

@raisedadead In that case, I will wait for the next PR after @ahmadabdolsaheb's changes have been merged.


Metadata/Add Arch x86: SuperTux.SuperTux version 0.6.2

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Any idea why is it failing?

Hey @vedantmgoyal2009,
When performing a manual validation on an x86 host machine, I am able to install successfully, but I have issues running the executable. When I launch the executable, a blank window appears, then closes out immediately. Please verify your x86 installer and make sure that it can launch the installed app successfully on an x86 machine.

Hey, can you please check again. On my machine nothing appears.


Support global default style for connector labels - font family, font size, background color

"I'm trying to customize our draw.io via Confluence Server, everything seems to be fine except for one thing - I can't figure out how to define the default style for connector labels - font family, font size, background color. Is it possible?"
Asked here: https://app.intercom.com/a/apps/fz6gxyi6/inbox/inbox/994666/conversations/55912901555561
P.S. I suggested experimenting with "defaultEdgeStyle" and font family/font color. I tested them but this doesn't always work due to multiple factors included (e.g. if the newly label is added or old one edited etc.) so the issue is created.

I manage to resolve the issue by placing the parameters at the beginning of the defaultEdgeStyle object:
"defaultEdgeStyle": {
    "fontFamily": "Verdana",
    "fontSize": "14",
    "fontColor": "#d2d2d2"
    ...
    // other parameters
},

When placed in the middle or at the end of the object, these parameters were ignored and draw.io was applying standard styling to labels.
I believe this information must be added to the documentation because in my case the need for strict order and precedence of parameters is not obvious and it's a long "try and fail" path to figure this out.

Thanks. Is it possible to record a video showing how the labels are changed exactly when parameters are put at the beginning?
I would like to be sure I am performing the same steps as you are. I see you are using drawio 9.5.3 and Confluence Server 7.8.3
I tried your solution and it failed at my place so I assume this could be related to drawio version.
What OS and browser version are you using?
Regards,

This also seems to work when placed in the middle or at the end of the object. What config did you try where it failed?


array ops

[x] zip
[x] unzip (same as split)
[x] join append
[x] split slice (the input will be split_off, output will be the second half)
[x] push
[x] pop
[x] resize
[ ] reverse

you'll want to take care of capacity at some point (shrinking after split for example)

i'll leave reverse.. i have other plans here..

test the push/pop method before adding reverse

yeah, push/pop with process works great now, no need for reverse :3


Deploy proposals only when option want_ is equal to 1

We would like to be sure that proposal will be applied only when
assigned option want_ is equal to 1.
For current condtions it is enough that for example
export want_barbican=0
and condition
[[ $want_barbican ]] || return

will not work

the alternative would be to change the defaults for the other to '' and 1, respecticvely. I have no strong opinion which one is better.

Problem is that in current condition check, [[ $want_murano ]] this will be true even when want_murano will be 0, 1, 2, 3, 4 ... I think would be better to have explicit $want_murano and others equal to 1


Improve file imports/reads

@MGlauer,
in our last talk you told me that you would do file imports/reads differently and that I could have a look at omi/dialects/base/parser.py. For me your approach  in __unpack_file() doesn't look that different from what I do. Except mine is not a function definition, and I look up the path with os.path.join(os.path.dirname(__file__) , which IMO is not a problem, because it should give on every operating system the correct path string for the JSON file which is in the same folder as the concerning python module.
@MGlauer, can you give me more hints, what you meant!

Oh, sorry. It seems I was not clear enough. The problem is __file__because its value depends on how you call this library and its behaviour after packaging is rather unpredictable as we had to learn the hard way
Use pkg_resources instead


Fix react tests

https://github.com/facebook/create-react-app/issues/8689 is causing our
tests to fail in the CI pipeline. As the comments suggest, downgrading
to react-scripts 3.4.0 fixes the problem.
In addition, fix a test warning due to a missing id field.
cc: @juliusv

Could you add a comment line pointing to this PR in package.json so we know why it's like that, and that we can try updating it again later but need to be careful?

How should I add a comment to package.json? A brief internet search shows that I could add a comments section but that is pretty far removed from the dependency.

@csmarchbanks I forgot for a moment that plain old JSON of course doesn't allow for comments. Ok, then we'll just have to live with it like that, yeah. Thanks!


Update issue templates

We had to create issue templates via github UI for some reason, so here they are! A bit improved from our existing templates which aren't being found/used by the issue template chooser currently.

This got very messed up creating them with the UI. Please ignore.


Tavern Link not Appearing in Collab?

Is the collab for TavernAI broken? The link doesn't appear in the collab version.
Never tried TavernAI before because I have an outdated OS that doesn't allow node.js, so I thought I'd try the collab, though it doesn't appear to be working.

It is because localtunnel that generates links is down and unavailable at the moment.


docs: Fix JSDoc and TypeScript type definitions

This package doesn’t modify the exts array, so the ReadonlyArray type is preferred.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@5223d20). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #60   +/-   ##
=========================================
  Coverage          ?   73.33%
=========================================
  Files             ?        1
  Lines             ?       45
  Branches          ?       16
=========================================
  Hits              ?       33
  Misses            ?        5
  Partials          ?        7

Impacted Files
Coverage Δ

src/index.js
73.33% <ø> (ø)

Continue to review full report at Codecov.

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

Thanks


Update Alpine to 3.7

This updates gitolite from v3.5.0 to v3.6.0:
$ ssh git@localhost info
hello user, this is git@5b52de295201 running gitolite3 v3.6.0-4851-g7817a6c9e1 on git 2.15.0

Gitolite 3.6+ has a nice feature: repo-specific hooks, which I'd like to use.

Thanks for your PR, @albers!


Homebrew package

Can this be made available as a homebrew package?

Hi @kishaningithub, definitely. Will look into this.

Need AUR package as well for Arch Linux

I have published AUR package
https://aur.archlinux.org/packages/csvlens
Pls check it out @YS-L

I don't know much about AUR packages but one question I have is why so many dependencies(Screenshot below)? Given rust can produce a statically linked binary can AUR not pull the platform specific binary from GitHub releases and publish it, that way dependencies would be zero.

AUR packages are not maintained by Archlinux [1], and is typically provided as source packages to be built locally, with the expectation that users should do the due diligence themselves to validate the safety of the packages they install. If a package is submitted into Archlinux (or Debian) as a binary package, it should be possible to ship the release binary itself, separating the build dependencies from runtime dependencies.
https://wiki.archlinux.org/title/Arch_User_Repository

csvlens is now available on homebrew: https://formulae.brew.sh/formula/csvlens

Thanks @madhur for the AUR package!


🐛Play multiple sounds at once

Fixed #46

Description of change

Multiple sounds can now be played at once. Playing one sound does not cut off the last one. The concurrent sound maximum is currently set to 8 but is easily configurable.

Codecov Report
Attention: 24 lines in your changes are missing coverage. Please review.

Comparison is base (c1ecc5a) 21.77% compared to head (a7dbf97) 21.23%.

Files
Patch %
Lines

crates/daktilo_lib/src/app.rs
0.00%
24 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #128      +/-   ##
==========================================
- Coverage   21.77%   21.23%   -0.54%
==========================================
  Files          11       11
  Lines         473      485      +12
==========================================
  Hits          103      103
- Misses        370      382      +12

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


http poller handle error

When http request fails Then a event should be raised with a tag _httppollfailure
This way this failure could be handle through a specific path in the pipeline to alert someone

I don't like the way it works. status codes should be tested/checked on filter if necessary to handle any use case


Allow users to download analysis results as geojson

After running an analysis, add the ability to download results as a geojson (in addition to the current CSV option).
This also relates to https://github.com/WFP-VAM/prism-frontend/issues/124 which is for all types of layers, not just analysis results

@wadhwamatic was this resolved in the new UX?


src/ba-transport.c: use correct mtu for SCO links

For SCO audio links, we must send data in chunks of the link mtu size to ensure correct timing. To use a constant value of 48 or 24 bytes is just wrong.
Obtain the link config after successful connection, because before then the mtu has not been negotiated with the remote device

WOW, so the correct solution for the MTU issue was in the plain sight
all the time. @borine many thx for this PR! I will test it and merge
as soon as possible :)

@borine please look at this:
https://github.com/pulseaudio/pulseaudio/commit/82e7fe413e86e9e3098d9676649e1c66e5996899
What do you think?
Unfortunately, I've got limited possibilities for testing; however,
I'll try to use all my Bluetooth devices which support HFP and see if
something will pop up. Also, I don't know whether such issue is BT
dongle specific or remote BT device specific or Kernel specific...

Yes I've seen that link; not sure what to make of it. I have no idea where the fixed MTU of 48 bytes for SCO came from either.
There are a number of differences in the pulseaudio code from bluealsa, and it may be some other issue that is breaking HSP for them - for example, pulse never sets the BT_VOICE property on the socket.
The only way to know if this commit improves HSP/HFP performance is to test with a selection of headsets and bt chips, some which previously worked well, and some that previously failed.
My suspicion is that the critical factor is the host controller - specifically that the driver or the firmware defaults vary between models, and neither bluealsa nor pulseaudio explicitly sets every possible property/option.
For example, I note that bluealsa does not explicity set BT_VOICE unless the connection is "transparent". Can we really be certain that the default is always BT_VOICE_CVSD_16BIT, or should we force that value for CVSD connections? I find the lack of documentation and source comments in the Bluez project to be extremely frustrating - even worse than the alsa project, which I had not thought possible!

I've tested it with my laptop (samsung np530) and unfortunately it does not work. The hci_devinfo() reports MTU to be 64 bytes for both read and write. With such settings, it seems that the BT chip hangs and kernel needs to reload driver (see dmesg output below). :(
[192261.948819] usb 2-1.5: USB disconnect, device number 6
[192261.950329] Bluetooth: hci0 sending frame failed (-19)
[192261.965325] Bluetooth: hci0 setting interface failed (19)
[192262.211936] usb 2-1.5: new full-speed USB device number 7 using ehci-pci
[192262.366581] usb 2-1.5: New USB device found, idVendor=8087, idProduct=07da
[192262.366583] usb 2-1.5: New USB device strings: Mfr=0, Product=0, SerialNumber=0

PS.
This happens when bluez-alsa is configured without mSBC support (you need updated master branch because there was a bug which has prevented proper HFP connection). When bluez-alsa is configured with mSBC and BT headset supports mSBC codec, the transparent voice is used which does not result in crash. However, there is no sound.... mSBC supports is not working correctly (at least for me).
PS2.
Build in USB BT dongle:
Bus 002 Device 007: ID 8087:07da Intel Corp.
Device Descriptor:
  bLength                18
  bDescriptorType         1
  bcdUSB               2.00
  bDeviceClass          224 Wireless
  bDeviceSubClass         1 Radio Frequency
  bDeviceProtocol         1 Bluetooth
  bMaxPacketSize0        64
  idVendor           0x8087 Intel Corp.
  idProduct          0x07da
  bcdDevice           78.69
  iManufacturer           0
  iProduct                0
  iSerial                 0
  bNumConfigurations      1

I need to do a lot more reading, the controller lockup is very surprising to me so there must be something fundamental I have not understood. Please leave this alone for now, I will get back if I find anything, but that is unlikely to be for a few days (or even weeks) now.

I've made some changes in the code in the area of HCI info. Right now it will be fetched during adapter creation (part of some "coming soon" change). I've pushed it right now, so it might bring some light in the area of MTU.
I've checked the values obtained by the hci_devinfo() during the bluealsa startup and after the SCO connection acquisition. And guess what... for me, these values are the same....
You can use this simple dump function:
debug("\n Dev: %s (%s), flags: %#x, type: %#x"
     "\n Features: %x %x %x %x %x %x %x %x"
     "\n PKT type: %#x, link policy: %#x, link mode: %#x"
     "\n ACL MTU: %d (pkts: %d), SCO MTU: %d (pkts: %d)",
     a->hci.name, batostr_(&a->hci.bdaddr), a->hci.flags, a->hci.type,
     a->hci.features[0],a->hci.features[1],a->hci.features[2],a->hci.features[3],
     a->hci.features[4],a->hci.features[5],a->hci.features[6],a->hci.features[7],
     a->hci.pkt_type, a->hci.link_policy, a->hci.link_mode,
     a->hci.acl_mtu, a->hci.acl_pkts, a->hci.sco_mtu, a->hci.sco_pkts);

PS.
There was one minor conflict between current master and this PR. I've resolved the conflict and created merge commit (new feature of GitHub?) and it seems thet this commit was committed in your branch... from which this PR was created o_O What a great way of trolling :D

I realized that my first commit got the mtu from the wrong socket - HCI instead of SCO; not sure if that alters the value returned, but needs fixing anyway. Unfortunately my local repo is out of sync with github after your merge so I can't upload any more commits atm. I'll look into sorting that out soon.
Meantime, I'm trying to make sense of the MTU values I'm seeing in the context of the packet format specifications and not having much joy at present. Will continue with this in the background over the next couple of weeks.

Unfortunately my local repo is out of sync with github after your merge so I can't upload any more commits atm.

Try something like this :) :
git remote add arkq https://github.com/Arkq/bluez-alsa.git
git pull --rebase arkq master
git push --force

After a lot of research, I've realised that the original premise of this PR was based on a misunderstanding of how the SCO MTU is agreed between BT adapters. So, I'm closing it here.
Having better understood the issues around HSP/HFP with Bluez, I have some ideas on what bluealsa can do to improve its support. I will open an issue ticket soon as a place to discuss them, and if we can agree on an approach there I will then raise a new PR.


How to use with beta builds?

for instance this
I see https://github.com/abarichello/godot-ci/blob/651e66176ab66ba920d9487614fd472f470612a8/Dockerfile#L22-L24 however not quite sure how to set them. Do I image: docker://barichello/godot-ci:3.5/rc6?

There are no container images provided for pre-releases.

@bend-n your GODOT_VERSION is still set to 3.4.4 in the workflow, change that to  GODOT_VERSION: 3.5.rc6

@bend-n this is the setup step in my builds
 - name: Setup
      shell: bash
      run: |
        mkdir -v -p ~/.local/share/godot/templates
        mv /root/.local/share/godot/templates/${GODOT_VERSION}.mono ~/.local/share/godot/templates/${GODOT_VERSION}.mono

I use the mono version, so it should be just removing the .mono at the end of the lines

The import errors are normal, i ignore the .import

I got it 🥳


Add ListModel::find_sorted and ListStore::find_or_insert_sorted

This PR adds two convenience methods: find_sorted to ListModel and find_or_insert_sorted to ListStore.
Both methods provide search functionality in O(log n). The naming schema is borrowed from the native ListStore methods.

It looks good to me, however I'm definitely not an expert on this matter so I'll let @sdroege and @bilelmoussaoui "judge" on this matter. :)

I'm not sure if it makes sense to add new API in the bindings directly. Are those actually that useful in general?

@bilelmoussaoui I obviously find these functions useful (there is log n search now where none was before), but they can be implemented on top of the existing API. I’m not sure what the aims of the C API are, maybe they try to be minimalistic? I would like to see them in the rust bindings to have it as useful and convenient as it gets, and if these functions were to later find their way into gio, one can swap them out.

I think it should be the other way around: having them in the C libraries before we bind them in the Rust bindings. We're not opposed to adding API on gtk-rs directly if they make sense though. It just seems like a very specific use case...

@GuillaumeGomez I think it’s genuinely useful, so I would try to get it into the C API. Adding it here first would just be a shorter path to have it actually usable in Rust. I don’t see getting it in here first doing any harm or does that clash with some policies you have regarding API stability?
ListModel::find_sorted is the more general and maybe more useful function of the two. It gives a fast search and fits with the existing API with special-casing a sorted model. It is roughly equivalent to Vec::binary_search_by.
ListStore::find_or_insert_sorted is a bit more specific in its usecase: Making sure, there is exactly one item of the given kind in the model; prioritizing an existing one. It is a relatively thin wrapper around ListModel::find_sorted, but it takes some lines to write out the logic. I lean towards saying it is useful, while being biased by having a problem at hand that it applies to ;)

I don’t see getting it in here first doing any harm or does that clash with some policies you have regarding API stability?

One word: maintenance.

I would also prefer to have this in either the C library or in a utility crate on top of the bindings. The bindings should really only bind to the C API and in addition provide integration API into the Rust ecosystem (and as part of that also potentially small API additions that make the API more Rust'y).

I'm not fan of adding a method to gio::ListModel this way, since the implementation should happen in the object implementing the interface and not be part of the ListModel. Also you can't really assume that the ListModel is sorted.
ListStore::find_or_insert_sorted on the other hand looks like something that can be easily implemented, and doesn't seam to be really useful in general except for specific cases.
Anyways i think this API change really needs to happen in gio and not in the bindings.

I guess let's close this here then. Thanks for your changes nonetheless! Maybe they can end up in some kind of gio utility crate, or maybe you can submit them to GLib in C so they can end up in all bindings.


Llama 3.1 405B fp16 Support in Distributed Mode

Feature request / 功能建议
I would like to deploy Llama 3.1 405B fp16 (>640GB but <640GB x2) in distributed mode using two servers with 8x80GB GPUs each. However, even with distributed mode, it seems that I still have to place the model file on a single worker, which overwhelms the GPU memory of one server instead of evenly distributing it across the memory of both workers. Is there a way to evenly or controllably distribute a single replica across multiple workers when deploying such a large model for inference? If this is already an existing feature, please let me know how to do it. Thank you.

Motivation / 动机
To support Llama 3.1 405B fp16 inference in distributed mode.
Your contribution / 您的贡献
N/A

We are working on this feature.

Hi. Any update on this feature?


[MB-9796] Small query change to allow renaming the duty_stations table without downtime

Description
This is a small tweak to the query that populates the duty location search results.
It shouldn't have any visible effect yet, but it sets up for a future migration.
We will be adding a migration that renames the duty_stations table. It will look something like this:
ALTER TABLE duty_stations RENAME TO duty_locations;
CREATE VIEW duty_stations AS SELECT * FROM duty_locations;

Creating the view allows us to do the migration before any queries have been updated, in theory.
In practice, I ran into one small issue that is fixed by this PR.

The problem is that Postgres is not quite as good at figuring things out for views as it is for tables, so when this query looks at the duty_stations view, we'll get an error because Postgres can't tell that duty_stations.provides_services_counseling will not change within any of the groups.
Reviewer Notes
Will this change the results of the query in a way that I'm not anticipating?
Setup

Create a new customer,
go through the onboarding UI up until the point where you select a duty station,
play with the search to make sure the results make sense.

I recommend comparing against another environment to make sure things match.
Make sure you test cases with abbreviations (for example, "naval air station" should give you results that start with "NAS").

After trying out a few searches, finish customer onboarding just to make sure that nothing breaks.

References

Jira story for this change
Slack thread with a little context on the migration this is setting up for

Messages

:book:
:link: MB-9796

  Generated by :no_entry_sign: dangerJS against 0761fb3b1a61db168758929b8ab36c7839837c4f


Replace/Merge hover_cols functionality with the new HoloViews 1.19 hover_tooltips approach

The new hover_tooltips approach allows for more customization, I suggest we replace (or make an alias of) the hover_cols arg in favor of hover_tooltips which adopts and mirrors the functionality in HoloViews.

duplicate


View and Download Resume

Research an npm package that will show a view of the Resume on Resume page along with a download capabilities/button

i would like to work on this issue


Define Scope of TRTF

This is related to determining the scope of the Trust Registry Task Force, with relationship to #48 .  My proposal is the following:

We define a trust model (see below)
We define the scope of the work here w.r.t a trust decision

At conclusion to this, my suggestion is that the TRTF normative scope is limited to support of trust services. The non-normative scope may extend outside this restricted scope. support of trust services needs to be further clarified, as various specifications that can allow for interop and extension of services that provide a trust context, to enable trust decisions. I would propose that the models provided are required to allow for extensibility later on, and assume that there will be additional contexts required for trust decisions that we will not necessarily know.
Note: We are not encoding the how  make a trust decision, i.e what business logic you use to make the trust decision. We are just providing the mechanics and framework to inject contexts to make a trust decision.

STRAWMAN
Trust Decision Model

$h(g(f(C, x)))$

where:

$f(C, x)$ is a trust embedding ( Trust decision framework mapping )
$g(f(C, x))$ is a trust decision
$h(g(f(C, x))) = z$ represents an action (or effect) based upon a trust decision. It chooses $z_i \in \mathbb{Z}$ where $\mathbb{Z}$ represents a set of possible effects from a trust decision.
$C$ is the decision context
$x$ is the set of claims

Scope: This task force is focused on supporting system around $c_{\text{trust services}}$ and the data models in $x$, which represent claims. While recommendations may be provided on the other functions, anything outside of $c_{\text{trust services}}$ and $x$ is considered non-normative and out of scope. Some ways this translates within the deliverable:

Specification Deliverable

Service Discovery
Containers
Data Models
APIs

Companion Guide:

Recommendations and best practices

${c_i, c_j} \in C$ where C is the decision context and $c_i, c_j$, are different data contexts.
$x$ represents claim data

related to https://github.com/decentralized-identity/trust-establishment/issues/46

[ ] @a-fox and @neil: to take different trust models today to help inform the discussion. Probably part of the companion.


Frame capturing is not working

Hello Friends,
I have installed FFMpeg on /var/www location on my Centos 6 machine. I have also installed PHP-FFMpeg on the same location.
Now I created a capture.php file in which I placed the code of capturing the frame from the video as described in the README.md.
I placed this capture.php and video.mp4 file in same /var/www/html folder and run the file but it's not generating any frame.
I have changed the binaries path in the file. I have also tried with placing this file on different location like put it in the folder of PHP-FFMpeg but it's not generating the frame.
It's also not displaying any error even my php error reporting is on.
Please help what should I do OR what I'm missing here?
Thanks in advance.

@virendravyas When reporting errors, always post your code and possible error messages please. The description of you is not reproducible.


how to get all information about movie by name?

var results = await _client.SearchMovieAsync(title, lang);
            var data = results.Results.FirstOrDefault(p => p.Title == title);

            return new MovieDataDetail
            {
                Id = data.Id,
                Title = data.Title,
                OriginalTitle = data.OriginalTitle,
                Overview = data.Overview,
                ReleaseDate = data.ReleaseDate,
                Popularity = data.Popularity,
            };

It works but I need all information. For example: imdbId, credits, genres, cast

Refer to the examples: https://github.com/LordMike/TMDbLib#examples
The point on more data is important :)


Add bootstrap broker string fields to Cluster status

this commit adds several new fields to the Cluster Status
struct to store the bootstrap broker string information for
Amazon MSK clusterss. These fields include:

BootstrapBrokerString
BootstrapBrokerStringPublicSASLIAM
BootstrapBrokerStringPublicSASLSCRAM
BootstrapBrokerStringPublicTLS
BootstrapBrokerStringSASLIAM
BootstrapBrokerStringSASLSCRAM
BootstrapBrokerStringTLS
BootstrapBrokerStringVPCConnectivitySASLIAM
BootstrapBrokerStringVPCConnectivitySASLSCRAM
BootstrapBrokerStringVPCConnectivityTLS

The setResourceAdditionalFields function is added to the
resource manager to fetch and set these fields from the AWS
API by calls the GetBootstrapBrokers API and populates
the corresponding fields in the ClusterStatus.
additionally, we're adding a customPreCompare hook function
to properly set default values for various Cluster spec fields
before comparing resources during the delta calculation - this
is fixing an observed bug that was impact the status fields
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/retest

/retest

/retest

/lgtm


[2.0] use of faker to generate random messages

[ ] A command to generate random names, addresses and text.

[ ]  A command to generate random names, addresses and text.

This issue will probably be closed as the openai do something similar and even more as done in #69

[ ]   A command to generate random names, addresses and text.

This issue will probably be closed as the openai do something similar and even more as done in #69

Confirmed.


Changing aligment of tiltle text

Perhaps it's very simple, but I can't understand how to change the direction of the text in the title (right alignment).
I can change aligment of paragraph by PP_ALIGN,
    p = tf.add_paragraph()     p.text = "This is a second paragraph that's bold"     p.font.bold = True     p.alignment = PP_ALIGN.CENTER
but it doesnt work with title. Tell me please.

name of slide
from pptx import Presentation
from pptx.util import Pt
from pptx.enum.text import PP_ALIGN
<img width="581" alt="8888" src="https://user-images.githubusercontent.com/20502236/30024429-737ece5c-917c-11e7-8407-4548cfd5ed89.PNG">

input_presentation = 'C:\\Users\Frostman\Sample.pptx'
slide_name = ['Name1','Name2','Name3','Name4','Name5']

my_presentation = Presentation(input_presentation)
slide_layout = my_presentation.slide_layouts[11]

**for slide_number in range (0,4):
    slide = my_presentation.slides.add_slide(slide_layout)
    title = slide.shapes.title
    title.text = slide_name[slide_number]

    title.alignment = PP_ALIGN.RIGHT**

Horizontal alignment applies to a paragraph, not a title (which is a shape). Something like:
for paragraph in title.text_frame.paragraphs:
    paragraph.alignment = PP_ALIGN.CENTER

thx


Assert Failed assem2str(): strlen(assem_stop_str[assem]) < size

Any value for -C <float> in contigs causes assert error:
% mccortex63 contigs -C 0.5 -f -o foo1.fa -p infer.ctp.gz infer.ctx

[25 Apr 2016 10:59:35-XuR][cmd] mccortex63 contigs -C 0.5 -f -o foo1.fa -p infer.ctp.gz infer.ctx
[25 Apr 2016 10:59:35-XuR][cwd] /mnt/seq/JOBS/MDU/J2016-06490/nullarbor.ST80/2014-23625
[25 Apr 2016 10:59:35-XuR][version] mccortex=86b4ffe zlib=1.2.7 htslib=1.3-37-gfc93dfc ASSERTS=ON hash=Lookup3 CHECKS=ON k=33..63
[25 Apr 2016 10:59:35-XuR] Taking number of kmers as genome size: 2,918,130
[25 Apr 2016 10:59:35-XuR][memory] 202 bits per kmer
[25 Apr 2016 10:59:35-XuR][memory] graph: 94.9MB
[25 Apr 2016 10:59:35-XuR][memory] paths: 30.1MB
[25 Apr 2016 10:59:35-XuR][memory] total: 95.1MB of 377.6GB RAM
[25 Apr 2016 10:59:35-XuR][hasht] Allocating table with 3,932,160 entries, using 60.2MB
[25 Apr 2016 10:59:35-XuR][hasht]  number of buckets: 131,072, bucket size: 30
[25 Apr 2016 10:59:35-XuR][graph] kmer-size: 63; colours: 1; capacity: 3,932,160
[25 Apr 2016 10:59:35-XuR][GPathReader] need 7216 paths 16810 bytes
[25 Apr 2016 10:59:35-XuR][GPathSet] Allocating for 7,216 paths, 7KB colset, 9.4KB seq => 143.3KB total
[25 Apr 2016 10:59:35-XuR][FileFilter] Loading file infer.ctx [1 colour]
[25 Apr 2016 10:59:35-XuR][GReader] 2,918,130 kmers, 58.4MB filesize
[25 Apr 2016 10:59:35-XuR][GReader] Loaded 2,918,130 / 2,918,130 (100.00%) of kmers parsed
[25 Apr 2016 10:59:35-XuR][hasht] buckets: 131,072 [2^17]; bucket size: 30; memory: 60.2MB; occupancy: 2,918,130 / 3,932,160 (74.21%)
[25 Apr 2016 10:59:35-XuR][hasht]  collisions  0: 2901076
[25 Apr 2016 10:59:35-XuR][hasht]  collisions  1: 16416
[25 Apr 2016 10:59:35-XuR][hasht]  collisions  2: 609
[25 Apr 2016 10:59:35-XuR][hasht]  collisions  3: 25
[25 Apr 2016 10:59:35-XuR][hasht]  collisions  4: 3
[25 Apr 2016 10:59:35-XuR][hasht]  collisions  5: 1
[25 Apr 2016 10:59:35-XuR][FileFilter] Loading file infer.ctp.gz [1 colour]
[25 Apr 2016 10:59:35-XuR][GPathSet] Allocating for 32,768 paths, 32KB colset, 384KB seq => 1MB total
[25 Apr 2016 10:59:35-XuR] Loaded 7,216 paths from 1,688 kmers
[25 Apr 2016 10:59:35-XuR][Assemble] Assembling contigs with 2 threads, walking colour 0
[25 Apr 2016 10:59:35-XuR][Assemble] Using missing info check: yes
[25 Apr 2016 10:59:35-XuR][Assemble] Stop traversal if step cummulative confidence < 0.500000
[25 Apr 2016 10:59:35-XuR][Assemble]   Writing contigs to foo1.fa
[25 Apr 2016 10:59:35-XuR][Assemble] Seeding with random kmers...
[src/tools/assemble_stats.c:43] Assert Failed assem2str(): strlen(assem_stop_str[assem]) < size
[25 Apr 2016 10:59:35-XuR] Assert Error
Aborted (core dumped)

Fixed - thanks for all the testing. Much appreciated.


chore: Add and upgrade dependencies

We need aws-cdk-lib and constructs in our codegen files because we are rendering
an L2 that has an L1 resource in it.
Due to jsii packaging we need these in dependencies otherwise a fresh install of the tool will
fail if you do not have aws-cdk-lib installed.
Also added blip on configuring AWS_REGION
Add small python example for SC

oh weird, we need cdk in both dev and normal because it is used by test files


Introduce GitHub Actions Workflow for Llama Stack Tests

What does this PR do?
Initial implementation of GitHub Actions workflow for automated testing of Llama Stack.
Key Features

Automatically runs tests on pull requests and manual dispatch
Provides support for GPU required model tests
Reports test results and uploads summaries

A couple questions:

can you attach some results of having run this workflow as you have been experimenting with it?
if I needed to run this workflow on an already committed commit on main/ let's say, is there a way to do it?
is there a way to manually start running this workflow on PRs?

(Some of these questions are due to my n00b-ness with everything GitHub :))

A couple questions:

can you attach some results of having run this workflow as you have been experimenting with it?

Here is the workflow being manually run off of Commit 435f34b that was pulled 11 hours ago.  Note that NO tests were collected for the marks: meta_reference and llama_3b:

if I needed to run this workflow on an already committed commit on main/ let's say, is there a way to do it?

Yes there is a way to do it.  You can specify it with the input that is currently labeled Branch to checkout.  Just pass in the SHA and it will use it as the checkout reference.  Branch to checkout can be used to specify the branch, tag, or SHA.  Here is a picture of the workflow when this occurs:

And these are the current inputs (and defaults) supported when manually running a workflow:

is there a way to manually start running this workflow on PRs?

The workflow will automatically run on PRs that change the following files:

    paths:
      - 'llama_stack/**/*.py'
      - 'tests/**/*.py'

Although answering your question: yes there is a way to also manually run the workflow referencing a PR; as mentioned above just specify the SHA of the top commit of the PR and the workflow will use that when checking out the repository.

@ashwinb Just updated this PR to temporarily not run on PRs until manual verification has been completed.
Testing has been verified to be the same via my fork's Run #135.


MudTreeView: Enhance Public Select Method

Description
As we discussed with @henon and @mikes-gh at #5856, treeview didn't have good options for programmatically changes.
But the PR at 5856 also didn't bring the exact requirement. If we want to select dynamic data like loaded from server, we need to a method that select with value, not with the TreeViewItem. So this PR added this option as method overload.
Where do we need this
In current situation, there is no way to select dynamic treeview item with right click on it. Now its possible. I also add a doc example that showed on this video
https://user-images.githubusercontent.com/78308169/206676816-f757f13b-6ca0-4fc4-a73c-62b12773f820.mp4
How Has This Been Tested?
Current programmatic change test enhanced.
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.
[x] I've added relevant tests.

Can you also make this example work by adding a public SelectedValue property?
https://try.mudblazor.com/snippet/ckGQvlQMeayRtVKV
In the snippet there are two trees which should be synced in terms of IsExpanded and SelectedValue. But only IsExpanded is synced, not the selection.

As per my previous comment I don't think this is the right way to go.  Adding Methods to manipulate the state of a component outside its normal lifecycle is not recommended.  We should be looking at a SelectedValue parameter

@henon ofc i can try this, but not sure i can do this with a little work.
@mikes-gh agree with you, but you can see this PR as the complementary part of the other PR, not a new thing. I also want to do an solid thing, but this PR has a tiny feature that will solve lots of things, its really worth. After list and select, i can work on treeview and arrange all methods again. It will be breaking change anyway.
private MudTreeViewItem<T> _selectedValue;
public HashSet<T> Items { get; set; }

As you see names should change, in current, value points the item and item points the value.

It shouldn't be hard. in OnParametersChanged check if SelectedValue is set then use your new method to set it. I agree with Mike, when SelectedValue parameter works we don't really need the other methods to be public.

in OnParametersChanged check if SelectedValue is set then use your new method to set it.

I don't understand this part.
And this look like not working easily, example i added the parameter
private T _selectedValueField;
        [Parameter]
        [Category(CategoryTypes.TreeView.Data)]
        public T SelectedValue
        {
            get => _selectedValueField;
            set
            {
                if ((_selectedValueField != null && value != null && _selectedValueField.Equals(value)) || (_selectedValueField == null && value == null))
                {
                    return;
                }
                _selectedValueField = value;
                _selectedValue = GetAllItems().FirstOrDefault(x => x.Value.Equals(value));
                UpdateSelected(_selectedValue, true).AndForget();
                SelectedValueChanged.InvokeAsync().AndForget();
            }
        }

This is not working, like i encounter on new select, selected value returns to null again etc. with lots of problems.

I offer to postpone the permanent solution later and handle this with bind:after, or the other pattern you find.

in OnParametersChanged check if SelectedValue is set then use your new method to set it.

I don't understand this part.

Sorry, I meant OnParametersSet or OnParametersSetAsync. In that method it is possible to check what the new value for the parameter SelectedValue is (if that param was changed).

@henon closing this PR as the the treeview codebase was rewritten and the PR code is no more relevant for the new codebase.


Implement DecodeAsFields for pointer types like Arc and Box

See https://github.com/paritytech/scale-encode/issues/18
Let's do the same here.

This appears to be the case here so closing this.


Use :landing_page_url to track approximate views of cases

This will be a good proxy for interest in particular cases, in the absence of being able to parse the Ahoy::Event data. It may even be superior to using events to track views, since cases viewed by the same person multiple times in a single visit count as separate events, thus potentially distorting its usefulness as a proxy for interest.

This is on production, graph shows total number of visits where a certain article was the landing page. Will create a separate issue to identify views over a discrete time period.


db.getSortedSetRange is not a function

Sorry to bother you again, this is probably something small, and likely my fault.
Here's the simple file I created:
var async = require('async');
var Data = require('../nodebb-plugin-import/server/data.js');
Data.eachPost(console.log,
  function (err) {
    if (err) {
      throw err;
    }
    console.log("DONE!!");
  }
);

When I run it I get this error:
/home/fidelix/www/anbient/forum/node_modules/nodebb-plugin-import/server/data.js:476
				db.getSortedSetRange(setKey, start, end, function(err, ids) {
				   ^

TypeError: db.getSortedSetRange is not a function
    at /home/fidelix/www/anbient/forum/node_modules/nodebb-plugin-import/server/data.js:476:8
    at Object.async.whilst (/home/fidelix/www/anbient/forum/node_modules/nodebb-plugin-import/node_modules/async/lib/async.js:675:13)
    at Object.Data.processIdsSet (/home/fidelix/www/anbient/forum/node_modules/nodebb-plugin-import/server/data.js:468:9)
    at Object.Data.processSet (/home/fidelix/www/anbient/forum/node_modules/nodebb-plugin-import/server/data.js:412:15)
    at Object.Data.each (/home/fidelix/www/anbient/forum/node_modules/nodebb-plugin-import/server/data.js:388:15)
    at Object.Data.eachPost (/home/fidelix/www/anbient/forum/node_modules/nodebb-plugin-import/server/data.js:290:15)
    at Object.<anonymous> (/home/fidelix/www/anbient/forum/node_modules/nodebb-plugin-import-ipboard/ipb.js:4:6)
    at Module._compile (module.js:409:26)
    at Object.Module._extensions..js (module.js:416:10)
    at Module.load (module.js:343:32)

Process finished with exit code 1

Any ideas why it's not defined? Looking at the db object it seems to have the proper configs (mongodb, localhost, etc)...

i think I know why, i  will fix it tomorrow

hey, I found the issue, but it's an easy fix, the thing is that you're trying to use the database before it had some time to load - there is a callback to db.init() but the issue with this one is if you're already running NodeBB at the same-time, the callback would never execute. I have a fix in mind, but it's Easter! so I am give you a very stupid workaround till i fix this the right way, hopefully next week.
timeout way

var async = require('async');
var Data = require('../nodebb-plugin-import/server/data.js');

// just wrap it in a timeout till it's ready.
setTimeout(function() {

  Data.eachPost(console.log,
    function (err) {
      if (err) {
        throw err;
      }
      console.log("DONE!!");
    }
  );

}, 3000); // 3 seconds worked for me, you might need more depending on how large your db-index is.

interval way

var async = require('async');
var Data = require('../nodebb-plugin-import/server/data.js');

var go = function () {
  Data.eachPost(console.log,
    function (err) {
      if (err) {
        throw err;
      }
      console.log("DONE!!");
    }
  );
};
// you can test it every 500ms
var interval = setInterval(function() {
  if (Data.db.client) {
    clearInterval(interval);
    go();
  }
}, 500);

Both solutions are very stupid, but if you're writing a one-off script, they should work

Thank you so much @akhoury. This worked!

Ops... let me leave it open...


feat: add server settings

Homarr

Thank you for your contribution. Please ensure that your pull request meets the following pull request:

[ ] Builds without warnings or errors (pnpm buid, autofix with pnpm format:fix)
[ ] Pull request targets dev branch
[ ] Commits follow the conventional commits guideline
[ ] No shorthand variable names are used (eg. x, y, i or any abbrevation)

bruh


Interface sometimes disappears

I see this more consistently on the Web Assembly version, and usually after opening the menu for the first time. I've also seen it happen in the Android emulator.
It may have to do with the menu animation.

This issue is fixed by https://github.com/dmeybohm/wisdom-chess/pull/19

I saw this again in both Firefox and Chrome. The error in Chromeconsole is:
108WebGL: INVALID_OPERATION: drawElements: no buffer is bound to enabled attribute
``

In Firefox the console errors are:

WebGL warning: drawElementsInstanced: Vertex attrib array 2 is enabled but has no buffer bound. 29
After reporting 32, no further warnings will be reported for this WebGL context.

Fixed in #32


ExpireEntityEvent not firing

According to its documentation, ExpireEntityEvent should fire when an Entity expires from the world.
I tried both given examples: "when an Endermite expires", "Lightning expires".
I watched them expire but my listener did not catch any ExpireEntityEvent, more suprisingly it did not catch an DestructEntityEvent either.
Setup

Tried both:

spongeforge-1.12.2-2503-7.0.0-BETA-2716, Forge version: forge-1.12.2-<IP_ADDRESS>5
spongevanilla-1.12.2-7.0.0-BETA-349

Java version: 1.8.0_144
Operating System: Windows

Reopen in https://github.com/SpongePowered/SpongeCommon.


nuitrack.h not found

Hi,
When I built the solution on windows with Cmake and tried to build on Visual studio I got nuitrack,h fille missing

I added the gesture folder to the include directories for the project and I got a lot more errors:

So I am thinking I didn't include the correct file location, Which nuitrack.h should I include in my project for this issue.
Thank you for your help.

Gesture/nuitrack.h#L4 is #include <nuitrack/Nuitrack.h>.
It is include header that contained in NuiTrack SDK. It is a necessary code.
It doesn't necessary to change if project is correctly generated using CMake.
Why did you change this line?

I was able to fix the issue.

Hi @pillaya21 , I met the same issue when I tried to build the samples. Would you please tell me how you fixed the issue? Thanks a lot!


Front-end flow unauthenticated risk

A sister issue in OpenGateway: https://github.com/GSMA-Open-Gateway/Open-Gateway-Documents/issues/52
The first step of the front-end flow is a call from the end-user device to the Aggregator, at this point the Aggregator must work out which Operator to forward the request onto; in the flow it talks about using TelcoFinder.
TelcoFinder may not be free, and indeed when using MSISDN TelcoFinder which uses HLR or equivalent it is very unlikely to be free.
Having TelcoFinder (or any compute heavy task) in the Authorisation Code flow is not going to work, authorisation tokens must be 'free' to compute, otherwise there is a huge financial risk through DDoS style attacks.

As per Yesterday's working group call (04/10), the issue will be discussed in the context of the GSMA technical stream, and anything that needs to be piggybacked into CAMARA will be raised later for everyone in CAMARA to review.


Unable to use templates with cluster name when passing userdata_override_base64

Describe the Bug
On order to launch node group with Bottlerocket OS I need to pass custom userdata, that templated like this:
[settings.kubernetes]
api-server = "${api_server}"
cluster-certificate = "${cluster_certificate}"
cluster-name = "${cluster_name}"

This template used to render local variable like this:
locals {
  bottlerocket_userdata = base64encode(templatefile("./templates/userdata.tpl",
    {
      api_server          = module.eks.eks_cluster_endpoint
      cluster_certificate = module.eks.eks_cluster_certificate_authority_data
      cluster_name        = module.eks.eks_cluster_id
    }
  ))
}

Which passed to module as:
...
userdata_override_base64 = local.bottlerocket_userdata
...

terraform plan stage fails with:
╷
│ Error: Invalid count argument
│
│   on .terraform/modules/eks_node_group/main.tf line 52, in data "aws_eks_cluster" "this":
│   52:   count = local.get_cluster_data ? 1 : 0
│
│ The "count" value depends on resource attributes that cannot be determined until apply, so Terraform cannot predict how many instances will be created. To work around this, use the -target argument to first apply only the
│ resources that the count depends on.
╵

Expected Behavior
Terraform plan should execute just fine.
Steps to Reproduce
I was able to narrow down templating issue:

userdata_override_base64 = module.eks.eks_cluster_id fails on plan with error from above
userdata_override_base64 = "fixed string" successfully passes plan

Environment (please complete the following information):
Anything that will help us triage the bug will help. Here are some ideas:
% terraform version
Terraform v0.15.3
on darwin_amd64
+ provider registry.terraform.io/hashicorp/aws v3.40.0
+ provider registry.terraform.io/hashicorp/kubernetes v2.2.0
+ provider registry.terraform.io/hashicorp/local v2.1.0
+ provider registry.terraform.io/hashicorp/null v3.1.0
+ provider registry.terraform.io/hashicorp/random v3.1.0
+ provider registry.terraform.io/hashicorp/template v2.2.0

Modules:

cloudposse/eks-cluster/aws 0.38.0
cloudposse/eks-node-group/aws 0.20.0

As fast I was able to debug, terraform bubbles error to topmost constructed variable, so it isn't real cause. Real error is in https://github.com/cloudposse/terraform-aws-eks-node-group/blob/0.20.0/userdata.tf#L49 around var.userdata_override_base64 == null check.

As a workaround I'm doing staged via terraform apply -target module.eks first and just terraform apply after. This makes outputs required for userdata template be available when planning eks_node_group module.

Looking into other issues, this looks like another instance of #37.

@z0rc What version of terraform-aws-eks-node-group was this? I cannot find any dependency of get_cluster_data on userdata_override_base64 to explain this behavior.

@z0rc Yes, this is not really a bug, so much as a limitation. This module expects its inputs to be available during the plan phase, and Terraform does not have the output of the template available at the plan phase. I wonder if this will work around the problem for you:
userdata_override_base64 = coalesce(local.bottlerocket_userdata, "===")

@Nuru proposed code didn't have any effect on the error, it's that with out without coalesce. I also tried kind similar approach using join function but error still remains.

I've hit the same using templatefile with the variable before_cluster_joining_userdata :
  before_cluster_joining_userdata         = templatefile("${path.root}/user_data/nodegroups_data.tpl", {
    PROXY_ADDRESS = aws_elb.registry.arn
  })
}

The error is the same:
╷
│ Error: Invalid count argument
│
│   on .terraform/modules/eks_node_group/main.tf line 52, in data "aws_eks_cluster" "this":
│   52:   count = local.get_cluster_data ? 1 : 0
│
│ The "count" value depends on resource attributes that cannot be determined until apply, so Terraform cannot predict how many instances will be created. To work around this, use the
│ -target argument to first apply only the resources that the count depends on.

Terraform version 0.15.5
Module version 0.20.0

@z0rc
Check the following snippet (it works for me, with bottlerocket AMI):
module "linux_nodegroup_1" {
  source  = ".../terraform-aws-eks-node-group.git?ref=tags/0.22.0"
  enabled = var.linux_nodegroup_1_enabled

  name                              = var.linux_nodegroup_1_name
  ami_image_id                      = var.linux_nodegroup_1_ami_image_id
  subnet_ids                        = var.vpc.private_subnet_ids
  cluster_name                      = module.eks_cluster.eks_cluster_id
  instance_types                    = var.linux_nodegroup_1_instance_types
  desired_size                      = var.linux_nodegroup_1_desired_size
  min_size                          = var.linux_nodegroup_1_min_size
  max_size                          = var.linux_nodegroup_1_max_size
  disk_size                         = var.linux_nodegroup_1_disk_size
  create_before_destroy             = var.linux_nodegroup_1_create_before_destroy
  cluster_autoscaler_enabled        = var.linux_nodegroup_1_cluster_autoscaler_enabled
  existing_workers_role_policy_arns = local.linux_nodegroup_1_existing_workers_role_policy_arns
  userdata_override_base64          = base64encode(data.template_file.linux_nodegroup_1_userdata.rendered)

  context = module.this.context
}

data "template_file" "linux_nodegroup_1_userdata" {
  template = file("${path.module}/templates/bottlerocket_userdata.toml.tpl")
  vars = {
    cluster_name     = module.eks_cluster.eks_cluster_id
    cluster_endpoint = module.eks_cluster.eks_cluster_endpoint
    cluster_ca_data  = module.eks_cluster.eks_cluster_certificate_authority_data
    node_labels      = join("\n", [for label, value in var.linux_nodegroup_1_kubernetes_labels : "\"${label}\" = \"${value}\""])
    node_taints      = join("\n", [for taint, value in var.linux_nodegroup_1_kubernetes_taints : "\"${taint}\" = \"${value}\""])
  }
}

@lyubomirdimov the only difference I see between your snippet and mine is I use templatefile function, where you're using data.template_file (which is actually not recommended). Still I tried this approach, which didn't work in my case unfortunately.
As I'm trying to dig into this issue, it looks like it's discussed at https://github.com/hashicorp/terraform/issues/26755 and at this moment of time would require refactroing this module in way where null checks won't be used for counts.

I can "fix" this particular issue by removing && var.userdata_override_base64 == null check from https://github.com/cloudposse/terraform-aws-eks-node-group/blob/0.22.0/userdata.tf#L49, after that terrafrom plan succeeds as expected. But this isn't a fix, as this breaks other use cases of this module.

@z0rc
I managed to reproduce the issue in my environment as well. The reason why i didn't see it, is because the cluster were existing before I introduced bottlerocket Nodegroups.
On clean deployment I get the error from the image below:

The module works like a charm with the default Amazon Linux 2 AMI, however I as well have been having lots of difficulties when using Custom AMIs, like Bottlerocket and Windows Core.

Okay, I give up trying to fix this and migrating away from this module.
My last attempt was trying to build my own aws_launch_template and pass it to module. This opened another can of worms, this time related to custom launch templates:
╷
│ Error: Invalid count argument
│
│   on .terraform/modules/eks_node_group/launch-template.tf line 107, in data "aws_launch_template" "this":
│  107:   count = local.enabled && length(local.configured_launch_template_name) > 0 ? 1 : 0
│
│ The "count" value depends on resource attributes that cannot be determined until apply, so Terraform cannot predict how many instances will be created. To work around this, use the -target argument to first apply only the
│ resources that the count depends on.
╵

It seems this module only reliably works when using majority of defaults to run AL2 nodes, running custom ami, userdata, launch template doesn't work and isn't actually tested.

@z0rc Thank you for all the effort you put into this issue, and I am sorry we have not fixed it yet. As you pointed out, fixing it will require breaking changes, and we have them on the drawing board. We plan to have this module updated in the near future.

It seems this module only reliably works when using majority of defaults to run AL2 nodes, running custom ami, userdata, launch template doesn't work and isn't actually tested.

This is not fair. All of the inputs were tested and work when given static inputs. The fair statement is that this module only reliably works when its inputs are known at plan time. It was imagined that you would be hard-coding values, not supplying derived values. It was also imagined that Hashicorp would fix this general issue.
In any case, the next version will be reworked so that it works with derived values as inputs. We will probably also simplify it so that it always creates a launch template instead trying to avoid it when possible.

@z0rc This and all the related issues should now be resolved with the release of version 0.25.0 of this module. Please give it a try and report back.

@Nuru sorry, but I won't be able to test and report on it. I already migrated away from module and migration back to module doesn't provide a value in my case.


[Feature] Custom floating menu

I'm submitting a
Feature request
Description
Love the project! Thanks for your hard work.
I'm trying to implement a floating menu that would appear above the line when some text is selected
I saw the issue for menu on the bottom of the editor. I was wondering if It's possible to do it in prosemirror bypassing ngx-editor toolbar altogether?

Thanks for any pointers you can offer.

I was wondering if It's possible to do it in prosemirror bypassing ngx-editor toolbar altogether?

You should be able to provide plugins to prosemirror directly via.
new Editor({
  plugins: [
    // prosemirror plugins
  ],
});

or register a plugin on the fly like.
const editor = new Editor();
editor.registerPlugin(plugin);

Refer bubble component as an example https://github.com/sibiraj-s/ngx-editor/blob/master/src/lib/components/bubble/bubble.component.ts
This is what that shows up when you click on a link in the editor. this is based on the example here https://prosemirror.net/examples/tooltip/.
Inject the component into the element with NgxEditor class. You should be able to write a floating menu.
Plus I am working on exposing commands so that would allow to build custom menu easily.
like
const editor = new Editor();

editor
  .commands()
  .bold()
  .insertText()
  .italic()
  .exec();

but it will take few weeks. as I am occupied with personal work here.
I hope this helps.

I was wondering if It's possible to do it in prosemirror bypassing ngx-editor toolbar altogether?

You should be able to provide plugins to prosemirror directly via.
new Editor({
  plugins: [
    // prosemirror plugins
  ],
});

or register a plugin on the fly like.
const editor = new Editor();
editor.registerPlugin(plugin);

Refer bubble component as an example https://github.com/sibiraj-s/ngx-editor/blob/master/src/lib/components/bubble/bubble.component.ts
This is what that shows up when you click on a link in the editor. this is based on the example here https://prosemirror.net/examples/tooltip/.
Inject the component into the element with NgxEditor class. You should be able to write a floating menu.
Plus I am working on exposing commands so that would allow to build custom menu easily.
like
const editor = new Editor();

editor
  .commands()
  .bold()
  .insertText()
  .italic()
  .exec();

but it will take few weeks. as I am occupied with personal work here.
I hope this helps.

Hi,
I create a component for doing this.
ballon.component.html
<ngx-editor-menu class="tooltip" [editor]="editor" > </ngx-editor-menu>
ballon.component.css (see Prosemirror tooltip)
:host {
    position: absolute;
      .tooltip {
        position: absolute;
        // pointer-events: none;
        z-index: 20;
        background: white;
        border: 1px solid silver;
        border-radius: 2px;
        padding: 2px 10px;
        margin-bottom: 7px;
        -webkit-transform: translateX(-50%);
        transform: translateX(-50%);
      }

      .tooltip:before {
        content: "";
        height: 0;
        width: 0;
        position: absolute;
        left: 50%;
        margin-left: -5px;
        bottom: -6px;
        border: 5px solid transparent;
        border-bottom-width: 0;
        border-top-color: silver;
      }

      .tooltip:after {
        content: "";
        height: 0;
        width: 0;
        position: absolute;
        left: 50%;
        margin-left: -5px;
        bottom: -4.5px;
        border: 5px solid transparent;
        border-bottom-width: 0;
        border-top-color: white;
      }

}

ballon.component.ts
import {
  Component,
  ElementRef,
  Input,
  OnDestroy,
  OnInit,
  Renderer2
} from '@angular/core';
import {
  Editor
} from 'ngx-editor';
import {
  EditorView
} from 'prosemirror-view';
import {
  Subscription
} from 'rxjs';

@Component({
  selector: 'app-ballon',
  templateUrl: './ballon.component.html',
  styleUrls: ['./ballon.component.scss']
})
export class BallonComponent implements OnInit, OnDestroy {
  @Input() editor: Editor;

  private view: EditorView;
  private updateSubscription: Subscription;

  constructor(
    private el: ElementRef,
    private renderer: Renderer2
  ) {}

  private setDomPosition(): void {

    this.showBubble();
    let state = this.view.state
    let {
      from,
      to
    } = state.selection;
    let start = this.view.coordsAtPos(from),
      end = this.view.coordsAtPos(to)

    let box = (this.view.dom.parentNode as any).getBoundingClientRect();

    let left = (start.left + end.left) / 4;

    let bottom = (box.bottom - start.top + 50);

    this.renderer.setStyle(this.el.nativeElement, 'left', `${left}px`);
    this.renderer.setStyle(this.el.nativeElement, 'bottom', `${bottom}px`);
  }

  private showBubble(): void {
    this.renderer.setStyle(this.el.nativeElement, 'display', 'flex');
  }

  private hideBubble(): void {
    this.renderer.setStyle(this.el.nativeElement, 'display', 'none');
  }

  private update(): void {
    const {
      state
    } = this.view;

    if (state.selection.empty) {
      this.hideBubble();
      return
    }

    // update dom position
    this.setDomPosition();
  }

  ngOnInit(): void {
    this.view = this.editor.view;

    this.updateSubscription = this.editor.onUpdate.subscribe(() => {
      this.update();
    });
  }

  ngOnDestroy(): void {
    this.updateSubscription.unsubscribe();
  }

}

Add position: relative; to .editor class in app.component.scss
Then replace
<ngx-editor-menu [editor]="editor" [toolbar]="toolbar" [customMenuRef]="customMenu">         </ngx-editor-menu>
by
<app-ballon [editor]="editor"></app-ballon> in app.component.html
Result :

I hope this can help you.

Hi,
I create a component for doing this.
ballon.component.html
<ngx-editor-menu class="tooltip" [editor]="editor" > </ngx-editor-menu>
ballon.component.css (see Prosemirror tooltip)
:host {
    position: absolute;
      .tooltip {
        position: absolute;
        // pointer-events: none;
        z-index: 20;
        background: white;
        border: 1px solid silver;
        border-radius: 2px;
        padding: 2px 10px;
        margin-bottom: 7px;
        -webkit-transform: translateX(-50%);
        transform: translateX(-50%);
      }

      .tooltip:before {
        content: "";
        height: 0;
        width: 0;
        position: absolute;
        left: 50%;
        margin-left: -5px;
        bottom: -6px;
        border: 5px solid transparent;
        border-bottom-width: 0;
        border-top-color: silver;
      }

      .tooltip:after {
        content: "";
        height: 0;
        width: 0;
        position: absolute;
        left: 50%;
        margin-left: -5px;
        bottom: -4.5px;
        border: 5px solid transparent;
        border-bottom-width: 0;
        border-top-color: white;
      }

}

ballon.component.ts
import {
  Component,
  ElementRef,
  Input,
  OnDestroy,
  OnInit,
  Renderer2
} from '@angular/core';
import {
  Editor
} from 'ngx-editor';
import {
  EditorView
} from 'prosemirror-view';
import {
  Subscription
} from 'rxjs';

@Component({
  selector: 'app-ballon',
  templateUrl: './ballon.component.html',
  styleUrls: ['./ballon.component.scss']
})
export class BallonComponent implements OnInit, OnDestroy {
  @Input() editor: Editor;

  private view: EditorView;
  private updateSubscription: Subscription;

  constructor(
    private el: ElementRef,
    private renderer: Renderer2
  ) {}

  private setDomPosition(): void {

    this.showBubble();
    let state = this.view.state
    let {
      from,
      to
    } = state.selection;
    let start = this.view.coordsAtPos(from),
      end = this.view.coordsAtPos(to)

    let box = (this.view.dom.parentNode as any).getBoundingClientRect();

    let left = (start.left + end.left) / 4;

    let bottom = (box.bottom - start.top + 50);

    this.renderer.setStyle(this.el.nativeElement, 'left', `${left}px`);
    this.renderer.setStyle(this.el.nativeElement, 'bottom', `${bottom}px`);
  }

  private showBubble(): void {
    this.renderer.setStyle(this.el.nativeElement, 'display', 'flex');
  }

  private hideBubble(): void {
    this.renderer.setStyle(this.el.nativeElement, 'display', 'none');
  }

  private update(): void {
    const {
      state
    } = this.view;

    if (state.selection.empty) {
      this.hideBubble();
      return
    }

    // update dom position
    this.setDomPosition();
  }

  ngOnInit(): void {
    this.view = this.editor.view;

    this.updateSubscription = this.editor.onUpdate.subscribe(() => {
      this.update();
    });
  }

  ngOnDestroy(): void {
    this.updateSubscription.unsubscribe();
  }

}

Add position: relative; to .editor class in app.component.scss
Then replace
<ngx-editor-menu [editor]="editor" [toolbar]="toolbar" [customMenuRef]="customMenu">         </ngx-editor-menu>
by
<app-ballon [editor]="editor"></app-ballon> in app.component.html
Result :

I hope this can help you.

Thanks @fredmabire . It is a good one.
But I didn't add it to the lib yet since it was difficult to calculate the position of nodes with images in it. I need to try if I can do one will node_views. couldn't find time.
Also this might overlap with the builtin link bubble that showsup when selection has a link or clicked on a link

Thanks @fredmabire . It is a good one.
But I didn't add it to the lib yet since it was difficult to calculate the position of nodes with images in it. I need to try if I can do one will node_views. couldn't find time.
Also this might overlap with the builtin link bubble that showsup when selection has a link or clicked on a link

No idea if this resource can be useful to you, the Tiptap editor (usable with VueJS) also uses Prosemirror and they have a system where the toolbar appears in a bubble.
https://github.com/ueberdosis/tiptap/blob/3b8ba27cc8871a7b8949fc423a8335edf7f9c3c8/packages/tiptap/src/Components/EditorMenuBubble.js
In any case I am interested in this project. Today I'm using Quill and I'll be taking the arguments in favor of NgxEditor but that's another subject.

No idea if this resource can be useful to you, the Tiptap editor (usable with VueJS) also uses Prosemirror and they have a system where the toolbar appears in a bubble.
https://github.com/ueberdosis/tiptap/blob/3b8ba27cc8871a7b8949fc423a8335edf7f9c3c8/packages/tiptap/src/Components/EditorMenuBubble.js
In any case I am interested in this project. Today I'm using Quill and I'll be taking the arguments in favor of NgxEditor but that's another subject.

Thanks @GitHubish . I just know Tiptap, not aware of its complete features. I will look into it.
Also, I will try to add floating menu components and try to fix conflicts with link bubble that already exists.

Thanks @GitHubish . I just know Tiptap, not aware of its complete features. I will look into it.
Also, I will try to add floating menu components and try to fix conflicts with link bubble that already exists.

Hi @sibiraj-s !
Yes I saw that there was a display problem with the link bubble.
This was just a brief example for @MateMahinKhan .
I am also interested in NgxEditor and I did not know TipTap editor either (thanks @GitHubish)  but some elements are interesting such as suggestions and tables.
Maybe I can help if you want.

Hi @sibiraj-s !
Yes I saw that there was a display problem with the link bubble.
This was just a brief example for @MateMahinKhan .
I am also interested in NgxEditor and I did not know TipTap editor either (thanks @GitHubish)  but some elements are interesting such as suggestions and tables.
Maybe I can help if you want.

@fredmabire cool if I could make you discover a new library.
For the @Mention feature there may be something inspired by Tiptap too and quill-mention that I use and that I had to fork for an internal project and rewritten in Typescript. Maybe I could help NgxEditor to have a module like this one.
Anyway great job @sibiraj-s

@fredmabire cool if I could make you discover a new library.
For the @Mention feature there may be something inspired by Tiptap too and quill-mention that I use and that I had to fork for an internal project and rewritten in Typescript. Maybe I could help NgxEditor to have a module like this one.
Anyway great job @sibiraj-s

Thanks @GitHubish.
NgxEditor supports plugins. See https://github.com/joelewis/prosemirror-mentions.
You can do this and no changes is required from the editor side for now. there are many community provided plugins. you can use them. If nothing is available we can add it.
new Editor({
  pluigins: [
    new MentionsPlugin()
  ]
})

Create a discussion if you need any help regarding this. Let's continue it there if required. lets keep this discussion for floating menu alone.

Thanks @GitHubish.
NgxEditor supports plugins. See https://github.com/joelewis/prosemirror-mentions.
You can do this and no changes is required from the editor side for now. there are many community provided plugins. you can use them. If nothing is available we can add it.
new Editor({
  pluigins: [
    new MentionsPlugin()
  ]
})

Create a discussion if you need any help regarding this. Let's continue it there if required. lets keep this discussion for floating menu alone.

I have added a bubble with minimal options as floating menu built in in v8.
It is also possible to make the current menu floating around selection with few lines of code. See https://stackblitz.com/edit/ngx-editor-floating-menu. The position calculation is not perfect yet. I will update that later.

I have fixed the positioning issue in demo. I will see if I could provide the same as an easy to use built in component with better positioning.

@sibiraj-s thank you it's already super cool!
The problem I often see, which would be great to fix is when the editor is in a modal and the parent has an overflow: hidden | scroll, floating toolbars are a problem if they are not created at the body level.
example : demo
Do you think it can be done?

Do you think it can be done?

Yes.
I have forked your example and made few changes to append to body. Checkout stackblitz

I just made rendering floating menu simpler in v8.0.0-beta.4
A Simple floating menu (default)
<ngx-editor [editor]="editor"> </ngx-editor>
<ngx-editor-floating-menu [editor]="editor"></ngx-editor-floating-menu> // place this whererver required.

To make the existing menu float
<ngx-editor [editor]="editor"> </ngx-editor>
<ngx-editor-floating-menu [editor]="editor">
  <ngx-editor-menu [editor]="editor"></ngx-editor-menu>
</ngx-editor-floating-menu>

You can place the floating menu anywhere in the HTML. just make sure its wrapper has postion: relative on it.
Demo: Default Floating Menu
Demo: Floating Menu with menu component

I don't know if its bug or feature but this info might be helpful for someone:
floating menu is not working when Ivy is disabled in tsconfig:
"angularCompilerOptions": {     "enableIvy": false,   }
You can try it here (Ivy is disabled in this example. To fix menu just enable it):
https://stackblitz.com/edit/ngx-editor-floating-menu-o7e8d1?file=tsconfig.json

thanks @wkulinski. Angular 9 comes with ivy enabled by default. It didn't occur to me to test in non ivy mode at the time of development. I will check this.


Excessive property warnings when using custom material and property lookups

Using a custom material to access data in property tables triggers excessive warnings when using property heavy datasets such as Cesium OSM Buildings.
Loading the below usda produces approximately 75,000 warnings.
osm_buildings_with_properties.zip
It may be better to only display warnings for properties that the material is trying to use, as opposed to every property in the tileset.

Yup, I'd like to fix this soon. https://github.com/CesiumGS/cesium-omniverse/pull/579 was the first step, but I have a better fix in mind that I haven't gotten to yet.

I still seem to be getting this, loading https://github.com/CesiumGS/cesium-omniverse-samples/blob/main/tutorials/07_style_by_properties/07_style_by_properties.usda when running a build of 1d4f98be7001c044a8d4b39759f50895e40a6738 which should have the latest #626 codebase


concatTextFile: init

Motivation for this change
Fixes https://github.com/NixOS/nixpkgs/issues/147589
Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

@pennae I wanted to give it an API similar to writeTextFile

huh, didn't realize writeTextFile uses paths of that form too. in that case consistency is more important.

@roberth that would make it inconsistent with writeTextFile

@roberth that would make it inconsistent with writeTextFile

Bug for bug compatibility is not a requirement, but implementing correctly it is not feasible now without a huge effort to get the overriding right, so my suggestion can wait and be implemented for all such functions later. It'd be a slightly breaking change because of the potential for up-preferences (../) in the file. I'll write an issue.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/426


clone the search interface to a new view, called "wall"

following the steps to the new milestone at https://groups.google.com/forum/#!msg/loklak/7azwyuslmTU/q9ueCSgwjZUJ
there is a preparation step necessary to be able to simultanously work on the twitter wall and the search interface: clone the "search" view to a "wall" view.
This looks like a simple thing to do, still I want to mention it to prevent irritation wether or not the existing search interface is to be extendet. It should not. "search" and "wall" shall evolve separately (while they can re-use code for each other)
This has Prio 1 because it's the first thing to do.

Also: there shall be NO top headline bar, and no menu. This page is considered to run in a browser kiosk / full screen mode.

thank you!


(OraklNode) Use RWMap for latest feed data storage instead of redis

Description
Use in memory RWMap to store and load latest feed data instead of redis
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist before requesting a review

[ ] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.

Deployment

[ ] Should publish npm package
[ ] Should publish Docker image

@coderabbitai review

@coderabbitai review

@coderabbitai review


Update to<EMAIL_ADDRESS>foundations

Closes #126
Command used:
$ npm run sync -- -t v3.4.1

This will also exclude the defaultPlugins by now, as they are only related to Camunda BPM, e.g. the deployment tool.

I'll double-check the new client extension feature to properly work with the Zeebe Modeler (e.g. by creating an example plugin).

For some reason, GitHub detects conflicts here which shouldn't be (since I removed them). I'd propose to do the merge to develop locally. We had this problems in the past, cf. https://github.com/zeebe-io/zeebe-modeler/pull/84#issuecomment-526527339
$ git checkout develop && git merge update-to-modeler341