Fix checkstyle plugin

Description
Fix checkstyle plugin usage.
Issues Resolved
#47
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@77f105f). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #56   +/-   ##
=======================================
  Coverage        ?   83.38%
  Complexity      ?      555
=======================================
  Files           ?       61
  Lines           ?     2058
  Branches        ?      146
=======================================
  Hits            ?     1716
  Misses          ?      236
  Partials        ?      106

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 77f105f...ed2fa46. Read the comment docs.

I see that the DCO check worked on this change which I would approve. Please separated it into its own PR from the check style modifications

Sure @peternied
I didn't intend to fix checkstyle in this PR but the CI was failing due to it. Will raise 2 separate PRs.

Hey -- it looks like you're actively working on this, but I'm confused.  https://github.com/opensearch-project/asynchronous-search/pull/55 was raised, but closed.  Are you raising a separate PR to close #47 ?
Thanks!
/C

Hey -- it looks like you're actively working on this, but I'm confused. #55 was raised, but closed. Are you raising a separate PR to close #47 ?
Thanks! /C

@CEHENKLE I have been suggested to raise a separate PR for DCO check workflow. I can raise that once this fix is merged.

@owaiskazi19 Have added a backlog item to use spotless formatting plugin
https://github.com/opensearch-project/asynchronous-search/issues/58

@CEHENKLE @owaiskazi19
closing this CR and have raised #60 to add checkstyle plugin as a dependency .


Add API Key query to examples

Fixing the examples which seem to have gotten out of date.

Thank you for updating this!


⚠ Bofrost.lu has degraded performance

In 30e96ab, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1287 ms

Resolved: Bofrost.lu performance has improved in 4a97b85 after 20 minutes.


Configurations in yaml not working

Dataset configurations cannot be created in YAML/README
Hello! I'm trying to follow the docs here in order to create structure in my dataset as added from here (#5331): https://github.com/huggingface/datasets/blob/8b8e6ee067eb74e7965ca2a6768f15f9398cb7c8/docs/source/repository_structure.mdx#L110-L118
I have the exact example in my config file for my data repo:
configs:
- config_name: main_data
  data_files: "main_data.csv"
- config_name: additional_data
  data_files: "additional_data.csv"

Yet, I'm unable to load different configurations:
from datasets import get_dataset_config_names
get_dataset_config_names('tsor13/test', use_auth_token=True)

returns a single split, ['tsor13--test']
Does anyone have any insights?
@polinaeterna thank you for adding this feature, it is super useful. Do you happen to have any ideas?
Steps to reproduce the bug
from datasets import get_dataset_config_names
get_dataset_config_names('tsor13/test')
Expected behavior
I would expect there to be two splits, main_data and additional_data. However, only ['tsor13--test'] test is returned.
Environment info

datasets version: 2.14.4
Platform: macOS-13.4-arm64-arm-64bit
Python version: 3.11.4
Huggingface_hub version: 0.16.4
PyArrow version: 12.0.1
Pandas version: 1.5.1

Unfortunately, I cannot reproduce this behavior on my machine or Colab - the reproducer returns ['main_data', 'additional_data'] as expected.

Whoa, in colab I received the correct behavior using my dataset. It must have something to do with my local copy of datasets (which again just failed).
I've tried uninstalling/reinstnalling to no avail

hi @tsor13 , I haven't been able to reproduce your issue on tsor13/test dataset locally either. reinstalling doesn't help?


Fix emscripten wheels

https://github.com/geoarrow/geoarrow-rs/actions/runs/10753602531/job/29823215237
Need to set the compiler flag

Closed by #771


Cannot Build EIM model

I am using a Jetson Nano, trying to generate an EIM from a custom Keras model.
TensorRT version 8
When I try to run APP_EIM=1 TARGET_JETSON_NANO=1 make -j, I get the error
In file included from ./edge-impulse-sdk/classifier/ei_fil_result_struct.h:25: ./model-parameters/model_variables.h:58:19: error: use of undeclared identifier 'trained_onnx_arena_size
What would be causing this error?  Thank you

There was an issue with a TensorRT deployment model_variables, it has now been corrected (arena size is set to 0 for TensorRT).


Remove viewer

It should be possible to download the viewer to the browser from the relay server.

Can you elaborate? What do you mean download the viewer?

Who ever wants to watch the stream shouldn't need to download anything.
Since the relay already exists, it would be more efficient to merge the
two. User visits the relay web site and downloads client side code to run
the viewer on the local browser
On Sun, Mar 24, 2019, 16:29 Fábio Maia<EMAIL_ADDRESS>wrote:

Can you elaborate? What do you mean download the viewer?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/fabiomaia/codestream/issues/4#issuecomment-475975101,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAIZ1N7gsFnJP3ZZZ6dDzOt-71EFwe92ks5vZ6f5gaJpZM4b9PQC
.


chore: bdd-nh-1586940440 to 0.0.1

chore: Promote bdd-nh-1586940440 to version 0.0.1

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

Needs approval from an approver in each of these files:

OWNERS

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


But if I call /live/song/get/track_data x y clip.name and the queried range contains empty clip slots
I don't get an error, but the value NIL for the empty slots.
The same happens for clip color and probably for other properties.
I definitly prefer NIL over an error. This might also solve the problem with starting listeners you described here:
https://github.com/ideoforms/AbletonOSC/issues/49

One thing to bear in mind is that calling start_listen for (e.g.) name or color on a clip_slot that is not populated does not work — because there is no clip in the clip_slot for Live to listen to. So if you wanted to keep a client updated with the correct colour of the slot contained in a clip, you would need to start listening for the presence of a clip in the slot, and then start listening to changes in that clip... which I appreciate may get a bit complicated...

I don't think the two are totally inconsistent; /live/song/get/track_data returns a record for every clip slot on a track, whether populated or not, and nil is the best way to indicate that a clip slot is unpopulated.
That said, I can see the logic in returning nil from queries that fail... as long as that logic ("nil means failure") can hold for every plausible query, so that it can be used as a reliable marker of failure. I've just been looking through the properties of each of the object classes, and I don't think there are any that can be nil...

Names (etc) can be empty strings, but not nil
List properties (e.g. /live/track/get/available_output_routing_types) can be empty lists, but not nil
Int properties that can be "unset" (e.g.  /live/track/get/playing_slot_index) tend to use -1 as a magic value

So I think probably returning nil would be a safe indicator of error.
Would it make it significantly easier to use in the TouchOSC case?


Zeah misc fixes

Fixes some issues with blending around Kourend Castle and Hosidius
Adds textures to the paths and the rest of Kourend Castle's walls
Reverts a texture from custom to OSRS vanilla to improve texture diversity.
Master/PR

Master/PR

Master/PR

Master/PR

Vanilla:

Master:

pr:

A last minute addition

Mount Karuulm textures:
Master/PR:


ARROW-14850: [R] Update ARROW_DEPENDENCY_SOURCE to default to AUTO

@github-actions crossbow submit all

@github-actions crossbow -g r

@github-actions crossbow submit -g r

The other thing we should do, if you haven't already, is document how to install all of the possible dependencies with system packages, which ones can't be installed as system packages, etc. This may be general C++ documentation, but we may also want an R-flavored version since we don't deal with all of the possible C++ dependencies in the R package (some are for features we don't enable, some are test/dev dependencies, etc.). Maybe the docs just start by pointing at ci/docker and all of the cpp docker files there: they install dependencies, set ARROW_DEPENDENCY_SOURCE=SYSTEM, and then also set a few other dependency sources as BUNDLED. Some dockerfiles even have comments explaining why (insufficient version, etc.).

OK, will be sure to cover this in https://github.com/apache/arrow/pull/11521


feat: set commit timeout #ntrn-79

https://hadronlabs.atlassian.net/browse/NTRN-79
checked only by printing in server config in cmd, it's hard to discover cometbft settings in runtime

I think it's base branch issues, waiting to rebase on fix of them

@NeverHappened you have merge conflicts by the way

https://github.com/neutron-org/neutron-tests/actions/runs/5764927281


docs: link to opensearch docs in sidebar

docs: link to opensearch docs in sidebar

Codecov Report
Base: 88.60% // Head: 31.52% // Decreases project coverage by -57.08% :warning:

Coverage data is based on head (22a5a26) compared to base (7d998f1).
Patch has no changes to coverable lines.

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #960       +/-   ##
===========================================
- Coverage   88.60%   31.52%   -57.09%
===========================================
  Files         155      155
  Lines        8050     8050
===========================================
- Hits         7133     2538     -4595
- Misses        917     5512     +4595

Flag
Coverage Δ

docarray
31.52% <ø> (-57.09%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

 Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


support for delete and/or redact of values in store API(s)

Currently the store APIs (esp. timeseries) don't have a delete or redact operation.
But, for example, if we cache tweets in a store then the EULA (and future GDPR) requires that we delete/redact the user content of a tweet in a timely manner if they delete the tweet in twitter. Similar principles apply to other drivers which cache data from external services (e.g. facebook, email) and the principle (GDPR, user control/right to be forgotten) also applies to any of our own drivers/services/apps which take content from users.
In some cases at least - like twitter - it might be better to preserve a stub of the record without the user content (i.e. a record that there was a message that was deleted) rather than delete it entirely. This will presumably need to be relatively efficient, and enforced within the store.

Does the GDPR actually require this? Given that we aren't storing anything at all -- the user (data subject) is responsible, surely?

Yes, maybe you can sidestep GDPR in some deployment cases (the default?!).
But the user can't sidestep the EULA requirements of twitter etc, e.g. if they are cacheing other user's tweets, so the technical facility should still be present IMO.
And if we have research deployments (i.e. a research project deploying databoxes to a group of participants to support a specific research activity) then I think this is also an important facility to have.

@jptmoore is looking into deleting/redacting data for the new store

@cgreenhalgh I don't think it's "sidestepping". IANAL but I don't see how the GDPR is relevant here. Or we'd all have to register as data controllers for storing our own bank statements and suchlike nonsense surely.
What sort of EULA requirements are you thinking of here specifically?

e.g. twitter developer terms

If Content is deleted, gains protected status, or is otherwise suspended, withheld, modified, or removed from the Twitter Service (including removal of location information), you will make all reasonable efforts to delete or modify such Content (as applicable) as soon as reasonably possible, and in any case within 24 hours after a request to do so by Twitter or by a Twitter user with regard to their Content, unless otherwise prohibited by applicable law or regulation, and with the express written permission of Twitter.

Reading those terms, they all seem to me to be framed as if providing a service consuming the Twitter API, not providing an application that the user runs against their own account. Wouldn't Twitter have to issue a request to do so to every Databox operator to invoke that? (Not to the author of the driver.)

My recollection is that the developer guidelines say that you can cache tweets, but you should revalidate the information in the cache, i.e. check if it has been deleted, before presentation to the user if you cache it for longer than that timespan. So the onus is on the user of the API. So logically I think a twitter driver should do something like that automatically (whichever legal entity is responsible for running it).

It's basically our job to do it! "you will make all reasonable efforts to
delete ...." if we have an app that collects and caches tweet...
but, in reality no one does it.. come GDPR, thigns might change
Twitter does not have an informing service API for now AFAIK
cheers
==
Hamed        https://haddadi.github.io
On 18 April 2018 at 17:54, Richard Mortier<EMAIL_ADDRESS>wrote:

Reading those terms, they all seem to me to be framed as if providing a
service consuming the Twitter API, not providing an application that the
user runs against their own account. Wouldn't Twitter have to issue a
request to do so to every Databox operator to invoke that? (Not to the
author of the driver.)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/me-box/databox/issues/198#issuecomment-382455805, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACy0rxKN3XHjmbGjFTWFEcDLpvnVV3PMks5tp2_dgaJpZM4Qh_5O
.


Fix types for mock's aliases

All the mocking API functions should also accept an object as an argument in the matcher parameter like in the mock function.

Hi @lourenci , and thanks for the contribution
This will fail when it hits master because there is a problem with dtslint validation but CI does not run on pull requests from other users at the moment (because of a setting somewhere I need to find and change).
You seem to know typescript. If you could run dtslint --expectOnly types locally and see if it passes that would be great. If you can figure out how to fix the type validation errors even better


STORM-1945 Fix NPE bugs on topology spout lag for storm-kafka-monitor (1.x)

PR for master: #1540

Fix several spots which are throwing NPE on TopologySpoutLag

Use a trick to get classname of Spout without requiring storm-kafka and storm-kafka-client

Move storm-kafka-monitor jar to toollib directory

since jars in extlib are added to worker classpath which we don't want for this jar

@priyank5485 @harshach Please take a look and comment. Thanks!

+1.

@HeartSaVioR while we are here can we fix the JSON response as well.
currently what we have is this
[{"spoutLagResult":"[{"topic":"topic","partition":0,"consumerCommittedOffset":1175610,"logHeadOffset":5634192,"lag":4458582},{"topic":"topic","partition":1,"consumerCommittedOffset":1251325,"logHeadOffset":5634192,"lag":4382867}]","spoutId":"spout","spoutType":"KAFKA"}]
we should be returning an array instead lets have something like this
{ "spoutId" : "spout"
"spoutType" :  "KAFKA"
"spoutLagResult" : [
"topic": // key is topic name
["partition0": {"consumerCommittedOffset":1175610,"logHeadOffset":5634192,"lag":4458582},
"partition2: {"consumerCommittedOffset":1175610,"logHeadOffset":5634192,"lag":4458582},
],
"topic2" :
["partition0": {"consumerCommittedOffset":1175610,"logHeadOffset":5634192,"lag":4458582},
"partition2: {"consumerCommittedOffset":1175610,"logHeadOffset":5634192,"lag":4458582},
]
}

@harshach
I guess you mean this:
[
   {
      "spoutId":"spout",
      "spoutType":"KAFKA",
      "spoutLagResult":{
         "topic":{
            "partition0":{
               "consumerCommittedOffset":1175610,
               "logHeadOffset":5634192,
               "lag":4458582
            },
            "partition2":{
               "consumerCommittedOffset":1175610,
               "logHeadOffset":5634192,
               "lag":4458582
            }
         },
         "topic2":{
            "partition0":{
               "consumerCommittedOffset":1175610,
               "logHeadOffset":5634192,
               "lag":4458582
            },
            "partition2":{
               "consumerCommittedOffset":1175610,
               "logHeadOffset":5634192,
               "lag":4458582
            }
         }
      }
   }
]

I agree this is better for removing duplicated thing and also easy to find by path. I'll update it.

It would be better for changing JSON response to file a new issue and make it out of scope for this PR, since STORM-1945 is a bugfix which is marked as critical.
Filed new issue for changing JSON response format: STORM-1950


Can this be used in a function component?

Question
I'm trying to use this within a function component, using the react useState hook, but it's causing poor behaviour when scrolling the map
function Map () {
  const [region, setRegion] = React.useState({
        latitude:  52.520008,
        longitude: 13.404954,
        latitudeDelta: 0.01,
        longitudeDelta: 0.1,
  })

  return (
        <MapView
          region={region}
          style={styles.map}
          onRegionChange={setRegion}
          rotateEnabled={false}
          scrollEnabled
          zoomEnabled
      >
  )
}

It seems like the onRegionChange isn't setting the region into the state quick enough, so the map scrolls and then gets jerked back. Is there anyway to make this useable with hooks or should I use a react component?

Please udpate your issue following the template or I'll delete it. You already put a lot of effort in giving details :)

Like this ☝ ? 😬

onRegionChange triggers A LOT and crossing the bridge is costly, maybe you can use onRegionChangeComplete?

I'm rendering a Circle around the region of the MapView, and I'd like the circle to move along as the map moves, which means onRegionChangeComplete won't work for me 😢 . I should use a React class component or is there a way to get onRegionChange to work?

You can try, but I don't know :(

I can't get it to work with a class component either 😭, it ends up too laggy. The best I can do is update the circle location with onRegionChangeComplete. It's not the exact behavior I want, but it's usable at least. If anyone has any other ideas let me know!

@peterzernia another thing you could try is to limit the number of decimals in the region to 6 (or less?) and  use useMemo to limit the rendering?
I don't use this module since a long time ago but I remember the change event has a lot of decimals in the coordinates

Do you need this to be in state? Do you need to pass the region back to the component? Consider using a ref instead and / or let the map save it's internal state and not manage it from outside if this isn't necessary.


Add lima/vscode ssh plugin dev environment

How to categorize this PR?
/kind enhancement
/area os
/os garden-linux
What this PR does / why we need it:
Adds a definition for a Garden Linux dev environment using lima which can be used with the VS Code SSH Remote plugin.
This adds a free software alternative for Garden Linux dev environments on non-linux host OS (except for the non-OSS VS Code parts).
SSH remote development allows a almost 'native' feel for editing/development in VS Code.
Which issue(s) this PR fixes:
Special notes for your reviewer:
Release note:

Nice 👍🏻
VS Code Remote Explorer does not seem to recognise the Included ssh config via:
echo "Include ${LIMA_HOME:-$HOME/.lima}/gl-dev/ssh.config" >> ~/.ssh/config

If I place the content of gl-dev/ssh.config directly in my .ssh/config, then it works as expected for me.
Looks to me like a bug in VS Code.. Including is way more acceptable than writing stuff in the ssh/config of the user directly in my opinion..

VS Code Remote Explorer does not seem to recognise the Included ssh config via:

I got this from here
Any issue with the shell you are using?
For me it correctly expands to
Include /Users/mylocalusername/.lima/gl-dev/ssh.config

inside the ~/.ssh/config file. I'm using zsh on the mac, but I think it should also work with bash or other posix shells.

The problem (I think) is my VS Code.
Include /Users/myuser/.lima/gl-dev/ssh.config is correctly added to the .ssh/config, but VS Code does not detect the lima-gl-dev config


[3.x] Fix touch events when using smartphone AR with WebXR

After https://github.com/godotengine/godot/pull/55869, Control's on a CanvasLayer will actually be shown on the screen when using smartphone AR. However, the Controls don't recognize any screen touch or drag events.
This PR will generate InputEventScreenTouch and InputEventScreenDrag events when using smartphone AR, so the Controls will actually work!
To implement this, I needed to use a piece of information from WebXR (XRInputSource.targetRayMode) that I think would also be useful for game developers, so this also adds a new method to the API: WebXRInterface.get_controller_target_ray_mode()
Discussed briefly on RocketChat with @akien-mga that this could be done only for 3.x for now, until we're actually able to test WebXR on 4.x.

Ah, looks like my local 3.x branch was out-of-date. I'll rebase, test and force push in a little bit.

Thanks!


Main page including all components created for accessibility testing

Pull Request Background
The work in this pull request was created as a proof of concept for our accessibility tests. One main components page was created, incorporating all components and then tests were run across the entire page.
GitHub Issue
Build Accessibility test page
What Changed

Created a new md file all-components.md that includes all components on one page, located at the URL /components/all-components
Updated accessibility tests to run on the new main accessibility page

How to Check Change

[ ] Make sure server is running at localhost:8000 and run npx cypress run in the command line to run accessibility tests. They should be passing.
[ ] Run sh ./ci/linter.sh to run linter tests. They should be passing.

Automated Testing
I created a new accessibility test in the Cypress spec file. This tests the new page that holds all components at /components/all-components

[x] All node tests are passing.
[x] All linter tests are passing.
[x] All accessibility tests are passing.

Additional Context
In the spec file, I commented out the accessibility tests that were created previously for the array of component URLs. I think future devs may benefit from seeing the example but this is open to discussion.

@LynnHerrick9 and @jeffmaher This is ready for review when you have time! Thank you!


why cv2.rectangle does not work meeting a numpy slice or flip?

import cv2
import numpy as np
image = np.random.rand(512,512,3)
image = image[:,:,::-1]
cv2.rectangle(image,(10,10),(100,100),(0,0,255))
cv2.imshow('',image)
cv2.waitKey(0)

when  I change image = image[:,:,::-1] to image = np.array(image[:,:,::-1]), it's ok

This is a issue tracker, not a generic Q&A forum. Please ask your question at https://answers.opencv.org/questions/ or at https://stackoverflow.com/.


Adding more scripture API

I was able to make it work.

Hi @Saorsa32
Which API are u suggesting, do you have the API doc/url? :)
Thanks

I wanted to add the LDS Scriptures, Here is a link to the page with the information I have. https://scriptures.nephi.org/ I was able to get what I needed to work, by downloading the Markdown version and using the Bible linker plugin. But maybe you would be able to to make it work with your plugin. Thanks


[SPARK-41810][CONNECT] Infer names from a list of dictionaries in SparkSession.createDataFrame

What changes were proposed in this pull request?
This PR proposes to support to infer field names when the input data is the list of dictionaries in SparkSession.createDataFrame.
For example,
spark.createDataFrame([{"course": "dotNET", "earnings": 10000, "year": 2012}]).show()

Before:
+------+-----+----+
|    _1|   _2|  _3|
+------+-----+----+
|dotNET|10000|2012|
+------+-----+----+

After:
+------+--------+----+
|course|earnings|year|
+------+--------+----+
|dotNET|   10000|2012|
+------+--------+----+

Why are the changes needed?
To match the behaviour with the regular PySpark.
Does this PR introduce any user-facing change?
No to end users because Spark Connect has not been released.
How was this patch tested?
Unittest was added.

cc @zhengruifeng

all related tests passed.
Merged to master.


HOSTEDCP-2063: [CPO Refactor] Refactor openshift-apiserver

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #
Checklist

[ ] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/lgtm

/override "Red Hat Konflux / hypershift-operator-main-on-pull-request"

/retest

/lgtm

/override "Red Hat Konflux / hypershift-operator-main-on-pull-request"


Issue #8. Add links in see tags

makeMLDoc.sh (updated)
Create the folder Publish to fetch the local documentations under the home directory as docs and source folder.
Current SEE links can link to the root level elements, e.g ML_Core.Types.NumericField. The SEE tag link can direct to ML_Core.pdf. Inside the same PDF, it requires full link to correctly fetch the tag


set className to a specific column

I just don't understand from the documentation nor from the code - how can I set the className for a specific column?

In the section API of the docs, ColumnDefinition component, a "cssClassName" prop is mentioned, but it doesn't work, like "customHeadingComponent", "headerCssClassName" and "width" (it sets style on body's td, but not in header's td) props.

Once a new version ships, set cssClassName and headerCssClassName on your <ColumnDefinition /> elements, e.g. this story.

<EMAIL_ADDRESS>includes this fix


fsverity_linux.go: Fix fsverity.IsEnabled() for big endian systems

I found out that unix.Syscall(..., uintptr(unsafe.Pointer(&return_value))) returns very big value.
If syscall returns 128 (not verity, just example) on other systems, it returns<PHONE_NUMBER>88 on s390x, so fsverity.IsEnabled is almost incorrect. Note that 128 << 32 ==<PHONE_NUMBER>88.
This happens because s390x is Big Endian, whereas other ones are Little Endian.
One fix is to change attr variable type to int32 , but this won't work if containerd is compiled against or used on system where C int is not 4 bytes wide.
This PR includes change of attr variable type to int32.

https://github.com/torvalds/linux/blame/master/include/uapi/linux/fs.h#L223-L224

Oh, didn't see this one. Then int32 assumption is very incorrect, but as we see int64 or long doesn't work with big endian machines.
Is there other ways to work around this issue for big endian machines?

I checked all possible output flags for FS_IOC_GETFLAGS and their total bitwise OR value is<PHONE_NUMBER>, which is less than uint32 maximum limit. Also, getting all extended attributes enabled is probably impossible, all i managed to get is 252 without verity and 1048576 with it.
--S-iadAc------------- 252
--------------------V- 1048576

Also, i asked about this on Golang issue tracker and got this response: https://github.com/golang/go/issues/70302#issuecomment-2471127093
If this is OK we can change attr variable to uint32 and merge it. I tested it briefly on s390x and it worked as expected:
[xbt573@localhost xd]$ lsattr test
--------------------V- test
[xbt573@localhost xd]$ ./xd test
1048576

#define FS_VERITY_FL			0x00100000 /* Verity protected inode */
// 0x00100000 == 1048576

I checked all possible output flags for FS_IOC_GETFLAGS and their total bitwise OR value is<PHONE_NUMBER>, which is less than uint32 maximum limit. Also, getting all extended attributes enabled is probably impossible, all i managed to get is 252 without verity and 1048576 with it.
--S-iadAc------------- 252
--------------------V- 1048576

Also, i asked about this on Golang issue tracker and got this response: golang/go#70302 (comment)
If this is OK we can change attr variable to uint32 and merge it. I tested it briefly on s390x and it worked as expected:
[xbt573@localhost xd]$ lsattr test
--------------------V- test
[xbt573@localhost xd]$ ./xd test
1048576

#define FS_VERITY_FL			0x00100000 /* Verity protected inode */
// 0x00100000 == 1048576

thx was that s390x system 32 or 64bit.. can move along but would be nice to know if we have an endian issue with c long 64bit..

Isn't c long int64 on 64 bit systems.. Should this be platform dependent?
https://github.com/torvalds/linux/blame/master/include/uapi/linux/fs.h#L223-L224

See the note in the man page: https://man7.org/linux/man-pages/man2/ioctl_iflags.2.html#NOTES


RSS Feed

I would like to have to option to add this website to my RSS Feed. Can you please create an feed.xml file?

You can find the feed here: http://screenings.io/feed.xml :)


Add --web-user-data-dir option for Chrome launch.

Description
This PR enables the use of the user-data-dir option on flutter run -d chrome command. I added the option as --web-user-data-dir run flag to follow the other web flags name pattern.
Related Issues
Closes: #47872 - Allow specifying --user-data-dir for chrome launch
Tests
I added the following test:
packages/flutter_tools/test/general.shard/web/chrome_test.dart: 'can use existing user data dir'
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[X] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[X] I signed the [CLA].
[X] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[X] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[X] I updated/added relevant documentation (doc comments with ///).
[X] All existing and new tests are passing.
[X] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[X] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read [Handling breaking changes].

[X] No, no existing tests failed, so this is not a breaking change.
[ ] Yes, this is a breaking change.

Approach generally looks good, Let me know when you're ready for a review

Approach generally looks good, Let me know when you're ready for a review

It's ready for a review.

Because of the lack of activity on this PR I'm going to close it to remove from our review queue. Please feel free to send us a new PR or discuss in an issue anytime.


Silences placed in future creates a dead-lock.

When creating a future-silence, the silence is created with a delete button that doesn't function. I see no clear way of disarming or deactivating the silence.

I got into the same issue when working on the new UI. When I want to delete a pending silence I get:

On it.

I fixed the issue semantically. But as we never delete but only expire, afterwards the silence still exists but has an effective duration of 0. It is still shown in the "pending" tab.
I think there are two options: Completely hide any silence at UI or API level where startsAt == endsAt or sort it into the "expired" tab even though the start lays in the future.

Another alternative would be to add this logic on the server in types.go/CalcSilenceState and set any Silence to Expired where startsAt == endsAt. What do you think? @w0rm @fabxc

I would prefer showing them as. It would be surprising to hide them or not,
just because they were expired before or after they begin. A silence that
will never be active has the same effect as one that did have an effect on
the past but not the future.
I think server side is the right place for this logic.
On Mon, May 22, 2017, 09:18 Fabian Reinartz<EMAIL_ADDRESS>wrote:

I fixed the issue semantically. But as we never delete but only expire,
afterwards the silence still exists but has an effective duration of 0. It
is still shown in the "pending" tab.
I think there are two options: Completely hide any silence at UI or API
level where startsAt == endsAt or sort it into the "expired" tab even
though the start lays in the future.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/prometheus/alertmanager/issues/783#issuecomment-303018808,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAICBrfzUwJqdHBqvp5XxjR2oOrLJKt_ks5r8TbVgaJpZM4NWglD
.

Hiding will cause confusion. I'd prefer that it is placed in expired.

In #811 @RSamal @fabxc and came up with a third solution. Instead of setting endsAt to startsAt we can set both to now. Thereby the Silence will get the expired state by default.


Tilting in shooters is problematic

If you hold down the Q button, the script works, but if you hold down the E button, the script stops working.

Ahah thanks for the cute diagram. It happens because "E" is default bind for temporarily disabling socd (see fifth row in the config file, and the readme about how it's structured). Rebind it to something else that you don't use and it should be good.


Cache gem dependencies in CI

Around 20 seconds is spent on installing gems, using gem caching it should make the tests even faster.

It looks like this caused a bunch of test failures. Is this something you are still interested in (I know it's been a long time), or shall I close this PR for now?

This seems to be incompatible with Appraisal. I'm not currently feeling the pain, so I'm going to close this. Thank you for trying!


Fixing https upgrade script

Script to generate list of domains which can be upgraded to https was giving incorrect results, example for domains where the certificate is invalid, it should not be upgraded.
verify_ssl=True enforces, even if certificate is present, it should be valid.
We currently only print the certificate exception, as a different PR we should save it somewhere for different analysis.
Usage:
python main.py db check_urls
cc @ecnmst

thanks @konarkmodi


Remove BeanUtils2/ConvertUtils2

@garydgregory its been a month an no sign from Nial.  Should we make this chnage?   If we do then we can work on formatting all the files of the project next,

@melloware
Sorry busy w day job. I want to take some time this week-end to catch up on Apache Commons.

No worries i totally understand.


Plugin not working android 9.0

When I build an application in android SDK level 28 or higher plugin not working in Android 9.0 (Pie)
but working fine on below android 9.0 devices

my app is working fine on android pie

I have got this plugin running in 2 apps and no problem with android 9.0 and SDK 28.
Can you give more information about your problem?
Say that it does not work without providing information about your project makes it very difficult to help you

@videmort thanks for your response
my code is:
cordova.plugins.CordovaCall.callNumber("0000000", onsuccess, onerror);
function onsuccess() {
alert('onsuccess');
}
function onerror() {
alert('error');
}
same code working fine on android 8.1 devices but not working on android 9.0
in both devices showing on onsuccess message

@IsmailAlam786   Did you call the function after the deviceready event.
Can you share this info:
Cordova:
Cordova CLI
Cordova LIB
Cordova Platforms
Cordova Plugins
System:
Android SDK Tools version
NodeJS
npm
OS

Cordova:
Cordova CLI =  9.0.0
Cordova LIB = 9.0.0<EMAIL_ADDRESS>Cordova Platforms = android
Cordova Plugins = CordovaCall
System:
Android SDK Tools version = 28
NodeJS = v10.15.3
npm = 6.4.1
OS = windows 10

@IsmailAlam786  cordova-android version? --> Run in your terminal cordova platform ls
Can you try if this method works?
cordova.plugins.CordovaCall.receiveCall('Test');

@videmort
cordova platform ls
Installed platforms:
android 7.1.4
yes this method cordova.plugins.CordovaCall.receiveCall('Test'); working fine on both devices

Try with lastest version of cordova android 8.1.0

with<EMAIL_ADDRESS>still the same experience. Receive works, callNumber doesn't.

Same issue faced on Android 11 device. Kindly advise.
Below code is not bringing up the dialer or making a call. onCallSuccess function gets called.
cordova.plugins.CordovaCall.callNumber("8590638740", onCallSuccess, onCallError);

node -v
v16.13.1

npm -v
8.1.4

cordova -v
11.0.0

cordova platform ls
Installed platforms:
android 9.0.0

cordova plugin ls
cordova-call 1.1.6 "CordovaCall"
cordova-plugin-appminimize 1.0.1 "AppMinimize"
cordova-plugin-network-information 3.0.0 "Network Information"
cordova-plugin-whitelist 1.3.4 "Whitelist"
cordova-plugin-x-toast 2.7.3 "Toast"

Android SDK Build Tools v32.0.0
API levels: 22 - 29
Testing on Android v11.


Update languages.md for PR#36 (zh-cn)

finished translation of Mercury tutorial 1

Relates to #36

Thanks! Closing as we solved it elsewhere.


API for posting and viewing orders

Closes: https://github.com/gnosis/dex-services/issues/1433
testplan:
cargo run

and in a new terminal
curl --location --request POST 'localhost:3030/v1/orders' \
--header 'Content-Type: application/json' \
--header 'Content-Type: text/plain' \
--data-raw '{
    "sell_amount": "0x10",
    "buy_amount": "0x10",
    "buy_token": "0xA193E42526F1FEA8C99AF609dcEabf30C1c29fAA",
    "sell_token": "0xFDFEF9D10d929cB3905C71400ce6be1990EA0F34",
    "owner": "0x63FC2aD3d021a4D7e64323529a55a9442C444dA0",
    "nonce": 1,
    "signature_v": 27,
    "signature_r": "0x07cf23fa6f588cc3a91de8444b589e5afbf91c5d486c512a353d45d02fa58700",
    "signature_s": "0x53671e75b62b5bd64f91c80430aafb002040c35d1fcf25d0dc55d978946d5c11",
    "valid_until": "0x0"
}'

Reposting my personal message to @fleupold  for more transparency:
I was trying to switch tokioo against async_std. Here is my code:
It compiles fine, but when I am running cargo run, I am getting the following error:
thread 'main' panicked at 'there is no reactor running, must be called from the context of Tokio runtime', /Users/alexherrmann/.cargo/registry/src/github.com-1ecc6299db9ec823/tokio-0.2.22/src/io/driver/mod.rs:201:9

somehow the warp lib wants to get started with tokio and not with async_std.
I found the following hint here as well:
2. Warp uses tokio as its async task runner behind the scenes. You won't have to care for it much to make the example work except "its alternative is async std and they are not that compatible currently".

Do you think that it is reasonable to stick to tokio under these given assumptions? I don’t know the libraries too well and can’t do a proper judgment
Also, I have seen that the join! commands are only available in the unstable variant.

Do you think that it is reasonable to stick to tokio under these given assumptions?

It looks like we are actually using the tokio runtime in the pricegraph (https://github.com/gnosis/dex-services/blob/7c7237b52d1d5614d73935d84fb706fd40f4c3f6/price-estimator/Cargo.toml#L21)
Maybe @nlordell recalls why we preferred async_std where possible. For the tests, I don't think you need to use await but can instead assert that everything returns now_or_never synchronously (since you shouldn't need to actually await on things in a unit test).

Warp requires tokio, which is why we use it in the price-estimator service.

I don't think you need to use await but can instead assert that everything returns now_or_never synchronously

This gets tricky with tokio timers, where, from what I recall, always yield. Personally, I really like using tokio::test. Asserting with now_or_never as @fleupold suggests can be useful when you want to assert that the future is expected to complete immediately, but this is not always what the test cares about.

Maybe @nlordell recalls why we preferred async_std where possible.

You can take a look at the short discussion @e00E and I had about why we use it in the driver gnosis/dex-services#843.  At the time, we were using the futures::executor quite extensively, so async-std seemed like an easier choice to simplify integration.
async-std is about providing an out of the box async version of the std library, (that being said, tokio is very complete and provides a very nice async-focused API, which in many cases is very similar to the std library).

They're both great projects with huge communities, that being said, crates like web3 (and now ethcontract since updating to the latest web3) use tokio internally, so it makes more sense now to go with tokio.

I am facing a weird issue:
Due to the fact that the new tokio::sync::rwlock is async, I can not implement a serialize function for the OrderBook struct for the serde trait Serialize., as the serialize function is required to be sync code. The trait needs a non-async serialize function.
This challenge is also explained here:
Unfortunately, there is no real solution proposed in this thread besides this very hacky way of just creating a new struct, which is not "underneath" a rwlock. ( as I pushed it in my latest commit).
Do you guys know about a better solution? @nlordell @fleupold

I don't think the proposed solution is very hacky. I found it a little weird to begin with that the Orderbook with the lock itself was serializable. In the future this class might grow and take more unserializable dependencies (e.g. a way to fetch balances or the like), thus having a raw unlocked plain old data model that is serializable seems good to me.

I also agree. I think de/serializable data should be plain data objects. Then types that control concurrent access would have RwLocks in order to share this data.


Use current CCG membership in presentations by practice API query

This will fix the chart at https://openprescribing.net/analyse/#org=practice&orgIds=14L&numIds=0703021Q0BB&denom=nothing&selectedTab=chart
This has no new tests -- updating the test fixtures is too much work as it breaks lots of other tests

Genuine test error:
======================================================================
ERROR: test_spending_by_practice_on_multiple_presentations (frontend.tests.test_api_spending.TestSpendingByPractice)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/code/openprescribing/frontend/tests/test_api_spending.py", line 530, in test_spending_by_practice_on_multiple_presentations
    rows = self._get_rows(params)
<snip>
ProgrammingError: column pc.pct_id does not exist
LINE 13: ... pr.presentation_code LIKE '0202010B0AAABAB') AND pc.pct_id ...
                                                              ^
HINT:  Perhaps you meant to reference the column "pr.pct_id".

I agree with the hint - although it could also be pc.code, but I that would probably result in a less performant query.
I also just spotted this in the test output, not sure how long that's been going on:
Traceback (most recent call last):
  File "./scripts/wait_for_postgres.py", line 16, in <module>
    host=environ['DB_HOST'],
  File "/usr/local/lib/python2.7/UserDict.py", line 40, in __getitem__
    raise KeyError(key)
KeyError: 'DB_HOST'

Gah, amateur hour!  Have fixed by changing .pct_id to .ccg_id.  (In my defence, it is a bit confusing...)

Per my concern about performance, this query has conditions on two different tables. It is likely to be more performant (as it could use one index) to apply compound conditions to the largest table first, i.e. against pr rather than pc.
I've not given this masses of thought so might be wrong. Would at least be a good idea to try out on staging before merging, if you've not already done so, and compare performance.

I am starting to review outstanding issues properly in the issue tracker, and noticed what I think describes this problem here - worth reviewing to check you agree they're the same issue and that this PR would fix that issue

Have benchmarked the queries, and the new query is faster (~520ms vs ~750ms, best of three).  The EXPLAIN output is 231 rows and so I haven't dug into why!
You're right that this comment and #479 are related.  They will be fixed in some cases by this PR, and in other cases by #846 (which is merged but we haven't done an import since the merge), depending on whether the analysis is for chemicals or presentations.

I'm happy if you want to merge & deploy then

Before:

After:


Support use of note names like "C4" or "G#3" as input to "play note" block?

Another idea for making the input to the "play note" more intuitive and musical: support both numeric input of MIDI note numbers (as in Scratch 2.0) and also note name/octave notation like "C4" or "G#3" or "Eb2" (similar to "scientific pitch notation). We could also automatically convert valid note names without an octave to the default octave. So these would all play the same note:
Play note 60
Play note C4
Play note C

I’m assuming the default octave is 4?
What would it play for invalid inputs? Silence?

Be careful with this enhancement request, as the note names do vary depending on the country. Ex. "B" is "H" in German, then there is the Latin countries' usage ("SI") etc. So it would need to be localized per user location in order to provide the best user experience.


cppzmq: 4.8.0 -> 4.8.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/zeromq/cppzmq/releases.
meta.description for cppzmq is: "C++ binding for 0MQ"
meta.homepage for cppzmq is: "https://github.com/zeromq/cppzmq"
meta.changelog for cppzmq is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 4.8.1 with grep in /nix/store/c9y2s0qvvymvm6fbl7bjz1zpcjzr9lr0-cppzmq-4.8.1

found 4.8.1 in filename of file in /nix/store/c9y2s0qvvymvm6fbl7bjz1zpcjzr9lr0-cppzmq-4.8.1

directory tree listing: https://gist.github.com/2af1c4372aaddf04bc1f9ea86942cd0b

du listing: https://gist.github.com/75f749a8fe3c1d61af25f854b5a83cc1

Rebuild report (if merged into master) (click to expand)

97 total rebuild path(s)

27 package rebuild(s)

27 x86_64-linux rebuild(s)
26 i686-linux rebuild(s)
18 x86_64-darwin rebuild(s)
26 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
aeon
cppzmq
gnss-sdr
gnuradio
gnuradio3_7Minimal
gnuradio3_8
gnuradio3_8Minimal
gnuradio3_8Packages.ais
gnuradio3_8Packages.gnuradio
gnuradio3_8Packages.limesdr
gnuradio3_8Packages.osmosdr
gnuradio3_8Packages.rds
gnuradioMinimal
gnuradioPackages.gnuradio
gqrx
horizon-eda
inspectrum
irods
irods-icommands
libsForQt5.libopenshot
libsForQt512.libopenshot
libsForQt514.libopenshot
libsForQt515.libopenshot
openshot-qt
plasma5Packages.libopenshot
qradiolink
ursadb

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/c9y2s0qvvymvm6fbl7bjz1zpcjzr9lr0-cppzmq-4.8.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A cppzmq https://github.com/r-ryantm/nixpkgs/archive/5f012c3a26b007ef54b0301997b31e82845f8bec.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/c9y2s0qvvymvm6fbl7bjz1zpcjzr9lr0-cppzmq-4.8.1
ls -la /nix/store/c9y2s0qvvymvm6fbl7bjz1zpcjzr9lr0-cppzmq-4.8.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

24 packages failed to build:

aeon
gnss-sdr
gnuradio
gnuradio3_7Minimal
gnuradio3_8
gnuradio3_8Minimal
gnuradio3_8Packages.ais
gnuradio3_8Packages.gnuradio
gnuradio3_8Packages.limesdr
gnuradio3_8Packages.osmosdr
gnuradio3_8Packages.rds
gnuradioMinimal
gnuradioPackages.gnuradio
gqrx
horizon-eda
inspectrum
irods
irods-icommands
libsForQt5.libopenshot
libsForQt512.libopenshot
libsForQt514.libopenshot
openshot-qt
qradiolink
ursadb

1 package built:

cppzmq

Maintainer pings
cc @abbradar for testing.

@ofborg eval


{Misc} Replace aliased errors with OSError

Description
Taken out from #30368
Motivation given in https://docs.astral.sh/ruff/rules/os-error-alias/
ruff check --select "UP024" --fix

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Thank you for your contribution! We will review the pull request and get back to you soon.

According to https://docs.python.org/3/library/exceptions.html

The following exceptions are kept for compatibility with previous versions; starting from Python 3.3, they are aliases of OSError.
exception EnvironmentError
exception IOError
exception WindowsError
Only available on Windows.

@atombrella, Thank you very much for the contribution!


Ubuntu 15.04 - proved to work?

Hi, is dokku-alt proved to work with Ubuntu 15.04?
I get the following error:
[19:58:18] ~/Projects/wdc> git push wdc master
Counting objects: 9119, done.
Delta compression using up to 4 threads.
Compressing objects: 100% (6177/6177), done.
Writing objects: 100% (9119/9119), 2.72 MiB | 0 bytes/s, done.
Total 9119 (delta 5489), reused 4157 (delta 2245)
remote: Access denied. No user supplied.
To dokku@*.*.*.*:wdc-app
 ! [remote rejected] master -> master (pre-receive hook declined)
error: failed to push some refs to 'dokku@*.*.*.*:wdc-app'

Ok, I don't know what was the problem but the issue is gone.


add total votes percentage

Motivation
This PR adds percentage of votes for each candidate. Will be helpful to see the relative difference for recounts
Before:

After:

Changes

[x] Ensured that you have rebased your branch with this repo's latest master branch.
[x] Ensured that relevant issues are linked, if this PR resolves any outstanding.
[x] Added a screenshot for all UI changes (you can drag the file into this edit box and it will be uploaded).
[x] Ensured that changes to auto-generated files have not been committed; in particular, *.html, *.csv, *.xml and *.json files.

I think this solves https://github.com/alex/nyt-2020-election-scraper/issues/339 in a less visual way too.


Add basic image preprocessing

Add basic image preprocessing, including:

conversion to grayscale
sharpening
threshold
anything else appropriate

Contrast adjustment has been added. However, it would be a good idea to rescale the image to a smaller size to make the pixel transformations take less time.


how to use MutliAgent task type

I see MutliAgent task type in the code. I wonder how can I use it? Does it mean loading two or more robot arms and hands in a environment? If yes, Could someone tell me how to use it in code?

Hi @nutsintheshell ,
Thank you for your careful observation. We don't have the multi-agent setting, the reason we have this parameter is because our code is modify from this repo: https://github.com/PKU-MARL/DexterousHands, which has the multi-agent setting. You can try their code If you are interested.
Hope this can help you.


This is all wrong, none of this works or is even well written.

[Enter feedback here]

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

ID: f5b066ca-c13e-9f25-3add-844012d105f1
Version Independent ID: 47ac81a1-d4a7-8d74-b59a-2cf143a9684c
Content: Get started with NuGet packages - Azure Artifacts
Content Source: docs/artifacts/get-started-nuget.md
Product: devops
Technology: devops-artifacts
GitHub Login: @ramiMSFT
Microsoft Alias: rabououn

@marc-downer-tdx Thank you for your feedback. The article has been updated few times since you posted the issue. I hope you will find this helpful.


Adds the PR template for Amundsen frontend

Mostly copied from Apache Airflow, and using the same in metadata repository.
Fixes: https://github.com/lyft/amundsenmetadatalibrary/issues/16
cc: @danwom @feng-tao

@verdan thanks. Could you add one for databuilder repo as well?

@verdan thanks for adding this! I'll have some updates to this later to specify what will be expected of changes to the application source (e.g. instruction about docstrings only applies to python code, and there's also a few other commands to add for testing)


optimize sec black list

use resource file and switcher to define blacklist .so user can set it by themself
use java resource path instead of the absolute path
use serialize.blacklist.file to define it.

CI failed.
Add this  to pom.xml
<build>
        <resources>
            <resource>
                <directory>src/main/resources</directory>
            </resource>
        </resources>


Fix negative indices error for edit commands

Fixes #214
I have also checked other similar commands, both deletes, mark/unmark and they do not have the same bug.

Codecov Report

Merging #233 (fa8e143) into master (c77ab1d) will decrease coverage by 0.16%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master     #233      +/-   ##
============================================
- Coverage     70.31%   70.14%   -0.17%
+ Complexity      901      899       -2
============================================
  Files           129      129
  Lines          2749     2737      -12
  Branches        360      360
============================================
- Hits           1933     1920      -13
- Misses          612      613       +1
  Partials        204      204

Impacted Files
Coverage Δ

...ogic/parser/contacts/ContactEditCommandParser.java
92.00% <100.00%> (-1.55%)
:arrow_down:

...book/logic/parser/tasks/TaskEditCommandParser.java
83.87% <100.00%> (+5.49%)
:arrow_up:

...skbook/logic/parser/exceptions/ParseException.java
50.00% <0.00%> (-50.00%)
:arrow_down:

...book/commons/exceptions/IllegalValueException.java
50.00% <0.00%> (-50.00%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


String.upper and String.lower should do more than just ascii

What the title says. The best way to do this would be by bringing in some external library for utf-8 handling. utf8proc seems like it would be a good fit.

Closing this as part of the gitlab migration: https://gitlab.com/FascinatedBox/lily/issues/314


How to access the hex inside the Hash()

blake3::hash(b"str") -> Hash(hex)  and I want the hex stripped out from the Hash() as a str
This change saves time so that the next person doesn't have to read the source code to locate .to_hex()

Good idea, thank you. I'll land this, and then make my preferred changes on top of it.


README alludes to the ability to generate python servers, but this doesn't seem to be the case

We're evaluating various RPC frameworks here at Patreon, and I suggested we look into Twirp since it seemed to have the ability to generate Python servers (we're a Python shop).
Specifically, the line (in README.md) This repo only has the official generators, which write out Go and Python code, while not specifically stating that Python servers are generated, also doesn't completely rule it out.
To prevent disambiguity, this might be something you want to add to the table, as well as reformatting to This repo only has the official generators for Go (client & server) and Python (client)

Pasting my comment from the slack channel.
We have an internal python server implementation that we use on a couple of services. We can clean it up and open source/contribute.
Current opinions up for debate:

Server is an asgi server (abstracted so wsgi should be possible without a lot of work)
We have a context object that we use similar to go. (There's no context cancelling on it yet)
The context object forces use of a logger, defaults to structlog.

@ofpiyush - any rough estimates on the timeline to clean up and eventually release?

Didn't realise there was pull for it!
I'll ping folks in the company on Monday and post an update as soon as I find a taker.

We figured that none of the python servers are open to public internet and there's no sensitive information in the repo so here's the link to our current version.
We'll remove verloop specific things and bad practices from here and make proper releases.
https://github.com/verloop/vtwirp/tree/vcontext
@spenczar what is the eventual goal here? Should we work towards merging this into the main repo some day?

@ofpiyush please feel free to submit a PR adding a link to vtwirp when you are ready. I think it makes the most sense for new clients and servers to exist in their own repositories to establish clear ownership.

@dpolansky we can close this now as the readme has also changed.


Pylance not working at all

Environment data

Language Server version: v2023.12.1
OS and version: Arch Linux
Python version (& distribution if applicable, e.g. Anaconda): Python 3.11.6

Code Snippet

All settings from settings.json related to Python:

    "python.languageServer": "Pylance",
    "python.analysis.autoImportCompletions": true,
    "python.analysis.autoFormatStrings": true,
    "python.analysis.completeFunctionParens": true,
    "python.analysis.enablePytestSupport": false,
    "python.analysis.logLevel": "Trace",
    "files.associations": {
        "*.py": "python"
    }

Repro Steps

Open a .py file
See that you're back to stone age with no autocomplete, suggestions, help menus on hover etc.

Expected behavior
To actually work.
Actual behavior
None of the features of Pylance are active: no autocomplete, no suggestions, no help when hovering over functions etc.
I have two PCs with Arch Linux and both exhibit the same behavior.
Logs

Logs for the Python extension:

2024-01-22 14:40:48.940 [info] Telemetry level is off
2024-01-22 14:40:48.940 [info] Experiments are disabled, only manually opted experiments are active.
2024-01-22 14:40:48.940 [info] Default formatter is set to null for workspace /home/reloadedd/Documents/programming/Python
2024-01-22 14:40:48.940 [info] Test server listening.
2024-01-22 14:40:48.940 [info] Python interpreter path: /usr/bin/python
2024-01-22 14:40:49.164 [info] Starting Pylance language server.
2024-01-22 14:41:04.211 [info] Discover tests for workspace name: Python - uri: /home/reloadedd/Documents/programming/Python/whatever.py

Logs for the Python Language Server:

2024-01-22 14:40:49.163 [info] (Client) You may install and use any number of copies of the software only with Microsoft Visual Studio, Visual Studio for Mac, Visual Studio Code, Azure DevOps, Team Foundation Server, and successor Microsoft products and services (collectively, the “Visual Studio Products and Services”) to develop and test your applications. The software is licensed, not sold. This agreement only gives you some rights to use the software. Microsoft reserves all other rights. You may not: work around any technical limitations in the software that only allow you to use it in certain ways; reverse engineer, decompile or disassemble the software, or otherwise attempt to derive the source code for the software, except and to the extent required by third party licensing terms governing use of certain open source components that may be included in the software; remove, minimize, block, or modify any notices of Microsoft or its suppliers in the software; use the software in any way that is against the law or to create or propagate malware; or share, publish, distribute, or lease the software (except for any distributable code, subject to the terms above), provide the software as a stand-alone offering for others to use, or transfer the software or this agreement to any third party.

Thanks for the issue. Where are you running pylance? Pylance is not licensed to run in anything but VS code or Visual Studio at the moment. That error message is what we display when we don't detect VS code.
If you're not using VS code, you can use Pyright instead (which Pylance is a fork of). It's an open source version of Pylance but with fewer features.

If you're wondering how to install VS code on ArchLinux, these docs give a pretty good outline:
https://code.visualstudio.com/docs/setup/linux

Hi. Thank you for the response. Indeed, I am not running the official VS Code. However, I was accustomed with it working up until recently that I've thought it was a bug.
I am running VSCodium - VS Code without MS branding/telemetry/licensing, therefore, indeed, there is not an issue with Pylance (although allowing it to run on VSCodium will be appreciated).
Thank you again. The issue can be closed.

Hi. Thank you for the response. Indeed, I am not running the official VS Code. However, I was accustomed with it working up until recently that I've thought it was a bug.
I was running a version of VS Code without telemetry, therefore, indeed, there is not an issue with Pylance (although allowing it to run on other variants of VS Code will be appreciated).
Thank you again. The issue can be closed.


Device will crash when receiving and reply the mDNS packets during pairing with chip-tool or android app

Problem
Device will crash when receiving and reply the mDNS packets during pairing with chip-tool or android app
i guess the issue results from the lwip pbuf management.
My platform is ARM + FreeRTOS + Lwip.  and i choose the minimal mDNS policy,
After getting the ip, device starts the DNS server and start to listen 5353 port,  during receiving and sending reply mDNS packets, the device will crash with data abort. i see the lr register point to mem_malloc.
Firstly in general idea , in the upstream, device Mac layer allocate the pbuf and Matter will free it by PacketBufferHandle after handling it within lambda event. in the downstream, Matter will allocate the pbuf by PacketBufferHandle  and device Mac layer will free it after sending it out.
After the analysis, i found the memory heap in lwip is messed(i use the default policy memory management in lwip), when i enable the sanity check in lwip, i can find "mem_free: illegal memory: non-linked: double free".
And i have checked the synchronization mechanism in lwip, i found it works as expected, it protects the mem_malloc and mem_free with mutex.
BTW, when i comment out the Lambda event handler,  the device will NOT crash even the commission is incomplete.
Proposed Solution
Still can't find root cause. i assume the lwip can handle the concurrent memory access.

@bzbarsky-apple @tcarmelveilleux  i found the root cause just now.  In device Mac layer, in upstream we should  call pbuf_alloc(PBUF_RAW, len, PBUF_POOL)  rather than buf_alloc(PBUF_RAW, len, PBUF_RAM);  What enlightens me is i notice that PacketBufferHandle allocate pbuf with PBUF_POOL.
It's a little tricky.

I'll make the pbuf_type a build configuration option, defaulting to PBUF_RAM.

@kpschoedel I think PBUF_POOL is the default value, keep the current design. And leave some comment there, hope it can catch the attention.

@bzbarsky-apple @tcarmelveilleux i found the root cause just now. In device Mac layer, in upstream we should call pbuf_alloc(PBUF_RAW, len, PBUF_POOL) rather than buf_alloc(PBUF_RAW, len, PBUF_RAM); What enlightens me is i notice that PacketBufferHandle allocate pbuf with PBUF_POOL. It's a little tricky.

I disagree that this is a root cause. The location of where we allocated SHOULD NOT impact this bug. A double-free is a double-free. It's possible the local change of PBUF_POOL to PBUF_RAM slightly affected the code paths or timing, but it is unlikely the root cause

Yeah, these days i want to make the root cause clear,  and i change the pbuf type to PBUF_RAM in the phy layer when allocating ram to convey packets to lwip layer,  the crash still happen always.(Before i just use PBUF_POOL for walk-around)
And i think it should be free for developers to use PBUF_RAM or PBUF_POOL to convey packets to upper layer. i still can't find the root cause.
Do you have any idea?

And i think it should be free for developers to use PBUF_RAM or PBUF_POOL to convey packets to upper layer.

Sure, but given that the upper layer is then supposed to free the packet and the packet does not itself know how it was allocated, that means the upper layer needs to be configured in a way that matches the lower layer in terms of what sort of allocation strategy is used, right?

And i think it should be free for developers to use PBUF_RAM or PBUF_POOL to convey packets to upper layer.

Sure, but given that the upper layer is then supposed to free the packet and the packet does not itself know how it was allocated, that means the upper layer needs to be configured in a way that matches the lower layer in terms of what sort of allocation strategy is used, right?

I think Matter SDK just use pbuf_free(), which internally can do the right thing according to the some information in the pbuf.

packet does not itself know how it was allocated

@tcarmelveilleux  Sorry , can you explain a little more? I assume pbuf internal has enough information to record how it organises

Ah, we just call pbuf_free and internally it does in fact track the type.  So yes, what @tcarmelveilleux said about this just being a double-free seems correct then, since "mixing" pbuf types should work fine.

Ah, we just call pbuf_free and internally it does in fact track the type. So yes, what @tcarmelveilleux said about this just being a double-free seems correct then, since "mixing" pbuf types should work fine.

@bzbarsky-apple Thanks, i agree that double-free cause this issue. But i have strictly implemented the synchronous mechnism(sys_arch.c based on FreeRTOS), i assume the lwip can guarantee the race of resource no matter we call pbuf_calloc and pbuf_free in whatever thread context. So i can't understand what caused the double-free. And the issue only happen when i use PBUF_RAM in mac layer. In other project, i always use PBUF_RAM , there is no problem.
(I observe that once Matter SDK broadcast the mDNS packets, the issue occur immediately. )


More Documentation

Documentation is pretty weak at the moment. Will attempt to make steady progress here.

+1. Also would be good to make a demo

Added some documentation in the form of README.md files in the main directories of the project. Will create a new issue to begin adding demo links (using Codepen or similar) to this documentation in the future.


dotnet : The term 'dotnet' is not recognized as the name of a cmdlet, function, script file, or operable program. Check  the spelling of the name, or if a path was included, verify that the path is correct and try again.

Problem encountered on https://dotnet.microsoft.com/learn/dotnet/hello-world-tutorial/install
Operating System: windows
Provide details about the problem you're experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.

Did you close and reopen the cmd window before running "dotnet"?

Closing this out; please reopen if you can provide additional detail.

I had this issue when setting up the environment for the first time in a new virtual machine.
I fixed by installing the latest PowerShell x64 version, then I ran this command and worked:

dotnet --version


Allow JSON for code bindings

On the root of a view I have three CSS classes which need separate toggling like so:-
  bindings:
    '@sk-tile':
      "classes: {
        'sk-is-saved':saved,
        'sk-is-hover':isMouseOver,
        'sk-is-loading':isLoading
      }"

I really don't like using a string of JSON-like code.
This is how it looked like in a Jade template:-
.sk-tile(role='sk-tile', data-bind="classes:{\
'sk-is-saved':saved,\
'sk-is-hover':isMouseOver,\
'sk-is-loading':isLoading }")

Both are ugly. I moved to code bindings to try make things nicer, but it didn't help so much.
Questions

Should I be creating a custom binding handler to avoid squeezing too much into the data-bind tag, or is there some other way to do it?

Can you allow bindings declarations to be plain JSON so the first example can be written as:

  bindings:
    '@sk-tile':
      classes:
        'sk-is-saved': 'saved'
        'sk-is-hover': 'isMouseOver'
        'sk-is-loading': 'isLoading'

I can't even use JSON.stringify with the first example because bindings (saved, isMouseOver, etc.) cannot be wrapped in quotes.

Is it possible to use code bindings and HTML bindings at the same time. HTML bindings make templates easier to understand, but when they get long it becomes ugly and hard to format.

Thanks!

@gmac Excuse me for my insistence. Has my PR any chance? Does it suite epoxy?


Dependecies on some libraries can be removed

cxf-bundle-compatible
can be replaced with:
cxf-rt-frontend-jaxws
cxf-rt-transports-http

Implemented in commit https://github.com/todvora/eet-client/commit/ba8600efaba3d826af93ec2dccc6afcf7bb77ace
Works fine and makes dependencies zip for non-maven users significantly smaller. Thank you!


Mahalanobis fix

Mahalanobis distance couldn't be used as a metric since it required the VI kwarg to be passed to the sklearn pairwise_distance function (that is the inverse of the covariance matrix between the fitted elements).
I also added tests for the mahalanobis case. To make the test work in newer Python version I had to bump pytest to a newer version.
I also fromatted the file with black to standardize line width.

Glad to see I'm not the only one that have found this method useful - thanks for your contributions! :)
The changes look good to me. I'll merge it to master for now, but before tagging and releasing, I will make a proper CICD and release pipelines (hopefully in the near future).


bump tmpnb

there was a tiny change required to update tmpnb.  It's still not 100% right, but it's good enough to get the point across tomorrow.

Might as well bump to tmp39 until the thebe PR https://github.com/oreillymedia/thebe/pull/50 is merged or a similar solution comes out.

I tried tmp39, but the interact bar doesn't work.  So, I kept going backward until it did.  We should probably talk with Zach about it tomorrow, but I doubt he can fix it tonight.

What's up with the interact bar? /cc @parente
I wonder if it's differences in ipywidgets...

ipywidgets                4.0.3                    py34_0
is the version of ipywidgets installed in the latest build. Just ran https://github.com/ipython/ipywidgets/blob/master/examples/Using Interact.ipynb on tmp39 and it seems fine FWIW.

Yeah, I think this is related to thebe bundling a specific version of e.g. the widgets. We'd need versioning of thebe tied to versions of the notebook (until we finish the broken apart JS libraries).


Installation of Voice-Kit on fresh stretch Raspbian image not working

Hi All
I've got the same issue like the closed issue https://github.com/google/aiyprojects-raspbian/issues/309.
It seems the Hacking.md instructions are completely outdated, as there is no more install-deps.sh nor the usage of env/bin/python.
It would be great if the project could be upgraded to run on a fresh Raspbian image, because I'd like to include it in another project.
Thanks

i followed the instructions and i keep getting a "no module named aiy" error.

@alonchis The missing aiy module is another issue and can be solved by copying the src/aiy directory into the same directory as whatever directory the script you're trying to use.  So, for example, to run a checkpoint script:
cp -R src/aiy checkpoints
source envbin/activate
(env) checkpoints/check_audio.py

@anuraj1 We just released new SD card image and pushed a lot of commits to GitHub repo, please take a look at new HACKING.md file.

Still no joy.  I just installed the new SD image, and still receive no response, with 100% google.internal.speech.S3OnePlatform.Assistant 499 errors.

Here the service is working, but crashes immedately with this error:
voice-recognizer.service: Failed at step EXEC spawning /home/pi/AIY-projects-python/env/bin/python3: No such file or directory

@dikkedimi Looks like voice-recognizer.service is from the old SD card image, we don't have virtualenv (/home/pi/AIY-projects-python/env/bin/python3) anymore.

@dmitriykovalev I've already removed all installed previous versions and followed the instructions by google. Still doesn't work. I've edited the .service file as well, trying different versions found on the forums while searching the received log messages. it's not working with the current instructions. If you can direct me to some correct instructions that work on any raspbian install that'd be more helpful. I'd already read that virtual env wasn't being used anymore but nowhere is there a modified instruction on how to set up the service by yourself. Probably all an effort by Google to make purchasing a Home device easier.

I'm going to toss my hat into the ring and mention that on a current Raspbian light, the process chokes out because of a file naming issue:
Reading package lists... Done
Building dependency tree
Reading state information... Done
The following additional packages will be installed:
  aiy-io-mcu-firmware aiy-vision-firmware
The following NEW packages will be installed:
  aiy-dkms aiy-io-mcu-firmware aiy-vision-firmware aiy-voicebonnet-routes
0 upgraded, 4 newly installed, 0 to remove and 0 not upgraded.
Need to get 2,441 kB of archives.
After this operation, 6,901 kB of additional disk space will be used.
Do you want to continue? [Y/n] y
Get:1 https://dl.google.com/aiyprojects/deb stable/main armhf aiy-io-mcu-firmware all 1.0-0 [12.5 kB]
Get:2 https://dl.google.com/aiyprojects/deb stable/main armhf aiy-vision-firmware all 1.1-0 [2,404 kB]
Get:3 https://dl.google.com/aiyprojects/deb stable/main armhf aiy-dkms all 1.1 [22.9 kB]
Get:4 https://dl.google.com/aiyprojects/deb stable/main armhf aiy-voicebonnet-routes all 1.0-0 [2,022 B]
Fetched 2,441 kB in 2s (943 kB/s)
Selecting previously unselected package aiy-io-mcu-firmware.
(Reading database ... 67904 files and directories currently installed.)
Preparing to unpack .../aiy-io-mcu-firmware_1.0-0_all.deb ...
Unpacking aiy-io-mcu-firmware (1.0-0) ...
Selecting previously unselected package aiy-vision-firmware.
Preparing to unpack .../aiy-vision-firmware_1.1-0_all.deb ...
Unpacking aiy-vision-firmware (1.1-0) ...
Selecting previously unselected package aiy-dkms.
Preparing to unpack .../archives/aiy-dkms_1.1_all.deb ...
Unpacking aiy-dkms (1.1) ...
Selecting previously unselected package aiy-voicebonnet-routes.
Preparing to unpack .../aiy-voicebonnet-routes_1.0-0_all.deb ...
Unpacking aiy-voicebonnet-routes (1.0-0) ...
Setting up aiy-voicebonnet-routes (1.0-0) ...
Setting up aiy-io-mcu-firmware (1.0-0) ...
ln: failed to create symbolic link '/usr/lib/systemd/system/aiy_mcu_updater.service': No such file or directory
dpkg: error processing package aiy-io-mcu-firmware (--configure):
 subprocess installed post-installation script returned error exit status 1
dpkg: dependency problems prevent configuration of aiy-dkms:
 aiy-dkms depends on aiy-io-mcu-firmware; however:
  Package aiy-io-mcu-firmware is not configured yet.

dpkg: error processing package aiy-dkms (--configure):
 dependency problems - leaving unconfigured
Setting up aiy-vision-firmware (1.1-0) ...
Errors were encountered while processing:
 aiy-io-mcu-firmware
 aiy-dkms
E: Sub-process /usr/bin/dpkg returned an error code (1)```

This also has the effect of breaking a lot of things, and renders my Pi unbootable.

Just mkdir /usr/lib/systemd/system and then the installation works

We released updated debian packages and new SD card image, please try to follow updated HACKING.md.


[Bug] Opening Crawls - Doesn't recognize the folder

I've got the journal folder with the default name but it doesn't recognize it.

I can't reproduce this, even by manually creating the folder.  Can you run the following in the console and provide the output here?
game.settings.get('ffg-star-wars-enhancements', 'opening-crawl-folder')
game.folders.getName(game.settings.get('ffg-star-wars-enhancements', 'opening-crawl-folder'))

You forgot about console.log. Yes. 0.7.9
Opening Crawls
VM1950:2 Folder {data: {…}, data: {…}, options: {…}, apps: {…}, compendium: null, …}apps: {}children: Array(0)length: 0__proto_: Array(0)compendium: nullcontent: []data: color: ""flags: {}name: "Opening Crawls"parent: nullsort: nullsorting: "a"type: "RollTable"_id: "rl9ljsR6RqKCZDSc"proto: Objectdepth: 1options: proto: Object_data: color: ""flags: {}name: "Opening Crawls"parent: nullsort: nullsorting: "a"type: "RollTable"_id: "rl9ljsR6RqKCZDSc"proto: Objectcollection: (...)displayed: (...)entities: (...)entity: (...)expanded: (...)folder: (...)hasPlayerOwner: (...)id: (...)limited: (...)link: (...)name: (...)owner: (...)parent: (...)permission: (...)sheet: (...)type: (...)uuid: (...)visible: (...)_id: (...)_sheetClass: (...)proto: Entity

Hmm. So it correctly sees the folder and that it exists.  Can you run
game.folders.getName(game.settings.get('ffg-star-wars-enhancements', 'opening-crawl-folder')).content.map(journal => { return { id: journal.data._id, name: journal.data.name} });

and post the output? Please expand it as it'll be collapsed and I need to see the contents. It should look something like this:

As a side note, which browser are you using? Neither Edge nor Chrome require console.log when interacting with the console; they auto-output the results of commands run in the console to the log.

Ok, misread that you wanted to do it the browser console. Thought about doing it by script.

I was using Edge.

Ok, so the problem is that the folder is returning no contents. Do you have other modules loaded? If so, can you unload them all and try again?

Didn't help. But here's what I did that fixed the situation.
I've deleted the folder. The module still detected it, so I've changed the name in the settings and had a new prompt that the folder didn't exist. Allowed it to create one, then created a new crawl and it worked.
I don't know how exactly this bug happened. I've created the folder myself at the start and from what I remember whenever I created a new crawl, it was outside the folder.


Broken - not able to load new files based on pasted-in links:

I've been using this plugin for years, but now I'm getting errors when I try to paste new links.  Old links already in my library seem to work fine, however.

@cosmoddd same problem...
can anyone still work with this plugin?

hi. @AdrianEddy if you have some free time, please release an update for this plugin. I couldn't enjoy listening music after this plugin broke :D
thanks for your plugin. 🧡

this broke again in the new version of AIMP

for anyone is looking here hoping for an update :D
r1me (+download link) is working on it and so far these features work:

sc likes
sc stream
sc my tracks and playlists
but still can't add a specific playlist/track from URL.


Extract image alternate text from keydef linktext

According to this email exchange on the DITA TC list, according to DITA 1.3 spec:

For empty <image> elements, effective content is used as alternate text, equivalent to creating an <alt> sub-element to hold that content.

If in my DITA Map I have a key definition:
<keydef keys="myImage" href="images/Chrysanthemums.jpg" format="jpg">
  <topicmeta>
    <linktext>Alternate text</linktext>
  </topicmeta>
</keydef>

and in a DITA topic I have a keyref to it:
<image keyref="myImage" id="image_tq4_4lc_p1b"/>

the image <alt> element should be computed automatically from the <linktext> set on the keydef <topicmeta>.

I wondered about this when working on #2719 but knew I wouldn't have time to add it in. I think the final refactoring @jelovirt did before merging that item should actually make fixing this easier, but I haven't tried yet. Would be nice to get it into 2.5.2 if we can.

Looking into this one, the current code inserts the alternate text directly inside <image> when the image does not already have alternate text. Of course that's not valid, but it's what I'd expect from the current code. It doesn't show up anywhere because X/HTML and PDF all explicitly look for <alt> and process that.
Gets a bit trickier here because technically, the deprecated @alt on images should probably also override anything specified in the key definition -- currently every test for "link text" is based on element content, not attribute content.

This one was bugging me so I kept thinking about it. My eventual logic may be partly related to the fact that all the keyref link text processing is in the endElement method so no easy access to test if @alt was specified. But ignoring that..

The @alt attribute was deprecated in favor of <alt> way back in DITA 1.1 (2007).
Keys were not even defined until DITA 1.2.
The key resolution here, that explicitly leads to <alt>, wasn't fully clarified until DITA 1.3 (2015).
Now, 10 years after it was deprecated, I don't think we should go out of our way to support @alt as a way to override the key reference feature that was added more recently.

With that in mind -- @jelovirt or @raducoravu any thoughts on this commit as a fix for the current bug?
https://github.com/robander/dita-ot/commit/80468032e8f0e961ce4bc835f2b6e94de9c72772
It currently breaks the integration tests because we've got a sample in there that changes, but I could fix that in a pull request.

I'm ok with that fix.

👍 There is a little bit of duplicate code between the two "writeAlt" methods, maybe a "writeAlt" which could received two parameters, the content as Element and the content as String, and depending on which one is not null, output that one...
One question, how about if there is an element derived from DITA image? Maybe it no longer has an alt element or maybe its "alt" element is not called "alt" anymore... Should we treat in the fix only DITA "image" elements (having the tag name "image") instead of matching the class attribute value?

For the duplicate code -- I've just used the same pattern we already have for writeLinktext, so I don't really want to create a new pattern here. Can change both later if we decide it's better to do as one method.
For specializations - we should not worry about that just for this case. The same theoretical issue comes up in almost any DITA processing that changes markup, so we shouldn't do something special here. For example - almost any DITA keyword/term/link/xref element could be specialized with EMPTY content model, but we still insert link text for those as well. Similarly, you could specialize a topic that doesn't allow <related-links> but we'll still generate map based linking. None of these break anything because we are not enforcing schema validity at this point, and if special processing exists for these conditions they'll just ignore the generated content.

@robander Ok.

Fixed in #2814.


moma 1.2.6

fix #49

RE-CHECK!

Automated adapter checker
ioBroker.moma

:thumbsup: No errors found
Add comment "RE-CHECK!" to start check anew


HarfBuzzSharp 2.6.1

Description of Change
This updates HarfBuzzSharp to version 2.6.1 and has added bindings for introduces OpenType Metrics API
This also adds some overloads to Buffer.cs that take an integer as codepoint to make the life easier when working with codepoints.
API Changes
List all API changes here (or just put None), example:
Added:

Definitions.cs
-- public enum OpenTypeMetricsTag : uint
-- public readonly structs OpenTypeMetrics
Font.cs
-- public OpenTypeMetrics OpenTypeMetrics { get; }
--- public bool TryGetPosition (OpenTypeMetricsTag metricsTag, out int position)
--- public float GetVariation (OpenTypeMetricsTag metricsTag)
--- public int GetXVariation (OpenTypeMetricsTag metricsTag)
--- public int GetYVariation (OpenTypeMetricsTag metricsTag)

PR Checklist

[x] Has tests (if omitted, state reason in description)
[x] Rebased on top of master at time of PR
[ ] Changes adhere to coding standard
[ ] Updated documentation

Hehe, more updates to harfbuzz. Seems every week there is a new version 😆

@mattleibow Could you update the XCode projects for me? I think when mac builds this is ready for review. Should OpenTypeMetrics be a nested struct within Font.cs

Having a look now. Thanks for the PR.

@Gillibald Before I merge this, I see you changed the enum from the Value = ('a' | 'b') format into the int value. Was there a specific reason? Enums are constants, so they would be compiled to the integer value anyway. But, at we could see the string values.

@mattleibow I think I changed it because Intellisence didn't work well with it if I remember correctly. If that isn't the case we should revert this and use bit shifting to produce the values. Too bad C# has no macros. To produce these values I wrote some code generation routine that reads the HarfBuzz headers and creates enums out of it.

I have changed the enum definition to use the bit shifting representation

Merged!
I want to get a release out before Christmas. Gonna try!


Update to test to fix arbitrary symbol limit

the betaUI handles this very well already when reading in contract metadata on a manually added non-auto-detected token. the prior changes were added just 4 weeks ago and seem arbitrary and effect our efforts. the now default betaUI handles this well already.

@estebanmino


jetporch init at unstable-2023-10-16

Description of changes
Initial commit of JetPorch version 0.0.1.
Things done
~Plan to build on Linux later, just wanted to get the PR put together.~

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

@FliegendeWurst I think I've completed all your suggestions, please let me know if there is anything else I can/should do.

@FliegendeWurst I think I've completed all your suggestions, please let me know if there is anything else I can/should do.

Yes, the commits should be the other way around. Otherwise the first commit doesn't evaluate.
After that's done we just need to wait for someone to merge the PR (I don't have the required permissions)

@FliegendeWurst order should be fixed.

This project has been discontinued. The original page is dead: https://www.jetporch.com/
Since I can't link to the original post on JetPorch, here's a community-contributed post: https://lucaberton.medium.com/farewell-to-jetporch-automation-b820021e512f


Add timeouts to aws_lambda_function for lambda's deployed in VPCs

Please add timeouts to resource "aws_lambda_function" "lambda_function" since deploying lambdas in VPCs is often extremely slow. In our case our code accesses AWS Neptune endpoints so the server-function needs deployment in a VPC which often times out.
timeouts {
    create = "30m"
    update = "30m"
}

(Or make it configurable)

Hi @jgeluk,
Apologies you ran into the issue, this has now been added to the module and released as part of v1.4.0.
Please update your terraform configuration to include the following
module "open-next" {
  source  = "RJPearson94/open-next/aws"
  version = "1.4.0"
}

Thanks for using the module
Kind Regards
Rob


Add __init__.py to become a regular package

A __init__.py is required to be considered a "regular package" (https://docs.python.org/3.11/glossary.html#term-regular-package). Without this the setuptools.find_packages() in setup.py returns an empty list.
Before:
$ python -m pip install git+https://github.com/Jaded-Encoding-Thaumaturgy/getfscaler@920b4451d7968eb2ea42729600e22d7a14a3858c
$ python -m getfscaler
.venv\Scripts\python.exe: No module named getfscaler

After:
$ python -m pip install git+https://github.com/sgt0/getfscaler@package
$ python -m getfscaler
usage: getfscaler [-h] [--native-height NATIVE_HEIGHT] [--native_width NATIVE_WIDTH] [--crop CROP]
                  [--frame FRAME] [--field-based FIELDS] [--swap] [--extensive] [--out] [--debug]
                  input_file
getfscaler: error: the following arguments are required: input_file

Also set program name for argparse so it doesn't appear as __main__.py.
Updated a couple of the GitHub actions too while I'm here.

Thanks!


about code

nice work! When will you release the source code.

This article has remained unnoticed after publication, and I'm almost prepared to quit making the code public. I have a trip planned for this upcoming week, so I'll be sharing the code in the first week of August, and hopefully the work will provide inspiration for you.

感谢您的回复，您的工作非常有意义，期待您的开源~
方兆岩
@.***

------------------ 原始邮件 ------------------
发件人:                                                                                "csnj0/AMA-Det"                                                                                @.>;
发送时间: 2023年7月20日(星期四) 下午5:31
@.>;
<EMAIL_ADDRESS>主题: Re: [csnj0/AMA-Det] about code (Issue #1)
This article has remained unnoticed after publication, and I'm almost prepared to quit making the code public. I have a trip planned for this upcoming week, so I'll be sharing the code in the first week of August, and hopefully the work will provide inspiration for you.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

The initial version has been uploaded, and all the experimental results in the paper can be reproduced. Please cite this paper if it inspires your research work. Thanks.

of course! thanks for your code~~~


how to limit the type of upload file

The version I used is:
angular ~1.2.0
ng-flow ~2
HTML :

      .......
           Upload Image

      .......
           Upload CSV

I want to upload file only in image (*.jpg, *.png, *.bmp) and csv format.
Accoridng to some suggestion, I used attribute flow-attrs="{accept:'.....'}" in HTML, but I found they seem not work. I have to check the file suffix in js method.
Could you please tell me if  we could have a better way?
Thanks

Hi,
using "flow-file-added " event handler you can limit the type of uploaded file.
In my project i am using like this...

flow-file-added="!!{png:1,gif:1,jpg:1,jpeg:1,csv:1}"

Hello @shailesh87
Thanks very much, it works.


Getting this error after deploying the node.js function

Error:
Function (MyHttpTrigger) Error: Failed to start language worker process for: node. node exited with code 1 Couldn't require bundle, falling back to Worker.js. Error: Your Function App is currently set to use Node.js version v6.5.0, but the runtime requires an Active LTS or Current version (ex: 8.11.1 or 10.6.0).,Error: Cannot find module './Worker.js'.
Session Id: b38a3536c5a641318389cec0f7084759
Timestamp: 2018-10-20T19:21:15.523Z

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

ID: 40b416d5-6ea5-39a5-9a2c-ab188d00a686
Version Independent ID: 855c85f7-53b3-59a2-3f42-fbccbba07d32
Content: Create your first function using the Azure CLI
Content Source: articles/azure-functions/functions-create-first-azure-function-azure-cli.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

@kristianjaeger Thank you for the detailed feedback. We are actively investigating and will get back to you soon.

Hi @kristianjaeger Sorry for inconvenience. Could you set your NodeJS runtime version to 8.11.1 and try again? This is done in Application Settings under Platform Features tab as documented here. Hope this helps, please let me know if you continue to encounter issues.

Thanks @mike-urnun-msft !  I will give that a try.  Cheers.

Hi @kristianjaeger Thanks again for your PR and contribution towards MS Docs!
We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


[1.317] Disabled axi (XYZ) should be more visible

Make the disabled buttons for the XYZ axis in the bottom of the screen more obviously disabled

Partial implementation is available in 1.317

This version is now available on the asset store, so closing ticket


Runtime Error in hellokan.ipynb

Closes #170
Thanks to @wkqian06

This should close also #36, given that the initial model and the other model are both initialized with device='cuda'. If they have different dtypes, though, they should be all casted to float(), for now.


Update content on start page

Context

Suggested content changes from @EmmaFrith
No longer open links on this page in new tabs

Changes proposed in this pull request
Before:

After:

Link to Trello card
https://trello.com/c/gp4IKjeb/1718-start-page-edit

@paulrobertlloyd - just a couple of small things which aren't the same as outlined here: https://docs.google.com/document/d/1PnKkbWbh4QgowOnS0whK8Wny5xxhGIeXJpuv3GNMryA/edit?usp=sharing

can you change the link text to 'Learn more about training to teach if you’re disabled' - rather than 'have a disability'
can you remove the full stops from the 2 sentences which are entirely link text

See here for examples on how to use punctuation with links on gov.uk: https://www.gov.uk/government/how-government-works. When the whole sentence is a link, and says something like 'Read more...' 'See...', we don't need the full stops.
Thanks!


Async tests fix for tests/integration_tests and tests/unit_tests directories.

Async tests fix for tests/integration_tests and tests/unit_tests directories.

https://github.com/opentensor/bittensor/pull/2040


[Bug]: Advanced Manual Setup for LN channel can result in crash

Describe the bug
Bitkit crashes if the Manual Setup flow form is not filled in and the Continue button is pressed.
(This may have persistently broken ldk, need to test more)
Reproduce

Go to Advanced settings -> Lightning Connections -> Add Connection
At Manual Setup page, tap 'Continue'
App crashes
On restarting Bitkit the LDK node never starts correctly (wallet restore required)

Screenshots / Recording
https://github.com/user-attachments/assets/16a55c09-859a-456a-a216-5bfa49a2a463
https://github.com/user-attachments/assets/df133def-72c0-4f85-bdc1-5d945cce034b
Operating system
iOS / Android
Bitkit version
1.0.5 (136)
Log output
crash-report.txt

Can confirm, the app is crashing when trying to add a peer with an invalid pubkey. I can add validation to check for correct length but that's not enough to prevent this.
https://github.com/user-attachments/assets/16a55c09-859a-456a-a216-5bfa49a2a463

Crashing on Android  as well:
https://github.com/user-attachments/assets/df133def-72c0-4f85-bdc1-5d945cce034b

it's weird it didn't crash for me

Yeah length check doesn't seem like enough here. Needs proper validation.

This should do the trick: https://github.com/synonymdev/react-native-ldk/pull/274
I'm testing with<PHONE_NUMBER>000000000000000000000000000000000000000000000000000000

@pwltr It's interesting you get a crash on android because I see a toast

Still strangely stopped being able to replicate the crash since Friday but I believe this does the trick: https://github.com/synonymdev/bitkit/pull/2275
Using the actual curve to validate the public key.

Unable to reproduce crash on android using 0d67faa2509b1ec083d475abc80e90802f8d7d28

The form validation now prevents proceeding without inputting a valid node id, and something for host and port.
If an node id is used that cannot be connected to then an appropriate toast error is displayed

Todo: test on iOS


DissociationFlux is not documented

I noticed that the DissociationFlux class is not documented in the API reference.

Actually I'll close this as apparently this has been fixed already in main.