Types from Token! as enum?

Disclaimer: This is my first time using syn, so I might have missed something obvious...
I ran into a little issue in my parser that could easily be solved if the various Token's that are provided by syn (the ones that are returned by the Token!  macro) were part of an enum, but sadly they're not. They do however have the Token marker trait, but I can't match on that, which is the crux of my issue.
So, basically what I'm asking is, why aren't the Token's an enum?

Our idiom for parsing benefits from treating tokens (like static_token, colon_token, eq_token, semi_token in the snippet below) as syntax tree nodes defined by a Parse impl. That lets them combine with generic parse functions like <Option<T> as Parse>::parse and Punctuated::<T, P>::parse_separated_nonempty. Parsing Punctuated in particular would be more complicated if the punctuation token type had to be an argument rather than a type parameter.

https://github.com/dtolnay/syn/blob/d9d5b25b9dbf79a893b18051c94392cc14e47863/src/item.rs#L1040-L1055


Crash when placing cables when NEE 2.0.0

Describe the bug
Crash when placing cables
To Reproduce
Place any kind of ME cables
Expected behavior
No crash
Additional context
crash-2022-11-27_13.26.03-client.txt
Environment
In pure pae2 environment,it's fine.

Minecraft Version:
AE2 Version: appliedenergistics2-rv6-stable-7-extended_life-v0.54.15
NEE Version: NotEnoughEnergistics-1.12.2-2.0.0
Forge Version: <IP_ADDRESS>60

https://github.com/PrototypeTrousers/Applied-Energistics-2/issues/191

This is a problem caused by nee's asm, I will release a version with pae2 support later

fix in 2.0.1


! operator and == true / false

Right now there is no language support for
if !condition
and
if condition == false

Verified that ! works & true == false does not.


No support for the ARM toolchain on... ARM?

Trying to install the STM32 boards in Arduino on a Raspberry Pi, but no luck: "Tool xpack-arm-none-eabi-gcc is not available for your operating system."

actually there is something unofficial...https://github.com/koendv/stm32duino-raspberrypi

I know @odo2063 but I would like a maintain able solution.

me too...but sometimes having at least something is better than nothing...

Right, but If I release this I will have to support it and answer to all related issue...
So user can use this following instructions provided by @koendv

Currently I'm waiting xpack support for arm then I will considered to add this support as mentionned in stm32duino/Arduino_Core_STM32#708

Hi Frederic,
Right now I am using stm32duino on a Raspberry Pi 4, 4gb ram, running 32-bit raspbian "buster". The Arduino IDE is stable. The built-in IDE editor is sometimes a bit slow. But in general, things work.
The upcoming version of the xpack gnu arm embedded gcc will support 32 and 64 bit arm linux hosts. To run stm32duino on arm linux, the only missing component is STM32CubeProg. Could you consider compiling STM32CubeProg for arm linux?
Since the introduction of the Raspberry Pi 4, these small arm linux systems are quite capable, and I believe that this market will grow rapidly, with more and more use cases, including for STM32 embedded software development, where the Arduino IDE is a major tool.
In the meantime, I don't mind sharing my solution. If you want to try stm32duino on raspberry, go to https://github.com/koendv/stm32duino-raspberrypi, copy the url of the .json file and paste the url in the Arduino IDE, File->Preferences "Additional Board Manager URLS". There are binaries for 32-bit (armv7l) and 64-bit arm (aarch64) linux.
regards,
koen

I confirm, work on the next release of the xPack GNU Arm Embedded GCC (v9.2.1-1.2), which will add support aarch64 and armhf, is under way.
Target date is end of March. I'll let you know when available.
But, as Koen mentioned, it would probably be useful to start preparing and build the other tools (STM32CubeProg?) for Arm 32/64-bits.

Could you consider compiling STM32CubeProg for arm linux?

Unfortunately, I could not "compile" it myself. I'm an user like you 😉
You should ask this on ST community.

Target date is end of March. I'll let you know when available.

Thanks @ilg-ul

Unfortunately, I could not "compile" it myself. I'm an user like you 😉
You should ask this on ST community.

This will probably take more time than getting the toolchain run on Arm.
If this is an open source project, perhaps it should not rely entirely on STM tools and also provide open source tools, like the ones used in Koen's solution.

This will probably take more time than getting the toolchain run on Arm.

Probably...

If this is an open source project, perhaps it should not rely entirely on STM tools and also provide open source tools, like the ones used in Koen's solution.

The main issue is that OSF does not support all STM32 series. Moreover it is only one upload method they are other provided. It is planned to provide the DFU utils  for all.

Unfortunately, I could not "compile" it myself. I'm an user like you 😉
You should ask this on ST community.

I'm a bit confused. Are you managing this project as a hobby? According to your GitHub account, you seem to be working for STMicroelectronics, so I expected this to be a project officially supported by STMicroelectronics.

The main issue is that OSF does not support all STM32 series. Moreover it is only one
upload method they are other provided. It is planned to provide the DFU utils for all.

Can you elaborate a bit? What STM32 series are not supported by the OSF tools?
I'm not an Arduino user, so correct me if I'm wrong, but, as for upload methods, I saw in Koen's solution several upload methods, using various bootloaders, in rom or in flash. What else would be needed?
And what do you mean by 'provide the DFU utils for all'?
I'm asking this because I'm trying to understand if I can further help this project.
I have a very elaborated build environment (called the xPack Build Box), which allows me to build binaries for Linux Intel 32/64, Linux Arm 32/64, Windows 32/64, macOS, and maybe we can use this to provide a consistent set of tools for the Duino users, in addition to the STM specific tools.

I'm a bit confused. Are you managing this project as a hobby? According to your GitHub account, you seem to be working for STMicroelectronics, so I expected this to be a project officially supported by STMicroelectronics.

No it is not a hobby even if several time I'm working at home even the week end... And yes it is supported by STMicroelectronics but it is first a community project. And a I said I'm a user of the STM32CubeProgrammer. Moreover, main ST Software ecosystem mainly support 3 OS as far as I know, that's why I said the bet way is to request on ST community to have STM32cubeProgrammer support for ARM arch.

Can you elaborate a bit? What STM32 series are not supported by the OSF tools?

STM32H7, STM32WB, STM32L5, STM32L4+,... some of the OSF projects start to introduce some of them but not stable yet or does not support all value line.

I'm not an Arduino user, so correct me if I'm wrong, but, as for upload methods, I saw in Koen's solution several upload methods, using various bootloaders, in rom or in flash. What else would be needed?

yes, that's why I said STM32CubeProgrammer is not the only way to upload so for those one this will be OK. I never said I will not add the ARM tooclchain just answered the STM32CubeProgrammer would not be possible while there is not ported on this arch.

And what do you mean by 'provide the DFU utils for all'?

Currently, DFU utils is used only for some target and I would generalize them for all.

I'm asking this because I'm trying to understand if I can further help this project.

Any help are welcome, as said this is a community project. 😉

I have a very elaborated build environment (called the xPack Build Box), which allows me to build binaries for Linux Intel 32/64, Linux Arm 32/64, Windows 32/64, macOS, and maybe we can use this to provide a consistent set of tools for the Duino users, in addition to the STM specific tools.

Thanks, that would be nice. I will get back to you if I need to made some build when I have a time slot to work on this, it will be very appreciated.

...  yes it is supported by STMicroelectronics ... the best way is to request on ST
community to have STM32cubeProgrammer support for ARM arch.

Don't you think that, from the internal STM procedures point of view, it is a bit weird (to not say terribly wrong) if, in order to communicate to your colleagues working on the STM32cubeProgrammer you need us to go to the ST community, instead of contacting them directly and getting an estimate when their tool will also be available for Arm 32/64-bit? If it is only a matter of a reasonable time, we'll wait, otherwise we need to consider alternate solutions.
And what exactly do you mean by 'ST Community'? I remember some time ago I reported some issues with compiling the STM packages with more recent GCCs, on a site that looked like a forum, but all that happened was that after several months the moderator mentioned that the issue will be passed to someone to investigate, and since then nothing else happened.

Don't you think that, from the internal STM procedures point of view, it is a bit weird (to not say terribly wrong) if, in order to communicate to your colleagues working on the STM32cubeProgrammer you need us to go to the ST community, instead of contacting them directly and getting an estimate when their tool will also be available for Arm 32/64-bit? If it is only a matter of a reasonable time, we'll wait, otherwise we need to consider alternate solutions.

Well, I could not forward all request internally else I will spent my time to do this, the way to go is to use https://community.st.com/
If several people show their interest to this more this will be considered, I guess.
Anyway, I already ask but I've no answer yet...

And what exactly do you mean by 'ST Community'? I remember some time ago I reported some issues with compiling the STM packages with more recent GCCs, on a site that looked like a forum, but all that happened was that after several months the moderator mentioned that the issue will be passed to someone to investigate, and since then nothing else happened.

Well this one: https://community.st.com/
I'm sorry if you didn't get quick answer or feedback 😕 Anyway this is the best way to get official support.

Here the answer:

There is no plan for CubeProgrammer to be supported on ARM arch host at least
in the short-mid term.

So, my advice is to request it on ST community as I suggest...

There is no plan for CubeProgrammer to be supported on ARM arch host at least
in the short-mid term.

In other words, that tool will take like forever to be available, thus the whole effort to provide the Arm toolchain was practically useless since without that tool you cannot release an Arm version of stm32duino.
Is this correct? Since waiting for a favourable outcome from a message in the ST community is like watching another episode of Mission Impossible. :-(

No this is not correct.
As I explained there is other upload methods than STM32CubeProgrammer.
Like @koendv  do using the BMP or dfu utils. And it exists other.
So this is not an issue that the STM32CubeProgrammer would not be available....

As I explained there is other upload methods than STM32CubeProgrammer.

Yes, there are other methods, but will this project use them? Since for now it seems all those methods were discontinued and replaced with the STM32CubeProgrammer.
As far as I know, the IDE works on any possible platform. The toolchain runs on all major platforms, Intel Linux 32/64, Arm Linux 32/64, Windows 32/64, Intel macOS 64.
Normally you should also release stm32duino on all 7 platforms, accompanied by existing open source tools, also built for all 7 platforms.
If STM provides STM32CubeProgrammer on a limited set of platforms, it should be added where available, not the other way around, to limit the number of supported platforms to follow those of STM32CubeProgrammer.
I'm convinced the community will appreciate using stm32duino on all 7 platforms.
At least from my part, the toolchain will support all of them.

It seems like we can't understand each other.
As you said you are not an Arduino user and so do not know how it works on the backstage.

The toolchain runs on all major platforms, Intel Linux 32/64, Arm Linux 32/64, Windows 32/64, Intel macOS 64.

For example, OK it runs on win32/64 but there is no way to differentiate them so the only way to support both is to release the 32 bits versions for windows.
About the limit of supported platform, you don't understand my point which was to avoid having to support all related issue because one or other tools does not support a dedicated series or value line. Anyway, as I said they are other way  which are already provided to support that and I will extend DFU utils usage and probably add open OCD way.
I do my best to have a coherent set of tools which can be maintainable and I always take in account all contributions (PR, ...) or suggestions to improve and help the community.
I really appreciated your contributions for the gcc toolchains and this enhanceand ease a lot this part.
The fact the STM32CubeProgrammer is not supported for ARM arch is not on my hands. So, please, be kind I'm not responsible of this and I have not power to change that. Few years ago, ST provided only windows tools so this is already a great step they support those one also. 🙄

it runs on win32/64 but there is no way to differentiate them so the only way to support both is to release the 32 bits versions for windows.

You mean the IDE does not know when running on win64? What a pity...
Anyway, I'll provide you the toolchain which already runs on all platforms (I did not release it since I'm still woking on gdb-py and gdb-py3, but this does not affect your IDE anyway). I hope you'll do the best out of it, including using the new Arm binaries, since I worked quite a lot to build them. I'm not happy that you are not using the win64 binaries, since this messes the analytics, but this is a different story.

You mean the IDE does not know when running on win64? What a pity...

I guess it known but it does not allow us to specify which tools to install/use....

I'm not happy that you are not using the win64 binaries, since this messes the analytics, but this is a different story.

I understand, but related to to IDE itself... maybe one day it will be supported or maybe it can be considered 32bits should no be provided anymore and switch to 64 bits... But I guess some user still having a 32 btis...

I guess some user still having a 32 bits...

Definitely. And as long as Microsoft will continue to support Windows on 32-bit machines, my binary tools, including the arm-none-eabi-gcc, in addition to the expected 64-bit binaries, will have their 32-bit siblings.
As for Arm, at this moment, most Raspberry Pi machines run 32-bit Linux-es, and although I do not like 32-bit OS-es very much, I think that both Linux 32 and Linux 64 deserve attention.
Can your IDE differentiate between Arm 32 and Arm 64, to install the proper binaries?

Can your IDE differentiate between Arm 32 and Arm 64, to install the proper binaries?

This is not my IDE 😉
But yes it is able to differentiate it like for Linux and Mac. Only windows have this restrictions.

Anyway we can hope the future release of the new Arduino Pro IDE will handle that.

Ok, thank you for the details.
The chances for me using Arduino in the near future are low, but I'll continue to support you with the toolchain.
Good luck with your projects.

Hi Frederick.
I got an answer from ST Community:

STM32CubeProgrammer is not supported on ARM architecture host .For linux only x86 architecture  is supported  .
Thanks for your help. At least we tried.

Welcome and thanks also for asking.
At least you get a quick answer 🙄

Hi Frederick.
I got an answer from ST Community:

STM32CubeProgrammer is not supported on ARM architecture host .For linux only x86 architecture  is supported  .

Thanks for your help. At least we tried.

for what do you need STM32CubeProgrammer anyway if you use Arduino?(just asking I have absolutly no clue, "onetime" Firmware update can be done on x86)

It is not mandatory... It is supported that's all and allows to flash over: SWD, DFU and Serial.
It can be easily extended to SPI, CAN,...


Obs phase1

Massive PR : Adds overlay, media, sfx, switches most var to let, and a lot of other stuff which will be documented in the path notes for 1.6.0


inbound_layers.append(res[inbound_layer_id]) KeyError: '140072123108824'

I had a problem when running reduce_keras_model, I get an error at line  55 in init.py at inbound_layers.append(res[inbound_layer_id])
I guess the problem happens when the model has a large number of filters
def get_simple_2d_model():
    inp = keras.layers.Input((60,60,1))
    x = Conv2D(32, 3, padding='same', kernel_initializer='random_uniform')(inp)
    x = BatchNormalization()(x)
    x = Conv2D(128, 3, padding='same', kernel_initializer='random_uniform')(inp)
    x = MaxPool2D((2,2))(x)
    x = BatchNormalization()(x)
    x = Conv2D(256, 3, padding='same', kernel_initializer='random_uniform')(inp)
    x = BatchNormalization()(x)
    x = MaxPool2D((2,2))(x)
    x = keras.layers.Activation('relu')(x)
    x = Conv2D(512, 3, padding='same', kernel_initializer='random_uniform')(x)
    x = BatchNormalization()(x)
    x = MaxPool2D((2,2))(x)
    x = Flatten()(x)
    x = Dense (512, activation = tf.nn.relu,kernel_regularizer=l2(0.01))(x)
    out = Dense(1, activation='sigmoid')(x)
    model= keras.Model(inputs=inp, outputs=out)
    return model

Can you please provide tensorflow and keras version?

I changed your code like that. And it works totally dine for me:
def get_simple_2d_model():
    import tensorflow as tf
    import tensorflow.keras as keras
    from tensorflow.keras.layers import MaxPool2D, Flatten, ReLU

    inp = keras.layers.Input((60,60,1))
    x = Conv2D(32, 3, padding='same', kernel_initializer='random_uniform')(inp)
    x = BatchNormalization()(x)
    x = Conv2D(128, 3, padding='same', kernel_initializer='random_uniform')(inp)
    x = MaxPool2D((2,2))(x)
    x = BatchNormalization()(x)
    x = Conv2D(256, 3, padding='same', kernel_initializer='random_uniform')(inp)
    x = BatchNormalization()(x)
    x = MaxPool2D((2,2))(x)
    x = keras.layers.Activation('relu')(x)
    x = Conv2D(512, 3, padding='same', kernel_initializer='random_uniform')(x)
    x = BatchNormalization()(x)
    x = MaxPool2D((2,2))(x)
    x = Flatten()(x)
    x = Dense(512)(x)
    out = Dense(1, activation='sigmoid')(x)
    model= keras.Model(inputs=inp, outputs=out)
    return model

strangely, it's working.  I am using TensorFlow 2.3.0. I get this error if I am loading a model with these params mentioned above. if I called function it works but if I loaded a pre-trained model it shows the error

May be it's because this line?
x = Dense (512, activation = tf.nn.relu,kernel_regularizer=l2(0.01))(x)
There are direct TF calls.

if I called this function, with no direct TF calls., I get the error
def get_simple_2d_model():
    import tensorflow as tf
    from tensorflow import keras
    from keras.layers import MaxPool2D, Flatten, ReLU,Conv2D,BatchNormalization,Dense,Input
    from keras.models import Sequential

    model=Sequential()

    model.add(Conv2D(32,kernel_2, input_shape=(60,60,1), strides = 1, padding='valid'))
    model.add(BatchNormalization())
    model.add(Conv2D(32,kernel_2))
    model.add(MaxPool2D(pooling))
    model.add(Dropout(dropout))
    model.add(BatchNormalization())

    model.add(Dropout(dropout))
    model.add(BatchNormalization())
    model.add(Flatten())
    model.add(Dense(512))
    model.add(Dropout(0.4))
    model.add(Dense(num_classes, activation = 'softmax'))

    return model


Make convenience methods like get_anndata() have expressive arguments

Should the API use types? Particularly to use in method arguments?
Advantages include:

Help users catch errors and typos.
Help communicate which parameters are valid.
IDE can provide auto-completion.

This design would arguably be more "Pythonic", in particular:

Explicit is better than implicit.
Flat is better than nested.

Current
adata = cellxgene_census.get_anndata(
    census = census,
    organism = "Homo sapiens",
    var_value_filter = "feature_id in ['ENSG00000161798', 'ENSG00000188229']",
    obs_value_filter = "sex == 'female' and cell_type in ['microglial cell', 'neuron']",
    column_names = {"obs": ["assay", "cell_type", "tissue", "tissue_general", "suspension_type", "disease"]},
)

Proposed
class Sex(Enum):
    FEMALE = "PATO_0000383"
    MALE = "PATO_0000384"
    UNKNOWN = "unknown"

adata = cellxgene_census.get_anndata(
    census = census,
    organism = "Homo sapiens",
    feature_ids: ['ENSG00000161798', 'ENSG00000188229'],
    sex = Sex.FEMALE,
    cell_type = ['microglial cell', 'neuron'],
    column_names = {"obs": ["assay", "cell_type", "tissue", "tissue_general", "suspension_type", "disease"]},
)

Note: obs_value_filter arguments would be optional, so users would only need to include their obs values of interest.
Questions

Are there any other possible values for var_value_filter other than feature_id? If not could it be simplified and more explicitly named as feature_ids and accept an array of strings rather than a string?

For a recent project (docs), I added some logic for specifying filters so you can do things like:
ensdb.genes(
    filter=gf.filters.GeneBioTypeFilter("lncRNA")
    & gf.filters.GeneRangesFilter("1:10000-20000")
)

We've defined &, |, ~ (and, or, not) to do more complex combinations.
How much do you want to support custom data collections? E.g. if I have my own local census, which may have extra fields. Is that meant to be supported here? If that's supported it would be nice to be flexible about what columns are named, which could be difficult if columns to filter on are specified by keyword arguments.

Emanuele suggested @aaronwolen may have thoughts on whether a change in somacore is on the table here.
Personally, I'm not sure this change is worth creating inconsistency between the APIs. But do like this change overall.


fix: load both deployer and authority priv keys into docker-compose's geth

Closes #1098
Overview
{:error, %{"code" => -32000, "message" => "unknown account"} is returned from geth when the childchain submits a block. This is due to how the docker-compose setup relies on the authority account to be unlocked but it was not configured in geth.
Changes

Configure the authority keystore into geth
Add --unlock "0,1" to unlock the two accounts
Provide a blank password file into geth since it is needed for the --unlock argument
Set --miner.etherbase to the deployer address so we do not depend on the accounts ordering which is prone to change

Testing
Tested with:
# In elixir-omg:
$ docker-compose up

# In omg-js:
$ npm run childchain-deposit
...
Alice's new rootchain balance:<PHONE_NUMBER>37316195423570985008687907853269984665640564039456737018433129639927
Alice's new childchain balance:<PHONE_NUMBER>00000000

$ npm run childchain-transaction
...
Created a childchain transaction of 0.05 ETH from Alice to Bob.
Submitted transaction. Transaction receipt: {
  "blknum": 1000,
  "txhash": "0xb3387025978fa8dbb88e36b121fe09b3d7f40aacb0f1aaed8612fc662a12d6b3",
  "txindex": 0
}
...

Coverage remained the same at ?% when pulling fa4b0482f667d31031d2762ed8d870b2fe7efe06 on 1098-unknown-account into 9159cbd19ff55aa6e522ebf8b8462cc8c2caa919 on master.


Different public address from Magic

I have two different public addresses for the following methods
import { EthNetworkConfiguration, Magic } from 'magic-sdk';

const magic = new Magic(process.env.NEXT_PUBLIC_MAGIC_API_KEY, {
  network: process.env.NEXT_PUBLIC_CHAIN_KEY as EthNetworkConfiguration,
});
await magic.auth.loginWithMagicLink({ email<EMAIL_ADDRESS>});

const provider = new ethers.providers.Web3Provider(magic.rpcProvider);
const magicSigner = provider.getSigner();
const address = await magicSigner.getAddress();
console.log(address)

and
import { useMagic, useAddress } from '@thirdweb-dev/react/evm/connectors/magic';
const connectWithMagic = useMagic();
const address = useAddress()

await connectWithMagic({ email<EMAIL_ADDRESS>})
console.log(address)

Here is the Sandbox to reproduce the issue: https://github.com/rekolt/magic-link
If I use the same email, I should have one single address, correct? Why do I have two different?

interesting. This might be because of the major version update magic did recentlyl. The react SDK uses an older version. We're working on an updated version in our new separate @thirdweb-dev/wallets package. Should help with this issue


Fix versioning in unit tests

The unit tests don't run out of the box because of some inconsistencies between what's hard-coded in .travis.yml. Frustrating trying to figure out which versions actually work together, but I figured it out.

I think I'll await on cleaning up this pull until https://github.com/chfw/pyexcel-io/pull/5 gets merged in. It would be great if you could push an update to pypi on that one so we can say pyexcel-io>=0.0.9 (since the unit tests on 0.0.8 are broken)
Thanks, @chfw!


general-info-relation-joined ERROR cannot open 5666/tcp (unit "nrpe/65"): conflicts with existing 5666/tcp (unit "nrpe/68")

018-07-02 04:06:15 DEBUG general-info-relation-joined ERROR cannot open 5666/tcp (unit "nrpe/65"): conflicts with existing 5666/tcp (unit "nrpe/68")
2018-07-02 04:06:15 DEBUG general-info-relation-joined Traceback (most recent call last):
2018-07-02 04:06:15 DEBUG general-info-relation-joined   File "/var/lib/juju/agents/unit-nrpe-65/charm/hooks/general-info-relation-joined", line 3, in
2018-07-02 04:06:15 DEBUG general-info-relation-joined     services.manage()
2018-07-02 04:06:15 DEBUG general-info-relation-joined   File "/var/lib/juju/agents/unit-nrpe-65/charm/hooks/services.py", line 67, in manage
2018-07-02 04:06:15 DEBUG general-info-relation-joined     manager.manage()
2018-07-02 04:06:15 DEBUG general-info-relation-joined   File "/var/lib/juju/agents/unit-nrpe-65/charm/hooks/charmhelpers/core/services/base.py", line 135, in manage
2018-07-02 04:06:15 DEBUG general-info-relation-joined     self.reconfigure_services()
2018-07-02 04:06:15 DEBUG general-info-relation-joined   File "/var/lib/juju/agents/unit-nrpe-65/charm/hooks/charmhelpers/core/services/base.py", line 192, in reconfigure_services
2018-07-02 04:06:15 DEBUG general-info-relation-joined     manage_ports])
2018-07-02 04:06:15 DEBUG general-info-relation-joined   File "/var/lib/juju/agents/unit-nrpe-65/charm/hooks/charmhelpers/core/services/base.py", line 234, in fire_event
2018-07-02 04:06:15 DEBUG general-info-relation-joined     callback(self, service_name, event_name)
2018-07-02 04:06:15 DEBUG general-info-relation-joined   File "/var/lib/juju/agents/unit-nrpe-65/charm/hooks/charmhelpers/core/services/base.py", line 326, in call
2018-07-02 04:06:15 DEBUG general-info-relation-joined     hookenv.open_port(port, protocol)
2018-07-02 04:06:15 DEBUG general-info-relation-joined   File "/var/lib/juju/agents/unit-nrpe-65/charm/hooks/charmhelpers/core/hookenv.py", line 670, in open_port
2018-07-02 04:06:15 DEBUG general-info-relation-joined     _port_op('open-port', port, protocol)
2018-07-02 04:06:15 DEBUG general-info-relation-joined   File "/var/lib/juju/agents/unit-nrpe-65/charm/hooks/charmhelpers/core/hookenv.py", line 660, in _port_op
2018-07-02 04:06:15 DEBUG general-info-relation-joined     subprocess.check_call(_args)
2018-07-02 04:06:15 DEBUG general-info-relation-joined   File "/usr/lib/python3.5/subprocess.py", line 581, in check_call
2018-07-02 04:06:15 DEBUG general-info-relation-joined     raise CalledProcessError(retcode, cmd)
2018-07-02 04:06:15 DEBUG general-info-relation-joined subprocess.CalledProcessError: Command '['open-port', '5666/TCP']' returned non-zero exit status 1
2018-07-02 04:06:15 ERROR juju.worker.uniter.operation runhook.go:114 hook "general-info-relation-joined" failed: exit status 1
2018-07-02 04:06:15 DEBUG juju.worker.uniter.operation executor.go:75 lock released
2018-07-02 04:06:15 INFO juju.worker.uniter resolver.go:102 awaiting error resolution for "relation-joined" hook
2018-07-02 04:06:15 DEBUG juju.worker.uniter agent.go:17 [AGENT-STATUS] error: hook failed: "general-info-relation-joined"

(by cgregan)

(by jog)
Marked this bug a duplicate of a similar issue the solutions QA team ran into caused by a bundle update. The bundle change installed two units of nrpe on the same machine. The solution was to update the bundle to avoid the second unit deploy.


Use standard-react instead of standard if presented.

close it with commit https://github.com/chenxsan/vscode-standardjs/commit/6d0a941f8916d9d553373b6c49c7e18a08aacbeb.


[test/plugin] immigrate postgresql scenario

Please answer these questions before submitting pull request

Why submit this pull request?

[ ] Bug fix

[x] New feature provided

[ ] Improve performance

Related issues

Bug fix

Bug description.

How to fix?

New feature or improvement

Describe the details and related test reports.

Add to workload 3 after #3645 merged(soon) please. And let's review this, @dmsolr

Merged, please move to jenkins workload 3

You have no the startup script. I have a suggestion, run it passed locally. And then configure the Jenkinsfile(Jenkinsfile-Agent-Test-3).

/run agent-plugin-test-3

I have filed the JIRA ticket to Apache INFRA, https://issues.apache.org/jira/browse/INFRA-19301
Waiting for the response.

/run agent-plugin-test-3

@wu-sheng ok

Move the job to move the task to workload1 group2. Let's continue this PR. We could changr the grouo after the workload3 fixed by infra.

Done

@dmsolr Is this good?

hi, It's still closed twice, what did I miss?

hi, It's still closed twice, what did I miss?

We don't have answer for that, still.

@wu-sheng @arugal the reason for closed twice is the code reason. first close is closeConnection() in sqlExecute.dropTable(ConstSql.DROP_TABLE_SQL) method. second close is finally cause code sqlExecute.closeConnection()

@wu-sheng @arugal the reason for closed twice is the code reason. first close is closeConnection() in sqlExecute.dropTable(ConstSql.DROP_TABLE_SQL) method. second close is finally cause code sqlExecute.closeConnection()

You need to fix it in code？

it is not a bug. Maybe ApmTest design for it.

If test codes are written in that way, I am good for don't change anything.

@dmsolr You should be able to approve as a valid reviewer already.

/run agent-plugin-test-3

@aderm The workload 3 is back online. Please move the job to workload 3, group 2.

Update the total number of tests.

/run ci

/run ci

Hi, you should merge first.

LGTM.


The predicted value prediect_y is the same for all

Hello:
Recently, I have been reading your article "EfficientFi: Toward Large-Scale Lightweight WiFi Sensing via CSI Compressio" and trying to implement the relevant code, but I encountered a problem in the process: The trained predicted value prediect_y is all the same value, which is inconsistent with the corresponding labels, resulting in the training error hovering around 1.78. After step by step inspection, it is found that the value of Loss2 is always around 1.78. I would like to ask if you have similar problems. Is there a solution? Thank you very much!

This means that your model has not converged. You need to ensure that the classification loss is minimized, and the MSE loss decreases progressively.


Import/Export settings pop-up box in Firefox 48 doesn't appear correctly

Defective ToolKit Feature Name:
Expected Behavior
Import/Export settings pop-up box should appear.
Actual Behavior
Import/Export settings pop-up box doesn't appear.
Pressing CTRL+V after pressing the import/export button shows the pop-up box but still it doesn't display correctly.
Steps to Reproduce the Behavior

Use firefox browser version 48 (ubuntu)
Go to add on settings
Click import/export button

Export of Toolkit Settings (Bottom Left of Settings Page)
(copied from chrome)
[{"key":"accountTransactionSearch","value":true},{"key":"accountsDisplayDensity","value":"0"},{"key":"accountsEmphasizedOutflows","value":true},{"key":"accountsSelectedTotal","value":true},{"key":"accountsStripedRows","value":true},{"key":"activityTransactionLink","value":true},{"key":"budgetBalanceToZero","value":true},{"key":"budgetProgressBars","value":"0"},{"key":"budgetQuickSwitch","value":true},{"key":"budgetRowsHeight","value":"0"},{"key":"categoryActivityPopupWidth","value":"0"},{"key":"changeEnterBehavior","value":false},{"key":"checkCreditBalances","value":true},{"key":"checkNumbers","value":false},{"key":"collapseExpandBudgetGroups","value":true},{"key":"collapseSideMenu","value":true},{"key":"colourBlindMode","value":false},{"key":"currentMonthIndicator","value":true},{"key":"daysOfBuffering","value":true},{"key":"daysOfBufferingHistoryLookup","value":"0"},{"key":"editButtonPosition","value":"0"},{"key":"enableRetroCalculator","value":true},{"key":"enlargeCategoriesDropdown","value":true},{"key":"enterInRegisterNow","value":true},{"key":"goalIndicator","value":false},{"key":"goalIndicatorWarningColor","value":true},{"key":"googleFontsSelector","value":"0"},{"key":"hideAOM","value":false},{"key":"highlightNegativesNegative","value":true},{"key":"importNotification","value":false},{"key":"incomeFromLastMonth","value":"1"},{"key":"l10n","value":"0"},{"key":"largerClickableIcons","value":false},{"key":"monthlyNotesPopupWidth","value":"0"},{"key":"options.dark-mode","value":true},{"key":"pacing","value":"0"},{"key":"printingImprovements","value":true},{"key":"privacyMode","value":"2"},{"key":"reconciledTextColor","value":"1"},{"key":"removePositiveHighlight","value":false},{"key":"removeZeroCategories","value":true},{"key":"reports","value":true},{"key":"resizeInspector","value":true},{"key":"rightClickToEdit","value":true},{"key":"runningBalance","value":true},{"key":"seamlessBudgetHeader","value":true},{"key":"showIntercom","value":true},{"key":"splitKeyboardShortcut","value":true},{"key":"splitTransactionAutoAdjust","value":true},{"key":"squareNegativeMode","value":false},{"key":"swapClearedFlagged","value":false},{"key":"toBeBudgetedWarning","value":true},{"key":"toggleSplits","value":true},{"key":"toggleTransactionFilters","value":"1"},{"key":"warnOnQuickBudget","value":true}]

System Settings
ToolKit Version: 0.7.1
Operating System (Mac/Windows/Etc): ubuntu 16.04
Browser (Chrome/Firefox/Safari): firefox
Browser Version: 48
Other Extensions: Firefox hello

@scias23 I'm on firefox 48 and it seems to be working? I'm using an Mac which may be the reason I'd have to make a ubuntu vm to test on ubuntu.
Are you still experiencing this issue?

Hi @joshmadewell , I think this is an isolated issue for ubuntu Firefox build. Not using ubuntu anymore so I can't comment if this may be still happening.


"Run Without Debugging" command has different behavior from "Run Python File" button

The Run>Run Without Debugging command (pulldown menu) has different behavior than the Run Python File (button at top right of UI).
The Run Without Debugging command appears to process launch.json, and picks up command-line arguments specified the .json.
The Run Python File button appears to ignore launch.json (does not pass command-line arguments to Python).
Both should process launch.json.
(If the different behavior is intentional, this should be made explicit in docs, with a reason.)

The "Run Python File" action in the editor header is the responsibility of the debug extension contributing the action.
The debug extension is free to define the semantics of these actions. Typically these actions operate on the editor contents independently from the debug configurations in the launch.json. This makes them more robust and deterministic because they do not rely on a currently selected launch config.
But a debug extension could easily implement these actions based on launch configurations defined in the launch.json.
Bottom line: the observed behavior is correct. Any documentation needs must be fulfilled by the extension contributing the actions because VS Code does not know the action semantics.

/extPython

It looks like this is caused by the Python extension. Please file the issue to the Python extension repository. Make sure to check their issue reporting template and provide them relevant information such as the extension version you're using. See also our issue reporting guidelines for more information.
Happy Coding!


{among us}--{Sahil Dasari}

I started coding when I was about 10 in my 5th grade. It interested me when I saw my sister code something. I saw so many lines of code and then, at an instant, I saw a calculator, or a flower drawing etc. She is my inspiration behind starting to code.
My drawing is an Among us impostor
I could add a marker instead of a pen and color in the drawing. (
Red for body and bag
Light blue for glasses
)
My impostor took about 4- 5 hours because I was a little new to the blot interface because it is a little different from python turtle. I also ran into many bugs, so taking inspiration from my sister, I explained everything to a rubber duck and quickly found the issues

Seems cool!

sussy
This is truly amazing.

I forgot to add 3 screenshots so here is my 3rd one.
Big Head and Big Right Leg, changed 3 parameters

It's a simple drawing but you captured the essence well. Nice work, you've earned your Blot.
You can order it here if you're a teen.


Form isn't rendered properly in Snapshot

Describe the bug
Form isn't being rendered properly in Snapshots.
To Reproduce
Used this code
// And/or enter code that reproduces the behavior here.
class SampleSnapshotTest: XCTestCase {

    func testDefault() {
        let vc = UIHostingController(rootView: SampleForm())
        vc.view.frame = UIScreen.main.bounds

        assertSnapshot(matching: vc, as: .image)
    }
}

struct SampleForm: View {

    var body: some View {
        Form {
            Section {
                Text("Item1")
                Text("Item1")
                Text("Item1")
                Text("Item1")
                Text("Item1")
            } header: {
                Text("Header")
            }
        }
    }
}

Expected behavior

Screenshots

Environment

swift-snapshot-testing version 1.10.0
Xcode Version 14.2 (14C18)
Swift 5.7.2
OS: 16.2

Duplicate of https://github.com/pointfreeco/swift-snapshot-testing/issues/667


Change visibility of Chip to public

Available for standalone usage. Fixes issue #8

ChipField doesn't use the server-side Chip as a component, but only as a DTO. Anyways, I think it doesn't harm to expose the Chip component, since it is already mapped to a standalong paper-chip.js


Documentation is misleading and incomplete

"Show all settings" checkbox is missing from Visual Studio 2022 Community version 17.3.4 with Python workflow. This article is not useful!!!

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

ID: 67031305-2f97-3aa4-ced1-b66092dd4d41
Version Independent ID: 37603282-7242-21de-fb56-9f1bbae9dc43
Content: Reformat your python code in Visual Studio - Visual Studio (Windows)
Content Source: docs/python/formatting-python-code.md
Product: visual-studio-windows
Technology: vs-python
GitHub Login: @rjmolyneaux
Microsoft Alias: rmolyneaux

@astroman133 -- Rick, thank you for your feedback.  Please consider these resources:

Visual Studio on Microsoft Q&A
Visual Studio MSDN Forum
Stack Overflow
Reddit

@rjmolyneaux -- Ryan, please look into this issue.

#reassign: @cwebster-99


[5.0] Incorrect error check in L2CAP channel open

When opening L2CAP channel, error callback is always executed. Problem is most likely in the error check that should be error == nil instead of error != nil.
Code snippet from Peripheral.swift:
let observable = delegateWrapper.rx_didOpenL2CAPChannel
    .take(1)
    .flatMap { [weak self] (channel, error) -> Observable<CBL2CAPChannel> in
        guard let strongSelf = self else { throw BluetoothError.destroyed }
        if let channel = channel, error != nil {
            return .just(channel)
        } else {
            throw BluetoothError.openingL2CAPChannelFailed(strongSelf, error)
        }
}

Hi @tomipu, you’re entirely right. Thanks so much!
@scibas could you send a patch to this?

It’s fixed by #184.
Thanks again for feedback!

Wow, that was fast!
Thanks guys, works like charm now.


Add ability to perform prediction on CPU

It is now possible to run prediction on CPU. Just set
instance_group [{ kind: KIND_CPU }]

in the config.pbtxt.
Closes #41
Limitation: It is currently not possible to run the container on a machine without a GPU. You can test this behavior by running the Docker container without the --gpus=all argument. RAFT will error out with
terminate called after throwing an instance of 'raft::cuda_error'
  what():  CUDA error encountered at:
file=/root/miniconda3/envs/triton_dev/include/cuml/raft/handle.hpp line=63:
call='cudaGetDevice(&cur_dev)', Reason=cudaErrorInsufficientDriver:
CUDA driver version is insufficient for CUDA runtime version

Update. I managed to get the RAFT handle to load lazily, so that you can run the container without --gpus=all.

I addressed all comments. I will add an E2E test tomorrow.

One other thought: To catch the whole range of errors that we ought to be concerned about for both GPU and CPU, it probably makes sense to run CPU-based tests on all of the same models we are running for the GPU tests. We don't have to completely generate new models; we can just re-use the ones that we've already created, which should save us a fair amount of time in CI. It also may make it easier to pinpoint errors where we're seeing different behaviors on GPU vs CPU deployments.

Tests now pass with concurrency=1. However, setting concurrency=8 results in random failures, such as:
AssertionError:
Arrays are not almost equal to 3 decimals

Mismatched elements: 1 / 128 (0.781%)
Max absolute difference: 2.
Max relative difference: 1.
 x: array([2., 2., 2., 2., 2., 2., 2., 2., 2., 2., 2., 2., 2., 2., 2., 2., 2.,
       2., 2., 2., 2., 2., 2., 2., 2., 2., 2., 2., 2., 2., 2., 2., 2., 2.,
       2., 2., 2., 2., 2., 2., 2., 2., 2., 2., 2., 2., 2., 2., 2., 2., 2.,...
 y: array([2., 2., 2., 2., 2., 2., 2., 2., 2., 2., 2., 2., 2., 2., 2., 2., 2.,
       2., 2., 2., 2., 2., 2., 2., 2., 2., 2., 2., 2., 2., 2., 2., 2., 2.,
       2., 2., 2., 2., 2., 2., 2., 2., 2., 2., 2., 2., 2., 2., 2., 2., 2.,...

tritonclient.utils.InferenceServerException: [StatusCode.INTERNAL] shared memory size specified with the request for output 'output__0' (4 bytes) should be at least 512 bytes to hold the results

Closing in favor of #82


Changed the priority of the CacheableResponseVaryListener

Changed the priority of the CacheableResponseVaryListener so that it runs after the cache headers have been added by the FrameworkExtraBundle
After thinking more about what I wrote in #171, I think it makes sense for this listener to have a lower priority by default.

Thanks yes that seems reasonable.


Issue executing ssm-diff init

Expected Output:

Something like 'ssm repo initialized'

Actual Output:

'int' object has no attribute 'splitlines'

Python Version: Python 2.7.15rc1
OS: Ubuntu Linux 18.04 upgraded from 16.04
If you need any further information, please let me know.
I'm not into python development, so i don't know what and how to provide.

Using a virtual env with python 3.6 works, so feel free to either close it or to add a mentioning in the readme for others that might accidentally try to use it the same way i did.

Hi @applike-ss,
Thanks for the feedback, and sorry for not stating out explicitly that Python 3 is required.
Will update readme accordingly

thanks! maybe even some more insight what requirements have to be met are useful as well. I changed the interpreter manually by changing the symlink of /usr/bin/python to the 3 version but that didn't help.
Would you add a suggestion to use a venv for other users to the readme? maybe that will help other people as well. (Or they know that this is in general a good idea to separate your working envs)
Nice tool!


Force installation of dependencies in entry points

Entry points affected:

build.py
make.py
test.py
project.py

They will install every dependency within requirements.txt that cannot be found with pkg_resources.working_set.require

@screamerbg please review

LGTM

One question @theotherjimmy : what's the time profile for this change? How much does it take per invocation ?
Plus, I am not certain this is what is expected from a python module? I mean setup.py is an entry point, however, if we consider these tools not to have that entry point, then there should be a consumer of these tools where those req should be (dependencies) and they should follow what python modules do, you install them prior a use, or at least quick check. and a user do what is required - install via pip, install as an egg, virtualenv, etc..

@0xc0170 the impact shouldn't be significant as pip install -r requirements.txt is only invoked if modules are missing. @theotherjimmy perhaps reload(site) should be part of the pip install... rather than applied every time, e.g. even if the modules are okay.


Add environment variables for backend service + missing tests

Added environment variables for backend service
Added missing tests

Codecov Report
Attention: Patch coverage is 72.46377% with 19 lines in your changes are missing coverage. Please review.

Project coverage is 84.98%. Comparing base (e1d230d) to head (4bdd526).

Files
Patch %
Lines

src/app/user-stats/user-stats.component.ts
83.01%
9 Missing :warning:

src/app/app.component.ts
38.46%
8 Missing :warning:

src/app/services/local-storage-checker.service.ts
33.33%
2 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##              dev       #5      +/-   ##
==========================================
- Coverage   85.76%   84.98%   -0.79%
==========================================
  Files          11       11
  Lines         267      313      +46
  Branches       18       13       -5
==========================================
+ Hits          229      266      +37
- Misses         38       47       +9

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


[Improvements]: Scheduled tasks

Scheduled tasks added here, but there are a few improvements that will be added:

[ ] Add email notifications for success/fail
[ ] Trim executions to last ~20 or so to limit db growth for historic runs
[ ] Execution time limits
[ ] Display next run time, total execution time
[ ] Probably some standardisation/consistency issues in here

I'd also love to be able to control the Scheduled Tasks as IAC by having a .crontab or similar file in the project's codebase. Is this a useful feature to track here or should it be considered separately?

[ ] Also consider adding Slack notifications for failed tasks.
[ ] Add tasks to each application individually, not just globally
[ ] The pop-up input is too small for long cron task, consider using a wider area on the dashboard.

A task of mine seems to have got stuck a few days ago and there's no way to cancel it, nor did the task re-attempt the next day. Could we supply a timeout or something?

We definitely need this:

Add email notifications for success/fail

By the way, any idea why my ScheduledTask is not finding my script?
sh: 1: runScript.sh: not found

My "Recent executions" seem to change order on page load. This sometimes means the most recently executed task (even if it's currently running) don't show up in the list. Does this sound right?
Would you be open to a PR that ensures the recent executions are the 20 most recent ones?

Also want to mention that failed cron jobs shows logs once I click on them, but successful ones does not. is this meant to be like that?

Is there any plan to integrate this with the log drains?

Is it possible to cancel stuck tasks?


Update README.md

Added example of Maestro

Would like to squash this commits if you want?

Please describe what you have change in commit description.

Please use ready for review label if your changes are to review.


Fixed issues 4 and 5

Here it is ! (I was feeling less lazy this morning)

Hey, question: would it break your use case, if I switched to a RFC 3986 validating regex, which would disallow standalone schema-only absolute URIs like just "http://" to be used with the fromString constructor? I would rather narrow the scope and use a known-valid and tested approach.

Go for it, I don't create schema only url.


Enhancements to classification mechanism

Add some convenience APIs for getting the classification at a certain offset
Provided classification ranges that are consecutive and cover the full source text of the node, including none classifications.
Add API TokenSyntax.tokenClassification to get the classification for a specific token

@swift-ci Please test


Pass the opt_direction value to the getConstrainedResolution function

The parameter was not used

Related to #9812. At least you can copy/paste a meaningful description for the option from there.

@fredj do you have time to work on this or would you prefer if someone else would finish this up?

I've pushed a commit that changes the description for the opt_direction param.


Hermes to periodically check for registered deployments and deploy the appscalesensor app.

https://ocd.appscale.com:8080/job/Daily Build/1320/

https://ocd.appscale.com:8080/job/Daily Build/1337/

https://ocd.appscale.com:8080/job/Daily Build/1341/


There is a space in column name "object_ id"

It should be "object_id" instead of "object_ id"

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

ID: 0df79411-afd1-930a-5248-525ddcd37229
Version Independent ID: a10e89b4-cb68-2d1e-ee64-c44db4035c22
Content: SQL Server Audit Records - SQL Server
Content Source: docs/relational-databases/security/auditing/sql-server-audit-records.md
Product: sql
Technology: security
GitHub Login: @VanMSFT
Microsoft Alias: vanto

@halfkilo -- thank you for your feedback.  I am updating the documentation.


Incorrect result for COUNT DISTINCT used as Window function

The query
WITH t AS (SELECT col0 AS a, col1 AS b FROM (VALUES(1,2),(1,1),(1,2),(2,1),(2,1),(2,2),(2,3),(2,4)) v)
SELECT *, COUNT(b) OVER(PARTITION BY a), COUNT(DISTINCT b) OVER(PARTITION BY a) FROM t

returns incorrect results. The last column contains ordinary counts instead of distinct counts.
The most straightforward fix is probably to reject the DISTINCT keyword inside window queries. At least that's what Postgres decided to do, and the SQL 2011 standard forbids the DISTINCT keyword inside window queries, too

Good catch! Indeed, DISTINCT should throw an error when used in Window functions. It seems like it is just ignored silently currently.

Should be fixed now, thanks for the report :)

thanks for the quick fix :)
Still reproes in https://duckdbdemo.project.cwi.nl/select.html, but I guess the server is just not updated yet

Ok, now that I took a look at your commit, I found another case which you probably want to forbid:
WITH t AS (SELECT col0 AS a, col1 AS b FROM (VALUES(1,2),(1,1),(1,2),(2,1),(2,1),(2,2),(2,3),(2,4)) v)
SELECT *, COUNT(b ORDER BY x) OVER(PARTITION BY a) FROM t

Interesting, I never saw this syntax before. I suppose it is used for aggregates where input order matters, e.g. STRING_AGG? I think in general we should forbid this for now, as it is unsupported right now.

Still reproes in https://duckdbdemo.project.cwi.nl/select.html, but I guess the server is just not updated yet

Indeed the server still needs to be updated.

Ok, now that I took a look at your commit, I found another case which you probably want to forbid:
WITH t AS (SELECT col0 AS a, col1 AS b FROM (VALUES(1,2),(1,1),(1,2),(2,1),(2,1),(2,2),(2,3),(2,4)) v)
SELECT *, COUNT(b ORDER BY x) OVER(PARTITION BY a) FROM t

Should be fixed as well now, and also disallowed the FILTER syntax for aggregates for now. Thanks again!


Purchase not working in iOS 16

When I try to make a purchase using command below nothing happens. The purchase sheet doesn't show up in iOS 16:
try await Mercato.purchase(product: product, quantity: 1, finishAutomatically: false, appAccountToken: nil, simulatesAskToBuyInSandbox: false)

Is anyone else having the same problem? Thanks for your help!

And no errors, nothing? I don't have any problems for now.
Try to test it locally with StoreKit configuration file.

I'm testing on the device. I'll try with a StoreKit configuration file.

@neowinston How is it going? Do you still have any issues with Mercato?

Yes, I'm still having problems.  The purchase sheet doesn't show up in iOS 16.

@neowinston I think, I was able to reproduce it when I tried to subscribe to auto-renewable subscription. I was getting success message right away without showing the purchase sheet from StoreKit. For me the issue was in the product item itself, two other subscriptions was working well and after re-creating a product on apple connect everything works well again.
Could you please, if you have a chance, verify what Mercato.purchase returns for you in this case.
let purchase = try await Mercato.purchase(product: product, quantity: 1, finishAutomatically: false, appAccountToken: nil, simulatesAskToBuyInSandbox: false)
print(purchase)

Thank you!

Here is more info: https://developer.apple.com/forums/thread/723126


Suggestion: multiple item buyout at once

At the moment, when you right-click an item, you can set a buyout for it and other stuff.
My suggestion: make it possible to right-click and drag mouse over items to select many of them and then apply the result of the popup window to each item selected. This would make it way easier for setting buyouts for rows of currency.
Basically:

OnMouseRightClickDown, when mouse is over an item, add that item to a list.
**When mouse is dragged to a new item, add that item to list too.
OnMouseRightClickUp show the Buyout/Offer/Price window.
Apply the data input in the window to every item in the list.

A better currency buyout system is something I'd like to add, I just don't know when I will have time to do it.


KeyError: b'AC' in Tajima'sD calculation

Hi, I tried to run the tajima option but keep getting errors.
This is the command I ran:
singularity exec  vcfkit:0.2.9--pyh5bfb8f1_0 vk tajima 5000 1000 SNV.vcf.gz
And this is the error:
CHROM	BIN_START	BIN_END	N_Sites	N_SNPs	TajimaD
Traceback (most recent call last):
File "/usr/local/lib/python3.6/site-packages/vcfkit/tajima.py", line 148, in
main()
File "/usr/local/lib/python3.6/site-packages/vcfkit/tajima.py", line 144, in main
for i in tajima(args[""]).calc_tajima(wz, sz, args["--sliding"], extra=args["--extra"]):
File "/usr/local/lib/python3.6/site-packages/vcfkit/tajima.py", line 82, in calc_tajima
AC = variant.INFO["AC"]
File "cyvcf2/cyvcf2.pyx", line 2138, in cyvcf2.cyvcf2.INFO.getitem
KeyError: b'AC'
I am using Singularity so I believe there should not be the python module incompatibility issue...
Thank you very much for any help.

https://github.com/AndersenLab/VCF-kit/issues/33
Looks like this.


Renaming methods/classes based on review.

Based on architecture board review:

Rename SerializationType -> SerializationFormat
endpoint -> fullyQualifiedNamespace

fullyQualifiedNamespace does not require a protocol (ie. https://) so removing that validation.

Adding SchemaRegistryVersion and exposing it in builder.

Related https://github.com/Azure/azure-sdk-for-java/issues/24221

API changes have been detected in this PR. You can review API changes here

API changes have been detected in com.azure:azure-data-schemaregistry. You can review API changes here


Integrate with syzkaller

syzkaller is a coverage-guided OS kernel fuzzer. It can generate BPF programs of low quality and could benefit from a high-quality BPF generator. It won't be as efficient as Buzzer in stressing BPF subsystem itself, but will uncover bugs in more complex interactions between BPF programs and other kernel subsystems.
I have a prototype for integration, which you can mostly ignore except for the proposed interface between syzkaller and Buzzer:
// This is supposed to be buzzer.Generate.
// progType is the program type (BPF_PROG_TYPE_SOCKET_FILTER/KPROBE/...).
// oldInsns is the program for mutation, if empty, generate a new one.
// Returns new/mutated program and number of used map fd's in bpf_attr::fd_array.
func BuzzerGenerate(progType int, rnd *rand.Rand, oldInsns []uint64) (insns []uint64, maps int)

Buzzer and syzkaller has some model mismatch as syzkaller generates and mutates programs offline. So it's not possible to embed actual map fd's into the program (they don't exist yet, and we are not even running on the target machine).
So I restricted it to use of only bpf_attr::fd_array, which syzkaller will fill with requested number of fd's later.
It is possible to use fd's embed in the program, but it will require some for of a-la ELF relocations (buzzer will need to say what offsets in the program should contain fd's and which of these refer to the same/different maps). I decided to left this out for now.
Any other ideas on how to design the interface? Anything I've missed? Do we need attach type here? Or map types? Or anything to make it possible to call C functions from the BPF program?
CC @a-nogikh @tarasmadan

Hey @dvyukov
Thanks for opening this, let's pursue this idea!
I was chatting offline with @meadori about how we could achieve this and here are some initial thoughts:

From a Buzzer perspective and eBPF generation perspective, the ebpf program generation is already kindof self contained, we think it would make sense to export that as a go module/library that could then be imported and used into syzkaller.  While we considered the option of fully integrating buzzer into Syzkaller and deprecating some other features/logic, we think there is a path moving forward where both fuzzers share a common "core" and coexist in harmony :)

Thanks for your prototype! I reviewed it quickly and it overall looks good to me. I am still learning how to use the fd_arrays in eBPF but I am confident it would not pose a problem. I can't come up with anything else that would be needed as part of that interface.

On the buzzer side, the eBPF generation library needs a bit of work, we would like to increase unit test coverage, refactor into a more easy to use interface, etc. I certainly can squeeze the necessary work to have a syzkaller friendly interface among that work.

Regarding map types: Perhaps instead of returning the number of map fds the program uses from fd_array we could return an array of map_types? where each position in the array corresponds to a map_fd and each value represents the expected map type for that fd

@meadori is there anything else you think I am missing from here?
Thanks again for opening this and pursuing this idea!

@thatjiaozi you covered everything that we talked about.  At first glance, the integration steps seem reasonable.  I will look more at the specifics this week.

Re 1: totally fine with me.
Re 2: AFAIU when using the bpf array, the program refers to maps using index (0, 1, 2, ...), and the actual map FDs are supplied in a separate array. This allows the program to be constant regardless of actual FD values.
Re 3: Mostly up to you, but I would prefer earlier integration to parallelize the work and shake our interface details (you provide a trivial implementation early but with the final interface, and then we improve and integrate in parallel).
Re 4: Looks reasonable to me. But I don't know if different map types have significantly different interfaces or not. If they do (program validation will fail too often with wrong map type), then it makes sense.

Sounds good to me on the point on 3. Let's shake the details of the interface ASAP and then we can split the work.
I'll sync with @meadori on this and submit a PR soon.

Also I have to admint I had no idea how to use fd_array in ebpf but I just figured out, the documentation is not very helpful but this comment (https://github.com/libbpf/libbpf/blob/master/include/uapi/linux/bpf.h#L1185) gave me all the clues I needed.
After some local hacking I managed to get it to work with buzzer, I think, at the buzzer side, we should entirely ditch the hardcoded map_fd values in favor of fd_arrays as they look way more flexible.


KAFKA-14616: Fix stray replica of recreated topics in KRaft mode

When a broker is down, and a topic is deleted, this will result in that broker seeing "stray replicas" the next time it starts up. These replicas contain data that used to be important, but which now needs to be deleted. Stray replica deletion is handled during the initial metadata publishing step on the broker.
Previously, we deleted these stray replicas after starting up BOTH LogManager and ReplicaManager. However, this wasn't quite correct. The presence of the stray replicas confused ReplicaManager. Instead, we should delete the stray replicas BEFORE starting ReplicaManager.
This bug triggered when a topic was deleted and re-created while a broker was down, and some of the replicas of the re-created topic landed on that broker. The impact was that the stray replicas were deleted, but the new replicas for the next iteration of the topic never got created. This, in turn, led to persistent under-replication until the next time the broker was restarted.

Question: why can't we delete these disappeared(deleted) logs directly? I think this can only happen when topic deleting while the node is offline. If so, then deleting them should be fine?

Question: why can't we delete these disappeared(deleted) logs directly? I think this can only happen when topic deleting while the node is offline. If so, then deleting them should be fine?

The bug only happened when we delete and recreate the topic while the broker is offline and there is overlap on partition assignment between the new and old topic on the offline broker.
A better way to explain in it is by the example I mentioned in the Jira where

we had offline broker that held replicas for partitions  0, 3, 4, 5, 7, 8, 9 of topic foo.test with id MfuZbwdmSMaiSa0g6__TPg which we deleted while the broker is offline so they aren't physically deleted.
Then before brining the offline broker we recreated foo.test again which now have partitions 0, 1, 2, 7, 8, 9 assigned to same offline broker but now the topic id is RzalpqQ9Q7ub2M2afHxY4Q. Notice here that partitions 0, 7, 8, 9 are common between the assignment of the deleted topic (topic_id: MfuZbwdmSMaiSa0g6__TPg) and the recreated topic (topic_id: RzalpqQ9Q7ub2M2afHxY4Q).
When the broker come-back the ReplicaManager and LogManager will not complain and move one to create the missing partitions' log dir for partitions 1, 2, 6 as there are already dirs for 0, 7, 8, 9 (despite the fact that the topic id is wrong here).
Then later BrokerMetadataPublisher will detect the strays (which are 0, 7, 8, 9) and deleted them.
Now the broker is online but has only partition 1, 2, 6 and we have no other way to bring the broker to create partition 0, 7, 8, 9 without restarting the broker.

which lead to permanent under-replication until someone restart the broker. It could get worse on larger clusters if we have more than 1 offline broker and the same issue happened to all of them. We might end up with partition in permeant at-min-isr or permeant under-min-isr.

Question: why can't we delete these disappeared(deleted) logs directly? I think this can only happen when topic deleting while the node is offline. If so, then deleting them should be fine?

It probably would be fine, but I'm a bit nervous about it. What I'd really like is to delete them after a time delay of a few hours, but the code doesn't support that currently. We should add this functionality.

committed

When not merging via github we should include the PR number in the commit message otherwise the PR and commit are not linked. When digging through history it's often important to find the PR and discussion to understand some changes.
Commit: https://github.com/apache/kafka/commit/f7feb43af351efa25be37cb78814c19bd28a22a2


fix: support editable install with multiple roots

Python packages may have multiple root modules where each is installed to its own folder under site-packages.
When using new setuptools editable install in such cases griffe, when used by mkdocstrings fails to find the correct paths.
The root cause it that griffe assumed a variable named MAPPING in the .py file created by the editable install, and assumed that this variable is a dict with single entry.
When a package has multiple roots - then this dictionary contain multiple entries.
This PR aims to handle such cases.
I have no knowledge about editables so I could not fix that case.
The unit test added simulates the case, and this has been tested against a real complex package with multiple roots using setuptools 65 editable install.

Hi. 😄
My first attempt in PR here.
I hope I was able to properly follow the contributing guide.
If there is something that needs to be changed - let me know.


Data sample on table properties not using limit

Description
When going to the Data tab on table properties DBeaver throus this query to redshift, without LIMIT (I have 200 as limit)
SELECT
x.*
FROM
schema.table x
DBeaver Version
23,3,0
Operating System
maOS 14
Database and driver
Redshift
Steps to reproduce
Double click in a tabla on Database Navigator, go to the Data tab, and look into the Redshift console for the queries been run on the cluster.
Additional context
No response

Hello @alejandrofielm
Please provide a screen from your preference page.
Preferences -> Editors -> Data Editor page.

Here it goes!

Could you please try to turn on the "Use SQL to limit fetch size" and check data reading again?

That was it, I thought it was enabled by default and worked according to this:

why is not enabled by default?
Thank you!

Unfortunately, this is a database-specific setting, and it doesn't work in all databases that DBeaver supports.

Thanks, will change it manually from now


Added Travis Badge, codecov badge, and Apache badge. Also changed REA…

Added Travis Badge, codecov badge, and Apache badge. Also changed README.

Codecov Report

Merging #68 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #68   +/-   ##
=======================================
  Coverage   95.97%   95.97%
=======================================
  Files           3        3
  Lines         273      273
=======================================
  Hits          262      262
  Misses         11       11

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 dced15a...afb8ae0. Read the comment docs.


Excluding files from sources does not seem to work.

Hello
First of all, thanks a lot for this useful task.
We have been successfully using this task, but not we'd like to exclude certain files.
I have tried multiple patterns, but none seem to work:
**/*.yml
!**/02-build-config.yml

**/*.yml
!02-build-config.yml

**/*.yml; !**/02-build-config.yml

**/*.yml; !02-build-config.yml

I even tried starting with @, but this is for variables so did not expect it work. And as expected it didn't.
Could you advise on how we would be able to exclude certain files from the task? Or highlight what I am doing wrong here?
Thanks in advance and happy holidays!

FYI: As a workaround, I have created two directories. And now in my Azure Pipeline I launch the task on the correct folder.
Would be nice to have the files in the same subdirectory though.

Hi @shifqu,
The pattern which will work is the following:
**/*.yml;!**/02-build-config.yml

I forgot to trim values when splitting on the ; so the space you have after the semi-colon is part of the negative pattern. I'll fix this in the next release.


Waypoint: Passing Values to Functions with Arguments [Beta]

I think the first line should either end with "a parameter" or "parameters". "form of parameter" sounds weird.

Also I keep getting "RangError: Maximum call stack size exceeded" when running

@CaroleAnneHannon I don't get any range errors. What's your code?

New text:

@SaintPeter it's into the function, not in to the function.

Got it.

Hello,
I get the message RangeError: Maximum call stack size exceeded with the following code:
function myFunction(a, b) {
  console.log(a + b);
}
myFunction(10, 5);


Terragrunt output isn't commented on pushes

Describe the bug
If a push is made to an open PR, the terragrunt output is not commented in the PR when tg_comment: 1.
To Reproduce

Open a PR that will run a github action with this action inside of it. Have tg_comment: 1 to indicate that there is to be a comment on the PR. Ensure that this action can be run on pull request opens and on pushes.
Push a change to the PR.
Notice that the plan is not printed into the PR.

Expected behavior
The output of the terragrunt command should be in the PR.
Nice to have

[ ] Terminal output
[ ] Screenshots

Versions

Terragrunt Action version: v1
Environment details (Terragrunt version, Terraform version, etc.):

Terraform: 1.5.7
Terragrunt: 0.51.7

Additional context
This is useful if the developer sees something in the plan that needs to be changed and wants to update the code in the PR. With this bug, it looks like the plan isn't changing. They have to go into the github run to actually see it.

Turns out the issue was that I needed the synchronize event on my github action for this to work. This works as expected.

Turns out the issue was that I needed the synchronize event on my github action for this to work. This works as expected.

@aiell0 I am running into issues getting my output commented on the PR. Could you provide additional details about your fix? Are you referring to on.pull_request: [synchronize]?


error downloading images

Expected behavior
non-local images download
Actual behavior
docker: error pulling image configuration: Get https://dseasb33srnrn.cloudfront.net/registry-v2/docker/registry/v2/blobs/sha256/a3/a39777a1a4a6ec8a91c978ded905cca10e6b105ba650040e16c50b3e157272c3/data?Expires=1485134748&Signature=fM2OmECGmCS6mONkQPTWoIQ8ZqKPHPFW69Fhe24uD9-Ir~Az2YsZjC5iwgmFYgh6TuXFNciIxAzHmh0N65H1dxmBsroLtv5PAjUAJCTCnam4nsaJDO3PNTCEAVQp65M0pz5vee8C5n980uQ5GaTEhZkFPHbYqA1zs9aIYI52FRY_&Key-Pair-Id=APKAJECH5M7VWIS5YZ6Q: x509: certificate has expired or is not yet valid.
Information about the Issue
i can't pull any images
Steps to reproduce the behavior

...
...

No information provided or steps to replicate - Closing


An SMTP/IMAP package for threebot

Create an email package based on smtp/imap  under /threebotpackages/threebot/email
story
As a 3bot owner I want to be able to

add an email account and sync it
create/send emails
attach files in my email
receive emails
search emails

I also expect to display and save  emails through the 3botApp so that my emails will be stored on my 3bot but still accessible from my 3bot app
As it is stored in bcdb we would also need models to be stored

Part
Layer
Protocol or Format

Server
Syncer
IMAP Server

Server
Email
SMTP

Here we ONLY focus on the server side as the front side will be done by the jimber guys as they are developing the 3bot app

[ ] IMAP server using BCDB

[ ] SMTP server using BCDB

[ ] Add models corresponding to the files stored

[ ] actors

[] UI on 3bot app

[ ] Deployment

[ ] package.py
[ ] package factory
[ ] start paramaterized

[ ] Testing

[ ] sync account
[ ]  send email with attachment
[ ]  display emails
[ ] search emails
[ ] delete emails

@grimpy @RafyAmgadBenjamin please link your work and update progress here

IMAP work is being done at https://github.com/threefoldtech/jumpscaleX_libs/tree/development_imap/JumpscaleLibs/servers/mail
Currently basic functionality works for evolution/thunderbird there are still some caveats with attachements


To be defined

While implementing system into a larger codebase, discovering issues and fixing issues that prevent from code to work efficiently.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

7 of 24 (29.17%)  changed or added relevant lines in 2 files are covered.
37 unchanged lines in 5 files lost coverage.
Overall coverage decreased (-0.3%) to 89.529%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

validation/verifier.go
6
8
75.0%

sources.go
1
16
6.25%

Files with Coverage Reduction
New Missed Lines
%

ast/tree.go
1
73.97%

ast/function.go
3
97.65%

validation/verifier.go
3
89.47%

ast/reference.go
4
81.7%

sources.go
26
67.73%

Totals

Change from base Build<PHONE_NUMBER>:
-0.3%

Covered Lines:
10979

Relevant Lines:
12263

💛  - Coveralls


Dynamically ignore PRs

Follow up of #571
Problem
With the tag_resolver.filter and tag_resolver.transformer feature we can now filter and ignore tags when building a PR "diff" between 2 tags.
For example, it's possible to search for the "previous" version of a tag matching a prefix (e.g. api-v1.0.1 and api-v1.1.0 while ignoring client-v1.0.2 between). The action would then consider PRs between those api- tags and not treat client-v1.0.2 as the previous version of api-v1.1.0.
What's currently not easily doable is to ignore PRs that shouldn't make it into a changelog based on the tag value.
While one can configure ignore_labels, this is static and applies to all "release scopes".
The goal is to dynamically ignore PRs one certain conditions.
In the given example, a PR labelled documentation but meant for the api should only be included in the changelog if the release tag also contains api-. For any client releases, that PR should not be included.
Workaround
It should be possible to use the exhaustive option in the category labels. To make it dynamic, one would need to patch the config to all categories before building a changelog (e.g. with jq).
However, there's a limitation: If a category "bugfixes" is built with either bug or fix, when there's api label and exhaustive=true, then a PR would need both bug AND fix (before it was OR). Alternatively, only bug/api pairs or fix/api are allowed
Proposal 1
Add a config option filter_labels.
Only PRs having one label from this list is further categorized based on other, additional labels. Alternatively, all labels are required. Maybe configurable with a method param?
This allows to categorize PRs based on labels like documentation or bug without requiring a scoped label, e.g. api/documentation and api/bug. One can just label documentation and based on the tag, the action would automatically figure out if a documentation PR is appropriate or not.
There is a limitation: Requiring a dedicated label for each "release scope" (e.g. api and client) works well for small list of release scopes. For highly dynamic release scopes it becomes more and more a pain to maintain those labels on GitHub. That's where Proposal 2 might be an alternative.
Proposal 2
Extract a label from label extractor and then filter on those.
Consider the following config snippet:
  "ignore_labels": [
    "ignore"
  ],
  "label_extractor": [
    {
      "pattern": "^(?!\\[api\\])",
      "method": "replace",
      "target": "ignore",
      "on_property": "title"
    }
  ],

Any PR that doesn't have the [api] prefix at the beginning of the PR title internally gets the ignore applied, so that the action will ignore unrelated PRs. This label extractor feature already exists today, but somehow it doesn't really work.
I need to confirm/debug this first to exclude a config error. It's possible that this proposal is obsolete if that approach already works but I did an error on my end.
This method would be more flexible and dynamic due to not requiring an existing set of labels compared to Proposal 1, but it makes it more fragile (depending how good the pattern is).

So far I can confirm that the workaround with exhaustive works.

Regarding proposal 2.
So, the config snippet provided does work like the following:
ℹ Added label ignoregenerate a changelog after helm-chart-releaser published release
ℹ Added label [fronius-stack] bogus change to test changelog generator
ℹ Added label ignore[fronius-exporter] bogus change to test changelog generator that should be ignored

patched transform.ts:
        for (const label of extracted) {
          core.info('ℹ Added label ' + label)
          pr.labels.add(label)
        }

I misunderstood the target property (based on its name) to be the label that will be applied if the pattern matches. However, it's the string that gets applied where pattern matches.
I'll try adapting my pattern, maybe I can do some regex so that the whole title gets replaced with ignore

So, with a pattern like         "pattern": "^(?!\\[fronius-stack\\])(.*)", I get the following additional labels on a PR:
ℹ Added label ignore
ℹ Added label [fronius-stack] bogus change to test changelog generator
ℹ Added label ignore

This now works as intended. Internally, the PR will get another label that is exactly like the title, not sure about the implications of this. One could label a PR bug and then name it documentation and it would come up in 2 different categories... It would be a rare edge case though and I don't think an action is required here. Maybe you know a better pattern/target?
I would still prefer proposal 1 for a more robust approach, especially for a finite set of "release scopes"/labels.

Glad you could resolve it. Did you get the project set-up locally now and used a test to quicker iterate?
I'll have a look, I think the pattern should be possible to be easier. which would in that case make solution 2 the cleaner solution I believe.

Did you get the project set-up locally now and used a test to quicker iterate?

I cloned the feature/572_test branch and edited the config and src files until I got a working npm test result. Didn't figure out how I could debug in VScode or run only a selective test (not a TS/js dev)

If you are in the terminal view of VSCode, there's the plus button on the right side, chose JavaScript Debug Terminal

And then within that terminal, just start the test in a single file like:
npm test -- fronius.test.ts

it's...beautiful
Thanks for the VSCode hints
I got it working, but I've had to choose the workaround with the exhaustive option. Some PRs are bot-generated and thus I can't rely on label extractor to attach labels from titles.


Change DFIR IG to IR IG

Describe the solution you'd like
See IR instead of DFIR

A couple things you have do for this:
Change this from "DFIR" to "IR":
https://github.com/ritsec/ops-bot-iii/blob/ab94a06d5988a9edbf06feef69865295d184a458/ent/schema/signin.go#L33
Generate Code from ent schema:
go generate ./...

Change these from "DFIR" to IR:
https://github.com/ritsec/ops-bot-iii/blob/ab94a06d5988a9edbf06feef69865295d184a458/commands/slash/signin.go#L43-L44
https://github.com/ritsec/ops-bot-iii/blob/ab94a06d5988a9edbf06feef69865295d184a458/commands/slash/signin.go#L119-L120


Add nestjs cli completions

Description

Nestjs is a backend development framework for nodejs and has its CLI to scaffold the project. This PR contains the bash completion for the CLI
Motivation and Context

How Has This Been Tested?

Locally on Linux OS
Screenshots (if appropriate):

Types of changes

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

Checklist:

[x] My code follows the code style of this project.
[ ] If my change requires a change to the documentation, I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] If I have added a new file, I also added it to clean_files.txt and formatted it using lint_clean_files.sh.
[ ] I have added tests to cover my changes, and all the new and existing tests pass.

@davidpfarrell @NoahGorny @gaelicWizard Ping 😄

Not required


Chrusher backfiller hash

Check the hashes of backfilled segments (as per #80). Also some more metrics.

Fixes #80


Add base target blank when rendered in iframes

What
Add a script to determine whether or not the current page sits within an . If it does we change the base target to be blank so that all links will open in a new window (outside the iframe).
Why
To allow people to check links (especially when they point to an non-secure (http) location) while previewing pages in Content Publisher.
Trello card

Had a quick look, code looks great! Was it copied from somewhere? Asking as it seems a mature solution already, and if so the commit could refer to the repository from which it was originally copied.

No, it happens that sometimes I end up writing mature solutions, although this time doesn't seem to cover all of our needs — I may need to iterate over all links. If you're referring to the iframe detection I first wrote it for the cookie banner component, but I don't think it's worth abstracting it yet - especially while this code lives in static.

Have left a couple of comments, though I don't have the context of any of the kickoff conversations so take my comments with a pinch of salt

The was no kickoff for this story.

Thank you for the review @ChrisBAshton. Based on @kevindew's comment I extracted this module and only load it if the draft environment is setup (meaning it's not loaded for everyone but only for the draft stack which is used for previews). Do you mind reviewing this again? I left it a separate commit to ease the 2nd review.

I'm going to merge this. @kevindew if any of your comments needed to be addressed I'm happy to iterate this over afterwards.


Deployed Endpoint App Insights Not Capturing Full Data [BUG]

Describe the bug
The application insights for PromptFlow as a deployed endpoint cuts off the full data.
When the content provided to the prompt is too long it cuts it off and does not provide the full data payload.
How To Reproduce the bug
Steps to reproduce the behavior, how frequent can you experience the bug:

Deploy a PromptFlow endpoint
Add an input argument which requires 24611 characters (or a large number)

Expected behavior
Would expect all the content used in the PromptFlow would be included in application insights, but these fields are cut off.
Additional context
I understand that there may be size/cost issues with retaining all data. But there should be some kind of notice like TRUNCATED DUE TO SIZE so it does not appear that there was an error in processing.

Hi @adampolak-vertex , thanks for reaching us. Promptflow will not limit data size, I guess the data size was limited by Azure App Insights. I found some reference about this limitation:
https://learn.microsoft.com/en-us/answers/questions/543396/how-to-increase-the-size-of-messages-logged-in-app
https://learn.microsoft.com/en-us/azure/azure-monitor/app/api-custom-events-metrics#limits


Google Docs

If copy is disabled in google docs it still doesnt work

the context menu only says as option save image

You means "Selection and Copying Restorer (Universal)" right?
added // @exclude             https://docs.google.com/*
should exclude Google Docs now.

yep, now it doesnt give me the image context menu :)


Upgrade to Semantic Kernel 1.4

Motivation and Context
Chat Copilot hasn't been upgraded to SK 1 and this hampers development.
Description
Upgrade to SK 1.4. It has OpenAI's function calling working but, other than that, no planning functionality.
Decision to remove planners:

The old action, sequential and stepwise planners are deprecated
The function-calling capabilities baked in OpenAI obsoleted most of their use
The code required to support planners as that functionality existed vastly needed to be re-written
Instead of re-writing that code and then finally merging the upgrade to main, the code was removed so that those not requiring planning could move on to the latest version of SK while those depending on planning can stay on the older CC version until planners are added back in (if this is eventually done - depending on demand and the amount of work we want to put in supporting CC)

Contribution Checklist

[ ] The code builds clean without any errors or warnings
[ ] The PR follows the Contribution Guidelines and the pre-submission formatting script raises no violations
[ ] All unit tests pass, and I have added new tests where possible
[ ] I didn't break anyone :smile:

Thanks!!!

Do you remove planner's kernel?

Yes.  The idea is that OpenAI's function-calling can figure out when to call functions without requiring a plan to do so.
Let me know if you think the lack of explicit planner will be a problem.

@glahaye I have tried to use the Karna and Websercher plugins, but since the plan does not activate, they do not launch these and OpenAI responds directly. Is this the expected workflow?
I had to correct some exceptions that were thrown when trying to use the chat. For example, the duplication of the chatid in the list.
Thank you very much because thanks to this project I can understand much more about the functioning of SK.

@glahaye I have tried to use the Karna and Websercher plugins, but since the plan does not activate, they do not launch these and OpenAI responds directly. Is this the expected workflow?

I am trying to fix the automatic function-calling today.  When it works, plugins should automatically be invoked when they are relevant.

I had to correct some exceptions that were thrown when trying to use the chat. For example, the duplication of the chatid in the list. Resolved using last commit.

If you encounter problems, please feel free to enter them in this PR so that I can address them.

Thank you very much because thanks to this project I can understand much more about the functioning of SK.

My pleasure!

@glahaye I have tried to use the Karna and Websercher plugins, but since the plan does not activate, they do not launch these and OpenAI responds directly. Is this the expected workflow?

I am trying to fix the automatic function-calling today. When it works, plugins should automatically be invoked when they are relevant.

I am trying to use the websearcher plugin that has the solution, but it seems that it is not able to invoke it. Before migrating to SK 1.4 using the planner, I made the request.
I have tried using the Klarna plugin and it seems to work.

@ezar I'll see what I can do about the websearcher plugin. I'm not officially given any time for this project anymore so it might take a bit :/


Add organisation endpoint

Commit for adding the organisation endpoint that returns organisation names and ROR ids.
Add:

Organisation endpoint

I think I created some conflicts because I manually uploaded the main.py file earlier to the GitHub main branch, but deleted it because I wanted to add it via this branch.


When creating Child Bridges, GUI only displays first bridge.

Describe The Bug:
Creating multiple child bridges via JSON config's "Add Platform" function, allows the manual creation of multiple children, but when you try to access SETTINGS for this plugin, only the first child bridge is displayed (forcing you to configure via text, or configure one at a time to get the JSON syntax, so you can then paste it into the subsequent children).
To Reproduce:
Go into JSON Config and select "Add Platform". Paste in a legitimate config in this newly-created platform, then save.  When back on the Plugins page, select SETTINGS, and you'll only see the first child.
Expected behavior:
When visiting settings and multiple children/platforms created, each one should be displayed.
Logs:
Show the Homebridge logs here, remove any sensitive information.

 You could trying creating an issue on https://github.com/oznu/homebridge-config-ui-x to get them to enhance it.

@mantorok1, not to dispute you on this, as you clearly know more about this stuff than I (I couldn't write a plug-in if someone paid me to) :), but I have that very functionality on an Elk M1 plugin, which is why I knew it should be visible here as well. See in the attached pics, I was able to add 2 additional ELKs from directly into the GUI (on that plugin), and that I have an ADD PLATFORM button in the plugin's SETTINGS GUI:

Interesting. I know when child bridges were first introduced it wasn't well supported by the Config UI. Maybe it's improved since. I'll do some investigation and see what I can do

Cool. Thank you. I worked around it for now by creating the lighting config, then pasting that JSON into a second child, then building out the security config (and leaving it in the first position), but would be good to have in the roadmap…. Thanks.

I've had a quick look at the ELKM1 plugin and one thing I noticed that was different was in the config.schema.json it has the singular key set to false but the OmniLink plugin has this set to true. If you have time and you are comfortable editing this file you could try changing it to true and see if it solves the problem.

I don't mind SSH'ing into it and trying that, but you'd have to tell me the path to edit - I know just enough to be dangerous. ;)

Thanks for trying it out for me. I'll add it to the next version of the plugin.
I've not seen that log message before but then I only use a single config for my Omni (which runs in a child bridge). It doesn't make a lot of sense since you need multiple configs in your case and Homebridge is supposed to support it. When I get some time I'll try to replicate the issue.
It could be some sort of glitch. I found the same sort of issue mentioned for another plugin here. It was suggested to remove the plugin, clear the accessory cache and delete it completely from HomeKit then try adding it all again. Make sure you backup you configs first though.

This had been fixed in version 1.5.10


How does Artsy do payments?

Subscriptions, aka galleries pay a monthly fee: Artsy has a standard account for subscriptions and we create charges daily in code (we don't use Stripe subscriptions).

Invoicing, aka partners charge customers when they sell a work: we use Stripe Connect, partners create a stripe standard account which is connected to Artsy's SaaS account through the connect platform where we create charges on their behalf via API and customer pays whichever way they want.

Auctions, aka customers win lots at an auction: we create a charge in the Artsy standard account and pay out the customer once we have collected all the money in bulk.

Re. auctions, we also use valid stored cards as a sort of validation mechanism for bidders.
Re. invoicing, note that partners link bank accounts in order to collect their payments.

More from @joeyAghion:
We've explored using their marketplace offering for all of e-commerce and auction and invoice payments, but in each case we've chosen to stick with the traditional product (for more control, mostly). Subscriptions charges are our most typical use case, because those funds are for us and not being held for a 3rd party.
To answer more generally, we use Stripe for validating, tokenizing, and storing cards. We sometimes create charges against those stored cards (for any of the purposes listed above). We have the ability to continue charging cards even after their numbers change, in some cases (when the network offers that integration with Stripe).

This issue is light on activity. Feel free to comment if you are inspired to write this blog post!


ISSUE-15: (Windows) Missed symlink to phar at bin folder

Short description of what this resolves:
Fixed Windows OS issue with bin folder
Changes proposed in this pull request:

For Windows environments suggested using copy instead of symlinks

Version: 1.x
Fixes:

ISSUE-15

@inri13666
Sorry for the late response and thank you for the contribution! I would prefer to create a new copy method instead of hiding this in the symlink method.
I made a new branch for this change called "copy-file-for-windows". Could you maybe try it?
{
	"require-dev": {
  		"tm/tooly-composer-script": "dev-master#f2d146b98a9fe83b28624515aa143dbcd40b79cc"
	}
}

@inri13666
Give me a sign and I'll merge this asap.

following code do not work on windows

please see the following comment
https://github.com/tommy-muehle/tooly-composer-script/commit/4cd775d54e4f362ab135775d59f65588d0c09711#r32305820

Fixed in the latest master. I'll also release a new version soon. Thanks for your contribution again!


Optimisation for pandas apply

In a few cases, use of pandas apply takes up a reasonable fraction of time, e.g., in frequencies functions when creating variant labels.
In benchmarking it's faster to use built-in map() with itertuples(). It's also faster to select only necessary columns first. E.g.:

Even faster...


stm32: remove TIMX singleton when used on timer driver

After multiple ways of looking at this, this is the best solution I could think of.

bors r+


Fix AnimationStateMachine cannot connect nodes bug

Fixed #65988.
The bug was that the unlocking of updating_transitions in _rename_transitions() was unexpectedly inside the for loop, so if the transition did not exist, it would not enter the for loop and updating_transitions would not be unlocked. Maybe it's a rebase or a copy-paste error.
In addition, some lacked signals have been added.

Thanks!


Tcp active open (wip)

This is an attempt at providing a way to initiate a TCP connection (see https://github.com/m-labs/smoltcp/issues/7).
I added an example to test the changes but this is not working yet. Here is what I get when running the client side:
[     0.026s] (socket::set): [0]: adding
[     0.026s] (socket::tcp): [0]<IP_ADDRESS>:6979:<IP_ADDRESS>:6970: state=CLOSED=>SYN-SENT
[     0.027s] (socket::tcp): [0]<IP_ADDRESS>:6979:<IP_ADDRESS>:6970: sending SYN
-> EthernetII src=02-00-00-00-00-02 dst=02-00-00-00-00-01 type=IPv4
   \ IPv4 src=<IP_ADDRESS> dst=<IP_ADDRESS> proto=TCP
    \ TCP src=6979 dst=6970 syn seq=0 ack=0 win=64 len=0 mss=1446
<- EthernetII src=02-00-00-00-00-01 dst=02-00-00-00-00-02 type=IPv4
   \ IPv4 src=<IP_ADDRESS> dst=<IP_ADDRESS> proto=TCP
    \ TCP src=6970 dst=6979 rst seq=0 ack=1 win=0 len=0
[     0.028s] (socket::tcp): [0]<IP_ADDRESS>:6979:<IP_ADDRESS>:6970: unacceptable RST|ACK in response to initial SYN

So the client sends a SYN+ACK instead of SYN, but I don't understand why yet.

I got this wrong... State should be switched to ESTABLISHED after acknowledging the SYN-ACK...
Will continue tomorrow.

Don't bother. I am unhappy with the code quality anyway. I'll implement this myself.

Well that's pretty harsh. What is particularly bad about my code? Would you mind mentoring me instead? I'd like to learn, that's primarily why I'm playing with redox and smoltcp.

What is particularly bad about my code?

No tests (an example doesn't count as test), a large amount of missed subtleties of TCP. Ideally I aim for 100% condition/branch coverage though Rust makes it somewhat hard to achieve with all the overflow checks.

Would you mind mentoring me instead?

It took me weeks of full-time work to get a rudimentary understanding of how TCP works and how it should be implemented. I have neither time nor the depth of understanding to transfer that to someone else.
This isn't something about you personally and I intend you no offense. It's rather that my primary goal is to have a TCP stack that actually works decently, and mentoring someone is an extremely inefficient way to get there.

That being said, I will gladly accept any contributions (or mentor contributors) that don't touch the TCP state machine specifically.


Use one MediaHandler per Session

Having a single handler per session (instead of a separate handler for each early dialog) provides at least two benefits:

Reusing the same media for multiple early dialogs would be easier to do automatically, without the application needing to know anything about it.
WebRTC.MediaHandler.getRemoteStreams could return the early media streams if available, and the on-call streams otherwise (again, without the application having to dig out the early dialog and its streams).

We are currently playing with a new MediaHandler per getDescription/setDescription (to get fresh peerConnections for reINVITE scenarios and on-the-fly handling of adding/removing screens).
Will keep you posted on our progress if that is an approach that would be satisfactory for merging into upstream as well.

This is fixed in 0.8.0.


Adapt to PSR-7

Starting the development server after installing latest dev-master gives you the following error

This is due to this code in the standardcontroller

We will have to adapt it  to psr-7 implementation in latest Flow version

Fixed with https://github.com/neos/flow-welcome/commit/fa290a814ce406eca3d57ab0a22730cf1b124d69 ;)


docs are redundant

Seems that https://github.com/kubevirt/kubevirt/tree/master/docs/design and https://github.com/kubevirt/kubevirt/tree/master/docs/ contains two similar documents.

We have a more recent description in docs/component.md, so we should keep the content of that file.