Include an estimated time remaining in the console output?

Loving this but I was wondering how long it would take to finish running this script when I was (still am!) using it.
Should be easy to calculate the remaining time using the time values given from the .ini file.
I had to do the math myself and ended up with about 31 hours total but it'd be nice to see the progress remaining and maybe a % complete!
Just a suggestion 😊

Hi, thanks for the suggestion, I added it with the last commit:
https://github.com/rubennp91/mcloady/commit/ca4380ea89da5a5728895faebfc85179fd8d5953


Update documentation

Describe changes
In this pull request, I made the following changes to the "fetch-runs-after-execution.md" file:

Corrected a command that was not working as expected.
Changed the command "zenml pipelines list" to the correct command "zenml pipeline list."

These changes address an issue where the incorrect command was causing confusion and preventing users from listing pipelines properly.
Pre-requisites
Please ensure you have done the following:

[x] I have read the CONTRIBUTING.md document.
[x] I have updated the documentation to reflect the corrected command.
[x] I have added tests to cover my changes.
[x] I have based my new branch on develop and the open PR is targeting develop.

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Other (add details above)

@strickvl , I have completed the pull request based on your recommendations. Could you please take a moment to review it? Your feedback and insights would be greatly appreciated.

@VishalKumar-S Thanks for this catch!


Implement Singly Linked List

We need to implement the singly linked list using ruby.

hi @ShehrozIrfan . I would like to contribute to this

@navaneethkp36
Assigned.

Please assign this to me

@akash-1107
Assigned.

@ShehrozIrfan un-assigning myself since I wont be able to complete this due to other tasks.

@ShehrozIrfan I would like to work on that, please assign me this task

@kunaldhongade
Assigned.


NotImplementedError: Elliptic cone force is not implemented yet.

Intro
Hi!
I am trying to use MuJoCo-MJX with CMUHumanoid from dm-control. However I'm getting the following error.
NotImplementedError: Elliptic cone force is not implemented yet.

However in the documentation of version 3.2.0 it is mentioned that
Added support for [elliptic friction cones](https://mujoco.readthedocs.io/en/stable/XMLreference.html#option-cone).
My setup
Mujoco versions

mujoco                       3.2.3
mujoco-mjx                   3.2.3

Python and OS version

python 3.11.8
ubuntu 22.04

Minimal model for reproduction
minimal XML link: https://github.com/google-deepmind/dm_control/blob/main/dm_control/suite/humanoid_CMU.xml
Code required for reproduction
import mujoco
from mujoco import mjx

mj_model = mujoco.MjModel.from_xml_path(xml_path)
mj_data = mujoco.MjData(mj_model)
print(mj_data.qpos, type(mj_data.qpos))

mjx_model = mjx.put_model(mj_model) # error here
mjx_data = mjx.put_data(mj_model, mj_data)

print(mjx_data.qpos, type(mjx_data.qpos), mjx_data.qpos.devices())

mjx_data = mjx.forward(mjx_model, mjx_data)

Confirmations

[X] I searched the latest documentation thoroughly before posting.
[x] I searched previous Issues and Discussions, I am certain this has not been raised before.

Hi @kinalmehta, I am getting the same error for a different XML model. Were you able to resolve this?

I switched to using pyramidal cone till this is fixed
<option cone="pyramidal"/>


boot test defconfig+sanitizers

we should boot test asan, ubsan, and friends

WIP: https://github.com/nathanchance/continuous-integration2/commit/e9416649764dee6502218dddf2f090334a28b037
KASAN KUnit tests are failing for at least x86_64 but that is not clang specific: https://lore.kernel.org/r/YQXMHnWRsmfzKK00@archlinux-ax161/
Certain arm64 builds with KASAN panic with ToT LLVM but my module_{c,d}tor linker script patch fixes that, leading me to believe that the sections might be getting placed too far away without that change.

https://github.com/ClangBuiltLinux/linux/issues/1432 is fixed (mainline+)


Microsoft.Sql is missing azureresourceschema configuration for API version 2020-11-01-preview

The following file is missing configuration for API version 2020-11-01-preview:
https://github.com/Azure/azure-rest-api-specs/blob/master/specification/sql/resource-manager/readme.azureresourceschema.md
It has been defined here: https://github.com/Azure/azure-rest-api-specs/blob/master/specification/sql/resource-manager/readme.md

Resolved with #13862

Thanks @leni-msft!


Validate the syntax of the bundle reference.

See #327 .
The CLI—or even better the DSS—should validate the syntax of the bundle reference (UUID and version). The status code returned by the DSS should not be 404 but rather 400 due to invalid syntax of bundle UUID and version. It'd be preferred that the DSS do that, rather than baking in an assumption of what constitutes a valid version into the CLI.

Issue moved to HumanCellAtlas/data-store #2103 via ZenHub


Check Images and Links in specification documents

Why is this needed:
We recently moved to this repository and did a little restructuring of documents and their locations.
It would be great if we can please check all the images and links in spec documents to make sure they are still valid and work.
Thanks!

Links checked using https://github.com/tcort/markdown-link-check

@manuelstein yeah i added that after opening this issue :) feel free to close this one if you wish.

I haven't checked images. The tool I used doesn't do embedded HTML, so I was thinking of using markdown style images. Since that wouldn't allow scaling (heights in use cases), I was thinking of resizing images, so we can have pure markdown. wdyt?


Added the Authentication Functionality

I added the authentication functionality like signup, login and logit with proper pop-up messages and handled the database in backend

@PSS2134 Kindly check and review it, merge the PR, It took 1 week to add authentication

@ankitgupta143 Nice job, share a demonstration video of your changes (like how it works)

@ankitgupta143  resolve  conflicts

@HimanshuNarware @PSS2134 I made a PR request, please check it.

@ankitgupta143 Nice job, But do one thing again raise PR , and for now I am closing this PR

@HimanshuNarware @PSS2134 I again request you the PR request, please check it


Fixed: Missed variables in the TranslationRepository class

Some variables were maybe left from previous sources...

thanks!


You can't set barTintColor on NavigatoriOS

I see that in the code that you can set the barTintColor property on NavigatoriOS, but the property isnt actually given to the Objc code.
Is there a another way to set the background of the navigator or is this a bug?

This is most probably a bug. We're not using the navigatorios in any of our real apps so it likely has bugs/unimplemented features

@vjeux out of curiosity, what do you guys use for the navigation?

The Navigation component. It supports left/right navigation with swipe to go back and custom header (we implemented breadcrumbs instead of the standard ios one). Also supports coming from the bottom and swiping down from the top to dismiss.
It is written purely in js and reacf

I would love to see the example of implementing an app using Navigation component. NavigationIOS has so many limitations. :sob:

@igodzp https://github.com/facebook/react-native/blob/master/Examples/UIExplorer/Navigator/NavigatorExample.js

@hyusetiawan Thank you so much! Can't believe I missed that one haha

This has been fixed!


can i change Alt key to FN key?

Hi,
Is it possible to change the ALT key to Function key (FN). so instead of ALT-1 I can use FN-1 (not F1).
The main reason is that I use ALT-left and Alt-right arrow to move between different panes in Windows Terminal.

I don't believe FN is a "real" key which is detectable by the OS, but instead a pseudo-key used by the keyboard FW to modify the key-press event sent to the OS, As such, this change is probably not doable.
Someone correct me if I'm wrong :)

Hi, I ended up changing the Windows Terminal keybindings. Instead of FN key I used CTRL-H and CTRL-L to move the courser left and right between panes.
I don't like that Workspacer is forcing to use arrow keys to move between spaces. . (as far as i know)


Kubernetes Objects: Updated links to docs resources

This change is

@nkuba Signing the CLA is a must for the PR to be merged.

@tengqm CLA status has updated to signed.

/lgtm

/assign @steveperry-53

/approve


Exclusive access to the camera

I am trying to implement the ZXing barcode library into my Xamarin.Forms project.
Because I am having a few problems there I tried the android demo project from the library: Sample.Android.
This one seems to work well but it crashes when putting the app into background and getting it back into foreground with the following exception:
[MonoDroid] UNHANDLED EXCEPTION:
[MonoDroid] System.Exception: Couldn't get exclusive access to the camera
[MonoDroid] at ZXing.Mobile.ZXingSurfaceView.GetExclusiveAccess () <IL 0x00033, 0x00150>
[MonoDroid] at ZXing.Mobile.ZXingSurfaceView.StartScanning     (ZXing.Mobile.MobileBarcodeScanningOptions,System.Action`1<ZXing.Result>) <IL 0x00046, 0x0028b>
[MonoDroid] at ZXing.Mobile.ZXingScannerFragment.scan () <IL 0x00012, 0x0007f>
[MonoDroid] at ZXing.Mobile.ZXingScannerFragment.OnResume () <IL 0x000fd, 0x00617>
[MonoDroid] at Android.Support.V4.App.Fragment.n_OnResume (intptr,intptr) <IL 0x0000a, 0x0009f>
[MonoDroid] at (wrapper dynamic-method) object.2ec0fbbb-3a10-48d4-b1ab-3fe72444ec91 (intptr,intptr) <IL 0x00011, 0x0003b>

Does anybody know how to fix this problem?

Also having this problem, would love a release for this fix! Thanks so much, I love the package.

Hi,
I have a couple of users that experience this error on a regular basis.
Do you know where it comes from ? Just to know if I can add a check to avoid the application to crash.
Thanks !

friend there any way to do direct on the library

This should be fixed by #205
Please reopen if it is still an issue after this patch.

Awesome, you're the best !

good morning, I'm not very good with someone could publish libraries and DLL already adjusted mistake and thank you

https://www.nuget.org/packages/ZXing.Net.Mobile/1.5.3 is available on nuget.

Hey, we are seeing this pop up now from time to time. Our app is a Xamarin Forms/PCL project with ZXingNetMobile <IP_ADDRESS>. It is intermittent but happening regularly. Any suggestions? I do not know how to reopen this thread (or whether it should be reopened and a new one entered.) Thanks!

This looks like StartScanning was called multiple times before the first scan is finished... Maybe evaluate your code to ensure that cannot happen?

I am assuming what you are seeing is the call stack. In this case, we are using a custom overlay and  ScanContinuously. The above call is a result of that call. I put a breakpoint on the _ScanContinuously _and it is not being called more than once. Is that what you mean? Or is there something external we may be calling?

I've seen this happen rarely with the version <IP_ADDRESS> from NuGet on some Android devices.  The stack trace is the same as moflaherty shows.  I'm simply adjusting the TopText & BottomText and calling Scan() from a MvvmCross ViewModel.

I still get a (this?) crash when I open the scanner, then go to HOME and then back to app.

I'm facing the same issue as xillie and moflaherty using scanner with CustomerOverlay.

I still get a (this?) crash when I open the scanner, then go to HOME and then back to app.

I'm currently using version 2.1.0-beta1 from NuGet (Xamarin.Android), but had this problem also with earlier stable versions. Any further progress on this issue?

System.Exception
Couldn't get exclusive access to the camera
at ZXing.Mobile.ZXingSurfaceView.GetExclusiveAccess () <0x8e9e3880 + 0x000c4>
at ZXing.Mobile.ZXingSurfaceView.OnSurfaceCreated () <0x8e9e2438 + 0x000d7>
at ZXing.Mobile.ZXingSurfaceView.SurfaceCreated (ISurfaceHolder holder)

@lukegore PR #325 should fix this, you can try it

Thank you! works like a charm!


In Annotation search, Ontology (aspect) filter only has one letter for aspect

It would be better if it had the whole text for the aspect, it would be much clearer if the full text was there, ie. Biological Process rather than P.
Mostly an issue for us downstream with the additional aspects we host.  Hard to know which letter corresponds to which aspect.

amigo and solr don't really care what's in there--it's all just strings. This is coming from one of the columns of the GAFs, as reproduced with the getAspect() function in owltools.
I'm not sure what the consequences would be in changing that, but I imagine minor to none.

Tagging @cmungall and @hdietze in case there is some use I'm not thinking of here.

Not sure if PAINT uses it, but otherwise I think it would be better to
expand this.
The wider context for Justin's request is I suspect other amigo
installations will want a wider range of ontologies here. We should
check the loader behavior - does it just pass through the obo namespace
is it's not in the hardcoded GO mapping?
Ultimately people will want to use the relationship type rather than
aspect here, it's more meaningful.
http://www.geneontology.org/specifications/gpad/gpad-1.html#default-annotation-relations
On 10 Nov 2015, at 17:31, kltm wrote:

Tagging @cmungall and @hdietze in case there is some use I'm not
thinking of here.

Reply to this email directly or view it on GitHub:
https://github.com/geneontology/amigo/issues/279#issuecomment-155625784

Talked with @elserj , will probably just add additional ontology information to the schema and the loader. See ont-config.yaml.

The fix will entail adding a new field (or fields) to the schema, in addition to a new load line to the owltools loader (GafDocumentLoader ?).


Zoom level is changing after printing

Attach (recommended) or Link to PDF file here:
Configuration:

Web browser and its version: Firefox nightly
Operating system and its version: Mac OS 12.5

Steps to reproduce the problem:

Open a pdf and set the zoom level to auto
click on the print button
dismiss the print dialog
the pdf is unzoomed (around 80%)

Before printing:

After printing:

This affect both the Firefox PDF Viewer as well as the generic viewer (i.e. https://github.com/mozilla/pdf.js#online-demo), and according to mozregression it's caused by the following commit:
2022-08-16T12:38:14.836000: DEBUG : Found commit message:
Bug 774398 - Hack to make print and viewport media query listeners fire when printing. r=geckoview-reviewers,owlish,dholbert

This still doesn't fire on print settings changes, so it uses the
default page size. Which is probably better than nothing, but...

To make viewport-size media-query listeners work more generally for
printed documents, we would need to re-clone the top document
unconditionally for all print settings changes, which needs front-end
work at least, and is dubious if the page changes dynamically.

Differential Revision: https://phabricator.services.mozilla.com/D150499

2022-08-16T12:38:14.836000: DEBUG : Did not find a branch, checking all integration branches
2022-08-16T12:38:14.840000: INFO : The bisection is done.
2022-08-16T12:38:14.868000: INFO : Stopped

I get this exception in the console:
offsetParent is not set -- cannot scroll [viewer.js:2958:13](resource://pdf.js/web/viewer.js)
    scrollIntoView resource://pdf.js/web/viewer.js:2958
    #scrollIntoView resource://pdf.js/web/viewer.js:10430
    scrollPageIntoView resource://pdf.js/web/viewer.js:10693
    _setScaleUpdatePages resource://pdf.js/web/viewer.js:10475
    _setScale resource://pdf.js/web/viewer.js:10556
    set currentScaleValue resource://pdf.js/web/viewer.js:9914
    webViewerResize resource://pdf.js/web/viewer.js:2268
    dispatch resource://pdf.js/web/viewer.js:3543
    windowResize resource://pdf.js/web/viewer.js:1954
    triggerPrinting resource://pdf.js/web/viewer.js:1849
    webViewerPrint resource://pdf.js/web/viewer.js:2303
    dispatch resource://pdf.js/web/viewer.js:3543
    bindListeners resource://pdf.js/web/viewer.js:13911
    (Asynchrone : EventListener.handleEvent)
    #bindListeners resource://pdf.js/web/viewer.js:13899
    Toolbar resource://pdf.js/web/viewer.js:13851
    _initializeViewerComponents resource://pdf.js/web/viewer.js:751
    initialize resource://pdf.js/web/viewer.js:494
    run resource://pdf.js/web/viewer.js:790
    webViewerLoad resource://pdf.js/web/viewer.js:15275
    (Asynchrone : EventListener.handleEvent)
    <anonyme> resource://pdf.js/web/viewer.js:15283
    <anonyme> resource://pdf.js/web/viewer.js:15285
    <anonyme> resource://pdf.js/web/viewer.js:15287

The problem is that "resize" event, on the original window, is now triggered during printing which is wasn't before https://bugzilla.mozilla.org/show_bug.cgi?id=774398 landed.
We could easily hack-around that for now, which given the dates in https://wiki.mozilla.org/Release_Management/Calendar probably makes sense to do here.


[FEATURE] Add port for prometheus monitoring to mount pod definition

What feature you'd like to add:
Add port for prometheus monitoring to juicefs-mount pod definition(s)
Would just require a small addition for the containers spec...
    ports:
      - containerPort: 9567
        name: prometheus

Why is this feature needed:
Would facilitate easy prometheus monitoring via PodMonitor CRD from the prometheus-operator
Example PodMonitor definition...
apiVersion: monitoring.coreos.com/v1
kind: PodMonitor
metadata:
  name: juicefs-mounts-monitor
  namespace: monitoring
  labels:
    name: juicefs-mounts-monitor
spec:
  namespaceSelector:
    matchNames:
      - kube-system
  selector:
    matchLabels:
      app.kubernetes.io/name: juicefs-mount
  podMetricsEndpoints:
    - targetPort: prometheus
      path: '/metrics'
      scheme: 'http'
      interval: '5s'

Other Considerations
Would require updating port if the port is otherwise configured and would require disabling port if monitoring features are disabled.

@dbotwinick this was reworked and released a few days ago and works nicely on my cluster 👍
The only thing that changed was the port name.
apiVersion: monitoring.coreos.com/v1
kind: PodMonitor
metadata:
  name: juicefs-mounts-monitor
  namespace: monitoring
  labels:
    name: juicefs-mounts-monitor
spec:
  namespaceSelector:
    matchNames:
      - kube-system
  selector:
    matchLabels:
      app.kubernetes.io/name: juicefs-mount
  podMetricsEndpoints:
    - targetPort: metrics
      path: "/metrics"
      scheme: "http"
      interval: "5s"

closed by https://github.com/juicedata/juicefs-csi-driver/pull/552


fix: move tag_options from picture element to img element in ix_picture_tag

Description
Currently, the tag_options param is being applied to the <picture> element when using ix_picture_tag. This makes it difficult to actually style the picture element, especially with frameworks like Tailwind, because the browser renders the matching source within the <img> element. This PR moves tag_options so that it is applied to the img tag instead of the picture tag.
A more flexible implementation would allow setting tag_options on both the picture element and the img element. Feel free to update if desired. I didn't want to mess with the method signature.
Checklist: Bug Fix

[ ] Each commit follows the Conventional Commit format: e.g. chore(readme): fixed typo. See the end of this file for more information.
[ ] All existing unit tests are still passing (if applicable)
[ ] Add new passing unit tests to cover the code introduced by your PR
[ ] Update the readme
[ ] Update or add any necessary API documentation
[ ] Add some steps so we can test your bug fix

Steps to Test
One would expect this to center the image, but it does not because it's being applied to the picture element instead of the contained img element.
.mx-auto {
  margin-left: auto;
  margin-right: auto;
}

<%= ix_picture_tag('image.jpg',
  tag_options: { class: 'mx-auto' },
  url_params: { w: 150, h: 150 },
  breakpoints: {
    '(min-width: 768px)' => {
      url_params: { w: 200, h: 200 }
    }
  })
%>

Hey @meltingice 👋 thank you for this useful PR! We're going to discuss this internally and get back to you on this soon, since this will likely be required to be a breaking change if we change this behaviour - potentially we can do this by changing the method signature.
In the meantime, we're going to need the tests to be updated and passing before this is merged. Do you think you'll be able to update the tests? If not, we can take care of this before merging.

No problem, just updated the specs.

Hey @meltingice 👋  so we discussed this internally, and since we would like to not introduce a breaking change here with the parameter change, instead we would like to have this as a second parameter, img_tag_options, and leave tag_options unchanged. If you feel like updating your PR to reflect this change, we'll happily accept it. Otherwise we'll get this on the list to tackle soon 👍

Totally understandable. I've got quite a bit on my plate so I'll leave the changes to you guys. Thanks!

I'm going to go ahead and close this for now @meltingice but we'll use this as a base when we get around to this work! Thanks again 😄


Fix various issues raised by Sonar

Codecov Report

Merging #14867 (f3b4819) into main (bc4eb21) will increase coverage by 0%.
The diff coverage is 52%.

@@          Coverage Diff           @@
##            main   #14867   +/-   ##
======================================
  Coverage     65%      65%
======================================
  Files        551      551
  Lines      25887    25889    +2
  Branches    3672     3672
======================================
+ Hits       16900    16902    +2
  Misses      8294     8294
  Partials     693      693

Impacted Files
Coverage Δ

...rc/client/activation/node/languageClientFactory.ts
36% <0%> (ø)

src/client/activation/node/manager.ts
27% <0%> (ø)

src/client/activation/progress.ts
30% <0%> (ø)

src/client/common/featureDeprecationManager.ts
40% <0%> (ø)

src/client/common/installer/productInstaller.ts
44% <0%> (ø)

src/client/common/utils/sysTypes.ts
56% <0%> (ø)

...bugger/extension/configuration/resolvers/helper.ts
14% <0%> (ø)

src/client/pythonEnvironments/base/locators.ts
62% <0%> (ø)

...ent/pythonEnvironments/discovery/locators/index.ts
84% <ø> (-1%)
:arrow_down:

src/client/terminals/codeExecution/helper.ts
14% <0%> (ø)

... and 10 more


Removed the class TestControlBase and added the class TestControlHelper

Some time ago I created a base class for the TestControl* controls. Unfortunately this prevented them from opening in design mode.
This PR removes that base class and puts that code in a new class called TestControlHelper which is contained in each of the TestControl* classes. This is not very elegant, but better than duplicating the code in all of the TestControl* classes IMHO. Hopefully more code will be moved into TestControlHelper in the future.

Changed. https://ericlippert.com/2014/09/15/internal-or-public/ is a great read on the subject.


not  php AMQPConnection

This SO thread might be of help: https://stackoverflow.com/questions/8199435/php-fatal-error-class-amqpconnection-not-found/8199571


Adding usil.pe

Adding Universidad San Ignacio de Loyola

@Milumon I am sorry, the domain usil.pe is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


Network option to build configurations in docker-compose is not documented

Problem description
Version 3.4 of the Docker compose file introduced the network option to build configurations: https://github.com/docker/docker.github.io/blob/master/compose/compose-file/compose-versioning.md#version-34
This option is not explained anywhere in the documentation.
Problem location

I couldn't find the information I wanted. I expected to find it near the following URL: https://docs.docker.com/compose/compose-file/#build

Closing this ticket due to its age, and the impending refactor. If you think this is in error, feel free to reopen. Thanks!


Error when importing PySwip

I get this error whenever I try to run from pyswip import Prolog
>>> from pyswip import Prolog
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/site-packages/pyswip/__init__.py", line 29, in <module>
    from pyswip.prolog import Prolog
  File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/site-packages/pyswip/prolog.py", line 28, in <module>
    from pyswip.core import *
  File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/site-packages/pyswip/core.py", line 568, in <module>
    (_path, SWI_HOME_DIR) = _findSwipl()
  File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/site-packages/pyswip/core.py", line 417, in _findSwipl
    (path, swiHome) = _findSwiplDar()
  File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/site-packages/pyswip/core.py", line 371, in _findSwiplDar
    (path, swiHome) = _findSwiplFromExec()
  File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/site-packages/pyswip/core.py", line 93, in _findSwiplFromExec
    rtvars = dict((name, value[1:-1]) for name, value in ret) # [1:-1] gets
  File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/site-packages/pyswip/core.py", line 93, in <genexpr>
    rtvars = dict((name, value[1:-1]) for name, value in ret) # [1:-1] gets
ValueError: not enough values to unpack (expected 2, got 1)

I use Prolog 7.6.4
Python 3.8
and macOS

I face the same problem but with swipl 8.0.3, python 3.8 macOS Mojave (10.14.6).
So pyswip is unusable for me at this point :-(.

I actually solved this problem, So first of all I had to use swipl 7.6.4 as that's the one that will work with macOS Mojave so you should downgrade to the 7.6.4 version and then try running the following commands:

export PATH=$PATH:/Applications/SWI-Prolog.app/Contents/swipl/bin/x86_64-darwin15.6.0
export DYLD_FALLBACK_LIBRARY_PATH=/Applications/SWI-Prolog.app/Contents/swipl/lib/x86_64-darwin15.6.0

I actually don't remember if that's all I needed for it to get working or not but I'm pretty sure that these steps helped. So if that didn't work tell me so I can remember what other steps needed.

What worked for me was that I had to import swi-prolog using Homebrew as explained in this article: https://www.swi-prolog.org/build/macos.html
I was able to figure this out by reading the _findSwiplFromExec() function in the core.py file and turns out this error was because swi-prolog was not installed on my local system

I actually solved this problem, So first of all I had to use swipl 7.6.4 as that's the one that will work with macOS Mojave so you should downgrade to the 7.6.4 version and then try running the following commands:

export PATH=$PATH:/Applications/SWI-Prolog.app/Contents/swipl/bin/x86_64-darwin15.6.0
export DYLD_FALLBACK_LIBRARY_PATH=/Applications/SWI-Prolog.app/Contents/swipl/lib/x86_64-darwin15.6.0

I actually don't remember if that's all I needed for it to get working or not but I'm pretty sure that these steps helped. So if that didn't work tell me so I can remember what other steps needed.

How to this the problem, i tried the solution you wrote but its not working.

Having same issue. Tried all the above solutions. Tried with homebrew. The 7.6.4 seems to solve the issue but the library that is currently in use in my program require swi prolog of 8.2 version or later. In which I cannot find any darwin file as suggested in the comment https://github.com/yuce/pyswip/issues/76#issuecomment-958943314
Can someone help me how can i create darwin file in macos ?

What worked for me was that I had to import swi-prolog using Homebrew as explained in this article: https://www.swi-prolog.org/build/macos.html
I was able to figure this out by reading the _findSwiplFromExec() function in the core.py file and turns out this error was because swi-prolog was not installed on my local system

This fixed it for me. Maybe the INSTALL README should reflect that this is the best way to install SWIPL on Mac?

I am having the same issues as described above, I am struggling with trying to solve the issue and my tutors at University don't seem to know how to use it! I am using swi-prolog 9 and pyswip in my project, any help would be much appreciated

The latest version v0.3.0 should fix this problem.
Please feel to reopen the issue if the the problem persists.


Add tests for WriteDateValue and WriteDelimitedFile

Add tests for new command features to write separate files and control how header comments are written.

I added the tests.  Close this issue.


Rotational Angles between frame 6 and 7

In the manipulator Kinematics part of the notebook, between frames 6 and 7, the angle is shown to be 180 degrees around x. To be around x is pretty obvious but when I built the robot on paper using the axes selected by you, I found that angle to be -90 degrees since there is rotation around x to match the other axes(Y6 and Y7 respectively.) I am not sure if there is something I really miss out on.

Hi @followthwhiterabbit, thanks for noticing this. The 180 degree angle is correct but the frame allocated to 6 was incorrect. I have fixed the graphic to reflect this.


State-based icons and entity image

Great job! Convenient, beautiful, versatile. Thank you very much for your work!
It would be great to be able to replace the icon with the entity image and also have the state-based icons, like it realized in default button-card

State based icons are already implemented.
When  #48 is merged and there is an attribute entity_picture this will be used instead of the icon.


Bug: Tab component page opens halfway down the page

When you open the Tabs component page, the focus drops to the Live demo, rather than positioning the user at the top of the page. This is consistent behavior across different browsers: Firefox, Chrome, and Safari.
https://www.carbondesignsystem.com/components/tabs/usage
@alisonjoseph I can fix if you tell me what to do!

V weird, also noticed this other fun bug on the Form page  when testing to make sure other pages weren't broken 😱

this should be resolvable once https://github.com/carbon-design-system/carbon/pull/7410 gets released


[SMALLFIX] Improve runUfsTests console message and tests

Previously, the ./bin/alluxio runUfsTests will log all the messages on the console. When running against object stores, many retry messages will show on the console to make users confusing. Moving the logs to user.log. Fix one UFS test.

@madanadit PTAL, thanks

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4000/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4005/
Test PASSed.

@calvinjia PTAL, addressed all the comments @madanadit left. Thanks

@calvinjia PTAL, thanks

alluxio-bot, merge this please.

alluxio-bot, cherry-pick this to branch-2.0 please.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4016/
Test PASSed.


Hide hamburger menu in Follower UI

Just found in code that hamburger is hidden if user is not logged in.


Feature: Models marked as readonly shouldn't allow save buttons in Edit view

First, thanks, because rails_admin is awesome.
Business:
It would be nice if the rails_admin Edit view did not display the save buttons when model.readonly? # => true

Right now if the model is readonly then an ActiveRecord::ReadOnlyRecord error is thrown when attempting to save.

I might also expect that all form fields on such a model are automatically marked as read_only.
Thoughts? Thanks!

This was an incomplete idea, because readonly? isn't forever and can be dynamic, including based on field values, which means you can edit it to not be read-only any more. In Rails it's only evaluated at the moment of save, but rails_admin has now made it matter at the moment of view.
Now I have to figure out how to tell rails_admin that it's okay to allow editing of a particular readonly? object.


166 ascii details

Some final details for the page.

This includes some updates to the YT video description generator as well.


KAFKA-6765: Handle exception while reading throttle metric value in test

Quota tests wait for throttle metric to be updated without waiting for requests to complete to avoid waiting for potentially large throttle times. This requires the test to read metric values while a broker may be updating the value, resulting in exception in the test. Since this issue can also occur with JMX metrics reporter, change synchronization on metrics with sensors to use the sensor as lock.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@junrao Thanks for the reviews, I have made the updates from the last review. Test failure is unrelated, merging to trunk.

Merging to 1.1 as well since the fix for KAFKA-6870 depends on this.


checkpoint is not saved while training the mode in fish-diffusion-2.1.0

/root/anaconda3/envs/fish/lib/python3.10/site-packages/lightning_fabric/plugins/io/torch_io.py:61: UserWarning: Warning, hyper_parameters dropped from checkpoint. An attribute is not picklable: Can't pickle local object 'EvaluationLoop.advance..batch_to_device'
rank_zero_warn(f"Warning, {key} dropped from checkpoint. An attribute is not picklable: {err}")

You can ignore this warning message.


How to reinitialise the autoScroll when a new element is added to the group?

How do we reinitialise the autoScroll for a new column that is dynamically added?
For example, consider we have Trello like board where there are two columns like "Todo" and "In progress". Consider we have already initialised the autoScroll for these two columns on load to inherit the autoscrolling property when we drag the cards around. Now i am adding a new column named "Done" dynamically without any page refresh. How do i reinitialise the autoScroll function on the 3rd column in this case. I have tried making a function like shown below whenever i add new columns and called it when i create new columns.
initScroll() {     autoScroll([document.querySelector('.list-container')], {         margin: 20,         maxSpeed: 5,         scrollWhenOutside: true,         autoScroll: function(){             //Only scroll when the pointer is down, and there is a child being dragged.             return this.down;         }     }); }
But this doesn't work and i see that dragging cards around after calling this function multiple times makes the app so slow, seems like some kind of recursive function calls is happening internally if the function initScroll() is made more than once without a page referesh.  I would like to understand a workaround for this. Is this a bug?

Added a temporary fix to this issue in my example like so. As per the above details i was calling initScroll whenever a new column was added. Now instead of calling initScroll method i created another method
`
resetScroll() {
// Remove all old column references
[...Array.from(this.elRef.nativeElement.querySelectorAll('.columns'))].forEach(el => this.scrollElements.remove(el));

// Add new column references
[...Array.from(this.elRef.nativeElement.querySelectorAll('.columns'))].forEach(el => this.scrollElements.add(el));

}
`
This works in my case without any issues. Call this function whenever you want to reset the scroll conditions. Dont know if this is the wright thing to do. Please do comment.


Add ECC_SIGNATURE_ALGORITHM env integration test

Add an integration for ECC CA certs. There was a regression a few months back that could have been caught sooner had an integration test existed.
This is based on tests/integration/security/ca_custom_root. Re-factor to follow once this passes tests.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[X] Security
[X] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[X] Does not have any changes that may affect Istio users.

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

/test all

@jacob-delgado: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
84b6fc7e39ba9031dd0e7e26d70dcfed635c5034
link
/test unit-tests_istio

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

@jacob-delgado: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
84b6fc7e39ba9031dd0e7e26d70dcfed635c5034
link
/test unit-tests_istio

integ-security-k8s-tests_istio
84b6fc7e39ba9031dd0e7e26d70dcfed635c5034
link
/test integ-security-k8s-tests_istio

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

/test all

@jacob-delgado: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-security-k8s-tests_istio
84b6fc7e39ba9031dd0e7e26d70dcfed635c5034
link
/test integ-security-k8s-tests_istio

unit-tests_istio
e53c660766dccb07768d3f6ef141f4a1d9c21f05
link
/test unit-tests_istio

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

@jacob-delgado: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
e53c660766dccb07768d3f6ef141f4a1d9c21f05
link
/test unit-tests_istio

integ-security-k8s-tests_istio
e53c660766dccb07768d3f6ef141f4a1d9c21f05
link
/test integ-security-k8s-tests_istio

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

/test all

@jacob-delgado: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-security-k8s-tests_istio
e53c660766dccb07768d3f6ef141f4a1d9c21f05
link
/test integ-security-k8s-tests_istio

unit-tests_istio
56651266badc93ade573613fdbd708049b2d2277
link
/test unit-tests_istio

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

/test all

@jacob-delgado: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-security-k8s-tests_istio
e53c660766dccb07768d3f6ef141f4a1d9c21f05
link
/test integ-security-k8s-tests_istio

unit-tests_istio
ba0b514ede5a091e3e5df141d415148d7e377b6c
link
/test unit-tests_istio

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

/test all

/test all

/test all

/test all

@jacob-delgado: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-security-k8s-tests_istio
c8762bb9cbefcb0d0278f8f07298d869df3610bd
link
/test integ-security-k8s-tests_istio

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

/test all

@jacob-delgado: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-security-k8s-tests_istio
e4bf03b40d581743c1be01f2ec7d09117b82a674
link
/test integ-security-k8s-tests_istio

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

/test all

/test all

@jacob-delgado: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-security-k8s-tests_istio
e4bf03b40d581743c1be01f2ec7d09117b82a674
link
/test integ-security-k8s-tests_istio

lint_istio
23cd94918f7732d310c4797575ea8dc78518dda6
link
/test lint_istio

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

@jacob-delgado: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-security-k8s-tests_istio
e4bf03b40d581743c1be01f2ec7d09117b82a674
link
/test integ-security-k8s-tests_istio

lint_istio
23cd94918f7732d310c4797575ea8dc78518dda6
link
/test lint_istio

gencheck_istio
23cd94918f7732d310c4797575ea8dc78518dda6
link
/test gencheck_istio

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

@jacob-delgado: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
23cd94918f7732d310c4797575ea8dc78518dda6
link
/test lint_istio

gencheck_istio
23cd94918f7732d310c4797575ea8dc78518dda6
link
/test gencheck_istio

integ-security-k8s-tests_istio
23cd94918f7732d310c4797575ea8dc78518dda6
link
/test integ-security-k8s-tests_istio

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

@jacob-delgado remember to cherry-pick this to older releases 1.6 and 1.7, thanks!

/cherry-pick release-1.7

@jacob-delgado: once the present PR merges, I will cherry-pick it on top of release-1.7 in a new PR and assign it to you.

In response to this:

/cherry-pick release-1.7

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

/cherry-pick release-1.8

@jacob-delgado: once the present PR merges, I will cherry-pick it on top of release-1.8 in a new PR and assign it to you.

In response to this:

/cherry-pick release-1.8

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

@jacob-delgado: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
23cd94918f7732d310c4797575ea8dc78518dda6
link
/test gencheck_istio

integ-security-k8s-tests_istio
23cd94918f7732d310c4797575ea8dc78518dda6
link
/test integ-security-k8s-tests_istio

lint_istio
e6e46b38e64e217b9d8b5ce41ff1489449efd065
link
/test lint_istio

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

@jacob-delgado: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-security-k8s-tests_istio
23cd94918f7732d310c4797575ea8dc78518dda6
link
/test integ-security-k8s-tests_istio

lint_istio
e6e46b38e64e217b9d8b5ce41ff1489449efd065
link
/test lint_istio

gencheck_istio
e6e46b38e64e217b9d8b5ce41ff1489449efd065
link
/test gencheck_istio

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

@jacob-delgado: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-security-k8s-tests_istio
23cd94918f7732d310c4797575ea8dc78518dda6
link
/test integ-security-k8s-tests_istio

lint_istio
e6e46b38e64e217b9d8b5ce41ff1489449efd065
link
/test lint_istio

gencheck_istio
e6e46b38e64e217b9d8b5ce41ff1489449efd065
link
/test gencheck_istio

analyze-tests_istio
e6e46b38e64e217b9d8b5ce41ff1489449efd065
link
/test analyze-tests_istio

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

@jacob-delgado: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
cd964cc5c2435b9388aa9ff6feb0736f0b4995a5
link
/test lint_istio

analyze-tests_istio
cd964cc5c2435b9388aa9ff6feb0736f0b4995a5
link
/test analyze-tests_istio

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

@jacob-delgado: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
cd964cc5c2435b9388aa9ff6feb0736f0b4995a5
link
/test lint_istio

analyze-tests_istio
cd964cc5c2435b9388aa9ff6feb0736f0b4995a5
link
/test analyze-tests_istio

integ-telemetry-k8s-tests_istio
cd964cc5c2435b9388aa9ff6feb0736f0b4995a5
link
/test integ-telemetry-k8s-tests_istio

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

@jacob-delgado: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-telemetry-k8s-tests_istio
cd964cc5c2435b9388aa9ff6feb0736f0b4995a5
link
/test integ-telemetry-k8s-tests_istio

analyze-tests_istio
bd650cd3aae56e79f36e64455449504045da84ca
link
/test analyze-tests_istio

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

/retest

@jacob-delgado: #28894 failed to apply on top of branch "release-1.8":
Applying: Add integration test for ECC_SIGNATURE_ALGORITHM
Using index info to reconstruct a base tree...
M	pkg/test/framework/features/features.yaml
Falling back to patching base and 3-way merge...
Auto-merging pkg/test/framework/features/features.yaml
CONFLICT (content): Merge conflict in pkg/test/framework/features/features.yaml
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 Add integration test for ECC_SIGNATURE_ALGORITHM
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to this:

/cherry-pick release-1.8

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

@jacob-delgado: new issue created for failed cherrypick: #29821

In response to this:

/cherry-pick release-1.8

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

@jacob-delgado: #28894 failed to apply on top of branch "release-1.7":
Applying: Add integration test for ECC_SIGNATURE_ALGORITHM
Using index info to reconstruct a base tree...
M	pkg/test/framework/features/features.yaml
Falling back to patching base and 3-way merge...
Auto-merging pkg/test/framework/features/features.yaml
CONFLICT (content): Merge conflict in pkg/test/framework/features/features.yaml
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 Add integration test for ECC_SIGNATURE_ALGORITHM
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to this:

/cherry-pick release-1.7

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

@jacob-delgado: new issue created for failed cherrypick: #29822

In response to this:

/cherry-pick release-1.7

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


Key offset for negative keys

When running retarget_metahuman_animation_sequence(), there's an issue with the controls that have both positive and negative values, where it's appending negative keys to the curve rather than merging the curves, so all negative curves are falling outside of the time range, like this...

I think this might be happening during pm.pasteKey.
Will let you know if I figure out a fix.

I have a hacky solution. Here I'm adding a new function for adjusting the negative keys after everything's processed. I've included an edited version of retarget_metahuman_animation_sequence that incorporates this...
`def align_negative_keys(attribute, start_frame, end_frame):
'''
When imported, sliders with negative values are incorrectly timed, playing after
positive values. This function reintegrates positive and negative values with correct
timing.
'''
attribute = str(attribute)
keys = cmds.keyframe(attribute, query=True)
if keys:
# Iterate over each key
for key in keys:
# Get the value of the key
value = cmds.keyframe(attribute, time=(key, key), query=True, eval=True)
# If the value is 0.0
if value and value[0] == 0.0:
# Delete the key
cmds.cutKey(attribute, time=(key, key))
keys = cmds.keyframe(attribute, time=(end_frame, end_frame + end_frame), query=True)

if keys:
    # Offset each keyframe
    for key in keys:
        # Move the keyframe by the offset amount
        value = cmds.keyframe(attribute, time=(key, key), query=True, eval=True)
        cmds.setKeyframe(attribute, time=key - end_frame, value=value[0])

cmds.cutKey(attribute, time=(end_frame, end_frame+end_frame))

@show_wait_cursor
def retarget_metahuman_animation_sequence(fbx_path, namespace=DEFAULT_NAMESPACE):
'''
Imports the fbx animation into the scene and connects the curve data to the control rig.
Then will bake the animation on the controls and clean up the scene
To keep the file size small and still bring in the animation, here are suggested settings
Unreal FBX Export Options:
    * FBX Export Compatibility: 2020
    * Set ONLY these check-box's to True
        * Export Morph Targets: True
        * Export Preview Mesh: True
        * Map Skeletal Motion to Root: True
        * Export Local Time: True

Args:
    fbx_path (str): absolute path to fbx animation
    namespace (str): rig namespace, so we can gather all the controls

Returns:
    tuple(str, str): elapsed time to complete, error message
'''
start_time = time.time()
load_plugin()

error_msg = ''
elapsed_time = ''

# Build control mapping
controllers, error = get_controllers(namespace)
if error:
    return elapsed_time, error

new_nodes = import_fbx_animation(fbx_path)

# 30fps
pm.currentUnit(time='ntsc')

# Check if animCurves came in
anim_curves = pm.ls(new_nodes, type=pm.nt.AnimCurve)
if not anim_curves:
    error_msg = "No animation curves present!\n\n" \
                "Ensure the exported animation is from an animation sequence!"
    pm.delete(new_nodes)
    return elapsed_time, error_msg

new_joints = pm.ls(new_nodes, type=pm.nt.Joint)
root_joint = get_root_joint(new_joints)
if not root_joint:
    pm.delete(new_nodes)
    error_msg = 'Did not find the root joint from: {}.\n' \
                'Ensure the exported animation is from an animation sequence!'.format(fbx_path)
    pm.delete(new_nodes)
    return elapsed_time, error_msg

# Get the range of keys
start_frame, end_frame = get_key_frame_range(root_joint)

controlAttrList = []

# Take the mapping data and copy animation keys over to the proper control.channel
for controller in controllers:
    for control_attr, expression_data in controller.control_mapping.items():
        for index, (expression, driver_value) in enumerate(expression_data):
            # Incoming anim data is 0-1 but is driven by controls that can
            # move -1 to +1, so we're remapping the data as we go
            if root_joint.hasAttr(expression):
                # Control moves in the negative
                if driver_value == -1.0:
                    controlAttrList.append(control_attr)
                    # Get the anim curve and scale it by -1
                    anim_curve = root_joint.attr(expression).inputs(type=pm.nt.AnimCurve)
                    if anim_curve:
                        pm.scaleKey(anim_curve[0], valueScale=-1.0)
                        copied = pm.copyKey(root_joint.attr(expression))
                        if copied:
                            try:
                                pm.pasteKey(control_attr)
                            except RuntimeError:
                                logger.error('Failed to paste keys to {}'.format(control_attr))
                    else:
                        logger.error('No animation curve for {}'.format(root_joint.attr(expression)))
                # Control moves in the positive
                else:
                    copied = pm.copyKey(root_joint.attr(expression))
                    if copied:
                        try:
                            pm.pasteKey(control_attr)
                        except RuntimeError:
                            logger.error('Failed to paste keys to {}'.format(control_attr))
            else:
                logger.warning('{} does not have {} in the name. This will be skipped!'.format(root_joint, expression))

for attribute in controlAttrList:
    align_negative_keys(attribute, start_frame, end_frame)

# Clean up
pm.delete(new_nodes)

pm.playbackOptions(animationStartTime=int(start_frame), animationEndTime=int(end_frame))

delta_time = time.gmtime(time.time() - start_time)
elapsed_time = str(time.strftime("%H:%M:%S", delta_time))
logger.info("Transfer completed in: {}".format(elapsed_time))
return elapsed_time, error_msg`

Hi @Danlowlows ,
Sorry for the delay. Been away. Thanks for digging in. I think I see the issue now. The default behavior for pm.pasteKey() is insert. Haven't tested this yet but I believe adding the kwarg, option='merge' would produce the desired behavior. I'll test this out when I can.

Hi @swnolan. Thanks for the reply.
I did try using option='merge', but the problem there is that both the positive and negative curves are keyframe-per-frame, even when the value of the attribute is zero. If you set option='merge', it's first pasting the negative keys, then overwriting them completely with the positive curves. Basically it stomps all the non-zero values from the negative curve, with zero values from the positive curve.
If there's an option to mask values when copying or pasting, and stripping out any zero keys, that'd probably do it. Alternatively, what I'm doing in the code above, is to let it paste both the positive and the negative curves, then "move" the negative keys back to where they should be. I put "move", in quotes cause I couldn't get Maya to actually offset these keys, I assume since it crosses existing keyframes on the curve, so when I say "move", I'm just setting a new keyframe and then wiping the old one. Like I say though, that's just a time consuming process and I figure there has to be a better way.

Ah, yeah, this makes total sense. I see what you're saying about the extra time to process with looping through each of the keyframes to extract out the values. Not ideal of course. Appreciate the insights and code. Let me tinker with this some more before I work in what you have. While I certainly don't want to add a bunch more time to the process, but I certainly don't want to be putting bad data onto the controls. Again, thanks for finding this and providing a solution. This darn work stuff tends to get in the way of this side stuff ;)

I'm working up a new solution. For each face controls, there can be either 1 or 2 input anim curves to drive each X or Y channel. If there's 2 input curves (expressions) for each channel, then take the one that needs to be -1 scaled and the other positive curve and plug these 2 curves into a blendWeighted node. Connect the output of the blendWeighted node into the control attribute. Keep a list of the blendWeighted nodes created and connected up the channels. Loop through the lists and bake out the connected channels and delete the blendWeighted nodes.
For the other controls with only 1 input anim curve, simply copy/paste. While the bake process will take a bit more time but since it'll be only be on a smaller subset of controls, it might not be too bad. But at least the results should be correct...at least in theory. Will just take the main processing loop in the retarget_metahuman_animation_sequence() call a bit more logic to implement. Fun puzzle :D

That’s a good idea! Worth a shot. :)

Here's updated code. Should handle negative curves. Should be a pretty minimal impact on the time to process.
Give it try and let me know if this works for you. If so, I'll update main.
`
@show_wait_cursor
def retarget_metahuman_animation_sequence(fbx_path, namespace=DEFAULT_NAMESPACE):
'''
Imports the fbx animation into the scene and connects the curve data to the control rig.
Then will bake the animation on the controls and clean up the scene
To keep the file size small and still bring in the animation, here are suggested settings
Unreal FBX Export Options:
	* FBX Export Compatibility: 2020
	* Set ONLY these check-box's to True
		* Export Morph Targets: True
		* Export Preview Mesh: True
		* Map Skeletal Motion to Root: True
		* Export Local Time: True

Args:
	fbx_path (str): absolute path to fbx animation
	namespace (str): rig namespace, so we can gather all the controls

Returns:
	tuple(str, str): elapsed time to complete, error message
'''
start_time = time.time()
load_plugin()

error_msg = ''
elapsed_time = ''

# Build control mapping
controllers, error = get_controllers(namespace)
if error:
	return elapsed_time, error

new_nodes = import_fbx_animation(fbx_path)

# 30fps
pm.currentUnit(time='ntsc')

# Check if animCurves came in
anim_curves = pm.ls(new_nodes, type=pm.nt.AnimCurve)
if not anim_curves:
	error_msg = "No animation curves present!\n\n" \
	            "Ensure the exported animation is from an animation sequence!"
	pm.delete(new_nodes)
	return elapsed_time, error_msg

new_joints = pm.ls(new_nodes, type=pm.nt.Joint)
root_joint = get_root_joint(new_joints)
if not root_joint:
	pm.delete(new_nodes)
	error_msg = 'Did not find the root joint from: {}.\n' \
	            'Ensure the exported animation is from an animation sequence!'.format(fbx_path)
	pm.delete(new_nodes)
	return elapsed_time, error_msg

# Get the range of keys
start_frame, end_frame = get_key_frame_range(root_joint)

# Take the mapping data and copy animation keys over to the proper control.channel
blend_weighted_nodes = []
controls_attrs_to_bake = []
for controller in controllers:
	for control_attr, expression_data in controller.control_mapping.items():

		# Connect control channel that has more than one anim curve driving it
		if len(expression_data) > 1:
			anim_curves = []
			for index, (expression, driver_value) in enumerate(expression_data):
				if root_joint.hasAttr(expression):
					anim_curve = root_joint.attr(expression).inputs(type=pm.nt.AnimCurve)
					if anim_curve:
						# Control moves in the negative
						if driver_value == -1.0:
							# Get the anim curve and scale it by -1
							pm.scaleKey(anim_curve[0], valueScale=-1.0)
					else:
						logger.error('No animation curve for {}'.format(root_joint.attr(expression)))

					anim_curves.append(anim_curve[0])
				else:
					logger.warning('{} does not have {} in the name. This will be skipped!'.format(root_joint, expression))

			# Connect anim curves
			bw_node = pm.createNode('blendWeighted')
			for i, anim_curve in enumerate(anim_curves):
				anim_curve.output.connect(bw_node.input[i])
			bw_node.output.connect(control_attr)
			controls_attrs_to_bake.append(control_attr)
		else:
			for index, (expression, driver_value) in enumerate(expression_data):
				if root_joint.hasAttr(expression):
					anim_curve = root_joint.attr(expression).inputs(type=pm.nt.AnimCurve)
					if anim_curve:
						copied = pm.copyKey(root_joint.attr(expression))
						if copied:
							try:
								pm.pasteKey(control_attr)
							except RuntimeError:
								logger.error('Failed to paste keys to {}'.format(control_attr))
				else:
					logger.warning('{} does not have {} in the name. This will be skipped!'.format(root_joint, expression))

# Bake controls
pm.bakeResults(controls_attrs_to_bake,
				time=[int(start_frame), int(end_frame)],
				preserveOutsideKeys=True,
				sparseAnimCurveBake=False,
				sampleBy=1,
				oversamplingRate=1,
				removeBakedAttributeFromLayer=False,
				removeBakedAnimFromLayer=False,
				shape=False,
				controlPoints=False,
				disableImplicitControl=True)

# Clean up
pm.delete(blend_weighted_nodes)
pm.delete(new_nodes)

pm.playbackOptions(animationStartTime=int(start_frame), animationEndTime=int(end_frame))

delta_time = time.gmtime(time.time() - start_time)
elapsed_time = str(time.strftime("%H:%M:%S", delta_time))
logger.info("Transfer completed in: {}".format(elapsed_time))
return elapsed_time, error_msg

`

Compared blink anim curves from UE to Maya and appear to be aligned now

Tested your fix and it's working well on my side. Like you say, difference in processing time is negligible. Great job!

Great! Main is updated now! Thanks for testing and helping out!


[PAD-2450] feat: migrate empower dbt to bq v2

See also PRs in airbyte-configs and airflow-dags (TK)
TODO after merging:

[ ] update destination namespaces in V2 syncs to remove _v2_test suffixes
[ ] merge airflow-dags PR (TK)
[ ] trigger DAG with refresh_matviews: True
[ ] merge airbyte-configs PR

I've been running this locally using datasets run on V2 and things seem to be checking out!


Change the Hover font family of vscode

I don't find how to change the hover font family of vscode

The question gets on my nerves

/duplicate https://github.com/xtermjs/xterm.js/issues/2979


Add Qt GUI refresh w/branding updates

Align Qt wallet better with new style guidelines and consistent branding
Removes old non-default themes due to brittle design (they don't play well with new redesign)
Remove small 3 buttons underneath PS button

The try and reset buttons are not necessary
The PS info modal is still available via the help menu

This is an alternative to #2204
Please see individual commits for more granular info on changes.

+68 −7,547 - nice :)

Few notes:

should run contrib/devtools/optimize-pngs.py to shrink new images a bit (Total reduction: 98903 bytes);
should drop images/light/about.png because it's no longer used after f924607;
would prefer the old "remove.png" icon (the one used in Console tab, the new one looks weird there imo).

All these changes in one commit https://github.com/UdjinM6/dash/commit/ad784aa4dcb1ca7468ce7c87d5f09fa43c756287
Could also adjust masternodes.png icon to match new style while at it I guess.

Added your commit, thanks @UdjinM6 . I've reached out to Suba to help w/the MN icon, and he said he found some more that he missed, so will wait for a patch from him (should be pretty quick).

General overview below as of https://github.com/dashpay/dash/pull/3000/commits/33d0fee13777f667135ac51afcab2908a1a07b7d

Screens from Light-HiRes theme on a HiDPI monitor.
Please note that in all screens multiple of the bottom right icons are missing:

Text very often overlaps or wraps strangely.  Table columns are too small very often, but I assume this is not in the scope of this PR.

Rebased onto latest develop and force-pushed newest changes.

I've rebased this after splitting out the extra themes removal and re-grouped into new commits (the old commit history was kind of a mess).
Should be ready for review and testing now.

8ed6afc99f9c82be56fa3e57e27b6c52684efb53 I can't see what's changed/updated here
ea3d6f5d9a2bd400b7fd47046714f8ff21e85721 Most icons are without transparancy now
src/qt/res/icons/address-book.png
This seems to be the wrong icon (doesn't look like an address book) and also is missing transparancy
src/qt/res/icons/tx_inout.png, tx_input.png, tx_output.png
I liked the older ones much better. With the new ones it's less clear what they mean. Left and right is easier associated with incoming/outgoing while up/down gives some room for more interpretations (balance gone up?).
Also, the in+out icon has much smaller arrows then the individual ones, which is strange

Why 2df07ff? It's not needed imo (splash image colors are adjusted automagically).

@UdjinM6 re: https://github.com/dashpay/dash/commit/2df07ff7d30664e0f18ad2dfbeb85ec602ef6245, I didn't know why it was removed in 3141 so assumed it should not have been.
@iamsuba Can you help answer these questions about icons that @codablock asked above?

@codablock  @nmarley

Most icons are without transparancy now
Yes, they are. There was a reason I went with non-transparent icons. I don't remember exactly why. I will try doing transparent icons and see the issues.

src/qt/res/icons/address-book.png
This seems to be the wrong icon (doesn't look like an address book) and also is missing transparancy
I will update to a new icon.

src/qt/res/icons/tx_inout.png, tx_input.png, tx_output.png
I liked the older ones much better. With the new ones it's less clear what they mean. Left and right is easier associated with incoming/outgoing while up/down gives some room for more interpretations (balance gone up?).
The icons where done this way to make is consistent with the dishpan and native wallet redesign icons, where we are using simple up and down arrows.

Also, the in+out icon has much smaller arrows then the individual ones, which is strange
The sizes are different because, the same icons are used in the transaction history pages. these sizes make it look good on transaction history pages but not in homepage. I guess we could resize the icons to be smaller.

Transparency has been fixed in icons now, directions fixed and sizes should be correct.
@UdjinM6 I reverted the testnet splash screen commit
@codablock As for your question about https://github.com/dashpay/dash/commit/8ed6afc99f9c82be56fa3e57e27b6c52684efb53, I believe this is where UdjinM6 ran the optimize images script, but I can't say for sure. I picked this from an old commit of his as the base for this work. But that's what it looks like since the file size is reduced for all these.

Ah, yes, we must run new images through mogrify (to fix nasty error messages) and optimize-pngs.py (to reduce image size). The later has to be fixed first though https://github.com/UdjinM6/dash/commit/d8f6ed7d793476c661a6db9f9497a5470df4f54b, we missed it in #3141 🙈 Changes for this PR would be https://github.com/UdjinM6/dash/commit/301121f475b4eb91febe57c0b6deca463c14210f and https://github.com/UdjinM6/dash/commit/5d927284dc63ac1083276ce1565e20d301ae4bd1 accordingly.
I'm still not happy with the new "remove" icon tbh https://github.com/dashpay/dash/pull/3000#issuecomment-505168279. I tried to make the icon in Console tab a bit larger https://github.com/UdjinM6/dash/commit/1e283140109c97bafd4f91fca2b0462392923fcd and it's a bit better that way IMO but it's still weird. I would probably copy the old one into some new file and use it in Console instead. Or maybe we could use a more general delete/remove icon like a trashcan or smth that would fit in both places instead?

@UdjinM6 Cherry-picked your changes except for the last one, I didn't see where this remove.png was showing up on the debug console 'til now. I agree the old one looks better there, restored the old remove icon as console_remove.png.

@codablock need me to test hires? I've got 4k screens.


Add esp32_devkitc_wrover to some tests and samples

Add esp32_devkitc_wrover boards to supported platforms:
tests/drivers/can/api
samples/drivers/ipm/ipm_esp32
samples/drivers/dac
samples/drivers/adc
samples/boards/esp32/flash_encryption

@henrikbrixandersen FYI, we have been improving our internal CI to not only build sample cases but also testing on real device. That is the reason we have been enabling lots of esp32 testcases like this.

@henrikbrixandersen FYI, we have been improving our internal CI to not only build sample cases but also testing on real device. That is the reason we have been enabling lots of esp32 testcases like this.

That's great news. Did you solve the flash-before-console twister issue locally?

@henrikbrixandersen FYI, we have been improving our internal CI to not only build sample cases but also testing on real device. That is the reason we have been enabling lots of esp32 testcases like this.

That's great news. Did you solve the flash-before-console twister issue locally?

Yep. We are using a patch locally and soon will work on a PR so others can make use of it.


Create an abstraction library

Motivation
For the extensibility purpose- caching library,..., we need to separate basic interfaces/models to the separate library.
I created a  Kentico.Kontent.Delivery.Abstractions library, which contains a minimum required dependencies for extract IDeliveryClient interface.
Do you know, which other parts of the Delivery SDK should be extracted to *.Abstractions library? @alesk-kentico, @petrsvihlik
In the next steps, we separate this library to own NuGet package.

@alesk-kentico if we decide to release the abstractions as a separate package (which we agreed we want), then I agree that we should try to keep the abstracted scope minimal.
on the other hand, the vast majority of the code has been abstracted already in the existing package and is just being moved to the new one. so...in that case, I don't object to moving the existing abstractions.


[WIP] New .pot file generated from running rake locale:update

@mzazrivec I did not check in  locale/*/*.po because I am trying to understand the rationale that I inquired about here

Replaced by https://github.com/ManageIQ/manageiq-release/pull/143


Fix/content 3 4

This pull request has been deployed to Vercel.

Latest commit:
c04f223

✅ Preview:
https://polkadot-bootcamp-3t2ttbec1-openedu101.vercel.app

🔍 Inspect:
https://vercel.com/openedu101/polkadot-bootcamp/GhQn8Zf1ChMKb8SLQpaDHbC8Rgqt

View Workflow Logs


Option to remove site from List API while still being functional

There should be a setting within the JSON file to indicate that a site is functional (defunct: 0) but may not support BotBlock and its API methods to post or get bot data.
There are currently two lists that match this case:

Top.gg: Intentionally blocks requests from the BotBlock API IPs as the owner is banned I guess
Discordextremelist.xyz: Uses IP-based rate limiting without API key, resulting in a higher amount of rate limits for BotBlock itself.

These are the sites I'm aware of that either have problems with the BotBlock API or intentionally deny support for it.
A setting to then f.e. add a notice to the Site info may be useful. Or perhaps remove it completely from the API in such cases.
I hope you get what I want to say.

I'm not sure it would make sense for us to actually remove a list that has an API from BotBlock's API integration. If folks are having trouble with those lists through BotBlock, the best bet would be for them to stop including them. We list the API information for those lists, and make use of it, because those lists have a published API.
I'd also call out that for both of the lists you mention, the issue was with the old deployment model for BotBlock where all traffic originated from a single source IP. BotBlock is now deployed globally with Cloudflare Workers, so I doubt either of these is an active issue anymore.
If we do need to add a note on the site about lists, the existing content field on lists should be used for this -- it should render as a notice on the list page.

Fair enough.


Pid equation test

Fixes https://github.com/Deltares/Ribasim/issues/473.
There are some complications with the PID derivative term that make it rather complex to implement. The core of the problem is that the derivative term makes the storage change of the listened to basin dependent on itself:
$$
\frac{\text{d}u}{\text{d}t} = \sum F + \sum Q - \phi(u)\left[ K_pe + K_iI + K_d\left(\dot{\text{SP}} - \frac{\text{d}u}{\text{d}t}\right) \right]
$$
where $\phi(u)$ is defined the same as here: https://github.com/Deltares/Ribasim/pull/465.
This computation is further complicated by the clipping of the pump flow value to its minimum or maximum. I don't think it is possible to create an analytical solution of the above equation that converges to the setpoint with only positive flow without using the derivative term (of course that shouldn't be the main argument to keep the derivative term). Here is what the solution looks like (assuming this is correct): https://www.desmos.com/calculator/jqbpnlv8pi.
There is already a PidControl test that converges to the setpoint without the derivative term with the pump turning of when the PID control dictates negative flow. I could attempt to formulate an analytical solution of that one.
The reason I started reformulating the derivative term of PID control is that currently the right hand side $\frac{\text{d}u}{\text{d}t}$ of the above equation was taken from not the current but the previous call to water_balance!, which I thought could lead to problems (since I got ill behaved solutions).

I may have done did the thing though


Update README.md

added some headers

Hey @AyanaAlemayehu , I wasn't able to accept your pull request because I merged another pull request with the same content.  Thanks for the contribution though!


[BUG] Microsoft.Azure.WebJobs.Extensions.ServiceBus delay between batches of session-based messages

Library name and version
Microsoft.Azure.WebJobs.Extensions.ServiceBus, 5.3.0
Describe the bug
Didn't realize v5.x moved here, see info from this other issue in the old repo. https://github.com/Azure/azure-functions-servicebus-extension/issues/180#issue-1163179060
TLDR: A simple ServiceBusTrigger is seeing a delay after processing MaxConcurrentSessions messages that appears related to the TryTimeout setting.
Expected behavior
To keep processing messages until there are no more messages in the queue.
Actual behavior
Process 8 messages, delay of 1 minute, process 8 messages, repeat until queue is empty.
This is actually causing the Azure scaler to scale-out the function app to max nodes as the queue does not get emptied fast enough.
Reproduction Steps
Function:
using System.Threading.Tasks;
using Microsoft.Azure.WebJobs;
using Microsoft.Azure.WebJobs.ServiceBus;
using Microsoft.Extensions.Options;

namespace FunctionApp1
{
    public class Function1
    {
        [FunctionName("Function1")]
        public async Task RunAsync([ServiceBusTrigger("queue-name", IsSessionsEnabled = true)] string json)
        {
        }
    }
}

host.json
{
  "version": "2.0",
  "logging": {
    "logLevel": {
      "default": "Debug"
    },
    "applicationInsights": {
      "samplingSettings": {
        "isEnabled": true,
        "excludedTypes": "Request"
      }
    }
  }
}

local.settings.json
{
  "IsEncrypted": false,
  "Values": {
    "AzureWebJobsStorage": "UseDevelopmentStorage=true",
    "AzureWebJobsServiceBus": "Endpoint=sb://.servicebus.windows.net/;SharedAccessKeyName=RootManageSharedAccessKey;SharedAccessKey=",
    "FUNCTIONS_WORKER_RUNTIME": "dotnet"
  }
}

Environment
This happens locally and appears to be happening when deployed in Azure. Also noticed similar behavior on dotnet-isolated worker with similar code/setup except that it would process 2000 messages and then wait because the MaxConcurrentSessions value is 2000 instead of 8 as it was in the in-process worker.

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

You can set the SessionIdleTimeout in host.json to configure how long to wait before closing the current session and moving on to another when no new messages are coming in.

That's the thing. There are more messages in the queue but the trigger/function app is still waiting between batches.

That's the thing. There are more messages in the queue but the trigger/function app is still waiting between batches. /unresolve

Yes, but the messages are in different sessions, right? Or are you saying there are remaining messages in the same session that are not being processed?

Yes, it's different sessions, so this is waiting for other messages in the same sessions then...

@smokedlinq, did you get any workaround/fix on this?.
We are also facing similar issue.

Yes I lowered the SessionIdleTimeout setting to 1 second. The session will wait for more messages before giving up. The default is 1 minute.


[hevce] Add check for intra refresh with B-frames

Intra refresh isn't supported with B-frames.
Fixes #2571

@yskuridi , is it ready for merge?


[stable/sysdig] Remove auditLog.clusterIP value

We don't need to use clusterIP in advance:

If using webhook, we can deploy the agent and then configure webhook
to post audit logs to the clusterIP.
If using AuditSink, we can rely on KubeDNS for reaching the service.

Signed-off-by: Néstor Salceda<EMAIL_ADDRESS>
Is this a new chart

NOTE: We're experiencing a high volume of PRs to this repo and reviews will be delayed. Please host your own chart repository and submit your repository to the Helm Hub instead of this repo to make them discoverable to the community. Here is how to submit new chart repositories to the Helm Hub.

What this PR does / why we need it:
This PR removes the auditLog.clusterIP need.
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/assign @bencer

/lgtm
/approve


zed-editor: 0.154.3 -> 0.154.4

https://github.com/zed-industries/zed/releases/tag/v0.154.4
Things done
Note that I switched to gitUpdater to ignore the pre releases and avoid PRs like #345500

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

I didn't know about this gitUpdater !

I didn't know there was more than one but I did recall one having the ability to ignore rc,beta...


Crash: dyld: Symbol not found: OBJC_CLASS$_Reachability

Steps to reproduce
I just installed the pod on an existing app and it crashed. However, when I tried it on a new Xcode project, there was no crash and it was working perfectly.
What could help pinpoint the issue is that I previously already have the Reachability.swift file copied directly on my existing app. However, when I deleted it, the crash is still there. I also have Reachability library by Tony Million through another pod, but I have no idea which one.
Expected behavior
No crash on app launch.
Actual behavior
App crashed on launch with the following crash logs.
dyld: Symbol not found: _OBJC_CLASS_$_Reachability
  Referenced from: /Users/abc/Library/Developer/CoreSimulator/Devices/387D652C-CB80-4739-88B1-DD04ACAFA8D6/data/Containers/Bundle/Application/749E2EB8-AD62-438E-8EE8-7AAF5E8C3028/LalaFood-User.app/Frameworks/ChatSDK.framework/ChatSDK
  Expected in: /Users/abc/Library/Developer/CoreSimulator/Devices/387D652C-CB80-4739-88B1-DD04ACAFA8D6/data/Containers/Bundle/Application/749E2EB8-AD62-438E-8EE8-7AAF5E8C3028/LalaFood-User.app/Frameworks/Reachability.framework/Reachability
 in /Users/abc/Library/Developer/CoreSimulator/Devices/387D652C-CB80-4739-88B1-DD04ACAFA8D6/data/Containers/Bundle/Application/749E2EB8-AD62-438E-8EE8-7AAF5E8C3028/LalaFood-User.app/Frameworks/ChatSDK.framework/ChatSDK

Hoping someone could help. Thank you.

For the time being, in order to proceed with my task at work, I just manually copied the files I need from PusherSwift, instead of doing a pod install.

Reachability is now no longer a dependency of this SDK, as of the v9.1.0 release. Upgrading to the latest version should resolve this issue.


Question on functionality

Ok so I am quite new to coding and I am taking a class in school based on learning coding, so I might be extremely ignorant. So I am trying to make a project that draws letters on the screen from pre-made paths using Javascript. At the moment I have a custom svg for every letter and I basically make a clone of that svg depending on what letter I press on the keyboard using id's such as svgA, svgB, svgC, svgD, etc... Is there a way to do this without using an svg per letter? And at the moment even with a custom svg per letter it doesnt seem to work.
var startXPos = 100;
var startYPos = 50;
var drawing = false;
var availableLetters = {
"A": true,
"B": true,
"C": true,
"D": true,
"E": true,
"F": true,
"G": true,
"H": true,
"I": true,
"J": true,
"K": true,
"L": true,
"M": true,
"N": true,
"O": true,
"P": true,
"Q": true,
"R": true,
"S": true,
"T": true,
"U": true,
"V": true,
"W": true,
"X": true,
"Y": true,
"Z": true
}
document.addEventListener("click", function() {
if (drawing == false) {
console.log("started vivus?")
drawing = true;
new Vivus('svgA', {
duration: 200
}, function() {
$(function() {
jQuery.Color.hook("fill");
})
});
setTimeout(function() {
drawing = false;
}, 2000)
}
});
document.addEventListener("keyup", function(e) {
if(availableLetters[String.fromCharCode(e.keyCode).toUpperCase()]) {
var clonedLetter = $('#svg' + String.fromCharCode(e.keyCode).toUpperCase()).clone(true);
clonedLetter.attr('id', 'current');
document.body.append(clonedLetter);

new Vivus('current', {
    duration: 200
}, function() {
    $(function() {
      console.log("Started new Vivus");
        jQuery.Color.hook("fill");
        $('letterA').toggleClass('path', 1000)
        clonedLetter.attr('id', '')
    })
});

}
else {
console.log("Letter is not Available")
}
})

Hi there,
I try to think about another way of doing what you would like to do, but unfortunately you will have to use one SVG per letter. However, you could hide them all and display the correct one when needed and reset/start the animation, instead of cloning SVGs on the go.
If you would like some help to debug, it would be nice to put your code on a JSfiddle. It makes it easier to hack and test.
But from what I can see right now, I think the issue might come from the start option. By default it's inViewport, while in your case autostart would be more appropriate.
new Vivus('current', {
  duration: 200,
  start: 'autostart'
}, function() {
  // ...
});

Ok thank you for the help. As I said I am really new to coding and I would love to learn wherever I can. I have a semi-working version. I made the svg's much smaller and I can type out multiple letters and they will animate but there has too be a delay in the typing or else if a letter is still animating the next letter will appear fully and the letter currently being drawn will restart it's drawing. I am still working on it though so any functionality within Vivus that would be useful to my program would be wonderful :) Thanks again!


"strategy.shouldEnter/shouldExit" does not behave exactly as "tradindRecord.getEntry/getExit"

I guess, there is a bug in "StopGainRule". When running the strategy with a "TradingRecord" by "TradingRecord#getEntry/getExit" all works as expected. However, when using it by  "Strategy#shouldEnter/shouldExit" on each tick, then this rule is not followed! Please check it out.
I am wondering if Strategy#shouldEnter/shouldExit behave wrong in other Rules as well in compare to "TradingRecord#getEntry/getExit".
Why is the result of TradingRecord#getEntry/getExit not exactly the same as "Strategy#shouldEnter/shouldExit"?

I don't understand. Please provide more elements. Try to debug the code to see if something is wrong.

You can test it easily by yourself. Take a dataset (for example one week) and use a "StopGainRule" as a Exit-strategy (Entry-strategy does not matter).
Rule stopGain = new StopGainRule(closePrice, Decimal.valueOf("1.5"));
Now use the dataset with the strategy and print its result. One time with "Trading Record" and the other time with "shouldEnter/shouldExit":
TradingRecord past = series.run(s);

 for(Trade t: trades){
log.info("entry - {}", t.getEntry().getIndex());
log.info("exit - {}", t.getExit().getIndex());
}

Use the same dataset with the same strategy and use the tick of the timeseries:
if (s.shouldEnter(i)) {
log.info("entry - {}", i);
}

 } else if (s.shouldExit(i)){
log.info("exit - {}", i);
}

You can see easily that the Exit by "s.shouldExit" is never called (even the stop-gain rule should happen) while the same series with trading record "t.getExit().getIndex()" is called. "TradingRecord" makes it right (with having the appropriate trades), while "shouldExit" behaves wrong with the same dataset and the same exit-strategy (because shouldExit it not called even stopgain-rule should be fired).

I analysed the TradingRecord and it tells me that "Exit-Rule" is not called even the StopGainRule-Strategy should be fired.
---------- TradingRecord ----------
11:58:14,247 INFO  [stdout:71] orders: [Order{type=BUY, index=14, price=1.0460000000000002629008122312371, amount=NaN}]
11:58:14,248 INFO  [stdout:71] buyOrders: [Order{type=BUY, index=14, price=1.0460000000000002629008122312371, amount=NaN}]
11:58:14,248 INFO  [stdout:71] sellOrders: []
11:58:14,248 INFO  [stdout:71] entryOrders: [Order{type=BUY, index=14, price=1.0460000000000002629008122312371, amount=NaN}]
11:58:14,248 INFO  [stdout:71] exitOrders: []
11:58:14,249 INFO  [stdout:71] trades: []
11:58:14,249 INFO  [stdout:71] startingType: BUY
11:58:14,249 INFO  [stdout:71] currentTrade: Entry: Order{type=BUY, index=14, price=1.0460000000000002629008122312371, amount=NaN} exit: null

satisfied = currentPrice.isGreaterThanOrEqual(entryPrice.multipliedBy(gainRatioThreshold));
To solve your problem (at the moment) you could simple change StopLossRule with StopGainRule

Fixed by #172


graphs in hidden views not sized properl

I have several graphs under different navigation tabs which are loaded with data when the web page loads, but all but the first tab which is visible when the page loads are not sized correctly even if I reload the data when entering the tab. If I resize the window then they appear correctly. How can I force them to size correctly without a resize?

just a follow up query about this. graphs which are initially hidden, now renders in the correct size (initial load), but the legend is "squished", as in the following:
http://plnkr.co/edit/PRSO0W9UUBGAjbHMpDpH?p=preview
The column graph is initially hidden. Legend renders properly once a series is clicked.
But after doing a window resize, the hidden graph's legend becomes squished again, the graph height is fine, and the width I think is based on the previous width prior to resize.
Does anyone have any workaround for this scenario?
Thanks.


Checking of Config class

OS: Linux
Python version: 3.7.3
Pydantic version: 0.24

It it would be useful to check the Config class to contain only supported options, to avoid the situation illustrated below:
from pydantic import BaseModel

class C(BaseModel):
    class Config:
        # note the intentional extra "s" in the option name
        validate_assignments = True

    f1: int = 0

o = C()
# hey, why doesn't this fail?.. Oh, a typo...
o.f1 = []

Disagree, can be useful to add other things to config for custom models.


Clarify fiona.open's encoding keyword arg

I told @NelsonMinar that the intent of this argument was to override encodings found in data when reading, but after reviewing docs, code, and my notes I realize that's not correct. This is what the argument is about:
In 'w' mode, it defines the encoding of the new dataset. If unspecified, the dataset's encoding will be ISO-8859-1 (for shapefiles) or sys.getpreferredencoding().
In 'r' mode, the encoding argument is intended to make up for encoding detection defects in GDAL < 1.9. See the closing paragraph of http://toblerity.org/fiona/manual.html#reading-vector-data. In GDAL versions >= 1.9 it shouldn't be needed and generally has no effect.

Thanks for the follow-up! FWIW of about 6 non-ASCII shapefiles we've seen in OpenAddresses, GDAL 1.11 detects encodings of 5 of them correctly. The 6th one it does not but Fiona then assumes ISO-8859-1 which seems to be correct for this file.
For writing files, have you considered making UTF-8 the default encoding? I don't know what's most likely to be what your users want, I just hate to see anything that's not Unicode as a default.

UTF-8 instead of the preferred system encoding? I've been trying to stick as closely as possible to the text file processing of Python 3 but I agree that UTF-8 would be better.

Oh I forgot about the preferred system encoding, that's probably better. FWIW it's UTF-8 in every Python environment I have.


Added logging to console for debugging.

Logging placeholders in the same style as previous versions added to

python/Interscale_hub/InterscaleHub.py
python/Interscale_hub/pivot.py
python/placeholders/Simulation_mock.py

Some log formatting to remove log error messages.

moved import statements to the start of the file (PEP8).
All three locations are duplicate code but marked with a TODO for proper logging -> included in the next iteration when we use the cosim manager (incl. logging).


pull request for e2e test at 01/16/2017 17:15:13

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


#ext_proc Move setUpstreamHost from openStream to onReceiveMessage, onGrpcError and onGrpcClose

Commit Message: Move setUpstreamHost from openStream to onReceiveMessage, onGrpcError and onGrpcClose
Additional Description: upstream host is only available when handling response. onGrpcError and onGrpcClose are corner cases.
Risk Level: Low
Testing: Modified existing test.
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

/assign @tyxia

/assign @tyxia

Calling setUpstreamHost in closeStream can set it correctly for all scenarios. Another earlier place is onReceiveMessage, but it doesn't handle the scenarios when side stream is closed with error, and it's not necessary to set it early.

/retest

/assign-from @envoyproxy/envoy-maintainers

@wbpcode  Could you please take a look. This PR should be straightforward. Thanks!


[SPARK-11792] [SQL] [FOLLOW-UP] Change SizeEstimation to KnownSizeEstimation and make estimatedSize return Long instead of Option[Long]

https://issues.apache.org/jira/browse/SPARK-11792
The main changes include:

Renaming SizeEstimation to KnownSizeEstimation. Hopefully this new name has more information.
Making estimatedSize return Long instead of Option[Long].
In UnsaveHashedRelation, estimatedSize will delegate the work to SizeEstimator if we have not created a BytesToBytesMap.

Since we will put UnsaveHashedRelation to BlockManager, it is generally good to let it provide a more accurate size estimation. Also, if we do not put BytesToBytesMap directly into BlockerManager, I feel it is not really necessary to make BytesToBytesMap extends KnownSizeEstimation.

@davies how about this version?

Test build #46219 has started for PR 9813 at commit 329c72f.

LGTM

Test build #46219 has finished for PR 9813 at commit 329c72f.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/46219/
Test PASSed.

Thanks! Merging to master and branch 1.6.


Plugin doesn't load correctly

On Grafana 6.5.0, I tried to use the Flowcharting plugin, version 0.5.0 installed from Grafana CLI inside a Docker image but when I select the Flowcharting Vizualisation, the interface doesn't load completely and I can't configure anything.
I can see inside the Javascript console the following error:

An idea how to fix it?

Hi
Yes, i have the same, but 6.5 of grafana is a nighty build, not stable.
BR
Arnaud

FI,
https://github.com/grafana/grafana/issues/20255
thx for your attention
BR
Arnaud

I found a bypass

try 0.6.0 in download section in documentation