v1

What now?
As we've reached the point where the project's codebase has grown to be more than just "few lines to wrap Simply Plural" and leaning towards being a full-fledged site meant to be deployed at scale, I think it's time to take a break for few minutes and look through the current implementation.
Node.js is a stable and performant backbone for the backend, however it's design may also be a struggle in the future. I have for a little while had thoughts about using Go instead.
Node.js's architecture is single threaded event-pool based. A lot of tasks that Plurali does or would benfit from conurrency, which is Go's strength and Node just isn't there - I hear you say "b.but Node has a woker" - yes, and Go has goroutines, which unlike Node aren't a sad excuse of a workaround for concurrency. This can be amazing for WebSockets - at the moment there is a - and I'm very aware of it - poor implementation of "Watchers" (Client for Simply Plural's WS which watches for data changes and invalidates/refetches the data as needed). It's poor for a reasons - the lack of capability from Simply Plural WS to watch over multiple users, which is understandable given the fact that it's mainly meant for the mobile app, but also a reason why the backend needs to open a WS connection for each user. That on it's own is already heavy on the network infrastructure, and certainly for Node. With go(routines), each connection can be on it's own thread, which (obviously) wouldn't solve the network issue but would lay off a lot of stress from the main thread, therefore better throughput and all that jazz and buzzwords.
Node and any (known to me and worth considering) JS runtimes are also interpreted, because JS is after all, an interpreted language (please do not mistake compiling for transpiling, we love to call it that but isn't the same). Go is compiled, therefore taking advantage of barebones perfomance. At the same time, Go was made with multi-core CPUs in mind, which together with abovementioned concurrency would be a great performance improvement.
The unfortunate disadvantages of Go is the ecosystem, loss of shared data-types with the frontend codebase and there isn't quite a replacement for Prisma that works just as well, though database is also a factor that may change and therefore Prisma not be a concern either way.
Even if we stay with Node.js, I would like to migrate to a full-fledged framework like Adonis.js. As the project grows with complexity, so does the code and while the performance of Fastify may be intriguing, re-inventing the wheel while it's been invented so many times before is an anti-pattern very well known in the JS community (staring into the soulds of frontend framework creators). Validation, authentication, caching, services (the IOC container being namely one of Adonis' outstanding features), all that good jazz... sure, I can do all that, but at the cost of mostly having to repeat myself (for example User: the type itself, transforming it for response data (transformUser function and everything related to it), validating requests (xxx schemas with fluent-json-schema), etc etc. And of course keeping all those points in sync - fun one indeed. Adonis in particular wouldn't even compromise on performance that much, as it's basically "toe-to-toe" with Fastify, and while the move may mean rewriting half the codebase, it comes at a benefit of cleaner codebase with a lot of available functionalities ready to be used.
S3 - Right now I use DigitalOcean Spaces, they're fully S3-compatible, provide a CDN for the storage and the pricing is not gonna bankrupt me (at least not yet). As for "lock-in", given the fact that a lot of storage providers are also S3-compatible, it doesn't seem to be that big of an issue. Not to mention that with Adonis, we get the Drive (and Attachment) abstraction which solves this concern very efficienty.
MongoDB - It has gotten us to way different numbers than the initial start with MySQL and later PostgreSQL, which makes total sense given the fundamental differences between these databases, however, ScyllaDB, fully Cassandra&DynamoDB-compatible database would throw us out the planet with the difference in throughput and reads/writes and the ability to learn&optimize itself (and much more, just too tired to name all of it). As it's cassandra/dynamo compatible, the available ecosystem widens even further than it's own and moving to Scylla may end up being the first change to happen.
And Redis - in short, it's fast. Plurali at the moment only uses the very basic key/value functionality of it and therefore I don't think any other technology is needed. With that said, drop-in forks like Dragonfly or KeyDB might be worth checking out for production. No code changes should be required as far as I am aware.


Automatic schema creation from basic settings

Is your feature request related to a problem? Please describe.
When users provide default dataset settings they should also see a detailed graph connected based on these settings.
Describe the solution you'd like
If a user defines feature X as the default anchor and feature Y and Z as the default links then they should see this reflected not only on the overview network but also on the detail network. E.g. the default schema in that case should be feature X connected directly to feature Y and feature Z with appropriate relationships.
Additional context
#24 should be done before this since it is needed to infer appropriate relationships.

implemented in the improvements branch


fix: java importing crd fails

Fixes https://github.com/awslabs/cdk8s/issues/234
Replace / with .
Tested locally with the CRD example I'm writing for Java.
Signed-off-by: campionfellin<EMAIL_ADDRESS>By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

Will write tests for the base import method tomorrow (or later this week) and update the PR.

Will write tests for the base import method tomorrow (or later this week) and update the PR.

Okay let’s merge this in the meantime and add a test in a subsequent PR


EDI network streaming

the software is really great. What is missing for analyzing a multiplex is the ability of receiving data by EDI network streaming (ETI over network - ETSI ETS 300 799, tcp or udp).
Would be great if this could be implemented in future.
Can be tested with ODR-DabMux.

This is definitely out of scope.

@sronline please use odr-dabmod

If you are on Linux or macOS, you can create FIFO (named pipe) and use it as output for odr-dabmod and raw file input for AbracaDABra. This way you can avoid creating huge IQ files.

I'm using odr-dabmux and odr-dabmod on linux and AbracaDABra on Windows. Is there a way to get the data live to AbracaDABra?

Reopening this issue for the discussion about possible workaround although the previous statement is still valid, direct support in application is out of scope.
According to this post it seems that might be possible workaround that could work on Linux and macOS. I am not sure about Windows, maybe MinGW64 could help. Note that currently the proposed command sequence does not work because application does not recognize the server as valid RTL-TCP server but this is should be an easy modification. But I do not know how to test it, are those EDI streams available to public so that I can use some for testing?
The other option could be to use named pipe (FIFO), that might already work on Linux but I have not tried.

If you can change that the server is recognized as valid RTL-TCP server, I can test it. Perhaps adding a new Device named TCP? Using named pipes with Windows seems not to be easy.

Named pipes on Win works different way. The change is already pushed, if you compile from source, you can try. New device is not introduced, application accepts server that does not have expected identification and uses it as RTL-TCP with unkonwn tuner.

To test the change, I'm trying to build the windows version from source, but it fails at linking.
Do you have a hint for me?
I'm using Windows 10, Qt 6.7.3. External libs (mpg123-1.32.7) are compiled with mingw1120_64 and copied to folder ../AbracaDABra-libs
The missing symbol __imp_PathIsRelativeW of libmpg123 seems to be releated to Shlwapi.
Set PATH=C:\Qt\Tools\mingw1120_64\bin;%PATH%
Set PATH=C:\Qt\Tools\CMake_64\bin;%PATH%
Set PATH=C:\Qt\6.7.3\mingw_64\bin;%PATH%
cmake -G "MinGW Makefiles" -DQCUSTOMPLOT=OFF -DUSE_SYSTEM_QCUSTOMPLOT=ON -DUSE_PORTAUDIO=OFF -DUSE_SYSTEM_MPG123=OFF -DUSE_SYSTEM_PORTAUDIO=OFF ../
mingw32-make

Building the apps under Windows is always painful :-( Your steps seem to be ok, but I do not do it this way. I use MinGW64 environment to build all libs and to install them is dedicated folders. And also mpg123 I compiled last time is mpg123-1.29.3, maybe this could be a problem.
I build libs following way.

Create AbracaDABra-libs/src folder
Unpack or checkout all libs sources to this directory
I have following script located in  AbracaDABra-libs/src folder:

#! /bin/bash

SRC_DIR=$PWD
INSTALL_DIR=$SRC_DIR/../
JX=-j4

# libusb
echo "============= libusb ============"
cd $SRC_DIR/libusb-1.0.26/
make clean
./configure --prefix=$INSTALL_DIR
make $JX && make install

# rtlsdr
echo "============ rtlsdr ============"
cd $SRC_DIR/rtlsdr
[ -d build ] && rm -Rf build
mkdir build
cd build
PKG_CONFIG_PATH=$PKG_CONFIG_PATH:$INSTALL_DIR/lib/pkgconfig/ cmake  -G "MSYS Makefiles" .. -DCMAKE_INSTALL_PREFIX=$INSTALL_DIR
make $JX && make install

# faad
echo "============ faad2 ============"
cd $SRC_DIR/faad2
make clean
./bootstrap
./configure --prefix=$INSTALL_DIR
make $JX && make install

# fdk-aac
echo "============ fdk-aac ============"
cd $SRC_DIR/fdk-aac/
[ -d build ] && rm -Rf build
mkdir build
cd build
PKG_CONFIG_PATH=$PKG_CONFIG_PATH:$INSTALL_DIR/lib/pkgconfig/ cmake  -G "MSYS Makefiles" .. -DCMAKE_INSTALL_PREFIX=$INSTALL_DIR
make $JX && make install

# mpg123
echo "============ mpg123 ============"
cd $SRC_DIR/mpg123-1.29.3/
make clean
./configure --prefix=$INSTALL_DIR
make $JX && make install

# portaudio
echo "============ portaudio ============"
cd $SRC_DIR/portaudio/
make clean
./configure --prefix=$INSTALL_DIR
make $JX && make install

# airspy
echo "============ airspy ============"
cd $SRC_DIR/airspyone_host
[ -d build ] && rm -Rf build
mkdir build
cd build
PKG_CONFIG_PATH=$PKG_CONFIG_PATH:$INSTALL_DIR/lib/pkgconfig/ cmake  -G "MSYS Makefiles" .. -DCMAKE_INSTALL_PREFIX=$INSTALL_DIR
make $JX && make install

# missging libs
cp $INSTALL_DIR/bin/libairspy*.a $INSTALL_DIR/lib/

# SoapySDR
echo "============ SoapySDR ============"
cd $SRC_DIR/SoapySDR
[ -d build ] && rm -Rf build
mkdir build
cd build
PKG_CONFIG_PATH=$PKG_CONFIG_PATH:$INSTALL_DIR/lib/pkgconfig/ cmake  -G "MSYS Makefiles" .. -DCMAKE_INSTALL_PREFIX=$INSTALL_DIR
make $JX && make install

# SoapyAirspy
echo "====== SoapyAirspy ======"
cd $SRC_DIR/SoapyAirspy
[ -d build ] && rm -Rf build
mkdir build
cd build
PKG_CONFIG_PATH=$PKG_CONFIG_PATH:$INSTALL_DIR/lib/pkgconfig/ cmake  -G "MSYS Makefiles" .. -DCMAKE_INSTALL_PREFIX=$INSTALL_DIR
make $JX && make install

# SoapyRTLSDR
# not compatible with Oldenburger's RTLSDR
echo "====== SoapyRTLSDR ======"
cd $SRC_DIR/SoapyRTLSDR
[ -d build ] && rm -Rf build
mkdir build
cd build
PKG_CONFIG_PATH=$PKG_CONFIG_PATH:$INSTALL_DIR/lib/pkgconfig/ cmake  -G "MSYS Makefiles" .. -DCMAKE_INSTALL_PREFIX=$INSTALL_DIR
make $JX && make install

########################################################################
# Done
cd $SRC_DIR

I open MinGW64 shell, navigate to AbracaDABra-libs/src folder and run ./build.sh

NOTE: you need to align the script for your source directories (libUSB and mpg123 have version in name, etc.) and remove libs you do not need (like SoapySDR). Good luck!

Yes, it is really painful to build under Windows.
Thank you very very much for your build instructions, I was now able to build the source on my way with the following modification:
`diff --git a/CMakeLists.txt b/CMakeLists.txt
index e1ce70a..eac4291 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -107,6 +107,8 @@ if(WIN32 OR (APPLE AND APPLE_APP_BUNDLE))
set(USE_SYSTEM_AIRSPY       OFF)
set(USE_SYSTEM_SOAPYSDR     OFF)
set(USE_SYSTEM_QCUSTOMPLOT  OFF)
+

set(CMAKE_CXX_STANDARD_LIBRARIES -lShlwapi)
endif()`

This allowed linkage of libmpg123.
To stream the data I adapted the command from rundfunkforum:
odr-dabmod mux2stream.ini | nc -l <IP_ADDRESS> 1234
The TCP connection to odr-dabmod can be established :-)
The channels are listed, but I am not able to get a stable transmission.
I'm trying to figure out if that it is caused by commands sent to the TCP socket which are received via nc or a blocking issue. odr-dabmod detects errors and restarts the modulator.

When I connect with telnet the data runs fine and odr-dabmod does not run into an error. In telnet when I type chars I can see them in in the console of odr-dabmod.
I tried with socat und unidirectional mode instead of nc:
odr-dabmod mux2stream.ini | socat -U -d -d -d TCP-LISTEN:1234,BIND=<IP_ADDRESS> -
But the issue is the same.
Perhaps switching from TCP to UDP could help.

I will check the issue with missing lib on my side and fix it int the repo, thanks for reporting it.
What is the data source for odr-dabmod? The point is that RTL-TCP input is designed in the way that the server is "clock master". Application just reads what is available in the socket. If it cannot provide proper timing then it will not work. For example if odr-dabmod reads data from ETI file and generates U8 IQ output that is passed to nc, then this configuration will not work because anytime AbracaDABra tries to read input, it will get new data until the file is read by odr-dabmod. I am not sure if the explanation is clear. There is RAW file input designed for input from file, all other inputs are designed for live stream with defined sampling rate.

I have just updated all libraries for Windows and I do not need to link with Shlwapi. Maybe it is W11 related, I still build with W10.

I did a new setup of ODR in a virtual machine to send the data to AbracaDABra.
There it works now with the rtl tcp device :-) It seems on the other system it was a cpu performance or network issue.
Source of odr-dabmod is a live essemble created by ODR-DabMux, which sends its data by edi to odr-dabmod.
The the big advantage of an EDI network streaming input in AbracaDABra would be that udp multicast streams could be used. In this case odr-dabmod would not be needed and simply EDI of ODR-DabMux could be used.

Yes, it coudl be used but it would require significant effort to implement support for EDI/ETI and major rework of the application that is designed to receive DAB signal from antenna. Decoding from EDI/ETI is different approach. Since ODR-Dab tools can be used to support this rare use-case I still think it is out of scope of AbracaDABra.
I will keep this issue open for some time as a discussion topic in case tehere is anythign I can do to improve support for stream from ODR-DabMux.

@KejPi if you need, i can provide you different official EDI/tcp for testing.


charts version upgrade of stable/grafana

change stable/grafana to version 2.2.5

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: rawlingsj
If they are not already assigned, you can assign the PR to them by writing /assign @rawlingsj in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Hi @alphabettispaghetti. Thanks for your PR.
I'm waiting for a jenkins-x member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

change stable/grafana to version 2.2.5

/ok-to-test


fix: properly catch typeguard error in trace tree

Fixes this internal sentry issue https://weights-biases.sentry.io/issues/4443133957/?environment=prod&project=6620563&query=!error.type:WeaveReponsesDifferException+!error.type:WeaveTypescriptServiceError+!error.type:WeavePerformanceRegressionException+release:latest+stack.stack_level:0&referrer=issue-stream&sort=freq&statsPeriod=7d&stream_index=8.
We were catching the wrong error type in our new trace tree _setattr_with_typeguard(). This catches the correct one.

https://wandb.atlassian.net/browse/WB-15391


Make v2_meta default to true for templates

Closes: #

[ ] Docs
[ ] Tests

Testing Strategy:

@chaance stacks are done in https://github.com/remix-run/indie-stack/pull/203, https://github.com/remix-run/blues-stack/pull/163 & https://github.com/remix-run/grunge-stack/pull/126
I'll create a PR for updating the examples asap


prevent unnecessary consul restarts on puppet runs

if you are running consul RPC on a custom port, the startup script can now see that the service is running

@mdelagrange It would be awesome if there are some tests to go along with this since there are a lot of moving parts to this.

@aj-jester yep that all sounds good.  i'll get to work sharpish.

@aj-jester updated per your recommendations.  thanks for the feedback!

@mdelagrange This looks stupendous! Nice work dude :100:
:bullettrain_side: LGTM

Fixed.

Can we hold up on merging this? I just want to make sure we don't commit just to revert all this back because of https://github.com/solarkennedy/puppet-consul/issues/174

To me it sounds like @pforman was waiting for this to be merged before he does his PR. Regardless of the solution to #174 it's going to need this logic to work properly on a custom host or port.

We want to understand how consul behaves if leave_on_terminate is set to true and TERM signal is sent with consul leave executing right before the TERM.

I don't have any issue with this, and it won't greatly affect anything I've been talking about.  As @mdelagrange says, the RPC info is going to need to be there anyway.  If it's merged, I'll do my TERM-based PR from the new HEAD.  If we do them at the same time, it would be a definite conflict, as we're changing the very same lines in the sysv script.
The only thing it would change for me is the addition of the ${RPC_ADDR}, which I'd preserve.

ok this should be good to merge :+1:

Thanks! I'll work on porting this feature to the other init systems.

@solarkennedy oh thats awesome! It was on my list but now I can cross it off :stuck_out_tongue:
Thanks.


issue - wont execute

Traceback (most recent call last):
File "faitagram.py", line 254, in
main()
File "faitagram.py", line 41, in main
if os.path.exists(wordlist) == False:
File "/usr/lib/python2.7/genericpath.py", line 26, in exists
os.stat(path)
TypeError: coercing to Unicode: need string or buffer, NoneType found

Is it fixed?
On Sun, Jul 8, 2018 at 8:15 AM Justin Chang<EMAIL_ADDRESS>wrote:

Closed #91 https://github.com/Juniorn1003/Faitagram/issues/91.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Juniorn1003/Faitagram/issues/91#event-1721878058, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AefBMPY6hWyOQluWY8uD3ycyHfuFm2c3ks5uEiIjgaJpZM4U8YAX
.

--
Electronic Privacy Notice: This Email, and ANY attachments, contains
information THAT IS covered by Electronic Communications Privacy Laws, and
is also confidential, proprietary in nature and NOT subject to public
disclosure.  If you are NOT the intended recipient, please be advised that
you are LEGALLY prohibited from retaining, using, copying, distributing, or
otherwise disclosing this information in part or in whole in ANY manner.
Instead, please reply to the sender that you have received this
communication in error, and then IMMEDIATELY delete it. Thank you in
advance for your cooperation.


Need input onChange function

Please I need this function [onChange] in [input] to be handle in my project, I download the project and try to modify it with my project it takes a lot and a lot of change
so, please help me...

Please I need this function [onChange] in [input] to be handle in my project, I download the project and try to modify it with my project it takes a lot and a lot of change
so, please help me...

Now they have a widget property called handleTextInputChange, just use it on your <Widget ... />
<Widget
  title={title}
  senderPlaceHolder={'Type a message...'}
  handleNewUserMessage={onClientMessage}
  handleTextInputChange={console.log}
/>

will be called on every change on client input


Auditor v2: rule configuration validation, generic rule skeleton

@carise is this ready to submit?

no, not yet.
On Tue, Dec 12, 2017 at 10:47 AM, mirons<EMAIL_ADDRESS>wrote:

@carise https://github.com/carise is this ready to submit?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/forseti-security/pull/753#issuecomment-351150017,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAA4BBVaYhrbWPzuxiDXxmbTutD7bpNTks5s_spIgaJpZM4QL9zF
.


Update README.md - lowered "c" for consistency

The client class has as its static method on_message to use it in plugins. You can use 'Client' or 'client' however you want. This is not a mistake.


refactor: Simplify cron scaler code

The current cron scaler code creates a cron object and starts it (which creates a goroutine and a lot of data to manage a cron engine) just so that it can find the next activation time. It also re-parses the cron strings on each iteration. This update only parses on creation of the trigger, then uses the cron schedule object directly to get the next activation time.
Checklist

[X] I have verified that my change is according to the deprecations & breaking changes policy
[X] Changelog has been updated and is aligned with our changelog requirements
[X] Commits are signed with Developer Certificate of Origin (DCO - learn more)

/run-e2e cron

/run-e2e cron

and now DCO xD

@joelsmith  Do you mind also refactoring it with the new declarative scaler configuration?  https://github.com/kedacore/keda/issues/5797

@joelsmith Do you mind also refactoring it with the new declarative scaler configuration? #5797

I have refactored it in #6098


[bug]: Combo Box TypeError: undefined is not iterable (cannot read property Symbol(Symbol.iterator))

Describe the bug
I copy pasted the sample combo box code in my react app the popover trigger appear. when click open this error occurred in the console.
Affected component/components
Command Combobox
How to reproduce

install popover and command
copy paste the sample code on the website

Codesandbox/StackBlitz link
No response
Logs
TypeError: undefined is not iterable (cannot read property Symbol(Symbol.iterator))
The above error occurred in the <ForwardRef> component:

System Info
Chrome

Before submitting

[X] I've made research efforts and searched the documentation
[X] I've searched for existing issues

This is a problem with command group of  cmdk library. The command list component seems to wrok fine

In the example, you need to add the value and onValueChange properties to the <Command /> component.
See more details here: https://github.com/pacocoursey/cmdk?tab=readme-ov-file#command-cmdk-root

Due To the version update in cmdk now you have to wrap the  around  and in the component replace data-[disabled] to data-[disabled="true"]

@reckek I do not believe this is the core issue, but I do believe the issue is upstream
Using this cmdk example: https://github.com/pacocoursey/cmdk?tab=readme-ov-file#use-inside-popover
You can reproduce this error by swapping List for Group

Opened an issue upstream https://github.com/pacocoursey/cmdk/issues/284

I found that the problem was with the cmdk package version 1.0.0. So I removed the package using npm r cmdk and then reinstalled a lower version using npm i<EMAIL_ADDRESS>and it worked Alhamdulillah (all praise to Allah).

Hi, I have been trying to find information about the error I have, but it seems to be something... strange.
When I click on the combobox, Popover try to display and the following error is generated:

FormField component (copy of Form example):
<FormField
    control={control}
    name='nationality'
    render={({ field }) => (
      <FormItem className='flex flex-col'>
        <FormLabel>Nationality</FormLabel>
        <Popover>
          <PopoverTrigger asChild>
            <FormControl>
              <Button
                variant='outline'
                role='combobox'
                className={cn(
                  'w-full justify-between',
                  !(field.value && field.value.nationality) &&
                    'text-muted-foreground'
                )}
                disabled={!nationalities}
              >
                {field.value && field.value.nationality
                  ? nationalities?.find(
                      (nationality) =>
                        nationality.nationality ===
                        field.value.nationality
                    )?.nationality
                  : 'Select an option'}
                <ChevronsUpDown className='ml-2 h-4 w-4 shrink-0 opacity-50' />
              </Button>
            </FormControl>
          </PopoverTrigger>
          <PopoverContent className='p-0'>
            <Command>
              <CommandInput placeholder='Search Nationality...' />
              <CommandList>
                <CommandEmpty>
                  Not found.
                </CommandEmpty>
                <CommandGroup>
                  {nationalities?.map(
                    (nationality) => (
                      <CommandItem
                        value={nationality.nationality}
                        key={nationality.id}
                        onSelect={(value) =>
                          console.log('Selected', value) //Test
                        }
                      >
                        <Check
                          className={cn(
                            'mr-2 h-4 w-4',
                            nationality.nationality ===
                              field.value.nationality
                              ? 'opacity-100'
                              : 'opacity-0'
                          )}
                        />
                        {nationality.nationality}
                      </CommandItem>
                    )
                  )}
                </CommandGroup>
              </CommandList>
            </Command>
          </PopoverContent>
        </Popover>
        <FormMessage />
      </FormItem>
    )}
/>

And this is the CommandItem component:
const CommandItem = React.forwardRef<
  React.ElementRef<typeof CommandPrimitive.Item>,
  React.ComponentPropsWithoutRef<typeof CommandPrimitive.Item>
>(({ className, ...props }, ref) => (
  <CommandPrimitive.Item
    ref={ref}
    className={cn(
      'relative flex cursor-default select-none items-center rounded-sm px-2 py-1.5 text-sm outline-none data-[disabled="true"]:pointer-events-none data-[selected="true"]:bg-accent data-[selected="true"]:text-accent-foreground data-[disabled="true"]:opacity-50',
      className
    )}
    {...props}
  />
))

The version of cmdk is 1.0.0
Any suggest how to fix 'o' prop type?

Hi, I have been trying to find information about the error I have, but it seems to be something... strange.
When I click on the combobox, Popover try to display and the following error is generated:

FormField component (copy of Form example):
<FormField
    control={control}
    name='nationality'
    render={({ field }) => (
      <FormItem className='flex flex-col'>
        <FormLabel>Nationality</FormLabel>
        <Popover>
          <PopoverTrigger asChild>
            <FormControl>
              <Button
                variant='outline'
                role='combobox'
                className={cn(
                  'w-full justify-between',
                  !(field.value && field.value.nationality) &&
                    'text-muted-foreground'
                )}
                disabled={!nationalities}
              >
                {field.value && field.value.nationality
                  ? nationalities?.find(
                      (nationality) =>
                        nationality.nationality ===
                        field.value.nationality
                    )?.nationality
                  : 'Select an option'}
                <ChevronsUpDown className='ml-2 h-4 w-4 shrink-0 opacity-50' />
              </Button>
            </FormControl>
          </PopoverTrigger>
          <PopoverContent className='p-0'>
            <Command>
              <CommandInput placeholder='Search Nationality...' />
              <CommandList>
                <CommandEmpty>
                  Not found.
                </CommandEmpty>
                <CommandGroup>
                  {nationalities?.map(
                    (nationality) => (
                      <CommandItem
                        value={nationality.nationality}
                        key={nationality.id}
                        onSelect={(value) =>
                          console.log('Selected', value) //Test
                        }
                      >
                        <Check
                          className={cn(
                            'mr-2 h-4 w-4',
                            nationality.nationality ===
                              field.value.nationality
                              ? 'opacity-100'
                              : 'opacity-0'
                          )}
                        />
                        {nationality.nationality}
                      </CommandItem>
                    )
                  )}
                </CommandGroup>
              </CommandList>
            </Command>
          </PopoverContent>
        </Popover>
        <FormMessage />
      </FormItem>
    )}
/>

And this is the CommandItem component:
const CommandItem = React.forwardRef<
  React.ElementRef<typeof CommandPrimitive.Item>,
  React.ComponentPropsWithoutRef<typeof CommandPrimitive.Item>
>(({ className, ...props }, ref) => (
  <CommandPrimitive.Item
    ref={ref}
    className={cn(
      'relative flex cursor-default select-none items-center rounded-sm px-2 py-1.5 text-sm outline-none data-[disabled="true"]:pointer-events-none data-[selected="true"]:bg-accent data-[selected="true"]:text-accent-foreground data-[disabled="true"]:opacity-50',
      className
    )}
    {...props}
  />
))

The version of cmdk is 1.0.0
Any suggest how to fix 'o' prop type?

@Kelvin274 I am having the same issue, trying to fix it

@Kelvin274 Strangely, if I separate the file into an isolated combo-box.tsx component, the error goes away.


[TASK] Add TYPO3 backend users to introduction environment

Fixes: #94

Tried it out, it works great


Extra [ displayed in define keybinding widget under Portuguese (Brazil)

From @Tekbr
https://github.com/Microsoft/vscode/issues/23685#issuecomment-303145958
https://github.com/Microsoft/vscode/issues/26945#issuecomment-303145991
As for me pushing ´, the problem still continues.
Steps:
1- Open Defines keybinding
2 - Press ´.
3- Click Define keybinding.
4- Press [.
Now it shows oem_6[.

Thanks @alexandrudima for the quick fix. 😄 👏 👏


Getting error due to bootstrap5 while installing package

I am using latest yii2 version 2.0.45. I am getting error while installing package
Problem 1
- yiisoft/yii2-bootstrap 2.0.0 requires bower-asset/bootstrap 3.2.* | 3.1.* -> found bower-asset/bootstrap[v3.1.0, v3.1.1, v3.2.0] but the package is fixed to v5.3.1 (lock file version) by a partial update and that version does not match. Make sure you list it as an argument for the update command.
- yiisoft/yii2-bootstrap[2.0.1, ..., 2.0.8] require bower-asset/bootstrap 3.3.* | 3.2.* | 3.1.* -> found bower-asset/bootstrap[v3.1.0, ..., v3.3.7] but the package is fixed to v5.3.1 (lock file version) by a partial update and that version does not match. Make sure you list it as an argument for the update command.
- yiisoft/yii2-bootstrap[2.0.9, ..., 2.0.11] require bower-asset/bootstrap 3.4.* | 3.3.* | 3.2.* | 3.1.* -> found bower-asset/bootstrap[v3.1.0, ..., v3.4.1] but the package is fixed to v5.3.1 (lock file version) by a partial update and that version does not match. Make sure you list it as an argument for the update command.
- lajax/yii2-translate-manager[1.8.0, ..., 1.8.1] require yiisoft/yii2-bootstrap ~2.0.0 -> satisfiable by yiisoft/yii2-bootstrap[2.0.0, ..., 2.0.11].
- Root composer.json requires lajax/yii2-translate-manager ^1.8 -> satisfiable by lajax/yii2-translate-manager[1.8.0, 1.8.1].
Looks package is using old bootstrap version.
Any body have idea to fix this issue?
FYI: @lajax @moltam and others

Hi gs63875, I made a fork and changed everything to BS5.
Let me know if you are still interested and I will check to make the fork public.

Yes please. I don’t mind and thank you for you support.
сб, 15 черв. 2024 р. о 09:18 gb5256 @.***> пише:

Hi gs63875 https://github.com/gs63875, I made a fork and changed
everything to BS5.
Let me know if you are still interested and I will check to make the fork
public.
—
Reply to this email directly, view it on GitHub
https://github.com/lajax/yii2-translate-manager/issues/155#issuecomment-2169155606,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAPZ3OYYILBT54FQERSDR53ZHPMDPAVCNFSM6AAAAABJLNVZXKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRZGE2TKNRQGY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Hi gs63875, I made a fork and changed everything to BS5. Let me know if you are still interested and I will check to make the fork public.

Please make it public


Reflect multi-version compatibility in source file names

This change implements the suggestion mentioned in #156 to rename the
source files from ol3 to simply ol so as to reflect the
compatibility from OpenLayers 3 up to and including OpenLayers 6.
The example file as well as the documentation have been updated to be
consistent with the filename generalisation.
Please review these changes! I've checked that the example works, but it's still possible that I might have missed something as part of the renaming process.
If you'd like anything changed or want this commit split into multiple commits or whatever, please don't hesitate to let me know!

hmm, I guess we would have to consider this a breaking change because the files are no longer available at the old paths 🤔

Hrm, true. Hadn't thought about that aspect.  If you don't consider the patch to be merge-worthy, I'm totally ok with it :-)  I saw the suggestion and thought that it might be helpful.
As I was thinking about your comment I had a thought: is anyone still using OpenLayers 2?    If not, then maybe a breaking change isn't such a problem and one can support OL3+ moving forward. Just an idea...

Come to think of it: surely if a user depends upon this filename then they will be using a very old version of OpenLayers anyway and can thus specify version 0.4.0 of sidebar-v2 in order to maintain the expected functionality.  What do you think?


Segfault when starting on Ubuntu 18.10

Hi,
I just recently updated to Ubuntu 18.10. Now I'm getting a segfault error when trying to start the installed version (via the .deb release package or snap version).
The error in my console (via journalctl -f).
Okt 24 12:09:09 JonasT440p kernel: fromscratch[15987]: segfault at e6e0c0 ip<PHONE_NUMBER>e6e0c0 sp 00007fffb86776d8 error 15 in fromscratch[200000+1603000]
I don't think this is an error in your code. But maybe a version update of some of the dependencies might work? I will have a look soon if building fromscratch myself might solve the problem.

Just to check that i'm reading this right: It happens in both the .deb and the snap version?

Can you let me know if this still happens in 1.4.3?


[Initial POC] Adding IntelliSense support for debug console

Description:
Implement initial POC with the initial design https://github.com/ballerina-platform/ballerina-lang/issues/32406

Closing with https://github.com/ballerina-platform/ballerina-lang/pull/33152


Website Redesign: "Learn" Page 1- Top- UI

User Story
AS a New Orcahome user I WANT to see an easy to read webpage when I navigate to the Orcasound "Learn" page SO THAT I can easily learn about Orcas.
Description
The intent of this story is to implement the UI finalized design from the Orcasound UX/UI team for the website redesign of the "Learn" Page. This Issue covers ONLY the top picture and links on the page (see image below).
NOTES:
-This is the UI only, links can be added at a later iteration
-Header/Footer already completed on different pull request #14
-Use Material UI for the styling
Acceptance Criteria
GIVEN I open the Orcasound Learn page WHEN I look at the page THEN I see a UI that matches the attached UX/UI design.
https://www.figma.com/file/YdvbAjHETokV2x1LBHaElj/Orcasound-Website-Redesign-2021?node-id=298%3A290

Hi  @hgordon6634 ! I'd like to contribute to this issue . I have good experience working in open source and converting UI designs to frontend code in Next.js.

Hey @ShwetKhatri2001 ! We would love your help!!
However, these issues aren't QUITE ready to go (I just created them and was getting feedback from our developer volunteers about them and need to make some tweaks and split them up more). They should be ready to work on around Wednesday of next week.
For now, why don't you hop on our Slack and introduce yourself in the Homepage-Redesign channel :) That way I can let you know when the issues are ready to go! https://orcasound.slack.com/archives/C025AV698S0

@Daggerhub - This is good to go, I will be adding two more user stories (issues) for the Learn page. This is just for the top part.

Resolved by #73


will not start

when i click on the loader to run it will not start up how to fix?

Check if your antivirus software is getting in the way. You might have to add exceptions.
The game in DirectX11 mode must be running already.
If it doesn't work, try running as Administrator.

it was working fine befor i updated the game today i always ran it in admin i just thout i would ask to see why it happend

oh its working now my pc reset all settings ty for help


Fix golint warnings for daemon/execdriver/*

Addresses: #14756
Signed-off-by: Qiang Huang<EMAIL_ADDRESS>

All in all I think most of the doc strings could use some more detail as things like "Foo stores information about foos" isn't all that helpful.

I agree with @cpuguy83

@cpuguy83 @LK4D4 updated comments as you suggested and added some extra comments for some other functions/structs.
Feel free to point them out if you think they need more details, I just thought most of them are quite self explained, which is why they have no docs in the first place.

@cpuguy83 @LK4D4 updated, thanks.

@hqhq Great work on this! This is a fantastic start.
I did notice we have slightly different doc strings between native/lxc on stuff that should basically be the same as they are just implementing some interface.
I wonder if it's good enough to put the doc details on the Driver methods in the interface definition and just mention in the actual drivers this implements the exec driver <foo> interface, and any pertinent details required for that particular implementation.

@cpuguy83 Updated, now all common docs are in interface, it's easier to maintain, what do you think now?

LGTM

@hqhq Could you rebase it so you can add the daemon/execdriver package and sub-packages to hack/make/validate-lint :wink: .

@vdemeester Thanks, updated.

ah, driver_unsupported*.go needs some lint too :blush:.

Fixed, should'v used golint xxx/*.go :smile:

needs rebase sorry

@jfrazelle done.

LGTM

@hqhq Thanks!
LGTM


Provide mapping between requests and response

The library should provide some way to map between request and response, such that given a request model, it's possible to retrieve its corresponding response model.
One way would be to add a field to requests, but that would introduce a runtime dependency between request and response modules, which seems like a good idea to avoid.
Another way is to provide a function that traverses the module tree, this seems easy to implement but a bit ugly ...

I've been tinkering a bit with a use case that needs this, for dynamically loading the right model based on incoming requests.
What I'm landing in would be the optimal design here, is code generation to produce a kio.schema.index module, containing a mapping from (api_key, api_version) to modules and names. The signature of the loader function would like this.
def load_response_schema(api_key: i16, version: i16) -> type[ResponsePayload]: ...
def load_request_schema(api_key: i16, version: i16) -> type[RequestPayload]: ...
# Data and header schemas do not have API keys, so would be exposed by name instead.
def load_header_schema(name: str, version: i16) -> type[Entity]: ...
def load_data_schema(name: str, version: i16) -> type[Entity]: ...

In addition to loading the main schema type, it's likely also sensible to expose a mechanism for loading the respective modules:
def load_payload_module(api_key: i16, version: i16) -> type[ModuleType]: ...
def load_entity_module(name: str, version: i16, entity_type: EntityType) -> type[ModuleType]: ...

For reference, I did an experiment with dynamically importing the full schema (all entity modules): this takes between 2-3 seconds.
https://gist.github.com/antonagestam/86f96f3dccee8c8ff7711f9647c5783a

Draft implementation here: https://github.com/Aiven-Open/kio/pull/150/. Missing code generation and tests.


[stable/redmine] adds configurable liveness and readiness probes

Signed-off-by: Dmitrii Ermakov<EMAIL_ADDRESS>What this PR does / why we need it:
PR allows configuring liveness and readiness probes for stable/redmine.
I had a problem:
liveness initialDelaySeconds: 300 was not enough for my environment to load so container continuously restarted.
I had to set the delay to 600 seconds for my environment.
I hope adding configurable timings might help in case of slow environment.
Default values for probes remain the same.
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

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

The DCO is failing, so remember to sign your commits. Thanks!

My request fails "version bump". Could you give me advice if I should change version from 13.0.1 to 13.0.2 or 13.1.0?

My request fails "version bump". Could you give me advice if I should change version from 13.0.1 to 13.0.2 or 13.1.0?

I'd say a minor since you are implementing a new feature. Thanks!

@carrodher, please, start the merging process.

There are some conflicts, happy to add the proper tags once resolved.

/ok-to-test

@carrodher thank you.
I hope I fixed the conflicts.

/lgtm


form can't submit when outside the Modal

Bug report 🐞
form works when the form is inside the Modal component but the Modal.Action loses its default styling.
When wrapping the Modal inside the form, the submit button doesn't work
Version & Environment
"@zeit-ui/react": "^1.6.0-canary.2",
"preact": "^10.1.0",
OS: Windows 10 64bits
Google Chrome Version 81.0.4044.138 (Official Build) (64-bit)
Expection
Expecting the form to be submitted when form wraps modal
Actual results (or Errors)
it doesn't submit

Here is an online example, Molda with Form seem to work correctly, can you provide more information to help me reproduce the issue? (Or  a online sample?)

The form attribute worked like a charm. I really appreciate your support buddy, best I've ever got. Thank you for your patience ! 🙏


GitHub: New repository

Required information:

Details to your GitHub repository:
repo name : Simple-data-exchanger-backend
repo name : Simple-data-exchanger-frontend
repo will be private and please fork repo from https://github.com/catenax-ng/tx-managed-simple-data-exchanger-backend & https://github.com/catenax-ng/tx-managed-simple-data-exchanger-frontend respectively

Details to GitHub team to grant access: product-dft

validation / approved by: MehranRoshandel
Thanks in advance.

Hi @hzierer
I am checking with team and confirm.

@hzierer  Please create empty private repos, don't need to fork from  above mention repository.

@hzierer any update on above request.

@hzierer any update on above request. Please create the above repos so that we can start pushing code.

Hi @adityagajbhiye9,
looking at the comment history, I'm not sure if I get your request.
What are you going to implement in this repo?
If it is related to the simple-data-exchanger, you should do that in the existing fork.
If your contributions include company IP or anything similar, that would require it to be private, you should think about doing it outside the catenax-ng organization.
Could you please clarify what this repo will be used for?

Sure, I am discussing with team.

Hi @adityagajbhiye9, any update on this?

Please close this issue. We have change the course of action.

ok thanks for the update


win32: Make the main item of the installer readonly

This is related to k-takata/vim-win32-installer#3.
Current win32 installer can uncheck "Vim executables and runtime files"
item, but it is nonsense.  Make the item readonly.

Codecov Report

Merging #2106 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2106      +/-   ##
==========================================
+ Coverage   73.97%   73.98%   +<.01%
==========================================
  Files          90       90
  Lines      131716   131716
  Branches    28941    30643    +1702
==========================================
+ Hits        97436    97444       +8
+ Misses      34276    34249      -27
- Partials        4       23      +19

Impacted Files
Coverage Δ

src/version.c
79.29% <0%> (-1.02%)
:arrow_down:

src/libvterm/src/screen.c
71.96% <0%> (-0.38%)
:arrow_down:

src/ex_cmds.c
77.71% <0%> (-0.09%)
:arrow_down:

src/gui.c
46.34% <0%> (-0.06%)
:arrow_down:

src/eval.c
80.72% <0%> (-0.05%)
:arrow_down:

src/os_unix.c
53.22% <0%> (+0.04%)
:arrow_up:

src/gui_gtk_x11.c
47.86% <0%> (+0.2%)
:arrow_up:

src/window.c
81.54% <0%> (+0.46%)
:arrow_up:

src/gui_beval.c
44.47% <0%> (+4.6%)
:arrow_up:

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 4033c55...07fc0f3. Read the comment docs.


feature: Always accept identity observable factors

This enables simultaneous measurement of qubit-wise commuting Pauli
words.
Issue #, if available:
Description of changes:
Testing done:
Merge Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.
General

[x] I have read the CONTRIBUTING doc
[x] I used the commit message format described in CONTRIBUTING
[x] I have updated any necessary documentation, including READMEs and API docs (if appropriate)

Tests

[x] I have added tests that prove my fix is effective or that my feature works (if appropriate)
[x] I have checked that my tests are not configured for a specific region or account (if appropriate)

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

Looks good to me. I'd like to test it out on my end before approving though.


nullable field isn't nullable if also restricted to a list of choices

Describe the bug
Fields with nullable=True aren't considered nullable when also using choices=...
To Reproduce
Create a model with the following:
from enum import Enum
from ormar import Model, Integer, String
import sqlalchemy
import databases

URL = "postgresql://..."  # url to postgres db

metadata = sqlalchemy.MetaData()
database = database.Database(URL)

class MyChoices(Enum):
    A = "A"
    B = "B"

class Item(Model):
    class Meta:
        tablename="items"
        metadata = metadata
        database = database

    id = Integer(primary_key=True)
    my_field = String(max_length=1, nullable=True, choices=list(MyChoices))

app = FastAPI()
app.state.database = database

@app.on_event("startup")
async def startup() -> None:
    database_ = app.state.database
    if not database_.is_connected:
        await database_.connect()

@app.on_event("shutdown")
async def shutdown() -> None:
    database_ = app.state.database
    if database_.is_connected:
        await database_.disconnect()

@app.post("/upload/")
async def upload(item: Item):
    await Item.save()

Upon trying a POST request to that endpoint, if my_field is set to None, it will fail with a ValueError:
{"detail": [{"loc": ["body", 0, "__root__"], "msg": "my_field: 'None' not in allowed choices set: ['A', 'B']", "type": "value_error"}]}

Expected behavior
The model accepts "A", "B", or None for my_field, because it has a set of valid choices and is nullable. It can be worked around by adding None to the enum, but that does not seem natural to me:
  class MyChoices(Enum):
      A = "A"
      B = "B"
+     NONE = None

Versions

Database backend used: postgres
Python version: 3.9.7
ormar version: 0.10.19
pydantic version: 1.8.2
fastapi version: 0.68.1

That's the same issue as described in #353, i will introduce the extra parameter on Model meta, that will allow to ignore or forbid the extra fields

I don't think it is the same issue. I am only sending it the required/allowed fields, but the choices= restriction on the nullable field isn't accepting None unless I put None in my list of choices

Yep you are right, nullable should still cause the field to be optional. Thank you for catching that!

I don't think it is the same issue. I am only sending it the required/allowed fields, but the choices= restriction on the nullable field isn't accepting None unless I put None in my list of choices

Sorry last post was in wrong issue 🤦

oh no problem, thanks!

Fixed in 0.10.20


Missing MyIntentUtils in Android: Layout Customization

Hello,
I'm creating custom notifications supporting me in your Wiki documentation
My code of Custom Push-Notification Class
`public class MyPushNotification extends PushNotification {
public MyPushNotification(Context context, Bundle bundle, AppLifecycleFacade appLifecycleFacade, AppLaunchHelper appLaunchHelper, JsIOHelper jsIoHelper) {
    super(context, bundle, appLifecycleFacade, new AppLaunchHelper(), new JsIOHelper());
}

@TargetApi(Build.VERSION_CODES.LOLLIPOP)
@RequiresApi(api = Build.VERSION_CODES.KITKAT_WATCH)
@Override
protected Notification.Builder getNotificationBuilder(PendingIntent intent) {
    final Resources resources = mContext.getResources();

    // First, get a builder initialized with defaults from the core class.
    final Notification.Builder builder = super.getNotificationBuilder(intent);

    // Set our custom overrides --

    // Enable 'extended' layout (extends on down-stroke gesture):
    final Notification.BigTextStyle extendedNotificationStyle =
            new Notification.BigTextStyle()
                    .bigText(mNotificationProps.getBody()); // "4:15 - 5:15 PM\nBig Conference Room"
    builder.setStyle(extendedNotificationStyle);

    builder.setPriority(PRIORITY_HIGH);

    Set custom-action icon.
    builder.setSmallIcon(R.drawable.meeting_icon)
            .setColor(resources.getColor(R.color.notification_bkg_color)); // Blue-ish

    // Add 'email guests' action.
    Notification.Action emailGuestsAction = new Notification.Action(
            R.drawable.action_email_guests,
            resources.getString(R.string.action_email_guests),
            MyIntentUtils.getComposeEmailIntent(mNotificationProps.asBundle().getStringArrayList("invited")));
    builder.addAction(emailGuestsAction);

    return builder;
}

}`
Problem
I can't find the "MyIntentUtils" and I don't know if I have to create it (and how) or if you forgot to share that part of the code in the wiki documentation.
I'll appreciate if you can help me with this. It is the only thing where I'm stuck.
Past Issue asking the same question. #119
Thank you!

Same thing here.
We are able to use native notifications but we just cant figure out how to pass the notification object to react native so that we can use the same functions as the ones you guys implemented for iOS. Just like the example below.

constructor() {
  NotificationsIOS.addEventListener('pushKitRegistered', this.onPushKitRegistered.bind(this));
  NotificationsIOS.addEventListener('pushKitNotificationReceived', this.onPushKitNotificationReceived.bind(this));
  NotificationsIOS.registerPushKit();
}

onPushKitRegistered(deviceToken) {
	console.log("PushKit Token Received: " + deviceToken);
}

onPushKitNotificationReceived(notification) {
    console.log('PushKit notification Received: ' + JSON.stringify(notification));
}

componentWillUnmount() {
	// Don't forget to remove the event listeners to prevent memory leaks!
	NotificationsIOS.removeEventListener('pushKitRegistered', onPushKitRegistered(this));
}

This is what you guys have for android:
import {NotificationsAndroid} from 'react-native-notifications';

// On Android, we allow for only one (global) listener per each event type.
NotificationsAndroid.setNotificationReceivedListener((notification) => {
	console.log("Notification received on device in background or foreground", notification.getData());
});
NotificationsAndroid.setNotificationReceivedInForegroundListener((notification) => {
	console.log("Notification received on device in foreground", notification.getData());
});
NotificationsAndroid.setNotificationOpenedListener((notification) => {
	console.log("Notification opened by device user", notification.getData());
});

The question is, how can we link a custom notification (notification with actions) in Android to the methods above so that we can get the notification object and do the corresponding functionalities?
If you guys could show the way or share some code snippet it would be awesome. Just want to say that the library is amazing. Thank you beforehand for the help and for the awesome job!

@lmrarturo just wondering if you had any advance with this topic, or any of you guys. Thanks for the help!

@lmrarturo and Wix team.. Still haven't achieve a good solutions for this topic. I don't kow if you guys have any updates. Thanks!

@pepesrz91 @lmrarturo I'm not sure what you mean by this. MyIntentUtils is a theoretical class, mentioned inside a theoretical block which you need to adjust so as to make it fit for you app. In the wiki doc the code merely describes how to achieve something like the result in the preliminary screenshot, nothing more. It is up to you to create the necessary tools and logic since it's very tightly coupled with your product.

Hey @d4vidi!
Thanks a lot for the response.
We understand that MyItentUtils is a theoretical class and we are able to implement the proper logic and functionality in Android for the notifications, but it doesn't work that well for us because our notification system interacts with our application state which we handle by using Redux.
So our main issue is that we don't know and understand well how can we connect our own notification categories, created natively android,  to the functions you guys have already implemented as:
onNotificationReceivedForeground(notification, completion) { ... };
onNotificationOpened(notification, completion, action) {...};

So that's why we related this issue to missing MyIntentUtils because we don't know exactly if we can actually connect, sort of speak, our own notifications categories and actions to the functions implemented that you guys implemented to catch notifications in React Native. And if it is possible, we are totally lost on how to implement it correctly.
If you guys could point us to the right direction it would be awesome. Thanks for the help!

@pepesrz91 you're not suppose to catch notifications in RN at this point. The native handling code you're referring to simply posts a notification in the device's notification center, nothing more. You do need to bundle an intent (over a PendingIntent) but again, that's product-specific.


Switch to a socket communication between the main thread and the browser

Thank you for filing an issue!

Version

0.4.3
Description

Currently we rely on calling the rely server or setting urls in the browser view for communicating between the main thread and the browser view. We probably should use https://github.com/electron/electron/blob/master/docs/api/ipc-main.md for a more standardized way of communication.

There is no value to the user
It only reduces technical dept

dupe


DSD-973: Adding SkipNavigation to Header

Fixes JIRA ticket DSD-973
This PR does the following:

Adds the SkipNavigation component to the Header component.
Updates the DOM for the Header component to include a header HTML element.

How has this been tested?
Storybook.
Accessibility concerns or updates

Needed for accessible web pages.

Checklist:

[ ] I have updated the Storybook documentation accordingly.
[ ] I have added relevant accessibility documentation for this pull request.
[ ] All new and existing tests passed.

Front End Review:

[ ] View the example in Storybook

Tugboat has finished building the preview for this pull request!
Link:

https://pr1005-pvylonapcsthqhbme4mhocomcidejezr.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/627d7cfb3ecac077f70fcf94

Tugboat has finished building the preview for this pull request!
Link:

https://pr1005-pvylonapcsthqhbme4mhocomcidejezr.tugboatqa.com

Dashboard:

https://dashboard.tugboatqa.com/627d7cfb3ecac077f70fcf94


Bugfix/close db connections

Previously, we used to leave the DB connection open while the
cluster was registered with the operator, potentially resutling
in dangled connections if the operator terminates abnormally.
Small refactoring around the role syncing code.

Coverage decreased (-0.009%) to 9.222% when pulling 09adc5eaa1883b33dc34a23b0fab6970a4373cdb on bugfix/close_db_connections into 8b58782a4a02b06833d8006fed713222a1649418 on master.

lgtm

👍

👍


Fix PackedFloat32Array get index not working

Closes #44812
Getting index value of PackedFloat32Array (such as myFloat32Array[0]) didn't work and adding the missing code make it work.

Thanks!


Markdown is not rendered in attachment texts

Description
Hi everyone,
we use quite a few self build integrations to post messages from e.g. CI/CD or Monitoring into our customer channels via API or Webhooks. To have the ability to set the message color to quickly distinguish info/warning/error we wan't the messages to have a color which is afaik only possible when putting the message into the attachment field. All of this is working great.
But when we use markdown in the attachment text, it just show the raw markdown in the Android App.
This behaviour can only be observed under Android. Both iOS and the Desktop App render markdown in the attachment test as expected.
Apparently #732 only applies to the message text itself and not to message inside the attachment.
Best regards,
Felix
Devices and Versions
Your Rocket.Chat.Android version: 3.1.1
Your Rocket.Chat Server version:  0.96.2

Mobile device model and OS version:  Andorid 8.1 on One+ 5T
Steps to reproduce

Use the API or a webhook to post a message into the channel which uses the following body: attachments: [{"text": "**Hello World**}]

@fleaz Thank you for this detailed description of the issue! 👍


Invalid contract call when using useContractCall

Using the code:
  const [paused] = useContractCall({     abi: contractInterface,     address: contractAddress,     method: "paused",     args: [],   }) ?? [];
Causes the error: Invalid contract call: address=0xCEb299e0f8B6fA3Ef5D698a318806b950275D0eF method=paused
Contract is deployed on Rinkeby. Abi provided contains required method.
What am i doing wrong?

I'm getting a similar issue when trying to call a contract on my localhost (hardhat). useContractFunction works great, but useContractCall is throwing an error no matter what method name I pass in (in this case a getter called mintPrice).
The specific error I'm getting are:
Client-side: failed to check transaction hash: undefined Error: invalid hash (argument="value", value=undefined, code=INVALID_ARGUMENT, version=providers/5.4.1) in
Hardhat-side:
code: -32603 message: "Error: Transaction reverted: function selector was not recognized and there's no fallback function"
`eth_call
Contract call:       MyContract#
From:                0xf39fd6e51aad88f6f4ce6ab8827279cfffb92266
To:                  0xa4899d35897033b927acfcf422bc745916139776
Error: Transaction reverted: function selector was not recognized and there's no fallback function`
import Image from 'next/image'
import styles from './Mint.module.css'
import { useContractFunction, useContractCall, useEthers } from '@usedapp/core'
import { utils, providers } from 'ethers';
import { Contract } from '@ethersproject/contracts'
import abi from '../../data/test.abi'

export default function Mint(){
    if(typeof window === 'undefined') {
         // Don't call these functions serverside to prevent 'window = undefined' errors
        return(

Price 0.35ETH  <button className={styles.mintButton} onClick={() => {mint('0.1')}}>Mint</button>

             Price will increase after each mint. Please wait for your transaction to complete before minting the next map.

)
    }

    const { account } = useEthers()
    const cncInterface = new utils.Interface(abi)
    const cncContractAddress = '0xA4899D35897033b927acFCf422bc745916139776'

    const provider = new providers.Web3Provider(window.ethereum);
    const signer = provider.getSigner(account)
    const contract = new Contract(cncContractAddress, cncInterface, signer)

    // Get mint price
    const [price] = useContractCall({
        abi: cncInterface,
        address: cncContractAddress,
        method: 'mintPrice',
        args: []
    }) ?? [];
    console.log(`price: ${price}`)

    // Setup mint function
    const { state, send } = useContractFunction(contract, 'mint', { transactionName: 'Mint' });

    const mint = (amount) => {
        send({value: utils.parseEther(amount)});
    }

   return(

Price 0.35ETH  <button className={styles.mintButton} onClick={() => {mint('0.1')}}>Mint</button>

               Price will increase after each mint. Please wait for your transaction to complete before minting the next map.

   )
}

Same issue here

Same issue here

Same issue

UseContractCall is deprecated, please use useCall now:)


Sample for ItemsRepeater with ConnectedAnimation

Is this a totally new sample, an addition to an existing sample page, or a modification of an existing sample page?

[x] New sample
[x] Addition to existing sample
[ ] Modification to existing sample

If this is an addition/modification to an existing sample page, which one?
There are sample page for ConnectedAnimation with both ListView and GridView. WinUI has introduced ItemsRepeater sometime back, it would be great to have a sample that shows how to do connected animation for forward and backward page navigation.
Describe the sample

The sample should demonstrate following

A simple forward and backward page navigation with ItemsRepeater. It should correctly work when the user has scrolled a bit the ItemsRepeater present inside the ScrollViewer (This is important for backward navigation to restore the scroll position).

The streaming media apps use something similar

It would be great to see a sample in which a ItemsRepeater containing multiple ItemsRepeaters. The forward and importantly backward page navigation with connected animation should work.

As of today there are two containers that can host ItemsRepeater to support scrolling. ItemsScrollViewHostand the upcomingScrollViewer`. It would be great if the the samples could include both containers.

Please consider if NavigationCacheMode=Required could have influence on the samples.

Is your sample request related to a problem? Please describe it

@jesbis @SavoySchuler (You made a callout to help you prioritise things in WinUI)
I am trying to add connected animation to a app similar to the diagram shown above, which I hope to launch before the Neo. It is a bit frustrating that there is very little documentation to pull this off. I am spending lot of my time doing experimenting to understand the behaviour of these classes, whereas a sample could have saved me valuable time and I believe @ranjeshj 's too .
What I a describing could benefit the broader community as it could help create these class of applications.
Additional context

I am in discussions with @ranjeshj at #microsoft/microsoft-ui-xaml/issues/1848. However, I think it would help everyone if there are some samples too.

Since ItemsView is the recommended control for items collections now, I believe it will be beneficial to create connected animation samples for ItemsView instead of ItemsRepeater.
For context, ItemsView is based off of ItemsRepeater anways.


Create maxCoins.cpp

https://leetcode.com/problems/maximum-number-of-coins-you-can-get/
Pull Request Template
What have you Changed(must)
Write here what you have changed in the codebase
Issue no.(must) -
Pr will be close and marked as spam. If issue number not found or issue was assigned to someone else.
Marking as spam can block your account from HacktoberFest.
Self Check(Tick After Making pull Request)

[ ] This issue was assigned to me if not it will be marked as spam.
[ ] My file is in the proper folder
[ ] I am following clean code and Documentation.
[ ] I have added the title and what program will do first in the file
[ ] I have named the file as (Problem #). (Problem Title(in Camel Case)).(extension)
ex. 2. AddToNumbers.py

README - How to Contribute

@vijay532 you have to mention the issue number in the PR Body.


Cropped Spritefonts

http://puu.sh/jwGOO/c6943d0459.png
Char bounds are not meassured correctly and cause wierd artefacts.

May the Ceiling causes the cropping..
https://github.com/ThuCommix/Sharpex2D/blob/api-2-working/ContentPipeline/ContentPipeline/Exporters/SpriteFontExporter.cs#L128

https://github.com/ThuCommix/Sharpex2D/commit/738d627339a72c6a210bbd6bb7206c4aa1a3c623
https://github.com/ThuCommix/Sharpex2D/issues/38


Gradle plugin resolution fails intermittently during CI builds

This is a problem with Testcontainers' build process, rather than an issue with the library that users would experience.
This was mentioned in #894, but I think we've been seeing this problem quite regularly in the last few weeks. The symptom is most/all of the CI jobs for a particular revision fail at the point of downloading Gradle plugins from https://plugins.gradle.org. I suspect this is either due to problems with the Gradle plugin repo servers (which I think are on Bintray) or a network connectivity issue between CI and those servers.
If we can do anything to make this less likely to occur, it would make it easier for people submitting/reviewing PRs.

Here's an example: https://circleci.com/gh/testcontainers/testcontainers-java/4562


Vulnerability roundup 84: gcc-9.3.0: 1 advisory

search, files

[ ] CVE-2019-15847 CVSSv3=7.5 (nixos-unstable)

Scanned versions: nixos-unstable: 0f5ce2fac0c. May contain false positives.

See also: #88298

CVE-2019-15847: Fixed upsteam at e99bfdd, released as version 7.5.0, 8.4.0, 9.3.0 and 10.1.0. See PR 91481.
nixos-unstable seems unaffected now.

POWER9 architecture specific. Ignore.


Support storing metadata to database for remote videos

Currently, using remote videos will download the metadata and save to /videos, it will be good if it can save the metadata to database by using Prisma or something.
### Tasks

Thanks for the suggestion! CMSs and alternative databases are on our radar as a next step for the project. We'll wrap in remote videos and consider Prisma as part of that.


Sprint 1 objectives

Self Grading Checklist
All self and by peer grading must be complete prior to Teacher review date(s).  Show a direct link to the tangible(s) for each line item.
Sprint1 Key Objectives
During the sprint you need to complete all of these key objectives and find and complete supporting hacks that show level of command you are declaring.
Rank to Ratio, rank is an integer
0 = .00 work, but faking understanding/accomplishment
1 = .55 incomplete assignment, big gaps
2 = .75 meeting some objectives, multiple little gaps
3 = .85 meeting most objectives, one little gap
4 = .90 meeting all objectives
4+Mastered-Y = .95 exceeding all objectives, requires links and  office hours verification

Skill
Mastered (Y/N)
Rank (1-4)
Ratio
Notes

Laptop Verification or Cloud Workspace
[ Y]
4
0.0
verification by Linux commands notebook

VSCode
[Y ]
4+
0.0
 live change code, commit, make, run localhost, test, debug

Show VSCode GitLens
[ Y]
4
0.0
verify commits, show change, comments, reset to commit, run on old commit

GitHub student_2025 repo
[ Y]
4+
0.0
template create, push changes, verify id, review analytics

GitHub Pages Actions
[ Y]
4
0.0
review runs and verify runtime, receive demo

Hacks Tools & Equipment Play
[Y ]
0
0.0
[[Hacks]](https://docs.google.com/document/d/1TNJB2rs7DMnU2wM6uLajalZe5KcFKcz0z90dYjjz_io/edit)(https://nighthawkcoders.github.io/portfolio_2025/jupyter/notebook/python#hacks)

Hacks GitHub Pages Playground
[ ]
0
0.0
[Hacks](https://nighthawkcoders.github.io/portfolio_2025/github/pages/hacks)

Hacks SASS Basics
[ ]
0
0.0
Hacks

Hacks Frontend
[ ]
0
0.0
Hacks

Pair Programming
[ ]
0
0.0
tangibles that show coop work

Total

0.0

Average

0.0

Learning Habits Table
Review criteria in habits.  In this review, pairs should rate each other base on their experience with the person on the line item.
Rank to Ratio, rank is an integer
1 = .55
2 = .75
3 = .85
4 = .90
4+Mastered-Y = .95

Skill
Mastered (Y/N)
Rank (1-4)
Ratio
Notes

Attendance
[ ]
0
0.0

Work Habits
[ ]
0
0.0

Behavior
[ ]
0
0.0

Timeliness
[ ]
0
0.0

Tech Sense
[ ]
0
0.0

Tech Talk
[ ]
0
0.0

Tech Growth
[ ]
0
0.0

Advocacy
[ ]
0
0.0

Comm & Collab
[ ]
0
0.0

Integrity
[ ]
0
0.0

Organized
[ ]
0
0.0

Total

0.0

Average

0.0

Message announcements
mobile icon
Stay up to date from anywhere with the Slack mobile app

Snake game

Turtle RPG

0.91

Hacks completed
Showed knowledge about HTML (soft skills)
Good imgs on the checklist


Automatic migration

Resolves #504

Perhaps you can link successful GitHub Action logs?

https://github.com/whilefoo/ubiquibot/actions/runs/5562861742

Looks good! The only thing that seems a little suspicious to me is that it said it failed to save a database password or something. I presume that it won't be an issue?
https://github.com/whilefoo/ubiquibot/actions/runs/5562861742/jobs/10161500340#step:4:10

it failed to save a database password or something. I presume that it won't be an issue?

It's only complaining about that because we are running this in CI

added secrets. merging


Strict typescript

Finally :) 500+ errors cleaned up, "strict": true is now enabled in tsconfig.json.
Also:

save history simultaneously with question data (I wanted to do this anyway, and weird prisma types forced me to do this for this branch; one possible downside is that fetching a platform twice in a row now results in multiple history records in the same day, but that shouldn't usually happen, and if it happens because of manual runs of npm run cli PLATFORM, then it should also be easy to clean up manually with DELETE FROM history WHERE ...)
update squiggle (old squiggle-experimental didn't have type definitions)
minor refactorings; hope I didn't break any platform-fetching code, I had to refactor some of it blindly, but if something breaks, it'll be easy to fix
moved code for calculating stars into platform modules, as previously discussed (I forgot the comment where I mentioned it)

Is this ready to be merged?

Is this ready to be merged?

Almost, I wanted to figure out #80 first.

Rolling back temporarily due to 500 error on frontpage. Investigating.

It was a dumb circular dependency bug, fixed in master and re-deployed.
And then there was also an issue with replaceAll, which was missing in Node 14, and I had to switch Vercel to Node 16; @NunoSempere, please let me know if you think replaceAll polyfill is still necessary (I removed it), I think it's supported well enough by all modern browsers (maybe we should still support IE 11 or Edge <85 for some reason?..)

let me know if you think replaceAll polyfill is still necessary
Doesn't seem necessary


Icons don't appear correctly in a browser called Supermium

https://lichess.org/forum/team-l1chess-tools-users-team/two-new-features-you-can-enable-in-v23101#5

The browser is not at fault, it's the OS. User still uses Windows 7. A patch on the OS solved the issue.


built-in regular expression support in string field schema validation

Describe the solution you'd like
BoSchema {

:: aStringField regexp "[a-z]+" // only lower-case letters allowed
}

schema will be refactored with the compiler plugin, won't work on the details until it's complete


Add support to bind socket to FIB on a FreeBSD

Hi.
This patch simply add support SO_SETFIB option to socket on a FreeBSD+ OS.
It allow to build VRF-aware applications on FreeBSD and partially solve problem with missing Bind-To-Device option on BSD.

Thanks @wladwm.


Need information on headless chromium version used and it's source

Hi @diegoparrilla
Thank you for providing this layer. Can you please provide more information on which version of chromedriver and headless chromium are you using in the layer and it's source. It will help us in updating the binaries if needed in the future.

https://github.com/diegoparrilla/headless-chrome-aws-lambda-layer/blob/main/Makefile#L14


Chore/remove unused deps

Codecov Report

Merging #1390 (e84aa96) into master (cdc6abb) will increase coverage by 0.01%.
The diff coverage is 0.00%.

 Learn more


Hydrator does not accept items from AE 2 export bus/interface

Minecraft version & Mod Version:
1.12.2, 1.8.6
Single player or Server:
Server (All the mods 3)
Describe problem (what you were doing / what happened):
Tried to automate the production of black concrete with AE 2. First put an ME interface on every side of the hydrator except bottom (water source) and added a processing recipe 1 black concrete powder -> 1 black concrete. Didn't take items. Then put an export bus on all sides except bottom and set it to export black concrete powder. Didn't take items. Then tried chest + Thermal Dynamics itemducts -> worked.
I guess I can work around by connecting an interface to an itemduct but it would be nice if you could export items to the hydrator directly.
Edit: The import bus doesn't pull items out either.
Edit 2: make that it doesn't output items at all, n'either with an itemduct nor with a chest directly placed next to the hydrator.
Crash log (if any):
(If this is a suggestion and not a bug report, delete all this and just write your suggestion!)

This is a weird one.  Normally if i get something working with pipes and hoppers it works with everything.
For the Hydrator im running it now and hoppers alone are working. But just like you said, item ducts work too.
If you have time, can you test some other Cyclic machine blocks and see if they work?  I havent used AE2 in a long time; im just wondering if its ONLY Hydrator + AE2 that has issues, or if its all Cyclic blocks + AE2

https://minecraft.curseforge.com/projects/cyclic/files/2508261


Increase Load, Save and other methods of OdfPackage performance

Hi,
This issue might be related in some way to the #10 issue.
I'm currently working on a project heavily depending on your library. At the moment, I'm trying to speed up the process and I've noticed with the help of a profiler, that some odfpackage methods are quite slow (not horribly but somewhat a bit). All theses methods are related to the ZipArchiveOutputStream library. I read the code, and at some places, I think i could modify those methods, and uses the java NIO API more, in order to gain some performance.
An example, would be, on the org.odftoolkit.odfdom.pkg.OdfPackage#getBytes method where, instead of using it, but using the Filesystems API (part of NIO) on the Odt document saved on the disk, it would be ten times faster (depending on the ZipEntries size).
Also, all the performance would only benefit files on the disk, and not the one from inputStream, so their would be some branching to do, in order to only use NIO API with Odt backed up by the disk.
What do you think about that? I can help if you want, also I can help with the performances tests, as I have written some on my project.
Regards,
Pierre Lemaigre

hi Pierre,
i suspect a lot of this IO code was written around 2009 when it had to run on versions of Java that are long obsolete.
if you can measurably improve the performance there, please send PRs.
we could also upgrade the Apache zip library to a newer version, but we don't have any involvement with that project, so if you'd need to do changes there it could cause some delays...

@Pierre-Lemaigre I second what Michael has written, we are grateful for any help!
In addition, performance tests are especially most welcome. I always struggled with those, it worked if the before/after was tested on the same machine, but this did not work with automation... like GitHub regression tests with every commit...
Thanks in advance, Pierre!
Svante


Fix most trackpad zoom jank

This removes zoom command throttling and the sketch mode zoom to use the mouse zoom for sketch mode which always seems super smooth.
This seems to fix most of the issues. There is a bit of lag if you zoom heavily in and out, but I assume that's more of an architectural challenge than a code challenge. I think throttling the rate of commands being sent only exacerbates any perceived latency.
Fixes #1875

I naively just hit retry on the playwright tests, but the same two tests failed both on ubuntu and macos, and they both have some scrolling zoom in the tests so it's not intermittent, I'll look into it.


MinGW Build Regression

The fix for https://github.com/tbeu/matio/issues/187 caused a build regression in MinGW for me (amd64, CMake Buildsystem):

MinGW (for compatibility w/ build systems that expect glibc) allows the user to specify -lm for the math functions, and automatically maps them to the correct CRT. But it does not provide a -lc, because normally nobody explicitly specifies that.
The fix for the OpenBSD issue caused the build system of matio to automatically add -lc to the compiler line due to the commit https://github.com/tbeu/matio/commit/f058e5dfbaa8bc49fbe76631b6dbf7637be04b10.

This causes MinGW builds to fail, since HAVE_LIBM is true on those systems, but -lc doesn't work there.
The least wrong simple fix I could come up with (without testing on OpenBSD, which I don't use) was to simply also check for -lc and then only link to that if it's available. The following patch works for me on Linux, macOS and Windows:
--- a/cmake/compilerOptions.cmake       2023-11-12 13:25:19.000000000 +0100
+++ b/cmake/compilerOptions.cmake       2024-02-05 16:29:48.804984158 +0100
@@ -70,6 +70,13 @@ endforeach()

 include(CheckLibraryExists)
 check_library_exists(m pow "" HAVE_LIBM)
+# OpenBSD also requires -lc (See
+# <https://github.com/tbeu/matio/issues/187> for
+# details), but that doesn't exist on all platforms
+# that do have libm (most notably not MinGW), so
+# explicitly check for the existence of -lc before
+# linking against in in src.cmake.
+check_library_exists(c fopen "" HAVE_LIBC)

 include(CheckCSourceCompiles)
 set(TEST_CODE_DECIMAL_POINT "
--- a/cmake/src.cmake   2023-11-12 13:25:19.000000000 +0100
+++ b/cmake/src.cmake   2024-02-05 16:26:10.244019256 +0100
@@ -68,7 +68,10 @@ if(STDINT_MSVC)
 endif()

 if(HAVE_LIBM)
-    target_link_libraries(${PROJECT_NAME} PUBLIC m c)
+    target_link_libraries(${PROJECT_NAME} PUBLIC m)
+    if(HAVE_LIBC)
+        target_link_libraries(${PROJECT_NAME} PUBLIC c)
+    endif()
 endif()

 if(MSVC)

While this will solve the issue, the most appropriate fix would probably be to only do that if OpenBSD requires this, for example performing a --no-undefined link check, and only add -lc if that is required to avoid the linker error with that flag. However, I can also open a pull request for my workaround if that's more the route you'd want to go in.

Thanks for the report. Need to fix it.
@seanm FYI

I've setup a MinGW build CI that confirms the regression: https://github.com/tbeu/matio/actions/runs/7822032818/job/21340240188

I've now attempted to write a proper generic test that doesn't depend on the OS but doesn't also add -lc on operating systems where that isn't necessary. See PR #238.
Since I don't have an OpenBSD system, I can't test that case, but I did take care when writing the CMake test. If something comes up on OpenBSD, this should hopefully be easily fixable.


Add phrase support to trigger builds

To the best of my knowledge, the only way to run a build with a PR again is to commit code. At scale, sometimes tests fail intermittently. Moreover, time based tests can fail on occasion and code changes aren't necessary for the build to pass. I'd like the ability to trigger another build by making a comment on the PR -- "test this please". Jenkins PR builder offers this support. Any chance we can get this for concourse?
https://plugins.jenkins.io/ghprb

Please see #46.


La haus updates

This PR was created to have our changes.
This is the branch that production code uses.

Deprecated


torch: attention softmax as float32 for bf16

@ruomingp good question. the unit test is running on CPU. May need to figure out how pytorch community is running bfloat16 for their implementation


Texture Atlas Crash

What is your question?
so i saw that you added 'animateatlas' an dont know how to use it an so i added doxxie as a test an compiled the game an went to a random song an, chart editor an selected doxxie an reloaded the song an crashed did i do something wrong?

i did what BB did but is it different?

just fixed. turns out the texture atlas stuff only worked in the mods folder


Text Mining: Text to highlight Target and Disease not always displayed

Observed behaviour
When looking at the Text Mining Evidence for the following Associations, for some PMIDs the text that contains Target and Disease is not displayed and therefore the items are not highlighted.
http://www.targetvalidation.org/evidence/ENSG00000174059/EFO_0002428
Papers: Nr. 2 (PMID: 28321124), Nr. 3 (28920959) and Nr. 5 (PMID: 29100302) - in these three examples the abstract stops mid-way through a sentence.
http://www.targetvalidation.org/evidence/ENSG00000091704/EFO_0000616
Papers: Nr. 3 (PMID: 28834963) and Nr. 4 (PMID: 28966012) - in these two examples the abstract also is cut-off mid-way through a sentence.
Expected behaviour
Target and Disease highlighted.
Environment
[How did you see the bug?]
Version: on website
Browser: Safari
First raised here: https://github.com/opentargets/webapp/issues/240

Issue moved to opentargets/platform #185 via ZenHub


Markdown file support

Describe the bug
When I configure the filename rule to be able to add a rule on .md files, the rule does not work.
Expected behavior
.md files are linted.
Plugin configuration
'check-fike/filename-naming-convention': ['error', {
  '**/*.{md}': '+([a-z])',
}],

Project structure
.
└── my-app/
    └── READme.md

Desktop (please complete the following information):

OS: Windows

Additional context
When I set the READme.md extension to .js, it works.
Seems like it's only works for js, jsx, ts and tsx extensions.

@joris-fonck-loreal  ESLint is a tool designed for analyzing and checking JavaScript code for syntax and stylistic errors. If you want to lint markdown file, you need to install a another parser for ESLint https://www.npmjs.com/package/markdown-eslint-parser. You can do like this:
{
  "plugins": ["check-file"],
  "overrides": [
    {
      "files": ["**/*.md"],
      "parser": "markdown-eslint-parser"
    }
  ],
  "rules": {
    "check-file/filename-naming-convention": [
      "error",
      {
        "**/*.{js,md}": "KEBAB_CASE"
      }
    ]
  }
}


wrong entry for thingsboard pubsubclient library.

is: "thingsboard/pubsubclient": "^2.9.3",
must be: "tingsboard/TBPubSubClient" "^2.9.3"
thanks

@imbeacon Please adjust the library.json file accordingly.

I forked repository and adjusted library.json. and created a pull request...


Enable stage length input

What's changed

Added form input for stage length.
Added the accordion group to put the timeline in.
Added basic lifeline input

What's next

I think @emily-plummer wanted to look at stage length styling. This should now be possible!

Coverage increased (+0.0%) when pulling db25f6cf6fc1e1136a11fb8dbedf1f03419f7294 on simonv3:add-times into e7021592d39fcfcb14824f9dcbe9c082c5c23e31 on openfarmcc:master.

Coverage increased (+0.0%) when pulling db25f6cf6fc1e1136a11fb8dbedf1f03419f7294 on simonv3:add-times into e7021592d39fcfcb14824f9dcbe9c082c5c23e31 on openfarmcc:master.


Update stable version in repo from 0.0.27 to 0.0.32

Think about update stable version to 0.0.32
Version: stable=0.0.27 (118 days old) => latest=0.0.32 (15 days old)
Installs: stable=736 (58.97%), latest=170 (13.62%), total=1248
Click to edit

This issue seems to be outdated.
This issue suggests to update the stable version of this adapter to 0.0.32 but in the meantime an update to version 0.1.2 is suggested.
So this issue will be closed and replaced by an updated one.


Log with multiple outputs

Enable a way to log for multiple outputs. This change will apply to lakefs and lakectl.

The logging.output (for lakeFS) and --log-output (for lakectl) can accept a list of comma separated outputs to use.
The string "-" represents standard output
The string "=" represents standard error
Output files will be automatically rotated using lumberjack library

hey @nopcoder , some comments within the pull request. Also, I must admit that using "-" and "=" for picking stderr or stdout is a bit confusing. I think that we should either using the plain-text "stderr" / "stdout" (or the Linux's full path if it makes more sense to you - "/dev/stderr" and "/dev/stdout", or the Linux output redirection flags, "1" for stdout and "2" for stderr)

Thanks for the review. The "-" is there for backward compatability with the current lakefs configuration.
The value "-" is usually used in command line application like aws cli where a filename can use - to specify the standard output.
It is always true we can specify the linux path to stderr and stdour and it will work, but for the windows users and to have something that will work on all cases I've kept '-' and added '=' (as far fd 2) for the stderr.


Capture information about alternate heating sources used at schools

@ldodds i think this is ready for a review.  will remove all the old alt heating source code from https://github.com/Energy-Sparks/energy-sparks/pull/2152/files in a separate pr.

hi @ldodds think this one's ready for another review now...


.NET: Linter runner doesn't work

Issue

dotnet build - is the correct linter command (it doesn't require to provide any files paths.
Version and model info
Aider v0.37.0
Models: claude-3-opus-20240229 with diff edit format, weak model claude-3-haiku-20240307
Git repo: .git with 33 files
Repo-map: using 10000 tokens

Thanks for trying aider and filing this issue.
Does running dotnet build <filename> work for you if you just run it on the command line? That's all aider is doing.

Yeah, I understand. It's not working from cmd too. dotnet build accepts a solution or project path as an argument. C# and F# are compilable languages. So we need an ability for aider to make it to not inject anything into the command line.

I believe dotnet build is a quite comprehensive make/compile/build command. Not a linter.

You're right. Alternative true way is to make aider interact with the LSP (with its diagnostics), but it's much harder to implement and also not all IDEs use it. Anyway, I think the feature is a good candidate for a separate issue.
Speaking about dotnet build - yes, it's not a perfect option, but it works (I had to use it via tests with aider), so I believe we should allow developers to set this command as a linter command (or even add a new command).

Using aider --test-cmd "dotnet build" might make more sense.

Using aider --test-cmd "dotnet build" might make more sense.

Paul, but it's not tests. For tests we have dotnet test. What do you think about adding the 3-rd feature "building" or "compilation"? It can be like build-cmd.

Using aider --test-cmd "dotnet build" might make more sense.

This is what I wanted to use, but that does not work for me. In fact, trying to run any command with arguments does not work as I expect. Something wrong with the quoting.
https://github.com/paul-gauthier/aider/issues/741

Specifically, to this issue I see two obvious options:

Call a shell script that invokes dotnet build and ignores the args. Requires no aider changes.
Add an option to not pass the file name to the linter command.

So the model for aider is that the lint step is something which is done to specific files which have been edited. So it's expected to take the filename(s) of source files on the cmd line.
The test step is a "global" operation, not applied to specific file.
Why not do --test-cmd "dotnet build && dotnet test" to both build and test?

Why not do --test-cmd "dotnet build && dotnet test" to both build and test?

@paul-gauthier Paul, because it's not the same operations :)
Build is a build and test is a test) it's often when a project doesn't even have any tests.
Many languages have a build/compile feature and it'll be nice to see support of building in aider.
If you wish I can even implement it...

It's called --test-cmd but it doesn't need to be used for testing. It's just used for any repo-wide checks that you want aider to do. For many projects, that's testing. It could be building. Or it could be both. Whatever you like for your project.
There's no need for a separate "--build-cmd". That just adds complexity and I don't think it does anything more than using --test-cmd for any/all repo-wide checks.

I'm going to close this issue for now, but feel free to add a comment here and I will re-open or file a new issue any time.


Axiom Node is deprecated with replacement being Axiom JS

Axiom Node's site states: Deprecated, use axiom-js instead
My attempts to integrate axiomhq/js in place of the current implementation in this repository have not been successful. Errors as described in this issue on their repository.
Any help to migrate this repository to axiomhq/js would be helpful for others trying to do the same.

Thanks I will try and get this updated.  I will also accept a PR for this.

fixed.