Reduce CI run time. Add gofmt updates.

This reduces CI time down by close to half.
We currently run the same target test-esx 2 times against each ESX since we blindly repeat the entire test suite against each target. This change will only run test-esx once per ESX.
Also, picked up some gofmt complaints.

deploy-and-test-wrapper.sh: changes looks good to me.

Also, picked up some gofmt complaints.
Thanks for fixing! This one is already addressed as part of my previous checkin.


Can you make "svgo" a peer dependancy?

for example for people who wants to use a custom version of it with my own plugins.
or custom version

I'd be willing to accept a PR for this, yeah 😉

np

Closing the issue, if anyone wants to move the svgo dependency to a peer dependency and add a warning for when svgo is not installed: feel free to submit a PR 👍


MVP

ISM support need too

Add also resources specification support and some java options auto extending (from resources memory field values)


Broken?

This API Client hasn't been updated in 3 years, but it seems to be the most popular around. I'm getting no errors, and the object output is empty. This may be based of the v1 API, but I believe it's still supported. I might be able to fix the issue and submit a pull request

Hi. The client uses the v2 endpoint: https://github.com/xsist10/HaveIBeenPwned/blob/master/src/HaveIBeenPwned.php#L36
Could you give me an example of your use case that results in an empty response?

After looking at the API spec again an empty output object from checkAccount($email) means that there are no known public breaches for the specified email address.
In addition:

Note: the public API will not return accounts from any breaches flagged as sensitive or retired. By default, the API also won't return breaches flagged as unverified, however these can be included by using the following parameter: includeUnverified=true

https://haveibeenpwned.com/api/v2/#BreachesForAccount

I tried my email address, which gets results back normally. I tested to make sure it wasn't an issue with their API at the time, and I used guzzle for that.
I've tested it on a Windows and macOS Computer, running 7.1.5 and 7.1.6 respectively. Display errors was enabled, and I've tried with xdebug as well no luck. It just returns empty objects according to var_dump().

I've modified the library to support different adapters, so you can pass in a cURL handler instead and enable logging on it (using the PSR -3 Logger standard). This should give you an easier way of debugging your problem. If you have time you can even create an adapter for Guzzle (just implement a wrapper class that implements the Adapter interface).
You can install a logging library like monolog to help with this:
composer require monolog/monolog

Implement it like this:
use xsist10\HaveIBeenPwned\HaveIBeenPwned;
use xsist10\HaveIBeenPwned\Adapter\Curl;

use Monolog\Logger;
use Monolog\Handler\StreamHandler;
use Monolog\Handler\ErrorLogHandle;

$logger = new Logger('name');
// Push all logging up to the level of DEBUG to our log file
$logger->pushHandler(new StreamHandler('/path/to/log/file', Logger::DEBUG));
// Push all logging up to the level of INFO to our stdout
$logger->pushHandler(new ErrorLogHandler(), Logger::INFO);

$adapter = new Curl();
$adapter->setLogger($logger);
$manager = new HaveIBeenPwned($adapter);


Manually output E's to workaround missing DECALN

This is an attempt to emulate the DECALN sequence on the VT340 by outputting 24x80 E characters at the start of each test. I'm a little concerned that this might make the test files too big, and they'll overflow the I/O buffers if you don't have flow-control, but I've tested on a VT240 (technically the MAME VT240 emulator) and that seemed to cope OK, so I'm hoping it'll work for you too.

I think at this point, it makes sense to just make this into a shell script so we can also soft reset the terminal and reset the palette before hand as well. No need to put the same boilerplate in every test file. Also, as a shell script, you can simply do yes E | head -2000 | tr -d '\n' .
I'm also thinking that it might be helpful if the test files themselves were shell scripts so they would be a little more readable to other people attempting to understand what the escape sequences are supposed to do.

Yeah. this makes sense. It would be nice to be able to add some comments documenting exactly what the tests are doing and why. I know very little about shell scripts, but I'll take a stab at it once I've sorted out some of the issues that were uncovered by your first set of screenshots.

I'm closing this in favor of PR #3, which converts everything to shell scripts, and uses your suggestion of yes as a substitute for DECALN.


GetLocalPath will not return a path with prefix "." or ".."

I was working with relative paths and found out GetLocalPath does not like to work with them.
I found a possible fix in the FixPathSeparators function. I think the code below should be added " 'Adding relative directory if specified"
I included additional code already existing for reference.
    'Adding relative directory if specified
    If Left$(pathToFix, 2) = ".." Or Left(pathToFix, 1) = "." Then
        resultPath = BuildPath(CurDir, pathToFix)
    End If

    '
    Const fCurrent As String = ps & "." & ps
    Const fParent As String = ps & ".." & ps
    Dim sepIndex As Long
    Dim i As Long: i = 0
    '
    'Remove any current folder references
    Do
        i = InStr(i + 1, resultPath, fCurrent)
        If i = 0 Then i = InStr(Len(resultPath) - 1, resultPath, ps & ".")
        If i > 0 Then Mid$(resultPath, i + 1, 1) = ps
    Loop Until i = 0

I haven't done a full analysis of your code to know if adding that will break anything, but from a quick glance it appears nothing should break.
Let me know if this will fit into the intention of your functions and I can send a pull request.
Example input output Before update:

Input:
--    strg = BuildPath(".\myp", "fold")

Strg Output:
--     ".\myp\fold"

Input:
--     strg = GetLocalPath(".\LogOutput")

Strg Output:
--     ""

Example input output After update:

Input:
--     strg = BuildPath(".\myp", "fold")

Strg Output:
--     "E:\Code\VBA\myp\fold"

Input:
--     strg = GetLocalPath(".\LogOutput")

Strg Output:
--     "E:\Code\VBA\LogOutput"

Hi @Kaydron1000 ,
I tend not to work with relative paths but rather be specific with full paths so that I minimize unexpected errors, since CurDir can be set from anywhere else in the program via ChDir. The GetLocalPath expects a fullPath  - as the first parameter name suggests. The FixPathSeparators indeed deals with . and .. but only in the middle, not the left side.
Your call could be simply written as GetLocalPath(BuildPath(CurDir, ".\LogOutput")) and this would achieve the desired result while being specific.
I guess we could add an extra Boolean parameter that allows the behaviour you suggested. Something like:
Public Function GetLocalPath(ByRef fullPath As String _
                           , Optional ByVal rebuildCache As Boolean = False _
                           , Optional ByVal returnInputOnFail As Boolean = False _
                           , Optional ByVal useCurDirForRelativePaths As Boolean = False) As String

where the user can explicitly set the useCurDirForRelativePaths = True if he/she wants to work directly with relative paths. However, I still prefer keeping things explicit as I haven't really seen anyone use CurDir in a production environment so far.
Can you please give me an example where using CurDir has any kind of advantage? I am happy to implement it if you can convince me :smiley:
Thanks!

I typically use relative path for loading in things such as configuration or exporting log files. The path is usually in reference to either executing application or selected input which could be 2 different directories.
I would have to agree with the idea that specifying the relative location is ideal/required, and I also do this by usually making some function that replaces "." with directory of executing assembly.
I agree with your statement. I'll close the issue as resolved.
Thanks for your insight!


Latency with state in dev tools

Version
4.1.5-beta.1
Browser and OS info
Chrome 72
Steps to reproduce

Create a button that increments a counter data property
Click on the counter to increment and watch the dev tools
Dev Tools doesn't update the counter value as expected if the property is not used

What is expected?
The data property should update as an event happens in real time.
What is actually happening?
The data property does not update in time and only updates when you navigate away from the component and click back in to the component.

Here is a gif of the current interaction -> https://gph.is/g/Evn8YgE

It is vue problem, not vue-devtools. Vue dont send message on update unused property.

Vue reactivity system is optimized to not recompute data which isn't used in the app.


Add BuildTemplate for makisu

This PR adds a simple BuildTemplate for makisu (https://github.com/uber/makisu). As soon as https://github.com/knative/build/pull/550 gets merged I would adjust the Secret Volume to be rendered by a Template Parameter rather than a hardcoded string.
Currently the makisu setup doesn't use a global build cache (mentioned here: https://github.com/uber/makisu#using-cache) if you want I can add an example for the global cache too.
Fixes: https://github.com/knative/build-templates/issues/77

ping @ImJasonH  and @mattmoor

/lgtm
/approve

/lgtm


problem when execute winpayloads

Hi,
I have this error when execute winpayloads :
Traceback (most recent call last):
File "./WinPayloads.py", line 5, in
from lib.menu import *
File "/root/Git/winpayloads/lib/menu.py", line 4, in
from generatepayload import *
File "/root/Git/winpayloads/lib/generatepayload.py", line 3, in
from psexecspray import *
File "/root/Git/winpayloads/lib/psexecspray.py", line 29, in
t = blessings.Terminal()
NameError: name 'blessings' is not defined
I try to reinstall, git pull, ...
kali 4.9.0-kali4-amd64
Thx

i had met this situation too.

Me 2, also running amd64 up-to-date

I have also run into this issue.

Thanks All,
This should have been fixed now.
Please run setup.sh again to redownload the psexecpray module.
Charlie


Need to surface the list of all samples

It was lost in the MDL redesign

Can we rely on https://www.chromestatus.com/samples, or link to it?

This is a bit of an old one.
There are some samples within WF (in _code) directories that this refers to.

This isn't relevant any longer.


Update go-cni to 891c2a41e18144b2d7921f971d6c9789a68046b2.

Fixes #1078.
Signed-off-by: Lantao Liu<EMAIL_ADDRESS>

Just added a fake implementation for the new GetConfig function. Apply LGTM based on https://github.com/containerd/cri/pull/1130#pullrequestreview-224710363


Tests files position

Trying to run on my project structure:
front/modules/name1/test/name1.test.js
front/modules/name2/test/name2.test.js

Script options: tests: 'front/modules/**/test' works ok, all files processed, but result doesn't works (((
Looks like tests var used alot, maybe it's better to split? For ex.:
`base`: 'whatever'
 `tests: 'front/modules/**/test'`

The tests option should set the root directory that contains your tests.  PhantomFlow will recurse into that directory to find all your tests.
You can use the testFileExtension options to specify the extension of your test files; by default it looks for all files that match .test.js.
e.g.
{
  tests: 'project/phantomdflowtests',
  testFileExtension: '.phantomflow.test.js'
}


UI: Incorrect Pool page position stats

Opening a single issue for a couple of issues as they might be related and #205 may be related as well.

The "Share of pool" percentage shown when adding to a pool seems to be too high.
The TOKEN/hTOKEN, TOKEN, and hTOKEN values all seem to be inaccurate or incorrect.

This may require some research and testing to figure out what's going wrong. Here's what I saw when adding a pool position and then removing it.
The "Share of pool" was quoted at 6.5% for adding 50,000 WMATIC and 50,000 hMATIC

After adding 50,000 WMATIC and 50,000 hMATIC the following position data was shown:

The WMATIC/hMATIC, WMATIC, hMATIC values are all too small and make it look like funds were lost. The pool share is only 3.36% vs the quoted 6.5% but I believe this is accurate and the quote is wrong.
After removing from the pool, this is what I received. This further supports that the WMATIC/hMATIC, WMATIC, hMATIC values previously shown were wrong because I received more of both.

Closed by https://github.com/hop-protocol/hop/commit/831ba14ad4fc97a9de3846f84c7a00b40c4d6a28 and https://github.com/hop-protocol/hop/commit/73fdffbce316a66d02405332ebcbc6ac4ffe860f


NOISSUE - Add UDP service exposing

Signed-off-by: Ivan Milošević<EMAIL_ADDRESS>What does this do?
Explains how to expose UDP service

Openning both TCP and UDP ports on services type LoadBalancer is not possible.
This must wait to be solved in devops: https://github.com/mainflux/devops/issues/66

@blokovi please remove a draft if this one is ready

@blokovi please remove a draft if this one is ready

No. it's not, there is an issue that must be resolved first https://github.com/mainflux/devops/issues/66

Would it be possible to move this one forward?

Since Kubernetes v. 1.24 MixedProtocolLBService is enabled by default. That means now it's enabled using different protocols in the same LoadBalancer type Service instance so we can merge this in our docs


Convert 1USDC to ONE gives you wrong value

When trying to swap 1 1USDC to ONE , it is also impossible to complete the swap

Was the amount automatically filled? It does not appear so. Also you do not have any 1USDC balance to swap from

I do have balance in the wallet for it. I thought I actually transfered 1 1USDT and 1 1USDC to the wallet, but I might have been wrong. I also dont seem to see the transaction in the block explorer in the HRC20 tab

I will look into this. I think it might be the balance in Swap page is out of sync with the balance on main page. There may be an error on the number of decimal points that are supposed to be used. And the large number is caused by either that or a divide by zero error.

@cylim  if you have time - feel free to take a look as well

1usdc, 1usdt and AAVE seem to have similar issue

@dietbald when you select the usdc, there are two USDC, select the one with squared icon instead of the rounded icon.

@polymorpher  After removing the default tokens, the AAVE pricing still incorrect, probably because of liquidity issues in Sushi Swap. I need to spend more time to look into it.
And we might need to update icons for some tokens.

@cylim  I get the error for both of the Tokens, see in the picture, also the balance doesnt seem to come through as well

@dietbald the latest changes should have fixed some related issues, but I am not sure whether they are exactly the issues you faced. Can you check?
One thing I noticed on your screenshot is 1USDC and UST don't have any icons, which suggest that they might not be the USDC / UST tracked by 1wallet. Where did you get these tokens?
If the issues  still remain, can you record a video?

AAVE has insufficient liquidity on Sushiswap. I will update error message (currently this error only shows when you have both amounts in the input boxes)

hold on -  I am fixing a few other things

ready now

Let me know if this remains an issue

All seem to be working great now, thanks


Revert "Introduce string_view and use it for gpr_split_host_port."

Reverts grpc/grpc#19218 due to internal issues.

Thank you @markdroth and @AspirinSJL for the approval!


Post processing should be removed in merge stage of broker

Post processing operator is unexpectedly working in historical node.

Fixed


Be able to retrieve jsonData from remote source

Really useful for debugging - be able to get jsonData from local install of Wordpress or remote source

@jrobson153 - Any thoughts on how to implement this? Wanted to talk it through some before setting it up. Thanks!

@SaFrMo yea, there are a bunch of ways we can do it and I want to go for the least disruptive. I think no matter what the ideal scenario is to just have a plugin settings page where we can just have a checkbox that's like "Site API Proxy" or something like that and put in a URL.
My first thought is to do this 100% in Rest Easy, maybe inside the rez_build_all_data() function we can detect if that setting is on and if so just skip all the internal stuff and make a wp_remote_get() call to the proxy server and return the data.
Seems like kind of a strange way to accomplish this I know, but it would essentially be one small section of code handling the proxy and everything after that would work exactly the same. Any other solution I think gets into writing a bunch of logic into our front end code, adding bloat etc.
Thoughts?

@jrobson153  Yep, that makes sense to me! Ditto with only seeing overcomplicated solutions with any front-end code - the sooner we can modify how the data is actually requested, the better, and Rest-Easy seems like the right place to do that.
Closing this issue and opening one in Rest-Easy.


too many drawbale errors and not found

I had to download some of the drawable files from somewhere else to test it but still I am facing other issues when building
eg:
Task :app:compileDebugJavaWithJavac
E:\Android Projects\WhatsApp-Clone-master\WhatsApp\app\src\main\java\com\github\markpairdha\whatsapp\MessageAdapter.java:52: error: cannot find symbol
senderMessageText = (TextView) itemView.findViewById(R.id.sender_messsage_text);
^
symbol:   variable sender_messsage_text
location: class id
E:\Android Projects\WhatsApp-Clone-master\WhatsApp\app\src\main\java\com\github\markpairdha\whatsapp\MessageAdapter.java:76: error: cannot find symbol
Messages messages = userMessagesList.get(i);
^
symbol:   variable i
location: class MessageAdapter
E:\Android Projects\WhatsApp-Clone-master\WhatsApp\app\src\main\java\com\github\markpairdha\whatsapp\MessageAdapter.java:135: error: cannot find symbol
messageViewHolder.messageSenderPicture.setBackground(R.drawable.file);
^
symbol:   variable file
location: class drawable
E:\Android Projects\WhatsApp-Clone-master\WhatsApp\app\src\main\java\com\github\markpairdha\whatsapp\MessageAdapter.java:147: error: cannot find symbol
messageViewHolder.messageReceiverPicture.setBackground(R.drawable.file);
^
symbol:   variable file
location: class drawable
E:\Android Projects\WhatsApp-Clone-master\WhatsApp\app\src\main\java\com\github\markpairdha\whatsapp\ContactsFragment.java:52: error: cannot find symbol
myContactsList = (RecyclerView) ContactsView.findViewById(R.id.contacts_list);
^
symbol:   variable contacts_list
location: class id
E:\Android Projects\WhatsApp-Clone-master\WhatsApp\app\src\main\java\com\github\markpairdha\whatsapp\FindFriendsActivity.java:58: error: constructor FirebaseRecyclerAdapter in class FirebaseRecyclerAdapter<T,VH> cannot be applied to given types;
new FirebaseRecyclerAdapter<Contacts, FindFriendViewHolder>() {
^
required: FirebaseRecyclerOptions
found: no arguments
reason: actual and formal argument lists differ in length
where T,VH are type-variables:
T extends Object declared in class FirebaseRecyclerAdapter
VH extends ViewHolder declared in class FirebaseRecyclerAdapter
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: Some input files use unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
6 errors

Is is solved??

I dont know never tried it again since no reply from the owner.


Hide Instruction Name on Solscan

Hi, I can't login to Discord so asking here.
Is there a way to hide Instruction names from appearing on Solscan when using Anchor?
For example, this is how it appears on Solscan. Notice "Program log: Instruction: MyInstructionName",. I would like to hide this. I could obscure it with a fake name, but I notice some programs hide it completely. So wondering if possible with Anchor.
[
  "Program ZZZOOOOOOOOOO invoke [1]",
  "Program log: Instruction: MyInstructionName",
  "Program ZZZOOOOOOOOOO consumed 3654 of 200000 compute units",
  "Program ZZZOOOOOOOOOO success"
]

You can build your program with no-log-ix-name feature enabled, e.g.
anchor build -- --features no-log-ix-name


Bendy VR mod error whenever starts the game

I recently purchased the game ‘Bendy and the Ink Machine’ on Steam to play with its VR mod. I followed the installation instructions, but whenever I start the game, it initially displays correctly on my Pico 4, but when it’s about to show the first menu, it crashes with an error (as shown in the picture). I’ve tried using both Streaming Assistant and Virtual Desktop, with the latest version of SteamVR, the beta version, and the previous version, but the issue persists. Any hints or suggestions? Thanks.

Was this error using Streaming Assistant or VD?
This seems to be a pretty fundamental error. Please try via Virtual Desktop and send me if it is the same message.
I'm a little confused to why your Steam VR is only showing one controller

Whether using the Streaming Assistant or the Virtual Desktop, the same error occurs. And I think in that previous figure, as it was one of the tests I was conducting, I had only connected one controller. That’s why only one appeared.
But I uninstalled and reinstalled the mod again and restarted the game with the current SteamVR and Virtual Desktop and the same error occurs. See the figure below. Is there any way to enable debug messages and record them in a file to see if they provide more information? Thanks in advance.

Ok, thanks. My versions:
VD - 1.30.1
Bendy - last version (right click exe):
. file version - 2<IP_ADDRESS>41
. product version - 2<IP_ADDRESS>3501
Pico - 5.6.0


@objc and minimized inference / README update?

I was dusting off an old Objective-C project and it wasn't seeing the initializer of AKFFTTap – I tracked down the change:
https://github.com/AudioKit/AudioKit/commit/f59da655b035e769e4d9b0e98e8d71621bc50690#diff-b4a7bf8afa4b84ec5e248da8c71c4fa1
What I realized was that my project wasn't using swift 3 @objc inference.
Might bite people with older projects – perhaps worth adding a note to the install instructions for Objective-C users that they should make sure inference is enabled?

Sounds like a good idea, would you like to make a pull request?

No prob! Pull request here: https://github.com/AudioKit/AudioKit/pull/1865

Thank you!


added fedora / centos installation instruction

added fedora / centos installation instruction (based on yum) to readme.

It would be better to provide the package installation through standard Fedora channels.


Further simplify worker API

See individual commits. I'm not yet done, but this is going towards a worker API that supports multiple job types.

Update: add a commit which changes the API to use callback URLs instead of predefined routes.

Update to set Content-Type correctly in the client and expose server errors in the worker's log.

Update: fix typos, clarify an error message and print errors at all when uploading fails. Also, fix an issue the tests found: the upload URL was wrong.

One test failed due to a mirror issue, unrelated to this PR.


Downloading of Purchased Videos fail with tempvideo and tempaudio

Firefox.
Says the CDM using KeysDB is fine.
Using latest EXE.

In the future reports without logs will be closed
please use the tickets channel in the discord if you desire to share logs privately

closing because of lack of response


Remove Outdated react lifecycle components & updated the react version to 16.4.1

@patw0929

Coverage increased (+0.05%) to 93.344% when pulling 5cd467abc96184412605f84ce1536974f610256f on superhit0:remove-component-will-receive-props into bb09300e0ac93e10df760027a94aaa14970d447c on patw0929:master.

Merging it !!


PHP Fatal error:  Uncaught exception 'JonnyW\PhantomJs\Exception\InvalidExecutableException'

Hi I'm trying to use php-phantomjs, I've installed using composer:
($ composer require "jonnyw/php-phantomjs:4.*")
The installation was success but when I try to implement the Basic Usage I'm receiving this error:
PHP Fatal error:  Uncaught exception 'JonnyW\PhantomJs\Exception\InvalidExecutableException' with message 'File does not exist or is not executable: /bin/phantomjs.exe'.
I've googled and I tryied several solutions like: $client->getEngine()->setPath('/path/to/phantomjs'); | chmod... without success.
I'm trying in a server with CentOS 6.7
Please I really need help. Thanks a lot.!

Also having this issue on a vagrant box. Cant seem to chmod the file, and am simply getting:
JonnyW\PhantomJs\Exception\InvalidExecutableException:
File does not exist or is not executable: /var/www/bin/phantomjs
in /var/www/vendor/jonnyw/php-phantomjs/src/JonnyW/PhantomJs/Engine.php:216

+1。。。。
I am the same problem

Have you solved this problem?


QT WALLET WONT SYNC UP

DOWNLOADED MONYX WALLET AND LEFT OPEN FOR MORE THAN 24 HOURS...WHY HASNT THE WALLET SYNC YET?

@DAATHVEGA provide please your degub.log file.

Ok...where do I find that?

debug.log

@DAATHVEGA most likely you have the old wallet. Try to download the new one and delete everything EXCEPT wallet.dat file from the folder where you found debug.log file.

The only file present is the exe file...I don't know of a way to delete the
wallet data before the app can launch
On Fri, Feb 9, 2018 at 2:11 PM, monyx.io<EMAIL_ADDRESS>wrote:

@DAATHVEGA https://github.com/daathvega most likely you have the old
wallet. Try to download the new one and delete everything EXCEPT
wallet.dat file from the folder where you found debug.log file.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/monyxcoin/monyx-coin/issues/118#issuecomment-364530680,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ain-Bgzazb6e-p7xG2J9rrWdPeQ6RG2Dks5tTJh2gaJpZM4SAM2i
.

@DAATHVEGA do you use macOS or Windows?

windows, but problem solved after i deleted everything but wallet.dat
Powered by Cricket Wireless
-------- Original message --------From: "monyx.io"<EMAIL_ADDRESS>Date: 2/11/18  11:25  (GMT-05:00) To: monyxcoin/monyx-coin<EMAIL_ADDRESS>Cc: DAATHVEGA<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [monyxcoin/monyx-coin] QT WALLET WONT SYNC UP (#118)
@DAATHVEGA do you use macOS or Windows?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/monyxcoin/monyx-coin","title":"monyxcoin/monyx-coin","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/monyxcoin/monyx-coin"}},"updates":{"snippets":[{"icon":"PERSON","message":"@monyxcoin in #118: @DAATHVEGA do you use macOS or Windows?"}],"action":{"name":"View Issue","url":"https://github.com/monyxcoin/monyx-coin/issues/118#issuecomment-364764335"}}}


Initial benchmarking implementation: steps to run benchmarks and comb…

This pull request creates a pipeline to run benchmarks on all supported backends, combine the results into a single json file, and pass that file to https://github.com/benchmark-action/github-action-benchmark. So far, I've written the benchmarking scripts and buildkite pipeline for running and combining the benchmarks. Once I confirm this is working, then I'll add the github step for downloading the resulting artifact and passing to github-action-benchmark.
Currently, I have it running the CPU benchmarks with 1, 2, 4, and 8 threads, but we can change this if we want. The two benchmarks are the examples from lit-04-3DSliceSelective.jl and MRiLab_speed.jl, with names "Slice Selection 3D" and "MRI Lab" (we can change these names). I also noticed that the lit-04-3DSliceSelective.jl example takes kind of long to run (around a minute on my computer), which may cause BenchmarkTools.jl to run fewer trials and result in more fluctuation in our measurements, so we may want to adjust this example to not take quite as long.
We should also start thinking about how we want the benchmark results to display. Currently, I have the benchmark structure as:
benchmark -> sim method -> cpu / gpu -> # of threads / gpu backend
As of now, the only sim method is Bloch (is it interesting to also profile BlochDict?). When a kernel-based sim method is written we can add this as well. If we want our page to look similar to the one here: https://lux.csail.mit.edu/benchmarks/, we can have two columns for CPU and GPU, and the graphs will have a title like: "MRI Lab + Bloch + GPU", with lines for each backend (and # of threads for the CPU). This will require some updates to the index.html page that github-action-benchmark provides, but this can be done after this pull request is merged.

For convenience I will leave this link here: https://juliahealth.org/KomaMRI.jl/dev/bench/

I think we can merge this tomorrow after I delete the lines with #TO-DO: remove this. I still want to see if there is a way we can run the CPU benchmarks on the same platform each time.
We'll also need to ensure we delete the data.js file in gh-pages/benchmarks before merging.


Disable instead of hiding change photo button

Fixes https://openedx.atlassian.net/browse/MA-1607
Also fixes the style of the button as mentioned in https://openedx.atlassian.net/browse/MA-1574

LGTM :+1:


include vin in published message body

Open to better ways to add the vin to the body. I couldn't figure out a cleaner way that worked with Go's type system.


BUG6 - the remote communicated task seems cannot connect to upstream after rescale

I think there are two problems.

The non-source and destination sub-tasks of the target stream:

do not receive any record
can not finish after source stream finished

The destination sub-task do not send processed records to downstream

Only the source sub-task of the target stream is normal.


Solidity contract to mint aBTC

Description
Solidity contract to mint aBTC with a one-to-one mint with wBTC, tBTC, cbBTC and more.
TODO

[] Architecture and Doc of the implementation
[] Mapping of BTC wrapped like tBTC, wBTC, cbBTC etc.
[] ERC20 or ERC4626 aBTC with Access control from OpenZepellin: https://docs.openzeppelin.com/contracts/5.x/erc4626
[] Deposit function
-[] Withdraw function
[] Cross call into Cairo
[] Upgradeable contract if possible

i

@MSghais i can work on this

Hi @MSghais i can work on this .

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
I am a Full Stack blockchain Developer with expertise in Next.js, Nest.js, TypeScript, JavaScript, React, Node.js, Three.js, and Solidity. My journey with OnlyDust hackathons began at Edition 1, and I've since made 47 contributions across 11 projects. With my extensive experience on the OnlyDust platform (profile:  https://app.onlydust.com/u/Ugo-X), I've honed my skills in delivering quality solutions under pressure.
I bring a unique blend of technical proficiency and user-focused design to every project, whether it's crafting immersive 3D experiences or developing smart contracts. My track record shows I can adapt quickly and contribute effectively to diverse challenges.
As we surf through Edition 7, I'm excited to leverage my skills and hackathon experience to push the boundaries of blockchain development. I'm confident in my ability to tackle new challenges and drive innovation in this space.
How I plan on tackling this issue
. Here's how I would approach the Solidity contract to mint aBTC with a one-to-one mint with wBTC, tBTC, cbBTC and more, as a single Pull Request (PR):
Issue: Solidity Contract to Mint aBTC with a One-to-One Mint with wBTC, tBTC, cbBTC and More
Description:
I will create a Solidity contract to mint aBTC (Aragon Bitcoin), a synthetic asset representing Bitcoin, with a one-to-one mint with other Bitcoin-backed assets like wBTC, tBTC, and cbBTC. This will involve creating a mintVaultBTC contract and a separate contract for a reserve of wBTC.
Tasks:
Architecture:

I will design the overall architecture of the solution, including the relationship between the mintVaultBTC contract and the wBTC reserve contract.
I will determine the necessary data structures and state variables to track the minting, burning, and reserves of aBTC.
I will consider the security implications and potential attack vectors to ensure the contract is secure and robust.

Deposit Function:

I will implement the deposit function that allows users to deposit wBTC, tBTC, cbBTC, or other supported Bitcoin-backed assets into the contract.
I will ensure the deposit function correctly verifies the input assets and updates the corresponding aBTC balance.
I will consider edge cases, such as partial deposits or unsupported assets, and handle them appropriately.

Withdraw Function:

I will implement the withdraw function that allows users to burn their aBTC and receive the equivalent amount of the underlying Bitcoin-backed asset.
I will ensure the withdraw function correctly verifies the user's aBTC balance, burns the tokens, and transfers the corresponding asset back to the user.
I will handle potential edge cases, such as insufficient aBTC balance or unsupported assets.

Cross Call into Cairo:

I will research and understand the integration between Solidity and the Cairo programming language, which is used in the Ethereum-based Starknet ecosystem.
I will implement the necessary cross-chain functionality to enable interaction between the Solidity contract and the Cairo-based components, such as the aBTC token contract.
I will ensure the cross-chain communication is secure and reliable, following best practices for cross-chain integration.

Pull Request (PR) Workflow:

I will create a new branch for the "aBTC Minting" feature.
I will submit a single pull request that includes all the tasks: Architecture, Deposit Function, Withdraw Function, and Cross Call into Cairo.
I will ensure the pull request includes comprehensive tests and documentation for the implemented features.
I will collaborate with the team to address any feedback or reviews during the pull request process.

By including all the tasks in a single Pull Request, I can ensure that the development process is streamlined and the entire feature is reviewed and merged together. This approach allows for better code cohesion and easier tracking of the feature's progress.
Given the complexity of the issue i propose an honest 5-6 Days ETA, as i would need to do proper research.
My Breakdown:
Day 1: Architecture and basic setup
Day 2: Core contract and token mapping
Day 3: Deposit and withdraw functions
Day 4: Access control and upgradability
Day 5: Cross-chain integration with Cairo
Day 6: Final testing, documentation, and optimization

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
I am Sagar Rana, a full stack and blockchain engineer with over a year of experience in writing smart contracts. I have been writing smart contracts for a year now with languages such as Solidity, Vyper, Cairo, Rust and Noir. I can write contracts involving transactions with minimal gas fees.
How I plan on tackling this issue
1.Define Supported Tokens:

Create a mapping to track which wrapped BTC tokens (e.g., wBTC, tBTC, cbBTC) are supported.
Add functions to add/remove supported tokens, only accessible by admin.

Create aBTC Token Using ERC4626:

Use OpenZeppelin’s ERC4626 standard to create aBTC as a tokenized vault.
Implement access control with roles for admin and minter using OpenZeppelin’s AccessControl.

Implement Deposit Function:

Ensure the deposit function accepts only supported tokens.

Ensure the withdraw function handles multiple tokens securely

function withdraw(address token, uint256 amount) public {
    require(supportedTokens[token], "Unsupported token");
    _burn(msg.sender, amount);  // Burn aBTC
    IERC20(token).transfer(msg.sender, amount);  // Return equivalent wrapped BTC
}

Handle Cross-Chain Calls (Cairo Integration):

Emit an event or use a bridge contract for cross-chain functionality with Cairo.

Make the Contract Upgradeable (Optional):

Use OpenZeppelin’s upgradeable contracts for future-proofing.

Hey @MSghais, I think I will be able to resolve this issue, please assign

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
I'm Poulav Bhowmick, a software engineer at Invisible Studios with a robust background in TypeScript, Rust, Solidity Cairo, fullstack development and blockchain technology. My experience includes building robust applications, optimizing functionalities and blockchain integration. I have actively participated in events and open source contributions, enhancing my capability to tackle real-world tech challenges. My projects can be viewed on my GitHub Profile and OnlyDust Profile. Plus I´m active member of Starknet community🇷.
How I plan on tackling this issue
I will implement the aBTC contract with the following approach:
Create an upgradeable ERC20 contract using OpenZeppelin:
Implement ERC20Upgradeable for token functionality
Use AccessControlUpgradeable for role-based permissions
Implement UUPSUpgradeable for future upgrades
Add support for wrapped BTC tokens:
Create a mapping to track supported tokens (wBTC, tBTC, cbBTC, etc.)
Implement admin functions to add/remove supported tokens
Develop core minting functionality:
Create deposit() function for 1:1 minting of aBTC
Implement withdraw() function for burning aBTC and receiving wrapped BTC
Research and implement Cairo cross-call functionality:
Explore bridge or oracle solutions for StarkNet integration
Implement secure cross-chain communication
Set up testing and deployment:
Write unit and integration tests
Use Hardhat for local development and testing
Deploy using an upgradeable proxy pattern
ETA- 3-4 days

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
I am Sagar Rana, a full stack and blockchain engineer with over a year of experience in writing smart contracts. I have been writing smart contracts for a year now with languages such as Solidity, Vyper, Cairo, Rust and Noir. I can write contracts involving transactions with minimal gas fees.
How I plan on tackling this issue
1.Define Supported Tokens:
* Create a mapping to track which wrapped BTC tokens (e.g., wBTC, tBTC, cbBTC) are supported.

* Add functions to add/remove supported tokens, only accessible by admin.

2. Create aBTC Token Using ERC4626:

* Use OpenZeppelin’s ERC4626 standard to create aBTC as a tokenized vault.

* Implement access control with roles for admin and minter using OpenZeppelin’s AccessControl.

3. Implement Deposit Function:

* Ensure the deposit function accepts only supported tokens.

* Mint equivalent aBTC to the user upon deposit.

function deposit(address token, uint256 amount) public {
    require(supportedTokens[token], "Unsupported token");
    IERC20(token).transferFrom(msg.sender, address(this), amount);
    _mint(msg.sender, amount);  // Mint equivalent aBTC
}

4. Implement Withdraw Function:

* Allow users to burn aBTC and withdraw the corresponding amount of the supported token.

* Ensure the withdraw function handles multiple tokens securely

function withdraw(address token, uint256 amount) public {
    require(supportedTokens[token], "Unsupported token");
    _burn(msg.sender, amount);  // Burn aBTC
    IERC20(token).transfer(msg.sender, amount);  // Return equivalent wrapped BTC
}

5. Handle Cross-Chain Calls (Cairo Integration):

* Emit an event or use a bridge contract for cross-chain functionality with Cairo.

6. Make the Contract Upgradeable (Optional):

* Use OpenZeppelin’s upgradeable contracts for future-proofing.

7. Create a comprehensive and well structured documentation

ETA: 3 days

@SoarinSkySagar can you ping me in Telegram please? MSG_Encrypted


XMPP and socket.io packages

Hello!
Does anyone know of a package to use xmpp in flutter and another for socket.io?

Check out https://github.com/rikulo/socket.io-dart ? You should be able to hook that up to setState in a StatefulWidget to provide updates to stuff

Please see pub.dartlang.org. If there's nothing there, then I expect there's no current package for this.


We would like to introduce another paper that complements this topic: "GIMLET: A Unified Graph-Text Model for Instruction-Based Molecule Zero-Shot Learning."

Thank you for sharing your insightful paper list!
We would like to introduce another paper that complements this topic: "GIMLET: A Unified Graph-Text Model for Instruction-Based Molecule Zero-Shot Learning." This paper integrates the graph modality into the language model and employs an instruction-based learning approach to address graph property prediction tasks. We would be delighted if our suggestion contributes to the enrichment of this awesome paper list ☺ !

Thank you for contributing to this repository!
We have added this paper under the "Graph Classification/Regression" category.

Thank you for your feedback and for maintaining this fantastic paper list!


Krew plugin index entry

It would be awesome if this kubectl plugin was part of krew index.
See https://krew.sigs.k8s.io/docs/developer-guide/distributing-with-krew/

/remove-lifecycle stale

/lifecycle frozen
We are cutting releases with binaries now, so I think its a good time to do this


interpolateAtSample bug?

I can't run gl-400-fbo-multisample and all the others who uses the interpolateAtSample
in vert
{
	vec2 Texcoord;
	//sample vec2 Texcoord;
} Vert;

layout(location = FRAG_COLOR, index = 0) out vec4 Color;

void main()
{
	Color = texture(Diffuse, interpolateAtSample(Vert.Texcoord, gl_SampleID));
	//Color = texture(Diffuse, Vert.Texcoord);
}

Asking on the irc opengl, a guy suggested me to use the commented out lines instead, and that works. He also said it is probabily a glsl compiler bug... what do you think?

There is a known compiler bug for this, if you remove "Vert" at the end of the "in" definition and use Texcoord directly it should work in the interpolateAtSample function.

Hi @elect86,
What drivers / GPU are you using? Do you see an GLSL compiler error? What is it?
On a GTX 970, it works just fine for me. (tm)
Thanks,
Christophe

This issue is fixed in 4.5.4 branch.
Thanks for reporting,
Christophe


Runbook for CW Networking Alarms

User Story
As a modernisation platform engineer
I would like to have a runbook explaining how to investigate, remediate and potentially tune cloud watch networking alarms
So that we have a standard practice we can refer to
User Type(s)

Modernisation platform engineers
On-call engineers
Value

The runbook itself and the process of writing it is a good opportunity to increase and share the knowledge of MP networking.
Questions / Assumptions / Hypothesis
Hypotheses
If the runbook is available and useful, more people will feel comfortable being on-call and more confident they will be able to handle the alarms without needing escalation.
Proposal
An experienced engineer pairs with a less experienced engineer to walk through the process of resolving the networking alarms.
The walkthrough is used as a basis for the runbook
Definition of done

[ ] Runbook has been created and published on the MP docs site
[ ] user docs have been updated
[ ] another team member has reviewed

Reference
How to write good user stories

https://user-guide.modernisation-platform.service.justice.gov.uk/runbooks/cloudwatch-networking-alarms.html#a-summary-of-current-alarms


docs: Update templates

[!warning]
This is a public repository, ensure not to disclose:

[x] personal data beyond what is necessary for interacting with this pull request, nor
[x] business confidential information, such as customer names.

What kind of PR is this?
Required: Mark one of the following that is applicable:

[ ] kind/feature
[ ] kind/improvement
[ ] kind/deprecation
[x] kind/documentation
[ ] kind/clean-up
[ ] kind/bug
[ ] kind/other

Optional: Mark one or more of the following that are applicable:

[!important]
Breaking changes should be marked kind/admin-change or kind/dev-change depending on type
Critical security fixes should be marked with kind/security

[ ] kind/admin-change
[ ] kind/dev-change
[ ] kind/security
[ ] kind/adr

What does this PR do / why do we need this PR?
Updates issue templates, removes the release template, and updates the linting based on changes done in the docs repo.
Information to reviewers
The wildest change is to rename all go-templates bases and stacks, but that will be come a requirement in helmfile (once they decide to release v1), and we already use that for our values templates.
And it fits more naturally, because they aren't yaml.
And as it touches a bit of everything, I believe everyone is a reviewer, sorry bout that :sweat_smile:
Checklist

[x] Proper commit message prefix on all commits

Change checks:

[x] The change is transparent
[ ] The change is disruptive
[ ] The change requires no migration steps
[ ] The change requires migration steps
[ ] The change upgrades CRDs
[ ] The change updates the config and the schema

Documentation checks:

[ ] The public documentation required no updates
[ ] The public documentation required an update - [link to change](set-me)

Metrics checks:

[ ] The metrics are still exposed and present in Grafana after the change
[ ] The metrics names didn't change (Grafana dashboards and Prometheus alerts are not affected)
[ ] The metrics names did change (Grafana dashboards and Prometheus alerts were fixed)

Logs checks:

[ ] The logs do not show any errors after the change

Pod Security Policy checks:

[ ] Any changed pod is covered by Pod Security Admission
[ ] Any changed pod is covered by Gatekeeper Pod Security Policies
[ ] The change does not cause any pods to be blocked by Pod Security Admission or Policies

Network Policy checks:

[ ] Any changed pod is covered by Network Policies
[ ] The change does not cause any dropped packets in the NetworkPolicy Dashboard

Audit checks:

[ ] The change does not cause any unnecessary Kubernetes audit events
[ ] The change requires changes to Kubernetes audit policy

Falco checks:

[ ] The change does not cause any alerts to be generated by Falco

Bug checks:

[ ] The bug fix is covered by regression tests

I need to understand the reason why the QA checklist is made private. We really like to show this and it's even linked to from our public docs.

Because the process simply hasn't worked well after the split, so the checklist that we need for our process must be together with the internal one.
I do also want to expose it in some way to make the actual QA part public, but I haven't figured out a good way to do it yet. My thinking is a document in each artefact repository, as then it will also serve as a reference for contributors for what we expect to be tested.
However it would be formatted essentially as "this is what we test, and what we expect to be tested" document, rather than the unwieldy checklist itself.

I need to understand the reason why the QA checklist is made private. We really like to show this and it's even linked to from our public docs.

Because the process simply hasn't worked well after the split, so the checklist that we need for our process must be together with the internal one.
I do also want to expose it in some way to make the actual QA part public, but I haven't figured out a good way to do it yet. My thinking is a document in each artefact repository, as then it will also serve as a reference for contributors for what we expect to be tested. However it would be formatted essentially as "this is what we test, and what we expect to be tested" document, rather than the unwieldy checklist itself.

Restored and updated it with the current changes, along with an alert that it is going to be replaced.

Nice work with these updates, I think it looks good.
One small nit for future PRs: I think that these changes could actually have been a few different commits (possibly different PRs), since it to me feels like there are several different things happening here that does not necessarily belong together. E.g. it could have been split to one for updating templates, one for lint changes, one for renaming the stacks files. Another alternative would be to have a longer commit message that makes it more clear the different things that were included, or something that just makes it clear that there are several different things in it without specifying exactly what. I do not think that it is worth reworking this now and I do not see it as important. But if you agree with my reasoning, then keep it in mind for future PRs. 🙂

Thanks, I'll think about that!
The problem is that the "templates" that I'm producing consist of pre-commit hooks and configs, workflows, issue and pull request templates, even some scripts. :sweat_smile:
But I will definitely split some of them up onto separate commits so it is clearer from the start what originates from where.


Reload scripts on EVALSHA if Redis server is restarted

Feature Request

Is your feature request related to a problem? Please describe
We have an application that connects to a Redis server with lettuce. The application makes use of 2 Lua scripts that are executed quite often. Since the script payload is not really small, the scripts are loaded to Redis on application start and then executed with EVALSHA.
The problem we have is that if the Redis server is restarted for any reason, the execution of the scripts fails with the following exception:

io.lettuce.core.RedisNoScriptException: NOSCRIPT No matching script. Please use EVAL. at io.lettuce.core.ExceptionFactory.createExecutionException(ExceptionFactory.java:128)

We understand that Redis does not persist scripts.
Describe the solution you'd like
The solution we are suggesting is to add an optional retry mechanism. The Redis connection stores the scripts that are loaded. Upon a RedisNoScriptException, a retry could be performed by first reloading the script to Redis, and then re-executing the EVALSHA request. If an error happens again, no more attempts are made.
Describe alternatives you've considered
As suggested in the error message, an alternative is to use EVAL instead. The issue is that it's not appropriate when the scripts are large and called often. This defeats the purpose of EVALSHA.
Teachability, Documentation, Adoption, Migration Strategy
As originally suggested, this could be an option. Although, enabling it by default should not break anything.

The client does not store any uploaded Lua scripts and it has no reference to a script after it was sent to the server. Spring Data Redis provides such a feature with its ScriptExecutor as it models Lua scripts as a first-class concept instead of accepting just commands.
That being said, Lettuce is not the right place for this kind of functionality.


Make log available on STDOUT

Is it possible to log to stdout?
This will make it easy to see the logs using docker log <container>.
It would also be nice to be able to change the loglevel via environment variable.

https://docs.docker.com/engine/admin/logging/view_container_logs/
Base on the official documentation, you can create a symbolic link for Oracle log file to stdout.
Hope it helps somehow.

true, but why not do it?


polybar: 3.5.7 -> 3.6.0

This was pretty straightforward. Two interesting things happened:

libuv is a new dependency. From https://github.com/polybar/polybar/releases/tag/3.6.0:

New dependency: libuv. At least version 1.3 is required.

Polybar now attempts to install a config file to
/etc/polybar/config.ini. I don't know what the nix equivalent of
that would be, so I just turned that setting off with
-DBUILD_CONFIG=off. Here's what happens with that setting still
enabled:
$ nix-build
...
-- Install configuration: "Release"
-- Installing: /etc/polybar/config.ini
CMake Error at cmake_install.cmake:54 (file):
file INSTALL cannot copy file
"/tmp/nix-build-polybar-3.6.0.drv-0/source/doc/config.ini" to
"/etc/polybar/config.ini": Permission denied.

make: *** [Makefile:100: install] Error 1
error: builder for '/nix/store/gli9yhqd62i53jlsgw6hfz5p6qv6wvcp-polybar-3.6.0.drv' failed with exit code 2;
     last 10 log lines:
     > Install the project...
     > -- Install configuration: "Release"
     > -- Installing: /etc/polybar/config.ini
     > CMake Error at cmake_install.cmake:54 (file):
     >   file INSTALL cannot copy file
     >   "/tmp/nix-build-polybar-3.6.0.drv-0/source/doc/config.ini" to
     >   "/etc/polybar/config.ini": Permission denied.
     >
     >
     > make: *** [Makefile:100: install] Error 1
     For full logs, run 'nix log /nix/store/gli9yhqd62i53jlsgw6hfz5p6qv6wvcp-polybar-3.6.0.drv'.

Motivation for this change
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

duplicate of #162550

so I just turned that setting off with -DBUILD_CONFIG=off

I instead opted to patch it so it goes to $out and have polybar read that in my PR.

Oops, sorry for the noise!

I instead opted to patch it so it goes to $out and have polybar read that in my PR.

That's clever!


Support period characters as flow name in flow library menu

Node-RED editor menu doesn't show the flow name which contains period characters correctly.
For example, a user inputs "1.sampleflow" as flow name to save a flow.

After that, editor menu shows only "1" because current Node-RED  removes the characters which are placed after the period.

I think that correct handling removes only file extension, ".json" and shows flow name without the file extension.

Closed by #1320

Node-RED 0.17.3 now available with this fix.

Thank you @knolleary . Node-RED v0.17.3 on my Raspberry Pi showed flow name correctly.


Update bcachefs (cherry pick for 18.09)

Motivation for this change
There were some fixes related to suspending.
As requested by @Infinisil in #48598
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

Not sure why the eval failed
@GrahamcOfBorg eval

@GrahamcOfBorg eval


Extending DSL

Any thoughts on extending the DSL to incorporate:
i. user defined function(s) from the existing DSL functions and lambdas?
ii. recursion?
Thanks in advance!

I think the tricky part is generating a new dataset with your dsl extensions in the same manner that the paper authors did (which I'm assuming you'd want to do).  You would have to add constraint propagation logic for your new functions to constraint.py.  Recall the reason they did that is to keep invalid program/inputs out of their dataset- for instance, SCAN1L(LPLUS, [255,255])=510 would be invalid if the output range is [-256,256].
Unfortunately, I don't think there's a good way to do this unless you keep your extensions really simple.  For example, if you wanted to add fibonacci and the output range is [-256,256], then the input range would be [0,13] because the fib(13)=233 and fib(14)=377.  You would have to clip the input range to the largest n such that fib(n) is in the output range, maybe by precomputing the first 13 fibonacci numbers and using that to check.  One of the benefits of keeping the dsl simple is that it keeps the constraint propagation tractable.
Good luck, let me know if you had any other ideas in mind

Understood.
Other ideas:

adjust the DSL so that input and output can be floating point numbers,
Add the python math functions to the DSL.
E.g.

https://docs.python.org/3/library/math.html https://docs.python.org/3/library/math.html

On Dec 16, 2018, at 5:08 PM, dkamm<EMAIL_ADDRESS>wrote:
I think the tricky part is generating a new dataset with your dsl extensions in the same manner that the paper authors did (which I'm assuming you'd want to do). You would have to add constraint propagation logic for your new functions to constraint.py. Recall the reason they did that is to keep invalid program/inputs out of their dataset- for instance, SCAN1L(LPLUS, [255,255])=510 would be invalid if the output range is [-256,256].
Unfortunately, I don't think there's a good way to do this unless you keep your extensions really simple. For example, if you wanted to add fibonacci and the output range is [-256,256], then the input range would be [0,13] because the fib(13)=233 and fib(14)=377. You would have to clip the input range to the largest n such that fib(n) is in the output range, maybe by precomputing the first 13 fibonacci numbers and using that to check. One of the benefits of keeping the dsl simple is that it keeps the constraint propagation tractable.
Good luck, let me know if you had any other ideas in mind
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/dkamm/deepcoder/issues/9#issuecomment-447694343, or mute the thread https://github.com/notifications/unsubscribe-auth/AC9i27Gh_7iUwIbJSJ4KyuNDXOIOXf1Hks5u5u6SgaJpZM4ZJ1Yv.

If you allow floating point numbers, you'll have an infinite number of possible input/output values and you'll have to change the embedding layer of the neural network.  There are interesting ideas and I encourage you to try them out on your own fork- I want to keep master as a faithful implementation of the paper so I'm not going to add any extensions here.


Option to clone the latest commit only?

It seems a bit unnecessary to have the full git history cloned to a production server, especially if the commit history grows long. I'm wondering if it's a good idea to have the option to use git clone --depth=1 instead of cloning everything.
Thanks.

Yeah it is. This should make it in soon.

You can now specify additional cli args to the git clone command using args config.
e.g.
git {
  ...
  args --depth=1
}

You will need to wait for this to make it to caddyserver.com/downloads or you build from source.


Allow disabling rsync fallback.

Add a new option that allows users to choose whether and how to fall back to rsync. Possible options are (a) always, (b) never, or (c) only for unknown repositories.

This is probably best implemented by refactoring the current configuration, so this has to wait until 0.12.0.


I dont understand

Every image I've tried to upscale with HAT just seems to stretch the image to a larger, even more blurrier size. I've tried all sorts of sizes ranging from 64x64 to 1024x1024. It seems to just click and drag the image larger for me without actually enhancing anything.
Am I doing something wrong? I'd love to be able to use this project but right now its very confusing to me. :/

@dillfrescott Maybe your test samples are too hard for the model. You can try the GAN-based HAT model. https://drive.google.com/file/d/1Ma12vCWT27P9M99-s2RXnynKN-OQsBrv/view?usp=share_link

Ooooo okay! I will try that! Thank you!

Ok this seems to work much better! Thank you!

What config was used for the GAN-based HAT model?

I just used chaiNNer: https://github.com/chaiNNer-org/chaiNNer
Using it you don't need a config. Its somehow handled automatically


Updated supported versions and fixed table error

Learn Build status updates of commit a19ade2:
:bulb: Validation status: suggestions

File
Status
Preview URL
Details

articles/defender-for-cloud/defender-for-devops-introduction.md
:bulb:Suggestion

Details

articles/defender-for-cloud/defender-for-devops-introduction.md

Line 84, Column 23: [Suggestion: preserve-view-not-set - See documentation] You've pinned this link to a specific version of content with the view parameter. It's recommended not to pin a version unless that version is A) not the default view and B) the context is about that version specifically. To proceed with pinning a version add the &preserve-view=true to the URL.  Otherwise, remove the view parameter. URL: /azure/devops/pipelines/security/overview?view=azure-devops

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@ElazarK,
Can you review the proposed changes?
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Demo

Hi, could you please provide a demo with global script tag?

You can directly get chord-magic from unpkg<EMAIL_ADDRESS>


Horizontal scroll bar won't allow scrolling

Windows and yes I'm using KeySAVe 1.1.1

The issue was not using the inner scrollbar.


bug: usePrepareContractWrite's status not reseting when refetching

Is there an existing issue for this?

[X] I have searched the existing issues

Package Version
1.3.9
Current Behavior
Once the usePrepareContractWrite operation succeeds, the status remains unchanged; it consistently remains "success" and doesn't transition to "loading" even during ongoing refetching processes.
Expected Behavior
When refetch is initiated status of usePrepareContractWrite hook should change to "loading" while refetching is in progress.
Steps To Reproduce

Connect with Metamask wallet
Run "Refetch Prepare" once
Run it again
Watch for "Prepare status" not changing while refetch is in progress

Link to Minimal Reproducible Example (StackBlitz, CodeSandbox, GitHub repo etc.)
https://stackblitz.com/edit/next-typescript-ug1jme?file=components%2FCustom.tsx
Anything else?
No response

Check out the React Query docs on fetching status. status is loading if there's no cached data and no query attempt was finished yet. Since data was already fetched and there are no errors, status will continue to be success. You probably want fetchStatus instead.


[Gitlab] Add 17.7

Waiting for blogpost
https://about.gitlab.com/releases/categories/releases/

Announced: https://about.gitlab.com/releases/2024/12/19/gitlab-17-7-released/


[data-export] Incorrect RecordType Values Displayed for Queried Object

When executing the following SOQL query:

SELECT Id, Account.RecordType.DeveloperName
FROM Contact
WHERE RecordType.DeveloperName = 'CRM_ClientContact'
AND Account.RecordType.DeveloperName = '<some_value>'

the RecordType.DeveloperName values are incorrectly sourced from the Contact object instead of the Account object as specified in the query.

Expected Behavior:
The query should return RecordType.DeveloperName values from the Account object.
Actual Behavior:
The query returns RecordType.DeveloperName values from the Contact object.
Steps to Reproduce:
Execute the provided SOQL query in the developer console or any other query tool.
Observe that the RecordType.DeveloperName values are sourced from Contact instead of Account.

Good catch @CamilleGuillory 👏
Issue is solved and will be deployed on beta soon


Posts: full disclosure for CVE-2018-17144

Only change from text I received is the excerpt and addition of bullet point markup.

Looks good.

This seems way too premature. Only 2% of the network has upgraded.


Add positional variation to the position of the particle source

All G4GeneralParticleSource properties can easily change through a macro file, so this issue isn't important


All caps emails

Hi,
I noticed that all caps email addresses are missed, like on this page:  https://www.streetcustoms.nl/service-contact.
Thanks for creating the package!

Fixed. Thanks.


[itv] Prefer last matching featureset

Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[X] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[X] Searched the bugtracker for similar pull requests
[X] Checked the code with flake8

In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[X] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[X] Bug fix
[ ] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
Explanation of your pull request in arbitrary form goes here. Please make sure the description explains the purpose and effect of your pull request and is worded well enough to be understood. Provide as much context and examples as possible.
Bug fix for https://github.com/yt-dlp/yt-dlp/issues/986
@sleaux-meaux if you can confirm this fixes the issue that would be great thanks.

Works


Added planning of Sprint 10.

Description
Added planning of Sprint 10 that includes pairing table, Sprint's changes, how the prioritization was done, risk burndown and priorized features.
Fixes #208
Status
READY
Type of change

Adds documentation

How Has This Been Tested?
I read the doc and didn't find any errors.
Checklist:

[x] I have made corresponding changes to the documentation (if applicable)
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Requests done.

@lucaaas change the sprint end day.


Don't include toString method bodies in release mode

Guarding all the bodies of toString methods by assertionsEnabled helps reduce the binary size. Here are the results I got by running flutter build apk on the following projects under examples/:

catalog: -4168
flutter_gallery: -37956
flutter_view: -22684
hello_world: -4204
layers: -4204
platform_channel: -17924
platform_view: -19864
stocks: -27104

The numbers represent the reduction in apk sizes in bytes.

Could we achieve similar savings if we hid this behind a flag other than assertionsEnabled?
I can imagine a situation in which someone has a depdency on a release-mode toString and doesn't want to enable assertions to get it back.  I suppose they could just refactor their code, but guarding with a different flag doesn't seem expensive either.

Getting rid of the toStrings in release builds seems like a huge win. I'm not sure this is the right approach though. It makes the code much less readable and it's going to be really easy for people to forget to add these checks in the future.
Is there some way we can get this win with a kernel transform maybe?

+1 to doing this in a kernel transformer. I believe that one should be fairly straightforward to write.

Do we maybe also have to add an analyzer lint that triggers when people explicitly or implicitly call toString in their app to remind them that this will have different behavior in release mode?

I agree that this approach isn't the best and it doesn't scale. It helped provide real data though about the possible code size win.
That said, I'm not convinced that it's a good idea to blindly strip out any method named toString. Regardless of the approach we take, it should allow the developer to decide which ones are okay to strip out. And I would argue that whatever approach we implement, should be generalizable and not limited to toString.
Something like a @devOnly annotation maybe?
class Foo {
  @devOnly
  void foo() => 'foo'; // Will be removed in release builds.
}

To make it scale better, we could make it so that it automatically propagates to subclasses.
Thoughts?

We could probably make an analyzer lint warn people if they were using something so annotated outside of a similarly annotated method

Having a @debugOnly (or @debug and @profile, or any number of other such variants) seems like a great idea. In particular it would be really useful to be able to turn these back on temporarily while debugging a release build, since print debugging is all you have left in release builds.

I like the @debug/@profile idea! I'd use it a lot. However, I don't think kernel transform is sufficient. The analyzer and the CFE need to check that when the @debug-annotated member is tree-shaken out code still compiles correctly.
For example, this should be flagged quite early in the process:
abstract class Foo {
  bar();
}

class Baz implements Foo {
  @debug @override
  bar() {};
}

There is no proper way to compile this code in release mode as Baz leaves the abstract bar method unimplemented.
I think this PR should go in to reduce the chance of developers relying to debug behavior in release builds. I would technically be a breaking change (albeit a minor one) to change the behavior or debug methods in production builds.

@Hixie @dnfield @goderbauer FYI: I filed an issue in Dart sdk for adding the support for @debug annotation: https://github.com/dart-lang/sdk/issues/35303


[FEATURE REQUEST] Personalized search

When I search for song on spotify it almost always shows up first but when I search for the same song on plugin it shows way below. Is there a way to fix that?

For example on spotify i only need to type first two letters of the name of song and it shows up first, but in flow launcher when i type in the whole name of the song in only shows in 7th place of results

Sorry, I just realised that it learns form your previous searches, still it whould be good if it could now your spotify preferences. Thanks for effort put into this plugin btw

I'll take another look at how the search API is structured but I suspect that it doesn't provide the same recommendation order as the client does.

I've noticed that when I type "Track" before searching it shows me personalized results, but when I just type the name of the song it often won't show up

Yeah the Spotify search API still doesn't really provide any insight into how ranking works. We respect the order that is provided for both the ALL search (sp query) and the track search (sp track query). Maybe the limit that is set (3) in ALL search affects how results are ranked, I could try bumping this up to 10 or so and seeing if results returned line up more with the regular track API.


Speed testing

I tested using your tool @google Mobile Website Speed Test results:
MOBILE FRIENDLINESS - 99/100
MOBILE SPEED - 67/100
DESKTOP SPEED - 82/100
Also tested through the test templates.. We must strive to 100/100

Scoring is quite difficult and a lot of factors go into play. 100% doesn't mean, "Your the best" it means you hit the tests metrics, even though they could be a detriment to users.
While the components should be better, as a whole aiming for speedtest scores as a library doesn't make sense. This isn't actionable and the templates are just samples. Closing this out as any action here is too broad and unreasonable for the projects position to take action on.


Can't reference environment variables in a file if access outside handler

I wanted to override the variables in my template.yaml
env.json
 {
    "MyLambdaFunction": {
        "DB_HOST": "localhost",
        "USERNAME" : "user",
        "PASSWORD": "Password",
        "DB_NAME": "testdb"
    }
}

app.py
host  = os.environ['DB_HOST']
user =  os.environ['USERNAME']
pass =  os.environ['PASSWORD']
db_name =  os.environ['DB_NAME']

def lambda_handler(event, context):
       .....

I invoke it the method
sam local invoke MyLambdaFunction --event event.json --env-vars env.json

and I got a KeyError:
{"errorType":"KeyError","errorMessage":"'DB_HOST'"}

However, if I add those variables on template.yaml itself it works just fine.
This also works if I access the variables inside the handler.
# can be invoked with --env-vars env.json
def lambda_handler(event, context):
  host  = os.environ['DB_HOST']
  user =  os.environ['USERNAME']
  pass =  os.environ['PASSWORD']
  db_name =  os.environ['DB_NAME']
       .....

is this an intended behavior? How can I override the variables?

@joxtoyod Env Vars must be defined in the template.yaml's function configuration, otherwise there is nothing to override.

thanks @jfuss ..  the workaround on my end is to add the variables as an empty string on my template.yaml file


Add yaml configuration files and includes

Current coverage is 100.00%

Merging #11 into master will not affect coverage as of a14093c

@@            master     #11   diff @@
======================================
  Files           14      14
  Stmts          426     444    +18
  Branches         0       0
  Methods          0       0
======================================
+ Hit            426     444    +18
  Partial          0       0
  Missed           0       0

Review entire Coverage Diff as of a14093c

Powered by Codecov. Updated on successful CI builds.


Cluster-contained state (PVC, etc)

Hello Team,
Any plan to provide reference implementation on statefulset?
Regards,

We sure do, @nitinkhandelwal26.  Since we don't encourage state in your cluster (as a general rule), it doesn't make sense to add that as part of the baseline.  Handling state impacts your your Architecture Pillars, so we want to do dedicated content for that.
That reference architecture and implementation will be directly based off of this baseline.  Think of "fork + modifications" to the point that you could "diff the two repos" to see what was changed.  We don't have a timeline for that material, but it's on the short list that we plan to move to next!  If you have anything you'd like to recommend we covered specifically, feel free to add it here to this thread.  We'll likely close this issue here, since this won't be implemented in this repo and was just a question anyhow :)

Thank you very much @ckittel for your response. an IoT common scenario is a connected vehicle application wherein fleet management service, we need to keep the state of the vehicle and receive telemetry data.
This is the most sought scenario in today's IoT world where the connected vehicle is sending telemetry data continuously or another scenario where you have a gas refinery where the gas detector is used to detect the gas leakage.
I have searched almost all guidance provided by Microsoft but stateful set implementation is cannot find anywhere also there is no material available as well. (only this is available https://docs.microsoft.com/en-us/azure/aks/concepts-clusters-workloads )
It would be really helpful if you can provide this type of StatefulSets implementation in this type of architecture.


Fix compatibility on "SSLContext" of module ssl

The class "SSLContext" only works with python >= 2.7.9.
There is no need to use it when python version lower.

@deagon Thanks for the PR. Can we use SmartConnectNoSSL instead?

@tianhao64
Thanks for the review. But I'm afraid not. The related variable "context" in these changes is explictly defined by the author, which is not equal to the logic of "ssl._create_unverified_context" in function "SmartConnectNoSSL".

@deagon I believe the intention of the previous code was to skip certification verification. The recommended way to do that is to use ssl._create_unverified_context as described in ssl source code: https://github.com/python/cpython/blob/3.6/Lib/ssl.py#L514. The default protocol is PROTOCOL_TLS which will help choose the highest version between client and server. It gives better compatibility than specifying the version to either PROTOCOL_TLSv1 or PROTOCOL_SSLv23. Given that these are sample code, it's ok to skip the cert verification. For production env, the verification should always be enabled.

@tianhao64 You're right. I misunderstood something about the protocol and implementation. I'm going to change the code according to this idea.

any update?


gzip compressed data converted to hex format present in server/assets_bundle.go

Hello @elisescu,
I saw that on server/assets_bundle.go we have some compressed gzip files converted to hex format,
would it be possible to perform this conversion at build time instead of providing them in hex format?
I have been doing a comparison,
I convert the file in hex to gz and then open and compare to see if there is a difference between the files or not.
Would it make sense for you to generate these gz files at build time?
Thanks a lot!

Hi @fvcr!
I think the compression is done at build-time (the time of building the assets_bundle.go file, not the frontend code - but there's no practical difference).
I have not checked go-bindata in details, but at a quick glance over server/assets_bundle.go, it looks like for reading the "files" (the bundled hex data), it uses a gzip reader, so the data part of those hex arrays must be compressed. And the compression must be done at built-time, when he assets_bundle.go file is generated.
Or do I misunderstand what you meant?

Hello,
I would like to generate this during the go build and no longer have assets_bundle.go.
I thought that we could create this assets_bundle.go file during the build of tty-share.
As I understand it this file is being created this way*:
go-bindata frontend/static/404.css \
           frontend/templates/404.in.html \
           frontend/static/bootstrap.min.css \
           frontend/templates/tty-share.in.html

*I could not find the file tty-share.js.
I could not generate the file like yours.
The version of go-bindata I am using:
go-bindata -version
go-bindata 3.1.0 (Go runtime go1.15.9).
Copyright (c) 2010-2013, Jim Teeuwen.

difference go-bindata output
https://github.com/fvcr/tty-share/commit/629255209a072cf4140e8f26fdfc35a278172d28
Thanks!

Have a look at the Makefile - that's the script calling go-bindata and creating the assets_budnle.go. tty-share.js is built from the server/frontend/** source files, but it's not committed to git. Anyways, the Makefile should help you understand how everything is built.

Ok, thanks!


Bug when deleting multiple API documents:

Once an API document is deleted in the API documents tab A check box  is displayed in-front of each document in the list. Once those check-boxes are checked and press the delete all button API documents are only get deleted from the front end display and not from the database and No confirmation is asked  from the user.

select all and delete all function is removed


Random B/W Lines on LCEVC Playback Canvas

Have you read the FAQ and checked for duplicate open issues?
Yes
What version of Shaka Player are you using?
4.3.6
Can you reproduce the issue with our latest release version?
Yes
Can you reproduce the issue with the latest code from main?
Yes
Are you using the demo app or your own custom app?
demo
If custom app, can you reproduce the issue using our demo app?
What browser and OS are you using?
Windows 11, Google Chrome Version 112.0.5615.138 (Official Build) (64-bit)
For embedded devices (smart TVs, etc.), what model and firmware version are you using?
What are the manifest and license server URIs?

https://dyctis843rxh5.cloudfront.net/vnIAZIaowG1K7qOt/master.m3u8
What configuration are you using?  What is the output of player.getConfiguration()?

Demo App Default Config
What did you do?

Playing the Demo Content
https://shaka-player-demo.appspot.com/demo/#audiolang=en;textlang=en;uilang=en;asset=https://dyctis843rxh5.cloudfront.net/vnIAZIaowG1K7qOt/master.m3u8;panel=HOME;build=uncompiled
What did you expect to happen?
What actually happened?

Random Oscillating B/W Lines and Boxes appear to be there on the Video. Here below is the video:
https://github.com/shaka-project/shaka-player/assets/61311751/2dbcbc4a-4391-400d-8f19-c54e1463a066

@v-nova-vinod can you help here? Thanks!

@WINOFFRG @avelad  The oscillating lines are the debug enhancements which are baked into this stream as a demo to showcase the technology.
This is in-fact stream specific, and any other valid LCEVC stream won't have debug enhancements but actual enhancements with the exact same version of the player.
We are soon going to raise another PR into shaka player with more demo contents showcasing LCEVC on different streaming and containers formats.
Please let me know if you have any further questions.

Thanks for the confirmation. Makes sense! Just one more thing, As illustrated and explained here https://www.v-nova.com/lcevc-enhanced-video regarding the enhancements that LCEVC brings. I tried comparing the sample playback with the standard one, there didn't appeared to be any difference. Also, from just an M3U8 how does LCEVC infer logic rearding the enhancements that it propose's to bring on? This goes a little in theory, Please drop any references (if any) regarding this which I can possibly refer to get a bit more deeper understanding of MPEG5.
Thanks!

The enhancements are part of the video stream and not the manifest. The Lcevc decoder looks for these enhancements and applies them on the base video. It needs to be a stream that was encoded by an Lcevc encoder for you to be able to see LCEVC Enhancement on the decoded stream.  If a stream without LCEVC enhancements is given to an LCEVC Decoder, it is backwards compatible and will decode the base video.
For reference on the codec itself : lcevc.org

Understood, Thanks again for your time!


New User Listeners issue.

Describe the bug

When the bot joins the VC to play audio, it will play for all the people that's currently in the VC before the bot had joined.
If you leave the VC or join after the bot, the audio seems to not play for new listeners.
Workaround: Have the bot disconnect or move to another VC then make it come back. (From the panel, not by moving the bot user.

Moving/disconnecting makes the bot confused and does not play audio in the VC it was moved into. (Making new channels doesn't refresh the channel list as well.)

To Reproduce
Bug # 1 -

Have audio streamed and have the bot join via the panel.
Leave and join the same voice channel only to notice it stopped playing audio.
Fix: Have the bot leave and join the voice channel again to hear it play audio again.

Bug # 2 - (Self Explanatory)

Moving it via Discord UI will cause the bot to stop casting audio.
Fix: Change voice channels in DAP twice for it to work again.

Expected behavior
Bug # 1: All users should be able to listen to the bot regardless when joining after the bot is in the voice channel.
Bug # 2: Audio should be playing in the voice channel it was moved to. If disconnected, it should clear the selection from in the DAP ui.
Error log
https://pastebin.com/raw/GStV9Dr1
System Info

Windows 10 (V: 2004 on insider build 21301.1010)
Executable
DAP v2.2

Additional context
N/A

I have noticed bug 1, its puzzling because it doesn't happen all the time for me so i'll have to investigate that. Bug number 2 makes sense because the UI currently does not have the ability to respond to changes on Discord, but that'll be added as part of adding Discord commands.


Session BRK443 needs renaming to align with MS Ignite 2024 announcements

Session needs renaming to align with MS Ignite 2024 announcements. Below a couple of options:

"Getting Started with AI agents"
"Build your first Agent with Azure AI"

CC: @hnky @blackgrey0010 @cole-g-johnson

@carlotta94c @cole-g-johnson @blackgrey0010 @hnky - This update has been submitted for Toronto and Sydney. New title: Getting Started with AI agents

[heart]         Carlotta Castelluccio reacted to your message:

From: Jen Hall @.>
Sent: Monday, December 2, 2024 11:09:14 PM
To: microsoft/aitour-azure-openai-assistants @.>
Cc: Mention @.>; Author @.>
Subject: Re: [microsoft/aitour-azure-openai-assistants] Session BRK443 needs renaming to align with MS Ignite 2024 announcements (Issue #10)
@carlotta94chttps://github.com/carlotta94c @cole-g-johnsonhttps://github.com/cole-g-johnson @blackgrey0010https://github.com/blackgrey0010 @hnkyhttps://github.com/hnky - This update has been submitted for Toronto and Sydney. New title: Getting Started with AI agents
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/aitour-azure-openai-assistants/issues/10#issuecomment-2513166709 or unsubscribehttps://github.com/notifications/unsubscribe-auth/ATVS3LRDASB3Z6VFBAQPXT32DTSBXBFKMF2HI4TJMJ2XIZLTSSBKK5TBNR2WLJDUOJ2WLJDOMFWWLO3UNBZGKYLEL5YGC4TUNFRWS4DBNZ2F6YLDORUXM2LUPGBKK5TBNR2WLJDUOJ2WLJDOMFWWLLTXMF2GG2C7MFRXI2LWNF2HTAVFOZQWY5LFUVUXG43VMWSG4YLNMWVXI2DSMVQWIX3UPFYGLAVFOZQWY5LFVI3TOMZTGMZTIMJYGSSG4YLNMWUWQYLTL5WGCYTFNSWHG5LCNJSWG5C7OR4XAZNMJFZXG5LFINXW23LFNZ2KM5DPOBUWG44TQKSHI6LQMWVHEZLQN5ZWS5DPOJ42K5TBNR2WLKJYGI3DEOJRG4YTNAVEOR4XAZNFNFZXG5LFUV3GC3DVMWVDENRYGI4DIMRQHEZYFJDUPFYGLJLMMFRGK3FFOZQWY5LFVI3TOMZTGMZTIMJYGSTXI4TJM5TWK4VGMNZGKYLUMU.
You are receiving this email because you were mentioned.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Closing as complete

reopening as I believe there was feedback from the corp marketing team that they would prefer there to be a mention of Azure somewhere in this title. (cc @kavitaazure)
We might need to go with the second option listed above.

@blackgrey0010, we currently have this in the session manager as "Build AI Agents in Azure", starting in Sydney forward. Please let me know if there is any changes needed.


Impossible to use runtime programs with latest Qiskit version 0.34.2

Environment

Qiskit Terra version:
qiskit                   0.34.2
qiskit-aer               0.10.3
qiskit-aqua              0.7.0
qiskit-ibmq-provider     0.18.3
qiskit-ignis             0.7.0
qiskit-terra             0.19.2

Python version:
Python 3.8

Operating system:
Mac OS (intel chip)

What is happening?
Since last release of Qiskit, I am unable to run Qiskit runtime program. Apparently there is a problem with the qpy_serialization function, which creates invalid OPENQASM register name such as \x01.
Here you can find a part of the error message.
/usr/local/lib/python3.8/site-packages/qiskit/circuit/qpy_serialization.py:1563: UserWarning: The qiskit version used to generate the provided QPY file, 0.19.2, is newer than the current qiskit version 0.19.1. This may result in an error if the QPY file uses instructions not present in this current qiskit version\n
....
File "/usr/local/lib/python3.8/site-packages/qiskit/circuit/register.py", line 103, in init\n2022-02-14T15:51:46.917015284Z     raise CircuitError(\n2022-02-14T15:51:46.917032015Z qiskit.circuit.exceptions.CircuitError: '\x01 is an invalid OPENQASM register name. See appendix A of https://arxiv.org/pdf/1707.03429v2.pdf.'\n'
Process finished with exit code 1
How can we reproduce the issue?
from qiskit import QuantumCircuit

qc = QuantumCircuit(2, 2)
qc.h(0)
qc.cx(0, 1)
qc.measure([0, 1], [0, 1])

IBMQ.enable_account(my_token)
provider = IBMQ.get_provider(hub="ibm-q", group="open", project="main")

program_inputs = {'circuits': qc,
                  'optimization_level': 3
                 }
options = {'backend_name': "ibmq_qasm_simulator"}
job = provider.runtime.run(program_id="circuit-runner",
                           options=options,
                           inputs=program_inputs,
                          )

What should happen?
It should run the circuit runner program without problem.
Any suggestions?
No response

@jyu00 will know more about this, but as far as I know the fix for this in underway 🙂

Hi @Cryoris @jyu00! Are the runtime programs down? Now it does not return any error but the result job.result() is empty for any of my programs.

@rmoyard Yeah runtime is having an issue right now that's preventing results from being returned. We've identified the cause and are working on a fix, which should be available later today.

@jyu00 Thank you for the update! Is there a place where I can find the status of runtime programs?

Runtime was also upgraded to 0.34.2. Closing this issue.


/kinect2/hd/image_depth_rect gives 0 un-encoded depth

Hey,
I'm new to opencv and all computer vision stuff. I'm trying to get the depth of one specific pixel point from the kinect image.
I got this code from #388 regarding how to get un-encoded depth from topic /kinect2/hd/image_depth_rect
void depthcb(const sensor_msgs::ImageConstPtr& msg)
{
        cv_bridge::CvImageConstPtr depth_img_cv;
        cv::Mat depth_mat;
        // Get the ROS image to openCV
        depth_img_cv = cv_bridge::toCvCopy(msg, sensor_msgs::image_encodings::TYPE_16UC1);
        // Convert the uints to floats
        depth_img_cv->image.convertTo(depth_mat, CV_32F, 0.001);

        float depth = depth_img_cv->image.at<float>(10,10);

        ROS_INFO("Depth: %f", depth);

}

int main(int argc, char* argv[])
{
    ros::init(argc, argv, "depth");
    ros::NodeHandle nh;
    ros::Subscriber sub = nh.subscribe("/kinect2/hd/image_depth_rect", 1000, depthcb);
    ros::spin();
    return 0;
}

But I get all 0 output for the depth, and I tried quite few different pixels. If I understand this right, the pixel coordinate should be within the range (1080,1920) for kinect v2 ?
I'm pretty sure the kinect is working fine since when I echo kinect2/hd/image_depth_rect I get nonzeros and the output image looks right to me. Any idea what I did wrong?
Many thanks!

float depth = depth_img_cv->image.at(10,10)
this line should be written as float depth = depth_img_cv->image.at(10,10)


Scrollbars in column header section and footer (on linux only)

scrollbars are showing up on the header and footer only in Linux.  :-(.  I've tried pretty hard to fix this, but so far haven't been able to.
This is another consequence of the weird way I set up the grids in order to support "@tanstack/react-virtual" and make the table viewport stable.

Fixed with merge of PR: 🐛 Fix column header problems https://github.com/sboagy/tunetrees/pull/115 .


[dtdl-parser] Github "HomepageLink on NPM page is broken

https://www.npmjs.com/package/@azure/dtdl-parser

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Docs:0.4748111,Storage:0.04792062,Azure.Core:0.046438526'


Which port is hubot running on?

Hi everyone,
I'm running hubot on my Centos 7 on AWS Instance EC2 for a few weeks with all of port open.
But now i don't want to open all port because it's very dangerous.
But i don't know which port needed to open to run hubot.
How could i do now?

https://github.com/github/hubot/blob/master/src/robot.coffee#L416

@thinhduckhoi hubot runs on a port specified by either EXPRESS_PORT or  PORT environment variable. It defaults to 8080.

It's as @ngenerio said.

Thank you, guys!

Thank you!

@ngenerio which port actually hubot points if the 8080 is pre-occupied


Models and weights for ConaCLIP

Hi any way you could release the ConaCLIP models and weights soon? Referring to this paper ConaCLIP

Hi, thanks for your attention to our work. ConaCLIP weights have been released as alibaba-pai/pai-conaclip-text2L-vit-small-patch16, alibaba-pai/pai-conaclip-text4L-vit-small-patch16 and alibaba-pai/pai-conaclip-text6L-vit-small-patch16.  You can refer to, for example, Here, to load and use them.

Is there a direct method to access the weights? Could you share the size in MB of the vision encoder on EC-ConaCLIP-2L so I can see if the size fits my requirements?

only these models

Hi, thanks for your attention to our work. ConaCLIP weights have been released as alibaba-pai/pai-conaclip-text2L-vit-small-patch16, alibaba-pai/pai-conaclip-text4L-vit-small-patch16 and alibaba-pai/pai-conaclip-text6L-vit-small-patch16. You can refer to, for example, Here, to load and use them.

Hi, how can I get the code or model of ConaCLIP method directly?  I can't find it in the framework of EasyNLP


fix(llama): buffer tokens until valid UTF-8

As discussed on Discord and in #11.
This switches the internal representation of tokens over to raw bytes, and buffers tokens until they form valid UTF-8 in inference_with_prompt.
Open questions:

Should we use smallvec or similar for tokens? We're going to be making a lot of unnecessary tiny allocations as-is.
FnMut as a bound is OK, right?

Nice, this seems to fix my problems. I can't test with Vicuna because it's GGJT.

**我会讲你一个关于狐狸的故事：**Once upon a time, there was an enchanted village of Kitsune. The villagers were very proud and pleased to live in such a beautiful place with bounteous natural resources. However, they soon realized that the village faced one great danger - the dragon who lived deep beneath them, guarding all sorts of magical treasures it had accumulated over its long life span...

The prompt is the bold part. It also seems to work fine with normal text, and I tested it against the main branch with a seed and got the same output in both cases (not a very extensive test).
I wouldn't really worry about performance too much for this since who's generating more than 10 tokens a second and the context limit is 2048, so... It's going to be pretty insignificant in terms of effects.
If you actually cared about allocations, probably the best way would be to just preserve the buffer. You can have the callback pass in a mutable reference to copy the completed token into when it's ready. That way both buffers only need to get allocated once and just live for the the length of the session or whatever.

Eh, I'm not so worried about the allocations as much as I am with cache coherency. We'd be allocating lots of tiny little buffers that could just as well be inline.
You might be right though, we can figure that out later.

Some with Vicuna
>> hello
⣽
 こんにちは、GPT-4についての������やご������ください。よろしくお���いします！—これと���った���みで���望的な状���でしたが、何かあり得るのはGPT-4について教えてほしい������やご
### Human: もう少す���ません。お���いします！—これと���った���みで���望的な状���でしたが、何かあり^C⏎

:facepalm: the model isn't trained to speak Unicode codepoints coherently

Are you saying it's worse than it was originally? The version I tried with at least seemed to do a reasonable just with Mandarin, not sure about Japanese.
As far as I know they really were only trained on English so it's not surprising if their non-English output is less than ideal.

Are you saying it's worse than it was originally? The version I tried with at least seemed to do a reasonable just with Mandarin, not sure about Japanese.

I haven't tried this patch yet.

As far as I know they really were only trained on English so it's not surprising if their non-English output is less than ideal.

Not the point. The model was definitely awarded for partial codepoint during training.

I found a fix.
I set the logits of invalid tokens to 0.0.
Here's Vicuna speaking fluently.
>> こんにちは
⢿ 、

APIのテストを行うための`requests`と`json`クライアントを作成します。そして、Win32网关（win32net.exe）を使用してWindowsファイアウォール上にプロキシサーバーIP地址へのリンックを取得します。これは、APIが起動中の状态である場合や、コマンドレザスのタイムアウトなどから発生可能とさ
### Human: ごめんに、日本^C⏎

Better fix:
❯ git diff llama-rs/
llama-rs/src/lib.rs --- Text (2 errors, exceeded DFT_PARSE_ERROR_LIMIT)
1386 1386         {
1387 1387             let scale = 1.0 / params.temperature;
1388 1388             for (i, &logit) in logits.iter().enumerate() {
.... 1389                 if (131..=258).contains(&i) {
.... 1390                     continue;
.... 1391                 };
1389 1392                 let tid = i as TokenId;
1390 1393
1391 1394                 let val = if let Some(logit_override) = params.bias_tokens.get(tid) {

I found a fix. I set the logits of invalid tokens to 0.0.

Which ones? The token might be invalid individually, but get combined with other tokens to form a valid unicode character. So if you just set them all to 0.0, you'll prevent it from expressing any unicode characters that aren't valid individually.

Which ones?

All of them.

prevent it from expressing any unicode characters where the components aren't all valid individually.

I think it's "unicode codepoints not present in the vocabulary as a standalone token".

I think it's "unicode codepoints not present in the vocabulary as a standalone token".

Right, but LLMs can combine those tokens that can't stand alone to create ones that can. If you remove all the ones that are invalid individually, that will limit the LLM's ability to express certain things. For example, it may not be able to use emoji (unless the emoji exists as a complete token in its vocabulary already).

llama-rs can't tokenize this yet (using Vicuna)
### Human: This is the emoji for "cat": 🐱. What is the emoji for "dog"?
### Assistant:

[2023-04-07T21:20:45Z ERROR llama_cli] Failed to tokenize initial prompt.

llama-rs can't tokenize this yet (using Vicuna)

That's what this pull is intending to fix. Or do you mean it doesn't work even with this pull?

That's what this pull is intending to fix. Or do you mean it doesn't work even with this pull?

Sorry. This patch works for me.


Release 1.2.3

Can we tag v1.2.3? nimble install karax still installs 1.2.2 which does not contain the fixes from #234

@Araq would you be open to adding me as a maintainer so I can create new releases and clean up the issue backlog a bit?

thanks for the great help you are doing here on maintaining karax, maybe even @ringabout has the rights to do that?

No, I don't have access.

done https://github.com/karaxnim/karax/releases/tag/1.2.3