USB Gadget

There are plenty of small single board computers that have USB OTG port that can be configured to act as HID device.
One can connect keyboard to host port of such SBC, PC to OTG or device port and use SBC as USB to USB converter that can remap keys, add layers, macros etc.
As a proof of the concept I modified keyd to write HID reports via USB Gadget instead of sending evdev events to uinput:
https://github.com/gkbd/keyd/tree/gadget
I'm testing it now and so far it looks promising, tried plugging it in Linux, Windows, Chrome OS and Android devices.
It's based on version 1.3.1 though, will take a look at 2.0 later.
If there is interest in making virtual input device configurable and adding USB Gadget as one of the output options I can eventually make sure that changes are upstreamable.
Otherwise I can keep it as proof of the concept for a while and decide later how to proceed.

This indeed looks quite promising!
I have been contemplating adding some degree of multi platform support, which would start by factoring out the evdev/uinput specific code and allow for arbitrary input and output sinks. Once done, it should be relatively straightforward to add USB Gadget as an output.
I'm curious to know what you are running this on.

Good to hear that.
At the moment I'm running it on Cubietruck, one of the early ARM devboards that I had lying in basement.
Eventually I want to try it on smaller boards like MangoPi MQ, but it might take some time before RISC-V devices like this gain mainline Linux support.
Until then I can keep prototyping it on ARM or Aarch64 devices, can pick low hanging fruit Pis first.

Interesting. How do these compare to the dedicated controllers? Are they battery powered, or is power supplied by the host?
I have winnowed out some space for alternative virtual keyboard implementations in the latest commit.
See 6819c1cd0 for a sample implementation which can be built with make vkbs-stdout and prints to stdout.  (note that if you run the resultant binary you will have to terminate with the panic key sequence).

Very good solution. Thank you. Will try it.
Some SBCs can be powered by host PC, more power hungry ones are powered from barrel plug or GPIO pins.
Dedicated controllers are good option too, some run TMK/QMK even.

Pushed USB gadget output sink and helper files to
https://github.com/gkbd/keyd/tree/gadget2
After your changes it was indeed much easier to add new output option.
And vkbd-stdout was also useful both as example and on it's own.
Thanks for that.
Let me know if further changes are required.
Can send PR if no other refactoring is needed at this stage.

Sounds good, go ahead and create a PR.

Merged.

Merged.


Signet::OAuth2::Client#decoded_id_token should take optional &keyfinder block for JWT.decode

Closes #95

@googlebot I signed it!


is there a way can create window at thread other than the main thread?

@LmanTW Not at the moment, and sadly I have no plans to fix this. In the beginning I was hoping to make this lib support worker threads but there are some obstacles to this. The biggest one is that the SDL event loop itself does not play well with threads.
It would be best if you only ever require() and use this lib from the main thread. If you need to interact with sdl from a worker therad you should post a message to the main thread and handle it there. I will add code in future versions to enforce this.


Improve script verify-gofmt.sh

It need flag "-w" to format bad files. So it's better to add this message into script.

/lgtm
thank you


[Device Support Request] Tuya 214C Ultrasonic water meter with valve

Problem description
This is a Smart Ultrasonic Water Flow Meter with Tuya Zigbee App Control.
The device can be paired with ZHA, but no sensors are available.
The device:
https://www.alibaba.com/product-detail/Smart-Ultrasonic-Water-Flow-Meter-With_1600722839075.html
Device model: "TS0601"
Manufacturer: "_TZE200_vuwtqx0t"
Solution description
Add support for this device
Device signature
Device signature
{
  "node_descriptor": {
    "logical_type": 2,
    "complex_descriptor_available": 0,
    "user_descriptor_available": 0,
    "reserved": 0,
    "aps_flags": 0,
    "frequency_band": 8,
    "mac_capability_flags": 128,
    "manufacturer_code": 4417,
    "maximum_buffer_size": 66,
    "maximum_incoming_transfer_size": 66,
    "server_mask": 10752,
    "maximum_outgoing_transfer_size": 66,
    "descriptor_capability_field": 0
  },
  "endpoints": {
    "1": {
      "profile_id": "0x0104",
      "device_type": "0x0051",
      "input_clusters": [
        "0x0000",
        "0x0004",
        "0x0005",
        "0xef00"
      ],
      "output_clusters": [
        "0x000a",
        "0x0019"
      ]
    }
  },
  "manufacturer": "_TZE200_vuwtqx0t",
  "model": "TS0601",
  "class": "zigpy.device.Device"
}

Additional information
No response

same for https://www.aliexpress.com/item/1005007353486146.html
213E device is similar:
[0xed9d] Got Node Descriptor: NodeDescriptor(logical_type=<LogicalType.EndDevice: 2>, complex_descriptor_available=0, user_descriptor_available=0, reserved=0, aps_flags=0, frequency_band=<FrequencyBand.Freq2400MHz: 8>, mac_capability_flags=<MACCapabilityFlags.AllocateAddress: 128>, manufacturer_code=4417, maximum_buffer_size=66, maximum_incoming_transfer_size=66, server_mask=10752, maximum_outgoing_transfer_size=66, descriptor_capability_field=<DescriptorCapability.NONE: 0>, *allocate_address=True, *is_alternate_pan_coordinator=False, *is_coordinator=False, *is_end_device=True, *is_full_function_device=False, *is_mains_powered=False, *is_receiver_on_when_idle=False, *is_router=False, *is_security_capable=False)

  "data": {     "ieee": "**REDACTED**",     "nwk": 55451,     "manufacturer": "_TZE200_zlwr0raf",     "model": "TS0601",     "name": "_TZE200_zlwr0raf TS0601",     "quirk_applied": false,     "quirk_class": "zigpy.device.Device",     "quirk_id": null,     "manufacturer_code": 4417,     "power_source": "Battery or Unknown",     "lqi": 112,     "rssi": -72,     "last_seen": "2024-08-20T21:03:01",     "available": true,     "device_type": "EndDevice",     "signature": {       "node_descriptor": "NodeDescriptor(logical_type=<LogicalType.EndDevice: 2>, complex_descriptor_available=0, user_descriptor_available=0, reserved=0, aps_flags=0, frequency_band=<FrequencyBand.Freq2400MHz: 8>, mac_capability_flags=<MACCapabilityFlags.AllocateAddress: 128>, manufacturer_code=4417, maximum_buffer_size=66, maximum_incoming_transfer_size=66, server_mask=10752, maximum_outgoing_transfer_size=66, descriptor_capability_field=<DescriptorCapability.NONE: 0>, *allocate_address=True, *is_alternate_pan_coordinator=False, *is_coordinator=False, *is_end_device=True, *is_full_function_device=False, *is_mains_powered=False, *is_receiver_on_when_idle=False, *is_router=False, *is_security_capable=False)",       "endpoints": {         "1": {           "profile_id": "0x0104",           "device_type": "0x0051",           "input_clusters": [             "0x0000",             "0x0004",             "0x0005",             "0xef00"           ],           "output_clusters": [             "0x000a",             "0x0019"           ]         }       },       "manufacturer": "_TZE200_zlwr0raf",       "model": "TS0601"     },     "active_coordinator": false,     "entities": [       {         "entity_id": "update.tze200_zlwr0raf_ts0601_firmware",         "name": "_TZE200_zlwr0raf TS0601"       }     ],     "neighbors": [],     "routes": [],     "endpoint_names": [       {         "name": "SMART_PLUG"       }     ],

part of solution might be here: https://github.com/Koenkk/zigbee2mqtt/issues/21255

Has anyone managed to get a ZHA quirk working for the 213E Tuya meter (TS0601 TZE200_zlwr0raf )?
I have been trying to adapt a quirk working by trial and error due to lack of experience in quirks, but I have not got it working, all I am after is the total water used and battery status, but could not get this working.
Any help will be greatly appreciated.
Thank you.
AJ Ruba

There is a implementation for this device for Tuya Local.
Maybe this could help for ZHA as well:
https://github.com/make-all/tuya-local/blob/main/custom_components/tuya_local/devices/zigbee_ultrasonic_watermeter.yaml

There is a implementation for this device for Tuya Local. Maybe this could help for ZHA as well:
https://github.com/make-all/tuya-local/blob/main/custom_components/tuya_local/devices/zigbee_ultrasonic_watermeter.yaml
Issue: make-all/tuya-local#2412

That's a nice DP listing, should be easy to build out a v2 quirk from that. See https://github.com/zigpy/zha-device-handlers/wiki/Tuya-‐-v2-Quirk-with-TuyaQuirkBuilder and https://github.com/zigpy/zha-device-handlers/blob/dev/tuya.md also the current quirks here would be a good starting point https://github.com/zigpy/zha-device-handlers/blob/dev/zhaquirks/tuya/ts0601_valve.py


Output exceptions when verbosity level is increased

I'm getting exceptions on several files (internal error), but I don't see a way to further debug those errors.
I assume that there's an exception being caught, it would be great if I could see it.

Yes, they are hidden for end-user.
Please, take a look into ErrorsManager

Do you have any additional documentation about using that?

No (just one that's inside file itself), this is class just to store that info you saw on console.
When I want to debug I just add some dumping in report method.
BTW, in 2.x version I would like to have some autogenerated bug reports with all info that is not needed for end-user, but essential to fix it.


Keras-autodoc and dataclass

🐛 Bug
Describe the bug
It seems keras-autodoc has a problem parsing a dataclass like BBox, the following error occur when running autogen.py:
  File "/Users/lgvaz/anaconda3/envs/mantis/lib/python3.7/site-packages/keras_autodoc/utils.py", line 73, in get_class_from_method
    meth.__qualname__.split('.<locals>', 1)[0].rsplit('.', 1)[0])
AttributeError: module 'mantisshrimp.core.bbox' has no attribute '__create_fn__'

Removing @dataclass from BBox fixes the issue.
I see two solutions here:
Solution 1
Fork keras autodoc and fix the issue (it should be relatively simple), we can make a PR to the original repo and keep use the fork until they merge it.
Solution 2
Stop using dataclass. Kind of a radical solution, dataclasses offer us some nice benefits... But if we fail to fix it, we can just use normal classes

@lgvaz, I would go with Solution 1, take from there. By filing a PR, we are also giving back to the keras_autodoc team who built a very useful package.

@ai-fast-track is this one still relevant or can we close?

@ai-fast-track wdyt?


chore: Remove TypeScript tooling from noir-projects.

As title.

Docs Preview
Hey there! 👋 You can check your preview at https://65e0cc77dd701506a39cf195--aztec-docs-dev.netlify.app

Benchmark results
No metrics with a significant change found.

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Values are compared against data from master at commit c7c24b2d and shown if the difference exceeds 1%.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
64 txs

l1_rollup_calldata_size_in_bytes
4,932
17,348
33,892

l1_rollup_calldata_gas
63,000
232,412
458,716

l1_rollup_execution_gas
190,900
493,670
897,926

l2_block_processing_time_in_ms
1,196 (-2%)
4,969 (+8%)
9,035 (-1%)

note_successful_decrypting_time_in_ms
200 (-1%)
563 (+1%)
1,012

note_trial_decrypting_time_in_ms
10.0 (+17%)
82.8 (+10%)
37.7 (-25%)

l2_block_building_time_in_ms
16,225
63,984 (-1%)
128,531

l2_block_rollup_simulation_time_in_ms
12,373 (+1%)
48,778 (-1%)
98,192

l2_block_public_tx_process_time_in_ms
3,818 (-2%)
15,132
30,204

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
15,224 (-1%)
29,007

note_history_successful_decrypting_time_in_ms
1,249 (-5%)
2,426 (-5%)

note_history_trial_decrypting_time_in_ms
110 (-5%)
146 (-22%)

node_database_size_in_bytes
18,600,016
35,344,464

pxe_database_size_in_bytes
29,923
59,478

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
248
44,736
28,001

private-kernel-ordering
178
52,625
14,627

base-rollup
1,312
175,343
933

root-rollup
71.2
4,192
825

private-kernel-inner
315
73,715
28,001

public-kernel-app-logic
193 (+1%)
32,254
25,379

merge-rollup
5.77 (-2%)
2,712
933

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
0 deployed contracts

tx_size_in_bytes
19,147

Transaction processing duration by data writes.

Metric
0 new note hashes
1 new note hashes

tx_pxe_processing_time_ms
2,612 (+1%)
1,340

Metric
0 public data writes
1 public data writes

tx_sequencer_processing_time_ms
0.0309 (-4%)
472


Add guidance on asking for names

This PR adds guidance to the Payment card details pattern saying not to assume that the name on a user's card will be the same as they enter elsewhere in the service.
This addition came from the Credit card details file on Dropbox Paper which has been archived as part of the migration work.

You can preview this change here:
Built with commit 5aca9d6ef3a7c656770959538bdaa414925cbb64
https://deploy-preview-589--govuk-design-system-preview.netlify.com


fixes the case where existing brooklyn entities have their sensors recreated as static sensors

Per comments above I'll merge this and incorporate the fix into #57.


Applying LoudnessEQ on input

Hello,
A while ago, I (unsuccessfully) tried modifying this plugin to apply on mic input.
When & if you have time, could you please create a version of this plugin that can be applied on input stream(s), via EasyEffects?
In theory, this should help normalize audio levels if the mic is too far/close.

Hello.
Using it in input should be quite easy. You need to copy the original in  ~/.config/easyeffects/input folder, then modify it with a text editor changing only output to input at the 2nd line.
You should view it in presets menu, under the input tab.
Anyway I did it here: LoudnessEqualizer.txt
Just rename it with json extension and copy in the folder mentioned above.

I wonder why/where I had trouble 😮 ... seems easy enough.
Thank you, will aim to try later today.

Hi. I am no coder and not even sure how to use github properly. Just want to thank digitalone1 a million times for solving the movie dialogue issue via the loudnesseqaulization preset.
I've been using VLC (compressor enabled) on Windows for years (decades actually), but MX Linux (with Pipewire) and Loudeness Equalization preset hands down beats the audio quality on Windows. I say this after almost 25+ years of using PC!
I am never going back to Windows! Thank you so much for creating the LoudnessEqualization preset!


Podman compatible?

Would you consider handling podman based systems with your script?

I created a pull request for you to review with the required code.

Since their code is basically the same, I thought it would be better to add a parameter to specify the container to use.


[#22] Finishing cleaning the structure so that everything has only one place

This issue will need #19, #20, #21 to be resolved.
The idea is that these three issues will change the backbone of the code, with substitution of previous elements. Once they are implemented, we should clean all the code to get a good overview of the next tasks

That should be good using #28 as reference.


Reindent only when dash was typed at the start of line

Reported by Daniel Hahler (@blueyed).

@zyx-i please remove my personal email.
You can use git at thequod de instead.

I could also create a new PR myself, just let me know.
I'll do so right away next time.

closing, should have been included


Get information from last selected leaf node

Hi All,
For my application, I would like to be able to get information from the most recently selected leaf node. Is there a function to achieve this?
I understand that I can use 'get_selection' or 'selection_callback' to get the list of all selected nodes. But it appears that I could not use either of these two functions to achieve my goal.
Any suggestions will be highly appreciated.
Thanks a lot. Best regards,
Wen

Dear @lubwen,
This is an interesting use case, and is not directly supported by the API at the moment.
However, it does seem possible to hack this together with the existing API and some JavaScript. Using selection_callback, you can try writing the selection to a variable, and when a new selection is made, comparing the new one with the old one to determine what was added.
I'm not entirely sure what you intend, so it's tough to determine new functionality for the API, or how it ought to work to support other, related use cases. For instance, entire clades may be selected, as well as all leaves/internal nodes.
Best,
Stephen

Dear @stephenshank,
I figured this out. This can be implemented from within the 'style_nodes' method using element.on('click', function() {...}).
Thanks.
Best,
Wen


Change algorithm for how public coords for obscured obs are calculated

Currently when an observation is obscured (via geoprivacy setting or via associated threatened taxa) we pick a uniformly distributed random cardinal direction from the true point and pick a point a uniformly distributed random distance from 0 to 10km in that direction to calculate a position to display the public location  https://github.com/inaturalist/inaturalist/blob/master/app/models/observation.rb#L1797
Unfortunately, when there are many many obscured observations on the same location (e.g. someone's house) its easy to estimate the private location as the center of the circle made up of public locations.
This proposal is to change to the following algorithm to avoid this problem:

Calculate the floor of the latitude and longitude rounded to some significant figure (e.g. y = 0.1 degrees) to represent the lower left hand corner of a y-by-y degree cell that contains the private location
Add a uniformly distributed random lat and lon offset for this lower left hand corner between 0 and y
Calculate the public coordinate as private_lat+lat_offset and private_lon+lon_offset.

The disadvantages of this approach are that the size area from which the private location is drawn will change from consistently 314km^2 to some value at the equator that approaches 0 at the poles since the distance of one degree of longitude varies with latitude.
For simplicity and since the existing 10km offset was pretty arbitrarily chosen, I suggest we pick a value of y=0.1 for the cell size which will vary from an obscured area of 11x11=121km^2 at the equator, to 9x11=99km^2 at the latitude of San Francisco to 5x11=55km^2 at the latitude of Anchorage. Thoughts?
(here's a nice lat/lon to km converter http://www.nhc.noaa.gov/gccalc.shtml)

Cell size sounds fine to me. Let's see what it looks like.

sounds great, i'd be able to fix all of my observations from home that i
used a really weird self-obscuring thing on.
On Mon, Jul 27, 2015 at 8:19 PM, Ken-ichi<EMAIL_ADDRESS>wrote:

Cell size sounds fine to me. Let's see what it looks like.
—
Reply to this email directly or view it on GitHub
https://github.com/inaturalist/inaturalist/issues/649#issuecomment-125384578
.

--
Charlie Hohn
Montpelier, Vermont

here's an example of exactly what Scott described:

that looks good to me - maybe I did my math wrong if thats a 0.1 degree grid, the areas look about the same

oh, sorry you're right - not exactly what you said. Instead of:

Add a uniformly distributed random lat and lon offset for this lower left hand corner between 0 and y

I did between -y and y, effectively quadrupling the areas you suggested. Here is what it looks like between 0 and y

Maybe the larger area (-y to y) will work and isn't too large?

Sounds great to me

I put the new algorithm on Staging and re-computed all obscured obs coordinates http://staging.inaturalist.org/observations/map?type=map&color=grey#10/40.545/-77.275 . I did end up making one change, which is instead of using a floor function I ended up rounding, which was needed when using -y to y instead of 0 to y. The obscured area remains the same 4x Scott's original calculation.
The Staging tileserver is a little slow, but it's worth checking out the Bay Area which is now a little blocky, but I think that's fine. The other thing we could do if that's a problem is combine this approach with the original and first round the coordinates then do the circular obscuring. The main original problem was the origin of obscuring was the original observation location. Scott's idea is a great one because it changes this origin, which I think is the key. Whatever shape we use to obscure the coordinates from there is pretty much irrelevant. For what it's worth the rectangles are easier to compute, but the different is probably negligible.

this is finished, deployed, and all observations with obscured coords have been updated. Just waiting on the needs_id release and then we'll be ready to get back to the obscured observations in map tiles

According to https://github.com/inaturalist/inaturalist/blob/23e6388fa0da1fd45da038f4978a8e8316a9ea0f/app/models/observation.rb#L2311, an observation is not mappable if it's public_positional_accuracy exceeds M_TO_OBSCURE_THREATENED_TAXA, and according to https://github.com/inaturalist/inaturalist/blob/23e6388fa0da1fd45da038f4978a8e8316a9ea0f/app/models/observation.rb#L2291, the public_positional_accuracy is defined as the positional_accuracy or M_TO_OBSCURE_THREATENED_TAXA, which means that if you create an observation with greater than 10,000 m positional_accuracy, it never gets shown on a map. Was that intentional, i.e. did you guys want to not map anything that had a positional_accuracy greater than that we automatically applied for coordinate obscuration?
I ask because we are still using M_TO_OBSCURE_THREATENED_TAXA to determine public_positional_accuracy and thereby its mappable state, even though this issue made it so we don't even use that for coordinate obscuration anymore. I have a fix that's a part of #865, but I wanted to clarify this point with you guys before committing it.


splitting kubernetes service ip with regex

What did you do?
I have pods with jmx-exporter. I would like to scrape only the exposed ports.
I do use the annotation "prometheus.io/port" but to be that only the exposed port is scraped in case some pods/services dont have the annotation.  To do so I am trying to replace the ports in the addresses that I get from the service endpoints.
I have multiple ports exposed in the pods so that with the below config I get endpoints in prometheus like
http://<IP_ADDRESS>:1111 http://<IP_ADDRESS>:2222 http://<IP_ADDRESS>:3333 http://<IP_ADDRESS>:4444
I would like to split up the pod IP and append the port number I would like to scrape.
I don't seem to  be able to make use of the regexes...
What I need is something like
      - source_labels: [__meta_kubernetes_service_annotation_prometheus_io_port]
        action: replace
        target_label: __address__
        regex: functioning regex
        replacement:  ${1}:2222

where ${1} is the service IP and 2222 is the port I want.
Can anybody help me with that?
Environment
OpenShift

Prometheus version:
1.6.2

Prometheus configuration file:

    - job_name: 'kubernetes-test'
      kubernetes_sd_configs:
      - role: endpoints

      relabel_configs:
      - source_labels: [__meta_kubernetes_service_annotation_prometheus_io_scrape]
        action: keep
        regex: true
      - source_labels: [__meta_kubernetes_service_annotation_prometheus_io_port]
        action: replace
        target_label: __address__
        regex: (.+)
        replacement:

It makes more sense to ask questions like this on the prometheus-users mailing list rather than in a GitHub issue. On the mailing list, more people are available to potentially respond to your question, and the whole community can benefit from the answers provided.

Thanks for the info. I'll do that.


Opacity and SVG issue

Hello everyone, I am hoping someone may have found a solution to the problem I have encountered. I know that SVG support in version 0.4.1 was being worked on and that in version 0.5.0 it has been addressed with the full rewrite. However here is the issue I have ran into. My web tool has the option to select a parcel to obtain information about the result as seen in the folllowing picture.

Now when you use version 0.4.1 you are able to get the screen capture and the opacity of the information remains, but the graphic on the map itself does not show as seen next:

So I then attempted to use the 0.5.0 prerelease version and I was able to get the graphic to show, but now all the transparencies for what was in the map are now lost as seen in the next image:

Has anyone been able to resolve this issue with either version and/or have any ideas on how one might be able to combine the opacity ability of version 0.4.1 with the graphic/SVG display that is found in version 0.5.0. I am at a loss right now and I am opting to go with version 0.4.1 so my users can at least print the page with 90% of the functionality needed, but getting that last 10% of getting the graphic to show would be greatly appreciated.
For access to the full code please feel free to visit: http://gis.florenceaz.gov/Web_Maps/Address_Search.html

It seems like html2canvas doesn't handle the intersections of SVGs and CSS styles. You can fix issues like this by setting the opacity attribute on the given element or adding a style tag to your svg. Maybe html2canvas will handle this issue in the future, but for now these are workarounds.

Is this still an issue with v1.0.0? If so, could you please share an example on jsfiddle.


Allow composition token to have multiple fill properties

Is your feature request related to a problem? Please describe.
Figma allows multiple fills to be applied on a layer. Currently the plugin does not support this.
Describe the solution you'd like
Allow composition token to have multiple fill properties defined.

Original Issue
From @chrisyalamov
Big +1 for this feature. Looking at the Group Tokens section of the draft spec for design tokens (https://design-tokens.github.io/community-group/format/#groups-versus-composite-tokens), it seems to describe something similar to the composition tokens feature in Tokens Studio:
Groups are for arbitrarily grouping tokens for the purposes of naming and/or organization.
Perhaps the plugin could allow for a composition token to contain several color/gradient tokens which apply the 'fill' property?

Shouldnt this be part of a Composition Tokens M2? Feels like that would be a good fit

This request has been moved to our new feedback tool. Go check out https://tokensstudio.featurebase.app/ to create a free account. You can upvote and comment on existing posts, make a new request, and subscribe to be notified when requests move into our roadmap.

Added to Featurebase Roadmap : 🪙 ➕ Support multiple tokens per layer of the same type


fix(chat): add default values in application dialog (Issue #2477)

Description:
Need to add default value for entrypoint/name/version/icon fields
Issues:

Issue #2477

UI changes

Checklist:

[x] the pull request name complies with Conventional Commits
[x] the pull request name starts with fix(<scope>):, feat(<scope>):, feature(<scope>):, chore(<scope>):, hotfix(<scope>): or e2e(<scope>):. If contains breaking changes then the pull request name must start with fix(<scope>)!:, feat(<scope>)!:, feature(<scope>)!:, chore(<scope>)!:, hotfix(<scope>)!: or e2e(<scope>)!: where <scope> is name of affected project: chat, chat-e2e, overlay, shared, sandbox-overlay, etc.
[x] the pull request name ends with (Issue #<TICKET_ID>) (comma-separated list of issues)
[x] I confirm that do not share any confidential information like API keys or any other secrets and private URLs

/deploy-review

/deploy-review

/deploy-review

/deploy-review

/deploy-review

/deploy-review


Correction for raw affiliation Department of Pediatrics, Western University, London, ON

Correction needed for raw affiliation Department of Pediatrics, Western University, London, ON
raw_affiliation_name: Department of Pediatrics, Western University, London, ON
new_rors: 02grkyz14
previous_rors: 02agqkc58
works_examples: W4378745278;W4389178866
contact: 123d559f5050b3879488fa31357c115c:0e8056c6545edd75 @ uwo.ca

This issue was accepted and ingested by the OpenAlex team on 2024-12-21. The new affiliations should be visible within the next 7 days.


conformance: add attachedRoutes to conformance tests

What type of PR is this?
/kind feature
/area conformance
What this PR does / why we need it:
Add attachedRoutes to conformance tests.
Which issue(s) this PR fixes:
https://github.com/kubernetes-sigs/gateway-api/issues/1543
Does this PR introduce a user-facing change?:
NONE

Thanks @ChaningHwang!
/ok-to-test


Kokoro basictests mac build fails with: Invalid JWT: Token must be a short-lived token and in a reasonable timeframe

Some problem with the service account key on macos workers:
2017-08-08 20:06:25,635 Attempting refresh to obtain initial access_token
2017-08-08 20:06:25,637 Refreshing access_token
2017-08-08 20:06:25,754 Failed to retrieve access token: {
 "error": "invalid_grant",
 "error_description": "Invalid JWT: Token must be a short-lived token and in a reasonable timeframe"
}

Traceback (most recent call last):
  File "workspace_c_macos_dbg_native/tools/run_tests/run_tests.py", line 1599, in <module>

Not sure if it's a one-off or not.
https://sponge.corp.google.com/invocation?id=4995ab68-656d-40bd-88ff-ee765a52e425&searchFor=

After manually checking the last 6 MacOS dbg builds, I don't see this issue at all.
@dgquintas, can you add this error to your Kokoro issue tracker to see if this is a consistent issue? Thanks

Unfortunately it's still happening. 22 instances over the last week.

https://stackoverflow.com/questions/36189612/token-must-be-a-short-lived-token-and-in-a-reasonable-timeframe - It looks like poor computer clock synchronization might be the issue. I'll report this to the Kokoro team.

@dgquintas - do you know if this is still happening?

I haven't seen it happen in a while.
On 12 September 2017 at 17:57, Matt Kwong<EMAIL_ADDRESS>wrote:

@dgquintas https://github.com/dgquintas - do you know if this is still
happening?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/grpc/grpc/issues/12129#issuecomment-329026240, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAHVmVoEA2R8eG8OSdg3DMCbGK6PyCzlks5shyhpgaJpZM4OyJyI
.

This has started happening again on master


LazyLoader null check missing

What will happen if object will be created by this constructor
public Post() {}
and after that this property will be accessed:
public Blog Blog
{
    get => LazyLoader.Load(this, ref _blog);
    set => _blog = value;
}

LazyLoader in this situation will be null. Right?

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

ID: 956653a9-3acf-e361-94a7-2eff333619c9
Version Independent ID: 7c665098-afcc-1ba0-d6a2-0262ae921575
Content: Lazy Loading of Related Data - EF Core
Content Source: entity-framework/core/querying/related-data/lazy.md
Product: entity-framework
Technology: entity-framework-core
GitHub Login: @roji
Microsoft Alias: avickers

@MaximMikhisor The Load extension method is designed to no-op when the LazyLoader is null.


Automatic error report

Once I upgrade both of my spyder and spyder kernels, the spyder then pops up an internal error, below us the system details of my spyder:

spyder-kernels (1.0.1) - Jupyter kernels for the Spyder console INSTALLED: 1.0.1 (latest)
spyder-notebook (0.1.2) - Jupyter notebook integration with Spyder
spyder-terminal (0.2.4) - Spyder Plugin for displaying a virtual terminal (OS independent) inside the main Spyder window
spyder-reports (0.1.1) - Spyder-IDE plugin for Markdown reports using Pweave.
spyder-unittest (0.3.1) - Plugin to run tests from within the Spyder IDE
spyder-line-profiler (0.1.1) - Plugin for the Spyder IDE that integrates the Python line profiler.
spyder-memory-profiler (0.1.2)  - Plugin for the Spyder IDE that integrates the Python memory profiler spyder (3.3.0) - Scientific PYthon Development Environment INSTALLED: 3.2.8 LATEST:    3.3.0
spyder-conda (0.1.0)
kn-spyder-tag (3.2.8) - Python project to create a small buzz network associating tags that are most likely to create buzz
spyder.example (0.0.1) - A Spyder Plugin example package
spyder.autopep8 (0.0.1) - A plugin to run the autopep8 python linter from within the spyder editor

the spyder then pops up an internal error

What's the internal error?

I guess you were able to make 3.3.0 work, which solves the problem you reported.


Extract Room from Document and move Presence from Client to Room

Description:
Extract Room from Document and move Presence from Client to Room
AS-IS

TO-BE

Related to

https://github.com/yorkie-team/yorkie-js-sdk/issues/379
https://github.com/yorkie-team/yorkie-js-sdk/issues/302

Why:

Topics cannot be separated if a client attaches two different documents.
We can use transactions by combining the document and the presence. This can reduce the workload by representing Text's selection as presence because presence is cheaper than Document(CRDT).

Move Presence to Document

[x] Go: https://github.com/yorkie-team/yorkie/pull/582
[x] JS SDK: https://github.com/yorkie-team/yorkie-js-sdk/pull/574
[x] iOS SDK: https://github.com/yorkie-team/yorkie-ios-sdk/pull/94
[x] Android SDK: https://github.com/yorkie-team/yorkie-android-sdk/pull/130


feat: Pretty print

It uses https://golang.org/pkg/text/tabwriter/.
Before:

After:

I simplified many code structures with this pull request. So you may be able to apply simply your patches using tabwriter.
Please check and customize here for pretty print. Example as follows:
func defaultWriter(delim string) io.Writer {
        if delim == "\t" {
          // pretty print customize is here
          return CUSTOMIZED_WRITER
        }

	return os.Stdout
}

Thank you!

https://github.com/goldeneggg/lsec2/pull/23


Add user feedback buttons to website

https://www.docsy.dev/docs/adding-content/feedback/#user-feedback

/assign

PR to address this: https://github.com/dapr/docs/pull/1767
Note: on some pages there isn't sufficient space between the User Feedback widget and core website content, and I didn't find a way to configure the text size/spacing in the widget. I've created a feature request to the Docsy team to allow us to fix this, but it might be something we have to live with for now.


DEV: Update ruby linting

Thanks @CvX @Flink. I think @angusmcleod has a big PR in the works, I'm going to hold off on merging this for the time being to avoid any major conflicts between main and his branch.
But should be able to merge next week.

Redo in #49


QUESTION: Should it be recursive?

i.e. If Package A depends on Package B. What should happen? At the moment it appears to build Package A as is, and assume Package B will be built independently and added along side it in the including app. Is this correct?

@ryanbooker that's correct. Much like Swift Binary Target support we don't attempt to solve dependencies as that would be a large problem to undertake.
We do support building your dependencies directly though – the SwiftPM generated xcproject bundle includes targets for everything in the package graph, so you can include them in your build list: swift create-xcframework --list-products will show you everything that is available to be built.
Maybe we should add in a --build-all option as well?
Apologies for the delayed reply!


Add discord link

What

Adds discord link (closes #82)
Removes timeline as it's not relevent anymore

:tada: This PR is included in version 0.17.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


fast trade bugs out with villager shop keepers plugin

im using Fast trading 0.2.0+1.21 on mc 1.21.1 with fabric api 1.21.1 Fabric API 0.110.0+1.21.1
https://github.com/user-attachments/assets/4b095370-697f-4dd1-8a7f-89a43bd48fe1
latest.log here is my log file

Have you tried changing the configs?
There's an option called autofillBehavior that should fix the problem


Updated VesselVAE and added NORAH

I've updated the repo link in VesselVAE that was not right. Also added https://link.springer.com/chapter/10.1007/978-3-031-43993-3_8

awesome, thank you!


Orientation property of PWA

Hi all,
I generated an APK (and I'm posting it to Google Play) from my PWA: android.cromimi.com
I used PWABuilder which used Bubblewrap to generate an APK from my PWA
It seems that the generated APK does not respect the orientation of the PWA (landscape only).
Is there any way to solve the problem ?
Regards,
Benoît

This looks like a duplicate of #281

Closing as duplicate of #281


Feature/update package data

Changes package.json and package-lock.json to reflect change to Digital UK namespace.

Created in error


Couldn't find all answers

Hello,
I am trying to fine-tune tapas on my table following the steps  here. I generated my dataset by adding coordinators and answer_text as follows:

but when I'm running TableDataset, an error appears: ValueError: Couldn't find all answers. Can you help me with this error? How can i fix it?

Hi,
I am running into the exact same issue, would you mind sharing how you resolved it ?
Thanks,

+1 Me too

I also had this issue. It seems the table I was using was too large (163 rows x 16 columns) and perhaps as a result when truncating the table some of the answer values were removed.
To solve this I just made the table smaller (20 rows x 7 columns) and updated my questions and answer locations accordingly. I am unsure if reducing both the number of columns and rows is necessary, or what the maximum possible table size is. Perhaps someone else can provide more insight here?

I am also facing same issue. would you mind sharing how you resolved it ?
Thanks


Added the option to load OpenFX plugins

Description of changes
Please consider that this is my first PR
If you need plugins to test this feature, you can use https://codeberg.org/BENDI/openfxmisc-flake.

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[x] Fits CONTRIBUTING.md.

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

I have no problem with this but I am away and can't test until Monday

Additionally, I’m curious why one wouldn’t instead set this in their user environment. Particularly as the env var is “standard” and transcends just Davinci Resolve.

When I initially set out to add this feature, I was not aware of the env var.
Not including the env var in the wrapper would mean that the store paths would have to be appended to the environment variable in the global context. I think this could be done via NixOS module? With audio plugins for example, there is no such module, necessitating the addition of a big block of code to the NixOS config [1].
Another option would be having to include the plugin via e.g. nix shell, but I think this workflow is more suited to software development. Having all plugins available all the time while editing surely is preferable to needing a shell.nix for every project you work on?
[1] https://wiki.nixos.org/wiki/Audio_production

A whole custom module isn’t really needed, and environment.variables.OFX_SEARCH_PATH = lib.concatStringsSep ":" [ … ]; wouldn’t look much different than a theoretical module.

Did you mean OFX_PLUGIN_PATH or is there another variable I am not aware of?

To be clear: not saying adding it to the wrapper doesn’t make sense or is bad, just enumerating the solution space a bit.

Of course. I just wanted to explain my thought process in detail, no worries.
After thinking about it a little longer, I believe there are three equally valid solutions:

Adding the option to the package. This has decent discoverability and is expressive of what users want to do with the software. The isolation of the plugins to resolve only might be desirable. In this case I believe the wrapper should in fact override the env var, so that only the plugins, which are explicitly installed, are loaded.
Creating a module. This is great for discoverability (search.nixos.org), but is, as you stated, kind of superfluous due to its simplicity. It would pave the way for more plugin options in the video space to be added though.
Do not add anything to nixpkgs, create a wiki article just like the one I linked above instead. I think this option does not have the best usability, because users first have to find the wiki article which would not be written for resolve directly and then deal with the somewhat confusing fact that adding a package to the list of strings to be concatenated adds it to  the system closure and effectively "installs" it. A module could get around this issue by renaming the option. Then again, overriding an input of the resolve package also is not straightforward.

I put the PR on draft while we decide on what to do.
@amarshall which solution do you think is best?


Consider getting rid of BytesMut in favor of only Bytes

Currently, Bytes represents an immutable, potentially shared, view into a memory region. BytesMut represents a unique view into a memory region, which means that the memory can be mutated.
Perhaps there is a way to collapse these two types into a single type.
Pros

Less types

Cons

Mutation may have to implicitly copy the memory into a new buffer... this expense may not be obvious. It may also be that extra runtime state may be needed to track if a handle has a unique view or not.

Hm yeah so in some sense all "change the capacity" operations already run the risk of copying all the bytes (e.g. a realloc). As a result the major downside here to me is that otherwise cheap operations like as_mut_slice now have the potential to be expensive.

May I suggest an idea? Not sure it works, but still.
Drop BytesMut struct and introduce
struct BytesRefMut<'a> {
    bytes: &'a mut Bytes,
    begin: *mut u8, // valid pointer even if kind == KIND_INLINE
    len: usize,
    cap: usize
}

impl Drop for BytesMutRef {
    fn drop() {
        flush len if kind == KIND_INLINE
    }
}

impl Bytes {
    // deep copy if refcount > 1
    fn unshare(&mut self) -> BytesMutRef { ... }
}

all mutation operations reside in BytesMutRef object
refcounter is checked only in unshare operation (unlike mutation operation on Bytes object)
unshare can be expensive (because of copying of buffer), but it is explicit now
repeated put_u8 operations are cheaper on BytesMutRef than on BytesMut, because kind is checked only twice (in unshare and in drop), not on each put_u8
that object may have cheap push operation

I don't think this idea will pan out. So, I'm going to close this issue.


Filled orders not applying in the bot

The issue is that the bot puts an order in a decent time, but after the order is filled, it doesn't start tracking it, it doesn't acknowledge that there is already a position to trade with.
So after the bot buys and it is filled, it just ignores it.

I have the same problem

I've tracked this to the point where the bot deletes the last-buy-price, there are two places where it happens. I don't know which one is the issue, however, I suspect it's because the balance calculation is done on old data. Sometime it works, sometimes it'll delete the last-buy-price.
I have commented out the two places where the last-buy-price is deleted and it works fine, it just keeps the last-buy-price in the UI which I'm not too fussed about.

@kaziqta
I am not sure I understood the issue. The bot does not need to track the order as it is working based on the coin balance.
Can you elaborate on what the actual issue is?

@kaziqta
I am not sure I understood the issue. The bot does not need to track the order as it is working based on the coin balance.
Can you elaborate on what the actual issue is?

Sometimes, when it places an order and the order gets filled, nothing happens in regards to the bot, it ignores it.

@kaziqta That kind of issue never happened to me yet. Without any log, I wouldn't be able to find what went wrong.

It may be something from a performance point of view, will check out logs, but nothing there for now.


[channel key]

where do i get the android channel key ?
Is it in the Mobile SDK

You can get channel key from Zendesk admin page.


Incorporating icons, like Academicons

A suggestion: to make the theme more welcoming to academics looking for a more stylish personal website than other Hugo theme options, consider making it easy to include Academicons.
I assume it doesn't take many steps to do, if you already know how. The thing is, of course, learning how!
On a related issue, perhaps it is worth mentioning something about font awesome icons too? For example, are are the free ones built-in and ready to incorporate with the right code, like you do in themes/hugo-scroll/index.html? Or does the user need to take additional steps to be able to make use of other icons?

Hi @damonlynch, I see you are making good progress on your website 👍🏻. Let me reply in two parts:
1. Using external icon sets (e.g. Academicons)
You simply need to include the respective stylesheet; in your case, that would be
<link rel="stylesheet"<EMAIL_ADDRESS>
in the -tag of your site.
To achieve this, you may create a layouts/partials/custom_head.html in your site's root directory containing this line:
<link rel="stylesheet"<EMAIL_ADDRESS>
With the stylesheet available you can then use the icons as it is described on their website, i.e., using -tags like

in your .html-content.
If you want to use icons in .md-files, you could enable HTML-rendering via adding
[markup]
    [markup.goldmark]
        [markup.goldmark.renderer]
            unsafe = true

to your config.toml.
2. What's built-in?
You are correct, this is a little bit of a hidden feature at the moment 😸, but as you already saw, there is an icon on the header of the start page. This theme currently includes font-awesome v4.0.3 for that purpose. You can check out https://fontawesome.com/v4.7.0/cheatsheet/ for a list of all icons.
You can use all those icons in .html via -tags . Again, if you want to use theme in .md-files, you need to enable unsafe-rendering for markdown, see above.
2.5 😅 Future Work
To make using icons in .md-files a lot nicer / user-friendly, we could introduce a shortcode. Then, you would not need to enable unsafe-rendering and could write something like
{{}} Mail me!

in your usual .md-file.
You can add this functionality locally by creating a layouts/shortcodes/i.html-file with the following content:
{{ with .Get "class" }}{{ end }}

I hope that helps 😄 . Let me know if this works for you.
Kind regards,
Jan

Hi @damonlynch, I saw that you added icons to the »contact«-section of https://damonlynch.net/, so I suppose we can close this issue.

By the way, I really like what you did with the theme. https://damonlynch.net/ looks very nice 👍🏻 .

Thanks Jan for the detailed instructions, and for your kind words! Do you want the theme's README.md updated to include some of this information? Of course you probably don't want to overwhelm that page with extraneous information, but I suppose a hint that it's possible to add these features easily might provide enough encouragement for others to also use your theme.
If you do like this idea, I can try to figure out how to do it in git and make a pull request. I'm very new to git but I need to figure git basics out at some point.

Hi @damonlynch, I took the time

to update the included font-awesome-icons to v4.7.0 (https://github.com/janraasch/hugo-scroll/commit/edc14934d0c7e8ff4e92ea5a0c5f47e865d11691)
add an icon-shortcode and add an "Using icons"-section to the README.md (https://github.com/janraasch/hugo-scroll/commit/bde829194418d599c41187d6b54cbe7bb1d3ce80).

What do you think? Is that good enough?
PS:

I'm very new to git but I need to figure git basics out at some point.

If it's just about editing some .md-files for documentation, you might also want to try to use the Github-UI directly to make up a pull request.


Sm/installer promotion fix

What does this PR do?
resolves the issue that was causing installers to point at old versions
What issues does this PR fix or reference?
@W-0@

I prefer having the separate jobs for packaging/uploading and promoting. Why consolidate them into one?

Agreed, but it wasn't working (the oclif promote command is uploading targeted tarballs anyway and there wasn't a good way to prevent it from doing that without making bigger changes to oclif).

Isn't the checkout step pulling the latest commit on the main branch so it should be the same across jobs in a workflow?
Yes...any reason you're asking?

Never mind, I thought whatever was checked out had something to do with what the promote command did but seems like it doesn't.


chore(contribs/accenture-bif): delete old code

Fixes #300
Deleted the whole folder with the old simple
asset transfer examples instead of applying a
targeted fix because this is the lowest effort
solution that gets the job done and we want
to focus on moving forward not keep fixing
up the old code that's really just there for
posterity's sake. This will also break some
links that are out there on the internet but
the sooner we fix those the better anyway.
Considered as an alternative just removing
the fabric specific parts of the old simple asset
transfer code but that also would've ended up
being a surgical move because of the dependencies
within the scripts that do the foriegn validator
key pre-sharing so instead of turning this into
a rabbit hole I'd just opted to take the whole
thing out and spend the saved up time on making
sure that we have the simple asset transfer
example done within the new codebase.
Signed-off-by: Peter Somogyvari<EMAIL_ADDRESS>

cc: @ryjones @jonathan-m-hamilton

@swinslow LGTM


cmd/go2go: Panic in type checker

What version of Go are you using (go version)?

Whatever is deployed at go2goplay.golang.org as of 11/11/2020 1:21 PM CST.

Does this issue reproduce with the latest release?
Presumably... I only tried it at go2goplay.golang.org
What operating system and processor architecture are you using (go env)?
go env Output
$ go env

What did you do?
I tried to make this example compile. I don't claim that it's valid syntax, but it causes a panic in the typechecker, so I figured I'd raise it in any case.
https://go2goplay.golang.org/p/rC4sofqXuA5
What did you expect to see?
A compiler error (I think the use of [S] at line 16 is wrong, but don't quote me).

This is a bug in the type-checker in the prototype; but it works with the most-up-to-date implementation in dev.typeparams:
$ go tool compile -G x.go
x.go:16:9: invalid operation: cannot index FT.Map (value of type func(_ FT, f func(T) S) S)

where x.go contains the linked program, and the compiler was built from the dev.typeparams branch (note that the compiler doesn't generate code - it just type-checks).
cc: @findleyr

@griesemer Ok cool! This issue can be closed as far as I'm concerned.
I'll check twice before submitting anything from the go2go playground again.

Please keep reporting. We will double-check both implementations - and even if we don't update the prototype, we'll add problem cases as tests to dev.typeparams if they are not covered already by existing tests.
Also, I'll keep this open for now until we have verified that both the compiler and a future go/types behave as expected with respect to this problem case, and/or we have added a matching test case.
Thanks!

Closing as this works with a proper error message in Go 1.18.


Documentation - statement of need

Hi there,
For my review,

Documentation

[ ] A statement of need: Do the authors clearly state what problems the software is designed to solve and who the target audience is?

I couldn't see this information clearly in the docs. It was clear in the software paper, though.

I think the docs are still missing

who the target audience is?

and it could be much clearer about what it calculates, i.e. the Bayesian evidence. I think both should be in the first few (if not the first) paragraph of the docs.

This is a good point - now looking at it again you are definitely right. I have added a sentence saying what nested sampling calculates as the first sentence of the homepage here https://dypolychord.readthedocs.io/en/latest/.
I am going to provisionally close this issue but let me know if you think it should be reopened!

I should add: I think it is implicit that the target audience is "people who want to calculate Bayesian evidences and/or posterior samples from a likelihood and prior", but let me know if you think this should be further clarified.


Invalid log message in TokenRequestValidator

[x ] I read and understood how to enable logging

Issue / Steps to reproduce the problem
If an authZCode.ClientID (XXXX) does not match the requested clientID (YYYY), the log message shows:
Client XXX is trying to use a code from client XXX
Current Code (line 203):
LogError("Client {clientId} is trying to use a code from client {clientId}", _validatedRequest.Client.ClientId, authZcode.ClientId);
Proposed Code:
LogError("Client {0} is trying to use a code from client {1}", _validatedRequest.Client.ClientId, authZcode.ClientId);
The log is obviously getting confused about which client ID to use.

PR?

Happy to create a PR.  How do I get permission to push a PR branch up?  I can sign the https://cla.dotnetfoundation.org/ form but there is no IS4 project to select.

Oh - thanks for making me aware of that - the project name is wrong - you need to choose "Thinktecture.IdentityServer".
Just start with the PR - the rest will be taken care of by the "CLA Bot"

This looks like it will happen on line 512 as well.
LogError("{clientId} tries to refresh token belonging to {clientId}", _validatedRequest.Client.ClientId,
refreshToken.ClientId);
So it might be worth updating your pull request with this one too.

@initialspark Good catch.  Updated the PR

merged, thanks.


Broken page link

The page has some text about Microsoft Defender for Kubernetes being deprecated, and recommending use of Defender for Containers. However, the link to Defender for Containers takes you to Defender for Servers, which seems to be something different (though maybe you are supposed to enable Defender for Servers on your K8s nodes and maybe the link is right and the text wrong; I'm now quite deeply confused about the Microsoft Defender ecosystem).

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

ID: 372412d3-28c0-79d3-2c5c-2565d943a18f
Version Independent ID: 3bcef49b-b2be-4680-e762-4135e8ea9c79
Content: Microsoft Defender for Kubernetes - the benefits and features
Content Source: articles/defender-for-cloud/defender-for-kubernetes-introduction.md
Service: defender-for-cloud
GitHub Login: @bmansheim
Microsoft Alias: benmansheim

@plwhite Thank you for catching this. We will investigate and update the link .

@plwhite We have initiated a change request and this will be updated once in teh next release cycle in 24-48 hours max.

Thanks - always nice to see these simple things quickly resolved.

@plwhite You're right. That link was supposed to go to the Defender for Containers page. Sorry to confuse you.
The fix was merged and will be live in a few hours.
Thanks for bringing to our attention!

#please-close


Update build instructions.

Please update build instructions to issue a
git checkout v0.22 or whatever is the latest tag before modifying Makefile & building as the main branch mayn't be the most stable.

a bit late to the party, but I've written instruction to use GNU Make on this PR


Build not updated with 5.0.20?

Not sure what's happened but it seems like #671 wasn’t actually included in the latest published version of @styled-system/css, 5.0.20.

Thanks! Yeah, that's odd, will try publishing a patch now

Republished as 5.0.21 if you can confirm that it's working now

@jxnblk yep all good now, thanks!


Remove QSPI inout port and replace with input/output

https://github.com/chipsalliance/caliptra-rtl/issues/113

Fixes QSPI ports
Fixes smoke tests
Adds qspi/uart to regression

Before we complete the PR, can you also fix the qspi ports here?
https://github.com/chipsalliance/caliptra-rtl/blob/dev-goog-qspi/src/integration/uvmf_caliptra_top/uvmf_template_output/project_benches/caliptra_top/tb/testbench/hdl_top.sv

Before we complete the PR, can you also fix the qspi ports here? (line 200) https://github.com/chipsalliance/caliptra-rtl/blob/dev-goog-qspi/src/integration/uvmf_caliptra_top/uvmf_template_output/project_benches/caliptra_top/tb/testbench/hdl_top.sv

done!


Prefixed string literals

Prefixed string literals will right now emit the prefix as is:
#define S L"string"

emits
const S = L"string"

leading to an error when the file is included since no L-string macro is defined.
Perhaps the prefix can just be removed, alt a no-op L-string macro can be defined since the cconvert from the type signatures should take care of it (albeit with a copy for String -> Cwstring).

The hard part is that libclang doesn't expose enough APIs for identifying the L-string macros.


MAC arm64

Can you please add a version that is compatible with MAC arm64?

Should already be in there. Are you having trouble?


Multi-table CTL

Implement multi-table CTL allowing a table to be looked up from several other tables.

Not really about this PR, but I notice we're using _next and _right, with the same meaning it seems. We should probably standardize it. I think _next seems more fitting since it's a "vertical" offset.

Yeah I agree next makes more sense. I can make a PR to standardize this 👍


[PE-D][Tester C] Documentation says that user can input a calorie using the add function where actually the user can only input an integer for calorie

Should be more descriptive like calorie can only be an integer or change implementation to allow decimal calories as well.

Labels: type.DocumentationBug severity.Medium
original: jwdavis0200/ped#9

Issue resolved


Implement graph driver restore on reboot

This fixes reference counting for overlayfs, devicemapper, and zfs to restore the initial count by checking if the path that is to be reference counted is already mounted for when the daemon dies with running containers.
It basically changes the reference counter from using the id of the layer to using the path so that we can check if it is reference counted or not.
Overlay needed additional changes because it was using a path cache and that could be removed.

There is already refcounting in the layer store, this seems like the best place to keep things since other subsystems should be interacting only with the layer store.
I'd added the ref counter at this level because of fsdiff, but it seems like we should be able to move this so the interactions from fsdiff go through the layer store.

@cpuguy83 you cannot do that at that level because these drivers mount themselves.

@crosbymichael This still requires #21732 to work properly, right? I wonder if we are ready to make the live-restoring default in master as there are still unresolved issues around networking.

@tonistiigi this works without that PR.  I verified using things like diff and the correct layer counts are updated without having to persist path data to disk and read it after restore.

@cpuguy83 @tonistiigi ok, i've added a fast path for checking if a path is mounted in the reference counter.
I also added the reference counter to aufs as it is the last driver that does mounts but did not have any reference counting.  I just have to work on removing the code from the layer store because I don't think a solution like #21732 will be needed now as it wouldn't fit every driver anyways.

Looks like build is failing;
21:58:33 # github.com/docker/docker/layer
21:58:33 c:\go\src\github.com\docker\docker\layer\layer_test.go:679: rl.activityCount undefined (type *referencedRWLayer has no field or method activityCount)
21:58:33 c:\go\src\github.com\docker\docker\layer\layer_test.go:680: rl.activityCount undefined (type *referencedRWLayer has no field or method activityCount)

@jstarks @jhowardmsft
for this error on tp5 tests:
23:46:31 ... "restart 56528aba7109c156db0432f1a9e67269ee9cde1cdd698874ba6e5f1da3622469" failed with errors: Error response from daemon: Cannot restart container 56528aba7109c156db0432f1a9e67269ee9cde1cdd698874ba6e5f1da3622469: hcsshim::ActivateLayer failed in Win32: The process cannot access the file because it is being used by another process. (0x20) id=56528aba7109c156db0432f1a9e67269ee9cde1cdd698874ba6e5f1da3622469 flavour=1

Is there a call that we can make to hcsshim that will tell us if the layer is already active or mounted?

@swernli Can you look into this?

LGTM for the Windows graphdriver.

ping @tonistiigi @cpuguy83 PTAL

cc @dmcgowan

@dmcgowan any other comments/changes?  i'll make them in a batch when you are finished with the review

Just that one nit, otherwise overlay and layerstore LGTM

Probably a good idea to ask @rhvgoyal for review of this.

@rhvgoyal can you please checkout the devmapper changes made in this pr?

@rhvgoyal you are commenting on outdated commits

@crosbymichael  Oops. Let me look at the latest one.

@rhvgoyal no problem, just wanted to let you know before you got too far

LGTM

LGTM

It has merge label and passed CI. I'm merging.


Crypto.com - Error parsing data on multiple bots using a single API key

Describe the bug
Using crypto_com as exchange and using single API key to multiple instance show a prasing error (crypto_com_exchange - Error parsing data)
Steps to reproduce
Here are the steps to reproduce the issue (see attachments in the section below):

Create simple config on PMM and use crypto_com as your exchange
Create multiple bots and run, after few seconds/minutes there will be an error
...

2021-06-03 13:45:58,061 - 1 - hummingbot.client.hummingbot_application - INFO - start command initiated.
2021-06-03 13:45:58,493 - 1 - hummingbot.connector.exchange.crypto_com.crypto_com_exchange - INFO - Network status has changed to NetworkStatus.CONNECTED. Starting networking...
2021-06-03 13:45:58,861 - 1 - hummingbot.connector.exchange.crypto_com.crypto_com_order_book_tracker - INFO - Initialized order book for EGLD-USDT. 1/1 completed.
2021-06-03 13:46:00,180 - 1 - hummingbot.connector.exchange.crypto_com.crypto_com_exchange - ERROR - Error parsing data from https://api.crypto.com/v2/private/get-account-summary. Error: Expecting value: line 1 column 1 (char 0)
Traceback (most recent call last):
  File "/home/hummingbot/hummingbot/connector/exchange/crypto_com/crypto_com_exchange.py", line 338, in _api_request
    parsed_response = json.loads(await response.text())
  File "/home/hummingbot/miniconda3/envs/hummingbot/lib/python3.8/json/__init__.py", line 357, in loads
    return _default_decoder.decode(s)
  File "/home/hummingbot/miniconda3/envs/hummingbot/lib/python3.8/json/decoder.py", line 337, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/home/hummingbot/miniconda3/envs/hummingbot/lib/python3.8/json/decoder.py", line 355, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

Screenshots

Release version
v.0.39.0
Attachments
// Please attach your config file and log file to help us debug. It would be difficult for us to help you without those!
logs_conf_pure_mm_1.log

log from one of my bot
logs_mkr.log- log from user
conf_pure_mm_1.txt - config used

WARNING: Do NOT publish any exchange API keys or your wallet's private key. Whoever has access to them may steal your assets!
┆Issue is synchronized with this Clickup task by Unito

This was initially reported to the team that it may have been caused by hitting the API rate limit. Looking at the API docs, that issue should throw a TOO_MANY_REQUESTS error which is not in the logs.
https://exchange-docs.crypto.com/spot/index.html#rate-limits
@cali-tomas can you run another test - instead of running multiple instances, try creating a bot that creates a total of 10 orders with 30 seconds order refresh time? Send the results here with the information packets.

This is the result of my testing. The bot works well with 8 order levels (Set to 10 but not enough balance) and 30sec refresh time

strategy: pure_market_making
exchange: crypto_com
market: EGLD-USDT

bid_spread: 4.0
ask_spread: 5.0
minimum_spread: -100.0
order_refresh_time: 30.0
max_order_age: 1800.0
order_refresh_tolerance_pct: 0.0
order_amount: 0.13

price_ceiling: -1.0
price_floor: -1.0

ping_pong_enabled: false

inventory_skew_enabled: false
inventory_target_base_pct: 50.0
inventory_range_multiplier: 1.0
inventory_price: 1.0

order_levels: 10
order_level_amount: 0.001
order_level_spread: 0.001

filled_order_delay: 60.0

hanging_orders_enabled: false
hanging_orders_cancel_pct: 10.0

order_optimization_enabled: false
ask_order_optimization_depth: 0.0
bid_order_optimization_depth: 0.0

add_transaction_costs: false

price_source: current_market
price_type: mid_price
price_source_exchange:
price_source_market:
price_source_custom_api:
take_if_crossed:
order_override:

2021-06-08 12:47:38,001 - 1 - hummingbot.strategy.pure_market_making.pure_market_making - INFO - (EGLD-USDT) Creating 9 bid orders at (Size, Price): ['0.13 EGLD, 83.5546 USDT', '0.131 EGLD, 83.5537 USDT', '0.132 EGLD, 83.5529 USDT', '0.133 EGLD, 83.552 USDT', '0.134 EGLD, 83.5511 USDT', '0.135 EGLD, 83.5503 USDT', '0.136 EGLD, 83.5494 USDT', '0.137 EGLD, 83.5485 USDT', '0.001 EGLD, 83.5476 USDT']
2021-06-08 12:47:38,126 - 1 - hummingbot.connector.exchange.crypto_com.crypto_com_exchange - INFO - Created LIMIT_MAKER BUY order HBOT-B-EGLD-USDT-1623156458001707 for 0.131 EGLD-USDT.
2021-06-08 12:47:38,126 - 1 - hummingbot.core.event.event_reporter - EVENT_LOG - {"timestamp":<PHONE_NUMBER>.0, "type": "OrderType.LIMIT_MAKER", "trading_pair": "EGLD-USDT", "amount": "0.131", "price": "83.5537", "order_id": "HBOT-B-EGLD-USDT-1623156458001707", "exchange_order_id": null, "leverage": 1, "position": "NILL", "event_name": "BuyOrderCreatedEvent", "event_source": "crypto_com"}
2021-06-08 12:47:38,136 - 1 - hummingbot.connector.exchange.crypto_com.crypto_com_exchange - INFO - Created LIMIT_MAKER BUY order HBOT-B-EGLD-USDT-1623156458001809 for 0.136 EGLD-USDT.
2021-06-08 12:47:38,136 - 1 - hummingbot.core.event.event_reporter - EVENT_LOG - {"timestamp":<PHONE_NUMBER>.0, "type": "OrderType.LIMIT_MAKER", "trading_pair": "EGLD-USDT", "amount": "0.136", "price": "83.5494", "order_id": "HBOT-B-EGLD-USDT-1623156458001809", "exchange_order_id": null, "leverage": 1, "position": "NILL", "event_name": "BuyOrderCreatedEvent", "event_source": "crypto_com"}
2021-06-08 12:47:38,144 - 1 - hummingbot.connector.exchange.crypto_com.crypto_com_exchange - INFO - Created LIMIT_MAKER BUY order HBOT-B-EGLD-USDT-1623156458001791 for 0.135 EGLD-USDT.
2021-06-08 12:47:38,144 - 1 - hummingbot.core.event.event_reporter - EVENT_LOG - {"timestamp":<PHONE_NUMBER>.0, "type": "OrderType.LIMIT_MAKER", "trading_pair": "EGLD-USDT", "amount": "0.135", "price": "83.5503", "order_id": "HBOT-B-EGLD-USDT-1623156458001791", "exchange_order_id": null, "leverage": 1, "position": "NILL", "event_name": "BuyOrderCreatedEvent", "event_source": "crypto_com"}
2021-06-08 12:47:38,152 - 1 - hummingbot.connector.exchange.crypto_com.crypto_com_exchange - INFO - Created LIMIT_MAKER BUY order HBOT-B-EGLD-USDT-1623156458001753 for 0.133 EGLD-USDT.
2021-06-08 12:47:38,152 - 1 - hummingbot.core.event.event_reporter - EVENT_LOG - {"timestamp":<PHONE_NUMBER>.0, "type": "OrderType.LIMIT_MAKER", "trading_pair": "EGLD-USDT", "amount": "0.133", "price": "83.5520", "order_id": "HBOT-B-EGLD-USDT-1623156458001753", "exchange_order_id": null, "leverage": 1, "position": "NILL", "event_name": "BuyOrderCreatedEvent", "event_source": "crypto_com"}
2021-06-08 12:47:38,160 - 1 - hummingbot.connector.exchange.crypto_com.crypto_com_exchange - INFO - Created LIMIT_MAKER BUY order HBOT-B-EGLD-USDT-1623156458001774 for 0.134 EGLD-USDT.
2021-06-08 12:47:38,160 - 1 - hummingbot.core.event.event_reporter - EVENT_LOG - {"timestamp":<PHONE_NUMBER>.0, "type": "OrderType.LIMIT_MAKER", "trading_pair": "EGLD-USDT", "amount": "0.134", "price": "83.5511", "order_id": "HBOT-B-EGLD-USDT-1623156458001774", "exchange_order_id": null, "leverage": 1, "position": "NILL", "event_name": "BuyOrderCreatedEvent", "event_source": "crypto_com"}
2021-06-08 12:47:38,200 - 1 - hummingbot.connector.exchange.crypto_com.crypto_com_exchange - INFO - Created LIMIT_MAKER BUY order HBOT-B-EGLD-USDT-1623156458001732 for 0.132 EGLD-USDT.
2021-06-08 12:47:38,201 - 1 - hummingbot.core.event.event_reporter - EVENT_LOG - {"timestamp":<PHONE_NUMBER>.0, "type": "OrderType.LIMIT_MAKER", "trading_pair": "EGLD-USDT", "amount": "0.132", "price": "83.5529", "order_id": "HBOT-B-EGLD-USDT-1623156458001732", "exchange_order_id": null, "leverage": 1, "position": "NILL", "event_name": "BuyOrderCreatedEvent", "event_source": "crypto_com"}
2021-06-08 12:47:38,210 - 1 - hummingbot.connector.exchange.crypto_com.crypto_com_exchange - INFO - Created LIMIT_MAKER BUY order HBOT-B-EGLD-USDT-1623156458001824 for 0.137 EGLD-USDT.
2021-06-08 12:47:38,210 - 1 - hummingbot.core.event.event_reporter - EVENT_LOG - {"timestamp":<PHONE_NUMBER>.0, "type": "OrderType.LIMIT_MAKER", "trading_pair": "EGLD-USDT", "amount": "0.137", "price": "83.5485", "order_id": "HBOT-B-EGLD-USDT-1623156458001824", "exchange_order_id": null, "leverage": 1, "position": "NILL", "event_name": "BuyOrderCreatedEvent", "event_source": "crypto_com"}
2021-06-08 12:47:38,250 - 1 - hummingbot.connector.exchange.crypto_com.crypto_com_exchange - INFO - Created LIMIT_MAKER BUY order HBOT-B-EGLD-USDT-1623156458001644 for 0.130 EGLD-USDT.
2021-06-08 12:47:38,252 - 1 - hummingbot.core.event.event_reporter - EVENT_LOG - {"timestamp":<PHONE_NUMBER>.0, "type": "OrderType.LIMIT_MAKER", "trading_pair": "EGLD-USDT", "amount": "0.130", "price": "83.5546", "order_id": "HBOT-B-EGLD-USDT-1623156458001644", "exchange_order_id": null, "leverage": 1, "position": "NILL", "event_name": "BuyOrderCreatedEvent", "event_source": "crypto_com"}
2021-06-08 12:47:38,263 - 1 - hummingbot.connector.exchange.crypto_com.crypto_com_exchange - INFO - Created LIMIT_MAKER BUY order HBOT-B-EGLD-USDT-1623156458001840 for 0.001 EGLD-USDT.
2021-06-08 12:47:38,263 - 1 - hummingbot.core.event.event_reporter - EVENT_LOG - {"timestamp":<PHONE_NUMBER>.0, "type": "OrderType.LIMIT_MAKER", "trading_pair": "EGLD-USDT", "amount": "0.001", "price": "83.5476", "order_id": "HBOT-B-EGLD-USDT-1623156458001840", "exchange_order_id": null, "leverage": 1, "position": "NILL", "event_name": "BuyOrderCreatedEvent", "event_source": "crypto_com"}```

Using one single api key every three instances solved the issue for me.
Your mileage may vary if you have more order levels or higher or lower refresh time

Tested on the current version using PMM same API and markets getting the same behavior
2022-02-25 04:34:00,129 - 1 - hummingbot.connector.exchange.crypto_com.crypto_com_exchange - ERROR - Error parsing data from https://api.crypto.com/v2/private/get-account-summary. Error: Expecting value: line 1 column 1 (char 0)
Traceback (most recent call last):
  File "/home/hummingbot/hummingbot/connector/exchange/crypto_com/crypto_com_exchange.py", line 343, in _api_request
    parsed_response = json.loads(await response.text())
  File "/home/hummingbot/miniconda3/envs/hummingbot/lib/python3.8/json/__init__.py", line 357, in loads
    return _default_decoder.decode(s)
  File "/home/hummingbot/miniconda3/envs/hummingbot/lib/python3.8/json/decoder.py", line 337, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/home/hummingbot/miniconda3/envs/hummingbot/lib/python3.8/json/decoder.py", line 355, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/hummingbot/hummingbot/connector/exchange/crypto_com/crypto_com_exchange.py", line 558, in _status_polling_loop
    await safe_gather(
  File "/home/hummingbot/hummingbot/core/utils/async_utils.py", line 22, in safe_gather
    return await asyncio.gather(*args, **kwargs)
  File "/home/hummingbot/hummingbot/connector/exchange/crypto_com/crypto_com_exchange.py", line 579, in _update_balances
    account_info = await self._api_request("post", CONSTANTS.GET_ACCOUNT_SUMMARY_PATH_URL, {}, True)
  File "/home/hummingbot/hummingbot/connector/exchange/crypto_com/crypto_com_exchange.py", line 345, in _api_request
    raise IOError(f"Error parsing data from {url}. Error: {str(e)}")

This happens when you create multiple bots with same API, markets with only 1 set of order at the same time. When I tested it by just using two bots w/ multiple orders [6 and 10] instead of multiple bots w/ single order, it seems to work just fine.

Closing ticket, workaround for this issue is to setup a config with multiple order levels rather using multiple testbots for same market on a single order


Add in a Gradle Property to enable the full execution of the the plugin, without interfering with typical Gradle Builds.

When developing invert with org.gradle.configureondemand=true, the :invert task would run on all subprojects as one would expect.  In order to do this, all projects are marked as dependencies requiring a full configuration of all the projects.
In order to avoid this in standard development, we should add a flag to only do this when a report needs to be generated.
./gradlew :invert -Pinvert.enabled=true or something like this.

In a recent conversation, it sounds like this can be accomplished without subprojects if the correct inputs/outputs are specified, and Gradle will wire it up.
Here is where the code is in invert now:
https://github.com/square/invert/blob/4eb9661df275bc1dbce869ce674eca7ac75dd608/invert-gradle-plugin/src/main/kotlin/com/squareup/invert/InvertGradlePlugin.kt#L39-L64
This is bad because:

It uses afterEvaluate
It uses .subprojects
All of this makes it "work", but requires unnecessary configurationo.

As more work continues, we are prioritizing functionality over Gradle performance.  Because of this, this flag is more important for people that want to install invert directly into their apps.
The approach we use for Invert in open source and Square both use an Gradle --init-script to accomplish this.  This is actually a great way that we can share configurations for Invert like Kotlin Projects vs Java vs Android vs Server (etc) to have a different set of Collectors.
Going forward this is something that should be added.
I'm proposing a invert { enabled true } flag in the configuration DSL.
This would:

Allow us to do this in --init-scripts without having to pass in additional gradle flags.
Allow teams that choose to install Invert directly to add their own gradle flag to manipulate this value in the configuration.


added support to control storedge interface

This PR adds support to control SolarEdge StorEdge interface. Check out the documentation to find out how to use it:
https://www.photovoltaikforum.com/core/attachment/88445-power-control-open-protocol-for-solaredge-inverters-pdf/

Excellent, this is how I resolved it too a while back [here] (https://github.com/nmakel/solaredge_modbus/issues/36#issuecomment-1412904827) - since this is already covered by the inverter itself, I was hoping one day to sort out the change between big and little endian when reading certain parameters without having to specify this separately

https://github.com/nmakel/solaredge_modbus/pull/100#issue-2311779625


[ASGenerator] Improved Generate Function. Adds return statement with default value to body of function if needed and posible

@Neverbirth please review this PR

@Neverbirth what do you think about this PR?

I think this should be a setting. Some developers (me included) don't like this kind of default implementation as it may make code to implement to be forgotten, others prefer to throw exceptions as default implementations, etc.

In case it was not clear, I'm Ok with merging this, but after some setting is added for it. For instance, I know some of my colleagues would be happy with it, but again, not everyone.

Well, I'll try to add this setting today or tomorrow after the main job

@Neverbirth done, please check


PullToRefreshLayout is always visible

PullToRefreshLayout should be visible only when you start dragging it down. Now it is always visible and I have to set the background of  the recyclerView to 100% opacity.

Can you give more details?
-Now (after last update)?
-Always visible? For me - it's not. Screenshot or video would be great.

I have the same problem, tried with the setVisibility(View.INVISIBLE) but the whole listview gone invisible.
The problem is as follow:
-When there is no item in Listview, the animation is shown, even without pulling (in fact you can't even pull it to refresh)
-When there is an item, you can see the images of the animation from the margin between the rows, if there is only one row, you can see half of the image which is so awful.
please help us to hide them while we don't need them.
Btw, thank you for such a beautiful library.

@DavidPacioianu @HamedMP now it is clear :)
I've updated the "dev" branch with a fix for this issue. It would be great if you pull it and try in your projects. I've also spend some time to test it.
If everything is ok - I'll update the maven repo too.

I'll try to check it at the moment and let you know, but before that,
I just updated to version 1.1.2 and the problem solved 50%, without any interaction to listview, the animation is not shown, but when you touch it or ..., it shows again, so only when the view is created everything is ok until you touch something.

@HamedMP now it's only possible to include dev branch as module to your project. Sample uses it in this way too, so you can check how it's done.

I have tried the dev branch module and the recyclerView padding is no longer working and the only thing that happens now is to make the recyclerview opaque, which is not what I expected ( I expected that the refreshLayout will be invisible) and because of that , if set a top margin for the list item it (the margin) becomes opaque and will cover a part of the refreshLayout.

@DavidPacioianu I haven't tested the library with recyclerView yet. Gonna do this asap (at the moment I don't have enough free time) and post an update here.

Any update on this? I'm having the same problem. This is what it looks like for me:

@alokedesai do you use RecyclerView? If not - try to use dev branch, this issue must be fixed there.
I haven't tested this lib with RecyclerView and at the moment I do not have time to do this.
Any pull-requests with fixes will be appreciated :)

First off, amazing job on this project. Animation looks incredible.
I too am using this with the RecyclerView (Specifically android.support.v7.widget.RecyclerView) and it looks exactly as alokedesai's does where the image end up taking up a section of the background. Just wanted to ping it as well as I look forward to including this in my app.
-Sil

@PGMacDesign @alokedesai I've just updated master branch and maven repo with version 1.2.0.
You can check out the new sample with both ListView and RecyclerView examples.
Please leave a feedback if you have any issues with the library.

Wow, Thank you very much, it's solved ^_^
(The problem with listview solved)
On Sat, May 30, 2015 at 6:03 PM, Oleksii<EMAIL_ADDRESS>wrote:

@PGMacDesign https://github.com/PGMacDesign @alokedesai
https://github.com/alokedesai I've just updated master branch and maven
repo with version 1.2.0.
You can check out the new sample with both ListView and RecyclerView
examples.
Please leave a feedback if you have any issues with the library.
—
Reply to this email directly or view it on GitHub
https://github.com/Yalantis/Phoenix/issues/9#issuecomment-107039157.

Shliama, fantastic work! It is flawless with a recyclerview now. Thank you so much for your time and brilliance with this Library.


Add pwd in search paths

Fix # 403

LGTM as soon as Travis agrees

Fixed.
For %%pythran magic, it would not have been a problem because pwd is already added in sys.path as it is an interactive session.

On Sun, Mar 22, 2015 at 10:45:22AM -0700, pbrunet wrote:

Fixed.
For %%pythran magic, it would not have been a problem because pwd is already added in sys.path as it is an interactive session.

Correct! Thanks for the fix!

Changes Unknown when pulling 3edac53a51b397cb051575b82943cf4b48d7eba1 on pbrunet:error403 into ** on serge-sans-paille:master**.

Changes Unknown when pulling 3edac53a51b397cb051575b82943cf4b48d7eba1 on pbrunet:error403 into ** on serge-sans-paille:master**.

Changes Unknown when pulling 3edac53a51b397cb051575b82943cf4b48d7eba1 on pbrunet:error403 into ** on serge-sans-paille:master**.


plugins should reference global variable

To prevent a script from being loaded and reloaded and reloaded, Vim files add (*.vim) typically add the following to the top of them:
if exists('g:loaded_vim_prettier') || &cp
    finish
endif
let g:loaded_vim_prettier = 0.2.3

That way they get loaded once and only once.  Once the functions and initialization is complete, there is no need to re-source this file.

Hi @dfishburn,
Please not that we have this logic on https://github.com/prettier/vim-prettier/blob/master/plugin/prettier.vim#L14
if exists('g:loaded_prettier')
  finish
endif
let g:loaded_prettier = 1

One thing I am not certain on, when you reference a function in the autoload file, whether Vim automatically sources that file first and then calls it or whether it checks if the function exists and does not source it.
I just that standard patter in all my files.

Hi @dfishburn
Autoloaded function get evaluated only once and are also lazy loaded. This is a good reference: http://learnvimscriptthehardway.stevelosh.com/chapters/53.html
Some quotes from the above link

 :call somefile#Hello()
When you run this command, Vim will behave a bit differently than a normal function call.
If this function has already been loaded, Vim will simply call it normally.
Otherwise Vim will look for a file called autoload/somefile.vim in your ~/.vim directory (and any Pathogen bundles).
If this file exists, Vim will load/source the file. It will then try to call the function normally.

Once again thanks @dfishburn  for raising this issue,
Feel free to comment/re-open this issue if you have any further questions or any info around it

Actually from that web link you provided:
Vim already has a definition for example#Hello, so it doesn't need to reload the file, which means:

The code outside the function wasn't run again.
It didn't pick up the changes to the function.

Now run :call example#BadFunction().

You'll see the loading messages again, as well as an error about a nonexistent function.

But now try running :call example#Hello() again. This time you'll see the updated message!

That is the case I am talking about.
Once you reference a different function in the same file, a function that Vim has not used yet, it sources the file again.
Had you not defined all your functions using function!, you would have seen an error (function already exists) each time you call a different function.
So, for efficiency you do want the finish guard at the top of that file.

Will look into this later and get back to this issue, in the meantime will leave it open until I can can check on it more in depth

Hi @dfishburn,
In order to debug the above statement I have just performed the following test, edited autoload/prettier.vim and included on the top the following line:
echom "loading vim-prettier"

Then I did the following steps:

Opened a new file
Executed :Prettier
Executed :PrettierCliVersion
run the command :messages to check for the printed messages

outcome:

"loading vim-prettier" got printed only once regardless of the multiple calls, so the autoload file got only loaded once

If you can provide a working example of that I will be happy to re-open this issue.


Republish orders automatically

Discussed in https://github.com/lnp2pBot/bot/discussions/462

Originally posted by jonthdiaz January  8, 2024
I would like to know if there is a way to create automatic orders, each day I have to create the orders, it is a bit of a waste of time.

@jonthdiaz here an issue to keep track of it


Add click functionality to interact with DCS clickable cockpits

Add GetDistanceBetweenHandTipAndThumb and IsThumbClick to implement a click (mouse down and mouse up) vs just mouse down so that a mouse click can interact with a DCS clickable cockpit

merge


Update Haskell package set to LTS 16.20 (plus other fixes)

This PR is test-built by Hydra at https://hydra.nixos.org/jobset/nixpkgs/haskell-updates. I'll fix up the remaining errors and merge it on Friday, 2020-10-30 20:00 +02:00. You can watch this live on Twitch at https://www.twitch.tv/peti343. In addition to the chat features offered by Twitch, there is also a voice conference at https://discord.gg/YTEa3XR that viewers can use to chat with me and with each other.

Next week we'll merge https://github.com/NixOS/nixpkgs/pull/102383.


RunUAT failure - Ubuntu 20.04 - UE5 - Size mismatch

I have done builds on fresh multiple machines now, including an AWS EC2 instance, and over the past few months I consistently get this failure on a fresh clone of the ue5-main branch. I really thought it was my environment first, but now having done in 3 separate environments/systems I am convinced there is an issue.
When running ./RunUAT.sh BuildPlugin -Plugin="$CESIUM_FOR_UNREAL_DIR/CesiumForUnreal.uplugin" -Package="$CESIUM_FOR_UNREAL_DIR/../packages/CesiumForUnreal" -CreateSubFolder -TargetPlatforms=Linux
it fails on every file with a size mismatch error
Size mismatch /home/ubuntu/develop/cesium-unreal/extern/cesium-native/extern/KTX-Software/other_lib/mac/Debug/SDL2.framework/Versions/A/Frameworks/hidapi.framework/hidapi = 23 to /home/ubuntu/develop/packages/CesiumForUnreal/HostProject/Plugins/CesiumForUnreal/extern/cesium-native/extern/KTX-Software/other_lib/mac/Debug/SDL2.framework/Versions/A/Frameworks/hidapi.framework/hidapi = 130
When i execute this on my corporate VPN, which is sometimes subject to packet loss and significant latency/jitter I even get a timestamps are not valid (because it took longer than 2 seconds for the copy to complete).
The resolution I have used is to modify the source code of the UE5 AutomationTool Utils so that it does not do any of these file copy verify checks. Primarily by commenting out these lines and making sure Retry is never set to True
https://github.com/EpicGames/UnrealEngine/blob/release/Engine/Source/Programs/AutomationTool/AutomationUtils/Utils.cs#L418-L438
Link to Issue posted on UE5 Forum

This is usually caused by git-lfs not being installed on your system.

Actually, sorry, I'm misremembering. Your problem sounds the same as this one, though:
https://github.com/CesiumGS/cesium-unreal/issues/955#issuecomment-1238924646

Thanks @kring that indeed is the issue.
"the Unreal Build Tool can't cope with the symlinks in the KTX-Software submodule."
After running the cmake build of cesium for unreal. i deleted the cesium-unreal/extern folder and was able to build the plugin with UAT.


Fix mixins

Prefix all mixined methods with createheatjs$
Add MixinExtras to use in the now merged Mixing and Packing category mixins (because the content of the mixin is identical in both we can use the behaviour of targetting multiple classes in one mixin to get both at once!)
Rewrite enum extension code to only allocate a new array once
Move the inject in BasinCategoryMixin to later so that no code needs to be copied (I'm not sure what the point of this mixin is tho, it just stops some additional JEI Ingredients from being added)
Centralize remap = false in annotations to the main Mixin annotation.
Fixes #1

I have tested this and it works as well as it did before the changes (that is, the ice spikes recipe doesn't seem to work when you actually try it, but the diamond one does and JEI is fine).
If you have any questions about why I made any of the changes, or how one of the mixins works now I'm happy to answer them.

Wow, these are great

I have a question to ask you, in Create HeatLevel will be in SEETHING is greater than KINDLED, this judgment is hard-coded to write dead in the enum HeatCondition method testBlazeBurner (), in HeatData in the priority field is to recall to a similar role however, I have not thought of a good solution for the time being. For similar CRYOTHEUM is -1Prior I have not thought of a good solution, so this function has been empty, do you have any good ideas?

The way forge does it for it's tool tier system is it allows you to insert before/after an existing thing rather than using hardcoded indices. So for here you could have a system where you insert CRYTHEOM before KINDLED, and BLAZE after SEETHING. Implementing this is reasonably easy too, you just loop through until you find the target then add the new thing before/after that. (note that this should be maintained in a separate list as you really shouldn't modify the order of an enum)


Cookbook Updated But Not Showing in Supermarket

Glad to see this cookbook adopted and bumped to version 3.0.0 but the supermarket is still showing 2.1.0 as the latest.

The metadata version has been bumped, but the cookbook has not yet been released. In the future we'll avoid dumping the version until we actually do a release since it's a big confusing

https://github.com/sous-chefs/openvpn/milestone/1

Hi @kylegoch we've released v3 of the cookbokok to ship the current set of fixes. I would recommend a higher than normal case of wary-ness as this is a major cookbook version bump


Add query code suggestions

Is your feature request related to a problem? Please describe.
Describe the solution you'd like
Show code suggestions on key down
Describe alternatives you've considered
Additional context

:tada: This issue has been resolved in version 1.7.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Line 1.2.0 Supermarket Release does not compile

resourses/filter_lines.rb looks like it references a class, Line::OS, that does not exist in the release.
The file also does not exist in the 1.2.0 branch's source code.
Here is an output of the chef compile error:
================================================================================
Recipe Compile Error in /var/chef/cache/cookbooks/line/resources/filter_lines.rb
================================================================================

NameError
---------
uninitialized constant Line::OS

Cookbook Trace:
---------------
  /var/chef/cache/cookbooks/line/resources/filter_lines.rb:19:in `class_from_file'

Relevant File Content:
----------------------
/var/chef/cache/cookbooks/line/resources/filter_lines.rb:

 12:  # distributed under the License is distributed on an "AS IS" BASIS,
 13:  # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 14:  # See the License for the specific language governing permissions and
 15:  # limitations under the License.
 16:  #
 17:
 18:  property :backup, [true, false], default: false
 19>> property :eol, String, default: Line::OS.unix? ? "\n" : "\r\n"
 20:  property :filter, [Method, Proc, Array]
 21:  property :filter_args, Array
 22:  property :path, String
 23:
 24:  resource_name :filter_lines
 25:
 26:  action :edit do
 27:    new_resource.sensitive = true unless property_is_set?(:sensitive)
 28:    eol = new_resource.eol

Platform:
---------
x86_64-linux

hey,
apologies about this, we have an incoming 2.0 release that will fix this. In the mean time the best thing to do it to ping to 1.1. Thanks for the heads up!

Closed by https://github.com/sous-chefs/line-cookbook/releases/tag/v2.0.0


1088 curation status

Adds a curation top-level object to AGS and sighting pipeline_status.

Draft while I add a test for sighting status.


Add some things to Settings.settings that had been manually added to Settings.Designer.cs

Settings.Designer.cs should never be edited by hand.
Visual Studio generates it from the file Settings.settings whenever Settings.settings is modified.
Visual Studio provides an editor for Settings.settings, but it's sometimes easier to edit the XML in Settings.settings directly.
If you have a complicated setting value type that cannot be represented by the XML in Settings.settings, then you can add the setting to Settings.cs, but never Settings.Designer.cs.
(Also, removed setting "DetectionsFontSize" since it was not being used, and I could not figure out what default value it should have).

Seems good, you even corrected "detection" to be "detections".


Add teuthology-update-inventory command

This ought to also replace teuthology-updatekeys, but might not be as
fast yet.
Signed-off-by: Zack Cerza<EMAIL_ADDRESS>

Refer to this link for build results (access rights to CI server needed):
http://jenkins.ceph.com//job/teuthology-pull-requests/1361/
Tests passed for this pull request.

Refer to this link for build results (access rights to CI server needed):
http://jenkins.ceph.com//job/teuthology-pull-requests/1364/
Tests passed for this pull request.


Implement opa_abort

What would be an easy .rego to force an abort to happen (unit test)?
@srenatus sorry for bothering you yet again - I asked in https://github.com/open-policy-agent/opa/issues/3083 however I still turned up empty-handed how to nicely force an abort. I don't want to blindly implement the feature without a test.

No worries! Any conflict error would trigger this, like
package p

r { true }
r = false { true }

q { r }

Not near a computer right now, but p/r or p/q should trigger this.

Thanks, this works for opa_abort being called.
Note to self:
string info = _envMemory.ReadNullTerminatedString(_store, addr);
works fine, however,
throw new PolicyEvaluationAbortedException(info);
then leads to

which in turn means my unit test expecting PolicyEvaluationAbortedException craps out:
      Expected: <Opa.Wasm.PolicyEvaluationAbortedException>
      But was:  <Wasmtime.TrapException: abort.rego:4:1: var assignment conflict
    wasm backtrace:
        0: 0xbdb7 - <unknown>!opa_abort
        1:  0x9b7 - <unknown>!opa_runtime_error
        2: 0x129d2 - <unknown>!g0.data.aborttestpkg.r
        3: 0x12a03 - <unknown>!g0.data.aborttestpkg.q
        4:  0x814 - <unknown>!eval

       at Wasmtime.Function.Invoke(StoreContext context, ReadOnlySpan`1 arguments)
       at Wasmtime.Function.Invoke(IStore store, Object[] arguments)
       at Opa.Wasm.OpaPolicy.Policy_eval(Int32 ctxAddr) in D:\GitWorkspace\dotnet-opa-wasm\src\Opa.Wasm\OpaPolicy.Functions.cs:line 72
       at Opa.Wasm.OpaPolicy.ExecuteEvaluate(String json, Nullable`1 entrypoint) in D:\GitWorkspace\dotnet-opa-wasm\src\Opa.Wasm\OpaPolicy.cs:line 275
       at Opa.Wasm.OpaPolicy.Evaluate(String json) in D:\GitWorkspace\dotnet-opa-wasm\src\Opa.Wasm\OpaPolicy.cs:line 225
       at Opa.Wasm.UnitTests.AbortTests.<>c__DisplayClass0_0.<AbortTest>b__0() in D:\GitWorkspace\dotnet-opa-wasm\src\Opa.Wasm.UnitTests\AbortTests.cs:line 18
       at NUnit.Framework.Assert.Throws(IResolveConstraint expression, TestDelegate code, String message, Object[] args)>

@srenatus Now this is interesting - opa_abort is being called first (I removed my throw to test), and then TrapException from the runtime kicks in too. My question then is - what is the use case for opa_abort when the runtime would croak anyways?

There's an unreachable instruction after the call to opa_abort. It's not expected to return.

True, when I remove my throw the TrapException is what you just said:
Wasmtime.TrapException
  HResult=0x80131500
  Message=wasm trap: unreachable
wasm backtrace:
    0: 0xbdbd - <unknown>!opa_abort
    1:  0x9b7 - <unknown>!opa_runtime_error
    2: 0x129d2 - <unknown>!g0.data.aborttestpkg.r
    3: 0x12a03 - <unknown>!g0.data.aborttestpkg.q
    4:  0x814 - <unknown>!eval

... If I remember correctly. Not on a computer, can't check.

My question then is - what is the use case for opa_abort when the runtime would croak anyways?

It conveys more information than just a naked unreachable exception.

Dumping my ideas just to think out loud:

opa_abort gets called. I get the detailed information. If I throw, that detailed information ends up as the message for Wasmtime.TrapException in Policy_eval. Otherwise only the generic info is available there.
So I don't need my custom exception class in opa_abort.
I want the user to see the more detailed information, throw new Exception will do then.

Question is do I want to surface Wasmtime.TrapException? Or wrap it?

I decided to go with Wasmtime.TrapException for the moment.


Fix the way graphics are referenced in this file

@pspinrad

Thanks @lparis -- will cherry-pick to other versions..