feat: RowFilter is a valid pruning predicate

The appearance of union makes me think this whole file needs to be cleaned up. These functions should probably all live on the rewriter who can accumulate all the references. I figure we can clean this up at some later point.

tbh calling RowFilter All and promoting it to vortex-expr would probably be the right change


chore(overlay): add attach method to PositionStrategy interface

After fixing the bad import this seems fine (same as the attach on ScrollStrategy)

This is technically a breaking API change, but I don't think anyone inside Google will be affected by it


Validate Project URL

closes #640
Shows an error message if the project URL is not valid in the tag creator.

The 'Test & publish code coverage' stage is failing. We will need to take a look at that and see why it is failing.


FIX : Helpful-material page scroll up button #1000

The Scroll upp button in helpful-material page was missing the arrow icon. Added arrow icon, used the same svg that is being used on homepage(index.html)

Fix #1000 Thanks :)


Datenschutz URL outdated / non-existent

https://github.com/criticalmaps/criticalmaps-android/blob/905c15516881ee46a550184f52e337c16fa9b43c/app/src/main/java/de/stephanlindauer/criticalmaps/Main.java#L134

:+1: thx!
can you point this to the develop branch?

oh wow, github can change the pr base branch. nice! :)

oh wow, github can change the pr base branch. nice! :)

Yes, they finally managed to add that. Since a while you can also push to PR branches from forks, if allow edit from maintainers is checked. Quite useful for minor amendments.


Return union of all envs from all workspace folders

Closes #17966
FYI @rchiodo regarding https://github.com/microsoft/vscode-python/pull/17515#issuecomment-928164544

@karrtikr does this mean when we're now requesting interpreters for all worksapces, we don't need to pass the Uri of each workspace.
I.e. is this change going to work in the API exposed to the Jupyter extension?
Asking as we iterate through all workspace folders today & combine the list of interpreters.

Yep, passing resource as undefined should do.


temporary request and response should be released in server & client mode

When using the server and client pattern in eCal, there's a significant memory leak problem especially when big data chunks are present in the request sent by a client.
This pull request aims to fix this issue by deleting the request and response variables at the end of the "call" method.

Thank you for reporting. Just checked it again and yes .. we have a leak here.


[HOLD] Provide a way to run podman service only

Without the Openshift cluster.
Fixes #1097.
Still WIP but creating PR already to get early feedback.
Testing

crc start --disable-cluster -b PATH_TO_BUNDLE
crc status

The start command should say Podman service now available and status command should have this line:
Podman:       	 Service available

How long does it take to start with just podman compared to starting a whole cluster?

Since clients aren't available we can not provide this functionality at this time.

podman support will be handled this quarter (https://github.com/code-ready/crc/issues/1611). It will be faster to start again than fixing all the conflicts here. This PR and its comments is still good to keep in memory.

podman support will be handled this quarter (https://github.com/code-ready/crc/issues/1611). It will be faster to start again than fixing all the conflicts here. This PR and its comments is still good to keep in memory.


compilation does not work

After following the instructions, the resulting binaries like ddsim_simple from the build directory are dangling symlinks.

Hey Alexandru,
could you share your build log with us.
Did you build the ddsim_simple CMake target by calling cmake --build . --config Release --target ddsim_simple?

cmake --build . --config Release --target ddsim
-- The CXX compiler identification is GNU 7.4.0
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Setting build type to 'Release' as none was specified.
-- qfr update
-- using QFR submodule
-- dd_package update
-- using DD package submodule
-- googletest update
-- The C compiler identification is GNU 7.4.0
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Found PythonInterp: /usr/bin/python (found version "2.7.15")
-- Looking for pthread.h
-- Looking for pthread.h - found
-- Performing Test CMAKE_HAVE_LIBC_PTHREAD
-- Performing Test CMAKE_HAVE_LIBC_PTHREAD - Failed
-- Looking for pthread_create in pthreads
-- Looking for pthread_create in pthreads - not found
-- Looking for pthread_create in pthread
-- Looking for pthread_create in pthread - found
-- Found Threads: TRUE
-- Found Boost: /usr/include (found suitable version "1.65.1", minimum required is "1.50") found components: program_options
-- Boost version: 1.65.1
-- Configuring done
-- Generating done
-- Build files have been written to: /home/alexandru/github/ddsim/build
[ 18%] Built target dd_package
[ 87%] Built target qfr
[100%] Built target ddsim

After make in the build directory the file exists, but
cat ddsim_simple
cat: ddsim_simple: No such file or directory

trying chmod on the link
chmod +x ddsim_simple
chmod: cannot operate on dangling symlink 'ddsim_simple'

ls -alt ddsim_simple
lrwxrwxrwx 1 alexandru alexandru 52 Mär  3 16:55 ddsim_simple -> /home/alexandru/github/ddsim/build/apps/ddsim_simple

As stated in the README you need to build the ddsim_simple target since ddsim is just the library. ;-)

Thank you. It is very confusing why the executable exists as symlink without being compiled.

I’ll fix it tomorrow! Sorry for the confusion.


gatsby-next-example and ThemeProvider

[x] This is not a v0.x issue.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior 🤔
I would expect that the plugin in the gatsby-next example would support having a custom theme
Current Behavior 😯
8a67ecc547aeeddeb841f4b948ed7d3d5761b704 changed the plugin from ThemeProvider to the StylesProvider. This threw me off track and I had to dig through the older commits to see how to use ThemeProvider correctly in the current way
Context 🔦
It would be cool if the internal plugin in the example supported both ways so that if a person put a theme in the gatsby-config, it would use ThemeProvider instead of StylesProvider.

The latest version of the Gatsby example show how to use a custom theme. What's wrong with it?

Oh sorry I missed that it was in the TopLayout plugin now instead of the mui plugin.


Error during Points2Regions plugin

Hi there,
I'm trying to use the Points2Regions plugin on visium data.
I have installed tissuumaps in a conda environement on a mac. I have also installed sklearn, scikit-image and pandas.
When I try to run the plugin I get this error:
This plugin only works on a pip installation of TissUUmaps, with the extra packages: pandas, sklearn, skimage
Also, a side question, how does tissuumaps figure out which columns in my file correspond to genes/markers as I have other categorical variables in my csv file?
Thanks

Hi @kaizen89,
If you run TissUUmaps from the terminal, can you copy here the error message in the terminal after you start the plugin? You may have to start TissUUmaps with the --debug option.
You should be able to select the column you want to use in the plugin directly.
But note that this plugin is designed for data with individual genes points (like FISH or ISS) more than for Visum data (with multiple genes expressed in each spot).
It should be possible to adapt to the second ones, but could be a bit of work on the plugin (tag @axanderssonuu who wrote the plugin)


[grpc] Fix infinite loop in grpc trace

Describe the pull request

What does your PR fix? Fixes #10270
On Ubuntu, grpc might exhibit infinite loop. The related source issue: https://github.com/grpc/grpc/issues/21507, https://github.com/grpc/grpc/issues/21213. This problem has not been fixed in new version, adding a patch to fix it temporarily.
Which triplets are supported/not supported? Have you updated the CI baseline?
No
Does your PR follow the maintainer guide?
Yes

Based on the author's comment in https://github.com/grpc/grpc/issues/20690#issuecomment-545059209, I think the right path forward is to instead only install one of grpc++ and grpc++_unsecure.


Mesh CVE policy

Description
fixes #6156
Adds a CVE policy for Kong Mesh
@lahabana please review

Testing instructions
Preview link:
Checklist

[x] Review label added
[x] Conditional version tags added, if applicable.

Pinging @lena-larionova for approval :smile:


Fix namespace tagging for latency metrics

What does this PR do?
Fix namespace tagging for latency metrics. All latency metrics are tagged with the last namespace processed
Motivation

2050-wayfair-aerospike-latency-metrics-shows-n-a-for-0-values

Additional Notes

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

Looks good, some small comments. Would be nice to have a test.

re: the test might take a while. Is it possible to handle that in a different PR?


Bug-Report: Tenacity 8.4.0

Dear Devs,
The tenacity package just released the version 8.4.0. Unfortunately, this update brakes the luigi import most likely due to an API change of tenacity. More specifically, the error occurs when importing the rpc module. Please find below the traceback. Judging by the description this only happens for python>3.8. python==3.6 still installs tenacity==8.2.2
>>> import luigi
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "<path_python>/lib64/python3.9/site-packages/luigi/__init__.py", line 34, in <module>
    from luigi import rpc
  File "<path_python>/lib64/python3.9/site-packages/luigi/rpc.py", line 33, in <module>
    from tenacity import Retrying, wait_fixed, stop_after_attempt
  File "<path_python>/lib64/python3.9/site-packages/tenacity/__init__.py", line 653, in <module>
    from tenacity.asyncio import AsyncRetrying  # noqa:E402,I100
ModuleNotFoundError: No module named 'tenacity.asyncio'

Same error here

Same error, setting tenacity<8.4.0 was a temporary solve

The issue is also now reported in tenacity itself. https://github.com/jd/tenacity/issues/471

Fixed by https://github.com/jd/tenacity/releases/tag/8.4.1


add wet day frequency correction

Because of dry-day GCM biases and the mixed discrete-continuous nature of precip data, we need to add a wet day frequency correction to our downscaling workflow. This will involve two steps:

pre-processing: replace zeros in obs and model data with nonzero uniform random values below a trace threshold (probably we will set this to be 0.05 mm/day)
post-processing: all values in the bias corrected data below that threshold will be set to zero

@brews what's your opinion on this being one vs two services? I think it could just be one with a "pre-process" or "post-process" option argument, that seem reasonable to you?

@dgergel Hmmm. Thinking about making the interface natural... I feel like you're describing it as an option to bias-correction. How about start by setting it as a boolean flag to biascorrect:
dodola biascorrect {insert usual options here} --wetday-correction

This feels pretty natural to me. It would be an option to the bias correction service, but you can implement it in the core however you'd need, of course.

It needs to be more than an option to bias correction because it will also come into the pipeline after downscaling (the post-processing step is after both bias correction and downscaling). So, I'm fine with the above syntax, but we will also need something along the lines of:
dodola wetday_correction <input zarr store> <output zarr store> --post-process

So, seems like we could have pre-process and post-process flags to the wetday_correction service, and add it as a boolean flag to the bias correction service? @brews how does that sound?

@dgergel @dgergel Hmm. So then the post-processing step is identical for downscaling?
What if we add a similar flag to dodola downscale, rather than making it an explicit, new step:
dodola downscale <blah blah blah> --wetday-correction

The wet-correction in downscaling and bias correction might share the same functions/classes in dodola/core.py (which we can unit test)?
Does this solve your issue or it might be that there is something critical that I'm totally missing?


Update requirements.txt

I got ModuleNotFoundError: No module named 'colorama' without this. So I added this to the requirements.txt.

That's right, I forgot


Provide upstart skeleton?

At Transloadit, we're currently generating the following Upstart file to launch tusd:
$ cat /etc/init/tusd.conf
stop on runlevel [016]

respawn
respawn limit 10 5

limit nofile 32768 32768

pre-stop exec status "tusd" | grep -q "stop/waiting" && initctl emit --no-wait stopped JOB="tusd" || true

script
  set -e
  rm -f "/srv/shared/tusd-log-fifo"
  mkfifo "/srv/shared/tusd-log-fifo"
  ( logger --tag "tusd" <"/srv/shared/tusd-log-fifo" & )
  exec >"/srv/shared/tusd-log-fifo"
  rm -f "/srv/shared/tusd-log-fifo"
  exec bash -c "cd /srv/current \
    && source env.sh \
    && exec sudo -HEu transloadit-api2 PATH=\${PATH} LD_LIBRARY_PATH=\${LD_LIBRARY_PATH} \
      /srv/current/stack/bin/tusd \
        -host <IP_ADDRESS> \
        -port 1080 \
        -base-path /resumable/ \
        -hooks-dir /srv/current/api2/bin/tusd-hooks \
        -max-size<PHONE_NUMBER>0 \
        -s3-bucket tmp.transloadit.com \
      2>&1"
end script

Then our rsyslog is configured like so:
$ cat /etc/rsyslog.d/49-tusd.conf
if $programname == 'tusd' then /srv/shared/log/tusd.log
if $programname == 'tusd' then @logs.papertrailapp.com:34719
if $programname == 'tusd' then stop # no other actions to avoid duplicates

This handles things like logging (both to local file via syslog as well as centralized to papertrail), running tusd as a non-privileged user, injecting environments with a privileged user, and respawns.
Now this config may not work for everyone. I would also not be surprised if people in the tus community know better ways. And so the thought arises, maybe it's useful to offer a few community curated startup templates, initially for both Upstart and systemd (but that could later be expanded to support other platforms), so that the binary is easier deployed in production environments, and users run tus in a safer, and pretty much all in an identical way.
This could be provided by the tus community either in the form of:

A developers's guide with some snippets in the best practices, ready for copy/paste.
In a ./templates/ directory or similar
In code, so that one could type tusd os_startup or similar, and it would ask some questions, and output the appropriate startup file, that the user could | sudo tee /etc/to/the/right/place (avoiding tusd writing itself and having to ever run it as root)

Thoughts?

I am not able to maintain such an upstart skeleton for myself, so I feel quite uncomfortable with having something around which I do not understand well enough.

What do you think if we add an entry to our newly created FAQ section linking to this specific issue for a template to use?

I think that's a good start! I don't think this issue will allow for pleasant iteration though, so i'd be even better to inline the snippet there, so that our peers can spot errors and send PRs against it. Of course, comments can be left here, but I probably wouldn't bother keeping track of a thread like that, vs a single item and latest version that one can look at.

After thinking a bit more about this, we could add an entry in the newly created docs/ folder containing the upstart configuration alongside some explanation about why certain settings were chosen and how to actually use it. However, having never worked with upstart or systemd I do not see myself capable of writing some a document. Would you, Kevin, have some spare time to write a few sentences up?

Not on the short term, but I can add this to my todo. I feel we could already add the paste, and then iterate on it as we go, applying the McDonalds Theory

Upstart has been discontinued in the meantime, so we can close this.


error messsage: stateless function components cannot have refs

Using react-native-video 0.8.0-rc and react-native 0.26.1
When I am removing a video player which has a ref attached to it, I am getting this error

@chetankumar01 is this still an issue with the latest react-native releases?

No.. it's not. Thanks
Sent from [Nylas N1](https://link.nylas.com/link/aob6oa4m5qlfh8p2ytxjy3nty
/local-7eb70fbb-7dfa/0?redirect=https%3A%2F%2Fnylas.com%2Fn1%3Fref%3Dn1&r=cmVw
bHkrMDAxODRjZmI2NjUzNDk3MDE4Yzk2ZjdhMjI3OGIwNGI1YjYzOWFmNmZlMGMwY2I3OTJjZjAwMD
AwMDAxMTNkYzQ1YTk5MmExNjljZTA5NTc1NWZjQHJlcGx5LmdpdGh1Yi5jb20=), the
extensible, open source mail
client.![](https://link.nylas.com/open/aob6oa4m5qlfh8p2ytxjy3nty/local-
7eb70fbb-7dfa?r=cmVwbHkrMDAxODRjZmI2NjUzNDk3MDE4Yzk2ZjdhMjI3OGIwNGI1YjYzOWFmNm
ZlMGMwY2I3OTJjZjAwMDAwMDAxMTNkYzQ1YTk5MmExNjljZTA5NTc1NWZjQHJlcGx5LmdpdGh1Yi5j
b20=)
On Aug 30 2016, at 12:19 am, Matt Apperson<EMAIL_ADDRESS>wrote:

[@chetankumar01](https://github.com/chetankumar01&r=cmVwbHkrMDAxODRjZmI2NjUz
NDk3MDE4Yzk2ZjdhMjI3OGIwNGI1YjYzOWFmNmZlMGMwY2I3OTJjZjAwMDAwMDAxMTNkYzQ1YTk5Mm
ExNjljZTA5NTc1NWZjQHJlcGx5LmdpdGh1Yi5jb20=) is this still an issue with the
latest react-native releases?

--
You are receiving this because you were mentioned.
Reply to this email directly, [view it on GitHub](https://github.com/react-
native-community/react-native-video/issues/227#issuecomment-243217778&r=cmVwbH
krMDAxODRjZmI2NjUzNDk3MDE4Yzk2ZjdhMjI3OGIwNGI1YjYzOWFmNmZlMGMwY2I3OTJjZjAwMDAw
MDAxMTNkYzQ1YTk5MmExNjljZTA5NTc1NWZjQHJlcGx5LmdpdGh1Yi5jb20=), or [mute the
thread](https://github.com/notifications/unsubscribe-auth/ABhM--LlXB9yZHzoJN6L
5F5Wi9sDe3Hdks5qkympgaJpZM4ImZob&r=cmVwbHkrMDAxODRjZmI2NjUzNDk3MDE4Yzk2ZjdhMjI
3OGIwNGI1YjYzOWFmNmZlMGMwY2I3OTJjZjAwMDAwMDAxMTNkYzQ1YTk5MmExNjljZTA5NTc1NWZjQ
HJlcGx5LmdpdGh1Yi5jb20=).![](https://github.com/notifications/beacon/ABhM-
y586p8aGEMn7-w1RCpiliSxVaApks5qkympgaJpZM4ImZob.gif)


Making the parameters of operators queryable through their names

When a user pauses the execution and wants to query some internal parameters (especially while training ML models), we need to make the parameters queryable through their name. This PR is the first step in that direction

You are right about showing it to the user. I thought about it while planning this feature, but dont have a way out yet.


Create website GitHub workflow

Create a GitHub workflow for the website which does the following:

[ ] Install npm packages
[ ] Run ESLint
[ ] Run Jest tests

Requires

#105


wheelchAI missing

Hi,
v2 is using wheelchAI which I could not find anywhere.
Any information where I can find it?
Thanks!
Kilian

Hi there,
this is actually just a naming issue. All the utils that were previously in wheelchAI are now found in the utils folder. Just import that and you should be good to go.

Thanks!


feat: add components/directives/pipes/services to barrel when generat…

…ed into a shared dir

One nit to clear the function, LGTM.


0088 - Merge Sorted Array

Description of the Problem
You may copy that from LeetCode
Code
Given two sorted integer arrays nums1 and nums2, merge nums2 into nums1 as one sorted array.

Note:

The number of elements initialized in nums1 and nums2 are m and n respectively.
You may assume that nums1 has enough space (size that is equal to m + n) to hold additional elements from nums2.
Example:

Input:
nums1 = [1,2,3,0,0,0], m = 3
nums2 = [2,5,6],       n = 3

Output: [1,2,2,3,5,6]

Constraints:

-10^9 <= nums1[i], nums2[i] <= 10^9
nums1.length == m + n
nums2.length == n

Link To The LeetCode Problem
LeetCode

Please assign this to me

Can you assign this work to me.

Please assign to me @vJechsmayr

please can u assign this to me ? @vJechsmayr

@Mrsterius @24Cipher The one who created a Issue will be prefered for assignment!

@anupamabauri Sir please assign this issue to me. I can easily contribute to it.

#402


docs: add links to top with jump2header

Added links to the table of contents for better navigation.
Demo
Generated with:
jump2header --start "Quick Start" --slug "contents"

If you don't like the PR but interested in links to top see jump2header for more options.

This pull request #1072 is assigned to @fabriciofx/z, here is why; the budget is 15 minutes, see §4; please, read §27 and when you decide to accept the changes, inform @paulodamaso/z (the architect) right in this ticket; if you decide that this PR should not be accepted ever, also inform the architect; this blog post will help you understand what is expected from a code reviewer; there will be no monetary reward for this job

@strdr4605 it seems fine to me. @paulodamaso can you merge it, please?

@strdr4605 just for curiosity: is is possible change the link icon (the arrow)?

@fabriciofx Yes. At the moment 5 icons are available:

@paulodamaso which them do you prefer?

@paulodamaso It will look like that.  To me, it looks a bit odd, but the feature itself is nice

@paulodamaso It will look like that. To me, it looks a bit odd, but the feature itself is nice

This is how it looks on my machine (macOS).

@strdr4605 Thank you for the PR, but I wouldn't like to have a dependency for generating the headers for the README. Also, I think that these images are a little cumbersome, I'd prefer a thinner approach, like just ^ for example

@paulodamaso
jump2header is not a dependency. You can use it one time to generate links for 30+ sections, then all new sections and links can be added manually.
It's not possible to add links as text in headers as it will break the anchor (only emojis).
I suggest placing links bellow the header or at the end of the section.
In this case, we can use custom text instead of emoji. Example

@strdr4605 I don't want that every contributor that edit the readm.md file must execute this script, so that's why I'm closing this issue

@sereshqua/z please review this job completed by @andreoss/z, as in §30; the job will be fully closed and all payments will be made when the quality review is completed

Code review was too long (28 days), architects (@paulodamaso) were penalized, see §55

Pull request #1072 was not merged, no payment for ARC, see §28

@0crat quality bad

Quality is low, no payment, see §31


[Gradle + TestNG] Need for a new Listener to notify TestNG xml listener initialization failures

TestNG Version
7.8.0

Note: only the latest version is supported

Context: Gradle + TestNG

Gradle useTestNG configuration:(Expand)
tasks.test {
    useTestNG {
        listeners.add("com.listeners.DefaultSuiteListener") // registered first before registering other listeners from xml

        suites("src/test/kotlin/config/functional_tests.xml")
    }
}

TestNG XML configuration:(Expand)

<suite name="Functional Tests" verbose="1" parallel="methods" configfailurepolicy="continue">
    <!--  listeners -->
    <listeners>
        <listener class-name="com.listeners.SuiteListener"/> <!-- Something goes wrong in initialising -->
    </listeners>
    <test name="Fun Test">
        <packages>
            <package name="com.tests.*" />
        </packages>
    </test>
</suite>

Expected behavior

On TestNG Listeners initialisation failure, already configured listeners(if any) need to be notified, something like ListenerConfigStatusNotifier/ListenerSetupReporter

Actual behavior

TestNG throws TestNGException and gradle logs this exception only in debug mode(gradle test --debug) or one need to implement and register gradle API's TestListener to gradle lifecycle, however this is not flexible option as gradle API internal to build.gradle.

I'm happy to take up this request once we agree on the need for it.
Is the issue reproducible on runner?

[x] Shell
[ ] Maven
[x] Gradle
[ ] Ant
[ ] Eclipse
[ ] IntelliJ
[ ] NetBeans

Test case sample

Please, share the test case (as small as possible) which shows the issue

here is the sample project to reproduce and demonstrate the problem.
https://github.com/sravanmedarapu/GradleTestNgScratchpad/tree/main

@sravanmedarapu - For a second, lets take gradle out of the equation. Can you please help share a sample that just uses the TestNG API and create a main method that can be used to reproduce the problem? I would like to start from there. That way we know if its the build tool integration with TestNG that is broken, or if its really TestNG.
I say this because I think Gradle embeds a testng version in it and that is perhaps not really the latest version of TestNG

@krmahadevan here is the sample.
Providing context on the Gradle flow, here's how Gradle launches TestNG and handles exceptions:

Gradle initiates TestNG to execute tests. If any problems arise with the listener or suite setup/execution during this process, the TestNG API should be able to inform about these failures.
Gradle catches exceptions thrown by TestNG and logs them in debug mode or notifies the Gradle listener. However, this Gradle listener is limited to the build script (build.gradle or build.gradle.kts) and cannot be extended to Kotlin/Java projects to perform additional actions, such as reporting to Slack, Elastic, etc.

Please note that the Gradle listener in the build script can be utilized for various purposes related to the build process, but it may not be directly accessible in external Kotlin/Java code. However, other mechanisms, such as custom TestListeners within TestNG, can be used to achieve additional actions beyond the build script's scope.

@sravanmedarapu - Thanks for sharing that sample. When you run the main method TestNG explicitly fails at instantiation. I think this is the same behaviour using Maven as well.
Shouldn't Gradle also be doing the same thing ?
We could build a ListenerConfigStatusNotifier/ListenerSetupReporter but the same instantiation problem could happen with them as well right? So where would this end?
@juherr your thoughts ?

TestNG throws exceptions when something goes wrong outside the test perimeter.
Here, the bootstrap is failing and testng throws an exception as expected.
If something should be fixed, it should be in the integration.
Maybe a dedicated exception could be used if it helps.

@krmahadevan / @juherr
Agree that maven is bit more clear on reporting/logging the problem.

Gradle could do a better job in reporting/logging, will check gradle team on simplifying the logging. However Gradle could only control logging/exception handing.

Shouldn't Gradle also be doing the same thing ?

Absolutely, TestNG provides notification/listener mechanisms at various levels, such as Execution, Suite, Test, and more. Incorporating a initialize/pre-setup layer with a comprehensive listener would indeed cover the entire lifecycle.

 Current TestNG.run() implementation(Click)
  public void run() {

  initializeEverything(); // <=== Not covered with listeners
  sanityCheck();          // <=== Not covered with listeners

 // Everything else covered with listeners
  runExecutionListeners(true /* start */);

  runSuiteAlterationListeners();

  m_start = System.currentTimeMillis();
  List<ISuite> suiteRunners = runSuites();

  m_end = System.currentTimeMillis();

  if (null != suiteRunners) {
    generateReports(suiteRunners);
  }

  runExecutionListeners(false /* finish */);

...
}

We could build a ListenerConfigStatusNotifier/ListenerSetupReporter but the same instantiation problem could happen with them as well right? So where would this end?

Indeed, having a Listener mechanism in TestNG grants more control over additional activities beyond just logging or throwing exceptions. It allows for custom actions such as reporting build problems to Teamcity, storing relevant events in a database or Elastic, and sending notifications to Slack or other communication channels.

@sravanmedarapu - The question is more about how does one deal with instantiation failures on the newer bootstrap listeners, because from what I see, we can still end up with problems instantiating them via reflection. I am more interested in that part of it.
So the question is definitely not about the intent of introducing newer listeners, but the concern is more around the problem will exist even there as well.
Also even if TestNG does provide such a listener, how would a Gradle driven test consume it because from what you shared so far I understand that:

Gradle gobbles all exceptions raised by TestNG and logs them in debug mode.
The Gradle provided listeners which can eavesdrop into such problems are confined to be accessible only from within the realm of a Gradle build file.

The crux of the problem seems to be with Gradle gobbling exceptions from TestNG (Not sure what would the rationale behind that be, especially when TestNG is the test runner that it is piggy backing for test execution). I was hoping that if we can get that part sorted out from Gradle wherein they don't swallow exceptions and instead let them bubble up and fail the build, maybe the issue would get resolved (Please bear if that sounds naive, but that's the level of my knowledge with respect to Gradle and its internals)
@juherr

Maybe a dedicated exception could be used if it helps.

Currently we throw a TestNGException . We can throw something else, but that doesn't change anything because Gradle is basically catching Throwable. So anything we throw is going to be gobbled up.
# Created at 2023-07-26T18:37:29.347
org.testng.TestNGException:
Couldn't find a constructor in class org.testng.MyListener
        at org.testng.internal.objects.InstanceCreator.newInstance(InstanceCreator.java:57)
        at org.testng.ITestObjectFactory.newInstance(ITestObjectFactory.java:10)
        at org.testng.TestNG.setListenerClasses(TestNG.java:684)
        at org.testng.TestNG.configure(TestNG.java:1549)
        at org.testng.TestNG.configure(TestNG.java:1768)

@krmahadevan here is the draft of what I am thinking https://github.com/testng-team/testng/compare/master...sravanmedarapu:testng:master
This should work with TestNG + Gralde flow or whoever using TestNG runner, happy to test with bootstrap listeners with bit of direction.

we can still end up with problems instantiating them via reflection

It's just not about logging only, as briefed earlier Gradle has TestListener to notify these events which can be used for logging but it's limited to gradle build script. My request is to bring this ability into TestNG API i.e. kotlin/java projects which gives flexibility to do extra activities by utilising framework.

Also even if TestNG does provide such a listener, how would a Gradle driven test consume it because from what you shared so far I understand that:

Gradle gobbles all exceptions raised by TestNG and logs them in debug mode.
The Gradle provided listeners which can eavesdrop into such problems are confined to be accessible only from within the realm of a Gradle build file.

In general, Listeners remain within the suite perimeter, at least from the perspective of XML configuration. TestNG's lifecycle includes various listeners to notify about Tests, Suites, execution failures, and statistics. Given that Listeners are an integral part of this lifecycle, it's reasonable to expect a dedicated one for handling Listener failures as well.
<suite name="Suite">
  <listeners>
    <listener class-name="ListenerWithRuntimeError"/>
  </listeners>
  <test name="Test">
      <classes>
          <class name="SampleTestClassSample"/>
      </classes>
  </test>
</suite>

Introducing this new listener would not only prove beneficial for our team but also for the wider TestNG user community. Would request you to give consideration to this request.

@sravanmedarapu - I am not sure if you are getting the fundamental question that I am asking.
What would happen if TestNG fails to instantiate this new listener? TestNG will continue to throw an exception which is again going to be gobbled up by Gradle. So what are we solving here? This is what I would like to understand.
Introducing another listener into the mix is NOT going to solve the problem of Gradle swallowing TestNG triggered exceptions thrown from listener instantiation failures. How would that issue be addressed by this PR is what I am trying to figure out here.

In short, what I would like to know is :

If TestNG fails with instantiating the existing listeners, this new listener would get notified.
Who would get notified if TestNG fails to create an instance of this new listener ? Given the fact that this new listener is also a TestNG known listener.

@juherr - Please chime in, incase I am missing something obvious here. Basically the ask is to figure out a way to be notified of TestNG bootstrap failures.
We have two options for such notifications:

We can throw a special exception. But I guess it wouldnt work with Gradle because its gobbling all Throwable
We can introduce a special call back. But this would mean that TestNG should instantiate this call back and reflection is the only way of doing it. TestNG can fail to instantiate this new callback as well. So we are back to square one.

In my opinion, the "fix" can only be done in 3rd party integration where the runner lifecycle is managed.
In case of gradle, it should not be too complicated to create a custom plugin using the TestNG api if the embeded one is not good enough.

@krmahadevan Apologies for not being precise here.
Yes, If TestNG fails to instantiating existing listeners(listeners mentioned in XML files), the proposed ListenerSetupReporter will be notified because new listener registered first before TestNG initialise XML listeners(using gradle API useTestNG build script).

If TestNG fails with instantiating the existing listeners, this new listener would get notified.

This exception will gobbled by gradle but can be covered by Gradle TestListener to log.

Who would get notified if TestNG fails to create an instance of this new listener ? Given the fact that this new listener is also a TestNG known listener.

@sravanmedarapu - A TestNG listener can be wired in using any means, but eventually the instantiation is always going to be via reflection. So any failure in instantiating a TestNG listener should be noticeable via the TestListener from gradle which I believe can be configured to be aborted as well.
Please refer to the comment from https://github.com/testng-team/testng/issues/2937#issuecomment-1653520145
I think we are fixing the problem in the wrong place. The origin of the problem is NOT testng but the integration between Gradle and TestNG. IMO, the fix should also be there and NOT at TestNG.

I believe that is essentially what is being said in this issue as well, which you created at Gradle project front https://github.com/gradle/gradle/issues/25857#issuecomment-1653298479

@krmahadevan, thank you for taking the time to review my argument.
I understand that I may not have been able to persuade enough with my points. Nevertheless, I will leave the draft of my proposal here for future reference and consideration.

@juherr - I was thinking of closing this off, since the issue is not within TestNG. Your thoughts ?

@krmahadevan yes, agree

Closing this issue since there's nothing to be done from TestNG side.


THRIFT-1857 Replace implicit relative imports with explicit relative imports.

If nobody is going to look at it for 2 years...

Maybe I will...

Hi, I know there's a long backlog here.  We've worked it down from 120 open PRs to under 100 in the last month.  Typically it has been up to the submitter to refresh the pull request if it does not pass the CI builds and once they do, it has been up to the submitter to email the dev mailing list to get the attention of a committer to finish the commit.
If you can rebase against master and reopen/push and it passes the CI build then we'll be in a much better position to merge it.


Update package.json (Deprecation)

@ForsakenHarmony the test is failing since node-globule is not supported on node 0.1 anymore, however gaze is.
I suggest updating the minimum supported gaze version to 0.3 too and see if you can get the approved. I would do this myself but this PR is coming from an unknown repository.

Closing as ^1.0.0 will install the latest minor version (because of ^) so 1.1.0 will get installed already.


360 video support on iOS question

Do I understand correctly that 360 videos are not supported on iOS?
It says under “Supported platforms” on your website “360 videos do not work in a browser on iOS.” and this video does not play when viewed on my iPhone 6S on iOS v10.3.2:  https://storage.googleapis.com/vrview/examples/video/index.html (Note: The video shows and gyroscope works, but the gorillas do not move around).
I wonder why this does not work as there are other video players that do support 360 videos on iOS, examples:

Videojs-panorama
JWPlayer

My questions:

Is there a specific reason for the non-support?
Are there plans to support iOS going forward?

i think didn't about support issue,
that's the video do not autoplay on safari browser .

Is it true that all you need is to add a play()  in order to watch the video?


Handle top level lists

Fixes issue described in #114.
Uses remap from the boltons package (https://boltons.readthedocs.io/en/latest/iterutils.html#boltons.iterutils.remap) to iterate over the dictionary of variables and recursively look into lists to call dict() on each model within each list.
Also removes Nones from the dictionary to handle optional inputs.

Also removes Nones from the dictionary to handle optional inputs.

This isn't strictly necessary so happy to remove this bit.

Also removes Nones from the dictionary to handle optional inputs.

This isn't strictly necessary so happy to remove this bit if you would prefer.

If I understand correctly, this change makes it impossible to pass None/null explicitly?

Yes. I had to do a bit of digging today to get my head around how this should work.
The issue I'm having is that I have a lot of optional arguments and optional fields within them.
The current generated code sets every missing value to None, and then sends "field": null.
Our GraphQL server interprets missing arguments/fields as semantically different to null values and therefore I want to remove them from the json. I spent some time today wondering if this is actually a bug in our server/the framework we are using, but the GraphQL language spec makes it clear that they are semantically different.
I had assumed that None representing a missing optional argument would be far more common than explicitly sending null, and that this change would result in payloads that are closer to what a user intended (and therefore should be the default behaviour). That might not be the case and if so we can raise an issue for null handling in general and discuss there. I have a similar fix in base_model.py for stripping nulls which would raise the same discussion points.

I'm not a fan of choosing one of these options. I wonder if it is a way to distinguish the default None from user-provided None. Maybe we should use something different for optional arguments and input fields, (graphql.Undefined?) and then remove only this from the payload. @rafalp What do you think?

I was going to point you at this discussion from openapi-python-client which explains their "Unset" concept, however that points at some Pydantic features that may save a lot of trouble here.
Should I create a new issue for the null handling as it clearly requires further discussion?

Should I create a new issue for the null handling as it clearly requires further discussion?

That would be great, thank you!

I can squash this to remove the commits related to null values before merging.

Deleting nulls from input values is wrong. As mentioned above in Python GraphQL "key": None and no key in input have different meaning. This behavior was implemented in Graphene 3 and it caused massive pain to the developers. Adding this behavior to client will make it useless for Saleor GraphQL, which is our primary use case.
Three solutions come to mind:

Check if there's way to get information from model which fields were initialized from user given values.
Undefined type for no presence of field (there's one in GraphQL utils, but we could re-export it for convenience or define our own).
Inputs as typed dicts.

Deleting nulls from input values is wrong. As mentioned above in Python GraphQL "key": None and no key in input have different meaning. Behavior normalizing those two paths into one was implemented in Graphene 3 and it caused massive pain to the developers. This behavior in client will make it problematic for Saleor GraphQL, which is our primary use case.
Three solutions come to mind:

Check if there's way to get information from model which fields were initialized from user given values.
Undefined type for no presence of field (there's one in GraphQL utils, but we could re-export it for convenience or define our own).
Inputs as typed dicts.

I've raised a new issue for the null discussion (#121), but it looks like Pydantic can handle unset values within the model. We just need a solution for the top level variables which can probably be done with a marker such as Undefined

Please rebase on main branch


Report when Black errors out

Currently, running Black via VS Code (specifically, it's set as my default formatter in the Python extension, so I just hit Alt+Shift+F) will produce no visible result in two cases:

The code is already well-formatted, or
Black failed to reformat the code (in which case it will just leave it untouched, unlike some other tools like autopep8 which in these cases will produce reformatted, but broken, code).

These two situations are exact opposites: the first one is fully normal, the second one is really bad. There should really be some difference in behaviour between these two -- a warning or an error should pop up if Black failed to run.

Hi @ooa113y , thank you for the suggestion! We have marked this issue as "needs decision" to make sure we have a conversation about your idea. We plan to leave this feature request open for at least a month to see how many 👍 votes your idea gets to help us make our decision.

#8763

@luabud if you run black with a failure and check the return code we can see if it differs from the error for when there's code that needs formatting (if it does that when you run w/o --check) then that will tell us if idea 2 is possible.
As for idea 1, I probably left it out as it spits out so much stuff that it makes using the Output view really hard. Perhaps we should just finally break out all the major areas of the extension into separate channels so that formatting can be hard to read but it's isolated from other things like interpreter discovery and linting commands?

I stumbled across that issue as I too find it irritating that there is no difference in black correctly formatting a file and erroring out.

I think there are two problems here:
1. Because we run black with the --quiet flag, the error is not even shown on the Output window. @brettcannon do you remember why it was decided to use --quiet with black? We don't do that with autopep8.

Actually running black with or without the --quiet flag does return an error output (just not so many emojis)
black .\view.py
error: cannot format view.py: Cannot parse: 270:26:         artifact_filename = build.get("artifacts_file").get("filename")
Oh no! 💥 💔 💥
1 file failed to reformat.

vs
black --quiet .\view.py
error: cannot format view.py: Cannot parse: 270:26:         artifact_filename = build.get("artifacts_file").get("filename")

From the Readme.md of psf/black

-q, --quiet                     Don't emit non-error messages to stderr.
Errors are still emitted; silence those with
2>/dev/null.

Maybe this helps?

We now support https://marketplace.visualstudio.com/items?itemName=ms-python.black-formatter extension which may not have this problem, please try it out and let us know if it's still an issue.

This feature is now available in the https://marketplace.visualstudio.com/items?itemName=ms-python.black-formatter extension.
You can use black-formatter.showNotification to enable notifications on error. Any error or warnings are always reported to the logs.


pull

What type of PR is this?

What does this PR do / why is it needed ?

Which issue(s) this PR fixes:

Fixes #
Other notes for reviewers:
Does this PR introduce a user-facing change?

pull

rebase


Optical Character Recognition

Feature request
I was thinking it would be awesome if you could scan text or documents with your phone's camera.
This is not at all important, since there seem to be solutions to doing in the browser already:

Tensorflow.js has a demo
Tesseract.js seems to be popular too.

Motivation
Allowing a device to 'read' the world could be a valuable building block. It could simplify creating interesting pipelines when combined with the existing Transformers.js features. E.g.

Detect signs and other objects in public space, OCR them, translate them, and speak them out loud, or show the translation on top of the image. This could be useful during travel, or for people who aren't great at reading.
Aid in scanning in documents, which can them be analyzed with the endless other text manipulation features. Create summaries of texts, scan for safe/good options in restaurant menus, save businesscards, read licenseplates, create games, etc.

Your contribution
I can help research models, aid in testing and design/build a demo.

Seems it's already supported through:
https://huggingface.co/Xenova/donut-base-finetuned-docvqa
More details:
https://huggingface.co/naver-clova-ix/donut-base-finetuned-docvqa


[uss_qualifier] f3548 - GEN0300 verify interactions with interoperability test instance

Extends the f3548 aggregate check to look for queries that interact with the test instance in order to validate requirement n° GEN0300.
If no interaction is found, the check is skipped.
Otherwise, the check is reasonably lax: for each possible interaction with the test endpoint, it is enough to have a single successful query to pass.

@BenjaminPelletier this approach is slightly different than what I remember discussing ~10 days ago (which I wrote down as "find a check that interacts with the test instance and mark GEN0300 as checked).
As I ran into the QueryType constants that seem include the interactions with the test instance I opted to add a test case to the aggregate checks at the end.

@BenjaminPelletier this approach is slightly different than what I remember discussing ~10 days ago (which I wrote down as "find a check that interacts with the test instance and mark GEN0300 as checked").
As I ran into the QueryType constants that seem to include the interactions with the test instance I opted to add a test case to the aggregate checks at the end.

This seems like a reasonable approach to me, and probably superior to what I was originally thinking.  GEN0300 says the USS needs to provide a test instance and I think this approach effectively identifies when that requirement isn't met with a fairly small amount of implementation.


Node-red crashes on websocket request

Current Behavior
when running a flow on node red creating a websocket i crashed node-red see log below. on linux im not able to get this running. on windows no issues however.
27 Aug 22:20:02 - [red] Uncaught Exception:
27 Aug 22:20:02 - Error: Error send Key
at /home/pi/.node-red/node_modules/samsung-tv-control/lib/samsung.js:55:23
at WebSocket. (/home/pi/.node-red/node_modules/samsung-tv-control/lib/samsung.js:289:17)
at WebSocket.emit (events.js:314:20)
at Receiver.receiverOnError (/home/pi/.node-red/node_modules/ws/lib/websocket.js:951:13)
at Receiver.emit (events.js:314:20)
at errorOrDestroy (internal/streams/destroy.js:108:12)
at onwriteError (_stream_writable.js:418:5)
at onwrite (_stream_writable.js:445:5)
at Receiver.startLoop (/home/pi/.node-red/node_modules/ws/lib/receiver.js:152:5)
at Receiver._write (/home/pi/.node-red/node_modules/ws/lib/receiver.js:78:10)
at doWrite (_stream_writable.js:403:12)
at writeOrBuffer (_stream_writable.js:387:5)
at Receiver.Writable.write (_stream_writable.js:318:11)
at TLSSocket.socketOnData (/home/pi/.node-red/node_modules/ws/lib/websocket.js:1045:35)
at TLSSocket.emit (events.js:314:20)
at addChunk (_stream_readable.js:297:12)
nodered.service: Main process exited, code=exited, status=1/FAILURE
Expected Behavior
No response
Steps To Reproduce
No response
Example flow
paste your flow here

Environment

Node-RED version: latest
Node.js version: latest
npm version: latest
Platform/OS: ubuntu 20 server
Browser: chrome

After lot of testing i found the "problem" seems that the Samsung TV does not allow connections from different subnets :( i use a lot of VLAN's. i have no setup node-red on same subnet as TV and it is now working.

It would still be helpful if you were to raise an issue against the samsung-tv node. It should be handling that error without crashing Node-RED.

Just did on the right contrib thanks....


Argument error in attention layer initialization.

I am running the provided plot notebook in the attention model. when I run this line.
import experiments.attention_models.baseline_model
reload(experiments.attention_models.baseline_model)
from experiments.attention_models.baseline_model import get_network

model_path = os.path.join(ROOT, "experiments/attention_models/models/mnist_att_params2.gz")
network = get_network(model_path, disable_reinforce=True)

i get this error:
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-6-7d813e2daac2> in <module>()
      4
      5 model_path = os.path.join(ROOT, "experiments/attention_models/models/mnist_att_params2.gz")
----> 6 network = get_network(model_path, disable_reinforce=True)

D:\Python Directory\winPython 2.7\deepy\experiments\attention_models\baseline_model.py in get_network(model, std, disable_reinforce, random_glimpse)
    211     """
    212     network = NeuralClassifier(input_dim=28 * 28)
--> 213     network.stack_layer(AttentionLayer(std=std, disable_reinforce=disable_reinforce, random_glimpse=random_glimpse))
    214     if model and os.path.exists(model):
    215         network.load_params(model)

D:\Python Directory\winPython 2.7\deepy\experiments\attention_models\baseline_model.py in __init__(self, activation, std, disable_reinforce, random_glimpse)
     23         self.gaussian_std = std
     24         #super(AttentionLayer, self).__init__(activation)
---> 25         super(AttentionLayer, self).__init__(10, activation)
     26
     27     def initialize(self, config, vars, x, input_n, id="UNKNOWN"):

TypeError: __init__() takes at most 2 arguments (3 given)

I error occurs when the AttentionLayer class initializes its parent class 'NeuralLayer'
super(AttentionLayer, self).__init__(10, activation)
I looked at the NeuralLayer implementation and found that it indeed takes only one argument besides self.
class NeuralLayer(object):

    def __init__(self, name="unknown"):

Can you please look into this error.

For attentional mechanism please see https://github.com/zomux/neuralmt.
Sorry, as in version 2.0, many major APIs are redeigned, so some codes for LM do not work. I will update them later


Consolidated proxy-related configuration options and constants under Kamal::Configuration::Proxy

Currently, some configuration options for kamal-proxy are stored in the Kamal::Configuration::Proxy, but some are in the Kamal::Configuration itself, and prefixed with proxy_. This inconsistency leads to a harder to understand codebase, but also limits refactoring options.
For example, making proxy image configurable would allow to test Kamal with a custom kamal-proxy image.

Thanks for this PR, @kpumuk!
There's two different types of configuration here - app level deploy time configuration for the proxy (i.e. everything under the proxy key in deploy.yml) and then static boot time configuration for the proxy.
Maybe the stuff that you are moving from Kamal::Configuration could go in a new Kamal::Configuration::KamalProxy class instead?

Thanks for the response. It is a very interesting problem to deal with the nature of Kamal proxy, that is at the same time global (as in shared between multiple processes) and app-specific (as in routing configuration).
I wanted to be able to configure custom container image, but now that you pointed out the difference, it might be a footgun to allow to do so. E.g. if somebody has two apps deployed, and one has a config like:
proxy:
  image: doodle/kamal-proxy:v15.0.0
  ssl: true

... apps will fight against each other on kamal proxy reboot. On the other hand, currently one needs to edit source code of Kamal to test against a custom build of kamal-proxy.

There's a kamal proxy boot_config command for the global config. It stores it on each server we are deploying to, since as you say it may be shared across multiple apps.
We could maybe allow the image to be set there,
We also have a check for the kamal-proxy version to ensure it is new enough. I guess we could allow a different image but keep the same version check. If you were developing a custom version you'd want to ensure that it was up to date with the version kamal requires and you should match version numbers with basecamp/kamal-proxy.

Will close this one for reconsideration. Thank you for you input, @djmb


Undistortion of ColorCamera node

Hi, recently with new update 2.28.0, there is support of undistortion. However, it is only supported for Camera node, not for ColorCamera right?
I tried the Camera node, but due to the lack of documentation about possible resolution etc. I cannot use it, because the resolution is set directly and not via ISP scale. Moreover, if I set the same resolution for Camera node like the one after ISP scaling in ColorCamera, it ouputs errors.
My question and wish is, is it possible to have some description of such usage? Or is it somewhere already?
Thanks a lot.

However, it is only supported for Camera node, not for ColorCamera right?

Correct.
You should be able to set a size for each output (setSize(), setVideoSize(), ..) which will optimally set the resolution and downscaling to achieve that size.
There is currently no in depth description; I am thinking of making a calculator for it.
Thanks,
Jaka


Running TGI docker container on a Mac M1

Feature request
Hi all,
has anyone experience with getting a LLM supported by the Hugging Face module "text-generation-inference" up and running in a docker container locally on a Mac M1?
I was planning to deploy a Llama-2-7B-Chat on my local device. Although the documentation only mentions NVIDIA GPUs I was wondering whether this is also possible on my Mac? A guy on YouTube mentioned that he succeeded, but unfortunately didn't provide any details except of adding the platform flag to the docker run command.
My command in the terminal so far is:
docker run --gpus all --shm-size 1g -p 3000:80 -v /llama2:/data --platform linux/amd64 ghcr.io/huggingface/text-generation-inference:latest --model-d meta-llama/Llama-2-7b-chat --num-shard 1
But it seems that it doesn't find the right GPU. The NVIDIA drivers require to have the NVIDIA Container Toolkit installed. Not sure if something similar exists for M1? Or is it because of an incorrect implementation on my part?
docker: Error response from daemon: could not select device driver "" with capabilities: [[gpu]]
Motivation
If it's because of an incorrect implementation on my part I would really appreciate any comment/ idea that points me into the right direction; if it's just not possible yet I think it would be a nice feature as there are probably a lot of mac users out there that would benefit from a supported solution
Your contribution
Not sure how, but of course I'd love to contribute in any way possible

I don't think M1 GPU are exactly CUDA devices.
This is not intended to work on M1/M2 and probably will not work. Unless you have an external GPU connected by some means.
Closing this as M1 is not officially supported as a target and probably won't be for the foreseeable future.
There are other projects more targeted for M1 (by huggingface or others).
I'll let others comment as I might not be the most up-to-date with this.


FTD option does not work for DPoE or PacketCable always display DOCSIS FTD

Found this in v2.1

Fixed in v2.1.1


Change in a recipe of fruit salad

Dear collaborators,
I would like to change the ingredients of fruit salad to make it more healthy and tasty.

recipe approved


Is there a way to manage z-index with this?

I am using this to make draggable windows in a project. i need to bring a window to the front when it is being dragged, is there a way to do this with z-index?

Yes,
You can use the onpositionchange event handler, e.g.
....
            onPosChanged: function(positionDiff, absolutePosition, event) {
                // el is the element that been dragged, you can find it using event.target or ref
                el.style.zIndex = "1";
            }


Black video from library after compress

Describe the bug
Some .MP4 videos selected from the library sent by WhatsApp. After select it turned fully black with audio.

Environment

Device: [iPhone 11 Pro]
OS: [13.4]
Xcode Version [11.4]
Swift Version [5.0]

Installation Type

Cocoapods

@brhoomjs  have you tried on a real device? Seems the issue reproduces only on a simulator

Same issue faced when I tried to pick the boomerang videos with 3 second

@xxKRASHxx  This issue happens on a real device as well.

@brhoomjs have you tried on a real device? Seems the issue reproduces only on a simulator

For all devices with all versions.

I have the same problem with real devices. Mainly I have seen it to happen in videos that were recorded on Android device. not always. but very frequently with this set up.

I have the same problem with real devices. Mainly I have seen it to happen in videos that were recorded on Android device. not always. but very frequently with this set up.

Thanks :)


Include BeeKeeb's Smiling Orca

Hey there, thanks for this great repo!
It could be nice to include those as well: https://www.thingiverse.com/thing:5425764

Oh hadn't seen them released outside of the beekeeb store before now!
Will definitely include them too, been wanting to try them myself and I just so happen to have a spool of nylon filament on the shelf 😊
I made the repo for myself to keep track of keycaps I found and tried out, happy to see other people find it useful!

Added Smiling Orca!


travis: Fix NODEJS_VERSION deployment check

The check didn't actually evaluate the NODEJS_VERSION variable in the
conditional. In fact, it assigned to the variable.

Is == the right syntax here? That’s what I expected when I first saw this change so I looked atthe Travis docs butthe examples there seem to use single =

So = is the operator defined by POSIX. == is an extension supported by bash. https://unix.stackexchange.com/questions/168284/unix-test-when-to-use-eq-vs-vs-in-test-commands
Travis evaluates its conditionals using the bash test command:
condition: You may set bash condition with this option. This must be a string value, which will be pasted into a bash expression of the form if [[ <condition> ]]; then <deploy>; fi It can be complex, but there can be only one. For example, $CC = gcc.

That's why it supports the non-POSIX compliant ==. I can see an argument for switching all of the Travis config to use =. What do you think? I'm fine either way.

Ehh I didn't realize there were other spots where we used this.  If all of our existing code uses ==, let's just leave it.


No live Graph View on dendron-site

Is it possible to show interactive Graph View on dendron's public site?
I understand there's a discussion https://github.com/dendronhq/dendron/discussions/615 around this topic, but there could be something easily borrowed from D3.js https://observablehq.com/@d3/force-directed-graph.

I guess this one is more coherent to Dendron's purpose:
https://observablehq.com/@icarofonseca/design-theory-mindmap

When it comes to graph rework improvements, that discussion is the perfect place to add comments on the future of how graph view may continue to be implemented and capable of embedding.
I've added a comment with your suggested resource links here:

https://github.com/dendronhq/dendron/discussions/615#discussioncomment-2192455

Feel free to add additional information in response to that discussion comment, or add new comments to the discussion. We value community feedback on how the implementation can be done! :)
Since issues in this repo are meant for the wiki/docs for Dendron, rather then Dendron features/enhancements (which can be raised at https://github.com/dendronhq/dendron), I'll be closing this issue.


Uncaught TypeError

PHPMD version: 2.8.2
PHP Version: 7.3.17
Installation type: composer
Operating System / Distribution & Version:

I am running phpmd in a Azure DevOps pipeline using a ubuntu image, and I am getting this error. The same command runs fine on my local machine, using the same composer install.
PHP Fatal error:  Uncaught TypeError: Argument 1 passed to PDepend\Source\AST\AbstractASTCallable::addChild() must implement interface PDepend\Source\AST\ASTNode, null given, called in /home/vsts/work/1/s/src/web/vendor/pdepend/pdepend/src/main/php/PDepend/Source/Language/PHP/PHPParserVersion70.php on line 183 and defined in /home/vsts/work/1/s/src/web/vendor/pdepend/pdepend/src/main/php/PDepend/Source/AST/AbstractASTCallable.php:143
Stack trace:
#0 /home/vsts/work/1/s/src/web/vendor/pdepend/pdepend/src/main/php/PDepend/Source/Language/PHP/PHPParserVersion70.php(183): PDepend\Source\AST\AbstractASTCallable->addChild(NULL)
#1 /home/vsts/work/1/s/src/web/vendor/pdepend/pdepend/src/main/php/PDepend/Source/Language/PHP/AbstractPHPParser.php(1265): PDepend\Source\Language\PHP\PHPParserVersion70->parseCallableDeclarationAddition(Object(PDepend\Source\AST\ASTFunction))
#2 /home/vsts/work/1/s/src/web/vendor/pdepend/pdepend/src/main/php/PDepend/Source/Language/PHP/AbstractPHPParser.php(1177): PDepend\Source\Language\PHP\AbstractPHPPars in /home/vsts/work/1/s/src/web/vendor/pdepend/pdepend/src/main/php/PDepend/Source/AST/AbstractASTCallable.php on line 143

PHP 7.3.17-1+ubuntu18.04.1+deb.sury.org+1 (cli) (built: Apr 19 2020 07:44:26) ( NTS )
Copyright (c) 1997-2018 The PHP Group
Zend Engine v3.3.17, Copyright (c) 1998-2018 Zend Technologies
    with Zend OPcache v7.3.17-1+ubuntu18.04.1+deb.sury.org+1, Copyright (c) 1999-2018, by Zend Technologies
    with Xdebug v2.9.3, Copyright (c) 2002-2020, by Derick Rethans

@johnwc If I read your stack trace I see that it is a bug in PDepend. But still it is strange. You wrote that you instal it with composer, do you use a lock file or is it possible that the used versions are not the same? Is your local PHP version the same as the one on Azure?

@tvbeek  I do have a lock file, so it should be the same as local. Is there a way to get phpmd to verbose or debug log?

vendor/pdepend/pdepend/src/main/php/PDepend/Source/Language/PHP/AbstractPHPParser.php(1265): PDepend\Source\Language\PHP\PHPParserVersion70->parseCallableDeclarationAddition(Object(PDepend\Source\AST\ASTFunction))

This means it fails trying to parse a closure such as function () {...} I don't know if it's a big clue, it depends how often you use closures.

If you install PHPMD with composer you also have PDepend in vendor/bin/pdepend, With that you can try to run pdepend to see where it failed, example command:
vendor/bin/pdepend --debug --summary-xml=summary.xml path/to/scan

Is there a way to get phpmd to log what directory or file it's sending to pdepend, so that I can pass that to pdepend by itself like you mentioned?


Django >= 4.0  error for RP (on OpenSuse 15.3)

Hello,
not Django/Python expert.
Trying to follow instruction here
https://github.com/italia/spid-cie-oidc-django/blob/main/docs/SETUP.md
for a basic Spid OIDC RP,
I'm getting this:
pip install spid-cie-oidc
Defaulting to user installation because normal site-packages is not writeable
Collecting spid-cie-oidc
  Using cached spid_cie_oidc-0.4.16-py3-none-any.whl (436 kB)
Collecting pydantic
  Using cached pydantic-1.9.0-cp36-cp36m-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (11.0 MB)
ERROR: Could not find a version that satisfies the requirement Django>=4.0 (from spid-cie-oidc) (from versions: 1.1.3, 1.1.4, 1.2, 1.2.1, 1.2.2, 1.2.3, 1.2.4, 1.2.5, 1.2.6, 1.2.7, 1.3, 1.3.1, 1.3.2, 1.3.3, 1.3.4, 1.3.5, 1.3.6, 1.3.7, 1.4, 1.4.1, 1.4.2, 1.4.3, 1.4.4, 1.4.5, 1.4.6, 1.4.7, 1.4.8, 1.4.9, 1.4.10, 1.4.11, 1.4.12, 1.4.13, 1.4.14, 1.4.15, 1.4.16, 1.4.17, 1.4.18, 1.4.19, 1.4.20, 1.4.21, 1.4.22, 1.5, 1.5.1, 1.5.2, 1.5.3, 1.5.4, 1.5.5, 1.5.6, 1.5.7, 1.5.8, 1.5.9, 1.5.10, 1.5.11, 1.5.12, 1.6, 1.6.1, 1.6.2, 1.6.3, 1.6.4, 1.6.5, 1.6.6, 1.6.7, 1.6.8, 1.6.9, 1.6.10, 1.6.11, 1.7, 1.7.1, 1.7.2, 1.7.3, 1.7.4, 1.7.5, 1.7.6, 1.7.7, 1.7.8, 1.7.9, 1.7.10, 1.7.11, 1.8a1, 1.8b1, 1.8b2, 1.8rc1, 1.8, 1.8.1, 1.8.2, 1.8.3, 1.8.4, 1.8.5, 1.8.6, 1.8.7, 1.8.8, 1.8.9, 1.8.10, 1.8.11, 1.8.12, 1.8.13, 1.8.14, 1.8.15, 1.8.16, 1.8.17, 1.8.18, 1.8.19, 1.9a1, 1.9b1, 1.9rc1, 1.9rc2, 1.9, 1.9.1, 1.9.2, 1.9.3, 1.9.4, 1.9.5, 1.9.6, 1.9.7, 1.9.8, 1.9.9, 1.9.10, 1.9.11, 1.9.12, 1.9.13, 1.10a1, 1.10b1, 1.10rc1, 1.10, 1.10.1, 1.10.2, 1.10.3, 1.10.4, 1.10.5, 1.10.6, 1.10.7, 1.10.8, 1.11a1, 1.11b1, 1.11rc1, 1.11, 1.11.1, 1.11.2, 1.11.3, 1.11.4, 1.11.5, 1.11.6, 1.11.7, 1.11.8, 1.11.9, 1.11.10, 1.11.11, 1.11.12, 1.11.13, 1.11.14, 1.11.15, 1.11.16, 1.11.17, 1.11.18, 1.11.20, 1.11.21, 1.11.22, 1.11.23, 1.11.24, 1.11.25, 1.11.26, 1.11.27, 1.11.28, 1.11.29, 2.0a1, 2.0b1, 2.0rc1, 2.0, 2.0.1, 2.0.2, 2.0.3, 2.0.4, 2.0.5, 2.0.6, 2.0.7, 2.0.8, 2.0.9, 2.0.10, 2.0.12, 2.0.13, 2.1a1, 2.1b1, 2.1rc1, 2.1, 2.1.1, 2.1.2, 2.1.3, 2.1.4, 2.1.5, 2.1.7, 2.1.8, 2.1.9, 2.1.10, 2.1.11, 2.1.12, 2.1.13, 2.1.14, 2.1.15, 2.2a1, 2.2b1, 2.2rc1, 2.2, 2.2.1, 2.2.2, 2.2.3, 2.2.4, 2.2.5, 2.2.6, 2.2.7, 2.2.8, 2.2.9, 2.2.10, 2.2.11, 2.2.12, 2.2.13, 2.2.14, 2.2.15, 2.2.16, 2.2.17, 2.2.18, 2.2.19, 2.2.20, 2.2.21, 2.2.22, 2.2.23, 2.2.24, 2.2.25, 2.2.26, 2.2.27, 3.0a1, 3.0b1, 3.0rc1, 3.0, 3.0.1, 3.0.2, 3.0.3, 3.0.4, 3.0.5, 3.0.6, 3.0.7, 3.0.8, 3.0.9, 3.0.10, 3.0.11, 3.0.12, 3.0.13, 3.0.14, 3.1a1, 3.1b1, 3.1rc1, 3.1, 3.1.1, 3.1.2, 3.1.3, 3.1.4, 3.1.5, 3.1.6, 3.1.7, 3.1.8, 3.1.9, 3.1.10, 3.1.11, 3.1.12, 3.1.13, 3.1.14, 3.2a1, 3.2b1, 3.2rc1, 3.2, 3.2.1, 3.2.2, 3.2.3, 3.2.4, 3.2.5, 3.2.6, 3.2.7, 3.2.8, 3.2.9, 3.2.10, 3.2.11, 3.2.12)
ERROR: No matching distribution found for Django>=4.0 (from spid-cie-oidc)

Is 4.0+ really necessary ?
Looked some official repos but seems not available (Opensuse 15.3 here)
Thanks.
Diego

Please use a virtualenv or install It via pip as root
It works also with Django 3 but we wont support it

Hi. Tried something like what listed on page but not working.
is it possible to show here or there the exact commands?
I did:

virtualenv -p python3 env (as root)
git clone https://github.com/peppelinux/spid-cie-oidc-django
git clone https://github.com/peppelinux/spid-cie-oidc-django
cd spid-cie-oidc-django
pip install --upgrade pip (--> Requirement already satisfied: pip in /home/diegone/GIT7/env/lib/python3.6/site-packages (21.3.1))
pip install -e .   (--> ERROR: Could not find a version that satisfies the requirement Django>=4.0 (from spid-cie-oidc) )  (same output)

Sorry but not a python expert at all ,and was looking for a set and forget spidOIDC RP

Also tried (first) with these as root
:

pip install spid-cie-oidc
pip3 install spid-cie-oidc

Please don't use python3.6, unfortunately it is EOL nowaday.
At least py37
it seems that your python interpreter is deprecated for Django 4.
Please type python and check what version you're using

You can even use the docker-compose to have a working platform for non-development need

output for pyhton:
python
Python 3.6.15 (default, Sep 23 2021, 15:41:43) [GCC] on linux
Type "help", "copyright", "credits" or "license" for more information.
btw, there's some how-to for the docker option?

https://github.com/italia/spid-cie-oidc-django#docker-compose

Hi Giuseppe.
Very kind to ask. I meant to reply and try your suggestions these days, but
had a reallocation of tasks and could not proceed, even if interested.
I may retry in a couple weeks, possibly.
One question though:
When is the final 1.0 or at least.. the minimal, normative-bound and valid
Milestone due?
Am I correct saying that I read it was planned, like a couple of weeks from
now?
Thanks for you interest.
Diego
Il mar 22 mar 2022, 17:43 Giuseppe De Marco @.***> ha
scritto:

@longstoned https://github.com/longstoned did you find the way to get
the demo running for your needs?
—
Reply to this email directly, view it on GitHub
https://github.com/italia/spid-cie-oidc-django/issues/169#issuecomment-1075380113,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYKE6H7Y6W5K2XF7ODBQK7TVBH2EXANCNFSM5RF7MJ3A
.
You are receiving this because you were mentioned.Message ID:
@.***>

This 0.4.0 is stable, we'll go ahead this week for 0.5.0.
the 0.6.0 will only have QaD tests if AgID and IPZS defines a spec document, otherwise we'll stay on hold
All to say that we have a stable release without a compliant SSO/ACR-refresh-token policy (https://github.com/italia/spid-cie-oidc-django/issues/123) and this bug (https://github.com/italia/spid-cie-oidc-django/issues/170)
all the other are minor bugs. the v0.5.0 will fix all these and improve the documentation

Hi,
we consolidated the docker-compose with all the example projects up and running


memory unit formatter

https://github.com/PragTob/benchee/issues/121

Shockingly we don't seem to have functions yet to scale units down or  convert from one unit to the other :scream:
That can be done in a separate ticket though :)


regression disable runtime filter for pipelineX test cases

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


location and geocoding update

Update on #97
Issue Addressed

[x] Map shows in each user's card
[x] Map is centered around the location that is registered in the user's json and a marker is displayed as well
[x] Location is being interpolated using Google's Geocoding services, it is so much faster than Ninjas and more reliable. Ninjas is erroneous with some locations.
[x] Map is rendered in full size and scroll bar appears when the card's default size isn't enough to display it all.

@sherikovic excellent job on the recent updates! I noticed that you've made it possible for users to exit the user card by simply clicking outside of the box. Additionally, the image lighting issue has been resolved by removing the blue border that made it appear clickable. The elimination of the satellite and map buttons has resulted in a more aesthetically pleasing Google Maps display with increased real estate. I'm very impressed that you were able to effectively fetch data from the JSON file, convert it to longitude and latitude, and accurately plot it on the Google Maps API. We'll create a separate issue for the flyover effect, as this is a future UI enhancement.
Our goal is to always offer new UI/UX enhancements for developers to enhance their skills and showcase them as proof. Great work! I'll proceed with merging it into the main and double-check that the API key is correctly fetching on the main.

This feature now closes the issue #97


More documentation for the app?

Hi Shawn, thank you for making this app.
One issue I am facing is the lack of documentation to understand your code. Though my main query is this:
How do you receive and process the H264 video stream from the Raspberry pi, and which java files contain the functionality for it?
What kind of libraries are used?
What does player.py do?

The video activity (VideoActivity.java) creates the video fragment (VideoFragment.java), which is where all the real work is done. The video fragment uses one of the reader classes (usually TcpIpReader) and the main loop starts with:
while (!isInterrupted())
{
    int len = reader.read(buffer);

The TcpIpReader class is given an IP address and a port and it opens a socket on that address/port. The main loop continuously reads the stream from the socket, breaks the stream up into NAL units, and feeds the NAL units into the decoder, which then feeds the decoded frames directly into the surface it has been configured with.
The only library being used is ca.frozen.library, which is another one of my open source projects. It's at:
https://github.com/ShawnBaker/Library

player.py was intended to run on an RPi connected directly to a TV where a remote control would be used to select which stream (channel) you want to view. I didn’t finish or test it.


The test files in the sql folder are missing the _test suffix

Environment

Ecto SQL: master

Yes, that's because they do not run by themselves, they need to be required by the specific adapters and the adapter may not support all of them!


chore(package): update github-repository-provider to version 2.1.13

Coverage decreased (-0.1%) to 81.713% when pulling db61df460480fc1f57adc51871259d45e7e2e791 on greenkeeper/github-repository-provider-2.1.13 into 7798498b0c9681cc3b0537997d0fcc7aa19db6ab on master.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


unknown variant rich_text

I'm hitting some issues when parsing valid blocks sent back from the Slack API because they include type of rich_text which doesn't appear to be supported. And I'm struggling to find it anywhere in the Slack blocks documentation, so it might be undocumented?
Specifically, the error:
---- should_send_acknowledgement_of_event_messages stdout ----
thread 'should_send_acknowledgement_of_event_messages' panicked at 'called `Result::unwrap()` on an `Err` value: Error("unknown variant `rich_text`, expected one of `section`, `divider`, `image`, `actions`, `context`, `input`, `header`, `file`", line: 0, column: 0)', client/tests/web_sockets.rs:97:27

And the Slack message, which is a response from the api/chat.postMessage action, some identifying details omitted:
{
  "ok": true,
  "channel": "<ommitted>",
  "ts": "1686086970.626639",
  "message": {
    "bot_id": "<ommitted>",
    "type": "message",
    "text": "foobar",
    "user": "<ommitted>",
    "ts": "1686086970.626639",
    "app_id": "<ommitted>",
    "blocks": [
      {
        "type": "rich_text",
        "block_id": "<ommitted>",
        "elements": [
          {
            "type": "rich_text_section",
            "elements": [
              {
                "type": "text",
                "text": "foobar"
              }
            ]
          }
        ]
      }
    ],
    "team": "<ommitted>",
    "bot_profile": {
      "id": "<ommitted>",
      "app_id": "<ommitted>",
      "name": "<ommitted>",
      "icons": {
        "image_36": "<ommitted>",
        "image_48": "<ommitted>",
        "image_72": "<ommitted>"
      },
      "deleted": false,
      "updated":<PHONE_NUMBER>,
      "team_id": "<ommitted>"
    }
  },
  "warning": "missing_charset",
  "response_metadata": {
    "warnings": [
      "missing_charset"
    ]
  }
}

Hi Mattsi! Just noticed this, I'll work on adding support for this tomorrow! :smiley:

Good stuff :+1: For what it's worth I've since discovered other undocumented parts of the Blocks API, like the link element and the rich_text_section element. They are returned by the API and work in the Block Kit Builder but aren't in the documentation.
I've been hacking together my own minimal implementation here that includes these non-documented types, if it helps as a reference: https://github.com/Mattsi-Jansky/termibot/tree/rewrite-in-rust/client/src/models/blocks


render() got an unexpected keyword argument 'renderer'

Hi,
I am using django 2.2.1 and installed your latest version. However the Django admin throws following exception when I try to add a new model, which uses HTMLField:
render() got an unexpected keyword argument 'renderer'

I tried but not it's working fine
Python: 3.7
Django: 2.2.1
Django-TinyMCE: 2.8.0
I have tried with fresh setup. Just Django & Django-TinyMCE and sample model as below:
from django.db import models
from tinymce.models import HTMLField

class MyModel(models.Model):
    content = HTMLField()


Fix #4675: Handle ES2021 in the ClosureLinkerBackend.

This was probably forgotten in 07b0250e62f08e0c37fac0086a373225dbd858cb.

This was pending an upgrade of Node.js on our CI. We now need at least v15 which added support for ||= and &&=. The new version is v16.15.1. Thanks to our sysadmin!

@gzm0 I had to add an additional commit "Serve all of test-suite/ for testHtmlJSDom." to make the CI pass on the more recent Node.js.


macOS Sierra Beta Unable to play music

OS:
macOS Sierra 10.12 Beta (16A270f)
Issue Descriptions:
Every time I open it, it successfully loads music, even allowing me to search for music. The problem does not occur until I attempt to play a song. When I click on a song to play it, the little audio bars start to move, but the bottom area showing the progress of the song and the play/skip buttons are all grayed out and it tells me that it couldn't play the song and skipped it.
Steps to Reproduce:
Open the app. Go to a song. Click play. It will give the error.

Unfortunately I'm going to follow the pattern of CEF and Electron issues here.  Basically until Chromium supports Sierra I can't put any effort into bugfixing it.
This appears at first glance to be a DRM MP3 issue on Sierra.  But yeah, I can't fix that Chromium need to.
Closing as Upstream and out of scope.

Thank you for your quick response. :)
On Aug 9, 2016 11:54 AM, "Samuel Attard"<EMAIL_ADDRESS>wrote:

Unfortunately I'm going to follow the pattern of CEF and Electron issues
here. Basically until Chromium supports Sierra I can't put any effort into
bugfixing it.
This appears at first glance to be a DRM MP3 issue on Sierra. But yeah, I
can't fix that Chromium need to.
Closing as Upstream and out of scope.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/MarshallOfSound/Google-Play-Music-Desktop-Player-UNOFFICIAL-/issues/1434#issuecomment-238616933,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMWqx4nE4GMCnCe9rMTTlNe2dzPO7VYEks5qeLCpgaJpZM4JgRVD
.


Minor changes

The problem
Solution provided

I sent you an invite to perl6 GitHub org that you can accept here: https://github.com/perl6/
Once you accept it, you will be able to merge this PR yourself and commit any further changes directly.

Thanks!


Disabled pentest deployment environment.

Context
The pentest environment isn't currently required and is to be removed. Before it can be removed it needs to be removed from the release pipeline code to prevent it being redeployed after its been deleted.
Changes proposed in this pull request

Commented out the pent test environment code to prevent it running in future releases. I've not removed it entirely in case its required again for the pen test in a couple of weeks.

Guidance to review
N/A
Link to Trello card
N/A
Env vars

[ ] If this PR introduces new environment variables, they have been added to all necessary parts of the Azure config based on the documentation in the README

Does this require extra config to turn off the true/false option for this env in the release pipeline workflow?

The variable in the pipeline will just become ineffective, but its presence is no controlled in the YAML file, that's done from in the DevOps GUI. We can remove it, but it we need it again in a couple of weeks I'd suggest just leaving it there for now since I can't do any harm once the pipeline code is removed.


Revert for SK version 6 bump

Motivation and Context
Plugins stopped working with bump.
Description
This reverts the bump change.
The reason for a separate PR instead of a GitHub revert PR, is that we also need to include the 403 workaround so the PR can be merged.
Contribution Checklist

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

NA


How to use regex pattern subscription?

Versions
Please specify real version numbers or git SHAs, not just "Latest" since that changes fairly regularly.
Sarama Version: master
Kafka Version: 2.2
Go Version: 1.12
Configuration
Default
Logs
[Sarama] 2019/05/29 00:08:40 Initializing new client
[Sarama] 2019/05/29 00:08:40 ClientID is the default of 'sarama', you should consider setting it to something application-specific.
[Sarama] 2019/05/29 00:08:40 ClientID is the default of 'sarama', you should consider setting it to something application-specific.
[Sarama] 2019/05/29 00:08:40 client/metadata fetching metadata for all topics from broker localhost:9092
[Sarama] 2019/05/29 00:08:40 Connected to broker at localhost:9092 (unregistered)
[Sarama] 2019/05/29 00:08:40 client/brokers registered new broker #0 at <IP_ADDRESS>:9092
[Sarama] 2019/05/29 00:08:40 Successfully initialized new client
[Sarama] 2019/05/29 00:08:40 client/metadata fetching metadata for [test.*] from broker localhost:9092
[Sarama] 2019/05/29 00:08:40 client/metadata fetching metadata for [test.*] from broker localhost:9092
[Sarama] 2019/05/29 00:08:40 Closing Client
panic: kafka server: The request attempted to perform an operation on an invalid topic.

Problem Description
Is there a way to use regex pattern in subscription? Just like in java:
void	subscribe(java.util.regex.Pattern pattern, ConsumerRebalanceListener listener)
I try the code below but got the error logs:
...
    sarama.Logger = log.New(os.Stdout, "[Sarama] ", log.LstdFlags)
    s, _ := regexp.Compile("test.*")
...
    partitionConsumer, err := consumer.ConsumePartition(s.String(), 0, sarama.OffsetNewest)
....

Although there is no built-in way to do this, you can do this quite easily:
...
        // Fetch a client.
	client, err := sarama.NewClient(brokers, cfg)
	if err != nil {
		// ...
	}

        // Fetch all available topics
	availableTopics, err := client.Topics()
	if err != nil {
		// ...
	}

        // Filter topics we care about
	var subTopics []string
	reg, _ := regexp.Compile("test.*")
	for _, t := range availableTopics {
		if reg.MatchString(t) {
			subTopics = append(subTopics, t)
		}
	}

        // Consume
	consumer, err := sarama.NewConsumerGroupFromClient(groupID, client)
	if err != nil {
		// ...
	}

	consumer.Consume(ctx, subTopics, handler)
...

Thanks for your comment.


Typo fix in master_session.cc comment

Just a typo fix.

Can one of the admins verify this patch?

Thanks for the fix!


Error when lane_id is not 0 but type is none

When I load an .xodr file where a right lane has as type 'none', and call RoadNetwork.get_roads(), I get the error:
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/home/aron/.local/lib/python3.8/site-packages/pyxodr/road_objects/network.py", line 153, in get_roads
    self._link_roads()
  File "/home/aron/.local/lib/python3.8/site-packages/pyxodr/road_objects/network.py", line 123, in _link_roads
    road._link_lane_sections()
  File "/home/aron/.local/lib/python3.8/site-packages/pyxodr/road_objects/road.py", line 420, in _link_lane_sections
    lane_section._link_lanes()
  File "/home/aron/.local/lib/python3.8/site-packages/pyxodr/road_objects/lane_section.py", line 239, in _link_lanes
    for lane in self.lanes:
  File "/home/aron/.local/lib/python3.8/site-packages/pyxodr/road_objects/lane_section.py", line 170, in lanes
    lanes = self.left_lanes + self.right_lanes
  File "/usr/lib/python3.8/functools.py", line 967, in __get__
    val = self.func(instance)
  File "/home/aron/.local/lib/python3.8/site-packages/pyxodr/road_objects/lane_section.py", line 165, in right_lanes
    return self.__get_lanes_by_orientation(LaneOrientation.RIGHT)
  File "/home/aron/.local/lib/python3.8/site-packages/pyxodr/road_objects/lane_section.py", line 111, in __get_lanes_by_orientation
    if lane_obj.type not in self.ignored_lane_types:
TypeError: 'in <string>' requires string as left operand, not NoneType

EDIT: This only happens when I set ignored_lane_types='shoulder', if I do not specify lanes to ignore this is no issue.
The specification of this lane in the xodr file is given below, and if I change the type from "none" to something, the error is not given. The xodr file is generated by Matlab RoadRunner, so I guess this is an artefact on that side. I have solved it for now by ignoring the lane when it's type is None, but I'm not sure how this influences the outcome.
<lane id="-1" type="none" level="false">
<width sOffset="0.0000000000000000e+0" a="5.0000000000000001e-3" b="0.0000000000000000e+0" c="0.0000000000000000e+0" d="0.0000000000000000e+0"/>
<width sOffset="1.0977543679446107e+1" a="5.0000000000000001e-3" b="0.0000000000000000e+0" c="0.0000000000000000e+0" d="0.0000000000000000e+0"/>
<roadMark sOffset="0.0000000000000000e+0" type="none" material="standard" color="white" laneChange="none"/>
<roadMark sOffset="2.0061081095444564e+0" type="none" material="standard" color="white" laneChange="none"/>
<roadMark sOffset="1.0034897775404445e+1" type="none" material="standard" color="white" laneChange="none"/>
<speed sOffset="0.0000000000000000e+0" max="3.5000000000000000e+1" unit="mph"/>
<userData code="vectorLane">
<vectorLane sOffset="0.0000000000000000e+0" laneId="{afee3bff-c14b-4b80-8c5a-5c42e23b817e}" travelDir="undirected"/>
<vectorLane sOffset="1.0977543679446107e+1" laneId="{b47e6813-9b35-408b-a61f-782177cf7f9c}" travelDir="undirected"/>
</userData>
</lane>

@aertssenaron  Sorry for late, as I haven’t worked on this project for quite a while.
In this project, the issue you mentioned about ignored_lane_types is indeed something that hasn’t been fully addressed in the OpenScenario (1.0.0) & OpenDrive format files (1.6). So, encountering such issues is expected. The solution you implemented is also a method I used in the earlier stages.
From what I observed using visualization software back then, the impact seemed minimal. However, there are a few key points you may want to check:

Impact on Road Geometry: Ignoring lanes marked as 'none' should not affect the computation of usable lanes, but it might impact the overall geometry if these lanes contribute to lane widths or road markings.
Road Markings: Road markings (if present) associated with these lanes might also be skipped, which could be relevant depending on your use case.
Lane Linking: In cases where the 'none' lanes form boundaries or are used for linking lanes, this may affect how lanes are connected, potentially leading to unexpected gaps or breaks in the road network.

To resolve this issue, you may want to check if ASAM OpenDRIVE has any updates or fixes (I only have the old version linked in my README.md).
Alternatively, a possible workaround is to modify the compiled Python script, similar to the following:
def __get_lanes_by_orientation(self, orientation):
    lanes = []
    for lane_obj in self._lanes:
        if lane_obj.orientation == orientation:
            # Only process if the lane type is not 'none' and respects the ignored types.
            if lane_obj.type is not None and lane_obj.type not in self.ignored_lane_types:
                lanes.append(lane_obj)
    return lanes


Syntax error in Grid Lists

What RMWC Version are you using [major.minor.patch]:

Name your build system [Webpack, Rollup...]:

Describe the bug with as much detail as possible:

What happened, and what was supposed to happen:

Solved by https://github.com/rmwc/rmwc/pull/1038. Closing.


Suggestion: Remove generated code from repo

After seeing #4884, I'd like to ask about this. Having diagnosticInformationMap.generated.ts in the repo seems fine since it's part of the build, but since we want to check if for duplicate IDs we should consider incorporating the generation of the code into the standard build process, rather than a script run by hand as-needed.
The only downsides I can see are that editor intellisense for the Diagnostic object won't work after a clean clone until after a build or pre-build is run, and we'll lose the ability to simply run tsc over the compiler folder and get a functioning compiler (you have to generate the generated code first).
As far as upsides go, adopting this strategy should result in fewer merge conflicts in diagnostic messages and, additionally, will add a warning in the form of a failed build in a PRs when someone tries to add a message with an identical message id to one in master.
To do this, we should remove the generated code file from the repo, add the filename to the .gitignore, and make the script to rebuild it run as part of the build process.
Does anyone feel strongly about this?

Since one other downside would be increased build time for tsc it's worth noting that jake generate-diagnostics today takes a negligible amount of time, < 1s.

sounds reasonable to me. we can give it a try and if we find missing the file affecting the productivity we can always put it back, though i doubt it.


Starting a language server in VS Code with no open folders takes down the server

Starting a language server in VS Code with no open folders takes down the server.
Unhandled exception. System.ArgumentNullException: Value cannot be null. (Parameter 'inner')
   at System.Linq.ThrowHelper.ThrowArgumentNullException(ExceptionArgument argument)
   at System.Linq.Enumerable.Join[TOuter,TInner,TKey,TResult](IEnumerable`1 outer, IEnumerable`1 inner, Func`2 outerKeySelector, Func`2 innerKeySelector, Func`3 resultSelector)
   at OmniSharp.Extensions.LanguageServer.Server.LanguageServerWorkspaceFolderManager.<Refresh>b__8_2(Container`1 workspaceFolders)
   at System.Reactive.Linq.ObservableImpl.Do`1.OnNext._.OnNext(TSource value) in /_/Rx.NET/Source/src/System.Reactive/Linq/Observable/Do.cs:line 38
--- End of stack trace from previous location where exception was thrown ---
   at OmniSharp.Extensions.LanguageServer.Server.LanguageServerWorkspaceFolderManager.OmniSharp.Extensions.LanguageServer.Protocol.Server.IOnLanguageServerStarted.OnStarted(ILanguageServer server, CancellationToken cancellationToken)
   at OmniSharp.Extensions.LanguageServer.Server.LanguageServer.Initialize(CancellationToken token)
   at OmniSharp.Extensions.LanguageServer.Server.LanguageServer.From(LanguageServerOptions options, IServiceProvider outerServiceProvider, CancellationToken cancellationToken)

@kvarv thanks for the report, I was able to track down the null ref and added some additional tests in #346


Interested in migrating to GitHub Actions?

Hi @chrisalley, thanks for the invitation.
Are you interested in migrating from Travis to GitHub Actions? I've seen that the latest build is from 5 months ago
I've did this several times in the past and it would require negligible effort on my side

Thanks for accepting the invite to the Pundit Community organisation. I'm looking at getting more people involved in the maintenance of this gem over time.
Migrating to GitHub Actions seems like a sensible step forward and could make a nice PR. Thanks.


Build: Enforce consistent line endings

C.f., gulpjs/vinyl-fs#269

Coverage remained the same at 98.361% when pulling 2f7e4220e389512da89ea6741475b3f2b0549d46 on erikkemperman:consistent-line-endings into 28c2aa2a3f8782a995b47ed051ab52885c705024 on gulpjs:master.


Spelling error: paternAny should be patternAny

SDK Platform
v4 branch
URL of Code Sample
https://github.com/Microsoft/BotBuilder-Samples/blob/v4/javascript_ts/13.basic-bot/src/bot.ts
probably in others
Problem Description
One line spells pattern correctly and the other is missing the second "t".
// Supported LUIS Entities, defined in ./dialogs/greeting/resources/greeting.lu
const USER_NAME_ENTITIES = ['userName', 'userName_paternAny'];
const USER_LOCATION_ENTITIES = ['userLocation', 'userLocation_patternAny'];
Do you want a pull request?

Also needs a fix in the .lu file - https://github.com/Microsoft/BotBuilder-Samples/blob/master/samples/javascript_typescript/13.basic-bot/resources/greeting.lu


Unable to get attachments from Jira for few work items

I'm working to export a huge number of work items from Jira to AzDo. There are a few work items where the attachments do not get exported. The only common factor I was able to find for such work items is that they are ".msg".
Tool Version which is used - 2.3.117
There is a mismatch between the Attachments Journal, the log and Attachments Folder.
AttachmentsJournal.txt and Attachments Folder:
482832;12948709
485070;12948710
Error from the log - Import:
[W][04:08:59] Cannot find info for attachment '482659', skipping. Reason 'Response Content: {"errorMessages":["The attachment with id '482659' does not exist"],"errors":{}}'.
[W][04:09:00] Cannot find info for attachment '482660', skipping. Reason 'Response Content: {"errorMessages":["The attachment with id '482660' does not exist"],"errors":{}}'.
Error from the log - Export:
[E][04:19:23] Could not find migrated attachment '[Removed] 482659/'.
[E][04:19:24] Could not find migrated attachment '[Removed] 482660/'.

Hello!
Have you tried to run the latest version of the migration tool?
https://github.com/solidify/jira-azuredevops-migrator/releases


Matrix subtraction

Perform matrix subtraction. Object parameter may have to be added.

please assign this to me

can i please work on this issue?

I'd like to work on this


Scrum 1 new story

merging code updates

Hi @chrishickey-jds - It seems like this might not have been intended as a PR against the sample repo, so closing. If that was in error feel free to reach back out.


Better config support

It would be better if Clio's config can :
1 Warn user if there is any unsupported/unrecognised config
2 Config can overwrite by command line @kuznetsss

1 will be fixed in #1184.
2 decided to not do.


bug: can't use sql in $onUpdateFn, value.toISOString is not a function

Provide environment information
System:
OS: Linux 5.15 Ubuntu 20.04.6 LTS (Focal Fossa)
CPU: (16) x64 AMD Ryzen 7 3700X 8-Core Processor
Memory: 3.74 GB / 9.72 GB
Container: Yes
Shell: 5.8 - /usr/bin/zsh
Binaries:
Node: 20.12.2 - ~/.nvm/versions/node/v20.12.2/bin/node
npm: 10.5.0 - ~/.nvm/versions/node/v20.12.2/bin/npm
pnpm: 9.2.0 - ~/.nvm/versions/node/v20.12.2/bin/pnpm
bun: 1.0.29 - ~/.bun/bin/bun
Describe the bug
tRPC Error on 'post.update'  [TRPCError: value.toISOString is not a function] {
  code: 'INTERNAL_SERVER_ERROR',
  name: 'TRPCError'
}

It seems that we can't use sql in $onUpdateFn? Did this used to work? https://github.com/drizzle-team/drizzle-orm/issues/2212#issuecomment-2114083803
Link to reproduction
https://github.com/t3-oss/create-t3-turbo
To reproduce
Update a post
Additional information
No response

@trevorpfiz looks not related with this repo. am i missing something?

@trevorpfiz looks not related with this repo. am i missing something?

https://github.com/t3-oss/create-t3-turbo/blob/791c06f64f736bb28eeab832e848ae812a67066f/packages/db/src/schema.ts#L22

still an issue.https://github.com/drizzle-team/drizzle-orm/issues/2212 works with mode: string when update db.

I encounter the same question and solve it by changing the callback to:
updatedAt: timestamp("updated_at", {
    mode: "date",
    withTimezone: true,
  }).$onUpdateFn(() => new Date()),

See also
https://github.com/drizzle-team/drizzle-orm/issues/1113
https://github.com/drizzle-team/drizzle-orm/issues/2212


Fix CMake action build error

CMake build correctly locally but it doesn't in actions because the C version. It doesn't recognize C23 for some reason.

GCC needes to be >= 13.0 and currently ubuntu-latest is on 11.0.

GCC seems to work now, IDKW.


Feature Request: Allow / Use Canvas for Participant Data Aggregation

Checklist

[X] I have read the plugin's documentation.
[X] The feature would be useful to more users than just me.

Feature Requested
I think using Canvas pages for items (i.e. participants) would work well and be a good feature. I would like to have, for example:
Canvas -- Participant SG

notes from my interview with SG (pdf file)
coded transcript with my interview with SG
Audio of SG.
Maybe individual cards with sections of interviews marching a query (or an embedded search). Canvas cards could be associated with codes if relevant. idk, just thinking out loud with this last one.

I imagine a template could be easily built to aid with this, but mostly wanted to get the canvas request in!
Relevant Screenshot
No response

Hmm, could you elaborate a bit more how that is supposed to work? I mean, this plugin is mostly working with "textual" QDA, stuff like coding video or audio segments would require an entirely new plugin and is therefore somewhat out of the scope.
Or do you mean the ability of coding/extracting from kanban cards?

okay, coding from canvas files is now supported.

Thank you!! I'm sorry I never circled around back to this to give you an
idea of what I meant but I think you got it! I was also thinking of using
this as a way to code additional documents, PDFs, photos, etc. Appreciate
your labor!
On Sat, Mar 2, 2024, 7:27 AM pseudometa @.***> wrote:

okay, coding from canvas files is now supported.
—
Reply to this email directly, view it on GitHub
https://github.com/chrisgrieser/obsidian-quadro/issues/1#issuecomment-1974829341,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARN2AOGXYQRAT5522MNJT73YWHVVRAVCNFSM6AAAAABDIXTKYCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZUHAZDSMZUGE
.
You are receiving this because you authored the thread.Message ID:
@.***>

yeah, I just figured that I don't need to understand how coding from a canvas works to enable doing all operations from a canvas file 😆
(coding nevertheless requires you to be in a markdown note in a canvas.)