[Quesiton] Reply messages with mention and without and private messages

how to make the bot reply messages without mention and make them reply private messages, and if u guys know, make a command like ~calc (math calc) just like ~calc 29 * 2 ~calc 28 + 88

I think you're better of by visiting these links first:
Learn Javascript:
https://www.codecademy.com/
https://developer.mozilla.org/en-US/docs/Web/JavaScript
Learn how to use Node.js:
https://nodejs.org/dist/latest-v5.x/docs/api/
Read Discord.js's docs & Examples:
http://discordjs.readthedocs.org/en/indev/docs_client.html
https://github.com/hydrabolt/discord.js/tree/master/examples
After you've got the basics down. You can then proceed to use this library to program a bot. Such issues as yours aren't a problem with the library, but your lack of knowledge on how to use it. Which the above links should help.

I would like to know this as well, as it's not a knowledge issue, rather a question regarding the library's functions. The docs do not mention if you must use RegEx to split the mentions from the messages (which is a hassle to learn anyways, and therefore should not be treated as common knowledge) or if there's a command to return the clean message content.

Can't you just override Discord.Message.prototype.reply?

I don't think rewriting a method in a module is much easier than using RegEx. Really, the best way would be to have simple member like a Message.InnerContent separate from the main Message.Content, or have a Message.Content.NoMentions submember.


Events: used persist instead of clone

I experienced the same issue than 46 and fixed it with this.
I understand that you clone the events because the SyntheticEvent is pooled and the properties are set to null. React gives a way to persist the events so we can get rid of cloneDeep and its heavy memory cost and the lodash dependency 🍾 !

great


Integration Testing a Consumer

We are starting on integrating Kafka and racecar into an app.  Looking at the documentation and the code, it's not clear to me how we can integration test a producer-consumer pair from within our test suite.
that is, while in production the produced events will be coming from an external service, in our test suite we would like to run a consumer, produce what should be an event that matches what will be produced in production, and assert against the expected results of the consumption.
anything that exists for this that y'all can point me to, or a discussion of the challenges of doing so?
both for the gem and the reply, many thanks!

Assuming you're on 2.0 alpha:

If you just want to test your behaviour of your implementation, you can use something like this:

msg = instance_double(::Rdkafka::Consumer::Message,
  key:       key.to_s,
  payload:   payload ? payload.to_json : nil,
  topic:     topic,
  partition: 0,
  offset:    offset,
)

YourConsumer.new.process_batch([msg, msg])

Obviously that's not too great, because it requires you to have internal knowledge about how racecar works, though you can also just dump an example message on production and use whatever it outputs.

If you want to cover more code with your integration test, then it becomes hairy relatively quickly. If you start a Racecar::Runner (which receives your Racecar::Consumer) in a separate thread and mock on rdkafka-ruby level, you might cover some of Racecar's behaviour. But the runner doesn't have a concept of "exit after receiving message", so you'll have to call stop at the right time, raise from your consumer under test or other such approaches which don't seem too appealing. Additionally you'll mostly have to mock librdkafka away, which also contains a big chunk of logic with regards to retries, offset handling, etc. Long story short, in the current state this approach seems not worth it.

There's obviously the option to run Kafka and a Racecar consumer in a docker-compose or similar environment, then produce an event to Kafka and observe the result.

From gut feeling I'd not use 2). We never implemented 3) for this project so I don't have code to share.
We're currently using 1) plus a staging environment with alerting to test our projects on top of racecar and it has been fine so far.

Thank you very much for the reply.  We're evaluating our options.  We may end up going with (3).  If so, I will aim to update this issue with our findings.

So,
As we near launching Kafka as part of our infrastructure and App development,  I'll summarize what we've ended up doing for posterity.
As for tests that are part of our build, we have settled on approach (1).   Something like approach (2) was attempted, but proved too difficult and not worth the effort.
There is also a local-development win here for approach 3:  We added kafka and zookeeper images to our docker-compose, and we created a very simple ruby producer for local development.
# KafkaProducer
# purpose: to be able to send kafka messages from console in local development.

class KafkaProducer
  BASIC_CONFIG = {
    "bootstrap.servers": "kafka:29092",
  }.freeze

  def self.produce(message = {})
    new(BASIC_CONFIG).produce(message)
  end

  def initialize(configuration)
    @configuration = configuration
  end

  def producer
    @producer ||= Rdkafka::Config.new(@configuration).producer
  end

  def produce(topic:, key:, payload:)
    producer.produce(
      topic: topic,
      key: key,
      payload: payload
    )
  end
end

We can then, in different term windows:

spin up a racecar consumer locally
open up a rails console to produce messages using the above producer, taking a sample message from the actual producer.

This helped us debug our consumer.
also, if anyone is using docker, and is having trouble getting the rdkafka gem to build properly, we needed to compile it from source in our Dockerfile:
ENV LIBRDKAFKA_SOURCE https://github.com/edenhill/librdkafka/archive/v1.2.0.zip

RUN curl -L $LIBRDKAFKA_SOURCE -o librdkafka-1.2.0.zip && \
    unzip librdkafka-1.2.0.zip && \
    cd librdkafka-1.2.0 && \
    ./configure --install-deps --source-deps-only && \
    make && \
    make install && \
    ln -s /usr/local/lib/librdkafka* /usr/lib/ && \
    cd .. && \
    rm -rf librdkafka-1.2.0 librdkafka-1.2.0.zip

sincerely,
denverCoder10

oh and also, thanks again for the guidance @breunigs !


Fix TWA links in README

@pampas93 thanks for the contribution. We need you to accept the CLA in order to merge.

I was unable to complete the CLA with my current hotmail email. I'll create a new PR with the CLA signed ID

@andreban Created new PR #712


Use powershell preparer

Switches over to the arm template. next step is to add the samples as part of the pipeline

/azp run python - tables - tests

Also fixes #12614


[BUG] Crash on doWeatherStuff

[Please fill out the form below and delete the sections in square brackets after reading them]
Version of Advanced Rocketry

1.6.0.-162

Have you verified this is an issue in the latest unstable build

[x] Y

Version of LibVulpes

0.4.1.-1

Version of Minecraft

1.12.2

Does this occur without other mods installed

[x] N

At least, not that I've been able to discern. I'm using AdvancedRocketry for the basic dimension, JustEnoughDimensions for a series of tweaks, including making this the spawn dimension. I'm also using BiomeTweaker to create the two biomes permitted on this world. At first, weather was allowed in those biomes but now it's disabled via BiomeTweaker and this still occurs.
JustEnoughDimensions 1.6.0-dev.20190711.220109
BiomeTweaker v3.2.354
Crash report or log or visualVM (if applicable)

https://gist.github.com/MuteTiefling/04d1252e3d7d5e8e3a936e567747c3f3
https://github.com/maruohon/justenoughdimensions/issues/47

Description of the problem
After sitting around in the world for a period of time, the game crashes with the attached errors. Including snippet here for ease of future searches.
If you'd like to see any configs, I'll happily provide them.

Thread: Server thread
Stacktrace:
at zmaster587.advancedRocketry.world.provider.WorldProviderPlanet.doWeatherStuff(WorldProviderPlanet.java:132)
at zmaster587.advancedRocketry.world.provider.WorldProviderPlanet.updateWeather(WorldProviderPlanet.java:124)
at net.minecraft.world.World.func_72979_l(World.java:2564)
at net.minecraft.world.WorldServer.func_72979_l(WorldServer.java:1236)
at net.minecraft.world.World.func_72835_b(World.java:2541)
at net.minecraft.world.WorldServer.func_72835_b(WorldServer.java:179)

This crash happens because JED allows using per-dimension WorldInfo overrides, ie. dimensions can have their own separate WorldInfo object, with their own separate values. JED does this via reflection, by changing the WorldInfo object of those dimension from the default DerivedWorldInfo into a custom WorldInfoJED object (which extends the base WorldInfo class).
It looks like AR has a check for Sponge in that piece of code, but if that is false, then it just assumes the WorldInfo to be DerivedWorldInfo without checking instanceof. (Off-topic: I kinda find it odd that it would change the weather of the overworld to get the planet to not rain... but I guess it's just due to the vanilla "all dimensions share everything" philosophy.)

Well, I've managed to get around this by using JED to disable weather in just the one dimension by applying the gamerule doWeatherCycle false there. @maruohon thanks for the comments, they got me to this workaround


[ Feature ] Add clip and model weights to the Power Lora Loader

The new Power Lora Loader is a great idea !
Is it possible to incorporate individual clip and model weights, to this new design ?
I think they would look good side by side, with the variable name next to the value, e.g.:
< clip 1.00 >  < model 1.00 >

Yes! I plan to do this, you're using the first version :)

You can now change to a single strength, or a dual model & clip from the node Properties!

Looks great.
I like the top line, with the text and the toggle all button.
But  since the objective is also more compact design, could you add an option to hide the [ + Add Lora ] button.
If hidden, the loras could be added  via right click menu instead.
Alternatively you could make a  smaller  [ + Add Lora ] buttom and put it the top line.


Update Nightly Run

What does this PR do?:
Summarize the changes. Are any stacks or samples added or updated?
This PR skips the java-quarkus stack entirely as it is failing our nightly run. Prior to this PR only the community starter was failing but now the entire stack is. An issue is going to be opened to investigate and fix the stack.
Which issue(s) this PR fixes:
Link to github issue(s)
N/A
PR acceptance criteria:

[ ] Contributing guide
Have you read the devfile registry contributing guide and followed its instructions?
[ ] Test automation
Does this repository's tests pass with your changes?
[ ] Documentation
Does any documentation need to be updated with your changes?
[ ] Check Tools Provider
Have you tested the changes with existing tools, i.e. Odo, Che, Console? (See devfile registry contributing guide on how to test changes)

How to test changes / Special notes to the reviewer:

Issue opened to investigate the skipped stacks and fix their failures so our nightly run is more accurate: https://github.com/devfile/api/issues/1530

FYI @thepetk @michael-valdron this should be good to go to temporarily fix our nightly run failures and stop the alerts. An issue is opened already for fixing the java-quarkus stack moving forward!


Merge pull request #2 from adriankumpf/master

bring fork up2date

Sorry, my bad - wanted it the other way.


CIV-4666 - Features - DQ - Shared Expert

JIRA link
https://tools.hmcts.net/jira/browse/CIV-4666
Change description
Implement shared expert page and appropriate tests
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[x] No

closing as #856 is used instead


Start using DifferentiationInterface.jl

Seeing Optimization.jl attempt to use DI, it seems like we have most of the features in place. We might be able to drop a lot of unnecessary code my making that shift!

@gdalle I am nearly done integrating DI into NonlinearSolve. The only feature I don't know how to use is specifying

A sparse jacobian as the direct prototype. SparseDiffTools has a JacPrototypeSparsityDetection(; jac_prototype)
If I already have the matrix coloring PrecomputedJacobianColorvec(; jac_prototype,          f.colorvec,          partition_by_rows = ADTypes.mode(ad) isa ADTypes.ReverseMode)

Currently the code is like:
@inline function __sparsity_detection_alg(f::NonlinearFunction, ad::AutoSparse)
    if f.sparsity === nothing
        if f.jac_prototype === nothing
            is_extension_loaded(Val(:Symbolics)) && return SymbolicsSparsityDetection()
            return ApproximateJacobianSparsity()
        else
            jac_prototype = f.jac_prototype
        end
    elseif f.sparsity isa AbstractSparsityDetection
        f.jac_prototype === nothing && return f.sparsity
        jac_prototype = f.jac_prototype
    elseif f.sparsity isa AbstractMatrix
        jac_prototype = f.sparsity
    elseif f.jac_prototype isa AbstractMatrix
        jac_prototype = f.jac_prototype
    else
        error("`sparsity::typeof($(typeof(f.sparsity)))` & \
               `jac_prototype::typeof($(typeof(f.jac_prototype)))` is not supported. \
               Use `sparsity::AbstractMatrix` or `sparsity::AbstractSparsityDetection` or \
               set to `nothing`. `jac_prototype` can be set to `nothing` or an \
               `AbstractMatrix`.")
    end

    if SciMLBase.has_colorvec(f)
        return PrecomputedJacobianColorvec(; jac_prototype,
            f.colorvec,
            partition_by_rows = ADTypes.mode(ad) isa ADTypes.ReverseMode)
    else
        return JacPrototypeSparsityDetection(; jac_prototype)
    end
end

ApproximateJacobianSparsity would map to DenseSparsityDetector, and I will clean up the above logic a bit, but we do allow users to pass in a prototype matrix and a colorvec.

For passing a known sparsity pattern, you can use ADTypes.KnownJacobianSparsityDetector instead of a detector from Symbolics.jl or SparseConnectivityTracer.jl. Just put it inside AutoSparse and it should work fine.
For passing the color vector I don't have a good answer right now, because this logic is handled by SparseMatrixColorings.jl and the coloring result types there are a bit more complex than just a vector of colors (this is especially true for Hessian coloring, which didn't use to be a thing in SparseDiffTools.jl). Let me think about it a little.

Thanks, I am fine with some manual combination of SparseMatrixColorings and DI if that is easy. This functionality is not exposed to the user so we have some leeway here.

The main issue I see is that this code mixes sparsity detection and coloring, like SparseDiffTools does, whereas the AutoSparse interface separates those two. Any idea how to get around that?

the sparsity_detection_alg can be re-written to return an AutoSparse type. Essentially like this:

ad isa dense type.

if sparsity is provided -> AutoSparse(ad;. ....) and some other cases with known sparsity patterns and such
if no sparsity info provided use dense ad

if ad is already a sparse type

Ideally there is no sparsity information
If there is in the problem definition, try resolving it. (the details here are not super important, but I will document it at some point)

The only part that doesn't fit in here is that we can have the colorvec, so we just need to do the AD in that case

Maybe I could add a KnownColorsColoringAlgorithm to SparseMatrixColorings, mirroring the KnownJacobianSparsityDetector?

I think that would do it

Approximate colorings can be used in order to give an accelerated Jacobian. Is there a way to do approximate colorings?
Also, using colorings on a dense Jacobian can be useful if the sparsity would is not enough and would slow down the LU. Can you sparse fill a dense matrix?

At the moment there is no approximate coloring but I could add it to DI.
As for filling matrices with the results of sparse AD, SparseMatrixColorings should work with any mutable AbstractMatrix. However it is only optimized for SparseMatrixCSC at the moment. More structured matrix types could be added in package extensions.

I think the main case of it is using a chosen color vector with a dense matrix. The reason for that is again if sparse LU is slow on the matrix (sparse matrices really need to be "sufficiently large" for sparse factorization to make sense) but coloring still makes sense to cut down the Jacobian cost.

Is it possible to query DI to get a Jacobian structure (basically the result of DI.jacobian but not computing the actual jacobian)? It is needed to ensure that we can create a concrete cache on init.
Note that similar doesn't work because jacobian might give a SMatrix.

Currently, we have four variants of coloring in SparseMatrixColorings.jl.
For some of them, we can easily use a user-provided coloring (vector of integers) and perform the decompression without loss of performance.

Row coloring (optimal)
Column coloring (optimal)
Star coloring (possible but slow without the structure StarSet)
Acyclic coloring (not possible; the structure TreeSet is needed)

StarSet and TreeSet are structures generated during the coloring.
Star and acyclic coloring are generally used on Hessians, while row and column coloring do not exploit symmetry and are mostly used for Jacobians.
I think just supporting colorvec as input for an expected row and column coloring is the best alternative.
You can use it for both Jacobians and Hessians.
You will keep all the features from SparseDiffTools.jl.

Is it possible to query DI to get a Jacobian structure (basically the result of DI.jacobian but not computing the actual jacobian)? It is needed to ensure that we can create a concrete cache on init.

Not at the moment no. Are we talking about sparse AD or general AD?
If you're creating a cache that means you're gonna run jacobian many times, so I think a good strategy is just to actually run it once to get a cache?

Note that similar doesn't work because jacobian might give a SMatrix.

But then what kind of cache do you want to get in this case? similar will give you an MMatrix, which seems like the only reasonable choice?

Not at the moment no. Are we talking about sparse AD or general AD?
If you're creating a cache that means you're gonna run jacobian many times, so I think a good strategy is just to actually run it once to get a cache?

Sorry I meant cache for NonlinearSolve. I am currently running it once to get the structure as you mention.
My structure I mean, if the returned J is mutable then DI.jacobian! should work on it and if it is immutable then I shouldn't be able to do DI.jacobian!

Then I would recommend you just copy the output of DI.jacobian? But keep in mind that I make no guarantee about the exact return type of the Jacobian matrix, because the backends don't either. So even if you give SVectors as input, with one backend you may get SMatrix, with another MMatrix, and with a third one a plain Matrix. The only thing that is guaranteed is the correctness of the values inside.

Right. This is what I will do (am doing on master) and seems like a fine solution since users caring too much about performance (1 jac call) should be using the caching interface anyways

@avik-pal does this do what you want in terms of fixing the vector of colors?
https://github.com/gdalle/SparseMatrixColorings.jl/pull/127

I think so. Do you need to make the conversion to sparse array? If you have a BandedMatrix it seems like a bad idea to make it Sparse because then the jacobian function with presumably return a sparse array instead of a banded matrix and linear solve will hit a regression

At the moment yes, there will be a conversion to SparseMatrixCSC because custom matrix types (like banded) are not yet supported by SparseMatrixColorings.jl. However, with ConstantColoringAlgorithm, this conversion to SparseMatrixCSC will only happen once (at creation).
What matters to me right now is getting the necessary functionality up and running, then we can see what the benchmarks tell us about performance regressions. But a fair warning that may not be obvious to everyone: when switching to the DI ecosystem, there will be performance regressions in some cases, and performance improvements in others. A good example is the Brusselator AD benchmark, where sparsity detection is 100x faster, coloring 10x faster, and differentiation 2x slower (although this has probably improved with the latest DI and SMC releases, I'll have to run it again).
To adapt to your use cases and milk the last drops of performance, first we need to get DI into NonlinearSolve, and then we'll identify the bottlenecks together.

Let me make a branch using DI for sparse jacobians, but I will hold off on merging that for now.

FYI the new version of SparseMatrixColorings that includes ConstantColoringAlgorithm is out

I re-ran the Brusselator sparse AD benchmark with the latest versions of everything, and DI is now on par with SparseDiffTools: https://docs.sciml.ai/SciMLBenchmarksOutput/dev/AutomaticDifferentiationSparse/BrusselatorSparseAD/


App Crash - Android

App crashed during testing 2 times, when I reopened, after the first time, the names shows as 0x instead of user name. Other than group chat - group chat remained with set name.

Here is a list of crashes on Android:
https://converse-app.sentry.io/issues/5619722109/events/cf604065104b417a6ca77436e2d699cf/?project=4504757120729088
https://converse-app.sentry.io/issues/4912444214/?environment=prod&project=4504757120729088&query=is:unresolved+os.name:Android&statsPeriod=24h&stream_index=1
https://converse-app.sentry.io/issues/5595661243/?environment=prod&project=4504757120729088&query=is:unresolved+os.name:Android&statsPeriod=24h&stream_index=12
https://converse-app.sentry.io/issues/5655074139/?environment=prod&project=4504757120729088&query=is:unresolved+os.name:Android&statsPeriod=24h&stream_index=16
@nplasterer @insipx @cameronvoell any of those ring a bell?
You can check the logs ("Breadcrumbs") by scrolling a bit, there sometimes are libxmtp errors

Okay just going through this list and going to add color to each of the issues I see
UNIQUE constraint failed: group_messages.id

https://converse-app.sentry.io/issues/5619722109/events/cf604065104b417a6ca77436e2d699cf/?project=4504757120729088
[libxmtp] receive error ReceiveErrors([OpenMlsProcessMessage(ValidationError(WrongEpoch)), Storage(DieselResult(DatabaseError(UniqueViolation, "UNIQUE constraint failed: group_messages.id"))), Storage(DieselResult(DatabaseError(UniqueViolation, "UNIQUE constraint failed: group_messages.id")))])

Maybe @insipx can speak to this one. This hopefully shouldn't be crashing the app if the group message was already written than it should just silently error and not break everything.
Expected URL scheme 'http' or 'https' but was 'eip155'

https://converse-app.sentry.io/issues/4912444214/?environment=prod&project=4504757120729088&query=is%3Aunresolved+os.name%3AAndroid&statsPeriod=24h&stream_index=1
java.lang.IllegalArgumentException(Expected URL scheme 'http' or 'https' but was 'eip155')

This looks like a java error. Are you certain this is being thrown from our code? Can you give me more details on where and what is triggering this?
API client error: identity error: status: ResourceExhausted

https://converse-app.sentry.io/issues/5595661243/?environment=prod&project=4504757120729088&query=is%3Aunresolved+os.name%3AAndroid&statsPeriod=24h&stream_index=12
Error: Call to function 'XMTP.createFromKeyBundle' has been rejected.
→ Caused by: org.xmtp.android.library.XMTPException: Failed to create client: uniffi.xmtpv3.GenericException$Generic: Generic Error: API client error: identity error: status: ResourceExhausted

This looks like an identity error thats maybe also related to the database. ResourcesExhausted feels like we have too many calls going to the database. So the previous calls to the database seem to be locking the database to some extent which then makes login fail. Probably a question for @insipx again.
OpenMlsProcessMessage(ValidationError(UnableToDecrypt(AeadError)))

https://converse-app.sentry.io/issues/5655074139/?environment=prod&project=4504757120729088&query=is%3Aunresolved+os.name%3AAndroid&statsPeriod=24h&stream_index=16
[libxmtp] receive error ReceiveErrors([OpenMlsProcessMessage(ValidationError(WrongEpoch)), OpenMlsProcessMessage(ValidationError(WrongEpoch)), OpenMlsProcessMessage(ValidationError(WrongEpoch)), OpenMlsProcessMessage(ValidationError(WrongEpoch)), OpenMlsProcessMessage(ValidationError(UnableToDecrypt(AeadError))), OpenMlsProcessMessage(ValidationError(WrongEpoch)), OpenMlsProcessMessage(ValidationError(WrongEpoch)), OpenMlsProcessMessage(ValidationError(WrongEpoch)), OpenMlsProcessMessage(ValidationError(WrongEpoch)), OpenMlsProcessMessage(ValidationError(WrongEpoch)), OpenMlsProcessMessage(ValidationError(UnableToDecrypt(AeadError))), OpenMlsProcessMessage(ValidationError(UnableToDecrypt(AeadError))), OpenMlsProcessMessage(ValidationError(WrongEpoch)), OpenMlsProcessMessage(ValidationError(UnableToDecrypt(AeadError))), OpenMlsProcessMessage(ValidationError(UnableToDecrypt(AeadError))), OpenMlsProcessMessage(ValidationError(UnableToDecrypt(AeadError))), OpenMlsProcessMessage(ValidationError(UnableToDecrypt(AeadError))), OpenMlsProcessMessage(ValidationError(UnableToDecrypt(AeadError))), OpenMlsProcessMessage(ValidationError(WrongEpoch)), OpenMlsProcessMessage(ValidationError(WrongEpoch))])

I was nearly certain that we fixed this issue in the latest release. Is it possible this was someone on an older build. We shouldn't be seeing this one any more going forward.

Thanks @nplasterer !
Yeah the API Client was not supposed to be there sorry.
Ok let's ignore the last one for now and see if it comes back, and let's see what @insipx has to say about the other two?

I'll be looking into these today and see what I can find to come up with more context!
Just from the error messages
1.) The first one seems to happen because of inserting a group message that already exists in the db. Could be a race like with PublishIntents or a number of things happening there
2.) This ones interesting eip155 is an Ethereum Improvement Proposal concerning ChainIds. It lets us know which endpoint to query based on a chain identifier. So something is happening where were trying to reach an https url but instead using a url with eip155 presumably to specify the chainid. There's a wrong URL somewhere but also will look into this today to figure out what's triggering it
3.) This ones definitely a gRPC Error and can actually be because of too many API Calls to the backend. I'll look into quota limits here, not too familiar with how gRPC handles these things but maybe we're just hitting the API too much with certain calls

I think we can close this task and open specific ones for Sentry crashes we see in the future.


Support building against network-<IP_ADDRESS>

Latest version fails with:
[4 of 8] Compiling System.Log.Handler.Growl ( src/System/Log/Handler/Growl.hs, dist/build/System/Log/Handler/Growl.o )

src/System/Log/Handler/Growl.hs:21:1: error:
    Could not load module ‘Network.BSD’
    It is a member of the hidden package ‘network-<IP_ADDRESS>’.
    Perhaps you need to add ‘network’ to the build-depends in your .cabal file.
    Use -v to see a list of the files searched for.
   |
21 | import Network.BSD
   | ^^^^^^^^^^^^^^^^^^

The Network.BSD module now lives in the network-bsd package.

@phadej @23Skidoo Have any of you been able to make any progress? We have a bug blocked by this.


Cannot program STM8S105K6 anymore

I cannot program my STM8S105K6-based boards anymore, which used to work.
Versions prior to 1d91c125705a6a1a41ec84a33b4a8f2c39ae2de8:
> ./stm8gal -p /dev/ttyUSB0 -u 2 -R 1 -w /tmp/arduino_build_28225/Blink.ino.hex

stm8gal (v1.2.0 beta)
  load Intel hex file 'Blink.ino.hex' ... ok (2.7kB)
  reset STM8 and press <return>
  open serial port '/dev/ttyUSB0' with 230.4kBaud ... ok
  synchronize ... ok (ACK)
  determine device ... ok (STM8S; 32kB flash; BSL v1.3)
  Uploading RAM routines ... ok (304B from 0x00a0)
  write 2.7kB (0x8000 to 0x8aa5) ... ok
  read 2.7kB (0x8000 to 0x8aa4) ... ok
  verify memory ... ok
  activate bootloader ... ok
  jump to address 0x8000 ... ok
done with program

1d91c125705a6a1a41ec84a33b4a8f2c39ae2de8 and later:
> ./stm8gal -p /dev/ttyUSB0 -u 2 -R 1 -w /tmp/arduino_build_28225/Blink.ino.hex

stm8gal (v1.2.0 beta)
  load Intel hex file 'Blink.ino.hex' ... ok (2.7kB)
  reset STM8 and press <return>
  open serial port '/dev/ttyUSB0' with 230.4kBaud ... ok
  synchronize ...

error in 'bsl_sync()': no response from BSL, exit!

They are connected via a CH340G USB-to-serial interface on the same board and I'm using Linux.

hi roybaer,
fixed it (hopefully). Can you please check branch develop again and provide feedback? Thanks in advance!
 This led to RTS and DTR being always active. My boards don't have DTR connected which is why I didn't see it before. But your sduino UNO is reset by DTR, which led to the observed loss of communication.
Thanks again for pointing this out, and also for the really nice board! Have a great day :-)

I can confirm that this solved the problem.

:-)
I'll merge the changes back to main branch


No support for armv7l (Raspberry pi 4)

Currently there is no support for the armv7l architecture. Would this be possible to support?

I'm not immediately familiar with armv7l. None of the existing arm builds work? What error do they produce?

The error happens when running the install script, but upon further inspection it seems that armv7l is not checked for. I assume that armhf would work too. Trying it out now.

This is the error btw:
==> [ERROR] Unknown Architecture: armv7l
==> Linux raspberrypi 5.4.51-v7l+ #1333 SMP Mon Aug 10 16:51:40 BST 2020 armv7l GNU/Linux
==> Please report this, along with the above OS details:
https://github.com/davidnewhall/unpackerr/issues/new

Got it! Yes, please let me know if the armhf build works for you and I'll update the install script.

I managed to install it with armv7l and it is now running as a service. However, I do not have any data to test it with at this moment but I assume that would work fine too 👍
Great job on this application. I am sure it will provide a great use for me when I have it fully set-up


GetPage kind param is neither fully respected nor really needed

.Site.GetPage takes a kind arg, but its implementation only partially respects it. For kind "page", it looks up only page Pages, as expected. But for any of the four node-kinds, home, section, taxonomy, and taxonomyTerm, it doesn't similarly limit itself.
For example:

.Site.GetPage("home", "posts") doesn't return the one and only page of kind "home", but instead returns the "posts" section page (it's a coincidence that the returned page is the "home" page of the section).
.Site.GetPage("section", "categories") returns a page of kind taxonomyTerm if one or more pages have a category value in their front matter.
.Site.GetPage("taxonomy" "funk") won't return the taxonomy page for "music", but will either fail or return a "music" section or taxonomyTerm page if one existed. Taxonomy GetPage calls must use the fully qualified name, e.g. .Site.GetPage("taxonomy", "tags/funk"). But then the "taxonomy" arg is redundant, as the "tags/" prefix already identifies this as a request for a taxonomy page as well as specifying which taxonomy ("tags").

This strange behavior is mostly innocuous, as the above are strange corner cases, and as the paths for any two node kinds, regardless of type, can never be the same.
But if the kind arg is eliminated:

eliminates the potential for confusion/surprise
simplifies the API
gives .Site.GetPage a signature consistent with the ref/relref shortcodes, which perform essentially the same function.

Pull request forthcoming...

This was suggested a while ago here too: https://github.com/gohugoio/hugo/issues/4652.
.. but I didn't know that the behavior was inconsistent (as I never tried putting incorrect Kinds).
I'd be interested in seeing what you propose in the PR. Would it break all the existing GetPages?

It think it will save time in the long run if you bring these issues you open to some kind of conclusion before you arrive with a PR.
The big question, in this case, is if it can be done without breaking existing code.

@bep What are the benchmarks I should run for this?


[WIP] Fix/shared model tx validation fix

What is this pull request?
Fixes stateless validation
Why do you implement it? Why do we need this pull request?
This PR should fix stateless validation bug, which causes the following problem.
When we create CommandsValidator which validates tx, we invoke validate method, where we go through every command and call boost::apply_visitor using CommandsValidatorVisitor where for every possible command we have implemented operators to perform validation logic. Problem is that visitor wrongly deduces the command type which should be validated and invokes wrong operator. For example when we do boost::apply_visitor to AppendRole for some reason invokes RemoveSignatory operator, which is wrong.
First reason was found and already fixed: the order of commands in commands.proto should be the same as in variant template parameters
How to use the features provided in the pull request?
Please check commands_validator_test and see that when we invoke validator on transaction containing AppendRole we see that validator tries to validate RemoveSignatory command (run commands_validator_test and see in printings that validator is invoked for RemoveSignatory)
Details/Features
Already fixed order of commands in commands.proto

Codecov Report

Merging #747 into feature/shared_model will increase coverage by 0.79%.
The diff coverage is 14.68%.

@@                   Coverage Diff                    @@
##           feature/shared_model     #747      +/-   ##
========================================================
+ Coverage                 41.51%   42.31%   +0.79%
========================================================
  Files                       395      396       +1
  Lines                     13336    13467     +131
  Branches                   7182     7265      +83
========================================================
+ Hits                       5537     5699     +162
+ Misses                     1414     1219     -195
- Partials                   6385     6549     +164

Impacted Files
Coverage Δ

shared_model/validators/commands_validator.hpp
39.55% <0%> (+30.6%)
:arrow_up:

...el/backend_proto/shared_proto_transaction_test.cpp
18.18% <0%> (+2.79%)
:arrow_up:

shared_model/validators/answer.hpp
83.33% <100%> (+3.33%)
:arrow_up:

...hared_model/validators/commands_validator_test.cpp
15.07% <15.07%> (ø)

...hared_model/interfaces/commands/transfer_asset.hpp
4.34% <0%> (+4.34%)
:arrow_up:

shared_model/interfaces/commands/create_role.hpp
5.88% <0%> (+5.88%)
:arrow_up:

...d_model/interfaces/commands/add_asset_quantity.hpp
5.88% <0%> (+5.88%)
:arrow_up:

...hared_model/interfaces/commands/create_account.hpp
6.25% <0%> (+6.25%)
:arrow_up:

...red_model/interfaces/commands/grant_permission.hpp
7.14% <0%> (+7.14%)
:arrow_up:

... and 25 more

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 b33d71a...6e8648e. Read the comment docs.

You've forgot to cleanup the generated pb files, that's the problem
I think we need some proto<>variant_idx map as we have in old model, otherwise we have pain in debugging such things
Please remove (by amending) debugging fixes in the pr


Fixed #27486 -- Fixed Python 3.7 DeprecationWarning in intword and filesizeformat filters.

Previously, running the Django test suite with warnings enabled resulted in several of the form:
…/django/django/utils/translation/trans_real.py:325: DeprecationWarning: Plural value must be an integer, got float
  return getattr(t, translation_function)(singular, plural, number)

The rationale for this warning is documented in BPO-28692: https://bugs.python.org/issue28692.
For filesizeformat, the filesize value is expected to be an int -- it fills %d string formatting placeholders. It was likely coerced to a float to ensure floating point division on Python 2. Python 3 always does floating point division, so coerce to an int instead of a float to fix the warning.
For intword, the number may contain a decimal component. In English, a decimal component makes the noun plural. A helper function, round_away_from_one(), was added to convert the float to an integer that is appropriate for ngettext().
https://code.djangoproject.com/ticket/27486

@jdufresne Thanks for this patch :+1: Should we document a new limitation (see comment), e.g. for Polish we changed

1,2 milionów  to 1,2 miliony, or
1,3 milionów to 1,3 miliony.

Should we document a new limitation

Works for me. I added a note to docs/ref/contrib/humanize.txt let me know if it is what you had in mind. Suggestions on wording are welcome.

I added release notes, and changed wording a bit.

Thanks for the docs complement. Maybe write a short notice to django-i18n before the commit?

@claudep Yes, sounds good :+1: . I've just joined this group added a notice.

MariaDB failure is not related with this PR.


Tag Releases

I should start tagging new releases and look into if it's possible to retroactively tag older versions.

We should be able to do this by specifying the commit's checksum...
Sample commit: 3a7bfbc
$ git tag -a v1.0.0 3a7bfbc

@vutran  is there a way to figure out the commit for a NPM _ackage ?

npm usually reads from the version specified from package.json.
If you've been using npm version <something> to bump, it should have automatically created git tags locally as well as a new commit that bumps the version in package.json. It makes it really easy to see where the tag points are in the commit history.

Cool will try that in future :) and will have to manually tag the older releases I guess.


Add precommit linting + get repo passing all checks

This adds the following pre-commit checks:

yapf
pyright
isort
inserting our license header into every python file
docformatter
pydocstyle
yamllint
yamlfmt

This is mostly a copy-pasted .pre-commit-config.yaml, pyproject.toml, and .yamllint.yaml  from the streaming repo. The main differences are:

I restricted the Python line width to 80 rather than 99. Since we expect users to fork this repo and add to the source code directly, figured it's safer to accommodate people who prefer short lines.

If you want 100-char lines but the existing code only uses 80, it doesn't hurt you. But if you want 80-char lines and the existing code uses 100, you have tons of hideous line wrapping that makes using our examples painful.

I added yamlfmt since our yamls were nowhere near compliant with yamllint and I didn't want to manually change them all. The one subtlety here is that it demands that list items not have the bullet indented relative to the parent (looks fine, just slightly different than streaming + composer yamls).

The rest of the code changes are just autoformatting + a few manual interventions to appease the linters.
Deferred to follow-up PRs:

Testing (these hooks only lint/format)
CI

Mostly looking for @bandish-shah to review this; repo just auto-requested everyone since you're now codeowners of your respective repos. That said, if you want to quickly look at the changes in your directory to make sure none of the semantics changed, that would be great--wouldn't be surprised if at least one bug got introduced.


API Final Project

What a journey.
I learned a lot and I really hope this meets expectations.
It was a pleasure meeting you in person.
Cheers,
Alan

Nice job!


Neuralynx Header Error

I am trying to import neuralynx data into python using neo. The code I'm running is:
import neo
dirname = (specify directory name)
reader = neo.io.NeuralynxIO(dirname = dirname, filename = (file name in directory))
The error I am getting is "OSError: No matching header open date/time for application {} version Cheetah. Please contact developers."

Hi,
did you try with the new matser ?

Hello,
Yea I updated the package and am using the latest version. I had previously tried these commands with the older version and it didn’t work, and so I saw another post on GitHub where the suggestion was to update the software. I am still having the same issue since updating.

From: Garcia Samuel @.>
Sent: Tuesday, March 22, 2022 4:01:46 AM
To: NeuralEnsemble/python-neo @.>
Cc: Demetrios Neophytou @.>; Author @.>
Subject: [EXTERNAL]Re: [NeuralEnsemble/python-neo] Neuralynx Header Error (Issue #1104)
Hi,
did you try with the new matser ?
—
Reply to this email directly, view it on GitHubhttps://github.com/NeuralEnsemble/python-neo/issues/1104#issuecomment-1074848677, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AXAZHHIPOINRTRCMPTNLOKTVBF46VANCNFSM5RIQIKFA.
You are receiving this because you authored the thread.Message ID: @.***>

Hi @dneophytou Can you provide a small test file, so I can check which Cheetah version and header layout you are using? You can provide a file here without making it publicly accessible https://amubox.univ-amu.fr/s/5Cd2BedrfJs9BQf

I have provided a file via that link. The file is titled with my username at the beginning of the file name. Please let me know if you need anything else.

Hi @dneophytou Can you provide a small test file, so I can check which Cheetah version and header layout you are using? You can provide a file here without making it publicly accessible https://amubox.univ-amu.fr/s/5Cd2BedrfJs9BQf

I've already uploaded a sample file, but I've just also uploaded a screenshot of the exact error message I am getting. Not sure if that'll be anymore help, but wanted to include it just in case there's an error that I'm missing.

I opened a PR here https://github.com/NeuralEnsemble/python-neo/pull/1107 that should fix this issue. Can you confirm?

Hi Julia, yes this worked! Thank you so much for your help!

I encountered the same issue today. This is a screenshot of the error.

Also, I noticed that there is a small bug in the line that raise the OSError. I think .format(an, av) is supposed to fill the first {} with an and the second {} with av. However, only the second string after the + calls .format(an, av) in the source code, which is why the error message is weird.


[stable/sonarqube] Update default service type to ClusterIP

Signed-off-by: Steve Hipwell<EMAIL_ADDRESS>What this PR does / why we need it:
Changes the default service type from LoadBalancer to ClusterIP as well as making the port handling between service and deployment easier to understand. Without this change this chart will create external resources by default if installed to a cluster where this is possible. This is a safer default value.
Which issue this PR fixes

fixes #17115

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

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

/assign @tsiddique

/assign @tsiddique

/retest

/ok-to-test
/assign

@rjkernick if you're happy with this could you flag it?

/lgtm


Check addresses supplied by users for conflict with existing routes

Fixes #437, #480.
Issues a warning but carries on.
Note that we are still not checking ranges passed to IPAM; I'm still considering whether that is a good idea.

Where does the warning pop up when using the proxy? In the CLI? The proxy logs?

Where does the warning pop up when using the proxy? In the CLI? The proxy logs?

Nowhere, it seems.

#1056 created for proxy issue


how to creat a arbitrary rnn layer

If I want to creat a rnn layer like this, and set the recurrence=5, does is easy to create this rnn operation by mxnet interface?

Where can I get the tutorial to construct the rnn layer?

mx.sym.RNN

@chenboheng It can be implemented by sharing the weight/bias symbols. The LSTM example may be helpful https://github.com/dmlc/mxnet/blob/master/example/rnn/lstm.py#L17-L113 .

@piiswrong mx.sym.RNN hasn't been implemented.

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


[FEATURE] JRTB-2: add telegram bot to the project

Create stub telegram-bot for the project.
Acceptance criteria:
stub answer via telegram-bot added to the project

https://github.com/vkostar/telegram_bot/pull/8


Teams for Outlook

hi.
hi guys.
i have a problem when a try to create a meeting with Teams.
throw me a notificacion about it: try later etc
im reinstall office, complements and teams but still problem
somebody help me please?
PD: my english is rusty

@diegarchovilla Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.

after formatting teams works perfectly but is not a rational option.
so i open a service ticket. bb2
tks


docs: update code block language

:tada: This PR is included in version 1.1.0 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


chore: Refactor mlflow inputs to use pydantic

Describe your changes
Refactor mlflow inputs to use pydantic. This is based on the pydantic usage in the sagemaker-notebooks module.
Also adding support for custom tags.
Checklist before requesting a review

[x] I updated CHANGELOG.MD with a description of my changes
[x] If the change was to a module, I ran the code validation script (scripts/validate.sh)
[x] If the change was to a module, I have added thorough tests
[x] If the change was to a module, I have added/updated the module's README.md
[x] If a module was added, I added a reference to the module to the repository's README.md
[x] I verified that my code deploys successfully using seedfarmer apply

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: AIOpsModuleTestsInfrastruct-doMLXEYsnmxr
Commit ID: 35129ef10e301f6bf721df487030ed8515e044b9
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AIOpsModuleTestsInfrastruct-doMLXEYsnmxr
Commit ID: 950a2abca436e16629c3287c313f71315527c743
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AIOpsModuleTestsInfrastruct-doMLXEYsnmxr
Commit ID: 400a4010ebc095317b1dce9c25f83f98afef8d61
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AIOpsModuleTestsInfrastruct-doMLXEYsnmxr
Commit ID: 64d956d8f3d94eab19826ad43137418bf5a31584
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AIOpsModuleTestsInfrastruct-doMLXEYsnmxr
Commit ID: 31411adfc1215af4d030623f89f3fb7081ad7d42
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AIOpsModuleTestsInfrastruct-doMLXEYsnmxr
Commit ID: ecdc983f4113873b8f86b1799401f713f0254b16
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AIOpsModuleTestsInfrastruct-doMLXEYsnmxr
Commit ID: 2ec63f8e979c5a149f2b6a836f2b1b9b47ae563d
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


AttributeError: 'RunTarget' object has no attribute 'sources'

Hi, I have an issue running the wrapper with meson: 0.47.1 and CLion
Traceback (most recent call last):
File "/usr/local/bin/meson", line 29, in
sys.exit(mesonmain.main())
File "/usr/local/lib/python3.6/dist-packages/mesonbuild/mesonmain.py", line 379, in main
return run(sys.argv[1:], launcher)
File "/usr/local/lib/python3.6/dist-packages/mesonbuild/mesonmain.py", line 287, in run
return mintro.run(remaining_args)
File "/usr/local/lib/python3.6/dist-packages/mesonbuild/mintro.py", line 235, in run
list_target_files(options.target_files, coredata, builddata)
File "/usr/local/lib/python3.6/dist-packages/mesonbuild/mintro.py", line 117, in list_target_files
sources = t.sources + t.extra_files
AttributeError: 'RunTarget' object has no attribute 'sources'
Which version of meson should I use ?

Sorry for the late answer.  (I was not on the watch list for the project) I think the issue should be solved by the newest version of meson-cmake-wrapper. If not then open a new issue. The next time I will respond faster. ;)


fix(auth): ignore case when comparing http headers

ensure authorization header is correctly removed, so no duplication of token when header is added down a few lines
adjusted similar cases for other headers

Checklist

[x] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

Http.HeaderNames.AUTHORIZATION has value "Authorization", while entry.getKey() has value "authorization". Thus this fix.


STRA for FAM

As a MISO
I want to understand the secuirty risks of an app so that I can
so that make sure it follows ministry approved best practices and will not expose security risks
Acceptance Criteria:
MISO has completed STRA to review
Definition of Done

[x] Ready to Demo in Sprint Review
[x] Peer Reviewed by 2 people on the team
[ ] STRA has been submitted

Intake form:
https://apps.nrs.gov.bc.ca/int/confluence/download/attachments/83564991/Security Threat and Risk Assessment - Intake Form - NRM.docx?version=2&modificationDate=1612912602000&api=v2
Checklist:
https://apps.nrs.gov.bc.ca/int/confluence/download/attachments/83564991/LITE Application and Software Assessment - NRM.docx?version=2&modificationDate=1615933962000&api=v2
STRA docs at https://apps.nrs.gov.bc.ca/int/confluence/x/Q4OKCQ

@gormless87 @ConradBoydElliottGustafson  should this be an epic? It's too large one ticket and we should break it down more. If we want to create a task I suggest something like "submit STRA for review" and link to all the dependencies (all the tickets and lead up to the submission). This way we can pull pieces of into the sprint and close the tickets as we complete the tasks.

It's from April 27, before we started working with Chris. I think this placeholder has been rendered redundant.
A new ticket "submit STRA for review" makes sense. I don't believe we have captured all the security work in tickets, so it's hard to retroactively pull them all into an epic at this point. I could be wrong!

@gormless87 let's close this ticket and write a new one based on comments above.


docs: remove duplicated state property in the example

Description
Remove duplicated property state in the example located in README.md file.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 93.402%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1454

Relevant Lines:
1484

💛  - Coveralls


Adjust tests according to grammar change

ConstOffsets now requires ImageGatherExtended
Int8 does not require Kernel anymore

See https://github.com/KhronosGroup/SPIRV-Headers/commit/3a9bf0b12a5b1a0a4996b0066f1451345c293546

Approved.
Failures were due to a SPIRV-Headers update that changed capability dependencies.
KhronosGroup/SPIRV-Headers@3a9bf0b
This caused failures in SPIRV-Tools validation tests, since they check some of those dependencies.
All future bot builds will fail until this patch goes in, so I'll push this to master now.


Enable join_notes to join more than 2 notes together, if possible.

Add a parameter, max_notes, to do so.
There are 2 options:

Join greedily as many notes as possible up to max_notes.
Choose a random number of notes up to max_notes and join that many.

This will allow join_notes to always reverse split_notes. Currently, split_notes is the only one that cannot be reverse with another (or the same) degradation.

Related to #29


Enable Comet aggregation (partition + final) as a whole

What is the problem the feature request solves?
Currently we treat partial and final aggregation operators separately during Comet planner. So theoretically you could get a Comet partial aggregation + Spark final aggregation.
The issue of this combination is that some aggregation functions in DataFusion may use unsigned integer types which cannot be properly mapped to Spark data type (e.g., Uint64 -> LongType). If we have a Comet partial aggregation + Spark final aggregation, it is possibly overflowing in runtime.
Actually I think only partial aggregation in Comet doesn't help too much. Because it means Comet shuffle is not enabled. Only partial aggregation directly on top of a Comet Scan will be transformed to Comet partial aggregation in such cases. I think it is very limited.
I think we can treat partial + final aggregation as a whole and enable/disalbe Comet aggregation (partition + final) together.
Describe the potential solution
No response
Additional context
No response

cc @huaxingao @sunchao

I re-think about this. Actually if there is any unsupported types like UInt64 from DataFusion partial aggregation state. Even we make partial and final aggregation as a whole, it doesn't solve the issue. It is because we need shuffle such unsupported arrays like UInt64. But we cannot arbitrarily assign an UInt64 array to Spark LongType like #216 did. Spark LongType is supported to be an Int64 array. Wronly binding an Uint64 array with LongType doesn't work with shuffle.


panicked at core\translate\emitter.rs:186:51:

Hi,
Too much panic occurs when running SQL.
An example:
limbo> create table t1 (a text);
limbo> insert into t1 values('a text for test');
limbo> insert into t1 values('a text for test2');
limbo> insert into t1 values('a text for test23');
limbo> insert into t1 values('a text for test234');
limbo> insert into t1 values('a text for test2345');
limbo> insert into t1 values('a text for test2345');
limbo> select a from t1 group by a order by a;
thread 'main' panicked at core\translate\emitter.rs:186:51:
called Option::unwrap() on a None value
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace
I couldn't debug with VsCode. The error message: Cargo has produced more than one matching compilation artifact.
rustup -V
rustup 1.27.1 (54dd3d00f 2024-04-24)
info: This is the version for the rustup toolchain manager, not the rustc compiler.
info: The currently active rustc version is rustc 1.83.0 (90b35a623 2024-11-26)
Os: Windows 11

hi, thanks
should be fixed here https://github.com/tursodatabase/limbo/pull/486

Thanks for the reply.
I can debug the codes I have written. But I can't debug for Limbo.
The error message: Cargo has produced more than one matching compilation artifact.
How can I debug for Limbo?


[v11] Recaptcha Enterprise integration

Description
Verify Recaptcha Enterprise in v11.  Since it relies on reflection, the code is a bit tricky. Follow-up to #13151

Verify retrieveRecaptchaToken reflection and functionality
Full end-to-end testing of Recaptcha Enterprise integration

Reproducing the issue
Click Initialize Recaptcha Config in the Sample App
Firebase SDK Version
11.0.0
Xcode Version
15.3
Installation Method
Swift Package Manager
Firebase Product(s)
Authentication
Targeted Platforms
iOS
Relevant Log Output
No response
If using Swift Package Manager, the project's Package.resolved

Expand Package.resolved snippet

Replace this line with the contents of your Package.resolved.

If using CocoaPods, the project's Podfile.lock

Expand Podfile.lock snippet

Replace this line with the contents of your Podfile.lock!

More investigation today on the following block of code:
Two issues:

Missing defaultAction in map leads to not linked failure:

(lldb) po action
FirebaseAuth.AuthRecaptchaAction.defaultAction
(lldb) po actionToStringMap
▿ 3 elements
  ▿ 0 : 2 elements
    - key : FirebaseAuth.AuthRecaptchaAction.signUpPassword
    - value : "signUpPassword"
  ▿ 1 : 2 elements
    - key : FirebaseAuth.AuthRecaptchaAction.signInWithPassword
    - value : "signInWithPassword"
  ▿ 2 : 2 elements
    - key : FirebaseAuth.AuthRecaptchaAction.getOobCode
    - value : "getOobCode"

Fails to do reflection:

po NSClassFromString("RecaptchaAction")  as? any RCAActionProtocol.Type
nil

Why does this cast fail?


Minor cleanup to uncorrelate function

Removed trailing spaces, lines pertaining to dhdl in uncorrelate function when using dE (lines 217-228), and removed line 774 which was calling estimatewithMBAR twice when plotting dF_t.

I tested decorrelation with dEs using my desmond parser. Will start a new pull request for it once this is merged?

Since this is cosmetic, not functional, I'm merging.
And yes, start a new pull request once I merge. (Though in principle you could have started it first and just updated by merging this one into your pull request once I merge it. It sounds complicated but it's actually simple.)


Question about connection problems

Neither the gameboy nor the Flipper recognise the connection. I've connected seemingly everything accordingly. If I'm wrong please correct me by the following pictures:

This is how it looks when it's attached to the Flipper:

Just to clear things up, I've used a multimeter to check if everything is connected correctly and judging by the multimeter it is. None of the channels of the cable are connected to multiple GPIO ports. I've tried out an older version of the Flipper App and the newest release, neither worked. Also, using the 5V doesn't make any difference. Any ideas?

Do not connect the 5v because you could kill the Gameboy or the flipper.

Thank you very much that did the trick.


Recipe on how to polyfill code intended for browser environment?

For a npm library I use tape for unit tests. But that library includes some other packages intended for browser environment only. This causes my tape unit tests to crash with errors like ReferenceError: navigator is not defined
What's your recommendation here? Is there anything I can polyfill my tape tests before this happens?

FYI if your code is already trying to distinguish browser vs node to be able to run in both, you can avoid the error with a typeof operator test instead of testing navigator directly:
if (typeof navigator !== 'undefined') {
  ...browser stuff...
} else {
  ...node stuff...
}

yup, thanks, already did that:
https://github.com/binarykitchen/videomail-client/blob/master/src/wrappers/visuals/replay.js#L10


disability support: PowerToys settings for numbers should have a number input along with number slider

Description of the new feature / enhancement
The number slider can be difficult to use for folks with certain disabilities. There should be an avenue for directly entering a number.
Scenario when this would be used?
Selecting numbers for settings.
Supporting information
N/A

For reference: https://github.com/microsoft/PowerToys/pull/21382

If anyone needs an idea of what I am talking about, try to change the number slider from 3 to 4 when you have a hand disability that makes it difficult to do tiny movements. I hope y'all can fix this. :/

What happens when you set focus to the slider and press arrow keys? (left/right)

@Jay-o-Way I swear I tried that earlier and it wouldn't work but I just tried again and it works as expected. I suppose this is a non-need now. Thanks!


Fix the walletDB wrong query results

duplicated results on timestamp (other?)
index file names in form (lenght)value-primaryKey

detect and fix duplicates at the level of the indexes


Raw nodes parsed as text

Hi,
I've encountered some issues using remark-iframe.
It seems that it is a bug from hast-util-raw.
I just created a PR to highlight the problem. A fix will follow soon.
See you soon :D
Edit: Fixed ?
Edit2: Oh ! And @vhf, he helped me a lot to find out the problem. Just to keep him in the loop.

Hi,
It's me, ... again ^^"
I just merge the two PR and i will close this issues by monday. To no pollute this repo.
💕

I was working on updating to the new version of parse5 and that was harder than expected. Sorry for the delay.
Thanks for all the work @arobase-che! 💪


Tab is renamed?

I believe the tab has a new name Recommended actions instead of Improvement actions.

This remark holds for other pages too, like this one

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

ID: 0f3bdcfa-e925-2280-0e37-e054bde0462f
Version Independent ID: 0f3bdcfa-e925-2280-0e37-e054bde0462f
Content: Microsoft Defender for Identity's security posture assessments
Content Source: ATPDocs/security-assessment.md
Service: microsoft-defender-for-identity
GitHub Login: @dcurwin
Microsoft Alias: dacurwin

Thank you for your comment. We'll investigate and get back to you.

@andrePKI  - Thanks for pointing that out. I updated the text and images on the MDI page.
As far as the other page you linked to, I suggest you open a GitHub issue there.


made air-to-vac conversion param when reading linelist

updated parameter names here and in EchelleCCFs to be consistent and in docstrings


[BUG]: FreqCacheEmbeddingBag._weight.ProcessGroup is initialized before _weight initialized

🐛 Describe the bug
When I init a DLRM with ParallelFreqAwareEmbeddingBag, the bug is reported as the following:

I believe that is because the ParallelFreqAwareEmbeddingBag init its _weight.ProcessGroup before the _weight is initialized.
After I swap above line with its next line, the traceback shows another error:

It looks like some api update issue.
Environment
No response

Thanks for the feedback. I will have a look.


Perf attribution by time period

this pr adds perf_attrib_periods which splits perf attrib calls into different time periods

Sorry, I just realized that the result is not changed where we do the slicing. As such, we don't need to push that logic to perf_attrib but can just slice the output data as we need it. As such, we should hold off on this PR and wait for the tear-sheet to be pushed where we can add this functionality.


Move LinearTask in System component

This PR moves the LinearTask in system component. Discussing with @S-Dafarra we decided to remove setKinDyn() and initialize from the LinearTask. We also decided to add isValid() to check if the content of getA() and getB() are valid.
This PR removes also the Task in TSID. Now all the tasks in IK and TSID inherits from System::LinearTask
cc @isorrentino

In which header is now LinearTask.h defined? I can't find it.

In which header is now LinearTask.h defined? I can't find it.

The CI seems to agree. :D

My fault. I forgot to commit the file. Done in  16e96b0dcb9a33589abb79fff3e2228bc8a3967e

brew, I :heart: you
Error: Failed to download resource "sqlite"
Download failed: https://homebrew.bintray.com/bottles/sqlite-3.34.1.catalina.bottle.tar.gz

I wonder whether it could make sense to check that every task is valid when solving the IK :thinking:

I wonder whether it could make sense to check that every task is valid when solving the IK 🤔

Yes I add a check


Secret not found in keyring

I use NixOS + Sway + LightDM. Gnome-keyring is running.

Can you send the contents of the log file here? Also, check out this wiki page for usage.

How do I do this?
Discordo also doesn't work on a full GNOME installation

I use NixOS + Sway + LightDM. Gnome-keyring is running.

Please provide more details about your environment and log files.

which log files?

which log files?

Discordo writes its log in the {XDG_CACHE_HOME}/discordo directory.

logs are empty under ~/.cache/discordo
also, echoing $XDG_CACHE_HOME returns null

Same error on Arch Linux with i3wm.
here is part of the log (anonymized, I guess)
 source=~/.cache/yay/discordo-git/src/discordo/cmd/application.go:40 msg="failed to login" err="The name is not activatable"
time=2024-09-30T20:42:27.270+02:00 level=INFO source=~/.cache/yay/discordo-git/src/discordo/cmd/state.go:49 msg="new HTTP request" method=GET path=/api/v9/guilds/123412341234/members/123412341234134
mime=2024-09-30T20:42:29.291+02:00 level=INFO source=~/.cache/yay/discordo-git/src/discordo/cmd/state.go:49 msg="new HTTP request" method=GET path=/api/v9/channels/123123412341234/messages
time=2024-09-30T22:37:00.675+02:00 level=ERROR source=~/.cache/yay/discordo-git/src/discordo/main.go:27 msg="failed to run" err=EOF

FYI it happens on gnome too

I might be wrong, but I think that configuring gnome-keryring fixed this for me


Robot does not appear in RViz

There's no .rviz config file in this repo, so the file path constructed here is invalid.

The above comment's issue is fixed but there are still problems with the MoveIt launch

Hi @danzimmerman
don't know if this has something to do with the rviz config, but your repo also has the issue mentioned here: https://github.com/UniversalRobots/Universal_Robots_ROS2_Gazebo_Simulation/issues/25

still problems with the MoveIt launch

Are these problems fixed or is this repo merged into the official one from UR?

Neither have happened yet, no.
I think that it's likely that it will be a while for some of these things to happen since it would make a lot more sense to build changes like this on top of some of the reorganization being considered for ur_description.
Keep in mind that the original UR Gazebo Simulation repo has not been officially released, so I don't imagine it will be a top priority for UR maintainers at the moment.
You might try the UR Ignition Simulation package to see if that's working.
I'm working on this as a user who wanted to get Gazebo up and running and share my progress so far, but I have pretty limited time at present to work on it.
Will let you know if I get MoveIt fixed but it looked like a bit of a complicated change. Doable, but easier and cleaner if ur_description gets reorganized.

Yeah no worries about not fixing it on your own. I understand that the reorganisation is worth the wait. I just wondered if it's already working or not. I hope UR releases the package soon.

Going to close this and open #5 to track MoveIt separately.


base64 encoding for icon

Signed-off-by: Jingfu Wang<EMAIL_ADDRESS>Please specify the area for this PR
What does does this PR do / why we need it:
Which issue(s) this PR fixes:
Fixes https://github.com/devfile/api/issues/450
PR acceptance criteria:

[x] Test (WIP)

[x] Documentation (WIP)

How to test changes / Special notes to the reviewer:
Build and deploy the registry to your own cluster then test the following endpoints:
/index?icon=base64
/index/sample?icon=base64
/index/all?icon=base64

@johnmcollier @maysunfaisal All comments are addressed, please review again, thanks.

@johnmcollier It's approved by Maysun, can you review it as well to see if it looks good to you?


buck2: unstable-2023-10-15->unstable-2023-01-15

Description of changes
just an update of buck2
thank you for the automated script!
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

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

1 package built:

buck2

Yeah I've got a private build of buck2 for now so I haven't updated the binary recently. Thanks!


[Event Request] codeunit 12 "Gen. Jnl.-Post Line".PostEmployee - OnAfterEmplLedgEntryInsert

Please add a new event (similar to customer/vendor event):
  local procedure PostEmployee(GenJnlLine: Record "Gen. Journal Line")
  begin
    ...
          EmployeeLedgerEntry.Insert(true);

          // Post detailed employee entries
          DtldLedgEntryInserted := PostDtldEmplLedgEntries(GenJnlLine, TempDtldCVLedgEntryBuf, EmployeePostingGr, true);

+         OnAfterEmployeeLedgEntryInsert(EmployeeLedgerEntry, GenJnlLine, DtldLedgEntryInserted);

          // Posting GL Entry
          if DtldLedgEntryInserted then
    ...
  end;

+ [IntegrationEvent(true, false)]
+ local procedure OnAfterEmployeeLedgEntryInsert(var EmployeeLedgerEntry: Record "Employee Ledger Entry"; GenJournalLine: Record "Gen. Journal Line"; var DtldLedgEntryInserted: Boolean)
+ begin
+ end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Unable to upgrade from 0.2.0 to 0.3.0

I encountered an issue while upgrading from active_hashcash version 0.2.0 to version 0.3.0. I am unable to execute the migrations successfully.
Steps to reproduce:

Create a new Rails project:

rails new hashcash-upgrade

Install active_hashcash gem version 0.3.0:

gem install active_hashcash -v 0.3.0

Attempt to run migrations:

bundle exec rails active_hashcash:install:migrations

Error message:
rails aborted!
Don't know how to build task 'active_hashcash:install:migrations' (See the list of available tasks with `rails --tasks`)
Did you mean?  active_storage:install:migrations
               action_mailbox:install:migrations

Thanks for the report. I forgot to include the db directory into the gemspec ... Curiously, there is no issue when the gem is installed via a local path or a Git URL. I published the version 0.3.1.

I really appreciate your help with this! I updated the gem to version 0.3.1 and the migrations went through without a problem. Thanks a lot for your support!


🤖 - Investigate memory leak

Description
https://blogs.windows.com/msedgedev/2021/12/09/debug-memory-leaks-detached-elements-tool-devtools/

there are still problems

I've managed to isolate it on Stackblitz, looks like it might be Angular animations bug:
https://github.com/angular/angular/issues/50533


Brochure launch

This is an issue to track the launch of the brochure site, a landing page where we can start to send people.
Here's a list of things that we need to accomplish before the site goes live:

[x] Text for the landing page: https://docs.google.com/document/d/1SHgzpQOcVonacH7mk9NUERKf-mpvxujQJ11tHlslFsg/edit#heading=h.mqupf8k02jq8
[x] A questionnaire for interested institutions: https://github.com/2i2c-org/2i2c-brochure/issues/1
[x] A page that describes the kinds of cloud infrastructure that we can provide: https://github.com/2i2c-org/2i2c-brochure/issues/2
[x] A "funders brief"? AKA, a page that describes the organization and its mission, and has a few connection points for organizations that may wish to support 2i2c.
[x] Finally: update this repository with the landing page docs

I think that at least the first and second items are important to have before the pedagogy meeting next week. @colliand what do you think?

@colliand what do you think about the current state of the materials at 2i2c.org?  We've got some explanatory text up there, but what can be improved? In addition, are we OK with "just" a contact form? Or do we want to link to something more like a survey for now?

This now exists!


Support for "integer" type

Great that this project was started!
Just started testing and found that type "integer" is not supported. However, specs define two numeric types: "number" and "integer": https://json-schema.org/understanding-json-schema/reference/numeric.html
Would you consider implementing integer as a supported type? If so, I'd be happy to draft a PR.

Hm, I didn't realize that JSON Schema had integer. It's not in the core spec, only in the validation spec. I assume that this is become JSON Schema doesn't consider it an "instance type". A PR would be appreciated.


[Task]: Fix unclaimed amounts showing in available balance

Task description
Reported by C3PO-all in iota since 2016#4792 on discord https://discord.com/channels/397872799483428865/998627792717627443/1001019281561096242 the balance seems to be including unclaimed transactions.
Requirements
Fix the balance so that it does not include pending transactions.
Creation checklist

[ ] I have assigned this task to the correct people
[X] I have added the most appropriate labels
[X] I have linked the correct milestone and/or project

Cannot reproduce that, also during my development time, i never encountered it really
I think the error here was that he already claimed it, but i know that we had a bug that we didn't correctly update claimable transactions,when they were claimed.


Don't expose port in Dockerfile

While the line EXPOSE 8080 of the Dockerfile can be useful in some cases, there are also reasons against it.
The only benefit I see is that it is not necessary to add -p 8080 with docker run if one wants to keep the default port and enable external access to the service.
But a port that is once exposed can't be unexposed.
Example: If one adds an nginx reverse-proxy Docker container before the microbin service, then one would probably want that only the nginx service is reachable from the outside (e.g. on ports 80 and 443). So one would use -p 80 -p 443 for nginx but omit the -p option with microbin. If one then runs docker ps, one can see that the port 8080 of microbin is still exposed. Docker ignores some firewalls such as ufw resulting in external access directly to microbin.
There are some workarounds for this such as

binding the exposed port to localhost, e.g. <IP_ADDRESS>:8080
adding additional firewall rules
using a script like docker-copyedit to create a new Docker image without any exposed ports

I think it is cleaner to omit the EXPOSE 8080 by default and if really needed, one can add it in a derived Dockerfile.

Sorry, I mixed up the usage of publish and expose. Ports can't be published in a Dockerfile. External access is only allowed to published ports.


Add CI workflow to test client

It's not clear what the issue is. The karma test runner seems to run all the tests successfully from the log shown, but then npm "crashes".
> ng test --browsers=ChromeHeadlessCi --no-watch --code-coverage

- Generating browser application bundles (phase: setup)...
Compiling @angular/core : es2015 as esm2015
Compiling @angular/core/testing : es2015 as esm2015
Compiling @angular/common : es2015 as esm2015
Compiling @angular/platform-browser : es2015 as esm2015
Compiling @angular/platform-browser-dynamic : es2015 as esm2015
Compiling @angular/platform-browser/testing : es2015 as esm2015
Compiling @angular/compiler/testing : es2015 as esm2015
Compiling @angular/platform-browser-dynamic/testing : es2015 as esm2015
23 07 2021 03:01:14.324:INFO [karma-server]: Karma v6.3.4 server started at http://localhost:9876/
23 07 2021 03:01:14.326:INFO [launcher]: Launching browsers ChromeHeadlessCi with concurrency unlimited
23 07 2021 03:01:14.328:INFO [launcher]: Starting browser ChromeHeadless
✔ Browser application bundle generation complete.
23 07 2021 03:01:18.288:INFO [Chrome Headless 92.0.4512.0 (Linux x86_64)]: Connected on socket zMv9RNxjk1TBrbuPAAAB with id 50069727
Chrome Headless 92.0.4512.0 (Linux x86_64): Executed 0 of 0 SUCCESS (0 secs / 0 secs)
Chrome Headless 92.0.4512.0 (Linux x86_64): Executed 0 of 0 SUCCESS (0.009 secs / 0 secs)
TOTAL: 0 SUCCESS
✔ Browser application bundle generation complete.

=============================== Coverage summary ===============================
Statements   : 100% ( 3/3 )
Branches     : 100% ( 0/0 )
Functions    : 100% ( 0/0 )
Lines        : 100% ( 3/3 )
================================================================================
✔ Browser application bundle generation complete.
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>test:ci: `ng test --browsers=ChromeHeadlessCi --no-watch --code-coverage`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>test:ci script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/runner/.npm/_logs/2021-07-23T03_01_20_131Z-debug.log
Error: Process completed with exit code 1.


Logical operators evaluate left-hand side twice

For context, I am using lua-in-js for unit testing redis scripts in node. It mostly works great for this, thanks! However I noticed that mocks were getting called multiple times in some scripts when I didn't think they should be.
Turns out that some logical operators appear to evaluate the LHS twice. This makes sense as it looks as transpilation for them is implemented via simple string substitution: https://github.com/teoxoy/lua-in-js/blob/3d4f9e2865e979d2eb8ef205f666c0852b283eab/src/parser.ts#L306
This should probably be updated to assign the result of left to a local variable first, assuming you have some mechanism to maintain hygiene and avoid unintended shadowing? I'm no Lua expert but I think it is not supposed to eval LHS twice. Please correct me if I'm understanding something wrong, I really just glanced over and made a lot of assumptions!

It does look like the LHS will get evaluated twice.

Fixed with ca94a68d03c201c30b6da0fd05f376449bcba88d (part of v2.2.4).


Inconsistent {{@type}} expansion in macros

If the macro is inside a module, the @type is referring to the container module instead of the type where the macro is expanded.
Running the docs tool with the following code exhibits the differences. But the issue affects also the generated code.
I was expecting to always be the type were the macro is executed (since the otherone can be explicitly mentioned)
module XXX
  macro m
    # This will be XXX (not ideal) `{{@type}}`
    def foo
      {{@type}}
    end
  end
end

macro m
  # This will be Foo (ideal) `{{@type}}`
  def bar
    {{@type}}
  end
end

class Foo
  XXX.m
  m
end

pp! Foo.new.foo # => XXX
pp! Foo.new.bar # => Foo

Extracted from #8994

Right now you can do:
UInt8.slice(1, 2, 3)

That generates:
Slice(UInt8)...

The macro uses {{@type}} to get the UInt8 type.
Essentially, it seems there's no way to fix this unless we have a way to refer to where the macro is called, and where the macro is defined.

Right now self inside macros gives an error because it has no meaning. We could make self work for this purpose: it's the type where the macro is defined. On the other hand, @type is the type where the macro is expanded.
When the macro happens inside a method, like:
class Foo
  def foo
    {{@type}}
  end
end

then @type and self will be the same. In the examples provided by @bcardiff @type will work as he expects it, which I think is what's expected, and self will return XXX inside XXX.m but will give an error saying "there's no self" inside the top-level macro m... just like what happens with regular methods.
However, this is a breaking change.


Currently unable to use quotes within variable definitions

A potential fix would be to have variable names need to be a parameter.
It might be a helpful change, could eventually allow using the inside of the element to write in default values...


Create minimum_pizza.cpp

This is not a spam repository,
Don't spam with single codes
If you want to contribute, take up an issue, solve maximum codes that you can and then then make a pull request


bugfix on response withBody StreamInterface

This is supposed to be a bug fix for #2856 but OP should instead emit the guzzle response directly instead.


Mobile authenIcarian is limited when authenticating using the Azure.Identity methods

[Enter feedback here]

Document Details
Can more mobile friendly authentication methods be provided in the documentation?
A majority of the Azure.Identity methods used in the docs are restricted from use on mobile.
A goodvexample is ClientSecretCredential method which should be allowed on mobile platform development.
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 918d059f-8e92-166e-e98d-f89a097c67f8
Version Independent ID: 2a956f03-1dbe-086a-9b07-0ae1217c50fb
Content: BlobContainersOperationsExtensions.SetLegalHold(IBlobContainersOperations, String, String, String, IList<String>, Nullable<Boolean>) Method (Microsoft.Azure.Management.Storage) - Azure for .NET Developers
Content Source: xml/Microsoft.Azure.Management.Storage/BlobContainersOperationsExtensions.xml
Service: storage
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

@norton287 Redirecting this to relevant team to update the documentation accordingly.


Content Searching Errors When Enumerating Shares

Describe the bug
When using the --content flag, the tool errors out and throws Python exceptions.
To Reproduce
cme smb <IP> -u <uname> -p <pass> --spider <share> --pattern <pattern> --content
Expected behavior
It searches through file contents without errors and finds what exists based on the command
Screenshots

Crackmapexec info

OS: Kali
Version of CME = 5.1.1dev
Installed from apt or using latest release?

Installed via pipx instructions on install wiki

I found a solution, run your command with "--continue-on-success" at the end.
Example: crackmapexec smb -u users.txt -p passwords.txt --shares <IP_ADDRESS> --continue-on-success

Done


Implement Headers and MutableHeaders

Welcome to Codecov :tada:
Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


MQTT username Password

Where can I set the MWTT username and password as I cannot connect to the home assistnt mqtt server.

Works 1a! Thanks . is there any plan to add config file? Hardcoded seems a bit over kill for "normal users"

Hard coded ...works ! Thanks !


Add translation: Catalan

Provide the name and the two-letter code of the new language
Catalan, ca
Steps to add the translation

Add a folder named like the two-letter code of the language you want to add under public/_locales/
Copy the file public/_locales/en/messages.json to your new folder
Translate all values of that file (please leave the keys untouched!) and create a pull request

I can add this translation

Awesome, thank you for this suggestion! I assigned this issue to you.


Remove StyleConfig

We decided not to save the configuration in the database.
https://trello.com/c/pDvJ8BbS

If this table is empty on production and not used anywhere else.. this looks fine to me.

The table is empty in production:
hound-production::SILVER=> select * from style_configs;
(No rows)

Also is Owner table used anywhere? I can see it gets filled but is it used anywhere?

But we can do Owner table in another PR or issue

@arunagw Since the Owner is pretty coupled with the StyleConfig - I opted to delete the owner here aswell.

Sounds good. Let's wait for more review here before merging this in master.

Yep, Owner was introduced as a precursor to organization level config. See #586. I don't think it was ever used for anything else, but perhaps someone is using it for statistics? @gylaz or @salbertson ? If not, is there meaningful data we can get from it that we aren't?

Owner is not being used for stats. Why are we removing the owner code? Is it causing issues?

I opted to delete it since it's dormant code, we're not using it for anything.
Will we? Or does it only make sense to delete the style-config?

Teo Ljungberg

On 30 okt. 2015, at 18:30, Scott Albertson<EMAIL_ADDRESS>wrote:
Owner is not being used for stats. Why are we removing the owner code? Is it causing issues?
—
Reply to this email directly or view it on GitHub.

Owner was added to back fill information about GitHub users/orgs. We will be adding org-level configuration soon-ish, so this is probably good to keep. As for StyleConfig, I believe it can be removed.

@gylaz Changed in 1f79b79

:+1:


EventEmitter Not Found

Using the latest code with just a console.log(EventEmitter) yields an error (EventEmitter is undefined). I reverted to version 4.1.0 (because it was on the jsfiddle example) which ran smoothly. Seems like the latest version isn't working though.

How are you loading the script, can you share your example?

I downloaded EventEmitter.min.js from the master branch and load it as follows:
<head>
  <meta charset="utf-8">
  <meta http-equiv="X-UA-Compatible" content="IE=edge,chrome=1">
  <title>Test Page</title>
  <meta name="description" content="">
  <meta name="viewport" content="width=device-width">
  <script src="lib/svg.js"></script>
  <script src="lib/EventEmitter.min.js"></script>
  <script src="http://localhost:12346/livereload.js"></script>
</head>
<body>
  Test
  <script>
    console.log(EventEmitter);     // <--- Problem
    var bus = new EventEmitter();
    console.log(bus);
  </script>
</body>

The console just states EventEmitter is not defined.
However; when I replace the EventEmitter code using the older code from http://jsfiddle.net/Wolfy87/qXQu9/, the above code works without a problem.

Hm, are you sure the script has loaded by that point? It should block until done, but I'd just confirm that. This works fine for me: https://jsfiddle.net/pje48erg/

The fiddle initially failed on my system due to explicit mime type checking, but I modified it slightly (using rawgit.com instead of github), which worked.
I'm using this within an Electron app, which still fails even after the modification, despite assuring the DOM/script is fully loaded before logging the EventEmitter. It seems like an environment issue, not necessarily your library. I still don't see how the 4.1.0 version works while the latest doesn't, but I haven't looked at the diff in depth yet.
Since Electron can leverage node's EventEmitter, my project doesn't necessitate the use of your EventEmitter, but my goal was to reuse the code in a browser version of the app. I realize this is an edge case. If you're interested in getting this to work within Electron, I'm happy to keep troubleshooting. Otherwise, I'm just as content closing this issue.
Thanks for your help and the time you put into this code base.

I came across this issue with other libraries too. I'll spare the deep details. Electron was allocating the EventEmitter to modules.export as opposed to the global/window.
If anyone else is doing some mad science with this library and Electron as I have been, the simple-yet-not-immediately-obvious fix is to add the following immediately after including the EventEmitter.
EventEmitter = module.exports;


ENH: viz tweaks

bring back ica component numbers in plot_sources_evoked

cc @jaeilepp @Eric89GXL

ping can we merge?

+1


Standardise our code formatting using automated tools in pre-commit, e.g., pyupgrade, black, flake8

Description of problem and opportunity to address it

Context to understand the problem
When working on code, I find it very beneficial to have a standardised format as I find it easier to quickly parse and understand the content. I think a standardised format also helps reviewing PRs as we can be sure the diffs presented are actual changes, not just stylistic choices. I also like the automate these kinds of formatting so that I can write code without worrying that I'm conforming to the style or not (thought I've found over the years, my natural writing style conforms more and more).
We already use pre-commit to auto-format our markdown and YAML files, the latter being super important given that most of this repository is helm configuration.
I would like to propose adding pyupgrade, black and flake8 to our pre-commit config for autoformatting our Python files as well.
Proposed solution
I would like to add the below dependencies to our existing pre-commit config:
- repo: https://github.com/asottile/pyupgrade
  rev: v2.31.0
  hooks:
  - id: pyupgrade
- repo: https://github.com/psf/black
  rev: 21.12b0
  hooks:
  - id: black
- repo: https://github.com/pycqa/flake8
  rev: 4.0.1
  hooks:
  - id: flake8
    args: ["--ignore=E501,W503"]

What's the value and who would benefit
I think this would allow us to maintain tidier code that is easier to work on as a team. Having it applied by pre-commit means that there's no additional overhead to engineers, they push their code as-is and pre-commit will format it.
@consideRatio and I would greatly benefit from implementing this standardised formatting before we take a deep dive into the deployer code as we work towards https://github.com/2i2c-org/infrastructure/issues/879 (and related issues).
We only have five open PRs at the minute and only one of them touches any Python files (and only two commented lines) so there will be a very small chance of introducing merge conflicts if we take this action soon.
Implementation guide and constraints
No response
Updates and ongoing work

[ ] Gain approval from the team to implement this
[ ] Open a PR updating the pre-commit config with the suggested tools and any automatically applied formatting from pre-commit
[ ] Document the use of these hooks in our team-compass, state that they have the team's blessing to be installed on any repo in the org a team member sees fit

I'm very happy about having both pyupgrade and black in a pre-commit config!
I also appreciate flake8 and are not strongly opinionated on the choice of config for it. When using flake8 with black, there are some notes in black's documentation. I think E501 is irrelevant if you configure flake8 to have a max-line-length of 88 as recommended there for compliance with black.
Practically, I suggest we start with a .flake8 config file like:
[flake8]
max-line-length = 88
extend-ignore = E203

After trying it out on this repo, we fix misc errors and add exceptions in the flake8 config if we feel its too strict to fix them.

I've found that following standards in formatting that the rest of the community follows as well is a great way to reduce cognitive overhead associated with code formatting. Those three hooks seem like a nice lightweight set to shoot for. +1 from me!
One thought - this could be a short PR to our team compass that just says "we use these 3 hooks and any repository that wants to use them has our blessing". That way you could "request review" from the team and make it clearer if people approve.

This sounds great! I'd only appreciate if we didn't split imports one per line (as some of the JupyterHub repos do). I'm hoping to open an issue in team-compass there about that soon...

This sounds great! I'd only appreciate if we didn't split imports one per line (as some of the JupyterHub repos do). I'm hoping to open an issue in team-compass there about that soon...

As far as I'm aware (and do please correct me if I'm wrong!), I don't believe any of the tools I'm recommending do this. I do use isort on my personal projects, but I thought that was a bit much for here.

I don't believe any of the tools I'm recommending do this

Oh actually, I might be wrong about this

@sgibson91 I don't think you are, but black will enforce line lengths and sometimes split up imports to multiple lines, but not make them dedicated and separate import statements.
For example, I think...
# example 1: fine with black
from module import func_1, func_2, func_3, func_4, func_5, func_6, func_7, func_8

# example 2: not fine with black
from module import func_1, func_2, func_3, func_4, func_5, func_6, func_7, func_8, func_9

# example 2 above reformats to:
from module import (
    func_1,
    func_2,
    func_3,
    func_4,
    func_5,
    func_6,
    func_7,
    func_8,
    func_9,
)

Just a note that I do not have strong opinions here one way or another

Big +1 on using these tools!
About the splitting imports, I do not have strong feelings either...


Add initial BYOB-based SLSA-generator

Summary

Adds the initial workflow for the Bring-Your-Own-Builder-based SLSA generator.
Currently the builder is hosted on my Github user, because the builder is under development. I am uploading it here to get it integrated early in the development process of the gradle builder. The BYOB-based generator should not disrupt any other workflows since it it needs to be manually triggered to run.
Release Note

Documentation

Fyi. I'm working with Adam on this.

Just ran another build which succeeded: https://github.com/AdamKorcz/sigstore-java/actions/runs/4609029248

What's the testing plan here? Are we running this on a tag seeing what happens?

Yes. Let's run it paralel to the existing build workflow.


Not Found page should be rendered for a non-existent article

/articles/wrong-id
/articles/wrong-id/edit
Should 404 and show a proper message.

This has been fixed.


ACC-270.6(Level A)Kyma-dashboard_Combo Box UI cannot be activated using keyboard

1.1 Internal Message Details
Internal Message:
Test Plan:	ACC SAP Kyma runtime 2024
Message Priority:	High
Affected Check Point:	ACC-270.6
Affected UI:	Combo Box
Short text:	ACC-270.6(Level A)Kyma-dashboard_Combo Box UI cannot be activated using keyboard
1.2 Application Details
Application Name	Kyma-dashboard
Application URL:	https://canary.cockpit.btp.int.sap/cockpit#/globalaccount/7304343e-1ce0-412b-a955-5948bab9d2c8/subaccount/e0ec1134-2770-40a2-910b-2b8e32ec8f60/subaccountoverview
User:	SSO
Technology:	SAP UI5
UI5 Version:	1.124.1
1.3 Test Environment Details:
GitHub	https://github.com/kyma-project/busola/issues
Web browser:	Google Chrome - Version 127.0.6533.73 (Official Build) (64-bit)
Operating System:	Windows 11 Enterprise Version 22H2
Screen Reader:	JAWS 2024.2405.83 ILM 64 bit for Windows
2. Screen Flow
•	Login into the application using the URL provided.
•	URL: https://canary.cockpit.btp.int.sap/cockpit#/globalaccount/7304343e-1ce0-412b-a955-5948bab9d2c8/subaccount/e0ec1134-2770-40a2-910b-2b8e32ec8f60/subaccountoverview.
•	Activate ‘Console URL:Link to dashboard’ in overview page.
•	Activate ‘Namespaces overview’ left navigation section.
•	Activate ‘Deployments’ item.
•	Activate ‘Create’ button.
3. Issue
Synopsis: UI cannot be activated using keyboard.
Observed Behavior:
Last list item in all unit of measurement combo box within ‘Deployments’ page is not keyboard accessible in forward navigation.
Expected Behavior:
Each & every UI, must be accessible using the keyboard.
Keyboard is the prerequisite for support of assistive technologies to enable users with disabilities to read and operate software applications, hence all the UIs must be accessible with keyboard.

Kindly refer to attached error and issue processing document for more details.
Regards,
Huda

ACC-270.6(Level A)Kyma-dashboard_Combo Box UI cannot be activated using keyboard.docx
Issue processing-09.2021 (1).docx


fix(bundler): fix order of webview2 installer args in nsis bundle

Funnily enough it must be /silent /install = /install /silent doesn't work and results in a crash
What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Docs
[ ] New Binding issue #___
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change?

[ ] Yes, and the changes were approved in issue #___
[x] No

Checklist

[ ] When resolving issues, they are referenced in the PR's title (e.g fix: remove a typo, closes #___, #___)
[ ] A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.
[ ] I have added a convincing reason for adding this feature, if necessary

Other information

Yeah just found out because i wanted to test the download itself and got a 0xWhatever error -> https://github.com/MicrosoftEdge/WebView2Feedback/issues/555#issuecomment-818214035 🤣


Typo in save_model code

In src/utils/utils.py, line 119 - 121:
def save_model(mode, path):
    torch.save(model.state_dict(), path)
    return

If my understanding is correct, the first parameter here should be "model" instead of "mode".
This function is called in src/runner/SingleRunner.py, line 158:
utils.save_model(self.model, self.args.model_path)

Thank you! The typo has been fixed.

fixed

fixed