Print queue eventually installed but printer invisibleon the network

Adding a print queue to the gutenprint Application seemed to be straightforward:
root@test-new:~# gutenprint-printer-app devices
snmp://NPI2CA3D7
usb://EPSON/Stylus%20Photo%20RX420?serial=LJ4O40407152301540

gutenprint-printer-app add -d RX420 -m epson--stylus-photo-rx-420--en -v usb://EPSON...
The response was
gutenprint-printer-app: Unable to add printer: Unsupported smi55357-device-uri uri value.

I looked at the stae file after using the web interface and altered the URI to cups:usb://EPSON... and the queue was added successfully. What is going on here?
Continuing:
root@test-new:~# lpstat -e
ENVY4500
root@test-new:~#

No sign of the printer until systemctl restarts the gutenprint-printer-app service. Why can this not be done automatically on successful queue installation?

This looks like that you Printer Application did not correctly do the Avahi registration the first time you started it. After the restart it did and therefore the printer got visible. I do not know why on the first start the Avahi registration failed.
Normally, you do not need to restart a PAPPL-based Printer Application after adding a printer. On its very first start, when there is still no printer added, it should already register at Avahi. avahi-browse and avahi-discover should already show entries for the web interface and IPP System Service interface of the Printer Application. If so, if you add a printer, it will also appear immediately, as "Internet Printer" (IPP), "Secure Internet Printer" (IPPS), and "UNIX Printer".
When you use the command line to add a print queue, put the URI in single quotes. It contains many characters which get messed up by the shell
gutenprint-printer-app add -d RX420 -m epson--stylus-photo-rx-420--en -v 'usb://EPSON...'

Also there is a bug in the CUPS package in Debian and also in Ubuntu 22.10 which prevents the Printer Application from correctly working (at least the web interface does not work);

https://launchpad.net/ubuntu/+source/cups/2.4.2-1ubuntu4
http://launchpadlibrarian.net/650318292/cups_2.4.2-1ubuntu2_2.4.2-1ubuntu4.diff.gz

The links show my fix in the Ubuntu package. ./configure must be called with --with-tls=gnutls and not any more with --enable-gnutls.

Ok. Thanks. I'll look into this a bit more. However, what is the the significance of
-v cups:usb://EPSON...

as opposed to
-v usb://EPSON...

The first form is implemented by the web UI and works. The second form doesn't work. I did try double and single quotes round the URU - with no effect.

pappl-retrofit supports both the backends which come with PAPPL and any CUPS backends. The PAPPL backends have the schemes usb:, socket:, ... and to support the CUPS backends one needs to add custom backends to the PAPPL-based Printer Application. A custom backend has a custom scheme. So I created a custom umbrella backend with the scheme cups:. The scheme name always ends with a colon, an URI starts with the scheme and after the colon it is completely to the backend which syntax the rest of the URI has.
So I created the custom scheme cups: to support the CUPS backends. The syntax for the URIs is simple, cups: plus the original URI of the CUPS backend. so
-v cups:usb://EPSON...

means that the "usb" backend of CUPS is used by the Printer Application, and
-v usb://EPSON...

means that PAPPL's built-in "usb" backend is used.
Which backends are available through the cups: scheme depends on which backend executables are in the directory where the Printer Application is searching and also on the Printer Applications configuration settings as there one can use an include list and an exclude list for backends.
The Legacy Printer Application of pappl-retro-fit for example should make all classically installed CUPS drivers available as Printer Application, therefore it points its search directory for CUPS backends to /usr/lib/cups/backend/ to have the same backends available as CUPS has.
With all this pappl-retrofit can also retro-fit classic CUPS printer drivers which come with their own backends.

If you do not get the URIs cleanly submitted into the Printer Application via the command line interface, there is probably a bug in PAPPL. You need to report it there with a debug log of the Printer Application attached.

I have fixed michaelrsweet/pappl#253, as michaelrsweet/pappl#262 so the bug is actually in PAPPL and therefore I am closing this bug report.


Fixed Java Core Issue 1592 - Find root cause of ConnectionError

TJWS close connection every 100 requests processed in case keep-alive is true. 100 times response header has connection: keep-alive and ‘keep-alive: …, max=100’.
However, some client does not take care max parameter of keep-alive header value. And the client reuse the socket (connection) for 101 times request, it seems fails to send if TJWS takes time  to close the socket.
As solution, to set connection: close and remove keep-alive header value if times of requested equals with maximum times connection used.

As we started to modify TJWS, we could also move the code that fixes socket timeout into TJWS directly.

@pasin Agree.


Calculating Item Heights for Nodes with Overflow [feature request]

Hey there,
Thanks for making this!
There is an assumption in your getItemHeight function that creates an inflexibility around some use cases which I will attempt to describe.
In cases where an element has overflowing children - regardless of what the parent's overflow value is set to, this calculation results in a value that accounts for the children's full dimensions (exceeding the parent's dimensions). This does make sense.. if I have a div whose overflow is visible, and its children exceed it's bounds, then presumably I would want the grid to align with the outer bounds of the child item, not the parent. This is not true if I want to animate the height of a grid item as described below.
A little background - I'm working on a pinterest style grid that has animated expanding/collapsing items in it. I've been fighting the stack-grid layout algorithm a bit since I need to coordinate when to update the layout around the animations (since I'm animating height). I refactored some code so that my grid items' height doesn't animate allowing the grid to update immediately with the correct height. At the same time, I am transitioning the height of a grid item child node. This works great when expanding since the grid-item becomes it's final height instantly while the inner child animates to that height. On collapse, however, I have to have overflow: visible on the parent in order to be able to see the transition on the child's height but at the same time, stack-grid wants to immediately measure and transition the grid, but because it uses scrollHeight the calculation is wrong (it accounts for the overflowing child).
I wonder if this is too crazy of a use case to warrant an option - perhaps you could allow the user to define the function that getItemHeight uses to extract the height from the node? If that were the case, I could just pass a function that ignores scrollHeight and just uses clientHeight or offsetHeight.
Thanks!

On another subject (and I can create a separate issue if that's desired), since the container element type can be customized (via component prop) it would make sense to also allow users to define the grid item container element (perhaps via gridComponent or something).
e.g. I want to make the wrapper into a ul and the grid items into li's but right now the <span> that wraps each grid item is hard-coded and passing component="ul" results in invalid markup where span are the directly children of the list instead of li elements.

Hi @solomonhawk Thank you for Issues :smiley:

perhaps you could allow the user to define the function that getItemHeight uses to extract the height from the node?

It seems to be a little special use case, but I agree with the option addition :+1:
If possible send us a PR and it will be very helpful :pray:

As you can see, the gridComponent option is essential for adjusting markup.
I would like to respond immediately :muscle:

@solomonhawk  Released a version including itemComponent props as v0.7.0 :tada:.

Awesome! Thanks @tsuyoshiwada. I'll try to find time to prepare a PR for the getItemHeight option.


Service Request by Monika about Specification checks

[x] Qualtrics Service Request Form was received from User
[ ] e-mail acknowledging that User's Qualtrics Service Request Form was sent
[ ] Problem NOT RESOLVED
[ ] Problem PARTIALLY RESOLVED
[ ] Problem FULLY RESOLVED and Issue is Closed
[ ] Qualtrics Post-Service Request Survey was sent to User

Use comments feature (below) to document:

A clear and concise description of the solution.

A clear and concise description of any alternative solutions or features you've considered.

Any additional context, such as screenshots about the problem.

From: David Lounsbury<EMAIL_ADDRESS>Sent: Wednesday, September 22, 2021 3:10 PM
To: david lounsbury<EMAIL_ADDRESS>Subject: IPA Tracker Support Request Form Submitted
IPA SERVICE REQUEST SUMMARY

CONTACT INFORMATION:

Monika  Salvage
Email<EMAIL_ADDRESS>Phone:<PHONE_NUMBER>

COUNTY

Cayuga

WHICH FEATURE(S):

SPECIFICATION: Creating, updating, deleting information to define or describe a SMART Strategy

BRIEF PROBLEM DESCRIPTION:

The C2 strategy shows all red marks under specification status as if no info was entered???

URGENCY:

50

ANYTHING ELSE TO SHARE:

this was already addressed


Update version strings for 1.10

Unsure if anything in the support table needs to be updated. Copy/pasted entries from TF 1.9


Improve unit test coverage of Feature parser/encoder helper

Specifically, in org.bdgenomics.adam.rdd.feature.Features. We need to improve coverage of the ontology term and dbxref codes, as well as the GFF tools.

I can pick this issue up.
@fnothaft  can you please explain what needs to be done here?

Closing as WontFix


Pull Request: Center Align Footer

Issue:
When zooming out the webpage, the footer is aligning to the left side instead of center, which affects the visual consistency and aesthetics of the website. (Issue #6)
Changes Proposed:
To address this issue, the following changes have been made:
 Added margin  : 0 auto;  to the footer CSS to center align it horizontally.
 Added justify-content : center;  to the footer CSS to ensure the content is centered horizontally within the footer.
How to Test:

Open the website.
Zoom out the webpage.
Scroll down to view the footer.

 Additional Notes:
These changes utilize  flexbox  properties to achieve a responsive and visually pleasing layout for the footer, ensuring it remains centered regardless of the zoom level of the webpage.
This PR resolves the issue of the footer alignment and enhances the overall user experience of the website.
Closing Notes:
This PR aims to improve the visual consistency and aesthetics of the website by addressing the footer alignment issue. Any feedback or suggestions for further enhancements are welcome. Thank you for reviewing!

Hey @ag19bhoomi
Kindly include screenshots for verification of the fix. Additionally, ensure that necessary changes are made to all relevant files to maintain consistency across all web pages.
Thanks!


Build fails: error: unknown type name 'j_decompress_ptr'

🐛 Describe the bug
FAILED: /usr/ports/misc/py-torchvision/work-py39/vision-0.18.0/build/temp.freebsd-14.0-STABLE-amd64-cpython-39/usr/ports/misc/py-torchvision/work-py39/vision-0.18.0/torchvision/csrc/io/image/cpu/decode_png.o
c++ -MMD -MF /usr/ports/misc/py-torchvision/work-py39/vision-0.18.0/build/temp.freebsd-14.0-STABLE-amd64-cpython-39/usr/ports/misc/py-torchvision/work-py39/vision-0.18.0/torchvision/csrc/io/image/cpu/decode_png.o.d -Wno-unused-result -Wsign-compare -Wunreachable-code -DNDEBUG -O2 -pipe -fstack-protector-strong -fno-strict-aliasing -O2 -pipe -fstack-protector-strong -fno-strict-aliasing -fPIC -DPNG_FOUND=1 -DJPEG_FOUND=0 -DNVJPEG_FOUND=0 -I/usr/ports/misc/py-torchvision/work-py39/vision-0.18.0/torchvision/csrc -I/usr/local/include/libpng16 -I/usr/ports/misc/py-torchvision/work-py39/vision-0.18.0/torchvision/csrc -I/usr/local/lib/python3.9/site-packages/torch/include -I/usr/local/lib/python3.9/site-packages/torch/include/torch/csrc/api/include -I/usr/local/lib/python3.9/site-packages/torch/include/TH -I/usr/local/lib/python3.9/site-packages/torch/include/THC -I/usr/ports/misc/py-torchvision/work-py39/vision-0.18.0/torchvision/csrc/io/image -I/usr/local/lib/python3.9/site-packages/torch/include -I/usr/local/lib/python3.9/site-packages/torch/include/torch/csrc/api/include -I/usr/local/lib/python3.9/site-packages/torch/include/TH -I/usr/local/lib/python3.9/site-packages/torch/include/THC -I/usr/local/include/python3.9 -c -c /usr/ports/misc/py-torchvision/work-py39/vision-0.18.0/torchvision/csrc/io/image/cpu/decode_png.cpp -o /usr/ports/misc/py-torchvision/work-py39/vision-0.18.0/build/temp.freebsd-14.0-STABLE-amd64-cpython-39/usr/ports/misc/py-torchvision/work-py39/vision-0.18.0/torchvision/csrc/io/image/cpu/decode_png.o -g0 -DTORCH_API_INCLUDE_EXTENSION_H '-DPYBIND11_COMPILER_TYPE="_clang"' '-DPYBIND11_STDLIB="_libcpp"' '-DPYBIND11_BUILD_ABI="_cxxabi1002"' -DTORCH_EXTENSION_NAME=image -D_GLIBCXX_USE_CXX11_ABI=0 -std=c++17
In file included from /usr/ports/misc/py-torchvision/work-py39/vision-0.18.0/torchvision/csrc/io/image/cpu/decode_png.cpp:3:
/usr/ports/misc/py-torchvision/work-py39/vision-0.18.0/torchvision/csrc/io/image/cpu/exif.h:200:40: error: unknown type name 'j_decompress_ptr'
  200 | inline int fetch_jpeg_exif_orientation(j_decompress_ptr cinfo) {
      |                                        ^
1 error generated.

Versions
0.18.0
pytorch-2.2.2
clang-18
Python-3.9

@yurivict  Can you please share the snippet you used to build torchvision? As well as the full output of
wget https://raw.githubusercontent.com/pytorch/pytorch/main/torch/utils/collect_env.py
# For security purposes, please check the contents of collect_env.py before running it.
python collect_env.py

I had the same error and got things to compile by editing vision-0.18.0/torchvision/csrc/io/image/cpu/exif.h. Specifically, I moved #include <jpeglib.h> to outside the #if JPEG_FOUND condition. Further down in this file you can see that it makes use of j_decompress_ptr both under the condition #if JPEG_FOUND and under #else. I'm not sure whether the latter is deliberate, as I can see that the relevant lines are:
#else // #if JPEG_FOUND
inline int fetch_jpeg_exif_orientation(j_decompress_ptr cinfo) {
  return -1;
}
#endif // #if JPEG_FOUND

This indicates that perhaps this used to be #if JPEG_FOUND but has been changed. I didn't test this yet, but my guess is that if these lines had been changed back to #if JPEG_FOUND instead of #else, then that would have also solved the problem, because j_decompress_ptr would not be needed anywhere.
Hope that helps!

Thanks @lrhammond .
I suspect we've fixed that build issue already (probably in https://github.com/pytorch/vision/pull/8342/files) because the lines you're referring to aren't present in the code anymore. I suspect the latest changes weren't pulled in when you tried to build from source?
I'll close this issue as I believe if it has been resolved, but please feel free to re-open a new one if the build keeps failing on the latest main branch.

0.18.1 (released on May 20 after your last comment on May 20) still has this problem.
log

Sorry folks, I would have to double check but what's probably happening is that we fixed the compilation issue on main but did not cherry-pick it onto the release branche(s).


CLANG fixed ops python [run_process_replay]

As requested by George

Yea you want the tests to be independent of the implementation, like the real hardware


Allow copy for "editor.dragAndDrop"

VSCode Version: 1.10.1
OS Version: Win10

I enabled editor.dragAndDrop and it works as expected.
But I also expected Ctrl+DragAndDrop to copy the selection to the new location (without using the clipboard) instead of moving it.
If you implement this, please change the mouse cursor while dragging it, similar to Word.

-- Sorry for the English, I used Google Translator --
Allow to DragAndDrop content to another VSCode tab and to other programs. This is currently not possible. (Without having to click Ctrl)
By clicking Ctrl it would only copy within the same file, right?

Indeed a dup of #21145
Apparently on a Mac you use the Alt-key when copying. On Windows it is the Ctrl-key.

@Tekbr yes you'll have Ctrl on Windows.


Ensemble Learning

Hello NVIDIA,
I am training some models on different datasets. It has come to my notice that when transfer learning, the model starts becoming better for the new dataset but its performance worsens on the previous. Therefore I decided to use ensemble learning to fix this. I have considered using AdaBoost (https://scikit-learn.org/stable/modules/generated/sklearn.ensemble.AdaBoostClassifier.html)
How could the sklearn implementation of this method be used with OpenSeq2Seq or an ensemble learning boosting approach in general?

If you use AdaBoost with Tensorflow, you can or use ScipyOptimizerInterface API, or implement new optimizer (see for example tf/contrib/optimizers


optimize logic and reduce alloc

caller will pass pointer to the callee so it can write the result there, instead of callee allocating and returning new value
add GetOutputAmountV2 to return only necessary data (without calling NewPool again)
optimize MulDiv following univ3-core code (avoiding bigInt)

I'm reviewing your PR, but please give me some time to understand it.

Thanks a lot, man!
Yes, this PR only contains optimizations on top of part1, most of the bigInt replacements are there (https://github.com/KyberNetwork/uniswapv3-sdk/pull/16), please help to also review it when you have some time (this is not a bug fix so nothing urgent that you had to work so late like that)


Save Button was not properly visible fixed UI issue

Updated Text Field to OutlinedText Field
added card to list
updated one gradle deprecation

Look and fill improvements

Hi @pravinkurhade , thanks for your PR! In general, it adds too much complexity so I would try to focus only on the button not being displayed in certain scenarios. I'll try to get to this soon.


How to use SecretValue creating a skill?

https://github.com/aws-samples/aws-devops-blog-alexa-cdk-walkthrough/blob/73cbfd23ef1a5281b6080c2135f1142724b38027/lib/alexa-cdk-stack.ts#L18
I tried to use this example:
    const PARAM_PREFIX = '/xxxx-ha-skill/'
    const alexaVendorId = ssm.StringParameter.valueForStringParameter(this, `${PARAM_PREFIX}alexa-developer-vendor-id`);
    const lwaClientId = ssm.StringParameter.valueForStringParameter(this, `${PARAM_PREFIX}lwa-client-id`);
    const lwaClientSecret = cdk.SecretValue.secretsManager(`${PARAM_PREFIX}lwa-client-secret`);
    const lwaRefreshToken = cdk.SecretValue.secretsManager(`${PARAM_PREFIX}lwa-refresh-token`);

    const skill = new Skill(this, 'Skill', {
      endpointLambdaFunction: skillBackendLambdaFunction, // @aws-cdk/aws-lambda.IFunction object containing backend code for the Alexa Skill
      skillPackagePath: 'src/skill-package', // path to your skill package
      alexaVendorId: alexaVendorId, // vendor ID of Alexa Developer account
      lwaClientId: lwaClientId, // client ID of LWA Security Profile
      lwaClientSecret: lwaClientSecret, // @aws-cdk/core.SecretValue object containing client secret of LWA Security Profile
      lwaRefreshToken: lwaRefreshToken // @aws-cdk/core.SecretValue object containing refresh token of LWA Security Profile
    });

but it doesn't work:
Error: Resolution error: Synthing a secret value to . Using a SecretValue here risks exposing your secret. Only pass SecretValues to constructs that accept a SecretValue property, or call AWS Secrets Manager directly in your runtime code. Call 'secretValue.unsafeUnwrap()' if you understand and accept the risks..

Unfortunately, adding .unsafeUnwrap() doesn't work either, because that returns a string, not a SecretValue.  I cannot figure out a workaround for this.  The fact that new Skill() accepts a SecretValue tells me it should work.
Versions of everything:
  "devDependencies": {
    "@types/jest": "^29.5.11",
    "@types/node": "20.14.2",
    "aws-cdk": "2.144.0",
    "ask-cli": "^2.10.0",
    "jest": "^29.7.0",
    "ts-jest": "^29.1.2",
    "ts-node": "^10.9.2",
    "typescript": "~5.4.5",
    "esbuild": "^0.21.4"
  },
  "dependencies": {
    "aws-cdk-lib": "2.144.0",
    "cdk-alexa-skill": "^2.0.4",
    "constructs": "^10.3.0",
    "source-map-support": "^0.5.21"
  }

ok I found part of the answer HERE
In a nutshell, it's saying to get rid of the warning by adding:
    "@aws-cdk/core:checkSecretUsage": false,

to cdk.json.  Note though that the stackoverflow says:

Ideally you'd address the root cause (perhaps the 3rd Party Skill construct?) so the check can remain enabled. But disabling the flag should silence the error.

which I believe to be the right solution ...

I confirmed that by unchecking checkSecretsUsage it still does the right thing i.e. it did not leak my secrets into the cloudformation template.  So that's good.  Disabling the check is still not a great idea, though,  It won't catch it if someone makes a mistake elsewhere.


Watching changes on mapMultiRowFields

In my case scenario, I need to set a property based on another property. Let me explain it better:
I have an array of invoice details:
details: [
    {
        amount: null
        quantity: null
        product: {
            id: 1
            name: 'Lorem ipsum',
            price: 1234
        },
    },
    {
        amount: null
        quantity: null
        product: {
            id: 2
            name: 'Dolor at',
            price: 3456
        },
    },
]

My store module simply looks like
state: {
        details: [],
},
getters: {
        getField,
},
mutations: {
        updateField,
}

and my component is mapping in this way
...mapMultiRowFields('invoice', [
        'details',
]),

Question is: how can I set de detail.amount with the detail.product.price when the product is selected/changed? I need to use it as a suggested price but keep the detail.amount bound to the state since the user can manually change the value.
I was thinking to overwrite the details state mutations but not sure if it's doable and how can it be done.
Also, I've found a vuex-watcher plugin but not even sure if it will work
Any help/recommendations will be really appreciated.

The way I handled this is with an @input directive. You pass in the row index as a parameter, call the updateField mutation in the method, using the row index in your path, e.g.
this.$store.commit('updateField', {path: `details[${index}].amount`, value: value})

For more advanced use cases, you might consider to use https://davestewart.github.io/vuex-pathify/#/ instead of this package.
Otherwise, you can use the approach @geoidesic provided.


How to update editor with dynamically filled in data?

I set up the JSON editor as documented. In my workflow there is some jQuery click handler on the text input element with some script dynamically populating text in. But when I call editor.getValue() later, the returned data does not have the updated value.
$('#json-editor input').on('click', function (e) {
  $(e.target).val('abc');
});

//.. later
var data = editor.getValue();
// data is not updated

Any idea what is missing? Thanks.

You might have to manually call editor.setValue(), give that a shot

Did anyone get the setValue function working? I'm trying to load json dynamically into the editor.
editor.setValue(validJson)


Add SparkPost support

@crisu83 you should disable Circle-CI, it's kinda useless. The Travis build is fixed now.


Create migration election class

Problem: We need a way to synchronize a halt to block production to manage migrations
Solution
Created the MigrationElection class

Codecov Report

Merging #2535 into master will decrease coverage by <.01%.
The diff coverage is 94.28%.

@@            Coverage Diff             @@
##           master    #2535      +/-   ##
==========================================
- Coverage   93.37%   93.37%   -0.01%
==========================================
  Files          43       44       +1
  Lines        2583     2596      +13
==========================================
+ Hits         2412     2424      +12
- Misses        171      172       +1


[ja_JP]kana of Japanese name by gender.

Changed person name for ja_JP
Updated README.md for ja_JP
I divided Japanese kana name by gender.

Thanks!


feat:Added automation for checking commit style

This will help beginners to follow the commit style which Keptn is following.
closes #94
Signed-off-by: Tarang Verma<EMAIL_ADDRESS>

this has been added..

I has been added


Json Outline pane doesn't use displayName tag

If a resource has tags ➝ displayName, Visual Studio's JSON outline view would display this value in the outline view rather than the 'name' variable. It seems this extension just uses the 'name' variable here. This is particularly useful to make the outline more readable when all the names come from variables/parameters.
Could this be supported for more friendly display names in the JSON outline pane?

Sounds like a good idea, thanks.  There was a similar request in #29 about having a setting.  Is there anything other than the displayName tag that you would want us to consider using?

@StephenWeatherford maybe icons of the resource type?
May not have to be exhaustive, but e.g. "type": "Microsoft.Web/sites" having a matching web app icon and a few things like this rather than the 'resource box' type icon.

FYI @damienpontifex we just released an update that includes this fix (specifically the displayName stuff - the icon stuff is tracked by https://github.com/Microsoft/vscode-azurearmtools/issues/92)


[Search Documents] Update CHANGELOG.md failing in nightly runs

Search Documents nightly test runs are failing with:

Error message:
##[error]The job running on agent azsdk-pool-mms-win-2019-general 69 ran longer than the maximum time of 60 minutes. For more information, see https://go.microsoft.com/fwlink/?linkid=2077134

For more details check here:

https://dev.azure.com/azure-sdk/internal/_build/results?buildId=1790655&view=results

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Search:0.4213692,Azure.Core:0.32739633,Azure.Identity:0.022421794'


DoubleConversion latest version upgradation.

Description
The react-native's latest version 0.66 uses 1.1.6 DoubleConversion version where as the latest DoubleConversion version is 3.1.6. Is there any plan of upgrading to latest double conversion version.
Version
0.65.1
Output of react-native info
System:
OS: macOS 12.0.1
CPU: (8) x64 Apple M1
Memory: 32.21 MB / 8.00 GB
Shell: 5.8 - /bin/zsh
Binaries:
Node: 14.17.5 - /usr/local/bin/node
Yarn: 1.22.11 - /usr/local/bin/yarn
npm: 6.14.14 - /usr/local/bin/npm
Watchman: Not Found
Managers:
CocoaPods: 1.11.2 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: iOS 14.5, DriverKit 20.4, macOS 11.3, tvOS 14.5, watchOS 7.4
Android SDK: Not Found
IDEs:
Android Studio: Not Found
Xcode: 12.5.1/12E507 - /usr/bin/xcodebuild
Languages:
Java: 1.8.0_131 - /usr/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: 17.0.2 => 17.0.2
react-native: 0.65.1 => 0.65.1
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
Steps to reproduce
In double conversion podspec the version is 1.1.6
Snack, code example, screenshot, or link to a repository
No response

The react-native's latest version 0.66 uses 1.1.6 DoubleConversion version where as the latest DoubleConversion version is 3.1.6. Is there any plan of upgrading to latest double conversion version.

Closing as this is a question.
To answer you: no. There are no immediate plans in bumping this dependency, unless there is a valid use case or a security patch that requires a bump.


vertically align the task list item

Vertically align the task list item  so that the list item's bullet always appears to the left of the first line of the task when the task's text wraps.  This addresses an issue where the list item's bullet will appear to the left of the last line of the task when the task's text wraps and , as discussed in gjtorikian/task-lists-js#5 and atom/markdown-preview#142

Arguably a better option is to add a class to all  elements identified as task list items, as proposed in #5, to allow straightforward styling of these elements.

Oops, this is generated from the Coffeescript.  Will think of something smarter and resubmit.

@brandoncurtis I think the file you want is index.coffee
Also, it's quite strange that this is generating a <label> tag as the actual HTML generated by task lists on github is just <ul>'s, <li>'s and <input>'s.


Typo in ArchiveReader.cs will cause build to fail in all configurations except Debug

If you look at Line 246 in "SharpCompress\Common\SevenZip\ArchiveReader.cs" you will notice extra line that is causing build to fail in any build configuration other than Debug:
#if DEBUG
                            Log.WriteLine("Complex Stream (In: " + coder.NumInStreams + " - Out: " + coder.NumOutStreams +
#endif
 ")");
                        }

So it is necessary to remove Line 246 completely in order to fix this issue.

Thanks for this, could you make this a Pull Request by chance?


why should i navigate with setTimeout?

i have to check some condition in componentDidUpdate lifecycle and if my condition is true, i navigated my screen to Home screen like this command:
componentDidUpdate() {
        if (this.props.accessToken && !this.state.appLoading) {
            if (firstTime) {
                firstTime = false;
                setTimeout(() => {
                    this
                    .props
                    .navigation
                    .navigate('Home');
                }, 500);
            }
        }
    }

i have InputText in my Home Page and after navigate to Home page, if i don't use the setTimeout in above code , when i tap on my InputText for first time, the inputText lost its focus

software
version

react-navigation
^1.0.0-beta.11

react-native
0.48.3

node
6.11.1

npm or yarn
3.10.10

Maybe the navigate method altered this.props.navigation so causing the current screen to re-render and get the focus.
If you don't have UI to be changed, you can try putting the logic in shouldComponentUpdate() and see if it works.
Better for this case though (I guess it's the login -> Home logic?), I think you should not use navigation.
Maybe something like this?
render() {
  if (this.state.appLoading) {
    return <AppLoading>;
  } else {
    if (this.props.accessToken) {
      return <Home>;
    } else {
      return <Login>;
    }
  }
}


[Feat.Req.] Json beautifier in datagrid call detail

When selecting a cell in datagrid and opening its content (SHIFT-Enter), it would be great to have, apart from already existing "Text" and "Hex", a view like "Json" (Xml, etc) with json string beautified (formatted, indented etc).
taht is a quick win, no? ;-)

hi @wiwwo ,
what db and datatype do you use?
as far as i can see it is already implemented, e.g. fo postgresql.

Hi! Maybe that is for fields that have json as datatype...
We want to store Jsons in DB without using the json datatype, but the normal varchar datatype (long story short: requirements, since that field could contain XML as well).
(Yes, my DB is postgresql)

thanks for clarifying and suggestion


Wrong RAM suggestion

Hi,
When setting up the server on a Raspberry Pi 4B+ with 8GB of RAM the script suggested 7.259GB.
Even though I only run the minecraft server on my Raspberry Pi it broke down, when starting chunky to pregenerate my world.
When decreasing the RAM to 6.8GB (max RAM - 10%) everything works fine.

Turn it way down.  To like 2GB.  This script was written for Pis that used less than 512MB.  There is no need to choke your system for memory to this extent.  Even 4GB would be extremely generous.  Hopefully that helps!

Okay thank you for the explanation. I'm aware that your script can't take anything into account but on your website you wrote, that the script will automatically suggest the maximum available RAM minus 10% margin. So when executing the script I thought I could go with the recommendation.

It's not really wrong. It's a suggestion/recommendation based on the recommendations for this server which is headless meaning you aren't really running anything else.

I'm running headless so this shouldn't be a problem.
What you wrote about reducing the allocated RAM totally makes sense. I will definitely try it.
By the way, I really like this script. While I'm not new to Linux and the Raspberry Pi, it's the first time I set up a Minecraft Server. Your script saved me a lot of time getting started. So thank you!

No worries at all.  To be honest if I wrote it today it would not have that at all (or really need it).  The only people who may still need to fight with the memory are probably people with 1GB Pis.  It's sort of a relic from back when we had Pis with < 1GB of RAM.
Back then you had to literally allocate your memory budget by the percentage.  These days it's far more likely to cause trouble than anything.  My instinct is to remove it completely but I know there are definitely people running it on 1GB Pis out there.  I doubt anyone is running it on anything less than that still but this level of tuning would still be required for those.
That's why it's a little bit of an anachronism though so I apologize for you getting caught up in that piece of it.  I haven't made it "smarter" because I feel like we're 1 generation away from probably dropping that altogether so it has always seemed like a waste of time.  It probably is time to start phasing it out though.
I could put the parameter at the top of the script as a manual override for anyone who still requires the functionality.  This is probably the directional I'll move in for the early-2022-ish season so that nobody is cut off but it will eliminate the confusion as you are not the first one to ask me about this (I get a lot more comments on my web site than I do GitHub issues opened so I've seen it more over there).
Thanks for the kind words about the script.  The feedback is much appreciated and I do want to make some changes here because of our conversation.  Take care!


Add support for the build target info

Task
https://github.com/scalameta/metals/pull/3380#pullrequestreview-874527874

Closed via https://github.com/scalameta/nvim-metals/pull/341


Client stats on partitioned topics missing some aggregated values

Describe the bug
In ProducerStatsRecorderImpl.updateCumulativeStats, it aggregates
numMsgsSent
numBytesSent
numSendFailed
numAcksReceived
totalMsgsSent
totalBytesSent
totalSendFailed
totalAcksReceived

but doesn't aggregate
sendMsgsRate
sendBytesRate
latencyPctValues[]

leading to incomplete stats for a parent partitioned topic producer. The consumer stats have a similar problem.
To Reproduce
Steps to reproduce the behavior:

Create a partitioned topic
Create a producer on that topic
Call producer.getStats(), the rate and latency values will always be 0

Expected behavior
The "parent" producer of a partitioned topic should have fully populated aggregated stats from all its "children" producers

I'm interested in trying to solve this issue, tks

#4482

@hellozepp Are you working for this issue?
@sleungtoast Can we move this issue to next milestone(2.4.1 or 2.5.0)?

@codelipenghui Yes,There are some changes that need to be confirmed.

@codelipenghui Yes we have a workaround so we are not blocked on this.

move to 2.7.0

Closed as stale and no one showed interest any longer. Please open a new issue if it's still relevant to the maintained versions.


Implement SourceLink

This work would see the implementation of SourceLink similar to the recent changes in XPlat Windows APIs

Thanks for reviewing @tom-made


Poetry cannot be installed with Pip 18.1 (released 5 Oct)

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

OS version and name: macOS 10.14
Poetry version: 0.11.5

Issue

Poetry installation via get-poetry.py fails with Pip 18.1.
22:30 ~/Downloads python3 -m venv test
22:30 ~/Downloads test/bin/pip install --upgrade pip==18.0
Collecting pip==18.0
  Using cached https://files.pythonhosted.org/packages/5f/25/e52d3f31441505a5f3af41213346e5b6c221c9e086a166f3703d2ddaf940/pip-18.0-py2.py3-none-any.whl
Installing collected packages: pip
  Found existing installation: pip 10.0.1
    Uninstalling pip-10.0.1:
      Successfully uninstalled pip-10.0.1
Successfully installed pip-18.0
You are using pip version 18.0, however version 18.1 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.
22:30 ~/Downloads curl -sSL https://raw.githubusercontent.com/sdispater/poetry/master/get-poetry.py | test/bin/python
Retrieving metadata

Installing version: 0.11.5
  - Getting dependencies
  - Vendorizing dependencies
  - Installing poetry

poetry (0.11.5) successfully installed!
22:30 ~/Downloads rm -rf test
22:31 ~/Downloads python3 -m venv test
22:31 ~/Downloads test/bin/pip install --upgrade pip==18.1
Collecting pip==18.1
  Using cached https://files.pythonhosted.org/packages/c2/d7/90f34cb0d83a6c5631cf71dfe64cc1054598c843a92b400e55675cc2ac37/pip-18.1-py2.py3-none-any.whl
Installing collected packages: pip
  Found existing installation: pip 10.0.1
    Uninstalling pip-10.0.1:
      Successfully uninstalled pip-10.0.1
Successfully installed pip-18.1
22:31 ~/Downloads curl -sSL https://raw.githubusercontent.com/sdispater/poetry/master/get-poetry.py | test/bin/python
Retrieving metadata

Installing version: 0.11.5
  - Getting dependencies
  - Vendorizing dependencies
  - Installing poetry
An error has occured: Command '('/Users/hermann/Downloads/test/bin/python', '-m', 'pip', 'install', '--upgrade', '--no-deps', '/var/folders/hk/jzlc3y0x0svcny3v758kmvwr0000gn/T/poetry-installer-v1mvs_fr/poetry-0.11.5-py2.py3-none-any.whl')' returned non-zero exit status 2.
Processing /var/folders/hk/jzlc3y0x0svcny3v758kmvwr0000gn/T/poetry-installer-v1mvs_fr/poetry-0.11.5-py2.py3-none-any.whl
Installing collected packages: poetry
Exception:
Traceback (most recent call last):
  File "/Users/hermann/Downloads/test/lib/python3.7/site-packages/pip/_internal/cli/base_command.py", line 143, in main
    status = self.run(options, args)
  File "/Users/hermann/Downloads/test/lib/python3.7/site-packages/pip/_internal/commands/install.py", line 366, in run
    use_user_site=options.use_user_site,
  File "/Users/hermann/Downloads/test/lib/python3.7/site-packages/pip/_internal/req/__init__.py", line 49, in install_given_reqs
    **kwargs
  File "/Users/hermann/Downloads/test/lib/python3.7/site-packages/pip/_internal/req/req_install.py", line 760, in install
    use_user_site=use_user_site, pycompile=pycompile,
  File "/Users/hermann/Downloads/test/lib/python3.7/site-packages/pip/_internal/req/req_install.py", line 382, in move_wheel_files
    warn_script_location=warn_script_location,
  File "/Users/hermann/Downloads/test/lib/python3.7/site-packages/pip/_internal/wheel.py", line 514, in move_wheel_files
    for row in sorted(outrows):
TypeError: '<' not supported between instances of 'int' and 'str'

22:32 ~/Downloads [2]

Bug in pip -- see https://github.com/pypa/pip/issues/5868

There has been a discussion in pypa/pip#5868 regarding this issue, and based on https://github.com/pypa/pip/issues/5868#issuecomment-430060402, it seems like the bug is in Poetry. If the maintainers agree that this is the root cause, it's an easy fix.

Actually this is not an issue in how Poetry builds wheels, so there is nothing to be done in the code.
It actually is an issue in the current installer (get-poetry.py). This has already been fixed on the develop branch by the implementation of a new installer (see #378) and the next 0.12.0 release will make this new installer available.
So, as far as I know, this is not an issue on pip's end.

@sdispater thanks for the update! I think there is currently some discussion whether or not this "issue" is really an issue or just needs to be handled properly for all RECORD files. #378 will resolve this issue for Poetry though.
https://github.com/pypa/pip/issues/5868#issuecomment-430282534

This has been fixed now that the get-poetry.py installer has been updated.


Changing window size cuts off the large text in Focus Timer

On the focus timer screen the center text below the timer gets cut off after changing the window size:

Fixed https://github.com/jenius-apps/ambie/commit/f6acea9a767df1c65138f007516c848201791c69


Handle stacks from internal exceptions sometimes thrown by Firefox

Discussed in csnover/tracekit#49
Upstream fix in csnover/tracekit#50

:clap:


Broken link -- System Restore

"System Restore" link is broken.

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

ID: 9d7bd11a-e693-26a0-d374-5cfb1d2335d3
Version Independent ID: bc9a2fb3-fa85-4ec0-8e0f-6e7c4906262b
Content: Volume Shadow Copy Service
Content Source: WindowsServerDocs/storage/file-server/volume-shadow-copy-service.md
Product: windows-server
Technology: storage
GitHub Login: @JasonGerend
Microsoft Alias: jgerend

#assign:xelu86

Hey @BR77BE,
Thanks again for contacting Microsoft. We value your feedback.
A documentation change has been made updating the broken link in question and will be live after the next business day. We will now be closing this thread.
Thank you and have a most pleasant day. #please-close


lack of spaces in rendered parameter docs

project folder structure (tree -L 2)

.
├── LICENSE.md
├── MANIFEST.in
├── README.md
├── docs
│   ├── index.md
├── mkdocs.yml
├── prysm
│   ├── __init__.py
│   ├── _richdata.py
│   ├── bayer.py
│   ├── ...
├── setup.cfg
├── setup.py
└── tests
    ├── __init__.py
    ├── test_bayer.py
    ├── ...

mkdocs.yml configuration file contents

site_name: prysm documentation
site_url: https://prysm.readthedocs.io/
theme: material

plugins:
  - search
  - mkdocstrings

mkdocstrings version: [e.g. 0.10.2]

# Name                    Version                   Build  Channel
mkdocstrings              0.15.2             pyhd8ed1ab_0        conda-forge
pytkdocs                      0.11.1             py38h50d1736_0    conda-forge
jinja2                             3.0.1             pyhd3eb1b0_0
python                         3.8.11             h88f2d9e_1

I placed an identifier in the index as a first play with mkdocs and mkdocstrings.  My code uses numpy style docstrings.  The docs source
::: prysm.propagation.Wavefront
    rendering:
        show_root_heading: true

Rendered as

I expected line breaks between parameters, or the table format shown in the examples.  Did I make a mistake in the usage?  Any help is much appreciated.

missed the optional dependency pip install 'pytkdocs[numpy-style]'
It would be useful for pytkdocs to have this as a default feature - the docstring-parser dep is only 13kB and this is a quite major feature.  (I know making it optional was explicitly part of the PR's feedback...)

It would be useful for pytkdocs to have this as a default feature - the docstring-parser dep is only 13kB and this is a quite major feature. (I know making it optional was explicitly part of the PR's feedback...)

Sure it would be helpful for devs using the Numpy-style. But not for the others :smile:
I'm way more in favor of keeping dependencies to a minimum, or setting them as extras. Maybe we can reconsider if the Python handler is extracted from mkdocstrings and made optional as well.

I think you have to pick the users you want to target and optimize for those.  Users with numpy docstrings get nothing out of google style or rst-style docstrings and still have to have the machinery for those installed.  Since there isn't a de-facto way to write docstrings in python, I guess you have to pick your favorite kid and forget the rest (hide them in optional installs).
At the least, I would suggest adding the syntax for the optional installs to the installation part of the docs.
I think breaking out the primary "driver" and making it an optional dependency is very, very much not a good design decision -- I'm more willing than most users to dig into problems and find root cause (most would just turn away instead of spending > 1 hr figuring out how to make numpy-style docstrings work).  If this python targeting docs tool "didn't work" from basic installation because an optional dependency would needed, I would look elsewhere and never look back - that's a dead on arrival level of broken.

At the least, I would suggest adding the syntax for the optional installs to the installation part of the docs.

You are totally right, the docs have been improved since then but were not deployed yet. Sorry this was so troublesome to setup.
About a favorite kid: mine is Google. So it was the first style supported, and therefore is still the default. Google and RST styles are implemented without any additional dependencies, that's why they are natively available. A PR implementing Numpy-style parsing without any additional dependency would be welcome :slightly_smiling_face:
As for Numpy-style being the "primary" driver: we should probably get the numbers to proove that. I witnessed a certain amount of data-science oriented projects using mkdocstrings on GitHub, true, though didn't check the docstring style they use. I myself see Google-style more often than any other style, at least in the projects I contribute too. We are probably both biased towards a particular style.

I guess I just.......don't understand your viewpoint.  Why the hard line on this dependency?  It's not like the author of that PR pulled in Boost or opencv or some other behemoth that makes things unwieldly.
It demotivates me as a user to see apparent arbitrary top-down rules that spawn hours of debugging for prospective users of this tool when trying to use it to process an extremely common docstring format.

Again, sorry you spent hours debugging this. The next version of the docs will make it clear that you need the numpy-style extra. Maybe this could be set as an extra on mkdocstrings as well. If it is well documented, I don't see how that's a problem for users.


Fix test_Meshgrid

Probably this fixes #8268.  (Reopen the issue later if it does not.)
The outputs are six arrays of the shape (6, 5, 3, 3, 6, 3), which seems too big to test backward with float16, in the failed example.

Thanks. Were you able to replicate the original errors, and how many times did you run this to confirm ok?

Were you able to replicate the original errors, and how many times did you run this to confirm ok?

I've not tried it.  IMO, the PR removes some bad tests, while weakening tolerance of a test is often a trade-off action where the frequency of failures may be measured first.
The cause seems independent of backend because I found test failures with devices native:0 and cuda:0.

Repeated this change with --count 1000 and it failed once with input_arrs=4, indexing='ij', float16, cudaon the backward test. Maybe 4 is still too high?

To test indexing, I was not so brave to drop input_arrs=4.

indexing='xy' (default): 1, 0, 2, ..., N-1
indexing='ij': 0, ..., N-1

Jenkins, test this please

Thanks @hvy for measuring the failure rate.  I updated the PR comment to keep the issue open.

Jenkins CI test (for commit 1d11f5337d257c0d0ace72e0b90131164629eb55, target branch master) succeeded!


Add support for AWS Neptune

Pricing information is found here
https://aws.amazon.com/neptune/pricing/
I think pricing is specific to the instances. Neptune organizes instances in clusters, but afaik there is not pricing that is specific to a cluster. Terraform has modules for instance and cluster.

For neptune_cluster_instance:

Database instance (on-demand, <instance type>) with unit hours.  Iinstance type is determined from the tf instance_class.  The count field, if present, should be used to multiply the number of monthly hours.
For t3 instances, a monthly_cpu_credit_hrs usage parameter should be displayed as CPU credits with unit vCPU-hours.

For aws_neptune_cluster:

A storage_gb usage parameter displayed as Storage with unit GB
An monthly_io_request usage parameter displayed as I/O with unit 1M request
A backup_storage_gb usage param displayed as Backup storage with unit GB.  A similar parameter is used for documentDB and decides to display the price if backup_retention_period is present and > 1.  We should be consistent with that implementation.  This captures the footnote from the pricing page: There is no additional charge for backup storage of up to 100% of your total Neptune database storage for each Neptune database cluster. There is also no additional charge for backup storage if your backup retention period is 1 day and you don’t have any snapshots beyond the retention period.

For aws_neptune_cluster_snapshot:

A backup_storage_gb usage param displayed as Backup storage with unit GB.

@alikhajeh1, Hey, I want to work on it!

@caphrim007 just FYI this was released in v0.9.2 today, plz let us know if it works ok for you.


Propagate timestamp from docker-loghose

... to logzio-nodejs so that resulting log entry will have correct
timestamp. This is needed to order log entries properly in Kibana.

@lorf
What is the issue you encountered? logzio-nodejs adds @timestamp to the log...

The issue is that logzio-nodejs sends messages in batches (if I understand it right) and the timestamp it generates is the timestamp of a batch, and differs significantly from the time the message was generated at. Here the message was generated at 11:59:35.524, but @timestamp is 18:59:37.829:
October 8th 2018, 18:59:37.829 [2018-10-08T11:59:35.524Z] INFO  4918920e2c7e-1/BROKER: Moleculer v0.13.3 is starting...

This leads to confusion, but most importantly, Kibana sorts messages by that timestamp and as a result I see the messages in a completely random order within a batch in Kibana. This is seen using official logzio-docker docker image.

This change, however, does not solve the sorting issue completely, e.g. backtrace lines usually have same millisecond timestamp, so backtraces are completely messed up in Kibana.

@lorf
We have a nanosecond option in the nodejs shipper so it might be helpful to use it in your case.
Can you please update version to 3.0.0 and the readme file - mention that "field time is not present anymore from version 3.0.0"

@idohalevi Sorry for a delay.
I suppose @timestamp_nano in logzio-nodejs has the same batching problem. Since it's a string It also has the problem with alphabetical sorting of numbers (1234 will be less than 567 when compared alphabetically).
I reverted to not to delete time field for compatibility reasons, do I still need to bump the version?

@lorf
Sorry for the late response. Approved


After logging in, the dashboard display delay

In GitLab by @luotian-github on Oct 26, 2017, 15:33
If you are reporting a problem, please make sure the following information are provided:

Version of board (last git commit number) , docker engine and docker-compose.
Config files of board, you can get them by packaging "board.cfg" and files in the same directory, including subdirectory.
Log files, you can get them by package the /var/log/board/.
Other necessery information.

2017/10/20 git

In GitLab by @sokril on Nov 8, 2019, 08:13
closed


Build Golden Mode

We need to be able to run a ttnn model using only pytorch fallbacks associated with every operation.  The idea here is to provide a way to completely bypass running on device.
This feature will initially be used to help check the models, however the expectation is that this feature will also be beneficial when we integrate/upgrade to pytorch 2.0 as well.

Merged to main


[Bug]: Huge performance issue since the last upgrade

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Since the last version update to v3.10.16, we're facing strange performance issues, specifically during the build process.

Steps To Reproduce
Just upgrade to the latest version
Version
3.10.16

For now, I need to scale up the server. Anyway, had to shut down to rescale

Is this still persists? If yes, could you please give me an example repository, so I can check it?
Please reopen this issue if needed.


build: bump node to v20

Fixes the deprecation warning:
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: batista/lint-filenames@1b766f13475cbdd2df8f2b75d8afda9a48ce7d84. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.

@batista any chance you could review this PR?

@martinm82 thanks for the ping! I haven't been watching this repo closely for a while, I'm reviewing it now.

@batista thanks a lot. Any chance you could release a new version of the action?


v3.0.0 Release

Change Summary
Overview

This PR marks the v3.0.0 release.

Oh yeah, this project isn't auto-versioned so we really should have done this before the release. Oh well, we can try and remember it for next time.

Oh yeah, this project isn't auto-versioned so we really should have done this before the release. Oh well, we can try and remember it for next time.

Agreed. Before next release I also would like to change the name of some of the tags/releases in this repo -- right now we have inconsistent use of v in our release/tag versions


Speed up scenechange

15-20x faster
On qcif:
before:
encoded 600 frames, 39.575 fps, 52.29 Kb/s

after:
encoded 600 frames, 44.320 fps, 52.29 Kb/s

Because the code isn't implicitly upscaling chroma the results for high resolution might diverge a little, discussion regarding downscaling luma and/or decide the weight between luma and chroma are open.

Coverage decreased (-0.02%) to 75.378% when pulling b3ee6fbd7ca6e92bcfe9a1294fb730f076332d90 on rust-av:scenecut-speedup into 90c46ec0f9c9ef9bedccba4339244564de278c68 on xiph:master.

Ah, you'll need to fix rustfmt however.

Formatted, I'd land it today.

Good news! With the chroma scaling, no change in AWCY.
Bad news, rustfmt and clippy are complaining again.

I'll fix and merge once I'm back, the weighted version is still worse?
On Thu, Nov 14, 2019, 20:08 Josh Holmer<EMAIL_ADDRESS>wrote:

Good news! With the chroma scaling, no change in AWCY
https://beta.arewecompressedyet.com/?job=master-scenecut%402019-11-14T03%3A15%3A23.198Z&job=sceneut-speedup-chroma-dec%402019-11-14T05%3A56%3A41.206Z
.
Bad news, rustfmt and clippy are complaining again.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/xiph/rav1e/pull/1848?email_source=notifications&email_token=AAB2LJGD76FKIH5VZPTNLWLQTUWUVA5CNFSM4JMWTP3KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEEBO3CY#issuecomment-553840011,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAB2LJGLJT73V3YCM7B6VJTQTUWUVANCNFSM4JMWTP3A
.

The earlier version was still worse, yes. Although I have a suspicion that things like scrolling credits interact in a strange way with this algorithm--they look a lot like constant scene flashes, especially if luma is prioritized more heavily.
If we want to work around this, it probably requires revisiting using cost estimates to determine where scene changes should happen, but I'm not sure if it's worth investing the time in that at the moment, given that it's inherently much slower than this approach. (We might be able to do it much more cheaply and accurately though if we can dedup some of the code that's shared between lookahead and encoding.)


[Performance] [Speculative decoding] Speed up autoregressive proposal methods by making sampler CPU serialization optional

Background
Speculative decoding leverages the ability to cheaply generate proposals, and cheaply verify them to achieve speedup for memory-bound inference. Different methods of speculative decoding explore the frontier between cost of proposal, alignment with the target model, and cost of verification.
For example, Medusa produces very cheap proposals, but the quality of the proposals are strictly less than Eagle because the heads do not have access to the previous proposals. Eagle on the other hand pays more for the proposals by sampling autoregressively instead of 1-shot, but it brings the benefit of higher-quality proposals.
At the end of the day, what the user cares about will dictate which speculative technique is used. vLLM's job is to provide them with the option for best speedup for their use case.
Draft-model, EAGLE, and MLPSpeculator rely on autoregressive proposals. This means their top-1 proposals are higher-quality than Medusa, which gives vLLM an ITL reduction that is more flops-efficient than Medusa. This is what our speculative decoding efforts are focused on first -- afterward, we can support top-k proposals with Medusa so users who care more about ITL reduction can use vLLM.
Speedup autoregressive proposal methods
This issue is to speed up autoregressive proposal methods by optimizing the sampler. Specifically, the sampler performs wasted work by copying sampled values to GPU and serializing them into Python objects. In speculative decoding, we never use the python objects because we consume the raw sampled token ids / probabilities in their GPU tensors. This means that the copy and CPU serialization are pure overhead in speculative decoding.
How much overhead?
In profiling vLLM, I found that copy + serialization in the draft model takes ~441µs (cell J30). Note that the actual forward pass and sampling math of the draft model take (220µs + 639µs) = 859µs. This means that by removing the unnecessary copy and serialization, we can get 50% more draft tokens in the same time it takes with the copy and serialization enabled.
This difference is actually massive on the overall performance of speculative decoding.
Furthermore, the subsequent draft model forward pass must consume the output of the previous step. This allows us to reduce time spent in prepare_inputs. I don't have numbers here, but I expect a further ~150µs reduction per draft model step by this (~300µs to ~150µs).
The work
This issue is to:

Make the CPU copy and CPU serialization optional in vLLM's sampler (thus leaving sampled token ids on GPU), and then
passing those sampled token ids to prepare_inputs of the next draft model forward pass.

1. Make CPU serialization optional
Warm up task: Note a good warmup task to get familiar with the Sampler is to add an option to disable logprobs for a given Worker. This will also provide some speedup to spec decode (~2ms e2e step time), but isn't part of this issue.
Code pointers:

The sample method which does CPU-GPU synchronization and serialization
The actual for loop which does synchronization and serialization

2. Allow prepare_inputs method to work on-device
The on-gpu sampled token ids should be appended to the next prepare_inputs batch.

The prepare_inputs method which will need to consume inputs from GPU in this case.
Our fork version that does this

@Yard1 has a good point: the north star here is we can CUDA graph the proposal method, and use an on-device mechanism to prepare input for the next fwd pass. We should get there iteratively, as it should also work for other proposal types (eagle which uses a different flavor of prepare inputs).

Thanks for writing it up! I'll get started on this.

I'll be working on (2)

Hi @cadedaniel @alugowski, I have been looking into the CPU serialization overhead part of this. Is there any work already started on this? I'm available to collaborate on this, so do let me know how I can help.

#6338 is covering this now :)
cc @alexm-neuralmagic

Closed by @alexm-neuralmagic and @comaniac


Move all test to test folder + pytest parametrization

Up to now, some tests are at the end of the specific files and some are defined in the test folder.
We should move all of them to the test folder and use whenever needed the pytest parametrize functions

Maybe we should also remove all the unused prints once we move everything to the test folder or replace them with asserts + intelligible error messages

We should also cover in the tests the several checks that we may have that could raise errors
i.e. using pytest.raises
with pytest.raises(ValueError, match='What an horrible error!'):
      some_command_meant_to_raise_an_error()


Remove superfluous run.join

do not merge yet
I'm not sure the purpose of the Ember.run.join that was adding around pushing component cards, but its presence causes an issue with ember-wormhole that manifests when using a mobiledoc parser (e.g., when pasting in an image). I'm still pinpointing a focused test case for this bug.

Because the add callback is executed by Mobiledoc-Kit, we don't know for sure there is a runloop.
join uses a current runloop if it is present, and if it isn't present it creates one source here. There are possible timing issues for sure, but I'm certain there are (at the least) theoretical cases where a runloop needs to be created. In most practical ones there will already be a runloop, yes, but join should handle those cases just fine.
Seems like a tricky corner case, nice find.

@mixonic thanks for looking. This is cleaned up and has a test added. Ready to merge, if you want to :eyes: again.

Really great. Thanks sir.


JavaScript Formatter

I have
'There is no document formatter for 'javascript'-files installed.'
How can I solve this problem?

Thx!
solved.


ImportError: torch.utils.ffi is deprecated, how to use cpp extensions instead?

When I run python build.py, there is an error saying torch.utils.ffi is deprecated, is there an updated version available which uses cpp extensions? If not please let me know if you have any advice on how to update this code, thanks.

ah, I realised I wasn't using the required version of torch==0.4.1. I have change my torch version and I am no longer getting this error


FR: Search, open & close tab groups

Since chrome supports tab groups now, it would be great if I can search for a tab group in quick tabs. Also, we should be able to open/close the whole tab group from quick tabs.

I agree, I think that makes a ton of sense. I have to find some time to check out the API and see what's possible.
Evan

+1 for searching by tab group name by default and via a / command.

 As much as I'm a backend engineer (therefore not js expert) I would love to contribute to this feature if you are willing to have some help.

@babyman if you give me some general directions I can try to work to implement this feature.


test: add verification api integration tests

Description:
Test for ATL-6400
A bug was found

Checklist:

[x] My PR follows the contribution guidelines of this project
[x] My PR is free of third-party dependencies that don't comply with the Allowlist
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have checked the PR title to follow the conventional commit specification

you can merge the megalinter fixes
https://github.com/hyperledger/identus-cloud-agent/pull/1202


[SPARK-8402][MLLIB] DP Means Clustering

DP means is a non-parametric clustering algorithm that uses a scale parameter 'lambda' to control the creation of new clusters. This algorithm helps to cluster data points without specifying the number of clusters in advance.

Merged build triggered.

Merged build started.

Test build #35125 has started for   PR 6880 at commit 0c0a478.

Test build #35125 timed out     for PR 6880 at commit 0c0a478     after a configured wait of 175m.

Merged build finished. Test FAILed.

@mengxr Could you please say your comments on this PR ?

@FlytxtRnD I haven't checked the implementation yet. Some high-level comments:

Please follow the code style guide. I saw wrong indentation, extra spacing, vertical alignment in your code.
Move save/load and the example code to follow-up PRs. Keep this PR small to accelerate the code review.
Check the generated API doc. Usually this is the simplest way to find public APIs that should be private.

On the algorithm part, could you list a few successful stories about k-means vs. kp-means? Some benchmark result also helps.

Merged build triggered.

Merged build started.

Test build #36766 has started for   PR 6880 at commit 907f4f1.

Test build #36766 has finished for   PR 6880 at commit 907f4f1.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
  case class Params(

Merged build finished. Test PASSed.

@mengxr I have reduced the PR length so that it would be easier for you to review. The style issues have been fixed wherever they were observed.
I will change the paper name in the next update and  the benchmark results will also be ready asap.
Could you please review this updated PR and  give suggestions?

@mengxr  Could you please tell me how to generate the API docs? I run build/sbt unidoc as mentioned in https://github.com/apache/spark/blob/master/docs/README.md. But it ends in an assertion error. Please help.

@mengxr @jkbradley Gentle remainder.

@FlytxtRnD To generate the docs, I've always used jekyll (following the instructions on that same page).  I know that builds more than you want, but does that at leasdt work?
Sorry this PR is having to wait a bit for full review!

@jkbradley To generate docs, I installed jekyll.  In jekyll build command, it is showing error.
[info] Done updating. [error] (catalyst/compile:compile) Compilation failed [error] Total time: 601 s, completed 28 Jul, 2015 4:25:49 PM Moving back into docs dir. Making directory api/scala cp -r ../target/scala-2.10/unidoc/. api/scala jekyll 2.5.3 | Error:  No such file or directory - ../target/scala-2.10/unidoc/.
But SKIP_API=1 jekyll build is successfully completed. Could you please help me to solve this?

@FlytxtRnD You might need build/sbt clean first. Given the review bandwidth, we may not be able to make this into 1.5. So I will make another pass after the 1.5 feature freeze. In the meantime, it would be super helpful if you can help review some other PRs that are on the 1.5 roadmap, e.g. https://github.com/apache/spark/pull/5267 (bisecting k-means). Thanks!

Thank you @mengxr . We will take a look into the PR you mentioned.We are looking forward to have DP-Means in the 1.6 release. Thanks a lot for your kind support.

@mengxr We have updated the JIRA ticket to include the benchmark results as well..Could you please take a look and give your suggestions?

@FlytxtRnD I made another pass. Please follow the code style guide closely: https://cwiki.apache.org/confluence/display/SPARK/Spark+Code+Style+Guide. I will make another pass on the implementation after your updates.

@mengxr Thank you for all the suggestions. Will update soon

Merged build triggered.

Merged build started.

Test build #42899 has started for   PR 6880 at commit e796866.

Test build #42899 has finished for   PR 6880 at commit e796866.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
  case class Params(
      case class WeightedPoint(vector: Vector, count: Long)
class DpMeansModel(

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

Merged build finished. Test PASSed.

@mengxr @jkbradley I have incorporated the suggestions and changes and updated the PR. Could you please take another look ?

@mengxr Could you please have a look into this?

Thank you @yu-iskw for the review comments.. Will update the PR asap

Merged build triggered.

Merged build started.

Test build #44917 has started for PR 6880 at commit b088e46.

@yu-iskw PR is updated. Shall I include @since to the methods? Or is it done after getting merged? Please provide any other suggestions, if any.

Test build #44917 has finished for PR 6880 at commit b088e46.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  *   case class Params(\n  * class DpMeansModel(\n

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/44917/
Test PASSed.

@FlytxtRnD thank you for the update. We should add @since tags in the first commit.
Btw, I haven't read the original paper carefully yet. I'll review this PR in terms of the algorithm later.

@yu-iskw I didn't get your comment on @Since tags. We will be waiting for further review comments.

@yu-iskw  @jkbradley any other review comments, please.

Test build #54751 has started for PR 6880 at commit b088e46.

Test build #54751 has finished for PR 6880 at commit b088e46.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/54751/
Test FAILed.

Test build #55098 has started for PR 6880 at commit 23316d4.

Test build #55098 has finished for PR 6880 at commit 23316d4.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/55098/
Test FAILed.

Test build #55108 has started for PR 6880 at commit c25eae2.

Test build #55108 has finished for PR 6880 at commit c25eae2.

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/55108/
Test PASSed.

Test build #69330 has finished for PR 6880 at commit c25eae2.

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/69330/
Test PASSed.

I'm closing this few-year-old PR because the corresponding JIRA (https://issues.apache.org/jira/browse/SPARK-8402) was closed as "Won't Fix".


Some code style fixes

Looks good, have couple of suggestions.

@mucaho I have made the second commit, tell me your opinion on it - if it's not ok, i'll fix if; and if it is okay - i'll rebase 2 commits into one.

This PR inspired me to open #951. We will hopefully fix code styles automatically soon, so I suggest you don't bother too much with them, as there are ~25 more files that need these fixes too :)

Ok, I see. revert indentation changes?

Sorry for the long wait, I have been away from my working PC.
Did a final review, added some documentation code highlighting and merged with a6acb0df73a6fc51c62f415c8c2ed24ebd19d26b.


Functionality to create compute instances with managed identity.

Python SDK V2 Template for creating compute instance
Compute instance can be created with no managed identity / system assigned managed identity / user assigned managed identity
Associated parametrized YAML template for compute instance creation

@microsoft-github-policy-service agree company="Microsoft"


[Coverity CID :203537]Error handling issues in /samples/subsys/nvs/src/main.c

Static code scan issues seen in File: /samples/subsys/nvs/src/main.c
Category: Error handling issues
Function: main
Component: Samples
CID: 203537
Please fix or provide comments to square it off in coverity in the link: https://scan9.coverity.com/reports.htm#v32951/p12996

from b1e45b413ef #6391

This looks 'Fixed' in the report page (link below). Can someone confirm? @aasthagr
https://scan9.coverity.com/reports.htm#v39298/p12996/fileInstanceId=55470800&defectInstanceId=6797685&mergedDefectId=203537

Hi @jenmwms, this is not really an issue. However if you would like to fix it completely all calls of nvs_write() or nvs_delete() that are not using the return value can be changed to (void)nvs_write and (void)nvs_delete.

Thanks @Laczen!

I contemplated a patch that checked the return values, but it is bulky and checking the return in that context does not appear to add value. So it is cleaner to simply change those few calls to (void).


Algolia Search doesn't index code snippets/masterclasses

Similar to #3005

https://docsearch.algolia.com/docs/required-configuration/
https://docsearch.algolia.com/docs/tips/
https://discourse.algolia.com/t/best-practice-for-using-docsearch-on-a-closed-source-site/15325
https://www.algolia.com/blog/product/algolias-top-10-tips-to-achieve-greatly-relevant-search-results/
https://discourse.algolia.com/t/what-is-the-best-way-to-index-more-generic-html-documents-in-algolia/17387
https://discourse.algolia.com/t/whats-the-recommended-approach-to-recreate-docsearch-records/16292


Building docs: requirements.txt missing

In sklearn-evaluation/docs/README.md, it mentions

Building docs
Install requirements:

pip install -r requirements.txt

But there's no requirements.txt in this repo.

right,
I think this should work (note that this requires conda):
pip install invoke
invoke setup

can you give it a try? if it works, please open a PR with the updated instructions

right,
I think this should work (note that this requires conda):
pip install invoke
invoke setup

can you give it a try? if it works, please open a PR with the updated instructions

It worked:

Opening a PR right now :)


How to make login session persist?

When a user logs in (by clicking the Login with Google button), and then refreshes the page, they are asked to login again. How do we make this login state persist across website refreshes?
This issue can be seen in the https://subscription.streamlit.app/ app

If I'm not mistaken, streamlit doesn't provide cookies and that's why the user loses the login after refreshing the page.

yeah this is exactly it, i've been trying to add it to cookies but that doesn't work either, because then someone could just edit their cookies to someone who is subscribed and get in easily. So it would be way less secure that way.

closing this as it is a dupe of https://github.com/tylerjrichards/st-paywall/issues/23


[Q]Graphite render API from and until time understanding, from is not working as expected

Hello Team,
We are using graphite to store the api hit count, the retention policy setup in our system is
[sample metrics]
pattern =^metrics\.(.*)\.api.*
retentions = 10s:5m,1m:1d,1h:30d,1d:1y,30d:10y

Aggregation
[default_sum]
pattern = .*
xFilesFactor = 0.5
aggregationMethod = sum

We have some scheduled jobs which run at every 30 minutes and pick the data from render api for 30minutes , suppose say if job is running at 12:30 AM , then we will frame the from time as 12:00 and until time as 12:30PM
http://localhost:32323/render?target=sum(metrics.*.*.*.*.*.get.*.*.success.*.count)&format=json&noNullPoints=true&from=12:00_20200519&until=12:30_20200519 `
Below are some question

is there any issue getting the data for every 30 minutes , based on the retention policy , is i will get duplicate count for this
regarding from time , suppose say we have data points at 12:00:10 , 12:00:20, 12:00:30 second then after first archive the data points will merge to one datapoints ie will be 12:00 , when we fetch the data "from=12:00_20200519" then i am not getting the datapoint which is stored at 12:00 which is giving the data from 12:01 so here i am loosing the data for 1 minute
Can you let me know is graphite works on below principle
from > 'date' and until <= 'date'
how should i cover the 1 minute data loss from render api

Hi @panditb

TBH I didn't really understand a question, could you please elaborate?
2,3,4. I'm not aware of such behavior. Do we really aggregating data point in next archive in whisper with 1 point forward? @DanCech @piotr1212 ?

Isn't it due to https://github.com/graphite-project/whisper/issues/230


Print Hello world without using semi-colon

What the program do?
It is a question asked in a interview. It is related to competitive programming.
In what programming language it is written?
C++

This pull request exeeds the max additions limit. Please fix it and reopen


-o is used for both --overview and --outage

One should have another letter then.

Only relevant when adding --outage.


Does not work on pandora!

Pandora has an ancient version of g++ (6.3.0) and thus cannot compile the code.
The problem is mostly from the tuple deconstructor
(auto [sfile, package, name] = createFile() => line 440)

Problem fixed


Eclipse scala IDE shows errors related to BuildInfo and conf . workflow executor/assembly also fails

The error is caused by an SBT plugin that puts a commit's hash into the code. Since the .git directory is missing, I assume that you have downloaded the source code via "releases" subpage using one of the tags.
To solve the problem please checkout the repository.
git clone git@github.com:deepsense-io/seahorse-workflow-executor.git
If you want to use 1.0's code then please check out the appropriate tag:
git checkout v1.0.0
Also, consider checking out 1.1 branch as it contains new features and bugfixes. Furthermore, you can also take a look at master branch (our development branch).
If you are interested in using rather than building then probably you want to download a Vagrant image with ready-to-use Seahorse from http://seahorse.deepsense.io/downloads.html


Systray with option to quit program?

Good suggestion that sounds useful, I think I'll add that to the next version.


Mat/Pat: Show workflow based on eligibility using appointment status from ODS

User Story
As a mat/pat applicant I want go to the appropriate workflow based on my eligibility that is decided by my appointment status
Acceptance Criteria

[ ] In the basic info screen, if appointment status is ~J or K~ K or L, go to the not eligible screen
[ ] For all other appointment statuses go to the next screen in eligible workflow

Design
Employee Info - https://miro.com/app/board/uXjVOWaGSE0=/?moveToWidget=3458764544376295648&cot=14
Not eligible screen - https://miro.com/app/board/uXjVOWaGSE0=/?moveToWidget=3458764543687588220&cot=14
Miro board - https://miro.com/app/board/uXjVOWaGSE0=/?moveToWidget=3458764543687588226&cot=14

@Stella-Archer
I believe you are looking for appointment status and category based on the values you shared.  That is available in the existing endpoint used by the Telework forms: https://analytics-testapi.psa.gov.bc.ca/apiserver/api.rsc/Datamart_Telework_employee_demographic/
Column names: appointment_status and empl_ctg

@T854236 I can't seem to access this link. I tried with my idir and password. Do i need to put ina request to Aki and the team to get access?

@HasanSifat
If you are looking to make a call to the API endpoint, yes you'll need an account.  Perhaps leverage the one provisioned for Telework?
If you are only looking for information, you can access it here: https://analytics-testapi.psa.gov.bc.ca/apiserver/api.rst#Datamart_Telework_employee_demographic

Thanks @T854236 this is much more helpful

Connie updated us that it is 'K' and 'L' that do not receive benefits.

I checked our API and we currently not getting the employee's appointment status from ODS. We need to add that to the backend and then will be able to use it on the form.

@iman-jamali-fw
To confirm, you are wanting appointment status and not employment status?

@T854236
Yes, based on the ticket, appointment status.
By the way, I assumed this is included in the https://analytics-testapi.psa.gov.bc.ca/apiserver/api.rst#Datamart_Telework_employee_demographic. I didn't test out that endpoint, having difficulty connecting to BC Gov VPN on my EY laptop. I just confirmed that we currently not recording it in our backend (to be passed to the form).

@iman-jamali-fw
Hi Iman, there is an appointment_status field with possible options: Regular, Auxiliary and ABC.  Is that what you were looking for?

Tested and verified.
Not eligible screen appears if 'k' or 'l' is typed in the appointment status. Although, if the alphabets are capitalized, the not eligible screen does not appear. Not sure if that makes a difference.

@ayushdamani good catch. The statuses will autopopulate capital K and L. Fixed the conditional.

This is not working in test. Seems like the logic has not been implemented in test yet
Recording

Seems like it reverted back for some reason. Fixed it again.

The bug with capitalized K & L still exists not going to "not eligible" screen still exists
If R, the next screen is confirmation page & not the Alternate contact info screen

Recording ->
https://bcgov.sharepoint.com/:v:/r/teams/03991/Shared Documents/Maternity and Paternity/Test Screenshots %26 Recordings/Screen Recording 2023-04-24 at 2.06.37 PM.mov?csf=1&web=1&e=m4tYOX

capitalized K and L work for me. It leads to the not eligible screen in test. Looks like the confirmation page was added when I was gone, do you know when it is supposed to be shown? Is this the confirmation page for K and L people?

Just checked, capitalized K & L are not working for me in the test environment.
Lower case 'k' and 'l' are working perfectly.

Capitalized K & L are working. Considering the field will not be editable and the value will be pulled from DB, we should be okay.
Pending issue -

R is still going to confirmation screen, instead of alternate contact screen. confirmation screens should only come on submission

the confirmation page is not part of the form, so I have hidden it.
It will be a success page redirect like the other forms.

@chriscaldwell-psa
The label "Date/Time" should be "Start Date" on the not eligible page

Works as expected


Promotes on aiMove

Is there a way to know to what piece the engine promoted into  or it will always be a queen? since it returns {"A7": "A8"} for example.

If the player is played by computer, then it is always the queen.
If the player is played by human, it is always the queen by default, but it can be changed by setPiece  afterwards.
If you need to update/change this logic, just let me know.

Thanks, this is not a problem, I couldn't implement stockfish, their documentation is not so simple as yours


[Efficient Conformer] Support ONNX GPU export, add librispeech results, and fix V2 streaming decode issue

Support ONNX GPU export.

After exporting onnx, the loss of CER is small. Using the train_u2++_efficonformer_v1_stream.yaml with causal:true, the CER increases from 9.30% to 9.33% in our dataset.

Add librispeech results and conf:

train_u2++_efficonformer_v1.yaml
train_u2++_efficonformer_v2.yaml

Add streaming conf in aishell:

train_u2++_efficonformer_v1_stream.yaml

Fix bug of V2 streaming decode:

Fix issue https://github.com/wenet-e2e/wenet/issues/1690#issuecomment-1434867574

Many thx!


New architecture support

Is this usable by now? I want to copy off some of this library's code to another library

@MicroDroid Sorry, just now seeing this. The example is mostly working, but my intention is to wrap this work this week. I can ping you again when I get it wrapped.

No worries :) Yeah I'd appreciate a ping 👍

@MicroDroid Hey, I merged this work in. It's not super tested yet, though everything appears to be working in the example app...going to work on getting it moved into our app at some point here this week and will start working out kinks at that point.

wow that was fast, I'll try it out and let you know if something is broken (cause I tried this before the recent changes, and something was)👍
thanks either way :)


How to specify path to a Lua interpreter?

I want to use lqc with specific version of Lua interpreter, but I don't understand how to do it.
Shebang in bin/lqc prompt that always used lua interpeter found in $PATH:
#!/usr/bin/env lua

local app = require 'lqc.cli.app'
app.main(arg)

How to use specify path to another Lua interpreter without patching lqc source code?
Please advice.

It's been a while since I used Lua/this library, but in the CI I test my library against various Lua versions (5.1, 5.2, LuaJIT, ...).
The shebang uses /usr/bin/env instead of lua directly, which makes it possible to use different Lua versions.
If I remember correctly, your luarocks package manager is linked to your lua version. So if you have Lua 5.2 for example, it will install lqc for 5.2 (see installation instructions in README how to install this library).
Hope this helps. :smile:

@luc-tielen Yeah, looks like it is possible to set any interpreter via env variable LUA.
BTW is it possible to use library without cli runner?

Yes that's possible, in the end the CLI calls this lua file, which should be installed via luarocks as well.

Thanks!


feat: recommend a TC remediation per Dependency' issue

Hi @ruromero ,
I'm on PTO today, will return tomorrow, this is the final part, in the meantime i neutrilized the relevant tests, as i still working on them, will finish them tomorrow morning.
in the meantime, please review the code.
Thanks and have a good day.


Some null fields in the EIA860 data

Describe the bug
There are some null fields in a few areas of the EIA860 data. I reinitialized my data this morning. I believe I do have the latest git repository on my machine as well (downloaded last night). I tracked down a few examples of what I am seeing.
To Reproduce
Please see the screen shots attached. As one example, minimum load is null for one unit of Plant EIA ID 54.
For another plant, 6483 (EIA code), the state/county are null.
This was not the case previously. In the raw EIA 860 data this information is not null.
Thanks for the help! Let me know if you need more information from me or you believe it is a potential setup issue on our side.
Best,
Bri

For the Minimum Load it looks like the nulls are showing up where EIA860 reports 0MW if that is anything helpful.

Hello there, I'm just following up on what I've noticed here. Here's another example of where I see this happening:
SELECT --sum(g.capacity_mw)
g.plant_id_eia,
g.generator_id,
g.technology_description,
p.plant_name,
p.county,
p.state,
p.latitude,
p.longitude,
g.capacity_mw,
g.nameplate_power_factor,
g.summer_capacity_mw,
g.winter_capacity_mw,
g.minimum_load_mw,
pm.operating_date,
g.retirement_date,
g.time_cold_shutdown_full_load_code,
pm.prime_mover_code,
g.energy_source_code_1
FROM generators_eia860 g
inner join plants_entity_eia p on g.plant_id_eia =p.plant_id_eia
right join generators_entity_eia pm on (g.plant_id_eia =pm.plant_id_eia and g.generator_id =pm.generator_id)
where
g.report_date = '2017-01-01'
--and f.report_date = '2017-01-01'
--and g.energy_source_code_1 in('BIT', 'RC','SGC','SUB','LIG')
and g.plant_id_eia = '621'
--and pm.prime_mover_code = 'ST'
--and f.prime_mover_code in ('ST')
--and f.fuel_type ='BIT'
and p.state not in ('AK', 'HI')
and g.operational_status_code in ('existing','OP','OS','SB','OA')
order by g.plant_id_eia, g.capacity_mw;
I had to repopulate my database recently. I wasn't seeing this before then though. Maybe I should re-initialize?

Thanks for the help!

Hey @briannacote! Sorry for the slow response on this!! Most of these harvested fields are not perfect. We choose to pull the most consistent record as reported across all the EIA tables and years, but we also required a "strictness" level of 70%. That means at least 70% of the records must be the same for us to use that value. So if values for an entity haven't been reported 70% consistently, then it will show up as a null value. I built in the ability to add special cases for columns where we want to apply a different method to, but the only ones we added was for latitude and longitude because they are by far the dirtiest.
Overall this results in harvested records for upwards of 98% when all of the years are being ingested, and higher rates for single years of data. Every column but the lat/long columns must produce 95% of acceptable values otherwise the initialization would fail due to assertions in the code. Because lat/long is much more messy we set this constraint to 92% for those columns.
We decided that for now this was generally acceptable based on the low number of null fields, but has this been a blocking issue for you? I think the simplest way to get around this would be to select a few columns that are causing you trouble and run them through some special case harvesting process. The absolute easiest would be to run them through the same selection process but with a lower strictness rate would decrease the number of null fields, but I'm not sure if that is actually a good idea. The other most obvious option is to choose the most recently reported record if there isn't a consistently reported value. Again, that wouldn't be perfect by any means but it would certainly reduce user confusion.

That makes sense. I like the idea of using the last record if it does not pass the strictness test, perhaps with a flag that indicates that? Does the test include years for when the plant was in the "proposed" dataset? I think that dataset often doesn't include lat/long or much location information... Thanks!

I know we compile the proposed and retired tabs all into one table, so I think so.. I don't remember exactly. Although the harvesting selection process does ignore null values, so I don't think that's the problem. When I was building out the harvesting process I spent a while trying to determine why the lat/long was so messy and it seemed like there were just truly tons of inconsistent records without any major pattern.
I'm also more and more liking the take the most recent record method as the fall back. It will almost certainly be incorrect some of the time, but it will match what is reported and that will cause much much less confusion.

subsumed within Issue #446


request to provision "Microsoft 365 learning pathways"  template failed

Please report the issue here providing the following provisioning action correlation ID: 734d8e6a-e736-47ac-9966-3f673103d2a9 !
My issues seems the same as https://github.com/SharePoint/sp-provisioning-service/issues/236
I never get the window 'Permissions requested'.

Hi,
Thanks for your feedback. Most likely the default language of your tenant is not in the list of supported languages.
Here are the localizations that the template supports:
LCID="1033" Name="English (United States)"
LCID="1031" Name="Deutsch (Deutschland)"
LCID="1036" Name="français (France)"
LCID="1049" Name="русский (Россия)"
LCID="2052" Name="中文(中国)"
LCID="1041" Name="日本語 (日本)"
LCID="1046" Name="português (Brasil)"
LCID="3082" Name="español (España, alfabetización internacional)"
LCID="1040" Name="italiano (Italia)"
Please, let us know if this the case. Thanks.

Thank you for replying.
Tenant language is Dutch.
As requested on https://docs.microsoft.com/en-us/office365/customlearning/custom_provision_ml  the Tenant Admin account doesn't have a language selected.
It is alright that the language of the Learning Path is English.


chore: switch to travis

can you please test with https://github.com/ipfs/aegir/pull/334 to make sure it works only with npx aegir dep-check


Improve Operator Docs: Prerequisites and Current Testnet Parameters

General improvements to style and formatting.

I'm going to merge this one. We can revisit whether we want to add VERSION to env later.


Make package names doc links

See if this helps prevent the initial misclick through to package docs

Can you put <docs> pill next to each docs link (package name), and add a GitHub icon next to the <code> pill?

I can shift the  pill to the left, which will keep them all aligned:

I can add the GitHub cat but that (a) I don't know how to float right (b) haven't put effort into finding out because I think having three pills/icons is a bit much and will confuse users that clicking a pill takes them somewhere but clicking the cat opens the panel

Another idea, how about adding a TOC (of depth 1) in each collapsible package box? So I can jump to "reference documentation" that I'm interested in (99.99% times).

I can do something like this:

@JoelPasvolsky, I like how the <docs> pill looks on the left.
Re: github icon -- I meant we group it with the <code> pill, e.g. <(octocat) code>, or (octocat) <code>, so it's clear the link will take you to GitHub source...

Re: github icon -- I meant we group it with the <code> pill, e.g. <(octocat) code>,

I had a look at doing that but don't see that bootstrap pills support an icon


[HUDI-1969] Support reading logs for MOR Hive rt table

Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contributing.html before opening a pull request.

What is the purpose of the pull request
(For example: This pull request adds quick-start document.)
Brief change log
(for example:)

Modify AnnotationLocation checkstyle rule in checkstyle.xml

Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Codecov Report

Merging #3033 (7b1f0e5) into master (a658328) will increase coverage by 15.69%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             master    #3033       +/-   ##
=============================================
+ Coverage     55.13%   70.83%   +15.69%
+ Complexity     3866      385     -3481
=============================================
  Files           487       54      -433
  Lines         23613     2016    -21597
  Branches       2528      241     -2287
=============================================
- Hits          13020     1428    -11592
+ Misses         9436      454     -8982
+ Partials       1157      134     -1023

Flag
Coverage Δ

hudicli
?

hudiclient
?

hudicommon
?

hudiflink
?

hudihadoopmr
?

hudisparkdatasource
?

hudisync
?

huditimelineservice
?

hudiutilities
70.83% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...apache/hudi/common/fs/HoodieWrapperFileSystem.java

...java/org/apache/hudi/hive/util/HiveSchemaUtil.java

...a/org/apache/hudi/cli/commands/CommitsCommand.java

...di/common/table/timeline/HoodieActiveTimeline.java

.../org/apache/hudi/metadata/HoodieTableMetadata.java

...main/java/org/apache/hudi/dla/HoodieDLAClient.java

...oning/compaction/CompactionV2MigrationHandler.java

...he/hudi/common/model/BootstrapBaseFileMapping.java

...src/main/java/org/apache/hudi/dla/DLASyncTool.java

...rg/apache/hudi/io/storage/HoodieParquetReader.java

... and 422 more

Hi @vinothchandar , can you take a look for this, i found that there was a TODO here, and HUDI has global index now and we can remove the TODO and support reading logs.

@n3nash Can you take a look, this is important to our flink hive users, thanks in advance.

@danny0405 on it. Going to spend the next week, fully in reviews. So we will move this along

@danny0405 on it. Going to spend the next week, fully in reviews. So we will move this along

The pure logs file group was never supported before, should we support that in this PR ?

@danny0405 I see less reasons for this outside of HDFS, which can actually take appends. For cloud storage, wrtiing a base file is probably better and now we also have things like clustering which can then control file sizing. So we need not rely only on compaction for e.g

@danny0405 I see less reasons for this outside of HDFS, which can actually take appends. For cloud storage, wrtiing a base file is probably better and now we also have things like clustering which can then control file sizing. So we need not rely only on compaction for e.g

Thanks @vinothchandar , i know that maybe the cloud storage is in major for North America users, but in China, most of the people still use the HDFS yet, and they still use Hive as the query engine most often, from the perspective of functional integrity, i think we should support it instead of throwing a confusing exception or ignoring the logs silently.

ok makes sense.

Marked this a release blocker, so we can get it into the release.

Just one comment. The logic seems correct. Can we add tests that explicitly send inserts to logs and see if its all returned correctly

Thanks @vinothchandar , before this patch , we assume that each file group has a base parquet fie PLUS logs with only UPDATE records, this is true for BloomFilter index, but with global index like flink, the logs are not only UPDATEs but can also be INSERTs. So the new logic is to read the parquet first then the logs.
So the test actually tested the case, with first 100 log records to update the old one then another 20 insert records for pure INSERTs.
While this patch does not solve the case to read pure logs file group, because after debug, i found that this input format found the file groups based on the visible parquet first, the logs are hidden file.
I have add one line comment to the test case.


VS Code freezes on launch

Pls help.  VS code freezes on launch.  Cannot click on anything in window.  Totally unresponsive. Works fine with extensions disabled (command below).

VSCode Version: latest stable 1.14.0
OS Version: OSX 10.12.5

Steps to Reproduce:

Download latest version of VS Code from code.visualstudio.com
open program with 'code' command in terminal or clicking on icon in dock. Watch it freeze instantly upon opening.

Reproduces without extensions: Yes/No   NO

VSCode Version: latest stable 1.14.0
OS Version: OSX 10.11.6

I have exactly the same problem

Possibly #30407
Do you have the atefth.ruby-on-rails-snippets extension installed? If so, try uninstalling it

@mjbvz That's the problem. Thanks for pointing it out.

#30407

ubuntu 16.04  I have exactly the same problem VSCode Version: latest stable 1.14.0

@Serg-zefir Make sure you're running the latest release 1.14.2.