按照步骤操作,机器人没有任何回应

I followed the tutorial, but sending a message to the bot doesn't get any response

windows x64 Running will display this page, and then exit automatically

And when I run chatgpt-telegram.exe, I don't go to the browser, so I added the cookie

config file must be .env ,and cant have any name ,just like config.env

have you used proxies like clash or Shadowsocks?


Font is not exporting to Figma.

All are converting to Roboto.

hey @NoobPro404 - if you are talking about when you view the exported content in Builder, that would be because you haven't uploaded the font to Builder. You can do so by making a free account and adding it in your settings page (builder.io/account/space) and assuming you give it the same name as in Figma, it should pull in as expected
Also, any exported code should have the proper font as well, regardless of if it's uploaded in Builder


support deprecated finders which has a options hash with the :conditi…

…ons as :where
has_many :attachments, conditions: 'xxx' should not be cached, but
has_many :attachments, conditions: proc { |x| x.foo } should be
arity of the scope is always 1 with rails 4.0 / deprecated finders, so we have to look a bit deeper
#<ActiveRecord::Associations::Builder::DeprecatedOptionsProc:0x007fe1687be368
@options={:order=>"created_at ASC, id ASC", :where=>"parent_id IS NULL"}>

@pschambacher

:+1:


About the main file

Hi, I'm going to host this project on cdnjs.
https://github.com/cdnjs/cdnjs/issues/7536
But I meet a problem.
The main of the package.json is load-awesome.js.
But I cannot find load-awesome.js in this project.
Can you help me? Thanks. :smile:

+1


[Bug] disabling NC/LS is broken as of alpha 5

LS/NC do not properly disable as of alpha 5, likely missing an isEnabled check somewhere in loading of views

Fixed, will be included in next release.


Notify user that this's on test network

Right now user just open the wallet and got 1,000 Coins, So we should notify that you're currently using test network (no value at all).

[x] Update banner with testnet symbol in README.md.
[ ] Update banner with testnet symbol in the application.

Thanks


Multiple filter windows are created on incremental update

Problem
Filter windows are created every time items are enqueued.
Environment
Neovim v0.7.0
Minimal vimrc
set runtimepath+=~/.cache/dein/repos/github.com/vim-denops/denops.vim
set runtimepath+=~/.cache/dein/repos/github.com/Shougo/ddu.vim
set runtimepath+=~/.cache/dein/repos/github.com/Shougo/ddu-ui-ff
set runtimepath+=~/.cache/dein/repos/github.com/Shougo/ddu-kind-file
set runtimepath+=~/.cache/dein/repos/github.com/Shougo/ddu-source-file_rec

nnoremap <Space>f
      \ <Cmd>call ddu#start({'sources': [{'name': 'file_rec'}]})<CR>

call ddu#custom#patch_global({
      \ 'ui': 'ff',
      \ 'uiParams': {'ff': {
      \   'split': 'floating',
      \   'autoResize': v:true,
      \ }},
      \ })

autocmd FileType ddu-ff call s:ddu_ff_my_settings()
function! s:ddu_ff_my_settings() abort
  nnoremap <buffer> i
        \ <Cmd>call ddu#ui#ff#do_action('openFilterWindow')<CR>
  nnoremap <buffer> q
        \ <Cmd>call ddu#ui#ff#do_action('quit')<CR>
endfunction

autocmd FileType ddu-ff-filter call s:ddu_filter_my_settings()
function! s:ddu_filter_my_settings() abort
  inoremap <buffer> <CR> <Esc><Cmd>close<CR>
endfunction

Reproduce steps

start nvim and cd to large directory (which has more than 200000 files)
execute <Space>f (start ddu)
type i (open filter window) immediately after the ddu window appears
type <CR> (close filter window) immediately after the filter window appears
wait for items are enqueued

Screenshot

Thank you, but though I closed the filter window, the filter window is automatically opened when the items are enqueued.

Fixed.  Please update.

It seems fixed. Thank you!


Regaining quests are run together during the same campaign

this visual glitch occurs when you lose all holds other than solitude after you get reunification of skyrim also occurs with skirmish and mine and farm quests
(an addition qol change could be to deactivate the waypoints for town,fort,mine and mills etc when a battle for them occurs)

@LABWAZHERE This is strange and potentially bad. Can I see the CWO Debug screen while this is appearing? Every campaign should be offense/defense of one hold at a time.

update: fresh save, still occurred, only with Morthal skirmish at stonehills triggered reunification of skyrim, could that be causing it?

          couple of other things, concurrent sieges still occured

This happened (dancing objective marker)

and liberate markarth quest popped up after winning stonehills (happened a second time as well)
Originally posted by @LABWAZHERE in https://github.com/schofida/CWORR/issues/12#issuecomment-1492794698

So this issue I am still unsure of how you got this to happen. In all of my testing I have never seen this.
So I am not sure if you use Mod organizer or not, but I snuck an INI override into the mod folder. It should turn Papyrus (Skyrim's scripting engine) logging on. Mod Organizer has a feature to allow mods to tweak ini settings so it should 'Just Work'.
If it did, in your My Documents\My Games\ folder, next to you Saves folder, there should hopefully also be a logs folder. If there is, can you zip up the folder and attach to this issue?

i use vortex and i have no clue how to use papyrus on vortex and i have so many mods that moving over to MO would be a nightmare (tried once, never again) i may try to convert them but i have 100 something mods soooooo.......
on another note i have a theory on this myself (coming from an uneducated person) my theory is this is leftover code from conquest (disabled it but idk), this theory has some credit as in conquest when you attack forts you hear a horn being played and i heard one when i got these pictures. but this also is debunked by the fact i never heard it during the fort battles of the most recent update and i haven't uninstalled conquest since then (only disabled)

could this be leftover code from the base game? since the base game doesn't have the sieges when completing these missions it could be the game thinking it is the base game code and activating the "next" hold

I think that I might have gotten to the bottom of this. I will hopefully have fixed in the next release tentatively scheduled for tomorrow evening.

the visual glitch still occurs, yet to determine if regain appears

I will close this. Unfortunately you cannot test this on your existing save.
I discovered thanks to you that setting the quest stage to the 'So and So Completed' stage does not automatically clean up the quest objectives so when you get the same quest, those uncompleted objectives are still hanging out there. I made sure to complete/fail all objectives programmatically before stopping the quest. So it should fix the issue going forward but your save will still have those stale objectives unfortunately.

Issue still persists at least in eves stormcloak save


Addons depend on module state, deduplication required (Cannot read property 'removeListener' of null)

Hello,
I was setting up storybook-addon-knobs for a project and had quite a bit of trouble getting it running. The communication channel setup was failing, breaking the decorated component that should have been rendered with the story with the following trace:
Cannot read property 'removeListener' of null
TypeError: Cannot read property 'removeListener' of null
    at WrapStory.componentWillUnmount (http://localhost:9001/static/preview.bundle.js:52084:26)
    at ReactCompositeComponentWrapper.unmountComponent (http://localhost:9001/static/preview.bundle.js:38889:15)
    at Object.unmountComponent (http://localhost:9001/static/preview.bundle.js:3250:23)
    at Object.unmountChildren (http://localhost:9001/static/preview.bundle.js:38332:26)
    at ReactDOMComponent.unmountChildren (http://localhost:9001/static/preview.bundle.js:43116:29)
    at ReactDOMComponent.unmountComponent (http://localhost:9001/static/preview.bundle.js:40482:11)
    at Object.unmountComponent (http://localhost:9001/static/preview.bundle.js:3250:23)
    at Object.unmountChildren (http://localhost:9001/static/preview.bundle.js:38332:26)
    at ReactDOMComponent.unmountChildren (http://localhost:9001/static/preview.bundle.js:43116:29)
    at ReactDOMComponent.unmountComponent (http://localhost:9001/static/preview.bundle.js:40482:11)

Tracing the problem and finding a solution, I found out the following:

The immediate source of the error is this property call: https://github.com/kadirahq/storybook-addon-knobs/blob/master/dist/components/WrapStory.js#L65
storybook-addon-knobs requires the channel in storybook-addons to be set https://github.com/kadirahq/storybook-addons/blob/master/dist/index.js#L17
The channel is passed on to knob-decorated stories which will fail without it https://github.com/kadirahq/storybook-addon-knobs/blob/master/dist/KnobManager.js#L77
The channel is set when initializing storybook https://github.com/kadirahq/react-storybook/blob/master/dist/client/preview/index.js#L71

After some debugging I confirmed that storybook and storybook-addon-knobs each received their own instance of storybook-addons. Thus, the former would set the channel on one instance and the latter would assume it to be there, only to run across a null.
Given this problem, the solution was to deduplicate storybook-addons. The simplest way to do that I found from https://github.com/webpack/webpack/issues/1165, adding a resolution alias to webpack:
resolve: {
  alias: {
    '@kadira/storybook-addons': require.resolve('@kadira/storybook-addons')
  }
}

Is this expected behavior or a bug?
Thanks for the effort on this tool, it's been very helpful once I arrived at a functioning configuration.

Annex
My addons.js (copy paste from the documentation):
// Default storybook addons
import '@kadira/storybook/addons';

// Knobs
import '@kadira/storybook-addon-knobs/register';

The failing story:
/* @flow */

import React from 'react';

import { storiesOf } from '@kadira/storybook';
// $FlowFixMe
import { withKnobs, boolean } from '@kadira/storybook-addon-knobs';

storiesOf('Show', module)
  .addDecorator(withKnobs)
  .add('conditional rendering', () =>

The following element is rendered conditionally:

      <Show when={ !boolean('hide element', false) }>

Toggle the 'hide element' knob to hide.

      </Show>

  );

function Show(props: { when: boolean, children?: any }) {
  if (props.when) {
    return props.children;
  } else {
    return null;
  }
}

Hi @Ezku, thanks for reporting. We'll look into it ASAP.

This is pretty weird since storybook-addons is a peer dependancy in all addons and it's a dependancy only in the @kadira/storybook.
Yep we can do that Webpack hack. But we assume we don't need to.
Anyway, are you using any linked NPM repos for storybook?

@arunoda This does happen with npm version 2 because it installs two copies. Just checked. Do we need to support npm v2?

Yep. It seems like we need that webpack alias in storybook.
Could you try using that?

Thanks for your attention on the issue. I can confirm that I'm on npm 2.x, @roonyh.

Published a new @kadira/storybook(v2.15.0) version with webapck require.alias as you mentioned.
Now it should work properly.

I'm closing this now. But, feel free to reopen if needed.
And thanks a lot on pointing out the issue with a proper fix.


No support for bit operations

From the spec:

Bitwise operations:
  int ^ int                    # bitwise XOR
  int & int                    # bitwise AND
  int | int                    # bitwise OR
  int << int                   # bitwise left shift
  int >> int                   # bitwise right shift (arithmetic)

Plus:

~ number        unary bitwise inversion (int)

We don't support these operations.

This landed, so one spec violation down!


chore: go version bump to 1.21

... and use "slices" package instead of "golang.org/x/exp/slices"

@xoxys, could you add owncloudci/golang:1.21 to our registry?

hmmm... fails because of https://github.com/owncloud-ci/golang/blob/main/v1.21/Dockerfile.amd64 ... but this is part of this PR. What is the strategy here @xoxys?

I forced merged it :+1:


Conda Issue

The type or namespace name 'Conda' could not be found (are you missing a using directive or an assembly reference?)

Could you add more information

Yes, when i integrated the package i got the error
This type of namespace Conda does not exist in current Context.
I integrated package twice but same error.
May be some Conda class missing or some library

The Conda class comes from the dependency package CondaExtensions.
Your error message would say that this package is not being loaded.
Can I ask how you are installing the package? Is it through OpenUPM or GH and have you added the scoped registries as per the readme?
Thanks

I did it through git hub
I downloaded zip and then dragged folder in unity

If you do that - you do not get the dependencies.
This is a Unity Package Manager package - it is designed to be installed through the Unity Package Manager.
However - if that is the way you prefer to work then you also need to do the same thing for Conda-Extensions

Thank you its working


refactor: ics27 indicator tests of deterministic error codes and message responses

Description
Readjusts the acknowledgement test which ensures the error code is deterministic (and thus can be included in the acknowledgement without halting the network.
In v0.35, tendermint made the construction of the LastResultsHash private. In order to maintain the spirit of the original test which allows us to automatically detect when our expected behaviour changes, I have added in process tests which commit a single transaction via tendermint. I then reconstruct the expected hash and ensure they are equal. To show the code is deterministic, I reconstruct the expected hash again with a different error code and show they are not equal. The same is done for successful transaction (a different message response is used)
I have updated the handling of transaction responses in the ics27 acknowledgement as specified in ADR 003
closes: #XXXX

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against correct branch (see CONTRIBUTING.md)
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards.
[ ] Wrote unit and integration tests
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[ ] Added relevant godoc comments.
[ ] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Review Codecov Report in the comment section below once CI passes

would appreciate a post merge ack from another person cc @seantking @damiannolan
The changes are pretty straightforward, but always nice to have 2 pairs of eyes on important changes


How to understand downsampling

Hello, I'm very sorry to bother you again. When I read your source code, I don't quite understand how your downsampling is calculated. Could you explain it to me, because it is just a simple description in your paper.

Essentially there are 3 phases, I take Ctxt FHEController::downsample1024to256(const Ctxt &c1, const Ctxt &c2) { as an example:

This is needed because some slots will be empty, because the previous convolution has some blank values (because of the stride equal to 2)

Can you explain  line 346 what is in the main.cpp
vector<Ctxt> res1sx = controller.convbn1632sx(boot_in, 4, 1, scaleSx, timing); //Questo e lento
vector<Ctxt> res1dx = controller.convbn1632dx(boot_in, 4, 1, scaleDx, timing);  //Questo e lento
What does this do?  I don't understand it from this place. Can you explain what sx and dx are? How can I understand them better?

I refer you to this, especially the image

Oh, I have understood sx and dx, but I still can't connect the picture you gave me in the downsampling with the code. Could you give me an example? Thank you very much.

The fourth block (in figure) has two branches: sx and dx

both do a downsampling (on the sx immediately, on the dx after the first convolution)

Yes, I already know this, what I don't understand is the line corresponding to the code in the figure below.

Ah ok, I think the "re-arranging" figure is the procedure performed inside the Conv2D blocks, since they have a stride of {2, 2}. This means that the kernel window is shifted by 2 positions (and not by 1, as in previous convolutions), leaving one block empty in our ciphertexts (because the HE convolution assumes that the stride is equal to {1, 1}). For this reason, we have to re-arrange the values in order to fill the empty slots

Simulation of FHEController 1191-1194
def rot(nums,index):
    index=index%len(nums)
    if(index>0):
        return nums[index:]+nums[:index]
    else:
        return nums[-index:]+nums[:-index]

def create_vec(n):
    res=[]
    for i in range(1,n+1):
        line=[i]
        res.append(line)
    return res

def vec_add(vec1,vec2):
    for i in range(len(vec1)):
        if vec2[i]==[0]:
            continue
        elif vec1[i]==[0]:
            vec1[i]=vec2[i]
        else:
            vec1[i].extend(vec2[i])
    return vec1

def vec_mult(vec1,vec2):
    for i in range(len(vec1)):
        if vec2[i]==0:
            vec1[i]=[0]

    return vec1

def gen_mask(n):
    mask=[]
    ci=n
    for i in range(1,32*32*32+1):
        if ci>0 :
            mask.append(1)
        else:
            mask.append(0)
        ci-=1
        if ci<=-n:
            ci=n
    return mask

if __name__ == '__main__':

    with open('test.txt','w') as f:
        fullpacke=create_vec(32*32*32)
        fullpacke=vec_mult(vec_add(fullpacke,rot(fullpacke,1)),gen_mask(2))
        f.write(str(fullpacke))
        f.write('\n------------------------------------------------------------------------------------------\n')
        fullpacke=vec_mult(vec_add(fullpacke,rot(fullpacke,2)),gen_mask(4))
        f.write(str(fullpacke))
        f.write('\n------------------------------------------------------------------------------------------\n')
        fullpacke=vec_mult(vec_add(fullpacke,rot(fullpacke,4)),gen_mask(8))
        f.write(str(fullpacke))
        f.write('\n------------------------------------------------------------------------------------------\n')
        fullpacke=vec_add(fullpacke,rot(fullpacke,8))
        f.write(str(fullpacke))

I am very sorry that I have been simulating your program, the numbers in my code represent subscripts, maybe there is something wrong with my writing, I am different from the diagram you gave, could you give me an example about the subsampling code? Thank you very much.

Ah ok. The "re-arranging" figure is the procedure performed inside the Conv2D blocks, since they have a stride of {2, 2}. This means that the kernel window is shifted by 2 positions (and not by 1, as in previous convolutions), leaving one block empty in our ciphertexts (because the HE convolution assumes that the stride is equal to {1, 1}). For this reason, we have to re-arrange the values in order to fill the empty slots

how do you select the rotation or mask index in the downsample function? about the index 1,2,8? Are these numbers related to the stride or others?

nums = [0, 2, 4, 6, 8, 10, 12, 14, 16, 18, 20, 22, 24, 26, 28, 30, 64, 66, 68, 70, 72, 74, 76, 78, 80, 82, 84, 86, 88, 90, 92, 94, 128, 130, 132, 134, 136, 138, 140, 142, 144, 146, 148, 150, 152, 154, 156, 158, 192, 194, 196, 198, 200, 202, 204, 206, 208, 210, 212, 214, 216, 218, 220, 222, 256, 258, 260, 262, 264, 266, 268, 270, 272, 274, 276, 278, 280, 282, 284, 286, 320, 322, 324, 326, 328, 330, 332, 334, 336, 338, 340, 342, 344, 346, 348, 350, 384, 386, 388, 390, 392, 394, 396, 398, 400, 402, 404, 406, 408, 410, 412, 414, 448, 450, 452, 454, 456, 458, 460, 462, 464, 466, 468, 470, 472, 474, 476, 478, 512, 514, 516, 518, 520, 522, 524, 526, 528, 530, 532, 534, 536, 538, 540, 542, 576, 578, 580, 582, 584, 586, 588, 590, 592, 594, 596, 598, 600, 602, 604, 606, 640, 642, 644, 646, 648, 650, 652, 654, 656, 658, 660, 662, 664, 666, 668, 670, 704, 706, 708, 710, 712, 714, 716, 718, 720, 722, 724, 726, 728, 730, 732, 734, 768, 770, 772, 774, 776, 778, 780, 782, 784, 786, 788, 790, 792, 794, 796, 798, 832, 834, 836, 838, 840, 842, 844, 846, 848, 850, 852, 854, 856, 858, 860, 862, 896, 898, 900, 902, 904, 906, 908, 910, 912, 914, 916, 918, 920, 922, 924, 926, 960, 962, 964, 966, 968, 970, 972, 974, 976, 978, 980, 982, 984, 986, 988, 990]

ori = fill_with_zeros(nums)
# print(output)

m = list(ori)

if len(m) < 1024:
    for i in range(1024-len(m)):
        m.append(0)

ori = np.array(m)

# print(ori.shape)

shift = left_rotate(list(ori), 1)

# print(shift.shape)

# print(ori[:64])
# print(shift[:64])

merge = np.array(ori) + np.array(shift)

# print(merge[:64])

def genmask(n,length):
    c = n
    lists = []
    for i in range(length):
        if c >0:
            lists.append(1)
        else:
            lists.append(0)
        c -= 1

        if c <= -n:
            c = n

    return np.array(lists)

mask = genmask(2, len(merge))

# print(mask)

fullpack_step1 = merge * mask

# print("fullpack_step1" ,fullpack_step1[:64])

# step2
shift = left_rotate(list(fullpack_step1), 2)

# print(shift[:64])

mask = genmask(4, len(shift))

merge = shift + fullpack_step1

fullpack_step2 = mask * merge

# print(fullpack_step2[:64])

# # step3

shift = left_rotate(list(fullpack_step2), 4)

merge = shift + fullpack_step2

mask = genmask(8, len(fullpack_step2))

fullpack_step3 = mask * merge

# print(fullpack_step3[:64])

# print(left_rotate(list(fullpack_step3),8)[:64])

finallOut = fullpack_step3 + left_rotate(list(fullpack_step3),8)

print(finallOut[:128])
# print(finallOut)

def mask_first_n_mod(n, padding, pos):
    mask = []
    for m in range(1):
        for j in range(pos*n):
            mask.append(0)

        for j in range(n):
            mask.append(1)

        for j in range((padding-n-(pos*n))):
            mask.append(0)

    return np.array(mask)

orix = finallOut

print(orix[:32])

downsample = np.zeros(orix.shape[0],dtype=np.uint32)
for i in range(16):
    mask = mask_first_n_mod(16, 1024, i)

    # print(mask[:32])

    maksed = mask * orix

    downsample = downsample + maksed

    # print(i, downsample[:64])

    # print(i)
    if i < 15:
        orix = left_rotate(list(orix), 64-16)

# print(ori[:32])
print(downsample)

Simulation of FHEController 1191-1194
def rot(nums,index):
    index=index%len(nums)
    if(index>0):
        return nums[index:]+nums[:index]
    else:
        return nums[-index:]+nums[:-index]

def create_vec(n):
    res=[]
    for i in range(1,n+1):
        line=[i]
        res.append(line)
    return res

def vec_add(vec1,vec2):
    for i in range(len(vec1)):
        if vec2[i]==[0]:
            continue
        elif vec1[i]==[0]:
            vec1[i]=vec2[i]
        else:
            vec1[i].extend(vec2[i])
    return vec1

def vec_mult(vec1,vec2):
    for i in range(len(vec1)):
        if vec2[i]==0:
            vec1[i]=[0]

    return vec1

def gen_mask(n):
    mask=[]
    ci=n
    for i in range(1,32*32*32+1):
        if ci>0 :
            mask.append(1)
        else:
            mask.append(0)
        ci-=1
        if ci<=-n:
            ci=n
    return mask

if __name__ == '__main__':

    with open('test.txt','w') as f:
        fullpacke=create_vec(32*32*32)
        fullpacke=vec_mult(vec_add(fullpacke,rot(fullpacke,1)),gen_mask(2))
        f.write(str(fullpacke))
        f.write('\n------------------------------------------------------------------------------------------\n')
        fullpacke=vec_mult(vec_add(fullpacke,rot(fullpacke,2)),gen_mask(4))
        f.write(str(fullpacke))
        f.write('\n------------------------------------------------------------------------------------------\n')
        fullpacke=vec_mult(vec_add(fullpacke,rot(fullpacke,4)),gen_mask(8))
        f.write(str(fullpacke))
        f.write('\n------------------------------------------------------------------------------------------\n')
        fullpacke=vec_add(fullpacke,rot(fullpacke,8))
        f.write(str(fullpacke))

I am very sorry that I have been simulating your program, the numbers in my code represent subscripts, maybe there is something wrong with my writing, I am different from the diagram you gave, could you give me an example about the subsampling code? Thank you very much.


Bootloader

First we need to make a bootloader 512 byte in rust so we can tell the BIOS to run the OS

Basic one added


How can I customize my feedback behaviour for android ?

I am trying to do something of this sort -> https://docs.testfairy.com/Android/Customizing_feedback_dialog.html
But  don't see a way to do it via this module? Can someone please help

Hi @apurvajain
What exactly are you trying to do?
Can you please email us to support at testfairy dot com?
Thanks!

@yairba I have tried email multiple times from our enterprise account but did nit get any response.
What I am trying to achieve?
I have made a react native application, the app is upload to the testfairy. We need to collect a feedback from our testers in form a multiple choice questionnaire.
Which is available as in option in android SDK -> https://docs.testfairy.com/Android/Customizing_feedback_dialog.html
I want to do similar stuff but via react native testfairy bridge.
Please help me. Thanks in advance.


🐛 [Accounts]: Wrong IDP selection error will not pop-up again a second time

Issue:
If a user selects the wrong SSO IDP when loging in the flow finishes and an error pop occurs. If the user selects the wrong SSO IDP any additional times the error pop-up is no longer shown.
Expectation:
The error pop-up must occur everytime a user selects the wrong IDP for the registered email

@shrunyan
This might be an issue in the login component in the material repo.
@agalin920 @theofficialnar please confirm.

@mattezra after checking this out on manager-ui, the ssoError does not get cleared when the error alert message is closed. So when the same error message happens on a failed login attempt, the error alert component no longer shows up. I don't know how the error is handled on the website though so I can't provide any input on that.
But it's worth creating a bug ticket on the manager-ui repo so this can be resolved there.


EEI: Account handle

The main issue with the cost of e.g. getBalance() is the fact that the account lookup in the database might be needed. This idea is to split the account loading from the accessing account metadata.
Instead of
getBalance(address, resultOffset);

we should have
handle = loadAccount(address);
getBalance(handle, resultOffset);

Similarly to getBalance() there should be getters for code hash, code, code size and others.
Pros

The cost of account lookup is separated from the cost of the getter.
Getting information about the current account should be cheap because it is already loaded (the handle to the current account could be predefined).
The cost of accessing the same account multiple times is lower.

Cons

Handles must be deterministic as they may leak. Simple solution would be to have an array of loaded accounts and return the index in this array. Each call to loadAccount would append new entry to the array.
Contracts are responsible of accounts management.

Alternatives

Just dump account matadata to memory: loadAccount(memoryOffset). This is simpler, but might waste some memory when contract is not interested in all data. Is is also not extensible, i.e. we cannot change the account representation in the future.

Extension of the alternative 1 where the contract specify the bit mask of account fields it is interested in, e.g. loadAccount(BALANCE | NONCE, memoryOffset). This at least allows adding more fields to the account in the future. But the output would be a mess, especially when getting the account code is considered.

split the account loading from the accessing account metadata

Makes sense to me, and seems to better reflect the actual I/O cost of the operation.

I believe that's not enough. If you deploy a contract that supports the current version 1 there is no way to support future version 2.

Could you explain me why that matters? A contract is written against version 1 and explicitly requests v.1 and gets the result in v.1, that sounds good to me and I can't see a reason why it should care about v.2 or something (unless you want to do something like a dynamic proxy that requests and forwards some account data and while it should be forward-compatible).

Maybe the contract should specify what account fields it is interested in.

Yeah, that's what i meant by bitflags : )

Could you explain me why that matters? A contract is written against version 1 and explicitly requests v.1 and gets the result in v.1, that sounds good to me and I can't see a reason why it should care about v.2 or something (unless you want to do something like a dynamic proxy that requests and forwards some account data and while it should be forward-compatible).

Yes, that will work. For that I'd rather suggest using different function names for new versions. E.g. loadAccountV2().
Previously, I though you meant to return the format version number as a part of the account structure.

Previously, I though you meant to return the format version number as a part of the account structure.

Ah sorry for that, I was writting on the go back then. Should have make this more clear

@pepyakin do you have anything similar in pwasm?

Nope, we don't!


[BUG] Crash on start (with or without camera connected)

Information

Version: 3.0.3
libgphoto2 version: 2.5.27-1

When I tried this out today, on my Arch machine, it crashes node instantly when trying to get the first camera - even if there is a camera connected or not.
Example
const CameraList = require("@tsed/gphoto2-driver").CameraList;
new CameraList().load();

Output
$ node -e "const CameraList = require('@tsed/gphoto2-driver').CameraList; new CameraList().load();"
[2021-11-18T01:00:52.878] [DEBUG] [default] - Load library from libgphoto2
[2021-11-18T01:00:52.941] [DEBUG] [default] - Load port info list...

#
# Fatal error in , line 0
# Check failed: result.second.
#
#
#
#FailureMessage Object: 0x7ffc7817a880
 1: 0x55b4a805e4b3  [node]
 2: 0x55b4a8b4da88 V8_Fatal(char const*, ...) [node]
 3: 0x55b4a848bccc v8::internal::GlobalBackingStoreRegistry::Register(std::shared_ptr<v8::internal::BackingStore>) [node]
 4: 0x55b4a81efb4f v8::ArrayBuffer::GetBackingStore() [node]
 5: 0x55b4a7f98976 napi_get_typedarray_info [node]
 6: 0x7fe519bab430  [/home/me/node_modules/ref-struct-napi/node_modules/ref-napi/prebuilds/linux-x64/node.napi.node]
 7: 0x7fe519bb0f5f Napi::details::CallbackData<Napi::Value (*)(Napi::CallbackInfo const&), Napi::Value>::Wrapper(napi_env__*, napi_callback_info__*) [/home/mef/server/node_modules/ref-struct-napi/node_modules/ref-napi/prebuilds/linux-x64/node.napi.node]
 8: 0x55b4a7f8f123  [node]
 9: 0x55b4a821ecde  [node]
10: 0x55b4a821f9bf v8::internal::Builtin_HandleApiCall(int, unsigned long*, v8::internal::Isolate*) [node]
11: 0x55b4a8a1c6b9  [node]

@olemartinorg The plugin is only supported for nodejs v12. There is a know issue with v14 and higher version here: https://github.com/node-ffi-napi/ref-napi/issues/47
If you downgrade your nodejs version to v12, it should works as expected.
I'm not an expert in C++ and nodejs so I'm not able to fix that. I can try to find another way to manipulate array pointer without using ref-array-napi, but... I'm not sure how I'll doing that. Any help is welcome :)
See you
Romain

Oh, thanks for the explanation! I'm also not familiar enough with C++ to figure that out. Do what you want with this issue, I found other ways around the problem.

Oh ok? What is it?

I just call gphoto2 on the command line instead. It works well enough, and I've made workarounds for the things that are less than stellar.


[5.2] Blade or behaviour

This is a proposal that brings to or operator the same ideia (but not the behavior, of course) of the Null Coalesce Operator.
The ideia of or in Blade is to echo a variable if exists, otherwise a default value is printed. And, of course, all of this using a ternary statement. So, instead of:
{{ isset($name) ? $name : 'Default' }}

We just write:
{{ $name or 'Default' }}

Cool!
But, knowing the ideia of or is to be a short-cut to print data, maybe is a good thing it also verify if the value is not null. Something like:
{{ isset($name) && !is_null($name) ? $name : 'Default' }}

If this change is a breaking change? I don't think so. Who needs to print a null value?
What you guys think?

I think this was already proposed and rejected was it not?

Or, just write...
{{ $name ?? 'Default' }}

I do not remember. But if was closed before: :+1:
Yeah, that works fine for PHP7 people. And to be honest or is useless on PHP7 (since we have ??).
But on my project with PHP 5.6 this is a good thing. I'll extend Blade. :)

I tested options for this back in #10135. At the end of the day, it's not worth trying to support every possible edge case for the or syntax. Ideally I'd like to see it deprecated and suggest using standard PHP syntaxes like short ternary or null coalesce.

@cryode Yeah, this gonna happen naturally I guess.


Update Puppeteer to version 10

Hopefully the updated Chromium-version might help reduce the number of intermittent test failures.
Please find additional information at https://github.com/puppeteer/puppeteer/releases/tag/v10.0.0

/botio test

I haven't gone through everything, but on windows it looks like chrome has changed how subpixel anti-aliasing works.

Thanks!
/botio makeref


Cannot open pull-request when base is not existing as local branch

[RuntimeException]
fatal: No such ref: '1.x'

Should use the base of the org org-name/base-name for fork-point, which also prevents getting the wrong default title :smile:

not sure if i understand the full scenario for this

If you open a pull-request (without the --title option) Gush will try to guess the Title by getting the first commit of the source branch. To make this possible it needs an existing base branch (the base is used as a real branch, so if its not checked out with an actual name you get a fatal error).
If you take an issue with a different base (which is not checked out already) and then later open a new pull-request the base used for taking is none existent.
In this case the 1.0 branch was renamed to 1.x (remotely), but I did not rename it locally yet, so Git failed with a fatal error.


Update TaskNodes Checkbox visual density to depend on the ThemeData.visualDensity

We're updating Checkbox default visual density to not depend on  ThemeData.visualDensity and default isualDensity.standard  in order to match Material Design 3 specifications in https://github.com/flutter/flutter/pull/159081.
This could produce visual changes for the users, however, if the users don't want to follow M3 compliance then they can override the Checkbox.visualDensity to depend on ThemeData.visualDensity.
This PR makes such override to pass customer testing suite in the Flutter PR as super_editor is part of this suite.

@TahaTesser which tests are failing without this modification?

@TahaTesser which tests are failing without this modification?

This is the test
https://github.com/superlistapp/super_editor/blob/90f4fe0729807dd8ed63260e29eb462522e1f949/super_editor/test/super_editor/supereditor_input_keyboard_actions_test.dart#L1262-L1275
Here is a minimal sample to reproduce the behavior.

expand to view the code sample
import 'package:flutter/material.dart';
import 'package:super_editor/super_editor.dart';

void main() => runApp(const MyApp());

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    final composer = MutableDocumentComposer();
    final editor = createDefaultDocumentEditor(
        document: MutableDocument(
          nodes: [
            TaskNode(
                id: '1',
                text: AttributedText('This is a task'),
                isComplete: false),
            ListItemNode.unordered(
                id: '2', text: AttributedText('This is a list item')),
          ],
        ),
        composer: composer);

    return MaterialApp(
      home: Scaffold(
        body: SuperEditor(
          editor: editor,
          componentBuilders: [
            TaskComponentBuilder(editor),
            ...defaultComponentBuilders,
          ],
        ),
      ),
    );
  }
}

With https://github.com/flutter/flutter/pull/159081

With https://github.com/flutter/flutter/pull/159081  and this PR

@angelosilvestre can you cherry pick this?


Adds PATCH method

This pull request contains support for HTTP method PATCH and several other commits needed to fix issues because of switch to latest Sinatra for testing.

Any updates on this topic?

Since Sinatra 1.3 is released I think we can now merge this pull request. @brynary, what do you think?

Any updates on this pull request?  It would be nice to have patch.

+1

What's the status of this?

+1

Merged. Thanks!


JavaScript

JavaScript
User Story
As a bootcamp student
I want the prework notes to be structured on a webpage
so that I can easily find and read info
Acceptance Criteria
Given a Prework Study Guide website
When I view the study guide
Then I can see the four topics I need listed along with a suggestion on what I should study first

Forgot to close this earlier but this is also done!!


cli: Improve pruning documentation

This PR improves the documentation of the pruning modes
exposed as CLI flags.
This specifies what the options mean (ie archive, archive-canonical, or number),
and what the two pruning modes are responsible for (state vs block pruning).
While at it, remove the old note that stated that only finalized blocks
are subject to pruning (which is not entirely true anymore, since we are
also pruning displaced branches -- forks).
Closes #12811.

bot merge

Thank you @lexnv


feat: add tsl0922/ttyd

tsl0922/ttyd: Share your terminal over the web
$ aqua g -i tsl0922/ttyd

How to confirm if this package works well
Reviewers aren't necessarily familiar with this package, so please describe how to confirm if this package works well.
Please confirm if this package works well yourself as much as possible.
Command and output
$ ttyd --help
ttyd is a tool for sharing terminal over the web

USAGE:
    ttyd [options] <command> [<arguments...>]

VERSION:
    1.7.3

OPTIONS:
    -p, --port              Port to listen (default: 7681, use `0` for random port)
    -i, --interface         Network interface to bind (eg: eth0), or UNIX domain socket path (eg: /var/run/ttyd.sock)
    -U, --socket-owner      User owner of the UNIX domain socket file, when enabled (eg: user:group)
    -c, --credential        Credential for basic authentication (format: username:password)
    -H, --auth-header       HTTP Header name for auth proxy, this will configure ttyd to let a HTTP reverse proxy handle authentication
    -u, --uid               User id to run with
    -g, --gid               Group id to run with
    -s, --signal            Signal to send to the command when exit it (default: 1, SIGHUP)
    -w, --cwd               Working directory to be set for the child program
    -a, --url-arg           Allow client to send command line arguments in URL (eg: http://localhost:7681?arg=foo&arg=bar)
    -R, --readonly          Do not allow clients to write to the TTY
    -t, --client-option     Send option to client (format: key=value), repeat to add more options
    -T, --terminal-type     Terminal type to report, default: xterm-256color
    -O, --check-origin      Do not allow websocket connection from different origin
    -m, --max-clients       Maximum clients to support (default: 0, no limit)
    -o, --once              Accept only one client and exit on disconnection
    -B, --browser           Open terminal with the default system browser
    -I, --index             Custom index.html path
    -b, --base-path         Expected base path for requests coming from a reverse proxy (eg: /mounted/here, max length: 128)
    -P, --ping-interval     Websocket ping interval(sec) (default: 5)
    -6, --ipv6              Enable IPv6 support
    -S, --ssl               Enable SSL
    -C, --ssl-cert          SSL certificate file path
    -K, --ssl-key           SSL key file path
    -A, --ssl-ca            SSL CA file path for client certificate verification
    -d, --debug             Set log level (default: 7)
    -v, --version           Print the version and exit
    -h, --help              Print this text and exit

Visit https://github.com/tsl0922/ttyd to get more information and report bugs.

Thanks!


Automate gem releases

As the title said, it is an idea worth consider automating the releases of new gem versions
Benefits

Faster access of the latest improvements and features for gem users
Elimination of issues asking for creation of new releases
Potentially making it easier to compare when a problem what introduced across different versions of gitlab gems

Potential approaches

semantic-release

I saw that the mentioned approach have the necessity to standard all commit messages prefix, and this could be a problem whether it's an open source project (Ok, we have tools to enforce valid commit messages, but I think that is not worth).


NEW_RELIC_LICENSE_KEY in different region on AWS Secrets Manager

Description
I discovered that if the secret of the NEW_RELIC_LICENSE_KEY is not in the same region, this will cause problems as the layer is not able to find the secret and I get the following error:
[NR_EXT] Failed to retrieve New Relic license key No license key configured

I then tried to work around it by looking in the extension code and found the env variable NEW_RELIC_LICENSE_KEY_SECRET. So I placed my secret arn inside it. However this ended up in the following message:
[NR_EXT] Failed to retrieve New Relic license key ResourceNotFoundException: Secrets Manager can't find the specified secret.

The secret is there in the specified arn however I guess the region is ignored.
Steps to Reproduce
Create a sample and place the secret in a different region and point to it.
Expected Behavior
The secret should be pulled from the different region.
Relevant Logs / Console output
Mentioned in the description already
Your Environment
Nodejs with AWS node 18 and respective layer

We will work on this issue.

After conducting an analysis, we find that retrieving secrets from a cross-region Secrets Manager can introduce latency exceeding 1 second to the invocation Duration. We recommend utilising the Secrets Manager in the same region as the Lambda function or storing the License Key as a Lambda environment variable NEW_RELIC_LICENSE_KEY.


Add Fluent namespace to ConfigError in v0.14 API based plugins

One user reports following error:
2016-09-09 09:01:49 +0000 [error]: fluent/supervisor.rb:567:rescue in main_process: unexpected error error="uninitialized constant Fluent::Plugin::GrepFilter::ConfigError

👍  - but, it's better to add tests to assert_raised for these configuration errors.

but, it's better to add tests to assert_raised for these configuration errors.

I agree but this changes several plugins across plugin types.
I think separating PR is good for me.
We have a room for test improvement, use data instead of each and more.

Ah, you're right.


[Model] OOM-B099

Actually this is a skin
Author = Jefe
Model author = EvilEngine
Droids VP Author = Sergeant Todd
JunkVoice VP Author = Slyrr
Source=Readme

URL: https://unrealarchive.org/models/unreal-tournament-2004/O/oom-b099_be878bce.html
Hash: be878bcedffef7e080212b29c47ad1e35407c16f
Current name: OOM-B099

Changed type and name (Battledroids), and set author to Jefe in lieu of multiple authors support.


Localisation

Looks good, but in the meantime other commits have affected code this changes, so need to rebase to make the two fit together. I'd also like to squash the commits down so it's just one big commit.
First of all, presuming you're using the Git command line, can you do (apologies if you know this already):
git remote add upstream https://github.com/rnicoll/cate.git # Tell Git "upstream" is my copy of cate
git fetch upstream # Get everything from upstream
git rebase upstream/master # Base your changes on latest upstream
It will complain about merge conflicts - you'll need to go into the files it says there's problems with (MainController.java in this case), and look for sections with <<<<<<< near them. That will show you what the old code was, and what the new code is, and you need to reconcile the two. When done, "git add" the file to let Git know, and then "git rebase --continue" to continue rebasing. Make sure it all works okay afterwards!
Then, can you do "git rebase -i HEAD~4" (Git rebase, interactive, back 4 commits from HEAD), and on everything but the last commit change "pick" to "fixup" to merge the commits.
Lastly, once that's all rebased, do "git push -f origin " to push the whole lot back.

Many thanks!


Can't use Timber as parent theme with new folder structure

We use Timber as a parent theme and when we try to activate our child theme it says that "The parent theme is missing"

Timber as a parent theme

So you use the starter-theme as a parent theme.
AFAIK the starter-theme is for starting working on it. README says: a dead-simple theme.
Having a parent theme is for not overwriting the the theme's code but the starter theme won't really have new features.
Despite the above I start investigating what is going on...

BTW Welcome to GitHub!

Results are coming in :)
/*
 * Theme Name: Another Starter Theme
 * Template: starter-theme/theme
 */

Add /theme to you child theme's meta header as depicted above.

See #105 for more info on resolution


v4.3 vs v4.2: esp_restart() results in download mode on very same hardware (IDFGH-5050)

Environment

Module or chip used:  [ESP32-WROOM-32E]
IDF version (run git describe --tags to find it): v4.3-dev-3175-g9a2d25191
Build System:         [Make]
Compiler version (run xtensa-esp32-elf-gcc --version to find it): (crosstool-NG esp-2020r3) 8.4.0
Operating System:     [Linux]
Using an IDE?: [No]
Power Supply:         [external]

Problem Description
We have a working hw-sw setup that worked with idf versions 3.x, 4.0, ,4.2. Now we migrated to 4.3 and experience that after calling esp_restart(), board boots into uart download mode. (systematically happens every time)
As hw has not changed a bit (neither our sw) I guess some gpio handling changed between the two revisions. Can you highlight what major changes has been done in the area or can you explicitly point out what to change to make it work?
Thanks,
Viktor

It seems as if v4.3 does not restore gpio settings to default while v4.2 did. I wonder if it is a feature or a bug.

The issue is caused by a block in esp_restart() that was cut short by RTC watchdog. RTC watchdog reset does not restore gpio state to default - is this a bug? Please check https://github.com/espressif/esp-idf/issues/6836 for details.


ARM Deployment Task-Error for Bicep File Paths(#16789)

Task name: AzureResourceManagerTemplateDeploymentV3
Description: To fix ARM Template deployment Task fails for bicep files when the path includes spaces
Documentation changes required: (N)
Added unit tests: (N)
Attached related issue: (Y) [https://github.com/microsoft/azure-pipelines-tasks/issues/16789]
Checklist:

[x] Task version was bumped - please check instruction how to do it
[ ] Checked that applied changes work as expected

@microsoft-github-policy-service agree

@manolerazvan / @rvairavelu  - can this PR be reviewed please? This issue with the ARM Deployment task not supporting spaces in bicep folder paths is super frustrating.

Hi @venkymanda, thanks for creating this PR, I noticed there are few conflicts and also build wasn't completed.
In order to unblock PR checks, you need to bump the task minor version to 239, need to build these changes locally and later commit the changes.
Just to be sure that your code changes isn't breaking anything and serving the purpose, can you follow the below simple steps to test real pipeline scenario?
Local task test
Have code changes ready (checkout to this branch), navigate to task root path ie C:...\azure-pipelines-tasks
Note: tfx build works only on node 8 and 10, so make sure to use node 8/10 while testing the task
Step 1: run "npm i"
Step 2: Build:

node make.js build --task AzureResourceManagerTemplateDeploymentV3

Step 3: Install tfx:

npm install -g tfx-cli

Step 4: Login:

tfx login
Service URL : [your task test org url]
PAT :

Step 5: Upload:

tfx build tasks upload --task-path C:\AzurePipelineTask\azure-pipelines-tasks\
_build\Tasks\AzureResourceManagerTemplateDeploymentV3

New task version will be uploaded to your org, please test the changes by running the pipeline.
Step 6: Delete: // optional and can be done after all the testing is completed.

tfx build tasks delete --task-id {           }

updated PR existing here https://github.com/microsoft/azure-pipelines-tasks/pull/20048

@manolerazvan please close / abandon this PR as there is a PR with unit tests and quotes around the path with file extension appended https://github.com/microsoft/azure-pipelines-tasks/pull/20048 available and ready for merge.
Highly appreciated.

@v-mohithgc  thank you let me do that

@manolerazvan / @rvairavelu - This https://github.com/microsoft/azure-pipelines-tasks/issues/16789 with the ARM Deployment task not supporting spaces in bicep folder paths is super frustrating.
Please close / abandon this PR as there is a PR with unit tests and quotes around the path with file extension appended https://github.com/microsoft/azure-pipelines-tasks/pull/20048 available.
Highly appreciated.


Easier local deployment

Rather than having the manual developer doc instructions, can we wrap everything inside a docker container and single executable script?

Below is a simple script that I am using to deploy a local instance of the app. Also offers the option to restart the database. I don't think we should commit this to the repo:
#!/bin/bash

# REMEMBER TO ACTIVATE YOUR PYTHON VIRTUAL ENVIRONMENT BEFORE RUNNING THIS SCRIPT!
# THIS SCRIPT SHOULD BE RUN FROM THE ROOT LEVEL OF THE REPOSITORY

# Load the environment variables
source .secrets/dtenv_localdb.sh

# Take in an argument called 'restart-db' to optionally restart the database
if [ "$1" == "restart-db" ]; then
    echo "Restarting the database"
    # stop and delete server container
    docker stop $DT_SQL_DBNAME
    docker rm $DT_SQL_DBNAME

    # start the postgres docker container
    docker run --name $DT_SQL_DBNAME -e POSTGRES_PASSWORD=$DT_SQL_PASS -p 5432:5432 -d $DT_SQL_USER

    # create the database
    createdb --host $DT_SQL_HOST --username $DT_SQL_USER $DT_SQL_DBNAME -W
fi

# Run the backend. This will run the backend in the background to leave the terminal open for the frontend.
cd dtbase/backend
./run_localdb.sh &

# Run the frontend
cd ../webapp
./run.sh

# NOTE: THE BACKEND WILL CONTINUE TO RUN IN THE BACKGROUND AFTER THE TERMINAL IS CLOSED.


python312Packages.google-cloud-asset: 3.26.3 -> 3.26.4

Changelog: https://github.com/googleapis/google-cloud-python/blob/google-cloud-asset-v3.26.4/packages/google-cloud-asset/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

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

[ ] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 342662 run on x86_64-linux 1

4 packages built:

python311Packages.google-cloud-asset
python311Packages.google-cloud-asset.dist
python312Packages.google-cloud-asset
python312Packages.google-cloud-asset.dist


Split conformance tests into 4 major workflow categories

Resolves #108

LGTM. Can somebody retrigger the tests? Looks like intermittent:
"ERRO Deadline exceeded: try increase it by passing --deadline option "

For an example of what these test results look like, see the table on the README here:
https://github.com/bloodorangeio/oci-distribution-conformance-results/tree/split-tests

LGTM
Good stuff


Browser heading

Can I modify browser heading for Hygieia dashboard UI? I tried modifying favicon.ico but not working.

I presume you're talking about the static header with the Hygieia logo?
If so, you should be able to edit src/app/dashboard/views/navheader.html to your heart's content.


Add new .spec.CSI field in storagecluster

This commit adds new field .spec.CSI to handle
read affinity settings.
This commit also makes use of Rook's Cephcluster
.spec.CSI section to pass read affinity and
cephfs kernel mount options.

/hold
needs one more rook pr to be merged to fix missing DeepCopyInto() of ReadAffinitySpec.

/unhold


Improve the model for the sprig on front page

The model for the sprig on the front page is fairly rudimentary. I am interested in cleaning up / optimizing the model to make it look nicer and perhaps more performant.

@Sheepy3 can you specify what specifically you'd improve / optimize, and how?  Also, highly suggest separating out perf work (which I'd file under tech debt) vs any cosmetic improvements/refinements/geometry modifications (feature request).


Add support for snaps

Signed-off-by: Dragan Stancevic<EMAIL_ADDRESS>

Closing due to inactivity. If you find some time to implement the required changes (+ to update the snapcraft.yaml file to RetDec version 3.3, which is the current version), please re-open.

There is a currently a snap for this in the snap store. It hasn't been updated since 2018.


Add a command that redos the last command that was executed

Please add a "repeat last command" command and keybinding, as sometimes found in other other apps, e.g. Microsoft Office.
I want to press a shortcut, and for it to repeat whatever the last command was, so I don't have to find it again in the menu or the ctrl-shift-p box.
This is especially useful when doing a lot of editing.

One thing that I don't think is super clear from the description of this issue is that what I think should be added is the ability to re-run the last command, no matter where it originated from. I wouldn't want it to just "re-run the first command that shows up in the command palette" because that won't always be the last command you ran. For example, hitting "CMD+W" to close an editor doesn't show up in the command palette as the "last command", but that's what I'd want to be the last command if this feature was implemented.

I'm closing this as this functionality is available - ctrl-shift-p already shows the last command, so just pressing enter performs this task.
@macintacos Please open another issue for that, it's quite different.

@lonix1 while I will end up creating another issue for what I described, why close this one? There isn’t a command to re-do the first command that shows up when you hit “Ctrl+shift+p”, so I’d argue that this functionality is still not available in the way that you initially described. There’s no way to map a shortcut to execute the last command you ran (ignoring my requirements that I mentioned in my previous comment), is there?

ctrl-shit-p and enter - repeats the last command

But that's not really a standalone command to redo the last action, that just happens to be the way the command palette works.
I had thought that this issue was covering "there should be an explicit command in VSCode to execute the last command", one that presumably you could map at will so that it can be a completely different shortcut than pressing ctrl-shift-p and then hitting "enter" (which is one more key than I'd want to be honest).
I guess I'll open another issue that asks for that, but it's just going to look like a duplicate of this one.

ok

I agree with @macintacos, an explicit action with this behavior would be more convenient (to map it to a concrete shortcut at your will).
Is there any expectations or ideas on this?

Ctrl-shift-p + enter works if the command did not have arguments. Otherwise the argument selection needs to be repeated. Would be great to have a shortcut that just repeats the last command as previously executed.

If you want to repeat a command in the terminal, try pressing the “up arrow key + ENTER”. This worked for me. Hope this helps. :)

If you want to repeat a command in the terminal, try pressing the “up arrow key + ENTER”. This worked for me. Hope this helps. :)

@aprilsmartt this is about VSCode commands, not terminal commands.

@aprilsmartt this is about VSCode commands, not terminal commands.

3 years have passed since this issue was first opened. Any progress?

is there any progress on this matter? would love to have a command id which i can use for a keybinding to quickly repeat my last action. it would be easier than to redo some weird shortcuts with 5 fingers in crazy positions

With the shell integration, you can do it in two steps: open the "run recent command" panel (which you can bind to ctrl+r for example), then press enter: . The commandId is workbench.action.terminal.runRecentCommand.
https://code.visualstudio.com/docs/terminal/shell-integration#_run-recent-command

I would also very much appreciate this after getting used to having it in editors like Vim or Helix.
Just to be clear, ctrl + shift + P, enter works but only if you actually executed the command through the command palette. For me this is an important distinction; I want a key I can press to repeat anything, even trivial commands such as cursorUp or workbench.action.closeActiveEditor. Is there a history of executed commands? If so, I would be nice to be able to repeat whatever the last one was.

is this change being considered?


Fix Typo in CONTRIBUTING.md

Fixes #503
I have fixed the typo in CONTRIBUTING.md.
This PR is part of GSSoC'23.

Good @Sriparno08, keep contributing to fintech repo.


Fix iterating over a FileCollection containing dependency artifacts from a thread that is not managed by Gradle

Fixes #?
Context
This used to work by coincidence when the file collection was declared as a task input or eagerly resolved in a task action, though was not really supported, and broke in Gradle 7.4. This PR restores the behaviour and adds some tests (thereby making it supported, I guess).
It is still an error to trigger dependency resolution from a thread that is not managed.
Contributor Checklist

[ ] Review Contribution Guidelines
[ ] Make sure that all commits are signed off to indicate that you agree to the terms of Developer Certificate of Origin.
[ ] Make sure all contributed code can be distributed under the terms of the Apache License 2.0, e.g. the code was written by yourself or the original code is licensed under a license compatible to Apache License 2.0.
[ ] Check "Allow edit from maintainers" option in pull request so that additional changes can be pushed by Gradle team
[ ] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective
[ ] Provide unit tests (under <subproject>/src/test) to verify logic
[ ] Update User Guide, DSL Reference, and Javadoc for public-facing changes
[ ] Ensure that tests pass sanity check: ./gradlew sanityCheck
[ ] Ensure that tests pass locally: ./gradlew <changed-subproject>:quickTest

Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation
[ ] Recognize contributor in release notes

@bot-gradle test and merge

OK, I've already triggered a build for you.


Support filtering of baselines

atm the tool is able to analyze existing baselines and provide a summary as a result, grouped into categories:
Overall-Errors: 41
  Classes-Cognitive-Complexity: 70
  Deprecations: 2
  Invalid-Phpdocs: 5
  Unknown-Types: 1
  Anonymous-Variables: 4
  Native-Property-Type-Coverage: 1
  Native-Param-Type-Coverage: 27
  Native-Return-Type-Coverage: 4
  Unused-Symbols: 3

it would be great if I could filter existing baselines, based on category, and get back a new baseline.
this would allow me to work with focus on a certain problem area, without manually searching the existing full-project-baseline

example 1 (which outputs all filtered content on stdout)
phpstan-baseline-analyze '*phpstan-baseline.neon' --filter=Deprecations

example 2(which outputs the content into new-baseline.neon)
phpstan-baseline-analyze '*phpstan-baseline.neon' --filter=Invalid-Phpdocs > new-baseline.neon

the result should be rendered in neon-format, like the default PHPStan baseline, so we can further process it with whatever tool which supports the default phpstan baseline format

Additional ideas

regenerate/filter existing baselines so all errors machting the filters will be deleted from the baseline. That way a regular phpstan analysis will match these errors
filter by phpstan native error identifiers


Assignments should not be made from within sub-expressions

What is the code smell/issue?
Assignments should not be made from within sub-expressions
How is this code smell/issue relevant?
Assignments within sub-expressions are hard to spot and therefore make the code less readable.
How is this issue resolved?
We have removed assignment out of this expression and have used the add function after the assignment operation

Looks good


Add javadocs for components and system testinhg

Reviewer checklist

[ ] Ensure relevant issues are linked (description should include text like "Fixes #")
[ ] Ensure any appropriate documentation has been added or amended

Coverage remained the same at 100.0% when pulling eb7dc776c208f0b346851a9467a246bf74cd79d1 on docs into bf6ab74121ecae5621161efd20b09ad9ac063743 on main.


Error Reporting not responding

I was enabling error reporting to determine my issues and concerns on iambic import issues I am facing.  And the https connection to error-reporting-prod.iambic.org/report_exception is notwortking or timing out.
To Reproduce
Steps to reproduce the behavior:

Enable Error Reporting in Config
core:
exception_reporting:
enabled: true
include_variables: false # Do not enable this feature unless you are certain that no sensitive information will be included in the report
automatically_send_reports: true
email_address: address

Error Message
2024/06/26 14:13:02 [error    ] Unable to run exception reporting logic
error=HTTPSConnectionPool(host='error-reporting-prod.iambic.org', port=443): Max retries exceeded with url: /report_exception (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x7f1c62292c40>: Failed to establish a new connection: [Errno -2] Name or service not known'))
Community Engagement
Your vote counts! Please support this bug report by adding a 👍 reaction to the original issue, which will aid the community and maintainers in addressing this problem.
Please refrain from adding "+1" or "me too" comments, as these create unnecessary noise for issue followers and do not help in prioritizing the issue. If you wish to contribute to solving this issue or have submitted a pull request, please leave a comment.

Thanks for the heads up. I'll try to get it back up and running within the next week.


Patient Document

Patient Document
Create a new model which stores the patient's document

Add in the time entered and the document which they stored

Allow for CRUD of document

Upload to Google Drive, try with dummy data


webgl examples not working on macos/safari

http://localhost:8080/parquet.html
[Error] Unhandled Promise Rejection: TypeError: null is not an object (evaluating 'ctx.getExtension')
(anonymous function) (filament.js:196)
App (parquet.html:42)
(anonymous function) (parquet.html:36)
taskFinished (filament.js:65)
(anonymous function) (filament.js:126)
promiseReactionJob
commit 1c726fcef6c9838a9ee17eb358d34368bf8d6c9b (origin/master, origin/HEAD, github/main, master, github_master)
Author: Philip Rideout<EMAIL_ADDRESS>Date:   Fri Aug 14 12:04:02 2020 -0700
Safari Version 13.1.2 (156<IP_ADDRESS>.3)
Macos Version 10.15.6 (19G2021)

That's because Filament requires WebGL2 which is not enabled in Safari yet (it should be in the new macos). Please use Firefox or a chromium based browser in the meantime.

@romainguy does chrome on ios support webgl2?  On mac, it appears to work.

Apple forbids to use a different engine than Safari’s on iOS. So Chrome uses WebKit there.


chore: enable colored help screen

Probably personal preference, but I find it easier if commands have
colored help as it is easier to distinguish the different sections of
the help screen.
People who do not want colors in CLIs almost always have NO_COLOR
set anyway, so this won't affect those people, as clap follows the no
color specification: https://no-color.org/

@SirWindfield Thank you for your contribution! I feel the same way!


Profiling not working on latest proxy

The current build does not support heap profiler using proxyv2:latest image
possibly from
commit 4c0d2d2a36996f2ec7d00b519ea90068629a2c55
Author: Dmitry Rozhkov<EMAIL_ADDRESS>Date:   Wed Oct 7 20:32:22 2020 +0300

    memory: switch to the latest tcmalloc for x86_64 builds (#13251)

We probably need to adjust how proxy is build

Reverting to gperftcmalloc to unblock release
Future plan here #28250

I also could not do cpu profile, not sure if that is the same root cause?

Yes. The profiler is gperftools functionability
See
https://github.com/google/tcmalloc/issues/2#issuecomment-587161590


Need to update ReadMe doc

As you have changed the "firebaseConfig' constants names, so please update README.md file with updated env file variables.
It should start with "VITE_", like -
VITE_FIREBASE_API_KEY=AIzaKJgkjhSdfSgkjhdkKJdkjowf
VITE_FIREBASE_AUTH_DOMAIN=yourauthdomin.firebaseapp.com
VITE_FIREBASE_DB_URL=https://yourdburl.firebaseio.com
VITE_FIREBASE_PROJECT_ID=yourproject-id
VITE_FIREBASE_STORAGE_BUCKET=yourstoragebucket.appspot.com
VITE_FIREBASE_MSG_SENDER_ID=43597918523958
VITE_FIREBASE_APP_ID=234598789798798fg3-034

Oh yes I forgot to edit the README file 😅. Thank you


API review feedback for core

Closing this as duplicate https://github.com/Azure/azure-sdk-for-java/issues/5686


Spfx web part - not able to programmatically open property pane in Teams

Category

[ X] Question
[ ] Typo
[ ] Additional article idea

Question
I have an Spfx web part using the PnP Placeholder control (https://pnp.github.io/sp-dev-fx-controls-react/controls/Placeholder/) and having the button open the property pane with the following 'onConfigure' handler:
props.context.propertyPane.open();
This works fine when the web part is on a SharePoint page. But when added as a Teams tab, the above code only works, if the property pane has already been opened before (by clicking 'Settings' from the tab dropdown menu).
If the web part loads and the user immediately clicks the button, then nothing happens.
It seems some javascript for property pane is not properly loaded until 'settings' has been clicked.
Environment details (if applicable)

Your Developer Environment:  Windows 10
Target Environment: Teams
Browser(s): [e.g. Chrome v79 | Safari v22 | FireFox v31 | IE v11 | Edge | Chromium Edge v79]
Tooling: SPFx v1.11.0

Sorry for the delay.  Teams doesn't really allow you to modify the settings of an app w/o going through the settings flow.  Thus, the property pane won't open outside of that flow.


[Feature][3.5][Ready] Custom headings and titles

Adds the ability to change what the title, heading and subheading of each CRUD page says.

title = what shows up in the browser tab (meta title)
heading = biggest text on page (h1)
subheading = smaller text next to the biggest text on page

The second parameter is the controller method where it will be used. The action.
Example:
$this->crud->setTitle('NEW ENTRY', 'create');

$this->crud->setHeading('Invoices', 'show');
$this->crud->setSubheading('Showing the details of an invoice', 'show');

$this->crud->setHeading('Monsters', 'index');
$this->crud->setSubheading("A huge entity which makes no sense, that we've created so you can see a lot of features", 'index');

There are a few differences from #1347

added Heading customization (can include HTML)
added Subheading customization (can include HTML)
single API call for each of the customizations (no more setShowTitle), which on the one hand makes it easier to remember and allows the same functionality on future Operations, on the other hand is less sexy;
connected the title to the Current Controller Method (the ACTION); that's what's used as a second parameter; this might not be as intuitive as one wants, since you do have to know the controller method where it's used... but it does allow us to use the same feature in all future actions too;

My reservations regarding this PR are:

I'm not really sure a $fallback is needed as parameter. We mostly use that in the views. But  maybe plain old null-coalescing operator $crud->getTitle() ?? 'Some fallback' would have been easier to read than $crud->getTitle('Some fallback');
Not sure about using the Action as a second parameter; it's definitely more future-proof; and customizable; it allows people to use it for their own controller methods; but... it does require people to know what controller method is being used:

for create, it's create
for update, it's edit
for preview, it's show
for listing, it's index
for revisions, it's listRevisions
for reorder, it's reorder

Thoughts? @AbbyJanke ?

Ok so I removed the $fallback - I was right, much easier to read like this :-) Don't you just love it when you're right? :-))
Would love some feedback on using ACTION as the second parameter - what you guys think, if it's too cryptic or it's actually a good thing because it makes people look at the underlying code. If it's good because it makes it easier to understand WHERE it happens or bad because it prevents us from changing the method names in the future:

for create, it's create
for update, it's edit
for preview, it's show
for listing, it's index
for revisions, it's listRevisions
for reorder, it's reorder

A sexier option would be to use the OPERATION name, but this feature will only be introduced in 3.5. Plus, what if an operation has multiple methods with views? You'd need to customize each one to a different thing and you couldn't do that by specifying the operation - you'll still need to specify the controller method.

perfect to merge this with #1504 ;-) :-D

@tswonke agreed!

what about allowing an array for actions?
e. g. $this->crud->setTitle('NEW ENTRY', ['create', 'edit']);

As long as available actions are commented in the DocBlocks and are as close as possible to existing (e. g. access), I think they are ok.
In the future (4.0...?) one can think about something more "change-stable"...

@tabacitu Was going through all my PRs and to pull some files i had commited before but not merged yet, and noticed that we merged custom headings and title but there is no documentation that i can find on it for the website. was going to commit an edit for the docs but couldn't decide where to put it.
TLDR; need documentation for this.

@AbbyJanke I added the docs a while back, I don't know if it was before/after your post though... Here they are, under Operations - https://backpackforlaravel.com/docs/3.6/crud-operations#titles-headings-and-subheadings
In hindsight, maybe we should put links to this in other places, maybe in each operation...


nullable timestamps not null when in $dates array

What was the conclusion on this issue?
Closed after a week and no description provided!

Laravel developers covering up the pitfalls of Carbon dating. Now that's a conspiracy!

I solved this! Under Model.php, under attributesToArray() function, it looks like they tried to catch this by setting a continue under the foreach to this:
if (! isset($attributes[$key])) { continue; }
Obviously, this will not catch an empty string.  So, slightly modifying it to this:
if (! isset($attributes[$key]) || $attributes[$key] == "" || null == $attributes[$key]) { continue; }
Will solve the problem, and I don't think this deviates too much from the intent of the framework in this spot.


Move data files into separate directory

This will declutter the data handling a bit and make the program
more flexible because all the inputs can be configured in one place.
It also avoids typos in filenames throughout the code and missing
filename updates during refactoring.

Yes, it's a good idea to move files into a separate directory. Also, avoiding typos in file names is also a good change. I plan on getting around to fixing those issues.
There are a couple changes that I think would introduce bugs, such as in the consolidate_json function. I'll look more closely at the changes at some point. Although, the bugs might become a non-issue, since I might get rid of multiprocessing anyway. It was just a way for me to practice using multiple processes; threads would work better in this case anyway, since scraping is I/O constrained.


Add unit test that reveals a bug in HierarchicalIterative

When using comparison types different from Equality, access to right hand side fails.

The latest commits fix the unit test. I will try with other examples before merging.


Add tests for cli tx-build-raw command for different scenarios

This removes Excercise5 and Excercise6 under the Pioneers module, as they are covered by added tests.

Is there any way for me to see the logs for why the ci/pr failed?

@mesudip are you still interested in this PR?

Closing because of inactivity


Rationalise all stylesheets to SCSS

Some were in SCSS, others were in SASS. SCSS is more familiar, so let's stick with that.

Looks good to me; I wonder if there's anything we can do to get git to recognise that these are all moves and then changes within the moved files, though?

@SteveMarshall there is: set diff.renames = true. Git doesn't record moves, but detects them. There's not much we can do to make GitHub detect them, however.

To clarify that a bit, if this commit is restructured so that the rename and the changes are separate commits, the default behaviour of Git will still list the rename as deletions and additions, unless diff.renames is set.
In this case, the files don't have any history beyond the fact that they were imported from the old PVB (where their history was already mostly obscured by the "open sourcing" dump) so I don't think it helps at all.

Yeah, I figured that was the case, but thought I should mention it just in case you knew of some magic trick to make renames more obvious to GitHub (I certainly don't). Given that, I'll merge…


ceph-salt-formula: tell ceph orch the right number of mgrs

Fixes: https://github.com/ceph/ceph-salt/issues/101
Signed-off-by: Nathan Cutler<EMAIL_ADDRESS>

in general, it's a bit problematic if we try to be more intelligent than cephadm. There is a PR for cephadm to make the command line arguments to finally make sense: https://github.com/ceph/ceph/pull/33523

@sebastian-philipp I don't feel like ceph-salt is "tyring to be more intelligent" than cephadm/orch. It's just issuing the documented command for creating the remaining MGRs. Presumably the documentation will change and then ceph-salt will have to be changed again (sigh).


HTTP health checks

Hi!
What is the best way for health checking HTTP-based services? For instance we have Spring Boot app and it exposes /health endpoint. Is something like this supported in CF?
Thanks!

Hi @bsideup,
Currently Cloud Foundry will not check your applications for health based upon a user provided endpoint. The DEAs monitor the process of each instance of your app and use that to check on the state of your instances. More accurate and specific health checking is not built in to open-source CF.
I'm curious about the use-case for this. Are you just trying to make sure that your app is up, or is this for some sort of service discovery? For uptime checks you can also try Pingdom or other related services.

@zaksoup healthchecking is important for rolling updates, self healing, and few more cases :) This is the reason why I surprised that it's not available in CF :)

@bsideup What do you mean by self-healing? We (as in CF) monitor your app's pid and that it is listening on the bound port. The health monitor will make sure that if any instance dies it is brought back up. That being said, it is not possible to configure the health monitor to hit a specific endpoint or do much more than attempt to restart the app if it stops responding on the bound port.
As for rolling updates, we don't actually support true rolling updates. We recommend a blue-green deploy process for deploying zero-downtime updates to your apps. Please see: https://docs.cloudfoundry.org/devguide/deploy-apps/blue-green.html

Hi @bsideup,
We are closing this issue due to inactivity. Hopefully your concerns have been addressed, but feel free to reopen this if you have more questions.
Thanks,
@danlavine && @dkoper, CF Release Integration Team


Add support for non-ascii fields in settings.

This patch adds support for using unicode strings that contain non-ascii characters in settings.
Trying to use non-ascii characters for contact person's given name for example, would fail with unicode encode errors in various places.
The most common settings that may contain non-ascii characters are:
{
  ...
  "contactPerson": {
    "technical": {
      "givenName": "uTéçhnïçäl Nämé",
      ...
    },
    "support": {
      "givenName": u"Süppört Nämé",
      ...
    }
  },

  "organization": {
    "en-US": {
      "displayName": u"Sérvïçé prövïdér",
      ...
    }
  },
  ...
}

Thanks for contribute.
Can you add custom test settings and specific unit test instead of modify the settings and the test that we had?

Thanks for the quick response @pitbulk!
Sure, I will do that tomorrow.

@pitbulk I made the requested changes and rebased.


UI issue of 'Submit for approval' button under purchase module

[x] bills

[x] purchase order

[x] make above module submit for approval as like that of vendor credit

issue fixed !


ARROW-3982: [C++] Allow "binary" input in simple JSON format

Since rapidjson doesn't validate UTF8 by default, we can represent arbitrary binary bytes in the JSON input (bytes < 0x20 have to be represented as unicode escapes).

Hm looks like this got touched by https://github.com/apache/arrow/commit/ce9c6e3914274dcaf7806159ea5373e0cb632727. I will rebase quickly


Output build ID

Occasionally the ovr platform util seems to get stuck on uploading symbols, but only when in combination with uploading a build.
For us, it's useful to separate the two processes. To do this, we need to read the Build ID from the output of the upload-quest-build command and output it as a variable.
Currently we use powershell to do this which is fine but it would be cleaner if this action did so
For example:
- name: Download OVR Platform Util
  shell: pwsh
  run: curl "https://www.oculus.com/download_app/?id=1076686279105243" -L -o ovr-platform-util.exe --silent

- name: Upload App Lab Build
  id: app_lab_upload
  shell: pwsh
  run: |
       $success = $commandResult -match 'Build ID: ([0-9]*)'
          if ($success) {
            $buildId = $Matches[1]
            echo "Build ID: $($buildId)"
            echo "BUILD_ID=$buildId" >> $env:GITHUB_OUTPUT
          } else {
            Write-Warning "Unable to find build ID in command output"
            echo $commandResult
          }

- name: Upload App Lab Symbols
  shell: pwsh
  run: "./ovr-platform-util upload-debug-symbols --parent ${{ steps.app_lab_upload.outputs.BUILD_ID }} --app-id ${{ secrets.META_APP_ID }} --app-secret ${{ secrets.META_APP_SECRET }} --debug-symbols-dir ./symbols_folder --debug-symbols-pattern *.so --disable-progress-bar"

Interesting I haven't had much issues with uploading symbols.
This is a good addition.

@camnewnham btw you can use my setup-ovr-platform-util step to download as well if needed.


FORGE server java8 output debug log

A couple of days ago the server started to output debug logs files in the log folder.
I did not change any config or properties to enable it.
How do I stop the debug log from being generated?
image: itzg/minecraft-server:java8
  - TYPE=FORGE
  - VERSION=1.12.2

.forge-installed-1.12.2-<IP_ADDRESS>55

Did a quick Google search and I'm not finding any obvious answers that I could work into the image
https://www.google.com/search?q=forge+enable+debug+logs

Something related to this maybe?
[main/DEBUG] [FML]: Injecting tracing printstreams for STDOUT/STDERR.


Consolidate time specification to one of ms or s

Promises:

promise timeout time is specified in absolute ms
created_at, completed_at is in ms

Schedules

promise timeout time is specified in absolute ms
created_at is in ms

Locks

lock expiry_in_seconds is specified in relative s
expires_at in relative ms

I propose we choose either seconds or milliseconds and stick to it consistently, aka what granularity do we want to expose to the user? On a database level let's store everything as ms so as to maintain maximum granularity if we ever choose to switch.
Corollary:  should we indicate the units in the key name, eg: timeout or timeoutInMs? Whichever we choose we should also use consistently.

Since ms is already the convention let's go with ms. This keeps the server (and database) at a low level of granularity and conveniences can be implemented on an sdk level.

Part of: #resonatehq/resonate#2


Hyper not loading .hyper.js config

I installed hyper v to run with bash, but I change the shell and it doesnt happen anything.
I also tried to change font family to Fira Code and it doesnt work too.
Windows version	10.0.18362 compilation 18362
I'm editing the file on -> AppData\Roaming\Hyper -> .hyper.js

copied config from another guy and it worked

I did the same but it's still not working here for me. Using MacOS Big Sur.


Sigrok / PulseView compatibility seems to be broken

Hello,
I tried to use HydraFW with Sigrok according to the wiki article ( https://github.com/hydrabus/hydrafw/wiki/HydraFW-SUMP-guide ). However after entering SUMP mode PulseView doesn't find the device (screenshot: https://afiskon.ru/s/33/65d517b933_temp.png ), nor 'sigrok-cli --scan` does:
$ sigrok-cli --scan
The following devices were found:
demo - Demo device with 12 channels: D0 D1 D2 D3 D4 D5 D6 D7 A0 A1 A2 A3

I'm using HydraFW compiled from source, commit cb077a71 (Thu May 24 08:21:25 2018 +0200). Here are versions of PulseView and sigrok-cli, they are relatively new as well:
$ pulseview --version
PulseView 0.5.0-git-1931b5f
$ sigrok-cli --version
sigrok-cli 0.8.0-git-a30c837

I didn't investigate the problem deeply, maybe this is a Sigrok's bug. I will report this issue to sigrok-devel@ mailing list as well. From what I know BusPirate compatibility is broken in Sigrok too https://sigrok.org/bugzilla/show_bug.cgi?id=1136
If anyone can reproduce this issue I suggest to edit the wiki page accordingly.

Hi !
Thanks for the report !
Using the last revision from master, I tested it with sigrok 0.7 :
$ sigrok-cli --version
sigrok-cli 0.7.0
$ sigrok-cli --driver=ols:conn=/dev/ttyACM0 --scan
The following devices were found:
ols - HydraBus with 16 channels: 0 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

Maybe there is something different with sigrok 0.8. Could you please try again using the following command and send me the result ?
sigrok-cli --driver=ols:conn=/dev/ttyACM0 --scan -l 5
Also, since https://github.com/hydrabus/hydrafw/commit/ca90ec2a30b7e80f157d30756444a9b279ad2f90, hydrabus can be detected without the need to enter the sump command.

My bad, it actually works. I realized that I should choose "Serial Port" instead of "USB" in PulseView's dialog: https://afiskon.ru/s/92/86885bbfdb_temp.png


add a config option for "branch indicator on new line"

Recently a contribution was made that placed the branch indicator (in the file panel) on a new line. However, in my projects I generally consider vertical real-estate in the file panel to be at a premium--I consider a long branch name occasionally overflowing a much lesser evil than permanently decreasing the number of files I can see in the file panel by 2. I seldom have very long branch names so this isn't a big enough issue for me I guess. Plus, I think it looked better the way it was.
How about adding a config option so that users can decide for themselves whether to place the indicator on a new line? Or possibly just placing it on a new line if there isn't enough room on the same line without overflowing?
Thanks!

cc @joseporto https://github.com/zaggino/brackets-git/pull/1278


[#192] Send a snapshot done key at the beginning of every streaming iteration

This PR modifies the code logic so that at the beginning of every iteration for streaming, we send out a key which has the snapshot_done_key as its key to tell the server that it is the point where snapshot has been completed.
Additionally, this closes #192

Not needed, these changes were required for colocated table snapshot and will go in that PR only.


Report linting errors using Julia's Logging stdlib...

...or at least have the option to. The motivation here is that it is very easy to work with logging records, which makes it easier to create new printing/interaction methods. I'm particularly interested in generating GitHub annotations from JET.jl results. See here for an example of what annotations look like for test failures.

Could you tell me how do you want to use JET (maybe with an example snippet or something), please ?
I'm not sure how JET's current API (i.e. report_xxx(io::IO, ...) or analyze_xxx(...)) is not enough for your use case.

Sure, if you look at
https://github.com/adamslc/JuliaGHA.jl/pull/5/files#file-test-runtests-jl-L16,
you should see a GitHub annotation indicating that the test failed. To do this, I am using a modified @testset that reports test failures as log messages rather than printing to stderr. Those messages can then be consumed, and turned into GitHub annotations.
If JET.jl also produced log messages, then the same thing could be done with the linting results. The goal of this is to surface testing failures into the GitHub UI (instead of hidden in a log file), which I think will improve the accessibility of submitting PRs.

hm, why do you need to use functionalities of Logging module ?
Just using something like
out = IOBuffer() # or whatever IO
report_file(out, yourfile)
s = String(take!(out))
# render `s` or whatever

won't suffice ? (still I think you will want to use analyse_xxx, which allows you to directly interact with errors collected by JET)

You're right that I could simply parse the output from this log in the way you suggest:
https://github.com/adamslc/JuliaGHA.jl/pull/5/checks?check_run_id=2392399247
However, it seems like an unnecessary amount of work to "stringify" the linting errors, only to parse them back into Julia to create the annotations.
You shouldn't feel obligated to work on this, but I do think that some more general output format will eventually be needed for interop with IDEs. Perhaps this sort of output would be more natural to add when that system has been designed.

Okay, so I think you want to interact with the result of analyze_xxx, rather than report_xxx entries.
For example JET.analyze_file returns res::JET.VirtualProcessResult, and you can implement a frontend code to render res.toplevel_error_reports::Vector{JET.ToplevelErrorReport} or res.inference_error_reports::Vector{JET.InferenceErrorReport} in GitHub annotations.
Go ahead for the documentation of JET's report interface, which should explain the API of JET's error reports (but please don't hesitate to ask questions, I just wrote the documentation this morning and it's not really matured).
For the frontend implementation, you may want to reuse JET's frontend code (those defined in print.jl to render error reports in io::IO).

That looks like exactly what I need! Thanks for taking the time to point it out. Sorry that I hadn't understood the difference between report_* and analyze_* earlier.

You're welcome, looking forward to your application :) That should be very useful.

This is now well documented: https://aviatesk.github.io/JET.jl/dev/internals/#Analysis-Results


A error in plugin source

Hey,
I clone your code to my compute,and I find a  place in "actor_plugin/src/ActorPlugin_ros.cc"
that make me puzzled.In 170 row, you user"diffDirection = diff.Normalize()"  to get the direction of "diff",however, Normalize()function will make diff be a unit. I think this may  influence the computation of scocial force. The Normalize() source code is below.
By the way, when I use social force model to Simulate human behavior, I find in complex enviroment, such as many actors, it's diffcult to completely avoid collision ..Do you find similar problem in your experiment?
/// \brief Normalize the vector length
/// \return unit length vector
public: Vector3 Normalize()
{
T d = this->Length();
    if (!equal<T>(d, static_cast<T>(0.0)))
    {
      this->data[0] /= d;
      this->data[1] /= d;
      this->data[2] /= d;
    }

    return *this;
  }

Hi,
For the first question, please check ignition::math::Vector3d::Normalize();
For the second question, of course, the social force model is not a perfect model. If you check this demo video of the social force model, you will clearly see a lot of collisions. That's why we were looking for better navigation strategies among pedestrian environments in this paper.

Hi
Ok,for first question,maybe our ignition::math::Vector3d::Normalize()‘s version are not same, when I try to print the length of the diff after use this function, it's always 1.And there is another function name normalized() that just return a unit.And thank you for your solution,best wishes!


[QST] Citation?

Hi, do you have a preferred way to cite this work? I've checked the README, readthedocss, old github issues, etc. and haven't seen it mentioned. Apologies if I've missed it somewhere.
Thanks in advance

Hi @rijobro, thanks for checking
The best option at the moment is probably this Zenodo citation corresponding to our SciPy 2021 presentation: https://zenodo.org/record/5151998#.YcDYM7tOmGc
Given that much of cucim.skimage is based on code from scikit-image, you may also wish to cite scikit-image itself. Citation info for it is given on the front page of https://scikit-image.org. We hope to submit a newer scikit-image publication in 2022 (the original one is from 2014).

Great, thanks for the info!
Might be worth adding to the README or somewhere else obvious?


Use $HOME instead of ~ in PATH env variable

Hi folks,
I was running some experiments with the InnovationEngine project, which leverage the CloudShell to execute the documentation, and I faced a problem with the the PATH env variable. In particular, I noticed some paths within PATH are using ~ instead of $HOME and some tools don't parse ~ thus they aren't able to execute the binaries located on those paths. An example of the tools impacted is kubectl. Here the issue reported upstream. The solution is to use $HOME instead of ~, which shouldn't have any impact on tools already working with ~. I tried in other online shells and systems I could try, and none of them are using ~ within PATH.
Thanks

Hi @blanquicet, thank you for making a PR. We will look into this and get back to you!

The test cases are passing: https://github.com/Azure/CloudShell/actions/runs/7062144530/job/19225319429.

The test cases are passing: https://github.com/Azure/CloudShell/actions/runs/7062144530/job/19225319429.

Yep, but it is just testing that the PATH contains the correct paths, not that the binaries inside those paths can be executed. I know, they should be, but it would be great if we have a test for it. I'm not sure to have free slots to work on this in the next weeks so, I propose merging it as it is and then work on a more specific test on a secondary PR. WDYT?

I found the issue. In my initial approach I was adding the $HOME string literally in the PATH env variable and it was not being evaluated within the container. Now, I'm adding it with the .bashrc script which is executed every time a new shell is opened within the container so it will be able to resolve $HOME correctly for each user.

Hi @darrentu, friendly ping to have this PR merged 🙂. Thanks!


Operations involving localized objects have multiple instances of the same locale criteria serialized

Need to find a way to have only one referenced everywhere

Trying to share the same locale criteria among operations serialized together triggers a bug in the serialization


feat: support node v12+

I updated the indy-sdk to support node v12+: https://github.com/hyperledger/indy-sdk/pull/2389
This PR loosens the node engine requirement to support 12+ and also adds a test martrix to test for node 12, 14 and 16.
Prettier had some updates to their defaults I think. that's where the other changes come from

Codecov Report

Merging #294 (70a6d3b) into main (ec95d83) will not change coverage.
The diff coverage is 66.66%.

@@           Coverage Diff           @@
##             main     #294   +/-   ##
=======================================
  Coverage   88.06%   88.06%
=======================================
  Files         218      218
  Lines        3922     3922
  Branches      452      452
=======================================
  Hits         3454     3454
  Misses        468      468

Impacted Files
Coverage Δ

src/modules/credentials/models/Credential.ts
75.00% <0.00%> (ø)

...c/modules/credentials/models/IndyCredentialInfo.ts
60.00% <0.00%> (ø)

src/modules/connections/ConnectionsModule.ts
67.60% <100.00%> (ø)

src/modules/proofs/models/ProofRequest.ts
100.00% <100.00%> (ø)

src/modules/proofs/models/RequestedCredentials.ts
100.00% <100.00%> (ø)

src/modules/proofs/services/ProofService.ts
81.85% <100.00%> (ø)

Continue to review full report at Codecov.

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


Configure Monitors Via Code

My team currently uses OneUptime Cloud to monitor various resources and present their status on status pages to our customers. It works great and we're very happy with it.
I'm wondering if it is possible to configure our monitors via code? It would be nice to have our code in source. Is there a way to use OneUptime as Infrastrucutre as code? rather than the UI directly?

This is under development and will be laucnhed by end of Dec.


fix: Fix bug if inner sub set is empty causing NPE [tentative]

(I haven't tried but I think this might be the issue)
Tries to fix https://github.com/flame-engine/flame/issues/3347 (needs to validate)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 97.345%

Totals

Change from base Build<PHONE_NUMBER>6:
0.02%

Covered Lines:
110

Relevant Lines:
113

💛  - Coveralls


'style_preset' is not included

Hello,
Stability AI just added style_preset into their API but it's not included in your client.
Any way it's going to be added soon?
https://api.stability.ai/docs#tag/v1generation/operation/textToImage

looks like it's part of extras, which is listed in the proto as "// for development use" so that might not be reliable going forward


Add --projects option Jest CLI Options docs

Summary
With Jest 20 there was new CLI option introduced --projects. This PR proposes documentation changes to include this new flag. It probably needs some rewording, but I wanted to open this PR to keep track of it.
Test plan
No tests required. Documentation change only.

Thanks!


Issue 75

fixes #75

Closing due to inactivity. Feel free to reopen, but keep code in separate PRs where possible to make things easier to review.


Show the maven coordinates in the Readme file

Hi,
Your extension is clearly deployed to maven central:
<dependency>
  <groupId>engineering.everest.axon</groupId>
  <artifactId>crypto-shredding-extension</artifactId>
  <version>...</version>
</dependency>

I think it would be helpful if those coordinates would be visible in the readme file which is shown on the first page of your Github project. Maybe even use a maven badge.

G'day @nils-christian
Apologies for leaving the issues you raised in limbo for nearly 2 years! This repository was not being actively monitored and notification emails weren't being received.
Thank you for taking the time to review this library and to provide feedback. It's very much appreciated!
I'm going to address some of the smaller issues that you have raised based on whether it helps with our current customer's project. Some, like this one, have been at the bottom of a long list of things we should get around to doing for way too long.
Again, thank you and sorry for taking such a long time to get back to you.


Docs.MS Release Updates for Azure.Storage.Files.Shares

Docs.MS Release Updates for Azure.Storage.Files.Shares

Docs Build status updates of commit 19681d7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

api/overview/azure/storage.files.shares-readme.md
:white_check_mark:Succeeded
View

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.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Go install fails

Hi, I tried to install this using go install github.com/sachaos/viddy@latest, and it's failing with this error :slightly_frowning_face:
go install: github.com/sachaos/viddy@latest (in github.com/sachaos/viddy@v0.3.3):
        The go.mod file for the module providing named packages contains one or
        more replace directives. It must not contain directives that would cause
        it to be interpreted differently than if it were the main module.

@farzadmf I've released 0.3.4 and fix the bug. Thank you for your bug report!

Yup, thank you @sachaos for the quick action, I saw your PR and tried with go install ...@latest, and it's working fine now
And ... thank you for the amazing tool, LOVE the idea and concept, and kudos to you for coming up with the idea. Keep up the good work