Support non-int channel_ids

Description
SpikeInterface allows channel_ids to be of any type, but currently the write_recording function expects int
Steps to Reproduce
import spikeinterface.full as si
from nwb_conversion_tools.utils.spike_interface import write_recording
from datetime import datetime

rec, sort = si.toy_example(num_segments=1)
rec_str = rec.channel_slice(channel_ids=rec.get_channel_ids(),
                            renamed_channel_ids=[str(ch) for ch in rec.get_channel_ids()])
print(rec_str.get_channel_ids())

start_time = datetime.now()
metadata = {"NWBFile": {'session_start_time': start_time}}
write_recording(rec_str, save_path="test_nwb_str.nwb", metadata=metadata)

Error
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
/var/folders/vl/0vmqjsjn0dg8ycnxdx_mqrdw0000gr/T/ipykernel_71428/3145529863.py in <module>
----> 1 write_recording(rec_str, save_path="test_nwb_str1.nwb", metadata=metadata)

~/Documents/Codes/catalyst/nwb-conversion-tools/nwb_conversion_tools/utils/spike_interface.py in write_recording(recording, save_path, overwrite, nwbfile, starting_time, use_times, metadata, write_as, es_key, write_electrical_series, write_scaled, compression, compression_opts, iterator_type, iterator_opts)
    977                 nwbfile = make_nwbfile_from_metadata(metadata=metadata)
    978
--> 979             add_all_to_nwbfile(
    980                 recording=recording,
    981                 nwbfile=nwbfile,

~/Documents/Codes/catalyst/nwb-conversion-tools/nwb_conversion_tools/utils/spike_interface.py in add_all_to_nwbfile(recording, nwbfile, starting_time, use_times, metadata, write_as, es_key, write_electrical_series, write_scaled, compression, compression_opts, iterator_type, iterator_opts)
    825         assert isinstance(nwbfile, pynwb.NWBFile), "'nwbfile' should be of type pynwb.NWBFile"
    826
--> 827     add_electrodes_info(recording=recording, nwbfile=nwbfile, metadata=metadata)
    828
    829     if write_electrical_series:

~/Documents/Codes/catalyst/nwb-conversion-tools/nwb_conversion_tools/utils/spike_interface.py in add_electrodes_info(recording, nwbfile, metadata)
    753     add_devices(nwbfile=nwbfile, metadata=metadata)
    754     add_electrode_groups(recording=recording, nwbfile=nwbfile, metadata=metadata)
--> 755     add_electrodes(recording=recording, nwbfile=nwbfile, metadata=metadata)
    756
    757

~/Documents/Codes/catalyst/nwb-conversion-tools/nwb_conversion_tools/utils/spike_interface.py in add_electrodes(recording, nwbfile, metadata, exclude)
    438                 electrode_kwargs.update(dict(group=nwbfile.electrode_groups[str(group_id)], group_name=str(group_id)))
    439
--> 440             nwbfile.add_electrode(**electrode_kwargs)
    441     # add columns for existing electrodes:
    442     for col_name, cols_args in elec_columns_append.items():

~/anaconda3/envs/nwb2/lib/python3.8/site-packages/hdmf/utils.py in func_call(*args, **kwargs)
    580         if is_method:
    581             def func_call(*args, **kwargs):
--> 582                 pargs = _check_args(args, kwargs)
    583                 return func(args[0], **pargs)
    584         else:

~/anaconda3/envs/nwb2/lib/python3.8/site-packages/hdmf/utils.py in _check_args(args, kwargs)
    573                 if parse_err:
    574                     msg = '%s: %s' % (func.__qualname__, ', '.join(parse_err))
--> 575                     raise ExceptionType(msg)
    576
    577             return parsed['args']

TypeError: NWBFile.add_electrode: incorrect type for 'id' (got 'str_', expected 'int')

Environment
latest versions of NCT, hdmf, SI
Checklist

[ ] Have you ensured the feature or change was not already reported?
[ ] Have you included a brief and descriptive title?
[ ] Have you included a clear description of the problem you are trying to solve?
[ ] Have you included a minimal code snippet that reproduces the issue you are encountering?
[ ] Have you checked our Contributing document?

Yep, we're aware of this. Supposed to be a part of https://github.com/catalystneuro/nwb-conversion-tools/pull/374 but that seems to be dragging a bit. Happy to take an isolated updated for this, then.


Added XCode project file

Added XCode project file so we can view/modify MD within a familiar IDE.  Works in both XCode 8 and XCode 9 beta.  The latter includes a new MD editor.

Good idea. Thanks for the contribution!


Filter gcloud output when checking installed components

When we check whether the required components are installed for gcloud the command returns all components with their corresponding status. We should use the new filter option to be released soon (soon wrt 4/15/2016) to reduce the output from gcloud to the necessary components. Most likely the following command will give the desired result:
gcloud components list --filter=id:app-engine-java --format='value(state.name)'

Is there a way to have gcloud stop checking for updates?  The update messages make parsing difficult…
$ gcloud components list --format=json

Your current Cloud SDK version is: 103.0.0
The latest available version is: 105.0.0

[
  {
    "current_version_string": "2016.03.28",
    "id": "core",
    "is_configuration": false,
    […etc…]
  }
]
To install or remove components at your current SDK version [103.0.0], run:
  $ gcloud components install COMPONENT_ID
  $ gcloud components remove COMPONENT_ID

To update your SDK installation to the latest version [105.0.0], run:
  $ gcloud components update

The version/update related output is in fact written to stderr, so stdout contains only the json formatted result (e.g.  gcloud components list --format=json 2>/dev/null)

custom runtime is gone


Question : Do we have any support to save the HTML source of the page on fail?

Scenario :
I'd like to see the HTML source of the page where the failed test ran on.

Hello, this question is unrelated to Cluecumber.
Cluecumber shows everything that is being attached to scenarios in your test framework (and is thus included in the Cucumber JSON files). So if you attach the page's source, it will show up in the report.

oh neat! thank you! I'll create a hook then. Thank you so much!

@laxersaz  - would you happen to know the best way to embed the String in the report?

In a Cucumber hook you can embed custom data in a scenario.
In Java it would be something like:
scenario.embed(screenshot, "image/png", "My screenshot");

You can also embed text, html or pretty much whatever you want.


Wrapper upgrades PRs doesn't run any tests

See for example this PR:
https://github.com/gradle/wrapper-upgrade-gradle-plugin/pull/188
It checks only the DCO but don't execute other tests.
This is because the PR were created by an GH Action using the build in secrets.GITHUB_TOKEN.
See https://github.com/gradle/wrapper-upgrade-gradle-plugin/blob/fa846e89de0c929df75400d66e42beb2c7d05062/.github/workflows/wrapper-upgrade-execution.yml#L15 and https://github.com/gradle/wrapper-upgrade-gradle-plugin/blob/fa846e89de0c929df75400d66e42beb2c7d05062/.github/workflows/wrapper-upgrade-execution.yml#L37
I guess it make sense that also the bump wrapper PRs run all of the existing tests.
To do this, one option is to provide another secret (from another user) and invite them to this repository as a collaborator with write access. Then using this secret inside the action.
The user will then create the pull requests, instead of the "GitHub Action Bot".
You might want to use your bot-githubaction user for that? 😁

Good catch! I was surprised to learn that and found the formal explanation https://docs.github.com/en/actions/security-guides/automatic-token-authentication#using-the-github_token-in-a-workflow.

When you use the repository's GITHUB_TOKEN to perform tasks, events triggered by the GITHUB_TOKEN, with the exception of workflow_dispatch and repository_dispatch, will not create a new workflow run. This prevents you from accidentally creating recursive workflow runs.

Another option to avoid using another token is to manually trigger the check; I think that's acceptable, given it only happens occasionally.

Another option to avoid using another token is to manually trigger the check; I think that's acceptable, given it only happens occasionally.

So in this case your actions requires an workflow_dispatch with an parameter for the branch to checkout.
And then your team doesn't have to forgot about it to run when there is an wrapper upgrade PR 😅
Sorry but I guess this will not happen 😁
I'm not sure where this PR is comming from.
But it seems that this is working fine with the correct bot-gradle token 🤔

Thanks for the awareness 👍. I'm closing this for now as we prefer not to reference other tokens on this project; we'll see how it goes.


Delete custom GitHub issue templates

Purpose
We've defined generic issue templates for all Ethyca repos here: https://github.com/ethyca/.github/tree/main/.github/ISSUE_TEMPLATE
Removing the templates from this repo allows it to automatically pull in the organization templates for consistency.
Changes

Deletes .github/ISSUE_TEMPLATE

Checklist

~Update CHANGELOG.md file~
[X] Applicable documentation updated (guides, quickstart, postman collections, tutorial, fidesdemo, database diagram.
If docs updated (select one):

[X] documentation complete, or draft/outline provided (tag docs-team to complete/review on this branch)
~documentation issue created (tag docs-team to complete issue separately)~

~Good unit test/integration test coverage~
~This PR contains a DB migration. If checked, the reviewer should confirm with the author that the down_revision correctly references the previous migration before merging~
~The Run Unsafe PR Checks label has been applied, and checks have passed, if this PR touches any external services~

Ticket
n/a

no clue why the build broke on this 🤷


Improve documentation of sidecar.istio.io/inject as a label

I see discrepancies in the documentation on this.
Being referred to as a label
Being referred to as an annotation
Which is it?

https://github.com/istio/api/pull/1997 is work in progress to stop using annotation
Both work, the label is new and better than annotation. The label avoids additional calls to istiod when set to false and allows usage even when the namespace is not explicitly labeled when set to true.

@howardjohn thanks! I figured as much, what with all the good changes Istio has had recently, especially to the docs.
I assume if I'm using 1.10.2 I can use the label without any gotchas, then?

Yep! I slightly repurposed the issue to track improving docs here, as there is some catch-up to do

Ok, thank you very much! You want I should leave it open, then?

sidecar.istio.io/inject as a pod annotation is invalid on istio v1.13.1, label is ok.  Is it a bug? @howardjohn

how to implement this issue?
I found it is AUTO-GENERATED by the istio/api, should we move or copy the defined of sidecar.istio.io/inject from annotations to labels?

Its both an annotation AND a label. Label is preferred


Improve employee request validation

Improve validation for existing employee:
Add new validations (restrictions) in case employee_id exists in Request $.employee_request.employee_id:

[x] Check that party birth_date from request $.employee_request.party.birth_date is equal to birth_date in DB for this employee_id
If not, return 409

{
    "meta": {
        "url": "http://dev.ehealth.world/api/employee_requests",
        "type": "object",
        "request_id": "jdla6e10jrr5q6b0c6464pldvplo9n2u",
        "code": 409
    },
    "error": {
        "type": "request_conflict",
        "message": "birth_date doesn't match"
    }
}

[x] Check that employee start_date from request $.employee_request.start_date is equal to start_date in DB for this employee_id
If not, return 409

{
    "meta": {
        "url": "http://dev.ehealth.world/api/employee_requests",
        "type": "object",
        "request_id": "jdla6e10jrr5q6b0c6464pldvplo9n2u",
        "code": 409
    },
    "error": {
        "type": "request_conflict",
        "message": "start_date doesn't match"
    }
}

Tested


Create Prim.cpp

Created file for Prim's Algorithm in graph that was still missing.

Can anybody tell me , why i am getting 1 check failed - cpplint_modified_files . I have tested my code on so many test cases , checked indentation and much more. But still getting one check failed with exit status 1.

Click the Details button as the twisty next the the ❌ to see...
 1 files contain uppercase characters:
graph/Prim.cpp

Thank you @cclauss for your contribution. Now waiting for merging it

@cclauss can we merge this with master?


Incorrect output for IdleTime

Sometimes I am seeing strange values for IdleTime such as:
IdleTime    : 24692.13:29:00
With the IdleStartTime switch it shows a sensible value so this looks like some parsing issue on the QUser output.  It seems to show these odd values when the sessions has only been idle a very short time.

Can you show me a comparison between quser and Get-Quser? Without knowing what the stdout value is, it's impossible for me to troubleshoot.

OK - battled to get this to repeat - then remembered I was doing it from anotherserver iwhen I saw issue ... for same remote server using quser itself I see this when run from a Windows Server 2008R2 box:
user1              rdp-tcp#1          30  Active          .  12/11/2020 11:23
user2             rdp-tcp#111        42  Active  24692+13:29  16/11/2020 08:58
user3              rdp-tcp#88         55  Active          8  17/11/2020 12:08
Same set of users whne run from Windows Server 2019 box:
user1              rdp-tcp#1          30  Active          1  12/11/2020 11:23
user2             rdp-tcp#111        42  Active          .  16/11/2020 08:58
user3              rdp-tcp#88         55  Active         14  17/11/2020 12:08
So it looks like it is some differnece in how old quser output stuff - could well be a bug in quser!  Becasue user is supposedly active so can't see how they have beeen idle like that - perhaps a bug in old quser if user logged on for days like this.

I appreciate you following up. I agree, it doesn't appear to be an issue with parsing the output.


Removal of "prometheus.io/scrape", etc. from kubernetes example

I am a bit puzzled by this commit:
https://github.com/prometheus/prometheus/commit/03a9e7f72e072c6d29f422425d8acd91a957836b
Is there a reference where this was discussed? Even though this is a convention and not something that needs to be done that way, I fail to see how completely removing this and replacing it with pseudo code is helping users. Even though it wasn't formalized, having a sort-of-agreed-upon way of doing annotations for prometheus in kubernetes is valuable, in my opinion.

@Bplotka : maybe you could comment?

@schweikert sure, can comment. (:
Here is the reference for the discussion and ALL the reasons and pros & cons of that change in examples: https://github.com/prometheus/prometheus/pull/4131
Let us know if something needs to be explained in further details.

I don't know why commit does not reference the PR ):

Thanks for the link to that github issue. That's what I was looking for. I searched for it, but failed to find it :)
Anyway, personally I think that this is a bit of a shame. I think the old suggested way has the advantage of being very simple and I find it to be very useful for most of my use cases. I don't have often the problem of having two containers that need scraping in the same pod. Even for that use case, I was thinking that you could spin up a low-resource consumption sidecar that proxies the request and merges metrics from all the containers that you want scraped. This would have the advantage of keeping the simple use case simple, and making the complex use case possible.

I agree that you can still use annotation if your use case is simple. In fact, what we did in this PR we just tried to make it clear and explain more how to use simple annotations and how it works. We did not remove any example. We just made it clear that this is only an example and not "recommended" and "golden" way for doing this.
The main reason of the change is the fact that people were referencing those annotations as the perfect "way" and that Prometheus recommends it etc etc, while it does not recommend anything in this field - it just gives different ways to handle different use cases.
Do you think that adding more explanation as we did in this change was a wrong idea?

The change removed for example prometheus.io/scrape and replaced with example.io/should_be_scraped. I think that's a bit too drastic and means that all users that want to configure this, need also to think about how to name that annotation. Having a sort-of-standard for that simple use case was useful.

I have added a reply to this google groups thread:
https://groups.google.com/forum/#!topic/prometheus-users/ihMUWtX477Q

This issue probably can be closed, considering that the change was reviewed and agreed on. Move the discussion to prometheus-users. Thanks again Bplotka for your answer. Also, I learned today about how to find a pull request related to a certain github commit (there is actually a link to #4131 when looking at the commit: https://github.com/prometheus/prometheus/commit/03a9e7f72e072c6d29f422425d8acd91a957836b)


Inline comments without tags are not very readable

Hi,
When both jsdocUseInlineCommentForASingleTagBlock  and jsdocExperimentalFormatCommentsWithoutTags are true
Comments without tags are collapsed to a single line if able.
That's ok and more readable for quick type def like /** @type {string} */
But for a simple description, we get something like this:

This feels (to me) much more readable:

Another example with class:

Do you think this is smt that can be changed or added as a setting?

Hey @mech-tools 👋
I'll try and take a look today and see if there's an easy fix, but I won't promise you anything, the jsdocExperimentalFormatCommentsWithoutTags is not something I plan to get out of experimental; it was a request that was easy enough to add, but not a use case I plan to support nor expand on it.

Hi @homer0
Thanks!
Here is my workaround for the time being:
import { get, override } from<EMAIL_ADDRESS>import { loadFns } from<EMAIL_ADDRESS>import { getPlugin } from<EMAIL_ADDRESS>import { getRenderer } from<EMAIL_ADDRESS>import { render } from<EMAIL_ADDRESS>
loadFns();

const customGetRenderer = (options) => {
  const renderer = get(render)(options);
  return (column, block) => {
    const padding = " ".repeat(column + 1);
    const prefix = `${padding}* `;
    const lines = renderer(column, block);

    if (
      lines.length === 1 &&
      block.tags.length > 0 && //<------ Added this line
      options.jsdocUseInlineCommentForASingleTagBlock
    ) {
      return `* ${lines[0]} `;
    }

    const useLines = lines.map((line) => `${prefix}${line}`).join("\n");

    return `*\n${useLines}\n${padding}`;
  };
};

override(getRenderer, customGetRenderer);

export default get(getPlugin)();

hi @mech-tools
You have no idea how happy it made me to see someone actually used the whole setup to extend the plugin xD!!
So, given that you took your time to learn the api, I added something I hope will fit your use case: jsdocExperimentalIgnoreInlineForCommentsWithoutTags.
🤘

Thanks @homer0. Although, now I have no reason anymore to use the plugin setup ^^


Week 2 Individual - Nathan

Total Score: 4.8/5
Individual Score: 2.8/3 ( GRADED BY : Armaan, He also graded Derrick )
Team Score: 2/2
Individual GitHub pages (3 points) - Checkpoint Grading

[x] Organize files, directories and menus for the first 3 weeks.
[x] Write Factorial function using classes, providing implementation of call.
[x] Select your own Math function. Write it in Imperative and OOP form

Tangibles:

LINK for files which has folders for weeks and organization
LINK to factorial.py
LINK to combo.py

Total Score 4.8/5
Derrick you did a great job organizing the files, implementing class and I love that you challenged yourself to doing the extra credit but your code does not show the palindrome being false. So I am giving you some credit on the Extra Credit which is enough to give you a 3/3. The only thing I can recommend is adding more clear comments to what the code is doing as some are long and hard to understand for people that don't know the code.
Team Score was a 1.8/2 with Mr. Mortenson
Individual Score was 3/3


docker image needs to install 'less'

I'm getting this message when running pulsar sql
presto> show schemas in pulsar;
ERROR: failed to open pager: Cannot run program "less": error=2, No such file or directory

I was able to temporarily resolve this by running apt-get update && apt-get install -y less

@mingfang
Can you describe your context in detail?
For example, whether the container environment was used or the local environment, what command was used to start the pulsar sql?

docker run -it apachepulsar/pulsar-all:latest bash
root@333459a84f05:/pulsar# bin/pulsar sql --server http://pulsar-sql.<IP_ADDRESS>.nip.io:80
presto> show schemas in pulsar;
ERROR: failed to open pager: Cannot run program "less": error=2, No such file or directory
       Schema
--------------------
 information_schema
 public/functions
(2 rows)

Query 20190822_042639_00000_sw7tk, FINISHED, 4 nodes
Splits: 53 total, 53 done (100.00%)
0:02 [2 rows, 44B] [1 rows/s, 23B/s]


Support Data Wrangling within Power Query

Support Data Wrangling in new version of ADF when it becomes a new type of object.
https://twitter.com/KromerBigData/status/1352421359482093568

Power Query (Wrangling Data Flow) are still in 'dataflow' folder, so deployment looks like for (mapping) data flows.
Already supported.


Avoid caching original console.error

only cache before switching to a no-op

Using the original console.error can be problematic in codebases with a modified console.error. e.g. spied console.error for test assertions

This is an issue in our codebase. We spy on console.error at the start of each test and assert 0 calls at the end of the tests. Going through this code path removes this functionality because it is restoring it to the original console.error reference.

This is great! Can you rebase so I can merge?

How about a regular merge commit of latest master? Will rebase if that is preferable.

I would prefer a rebase. Otherwise I can just squash your commits with merge, but then it appends me to the commit history. Would rather give you full credit

Done and done. Appreciate the quick turnaround!


Cannot parse ISO8601 time-only string

Given a string, "09:00:00", the ISO8601Parser fails to parse it, and returns 00:00:00 as the time.

Thank you, fixed in https://github.com/malcommac/SwiftDate/commit/2699a1357e6f06da7aabd4dcb94fcc97bb05a4b2.


Design the Formal Language Grammar of COOL in BNF.

Consult the COOL Reference Manual.
Formulate COOL Grammar as a Context-Free Grammar (CFG) in Backus–Naur Form (BNF).
Write the CFG in a markdown document.
Write the CFG to be parsed by an LALR parser.

Context-Free Grammar in Backus-Naur Form was written in /docs/CFG.md.


Why not keep the same key combination to remove/delete logs as turboConsoleLog?

I recently discovered your extension and think it's fantastic! I have a suggestion: why not align with turboConsoleLog and set ctrl+alt+d / ctrl+opt+d as the default shortcut for deleting debug messages?
I find ctrl+alt+e a bit confusing. With the Salesforce extension pack, the suggested key combination doesn't conflict with any other shortcuts.
I managed to change it myself by overwriting the command turbo-apex-debug.removeAll. If you'd like, I can create a PR to add this configuration to the README. It would be great to have this as the default setting!
Thanks for creating this extension!

Hey @SotoAugusto, thanks a lot for the feedback! I'm glad I could simplify the process for someone, even just a little.
Yes, I completely agree with you. Changing the shortcut to the default one used by turbo-console-log makes it more consistent. I just updated the README and the extension – now that's the default shortcut. Thanks again! ❤


Capy retrieves the wrong picture to show in the articles feed for this RSS feed

Steps to Reproduce
RSS feed in question:
https://prensasincensura.com/feed/
Expected behavior
When navigating the articles feed, where it shows the title with a short description and an image of the article, this RSS feed always shows an image of the author of the article instead of an image of the article's topic.
Screenshots or screen recording

Device, Operating system, and app version
Google Pixel 9 Pro, Android 15, Capy Reader 2024.11.1075-dev

Hey @cartomichelines. Thanks for the bug report. I think I have a fix for this to exclude author images. I'll include it in the next build. Here's a preview of what that looks like

A fix for this is live in 2024.11.1076-dev


Refactor rdkit.ML.Cluster.Butina to use numpy to perform on large datasets.

Reference Issue
Fixes #6166
What does this implement/fix? Explain your changes.
I attempted to use the existing Butina logic to cluster 99k molecules, and it eventually runs out of memory on a 16GB of RAM machine.
With these changes it runs successfully on a dataset of that size.
Luckily this is <60 lines of pure Python that don't use anything else in the system.  It passes your unit test, and also has produced identical results to the old code on our internal datasets.
Hope this is useful to others.
This replaces the list-of-lists with an numpy.array(dtype=bool) to track which pairs of molecules pass the threshold, which is smaller.  It creates the list empty, but full-size at the beginning and the loops of code flip the bits instead of grow the list.

Thank you for this. What is the current status of it?

@sakoht : any plans to take this up again?

@greglandrum how about refactoring this to use SciPy sparse array? It would solve both problems, since sparse arrays allocate memory only for non-zero elements. It would introduce the SciPy as a dependency, although many projects utilizing RDKit in Python already use it as e.g. scikit-learn dependency.

@sakoht : any plans to take this up again?

@greglandrum, @ilkersigirci Sorry I didn't see your messages way back last Summer.  I meant to circle back and make the requested changes to support both paths of logic and repo standards, but have been swamped with work, so we just used my branch.
Glad to go through and update things now that I have a bit more time.

@greglandrum how about refactoring this to use SciPy sparse array? It would work in both cases, since sparse arrays allocate memory only for non-zero elements. It would introduce the SciPy as a dependency, although many projects utilizing RDKit in Python already use it as e.g. scikit-learn dependency.

@greglandrum that sounds like a clever idea.  Since we would be letting the caller specify sparse/dense flag and going through different code paths, this makes the sparse code path simpler, and a shared code path will be less likely to diverge.
I think the key question would be what the project maintainers would prefer?

@sakoht, I would prefer to get this one merged and then discuss adding an optional path through the code for sparse arrays. I don't want to add a hard scipy dependency, but we could definitely have it be optional.


changed second check for total > 0

Why double check for total > 0?

Thanks for the PR, but I prefer the code the way that it currently is.


Some project js files have no error prompts, and some project js files have error prompts

projects A:

projects B:

In the past, these projects all had error prompts, but there were no error prompts for some projects recently.

We closed this issue because it is a question about using VS Code rather than an issue or feature request. Please search for help on StackOverflow, where the community has already answered thousands of similar questions. You may find their guide on asking a new question helpful if your question has not already been asked. See also our issue reporting guidelines.
Happy Coding!


Retry & exponential backoff retry for batchPut doesn't work for ProvisionedThroughputExceededException

Retry & exponential backoff retry for batchPut doesn't work for ProvisionedThroughputExceededException. We have to handle manually if ProvisionedThroughputExceededException is thrown.
I am curious to know if there's any plan to include that in the code itself or we should continue handling it all by ourselves.

@infigenie I can't speak to the plan here (full disclosure: I am an Amazon employee but just another user of this library, not a maintainer), but just a note that what you bring up here sounds like the described/spec'd behavior from the readme to my reading:

Partial successes (i.e., BatchGetItem operations that return some responses and some unprocessed keys or BatchWriteItem operations that return some unprocessed items) will retry the unprocessed items automatically using exponential backoff.

I'm guessing you're referring to ProvisionedThroughputExceededException which fails a whole batch operation. This behavior is rooted in the AWS DynamoDB API and client (e.g. see the batchWriteItem docs) where partial batch failures are expressed via an UnprocessedItems field in the response and full batch failures are expressed via ProvisionedThroughputExceededException.
My guess is the idea here is to behave similarly to the underlying client and that users of the library need to handle retries in this case. I don't know where this is more intentional or incidental to the implementation, but it seems to be the requirement at this point at least.

I'm also experiencing the same thing (and what @rationull says makes sense to me), and would expect DataMapper to handle retries for throughput exceptions as well.  Having to handle retries/backoff myself for this exception misses the whole point of library retries/backoff.


download dataset

Many thanks for providing this package.
When I run this code, the dataset is not to be downloaded.
These two ULR cannot be accessed now as follows:
#self.url = "http://<IP_ADDRESS>" #旧服务器
self.url = "http://<IP_ADDRESS>:9000/" #新服务器
Could you provide another URL for dataset download? or other methods to get the dataset.
Miao

Hi @MiaoHu-Pro, thanks for your attention. We'll provide a new URL soon.

Hi @MiaoHu-Pro, thanks for your attention. We'll provide a new URL soon.

@jinzhuoran  Thanks for your reply.  Please provide the new URL as soon as possible.

Dataset can be downloaded in Baidu Pan (key: p986)

Many thanks for providing the dataset.

@MiaoHu-Pro Can you share the datasets please ? I cannot find it
Thank you
Best
Gaoussou


cs: adds logging for the iteration where the rebalance was a no-op

Changes proposed in this PR

log when the rebalance was a no-op
minor refactoring to sort keys and remove unused values

Why are we making these changes?
Helps us quickly identify why a rebalance iteration was a no-op.

@dposada build is green, can we merge?

I am good with merging. Should we wait for @pschorf to approve?


x/net/http2/h2c: H2C handler uses incorrect headers to detect upgrade

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

$ go version
go version go1.16.2 linux/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/pivotal/.cache/go-build"
GOENV="/home/pivotal/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/home/pivotal/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/pivotal/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GOVCS=""
GOVERSION="go1.16.2"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/dev/null"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build592242489=/tmp/go-build -gno-record-gcc-switches"

What did you do?
Running a simple h2c server:
package main

import (
        "fmt"
        "net/http"

        "golang.org/x/net/http2"
        "golang.org/x/net/http2/h2c"
)

func main() {
        h2s := &http2.Server{}

        handler := http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
                fmt.Fprintf(w, "Hello, %v; HTTP Version: %v", r.URL.Path, r.Proto)
        })

        server := &http.Server{
                Addr:    "<IP_ADDRESS>:8080",
                Handler: h2c.NewHandler(handler, h2s),
        }

        err := server.ListenAndServe()
        if err != nil {
                panic(err)
        }
}

What did you expect to see?
Based on my reading of RFC 7540, Section 3.2, the required headers for an h2c upgrade are Upgrade: h2c and HTTP2-Settings:

The client does so by making an HTTP/1.1 request that includes an Upgrade header field with the "h2c" token.  Such an HTTP/1.1 request MUST include exactly one HTTP2-Settings (Section 3.2.1) header field.

Curling with those headers, I would expect to receive a HTTP/1.1 101 Switching Protocols response.
What did you see instead?
The h2c handler does not accept the upgrade:
$ curl localhost:8080 -H "Upgrade: h2c" -H "HTTP2-Settings: AAMAAABkAARAAAAAAAIAAAAA" -vvv
*   Trying <IP_ADDRESS>:8080...
* TCP_NODELAY set
* Connected to localhost (<IP_ADDRESS>) port 8080 (#0)
> GET / HTTP/1.1
> Host: localhost:8080
> User-Agent: curl/7.68.0
> Accept: */*
> Upgrade: h2c
> HTTP2-Settings: AAMAAABkAARAAAAAAAIAAAAA
>
* Mark bundle as not supporting multiuse
< HTTP/1.1 200 OK
< Date: Mon, 26 Apr 2021 19:09:10 GMT
< Content-Length: 32
< Content-Type: text/plain; charset=utf-8
<
* Connection #0 to host localhost left intact
Hello, /; HTTP Version: HTTP/1.1

This is because the h2c package is not checking the appropriate headers:
// isH2CUpgrade returns true if the header properly request an upgrade to h2c
// as specified by Section 3.2.
func isH2CUpgrade(h http.Header) bool {
	return httpguts.HeaderValuesContainsToken(h[textproto.CanonicalMIMEHeaderKey("Upgrade")], "h2c") &&
		httpguts.HeaderValuesContainsToken(h[textproto.CanonicalMIMEHeaderKey("Connection")], "HTTP2-Settings")
}

Source: https://github.com/golang/net/blob/5f58ad60dda6b6eba34c424201d17c9fdc37953d/http2/h2c/h2c.go#L373-L378
While the Connection: HTTP2-Settings header does appear in the example in Section 3.2, I can't find anything in the spec that says it is required or that it should be used to detect h2c upgrades.
After adding the Connection: HTTP2-Settings header, the h2c upgrade succeeds:
$ curl localhost:8080 -H "Upgrade: h2c" -H "HTTP2-Settings: AAMAAABkAARAAAAAAAIAAAAA" -H "Connection: HTTP2-Settings" -vvv
*   Trying <IP_ADDRESS>:8080...
* TCP_NODELAY set
* Connected to localhost (<IP_ADDRESS>) port 8080 (#0)
> GET / HTTP/1.1
> Host: localhost:8080
> User-Agent: curl/7.68.0
> Accept: */*
> Upgrade: h2c
> HTTP2-Settings: AAMAAABkAARAAAAAAAIAAAAA
> Connection: HTTP2-Settings
>
* Mark bundle as not supporting multiuse
< HTTP/1.1 101 Switching Protocols
< Connection: Upgrade
< Upgrade: h2c

This is causing problems because we are running our H2C apps behind a httputil.ReverseProxy, which removes Connection headers, thereby blocking the h2c upgrade.

cc @bradfitz @tombergan

My colleague, @ameowlia, pointed out that Section 3.2.1 of the spec does say that Connection: HTTP2-Settings should be sent by the client:

Since the upgrade is only intended to apply to the immediate connection, a client sending the HTTP2-Settings header field MUST also send "HTTP2-Settings" as a connection option in the Connection header field to prevent it from being forwarded (see Section 6.1 of [RFC7230]).

Still, this feels like an odd choice for detecting h2c upgrades, especially in lieu of checking for the HTTP2-Settings header itself. Furthermore, this check isn't compatible with Golang's ReverseProxy implementation, as I described above.
For comparison, I testing the same request against a simple Python server that supports h2c:
#!/usr/bin/env python3

from quart import Quart, request

app = Quart(__name__)

@app.route("/")
async def hello():
    return f"Hello, {request.full_path}; HTTP Version: {request.http_version}."

if __name__ == "__main__":
    app.run(port=8081)

It successfully upgraded without the Connection: HTTP2-Settings header:
$ curl localhost:8081 -H "Upgrade: h2c" -H "HTTP2-Settings: AAMAAABkAARAAAAAAAIAAAAA" -vvv
*   Trying <IP_ADDRESS>:8081...
* TCP_NODELAY set
* Connected to localhost (<IP_ADDRESS>) port 8081 (#0)
> GET / HTTP/1.1
> Host: localhost:8081
> User-Agent: curl/7.68.0
> Accept: */*
> Upgrade: h2c
> HTTP2-Settings: AAMAAABkAARAAAAAAAIAAAAA
>
* Mark bundle as not supporting multiuse
< HTTP/1.1 101
< date: Mon, 26 Apr 2021 20:54:51 GMT
< server: hypercorn-h11
< connection: upgrade
< upgrade: h2c

cc @fraenkel

I don't believe this is an issue given that this is following the spec as identified above.
From https://datatracker.ietf.org/doc/html/rfc7540#section-3.2.1
A request that upgrades from HTTP/1.1 to HTTP/2 MUST include exactly
one "HTTP2-Settings" header field.

 a client sending the HTTP2-Settings header field MUST
   also send "HTTP2-Settings" as a connection option in the Connection
   header field

Based on that, should httputil.ReverseProxy preserve the Connection: HTTP2-Settings header? There is already precedent for preserving headers needed for upgrades: https://github.com/golang/go/blob/ce92a2023ccd77ca609126aa8a6e881c9def57f0/src/net/http/httputil/reverseproxy.go#L272-L277
Otherwise, it seems impossible to implement the H2C upgrade flow spanning a ReverseProxy.

See https://blog.assetnote.io/2021/03/18/h2c-smuggling/

I see why it makes sense to strip HTTP2-Settings from the Connection header as a default.
Would you be open to making the preserved headers configurable on ReverseProxy? In our case, we have a trusted client and also don't enforce any access controls via our reverse proxy, so I don't think h2c smuggling is a concern. If not, I suppose we can always shim the ReverseProxy's Transport to add the headers back.

Closing as this appears to be working as intended.
See #53002 for the new design for ReverseProxy that can handle hop-by-hop headers


cimg,gmic,gmic-qt: 3.2.1 -> 3.2.2

Description of changes
I updated the three G'MIC packages (which required one upstream PR to fix gmic-qt) and made a few drive-by fixes
Changelog: https://discuss.pixls.us/t/on-the-road-to-3-3/34786 (notably includes a critical bug fix to enable mutexes for the command line tool when using parallel computing)
Drive-bys:

Fix repo owner for cimg and gmic, since the repository was moved a while ago to GreycLab
Remove unused args from packages
Make gmic-qt use stdenv.mkDerivation instead of qt5.mkDerivation (see #180841, and this would also be a requirement to building with Qt6)

Upstream PR to fix CMakeLists: c-koi/gmic-qt#176
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] 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

[x] 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/)
23.05 Release Notes (or backporting 22.11 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.

nixpkgs-review
Result of nixpkgs-review run on x86_64-linux 1

10 packages built:

cimg
darktable
gimp-with-plugins
gimpPlugins.gmic
gmic
gmic-qt
imgcat
openroad
phash
photoprism

Also, given that both gmic and gmic-qt have no listed maintainers, should I just go ahead and add myself? I'm happy to adopt them (should I add myself to cimg too since the three are tightly coupled?)

I made a small license fix and added myself as maintainer to all three. Let me know if this is problem

Can we backport this to 22.11 since the update includes a critical bug fix?

Actually I just noticed 22.11 is still on G'MIC 3.1 (and so 3.1 -> 3.2 would be a bigger upgrade than 3.2.1 -> 3.2.2). I don't see any specific mentions of breaking or incompatible changes at https://discuss.pixls.us/t/release-of-gmic-3-2/30567, though, so I think it can still be a candidate for backporting
I'll make a manual backport shortly

So this might not be backportable. gmic-qt-krita was removed upstream in 3.2.0 so this actually would be a breaking change against 22.11
Do you think you could just use gmic from nixos-unstable until 23.05 @Zahrun?

Sure, no problem, thank you!
Let's wait for 23.05. I guess if anyone faces the crash, they will find this related issues.
Fetching from unstable works well.


QA changes for summary list update

https://trello.com/c/CXGExCLO/55-3-replace-summary-table-with-summary-list-on-g-cloud-service-page-and-dos-opportunity-page

Adjust the margin on the Question and Answer session link on Brief pages.

BEFORE

AFTER

Adds a "not empty" check to Brief pages to match Service pages and not display a heading for an empty section.
Bumps Content Loader.
Shows empty attributes on Brief pages (but not on Service pages) - this is to conform with previous behaviour.

To do

Test to make sure <h2> don't display above sections if they are empty.
Test to make sure empty questions are displayed on Brief pages but not on Service pages

Looks like we need to put the question and answers in the HTML key of the summary list, I've found an example of a Q&A where the question includes HTML.

http://localhost/digital-outcomes-and-specialists/opportunities/12024


Specification + Zig Code is not quite enough to reimplement in C

For the record, I just wrote 4500 lines of C++ code to parse SVGs using a peephole parser and a 64 byte capture window. It parses top down, rendering and tossing as it goes. It uses my graphics library's new vector drawing canvas.
With that said, I think it's fair to say I'm pretty good at reading specs and code if I managed that.
However, try as I might, I cannot produce an implementation in C++ based on your documentation and SDK code.
The Zig is I'm assuming complete and functional. The problem is most of the pertinent stuff is buried in layers of local functions and other things that make the code less clear than it maybe could be,
Ideally, your reference implementation would be in C. That would fulfill a much wider audience than Zig, which is a boutique language.
Now, I'm not sure where to go from here. I was planning on helping you integrate what SVGO does into your own converter app, but I can't do that if I don't understand the TVG binary format. (I'm not bothering with the text format because I don't need it, but understanding the binary format fully will give me a complete picture of TinyVG's capabilities)
Do you know C++? (no STL, it's very Cish code) - if so would you be willing to review some parsing code to tell me where I went wrong reading your specs?
I'm sorry but they're just not clear enough, and neither is the code (at least for a non-zig coder, like most coders)
If I can get it right, I'll give you a C or C++ reference implementation.

nevermind I think I'm past the hump. I'll post a reference implementation when it's complete.

Could you please write which parts you had problems with understanding?

I misinterpreted the sizes array that comes with the path segments. It was my fault. I missed the part about each count being +1, and was only doing that for the overall count itself, not the subcounts, leading my whole binary mess to be off. Naturally, I didn't figure that out despite banging on it until after I posted this.
I did run into another issue. You do even-odds for your overlap on your rendering. My canvas control can set even-odd vs non-zero but not for overlapping regions. It follows the behavior of SVG more or less. Unfortunately, it's based on plutovg and I don't know enough about the rasterization code to modify it to accept an even-odd overlap handling scenario.
Therefore, I've had to scrap TVG support for now. I do however, have it 80% implemented in C++ using my vector canvas, it just doesn't render a lot of things correctly due to the above.
I am currently working on a major update of my graphics library, but I may be able to squeeze in time to do a reference implementation at least of the parsing, in C.

I should add, the primary help I've found from the TVGT format is debugging my my TVG binary handling code!

I did run into another issue. You do even-odds for your overlap on your rendering. My canvas control can set even-odd vs non-zero but not for overlapping regions. It follows the behavior of SVG more or less. Unfortunately, it's based on plutovg and I don't know enough about the rasterization code to modify it to accept an even-odd overlap handling scenario.

That should be standard behaviour of SVG tho. Self-overlapping paths do cancel themselves out and a SVG renderer should be able to handle this.

If that's the case then something else is going on that I do not understand.
Chart.svg
This renders correctly for me in SVG, but when I render the TVG version, the inside of the Os, and the B are filled in, like it's not chopping out inner sections. I'd show you a render, but I'm busy tearing apart my rasterizer in order to optimize it.
Your pathing in the final TVG is much different than in the SVG from what I can tell so I can't compare my debug sessions, but I'm kind of relying on the fact that I wrote both the SVG and TVG code back to back so they're pretty similar, given the different formats involved.
So with all that said, I am at a loss.

i'll try taking a look soon. feel free to join the Discord channel for further debugging :)


Multi color ascii

Implemented multi color ascii.
To color ascii just add {n} to the ascii to change every character after it to that color. N can be any single digit number, so only up to 10 different colors per ascii.
I only changed the C and rust logo to multiple colors so far, but the others still work even if they don't have the annotations.

Impressive work @xynxynxyn
We are getting very close to what Neofetch can do...in Rust.
Thanks again


target origin does not match with recipient window's origin

Failed to execute 'postMessage' on 'DOMWindow': The target origin provided ('https://www.youtube.com') does not match the recipient window's origin ('http://mediaelementjs.com').
i am using mediaelement player and got the same error. So just posting here if you people give some solution. I tried several ways but no success.
Any idea how to resolve this issue ?

@debasispanda Seems like there is an issue related HTTP and HTTPS and asynchronous requests; it seems intermittent and doesn't affect the player at all (https://help.clipchamp.com/hc/en-us/articles/216329537--API-How-do-I-fix-the-Failed-to-execute-postMessage-on-DOMWindow-error-)


Add issue templates to the repo

In sir-lancebot, there are currently issue templates for ease of use, and uniform issues. I suggest we either move the sir lancebot issue templates to the repo or create new ones.

I would like this too, since there have been many times I've had to go copy it from the Sir Lancebot repository. This would make it easier.

Duplicate of python-discord/organisation#268

When is that?

There is currently nothing scheduled. @Xithrius, @Akarys42 thoughts?

Since I do not have access to that issue, im assuming that it is for issue templates across repos

I don’t think we reached a consensus on this, although here are my thoughts:
I’m not sure if locking ourselves in with a template issue is a good idea. I think having this freedom of structuring it as we want makes us more efficient and we can assume most of our users know how to write a good issue. It feels also quite bureaucratic.
It isn’t mentioned in the issue but a contrario I’d say copying the PR template over would be a good idea. You don’t need much freeway when writing PRs and it can force everyone to structure it well.

I'd be fine with just going with the PR template, but I think at least having templates would be a good idea. We could provide a blank template for people to use as well.

Whats the status on this?

We're probably not going to add templates, as this repository is meant for more advanced users after coming from sir-lancebot.
Along with that, I agree that there should be no need for a format of PRs in terms of initial comments.

I've seen it brought up a few times now, and I disagree with the notion that "advanced" users don't find templates helpful.
Templates have a few benefits:

They guide beginners about what they should add,
but 2. They also guide any user about what they should add.

You can know what you should add, but still forget something. I also personally find it easier to give the required details, when it's more of a questionnaire than an open-ended response. I'd personally like to see issue templates in this repo as well.
I don't have as strong an opinion about PR templates.

Should we have issue templates that are more simplified than sir-lance's?

There are a few things in lance's template that I would like changed, but besides those, I think having the templates be identical would be the easiest choice, both in terms of maintenance and user-friendliness.

After giving it some more thought, I think both the issue and PR templates should be copied from the sir-lancebot repository and put into this one.
Besides that, they should not be enforced as not all PRs require the templates that we will be providing.

No problems here, would love to have them. 👍

The format of the templates was discussed a long time ago in python-discord/organisation#268 but never actioned. I don't think we really came to a conclusion. I personally think some templates need changes. We should also invest in https://docs.github.com/en/communities/using-templates-to-encourage-useful-issues-and-pull-requests/configuring-issue-templates-for-your-repository as Chris mentioned in that issue.

Whats the status of this?

TBH We can have issue templates and still allow a user to not use the templates, so its very easy to have the best of both worlds.
https://docs.github.com/en/communities/using-templates-to-encourage-useful-issues-and-pull-requests/configuring-issue-templates-for-your-repository#configuring-the-template-chooser

You can encourage contributors to use issue templates by setting blank_issues_enabled to false. If you set blank_issues_enabled to true, people will have the option to open a blank issue.


Can't link library using absl::string_view in api compiled with c++11 against program using this api compiled with c++17

I'm trying to link a program compiled with c++17 against a library compiled with c++11 that has absl::string_view in its api. Because my program gets std::string_view and the library has absl::string_view, they are binary incompatible and this can't be done.
It would be nice if I could specify using absl::string_view over std::string_view even when std::string_view is available so I can use both this library and c++17.
can ABSL_HAVE_STD_STRING_VIEW be made configurable?

If I understand what you're trying to do correctly, we don't support depending on a compiled representation of Abseil. From https://abseil.io/about/compatibility:
"""
Do not depend on a compiled representation of Abseil. We do not promise any ABI compatibility — we intend for Abseil to be built from source, hopefully from head. The internal layout of our types may change at any point, without notice. Building Abseil in the presence of different C++ standard library types may change Abseil types, especially for pre-adopted types (string_view, variant, etc) — these will become typedefs and their ABI will change accordingly.
"""
Is compiling both libraries at the same time, with the same compiler options, a feasible workaround?

I think I might call this a bug in TensorFlow.  Binary distributions that include Abseil should either use an LTS branch of Abseil, in which case the inline namespaces that we include would prevent these ODR violations, or they should hide all Abseil symbols from the linker.  Either way, TensorFlow could package their library in such a way that this wouldn't be a problem.

Ah I see.
I can't do that because recompiling the third-party library (tensorflow) is infeasible.
The workaround I'm using currently is to do:
#include "absl/base/config.h"
#undef ABSL_HAVE_STD_STRING_VIEW

before including the tensorflow headers, which is not great.
Feel free to close this issue.

For others stumbling over this issue, the following is still a valid workaround for me:
#include "absl/base/config.h"
#undef ABSL_HAVE_STD_STRING_VIEW
#undef ABSL_USES_STD_STRING_VIEW

For others stumbling over this issue, the following is still a valid workaround for me:
#include "absl/base/config.h"
#undef ABSL_HAVE_STD_STRING_VIEW
#undef ABSL_USES_STD_STRING_VIEW

Don't do that, it is going to be brittle. The supported solution is to edit options.h.
https://github.com/abseil/abseil-cpp/blob/48f72c227b94b06387106f71d4450b31e88e283b/absl/base/options.h#L133-L157
See also:
https://abseil.io/docs/cpp/guides/options
https://abseil.io/blog/201901115-options

So there is no safe way to do that from "user space" without editing the lib?


Play command

Error whilst ytsrSyntaxError: Unexpected token < in JSON at position 0

This is a known problem with ytsr.
For a hotfix you can use my own fork of the module: npm i https://github.com/MatteZ02/node-ytsr
I will update the module when the fix is stable.


Mouse Unusable (e.g., Logitech MX Master 3)，When using a combined keyboard and mouse receiver。

When using a combined keyboard and mouse receiver, the mouse (e.g., Logitech MX Master 3) is not functioning properly. Specifically, the mouse does not work at all. We would like to know if this issue is specific to the Logitech MX Master 3 or if other Logitech mice also experience similar problems.

Yes, I tried version pre-1.3. The issue still persists with this version.
Thank you for your help!

Which color does the logo on the USB receiver have?

I have MX Master 3S mouse and MX Mechanical Mini plugged through combined keyboard and mouse receiver (bolt, light green logo) without any issues on version pre-1.3

Can you hook up a serial debug cable to GPIO0 and post the output?

The receiver is black, and the logo on it is "logi" in lowercase letters, not the "G" logo.
I tested with the latest version today, and the issue still persists. The mouse is not recognized at all; the cursor doesn't even move.
However, the Logitech receiver with the "G" logo works fine.
Thank you for your help!

Can you hook up a serial debug cable to GPIO0 and post the output?

Ah I've got a mouse now so I can reproduce this, I'll get back to you!

Thank you very much! I am waiting for your response online.

It seems it depends on whether the receiver is connected to a hub or directly to the pico 🤔

I usually connect it to a hub, but it doesn't work even when I connect it directly during testing. Also, we need to use a hub to connect both the keyboard and the mouse.
Is there any good solution or workaround for this issue?

I think I know where the issue is, its in the mouse report initialization, I'll get back to you!

Should be fixed in the latest pre-release: https://github.com/No0ne/ps2x2pico/releases/tag/pre-1.3
If not please leave a comment.


ISPN-6412 Make RESTReplicationIT stable

https://issues.jboss.org/browse/ISPN-6412
As @danberindei suggested here I added clean up code to all places where RESTHelper had been used.

Needs rebasing

Otherwise the PR looks good

ExpirationIT and ExampleConfigsIT fail with "java.lang.Exception: Method clearServers() should be static"

Rebased and comments addressed

Rebased

looks good to me. @galderz @danberindei any other comments?

No more comments @pruivo

this was already integrated. closing it. thanks @slaskawi !


[SPARK-26832][SQL][PYTHON] Avoid project creation per record at Python's grouped vectorized UDF

What changes were proposed in this pull request?
Looks the projection is being created per each output record in Python's grouped vectorized function.
How was this patch tested?
Existing tests should cover.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/7726/
Test PASSed.

Test build #102059 has started for PR 23739 at commit b9eed23.

Test build #102059 has finished for PR 23739 at commit b9eed23.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102059/
Test FAILed.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/7727/
Test PASSed.

Test build #102060 has started for PR 23739 at commit b9eed23.

Test build #102060 has finished for PR 23739 at commit b9eed23.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102060/
Test FAILed.

retest this please

Test build #102062 has started for PR 23739 at commit b9eed23.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/7729/
Test PASSed.

Test build #102062 has finished for PR 23739 at commit b9eed23.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102062/
Test PASSed.

Thanks @HeartSaVioR . Adding @BryanCutler and @icexelloss as well.

D'oh, sorry guys, I updated the JIRA and PR. I misread it.

Hm, let me just leave this closed. Minor style fixes are not quite encouraged anyway. I might likely touch this code soon since I am working on vectorized R native udfs (gapply and dapply). Let me fix it if I happen to touch this code later.

BTW, thanks for reviewing this guys.


Updated clipboard vendor to add wayland support

gopass -c was broken on wayland. By updating the clipboard vendor package, this issue is solved.
I would appreciate, if you could increment the minor release version and publish a new release, so that this get's fixed upstream by the major linux distributions. Thanks!

Codecov Report

Merging #1118 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1118   +/-   ##
=======================================
  Coverage   62.57%   62.57%
=======================================
  Files         184      184
  Lines       10239    10239
=======================================
  Hits         6407     6407
  Misses       3033     3033
  Partials      799      799

Continue to review full report at Codecov.

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

Thanks! Will try to do a new release soon.

Ping for a release, please and thank you!

Done

Seems to me it's back again. --clip option doesn't work


Create animals table

In this project I create a relational database named vet_clinic. I have accomplished the following:

Create table named animals with the following columns:
id, name, date_of_birth, escape_attempts, neutered, weight_kg.
Insert animal data into the table.
Write queries to get requested data.

Having done this, I have copied the SQL into the appropriate files.
Here are the screenshots for the queries:

Find all animals whose name ends in "mon".

List the name of all animals born between 2016 and 2019.

List the name of all animals that are neutered and have less than 3 escape attempts.

List the date of birth of all animals named either "Agumon" or "Pikachu".

List name and escape attempts of animals that weigh more than 10.5kg

Find all animals that are neutered.

Find all animals not named Gabumon.

Find all animals with a weight between 10.4kg and 17.3kg (including the animals with the weights that equals precisely 10.4kg or 17.3kg)

Thank you for your time


sh: clean up some issues caught by shellcheck

The most important bits here are quoting the $@ invocations, especially in vg
globalExec; without this, trying to pass arguments containing spaces to the
program exec'd by vg globalExec will fail.  Some other instances are helpful if
your home directory has a space in its path. Some of the other quotes are
perhaps extraneous (eg, with echo), but it's a good habit to be in anyway.

Note: I don't know much about zsh and haven't tested there.

Good finds. Hadn't heard about shellcheck before but installed it right away, seems to give good suggestions.

BTW, as far as dropping the eval in globalExec: keeping it in seemed to break vg globalExec echo 'foo           bar' and I think it still allows for most of the things you'd expect from an eval (eg you can globalExec a cd and it works), but I'm not an expert.


Support for \langle and \rangle

The is a feature request to add \langle and \rangle to the auto-completion list.
Additionally, I'd like \langle to auto-complete with \rangle (the same behavior in parentheses and brackets)
\langle is usefull and replaces \left< when you don't want latex to resize depending on what is inside the brackets.

Good call :)

Oh wow. accidently closed the issue. ^^


[Feature request] Display full text on hover

🐣 Is your feature request related to a problem? Please describe.
Right now when the text is too long, it will just get cropped, with no way to see the whole string (besides inspecting the element).

💡 Describe the solution you'd like
Displaying the full string when hovering mouse over the cropped text
❓ Describe alternatives you've considered
Opening the full text in a cute dialog might also be an option :)

Implemented. Will be a part of the next release :)

I do not think this issue is fixed in 3.1.1. I hover over the trimmed text but noting happens

It should show up after a short delay

It does not work :))))
I have now tried firefox and brave (based on chromium)


ESP-IDF release ZIP file has the .git folder in it.

Is there a reason to include the .git folder in the ZIP files from the Releases page?
This adds about 200MB worth of compressed data that no one really needs if they're using the release as a point-in-time snapshot anyways.

Hi @nuket, we do keep .git directory in those archives on purpose. The intention is that users still use git to update IDF or switch between versions. Packaging the entire working copy into a release archive is a workaround for users who can't perform git clone --recursive https://github.com/espressif/esp-idf.git due to unreliable Internet connectivity.

Ahh, I see. That makes sense.

Are you already using the --depth parameter to reduce the .git sub-folder size when git cloneing prior to zipping?

@nuket Hi, could you help share if any updates for this issue? Can this issue be closed? Thanks.

No updates from my side, this issue can be closed.


Function for submitting the form programatically?

Hi, I want to submit the form in an event listener, but I cannot find any function for that in the api doc.
https://api.felte.dev/
I have then tried out the traditional form.submit() (and reference the <form> element with bind:this) in Javascript, but seems like Felte is not catching the submit there.
const handleKeydown = (e: KeyboardEvent) => {
    if(e.code === "Enter"){
        formElem.submit()
    }
}

Did I missed something in the documentation? Or is this function not there yet? I would love to help with the doc or the code.

Hello! https://api.felte.dev right now is more aimed at people trying to make sense of all the helper libraries in order to expand them rather than for people just trying to use Felte.
For your specific use case, you may use the createSubmitHandler helper function!
Regarding form.submit(), traditionally calling that does not trigger a submit event, neither does it call onsubmit handlers, that's why Felte's not able to catch it. Overriding the method so it uses Felte could be possible, but since that use case is covered by createSubmitHandler I didn't think it'd be worth it.

As an addendum to this, MDN docs seem to indicate the proper way to do this (by calling a method from the form element) is by using HTMLFormElement.requestSubmit. I see it actually does not work on Safari. So I've just published a patch on<EMAIL_ADDRESS>that just calls our submit handler if the browser does not support it or no polyfill has been added.
So calling form.requestSubmit() should work for your use-case, and starting on 0.8.3 it also works in Safari.

I'm closing this, since there was no further concerns and there's now two ways to do this: requestSubmit and createSubmitHandler.
Thanks for using Felte!

@pablo-abc Thank you for the help again, I have done what I want this morning with createSubmitHandler, which is really easy. Cheer!


fix: quality-gate issues

fix code-smells reported by SonarCloud

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

194 of 202 (96.04%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.003%) to 96.04%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/TypealizR/Core/GeneratorOptions.cs
47
49
95.92%

src/TypealizR/Core/ResxFileSourceGeneratorBase.cs
69
75
92.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.003%

Covered Lines:
1475

Relevant Lines:
1512

💛  - Coveralls


Consumer groupId issue with multiple instance running

Hi,
I am facing the issue in consuming the event from topic.  There are 3 partitions for topic and also there are three consumers in the same group Id that are consuming this topic. I tried to push 1000 message at the same time to the topic but only one consumer handles messages. I really want that each consumer in group will process a part of all message to reduce  whole processing time.

I'm using _comsumer.Assign to set Consumer listen from specific partition.

it sounds like you are writing all the messages to the same partition?

@mhowlett  this is another consumer

@mhowlett I'm not sure but let me check producing part.

i have a same question

recommend inspecting the topic using some independent means, perhaps the kafka-consumer-group tool, confluent control center, or just looking at the relative size of the partitions on disk in the kafka data directory. this is not likely a problem with the consumer.


[LiveComponent] deferred/lazy loading

From https://github.com/symfony/ux/issues/102

Future G) Potentially add support for "lazy" components (they don't load until they are visible) or at least document how one could use a lazy Turbo Frame nicely with a component URL. For example, on page load, a component has a "loading" animation. Then, it loads via Ajax and the area is updated.

Livewire's docs for this feature

A bit of a DX brainstorm:

Should the component class know it's lazy? (livewire does - example stores the loaded state)
Should the component template know it's lazy? (livewire does - example with wire:init)
Should the calling component function mark it as lazy? (livewire does not)

I'm sort of leaning towards a combination of 2 and 3 (opposed to livewire's 1 and 2):
<twig:MyComponent data-defer="true" />

{# MyComponent.html.twig #}

<div {{ attributes }}>
    {% if isInitializing %} {# new attribute provided by live components #}
        Loading... (spinner or maybe a cool animated placeholder skeleton)
    {% else %}
        ... expensive operation ...
    {% endif %}
</div>

On the first render, isInitializing would be true. The js would know to, as soon as the component is initialized on the page, immediately re-render. During this render (and subsequent renders), isInitializing would be false.
At least one problem with this idea: forcing the component to be lazy in the component template (<div {{ attributes }} data-defer="true"> wouldn't work. This would be too late to know to set isInitializing.
I'm very open to other ideas.
// cc @WebMamba, @sneakyvv any interest in this feature? I'd love some thoughts.

One thing I like about the Livewire way of doing things is that it doesn’t add any new concepts: you create an action, then the only magic is that you ask the action to be automatically executed.
With that in mind, what about something like:
<div {{ attributes.defaults({
    'data-action': 'appear->live#action',
    'data-action-name': 'loadPosts',
}) }}>
<div>

Or, you should be able to equally do this from the outside:
<twig:MyComponent data-action="appear->live#action" data-action-name="loadPosts" />

This takes the normal "action-calling" syntax + Stimulus's normal data-action syntax (which is data-action="EVENT->CONTROLLER#ACTION"). The new addition would be an appear event (name could be changed) that we trigger internally when the element "comes into the viewport. This is modeled off of https://github.com/stimulus-use/stimulus-use/blob/main/docs/use-intersection.md
If you want to simply "load immediately on page load", then instead of appear, I think that this would already work, though the name is a bit cumbersome (the live:connect - we could shorten to connect or load).
<twig:MyComponent data-action="live:connect->live#action" data-action-name="loadPosts" />

Hum I am not really agree here. Since the laziness of a component is a template concern, only the template should know that the component is lazy.
My proposal is as follow:
<twig:MyComponent data-lazy />

You only have to set this data-lazy attribute, if the data-lazy attribute is set the mount method will be called by ajax when the compose is visible, if not the mount method is call as before.

@WebMamba I think this is valid, but a separate level of laziness. Here are the 2 cases:
A) (existing idea): the component renders, but it has a flag to avoid doing something heavy. That flag changes on load, and then you do the heavy stuff. An advantage of this is that you can easily control the "loading" state - e.g. "Loading...".
 I like this, but it could be tricky in practice. You would need a way to control the loading state (e.g. "Loading...") and you would need to be able to construct a URL that would render the component, which would come from only the "input props" (as we'd want to avoid actually mounting the component). There is a component_url() Twig function, but that mounts the component then dehydrates it. In this case, you would need to create a URL based off of the "input props", which are not something that we normally need to try to dehydrate. My guess is that we could only reasonably make this work if we required all "input props" to be scalars (e.g. so we could just pop them onto a URL as query params).

I like all the ideas here, but I'm wondering if this is taking the (not yet accepted/merged) live embedded components into account. (#913).
I guess it would work well together since these can also be loaded lazily, and wouldn't be impacted by any of this, right?🤔
Would a component that's being replaced in the DOM again be treated the same and potentially be loaded lazily? Even if it's being replaced within the current viewport?

Btw, Livewire 3's lazy feature is like @WebMamba suggested :)

My proposal is as follow:
<twig:MyComponent data-lazy />

You only have to set this data-lazy attribute, if the data-lazy attribute is set the mount method will be called by ajax when the compose is visible, if not the mount method is call as before.

The key thing is that: the component is NOT mounted initially. I as mentioned, this means that you actually need to dehydrate the input props, which is not something that's done anywhere else. But, Livewire does this. Likely, we would need to require the input props to be scalars so that we can just serialize them easily. I think that Livewire does NOT require this, but that means they leak code info (e.g. prop foo is an App\Entity\Foo object) to the frontend, which I think they are generally ok with, but we have avoided thus far.
@sneakyvv

guess it would work well together since these can also be loaded lazily, and wouldn't be impacted by any of this, right?🤔

My guess is that this won't be impacted by your work over there.

Would a component that's being replaced in the DOM again be treated the same and potentially be loaded lazily? Even if it's being replaced within the current viewport?

Not sure what you mean here.

This has been fixed for a while.


Debian repository returning 500 error, prevents install

The gemfury repositories for this project seem to be down
Err:11 https://davidbailey00.fury.land/apt  InRelease
  500  Internal Server Error [IP: <IP_ADDRESS> 443]

I'm getting a 500 too :cry:

Have tried several times and it partially worked then failed while doing repo update
Ign:11 https://davidbailey00.fury.land/apt  InRelease
Ign:15 https://davidbailey00.fury.land/apt  Release
Ign:16 https://davidbailey00.fury.land/apt  Packages
Err:16 https://davidbailey00.fury.land/apt  Packages
  404  Not Found [IP: <IP_ADDRESS> 443]

On another try I got a 503
Err:12 https://davidbailey00.fury.land/apt  InRelease
  503  Service Unavailable [IP: <IP_ADDRESS> 443]

Looks like the new hosting is not very stable?

Sorry, this is outside of my control :( Check Gemfury's status page: https://status.fury.co/ (doesn't say anything at time of commenting)

I'm going to close this assuming it's now working - please let me know if there are still problems


Add callback function for angular controller

The standard method of loading content using $http in Angular has the option to specify a success callback when the content has loaded, so that in the controller, you can initiate scope bindings dependent on data.
The angular-sails-bind directive has no such thing. I implemented it myself in https://github.com/arryon/angular-sails-bind/commit/0ec49070f32bce1c384d023d5f5781b1fdc76528 but probably there is a better way to do this, like the default .success() and .error() paradigm. Any ideas?

I have solved this by adding a $watch to the object and passing a function to the watch.

I've merged your changes from your fork to do it, thanks!


Add GitHub Action that adds context-v2 label everywhere

I'm putting the context-v2 label on every new issue, discussion and PR manually to mark it as having been created in the context of tailwind-merge v2. Would be nice to build a GitHub action that does this automatically whenever a new issue, discussion or PR is created in this repository.
Here are the docs for GitHub Actions: https://docs.github.com/en/actions
And here is the workflow syntax explained: https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions
If anyone wants to pick it up, I'm accepting PRs for this and also happy to help if needed. 😊

Hi @dcastil,
I have created the context-v2-label  branch and added the github action file to that branch. Now how can I test this action?
Can you help me


[Bug] screentogif 2.38.1 blocks ffmpeg 6.0 update to in chocolatey

Describe the bug
I have ffmpeg installed in chocolatey, but can't update it from 5.1.2 to 6.0 due to screentogif:
You have ffmpeg v5.1.2 installed. Version 6.0 is available based on your source(s).
[NuGet] One or more unresolved package dependency constraints detected in the Chocolatey lib folder. All dependency constraints must be resolved to add or update packages. If these packages are being updated this message may be ignored, if not the following error(s) may be blocking the current package operation: 'screentogif 2.38.1 constraint: ffmpeg (>= 4.4.1 && < 6.0.0)', 'screentogif.portable 2.38.1 constraint: ffmpeg (>= 4.4.1 && < 6.0.0)'
Re-attempting package dependency resolution using additional available package information...
[NuGet] One or more unresolved package dependency constraints detected in the Chocolatey lib folder. All dependency constraints must be resolved to add or update packages. If these packages are being updated this message may be ignored, if not the following error(s) may be blocking the current package operation: 'screentogif 2.38.1 constraint: ffmpeg (>= 4.4.1 && < 6.0.0)', 'screentogif.portable 2.38.1 constraint: ffmpeg (>= 4.4.1 && < 6.0.0)'
Progress: Downloading ffmpeg 6.0... 100%

ffmpeg v6.0.0 [Approved]
ffmpeg package files upgrade completed. Performing other installation steps.
Extracting 64-bit C:\ProgramData\chocolatey\lib\ffmpeg\tools\ffmpeg-release-essentials.7z to C:\ProgramData\chocolatey\lib\ffmpeg\tools...
C:\ProgramData\chocolatey\lib\ffmpeg\tools
Removing extracted archive.
Sleeping for 2 seconds to allow anti-viruses to finish scanning...
Renaming ffmpeg directory to common name (Try 1 / 3)
Successfully renamed directory.
 ShimGen has successfully created a shim for ffmpeg.exe
 ShimGen has successfully created a shim for ffplay.exe
 ShimGen has successfully created a shim for ffprobe.exe
 The upgrade of ffmpeg was successful.

It looks like tried to install ffmpeg 6.0 but doesn't actually do it.

Yes, ScreenToGif doesn't support v6 of FFmpeg yet.
I'm updating the app now to add support for it.

Why ffmpeg 6.0.0 and not 6.x?

Constrain removed from the package.

Thanks!


Bug fix for CertificateInformation and add additional properties to ApiContract

Description
https://github.com/Azure/azure-rest-api-specs/pull/3614
https://github.com/Azure/azure-rest-api-specs/pull/3609
https://github.com/Azure/azure-rest-api-specs/pull/3430

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

[x] Please add REST spec PR link to the SDK PR
[x] I have read the contribution guidelines.
[x] The pull request does not introduce breaking changes.

General Guidelines

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

SDK Generation Guidelines

[x] If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
[x] The generate.cmd file for the SDK has been updated with the version of AutoRest, as well as the commitid of your swagger spec or link to the swagger spec, used to generate the code.
[x] The *.csproj and AssemblyInfo.cs files have been updated with the new version of the SDK.

@shahabhijeet  looks to be some issue with infra

@dsgouda updated the SDK generation to be agnostic of version.

@shahabhijeet @dsgouda a gentle ping on this!

Apologize for the delay


update Constraints in the example not work

Can you try updating the pod to latest version (1.1.0)?, if not.
Or tweak the priority like this:
[make.width.equalTo(@(self.buttonSize.width)) priorityLow];         [make.height.equalTo(@(self.buttonSize.height)) priorityLow];         [make.width.lessThanOrEqualTo(self) priorityHigh];         [make.height.lessThanOrEqualTo(self) priorityHigh];
it generates some warnings, but only at the moment the button width reaches the superView width.


Would it be interesting to support zstd?

I saw a fast algorithm with good compression:
https://github.com/facebook/zstd
I don't know if it would bring improvements in relation to the one used by zpaq, nor how complicated it would be to include support for it, but I leave it here as a suggestion for analysis. :)

Thank you
Embedding new compressors is not an easy task, because zpaq use its own virtual machine (yes!) to describe the algorithms used
I am currently developing an advanced chunked-extraction mechanism to allow 100% verification with free space less than the overall size of the archive. The first objective is safety, then efficiency

Thanks for the feedback.

The first objective is safety, then efficiency

I'm happy with this choice :)


error on zat server

I am trying to follow this article to get started with Zendesh apps tools :
$ zat new
then
$ zat server
and I have the following error on my browser http://localhost:4567/
Sinatra doesn’t know this ditty.

Try this:
# in Users/user/.rbenv/versions/2.5.0/lib/ruby/gems/2.5.0/gems/zendesk_apps_tools-2.8.3/lib/zendesk_apps_tools/server.rb
class ZendeskAppsTools::Server
  get '/' do
    "Hello World"
  end
end

I am supposed to get the content of the iFrame "Hello, World!"
I tried to modify server.rb as suggested, but then the browser display my plan text, instead of serving the iFrame.
I have the latest version of Ruby and ZendeskAppsTools
ruby 2.5.0p0 (2017-12-25 revision 61468) [x86_64-darwin17]
gem update zendesk_apps_tools
Updating installed gems
Nothing to update

I am following the same steps but getting an encoding error and don't know how to force the server to run in UTF 8. (ruby 2.3 32 bit on win10)

You're not supposed to go to localhost:4567. You're supposed to go to https://subdomain.zendesk.com/agent/?zat=true (but replace "subdomain" with your subdomain). See Testing your app locally in a browser for more.

@H-x-c have you tried setting the RUBYOPT as suggested in the error message?

@chuck-wood I am still getting the error, however are still able to carry on with what I am doing. I am setting it in the iframe instead. Not on the command line.

@H-x-c You should probably open a separate issue for this; or you might have better luck posting on the Zendesk app community. They are pretty responsive.

I had the same issue. It seems ZAT now uses app_scaffold tool and the documentation is not updated here
So, after creating ZAT app, just follow instructions:
Setup

Clone or fork this repo
Change (cd) into the app_scaffold directory
Run yarn install

You can use either yarn or npm as package manager and run the scripts with the corresponding commands.
...
Running locally
To serve the app to your Zendesk instance with ?zat=true, run
yarn run watch
zat server -p dist

The problem is that Zendesk has not updated to secure protocol in the code. If you open console in Chrome, you will see that the iframe.html is not displayed due to mixed content:

@lawhi The workaround for this is outlined in one of the HC articles .
Allowing unsafe scripts in Chrome will let you bypass the mixed content issue.


Hide init task from ./gradlew tasks output when build file exists

Expected Behavior
./gradlew tasks should not display init task when build file already exists.
What is the use of running ./gradlew init in a project that already has a build file?
That task should be hidden automatically.
Current Behavior
./gradlew tasks always prints init - Initializes a new Gradle build even though the task would be skipped like
$ ./gradlew init

> Task :init SKIPPED
The build file 'build.gradle.kts' already exists. Skipping build initialization.

Context
https://github.com/gradle/gradle/issues/10133

Hi @vlsi
The recent activity is mostly us making sure that all issues closed by the stale bot are properly marked as "not planned" instead of looking like regular completed issues.
This is an area that the team cannot currently focus on. So its state reflects this now.


ImportError: cannot import name 'image_converter' from 'carla' - Carla 0.9.13, Windows 10, Human_Control code - G920 steering wheel

Hi everyone!
CARLA version: 0.9.13
Platform/OS: Windows 10
Problem you have experienced: When I try to execute the human_control.py it throws me the error: "ImportError: cannot import name 'image_converter' from 'carla'".
What you expected to happen: To successfully setup G920 steering wheel with manual controls.
Steps to reproduce:

Imported and saved the file human_control.py in the path PythonAPI/examples from https://github.com/rohitgajawada/carla/blob/intern/PythonClient/human_control.py
Ran the human_control.py from the command prompt at example folder.
Simultaneously launched the CarlaUE4Editor and was in the play mode.

Other information (documentation you consulted, workarounds you tried):

Tried to install the .egg in the PythonAPI/carla/dist folder using easy_install, pip and pip3 it results in error "'easy_install' is not recognized as an internal or external command operable program or batch file."
With pip & pip3 it throws error "ERROR: Invalid requirement: 'C:\Users\hcelab\carla\PythonAPI\carla\dist\carla-0.9.13-py3.8-win-amd64.egg' Hint: It looks like a path. The path does exist."
My setuptools are all updated and the requirement is satisfied too.
I can even see easy_install.py under the setuptools/commands folder.

The full error w.r.t Carla is shown below:

Could someone please help me or at least point me in the right direction. Any help is much appreciated.
Thanks in advance.

Hello,
I am also facing the same issue. Did you manage to fix it?

where can i find carla\PythonAPI\carla\dist\carla-0.9.13-py3.8-win-amd64.egg ????


POP3 fixes

Hi,
This PR fixes POP3 connection (READONLY not handled) and empty mailbox.
About the weird additional check, with pop3, imap_open will trigger an error logged in imap_errors and displayed at the imap_close (or auto close as the end of the page) but I don't want to ignore the errors if it's not really linked with the empty mailbox, thus my additional check.

It fixes https://github.com/Webklex/laravel-imap/issues/402

Hi @Korko ,
many thanks for your pull request!
Best regards,


Make work in editing mode

Currently this only works in reading mode. It'd be nice if it could also work in editing mode (akin to the built-in auto-linking of URLs).
I think it's possible, by making a view plugin and adding decorations, but am not 100% sure.

This would be a game-changer for this plugin. I work most of the time in live preview. (I assumed you meant be "editing mode" the live preview and not the Source mode)

Agh! Thats why it looks like it didnt work! I'm never ever in 'Reading' mode as live in editing with the live preview thing (so it edits when I cursor over the feature in the note).
A big +1 for me on this one.. my personal workflow / usage of Obsidian makes this AWESOME plugin redundant =(
If I ever get round to learning how to write plugins for Obsidian I'll chip in, but I know nada at the moment I'm afraid.

@danstreeter How do you make this work in reading mode? Do you type in the phrase to replace then switch to it? I tried that and it doesn't work.
@kemayo Clearly this plugin isn't well documented, and its very unprofessional for the readme to say "I'm so very sorry." for no good reason.

@kemyao, I shall try to compensate the bad vibes send from @YousufSSyed.
Thank you for publishing your work, hoping it will be helpful for others. I (and I assume a lot of other people) appreciate it very much ! And no one with a little bit of empathy would expect that you would provide professional code, readme and support for free.
Thank you again very much !
:heart:

Thanks for all the hard work guys. I came here while searching for a similar feature of GitHub's smart links.
I just started with Obsidian, so live preview is native to my workflow and I thought this wasn't working 😅


temporary stringify fix

Temporary fix for #807
The bug happens when iterating over _recordValues and find a reference to another MegamorphicModel...
record.debugJSON() returns the recordData's, _data, the last known data from the server inside recordData._data... so it returns the reference and this method is modifying the reference, which leads to this bug... this is always happens, and leads to other kind of more "serious" bugs like changedAttributes stuff, maybe this also helps #769 ?

also since m3 lazily access/calculates stuff via unknownProperty , in the scenario of a corrupted/stringified recordData._data, accessing the props returns the stringified value.

The easy solution for now is just cloning, what do you think @SYU15 @igorT ?

@betocantu93 @SYU15 thanks so much for digging in to this issue!  This looks good I'll be very happy to merge once @SYU15's comments are addressed.
Thanks again


Slow mysql query encountered when browsing the Network Tab

When using a broader IP Range (e.g multiple /16), a particular query is getting slow:
SELECT COUNT(user_ip_address.public_ip_address) FROM user_ip_address  INNER JOIN vlan ON user_ip_address.vlan_db_id=vlan.id WHERE user_ip_address.data_center_id  = 1 AND  (vlan.vlan_type = 'DirectAttached' );

ISSUE TYPE

Improvement Request

COMPONENT NAME
API, Mysql Backend
-->
CLOUDSTACK VERSION
<IP_ADDRESS>
CONFIGURATION
Advanced Networking, VLAN isolation
OS / ENVIRONMENT
Ubuntu 22.04 LTS
SUMMARY
When using a broader IP Range (e.g multiple /16), a particular query is getting slow:
SELECT COUNT(user_ip_address.public_ip_address) FROM user_ip_address  INNER JOIN vlan ON user_ip_address.vlan_db_id=vlan.id WHERE vlan.vlan_type = 'DirectAttached' AND user_ip_address.data_center_id  = 1  AND user_ip_address.state != 'Free'  AND user_ip_address.removed IS NULL;

This could be resolved by adding a (missing) key:
ALTER TABLE `user_ip_address` ADD KEY `fk_user_ip_address__state` (`state`);

STEPS TO REPRODUCE
Having a few /16 IPv4 Ranges assigned,
Enable mysql slowlog with a timeout of e.g. 2 seconds, and watch it happen.
EXPECTED RESULTS
Having the result faster.
ACTUAL RESULTS
Waiting for the result for about 2 seconds. (At least for my setup)

Is this critical/blocker issue @DaanHoogland ?

@rohityadavcloud , for anybody using /16 nets it is a blocker and I don´t know a workaround. I would say it is critical. Any reason it would not be?

@s-seitz I am assuming you tested this to resolve your issue?
If not please try with PR #7028


Missing branches detail missing from JSON format

Describe the bug
Missing branches detail missing from JSON format
To Reproduce
How can we reproduce the problem?  Please be specific. Don't link to a failing CI job. Answer the questions below:

What version of Python are you using?

$ python3 --version
Python 3.9.13

What version of coverage.py shows the problem? The output of coverage debug sys is helpful.

$ python3 -m coverage debug sys
-- sys -------------------------------------------------------
               coverage_version: 6.4.2
                coverage_module: /opt/homebrew/lib/python3.9/site-packages/coverage/__init__.py
                         tracer: -none-
                        CTracer: available
           plugins.file_tracers: -none-
            plugins.configurers: -none-
      plugins.context_switchers: -none-
              configs_attempted: .coveragerc
                                 setup.cfg
                                 tox.ini
                                 pyproject.toml
                   configs_read: -none-
                    config_file: None
                config_contents: -none-
                      data_file: -none-
                         python: 3.9.13 (main, May 24 2022, 21:13:51) [Clang 13.1.6 (clang-13<IP_ADDRESS>)]
                       platform: macOS-12.5-arm64-arm-64bit
                 implementation: CPython
                     executable<EMAIL_ADDRESS>                   def_encoding: utf-8
                    fs_encoding: utf-8
                            pid: 54555
                            cwd: /Users/juan/project/slipcover
                           path: /Users/juan/project/slipcover
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>                                 /opt/homebrew/lib/python3.9/site-packages
                                 /Users/juan/project/slipcover2
                                 /Users/juan/tmp/flask/src
                    environment: HOME = /Users/juan
                   command_line: /opt/homebrew/lib/python3.9/site-packages/coverage/__main__.py debug sys
                sqlite3_version: 2.6.0
         sqlite3_sqlite_version: 3.39.2
             sqlite3_temp_store: 0
        sqlite3_compile_options: ATOMIC_INTRINSICS=1, COMPILER=clang-13.1.6, DEFAULT_AUTOVACUUM,
                                 DEFAULT_CACHE_SIZE=-2000, DEFAULT_FILE_FORMAT=4,
                                 DEFAULT_JOURNAL_SIZE_LIMIT=-1, DEFAULT_MMAP_SIZE=0, DEFAULT_PAGE_SIZE=4096,
                                 DEFAULT_PCACHE_INITSZ=20, DEFAULT_RECURSIVE_TRIGGERS,
                                 DEFAULT_SECTOR_SIZE=4096, DEFAULT_SYNCHRONOUS=2,
                                 DEFAULT_WAL_AUTOCHECKPOINT=1000, DEFAULT_WAL_SYNCHRONOUS=2,
                                 DEFAULT_WORKER_THREADS=0, ENABLE_COLUMN_METADATA, ENABLE_FTS3,
                                 ENABLE_FTS3_PARENTHESIS, ENABLE_FTS4, ENABLE_FTS5, ENABLE_GEOPOLY,
                                 ENABLE_MATH_FUNCTIONS, ENABLE_PREUPDATE_HOOK, ENABLE_RTREE, ENABLE_SESSION,
                                 MALLOC_SOFT_LIMIT=1024, MAX_ATTACHED=10, MAX_COLUMN=2000,
                                 MAX_COMPOUND_SELECT=500, MAX_DEFAULT_PAGE_SIZE=8192, MAX_EXPR_DEPTH=1000,
                                 MAX_FUNCTION_ARG=127, MAX_LENGTH=1000000000, MAX_LIKE_PATTERN_LENGTH=50000,
                                 MAX_MMAP_SIZE=0x7fff0000, MAX_PAGE_COUNT=1073741823, MAX_PAGE_SIZE=65536,
                                 MAX_SQL_LENGTH=1000000000, MAX_TRIGGER_DEPTH=1000,
                                 MAX_VARIABLE_NUMBER=250000, MAX_VDBE_OP=250000000, MAX_WORKER_THREADS=8,
                                 MUTEX_PTHREADS, SYSTEM_MALLOC, TEMP_STORE=1, THREADSAFE=1

What versions of what packages do you have installed? The output of pip freeze is helpful.

$ python3 -m pip freeze
attrs==21.4.0
click==8.1.3
coverage==6.4.2
cycler==0.11.0
-e git+ssh://git@github.com/pallets/flask.git@dba2be9311d2abfd1b973c6bb578f416ee771fda#egg=Flask
fonttools==4.34.4
importlib-metadata==4.12.0
iniconfig==1.1.1
itsdangerous==2.1.2
Jinja2==3.1.2
kiwisolver==1.4.4
MarkupSafe==2.1.1
matplotlib==3.5.2
numpy==1.23.1
packaging==21.3
Pillow==9.2.0
pluggy==1.0.0
py==1.11.0
py-cpuinfo==8.0.0
pyparsing==3.0.9
pyperf==2.4.0
pytest==7.1.2
pytest-repeat==0.9.1
python-dateutil==2.8.2
six==1.16.0
-e git+ssh://git@github.com/jaltmayerpizzorno/slipcover.git@1164e0d5449eae9e1b1ed4eda663475144fb1cb8#egg=slipcover
tabulate==0.8.10
tomli==2.0.1
typer==0.6.1
Werkzeug==2.2.1
zipp==3.8.1

What code shows the problem?  Give us a specific commit of a specific repo that we can check out. If you've already worked around the problem, please provide a commit before that fix.

x = 0
if x >= 0:
    x += 1
x += 2

What commands did you run?

$ python3 -m coverage run --branch b.py; \
   python3 -m coverage report -m; \
   python3 -m coverage json; \
   python3 -m json.tool coverage.json
Name    Stmts   Miss Branch BrPart  Cover   Missing
---------------------------------------------------
b.py        4      0      2      1    83%   2->4
---------------------------------------------------
TOTAL       4      0      2      1    83%
Wrote JSON report to coverage.json
{
    "meta": {
        "version": "6.4.2",
        "timestamp": "2022-07-28T13:40:37.638100",
        "branch_coverage": true,
        "show_contexts": false
    },
    "files": {
        "b.py": {
            "executed_lines": [
                1,
                2,
                3,
                4
            ],
            "summary": {
                "covered_lines": 4,
                "num_statements": 4,
                "percent_covered": 83.33333333333333,
                "percent_covered_display": "83",
                "missing_lines": 0,
                "excluded_lines": 0,
                "num_branches": 2,
                "num_partial_branches": 1,
                "covered_branches": 1,
                "missing_branches": 1
            },
            "missing_lines": [],
            "excluded_lines": []
        }
    },
    "totals": {
        "covered_lines": 4,
        "num_statements": 4,
        "percent_covered": 83.33333333333333,
        "percent_covered_display": "83",
        "missing_lines": 0,
        "excluded_lines": 0,
        "num_branches": 2,
        "num_partial_branches": 1,
        "covered_branches": 1,
        "missing_branches": 1
    }
}

Expected behavior
I'd expect the missing 2->4 branch to be shown somewhere in the JSON.

Completed as of 95195b13.

This is now released as part of coverage 6.5.0.


Excluding opera mini from browsers list

We get a lot of browser feature not supported warnings from opera mini. I have a hunch opera mini is included in the > 5%

I was doing some testing with where this config should live and discovered that it's not picked up by the stylelint plugin unless it's in the stylelint config. I know it should be, but there's some bug in the plugin or unfinished feature. So I'm changing this PR to move the browserstlist to the stylelint config.
lib/mixins/layout.scss
 25:3  ⚠  Unexpected browser feature "css-media-resolution" is only partially supported by IE 10,11, Safari 10,10.1, iOS Safari 10.0-10.2,10.3, Opera Mini all, Blackberry Browser 7,10, IE Mobile 10,11   plugin/no-unsupported-browser-features


CLion on Ubuntu Linux?

I see part of this script is dependant on XCode (Mac), Is there any way to use this with CLion on Ubuntu Linux?

You will have to modify it accordingly.