New update broke everything

The game got an update this morning. Live tracking and log recovery doesn't work anymore.

20221018-1.log
Example log file - 1 round of Sweet Thieves

Hmm, possible cause
10/18/2022: [GameStateMachine] Replacing FGClient.StateMainMenu with FGClient.StateMatchmaking
10/18/2022: [StateMatchmaking] Begin matchmaking solo
Compared to a log from 19th Sept
19:30:01.317: [GameStateMachine] Replacing FGClient.StateMainMenu with FGClient.StateFNMMSMatchmaking
19:30:01.317: [StateMatchmaking] Begin party communications

Hmm, so looking at the event outcome - it seems that stamps have the date rather than time stored.
Taking this win from Sweet Thieves
10/18/2022: [ClientGameManager] Handling unspawn for player FallGuy [17] XXX (ps5) (FG.Common.MPGNetObject)
10/18/2022: [ClientGameManager] Handling unspawn for player FallGuy [27] XXX (ps4) (FG.Common.MPGNetObject)
10/18/2022: [ClientGameManager] Handling unspawn for player FallGuy [16] XXX (win) (FG.Common.MPGNetObject)
10/18/2022: [ClientGameManager] Handling unspawn for player FallGuy [18] XXX (win) (FG.Common.MPGNetObject)
10/18/2022: [ClientGameManager] Handling unspawn for player FallGuy [22] XXX (win) (FG.Common.MPGNetObject)
10/18/2022: [ClientGameManager] Handling unspawn for player FallGuy [21] XXX (win) (FG.Common.MPGNetObject)
10/18/2022: [ClientGameManager] Handling unspawn for player FallGuy [19] XXY (win) (FG.Common.MPGNetObject)
10/18/2022: Cannot cycle spectators when not using the player spectator camera.
MPG.Utility.CustomLogHandler:LogFormat(LogType, Object, String, Object[])
UnityEngine.Logger:Log(LogType, Object, Object)
MPG.Utility.MPGMonoBehaviour:LogWarning(String)
FG.Common.CameraDirector:CyclePlayerSpectatorCamera(Boolean)
FG.Common.CameraDirector:RemoveSpectatorCameraTarget(GameObject)
FGClient.ClientGameManager:_untrackPlayerSpawn(MPGNetObject)
FGClient.ClientGameManager:_handleClientUnspawnNetObject(MPGNetObject)
System.Action1:Invoke(T) FG.Common.MPGNetObjectManager:UnspawnNetObject(MPGNetID, UnspawnGameObjectPolicy) FGClient.ClientGameManager:HandleServerUnspawnNetObject(GameMessageServerUnspawnObject) System.Action1:Invoke(T)
FGClient.ClientNetworkMessageProcessor:processMessage(GameConnection, GameMessageBase)
FG.Common.UnityNetworkMessageHandler:HandleAndFreeInboundGameMessage(MessageEnvelope)
FG.Common.UnityNetworkMessageHandler:PeekAndHandleMessage(NetworkMessage, GameConnection)
Spine.Unity.InstructionDelegate:Invoke(SkeletonRendererInstruction)
UnityEngine.Networking.NetworkConnection:HandleReader(NetworkReader, Int32, Int32)
UnityEngine.Networking.NetworkClient:InternalNetworkDataReceive()
UnityEngine.Networking.NetworkClient:NetworkDataReceive()
(Filename: C:\buildslave\unity\build\Runtime/Export/Debug/Debug.bindings.h Line: 39)
10/18/2022: [ClientGameManager] Handling unspawn for player FallGuy [20] XXX (win) (FG.Common.MPGNetObject)
10/18/2022: [ClientGameManager] Handling unspawn for player FallGuy [24] XXX (switch) (FG.Common.MPGNetObject)
10/18/2022: [ClientGameManager] Handling unspawn for player FallGuy [23] XXX (win) (FG.Common.MPGNetObject)
10/18/2022: [ClientGameManager] Handling unspawn for player FallGuy [25] Chℝɪsproツ (switch) (FG.Common.MPGNetObject)
10/18/2022: [ClientGameManager] Handling unspawn for player FallGuy [26] Moppel2099 (ps4) (FG.Common.MPGNetObject)
10/18/2022: ClientGameManager::HandleServerPlayerProgress PlayerId=5 is succeeded=True
10/18/2022: ClientGameManager::HandleServerPlayerProgress PlayerId=9 is succeeded=True
10/18/2022: ClientGameManager::HandleServerPlayerProgress PlayerId=4 is succeeded=True
10/18/2022: ClientGameManager::HandleServerPlayerProgress PlayerId=8 is succeeded=True
10/18/2022: ClientGameManager::HandleServerPlayerProgress PlayerId=12 is succeeded=True
10/18/2022: ClientGameManager::HandleServerPlayerProgress PlayerId=10 is succeeded=True
10/18/2022: ClientGameManager::HandleServerPlayerProgress PlayerId=11 is succeeded=True
10/18/2022: SquadManager::GetSquadScore squadId0 not found return 0
10/18/2022: ClientGameManager::HandleServerPlayerProgress PlayerId=2 is succeeded=True
10/18/2022: ClientGameManager::HandleServerPlayerProgress PlayerId=3 is succeeded=False
10/18/2022: ClientGameManager::HandleServerPlayerProgress PlayerId=6 is succeeded=False
10/18/2022: ClientGameManager::HandleServerPlayerProgress PlayerId=1 is succeeded=False
10/18/2022: ClientGameManager::HandleServerPlayerProgress PlayerId=7 is succeeded=False
10/18/2022: Team 0 score = 0
10/18/2022: Team 1 score = 12
10/18/2022: SquadManager::GetSquadScore squadId0 not found return 0
10/18/2022: [ClientGameManager] Server notifying that the round is over.
I'll see if ShootMe can see anything useful, but I'm guessing that if there's no actual timestamp (like 19:30:01.317) , there is nothing to calculate from.

As adamkelly86 has shown, the log lines have had their timestamp changed to a datestamp which means round times can't be calculated. However, everything else appears to be the same, so it should be possible to still track round progress and wins/losses.

I don't see any information about round names in the log either, unless I'm missing something.

There are two ways the tracker gets the round, from parsing either the round loading line itself e.g.
10/19/2022: [StateGameLoading] Finished loading game level, assumed to be round_gauntlet_02_ss2_launch. Duration: 2.39s
or if it can't work it out from that, then from the splash screen loading line e.g.
10/19/2022: [StateGameLoading] Loading game level scene FallGuy_Gauntlet_02_01 - frame 13343
where gauntlet_02 is the internal name for Dizzy Heights (each round has its own internal name).

With the removal of the timestamps it will take some effort to change everything to work close to like it used to. I don't really have that much time these days to do that unfortunately. If anyone else can figure something out and send a pull request I would use that, but don't have the time to do it myself at this point.

so @ShootMe auto update system will update and fix this issue automatic or you need give new file?

Can you provide me with the log file before the update?

@adamkelly86 @ShootMe Can you provide me with the log file before the update?

You can look at some of my logs from last june.
Player-prev.log
Player-prev.log
Player.log
Player.log
Player-prev.log

It's not perfect, but I've modified the source code. Just overwrite the existing folder and run. please test.
FallGuyStatsManualUpdate.zip

It's not perfect, but I've modified the source code. Just overwrite the existing folder and run. please test.
FallGuyStatsManualUpdate.zip

It doesn't work, the problem persists. My stats not updating after a round is over

ok

2022-10-29: [GameSession] Changing state from Countdown to Playing
2022-10-29: [ClientGameManager] Handling unspawn for player FallGuy [52] julyan-papa (ps4) (FG.Common.MPGNetObject)
Guys. I've been reviewing the source code all day today, but since the log file does not provide the start and end time information of the round after the update, there seems to be a limit to the way to read the current FallGuysStats log file. Therefore, the lap time function of the round is abandoned, and information such as the timer function, number of people, and ping can be displayed.

2022-10-29: [GameSession] Changing state from Countdown to Playing
2022-10-29: [ClientGameManager] Handling unspawn for player FallGuy [52] julyan-papa (ps4) (FG.Common.MPGNetObject)
Guys. I've been reviewing the source code all day today, but since the log file does not provide the start and end time information of the round after the update, there seems to be a limit to the way to read the current FallGuysStats log file. Therefore, the lap time function of the round is abandoned, and information such as the timer function, number of people, and ping can be displayed.
I'm not sure why Mediatonic suddenly deleted the time information from the log file, but if they leave the time information in the log file again through a later update, then all functions will be possible.

you update the tracker.exe or not?

What does tracker.exe mean?

What does tracker.exe mean?

FallGuyStatsManualUpdate.zip i mean

If you download FallGuyStatsManualUpdate.zip that I posted in the above comment and overwrite it in the existing folder, the timer function, number of people, and ping will be displayed.

If you download FallGuyStatsManualUpdate.zip that I posted in the above comment and overwrite it in the existing folder, the timer function, number of people, and ping will be displayed.

The problem with your file is that it doesn't update the stats after a completed round. We've had this bug for 2 weeks and the creator of the software does nothing to fix it.

I have taken a stab at this myself, would like some feedback on my fork: https://github.com/ThreesFG/FallGuysStats
It reconstructs the log file by inserting timing data on the fly (requires the stat tracker to be running before starting the show). Round timings won't be exact but appear to be pretty close (within ~0.1 second). All tracking from 8.0.0 should be functional again, aside from the Ping on the overlay. I made it create a backup copy of the database file when running my version for the first time (juuuuuust in case).
I haven't spent a ton of time on it, so would like some confirmation that I'm not breaking things before submitting a pull request.

I have taken a stab at this myself, would like some feedback on my fork: https://github.com/ThreesFG/FallGuysStats
It reconstructs the log file by inserting timing data on the fly (requires the stat tracker to be running before starting the show). Round timings won't be exact but appear to be pretty close (within ~0.1 second). All tracking from 8.0.0 should be functional again, aside from the Ping on the overlay. I made it create a backup copy of the database file when running my version for the first time (juuuuuust in case).
I haven't spent a ton of time on it, so would like some confirmation that I'm not breaking things before submitting a pull request.

Giving it a try now and it seems to be working. Received an error message to start but doesn't seem to be affecting the stat tracking so far.

@bermanskid do you recall what the error message said? I have it show an alert the first time you run my version (info on timings not being exact and usage instructions), so want to rule out that you just saw that message.

That alert did pop up. The error message popped up after. I’ll try to get a screenshot of it next time I play.
Other than that the tracker seemed to be working well. Appreciate the work on that!

On Oct 31, 2022, at 7:30 AM, ThreesFG @.***> wrote:
@bermanskid https://github.com/bermanskid do you recall what the error message said? I have it show an alert the first time you run my version (info on timings not being exact and usage instructions), so want to rule out that you just saw that message.
—
Reply to this email directly, view it on GitHub https://github.com/ShootMe/FallGuysStats/issues/141#issuecomment-1297177290, or unsubscribe https://github.com/notifications/unsubscribe-auth/A35D3SFU4MNAYJUQ5FZO2TDWF7JXBANCNFSM6AAAAAARIFT5J4.
You are receiving this because you were mentioned.

So the error message hasn't popped up today. I even restarted my PC to see if it would, but nope. Tracker is still working though!

@neokidaze For your Log Path in the Settings screen, make sure this is either blank or is pointing at the logs folder. Your screenshot suggests this value was set to "C:\Program Files (x86)\Steam\steamapps\common\Fall Guys\FallGuys_client_game.exe" , which is incorrect.

@ThreesFG Understood! Thank you.

I have submitted a pull request to fix this issue and support the new log format: https://github.com/ShootMe/FallGuysStats/pull/151

How do you download the fix? anything I click on wont save to pc

How do you download the fix? anything I click on wont save to pc

Simply download the latest FallGuysStats' version (1.130). Chek the README.md file. ;)


To Enable startlistening feature of bing speech in webchat automatically

Hi, I have already integrated the bing speech api using speechrecognizer. It is working as expected. Now we had a requirement to enable the speech listening capability once an event/action is completed in the webchat.
How can we enable the speech listening automatically?

You could click the microphone from javascript with document.getElementsByClassName("wc-mic")[0].click().


Crashes and closes when moving files or deleting folder

VSCode Version: 1.35.0
OS Version: Manjaro 18.0.4 Illyria

Steps to Reproduce:

Try to move two files into a folder
Try to delete a folder, empty or not

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

I just tried both of those things on 1.35.0 and Linux Fedora 29 and can report they work fine. For the moving I did a drag and drop using the explorer.

I just tried both of those things on 1.35.0 and Linux Fedora 29 and can report they work fine. For the moving I did a drag and drop using the explorer.

Should I think it's an issue of my operating system¡

@JulianH99 Probably, but @Tyriar is the expert and will find out in due course I imagine. He added it to the Backlog which means it will be looked at in the future.
For development I highly recommend Fedora, here is my setup script if you want to try it out. It installs and sets up VS Code for you along with a lot of cool extras:
https://github.com/David-Else/fedora-ultimate-setup-script

Tested on ubuntu 18.04 too, and works fine. I'm starting to think my Manjaro is the problem. Gonna give a try to fedora again soon as your advice says @David-Else

I find the same issue using kubuntu 19.04. It only happens when I try to delete a file; when I do that vscode becomes irresponsive. I am using the snap version of vscode.


Adaptive media does not start after play button click - Safari browser

Screenshots
After adaptive media load:

After play button clicked:

Version
CDN v4.6.0
iPhone
Describe the bug
After clicking the play button of an adaptive media (video) the media control's action panel appears on the box and the default play button switches to pause but the actual video does not start playing.
Steps to reproduce

Use web chat with adaptive media
Click on the play button of the video.
The video action controls appear on the box but video does not start.

Expected behavior
The video should start and take the full screen.
Additional context
In the provided screenshot 2 if I click on full screen action icon then the video will take the full screen and start playing.
AdaptiveCard Json
{
  "type": "AdaptiveCard",
  "version": "1.1",
  "body": [
    {
      "type": "TextBlock",
      "size": "large",
      "isSubtle": true,
      "text": "Orientation Video"
    },
    {
      "type": "Media",
      "id": "orientationVideo",
      "sources": [
        {
          "mimeType": "video/mp4",
          "url": "some video url"
        }
      ]
    }
  ],
  "actions": [
    {
      "type": "Action.Submit",
      "data": {
        "CardType": "OrientationVideoCard",
        "ModelType": "System.String",
        "ActionTitle": "Done"
      },
      "title": "Done"
    }
  ]
}

@arman-g Can you please add the JSON for your Adaptive Card?

@arman-g Can you please add the JSON for your Adaptive Card?

Added, thanks.

@arman-g Where is your video hosted - YouTube, Vimeo, self-hosted?

It is self hosted.

@tdurnford just curios how do you use YouTube in a adaptive media?

@arman-g When I reproduce your issue it looks like the video is taking a long time to load. If you let the chat sit for a couple of minutes, does the video start to play? I'll need to investigate further if this is a Web Chat, Adaptive Card, or video hosting issue.
I'm not sure if Adaptive Cards supports YouTube Media; however, Web Chat supports Video Cards that support both YouTube and Vimeo Videos. Take a look at the code snippet below.
Bot Framework SDK v4.6 (Node)
await context.sendActivity({
  type: 'message',
  attachments: [{
    contentType: 'video/mp4',
    contentUrl: 'https://www.youtube.com/watch?v=rIJRFHDr1QE',
    name: 'Vision Keynote Highlights // Microsoft Build 2019'
  }]
});

@tdurnford on desktop using chrome, edge or firefox it works as excepted. Starts playing as soon the overlay play button is clicked. The problem happens on iPhone and I am not sure if this is a safari problem or iPhone. One think for sure, on iPhone when you play a video it automaticly goes to full screen. I think it has something to do with this. I mean maybe in order for this to work on iPhone when the overlay play button is clicked it should also make it full screen. I acspect this to be handeled by the framework but just giving my two cents whati it is worth :). In my case if you click on the full screen action button, or pause and play button right after the overlay play button click, it will start playing with out waiting 1 or two minutes as in your tests.
Thanks for the sample!

@tdurnford My farther test shows that this happens on Safari browser only. So this is not a iPhone issue but Safari.

@arman-g we can't reproduce this behavior. Is it possible the video is loading very slowly in your environment?

Hi @cwhitten, I don't think it is related to the video loading. The problem can only be reproduced on iPhone and safari on IOS.

On iPhone, when the play button is tapped we get the screen that I provided in the below image. Video will start only if I tap on full screen icon or tap on pause and play again.

On Safair on IOS, when play button is clicked we get similar screen like the image provided below. Video will start playing if I click the full screen icon or click pause and play icon again.

Hi @arman-g, I'm not reproducing your scenario unfortunately. When I press the play button, the video goes into fullscreen mode but immediately starts playing. This is on both Safari and Chrome. My understanding is that your video doesn't play unless you tap twice, is that correct?

I'm hoping another customer has run into your issue so that we can get more data.

@corinagum The Mock Bot Video command is a video card. This issue is related to videos in Adaptive Cards. I'm working on trying to repro the issue outside the scope of Web Chat to see if it's an issue with Adaptive Cards.

@arman-g This appears to be an issue with Adaptive Cards - not Web Chat. I've opened an issue in the Adaptive Cards Repo and you can track the progress of the issue here. Thanks for reporting the issue!


[BD-46] docs: remove duplicated menu title

Description
Remove duplicated menu title
Issue
Deploy Preview
https://deploy-preview-2756--paragon-openedx.netlify.app/
Merge Checklist

[ ] If your update includes visual changes, have they been reviewed by a designer? Send them a link to the Netlify deploy preview, if applicable.
[ ] Does your change adhere to the documented style conventions?
[ ] Do any prop types have missing descriptions in the Props API tables in the documentation site (check deploy preview)?
[ ] Were your changes tested using all available themes (see theme switcher in the header of the deploy preview, under the "Settings" icon)?
[ ] Were your changes tested in the example app?
[ ] Is there adequate test coverage for your changes?
[ ] Consider whether this change needs to reviewed/QA'ed for accessibility (a11y). If so, please add wittjeff and adamstankiewicz as reviewers on this PR.

Post-merge Checklist

[ ] Verify your changes were released to NPM at the expected version.
[ ] If you'd like, share your contribution in #show-and-tell.
[ ] 🎉 🙌 Celebrate! Thanks for your contribution.

Thanks for the pull request, @khudym!
When this pull request is ready, tag your edX technical lead.

@khudym 🎉 Your pull request was merged!  Please take a moment  to answer a two question survey so we can improve your experience in the future.


Improving wms test

I improved the WMS Test so that a Visible Picture is tested. So to discover Regressions faster.
On the way to this I fixed some warnings and added some more log messages (Because during debugging it worked but not on running).
I to fixed that to many changed events where fired when the property didn't really change anything.

I think too that the Extent should be set in an Asynchronous Init Call.


Unable to load the service index for source https://pkgs.dev.azure.com/devdiv/_packaging/dotnet-core-internal-tooling/nuget/v3/index.json

This is hitting a lot of our official builds, see https://dev.azure.com/dnceng/internal/_build/results?buildId=2446012 for an example.
+ @markwilkie

This was related to when we had to manually rotated PATs for service connections. Closing.


'Go to definition/Peek definition' does not select correct file for inherited classes

Issue Type: Bug
Upon 'Go to Definition' or 'Peek Definition' action, VS Code does not display the correct location. Line is selected properly, while file is not.
Example:

Class 'Foo' is defined in a file: a.py on a line 42

'Foo' has a method called 'doSomething()'

Class 'Bar' is defined in a file: b.py, and:

imports file a.py

'Bar' constructor inherits and initialize class 'a.Foo'

If I create 'self.doSomething()' call inside any 'Bar' method and try to 'Go to Definition', VS Code puts me in file b.py::42 (not a.py::42 as expected).

Works for normally created class instances.
Extension version: 2019.9.34911
VS Code version: Code 1.38.1 (b37e54c98e1a74ba89e03073e5a3761284e3ffb0, 2019-09-11T13:35:15.005Z)
OS version: Windows_NT x64 10.0.17763

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-4570 CPU @ 3.20GHz (4 x 3193)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwareoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_deferred_display_list: disabled_offskia_renderer: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.89GB (10.05GB free)

Process Argv

Screen Reader
no

VM
0%

Ping! Any news here? :)


[ListItem] with rightIconButton and containerElement with Link from react-router

Problem description
Having problems with a ListItem with containerElement with a Link from react-router and a rightIconButton. When clicking on the rightIconButton, the containerElement is also triggered. I have not been able to figure out a work around to this.
Link to minimal working code that reproduces the issue
import React from "react";
import PropTypes from "prop-types";
import FloatingActionButton from "material-ui/FloatingActionButton";
import ContentRemove from "material-ui/svg-icons/content/remove";
import {ListItem} from "material-ui/List";
import {Link} from "react-router";

function WorkoutListItem(props) {
  return (
    <ListItem
      primaryText={props.workout.title}
      secondaryText={props.workout.date}
      containerElement={<Link to={`/workouts/${props.workout.slug}-${props.workout.cuid}`}/>}
      rightIconButton={
        <FloatingActionButton
          mini={true}
          secondary={true}
          onClick={props.onDelete}
        >
          <ContentRemove />
        </FloatingActionButton>}
    />
  );
}

WorkoutListItem.propTypes = {
  workout: PropTypes.shape({
    title: PropTypes.string.isRequired,
    date: PropTypes.string.isRequired,
    slug: PropTypes.string.isRequired,
    cuid: PropTypes.string.isRequired,
  }).isRequired,
  onDelete: PropTypes.func.isRequired,
};

export default WorkoutListItem;

Versions

Material-UI: 0.18.0
React: 15.1.0
Browser: Google Chrome

I have tried out your example and using event.preventDefault() in props.onDelete seems to prevent the link from being triggered.

Thanks @jeroenwienk, that did the trick! :)

I also have a question regarding this one. I have a listItem with nested List but when I click the right icon button it also triggers the link. How will I prevent it? Thank you.


Add light variants

By reversing base00-base07. Fixes #1.
To save time I used a ruby script:
Dir.glob("atelier-*.yaml").each do |file|
  next if file =~ /-light/
  lines = File.readlines(file).map { |line| line.split(": ") }
  base00 = lines.index { |l| l.first == "base00" }
  range = base00..base00+7
  dark = lines[range]
  lines[range] = dark.map(&:first).zip dark.map(&:last).reverse
  File.open(file.sub(".y", "-light.y"), "w") do |f|
    f.puts lines.map { |l| l.join(": ") }.join
  end
end

Thanks for this, didn't realise and haven't looked into this new setup much, but will do..., I hope this will work out though (in/with the new setup,) because I really think these colorschemes should work in both dark and light variants: they were assembled that way.


Override non existing dependancies

Override non existing dependancies; build was failing because sourcegraph doesn't have this repos anymore.

Approved assuming that an implicit dependency needs to be replaced for functioning.

Please verify the original depender of replaced packages is working as expected


Add Google Analytics

This will be made up when we transfer the site to its original domain name - www.myclario.com so I can assign the tracking code to this domain.


[BUG] Can we have separate app icon for DARK mode and Light Mode

I am not sure if we can set Dark mode/ Light mode app icon for the mac Application. Can you please help me on this

Hi @SohailUrRahmanGit with App Icon you mean the Icon of the app itself in the Application folder or the default pop-up icon that appear on the dialog?

Both the scenarios

You can already define 2 different pop-up icon based on system appearance as you can see

Mind that when you rebrand the app.
The App Icon can't have 2 different version based on system appearance. It must be consistent.

Thank you for the. pop-up icon information , I would like to set App Icon too in Dark mode and light mode. I have done the feature in iOS but not sure if mac supports Setting up two different app icon

Hi @SohailUrRahmanGit no this doesn't looks to be possible for macOS Applications.


how to install chatterbot in conda

hello i want ask you how to install chatterbot package with conda. thanks for Thank you to answering me

Hi @rachid127, I've never used conda before, but I am sure that their documentation must include instructions on how to install Python packages.

You can use pip to install the package on conda @rachid127


[New Model]  Add HaloNet

Add HaloNet and align ported weights performance.
Reference paper: https://arxiv.org/pdf/2103.12731.pdf
Timm impl: https://github.com/rwightman/pytorch-image-models/blob/master/timm/models/byoanet.py

HaloNet is now released and merged, so I close this issue


ISPN-5345 Allow eviction for based on approximation of size compared to element count

https://issues.jboss.org/browse/ISPN-5345

Added in size calculation for byte[] key/value pairs with LRU

This supersedes' @wburns https://github.com/infinispan/infinispan/pull/3467 and introduces the following changes:

add thresholdPolicy=COUNT|MEMORY to choose how to evict entries
add thresholdSize which replaces maxEntries and works for both policies
add a warning when not running under HotSpot
add a MemoryUnit utility class inspired by TimeUnit

@tristantarrant I liked your previous suggestion much better... I'd rather have two different attributes/elements with clear names than a generic thresholdSize that doesn't say anything.
To add yet another suggestion: How about deprecating the <eviction max-entries="x"/> element and adding <eviction-count max-entries="x"/> and <eviction-memory max-size="x"/> to replace it? Would that complicate things too much?

Supporting that for embedded is reasonable, it is server which always causes head-scratching and pain, since we need to think about how to represent those "nodes" in the management model in a non ambiguous way

@danberindei using type and size now

CI failures are unrelated

They are gone

Silly, me, fixed now

Integrated, thanks Tristan!


app: Performance Lag in silk fountains.scn

In master there is this performance lag in silk fountains.scn. All are scenes perform quite like before.
https://user-images.githubusercontent.com/35926530/155843324-1c777f1b-5ffb-4c61-8e2e-0803710aedc2.mp4
https://user-images.githubusercontent.com/35926530/155843327-239c2508-2193-4506-acfd-933fb7488372.mp4

That UI is not accurate, it was never done correctly, and should not be used for performance bug reports.

Got it. But there is also a visible lag in the scene as you can see it during the avatar movement in above videos.


test framework: add optional upgrade step

longhorn/longhorn#746
Signed-off-by: Mohamed Eldafrawi<EMAIL_ADDRESS>

We also need to have a test case which can simulate the steps in https://github.com/longhorn/longhorn-tests/wiki/Manual-Test-Plan#longhorn-upgrade-test , to make sure our test is closer to the real environment.

Closing this PR in favor of longhorn/longhorn-tests#243


Convert bid & auction datetimes from UTC to the browser's local time

Description
UTC requires a time conversion that most people don't know. Let's update the instances where the Cadastre displays a UTC time to local time to help the user out.

Transaction Panel
Outstanding Bid - Response Deadline (both for the current licensor & everyone else)
Fair Lanch Claim - Auction End (instructions & transaction page)
Foreclosure Auction - Auction End (instructions & transaction page; for current licensor & everyone else)
Portfolio Modal - Action Date (only the date is shown but depending on timezone, the offset might change the date)

Closed with https://github.com/Geo-Web-Project/cadastre/pull/244


⚠ Bofrost.at has degraded performance

In add31a8, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2206 ms

Resolved: Bofrost.at performance has improved in bdce5e8 after 30 minutes.


update go-topaz-ui:0.1.8

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1210

Relevant Lines:
4533

💛  - Coveralls


Make rake-db import optional again

I upgraded from 1.24 to 1.27 and noticed that rake-db now requires import option. However, since I'm passing migrations collection directly, I don't think rake-db is actually using import.
If that is so, I propose to make import optional again, and instead crash in runtime with the suggested implementation included in the error message.
Alternatively, make the options type context-aware with something like:
{
  migrations: ModuleExportsRecord
  migrationsPath?: never
  import?: never
} | {
  migrations?: never
  migrationsPath?: string
  import(path: string): Promise<unknown>
}

or perhaps refactor it altogether:
migrations: ModuleExportsRecord | { path?: string; import(path: string): Promise<unknown> }

(UPD: both are records/objects so tech wise this won't work as is, but I'm keeping it just as something to consider).

Done, now import is optional when you have migrations.
The import is also used for code generation, not only for loading migrations. It will throw a runtime error if it's not set and you're trying to use db generate (g).


Share button does not return JSON

The share button doesn't return it in JSON, just plain old LaTex, it should return in JSON just for more controlability

Looks like this design is evolving nicely.


VoicePipelineAgents sometimes responds twice to a single user message

It happens frequently that the agent responds twice to the same user message. Based on the logs, it looks like the same user message gets added to the context and sent to TTS, and multiple responses to the same message are generated and played sequentially.
Agent config:
const agent = new pipeline.VoicePipelineAgent(
  vad,
  openai.STT.withGroq({ detectLanguage: true }),
  new openai.LLM({ model: 'gpt-4o', temperature: 0.7 }),
  new openai.TTS({ voice: 'onyx' }),
  {
    chatCtx,
    allowInterruptions: true,
    minEndpointingDelay: 1000,
    beforeLLMCallback: (_, ctx) => {
      const lastMessage = ctx.messages[ctx.messages.length - 1]
      if (lastMessage) {
        logger.debug({
          participantId,
          role: getReadableRole(lastMessage.role),
          content: lastMessage.content,
        })
      }
    },
    beforeTTSCallback: async (_, source) => {
      const messageChunks: string[] = []
      if (typeof source === 'string') {
        messageChunks.push(source)
      } else {
        for await (const chunk of source) {
          messageChunks.push(chunk)
        }
      }
      const message = messageChunks.join('')
      logger.debug({
        participantId,
        role: getReadableRole(llm.ChatRole.ASSISTANT),
        content: message,
      })
      return message
    },
  }
)

Log output:
{"message":"Agent waiting for participant"}
{"participantId":"responsive-nodejs","message":"Starting agent for participant id responsive-nodejs"}
{"participantId":"responsive-nodejs","role":"Assistant","content":"Hi! I’m your AI personal trainer. How can I assist you today?"}
{"participantId":"responsive-nodejs","role":"User","content":" Hey, how are you doing?"}
{"participantId":"responsive-nodejs","role":"Assistant","content":"I'm doing great, thanks for asking! How about you?"}
{"participantId":"responsive-nodejs","role":"User","content":" I'm hungry."}
{"participantId":"responsive-nodejs","role":"Assistant","content":"Ah, the classic case of the grumbling tummy! Any thoughts on what you're craving?"}
{"participantId":"responsive-nodejs","role":"User","content":" I have a sweet tooth.  Would you please suggest me something sweet and healthy?"}
{"participantId":"responsive-nodejs","role":"User","content":" I have a sweet tooth.  Would you please suggest me something sweet and healthy?"}
{"participantId":"responsive-nodejs","role":"Assistant","content":"Sure thing! How about a nice bowl of Greek yogurt topped with some berries and a drizzle of honey? It's sweet, satisfying, and packs a protein punch."}
{"participantId":"responsive-nodejs","role":"Assistant","content":"Sure thing! How about some Greek yogurt with a drizzle of honey and a sprinkle of nuts? It's sweet, satisfying, and packs a nice protein punch. If you're feeling a bit fancy, you could toss in some berries too!"}

Environment:

"@livekit/agents": "^0.5.2"
"@livekit/agents-plugin-openai": "^0.7.2"

did this start with 0.5.2 or was it already happening before?

It's been happening over the past few versions, so it's not a new issue

I got same issue on 0.6.0, when input voice is not continuous.
[16:17:45.693] DEBUG (15495): started playing the first time
speechId: "d1056d96-57e1-4127-9960-560bf147673c"
[16:17:45.833] DEBUG (15495): committed user transcript
userTranscript: "I'm looking for hotel and of curation and redression."
 Could you please provide a bit more detail about your needs, such as location and specific interests or requirements?"
interrupted: false
speechId: "d1056d96-57e1-4127-9960-560bf147673c"
[16:17:57.761] DEBUG (15495): playout finished
speechId: "d1056d96-57e1-4127-9960-560bf147673c"
interrupted: false
[16:17:57.762] DEBUG (15495): started playing the first time
speechId: "ac65470d-5310-4893-98c6-1edc68b0a42f"
[16:17:58.262] DEBUG (15495): committed user transcript
userTranscript: "I'm looking for hotel and of curation and redression. Hanau A nós."
[16:18:07.846] DEBUG (15495): playout finished
speechId: "ac65470d-5310-4893-98c6-1edc68b0a42f"
interrupted: false
[16:18:07.846] DEBUG (15495): committed agent speech
agentTranscript: "It seems like you might be looking for information related to accommodations and possibly something specific in Hanau. Could you clarify your request or provide more details?"
interrupted: false
speechId: "ac65470d-5310-4893-98c6-1edc68b0a42f"


[DO NOT MERGE] YLLD -> Introducing Yobama LLD edition one way to manage all at once, now with cool graphic design!

📝 Description
YLLD -> Yobama Ledger Live Desktop.
The prophecy is true that one day the great Master of Jedi shall be combined with one of the coolest presidents alive.
❓ Context

Impacted projects: live-desktop
Linked resource(s): ``

✅ Checklist

[x] Test coverage
[x] Atomic delivery
[x] No breaking changes

📸 Demo

🚀 Expectations to reach
Please make sure you follow these Important Steps.
Pull Requests must pass the CI and be internally validated in order to be merged.

YES WE CAN!
kudos to the best design

the people deserve Yobama. merge it.

pls merge i need to learn from the great master yobama

Hi @valpinkman,
I've been doing some research in terms of the popularity of YOBAMA, and it seems like this PR would drastically assist with the next Obama campaign. We might want to merge it as the US government would pay for any release fees.
Do we have a planned date already? My FBI agent is on the needles.

is this a psyop

You won't be closed till it's merged

 You won't be closed till it's merged

not closing it don't worry

Hello yes, do not close the PR, the Actual President is not willing to give up. Desu.

When will this be merged?  This feature is crucial for usability.


Input element of type number calls onChange twice when clicking increase or decrease buttons

I'm experiencing this issue using version 3.9.3 but have reproduced it in the codesandbox using v4 - https://09w6k.csb.app/

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
onChange is called twice when clicking the input number buttons.

Expected Behavior 🤔
onChange should be called once when clicking a button.

Steps to Reproduce 🕹
Try to look at this codesandbox https://09w6k.csb.app/
Clicking either the plus or minus buttons creates a double log statement.
Context 🔦

Your Environment 🌎

Tech
Version

Material-UI
v3.9.3

React
16.8.30

Browser
Chrome 77 beta

TypeScript
3.4.5

etc.

@mellson I can't reproduce it. Can you reproduce the issue with a native <inpu> element?

@oliviertassinari thanks for the quick reply 👍I just tested with a native <input> same problem. Thanks for the suggestion. It's not related to Material UI then.
I am running the lates macOS 10.15 Beta (19A536g) and I just tested that I have this problem on all Chromium browser. Luckily both Firefox and Safari seems to work, so I'll just stick with them for now.
Thanks again for guiding me in the right direction.

I am experiencing the same problem.
Environment:
Browser: Chrome 92.0.4515.131 (Official Build) (x86_64)
React: ^16.14.0
OS: macOS v.10.14.6


option: turn off checking for updates

please give us the option to turn off automatic update checks. in certain environments it's undesirable.

Approved. Will work on this.


Add startTour

I've noticed you did not yet had integrated the startTour functionality.
Here is a PR to add this functionality.
I hope you will approve this PR.
Kind regards

Hey @gertjanwytynck thanks for this PR.
I'm interresting too for using startTour feature of intercom.
@johnnynotsolucky It's possible to accept this PR and release a new version ? :)
Thanks a lot :)

@gertjanwytynck Looks like this PR gets rid fo the trackEvent handler?


Fix sxiv errors

Switch arguments to avoid sxiv error messages

Quick question, how does that cause errors? I havent gotten any errors so far.

Try the script (the original version) launching it in the terminal: sxiv should complain about -N and -b flags.
sxiv: -N: No such file or directory
sxiv: -b: No such file or directory

Oh okay, got it :+1:


Add service graph flag

Updated Helm command
Updated datasource agent scripts
Added checkbox to agent ui

Will not be implemented, it will be part of the SPM flag.


If I dont want to use patternPrefix and helperPrefix,

Is there any way to configure that?

No, currently this is not possible.
But we are open for suggestions.

Thanks a lot for your PR.
I was not shure what you wanted to achieve exactly.
As i understand you now - with your boolean switch you still allow prefixes. Selectors like .block, .block__element are valid as well as .a-block and a-block__element, ...
We could also call them block names. So far so good.
Because the prefixes are a core concept in our naming convention your PR results in some inconsistencies.

The selector -block is rejected with the misleading message: "Expected class name "-block" to use the [prefix]-[block] syntax."
The selector --block is rejected with the message: "Expected class name "--block" to use the [prefix]-[block]--[modifier] syntax."
The selctor m--modifier shouldn't be rejected (it's rejected because m is a default prefix).
state-m--state, 'state-m-block' or m--modifier__block are also rejected because of the default configuration
...

We're not at the end ;-)
and...
What do you think about disabling the individual configuration parameters?
"plugin/stylelint-bem-namics": {
    "patternPrefixes": false,
    "helperPrefixes": false,
}

Thanks for your explaining,
Actually, I was also considering those situations which you just mention,
When I recognize If we want to handle all the situation,  we should refactor the function of parseClassName to let it parse class name into an object which is holding namespace/helper-prefix/pattern-prefix no matter if these prefixes exist.
but I was little rush to close those prefixes for my own needs, sorry for that.
and
I think it is a good idea to disabling the individual config in that way.


Remove entangler map test

Summary
This PR removes the entangler map test, which used to test the entangler_map utilities in qiskit.utils and was probably kept under the algorithm tests for historical reasons. These utilities are soon to be removed in Qiskit 1.0 and not really used anywhere in qiskit_algorithms, so I believe that this test can be removed without further consequence.
Details and comments
Related to:

https://github.com/Qiskit/qiskit/pull/11323
https://github.com/Qiskit/qiskit/pull/11324

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
6464

Relevant Lines:
7137

💛  - Coveralls


rpi-base.inc: Add infrared dtbo

Add device tree binary overlays gpio-ir-tx and gpio-ir-tx to
support peripherals for sending and receiving infrared signals.
Signed-off-by: Leon Anavi<EMAIL_ADDRESS>
- What I did
Included gpio-ir-tx and gpio-ir-tx dtbo which are required for peripherals for sending and receiving infrared signals (for example using LIRC).
- How I did it
By adding gpio-ir-tx and gpio-ir-tx dtbo to variable RPI_KERNEL_DEVICETREE_OVERLAYS.
In the mean time I have submitted a patch to the upstream of meta-oe for upgrading LIRC to version 0.10.1. If it is approved I will carry on with LIRC bbappend file specific for Raspberry Pi.
Best regards, Leon

Hi @agherzan, @kraj,
Kind reminder. It has been more than a month. Could you please review this patch?
Thanks,
Leon

Merged. Thanks.


settings_file_path as .json

added a condition to check also for .json extension/format.

https://github.com/egabancho/flask-sso-saml/commit/8f03ca4934643d7c50c3e3709b9d113de7a1364e


Build classifier

Hi. I am very impressed with your paper. So, I tried to apply your algorithm to my data, but I ran into a problem.  Could you please elaborate on the function arguments corresponding to build_tree(persistence.py) in your code?. We would really appreciate if you could provide sample code using example data.

Thanks a lot for your interest in this project! Sure, we will ASAP make a release that facilitates this part of the workflow and provides additional functionalities.


keyd generates extraneous control down/up for keys in layer with :M modifier

First of all, thank you for your amazing work, keyd is life changing program that makes me more productive and happy every day of my life.
So I have a home_arrows layer which is activated by leftmeta key, obviously I still want super key so this layer has :M modifier.
However when using my shell I noticed that when I (accidentally) press both keys from this layer (when one key wasn't released yet and other is pressed), some weird escape sequences are inserted in the shell's input prompt. I ran keyd monitor and saw that when I hold leftmeta and press e.g. j (left) key, the following is logged:
keyd virtual keyboard	0fac:0ade	leftcontrol down
keyd virtual keyboard	0fac:0ade	leftmeta up
keyd virtual keyboard	0fac:0ade	leftcontrol up
keyd virtual keyboard	0fac:0ade	left down
keyd virtual keyboard	0fac:0ade	left up
keyd virtual keyboard	0fac:0ade	leftmeta down
keyd virtual keyboard	0fac:0ade	leftcontrol down

As you can see there's control events (I did not press anything but the leftmeta and j).
Here's minimal reproducible config:
[ids]
*

[main]
leftmeta = layer(home_arrows)

[home_arrows:M]
# arrow keys on the home row
j = left
k = down
l = up
; = right

When I removed :M from the layer header control events disappeared, but this is not an option for me since I still need super key (and my right super key is mapped to different layer).

I experience something similar in v2.4.3, but where meta is also returned when unwanted when using overload.
 From the man page, GLOBALS section:
*disable_modifier_guard*: By default, keyd will inject additional control keypresses where necessary in order to prevent programs from seeing additional modi‐
           fier taps (E.g alt in firefox). If set, this option disables that behaviour. (default: 0)

Yes I've found this recently too so it seems this is intentional. Well it haven't bothered me much anyways, so closing


Use AWS EC2 Linux/Unix as your local machine to deploy Akto

Hi,
I was going through the Akto repositories and the deployment in AWS through the CloudFormation template was taking more time and was utilizing more resources as well (Although the architecture of that is pretty solid). So, I was thinking of contributing a simple beginner-friendly method wanting to test Akto in the public cloud (in this case, AWS) but not utilizing more resources.
Here's the output of README.md after doing modifications! You can consider adding this if you find this useful!

Thank you, have a great day ahead!

Thanks @sumitNITS ! I have put in extra pointres for good security practices while installing Akto in Cloud.

Awesome explanation @ankush-jain-akto
Thanks for the correction.


Terminal colours not working correctly with bash on Windows

[x] I am on the latest Hyper.app version
[x] I have searched the issues of this repo and believe that this is not a duplicate

OS version and name: Windows 10
Hyper.app version: 1.3.1
Link of a Gist with the contents of your .hyper.js: https://gist.github.com/jameshclrk/83bf7c015a08c508426220b17d1eda04
Relevant information from devtools (CMD+ALT+I on Mac OS, CTRL+SHIFT+I elsewhere): N/A
The issue is reproducible in vanilla Hyper.app: Is Vanilla

Issue
After running a command that outputs colours to the terminal, the text in the terminal is inverted.
This doesn't happen in any other terminals I've tried.
Hyper:

cmd.exe:

powershell:

mintty:

I had to reset my CMD colors before the colours in bash on Windows behaved normally. After that I could use a theme for Hyper.
https://www.sevenforums.com/tutorials/401170-command-prompt-restore-default-personalization-settings.html

I'm noticing what I believe is the same issue in Bash on Ubuntu on Windows, using vanilla Hyper.app (default index.js). Here's a screenshot of a directory view:

The directory labels (example: node_modules) are using blue as foreground and green as background. The same blue is used for the path's background color and the green is used as the text color for the files. In the vanilla theme, this isn't unreadable enough for me to notice as unusual at first, but with different themes you can see how significant of an issue it is:
hyperterm-atom-dark

hyper-solarized-dark

In the case of the latter theme, you can see in this related issue what the directory colors are supposed to look like. The background orange is apparently what the git branch status uses.

I agree that this problem is caused by WSL, because on my PC, some other terminal emulators also encountered this.
Integrated Terminal of Intellij Idea:

Integrated Terminal of Visual Studio Code:

@chilblane The directory colours there are a different issue, it is to do with Bash on Ubuntu on Windows setting permissions incorrectly by default.
That can be solved be fixing the umask. See Microsoft/BashOnWindows#352
Of course, you'll have to fix the permissions for each directory already affected, but new ones should be fixed.

Hey there, please try again with the new Hyper v2 release and open a new issue if the problem stills exists. Thank you!

I'm afraid that this issue remains in the 2.0 version.
OS version: Windows 10 Pro Build 16299.371
Expected rendering:

Thanks for reporting back!

Yes, vscode has the same problem.

It is still happening. @Stanzilla did you mark the issue on xterm.js? I couldn't find it, but it would be nice to know the status of this...
Interesting fact: it works like a charm not only on PowerShell but also on plain old cmd.exe. This is indeed a xterm.js only bug 😢

No, I never filed an issue for that :(

this worked for me and it wasn't too complicated, so thought I'd share :) scroll down to the section about dircolors:
https://medium.com/@Andreas_cmj/how-to-setup-a-nice-looking-terminal-with-wsl-in-windows-10-creators-update-2b468ed7c326


Function Call Result Role=Tool Support (Identifier missing)

Is not clear in the documentation but currently there's no documentation providing a way to identify to the model what is the function related to a Author tool response. So the model can iterate back over a function result.
Highlighted in red on this OpenAI Function Calling Flow.

In an OpenAI Request we would normally see a message with an id pointing to what was the function
Tool Call Differencies OpenAI vs Ollama.
// OPENAI
{
      "role": "assistant",
      "content": "",
      "tool_calls": [
       {
          "id": "1", // Identifier (Not present in Ollama response)
          "type": "function", //Type (Not present in Ollama response)
          "function":  {
              "name": "calculate",
              "arguments": "{\n\"expression\": \"3 - 2\"\n}"
       }]
},

// OLLAMA
    "message": {
        "role": "assistant",
        "content": "",
        "tool_calls": [
            {
                "function": {
                    "name": "calculate",
                    "arguments": {
                        "expression": "3 - 2"
                    }
                }
            }
        ]
    },

Tool Response Differencies OpenAI vs Ollama
// OPEN AI
{
      "role": "tool",
      "tool_call_id": "1", // Identifier (Not available as part of OllamaSharp `Message` type
      "content": "Error: Function call processing failed. Error: Function call arguments were invalid JSON."
},

// OLLAMA
??? (No examples or documentation)

I am not sure if you found an issue with my tool implementation or the Ollama API docs.
It seems to be a lack of information in the  Ollama API, at least according to the docs. But if so, what can I do here?

This is something to keep track of, as of now, there's no way to identify which function the Message.Tool refers to and there's no clear documentation on how to do so in Ollama.
So in a scenario where you advertise multiple functions, we can't provide to the model what was the result of which.

Do you know of an issue in the Ollama repository?

Closing this as the issue actually is in the Ollama's API.

https://github.com/ollama/ollama/issues/7510


fix: allow redirect to external url

resolves #770

Codecov Report

Merging #2055 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #2055   +/-   ##
=======================================
  Coverage   95.53%   95.53%
=======================================
  Files          15       15
  Lines         964      964
=======================================
  Hits          921      921
  Misses         43       43

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 ff70623...c63e263. Read the comment docs.

@pi0
About 2.7kb(gzipped)


[Question] Parameter adjustments on training male voice with mb_melgan conf

First of all, let me say I'm really grateful for your work. Really using this repo well and achieved a lot from here. Great work!
So the problem is that while training voices with mb_melgan, I find it hard to train  with male voices rather than female voices in mb-melgan. It shows good quality in female voice while not in male voice.
I trained single male voice (15 hours, librivox meian) in vctk multi_band_melgan.v2.yaml. i converted the sampling rate from 22050Hz to 24KHz and the sampling bit from 32 to 16. In the synthesis step, I transformed the original audio to Mel by espnet STFT first and apply it to the trained vocoder as to avoid any contribution from acoustic model. The sound quality is a bit disppointing as i could hear some howling noises with checkboard pattern in it.
I used default vctk multi_band_melgan.v2.yaml and found out the parameter settings in vctk is as same as those of ljspeech which is female dataset except fft_size, hop_size, and win_length.
I wanna make some parameter adjustments that would fit into male voice dataset.
So I am asking you for advice on this matter.

Do you think those discrepant parameters above would have affected the differing qualities regarding gender?
I feel like I have to somehow touch discriminator and stft loss settings. do you think this is the right way?

Thanks in advance.

Hi @windtoker.
Actually, I have no so much experience with male dataset since there is no good single-male speaker free corpus, but I will try to answer your question.

In the synthesis step, I transformed the original audio to Mel by espnet STFT first and apply it to the trained vocoder as to avoid any contribution from acoustic model.

Do you mean the synthesis with the natural features (analysis-synthesis)?

I used default vctk multi_band_melgan.v2.yaml and found out the parameter settings in vctk is as same as those of ljspeech which is female dataset except fft_size, hop_size, and win_length.

Yes but upsample_scales is also different.

Do you think those discrepant parameters above would have affected the differing qualities regarding gender?

If the f0 of the speaker is very low, it is better to tune fmin (default=80).
And in general, it is better to remove the silence as much as possible, especially in the beginning and the end of utterances. For example, in my VCTK recipe, I use force alignment results to remove the silence.

I feel like I have to somehow touch discriminator and stft loss settings.

lambda_adv will affect the performance. In my experience, small lambda_adv decreases performance.
And I saw some papers using more multi-resolution STFT loss with different window/shift settings (currently 3 types).
And another important setting is optimizer settings.
You can see some discussion in #143.

Thanks for the comment.
Do you mean the synthesis with the natural features (analysis-synthesis)?
-> yes. I guess what you are saying is what I meant.
Considering the proposals with linked issue above, guess I will lower the fmin range to 0 and increase lambda_adv a bit. also gonna fine-tune PQMF filter configuration look for some references regarding multiple stft loss.
thanks again!


ci: Terminate aws instances on abort

It looks like di-tester isn't properly terminating instances when we abort.
The best way to start debugging this would probably be to manually test the kill_aws function in di-tester/bin/di-tester.

https://trello.com/c/Ilqh3xl9/35-ci-terminate-aws-instances-on-abort


Fisheye calibrate failure with specific arguments

Fisheye calibrate() function fails with seg fault when std::vector<cv::Mat> is used instead of std::vector<cv::Vec3d> as OutputArrayOfArrays parameters rvecs, tvecs.

This should work fine with Mat outputs, one row/col per vector.
Using std::vector<Mat> with 3 elements per Mat provides a huge overhead.

There is error in initUndistortRectifyMap function.
In "for" cycles wrong calculating of coordinates. We need to calculate coordinates relative to the center. For me I replaced the line 461:
double x = _x/_w, y = _y/_w;
by this:
double x = fscale*(_x / _w - size.width / 2) / size.width, y = fscale * (_y / _w - size.height / 2) / size.width;
But it work only with "eye" rotation vector. We need to replace this function with correct calculation of coordinates.


Roast depends on Rakudo when it shouldn't

Checking out roast and running perl fudgeandrun $(sed 's/#.*//' spectest.data | grep '\.t') using the system wide installed rakudo master yields the following test failures:
[ 1809s] Test Summary Report
[ 1809s] -------------------
[ 1809s] S03-metaops/misc.t                                       (Wstat: 256 Tests: 0 Failed: 0)
[ 1809s]   Non-zero exit status: 1
[ 1809s]   Parse errors: No plan found in TAP output
[ 1809s] S26-documentation/block-leading-user-format.t            (Wstat: 256 Tests: 0 Failed: 0)
[ 1809s]   Non-zero exit status: 1
[ 1809s]   Parse errors: No plan found in TAP output
[ 1809s] integration/advent2010-day03.t                           (Wstat: 512 Tests: 2 Failed: 2)
[ 1809s]   Failed tests:  1-2
[ 1809s]   Non-zero exit status: 2
[ 1809s]   Parse errors: Bad plan.  You planned 12 tests but ran 2.
[ 1809s] Files=1361, Tests=188669, 1777 wallclock secs (24.89 usr  2.58 sys + 2077.74 cusr 109.43 csys = 2214.64 CPU)
[ 1809s] Result: FAIL

S03-metaops/misc.t and S26-documentation/block-leading-user-format.t fail because they try to load a module called Test::Helpers which seems to be part of rakudo's tests. integration/advent2010-day03.t fails because it expects the files Makefile CREDITS LICENSE README.md which are clearly part of rakudo and not roast.
This is blocking automated spectests for every rakudo commit on the Open Build Service.

S03-metaops/misc.t was easy to fix. Done so in commit c2666ff0c346e150cefc3b108aa7d2930f76e7b4

From Test::Helpers apparently what's used is is-run. Could probably be ported to the file itself, since it's not used from anywhere else. Let me try that.

Reopening because there are still problems with integration/advent2010-day03.t Sorry about that.

This last one is probably redundant, since it's actually testing dir, which is tested elsewhere. Optionally, we can simply test other files.

Test::Helpers is part of roast -- https://github.com/Raku/roast/tree/master/packages/Test-Helpers

On Sonntag, 10. Mai 2020 18:14:45 CEST Nick Logan wrote:

Test::Helpers is part of roast --

Yes and no. Roast contains a Test::Helpers distribution but that's an entirely
different thing from the Test::Helpers module in Rakudo's t/lib

Yes and no. Roast contains a Test::Helpers distribution but that's an entirely
different thing from the Test::Helpers module in Rakudo's t/lib

Ah. Then sounds like they should use https://github.com/Raku/roast/blob/6d23891a3a7e38ebdd0d6ec6b7c79a83a35c7445/packages/Test-Helpers/lib/Test/Util.pm6#L107


[Tooltip] Tooltip as a child takes onClick props from parent by default

What's the problem? 🤔
When using MUI Tooltip as a child of an interactive element, it takes onClick props from parent by default. Is this a bug, or intended behaviour? There is a workaround where you can set e.stopPropagation to componentProps, but it would be better if it takes it as default prop.
What are the requirements? ❓
No response
What are our options? 💡
No response
Proposed solution 🟢
No response
Resources and benchmarks 🔗
No response

Can you share a CodeSandbox that shows how you use it? Normally, the interactive element should be wrapped with Tooltip, not the other way around?

Hello @siriwatknp , sorry for not responding, this task fell behind. I prepared simple codesandbox example. Try to copy text from tooltip, it changes background color of parent component. Is this usage wrong?
https://codesandbox.io/s/react-mui-forked-mebvog?file=/index.js

Hi @siriwatknp , anything new on this issue? It is still open. Thanks

Even I faced the same issue.
<TableCell onClick={handleClick}>
   <Tooltip
      title='My Awesome Title'
      PopperProps={{
         onClick(e) {
            e.stopPropagation();
         }
      }}
   >
      My Awesome Title
   </Tooltip>
</TableCell>

After attaching a click event handler to PopperProps which tell them to stopPropagation() fixed my issue :)

+1.
I just ran into this today; I have a card component that has a descendant tooltip and I noticed that clicking on this tooltip (in order to select its text) was triggering the card's onClick! Easy enough to workaround, just wanted to bump this for visibility.


Guidance on Timeout Redirect URL and Mobile App Login Redirect URL?

This documentation is missing guidance on Workday configuration items. Specifically, the Workday Single-Sign On settings ask for the following two items:

Timeout Redirect URL
Mobile App Login Redirect URL

For integration with Azure AD, what should these URLs be?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 18eabd03-e535-e846-e4ad-c4394f581013
Version Independent ID: 1d98ce13-5bb7-9cb8-c2b7-3c711a7e03e1
Content: Tutorial: Azure Active Directory integration with Workday
Content Source: articles/active-directory/saas-apps/workday-tutorial.md
Service: active-directory
GitHub Login: @jeevansd
Microsoft Alias: jeedes

@cap273 Thanks for your question. We are checking on this and will respond to you soon.

I believe both of these would just be the sing-on/sign-in URL.

I am assigning this issue to the content author to update as appropriate. @jeevansd

I can't get Workday-initiated timeout to actually work with the Timeout Redirect URL being the Sign-on/Sign-in URL. The user is redirected to the sign-on/sign-in URL after the timeout period, but the user is already authenticated and is not prompted to actually re-sign in.

@cap273  For Workday to force the authentication all the time you should enable this in the Workday SAML SSO configuration.

For Mobile URL use the Azure AD sign on URL. That way when users are using the Workday Mobile app then they will be redirected to do the authentication with Azure AD on that endpoint.

#please-close

@jeevansd, thank you for your response. I did already have the option to ForceAuthn Only selected: https://i.imgur.com/pDe3KlN.png
I don't see how that option is related to the missing guidance on what the Timeout Redirect URL should be. Timeout still doesn't work with the Timeout Redirect URL being the Sign-on/Sign-in URL.

Resolution: Always Require IdP Authentication had to be set for the entire Workday, not just for a specific Workday tenant. That functionality (enabling this setting for only a specific Workday tenant) was not working properly. This was confirmed with Workday support.


ModuleNotFoundError: No module named '_tkinter'

Message appears when attempting to start !!!!!!!!!!!!!!!!!
Traceback (most recent call last):
File "/home/dks384/meshtastic_chat_desktop/meshtastic_chat_desktop.py", line 3, in < module >
import tkinter as tk
File "/usr/local/lib/python3.12/tkinter/init.py", line 38, in
import _tkinter # If this fails your Python may not be configured for Tk
^^^^^^^^^^^^^^^
ModuleNotFoundError: No module named '_tkinter'

worksystem
Raspberry 4
Raspberry PI OS 64bit
Python 3.12.4

Hi there,
Thank you for reporting this issue. The error message indicates that the _tkinter module is missing. This module is required for tkinter to work and is often not included by default in some Python installations.
Here are the steps to resolve this issue on a Raspberry Pi running Raspberry Pi OS 64-bit:

Update your package list:
sudo apt update

Install the Tkinter package:
sudo apt install python3-tk

Verify the installation:

Open a Python shell by running python3 in your terminal.
Try to import tkinter with the following command:import tkinter as tk

If there are no errors, the installation was successful.

Re-run your application:
python3 /home/dks384/meshtastic_chat_desktop/meshtastic_chat_desktop.py

These steps should resolve the issue. Please let me know if you encounter any further problems.
Best regards,
Lane


Feat: Add support for pixi package manager

Add support for pixi package manager, uses pixi.toml for parent component. Uses or generates the pixi.lock file to get package information.
Contains Additional attributes contained in pixi.lock file, as there are some dependencies which are different for Operating Systems:
"properties": [
      {"operatingSystem": package['subdir'],},
      {"build_number": package['build_number']},
      {"build": package['build']}
    ]

output:
bom.json

Please also suggest If I should try to fix the above given warning about path being user controlled variable.

Could you kindly check the snapshot tests?

@aryan-rajoria Pillow is being installed fine according to the log, so this is related to something else. Since snapshots are failing for the same reason for different PRs, I believe this is related to a change outside our codebase (like on the github runner). I have updated the snapshot with this missing piece of data.


bcm2710-rpi-cm3.dtb missing from KERNEL_DEVICETREE for raspberrypi3-64 in sumo

The file bcm2710-rpi-cm3.dtb is missing from the KERNEL_DEVICETREE list for the raspberrypi3-64 machine target in sumo branch.
Unable to boot a Raspberry Pi Compute Module 3 using 64-bit poky Linux targeted for raspberrypi3-64 built in sumo branch.
Steps to reproduce the issue:

Build poky Linux for raspberrypi3-64 machine target, at head of sumo
Copy to eMMC on Compute Module 3 and try to boot it

It gets stuck on the rainbow screen, because the bcm2710-rpi-cm3.dtb file was not placed in the boot partition of the image during the build.
The Compute Module 3 should boot fine, continuing past the rainbow screen, and on to the rest of the boot.
The solution is to list bcm2710-rpi-cm3.dtb in the KERNEL_DEVICETREE listing in raspberrypi3-64.conf, in meta-raspberrypi/conf/machine
This problem was observed in the sumo branch.

@stevep-korg This should do it: https://github.com/agherzan/meta-raspberrypi/pull/289

Fixed by #289 .

Thanks very much, Andrei! What is the process for such fixes being merged into the other branches like sumo?  (I work in sumo branch, so I won't currently see this fix there, I don't think.)

Andrei, we may need to enhance this fix soon, or temporarily back it off.  I just discovered this error when it was trying to use that change locally:
|   CHK     scripts/mod/devicetable-offsets.h
| make[3]: *** No rule to make target 'arch/arm64/boot/dts/broadcom/bcm2710-rpi-cm3.dtb'.  Stop.
| arch/arm64/Makefile:133: recipe for target 'broadcom/bcm2710-rpi-cm3.dtb' failed
Perhaps there is also another issue with a missing dts or dts entry and therefore this in not resolving.
Sorry for this complication.  I thought I had tested this and seen that it would build OK before suggesting the change.
Please let me know your thoughts.

This additional problem must stem from the lack of the bcm2710-rpi-cm3.dts in the arch/arm64/boot/dts/broadcom/dts folder.  That's unfortunate.

Hi Andrei,
I don’t know how to do this process, because I’m new to it.
Should I clone the repo in my github account and do some work there?
Sorry, I don’t understand the workflow of how to achieve this.  Is there a link with some directions about how to do this?
(I guess PR is Pull Request.)
Another problem is that the companion dts file is also missing from sumo, so another fix is needed in sumo besides putting the dtb in the KERNEL_DEVICETREE list.
Thanks,
Steve

On Jul 16, 2018, at 4:36 PM, Andrei Gherzan<EMAIL_ADDRESS>wrote:
The usual process is that people who need them would PR the fixes on top of stable branches. So I encourage you to push this fix on top of sumo and PR it here.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/agherzan/meta-raspberrypi/issues/284#issuecomment-405373656, or mute the thread https://github.com/notifications/unsubscribe-auth/ACAgSHsenrReO0F9WR06io9xliRHMCMrks5uHPlIgaJpZM4VL3Vv.


xhamster2.desi

xhamster2.desi

I forgot to add this one:
chat-avenue.com
should I make another request?
you mentioned that reporting the sites to the sources are better than reporting them here. But I don't know which source should I report porn sites to. I would appreciate any suggestions.

Thanks John @John889.  This is OK, I'll just raise the curator from here.
Ping @sinfonietta.

Thanks @sinfonietta.
John @John889 this will be live in the next release, probably later today.
Closing.


Publish 8.0-20230211005621+0000

Publish 8.0-20230211005621+0000

@bot-gradle test and merge

I've triggered a build for you.


[BUG] Refresh browser for the virtualization management page of accessing Harvester would display empty or page not found error

Rancher Server Setup

Rancher version: v2.9.0-alpha5 and v2.9-head (4c5e874)
Installation option (Docker install/Helm Chart): Helm Chart

Harvester Setup

Harvester version: v1.3.1 and v1.2.2

Describe the bug

With Harvester have imported in Rancher
When access Harvester in Rancher virtualization management.

After the Harvester cluster content come up

On Harvester v1.3.1 + Rancher v2.9.5-alpha5

When we try to fresh the page via browser function (or F5), It would display empty page content
https://github.com/rancher/dashboard/assets/29251855/46dee9f4-3a7a-4578-9091-2134e90798a0

On Harvester v1.2.2 + Rancher v2.9-head

When we try to fresh the page via browser function (or F5), It would display Error 404: This page could not be found

To Reproduce

Use helm to provision the Rancher v2.9.0-alpha5

Repo url: https://releases.rancher.com/server-charts/alpha

Create a v1.3.1 Harvester cluster
Import Harvester to Rancher
Access Harvester from Rancher virtualization management page
Refresh the page

Expected Result

Every time refresh the Harvester cluster content page should display all content well.

This is solved when running locally, but not in a production build.
In addition, when not logged in and navigating to a harvester cluster's page, there's a different error
getters.js:24 Uncaught (in promise) Error: Unknown schema for type: management.cattle.io.cluster
    at getters.js:24:13

which ultimately points at a findAll action. That part is reproducible in both local and production builds

@TachunLin how's testing for this looking? 2.9.0's release date is targeted for next week and QA is wrapping up other tickets in the next day or so.

Tests pass on v2.9-dd23578d3e3499f54d80c4ed88cc52c44ee69539-head

Verified fixed on Rancher v2.9-6e9cf233a87796ee9a96e73c096a82101fc9c2ff-head with Harveter v1.3.1 and v1.2.2.
Result

On Rancher v2.9-head with Harvester v1.3.1, after refreshing the virtualization management page, the content can loaded well without empty.

https://github.com/user-attachments/assets/0b794592-bf0b-4592-b86d-106ea8d299ea

On Rancher v2.9-head with Harvester v1.2.2, after refreshing the virtualization management page, the content can loaded well without showing Error 404: This page could not be found

https://github.com/user-attachments/assets/deea3d0e-1d1d-4c57-8b08-008cfb280ef9
Test Information

Test Environment: Single nodes harvester on kvm machine
Harvester version: v1.3.1 and v1.2.2
Rancher version: v2.9-6e9cf233a87796ee9a96e73c096a82101fc9c2ff-head


New border type: Image 🖼

Added the ability to select the fill of the avatar border using a custom image
Thank you for your work ❤

Thank you @PonomareVlad ! A very nice feature.


[NCL-7282] Use absolute imports rather than relative

Hi @DnsZhou , patrik rebased this now and applied some additional changes, can you review it again + test it locally?


attachable property not supported in docker-compose v3 format

@dnephin
Description
it's not possible to set attachable: true in the global network key.
Steps to reproduce the issue:
networks:
  my-network:
    driver: overlay
    attachable: true

Describe the results you received:
attachable Additional property attachable is not allowed
Describe the results you expected:
just creating an attachable network
Additional information you deem important (e.g. issue happens only occasionally):
Output of docker version:
Client:
 Version:      1.13.0-rc7
 API version:  1.25
 Go version:   go1.7.3
 Git commit:   48a9e53
 Built:        Fri Jan 13 21:41:57 2017
 OS/Arch:      darwin/amd64

Server:
 Version:      1.13.0-rc7
 API version:  1.25 (minimum version 1.12)
 Go version:   go1.7.3
 Git commit:   48a9e53
 Built:        Fri Jan 13 06:37:02 2017
 OS/Arch:      linux/amd64
 Experimental: true

Output of docker info:
Containers: 1
 Running: 1
 Paused: 0
 Stopped: 0
Images: 10
Server Version: 1.13.0-rc7
Storage Driver: overlay
 Backing Filesystem: extfs
 Supports d_type: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local local-persist
 Network: bridge host ipvlan macvlan null overlay
Swarm: active
 NodeID: wx2cm0cgduy2r82wgkw0pgayy
 Is Manager: true
 ClusterID: mhdentu8t3yzti0z62r5h1hqh
 Managers: 3
 Nodes: 6
 Orchestration:
  Task History Retention Limit: 5
 Raft:
  Snapshot Interval: 10000
  Number of Old Snapshots to Retain: 0
  Heartbeat Tick: 1
  Election Tick: 3
 Dispatcher:
  Heartbeat Period: 5 seconds
 CA Configuration:
  Expiry Duration: 3 months
 Node Address: <IP_ADDRESS>
 Manager Addresses:
  <IP_ADDRESS>:2377
  <IP_ADDRESS>:2377
  <IP_ADDRESS>:2377
Runtimes: runc
Default Runtime: runc
Init Binary: docker-init
containerd version: 03e5862ec0d8d3b3f750e19fca3ee367e13c090e
runc version: 2f7393a47307a16f8cee44a37b262e8b81021e3e
init version: 949e6fa
Kernel Version: 4.8.14-docker-1
Operating System: Debian GNU/Linux 8 (jessie)
OSType: linux
Architecture: x86_64
CPUs: 4
Total Memory: 3.857 GiB
Name: d0001.ams.tc.xxxxxx.com
ID: I2Z7:LP7H:TNVC:WZ7A:3O6B:OKSD:XET5:G62C:YHRE:JQYZ:OZ3F:YPUU
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): false
Username: raarts
Registry: https://index.docker.io/v1/
Labels:
 provider=generic
Experimental: true
Insecure Registries:
 <IP_ADDRESS>/8
Live Restore Enabled: false

Additional environment details (AWS, VirtualBox, physical, etc.):

/cc @dnephin @vdemeester

Yeah... This would be pretty nice especially for local development...

Hum thinking of docker/compose, should this be just ignored (maybe a warning then) when using docker-compose (which doesn't talk to the swarm) ?

Looks like the bridge driver already ignores the option;
$ docker network create --attachable -d bridge foobar
6a552771d516ae246376ec8ba15846f0a05e879f8e5a6d730ddccca698e25571

$ docker network inspect foobar

[
    {
        "Name": "foobar",
        "Id": "6a552771d516ae246376ec8ba15846f0a05e879f8e5a6d730ddccca698e25571",
        "Created": "2017-01-20T14:25:26.398470731Z",
        "Scope": "local",
        "Driver": "bridge",
        "EnableIPv6": false,
        "IPAM": {
            "Driver": "default",
            "Options": {},
            "Config": [
                {
                    "Subnet": "<IP_ADDRESS>/16",
                    "Gateway": "<IP_ADDRESS>"
                }
            ]
        },
        "Internal": false,
        "Attachable": true,
        "Containers": {},
        "Options": {},
        "Labels": {}
    }
]

Looking at the above;

wondering; is attachable an option of the overlay driver (should it actually be --opt attachable?
should it not actually set the attachable option? If the bridge driver does not support attachable it should not actually show up in the network specification?

ping @mavenugo @aboch ^^

@vdemeester In docker-compose this field is always enabled, as of version Compose format v2.1 (https://github.com/docker/compose/pull/4232).

@thaJeztah it is not a driver specific option. It is a swarm-mode option to allow both services and unmanaged containers to run in the same network. As of 1.13.0, overlay driver is the only supported driver in swarm-mode. But with 1.13.1, we are trying to bring-in plugin support in swarm-mode and this --attachable option will be used by networks using other plugins as well.

Would love to see this. Being able to docker-compose run using the service definitions in our V3 docker-compose.yml file would make it much easier to run one-offs such as crons, database migration scripts, etc.

+1.
I am also having issues dealing with one-off  containers in V3 compose-managed service stacks and swarm overlay networks. After running:
$ docker stack deploy -c docker-compose.yml my_service

The only way I have been able to connect a one-off container into a service stack is to create another service attached into the my_service network with a sleep command which seems goofy:
$docker service create --name temp_service  --network my_service_default centos:centos7 sleep 5000
$docker exec -it temp_service.1.qk16sb2xgvpygqkgy03d0k03l bash

Guys, any updates about this issue?

This was added in #30742. It will be in Compose file v3.2 format in the next release.


Improve examples pages on mobile devices a bit

Only show navigation when there is enough space.
This does not make the example page look perfect on narrow screens, but at least it makes the example navigation and the examples themselves usable again.

Before:

After:

Looks like an improvement.  Please merge.


fix pandas to <= 2

unit test and e2e work w/ pandas 2.0.3
@clampr  why was <2 introduced?

@clampr  can you take a look at the test, I can't repro the error.

Brought back Pandas v2 support in v1.6.7. Closing this PR. Thanks @eisber :)


Debugging coverage performance?

I'm not sure if an issue is the right way to ask for help, if it's not I apologize, but I couldn't find anyplace better.
I'm a maintainer on the Cesium project and we've recently switched our 200,000 line JavaScript code base over to use karma (over 7200 jasmine unit tests). The entire test suite runs in less than 40 seconds.
I've used istanbul in my other (smaller) projects, and love it.  That being said, we are now trying to switch over code coverage in Cesium from an ancient version of JSCoverage to istanbul via karma-coverage and running into a major performance hurdle.  Our old system takes about 90 seconds total to instrument and cover.  However, running the same tests via istanbul takes over 7 minutes (and only if I comment a couple of tests that were taking upwards of 5 minutes each).  I also had to jack up browserNoActivityTimeout and browserDisconnectTimeout in karma just to try and get it to run to completion (and that has only happened once so far and took 10 minutes, the results were not repeatable.
I can't say for certain, but it seems earlier tests run way faster and things just get slower as the tests go on.  A few seemingly innocuous tests seem to take multiple minutes to complete on their own (but execute in seconds when not run under coverage).  I'm not sure it's any particular test causing issues though.
I've tried multiple launchers and browsers in case that was the problem (we usually use Chrome by default), but all browsers exhibited the same performance issue.
It happens on both Windows and Linux.
While I don't expect anyone to magically solve my problem, I am hoping that someone can provide tips for tracking down whatever the root cause of the problem may be.  Perhaps there are some options I can tune or extra logging commands I can run to figure out what is taking so long.  I'd be more than happy to submit a pull request with performance improvements if it turns out that is the problem (vs some bug slowing things down)
Here's the relevant configuration we are using:
https://github.com/AnalyticalGraphicsInc/cesium/blob/coverage/Specs/karma-main.js
https://github.com/AnalyticalGraphicsInc/cesium/blob/coverage/Specs/karma.conf.js
https://github.com/AnalyticalGraphicsInc/cesium/blob/coverage/gulpfile.js#L281
Any help would be greatly appreciated, and thanks again for a great project.

@mramato I just ran into a performance problem, and I'm hoping my findings can help you.
Istanbul uses https://github.com/mklabs/node-fileset at startup, and inputs **/*.js as the include parameter by default. It also inputs **/node_modules/** as the exclude parameter by default.
In theory this seems rational. But what it actually does is first execute the include using the node-glob module, and then perform a filter using the exclude param.
Which means that at startup, even though node_modules is listed as an exclude, it still traverses the entire folder.
On our codebase at work, the time it takes to do so is quite extreme (these are the defaults istanbul uses https://github.com/istanbuljs/istanbul-api/blob/929abf6922e21eebcea694e9f8b0058fbfb207d7/lib/file-matcher.js#L25-L32):
$ time node -e 'require("fileset")("**/*.js", "**/node_modules/**", function () {})'

real	0m9.743s
user	0m7.807s
sys	0m2.651s

Removing the **/*.js and being just a bit more specific greatly helps:
$ time node -e 'require("fileset")("src/**/*.js", "**/node_modules/**", function () {})'

real	0m0.654s
user	0m0.529s
sys	0m0.192s

So I fixed this by changing my istanbul invocation:
instead of: istanbul cover _mocha
do: istanbul cover -i 'src/**/*.js' _mocha
Execution time dropped in half!

Thanks for the suggestion @kirbysayshi, unfortunately your issue doesn't appear to be the source of my problem.  I'm not running the cover on the command line, but instead running programatically through karma.  I was also already specifying my own include filter.  I spent some time debugging and tweaking the code (I even hardcoded some changes in file-matcher) but nothing in that area had any affect.  Thanks anyway.

Sorry to hear that @mramato ! Did you try hard coding a list of files for file-matcher to return? Just to remove any chance of this issue being the source of your performance woes?

Did you try hard coding a list of files for file-matcher to return?

Yes,I went into the node_modules directory and hardcoded values, but as far as I could tell running with karma doesn't even take this code path (because it uses the programmatic API to istanbul and file-matcher.js only appears to get called when using the command line (as far as I can tell).
Thanks again anyway.


complient requirements as per #33

I have updated the master branch since the dev branch requirement are very different.

This commit has been added to the following release
https://github.com/codejamninja/sphinx-markdown-builder/releases/tag/0.5.3


Docs say<EMAIL_ADDRESS>but should say<EMAIL_ADDRESS>

As per the issue title.
The examples in the documentation are as follows:
# Compliance
- name: Advance Security Compliance Action
  uses<EMAIL_ADDRESS>
Should be:
# Compliance
- name: Advance Security Compliance Action
  uses<EMAIL_ADDRESS>
Its a tiny bit confusing.

@CallumHibbert you are absolutely right and thanks for spotting my mistake. PR coming shortly

@CallumHibbert Done and merged 🚀


Cookie warning

closes #21
I added a cookie warning using bootstrap to the best of my ability. I added one new CSS rule to the main.css file and used Vanilla JS in the base.html file. I'm not familiar with Russian or Tartar so I used Google Translate (Google Translate sometimes miss translates in my experience, this doesn't happen often but it's worth mentioning).
Another thing worth mentioning is that after running the makemigrations command a new migration was created. I didn't edit any models so I'm assuming this was just an edit that was never migrated.
New CSS
.cookie-allow-tag{
    bottom: 1em;
    max-width: 500px;
    width: 90%;
    z-index: 1;
}

New HTML

        {% translate 'CookieWarning' %}
        <button class="btn btn-light mt-1" onclick='localStorage.setItem("cookie-allow-tag", "true"); this.parentNode.remove();'>{% translate 'CookieWarningAccept' %}</button>

Hello there, really appreciate your PR.
I added a few comments for translation strings, everything else looks fine. You also reminded me how important to check on makemigrations . If you make that changes, I'll be happy to merge PR.

Thanks for your feedback, I expected some translation issues. I won't be able to make any fixes till the end of the week but, I'll get them fixed as soon as I can.


Helper function to parseADA to Lovelace

Discussed in https://github.com/input-output-hk/marlowe-ts-sdk/discussions/143

Originally posted by nstanford5 December 21, 2023
Most DApps will present users with the ability to input token amounts to a UI component. Users typically think in terms of ADA amounts, but we need to send Lovelace amounts to the blockchain.
Should the SDK include a simple helper function to parseADA to Lovelace?
Something like..
const parseADA = (num: number) => { return num * 1000000; };
On the other side of that, when querying the blockchain for amounts -- it will return Lovelace that needs to be formatted to ADA to present to users.
Should we also include a formatLovelace function?

As I responded in the discussion, I believe this should be part of a bigger thing that we can call TemplateParameters/ContractScheme/etc that is similar to the Marlowe Extended feature.
This could allow us to define a DSL for contract parameters with assertions (amount > x, deadline 1 > deadline 2, etc), with a human description (that can help us with Marlowe to english explainer) and with a proper serialization mechanism so we can simplify this type of code.
The proposed function parseADA is misleading as it is not parsing anything, it could be called lovelaceToAda, but multiplying a number for 1 Million is trivial and does not belong in the SDK IMO.


timeouts silently ignored

What happened?
When trying to convert a resource with timeouts, those get silently dropped. This seems expected given this comment here: https://github.com/pulumi/pulumi-converter-terraform/blob/1a9e315cfd7fed08e19c815ab68859ee331a0b65/pkg/convert/tf.go#L1736
This seems to cause: https://github.com/pulumi/pulumi-aws/issues/4391
I found this similar existing issue https://github.com/pulumi/pulumi-converter-terraform/issues/104, but the difference here is that we're not getting an error.
Example
resource "aws_security_group" "example" {
  name = "izizavle"

  timeouts {
    delete = "2m"
  }
}

Output of pulumi about
n/a
Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

I believe we need to add support for timeouts to PCL and update programgen to be able to emit it. Then we can make the converter set it.


The document lacks detail description

It's a amazing dock component and I can hardly find any other react-based dock components in the entire community, but it lacks the detailed document,  for example, what's the possible value of the property "dockStyle" ?

Solved. Use react-split-pane to replace it.


fun command

make a command that can display images from url
but not normal displaying of images
i want it to scale the image to any desired width, and then process each pixel of the image, and be able to print said image to console by using 1 coloured shaded box unicode character (▓▒░) per pixel
it's gonna be difficult, but i mean like,,,, once i get it to work it'll be cool

hahahhahaahaha basically impossible


API Documentation

I'm writing an API documentation generator. It's currently using NuDoq to parse the assembly.
What would be the best way to inject this into Static.Web processing? It will be generating content, metadata, and possibly more.
There are no input files needed. It will just need to read some config values, like where to find the assembly it's generating the docs from.

Cool! Just FYI, Statiq will eventually have in-the-box API documentation support via Statiq Docs but I'm all about keeping stuff open so lots of different techniques can be used as needed (just wanted to make sure you knew it was coming).
My first instinct would be to add the NuDoq code to a module and then add that module to a new pipeline (let's call it Api). Then add the Api pipeline as a dependency of the Input pipeline:
.ModifyPipeline(nameof(Pipelines.Inputs), x => x.Dependencies.Add(nameof(Api)))

You could have the module generate a document-per-symbol, or maybe a document-per-class. Then, since it's a dependency of the Inputs pipeline you can iterate over those documents in other pages and layouts or create archives from them.
Does that make sense for a starting point?


If used "output-directory = " to a different spot extracted subtitles doesnt get transfered back

In the "config.ini" under "[Converter]" "output-directory = ".
If i added a location for it to convert in, once it completes and move it back to the correct spot, It leaves the extracted subtitles behind in the output-directory location.

Please include logs and use the issue template or its very difficult to assess your problem

6abafed8c137631b68ed1f37e36e06b33210f7d7
Got lucky this time, see if that fixes it but in the future please include relevant info

Another great work indeed.
Question: since it is using one output folder and not creating subfolders. If Radarr and Sonarr both run at the sametime to recode would that mess it up?

Shouldn't make a difference. It's not changing the folder or operating on any wildcards it's still referencing the specific files

Just noticed that when subtitles are imported back it loses the "forced" and also adds the "1" and "2".
Before:
Chupa (2023) {imdb-tt14923260} [WEBDL-1080p][AC3 5.1][x264]-PEPE.en.forced.srt
Chupa (2023) {imdb-tt14923260} [WEBDL-1080p][AC3 5.1][x264]-PEPE.en.srt
After:
Chupa (2023) {imdb-tt14923260} [WEBDL-1080p][AC3 5.1][x264]-PEPE.1.en.srt
Chupa (2023) {imdb-tt14923260} [WEBDL-1080p][AC3 5.1][x264]-PEPE.2.en.srt

Is there a way to add ".mvk.orginial" to the orginial file even when the converting "out-directory" is in a different folder? That way Plex Schedule wont scan the file while converting.


Protect against divide by zero when SELECTing

Can we somehow just use AVG here? This SQL feels overly complex for what must be a pretty common operation.
grep for '* 100' probably.
Seems like MySQL 5.7 might be more strict here by default instead of giving NULL.

a62d7a6 fixes these. I think I got them all. I'm using 5.7 on local now so should find any obscure issues as I go.


Support Rails 5.1.x

All of the currently released Solidus versions still support Rails 5.1.x, and I think it's important to try to continue to support it as long as possible in this extension in particular.
Many Solidus extensions rely on solidus_support v 0.5 and greater, and at the moment our store cannot get any of the latest commits from solidus_tax_cloud or solidus_handling_fees or solidus_stripe or solidus_related_products or . . . you get the idea . . . until we transition our application to Rails 5.2. (This is a huge undertaking, because it means somehow getting away from the deprecated solidus_active_shipping!)
Because this extension is so widely used (indeed that's the whole idea!), I believe it is important to have compatibility requirements that are as loose as possible, otherwise any requirements here will constrain the entire ecosystem of a user's Solidus extensions.
I'm not familiar with what if anything changed in activesupport from v5.1 to 5.2, but all of the specs are green and, as I say, all current versions of Solidus support 5.1 so I think we should endeavor to continue to do so until there is a convincing reason otherwise!

@brchristian thanks for the PR! You're raising some valid concerns here.
I was thinking it may be an even better idea to remove the version constraints on activesupport entirely. Instead, let's rely on solidus_core etc. specifying the required Rails version.
What do you think?

@aldesantis I support this idea! I think that makes a lot of sense. And of course individual extensions can have constraints on the Rails version, too, if they make use of version-specific Rails features.

Hi @aldesantis I’ve gone ahead and done exactly as you suggested, removing the constraint on activesupport entirely. What do you think?
One thing to note is that ideally there might be some way to backport this to an 0.5.x release! Otherwise I’m going to have to get each and every one of the extensions we use up to supporting 0.6.x, which is doable but will take probably a half dozen PRs across different repos.

@jarednorman great idea, added it!

@aldesantis @jarednorman Anything I can do to help this along? We’ve been running this in our own fork in production for a few months now and would love to rejoin the party on the master branch!

@brchristian sorry, somehow I missed the notification for this one! LGTM, @jarednorman what do you think?

Terrific, thanks all!


onLoadFinished arguments wrong with SlimerJS

SlimerJS passes 3 arguments to the onLoadFinished callback, while PhantomJS only passes 1.
If one registers an onLoadFinished callback taking 1 argument and is using SlimerJS, the callback will be called with an array of the 3 arguments as the first argument.
I believe the problem stems from this section. It appears this issue will happen whenever the callback has one argument but is called with multiple arguments.
This is related to johntitus/node-horseman#180

Suggestions? No idea why that magic with wrap/unwrap arrays was done, looks suspicious. It existed in v1.0 and we did not changed anything there. I have no principal objection to bump version to 3.0 and remove all legacy mess.
@baudehlo may be you remember something about that code :) ?

My suggestion would be to always use .apply rather than .call, like you said. I also think it looks like it is doing something weird...

I have no recollection of anything - sorry - I haven't worked on this code
in years.
On Thu, May 12, 2016 at 10:43 AM, Alex Layton<EMAIL_ADDRESS>wrote:

My suggestion would be to always use .apply rather than .call, like you
said. I also think it looks like it is doing something weird...
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/baudehlo/node-phantom-simple/issues/133#issuecomment-218779300

@awlayton could you do a PR with test?

I am working on a paper right now. I might be able to get to it this weekend, maybe.

I tried to make a test, but now I can't get the bug to happen @puzrin. What SlimerJS were you using @winghouchan?

I'm using SlimerJS 0.10.0 @awlayton.

I have a feeling that the problem caused by using .call rather than .apply in this situation is a lot more expansive. This is coming from a case I just witnessed with a callback on the consoleMessage event. When running the callback in PhantomJS, 3 arguments are expected:
const webPage = require('webpage');
const page = webPage.create();

page.onConsoleMessage = (msg, lineNum, sourceId) => {
  console.log(`CONSOLE: ${msg} (from line #${lineNum} in ${sourceId})`);
};

However a consoleMessage callback in SlimerJS gets passed the first argument with an array of elements corresponding to each expected argument.
Perhaps more investigation is required to see how far this issue expands exactly.

Yes the problem potentially affects all callbacks @winghouchan. Perhaps you have more time to make a test and fix for this issue?

I'm certainly open to putting a PR together for this. However my time is also limited so it won't be something I'm prioritising right now.

Can we except a merge ?


Add some bits to the readme

The readme is rather sparse, let's go ahead and update it a bit.
------------------------ >8 ------------------------
Do not modify or remove the line above.
Everything below it will be ignored.
Requesting a pull to mattr-:master from mattr-:updates
Write a message for this pull request. The first block
of text is the title and the rest is the description.
Changes:
3c8e93c (Matt Rogers, 29 minutes ago)
more testing
f4ae45e (Matt Rogers, 6 days ago)
add readme

this is better, but not quite there yet.


For mercycorps/TolaActivity#2457, fix modalized PNotify in result form.

Yep, that's the goal.

Yep, that's the goal.