Use appropiately qualified names in blas and rng

Description
Some types in the blas and rng domain did not have properly qualified names. This has caused trouble when using different SYCL implementations. This PR fixes this for these two domains.
related to #99
All Submissions

[ ] Do all unit tests pass locally? Attach a log.
[ ] Have you formatted the code using clang-format?

@sbalint98 Could you please resolve the conflicts?


Temp (response) queue won't be auto-deleted due to channel remains open

Am I right thinking that RR::Client#request
https://github.com/jakesgordon/rack-rabbit/blob/0fc9fb8dc0dff3837d3376bf2adf7505be6a9981/lib/rack-rabbit/client.rb#L50
does not close channel after it received response?
So the temporary queue won't be auto-deleted, as the connection (channel) is still open.

After some research I found out that rack-rabbit uses single channel for all its temp (response) queues.
On every sync request that rack-rabbit does, it creates temporary queue on RabbitMQ broker, and tells the broker to auto-destroy this queue right after the channel/connection to the broker will be closed.
It's fine if you instantiate new TCP connection to RabbitMQ every time you make synchronous request. So connection opened, temp queue created, actual request sent to exchange, response received from temp queue, connection closed, broker auto-deleted temp queue due to connection closed event. Done.
However, when you prefer to increase throughput and efficiency of your connection to RabbitMQ (in my case I've got 2x increase in throughput this way) and keep TCP connection open for further synchronous requests, that opened channel becomes a source of problems.
By keeping channel open, RabbitMQ does not see any reason to auto-delete created temp queue, so the temp queue for every request made will stay alive permanently. Thus, RabbitMQ will keep created and used temp queues until it overflow at some point.
To deal with this issue we need to manually delete temp queue right after we receive response from that queue.
To fix this it needs to add reply_queue.delete(if_empty: true) right before this line https://github.com/jakesgordon/rack-rabbit/blob/0fc9fb8dc0dff3837d3376bf2adf7505be6a9981/lib/rack-rabbit/client.rb#L65
Pull request will follow.


Implement gadget to show TCP RST packets

Current situation
Users cannot see TCP RST packets.
If the byob or tcpdump gadgets are merged, they might provide a way to see TCP RSTs, but I think a dedicated gadget is still useful.
Impact
Difficult to investigate or notice TCP problems.
Ideal future situation
$ ig trace tcprst
NODE             NAMESPACE  POD    CONTAINER  PID      COMM  IP SRC                    DST
minikube-docker  default    shell  shell      4132704  curl  4  p/default/shell:37418  p/default/nginx-8f458dc5b-

Ideally, the gadget uses the socket enricher and the kube ip resolver.
Implementation options
It could be a networking gadget like dns and sni. I don't see a single place in the kernel responsible for sending TCP RST, so we might not be able to use a kprobe/tracepoint with the reason.
Additional information
There could be several reasons for sending a TCP RST:

The server refuses the connection of the client
The connection is already closed on one side
Iptables with -j REJECT --reject-with tcp-reset
A router doing NAT with a full connection tables closes some old connections

This should now be implemented as an image-based gadget instead of a built-in gadget.
We can start with a socket filter without saying which function the TCP RST packet comes from. Using socket filters and kprobes in the same gadget might be challenging because of https://github.com/inspektor-gadget/inspektor-gadget/issues/2371.

Related: https://github.com/blanquicet/tcprst-gadget


x NOT IN y works but NOT (x IN y) doesn't

Describe the bug
in running some benchmarking, was getting errors like:
Exception: This feature is not implemented: Physical plan does not support logical expression InSubquery(InSubquery { expr: Column(Column { relation: None, name: "ps_suppkey" }), subquery: <subquery>, negated: false })

tracking this down shows x NOT IN y works but NOT (x IN y) doesn't
To Reproduce
probably a more succinct way, but:
import datafusion

ctx = datafusion.SessionContext()

tables = {
    "part": {
        "p_partkey": [1, 2, 3, 4, 5],
        "p_brand": ["Brand#45", "Brand#45", "Brand#46", "Brand#47", "Brand#48"],
        "p_type": ["MEDIUM POLISHED", "MEDIUM POLISHED", "MEDIUM POLISHED", "MEDIUM POLISHED", "MEDIUM POLISHED"],
        "p_size": [49, 14, 23, 45, 19],
    },
    "partsupp": {
        "ps_partkey": [1, 2, 3, 4, 5],
        "ps_suppkey": [1, 2, 3, 4, 5],
    },
    "supplier": {
        "s_suppkey": [1, 2, 3, 4, 5],
        "s_comment": ["Customer Complaints", "Customer Complaints", "Customer Complaints", "Customer Complaints", "Customer Complaints"],
    },
}

for table_name, data in tables.items():
    ctx.from_pydict(data, name=table_name)

sqlA = """
select
    p_brand,
    p_type,
    p_size,
    count(distinct ps_suppkey) as supplier_cnt
from
    partsupp,
    part
where
        p_partkey = ps_partkey
  and p_brand <> 'Brand#45'
  and p_type not like 'MEDIUM POLISHED%'
  and p_size in (49, 14, 23, 45, 19, 3, 36, 9)
  and ps_suppkey not in (
    select
        s_suppkey
    from
        supplier
    where
            s_comment like '%Customer%Complaints%'
)
group by
    p_brand,
    p_type,
    p_size
order by
    supplier_cnt desc,
    p_brand,
    p_type,
    p_size;
"""

ctx.sql(sqlA).collect() # this is fine

sqlB = """
SELECT
  p_brand,
  p_type,
  p_size,
  COUNT(DISTINCT ps_suppkey) AS supplier_cnt
FROM partsupp, part
WHERE
  p_partkey = ps_partkey
  AND p_brand <> 'Brand#45'
  AND NOT p_type LIKE 'MEDIUM POLISHED%'
  AND p_size IN (49, 14, 23, 45, 19, 3, 36, 9)
  AND NOT ps_suppkey IN (
    SELECT
      s_suppkey
    FROM supplier
    WHERE
      s_comment LIKE '%Customer%Complaints%'
  )
GROUP BY
  p_brand,
  p_type,
  p_size
ORDER BY
  supplier_cnt DESC NULLS LAST,
  p_brand,
  p_type,
  p_size;
"""

ctx.sql(sqlB).collect() # this is not fine

# Exception: This feature is not implemented: Physical plan does not support logical expression InSubquery(InSubquery { expr: Column(Column { relation: Some(Bare { table: "partsupp" }), name: "ps_suppkey" }), subquery: <subquery>, negated: false })

Expected behavior
query works in both cases
Additional context
SQLGlot makes the above transformation that should be valid, which is how I hit this initially

take


ContainerName_s property is missing in ContainerInstanceLog_CL schema

ContainerName_s property is missing in ContainerInstanceLog_CL schema

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

ID: 160bf6c0-980c-bf64-8828-cda81c3ccb83
Version Independent ID: 26e35469-0fdc-776f-4b23-a9488fc9c2e3
Content: Collect & analyze resource logs - Azure Container Instances
Content Source: articles/container-instances/container-instances-log-analytics.md
Service: container-instances
GitHub Login: @macolso
Microsoft Alias: macolso

Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. Though the scope of our feedback channel here on GitHub covers specific documentation fixes, we can help redirect you to the right support channel to get an answer to your question: [reference to alternative product/service or community endpoint]. Because this issue does not relate to documentation, we are closing this issue. #please-close


Provide a method to set labels to the k8s resources of the workflow

Description
We'd like a way that enables us to label K8s resources related to the Sonataflow CR. E.g have the same labels appear in k8s deployment, pod, service, route and so.
Today, only default labels are applied to those resources (app=workflow-name, sonataflow.org/workflow-app=,  sonataflow.org/service=)
A private case of it is Backstage plugins that interact with K8s cluster (ci, gitops and others).
In backstage, to view workflow resources related to a component under the CI (tekton) and K8s tabs, a specific label should be set on the pods (e.g. backstage.io/kubernetes-id=application-onboarding).
By enabling the label, the user can view pipelines related to the workflow as in the attached image:
For the CI:

And for K8s:

Implementation ideas

Propagate Sonataflow CR labels to the k8s resources.
Extend the Sonataflow CR API to support it.

I'm in favor of the first option.

I think we can propagate the labels from the SonataFlow CR to the resources. So any custom labels added there by users would also be added to the resources. Is this a reasonable approach for you?

I think we can propagate the labels from the SonataFlow CR to the resources. So any custom labels added there by users would also be added to the resources. Is this a reasonable approach for you?

That's sounds great :-)

@masayag go ahead and send a PR! :D


CSV updates

Pull Request Test Coverage Report for Build 441

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

Totals

Change from base Build 439:
0.0%

Covered Lines:
928

Relevant Lines:
1355

💛  - Coveralls


Does not work if token and no query specified

in IGXClient  - (NSString *)queryStringWithPath:(NSString *)path
needs to be:
	NSString *input;
	if (nil != query && 0 != query.length)
		input = [[NSString alloc] initWithFormat:@"%@%@?%@", self.token, path, query];
	else
		input = [[NSString alloc] initWithFormat:@"%@%@", self.token, path];

instead of
NSString *input = [[NSString alloc] initWithFormat:@"%@%@?%@", self.token, path, query];
In the existing code, if query is empty, it adds a '?' at the end of the encoder input string which leads to the wrong encoded string.

Hey @simsea, should be all better as of #7. Let me know if you're still seeing any issues around this!


Update CL xrefs to species-specific AOs

See the CL subset here:
https://github.com/obophenotype/uberon/issues/1288
Note: editing xrefs in Protege is horrible. We should look to another mechanism, unless someone really wants to do this i Protege

Yeah, I did it manually in the OWL file :/

ZFA
ZFA maintains xrefs to CL

The ZFA to CL issues are an uberon pipeline issue, not of relevance on this ticket
Individual issues have been fixed by @cerivs e.g. https://github.com/cerivs/zebrafish-anatomical-ontology/issues/129

XAO
It looks like XAO has CL xrefs and CL has XAO xrefs. Need to check consistency

WARNING: This issue has been automatically closed because it has not been updated in more than 3 years. Please re-open it if you still need this to be addressed – we are now getting some resources to deal with such issues.


FEAT: Add Single Video Page and Like state persisting bug

📝 Description
A single video page is where the user could see the video and find all details about the video.
🚀 New behavior

Any video could be viewed dynamically
Video could be liked from single video page
Liked Video's Persisting bug of #12 is resolved

🖼 Preview Link
https://vistream-react-version-oht6tiij5-vishalpatil18.vercel.app/
💣 Is this a breaking change?

[ ] Yes
[x] No

🔬 Is the code self-reviewed?

[x] Yes
[ ] No

📋 Is the code well-formatted?

[x] Yes
[ ] No

🔗 Related to Issue

#13
#12

Functionality is working asexpeted
UI looks awesome.
Files are structured properly which helps while reviewing.
Great work @VishalPatil18


Add skip-if-larger params for pngquant

pngquant worker will not allow bigger files anyway, but I assume that using this switch can make it faster to fail.
Please add changelog entry and squash-rebase on master.

Superseded by #181


iOS build

We definitely need to improve build configuration for iOS (XCode 8.1).
I've made some research and build custom version of lime, found how to compile signed .ipa with provisioning profile and identity.
Configuration now looks like
<certificate team-id="XXXXXXXXX" provisioning-profile="xxxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxx" identity="iPhone Developer: Ilya Malanin (XXXXXXXXX)" />
During the build step you should just pass to xcodebuild two additional parameters:
IOSHelper.build:
		if (project.certificate != null) {
			if (project.certificate.provisioningProfile != null) {
				commands.push("PROVISIONING_PROFILE="+project.certificate.provisioningProfile);
			}

			if (project.certificate.identity != null) {
				commands.push("CODE_SIGN_IDENTITY="+project.certificate.identity);
			}
		}

Thanks! This was just committed :smile:
If you pull the newest Lime version, try to lime rebuild tools, then see if the identity and profile work for you properly.
I changed the way that <certificate /> works internally, here's how you set this up for iOS:
<certificate team-id="" identity="" />
<config:ios provisioning-profile="" />

or
<config:ios team-id="" identity="" provisioning-profile="" />

Great news Joshua! Thank you! I'll check it in a next week (right now I don't have mac on hand)

@jgranick I've tried to compile with rebuilded tools, and found that signing is not work properly.
You should add this parameters on the build step, not on the signing. Arguments CODE_SIGN_IDENTITY and PROVISIONING_PROFILE should be provided to "xcodebuild" (in the build step), not for the "codesign".

Thanks, I'll look into this :smile:

I believe this is resolved, can you tell if the current release of OpenFL and Lime is working here?


Unable to open Dbeaver after updating to 7.3.2-1

System information:

Manjaro Linux
7.3.2-1
1.8.0_275
Office plugin & Apache POI plugin

Connection specification:

Database name and version - none
Driver name - none
Do you use tunnels or proxies (SSH, SOCKS, etc)? - no

Describe the problem you're observing:
After updating dbeaver to the version in the title (I used the system's package manager as always), every time I try to open dbeaver I get a small error window:

From the log it seems that some eclipse modules are gone...has this happened to anyone?
Steps to reproduce, if exist:
Include any warning/errors/backtraces from the logs
1611316665339.log

Same here, I downgrade dbeaver to 7.3.0-1, works fine.

Same here, I downgrade dbeaver to 7.3.0-1, works fine.

Thank you for your comment, I guess I'll do that for now.
Hope that this gets fixed meanwhile :)

Thank you for your comment, I guess I'll do that for now.
Hope that this gets fixed meanwhile :)

I hope so too. But this project is too big so I can only report this issue...
This is what I see and its log.
SS

Log
https://gist.github.com/conao3/9390443dba855901538c4747ee8829b8

I hope so too. But this project is too big so I can only report this issue...
This is what I see and its log.
SS

Log
https://gist.github.com/conao3/9390443dba855901538c4747ee8829b8

You have exactly the same log output as me.
I followed your advice and downgraded, and everything is working now :)

You have exactly the same log output as me.
I followed your advice and downgraded, and everything is working now :)

did you report downstream to manjaro as well? the bug might also be in packaging

did you report downstream to manjaro as well? the bug might also be in packaging

Thanks, I opened report at archlinux forum.
https://bugs.archlinux.org/task/69430

Thanks, I opened report at archlinux forum.
https://bugs.archlinux.org/task/69430

I confirmed this issue completely packaging issue.
$ curl -LO https://dbeaver.io/files/dbeaver-ce-latest-linux.gtk.x86_64.tar.gz
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   154  100   154    0     0    221      0 --:--:-- --:--:-- --:--:--   221
100   154  100   154    0     0    178      0 --:--:-- --:--:-- --:--:--   178
100 93.8M  100 93.8M    0     0  7585k      0  0:00:12  0:00:12 --:--:-- 7331k

$ tar xf dbeaver-ce-latest-linux.gtk.x86_64.tar.gz

$ cd dbeaver/
.   configuration  dbeaver.desktop  dbeaver.png      features  jre       p2       readme.txt
..  dbeaver        dbeaver.ini      .eclipseproduct  icon.xpm  licenses  plugins

$ ./dbeaver
<works fine!>
<'About' says its version is Version <IP_ADDRESS>101161839>

I confirmed this issue completely packaging issue.
$ curl -LO https://dbeaver.io/files/dbeaver-ce-latest-linux.gtk.x86_64.tar.gz
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   154  100   154    0     0    221      0 --:--:-- --:--:-- --:--:--   221
100   154  100   154    0     0    178      0 --:--:-- --:--:-- --:--:--   178
100 93.8M  100 93.8M    0     0  7585k      0  0:00:12  0:00:12 --:--:-- 7331k

$ tar xf dbeaver-ce-latest-linux.gtk.x86_64.tar.gz

$ cd dbeaver/
.   configuration  dbeaver.desktop  dbeaver.png      features  jre       p2       readme.txt
..  dbeaver        dbeaver.ini      .eclipseproduct  icon.xpm  licenses  plugins

$ ./dbeaver
<works fine!>
<'About' says its version is Version <IP_ADDRESS>101161839>

Ok, that's it then!
Thank you for looking into it @conao3.
Closing the issue.

Ok, that's it then!
Thank you for looking into it @conao3.
Closing the issue.

I am running Archlinux and was also seeing this error after upgrade. The issue for me was I was using jdk8 as the default java (for development purposes). When I installed jdk11 and used archlinux-java to change teh default java version, everything was working fine. What gave me a clue to try this was that I noticed a change around the time the issue started happening for me:
https://github.com/archlinux/svntogit-community/commit/8c91aa2e07a2571dcee16515a97af05ab3c2e71d#diff-37538beb61ff63edebbf735dfcf39e5d732f49183d6beb097169d971875ca422R11

I am running Archlinux and was also seeing this error after upgrade. The issue for me was I was using jdk8 as the default java (for development purposes). When I installed jdk11 and used archlinux-java to change teh default java version, everything was working fine. What gave me a clue to try this was that I noticed a change around the time the issue started happening for me:
https://github.com/archlinux/svntogit-community/commit/8c91aa2e07a2571dcee16515a97af05ab3c2e71d#diff-37538beb61ff63edebbf735dfcf39e5d732f49183d6beb097169d971875ca422R11


Remove cycles from connectivity queries

Does this PR close any open issues?
No
Found here and potentially before: https://github.com/multinet-app/multimatrix/pull/409#pullrequestreview-1034719675
Give a longer description of what this PR addresses and why it's needed
Removes the cycles from the connectivity query. This should stop the start and end node being the same in multi hops. I have removed all 2 hop and 3 hop cycles. These are the only ones that can be present with our current number of hops (max = 3).
I also fixed an issue with the node/edge attribute selection in the query. Previously we would select n1.attribute, but for some variable names that causes problems. I've updated it so we select n1.`attribute`
Provide pictures/videos of the behavior before and after these changes (optional)
Changes are in the query so will change query results, but there are no UI changes.
Are there any additional TODOs before this PR is ready to go?
TODOs:

[x] Check for 3 hop cycles, 4 hop cycles, etc.
[ ] Test that Crystal's broken query now works

Test Query for 2 Hop: This is the go-to query for Graffinity. Works well with these recent changes to Multinet querying.

In comparison, if the middle node is left to be an approximate match, I get the spinning wheel of doom

Should this be a toggle? There are potentially cases where we'd like the cycle to show. Maybe we even want just the start and end the same and no cycles in the middle?


ci: edit PR message to display chromatic deployment

Issue
💝
Description
edit PR message to display chromatic deployment
Additional context

📚  View the storybook of this branch here

Gravitee.io Automatic Deployment
A prerelease version of this package has been published! 🚀
To use it in your project run the following lines:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

Gravitee.io Automatic Deployment
A prerelease version of this package has been published! 🚀
To use it in your project run the following lines:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


feat(DIA-2): new shows rails in about tab

This PR resolves DIA-2
Description

Implements the new Shows rails
Remove unused component SmallList

iPhone 14 Pro Max
https://github.com/artsy/eigen/assets/15792853/6f57cbac-2e77-4376-b45f-0b574c5cbc95
iPad

cc @artsy/diamond-devs
PR Checklist

[x] I have tested my changes on iOS and Android.
[x] I hid my changes behind a feature flag, or they don't need one.
[x] I have included screenshots or videos, or I have not changed the UI.
[x] I have added tests, or my changes don't require any.
[x] I added an app state migration, or my changes do not require one.
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

new shows rails in Artist about tab - araujobarret

iOS user-facing changes

Android user-facing changes

Dev changes

Need help with something? Have a look at our docs, or get in touch with us.

also bonus thanks for the tablet screenshots 👏

wandering how we handle that in other rails within eigen, my only concern would be having a huge item in a smaller device (like iphone SE, or even 7,8).

What is the smaller we consider in Eigen? Xs or SE?

@araujobarret nah that looks more than okay 🔥 (I think SE is smaller than XS)


feat: Cookbook Onboard (AI Assistant) integration

Preview
Preview link: https://qrl-docs-cookbook.vercel.app/
Ask Cookbook AI Assistant
Cookbook's Ask Cookbook AI assistant and co-pilot is trained on all existing QRL resources (source code, docs website, etc.) and is available as a standalone modal, embeddable as a button on any page (recommended for technical docs). It answers developer questions about building on QRL, acting as an enhanced and streamlined technical documentation search tool as well as a Solidity coding co-pilot.
Ask Cookbook AI can also access context from thousands of data sources indexed by Cookbook.dev in addition to QRL-specific data sources, providing the best blockchain developer-focused answers of any chatbot on the market. Cookbook will assist in the tuning and calibration of the AI assistant to ensure the highest answer quality.

Thanks, but a non-trivial change like this would need discussion before a PR is considered -><EMAIL_ADDRESS>


RequestLogFactory custom logging format

Hi,
it appears to me, that the RequestLogFactory ignored all log formats that are passed in the configuration. It also hardcodes the RequestLogFactory as a layout. I know one can overwrite this factory and create a custom logger, however it would be nice if this could be done through configuration.
Thanks,
Artur

Hi,
I think this is done on purpose, take a look at the "Log Format" chapter in the manual:
Dropwizard’s log format has a few specific goals:

Be human readable.
Be machine parsable.
Be easy for sleepy ops folks to figure out why things are pear-shaped at 3:30 AM using standard UNIXy tools like tail and grep.

That is fine, but you can define your own log formats for your other loggers. Why not be able to have your own log format for request logging?

@pandaadb https://github.com/dropwizard/dropwizard/pull/1415 adds the ability to configure the format of the request log. It is available in master and targeted for the 1.0.0 release.

Closing this because the feature is implemented in 1.0 (if this is not the case, or there is a bug, please report back to open the issue back up).


Use django signals for performing operations on related objects

Is your feature request related to a problem? Please describe.
I see many instances where related objects are saved/modified in save operation of other object. This logic should be detached form using django signals.
Describe the solution you'd like
Use Django's post_save signal to create relate objects and sending email notifications.

Honest question: why?

I believe it will improve extensibility of the project.

This is definitely worth exploring but will require some significant code changes I think. Will put it on the list for a future major version. Thanks for the suggestion!

@pandafy - feel free to raise a PR to start moving towards this pargidgm if you really want to make it happen.


doc: change link to presentation to Fosdem 2019

Fosdem 2019 presentation is more up-to-date.

Thank so much @tomsquest

You're welcome!


JSON-LD

Please support output of JSON-LD.

Thanks for this early issue.
Added your feature request to some later stages of development, cause right now the focus is on very basic functionality.


When will open source code be available

It's a fascinating piece of work and we look forward to the code being open-sourced for more engagement and advancement soon!

Hello, sorry for the delay. The code will be released this week. Thanks!

Hello, sorry for the delay. The code has been  open-sourced.


when i run  "roslaunch kinect2_bridge kinect2_bridge.launch",it crashed , how can i resolve it?

[ INFO] [1495530398.216005983]: Loading nodelet /kinect2_bridge of type kinect2_bridge/kinect2_bridge_nodelet to manager kinect2 with the following remappings:
[ INFO] [1495530398.307841897]: [Kinect2Bridge::initialize] parameter:
base_name: kinect2
sensor: default
fps_limit: -1
calib_path: /home/ronan/catkin_ws/src/iai_kinect2/kinect2_bridge/data/
use_png: false
jpeg_quality: 90
png_level: 1
depth_method: default
depth_device: -1
reg_method: default
reg_device: -1
max_depth: 12
min_depth: 0.1
queue_size: 5
bilateral_filter: true
edge_aware_filter: true
publish_tf: false
base_name_tf: kinect2
worker_threads: 4
[Info] [Freenect2Impl] enumerating devices...
[Info] [Freenect2Impl] 14 usb devices connected
[Info] [Freenect2Impl] found 0 devices
[ERROR] [1495530398.767375377]: [Kinect2Bridge::initDevice] no Kinect2 devices found!
[ERROR] [1495530398.773057216]: [Kinect2Bridge::start] Initialization failed!
[FATAL] [1495530398.840791856]: Failed to load nodelet '/kinect2_bridgeof typekinect2_bridge/kinect2_bridge_nodeletto managerkinect2'
[kinect2_bridge-3] process has died [pid 23499, exit code 255, cmd /opt/ros/indigo/lib/nodelet/nodelet load kinect2_bridge/kinect2_bridge_nodelet kinect2 __name:=kinect2_bridge __log:=/home/ronan/.ros/log/147bc644-3f97-11e7-ab91-848f69cd9abe/kinect2_bridge-3.log].
log file: /home/ronan/.ros/log/147bc644-3f97-11e7-ab91-848f69cd9abe/kinect2_bridge-3*.log
[kinect2_bridge-3] restarting process

Try to connect a kinect2 to you pc: [ERROR] [1495530398.767375377]: [Kinect2Bridge::initDevice] no Kinect2 devices found!

need to install the package ros--image-pipeline


"Unhandled promise rejection" error

I have a pretty simple Azure/cli step in my pipeline that's giving me the error:
(node:3210) UnhandledPromiseRejectionWarning: Error: az cli script failed.
    at /home/runner/work/_actions/azure/CLI/v1/dist/index.js:1:23964
    at Generator.throw (<anonymous>)
    at rejected (/home/runner/work/_actions/azure/CLI/v1/dist/index.js:1:20436)
    at processTicksAndRejections (internal/process/task_queues.js:97:5)
(node:3210) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:3210) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

This is the task code:
      - name: Dewhitelist IP
        if: always()
        uses: azure/CLI@v1
        with:
          azcliversion: 2.30.0
          inlineScript: |
            az acr network-rule remove --name retailde --ip-address ${{steps.runner_ip.outputs.runnerip}}

Hi @tdevopsottawa Can you please provide us with complete logs as there should be a ERROR field with correct error message.

Hi @t-dedah,
I've received this a few times today.  No explanation for it.  Had worked on same Self Hosted Runner (Linux) yesterday with zero code changes since.
Error:
ERROR: 'bytes' object has no attribute 'get'
Error: Error: az cli script failed.
cleaning up container...
MICROSOFT_AZURE_CLI_16488[18](https://<redacted>/runs/5788376876?check_suite_focus=true#step:5:18)864547_CONTAINER

(node:913745) UnhandledPromiseRejectionWarning: Error: az cli script failed.
    at /home/cfadmin/actions-runner/_work/_actions/azure/CLI/v1/dist/index.js:1:23964
    at Generator.throw (<anonymous>)
    at rejected (/home/cfadmin/actions-runner/_work/_actions/azure/CLI/v1/dist/index.js:1:[20](https://github.com/Carfinance247/Carfinance.Events.Service/runs/5788376876?check_suite_focus=true#step:5:20)436)
    at processTicksAndRejections (internal/process/task_queues.js:97:5)
(node:913745) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:913745) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Can this ticket be reopened please?
Warmest regards,
Garrard.

Hello, I am also see this error.  Is there any guidance on how to resolve it?


feat(pulse): Show volume in decibels

Add  %decibels% token in pulseaudio module.
Implements #1886.
Tested on two archlinux devices and everything seems fine.

Closes #1886


SVG logos are not rendered in documents

Edit e.g. Delivery note settings and add an SVG logo
Render document for any order
Of course, this also causes problems during and after migration. Documents with svg logo broken as a result

[public] Shopware automated response: This issue is linked to the internal issue: NEXT-39167
[created from NEXT-39167, comment 505473]


Empty out arrays, object counts when context is zombified

With this changeset (which will be released as 0.7.3), we mark our CoreDataSource as having a zombie context earlier in its lifecycle (before we notify listeners about the data source emptying out) so that when those listeners (usually collection/table view data sources) ask our CoreDataSource how many objects it has, we can return the "accurate" number (which is 0), which prevents inconsistency exceptions.
Objects can still get at specific objects using object(at:) and passing in the index paths if they need to, before those objects' backing contexts go away.

Since this is the same fix I saw you working on on Friday, I'll say this LGTM, but others should take a look as well.

Yeah lemme see about adding a test for this.


Cypress automated test for "Tutorial - Navigation and Links"

Summary
Addition of a Cypress test to test navigation of the tutorial that displays when the user first signs up/signs into Mattermost.
The test:

Creates a new user via the API and signs in.
Clicks through the pages using the dots at the bottom of the tutorial to ensure the proper page displays for each dot.
Clicks through the Next button to ensure the next page displays, and:

Clicks the icons link on the second step to ensure that the download page is fetched (but does not display, as it seems Cypress does not handle new tabs/windows well).
Clicks the Off Topic link in the sidebar so that the Next button on the third page of the tutorial displays that channel instead of Town Square.

Creates a second user via the API and signs in.
Clicks the Skip Tutorial link and ensures the Town Square channel displays.

This pull request adds two methods to the API commands to create and sign in users without setting the preferences to bypass the tutorial
Ticket Link
-Github Ticket 10234: Write an automated test using cypress for "Tutorial - Navigation and Links"
-Jira Ticket MM-13929
Related Pull Requests
None.

Thanks @bnoggle for the pull request!
Per the CONTRIBUTING.md file displayed when you created this pull request, we need to add you to the list of approved contributors for the Mattermost project.
Please help complete the Mattermost contribution license agreement?
This is a standard procedure for many open source projects. Your form should be processed within 24 hours and reviewers for your pull request will be able to proceed.
Please let us know if you have any questions.
We are very happy to have you join our growing community! If you're not yet a member, please consider joining our Contributors community channel to meet other contributors and discuss new opportunities with the core team.

@bnoggle This will also need to sync with master is well for the checks to pass.

LGTM. Great work @bnoggle


Release next

Features
Add pages/update existing pages. Mention in Jan 2024 release.

[x] Clickhouse (@scsmithr)
[ ] BSON table funcs (@tychoish)
[ ] Crypto/hashing table funcs (partition_results, etc) (@tychoish)
[x] read_sqlserver (@scsmithr)
[ ] COPY TO bson (@tychoish)
[ ] #209 (@greyscaled)
[ ] Cloud SQL Workspace (@greyscaled)
[ ] Cloud Connection Dialog (@greyscaled)
[ ] Cloud Auth Github (@greyscaled)
[ ] Cassandra

Fixes
Just mention in Jan 2024 release.

[x] Fix iceberg table version reading

To double check: https://github.com/GlareDB/glaredb/compare/v0.7.1...main

there are PRs open on the meta repo with blog posts about all of the things with my name next to them.

We haven't documented any scalar functions which the hashing and partitioning functions count as, and it feels weird to put them in the same category as the table functions.

Crossed it out. We'll want to add those in at some point, and maybe generate some/most of the documentation from glare catalog. Could be interesting to just generate a cheat sheet for scalars ordered by name or something.


Remove domain from username for non-unique id secondary userstores

Description
Remove domain from username claim for non-unique id secondary userstores, when calling SCIM endpoint
Related Issue

https://github.com/wso2/product-is/issues/20443

This issue could not be reproduced in master


Add a motd for useful BACH variables

This PR fixes issue #237.
Welcome to Ubuntu 12.04.5 LTS (GNU/Linux 3.2.0-86-virtual x86_64)

You have logged into a member of a BACH cluster.

Environment: Test-Laptop
Bootstrap node IP: <IP_ADDRESS>
Roles:
  - role[BCPC-Bootstrap]

Added Zabbix and Graphite URLs to the motd.
Welcome to Ubuntu 12.04.5 LTS (GNU/Linux 3.2.0-86-virtual x86_64)

You have logged into a member of a BACH cluster.

Environment: Test-Laptop
Zabbix URL: https://<IP_ADDRESS>:7777
Graphite URL: https://<IP_ADDRESS>:8888
Bootstrap node IP: <IP_ADDRESS>
Roles:
  - role[BCPC-Bootstrap]

Tested this on a dev cluster and looks good.


Master NSG has 2 rules for port 443 and none for 8443 port

Openshift 3.9 installation has 2 rules for port 443 and none for 8443 port.
Could you please have a look?

We are no longer using port 8443.  We changed to using 443 instead.

Closing this issue as we no longer use port 8443.


fix(test): fix failed test cases due to race condition

Which issue(s) this PR fixes:

Issue #
What this PR does / why we need it:
Special notes for your reviewer:
Additional documentation or context

@FrankYang0529 The CI is queued and stuck. Can you help check it? Thanks.
https://github.com/longhorn/sparse-tools/actions/runs/9764492417/job/26953145525


Skip flaky mcs test

Will continue debugging in https://github.com/istio/istio/pull/35767
For https://github.com/istio/istio/issues/34051
This test is blocking productivity
Please provide a description of this PR:

/cherrypick release-1.12

@howardjohn: new pull request created: #35922

In response to this:

/cherrypick release-1.12

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


Don't Change Location with New Update

Description of the new feature / enhancement
During updating, «PowerToys» changes installation location. The app automatically removed from chosen by me location and established on default folder (C:\Program Files\PowerToys).
Scenario when this would be used?
Every upcoming updates.
Supporting information
No response

cc: yuyoyuppe

Thanks for the report! I was able to reproduce it and we'll implement the fix for it.

This has resolved with 0.47 release


Add a nix flake setup

This adds a nix flake setup to lake and a corresponding CI. Eventually I would hope that Lake and nix flake will be somewhat interoperable in the sense that they both support content addressed and deterministic builds.

I feel weird for being the one to object to this PR, but: who will update the lean input when leanpkg.toml's lean_version is updated? As Mac is a Windows user, this looks like a recipe for frequently-broken CI.

@Anderssorby, @Kha makes a good point, how can this be feasibly maintained if I am not using Nix? Is there a way for me to manually update the Lean version when I update the version in leanpkg.toml?

@tydeu  True, it is most convenient if there is a nix user which can run nix flake update occasionally. I guess it is possible to manually update the revision in flake.lock, but you also need to update the NAR hash which you need nix to know, but as long as you have some contributors which can do this update it shouldn't be a big problem. You want to support unix right? So you need to test it occasionally in that environment anyway. You could also disable the nix CI if it gets annoying. I'm planning on using this in my experiments with Lean and Yatima going forwards.

You want to support unix right? So you need to test it occasionally in that environment anyway.

As Lake is written in Lean, the Unix support comes largely as a by-product from Lean itself. Lean is cross-platform, so I can develop the project on my Windows machine and have it still work on Linux and MacOS. The problem here is that in order to support Nix build, I also need to have a local Nix installation, which I do not have. This is not the case for the other builds. For instance,, there is a MacOS build for Lake in the CI, but I have never tested that locally on a Mac (and may never do so).

Also, it concerns me that you need to have Nix installed in order to update a build configuration. This strikes me as a major flaw with Nix. While I understand that you may need Nix in order to compute the integrity hash, such a hash should not be required to perform builds. For example, in npm, the package.lock (which contains integrity hashes) is a useful but necessary component for building.

You seem to be talking about two different things at once - requiring a lock file, and requiring the build tool for updating/generating it. Nix doesn't do the first one either (but we of course want Lean packages to be locked against Lean), and the second is just as true for npm or any other tools that uses a (non-standardized) integrity hash, I would assume

@Anderssorby If you want to use Lake via Nix, someone should just improve Nix' submodules support so we can extend https://github.com/leanprover/lean4/pull/683 with Nix support :) (likely after it is merged)

Nix doesn't do the first one either (but we of course want Lean packages to be locked against Lean),

Ah, my mistake, I misunderstood. In that case, I could just delete the lockfile when a change makes it outdated and wait until some other contributor decides to regenerate it and add it back.

Yes, that would work since going forward we will not make incompatible changes to Lean without immediately repairing Lake, so it's never wrong to fall back to the newest Lean in absence of a lock file. So I would probably suggest to commit only the flake.nix (and secondary files) and leave out flake.lock and the CI setup, since there is no maintainer to maintain them as you pointed out.

@Kha Is there any harm in keeping the CI setup? It can serve as a nice test that such a build does indeed work and it doesn't need the lockfile, right?

@Kha @tydeu I've removed the flake.lock and nix CI. I'm also looking into ways to use lake from within nix.

@Anderssorby I think that in the adding flake.lock resulted its deletion not being committed.

@tydeu It's fixed now.


Feature/metadata

Coverage remained the same at 100.0% when pulling b902865af709a8d9a2f39b22dc626faeea93fd07 on feature/metadata into 0f9ccde32aad43e1fce53c5f475a0c0cb02ccd3a on master.

Coverage remained the same at 100.0% when pulling 715123b6ea90fa1e63f58b74ef82fa0b4a31a513 on feature/metadata into 0f9ccde32aad43e1fce53c5f475a0c0cb02ccd3a on master.

Coverage remained the same at 100.0% when pulling 57e39ab0b4abd5ac4d0baa660065f5b2bbc2d25e on feature/metadata into 0f9ccde32aad43e1fce53c5f475a0c0cb02ccd3a on master.

Coverage remained the same at 100.0% when pulling 4e127c14670fa292d3b21597c6c98f474f1f4715 on feature/metadata into 0f9ccde32aad43e1fce53c5f475a0c0cb02ccd3a on master.


Remove scrollviewer around accounts view.

This breaks the scrollviewer in the listbox used to present the list
of accounts.
Fixes #103.

So is this fix win10 only or will it fix the lack of scrollbar on accounts for win7 too?

fixes it everywhere.  the fix had nothing to do with the os.

OK


Print advanced instructions after ghpc deploy

Print advanced instructions after ghpc deploy;
Change WriteDeployment to take deploymentDirectory;
Meet test coverage by reducing number of lines.

After you make the requested changes, please also update the description to include a "before" and "after" of what the commands look like.

$ make && rm -rf hello/ && ./ghpc create tst.yaml $TOOLKIT_DEF_VARS > ~/tmp/after
$ g co develop
$ make && rm -rf hello/ && ./ghpc create tst.yaml $TOOLKIT_DEF_VARS > ~/tmp/before
$ diff ~/tmp/after ~/tmp/before
$ # no diff

$ make && ./ghpc deploy hello > ~/tmp/after
$ g co develop
$ make && ./ghpc deploy hello > ~/tmp/before
$ diff ~/tmp/after ~/tmp/before
<
< ###############################
< Find instructions for cleanly destroying infrastructure and advanced manual
< deployment instructions at:
<
< hello/instructions.txt


[task-planning] Plan the order of picking up objects in different aisles

[ ] Determine criteria for picking up
[ ] Implement if based planning
[ ] A-star/heuristic other planning options

Solved by Travelling Salesman Problem


Gradle build should have "offline" build mode

To test if the entire project is self-contained we should have a way to build with gradle without using any external servers.
Probably mostly amounts to setting the repositories to only flatDir

Offline build with
Offline Requirements:

JDK
Gradle
pandoc and texlive to generate userguide.pdf

The dist can be built offline with
gradle -b build.offline.gradle --offline build

Replace build to run another Gradle task.
Offline limitations/workarounds:

Integration test are not prepared. To run offline integration tests you are responsible for running ./src/test/resources/scripts/create_symlinks on the locations specified in the xenon.test.properties file.
Publishing to bintray must be done manually
No test coverage, coverage report generation requires download
No SonarQube analysis

TODO
Add task to download jars to lib.

What do you mean by downloading jars to lib? The Xenon dependencies are there already?

Offline build mode works, the todo in the comment is not essential for building offline, and will become a separate issue.

Moved TODO to own issue #267


[Docs]:Edge shape documentation added

Notes for Reviewers
This PR adds Edge shapes to the Meshery docs on a separate URL/page.
Signed commits

[x] Yes, I signed my commits.

🚀 Preview for commit bc4eac58bffbc3e6135fe375588f185569e18553 at: https://66c465448ec781eddf1897a7--meshery-docs-preview.netlify.app

@SAHU-01 the reason for the svgs not being displayed is the following error:

props in the  e.g. width, height, fill etc. will be needed to be filled with their appropriate values.

@vishalvivekm I've fixed all of these, thank you for helping me point this out!

🚀 Preview for commit ff37e13abbf37b1c0f181a46b467bde7a29fa7cd at: https://66c46c9b051680f993d730ee--meshery-docs-preview.netlify.app

@SAHU-01 You can use the reusable include to show the edges details without having to repeat the markup and css
https://github.com/meshery/meshery/pull/11701/files#diff-368a87601b658e84de4946da33db827f08f31fc4d023e42e46841da77abe0510R20-R91

🚀 Preview for commit 7450f4819ddef449f45ee732b60434f6e234fc49 at: https://66c487a416d4a0067f4978fa--meshery-docs-preview.netlify.app

🚀 Preview for commit 52c93a2351b033e10bcb459cca3b74418d07a20e at: https://66c4921b47eb41150f4339e1--meshery-docs-preview.netlify.app

🚀 Preview for commit 0ee100f3009cd8b88e1fdc5539eb3e504714a0c5 at: https://66c6ea0bd7e7ad189b719cbe--meshery-docs-preview.netlify.app

🚀 Preview for commit 79064c1f82fe80bc3c8d0ecb0aaa5f66f2d793f9 at: https://66c856879d776755deca970c--meshery-docs-preview.netlify.app

🚀 Preview for commit ca6b17cb68bd178f6472945dda3a1a722862b1e6 at: https://66c859f6d921d35cca7a7ed6--meshery-docs-preview.netlify.app


[Text Translation] Updating samples

Updating Text Translation samples.

API change check
API changes are not detected in this pull request.


[do NOT merge yet] Update the_geom_webmercator from the_geom #263

Fixed in upstream project: https://trac.osgeo.org/gdal/ticket/6565
Cclosing this ticket


Bug: events on a Sprite don't work on initial run using Vite

Current Behavior
When I create a sprite and add an onPointerDown property that changes the tint property of the sprite, it doesn't work on initial pnpm run dev, but it works after changing anything in the file and saving it (after an HMR)
Expected Behavior
I expect that after running the project with pnpm run dev, when I click the sprite it should change it's color.
Steps to Reproduce

Open live example on Stackblitz.
Wait until it shows the bunny on the preview tab.
Try to click the bunny and nothing happens.
Edit the App.jsx file (it could be adding blank line) and let HMR acts.
Try to click the bunny and now it works, the bunny change it's color successfully.

Environment

@pixi/react version: 8.0.0-beta.7
pixi.js version: 8.2.6
React version: 18.3.1
ReactDOM version: 18.3.1
Browser & Version: 126.0.6478.183 (Official Build) (x86_64)
OS & Version: MacOS Sonoma 14.5
Running Example: example on Stackblitz

Possible Solution
No response
Additional Information
I don't know if it's also happening with another pixi components.
But I could see that if i move the onPointerDown and eventMode properties to the <container> tag, it's working like expected!

This same exists when using a combination of React and Valtio, initially I assumed it a problem on how I was managing my state.


Set UIOffset in a view

I want to HUD show in view, and set offset, e.g.:
SVProgressHUD.setContainerView(myView)
SVProgressHUD.setOffsetFromCenter(UIOffset(horizontal: 100, vertical: 100))
SVProgressHUD.show()

no any effect, it still stay in center. how to change?
look forward to your early reply, thank you!

same issue in swift, iOS10.3...

saw this PR : https://github.com/SVProgressHUD/SVProgressHUD/pull/738
why not yet included ?


multiplayer: kick player kicks the host

After kicking a player out and getting kicked out, I also had problems joining other games. I had to open a new multiplayer tab to join other games.

We were running against an out of date server in testing :(. Works fine now.


Emacs tree-sitter support

It would be interesting if we have support for Emacs. Currently, I'm setting Emacs and tree-sitter using https://github.com/emacs-tree-sitter/elisp-tree-sitter to work on my local machine. I can help try integrating on it. :)

Sounds good, feel free to create an integration :D Is there anything necessary on this end to support it?

Cool! For some existing queries see highlights.scm in this repo and also https://github.com/nvim-treesitter/nvim-treesitter/pull/1904. You probably can reuse most of that and just change the @annotations to whatever is expected :)

Sorry for not responding for a long time. I've trying with highlight.scm from nvim-tree-sitter. I tried it with lib/elixir/lib/code/formatter.ex in elixir repository and the result looks like:
Before:

After:

I found an issue that some atoms in required_parens_on_binary_operands got highlighted and some not, I'm not sure why. But overall looks good.

Hmm, the module attributes look off, and some of the keywords like def, case are not highlighted properly. You can play around with queries order, I think the built-in highlighter always uses whatever matches first (ordering from most to least specific queries), while nbim-tree-sitter works the other way round (subsequent highlightings take precedence).
FTR here's what I get using the built-in highlighter with these queries.

Thanks for your advice. The syntax highlighter is pretty straightforward than I thought.

I opened a branch on my fork here. I need help testing before sending a patch to upstream. :)

Update the progress. I sent a PR to the tree-sitter-langs package to register a highlight query here.

I am playing with the feature/tree-sitter branch and have some version working with font-locking, indentation and navigation. I don't know tree-sitter very well and have not worked a lot on major modes, so not sure if what I am doing is correct. Maybe we can merge some version of this into the elixir-mode repository for when emacs 29 is out?
It is ready enough for me to use it during the week and tweak an iron out: https://github.com/wkirschbaum/elixir-mode/blob/main/elixir-mode.el

Not sure if this is what you’re looking for for the moduledoc issue https://github.com/wingyplus/tree-sitter-langs/blob/elixir-queries/queries/elixir/highlights.scm

@wingyplus i see the arrows are misleading. i meant those as an advantage, not an issue. I used the highlights file to generate the treesit-settings function, it is almost the same except for some tweaks.
There is a moduledoc issue, but seems to be either emacs treesit bug ( i see there are some fixes still rolling in ) or a elixir treesitter bug. I will investigate a bit this week.

Ah I see. If we can modify string after doc attributes (moduledoc, doc, typedoc) into a comment that’s would be great.

This is an issue that I see almost everywhere that I see tree-sitter-elixir being used. It happens in Zed and Nova.
It makes other editors I want to try unusable except for VSCode because it drives me nuts (I write a lot of docs in my code).
e.g. https://github.com/zed-industries/community/issues/814
I wish I knew what GitHub was doing to fix it on their end, since they apparently also use tree-sitter-elixir.

Hi @ryanwinchester, I think wkirschbaum/elixir-ts-mode has already marked it as comment style. :sunglasses:
https://github.com/wkirschbaum/elixir-ts-mode/blob/main/elixir-ts-mode.el#L70

Is this not something that tree-sitter-elixir could fix, or does every editor that uses it have to set that up on their own?

In fact, the highlight provided by tree-sitter-elixir also marks it as comment style.
https://github.com/elixir-lang/tree-sitter-elixir/blob/main/queries/highlights.scm#L8
So maybe those editors don't set the correct colors for the comment.doc style.

Weird. It doesn't look correct in here either: https://elixir-lang.org/tree-sitter-elixir

Because it uses a simple version of the highlighting rules.
You can copy the complete highlighting rules I mentioned above into the lower left corner of the demo web page.

Because it uses a simple version of the highlighting rules. You can copy the complete highlighting rules I mentioned above into the lower left corner of the demo web page.

Yes I did, and it is still not correct for me. Does it work for you?

@ryanwinchester tree-sitter-elixir handles the parsing part, that is turning code into meaningful nodes. The rules for highlighting nodes are generally up to the user (or in this case the editor projects). In this repo we have queries/highlights.scm, which is what GitHub uses and we explicitly mark docstrings as comments:
https://github.com/elixir-lang/tree-sitter-elixir/blob/b20eaa75565243c50be5e35e253d8beb58f45d56/queries/highlights.scm#L7-L21

Weird. It doesn't look correct in here either: https://elixir-lang.org/tree-sitter-elixir

The demo page isn't meant for highlighting, it's a playground where you can see how the code gets parsed and then match queries against the parse tree. The colors are only to distinguish the query results.

In this repo we have queries/highlights.scm, which is what GitHub uses and we explicitly mark docstrings as comments

Thanks, so it's easy-mode for editors if they use that? Thanks :)

@ryanwinchester

Thanks, so it's easy-mode for editors if they use that?

Yeah, when integrating with the given editor you'd use pretty much the same query and just make sure to use node names that the editor understands for highliting (e.g. @comment).
Here is an example in Nvim :)

this is what it currently looks like in elixir-ts-mode

Should comment use the same font  as doc comments?

i don't know about other editors, but in emacs it is really easy to modify the faces. giving fever options is not better.

Maybe we're getting into subjective territory but as long as it's comment-like and not rendered like a normal module attribute + string, it's good.
Personally I prefer it to just be the same as comments. Like how GitHub and VSCode handle it.
@doc "foobar"

@moduledoc """
foobar
"""

# comment

"string"

~S"""
foobar
"""

@ryanwinchester nothing stops you in emacs and currently elixir-ts-mode to set them to the same face. if we tag moduledocs with comment then the user won't be able to easilty specify them as two different faces if they want to. I don't think it is subjective, as it is now you have more options to customize your fontification and don't think we should remove that option.

@ryanwinchester nothing stops you in emacs and currently elixir-ts-mode to set them to the same face. if we tag moduledocs with comment then the user won't be able to easilty specify them as two different faces if they want to. I don't think it is subjective, as it is now you have more options to customize your fontification and don't think we should remove that option.

No, I agree.

@wingyplus is there any reason for this to still be open? Is there anything which needs to be done? In the next month or two I will try to get elixir-ts-mode into emacs core and currently we have elixir-ts-mode and heex-ts-mode on MELPA for the next year or so.

@wkirschbaum Thank you for creating those ts-modes. I'm very impressed.
So I'll close it now. 🙇


Make use of registered services for log writers

I wanted to create a more complex log writer (create YouTrack issues). This one needs more config than just a output url (like username, password, url, project name...). With this PR you can register a service and use it as log writer. If the registered service has the method setOptions, the options array from the writer config will be applied to it.

Oh wow, PHP 5.3 is supported. Hmm. Should I fix the use statement?

Oh wow, PHP 5.3 is supported. Hmm. Should I fix the use statement?

I am not keen on 5.3, but it is a fairly simple fix to continue to support it. Replace the trait with a setter?

Done ;-)

Great!
Coverage dropped, can you cover new methods please https://scrutinizer-ci.com/g/eddiejaoude/zf2-logger/inspections/5562d294-c242-499f-98b1-3270b989d560

I will, but I'm busy right now. Within the next few days...

Sure :smile:


Request for AAD Connect Cloud Sync info

Please update to include information about how to best use the new AAD Connect Cloud sync, specifically in regards to its potential role in disaster recovery.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: ddae5ea7-6aa1-1f50-136d-985a95d53ba6
Version Independent ID: d3b98641-4e2b-16dd-aa73-8d08cd2fb395
Content: Azure Active Directory Identity and access management operations reference guide
Content Source: articles/active-directory/fundamentals/active-directory-ops-guide-iam.md
Service: active-directory
Sub-service: fundamentals
GitHub Login: @martincoetzer
Microsoft Alias: martinco

@DeanGross
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

We've started our investigation but believe most of the recommendations will remain similar. We will update this as we learn more

#please-close


changes to polling APIs to allow polling all types of requests

[ ] Change ServiceConfigurationChangeRequest to ServiceChangeRequest
[ ] Change method getPendingConfigurationChangeRequest to getPendingServiceChangeRequest
[ ] Change method updateConfigurationChangeResult  to updateServiceChangeResult.

let me do it


Travis CI: Add Python 3.7

https://devguide.python.org/devcycle/#end-of-life-branches Python 3.4 is end of life this codebase really need to continue to support it?

Thank you!
We should generate the golden data for 3.7 rather than ignore failures for it. Would you do that in this PR?
python3.7 setup.py test -s pasta.base.annotate_test.generate_goldens

will create a starting point for 3.7.
As for 3.4, if ti becomes much work to maintain I'll consider dropping support for it. For now I don't think it's a big issue.

3.7 and 3.8 are included in travis build now.


Using module in advanced application template

Can this module be used with advanced application template?
I want the administrative functions be enabled in backend and registraction, profile etc functions be enabled in frontend. There should be convenient way to do it easy.

Yes.

If the module can be used in advanced application template by the proper configuration - THAT'S GREAT. It's hard to find all opportunities without full doc. I have seen the request to contribute to markdown in #21. But IMO it's hard to contribute to documentation when the person is not the author of the code. The person doesn't realize all features and intentions.

You may read through this webtip to configure different sessions for frontend and backend.
I will be creating a separate app template (based on advanced, app-practical-a apps) that will have this module preinstalled with configurations.

Updated in docs.

@kartik-v Thanks!!!


Support for multiple values in length.is validation

Example:
var constraints = {
    length: {
        is: [5, 10] // length is 5 or 10 (array for multiple values)
    }
};

var value = 'hello';

validate.single(value, constraints);

Hmm, this feels like a highly specific use case, could you explain a bit more what you're trying to do?

In Russia individual taxpayer number (ITN) is 10 or 12-digit code. Maybe other identifiers can have different length

I think this is better done with a regex validation instead, that way you can handle formatting too.
Something simple like this might work:
var constraints = {
    pattern: "\\d{5}|\\d{10}"
};
validate.single("1234", constraints);
// => ["is invalid"]
validate.single("12345", constraints);
// => undefined

It is also good. I agree that it's specific use case

I just checked and it does not work :)
var constraints = {
    format: {
        pattern: "\\d{5}|\\d{10}"
    }
};

validate.single("1234", constraints); // is invalid
validate.single("12345", constraints); // undefined
validate.single("1234567890", constraints); // is invalid

From source:
match = pattern.exec(value); // value = "1234567890"
if (!match || match[0].length != value.length) { // match[0] = "12345", not "1234567890"
    return message;
}

Correct regexp is ^(\d{5}|\d{10})$


Auto-completion

Add auto-completion support. Something similar to what https://atom.io/packages/language-ledger and https://github.com/be9/atom-ledger have should do.

It's currently working for account names. I still need to make it work with payees and possibly to make it smart enough to know when it has to autocomplete with an account and when it has to autocomplete with a payee